python3Packages.policy-sentry: 0.11.16 -> 0.11.17

Motivation for this change
Update to latest upstream release 0.11.17
Change log: https://github.com/salesforce/policy_sentry/releases/tag/0.11.17
Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 139475 at 0a013b15 run on x86_64-linux 1

2 packages failed to build:

python38Packages.cloudsplaining
python39Packages.cloudsplaining

2 packages built successfully:

python38Packages.policy-sentry
python39Packages.policy-sentry

Note that build failures may predate this PR, and could be nondeterministic or hardware dependent.
Please exercise your independent judgement. Does something look off? Please file an issue or reach out on IRC.

https://github.com/salesforce/cloudsplaining/issues/223


moralisAuth on Connect Button

Right now if I have the following:
<ConnectButton moralisAuth={false} />

However I've added my server URL and appId, it still tries to have me do moralis authentication. Am I using the moralisAuth wrong?
It says:
https://github.com/web3ui/web3uikit/blob/9bb64014b8feb7aef2305d9fdcc5f4ca746048fe/src/components/ConnectButton/types.ts#L3
And I've set it to false, but it still tries to moralis auth?

Ah, if your provider looks like:
<MoralisProvider appId={APP_ID!} serverUrl={SERVER_URL!} initializeOnMount={true}>

It will auto try to moralis authenticate you. Is this intentional? Can we connect to moralis but not be authed as a moralis user? Or do we have to be authed?

fixed on 0.1.77


feat: add theme gradients

[ ] Allows to select custom colors or gradients by editor sidebar
[ ] Add base gradients to editor themes
[ ] Add custom color component with color/gradient
[ ] Add possibility to select custom color by hex/rgba or gradient (linear/circular gradient)

Preview #2


Setup connection as Foreground Service

This will allow a different app on top instead of the letsrobot robot app and allow it to continue running without being killed randomly.
This will not prevent the app from being killed if the phone is locked, but that can be handled by disabling battery optimization for the app

This is how everything is connected now, and everything is on their own threads. Service holds all components in the same component array

Service is done, now need to make it actually a foreground service. Right now, it has no notification and works only as a bound service


Can not save the password of syncing in Preferences.

How to save the password? It will always be null, no matter what I do.
The post data in http body is always:
Email=xxxxxxx%40gmail.com&Passwd=

Which file is config file of this, can i just edit it?

Which system language are you using?

Thx for reply.
Chinese (Simplified).

I hope pull request #1245 will also fix this.


Issue #6991 Fixed

Issue Reference

#6991
Description

UI Issue Fixed
How To Test This?

Go to Admin -> Catalog -> Products -> Add Product -> Create a new booking product.
Inspect the details in mobile view.
Select the booking type - appointment
Select the Same Slots all days - yes or no.
In both the selections of slots we will be able to see the UI issue.
Documentation

[ ] My pull request requires an update on the documentation repository.

@JunaidJaved248
Please resolve the conflicts.

@Sanjaybhattwebkul Conflicts Resolved

@JunaidJaved248
Not fixed, getting same issue, please find the screenshot here

@Sanjaybhattwebkul Check now Please

@JunaidJaved248
Still not fixed, please find the screenshot Here

@Sanjaybhattwebkul Issue was related to date icon was not aligned. That Fixed now, But now in recent attached screenshot you are pointed cross icon, can you elaborate it?

@JunaidJaved248
Yes, you are right but the cross icon is not in alignment, so we need to fix it, please take a reference from here for cross and date icon's alignment.

Closing this pull request as no activity from the last 16 days.
Feel free to open a new pull request once done.


Make some AvaloniaProperty APIs internal.

What does the pull request do?
Makes some APIs on AvaloniaProperty and derived classes internal:

Constructors are now internal: properties should only be created using the Register* methods anyway
Notifying is now internal, this should only be needed by DataContext and the constructor which set this was already made internal
Base OverrideMetadata methods are now internal: the implementations on StyledProperty should be used

These APIs were always intended to be internal anyway: there should be no need outside our own unit tests to create AvaloniaProperty instances without registering them. If I'm wrong, please let me know!
Breaking changes
As above.

You can test this PR using the following package version. 11.0.999-cibuild0034786-beta. (feed url: https://pkgs.dev.azure.com/AvaloniaUI/AvaloniaUI/_packaging/avalonia-all/nuget/v3/index.json) [PRBUILDID]


Ubuntu 24 compatibility

Question
Does the script work for Ubuntu 24?
Environment (please complete the following information):

Ansible Version: [e.g. 2.16.4]
Host Python Version: [Python 3.12.2]
Ansible Server Python Version: [Python 3.12.2]
Additional Details:
As subject.  Thanks.

hi @patrickfong
I have not current had chance to test this, i know there are some changes that will likely be changed when CIS is released for 24.04.
Many thanks
uk-bolly

hi @patrickfong
I have now written the first release of ubuntu24-cis , which is currently in subscriber access area. This should be officially available in around 10 weeks for public.
Kindest regards
uk-bolly

Hi @uk-bolly,
This is really cool! I'm in the process of implementing CIS compliance on our Ubuntu 24 machines. I would love to use and contribute to your work. Would you be able to share a link to the repository?

hi @sheikhomar
Thank you, It is currently released to subscribers using the early access. It should be released to the community early in the new year for other contributors.
It will then be open for all to feedback back an assist in improving.
Many thanks
uk-bolly

@uk-bolly How do I subscribe and get early access?

@uk-bolly How do I subscribe and get early access?

Hi @sheikhomar, please reach out to me at<EMAIL_ADDRESS>and we can have further discussions.


Base Skelton

[x] Created template
[x] Readme.md
[x] Setup for python package
[x] tests setup
[x] research dir created

its done


[MATH] Fix generation for chat models & fix normalization for predictions

This PR fixes two issues with evaluating chat models that I spotted after working on #140:

stop_tokens enforces premature termination and should not be applied for chat models. For example, in math we specify \n as the stop token, which leads to answers terminating too early (especially in zero-shot). Instead, for chat models we should let the termination happen naturally at the EOS token. The proposed fix now checks for the presence of a chat template and disables stop_tokens if it exists (a cleaner way would be to infer this from --use_chat_template, so please let me know how I can propagate this arg to the model!)
The math_normalizer formats the predictions incorrectly. I am not sure why we distinguish the gold and prediction strings, but removing this logic led to sensible results (I manually inspected the details to see the qem matches the responses). See this Colab where setting is_gold=False gives the wrong normalization.
Makes the _remove_right_unit() method more robust to failure. In this case, it doesn't throw an exception if more than two instances of \\text{} are present. See Colab for example of failure and fix

I'm happy to iterate on the PR to find a cleaner approach if needed and run some other evals to ensure we don't break things. For now it's a quick hack to unblock me :)
Reference evaluations from this branch
deepseek-ai/deepseek-math-7b-instruct

Paper value (unsure if they use sampling): 46.8%

I had to tweak the prompt formatting to match their zero-shot CoT:
def math(line, task_name: str = None):
    return Doc(
        task_name=task_name,
        query=f"{line['problem']}\nPlease reason step by step, and put your final answer within \\boxed{{}}.",
        gold_index=0,
        choices=[f" {line['solution']}"],
    )

Task
Version
Metric
Value

Stderr

all

qem
0.4114
±
0.0181

lighteval:math:_average:0

qem
0.4114
±
0.0181

lighteval:math:algebra:0
0
qem
0.6858
±
0.0135

lighteval:math:counting_and_probability:0
0
qem
0.3439
±
0.0218

lighteval:math:geometry:0
0
qem
0.3549
±
0.0219

lighteval:math:intermediate_algebra:0
0
qem
0.2204
±
0.0138

lighteval:math:number_theory:0
0
qem
0.3370
±
0.0204

lighteval:math:prealgebra:0
0
qem
0.6670
±
0.0160

lighteval:math:precalculus:0
0
qem
0.2711
±
0.0190

Ah I see the unit tests are failing with this change - will take a look

I took a look at the failing test case (it's just 1) and this is the problematic element of reference_scores/harness_metrics.json:
{
                "example": "Problem: Determine the remainder when $$1+12+123+1234+12345+123456+1234567+12345678$$is divided by $5$.\nAnswer:",
                "instruction": "",
                "full_prompt": "Problem: Determine the remainder when $$1+12+123+1234+12345+123456+1234567+12345678$$is divided by $5$.\nAnswer:",
                "predictions": [
                    " The remainder is $1$."
                ],
                "truncated": [
                    0
                ],
                "padded": [
                    0
                ],
                "num_effective_few_shots": 0,
                "num_asked_few_shots": 0,
                "qem": 1.0,
                "hashes": {
                    "example": "fb8657348e7275f8",
                    "full_prompt": "fb8657348e7275f8",
                    "input_tokens": "f379080e78012326",
                    "cont_tokens": "e8d8f369a6896352"
                },
                "choices": [
                    [
                        " If we look at the residues mod 5, we have  \\begin{align*}\n&1+12+123+1234+12345+123456+1234567+12345678\\\\\n&\\qquad\\equiv 1+2+3+4+0+1+2+3 \\\\ &\\qquad\\equiv 16 \\\\ &\\qquad\\equiv \\boxed{1} \\pmod{5}.\\end{align*}"
                    ]
                ],
                "gold_index": 0
            },

What is strange is that math_normalizer correctly parses the choices content, but looking at the test logic, it seems we only rely on predictions which is incorrect for the MATH benchmark AFAICT (we always measure against \\boxed{} answers).

I took a look at the failing test case (it's just 1) and this is the problematic element of reference_scores/harness_metrics.json:
{
                "example": "Problem: Determine the remainder when $$1+12+123+1234+12345+123456+1234567+12345678$$is divided by $5$.\nAnswer:",
                "instruction": "",
                "full_prompt": "Problem: Determine the remainder when $$1+12+123+1234+12345+123456+1234567+12345678$$is divided by $5$.\nAnswer:",
                "predictions": [
                    " The remainder is $1$."
                ],
                "truncated": [
                    0
                ],
                "padded": [
                    0
                ],
                "num_effective_few_shots": 0,
                "num_asked_few_shots": 0,
                "qem": 1.0,
                "hashes": {
                    "example": "fb8657348e7275f8",
                    "full_prompt": "fb8657348e7275f8",
                    "input_tokens": "f379080e78012326",
                    "cont_tokens": "e8d8f369a6896352"
                },
                "choices": [
                    [
                        " If we look at the residues mod 5, we have  \\begin{align*}\n&1+12+123+1234+12345+123456+1234567+12345678\\\\\n&\\qquad\\equiv 1+2+3+4+0+1+2+3 \\\\ &\\qquad\\equiv 16 \\\\ &\\qquad\\equiv \\boxed{1} \\pmod{5}.\\end{align*}"
                    ]
                ],
                "gold_index": 0
            },

What is strange is that math_normalizer correctly parses the choices content, but looking at the test logic, it seems we only rely on predictions which is incorrect for the MATH benchmark AFAICT (we always measure against \\boxed{} answers).
I think the correct approach would be to put some of the choices responses under predictions (with qem=1) and the remaining then should have qem=0

Oh I think this is because you modified the prompt function by adding CoT. Doing this makes the test fail because we check that the prompt is correct.

Oh I think this is because you modified the prompt function by adding CoT. Doing this makes the test fail because we check that the prompt is correct.

Hmm I don't think I have added CoT anywhere - can you clarify what you mean? My understanding is that we need to be parsing choices instead of predictions or put //boxed{} in the answers of predictions.
Perhaps @clefourrier can comment on this final point wrt to 1 failing test?

I'll take a look in the afternoon :)

Ok, so current tests are simply failing because of an import problem, which I fixed, and now the problem is that some of the results changed because of the difference in prompting. @lewtun would you be OK with us having 2 configs, one for math_cot, one for math?

Ok, so current tests are simply failing because of an import problem, which I fixed, and now the problem is that some of the results changed because of the difference in prompting. @lewtun would you be OK with us having 2 configs, one for math_cot, one for math?

Absolutely, this would be fine with me!


Throws an exception if workspace/didChangeConfiguration settings is empty/null

It tries to get python key from settings even if settings is null. Which makes Newtonsoft.Json throw an exception (Newtonsoft.Json.Linq.JToken.get_Item(Object key) InvalidOperationException).

LS can't work without proper settings such as interpreter path, etc. So this is expected.

I'm sending those in initializationOptions, so the server starts. It just doesn't enable the defaults in didChangeConfiguration because of this exception.

OK, it was not clear from the title. Which field is specifically null in your case?

Ah sorry about the unclear report. It happens in DidChangeConfiguration(JToken token, CancellationToken cancellationToken) function at line pythonSection = rootSection?["python"];, which is settings["python"].

@jakebailey worked on settings and related.

Hm
var rootSection = token["settings"];
var pythonSection = rootSection?["python"];

seems to be handling null. Or the settings is missing too?

Yes, it fails when the settings is null. It happens for example when using with LSP modes for Emacs.


Exception with Embedded Descriptions

When using discord notifier with plastic SCM plugin when there are changes example:

[cs:611@Name@Name@cloud]

The discord plugin crashes with the following exception:
java.lang.NullPointerException 	at nz.co.jammehcow.jenkinsdiscord.util.EmbedDescription.<init>(EmbedDescription.java:35) 	at nz.co.jammehcow.jenkinsdiscord.WebhookPublisher.perform(WebhookPublisher.java:134) 	at hudson.tasks.BuildStepMonitor$1.perform(BuildStepMonitor.java:20) 	at hudson.model.AbstractBuild$AbstractBuildExecution.perform(AbstractBuild.java:744) 	at hudson.model.AbstractBuild$AbstractBuildExecution.performAllBuildSteps(AbstractBuild.java:690) 	at hudson.model.Build$BuildExecution.cleanUp(Build.java:196) 	at hudson.model.Run.execute(Run.java:1866) 	at hudson.model.FreeStyleBuild.run(FreeStyleBuild.java:43) 	at hudson.model.ResourceController.execute(ResourceController.java:97) 	at hudson.model.Executor.run(Executor.java:429)
Note: I tried to post this on the new github repo on jenkinssci but there was no issues section :(

Sorry for the lack of issue section there, I forgot to turn them on.
Please install this version of the plugin and let me know if it works.

I updated the plugin with the one you linked. I now get this exception
java.lang.StringIndexOutOfBoundsException: String index out of range: 6 	at java.lang.String.substring(Unknown Source) 	at nz.co.jammehcow.jenkinsdiscord.util.EmbedDescription.<init>(EmbedDescription.java:37) 	at nz.co.jammehcow.jenkinsdiscord.WebhookPublisher.perform(WebhookPublisher.java:142) 	at hudson.tasks.BuildStepMonitor$1.perform(BuildStepMonitor.java:20) 	at hudson.model.AbstractBuild$AbstractBuildExecution.perform(AbstractBuild.java:744) 	at hudson.model.AbstractBuild$AbstractBuildExecution.performAllBuildSteps(AbstractBuild.java:690) 	at hudson.model.Build$BuildExecution.cleanUp(Build.java:196) 	at hudson.model.Run.execute(Run.java:1866) 	at hudson.model.FreeStyleBuild.run(FreeStyleBuild.java:43) 	at hudson.model.ResourceController.execute(ResourceController.java:97) 	at hudson.model.Executor.run(Executor.java:429) Build step 'Discord Notifier' marked build as failure

A fixed version, should work now.

It looks like that fixed version no longer crashes so far. Thanks!


Installation Guide on Laravel Blade

hi @sveltecult,
i really appreciate your work on Franken UI. but can you give us installation guide on Laravel please. because, for the first time i read your docs, it seems i have little bit confused when trying to implement things on my Laravel application.

Hello, what part of the docs are you confused?
You can try and explore this Laravel template: https://github.com/sveltecult/franken-ui-template-laravel this is a little bit outdated but everything is pretty much the same.

i've been read your docs completely and i can understand now. Massive thanks for providing this UI, is really helped me on my current project. especially Overlay and Select components. Hoping to see search functionality on your docs ASAP.

Great! Glad that it helped your project. Should you have any questions, just let me know. Closing this for now.


add long long support

Sorry to take forever responding to this. I think the right approach is to switch to the stdint types: int32_t, int64_t, etc. It's just a significant amount of work that hasn't been done yet. As an intermediate step, this would make sense if it was int64_t vs. 'long long'.

This is replaced by PR #441. Please review! Closing this version.


[Frontend] Add support for logical or, and and not in autograph

[sc-41303]

Hi @josh146. I expect this PR to have corner-cases, so I 'd like to know what we want to support at the first place here. Do we have any use-case in mind, e.g. among the tutorial algorithms which I could check?


DOCS-9203 Revising overview and adding feature support matrix

See also: https://github.com/mulesoft/docs-runtime-fabric/pull/309


Add Support for Message Headers (KIP-82)

See https://github.com/spring-projects/spring-kafka/issues/352

Duplicated by #170


Implementing the Configuration and Client Stores.

Just to let you know, I have taken your project as the basis to :

upgrade the code to .netcore  3.0
implement the TokenService and TokenCleanupHost
implement the resource and client stores.

I have not forked this project because I felt there were too many differences with the upstream IdentityServer repository. Insted I created a mirror.
Thank you very much for starting this project.
I hope you don’t mind me taking another direction.

Sounds great!


Express URLs and allow to go back using the browser back button

Story
As a user who is going through the momentum bot flow, I want to go back to the previous step or I want to go directly to the view where I can start the bot
Done when

[ ] I can go back to the previous step using the browser back button

[ ] from search results to search input parameters
[ ] from swap/start bot view to search results

[ ] I can go directly to the swap/start bot view by typing in the url in the browser with the token address as passed in parameter

What is tokenID? We don't have any tokenID.

I mean token address. Updating the description

And for the second factor, currently, we can't go directly to the swap/start bot view page by entering parameters.
In this case, I can send the token address and volume value, etc as parameters, but I can't get the information of this token needed to show and start the bot. It means that we can't get the 6, 12, 24H volume, token name, symbol, and image that the current API is returning. to make this possible, a separate API is required for me.
like Get bot info of selected token @verabehr

ok, let's discuss next week what's possible. I'll add this to the list of items in notion

removing that part of the ticket. This is reallly the stop loss bot and will be done separately


Ability to send Oauth Events from System Listeners

Describe your problem(s)
There is a requirement on sending OAuth related events to revoke the cache at every token revocation happens at the IAM side. Currently, token events get triggered only from the token endpoint related activities. Therefore we need support to send system level token revocation events.

This is fixed in https://github.com/wso2-extensions/identity-inbound-auth-oauth/pull/1443


Where to see the documentation for blogger?

I want to post to blogger using php library but I don't see the documentation anywhere.
Can someone pass me the link?

Try here: https://developers.google.com/blogger/docs/3.0/libraries#php


gcc 5.0+ ?

why is gcc 5.0+ needed to build gearshifft?

I use this feature from g++5.0 in fft_abstract.hpp for makePlan():
template<template typename X> struct D; // OK
https://gcc.gnu.org/gcc-5/changes.html#cxx

Do I understand right, that you only require gcc5+ for using template template parameters?
http://en.cppreference.com/w/cpp/language/template_parameters
those are a feature of C++ since the 2003 AFAIK.
PS. It's considered a courtesy to allow the creator of an issue to close it again. ;)

sorry, my bad, I thought I have to do it. I reopen it.
To your question: I use typename in a template-template and this was not possible before gcc5.0. With a previous version it fails to compile.

can you provide a minimal "not compiling example" with gcc4.9? I've used template template parameters alright with gcc <5.0!

it is not the nested template, but the nested template-typename construct. try it out:
http://pastebin.com/zQpGPAFf


Arrows separators does not appear properly

environment

vim: neovim v0.3.1
vim-airline: master
OS: Linux Mint
Have you reproduced with a minimal vimrc: No
What is your airline configuration: ???
if you are using terminal:
terminal: gnome-terminal
$TERM variable: xterm-256color
color configuration (:set t_Co?): set t-Co=256
if you are using Neovim:
does it happen in Vim: Yes

I am migration from MacOS back to linux(mint) my Mac airline neovim configuration is not working properly. I've installed the Powerline fonts, when I copy and past the airline_left_sep, for example, from other closed issues, It was pasted as the same character showed as in the screenshot below.
What should I do?
actual behavior

expected behavior
https://raw.githubusercontent.com/wiki/vim-airline/vim-airline/screenshots/demo.gif

What kind of separator would you like to have?

Am 01.04.2019 um 08:00 schrieb Walter Alves<EMAIL_ADDRESS>environment
vim: neovim v0.3.1
vim-airline: master
OS: Linux Mint
Have you reproduced with a minimal vimrc: No
What is your airline configuration: ???
if you are using terminal:
terminal: gnome-terminal
$TERM variable: xterm-256color
color configuration (:set t_Co?): set t-Co=256
if you are using Neovim:
does it happen in Vim: Yes
I am migration from MacOS back to linux(mint) my Mac airline neovim configuration is not working properly. I've installed the Powerline fonts, when I copy and past the airline_left_sep, for example, from other closed issues, It was pasted as the same character showed as in the screenshot below.
What should I do?
actual behavior
expected behavior
https://raw.githubusercontent.com/wiki/vim-airline/vim-airline/screenshots/demo.gif
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Hi @chrisbra,
I would like to have the same separator as in the gif(expected behavior, a kind of colored and filled ">" at the end). I copied the character from other tutorials and from some github issues which was solved by installing the Powerline fonts, but even I adding these fonts, when I past it in the airline_left_sep, for example, it becomes the separator which is being showed now in the A section(in the side of "NORMAL"). Is there anything I would do to debug it?

You don't need to set all the separators manually. Just set the powerline config option (see help) and make sure to use a powerline enabled font.

Am 01.04.2019 um 14:11 schrieb Walter Alves<EMAIL_ADDRESS>Hi @chrisbra,
I would like to have the same separator as in the gif(expected behavior, a kind of colored and filled ">" at the end). I copied the character from other tutorials and from some github issues which was solved by installing the Powerline fonts, but even I adding these fonts, when I past it in the airline_left_sep, for example, it becomes the separator which is being showed now in the A section(in the side of "NORMAL"). Is there anything I would do to debug it?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.


When opening the Launcher I get an error

the error is:
System.Exception: Not enough disk space for repairing. Required space [547338323], available space [0]
The Launcher is trying to download the game folders/files but the error gets thrown in the console. I have enough space and I am on Mac (Monterey 12.1).
Kind regards,
Paul

Hi Paul,
that's a problem in the Mono runtime on Mac. The only way to work around it is to disable the disk space check when on Mac: _context.Settings.EnableDiskSpaceCheck = false; and you will be fine to go.

Ah oke, but where do I set this? (the _context.Settings.EnableDiskSpaceCheck = false;)

In LauncherBase.cs! You can set it in the CreateSettings function (by writing settings.EnableDiskSpace = false;) or in CreateContext function (by writing context.Settings.EnableDiskSpace = false;).

Mm oke so now I have to find a way to not edit the asset code but somehow extend/override it. Thanks for your help! I will test things out.

This bugfix will be included in the next release anyway, so don't worry about that! I'll mark this as bugfix request, just to keep track of it.

Great! Thank you very much I really appreciate that.

One little off-topic question.
When I built and tested the launcher (it downloaded the game logic!) it says this in the manual (under Testing the Launcher)
"From now on you can run directly your game".
Does this mean from now on I have to start the game by using the gameName.app (or gameName.exe in Windows) or always use the LauncherName.app (or Launcher.exe in Windows)?

It's up to you and how you like your game to behave. You can start your game.exe and it will redirect to launcher.exe if an update is found.
Or you can directly start launcher.exe and it will redirect to game.exe if no updates are found.

Oke sounds good. I’m Thanks again!
ps: My comments about this can be removed.  Also: join us on Discord if you like! We can answer all your questions more quickly and you will be able to discuss with other customers that already work on Mac with PATCH!

I improved the error detection for this particular situation. Now it tries to recognize scenarios where the feature is not supported by the OS and tries to proceed anyway.
Commit ref: 28760cda


Is there an easy way to use this library with typescript?

I'm trying to port a project I had in JS to TS, and I have a strong dependency on this library. I've tried to find the definitions file without any luck. Is there already a way, or a plan, to make this library usable with Typescript?

Easy way would be to add allowJs to your tsconfig.json:
{
    "compilerOptions": {
        "allowJs": true,
        "esModuleInterop": true,
        "module": "esnext",
        "moduleResolution": "node",
        "strict": true,
        "target": "esnext"
    }
}

You don't get the benefits of autocompletion for the sitemap module, but your TypeScript code still compiles just fine in strict mode with no warnings.
const sm = require("sitemap");
const sitemap = sm.createSitemap({hostname: "https://example.com"});
sitemap.add({url: "/mypage", changefreq: "daily"});
console.log(sitemap.toString());

I just learned some basic TS, I'll look at this this weekend

It should be easy now to use sitemap from typescript.
#183 was merged.


Assertion error in Triton/XeTLA benchmark

python: /runner/_work/intel-xpu-backend-for-triton/intel-xpu-backend-for-triton/third_party/intel/lib/TritonIntelGPUToLLVM/Utility.h:113: mlir::Value mlir::LLVM::intel::getSharedMemoryBase(mlir::Location, mlir::ConversionPatternRewriter&, mlir::Operation*): Assertion `op->hasAttr("allocation.offset")' failed.
...
/runner/_work/_temp/673946f5-6b4d-405f-9edf-bfa8c094c243.sh: line 2: 66330 Aborted                 (core dumped) python fused_softmax.py --reports $REPORTS

https://github.com/intel/intel-xpu-backend-for-triton/actions/runs/9982726573/job/27589014198#step:16:25
Note that the yesterday's run was successful, so there is a recent commit that broke the benchmark.

I can run successfully in x1(Agama 914.32 PTDB 0.5.2) based on latest commit

CI also works with the latest commit: https://github.com/intel/intel-xpu-backend-for-triton/actions/runs/9995978002

@ESI-SYD Start to fail again, please take a look. https://github.com/intel/intel-xpu-backend-for-triton/actions/runs/10054870453

The issue has been work around in PR #1695 .
We need to investigate the root cause of this issue and fix it properly as high priority.

Another failure in the last nightly run: https://github.com/intel/intel-xpu-backend-for-triton/actions/runs/10135461177/job/28036030570

Also in E2E performance workflows: https://github.com/intel/intel-xpu-backend-for-triton/actions/runs/10129183788/job/28008945266#step:29:625

@pbchekin We have checked these workflows that have experienced failures, and they have one thing in common, which is the use of cached trtion instead of build from source
Can we replace the cache on the top of LLVM target?

@pbchekin We have checked these workflows that have experienced failures, and they have one thing in common, which is the use of cached trtion instead of build from source Can we replace the cache on the top of LLVM target to expose more details if crash happens?

As far as i understand, we use cache for Triton wheels and we use commit id as a cache key. So for each new commit in llvm-target you have a new Triton wheel.

@ESI-SYD potentially you can remove using Triton cached wheels for Triton/XeTLA benchmark and check if that helps. We also can disable cache for E2E workflow, if that helps.


Piri Piri Sauce

Original Portuguese Piri Piri Hot Sauce, 1 bottle
Corisca Hot Crushed Peppers, 2 spoonfuls
Red Wine Vinegar, 10 dashes
Dijon Mustard, 3 spoonfuls
Garlic Seasoning, to taste
Linguiça Seasoning, to tase
MSG, to taste
Maple Syrup, 2 spoonfuls
Lemon Juice, to taste

Emulsify in Immersion Blender

Too much vinegar, needs more acid


Broken links to OS specific upgrade pages

Appears the target prevented the redirect to the correct page. Removing the .md manually directs to correct page.

Proposed changes

Unreleased project version (optional)

Related issues (optional)

Docker Enterprise docs have been moved to Mirantis. If there's still an issue, could you report this over there? https://docs.mirantis.com


Add Windows support

Summary

Adding Windows support.
Test Plan

What's required for testing (prerequisites)?
See this getting started guide for the steps to create a ReactNativeWindows app.
What are the steps to reproduce (after prerequisites)?
Add WebView to ReactNativeWindows app and test.
Compatibility

OS
Implemented

iOS
❌

Android
❌

Windows
✅

Checklist

[x] I have tested this on a device and a simulator
[x] I added the documentation in README.md
[ ] I mentioned this change in CHANGELOG.md
[ ] I updated the typed files (TS and Flow)
[ ] I added a sample use of the API in the example project (example/App.js)

whaaaaatt @kaiguo this is awesome

One thing that would help me is having windows part of the example app. I can boot up a windows VM since I am not too familiar with the windows code base. So I can approve with manual testing!

I agree with @safaiyeh, I think that the example/ app needs to be extended to support windows for this to be merged.

Also if it could be squashed to 1 commit first before it's merged that would be awesome.

One thing that would help me is having windows part of the example app. I can boot up a windows VM since I am not too familiar with the windows code base. So I can approve with manual testing!
@safaiyeh Yeah i'm working on adding a sample app, will ping you when it's ready fro review.

Also if it could be squashed to 1 commit first before it's merged that would be awesome.

Sure I'll squash to 1 commit before merging.

Hi @safaiyeh, I'm trying to run the example app but got a "Module AppRegistry is not registered callable module" error (tested on master branch with Android emulator, I'm seeing the same error in the windows app I just added too). Is this known?

@safaiyeh, I added a windows test app in the example folder, but hit some weird metro issues I wasn't able to resolve :/
You can try yarn start:windows and see if that works for you. If it doesn't work, I've created a separate test app here with some instruction in readme.

Thanks @kaiguo I'll set up some time to test tomorrow.

@safaiyeh did you get a chance to check out the sample app?

We'll want to add Windows to each of the "supported platforms" sections in all of the documentation, where appropriate.

Type
Required
Platform

function
No
iOS, macOS, Windows

@jamonholmgren doc should be updated now.
Also @safaiyeh I've fixed the sample app, so you should be able to just do yarn start:windows and run the test app. I updated the instructions here.

Apologies for the wait, I'll be testing today @kaiguo. Setting up my Windows VM and all ❤

I got it to work on my VM! LGTM, I'll land this right now. Thank you @kaiguo for responding to all the comments and updating accordingly, excited for this.

Thanks @safaiyeh!


how to build develop branch?

i'm trying to build on centos7 (the stable version works) but the develop branch doesn't build. It seems that the include path for mbedtls are different.

In file included from src/task/net.c:3:0:
./src/server.h:42:25: fatal error: mbedtls/ssl.h: No such file or directory
#include <mbedtls/ssl.h>

However when I solve those by symlinking to the needed location it still can't find the mbedtls *.so files.

/usr/bin/ld: cannot find -lmbedtls
/usr/bin/ld: cannot find -lmbedx509
/usr/bin/ld: cannot find -lmbedcrypto

I can not find how to build those. Is there a fix for this? May be i need to switch to a different branch for mbedtls?
Regards.

Which mbedtls are you trying to use?  Is it centos7?  The stable version uses a bundled mbedtls, so won't have this problem.
You can still build with the old make file by doing a make -f localmbedtls.mak for now, but if the default makefile doesn't work on a centos with mbedtls installed I want to know.

Hi,
make -f localmbedtls.mak worked great, thanks. But centos7 packaged version doesn't work, here are the errors:

/root/mongrel2/mongrel2/src/connection.c:299: undefined reference to mbedtls_sha1_starts' /root/mongrel2/mongrel2/src/connection.c:300: undefined reference to mbedtls_sha1_update'
/root/mongrel2/mongrel2/src/connection.c:301: undefined reference to mbedtls_sha1_finish' build/libm2.a(server.o): In function Server_init_rng':
/root/mongrel2/mongrel2/src/server.c:177: undefined reference to mbedtls_havege_init' /root/mongrel2/mongrel2/src/server.c:179: undefined reference to mbedtls_havege_random'
build/libm2.a(websocket.o): In function websocket_challenge': /root/mongrel2/mongrel2/src/websocket.c:64: undefined reference to mbedtls_sha1'
collect2: error: ld returned 1 exit status
make: *** [bin/mongrel2] Error 1
make: *** Waiting for unfinished jobs....
build/libm2.a(connection.o): In function Connection_fingerprint_from_cert': /root/mongrel2/mongrel2/src/connection.c:299: undefined reference to mbedtls_sha1_starts'
/root/mongrel2/mongrel2/src/connection.c:300: undefined reference to mbedtls_sha1_update' /root/mongrel2/mongrel2/src/connection.c:301: undefined reference to mbedtls_sha1_finish'
build/libm2.a(server.o): In function Server_init_rng': /root/mongrel2/mongrel2/src/server.c:177: undefined reference to mbedtls_havege_init'
/root/mongrel2/mongrel2/src/server.c:179: undefined reference to mbedtls_havege_random' build/libm2.a(websocket.o): In function websocket_challenge':
/root/mongrel2/mongrel2/src/websocket.c:64: undefined reference to mbedtls_sha1' collect2: error: ld returned 1 exit status make: *** [tests/handler_tests] Error 1 build/libm2.a(connection.o): In function Connection_fingerprint_from_cert':
/root/mongrel2/mongrel2/src/connection.c:299: undefined reference to mbedtls_sha1_starts' /root/mongrel2/mongrel2/src/connection.c:300: undefined reference to mbedtls_sha1_update'
/root/mongrel2/mongrel2/src/connection.c:301: undefined reference to mbedtls_sha1_finish' build/libm2.a(server.o): In function Server_init_rng':
/root/mongrel2/mongrel2/src/server.c:177: undefined reference to mbedtls_havege_init' /root/mongrel2/mongrel2/src/server.c:179: undefined reference to mbedtls_havege_random'
build/libm2.a(websocket.o): In function websocket_challenge': /root/mongrel2/mongrel2/src/websocket.c:64: undefined reference to mbedtls_sha1'
collect2: error: ld returned 1 exit status
make: *** [tests/chunked_tests] Error 1

centos installs this version:
Version     : 2.7.0

Pull Request #324 should help :wink:

Not sure it helps actually... Your CentOS-provided mbedTLS version may have sha1 and havege module deactivated... You can check this in the config file /usr/include/mbedtls/config.h

@dimovnike does make -f localmbedtls.mak not work for you?  It should be entirely independent of the system mbed TLS version.

yes, make -f localmbedtls.mak works.

@dimovnike #328 has a possible fix for this, let me know if it doesn't work for you


Component does not return parent id

In case subNav is empty (And user selected on root element)
<Navigation
onSelect={({itemId}) => {
will return itemId
}}
In case subNav is not empty (And user selected on root element)
<Navigation
onSelect={({itemId}) => {
will return not return anything
}}
Which is a problem as in my case the Root menu elements render the page, where sub Nav items, are only used to scroll to position. Anyway when component sometimes return ItemId and in some conditions does not it's not consistent behavior of it.

Do you have a fix for this?
I will merge it asap.

Also, try to create a sandbox with your issue for me to look into.

Fixed the case if subNav is [] in v1.7.1.

Also, for consistency, every click will call the callback method passed in props.
Other issues are also fixed as well from v.1.8.0


Error message: nvm is not compatible with the npm config "prefix" option: currently set to "/opt/homebrew"

Operating system and version:
nvm debug output:

nvm --version: v0.36.0
$SHELL: /bin/zsh
$SHLVL: 2
whoami: 'samholmes'
${HOME}: /Users/samholmes
${NVM_DIR}: '${HOME}/.nvm'
${PATH}: /opt/homebrew/bin:/opt/homebrew/sbin:${HOME}/.deno/bin:/usr/local/sbin:${HOME}/bin:/opt/homebrew/bin:/opt/homebrew/sbin:${HOME}/.deno/bin:/usr/local/sbin:${HOME}/bin:/usr/local/bin:/usr/bin:/bin:/usr/sbin:/sbin:/usr/local/go/bin:/Library/Apple/usr/bin:${HOME}/.cargo/bin:/Applications/kitty.app/Contents/MacOS:/usr/local/bin:${HOME}/Library/Android/sdk/emulator:${HOME}/Library/Android/sdk/tools:${HOME}/Library/Android/sdk/tools/bin:${HOME}/Library/Android/sdk/platform-tools:/opt/homebrew/bin:${HOME}/Library/Android/sdk/emulator:${HOME}/Library/Android/sdk/tools:${HOME}/Library/Android/sdk/tools/bin:${HOME}/Library/Android/sdk/platform-tools
$PREFIX: ''
${NPM_CONFIG_PREFIX}: ''
$NVM_NODEJS_ORG_MIRROR: ''
$NVM_IOJS_ORG_MIRROR: ''
shell version: 'zsh 5.8.1 (x86_64-apple-darwin21.0)'
uname -a: 'Darwin 21.6.0 Darwin Kernel Version 21.6.0: Wed Aug 10 14:28:23 PDT 2022; root:xnu-8020.141.5~2/RELEASE_ARM64_T6000 arm64'
checksum binary: 'shasum'
OS version: macOS 12.5.1 21G83
curl: /usr/bin/curl, curl 7.79.1 (x86_64-apple-darwin21.0) libcurl/7.79.1 (SecureTransport) LibreSSL/3.3.6 zlib/1.2.11 nghttp2/1.45.1
wget: not found
sed: /usr/bin/sed
cut: /usr/bin/cut
basename: /usr/bin/basename
rm: /bin/rm
mkdir: /bin/mkdir
xargs: /usr/bin/xargs
git: /usr/bin/git, git version 2.32.1 (Apple Git-133)
grep: /usr/bin/grep, grep (BSD grep, GNU compatible) 2.6.0-FreeBSD
awk: /usr/bin/awk, awk version 20200816
nvm current: system
which node: /opt/homebrew/bin/node
which iojs: iojs not found
which npm: /opt/homebrew/bin/npm
npm config get prefix: /opt/homebrew
npm root -g: /opt/homebrew/lib/node_modules

nvm ls output:

       v10.23.0
       v12.19.0
       v14.15.0
       v14.17.5
       v16.15.1
->       system
default -> lts/* (-> v16.15.1)
node -> stable (-> v16.15.1) (default)
stable -> 16.15 (-> v16.15.1) (default)
iojs -> N/A (default)
unstable -> N/A (default)
lts/* -> lts/gallium (-> v16.15.1)
lts/argon -> v4.9.1 (-> N/A)
lts/boron -> v6.17.1 (-> N/A)
lts/carbon -> v8.17.0 (-> N/A)
lts/dubnium -> v10.24.1 (-> N/A)
lts/erbium -> v12.22.12 (-> N/A)
lts/fermium -> v14.19.3 (-> N/A)
lts/gallium -> v16.15.1

How did you install nvm?
install script in readme
What steps did you perform?

Open terminal app
Open a sub-shell with zsh
Run nvm ls

What happened?
On step 2, an error message appears:
nvm is not compatible with the npm config "prefix" option: currently set to "/opt/homebrew"
Run `npm config delete prefix` or `nvm use --delete-prefix v16.15.1 --silent` to unset it.
Make sure your username (samholmes) matches the one in your $HOME path.
See the "macOS Troubleshooting" section in the docs for more information.

And, on step 3, the selected node version is system.
What did you expect to happen?
No message is shown when the sub-shell starts and the selected node version should be v16.15.1.
Is there anything in any of your profile files that modifies the PATH?

.zshrc
.zshenv
.cargo/env

I suspect your node-installed homebrew might come with a hardcoded npmrc that sets prefix. What output and exit codes do you get from the following commands:

nvm_npmrc_bad_news_bears /opt/homebrew/lib/node_modules/npm/npmrc
nvm_npmrc_bad_news_bears /opt/homebrew/etc/npmrc
nvm_npmrc_bad_news_bears $HOME/.npmrc

?

hm, that suggests that zsh isn't even sourcing nvm.sh at all. Can you . $NVM_DIR/nvm.sh and try again?

@ljharb I did . $NVM_DIR/nvm.sh and then ran which nvm_npmrc_bad_news_bears without any avail. I took a peak in the nvm.sh file and found no reference to nvm_npmrc_bad_news_bears.

which won't ever work with nvm, nvm is a bunch of sourced shell functions. After sourcing the file, you should just be able to run it.

which does work for other shell functions such as nvm_install_latest_npm (it writes out the function implementation). I ran nvm_npmrc_bad_news_bears again after sourcing and still got the same results.

ah, that's a zsh-specific behavior then :-) use command -v to be posix-compliant.
I think the issue is that you're using an ancient version of nvm - v0.36.0. Can you upgrade to v0.39.1?

Upgrading got ride of the "prefix" message, though the default node is still system from sub-shells. What determines the default node version used?

nvm alias default node will set it to "node" (ie, the latest available); or, if you have it in a .nvmrc file in the current directory or any parent directory, its contents will determine it.

Sorry for the delay here.
What, specifically, do you have in your profile files that alters the PATH after those nvm lines? In particular, if anything homebrew-related prepends itself to the PATH afterwards, nvm will never be able to take over from homebrew's node.

After NVM_DIR settings in my .zshrc:
export ANDROID_HOME=$HOME/Library/Android/sdk
export PATH=$PATH:$ANDROID_HOME/emulator
export PATH=$PATH:$ANDROID_HOME/tools
export PATH=$PATH:$ANDROID_HOME/tools/bin
export PATH=$PATH:$ANDROID_HOME/platform-tools

This is very confusing. What does env | grep -i prefix print out?

HOMEBREW_PREFIX=/opt/homebrew

Same print in both regular shell, and sub-shell (i.e. after running zsh).

aha! the error message above says the "npm config" prefix option. Is there an .npmrc in your project dir, or in $HOME/.npmrc?

There is a $HOME/.npmrc file, but it just contains:
//registry.npmjs.org/:_authToken=***

I did brew uninstall node and now nvm will always take precedent over "system" because /opt/homebrew/bin (in the path) does not contain a node bin

That's certainly a workaround, but nvm should be putting its own directories first in the PATH, and the homebrew ones shouldn't be added after it.

I found this in my .zshrc:

#
# homebrew setup, following https://noahpeeters.de/posts/apple-silicon/homebrew-setup/
#
if [ -d "/opt/homebrew/bin" ]; then
    export PATH="/opt/homebrew/bin:$PATH"
fi

function ibrew() {
   arch --x86_64 /usr/local/bin/brew $@
}

# variables needed to properly install things under intel or m1

ARCH="$(uname -m)"
case ARCH in
    i386)   ARCH="386" ;;
    i686)   ARCH="386" ;;
    x86_64) ARCH="amd64" ;;
    arm)    dpkg --print-architecture | grep -q "arm64" && ARCH="arm64" || ARCH="arm" ;;
esac

# commenting this out removed the error
# if [[ "${ARCH}"  == "arm64" ]]; then
#     PREFIX="/opt/homebrew"
# else
#     PREFIX="/usr/local"
# fi

I think this has been answered.

I got this error with zsh because I'm loading the homebrew environment variables with this command in .zprofile:
eval "$(/opt/homebrew/bin/brew shellenv)"

And was loading NVM in .zshenv per the suggestion from React Native so that Xcode can find node:
export NVM_DIR="$HOME/.nvm"
[ -s "$NVM_DIR/nvm.sh" ] && \. "$NVM_DIR/nvm.sh"  # This loads nvm

I resolved it by adding the unset command as the NVM output suggests:
unset PREFIX # Workaround for homebrew
export NVM_DIR="$HOME/.nvm"
[ -s "$NVM_DIR/nvm.sh" ] && \. "$NVM_DIR/nvm.sh"  # This loads nvm

In fact, I switched to https://github.com/lukechilds/zsh-nvm, which loads NVM as a zsh plugin (using manual installation to ~/.zsh-nvm/), and now get autocomplete and lazy loading. 🎉 In .zhenv:
# NVM as zsh plugin: https://github.com/lukechilds/zsh-nvm
# Upgrade NVM with `nvm upgrade`
unset PREFIX # Workaround for homebrew
export NVM_COMPLETION=true
export NVM_LAZY_LOAD=true
export NVM_LAZY_LOAD_EXTRA_COMMANDS=('yarn')
source ~/.zsh-nvm/zsh-nvm.plugin.zsh

I solved the issue by adding unset PREFIX in .zprofile
# Brew
eval "$(/opt/homebrew/bin/brew shellenv)"

# NVM
unset PREFIX # Workaround for homebrew
export NVM_DIR="$([ -z "${XDG_CONFIG_HOME-}" ] && printf %s "${HOME}/.nvm" || printf %s "${XDG_CONFIG_HOME}/nvm")"
[ -s "$NVM_DIR/nvm.sh" ] && \. "$NVM_DIR/nvm.sh"


Theme file catppuccin-frappe.rasi was modified.

Look like catppuccin-frappe.rasi was accidentally modified by ab9fc7372b06f21e22286f8ce897747dbc0e72b2. It makes this theme doesn't work anymore.

Ah, apologies for overlooking this in ab9fc73.
I can't merge this PR though, since it just deletes everything from justTOBBI's fork.
Could you please undo the changes for the catppuccin-frappe.rasi file only?
Otherwise, I'll get to it, but it may take me a while.

I changed only catppuccin-frappe.rasi file.

oh... 🤦 Clicked on the wrong thing.
My bad. Thanks for your PR. 👍


Workflow and domain page responsiveness

Summary
Handle workflow & domain pages responsiveness.
Fixes

Increase spacing in wrapped workflow breadcrumbs
Wrap workflow history group date in small screens
Wrap workflow history action buttons on small screens
Hide workflow history compact list on small screens
Wrap domain info in small screens

Screenshots

I edited the markdown in the PR summary to fix the headings 😅


Versionbot: Add versionbot file and config

Can one of the admins verify this patch?

This file is not needed anymore. Closing PR


UI changes - My Applications

Address the following:
User interface (buttons, links, and visual styling)
Interaction design
Labels, headlines, and written text
Navigation structures and content classification
Accessibility (contrast, visual focus indicators, text alternatives, etc.)

My Applications design review Figma link: https://www.figma.com/file/LdBNHPjD3mlIggDrGYljs9/API-Services-Portal?node-id=1789%3A684
To edit, you need to login or sign up.


Scope presubmits by version

Only run v1beta2 tests if relevant or common code is changed. Same for v1.

This change is

Coverage remained the same at 76.744% when pulling 8b4f6ab30a2b51aa4890a5c47c29792b2c5f61e4 on richardsliu:scope_tests into 4a46eba093bbab2860f42223385e392741fd8793 on kubeflow:master.

/assign @gaocegege


UseSorting - Add parameter to specify a sorting fallback when one is not sent in the query

Is your feature request related to a problem?
When paging is used, sorting should be used.
An API author, at the moment, has no built-in way to specify what to sort by if the caller did not pass an order argument.
The solution you'd like
Two extension methods (similar to the ones below) and two exclusive options on UseSorting, to wit: string OrderByFallbackPropertyName and bool OrderByFallbackToPrimaryKey (these options would be exclusive)
public static IQueryable<T> OrderByArgumentOrDefault<T>(
    this IQueryable<T> query,
    IResolverContext context,
    Func<IQueryable<T>> func)
{
  return context.HasOrderArgument() ? query : func.Invoke();
}

    public static IQueryable<T> OrderByArgumentOrPrimaryKey<T, TKey>(
        this DbSet<T> dbSet,
        IResolverContext context)
        where T : class
    {
        if (context.HasOrderArgument())
        {
            return dbSet;
        }

        PropertyInfo? property = dbSet.EntityType.FindPrimaryKey()?.Properties.First().PropertyInfo;

        if (property == null || property.PropertyType != typeof(TKey))
        {
            return dbSet;
        }

        ParameterExpression parameterExpression = Expression.Parameter(typeof(T), "instance");
        MemberExpression propertyExpression = Expression.Property(parameterExpression, property);

        var lambda = Expression.Lambda<Func<T, TKey>>(propertyExpression, parameterExpression);

        return dbSet.OrderBy(lambda);
    }

Product
Hot Chocolate

That is actually not true :)
[UsePaging]
[UseFiltering(typeof(AssetFilterInputType))]
[UseSorting(typeof(AssetSortInputType))]
public IQueryable<Asset> GetAssets(AssetContext context, IResolverContext resolverContext)
    => resolverContext.ArgumentLiteral<IValueNode>("order").Kind is SyntaxKind.NullValue
        ? context.Assets.OrderBy(t => t.Symbol)
        : context.Assets;

Also there is a fluent API for this as well.

I mostly want the control in the attribute
The extensions were things we wrote that I was offering up
Having one that just figured out the primary key could be useful to others

@stand-sure  in v13 we add a sorting context. it's pretty easy to detect if sorting is provided
so you can easily do
if(!sortingIsProvided)
{
   queryable = queryable.OrderBy(x => x.Id);
}
return queryable;

That looks useful
Thanks


Generalize (re)render method

The static render and rerender methods of <sig-nal> are great, except for when we want to do more than assign to a property of a DOM node.
Take the case of manipulating the .classList property: instead of assignment, you choose a method (e.g. toggle) and supply a parameter (the class name as a string literal), then invoke said method.
We can generalize things if we can supply an optional transform function (default: the existing assignment).
The new rerender(transform) signature would be backwards incompatible with respect to existing hydration specs: we would now need to specify rerender() function application in all those places where hydration specs previously used just rerender, i.e. mere reference to the function without. application.
The classList case would use e.g.
rerender(({domNode, name, signal}) => domNode[name].toggle(signal.value))

Think this through a bit more.
Then implement it!

It seems better w.r.t. backwards compatibility and simplicity for simple cases to not extend rerender.
Instead, I propose to implement a new method renderWith({type,name, signal, signals, init, kind, domNode, domNodes, e) => /* callback body */).
The classList case would then be
renderWith(({domNode, name, signal}) => domNode[name].toggle(signal.value))


fix: add support for storage key

What kind of change does this PR introduce?
Address #19 to allow developer to configure storageKey when using createBrowserClient() client side. Not relevant for server-side as cookies are used there.

Wait why not createServerClient? These values must be synced up, otherwise wrong cookies will be read.

Initial consideration was that on server side they'd be able to set it via: options?.cookieOptions?.name  - was under impression from description that we primarily to want to allow the option of setting storageKey via options?.cookieOptions?.name on browser
It seems fine to allow this on createServerClient as well though. Pushed up to support on createServerClient as well

Hey @hf @j4w8n,
Rounding out old PRs here - could I trouble y'all for a review when free? Happy to close too if this is not something we wish to pursue

I just tested things as they are, in v0.5.1, and had no issues setting the cookie name via { auth: { storageKey: 'thing' } }. The ...auth?.options lines take care of this behavior. Has anyone been able to reproduce the issue? It tried to see if things changed with another PR, but didn't find anything.
No matter what, we should consider changing the order of things though. For instance, ...auth?.options is before ...(options?.cookieOptions?.name in the browser client, but they're reversed in the server client. Which means if someone happened to set both, and the values were different, then they'd get inconsistent results. I doubt someone would do those things, but it might benefit us to align the order.

Actually, perhaps as a side effect, @hf look to have resolved the user's issue with the v0.4.0 rewrite. Y'all can double-check, but I think we can close this PR.
@J0

Sounds good, thanks for the input!


Finishing exchange migration to enum

Description
In my previous PRhttps://github.com/whittlem/pycryptobot/pull/547, I introduced an exchange enum to avoid using strings when comparing exchanges, however, I kept the same behaviour, but I was comparing with the Enum value.
This change completes the migration and starts comparing the enums, instead of Strings.
Fixes # (issue)
Type of change
Please make your selection.

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update
[ ] This change requires a new dependency
[x] Code Maintainability / comments
[x] Code Refactoring / future-proofing

How Has This Been Tested?
Running bot in Live Mode, Non-live Mode and Sim Mode with --exchange being specified in the command line and getting the exchange from the config.json
Checklist:

[ ] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[ ] I have added pytest unit tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[x] Any dependent changes have been merged and published in downstream modules
[x] I have checked my code and corrected any misspellings

Note: this was a 2 parts migration:
1 - Introduced Enum and started comparing the values e.g. "binance, "coinbasepro".
2 - Typed every place where exchanges are used to use Enum instead of String an now the app compares Exchange.BINANCE == Exchange.BINANCE

@edrickrenan thanks for this. There are a lot of conflicts. Can you please check them and resolve? Thanks

@whittlem, the conflicts have been resolved.


Render CFStrings inline

I've noticed this method of lifting it results in the string always being lifted, even when it is used in dead code.
I figure this is probably a good thing, since it can expose nopped out logging info or erroneous dead code that HLIL would've hidden evidence of completely otherwise


Link to city map not being generated correctly

Foundry Version 10 - Build 288
module version 0.3.1
The link inside the burgs journal is incomplete/wrong so that it always generate a coastal city
for Reference, link in the journal
https://watabou.github.io/city-generator/?&citadel=1&walls=1&seed=3392497470474&name=Geth&population=1645&gates=-1
when you open it, you're redirected to
https://watabou.github.io/city-generator/?size=15&seed=3392497470474&name=Geth&population=1645&greens=0&farms=1&citadel=1&urban_castle=0&plaza=1&temple=1&walls=1&shantytown=0&coast=1&river=0&gates=-1&sea=0

link in the azgaar site
https://watabou.github.io/city-generator/?name=Geth&population=1645&size=16&seed=3392497470474&river=0&coast=0&farms=0&citadel=1&urban_castle=1&hub=false&plaza=0&temple=0&walls=1&shantytown=0&gates=-1

As much as I would love to solve this, it would require an extensive amount of work to replicate how it is done in FMG.
If anyone is curious, it's described here: https://github.com/Azgaar/Fantasy-Map-Generator/blob/a617dda3ad7dbc50aee384a1d0e0bd3a2f98035b/modules/ui/editors.js#L298
The main issue revolves around a LOT of missing functions and global variables that are incredibly difficult to recreate with my lack of knowledge on this part. I've updated to include support for the urban_castle parameter and the coast parameter so hopefully that will make it at least slightly better. I'll leave this issue open, however.


Add a plugin to enable log output to a file

Description
This PR adds a new plugin to the framework, which stores log messages to a file. This is done in addition to the default log output defined by the respective platform. So if you e.g. enable this on Windows, you still get the log messages in the separate console window, but also in a text file saved to disk.
Example usage:
vulkan_samples.exe sample afbc --log-file afbc.txt
Refs #372
General Checklist:
Please ensure the following points are checked:

[x] My code follows the coding style
[x] I have reviewed file licenses
[x] I have updated the header Copyright to reflect the current year (CI build will fail if Copyright is out of date)
[x] I have commented any added functions (in line with Doxygen)
[x] I have commented any code that could be hard to understand
[x] My changes do not add any new compiler warnings
[x] My changes do not add any new validation layer errors or warnings
[x] I have used existing framework/helper functions where possible
[x] My changes build on Windows, Linux, macOS and Android. Otherwise I have documented any exceptions
[x] My changes do not add any regressions
[x] I have tested every sample to ensure everything runs correctly
[x] This PR describes the scope and expected impact of the changes I am making

Merge on 3 approvals


Fatal error Missing file

Since this morning dxvk stopped working for Borderlands 2 and TPS.
Played with it flawless the last weeks. Now the game doesn‘t even launch and i get a fatal error popup with „missing file“ and something like „debug code 0==0“.
Can‘t describe it accurately because i‘m not at my pc right now. As soon as i delete the dxvk files the game starts. Reinstalling the games doesn‘t help.
win11
Rtx4090
Current release driver
Tried Dxvk version 2.3 , 2.4 and 2.4.1

That does not sound like a DXVK problem at all. It sounds like the game broke in general regardless of DXVK.

Attach dxvk logs

Win11, rtx4080laptop, steam version
Everything works, game runs smooth, no crash, no stutter
suggesting verify game file integrity first.

Borderlands2_d3d9.log
Verfied game files right now, same error for both BL2 and TPS. Logfile seems to be empty. what is this?

No idea tbh

Seems introduced by some overlay software.
Uninstall Geforce Experience (and NVIDIA app), uninstall RTSS, disable steam overlay, and try again.

@c00r5F Check the health of your hdd/ssd/nvme storages, just to make sure they're not the problem.
Hard Disk Sentinel
https://www.hdsentinel.com/download.php
https://hdsentinel.com/blog/how-to-check-ssd-health
CrystalDiskInfo
https://crystalmark.info/en/software/crystaldiskinfo/

Seems introduced by some overlay software. Uninstall Geforce Experience (and NVIDIA app), uninstall RTSS, disable steam overlay, and try again.

And in what case should be this the problem, when i never changed anything from one day to another? The game worked fine with dxvk for months.

@c00r5F Check the health of your hdd/ssd/nvme storages, just to make sure they're not the problem.
Hard Disk Sentinel https://www.hdsentinel.com/download.php https://hdsentinel.com/blog/how-to-check-ssd-health
CrystalDiskInfo https://crystalmark.info/en/software/crystaldiskinfo/

Hard drive is fine, any other game is working fine and how can it be the hard drive, if the games work fine without dxvk, but natively with dx9 it‘s running as it‘s supposed to be.
Don‘t take anything peronal here, but i‘m confused how that could be with only the implementation of dxvk in those games

And in what case should be this the problem, when i never changed anything from one day to another? The game worked fine with dxvk for months.
Some overlay softwares contain self-update feature and may update in the background without notification. You haven't seen any change, but the environment does changed due to these softwares. For example, Google never asks you to approve update of Chrome, but you should be using latest Chrome now.
Double check whether you have installed these softwares:

NVIDIA app & GeForce experience
RTSS
Overwolf
OBS
SteelSeries GG
If installed, try uninstall them all and test again.
Although in most cases these software should work fine, but in some rare conditions there are conflicts.

disabling nvidia overlay didn't help. it's the only overlay i'm running.

disabling nvidia overlay didn't help. it's the only overlay i'm running.

Then i have no idea.
Can you have another clean installation copy of Windows? There may be some environment conflict for your current installation.

I'm running borderlands 2 with nvidia overlay enabled, with rtss enabled, with wallpaper engine in background, and with rainmeter in background, rtx 4070ti, dxvk 2.4.1 and latest nvidia driver, no issues, win10 x64


Fix 3 tests using SerializerFeature.MapSortField

Existing tests are flaky because they rely on the ordering of elements in a map. To fix it, use SerializerFeature. MapSortField to force ordering when convert JSON to string, so the converted string will be deterministic.
The flaky tests were found by running NonDex (https://github.com/TestingResearchIllinois/NonDex).

Codecov Report

Merging #3571 (65fcbcc) into master (37ed66b) will decrease coverage by 0.03%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master    #3571      +/-   ##
============================================
- Coverage     80.41%   80.37%   -0.04%
+ Complexity     7491     7490       -1
============================================
  Files           157      157
  Lines         27929    27960      +31
  Branches       6737     6748      +11
============================================
+ Hits          22459    22474      +15
- Misses         3462     3469       +7
- Partials       2008     2017       +9

Impacted Files
Coverage Δ
Complexity Δ

...main/java/com/alibaba/fastjson/util/FieldInfo.java
85.76% <0.00%> (-1.00%)
96.00% <0.00%> (-3.00%)

...ba/fastjson/parser/deserializer/Jdk8DateCodec.java
77.38% <0.00%> (-0.80%)
107.00% <0.00%> (+1.00%)
:arrow_down:

.../fastjson/parser/deserializer/MapDeserializer.java
85.65% <0.00%> (-0.19%)
69.00% <0.00%> (+2.00%)
:arrow_down:

...n/java/com/alibaba/fastjson/util/JavaBeanInfo.java
86.89% <0.00%> (-0.17%)
217.00% <0.00%> (-1.00%)

...java/com/alibaba/fastjson/parser/ParserConfig.java
80.61% <0.00%> (-0.15%)
227.00% <0.00%> (-1.00%)

...com/alibaba/fastjson/parser/DefaultJSONParser.java
87.47% <0.00%> (-0.08%)
323.00% <0.00%> (ø%)

...ava/com/alibaba/fastjson/serializer/JodaCodec.java
54.08% <0.00%> (+0.82%)
52.00% <0.00%> (+2.00%)

Continue to review full report at Codecov.

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


Disable nightly shared services E2E

What is being addressed
Gitea fails to install in the environment representing the main branch. As the issue is related to KV, recovering secrets, etc. it's notoriously hard to solve...
To bring some more stability to the nightly E2E run, I suggest to remove it for now.

/test-force-approve

/test-force-approve


[SMALLFIX] Disable secondary master in the integration tests

It slows down the tests. Disable for now.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/14639/
Test PASSed.

Merged build finished. Test PASSed.


chore: update polkadot to v1.13.0

Link to issue: https://github.com/r0gue-io/base-parachain/issues/34

[x] Node
[x] Runtime

Compilation Outcome

✅ rustc 1.77.2 (25ef9e3d8 2024-04-09)

CHANGELOG

[x] Update crates to polkadot 1.13.0 with psvm -v 1.13.0
[x] Remove parameterized-consensus-hook feature flag. Mentioned here

- cumulus-pallet-parachain-system = { version = "0.12.0", default-features = false, features = ["parameterized-consensus-hook"] }
+ cumulus-pallet-parachain-system = { version = "0.13.0", default-features = false }

[x] The RuntimeGenesisConfig generic type parameter was removed from GenericChainSpec struct. Mentioned here

- pub type GenericChainSpec = sc_service::GenericChainSpec<(), Extensions>;
+ pub type GenericChainSpec = sc_service::GenericChainSpec<Extensions>;

[x] Bound XCMP queue
[x] Pin dependency version

mmr-rpc = "=35.0.0"
polkadot-node-core-pvf-common = "=14.0.0"
sc-network-types = "=0.12.0"
sp-runtime = { version = "=38.0.0", default-features = false }
sc-client-api = "=35.0.0"

I am getting an error:
use of undeclared crate or module `frame_metadata_hash_extension`

I think it only needs to import frame-metadata-hash-extension (https://github.com/r0gue-io/contracts-parachain/blob/main/Cargo.toml#L43).
Before merging this PR, I would like to merge these two PRs first so that we can have everything in v1.12 before moving base-parachain to v1.13.

https://github.com/r0gue-io/contracts-parachain/pull/7
https://github.com/r0gue-io/assets-parachain/pull/6


feat: flavored types

Less strict version of brand. The code is basically the same, just a couple of more test cases.

Thank you for creating this PR. Can you explain the pros and cons of flavor vs. brand? What are the use cases for flavor? Do other schema libraries have the same or a similar feature in addition to brand?

Thank you for creating this PR. Can you explain the pros and cons of flavor vs. brand? What are the use cases for flavor? Do other schema libraries have the same or a similar feature in addition to brand?

I'm not aware of other validation libraries using flavored types in addition to branded ones, but there's a database schema library called kanel that makes use of them. The purpose there is to avoid mixing up ids of different entities, but not require the use of a validation library or custom functions for casting new ids to be used as arguments.
I prefer using flavored types for ids and ISO timestamps because all the existing values are more or less guaranteed to be flavored from either validated API inputs or database queries. There's a loss in ergonomics for no or very little extra safety when branded types force me to implement wrapped functions a la createUserUuidv4() or createUuidv4('UserId') or toIsoTimestamp(new Date()). There are some more examples in this blog post.
tl;dr - flavored types don't get in my way, but will stop me from mixing up validated data (and all my data is validated.) In critical places I use runtime assertions and/or branded types.

Is the only difference that the FlavorSymbol key is optional while the BrandSymbol key is required?

Is the only difference that the FlavorSymbol key is optional while the BrandSymbol key is required?

That's right.

Am I understanding correctly that a flavored type can be passed on to an unflavored type, but not to another flavored type, and that's the difference with a branded type and the idea behind flavor? Here is an example:
type A = string & { foo?: 'A' };
type B = string & { foo?: 'B' };

function createA(): A {
  return 'hello';
}

const test1: A = createA();       // This works
const test2: string = createA();  // This works
const test3: B = createA();       // But this does not work

Yes, I added similar examples to the docs here.
In essence:

Flavored<string, 'a'> is a valid string and vice versa
Flavored<string, 'b'> is not a valid Flavored<string, 'a'>

In general, I like the idea of a flavor action. I am just careful not to make the library too complicated. It will be very important to document the difference to brand well so that users do not get confused. Do you have an alternative name for this action?

Maybe weakBrand? Could even use the same symbol for convenient interop.

Here is a list of names with Brand as a suffix:

weakBrand
softBrand
partialBrand
optionalBrand

And here is a list of completely new and independent names:

flavor
mark
tag
flag
tint

What's your favorite so far?

I like weakBrand the most + using the same symbol. I'll update the PR if you're ok with that.

I am not sure we should use the same symbol. This could cause problems when people extend existing schemas that end up using both actions. Let me think about the name a bit more. I'm not sure what's best yet.

Thanks for your feedback. I will have to investigate this before moving forward. I may prioritize our v1 RC first before doing this.


[SPARK-23161][PYSPARK][ML]Add missing APIs to Python GBTClassifier

What changes were proposed in this pull request?
Add featureSubsetStrategy in GBTClassifier and GBTRegressor.  Also make GBTClassificationModel inherit from JavaClassificationModel instead of prediction model so it will have numClasses.
How was this patch tested?
Add tests in doctest

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/3516/
Test PASSed.

Test build #91053 has finished for PR 21413 at commit 16d19f4.

This patch fails MiMa tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
class GBTClassificationModel(TreeEnsembleModel, JavaClassificationModel, JavaMLWritable,

Merged build finished. Test FAILed.

jenkins retest this please

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/3527/
Test PASSed.

Test build #91072 has finished for PR 21413 at commit 16d19f4.

This patch passes all tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
class GBTClassificationModel(TreeEnsembleModel, JavaClassificationModel, JavaMLWritable,

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/91072/
Test PASSed.

Test build #91116 has started for PR 21413 at commit 0631f8d.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/3558/
Test PASSed.

Test build #91127 has started for PR 21413 at commit 76b57f4.

Test build #91127 has finished for PR 21413 at commit 76b57f4.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/91127/
Test PASSed.

Test build #91280 has started for PR 21413 at commit 714ab33.

Test build #91280 has finished for PR 21413 at commit 714ab33.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/91280/
Test PASSed.

Test build #91282 has started for PR 21413 at commit d8f3906.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/3689/
Test PASSed.

Test build #91282 has finished for PR 21413 at commit d8f3906.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/91282/
Test PASSed.

Merged build finished. Test PASSed.

merged to master, thanks @huaxingao !

Thanks a lot @BryanCutler for your help!

Thanks a lot @BryanCutler for your help!


Build up an overridable config system

We will need a basic config system that stores config in some sort of container.  Perhaps in an object or simply a dictionary?

@spatialbits Ideas?

@gregseb can you describe a little what you are thinking the config system should manage and how it would cooperate with the geomark module?

Anything that could possibly change in the future, and a debug setting.
So at the top of my head.... the base_url and debug.   Chances are base_url will likely never change.  But it could and although I'd like to say we'll be plenty fast enough to update the code we might not be fast enough for someone, so to be safe we should just declare it as a config option.  Plus its a convenient way to keep it somewhere easily accessible by any other modules we create.

Nice work @gregseb


[Bug] "TypeError: Cannot read properties of undefined (reading 'length')"

Describe the bug
Similarly to https://github.com/eikofee/eikonomiya-front/issues/8, when I try to access informations of certain UIDs, there is an error instead. But since the error seems to be different and be thrown from a different place, I report this issue separately. When trying to access 704207445, I get:
TypeError: Cannot read properties of undefined (reading 'length')
    at f.parseEffect (/app/.next/server/chunks/667.js:1:10130)
    at f.getWeaponEffects (/app/.next/server/chunks/667.js:1:13275)
    at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
    at async f.loadPlayerData (/app/.next/server/chunks/667.js:1:16301)
    at async s (/app/.next/server/app/[uid]/page.js:1:2213)
[Error: An error occurred in the Server Components render. The specific message is omitted in production builds to avoid leaking sensitive details. A digest property is included on this error instance which may provide additional details about the nature of the error.] {
  digest: '3653815343'
}

To Reproduce
Steps to reproduce the behavior:

Try to access http://localhost:3000/704207445

Additional context
The list of non working UIDs is of course not exhaustive, but they don't work on my side. Note that Enka returns for all of these a valid avatarInfoList field. In fact, I suspect the application either to not handle properly the avatar of the player. Indeed, on some working UIDs, it is stored as this:
{
    "playerInfo": {
        "profilePicture": {
            "id": 7400
        }
    ...
}

While on the UID I listed above, the structure is different:
{
    "playerInfo": {
        "profilePicture": {
            "avatarId": 10000015,
            "costumeId": 201501
        }
    ...
}

Mostly fixed on data side.
Assets in the profile are bugged though but should be fixed with 02b0033 and 873ade7 and upcoming fix.


Permission generation and registering using single file(Permissions)

Replace Permission generation using PermissionModule with Permission generation using Permissions class and reflection

For the issue #138


front-admin : view owner

List customer(company) + associated customer right account

List of request to become customer

ability to create customer(company) + customer accounts

List of customer request ? (issue or else ?)

ticket wasn't clear enough


Correct Layout reproducer

When the CI Job reproducer was introduced, we missed the need of a new
parameter in order to inject the actual network layout we're consuming
in a non-CI job case.
This patch also allows to bootstrap CRC directly during the reproducer
run, provided we define it in the layout (it's usually the case).
As a follow-up, we may want to properly manage pull-secret copy from the
host running the playbook onto the hypervisor, in case those are two
distinct nodes. Since this is a feature shared with devscripts, we may
want to get a common role for that.
We also update the doc to reflect this change, since we don't need to
have a CRC pre-provisionned anymore. You still may have it ready, but
it's not mandatory anymore.
As a pull request owner and reviewers, I checked that:

[X] Appropriate testing is done and actually running
[X] Appropriate documentation exists and/or is up-to-date:

[X] Content of the docs/source is reflecting the changes

/hold
made a mistake - missing the uri: qemu:///system for virt commands.

/unhold

recheck


Docker image can't be pulled

Hi,
I am trying to pull the Docker image but I get the following error:
$ docker pull crowdsecurity/cloudflare-worker-bouncer

Using default tag: latest
Error response from daemon: manifest for crowdsecurity/cloudflare-worker-bouncer:latest not found: manifest unknown: manifest unknown

Hi, I am trying to pull the Docker image but I get the following error:
$ docker pull crowdsecurity/cloudflare-worker-bouncer

Using default tag: latest
Error response from daemon: manifest for crowdsecurity/cloudflare-worker-bouncer:latest not found: manifest unknown: manifest unknown

We are currently working on getting the image uploaded we had some issues in the build pipeline


new coordinate properties

Which issue(s) are closed by this pull request?
Closes #353
Changes proposed in this pull request:

include properties for each domain and convention, such as, .cartesian, etc
include "different constructor" for each domain and convention, such as .from_sperical_colatitude(),...
coordinates are always saved in cartesian coordinates
add Deprecation for old set_sph and get_sph ...
init will be changed to init(x, y, z, comment="", weights="")
sh_order will be shifted to SamplingSphere
add PyfarDeprecationWarning

todo

[x] update example
[x] fix and update documentation

ready for review doc

please also have a look on the notebook

we decided to add a link to the conepts for each coordinate system property and constructor

We decided to change the names of theta and phi angles. The best suggestion so far is wipe and shoot.

all suggestions applied

Like @f-brinkmann stated, the concepts image should still be updated.


Update CDataInterface.rst （Did you write it wrong? I think it should be API）

Did you write it wrong? It should be api

Rationale for this change

What changes are included in this PR?

Are these changes tested?

Are there any user-facing changes?

Did you write it wrong? I think it should be API

Thanks for opening the issue.
ABI is correct. This is talking about the Application Binary Interface: https://en.wikipedia.org/wiki/Application_binary_interface
I am closing the PR.


Fatal error: Unexpectedly found nil while unwrapping an Optional value

How frequently does the bug occur?
Always
Description
We have a server function that has no arguments:
let realmResponse = try await app.currentUser?.functions.deleteAccount()
After performing the call the app crashes on FunctionCallable see screenshot:

Stacktrace & log output
RealmSwift/Sync.swift:1144: Fatal error: Unexpectedly found nil while unwrapping an Optional value

Can you reproduce the bug?
Always
Reproduction Steps
No response
Version
10.52.3
Are you using encryption?
No
Platform OS and version(s)
17.5
Build environment
Xcode version: 15.4.0

It looks like both the type and implementation of the async version of user.functions are very incorrect.

It looks like both the type and implementation of the async version of user.functions are very incorrect.

What exactly do you mean?
Following the documentation I don't see any issue.

Well yes, the documentation isn't going to say "we implemented this incorrectly".

And how should I use it correctly?


Adding Twitter and Social image preview including description

When sharing on socials, this change will preview an image, title, and the actual description of the note (if not home/index page).
An example of how Twitter shows it:

I didn't and am unsure when I will get to it.


Move CORS sample policy out of ./test

See #123.

@specialkk Can you rebase this branch? LGTM otherwise.


When downloading source as a zip, loading in gamemaker fails to load all sprites

did i do something wrong?

@alexkisser Please follow the Bug Report template provided in this repository; the requested information if important for helping to diagnose the issue.

ok


fix: handle error exchange code for session

What kind of change does this PR introduce?

Fixes https://github.com/supabase/auth-js/issues/782

What is the current behavior?
Please link any relevant issues here.
What is the new behavior?
Feel free to include screenshots if it includes visual changes.
Additional context
Add any other context or screenshots.

see #946


Windows NPM Fallback

Describe the bug
A clear and concise description of what the bug is.
When running reflex run bun errored and reflex didn't fall back to Windows it just stated make an issue on the bun GitHub.
Additional context
Add any other context about the problem here.

fixed in #3138