Replaced From<f64> with TryFrom<f64>

To enable usage in environments where panic is untenable.
Note: This is a breaking change.
Closes #2

@metasim Thanks. I quickly reviewed the PR. It generally looks good; however, there isn't a test for the case where the conversion fails. I like to be careful to check the edge cases, especially around the "boundaries" between valid and invalid.

@xpe I thought this should cover it (via the #[should_panic])? If not, I can try to think up some more edge cases.
https://github.com/xpe/q-num/pull/3/files#diff-e6012b80fc130c96ba4662a6a8dc26bb7c6d06caa6f173af5e38f43a0296381dR117

Or maybe you'd prefer an assert!(Q::try_from(16.00).is_err());?

Ah, I missed the should_panic case you included. Sorry about that.
Yes, I agree assert!(Q::try_from(16.00).is_err()); is cleaner.

@metasim Thanks!

@xpe You're welcome. Sorry I didn't get to fixing that test before you merged, but if there's a next time I'll update that. Appreciate you accepting this.


Tutorial Translation

Tutorial translation is going to be a major undertaking. I think it will need to be done in smaller chunks. Creating a separate issue to track its progress.
Sections:

[ ] Before We Start the Tutorial ( @3sanket3 )
[ ] Setup for the Tutorial
[ ] Overview
[ ] Completing the Game
[ ] Adding Time Travel

do you still need a contributor

Uh, is this still being done? I think I may be able to help. Gujarati is my mother tongue...

@ridays2001 Please take any page that you want to work on here #1 👍

Oh, Okie. Will check that. Thanks 👍🏻


Feature request: save watched variables

Hey! thanks for the extension, improves a bit the lackluster breakpoint support in vscode.
I was thinking of potential features...

Storing watched variables seems like it could be a great addition on top of the breakpoints!

Interesting!
I'm thinking of improving and updating this extension very soon :)
Also, if you can, please consider supporting me
Thank you


Azure immutable storage seems to work for time-based version-level scope policies and SAS authentication

This documentation states that BACKUP TO URL does not support Azure immutable storage. After some tests, I was able to successfully backup and restore a 2 GB database in this specific setup:

SAS Token based authentication (block blobs)
Time-based immutable policy
Version-level scope enabled

I did not perform exhaustive tests, but can you please confirm that immutable storage is actually supported in such specific scenarios?

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

ID: cc04e819-4cfd-5b23-404b-8d4ed41afd21
Version Independent ID: d254a232-e05d-0d52-0e25-7e83866f12e6
Content: SQL Server Backup to URL - SQL Server
Content Source: docs/relational-databases/backup-restore/sql-server-backup-to-url.md
Product: sql
Technology: backup-restore
GitHub Login: @LitKnd
Microsoft Alias: kendralittle

@helderpinto -- Hélder, thank you for your feedback.  Please consider these resources:

SQL Server on Microsoft Q&A
DBA Stack Exchange
Stack Overflow

@LitKnd -- Kendra, please look into this issue.

Hello @helderpinto ,
Apologies for the delay in responding to this, and thanks for raising this with us. I'm working on identifying the answer as to whether this is supported.
Cheers,
Kendra Little
Database Docs Team

Hello,
Thanks again for your patience. I am reassigning this issue as we continue to investigate.
Cheers,
Kendra
#reassign: @rothja

@WilliamDAssafMSFT William, could you please take a look at this question that potentially needs a doc update when you get some time? Thank you! #reassign:williamdassafmsft

Thanks @helderpinto for contacting us and taking the time to document and test this. It is entirely possible that this is possible but also not supported. Reaching out to the product group, will update the docs as any information becomes available. Thanks again for this feedback, your suggestions help improve our documentation.


Difficult to acquire sample scenes

Each Tutorial that mentions a sample scene to load as a basis for that tutorial should ideally lead to a page that describes where to find and how to download that sample. This eliminates dependencies on level of expertise and level of  understanding of current organization of entire MRTK offerings.

We appreciate your feedback and thank you for reporting this issue.
Microsoft Mixed Reality Toolkit version 2 (MRTK2) is currently in limited support. This means that Microsoft is only fixing high priority security issues. Unfortunately, this issue does not meet the necessary priority and will be closed. If you strongly feel that this issue deserves more attention, please open a new issue and explain why it is important.
Microsoft recommends that all new HoloLens 2 Unity applications use MRTK3 instead of MRTK2.
Please note that MRTK3 was released in August 2023. It features an all new architecture for developing rich mixed reality experiences and has a minimum requirement of Unity 2021.3 LTS. For more information about MRTK3, please visithttps://www.mixedrealitytoolkit.org.
Thank you for your continued support of the Mixed Reality Toolkit!


GD-156: Add an exception hook to report exceptions as test failures that are caught by Godot

Discussed in https://github.com/MikeSchulze/gdUnit4Net/discussions/148

Originally posted by bjkarasek November  1, 2024
In documentation (https://mikeschulze.github.io/gdUnit4/testing/assert-error/) it seems that assert-error is not supported in C#. Does that mean there is no way to make tests fail when you print GD.PrintErr or GD.PushErr? Is there a workaround where I can "log" something that will make the tests fail?
Background
By default, Godot intercepts all exceptions triggered by the runtime within the scene tree processing in order to avoid unexpected program crashes. All caught exceptions are just logged as error in the Godot log.
GodotSharp.dll
 catch (Exception ex)
      {
        ExceptionUtils.LogException(ex);
        *outRet = new godot_variant();
        return godot_bool.False;
      }

      ...

    public static void LogException(Exception e)
    {
      try
      {
        if (NativeFuncs.godotsharp_internal_script_debugger_is_active().ToBool())
          ExceptionUtils.SendToScriptDebugger(e);
        else
          GD.PushError(e.ToString());
      }
      catch (Exception ex)
      {
        ExceptionUtils.OnExceptionLoggerException(ex, e);
      }
    }

Install an exception hook
We can hook into the Godot exception to add a custom exception handler that will cause a test to fail.
The test should fail with the stacktrace to the original exception

Reopen, because of correct caught exceptions are reported as failure.
  try
        {
            if (frameCount == 5)
                throw new InvalidProgramException("Exception during scene processing inside a catch block");
        }
        catch (InvalidProgramException)
        {
            // ignore
        }


Specific shortcut settings are not being saved after refreshing page

Actions before raising this issue

[X] I searched the existing issues and did not find anything similar.
[X] I read/searched the docs

Steps to Reproduce

Go to settings and select shortcuts tab.
Change the shortcut for Switch label to xxx under the Objects sidebar to a new key combination.
Refresh the page.
The shortcut is resettled to default.

Expected Behavior
After saving this change, the shortcut works and remains active even after refreshing the page. I have tested some shortcuts under General and Annotation page, and they function properly without resetting to the default settings.
Possible Solution
Should the same logic for shortcuts under General be applied here as well?
Context
No response
Environment
Self-hosted CVAT (v2.18.0) with customized permission controls, particularly for data exportation.

Reproducable, thanks

Can be fixed by adding this line
sequences: keyMap[key]?.sequences ?? updatedComponentShortcuts[key].sequences,
to label-list.tsx in the below useEffect
    useEffect(() => {
        const updatedComponentShortcuts = JSON.parse(JSON.stringify(componentShortcuts));
        for (const [index, labelID] of Object.entries(keyToLabelMapping)) {
            if (labelID) {
                const labelName = labels.find((label: any) => label.id === labelID)?.name;
                const key = makeKey(+index);
                updatedComponentShortcuts[key] = {
                    ...updatedComponentShortcuts[key],
                    nonActive: false,
                    name: `Switch label to ${labelName}`,
                    description: `Changes the label to ${labelName} for the activated
                        object or for the next drawn object if no objects are activated`,
                    sequences: keyMap[key]?.sequences ?? updatedComponentShortcuts[key].sequences,
                };
            }
        }

        registerComponentShortcuts(updatedComponentShortcuts);
    }, [labels]);


Pathway contaminant_plant is not a standard pathway

@LienReyserhove : very likely do you mean contaminant_on_plants?
This occurs in the following taxa:
https://www.gbif.org/species/181971925
https://www.gbif.org/species/181972034
https://www.gbif.org/species/181972036

See #10


Create many to many relation with unlimited records

when I create a schema I can create a relation with another collection but I must enter max select. What if I need the max select is infinity ?

Currently you can't, you have to type the max allowed js integer<PHONE_NUMBER>740991) for example as a workaround of the validator. This is already "fixed" in the refactoring related to #376.

Since recently there was a discussion about this, I'll add a note here in case someone ended here in the future - support for storing "unlimited" multiple relation field values was reverted as part of the latest refactoring and will be removed in PocketBase v0.23.0. The multiple relation field support is usually intended primarily for small/finite number of items like labels, tags, categories, groups, roles, etc. Storing thousands of relations as part of a single db row not only makes it almost impossible to manage from the Admin UI but also is very inefficient because db indexes cannot be utilized properly. For those cases where a lot of multiple relations need to be assigned to a single record, a more traditional SQL approach would be better (either as single one-one relation or in a junction collection with 2 relation fields). One big limitations with earlier versions was the lack of easy filtering and expanding non-directly related entries but this is already solved with the recent back-relation support. So to summarize, with PocketBase v0.23.0 all multiple valued fields (file, select, relation) will have consistent behavior and the MaxSelect option will be required to be > 1 for storing multiple values.


PKG: Cannot 'update' bucket in OSS with 'newer' all-access token

I created a new all-access token with the UI.  I used the new token with a simple yml template:
testBucket.copy.yml.txt
Since the bucket did not exist in OSS, when I first ran the pkg/template it DID create the bucket.
The second time I run pkg it errors with:
>bin/darwin/influx pkg --org-id 03ed7f40b511a000 -f ../templateExport/testBucket.yml -t "***=="
Error: Read:buckets/fea260040a1fd613 is unauthorized.

Curious, I tried an old all-access token from ? two months ago.  Running pkg with that token successfully runs the 'update'

STDOUT in terminal:
2020-03-02T23:28:50.122443Z	error	failed to dry run pkg	{"log_id": "0LJMteI0000", "service": "pkger", "orgID": "03ed7f40b511a000", "userID": "03ed7f40a1d1a000", "error": "read:buckets/fea260040a1fd613 is unauthorized", "took": "0.439ms"}
2020-03-02T23:28:50.122476Z	error	api error encountered	{"log_id": "0LJMteI0000", "handler": "pkger", "error": "read:buckets/fea260040a1fd613 is unauthorized"}

Can you paste the relevant output from the equivalent of
curl https://SOMEPLACE.SOMEWHERE.cloud2.influxdata.com/api/v2/authorizations \
   -H 'cookie: session=your-session-token-from-your-cookie=='

There will be sensitive information (tokens) in the output, so redact as needed. Example:
{
  "authorizations": [
    {
      "id": "053c445dc52b3000",
      "token": "***",
      "status": "active",
      "description": "sharedReadTokenLocalUser1",
      "permissions": [
        {
          "action": "read",
          "resource": {
            "type": "buckets",
            "id": "03cf84c6d664e000",
            "name": "Bucket",
            "org": "organization"
          }
        }
      ]
    },
    {
      "id": "053d7ca7e4cca000",
      "token": "***",
      "status": "active",
      "description": "you can read from my bucket",
      "permissions": [
        {
          "action": "read",
          "resource": {
            "type": "buckets",
            "id": "03cf84c6d664e000",
            "name": "Bucket",
            "org": "organization"
          }
        }
      ]
    }
  ]
}

This is a local temp OSS instance, so no redaction required:
pkgTokenInvestigation.txt

Guessing which two tokens you are using, old and new, it looks like there is a difference in permissions (apart from the newer created one including checks/notifications grants).  The newer permissions are org specific to org 03ed7f40b511a000 whereas the old permissions on oss are for everything, this was likely fallout from the original pr to clean this up that also caused the all-access token issues the other day. There is also a missing write orgs permission in the new token vs one created in the past. None of this should have any affect on the original issue though, as the token you are using has permission to read buckets on the org you are using. This definitely seems like an issue deeper in the authentication layer.
{
  "action": "read",
  "resource": {
    "type": "buckets",
    "orgID": "03ed7f40b511a000",
    "org": "nisley"
  }
}

New Token - l3fejkXWSe9WHf2AOKo3Z4TlYnpEkxaPc-IVbOFKEP5wjx6KRd6jTUrXLtqg6JB5sVLk3j9ZdF18PyGGOGoCiw==
Old Token that worked - zx-xrxmnD92Z6gUE-GSlIVqTV0KXxXJFO4_qPapWG-cxkVPaJJwg0A2nicJsNIrotwKPlDDnTbey1Piqof2fZQ==


Request

Hello everyone,
Can someone please edit the config.js file and make like a quick tutorial with some examples of what you should write on the different modules?
Example: cards_sold: '', // An example of what sould go here
Thanks !
Best Regards,
w0zol.

referenced, and patched in pull 18

@SloRunner Thank you so much !

no problem ;) @shanetechwiz can you merge this?


Disable simd-accel for windows

The simd-accel feature currently fails to build on Windows.  Until somebody wants to fix this properly, let's follow the path of android and ios and ensure simd-accel does nothing on windows.
eg:
  process didn't exit successfully: `C:\msys64\home\mvines\solana\target\debug\build\reed-solomon-erasure-61e557d7bc126d5c\build-script-build` (exit code: 1)
--- stdout
TARGET = Some("x86_64-pc-windows-msvc")
HOST = Some("x86_64-pc-windows-msvc")
CC_x86_64-pc-windows-msvc = None
CC_x86_64_pc_windows_msvc = None
HOST_CC = None
CC = None
CFLAGS_x86_64-pc-windows-msvc = None
CFLAGS_x86_64_pc_windows_msvc = None
HOST_CFLAGS = None
CFLAGS = None
CRATE_CC_NO_DEFAULTS = None
CARGO_CFG_TARGET_FEATURE = Some("fxsr,sse,sse2")
DEBUG = Some("true")
running: "C:\\Program Files (x86)\\Microsoft Visual Studio\\2019\\Community\\VC\\Tools\\MSVC\\14.21.27702\\bin\\HostX64\\x64\\cl.exe" "-nologo" "-MD" "-O2" "-Z7" "-Brepro" "-W4" "-march=native" "-std=c11" "-FoC:\\msys64\\home\\mvines\\solana\\target\\debug\\build\\reed-solomon-erasure-2412293776b1fcee\\out\\simd_c\\reedsolomon.o" "-c" "simd_c/reedsolomon.c"
cargo:warning=cl : Command line warning D9002 : ignoring unknown option '-march=native'
cargo:warning=cl : Command line warning D9002 : ignoring unknown option '-std=c11'
reedsolomon.c
C:\Users\mvines\.cargo\registry\src\github.com-1ecc6299db9ec823\reed-solomon-erasure-4.0.0\simd_c\reedsolomon.h(24): fatal error C1083: Cannot open include file: 'unistd.h': No such file or directory
exit code: 2

Thanks for the report!
This looks like a problem with mvsc (or specifically lack of unistd.h). I don't know if I can detect which C compiler is being used, so I'm not sure if a proper fix can be done.

So removing include of unistd.h seems to build fine - I guess my modifications negated the need of the header file.
I'll try this in another PR and see if any test fails

Sweet.  I tried simply removing unistd.h, but then found that cl.exe tripped up on the gnu attribute at https://github.com/darrenldl/reed-solomon-erasure/blob/fa406eec86fc13c840abb3461e7189254e2cc488/simd_c/reedsolomon.h#L40

Coverage increased (+0.0006%) to 96.29% when pulling a2173c756b4a8a806b0ec86c5192b09498a80d6b on mvines:win into fa406eec86fc13c840abb3461e7189254e2cc488 on darrenldl:master.

Ah, so close.
From the looks of it, it seems like I can also remove that line, as the C api is only ever called from Rust, and there's no null pointer during the calls made in this lib afaik.
I'll delete the line from the no-unistd-h branch and see how it goes.

@mvines Could you check if branch no-unistd-h works fine on your system?
I just realised Travis CI builds don't use simd-accel feature, so I'll need to push a commit later to amend that.
Cheers

@darrenldl - thanks so much for the quick turn around, yep the no-unistd-h branch builds for me on Windows.
I get a number of warning: use of deprecated item 'libc::uint8_t': Use u8 instead. warnings but 🤷♂
Would it be possible to make a 4.0.1 release with this fix quickly so we can pick it up from crates.io?  Thanks again!

Closing in favour of #64

@mvines

thanks so much for the quick turn around

That was an embarrassing amount of delay in past 6 months, due to various things, including starting my PhD, so it's good that you did not experience that, though your colleague might have, ha.

warning: use of deprecated item 'libc::uint8_t': Use u8 instead

Oh yeah, good reminder, I saw those in my system as well, I'll fix that as well.
Tracked at issue #65

Would it be possible to make a 4.0.1 release with this fix quickly so we can pick it up from crates.io?

Give me a couple hours, I have to attend some stuff today.

so it's good that you did not experience that, though your colleague might have, ha.

Sagar sits right beside me :)

Give me a couple hours, I have to attend some stuff today.

Awesome, thanks so much!


fix(css): clean id for postcss process

Description

Fix #5097
If a CSS module is loaded via client and server, the id may differ in ?direct due to import/fetch limitations.
A custom generateScopedName may be used that derives from the file name, but because of ?direct, it generates a different hash even if the content is the same.
This PR cleans the id before passing to postcss.
Additional context

I'm not sure if there are postcss plugins out there that rely on the query params to do something special, but that sounds hacky in the first place 🤔 And if so, this PR would break those usecase.

What is the purpose of this pull request?

[x] Bug fix
[ ] New Feature
[ ] Documentation update
[ ] Other

Before submitting the PR, please make sure you do the following

[x] Read the Contributing Guidelines.
[x] Read the Pull Request Guidelines and follow the Commit Convention.
[x] Check that there isn't already a PR that solves the problem the same way to avoid creating a duplicate.
[x] Provide a description in this PR that addresses what the PR is solving, or reference the issue that it solves (e.g. fixes #123).
[x] Ideally, include relevant tests that fail without this PR but pass with it.

Whadaya know we indeed have test to prevent the breaking change 😄

Wait I was the one who approve https://github.com/vitejs/vite/pull/7827 too 🙈 Was thinking of just removing the ?direct part. cc @sapphi-red

Yeah, I think we could remove ?direct as long as we preserve the other parts.


Add: added support for spanish docs and translated a couple of docs

Thanks for reviewing @ernesto-customink! Welcome to Team Starlight @dreyfus92 🥳


JDBC driver

Support for a generic JDBC driver would be incredibly useful.
I'm not sure how feasible this would be?

It's doable, but as far as I know, there is no self-contained package distributed in npm.
I looked into this at the time I wrote a connector based on beeline  (this beeline-based connector isn't currently distributed with Falcon).
At that time, I found node-jdbc, but I decided not to use it, because one has to install a JDBC driver for node-jdbc to work, and I didn't really want to have that burden.


PiKVM #unsupported #selfsupported

Not officially endorsed by the PiKVM team

yes,We are also enthusiasts. In addition, we provide the latest test video：https://www.youtube.com/watch?v=aehOawHklGE

I am aware but this project needs to be more transparent. Its especially unfair to those that do not know and purchase this hardware that this is not approved hardware for pikvm and that it will not be supported.

Hello, what do you mean by being more transparent? I hope to provide more support for pikvm hardware.

You keep promoting PiKVM but PiKVM will NOT support this project, when PiKVM matures it will mature with its own hardware and not yours and will eventually break. This needs to be told to the customers, which you are not doing. This is what I mean by being transparent.
Spamming a channel or its members, ether directly by you or indirectly also doesn't help your cause.

@arch1mede - Are you saying that PiKVM will not be supporting the DIY version in the future?

@arch1mede - Are you saying that PiKVM will not be supporting the DIY version in the future?

PiKVM does NOT support THIS project in any official capacity and will not tailor any changes to make sure that PiKVM will work. This is all #selfsupported. I think its been made clear, so will no longer respond to this thread.

@arch1mede - Are you saying that PiKVM will not be supporting the DIY version in the future?

PiKVM does NOT support THIS project in any official capacity and will not tailor any changes to make sure that PiKVM will work. This is all #selfsupported. I think its been made clear, so will no longer respond to this thread.

I think we all know that given nowhere does it say that it is supported by PiKVM.

The project name PiKVM is used everywhere to promote this un-official project, causing potential customers to think this is part of the official PiKVM project.
Remove every mention of "PiKVM" and use "Pi-based KVM" instead. Create your own software image for your project to avoid any confusion!
The PiKVM project team WILL NOT assist your users. They should be made aware of this fact too! Support is exclusively yours to provide.

This project arguably takes the work done by PiKVM project and improves on it substantially by improving the physical form factor.
I've had a chance  to look and play with both solutions, this project ends up with a cleaner/simpler user setup experience. And in Australia $96 cheaper. Wasn't that the goal of the project in the first place?
In my experience, The seller at AliExpress provided all necessary support to get the product working.
Counter to this was any mention or question of this product is instantly removed from the Official PiKVM discussion channel, of course users are going to be confused when you try to Streisand effect a competing product away but that is your fault, not @ThomasVon2021
Maybe you need to acknowledge and embrace @ThomasVon2021 superior product design rather than be jealous of them. On official channels, Acknowledge there will clones of your product, try and explain why they are hurting your project rather than try an obfuscate information about them.


Removing ConvBinWinogradRxSf3x2 from pdb

Removing ConvBinWinogradRxSf3x2 from mi200 104

Reasoning:

MIOpen(HIP): Warning [FindSolutionImpl] Invalid config loaded from Perf Db: ConvBinWinogradRxSf3x2: 109.
...it is seen on system with MI210 (but our other systems uses MI250x)

MI210 has 104 CUs. Winograd's tuning config value should not exceed the number of available CUs.T he reason of the warning is that tuning configs were obtained on MI250 and then copied to MI210's db. As a fix, I recommend removing Winograd's tuning configs from MI210's db. The HeuristicInit of Winograd is very capable so I do not expect any performance drops.


Put operation for passive in serverstore impl

This PR introduces a put operations on ServerStoreImpl only to be used by EhcachePassiveEntity, i.e. why it is not part of the ServerStore api as it is not supposed to be used by clients.
Also the implementation is kind of a provisional thing for now, so that work for AP can move forward. The implementation for put needs to be rewritten in much proper way

See #1448


img2pdf init at 0.3.1

Motivation for this change
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[ ] NixOS
[x] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[x] Fits CONTRIBUTING.md.

Success on x86_64-linux (full log)
Attempted: img2pdf, pdftk
Partial log (click to expand)
/build/img2pdf-0.3.1
post-installation fixup
shrinking RPATHs of ELF executables and libraries in /nix/store/c4hsi0811jkln1xmw3gr4dy1p3wflp2q-img2pdf-0.3.1
strip is /nix/store/gpc2wld1s0c6qzx9326cwn1wcx29xzsj-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/c4hsi0811jkln1xmw3gr4dy1p3wflp2q-img2pdf-0.3.1/lib  /nix/store/c4hsi0811jkln1xmw3gr4dy1p3wflp2q-img2pdf-0.3.1/bin
patching script interpreter paths in /nix/store/c4hsi0811jkln1xmw3gr4dy1p3wflp2q-img2pdf-0.3.1
checking for references to /build in /nix/store/c4hsi0811jkln1xmw3gr4dy1p3wflp2q-img2pdf-0.3.1...
wrapping `/nix/store/c4hsi0811jkln1xmw3gr4dy1p3wflp2q-img2pdf-0.3.1/bin/img2pdf'...
/nix/store/c4hsi0811jkln1xmw3gr4dy1p3wflp2q-img2pdf-0.3.1
/nix/store/y2h57abqcf4i738hplp74d7793vsxrxx-pdftk-2.02

Unexpected error: command failed with exit code 1 on aarch64-linux (full log)
Attempted: img2pdf, pdftk
Partial log (click to expand)
Successfully installed img2pdf-0.3.1
/build/img2pdf-0.3.1
post-installation fixup
shrinking RPATHs of ELF executables and libraries in /nix/store/wf8l2w2m3rp5k1hd8y4rs0yqz8j77799-img2pdf-0.3.1
strip is /nix/store/ah0va6j4cnwj9nx4j6rwcfc8nh785jwm-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/wf8l2w2m3rp5k1hd8y4rs0yqz8j77799-img2pdf-0.3.1/lib  /nix/store/wf8l2w2m3rp5k1hd8y4rs0yqz8j77799-img2pdf-0.3.1/bin
patching script interpreter paths in /nix/store/wf8l2w2m3rp5k1hd8y4rs0yqz8j77799-img2pdf-0.3.1
checking for references to /build in /nix/store/wf8l2w2m3rp5k1hd8y4rs0yqz8j77799-img2pdf-0.3.1...
wrapping `/nix/store/wf8l2w2m3rp5k1hd8y4rs0yqz8j77799-img2pdf-0.3.1/bin/img2pdf'...
error: build of '/nix/store/a22cq7xh60i5y99j62krx22xd2w0llgg-pdftk-2.02.drv' failed

Failure on x86_64-darwin (full log)
Attempted: img2pdf, pdftk
Partial log (click to expand)
make[3]: Leaving directory '/private/tmp/nix-build-gcj-6.4.0.drv-0/build/build-x86_64-apple-darwin/fixincludes'
make[2]: *** [Makefile:2777: all-build-fixincludes] Error 2
make[2]: Leaving directory '/private/tmp/nix-build-gcj-6.4.0.drv-0/build'
make[1]: *** [Makefile:18768: stage1-bubble] Error 2
make[1]: Leaving directory '/private/tmp/nix-build-gcj-6.4.0.drv-0/build'
make: *** [Makefile:19073: bootstrap] Error 2
builder for '/nix/store/l88cyq8rkf5qmarfqxvdmzycf1jis7sq-gcj-6.4.0.drv' failed with exit code 2
cannot build derivation '/nix/store/8dyc6j7qpxj1l0cq13ccfdcv6z50p9bk-gcj-wrapper-6.4.0.drv': 1 dependencies couldn't be built
cannot build derivation '/nix/store/irkyfknwdl9zc8xq40npxj7vv20hbcjh-pdftk-2.02.drv': 2 dependencies couldn't be built
error: build of '/nix/store/irkyfknwdl9zc8xq40npxj7vv20hbcjh-pdftk-2.02.drv' failed

Success on aarch64-linux (full log)
Attempted: img2pdf
Partial log (click to expand)
/nix/store/wf8l2w2m3rp5k1hd8y4rs0yqz8j77799-img2pdf-0.3.1

Success on x86_64-linux (full log)
Attempted: img2pdf
Partial log (click to expand)
Successfully installed img2pdf-0.3.1
/build/img2pdf-0.3.1
post-installation fixup
shrinking RPATHs of ELF executables and libraries in /nix/store/c4hsi0811jkln1xmw3gr4dy1p3wflp2q-img2pdf-0.3.1
strip is /nix/store/gpc2wld1s0c6qzx9326cwn1wcx29xzsj-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/c4hsi0811jkln1xmw3gr4dy1p3wflp2q-img2pdf-0.3.1/lib  /nix/store/c4hsi0811jkln1xmw3gr4dy1p3wflp2q-img2pdf-0.3.1/bin
patching script interpreter paths in /nix/store/c4hsi0811jkln1xmw3gr4dy1p3wflp2q-img2pdf-0.3.1
checking for references to /build in /nix/store/c4hsi0811jkln1xmw3gr4dy1p3wflp2q-img2pdf-0.3.1...
wrapping `/nix/store/c4hsi0811jkln1xmw3gr4dy1p3wflp2q-img2pdf-0.3.1/bin/img2pdf'...
/nix/store/c4hsi0811jkln1xmw3gr4dy1p3wflp2q-img2pdf-0.3.1

Success on x86_64-darwin (full log)
Attempted: img2pdf
Partial log (click to expand)
/nix/store/hxdjp4ry8phmr0qngp34jdxgfw6y0n2c-img2pdf-0.3.1

Success on aarch64-linux (full log)
Attempted: img2pdf
Partial log (click to expand)
/nix/store/wf8l2w2m3rp5k1hd8y4rs0yqz8j77799-img2pdf-0.3.1

Success on x86_64-linux (full log)
Attempted: img2pdf
Partial log (click to expand)
/nix/store/c4hsi0811jkln1xmw3gr4dy1p3wflp2q-img2pdf-0.3.1

Success on x86_64-darwin (full log)
Attempted: img2pdf
Partial log (click to expand)
  The script img2pdf is installed in '/nix/store/hxdjp4ry8phmr0qngp34jdxgfw6y0n2c-img2pdf-0.3.1/bin' which is not on PATH.
  Consider adding this directory to PATH or, if you prefer to suppress this warning, use --no-warn-script-location.
Successfully installed img2pdf-0.3.1
/private/tmp/nix-build-img2pdf-0.3.1.drv-0/img2pdf-0.3.1
post-installation fixup
strip is /nix/store/251fvx4mx9q4zarbca5cciasmn66p668-cctools-binutils-darwin/bin/strip
stripping (with command strip and flags -S) in /nix/store/hxdjp4ry8phmr0qngp34jdxgfw6y0n2c-img2pdf-0.3.1/lib  /nix/store/hxdjp4ry8phmr0qngp34jdxgfw6y0n2c-img2pdf-0.3.1/bin
patching script interpreter paths in /nix/store/hxdjp4ry8phmr0qngp34jdxgfw6y0n2c-img2pdf-0.3.1
wrapping `/nix/store/hxdjp4ry8phmr0qngp34jdxgfw6y0n2c-img2pdf-0.3.1/bin/img2pdf'...
/nix/store/hxdjp4ry8phmr0qngp34jdxgfw6y0n2c-img2pdf-0.3.1


[API] How do you get the final permission value?

hey,
I'm trying to get the final permission value out of default, claim, inherit and override. I tried using claim.getPermissionValue() but it did always return undefined so it ignored the others.
What's the best way of doing this?

You need to provide more information as what you stated tells me nothing.

I want to get the final result of the block-break flag at a specific location so I can check if x player is allowed to break blocks at that location because of a flag or being trusted on the claim.

As I already said above, paste the code you are using.

claim.getPermissionValue(player, ClaimFlag.BLOCK_BREAK, "any:any", ClaimContexts.BASIC_DEFAULT_CONTEXT)
claim.getPermissionValue(player, ClaimFlag.BLOCK_BREAK, "any:any", ClaimContexts.BASIC_OVERRIDE_CONTEXT)

Does return undefined too.

@bloodmc does the getPermissionValue only get the permission flags and not the default flags configured in the configs?
I did the following and in both cases I get a empty map for a basic claim.
System.out.println(claim.getPermissions(ClaimContexts.BASIC_OVERRIDE_CONTEXT));
// returns
{}
System.out.println(claim.getPermissions(ClaimContexts.BASIC_DEFAULT_CONTEXT));
// returns
{}


Port reuse

5060 has already been used as a service listener. How to use the service link to send request data through port 5060 instead of response data? I reported that the port is occupied using a new client
`
tx.Respond(sip.NewResponseFromRequest(req, sip.StatusOK, "OK", nil))
		catalog := sip.NewRequest(sip.MESSAGE, sip.Uri{Host: "localhost", Port: 5060})

		params := sip.NewParams()
		params["branch"] = sip.GenerateBranch()
		catalog.AppendHeader(&sip.ViaHeader{
			Transport:       "UDP",
			ProtocolName:    "SIP",
			ProtocolVersion: "2.0",
			Host:            "<IP_ADDRESS>",
			Port:            5060,
			Params:          params,
		})

		uri := sip.Uri{
			User: "37070000082008000003@3707000008",
			Host: "<IP_ADDRESS>",
			Port: 5060,
		}
		catalog.AppendHeader(&sip.FromHeader{
			DisplayName: "SipServer",
			Address:     uri,
			Params:      sip.NewParams().Add("tag", sip.GenerateTagN(20)),
		})

		catalog.AppendHeader(&sip.ToHeader{
			DisplayName: req.From().DisplayName,
			Address:     req.From().Address,
			Params:      sip.NewParams(),
		})

		deviceid := req.From().Address.User
		catalog.SetBody(GetDeviceInfoXML(deviceid))
		catalog.SetDestination(req.Source())

		uaa, _ := sipgo.NewUA(
			sipgo.WithUserAgent("SIPGO"),
			// sipgo.WithUserAgentIP(*extIP),
		)
		ctxx := context.Background()

		log.Logger.Info().Msgf("TransactionRequest %v", catalog.String())

		client, _ := sipgo.NewClient(uaa, sipgo.WithClientNAT(), sipgo.WithClientPort(5060))`

Hi @srymurphy this one is problematic. I guess you want to reuse UDP listener, and so far
todo this you need to say on client same Hostname and Port.
Port will not be enough, you need to set both.

Hi @srymurphy this one is problematic. I guess you want to reuse UDP listener, and so far todo this you need to say on client same Hostname and Port. Port will not be enough, you need to set both.

client, _ := sipgo.NewClient(uaa, sipgo.WithClientHostname("<IP_ADDRESS>"), sipgo.WithClientPort(5060))
TransactionRequest listen udp <IP_ADDRESS>:5060: bind: Only one usage of each socket address (protocol/network address/port) is normally permitted
Adding the host setting has no effect. Can we use the server-side socket object to directly send requests

Hi @srymurphy . Can you pls explain how are you binding.
Are you binding maybe on all interfaces. Normally this works only in case of static IP.

Can we use the server-side socket object to directly send requests

By default sipgo reuses connection if the remote addr match.
You should probably achieve this by forcing SetDestination and getting Source from Server request.
So in case you received request on 5060 UDP it should go out on same.
Still consider in dialog above is not recommended. But I see you are actually just passing MESSAGE. request.


Checkout Footer / Checkout Policies

Please list the related package(s)
checkout-ui-extensions
If this related to specific APIs or components, please list them here
AFAIK there is no API or Component for that
Is your feature request related to a problem? Please describe.
I see no way to modify the checkout footer or to modify the modals where the policies are being rendered
Describe the changes you are looking for
Being able to modify checkout footer, especially regarding policy links - props like href or target. Being able to add other links and remove standard policy links.
Describe alternatives you’ve considered
I haven't found any alternative
Additional context
We have added policies as a HTML code in the shop settings and they are rendered beautifully in the shop frontend, but the checkout modal box doesn't apply the html and renders pretty much plain text. In the checkout.liquid we solved it by modifying policy links with javascript, adding target="_blank". We also have another store running where we replace policy links with some other links.

hello there! We are currently working on a project to enable header & footer customizations. As part of this project, new extension targets in the Checkout header & footer will be added.
You will be able to create a footer extension with custom links & modals then.

The above-mentioned feature is live. Here is a relevant tutorial. I'm closing this out as done, but feel free to circle back with questions.


Walmart brand filter list

Hello, can we have the ability to also filter walmart.com brands? thanks

Currently no, but eventually.
The vision for this addon is not to filter out "generics."
That being said I am planning to add personal apps allow and block lists.  Once that's in you can add any brands you specifically don't want to see.


Big load of changes

I've been integrating other people's changes as well as making my own in order to get this library working with recent devices/files/python versions. As a result,  this PR isn't really focused on a specific issue but instead has a random grab-bag of commits.
If you would like to pick and choose which ones to use, feel free, they're mostly divided up into pretty reasonable chunks that could be cherry-picked.
Highlights (see commit messages for details):

Python 3 support (maybe still works with 2? I didn't test it. Dropped Python 2 support. Python 2.7 still supported)
more tests
support for chained fit files
updated to the latest version of the SDK (including dev fields)
fixed bugs
more flexible processing class API (used to implement converting all speeds to km/h)

Getting AttributeError: 'module' object has no attribute 'zip_longest' trying to run.

@sanderroosendaal

Python 3 support (maybe still works with 2? I didn't test it)

Seems it's not working in Python 2, I'll update the PR description.

Yes, that is Python 2.
You implemented all the things I need (developer fields support). My site runs on Python 2.7 and is now using the ng branch of python-fitparse.

Added Python 2.7 support back in. If anyone finds a file that doesn't parse properly (in 2 or 3), let me know. It works with everything I could find to throw at it using both 2 and 3.

Carey would you like the maintain this package? I've orphaned it. Got a minute to sync up? I'm at<EMAIL_ADDRESS>

@dtcooper Sent you an email

In records.py there is a temp.raw_value in line 378 which is not defined. This causes FitFile to choke on one of the attached FIT files when getting the messages
fitfiles.zip

@sanderroosendaal whoops, fixed now. I also added one of your files to the repo as a test file. I can remove it if that's not ok with you.

FYI: Approved for merge @pR0Ps :)

@pR0Ps All good to kill the ng branch when you're done. ;D


Fix: velaux addon hint after enable

Signed-off-by: qiaozp<EMAIL_ADDRESS>Description of your changes
There are some changes in vela port-forward, this PR fix the legacy prompt when enable velaux addon.

I have:

[x] Read and followed KubeVela's contribution process.
[ ] Related Docs updated properly. In a new feature or configuration option, an update to the documentation is necessary.
[x] Run make reviewable to ensure this PR is ready for review.
[x] Added backport release-x.y labels to auto-backport this PR if necessary.

How has this code been tested

Special notes for your reviewer

How can I print additional messages after enabling terraform-xxx? Like
$ vela addon enable terraform-alibaba
...
terraform-alibaba is successfully enabled.

Please authenticate terraform-alibaba provider by `vela provider add  terraform-alibaba`

How can I print additional messages after enabling terraform-xxx? Like
$ vela addon enable terraform-alibaba
...
terraform-alibaba is successfully enabled.

Please authenticate terraform-alibaba provider by `vela provider add  terraform-alibaba`

Maybe make a more general post-enable hook function?


Add default channel to conda on RTD

As discussed here: https://github.com/pydata/xarray/issues/1106
I also added a section somewhere to print the packages versions on RTD. It will be useful to pin the versions again once it works, and is useful enough to stay on the docs I think. I just didn't really knew where to put it, other ideas welcome!

We print version info in conf.py, which shows up in the build logs on RTD:
https://github.com/pydata/xarray/blob/master/doc/conf.py#L18

Uh, this is a good candidate for a from __future__ import print_function

indeed!


Remove concept of ignored / condemned paths from Cache#transform.

This additional complexity is no longer necessary now that
didTransform events are no longer being emitted from the Cache
when related inverse ops are applied.

Nice! The handling of the removed paths did feel a little awkward.


Python packaging

The packaging currently relies on a conf.py file which is a bit of an outdated practice.
I highly suggest switching to using a pyproject.toml and setup.cfg files which is the current recommended practice: https://packaging.python.org/en/latest/tutorials/packaging-projects/#configuring-metadata
You can check out the twoaxistracking package as inspiration: setup.cfg and pyproject.toml. Note how the versions of the documentation packages are all fixes (this is recommended to avoid your documentation breaking in the future as new releases become available).
I'm partly recommending this because when I tried to build the documentation, then my environment didn't have Sphinx for example. If using the above approach, one can then get all the documentation packages by installing as:
pip install pydem[doc]

Hmm hang on a second, let me look into this a bit more. It seems that you are using a pyproject.toml

The second [environment.yml](https://github.com/trchudley/pdemtools/blob/main/docs/environment.yml) file in the docs folder threw me off.
My recommendation would be to get rid of both of the environment.yml files and include the dependencies in the pyproject.toml as described here: https://packaging.python.org/en/latest/guides/writing-pyproject-toml/

Here's a bit more info:

Starting with PEP 621, the Python community selected pyproject.toml as a standard way of specifying project metadata. Setuptools has adopted this standard and will use the information contained in this file as an input in the build process.

Source

Thanks Adam. Yes, pdemtools itself is using a pyproject.toml file. However, the readthedocs/sphinx website is built on top of a sample repo that used a conf.py file. I've attempted to look into this and it doesn't look like sphinx fully/consistently supports pyproject.toml files (yet?). The official documentation still describes conf.toml files. Our chosen website theme also still recommends a conf.py file in the setup tutorial. I don't think this is simply a case of outdated docs - even 'big' projects like xarray are still using conf.py files in their docs. The sphinx-pyproject project, which claims to move "some" (?) of the sphinx configuration into a pyproject.toml, is still in development mode. I guess some non-Python sphinx background processes in the makefile may depend on the conf.py file in an unusual manner?
I am very much an end-user of the sphinx/readthedocs tools and getting into the weeds of the sphinx configuration is making me a little anxious about breaking things. If it's all right can I defer moving the website to a toml file until Sphinx community development and tutorials/support surrounding pyproject.toml is more mature?
As an interim, I have included a README.md file in the docs directory that outlines how users can edit and build the website (although I very few outside the core pdemtools team should really need to). Is this acceptable to you?

@trchudley Sorry for the poor explanation of this issue. I was not concerned with the conf.py for the docs, but initially I thought you were using a conf.py for the packaging (though this is not the case). The changes in #39 is really what I meant, i.e., removing the two conda environment files if possible.

Thanks Adam. I've had a look at the pull request.
I definitely wouldn't want to remove the root directory yaml file. This is necessary to set up our github actions (e.g. testing), and is the recommended way of installing the dev environment. This is so that we can enforce installation via conda, given our previously discussed issues surrounding GDAL and pip. A base environment yml file appears to be common in peer projects for dev install purposes (e.g. Penny How's pypromice, also a JOSS paper, recommends dev installation via the yml file).
As for the docs yml file, this is for the same reason: so that we can enforce readthedocs to create the website within a GDAL-friendly conda environment (hence the inclusion of the conda: environment: docs/environment.yml lines that you recommended deleting). Whilst it looks like introducing pdemtools[doc] optional dependencies would allow us to at least synthesise this into the root directory yaml file, I am not aware of a way for readthedocs to infer an optional dependency install: this is the reason for the seperate docs/environment.yml file.
I'm open to other ideas if I've missed anything. Perhaps it might be sufficient to rename docs/environment.yml to doc/readthedocs_env.yml or similar to avoid confusion?

These are fair arguments.


WidgetExt::hide_on_delete useless in current form

fn hide_on_delete(&self) -> bool

should be connectable to
fn connect_delete_event<F: Fn(&Self, &Event) -> Inhibit + 'static>(
    &self,
    f: F
) -> SignalHandlerId

which doesn't work out that well.

https://gtk-rs.org/docs/gtk/trait.WidgetExt.html#tymethod.hide_on_delete
https://gtk-rs.org/docs/gtk/trait.WidgetExt.html#tymethod.connect_delete_event

Yes it should return an Inhibit, needs a manual implementation for that. Do you want to create a PR?
As workaround for now you can do
window.connect_delete_event(|window| Inhibit(window.hide_on_delete(window)))

Actually, |window, _| :)
I will have a look at the manual implementations tomorrow. Doesn't sound that complicated

Ok :) Let me know if you have questions


Configure whenever and add a first job

refs #469

Coverage remained the same at 80.35% when pulling 8d948bd2dbc02a51b03197ce9aeb6f49d5f5f3bf on issue/469-whenever into 379582fbb27d5a2c87c1e497b0543a0ae39c5f2f on master.

This looks great @hackmastera!


Add support for org.antlr:antlr4-runtime:4.11.1

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]

No.

Describe the solution you'd like
A clear and concise description of what you want to happen.

Add support for org.antlr:antlr4-runtime:4.11.1.

Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.

Null.

Additional context
Add any other context or screenshots about the feature request here.

Metadata generated at https://github.com/linghengqian/graalvm-trace-metadata-smoketest/issues/1 .

Linkes to https://github.com/apache/shardingsphere/issues/21347 .

I'm working on this issue.

Verification at https://github.com/apache/shardingsphere/issues/29052 shows that org.antlr:antlr4-runtime does not require GraalVM Reachability Metadata.


vimPlugins.jsonfly-nvim: init at 2024-04-12

Description of changes
Adds the jsonfly.nvim NeoVim plugin
Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] 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
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.05 Release Notes (or backporting 23.05 and 23.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

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Result of nixpkgs-review run on x86_64-linux 1

1 package built:

vimPlugins.jsonfly-nvim

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-ready-for-review/3032/3799


/vc investors in company_name

Another great command IMO would be to be able to look up the investors in a given company, eg
/vc investors in uber

That will surely be much more interesting and helpful, thank you! @ernopp


Some step definitions are getting really long.
I know that there is a way to disable this only in certain directories, but that didn't work on my machine,
so this is a temporary fix until we figure out how to do this. I don't want this to hold up the review process.

This is the last PR in this series, and it passes the tests.


[FABCAG-7] introduce ledger [FABCAG-9] introduce collection

Partially resolves:
FABCAG-7
FABCAG-9
Introduces:

Ledger Object
Collections Object

Does not resolve:

integration tests, these will be built out once the underlying interaction with the world state is complete

Need to re-work


Add a Codacy badge to README.adoc

With this addition, it will help to gauge the quality of our code better. Great find!


Feature request - Batch export, export selected objects as individual GLB's.

Is your feature request related to a problem? Please describe.
Working with a variety of GLB projects, used for customization software, I'll often need to export 5-20 different variations/pieces of my models. A batch export setting would be ideal for this situation, where all of my existing saved export settings can be used to export the set as individual GLBs in one go.
Describe the solution you'd like
Add a checkbox for "Batch export" that exports my selected objects as individual GLBs, or alternatively turn the "Selected Objects" checkbox to a dropdown enum that has "All visible" "Selected Objects" and "Batch Export" (Or "Selected (Batch)")

I second that.

This leads to lots of question that really depend on your pipeline/workflow:

Do you want individual glb to be world position or any individual center in world?
What about animations? What if your object is not animated but parented to an animated object?
etc...

Adding option for all these cases will add much more complication in UI that we may really want.
(BTW, I think there is already multiple addons that are doing it, on top of this I/O addon)


Make CDeterministicMN::internalId private to make sure it's set/accessed properly

Refactoring only, based on #3504 atm to avoid merge conflicts.

Makes overall sense 👍 What about even getting rid of the setter and make it only accessible by constructor and serializations like in here 653cb219918b528e8e74617ff795d0837078b010?

@xdustinface added your patch, thanks! 👍 I tweaked ctors a bit to ensure we never create DMNs with the initial internalId value/no id specified. There is one exception where "no id" is a desired (intermediate) outcome - the upgrade logic for CDeterministicMNListDiff_OldFormat. Have to use a workaround for this specific case which IMO is ok-ish because they are just temporary data structures, see NOTE in code.

utACK! Agree with the workaround and the tweaks are looking good 👍

Rebased, should be clean and ready to merge now.


Update Android service to use plugin capabilities for sharing intents

Fixed in https://github.com/learningequality/kolibri-installer-android/pull/68


search-as-primary-CTA JIRA-ID-CXPUI-4

Description
JIRA Ticket: https://newrelic.atlassian.net/browse/CXPUI-4
Previous, Search bar was below the guided install
Now, search bar is in banner below the text and also removed the view button
Reviewer Notes
If there are links or steps needed to test this work, add them here.
Related Issue(s) / Ticket(s)
If there are any related GitHub Issues or JIRA tickets, add links to them here.

issue

Screenshot(s)
OLD:

NEW:

Use Conventional Commits
Please help the maintainers by leveraging the following conventional commit
standards in your pull request title and commit messages.
Use chore

for minor changes / additions / corrections to content.
for minor changes / additions / corrections to images.
for minor non-functional changes / additions to github actions, github templates, package or config updates, etc

git commit -m "chore: adjusting config and content"

Use fix

for minor functional corrections to code.

git commit -m "fix: typo and prop error in the code of conduct"

Use feat

for major functional changes or additions to code.

git commit -m "feat(media): creating a video landing page"

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

Done changes in Incorrect repository


qbittorent api extend

According to webapi documentation find_torrent can be reimplemented with this api call: https://github.com/qbittorrent/qBittorrent/wiki/WebUI-API-Documentation#get-torrent-generic-properties
which has addition_date property.

It doesn't have name property, that's why I used list since it has both.

Why wontfix?

Closing since the name property necessary for the implementation is missing in https://github.com/qbittorrent/qBittorrent/wiki/WebUI-API-Documentation#get-torrent-generic-properties


Support Ethers v6 and Hardhat Toolbox 3.0.0

Currently, if you would like to use Hardhat Toolbox 3.0.0, it requires dependencies on hardhat-ethers 3.0.4 and Ethers v6. This leads to an incompatibility with Smock, because smock requires hardhat-ethers v2 and Ethers v5.
It would be nice if Smock was updated to support Ethers v6. Unfortunately, I think this might require removing the hardhat-waffle dependency, as there is not a v3 of that since it is dead (Hardhat has moved on to using chai matchers and hardhat network helpers which they claim is an improvement over waffle).
Ethers v6 has substantial changes, specifically the elimination of BigNumber. Its probably better to deal with that big update sooner rather than later.
Can you please update to support Ethers v6 and also the hardhat 3/toolbox approach? If not, can you provide insight into if that update will ever occur?
Thank you.

Upvoted this issue. I ran into the sample problem using current smock version.

Deps
    "@defi-wonderland/smock": "^2.3.5",
    "hardhat": "^2.17.3",
    "@nomicfoundation/hardhat-toolbox": "^3.0.0",
    "@typechain/ethers-v6": "^0.5.0",
    "@typechain/hardhat": "^9.0.0",
    "ethers": "^6.7.1",

When running test like below,
import { FakeContract, smock } from '@defi-wonderland/smock';
it.only("some words" , async function TestMock() {
      const fakeCn = await smock.fake<Assembly>('Assembly')
      console.log({fakeCn})
})

Hardhat complains throw below logs.
Error [ERR_PACKAGE_PATH_NOT_EXPORTED]: Package subpath './lib/utils' is not defined by "exports" in C:\Users\user\Desktop\code\web3-snippets\node_modules\.pnpm\@ethereum-waffle+provider@4.0.5_@ensdomains+ens@0.4.5_@ensdomains+resolver<EMAIL_ADDRESS>

Hello, like @mvillere asked, do you have plans to support ethersV6? I love this package and I'd like to continue using it but upgrading to hardhat3 and ethersV6 is inevitable for most of us.

any news on that?
I found that one fork tried to update it to ethers-v6 (@puuuuh/smock), but I couldn't make it work.

Any updates on this one?


Fixes error: Correctly sets YouTube consent cookie

Hey,
Great work with your repo. I've seen that YouTube made some changes recently that make XCDYouTubeKit fail in many regions. The issue was "discussed" in #523.
I implemented the same solution as in youtube-dl commit 14f29f. It basically has to set a "consent-cookie" so YouTube sees you agree with their terms and conditions, cookie rules or whatever.
Would be happy to hear your feedback

@ekincelik  Still it is not working with some urls and giving me error with reason string "Video unavailable". Please check it.
Domain: XCDYouTubeVideoErrorDomain
Code: -2
Below Url is not working
https://www.youtube.com/watch?v=J6R4u8jSkh4

@ekincelik Still it is not working with some urls and giving me error with reason string "Video unavailable". Please check it.
Domain: XCDYouTubeVideoErrorDomain
Code: -2
Below Url is not working
https://www.youtube.com/watch?v=J6R4u8jSkh4

Works for me with my pull request

Works, thanks!

I had downloaded the code from alexeichhorn:fix/youtube-consent branch and added the required XCDYoutubeKit folder into project, because there is no new pod version available for this fix.
I run the project on real device and observed that https://www.youtube.com/watch?v=J6R4u8jSkh4 url gives me same error video unavailable.
Am I doing anything wrong here, how it’s working for u and not for me. Please suggest we depends on this lib and in a release phase. Please advise.

I tested your video using the demo application in the repository, which worked, and used SwiftPM to integrate the framework in my application where I haven't tested your video, but a lot of others.
There's a chance this problem is region based, so maybe try to use a VPN to test some other regions. I myself tested it in Switzerland.

hi @Praveen-Balpande
I used alexeichhorn:fix/youtube-consent branch   too. Your video is working for me.
You can check response in handleConnectionSuccessWithData method on XCDYouTubeVideoOperation.m class in order to understand what the problem is.

Hi everyone,
I am getting this error while playing video id "E_lByLdKeKY"
2021-05-13 16:39:53.974689+0530 Quizzwizz[1362:383440] [XCDYouTubeKit] Failed to decode response from https://www.youtube.com/get_video_info?el=embedded&hl=en&ps=default&video_id=E_lByLdKeKY (response.textEncodingName = utf-8, data.length = 0)
2021-05-13 16:39:53.975118+0530 Quizzwizz[1362:383440] [XCDYouTubeKit] Video operation finished with error: The operation couldn’t be completed. (XCDYouTubeVideoErrorDomain error -3.)
Domain: XCDYouTubeVideoErrorDomain
Code:   -3
User Info: {
}
Couldn't get the video from video id
please note the video works on simulator but giving error on device. please help as we are about to release our app tomorrow.

It works for me, thanks a lot
Jarek

It works for me, thanks a lot
Jarek

please help me also...in my app most of the videos are not getting played..:(

It works for me, thanks a lot
Jarek

please help me also...in my app most of the videos are not getting played..:(

Hi
I have added new lines from
https://github.com/0xced/XCDYouTubeKit/pull/524/commits/893401b5dcfefa300e324bbe93af6ab2be49e384
to the file:
XCDYouTubeKit/XCDYouTubeVideoOperation.m
Hope that helps
Best
Jarek

Can you add the fix from other pull-requests:
NSDictionary *query = @{ @"video_id": self.videoIdentifier, @"hl": self.languageIdentifier, @"el": eventLabel, @"ps": @"default", @"html5" : @"1" };

@sherwinzadeh I've merged both pull requests in the master branch of my fork. I use it myself in SwiftPM for my own project.

@sherwinzadeh I've merged both pull requests in the master branch of my fork. I use it myself in SwiftPM for my own project.

Hi, can you please write what is the correct pod file entry for your fork?


NetworkManager: fix VPN service source locations

Motivation for this change
These have all moved to lib/NetworkManager/VPN in the plugin build outputs rather than etc/NetworkManager/VPN.  This fixes the symlinks so NetworkManager can find its vpn plugins again.
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[x] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[x] Determined the impact on package closure size (by running nix path-info -S before and after)
[x] Fits CONTRIBUTING.md.

See https://github.com/NixOS/nixpkgs/pull/46201

ugh sorry, I forgot to rebase against current master before opening this PR.


fix: ToolBar title not avoiding traffic lights when no sidebar is present

Closes #371
Pre-launch Checklist

[x] I have incremented the package version as appropriate and updated CHANGELOG.md with my changes
[x] I have added/updated relevant documentation
[x] I have run "optimize/organize imports" on all changed files
[x] I have addressed all analyzer warnings as best I could

@whiplashoo @Adrian-Samoticha can I get a sanity check on this, please? Run an app of yours based on this branch and make sure everything still works properly.


Feature Request: Windows Support

Durdraw 0.15 actually worked alright in Windows (in Command Prompt, Windows Terminal, etc) with windows-ncurses installed, but the latest build (0.17.2) has issues.
The issue seems to have started when I started using curses panels. Curses.panel.move() returns ERR. I haven't investigated too deeply, but some possible theories are that the Windows curses clone does not fully implement panels, or maybe that it needs a bit more care with the order things are done in. It would be nice if someone would help track down and fix the Windows issues.

This appears to be caused by windows-curses having an incomplete or broken implementation of panels:
zephyrproject-rtos/windows-curses#11

It looks like, in addition to WSL, Durdraw can be used in Windows through Docker. This works in the Command Prompt Window, Powershell window, etc. No obvious problems.
The upside is that it works. The downside is, Docker is a pretty heavy requirement. A self-contained .exe would be nicer.


feat(render): Add slots to allow custom navigation and pagination

Allow custom components to be used for rendering pagination and navigation, with access to the carousel provider.
The following named slots have been added:

pagination
navigation

<carousel>
  <slide>content</slide>
  <custom-pagination slot="pagination" />
  <custom-navigation slot="navigation" />
</carousel>

Description
The pagination and navigation included in vue-carousel is somewhat limited. Adding named slots to allow custom components to be used to render these elements adds more flexibility without impacting the default usage. Without slots, its not possible for custom components to access the carousel provider.
Motivation and Context
To allow for more custom rendering of navigation and pagination elements.
How Has This Been Tested?
Using vue-play in the browser. This is a very simple change, it should not impact the current default behavior.
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist:

[x ] My code follows the code style of this project.
[ x] My change requires a change to the documentation.

[ x] I have updated the documentation accordingly.

[ ] I have included a vue-play example (if this is a new feature)

I haven't added an example to vue-play, the default pagination and navigation components already verify the feature is working correctly.

Hey @adam-26 this is great! Thanks a million for the PR. I'll merge in with the next release, should be sometime this week :octocat:


CAPT-1121 Amend exit screen for users who select 'none of the above' for their ITT year - LUP schools

CAPT-1121

There is an additional line of copy to add which is specified in the Jira ticket.

This line has already added.


Option to keep navigation drawer always open

In flutter web would be very convenient in some cases to have the drawer fixed open, so you can interact with other content in the page and it stays open.
Many websites have this behavior. Such as youtube, gmail, google news, etc.

Removing the web label because this is a general feature request for all big screens.
/cc @HansMuller

I am starting to develop a web app and this function would be excellent to have...

Any update on this one?

Any updates here?

It would be awesome. Also, it follows the material design guideline for the web.

Any update regarding this?

We need gmail-like look and behavior for drawer in web apps. Please give us any estimation for this feature.

Any updates on "standard" and "bottom" drawers? https://material.io/components/navigation-drawer

You can get an implementation of a working responsive drawer scaffold here: https://gitlab.com/testapp-system/testapp-flutter/-/blob/mobile/lib/drawer.dart#L219

Any update?

This will be useful.

Hope the devs prioritize this feature

It would be awesome feature for web applications. Any update on this?

Any updates for this feature ?

Opting this!

Opting this!

One workaround I use is having a Row for Scaffold.body, and then making the first widget a Drawer, and the second the actual body. By not including it in the Scaffold, the AppBar doesn't take control of the Drawer.
So something like this:
class FullScreenPage extends StatelessWidget {
  @override
  Widget build(BuildContext context) => Scaffold(
    appBar: AppBar(title: Text("Big screen web app")),
    body: Row(
      children: [
        Drawer(/* Menu items here */),
        Placeholder(/* Normal body here */),
      ]
    )
  );
}

One workaround I use is having a Row for Scaffold.body, and then making the first widget a Drawer, and the second the actual body. By not including it in the Scaffold, the AppBar doesn't take control of the Drawer.
So something like this:
class FullScreenPage extends StatelessWidget {
  @override
  Widget build(BuildContext context) => Scaffold(
    appBar: AppBar(title: Text("Big screen web app")),
    body: Row(
      children: [
        Drawer(/* Menu items here */),
        Placeholder(/* Normal body here */),
      ]
    )
  );
}

Right! Thank you.
It works!

Right! Thank you.
It works!

One workaround I use is having a Row for Scaffold.body, and then making the first widget a Drawer, and the second the actual body. By not including it in the Scaffold, the AppBar doesn't take control of the Drawer.
So something like this:
class FullScreenPage extends StatelessWidget {
  @override
  Widget build(BuildContext context) => Scaffold(
    appBar: AppBar(title: Text("Big screen web app")),
    body: Row(
      children: [
        Drawer(/* Menu items here */),
        Placeholder(/* Normal body here */),
      ]
    )
  );
}

The thing is, with this approach, you will lose back navigation from the appbar :(

One workaround I use is having a Row for Scaffold.body, and then making the first widget a Drawer, and the second the actual body. By not including it in the Scaffold, the AppBar doesn't take control of the Drawer.
So something like this:
class FullScreenPage extends StatelessWidget {
  @override
  Widget build(BuildContext context) => Scaffold(
    appBar: AppBar(title: Text("Big screen web app")),
    body: Row(
      children: [
        Drawer(/* Menu items here */),
        Placeholder(/* Normal body here */),
      ]
    )
  );
}

The thing is, with this approach, you will lose back navigation from the appbar :(

@Levi-Lesches' solution works for the question and for web/Android, but to follow the guidelines on iOS you need to remove the AppBar - what iOS calls a Split View.  It's the way iOS Mail app looks like on iPad.
https://developer.apple.com/design/human-interface-guidelines/ios/views/split-views/

Is there already an out-of-box solution available or is @Levi-Lesches' example the go-to workaround for now?

There are a few packages on pub.dev, such as adaptive_navigation (by the Material team) and adaptive_scaffold, but I personally found that neither fit the feel or functionality of my app, so I built my own version (more complex than my comment above). It's certainly not impossible and allows you to have greater control than those packages can give you.

Any updates for this feature?

This is a pretty critical item for meeting the Material Design standards while building multi-platform apps. It would be nice to see something implemented soon

im actually looking for the same thing, does anyone know how to do this?

See my comment above, which is basically keeping the Drawer in your Scaffold's body instead of in the designated Scaffold.drawer field.

Maybe some may be happy with a navigation rail instead: https://api.flutter.dev/flutter/material/NavigationRail-class.html

Maybe some may be happy with a navigation rail instead: https://api.flutter.dev/flutter/material/NavigationRail-class.html

Thats a good option

Adding this to the Scaffold API would be complicated as it would mean moving the drawer into the content area. It would be better to have the application manage this itself. In addition, as pointed out above, we now have the Navigation Rail widget which can do this automatically and keeps the layout of the content area in control of the app (it also appears to be the direction that the Material Design spec is promoting).
Given that options, I am going to close this issue.

For future GitHub comments, please consider reducing usage of pronouns "this" and "it", for those of us less familiar with the problem space.
Specific examples:

do this automatically

it also appears to be the direction that the Material Design spec is promoting

Those pronouns compound my confusion as to the precise relevance of Navigation Bar in a thread about Navigation Drawer. I don't understand the problem space enough to construct a clarifying question. I'm not asking for clarification now. It's just a suggestion to avoid pronouns in the future.


Update en

Updated the English file with some bad words.

commenting here vs noising it up with a new new issue / PR:
comparing your list against a custom list:
>>> [x for x in bad.bad_words if x not in en]
['christ', 'cocaine', 'condoms', 'fatass', 'hooters', 'pooppoop', 'poo']


Failing to parse https://registry.npmjs.org/react-router

https://registry.npmjs.org/react-router
error: Error: contentExtractor: Unexpected token \(…)
somewhere here: https://github.com/tulios/json-viewer/blob/master/extension/src/json-viewer/content-extractor.js#L17

It might be related to the content present in this response, other resources are working, e.g:
https://registry.npmjs.org/file-loader


signal-desktop: 5.52.0 -> 5.53.0

Description of changes
https://github.com/signalapp/Signal-Desktop/releases/tag/v5.53.0
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)
[x] 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

[ ] 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.11 Release Notes (or backporting 22.05 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

[x] Fits CONTRIBUTING.md.

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

1 package built:

signal-desktop


fixed Date.parse exception

It's annoying to have this exeception firing constantly when debugging.  I fixed it.

@sgehrman thanks! Please change your commit to staging (not master), then I'll merge it.

@sgehrman ^^


Provide a description for soda-sql-dbt that will show up on PyPI

Right now the PyPI package page for soda-sql-dbt is empty and it doesn't look all that re-assuring. We should at least have some information that points to the main repo of soda-sql core.
@vijaykiran @janet-can if you have some guidelines feel free to drop them in here and I can take care of opening a PR.

We also have a pending issue for all the packages : https://github.com/sodadata/soda-sql/issues/145 -  I'll add them next week, since we recently removed the beta.

Ah yeah 🤦 I should have done a quick search before. Feel free to close this issue in favour of the other ones @vijaykiran


Support for defmt logging

https://github.com/knurling-rs/defmt is a new approach to tracing/logging from embedded devices which dramatically reduces bandwidth use by deferring formatting to the host, and using indexed format strings.
I think this is a very good fit for embedded smoltcp, since other than in low-traffic situations it is currently basically impossible to use smoltcp's (extensive) logging.  With defmt, this might become feasible.
Unfortunately, format strings are not compatible since for compactness, defmt format strings need to contain more information about the formatted type (e.g. {:u8}).
Since defmt isn't meant for non-embedded use, smoltcp would have to switch between them using a feature flag.  The easiest way I could see to support both syntaxes is to use a custom proc macro (net_trace and net_debug are already custom declarative macros anyway), which outputs two versions of the logging call depending on the feature flag selected (and none if logging is switched off).  However, it's also conceivable for defmt to implement that, so I opened https://github.com/knurling-rs/defmt/issues/149 there.
cc @thalesfragoso @adamgreig (stm32-eth developers)

since other than in low-traffic situations it is currently basically impossible to use smoltcp's (extensive) logging

You can write the logs to a ring buffer, and then trigger a snapshot of it through some condition. The utility of this is largely limited to lockups, and while that's nice, I agree with you overall.

However, it's also conceivable for defmt to implement that, so I opened knurling-rs/defmt#149 there.

I would much prefer that to happen. A separate crate that implements just the proc macro would also be fine.

Thanks for the feedback! In any case, I'll wait for defmt to stabilize enough for a crates.io release before I try to implement anything...

I have a PoC of this here https://github.com/akiles/smoltcp/commit/f4cc2c956bfa6db5fea4cdc79078fa0b6bdc9db0
It's quite ugly, and it required changing all {} to {:?}. It builds for both defmt and log though!
The ideal solution would be some macro that takes a "supercharged" format string and converts it to a defmt format string or a log format string depending on enabled features...

Done in #455! With defmt 0.2 there's no longer a need to change {} into {:?}


GitHub bot reports "edited" when a comment gets hidden

GitHub has a feature to hide a comment on an issue or PR:
https://docs.github.com/en/communities/moderating-comments-and-conversations/managing-disruptive-comments#hiding-a-comment
This can be useful when a comment isn't helpful for reading a thread, for any of a range of reasons — from outright abuse, down through misdirected off-topic comments, down to comments that were perfectly sensible when made but are outdated and have become clutter.
But apparently some of the time, when using this feature, the comment ends up getting repeated by Zulip's GitHub bot, saying the comment was "edited". That's bad behavior — the point of using this feature is that the comment is, at best, noise.
Instead, when a comment gets hidden, the behavior I'll personally want is that the bot doesn't send any message about that at all.
Potentially other users configuring the bot might want messages for such events. In that case, they should (a) be possible to filter separately, so that users who don't want them can screen them out; (b) use an accurate verb like "hid", instead of "edited"; (c) leave out the comment's body.
Observed examples
For examples, see the three bot messages starting here:
https://chat.zulip.org/#narrow/stream/494-mobile-github/topic/zulip-flutter/near/1899311
The behavior is a bit odd in that it doesn't seem to be deterministic, at least not on any pattern I can see. If you look at the issue those messages point to, there are six comments "marked as outdated" (they're six of the latest seven comments on that issue, as of now when I'm reporting this issue), including the three that the bot reported as "edited". All six of those, I marked as outdated today, in the span of a couple of minutes.
Of the six comments I marked as outdated, two were by me and four by another contributor. The ones the bot reported cut across both categories: they include one by me and two by the other contributor.
How to approach this issue
The next step needed will be to get an example of what the webhook payload from GitHub looks like when this happens. The ideal way to do this is to reproduce it in a test repository, so that'd be the first thing to try. If that doesn't work, we can set up to collect logs for the next time it happens in the wild; see a chat discussion in the context of #26136.
Zulip Server and web app version:

[ ] Zulip Cloud (*.zulipchat.com)
[x] Zulip Server 8.0+ (namely, chat.zulip.org)
[ ] Zulip Server 7.0+
[ ] Zulip Server 6.0+
[ ] Zulip Server 5.0 or older
[ ] Other or not sure

@gnprice I would like to work here.

@zulipbot  claim

@zulipbot claim

@zulipbot abandon

@zulipbot claim

@Niloth-p can you fix this one?


Write manual test cases for ManitobaPage.

Please write a manual test case for "Manitobapage: https://prayersconnect.com/mosques/search/Winnipeg%252C%2520Manitoba%252C%2520Canada/49.8954221%2C-97.1385145/49.8954221%2C-97.1385145" which involves breaking down the functionality of the webpage into specific steps that can be tested.
Please use this Google sheet for your test cases.
https://docs.google.com/spreadsheets/d/1ci-_ZZpDDkxCGeviDORBB0CPspBzZBmmSnvbjvDf9KE/edit?usp=sharing

Completed and submitted herewith:
https://docs.google.com/spreadsheets/d/1VG999MAq6GLH0lbUGhgr7P7oVcW5jUbI4AyEhYjf62U/edit?usp=sharing

Thank you.


[Dev] New package

@frapac @jbcaillau @PierreMartinon
What do you think we need to do before trying to register OptimalControlProblems.jl into the General Julia registry?

@ocots go!
NB. @0Yassine0 would be nice to use the new syntax ocp = @def ... (instead of @def ocp ...)

@JuliaRegistrator register()

Registration pull request created: JuliaRegistries/General/115146
Tip: Release Notes
Did you know you can add release notes too? Just add markdown formatted text underneath the comment after the text
"Release notes:" and it will be added to the registry PR, and if TagBot is installed it will also be added to the
release that TagBot creates. i.e.
@JuliaRegistrator register

Release notes:

## Breaking changes

- blah

To add them here just re-invoke and the PR will be updated.
Tagging
After the above pull request is merged, it is recommended that a tag is created on this repository for the registered package version.
This will be done automatically if the Julia TagBot GitHub Action is installed, or can be done manually through the github interface, or via:
git tag -a v0.1.1 -m "<description of version>" e28285cb58c3a56e70584679b8468bf40125509f
git push origin v0.1.1

Also, note the warning: This looks like a new registration that registers version 0.1.1.
Ideally, you should register an initial release with 0.0.1, 0.1.0 or 1.0.0 version numbers
This can be safely ignored. However, if you want to fix this you can do so. Call register() again after making the fix. This will update the Pull request.


swupdate: remove non-essential swupdate services

swupdate-progress.service
<EMAIL_ADDRESS>
Signed-off-by: Tariq Ansari<EMAIL_ADDRESS>

would have been nice to have explanations what they were about and why we didn't need them

would have been nice to have explanations what they were about and why we didn't need them

Yes, always try to give an answer to "why?".


Yet again mixcloud not working

Please follow the guide below

You will be asked some questions and requested to provide some information, please read them carefully and answer honestly
Put an x into all the boxes [ ] relevant to your issue (like this: [x])
Use the Preview tab to see what your issue will actually look like

Make sure you are using the latest version: run youtube-dl --version and ensure your version is 2017.08.18. If it's not, read this FAQ entry and update. Issues with outdated version will be rejected.

[x ] I've verified and I assure that I'm running youtube-dl 2017.08.18

Before submitting an issue make sure you have:

[ x] At least skimmed through the README, most notably the FAQ and BUGS sections
[x ] Searched the bugtracker for similar issues including closed ones

What is the purpose of your issue?

[ x] Bug report (encountered problems with youtube-dl)
[ ] Site support request (request for adding support for a new site)
[ ] Feature request (request for a new functionality)
[ ] Question
[ ] Other

The following sections concretize particular purposed issues, you can erase any section (the contents between triple ---) not applicable to your issue

If the purpose of this issue is a bug report, site support request or you are not completely sure provide the full verbose output as follows:
Add the -v flag to your command line you run youtube-dl with (youtube-dl -v <your command line>), copy the whole output and insert it here. It should look similar to one below (replace it with your log inserted between triple ```):
youtube-dl -g https://www.mixcloud.com/LiveSetsat/solomun-live-tomorrowland-2017-belgium-main-stage-23072017/ --verbose
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: [u'-g', u'https://www.mixcloud.com/LiveSetsat/solomun-live-tomorrowland-2017-belgium-main-stage-23072017/', u'--verbose']
WARNING: Assuming --restrict-filenames since file system encoding cannot encode all characters. Set the LC_ALL environment variable to fix this.
[debug] Encodings: locale ANSI_X3.4-1968, fs ANSI_X3.4-1968, out ANSI_X3.4-1968, pref ANSI_X3.4-1968
[debug] youtube-dl version 2017.08.18
[debug] Python version 2.7.5 - Linux-2.6.32-042stab123.9-x86_64-with-centos-7.3.1611-Core
[debug] exe versions: ffmpeg 2.6.8, ffprobe 2.6.8
[debug] Proxy map: {}
ERROR: LiveSetsat-solomun-live-tomorrowland-2017-belgium-main-stage-23072017: Failed to parse JSON  (caused by ValueError('Extra data: line 1 column 2 - line 4 column 91 (char 1 - 216)',)); please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
Traceback (most recent call last):
  File "/usr/local/bin/youtube-dl/youtube_dl/extractor/common.py", line 676, in _parse_json
    return json.loads(json_string)
  File "/usr/lib64/python2.7/json/__init__.py", line 338, in loads
    return _default_decoder.decode(s)
  File "/usr/lib64/python2.7/json/decoder.py", line 369, in decode
    raise ValueError(errmsg("Extra data", s, end, len(s)))
ValueError: Extra data: line 1 column 2 - line 4 column 91 (char 1 - 216)
Traceback (most recent call last):
  File "/usr/local/bin/youtube-dl/youtube_dl/YoutubeDL.py", line 776, in extract_info
    ie_result = ie.extract(url)
  File "/usr/local/bin/youtube-dl/youtube_dl/extractor/common.py", line 433, in extract
    ie_result = self._real_extract(url)
  File "/usr/local/bin/youtube-dl/youtube_dl/extractor/mixcloud.py", line 108, in _real_extract
    play_info = self._decrypt_play_info(encrypted_play_info, track_id)
  File "/usr/local/bin/youtube-dl/youtube_dl/extractor/mixcloud.py", line 74, in _decrypt_play_info
    video_id)
  File "/usr/local/bin/youtube-dl/youtube_dl/extractor/common.py", line 680, in _parse_json
    raise ExtractorError(errmsg, cause=ve)
ExtractorError: LiveSetsat-solomun-live-tomorrowland-2017-belgium-main-stage-23072017: Failed to parse JSON  (caused by ValueError('Extra data: line 1 column 2 - line 4 column 91 (char 1 - 216)',)); please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.

...
<end of log>

If the purpose of this issue is a site support request please provide all kinds of example URLs support for which should be included (replace following example URLs by yours):

Single video: https://www.youtube.com/watch?v=BaW_jenozKc
Single video: https://youtu.be/BaW_jenozKc
Playlist: https://www.youtube.com/playlist?list=PL4lCao7KL_QFVb7Iudeipvc2BCavECqzc

Note that youtube-dl does not support sites dedicated to copyright infringement. In order for site support request to be accepted all provided example URLs should not violate any copyrights.

Description of your issue, suggested solution and other information
This is the master branch.

Either version not working :(

@dstftw Sergey, tagging you in case you have not seen.
If yes, then I am sorry.
Thanks.

It's not yet released.


transformation + (smoothed_trajectory – trajectory)?

Why design the transformation matrix this way?What does smoothed_trajectory – trajectory stands for?

Thanks for closing this issue as these questions aren't the type of thing issues are intended for.  Please see the README for questions like this:

Python video stabilization using OpenCV. Full searchable documentation here.

This module contains a single class (VidStab) used for video stabilization. This class is based on the work presented by Nghia Ho in SIMPLE VIDEO STABILIZATION USING OPENCV. The foundation code was found in a comment on Nghia Ho's post by the commenter with username koala.


FILMED & DISTRIBUTED ME INSIDE MY APARTMENT [ NO CONSENT ]

PEEPING TOMS
WWW.WILSONELSER.COM

Memo Style1.5.pdf

Your message
To<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>MSKYLINE BROKER<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>'MIKE DANILACK'; 'PAM OLSON'; 'PETER MERILL'; MSRB
Cc<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>'THE NEW YORK TIMES'
Subject: RE: *** LOCATED THE VIDEOGRAPHER FOR THE ZUCKERS… IN THE FEDERAL RECORD.
Sent: 2/6/2022 4:42 AM
was read on 2/8/2022 8:37 AM.
was read on 2/8/2022 8:37 AM.

Memo Style2.pdf

Your message
To<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>Cc<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>Subject:  RE: ***FWD.:::

  *** ***
Sent:  2/4/2022 2:09 AM
was read on 2/4/2022 2:12 AM.

  *** ***
Sent:  2/4/2022 1:55 AM
was read on 2/4/2022 2:00 AM.

Your message
To<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>Cc<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>Subject:  RE: ***FWD.:::

 HUNG UP...
Sent: 2/6/2022 2:33 PM
was read on 2/6/2022 6:49 PM.

 IN THE FEDERAL RECORD  (1901)
Read      LOCATED THE VIDEOGRAPHER FOR THE ZUCKERS… IN THE FEDERAL RECORD  (2001)
Read      LOCATED THE VIDEOGRAPHER FOR THE ZUCKERS…

County Clerk   § 11 440 Tampering with or fabricating physical evidence
https://github.com/BSCPGROUPHOLDINGSLLC/WILSONELSER-ZUCKER

https://github.com/BSCPGROUPHOLDINGSLLC/WILSONELSER-ZUCKER/pull/4
LAURIE ZUCKER ON AUGUST 10TH, 2020
8209: CRDMEMBER FILE - MORGAN STANLEY

COLIN BROOKS

COLIN BROOKS

PRIYA DARSINI RAGHURAM

PRIYA_DARSINI_RAGHURAM 1

PRIYA_DARSINI_RAGHURAM 2 - STFGX

STATE FARM LIFE INSURANCE COMPANY
https://github.com/WILSON-ELSER-STATEFARM-SULLIVAN-ZUCKER/FEDERAL-USE-AND-DISTRIBUTION-ONLY
REFERENCES.
https://github.com/BSCPGROUPHOLDINGSLLC/INDEX153974/files/9237684/document.index.TCR5.1.pdf
https://github.com/BSCPGROUPHOLDINGSLLC/INDEX153974/issues/23


Implemented cancelBeacon and buildBeacon hooks for Android

Change Type (required)
Indicate the type of change your pull request is:

[ ] patch
[ ] minor
[ ] major

Does your PR have any documentation updates?

[ ] Updated docs
[ ] No Update needed
[ ] Unable to update docs

📦 Published PR as canary version: 0.9.2--canary.542.18345

Try this version out locally by upgrading relevant packages to 0.9.2--canary.542.18345

/canary

/canary

/canary

fix lint and I think this is good to go

/canary


Option to disable image preview scrolling.

Occasionally I notice images in Phanpy "scrolling". This happens both on Preview Cards for links and images that have been truncated for display on the timeline.
I think this happens when the post/image has the focus (although I'm not really sure what that means on mobile!)
It's a really neat feature and I like it at a technical/UX level but... I find animations really visually distracting and so, personally, I'd rather not have them
Is your feature request related to a problem? Please describe.
This does not relate to a specific problem.
However as above I (and I understand this to be a not uncommon issue) find animations overly distracting.
Describe the solution you'd like
An option in the settings menu that would disable the automated "scrolling" that happens to images that have the focus.
Describe alternatives you've considered
The only other option I can think of is disabling it completely but obviously that isn't really an acceptable one especially given I recognise this is primarily a person project.
Or not implementing and just putting up with it I guess.
Additional context
I really like Phanpy and it's now my default app. I hope this request makes sense because I really wasn't sure on some of the terminology and if my assumption of when this happens is correct.

Yes, please!  I came to this repo to open an issue to request this.  I'm happy to see that somebody else has already done that.
The scrolling of images upon mouseover is annoying to say the least, and distracting to be fully honest.  Animation of this kind can be a problem for neurodivergent users, causing an accessibility issue.


User provided connect kwargs should have precedence over the default.

Closes #96
Example
with snowflake_connector.get_connection(autocommit=False) as conn: used to yield a conn object that still has autocommit set to None. Now it's set to False as the dev asked.
Screenshots

Checklist

[x] References any related issue by including "Closes #" or "Closes ".

If no issue exists and your change is not a small fix, please create an issue first.

[x] Includes tests or only affects documentation.
[x] Passes pre-commit checks.

Run pre-commit install && pre-commit run --all locally for formatting and linting.

[x] Includes screenshots of documentation updates.

Run mkdocs serve view documentation locally.

[x] Summarizes PR's changes in CHANGELOG.md

Thanks for the PR @JohnLemmonMedely! Can you please add a short description of this PR in CHANGELOG.md under the 'Unreleased` section?

@desertaxle I just added it. I also squashed it down to single commit. Please let me know if you need anything else.


DATAJDBC-356 - Support for large datasets with stream

This is a work in progress
I decided to open the pull request so it is easier to keep track of changes and get feedback.
I stared working on a solution for DATAJDBC-356. I did not started mybatis support yet and will start working on tests and better documentation / examples, but would like to discuss the solution API and implementation wise.
For this case, I created some support classes to return an SqlRowSet backed by a connected ResultSet and it's DataSource. I did it this way so I can keep track of all resources and close them when appropriated.
On the API side, I created a new @QueryStream annotation, to specify the ResultSet fetchSize and to not break current uses of the API (client code that don't close the resulting Stream).
I would much appraciate some review on this work in progress, as it is my first Spring contribution.

@schauder @detinho Is this still being worked on? We just realized that Spring Data JDBC is not capable of streaming large result sets and we are encountering the first memory issues because of this.

@denniseffing I guess the blocker issue here is if we must add stream support on JdbcTemplate first or not.
As @schauder pointed out, there is a issue on spring-jdbc and I'd be glad to work on it if there aren't any blockers or even continue on this implementation if it still makes sense to do it here.

@detinho Thanks for the fast response. I commented on the linked issue as well. It would be awesome if Stream support would be added in the near future.

The framework issue is fixed now. A PR with an implementation based on that would be highly appreciated.

@detinho Would you mind working on a PR for this? Take a look at the new queryForStream methods in JdbcTemplate. See https://github.com/spring-projects/spring-framework/issues/18474 for context.

@denniseffing I'll take a look at it through the week.

I created a new PR (#903) because we couldn't wait for the change.

I created a new PR (#903) because we couldn't wait for the change.

This PR got superseded by #903 which very much looks like we might be able to merge it soon.
Thanks everybody, although id didn't got merged it helped move this feature forward.

This PR got superseded by #903 which very much looks like we might be able to merge it soon.
Thanks everybody, although id didn't got merged it helped move this feature forward.


Add param distance/duration quota & allow to drop destinations

Fix #000 (<-- Add issue number here)
IMPORTANT: Each pull request (PR) should resolve an issue. Please create the issue if it hasn't been created, unless you're fixing typos or making minor changes to documentation or infrastructure. In that case, just leave the issue number as #000.
Details
If the PR title isn't enough to describe your PR, please provide details here.
Checks
Make sure these boxes are checked before your PR is ready to be reviewed and merged. Thanks!

[x] PR is concisely and descriptively titled 📑 and links to the original issue 🔗.
[x] Code is in uniquely-named feature branch and has no merge conflicts 📁.
[x] Tests pass -- look for a green checkbox ✔ a few minutes after opening your PR.
[ ] @mentions of the person or team responsible for reviewing proposed changes. By default, use @BadwaterBay/reviewers.
[ ] (Optional) Screenshots/GIFs are attached 📎 in case of UI updates.

We're happy to help you get this ready. Don't be afraid to ask for help, and don't be discouraged if your tests fail at first.

Thank you for contributing to this project!

Codecov Report

Merging #95 into master will increase coverage by 2.67%.
The diff coverage is 81.01%.

@@            Coverage Diff             @@
##           master      #95      +/-   ##
==========================================
+ Coverage   59.44%   62.11%   +2.67%
==========================================
  Files           5        6       +1
  Lines         143      161      +18
  Branches        8       10       +2
==========================================
+ Hits           85      100      +15
- Misses         56       59       +3
  Partials        2        2

Impacted Files
Coverage Δ

src/core/get_distance_matrix.py
0.00% <0.00%> (ø)

src/core/find_route.py
83.56% <83.56%> (ø)

src/core/helpers/helper.py
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

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


"WARNING: failed to load clp.lua\n" should print to stderr

Thanks, should be fixed by 1318913ff4e5084a30d33d22f0c87f61dabf59de


Set TestFX version to 4.0.5-alpha to prevent breaking changes to GUI tests

Coverage remained the same at 90.405% when pulling bddc876a41d57230df4cf6b321ecdead0db666fe on hotfix-guitests into 8cb5e139d6ad1af321b60c2d8d1afc1ec54d2e28 on master.


Add syntax highlighting to readme

Add JavaScript syntax highlighting and fix one code block's placement. Very small commit, just helps readability. Hope this is helpful.

Thanks for this


Split auto created sessions and manually saved sessions into different search-lenses

Hi @danielyrovas, could you elaborate on what you mean by this?

My idea was that if you invoke the SaveSession command to save a session manually then you could access that in a seperate list with search lens than the the sessions that are saved on quit. I think if I use auto session for a while the number of sessions that I have saved will continually accrue and could make it more difficult than it needs to be to find a specific session. If there is 10 or so sessions that you want to have to come back to regularly, you could save them manually and then have them display in a separate list.

@danielyrovas I see. Maybe there's just a better way of achieving the same result. Could look into implementing frecency of some kind in the picker, so you'd get those 10 sessions you use the most ranked higher. Food for thought.

yes that would likely be a cleaner way to provide the same functionality. A 'pin to top' method could also work.
Thanks for the awesome plugin, I'm liking it a lot atm.


Bug with calculation causes all tokens to be incorrect on some devices, for Google Authenticator 2.49

I encountered always wrong verification code on my phone, e.g. 396731, correct is 396709.
The correct code is found by using php's Google Authenticator's source code.  The wrong code is 22396709 mod 999999, instead of mod 1000000  (always reproducable).  The tested phone is Lenovo S930, android 4.4.2.
Not sure if the bug is related to the logic in this project. Thus reported here for reference.
Some corresponding parameter of the final code 396709
hmac output:  287f5c148333f125aa4507950eca8155bf25556e
final 32-bit integer before mod 1000000:  22396709

Sounds like it's the same as https://github.com/google/google-authenticator-android/pull/4 .
Status of that is that it's fixed in this open source version, and will be in the next closed source release. When that will be is out of my domain.


Last Week in Pony - July 3, 2022

Audio from the June 28th Pony Development sync is available: https://sync-recordings.ponylang.io/r/2022_06_28.m4a

Version 0.51.1 of ponylang/ponyc has been released.
See the release notes for more details.

I've started an external package that offers enhanced terminal application development APIs. This offers compatibility with the term package's ANSI, ANSITerm and ANSINotify types while defining a core API that offers more advanced capability.
https://github.com/nogginly/termax.pony
So far I have been testing on macOS using Terminal.app. I would love to get some help running the example apps on other platforms (Linux, Windows). In particular, there are two examples that I would like to get feedback on:

examples/formatting which can be used to test the various text formatting options supported.
examples/mousing which can be used to test mouse input handling