Bump pocketsphinx/sphinxbase

Description of changes
Pocketsphinx was last updated >5 years ago in nixpkgs. Though upstream has yet to put out a tagged release, this update should rid us of more Python 2 dependencies (#148779).

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[x] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
22.05 Release Notes (or backporting 21.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[ ] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 169609 run on x86_64-linux 1

3 packages built:

parlatype
pocketsphinx
sphinxbase

I wouldn't be opposed to removing it altogether. Only parlatype seems to depend on this, and parlatype is unmaintained in nixpkgs as well (the only bumps I've seen are from r-ryantm and the original submitter @melchips never left any review comments on bumps). (Parlatype is still working though, I tested it with this PR)
There's also a haskell package (cut-the-crap) depending on this, but it's marked broken...
So I think it may be better to remove these packages.

Made a PR to remove everything. #170124


Fails to save pages that have a # character in the page's title

I'm not sure if this is a problem with the extension, or the API being used by the extension.
Steps to recreate:

Go to https://fsharpforfunandprofit.com/rop/
Click on Logseq Quick Capture

Expected:
Page to be entered into Logseq with the full title and URL linked:
**12:16** [[quick capture]]:  [Railway Oriented Programming | F# for fun and profit](https://fsharpforfunandprofit.com/rop/)
Actual:
**12:16** [[quick capture]]:  Railway Oriented Programming | F
text including and after the # is cut-off.

I have included a fix in v0.2.3. It is currently pending the webstore review.

Thanks for the prompt fix!

Fix should be in already. Let me know how it goes.

Update works great, thanks!


correct "requirements" without "s"

Install Required Libraries:

pip install -r requirements.txt

but : https://github.com/Asi0Flammeus/LLM-Translator/blob/main/requirement.txt

Resolved with commit aeae04f. Requirements file has been renamed and cleaned up with only necessary libraries.


Mark derived code as #[automatically_derived]

This should help suppress some of the clippy warnings that keep popping up

Fixed by 09dd49342b8b9de42c9bbccaf121a16d49cb6472


Target link lib

Reduces scope of many library symbols

I forgot, but either a unittest or one of the calculator needed symbols.

Ok


ParseTests TestArrayParse and TestMultilineValueParse failure

This is a great project!  Please keep it up!
I just want to note that TestArrayParse and TestMultilineValueParse will fail on Windows in some cases if there is a newline mismatch.
In these cases, the "input" variable has newlines encoded in the .CS file as \n but the tests check for equality with newlines encoded with \r\n.  Aside from the newline mismatch, the tests work perfectly.
> dotnet test TommyTests
Microsoft (R) Build Engine version 16.4.0-preview-19529-02+0c507a29b for .NET Core
Copyright (C) Microsoft Corporation. All rights reserved.
Test run for C:\tommy\TommyTests\bin\Debug\net46\uMod.Tests.dll(.NETFramework,Version=v4.6)
Microsoft (R) Test Execution Command Line Tool Version 16.3.0
Copyright (c) Microsoft Corporation.  All rights reserved.

Starting test execution, please wait...

A total of 1 test files matched the specified pattern.
  X TestArrayParse [61ms]
  Error Message:
   Assert.AreEqual failed. Expected:<Just to make sure
we still work as expected
because reasons>. Actual:<Just to make sure
we still work as expected
because reasons>.
  Stack Trace:
     at TommyTests.Utils.TomlNodesAreEqual(Assert assert, TomlNode expected, TomlNode actual, Boolean ignoreComments) in C:\tommy\TommyTests\Utils.cs:line 20
   at TommyTests.Utils.TomlNodesAreEqual(Assert assert, TomlNode expected, TomlNode actual, Boolean ignoreComments) in C:\tommy\TommyTests\Utils.cs:line 41
   at TommyTests.Utils.TomlNodesAreEqual(Assert assert, TomlNode expected, TomlNode actual, Boolean ignoreComments) in C:\tommy\TommyTests\Utils.cs:line 50
   at TommyTests.ParseTests.TestArrayParse() in C:\tommy\TommyTests\ParseTests.cs:line 338

  X TestMultilineValueParse [1ms]
  Error Message:
   Assert.AreEqual failed. Expected:<The first newline is
trimmed in raw strings.
  All other whitespace
  is preserved.
>. Actual:<The first newline is
trimmed in raw strings.
  All other whitespace
  is preserved.
>.
  Stack Trace:
     at TommyTests.Utils.TomlNodesAreEqual(Assert assert, TomlNode expected, TomlNode actual, Boolean ignoreComments) in C:\tommy\TommyTests\Utils.cs:line 20
   at TommyTests.Utils.TomlNodesAreEqual(Assert assert, TomlNode expected, TomlNode actual, Boolean ignoreComments) in C:\tommy\TommyTests\Utils.cs:line 50
   at TommyTests.ParseTests.TestMultilineValueParse() in C:\tommy\TommyTests\ParseTests.cs:line 561

Test Run Failed.
Total tests: 11
     Passed: 9
     Failed: 2

Thank you for considering this.

Hi,
thanks for pointing this out! The problematic behaviour depends on your git configuration: if you checkout files in CRLF, the tests do pass fine on Windows. However, if you checkout in LF, the tests do indeed fail.
Naturally, the same issue persists on Linux but in a reversed way.
I'll update the assertion helper to make it agnostic of newline styles for the time being.
Thanks again!


Traffic Dump: Add an HTTP/3 AuTest

Adding an HTTP/3 test for Traffic Dump. This test is currently skipped
because ATS does not have the TS_EVENT_HTTP_SSN_* hooks implemented for
QUIC connections yet. Although the test is skipped, I plan to commit
this change now because it adds the following value:

With minor modifications the test should work when these hooks are
implemented.
The test can be used to help development when we decide to implement
these hooks for QUIC.
This test provides an example AuTest for completed HTTP/3 transactions
which we currently do not have.
This commit updates the Proxy Verifier extensions for HTTP/3 which will
be helpful for future QUIC tests using Proxy Verifier.

Cherry-picked to 9.1.x, to resolve some merge conflicts for other PRs.


Support for incrementing and decrementing counter cache columns on associated objects for ActiveRecord>=4.x

Hi, we came across a problem while we were developing our software which was that counter cache columns weren't being decremented when deleting an associated object and they were also not being increment when recovering an associated object. I think I managed it pretty well with help from pull request #40  .

i could not find anything in the documentation. are counter-caches currently supported?

@phoet  No, counter-caches are currently not supported. I'm sorry my initial comment was misleading. What I meant was that it would be useful to have that functionality.

i think that it would be worth putting in some caveat section of the readme, unless ofcourse, you fix it through this PR 😄

@phoet this PR should fix it, but maybe I should also update the readme when and if it gets merged

@shadydealer the travis test-matrix has a lot of failures. i saw some where gems were not found. i dont  think that this is expected, could you push another commit or trigger the build some other way?

@phoet Yeah, I saw that too but i don't know how to fix it since the gems are specified in the .gemfile of each version. I just triggered another build, but i doubt it'll work since the last 3 build all fail with the same errors.

thats not great... could you try updating the gemfiles, maybe some dependencies got yanked? alternatively add them to the test-matrix as allowed errors, that would at least make the travis build green 😏

@phoet I tried fixing the problem, but sadly I couldn't. I'll add the files as allowed errors as you suggested.

@phoet I added them as allowed errors like you suggested. Build is green, hopefully someone who can fix those sqlite3 bugs comes around. Thanks for your help.

Hi @shadydealer the build failures should be fixed now. Would you mind rebasing this pull request onto master (minus the allowed failures)?

Hi @mvz I just did and i also made it so incrementing and decrementing the counter cache column works for ActiveRecord>=4.x

@shadydealer looks like something went wrong with your rebase and you merged the remote branch back in. I'm going to fix that up and force push your branch for final review.

@phoet would you like to take another look at this now that the build is green?

👍 go ahead!

@raelgc I just released 0.6.1.


Add a license to this repo

This repo doesn't have license. It would be great to add a license to this repo.

done! https://github.com/shakao/arcade-sprite-pack/pull/26

Thanks so much @shakao


[Solved] eslint can't not find the module "babel-eslint" when I have installed the module babel-eslint?

I found a solution to #82.  You have locked up that issue and no more comments are allowed. So I am adding a new issue with my solution to help anyone else that runs into it.
Issue
babel-eslint cannot be found when loading VSCode.

I noticed that this only happened when I opened a certain project, it seems to work fine for my main project.
In the Output window I saw the line:
[Info  - 11:02:44 AM] ESLint library loaded from: E:\u\temp\node_modules\eslint\lib\api.js

Repair
This was an unused node_modules directory.  So I deleted the unused node_modules directory and the ESLint found my global node_modules directory.
Problem solved.

Or bot locks it :-)
Thanks for the issue and the solution. Closing the issue. Please ping if you think an action is needed.


add ALFAcashier.com to the exchange list on bitcoincash.org website

We've modified all urls to the bitcoincash. Sorry for that. Here is a new pull request.
Thank you.

Thanks for making the change. Really appreciate it. =)

Hi. Is there any ETA on the merge?
Thanks.

Will be doing it tonight. At work atm.

Thanks a lot!


Support --no-browsers command line flag in karma builder

Command
test
Description
I often prefer to run tests using Karma's /debug.html, instead of letting it auto-run. I usually set browsers: [] in karma.conf.js, so that it doesn't auto-lunch a new browser. Instead I navigate to /debug.html myself.
Karma has a command line flag for this: --no-browsers. When I try to use it I get this error:
$ ng test ng-core --no-browsers
Error: Schema validation failed with the following errors:
  Data path "/browsers" must be string.

Describe the solution you'd like
Since Angular 15 no longer scaffolds karma.conf.js, it would be great to be able to use the --no-browsers command line flag, so I don't need to keep my entire config file just to modify browsers.
Describe alternatives you've considered
No response

Thanks for this request.
However this currently do not fall in our roadmap.


Decide on use of Modernizr

In #518 the use of Modernizr for the theme was questioned. If we can remove this, it would make the build a bit simpler and the output a bit smaller.

If the feature detection is not used by the theme, is it still needed to include Modernizr?
If we do need Modernizr, do we need the kitchen sink build currently included?

According to https://github.com/rtfd/sphinx_rtd_theme/pull/248, Modernizr was only included for old IE support. I've personally removed it from my project and haven't noticed any issues.

I missed this but the 0.2% is for all docs sites hosted by Read the Docs. Essentially nobody visiting RTD is using IE<11 now. IE11 is still close to 2%.

I think it is safe to remove this now.

I we are just including it for IE<11, then it is safe to remove. Do we know precisely which detects we use if any?

Consider me +1 then. I'm fine dropping support for IE<11 as virtually no users use those old versions.
I do believe that dropping support should involve incrementing the version to 0.4.0 though. We also should be explicit in the docs about which browsers are supported.

We also should be explicit in the docs about which browsers are supported.

I agree, but such a statement for the theme depends on a statement for the entire RTD service (which I couldn't find, ping @ericholscher @agjohnson). We should not build a theme that only works on the latest browsers if the goal of RTD is to also support legacy browsers.

I don't have time at the moment to research some of the ramifications of dropping modernizer, but to put into perspective: even though 0.2% sounds low, the real number behind this is 500k+ pageviews a year. This number is more significant than the portion of our traffic probably. I'd like to figure out what sort of experience those 500k+ pageviews would have before dropping support for them.
If no one gets to it before I do, I'm just going to test the theme via saucelabs and an ancient browser/os combo. I'm curious what the page and nav content will look like. If it's at least usable, I think we can sacrifice the dependency on modernizer.

I've created a branch in #624 suitable for testing with older browsers.

Eeek Modernizr was removed a long time ago, time to close this.


On Python 3 use exist_ok with makedirs

Django 2.0 only works on Python 3.4+. This keyword is only present on
Python 3 and so this is safe to use since we don't need to worry about
exist_ok missing on Python 2.7

✨🍰✨


Missing error log from CI

The notebook tests in some cases don't report the error log:
http://ci.mxnet.io/blue/organizations/jenkins/GluonNLP-py3-master-gpu-doc/detail/PR-877/9/pipeline
Not sure if it's transient or consistent across runs

Triggered it again and the log is still missing. cc @leezu

Triggering again as the logs from your last run are already deleted (12 hour timeout)

I found some job status reports that indicate that your jobs were timing out. Are you sure #877 does not introduce any regression that prevents the transformer example from running? The Cloudwath log does not contain any output after minute 2 (and until it is killed at minute 30)

I'll open a PR to make sure the stderr output is also sent to Cloudwatch. Currently we can't access stderr output as it is only logged to a file but that file is never uploaded to S3 due to the job timeout.

Yes I also find that the job exceeded 30 mins. There was indeed a bug that causes the translation regression.


Removing colour from text

Hey,
Great library.
I'm trying to remove the green from this image.

Here is my code
replaceColor({
image: dir + '/' + image.src,
colors: {
type: 'hex',
targetColor: '#00ff00',
replaceColor: '#00000000',
deltaE: 20
}
})
I'm still getting a green tinge around the letters.
Do you know how I can completely remove all the green?

I am going to close this issue due to inactivity. If you have any questions, let me know.


How to make cmd send ctrl?

How did you map alt-backspace to send ctrl-w? Would I use that same technique to map cmd combos to ctrl combos?

I am looking for the exact same thing


When reloading settings, one needs to close game to refresh it

Describe the bug
If someone changes a setting and "applies" it, the setting would not be changed until you close the game and enter it
How to reproduce the bug
Just change a setting, and check the debug screen

Figured out how to do it, implementing it...

Fixed


Long Start-Up Time for yt-dlp_macos (nightly)

DO NOT REMOVE OR SKIP THE ISSUE TEMPLATE

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

Checklist

[X] I'm reporting a bug unrelated to a specific site
[X] I've verified that I have updated yt-dlp to nightly or master (update instructions)
[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 all URLs and arguments with special characters are properly quoted or escaped
[X] I've searched known issues and the bugtracker for similar issues including closed ones. DO NOT post duplicates
[X] I've read the guidelines for opening an issue

Provide a description that is worded well enough to be understood
yt-dlp_macos (nightly) takes 45 s to load.
https://github.com/yt-dlp/yt-dlp-nightly-builds/releases/download/2<IP_ADDRESS>707/yt-dlp_macos
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] If using API, add 'verbose': True to YoutubeDL params instead
[X] Copy the WHOLE output (starting with [debug] Command-line config) and insert it below

Complete Verbose Output
❯ time yt-dlp -vU --ignore-config
[debug] Command-line config: ['-vU', '--ignore-config']
[debug] Encodings: locale utf-8, fs utf-8, pref utf-8, out utf-8, error utf-8, screen utf-8
[debug] yt-dlp version<EMAIL_ADDRESS>from yt-dlp/yt-dlp-nightly-builds [705f5b84d] (darwin_exe)
[debug] Python 3.12.4 (CPython x86_64 64bit) - macOS-14.5-x86_64-i386-64bit (OpenSSL 3.0.13 30 Jan 2024)
[debug] exe versions: ffmpeg git-2024-07-04-bef77c6c (fdk,setts), ffprobe git-2024-07-04-bef77c6c, rtmpdump 2.4
[debug] Optional libraries: Cryptodome-3.20.0, brotli-1.1.0, certifi-2024.07.04, curl_cffi-0.7.0, mutagen-1.47.0, requests-2.32.3, sqlite3-3.45.3, urllib3-2.2.2, websockets-12.0
[debug] Proxy map: {}
[debug] Request Handlers: urllib, requests, websockets, curl_cffi
[debug] Loaded 1834 extractors
[debug] Fetching release info: https://api.github.com/repos/yt-dlp/yt-dlp-nightly-builds/releases/latest
Latest version<EMAIL_ADDRESS>from yt-dlp/yt-dlp-nightly-builds
yt-dlp is up to date<EMAIL_ADDRESS>from yt-dlp/yt-dlp-nightly-builds)
yt-dlp -vU --ignore-config  1.18s user 1.38s system 5% cpu 45.165 total

took 45s

Can you try https://github.com/yt-dlp/yt-dlp-nightly-builds/releases/tag/2<IP_ADDRESS>843 ?
yt-dlp --update-to<EMAIL_ADDRESS>

2<IP_ADDRESS>843 loads in 26 s:
❯ time ./yt-dlp_macos -v --ignore-config
[debug] Command-line config: ['-v', '--ignore-config']
[debug] Encodings: locale utf-8, fs utf-8, pref utf-8, out utf-8, error utf-8, screen utf-8
[debug] yt-dlp version<EMAIL_ADDRESS>from yt-dlp/yt-dlp-nightly-builds [d2189d3d3] (darwin_exe)
[debug] Python 3.12.4 (CPython x86_64 64bit) - macOS-14.5-x86_64-i386-64bit (OpenSSL 3.0.13 30 Jan 2024)
[debug] exe versions: ffmpeg git-2024-07-04-bef77c6c (fdk,setts), ffprobe git-2024-07-04-bef77c6c, rtmpdump 2.4
[debug] Optional libraries: Cryptodome-3.20.0, brotli-1.1.0, certifi-2024.07.04, curl_cffi-0.5.10, mutagen-1.47.0, requests-2.32.3, sqlite3-3.45.3, urllib3-2.2.2, websockets-12.0
[debug] Proxy map: {}
[debug] Request Handlers: urllib, requests, websockets, curl_cffi
[debug] Loaded 1834 extractors

Usage: yt-dlp_macos [OPTIONS] URL [URL...]

yt-dlp_macos: error: You must provide at least one URL.
Type yt-dlp --help to see a list of all options.
./yt-dlp_macos -v --ignore-config  1.09s user 1.12s system 8% cpu 25.934 total

Two random past versions
2<IP_ADDRESS>726 loads in 45 s:
❯ time ./yt-dlp_macos -v --ignore-config
[debug] Command-line config: ['-v', '--ignore-config']
[debug] Encodings: locale utf-8, fs utf-8, pref utf-8, out utf-8, error utf-8, screen utf-8
[debug] yt-dlp version<EMAIL_ADDRESS>from yt-dlp/yt-dlp-nightly-builds [12d8ea824] (darwin_exe)
[debug] Python 3.12.1 (CPython x86_64 64bit) - macOS-14.5-x86_64-i386-64bit (OpenSSL 3.0.11 19 Sep 2023)
[debug] exe versions: ffmpeg git-2024-07-04-bef77c6c (fdk,setts), ffprobe git-2024-07-04-bef77c6c, rtmpdump 2.4
[debug] Optional libraries: Cryptodome-3.20.0, brotli-1.1.0, certifi-2024.02.02, curl_cffi-0.5.10, mutagen-1.47.0, requests-2.31.0, sqlite3-3.43.1, urllib3-2.2.1, websockets-12.0
[debug] Proxy map: {}
[debug] Request Handlers: urllib, requests, websockets, curl_cffi
[debug] Loaded 1803 extractors

Usage: yt-dlp_macos [OPTIONS] URL [URL...]

yt-dlp_macos: error: You must provide at least one URL.
Type yt-dlp --help to see a list of all options.
./yt-dlp_macos -v --ignore-config  1.14s user 1.46s system 5% cpu 44.935 total

took 45s

2<IP_ADDRESS>713 loads in 39 s:
❯ time ./yt-dlp_macos -v --ignore-config
[debug] Command-line config: ['-v', '--ignore-config']
[debug] Encodings: locale utf-8, fs utf-8, pref utf-8, out utf-8, error utf-8, screen utf-8
[debug] yt-dlp version<EMAIL_ADDRESS>from yt-dlp/yt-dlp-nightly-builds [a40b0070c] (darwin_exe)
[debug] Python 3.12.1 (CPython x86_64 64bit) - macOS-14.5-x86_64-i386-64bit (OpenSSL 3.0.11 19 Sep 2023)
[debug] exe versions: ffmpeg git-2024-07-04-bef77c6c (fdk,setts), ffprobe git-2024-07-04-bef77c6c, rtmpdump 2.4
[debug] Optional libraries: Cryptodome-3.20.0, brotli-1.1.0, certifi-2023.11.17, mutagen-1.47.0, requests-2.31.0, sqlite3-3.43.1, urllib3-2.1.0, websockets-12.0
[debug] Proxy map: {}
[debug] Request Handlers: urllib, requests, websockets
[debug] Loaded 1819 extractors

Usage: yt-dlp_macos [OPTIONS] URL [URL...]

yt-dlp_macos: error: You must provide at least one URL.
Type yt-dlp --help to see a list of all options.
./yt-dlp_macos -v --ignore-config  0.81s user 1.14s system 5% cpu 38.933 total

took 39s

2<IP_ADDRESS>726 loads in 45 s

The startup time seems to vary a bit randomly then, and doesn't seem to be caused by any changes within the past 2 months. Was there ever a version of the yt-dlp_macos executable that didn't give you these long startup times?

This could be caused by MacOS Notarisation scanning all the files in the folder, does this continue to happen after the first  time(s) you run it?

Yes, did run multiple times… Somehow the 2<IP_ADDRESS>833 version did get faster with 21 s:
❯ /usr/bin/time -l -h -p yt-dlp -v --ignore-config
[debug] Command-line config: ['-v', '--ignore-config']
[debug] Encodings: locale utf-8, fs utf-8, pref utf-8, out utf-8, error utf-8, screen utf-8
[debug] yt-dlp version<EMAIL_ADDRESS>from yt-dlp/yt-dlp-nightly-builds [cc1a3098c] (darwin_exe)
[debug] Python 3.12.4 (CPython x86_64 64bit) - macOS-14.5-x86_64-i386-64bit (OpenSSL 3.0.13 30 Jan 2024)
[debug] exe versions: ffmpeg git-2024-07-04-bef77c6c (fdk,setts), ffprobe git-2024-07-04-bef77c6c, rtmpdump 2.4
[debug] Optional libraries: Cryptodome-3.20.0, brotli-1.1.0, certifi-2024.07.04, curl_cffi-0.7.0, mutagen-1.47.0, requests-2.32.3, sqlite3-3.45.3, urllib3-2.2.2, websockets-12.0
[debug] Proxy map: {}
[debug] Request Handlers: urllib, requests, websockets, curl_cffi
[debug] Loaded 1834 extractors

Usage: yt-dlp [OPTIONS] URL [URL...]

yt-dlp: error: You must provide at least one URL.
Type yt-dlp --help to see a list of all options.
real 21.59
user 1.01
sys 1.13
            63660032  maximum resident set size
                   0  average shared memory size
                   0  average unshared data size
                   0  average unshared stack size
               47923  page reclaims
                2139  page faults
                   0  swaps
                   0  block input operations
                   0  block output operations
                   0  messages sent
                   0  messages received
                   0  signals received
                 832  voluntary context switches
                1106  involuntary context switches
<PHONE_NUMBER>  instructions retired
<PHONE_NUMBER>  cycles elapsed
              851968  peak memory footprint

took 21s

The startup time seems to vary a bit randomly then, and doesn't seem to be caused by any changes within the past 2 months. Was there ever a version of the yt-dlp_macos executable that didn't give you these long startup times?

I don’t know of a version of the yt-dlp_macos executable that does not have these long startup times; first-time user of the executable, used to run the py script – this runs in under 2 s:
❯ /usr/bin/time -l -h -p /usr/local/bin/yt-dlp -v --ignore-config
[debug] Command-line config: ['-v', '--ignore-config']
[debug] Encodings: locale utf-8, fs utf-8, pref utf-8, out utf-8, error utf-8, screen utf-8
[debug] yt-dlp version<EMAIL_ADDRESS>from yt-dlp/yt-dlp-nightly-builds [cc1a3098c] (zip)
[debug] Python 3.12.4 (CPython x86_64 64bit) - macOS-14.5-x86_64-i386-64bit (OpenSSL 3.3.1 4 Jun 2024)
[debug] exe versions: ffmpeg git-2024-07-04-bef77c6c (fdk,setts), ffprobe git-2024-07-04-bef77c6c, rtmpdump 2.4
[debug] Optional libraries: certifi-2024.07.04, curl_cffi-0.7.0, pycrypto-3.20.0, sqlite3-3.46.0
[debug] Proxy map: {}
[debug] Request Handlers: urllib, curl_cffi
[debug] Loaded 1834 extractors

Usage: yt-dlp [OPTIONS] URL [URL...]

yt-dlp: error: You must provide at least one URL.
Type yt-dlp --help to see a list of all options.
real 1.99
user 1.32
sys 0.30
           110317568  maximum resident set size
                   0  average shared memory size
                   0  average unshared data size
                   0  average unshared stack size
               55624  page reclaims
                2248  page faults
                   0  swaps
                   0  block input operations
                   0  block output operations
                   0  messages sent
                   0  messages received
                   0  signals received
                 293  voluntary context switches
                1500  involuntary context switches
<PHONE_NUMBER>  instructions retired
<PHONE_NUMBER>  cycles elapsed
            66691072  peak memory footprint

This could be caused by MacOS Notarisation scanning all the files in the folder, does this continue to happen after the first time(s) you run it?

Yeah, this is my best guess too: some sort of Apple security scanning is happening.
yt-dlp_macos is a pyinstaller-bundled "onefile" executable, which means every time you execute it, it first needs to unpack its contents (~30-50MB of shared libraries and script files) into a temporary directory before yt-dlp itself is executed. These temp files are cleaned up once the executable has finished, so the security scanning would be invoked on every run, not just the first run.
An alternative would be the yt-dlp_macos.zip "onedir" build, where you unpack it yourself into a directory upon installation and no further unpacking is ever needed. The downside with the onedir build is that you can't use -U/--update-to; you would need to manually download updates from Github.
Another alternative is installing with homebrew. While not an official channel for yt-dlp releases, brew is a very well-managed project that is quick to push updates of yt-dlp. (This is not a viable option if you require yt-dlp to have browser impersonation support via curl_cffi, though.)

Yeah, makes sense.
So, yt-dlp_macos is kind of useless?!

So, yt-dlp_macos is kind of useless?!

There is something about your machine/configuration that is different. You were the first person to report such long startup times I am aware of. On my Mac, it takes ~1.5 seconds.

I personally just use pip install yt-dlp —pre -U

@NoNoNo just curious, did you make the yt-dlp binary executable by running chmod +x ?

Hello, this is happening for me as well.
I updated yt-dlp after a long time (due bad SSL certs for YouTube) and whenever I execute the new version it takes about ~10 seconds when it used to be instant-ish before:
time yt-dlp --version
2024.08.01
yt-dlp --version  0.72s user 0.21s system 8% cpu 10.705 total

Thank you.

@Y3K what was the last version that didn't have the long startup time?

Hello @bashonly , unfortunately I'm not sure and I deleted the previous binary, however I'd ballpark it around 6 months old.

A macOS user on discord was kind enough to try and "bisect" this for us, and found that the slow startup time presented for them in every release going back to 2022.04.08 (at which point we gave up).
Every issue that's been opened on the pyinstaller tracker about slow startup time (and there are several) has always reached a conclusion of "antivirus/security scanning interference" (or else "don't use --onefile", which is not a solution here).


The value of "offset" is out of range

Hello!
Based on WC3MapTranslator, I have created a parser that converts all content of a map file to JSON and back. My issue is that when testing with different maps, I receive similar errors. This occurs in various formats (sometimes .w3u parses correctly, sometimes not, and other formats as well).
However, the error always appears. What could be causing this?
(My ultimate goal is to configure the program for my project.)
The current version of Reforged is being used.
Example console log errors:
PS C:\Program Files\OSPanel\domains\WC3MapTranslator> node decode.js
File w3\war3map.doo successfully converted to output\doodads.json!
File w3\war3map.imp successfully converted to output\imports.json!
Error processing file w3\war3map.w3a: The value of "offset" is out of range. It must be >= 0 and <= 36986. Received 36990
Error processing file w3\war3map.w3b: The value of "offset" is out of range. It must be >= 0 and <= 258. Received 260
File w3\war3map.w3c successfully converted to output\cameras.json!
File w3\war3map.w3d successfully converted to output\doodads.json!
File w3\war3map.w3e successfully converted to output\terrain.json!
Error processing file w3\war3map.w3h: The value of "offset" is out of range. It must be >= 0 and <= 976. Received 980
File w3\war3map.w3i successfully converted to output\info.json!
Error processing file w3\war3map.w3q: The value of "offset" is out of range. It must be >= 0 and <= 6173. Received 6175
File w3\war3map.w3r successfully converted to output\regions.json!
Error processing file w3\war3map.w3t: The value of "offset" is out of range. It must be >= 0 and <= 728. Received 729
Error processing file w3\war3map.w3u: The value of "offset" is out of range. It must be >= 0 and <= 16611. Received 16613
File w3\war3map.wts successfully converted to output\strings.json!
File w3\war3mapUnits.doo successfully converted to output\unitsDoo.json!

+1 I got same issue

This issue seems to correspond to the response provided in this other thread:
https://github.com/ChiefOfGxBxL/WC3MapTranslator/issues/67

I was able to work-around the issue by finding a patched version of WC3MapTranslator from another project "patchwork-mapconverter", and copy them into my repo: https://github.com/pqhuy98/Warcraft3/tree/master/gold-rush/scripts/sandbox/wc3maptranslator

I saw your project, but I don't understand what I need to do to accomplish what I want. In the end, what I need is to be able to transform the files "war3map.w3e", "war3map.doo", and "war3mapUnits.doo" into JSON. I would greatly appreciate any help I could receive. Thank you very much!

You can copy the files in the folder I shared into your codebase. Then instead of importing the translator classes this package, you can directly import those classes in the files you just copied.

@ChiefOfGxBxL, hi!
With the release of Reforged 2.0, everything has been thrown into disarray. I haven't checked the current state of things because I've been steering my development in a different direction. I was recommended another library, and using it, I managed to transform map files.
I’ve been working on an engine designed to transfer data between maps and carefully merge data between them. The essence of my work is that a parent project-map transfers all its data to child project-maps, merging the data whenever possible. It also attaches new maps to itself and integrates all non-standard data and imports.
Unfortunately, I couldn't grasp the structure of the parsed data—it’s too complex to understand. I initially thought I’d be able to reassign RAW codes that overlap between two sets of custom maps, but I got lost in the processed data. I assumed the data was neatly organized, but the reality turned out to be much harsher. Developing such a system quickly and on my own isn't feasible.
On the other hand, using Laravel (PHP), I managed to implement data transfer (copying) of selected files from the parent to child projects—although this part was straightforward. I also implemented a cleanup process for WTS files, removing technical tags added during development in the World Editor. Additionally, I developed functionality for overriding core variables in the .j file. At some point, I may also learn how to delete individual map triggers from .j files. Furthermore, I created functionality to import shared data to child maps as well as individual files. Finally, I use PHP to connect to Ladik's MPQ Editor and automatically package everything into a .W3X archive.
The main issue with the system lies in connecting new projects to the current one. When transferring data, matching RAW codes for units and decorations get overwritten by my project’s data... which is a big problem. This remains the system's biggest bottleneck.
To publish my system on Hive, I’d need to rewrite it in C++ or something similar to make it work as an .exe rather than a web server. (I specialize in web development because that’s my main job.)

I was looking into w3t file and it seems like for file version 3 there are some introduced changes, like sets_count that is not currently processed by ObjectsTranslator.js, here is the link with new format changes i found https://github.com/stijnherfst/HiveWE/wiki/war3map(skin).w3*-Modifications


enabling multilingual requests

I have added the languageCode argument in several methods enabling the requests to multilingual agents.

Closing this PR due to the CLA issues.
Filed a new PR #134 which will apply the same changes brought up by @migoogle


gio: Add type parameter to ListModel and ListStore

Fixes #213
This will break any autogenerated functions using ListModel but we could add a special rule in gir to change that to ListModel<Object>.
Upcasting and downcasting works with a few rules:

ListModel<T> is covariant on T
ListStore<T> is invariant on T
methods of ListStore<T> are contravariant on T

I didn't really change subclassing, that still works the same except you have to do type Interfaces = (gio::ListModel<MyType>,);. There is a little bit more work that needs to be done if someone wants to do a generic object with type Interfaces = (gio::ListModel<T>,);, this PR not far off from that though.

Maybe there should still be a ListStore without a type parameter to support ones with a runtime dynamic type? ListStoreDynamic or something? Or I can change this to TypedListStore<T> and leave ListStore?

Wouldn't that be ListStore<glib::Object>?

Not unless we want to relax ListStore<T> to also be covariant on T. That can be done but it would be mean that ListStore::insert et al. could fail at runtime again, whereas right now it can't because of the invariance.

In GTK there are some we know the type of which could be implemented manually with a type parameter:

gdk::prelude::DisplayExt::monitors -> gio::ListModel<gdk::Monitor>
gtk::prelude::FileChooserExt::files -> gio::ListModel<gio::File>
gtk::prelude::FileChooserExt::filters -> gio::ListModel<gtk::FileFilter>
gtk::prelude::FileChooserExt::shortcut_folders -> gio::ListModel<gio::File>
gtk::Assistant::pages -> gio::ListModel<gtk::AssistantPage>
gtk::ColumnView::columns -> gio::ListModel<gtk::ColumnViewColumn>
gtk::ConstraintLayout::observe_constraints -> gio::ListModel<gtk::Constraint>
gtk::ConstraintLayout::observe_guides -> gio::ListModel<gtk::ConstraintGuide>
gtk::Notebook::pages -> gio::ListModel<gtk::NotebookPage>
gtk::Window::toplevels -> gio::ListModel<gtk::Widget>

All the rest of the custom models should be safe to take a ListModel<Object>

Unfortunately model.type_().is_a() or value.is_type() will still not be reliable with this approach, that should probably be documented


move external_url dependees out of constructor

Issue
This PR addresses #388.
Solution
Moving things depending on self.external_url outside of the constructor, so they are not called on removing the application.
Release Notes
I moved the logic depending on self.external_url to a separate function and invoked that in _configure() instead of __init__().
This way, that code is executed in every part of the lifecycle except the removal.

An event guard probably make more sense!

I was slow on the uptake for the review, but please don't directly check JUJU_HOOK_NAME. This isn't a promise Juju gives us, and even OF doesn't try to be that clever. catching ModelError is much less brittle


word_count_weight and valid_word_count_weight

Hi, I cannot still understand the word_count_weight and valid_word_count_weight. What they affect to the scoring? I've found that each default value is 0. and 1. in ctc_beam_scorer_klm.h

word_count_weight is a word-insertion bonus factor. valid_word_count_weight is the same thing, but only applies if the word is within the supplied dictionary. See http://deeplearning.stanford.edu/lexfree/lexfree.pdf


Fix inefficient Docker caching in CI

Seems like the Docker build isn't caching as nicely as it should. You may be able to view logs from this run:
https://github.com/uwsampl/lakeroad/actions/runs/8034507451/job/21946176589
This commit only changed things at/after the "build CVC5" stage, yet many of the steps are re-running. I can't currently see enough of the logs to know how far back it isn't caching, but at least from the Python dependencies section. The Python dependencies causing problems would make sense.

I think this was simply caused by the ADD . . having been pushed nearer to the top of the file.


Remove the GB label from GCE boot_disk_size description in provisioning dialog

Remove the GB label from the Boot Disk Size field description since the possible options include both GB and TB.

https://bugzilla.redhat.com/show_bug.cgi?id=1383309

Change looks good and fixes the reported issue but clearly sorting is still an issue.
@h-kataria Do you recall if there is a way to enable numeric sorting on this field?  The :boot_disk_size field currently has :data_type: string and changing it to integer resulted in the same sort order.

@gmcculloug is that something that could be a different BZ?

Yes, I was thinking the same thing.  We'll deal with sorting separately.

Euwe Backport details:
$ git log -1
commit 71c1fec274a1732aba2a19bde3771102785b1467
Author: Greg McCullough<EMAIL_ADDRESS>Date:   Mon Oct 24 09:25:56 2016 -0400

    Merge pull request #11991 from agrare/bz_1383309_remove_gb_from_boot_disk_size

    Remove the GB label from GCE boot_disk_size description in provisioning dialog
    (cherry picked from commit a9123611239a8b3a1c58881090b80489475dc2d3)

@simaishi Please make Euwe BZ from https://bugzilla.redhat.com/show_bug.cgi?id=1383309

Euwe BZ: https://bugzilla.redhat.com/show_bug.cgi?id=1388667


Mar 2017: Barreleye Driver Test with Loop / Stability Test once in a week

                                    Barreleye CT

Date
Build
CU
CT-PASS
CT-FAIL
New Defects Logged

1st Mar
v1.99.1-65-g07df806
OK
170
8
openbmc/openbmc#1052, openbmc/openbmc#1031, #314, #313

2nd Mar
v1.99.1-82-gbfd622e
OK
170
8

Current Blocking OpenBMC Issues

No.
Github#
Reason/Heading
Current Status

1

If there is any update here today, you can put it in slack or update here as well. I am closing as of now.


dark theme switch button

there should be any options for a dark theme switch button

dark theme detection is available now

done


fix(ngOptions): add ngEmptyValue attribute to ngOptions

Reference: angular#13334
This would allow an additional attribute on ng-options to provide a default value for options that lack values.

Thanks for the PR! Before we can merge we a few things:
[ ] Docs for the new feature
[ ] A test for the new feature
[ ] Squashing all commits
Can you work on improving this? If you need help with setting up a dev environment, check https://github.com/angular/angular.js/blob/master/CONTRIBUTING.md or ask here

Will do! Thanks for the instructions, I'll add docs and a test, then squash all the commits together.
I'm a little confused about commit messages - I thought I'd included the issue. Should I use "closes" rather than "references"?
Also, the original issue was opened as a bug, has it been reassessed as a feature? (No disagreement if it is! I'm just a newbie to the project and I want to make sure I understand)

"Closes" in the commit message closes the issue when your commit is merged into master. Which is what we want. :) I say it's a feature, because it's not a bug, it's just something that's missing. Or did this ever work before in some way?

That all makes sense to me. I don't believe this has worked before, but I just searched for the "PRs Plz" tag and grabbed one I could make sense of.

Cool, it's always good to see new contributors. These formal issues (commit message, multiple commits) can be handled when code & docs are ready, they are not blocking.

Thanks for your help. When correcting a previously made PR, should I add a comment to that effect?
If so, here is that comment.
If not, uh, never mind.

Yes, you should write a commit. Because subscribers don't get notifications for code pushes.
I wonder if the feature is "good enough", because it only works if the option is explicitly selected. I would expect it to work also on load, before any interaction has happened.

Okay, I believe I addressed your comments. Thanks for your time in giving those!
I don't really have anything for the pageload problem. I agree that it'd be better if it worked before selection, but that looks like a much trickier fix (at least for someone like me who's still getting his bearings in the Angular source code).

Good stuff. I have to think about setting the empty option on load. It's probably not necessary to do that.
For the use case described in the original issue, it's not problem, because the model would probably be undefined until it is selected again in the options. The only case I could imagine where this might be useful is when you set your model to a value that is not in your options. In that case, we reset the model to the empty value. So that case is covered.

Sorry, I keep finding things. ;)

Updated for most of your notes, with the exception of the one I replied about in your line notes. Thanks for your time in looking at these!

So... is this PR dead? I was trying to use this feature until I realized it was never actually merged. :disappointed:

TBH, now that select can be also used with ngRepeat/ngValue, I see little value in adding this (which is only useful in specific usecases).

I agree that this is not needed in core. Although I don't see how ngValue + select changes the assumptions here - both regular and ngOptions select will set the model to null if the empty option is selected.
In https://github.com/angular/angular.js/issues/1780, there are 2 workarounds for this problem, and I've added a 3rd one that implements emptyValue, but doesn't rely on private API.


The article on Object and Collection Initializers should mention and explain benefits of init and required.

I believe the article on Object and Collection Initializers should mention the init accessor as well as required modifier.
Specifically, it should be explained how these keywords enable advanced control over initializations, similar to how this section on Object Initializers with collection read-only property initialization does.
In brief:
init:

like a ctor param, but also enabling usage of object initializers;
another take: like a set accessor, but only during construction/init time, thus preventing mutability post-creation.

requried:

like init, but also must be set in object initializer, thus preventing only partially constructed objects (and precludes usage in ctor unless [SetsRequiredMembers] is used).

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

ID: 0efd6a6a-b52a-f6a2-d98b-f245b55115ff
Version Independent ID: 95025282-20a0-ded8-c6d2-3b8ffff600c4
Content: Object and Collection Initializers - C# Programming Guide
Content Source: docs/csharp/programming-guide/classes-and-structs/object-and-collection-initializers.md
Product: dotnet-csharp
Technology: csharp-fundamentals
GitHub Login: @BillWagner
Microsoft Alias: wiwagn

Thanks @konrad-jamrozik
That's a great suggestion.  I've added it to our backlog for when we update this article. I've also added the "help wanted" label in case anyone wants to submit a PR in the interim.


Add retry_rate_limit option to handle 429 response

References issue #88
This should do it, has not been fully tested, so please proceed with caution.

#123 fixes this.


More-inclusive name for command

Please provide details on the following items. Failure to do so may result in deletion of your feature request.
What's the user value of this feature request?
The name of the command cf delete-orphaned-routes may be difficult for those who have lost a parent.  Suggest renaming it to detached/disconnected/stranded/marooned/etc.
Who is the functionality for?
Everyone
How often will this functionality be used by the user?
Routinely (for deleting routes)
Who else is affected by the change?
Will not cause a breaking change; just have to remember the new name.
Is your feature request related to a problem? Please describe.
No IT problem
Describe the solution you'd like
Rename the command cf delete-orphaned-routes to detached/disconnected/unplugged/stranded/marooned/etc.
Describe alternatives you've considered
Leave it as-is until somebody complains (but would be better to be proactive)
Additional context
None
Note:
As of the GA of the v7 CLI, the v6 line is no longer under active development and will be updated only to patch CVEs or fix severe blocking defects with no workarounds.

Thank you @spfeiffe for bringing this to our attention. This is issue is added as feature request in our system.


Fixed #1292 and resulting issues from #1300

@mholt We already talked in length about the contents of this PR and here it is!
I tried to split up this PR in 4 commits to clearly separate the fixes. The commit messages probably say it all. 🙂

And you've confirmed it fixes your use case as described in #1292 and #1300?

Yeah passing HTTP/2 requests and WebSockets generally works flawless now. I tested it using self signed certs and Go 1.8 beta 2.

I'm still having issues with my gRPC backend though, because requests and responses seem to buffered by caddy somehow. Maybe they're even dropped - I've got to properly investigate that first. Due to that my gRPC server seems to only receive the first and last stream chunks. Any idea why that could be the case?
(I don't think it's because of the missing support for HTTP/2 push, because in my case it's a client sending a stream to the server!)

I'm still having issues with my gRPC backend though, because requests and responses seem to buffered by caddy somehow. Maybe they're even dropped - I've got to properly investigate that first. Due to that my gRPC server seems to only receive the first and last stream chunks. Any idea why that could be the case?

Nope :( Although I wonder if there is something up with the buffering somewhere in the proxy. For example, in #1310 there is a report of high memory use when proxying for big files.

@sanderhahn I was able to reproduce your problem. This only happens if you don't specify insecure_skip_verify.
@mholt Due to this I'd consider this PR incomplete. I'll try to fix this but I'm not entirely sure yet where to start, since unlike before this does not cause any debug/error output to be generated by Go.

@lhecker I'm not entirely clear on what the error is, but maybe dumping the HTTP request would be a helpful start? See https://golang.org/pkg/net/http/httputil/.

@mholt I actually found the root cause again only minutes after writing the comment. It's because I only add the DialTLS method to the transport if a transport.TLSClientConfig is provided, but that's not the case for verified TLS connections (i.e. transport.TLSClientConfig is nil).
So… I fixed that, but now to the next issue:
2016/12/28 15:59:35 http: TLS handshake error from [::1]:2816: EOF

sigh…

AHA! I forgot to add the errors stderr directive to that particular domain I'm testing on. I can't claim that I particularly like that behavior (swallowing errors by default) and IMHO you might want to consider changing it to stderr as the safer default.
Anyways: Fix inbound.

@lhecker Nice work :) Sorry about the default of hiding errors. We can probably open a discussion on that in a new issue if you want.

@sanderhahn Do you mind trying out the newest commit (cd8a992)?

@mholt I found the cause for #1310 and I think I fixed it for the most part. Unfortunately I'm still experiencing issues with my gRPC backend so I can't be fully sure yet. Do you mind if I attach the fix for that to this PR in case I finish with it before you merge this?
It's caused by this line. I fixed it by introducing a special case where the request body is not buffered if len(proxy.Upstreams) is 1 or less. If only one upstream exists we don't ever have to rewind the body and thus don't have to cache it right? That should really help improve performance for the common proxy case!

Additional note is that the -http2=false flag doesn't seem to be necessary anymore. The proxy works both with and without http2 enabled.

@lhecker Maybe make a new PR for the gRPC issue? I'll probably merge this soon unless somebody tells me to hold the presses.
Let's make you a collaborator on the project too so you can have push access. This way you can work in branches instead of forks and also help by reviewing and merging pull requests, etc. I really appreciate your efforts, you've done a great job!
@sanderhahn Thanks for testing it!

Oh, missed the second half of your comment @lhecker:

It's caused by this line. I fixed it by introducing a special case where the request body is not buffered if len(proxy.Upstreams) is 1 or less. If only one upstream exists we don't ever have to rewind the body and thus don't have to cache it right? That should really help improve performance (especially latency obviously) for the common proxy case!

I had never considered that. I guess not, then. I don't see where or why else we're using body other than to replay it for the next upstream.
I wonder if buffering has been causing some of other proxy-related issues too. It'd be interesting to see if their issues are fixed with that change too.
Let me know when all the changes for this PR are pushed. :)

Let's make you a collaborator on the project too so you can have push access. This way you can work in branches instead of forks and also help by reviewing and merging pull requests, etc. I really appreciate your efforts, you've done a great job!

Thanks! I probably won't be able to contribute much, because most of the current work resulted from me needing a viable nginx alternative as the frontend for a commercial product with HTTP/2 upstream support while being easy to mod. Caddy turned out to not be quite bug free yet so I simply contributed my fixes. It's quid pro quo right? 😅
Oh and I'll of course promise to handle in good faith with my new access rights to this repo though! And if I ever have a fix for caddy in the future I'll make sure to create a PR. 🙂

Let me know when all the changes for this PR are pushed. :)

Since @sanderhahn confirmed that it's working for him too I'd consider this PR to be finished and you can merge it whenever you see fit. This PR turned out to be mostly a fix for #1292, but since the issues from #1300 are quite closely related I think it's okay we squashed both together.

@mholt I pushed one further change just now. Please review it carefully, because I don't know wether it has any implications:
I looked into the git history of those 4 removed lines in 533039e but never found a explanation as to why they where added originally. Well apart from outreq.Close = false which can be removed without any concerns, because false is the default anyways.
But I found that the removal of those 4 lines has no side effects for my sites and seemed to be essential to make gRPC proxying work properly.

@lhecker We added those lines originally because we borrowed the ServeHTTP for ReverseProxy in https://golang.org/src/net/http/httputil/reverseproxy.go?s=3924:3999#L134 of the Go standard library.
It's quite possible, given all the handling/configuration of transports and directors and such in our case that we don't need to set these there.
But it's good to hear that's what makes gRPC work! And the tests still pass, so that's promising. I'll take a look.
Others are welcome/invited to review carefully too. @nemothekid - hope you're having a happy holiday, I dunno where you're at but maybe you'd have a chance to look this over and see if you notice anything?

@mholt I found one more bug… 😕 You're not handling any trailers in the proxy! Seeing as trailers are gaining quite some interest with HTTP/2 I'll update this PR with a fix soon.

@mholt I pushed the support for HTTP trailers just now. As usual I made sure to add tests (I extended TestReverseProxy).
Again: Without the fix it doesn't work at all, but after applying it it works flawless (trailers in general as well as proxying gRPC metadata, which partially relies on it). I also tested everything on my staging system (basic REST, WebSockets, gRPC).
The code stems from the current reverseproxy.go code just like it did before. The big difference is that the version in caddy was outdated, missing the trailer feature as well as containing a trailer-related bug ("Trailers" != "Trailer").

That's awesome, thanks @lhecker . Can't wait to take a look this weekend. :+1:

Thank you @lhecker!! You are welcome to hack on Caddy at any time. :smile:

Thank you @lhecker!! You are welcome to hack on Caddy at any time. :smile:


Use b4a.equals in key check in firewall

Trying hyperbeam in browser, this the only thing that stops it.

nice catch

3.0.2


Use BaseModel as default - simplify makeServicePlugin call

just trying to migrate to 2.0.  Need some clarification
in the backend a model and service is created like so
// Initializes the `circuits` service on path `/circuits`
const createService = require('feathers-nedb')
const createModel = require('../../helpers/nedb.model')
const hooks = require('./config.hooks')

module.exports = function () {
  const app = this
  const Model = createModel(app, 'config')
  const paginate = app.get('paginate')

  const options = {
    name: 'config',
    Model,
    events: [],  // custom events
    paginate
  }

  // Initialize our service with any options it requires
  app.use('/config', createService(options))

  // Get our initialized service so that we can register hooks and filters
  const service = app.service('config')

  service.hooks(hooks)
}

I am assuming that is how more or less f-v in V1.7 did it as one only had to pass the service name to the plug-in generator.
In 2.0 one now has to make and pass a BaseModel extended class in which   static model name is added
Can the mew makeServicePlugin function take care of all this if it sees a string passed (the service name) and not an object (with model and service)
Seems like that function should/could have access to the feathers object from the feathersVuex call and should be able to create the service as well as the baseModel class instance with the model name if that is what is required.   This IMO should be as simple as before if one doesn't need changes to the BaseModel
const SERVICES = ['config', 'network', 'hardware', 'circuits', 'switches']
plugins: SERVICES.map(name => service(name))

As it is I have to do this hard coding of an extended Class 6 times for my 6 services.  Seems a 'createModel` function should be added equivalent to that in feathers itself.  Then internally to makeServicePlugin it could be called.
Ok...so I probably should do a PR right :-).  Unfortunately I don't code in typescript (mainly cause of the transpiler step) but I do write in latest ES6-7/ESM if that is any help.

Doesnt work but like this, ideas here
https://stackoverflow.com/questions/33605775/es6-dynamic-class-names
function nameModel (name) {
  const nameIt = (name, cls) => ({ [name]: class extends cls {} })[name]

  class Model extends BaseModel {
    constructor (data, options) {
      super(data, options)
      this.name = name
    }
  static modelName = this.name
  }

  return nameIt(name, Model)
}

This is a pretty cool idea, really.  I can see two issues to address.
(1) This shorthand syntax won't support multiple feathers clients.  But MOST apps use a single API, so I feel like we should go for it.  This is basically a documentation issue.
(2) Lack of instanceDefaults support is a bigger issue, since that saves a lot of boilerplate with forms.

well that worked!  Seems worthwile to at least add as a helper function
this version you can specify a custom base if you made one elsewhere otherwise it uses BaseModel.  Could include your suggested static method but will need to be able to grab the api name from somewhere
const nameIt = (name, cls) => ({ [name]: class extends cls {} })[name]

function createModelClass (name, Base) {
  if (!Base) Base = BaseModel
  class Model extends Base {
    constructor (data, options) {
      super(data, options)
      this.modelName = name + 'i'
    }
  static modelName = name
  }
  return nameIt(name, Model)
}

So I'm back up and run with 2.0.  Now looking forward to refactoring things to take advantage of all the changes in 2.0.  @marshallswain so thankful for  your continued work on f-v.   Anything I can help with other than PRs?    Api Docs?  I like this project for those https://github.com/onury/docma.  Maybe some examples of usage including with Quasar?

per 1 it could take an api name and if not supplied can use the first (or only) one it can find.
per 2 my better version supports an already extended BaseModel which could include your static method installDefaults That one could be an included and exported as say ExtBaseModel which has those suggestions built in.

@dkebler I feel like the only thing standing in the way of getting this into 2.x is https://github.com/feathers-plus/feathers-vuex/issues/276.  I would be in favor of having your utility automatically used under the hood if a Model is not passed to makeServicePlugin.  Fixing that other issue will make it so that instanceDefaults are no longer required.

Here it is in action
https://github.com/feathers-plus/feathers-vuex/issues/252#issuecomment-537601295


Update redisson to 3.17.3

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

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

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

Superseded by #178.


fix jets --version outside of jets project

This is a 🐞 bug fix.

[ ] I've added tests (if it's a bug, feature or enhancement)
[ ] I've adjusted the documentation (if it's a feature or enhancement)
[ ] The test suite passes (run bundle exec rspec to verify this)

Summary
Show jets --version correctly when outside of Jets project. Right now it shows the rails components version 🤣
Context
Related #703
How to Test
Run jets --version outside of jets project.
Version Changes
Patch

Released in jets 5.0.9


More documentation on building, running, and debugging

The Start Contribution section describes how to initialize and build the project, and how to run the tests, but not how to run the built jscpd cli.
Please add instructions on how to run the built command line tool, specifically how to launch and debug the tool.
I have been trying to create a working Visual Studio Code debug launch configuration, but have been running into all sorts of issues about module resolution, etc.
Thank you!

In the<EMAIL_ADDRESS>the build process is much easier and not you can run just
pnpm build

in the root of the project and get results.
Also you can run pnpm dev and work with jscpd continuously with watching directory.


Add opt-out for token 22

It looks like it is currently impossible to opt out of token 22 dependency if you're using the token init functionality because of the following line
Ideally you'd be able to complete avoid token 22 dependency and still use the token init functionality

It looks like it is currently impossible to opt out of token 22 dependency if you're using the token init functionality because of the following line

Yeah, it's not currently possible not only because of that line but also the parts where we use token_interface.

https://github.com/coral-xyz/anchor/blob/4955a92593bbb58ee6457416dab1e4d33aedffa0/lang/syn/src/codegen/accounts/constraints.rs#L559
https://github.com/coral-xyz/anchor/blob/4955a92593bbb58ee6457416dab1e4d33aedffa0/lang/syn/src/codegen/accounts/constraints.rs#L718

Curious if there is a specific reason why you'd like to avoid token 2022 dependency given it will be the default soon?

Have a dependency conflict with the solana-program version. spl-token-2022 depends on ^1.14.12 and phoenix depends on =1.14.9

Ah, specifying a fixed version for solana-program is almost guaranteed to cause issues at this point since everyone has a different version. Even if we added a way to opt out of token 22 in Anchor, it would still get included in your dependencies via spl-associated-token-account. I think the fastest solution would be to fork/update Phoenix's solana-program dependency to ~1.14.9.

makes sense, i've done this, thanks!


Support IngressClass ingressclass.kubernetes.io/is-default-class: 'true'

Describe the feature request
IngressClass can have an annotation to be default.
With the following:
apiVersion: networking.k8s.io/v1
kind: IngressClass
metadata:
  name: istio
  annotations:
    ingressclass.kubernetes.io/is-default-class: 'true' # <------ HERE
spec:
  controller: istio.io/ingress-controller

and if I create an ingress without ingressClassName: istio, it is not associated with this class.
Describe alternatives you've considered
MeshConfig has an ingressControllerMode, I've used this as workaround:
istioctl install --set meshConfig.ingressControllerMode=DEFAULT

Affected product area (please put an X in all that apply)
[x] Docs
[x] Installation
[ ] Networking
[ ] Performance and Scalability
[ ] Extensions and Telemetry
[ ] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure
Affected features (please put an X in all that apply)
[ ] Multi Cluster
[ ] Virtual Machine
[ ] Multi Control Plane
Additional context

Also happens in istio 1.10.1 on vanilla k8s 1.21.1.
(IMHO it's a bug)


Doesn't seem to work after opening Runelite; only works after disabling and enabling the plugin

The plugin doesn't seem to work right after you open RuneLite; items just display their normal mouseover text. The issue can be fixed temporarily by disabling and re-enabling the plugin, but this needs to be done again every time you open RuneLite.
RuneLite version <IP_ADDRESS>

I've seen this happen intermittently. I've seen the plugin work upon first startup of RuneLite and other times it doesn't work. I think it's because the itemManager isn't always loaded properly when the Plugin tries to load? Idk, it's weird. It's due to a NullPointerException that gets thrown only sometimes, when an item id is passed to the itemManager to find an item. The thing is, the item IDs never change. And the way I'm building it never changes. So yeah, race condition is my hunch.

Hopefully when https://github.com/runelite/plugin-hub/pull/5299 is merged, this goes away. I'll keep an eye open too for this. I haven't been able to reproduce this with the changes in that commit.


[Feature] Multiple reconnect attempts

Currently your library supports automatically reconnecting to the MQTT broker. However there will be only ever be one reconnection attempt.
If the reconnection attempt fails, the library will stop reconnecting and not try again. This may be the desired behavior in some cases, but in many real-world cases you would expect to retry reconnecting until the MQTT becomes available again.
How about a configuration parameter maxReconnects which can either take a number (for example "3") for maximum of 3 reconnection attempts. If you set it to "-1", then it will try infinite times to reconnect.
maxReconnects: 1 // current behavior (default)
maxReconnects: 3 // try up to 3 times
maxReconnects: -1 // try infinite times (until MQTT is back up)

Hey there, I see this can be a useful feature. I'll return to this after checking the #94 discussion, where we might implement a custom reconnect callback, where this feature mentioned above can also reside.
I'll be unavailable at least in the next few days, but I'll assign myself to this issue once I am.
Thanks again for the issue.


Add test command

Same concept as build, but running the test command in each package.
Depends on packages already being built. So it would need to build them if they had changes.

this was done in #35


Error Deploying to Heroku (webpack-bundle-tracker)

When trying to deploy to heroku the webpack-bundler-tracker wasn't installed
I have to run
heroku config:set NPM_CONFIG_PRODUCTION=false
in order to make it works...
Hope it helps.

how could u install nom and node.js on heroku?
I can't do this.
so I meet the problem
Error reading ./webpack-bundle.json. Are you sure webpack has generated the file and the path is correct?

Same problem here.

Same here, subscribing to this issue.

Has anyone manage to solve webpack-bundle.json error?

I think this issue is resolved by now. "Deploy to heroku" button works, however, if you are deploying manually (eg. from CLI) remember to set proper Buildpacks.
Saleor needs heroku-buildpack-nodejs and heroku-buildpack-python to work properly.

I used
npm install webpack-bundle-tracker --save-dev
to create the package and then push it to heroku =)


Backward compatibility and API stability

Is your feature request related to a problem? Please describe.
I'm always frustrated when updating to a new version of authlib because of needless refactoring and especially removal of methods on models. Moving code around and renaming causes serious breakage between versions. We have many services using this lib and it's a total nightmare to update their dependencies (which we do regularly for security purposes). Because we have no idea what's changed with each release, and how our services will break we have to examine every single line and then perform full regression tests on everything using this lib. It's a terrible development experience, using a library should help development.
Describe the solution you'd like
My proposed solution is two fold

A focus on backward compatibility when developing this library.
Create a detailed change log/migration guide (for API and DB schema) for every release to help users of this library to migrate to newer versions.

Describe alternatives you've considered
Writing it ourselves or forking.
Additional context
None.

@nomad-software are you upgrading to 1.0.0 alpha versions?
Each breaking version change I will create a gist guide like:
https://gist.github.com/lepture/726e07a737b20f111d570d1d0437d5d4
https://gist.github.com/lepture/ecf8ccdbb2ef4b635799d5669308cb50

We've done a few upgrades to our services and each has had problems. At least one of our services is now running 1.0.0 alpha as I did that the other day and noticed its version number.
Those linked gists are very useful and would probably be better done as part of https://docs.authlib.org/

@nomad-software these links are on page https://docs.authlib.org/en/latest/changelog.html
e.g.
Breaking Changes: find how to solve the deprecate issues via https://git.io/JkY4f

For each release, I will keep it backward compatible for at lease >=2 versions, and there will be a deprecate message. I think your problem is upgrading from an early version to a very new version. But Authlib has no new releases for a long time now.

@nomad-software did you check the changelog?

Yeah, I had a look. It's very helpful, please continue this into the future. Thanks!


Archive is broken :(

The move the cloudflare and the non-selfcontained html broke the archive. YIKES!

partially resolved. the self contained html should work properly in the archive. We'll resolve the non-selfcontained archive in another update.

best way to do this is probably putting each commit in an individual folder then hyperlink to there


nvm not installing version of node like its supposed to on setup

needs to be tested now that we're using zsh-nvm

fixed


Caching causes subsequent images to fail to load.

https://github.com/RafaelBarbosatec/gif_view/blob/962d3ce8a360da49916888ee1178a9fc7ed8abe2/lib/gif_view.dart#L207C0-L215C11
The internal frame cache uses a map that makes keys out of known types, but does not cover all types.
For example, when you try to load gifs from files. (eg, using FileImage), it always uses an empty String as the key and thinks everything that isn't a NetworkImage, AssetImage, or MemoryImage is the same image.
final gifImage = FileImage(File(file.path));

return Center(child: GifView(image: gifImage));

This will cause the GifView widget to retain the first image it loads because every FileImage is identified in the map as the same key of empty String.

Fixed in 0.4.1 version.


Update changelog because 4.1.1 was never published

Changes proposed in this Pull Request:

Remove the 4.1.1 entry and roll it up to 5.0.0

wasn't 4.1.1 released?

wasn't 4.1.1 released?

That's react-components, this is react native components.
Also, don't approve this yet. I'm actually publishing 4.1.1 now because 4.1.0 dependencies are broken.

Closing in favor of #131


Create pk-loot-splitter plugin v1.0.0

Your nav button icon is too large. While nav buttons scale it down, it Java keeps it loaded in memory at the full size. Please scale it down so that's not an issue.
Speaking of nav buttons, instead of removing it on shutdown, you're adding it a second time.

Your nav button icon is too large. While nav buttons scale it down, it Java keeps it loaded in memory at the full size. Please scale it down so that's not an issue. Speaking of nav buttons, instead of removing it on shutdown, you're adding it a second time.

Fixed both of these issues I believe


brecq errors

When I use adaround algorithm for quantization, then it reports an error. This is why.
Traceback (most recent call last):
  File "/mnt/miniforge3/envs/torch113_cuda116/lib/python3.10/runpy.py", line 196, in _run_module_as_main
    return _run_code(code, main_globals, None,
  File "/mnt/miniforge3/envs/torch113_cuda116/lib/python3.10/runpy.py", line 86, in _run_code
    exec(code, run_globals)
  File "/mnt/Dipoorlet/dipoorlet/__main__.py", line 151, in <module>
    weight_calibration(onnx_graph, act_clip_val, weight_clip_val, args)
  File "/mnt/Dipoorlet/dipoorlet/weight_transform/weight_trans_base.py", line 57, in weight_calibration
    graph_after_wt = adaround(onnx_graph, graph_after_wt, act_clip_val, weight_clip_val, args)
  File "/mnt/Dipoorlet/dipoorlet/weight_transform/adaround.py", line 115, in adaround
    round_mask = learning_round_mask(
  File "/mnt/Dipoorlet/dipoorlet/weight_transform/adaround.py", line 152, in learning_round_mask
    loss = L2_norm(output, fp_output) + reg(ada_layer.module.round_mask, cur_iter)
  File "/mnt/Dipoorlet/dipoorlet/weight_transform/ada_quant_layer.py", line 117, in L2_norm
    return (pred - tgt).pow(2.0).sum(1).mean()
RuntimeError: The size of tensor a (360) must match the size of tensor b (368) at non-singleton dimension 2

Try using onnx.shape_inference(model) to see if it works?


Adding dark mode support

This fixes #1245 where the select2 fields don't honor the Django dark mode theme.
The fix requires using the CSS variables defined by the Django admin. These variables get used for both dark mode and light mode. For older Django versions < 3.2 before these variables were added, a fallback value is provided that matches the default styling that select2 was using.
Below you can see the comparison between the different fields and their states before and after these changes are applied.
Dark Mode Comparison

List comparison

Multiple choice field
Empty/Inactive
Before

After

Empty and dropdown open
Before

After

1 selected
Before

After

1 selected and dropdown open
Before

After

1 selected and searching
Before

After

Single choice field
Empty/Inactive
Before

After

Empty and dropdown open
Before

After

Choice selected
Before

After

Choice selected and dropdown open
Before

After

1 selected and searching
Before

After

Table comparison

Field Type
Field State
Before
After

Multiple Choice
Empty/Inactive

Multiple Choice
Empty and dropdown open

Multiple Choice
1 selected

Multiple Choice
1 selected and dropdown open

Multiple Choice
1 selected and searching

Single Choice
Empty/Inactive

Single Choice
Empty and dropdown open

Single Choice
Choice selected

Single Choice
Choice selected and dropdown open

Single Choice
1 selected and searching

Light Mode Comparison

List comparison

Multiple choice
Empty/Inactive
Before

After

Empty and dropdown open
Before

After

1 selected
Before

After

1 selected and dropdown open
Before

After

1 selected and searching
Before

After

Single choice
Empty/Inactive
Before

After

Empty and dropdown open
Before

After

Choice selected
Before

After

Choice selected and dropdown open
Before

After

Choice selected and searching
Before

After

Table view

Field Type
Field State
Before
After

Multiple Choice
Empty/Inactive

Multiple Choice
Empty and dropdown open

Multiple Choice
1 selected

Multiple Choice
1 selected and dropdown open

Multiple Choice
1 selected and searching

Single Choice
Empty/Inactive

Single Choice
Empty and dropdown open

Single Choice
Choice selected

Single Choice
Choice selected and dropdown open

Single Choice
1 selected and searching

Great contributions :tophat:

Which version of DAL should we use to have this PR included ? Currently the latest (3.9.4) has still this issue

Hello @trumpet2012 Can you please let me know which version of dal has this support?

@ram495 @daweedm It should be in version 3.9.7 and later


Check for build tags case insensitively.

#1194
When adding a new tag with the same text but different casing as an existing tag we shouldn't throw an exception.
The existing tag won't get overwritten but that's not a problem.

@stephenmichaelf,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot

looks good.


wrong size calculation for slice of structs with marshalers

The tagcode is size is counted twice
here
https://github.com/golang/protobuf/blob/master/proto/encode.go#L1006
and here
https://github.com/golang/protobuf/blob/master/proto/encode.go#L1017

I have a fix pending.  Might be a couple days before I can pull it into GitHub.

Ok but its a one line fix.
Just delete
https://github.com/golang/protobuf/blob/master/proto/encode.go#L1017

Yeah, and then you need to add tests, and fix other issues the new tests uncover, and synchronize a couple different repos...
A proper fix is more than one line.  It's on the way.

fair enough :-)
On Sep 27, 2016 6:25 PM, "Bryan C. Mills"<EMAIL_ADDRESS>wrote:

Yeah, and then you need to add tests, and fix other issues the new tests
uncover, and synchronize a couple different repos...
A proper fix is more than one line. It's on the way.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/golang/protobuf/issues/236#issuecomment-249918466,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABvsLX3FK8vQyDaRoW1vsgn0woMkjIyFks5quUOLgaJpZM4KF759
.

+1


Move item summary item retrieval to method

Preparatory work to have certain items be just sheet data.

Sure, I'll do that tomorrow when I wake up.

I've chosen to backtrack on adding item summary listeners to the global click listeners. I think we need to avoid collisions as much as possible. This change is all I really need.


ruby 2.0.0 is EOL

Ruby 2.0.0 maintenance ended Feb. 24th, 2016.

@chef/client-core RIP ruby 2.0.0

we'll want to make similar commits to ohai and the mixlibs and such...

Did we ever make an official RFC or something about our policy on EOL'd ruby versions?

https://github.com/chef/chef-rfc/blob/master/rfc034-ruby-193-eol.md
and
https://github.com/chef/chef-rfc/blob/master/rfc015-chef-12.md
its not written quite as clear as i'd like right now, but the waffling seems to surround the fact that we were creating wiggle room to support 2.0.0 on windows for as long as we were stuck with it.
the way the RFC's are written it looks like we could drop support on 2.1 as well since 2.2 is what we seem to be calling 'N-1'.  i don't think we need to do that until 2.1 EOL's though (and its been less than an hour since i merged the ruby-2.2 branch...)

:+1:

:+1:

I'm so glad we don't support Ruby 1.9.5 anymore.
:+1:

We should dump the Travis 2.0 tests too if we haven't already

Those are already gone

(which is actually a good reason to get this done -- because we have no coverage and don't want to ship broken 2.0 code that 2.0 clients can still slurp down)


Refactor converter and add CSS support

Complete refactor of the code responsible for SVG to Ruby conversion, with these highlights:

These classes are no longer needed: Parser, SVGSource, XMLNode and XMLText
Instead of all four, there is one: SVGNode
The SVGNode class handles all nodes, and renders them using ERB templates.
CSS (style tags) is now handled (more-) properly using the new class CSSData - closes #23
Inline styles (style attributes) are now handled properly and coverted to hash - closes #25

@gingermusketeer - hey, I believe this could be a good refactor, as it simplifies the conversion handling and adds support for CSS.
Note that I did not yet delete the old files - leaving them here for reference - and I did not update the specs yet - wanted your opinion first.

@gingermusketeer I guess you don't have time. I will merge now.


Remote syslog setup has been removed in 3.1.x < , misleading screenshot of UI

File: ee/ucp/admin/configure/store-logs-in-an-external-system.md

The screenshot of the remote syslog setup in the UI is misleading for UCP 3.1.x users as this feature has ben removed.


Lock screen is taking too long to appear when unlocking the phone

When the iPhone is unlocked, the wallet shows its balance before the lock screen shows up. Sometimes the balance even updates before the lock screen shows up.
It is happening on iPhone 7 Plus with Hathor Wallet v0.8.3.

@pedroferreira1 Maybe we have to put the lock screen when the app goes to background. I think we can add a handler that is called by the phone when either the app goes to background or the phone is locked. In this case, we can put either the lock screen or a "blank screen", just to prevent the balance from showing up.
What do you think?

Makes sense. I still need to check the code but we are probably redirecting to the Locked screen when the app is active again. We should redirect when the app goes to background.

Not necessarily when it goes to background. We actually only lock it if it's been in background for a period (I think 30s). That was our decision so the user wouldn't need to input the pin all the time, when he quickly switches apps.


Feature request: Add support for plain text events

For small events like "go for a run" It seems a bit excessive to create a whole note just for that purpose.
Would be nice to have the option of adding calendar entries that are simply text.
Entries that don't have to point to a note in the vault.

You might look at how the itinerary plugin defines events.  it allows multiple events to exist in a note.  actually if you start by using that format and enhance it, you'll be able to convert users of that plugin.  itinerary doesn't support the recurring events, but they do support coloring (background and text) in an event.  that plugin also uses the full-calendar library to show the calendar and supports the different formats.

I don't think supporting events with no backing in the Vault will be supported. Like @pcause mentioned, multiple inline events within a single note might cover your concerns, and since it's described in #28, I'm closing this issue in favor of that one.


Flush SRP's base64 code

Use the generic encoder/decoder.  But see the comments in https://github.com/openssl/openssl/pull/2672

Looking at the generic code, I got reminded of EVP_EncodeBlock and EVP_DecodeBlock, which are exactly what should be used.  This will be an easy fix.

merged, closing this.


[Feature Request] 3rd party reading state grouping

Why/User Benefit/User Problem
The mangadex API is in read only state. The reading state can't be changed. (Unfollowed, Reading, Compleated, On Hold, Plan To Read...) This resutls in a lot of ungrouped Manga. Neko currently only supports the mangadex Libary Grouping.
This would also be useful when MangaDex is down again. Then all the manga I read wouldn't be ungrouped after I click on the cover.
What/Requirements
I would like to have a option in the Libary Settings to choose the provider of the reading state so that when you group you libary by tracking status it would use the status of a 3rd party like MAL or Anilist.

The 3rd party status's are part of the grouping iirc.  Just skip over the unfollowed section

They are part of the filter, but not part of the grouping.

I mean this grouping. This only works with mangadex. At least I can't find a option to change it. I use Anilist. I didn't tested it with MAL

hmm yea maybe something changed but i thought it used to group by all trackers

So this coming from J2K it takes the first logged in service and uses that as the default, because MangaDex was always the first logged in service thats why it always grouped them like that.


Update azure-analytics-defender-easm module, reflect latest spec

There is change in spec repo that changed the structure of the endpoint.
https://github.com/Azure/azure-rest-api-specs/commit/690baaddc71ab873c83be33a74b4c0b9a044f594#diff-40d8b2cf9bb6658c4dbf874ca40a783caaff9fa377b605ccc5b8a9fe76f38005
Because of this, SDK cannot re-generate with latest spec. This changes breaks samples/tests.
Please update the azure-analytics-defender-easm module in Java to reflect latest spec.

update tsp-location.yaml
run tsp-client update
one line modify on module-info.java, e.g. https://github.com/Azure/azure-sdk-for-java/pull/39861/commits/091efe35725e767aa8a60516b492a2ee77468467
fix tests/samples
record

A draft PR is prepared for you. Feel free to either use it or make a new PR.
https://github.com/Azure/azure-sdk-for-java/pull/39861

I see there is another PR pending in spec. It would be fine if you do this on the next preview api-version.

@thang-bit
I've regenerated the lib for you.
Please see if you can schedule a release.


tests: switch to test copyParentCPUSet under tmpDir

test copyParentCPUSet() under tmpDir instead of using real
cgroups syatem path, make the test always worked whether
cpuset.cpus and cpuset.mems files in cpu cgroups exist or not.
Signed-off-by: Ace-Tang<EMAIL_ADDRESS>please help me check if the patch is properly, thanks.
ping @sameo

@Ace-Tang The patch looks good to me, but you need to create an issue for your PR and then add a Fixes line to your commit message.

oh, sorry, I do not konw issue is necessary， I will update soon

Thanks @Ace-Tang!
@sboeuf, @grahamwhaley - can you restart the 16.04 build for this PR? I cannot as I see a bunch of errors like this:
Error while checking in file to scm repository

... and they are covering up the "restart build" button! If I try to remove the message, I get an error that only an admin can purge log messages.

@jodh-intel Oh, I see what you mean by 'covering up' the button - there was a not-easy-to-see 'clear this error' link, so I pressed that, which then cleared it down and allowed me to hit the rebuild - should be in progress now

@grahamwhaley - thanks - that link is what threw the Jenkins perm error at me (I don't have your super-powers! ;)

@jodh-intel I have added you now, you should be able to rebuild from Jenkins

LGTM

Coveralls says coverage is increasing with that one: https://coveralls.io/builds/13561840


align referral created

🙏
On Wed, 23 Nov 2022, 11:38 am Feruz M, @.***> wrote:

@.**** approved this pull request.
lgtm
—
Reply to this email directly, view it on GitHub
https://github.com/ecency/ecency-vision/pull/1120#pullrequestreview-1191404429,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AVAB2UWS64IWUCQZKRRRUW3WJXX3JANCNFSM6AAAAAASIC44GQ
.
You are receiving this because you authored the thread.Message ID:
@.***>


feat: instanceID support for argo server. Closes #2004

Closes #2004
Make Argo server behaves the same as workflow controller:

Argo server only manages the workflows/CronWorkflows with the same instanceID (label xxxx/controller-instanceid), or wfs/cronWfs without instanceID, depends on if instanceID is configured in the ConfigMap.

argo cli without ARGO_SERVER ENV goes through kube api, argo list or argo cron list can see all the results. argo submit or argo cron create needs to give --instanceid=xxxx to create wf/cronWf targeting a specific controller.

argo cli with ARGO_SERVER ENV uses argo server, results of argo list and argo cron list might be different, depends on instanceID configuration in the ConfigMap.  As a result, argo submit or argo cron create by default will include the instanceID in the labels if it's configured in the ConfigMap.

Checklist:

[x] Either (a) I've created an enhancement proposal and discussed it with the community, (b) this is a bug fix, or (c) this is a chore.
[x] The title of the PR is (a) conventional, (b) states what changed, and (c) suffixes the related issues number. E.g. "fix(controller): Updates such and such. Fixes #1234".
[x] I have written unit and/or e2e tests for my change. PRs without these are unlike to be merged.
[ ] Optional. I've added My organization is added to the README.
[ ] I've signed the CLA and required builds are green.

@alexec - could you review it again?


ExtensionTarget

Update-AzConnectedMachineExtension : A parameter cannot be found that matches parameter name 'ExtensionTarget'.
Is this a not yet released module version? I currently have version Script     0.4.1      Az.ConnectedMachine
I would be interested in using this and testing with the -ExtensionTarget Capabilities on the Azure Powershell variant.

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

ID: 5ea00229-4162-25a0-2990-c287ec73f913
Version Independent ID: 92c7064f-7247-ce6e-469e-2de8f018aefb
Content: Enable VM extension using Azure PowerShell - Azure Arc
Content Source: articles/azure-arc/servers/manage-vm-extensions-powershell.md
Service: azure-arc
Sub-service: azure-arc-servers
GitHub Login: @johnmarco
Microsoft Alias: johnmarc

@ariendg Thanks for your comment! We will review and provide an update as appropriate.

@rpsqrd would be the Subject Matter Expert on this?

@ariendg I think you're calling the wrong cmdlet. To upgrade extensions, use Update-AzConnectedExtension (no "machine"), to update (reconfigure) an extension, use Update-AzConnectedMachineExtension. I apologize for how confusing the cmdlet names are. The Az CLI equivalents are a bit clearer: az connectedmachine upgrade-extension and az connectedmachine extension update.

Thanks for clarifying it @rpsqrd.
@ariendg, Closing this issue for now. If there are further questions regarding this topic or feedback on the referred Azure documentation, please do let us know and we will gladly continue the discussion.

@rpsqrd Yes, initially I got tricked by the cmdlet Names and the short descriptions in help. Thank you for pointing out.
What I tried to accomplish is this https://github.com/ariendg/PowerShellPublicADG/blob/main/AzureArc Update-AzConnectedExtensions for 1 Server.ps1
But it is still not doing what I expected like in the console. What am I missing here?

Do you get an error? Does it register in Activity Log? The code looks correct to me.

@rpsqrd
Yes I do see Operations called: Upgrade Extensions on Azure Arc machines
But no expected results like Change history on the properties.
FYI This is the output of my Script:
Updating Extension MDE.Windows to <IP_ADDRESS>
VERBOSE: Performing the operation "Update-AzConnectedExtension_UpgradeExpanded" on target "Call remote 'UpgradeExtensions' operation".
Updating Extension DependencyAgentWindows to <IP_ADDRESS>50
VERBOSE: Performing the operation "Update-AzConnectedExtension_UpgradeExpanded" on target "Call remote 'UpgradeExtensions' operation".
Updating Extension AzureMonitorWindowsAgent to <IP_ADDRESS>
VERBOSE: Performing the operation "Update-AzConnectedExtension_UpgradeExpanded" on target "Call remote 'UpgradeExtensions' operation".
Updating Extension AdminCenter to <IP_ADDRESS>3
VERBOSE: Performing the operation "Update-AzConnectedExtension_UpgradeExpanded" on target "Call remote 'UpgradeExtensions' operation".


rm_cropped_images from wagtail

My first pull request please double check it!
This is a command will remove all rendered images. Other packages have similar commands like sorl thumbnailing.
I hope this command is complete enough to cover situations with and without custom image model.

Thank you for looking into this @onno-timmerman :) Note for future reference – this looks like it could replace #759.

@onno-timmerman I was wondering if you have time to revisit this PR? It feels like a good addition that would help lots of Wagtail users.
The changes needed:

[ ] Make use of Image.get_rendition_model() (see https://github.com/wagtail/wagtail/blob/571b9e1918861bcbe64a9005a4dc65f3a1fe7a15/wagtail/images/signal_handlers.py#L26-L27)
[ ] drop the shutil.rmtree(image_root) bit, as the actual file deletion is done via signals.
[ ] add a test to confirm the deletion
[ ] revisit the spelling

Not sure where this PR is at but I have created https://github.com/wagtail/wagtail/issues/8107 as a issue to cover the general request for this kind of management command.

superseded by #8166


ICMP v6 packets and IP v6 packets containing Cisco EIGRP packets are not currently supported

The Processing class in the PacketCaptureAnalyzer.EthernetFrame.IPPacket.IPv6Packet namespace contains the following code:
// Processing of IP v6 packets containing an ICMP v6 packet is not currently supported!

// Just record the event and fall through as later processing will read off the remaining payload so we can move on
this.theDebugInformation.WriteInformationEvent(
    "The IP v6 packet contains an ICMP v6 packet, which is not currently supported!");

...

// Processing of IP v6 packets containing a Cisco EIGRP packet is not currently supported!

// Just record the event and fall through as later processing will read off the remaining payload so we can move on
this.theDebugInformation.WriteInformationEvent(
    "The IP v6 packet contains a Cisco EIGRP packet which is not currently supported!");

These information events capture the fact that ICMP v6 packets and IP v6 packets containing Cisco EIGRP packets are not currently supported by the Packet Capture Analyser
Suggestions for the necessary implementation are welcomed

Added processing for ICMPv6 packets on 20/02/2015
Added processing for IPv6 packets with hop by hop or destination option extension headers on 20/01/2015
Updated this issue to reflect those additions

Support added on 20/02/2015 so can close this issue


Configure browser tests to run against Saucelabs

See https://github.com/hughsk/smokestack for more info.

The tests run against Saucelabs now. The README widget isn't ok yet but that can wait.


Lecture05

Coverage decreased (-0.3%) to 44.505% when pulling ed73d644a315d08eba61f72f4b7509fb0af81845 on SergDob:lecture05 into 315d33f92428d9f59d2580d0102eb36ce0e3991e on rybalkinsd:lecture05.

Dobychin Sergey

+


tailwind-elements - adding custom css

I am using this carousel:
https://tailwind-elements.com/docs/standard/components/carousel/
I don't know how to add custom CSS, instead default for bellow div:

 ...

I should copy carousel-indicators to my CSS, and change it?
How practice is the best?

We've just release the new v1.0.0-beta1 version where we use almost only tailwind class system and all elements can be customize. Please update and check it out!
https://tailwind-elements.com/docs/standard/components/carousel/

Closing due to inactivity.


Intermittent validation error on test_pole_NR.glm

Problem description
When validating develop there is an intermittent fail of autotest powerflow/test_pole_NR.glm
Steps to reproduce
(Explain how to reproduce the behavior, e.g.,)
gridlabd -W powerflow --validate
Expected behavior
100% pass rate
System information

GridLAB-D version: HiPAS GridLAB-D 4.2.1-200107 (develop_fix_websites) Darwin
Operating system: Darwin dhcp-swlan-public-84-172.slac.stanford.edu 19.2.0 Darwin Kernel Version 19.2.0: Sat Nov  9 03:47:04 PST 2019; root:xnu-6153.61.1~20/RELEASE_X86_64 x86_64

Supporting information
NA
Attachments
NA

@aivanova5, when you find a simple reduced model that reliably causes the error, please attach it to this issue.

@ftuffner, FYI this is the issue we are using to track resolution of this bug.

The problem is likely in module/powerflow/pole.cpp@pole::presync() somewhere after line 270.

Initial workplace:

Change link_fault_on/off to line->status = OPEN/CLOSED.
Make sure NR solver is flagged to redo admittance (e.g., NR_admittance_change=TRUE)
Check that fault_check handles all links, not just switches.


tuya YSR-ZSW 3-Button Wireless Remote Switch

ADD: tuya YSR-ZSW 3-Button Wireless Remote Switch
LINK: https://www.aliexpress.com/item/1005004775066234.html (Battery Switch-3CH)

deployed on both dev an main channels


chore: remove the concept of target and host platforms

Its very unlikely that we will ever support cross platform builds. For
that reason, let's simplify the whole Makefile by removing the concept
of target and host platforms.
Signed-off-by: Juan Cruz Viotti<EMAIL_ADDRESS>

Oh, is cross-platform builds not a feature that electron-builder supports?

@lurch Up to a certain point. See https://github.com/electron-userland/electron-builder/wiki/Multi-Platform-Build

Don't expect that you can build app for all platforms on one platform

If your app has native dependencies, it can be compiled only on the target platform. prebuild is a solution, but most node modules don't provide prebuilt binaries.
macOS Code Signing works only on macOS. Cannot be fixed.

Both points affect us, plus I don't think its a battle worth fighting.

Fair enough :+1:
Rename target-convert.sh and I'll approve ;)

Cool, I just pushed that change :)

macOS Code Signing works only on macOS. Cannot be fixed.

This sounds like something that can be fixed by some mild reverse engineering :)

I think getting the MacOS CLI code-signed via pkg would be more important than fixing electron-builder to be able to code-sign MacOS binaries when running on Linux ;)

Yeah, but this sure sounds like a fun spare-time project to me :)

As long as Apple doesn't come after you under the DMCA ;-)

Reverse engineering is legal, unless stuff has been copied from whatever it is your tearing apart, afaik. Also... I'm in Germany, not the US ;P

:de: :tada:


Create CronJob for syncing spotify:main to dfds:main

[x] - Create CronJob for syncing spotify:main to dfds:main

CronJob in question - https://github.com/dfds/emcla-sandbox/tree/master/poc/backstage/upstream-sync

Thu 20 Aug 2020 11:00 AM
A CronJob has been created with accompanying secret and configmap in the 'backstage' namespace on Hellman.
It's configured to:

Pull the dfds/backstage repository, also configured the same repository as a remote named 'origin'
Add spotify/backstage repository as remote with the name 'upstream'
Fetch from 'upstream' remote
Checkout 'main' branch, and creates it if it doesn't exist
Merge with '--ff-only' flag from upstream/master to origin/main
Push changes to origin/main

The above will only merge if fast-forward is possible. If not, it'll merely exit with an error. Likewise the push will only happen if there's no conflict with the origin/master.

Thu 20 Aug 2020 11:00 AM
A CronJob has been created with accompanying secret and configmap in the 'backstage' namespace on Hellman.
It's configured to:

Pull the dfds/backstage repository, also configured the same repository as a remote named 'origin'
Add spotify/backstage repository as remote with the name 'upstream'
Fetch from 'upstream' remote
Checkout 'main' branch, and creates it if it doesn't exist
Merge with '--ff-only' flag from upstream/master to origin/main
Push changes to origin/main

The above will only merge if fast-forward is possible. If not, it'll merely exit with an error. Likewise the push will only happen if there's no conflict with the origin/master.


Bundle size benchmarks

It would be great to have a comparison of the bundle size of importing the functions from graphql and graphql-web-lite in order to have a better understanding of the significance of this package. I guess generating this could be automated via a Github action.

On every PR you can see a difference between an urql build with lite and graphql https://github.com/kitten/graphql-web-lite/pull/3#issuecomment-909499438

Oh, that is awesome! Thank you!