IXMP on Apple Silicon (ARM architecture)

During Solve:
ERROR Reported: java.lang.java.lang.UnsatisfiedLinkError
Error can be due to CPU/ Java architecture mismatch. Have tried both X86 and ARM builds of JAVA, still this error persists.

Code sample or context
--- Warning: The GAMS version [41.1.0] differs from the API version [24.8.3].
java.lang.UnsatisfiedLinkError: Can't load library: /Users/iiasa/opt/anaconda3/envs/env_x86/lib/python3.9/site-packages/ixmp/backend/jdbc/libgmdjni64.dylib
at java.base/java.lang.ClassLoader.loadLibrary(ClassLoader.java:2393)
at java.base/java.lang.Runtime.load0(Runtime.java:755)
at java.base/java.lang.System.load(System.java:1953)
at com.gams.api.gmd.(gmd.java:202)
at com.gams.api.GAMSDatabase.createGMDObject(GAMSDatabase.java:342)
at com.gams.api.GAMSDatabase.initializeAttributes(GAMSDatabase.java:328)
at com.gams.api.GAMSDatabase.(GAMSDatabase.java:256)
at com.gams.api.GAMSWorkspace.addDatabase(GAMSWorkspace.java:1024)
at com.gams.api.GAMSWorkspace.addDatabase(GAMSWorkspace.java:1010)
at at.ac.iiasa.ixmp.objects.Scenario.makeGamsDB(Scenario.java:1812)
at at.ac.iiasa.ixmp.objects.MsgScenario.toGDX(MsgScenario.java:1328)
Unexpected exception formatting exception. Falling back to standard exception
Traceback (most recent call last):
File "MsgScenario.java", line 1328, in at.ac.iiasa.ixmp.objects.MsgScenario.toGDX
Exception: Java Exception
The above exception was the direct cause of the following exception:
Traceback (most recent call last):
File "/Users/iiasa/opt/anaconda3/envs/env_x86/lib/python3.9/site-packages/IPython/core/interactiveshell.py", line 3460, in run_code
exec(code_obj, self.user_global_ns, self.user_ns)
File "/var/folders/jh/k9yd8qn1389drngmnwrj91740000gp/T/ipykernel_13760/2994721627.py", line 1, in
scenario.solve()
File "/Users/iiasa/opt/anaconda3/envs/env_x86/lib/python3.9/site-packages/message_ix/core.py", line 676, in solve
super().solve(model=model, solve_options=solve_options, **kwargs)
File "/Users/iiasa/opt/anaconda3/envs/env_x86/lib/python3.9/site-packages/ixmp/core/scenario.py", line 835, in solve
model_obj.run(self)
File "/Users/iiasa/opt/anaconda3/envs/env_x86/lib/python3.9/site-packages/message_ix/models.py", line 377, in run
result = super().run(scenario)
File "/Users/iiasa/opt/anaconda3/envs/env_x86/lib/python3.9/site-packages/ixmp/model/gams.py", line 276, in run
scenario.platform._backend.write_file(
File "/Users/iiasa/opt/anaconda3/envs/env_x86/lib/python3.9/site-packages/ixmp/backend/jdbc.py", line 579, in write_file
self.jindex[ts].toGDX(str(path.parent), path.name, False)
java.lang.java.lang.UnsatisfiedLinkError: java.lang.UnsatisfiedLinkError: Can't load library: /Users/iiasa/opt/anaconda3/envs/env_x86/lib/python3.9/site-packages/ixmp/backend/jdbc/libgmdjni64.dylib

I don't know if that makes a difference, but your path includes an env_x86, while your library seems to be for 64 bit: libgmdjni64.dylib. I will have to read up on that.
In the meantime, can you please make sure to check everything suggested e.g. in this answer, so check your java.library.path and PATH environment variable and see if they include the path to the ixmp library?

I did some casual searching for "libgmdjni" and "gmdjni", and found no direct references in the ixmp_source repo, nor much on the Internet in general.
".dylib" is a macOS library file, and (as @glatterf42 suggests) I guess the one version bundled with ixmp is x86_64/amd64-specific, with no analogous file for the Apple ARM architecture.
This may be a result of the process whereby the ixmp_source JAR and its dependencies are prepared, i.e. it may pick up dependencies for certain platforms and architectures, but this set may exclude the Apple ARM architecture.
To reiterate my point during the MESSAGE meeting: since ixmp_source is effectively unmaintained (we do not have a Java developer assigned to maintain it, including fixing bugs), this may go unfixed unless boosted to higher priority.

Thanks Paul and Fridolin. It seems that Pauls analysis is correct here. One possible solution is to recompile the Java Code (Eclipse/Xcode) while specifying the ARM architecture. Another  stopgap solution that I found is to use VMWare Fusion personal license (Free for now) along with Windows 11 ARM developer preview to run the code under emulation - very low performance hit.

Steps to enable MESSAGEix model development on Apple (ARM based CPU) --- Verified 17th April 2023.
1.) Install GAMS 40 - https://www.gams.com/40/
2.) Install Miniconda -  (Miniconda3 macOS Intel x86 64-bit pkg
3.) Create conda environment (Python version 3.8)- conda create --name ME_X86_P3.8 python=3.8
3.1) Ensure that the packages being installed next are coming from "pkgs/main/osx-64" conda channel
4.) Fork and install ixmp from source - https://docs.messageix.org/projects/ixmp/en/latest/install.html#from-source
5.) Install open JDK (ver. conda-forge/osx-64::openjdk-<IP_ADDRESS>-hcf210ce_1)- conda install -c conda-forge openjdk
6.) Fork and install MESSAGEix from source - https://docs.messageix.org/en/stable/install.html#from-source
7.) install pram - pip install pyam-iamc
8.) Install Jupyterlab - conda install -c conda-forge jupyterlab
The above steps are tested for M1Pro processor.
On 13th April 2023, GAMS native installer for Apple ARM was released as Beta. This is yet to be tested.

Steps to enable MESSAGEix model development on Apple (ARM based CPU) --- Verified 17th April 2023.
1.) Install GAMS 40 - https://www.gams.com/40/
2.) Install Miniconda -  (Miniconda3 macOS Intel x86 64-bit pkg
3.) Create conda environment (Python version 3.8)- conda create --name ME_X86_P3.8 python=3.8
3.1) Ensure that the packages being installed next are coming from "pkgs/main/osx-64" conda channel
4.) Fork and install ixmp from source - https://docs.messageix.org/projects/ixmp/en/latest/install.html#from-source
5.) Install open JDK (ver. conda-forge/osx-64::openjdk-<IP_ADDRESS>-hcf210ce_1)- conda install -c conda-forge openjdk
6.) Fork and install MESSAGEix from source - https://docs.messageix.org/en/stable/install.html#from-source
7.) install pram - pip install pyam-iamc
8.) Install Jupyterlab - conda install -c conda-forge jupyterlab
The above steps are tested for M1Pro processor.
On 13th April 2023, GAMS native installer for Apple ARM was released as Beta. This is yet to be tested.

Thanks for providing us with these steps! Do you know which of them (if any) are ARM-CPU specific?
Also, do you think you'll be able to test the new GAMS native installer?

Thanks, then we can update our installation instructions ahead of the community meeting and workshop. Please also let us know what you find for the GAMS ARM native build, whenever you find the time to do so.

Hi again @SiddharthJoshi-Git, I have a few more questions before updating the installation instructions:

Looking at the miniconda installers, there are M1 ARM installers.
Which conda channels did you end up using? Could you please post the output of conda list here?
Which command did you use to change the default conda channels?
Finally, what are the default conda channels for you? Could you please simulate starting the process from scratch and check that?

Hi Fridolin,
- Looking at the miniconda installers, there are M1 ARM installers.Did they fail for you so that you had to change to the Intel x86 64-bit pkg?

There are currently two methods of installing anaconda/miniconda on macOS with ARM chip like APPLE M1/M2. The first method relies on installing M1/M2 native arm installer that defaults to pkgs/main/osx-arm64/. The second method relies on X86 installer that defaults to /pkgs/main/osx-64/ . I tried installing using first method initially which lead to the original errors. Switching to second method lead to full compatibility using rosetta translation layer (this is automatic and we don't change any setting here). I don't yet know if we can install using first method and change packages to X86 on the fly. I presume that with first method, the python version being downloaded during venv initialisation is ARM one instead of X86.
Links:
https://www.anaconda.com/blog/apple-silicon-transition/
https://www.anaconda.com/blog/new-release-anaconda-distribution-now-supporting-m1
https://docs.anaconda.com/free/anaconda/reference/packages/old-pkg-lists/2022.05/py3.8_osx-arm64/

- Which conda channels did you end up using? Could you please post the output of conda list here?

With X86 installer, I did not change any default channel. The installer automatically pointed to /pkgs/main/osx-64/  repository.
Some outputs:

- Which command did you use to change the default conda channels?
Please see above. To summarise, the default channel is _**conda-forge/pypi**_, and the repository is _**https://repo.anaconda.com/pkgs/main/osx-64**_. No command was used to change the conda default channels and the installer takes care of this automatically
- Finally, what are the default conda channels for you? Could you please simulate starting the process from scratch and check that?
Please see above. Attached is the output of conda list

Maybe a test is still required to check if ARM installer can be coerced to install packages from X86 repository. This way all the native packages can be ARM (Faster speed) and all the MESSAGEix requirements come from X86 builds.

Thanks a lot, I updated the installation instructions accordingly.

Steps to enable MESSAGEix model development on Apple (ARM based CPU) --- Verified 11th April 2024.
1.) Install GAMS 46.4 using the package installer for macOS on Intel CPUs. Ensure you select 'customize' during installation and set the tick box to add to your Path variable, your .zprofile should have an entry that looks like this after installation:
PATH="/Library/Frameworks/GAMS.framework/Resources:${PATH}"
export PATH
Manual edits are likely to fail.
2.) Install Miniconda (Miniconda3 macOS Intel x86 64-bit pkg)- https://repo.anaconda.com/miniconda/Miniconda3-latest-MacOSX-x86_64.pkg
3.) Create and activate conda environment (Python version 3.10) -
$ conda create --name ME_X86_P3.10 python=3.10
$ conda activate ME_X86_P3.10
4.) Check that the packages being installed next are coming from the "pkgs/main/osx-64" conda channel using $ conda info
5.) Fork or clone and install ixmp from source - https://docs.messageix.org/projects/ixmp/en/latest/install.html#from-source
6.) Install open JDK (ver. conda-forge/osx-64::openjdk-<IP_ADDRESS>-hcf210ce_1) using $ conda install -c conda-forge openjdk
7.1) Run $ ixmp show-versions. This should show gams 46.4.
7.2) Navigate to ixmp repository and run $ pytest. This should pass.
8.) Fork or clone and install MESSAGEix from source - https://docs.messageix.org/en/stable/install.html#from-source
9.) Install pyam - $ pip install pyam-iamc
10.) Install Jupyterlab - $ conda install -c conda-forge jupyterlab
11.) Navigate to message_ix repository and run $ pytest. This should pass.


show error from TR_TADIR_INTERFACE

show error from TR_TADIR_INTERFACE
METHOD set_default_package + METHOD get_adt_objects_and_names moved due to SE80 vs eclipse

conflict, lets just close


Blackhole Controller with Blackhole Tanks only outputting fluid of first slot

Industrial Foregoing Version: 1.12.2-1.12.11-235
TeslaCoreLib Version: 1.12.2-1.0.15
When connecting an Ender Fluid Conduit from EnderIO (which can transfer any fluid) to a Blackhole Controller that contains multiple Blackhole Tanks, the controller will only output the fluid from its first slot, even if all other tanks receiving that fluid are full and there are other tanks that contain a different fluid that aren't full. Not sure if this is an issue with EnderIO or this mod, could be an edge case since most fluid transfer systems lock their pipes to one fluid and actually contain that fluid, while EnderIO transfer is instant without having any items/fluids in the actual pipes.

It works fine with other types of extraction methods like Industrial Foregoing conveyors


durable load balanced stream?

One use case I have for liftbridge is a scale out pub/sub model I can use to load balance across a backend (similar to typical rabbitMQ use cases).  The documentation for liftbridge and the SDK documentation seem to indicate that having a load balanced group listening on the same stream is possible, but I can't figure out how that is done. The closest I've seen is the SetCursor option, but that doesn't appear to be implemented in the client returned by Connect.
It looks like there is something called an ActivityStream that was recently introduced into liftbridge, but not available in the go API. Is that something that accomplishes what I'm looking for and I just need to patiently wait?

A load-balance group is a set of streams that receive messages from a NATS subject in round-robin fashion. This is done by creating multiple streams with the same subject and using the Group option with the same name. You could use this pattern to balance messages amongst a set of consumers.

A load-balance group is a set of streams that receive messages from a NATS subject in round-robin fashion. This is done by creating multiple streams with the same subject and using the Group option with the same name. You could use this pattern to balance messages amongst a set of consumers.

But the Publish API requires that you send to a stream name and not a nats subject?
Provided I'm reading the wrong thing and I need to call something else, how does the consumer checkpoint so Liftbridge knows that the message was successfully delivered?

Yeah, that is correct. It's more meant for the NATS layer, i.e. publishing with NATS clients to NATS subjects, rather than the Liftbridge stream layer. This probably needs to be documented better.
If you are working only at the streams layer, the better approach might be to use a single stream that is partitioned, then have a single consumer for each partition which checkpoints it's position. This pattern will become a lot more seamless with once consumer groups are available.

OK, i'll check back in a few months then, thanks


How to add ui-tree-handle dynamically?

Hi all,
I am trying to implement a feature when the element is click/selected by pressing a checkbox. i would like to add the ui-tree-handle directive into the element. is that posible?
Thanks!

Hello Kiann, you can use an "ng-attr" or an "ng-switch" to conditionally apply the ui-tree-handle attribute.
You should also be aware of the default behaviour for a tree that doesnt contain a ui-tree-handle.

If you don't add a ui-tree-handle for a node, the entire node can be dragged.

GLHF


[Enhancement] Update logging in all functions to utilize Sawmill

We not have a logger! Yay333t!! Time to do some proper clean up with all the log statements in the backend 👍

@LeviHarrison down for this as well?

Following up on our conversation over Discord yesterday, I think that there might have to be a few changes to Sawmill in order to support Emoji in logs. We could technically append them on the error messages that we pass to LogErr, but that's a little bit messy, and it would also lead to them showing up in the GCP Logs Viewer, where they aren't really needed. That's why I propose that we add one more parameter to LogErr which either takes an emoji or a type representing that emoji, such as sawmill.Network, which could be 🌐 and represent a network error. Thoughts?

Hey sorry I know I'm behind on this. Will take a look at this after the stream!

Okay so:

I feel like the logging in GCP would be valid for this as well no? Would definitely love to understand your reasoning behind this
I did something similar to what you are proposing for iOS! Essentially have some set emojis that are always added to the front of the log based on whether it was info/warning/error etc. Could you do something similar to this where we allow people to pass in an optional emoji for different severity levels and such?

Yes definitely feel free to make changes to Sawmill as needed!

 I did in turn pull your changes (it was easier). Hopefully everything should be good to go :D. I do need to rev the version for some pkgs for proper deployment to Firebase. But good to go!


Ina layout

Hi
The user-submitted Ina layout on your site has two N keys, and no -_ key. Not sure if the letters next to first N should shift left to add the missing key to the end of the row ...
Thanks.

Good catch. I'm not sure what their final intention was, but I've updated the layout to remove the duplicate key and replace it with a -_ key.


ERROR: Board can accept larger flash images

I tried to flash the program to my pix falcon on ubuntu 16.04 using
make  px4_fmu-v2_default upload.
However, I get the error and failed
ERROR: Board can accept larger flash images (2080768 bytes) than board config (1032192 bytes). Please use the correct board configuration to avoid lacking critical functionality.
When I used
make  px4_fmu-v3_default upload,
it works. Is there something wrong with the version of FMU?
Thanks for your hint.

No, you did what you were supposed to do.


Update bitflags and remove TryFrom<()> impls

Summary
The TryFrom<()> impls are a bit of a weird construct, and they now trigger build failures on nightly. By the same occasion also update bitflags and fix some bugs preventing the USB peripheral from building for SAMD11 targets.
Checklist

[ ] CHANGELOG.md for the BSP or HAL updated Not necessary - all changes are internal APIs only
[x] All new or modified code is well documented, especially public items
[x] No new warnings or clippy suggestions have been introduced

@ianrrees this may help getting CI green again :)

Cheers!  Yeah I've done the same thing with the bitflags just was waiting until we had the CI thing sorted to PR.
Minor suggestion - could we put the english name of the flags in the docstrings, eg ISF to ISF - Inconsistent Sync Field?

Cheers! Yeah I've done the same thing with the bitflags just was waiting until we had the CI thing sorted to PR.
Minor suggestion - could we put the english name of the flags in the docstrings, eg ISF to ISF - Inconsistent Sync Field?

Done!

  Hope you still get the GitHub points @jbeaurivage !


Check for submissionsEnabled on Submit page

It was showing the Submit form (the API endpoint is already checking if the submissions are enabled).

Nice catch Damu! ♥
Not sure why I implemented the submissionsEnabled check in the SubmitButton component, I think it's better your approach to not show the submission form, since it can have innacurate data from other hackathons🙏
Should we delete the submissionsEnabled logic from the button since it shouldn't be needed anymore?
Nitpick: pushed a styling change to have the same style of /admin "Access Denied" msg.

Removed the submissionsEnabled logic from the button!
Thanks!!

Removed the submissionsEnabled logic from the button!

Great! Merging 🙏


Invoke-WebRequest/RestMethod: Improve reliability of CTRL-C tests

Closes #19531

Lengthen the race window from milliseconds to seconds.
Don't sleep any longer than necessary if a task has already completed.

PR Summary
Addresses request by @daxian-dbw in https://github.com/PowerShell/PowerShell/pull/19330
Raised as #19531

PR Context
CTRL C tests rely races on timeouts that had time-frames in 100s of milliseconds. These may be too tight when running in CI where resources are stretched. Increased the timeouts and changed logic so it only waits as long as necessary if possible.
PR Checklist

[x] PR has a meaningful title

Use the present tense and imperative mood when describing your changes

[x] Summarized changes
[x] Make sure all .h, .cpp, .cs, .ps1 and .psm1 files have the correct copyright header
[ ] This PR is ready to merge and is not Work in Progress.

If the PR is work in progress, please add the prefix WIP: or [ WIP ] to the beginning of the title (the WIP bot will keep its status check at Pending while the prefix is present) and remove the prefix when the PR is ready.

Breaking changes

[x] None
OR
[ ] Experimental feature(s) needed

[ ] Experimental feature name(s):

User-facing changes

[ ] Not Applicable
OR
[ ] Documentation needed

[ ] Issue filed:

Testing - New and feature

[x] N/A or can only be tested interactively
OR
[ ] Make sure you've added a new test if existing tests do not effectively test the code changed

Tooling

[x] I have considered the user experience from a tooling perspective and don't believe tooling will be impacted.
OR
[ ] I have considered the user experience from a tooling perspective and opened an issue in the relevant tool repository. This may include:

[ ] Impact on PowerShell Editor Services which is used in the PowerShell extension for VSCode
(which runs in a different PS Host).

[ ] Issue filed:

[ ] Impact on Completions (both in the console and in editors) - one of PowerShell's most powerful features.

[ ] Issue filed:

[ ] Impact on PSScriptAnalyzer (which provides linting & formatting in the editor extensions).

[ ] Issue filed:

[ ] Impact on EditorSyntax (which provides syntax highlighting with in VSCode, GitHub, and many other editors).

[ ] Issue filed:

I'm seeing failure of an unrelated test on MacOS:
Test-Connection.Ping.Quiet works
at <ScriptBlock>, /Users/runner/work/1/s/test/powershell/Modules/Microsoft.PowerShell.Management/Test-Connection.Tests.ps1: line 90
90:             $result2 | Should -BeFalse

@daxian-dbw  I have made all changes as suggested in case this is exactly what you wanted. Let me know if you want me to dial back the delay a bit so the tests don't take so long to run. I split the brotli/deflate/gzip tests into separate cases as well.


Compiler reporting success despite a change in a module which invalidates another module

Using Elm 0.19 on Windows 10 Pro editing in VS Code, browser is Edge.
If you successfully compile a project, then make a change to a module which would cause another module not to compile, but do not change that affected module, recompiling the project reports success and running it causing uncertain results.
The example below results in nothing showing in the browser, but in my larger project I am getting undefined, which Elm promises I will not get!

To reproduce, create the index.html, Main.elm, MyModule.elm, and initial ChangedModule.elm from below, compile and run. Not too exciting.
Then change ChangedModule.elm as shown below. Recompiling reports success, but running now shows nothing.
index.html:
<!DOCTYPE html>

    <meta charset="UTF-8">

Main.elm:
module Main exposing (main)

import Html exposing (..)
import MyModule

view model =
    div []
        [ h1 [] [ text "First view" ]
        , MyModule.view
        ]

main =
    view ""

MyModule.elm:
module MyModule exposing (view)

import ChangedModule
import Html exposing (p, text)

view =
    ChangedModule.view "Mark"

initial ChangedModule.elm:
module ChangedModule exposing (view)

import Html exposing (Html, p, text)

view : String -> Html msg
view name =
    p [] [ text ("Your name is " ++ name) ]

final ChangedModule.elm:
module ChangedModule exposing (view)

import Html exposing (Html, p, text)

view : Bool -> String -> Html msg
view upperCase name =
    if upperCase then
        p [] [ text ("Your name is " ++ String.toUpper name) ]

    else
        p [] [ text ("Your name is " ++ name) ]

I tried this on my machine, but I get a compiler error after changing ChangedModule.elm.  It did compile successfully the first time, before I changed the `ChangedModule.elm, I am on a mac running 0.19.0.
Detected errors in 1 module.
-- TYPE MISMATCH ---------------------------------------------- src/MyModule.elm

The 1st argument to `view` is not what I expect:

8|     ChangedModule.view "Mark"
                          ^^^^^^
This argument is a string of type:

    String

But `view` needs the 1st argument to be:

    Bool

Hint: Elm does not have “truthiness” such that ints and strings and lists are
automatically converted to booleans. Do that conversion explicitly!```

Are you compiling with `elm make`, or something else?

I am using elm make from the terminal window in VS Code.
Must be something to do with Windows 10 or VS Code or both.
If I find out what it is, I'll report back.

I can give it a shot on a Windows machine tonight and report back as well.

I've successfully reproduced this using my own code.
Code, issue details and everything else you need can be found here:
https://github.com/icidasset/elm-0.19-issue
Let me know if I missed anything.
(Sidenote: this issue has been popping up in multiple projects)

@icidasset I couldn't reproduce using your project, but I think I experienced something similar after upgrading a dependency to a new major version.
What environment do you recreate this in?
I'll try to recreate my experience in an SSCCE too.

@absynce Hey. I can still reproduce the same issue using that project of mine. Did you follow the steps/procedure in the readme?
Environment:
macOS 10.14 (when I project, this was 10.13)
Elm 0.19.0 using asdf version manager
Let me know if you need more info.

Did some more testing on this (using my https://github.com/icidasset/elm-0.19-issue code)
I only have the issue when the elmjutsu plugin is active.
Tested this again with the new 0.19.1 alpha and didn't have the issue.
This is most likely related to the "filelock problem" which is addressed in 0.19.1.

I believe this is the same root problem as https://github.com/elm/compiler/issues/1956 which was fixed across these three commits:

https://github.com/elm/compiler/commit/a74201c5944c057a20ce5f3de08903c3cd7dd094
https://github.com/elm/compiler/commit/884ea201f2c2f6f478df3b311904acb8c02d538a
https://github.com/elm/compiler/commit/fa1971bf72b4798111e67a66dd56cbade6bbb19a

Can someone confirm that the particular manifestation here is fixed by 0.19.1?


Conversions makes overly strong assumptions and are more confusing than beneficial

For instance the Histogram implicitly assumes it's measuring a duration, which is a pretty strong assumption. Actually in your own example here https://metrics.dropwizard.io/4.0.0/getting-started.html#Histograms it's measuring bytes, which the conversion to duration will then turn into... what exactly?
I think I'd prefer a solution that did not try to convert at all, and just logged/reported whatever you gave it. I think the conversions are more confusing than beneficial.

Hello @rasmusbergpalm
Could you point to concrete lines in the source code of Histogram or Reservior where you see assumptions about the nature of measured numbers? I do not see them at all.

I cannot. Maybe it's been fixed or I've been confused. Thanks! I'll close the issue.


windows

Just had to make some minor changes and now this works on my windows machine, so I figured I should let the repository owner know.

i did every changes but not getting this error
"tbselenium.exceptions.TBDriverPortError: SOCKS port 9050 is not listening"
what should i do?

Make sure that you are using the right versions of tor browser, selenium and geckodriver. If you are, try using the socks_port argument to use a different port. I also made those modifications 7 months ago so they may not work on the most recent version of tbselenium.

@kjmcgrinder Sorry, I couldn't respond earlier to this PR. I may be interested in merging your code, but first you'll need to rebase your code and put Windows-specific code behind an if clause like if os.name == 'nt'

@gunesacar no worries, I'll see if I can make it a little cleaner so that it can run on windows OR linux when I get a chance and make a new pull request with the most recent commit.


Feature Request: Capability to download the audio provided by ChatGPT. READ ALOUD

Please add the capability to download the read aloud audio

added todo list 😊

+1


Change markdown library

We're punting on v3.3.3 right?

@mdo Right. We're skipping over it since bootstrap-sass had to use it for their Sass-specific emergency post-v3.3.2 bugfix.


[Deathwatch] [Bugfix] Fix renown attribute not saved

Submission Checklist
Required

[x] The pull request title clearly contains the name of the sheet I am editing.
[x] The pull request title clearly states the type of change I am submitting (New Sheet/New Feature/Bugfix/etc.).
[x] The pull request makes changes to files in only one sub-folder.
[x] The pull request does not contain changes to any json files in the translations folder (translation.json is permitted)

New Sheet Details

The name of this game is: <   >  (i.e. Dungeons & Dragons 5th Edition, The Dresden Files RPG)

The publisher of this game is: <   > (i.e. Wizards of the Coast, Evil Hat)

The name of this game system/family is: <   > (i.e. Dungeons & Dragons, FATE)

[x] I have followed the Character Sheets Standards when building this sheet.

[ ] I have authorization from the game's publisher to make this an official sheet on Roll20 with their name attached.
[ ] This game is not a traditionally published game, but a copy of the game rules can be purchased/downloaded/found at: <   >
[ ] This sheet is for an unofficial fan game, modification to an existing game, or a homebrew system.

Changes / Description

Save the renown attribute that wasn't saved since #9259
(I don't mean to shame the PR author. He was right to change the name, but he made a mistake that wasn't seen in the review)

Character Sheet Info Roll20 Internal Use only.


Failed to create local audio stream, source is not of type AudioDeviceInfo or MediaStreamTrack

Package Name: "@azure/communication-calling":
Package Version:  "1.0.0-beta.4",
Operating system: MacOS M1
[x] nodejs

version: v16.15.0

[x] browser

name/version: Edge Version 110.0.1587.49 (Official build) (arm64)

[ ] typescript

version:

Is the bug related to documentation in

[ ] README.md
[ ] source code documentation
[x] SDK API docs on https://docs.microsoft.com

Describe the bug
A clear and concise description of what the bug is.
https://learn.microsoft.com/en-us/azure/communication-services/quickstarts/voice-video-calling/get-started-raw-media-access?pivots=platform-web#place-a-call-with-a-custom-audio-stream
If you follow the above tutorial and use the LocalAudioStream and pass the custom audio stream, this fails with the following error
The sample code

    const createBeepAudioStreamToSend = () => {
        const context = new AudioContext();
        const dest = context.createMediaStreamDestination();
        const os = context.createOscillator();
        os.type = 'sine';
        os.frequency.value = 500;
        os.connect(dest);
        os.start();
        const { stream } = dest;
        return stream;
    };

    const AudioStream =  createBeepAudioStreamToSend();
    const localAudioStream = new LocalAudioStream(AudioStream);

Error
Uncaught (in promise) CallingCommunicationError: Failed to create local audio stream, source is not of type AudioDeviceInfo or MediaStreamTrack
    at new LocalAudioStream (sdk.bundle.js:166:288998)

To Reproduce
Steps to reproduce the behavior:

Follow the above tutorial and use the LocalAudioStream

Expected behavior
A clear and concise description of what you expected to happen.
LocalAudioStream only accepts AudioDeviceInfo or MediaStreamTrack as parameters, this should either be changed to support a MediaStream as well or the docs can be changed to this
const localAudioStream = new LocalAudioStream(AudioStream.getAudioTracks()[0]); which would pass in the MediaStreamTrack into the method.
Screenshots
If applicable, add screenshots to help explain your problem.
Additional context
Add any other context about the problem here.

@nellyk RAW audio is available in SDK version > 1.10.0-beta.1 ( https://learn.microsoft.com/en-us/azure/communication-services/quickstarts/voice-video-calling/get-started-raw-media-access?pivots=platform-web )
can you please try to switch and let us know if it works

Thank you upgrading to the version 1.10.0-beta.1 resolved the issue.
Only issue i can see now is that the error thrown is still requiring you to send a MediaStreamTrack although the method now accepts AudioDeviceInfo and MediaStream

Can you upgrade to 1.10.1-beta.1 please? There was an issue that was hotfixed.


undefined is not an object (evaluating 'e.createElementNS') React-Native

When I import the "import ApexCharts from 'apexcharts' "  undefined is not an object (evaluating 'e.createElementNS') React-Native this error occurs.

The library has no direct support for React-Native as of now. The undefined error is due to document not being available which will only be present in browsers.

ok, so how can it work for react-native. Can you suggest me any changes in code??

no news ? @sumitaryavrat you made it work on react-native ?


Enable github pages for the docs/ directory?

The website http://phaser.io/docs suggests:

These docs are for Phaser 2.6.2
If you're using Phaser CE then download the CE repository and open docs/index.html

And since GitHub now supports docs/ directory for zero config github pages, can we turn that on for easier documentation viewing?

Yeah good idea. Done.


chore: use error.New when error without format

@a8m, hello!
Please forgive me, but this case again


[video_player] android: add tests to make sure playing local assets is successful

We should add tests to make sure the local assets could be loaded by the video player plugin.
Issue: https://github.com/flutter/plugins/pull/3123
This PR fixed the issue, but was landed without tests: https://github.com/flutter/plugins/pull/3123
There is already a test available in the plugin, we just couldn't enable it on CI due to some issues. https://github.com/flutter/plugins/blob/master/packages/video_player/video_player/example/test_driver/video_player_test.dart#L17

Since it's been a while, we might need to re-investigate the test to see if we can enable it now https://github.com/flutter/plugins/blob/master/packages/video_player/video_player/example/test_driver/video_player_test.dart#L17

This should've been closed when https://github.com/flutter/plugins/pull/3219 was merged, right?


Connector RC Release: S3

All Connectors
These requirements are imposed on every connector, regardless of the selected features/tests in the above table.

[ ] Core Arrow Data Types are supported
[ ] For exceptions where a Core Arrow Data Type is not supported, print a warning and ignore the column instead of crashing or failing the query.
[ ] All known Major Bugs are resolved

Documentation

[ ] Documentation includes all information and steps for a user to set up the connector.
[ ] Documentation includes all known issues/limitations for the connector.
[ ] Documentation includes any exceptions made to allow this connector to reach RC quality (e.g. if a particular data type cannot be supported by the connector).
[x] The connector has an easy to follow quickstart.

Testing

[x] Integration tests to cover simple use cases based on the connector type, e.g. a Spicepod sourcing a file from an FTP server, reading latest GitHub issues, etc.

Conditional Criteria
The following features/tests are dependent on the required features/tests for the specified connector, from the Connector criteria table
Streaming

[x] Data is streamed when reading/performing queries from this connector.

Schema Inference
Support for schema inference in a connector is classified as:

✅: Full coverage. The connector supports native schema inference.
⚠: Partial coverage. The connector does not support native schema inference, but it requires no inference (e.g. static schema, etc).
❌: No coverage. The connector does not support native schema inference, and typically infers schema from the first row of results.

No Coverage

[x] The connector does not support native schema inference.

Federation
Support for federation in a connector is classified as:

✅: Full coverage. The connector supports full federation and query push down.
⚠: Partial coverage. The connector supports partial federation and query push down.
❌: No coverage. The connector does not support federation or query push down.

Partial Coverage

[x] The connector supports filter federation within a single dataset for common use case columns (e.g. select * from my_dataset where id = 1)

Common use case should be determined at the discretion of the DRI for the connector.
For example, the GitHub connector should support filter federation for the author, state and title of issues.

Test Coverage
Indexes are not required for test coverage, but can be introduced if required for tests to pass (e.g. due to performance characteristics, etc).
TPC-H

[x] End-to-end test to cover connecting to TPC-H SF1 for the connector type and benchmarking TPC-H queries (official and simple).

[ ] Connectors should run all queries with no Major Bugs.

[ ] ~A test script exists that can load TPC-H data at the designated scale factor into this connector.~
[x] The connector can load TPC-H at the designated scale factor, and can run all queries with no Major Bugs.
[ ] All Minor Bugs for TPC-H are raised as issues.

TPC-DS

[x] End-to-end test to cover connecting to TPC-H SF1 for the connector type and benchmarking TPC-DS queries (official and simple).

[ ] Connectors should run all queries with no Major Bugs.

[ ] ~A test script exists that can load TPC-DS data at the designated scale factor into this connector.~
[x] The connector can load TPC-DS at the designated scale factor, and can run all queries with no Major Bugs.
[ ] All Minor Bugs for TPC-DS are raised as issues.

ClickBench

[x] A test script exists that can load ClickBench data into this connector at the designated scale factor. Not applicable for S3. Simple file upload.
[x] All queries are attempted on this connector. No bug fixes are required for ClickBench.
[x] All ClickBench bugs are raised as issues.

Target EOW: Done-Done


PGRST_SERVER_PROXY_URI in docker-compose instructions is incorrect

I'm very new to PostgREST, so this might be wrong, but I think the instructions for the docker-compose install at https://postgrest.org/en/v7.0.0/install.html#docker are incorrect. In that compose file example, it specifies PGRST_SERVER_PROXY_URI should be http://<IP_ADDRESS>:3000. Regardless of what I put in this field, my openAPI response always showed the host as "<IP_ADDRESS>". After some poking around the code, it looks like this environment variable should actually be PGRST_OPENAPI_SERVER_PROXY_URI instead.
If someone can confirm this, I'm happy to make a PR to fix it in the docs, but I want to make sure this is the correct thing to do, first.

Yes, that's correct, this has been changed. PR welcome!

Ok, submitted!


Generate password

Added some javascript for random password generation
When creating or editing a user, you can either assign a password or just click the "Generate password" button which just fills out both password fields and turns their types into text so they are visible.

Thanks for your contribution! :) I've moved the button into the column of the password input fields.


Add Kong Kubernetes Ingress Controller conformance report for v0.8.1

What type of PR is this?
/kind documentation
/area conformance

What this PR does / why we need it:
Adds Kong Kubernetes Ingress Controller conformance report for v0.8.1.
Does this PR introduce a user-facing change?:

NONE

My overlooking @sunjayBhatia, fixed

Kong is passing the GatewayClassObservedGenerationBump test. We need to enable it. Setting this PR to hold to avoid merging without fixing it.
/hold /ok-to-test

I'll add you as a reviewer @mlavacca since you've got relevant context
/cc @mlavacca

whoops, nice catch @shaneutt!
/lgtm


Feature/onboarding profile

create a profile with profile picture and nick name

Codecov Report

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

@@          Coverage Diff           @@
##           master     #33   +/-   ##
======================================
  Coverage    62.5%   62.5%
======================================
  Files          20      20
  Lines         240     240
======================================
  Hits          150     150
  Misses         90      90

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 44d96cd...e016614. Read the comment docs.


Build Errors

When I try to build my iOS Cordova app after installing this plugin I now get the following error.
Any help would greatly be appreciated.
ld: symbol(s) not found for architecture x86_64
clang: error: linker command failed with exit code 1 (use -v to see invocation)
xcodebuild: Command failed with exit code 65

Even i am facing the same issue

@ddeakes  and @DigvijayJaswal , please which command did you use to install the plugin?
Maybe you have pulled from the wrong branch?
Please make sure you use cordova plugin add https://github.com/MenelicSoftware/cordova-plugin-brother-label-printer --save

@arcadius Thanks for the reply, that however still gives the same error when trying to run:
cordova build ios --save
I get:
ld: symbol(s) not found for architecture x86_64
clang: error
: linker command failed with exit code 1 (use -v to see invocation)

@ddeakes  i was able to make it work by adding libc inside Linked Libraries

duplicate symbol '_kDeviceType' in:
cordova-plugin-bxlbgatedevice/libMPosSDK.a(BxlDeviceObject.o)
BRPtouchPrinterKit.framework/BRPtouchPrinterKit(deviceSearchByBonjour.o)
ld: 1 duplicate symbol for architecture arm64
clang: error: linker command failed with exit code 1 (use -v to see invocation)
Not able to build, could you please help us resolve.

@sanarm123, Seems the problem here is that you are using 2 SDKs from different printer vendors and each SDK has the same duplicate symbol. One or the other SDK will likely have to change the name of the constant to avoid this problem. Can you provide your contact info and I will reach out to you outside of GitHub to discuss further.

Getting the same error : Undefined symbols for architecture x86_64:
whats the fix?

@vanerleo It seems from other answers above that the fix is to add libc++ as a linked library. Have you tried that?

@robr2112 the only way i found to fix this is to add
        <framework src="libstdc++.dylib" />

plugins/cordova-plugin-brother-label-printer/plugin.xml
on line 90

@robr2112
When using IOS emulator should findPrinters return printer attached to the Host MAC via USB?

@vanerleo, No, I would not expect simulator to find USB-connected printers. It might find using Ethernet if you're looking for a cable-based setup.

when building a release version getting an error
building for iOS Simulator, but linking in object file built for iOS, file 'Plugins/cordova-plugin-brother-label-printer/BRPtouchPrinterKit.framework/BRPtouchPrinterKit' for architecture arm64

any solutions for this?

@vanerleo, sorry I'm not sure. Sounds like a build system issue. Have you tried doing a "clean" or "rebuild all"? Or deleting "Derived Data" for your app?

@vanerleo Have you solved this yet? Are you using Xcode 12 by any chance? I just ran into a similar problem with Xcode 12.0.0. Problem doesn't happen with Xcode 11.6. Haven't tried 12.0.1 yet to see if it fixes this, but doesn't seem likely. The following StackOverflow post may provide an answer. I'm still reviewing this myself. https://stackoverflow.com/questions/63607158/xcode-12-building-for-ios-simulator-but-linking-in-object-file-built-for-ios

@robr2112 using Xcode Version 12.1 (12A7403)

@vanerleo OK, refer to the StackOverflow post I mentioned before. This is a new problem with Xcode12, which you will typically encounter when you build for Simulator in the Release configuration (which creates a Universal Binary, i.e. "Build Active Architecture Only" = NO). You may not have the problem when building for Debug configuration....unless you want to use a simulator that has arm64 processor...since Debug configuration will typically not try to create a UB. The SO post describes some ways to fix this for "Release" configuration by modifying the "Excluded Architectures" setting to exclude arm64 when building for Simulators.
I have verified that this does work, at least on a Native iOS project using only Xcode and not Cordova. I don't know how to resolve this in Cordova dev environment, as I am not a Cordova developer.
-Rob

@vanerleo, great that you could fix by moving to Ionic-Capacitor. Not sure why Cordova is not working? But at least you found a solution.
About the iOS13.8+ problem with WIFI Search:
I know that iOS14+ requires adding new settings for Bonjour Services to the Info.plist to allow search.
Didn't know this (may have) started at 13.8.
Please add the following information in the Info.plist:

Privacy – Local Network Usage Description

NOTE: IMO, this is optional only to customize the default system message if you want, but Apple might require it. Anyway, Apple now requires users to "opt in" to "Local Network" usage in Apps, for user privacy concerns.

Bonjour services (This is what will enable Search to work again)

_pdl-datastream._tcp
_printer._tcp
_ipp._tcp

closing as @vanerleo managed to get it working


Jd/hotfixes

Fix README example
Fix Dockerfile
Add back the swagger docs

Codecov Report

Merging #77 (a52d061) into main (0902a3a) will increase coverage by 0.32%.
The diff coverage is 100.00%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
@@            Coverage Diff             @@
##             main      #77      +/-   ##
==========================================
+ Coverage   68.40%   68.72%   +0.32%
==========================================
  Files           2        2
  Lines         288      291       +3
==========================================
+ Hits          197      200       +3
  Misses         91       91

Impacted Files
Coverage Δ

src/SimulationService.jl
75.47% <100.00%> (+0.35%)
:arrow_up:

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Un-mess up msys2's helpful PATH protection

:+1:

@mwrock : I changed it up.  I'm modifying omnibus in the mean time to pass the extra flags needed :)


Improve readability of parameter sweep

File: include/vipra/parameter_sweep/parameter_sweep.hpp
The parameter sweep is hard to read, need to improve the readability.

removing compile time polymorphism removed the majority of complexity here


Debug Develop code to verify Test cases

Our dev code is breaking because of test written please verify it and fix thhe test written.

ETA will be 19 jul


SSO Logout (Old 18)

In gitlab by @A-Lawrence on Aug 31, 2015, 01:10
When members log out of a subsystem, they should be sent to:
sso.vatsim-uk.co.uk/sso/auth/logout?ssoKey=Helpdesk&returnURL=http://somewhere.com
Here they should be offered the chance to logout of just that system or all systems.

In gitlab by @A-Lawrence on Aug 31, 2015, 01:18
Removing milestone: RELEASE :: 1.0.7 (automated comment)
ALawrence66 - 2013-09-29T01:14:05.708240+00:00

In gitlab by @A-Lawrence on Aug 31, 2015, 01:18
Corrected erroneous function calls for SSO Logout.  Resolve issue #488658 (18)
ÔåÆ <>
ALawrence66 - 2013-07-27T19:39:24.677146+00:00

In gitlab by @A-Lawrence on Aug 31, 2015, 01:18
Resolved issue #488658 (18) issue #19 issue #488657 (20) issue #21 issue #488656 (22) issue #23
ÔåÆ <<cset 9e27bee878b9>>
ALawrence66 - 2013-07-27T17:32:10.791479+00:00


Avieth/network merge 2

Description
A part of the incremental merge of networking/develop into develop.
Linked issue
N/A. This is basically custodial work.
Type of change

[~] 🐞 Bug fix (non-breaking change which fixes an issue)
[x] 🛠 New feature (non-breaking change which adds functionality)
[~] ⚠ Breaking change (fix or feature that would cause existing functionality to change)
[x] 🏭 Refactoring that does not change existing functionality but does improve things like code readability, structure etc
[ ] 🔨 New or improved tests for existing code
[ ] ⛑ git-flow chore (backport, hotfix, etc)

Developer checklist

[ ] I have read the style guide document, and my code follows the code style of this project.
[ ] If my code deals with exceptions, it follows the guidelines.
[ ] I have updated any documentation accordingly, if needed. Documentation changes can be reflected in opening a PR on cardanodocs.com, amending the inline Haddock comments, any relevant README file or one of the document listed in the docs directory.

Testing checklist

[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.

QA Steps

Screenshots (if available)

@coot @karknu @erikd PTAL

The nix build should be fixed if you rebase against develop to include #2835 which has already been merged.

CI failing with
Building cardano-sl verbosely..
--
  | + nix-build -A cardano-sl -o cardano-sl.root --argstr gitrev 7b609dbf40f1b569abe6c048bb7397dab6fe320d --argstr buildId 4134
  | error: infinite recursion encountered, at undefined position
  | (use ‘--show-trace’ to show detailed location information)

@iohk-devops Help please. Is this my fault?

@iohk-devops Help please. Is this my fault?

Probably not. Make sure your branch has the commit in #2835 .

It looks like CI might finally pass! Any objections to merging this @erikd @dcoutts ? Or can I get an approval?

Yeah, AppVeyor living up to the "irritation" part of its name.

There's a failing wallet test:
Test.Pos.Wallet.Web.Tracking.Sync.Pos.Wallet.Web.Tracking.evalChange Outgoing transaction from account to the same account.
It fails with seed 624026529 even on develop. I've been looking at fixing it but it's very weird.

It is super weird. We have a story in the backlog for investigating it, CSL-2473


Scully fails to render route from component with no styles/styleUrls defined

🐞 Bug report
Description
Running Scully with a component that doesn't include a styles or styleUrls property in the component decorator metadata fails to render the content.
I came across this issue while making single-file components, and removing unused properties from the component decorator metadata.
🔬 Minimal Reproduction
https://github.com/brandonroberts/scully-styles-bug

Clone repo
Install dependencies with yarn
Note that src/app/blog/blog.component.ts is missing a styles or styleUrls property in the component metadata
Run ng build
Run npm run scully
Note that scully fails to render the blog page because it can't find <scully-content>
Open src/app/blog/blog.component.ts and add styles: [''] to component metadata
Run ng build
Run npm run scully
Note that Scully completes successfully

It also fails if styles is defined as an empty array.
💻Your Environment
Angular Version:

    / \   _ __   __ _ _   _| | __ _ _ __     / ___| |   |_ _|
   / △ \ | '_ \ / _` | | | | |/ _` | '__|   | |   | |    | |
  / ___ \| | | | (_| | |_| | | (_| | |      | |___| |___ | |
 /_/   \_\_| |_|\__, |\__,_|_|\__,_|_|       \____|_____|___|
                |___/

Angular CLI: 9.1.9
Node: 12.18.0
OS: linux x64

Angular: 9.1.11
... animations, common, compiler, compiler-cli, core, forms
... platform-browser, platform-browser-dynamic, router
Ivy Workspace: Yes

Package                           Version
-----------------------------------------------------------
@angular-devkit/architect         0.901.9
@angular-devkit/build-angular     0.901.9
@angular-devkit/build-optimizer   0.901.9
@angular-devkit/build-webpack     0.901.9
@angular-devkit/core              9.1.9
@angular-devkit/schematics        9.1.9
@angular/cli                      9.1.9
@ngtools/webpack                  9.1.9
@schematics/angular               9.1.9
@schematics/update                0.901.9
rxjs                              6.5.5
typescript                        3.8.3
webpack                           4.42.0

Scully Version:

{
  "name": "scully-styles-bug",
  "version": "0.0.0",
  "scripts": {
    "ng": "ng",
    "start": "ng serve",
    "build": "ng build",
    "test": "ng test",
    "lint": "ng lint",
    "e2e": "ng e2e",
    "scully": "scully",
    "scully:serve": "scully serve"
  },
  "private": true,
  "dependencies": {
    "@angular/animations": "~9.1.11",
    "@angular/common": "~9.1.11",
    "@angular/compiler": "~9.1.11",
    "@angular/core": "~9.1.11",
    "@angular/forms": "~9.1.11",
    "@angular/platform-browser": "~9.1.11",
    "@angular/platform-browser-dynamic": "~9.1.11",
    "@angular/router": "~9.1.11",
    "@scullyio/init": "0.0.28",
    "@scullyio/ng-lib": "latest",
    "@scullyio/scully": "latest",
    "rxjs": "~6.5.4",
    "tslib": "^1.10.0",
    "zone.js": "~0.10.2"
  },
  "devDependencies": {
    "@angular-devkit/build-angular": "~0.901.9",
    "@angular/cli": "~9.1.9",
    "@angular/compiler-cli": "~9.1.11",
    "@types/node": "^12.11.1",
    "@types/jasmine": "~3.5.0",
    "@types/jasminewd2": "~2.0.3",
    "codelyzer": "^5.1.2",
    "jasmine-core": "~3.5.0",
    "jasmine-spec-reporter": "~4.2.1",
    "karma": "~5.0.0",
    "karma-chrome-launcher": "~3.1.0",
    "karma-coverage-istanbul-reporter": "~2.1.0",
    "karma-jasmine": "~3.0.1",
    "karma-jasmine-html-reporter": "^1.4.2",
    "protractor": "~7.0.0",
    "ts-node": "~8.3.0",
    "tslint": "~6.1.0",
    "typescript": "~3.8.3"
  }
}

🔥 Exception or Error

 ☺   new Angular build imported
Background servers already running.
Finding all routes in application.

----------------------------------
Using stored unhandled routes!.
   To discover new routes in the angular app use "npm run scully -- --scanRoutes"
----------------------------------
Pull in data to create additional routes.
Finding files in folder "./blog"
Route list created in files:
  "./dist/static/assets/scully-routes.json",
  "/home/robertsbt/projects/scully-styles-bug/dist/scully-styles-bug/assets/scully-routes.json",
  "/home/robertsbt/projects/scully-styles-bug/src/assets/scully-routes.json"

Route "" rendered into file: "./dist/static/index.html"
Route "/home" rendered into file: "./dist/static/home/index.html"

        ----------------
        Error, missing "" in route "/blog/___UNPUBLISHED___kbk617eb_MdeHUgI6aHEdhvLWRXpT6lE7p3NOJTSI"
        without  we can not render this route.
        Make sure it is in there, and not inside any conditionals (*ngIf)
        You can check this by opening "http://localhost:4200//blog/___UNPUBLISHED___kbk617eb_MdeHUgI6aHEdhvLWRXpT6lE7p3NOJTSI"
        when you serve your app with ng serve and then in the browsers console run:
        document.querySelector('scully-content')
        ----------------

DOMException {}
Route "/blog/___UNPUBLISHED___kbk617eb_MdeHUgI6aHEdhvLWRXpT6lE7p3NOJTSI" rendered into file: "./dist/static/blog/___UNPUBLISHED___kbk617eb_MdeHUgI6aHEdhvLWRXpT6lE7p3NOJTSI/index.html"
send reload

Generating took 2.8 seconds for 3 pages:
  That is 1.07 pages per second,
  or 936 milliseconds for each page.

  Finding routes in the angular app took 1 milliseconds
  Pulling in route-data took 16 milliseconds
  Rendering the pages took 2.06 seconds

I pushed a new version to npm that has this fixed


Fix types for Seed.

Makes the following changes:

Added float to the Union for Seed.
Added None to the Union for Seed.
Replaced Optional[Seed] with Seed everywhere.

See the following links for details:

https://bugs.python.org/issue32554
https://github.com/python/cpython/pull/15382
https://docs.python.org/3.9/library/random.html#random.seed

Currently typeshed is still using Any.

Checklist

[x] I have read contributing guidelines
[x] I'm sure that I did not unrelated changes in this pull request
[n/a] I have created at least one test case for the changes I have made

@ngnpopePlease, fix the issues (see details for github actions)

Sure. (I'd noticed but just don't have time to get back to it this weekend. Will resolve in a few days.)

Hopefully I've fixed all the issues. Sorry for the laziness and the noise!

@ngnpope You did a great job! Thank you very much!


claimtrie: allows '*' in claim name

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
12 unchanged lines in 2 files lost coverage.
Overall coverage decreased (-0.03%) to 51.096%

Files with Coverage Reduction
New Missed Lines
%

btcec/signature.go
3
92.82%

peer/peer.go
9
75.61%

Totals

Change from base Build<PHONE_NUMBER>:
-0.03%

Covered Lines:
22492

Relevant Lines:
44019

💛  - Coveralls


Feature request: allow more control over parameter matching

It would be great if I could have more control over parameter matching. Lets say I have a query that creates a node and takes a random uuid as input:
MERGE (u:User { userId: $userId, name: $userName }) return u;

My input params might be in the form of
const params = {
  userId: newUuid(), // function to generate a random uuid
  name: "user name"
}

neo-forgery configuration
const querySet:QuerySpec[] = [
        {
            name: 'create_user',
            query: `
              MERGE (u:User { userId: $userId, name: $userName }) return u;
            `,
            output: wrapCopiedResults(createUserResponse),
            params: {
              userId: "test_user_id",
              name: "user name"
            }
        }
    ]

This will result in an error: Error: your query was matched in your QuerySpec. But your params were not matched:
I would like to have some control here, my idea is to allow a lambda on each parameter to control the param checking, for example:
const querySet:QuerySpec[] = [
        {
            name: 'create_user',
            query: `
              MERGE (u:User { userId: $userId, name: $userName }) return u;
            `,
            output: wrapCopiedResults(createUserResponse),
            params: {
              userId: (value) => typeof value === "string",
              name: (value) => value === "test_user_id",
            }
        }
    ]

Hi @eamonnmcevoy, I hear.  It does seem sensible.
Just to confirm that I understand the use case, you would generate the mocked response using the actual generated random id ?

Not necessarily, I just want to ignore the parameter check for some parameters.

You could use mockSessionFromFunction.  But I get it.
(Just pointing out that as a workaround you could mock the newUuid() function now.)


codecov/project fail

Pls check https://github.com/apache/singa/pull/795/checks?check_run_id=1115935742

@moazreyad
Codecov check if the code is covered by test
I think will need to update this ignore path of this file https://github.com/apache/singa/blob/dev/.codecov.yml

@moazreyad It is very strange that the test analyzes the coverage on tensor.h in this PR which is totally unrelated

There is a problem in this PR because it installs singa in python 3.6  and executes the test cases from python3.7/site-packages folder which does not work. Please fix this different python versions problem.
Note also that the project coverage threshold can be set in .codecov.yml like this:
coverage:
  status:
    project:
      default:
        # basic
        target: auto
        threshold: 5%

The value of threshold can be chosen by the development team based on the quality needs. In the best case, every PR should not reduce the coverage because this means it either added code without test case or it removed (or disabled) the test cases for existing code. However, if the team think this is too strict requirement, we may choose the threshold to 5% or even more. But this means that a PR can be allowed to pass even if it reduces the code coverage too much.
The PR #795 reduces the coverage by 4.97%, so setting the threshold to 5% should make it pass. But this is not recommended solution. We can use it only if the team thinks it is too strict to keep the coverage change always zero or positive. In this case, we may allow threshold of 1% or 2% to pass. But 5% seems high to me and it means there is a lot of code that was not tested and there is a problem. And thanks to the strict code coverage, we found the different python versions problem. Without the strict coverage check, may be this problem will be hidden and will cause other problems later.

After fixing the python version issue, code coverage fails again..
I didn't change the source code in this PR #795

Seems it does not provide the suggestions on how to improve the code coverage.
for example, #797

After fixing the python version issue, code coverage fails again..

The coverage is now decreased by 0.01% instead of 4.97%. It still fails because we did not specify a threshold, so any decrease any in the coverage will make test fails. To solve this problem, either add a small threshold like 0.1 % to allow small drops in coverage generally in all PRs, or leave the threshold decision to each pull request reviewer to decide if the decrease in the specific PR is fine and it can be merged or it is not fine and must be fixed.

Seems it does not provide the suggestions on how to improve the code coverage.

Yes, it just reports the coverage results without suggestions on how to fix. This is similar to all the test cases which report the error but of course they do not usually suggest how to fix it.
To improve the coverage, the developer needs to investigate more how to create the suitable test cases to cover his new code, or how not to prevent current test cases from running (like the problem that we had in #795).

Thanks for the explanation.
I think we can let the reviewer to determine if the newly added code should be covered by unit tests.


Comment out or flag compiler generated functions

The PDB format uses pseudo and compgenx flags to specify these functions
https://github.com/microsoft/microsoft-pdb/blob/master/include/cvinfo.h#L1145
for example public: void __dflt_ctor_closure(); is one such function but there are less obvious ones like copy ctors, copy operators and other things that are not clear if they were actually implemented.

Ah good point, these functions are polluting the output indeed. Thanks for the pointers 👍


943 patch governance

This is a documentation fix to correct Issue 943.

@asb I'll let you weigh in on this change. At a minimum I think it should be
(see Companies house) - with parenthesis. But your call

@evelynmitchell The companies house reference feels a bit out of place in this sentence to me. I've commented on https://github.com/lowRISC/opentitan/issues/943 to get the Articles of Association added to the lowRISC web site.
Would that be sufficient to close this PR?

Yes. That's sufficient.
Evelyn Mitchell
On Fri, Nov 15, 2019 at 9:25 AM Philipp Wagner<EMAIL_ADDRESS>wrote:

@evelynmitchell https://github.com/evelynmitchell The companies house
reference feels a bit out of place in this sentence to me. I've commented
on #943 https://github.com/lowRISC/opentitan/issues/943 to get the
Articles of Association added to the lowRISC web site.
Would that be sufficient to close this PR?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/lowRISC/opentitan/pull/946?email_source=notifications&email_token=AAHV7Z2OWCQPKZBZ5QZWTYDQT3EQXA5CNFSM4JK5RN6KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEEF6WWI#issuecomment-554429273,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAHV7ZZGCZX5G4EYYQEV4JDQT3EQXANCNFSM4JK5RN6A
.

Thanks for the contribution Evelyn. It sounds like you're happy that we address this by making the Articles of Association more discoverable on the lowRISC website (as discussed in https://github.com/lowRISC/lowrisc-web/issues/40), so I'll mark this PR as closed.


chore: Update Github owner

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


nav is broken again

So it looks like nav is more or less the same inside, so long as you tune the wheel speeds for inside. Next I will test if it is WLD messing with nav outside


Remove fields from api_pb2.Image that are no longer used by the server

See https://github.com/modal-labs/modal/pull/17428
 Backward/forward compatibility checks

Check these boxes or delete any item (or this section) if not relevant for this PR.

[x] Client+Server: this change is compatible with old servers
[ ] Client forward compatibility: this change ensures client can accept data intended for later versions of itself

Note on protobuf: protobuf message changes in one place may have impact to
multiple entities (client, server, worker, database). See points above.

@prbot approve


Revert "Support cusparsecsrgeam2 and cusparsecsrgemm2"

Reverts cupy/cupy#3153
Windows can't build with cuSPARSE generic APIs support

pfnCI, test this please.

It looks different PR was being reverted..
This one: https://github.com/cupy/cupy/pull/3129?

Can't revert #3129 till this one is reverted first, since they conflict

Jenkins CI test (for commit 263bc2dd08a26c970b5bf58ee83657c46fc081b6, target branch master) succeeded!


api: deprecated version annotation introduction

Commit Message: api: deprecated version annotation introduction
Additional Description:
Adding a deprecated API version annotation to deprecated fields and enum values in proto files.
This is part of the work on adding minor/patch versioning work.
Risk Level: Low (adding annotation to existing protos).
Testing: Added and modified tests for the tooling (in tools/testdata).
Docs Changes:
Release Notes:
Platform Specific Features: None

The first commit 3ffd898 has the relevant scripts changes (including tests).
The second commit 8f9b997 contains the updates to all protos in api and generated_api_shadow.

/retest

@htuch, @phlax thanks for the comments!
It should be all set now.

/retest

@adisuissa can you merge main? @phlax I think we should stop merging format fixes until this PR merges.

@phlax I think we should stop merging format fixes until this PR merges.

yep k - apologies @adisuissa - py formatting has been a bit of shifting target and been a merge/rebase nightmare

docs are rendered here:
https://storage.googleapis.com/envoy-pr/15564/docs/index.html
maybe i misunderstand what is included here - or the purpose - but i expected to find some change to the docs re deprecation

@phlax I think we should stop merging format fixes until this PR merges.

yep k - apologies @adisuissa - py formatting has been a bit of shifting target and been a merge/rebase nightmare

No worries, I can fix the format if needed (it would just be easier to see the relevant changes if there were no formatting changes needed).

docs are rendered here:
https://storage.googleapis.com/envoy-pr/15564/docs/index.html
maybe i misunderstand what is included here - or the purpose - but i expected to find some change to the docs re deprecation

My plan is to add an explanation to api/API_VERSIONING.md (see draft: https://github.com/envoyproxy/envoy/pull/14943).
I can also update the docs, but not sure where. It seems that the API versioning points to the internal doc.
Another possibility is to add the explanation to this FAQ.

My plan is to add an explanation to api/API_VERSIONING.md

if we have a way of saying "this feature is deprecated in version X" it would be really helpful to include this in the docs that end users read - im still not sure if i understand the purpose exactly - or whether this provides that

/restest

/retest

if we have a way of saying "this feature is deprecated in version X" it would be really helpful to include this in the docs that end users read - im still not sure if i understand the purpose exactly - or whether this provides that

I agree that the docs should be updated, once the patch version capabilities will be added (until now the work has mainly added minor version capabilities).
Moreover, we'll need to clarify which deprecated fields are no longer supported in a specific version.
@phlax thanks for the thorough review and great comments!


Google Cast - Web Sender

Related Problem:
Casting content with Google Cast (Apple AirPlay is already supported in the community player)

Describe:
Add the ability to provide a Google Cast receiver ID to a vidstack player to enable Google Cast logic along with a community player button.
Alternatives:
I've been able to get this codelab to work with a standard video player, but not with vidstack:
https://developers.google.com/cast/codelabs/cast-videos-chrome

chromecast is..... annoying, while this will work fine for providers like video or hls [sometimes, not all chromecast devices support HLS to my understanding] it won't work for things like youtube, vimeo or local [blob] files, this could however be remedied by creating a custom chromecast app which essentially runs a website with only the vidstack player, is the scope of this only audio and video?

This has been implemented and released in 1.10@next see release notes. The scope is only video and HLS providers at the moment.  Supporting embeds for this is out of scope and not something I'm willing to explore right now. Thanks!


[RFC] New Relic Support?

First of all, thanks for your software. It's been a great experience working with it.
I moved my Symfony application to Roadrunner, which improved the performance, but New Relic integration does not seem to work.
I was wondering if anyone has successfully integrated it, or you have any tips on this.
Thanks in advance.

Parts of it can be solved with this but New Relic will still be missing a lot of information (like request url, headers, params, etc). It doesn't look like you can set these manually in the extension at the time.

Parts of it can be solved with this but New Relic will still be missing a lot of information (like request url, headers, params, etc). It doesn't look like you can set these manually in the extension at the time.

It might make more sense to add new relic as rr plugin rather than tackle PHP code.

It might make more sense to add new relic as rr plugin rather than tackle PHP code.

Possibly. Although allowing to configure a custom request would also solve the problem for other services with a similar architecture as RoadRunner (like PHP-PM).

Possibly. Although allowing to configure a custom request would also solve the problem for other services with a similar architecture as RoadRunner (like PHP-PM).

See this comment. A simple solution would be to offer something like PHP-PM does with populate-server-var that populates the $_SERVER variable which would allow New Relic to look up the given request details. This can be implemented in user land but it might still be convenient for RoadRunner to offer such an option.

See this comment. A simple solution would be to offer something like PHP-PM does with populate-server-var that populates the $_SERVER variable which would allow New Relic to look up the given request details. This can be implemented in user land but it might still be convenient for RoadRunner to offer such an option.


x/tools/refactor/rename: doesn't work on Windows

Go 1.7
Windows 64 Bit
I hope, I fixed the problem with go move package under windows.
bradfitz outcommented the tests for windows.
Now they work.
There was a tiny replace oversight -
from [unix] package path to [windows] file path.
https://play.golang.org/p/76oMBOgedE
Here is the patch
diff --git a/refactor/rename/mvpkg.go b/refactor/rename/mvpkg.go
index cd416c5..eed338a 100644
--- a/refactor/rename/mvpkg.go
+++ b/refactor/rename/mvpkg.go
@@ -80,7 +80,8 @@ func Move(ctxt *build.Context, from, to, moveTmpl string) error {
 			affectedPackages[r] = true
 		}
 		// Ensure directories have a trailing separator.
-		dest := strings.Replace(pkg,
+		dest := strings.Replace(
+			filepath.Join(pkg, ""),
 			filepath.Join(from, ""),
 			filepath.Join(to, ""),
 			1)
diff --git a/refactor/rename/mvpkg_test.go b/refactor/rename/mvpkg_test.go
index 674fe6c..ba2536a 100644
--- a/refactor/rename/mvpkg_test.go
+++ b/refactor/rename/mvpkg_test.go
@@ -12,7 +12,6 @@ import (
 	"path/filepath"
 	"reflect"
 	"regexp"
-	"runtime"
 	"strings"
 	"testing"

@@ -114,9 +113,9 @@ var _ foo.T
 }

 func TestMoves(t *testing.T) {
-	if runtime.GOOS == "windows" {
-		t.Skip("broken on Windows; see golang.org/issue/16384")
-	}
+	// if runtime.GOOS == "windows" {
+	// 	t.Skip("broken on Windows; see golang.org/issue/16384")
+	// }
 	tests := []struct {
 		ctxt         *build.Context
 		from, to     string

Hello. Go uses gerrit for patches. Please see https://golang.org/doc/contribute.html for details.

Submitted patch to Gerrit:
https://go-review.googlesource.com/#/c/32392/


Switch imgur images to a local image repository

As pointed out by @robzwolf, this is not the safest method since it relies on imgur which could go down and also take down our Readme with it. We need to move all images to an image folder in our repo.

This issue has been resolved. Closing it now


Only NuGet Admins should see the reflow link

I noticed a new link "Reflow" on the package detail page that shouldn't be visible for non admins - at least the control action is only accessable for NuGet Admins:
(the screenshot is from prod NuGet.org)

With this PR the link shouldn't be visible for normal NuGet users.
If a normal NuGet user should do a "reflow" (whatever that means), then the Authorize("Admin") attribute should be removed.

Already aplied to dev :-) (but not deployed) Thanks for the effort @robertmuehsig

You are pretty fast ;)


Checkpointer crashes on jsonplus with StructuredTools

Checked other resources

[X] I added a very descriptive title to this issue.
[X] I searched the LangGraph/LangChain documentation with the integrated search.
[X] I used the GitHub search to find a similar question and didn't find it.
[X] I am sure that this is a bug in LangGraph/LangChain rather than my code.
[X] I am sure this is better as an issue rather than a GitHub discussion, since this is a LangGraph bug and not a design question.

Example Code
tools = [
    StructuredTool.from_function(
         func=self.customer.find_customer,
         name='find_customer',
         description='Description',
    ),
    ... other tools like this one ^
]

memory = SqliteSaver(conn=sqlite3_conn)

create_react_agent(
   model=openai,
   tools=tools,
   state_modifier=prompt,
   checkpointer=memory,
)

Error Message and Stack Trace (if applicable)
Traceback (most recent call last):
File "/home/michael/PycharmProjects/oro-ai/.venv/lib/python3.11/site-packages/langgraph/pregel/__init__.py", line 1560, in invoke
    for chunk in self.stream(
  File "/home/michael/PycharmProjects/oro-ai/.venv/lib/python3.11/site-packages/langgraph/pregel/__init__.py", line 1248, in stream
    with SyncPregelLoop(
  File "/home/michael/PycharmProjects/oro-ai/.venv/lib/python3.11/site-packages/langgraph/pregel/loop.py", line 770, in __exit__
    return self.stack.__exit__(exc_type, exc_value, traceback)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/michael/.pyenv/versions/3.11.9/lib/python3.11/contextlib.py", line 601, in __exit__
    raise exc_details[1]
  File "/home/michael/.pyenv/versions/3.11.9/lib/python3.11/contextlib.py", line 586, in __exit__
    if cb(*exc_details):
       ^^^^^^^^^^^^^^^^
  File "/home/michael/PycharmProjects/oro-ai/.venv/lib/python3.11/site-packages/langgraph/pregel/executor.py", line 105, in __exit__
    task.result()
  File "/home/michael/.pyenv/versions/3.11.9/lib/python3.11/concurrent/futures/_base.py", line 449, in result
    return self.__get_result()
           ^^^^^^^^^^^^^^^^^^^
  File "/home/michael/.pyenv/versions/3.11.9/lib/python3.11/concurrent/futures/_base.py", line 401, in __get_result
    raise self._exception
  File "/home/michael/PycharmProjects/oro-ai/.venv/lib/python3.11/site-packages/langgraph/pregel/executor.py", line 70, in done
    task.result()
  File "/home/michael/.pyenv/versions/3.11.9/lib/python3.11/concurrent/futures/_base.py", line 449, in result
    return self.__get_result()
           ^^^^^^^^^^^^^^^^^^^
  File "/home/michael/.pyenv/versions/3.11.9/lib/python3.11/concurrent/futures/_base.py", line 401, in __get_result
    raise self._exception
  File "/home/michael/.pyenv/versions/3.11.9/lib/python3.11/concurrent/futures/thread.py", line 58, in run
    result = self.fn(*self.args, **self.kwargs)
             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/michael/PycharmProjects/oro-ai/.venv/lib/python3.11/site-packages/langgraph/pregel/loop.py", line 696, in _checkpointer_put_after_previous
    prev.result()
  File "/home/michael/.pyenv/versions/3.11.9/lib/python3.11/concurrent/futures/_base.py", line 449, in result
    return self.__get_result()
           ^^^^^^^^^^^^^^^^^^^
  File "/home/michael/.pyenv/versions/3.11.9/lib/python3.11/concurrent/futures/_base.py", line 401, in __get_result
    raise self._exception
  File "/home/michael/PycharmProjects/oro-ai/.venv/lib/python3.11/site-packages/langgraph/pregel/executor.py", line 70, in done
    task.result()
  File "/home/michael/.pyenv/versions/3.11.9/lib/python3.11/concurrent/futures/_base.py", line 449, in result
    return self.__get_result()
           ^^^^^^^^^^^^^^^^^^^
  File "/home/michael/.pyenv/versions/3.11.9/lib/python3.11/concurrent/futures/_base.py", line 401, in __get_result
    raise self._exception
  File "/home/michael/.pyenv/versions/3.11.9/lib/python3.11/concurrent/futures/thread.py", line 58, in run
    result = self.fn(*self.args, **self.kwargs)
             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/michael/PycharmProjects/oro-ai/.venv/lib/python3.11/site-packages/langgraph/pregel/loop.py", line 696, in _checkpointer_put_after_previous
    prev.result()
  File "/home/michael/.pyenv/versions/3.11.9/lib/python3.11/concurrent/futures/_base.py", line 449, in result
    return self.__get_result()
           ^^^^^^^^^^^^^^^^^^^
  File "/home/michael/.pyenv/versions/3.11.9/lib/python3.11/concurrent/futures/_base.py", line 401, in __get_result
    raise self._exception
  File "/home/michael/PycharmProjects/oro-ai/.venv/lib/python3.11/site-packages/langgraph/pregel/executor.py", line 70, in done
    task.result()
  File "/home/michael/.pyenv/versions/3.11.9/lib/python3.11/concurrent/futures/_base.py", line 449, in result
    return self.__get_result()
           ^^^^^^^^^^^^^^^^^^^
  File "/home/michael/.pyenv/versions/3.11.9/lib/python3.11/concurrent/futures/_base.py", line 401, in __get_result
    raise self._exception
  File "/home/michael/.pyenv/versions/3.11.9/lib/python3.11/concurrent/futures/thread.py", line 58, in run
    result = self.fn(*self.args, **self.kwargs)
             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/michael/PycharmProjects/oro-ai/.venv/lib/python3.11/site-packages/langgraph/pregel/loop.py", line 698, in _checkpointer_put_after_previous
    cast(BaseCheckpointSaver, self.checkpointer).put(
  File "/home/michael/PycharmProjects/oro-ai/.venv/lib/python3.11/site-packages/langgraph/checkpoint/sqlite/__init__.py", line 399, in put
    type_, serialized_checkpoint = self.serde.dumps_typed(checkpoint)
                                   ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/michael/PycharmProjects/oro-ai/.venv/lib/python3.11/site-packages/langgraph/checkpoint/serde/jsonplus.py", line 193, in dumps_typed
    return "msgpack", _msgpack_enc(obj)
                      ^^^^^^^^^^^^^^^^^
  File "/home/michael/PycharmProjects/oro-ai/.venv/lib/python3.11/site-packages/langgraph/checkpoint/serde/jsonplus.py", line 500, in _msgpack_enc
    return enc.pack(data)
           ^^^^^^^^^^^^^^
  File "msgpack/_packer.pyx", line 279, in msgpack._cmsgpack.Packer.pack
  File "msgpack/_packer.pyx", line 276, in msgpack._cmsgpack.Packer.pack
  File "msgpack/_packer.pyx", line 265, in msgpack._cmsgpack.Packer._pack
  File "msgpack/_packer.pyx", line 213, in msgpack._cmsgpack.Packer._pack_inner
  File "msgpack/_packer.pyx", line 265, in msgpack._cmsgpack.Packer._pack
  File "msgpack/_packer.pyx", line 213, in msgpack._cmsgpack.Packer._pack_inner
  File "msgpack/_packer.pyx", line 265, in msgpack._cmsgpack.Packer._pack
  File "msgpack/_packer.pyx", line 213, in msgpack._cmsgpack.Packer._pack_inner
  File "msgpack/_packer.pyx", line 265, in msgpack._cmsgpack.Packer._pack
  File "msgpack/_packer.pyx", line 232, in msgpack._cmsgpack.Packer._pack_inner
  File "msgpack/_packer.pyx", line 265, in msgpack._cmsgpack.Packer._pack
  File "msgpack/_packer.pyx", line 213, in msgpack._cmsgpack.Packer._pack_inner
  File "msgpack/_packer.pyx", line 267, in msgpack._cmsgpack.Packer._pack
  File "/home/michael/PycharmProjects/oro-ai/.venv/lib/python3.11/site-packages/langgraph/checkpoint/serde/jsonplus.py", line 232, in _msgpack_default
    obj.model_dump(),
    ^^^^^^^^^^^^^^^^
TypeError: BaseModel.model_dump() missing 1 required positional argument: 'self'

Description
SqliteSaver fails with create_react_agent() construct and StructuredTools
Actually, the same happens with the basic MemorySaver()
Versions:
langchain = "^0.3"
langchain-cli = "^0.0.31"
langchain-openai = "^0.2"
langgraph = "^0.2"
langgraph-checkpoint-sqlite = "^2.0.0"
System Info
System Information

OS:  Linux
OS Version:  #26-Ubuntu SMP PREEMPT_DYNAMIC Tue Jun 18 12:35:22 UTC 2024
Python Version:  3.11.9 (main, Aug 12 2024, 07:24:05) [GCC 11.4.0]

Package Information

langchain_core: 0.3.7
langchain: 0.3.1
langchain_community: 0.3.1
langsmith: 0.1.129
langchain_cli: 0.0.31
langchain_openai: 0.2.1
langchain_text_splitters: 0.3.0
langgraph: 0.2.32
langserve: 0.3.0

Other Dependencies

aiohttp: 3.10.8
async-timeout: Installed. No version info available.
dataclasses-json: 0.6.7
fastapi: 0.115.0
gitpython: 3.1.43
gritql: 0.1.5
httpx: 0.27.2
jsonpatch: 1.33
langgraph-checkpoint: 2.0.0
langserve[all]: Installed. No version info available.
numpy: 1.26.4
openai: 1.51.0
orjson: 3.10.7
packaging: 24.1
pydantic: 2.9.2
pydantic-settings: 2.5.2
PyYAML: 6.0.2
requests: 2.32.3
SQLAlchemy: 2.0.35
sse-starlette: 1.8.2
tenacity: 8.4.2
tiktoken: 0.7.0
tomlkit: 0.12.5
typer[all]: Installed. No version info available.
typing-extensions: 4.12.2
uvicorn: 0.23.2

Hi, can you provide a more complete example?
For example: Testing it like this is normal.
from dotenv import load_dotenv
from langchain_core.tools import StructuredTool
from langchain_openai import ChatOpenAI
from langgraph.checkpoint.memory import MemorySaver
from langgraph.prebuilt import create_react_agent

load_dotenv()

class MyTool:
    def add(self, a: int, b: int) -> int:
        """Add two numbers."""
        return a + b

    def get_tools(self) -> list[StructuredTool]:
        return [
            StructuredTool.from_function(
                func=self.add,
                name="find_customer",
                description="Description",
            ),
        ]

openai = ChatOpenAI(model="gpt-4o-mini")
app = create_react_agent(
    model=openai, tools=MyTool().get_tools(), checkpointer=MemorySaver()
)
thread_config = {"configurable": {"thread_id": "1"}}
print(app.invoke({"messages": "What is 1 + 1?"}, thread_config))

I'm not sure about your func=self.customer.find_customer, in what context it is used.

Ah, it didn't call the tool. I'll test it again.

I found the reason.
I was passing additional arguments from typed object to the invoke method dynamically, like this:
runnable.invoke(params.model_dump(), , config={'configurable': {
    'thread_id': params.thread_id,
}})

And when I started using the checkpointer it started failing.
Note: It may be worth adding to the docs that passing additional stuff might harm the executor when the checkpointer is utilized.
Thanks @gbaian10


Drop archive name from matches in legendastv provider: fixes #691

As discussed in #691: Using archive name to enrich subtitles information might lead to false positive matches

Coverage decreased (-0.004%) to 91.817% when pulling 5244e4fd792cf6dd49667be46572ede5f9c6c607 on ratoaq2:ltv-drop-archive-name into 7eb7a53fe6bcaf3e01a6b44c8366faf7c96f7f1b on Diaoul:develop.

Coverage decreased (-0.004%) to 91.817% when pulling 5244e4fd792cf6dd49667be46572ede5f9c6c607 on ratoaq2:ltv-drop-archive-name into 7eb7a53fe6bcaf3e01a6b44c8366faf7c96f7f1b on Diaoul:develop.

Coverage decreased (-0.004%) to 91.817% when pulling 5244e4fd792cf6dd49667be46572ede5f9c6c607 on ratoaq2:ltv-drop-archive-name into 7eb7a53fe6bcaf3e01a6b44c8366faf7c96f7f1b on Diaoul:develop.

Coverage decreased (-0.004%) to 91.817% when pulling 5244e4fd792cf6dd49667be46572ede5f9c6c607 on ratoaq2:ltv-drop-archive-name into 7eb7a53fe6bcaf3e01a6b44c8366faf7c96f7f1b on Diaoul:develop.


Omni for MultiMC

Add a new theme to the organization
If you have already created a theme, make sure you check the following requirements:

[X] The theme follows the template;
[X] The theme follows the Omni Color Palette;
[X] The branch name is main (we will keep this name pattern from now on);
[X] The theme has correct instructions on how to install it;
[X] The repository name is lowercase and contains just the application name. E.g. visual-studio-code.

Application: MultiMC
Application URL https://multimc.org/
GitHub URL: https://github.com/wictornogueira/multimc

Hey @wictornogueira. Thanks for creating the theme!
I have reviewed it, and we can include the theme in the organization. 🎉
I'll be sending an invite for you to join the organization. After that, you can transfer your repository to the org.
Just make sure to open a PR and include the theme on the main repository 👊🏻
Also, after the transfer, make sure to change all URLs to match the new repository link.

Hey, I saw that you transferred the repository!
Now you need to create a PR to add it to this main repository! If you need any help, let me know.
Don't forget to share the theme on forums, communities, and social media!
Thanks for creating it 💜


cannot clone due to 'examples/ansible-files/files/apache-artifact-2021-12-03T13:57:46.346551+00:00.json'

I tried to clone the repo in order to follow a course, but it failed with the following error
$ git clone https://github.com/rhmk/ansible-cli-container.git
Cloning into 'ansible-cli-container'...
remote: Enumerating objects: 117, done.
remote: Counting objects: 100% (117/117), done.
remote: Compressing objects: 100% (77/77), done.
remote: Total 117 (delta 38), reused 87 (delta 19), pack-reused 0Receiving objecReceiving objects:  85% (100/117)
Receiving objects: 100% (117/117), 26.56 KiB | 2.66 MiB/s, done.
Resolving deltas: 100% (38/38), done.
error: invalid path 'examples/ansible-files/files/apache-artifact-2021-12-03T13:57:46.346551+00:00.json'
fatal: unable to checkout working tree
warning: Clone succeeded, but checkout failed.
You can inspect what was checked out with 'git status'
and retry with 'git restore --source=HEAD :/'

Thanks for leaving this comment. This is strange error I haven't seen so far. I will investigate further and let you ASAP

where did you run this command? I can't reproduce your issue neither on Mac nor on Linux. It seems that you either try to checkout in a non-empty directory that contains examples/ansible-files/files/apache-artifact-2021-12-03T13:57:46.346551+00:00.json
This file does not exist in my repo.

As a first step I would try to follow the git recommendations. Ti get the environment it should be sufficient to just run the container issuing the above podman command

After this I managed to clone and it just cloned without this particular file. I'm Not sure why but thank you anyway for your time.


Add nanosecond timestamp type

Feature Request / Improvement
Analogous to the update in apache iceberg from sep 3, 2024, include nanosecond precision for timestamps.

This would be very helpful for high resolution time series data.

Are there any updates regarding this topic? It would be quite important for our project to have more precise timestamps


Cluster-Wide Session Re-Authorization

Environment

VerneMQ Version:  1.10.2
OS:  Linux (Kubernetes)
Erlang/OTP version (if building from source): N/A
VerneMQ configuration (vernemq.conf) or the changes from the default
Cluster size/standalone: 3 node

Expected behavior
vmq-admin session reauthorize works for a client from any cluster node.
Actual behaviour
Periodically we issue vmq-admin command to re-authorize subscriptions for a client in each node and it ends up killing the subscription.
Might be related to #1124

@ssmiller25 Thanks. I don't understand from your description what you do.
vmq-admin session reauthorize sometimes works, sometimes not, killing a subscription?
What does killing the subscription mean? what do you see logged? What auth plugins do you use?

@ioolkos adding to what @ssmiller25 said, we have a 3 node cluster setup and there's a shell script that runs on each node which periodically gets the list of connected clients using "vmq-admin session" and issue "vmq-admin session reauthorize" for each client. Since in cluster setup, the list contains the clients that are connected to other nodes as well, script ends up calling reauthorize command for those clients as well which ends up killing the subscription and the client stops getting messages. We have written a lua plugin with auth_on_subscribe hook which does gets called when "vmq-admin session reauthorize" is issued. In non-cluster setup everything works as expected.

@gsingh395 @ssmiller25
Okay, thanks for the additional information. Will have to check this.
In the mean time, couldn't you add --node to your vmq-admin session command, and then do some filtering or sorting in the script (so that you only call reauthorization for local sessions)?

Thanks @ioolkos  I wasn't aware you can query node column with vmq-admin session command. @ssmiller25 this might work for us.

vmq-admin session show --help shows you all the options.

Thanks @ioolkos.  That solution will work for us!


chore: example with clerk

Minimal example. If user is not signed in - uploading is prohibited

[ ] Add examples with pages dir

Will add example with pages dir

@juliusmarminge I see that you approved the changes, but can I ask you to rename the example to with-appdir-and-clerk
@markflorkowski wants to have similar but for pages directory

This looks solid - the pages/ example can be a separate PR


Using $_FILES to download?

How come you are using $_FILES in your script? As far as I know $_FILES is using for uploading files. Wouldnt it be more proper to use another temporary file name?

@ralphe-samson  yes, using $_FILES["file"]["tmp_name"] for downloading a file is one of acceptable ways.  This is working code for large download in PHP. Let me know if otherwise.


xiaomi show permission dialog twice when i request Location permission

Overview
xiaomi show permission dialog twice when i request Location permission，first dialog has 10 seconds countdown and auto deny operation，second has not  countdown。Is this a problem？
Expected

It should show dialog once。

Environment
-library：3.0.1

xiaomi 5   android 7.0

first dialog has 10 seconds countdown and auto deny operation

Sorry I don't get it...could you elaborate or paste the gif?
Anyway, please try without PermissionsDispatcher and compare the result.

With new update to support permission on xiaomi device, this issue is introduced.
The reason I found that is because these checking operation will both trigger permission dialogs:

AppOpsManagerCompat.noteOp()
ActivityCompat.requestPermissions()
I'm not sure how to avoid this but I think it should be a problem.

Like this.And this make another problem,if i request permission in onCreate(),the first dialog will show but the activity layout did not show.I try without PermissionsDispatcher and it has no problem.

Now,i use library 2.2.1,it work well.

I think Xiaomi has fixed this issue in later versions. So it is hard to decide when to do the double check. My workaround is to temporarily modify Build.MANUFACTURER other than "Xiaomi" to avoid the check.

So can I close this issue?

-library：3.3.1
xiaomi note3 android 8.1
I have the same problem.

”OnNeverAskAgain“  this method is called all the time！ in Nokie X7-android8.1；
i 'm use 3.3.1 version

Just merged https://github.com/permissions-dispatcher/PermissionsDispatcher/pull/548 so it will be fixed in next version may be(you can test with snapshot)?


Add shrinkwrap file

closes #957
Moves some dev-specific dependencies, and generates a shrinkwrap file as documented https://docs.npmjs.com/cli/shrinkwrap
Note: shrinkwrap will include devDependencies (currently does not) if we upgrade to NPM 4.x

Coverage remained the same at 83.226% when pulling 364f04e70bc5c28483cc17948f0142f42db4bdba on mparke:npm-shrinkwrap into e1efc9930a0f89ccbc90cebaef51e0f30d36e359 on linode:master.


Add support for LoadBalancing metrics

resolves: #51
Description
This PR enables support for CloudFlare LoadBalancer metrics.
Introduces the following changes:

  feat: Export LoadBalancer analytics metrics

This commit enables support for exporting LoadBalancer analytics
metrics. LoadBalancer's pool health is reported:

1 -> healthy
0 -> unhealthy

as long as the number of requests each pool receives from each CFL
network location[1]

https://developers.cloudflare.com/load-balancing/reference/load-balancing-analytics/#graphql-analytics

feat: Export browser map page views count

Export browser_map_page_views_count requests metric

Test plan
All changes have been tested locally with a local monitoring stack pulling metrics.

LoadBalancers pool health:

Browser map page views per family

@martinhaus for some reason I can't add reviewers, I would appreciate a review when you have some time :)
Thanks in advance!

LGTM
thank you for the contribution @axilleastr !


[BUG] Seems Blaze Enchanter can't accept the experience fluid.

Minecraft Version
1.20.1
Describe the Bug
With only create enchantment industry and create fabric installed, attempting to transfer experience fluid from the tank with experience fluid to the blaze enchanter does not result in any response.
Reproduction Steps

Place a fluid tank and a blaze burner.
Converting it to blaze enchanter using the enchanting guide.
Connecting the two with a pump.
Seems Blaze Enchanter can't accept the experience fluid.

Expected Result
Blaze Enchanter can accept the experience fluid.
Screenshots and Videos

Crash Report or Log
No response
Other Mods
create-fabric-0.5.1-f-build.1417+mc1.20.1
fabric-api-0.92.1+1.20.1
Mod Version
1.2.16

Try attaching the pipe from the bottom of the Blaze Enchanter.

Oh, that might be a feature, thanks.


localhost -> local.webaverse.com startup

The app requires valid https for many web features. We ship valid local certs for this purpose, for local.webaverse.com, which is purely localhost.
However, commonly people accidentally run npm run dev and then connect to localhost and bypass the warning to get a broken app, with many features not working.
To avoid this, we should update index.js to change localhost to redirect to the correctly working https://local.webaverse.com/ (possibly with a port suffix)

If the docs point to broken instructions, they should be updated as well.

Note that
npm run prod

Uses sudo to give a one-command working https://local.webaverse.com/, on the default port of 443.


[deprecated] Property settings is not allowed

Transferred.

Transferred.

ref
https://zenn.dev/newgyu/scraps/4c24bf3df804bd