llama3 does not return pure json

Testing gpt-researcher with llama3, i found that 3 times out of 4 llama3 will respond with json + some verbiage to prompt in generate_search_queries_prompt.
Not sure it is worth changing the prompt for sake of llama3 alone, but for documentation purposes here is the updated prompt that seems to work every time
before:
f'You must respond with a list of strings in the following format: ["query 1", "query 2", "query 3"].'

after
f'Your response must include list of the query strings in json format and nothing else. For example: ["query 1", "query 2", "query 3"]'

Hey @barsuna. I was searching for how to use llama with gpt researcher and stumbled upon this post. If possible, could you tell me how to get gpt researcher to work with llama 3?

@Dilip-17 there was same question on another issue, i added some pointers there
https://github.com/assafelovic/gpt-researcher/issues/520
the challenge is mostly not how to run, but having the gpu memory necessary to run llama3 - even the borderline usable (imo, opinions are divided on this) 4-bit quantized 70b model takes about ~43GB, i'd recommend Q6 which is close to 60GB

Hey working with different LLMs (other than the default OpenAI) required extra manual tweaking. Would love to learn from your experience if you find ways to make the code more generic!

To its credit, llama3 worked pretty much out of box with gpt-researcher (the only tweak needed was the prompt change above). It seems it is possible to stretch the context window to 16k without tuning (though i've done very limited testing of that).
So far progress with llama3 was difficult for things requiring function calling and in-prompt memory - autonomous agents, with single or 1 by 1 prompting agents things seem to be better.
Of course the main challenge remains the quality of reports, i'm currently trying to compare llama3 vs gpt4, it seems both are challenged somewhat and my belief is the likely direction to solve this is to balance automation/augmentation - let user do more if they wished.
Havent measured quality of embeddings and its impact on quality of report much either.

Great thank you for the feedback @barsuna ! Closing for now but feel free to open new threads if needed


discovery: fix silent remote orphan announcement proof rejection

In case of the situation when we receive remote announcement with channel id which pointing out to unknown channel the announcement have been silently rejected. Now such announcement will
be added to the waiting proof map.
Such solution has a serious drawback - by adding the announcement proof without information about channel itself (that this announcement have been received from the node eligible to sending it), we allow overwriting the waiting proof map by Eve node.

Coverage increased (+0.1%) to 74.081% when pulling 464c3143b05c98e7b0e05879bff2ee8faf27fccc on AndrewSamokhvalov:fix_remote_orphan_rejection into f9c78b7c2bf2801bfcd7dd4cf02c62565743c72a on lightningnetwork:master.


Enable Role Based Access Control based on Azure AD

Fixes #103
Changes proposed in the pull request:
When both var.enable_role_based_access_control & var.rbac_aad_managed are set to true, azure_rbac_enabled is set true as well
Its only possible to use azure_rbac_enabled when Azure Active Directory integration is Managed, meaning that Azure will create/manage the Service Principal used for integration.

Any progress about this?

Hello. Any news on this? I'm missing this feature in the module and it looks like the proposed solution could be very simple.

Hello @nitinnbisht , thanks for you pr, this feature has been supported in #199 so I'm closing this pr now.


docs/gettingstarted: Update AKS instructions

Refs: #13627

Thanks for the tips @joestringer. I've fixed the indentation and visually verified that the docs render correctly.


WIP: Applicative support

This adds support for the new applicative in F# 5.0.
The result CE will remain mostly the same. It will still return a Result<'a,'err>. and! only gives users some performance optimization.
The new validation CE will be a much bigger difference. It's return type will be Result<'a, 'err list>.  When given the example:
            let expected = Error ["Error 1"; "Error 2"]
            let actual = validation {
                let! a = Ok 3
                // and! b = Ok 2
                and! b = (Ok 2 : Result<_,string>)
                and! c = Error "Error 1"
                and! d = Error "Error 2"
                return a + b - c - d
            }
            Expect.equal actual expected "Should be Error"

using and! in this case will run each operation parallel resulting in a list of 'err to be given.  Currently you do have to use some type annotations to tell the compiler how to handle the overload resolution.  @cartermp, is there something I can do about my implementation so consumers don't have to specify that type annotation?
CE's to finish:

[x] Result CE
[x] Validation CE
[ ] AsyncResult CE
[ ] AsyncValidation CE
[ ] TaskResult CE
[ ] TaskValidation CE
[ ] JobResult CE
[ ] JobValidation CE

@TheAngryByrd this program runs without an annotation in the CE:
// Learn more about F# at http://fsharp.org
open System

type Validation<'a,'err> = Result<'a, 'err list>

[<RequireQualifiedAccess>]
module Validation =

  let ok x : Validation<_,_> = Ok x
  let error e : Validation<_,_> = List.singleton e |> Error

  let ofResult x =
    Result.mapError List.singleton x

  let apply f x =
    match f, x with
    | Ok f, Ok x -> Ok (f x)
    | Error errs, Ok _ | Ok _, Error errs -> Error errs
    | Error errs1, Error errs2 -> Error  (errs1 @ errs2)

  let retn x = ofResult (Ok x)

  let map2 f x y =
    apply (apply (retn f) x) y

  let map3 f x y z =
    apply (map2 f x y) z

  let bind (f : 'a -> Validation<'b, 'err>) (x : Validation<'a,'err>) : Validation<_,_>=
    Result.bind f x

  let zip x1 x2 : Validation<_,_> =
    match x1,x2 with
    | Ok x1res, Ok x2res -> Ok (x1res, x2res)
    | Error e, Ok _ -> Error e
    | Ok _, Error e -> Error e
    | Error e1, Error e2 -> Error (e1 @ e2)

[<AutoOpen>]
module ValidationCE =
    type ValidationBuilder() =
        member __.Return (value: 'T) =
            Validation.ok value

        member _.BindReturn(x: Validation<'T,'U>, f) : Validation<_,_> = Result.map f x

        member __.Bind
            (result: Validation<'T, 'TError>, binder: 'T -> Validation<'U, 'TError>)
            : Validation<'U, 'TError> =
            Validation.bind binder result

        member _.MergeSources(t1: Validation<'T,'U>, t2: Validation<'T1,'U>) : Validation<_,_> = Validation.zip t1 t2

    let validation = ValidationBuilder()

[<AutoOpen>]
module ValidationCEExtensions =

  // Having Choice<_> members as extensions gives them lower priority in
  // overload resolution and allows skipping more type annotations.
    type ValidationBuilder with
        member __.Bind
            (result: Result<'T, 'TError>, binder: 'T -> Validation<'U, 'TError>)
            : Validation<'U, 'TError> =
            result
            |> Validation.ofResult
            |> Validation.bind binder

        member _.BindReturn(x: Result<'T,'U>, f) : Validation<_,_> = x |> Validation.ofResult |> Result.map f

        member _.MergeSources(t1: Validation<'T,'U>, t2: Result<'T1,'U>) : Validation<_,_> = Validation.zip t1 (Result.mapError List.singleton t2)
        member _.MergeSources(t1: Result<'T,'U>, t2: Validation<'T1,'U>) : Validation<_,_> = Validation.zip (Result.mapError List.singleton t1) t2
        member _.MergeSources(t1: Result<'T,'U>, t2: Result<'T1,'U>) : Validation<_,_> = Validation.zip (Result.mapError List.singleton t1) (Result.mapError List.singleton t2)

let f () =
    let expected = Error ["Error 1"; "Error 2"]
    let actual = validation {
        let! a = Ok 3
        and! b = Ok 2
        and! c = Error "Error 1"
        and! d = Error "Error 2"
        return a + b - c - d
    }
    printfn "%A %A" expected actual
    ()

[<EntryPoint>]
let main argv =
    f()
    0 // return an integer exit code

However I have also noticed that in some cases a type annotation is needed. Thanks for testing this out though, @dsyme is looking for more folks to give it a shot

Ok I can also get it to compile just fine now. Please call an exorcist.

Thanks, @TheAngryByrd for taking this up. Due to some personal reasons, I am not in a position to actively work on this library.
I am checking the email notifications periodically, and I will jump in if required.
Thanks again :)

@demystifyfp No worries! I use this extensively at work so I’m fine with maintaining it fully. Would you be willing to add me as an owner of the nuget package so I can push whenever PRs get merged?

@cartermp I am once again asking for your support. I hope summoning you scares the code into behaving again.
I'm trying to add overloads for BindResult and MergeSources for the AsyncResult CE, however I cannot get both of these test cases to build.
     testCaseAsync "Happy Path AsyncResult" <| async {
            let! actual = asyncResult {
                let! a = AsyncResult.retn 3
                and! b = AsyncResult.retn 2
                and! c = AsyncResult.retn 1
                return a + b - c
            }
            Expect.equal actual (Ok 4) "Should be ok"
        }

     testCaseAsync "Happy Path Async" <| async {
            let! actual = asyncResult {
                let! a = Async.singleton 3
                and! b = Async.singleton 2
                and! c = Async.singleton 1
                return a + b - c
            }
            Expect.equal actual (Ok 4) "Should be ok"
        }

This is the compiler error I end up getting for the second test:
/Users/jimmybyrd/Documents/GitHub/FsToolkit.ErrorHandling/tests/FsToolkit.ErrorHandling.Tests/AsyncResultCE.fs(326,26): error FS0041: A unique overload for method 'MergeSources' could not be determined based on type information prior to this program point. A type annotation may be needed.Known types of arguments: Async<int> * Async<Result<(int * int),'a>>Candidates: - member AsyncResultBuilder.MergeSources : t1:Async<'T> * t2:Async<'T1> -> Async<Result<('T * 'T1),'a2>> - member AsyncResultBuilder.MergeSources : t1:Async<'T> * t2:Async<Result<'T1,'U>> -> Async<Result<('T * 'T1),'U>> [/Users/jimmybyrd/Documents/GitHub/FsToolkit.ErrorHandling/tests/FsToolkit.ErrorHandling.Tests/FsToolkit.ErrorHandling.Tests.fsproj]
/Users/jimmybyrd/Documents/GitHub/FsToolkit.ErrorHandling/tests/FsToolkit.ErrorHandling.Tests/AsyncResultCE.fs(326,17): error FS0041: A unique overload for method 'BindReturn' could not be determined based on type information prior to this program point. A type annotation may be needed.Known types of arguments: 'a * ( ^b * ( ^c *  ^e) ->  ^f) when ( ^b or  ^c) : (static member ( + ) :  ^b *  ^c ->  ^d) and ( ^d or  ^e) : (static member ( - ) :  ^d *  ^e ->  ^f)Candidates: - member AsyncResultBuilder.BindReturn : x:Async<'T> * f:('T -> 'a1) -> Async<Result<'a1,'a2>> - member AsyncResultBuilder.BindReturn : x:Async<Choice<'T,'U>> * f:('T -> 'a2) -> Async<Result<'a2,'U>> - member AsyncResultBuilder.BindReturn : x:Async<Result<'T,'U>> * f:('T -> 'd) -> Async<Result<'d,'U>> - member AsyncResultBuilder.BindReturn : x:Choice<'T,'U> * f:('T -> 'a2) -> Async<Result<'a2,'U>> - member AsyncResultBuilder.BindReturn : x:Result<'T,'U> * f:('T -> 'a2) -> Async<Result<'a2,'U>> - member AsyncResultBuilder.BindReturn : x:Task<'T> * f:('T -> 'a1) -> Async<Result<'a1,'a2>> - member AsyncResultBuilder.BindReturn : x:Task<Result<'T,'U>> * f:('T -> 'a2) -> Async<Result<'a2,'U>> [/Users/jimmybyrd/Documents/GitHub/FsToolkit.ErrorHandling/tests/FsToolkit.ErrorHandling.Tests/FsToolkit.ErrorHandling.Tests.fsproj]

We are using the hack of using extension members which has a lower priority when binding, so maybe this just isn't possible.
You can pull the latest at https://github.com/demystifyfp/FsToolkit.ErrorHandling/pull/75/commits/d9e960e119943b49f305c33c43ad6d186edec723 which has this issue. You can't use FAKE to build until https://github.com/fsharp/FAKE/pull/2499 is merged, so just cd tests/FsToolkit.ErrorHandling.Tests and run dotnet watch run.

Thanks, @TheAngryByrd. I have added you as an owner of the packages in NuGet.

Given the amount of work to do for AsyncValidation/TaskValidation/JobValidation and the lack of use cases around those currently, I'm opting to not implementing them for now.

I'm having issues with travis: https://github.com/dotnet/core/issues/4542#issuecomment-627619266
I'm going to accept this PR since I don't want to fight with it.  I'll open a separate issue to get it fixed.


[Menu] Fix RegExp error on keydown

Menu throws SyntaxError from onKeyDown handler.
To reproduce, go to http://www.material-ui.com/#/components/dropdown-menu, open a menu, hit "\" character on keyboard, see SyntaxError in console.
Caused by constructing a RegExp to find matching menu items. Characters are not filtered at all, allowing invalid RegExp.
This PR uses simple string functions instead of RegExp and adds a test case for this particular key (\).

[x] PR has tests / docs demo, and is linted.
[x] Commit and PR titles begin with [ComponentName], and are in imperative form: "[Component] Fix leaky abstraction".
[x] Description explains the issue / use-case resolved, and auto-closes the related issue(s) (http://tr.im/vFqem).

@liamcmitchell Good catch. Thanks


License - What type is it?

Hi,
I've just noticed the lack of a license file, which OSS license does the project use?
Cheers,
Tom

It's Apache licensed, as specified in the LICENSE file.


[csolution] how to specify debug interface ?

Hi,
I do not think it has been specified yet but is it possible to specify the debug interface to be used with the csolution ?
My use-case:
csolution-->cprj-->uvision : here I need to select ST-Link
I would like that the ST-Link choice is made when defining application project.
I agree this is not build.

Hi Fred,
as much as I see the value in making the above flow completely "automatic", at this point we focus on build and pretty much ignore flash programming and debug, however the Specification of CMSIS-Pack has provisions to specify the mounted debug adapter on a board, which seems like a useful and obvious default selection. During import of a project this information shall be used by an IDE to configure the right debug adapter.

@fred-r


make self-hosted docs slightly more readable

I have successfully set this amazing plugin up in my server. I am still very new to the self hosting world so I did spend some time trying to figure out the documentation for this.
I ended up copying the wrong commands because of the code block!
I know most experienced people would recognize it as the code output but for ease of copying the code block and readability I propose this change in the docs, including removal of $ :).

Thank you for your fixing up! I think these are perfect improvements!
I am so happy to merge this!
I appreciate your contribution!


ATS as Forward Proxy - For an Origin server with two IPs, and when the first IP is down, requests fail with 502 error

ATS as a Forward Proxy - For a url whose domain name resolves to two IPs and the first IP is down - ATS fails with a 502 error.
ATS never falls back to the second IP, which incidentally is up. Tried the same with Squid Proxy and it forwarded the request to the working IP.
Config:
proxy.config.http.connect_attempts_rr_retries 3
Is there some configuration that I need to set to ensure that if one of the IPs is not working, ATS should fallback to the working IP.

same as https://github.com/apache/trafficserver/issues/4214 ?

@hankai17 - exact same issue - but there doesn't seem to be any resolution. Also, #4214 is closed. Even if one of the IP is working, ATS throws a 502 error. Is there any configuration
I am missing or is it by design?

Have you tried adjusting this? That might cause ATS to move on before existing all upstream retries.

@SolidWallOfCode - proxy.config.http.connect_attempts_max_retries is unchanged, and is defaulted to 3. While there are only two IPs for the OS, it should have moved to the second IP after the first try, it did not.
proxy.config.http.connect_attempts_rr_retries defaults to 3. I will limit this parameter to 1 and see if that helps.
Thanks for the suggestion.

@vksiva Did you try setting proxy.config.http.connect_attempts_rr_retries to a lower value?

Refer https://github.com/apache/trafficserver/pull/4810. May resolved this.

@vksiva Please reopen if this is still and issue.


Fix undefined http_status causing the game to crash

When the HTTP event returns a status of 1, it indicates that the content is still being downloaded, according to the GameMaker manual. Therefore the http_status variable can sometimes be undefined, which may cause the game to crash.

Thank you very much!


dotnet publish (3.0)

[Enter feedback here]
.NET 3.0 breaking change - I had to change after upgrade from 2.2 to 3.0:
dotnet publish --configuration=Release --no-restore --output=../../dist src/Website
to
dotnet publish --configuration=Release --no-restore --output=dist src/Website

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

ID: 76f39eec-9dd1-6167-56ec-47b26e0b5063
Version Independent ID: f9f2b0f3-01eb-cf58-8767-194fe0fc0e44
Content: Migrate from ASP.NET Core 2.2 to 3.0 Preview
Content Source: aspnetcore/migration/22-to-30.md
Product: aspnet-core
Technology: aspnetcore-migration
GitHub Login: @Rick-Anderson
Microsoft Alias: riande

@pranavkm  who can review dotnet publish --configuration=Release --no-restore --output=dist src/Website

Solution structure is:
/src/Website/Website.csproj
App.sln

This issue was moved to aspnet/AspNetCore#14262


build: Move images to be based on Amazon Linux 2023

We get a few advantages with moving to Amazon Linux 2023, the most notable being that we get to use more packages provided by the OS rather than downloading some. For example, we can use the node and npm binaries from the RPM packages in AL2023.
We also take the opportunity to move to pulling base images from ECR rather than DockerHub.
Using curl rather than wget also means we don't need to add wget to the list of packages we're installing in the container, thus keeping things nice and minimal.

@stewartsmith really appreciate this pull request and the extra work you did here. I'm taking a look over the changes mainly to make sure this doesn't do anything funky to the OSS approval we received to publish these images.

Looks good, test failure is a flake. Thanks for this!


How to remove space auto complete activation

I have an annoying bug with tailwindcss, when I try to type a new class in className, it gives me an auto suggestion of unrelated things and ends up breaking my auto complete. For example, when I enter insert mode and press space to write the class, it gives me this unrelated suggestion:

and when I try to type the class I actually want, it doesn't capture the first character: (ignore buffer and luasnip)

here if i type j twice it actually give me the suggestion i want

I believe that if I find a way to remove the space from triggering the auto complete, it will work normally.

Show your nvim config. @Caixetadev

Show your nvim config. @Caixetadev

my config nvim


Fix to #3889 - Query: [Nav Prop Translation] SelectMany generates wrong SQL

Issue was that we were applying our standard navigation expansion logic to SelectMany queries (e.g. customers.SelectMany(c => c.Orders)
This resulted in queries like:
SELECT * FROM Orders as o
which is correct for the basic case, however gives invalid SQL for slightly complex scenarios, e.g. with filter:
customers.Where(c => c.CustomerPK == 1).SelectMany(c => c.Orders)
SELECT * FROM Orders as o
WHERE o.CustomerPK = 1
Fix is to always translate the navigations inside SelectMany into inner joins instead.
This will produce queries like:
SELECT * FROM Customers as c
JOIN Orders as o ON c.Id = o.CustomerFK
WHERE c.CustomerPK = 1
This will have to be modified when Many-Many relationships are introduces, as some of the navigations will have to be translated into CROSS JOINs
Also added template visitor for transforming QueryModel as we now have two instances where this functionality is used.

@anpete

@anpete new version up

:shipit:


Is it possible to use this library omitting features accessing iPhone camera because I do not use camera feature in my app and would like to remove NSCameraUsageDescription in info.plist.

Hello.
Is it possible to use this library omitting features accessing iPhone camera because I do not use camera feature in my app (only displaying barcode on UIImageView) and would like to remove NSCameraUsageDescription in info.plist.
In fact my app got a following mail from Apple and failed to submit:
ITMS-90683: Missing Purpose String in Info.plist

Your app’s code references one or more APIs that access sensitive user data.
The app’s Info.plist file should contain a NSCameraUsageDescription key with a user-facing purpose string explaining clearly and completely why your app needs the data.

Thank you.

For now, I will fill in NSCameraUsageDescription with some phrases and submit the app to App Store Connect because it might pass App Store Review. If it is denied, I will manually remove some files in ZxingObjC manually to remove NSCameraUsageDescription. Thank you.


[docs] Replace videos on S3 with ones on YouTube

What does this PR do?
Part of #7666.
TODO

[ ] Upload videos on YouTube (Taylor is working on this.)
[ ] Replace the current s3 links with youtube ones

List of videos

List of videos that are missing on YouTube
https://pl-bolts-doc-images.s3.us-east-2.amazonaws.com/pl_docs/trainer_flags/accumulate_grad_batches.mp4
https://pl-bolts-doc-images.s3.us-east-2.amazonaws.com/pl_docs/trainer_flags/amp_backend.mp4
https://pl-bolts-doc-images.s3.us-east-2.amazonaws.com/pl_docs/trainer_flags/amp_level.mp4
https://pl-bolts-doc-images.s3.us-east-2.amazonaws.com/pl_docs/trainer_flags/auto_lr_find.mp4
https://pl-bolts-doc-images.s3.us-east-2.amazonaws.com/pl_docs/trainer_flags/auto_scale_batch_size.mp4
https://pl-bolts-doc-images.s3.us-east-2.amazonaws.com/pl_docs/trainer_flags/auto_select_gpus.mp4
https://pl-bolts-doc-images.s3.us-east-2.amazonaws.com/pl_docs/trainer_flags/benchmark.mp4
https://pl-bolts-doc-images.s3.us-east-2.amazonaws.com/pl_docs/trainer_flags/callbacks.mp4
https://pl-bolts-doc-images.s3.us-east-2.amazonaws.com/pl_docs/trainer_flags/checkpoint_callback.mp4
https://pl-bolts-doc-images.s3.us-east-2.amazonaws.com/pl_docs/trainer_flags/check_val_every_n_epoch.mp4
https://pl-bolts-doc-images.s3.us-east-2.amazonaws.com/pl_docs/trainer_flags/cluster_environment.mp4
https://pl-bolts-doc-images.s3.us-east-2.amazonaws.com/pl_docs/trainer_flags/default_root_dir.mp4
https://pl-bolts-doc-images.s3.us-east-2.amazonaws.com/pl_docs/trainer_flags/deterministic.mp4
https://pl-bolts-doc-images.s3.us-east-2.amazonaws.com/pl_docs/trainer_flags/distributed_backend.mp4
https://pl-bolts-doc-images.s3.us-east-2.amazonaws.com/pl_docs/trainer_flags/fast_dev_run.mp4
https://pl-bolts-doc-images.s3.us-east-2.amazonaws.com/pl_docs/trainer_flags/flush_logs_every_n_steps.mp4
https://pl-bolts-doc-images.s3.us-east-2.amazonaws.com/pl_docs/trainer_flags/gpus.mp4
https://pl-bolts-doc-images.s3.us-east-2.amazonaws.com/pl_docs/trainer_flags/gradient_clip_val.mp4
https://pl-bolts-doc-images.s3.us-east-2.amazonaws.com/pl_docs/trainer_flags/limit_batches.mp4
https://pl-bolts-doc-images.s3.us-east-2.amazonaws.com/pl_docs/trainer_flags/log_every_n_steps.mp4
https://pl-bolts-doc-images.s3.us-east-2.amazonaws.com/pl_docs/trainer_flags/logger.mp4
https://pl-bolts-doc-images.s3.us-east-2.amazonaws.com/pl_docs/trainer_flags/log_gpu_memory.mp4
https://pl-bolts-doc-images.s3.us-east-2.amazonaws.com/pl_docs/trainer_flags/min_max_epochs.mp4
https://pl-bolts-doc-images.s3.us-east-2.amazonaws.com/pl_docs/trainer_flags/min_max_steps.mp4
https://pl-bolts-doc-images.s3.us-east-2.amazonaws.com/pl_docs/trainer_flags/num_nodes.mp4
https://pl-bolts-doc-images.s3.us-east-2.amazonaws.com/pl_docs/trainer_flags/num_processes.mp4
https://pl-bolts-doc-images.s3.us-east-2.amazonaws.com/pl_docs/trainer_flags/num_sanity_val_steps.mp4
https://pl-bolts-doc-images.s3.us-east-2.amazonaws.com/pl_docs/trainer_flags/overfit_batches.mp4
https://pl-bolts-doc-images.s3.us-east-2.amazonaws.com/pl_docs/trainer_flags/precision.mp4
https://pl-bolts-doc-images.s3.us-east-2.amazonaws.com/pl_docs/trainer_flags/prepare_data_per_node.mp4
https://pl-bolts-doc-images.s3.us-east-2.amazonaws.com/pl_docs/trainer_flags/process_position.mp4
https://pl-bolts-doc-images.s3.us-east-2.amazonaws.com/pl_docs/trainer_flags/profiler.mp4
https://pl-bolts-doc-images.s3.us-east-2.amazonaws.com/pl_docs/trainer_flags/progress_bar_refresh_rate.mp4
https://pl-bolts-doc-images.s3.us-east-2.amazonaws.com/pl_docs/trainer_flags/reload_dataloaders_every_epoch.mp4
https://pl-bolts-doc-images.s3.us-east-2.amazonaws.com/pl_docs/trainer_flags/replace_sampler_ddp.mp4
https://pl-bolts-doc-images.s3.us-east-2.amazonaws.com/pl_docs/trainer_flags/resume_from_checkpoint.mp4
https://pl-bolts-doc-images.s3.us-east-2.amazonaws.com/pl_docs/trainer_flags/sync_batchnorm.mp4
https://pl-bolts-doc-images.s3.us-east-2.amazonaws.com/pl_docs/trainer_flags/tpu_cores.mp4
https://pl-bolts-doc-images.s3.us-east-2.amazonaws.com/pl_docs/trainer_flags/track_grad_norm.mp4
https://pl-bolts-doc-images.s3.us-east-2.amazonaws.com/pl_docs/trainer_flags/truncated_bptt_steps.mp4
https://pl-bolts-doc-images.s3.us-east-2.amazonaws.com/pl_docs/trainer_flags/val_check_interval.mp4
https://pl-bolts-doc-images.s3.us-east-2.amazonaws.com/pl_docs/trainer_flags/weights_save_path.mp4
https://pl-bolts-doc-images.s3.us-east-2.amazonaws.com/pl_docs/trainer_flags/weights_summary.mp4
https://pl-bolts-doc-images.s3.us-east-2.amazonaws.com/pl_docs/yt/Trainer+flags+19-+early+stopping_1.mp4
https://pl-bolts-doc-images.s3.us-east-2.amazonaws.com/pl_docs/yt/Trainer+flags+4-+multi+node+training_3.mp4
https://pl-bolts-doc-images.s3.us-east-2.amazonaws.com/pl_docs/yt/Trainer+flags+7-+debugging_1.mp4
https://pl-bolts-doc-images.s3.us-east-2.amazonaws.com/pl_docs/yt/Trainer+flags+9+-+precision_1.mp4
https://pl-bolts-doc-images.s3.us-east-2.amazonaws.com/pl_docs/pl_docs_animation_final.m4v
https://pl-bolts-doc-images.s3.us-east-2.amazonaws.com/pl_docs/pt_trainer_mov.m4v
https://pl-bolts-doc-images.s3.us-east-2.amazonaws.com/pl_docs/pl_mod_vid.m4v
https://pl-bolts-doc-images.s3.us-east-2.amazonaws.com/pl_docs/pt_callbacks_mov.m4v
https://pl-bolts-doc-images.s3.us-east-2.amazonaws.com/pl_docs/pt_dm_vid.m4v

Before submitting

[x] Was this discussed/approved via a GitHub issue? (not for typos and docs)
[x] Did you read the contributor guideline, Pull Request section?
[x] Did you make sure your PR does only one thing, instead of bundling different changes together?
[ ] Did you make sure to update the documentation with your changes? (if necessary)
[n/a] Did you write any new necessary tests? (not for typos and docs)
[ ] Did you verify new and existing tests pass locally with your changes?
[ ] Did you update the CHANGELOG? (not for typos, docs, test updates, or internal minor changes/refactorings)

PR review
Anyone in the community is free to review the PR once the tests have passed.
Before you start reviewing make sure you have read Review guidelines. In short, see the following bullet-list:

[ ] Is this pull request ready for review? (if not, please submit in draft mode)
[ ] Check that all items from Before submitting are resolved
[ ] Make sure the title is self-explanatory and the description concisely explains the PR
[ ] Add labels and milestones (and optionally projects) to the PR so it can be classified

Did you have fun?
Make sure you had fun coding 🙃

Thanks for starting this @akihironitta! Most videos are in pytorch-lightning/docs/source/common/trainer.rst, but there are some more around I think...

@edenlightning Thanks for checking it out! I thought videos are only in mp4, but some are in m4v, so I've updated the list :]

@akihironitta Should I reopen this?


RID Objects are always "Invalid RID"

Godot version:
Beta 1
OS/device including version:
Windows 10 1903
Issue description:
No matter which function i use, the RID object always gives Invalid RID.
Steps to reproduce:
Try any of these:
var ps_rid = get_viewport().get_world().get_space() # ps_rid is Invalid RID
var vs_rid = get_viewport().get_world().get_canvas() # vs_rid is Invalid RID
var rigidBody = RigidBody2D.new()
var rb = rigidBody.instance()
add_child(rb)
print(rb.get_rid()) # prints Invalid RID

The only mention of Invalid RID in the code is in the Inspector (and mostly makes sense, as the RID is valid in the debugged game, not in the editor):
https://github.com/godotengine/godot/blob/37b230fe3af3c7e20a6f938d03b61a79b117d354/editor/editor_properties.cpp#L2060-L2068
I don't think the print statement can result in Invalid RID, it should print [RID] from what I can see.

Alright, that makes sense, BUT, trying to call any kind of method on that RID object just gives me:

The image is from:
	var vs_rid = get_viewport().get_world_2d().get_canvas()
	var ci_rid = vs_rid.canvas_item_create()

I havent been able to call any kind of method on any returned RID object.

Alright, that makes sense, BUT, trying to call any kind of method on that RID object just gives me:

The image is from:
	var vs_rid = get_viewport().get_world_2d().get_canvas()
	var ci_rid = vs_rid.canvas_item_create()

I havent been able to call any kind of method on any returned RID object.

The only mention of Invalid RID in the code is in the Inspector (and mostly makes sense, as the RID is valid in the debugged game, not in the editor):
https://github.com/godotengine/godot/blob/37b230fe3af3c7e20a6f938d03b61a79b117d354/editor/editor_properties.cpp#L2060-L2068
I don't think the print statement can result in Invalid RID, it should print [RID] from what I can see.

NVM, i think my confuzion was just with the Invalid RID. using VisualServer.canvas_item_create() with the visual server singleton instead of the RID seems to work

Looking back, the problem started when i was following the docs  in an attempt to replicate:
func _body_moved(state, index):
    # Created your own canvas item, use it here.
    VisualServer.canvas_item_set_transform(canvas_item, state.transform)

func _ready():
    # Create the body.
    var body = Physics2DServer.body_create()
    Physics2DServer.body_set_mode(body, Physics2DServer.BODY_MODE_RIGID)
    # Add a shape.
    var shape = RectangleShape2D.new()
    shape.extents = Vector2(10, 10)
    # Make sure to keep the shape reference!
    Physics2DServer.body_add_shape(body, shape)
    # Set space, so it collides in the same space as current scene.
    Physics2DServer.body_set_space(body, get_world_2d().space)
    # Move initial position.
    Physics2DServer.body_set_state(body, Physics2DServer.BODY_STATE_TRANSFORM, Transform2D(0, Vector2(10, 20)))
    # Add the transform callback, when body moves
    # The last parameter is optional, can be used as index
    # if you have many bodies and a single callback.
    Physics2DServer.body_set_force_integration_callback(body, self, "_body_moved", 0)

 Physics2DServer.body_add_shape(body, shape) That like is wrong. shape variable is not an RID. You have to get the RID first using get_rid(). But then i saw the Invalid RID in the debugger and i assumed that was the error :dizzy_face:.
So to sum up, first there needs to be a fix for the Invalid RID message and second for the docs. I will make the changes myself later with a PR. Thanks @bojidar-bg :punch:

Physics2DServer.body_add_shape(body, shape) That line is wrong. shape variable is not an RID. You have to get the RID first using get_rid()

Is Godot actually converting into a RID if the parameter inherits Resource?

Physics2DServer.body_add_shape(body, shape) That line is wrong. shape variable is not an RID. You have to get the RID first using get_rid()

Isn't Godot actually converting into a RID if the parameter inherits Resource? That may add to the confusion but I think that's what happens.

@Zylann Yeah, i think you're right 😵😵. Now I'm lost.

This might be just restating what has been already said, but I see an identical behaviour on the following:
var D:DynamicFont
var R:RID
var i:int
....
R = $Sprite.get_rid()
i = R.get_id()
print(R)
print(i)
....
D.size = 24
D.draw(R, Vector2(0.0, 0.0), "abc123")
In the output window I get:
[RID]
151
and get the following error from DynamicFont.draw()

Furthermore, if you somehow expose RID property, it won't be saved properly in a tscn, which will lead to parse error on opening. I've only stumbled upon this while accidentally exposing an RID as a property using C++ with some combination of property hints and usage. Exporting RID via GDScript and saving works alright.

I'm trying to use Physics2DServer.area_set_area_monitor_callback and with breakpoint inside callback function I always get Invalid RID in inspector view.

Sample project in Godot 3.2.2
bullet-shower.zip

Creating and working with RIDs directly using VisualServer works, so that Invalid RID message in the inspector is indeed misleading.


update nen_to_axioms.py

update create_restriction_node
update create_cardinality_node
update spo -- class/data_property/object_property

Does this result in functional code? I see that the g has been added as a parameter to a function, but I don't see any particular changes to the signature of the method?


Initial Version

Beep Beep

@simurai does CSS has a transition that allows for smooth increase in height when an element is appended to the container?

does CSS has a transition that allows for smooth increase in height when an element is appended to the container?

Not in an easy way if the height is using auto. It might can be hacked with animating min-height. Is for something like the gif in https://github.com/atom/notifications/blob/master/README.md.

Thew new Message class supports HTMLElement caching, means we will no longer be giving non-used messages as garbage and creating them when we need new ones. DOM operations are cpu-intensive, therefore we'd be keeping them at disposal and using them when necessary. Currently the max number is a hard-coded 250, but it won't be much complicated to get average number of insertions of the last ten times and use it as max

Iteartion #1 is complete :heart:


Carthage and Swift 3 Support

It would be great if you could provide Carthage support. And also releases for swift 2.3 and swift 3. Thanks in advance.

Just updated to Swift 3 syntax, have a try. Will investigate how to do the Carthage support later.

if linkedin app exist in my device then code just opens app. Does not ask for verification and does not come back.

did you follow up the linkedin sdk setup in this link? https://developer.linkedin.com/docs/ios-sdk

@tonyli508 yes everything seems done, looks like minor mistake is done. Here is my program:
that works perfectly if linkedin is not installed in my app.
//Initialize
private let linkedinHelper = LinkedinSwiftHelper(configuration: LinkedinSwiftConfiguration(clientId: K.LINKEDIN_KEY, clientSecret: K.LINKEDIN_SECRET, state: "DLKDJF46ikMMZADfdfds", permissions: ["r_basicprofile", "r_emailaddress"], redirectUrl: K.LINKEDIN_REDIRECT_URL))
//Click
linkedinHelper.authorizeSuccess({ (token) in
        PrintHelper.doPrint(data: token)
        //This token is useful for fetching profile info from LinkedIn server

        self.linkedinHelper.requestURL("https://api.linkedin.com/v1/people/~:(id,first-name,last-name,email-address,picture-url,picture-urls::(original),positions,date-of-birth,phone-numbers,location)?format=json", requestType: LinkedinSwiftRequestGet, success: { (response) -> Void in

            PrintHelper.doPrint(data:response)

            let json = JSON.init(response.jsonObject)

            PrintHelper.doPrint(data:json)
            //parse this response which is in the JSON format

        }) {(error) -> Void in

            PrintHelper.doPrint(data:error.localizedDescription)
            //handle the error
        }

    }, error: { (error) in

        PrintHelper.doPrint(data: error.localizedDescription)
        //show respective error
    }) {
        //show sign in cancelled event
    }

App Delegate
func application(_ app: UIApplication, open url: URL, options: [UIApplicationOpenURLOptionsKey : Any] = [:]) -> Bool {
    if LISDKCallbackHandler.shouldHandle(url)
    {
        return LISDKCallbackHandler.application(app, open: url, sourceApplication: options[UIApplicationOpenURLOptionsKey.sourceApplication] as? String, annotation: options[UIApplicationOpenURLOptionsKey.annotation])
    }
    else if LinkedinSwiftHelper.shouldHandle(url) {
        return LinkedinSwiftHelper.application(app, open: url, sourceApplication: options[UIApplicationOpenURLOptionsKey.sourceApplication] as? String, annotation: options[UIApplicationOpenURLOptionsKey.annotation])
    }
    else{
        return FB
    }

}
Plist
CFBundleURLTypes

CFBundleTypeRole
Editor
CFBundleURLSchemes

ABCD

NSAppTransportSecurity

NSAllowsArbitraryLoads

NSExceptionDomains

linkedin.com

NSExceptionAllowsInsecureHTTPLoads

NSIncludesSubdomains

NSExceptionRequiresForwardSecrecy

You miss LIAppId in your info.plist, 	LIAppId4127213


ESP-IDF application is not connecting to configured web socket, but always connecting to default web socket

Dear @matth-x,
I'm using ESP-IDF in my build, but it is always trying to connect to "ws://echo.websocket.events/"
while debugging, I found, it is taking the configuration from "ocpp-creds.jsn" and not from "menuconfig", Could you please tell me where to locate this file. and how to resolve this issue.
Logs:
[AO] info (./components/ArduinoOcpp/src/ArduinoOcpp/Core/Configuration.cpp:96): init new configurations container: ocpp-creds.jsn
4711 3 mongoose.c:3440:mg_connect       1 -1 ws://echo.websocket.events/
4713 3 mongoose.c:3440:mg_connect       2 -1 udp://<IP_ADDRESS>:53
4719 3 mongoose.c:4113:mg_send          2 0x36 0:0 34 err 0
[AO] info (./components/ArduinoOcpp/src/ArduinoOcpp/Core/Configuration.cpp:96): init new configurations container: /opstore.cnf
[AO] info (./components/ArduinoOcpp/src/ArduinoOcpp/Core/Configuration.cpp:96): init new configurations container: /txstore.jsn
[AO] info (./components/ArduinoOcpp/src/ArduinoOcpp/Core/Configuration.cpp:96): init new configurations container: /volatile
480f 3 mongoose.c:4239:read_conn        2 0x36 snd 0/0 rcv 0/2048 n=66 err=2
With regards

Hi @Bhuvi-Naveen,
In fact this config is a little misleading. The menuconfig is about the factory defaults. They are only used to initialize the ocpp-creds.jsn at the first boot up.
To load the new factory defaults, you can wipe the memory with the idf.py tool and start the firmware again.


Broken Links to Documentation on docs.rs for Multiple Solana Crates

Description
I have encountered multiple broken links on docs.rs for various Solana crates. These broken links lead to not a library page , making it difficult to understand certain parts if the codebase .
List of Broken Links
Here are the broken documentation links i could find :

Solana Accounts DB

URL: https://docs.rs/solana-acounts-db

Solana Cargo Registry

URL: https://docs.rs/solana-cargo-registry

Solana Stake Accounts

URL: https://docs.rs/solana-stake-accounts

Solana RPC Test

URL: https://docs.rs/solana-rpc-test

Solana Nonce Client

URL: https://docs.rs/solana-nonce-client

Solana PoH Bench

URL: https://docs.rs/solana-poh-bench

Solana Ledger Tool

URL: https://docs.rs/solana-ledger-tool

Solana Client Test

URL: https://docs.rs/solana-client-test

Expected Behavior
All links should direct to the appropriate and current documentation pages on docs.rs.
Suggested Action
Please verify whether the crates are still actively maintained and whether their documentation URLs need updating or if there are typos in the crate names in the URLs. If the documentation has been moved or if the crates have been deprecated, please update the links or the documentation references to reflect the current status.

do you mean this one?

If yes, I don't think they are broken links. they are truly bin crates. btw, the crates that you listed, only the first one is a lib (and you have typo. it is https://docs.rs/crate/solana-accounts-db). All others are either bin crates or haven't been published for a while.
could you please link me to some appropriate documentation that you have seen for bin crates or crates that haven't been published for a long time?


Issues with building client

Hi, so I followed the instructions using the test app you provided. I'm running into the following error when I do this command
$ cd app
$ meteor-build-client ../build -s ../config/electron.json -p ""

HLs-MacBook-Pro:app hl$ meteor-build-client ../build -s ../config/electron.json -p ""
Bundling Meteor app...
Command failed: /bin/sh -c meteor build ../build --directory
Supply the server hostname and port in the --server option for mobile app
builds.

 undefined

Ran into the same problem. I simply ran the meteor-build-client with -u option and pointed it to my server (still running locally)
meteor-build-client ../build -s ../config/electron.json -p "" -u http://localhost:3000

this gets over the error, but the app fails. I am not sure where the main.js and package.json are to be copied from ....

Hm, do you happen to know your MONGO_URL on your remote server?


Updated translation for French

New translation can be found for French at

https://docs.google.com/spreadsheets/d/16FtHGThz0-CBz_rf_9QO4zOV7oNmNk2BlwmtvKuzAhs/edit?usp=sharing

I also updated my information that where outdated.

Additional changes for FR translation, made on the file, essentially typographic changes (e.g.   before colon) and some improvements.
Not very satisfied with this “atomic save“ setting: it would help if there is more information on the reason(s) where a change on this setting is really needed (specific OS? text editors? ...)
Thank you @oellan for the work done! The previous translation was rather strange sometimes!!!


update history table during shipment

Working on the exploration for box movement data it appeared that we have to track box movements in the context of shipments.
Now changes in box product, size, number of items, location, and state at the time of reconciliation are tracked.

 2.0 Update history table during shipment

@HaGuesto re state: @aerinsol and I had a slack discussion with the agreement that I'll implement tracking of box state changes in the context of shipments.


implementation of sparql expression functions STRDT() and STRLANG()

I just realized that for the implementation of langMatches as a built-in-call function another helper function for handling language tags is necessary. After this pull request is done, i maybe should move the helper function here from StringUtils.h to a new file (maybe LanguageTagUtils.h)


update notices file for version 1.1.0

This change is

Review status: 0 of 1 files reviewed at latest revision, all discussions resolved.

Comments from Reviewable


Bandwidth Calculation Unit

https://github.com/mlc-ai/dlight-bench/blob/d16cf0723a37428b69cc5011ec40d450371aa9f3/q4f16_1-dequantize-gemv/degemv.py#L250-L261
The bandwidth calculated seems to be MB/s instead of GB/s.

CC: @spectrometerHBH

Probably won't fix.


modify access controll level

I want to use this module as a framework.

This is important for using cocoapods


[5.7] Add the ability to skip algorithm checking

See #25458.
Upon updating to 5.7 a portion of my user accounts can no longer login (#25458) due to the algorithm checking in place. I think it would be wise to allow us to toggle this feature as and when needed.
Although ideally there should be complete consistency in terms of the hashing algorithm one uses, sometime this is not the case. For me, I have hashes that have originated from other applications and unfortunately some of them appear to have used an older version of bycrpt.
https://en.wikipedia.org/wiki/Bcrypt#Versioning_history
Below we have an example of the issue that I am running into. Because PHP doesn't recognise my bcrypt hash.
Example
$hash = '$2a$10$Wrk/FoakWwkX/tT0A/No5uu3IZrVu/e27QHTgpjHlPUQS3HwK0ei2';
password_verify('Test12345', $hash); // bool(true)
password_get_info($hash); // array(3) { ["algo"]=> int(0) ["algoName"]=> string(7) "unknown" ["options"]=> array(0) { } }

Because the above returns an algoName of unknown the exception is thrown.
public function check($value, $hashedValue, array $options = [])
{
    if ($this->info($hashedValue)['algoName'] !== 'bcrypt') {
        throw new RuntimeException('This password does not use the Bcrypt algorithm.');
    }
    return parent::check($value, $hashedValue, $options);
}

https://github.com/laravel/framework/blob/5.7/src/Illuminate/Hashing/BcryptHasher.php#L60
PHP's password_get_info() function only recognises the $2y$ prefix. https://github.com/php/php-src/blob/master/ext/standard/password.c#L81
Solution
I have added an option that you can specify in you config to turn on and off the algorithm checking (obviously on by default).
    'bcrypt' [
        'check' => false
    ],

The issue in the ticket (#25458) is slightly different to my use case, but check it out.
If the is a more intelligent way of going about fixing this, do let me know.

I imagine that another name (like verifyAlgorithm or checkAlgorithmName) is easier to understand than the generic check.

Very true, verifyAlgorithm is a good suggestion - I have changed it.
If that is deemed too long, verify?

@taylorotwell I'm not sure you realise how that breaks existing apps, and will break also new apps in the future when new hashing algos will replace the existing ones?

I believe that this breaking change was a hasty decision.
I don't see any real gain out of this, for the sake of backward compatibility and to support future hashing algorithm, it should be consistent with how password_verify works.
I also opened an issue on the framework repo: https://github.com/laravel/framework/issues/25586

While I have overcome my issue using @taylorotwell's suggestion, I think @kamui545's has a decent argument.

Future Algorithms
Let's say we had a new bcrypt version ($2z) tomorrow and PHP updated their algorithm checker for PHP 7.3.
Upon updating PHP all (bcrypt) hashes checked on login would now throw an exception without updating Laravel - as the password hashes are no longer valid in Laravel's eyes.
What would be the solution in this situation?
Backwards Compatibility
See https://github.com/laravel/framework/issues/25586
What are your thoughts @taylorotwell?

This has been fixed in 5.7.4

Rejoice! 🎉

Defaulted verification to false so it just works how it has in years past.


fix: prevent timer callback race condition

What is this PR about?
This PR fixes a race condition where the timeout timer callback writes to already closed stdout or stderr channels, resulting in a panic: send on closed channel error.
Why is this change necessary?
When a timeout occurs, the timer callback (time.AfterFunc) writes an error message to the stderr channel and kills the process. However, if the stderr channel has already been closed (e.g., by the caller in RunPython3Code), this leads to a runtime panic.
The issue arises because timer.Stop() does not prevent an already-triggered timer callback from executing.
What changes have been made?
•	Added a sync.WaitGroup to track the timer callback execution.
•	Ensured that if timer.Stop() fails (indicating the timer has already triggered), the main process waits for the timer callback to finish before closing the channels.

@Yeuoly

Hi @Yeuoly
Thanks for approving the PR! It looks like the workflow is still pending approval to run. When you have a moment, could you approve it under the Actions tab?
Much appreciated!


Change ReadTheDocs URL to match distribution name

Currently we're using https://catalyst-cooperative-pudl.readthedocs.io which matches the Github repo name, but that's awfully long, and it seems more common for the URL to match the distributed package name, so given that we're using catalystcoop.pudl it seems like changing the URL to  https://catalystcoop-pudl.readthedocs.io would be better -- and we should do it before we publish it and links show up elsewhere.  This will mean deleting and re-creating the project at RTD.

Switched!  We've got docs at https://catalystcoop-pudl.readthedocs.io Pretty painless.


YouTube doesn't work

Every time I enter a YouTube link and hit "Generate subtitle File", I always get an error. I'm even running as admin and completely reinstalled the app from scratch and nothing helped.
I'm using the script with Pinokio.
Windows 11,
32 GB RAM,
Ryzen 5600x
RTX 3060 12GB VRAM

Hi. Can you try to update the pytubefix in the Pinoko terminal?
pip install -U pytubefix


Contracts of Validator does't work during package development but facade does

Laravel Version: 5.3.9
PHP Version: 7.0

Description:
I'm doing a package for laravel, for now it is simply a controller with validation in its method as following:
public function someMethod(Request $req)
{
  $validator = Validator::make($req->all(), [
    //...
  ]);

  //...
}

So, i need to load the class Validator. According to the laravel doc , it seems i should use contracts. So i add:
use Illuminate\Contracts\Validation\Validator;
use Illuminate\Contracts\Validation\Factory;

But i'll get error
Call to undefined method Illuminate\Contracts\Validation\Validator::make()

By using instead of above two
use Validator;

it is ok. So i think it might be a bug, contracts should also work.
Steps To Reproduce:

create a package which has a controller
use Validator contracts

use Illuminate\Contracts\Validation\Validator;
use Illuminate\Contracts\Validation\Factory;

invoke Validator::make in the method of the controller you created in the package

You can't call a static method on a contract. You need to dependency inject it.

@GrahamCampbell Not quite understand. It seems i still need to use facade cause the interface can not be instantiable.
use Illuminate\Contracts\Validation\Validator;
use Illuminate\Contracts\Validation\Factory;

	public function edit(Request $req, Validator $validator)
	{
		$validator = Validator::make(...

Does't work. Or i miss something?


operator: enhance configurability

add more config options (preserving the current default behavior) to enable future customizability

/cherry-pick release-4.14


Best practice for hosting niivue app on GitHub pages with Vite

Hi there!
today I had the chance to make some experiments with niivue, and here I want to share some things I've come up with.
First of all I've tried the react app example and that works out of the box:
Repo: https://github.com/kolibril13/niivue-vite-react-app
Page: https://kolibril13.github.io/niivue-vite-react-app/
next I've tried the basic vanilla JS example and that worked great as well.
However, here I had to first create a canvas element to make this work.
Repo: https://github.com/kolibril13/niivue-vite-vanillajs-app
Page: https://kolibril13.github.io/niivue-vite-vanillajs-app/
My question regarding the Vanilla JS example:
Is this how Niivue should be used, or can the below example still be improved?
Looking forward to feedback!
import { Niivue } from "@niivue/niivue";

// Create a canvas element
const canvas = document.createElement("canvas");
canvas.id = "gl";
canvas.width = 800;
canvas.height = 800;

// Append the canvas to the element with id="app"
document.querySelector("#app").appendChild(canvas);

// Make an array of volumes to load
let volumeList = [
    { url: "https://niivue.github.io/niivue-demo-images/mni152.nii.gz" },
];

const nv = new Niivue({ isResizeCanvas: false });
nv.attachTo("gl"); // attach to canvas with id="gl"
nv.loadVolumes(volumeList);

The vanilla JS example looks good to me. The only alternative would be that the canvas already exists in the html page and the JavaScript only attaches NiiVue to the canvas rather than creating the canvas in JS.

That's great to hear!
For ipyNiiVue was translating the above example into an anywidget widget by replacing document.querySelector("#app").appendChild(canvas) with   el.appendChild(canvas); like this:
import { Niivue } from "@niivue/niivue";

export function render({ model, el }) {
  const canvas = document.createElement("canvas");
  canvas.id = "gl";
  canvas.width = 800;
  canvas.height = 800;

  // Append the canvas to the element with id="app"
  el.appendChild(canvas);

  // Make an array of volumes to load
  let volumeList = [
    { url: "https://niivue.github.io/niivue-demo-images/mni152.nii.gz" },
  ];

  //
  const nv = new Niivue({ isResizeCanvas: false });
  nv.attachTo("gl"); // attach to canvas with id="gl"
  nv.loadVolumes(volumeList);
}

Unfortunately, I now get an error:
ncaught (in promise) TypeError: Cannot read properties of null (reading 'getContext')

Interestingly, the error goes away, and the widget does appear when I trigger the Hot Module Replacement.
Triggering Hot Module Replacement (HMR) means saving the changes and letting npm rebuild the JS without restarting the notebook and without reloading the page, here's a screen recording of this:
https://github.com/niivue/niivue/assets/44469195/9d1b90af-d900-4a19-9810-0200c96f38c9
Probably that has to do something with the way the canvas is added to the widget output and the async call of loadVolumes (as also mentioned here https://github.com/niivue/ipyniivue/issues/44#issuecomment-1880972855)
Further notes:

This error does only occurs in the anywidget vanillaJS version:
https://github.com/kolibril13/anywidget-ipyniivue-vanillajs/blob/main/js/widget.js
This error does not occur at all in the anywidget react version:
https://github.com/kolibril13/anywidget-ipyniivue/blob/main/js/widget.jsx

Any ideas?
cc. @christian-oreilly , @manzt , @cdrake

@kolibril13 javascript runs in parallel by design, but one must wait until your canvas exists before you can get the WebGL context from the context. So your problem is you can not attach a NiiVue instance to a canvas until the canvas is created, so you must await the completion of document.createElement("canvas") before you call nv.attachTo("gl").
Several solutions are described here but I am not sure what the most elegant method for anywidget work flow.
As an aside, you will want to ensure that isResizeCanvas: false is the correct for anywidget - this enforces the idea that the canvas has a fixed resolution that will not change to fit a container.
One final note, some computations are very slow to compute and so NiiVue will defer running them until requested. This is unintuitive, but it does allow you to change a large number of properties before waiting for a slow computation. This is described in issue 463. The call nv1.updateGLVolume(); forces NiiVue to apply all the deferred changes.

@kolibril13, why not use a different api that accepts the canvas element directly (attachToCanvas)?:
import { Niivue } from "@niivue/niivue";

export async function render({ model, el }) {
	let canvas = document.createElement("canvas");
	let container = document.createElement("div");
	container.style.height = "300px";
	container.appendChild(canvas);
	el.appendChild(container);

	let volumeList = [{ url: "https://niivue.github.io/niivue-demo-images/mni152.nii.gz" }];
	let nv = new Niivue();
	nv.attachToCanvas(canvas);
	nv.loadVolumes(volumeList);
	return () => {};
}

nv.attachToCanvas(canvas); solved this problem 🎉
Also thanks a lot for the video!
I especially like this part at in the middle where the terms stack webapis, task queue and event loop are explained in an intuitive visual way.


Compatibility Issues with PHP 8.3

Steps to reproduce :
Running duster fix / lint with PHP 8.3 with throw the following error.

@krlmrr im having the same error on php 8.1

Interesting, if I set PHP Monitor to PHP 8.2 the error goes away, but I am not able to recreate the error with PHP 8.1.

Workaround, replace uses: tighten/duster-action@v2 with   run: vendor/bin/duster lint

I also getting this error inside gitlab ci with vendor/bin/duster lint via php 8.3. On php 8.2 there is no error at all.

If you are using the duster-action I belive this is the problem:
https://github.com/tighten/duster-action/issues/7#issuecomment-1832035664
Hopefully, we can get a fix, with a fixed version, or PHP 8.3 compatibility.

+1 Bump
@mattstauffer  - Would you have a second for this?  😄

I've asked the Tighten folks who are helping me with Duster to take a look!

Hey everyone, this has been fixed in 2.6.0. Thank you for reporting and patience 🙏


setup.py should not install 'tests' package at top level

fixes #58

sure, now is: 1.2.1

Thank you for the fix!

Thanks for patching. (Only 48 left).


Don't copy inexistent certs folder in Dockerfile.django

When building Dockerfile.django with DOCKER_BUILDKIT=1 option in docker, the build fails with:
failed to solve with frontend dockerfile.v0: failed to build LLB: lstat /var/lib/docker/tmp/buildkit-mount548069851/certs: no such file or directory

[X] Give a meaningful name to your PR, as it may end up being used in the release notes.
[ ] Your code is flake8 compliant.
[ ] Your code is python 3.6 compliant (specific python >3.6 syntax is currently not accepted).
[ ] If this is a new feature and not a bug fix, you've included the proper documentation in the ReadTheDocs documentation folder. https://github.com/DefectDojo/Documentation/tree/master/docs or provide feature documentation in the PR.
[ ] Model changes must include the necessary migrations in the dojo/db_migrations folder.
[ ] Add applicable tests to the unit tests.
[ ] Add the proper label to categorize your PR.

@uncycler I added this option intentionally, see https://github.com/DefectDojo/Documentation/blob/master/docs/social-authentication.rst#saml-20
Idea is to enable generic way of inserting of private CA serts that are not included inside of certifi

@dsever Maybe just adding a certs directory with a readme.txt. We could put it inside /docker so it don't clobber the root. What do you think?

You mean just to have some content, not to break the build? If yes than I can add readme.txt and some instructions to it.

I update the PR to add a placeholder.


MainMenu.nib for 10.6, with Sparkle

Sorry for delay, was on vacation. Here's the work requested in the comment in #211. Needs more work to successfully build on 10.6 though; I get:
The following build commands failed:
        PBXCp Sparkle.framework build/Release/MacVim.app/Contents/Frameworks/Sparkle.framework

Need to make copying Sparkle.framework conditional (>= 10.7?) the same way that QuickLookStephen.xcodeproj is, but I'm not sure about the right commands to do what the Copy Frameworks phase is doing (is there a PBXCp utility I can call from the script phase?).

:+1:


pull request for e2e test at 01/10/2017 12:15:08

:white_check_mark: Validation status: passed
For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


We shouldn't let add a new measurement after the last one is valid

STR

Create a map of SF neighborhoods (present in data library).
Create a DO analysis.
Add a new measurement item.

We should not let the user add a new measurement until the previous one is selected and valid.
Caveats

If it is necessary, we will display a required tooltip. But we don't think we will needed.

This is closely related to Backbone Forms and nested components forms. Right now, this same component is used in custom legend form as well.

Merged into main branch!


Tests: Temporarily disable attr_backDeploy_evolution during swift_test_mode_optimize builds

Fixes the https://ci.swift.org/job/oss-swift_tools-RA_stdlib-RDA_test-simulator/ bot.

@swift-ci please smoke test and merge


feat: strip node: prefixes from our dists

🧰 Changes
I learned this evening that despite adding support for node: protocols in Webpack 5.40^1 it doesn't support them in resolve.fallback or resolve.alias^2 which is making it impossible to compile this library, by way of @readme/httpsnippet and @readme/oas-to-snippet, in our main app:

This is a tough one because I hate this ESBuild plugin I found but also I do kind of like the node: prefix, eventhough it doesn't really do anything other than adding visual sugar^3,  but maybe instead of stripping it from our dists we should just stop using it all together and remove the unicorn/prefer-node-protocol rule from our ESM ESLint config^4?
🧬 QA & Testing
Before
Curiously tsup and esbuild already remove it for CJS builds.

After

@kanadgupta Yeah agreed. Unfortunately it just seems like it's not possible to deal with these prefixes in Webpack at the moment.


TwilioAgent's receive_text and receive_call options are poorly documented.

The online documentation for the TwilioAgent does not seem complete.  The use case for the receive_call option isn't fully documented; what it does and why it might be useful are not specified.  The receive_text option is not documented at all (though it could be inferred that this is so that the agent could receive text messages for processing this may not be correct).
Additionally, it is not certain what the value of server_url would be if Huginn is behind a proxy of some kind (such as Nginx).  It seems reasonable that omitting the port Huginn is running on behind the proxy might be the correct thing to do, but this may not be the case.

Did you figure out your issue?  If so, a PR to update the docs would be awesome!

Maybe.  I've got notes but I haven't actually tried to implement them yet.  Let me do a few experiments, and if I'm right I will.

Okay, here's what I've got:
If Huginn is behind a web server of some kind (like Nginx), the value of server_url will be whatever the URL would be in the browser: https://huginn.example.com/.  If HTTP authentication is enabled, a separate username and passphrase need to be set in the .htaccess file and added to the URL (ref ticket #815), like this<EMAIL_ADDRESS> Twilio's TwiML aplication configurations work with this fine.
TwilioAgent will accept as its message any of the following keys: {{message}}, {{text}}, or {{sms}}
receiver_cell must contain the phone number TwilioAgent will send a message to.
At least one of the keys receive_text and receive_call must be set.  receive_text means that Twilio will send a text message.  receive_call means that the message will be run through a speech synthesis system and the destination number will receive a phone call.  Messages sent from free accounts will have a "Buy a real account" nag prepended to them.

Would it be worth writing this up on the wiki?

I think it would.  I'll get on it.

https://github.com/cantino/huginn/wiki/Using-the-Twilio-Agent

Thanks!  Are any parts of that worthy of also adding to the agent in a PR?

At the moment, no.  I don't think so, anyway.

Okay, it seems like you've figured this out now and documented it, so I'm going to close the issue.  Please reopen if more discussion is needed.


Unable to restore page revision

For Bug Reports

BookStack Version: v0.15.2
PHP Version: 5.6.30
MySQL Version: Ver 15.1 Distrib 10.1.21-MariaDB, for Linux (x86_64) using  EditLine wrapper

Expected Behavior
Page revision is restored when going to Revisions and selecting Restore on any of the revisions.
Actual Behavior
Error is encountered when attempting to restore.
Here's the whole capture.
ErrorException in EntityRepo.php line 103:
Undefined index: page_revision

in EntityRepo.php line 103
at HandleExceptions->handleError(8, 'Undefined index: page_revision', '/var/www/stacks/app/Repos/EntityRepo.php', 103, array('type' => 'page_revision')) in EntityRepo.php line 103
at EntityRepo->getEntity('page_revision') in EntityRepo.php line 114
at EntityRepo->entityQuery('page_revision', false) in EntityRepo.php line 141
at EntityRepo->getById('page_revision', '292') in EntityRepo.php line 1061
at EntityRepo->restorePageRevision(object(Page), object(Book), '292') in PageController.php line 426
at PageController->restoreRevision('server-notes', 'nasnetwork', '292')
at call_user_func_array(array(object(PageController), 'restoreRevision'), array('server-notes', 'nasnetwork', '292')) in Controller.php line 55
at Controller->callAction('restoreRevision', array('server-notes', 'nasnetwork', '292')) in ControllerDispatcher.php line 44
at ControllerDispatcher->dispatch(object(Route), object(PageController), 'restoreRevision') in Route.php line 204
at Route->runController() in Route.php line 160
at Route->run() in Router.php line 559
at Router->Illuminate\Routing\{closure}(object(Request)) in Pipeline.php line 30
at Pipeline->Illuminate\Routing\{closure}(object(Request)) in Controller.php line 44
at Controller->BookStack\Http\Controllers\{closure}(object(Request), object(Closure)) in Pipeline.php line 131
at Pipeline->Illuminate\Pipeline\{closure}(object(Request)) in Pipeline.php line 53
at Pipeline->Illuminate\Routing\{closure}(object(Request)) in Authenticate.php line 45
at Authenticate->handle(object(Request), object(Closure)) in Pipeline.php line 148
at Pipeline->Illuminate\Pipeline\{closure}(object(Request)) in Pipeline.php line 53
at Pipeline->Illuminate\Routing\{closure}(object(Request)) in Localization.php line 21
at Localization->handle(object(Request), object(Closure)) in Pipeline.php line 148
at Pipeline->Illuminate\Pipeline\{closure}(object(Request)) in Pipeline.php line 53
at Pipeline->Illuminate\Routing\{closure}(object(Request)) in SubstituteBindings.php line 41
at SubstituteBindings->handle(object(Request), object(Closure)) in Pipeline.php line 148
at Pipeline->Illuminate\Pipeline\{closure}(object(Request)) in Pipeline.php line 53
at Pipeline->Illuminate\Routing\{closure}(object(Request)) in VerifyCsrfToken.php line 65
at VerifyCsrfToken->handle(object(Request), object(Closure)) in Pipeline.php line 148
at Pipeline->Illuminate\Pipeline\{closure}(object(Request)) in Pipeline.php line 53
at Pipeline->Illuminate\Routing\{closure}(object(Request)) in AddQueuedCookiesToResponse.php line 37
at AddQueuedCookiesToResponse->handle(object(Request), object(Closure)) in Pipeline.php line 148
at Pipeline->Illuminate\Pipeline\{closure}(object(Request)) in Pipeline.php line 53
at Pipeline->Illuminate\Routing\{closure}(object(Request)) in EncryptCookies.php line 59
at EncryptCookies->handle(object(Request), object(Closure)) in Pipeline.php line 148
at Pipeline->Illuminate\Pipeline\{closure}(object(Request)) in Pipeline.php line 53
at Pipeline->Illuminate\Routing\{closure}(object(Request)) in Pipeline.php line 102
at Pipeline->then(object(Closure)) in Router.php line 561
at Router->runRouteWithinStack(object(Route), object(Request)) in Router.php line 520
at Router->dispatchToRoute(object(Request)) in Router.php line 498
at Router->dispatch(object(Request)) in Kernel.php line 174
at Kernel->Illuminate\Foundation\Http\{closure}(object(Request)) in Pipeline.php line 30
at Pipeline->Illuminate\Routing\{closure}(object(Request)) in Debugbar.php line 51
at Debugbar->handle(object(Request), object(Closure)) in Pipeline.php line 148
at Pipeline->Illuminate\Pipeline\{closure}(object(Request)) in Pipeline.php line 53
at Pipeline->Illuminate\Routing\{closure}(object(Request)) in ShareErrorsFromSession.php line 49
at ShareErrorsFromSession->handle(object(Request), object(Closure)) in Pipeline.php line 148
at Pipeline->Illuminate\Pipeline\{closure}(object(Request)) in Pipeline.php line 53
at Pipeline->Illuminate\Routing\{closure}(object(Request)) in StartSession.php line 64
at StartSession->handle(object(Request), object(Closure)) in Pipeline.php line 148
at Pipeline->Illuminate\Pipeline\{closure}(object(Request)) in Pipeline.php line 53
at Pipeline->Illuminate\Routing\{closure}(object(Request)) in CheckForMaintenanceMode.php line 46
at CheckForMaintenanceMode->handle(object(Request), object(Closure)) in Pipeline.php line 148
at Pipeline->Illuminate\Pipeline\{closure}(object(Request)) in Pipeline.php line 53
at Pipeline->Illuminate\Routing\{closure}(object(Request)) in Pipeline.php line 102
at Pipeline->then(object(Closure)) in Kernel.php line 149
at Kernel->sendRequestThroughRouter(object(Request)) in Kernel.php line 116
at Kernel->handle(object(Request)) in index.php line 54

Hi @domainzero, Thanks for reporting this. Sorry for the delay in my response and for the fact the issue is apparent.
I have fixed the issue and now added a proper test for page restores to help prevent a bug like this happening again. I've just pushed up a new release, v0.15.3, to cover this.

No worries. Good work!


chore: release main

:robot: I have created a release beep boop
baggie: 2.0.2
2.0.2 (2023-07-13)
Bug Fixes

core: Add src files to npm package (9f80019)
react: Add src files to npm package (4162903)

core: 1.3.2
1.3.2 (2023-07-13)
Bug Fixes

core: Add src files to npm package (9f80019)

react: 1.0.3
1.0.3 (2023-07-13)
Bug Fixes

react: Add src files to npm package (4162903)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/bag-of-tricks/baggie/releases/tag/baggie-v2.0.2 :sunflower:

:robot: Release is at https://github.com/bag-of-tricks/baggie/releases/tag/core-v1.3.2 :sunflower:

:robot: Release is at https://github.com/bag-of-tricks/baggie/releases/tag/react-v1.0.3 :sunflower:


README on marketplace does not show Prisma Logo

I think we can just remove the image.
Both on https://marketplace.visualstudio.com/items?itemName=Prisma.prisma and in VSCode the extension picture itself is visible:

Without the repetition, we have more space for the actually interesting content.


Enhancement: Move the variable value for map key to config.xml

It will be easily changable and efficient if the developer moves the map key variable from cli input to just read from config.xml

Impossible. In Android, you MUST put your API_KEY in the AndroidManifest.xml file that is not changeable from the application code.

Thanks for your quick reply.Actually i was talking about something like this

and i have seen similar configuration in this #907 issue as explained by you.

#907 is case for PhoneGap Build. Not CLI.


Importing library error

Hello ,
I have been trying to use the code yet when i am trying to import the library i get the same error
`~/opt/anaconda3/lib/python3.9/site-packages/shakenbreak/distortions.py in
7 import numpy as np
8 from ase.neighborlist import NeighborList
----> 9 from hiphive.structure_generation.rattle import (
10     _probability_mc_rattle,
11     generate_mc_rattled_structures,
~/opt/anaconda3/lib/python3.9/site-packages/hiphive/init.py in
4
5 from .cluster_space import ClusterSpace
...
----> 3 from numba.np.ufunc import _internal
4 from numba.np.ufunc.parallel import ParallelUFuncBuilder, ParallelGUFuncBuilder
5
SystemError: initialization of _internal failed without raising an exception`
I tried changing the numpy and numba version but it doesnt seem to solve the issue,
I was wondering if you could give me some suggestions

Hi Andres,
What are your versions of numpy and numba?
Also, might be worth trying to use a python virtual environment for ShakeNBreak to avoid version issues. In conda you can create one using:
conda create -n snb
conda activate snb

And then install ShakeNBreak inside it:
conda install -c conda-forge shakenbreak

Hello @ireaml
I tried with numpy 1.23.5 and 1.24.1 and 1.24.2
and numba 0.54.1 0.55 0.56.2
Since I already have an environment with other packages I wanted to include ShakeNBreak python package while keeping at least numpy 1.24.1  otherwise there seems to be an incompatibility issue only in ShakeNBreak

So numba requires numpy<1.24,>=1.18, which it's the incompatibility causing the error.
Can you try with numpy<=1.23.5 and numba==0.56.4? For me, with these versions (1.23.5 and 0.56.4 it works ok)

Hello,
I tried the same combination, no success either

Did you use pip or conda for installation?
Can you execute the following lines in a jupyter cell? (with your environment activated)
from importlib_metadata import version

print(f"Version of numpy:", version("numpy"))
print(f"Version of numba:", version("numba"))
print(f"Version of hiphive:", version("hiphive"))
print(f"Version of shakenbreak:", version("shakenbreak"))

I manage to make it wok but i use numpy 1.23.1
Version of numpy: 1.23.1
Version of numba: 0.56.4
Version of hiphive: 1.1
Version of shakenbreak: 23.2.8


Hi

Hu

Please close this issue, As soon as possible.
It doesn't serves any purpose.
Thank you,
Nitkarsh Chourasia.


Update & publish the new release guide/steps for devrel

based on what @bidoubiwa defined before us, I created a PR for that. I updated based on what I had in mind.
https://github.com/meilisearch/devrel/pull/76
@CaroFG please comment.
@react-learner please review the documentation part, and please, copyedit the text itself.
@bidoubiwa comments welcome based on your experience at MeiliSearch.
@curquiza I'll create the same for a new integration.

@curquiza I'll create the same for a new integration.

Not sure to understand what it means 😁

@curquiza I'll create the same for a new integration.

Not sure to understand what it means 😁

Food for thoughts @react-learner, but the MeiliSearch use Milestone to manage PR & commits for new version. I would prefer we use the same process. See https://github.com/meilisearch/MeiliSearch/milestone/15

Food for thoughts @react-learner, but the MeiliSearch use Milestone to manage PR & commits for new version. I would prefer we use the same process. See https://github.com/meilisearch/MeiliSearch/milestone/15

@curquiza I'll create the same for a new integration.

Not sure to understand what it means 😁

We will create the same kind of guide for when a new integration is released or an awesome new feature worth sharing for one or more integrations. Clearer?

@curquiza I'll create the same for a new integration.

Not sure to understand what it means 😁

We will create the same kind of guide for when a new integration is released or an awesome new feature worth sharing for one or more integrations. Clearer?

Not sure this is exactly the kind of guide you want to write but here is what I already wrote:

about the integration tools release: https://github.com/meilisearch/integration-guides/blob/master/guides/integration-tool-release.md
about the pre-release-week: https://github.com/meilisearch/integration-guides/blob/master/guides/pre-release-week.md

Not sure this is exactly the kind of guide you want to write but here is what I already wrote:

about the integration tools release: https://github.com/meilisearch/integration-guides/blob/master/guides/integration-tool-release.md
about the pre-release-week: https://github.com/meilisearch/integration-guides/blob/master/guides/pre-release-week.md

Awesome guides! It's more for the after, what do we do in #devrel. Mostly about sharing the good work your team is doing :D

Awesome guides! It's more for the after, what do we do in #devrel. Mostly about sharing the good work your team is doing :D


review our web security

e.g.
https://community.qualys.com/blogs/securitylabs/2015/04/16/security-issues-discovered-and-fixed-in-searchblox

https://community.qualys.com/blogs/securitylabs/2015/06/08/introducing-tls-maturity-model


Added a order by when fetching jobs (#5)

Closes #5
I don't really see how to properly test this? The current tests, like test_get_jobs already cover this?
Your ideas are welcome :>

Codecov Report

Merging #34 into master will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff          @@
##           master    #34   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files          11     11
  Lines         362    362
  Branches       24     24
=====================================
  Hits          362    362

Impacted Files
Coverage Δ

cabbage/postgres.py
100% <ø> (ø)
:arrow_up:

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 6c8f7dd...9370050. Read the comment docs.

I added the ORDER BY in cabbage_fetch_job.
I'm still unsure on testing this :o It doesn't break anything so that's good.

(Suggestion, not obligation, please merge when you feel like it’s ready)

Hmm, you know what ? i’ll merge now and if you want to do a test, you’re welcome in another PR :)


Reduce the size of the repo

Copy the repo to an archive repo. In this repo, filter out all old versions the landscape and serverless landscape and the trail map.
Create a new repo just to hold the trail map.
https://itextpdf.com/blog/how-completely-remove-file-git-repository
On build, copy the latest version of the trail map to /dist so that we serve the trail map from this app rather than GitHub (and get credit for doing so in Google Analytics).

@dankohn , here is a list of required actions

close all pull requests, clean all branch
remove all obsolete local branches
remove landscape, serverless, trail_map folder and commit
run bfg --delete-folders 'landscape', bfg --delete-folders 'serverless', bfg --delete-folders 'trail_map'
this removes these files from the history, also they should not present in the latest commit
run git reflog expire --expire=now --all && git gc --prune=now --aggressive to get rid of obsolete local files
finish with git push origin HEAD --force
I expect that eventually github will cleanup old files and the repo will be back to 37mb

Just forgot to mention, after that I need to update the code, I have a question, can we reference the new cncf/landscape-trailmap repo from the readme to show a trail map ?

Or do you want me to copy data to the dist folder and then server those files from the landscape.cncf.io for the readme ?

Implemented.
Also note this useful command:
git rev-list --objects --all \
| git cat-file --batch-check='%(objecttype) %(objectname) %(objectsize) %(rest)' \
| sed -n 's/^blob //p' \
| sort --numeric-sort --key=2 \
| gcut -c 1-12,41- \
| gnumfmt --field=2 --to=iec-i --suffix=B --padding=7 --round=nearest

Note that the original, unaltered repo is archived at: https://github.com/cncf/landscape-archive


Support Flight

No. Flighting doesn't mean live configuration outside Microsoft. So please do not mention  'flight' as live configuration here.

No. Flighting doesn't mean live configuration outside Microsoft. So please do not mention 'flight' as live configuration here.

I don't think so. App Store has an application called TestFlight

Just searched on the web. The word flight has no meaning or any relationship between live configuration.

Just searched on the web. The word flight has no meaning or any relationship between live configuration.

Looks interesting
https://aidanfinn.com/?p=17862


String Limit?

I am able to get the player working with a few songs, but from a bit of messing about I found that files with long path strings seem to overwhelm things at startup and the game refuses to launch.
If a string limit is real, how large can these be?
I have a playlist of 50 songs, all located on a separate HDD than the game, all files are mp3 and the playlist was saved via foobar2000 as some have stated would be best.
The songs that do work, have path strings about 64 char long, some of the longest file paths shown are about 116 char long

This is late, but its probably just the standard windows path limit of 206 characters. I think you can change the limit in the settings but idk why it would crash at 116.


Fix PopupPanel and PopupMenu menu styles

Fixes: https://github.com/godotengine/godot/issues/96517
Edit: Fixes: https://github.com/godotengine/godot/issues/67657 (border part)
This makes popup menus using PopupPanel and PopupMenus have same base look, as it is in Godot 3.
Panel border
Adding a default border to the PopupPanel to make it easier to differentiate from the background.
I believe this was removed in PR https://github.com/godotengine/godot/pull/45607 and then added back for PopupMenus in PR https://github.com/godotengine/godot/pull/48655 (also removed support for Border Size on PopupMenus)
Before:

After:

Border color
In Godot 3 both styles of popup menus uses the same border color, light on dark theme and dark border on light themes.
Godot 3 example:

1
2

Currently Godot 4 uses dark border for PopupMenus and with Border Size in Editor Settings set to 1 or 2 PopupPanel uses a light border in dark themes.
Before: with border size set

1
2

After: border size is set to deafult

1
2

Because the PopupMenu border is visible by default I went with that color for both in this PR.
Maybe there could be a editor setting for changing outline color, but that would need to be a proposal in that case.
Border Size
In Godot 3 both PopupPanel and PopupMenus uses the Border Size.
PopupPanel uses this already.
This PR also changes PopupMenus to use the editor setting Border Size for the outer border.
Separator width
Due to the change of using Border Size the separator line in PopupMenus was much more obvious that it overlaps the outer border. So this was also changed
Before:

After:

Compact
Before:

After:

Thanks!


Remove activityId when contacting ContextShare blob

Tile content http request should be kept as a “simple requests” without pre-flight.

 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.

I'll defer to someone (@ramanujam-raman? @calebmshafer?) who understands the ramifications of this change - I don't.

Currently when iModelJs request a tile from ContextShare, it will send two requests, a pre-flight(OPTIONS) and the tile itself(GET). This is because we send an x-activitity-id header, which force a pre-flight. More info on MDN.
The goal of this PR is to keep tile request as a simple request to lower by half the amount of requests done to ContextShare.

@calebmshafer How do the merge works? Do I need all 3 reviewers left?

Do I need all 3 reviewers left?

No. Green checkmark means changes approved.

@calebmshafer Should the section ## Source Code Edit Workflow in the main readme.md mention the need to run rush lint?

@calebmshafer Should the section ## Source Code Edit Workflow in the main readme.md mention the need to run rush lint?

@BeCharem yes we definitely should. Just opened #1419 with the update.


SH1106 feature request / LUMA display driver

wondering whether the script provided by splitti is running for someone here.
It is not for me as of now, but it has and was working perfectly, in v2.1, IIRC.
I have no idea where to look for the problem.
two different pi0's were used, one without any audio hardware attached, one with a working adafruit 2W-amp-bonnet.
Both are running the legacy version of buster (2021-12-02), one used dev, the other one the main branch.
The source for the script is:
https://splittscheid.de/phoniebox-display/
https://github.com/rm-hull/luma.oled/wiki/Usage-&-Benchmarking
thanks in advance

Luma OLED Python library: https://github.com/rm-hull/luma.oled


Endpoint for walletless recovery

We need an endpoint for the recovery of walletless keys. This basically just signs a message to the backend and return the answer.
$"{keyPair!.PublicKey}|{accessKey}|{DateTime.UtcNow.ToString("yyyy-MM-ddTHH:mm:ssZ")}";

this will be under
GET walletless/recover

This will transform into a generic endpoint for signing stuff.


chore: remove stale workflow

Overview
Since we're only now picking up maintenance, it doesn't make sense (and hasn't for a while) to auto-close PRs without reviews.

coverage: 94.337%. remained the same
when pulling<PHONE_NUMBER>7573ed3d0ce5629e27e68a984f2581 on no-stale
into 48002bc60144c25060bf91359620c4a0284e8284 on master.


different machines show different nesting levels for json output

I am running this script:
#!/usr/bin/env raku
use v6;
use WWW;

my @allRepos=jget qq!https://api.github.com/users/raku-community-modules/repos!;

say @allRepos.elems;

on two machines on the same network, running the same version of Raku:
$ raku -v
Welcome to 𝐑𝐚𝐤𝐮𝐝𝐨™ v2021.06-21-gdba9f56af.
Implementing the 𝐑𝐚𝐤𝐮™ programming language v6.d.
Built on MoarVM version 2021.06-6-g6977d0591.

and the same version of WWW, namely 1.005006. Furthermore, I have checked that
curl https://api.github.com/users/raku-community-modules/repos

returns identical output on both machines (the file is here; I ran that curl command on both and diff-ed the outputs; they're identical).
Nevertheless, on one machine the Raku script provided above returns 1, and on the other it returns 30. For the first machine, I have to furter unwrap the output to @allRepos.[0] to get to the 30 repos.
So there's an extra level of nesting on just one machine, but I don't understand how this can be, given that everything seems to be identical.

Sorry to be so late. This might be due to the JSON library that's been used. I don't think there's an easy fix for that other than pinning the JSON library version


Add a separate install tools command

Description
Add a separate install tools command. This allows for something like sudo az capi install which fixes issues when installing things on Linux.  This seems to be the pattern according to https://github.com/Azure/azure-cli/issues/2558#issuecomment-289919156
Fixes: #90
History Notes
Something I tried was using a temp path in the function download_binary if on linux but because this command was in a spinner it never prompted (it did but it got overwritten). It worked if I had recently used sudo.
  if system == "Linux":
         tmpfile = Path(tempfile.gettempdir()).joinpath(os.path.basename(install_location))
        urlretrieve(file_url, tmpfile)
        os.system(f"sudo mv {tmpfile} {install_location}")
  else:

This checklist is used to make sure that common guidelines for an Azure CLI pull request are followed.

[ ] The PR title and description has followed the guideline in Submitting Pull Requests.
[ ] I adhere to the Command Guidelines.
[ ] I adhere to the Error Handling Guidelines.

@DannyBrito I was wondering something along the same lines.  Maybe this should implement it with an argument --all with the idea that later we could install individual components via an argument --name it was needed?

I guess kind isn't always required, so we wouldn't want to install it by default. I like @jsturtevant's idea about making it installable with az capi install --all for now.


Throws an error with sapplings from BetterEnd in 1.18.2

Error: https://mclo.gs/gCm3kld
saplanting-fabric-1.18.2-1.2.4
better-end-1.1.1
Fabric loader 0.14.21

In short

If no other trouble, you can ignore.
Not fully on my side.

In detail:
Errors that start with "Some Errors occurred during planting this item:" means something not good has happened, but the mod Saplanting had catched these errors and deal with it with specific procedures(mentioned below). Which means if this error is not bringing troubles to you, it's OK to just ignore the error.
Anyway, since a detailed traceback is provided, I'll try to explain the error.
Interface cool.muyucloud.saplanting.mixin.SaplingGeneratorAccessor is a Mixin-class used to provide access to some information about a sapling.
In your case, method getTreeFeature is used to figure out what kind of shape the sapling(s) need to be placed so that the sapling(s) can grow (For example, dark-oak saplings cannot grow unless they're placed in shape of 2x2).
Additionally, this method relies on a class member treeFeature maintained by class SaplingBlock, which used to tell Minecraft some properties of a conversion from a sapling to a tree (like conditions to grow a tree, what kind of tree to grow). If this member is null, the Mixin class will throw an exception which cannot be expected until the mod invoke this method.
I've also experienced the mod Better End in some mod packs I like. I'd experienced these errors so I've checked the source code and found some reasons why some modders(not to any specific modder) didn't implement the member.

They leave it null because the mod they make is incomplete and haven't included a tree feature to tell Minecraft what kind of tree to grow, which would also be unable to tell the mod Saplanting how many saplings is required.
They seemed to be unwilling to use Mojang's classes to achieve some features they want, like BYG mentioned in issue #12. What I mean is that it's fantastic for all modders to achieve features via their brilliant codes, but it is a problem for me because I cannot predict all the classes created by modders from all over the world and learn how to figure out what kind of shape the spaling(s) needs to be placed from these classes.

However, mod Saplanting has a universal procedures to deal with all kinds of exceptions occurring when try to plant a sapling.

A try-catch structure surrounds the whole process of planting an sapling-item.
If any exceptions are catched during planting, this specific sapling-item instance will be marked containsError. All the instances with this mark will be excluded from planting. But any form of regeneration will remove the mark. (And of course, an exception and some info will be printed in log)
By default, if errors occurred during planting a type of a sapling, the ID of the type will be automatically recorded in a black list, which means all sapling of this type will be ignored by the mod Saplanting. This feature can be disabled by configuring.
So, it's more likely you will only witness the error once for each type of sapling.


Allow Hackney 1.9 and later

~> 1.8.0 or 1.6.5 is too strict; it should be ~> 1.8 or 1.6.5 to permit later compatible versions of Hackney.

@halostatue thank you so much for the contribution! 💖


Adding Nodes to a cluster

I've got cassandra-mesos running as a dcos package and it's working great with 3 nodes.
I want to add cassandra nodes but I cannot work out how to do it from the docs.
I've tried running:
dcos cassandra node cluster resize --json cassandra.dcos 5
which completes without any output but my cluster size and number of cassandra tasks remains the same.

it turns out I only had 4 nodes in my cluster - works fine going from 3 -> 4.
It probably would be better if it didn't fail silently though

This repository is deprecated.  Please use the latest supported framework here:  https://github.com/mesosphere/dcos-cassandra-service.


Compatibility with custom dataset

This branch was created to make the targene-pipeline compatible with datasets that are not the UKBB.

Hello there! How are we doing around here? Is it ready for review? When ready you can add me as a reviewer on all your PRs :)

To-do checklist has now been completed.

@roskamsh I've updated the new test configuration filename to match what it does, if the tests work, I'll merge and release! Thank you again for taking the time! :-)


Need to update Amazon packages for latest version 2017.03

This will be addressed with the next point release of Salt which is immanent

see https://github.com/saltstack/salt/issues/41051

This is just a matter of making the directory 2017.03 available on the site repo.saltstack.com yum/amazon/2017.03 (symbolic link to latest)

Updated repo.saltstack.com to allow for 2017.03 in amazon.
This fix did not need a new release of Salt, only an update of the website


[Paper] Refactor lava code

Refactoring the lava-based code for the paper experiments.

merged into #90 and continued there


🐛 Bug Report — Runtime APIs: No HTTP CONNECT support?

I was making a pass at implementing a very simply http-proxy using the new connect() function and when I tried calling my worker (started with wrangler dev on wrangler v3.2.0) using curl -x http://<IP_ADDRESS>:8787 https://www.example.com, I got:
workerd/server/server.c++:2363: error: Uncaught exception: kj/compat/http.c++:6983: unimplemented: CONNECT is not implemented by this HttpService

I couldn't find anything in the Cloudflare Worker docs stating whether the CONNECT http method is supported or not in a worker, but given the new connect() method, it seems like it should be.

Hey @shaunco — thanks for flagging this. Moving to the docs repo so that we clarify this.

CONNECT is not supported in CDN as well, it is not just a Workers limit.


Docker support

Thanks a lot for the great project, I'd like to contribute to it and help with the task!
Is your feature request related to a problem? Please describe.
No
Describe the solution you'd like
I'd like to package the app in a docker image
Additional context
Is there a recommended way or do you have any suggestion to package the app as static content without writing a single line of code?
Ideally it should be easy to maintain, like a Dockerfile with a simple static server
e.g. python -m  http.server 8000 to expose the static content.
The final goal is that I want to be able to package it as helm chart and deploy it on kubernetes
Thanks

Hey, thanks for wanting to contribute. The easiest way to use the static assets would be using the contents hosted on CDNs like jsdelivr. This would mean that the assets would be fetched from the jsdelivr.com domain. If you intend to load the assets within the image and serve it locally, you would need to download it from either a CDN like jsdelivr, or install the npm package for it as a dependency, which should include the assets, but that would require writing some code to get it working as expected (setting up a simple HTTP server in node to serve the assets from the altair-static dependency).

I should mention that someone has previously published a docker image for Altair https://github.com/LimeBeck/altair-docker

Thanks for the hints and the link to the repo!


eslint: fix lint issues with function order

After cleaning my node_modules folder cockpit-composer began complaining about the eslint issues fixed in this PR.

This issue seems isolated to my local env


feat(search-with-typeahead): v2 styles for search-with-typeahead

Related Ticket(s)
Closes #10790
Description
Fixes the styles for search-with-typeahead and brings the alternate variation down to 48px in height
Changelog
Changed

add sticky-search property to dotcom-shell story variation with the leadspace-with-search to be able to test out the scrolling

@annawen1 overall this looks really good! I saw a few things:
Default
- [ ] Blue line isn't accessible in dark themes
Let me know if  you need help with a token for this

- [ ]  Text indicator (vertical cursor) and text when typing into search box should be $text-primary

- [ ] Plex isn't loading in the Search input
But it is loading in the dropdown, super weird! I highlighted the as I hope you can see the slight difference between Helvetica (Input) and Plex (dropdown)?

Alternate
- [ ] Close button isn't a 48x48 square
Not quite sure what causing this

- [ ] Dividers between options
Were you able to look into bringing in the dividers between the dropdown options for the alternate style? If not no big deal!

@oliviaflory I believe things mentioned above should be fixed now!


consensus::State (de)serialization regressions

There are a couple problems with the serialization (or rather, deserialization) implemented in #676 which prevents existing consensus state JSON files from being read. (Note: we can support a new serialization, it's just the deserialization that needs to be backwards compatible)
null block_id field
tendermint_proto::serializers::optional does not appear to handle the field being explicitly null, e.g.:
{"height":"3820001","round":"0","step":2,"block_id":null}

This results in the following error:
invalid type: null, expected struct BlockId at line 1 column 50

total field of parts::Header
This was previously serialized as a string. It now appears to be serialized as an integer. This causes the following error when deserializing an existing file:
 invalid type: string "1", expected u32 at line 1 column 146

The total field was changed in Tendermint Go too. I can write a compatibility layer to be able to deserialize from string OR int, but we should keep in mind that serialization will follow the new normal.
This means that KMS can upgrade from an older version but cannot downgrade after a new version was executed.

@greg-szabo that sounds fine. worst case, people can hand-edit the files or blow them away


Detail error message

In section 4.1.1 it says the output would be "throw error", I think the error message could be added to declare what kind of error it is. Also in the initial state when it says size 1m x 1m x 1m I guess it is saying 1m * 1m * 1m?

Adding detailed error message and changing notation on size.


Added Commen Section for Interactivity

Related Issue

Issue No. #88

Changes you made :

About html
-CSS

Describe the changes :

Sir I have made a comment box using HTML and CSS only as when doing with php..I need to change the filename which I think would not be appropriate (from .html to .php) as .htaccess is not working on my local machine..
In my previous PR, I had a branch conflict so I made this by updating all and resolving it

It makes people who have went along with your services to interact with each other or raise a question regarding services.
Check list :

[x ] I have followed code style of this project.
[ x] I have read the CONTRIBUTING document.
[ x] All the aspects mentioned in the issue are covered.
[ x] The website is properly working after my changes.
[ x] 🌟 ed the repo

Type of change :

[ x] Bug fix (non-breaking change which fixes an issue)
[ x] New feature (non-breaking change which adds functionality)

Output Screenshots :

before...

after...

Please check and confirm it under Hacktoberfest 2022

Hello! Please confirm it under Hactoberfest -accepted 2022 label

Sir please confirm it and check

@Arun9739 Please confirm and merge my PR , I have contributed in improving your website!

Good to go @AryanSahoo-2003
LGTM 🌟

@AryanSahoo-2003 Your changes have created lot of problems
From next time make sure to do the changes properly.


card actions should be at the bottom when using grid and stretch

Hi, Thanks for the great libary.
I am trying to use cards of different possible heights (i.e. due to some having more content than others) and using grid to display them. When stretching the items in grid, the card height is properly adjusted, but it seems to be just the paper of the card, the card actions are not floated to the bottom.
Here is a very simple example using simplemediacard whose height has been set to %100:
https://codesandbox.io/s/8z5rn2mq72
Pleaset let me know if its possible to push the card actions to the bottom in such a setup, Thanks.

[X] This is a v1.x issue.
[X] I have searched the issues of this repository and believe that this is not a duplicate.

Expected Behavior

The CardActions should always be lined up to the bottom.
Current Behavior

CardActions are immediatelly below the contnet, which looks bad when different cards have different contents.
Steps to Reproduce

Link:

https://codesandbox.io/s/8z5rn2mq72

Context

Your Environment

Tech
Version

Material-UI
v1.5.0

React
16.4

browser
chrome

etc.

I would suggest using flexbox: https://codesandbox.io/s/y7pn35wwkx