YamlCache reads the entire file's contents to see if the file is in the cache

See original issue on GitLab
In GitLab by [Gitlab user @jonathanmaw] on Feb 1, 2019, 15:00
Background
YamlCache is a class that speeds up the parsing and provenancing of yaml by maintaining a cache of yaml parsing that is persistent across BuildStream executions.
Currently, it reads all of a file's contents to decide whether it has changed since it was put in the cache.
There should be an easier way to do that - statting a file and comparing its mtime works in most cases.
There are currently three distinct cases where we load yaml, listed in order of most to least common:

Element file from a checked-out project. mtime is very suitable for this.
Element file from a junctioned project. mtime is not suitable for this because these files are a checked-out Source each buildstream invocation.
Element file from a workspace of a junctioned project. mtime should be suitable for this, but I am not sure whether it actually loads the file from the workspace.

Task description

[ ] Write a test case to check whether changes in a workspace of a junction are being picked up.

[ ] If not being read, fix this.

[ ] Use the file's mtime when checking for changes in an element in a checked-out project, or a workspaced junctioned project.
[ ] Use the junctioning element's cache key when checking for changes in an element in a junctioned project - plus potentially changes to the Element API if acquiring this information breaks encapsulation.
[ ] Ensure that the yamlcache is being exercised for element checkouts, junctions and workspaced junctions with tests.

Acceptance Criteria
Unless completely unavoidable, YamlCache does not need to read the contents of a file to know whether it is cached.

In GitLab by [Gitlab user @jonathanmaw] on Feb 1, 2019, 15:56
I've tested these changes on linux, and on WSL, and any difference is extremely marginal.
In the files below, "no-read" is with my changes, and "with-read" is from master.
no-read-no-cache-show.cprofile
no-read-with-cache-show.cprofile
with-read-with-cache-show.cprofile
with-read-no-cache-show.cprofile
I suspect this just indicates that the major slowdown isn't from WSL, but is from not using an SSD.
I will retry these changes later on spinning rust over USB and see if there is a more pronounced improvement.

In GitLab by [Gitlab user @juergbi] on Feb 1, 2019, 16:00
Have you tried this with cold page/dentry cache?
echo 3 > /proc/sys/vm/drop_caches
This might not be supported on WSL, though, in which case you may have to reboot before each test to ensure a cold page/dentry cache.

In GitLab by [Gitlab user @jonathanmaw] on Feb 4, 2019, 15:54
mentioned in merge request !1126

In GitLab by [Gitlab user @jonathanmaw] on Feb 4, 2019, 17:24
I've tested these changes on WSL with the project directory on the external disk over USB, and the results are no better.
master-no-yamlcache.cprofile
master-with-yamlcache.cprofile
no-read-no-yamlcache.cprofile
no-read-with-yamlcache.cprofile
It would seem that there's an equal amount of latency to read the contents of the file compared to statting the file.
Next, I will test whether a difference exists on native linux (i.e. what if there is equal latency because of the ext4 emulation layer). If I fail to see a performance improvement there, then I will close this issue, as there is no meaningful performance gain from reading the inode over reading the contents.

In GitLab by [Gitlab user @jonathanmaw] on Feb 5, 2019, 12:49
On native linux, with the project on spinning rust over USB, with drop_caches, the time saving is minor (~2s of a ~40s runtime)
master-no-cache.cprofile
master-with-cache.cprofile
no-read-no-cache.cprofile
no-read-with-cache.cprofile

In GitLab by [Gitlab user @jonathanmaw] on Feb 5, 2019, 15:11
unassigned [Gitlab user @jonathanmaw]

In GitLab by [Gitlab user @jonathanmaw] on Feb 5, 2019, 15:12
Given the benefits of this optimisation are very small, I'm putting this issue down and looking at #895 instead

In GitLab by [Gitlab user @tristanvb] on Feb 5, 2019, 17:03

Given the benefits of this optimisation are very small, I'm putting this issue down and looking at #895 instead.

Fair enough, I have a suspicion that the benefits of this optimization are only very small because the hugest bst projects are fairly small (perhaps the cost of a stat() is similar to the full read of a file that is small enough to be transfered in a single buffer ?).
Judging from the performance gained by fixing workspaces to use mtimes instead of exhaustively (and quite needlessly) hashing file content, we will gain a lot of performance here for any project which consists of about 6GB of .bst files (e.g., like a built WebKit workspace).
Perhaps it's still worth swapping out the hashing algorithm for a local mtime approach too, just for the peace of mind that this wont cost more for larger files.

In GitLab by [Gitlab user @jonathanmaw] on Feb 15, 2019, 18:02
I had another look, this time about whether there are improvements when loading yaml from junctioned elements. I had the suspicion it might be a useful improvement because we don't need to make any syscalls to know whether it's cached (we already have the junction element's cache key).
master-with-cache.cprofile
no-read-with-cache.cprofile
There is no appreciable improvement on my linux laptop. It might still be worthwhile if there's a decent improvement on WSL.

In GitLab by [Gitlab user @danielsilverstone-ct] on Mar 29, 2019, 08:33
Since !1257 merged, we no longer have a YamlCache so this issue can be closed.

In GitLab by [Gitlab user @danielsilverstone-ct] on Mar 29, 2019, 08:33
closed


Snapshots of jails triggering property update.

It seems like in some cases snapshots are triggering property updates. I'm not sure if this needs to happen, and if it does, the way it is being done probably isn't right.

I suspect this is because the snapshot on the $POOL/iocage/jails dataset is being included as part of the recursive zfs list that checks the version for all the jails, even though it isn't a jail. This should be a very simple fix, I just need to set up a dev environment to make the change and do the testing. Until then, this is a harmless bug that just adds a little delay the first time iocage runs after a new snapshot of $POOL/iocage/jails has been taken.


fix: passed IncludePaths in node-sass render function

The PR fixes #53 by passing includePaths from config to node-sass's render function.
Moreover, I used tar-ball of this PR and fixed #53 in a branch
git clone -b fix/PR-54  https://github.com/arpit-agarwal/scss-bundle-issues.git
run npm install && npm start.
Theme file is generated as expected on destination location

@arpit-agarwal, good catch. CI/CD worked, but the token was not updated after the global npm tokens revocation.
What is interesting, why didn't it throw an error, but we'll figure that out and publish the new version soon.


[Bug]: When optimize fails to commit and then deletes a file, the optimize task will fail when retrying

What happened?
When optimize fails to commit and then deletes a file, the optimize task will fail when retrying.

We can see that the first commit failed and a file was deleted afterwards.

Failed to get status for file: hdfs://nameservice1/user/hive/warehouse/realtime arctic v3.db/zl ual nonrealtime deviceprofile/hive/hdfs par=20221206/1670310415839<PHONE_NUMBER>926017638/1-B-6002581311138640498-00000-9064569570000000001.parquet. caused by File does not exist: hdfs://nameservice1/user/hive/warehouse/realtime arctic v3.db/zl uanonrealtime device profile/hive/hdfs par=20221206/1670310415839<PHONE_NUMBER>926017638/1-B-6002581311138640498-00000-906456957-0000000001.parquet
When optimize retries, optimize loads the deleted file again, causing the task to fail.
Affects Versions
0.3.x
What engines are you seeing the problem on?
AMS, Optimizer
How to reproduce
Don't know how to reproduce the problem
Relevant log output
No response
Anything else
No response
Code of Conduct

[X] I agree to follow this project's Code of Conduct

Overwrite hive table failed when committing iceberg table success and committing HMS failed.
We should ignore committing errors to HMS when committing iceberg successfully.


feature: Unbind  in terminals to leave the default "clear" shortcut work

Did you check the docs?

[X] I have read all the LazyVim docs
[X] This is not a request for a new or existing extra (see above)

Is your feature request related to a problem? Please describe.
When I use the <c-\> keymap, it opens the terminal and that's perfect. But I often use "CTRL+L" to clear the terminal. Unfortunately, the <c-l> keymap makes the cursor going in the "right" widow.
Describe the solution you'd like
Simply add this somewhere:
vim.api.nvim_create_autocmd("TermOpen", {
  pattern = "*",
  callback = function()
    -- unbind <c-l> in terminal
    vim.api.nvim_buf_set_keymap(0, "t", "<C-l>", "<C-l>", { noremap = true, silent = true })
  end
})

Describe alternatives you've considered
No other alternative, I added it inside my keymap to fix the problem - but maybe it could be the default
Additional context
No response

The main problem is that CTRL+L is a common shortcut for many terminals to refresh the view (to clear or redraw the TTY). Binding this shortcut is already an "issue" as we sometimes have some artifacts dues to some errors from plugins (for example).
I can fix it by scrolling, sometimes. But...
But, for floating terminal, it makes the cursor jumping behind, and we lose the focus - so we can break the buffer content that is hidden by the current terminal. And it's not that easy to gain focus again.
Really, C-l should never be binded by any plugin. It's a terminal shortcut, do not override it :)

I mean: c-l is already necessary in Neovim, even if we don't open a floating terminal, to redraw the terminal where NeoVim is displayed.

Screencast_2024-10-01-18-41-48.webm
This is the workflow I'm talking about. Also here's a search for <c-l> throughout Github. Lots of people use it even as a terminal mapping. So, I don't believe anyone can say

Really, C-l should never be binded by any plugin. It's a terminal shortcut, do not override it :)

People can do what they want with their configs.
Now, I don't use that particular workflow myself, but I can't doubt that it can be a legit workflow for some people. Probably that was the reason it was added as a default. I don't know for sure, because I didn't choose it, but that would be my guess.
Anyway, the maintainer will see this and will decide.

I'm also facing this problem and would like to find a solution.
I like using LazyVim's default ctrl + l to move windows around, but I also want to use ctrl + l to clear the terminal.
Since I use fish, I could save some stress by abbreviating c as clear.
However, this solution is not perfect. I hope someone has a better idea.

Just disable the terminal mapping specifically then. It's not really a problem when users can change the mappings to their liking. Users can override defaults in their personal configuration. The defaults can't satisfy every single user. And for most defaults there exists also a legit workflow.

I'm also facing this problem and would like to find a solution.
I like using LazyVim's default ctrl + l to move windows around, but I also want to use ctrl + l to clear the terminal.
Since I use fish, I could save some stress by abbreviating c as clear.
However, this solution is not perfect. I hope someone has a better idea.

My key map isn't working for you? It only disables c-l on terminals.

Add the below to your autocmds.lua:
vim.api.nvim_create_autocmd("TermOpen", {
  callback = function(ev)
    vim.keymap.set("t", "<c-l>", "<c-l>", { buffer = ev.buf, nowait = true })
  end,
})

fixed above issue by replaceing TermOpen with TermEnter callback:
vim.api.nvim_create_autocmd("TermEnter", {
  callback = function(ev)
    vim.keymap.set("t", "<c-l>", "<c-l>", { buffer = ev.buf, nowait = true })
  end,
})


Bump Python to 3.9 in Colab

It seems that Colab has switched to Python 3.9, and the current version of the notebook (based on 3.8) now raises an error at the 4th cell.
This PR fixes it, I've tested it on Colab.

Fixed in https://github.com/deepmind/alphafold/commit/2e6a78f08b3a31945ad0fad77f7cf4344ae73d13, thank you!


Happy test case for UserDeposit contract

This issue keeps track of listing and implementing happy path test cases for UserDeposit contract.

A user adds some deposit twice
A user withdraws some deposit
A service takes some deposit with an IOU

All of these have been tested already.


docs: issue linking

The issue link does not work:
Add What’s New section to docs (:issue:`10`)

on page
http://wholmgren-pvlib-python-new.readthedocs.org/en/docfixes/whatsnew.html#new-features

#22 isn't even merged yet! I want to keep the issues page focused on the code and documentation that is actually in pvlib/pvlib-python. You can reopen the issue if this is not resolved when it is merged.


Numbered Lines in Code Blocks

Hi, I just started using AsciidocFX to write some documentation for my codes.
Now I'm trying to show the line numbers in the code blocks. It doesn't seem to work
I read at Asciidoc's documentation something like that:
[source, java, numbered]

---
//Some Java code

---

Then the output shows the line numbers of the code.
I don't know if I'm doing wrong. Is it possible to do that?
Thanks

Hi, Asciidoctor.js and AsciidocFX use highlightjs, and it doesn't support line numbering.

Highlight.js still has no support for line-numbering, but http://prismjs.com/ has it. @mojavelinux what do you think to support prism.js ?

I recommend using prettify, which is integrated into core and supports line numbers.

Woow, it is good, thank you @mojavelinux

:+1:
Of all the syntax highlighters, I think prettify is one of the best. It doesn't support as many languages, but the languages it does support it supports very accurately because it's a real parser, not just a regexp matcher.

That's great! :+1:
@romulomourao You can use this feature in new release.

oh..that's awesome
thks @rahmanusta and @mojavelinux
:+1:

Please try version 1.4.6 https://github.com/asciidocfx/AsciidocFX/releases/tag/v1.4.6
You should use prettify.js like below
:source-highlighter: prettify

[source,java,numbered]
----
public class Hello{

}
----

I now close this issue, if you think your issue is not fixed, please don't hesitate to reopen or comment the issue.

I would like to use okaidia style with prettify and not the default. Is this possible?


Feature Suggestion: Using JSON-LD for Gizmo results

Description
Currently, Gizmo returns JSONs of strings which may or may not be valid n3 values. This is a non-standard behaviour that is hard to integrate with other systems.
Examples
Results of IRIs:
{ "@id": "bob" }

Results of literals:
"Bob"

Results of literals with language:
{ "@language": "en", "@value": "Bob" }

Results of literals with type:
{
    "@value": "2010-05-29T14:17:39+02:00",
    "@type": "http://www.w3.org/2001/XMLSchema#dateTime"
}

All together
{
   "id": { "@id": "bob" },
   "name": "Bob",
   "englishName": { "@value": "Bob", "@language": "en" },
   "timeCreated": {
      "@value": "2010-05-29T14:17:39+02:00",
      "@type": "http://www.w3.org/2001/XMLSchema#dateTime"
   }
}

Future Proposals

It might be better to change id default property

Took a deeper look into Gremlin. It seems like you need to be explicit about the return type you get there.
To get nodes:
g.V()

To get values:
g.V(1).values('name')

To get tags
g.V().as('a').out().as('b').out().as('c').select('a','b','c')

Which I find more explicit and simple

Here is a basic implementation: https://github.com/cayleygraph/cayley/pull/821

@dennwc what do you think about the previous comment?

@iddan It's definitely a nicer way for selecting results. May also help the optimizer to some extent.

Merged https://github.com/cayleygraph/cayley/pull/821


Upgrade Linux Kernel for flatcar-3510 from 5.15.120 to 5.15.123

Subject says it all.

We need to first release the next Stable with Kernel 5.15.120, https://github.com/flatcar/scripts/pull/981, not this one.
Added on-hold.

close and recreate


Closes: #1

updated validations.py python script.
Fixed the behaviour of validate_user function in validations.py by including good regular expression in script
Fixes #<issue_number_goes_here>

It's a good idea to open an issue first for discussion.

[ ] Tests pass
[ ] Appropriate changes to README are included in PR

@googlebot I signed it!


Manual Task Title Update failure

Describe the bug
Currently unable to change the title for a task of type Manual (this would likely be in the ManualTaskDesigner component)
To Reproduce
Drag and drop a manual task onto a flow. Try to change the title, you should see a bug
Expected behavior
As for the other tasks, a user should be able to give a custom name to a task.
Screenshots

Hi @BenjaminRuby
I'm not able to reproduce in dev, can you share which workflow you received this error?

Hey @Okisa is this making it into 3.4 or 3.4.1 as a bug fix?

Still not able to reproduce locally or in dev.

@Okisa @BenjaminRuby I believe this is still an issue in live for a certain workflow

fixed with 3.1.67

Thank you guys working on it, let me inform the user that status.

Resolved and verified. Part of Release 3.4.1


[Editor] Change the resize cursors to bidirectional variants

When resizing an editor we're currently using unidirectional cursors, please refer to https://developer.mozilla.org/en-US/docs/Web/CSS/cursor
Given that editors can (generally) be resized to become either smaller or larger, it seems overall more appropriate to use bidirectional cursors to make this clearer to the user.
Note that as mentioned in the MDN article some environments, which seems to apply to e.g. Windows 11, doesn't differentiate between the two cursor formats and simply use bidirectional ones unconditionally.
One additional benefit of these changes is that the relevant CSS rules become slightly more compact.

/botio-linux preview


show stats row counts for tabs that are missing them

Would be great if the es nodes, es indices, es tasks, and es recovery tabs could show a shown count (after filtering) and a total count (before filtering).  Pagination not required.

fixed in 9860d84


P2441R2 views::join_with

https://www.open-std.org/jtc1/sc22/wg21/docs/papers/2022/p2441r2.html
__cpp_lib_ranges_join_with : 202202L

完了


mxf2raw: extend --rdd6 option description

Fixes https://github.com/bbc/bmx/issues/44

Thanks, it's very useful to have this documented more thoroughly.
A few thoughts:

There's no indication given if the frames value was insufficient to allow complete program descriptions to be extracted for all programs
If there were multiple programs in the bitstream can they in practice be included in the XML? It looks like the XML only supports a single program description.

I don't think we have a use case for multiple programs -- if mxf2raw can't fully support this then this could be noted in the help text

Presumably mxf2raw assumes that the RDD 6 metadata is static except for both frame_count and description_text i.e. there are two non-static fields

Presumably mxf2raw only reads RDD 6 metadata (except for those two fields) from the first frame indicated in the frames commandline parameter? i.e. won't notice if other values actually vary

Perhaps remove the "Set the  ..." paragraph from https://github.com/bbc/bmx/tree/main/meta/rdd6_xml_creator to avoid conflicting text / having to maintain the text in two places

I've updated the PR to

improve the option description following your comments,
log a warning if the description text is not complete for all programs,
remove the redundant text in the rdd6_xml_creator app

The RDD 6 XML schema defines the desc_text element to have at most 8 programs. See table 6.2 of the RDD 6 spec.. E.g. if there is 5.1 and stereo audio then that is represented as 2 RDD 6 programs.

Just one outstanding item:

presumably mxf2raw only reads RDD 6 metadata (except for those two fields) from the first frame indicated in the frames commandline parameter? i.e. won't notice if other values actually vary

Yes, it assumes it is static and doesn't check if it isn't.

On this topic, the help text says:

The output XML will contain the static RDD 6 metadata

Perhaps you can just add something to convey whether this RDD 6 metadata being static is just an assumption, or whether mxf2raw actually checks that it's static (for the frames that it reads)?

I've changed it to say it assumes it is static and that it uses the non-first frames to accumulate the description text.

LGTM


auro-modal: Separate modal state from background opacity

Is your feature request related to a problem? Please describe.
The current state options for Auro modal (blocking or non-blocking) are tied to the opacity level used to darken the screen behind the modal. Darkening the entire screen by 20% feels excessive for a dialog telling you the seat has no window. A slight shadow would suffice for desktop and mobile to differentiate the modal overlay from the background content in this instance.
Describe the solution you'd like
Can the dialog's state (non-blocking vs. blocking) be separated from the level of darkening scrim used behind it? State should still be used to determine available interaction options (an X to close vs. 'yes or no' button options, respectively). The opacity level for the background could have variable options for darker, lighter, or transparent (none) based on your specific use case with the modal/dialog drop shadow being set to the standard shadow level for cards and other overlays.
Describe alternatives you've considered
An additional state option could be added. This would allow the darkest background option to be tied to the blocking state, the lightest tied to the non-blocking state, and a transparent background option to be added to the new state. I have not been able to come up with another other differentiator to address the functionality overlap between non-blocking and any new state.
Additional context
Screenshot, Auro modal with transparent background and drop shadow.

As discussed in the linked PR, the solution being proposed dismisses the intent behind having a dark background being associated with a modal dialog and a standard dialog having a light background. Simply defining these are loosely coupled options that dismiss this intent and thus drastically change their definition.
My question here is, why can't this dialog box have a light background? Is the user not being pulled from the previous interaction to a new interaction?

As discussed in PR #14, we'll be keeping the behavior of auro-dialog the same. A sticky feature as well as mobile compatibility in will be added to auro-popover. Closing this issue.


Can't run server

Hi all:
I downloaded on Windows, and extracted the ZIP file to "Downloads\alexa-app-server-master", and ran:
npm install
Which seemed to work, but then when I tried to run:
node server
I get an error: "Error: Cannot find module 'D:\Data\Amazon\alexa-app-server\examples\server\server"
There is a "server" folder in the "examples director"...
Note I am a TOTAL noob, working through the book "Building Voice-Enabled Apps with Alexa"...
Any and all advice is greatly appreciated.
John.

This is a library that you can use, but not "download & install". Maybe go through http://code.dblock.org/2017/02/16/building-a-simple-alexa-skill-live-code.html first?
Closing this but feel free to ask more questions.

Thanks for your response, but if this is not "installable", why does the readme.md say this:
Installation
npm install alexa-app-server --save

You can do that, but you don't need to download anything, it gets that from npmjs.org.


Quarterly report - 2024Q3

Quarterly report, 2024Q3

Reporting Period: 2024-07-01 - 2024-09-30
Highlights:

Set up data storage infrastructure and allocated backend compute resources to host the new VegBank system, including development and production environments.
Deployed the latest stable and security-patched version of the database management software system to the development environment.
Developed and deployed a configurable database migration framework for applying schema updates as needed both for completing the system upgrade, and for accommodating future schema changes in a flexible and robust manner.
Coded and tested database migration steps to make the original VegBank schema compatible with the new database system, and to automate loading of all existing VegBank data onto the new platform.
Initiated REST API design iterations to flesh out user stories and frame API requirements, and documented the base cases for new data insertion.
Created new code repository and initial scaffolding for the R client package (“vegbankr”).
Initiated internal design discussions around core workflows for preparing, uploading, and retrieving VegBank data using the R client.
Developed initial as hoc scripts for loading sample VegCAMP plot data into R, interrogating the schema, and conducting a preliminary assessment of compatibility with VegBank.

Issues:

https://github.com/NCEAS/vegbank2/issues/29 (completed in Q3)
https://github.com/NCEAS/vegbank2/issues/33 (initiated in Q3)
https://github.com/NCEAS/vegbank2/issues/8 (initiated in Q3)


Converted exclusive systems to parallel systems wherever possible

Closes #2767.
Converted:

play_queued_audio_system
gilrs_event_startup_system
gilrs_event_system
change_window

Couldn't test gilrs_event_* because I don't have a controller.

It is probably introducing ambiguity and some one-frame delays. From what I understand by default all the exclusive systems run at the start of the stage and this change makes it run in between. I would assume the same applies to gilrs_event_system.
I don't know how to make these systems run first without making them exclusive again or putting labels everywhere. Idk if putting labels everywhere is an acceptable solution, I don't think it is very maintainable.
For Reference:

Ambiguity before this change:-- "bevy_winit::change_window" and "bevy_audio::audio_output::play_queued_audio_system<bevy_audio::audio_source::AudioSource>"

Ambiguity after this change:-- "bevy_render::camera::camera::camera_system<bevy_render::camera::projection::OrthographicProjection>" and "bevy_winit::change_window"
-- "bevy_render::camera::camera::camera_system<bevy_render::camera::projection::PerspectiveProjection>" and "bevy_winit::change_window"
-- "bevy_ui::widget::text::text_system" and "bevy_winit::change_window"
-- "bevy_ui::flex::flex_node_system" and "bevy_winit::change_window"
-- "bevy_winit::change_window" and "bevy_text::text2d::text2d_system"

Idk if putting labels everywhere is an acceptable solution, I don't think it is very maintainable.

Yeah; there's a larger conversation ongoing about this: see #2747.
I suspect that the change_window system's ambiguities are going to genuinely cause issues. Can we move this into the stage before?

Can we move this into the stage before?

I think that will exacerbate the issue of one-frame delays because the majority of commands put into the Window::command_queue will probably come from CoreStage::Update. It might be best to keep it as an exclusive system for now. It is hard to say which is better without more testing.

@alice-i-cecile I did some testing with timing the duration it takes for changes triggered in scale_factor_override.rs example to show up in the flex_node_system (the only consumer of WindowScaleFactorChanged events).
postupdate non exclusive: 0.549728 ms - 10.027739 ms
postupdate exlcusive: 0.390465 ms - 0.620356 ms
update non exclusive: 0.390914 ms - 7.55568875 ms
update exclusive: 7.643498 ms - 11.235118 ms

I think the non exclusive ones change the order they are executed in across executions of the binary. They produce extremely varying results.

because the majority of commands put into the Window::command_queue will probably come from CoreStage::Update

Hmm, I agree. We could make another stage :/ I think the best solution here is probably to add a public label to change_window and then have all the other conflicting systems run before it.

I think the parallel ones change the order they are executed in across executions of the binary. They produce extremely varying results.

Yep, that's standard ambiguous ordering behavior :(

@alice-i-cecile Added the labels in. Ran a benchmark again and now its around 0.302998 ms - 0.411899 ms pretty reliably. There are no more ambiguities reported related to this change pr now.

@SarthakSingh31 can you rebase this on main? :) I'd like to get this merged but there are now merge conflicts

@alice-i-cecile rebased. It looks like gilrs_event_system was converted in a different pr so removed anything related to it from this pr.

Thanks! Ping @mockersf; can you merge this in?

the system add_clusters in bevy_pbr is an exclusive system that would now be executed before the change_window system. Is that an issue? @superdump ?

The system add_clusters in bevy_pbr is an exclusive system that uses Res and that would be executed before the change_window system after this change. Is that an issue?

My intution is that this should run after, both to preserve existing behavior and for correctness. @SarthakSingh31 can you make that change? We can revert it if our rendering experts tell us otherwise?.

The system add_clusters in bevy_pbr is an exclusive system that uses Res<Windows> and that would be executed before the change_window system after this change. Is that an issue? @superdump ?

Hmm. I think the current setup has add_clusters being run first as an exclusive system to ensure the presence of Clusters components on appropriate views, and then update_clusters is a normal system that overwrites the cluster configuration based on the window size. I think that suggests it only has to be correct when update_clusters is run. To complicate things further, I know that update_clusters is going to move to being called from assign_lights_to_clusters in an upcoming PR.

If add_clusters required windows to be correct and it is an exclusive system, then it would then require that change_windows is either exclusive and run before it, or it is run in an earlier stage, right?

If add_clusters required windows to be correct and it is an exclusive system, then it would then require that change_windows is either exclusive and run before it, or it is run in an earlier stage, right?

Huh, change_window is also an exclusive system. In that case, yes.
However, it looks like change_window also has no reason to be exclusive: https://github.com/bevyengine/bevy/blob/caf6611c62fddf1477e74a5cbe7c3e757446fbba/crates/bevy_winit/src/lib.rs#L48.

yup, this pr is changing change_window from exclusive to non exclusive
it seems add_clusters is set as exclusive only for ordering purpose though

yup, this pr is changing change_window from exclusive to non exclusive
it seems add_clusters is set as exclusive only for ordering purpose though

It isn't only for ordering. It inserts Clusters components on views which don't have them. Subsequent systems in PostUpdate expect Clusters to be present. I suppose if you accepted a 1-frame delay in the Clusters component coming into existence then it would be fine.
That said, I don't think add_clusters was specified to necessarily run after change_windows among the exclusive PostUpdate systems, so it could have run after. And, also as said, I think as long as update_clusters runs after change_windows, it should be fine, but we need to test it.

@mockersf fixed the ordering and resolved the new merge conflicts. I feel like update_clusters should be reported as ambiguous before this change but it isn't.

@SarthakSingh31 can you rebase this? I'd like to merge this in.

bors r+

I don't know if it's too late now, but it would be cool if this PR used the actual system as a label instead of a new SystemLabel now that we have this feature.


Drop down does not work with reactive forms (needs changedetector or onpush)

v 4.15.0
I'm using reactive forms and when initialising my form the writevalue is written correctly
Nevertheless, "Every subsequent update" of the reactive form does not refresh the value on screen
That is i debugged until propagateSelected and values are assigned correctly but the view is not refreshed
This is because i use reactive forms with onpush strategy so i propose that there's some change mechanism in place in drop down that refreshes the view like change detector markForCheck()
from a user experience
user selects a value from a dropdown

user hits reset ..nothing changes ( i''m using angular this.form.reset({})
user just touches any of the other controls and the form is reset

thanks

just for info #1536 does not solved the above, the above is related to this.form.reset({}

I made a simple code sandbox with a dropdown within a reactive form using the onpush strategy, I can't seem to break it but I must be missing something @huineng, if you want to take a quick look https://codesandbox.io/s/dropdown-reactive-forms-1bq7e?file=/src/app/app.component.html

I too am having issues reproducing this issue. I'm using Angular 11 and the latest version of carbon-components-angular. A minimal ng cli app that exhibits the issue would help with debugging.

I too am having issues reproducing this issue. I'm using Angular 11 and the latest version of carbon-components-angular. A minimal ng cli app that exhibits the issue would help with debugging.

yes, after installing latest version it works
thanks

yes, after installing latest version it works
thanks


PHP 8+ compatible

Please make the package PHP 8.0, 8.1 and 8.2 compatible.

Hi @RosiersRobin - thank you for your interest in this package.
I have been really busy lately and I forgot to do it. Would you be up for the challenge?
Looking forward to hearing back from you.

Hi @gocanto I made a PR for you https://github.com/gocanto/http-client/pull/9

Closed by https://github.com/gocanto/http-client/pull/10
Thanks.


[pokemon] add extractor for watch.pokemon.com

Please follow the guide below

You will be asked some questions, please read them carefully and answer honestly
Put an x into all the boxes [ ] relevant to your pull request (like that [x])
Use Preview tab to see how your pull request will actually look like

Before submitting a pull request make sure you have:

[x] At least skimmed through adding new extractor tutorial and youtube-dl coding conventions sections
[x] Searched the bugtracker for similar pull requests
[x] Checked the code with flake8

In order to be accepted and merged into youtube-dl each piece of code must be in public domain or released under Unlicense. Check one of the following options:

[x] I am the original author of this code and I am willing to release it under Unlicense
[ ] I am not the original author of this code but it is in public domain or released under Unlicense (provide reliable evidence)

What is the purpose of your pull request?

[ ] Bug fix
[ ] Improvement
[x] New extractor
[ ] New feature

Description of your pull request and other information
Adds a new extractor for updated Pokémon TV https://watch.pokemon.com/ (Issue #22517)
Older URLs and site are still working so the original extractor is not touched.
The new player website is a bit difficult to extract any metadata from as most of the content is loaded under <div id="playerContainer"> after the initial page load. Metadata such as description and thumbnails isn't possible to get from the original URL by itself so relying instead on a channels API that returns much more data than we really need. The response size is ~555KB at time of writing as a JSON array, which is just brute-force searched to find media with matching ID... let me know if there's a better approach. I feel if a user is downloading a video anyway, the additional JSON array download is negligible in the bigger scheme of things. This is skipped if the user is just listing formats.

Thanks for this dude

Thanks for this dude

Did you get this to work?
Mine fails on newest youtube-dl version
C:\Users\ChriZ\Desktop\New folder (4)>youtube-dl.exe https://watch.pokemon.com/da-dk/player.html?id=1975805c69c4475280e82678915bc24c
[generic] player: Requesting header
WARNING: Falling back on generic information extractor.
[generic] player: Downloading webpage
[generic] player: Extracting information
ERROR: Unsupported URL: https://watch.pokemon.com/da-dk/player.html?id=1975805c69c4475280e82678915bc24

Thanks for this dude

Did you get this to work?
Mine fails on newest youtube-dl version
C:\Users\xxx\Desktop\New folder (4)>youtube-dl.exe https://watch.pokemon.com/da-dk/player.html?id=1975805c69c4475280e82678915bc24c
[generic] player: Requesting header
WARNING: Falling back on generic information extractor.
[generic] player: Downloading webpage
[generic] player: Extracting information
ERROR: Unsupported URL: https://watch.pokemon.com/da-dk/player.html?id=1975805c69c4475280e82678915bc24

I compiled ssaquas fork, i dont think youtube-dl has accepted the changes

Any chance the developers see this, and accept the changes maybe?.
Thanks for your answer!

@dstftw apologies for the ping, any chance to have this reviewed? I think this PR has gotten buried.

@dstftw apologies for the ping, any chance to have this reviewed? I think this PR has gotten buried.

Hope @dstftw can implement this fix in next release <3
Please :)

Hi !
Any chance this branch could be commit to release? :)
Thanks for your work!
Regards

Following this thread as well - as I"m also interested in the fix for https://github.com/ytdl-org/youtube-dl/issues/22517

Following this thread as well - as I"m also interested in the fix for #22517 . I will try to use the fork for now.
EDIT: Can confirm - the fork seems to work. For anyone wanting to use it while waiting for it to make a release, just clone ssaqua's fork, enter repo, then run "python -m youtube_dl" (You will need python installed) with whatever options you want after it. Seemed to work fine. Thanks ssaqua!

Would it be possible to give me a download link for your modified youtube-dl please?, im not that keen with python and such :)

Following this thread as well - as I"m also interested in the fix for #22517 . I will try to use the fork for now.
EDIT: Can confirm - the fork seems to work. For anyone wanting to use it while waiting for it to make a release, just clone ssaqua's fork, enter repo, then run "python -m youtube_dl" (You will need python installed) with whatever options you want after it. Seemed to work fine. Thanks ssaqua!

Would it be possible to give me a download link for your modified youtube-dl please?, im not that keen with python and such :)

@Masterditto - Unfortunately it doesn't really work that way - I have no download link to share. @ssaqua forked the project, and made changes in it. That is already shared and available. If you want to use their changes, you need to follow my directions above (which are pretty standard and nothing special) to run their changes manually. Otherwise, I think you may need to wait until their PR is approved and this makes its way into the release branch, and likely gets published into a more non-developer friendly form.

@Masterditto I merged that PR in my fork. Updated binaries and pip package will come in the next few hours.


Feature/csh/max filters per channel

Description

Now event handler limits filters size per channel!

Thanks for the review! Now merging


Set the default bouncetime value to -666

Set the default bouncetime to -666 (the default value -666 is in Rpi.GPIO source code).
As-Is: if the bouncetime is not set, your setting for event detecting is silently down. And there is no notification that bouncetime is required.

Thanks!


add a default table view for TimeSheetTotalPerDay (fix #8)

Fixes issue #8
This formats TimeSheetTotalPerDay objects (from Get-TimeSheetTotalPerDay) as a table by default, subjects on the left.
Table-formatting widths are a bit subjective to get right, but I tested this in a terminal only 80 chars wide to a fullscreen one and it scaled pretty satisfactory for me.
Format-Table -AutoSize and Format-List are of course always available too.
https://learn.microsoft.com/en-us/powershell/module/microsoft.powershell.core/about/about_format.ps1xml?view=powershell-7.2

Nice one!
I'm constantly doing FT on this, so a custom format type is perfect.
I'll merge this in now and figure out if we want to exclude some other properties by default next time e.g. I rarely use any timespan formats that aren't the *Agg properties.


Adding Registration Feature

Adding both registration styling components and backend logic.

Also added form error handlers

And all the logic required in parse to manage user auth
Update:
Adding user persistance and login component. Also added protected routes!

Great work!
I would split his PR into 2, login and registration! This would make it easier for reviewers!
Same as before include more context in the PR about the project and plan!

Done!


Allow insertion of custom types for 'Json' type with the 'gen types typescript' command

Feature request
After having to manually change the JSON values of my types file every time I generate a new ones, it gets repetitive. It seems like it should possible to have those types replace automatically when the supabase gen types typescript is run.
Describe the solution you'd like
I'm not exactly sure how this would be approached, but it seems one possible solution would be to pass in a file path to a json file that mimics the organization of the generated types file, such as
{
[schemaName]: {
.. "Tables": {
.... [tableName]: {
....... [rowname]: string | number | boolean | null | {
............. ...typeJsonStructure
.......... } | {
............. ...typeJsonStructure
.......... }[]
...... }
.. },
.. "Views": {},
.. "Functions": {},
}
An example file would be like
{ public: {     Tables: {         user: {             myJsonColumn: {                 customParamOne: string                 customParamTwo: number             }         }     },     Views: {         userView: {             viewJsonColumn: {                 viewColOne: string:                 viewColTwo: {                     nestedOne: boolean;                     nestedTwo: number | string'                 }             }         }     } }
Describe alternatives you've considered
I have not seen anything related to JSON specifically, but @magick93 suggested support for Custom PostgreSQL types in this issue https://github.com/supabase/cli/issues/531

Hey @codewriter73, we've updated the docs to include the recommended way to merge/replace types. With this, you write the generated types to one file (e.g. database-generated.ts) and write the replacements in another file that imports the former (e.g. database.ts), so that way you don't have to keep manually replacing the types.

@soedirgo Thanks for pointing that out! I'll have to look into it more. I ended up writing my own script that uses typescript's compiler API to compare the generated DatabaseTypes.ts and my own AugmentTypes.ts to replace the types and output to a clean types file. I messed around with using abstract types to merge the two but I found it made it difficult to use typescript's language feature when trying to get a quick glance at the return types, etc


ark_java fails when tar is not installed

The Docker image centos:6 doesn't seem to have tar installed by default. When running the java cookbook, the java_ark provider fails on the extraction part. Should it check for the availability of tar and install the package when needed?
Side question: the docs mention that java_ark is deprecated in favor of the ark cookbook, but I don't see any dependency yet. Is this future work?

Yes I suppose that'd make sense as it's a hard dependency when using that provider. If you'd like to take a stab at putting that in and making a PR great! If not let me know and I'll get it in next release.
To your side question - there's no reason to use the java_ark resource directly when there's the ark cookbook.

Submitted a PR to install tar package right before java_ark resource is used


showing "real" diffs?

Pixel diffing is nice, but is there a way to tell this library that I care about "what is different" and not "highlighting the entire document just because there's extra vertical whitespace relatively high up in the image"?
For example, if I have two screenshots of a webpage, one with body { padding-top: 20px; } and one with body { padding-top: 20em; } then the only difference is going to be the extra whitespace. Everything else is identical, so having the entire image flagged as "diff" would be wrong (think of it as a github diff: a change that adds the letter a to the start of a README.md does not change the entire document. All it does is add an a, the rest of the document is still the same as before).
Is there a way to tell blink-diff to find diffs but with maximum shift detection?

Hello, I am not sure if this would work for what you're trying to do, but have you looked at the h-shift capacity? For a simple ignore horizontal diffs (over your entire image, I believe) it works pretty well.

I looked at more permissible h-shift and v-shift values, but that didn't really work. I'm trying to find libraries that do visual diffing in the same way we all understand text diffing should work (e.g. if you move something around, or add some text at the start, that move or add is the diff, not "almost the entire document has changed" =) so I was hoping blink-diff could do that. If it can't, no worries, it just means the search goes on.


After submission comments non-deterministically get stuck in pending despite transitioning to paid

This happens about 1 in 10 comments for me. The pending text flickers when it happens so I suspect there might be a rerender messing something up.
Repro: leave lots of comments in prod non-custodially until one gets stuck in pending, navigate to the comment and confirm it's actually paid.

This should be fixed in #1448


emulate chosen compiler in integer literal promotion

Closes https://github.com/Vexu/arocc/issues/690

@Vexu could you review changes now if you have a chance?

Thanks!


Calling liquibase CLI programmatically from different threads is broken by ThreadLocal changes in Scope

Search first

[X] I searched and no similar issues were found

Description
Since 4.27.0+ we've been seeing several errors like:
2024-07-05 14:10:55.416 ERROR l.i.c.LiquibaseCommandLine - Unknown option: '--show-summary=OFF'
picocli.CommandLine$UnmatchedArgumentException: Unknown option: '--show-summary=OFF'
	at picocli.CommandLine$Interpreter.validateConstraints(CommandLine.java:13662)
	at picocli.CommandLine$Interpreter.parse(CommandLine.java:13614)
	at picocli.CommandLine$Interpreter.processSubcommand(CommandLine.java:13904)
	at picocli.CommandLine$Interpreter.processArguments(CommandLine.java:13800)
	at picocli.CommandLine$Interpreter.parse(CommandLine.java:13591)
	at picocli.CommandLine$Interpreter.parse(CommandLine.java:13559)
	at picocli.CommandLine$Interpreter.parse(CommandLine.java:13454)
	at picocli.CommandLine.parseArgs(CommandLine.java:1552)
	at liquibase.integration.commandline.LiquibaseCommandLine.registerValueProviders(LiquibaseCommandLine.java:605)
	at liquibase.integration.commandline.LiquibaseCommandLine.execute(LiquibaseCommandLine.java:361)

I think I've managed to track the problem down to pull #5617 and specifically this change:
// Old code
private static ScopeManager scopeManager;

// New code
private static final ThreadLocal<ScopeManager> scopeManager = new ThreadLocal<>();

For non-singleton instances I suspect this isn't an issue (else surely there would have been other regressions or bug reports by now!) but it does significantly change the behaviour of Scope.getSingleton: previously this would return a global singleton instance (assuming we were using the default setup) where now it only returns a thread-local singleton - so it's possible for two different threads to see two different "singleton" instances. This in turn breaks any code which both:

relies on Scope.getSingleton, AND
might call liquibase from different threads

The specific bug above manifests because some static initialisers rely on Scope.getSingleton: for example the static initialiser for ShowSummaryArgument ultimately calls CommandArgumentDefinition.build() which contains this line:
Scope.getCurrentScope().getSingleton(CommandFactory.class).register(commandName, newCommandArgument);

Previously this would effectively register the showSummary option(s) globally for the whole JVM; but now the registration only happens for the CommandFactory singleton tied to this thread. And because static initialisers are only run once by the JVM (assuming no classloader shenanigans) that means that any other thread that comes along will get a different instance of CommandFactory which does not have these options registered - hence complaining about "Unknown option".
Steps To Reproduce

call new LiquibaseCommandLine().execute("liquibase update --show-summary=OFF") from thread A
wait for completion
call new LiquibaseCommandLine().execute("liquibase update --show-summary=OFF") from thread B

Expected/Desired Behavior
It should be possible to call into liquibase CLI multiple times inside the JVM from different threads, and have it not crash.
Liquibase Version
4.27
Database Vendor & Version
H2
Liquibase Integration
CLI
Liquibase Extensions
No response
OS and/or Infrastructure Type/Provider
No response
Additional Context
We run liquibase inside the Play web framework which provides a development mode with hot reload: this is extremely useful since it allows developers to edit changelogs while the webserver is running, have it reload, and any liquibase changes will automatically be applied with liquibase update. In order to support this it might end up running its hotswap code (including, in our case, liquibase) from different threads at different times.
Are you willing to submit a PR?

[X] I'm willing to submit a PR (Thank you!)

Hello @ashleymercer ! Thanks for the detailed ticket. Based on your findings I created PR  https://github.com/liquibase/liquibase/pull/6222 .
The added test was failing , but after that static was added to CommandFactory class it started to pass. And in fact it's a caching issue: we are re-reading static allocated parameters to an instance object. By changing it to static we use the caching and nothing is missed.
It the test now behaving as expected?

Hi @filipelautert - thanks for picking this one up! I'll wait for the next release and then test it again on my end


Issue with Bartender 5

Describe the bug
When accessing the Menu Bar Items in Bartender 5, the OnlySwitch icon creates a massive amount of space around itself which then requires you to scroll around it to locate other items in the settings menu.
To Reproduce
Install Only Switch. Open Bartender 5 settings. Under hidden menu bar items or shown menu bar items, you will reach an empty space, if you start scrolling, eventually the OnlySwitch icon will appear.
Expected behavior
There should not be extended space around the OnlySwitch icon.
Screenshots

Desktop (please complete the following information):
MacOS 14.6.1 (23G93)

Note - quitting and restarting Bartender after OnlySwitch install appears to fix the issue.


Update external links

Codecov Report

Merging #29 (eb5dfeb) into main (be04774) will not change coverage.
The diff coverage is n/a.

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

@@           Coverage Diff           @@
##             main      #29   +/-   ##
=======================================
  Coverage   26.74%   26.74%
=======================================
  Files          55       55
  Lines         602      602
  Branches      129      129
=======================================
  Hits          161      161
  Misses        390      390
  Partials       51       51

Impacted Files
Coverage Δ

...c/components/analytics/configure-analytics-card.js
33.33% <ø> (ø)

src/components/automation-hub/hub-card.js
6.12% <ø> (ø)

src/components/catalog/configure-catalog_card.js
33.33% <ø> (ø)

src/components/dashboard/configure-app-page.js
33.33% <ø> (ø)

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 be04774...840d23f. Read the comment docs.


updated go version to 1.21.8

Purpose of PR?:
This PR updates go version to latest one 1.22 including the CI.
Fixes #
Does this PR introduce a breaking change?
If the changes in this PR are manually verified, list down the scenarios covered::
Additional information for reviewer? :
Mention if this PR is part of any design or a continuation of previous PRs
Checklist:

[ ] Bug fix. Fixes #
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update
[ ] PR Title follows the convention of  <type>(<scope>): <subject>
[ ] Commit has unit tests
[ ] Commit has integration tests

Are the errors in CI due to old go.sum? cc @DelusionalOptimist

controller-gen is crashing with go 1.22. So our CI is also failing due the same reason.
https://github.com/golang/go/issues/65637

@VedRatan use kubernetes-sigs/controller-tools#880 , basically update the controller-gen module to 0.14 to fix it

Yes I am upgrading the controller-gen, will be pushing the changes now.

Sorry @VedRatan... had to take this over as it was blocking CI for all other PRs for the upcoming release. : (

No issues @DelusionalOptimist


merge package template from arlac77/npm-package-template

package.json

chore(devDependencies): remove<EMAIL_ADDRESS>chore(devDependencies): remove<EMAIL_ADDRESS>chore(devDependencies): add<EMAIL_ADDRESS>from template
chore(devDependencies): update<EMAIL_ADDRESS>from template
chore(scripts): update docs@documentation readme src/service-health-check.js --section=API from template
chore(scripts): add lint@documentation lint src/service-health-check.js from template

.travis.yml

chore(travis): merge from template .travis.yml

README.md

docs(README): update from template

Coverage remained the same at 69.388% when pulling f7f8849a7c25c64761349c7193943b096ae4b045 on template-sync-1 into ec72e2466f619f5adc818031e2de246e7c1b748c on master.

Coverage remained the same at 69.388% when pulling f7f8849a7c25c64761349c7193943b096ae4b045 on template-sync-1 into ec72e2466f619f5adc818031e2de246e7c1b748c on master.


Cursor going out of focus.

Hi Team,
I am using Jodit v 1.0.51, as soon as I place my cursor and make a single change say to add or remove a character, the cursor is going out of focus so again I need to click and need to do the changes.

Duplicate of https://github.com/jodit/jodit-react/issues/43, take a look :)

I fixed some of the known issues of Jodit in this repository, including this one.
See if it helps:
https://github.com/ShahrozeAli/react-jodit-fixes


[vcpkg baseline][log4cpp] Update version record

PR #15650 forgot to update the version record.
Update now.

Ping @ras0219 @ras0219-msft for merge this PR quickly.

Thanks!


Components are tough to edit

I love the concept, got it all setup and started editing.
I found a block I mostly liked but I wanted to remove a button in it, I couldn't
when I went to craft.js it seemed to be able to edit everything, remove a button no problem.
it also has a "customize" area with layers I can;t seem to access in destack.
am I doing something wrong to not see these features?

Thanks! You are not doing anything wrong.
Although Craft.js has an example with some styling this library is different. The reason is that the library has hundreds of components that I couldn't add custom style editors for.
Although this is currently very hard to do, there's an ongoing migration to v3 https://github.com/LiveDuo/destack/issues/104. This major update moves the library away from Craft.js and into a new vanilla HTML editor. The new editor will let you just edit the HTML as the use case you described.
You can use it right now here but note that there are multiple bugs in the editor that haven't been fixed yet.

@SuddenDevelopment The next major update v3 is out now and uses HTML as the templating language. That means if there's a button or anything else you want to change you can either change it in your browser DevTools or edit the template directly.

@LiveDuo you might want to take a look at Puck, their editor is very good but they lack templates.

@Immortalin A react based editor like Puck is something that has been considered before (ie. in v2). It comes with multiple complexities such as having to convert the HTML based component to react components and the increment in the bundle size. See https://github.com/LiveDuo/destack/issues/104 for more details.
The current alternative which is HTML based is better in a lot of different facets and allows to edit the HTML either by hand or using the browser DevTools.

The current editor's drag and drop user experience has a lot of room for improvement compared to Puck though

The popular browsers like Chrome, Safari and Firefox already have this functionality in DevTools and is a very flexible today to edit CSS styles or HTML. I highly suggest you try it out.

Puck maintainer here! Just stumbled across this, nice work.
FWIW, I think you could probably create a compatibility layer for the HTML -> React components, without to rewrite all the components. However, appreciate that it is indeed additional complexity.
On-hand if you need any input :)

Hey @chrisvxd,
Glad to have you around and appreciate the work you do on Puck.
Just for the record, this project was under a similar paradigm in v2 with Craft.js. That has a compatibility layer converting the HTML components to Craft.js syntax to render. In contrast, the current version has HTML components and a React based editor.
The goal of the project is to have it all in vanilla JS/HTML for 1) compatibility across all frontend frameworks and 2) source code simplicity and 3) longevity of the project. There's no much information about this right now but I'd write a more detail post about this in the next few days.
PS: The way to edit components now is to use the browser DevTools:
https://private-user-images.githubusercontent.com/370612/318197307-430540a1-b65d-4be6-8091-fae88d14a41a.mov


The application freezes when executing db.crypto

I am now developing an app using ionic 1 with pouchDB and pouchdb-adapter-cordova-sqlite. The app is freezes for about 10 seconds even the UI and the database is empty in iPhone 6s. I am tried it with the database with about 100+ records, it takes about 20 minutes.
Can I know is there something I missed, and what can I do in order to improve it?
My code as following:
`
$rootScope.DeviceReady.promise.then(function(){
            var db = new PouchDB(name,{
              auto_compaction: true,
              adapter: 'cordova-sqlite',
              iosDatabaseLocation: 'default',
              androidDatabaseImplementation: 2
            });

             db.crypto( 'password' );                        // the app freezes after this

}
`
library version:
iOS 9.3.5
angular#1.5.3
pouchdb#6.0.5
pouchdb-adapter-cordova-sqlite
crypto-pouch

it's running the key derivation in the main thread and safari web view doesn't support crypto api so it's all being done in javascript

I'm running into the same issue. It works like a dream on Android devices. However, on iOS the login process comes to a halt for 1-3 minutes while it does the db.crypto() command.
After reading this issue and the following part of the readme:

Chacha20-Poly1305 is also available and previous versions defaulted to this algorithm. You might consider using this if your app will primarily be used in browsers that don't support the web crypto api (e.g. safari).

I tried to make it more performant on my iOS devices by encrypting with this option:
options.algorithm = 'chacha20';
I'm not really noticing any difference however. Am I implementing that correctly, or is there something else I need to be doing in order to get this functioning properly?
Do you have any other suggestions to make the login/encryption process more bearable for the user? Thanks in advance.

so it's not the encryption that is the issue, it's the key derivation e.g. turning your human readable text into a binary bits.  You can do the key derivation your self if you want and and pass in a 32 byte browserify buffer as a key parameter of the options (you can omit the password and just pass the options as the first parameter)
So you can do something like
var buf = createBufferSomehow();
console.log(Buffer.isBuffer(buf));// needs to be true
console.log(buf.length);//needs to be 32
db.crypto( {key: buf});
// or since you are on safari
db.crypto( {key: buf, algorithm: 'chacha20'});

you can use this along with a key you derived some other way from your password, if you have access to like a web worker or whatever the safari web view equivalent is, you can use pbkdf2 in a background thread (that's the algo we use) or some other algo to generate the key from the password.

Hi Calvin,
I'm using crypto-pouch to encrypt the pouch-db's we use in an Ionic app but I'm also having issues with the cryptography being terribly slow on iOS devices. I've tried your suggestion and created a buffer using forge
var salt = forge.random.getBytesSync(128);
var key = forge.pkcs5.pbkdf2(password, salt, 100000, 32);
db.crypto({ key: key, algorithm: 'chacha20'});

100000 iterations was not manageable (also not on Android), 1000 iterations took about 2-3 seconds computing time on iOS. Using this generated key as an option for the db.crypto is not faster than without passing an options object.
Do you know of any other options I can try?
P.S: the plugin works flawlessly on Android. And it's awesome that it's just a 'feature toggle'.

the other option is to use a web worker to, where you could open the
database in that thread and have it receive messages from your main thread
and send back the decrypted data from the database
On Thu, Apr 6, 2017 at 9:07 AM wouterSeyen<EMAIL_ADDRESS>wrote:

Hi Calvin,
I'm using crypto-pouch to encrypt the pouch-db's we use in an Ionic app
but I'm also having issues with the cryptography being terribly slow on iOS
devices. I've tried your suggestion and created a buffer using forge
https://github.com/digitalbazaar/forge
var salt = forge.random.getBytesSync(128);var key = forge.pkcs5.pbkdf2(password, salt, 100000, 32);db.crypto({ key: key, algorithm: 'chacha20'});
100000 iterations was not manageable (also not on Android), 1000
iterations took about 2-3 seconds computing time on iOS. Using this
generated key as an option for the db.crypto is not faster than without
passing an options object.
Do you know of any other options I can try?
P.S: the plugin works flawlessly on Android. And it's awesome that it's
just a 'feature toggle'.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/calvinmetcalf/crypto-pouch/issues/53#issuecomment-292168540,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABE4n7VIekK2gEUGcwa0kSyhbDTRv_00ks5rtOOKgaJpZM4Mh_dt
.

Howdy, folks! We just released version 4.0 which changes a lot of things. Could you try again with the new version and let me know how it goes?
Since it's been four years, I'm going to close this issue for now. Let me know if you'd still like help!


fix: pgvector hang risk if table not exist #3883

FYI - had an error pop up after upgrading to the latest rev - works fine after commenting out the commit.
langchain/vectorstores/pgvector.py", line 168, in create_tables_if_not_exists
    self._conn.commit()
AttributeError: 'Connection' object has no attribute 'commit'

Hi @hanguofeng ,
Appreciate the quick response.
Thank you for your help.
-krb

I bumped up SQLAlchemy from 1.4.47 to 2.0.12
using PGVector 0.1.6, LangChain 0.0.160


issue with blank-java

there is no policy name 'AWSXrayWriteOnlyAccess' so that run deply.sh will be failed

Hey @chenkaijiekidd ,the 'AWSXrayWriteOnlyAccess' policy is present in the template.yml.
Please help me with the details, to understand the problem you're facing.


send-hits-to-discord/[REQUEST]

I want to send hits from other user to my discord from my config when they use my config the hits I want plz to get them in my discord any help

It's a planned feature. Just wait or implement it yourself if you cannot wait.

you can use discords webhook and implement that into your config to send hits pretty simple but it is frowned upon to hit log

You can do this easily by using the Discord Webhooks, but i dont advice this as it might get your account and/or server banned.


Parse() and getConfig() don't recognize helper classes

The AutoAtom feature's methods parse() and getConfig() are not matching helper classes such as Bfc
fyi @renatoi @thierryk

Need to make sure we have tests that include these classes going forward too

@src-code I'm not sure we should bother with helper classes. In my opinion, helpers.css should be treated the same as base.css. Also because we are not sure what would make sense to include and what the proper syntax should be. Like a base style sheet, some people may have their own helper sheet already.
I'm more concerned by the oldIE fallback for display: inline-block and the ib-box (for inline-block box) class we want to offer to replace D-ib Va-t and provide the oldIE patch as well. How should we do this? Unlike the helpers style sheet, I believe those should be part of atomic.css. May be "baked in", with any atomizer output?

I think this one is outdated, as we've talked about bringing back helpers but in their own ruleset?  But for now, Bfc and such have been removed from the Atomic ruleset.
Issue #80 covers the oldIE issue you raised here.


non-optimal alignment

ABACABA and ABA should be when taking into account the affine gap penalty aligned like:
ABACABA
ABA----
or
ABACABA
----ABA
BUT NOT:
ABACABA
AB--A--

This issue calls for a better debug output: https://github.com/kjenova/fogsaa/commit/756f8e2977a4afd935b392a5b80784ba96f1e083

The output seems OK now:
ABACABA
ABA----
x: 8, y: 4, score: -2
But why do we even need Ox and Oy!? We know, that the optimal alignment is going to be in cells[(endof(X) + 1, endof(Y) + 1)]!

ABAXXXABA XXXBA
ABAXXXABA
---XXXBA-
score: -5
This is not optimal, this is:
ABAXXXABA
---XXX-BA
score: -1

A
AB
ABA
ABAX
---X
ABAXX
---XX
ABAXXX
---XXX
ABAXXXA
---XXXB
ABAXXXAB
---XXXBA
ABAXXXABA
---XXXBA-
^
A
X
AB
X-
ABA
X--
ABAX
X--X
ABAXX
X--XX
AB
XX
ABA
XX-
ABAX
XX-X
I do not believe, that the queue works fine. Should add information about what was pushed on the queue. The case simplifies to
ABA
AB-

Yay, I think I know what's wrong:
	ret = t
	t.align = Align.undef

in pop()

jan@0:~/fogsaa$ julia fogsaa.jl ABA BA
ABA
B?-
ABA
BA-
jan@0:~/fogsaa$ julia fogsaa.jl BBA BA
BBA
B?-

@1: (2, 2); [-2, -2]

BBA
BA-
BBA
BA-
^ -2
< @1
jan@0:~/fogsaa$
doesn't even push other options for first case on the queue ...

Well, this is completely broken:
XY: [0, -2]
Xy: [-4, -4]
xY: [-4, -6]
BBA
B?-
XY: [-2, -2]
Xy: [-2, -2]
xY: [-4, -4]

@1: (2, 2); [-2, -2]

BBA
BA-
Xy: [-2, -2]
BBA
BA-
^ -2
< @1

Heck, we do not even need to calculate m! and M! like we do, we could recursively calculate it like in lemmas 1-3 here: http://www.nature.com/srep/2013/130429/srep01746/full/srep01746.html#methods

Still doesn't work...
XY: [0, -2]
Xy: [-2, -6]
xY: [-5, -5]

@1: (1, 1); [-2, -6]
XY: [-2, -2]
Xy: [-2, -4]
xY: [-5, -3]
@2: (2, 2); [-2, -4]
^ 0
< @2
ERROR: convert has no method matching convert(::Type{CellType}, ::Type{CellType})
in anonymous at /home/jan/fogsaa/fogsaa.jl:316
in include at ./boot.jl:245
in include_from_node1 at loading.jl:128
in process_options at ./client.jl:285
in _start at ./client.jl:354
while loading /home/jan/fogsaa/fogsaa.jl, in expression starting on line 251

New rule: we won't even count the gap, that is formed, because endof(X) != endof(Y), as a Score.gap_start

Eureka! It should be counted in gm, because that could mean it is in the middle of the sequence, but not in gM!

Nah, we should just add a bool telling us whether the first gap had been opened.

Well, BBA works now. But:
XY: [-4, -6]
Xy: [-2, -6]
xY: [-7, -7]

@1: (1, 1); [-4, -6]

ABBAC
-BA--
^ -1
< @1
Doesn't work.

 I meant branches instead of align all the time :)

ABB BB
XY: [-2, -4]
Xy: [0, -4]
xY: [-5, -5]

@1: (1, 1); [-2, -4]
XY: [0, -2]
Xy: [0, -4]
xY: [-5, -5]
@2: (2, 1); [0, -4]
XY: [0, 0]
Xy: [-1, -3]
xY: [-5, -3]
@3: (3, 2); [-1, -3]
ERROR: key not found: (0x0000000000000004,0x0000000000000004)
in getindex at dict.jl:617
in print_alignment at /home/jan/fogsaa/fogsaa.jl:213
in main at /home/jan/fogsaa/fogsaa.jl:252
in include at ./boot.jl:245
in include_from_node1 at loading.jl:128
in process_options at ./client.jl:285
in _start at ./client.jl:354
while loading /home/jan/fogsaa/fogsaa.jl, in expression starting on line 400

XY: [-4, -6]
Xy: [-2, -6]
xY: [-7, -7]

@1: (1, 1); [-4, -6]
XY: [-2, -4]
Xy: [-2, -6]
xY: [-7, -7]
@2: (2, 1); [-2, -6]
XY: [-4, -4]
Xy: [-3, -5]
xY: [-7, -5]
@3: (3, 2); [-4, -4]
XY: [-3, -3]
Xy: [-3, -5]
xY: [-8, -6]
@4: (4, 2); [-3, -5]
Xy: [-4, -4]

ABBAB
-B-A-
^ -3
< @2

I think that cells[endof(X) + 1, endof(Y) + 1] somehow isn't modified, because the score and alignment can't be the same twice:
XY: [-4, -6]
Xy: [-2, -6]
xY: [-7, -7]

@1: (1, 1); [-4, -6]
XY: [-2, -4]
Xy: [-2, -6]
xY: [-7, -7]
@2: (2, 1); [-2, -6]
XY: [-4, -4]
Xy: [-3, -5]
xY: [-7, -5]
@3: (3, 2); [-4, -4]
XY: [-3, -3]
Xy: [-3, -5]
xY: [-8, -6]
@4: (4, 2); [-3, -5]
Xy: [-4, -4]

ABBAB
-B-A-
^ -4
< @2
Xy: [-4, -4]
XY: [-4, -2]
Xy: [-4, -4]
xY: [-9, -5]
ABBAB
-B-A-
^ -4
< @4

ABBAB | AB doesn't print

I finally know what I fucked up: cells[(1, 1)] changes and that gives us a wrong M in pop(). Heck,  I should not have even let Queue see cells, I should not have even initialised cells[(1, 1)] — should have done that in main() — that would even probably lead to an optimization.

I knew it. We overwrite alignments["cross path" x, y] before we know that the popped alignments is optimal.

I think overwriting scores won't be a problem. It would mean: "we reached this score, but still we couldn't best the optimal alignment."

Because we know that the new alignment first diverges from the old one, then goes into uncharted territory and then maybe (if the score is better) merges with the old one, there will only be one road fork.


Getting Error while installing "Call to Undefined method Illuminate\Foundation\Application::share()"

Hi,
i was installing , but got "Call to Undefined method Illuminate\Foundation\Application::share()"
log shows
[2017-03-05 18:25:21] local.ERROR: exception 'Symfony\Component\Debug\Exception\FatalErrorException' with message 'Call to undefined method Illuminate\Foundation\Application::share()' in C:\xampp\htdocs\WooAccountManager\vendor\pixelpeter\laravel5-woocommerce-api-client\src\WoocommerceServiceProvider.php:44 Stack trace: #0 {main}
my composer.json is
{     "name": "laravel/laravel",     "description": "The Laravel Framework.",     "keywords": ["framework", "laravel"],     "license": "MIT",     "type": "project",     "require": {         "php": ">=5.6.4",         "laravel/framework": "5.4.*",         "laravel/tinker": "~1.0",         "pixelpeter/laravel5-woocommerce-api-client": "2.0"     },     "require-dev": {         "fzaninotto/faker": "~1.4",         "mockery/mockery": "0.9.*",         "phpunit/phpunit": "~5.7"     },     "autoload": {         "classmap": [             "database"         ],         "psr-4": {             "App\\": "app/"         }     },     "autoload-dev": {         "psr-4": {             "Tests\\": "tests/"         }     },     "scripts": {         "post-root-package-install": [             "php -r \"file_exists('.env') || copy('.env.example', '.env');\""         ],         "post-create-project-cmd": [             "php artisan key:generate"         ],         "post-install-cmd": [             "Illuminate\\Foundation\\ComposerScripts::postInstall",             "php artisan optimize"         ],         "post-update-cmd": [             "Illuminate\\Foundation\\ComposerScripts::postUpdate",             "php artisan optimize"         ]     },     "config": {         "preferred-install": "dist",         "sort-packages": true     } }

You're using the wrong version. Error is because of Laravel 5.4.
This module is compatible with Laravel 5.4 since v2.2.0.
Please update your composer.json to the correct version constraint composer require pixelpeter/laravel5-woocommerce-api-client ^2.0


Fix anchors in README.md

Summary:
Currently all anchors in README.md cause address change. E.g. style link in Props section should lead to
https://github.com/react-native-community/react-native-slider#style

but leads to
https://github.com/react-native-community/react-native-slider/blob/master/README.md#style

This PR fixes that.
Test Plan:
Try clicking on the same link in Props section with the fix.

Thanks 🏅


Sync command not interpreting source files properly

Which version of the AzCopy was used?
Note: The version is visible when running AzCopy without any argument
10.0.8
Which platform are you using? (ex: Windows, Mac, Linux)
Windows
What command did you run?
Note: Please remove the SAS to avoid exposing your credentials. If you cannot remember the exact command, please retrieve it from the beginning of the log file.
sync .\garbage https://wilxstorage.blob.core.windows.net/wilxcontainer --delete-destination true
What problem was encountered?
source files not interpreted correctly. My source directory(garbage) has two files in it: file.txt and file2.txt
Yet in my log file I see:
Starting transfer: Source "./garbage/garbage/file.txt" Destination "https://wilxstorage.blob.core.windows.net/wilxcontainer/garbage%2Ffile.txt". Specified chunk size 8388608
It looks like the directory name is being appended twice to form source paths, and then because sync cannot find those malformed source paths, it fails.
This also causes ALL destination blobs to be deleted if the delete-destination flag is specified, as there are no matches.
How can we reproduce the problem in the simplest way?
Run the command with a source directory and destination container
Have you found a mitigation/solution?
No

Hi @williexu, thanks for letting us know!
I was able to repro and confirmed that it's an issue specific to relative paths on Windows. I tried the same thing on MacOS and it worked fine. Please use full paths for now as a workaround.
I also realize that we have a lack of testing for relative paths, I'll look into how we can improve this. Thanks for the feedback!

Hi @williexu, I've fixed this issue for the sync command. It was a string processing bug. Really sorry for the inconvenience. I'll try to figure out how to add relative paths to our testing so that they are covered as well.

Fixed in 10.0.9.


How to nest these genericlinks?

I'd like to be able to have a genericlink as a property of a genericlink in order to create hierarchies for navigation.  Do you have any suggestions... I notice that the Property Types links https://github.com/Geta/geta-optimizely-genericlinks/blob/master/docs/advanced-property-handling.md#supported-property-types explicitly excludes blocks...

Because of the way this data is persisted, like a link with properties being attributes, makes nesting impossible. There is also the problem of getting the UI to handle nesting. The entire editor would have to be rebuilt from the ground up.


Literal string 'assign' and 'assume' policies should be re-implemented in troposphere

The current base 'assign' and 'assume' policies are created as literal strings in the code.  These should be re-implemented as troposphere/awacs Template Policy Statements.
Context
This is a first step towards more modular permissions, as it allows for more programmatic manipulation of the permissions, instead of simply appending to a dictionary generated from JSON or YAML.
Expected/Actual Behavior
No expected behavior change.

Yeah, I think this is a good idea I've never liked having these big static strings in the code anyway. There are some legacy policies still hardcoded in there as well, I think.
We should have a chat about the larger direction of this as well.


How to get 'after_move:filepath' info from yt-dlp embedded in a Python script?

DO NOT REMOVE OR SKIP THE ISSUE TEMPLATE

[X] I understand that I will be blocked if I intentionally remove or skip any mandatory* field

Checklist

[X] I'm asking a question and not reporting a bug or requesting a feature
[X] I've looked through the README
[X] I've verified that I'm running yt-dlp version 2023.07.06 (update instructions) or later (specify commit)
[X] I've searched known issues and the bugtracker for similar questions including closed ones. DO NOT post duplicates
[X] I've read the guidelines for opening an issue

Please make sure the question is worded well enough to be understood
I have no problem running yt-dlp embedded in python except for the issue of getting the final file path after post-processing.
If I run, for example, "yt-dlp https://www.youtube.com/watch?v=CHviUkAXVqA --print after_move:filepath --extract-audio --audio-format mp3 -f 'bestaudio[ext=mp3]/bestaudio'" in a linux terminal I get the final mp3 file path.
Despite spending many hours poring through the yt-dlp source, I have not been able to work out how to achieve the same result in python. I know it requires setting the correct option, but I am at a loss for exactly what that should be.
I've tried various combinations of ydl_opts['forceprint'] = {'after_move': '%(filepath)s'}, and then ydl_opts['forceprint'] = {'video': {'after_move': '%(filepath)s'}} but none of them works. Each combination results in one or more 'NA' printed to the terminal. I'm just guessing and getting nowhere so could somebody please advise how to achieve the '--print after_move:filepath' result in a python script?
Provide verbose output that clearly demonstrates the problem

[ ] Run your yt-dlp command with -vU flag added (yt-dlp -vU <your command line>)
[X] If using API, add 'verbose': True to YoutubeDL params instead
[X] Copy the WHOLE output (starting with [debug] Command-line config) and insert it below

Complete Verbose Output
[debug] Encodings: locale UTF-8, fs utf-8, pref UTF-8, out utf-8, error utf-8, screen utf-8
[debug] yt-dlp version<EMAIL_ADDRESS>[b532a3481] (pip) API
[debug] params: {'format': 'bestaudio[ext=mp3]/bestaudio', 'outtmpl': '/tmp/%(title)s.2023-08-15_17-41-11.%(ext)s', 'forceprint': {'video': {'after_move': '%(filepath)s'}}, 'verbose': True, 'postprocessors': [{'key': 'FFmpegExtractAudio', 'preferredcodec': 'mp3'}], 'compat_opts': set()}
[debug] Python 3.10.12 (CPython x86_64 64bit) - Linux-5.15.0-78-generic-x86_64-with-glibc2.35 (OpenSSL 3.0.2 15 Mar 2022, glibc 2.35)
[debug] exe versions: ffmpeg N-111639-gb300552dfb (fdk,setts), ffprobe N-111639-gb300552dfb
[debug] Optional libraries: Cryptodome-3.18.0, brotli-1.0.9, certifi-2020.06.20, mutagen-1.46.0, secretstorage-3.3.1, sqlite3-2.6.0, websockets-11.0.3
[debug] Proxy map: {}
[debug] Loaded 1855 extractors
[youtube] Extracting URL: https://www.youtube.com/watch?v=CHviUkAXVqA
[youtube] CHviUkAXVqA: Downloading webpage
[youtube] CHviUkAXVqA: Downloading ios player API JSON
[youtube] CHviUkAXVqA: Downloading android player API JSON
[youtube] CHviUkAXVqA: Downloading m3u8 information
[debug] Sort order given by extractor: quality, res, fps, hdr:12, source, vcodec:vp9.2, channels, acodec, lang, proto
[debug] Formats sorted by: hasvid, ie_pref, quality, res, fps, hdr:12(7), source, vcodec:vp9.2(10), channels, acodec, lang, proto, size, br, asr, vext, aext, hasaud, id
[info] CHviUkAXVqA: Downloading 1 format(s): 251
NA
[debug] Invoking http downloader on "https://rr3---sn-u2bpouxgoxu-coxd.googlevideo.com/videoplayback?expire=1692106874&ei=GizbZJ3hCeSa1d8P96Wb8Aw&ip=<IP_ADDRESS>&id=o-AHmbRm4TSXEaDNFnykYtNSTSBs723fClTACsnfZSw1jD&itag=251&source=youtube&requiressl=yes&mh=e_&mm=31%2C29&mn=sn-u2bpouxgoxu-coxd%2Csn-ntqe6n7k&ms=au%2Crdu&mv=m&mvi=3&pl=19&initcwndbps=1963750&spc=UWF9f5eS38tW9dG9GPGb4h4zGdGrmDQ&vprv=1&svpuc=1&mime=audio%2Fwebm&gir=yes&clen=382836&dur=24.181&lmt=1634049985162546&mt=1692084797&fvip=1&keepalive=yes&fexp=24007246%2C24362685&c=ANDROID&txp=5432434&sparams=expire%2Cei%2Cip%2Cid%2Citag%2Csource%2Crequiressl%2Cspc%2Cvprv%2Csvpuc%2Cmime%2Cgir%2Cclen%2Cdur%2Clmt&sig=AOq0QJ8wRgIhALtUvGFsFdo5_zfVDRxxUxOY91E_ckXMR8vDE2VmD7EcAiEA10QMPQmf5K-VpAuAxFcCuXYHfXKbCtVy3wByhTu5Qj0%3D&lsparams=mh%2Cmm%2Cmn%2Cms%2Cmv%2Cmvi%2Cpl%2Cinitcwndbps&lsig=AG3C_xAwRAIgbgrLsvvx58cDGISLC7f5oaXDz45Bla2aXFSP-kA-HEYCIAZNkny7wtQBc2koPmNZbQqen_qf0sr77qgnnnShIqTR"
[download] Destination: /tmp/＂Hey＂ -a VERY short Film.2023-08-15_17-41-11.webm
[download] 100% of  373.86KiB in 00:00:00 at 1.79MiB/s
[debug] ffmpeg command line: ffprobe -show_streams 'file:/tmp/＂Hey＂ -a VERY short Film.2023-08-15_17-41-11.webm'
[ExtractAudio] Destination: /tmp/＂Hey＂ -a VERY short Film.2023-08-15_17-41-11.mp3
[debug] ffmpeg command line: ffmpeg -y -loglevel repeat+info -i 'file:/tmp/＂Hey＂ -a VERY short Film.2023-08-15_17-41-11.webm' -vn -acodec libmp3lame -movflags +faststart 'file:/tmp/＂Hey＂ -a VERY short Film.2023-08-15_17-41-11.mp3'
Deleting original file /tmp/＂Hey＂ -a VERY short Film.2023-08-15_17-41-11.webm (pass -k to keep)

I think I've found the answer by trying almost every combination I could think of.
I found that most variations produced a wrong answer in the wrong part of the output, but some of them using ydl_opts['forceprint'] = {'after_move':  ...}
produced a wrong answer in the right part of the output.
By repeating that with different values and object types I eventually hit on:
ydl_opts['forceprint'] = {'after_move': ['filepath']}
That produced the correct answer as the last line of the output.
By adding:
import io
from contextlib import redirect_stdout

and
    f = io.StringIO()
    with redirect_stdout(f):
        with YoutubeDL(ydl_opts) as ydl:
            exitcode = ydl.download(url)

    out = f.getvalue()
    lastline = out.splitlines()[-1]

I have been able to get the final file path in the variable 'lastline' for further processing.

you could also do it like this
with YoutubeDL(ydl_opts) as ydl:
    info = ydl.extract_info(url)

filepath = info['requested_downloads'][0]['filepath']

Thank you! That's exactly what I've been looking for all this time. That's much better than my klunky attempt to extract the final filepath from stdout.
Is there some documentation anywhere that explains or lists such options for people not already familiar with the internals of yt-dlp?

extract_info is documented in https://github.com/yt-dlp/yt-dlp#extracting-information. For the field name, you could look at print/json.dump of the generated info.


VSCode lags and freezes, when OBS and Firefox are open in the background, or when left open for a long time.

VSCode gets so slow and laggy that a character that I type, appears on screen even after 5-6 seconds, and ultimately becomes unusable.
I notice this happening in two circumstances.
Mainly, when I've got Firefox open and OBS Studio recording a single tab (!), it slows down to the point that typing appears on screen often after 10 seconds.
It does so, not at the beginning of the recording, but well into it.
It stops slowing down when I close OBS and Firefox, and do a restart of VSCode.
The other circumstance when it freezes and lags is when I have it open with a single python file with 500 or so lines, and I leave VSCode open for too long. Even hitting enter to create a new line takes a second or two.
After a close,and re-open it works like new though.

Same over here

Please follow https://github.com/microsoft/vscode/wiki/Performance-Issues for reporting perf issues.

Will close current issue an re-report it correctly. Thanks @octref


fix troubleshoot button hover effect

📝 Description
This PR fix the hover effect of the troubleshoot USB button
❓ Context

Impacted projects: ledger-live-desktop
Linked resource(s): FAT-235

✅ Checklist

[ ] Test coverage
[X] Atomic delivery
[X] No breaking changes

📸 Demo
https://user-images.githubusercontent.com/29443638/189906330-5bd86960-ada4-4c55-ae7e-1642ab62b9b7.mp4

🚀 Expectations to reach
Please make sure you follow these Important Steps.
Pull Requests must pass the CI and be internally validated in order to be merged.

QA OK, fixed, can be merged


Old link broken(Further reading). Corrected link

Why:
Old link broken in Further reading: Sorting search results
corrected link

What's being changed:
In docs/content/search-github/searching-on-github/
Update file:
before old broken link: Sorting search results by later new link: Sorting search results

Check off the following:

[ Yes] I have reviewed my changes in staging (look for "Automatically generated comment" and click Modified to view your latest changes).
[ Yes] For content changes, I have completed the self-review checklist.

Writer impact (This section is for GitHub staff members only):

[ ] This pull request impacts the contribution experience

[ ] I have added the 'writer impact' label
[ ] I have added a description and/or a video demo of the changes below (e.g. a "before and after video")

@santoslopez
Thanks so much for opening a PR! I'll get this triaged for review :zap:

The original file is fine, I made a mistake viewing it.


2.0.0 to 2.0.3 update failure

Windows Version
Microsoft Windows [Version 10.0.19045.3448]
WSL Version
<IP_ADDRESS>
Are you using WSL 1 or WSL 2?

[X] WSL 2
[ ] WSL 1

Kernel Version
<IP_ADDRESS>
Distro Version
No response
Other Software
No response
Repro Steps
Run wsl --update --web-download --pre-release
Expected Behavior
Successful update
Actual Behavior
Fails with Error code: Wsl/UpdatePackage/E_ABORT
Diagnostic Logs
PS C:\Users\elsaco> wsl --version
WSL version: <IP_ADDRESS>
Kernel version: <IP_ADDRESS>-1
WSLg version: 1.0.57
MSRDC version: 1.2.4485
Direct3D version: 1.608.2-61064218
DXCore version: 10.0.25880.1000-230602-1350.main
Windows version: 10.0.19045.3448
PS C:\Users\elsaco> wsl --update --web-download --pre-release
Checking for updates.
Updating Windows Subsystem for Linux to version: 2.0.3.
Update failed (exit code: 1603).
Log file: C:\Users\elsaco\AppData\Local\Temp\wsl-install-logs.txt
Error code: Wsl/UpdatePackage/E_ABORT

wsl-install-logs.txt

Log of my failure

I had same failure, but https://github.com/microsoft/WSL/issues/10574#issuecomment-1743161774 gave the workaround: download the msi and upgrade that way ...

Thank you for reporting this. The issue is that the installation was not run as administrator:
Error 1925. You do not have sufficient privileges to complete this installation for all users of the machine.  Log on as administrator and then retry this installation.

This is already fixed in 2.0.3

Actually,  --web-download --update did not work even when run as administrator!  Contrary to non-admin no error was reported but version stayed 2.0.0.  Only .msi install worked for me to upgrade ...

I just tried to update from 2.0.2->2.0.3 and still needs to be administrator to update.

I just tried to update from 2.0.2->2.0.3 and still needs to be administrator to update.

That's expected. Updating WSL via the .msi requires administrator. You can still update via the store though


contributorRole/creatorRole links to deprecated controlled vocab

The pages for <creatorRole> and <contributorRole> link to deprecated controlled vocabulary lists.
They could both link to the current list that covers both tags?
Related to this, the examples show links to deprecated values.

Thank you for catching this issue! We've made the quick fix on the link, but please be aware that we're currently in the process of re-launching the website now, so examples and page layouts will change in the near future.

Awesome, thanks!


Why not use Lombok?

Why don't we try Lombok and reduce the boilerplate code like getters, setters, constructor creation etc..?
<dependency>
  <groupId>org.projectlombok</groupId>
  <artifactId>lombok</artifactId>
  <optional>true</optional>
</dependency>

Hello friend. I think it's a good suggestion to use Lombok dependency.
It can help the project to be more readable too.
If there is not any problem I'd be glad to work on this issue.
@RaveenDharmasiri

We're not going to start a debate around lombok here. We have no intention of using it.


require-jdk: migrate from AdoptOpenJDK API to Adoptium

The AdoptOpenJDK API has been deprecated for some time in favour of the Eclipse Temurin API

 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.

Can one of the admins verify this patch?


Replace packaging of Dotnet.ReproducibleBuilds via nuget in favor of dotnet pack

On my local machine the nuget pack invocation was throwing, so I converted all packaging to rely on native dotnet pack features.
Before:

After:

Before:

After:

The layouts, descriptions, transitive package dependencies, etc are the same.  Bonuses of this change include onboarding to SPDX License expressions and linking READMEs.

I also added release notes in the packages in preparation for another release.


issue with command $ vcpkg install OpenCL on Mac OS X

[Mac OS X 10.12.6 (Sierra), United States]
I couldn't use vcpkg to install OpenCL.
I'm not sure what all this means, so I'm just pasting in the end of the error message.
-- Building x64-osx-dbg
CMake Error at scripts/cmake/vcpkg_build_cmake.cmake:170 (message):
    Command failed: /Users/lucius/lsc/cpp/vcpkg/downloads/tools/cmake-3.11.4-osx/cmake-3.11.4-Darwin-x86_64/CMake.app/Contents/bin/cmake;--build;.;--config;Debug;--target;OpenCL;--;-v;-j1
    Working Directory: /Users/lucius/lsc/cpp/vcpkg/buildtrees/opencl/x64-osx-dbg
    See logs for more information:
      /Users/lucius/lsc/cpp/vcpkg/buildtrees/opencl/build-x64-osx-dbg-out.log

Call Stack (most recent call first):
  ports/opencl/portfile.cmake:67 (vcpkg_build_cmake)
  scripts/ports.cmake:71 (include)

Error: Building package opencl:x64-osx failed with: BUILD_FAILED
Please ensure you're using the latest portfiles with `.\vcpkg update`, then
submit an issue at https://github.com/Microsoft/vcpkg/issues including:
  Package: opencl:x64-osx
  Vcpkg version: 0.0.113-unknownhash

Additionally, attach any relevant sections from the log files above.

Here is the first of a long series of error reports in the log file:
In file included from /Users/lucius/lsc/cpp/vcpkg/buildtrees/opencl/src/758fc44360-e8fe074a5b/icd.h:53:
/Users/lucius/lsc/cpp/vcpkg/packages/opencl_x64-osx/include/CL/cl.h:708:74: error: expected function body after function declarator
                               cl_command_queue     /* command_queue */) CL_API_SUFFIX__VERSION_2_1;

Update: I got the version of OpenCL in my System/Library/Frameworks directory to work with Boost::Compute and g++. So I no longer need to install OpenCL on Mac OS X using vcpkg.

Hi @LuciusSchoenbaum, sorry for late.
I can't repro this issue on my machine, does it exists?

We haven't been able to repro this; if more information comes up, or this issue appears again, please reopen.


📘

I fixed some mistakes, made it easier to add things to it such as commands, and made it easier to safely migrate to a production environment.
I also added a typing for this too for a bit more help from Intellisense. (I'm a vscode user so idk if this will work for anyone else)

Thanks :D


Error: The JSON value could not be converted to Observatory.Framework.Files.ParameterTypes.MicroCategory.

I selected read all, which I've not done since bug #31 was fixed - I get the attached errors.
Version: 0.1.327.2251
Plugins:
Observatory BioInsights (v0.21.313.1557)
Observatory Botanist (v0.0.320.140)
Observatory Boxel Stats (v0.21.313.1600)
Observatory Evaluator (v0.21.291.1521)
Observatory Explorer (v0.1.327.2244)
Observatory Signal Monitor (v0.21.291.1449)
Observatory Herald (v0.0.327.2244)
Observatory Telegram Notifier (v0.21.320.142)
First error is:
The JSON value could not be converted to Observatory.Framework.Files.ParameterTypes.MicroCategory. Path: $.Category | LineNumber: 0 | BytePositionInLine: 412.
File: Journal.210722231223.01.log
Line (first 200 chars): { "timestamp":"2021-07-22T23:19:17Z", "event":"TradeMicroResources", "Offered":[ { "Name":"rdx", "Category":"Component", "Count":14 }, { "Name":"phneutraliser", "Name_Localised":"pH Neutraliser", "Cat
ObsErrors.txt

Clearly I need to start putting more of the journal line in the error message. Can you provide the full journal line for any of those events?

Journal.210722231223.01.log
Here you go :-)

Happened to me, just installed the last version:
`The JSON value could not be converted to Observatory.Framework.Files.ParameterTypes.CrimeType. Path: $.CrimeType | LineNumber: 0 | BytePositionInLine: 94.
File: Journal.211004002230.01.log
Line: { "timestamp":"2021-10-03T22:46:18Z", "event":"CommitCrime", "CrimeType":"onFoot_dataTransfer", "Faction":"Future of Arro Naga", "Fine":500 }
The JSON value could not be converted to Observatory.Framework.Files.ParameterTypes.CrimeType. Path: $.CrimeType | LineNumber: 0 | BytePositionInLine: 94.
File: Journal.211011201045.01.log
Line: { "timestamp":"2021-10-11T18:40:33Z", "event":"CommitCrime", "CrimeType":"onFoot_dataTransfer", "Faction":"Future of Arro Naga", "Fine":500 }`
Running Windows 10 Italian version.
The old "Observatory.exe" has no problems. HTH

Found both, should be fixed next release.
8094820ffd56be699c0c094c71d23df2323fb520
and
1bf3927acacea55fc11fdff59f1185ec06c2ee35
If you're using the installed version (not the standalone from the zip) replacing ObservatoryFramework.dll with this one might sort you both out. If it works for you guys I'll add it to the releases page.
ObservatoryFramework.zip

That has indeed fixed the bulk of the errores I was getting (it was +77). Just ran a read all again and I've now got only 2 errors (logs attached):
The JSON value could not be converted to System.Int64. Path: $.ShipsRemote[1].ShipMarketID | LineNumber: 0 | BytePositionInLine: 498.
File: Journal.210910205433.01.log
Line (first 200 chars): { "timestamp":"2021-09-10T23:17:10Z", "event":"StoredShips", "StationName":"Horrocks Vision", "MarketID":3225271296, "StarSystem":"Barotrites", "ShipsHere":[  ], "ShipsRemote":[ { "ShipID":0, "ShipTyp
The JSON value could not be converted to Observatory.Framework.Files.ParameterTypes.CrimeType. Path: $.CrimeType | LineNumber: 0 | BytePositionInLine: 101.
File: Journal.211005103835.01.log
Line: { "timestamp":"2021-10-05T10:46:47Z", "event":"CommitCrime", "CrimeType":"onFoot_carryingStolenGoods", "Faction":"Dakak Imperial Society", "Fine":12500 }
Journal.210910205433.01.log
Journal.211005103835.01.log

Perfect. I'll get those fixed as well then update the release page and provide a new hotfix dll. Once that's done I'll close this issue.

Release updated with repackaged installer, rebuilt standalone, and separate hotfix.
https://github.com/Xjph/ObservatoryCore/releases/tag/v0.1.327.2251


Solution to bootloops or kernel weird behaviour

I posted this in a comment but it deserves its own post as I have questions.
I had this issue which creeps up when I change to certain resolutions causing boot loops but sometimes undefined behavior in the kernel and it took me a while to figure this out but the first page at address 0 is already allocated causing the AllocatePages to fail. I assume it's reserved by the bootloader for itself? I have unallocated it and after that it allocates fine but then you are overwriting memory which also causes boot failures or problems in the kernel.
What does work is changing this line to:
Elf64_Addr segment = phdr->p_paddr;
to
Elf64_Addr segment = phdr->p_paddr + 0x1000;
This starts loading the kernel at page 2.
Then change:
void (KernelStart)(BootInfo) = ((attribute((sysv_abi)) void ()(BootInfo) ) header.e_entry);
to
void (KernelStart)(BootInfo) = ((attribute((sysv_abi)) void ()(BootInfo) ) header.e_entry + 0x1000);
This loads the kernel from page 2.
Now the kernel boots fine and everything works except interrupts. To fix this and I am not happy with this fix is in the kernel kernelUtil.cpp change:
interrupt->SetOffset((uint64_t)handler);
to
interrupt->SetOffset((uint64_t)handler + 0x1000);
I do not want to have to add this offset to certain things inside the kernel.
Is there any way to tell the compiler or linker to account for this 0x1000 offset and remap the addresses?
I am sure there is a way I just don't know how.
If there is another solution let me know.

It looks like the solution to above is not the changes I suggested above but rather change the kernel.ld script by adding this line:
. = 0x1000; just before _KernelStart = .;


Update README.md

I think there's a typo in the ##### Math mode section; original text read: "To switch to Text simply: label.labelMode = .label" and I think that last word is supposed to be "text" instead.

Thanks for the fix.


Make use of ElasticSearch optional

Adds a configuration parameter which, when set, completely disables use of ElasticSearch. Note that search operations which require ElasticSearch will silently fail by returning empty results.
The new configuration parameter is workflow.indexing.enabled with a default value of true. Setting it to false will completely disable use of ElasticSearch.
(Of course, with this configuration, significant parts of the Conductor UI will no longer function correctly. However, not all users of Conductor actually use the Conductor UI, so for such users that will not be an issue.)

Pull Request Test Coverage Report for Build 3773

3 of 34 (8.82%)  changed or added relevant lines in 5 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.1%) to 69.848%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

server/src/main/java/com/netflix/conductor/bootstrap/ModulesProvider.java
2
3
66.67%

server/src/main/java/com/netflix/conductor/noopindex/NoopEmbeddedElasticSearchProvider.java
0
2
0.0%

server/src/main/java/com/netflix/conductor/noopindex/NoopIndexModule.java
0
4
0.0%

server/src/main/java/com/netflix/conductor/noopindex/NoopIndexDAO.java
0
24
0.0%

Totals

Change from base Build 3755:
-0.1%

Covered Lines:
11418

Relevant Lines:
16347

💛  - Coveralls

@skissane This is a fantastic addition and we really appreciate your contribution.
Please resolve the minor comment and we can merge this.

when will this feature be released?😄


Add initial implementation of SchemeDictionaryDestination

Signed-off-by: Filipe Laíns<EMAIL_ADDRESS>

https://github.com/pradyunsg/installer/blob/main/noxfile.py#L23
This is intentionally 3.8 — since that’s the same as all the sessions in the noxfile, in case some contributor only has single version of Python in their environment. Bump everything to 3.9 if you’d prefer. :)

Friendly ping :blush:

Friendly ping :blush: