Your discord client: [ e.g. Powercord ]
Betterdiscord and Powercord
Screenshots
If applicable, add screenshots to help explain your problem.
Additional context
As I stated above, sometimes I can't click on channels after scrolling down. This happens both on powercord and betterdiscord.
I am using the SmoothScroll plugin, just like you suggested. However, this does not always happen.
In order to fix I need to disable the theme. Disabling the plugin doesn't solve it.
Anyway, I wanted to congratulate with you, as the theme is beautiful!
Regards!

This is a known issue, SmoothScroll does what it says, lets you scroll smoothly without it stopping all of a sudden and getting stuck while scrolling, thats the part this plugin fixes. The not being able to click part, I'm still trying to figure out how to fix that. 😂😂 This all happens because of discords lazy loading of those scrollers and lavender changing the size of those causes this.
If it is really getting annoying and unuseable for you, you can disable it completely by modifying the theme files but this will also change the appearance of the theme a little bit. To do that:
Go to your theme folder and open the main theme file of lavender (lavender.theme.css for BD or theme.scss for PC) and under the "Lists" comment delete all the lines that look like:
@import "https://lavender-discord.github.io/src/lists/any-name-here.css";

*Only delete the lines whose url match the above one. (Look at the link and see if it has /src/lists in itl
I hope that explanation was good enough 😂.
I'll close this for now, will reopen when I have a better fix for this.


Add arrow at the end of links

Hi, is it possible to add an arrow at the end of links that indicates the orientation of links?
Thanks.

Hi, for general questions, please use the mailing list. We use this list to track issues.
All links flow in the same direction; from the right-hand edge of a node to the left-hand edge of the next node. I appreciate for some cultures that is not necessary the 'natural' direction of flow, but that's what we have.


Cannot migrate without any modification

Bug description

Environment variables loaded from .env
Prisma schema loaded from schema.prisma
Datasource "db": MySQL database "..." at "..."
The migration<PHONE_NUMBER>3444_increase_url_length was modified after it was applied.
√ We need to reset the MySQL database "..." at "...".
Do you want to continue? All data will be lost. ... no

We haven't modified any migration file and it is in sync with our database.
We can't migrate our new schema. What can we do now?
We can't reset our production database. and it is absolute in sync now.
_prisma_migration

How to reproduce

Expected behavior
No response
Prisma information

Environment & setup

OS: Windows
Database: Mysql
Node.js version: 18.6

Prisma Version
Environment variables loaded from .env
prisma                  : 4.1.0
@prisma/client          : 4.1.0
Current platform        : windows
Query Engine (Node-API) : libquery-engine 8d8414deb360336e4698a65aa45a1fbaf1ce13d8 (at<EMAIL_ADDRESS>Migration Engine        : migration-engine-cli 8d8414deb360336e4698a65aa45a1fbaf1ce13d8 (at<EMAIL_ADDRESS>Introspection Engine    : introspection-core 8d8414deb360336e4698a65aa45a1fbaf1ce13d8 (at<EMAIL_ADDRESS>Format Binary           : prisma-fmt 8d8414deb360336e4698a65aa45a1fbaf1ce13d8 (at<EMAIL_ADDRESS>Default Engines Hash    : 8d8414deb360336e4698a65aa45a1fbaf1ce13d8
Studio                  : 0.467.0

ok, I can migrate now after delete the rollback record in the last migration.
It seems there is a checking issue on rollback record.

So the theory would be that migrate dev (that is the command giving you the error?) is still checking the checksum of the rolled back migration file although it is marked as "rolled back" in the database, and not the new hash from the successfully applied migration?

PS D:\project> prisma migrate dev
Environment variables loaded from .env
Prisma schema loaded from schema.prisma
Datasource "db": MySQL database "project" at "<IP_ADDRESS>:3306"
Database error code: 1062
Database error:
Duplicate entry '138-5' for key 'Article.Article_articleRawId_localeId_key'
Please check the query number 1 from the migration file.
PS D:\project> prisma migrate dev
Environment variables loaded from .env
Prisma schema loaded from schema.prisma
Datasource "db": MySQL database "project" at "<IP_ADDRESS>:3306"
The migration<PHONE_NUMBER>4950_add_article_raw_id_locale_id failed.
? We need to reset the MySQL database "project" at "<IP_ADDRESS>:3306".
Do you want to continue? All data will be lost. » (y/N)

What is the proper way to migrate? Everything time I failed the first time, after fixing the record and time to rerun the migrating. it will warn me all data will be lost, if I press "y" in accident, all data in production will be lost. IT'S TERRIBLE.

@shtse8 it sounds a bit like you are using migrate's development commands in production.
In development
use migrate dev and db push. The development db is considered "resettable" and "seedable" as its used to develop your migrations and to test/tweak them if needed. So if a migration fails in development, try to fix it and run migrate dev again (it will prompt to reset the development DB and then re-try to create/apply the migration).
In production
use migrate deploy only where Prisma will never prompt to reset the prod db. If a migration fails, use migrate diff to troubleshoot the failed migration and either move forward (recommended, use migrate resolve --applied to mark the migration to be fixed forward) or backwards (use migrate resolve --rolled-back to mark a migration to be rolled back)

@shtse8 it sounds a bit like you are using migrate's development commands in production.
In development use migrate dev and db push. The development db is considered "resettable" and "seedable" as its used to develop your migrations and to test/tweak them if needed. So if a migration fails in development, try to fix it and run migrate dev again (it will prompt to reset the development DB and then re-try to create/apply the migration).
In production use migrate deploy only where Prisma will never prompt to reset the prod db. If a migration fails, use migrate diff to troubleshoot the failed migration and either move forward (recommended, use migrate resolve --applied to mark the migration to be fixed forward) or backwards (use migrate resolve --rolled-back to mark a migration to be rolled back)

oh, thanks!!! Never knows there is a deploy option. let me try.
I think we can do better in the example when running prisma cli.
Examples

  Set up a new Prisma project
  $ prisma init

  Generate artifacts (e.g. Prisma Client)
  $ prisma generate

  Browse your data
  $ prisma studio

  Create migrations from your Prisma schema, apply them to the database, generate artifacts (e.g. Prisma Client)
  $ prisma migrate dev

  Pull the schema from an existing database, updating the Prisma schema
  $ prisma db pull

  Push the Prisma schema state to the database
  $ prisma db push

I think we can do better in the example when running prisma cli.

Yes you are completely right on that. Would you like to create an issue so we can schedule work on this? I am happy to do it otherwise.

I have a PR to change how we treat edited rolled back migrations in migrate dev — I am not convinced not resetting in this case is the right behaviour, but this is what we would do if we merge that: https://github.com/prisma/prisma-engines/pull/3105
Like Frederik said, deploy for production databases and migrate dev for development databases you can reset frequently is what Migrate is designed for.
If you reallly cannot afford to reset, a combination of migrate diff, db execute and migrate deploy — or a different migration tool in combination with db pull are better choices. We are aware this is not optimal, and we will work on this at some point, but it of course competes with other work.


ISSUE-2098 Bump Netty to 4.1.77

Make sure that:

[x] You have read the contribution guidelines.
[x] You have created a feature request first to discuss your contribution intent. Please reference the feature request ticket number in the pull request.
[x] You use the code formatters provided here and have them applied to your changes. Don’t submit any formatting related changes.
[x] You submit test cases (unit or integration tests) that back your changes.

Hi Benoit, we are looking forward to the patch release containing this fix...!

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


Removing Duplicates from LL

Removinfg Duplicates from Linked List

Changes made and new pull request made

Thanks for suggestion,  the necessary changes made

@Shalavya8103
Great work 💯
Contribution accepted 🚀
You can follow me and star this repo for further updates.
Have fun


Wallaroo cluster intermittently stalls and never finishes processing in CI tests

Integration and Integration-with-resilience tests fail sporadically on CI when the output that is supposed to trigger the test's completion never arrives at the sink.
Such an example is available at https://circleci.com/gh/WallarooLabs/wallaroo/4846.
The test is failing because not all expected output arrives at the sink.
Why not all output arrives at the sink is unknown at this point, and should be considered a high priority bug.
Unfortunately, I don't know of a reliable way to reproduce this bug quickly.

@rachelreese ping. so this is on your radar.

This has come up in https://github.com/WallarooLabs/wallaroo/pull/2297. Andy and myself are investigating, but so far no luck isolating the cause.

This turned out to be a bug in ponyc, which is addressed in https://github.com/ponylang/ponyc/pull/2840 by @dipinhora.
I'll close this issue once the bugfix is merged and our CI is updated to the new ponyc release.


Refactor with hook

Function components are already used.
useState Hook is used for set the value.
The functions are working perfectly in the browser, too.

STATUS: APPROVED 🎉✔🍾
Hello @huzaifakhan0308 👋
Your project is complete!  There is nothing else to say other than... it's time to merge it :shipit:
Congratulations! 🎉
Highlights

You documented your work professionally.✔
Your linters are passing all the checks.

Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, remember to tag me in your question so I can receive the notification.

As described in the Code reviews limits policy  you have a limited number of reviews per project (check the exact number in your Dashboard). If you think that the code review was not fair, you can request a second opinion using this form.


Install traffic-mesh via npm package

- Summary
This configures the CLI to load traffic-mesh via npm instead of via downloading it manually the first time netlify dev is run.
The package itself is quite heavy right now (40mb) because it contains the binaries for all operating systems, but we can always improve on that later by shipping a stripped package that downloads the right binaries on installation. For the MVP I wanted to avoid the additional moving parts this brings.
- Test plan
This should not change any behavior, so the existing tests should continue working.
- Description for the changelog
Install traffic-mesh component via npm package.
- A picture of a cute animal (not mandatory but encouraged)

The package itself is quite heavy right now (40mb) because it contains the binaries for all operating systems

Can anything be done to remove some bytes, such as removing debugging symbols, etc.?

but we can always improve on that later by shipping a stripped package that downloads the right binaries on installation.

The main concern with a bigger binary size is that this would add time when installing netlify-cli. Downloading on postinstall would add even more time, so I think the current solution might be best.

Can anything be done to remove some bytes, such as removing debugging symbols, etc.?

Yeah the binaries aren‘t stripped yet.

Follow-up: Stripping binaries removes any useful information from their stack traces (in case of panics), so I'd opt to leave the debug symbols in them for now and only remove those later.

Follow up 2: I'm working on leveraging optional dependencies to make npm only download the package you need for your platform.
This gives us the benefit of using npm for all the I/O and has only marginally increased complexity.
See https://github.com/netlify/traffic-mesh/pull/1217

Download size per platform is now:

macOS: 12.8MB
Linux: 16.6MB
Windows: 11.2MB

Hey @ehmicky @erezrokah, just wanted to ask your status on this PR. I'm happy to move forward with this PR and #1461, but I'm not sure about your schedule and / or whether you're currently blocked on other things. :)
I think I'll have the capacity to finish the integration next week, so up to you, whenever you have time.

This is looking ready to merge, sorry for the delay! Thanks for helping out here!
@erezrokah Is it good to merge for you too?

npm ci failed:
https://github.com/netlify/cli/runs/1341703190?check_suite_focus=true#step:4:4
Going to re-run it.


[Question] Are there new rules for attribute forwarding of Wrappers?

Question
Hi, are there new rules for attribute forwarding of Wrappers?
In previous versions  <1.0.0rc
The attributes of environments where propagated to the wrappers transparently.
Now I am experiencing many attribute errors, it seems they are not propagated any more or I am doing bad for new library specification?
If this is a NEW design structure please announce this clearly as a design option to be alerted of code breaking.
Ii there a new design documentation for how wrappers will work now ?
Thanks.

Yes, some wrappers need to be imported in a slightly different way, the changes should be included in the 1.0.0rc1 release

Thanks, a part from the importing from wrappers instead of the older routes, I am observing that they behave different with respect to env attribute access.
In the older versions since gym and gymnasium when your root env had an attribute say a Mujoco env containint the dt attribute you can access the attribute even if you top env is a wrapper of 4 layers. However now we are forced to use env.unwrapped.dt.
Is this now by design or a temporal bug ?

This is an explicit design goal to avoid a situation where a wrapper and environment shared the member variable name

Ok, so , by design and starting at version 1.0.0 we should explicitly call the attributes of root env by env.unwrapped.
Is that correct?

yes

Thanks @Kallinteris-Andreas, yes this is a by design change in v1.0.0 as otherwise users could access variables anywhere in the environment stack but importantly not modify them.
This change forces users to be more explicit with where they are accessing the variables from


Access is denied to settings folder

On install I've put my settings in Dropbox D:\Eugene\Personal\PC\Dropbox\Settings\MultiMedia\StaxRip\. Now when I open tools->Directories->Settings I get the following error
Access is denied
File: D:\Eugene\Personal\PC\Dropbox\Settings\MultiMedia\StaxRip\

tried to launch StaxRip as admin — didn't help
Window 10 with latest test build StaxRip x64 <IP_ADDRESS> beta

I only use gdrive and 1drive so I cannot test it. When .NET throws an exception something must be locked so it's not a StaxRip problem. I've added now some excepting handling to show error messages instead of crashing.

Thank you, it's fixed now (even though Chrome  tried to block your updated app this time ;)


version.py is missing in the asf folder

I just tried out to install asf version 2.0, and pip install . results in an error pointing towards the missing file.

LookupError: setuptools-scm was unable to detect version for /ess/qs/home/hahn/qc_on_qc/ActiveSpaceFinder-release/ActiveSpaceFinder-release.

Dear Walter,
I just tried cloning this repository and creating an install in a fresh conda environment with Python 3.11.
The installation worked out fine, and pip list reported version 2.0.0.
The source code does not contain asf/__version__.py; that is intentional, as the file is created automatically by setuptools_scm.

Dear Peter, thank you. I also created a "version.py" file manually and it did not help. The problem seems to be somewhere else.


CIDER version

Hello,
Are you able to advise how I can "pin" cider to the stable version please?
I tried putting (add-to-list 'package-pinned-packages '(cider . "melpa-stable") t) in my init-local.el but   I seem to have 20161007.924 installed but when I connect to a clojure repl started with lean repl I get the warning:
WARNING: CIDER's version (0.14.0-snapshot) does not match cider-nrepl's version (0.13.0). Things will break!
Thanks!
Andrew.

I think package-pinned-packages only affects installation, so you'll have to uninstall/delete the unwanted snapshot package, and reinstall the package so that you get the stable version instead.

Thanks.  I also needed to add the melpa-stable repo, and now I've got it sorted out.
Thanks for your help, and this great project as always
On Sat, Oct 8, 2016 at 4:50 PM +1100, "Steve Purcell"<EMAIL_ADDRESS>wrote:
I think package-pinned-packages only affects installation, so you'll have to uninstall/delete the unwanted snapshot package, and reinstall the package so that you get the stable version instead.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Great, glad you figured it out. Happy this config is helpful for you.


Missing link

In this section "To walk through the basic setup procedure on Azure, see Implement Oracle Data Guard on an Azure Linux virtual machine." the link to the mentioned article is missing

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

ID: 42a87992-3c2d-68cd-01b2-d41ea5e3b142
Version Independent ID: 5cf81a67-4e67-17d6-2b7c-f1ae3947088f
Content: Overview of Oracle Applications and solutions on Azure - Azure Virtual Machines
Content Source: articles/virtual-machines/workloads/oracle/oracle-overview.md
Service: virtual-machines
Sub-service: oracle
GitHub Login: @jjaygbay1
Microsoft Alias: jacobjaygbay

@cenk-ersoy
Thanks for the feedback! I have assigned the issue to content author to check and update the document as appropriate.

@jjaygbay1
Can you please check and add your comments on this doc update request as applicable.


Run JS tests

Main blocker here is loading of files.

Perhaps this should be its own issue. But I'll just comment here for now :)
I'm getting errors when trying to use zio-json in a Scala.js project.
[error] Referring to non-existent method java.lang.Class.getResourceAsStream(java.lang.String)java.io.InputStream
[error]   called from scala.util.PropertiesTrait.scalaProps()java.util.Properties
[error]   called from private scala.util.Properties$.scalaProps$lzycompute()java.util.Properties
[error]   called from scala.util.Properties$.scalaProps()java.util.Properties
[error]   called from scala.util.PropertiesTrait.scalaPropOrNone(java.lang.String)scala.Option
[error]   called from scala.util.Properties$.scalaPropOrNone(java.lang.String)scala.Option
[error]   called from scala.util.PropertiesTrait.scalaPropOrElse(java.lang.String,scala.Function0)java.lang.String
[error]   called from scala.util.Properties$.scalaPropOrElse(java.lang.String,scala.Function0)java.lang.String
[error]   called from scala.util.PropertiesTrait.$init$()void
[error]   called from constructor scala.util.Properties$.<init>()void
[error]   called from constructor zio.json.internal.Lexer$.<init>()void
[error]   called from private zio.json.JsonDecoder$.$anonfun$byte$1(scala.collection.immutable.List,zio.json.internal.RetractReader)byte
[error]   called from constructor zio.json.JsonDecoder$.<init>()void
[error]   called from private slides.Command$.$anonfun$decoder$7()zio.json.JsonDecoder
[error]   called from constructor slides.Command$.<init>()void
[error]   called from private catechism.Slides$.$anonfun$openWebSocketConnection$2(org.scalajs.dom.raw.WebSocket,slides.Command)void
[error]   called from catechism.Slides$.catechism$Slides$$$anonfun$openWebSocketConnection$1(org.scalajs.dom.raw.Event,com.raquo.airstream.eventstream.EventStream,org.scalajs.dom.raw.WebSocket)com.raquo.airstream.ownership.Subscription
[error]   called from catechism.Slides$.openWebSocketConnection(com.raquo.airstream.eventstream.EventStream,com.raquo.airstream.core.Observer)void
[error]   called from catechism.Slides$.view()com.raquo.laminar.nodes.ReactiveHtmlElement
[error]   called from catechism.App$.start()void
[error]   called from catechism.App$.$js$exported$meth$start()java.lang.Object
[error]   exported to JavaScript with @JSExport
[error] involving instantiated classes:
[error]   scala.util.Properties$
[error]   catechism.Slides$
[error]   catechism.App$

Thank you for the report, we will look into it.

Taking this.

Completed.


[macOS] "image not found" error for youtube playlist

Checklist

[x] I'm reporting a broken site support issue
[x] I've verified that I'm running youtube-dl version 2021.12.17
[x] I've checked that all provided URLs are alive and playable in a browser
[x] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[x] I've searched the bugtracker for similar bug reports including closed ones
[x] I've read bugs section in FAQ

Verbose log
$ youtube-dl -v --yes-playlist --audio-format mp3 https://www.youtube.com/playlist?list=PLgtfSiKjx3eDcbBThQPYJt8Mw1Bvs2Poa
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: ['-v', '--yes-playlist', '--audio-format', 'mp3', 'https://www.youtube.com/playlist?list=PLgtfSiKjx3eDcbBThQPYJt8Mw1Bvs2Poa']
[debug] Encodings: locale UTF-8, fs utf-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2021.12.17
[debug] Python version 3.6.9 (CPython) - Darwin-18.7.0-x86_64-i386-64bit
[debug] exe versions: ffmpeg present, ffprobe present, rtmpdump 2.4
[debug] Proxy map: {}
[youtube:tab] PLgtfSiKjx3eDcbBThQPYJt8Mw1Bvs2Poa: Downloading webpage
[download] Downloading playlist: Station Eleven Soundtrack
[youtube:tab] playlist Station Eleven Soundtrack: Downloading 33 videos
[download] Downloading video 1 of 33
[youtube] -XHqXhET5rQ: Downloading webpage
[debug] Default format spec: bestvideo+bestaudio/best
[debug] Invoking downloader on 'https://rr2---sn-hgn7rne7.googlevideo.com/videoplayback?expire=1646668732&ei=W9clYtX6O_2Ax_AP8_mLWA&ip=<IP_ADDRESS>&id=o-ADVVX_g7ui9bWRJXNPA23hgOTOBs5e0qWM1nnj--jY_B&itag=248&aitags=133%2C134%2C135%2C136%2C137%2C160%2C242%2C243%2C244%2C247%2C248%2C278&source=youtube&requiressl=yes&mh=Iv&mm=31%2C29&mn=sn-hgn7rne7%2Csn-hgn7yn7e&ms=au%2Crdu&mv=m&mvi=2&pl=22&gcr=fr&initcwndbps=1067500&vprv=1&mime=video%2Fwebm&ns=YRV2gWv0tDzT-cyevhpN2e0G&gir=yes&clen=13869095&dur=175.720&lmt=1615791144252167&mt=1646646785&fvip=1&keepalive=yes&fexp=24001373%2C24007246&c=WEB&txp=2311222&n=UvJnjN7-1YOsuPStn&sparams=expire%2Cei%2Cip%2Cid%2Caitags%2Csource%2Crequiressl%2Cgcr%2Cvprv%2Cmime%2Cns%2Cgir%2Cclen%2Cdur%2Clmt&lsparams=mh%2Cmm%2Cmn%2Cms%2Cmv%2Cmvi%2Cpl%2Cinitcwndbps&lsig=AG3C_xAwRQIhAPF8m6C9Cb4Ez7FckJnbVIhzE1sxic7IvzKeckI0nr3VAiACFVzWdjuejq1AOUZizKkDgjrIQm9h6TVNEU7G5S8t0Q%3D%3D&sig=AOq0QJ8wRAIgY-QWKM7ay49aBjQaFlQNvIAgQO_hhBcdPGKX30_ESeQCIA4q6tuFtLaoOX39jdOorX4BitufE4zCgLItqdMae-4-'
[download] Destination: Turn Around, Look At Me (Remastered 2003)--XHqXhET5rQ.f248.webm
[download] 100% of 13.23MiB in 02:47
[debug] Invoking downloader on 'https://rr2---sn-hgn7rne7.googlevideo.com/videoplayback?expire=1646668732&ei=W9clYtX6O_2Ax_AP8_mLWA&ip=<IP_ADDRESS>&id=o-ADVVX_g7ui9bWRJXNPA23hgOTOBs5e0qWM1nnj--jY_B&itag=251&source=youtube&requiressl=yes&mh=Iv&mm=31%2C29&mn=sn-hgn7rne7%2Csn-hgn7yn7e&ms=au%2Crdu&mv=m&mvi=2&pl=22&gcr=fr&initcwndbps=1067500&vprv=1&mime=audio%2Fwebm&ns=YRV2gWv0tDzT-cyevhpN2e0G&gir=yes&clen=3021703&dur=175.741&lmt=1615790775970025&mt=1646646785&fvip=1&keepalive=yes&fexp=24001373%2C24007246&c=WEB&txp=2311222&n=UvJnjN7-1YOsuPStn&sparams=expire%2Cei%2Cip%2Cid%2Citag%2Csource%2Crequiressl%2Cgcr%2Cvprv%2Cmime%2Cns%2Cgir%2Cclen%2Cdur%2Clmt&lsparams=mh%2Cmm%2Cmn%2Cms%2Cmv%2Cmvi%2Cpl%2Cinitcwndbps&lsig=AG3C_xAwRAIgQ_iGVDjME0Ae2ETMwMjRtIZzJEYctP_BMzH1-ZqrgV0CIDLh57JbRTFO6JMeaEEKc0UfrdhZWRPbWGqhYKWQJ9Xr&sig=AOq0QJ8wRAIgMfOMqFhBaDNiOA0VzDDEW9sf4kUpi-zCNChkNDvmomUCIBUX5dBCxMj0WwguxP7gwvD-4GdzvX6XwwURQwL8WHqY'
[download] Destination: Turn Around, Look At Me (Remastered 2003)--XHqXhET5rQ.f251.webm
[download] 100% of 2.88MiB in 00:36
[ffmpeg] Merging formats into "Turn Around, Look At Me (Remastered 2003)--XHqXhET5rQ.webm"
[debug] ffmpeg command line: ffmpeg -y -loglevel repeat+info -i 'file:Turn Around, Look At Me (Remastered 2003)--XHqXhET5rQ.f248.webm' -i 'file:Turn Around, Look At Me (Remastered 2003)--XHqXhET5rQ.f251.webm' -c copy -map 0:v:0 -map 1:a:0 'file:Turn Around, Look At Me (Remastered 2003)--XHqXhET5rQ.temp.webm'
[debug] dyld: Library not loaded: /usr/local/opt/libffi/lib/libffi.7.dylib
  Referenced from: /usr/local/opt/p11-kit/lib/libp11-kit.0.dylib
ERROR:   Reason: image not found
Traceback (most recent call last):
  File "/usr/local/bin/youtube-dl/youtube_dl/YoutubeDL.py", line 2115, in post_process
    files_to_delete, info = pp.run(info)
  File "/usr/local/bin/youtube-dl/youtube_dl/postprocessor/ffmpeg.py", line 526, in run
    self.run_ffmpeg_multiple_files(info['__files_to_merge'], temp_filename, args)
  File "/usr/local/bin/youtube-dl/youtube_dl/postprocessor/ffmpeg.py", line 238, in run_ffmpeg_multiple_files
    raise FFmpegPostProcessorError(msg)
youtube_dl.postprocessor.ffmpeg.FFmpegPostProcessorError:   Reason: image not found

Description
I am trying to download a playlist from youtube, which results in an "image not found" error.

ffmpeg is broken on your Mac (a shared library is missing). Reinstall it.
Or these people have had some success by creating a symlink for the missing library.

Yes, a re-install of ffmpeg seem to have fixed this problem. Thanks for the solution.


Support for custom webpack configurations

Command
I don't know which commands are affected
Description
My company uses webpack module federation (by WebpackModuleFederation plugin) for a step-by-step migration from angular-js to latest angular version.
The problem we face is that a development build is not working due to the 'ngDevMode is not defined' problem as it is already reported here: https://github.com/angular/angular-cli/issues/25830
We can workaround this by defining the ngDevMode manually:
config.plugins.push(
new webpack.DefinePlugin({
      ngDevMode: config.mode === 'development'
}));

But that is an angular internal.
I would appreciate it if the AngularWebpackPlugin would handle that directly.
Describe the solution you'd like
A rudimentary custom webpack configuration support. So that at least simple custom webpack configurations are supported.
Describe alternatives you've considered
No response

By design, custom webpack configurations are not supported. Employing such custom configurations directs you towards an undesirable path.

@alan-agius4 : If it is done by design what would be your suggestion? How you would you solve the initial problem of module federation? As far as I know this is not possible without a custom webpack configuration.

The tracking of official microfrontend support can be found at https://github.com/angular/angular-cli/issues/17556. However, it is important to note that there is currently no official support available.
Additionally, it's worth acknowledging that module federation is tied to the older build system utilizing Webpack, and it is not compatible with the new Esbuild pipeline, even with the use of custom plugins.
Nevertheless, there is an ongoing implementation by the community utilizing native import maps, which can be explored at https://www.npmjs.com/package/@angular-architects/native-federation.

ok, thank you


documentation/protocol issue with peek_ready()

peek_ready gives the first job in the using() tube, not the first job you would get from watching().
The documentation says:
To peek at the same job that would be returned by reserve -- the next ready job -- use peek-ready:
When in reality, peek_ready() returns the next job that will be consumed on the tube you are using, not watching. This completely threw me for a loop.
I wanted to double check and make sure this was intentional and confirmed before I changed the docs.

Thanks a lot for this careful attention, and my apologies for the confusion and derailment this has caused. You are absolutely correct, the quoted sentence is just plain wrong. I'm quite embarrassed that it survived for so long.
From the protocol specification:

The peek commands let the client inspect a job in the system. There are four variations [ed: peek, peek-ready, peek-delayed, peek-buried]. All but the first operate only on the currently used tube.

I'd very much appreciate if you go ahead and submit a pull request to fix this so that you are properly attributed as committer as well. If that's too much of a hassle, just let me know, and I'll push a fix with an attribution in the commit message.

I had totally forgotten about this. I'd rather you made the change yourself, if that's okay.


Uniqueness of song.id and song.cid

Asked this here but waiting on an explanation.  In the meantime we need to assume that song.id may change while cids are the way to determine uniqueness for a video.
http://support.plug.dj/hc/en-us/articles/202823766-Data-getMedia

Adjusted this to be unique based on format + cid


CICD SA needs org level role even to apply folder level org policies

https://cloud.google.com/iam/docs/understanding-roles#organization-policy-roles

CICD SA needs to be granted additional org level role when org_policies block is present.


Language server does not install using termux on android

Describe the bug
Installation occurs and works normally but language server does not work and at startup it produces the following error:
stylua: failed to install
lua-language-server: failed to install
To Reproduce
Clean install of android for ARM
Install termux app
Install neovim and kickstart dependencies
copy and paste install command
run nvim and wait for the installation to finish
exit nvim and run it again
Desktop

OS: Android REL 14 aarch64 on host samsung SM-A536E
Terminal: Termux

Neovim Version

NVIM v0.9.5
Build type: Release
LuaJIT 2.1.0-beta3

   system vimrc file: "$VIM/sysinit.vim"
  fall-back for $VIM: "
/data/data/com.termux/files/usr/share/nvim"

Run :checkhealth for more info

i'm sorry, but I'm afraid installing on android phones is not supported. Good luck and please feel free to write your solutions back here if you should find one.


RFC: Trait transformers

This is a roadmap item for wg-async. You can view the roadmap here: https://github.com/orgs/rust-lang/projects/28
As described in https://smallcultfollowing.com/babysteps/blog/2023/03/03/trait-transformers-send-bounds-part-3/

Not doing this at the moment; will revisit in the future.


Can't locate IPC/Run3.pm in @INC

t/00-spelling.t fails if IPC::Run3 is not installed:
Can't locate IPC/Run3.pm in @INC (you may need to install the IPC::Run3 module) (@INC contains: ...
BEGIN failed--compilation aborted at t/00-spelling.t line 7.
t/00-spelling.t .....
Dubious, test returned 2 (wstat 512, 0x200)
No subtests run

Fixed in a2e802b.


Quick Switch doesn't really work as advertised

Bug description:
"Select a service with TAB, ↑, ↓" doesn't work ...
seems only in apps that use those keys, e.g., Gmail, Whatsapp.
Steps to reproduce:

Go to Gmail
Type Cmd-S
Press ↑, ↓, tab
You can see stuff happening in Gmail ... not in the quick switcher.

Also, esc & enter don't work either ... only click on 'X' will close the switcher.
MacOS 10.14.6
Ferdi 5.5.0 (<IP_ADDRESS>6)

Same for me, happens on gmail, telegram, slack & other services.


Add Authenticating Users to Mobile

@ConnorLand

We are going to add a page to Designing Mobile User Interfaces called Authenticating Users, and we can explain the default process with anonymous users and link to Native SSO. We'll also rename Designing Mobile User Interfaces to something more generic.


Generating component into module has a case-sensitive inconsistency

Bug Report or Feature Request (mark with an x)

[x] bug report -> please search issues before submitting
[ ] feature request

Versions.

@angular/cli: 1.0.3
node: 6.10.3
os: linux x64
@angular/common: 4.1.2
@angular/compiler: 4.1.2
@angular/core: 4.1.2
@angular/forms: 4.1.2
@angular/http: 4.1.2
@angular/platform-browser: 4.1.2
@angular/platform-browser-dynamic: 4.1.2
@angular/router: 4.1.2
@angular/cli: 1.0.3
@angular/compiler-cli: 4.1.2
Repro steps.

Generate a module with an uppercase initial letter, e.g. Assignment3
Try to generate a component with the same name, putting it into the module

The log given by the failure.

installing module
  create src/app/assignment3/assignment3.module.ts
➜  ng-exercises git:(master) ✗ ng g component --module Assignment3 Assignment3
installing component
EEXIST: file already exists, mkdir '/home/francesco/sources/ng-exercises/src/app/assignment3'
Error: EEXIST: file already exists, mkdir '/home/francesco/sources/ng-exercises/src/app/assignment3'
    at Error (native)
    at Object.fs.mkdirSync (fs.js:923:18)
    at parts.reduce (/home/francesco/sources/ng-exercises/node_modules/@angular/cli/utilities/dynamic-path-parser.js:39:16)
    at Array.reduce (native)
    at Object.dynamicPathParser (/home/francesco/sources/ng-exercises/node_modules/@angular/cli/utilities/dynamic-path-parser.js:24:31)
    at buildFullPath (/home/francesco/sources/ng-exercises/node_modules/@angular/cli/utilities/resolve-module-file.js:34:46)
    at Object.resolveModulePath (/home/francesco/sources/ng-exercises/node_modules/@angular/cli/utilities/resolve-module-file.js:25:26)
    at Class.beforeInstall (/home/francesco/sources/ng-exercises/node_modules/@angular/cli/blueprints/component/index.js:107:39)
    at tryCatch (/home/francesco/sources/ng-exercises/node_modules/rsvp/dist/rsvp.js:539:12)
    at invokeCallback (/home/francesco/sources/ng-exercises/node_modules/rsvp/dist/rsvp.js:554:13)
➜  ng-exercises git:(master) ✗ ng g component Assignment3 --module assignment3
installing component
  create src/app/assignment3/assignment3.component.css
  create src/app/assignment3/assignment3.component.html
  create src/app/assignment3/assignment3.component.spec.ts
  create src/app/assignment3/assignment3.component.ts
  update src/app/assignment3/assignment3.module.ts

Desired functionality.

The problem is that it works fine when generating the Component using --module assignment3 but it doesn't work when trying --module Assignment3, which is inconsistent since the component was generated usingng  module Assignment3
Mention any other details that might be useful.

I guess the log result is pretty self-describing, no additional details needed.

@ShinDarth Is It happening on a new project? I tried to replicate same on a brand new project and seems to be working fine. Which operating system are you on?

yes it's a new project, my operating system is Ubuntu 16.04 - 64 bit


Duplicate text/html MIME type with nginx playbook

The Nginx v1.10.0 deployment includes an /etc/nginx/mime.types with the "text/html" MIME type declared within it.

The provided /etc/nginx/nginx.conf duplicates this on line 30 in the gzip_types declaration, leading to the following warning.

    [warn] 30382#30382: duplicate MIME type "text/html" in /etc/nginx/nginx.conf:30

Removing "text/html" from the gzip_types declaration permits Nginx to start without any warnings.

Kind Regards,
-Camisa

According to nginx documentation (https://www.nginx.com/resources/admin-guide/compression-and-decompression/) explicitly setting text/html is not needed for gzip, so it can be removed without change to functionality. Should have a pull shortly.

Fixed in above PR, please reopen issue if any related issues found


Updates to the FAQ for improved A2HS based on feedback

On line 172, instead of wrapping "FAQs below" in bold, please make it a link to the FAQ section.

On line 177, please remove the - - - as it's automatically added by the ## FAQs


Check in parent to see if they stayed - for collecting data - grants

I don't know where we would do this? Would it be on the admin system when they check in? Would it be something the lead teacher does after the class is done?

This issue was moved to CoderDojoChi/c2#67


How to pause/play animation??? Is there any way to do it?

Please check https://github.com/nithinpp69/react-native-circular-progress-indicator#play-pause-and-reanimate. This is available from V4.4.0.


Advanced elem factory

Hoist P/R Checklist
Pull request authors: Review and check off the below. Items that do not apply can also be
checked off to indicate they have been considered. If unclear if a step is relevant, please leave
unchecked and note in comments.

[ ] Caught up with develop branch as of last change.
[ ] Added CHANGELOG entry, or determined not required.
[ ] Reviewed for breaking changes, added breaking-change label + CHANGELOG if so.
[ ] Updated doc comments / prop-types, or determined not required.
[ ] Reviewed and tested on Mobile, or determined not required.
[ ] Created Toolbox branch / PR, or determined not required.

If your change is still a WIP, please use the "Create draft pull request" option in the split
button below to indicate it is not ready yet for a final review.

Pull request reviewers: when merging this P/R, please consider using a squash commit to
collapse multiple intermediate commits into a single commit representing the overall feature
change. This helps keep the commit log clean and easy to scan across releases. PRs containing a
single commit should be rebased when possible.

Tom reviewed this -- there are still a number of API related issues we need to review before going to develop here, but Tom and I agreed that the main change here -- meaningful validation of props -- is absolutely critical.  Merging this to keep the whole thing moving forward.
We will review all critical issues before going to develop.


Update alt text on jupyter.org

This PR updates existing alt text to Project Jupyter documentation. It includes alt text for

all images on the site except for those used on index.html (for no reason other than us running out of time)
renaming images that are interactive (like links) based on what the interaction does (more about functional image alt text on W3C)

If there are other images in this repo you think were missed, I'd love to know so we can address them in the future along with index.html's images.
If you have any questions about reviewing alt text, let me know and I'd be happy to tell you what I look for.
Thanks in advance for reviewing this PR! 🌻

This alt text was contributed by wonderful volunteers at one of the Jupyter accessibility workshops! They teamed up to make progress on so many images.
If you are curious how we have a commit authored by five people, feel free to check out our working PR or the How to Participate section of the event agenda.

The failing audit is definitely not related to this PR, so I am going to go ahead and merge it (the performance is hovering at 89 right now and our limit is 90)


adding a new language

Hi, is it possible to add support for a new language as slovenian/croatian/serbian ?
Do you have any procedure we can follow to train the model for that languages ?

Hey @tculjaga, it's possible but there's no guide yet! I don't have the bandwidth for the next few weeks though

Would you mind sharing some pointers? I don't mind taking a stab at it. I just successfully fine-tuned MMS for some Western African languages and hoping to build off of that.

Hi @ylacombe thanks for a quick response. I'd like to give it a try... all we need is a small and short guide (a bullet list is enough for start) how you do it on a specific language then we can try our best to battle through it :)

Would you mind sharing some pointers? I don't mind taking a stab at it. I just successfully fine-tuned MMS for some Western African languages and hoping to build off of that.

Hello, we're actually trying to train parler-tts for Indian languages and our pilot test gave great results. We followed exactly the same training process, and only changed to text encoder to mT5. It worked well!

Hey @SherryS997,
thanks for giving this feedback. I actually trained a version of Mini v1 which uses mT5 instead of T5, but it's only in English.
Do you think you'd be able to either open-source your model, or to share some snippet?
Thanks!

Hi @ylacombe,
Yes, we plan to open-source our model, along with the dataset, code, and captions, in the next month or two once we’re fully satisfied with the results. This work is part of the TTS research at AI4Bharat, and the model will be designed to support all 23 official languages of India, including English. We're also experimenting with various English accents and are optimistic that the model will handle those effectively as well.
In our pilot training, as stated before, we used mT5, which gave excellent results in terms of performance. However, we are transitioning to ai4bharat/IndicBERTv2-SS for the text encoder and tokenizer because mT5 only covers 12 of the 23 languages we aim to support. IndicBERTv2-SS offers broader language support, making it better suited to our goals.

Hey @SherryS997, let's speak over mail if that's okay with you: yoach [at] huggingface.co

Hi @ylacombe and @SherryS997,
I'm also very interested in training for a low resource language which is not supported by any tokenizer. I'd really appreciate it if you could share with me some advice as well.
Thanks!

Hi @ylacombe, Yes, we plan to open-source our model, along with the dataset, code, and captions, in the next month or two once we’re fully satisfied with the results. This work is part of the TTS research at AI4Bharat, and the model will be designed to support all 23 official languages of India, including English. We're also experimenting with various English accents and are optimistic that the model will handle those effectively as well. In our pilot training, we used mT5, which delivered excellent results. However, we are now experimenting with other tokenizers, as mT5 supports only 12 of the 23 languages we’re targeting. These alternatives will help us achieve broader language coverage for the project.

Great job. Looking forward to your open-source release.

Hi @ylacombe and @SherryS997, I'm also very interested in training for a low resource language which is not supported by any tokenizer. I'd really appreciate it if you could share with me some advice as well. Thanks!

You may need to build a tokenizer or better yet extend the flanT5 tokenizer. Then all you need to do is point to this tokenzer is the start json config.

Hey @SherryS997, I've been trying to fine-tune the parler-tts model for the NPTEL TTS dataset (basically Indian English). The loss doesn't seem to reduce wandb link. I have tried training longer (>12K) steps as well with no luck. How do I debug the issue? Do you think modifying the tokeniser would help?

If it's just English, no need to change anything wrt the tokenizer. I'll suggest using the IndicTTS dataset for Indian English. It is a high quality TTS dataset unlike the NPTEL dataset. I'd also recommend naming a few speakers as named speakers seem to be the best at the moment in capturing accents, styles, voice quality, etc.

@ylacombe Just curious: How do I train parler for the arabic language?


[CodeCompletion] Supress unavailable unresolved member completions

What's in this pull request?
Removes completions for unavailable/inaccessible enum cases and option set values from unresolved member completions.  Also marks deprecated completions as "not recommended".
Resolved bug number: (SR-)
rdar://problem/26335424

Before merging this pull request to apple/swift repository:

[ ] Test pull request on Swift continuous integration.

Triggering Swift CI
The swift-ci is triggered by writing a comment on this PR addressed to the GitHub user @swift-ci. Different tests will run depending on the specific comment that you use. The currently available comments are:
Smoke Testing
    Platform     | Comment
    ------------ | -------------
    All supported platforms     | @swift-ci Please smoke test
    OS X platform               | @swift-ci Please smoke test OS X platform
    Linux platform              | @swift-ci Please smoke test Linux platform

Validation Testing
    Platform     | Comment
    ------------ | -------------
    All supported platforms     | @swift-ci Please test
    OS X platform               | @swift-ci Please test OS X platform
    Linux platform              | @swift-ci Please test Linux platform

Note: Only members of the Apple organization can trigger swift-ci.

@swift-ci Please test and merge

@swift-ci Please test and merge

@swift-ci Please test and merge

Unrelated failures...

@swift-ci Please test and merge

Still unrelated failures.  Once more with feeling!

@swift-ci Please test and merge


Limit reserving to during restaurant open hours

The schedules endpoint returns information for business hours. I think think this might be related to when the restaurant services mealpal orders:
The jsonpath is:
$.schedules[].restaurant.open
$.schedules[].restaurant.mpn_open

https://github.com/edmundmok/mealpy/blob/3d10eb6be23e73cd4ad85429574b78274ec83dbb/tests/mealpy_test.py#L223-L226
This can be useful to prevent user from reserving during closed times. Also can be added to list restaurants command output, so the user will have information to set up valid argument inputs.

This will require investigation to verify this hypothesis, which will require an user with active subscription.


Refactor artifact cleanup

This pull request removes the artifact-cleanup.yml Workflow from all managed providers.
Instead, we will use the retention-days feature of the upload-artifact Action to have artifacts expire after 30 days.
Previously, we ran the artifact cleanup Workflow on a cron that would remove all artifacts older than 30 days.
With this change, we are setting artifacts to expire after 30 days instead of the default 90 days that upload-artifact provides.
This is a very likely fix for https://github.com/pulumi/pulumi-azure-native/issues/2558 and https://github.com/pulumi/pulumi-kubernetes/issues/2499, et al, since the Action we used in artifact-cleanup.yml caused a secondary rate limit error.
Related: https://github.com/pulumi/pulumi-aws/pull/2649.
h/t to @aq17 for finding this neat feature of upload-artifact :heart:

Remove artifact cleanup cron and set retention-days to 30 on Upload Artifact job itself
Regenerate tf providers
Native providers: Remove artifact cleanup workflow and add retention-days to upload-artifacts job
Generate native provider CI

for automation:
Fixes https://github.com/pulumi/pulumi-azure-native/issues/2558
Fixes https://github.com/pulumi/pulumi-kubernetes/issues/2499

@iwahbe based on feedback from @thomas11 I removed the retention days form any workflows triggered by tags. We will keep release and prerelease artifacts for the default 90 days.

@iwahbe based on feedback from @thomas11 I removed the retention days from any workflows triggered by tags. We will keep release and prerelease artifacts for the default 90 days.

That doesn't sound right to me. We are able to access release artifacts that are older than 90 days. This is the pulumi-aws v5.18.0, released mid-october of 2022. All the released artifacts are accessible.
The artifacts as stored as part of the release action are inaccessible, but that doesn't matter. We don't expect users to consume artifacts from of the release action, just the actual release.
I think a 30 day retention period is fine. There isn't a clear reason to special case these workflow artifacts compared to other workflow artifacts.
CC @thomas11 @mikhailshilkov


Add terraform-cloudflare-maintenance

Terraform module to create a responsive Maintenance Page using Cloudflare Workers.

Very cool!


Integrating with Apollo Server

Hi,
I'd like to integrate ReactQL Koa Server with an Apollo Server.
I've tried to setup a middleware ad described in the server documentation just before the router middleware in the /src/runner/app.ts file, but that does not work.
Any ideas?
Thank you

@ghiblin - please post some code. Hard to diagnose without it.


BUG!!!

Happened when I just added a chain (clicked on the chain button in the UI, then clicked on some build tasks tasks (1st, 2st, 3st, crashed on 4st)
NullReferenceException: Object reference not set to an instance of an object
at ChainErrand.ChainedErrandPacks.DiggablePack.GetChoreFromErrand (Diggable errand) [0x00000] in <3c925cda528e4e90aa8113b34f9bda07>:0
at ChainErrand.ChainedErrandPacks.AChainedErrandPack2[ErrandType,ChainedErrandType].GetChoreFromErrand (System.Object errand) [0x00000] in <3c925cda528e4e90aa8113b34f9bda07>:0    at ChainErrand.ChainHierarchy.ChainedErrand.ConfigureChorePrecondition (Chore chore) [0x0000f] in <3c925cda528e4e90aa8113b34f9bda07>:0    at ChainErrand.ChainHierarchy.Chain.CreateOrExpandLink (System.Int32 linkNumber, System.Boolean insertNewLink, System.Collections.Generic.Dictionary2[TKey,TValue] linkErrands, System.Boolean forceInsertAtLinkNumber) [0x00152] in <3c925cda528e4e90aa8113b34f9bda07>:0
at ChainErrand.ChainToolUtils.CreateOrExpandLink (System.Collections.Generic.Dictionary`2[TKey,TValue] linkErrands) [0x0003b] in <3c925cda528e4e90aa8113b34f9bda07>:0
at ChainErrand.ChainTool.OnDragComplete (UnityEngine.Vector3 cursorDown, UnityEngine.Vector3 cursorUp) [0x00147] in <3c925cda528e4e90aa8113b34f9bda07>:0
at DragTool.OnLeftClickUp (UnityEngine.Vector3 cursor_pos) [0x00175] in <93e6d51e7c504f4483a782aaaf7248c3>:0
at PlayerController.OnKeyUp (KButtonEvent e) [0x000ab] in <93e6d51e7c504f4483a782aaaf7248c3>:0
at KInputHandler.HandleKeyUp (KButtonEvent e) [0x0001c] in <0a61a0e7beaf44b6b425814476821935>:0
at KInputHandler.HandleKeyUp (KButtonEvent e) [0x0006f] in <0a61a0e7beaf44b6b425814476821935>:0
at KInputHandler.HandleEvent (KInputEvent e) [0x0001e] in <0a61a0e7beaf44b6b425814476821935>:0
at KInputController.Dispatch () [0x0001c] in <0a61a0e7beaf44b6b425814476821935>:0
at KInputManager.Dispatch () [0x00017] in <0a61a0e7beaf44b6b425814476821935>:0
at KInputManager.Update () [0x0002e] in <0a61a0e7beaf44b6b425814476821935>:0
at GameInputManager.Update () [0x000dc] in <0a61a0e7beaf44b6b425814476821935>:0
at Global.Update () [0x0005c] in <93e6d51e7c504f4483a782aaaf7248c3>:0

Could you please send a screenshot of the errand that causes the crash when added to a chain? Also, this is just the crash error message, not the full log. I would like to have the whole Player.log file which is located at .../User/AppData/LocalLow/Klei/OxygenNotIncluded/Player.log

My guees is that the bug occurs when I click on an unreachable task. In the first screenshot, this is the case, and the game crashes. In the second screenshot, I can add the digging task to the chain without issues.
Crash:

Works:

Crash player.log:
Player.log

Fixed in v<IP_ADDRESS>.


Peer review

Repository scan
Does the repo have an apt description?
Yes, the readme is well documentated, maybe it's an idea to push your sketches to your wiki once you've progressed more.
Is there a link to the live demo at the top?
No
Are any files in the repo that shouldn't be there like an .env file or node_modules?
No, the repo contains a nice gitignore file
Is there a section about the concept, install notes, data life cycle, external API, and real-time events?
Yes, this part is very well explained, especialy the api.
Understanding the concept
Read about the concept in the readme, do you understand what the app does?
I might be biased, because you've already explained the concept to me, but the concept is very clear and well needed
Data life cycle
Are all the app's components present in a visual diagram?
Yes, it's very visible and accessible. Maybe you can make the white text a little bigger but it's clear to me where the data is going
Is it clear what data is stored for each component?
Not in the visual representation, but the explanation covers everything pretty well. The image should just give you a global idea imo.
Is it clear how components communicate with each other? (socket, polling, OAuth)
Yes, aldo I do wonder if it would be a better idea to seperate your server and socket.
Can you follow where real-time data is created?
Sorta, the socket.io doesn't really stand out.
Real-time events
Are all custom events described?
No, but the explanation tells me what he's using (but I don't really know what for).
API information
Is it clear what data the external API offers?
Couldn't be more clear
Are instructions present on how to get a key from the API and how to connect to it?
Not in great detail, but the readme does give a endpoint and a documentation link (that should be enough)
Conclusion

The application is very cool! I wish I could see it, but there is no link yet :(
Maybe seperate your server and socket to get into the data a little bit more.
The code is obviously well written, wouldn't expect otherwise.
Add some things in your readme or wiki where you explain what you're doing with the data

Thanks Inju. I updated my readme with more information based on your feedback.


Refactoring for v0.1.0

Closes #25

Codecov Report

Merging #30 into main will increase coverage by 0.59%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main      #30      +/-   ##
==========================================
+ Coverage   87.43%   88.03%   +0.59%
==========================================
  Files           6        7       +1
  Lines         605      635      +30
==========================================
+ Hits          529      559      +30
  Misses         52       52
  Partials       24       24

Impacted Files
Coverage Δ

honeycombio/data_source_query.go
92.64% <100.00%> (ø)

honeycombio/data_source_trigger_recipient.go
93.75% <100.00%> (ø)

honeycombio/resource_board.go
78.63% <100.00%> (-0.71%)
:arrow_down:

honeycombio/resource_marker.go
87.27% <100.00%> (ø)

honeycombio/resource_trigger.go
82.30% <100.00%> (-0.27%)
:arrow_down:

honeycombio/type_helpers.go
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

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

Since this is dragging on, I'll be merging this already. Remaining work in #25 can be done with a new PR.
Tests fail because GitHub Actions still uses Terraform v0.13.0 while v0.13.1 is available. v0.13.0 has a regression which breaks the terraform version output.


Show/hide Horizontal grid lines: scaleShowHorizontalLines option doesn't work

When the scaleShowHorizontalLines option is set to false, the horizontal grid lines still show

Raised PR #9 to fix this


[Crash] CyanosLootableBodies-1.2.0-backport (Forge <IP_ADDRESS>6 + Forge <IP_ADDRESS>4)

Hello!    I already made a post about this over at Thermal Expansion so I hope you don't mind if I quote myself from there as well as some of that developer's remarks.
Previous Post:
Hopefully I've just done something stupid here and we can all laugh at me and move along. http://pastebin.com/wm3z4bBf should give you a list of the mods I've put together and http://pastebin.com/z2ew0ree should be the crash log. The list of mods is from me typing what appeared on the list in the "Mods" menu in my client. If there is a better/more readable method of displaying what I have installed then I will try to supply you with that. On to the issue! I got a crash apparently trying to render an energy cell inside of a lootable body container (inventory) from Dr. Cyano's Lootable Bodies mod. The log screamed at me about Thermal Expansion so I thought to try here first but if you think the issue is with the lootable bodies mod then I will post there as well. When I put a cell inside of a standard inventory (e.g. a vanilla chest) and attempt to access it I get no crash, it seems to just be when accessing the lootable body created after killing myself. I was getting some funny render glitches outside of the body gui though when i had the Cell in my hand. The included screenshot shows an example of this.

The effect stopped as soon as I switched to another slot in my hotbar. That screenshot was taken using Forge <IP_ADDRESS>6, after upgrading to <IP_ADDRESS>4 I have been able to reproduce the crash consistently, but not the weird rendering bug from the screenshot. Hope you can make more sense of this than I can! I look forward to hearing from you soon, thanks. :D
...
The cell is the only thing I have seen this happen with, as you can see in the screenshot i tried it with a few other blocks in hand and nothing bad happened. I did discover however that the weird effects from the screenshot in my first post are reproducible by dying with the item in hand. These tests were done in a new world just to make sure it wasn't about the save file.

:End Previous Post
Skyboy's Response:
The render issue is caused by lootable bodies not properly pushing all the necessary GL states before letting items render.
The metadata issue appears to be that lootable bodies is re-creating the item incorrectly and using the display damage value instead of the actual damage value (which is easily avoided by using the item it has (there's even a .copy() method if it needs copied) instead of attempting to re-create it in a wholly incorrect manner).
:End Skyboy's Response
I hope all this text isn't a bother, I just wanted you to have most of the info in one place to start with.  Here is a link to the original issue if you would like to read the whole thing https://github.com/CoFH/Feedback/issues/11.  I hope to hear back from you soon as I love your mod and would hate to not include it in my pack because of this issue.  Thank you so much for your time!

I'm looking into it now. Minecraft 1.8 does rendering differently and
has defferent methods for comparing items than Minecraft 1.7.10, which
has caused most of the bugs so far in the backport, so I suspect this is
another such bug.
-DrCyano
On 01/19/2015 01:09 PM, Carbon-Key wrote:

Hello!  I already made a post about this over at Thermal Expansion so I
hope you don't mind if I quote myself from there as well as some of
that developer's remarks.
Previous Post:
Hopefully I've just done something stupid here and we can all laugh at
me and move along. http://pastebin.com/wm3z4bBf should give you a list
of the mods I've put together and http://pastebin.com/z2ew0ree should
be the crash log. The list of mods is from me typing what appeared on
the list in the "Mods" menu in my client. If there is a better/more
readable method of displaying what I have installed then I will try to
supply you with that. On to the issue! I got a crash apparently trying
to render an energy cell inside of a lootable body container
(inventory) from Dr. Cyano's Lootable Bodies mod. The log screamed at
me about Thermal Expansion so I thought to try here first but if you
think the issue is with the lootable bodies mod then I will post there
as well. When I put a cell inside of a standard inventory (e.g. a
vanilla chest) and attempt to access it I get no crash, it seems to
just be when ac cessing the lootable body created after killing
myself. I was getting some funny render glitches outside of the body
gui though when i had the Cell in my hand. The included screenshot
shows an example of this.
2015-01-10_15 51 40
https://cloud.githubusercontent.com/assets/10481639/5805266/634b897a-9fdb-11e4-8df2-8674f43ddc95.png
The effect stopped as soon as I switched to another slot in my hotbar.
That screenshot was taken using Forge <IP_ADDRESS>6, after upgrading to
<IP_ADDRESS>4 I have been able to reproduce the crash consistently, but
not the weird rendering bug from the screenshot. Hope you can make
more sense of this than I can! I look forward to hearing from you
soon, thanks. :D
...
The cell is the only thing I have seen this happen with, as you can
see in the screenshot i tried it with a few other blocks in hand and
nothing bad happened. I did discover however that the weird effects
from the screenshot in my first post are reproducible by dying with
the item in hand. These tests were done in a new world just to make
sure it wasn't about the save file.
2015-01-10_23 10 21
https://cloud.githubusercontent.com/assets/10481639/5805298/94fd8b8a-9fdb-11e4-98a7-051caa399c02.png
:End Previous Post
Skyboy's Response:
The render issue is caused by lootable bodies not properly pushing all
the necessary GL states before letting items render.
The metadata issue appears to be that lootable bodies is re-creating
the item incorrectly and using the display damage value instead of the
actual damage value (which is easily avoided by using the item it has
(there's even a .copy() method if it needs copied) instead of
attempting to re-create it in a wholly incorrect manner).
:End Skyboy's Response
I hope all this text isn't a bother, I just wanted you to have most of
the info in one place to start with. Here is a link to the original
issue if you would like to read the whole thing CoFH/Feedback#11
https://github.com/CoFH/Feedback/issues/11. I hope to hear back from
you soon as I love your mod and would hate to not include it in my
pack because of this issue. Thank you so much for your time!
—
Reply to this email directly or view it on GitHub
https://github.com/cyanobacterium/DrCyanos_Lootable-Bodies/issues/5.

It looks like, based on the issue thread, that the problem is that the
energy cell reports itself to be damageable, but uses the damage value
for metadata. The solution, for now, is to upgrade to Lootable Bodies
version 1.3.0 and set item_damage_on_death=0 in the config file.
-DrCyano
On 01/19/2015 01:09 PM, Carbon-Key wrote:

Hello!  I already made a post about this over at Thermal Expansion so I
hope you don't mind if I quote myself from there as well as some of
that developer's remarks.
Previous Post:
Hopefully I've just done something stupid here and we can all laugh at
me and move along. http://pastebin.com/wm3z4bBf should give you a list
of the mods I've put together and http://pastebin.com/z2ew0ree should
be the crash log. The list of mods is from me typing what appeared on
the list in the "Mods" menu in my client. If there is a better/more
readable method of displaying what I have installed then I will try to
supply you with that. On to the issue! I got a crash apparently trying
to render an energy cell inside of a lootable body container
(inventory) from Dr. Cyano's Lootable Bodies mod. The log screamed at
me about Thermal Expansion so I thought to try here first but if you
think the issue is with the lootable bodies mod then I will post there
as well. When I put a cell inside of a standard inventory (e.g. a
vanilla chest) and attempt to access it I get no crash, it seems to
just be when ac cessing the lootable body created after killing
myself. I was getting some funny render glitches outside of the body
gui though when i had the Cell in my hand. The included screenshot
shows an example of this.
2015-01-10_15 51 40
https://cloud.githubusercontent.com/assets/10481639/5805266/634b897a-9fdb-11e4-8df2-8674f43ddc95.png
The effect stopped as soon as I switched to another slot in my hotbar.
That screenshot was taken using Forge <IP_ADDRESS>6, after upgrading to
<IP_ADDRESS>4 I have been able to reproduce the crash consistently, but
not the weird rendering bug from the screenshot. Hope you can make
more sense of this than I can! I look forward to hearing from you
soon, thanks. :D
...
The cell is the only thing I have seen this happen with, as you can
see in the screenshot i tried it with a few other blocks in hand and
nothing bad happened. I did discover however that the weird effects
from the screenshot in my first post are reproducible by dying with
the item in hand. These tests were done in a new world just to make
sure it wasn't about the save file.
2015-01-10_23 10 21
https://cloud.githubusercontent.com/assets/10481639/5805298/94fd8b8a-9fdb-11e4-98a7-051caa399c02.png
:End Previous Post
Skyboy's Response:
The render issue is caused by lootable bodies not properly pushing all
the necessary GL states before letting items render.
The metadata issue appears to be that lootable bodies is re-creating
the item incorrectly and using the display damage value instead of the
actual damage value (which is easily avoided by using the item it has
(there's even a .copy() method if it needs copied) instead of
attempting to re-create it in a wholly incorrect manner).
:End Skyboy's Response
I hope all this text isn't a bother, I just wanted you to have most of
the info in one place to start with. Here is a link to the original
issue if you would like to read the whole thing CoFH/Feedback#11
https://github.com/CoFH/Feedback/issues/11. I hope to hear back from
you soon as I love your mod and would hate to not include it in my
pack because of this issue. Thank you so much for your time!
—
Reply to this email directly or view it on GitHub
https://github.com/cyanobacterium/DrCyanos_Lootable-Bodies/issues/5.

Thanks for your help!  I changed the config and I'm having no issues now. :D


Unknown client when attempting to sign in

Hello,
I have two questions. I'm trying to sign in, here's my code:
        let credentialsKey = "????"
        let clientId = "from developer.tidal.com"
        let clientSecret = "from developer.tidal.com"

        let authConfig = AuthConfig(
            clientId: clientId,
            clientSecret: clientSecret,
            credentialsKey: credentialsKey
        )

        let auth = TidalAuth.shared

        let loginConfig = LoginConfig()
        let redirectUri = "https://juliette.page/tidal"

        guard let loginURL = auth.initializeLogin(redirectUri: redirectUri, loginConfig: loginConfig) else {
            print("Something is wrong")
            return
        }
        print(loginURL)

        TidalAuth.shared.config(config: authConfig)
        url = loginURL

My first question is where do I get my credentials key? I've looked around in the Tidal developer dashboard and can't find anything and the documentation isn't specific on what to put. Along with that when I attempt to sign in I get the error "Unknown client: waves" printed in the console. I have "waves" as the client name in the Tidal developer dashboard. Is there anything I can do or am I not supposed to be using this SDK?
Thank you have a good night!

Hi @juliettebee, thank you for your question!
Please try to call TidalAuth.shared.config(config: authConfig) before initializing login.
As for credentialsKey - it is just the key used to store credentials in the system keychain. In our example we use storage, but feel free to pick any unique name you like.
Let us know if it helps!

Thank you so much! Got it working!


Updating featured challenges/showcases

Updating homepage 11/7 and eliminating filler in showcase jsons.

Thanks for catching these!


Fix rest-generator instructions

When only adding the plugin and calling it directly, maven complains about the rest-generator dependency missing. Running install first fixes this problem.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Closed for now as not reproducible. We can reopen it if we find out today that the error occurs again.


option to change the port

Is it possible to add a parameter to use another port than 8080 in serve-mode?

Yes, you can by using --port or -p flag. For more details, you can run shiori serve -h :
$ shiori serve -h
Run a simple annd performant web server which serves the site for managing bookmarks.
If --port flag is not used, it will use port 8080 by default.

Usage:
  shiori serve [flags]

Flags:
  -h, --help       help for serve
  -p, --port int   Port that used by server (default 8080)

d'oh, now I feel stupid, thanks a lot for the info I could have found myself ;-)


[incubator][VC] remove endpoints syncer

endpoints will auto managed by super master endpoints controller.
Besides, kube-proxy works well with native endpoints generated by endpoints controller locate on super master.
Signed-off-by: zhuangqh<EMAIL_ADDRESS>

/lgtm
/approve


Remove duplicate functions

Signed-off-by: feihujiang<EMAIL_ADDRESS>

The duplication is intentional, as we are attempting to separate core and validator code.  In particular, core should not rely on anything in the validator package, but the reverse is also true.  I think this is mostly the case currently with the exception of the CLI utility for creating the genesis block.

@vaporos You mean  the validator package should not rely on anything in the core package either?
I don't think the duplication is a good idea. Maybe we need to put them into an separate package. then the validator and core packages rely on the separate package.


An example with Transformers to generate text + images

Hi,
I see very interesting the examples detailed for interacting with images with Transformers for VQA etc. However, how can we really generate text + images (with the right history context) with HF transformers?
I cannot see an example with this good feature of your work.
Thanks.

Hi, thanks for your attention. As we showed in Fig.3 in our paper(https://arxiv.org/pdf/2309.15112v2.pdf), the text-image interleaved generation is realized by multiple steps. The model first generates the text, then generates proper positions and corresponding captions for image injection. After we get candidate images by retrieval method, the model selects the best one based on the context text and images. For more details, please refer to our paper. For the code, please see our online demo(https://github.com/InternLM/InternLM-XComposer/blob/main/demo_asset/demo.md).


Added second classes for tracker and keyword so they are easier to find

Added a secondary, static class for the tracker and keyword so they can be found without determining the current user settings.

Very nice! LGTM


ViewSettingsCustomItem customControl content does not bind to any model

OpenUI5 version: 2.3.0
Browser/version (+device/version): Google Chrome  51.0.2704.103 m (64-bit)
Any other tested browsers/devices(OK/FAIL): FAIL
URL (minimal example if possible): http://jsbin.com/hakejo/edit?html,output
Steps to reproduce the problem:

Open Dialog by clicking Button
Click on Custom Filter named "My Custom Filter"

What is the expected result?
PlaceHolder for input, under the "My Custom Filter"  should display "Filter PlaceHolder"
What happens instead?
No placeholder is visible..
Any other information?
This issue occurs to on this one page sample but also when I have a fulll application running.
No model (e.g. i18n or an Odata model) is available under that "aggregation", customControl

Hi,
the sap.m.ViewSettingsDialog internally creates a sap.m.Dialog control, which is the thing that is actually displayed. You are setting the Model on the sap.m.ViewSettingsDialog, but the internal Dialog does not have any connection to the ViewSettingsDialog, data-binding-wise, so it does not get the Model.
I would say this is a bug in the ViewSettingsDialog. In similar situations addDependent() is used, and indeed the line
oViewSettings.addDependent(oViewSettings._getDialog());
would solve the problem.
I opened internal ticket<PHONE_NUMBER> for this issue.
Thanks for reporting!
Andreas

Hi Andreas,
I didn't really get why sap.m.ViewSettingsDialog is not directly inheriting from sap.m.Dialog? This would give the developer a lot more control.
BR
Chris


Allow setting custom volume labels for configmap and secret volumes

What this PR does / why we need it:
Allows supporting different bootstrap solutions without having explicit support for them. For instance right now we don't have a special kickstart volume.
With this change people can just use configmaps and secrets right now without explicit support of the specific bootstrap mechanism.
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #3284
Special notes for your reviewer:
Release note:
Allow setting custom volume labels for configmap and secret volumes

/override ci/prow/images


Mode to sort leaderboard by percentage and time

Doesn't need to be the default but would be nice as an option IMO :)

I.e. turning this...
...into this

Probably done in 91292be0546c43ad9429308ff9e6c1a4335943ba

Works great, ty :)
Do we want to do anything with the time bars as well or leave them as is?

The sort function for time was erroring, oops. Should be fixed in f3ed02b839a6815913c6d41d7518ebfe94847561

🎉

Do we want all graphs to be sorted like this or just the overall ones? Eg:

Feature and edition graphs
File/folder graphs

looks like it automatically applied to all of them, which seems fine to me.


Add TeamsCall and TeamsCallAgent to stateful

Update calling beta version
Add new beta feature flag "teams-call"
Extract common logic CallAgentCommon and CallCommon

What

Why

How Tested

Process & policy checklist

[ ] I have updated the project documentation to reflect my changes if necessary.
[ ] I have read the CONTRIBUTING documentation.

Is this a breaking change?

[ ] This change causes current functionality to break.

Actual PR review pending splitting off of the SDK version bump into its own PR.

Are we able to splti this PR into two -- one to just bump the calling SDK version and then one to add upport for TeamsCall?
This PR is fairly large and complex, so any ways you can think of to take smaller steps would also be good.

The bump one has been done by some other PRs (not from me), so currently it is 1.8.0-beta.1, I did merge it into this PR, so there is no bumpping needed anymore


Bug-report project does not run out-of-the-box

Package
Other
Package Version
newest
Laravel Version
^11.9
Livewire Version
No response
PHP Version
^8.2
Problem description
When creating a bug report on Github (https://github.com/filamentphp/filament/issues/new), I'm given the following example project to base my reproduction on: https://filament-issue.unitedbycode.com/
This project does not work out-of-the-box even though the page states that it does and it could also be more user-friendly.
Expected behavior
A list of things I ran into while setting-up the project:

The page states that the only requirement is to run php artisan serve. That is not the case. I also had to run:

php artisan key:generate
composer install?
php artisan migrate
php artisan db:seed or something else to create a user.

The composer.json doesn't contain filament as a dependency, even though it's present in the vendor directory.
The user that is pre-filled in the login form (App\Filament\Pages\Auth) does not match the seeded user.
App\Providers\Filament\AdminPanelProvider isn't registered.
http://<IP_ADDRESS>:8000/ shows the default Laravel app. This could redirect to the login form so I don't have to search for routes (which I couldn't find using php artisan route:list because they weren't registered).
The README.md is the default Laravel README so it didn't help me with any of the above.

Steps to reproduce

Create a bug report on https://github.com/filamentphp/filament/issues/new/choose.
Click the link under "Reproduction repository" (https://filament-issue.unitedbycode.com/).
Download and unzip the project.
Run the project using php artisan serve.
Go to http://<IP_ADDRESS>:8000/
See that it doesn't work out-of-the-box.

Reproduction repository (issue will be closed if this is not valid)
https://github.com/MarkKremer/filament-pr-2
Relevant log output
No response

Donate 💰 to fund this issue

You can donate funding to this issue. We receive the money once the issue is completed & confirmed by you.
100% of the funding will be distributed between the Filament core team to run all aspects of the project.
Thank you in advance for helping us make maintenance sustainable!

cc @ijpatricio

Thanks for the heads up, I'll fix it and report back here.


API for covid warriors data.

Is your feature request related to a problem? Please describe.
Govt of India has released data for corona warriors: https://covidwarriors.gov.in/default.aspx
That should be exposed as an API for devs to develop features around the same.
Describe the solution you'd like
Data has to be scraped and dumped in json and csv files and add to covid-api.
Verified that they don't have any APIs powering their solution that can be reused.
Additional context
We're starting to work towards it. Will need help in hosting on api.covid19.

i would be doing it in python

Wondering is this would be available somewhere in data.gov.in portal.

@gyanendrapatro for a start you can publish a MxN matrix of (states, metrics) in a json along with a timestamp of when the data was scraped. In case this data changes, we can run the script again and generate another snapshot of the same data.

@gyanendrapatro
Is this data being updated regularly by MOHFW? Could you confirm that first? Is it's static, it may not be useful. If it's quite dynamic - we should capture this information as it will help in doing capacity analysis across states.

This data is not rich, unique or recent enough to be included in this API. Anybody who needs it can capture those values from the website directly. Hence, closing.


Set ctx.inside_macro based on FmtVisitor.parent_context in  FmtVisitor::get_context

Fixes #5526
When creating new contexts with FmtVisitor::get_context we set the value of ctx.inside_macro based on the parent_context if it exists. This helps us remember that we're inside of a macro when creating new RewriteContext from recursively created FmtVisitors.
A similar issue to #5526 was reported in #3139, and remembering that we're inside of a macro in this way helps us go down a code path that resolved #3139 and was implemented in PR #3142.

Thanks for the additional detail. Could you take a look at the results from the diff check run https://github.com/rust-lang/rustfmt/runs/8171301629?check_suite_focus=true#step:4:324 and summarize when you get a chance?
Even if this is a change that should be made, is possible we could need to version gate to avoid introducing breaking formatting changes (though if a sufficiently strong argument could be made that this is a bug fix then I'd consider doing so without a gate)

There were three repos that reported diffs: rust-lang/rust, Rocket, and hyper.
The rust-lang/rust diff was just reiterating the change I made to the system test in this PR. The Rocket error was similar. Here's a modified code snippet from the Rocket diff:
fn test() {
    assert_eq!(multi, Multi {
        checks: vec![true, false, false],
        names: vec!["Sam".into(), "Smith".into(), "Bob".into()],
        news: vec!["Here".into(), "also here".into()],
        dogs: {
            let mut map = HashMap::new();
            map.insert("fido".into(), Dog { barks: true, trained: true });
            map.insert("George".into(), Dog { barks: false, trained: true });
            map.insert("bob boo".into(), Dog { barks: false, trained: false });
            map
        },
        more_dogs: {
            let mut map = HashMap::new();
            map.insert("My Dog".into(), Dog { barks: true, trained: true });
            map
        }
    });
}

the current master wants to add a trailing comma when formatting Dog { barks: false, trained: true } over multiple lines, however the current PR does not add the trailing comma.
The hyper diff seems more involved and I'll need to do more investigating. For now, here are the code snippets that get poorly formatted by this PR:
#[bench]
fn throughput_fixedsize_many_chunks(b: &mut test::Bencher) {
    bench_server!(b, ("content-length", "1000000"), move || {
        static S: &[&[u8]] = &[&[b'x'; 1_000] as &[u8]; 1_000] as _;
        BodyExt::boxed(StreamBody::new(
            stream::iter(S.iter()).map(|&s| Ok::<_, String>(s)),
        ))
    })
}

#[bench]
fn throughput_chunked_many_chunks(b: &mut test::Bencher) {
    bench_server!(b, ("transfer-encoding", "chunked"), || {
        static S: &[&[u8]] = &[&[b'x'; 1_000] as &[u8]; 1_000] as _;
        BodyExt::boxed(StreamBody::new(
            stream::iter(S.iter()).map(|&s| Ok::<_, String>(s)),
        ))
    })
}

And here's how those snippets are formatted:
#[bench]
fn throughput_fixedsize_many_chunks(b: &mut test::Bencher) {
    bench_server!(b, ("content-length", "1000000"), move || {
        static S: &[&[u8]] = &[&[b'x'; 1_000] as &[u8]; 1_000] as _;
        BodyExt::boxed(StreamBody::new(stream::iter(S.iter()).map(|&s| Ok::<
            _,
            String,
        >(
            s
        )),))
    })
}

#[bench]
fn throughput_chunked_many_chunks(b: &mut test::Bencher) {
    bench_server!(b, ("transfer-encoding", "chunked"), || {
        static S: &[&[u8]] = &[&[b'x'; 1_000] as &[u8]; 1_000] as _;
        BodyExt::boxed(StreamBody::new(stream::iter(S.iter()).map(|&s| Ok::<
            _,
            String,
        >(
            s
        )),))
    })
}


[r] Bug in nested-close logic

Describe the bug
Closing an experiment (or any collection) should recursively close all of its open descendants. Yet there is a scenario below in which something doesn't quite connect.
To Reproduce

Step 1: create a ./pbmc3k SOMA experiment:

https://gist.github.com/johnkerl/5f2e26d17dc930053cebbcb9970ecbcf

Step 2 (optional): do some queries on it (not necessary to repro the bug):

https://gist.github.com/johnkerl/bdcbd51e58a13b2daf788a43c04aeac5

Step 3: add new obsm and write it back:

https://gist.github.com/johnkerl/0d1f29c643a19afc25a46ddda8041d3e
The extra ms_rna_w$close() here shouldn't be necessary, but without it, obsm doesn't appear

Note that group writes (including member-adds) don't take effect until the group is closed.
Also note that spdl-logging shows when the close happens, and if the ms_rna_w$close() is commented out, that spdl-logging shows that pbmc3k/ms/RNA isn't getting closed.
Hypothesis: the private$.member_cache isn't including everything it should at https://github.com/single-cell-data/TileDB-SOMA/blob/21520a6c0ff922d272d25afa692cc48e2ea085f6/apis/r/R/TileDBGroup.R#L90
cc @aaronwolen @mojaveazure @mlin @pablo-gar
Also note this closing logic is my own from https://github.com/single-cell-data/TileDB-SOMA/issues/1346 -- so I may be the best person to fix this. Or, maybe a fresh pair of eyes is best ...
Versions (please complete the following information):

TileDB-SOMA version: today's main
Language and language version: R
OS: reproduced on MacOS and Linux both

I see the assignment but I am really not that deep into R6 magic.  Help would be appreciate.  Maybe @mojaveazure and I can poke together...

@mlin in the spirit of load-balancing can you help take a look here?

(Also taking a peek, as I'd promised @eddelbuettel)
Following the repro in this issue's description, and using the third gist, and with this diff applied
https://gist.github.com/johnkerl/556f30a0c9d36e1c95ee01c34efc7a72
I see that what I feared was true, is true.
Namely when I print exp$memfoo() (dumper/inspector from the gist above) I see fewer things than when I do ms_rna_w$memfoo().
This in turn suggests that perhaps that as called from that third gist's
exp <- SOMAExperimentOpen(exp$uri, "WRITE")
ms_rna_w <- exp$ms$get("RNA")

the ms_rna_w appears to not be a reference to the exp$ms$get("RNA").
I am made aware that S6 does have reference semantics, but, perhaps at some point in our implementation inheritance chain for the get method, we've inadvertently copying something -- ?

I actually spent several hours noodling around this issue, and have no fix.  Adding simple on.exit(soma_parent$close()) closes too soon for our nesting so I have no suggested fix.

A bit more background:

First gist in this PR's descriptions writes the SOMA experiment ./pbmc3k
The full structure looks like this

root group pbmc3k
  obs array
  ms group
    RNA group
      var array
      X group
        counts array

The third gist does

...
pbmc3k <- RunPCA(pbmc3k, features = VariableFeatures(pbmc3k), verbose = FALSE)
...
exp$close()
exp <- SOMAExperimentOpen(exp$uri, "WRITE")
ms_rna_w <- exp$ms$get("RNA")
...
write_soma(pbmc3k[["pca"]], soma_parent = ms_rna_w) # This updates the `obsm`
...

Using print statements, we find that

exp's member cache shows what is open for the exp R6 object is only obs and ms in its member cache.
ms_rna_w's member cache shows what is open for the ms_rna_w R6 object includes the new obsm as well

Crucially, TileDB groups have commit-on-close semantics

So if we only call exp$close() (implementation here then the new obsm is not persisted to storage, since the exp R6 object is not aware of it
If we do call ms_rna_w$close() then the new obsm is persisted to storage, since the ms_rna_w R6 objecy is aware of it
In this latter case we successfully see

root group pbmc3k
  obs array
  ms group
    RNA group
      var array
      X group
        counts array
      obsm group    # <----- new
        X_pca array # <----- new

Since the ms_rna_w R6 object was gotten from the exp R6 object via ms_rna_w <- exp$ms$get("RNA"), and since (naively) I believe R6 objects have reference semantics, the exp R6 object's member cache ought to traverse on exp$close() down to the same location as ms_rna_w, but, we see evidently that it does not

So there is an opportunity to do more debugging about R6 reference semantics.

Now that's a nice write-up!  (I used spdl::* rather than print myself but details).  One quick idea while reading this: "TileDB groups have commit-on-close semantics".  Could we work around this here (where we know things are nested) by explicitly calling a commit or write rather than waiting for the implicit one that happens too late?

by explicitly calling a commit or write rather than waiting for the implicit one that happens too late?

@eddelbuettel I love this too, but unfortunately, the "commit" or "write" function for tiledb groups is, in fact, spelled c-l-o-s-e. So our only protection here would be to call group$close() before the user requested it (and without them having requested it). Which, I think, is not a good user experience ...

Maybe we have to add a french variant e-c-r-i-r-e then.
Kidding aside maybe the cost of close-then-reopen is what we have to bear here.  Better than failing.

Found and fixed in sandbox. PR coming up tomorrow.

What was the issue?  In a real-quick reader's digest version?

This simple repro shows the bug. First make a getfoo and setfoo at the TileDBObject level:
diff --git a/apis/r/R/TileDBObject.R b/apis/r/R/TileDBObject.R
index 7907d385..cc53d9ae 100644
--- a/apis/r/R/TileDBObject.R
+++ b/apis/r/R/TileDBObject.R
@@ -46,6 +46,13 @@ TileDBObject <- R6::R6Class(
       spdl::debug("[TileDBObject] initialize {} with '{}'", self$class(), self$uri)
     },

+    getfoo = function() {
+      return(private$.foo)
+    },
+    setfoo = function(value) {
+      private$.foo <- value
+    },
+
     #' @description Print the name of the R6 class.
     class = function() {
       class(self)[1]
@@ -121,6 +128,8 @@ TileDBObject <- R6::R6Class(
   ),

   private = list(
+    .foo = NULL,
+
     # Pro tip: in R6 we can't set these to anything other than NULL here, even if we want to.  If
     # you want them defaulted to anything other than NULL, leave them NULL here and set the defaults
     # in the constructor.

Next try it out -- if exp$ms is the desired reference then we can set/get the .foo attribute on exp as well as exp$ms:
> exp$setfoo(100)
> exp$getfoo()
[1] 100
# Works

> exp$ms$setfoo(200)
> exp$ms$getfoo()
NULL
# Not good

> ms <- exp$ms
> ms$getfoo()
NULL
> ms$setfoo(300)
> ms$getfoo()
[1] 300
# Readback works as long as we retain `ms <- exp$ms`

The bug is simple: on TileDBGroup's set we do update the member cache here
https://github.com/single-cell-data/TileDB-SOMA/blob/ee266fbeec9b84178ca74be802dd6f0774ed10f4/apis/r/R/TileDBGroup.R#L139-L144
whereas on TileDBGroup's get we do not:
https://github.com/single-cell-data/TileDB-SOMA/blob/ee266fbeec9b84178ca74be802dd6f0774ed10f4/apis/r/R/TileDBGroup.R#L173-L183
The fix is simple:
private$add_cached_member(name, obj)

in TileDBGroup's get

For what it is worth I did create a local script reproducing this issue (and hinging on the undesirable explicit ms_rna_w$close() to flip between 'works-with-the-extra-close' and 'does not work'.  Applying the fix suggested in #1524 improves this and we no longer need ms_rna_w$close() --- which is quite promising.

#1524 is merged


Admin console not loading after fresh install from stable

Today I have made several attempts to setup a brand new instance of Ghost locally from stable. After going through the initial setup:
npm install -g grunt-cli
npm install
grunt init
npm start
The initial page will load at http://localhost:2368. When I navigate to http://localhost:2368/ghost the redirect to http://localhost:2368/ghost/setup occurs, but a blank page loads. After using Chrome Dev Tools with Ember diagnostics it appears that there is an issue within Ember itself. However, I don't have the knowledge to dig further myself.

Hey @mathewballard,
Sorry you're having trouble!  I think the one step you're missing is running grunt prod after grunt init finishes.  That will "build" all of the front-end assets, which is required if you're running Ghost from the GitHub repository instead of the .zip file.

Unfortunately that doesn't resolve the issue. Single uses of grunt init/ grunt prod or running them both yields the same results.

Yeah, sorry about that, I edited my comment above.

Error: Cannot read property 'template' undefined.
Wanted to get that up while I try out your suggestion.

Current stable depends on Ember 1.9.0, and the error above is from trying to call Ember.HTMLBars.template() which isn't available in Ember prior to 1.10.
Changing all calls to Ember.HTMLBars.template() to Ember.Handlebars.template() will let you start Ghost.

The real culprit is grunt-ember-templates which now defaults to using HTMLBars.
diff --git a/Gruntfile.js b/Gruntfile.js
index 86c8853..df95b16 100644
--- a/Gruntfile.js
+++ b/Gruntfile.js
@@ -382,6 +382,7 @@ var _              = require('lodash'),
                     options: {
                         templateBasePath: /core\/client\//,
                         templateFileExtensions: /\.hbs/,
+                        templateNamespace: 'Handlebars',
                         templateRegistration: function (name, template) {
                             return grunt.config.process('define(\'ghost/') +
                                 name + '\', [\'exports\'], function(__exports__){ __exports__[\'default\'] = ' +

Adding templateNamespace to the Gruntfile will build everything properly.

Yeah, I see that there was a breaking change from 0.5.0-alpha to 0.5.0.  That's not ideal but I'm sure they had reasons.
@mathewballard Probably the easiest thing to do is just npm uninstall grunt-ember-templates && npm install<EMAIL_ADDRESS>and then you'll end up with the version that works.
We've switched to HTMLBars on master, so in the next stable release this won't be an issue.

Thanks everyone, I got it up and running by following @davidolrik's suggestion.

One last thing for future reference. Here are the steps I followed leading up to the issue:
The only thing that I can think of that happened is me pulling down the repo shortly after the update for the Ember namespace change was made - I had noticed at the time that there were a lot of Ember related commits which sent me down this road during debugging. I then switched to stable and ran the
npm install -g grunt-cli
npm install
grunt init
npm start
I'm wondering if because I started with master and then switched to stable if something was left behind that caused the issue in stable.

No, the problem you were having was caused by the grunt-ember-templates module changing its behavior between 0.5.0-alpha and 0.5.0.  There really shouldn't have been a breaking change like that   without moving to something like 0.6.0 but it happens sometimes, especially when packages are still in version 0.x.x.

Nope, I had the same problem and I started with<EMAIL_ADDRESS>The problem is as @jaswilli stated, grunt-ember-templates changed defaults between 0.5.0-alpha and 0.5.0 and package.json has its version specified as ~0.5.0-alpha which means its ok for nom to get any 0.5.x version.

There shouldn't be any braking changes with out changing the major version number, but pre 1.0.0 anything can happen.

Yeah, everything built on node is so fragile that things like this happen. It is odd though, because I had pulled it down a day before and it ran fine, then the next day that wasn't the case.
With all that said, what is the ETA on the next stable release?

Making changes to the stable branch is undesirable, but I have added an npm-shrinkwrap.json which should, I believe, resolve this problem with the loosely-bound dev dependencies without actually changing the code.
I have been thinking about including the fully built assets on stable, but I'm not sure if that's warranted or desirable.
The shrinkwrap file seems to me to be a happy medium, we probably ought to include it in all releases. I will update the release scripts to include this file I think.

Hi Hannah ( @ErisDS )
For me I can NOT confirm that the npm-shrinkwrap.json content on stable is working. Still the blank error caused by the wrong calls for HTMLBars and Handlebars in generated templates.js.
What I can see so far, if you wish to solve it via npm-shrinkwrap.json you should or have to add dependencies for grunt-ember-templates as well.
But still I prefer #4920 suggestion for stable fix.
Tom


Missing log_level option for Python client

First of all, thanks for creating an awesome profiling tool. 👏🏾
Describe the bug you encountered:
The doc mentioned that we could configure the log_level.
import pyroscope

pyroscope.configure(
....
  log_level           = "info" # default is info, possible values: trace, debug, info, warn, error, and critical
)

When I configured the log_level, I got the error
    pyroscope.configure(
TypeError: configure() got an unexpected keyword argument 'log_level'

What did you expect to happen instead?
I expect the log_level parameter to exist and should only see certain logs.
How did you install pyroscope-rs?
I installed it with pip install pyroscope-io -U

pyroscope-rs version and environment
$ pip list | grep pyro
pyroscope-io          0.7.2
$ python3 --version
Python 3.10.1

I took a peak at the code and found that the log_level param is missing from the configure function
https://github.com/pyroscope-io/pyroscope-rs/blob/main/pyroscope_ffi/python/pyroscope/__init__.py#L12-L15
Digging a bit deeper, the param log_level was removed in this commit https://github.com/pyroscope-io/pyroscope-rs/commit/1696cd06db53a1d7ebbdcee556334156eed3429e.

First thanks a lot for this easy to use and awesome tool.
But I think there is a related but separate issue: seems the log levels mapping is incorrect
https://github.com/pyroscope-io/pyroscope-rs/commit/1696cd06db53a1d7ebbdcee556334156eed3429e#diff-9d372ec50bcb337bf0a01889d97d230d6d235bb15487175a17e4c7a1456ce285R51-R70
Here 50 => error, 40 => warn, 30 => info etc, while in python logging 40 => ERROR, 30 => WARNING, 20 => INFO etc


[WIP] Componentize

Tracking PR for componentization work.

@KeitIG I notices this code in some places:
constructor(props) {
    super(props);
    this.state = {};
}

even though there are no calls to get/set state.
What is the purpose of such code? Is it useful?

@YurySolovyov they are mandatory in React.

Does not seem to be mandatory if component is pure and don't have own constructor

class MyClass extends React.component {
render(){
return Hello { this.props.world };
}
}
The code above is perfectly valid. You don't have to call super() for every react component you create. However, if there is a constructor in your code, then you MUST call super

Feel the squash !

Let's go like that for now.