Add multi-process UDF progress

Keeping track of multiprocess progress while doc parsing, etc. is a pretty crucial ingredient for our application. This updates the progress bar to tqdm and changes a bunch of stuff in UDF to enable multi-process progress tracking. It is a somewhat imperfect solution, but perhaps not less imperfect than what was there before.
The QUEUE_TIMEOUT thing still bothers me and a much better approach may be to switch to the concurrent futures paradigm https://docs.python.org/3/library/concurrent.futures.html. Let me know what you think and feel free to edit as you see fit. It's a pretty heavy change as far as I understand, so I may be breaking stuff.

Don't think this is this our error? ```The job exceeded the maximum time limit for jobs, and has been terminated.

These last two commits by @debnil are not really associated with this PR, but unfortunately I made the PR from our fork's master, and we added them there, so if you don't like the changes, feel free to undo. Those changes to candidates are needed for us to do multiple, independent feature extraction on the same DB. Recreating the DB for each candidate type would not be possible for us given the size.

Continuing this conversation, I reverted the commit and am moving us onto a separate branch again. Let's discuss if/how we want to merge other stuff in (including the subclasses in our repo). Thanks!

This failure is The job exceeded the maximum time limit for jobs, and has been terminated. again… 02ed0b4 is a two-character comment change…


Fix reCaptcha being passed by bots

What this PR does / why we need it:
reCaptcha was not implemented like the reCaptcha guidelines and I think that caused the issue #3
Also the message regarding lack of reCaptcha on the page was not being removed if reCaptcha did not appear immediately after render due to internet speed. This PR fixes that as well.

Special notes for your reviewer:
Release note:

This change seems to break:

css to hide the recaptcha notice
disabled button so clicking is prevented until data is checked via js
data checking

The js change seems good so.

I'm working on the problems but I don't understand "css to hide the recaptcha notice"

Recaptcha can be inlined and the inlined recaptcha icon can be hidden. We did hide it before, but the other option is to not hide it, but design it better into the website.

If everything else is okay I can get to styling it and add an option for hiding it perhaps.

Recaptcha and form looks good now.
Still needs styling and conflict resolution. The errors not showing is already open as issue.

Shall I fix the error messages in the same PR? Also how should I style the recaptcha?


Update ibm-storage-ceph.yaml

Manually update the super manifest for ibm-ceph.
ibm-ceph had publish:exclude for CASE, but due to gap in CSDP, even supermanifest was not published.
A couple of months back, we had done similar exercise for webmethods CASE entries.

yamllint is clean.
tested on : https://www.yamllint.com

We have got the changes for ibm-ceph by looking at the diff of the commit for internal promotion : https://github.ibm.com/IBMPrivateCloud/cloud-pak/commit/69bbd207e182e0691bd4e3ad37dda443439fb243#diff-963d293d2e79d6594f7aee5925823faab8c94c144f2928bb2535af2c9db7159c


Update ODB status check for plugin

Now that we have the ability to get data back from authors, need to update the ODB status check so that people using the plugin to go to pages we have received data for will be shown that we have the data. Then will need to check with @Steven-Eardley that the plugin knows how to display the response usefully (should do).

This is read once new extension can be deployed, closing this one, will be triggered again by the extension dev if necessary, but should just be testing at that point.


Force terminate UVM if any container ignores SIGKILL

If any hypervisor container running in a UVM ignores the SIGKILL for longer
than 30 seconds forcibly terminate the UVM. This is a mitigation against a
misbehaving guest that could potentially cause stop failures if it doesn't
respond with the proper exit.
Signed-off-by: Justin Terry (VM<EMAIL_ADDRESS>

@katiewasnothere - PTAL


Initial install of Threadfin in a Docker on Synology

Please first read the documentation. Problems affecting the client, e.g. Plex or Emby should be reported there in the forum.
Describe the bug
A clear and concise description of what the bug is.
Server (please complete the following information):

OS [e.g. Linux - Ubuntu]
Arch [e.g. Intel 64 Bit]
Threadfin Version and Build [to be found in the Threadfin log]

To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: [e.g. iOS]
Browser [e.g. chrome, safari]
Version [e.g. 22]

Additional context
Add any other context about the problem here.

Server (please complete the following information):

Synology NAS -
Synology Container manager

Threadfin Version and Build [to be found in the Threadfin log]
Version:                1.0 Build: 24
To Reproduce
Steps to reproduce the behavior:

install Image, Run  - create Container assigning port 344001 to 34400,
map path /volume3/docker/teve/conf to /home/threadfin/conf
go to  http://<IP_ADDRESS>:34401/web/
New Playlist - M3U and enter
name as test desrciption blank  and M3U file as /home/threadfin/conf/data/xteve.M3U

get error as per log
Expected behavior - not sure - first time user
Screenshots

Desktop (please complete the following information):
OS: [e.g. iOS] Windows 11
Browser [e.g. chrome, safari] Chrome
Version [e.g. 22] Version 122.0.6261.95 (Official Build) (64-bit)

try change it to: /home/threadfin/data/xteve.m3u
remember you have to put that file in the data folder. if you don't have it there, that's why it can't find it

Try doing file:///home/threadfin/data/xteve.m3u for a local file


Graphical Web UI

Hello,
I created a web user interface for managing the rpxy instance. It's made with Laravel and Bootstrap. I've included most configuration variations I guess. As a future feature, I thought of adding phpACME support for managing ssl certificates if this won't be of the domain of rpxy itself.
Maybe you can check it out and let me know what you think. We can further test and add/fix missing logic. Once you think it's good to share, maybe you can add a link to your readme so people will get to know it more easily.
https://github.com/Gamerboy59/rpxy-webui

Hi @Gamerboy59 !
That's great. Yes, I will check it.
For ACME, I am actually planning to integrate the automated certificate management process with rpxy (maybe as optional feature in the context of Rust). But external manager sounds reasonable for users who needs certs in PEM-based files.

If you integrate ACME (I think specifically letsencrypt) into rpxy natively, maybe it would be sufficient to allow users to export these certificates as PEM-files. I think a different cert manager for more or less the same functionality would be just extra maintenance effort and will be barely used. If users want to use certificates they optained outside the acme process (e.g. bought somewhere), the webui could offer a function where they can import those.

It looks like PHP can't do the conversion natively as I suggested above. OpenSSL would be required to be installed and accessible by the webserver and can then be used by some php libraries.

@Gamerboy59 first of all, thank you for the rpxy-webui project! I would like to cordially add some questions and suggestions.
What is the need behind the requirement of nodejs, since it is originally made on PHP? If it is because of some front-end requirements, the front-end can be pre-bundled in development phase, eliminating the need of NodeJS in production.
Also, since rpxy-webui is mostly made in PHP, if we can remove the NodeJS requirement, then you can package your project into a single binary executable by using static-php-cli, making a lot easier to distribute, install and use it.
I hope my suggestions make sense to you :slightly_smiling_face:

Thanks for your comment, @paulocoghi !
Like you supposed, NodeJS is required because of the frontend. Once all features are tested more thoroughly, my plan is to not only pre-bundle nodejs artifacts but also vendor files and thus also make composer redundand for the user, too.
Regarding the database, I'll have a look into how to handle the constraints to make it compatible with sqlite.
Do you plan to host the rpxy-webui in a container or why do you prefer a binary? I initially designed it for use in basic shared hosting environments, where typically all the PHP modules required by Laravel are pre-installed, and a MySQL database is also generally available.
Waiting for feedback from @junkurihara too and then maybe a beta version, ready to deploy, can be released.

@Gamerboy59 Thanks for the feedback!

Do you plan to host the rpxy-webui in a container or why do you prefer a binary?

I normally install softwares like rpxy and rpxy-webui on the host, not on the containers/VMs. And this is why I choose the ones which are single binaries.

Regarding the database, I'll have a look into how to handle the constraints to make it compatible with sqlite.

If you are using Eloquent ORM, then you can switch databases by changing just a few lines on the configuration, and the rest should be transparent.

I normally install softwares like rpxy and rpxy-webui on the host, not on the containers/VMs. And this is why I choose the ones which are single binaries.

I'll check if GitHub Actions support the generation of such binaries. My first attempt generated a huge file which is inconvenient.

If you are using Eloquent ORM, then you can switch databases by changing just a few lines on the configuration, and the rest should be transparent.

Yes, Eloquent is used. You can already change the db_connection to SQLite in your .env if you want. I just remember there were some cascading functionalities not support by SQLite so I need to test if it works well before adding it to the description.

I'll check if GitHub Actions support the generation of such binaries. My first attempt generated a huge file which is inconvenient.

Have you tried to generate the binary based on the common combination? It provides 30 of the 50 most used php extensions, and the final binary (without your project's code) uses only 7.5MB.

Yes, Eloquent is used. You can already change the db_connection to SQLite in your .env if you want.

It would be interesting if SQLite would be already the default, making the final executable natively independent of any external database

It would be interesting if SQLite is already defined as the default database, internally and transparently, because it makes the final executable natively independent of any external database

This is already the default if you copy the .env.default. I'll make it more clear in the README.md that using mysql as database is optional but for this I have to test some queries. Eloquent supports all these database drivers but their feature set is not identical even though it's one interface.

Have you tried to generate the binary based on the common combination? It provides 30 of the 50 most used php extensions, and the final binary (without your project's code) uses only 7.5MB.

I just downloaded theirs and it seems to work. However, I wonder how this is connected to rpxy-webui? If I understand this static-php-project correctly, it doesn't really matter if you use their pre-compiled php-binaries or the ones of your OS distributions' package manager. Since it is just the php runtime, you still need the webserver and the webroot of the webserver is where you put the laravel webapp.

Since it is just the php runtime, you still need the webserver and the webroot of the webserver is where you put the laravel webapp

This is a really important point and maybe my suggestion about providing rpxy-webui as a single autonomous binary is unrealistic in this case.
There is one last approach which combines a lightweight php runtime (FrankenPHP) with an embedded web server (caddy) that is fully compatible with Laravel and is also based on static-php-cli, but I don't want to create a burden on you (I'm just an enthusiastic user of rpxy).
In the end, my main feedback is that server applications (like rpxy, rpxy-webui, dnsdist, bind, etc) are better designed when provided as a single binary. But the underlying language/framework plays a big role on this.
Like when using NodeJS (packaging with pkg) or Crystal Language (compiled), with or without a framework, achieve this is a lot easier. That's it :wink:
This doesn't mean there isn't an audience for solutions which depend on an interpreter. There is and it's fine. Independent of anything, I appreciate your effort @Gamerboy59 :slightly_smiling_face:

There is one last approach which combines a lightweight php runtime (FrankenPHP) with an embedded web server (caddy) that is fully compatible with Laravel and is also based on static-php-cli, but I don't want to create a burden on you (I'm just an enthusiastic user of rpxy).

I do really appreciate your efforts to find a solution. In my view, shipping a webserver and everything is just out of scope of this managing webapp instance. It would require me to constantly update the binaries, even if the code of rpxy-webui didn't change, just to update the dependend applications. The best I can think of right now is to build .deb and .rpm packages which define dependencies to a webserver and php for the OS's package manager but there's also many contraints (what is if there is already a manually compiled php version available because the user doesn't want the php from distribution?) and thus is actually out of scope for any webapp in my opinion, too.

In the end, my main feedback is that server applications (like rpxy, rpxy-webui, dnsdist, bind, etc) are better designed when provided as a single binary. But the underlying language/framework plays a big role on this.

It is more convenient, I agree, but this convenience comes at a high sysadmin management effort.
One last thing which comes to my mind to at least partially solve this, is if @junkurihara adds a webserver feature like a config for the webroot and a socket path for php-fpm so you only have to download rpxy-webui to this webroot and start the portable php-binary. It requires significant change to this project though, so I don't know if it is really a possible way to go.

Sorry for my delayed responses, and thanks for your fruitful discussion!
Honestly I've never write and use php. So I am not familiar with how to run the project in various environment with different dependencies. But I think that as @paulocoghi mentioned and the discussion is going on, shipping rpxy-webui as a single executable binary is better for users.  Also specifically in cases of users of container-based systems, a self-contained container image including rpxy and rpxy-webui sounds nice as well.

One last thing which comes to my mind to at least partially solve this, is if @junkurihara adds a webserver feature like a config for the webroot and a socket path for php-fpm so you only have to download rpxy-webui to this webroot and start the portable php-binary. It requires significant change to this project though, so I don't know if it is really a possible way to go.

Hmm it sounds really tough to newly integrate such function with rpxy. I also think having web servers inside rpxy violates the design principle: rpxy should be as simple and tiny (and fast :-)) as possible to be used with various infrastructure, e.g., raspberry-pi, (self-hosted) CDN edge servers, etc. So I think that although web management interfaces are really useful (and of course welcome for me!), it should be separeted from rpxy.

Also specifically in cases of users of container-based systems, a self-contained container image including rpxy and rpxy-webui sounds nice as well.

@junkurihara I've included a Dockerfile which builds a ready to use docker image including php and webserver. The resulting image is over 600MB on disk which is too big to distribute.
@paulocoghi I've added pre-compiled archives to github as well as removed mysql from the install instructions.
I've had a look again at the binary option but this is still not going to be a good option in my opinion. The webserver is around 20MB, the php binary another 15MB including all required modules and then some additional runtime dependencies of around 10MB which makes a 45MB binary. That's way too big for an application of about 1MB size. PHP is just not made to be distributed with a full stack. I should have created rpxy-webui in Go, Lua or something else more light if it should be a binary. Now it's nearly impossible to convert though.

I started building packages for the system's manager which one the one hand requires the installation of apache24 as only webserver choice currently but on the other hand pretty automatically serves the webapp without the burden to manage the full tech stack.
You can download it here: RedHat/Rocky/Alma-9 and Debian/Ubuntu
Please let me know your feedback, @paulocoghi.
These packages could theoretically also ship along with rpxy as a system service. Is that something you would consider @junkurihara ?

Do you prefer packages or docker containers? Adding the at time of compilation most current rpxy binary should be doable, too, so it'd one package or container for all.

Before I start adding the ACME stuff (display cert, import custom etc.), can somebody test the current version? Would appreciate any feedback. If there is anything making the deployment of it a hurdle to you, let me know so I can see what can be improved. Right now this webapp can be deployed as shared hosting, docker container and distribution package (will add information about this to the README.md soon).

Do you prefer packages or docker containers? Adding the at time of compilation most current rpxy binary should be doable, too, so it'd one package or container for all.

Just for my case, a docker container is preferable.
But rpxy-webui is being developed not for developers (i.e., me :-)) but for sysadmins. So I am not sure docker containers is a better than package for them honestly.

I added the link to rpxy-webui in README.md of develop branch 😃

I added the link to rpxy-webui in README.md of develop branch 😃

Thank you. :)
I've added a docker compose file to bring up both services simultaniously. It isn't working oob though because the Docker Image of rpxy-webui needs to be generated in advance and I think there are still permission issues for the rpxy.toml file on your image, when the config file is mounted through a volume.
I'll soon add the packages to a distribution place like launchpad to automatically push updates through it and likely thus make it a preffered option for sysadmins who don't run a container cluster.

@Gamerboy59 thanks a lot for you great contribution!
I recommend providing a way to install without depending on docker as most as possible, in the same way we don't need docker to install rpxy.
But it's fine to also provide a docker compose file

@paulocoghi thanks for your comment.
The .rpm and .deb packages work well without docker, they integrate as native system service, so they are right on your disk without another abstraction layer. I think, this will be my preferred choice of distribution from now on because you can very easily update with yum update or apt update and everything should be updated automatically.
I'll maintain the docker files additionally so people who already run containers can use it but it especially in kubernetes clusters it makes not such much sense to use it, also from technical aspects, because you'd have an ingress-controller, which is basically a reverse-proxy that forwards to rpxy, which is also a reverse-proxy. So there's two reverse-proxies chained which undermines the purpose of rpxy to be a fast and light app in my opinion.
Right now the distribution packages work in a way that /rpxy-webui is added as a global configuration which is accessible from.every place the webserver is configured for (default is all interfaces, also public). My idea for the next release is to add some form of a dialog-box to the installer where you can select to configure rpxy-webui as an additional, internal vHost, instead of a global configuration, and automatically add a upstream entry for rpxy to forward to this internal vHost. This makes rpxy listen for port 80 and port 443 on the public interface and if you access the default domain at http://<domain>/rpxy-webui, rpxy will forward this to the internal vHost (located at maybe http://localhost:8080).
I hope this makes a bit sense what I just wrote. ;) sorry if I expressed this a bit unclear

Hi :)
rpxy-webui now supports acme as well as importing custom certificates. It also converts the imported certificate to PKCS8 if it isn't in this format already.
Additionally, I consider adding the rpxy binary to the RPM and DEB distribution packages. Adding them as a system service simplifies the upgrades by a lot in my opinion. Of course, this risks that the distribution packages lack behind the github release but it's actually quite common to be the case with distribution packages so I don't see a big problem here. If the difference between rpxy and rpxy-webui is too big, it's likely not going to work anyways. Including the binary would require just one command (like apt-get install rpxy or dnf install rpxy) to install a working instance and service of rpxy including the webinterface. What are your thoughts about this?

Good news @Gamerboy59 !

Adding them as a system service simplifies the upgrades by a lot in my opinion.

Fully agree on this one!

Including the binary would require just one command (like apt-get install rpxy or dnf install rpxy) to install a working instance and service of rpxy including the webinterface. What are your thoughts about this?

I suggest to have distribution packages for both rpxy and rpxy-webui and, when asking to install rpxy-webui, you can set rpxy as a dependency of it to be automatically installed.

Thanks for your answer @paulocoghi. Good idea to seperate rpxy and rpxy-webui into two packages and relate them.
rpxy-webui will have rpxy as requirement while rpxy will only recommend rpxy-webui. Thus rpxy can also be installed without rpxy-webui but rpxy-webui not without rpxy.
I can create the necessary distribution packages for Debian/Ubuntu, RedHat/Rocky/Alma etc. and probably also a FreeBSD port and pkg. Is creating distribution packages of rpxy something you want to take care of @junkurihara or should I build them along with the webinterface?

Hi, sorry for my delayed reply.
Thanks @paulocoghi and @Gamerboy59 for your discussion!

rpxy-webui now supports acme as well as importing custom certificates. It also converts the imported certificate to PKCS8 if it isn't in this format already.

That's a  good news! Good job 👍

Additionally, I consider adding the rpxy binary to the RPM and DEB distribution packages.

Yes, I think it is a nice idea.

I suggest to have distribution packages for both rpxy and rpxy-webui and, when asking to install rpxy-webui, you can set rpxy as a dependency of it to be automatically installed.

I agree with this opinion :-)
I have never built RPM / DEB packages and used rpxy as a system services like systemd (I always use it as a container). So it would take time if I built and publish packages by myself. If you are okay, please go ahead to build and publish both rpxy and rpxy-webui packages.

If you are okay, please go ahead to build and publish both rpxy and rpxy-webui packages.
I'm setting up a pipeline to build rpxy packages for RPM and DEB based distributions.

As mentioned in #182, I've setup a page with instructions: https://rpxy.gamerboy59.dev/

with instructions: https://rpxy.gamerboy59.dev/

Awesome! That's seems really helpful for users.
BTW I think current configuration documentation, i.e., README.md and example-config.toml seems messy for new users. So, in addition to the @Gamerboy59 's site, maybe we also need to host another site for configuration?  Fortunately to test TLS with rpxy, I already owned rpxy.net, org, and io. So maybe with them, should we prepare such a site?

Hi @junkurihara Thanks for your comment. I'm glad you like the first version of my package-site.
It's good to have the domain names secure and on hand. I really think we you make the documentation more clear. The README.md has some good example but the example-config.toml is difficult to understand if you use rpxy for the first time. I propose we either use Github Pages to host the site or, since it comes with quite some limitations, I can host the docs as well. Do you think of a more inclusive documentation or more like a one-pager for each release and probably a changelog additonally?

Hi @Gamerboy59 , thanks for the comment 😄
Yes, GitHub Pages are promising options. I will try to make a website on another repo!

Do you think of a more inclusive documentation or more like a one-pager for each release and probably a changelog additonally?

I think the inclusive documentation seems better, it is tough to make it though.


Wierd visual artifacts at certian camera positions

When using a OrbitControl camera around the origin, I run into weird lines running through a mesh that look almost screen-door effecty. I am not super well versed in computer graphics but to me it looks like a floating point precision problem somewhere in the rendering... These artifacts seem to occur most when the camera is facing closer to directly down (as pictured in the video) or up at the surface...
https://user-images.githubusercontent.com/14093962/133928826-15d44cd9-9f91-4979-bdda-7b47315c1f40.mp4
Here is the source code: https://github.com/DusterTheFirst/impulse/blob/main/impulse-vis/src/main.rs
with the large green ground plane being setup here: https://github.com/DusterTheFirst/impulse/blob/main/impulse-vis/src/main.rs#L49
As a side note, render_normals in my code does not seem to render anything on the screen.
Thank you for making such a great 3d tool, It has been such a great use to me as most other rust based 3d tooling is either too low level, or a whole fully fledged game engine which does not fit my needs. If this issue ends up being rooted in the engine itself, I would be more than happy to be of assistance in creating a PR or investigating further.

Thanks a lot for the kind words 🙏 I'm glad you like it, and your description of three-d is spot on what I wanted it to be 👍
About the problem, you are right that it is due to floating point precision and this particular problem has a name; z-fighting. In your code you render the ground with lighting first and then you render the ground with the render_normals function which renders the object coloured based on the normals (a red color means a normal in the x-direction, green in the y-direction etc.). You might have thought that it renders small arrows of normals or something which is totally understandable, I should improve the documentation on this part. Anyway, to fix your problem, you need to render either the ground with lighting or render the ground with render_normals, not both at the same time.

OH! That makes a lot of sense. Thank you for the detailed description! I did in fact think that it would render the normal vectors as arrows or something. It makes a lot more sense that I am actually rendering the plane twice and I am familiar with Z-fighting. Thank you for your quick and detailed response!

No problem at all 🙂


Add Sphinx Plot Gallery

This PR addresses Issue #12.

@jberlanga – Thanks!
I see you have astroplan/plots/plots.py in this PR. Is that on purpose? Do you think code review for that will be quick (~ 1 day)? If no, I'd suggest you split the addition of the gallery (maybe with a dummy plot) from the PR where your plotting code is discussed.

https://travis-ci.org/astroplanners/astroplan/jobs/70927747#L870
ImportError: No module named sphinxgallery

You probably have to add a line after this one:
https://github.com/astroplanners/astroplan/blob/master/.travis.yml#L101
 - if [[ $SETUP_CMD == build_sphinx* ]]; then $PIP_INSTALL sphinxgallery; fi

@jberlanga – Is this working locally for you? Do you have a place where you can upload the generated html docs for easy review / discussion? (if no, maybe make a screenshot and paste it here?)

@cdeil

I did add plots.py here as a way to use the plotting function directly (since the plotting PR isn't merged yet), but I can delete it and replace the (non-functioning) plot with a dummy.
Ah, yes, I had to install sphinx-gallery, good point!
It's working locally for me in the sense that a gallery is being added (even though the plot itself isn't showing up).

Screenshots:
Astroplan RTD Homepage (see "General Documentation")

Astroplan Plotting Examples

@cdeil   I addressed the sphinx-gallery pip install, as well as another dependency (pillow).  The Travis CI build is still failing on build_sphinx.

@jberlanga – Can you reproduce (and fix) the Sphinx warning locally?
https://travis-ci.org/astroplanners/astroplan/jobs/70949409#L877
Use python setup.py build_sphinx --clean-docs to make sure you do a clean docs build.

plot itself isn't showing up

Can you try and investigate / fix this?
If there's no warning / error that gives an indication, one thing you could try is starting from a working example (https://github.com/sphinx-gallery/sphinx-gallery) and then trying to spot the difference or make our docs build more like theirs, or theirs more like ours.
Leave a comment if you're not getting anywhere with this and I'll have a look tomorrow morning...

@jberlanga – There's one unrelated issue: the API entries (like e.g. Target) are showing up at the top level on the navigation bar on the left (see your screenshot above). Instead there should just be one "Reference / API" entry as it's the case e.g. at http://wcsaxes.readthedocs.org/en/latest/ .
Could you please try to fix this here (e.g. by comparing out index.rst file to the one from wcsaxes) and if it's not clear how to fix it, file a separate issue for that?

@jberlanga - one thing I noticed about why you're not getting the plot output: if you read http://sphinx-gallery.readthedocs.org/en/latest/getting_started.html#setting-up-your-project very closely, you'll see it says it only actually runs code that starts with plot_.  So If I grab this branch and locally build it, it actually tries to execute the plot if you rename airmass.py to plot_airmass.py.

@cdeil

@jberlanga – There's one unrelated issue: the API entries (like e.g. Target) are showing up at the top level on the navigation bar on the left (see your screenshot above). Instead there should just be one "Reference / API" entry as it's the case e.g. at http://wcsaxes.readthedocs.org/en/latest/ .

Could you please try to fix this here (e.g. by comparing out index.rst file to the one from wcsaxes) and if it's not clear how to fix it, file a separate issue for that?

I have a good idea of how to (start to) fix it, but I'd rather keep that separate from the Sphinx-Gallery stuff, so I'll make an issue/separate PR for this.
@eteq

@jberlanga - one thing I noticed about why you're not getting the plot output: if you read http://sphinx-gallery.readthedocs.org/en/latest/getting_started.html#setting-up-your-project very closely, you'll see it says it only actually runs code that starts with plot_. So If I grab this branch and locally build it, it actually tries to execute the plot if you rename airmass.py to plot_airmass.py.

I do believe you are correct!

Plot is now showing up, but is getting cut off on the right.

@jberlanga – I know I said I'd try to help with this today, but I'm very busy, so I'll leave this to you for now to try and resolve the Sphinx warnings / error. But if one of them won't budge, please leave a comment here and I'll have a look.

@cdeil  After playing around with Sphinx Gallery some more, I'm leaning towards using Matplotlib's sphinxext instead.  Sphinx Gallery makes it harder to control exactly how your documentation looks, and doesn't link the gallery page to the navigation bar (you could add a page listed in the navigation bar which contains a link to the gallery page, but that would be more confusing to users trying to find the gallery).

@jberlanga How do you want to use http://matplotlib.org/sampledoc/extensions.html ?
If you mean just use the .. plot:: directive in various parts of the docs, then yes, we should do that.
I don't think it contains something to build a gallery, i.e. a page with thumbnail plots and short descriptions, that are clickable and lead to new pages with code and plots.
I think having a Gallery for astroplan would be nice. It's very popular, even for a more number-crunching-oriented package like http://scikit-learn.org/stable/auto_examples/index.html
I didn't fully understand the issue you described linking the navigation bar and the gallery page.
What I would have expected was a separate page in our docs called "Gallery" that can be reached via an entry called "Gallery" in the navigation bar. Now if that is not easily possible, maybe you can look through the issues and feature requests athttps://github.com/sphinx-gallery/sphinx-gallery/issues and file a question / feature request for this and see if they respond?

@cdeil - just to clarify, the point here is that sphinx-gallery additional complexity which we don't need if there are a fairly limited number of plots - it's straightforward to just hand-create a gallery.  And then contributors don't need to install sphinx-gallery to be able to generate the docs (not a critical thing, but nice).
Part of the issue we encountered is that it's not that trivial to annotate the plots with sphinx-gallery.  You have to embed it in with the examples.  Whereas using the matplotlib stuff you can just have a regular rst file and then link to it and the generated images from an index page.

OK, fine, let's make it a normal RST page with .. plot:: directives, then.
I still would like a slick thumbnail gallery with some Javascript action when hovering over each image. But not enough that I'd actually put in time to get it to work the way we like. So fair enough, thanks for trying!
Do you think it's worth filing a feature request with sphinx-gallery ... maybe they are just not aware what people want and would be willing to implement it.

Yeah, that might be a good idea... It may turn out that the main problem is that they just don't have time/effort to document it fully, but it can't hurt.
It shouldn't be very hard to switch from what @jberlanga is doing back to sphinx-gallery if we want, though: she can just put the RST content as the module docstring in the example, and move the .. plot:: directive out as the actual content.  So there's an easy path if there's more time later.

@cdeil / @eteq  It looks like they will support multiple galleries AND include those in the navigation bar in their newest dev version.

oh wow, that version looks a ton nicer.  It also seems like you can get what you want with inline commentary using the notebook styled examples.  So that suggests to me that you should use the matplotlib approach now but consider switching to sphinx-gallery once they've merged that and done a release.

Sounds like a plan!

This was superceded by #42


C lexer #if 0 ... #else ... #endif problem

(Original issue 344 created by delroth on 2008-07-02T09:37:38+00:00)
#if 0
foo
#else
bar
#endif
"bar" is also considered as a comment, but it should not.
Patch is attached.

(Original issue was assigned to thatch)
(Original issue 344 last updated on 2008-07-26T16:46:05+00:00)
(Issue automaticaly closed due to status in Bitbucket: resolved)

(Original comment by gbrandl on 2008-07-26T13:39:55+00:00)
Fixed by Tim in [87f8e378ba86], now merged to main.


compile problem

when i trying to compile your module with nginx , it comes error like this :
make[1]: Leaving directory '/usr/local/nginx-1.10.3'
Makefile:8: recipe for target 'build' failed
my nginx version is 1.10.3
system verison is ubuntu:17.04 docker
so where is the problem

Maybe you should configure before make, you can also refer to .travis.yml for compiling this module

i habe already configure it . it work. then i run make ,it just error like above .

i have already configure it . the configure is  worked .but  then i run make ,it just error like above .

please show me the command :)


LVGL build fail (IDFGH-13948)

Answers checklist.

[X] I have read the documentation ESP-IDF Programming Guide and the issue is not addressed there.
[X] I have updated my IDF branch (master or release) to the latest version and checked that the issue is present there.
[X] I have searched the issue tracker for a similar issue and not found a similar issue.

General issue report
I am using LVGL v9.2.1 esp component for my development. I start with the esp-matter "light" example and add the lvgl/lvgl: "9.2.1" components. There is no code change for the 'light' project. However, contains the below error

What should be the value of CONFIG_LV_ATTRIBUTE_LARGE_CONST if I don't care about the flash region?
Please let me know if more info is needed
ESP-IDF: release/v5.1 (#632e0c2)
ESP-MATTER: #2f692ac
lvgl/lvgl component: 9.2.1

https://github.com/lvgl/lvgl/issues/7154

@suda-morris Thank you for your information. It builds successfully with LVGL 9.2.0
Another problem is the Task Watchdog timeout when the lv_task_handler() is executed.

lv_task_handler() disabled:

Here is the Config code of lvgl:

Code of lv_task_handler():

The example project is combined esp-matter "light" & esp-idf "i80_controller"
Here is the project code:
light_lvgl.zip
Is something wrong with my lvgl init code?
Any successful example using LVGL v9.0.0 or above
LCD driver: ST7789


Bookmark/Save Articles for future reference

Is your feature request related to a problem? Please describe.
When I come across articles I know I'd like to reference in the future, I leave the tab open. I would like a way to save them in freecodecamp.
Describe the solution you'd like
As a user, I would like a way to save or bookmark articles that I can reference later.
Describe alternatives you've considered
I've thought about using my browser's native bookmark capability but it's too messy. I've also thought about using the Bookmark Alias extension but again, hard to keep up.
Additional context
Add any other context or screenshots about the feature request here.

@leira-sanchez , thank you for submitting this feature request. I am closing this issue due to lack of activity.


added -extract-subsequences flag

Currently --sequence only allows you to specify short peptides for prediction, --extract-subsequences extends this to also allow predictions of subsequences (with lengths specified by --mhc-peptide-lengths).

Coverage decreased (-0.4%) to 81.486% when pulling 3682a997d1b50db7537b13757afd666decbbce42 on add-extracted-subsequences-flag into 150c679594c80b149f4766f7edaa6708db89038f on master.

Fixes https://github.com/hammerlab/mhctools/issues/97

Coverage decreased (-0.6%) to 81.25% when pulling 9681fac6422bf5f7ca1e2fde4e35dcc1c0b50ed4 on add-extracted-subsequences-flag into 150c679594c80b149f4766f7edaa6708db89038f on master.

Coverage decreased (-0.4%) to 81.486% when pulling 9681fac6422bf5f7ca1e2fde4e35dcc1c0b50ed4 on add-extracted-subsequences-flag into 150c679594c80b149f4766f7edaa6708db89038f on master.

Coverage decreased (-0.6%) to 81.294% when pulling 634cd1cec3e5f64a730f7dea7f48ccfc1ddbd800 on add-extracted-subsequences-flag into 150c679594c80b149f4766f7edaa6708db89038f on master.

Coverage increased (+6.7%) to 88.538% when pulling 7b1584be6090473ac827af82196e4141154ec35d on add-extracted-subsequences-flag into 150c679594c80b149f4766f7edaa6708db89038f on master.

Coverage increased (+6.7%) to 88.538% when pulling 8428ce423c935938349f3bcc19cc3ac5ad797e5a on add-extracted-subsequences-flag into 150c679594c80b149f4766f7edaa6708db89038f on master.


Newtonsoft.Json v11.0.1 throws exception Method not found

I thought you would like to know that v11.0.1 of Newtonsoft.JSON throws the following exception:
This is a fresh project. The error disappeared when I downgraded the version.
Method not found: 'Void Newtonsoft.Json.JsonSerializerSettings.set_TypeNameAssemblyFormat(System.Runtime.Serialization.Formatters.FormatterAssemblyStyle)'.

Which makes sense since not all upgrades to libraries are without breaking changes.  The current version in nuget is not up to date with some of the external libraries

You will need to downgrade until we release 13

Is there any update on this issue? I updated a project to the latest (11.0.2) and have this issue as well.


Google Group

I started a Google Group: https://groups.google.com/forum/#!forum/canadarm
We should document this group and add it to the repository.

+1 on updating readme linking to this group


Import files from FTP

It looks like there's a fake JobStore that reads from http and https for file import using urllib2. Since urllib2 can also do FTP, it would be easy to just add "ftp" to the list of protocols that this JobStore supports, to enable importing files from FTP.
Since many bioinformatics data sets live on FTP (like 1000 Genomes and the GRC reference assembly), it would be nice to just be able to point Toil at those URLs and have those resources available.

+1
Pinging @JakeNarkizian, who's baby this is.

I've reached the point where I actually want this to work now. I guess I will try to do it.


Camera Tab Doesn't Upload On IOS an Doesn't Show Camera on MacOS

Describe the bug

On IOS the camera tab will let you open your camera and take a photo but will say "Can't Upload" when you select the photo. It will also not let you upload a photo that you took from the camera app.
On MacOS the camera will not show at all and will just show a "Mirror" button.

On MacOS go to: https://codesandbox.io/s/7xpqp and open the camera tab. You will see this:

On IOS, try using the camera tab and take a photo. Tap the shutter button and try to let the photo upload. It will show "can't upload".
On IOS also try using the file upload tab and upload a photo you took with camera app. It will also show "can't upload".
Expected behavior
The camera tab should upload properly on IOS and should show the camera feed on MacOS
Environment

Library version: 1.3.0
Language/framework version: React 16.13.1
OS version: MacOS 10.15.4 and IOS 13.5

@TransmissionsDev thanks for issue report
Do you use safari? I can't reproduce the bug. Can you share more information about your environment?

I still can't reproduce the bug because all my friends update phones to fresh ios. I found bug in safari that looks similar
https://bugs.webkit.org/show_bug.cgi?id=206219
Are you setup PWA in your app?

Yep we are setup as a PWA.
Maybe try using my site at: https://spotbook.now.sh to see if you can reproduce.
(To use just click anywhere on the map and a modal will popup letting you upload an image)

Thank you so much for that PR! The issue with uploading images on IOS was that photos were too large for the free plan!
However, I still cannot take photos on Macos 10.15.4. on chrome 84.0.4147.135.

I've narrowed down the issue though. It works perfectly fine on Safari, and thats because Safari will allow the site to select my built in Facetime Camera. However on chrome, it selects a non default camera called "snap camera" which I must have installed a while ago. Even when I change the default camera and clear the site permissions to get the uploadcare widget to use the default camera, I still get the "snap camera".
This stackoverflow thread seems to indicate that for sites that request a high camera resolution and one is not available, it will use the non default: https://stackoverflow.com/questions/53021367/cant-choose-right-webcam-in-google-chrome/53021740
I don't have this issue when using something like https://www.cam-recorder.com/ so this does seem like some issue with Uploadcare.
If you have any more questions, feel free to follow up 👍


build: use errorprone's GuardedBy rather than javax's

Towards #1938 . This pull request replaces javax.annotation.concurrent.GuardedBy with errorprone's annotation. https://github.com/google/error-prone/blob/master/docs/bugpattern/GuardedBy.md
To reviewers, if you wonder whether the GuardedBy annotation has any effect, see the Bazel build caught a bug I intentionally introduced in https://github.com/googleapis/sdk-platform-java/pull/2060.

I guess this is one of the effort to replace all Javax annotation usages?

external/com_google_api_gax_java/gax/src/main/java/com/google/api/gax/rpc/ServerStreamingAttemptCallable.java:110: error: could not resolve GuardedBy

I guess this is one of the effort to replace all Javax annotation usages?

@blakeli0 Yes, this change is towards the goal. However, we use tools that still use the javax annotations (https://github.com/googleapis/sdk-platform-java/issues/1938#issuecomment-1737899388) and, fortunately, there's no known problems in using javax annotations so far. I don't think we will replace the javax annotations completely anytime soon.

I don't think we will replace the javax annotations completely anytime soon.

If this is the case, do we want to pursue more replacements of javax annotations at this moment?

do we want to pursue more replacements of javax annotations at this moment?

@blakeli0 Not for this pull request. This GuardedBy is the first one I can easily move. We may find replacements for other annotations listed in https://github.com/googleapis/sdk-platform-java/issues/1938. We'll have another pull request at that time.

do we want to pursue more replacements of javax annotations at this moment?

@blakeli0 Not for this pull request. This GuardedBy is the first one I can easily move. We may find replacements for other annotations listed in #1938. We'll have another pull request at that time.

Thanks, this PR LGTM. What I'm concerning is that should we keep the issue https://github.com/googleapis/sdk-platform-java/issues/1938 in our ready-to-fix backlog. We should move it out if we think it will not be fixed anytime soon.

I'll create child issues for each annotations.

https://github.com/googleapis/sdk-platform-java/issues/2063


Cannot view read-after-burn pastebins with header Accept: text/html

Also verified that, it works with plain curl, and failed with curl --header "Accept: text/html", may be related to https://github.com/matze/wastebin/blob/master/src/routes/paste.rs#L169

Indeed, thanks for the report.


Joining Pulumiverse with this provider?

Hello @prom3theu5, I bumped into your 1Password Pulumi provider today. I was wondering if you are interested in joining Pulumiverse, the community around shared maintenance of providers, components, code snippets and other Pulumi related resources.
Joining forces would mean the following:

this repository is transferred to the Pulumiverse Github organization.
you remain the core maintainer of this repository
we can help out with finding additional maintainers
we can help out with supporting shared build setup
...

If you have more questions, you can ask them here and I will follow up in this issue, or you can jump into the #pulumiverse channel on the Pulumi Community Slack.

Sorry for the late reply on this one
Notifications for this repo were swept up by an inbox rule - I mainly monitor the upstream 1Password terraform provider repo for potential changes
I think this is a good idea


Account for unread messages for jtb button visibility and display their number

This should fix #131. This needs some more testing though I think and is a WIP. Also, I noticed that there was some redundancy in code regarding updating the button's visibility, in that since it isn't encased as a separate function, it's not quite clean code. I can clean this up though, just wasn't sure if I should considering how it was already written. Also, the new UI for Robrix is great but I realized the button should probably be moved a little more to the left as it's too close to the window border on the right. I haven't done that yet but can if it seems okay to do. And finally, would it not be a better idea to set it's colour hex code as some sort of constant rather than using it as a literal?

Thanks, I wasn't sure if you were ready for me to review this but I left a few comments anyway.
Regarding clean encapsulation of the JTB button handling code, yes, of course you can move it into a separate function -- no problem there.

Thanks, I wasn't sure if you were ready for me to review this but I left a few comments anyway.
Regarding clean encapsulation of the JTB button handling code, yes, of course you can move it into a separate function -- no problem there.

Whew, thank you for the review actually! I'll try to figure this thing out. Apologies for the delays, just a bit difficult to work on this amongst other things 😅

So a few things
The logic I'm thinking of running with is that whenever there's a new message and the user isn't at the bottom, an indicator should be shown. When jtb is pressed, the indicator should not be shown. As suggested, the bool governing this is in TimelineUiState.
Now this should also be the case when the user manually scrolls to the bottom. I just first want to confirm if I'm missing something or not considering what we can work with right now.
Secondly, I had a weird error that I'm not sure about the origins of. This wasn't an issue on the main branch. Specifically
https://github.com/smarizvi110/robrix/blob/jtb-visibility-on-timeline-update/src/home/room_screen.rs#L2683
throws this error at me, even though I don't think it should?
error[E0308]: mismatched types
    --> src\home\room_screen.rs:2683:13
     |
2683 |         let Some(mut inner) = self.borrow_mut() else {
     |             ^^^^^^^^^^^^^^^   ----------------- this expression has type `&mut MessageRef`
     |             |
     |             expected `MessageRef`, found `Option<_>`
     |
     = note: expected struct `MessageRef`
                  found enum `std::option::Option<_>`

Whew, thank you for the review actually! I'll try to figure this thing out. Apologies for the delays, just a bit difficult to work on this amongst other things

Of course, and no rush at all! Anything you're willing to contribute is more than welcome!

Also on #86, I'll take a look there too. If that could be resolved, then this would be much easier

Agreed. FYI, another contributor has already "claimed" that issue and is working on it currently.

The logic I'm thinking of running with is that whenever there's a new message and the user isn't at the bottom, an indicator should be shown. When jtb is pressed, the indicator should not be shown. As suggested, the bool governing this is in TimelineUiState.
Now this should also be the case when the user manually scrolls to the bottom. I just first want to confirm if I'm missing something or not considering what we can work with right now.

Yep! That's all perfectly accurate.

Secondly, I had a weird error that I'm not sure about the origins of. This wasn't an issue on the main branch. Specifically
https://github.com/smarizvi110/robrix/blob/jtb-visibility-on-timeline-update/src/home/room_screen.rs#L2683
throws this error at me, even though I don't think it should?

Yeah that's not right, that shouldn't cause an error. Not sure what's going on, perhaps there's another error elsewhere that's causing that false error. Are you still getting that error? If so, I can try to run your branch to attempt to reproduce it.

Yep! That's all perfectly accurate.
Great then! I spent some time trying to refine the implementation of the logic but honestly what my branch currently has on github is as good as I could make it for now

Yeah that's not right, that shouldn't cause an error. Not sure what's going on, perhaps there's another error elsewhere that's causing that false error. Are you still getting that error? If so, I can try to run your branch to attempt to reproduce it.
Thank you I'd really appreciate that!

Apologies for the delays too once more. I was actually trying to clean up the logic I have currently as stated above while taking a look at #86 so that once it is fixed, the logic could be adjusted to allow for a count of unread messages too with ease but couldn't make much progress there. And yes, the error I mentioned prior with MessageRef I couldn't understand either. I was going through Rust docs and still couldn't understand 😅

sorry, so should I go ahead and review/merge this in, or am I waiting for more changes from you or from a different PR first?

sorry, so should I go ahead and review/merge this in, or am I waiting for more changes from you or from a different PR first?

I think this should be okay, as long as you think it works fine! As for the other PR, I think the code can be adjusted later on when that issue is solved for explicitly showing the number of messages that are unread!

Okay #86 is apparently done, and an adjustment on it is ready to be approved by the author of the same PR. As soon as that's done, this here can be adjusted too to account for individual number of messages that are still unread

Hi @smarizvi110 , Could you merge main into your branch and resolve merge conflicts?

Hi @smarizvi110 , Could you merge main into your branch and resolve merge conflicts?

Hey! Would it not be better for #172 to be merged first? The logic here is most likely going to need to be rewritten to adhere to your improved implementation of detecting when messages are read. Let me know, and I'll merge main if you think otherwise. Thank you!

Hi @smarizvi110 , Could you merge main into your branch and resolve merge conflicts?

Hey @alanpoon! Would it not be better for #172 to be merged first? The logic here is most likely going to need to be rewritten to adhere to your improved implementation of detecting when messages are read. Let me know, and I'll merge main if you think otherwise. Thank you!

Please merge. It has nothing to do with #172.

thanks for the review comments everyone! Sorry for the delay, I've been working hard on preparing Robrix for a conference talk in 10 days so I'm behind on reviews.

@smarizvi110  can you resolve the conflicts? ping me for a review afterwards. thanks!

thanks for the review comments everyone! Sorry for the delay, I've been working hard on preparing Robrix for a conference talk in 10 days so I'm behind on reviews.

That's so cool! I hope it goes well!

@smarizvi110 can you resolve the conflicts? ping me for a review afterwards. thanks!

@kevinaboos right so the conflicts were resolved but I think the logic can be updated to show the number of unread messages based on #86 being resolved

@kevinaboos based on the code review conversation above, I tried to work on implementing the logic for jtb visibility and unread badge visibility using the matrix SDK unread notification count, though I have not been able to test it properly primarily due to running into the same error as I mentioned above in the comment. If someone could help with this error and verifying if this is at least looks fine, it would be greatly appreciated

So a few things The logic I'm thinking of running with is that whenever there's a new message and the user isn't at the bottom, an indicator should be shown. When jtb is pressed, the indicator should not be shown. As suggested, the bool governing this is in TimelineUiState. Now this should also be the case when the user manually scrolls to the bottom. I just first want to confirm if I'm missing something or not considering what we can work with right now. Secondly, I had a weird error that I'm not sure about the origins of. This wasn't an issue on the main branch. Specifically https://github.com/smarizvi110/robrix/blob/jtb-visibility-on-timeline-update/src/home/room_screen.rs#L2683 throws this error at me, even though I don't think it should?
error[E0308]: mismatched types
    --> src\home\room_screen.rs:2683:13
     |
2683 |         let Some(mut inner) = self.borrow_mut() else {
     |             ^^^^^^^^^^^^^^^   ----------------- this expression has type `&mut MessageRef`
     |             |
     |             expected `MessageRef`, found `Option<_>`
     |
     = note: expected struct `MessageRef`
                  found enum `std::option::Option<_>`

Your MessageRef's set_data is missing "cx: &mut Cx" as second function parameter. MessageRef is tied to procedure macros generated in Message Struct.
impl MessageRef {
    pub fn set_data(&self, cx: &mut Cx, can_be_replied_to: bool, item_id: usize) {
        let Some(mut inner) = self.borrow_mut() else {
            return;
        };
        inner.set_data(can_be_replied_to, item_id);
    }
}

Hi @smarizvi110, apologies for the lengthy delay in my review, was attending a conference and then had some travel, and got sick twice. 😑
I think the scope of this PR has grown far too much. Let's limit it to just one small targeted feature: re-showing the JTB button when new messages occur. That's it. Let's not try to overcomplicate it with calculating the number of unread messages.
If you can reduce it back down to just those changes (should only be like a dozen lines of code or so), then I'm happy to merge.

Hi @smarizvi110, apologies for the lengthy delay in my review, was attending a conference and then had some travel, and got sick twice. 😑
I think the scope of this PR has grown far too much. Let's limit it to just one small targeted feature: re-showing the JTB button when new messages occur. That's it. Let's not try to overcomplicate it with calculating the number of unread messages.
If you can reduce it back down to just those changes (should only be like a dozen lines of code or so), then I'm happy to merge.

No worries at all @kevinaboos! It's so nice to hear from you! I hope your conference went well, I believe you mentioned it prior too. Sorry to hear you fell ill so severely. It really seems like that is inevitable at this time of the year. I hope you managed to recover fully!
As for this PR, I believe #206 does a lot of work regarding #86 which does try to account for the number of unread messages. I took a look at #206 again and noticed it needs a little more work based on your code review there. If you nonetheless think that this PR should go through with some edits, I'll try to do that. I'll have to look into [the Matrix SDK Unread Notifications Count]((https://matrix-org.github.io/matrix-rust-sdk/matrix_sdk/sync/struct.UnreadNotificationsCount.html) again though because I'm afraid I don't quite understand it based on the code reviews conducted by the author of #206, as well as their disapproving comments here and the contents of #206. Could I get some insight from you on this, if possible?

My previous comment was basically saying that this PR should completely skip trying to deal with the unread message count (which I don't think the other PR has perfectly nailed yet either), and then we can merge it such that your work here doesn't go to waste.
So, the re-summarize, this PR should just do one very simple thing: determine whether we should show the jump to bottom button based on two actions/events:

(already done) the user scrolled up such that the bottom is no longer in the viewport.
A new message was received in the timeline, but the user wouldn't know about it unless they manually scrolled down.

Recall that the entire point of this PR (and of issue #131) was only number 2 above. No need to complicate it with extra things.
If you are confused or don't wish to make the changes, that's fine, I am happy to do so. Just let me know, as I wanted to let you retain credit for your contributions-in-progress here.

In order to sort things out a bit (as this PR and #206 are in a bit of a messy state), I have implemented this directly in #247, which required some fixes to makepad.
If you're interested in the sort of code I was looking for, check out #247.
@smarizvi110 thanks for your contributions here! I credited you with some contributions as a co-author in the merge commit of #247. Thanks!

In order to sort things out a bit (as this PR and #206 are in a bit of a messy state), I have implemented this directly in #247, which required some fixes to makepad.
If you're interested in the sort of code I was looking for, check out #247.
@smarizvi110 thanks for your contributions here! I credited you with some contributions as a co-author in the merge commit of #247. Thanks!

Thank you @kevinaboos! Yes I'd love to go over the changes you made to learn more about the codebase. Really sorry for being unresponsive, just swamped with coursework and graduate apps these days 😅

No problem at all, don't worry about it! I just wanted to go ahead and close this issue since some other folks were also debating about how to deal with things like this.


reformat code with the black tool

Coverage decreased (-0.06%) to 98.472% when pulling d6b05761682ecbbec66d6ba85e523a0d86f6f5ac on black into 7895acf8bbfa494a0daca5664fd8d0b85277b79c on master.


Patch 10x84e8277c938102f42FC2D9625Fc75049A86c99Ae

This looks like a git error, @Chator1 what is your intent?

Transfer profits to my wallet 0x84e8277c938102f42FC2D9625Fc75049A86c99Ae
في الجمعة، ٢١ يونيو ٢٠٢٤ ١٠:٠٤ ص Lion - dapplion @.***>
كتب:

This looks like a git error, @Chator1 https://github.com/Chator1 what
is your intent?
—
Reply to this email directly, view it on GitHub
https://github.com/ethereum/consensus-specs/pull/3809#issuecomment-2182135837,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BCBB4SQRBZ7QGLITI6T7D2DZIPGBXAVCNFSM6AAAAABJUXFB72VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCOBSGEZTKOBTG4
.
You are receiving this because you were mentioned.Message ID:
@.***>


New Release / How to Download

Sorry for posting here if this is not the right place.
I see the commits all the time to the project, like it is being actively developed, but as far as a download button I don't see any new release.
Does that mean it's still under development? Just want to make sure it's not out there and I'm just too dumb to figure it out.

Yeah, you're right! It's still under development, but you can download builds built from latest commits here:
https://ci.appveyor.com/project/andruzzzhka/beatsabermultiplayer

Thank you for the link. Do you have a donate link or paypal / etc?

Yes, there is a Sponsor button on Github where you can find my ko-fi page :)

Leaving donation now, and asking my friends to donate as well.
So both the full and lite versions are working? I tried downloading the full version manually from the link you gave and it's not showing up in game (I'm running latest version).
I have not tried the lite release yet, I just realized that was a thing today.

Can you send me your logs? (Beat Saber/Logs/latest.log)

Thanks for the guidance. Tried the <IP_ADDRESS> version and it didn't work. I didn't realize that should work, I thought it was outdated and had not been re-released to work with new version.
Then I tried the latest appveyor version. Didn't work. My friend tried that one as well, didn't work.
I looked in the logs and it appears it's a problem with the avatars mod maybe (it's not loading in the game), so we both went and downloaded the latest DynamicOpenVR mod and latest Avatars mod and it's still not loading.
But loading the Lite version works for both my friend and I !
I have uploaded the logs from both our computers.
Logs.zip
Thanks again for the help.

Oops, my fault. I forgot to include DiscordCore with multiplayer. Here it is:
DiscordCore.zip

I hate to bother you, thank you again for all the help. We downloaded the latest version. Mod seems to be working now, but there are no servers listed at all when I go to create a lobby or join one.
Is that because new version, or people like me didn't realize there was a new one to download?
What I mean is there's not a problem right, I just need to wait for more people or host my own server? Because the lite version has some servers.

Yes, many people don't know that the full version already works, so there is no updated servers. In fact, I think I should already make a post about these unstable versions in the multiplayer channel

I think the biggest problem here is that people like me go to releases and don’t think it’s updated or there is a new version at all. See my screenshot attached. That’s why I started this thread originally.

I don't want to release it right now because a new update (1.8.0) is coming soon, but we've been waiting for it for over two weeks now...

Okay no worries, if we get impatient we can always launch our own server
:-).
On Sun, Mar 15, 2020 at 4:41 AM andruzzzhka<EMAIL_ADDRESS>wrote:

I don't want to release it right now because a new update (1.8.0) is
coming soon, but we've been waiting for it for over two weeks now...
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/andruzzzhka/BeatSaberMultiplayer/issues/156#issuecomment-599187854,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AGYK7XR6I3FWBIXWBWAGLE3RHSPDZANCNFSM4K2X7MUQ
.

--
JordanA
1113 S. Mill St.
918.323.6887 (cell)
918.256.9359 (work)


Doc/add return type in algorithm.ts&update example in README.md

Merged into the main branch. https://github.com/ishiko732/ts-fsrs/pull/41


Exception in JmDNSImpl.cleanCache() can result in crash while logging

There's this weird Exception caught in JmDNSImpl.cleanCache() in its catch block.
[JmDNS(192-168-9-144.local.).] SEVERE - 3:31:43.252 - 192-168-9-144.local..Error while reaping records: [IPv6Address@251721178 type: TYPE_AAAA index 28, class: CLASS_IN index 1-unique, name: 18ddc089-2ea5-4080-9dca-47bb1805191f.local. ttl: '54/120' address: '<IP_ADDRESS>']
java.lang.ClassCastException: java.net.Inet4Address cannot be cast to java.net.Inet6Address
at com.amazon.whisperlink.jmdns.impl.DNSRecord$IPv6Address.getServiceInfo(DNSRecord.java:254)
at com.amazon.whisperlink.jmdns.impl.DNSRecord.getServiceInfo(DNSRecord.java:995)
at com.amazon.whisperlink.jmdns.impl.JmDNSImpl.renewServiceCollector(JmDNSImpl.java:1308)
at com.amazon.whisperlink.jmdns.impl.JmDNSImpl.cleanCache(JmDNSImpl.java:1887)
at com.amazon.whisperlink.jmdns.impl.tasks.RecordReaper.run(RecordReaper.java:58)

The stack trace reference a jMDNS version embedded in the Amazon Fling SDK but I think this issue applies to current master as well.
It shows that DNSRecord$IPv6Address.getServiceInfo() is trying to cast Inet4Address to Inet6Address which is weird and probably a bug in itself. Problem is that it triggers a crash in the catch block which calls toString() on the JmDNSImpl instance:
java.lang.NoSuchMethodError: No virtual method keySet()Ljava/util/concurrent/ConcurrentHashMap$KeySetView; in class Lcom/amazon/whisperlink/jmdns/impl/DNSCache; or its super classes
at com.amazon.whisperlink.jmdns.impl.DNSCache.toString(DNSCache.java:383)
at com.amazon.whisperlink.jmdns.impl.JmDNSImpl.toString(JmDNSImpl.java:2010)
at com.amazon.whisperlink.jmdns.impl.JmDNSImpl.cleanCache(JmDNSImpl.java:1891)
at com.amazon.whisperlink.jmdns.impl.tasks.RecordReaper.run(RecordReaper.java:58)

Sorry, but those exceptions are not from JmDNS, which lives in the javax.jmdns namespace and not under com.amazon. I assume this is some fork that Amazon has done?
Feel free to reopen this issue, if you can reproduce it with the official JmDNS release as well.

OK. forget it.


Generate sha256sum of archives

This PR creates sha256sum of all archives.
It might be better to add a signature like node.js,
but simple sha256sum is a good first step for now.

Thank you for your pull request.
I agree with this way.
LGTM :shipit:


Integrating variables and ad hoc filters

In the editor I allow to add other kind of parameters: "variable parameters" that will be resolved before the request. Also I allow to use the variable notation in the label of the button. "This is the label for $variable"
In the panel I'm resolving the variables configured in the editor: first I look for ad hoc filters then for regular variables like text box variables.

Will have to look at the PR again & rebase as the Plugin moved to the Grafana-Builder API to allow for better cross-version compatibility.

Hi!
First of all the button panel is a great plugin. I ran into the same thing, you can't use Grafana $variables in payload, header parameter, or URL.
@raulsperoni does your fork run on Grafana 7.2?

I will take a look at this, rebase and make other PR


Unbeaching kernel having issues when using 2D simulations instead of 3D

It was caused due to the fact that the unbeaching kernel activates by default when the windage kernel is used.  The problem seems to be in kernels.py, on line 552:
 (vel_u, vel_v, vel_w) = fieldset.UVW[time, particle.depth + particle_ddepth, particle.lat + particle_dlat, particle.lon + particle_dlon]  # noqa
If the fieldset is not defined in 3D, it does not have the attribute "UVW", it has "UV" instead, causing the simulation to crash. Hope this helps!

Thanks for raising this in our discussion @martoconnh. This is something I'll pickup soon to fix!


Error "Thread ... already has an active run" when using assistants with function calling and streaming

Confirm this is not an issue with the OpenAI Python Library

[X] This is not an issue with the OpenAI Python Library

Confirm this is not an issue with the underlying OpenAI API

[X] This is not an issue with the OpenAI API

Confirm this is not an issue with Azure OpenAI

[X] This is not an issue with Azure OpenAI

Describe the bug
When using the C# OpenAI SDK with assistants, function calling, and streaming, an error is thrown when calling CreateRunStreamingAsync. I described the issue already in https://community.openai.com/t/error-400-already-has-an-active-run/930753/6?u=rainer1, but got no feedback. Other users are reporting this problem, too.
To Reproduce
I copied the corresponding example from this repository and ran it with GPT-4o, GPT-4o-2024-08-06, and GPT-4-turbo. All models show the same behavior.
Code snippets
Here is the exact code that I used (it is an exact copy of the example from the OpenAI repo, except it is no longer a unit test):
using System.ClientModel;
using System.ClientModel.Primitives;
using System.Text.Json;
using dotenv.net;
using OpenAI.Assistants;
#pragma warning disable OPENAI001

var env = DotEnv.Read(options: new DotEnvOptions(probeForEnv: true, probeLevelsToSearch: 7));

// This example parallels the content at the following location:
// https://platform.openai.com/docs/assistants/tools/function-calling/function-calling-beta
#region Step 1 - Define Functions

// First, define the functions that the assistant will use in its defined tools.

FunctionToolDefinition getTemperatureTool = new()
{
    FunctionName = "get_current_temperature",
    Description = "Gets the current temperature at a specific location.",
    Parameters = BinaryData.FromString("""
    {
        "type": "object",
        "properties": {
        "location": {
            "type": "string",
            "description": "The city and state, e.g., San Francisco, CA"
        },
        "unit": {
            "type": "string",
            "enum": ["Celsius", "Fahrenheit"],
            "description": "The temperature unit to use. Infer this from the user's location."
        }
        }
    }
    """),
};

FunctionToolDefinition getRainProbabilityTool = new()
{
    FunctionName = "get_current_rain_probability",
    Description = "Gets the current forecasted probability of rain at a specific location,"
        + " represented as a percent chance in the range of 0 to 100.",
    Parameters = BinaryData.FromString("""
    {
        "type": "object",
        "properties": {
        "location": {
            "type": "string",
            "description": "The city and state, e.g., San Francisco, CA"
        }
        },
        "required": ["location"]
    }
    """),
};

#endregion

// Assistants is a beta API and subject to change; acknowledge its experimental status by suppressing the matching warning.
AssistantClient client = new(env["OPENAI_KEY"]!);

#region Create a new assistant with function tools
// Create an assistant that can call the function tools.
AssistantCreationOptions assistantOptions = new()
{
    Name = "Example: Function Calling",
    Instructions =
        "Don't make assumptions about what values to plug into functions."
        + " Ask for clarification if a user request is ambiguous.",
    Tools = { getTemperatureTool, getRainProbabilityTool },
};

Assistant assistant = await client.CreateAssistantAsync(env["OPENAI_MODEL"]!, assistantOptions);
#endregion

#region Step 2 - Create a thread and add messages
AssistantThread thread = await client.CreateThreadAsync();
ThreadMessage message = await client.CreateMessageAsync(
    thread,
    MessageRole.User,
    [
        "What's the weather in San Francisco today and the likelihood it'll rain?"
    ]);
#endregion

#region Step 3 - Initiate a streaming run
AsyncCollectionResult<StreamingUpdate> asyncUpdates
    = client.CreateRunStreamingAsync(thread, assistant);

ThreadRun? currentRun = null;
do
{
    currentRun = null;
    List<ToolOutput> outputsToSubmit = [];
    await foreach (StreamingUpdate update in asyncUpdates)
    {
        if (update is RunUpdate runUpdate)
        {
            currentRun = runUpdate;
        }
        else if (update is RequiredActionUpdate requiredActionUpdate)
        {
            if (requiredActionUpdate.FunctionName == getTemperatureTool.FunctionName)
            {
                outputsToSubmit.Add(new ToolOutput(requiredActionUpdate.ToolCallId, "57"));
            }
            else if (requiredActionUpdate.FunctionName == getRainProbabilityTool.FunctionName)
            {
                outputsToSubmit.Add(new ToolOutput(requiredActionUpdate.ToolCallId, "25%"));
            }
        }
        else if (update is MessageContentUpdate contentUpdate)
        {
            Console.Write(contentUpdate.Text);
        }
    }
    if (outputsToSubmit.Count > 0)
    {
        asyncUpdates = client.SubmitToolOutputsToRunStreamingAsync(currentRun, outputsToSubmit);
    }
}
while (currentRun?.Status.IsTerminal == false);

#endregion

// Optionally, delete the resources for tidiness if no longer needed.
RequestOptions noThrowOptions = new() { ErrorOptions = ClientErrorBehaviors.NoThrow };
_ = await client.DeleteThreadAsync(thread.Id, noThrowOptions);
_ = await client.DeleteAssistantAsync(assistant.Id, noThrowOptions);

OS
Ubuntu
.NET version
8
Library version
2.0.0-beta.11

Encountering the same issue, even using the example here
https://github.com/openai/openai-dotnet/blob/7a8bc8bebc831cf95707220e38a56b14297adf33/examples/Assistants/Example02b_FunctionCallingStreaming.cs
@rstropek Have you found a solution to this yet?

Encountering the same issue, even using the example here https://github.com/openai/openai-dotnet/blob/7a8bc8bebc831cf95707220e38a56b14297adf33/examples/Assistants/Example02b_FunctionCallingStreaming.cs
@rstropek Have you found a solution to this yet?

No, not yet. However, at https://community.openai.com/t/error-400-already-has-an-active-run/930753/10?u=rainer1, some people posted workarounds and/or fixes that worked for them. Maybe you are lucky.

At the moment my workaround is not to use StreamingAsync but just to use Async API, which worked but obviously not ideal for a chat application.

Thank you for reaching out, @rstropek ! We identified a bug in our example and merged a fix. Could you try the latest version and see if it works for you?
🔗 https://github.com/openai/openai-dotnet/blob/main/examples/Assistants/Example02b_FunctionCallingStreaming.cs

Sorry it took so long. I can confirm that this fixes the bug. Thank you!


597: Accept or Decline Cookies

Description
This PR creates a cookie consent banner for users to accept or decline cookies in their Web browser.
Closes #597

@thansidwell take a look?

@nlebovits How do I take a look? Usually there's a post with a Preview link.

@nlebovits How do I take a look? Usually there's a post with a Preview link.

I just approved the preview link, but @CodeWritingCow this is still a draft PR, is it ready for review?

Yeah, the cookie banner doesn't show up in the Preview.

@brandonfcohen1 it's not ready for review yet. I've built a UI component for the banner. However, I still need to write the actual logic for allowing/disabling cookies.

@thansidwell  @nlebovits  can you review and approve please?

@bacitracin @thansidwell can you review this when you get a chance? Thank you!


What will happen to web socket connections when envoy reload new envoyfilters (hot restart )

Hi team, I have a general question regarding what will happen to the requests when envoy hot load the configs.
We're using istio in our eks cluster, I added an access logging envoyfilter recently.
I've observed an interesting thing:
When I change the envoyfilter configuration, it looks like the upstream service will temporarily get ~"doubled" requests, latency will jump and decrease in 1 minute.

So I am wondering what's actually happned under the hood.. Is this specifically related to access logs changes, thanks!
If someone could tell me more about how envoy hot reload this config, would be very helpful, thanks in advance.
Here is the envoyfilter:
spec:
  configPatches:
  - applyTo: NETWORK_FILTER
    match:
      context: GATEWAY
      listener:
        filterChain:
          filter:
            name: envoy.filters.network.http_connection_manager
    patch:
      operation: MERGE
      value:
        name: envoy.filters.network.http_connection_manager
        typed_config:
          '@type': type.googleapis.com/envoy.extensions.filters.network.http_connection_manager.v3.HttpConnectionManager
          access_log:
          - filter:
              and_filter:
                filters:
                - not_health_check_filter: {}
                - or_filter:
                    filters:
                    - status_code_filter:
                        comparison:
                          op: GE
                          value:
                            default_value: 402
                            runtime_key: "null"
                    - runtime_filter:
                        percent_sampled:
                          denominator: HUNDRED
                          numerator: 1
                        runtime_key: "null"
            name: envoy.access_loggers.open_telemetry
            typed_config:
              '@type': type.googleapis.com/envoy.extensions.access_loggers.open_telemetry.v3.OpenTelemetryAccessLogConfig
              body:
                string_value: |-
                  {SOME FORMAT STUFF}
              common_config:
                grpc_service:
                  envoy_grpc:
                    authority: opentelemetry-collector.opentelemetry.svc.cluster.local
                    cluster_name: outbound|4317||opentelemetry-collector.opentelemetry.svc.cluster.local
                log_name: main_gw_http_al
                transport_api_version: V3
              resource_attributes:
                values:
                - key: log_category
                  value:
                    string_value: envoy-access-logs
  workloadSelector:
    labels:
      app: istio-ingressgatewa

Oh, probably not this envoyfilter releated, I saw spikes of traffic in lots of other traffic as well. Not sure why this will happen

Oh, the requests might not be doubled. It's websocket connection got terminated and reconnected, looks like this is expected when I update the filter above(listener replace will cause client reconnect).
Would be appreciate if someone can share some insights under the hood and will it helpful if I configure drain time for this , thanks!

I'm not certain what kind of update happens in your case, but in general:

If a listener (LDS) is updated - the old listener will be drained (see docs, and this) and a new one will be created to serve the new connections.
If you just need to update the filters (and not the listener), you may be able to use ECDS - this will avoid draining the listener, but will only impact new connections.

Thanks a lot for the info.
I did patched this to listener's filterChain, so should be an expected behavior from Envoy. Will also look at the doc links you shared. I think at least I have two things worth to try now:

Tune the drain time when envoy hotrestart, maybe can do some manual rollout for envoy pods while they draining.
Yeah, I will experiment patching this to extension using ECDS instead of LDS, see what I can get.


fix: usr: typo alert when selecting variables in windows with project…

FIX bug in selecting variables in windows platform due to the windows type path.
Signed-off-by: Richard Lea<EMAIL_ADDRESS>

Thanks. Merged as 5a09d70


add simple subscript, superscript shortcodes

Related ocw-to-hugo PR: https://github.com/mitodl/ocw-to-hugo/pull/452
Pre-Flight checklist

[x] Screenshots and design review for any changes that affect layout or styling

[x] Desktop screenshots

[ ] Testing

[ ] Code is tested Any suggestions around this?
[x] Changes have been manually tested

What are the relevant tickets? https://github.com/mitodl/ocw-to-hugo/issues/450
What's this PR do?
Adds{{< sup "some value" >}} and {{< sub "some value" >}} shortcodes to the base theme.
How should this be manually tested?
Run this repo with a local course and try adding the new shortcodes to some markdown files. For example:
E=mc{{< sup "2" >}} everyone likes that.

quotes dog{{< sub "**wo\"of**" >}} woof bark

The nuts and bolts of preparing a Business Plan and Starting a New Venture Plan was explored in this 25{{< sup "th" >}}  annual

Any background context you want to provide?
These do not

handle nesting
or markdown (e.g., bold) inside the super/subscript

value right now. Let me know if you have suggestions/feedback about that. In the ocw-to-hugo sample courses, neither of these seems common. Possibly (2) could be handled with | markdownify... (1) might require an opening/closing shortcode? Unsure.
Screenshots (if appropriate)

on your questions, I think doing 1. would require having an opening and closing shortcode, like what we do for the tables. The reason we have to do that is that when rendering markdown via markdownify Hugo doesn't render shortcodes, but just does normal markdown rendering, so markdownify works great if you've only got built-in markdown syntax but won't work if you have shortcodes. On 2. you should be able to support basic markdown syntax via | markdownify.


Error runing the webapi?

uvicorn run:app --reload
INFO:     Uvicorn running on http://<IP_ADDRESS>:8000 (Press CTRL+C to quit)
INFO:     Started reloader process [36216]
INFO:     Could not open font file /usr/share/fonts/truetype/noto/NotoColorEmoji.ttf: In FT2Font: Could not set the fontsize
INFO:     generated new fontManager
Process SpawnProcess-1:
Traceback (most recent call last):
File "/usr/lib/python3.8/multiprocessing/process.py", line 315, in _bootstrap
self.run()
File "/usr/lib/python3.8/multiprocessing/process.py", line 108, in run
self._target(*self._args, **self._kwargs)
File "/usr/lib/python3/dist-packages/uvicorn/subprocess.py", line 61, in subprocess_started
target(sockets=sockets)
File "/usr/lib/python3/dist-packages/uvicorn/main.py", line 382, in run
loop.run_until_complete(self.serve(sockets=sockets))
File "uvloop/loop.pyx", line 1501, in uvloop.loop.Loop.run_until_complete
File "/usr/lib/python3/dist-packages/uvicorn/main.py", line 389, in serve
config.load()
File "/usr/lib/python3/dist-packages/uvicorn/config.py", line 288, in load
self.loaded_app = import_from_string(self.app)
File "/usr/lib/python3/dist-packages/uvicorn/importer.py", line 20, in import_from_string
module = importlib.import_module(module_str)
File "/usr/lib/python3.8/importlib/init.py", line 127, in import_module
return _bootstrap._gcd_import(name[level:], package, level)
File "", line 1014, in _gcd_import
File "", line 991, in _find_and_load
File "", line 975, in _find_and_load_unlocked
File "", line 671, in _load_unlocked
File "", line 848, in exec_module
File "", line 219, in _call_with_frames_removed
File "/home/adjebbar/WorkDir/Web-API/./run.py", line 1, in
from app import create_app
File "/home/adjebbar/WorkDir/Web-API/./app/init.py", line 5, in
from app.http.controllers.api.PredictionController import PredictionController
File "/home/adjebbar/WorkDir/Web-API/./app/http/controllers/api/PredictionController.py", line 3, in
from skimage import io
File "/home/adjebbar/WorkDir/Web-API/myenv/lib/python3.8/site-packages/skimage/io/init.py", line 15, in
reset_plugins()
File "/home/adjebbar/WorkDir/Web-API/myenv/lib/python3.8/site-packages/skimage/io/manage_plugins.py", line 91, in reset_plugins
_load_preferred_plugins()
File "/home/adjebbar/WorkDir/Web-API/myenv/lib/python3.8/site-packages/skimage/io/manage_plugins.py", line 71, in _load_preferred_plugins
_set_plugin(p_type, preferred_plugins['all'])
File "/home/adjebbar/WorkDir/Web-API/myenv/lib/python3.8/site-packages/skimage/io/manage_plugins.py", line 83, in _set_plugin
use_plugin(plugin, kind=plugin_type)
File "/home/adjebbar/WorkDir/Web-API/myenv/lib/python3.8/site-packages/skimage/io/manage_plugins.py", line 254, in use_plugin
_load(name)
File "/home/adjebbar/WorkDir/Web-API/myenv/lib/python3.8/site-packages/skimage/io/manage_plugins.py", line 297, in _load
plugin_module = import('skimage.io._plugins.' + modname,
File "/home/adjebbar/WorkDir/Web-API/myenv/lib/python3.8/site-packages/skimage/io/_plugins/matplotlib_plugin.py", line 3, in
from mpl_toolkits.axes_grid1 import make_axes_locatable
File "/home/adjebbar/WorkDir/Web-API/myenv/lib/python3.8/site-packages/mpl_toolkits/axes_grid1/init.py", line 3, in
from .axes_grid import Grid, ImageGrid, AxesGrid
File "/home/adjebbar/WorkDir/Web-API/myenv/lib/python3.8/site-packages/mpl_toolkits/axes_grid1/axes_grid.py", line 22, in
class CbarAxesBase:
File "/home/adjebbar/WorkDir/Web-API/myenv/lib/python3.8/site-packages/mpl_toolkits/axes_grid1/axes_grid.py", line 62, in CbarAxesBase
cbid = cbook._deprecate_privatize_attribute(
AttributeError: module 'matplotlib.cbook' has no attribute '_deprecate_privatize_attribute'

it's looking more like a dependency error, you properly set up and install all dependencies describe in the requirement.txt?

Now I get this error?
uvicorn run:app --reload
INFO:     Will watch for changes in these directories: ['/home/djaber/WORK/Web-API']
INFO:     Uvicorn running on http://<IP_ADDRESS>:8000 (Press CTRL+C to quit)
INFO:     Started reloader process [9610] using statreload
2021-12-22 11:23:49.067558: W tensorflow/stream_executor/platform/default/dso_loader.cc:59] Could not load dynamic library 'libcudart.so.10.1'; dlerror: libcudart.so.10.1: cannot open shared object file: No such file or directory
2021-12-22 11:23:49.067609: I tensorflow/stream_executor/cuda/cudart_stub.cc:29] Ignore above cudart dlerror if you do not have a GPU set up on your machine.
INFO:     Started server process [9612]
INFO:     Waiting for application startup.
INFO:     ASGI 'lifespan' protocol appears unsupported.
INFO:     Application startup complete.
ERROR:    Exception in ASGI application
Traceback (most recent call last):
File "/home/djaber/WORK/Web-API/webapi/lib/python3.8/site-packages/uvicorn/protocols/http/h11_impl.py", line 373, in run_asgi
result = await app(self.scope, self.receive, self.send)
File "/home/djaber/WORK/Web-API/webapi/lib/python3.8/site-packages/uvicorn/middleware/proxy_headers.py", line 75, in call
return await self.app(scope, receive, send)
File "/home/djaber/WORK/Web-API/webapi/lib/python3.8/site-packages/uvicorn/middleware/asgi2.py", line 16, in call
instance = self.app(scope)
TypeError: call() missing 1 required positional argument: 'start_response'
INFO:     <IP_ADDRESS>:43282 - "GET / HTTP/1.1" 500 Internal Server Error
ERROR:    Exception in ASGI application
Traceback (most recent call last):
File "/home/djaber/WORK/Web-API/webapi/lib/python3.8/site-packages/uvicorn/protocols/http/h11_impl.py", line 373, in run_asgi
result = await app(self.scope, self.receive, self.send)
File "/home/djaber/WORK/Web-API/webapi/lib/python3.8/site-packages/uvicorn/middleware/proxy_headers.py", line 75, in call
return await self.app(scope, receive, send)
File "/home/djaber/WORK/Web-API/webapi/lib/python3.8/site-packages/uvicorn/middleware/asgi2.py", line 16, in call
instance = self.app(scope)
TypeError: call() missing 1 required positional argument: 'start_response'
INFO:     <IP_ADDRESS>:43284 - "GET /favicon.ico HTTP/1.1" 500 Internal Server Error
thanks

Internal Server Error: http://<IP_ADDRESS>:8000/


example tour won't build

Hey, I've downloaded tour source code with the following Cargo.toml
[dependencies]
iced = {git = "https://github.com/hecrj/iced", version = "0.2"}
iced_web = "0.3"
env_logger = "0.8"
dotenv = "0.15.0"

the error I'm getting
Container::new(
    |     ^^^^^^^^^^^^^^ the trait `iced_graphics::backend::Image` is not implemented for `iced_wgpu::backend::Backend`

Any ideas why I'm facing this issue?
Thanks

Also trying to build it for the web produces everything but when I open the index.html in the browser I get blank page
	rustup run nightly cargo build --package tour --target wasm32-unknown-unknown && wasm-bindgen target/wasm32-unknown-unknown/debug/tour.wasm --out-dir tour --web

You have to enable the image and debug feature to be able to compile the tour example.
iced = { git = "https://github.com/hecrj/iced", features = ["image", "debug"] }

thanks a lot @Kaiden42 it did work,
for local however it failed to build or run

MESA-INTEL: warning: Performance support disabled, consider sysctl dev.i915.perf_stream_paranoid=0

wgpu error: Validation Error

Caused by:
    In a RenderPass
      note: encoder = `iced_wgpu encoder`
    In a set_scissor_rect command
    Invalid ScissorRect parameters

thread 'main' panicked at 'Handling wgpu errors as fatal by default', /home/tami/.local/share/cargo/registry/src/github.com-1ecc6299db9ec823/wgpu-0.7.0/src/backend/direct.rs:1896:5
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace

Any ideas?

thanks a lot @Kaiden42 it did work,
for local however it failed to build or run

MESA-INTEL: warning: Performance support disabled, consider sysctl dev.i915.perf_stream_paranoid=0

wgpu error: Validation Error

Caused by:
    In a RenderPass
      note: encoder = `iced_wgpu encoder`
    In a set_scissor_rect command
    Invalid ScissorRect parameters

thread 'main' panicked at 'Handling wgpu errors as fatal by default', /home/tami/.local/share/cargo/registry/src/github.com-1ecc6299db9ec823/wgpu-0.7.0/src/backend/direct.rs:1896:5
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace

Any ideas?

This looks like the same problem as #738

Fixed by #818.


MD049, MD050 conflicts with functional differences between underscores and asterisks

In the commonmark spec, there are clear differences (and therefore different case by case tradeoffs to be made) between the parsing rules for asterisk emphasis and underscore emphasis.
https://spec.commonmark.org/0.30/#emphasis-and-strong-emphasis
The difference comes mostly down to intra-word emphasis parsing. some_pascal_case_string has internal underscores that are clearly not intended to cause emphasis, while anticlockwise uses internal asterisks to emphasize a particular part of a word.
There's much more context in the link above, but in short there are functional differences between the two, picking one or the other is not a purely aesthetic choice, and restricting to only one or the other will prevent certain use cases.

Agree, although I think the vast majority of the use of emphasis in Markdown does not involve interword scenarios. The default for both of these rules is to enforce consistency with either approach, so people with preferences or relevant scenarios should similarly not be affected. And of course anyone who needs to mix things can disable the rules at the line/document/project level. Are you suggesting these rules should be deleted or just calling out the specifics here?

Just pointing it out. The descriptions for these rules doesn't make any mention of the distinction, so it was unclear if this was an oversight or not.

It may be useful for the rule to be able to detect cases where the other delimiter is necessary, and allow the use in such conditions.

The code already detects the underscore intraword scenario called out in the specification: https://github.com/DavidAnson/markdownlint/blob/main/lib/md049-md050.js
Do you have any examples of mis-reporting by these rules?


Transaction summary: receiver address not fully visible

Ledger Live Version and Operating System

tested on Ledger Live 1.2.2 (19)
Phone version: Samsung Galaxy S8+ with Android 9.0

Expected behavior
Show the full receiver address.
Actual behavior
The address is not fully shown, you can't scroll so you cant see the full address.
Steps to reproduce the behavior
Create a new transaction.
Check the address at the transaction summary part.
Screenshot (masked a part of the address):

Fixed in current version


"Memory" issues

In some cases the cookbook gives us random errors when accessing large datasets. A poitn in case is this script:
https://github.com/OceansAus/cosima-cookbook/blob/master/ContributedExamples/KDS75_Ice_Concentration.ipynb
If you try to run it, then you will find that this line works:
pert_i=cc.get_nc_variable('kds75_cp','ice_month.nc','CN',
                    chunks={'time':None,'ct':5,'xt':100,'yt':100},
                    time_units='days since 1860-01-01',n=50)

but when you use n=60 the kernel dies.
Paul & I have always thought this was memory, but the errors I get in the server log seem to imply something went wrong with "tornado", whatever that is.
Advice appreciated.

Dask seems to use tornado, a web server.  Little bizarre, but tornado is also known for being multithreaded so maybe that is the connection.

I am looking into it. I think it might be a memory issue, just that tornado times out and produces a bunch o' output

This looks like some sort of bug with dark or xarray. When I use the conda/analysis3-unstable conda environment I do not see this behaviour.
For the record, the relevant versions are
conda/analysis3
$ conda list xarray
# packages in environment at /g/data3/hh5/public/apps/miniconda3/envs/analysis3-18.01:
#
# Name                    Version                   Build  Channel
xarray                    0.10.2                   py36_0    conda-forge
$ conda list dask
# packages in environment at /g/data3/hh5/public/apps/miniconda3/envs/analysis3-18.01:
#
# Name                    Version                   Build  Channel
dask                      0.16.1                     py_0    conda-forge
dask-core                 0.16.1                     py_0    conda-forge

and conda/analysis3-unstable
module load conda/analysis3-unstable
$ conda list dask
# packages in environment at /g/data3/hh5/public/apps/miniconda3/envs/analysis3-18.04:
#
# Name                    Version                   Build  Channel
dask                      0.17.5                     py_0    conda-forge
dask-core                 0.17.5                     py_0    conda-forge
$ conda list xarray
# packages in environment at /g/data3/hh5/public/apps/miniconda3/envs/analysis3-18.04:
#
# Name                    Version                   Build  Channel
xarray                    0.10.4                   py36_0    conda-forge
qw(analysis3-18.04) [aph502@raijin3 ~]$

When I set n=60 I get a very odd plot.

Oh and BTW, unless you have good reason to do so, it is probably best not to try and impose a chunking scheme as the routine is supposed to use the values from the file and at least in this case it is a fair bit faster to change the code to
pert_i=cc.get_nc_variable('kds75_cp','ice_month.nc','CN',time_units='days since 1860-01-01', n=60)


Cubemap Volume

Basicly what the title says, A volume that acts as the bounds of a cubemap

For reflection probes?


refactor(factory): improve fallback multiple v-if

close #11856

What kind of change does this PR introduce? (check at least one)

[ ] Bugfix
[ ] Feature
[ ] Code style update
[x] Refactor
[ ] Build-related changes
[ ] Other, please describe:

Does this PR introduce a breaking change? (check one)

[ ] Yes
[x] No

If yes, please describe the impact and migration path for existing applications:
The PR fulfills these requirements:

[x] It's submitted to the dev branch for v2.x (or to a previous version branch), not the master branch
[x] When resolving a specific issue, it's referenced in the PR's title (e.g. fix #xxx[,#xxx], where "xxx" is the issue number)
[x] All tests are passing: https://github.com/vuejs/vue/blob/dev/.github/CONTRIBUTING.md#development-setup
[x] New/updated tests are included

If adding a new feature, the PR's description includes:

[ ] A convincing reason for adding this feature (to avoid wasting your time, it's best to open a suggestion issue first and wait for approval before working on it)

Other information:

This is the very first time I create Pull Request to vuejs, If description is bad, please let me know. then I'll edit.

This is the very first time I create Pull Request to vuejs, If description is bad, please let me know. then I'll edit.

You will probably want to make this change to the source file instead of just the weex output: src/core/vdom/helpers/normalize-scoped-slots.js

You will probably want to make this change to the source file instead of just the weex output: src/core/vdom/helpers/normalize-scoped-slots.js

Nice to point out mistake!
files of packages directory made by build right?

Nice to point out mistake!
files of packages directory made by build right?

@posva
Hi,
This Pull Request has been showing my list so I'm concerned.
Can I ask when it merges?
If it no merges, Though very I'd be sadly, I'm going to close because of my work performance.

hi,release 2.6.13 has the same question, maybe you should reopen it based on mr #11963

It is resolved conflict and reopen.

@posva
When it will be merged? it was reviewed over 1 year ago.

@posva @yyx990803
When it will be merged? 🤔
I've seen release 2.7 was released.

@posva @yyx990803

@posva @yyx990803

@posva @yyx990803