Namespaced components not working as expected

Background
We've been using ViewComponent since early days and it's been great. Since we have so many components, each with three files (.rb, .html.erb and .css) each component resides in its own folder to help organisation.
Also full disclosure, we don't use the Component suffix i.e. we use Alert.new to save typing.
We now want to create two types of view components, matching what you do at GitHub i.e. as documented here.
In order to create the mentioned "App-specific ViewComponents" we need namespaced components. But the documented generator doesn't produce a renderable component.
This looks to be due to our folder structure of having each component in a folder (which is quite nice). Is there a way for namespaced components to work in harmony with our normal components, keeping the same folder structure?
I've detailed the steps below with a repo, many thanks for your time!
Steps to reproduce

Create new Rails app

Add a couple view components using folder structure app/components/component_name/component_name.rb
In order for this to work the following must be added to config/application.rb:
config.autoload_paths += Dir[config.root.join('app/components/*')]

Run namespaced component generator i.e. bin/rails generate component Sections::Example title content

Restart the server and try to render the component in a view i.e. <%= render Sections::ExampleComponent.new %>

Receive error uninitialized constant #<Class:0x0000000122a13c68>::Sections

Here's a repo reproducing the issue: https://github.com/AlecRust/namespaced-components-issue
Expected behavior
I expected to be able to create namespaced components, despite each of our existing components residing in a folder. I assume since the namespaced components generator creates a folder (unlike the standard generator) this is what's causing the issue.
Actual behavior
Namespaced components don't seem to work. Normal components seem fine.
System configuration
Rails version: Tested on 6.1 and 7.0
Ruby version: Tested on 2.7 and 3.1
Gem version: 2.49.0
Related issues

https://github.com/github/view_component/issues/401
https://github.com/github/view_component/issues/670

Hi @AlecRust! Thanks for opening this issue.
I find this use case interesting - it makes sense that the Ruby file should be at the same level of all sidecar assets. I do wonder if this should be considered as a feature request.
However, I feel like without coercing Zeitwerk in some way, this could be quite a difficult thing to achieve. autoload_paths only adds to the list of directories from which components are loaded - without changes to Zeitwerk's configuration, they won't be autoloader to your liking. Zeitwerk, by default, loads classes like this:
app/components/component_name/component_name.rb

gets translated to:
ComponentName::ComponentName

based on the file path. So app/components/component_name/component_name.rb should contain the constant ComponentName::ComponentName in order for it to be successfully loaded.
I've previously tampered with Zeitwerk's loading strategy using an initializer that calls this:
# frozen_string_literal: true

Rails.autoloaders.each do |autoloader|
  autoloader.inflector.inflect(
    'commands' => 'Command'
  )
end

This allowed for app/commands/thing_command.rb to point to Command::ThingCommand without the need for a parent command directory to define the namespace. You may be able to use this to reach the solution you'd like.

Thanks for your reply @boardfish I wasn't aware of the default Zeitwerk behaviour here.
I agree this could be treated as a feature request.
Until then it sounds like simplest solution to start using namespaced components is to match the docs by having all component files together in one folder. This will be a large folder but at least the files will still be grouped together.

Hey @AlecRust, it sounds like this behavior is to be expected based on how Zeitwerk functions. Please don't hesitate to reopen if I've missed something :)


Add a SIG Architecture - Enhancements spotlight article

Add a spotlight article on the SIG Architecture - Enhancements subproject, as part of the SIG Architecture series.
xref: https://github.com/kubernetes/contributor-site/issues/378

/assign
/cc @kikisdeliveryservice

Started getting answers. Movement is happening.

Answers are done, working on getting the markdown done. Open to volunteers to help with the markdown (particularly blogging shadows).

/remove-lifecycle stale


fix: sorting issue for suggested params

Description

by default sort() takes number into string, so in case 3 and 11, 11 is first. I added custom sort comparison and extended tests

You will be never boring with reading of documentation in JS


examples were not rendered fine in the documentation

Summary of changes
Indentation was wrong, so examples were not rendered correctly in the docs.
Checklist

[ ] New functions have typespecs, changed functions were updated
[ ] Same for documentation, including moduledocs
[ ] Tests were added or updated to cover changes
[ ] Commits were squashed into a single coherent commit

It was due to incorrect align of markdown.

Coverage increased (+0.04%) to 70.417% when pulling c752595a8a9d7d469cd923dafc3bd1321dd40334 on edmz:patch-1 into 7c736b6624b692b613a8ce2ec1086439780657e0 on bitwalker:master.

Fixed in master


Backup file system provider can not do buffered write

Does this issue occur when all extensions are disabled?: Yes/No

VS Code Version: 1.83-Insiders
OS Version: macOS

I was verifying https://github.com/microsoft/vscode/issues/192970 and found that large file piece tree buffers are still merged before write. Turns out that backup is always doing doWriteUnbuffered since it's using FileUserDataProvider, which doesn't have FileOpenReadWriteClose capability so it always runs https://github.com/microsoft/vscode/blob/3b60c12380f5a9782439be309d8d2642835b8822/src/vs/platform/files/common/fileService.ts#L394-L395
and unfortunately unbuffered write always concatenates all buffers first
https://github.com/microsoft/vscode/blob/3b60c12380f5a9782439be309d8d2642835b8822/src/vs/platform/files/common/fileService.ts#L1294

One interesting behavior I found is sometimes backup happens right after I press cmd+s to save, maybe the delayed backup kicks in right before I save, but it feels weird to have both a backup and save operation at the same time.

Good catch, but I think this is simply something that was forgotten when introducing the file user data provider. Since the underlying disk file system provider supports this, I see no reason why the file user data provider could not support this.
@sandy081 can you check?

Looks like this was an explicit decision in https://github.com/microsoft/vscode/commit/87bf1d80652c928d4c81cfa95b05bbf285e6dd84#diff-28073e52c2a7e44ac456dfb32c4f0794985cb4a4bcc7c0d99a033e8224fff52cR25 and probably done to enforce atomic writes.
I think we may be fine to bring back that capability. The file service is already optimising the calls to the provider in a way that writeFile is being called when the buffer is small:
https://github.com/microsoft/vscode/blob/2553649d48b6a8da349ac1fe1c56260d3f849e09/src/vs/platform/files/common/fileService.ts#L372-L390
On top of that: if any component comes with a buffer for writing, we never stream:
https://github.com/microsoft/vscode/blob/2553649d48b6a8da349ac1fe1c56260d3f849e09/src/vs/workbench/services/textfile/browser/textFileService.ts#L267-L275
Which I would assume is the case for all our settings when written to programmatically.

@bpasero Does it also need FileUserDataProvider to support FileSystemProviderCapabilities.FileOpenReadWriteClose ?

@sandy081 yes, that is what the IFileService will probe for before deciding which method to use. Since yours does not declare it, all writes are without streaming.

I experimented adding the FileOpenReadWriteClose capability to the FileUserDataProvider https://github.com/microsoft/vscode/pull/193228, with this backup streams again and no longer blocks the UI. Feel free to discard my PR if it's in the wrong direction

I wonder if a better approach for this would be to drop the FileUserDataProvider and simply expose a method on IFileService to set options for certain paths.
Basically this method would call into IFileService instead of requiring to subclass the disk file system provider:
https://github.com/microsoft/vscode/blob/fa99dace5ee3b35a070ca4970422621af07c2781/src/vs/platform/userData/common/fileUserDataProvider.ts#L54-L62

I liked that. Does it mean we should not use vscode-userdata scheme in Desktop?

@sandy081 I would not suggest to change anything with regards to URI or scheme, just simply add a way on the IFileService to enforce custom options when writing. Its a bit of a hack, but I find the custom file system provider to cause more issues. I would assume you can drop it then.

I am wondering it is not straight forward to use disk FSP without changing the scheme. The main purpose of vscode-userdata fsp is to handle the scheme, for eg., watching on user data scheme and getting change events on the same scheme. We should also adopt the disk FSP not to call uri.fsPath directly which I saw using it here
https://github.com/microsoft/vscode/blob/d8e69199806fe5409869dad46dfef926a36ff8f8/src/vs/platform/files/common/diskFileSystemProvider.ts#L217-L219

@sandy081 how about this then:

keep using vscode-userdata scheme as we do today ✅
keep using a fileUserDataProvider as we do today ✅
take in the change from Pung in https://github.com/microsoft/vscode/issues/193151 to enable streaming support ✅
revert the change from https://github.com/microsoft/vscode/commit/3b60c12380f5a9782439be309d8d2642835b8822 ✅
expose a method on IFileService to enforce atomic writes with the knowledge we have from https://github.com/microsoft/vscode/commit/3b60c12380f5a9782439be309d8d2642835b8822

I think then we get the best of all worlds:

we don't change schemes
we don't change file system providers
the decision whether to use atomic write or not moves into the IFileService

@sandy081 I went with a slightly different approach: since the file user data provider is used in many contexts, it makes more sense to keep the decision about which resources should get atomic read/write inside the provider. As such, I added new methods to enforce atomic read, write or delete: https://github.com/microsoft/vscode/pull/193228

This bug has been fixed in the latest release of VS Code Insiders!
@rebornix, you can help us out by commenting /verified if things are now working as expected.
If things still don't seem right, please ensure you're on version c72447e8d8aaa7497c9a4bd68bc4301584b92beb of Insiders (today's or later - you can use Help: About in the command palette to check), and leave a comment letting us know what isn't working as expected.
Happy Coding!


Integer overflows

building for Ipad MC775ZP/A with IOS version 9.3.5 (13G36) is failed with errors:
Integer literal '4023233417' overflows when store into 'UInt32'
Integer literal '2562383102' overflows when store into 'UInt32'

at lines:
var b = UInt32(0xefcdab89)
var c = UInt32(0x98badcfe)

Look at my workaround in Issues #1

I also have a pull request in with the same fix

Hi man, you just need change
    var a = UInt32(0x67452301)
    var b = UInt32(0xefcdab89)
    var c = UInt32(0x98badcfe)
    var d = UInt32(0x10325476)

to
    var a: UInt32 = 0x67452301
    var b: UInt32 = 0xefcdab89
    var c: UInt32 = 0x98badcfe
    var d: UInt32 = 0x10325476

all errors should go away.

gaoshanyu's answer solve my problem.


Adding About Page

Hey Kritika, can I be assigned this task? I'll start working as soon as further details are provided.

Okay
You can join the discord!

hey @Kritika30032002 Can you please assign me this issue under kwoc2022??

@Riya14-dangra You can take another issue which is not yet assigned to anyone else or create a new issue.

Heyy,
Sorry but I'm unable to join the discord, it shows an invalid link.

@MrEngineer500  Now try again


Decrease MSRV

Currently jiff MSRV is set to 1.70 but it can be lowered and still be compiled correctly. What is your policy on this?
It would be nice if the version could be lowered to 1.63 to make jiff compatible with Pyo3.

Nevermind, OnceLock requires 1.70+.

Yes, correct. IMO, Rust 1.70 is already pretty generous. Rust 1.63 is a very conservative MSRV. I'm guessing it's trying to target Debian. But Debian can just use an older version of Pyo3 that works on their version of Rust.

Also, Jiff's MSRV policy is here: https://github.com/BurntSushi/jiff?tab=readme-ov-file#minimum-rust-version-policy


Adding // to the end of the root route throws an Internal Server Error

What version of Remix are you using?
1.7.3
Steps to Reproduce
Spin up a new remix project from scratch. Just the basics. Remix Server. JavaScript.
Start the dev server and visit the browser at localhost:3000//.
Also, for fun check out https://remix.run//
Expected Behavior
I would expect this to not error. Ideally it would redirect back to the root route.
Actual Behavior
On production: An Internal Server Error is thrown that doesn't get caught by Error Boundaries.
On development: An Invalid URL error is thrown that also doesn't get caught by Error Boundaries.
TypeError: Invalid URL
    at new NodeError (node:internal/errors:372:5)
    at URL.onParseError (node:internal/url:553:9)
    at new URL (node:internal/url:629:5)
    at createRemixRequest (/Users/bkrall/content/remix-bug-server-error-on-route/node_modules/@remix-run/express/dist/server.js:67:13)
    at /Users/bkrall/content/remix-bug-server-error-on-route/node_modules/@remix-run/express/dist/server.js:37:21
    at /Users/bkrall/content/remix-bug-server-error-on-route/node_modules/@remix-run/serve/dist/index.js:48:7
    at Layer.handle [as handle_request] (/Users/bkrall/content/remix-bug-server-error-on-route/node_modules/express/lib/router/layer.js:95:5)
    at next (/Users/bkrall/content/remix-bug-server-error-on-route/node_modules/express/lib/router/route.js:144:13)
    at next (/Users/bkrall/content/remix-bug-server-error-on-route/node_modules/express/lib/router/route.js:140:7)
    at next (/Users/bkrall/content/remix-bug-server-error-on-route/node_modules/express/lib/router/route.js:140:7)

This is fixed in Remix 1.13.0 - we aren't doing any redirecting but instead are ensuring our adapters can correctly generate a URL with these trailing slashes.  It would still be up to the app to detect and redirect in a loader (or eventually in middleware once that lands).

@brophdawg11 I just filed an issue with the <Link> component improperly handling URLs in Webkit browsers that occur in Remix 1.13.0. The issue must be a regression (it would be insane if this behavior existed for more than a week, since it means all links are broken on Webkit browsers). The commit that fixes this issue looks likely to be the culprit; any chance you could double check that it didn't cause a regression? Thanks!

I ran into similar issue. I'm trying to deploy the remix app to our company cloud service. it works locally. what should I look at first? below is the error message
TypeError: Invalid URL
    at new NodeError (node:internal/errors:405:5)
    at new URL (node:internal/url:637:13)
    at createRemixRequest (/opt/bytefaas/toutiao/app/gdserver/node_modules/.pnpm/@remix-run+express@2.6.0_express@4.18.2_typescript@5.3.3/node_modules/@remix-run/express/dist/server.js:74:13)
    at /opt/bytefaas/toutiao/app/gdserver/node_modules/.pnpm/@remix-run+express@2.6.0_express@4.18.2_typescript@5.3.3/node_modules/@remix-run/express/dist/server.js:39:21
    at Layer.handle [as handle_request] (/opt/bytefaas/toutiao/app/gdserver/node_modules/.pnpm/express@4.18.2/node_modules/express/lib/router/layer.js:95:5)
    at next (/opt/bytefaas/toutiao/app/gdserver/node_modules/.pnpm/express@4.18.2/node_modules/express/lib/router/route.js:144:13)
    at next (/opt/bytefaas/toutiao/app/gdserver/node_modules/.pnpm/express@4.18.2/node_modules/express/lib/router/route.js:140:7)
    at next (/opt/bytefaas/toutiao/app/gdserver/node_modules/.pnpm/express@4.18.2/node_modules/express/lib/router/route.js:140:7)
    at next (/opt/bytefaas/toutiao/app/gdserver/node_modules/.pnpm/express@4.18.2/node_modules/express/lib/router/route.js:140:7)
    at next (/opt/bytefaas/toutiao/app/gdserver/node_modules/.pnpm/express@4.18.2/node_modules/express/lib/router/route.js:140:7)

After some dig in, I found the issue
In our could service, resolveHost is [fdbd:dc01:b:90::28]:dc01, which isn't a valid URL.
the source code
https://github.com/remix-run/remix/blob/2228ec33f6964bb5bf1373cd4f41cdff4a134301/packages/remix-express/server.ts#L96-L98


Support for Azure blob storage

Hi,
Thanks, I was looking for a self-hosted alternative to GitHub's LFS storage because I'm currently blasting through my LFS storage limit's. I came across your library and blog posts which is exactly what I was looking for.
The thing is I would like to store my LFS files in the Azure cloud, so I decided to code support for azure blob storage (basically the Azure equivalent of S3).
I'll open a PR once I'm done, so you can take a look.
Let me know what you think.
Stephanvs

Hi Stephan,
An Azure adapter would be fantastic! I intended to add one eventually, but I work more with AWS than Azure and just didn't get around to doing it (and also wouldn't be able to test it as well as the S3 adapter, considering I use that nearly every day!).
Let me know when you have something to review,
Alan

Thanks so much for merging my PR 💯

Thanks for your contribution :) I pushed verison 3.1.0 of the packages, they'll be indexed shortly but are ready for use:

https://www.nuget.org/packages/Estranged.Lfs.Adapter.Azure.Blob/3.1.0
https://www.nuget.org/packages/Estranged.Lfs.Adapter.S3/3.1.0
https://www.nuget.org/packages/Estranged.Lfs.Api/3.1.0
https://www.nuget.org/packages/Estranged.Lfs.Authenticator.BitBucket/3.1.0
https://www.nuget.org/packages/Estranged.Lfs.Authenticator.GitHub/3.1.0
https://www.nuget.org/packages/Estranged.Lfs.Data/3.1.0


allow several direction=*-values for some things like highway=street_lamp

Hello,
I wanted to ask whether it would be possible to allow more than just one direction=*-value for the highway=street_lamp preset. It's because one node with highway=street_lamp, can have two or more single lamps on/at it which shine into several directions. So often it might be necessary to tag such objects with something like direction=120;300 (two lamps with an angle of 180 between them). But I know it might be hard to do because in most cases, we want to have just one direction value to avoid mistakes. But even not everywhere, so. Some street lamps also have a kind of "360 degree"-glow, so the direction-value would be "0-360"...

Ah yeah, this does actually work if you type your values in the raw tag editor - but it does not work in the Direction field because of #5438.  The field is setup to be strictly numeric and it throws out the semicolon values.  I'm going to close here because it's really a bug captured on that other issue.


Reduce Polling Interval Option

Is your feature request related to a problem? Please describe.
Mint Mobile’s website has auto blocked my IP for too many logins.
Describe the solution you'd like
Ideally we have some way to fine tune the polling interval to keep Mint happy with less login attempts. Perhaps a polling interval option?
Describe alternatives you've considered
N/A
Additional context
They have banned my home IP, hopefully temporarily, because I login too often with HA. The Mint website no longer works from my home IP but if I change public IPs, like a mobile hot spot, it connects.

Sure. I can work on that.

Uh I have this happened to me and I'm on the phone with the "Tech" it's a nightmare trying to tell them that nothing is wrong with my service and I can access my account using mobile data and someone needs to unblock my home ip.

I’ve gone around in circles with Alex on Reddit from Mint Mobile. He said his IT department has many reports and is looking at it.
They basically implemented a web application firewall that triggers thinking this addon is a bot. I’m hoping the reduced polling to maybe once per day will work if they ever unban me because being able to monitor usage rocks.
He said it could take “up to 30 days”, but I suspect they outsourced their OpSec so they are confused.

So I've decided to leave Mint Mobile over this issue. They have blocked my ISP and their API even returns "BAD_ISP" yet their customer support staff will tell you that they don't block IP's and that you need to reach out to your ISP to fix their issues. They have put in a ticket before to solve this but upon follow up, say that the ticket was never worked.

I don't think they are trained for such a request, I don't think someone intentionally has blocked our IP. This is probably done as @zSprawl by a firewall. I can only hope this firewall whitelists us after X amount of time.

It probably is a firewall. They use an Imperva WAF as there web firewall but they don't know how to unblock my ISP, which I have Mint Mobile integration disabled for a few months now and can not manage my account anymore and everybody else in my building is blocked from Mint Mobile as well since we have a CGNAT and share IPv4 addresses. Blocking customer's IPs is not the answer and should instead preform a security challenge like a CAPTCHA. Since they are unwilling to work with you on the issue, I don't see a reason to continue using them for my service as their's other providers out there that will do better.

 We are all using Mint likely cause it’s $15 per month. I’m locked into an annual contract but I’d be interested if you find a better provider and integrate it with HA.
How hard is it to change the polling interval to 1 day if they ever unban us? It is a simple variable change in the code for polling interval?
Not gonna hold my breath but I’m stuck with Mint for a year anyhow. Hopefully they figure out the WAF issue as I suspect they’ve outsourced the security to a 3rd party.

@zSprawl I've actually already updated the code to be a variable for the polling interval that you can set to any amount of time. In addition, I refactored the code to make as little of API calls as possible to reduce the risk of getting banned. I just don't have a way to test it.
I'm moving over to Visible that's $40 a month with unlimited data. Due to that being unlimited, I'm not sure it's worth even tracking that data in Home Assistant.

Awesome.
Well best of luck. I’ll try the code out if they ever unban me. It appears to have hit a lot of people so hopefully eventually they are pressed to fix it.

@zSprawl Sounds good. I'll push out the code as a beta version so you can select it from HACS then.

Did that beta ever get pushed to GitHub? They finally unblocked me and my annual is going to expire next month, so I wanted to try one more time.

I think this repo has basically been abandoned. I'm not sure what the original fix was.  How long did it take for your IP to be un-banned?  I think I was banned this morning, I am adding the below to sensor.py (just below line 22), but I'm curious how long it'll take before it starts working again!
SCAN_INTERVAL = datetime.timedelta(hours=6)

I have no idea if it'll do the trick...

It took almost 9 months... I think it's because they got purchased by T-Mobile, or at least, it seems to have happened around that time.

That's annoying.  Could be it's related to the acquisition, it did (anecdotally) seem like a lot of churn happened surrounding Mint Mobile and their infrastructure during those weeks.  Though the purchase isn't completed yet, so legally it's a little early for them to be taking any actions.  It is possible they were doing some housekeeping at the same time, related or unrelated... But the big changes legally have to wait until the relevant government bodies approve the purchase (they're still separate legal entities for the moment).
That said, I can confirm the ban isn't an IP ban, but it is an account ban.  I got banned about a month ago, I've tried from multiple IP addresses via multiple different ISPs, but gotten access forbidden messages every time.  Would be great if there was at least some documentation letting us know how frequently we're allowed to ping the service before becoming banned.  Is it a firm number of requests per year?  Per contract term?  In any event, I'm locked out for a month now.  I'll cross my fingers it won't take 8 more months to become unbanned, though I may not stick with Mint Mobile in 2024 anyways.


Windows MSI build not always working

It looks like the MSI build for Windows distribution is not always working.
To better describe setup process to avoid pitfalls. Ideally add to CI as well #15.

Issue came from the latest release of the MSI builder, hence the difference between setups.
https://github.com/Nitrokey/pynitrokey/issues/33 should solve the problem.

To be precise:

cxfreeze is working but the MSI is not created
this is a known bug in cxfreeze
pinning the version to 6.2 worked, current git (master) should also work

Hi!
@daringer Could this be closed? This blocks the v0.4 release.

yep, can do.
cxfreeze can be kept at 6.2 for now, don't see a need for an update


Wordcount support

https://github.com/Xon/XenForo-WordCountSearch/issues/2
Add support for showing the wordcounts on a threadmark index, and a total for all threadmarked content. This issue will require changes to the above add-on and threadmarks.

Implemented in #52


Add Combobox alias for Lookups

[ ] Lookup: Uses new combobox HTML
[ ] Added Combobox component — Combobox

Still a prototype


Cut copy paste (right click) context dialogue is bugged

Overview

Insomnia Version: <IP_ADDRESS>6
Operating System:  macOS Sierra V10.12.1
Summary:  If you click on 'timeline' in insomnia client, highlight some words, click about in the UI you'll find that eventually wherever you click on your mac desktop (even on different monitors) the context menu appears, very odd

How To Reproduce

Go to timeline on a request
Right click and select COPY (click copy)
Without moving mouse... this weird duplication bug appears

Other Notes
Never seen this problem before, started occuring around 23 or 24/5/2017
Have to force quit the app to stop the issue quite often, yesterday caused a real headache

Totally embarrassed.
My Apple mouse is right clicking when it shouldn't, not witnessed it anywhere else but it's not an insomnia bug at all (!)

@TakesTheBiscuit this was actually a real bug #218 but it's fixed now 😄


[Site Request] rajdigitaltv.com

DO NOT REMOVE OR SKIP THE ISSUE TEMPLATE

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

Checklist

[X] I'm reporting a new site support request
[X] I've verified that I'm running yt-dlp version 2022.08.19 (update instructions) or later (specify commit)
[X] I've checked that all provided URLs are playable in a browser with the same IP and same login details
[X] I've checked that none of provided URLs violate any copyrights or contain any DRM to the best of my knowledge
[X] I've searched the bugtracker for similar issues including closed ones. DO NOT post duplicates
[X] I've read the guidelines for opening an issue
[ ] I've read about sharing account credentials and am willing to share it if required

Region
India
Example URLs
https://www.rajdigitaltv.com/movies/thenali/RAJTV_MOVIE_61d91b7b99c95163af7bcf46
Provide a description that is worded well enough to be understood
Rajdigitaltv is a subscription-based streaming site for Tamil movies. I don't know if the content is downloadable using yt-dlp. I hope you look into it.
Provide verbose output that clearly demonstrates the problem

[X] Run your yt-dlp command with -vU flag added (yt-dlp -vU <your command line>)
[X] Copy the WHOLE output (starting with [debug] Command-line config) and insert it below

Complete Verbose Output
[debug] Command-line config: ['-vU', '--cookies-from-browser', 'chrome', 'https://www.rajdigitaltv.com/movies/thenali/RAJTV_MOVIE_61d91b7b99c95163af7bcf46']
[debug] Encodings: locale cp1252, fs utf-8, pref cp1252, out utf-8, error utf-8, screen utf-8
[debug] yt-dlp version 2022.08.19 [48c88e0] (win32_exe)
[debug] Plugins: ['SamplePluginIE', 'SamplePluginPP']
[debug] Python 3.8.10 (CPython 64bit) - Windows-10-10.0.19044-SP0
[debug] Checking exe version: ffmpeg -bsfs
[debug] Checking exe version: ffprobe -bsfs
[debug] exe versions: ffmpeg 2022-07-28-git-6f7d3bde11-full_build-www.gyan.dev (setts), ffprobe 2022-07-28-git-6f7d3bde11-full_build-www.gyan.dev, rtmpdump 2.4
[debug] Optional libraries: Cryptodome-3.15.0, brotli-1.0.9, certifi-2022.06.15, mutagen-1.45.1, sqlite3-2.6.0, websockets-10.3
[Cookies] Extracting cookies from chrome
[debug] Extracting cookies from: "C:\Users\Alien\AppData\Local\Google\Chrome\User Data\Default\Network\Cookies"
[debug] Found local state file at "C:\Users\Alien\AppData\Local\Google\Chrome\User Data\Local State"
[Cookies] Extracted 3174 cookies from chrome
[debug] cookie version breakdown: {'v10': 3174, 'other': 0, 'unencrypted': 0}
[debug] Proxy map: {}
[debug] Fetching release info: https://api.github.com/repos/yt-dlp/yt-dlp/releases/latest
Latest version: 2022.08.19, Current version: 2022.08.19
yt-dlp is up to date (2022.08.19)
[debug] [generic] Extracting URL: https://www.rajdigitaltv.com/movies/thenali/RAJTV_MOVIE_61d91b7b99c95163af7bcf46
[generic] RAJTV_MOVIE_61d91b7b99c95163af7bcf46: Downloading webpage
WARNING: [generic] Falling back on generic information extractor
[generic] RAJTV_MOVIE_61d91b7b99c95163af7bcf46: Extracting information
[debug] Looking for Brightcove embeds
[debug] Looking for embeds
ERROR: Unsupported URL: https://www.rajdigitaltv.com/movies/thenali/RAJTV_MOVIE_61d91b7b99c95163af7bcf46
Traceback (most recent call last):
  File "yt_dlp\YoutubeDL.py", line 1442, in wrapper
    raise
  File "yt_dlp\YoutubeDL.py", line 1518, in __extract_info
    if process:
  File "yt_dlp\extractor\common.py", line 666, in extract
    if ie_result is None:
  File "yt_dlp\extractor\generic.py", line 3077, in _real_extract
    raise UnsupportedError(url)
yt_dlp.utils.UnsupportedError: Unsupported URL: https://www.rajdigitaltv.com/movies/thenali/RAJTV_MOVIE_61d91b7b99c95163af7bcf46

@Et-tman Website asks for a mobile number

Yeah. It requires phone number. I can share the number with you. But, it has OTP verification.

@Et-tman Website asks for a mobile number

Yeah. It requires phone number. I can share the number with you. But, it has OTP verification


Cone spells Getting Cover

So I have a cone spell attack and its set as a saving throw not as a ranged weapon attack but for some reason saving throws get covered added to them, I turned off this module and that problem went away, is this a bug or do I need to change how the spell works?

@XXDarrienX Are you using midi-QOL as it could be related to a bug I found was introduced a while ago.
If you are using midi-qol and have the Calculate Cover set to Simbul's, templates will not always work correct:

For now while I try and sort out the issue, disable the calculate (or use one of the 2 other modules if you have them) and I should hopefully have a solution soon.

Yes I am using Midi and I did have simbuls set for cover.

@XXDarrienX ok, just ran through the scenario's myself to ensure that everything is working as it should and it is.
The reason you were getting cover on some of the Infected Crew Members is probably because you've setup the Cover Calculator to have token's supply cover. So in this case, what is happening is that you put the template down, midi-qol runs the cover check (as per your settings) and cover check uses the token cover's to let midi know that some of the targets have cover.
I quickly ran this on my own live system to double check and it works like I mentioned above. So you can either remove Calculate Cover from the midi-settings or disable token's giving cover in this module. It's really up to you.
Hope this clears the issue for you.


Simplify RDKit MinimalLib updates

This PR implements two changes:

It removes the need to patch the RDKit_minimal.js loader whenever we update the wasm library version by passing the locateFile to the initRDKitModule function. This allows to keep the RDKit_minimal.js loader as generated by emscripten - no need to patch it anymore
The RDKit_minimal library version is stored in a single place for ease of updating while previously multiple files had to be modified at each update.
To keep things simple I have removed the versioning information from the JS loader library (which indeed is not changing when we update the wasm library, particularly as we do not need to patch it anymore thanks to this PR)

Thank you for the PR @ptosco!


SslContextBuilder is failing with Not supported using BoringSSL

TCN w/ BoringSSL fails with the following error. The change was introduced with https://github.com/netty/netty-tcnative/commit/913df47adc07c9b8b083cdb92789356d3e864e55 but I'm not sure it's supposed to happen with SslContextBuilder#keyManager(file, file, pass)?!
Caused by: java.lang.Exception: Not supported using BoringSSL
        at io.netty.internal.tcnative.SSLContext.setCertificateBio(Native Method) ~[netty-tcnative-boringssl-static-2.0.25.Final-osx-x86_64.jar:2.0.25.Final]
        at io.netty.handler.ssl.ReferenceCountedOpenSslContext.setKeyMaterial(ReferenceCountedOpenSslContext.java:787) ~[netty-all-4.1.36.Final.jar:4.1.36.Final]
        at io.netty.handler.ssl.ReferenceCountedOpenSslClientContext.newSessionContext(ReferenceCountedOpenSslClientContext.java:104) ~[netty-all-4.1.36.Final.jar:4.1.36.Final]
        at io.netty.handler.ssl.OpenSslClientContext.<init>(OpenSslClientContext.java:193) ~[netty-all-4.1.36.Final.jar:4.1.36.Final]
        at io.netty.handler.ssl.SslContext.newClientContextInternal(SslContext.java:777) ~[netty-all-4.1.36.Final.jar:4.1.36.Final]
        at io.netty.handler.ssl.SslContextBuilder.build(SslContextBuilder.java:452) ~[netty-all-4.1.36.Final.jar:4.1.36.Final]

Expected behavior
Actual behavior
Steps to reproduce
Minimal yet complete reproducer code (or URL to code)
Netty version
JVM version (e.g. java -version)
OS version (e.g. uname -a)

@rkapsi what Netty version and tcnative version ?

@normanmaurer 2.0.25.Final (in this case the netty-tcnative-boringssl-static for osx-x86_64 from Maven Central)

and which netty version @rkapsi ?

netty-all-4.1.36.Final

ah sorry... stupid me. I did not see it :( . Let me check.

@rkapsi do you use io.netty.handler.ssl.openssl.useKeyManagerFactory=false by any chance ?

:eek: I do
systemProperties['io.netty.handler.ssl.openssl.useKeyManagerFactory'] = 'false'

@rkapsi I see... Can you remove this and see if it fixes your problem ? I think it should.... In the meantime let me come up with a fix.

@normanmaurer confirm, appear to fix it.

@rkapsi after thinking a bit more about it I lean to just remove support for the system property. It was only present to allow easy switching to not using KeyManagerFactory when we introduce the feature at some point to give the user a chance to disable it if there is a bug. This should not be needed anymore at all as the feature is stable.
WDYT ?

@normanmaurer SGTM

@rkapsi https://github.com/netty/netty/pull/9162 PTAL


Sub Bulleted list formatting when removing keyword option is set

Great plug-in! Just what I've been looking for to do simple qualitative coding.
I discovered a small bug tonight in v.1.1.3. When "Extract content at the bulleted list item level" is checked and "Remove extracted paragraph keyword with end tag" is checked, the plugin removes not only the keyword, but the indentation of the sub bulleted list. For example,

Example 2 #test2

Sub 1
Sub 2

Results in,

Example 2 - Sub 1 - Sub 2

Instead of,

Example 2

Sub 1
Sub 2

Thx for finding the issue.  I'll take a look at it tomorrow. Seems it should be a simple fix.

@hp6393 I believe I have the fix completed. Doing some testing now. It might be good for you to manually install the fix first and verify. Let me know if you'd like to do that.

I've never installed a plug-in manually, but I'm willing to try.

@hp6393 I have to upload the .jpl file to here and then you just download it and copy to the joplin-desktop/plugins directory. If you don't want to fiddle with that, I'll just continue testing for a day or two and update the version to 1.1.4

Sounds easy enough. I'll try it.

@hp6393 - ok, the fix is here:
https://github.com/djsudduth/joplin-plugin-paragraph-extractor/blob/main/io.github.djsudduth.paragraph-extractor.jpl
Download and copy this jpl file to your joplin-desktop/plugins directory. You might want to save a backup of the existing version in case this doesn't work.
I added one additional thing - I was bothered by not having additional line feeds between extracted paragraphs from each page (adding an extra line feed only at the page level). It helps me see what chunks of paragraphs are part of the same source note. Let me know if this bothers you or you feel its useful. I suppose I could add an option in settings, or, just only add the line feed if the extracted note titles are selected in options.

I downloaded the jpl file and replaced the existing jpl in the plugins directory with the new version. I restarted Joplin and the plugin disappeared from my plugin list and all menus. I then tried manually installing the jpl using the "from file" option. It would not install. I noticed that the new version is quite a bit larger than the 1.1.3 version. Otherwise, the file seems ok, but it does not work.

Hmmmm..ok. I'll update the version today.

@hp6393 the new version 1.1.4 was just published. The Update button should show up on Paragraph Extractor in Joplin plugins i about 1 hour from now. Let me know how it goes!

I just updated without incident. I tested it on the text that I used the other day and everything worked as expected. The paragraph spacing looks nice.
Thanks for taking a look at this!

@hp6393 awesome! Thanks for checking it!


Added babel-runtime

Before sending a pull-request please open an issue to discuss new features or non-trivial changes.
Thanks for your contribution!
"babel-runtime": "^6.18.0" dependency to make these examples to work.

thanks!


Script for other kiss websites

Can you make one for other kiss websites? Specially kissasian? I used your script for kissasian but it only gives me empty links :( And thank you for making a batch downloader!

Hey, I just created a script for Kissasian and it is working on my end. Please try it and let me know.
https://github.com/aviaryan/Kissanime-Batch-Downloader/blob/all-kiss/kissasian.js
I am also planning to create script for KissCartoon.
You can track the progress at https://github.com/aviaryan/Kissanime-Batch-Downloader/tree/all-kiss

@Louise789 I have added the support for other Kiss websites now. Please check the repo homepage.

Thank you so much! It works perfectly fine for me!


1142 Longest Common Subsequence golang solution

File(s) Modified:  1142-Longest-Common Subsequence.go
Language(s) Used: golang
Submission URL: https://leetcode.com/submissions/detail/xxxxxxxxx/

Change the title to 1143 ticket and text of the submission link with passed tests.


Missing completions for initialiser list

Given this code:
String _lots(String a) => a * 50;

class Field {
  final String comment;
  Field(String comment) : comment = _lots(comment);
}

There are no errors or warnings, but if you delete the ots from _lots and invoke completion after the _l there are no entries in the results:
==> {"id":"15515","method":"completion.getSuggestions","params":{"file":"/Users/dantup/Dev/parser/lib/test.dart","offset":111}}
<== {"id":"15515","result":{"id":"1346"}}
<== {"event":"completion.results","params":{"id":"1346","replacementOffset":109,"replacementLength":2,"results":[],"isLast":true}}

https://dart-review.googlesource.com/c/sdk/+/84048

Fix LGTM, thanks!


Replace mysql_async with sqlx

mysql_async contribute 20% of project release binary. Cause the release binary reach 9.9 mb. The alternative is try using sqlx.

#16


Feature Request: CLI options for install

My Setup

My OS name and version: Windows 10
My current app version: 2.7

Description
I'm trying to push Rocket.chat out company wide. So I'm needing a way to push a silent install. I found the /S switch which does a silent install, but it installs locally to the users APPDATA folder and doesn't appear to launch Rocket.chat after the install is complete.
If we could have some CLI switches to set those options (which are both available in the GUI installer) that would be incredibly helpful. Not sure if the "Run at startup" gets selected with the /S, but that would be nice as well.
Current Behavior

The client installs, but locally for the user, not all users
After the install is complete, the application does not run
I do not know if it set the application to auto-start with login, since I just uninstalled from the test machine to keep working on switches.

Expected Behavior
Be able to set CLI flags for:

Install for all users
Run Rocket.Chat+ after install
Run at startup

There's a flag for installing for all users: "/allusers"
But the flags for running at startup and after install would be nice

Do you know if the /S option sets the flag for auto-run when the user logs in?

While the "/startup" flag doesn't exist, there is a option to enable it after instalation?

I would like to know if since 2018, new flags have been set up for an installation by GPO.
Flag /S is nice but other flags is needed for a complet GPO installation like :
Running a startup and after install.
Auto server IP
After that, the end user would only have to enter his login credentials.

The missing flags for "startup on windows boot" and "Start after install"  are breaking the silent install with /S just tested it with PowerShell. Installer just times out. Doing manuell installation with only /allusers or /currentuser works fine. For deploying over some software distribution in bigger enviroments this is an issue especially since your updater is currently not working. Or are you just lagging the documentation for these installation flags?

is there anybody here ? no answer for this post !
This features would be great !

This would be a good way to contribute to the project.

Hi, is this feature still required? Maybe I can have a try

Hi, is this feature still required? Maybe I can have a try

Yes, there is no starton boot option for MSI installer


BetterEnd breaks with Sparse Structures

When using BetterEnd combined with Sparse structures, none of the BetterEnd biomes spawn in. This also means that the central end island (which is normally considered its own biome) doesn't spawn in any of the obsidian towers

Hmm, that's interesting, I'll try to reproduce this when I can.
Thank you for the report.


[Fetaure Request] Seperate JWS Parsing in sendAttestationRequest method

Ohai.
First of all, good work on the plugin !
I would like to discuss a feature.
Currently we are using the following workflow

Generate a nonce on device/server
Send Attestation Request to Google
Decode & Verify Response on Device
Execute Relevant Logic.

One of the use cases while implementing SafetyNet requires JWS verification on server side. The second workflow is below.

Generate partial nonce on server
Generate partial nonce on device
Send Attestation Request to Google
Send JWS response to server
Verify JWS response with google on server
Decode JWS
Execute Relevant Logic

A small change to sendAttestationRequest will allow us to execute the second workflow. A developer can then choose to decode JWS on device or send the JWS back to server. All it requires is creating a separate method for JWS decoding.
function sendAttestationRequest(nonce, apiKey) {
    return RNGoogleSafetyNet.sendAttestationRequest(nonce, apiKey).then((result) => {
        return result;
    });
}

function decodeJWS(jwsResponse) {
    return jwtDecode(jwsResponse);
}

Nice! That sounds good to me. Would you like to do a PR for it?

👍 Sure.I will do that in a day or two.

@rajivshah3
I did a pull request.
Originally I wanted to separate the two functions. But I don't really want to break any working logic.
So instead added an extra function.

Thanks! #148


refactor: swap

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

GitHub release
npm package (@latest dist-tag)

Your semantic-release bot :package::rocket:


Doesn't Works with custom shellcode

im using custom shellcode with flag "-t", Equation Editor Crashes but your staged shellcode doesn't invokes
my shellcode i used while generating doc
command : 2.py -t 0 -c sc.txt -o sc.doc
generated doc : http://s000.tinyupload.com/?file_id=51767115386029640308

im using custom shellcode with flag "-t", Equation Editor Crashes but your staged shellcode doesn't invokes
my shellcode i used while generating doc
command : 2.py -t 0 -c sc.txt -o sc.doc
generated doc : http://s000.tinyupload.com/?file_id=51767115386029640308

Please share the shellcode,

sure, can i DM you somewhere like twitter or anywhere? would be comfortable on both side to communicate.
Thanks for replying back :)


Remove deprecated features for upcoming 0.8.0

Description
Remove deprecated kedro_mlflow.framework.context and cancel deprecation of kedro_mlflow.io.metrics.MlflowMetricsDataSet for upcoming 0.8.0 release
Development notes
Remove deprecated code
Checklist

[X] Read the contributing guidelines
[X] Open this PR as a 'Draft Pull Request' if it is work-in-progress
[ ] Update the documentation to reflect the code changes
[X] Add a description of this change and add your name to the list of supporting contributions in the CHANGELOG.md file. Please respect Keep a Changelog guidelines.
[ ] Add tests to cover your changes

Notice

[X] I acknowledge and agree that, by checking this box and clicking "Submit Pull Request":

I submit this contribution under the Apache 2.0 license and represent that I am entitled to do so on behalf of myself, my employer, or relevant third parties, as applicable.

I certify that (a) this contribution is my original creation and / or (b) to the extent it is not my original creation, I am authorised to submit this contribution on behalf of the original creator(s) or their licensees.

I certify that the use of this contribution as authorised by the Apache 2.0 license does not violate the intellectual property rights of anyone else.

Codecov Report

Merging #263 (4b4250c) into master (5080ec3) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff             @@
##            master      #263    +/-   ##
==========================================
  Coverage   100.00%   100.00%
==========================================
  Files           33        30     -3
  Lines         1037       923   -114
==========================================
- Hits          1037       923   -114

Impacted Files
Coverage Δ

kedro_mlflow/io/metrics/mlflow_metrics_dataset.py
100.00% <ø> (ø)

Continue to review full report at Codecov.

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


Cursor moves stragely and slowly

Hi,
I'm so happy that this piece of software is evolving so rapidly. I've recently had, however, a strage issue. If I want to move the cursor in edi_normal, it behaves stragely: it highlights a bunch of text, than moves slowly and puts the cursor to strange places. I also attach a gif, where you can see the problem. First, I hit two e-s, than the rest of the keypresses are w-s.

I also copy here the log file, which clearly shows that there is something on, but I cannot really understand why.
========== System Infomation ==========
[Windows]
Edition: Windows 10 Pro
Version: 23H2
Build Numbers: 10.0.22631
Architecture: x64
[win-vind]
Version: <IP_ADDRESS>
[Message] Version: huge
[Error] Access is denied.
[Error] move_fwd_word failed. An runtime exception occurred: failed opening clipboard (void __cdecl vind::bind::BindedFunc::error_process(const class std::exception &))
Do you have any idea how to resolve this?
Thank you for your time and help.

As default settings of word-motion, win-vind employs the behavior-oriented the original Vim. However, the external application cannot get the text strings of other applications without using the UIA or copy. As a result, the default word motion copies some characters and analyzes the text structure.
If you want to prioritize the speed of word motion, you can use the <move_fwd_word_simple> function instead. For example, enmap w <move_fwd_word_simple>.
You can refer to this document in the cheat sheet.
https://pit-ray.github.io/win-vind/cheat_sheet/functions/#move_fwd_word
Thanks.

Hi,
The strange/slowish movement happens even though I have the move_simple mapping. This happens to me in the latest version 5.11.3. When I switch to my previous version, <IP_ADDRESS>, it behaves fine.
See video:
https://github.com/pit-ray/win-vind/assets/39532610/ee6d3cfe-e077-4e7c-a72e-79895eef155f

@gilbh
I think the reason it is selected even with <move_fwd_word_simple> is because blockstylecaret_mode is set to flex. Please try set blockstylecaret_mode=solid or set noblockstylecaret.
https://pit-ray.github.io/win-vind/cheat_sheet/options/#block-style-caret

Many thanks.

set noblockstylecaret will not have any different in caret style between Edit and Normal -- that makes the work much harder, because no visual indication for the difference between them.

set blockstylecaret_mode=solid will show a difference in caret style, but it is fixed width and very difficult to work with on proportional text.

Plus, e function still takes a long time b/c requires text analysis ...
All of the above makes me ask: what has changed between the previous versions (say, 5.8) -- where the text analysis was much faster? What would it take to speed up text analysis in current version?
The above issues are major for working in Word, at least in my experience.
Sorry for "creating troubles" -- but just want to keep vind the coolest tool ever for working in Word ...
Thanks for your consideration of the above issues!

+1 to the above comment, word-motion is huge for me as I mainly use this tool for the vim-style text editing, and the lag is difficult to work with.


can't connect to kraken exchange

connect kraken results in an exception:
2023-02-01 16:23:39,553 - 1 - hummingbot.core.utils.async_utils - ERROR - Unhandled error in background task: local variable 'connector_config' referenced before assignment
Traceback (most recent call last):
File "/home/hummingbot/hummingbot/core/utils/async_utils.py", line 9, in safe_wrapper
return await c
File "/home/hummingbot/hummingbot/client/command/connect_command.py", line 63, in connect_exchange
await self._perform_connect(connector_config)
UnboundLocalError: local variable 'connector_config' referenced before assignment
Steps To Reproduce
Here are the steps to reproduce the issue (see attachments in the section below):

new HB instance, docker, v1.12.0
connect kraken

Screenshots
// If applicable, add screenshots to help explain your problem.
Release version
1.12.0
Optional: your discord username:
Ortal

Same problem here.

@rapcmia The issue was identified, this is not a potential "Bounty" correct? Anyway, it seems to be a hasty commit that introduced this bug just for Kraken

Hi @MementoRC good day,
Yes it was decided that HF would not add bounties for community fix for BRONZE connectors however it is possible for anyone from the community can host the amount to open a bounty. I check the issue and seems to be on the setup itself for the connector, Ill try to check with our team about it but we can't open a bounty for it. Thank you again for your time

@rapcmia The issue is a simple indentation (or addition of else: clause) that segregates the connector_config variable for Kraken


L2_error function; uj is undefined

Hi,
I'm working on documenting the callables, and i've noticed that the L2_error function body uses uj, which is not defined. Is it intended that we define this ourselves in the function body, or should it be replaced with uh?

It is a mistake, that part where uj is used should have been deleted.


Choose: Restyle the family cards

Following from #40, it would be good to make the style name labels smaller (8px), grey and not bold, and include their font-weight, more like in Google Fonts:
Google Fonts:

DDT:

Nice, except I see a bug, the regular is shown as,

Family undefined 400

I can't reproduce this issue on the newest version on gh-pages. Can you try again and let me know if it's still broken for you?

Fixed!


Created SQL_for_beginner

Added some important interview questions of SQL for beginner.
Created a folder named "SQL_for_beginner", and created a README.md file under the same folder.

sorry, but this PR doesn't fit the purpose of this repository


Add metadata fetch and selection feature with undo/redo and filename rename options

Add a new feature to enhance metadata management:
Workflow

Opens when "Download Metadata" is clicked for a selected song.
Displays probable metadata matches based on song title, artist name, and album name.

Fetching metadata

Retrieve probable matches including artwork, year, track number, and other relevant metadata.

User interaction

Allow selection of the best matching metadata.
Option to rename the audio file using the selected metadata.
Provide undo/redo functionality to revert or reapply changes.
Stage updates for user confirmation before applying.

Metadata fetch and selection view sketch
+---------------------------------------------------------------+
|                 Select Metadata for "Song Title"              |
+---------------------------------------------------------------+
| Fetching metadata matches for "Song Title" by "Artist" from   |
| "Album"                                                       |
| [Loading Spinner]                                             |
+---------------------------------------------------------------+
| | Artwork | Title       | Artist       | Album       | Year | |
| |---------|-------------|--------------|-------------|------| |
| | [img]   | Match 1     | Artist 1     | Album 1     | 2020 | |
| | [img]   | Match 2     | Artist 2     | Album 2     | 2019 | |
| | [img]   | Match 3     | Artist 3     | Album 3     | 2018 | |
| +-----------------------------------------------------------| |
+---------------------------------------------------------------+
|                      Selected Metadata                        |
+---------------------------------------------------------------+
| | Artwork Thumbnail |                                         |
| |-------------------|                                         |
| | Title:            | [Selected Title]                        |
| | Artist:           | [Selected Artist]                       |
| | Album:            | [Selected Album]                        |
| | Year:             | [Selected Year]                         |
| | Track Number:     | [Selected Track Number]                 |
| | Other Metadata... |                                         |
| +------------------------------------------------------------||
+---------------------------------------------------------------+
| Rename file with selected metadata [ ]                        |
+---------------------------------------------------------------+
| [Confirm]  [Undo]  [Redo]  [Cancel]                           |
+---------------------------------------------------------------+
| Status: [Status Message]  [Progress Bar]                      |
+---------------------------------------------------------------+

Acceptance criteria

Improved metadata accuracy and completeness.
Enhanced user experience with simplified metadata management.
Added flexibility with undo/redo options.
File renaming based on metadata.

File rename and track and year fetching still unimplemented

The track and year fields are not crucial for the core functionality of the application. If these fields are unavailable or incomplete, they can be left out without affecting the primary operations. The essential fields to focus on are the title, artist, album, and coverArt


Correct a traceback in vllm

What does this PR do?
We get this traceback:
File "/usr/local/lib/python3.10/site-packages/llama_stack/providers/adapters/inference/vllm/vllm.py", line 136, in _stream_chat_completion async for chunk in process_chat_completion_stream_response( TypeError: process_chat_completion_stream_response() takes 2 positional arguments but 3 were given
This corrects the error by deleting the "request" variable to match other inference adapters.
Mentioned in  #357
Feature/Issue validation/testing/test plan
Please describe the tests that you ran to verify your changes and relevant result summary.
python -m llama_stack.apis.safety.client localhost 5000

Thanks for the fix! This was due to https://github.com/meta-llama/llama-stack/pull/240 while remote:vllm wasn't merged yet at that time.


Default selection

Hi! I was wondering, is there any way to specify default selection?

I believe this might work:
<select-picker content={{mySelectContent}} selection={{myDefaultChoice}} action=(action "userPickedSomething")>

Then just preset the myDefaultChoice. Though I haven't tested this. It was a thought.

Could I remove the nothing selected option and make always the first selected by default?

The current logic has a lot of assumptions. The ability to customize it was an afterthought. This entire component needs to be refactored. Eventually I think it would be supported.
In the mean time the ember-power-select addon has a bootstrap theme and is far more customizable then mine.

Thank you for the tip! Anyway, congratulations for the work done, so far I don't need to use another one.


iam-eks-role cannot attach a policy created in the same module using role_policy_arns input

Description
When using iam-eks-role i cannot pass a role_policy_arn i created in the same module
Versions

Terraform: 1.1.4
Provider(s):

registry.terraform.io/hashicorp/aws: 3.74.0

Module:

terraform-aws-modules/eks/aws: 18.2.7
terraform-aws-modules/iam/aws//modules/iam-eks-role: 4.13.1

Reproduction
Using the code snippet below and
Code Snippet to Reproduce
resource "aws_iam_policy" "alb_load_balancer" {
    name        = "K8SALBController"
    path        = local.base_iam_path
    description = "Policy that allows k8s load balancer controller to provisione alb/elb"
    policy = file("${path.module}/policies/alb-policy.json")
}

module "aws_alb_controller_role" {
  source = "terraform-aws-modules/iam/aws//modules/iam-eks-role"
  //source  = "terraform-aws-modules/iam/aws//modules/iam-assumable-role-with-oidc"
  version = "4.13.1"

  create_role      = true
  role_name = "aws-alb-controller"
  role_path = local.base_iam_path
  role_description = "IRSA role for load balancer controller"

  role_policy_arns               = [aws_iam_policy.alb_load_balancer.arn]
  cluster_service_accounts = {
    "${var.cluster_name}" = [
      "kube-system:aws-alb-controller"
    ]
  }
  depends_on = [
    module.eks.cluster_id
  ]
}

Expected behavior
Policy should be attached
Actual behavior
I get the error below
Terminal Output
│ Error: Invalid for_each argument
│
│   on .terraform\modules\eks.aws_node_termination_handler_role\modules\iam-eks-role\main.tf line 76, in resource "aws_iam_role_policy_attachment" "custom":
│   76:   for_each = toset([for arn in var.role_policy_arns : arn if var.create_role])
│     ├────────────────
│     │ var.create_role is true
│     │ var.role_policy_arns is list of string with 1 element
│
│ The "for_each" value depends on resource attributes that cannot be determined until apply, so Terraform cannot predict how many instances will be created. To work around this, use the -target argument to first apply only the resources that the for_each depends on.

Additional context
Using
resource "aws_iam_role_policy_attachment" "custom" {
  role       = module.aws_alb_controller_role.iam_role_name
  policy_arn = aws_iam_policy.alb_load_balancer.arn
}

I am able to attach a policy.
Since this is a common policy i can user terraform-aws-modules/iam/aws//modules/iam-role-for-service-accounts-eks, but for a custom policy it wouldn't work

ya, this is an issue related to https://github.com/hashicorp/terraform/issues/4149
the policy you are referring to has to exist before it can be referenced in role_policy_arns (i.e. - terraform apply -target aws_iam_policy.alb_load_balancer.arn -y && terraform apply -y)

It's possible to write the module in a way that does not encounter this problem by taking a separate argument that is fully known, in this case the "name" of the policy, and using that in the for_each expression. I always prefer that pattern to avoid exactly this issue.

I don't follow, do you have an example?

To the O.P.s comment:

Since this is a common policy i can user terraform-aws-modules/iam/aws//modules/iam-role-for-service-accounts-eks, but for a custom policy it wouldn't work

Do you mean a custom policy in the sense of a customer managed policy? I was able to create a policy in a separate module and use aws_iam_role_policy_attachment to attach this new customer-managed policy to the role created by iam_role_for_service_accounts, and avoid the for_each issue.
module.eks_iam is a local module I'm using for IAM policy management, in the code block below.
module "iam_role_for_service_accounts" {
  source = "registry.terraform.io/terraform-aws-modules/iam/aws//modules/iam-role-for-service-accounts-eks"
  role_name = lower(var.cluster_info.name)
  version = "4.14.0"

  oidc_providers = {
    one =  {
      provider_arn = module.eks.oidc_provider_arn
      namespace_service_accounts = [ "${var.kube_namespace}:${var.kube_service_account_name}" ]
    }
  }
}

resource "aws_iam_role_policy_attachment" "serviceAccountPolicyAttach" {
  policy_arn = module.eks_iam.policy_arn
  role       = module.iam_role_for_service_accounts.iam_role_name
}

Screenshot of the Role created by this module, with my customer-managed policy attached:

I don't follow, do you have an example?

Yes, if you create a resource and also in the same config/state attempt to reference an attribute of that resource in the key of the for_each expression, then you will encounter the error mentioned in the OP:

The "for_each" value depends on resource attributes that cannot be determined until apply, so Terraform cannot predict how many instances will be created. To work around this, use the -target argument to first apply only the resources that the for_each depends on.

Note the "key" of the for_each expression is what must be known. That is important. The problem in this specific case is that the policy is being created in the same state, and so the ARN is unknown in a first apply, and the ARN is used as the for_each key.
My solution is to avoid referencing attributes of resources that are likely to be created in the same config/state. Instead of this:
resource "aws_iam_role_policy_attachment" "custom" {
  for_each = toset([for arn in var.role_policy_arns : arn if var.create_role])

  role       = aws_iam_role.this[0].name
  policy_arn = each.key
}

I would use either a map, or an object variable, and construct the for_each key from values that may be known in advance. Using a map(string), the key would be anything the user sets, presumably a "name" of the policy, and the value would be the ARN of the policy:
variable "role_policy_arns" {
  type    = map(string)
  default = {}
}

resource "aws_iam_role_policy_attachment" "custom" {
  for_each = var.create_role ? var.role_policy_arns : {}

  role       = aws_iam_role.this[0].name
  policy_arn = each.value
}

Using an object, you have a lot of flexibility in the data structure. I like lists of objects, but a map would be fine also.
variable "role_policy_arns" {
  type = list(object({
    name = string  # <-- used as for_each key, so cannot reference attribute of resource in same state
    arn  = string  # <-- only in the for_each value, so it is fine to reference attribute of resource in same state
)
  default = []
}

resource "aws_iam_role_policy_attachment" "custom" {
  for_each = {for policy in var.role_policy_arns : policy.name => policy if var.create_role}

  role       = aws_iam_role.this[0].name
  policy_arn = each.value.arn
}

thank you for sharing this @lorengordon - I will dive in tomorrow and check it out

@bryantbiggs were you ever able to figure out a solution?

Running into this as well, as I create a policy that this module doesn't have. I believe, and I'm not 100% sure so someone might want to test this, you can probably use vars in the depends_on field, in the resource. This essentially would tell Terraform that it has to wait on whatever is behind that variable (i.e. Terraform can't provide the variable until the resource is created).
If the above is the case, then I'd assume this would be a very easy fix. This stackoverflow post is a bit outdated, but I'm curious if this still applies today. It would make sense that it does.
It would also be nice to simply add a policy_inline field, where you can provide your own JSON, and it will inline the policy into the created role. Most of our policies were previously inlined anyway, and it has the benefit that the module has full control of its creation/destruction.

PR is up to resolve this if anyone wants to give it a try #250

And big thank you to @lorengordon for pointing out the workaround, this will help a lot as well as in other modules so thank you

This issue has been resolved in version 5.0.0 :tada:


ValueError: The channel dimension of the inputs should be defined. Found None.

How can I fix this?
I try to convret dartnet weights and cfg to keras *.h5 using convert.py.
This problem also exist 'python train.py'.
BUT 'python yolo_video.py --image' work.
info: tf1.10.0 keras2.2.4 CUDA9.2 cuDnn7.2.1 win10

I have the same problem. Have you solved this now?

how to solve the problem ？


Setup the food recipe project

Hi, we are two people in our project
@jubaer919
@Megagig

Here is the project link:

link

Hi @jubaer919 and @Megagig,  👋🏻👋🏻
Your project is complete!  There is nothing else to say other than... it's time to merge it :shipit:
Congratulations! 🎉

Highlights

[ ]  Descriptive PR⭐
[ ]  Kudos for working on the kanban board🤩
[ ]  Great job assigning the tasks to yourself 🚀
[ ] Good job adding a description and title to each issue 💪🏻

Optional suggestions
Every comment with the [OPTIONAL] prefix won't stop the approval of this PR. However, I strongly recommend you to take them into account as they can make your code better. Some of them were simply missed by the previous reviewer and addressing them will really improve your application.
Cheers, and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, remember to tag me in your question so I can receive the notification.

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


The regex in tmLanguage definition file cannot correctly match Unicode characters

VSCode Version:1.0.0
OS Version:Windows 10 professional (10.0.10586.0)

Steps to Reproduce:

Create a new colorization extension, and set the patterns as [\u0080-\uffff]
debug it.

The result is:

While if I search it in editor itself, the result looks correct.

@gjy1992 The colorization is based on TextMate grammars, which were built and defined by TextMate using the oniguruma regex library. When we colorize, we use the oniguruma regexes to evaluate text. Syntax at https://github.com/kkos/oniguruma/blob/master/doc/RE

OK, thank you.


diagnostics gaps

Environment info

Version: 2.1.3-b3379aa
Deployment: ESX
Customer: ** EGA/RAK**

Actual behavior
1. agent diagnostics does not return diags for all drives ~~
~~ - when taking diagnostics of a host with multiple drives we actually collect diags for one of the drives
2. when taking agent diagnostics we should add read_host_mappings output to diags

currently, read_host_mappings is very heavy (~40K per entry of an object). a lot of the data is unnecessary (digests and stuff) for both diagnostics and the UI. we should maybe redefine the API to only return the necessary data and use it foe UI and diags

Expected behavior

diagnostics should be collected for all drives
should collect shared information only once (e.g. noobaa.log is shared)

Steps to reproduce

Screenshots or Logs or other output that would be helpful

1 is irrelevant

Not relevant since moving to pv pool based model


⚠ SceneTime has degraded performance

In 5105478, SceneTime ($ST) experienced degraded performance:

HTTP code: 200
Response time: 1612 ms

Resolved: SceneTime performance has improved in ec32b2a after 6 minutes.


Update 'approleassignment add' to v1.0 API. Closes #1858

Update approleassignment add to v1.0 API. Closes #1858

Awesome @aakashbhardwaj619. We'll have a look at it asap 👏

Merged manually. Thank you! 👏


Update README - lock version to master instead of v1.0

The README goes into detail about why you should always reference a version in your pipelines
The initial part of the README when setting up the library shows setting the default version to v1.0
There's no really good reason to set the default version to v1.0.  Either update it to the latest, or update it to master.

From Jantman:
For what it's worth:
I really, really think that pinning to versions in pipelines (and pretty much everywhere else) is the right thing to do. If that isn't done, inevitably someone will need to agonize between not releasing a potentially-very-useful change or breaking customers' pipelines (perhaps in a non-obvious or dangerous way).
On the other hand...
I've left in my wake a giant pile of READMEs (for projects in various languages) that have installation examples showing really old version numbers. So... short of either (1) including a note after the example to check for the latest version number or (2) having the example literally pin to @<Latest Version - See Link Below>, I'm not sure what the clean solution to this is.


Document that "f" and "F" cannot be used in combination.

I would have hoped that I could display a date time with a minimum of 3 digits of milliseconds and a maximum of 6 digits, but unfortunately this doesn't appear to be supported.
mm:fff will produce:
2009-06-15T13:45:30.6174000 -> 30:617
As the documentation doesn't indicate, the expected result of mm:fffFFF could well have been, but isn't:
2009-06-15T13:45:30.6174000 -> 30:6174
2009-06-15T13:45:30.6100000 -> 30:610
Instead, mm:fffFFF will produce:
2009-06-15T13:45:30.6174000 -> 30:617617

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

ID: dda93f14-f1b5-fde6-e441-302cdac261ab
Version Independent ID: 96abc7c4-3719-82da-e271-881d62f0640f
Content: Custom date and time format strings
Content Source: docs/standard/base-types/custom-date-and-time-format-strings.md
Product: dotnet
Technology: dotnet-standard
GitHub Login: @Thraka
Microsoft Alias: adegeo

I don't think you can use anything in "combination" in that way. Formatting a string means using it as a template. Wherever f is in the string it replaces it with the seconds fraction. Wherever F is, it replaces it with the seconds fraction.
Also, F says that it trims trailing 0's so it wouldn't work for you even if you could combine them some how. 😞


Debug apk is  different with the release apk

I copied a demo from gallery-animation and I changed something.  The demo running well on debug mode(Android) ,but when I packaged on released mode ,the demo's animation  not worked。

Can you post a reproduction of the issue? We will need more information to help you resolve it.

@mraleph Sorry about that.I copied these files : home.dart, sections.dart, widgets.dart

I rename the  'home.dart'  to 'main.dart'  and I  added the 'main'   function in this dart file .

Like this,if I run it on debug mode(Android studio),the animation is the same of Gallery.But I build release with "flutter build apk",the animation is disappeared.
I hope you can understand my bad English.

@mraleph I got a picture:

the release apk like this,the auto_animation is not worked

Thanks! Can you also attach the output of flutter doctor so that we know which version of Flutter you are running?

@mraleph I tried to release the Gallery ,the issue disappeared,   it worked well   on both mode.   But when I copied the Animation module out  and  released it , the problems arise。

@pp126 I have tried reproducing this and unfortunately I was not able to.
I copied the animation module out and run it on its own - and works both in release and debug mode for me.
Could you put the code that reproduces the problem for you somewhere, e.g. create a repo on GitHub and put all the code there.
(Note: one thing to note is that animation demo also depends on some assets from flutter_gallery_assets. Did you copy those?)

@mraleph Thanks a lot,I have pulled my code on GitHub's repertory。https://github.com/pp126/animation_test

@mraleph Excuse me.I checked my code again,I couldn't found the problem out.Help please.

Unfortunately, I don't have access to either. @Hixie can somebody try to repro on S6/S8 if we have those in the device lab?

@mraleph Thank you very much. I will check my code again and test it on more devices.

@pp126 Is this still an issue?

Without additional information, we are unfortunately not sure how to resolve this issue. We are therefore reluctantly going to close this bug for now. Please don't hesitate to comment on the bug if you have any more information for us; we will reopen it right away!
Thanks for your contribution.

@zoechi Please open this issue.  Today I tried  again. I copy the same code from GitHub(https://github.com/pp126/animation_test),when I run with debug mode on Android device ，animation it's woked. But I run released mode ,it's not woked.
Flutter version: stable, v1.2.1

@mraleph I think you can reappearance of this bug .1,create a new flutter project .2,copy the dart code from https://github.com/pp126/animation_test.

Hi,
could you please tell me if this bug still appears on the latest stable version?

@escamoteur I have builded a release apk again.This bug appeared again.
code : https://github.com/pp126/animation_test

@escamoteur
,  This is my SDK version.

could you please try again with creating a new project and copy the files over? There has been a lot of changes in the meantime.

Could you perhaps make an gif of what happens in debug and what happens in release mode?

@escamoteur I created a new project,It dosen't work.

@pp126 thanks for sending the Video, I'm not sure what is wrong?

@escamoteur Sorry!  I mean  this page widget's spring disappeared. when I slide the widget,it should be auto slipping to top or bottom.But it doesn't work on release apk (debug mode is ok).

Could you please add a gif of the debug behaviour?

@escamoteur

I'm not sure that I saw a difference. Could you try to describe t so that I know what to look for?

@escamoteur

Hi @pp126
are you still experiencing this issue with the latest stable version of flutter?
If so can you please provide your flutter doctor -v
and a reproducible minimal code sample?
Thank you

Without additional information, we are unfortunately not sure how to resolve this issue. We are therefore reluctantly going to close this bug for now. Please don't hesitate to comment on the bug if you have any more information for us; we will reopen it right away!
Thanks for your contribution.
Could everyone who still has this problem please file a new issue with the exact descriptions what happens, logs and the output of 'flutter doctor -v'  please.
All system setups can be slightly different so it's always better to open new issues and reference related issues.


[SPARK-40938][CONNECT][PYTHON][FOLLOW-UP] Fix SubqueryAlias without the child plan when constructing Connect proto in the Python client

What changes were proposed in this pull request?

The SubqueryAlias in plan.py does not set child plan when constructing Connect proto. This PR fixes that issue by setting up the input field during proto construction.
Why are the changes needed?

Bug fix.
Does this PR introduce any user-facing change?

NO
How was this patch tested?

UT

Can one of the admins verify this patch?

Merged into master


No response received (possibly my adapter?)

Thank you for this amazing repo.  I wanted to share my "No response received" error with you, but I'm not sure if it is code or hardware to be honest.
My Hardware
I have a Pioneer WYS024GMFI22RL-10 with a separate OSK-103 Smart Key adapter.
My Mod
I ordered an ESP-01s with a level shifter (dipswitch PROG / UART version) and flashed ESP-Link 2.2.3v and successfully configured it for remote connections.  It turns out the USB for my OSK-103 was "keyed" and wouldn't fit in the port.  So, I de-soldered the USB port off of my OSK-103, and soldered it to my ESP-01S to USB adapter.  After completing the USB transplant, the Pioneer powers on the custom ESP key.
The Problem
I can't seem to successfully control the unit with any repos from Home Assistant / HACS, and the node project I created with your repo keeps giving me "No response received."  I'm wondering if the the custom ESP key might need to bypass the built-in USB to TTL adapter that comes with the Pioneer?  In other words, maybe it will work if I:
[ESP + Level Shifter] ----> (bypass built-in USB port / TTL adapter) -----> [custom USB cable directly to control board]
Before I mess with code any further, do you happen to know if my hardware config would be problematic?

app.js
`var express = require('express');
var path = require('path');
var cookieParser = require('cookie-parser');
var logger = require('morgan');
const axios = require('axios').default;
var indexRouter = require('./routes/index');
var app = express();
app.use(logger('dev'));
app.use(express.json());
app.use(express.urlencoded({ extended: false }));
app.use(cookieParser());
app.use(express.static(path.join(__dirname, 'public')));
app.use('/', indexRouter);
indexRouter.get('/mode', function(req, res) {
console.log('\nReceieved serial bridge mode request\n');
var mode = req.query.set;
var path = req.path;
var reqData = JSON.stringify(req.query);
console.log("reqPath: " + path + '\n');
console.log("reqData: " + reqData + '\n');
console.log('Mode: ' + mode + '\n');
setMode(mode);
return res.send("ok");
});
indexRouter.get('/getCapabilities', function(req, res) {
console.log('\nReceieved serial bridge capabilities request\n');
var path = req.path;
console.log("reqPath: " + path + '\n');
getCapabilities();
return res.send("ok");
})
module.exports = app;
const appliances = require('node-mideahvac')
var options = {
communicationMethod: 'serialbridge',
host: '<IP_ADDRESS>',
port: 23
}
var ac = appliances.createAppliance(options)
ac.on('status-update', data => {
console.log(Received updates: ${JSON.stringify(data)})
})
ac.initialize()
.then(response => {
console.log(Initialized: ${JSON.stringify(response)})
setInterval(() => {
  ac.getStatus()
    .catch(error => {
      console.log(`Error: ${error.message}`)
    })
}, 30000)

})
.catch(error => {
console.log(Error: Failed to initialize (${error.message}))
})
function setMode(mode) {
console.log("Received setMode command on server\n");
switch (mode) {

  case 'heat':

    ac.setStatus({ powerOn: true, setpoint: 75, temperatureUnit: 'fahrenheit' })
  .catch (error => {
    console.log(error.message)

  })

}

}
function getCapabilities() {
console.log("Received getCapablities command on server\n");
ac.getCapabilities(true)
.catch(error => {
console.log(error.message)
})
}
// ac.setStatus({ powerOn: true, setpoint: 24 }, true, true)
//   .catch(error => {
//     console.log(error.message)
//   })
// ac.setStatus({ powerOn: true, setpoint: 73 }, true, true)
//   .catch(error => {
//     console.log(error.message)
//   })
// ac.getCapabilities(true)
// .catch(error => {
//     console.log(error.message)
// })`
index.html
`<!doctype html>

Web Interface for Serial Bridge

`

First of all I am not aware of anyone else using the Pioneer model you are using but if the Midea Air App worked in combination with the OSK-103 the custom dongle should also work. The custom dongle should be connected to the AC exactly like the OSK-103 was connected. No response received seems to indicate there is no communication between the dongle and the AC. Can you provide any logs?

Yes, the Midea Air app does connect to the Pioneer.
I have logs from the ESP web server, but I don't think I have any on node-mideahvac.
The esp-link debug logs shows the following:
325816> serbridge: connection reset err=-9 <-- this was me resetting the node server
327660> Accept port 23, conn=0x3fff65b0, pool slot 0 <-- initialize command
327661> Accept port 23, conn=0x3fff63b8, pool slot 1 <-- this was me resetting the node server
This is the code that I am testing
const appliances = require('node-mideahvac')
var options = {
communicationMethod: 'serialbridge',
host: '<IP_ADDRESS>',
port: 23
}
var ac = appliances.createAppliance(options)
ac.on('status-update', data => {
console.log(Received updates: ${JSON.stringify(data)})
})
ac.initialize()
.then(response => {
console.log(Initialized: ${JSON.stringify(response)})
setInterval(() => {
  ac.getStatus()
    .catch(error => {
      console.log(`Error: ${error.message}`)
    })
}, 30000)

})
.catch(error => {
console.log(Error: Failed to initialize (${error.message}))
})
ac.setStatus({ powerOn: true, setpoint: 75, temperatureUnit: 'fahrenheit' }, true, true)
.catch(error => {
console.log(error.message)
})

if you add these lines to that code you will get debug logs:
`
const logger = require('winston')
logger.remove(logger.transports.Console)
logger.add(new logger.transports.Console({
format: logger.format.combine(
logger.format.timestamp(),
logger.format.colorize(),
logger.format.printf(event => {
return ${event.timestamp}: ${event.message}
})
),
level: 'silly'
}))
`

Thanks - here is the log.  Side question, are those hex values something I could test through the esp-link microprocessor or through Hercules TCP serial client?
2022-01-19T20:28:27.845Z: SerialBridge._processQueue: Entering
2022-01-19T20:28:27.846Z: SerialBridge._processQueue: Sending 'sendNetworkStatusNotification' command
2022-01-19T20:28:27.847Z: SerialBridge._write: Write aa1eac0000000000030d010104f401a8c0ff000101010000000000000000c1
2022-01-19T20:28:27.847Z: SerialBridge._processQueue: Entering
2022-01-19T20:28:27.847Z: SerialBridge._processQueue: Command in progress
2022-01-19T20:28:28.851Z: SerialBridge._processQueue: No response received in time for 'sendNetworkStatusNotification' command
2022-01-19T20:28:28.852Z: SerialBridge.initialize: Failed to send network status notification for <IP_ADDRESS>:23 - No response received
2022-01-19T20:28:28.852Z: SerialBridge._processQueue: Entering
2022-01-19T20:28:28.852Z: SerialBridge._processQueue: Sending 'setStatus' command
2022-01-19T20:28:28.853Z: SerialBridge._write: Write aa24ac000000000003024043080000000030000004000000000000000000000000000077f5
2022-01-19T20:28:29.856Z: SerialBridge._processQueue: No response received in time for 'setStatus' command
No response received

These hex values are the data that is send to the AC.
Apparently there is no communication between the dongle and the AC. This can have various causes, e.g.:

Faulty level shifter
Swapped rx and tx lines

You can try to swap the rx and tx connections.
Do you have more information on the ESP and level shifter you are using?

Rx / Tx are technically soldered, but here is what I'm using.
ESP + Level Shifter
I also picked up one of these if I decided to use an Arduino:
USB to TTL Converter

I might have found something.  The ESP-01 to USB shows [ 5v, gnd, +, - ]in this schematic but the [built-in converter](<img src="https://i.ibb.co/vkqWp2W/PXL-20220112-201823371.jpg) appears [ 5v, +, -, gnd ]
I'll try to make a breakout cable tomorrow to test this fix.

This will never work. You bought the wrong adapter. The interface between the dongle and the AC is not a real USB interface it only uses an USB connector. The adapter you bought contains an USB chip (CH340G) handling the communication over the USB connector (actually converting serial TTL to USB). The communication between the dongle and the AC is just plain serial TTL. Also that USB to TTL converter you have bought cannot be used for the same reason. You need something like this: https://nl.aliexpress.com/item/4001054021801.html

Thanks, I've ordered the proper adapter.

New adapter came in, and oddly, I'm still getting the same results.  I tried swapping rx/tx without any luck.  I noticed someone reported that the WYS 18 model was working, so I assume the WYS 24 (that I have) should work as well.  I continuity tested my connections to double check, but still no comms for some reason.

Yes I connect the original dongle as well as the custom dongle to my macbook when I am developing while reverse enginering. You need a ttl to usb convertor for that, I use something like this: https://nl.aliexpress.com/item/32273550144.html?spm=a2g0o.productlist.0.0.61926b81dQNfBY&algo_pvid=b2b9889e-2b4b-4b8c-92e2-e5e25268f99b&aem_p4p_detail=202201251437364196101537621480018464008&algo_exp_id=b2b9889e-2b4b-4b8c-92e2-e5e25268f99b-2&pdp_ext_f={"sku_id"%3A"12000021370119598"}&pdp_pi=-1%3B1.33%3B-1%3B-1%40salePrice%3BEUR%3Bsearch-mainSearch
You can try changing the pin assignment preset in esplink. I do not recall which setting I used and if there was a ESP01 preset available.
At the moment I use the following dongle: https://github.com/reneklootwijk/mideahvac-dongle with custom firmware, but before that I used an ESP01 with the levelshifter and esplink as firmware.
At th moment I am refactoring this nodejs code to support direct communication with the original dongle with dependency on the cloud. I need some days to finish this.

Just a heads up, I had a lot of fun w/ this project from a learning perspective.  It ultimately didn't pan out, but I did find a solution.
Initially, my goal was to control the mini-split with a Nest Thermostat using an Arduino to translate the 24v thermostat signal.
However, I found that the Samsung Smart Things hub can detect the Nest thermostat.  I use a trigger event from the Nest (i.e. when heat is turned on) in conjunction with a Switchbot Mini (IR blaster) to send the proper command for control.  This works perfectly so far.  It doesn't send a specific temperature to the VRF, but it does work with pre-programmed heating and cooling temps (or fan only as well).
It seems like the manufacturers want us to leave our mini-splits on basically 24/7 and allow their inverters to ramp up/down as needed, but I still feel like having the split shut off / on could be beneficial for energy savings.  Time will tell!


[Feature Request] Support Alerting

Support for alerting would be great, I see there was a Feature Request from a ling time ago that was closed.
https://github.com/Vertamedia/clickhouse-grafana/issues/3
Current backends supporting alerting.
https://grafana.com/docs/alerting/rules/#execution
Grafana Alerting looks like a possible replacement for InfluxDB+Kapacitor.

Could you give some examples about how to support alerting ?

Frankly speaking, it doesn't look like production ready - https://github.com/grafana/grafana-plugin-model

Ah, yeah, I didn't appreciate there was such a distinction of datasource(DS) plugins.
If Im understanding correctly, Alerting works for internal DS plugins, for a DS plugin that isn't internal, not everything is in place in Grafana to allow the plugin to support Alerting.
So either ClickHouse plugin would have to be internal, or support needs to be completed in Grafana to allow not internal plugins to be also support alerting.

Correct! External plugins are frontend-only and implemented in JS. Alerting requires some processes to run on Grafana's backend, because alerting should always work, even if user closed tab in browser. So there should be a way how to put all plugin's logic for sending requests and parsing responses on backend. So plugin authors should implement all this logic in Go (which is Grafana backend language), and Grafana authors find a way how to run this logic without harm to main process.
When this plugin was created few years ago, I also implemented alerting. But that's was possible only because I was running my own fork of Grafana. However, I wasn't able to keep up with Grafana's development pace so fork become outdated very fast.

@hagen1778  If We want to support clickhouse alert by ourselves. How much cost we need ?
any difficulty ？

@wuguanyu
Can't say, haven't looked for a year... You may want to check following code for examples:

backend branch - https://github.com/grafana/simple-json-datasource/tree/backend-updated
druid plugin - https://github.com/GoshPosh/druidplugin/tree/druid_alert

The parser in JS for CH responses may be found here - https://github.com/Vertamedia/clickhouse-grafana/blob/master/src/response_parser.ts
I have no idea if current Grafana's API for backend plugins is stable, so be ready for changes in future.

@hagen1778  Thanks a lot

closed by https://github.com/Vertamedia/clickhouse-grafana/pull/183
need to add dist/MANIFEST.txt and implements sign process
also need refactoring from  https://github.com/grafana/grafana-plugin-model/ to https://github.com/grafana/grafana-plugin-sdk-go


Upgrade to ui-router v1.x

Description
Current version of angular ui-router (0.3.x) does not support to route to angular 1.5 components.
However, last version (currently 1.0.0-beta.1) supports a new 'component' attribute, with various syntaxes.
Let's upgrade, or use this polyfill: https://github.com/softsimon/ui-router-route-to-components
We should also add an example in the gererator.
e.g. for a component named "fleet":
fleet/fleet.js
module.exports = {
  templateUrl: 'app/fleet/fleet.html',
  controller: Fleet,
  bindings: {
    todos: '=',
    filter: '<'
  }
};

/** @ngInject */
function Fleet($log) {
  this.$onInit = function () {
    $log.info('Hi there!');
  };
}

index.js
var FleetComponent = require('./app/fleet/fleet');
// [...]
angular
  .component('fleetComponent', FleetComponent);

route.js
.state('fleet', {
            url: '/fleet',
            component: 'fleetComponent'
        })

Config
{
  "generator-fountain-angular1": {
    "version": "0.5.4",
    "props": {
      "framework": "angular1",
      "modules": "webpack",
      "js": "js",
      "css": "scss",
      "resolved": "/usr/local/lib/node_modules/generator-fountain-webapp/node_modules/generator-fountain-angular1/generators/app/index.js",
      "namespace": "fountain-angular1",
      "argv": {
        "remain": [],
        "cooked": [],
        "original": []
      },
      "skip-cache": false,
      "skip-install": false,
      "sample": "todoMVC",
      "router": "uirouter"
    }
  }
}

Relevant Links

https://github.com/angular-ui/ui-router/issues/2627
https://github.com/angular-ui/ui-router/issues/2547

releated line code
https://github.com/FountainJS/generator-fountain-angular1/blob/master/generators/app/index.js#L59

https://github.com/FountainJS/generator-fountain-webapp/issues/65

Hi, can any one update when will be the next release so that i can assume that this feature is available?

Fixed in 0.7.0 (just released) 😉  @madasuk


Added support for Google Cloud Storage repositories using the existin…

…g s3 support to address #211.
Added a new package to handle GCS configuration. GCS repositories get specified
using gs://bucketname/prefix syntax, similar to gsutil.
Added tests for the various cases to config_test and location_test.

Thanks for your contribution!
Can you describe why this needs a separate prefix? As far as I understood the code, it is sufficient to call restic with --repo s3:https://storage.googleapis.com/bucket/prefix. Why the extra gs:?

It is equivalent, but I think it is easier to use that way.
This is similar to the s3:region/bucket/prefix shorthand for s3:https://s3-region.amazonaws.com/bucket/prefix

It is equivalent, but I think it is easier to use that way.

I appreciate your work, but to be honest, I think that this may not be enough for justifying 200 lines of extra code. Code that we have to maintain, and a URL scheme we'll never get rid of once users start using it: we'll have to support it for a long time.
What about the other services the minio s3 library supports (OpenStack Swift, Ceph, Riak), should we also add custom backend scheme names and extra code for them?
What do you think about documenting how to use the existing s3 backend to use GCS, instead of adding extra code?

Well the 200 lines are mostly tests. In theory since this is a simple
translation to the s3 scheme for now, we could reduce them and rely on the
existing s3 tests. I could even call s3.createConfig() to save an
additional dozen lines.
I do understand your concern regarding supporting the Url scheme, though.
Long term I would like to implement proper support for GCS using the native
security features of the platform instead of the legacy S3 secrets. Is that
something that you are still interested in supporting in Restic? Or do you
consider the existing support through minio good enough?
I assumed that from the existing discussion on #211 (and
https://github.com/restic/restic/pull/304 which you closed three days ago)
that you would like to see support for GCS but wanted it to pull in fewer
dependencies.  So I decided to support it using the existing minio library
as a first step. It certainly wasn't obvious to me that Restic supports GCS
before I started tinkering with it.
I will definitely add something to the documentation to make it easier to
discover.
On Tue, Feb 16, 2016 at 12:11 PM, Alexander Neumann <
<EMAIL_ADDRESS>wrote:

It is equivalent, but I think it is easier to use that way.
I appreciate your work, but to be honest, I think that this may not be
enough for justifying 200 lines of extra code. Code that we have to
maintain, and a URL scheme we'll never get rid of once users start using
it: we'll have to support it for a long time.
What about the other services the minio s3 library supports (OpenStack
Swift, Ceph, Riak), should we also add custom backend scheme names and
extra code for them?
What do you think about documenting how to use the existing s3 backend to
use GCS, instead of adding extra code?
—
Reply to this email directly or view it on GitHub
https://github.com/restic/restic/pull/449#issuecomment-184858989.

I extended the s3 implementation to allow a different repository layout (similar to what I asked for in #478) and used that new layout in the GCS implementation. I was able to copy a local restic repository into GCS and successfully ran restic check on it.
Let me know what you think.

Hey, thanks for your work. In order to ease the review this PR, may I ask you to:

Rebase the commits on the current master branch. This will be a bit of work since we switched to gb and gb-vendor for development, and the file system structure was changed.
Drop the commits that aren't relevant any more, and remove the .a files that went in by accident.

If I understand your code correctly, you implemented a new backend-scheme gs which uses the minio backend with a structure compatible to the local and sftp backends, i.e. with the packs below a second level of subdirectories (data/aa/bb[...]). Is that correct?

I had already rebased my pull request on the master branch after you made the switch to gb. I guess the Godeps were a remnant of the switchover. I removed those files from the pull request.
All the changes in this branch are necessary: I first introduced the gs: scheme to write to GCS, simplified it a bit and then changed the layout of the GCS repository by adding the handling of different layouts to S3.
So in the end this whole change introduces a way to store restic backups in GCS that can be easily synced on a filesystem basis.
As to if this is better than copy is up for debate, I guess. On one hand I can see the desire of not leaking backend implementation details, but on the other hand once you start writing backups it will be difficult to change the backend storage model without writing the complete repository again. So if two repository implementations happen to have the same layout on disk they will be synced for a while.
If we store the backups in a compatible way on filesystem we can give people the flexibility to use different ways to synchronize their backups between the different storage models. i.e. you can run restic backups to local storage, rsync that to your sftp server periodically and then simply restore directly from the sftp repository. That already works in the current implementation and I added support to make that work with the cloud storage as well: Sync your repository to S3/GCS/OneDrive etc. periodically in case you lose your local backup and then restore from the cloud in case of disaster.
As an aside: Another interesting side effect of my change is that you can experiment with the efficiency of local caching. Running a backup to a local repository and syncing to the cloud is much faster than running the backup the cloud directly. Ideally restic can approach the same speed in the long run.

One more thing: I noticed that the test coverage went down because there are no tests for the new repository layout. What is the best way for me to test this? i.e. where is the entry point for the existing S3 tests, so I can inject a different Config? I can see the Config in s3_test.go, but I don't know how that gets used during testing or how I can reuse that for GCS testing.

Just a quick heads-up: I haven't forgotten, I just don't have any time at the moment.

I'm really sorry that I didn't give any further feedback on this PR. I thought about it, though, and I'd like to move forward in supporting GCS with a native backend. I hope you're still interested! :)
I'm going to close this PR for now, let's move the discussion to the issue (#211).
As a side note, I've just merged #741 which slightly changes the backend API.


Update init.py

Get Last Value with parameter from now.
Works for me localy. Maybe there an elegant way to get the last value with only one http query, but this works for me.
Tested in my local Home Assistant instance.

I close this and make an new one with your tls changes.