MAINT: Remove old & unused code

I don't think there's any reason to keep these around... let me know if I've overlooked something

We do still need to require the ipython package because we use the display function. Other than that looks good!

I only removed the jinja2 requirement, which we no longer use. I added the ipython requirement here!

Ahhh, sorry I missed the coloring in the diff. Merging...

:cry:

I dare you to implement a full matplotlib rendered based on the 1.x
vega-lite spec ;-)
On Thu, May 26, 2016 at 4:25 PM, Thomas A Caswell<EMAIL_ADDRESS>wrote:

😢
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/ellisonbg/altair/pull/109#issuecomment-222022833, or mute
the thread
https://github.com/notifications/unsubscribe/AABr0JHRXdg2uEW1YxJCOSlmaAwxiYzzks5qFiv8gaJpZM4Ik0Xb
.

--
Brian E. Granger
Associate Professor of Physics and Data Science
Cal Poly State University, San Luis Obispo
@ellisonbg on Twitter and GitHub
<EMAIL_ADDRESS>and<EMAIL_ADDRESS>

That is not nice @ellisonbg, not nice at all.... :smiling_imp:
Some what more seriously, I have not been able to keep up with this.  Is the problem of understanding/implementing the data transformations solved?

Yes, the Vega-Lite documentation is greatly improved and descibes the
transformations very well now:
https://vega.github.io/vega-lite/docs/transform.html
On Thu, May 26, 2016 at 5:16 PM, Thomas A Caswell<EMAIL_ADDRESS>wrote:

That is not nice @ellisonbg https://github.com/ellisonbg, not nice at
all.... 😈
Some what more seriously, I have not been able to keep up with this. Is
the problem of understanding/implementing the data transformations solved?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/ellisonbg/altair/pull/109#issuecomment-222029608, or mute
the thread
https://github.com/notifications/unsubscribe/AABr0CFrd-RaxNnGpkpfFTOFcD_JZSsVks5qFjfBgaJpZM4Ik0Xb
.

--
Brian E. Granger
Associate Professor of Physics and Data Science
Cal Poly State University, San Luis Obispo
@ellisonbg on Twitter and GitHub
<EMAIL_ADDRESS>and<EMAIL_ADDRESS>

We've decided to narrow the scope and let the Vega-lite library do all the rendering for us, so we no longer have to worry about data transformations on our end.

But if we wanted a matplotlib renderer (I would still like to see this) we would need to understand the transformation.
Also, I have been thinking about APIs for doing things like regression (scatter + fit line). If you want that to work with different groupby's (faceting, color, etc), we would need to do those groupby's in Python, do line fits on each group and then generate a new dataset that has the fitted line data. Something like this:
RegressionLayer(raw_data).mark_line().encode(
    X('raw_x:Q'),
    Y('raw_y:Q'),
    Color('groupby1'),
    Row('groupby2')
)

This would take the raw data, do a groupby on the color and row and then do a fit on each of those groups. All of that would give back a dataframe that would yield the fitted line data. This would require us to have the transformation layers implemented in Python.


[Bug]: Inconsistency in program execution with named workers

Description
$title
Steps to Reproduce
When we run the following code it runs indefinitely.
import ballerina/io;

public function main() {
    worker w1 {
        io:println("Hello from worker w1");
        while true {
            io:println("loop");
        }

    }
}

gives
Hello from worker w1
loop
loop
loop
.
.
.

But when we add a runtime:sleep() to yield inside the worker, the program execution stops.
import ballerina/lang.runtime;
import ballerina/io;

public function main() {
    worker w1 {
        io:println("Hello from worker w1");
        while true {
            io:println("loop");
            runtime:sleep(1);
        }
    }
}

gives
Hello from worker w1
loop

Affected Version(s)
No response
OS, DB, other environment details and versions
No response
Related area
-> Runtime
Related issue(s) (optional)
https://github.com/ballerina-platform/ballerina-lang/issues/28536
Suggested label(s) (optional)
No response
Suggested assignee(s) (optional)
No response

Since in Ballerina we have cooperative multitasking by yielding, we cannot identify the change to a stable state till the worker yields. It is not good to forcefully exit the program at the end of the main function. We have to stop the program gracefully. Hence, the user has to expect this behaviour when using such infinite loops, and in this first code sample, the program will not exit though the main function ends.


Handle case where collection has no members

When using the ranked abundance app, we see:

(also fails on beta.mbio)
Logs:

  invalid class “VariableMetadata” object: Members must be non-empty for collection variables.

I'm going to guess this is related to https://github.com/VEuPathDB/microbiomeComputations/issues/31

decided this behavior is perfect! It's telling us exactly what's wrong. Means we have an issue with the data itself. Closing!


Refactor Rep to Reps for consistency

Codecov Report
Base: 72.15% // Head: 72.15% // No change to project coverage :thumbsup:

Coverage data is based on head (1fe8797) compared to base (78aa441).
Patch coverage: 94.73% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff            @@
##             master      #67   +/-   ##
=========================================
  Coverage     72.15%   72.15%
  Complexity      399      399
=========================================
  Files            70       70
  Lines          1232     1232
  Branches        125      125
=========================================
  Hits            889      889
  Misses          311      311
  Partials         32       32

Impacted Files
Coverage Δ

src/main/java/gim/logic/commands/AddCommand.java
100.00% <ø> (ø)

src/main/java/gim/model/util/SampleDataUtil.java
20.00% <ø> (ø)

src/main/java/gim/ui/ExerciseCard.java
0.00% <0.00%> (ø)

src/main/java/gim/model/exercise/Exercise.java
97.61% <87.50%> (ø)

src/main/java/gim/logic/commands/EditCommand.java
97.01% <100.00%> (ø)

...c/main/java/gim/logic/parser/AddCommandParser.java
100.00% <100.00%> (ø)

src/main/java/gim/logic/parser/CliSyntax.java
83.33% <100.00%> (ø)

.../main/java/gim/logic/parser/EditCommandParser.java
92.30% <100.00%> (ø)

src/main/java/gim/logic/parser/ParserUtil.java
97.22% <100.00%> (ø)

src/main/java/gim/model/exercise/Reps.java
80.00% <100.00%> (ø)

... and 1 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


fix: valueId metadata out of sync with ui

Fixes #3034

cc @AlCalzone, could you give this a try please?

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 7 (0.0%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.004%) to 23.477%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

lib/SocketEvents.ts
0
1
0.0%

lib/ZwaveClient.ts
0
6
0.0%

Totals

Change from base Build<PHONE_NUMBER>:
-0.004%

Covered Lines:
3818

Relevant Lines:
17179

💛  - Coveralls


[python-package] monotonic constraints don't work

Description
I've trained a monotonic binary classifier with 'monotone_constraints_method': 'advanced', but as per the example below, it doesn't seem to work : (
Reproducible example
import lightgbm as lgb
import pandas as pd

bst = lgb.Booster(model_file='model_file_test.txt')
df = pd.read_csv('test_data.csv')
df['pred_score'] = bst.predict(df.drop(columns=['pred_score']))

In the test_data.csv, the two records only differ in feature columns col_13 and col 15.

col_13
col_15

0
9
2

1
28.3156374
2.32535749

And in the model_file_test.txt,
feature_names=col_1 col_2 col_3 col_4 col_5 col_6 col_7 col_8 col_9 col_10 col_11 col_12 col_13 col_14 col_15 col_16 col_17 col_18 col_19 col_20 col_21 col_22 col_23 col_24 col_25 col_26 col_27 col_28 col_29 col_30 col_31
monotone_constraints=1 -1 1 1 1 1 1 1 1 1 1 1 1 1 1 1 1 1 1 1 1 1 1 1 1 1 -1 -1 -1 -1 -1

Features col_13 and col_15 are constrained to be monotonic increasing.
However, the pred_scores from bst.predict() seem to decrease as the feature values increase.

pred_score

0
0.41453699

1
0.41169457

Environment info
LightGBM version or commit hash: 4.0.0
Command(s) you used to install LightGBM:
python3.9 -m pip install lightgbm

But the default method works, right?

@yuanf8 Could you provide a real minimal reproducible example, i.e. including the model fitting part which is crucial here? Otherwise, it’s impossible to hunt the error.


libextractor: 1.6 -> 1.7

Semi-automatic update generated by https://github.com/ryantm/nixpkgs-update tools. This update was made based on information from https://repology.org/metapackage/libextractor/versions.
meta.description for libextractor is: '"Simple library for keyword extraction"'.

Checks done (click to expand)

built on NixOS
/nix/store/0v9g2s4wfsmdwdbgh80mxb8as4cqd7yj-libextractor-1.7/bin/extract passed the binary check.
1 of 1 passed binary check by having a zero exit code.
1 of 1 passed binary check by having the new version present in output.
found 1.7 with grep in /nix/store/0v9g2s4wfsmdwdbgh80mxb8as4cqd7yj-libextractor-1.7
directory tree listing: https://gist.github.com/6778cee4f7935f265da8f142df4ba5e6
du listing: https://gist.github.com/c4ed8fc1ad77b6e8cf8dd4ed036a05af

Test this build with Cachix (click to expand)

One time setup in nixpkgs Git checkout:
cachix use r-ryantm
git remote add r-ryantm https://github.com/r-ryantm/nixpkgs.git

Test this build:
git fetch r-ryantm && git checkout a065910680875f9e4be7b06c4bad41bf5653a453 && nix-shell --pure -I nixpkgs=. -p libextractor

No attempt on x86_64-darwin (full log)
The following builds were skipped because they don't evaluate on x86_64-darwin: libextractor
Partial log (click to expand)

a) For `nixos-rebuild` you can set
  { nixpkgs.config.allowUnsupportedSystem = true; }
in configuration.nix to override this.

b) For `nix-env`, `nix-build`, `nix-shell` or any other Nix command you can add
  { allowUnsupportedSystem = true; }
to ~/.config/nixpkgs/config.nix.

Success on x86_64-linux (full log)
Attempted: libextractor
Partial log (click to expand)
shrinking /nix/store/0v9g2s4wfsmdwdbgh80mxb8as4cqd7yj-libextractor-1.7/lib/libextractor/libextractor_rpm.so
shrinking /nix/store/0v9g2s4wfsmdwdbgh80mxb8as4cqd7yj-libextractor-1.7/lib/libextractor/libextractor_wav.so
shrinking /nix/store/0v9g2s4wfsmdwdbgh80mxb8as4cqd7yj-libextractor-1.7/lib/libextractor/libextractor_tiff.so
shrinking /nix/store/0v9g2s4wfsmdwdbgh80mxb8as4cqd7yj-libextractor-1.7/lib/libextractor/libextractor_nsf.so
gzipping man pages under /nix/store/0v9g2s4wfsmdwdbgh80mxb8as4cqd7yj-libextractor-1.7/share/man/
strip is /nix/store/90vmpr41dzsx350k5argycaf693hnl1l-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/0v9g2s4wfsmdwdbgh80mxb8as4cqd7yj-libextractor-1.7/lib  /nix/store/0v9g2s4wfsmdwdbgh80mxb8as4cqd7yj-libextractor-1.7/bin
patching script interpreter paths in /nix/store/0v9g2s4wfsmdwdbgh80mxb8as4cqd7yj-libextractor-1.7
checking for references to /build in /nix/store/0v9g2s4wfsmdwdbgh80mxb8as4cqd7yj-libextractor-1.7...
/nix/store/0v9g2s4wfsmdwdbgh80mxb8as4cqd7yj-libextractor-1.7

Success on aarch64-linux (full log)
Attempted: libextractor
Partial log (click to expand)
shrinking /nix/store/vh1zkrlv5a0011gllgm2sh7kgx7v9ysx-libextractor-1.7/lib/libextractor/libextractor_it.so
shrinking /nix/store/vh1zkrlv5a0011gllgm2sh7kgx7v9ysx-libextractor-1.7/lib/libextractor/libextractor_dvi.so
shrinking /nix/store/vh1zkrlv5a0011gllgm2sh7kgx7v9ysx-libextractor-1.7/lib/libextractor_common.so.1.0.0
shrinking /nix/store/vh1zkrlv5a0011gllgm2sh7kgx7v9ysx-libextractor-1.7/lib/libextractor.so.3.1.6
gzipping man pages under /nix/store/vh1zkrlv5a0011gllgm2sh7kgx7v9ysx-libextractor-1.7/share/man/
strip is /nix/store/7iyn7gn33i7xxjgmwf25k20246y6nd9d-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/vh1zkrlv5a0011gllgm2sh7kgx7v9ysx-libextractor-1.7/lib  /nix/store/vh1zkrlv5a0011gllgm2sh7kgx7v9ysx-libextractor-1.7/bin
patching script interpreter paths in /nix/store/vh1zkrlv5a0011gllgm2sh7kgx7v9ysx-libextractor-1.7
checking for references to /build in /nix/store/vh1zkrlv5a0011gllgm2sh7kgx7v9ysx-libextractor-1.7...
/nix/store/vh1zkrlv5a0011gllgm2sh7kgx7v9ysx-libextractor-1.7


$openai->completion (singular) works where opanai documentation specifies https://api.openai.com/v1/completions (plural)

Describe the bug
The README places links to OpenAI's reference documentation above the fold (at the very beginning), creating the impression that the library's methods have the same name as the path of the OpenAI endpoints.  For example, it may be common for new users of the library to assume Completions are accessed by a method names "completions" such as:

$openai->completion() (spelled singular)

since the linked OpenAI reference documentation for "completions" endpoint describes endpoint as
POST  https://api.openai.com/v1/completions (spelled plural)
Yet, the library's method is completion (spelled singular)

$openai->completion() (spelled singular)

This is made clear further down the README, but it's possible many new users will get sidetracked following links to OpenAI's reference and trying to copy the endpoint paths as library method names.  Suggest several possible fixes:

At the top of the Readme, calling out the fact method names are described further down the README.

Create a prominent table mapping the OpenAI Reference endpoints to the library's methods.

Thnx
To Reproduce
$openai->completion()
Code snippets
No response
OS
Linux
PHP version
PHP 7.4
Library version
openai v3

Thanks but I'm not sure this is indispensable, but Ill watch the feedback if we get a lot of requests about this feature. I'll take a look. now pr is closing


added Melophobia- Cage the Elephant

14. Melophobia 🦇
🎼 Music Composer:  Cage The Elephant
👍🏻 Best for:  Lightening your mood 🔥
🔗 Links: Spotify, Apple Music , Youtube

@asaw4
Thanks a lot for your contribution. What this is a soundtrack only list. 😅

this is an album . I am sorry, I didn't get you

It's good album but this list is of non lyrical music === soundtracks


Network traffic not forwarded from Windows browser to WSL2 distro

Windows Version
W10 21H2
WSL Version
WSL 2
Are you using WSL 1 or WSL 2?

[ ] WSL 2
[x] WSL 1

Kernel Version
No response
Distro Version
Ubuntu 24.04
Other Software
No response
Repro Steps
Set export BROWSER=/usr/bin/wslview in your .bashrc file.
Run aws configure sso in your bash terminal.
Expected Behavior
I expected the localhost response would reach the WSL2 machine.
Actual Behavior
After running aws configure sso, the default Windows browser opens via wslview. However, after completing the authentication in the browser, the localhost response doesn't reach the WSL2 machine.
Diagnostic Logs
No response

/question


Basic Schema.org usage for HTML version

See #7

This is the actual schema generated by the the README.md. It's not bad considering it's the github default output

If we generate a new version with HTML, lets use some extra logic for structured data. For now I'm not sure the best types to use for our type of content, but at least use something generic to start.

(The link to preview the last image is https://search.google.com/structured-data/testing-tool/u/0/#url=https%3A%2F%2Ftags.etica.ai%2Fpoc.html.)
And this is our outline, direct link https://validator.w3.org/nu/?doc=https%3A%2F%2Ftags.etica.ai%2Fpoc.html&showoutline=yes.

The default does not use tags like <section>, <article>, <aside>, etc. So heading level is implicit. Let's make it more explicit.
Both images can be used later as reference

Raw HTML is so flexible for fine tuning. Some heading levels where used just to make easier for automatic ToC with one VSCode extra extension where not ideal ones. But raw HTML for this specific project is better for sure. So lovely.

Version in production now:
Outline

https://validator.w3.org/nu/?doc=https%3A%2F%2Ftags.etica.ai%2F&showsource=yes&showoutline=yes

Schema (on Google Rich Snippets tool)

https://search.google.com/structured-data/testing-tool/u/0/?hl=pt-BR#url=https%3A%2F%2Ftags.etica.ai%2F


Feature to wrap up

Let's wrap up a docker build and composer

@jjhesk will have limited availability to check on pr but lets continue the discussion in the pr thread, will most prob merge after a few rounds, thanks for the submission


MissingPropertyException for "it" in closures

The code below throws a MissingPropertyException when using it inside of closures after version 1.5.
Example 1:
log = ''
(1..10).each{log += it}

Example 2:
list =["foo", "bar"]
index = list.findIndexOf{ it == "foo"}

Already filed as JENKINS-33468.

Hi guys, Is there any fundamental issue preventing fixing this? Any plan to fix it in near future?
Thanks.

I cannot use Jenkinsfile from SCM and I am blocked due to this

Is there any fundamental issue preventing fixing this?

Unknown until someone tries to fix it.

Any plan to fix it in near future?

Not currently. Feel free to upvote the JIRA issue or file a PR.

I am blocked due to this

Unlikely. There is a trivial workaround, as described in JIRA.

https://github.com/kohsuke/groovy-sandbox/pull/34/commits/1fffecf0e53f3901559acf707e7e4f766cd33d0d
This change adds support for implicit "it"

@jglick could you have a look at my PR submitted for this fix? thanks.

Refiled as https://github.com/jenkinsci/groovy-sandbox/pull/38 by @abayer.

Going by Git tags, looked like the fix for this was released in version 1.14.


Heading component is missing when you use Tailwind

Describe the Bug
When you wrap Body with Tailwind whenever I use Heading inside the body, its not showing up in the 'email dev', if i remove the tailwind, the Heading component show back up? Any fix or am i doing it wrong somehow?
Also how do reset all the component's css? like Text has margin i want to reset them.
Which package is affected (leave empty if unsure)
@react-email/tailwind
Link to the code that reproduces this issue
none
To Reproduce

  <Preview>{previewText}</Preview>
  <Tailwind>

      <Heading className="text-black text-[24px] font-normal text-center p-0 my-[30px] mx-0">
        Join {teamName} on Vercel
      </Heading>

  </Tailwind>

Expected Behavior
you should see  Join {teamName} on Vercel  on the screen but none show up

What's your node version? (if relevant)
No response

Closing as a duplicate of #1391.
It should be working under @react-email/tailwind 0.0.16-canary.1.


Not an actual issue

Sorry to ask this here but I dont know how else to conctact you and have been looking for this for a while...
How do you get that nice c# syntax highligthing in your screenshots on the main page? https://github.com/fakesharper/fakesharper
Do you use an extention and custom theme? Cause my syntax highligting in vscode is no where near this detailed.
Thank you.

Hi @thecrazy , it's one monokai theme

Thank you!
On Mon, Jul 5, 2021 at 8:38 AM jaqra @.***> wrote:

Hi @thecrazy https://github.com/thecrazy , it's one monokai theme
https://marketplace.visualstudio.com/items?itemName=azemoh.one-monokai
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/fakesharper/fakesharper/issues/19#issuecomment-874080433,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AASM3NAO7ZW4VEI3OUI4ZQTTWGRUVANCNFSM47TTIZYQ
.


Default env is loaded even if it is not specified.

Expected Behaviour
Gauge should load environments that is specified using the --env flag. If there are no environment specified it should load the default environment.
Actual Behaviour
Gauge is loading the default environment even if some other environment is specified in the --env flag.
Steps to Reproduce

Create a simple Gauge project
Add a new env foo with the following additional properties

my_foo_variable = bar

Edit the default/default.properties and add the following

my_default_variable = Definetly not Bar!!!

Print the my_foo_variable and my_default_variable in a step implementation and implement the step(eg in js)

step("Print environment variables", function() {
  console.log("The default variable is", process.env.my_default_variable)
  console.log("The foo variable is", process.env.my_foo_variable)
});

# Specification
## Test env variables
* Print environment variables

Now run gauge run --env=foo.
The output contains

The default variable is definetly not bar!!!
The foo variable is bar

Observe the line The default variable is definetly not bar!!! is printed. This means that the default env was loaded even if it not specified.

The default env should be loaded only if it is explicitly specified in --env flag or if --env flag is not used in the gauge command.

we need to be careful here, as this changes the behavior.
Currently, you can define a custom environment and only describe the properties, you like to change. The rest is loaded with the default values from the default env.
If you need to specify the default env, when loading a custom env, you would need to use all properties in your custom env or the properties are actually empty and this will break projects
As far as I see, this is also the case for the test cases in  env_test.go, only the properties that change are defined in the custom envs.

Unless I'm understanding it incorrectly, this isn't really a bug but really useful thing to load variables from default which applies to all environments.
You wouldn't really provide all variables in each env directory, that would be a lot duplication and harder to manage.
As @ajoecker pointed, this would be a breaking change for most projects.


ContainerImage deleting

ContainerImageRegistry association can destroy ContainerImage, but all images have to be archived due to reports and chargebacks.
Solves OpenShift pending specs

Cc @cben @Ladas

Cc @agrare you can merge, pls review

Thx to @Ladas to big help


Viki New Player like netflix

Checklist

[x] I'm reporting a site feature request
[x] I've verified that I'm running youtube-dl version 2020.03.24
[x] I've searched the bugtracker for similar site feature requests including closed ones

Description

I just noticed that VIKI.COM changed their player like netflix video player
They are now using manifest.mpd and not m3u8
Download direct from the link still working. But I can't download the actual video which  is from KOCOWA video.
Is there any way to download the kocowa video from viki?

Why are you closing? i just asking and suggesting for feature


local k3s cluster upgrade shows error in UI; Upgrade is successful

Setup

Rancher version: v2.6.4-rc10
Browser type & version: Chrome

Describe the bug
An error appears in UI when upgrading k8s version in local k3s cluster
To Reproduce

Fresh install of rancher v2.6.4-rc10 with local as v1.21.9+k3s1 k8s k3s cluster
Upgrade local k3s cluster (to either v1.22.7+k3s1 or v1.23.4+k3s1) from edit config
Reproduced

Result
Upon submission UI error is displayed, but upgrade is still invoked and successful
Expected Result
Upgrade is invoked and successful without errors
Screenshots

Additional context
Navigating back to Cluster Management from edit config resolved the error

Need to look at what we can do in the UI for when the backend disappears briefly during the upgrade.


New version: TextParse v1.0.2-DEV

Registering package: TextParse
Repository: https://github.com/queryverse/TextParse.jl
Created by: @shashi
Version: v1.0.2-DEV
Commit: 99e8851cd4104dad7d026fe04078e940151eb898
Git reference: master

@davidanthoff why did you add the DEV in there? Are you maintaining a different release branch?
This release will unblock the usage of a new version of DataStructures.jl, so it would be nice to get this out ASAP.
Thanks!

I use https://github.com/JuliaLang/PkgDev.jl#pkgdevtagpackage_name-versionnothing-registrynothing-release_notesnothing for tagging releases, and it adds the -DEV version to commits that are not tagged. I'll close this and I'll tag a new release here.

New PR https://github.com/JuliaRegistries/General/pull/20669.

thank you!


docs: fix simple typo, collisons -> collisions

There is a small typo in src/saux/ssearch.h.
Should read collisions rather than collisons.
Semi-automated pull request generated by
https://github.com/timgates42/meticulous/blob/master/docs/NOTE.md

Hi Tim, thank you for your pull request, nice thing the automated spell checker. I'm closing this because I'm not accepting PR's without previous non-trivial contributions to this repository. Also, I'm still preparing/considering the contribution rules ("future/contributing" branch), where I'll add the reasons for rejecting PR's.


Add mechanism of detect network disconnection and reconnect streaming

idea 1
Continue to recconect streaming until called shutdown()
fun connectHoge(handler: Handler): Shutdownable {
    val dispatcher = Dispatcher()
    dispatcher.invokeLater(Runnable {
        while (true) {
            val response = client.get("hoge")
            if (response.isSuccessful) {
                val reader = response.body().byteStream().bufferedReader()
                try {
                    while (true) {
                        val line = reader.readLine()
                        .....
                    }
                } catch (e: IOException) {
                    //waiting reconnect time
                    continue
                }
            }
        }
    })
    return Shutdownable(dispatcher)
}

I agree the idea. But I think that Streaming shouldn't have responsibility for automatic reconnection.
It would be better to provide a disconnect event and a reconnect function.
interface Retryable {
    fun retry()
}

interface Handler {
    // ...
    fun onDisconnected(retryable: Retryable)
}

It can delegate the reconnection strategy to the user.

OK. I try to write code according to this sample. Thank you!


Add Azure OpenAI support

This adds a new endpoint_type parameter to chat_server. To use with Azure:
    chatstream.chat_server("mychat", endpoint_type="azure")

Note that to use Azure, you will also probably need to set some values at the top of your app, as described in https://learn.microsoft.com/en-us/azure/cognitive-services/openai/how-to/switching-endpoints :
import openai

openai.api_type = "azure"
openai.api_key = "..."
openai.api_base = "https://example-endpoint.openai.azure.com"
openai.api_version = "2023-05-15"  # subject to change

I currently do not have access to Azure OpenAI, so I can't test this myself. It can be installed with:
pip install chatstream@git+https://github.com/wch/chatstream.git@azure

I have this working with Azure, using a deployment I created named my-gpt-35-turbo.
Example app:
import openai
from shiny import App, Inputs, Outputs, Session, ui

import chatstream

openai.api_type = "azure"
openai.api_base = "https://winstontest.openai.azure.com/"
openai.api_version = "2023-03-15-preview"

app_ui = ui.page_fixed(
    chatstream.chat_ui("mychat"),
)

def server(input: Inputs, output: Outputs, session: Session):
    chatstream.chat_server(
        "mychat",
        api_key=os.getenv("OPENAI_API_KEY"),
        model="gpt-3.5-turbo",
        azure_deployment_id="my-gpt-35-turbo",
        debug=True,
    )

app = App(app_ui, server)

Note that you must create a deployment with a custom name. In this case, it is "my-gpt-35-turbo", but you could call it anything. The screen to create it in the Azure control panel will look something like this:

Also note that this is created within an Azure resource named winstontest. The URL for that resource is https://winstontest.openai.azure.com/.


[cf-expandables] Complete usage.md

Fill it out with usage guidance akin to what was found in the older Topdoc docs.

Would https://github.com/cfpb/capital-framework/pull/284 be the best reference for updating this issue?

No, I think this branch that Jimmy started working on would be the best thing to reference: https://github.com/cfpb/capital-framework/compare/usage-update


Adding dynamic guide lines

Is your feature request related to a problem? Please describe.
When a COBOL source is open, guideline are displayed, such guideline should also be able to be defined dynamically.

Describe the solution you'd like

In the settings specific tabstops can be defined under the item coboleditor.tabstops, with an additional setting such a line should be displayed for each tabstop.

'Guide lines' are called rulers in VS Code.
I am not really sure what you mean by 'dynamic', since rulers can only be defined in two ways:

the user can setup their own rulers, as you described. (https://dev.to/brad_beggs/vs-code-vertical-rulers-for-prettier-code-3gp3)
specific extensions can provider rules, similar to what the COBOL extension does.

Any chance you can be more specific?

Closing due to inactivity.


SplFileObject Flags have no effect / empty lines cannot be ignored

I just updated from 7.0.1 to 7.1.0 and noticed that my internal tests failed due to a null value appended at the end of each CSV file that was imported. After some trial and error I figured that the SplFileObject falgs don't seem to have an effect any longer.
I'm missing the SplFileObject::SKIP_EMPTY flag in particular since it seemed to make sure that there's no null value at the end if the file terminates on a new line.
See the following test script:
<?php
use League\Csv\Reader;

include __DIR__."/vendor/autoload.php";

$path = __DIR__."/tmp.txt";
$str = "1st\n2nd\n";
$obj = new SplFileObject($path,"w+");
$obj->fwrite($str);
$obj = new SplFileObject($path,"r");
$reader = Reader::createFromFileObject($obj);

$flags = [
    "NONE" => 0,
    "READ_AHEAD" => SplFileObject::READ_AHEAD,
    "READ_AHEAD | DROP_NEW_LINE" => SplFileObject::READ_AHEAD | SplFileObject::DROP_NEW_LINE,
    "READ_AHEAD | SKIP_EMPTY" => SplFileObject::READ_AHEAD | SplFileObject::SKIP_EMPTY,
    "READ_AHEAD | DROP_NEW_LINE | SKIP_EMPTY" => SplFileObject::READ_AHEAD | SplFileObject::DROP_NEW_LINE | SplFileObject::SKIP_EMPTY,
    "DROP_NEW_LINE" => SplFileObject::DROP_NEW_LINE ,
    "DROP_NEW_LINE | SKIP_EMPTY" => SplFileObject::DROP_NEW_LINE | SplFileObject::SKIP_EMPTY,
    "SKIP_EMPTY" => SplFileObject::SKIP_EMPTY ,
];

foreach($flags as $flagName => $flag) {
    $reader->setFlags($flag);
    $lines = $reader->fetchAll();
    $vals = [];
    foreach($lines as $line){
        if($line == null){
            $vals[] = "<null>";
        }elseif(count($line)){
            $val = array_shift($line);
            if($val == null){
                $val = "<null>";
            }
            $vals[] = $val;
        }
    }
    echo count($lines). " lines [".implode(', ',$vals)."]\tfor ". $flagName."\n";
}

Output in 7.0.1
3 lines [1st, 2nd, <null>]	for NONE
3 lines [1st, 2nd, <null>]	for READ_AHEAD
3 lines [1st, 2nd, <null>]	for READ_AHEAD | DROP_NEW_LINE
2 lines [1st, 2nd]	for READ_AHEAD | SKIP_EMPTY
2 lines [1st, 2nd]	for READ_AHEAD | DROP_NEW_LINE | SKIP_EMPTY
3 lines [1st, 2nd, <null>]	for DROP_NEW_LINE
2 lines [1st, 2nd]	for DROP_NEW_LINE | SKIP_EMPTY
2 lines [1st, 2nd]	for SKIP_EMPTY

Please note the lines with SplFileObject::SKIP_EMPTY set: Those contain only 2 values (as expected).
Output in 7.1.0
3 lines [1st, 2nd, <null>]	for NONE
3 lines [1st, 2nd, <null>]	for READ_AHEAD
3 lines [1st, 2nd, <null>]	for READ_AHEAD | DROP_NEW_LINE
3 lines [1st, 2nd, <null>]	for READ_AHEAD | SKIP_EMPTY
3 lines [1st, 2nd, <null>]	for READ_AHEAD | DROP_NEW_LINE | SKIP_EMPTY
3 lines [1st, 2nd, <null>]	for DROP_NEW_LINE
3 lines [1st, 2nd, <null>]	for DROP_NEW_LINE | SKIP_EMPTY
3 lines [1st, 2nd, <null>]	for SKIP_EMPTY

Regardless of the flags, the output is always the same (3 lines, last one is null. I tried to figure out what was changed between those version but was unable to identify the cause - so this is my last desperate attempt to understand what's going on :)

Hi @paslandau,
I've just looked at your code and after some digging I've found the bug ... I'll release a patch version tomorrow.

I've push the bug fix in the master repo. So if you download the master branch you can test your code against a patch version. The 7.0.1 behaviour is the correct one. Tell me if all is good for you.
I'll try to incorporate your test into the CSV test suite.
By the way It would benefit everyone if you have other complementary tests to make a PR and add them to the package test suite

Hey @nyamsprod,
just updated to the current dev-master and it works as expected. Thanks for the fix!
For the sake of completeness, here's the dev-master output of the script in the op.
3 lines [1st, 2nd, <null>]	for NONE
3 lines [1st, 2nd, <null>]	for READ_AHEAD
3 lines [1st, 2nd, <null>]	for READ_AHEAD | DROP_NEW_LINE
2 lines [1st, 2nd]	for READ_AHEAD | SKIP_EMPTY
2 lines [1st, 2nd]	for READ_AHEAD | DROP_NEW_LINE | SKIP_EMPTY
3 lines [1st, 2nd, <null>]	for DROP_NEW_LINE
2 lines [1st, 2nd]	for DROP_NEW_LINE | SKIP_EMPTY
2 lines [1st, 2nd]	for SKIP_EMPTY

The output is identical to v7.0.1.
My tests are currently part of my utility package for file based operations (tests are over here) and somewhat deeply integrated. But I'll check if parts of them can be extracted to extend the current league/csv test suite.

version 7.1.1 is released which correct this bug

Great, thx!

@paslandau I have a question regarding League\Csv
Right now out of the box the reader uses SplFileIObject::DROP_NEW_LINE which is a bug
I want to switch this to SplFileObject::READ_AHEAD|SplFileObject::SKIP_EMPTY do you consider this as being a BC break or just a patch ?

Hey @nyamsprod
Depends.. SemVer states that the "public API must  not change within a patch/minor release". Did you specifiy the exact behaviour anywhere in the docs? If not, a patch might be okay - on the other hand, that might be a missing piece in the docs :)
I personally would bump up the major version because it might unexpectedly break existing code.
Cheers
Pascal

@paslandau
About semver, no public API is changed so if I were to strictly follow semver then this is not a BC break :+1: .  Moreover this change is really a bug fix has using SplFileIObject::DROP_NEW_LINE by default is a bug
I have publish a roadmap for v8.0 which takes into account this bug fix


Link librav1d against C dav1d and seek_stress instead of Rust versions

Closes #807 and #803.
When the test_rust meson option (on by default) is set, use a custom meson command to build librav1d.a and link dav1d and seek_stress with it instead of libdav1d.
Notes:

Using the subprojects feature of meson doesn't seem to be an option since it requires a specific directory layout.
Cross-compiling isn't supported.
The macOS build behaves differently than local macOS, possibly due to meson version differences. Didn't root cause.

Could you clarify, does this preserve the ability to build librav1d linked against Rust dav1d and seek_stress binaries?

Yes, cargo build works as usual this only affects how things are built when using meson. When you cargo build, it links against the Rust version of the dav1d CLI tools which are also fixed in this PR.

Also, could we use cargo's --profile option instead of --release?

I think that would be confusing. The idea is to match build types such that if we configure meson for a debug build, it links against libravid.a with debug symbols.


Test in IE with version 3.2.4

I have followed the steps in readme and can't get it to work on IE. Here's what I did:

<img src="my-image.jpg" class="myImg">
.myImg {object-fit: cover; font-family: 'object-fit: cover; object-position: bottom;';}
In my JS, i am importing it this way:

import objectFitImages from 'object-fit-images';
objectFitImages();

Not a bug in the polyfill, the problem is elsewhere and it needs debugging


Would like a VPN Gateway PIVPN

PiVPN Issue Template

Console output of curl install.pivpn.io | bash

Console output of pivpn add or pivpn add nopass

Console output of pivpn debug

Issue
PIVPN is incredible.  I like the simplicity and very secure.  Could you consider a fork where you make the pi a VPN gateway.  All traffic routes through Pi that is connected to VPN Host such as PureVPN or PIA or "Insert Openvpn compatible".   It could be either ethernet or combination wifi devices to ethernet to router.

While that would indeed be cool, I don't think that's in the scope of this project. I think you can use dd-wrt for this.

I want to use my raspberry Pi.  i don't think Rd-wrt would work with a pi.

https://wiki.openwrt.org/toh/raspberry_pi_foundation/raspberry_pi don't know if it will do what you want, but it's worth a shot


Do not fetch GBW file by default

ORCA stores molecular wavefunction in a binary GBW file, which can then be used to restart a calculation or used as a wavefunction guess. Currently, in OrcaCalculation we always fetch this file as part of the retrieved folder. However, this is not ideal since these files can get very big and it is not possible to delete them once the workflow finishes since they become part of the AiiDA provenance.
This PR is a subset of #72. Here I simply remove the GBW file from the default retrieve list. User can still add it when needed via inputs.metadata.options.additional_retrieve_list.
In #72, I proposed a new separate keyword that would instruct the calcjob to attach the GBW file as a SinglefileData output node, for easier manipulation. But as @mbercx explained in a detailed comment https://github.com/pzarabadip/aiida-orca/pull/72#issuecomment-1546078454, we might want to instead have a more general input node parent_calc_folder, which is in line how other AiiDA plugins handle these big files and restarts.
I am postponing that design discussion to a future PR, since I am now in a rush to publish a first working version of my App.
Hence this PR does only the minimal thing to unblock me. Since this is a breaking change, it should be released as a version 0.7.
Since we're doing a breaking change anyway, I decided to do a little cleanup and remove the settings input Dict since it is not really needed (at least for now).

the additional_retrieve_list is now supported directly in aiida-core since 1.6.0. https://github.com/aiidateam/aiida-core/issues/4423
orca command does not really support any additional command line parameters, so the cmdline keyword is useless. See the commit message for details.

Step towards #68

Codecov Report

Merging #73 (1ed347b) into develop (e267e8b) will decrease coverage by 0.06%.
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             @@
##           develop      #73      +/-   ##
===========================================
- Coverage    57.33%   57.27%   -0.06%
===========================================
  Files           15       15
  Lines         1589     1587       -2
===========================================
- Hits           911      909       -2
  Misses         678      678

Impacted Files
Coverage Δ

aiida_orca/parsers/__init__.py
88.46% <ø> (-0.43%)
:arrow_down:

aiida_orca/calculations/orca_orca.py
97.22% <100.00%> (ø)

Paging @mbercx in case you have a bit of time to give this PR your blessing. (it's very annoying the Github does not allow me to add you as a reviewer, not sure if this can be tweaked in repo settings?)

Ah, I seem to have missed this ping, apologies @danielhollas! As penance, I will review #72 in great detail once it's ready.

the additional_retrieve_list is now supported directly in aiida-core since 1.6.0. https://github.com/aiidateam/aiida-core/issues/4423

I actually wasn't aware of this one. 😅 I guess we can remove it from our settings too then!

@mbercx no worries, no penance needed. :smile:

@pzarabadip thanks for merging and publishing a new version! :clap:


Image gallery selection for DftQ images

It's live! You can specify title, images, and captions
Docs: https://docs.storysynth.org/guide/extensions.html#image-gallery


chore!: move noir out of yarn-project

Fixes #4107

ah yes almost forgot to tag a devrel: @signorecello

As discussed, lets also deprecate @aztec/noir-contracts tmrw after release!


Add current keyboard layout

Hi! First of all, awesome locker, thank for your work!
Can you add a small label with current keyboard layout? Because it is kinda annoying when you have to guess which language is currently in use, while trying to unlock your screen.

should be optional because many people (me for example haha) only use one keyboard layout.
If we add that it would probably be a config file option, what do you think @pavanjadhaw ?

@AUTplayed yeah we can do that

@AUTplayed yeah, sure it should be optional :) For me a config file option or a command line argument (like betterlockscreen --lock --show-layout) would be a pretty nice :+1:

This would be awesome

This would be great for me ;)

I have the same problem that sometimes I cannot unlock the screen because I am in wrong keyboard layout and there is no way to change it while in the lock screen.

Thanks for your request, this will be resolved soon, with the merge of the multi-monitor branch from the multilockscreen-fork :)

Merged with #249 into multi-monitor, so will be released as v4-beta soon :)

Hi! Thanks for the great locker!
After update to v4.2.0 I lose my layout indicator.
What happened?
How can I check it?
When I run
betterlockscreen --lock --show-layout
I got locker without layout indicator :(


klipperscreen_shutdown.

Hi Leadustin
why am i getting this error
all files in correct location
when i save and restart
Section 'gcode_shell_command klipperscreen_shutdown' is not a valid config section
Regards

Hi Leadustin
why am i getting this error all files in correct location when i save and restart
Section 'gcode_shell_command klipperscreen_shutdown' is not a valid config section
Regards

Have you installed the GCode Shell Comand?
via KIAUH and Option 4 Advanced. Actually, this is already installed when you install Klippain Shake And Tune

Hi
Thank you yes installed
I'll work it out later not important
Printer working great
Regards
On Mon, 9 Sep 2024 at 11:13 PM, leadustin @.***> wrote:

Hi Leadustin
why am i getting this error all files in correct location when i save and
restart
Section 'gcode_shell_command klipperscreen_shutdown' is not a valid config
section
Regards
Have you installed the GCode Shell Comand?
via KIAUH and Option 4 Advanced. Actually, this is already installed when
you install Klippain Shake And Tune
—
Reply to this email directly, view it on GitHub
https://github.com/leadustin/QIDI-up2date-english/issues/15#issuecomment-2338396220,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BCIQLITRVGM6VTKYDL3Z733ZVW3KRAVCNFSM6AAAAABN4JG64SVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGMZYGM4TMMRSGA
.
You are receiving this because you authored the thread.Message ID:
@.***>

Hi Thank you yes installed I'll work it out later not important Printer working great Regards
…
On Mon, 9 Sep 2024 at 11:13 PM, leadustin @.> wrote: Hi Leadustin why am i getting this error all files in correct location when i save and restart Section 'gcode_shell_command klipperscreen_shutdown' is not a valid config section Regards Have you installed the GCode Shell Comand? via KIAUH and Option 4 Advanced. Actually, this is already installed when you install Klippain Shake And Tune — Reply to this email directly, view it on GitHub <#15 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/BCIQLITRVGM6VTKYDL3Z733ZVW3KRAVCNFSM6AAAAABN4JG64SVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGMZYGM4TMMRSGA . You are receiving this because you authored the thread.Message ID: @.>

Have you also observed the file permissions accordingly? Changes in Mainsail or Fluidd reset the permissions. Therefore, set the permissions in a tool such as WinSCP.

Hi
Yes I used WinSPC
Ticked the boxes saved
Changed IP address
Question should the IP be the new screen using raspberry pi or the printer
I'm using the BTT K - touch as per picture
But setup stand alone screen with full klipper screen
Regards
On Tue, 10 Sep 2024 at 2:55 PM, leadustin @.***> wrote:

Hi Thank you yes installed I'll work it out later not important Printer
working great Regards
… <#m_-3567367709041428313_>
On Mon, 9 Sep 2024 at 11:13 PM, leadustin @.> wrote: Hi Leadustin why
am i getting this error all files in correct location when i save and
restart Section 'gcode_shell_command klipperscreen_shutdown' is not a valid
config section Regards Have you installed the GCode Shell Comand? via KIAUH
and Option 4 Advanced. Actually, this is already installed when you install
Klippain Shake And Tune — Reply to this email directly, view it on GitHub
<#15 (comment)
https://github.com/leadustin/QIDI-up2date-english/issues/15#issuecomment-2338396220>,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BCIQLITRVGM6VTKYDL3Z733ZVW3KRAVCNFSM6AAAAABN4JG64SVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGMZYGM4TMMRSGA
https://github.com/notifications/unsubscribe-auth/BCIQLITRVGM6VTKYDL3Z733ZVW3KRAVCNFSM6AAAAABN4JG64SVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGMZYGM4TMMRSGA
. You are receiving this because you authored the thread.Message ID: @.>
Have you also observed the file permissions accordingly? Changes in
Mainsail or Fluidd reset the permissions. Therefore, set the permissions in
a tool such as WinSCP.
—
Reply to this email directly, view it on GitHub
https://github.com/leadustin/QIDI-up2date-english/issues/15#issuecomment-2339820090,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BCIQLIQTX7C2E4HDHJY7JM3ZV2JXVAVCNFSM6AAAAABN4JG64SVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGMZZHAZDAMBZGA
.
You are receiving this because you authored the thread.Message ID:
@.***>

The IP must be that of the Raspberry. Just to be on the safe side. Have you installed Gcode Shell Command on the printer or the Raspberry?

Yes using the Raspberry pi IP address.
installed Gcode Shell command on the printer

You may have changed the syntax of the script in your edits. Deleted spaces that are required. Can you post the content of the Klipperscreen_shutdown.cfg?

[gcode_shell_command klipperscreen_shutdown]
command: ~/printer_data/config/Scripts/klipperscreen_shutdown.sh #Adjust the path of the shutdown script according to the printer. If the "Automatic Github backup" guide is installed, the path is correct.
timeout: 60
verbose: True
[gcode_macro Klipperscreen_Shutdown]
description: Remote Shutdown Klipperscreen
gcode:
RUN_SHELL_COMMAND CMD=klipperscreen_shutdown

You must indent the text from the beginning of the second line. Take another look at the original config from the instructions. There is a space at the beginning of the second line.
[gcode_shell_command klipperscreen_shutdown]
command: ~/printer_data/config/Scripts/klipperscreen_shutdown.sh
timeout: 60
verbose: True
[gcode_macro Klipperscreen_Shutdown]
description: Fährt  Klipperscreen herunter
gcode:
RUN_SHELL_COMMAND CMD=klipperscreen_shutdown

Hi
All working don’t know what I did because was trying so many options 🤪
Question does the wifi work after this update
I’m using Ethernet port
Tried the wifi and it’s not connecting/ working
Regards

I'm glad it works now. Wifi did not work directly with the printer. I can't say whether there are suitable drivers. I've never looked for them as I only use LAN. The Pi with Klipperscreen also works via WLAN, but I switched to LAN anyway.


chore: Use operator-sdk v1.22.2

What type of PR is this?
/kind cleanup
What this PR does / why we need it:
We were using v1.17.0 which was released in February.
Which issue(s) this PR fixes:
None
Does this PR have test?
Special notes for your reviewer:
Does this PR introduce a user-facing change?
NONE

Codecov Report

Merging #1044 (c3d947b) into main (e3e77aa) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main    #1044   +/-   ##
=======================================
  Coverage   50.09%   50.09%
=======================================
  Files          42       42
  Lines        4779     4779
=======================================
  Hits         2394     2394
  Misses       2299     2299
  Partials       86       86


chore: Update Container Images for Stackable Release 24.11

Part of stackabletech/issues#647.

[!IMPORTANT]
Supersedes https://github.com/stackabletech/issues/issues/626
Some of the work has already been done in that issue, so the tasklist below will link back to that.
After the meeting on 11/09 where we lock the versions, there might be minor bumps. The templates can be used from there, and links updated below.

Container Image Updates for Stackable Release 24.11

[!NOTE]
Update the product versions based on what has been decided upon in the Product Spreadsheet[^1].

[^1]: Currently this is a private spreadsheet
Replace the items in the task lists below with the subsequent tracking issue.

### Product Container Images
- [x] No additional Java versions to add (22 is the latest)
- [x] stackable-base https://github.com/stackabletech/issues/issues/626
- [x] ubi-rust-builders https://github.com/stackabletech/issues/issues/626
- [ ] https://github.com/stackabletech/docker-images/issues/845
- [ ] https://github.com/stackabletech/docker-images/issues/847
- [ ] https://github.com/stackabletech/docker-images/issues/833
- [ ] https://github.com/stackabletech/docker-images/issues/834
- [ ] https://github.com/stackabletech/docker-images/issues/835
- [x] hive https://github.com/stackabletech/issues/issues/626
- [x] kafka https://github.com/stackabletech/issues/issues/626
- [ ] https://github.com/stackabletech/docker-images/issues/836
- [x] opa https://github.com/stackabletech/issues/issues/626
- [ ] https://github.com/stackabletech/docker-images/issues/838
- [ ] https://github.com/stackabletech/docker-images/issues/837
- [x] trino https://github.com/stackabletech/issues/issues/626
- [ ] https://github.com/stackabletech/docker-images/issues/832

### Additional items which don't have a tracking issue
- [x] hello-world - ~bump to java 22/23?~
- [x] krb5 - no change
- [ ] https://github.com/stackabletech/docker-images/pull/896
- [x] testing-tools - no change
- [x] statsd_exporter (will be bumped to `1.27.0` in #837)
- [ ] https://github.com/stackabletech/docker-images/pull/879

Use https://github.com/stackabletech/docker-images/issues/831 from now on

Update: I'm checking the versions in docker-images match what was listed in the spreadsheet.

Checks are done


What is the status of this project?

Any useable features yes?

Seems deserted... found any alternatives in the meanwhile?


#1865 add support for custom JS/CSS on Job schedule form

Closes: #1865
What's Changed
Just an idea to get the ball rolling on this, can tidy up if required

Job definitions can now set a template_name prop the to use custom Job templates with custom JS & CSS ( same as object edit view etc )

Solid start @tbotnz!
Mind updating the example_plugin to demonstrate how one would use this?

@bryanculver done


API changes - expression parsing

From v. 2.0 the API will change and offer expression parsing.
import { parseExpression } from 'cherow';

// without options
parseExpression('a === b ? c : d % e');

// with
parseExpression('a === b ? c : d % e', { next: true });

This is still WIP, and I'm playing around with it locale.

I doubt it's any point to have this open now. This will be mentioned in the readme soon as the expression parsing is done. So closing. Re-open if I'm wrong.


wolfssl: 3.10.3 -> 3.11.0

Motivation for this change
Sync with latest upstream.
Things done

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

[ ] NixOS
[ ] macOS
[x] Linux (Fedora 24)

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

No package currently depends on wolfssl, so for testing I recompiled curl against wolfssl using the following shell.nix:
with import <nixpkgs> {};
let
  curlwolfssl = (curl.override {
    openssl = wolfssl;
  }).overrideDerivation (oldAttrs: {
    configureFlags =
      (builtins.filter (x: x != "--with-ca-fallback") oldAttrs.configureFlags)
      ++ ["--with-cyassl=${wolfssl}"];
  });
in
{
  my-env = stdenv.mkDerivation {
    name = "my-env";
    buildInputs = [
      curlwolfssl
    ];
  };
}

I tested the resulting curl binary and compared its output against my local curl binary provided by Fedora:
curl -2 https://nixos.org
curl https://nixos.org > 1.txt
/usr/bin/curl https://nixos.org > 2.txt
diff 1.txt 2.txt

The first line gives an error message "curl: (35) CyaSSL does not support SSLv2" confirming that CyaSSL - which is the old name for wolfssl still referred to by curl - is indeed used. Diff of output of local curl and this new curl shows identical outputs as expected.

Thanks!

Thanks for the quick merge!

you made the review easy with our example code.


MAF requires a workaround, due to a BioPerl issue for develop branch

Due to the BioPerl issue (bioperl/bioperl-live#147) I need to apply a workaround for MAF alignment imports.
Additionally, only required alignments are calculated by lastz
closes #52

Coverage decreased (-0.5%) to 99.484% when pulling 79c4e7ece583463288386f88d56ff59cc5b6e882 on hotfix/workaround-for-MAF-revcom-issue into 064d73bea471043c23629d4e0230a914ead8b23b on develop.

Coverage decreased (-0.5%) to 99.484% when pulling ab7520dd41cae11a2a29246ddef2233e28e7c19b on hotfix/workaround-for-MAF-revcom-issue into 064d73bea471043c23629d4e0230a914ead8b23b on develop.


Need integration test of complex routing + policy enforcement

Recent (and outstanding) changes to the existing integration test suites are removing a bunch of routing rule (and other) complexity from tests in an attempt to make tests more debuggable (and understandable).
However, this leaves a void in testing of complex mesh behavior. We need to add additional tests that attempt to "stress test", if you will, the mesh for more than just simple scenarios.

Removing "area/test and release" label as it is not meant for adding functional tests or test coverage.
Please assign issues to the appropriate functional area going forward.

andy while I agree the specific issues aren't necessarily test and release specific, the underlying theme across is the ease  (or not) of adding non flacky simple tests (applies to #3029 as well) - how easy those gets fixed will I guess inform on whether we are where we want from a test framework point of view.

@ldemailly yes that is part of what swat team is doing (i.e. less flaky framework), but labeling all test issues with "area/test and release" is not helpful to anyone.

I think a label to track test issues would be useful

The closest one is "kind/test failure".

looking more for kind/test coverage


Consider not converting spaces to   entities in kind=markdown

In getting the Microsoft PyLS working with Emacs, see https://vxlabs.com/2018/11/19/configuring-emacs-lsp-mode-and-microsofts-visual-studio-code-python-language-server/ -- I had to hack around the fact that PyLS converts all spaces in the Python source docstrings to   elements.
As far as I can tell, this is strictly speaking valid Markdown, although it's making life difficult for non-HTML based MarkDown renderers, like those in VIM and Emacs.
Would you consider disabling this function, or at least making it configurable?
In short, I would like contents with kind=markdown to use simple spaces instead of   entities.

I understand that you're being constrained to a certain extent by Visual Studio Code.
However, we've all seen zillions of markdown files, most of which don't contain this many   entities. Markdown renderers the world over seem to do a great job of showing these  -free files. :)
I would already be very happy if you could make an exception or configuration for only  , but I of course understand if you make the call that's not going to be reasonable for you to add.
With regard to your last question: Even when I specify preferredFormat as plaintext, I get those   entities, but only in some cases. For example, with import pandas as pd, hovering over the pandas will get me plaintext, whilst hovering over the pd will get me the same docs as markdown. Super strange.

Oh, I didn't mean that we wouldn't want to change things; I just wanted to write down my thoughts as to why I believe this happens to be the current behavior and some places to look and challenges to consider when writing a fix.
Those  s shouldn't be showing up in the plaintext output, for sure. Are you also certain that the client is sending plaintext in the client capabilities? We currently have two different places to set the documentation formatting (see #319).

I'm sending preferredFormat: plaintext as part of the initializationOptions, just like vs.code is doing (but they are sending markdown)
Doing this affects some things but not others. I can confirm that I still get markdown documentation as a response to textDocument/hover -- I looked at the raw json coming back from the LS.

Plaintext means no colors in VS Code. Since extension with Jedi traditionally provided colored code (as do other extensions), we should be doing the same.

@MikhailArkhipov The point was that even when the LS is instructed via its API to return plaintext, it still returns markdown and   entities, sometimes.
Also, I don't understand your reasoning with regards to TABs and &nbsp vs just spaces. In both cases you have to make the call how many spaces to use.  -entities only serve to act as further LS-imposed constraints on the vs.code markdown renderer (and other more compliant markdown renderers). (or are you translating TABs into another more specific html entity?)
Furthermore, from a software engineering standpoint, this really goes against separation of concerns. Ideally the MS LS should generate fully compliant, textbook markdown, whilst vs.code and other clients simply should simply render that compliant, textbook markdown.
Although markdown strictly speaking might allow for   and other html entities, that was never its intention. Remember that it started as a plaintext markup that could be equally well parsed by humans reading emails.
As I mentioned earlier: Fundamentally, most (all?) markdown renderers render  -less markdown without a hitch. I find it hard to believe that vs.code is the only one that has trouble with this.
All of that being said: I don't really care how you decide to deviate from what everybody else is doing, as long as you stay consistent in how you do it so I don't have to keep on updating my patches to Emacs. ;P

LS sends   in markdown triple-tick blocks with html language specified so they are part of HTML fragment. Works same way triples work here at GH. Typically markdown looks like block of triple-tick-python followed by triple-tick-html. See https://github.com/Microsoft/vscode/issues/49018 for an example.

VS Code always requests markdown, never plaintext, so I don't think fixing plaintext output to actually be plaintext will affect VS Code (not that plaintext can't contain   if it means it). I'm not sure how it would affect VS, though, if at all.
I'm a bit confused what triple backtick html blocks have to do with this, though. That's for code formatting HTML (like triple backtick then python would be). Example:
<body>
<p>Hello   world!
</body>

Is:
```html
<body>
<p>Hello   world!
</body>
```

That doesn't mean that HTML is rendered, but that it's highlighted as HTML. When I was looking into #328, I didn't see us outputting triple backtick html blocks at any point. Searching the repo for "html" doesn't net anything either. When we render
If we want that bottom docstring to be spaced as plaintext with the original characters, I'd think that a plain triple backtick block would work, or explicitly stating plaintext. IIRC we do the former.

All backtick blocks should be able to do that, like:
        I'm prefaced by 8 spaces!

Which is:
```
        I'm prefaced by 8 spaces!
```

With #596, the docstring handling has been completely rewritten for the new language server. There is no longer any manual spacing of elements, instead converting from the docstring to something markdown will be able to format. I don't think we'll be inserting any more HTML-specific whitespace (unless it can't be avoided, which I can't see happening). If plaintext is specified, all docstrings returned will come through unmodified, other than removing consistent indents.
I'll file another bug for handling LSP capability-based MarkupKind support, but that's a further improvement.


Support 2D bit arrays in structures. Optimise array indexing.

Support bit slicing of array elements in structs or unions e.g.
s.a[i][p:q]

Array indexing when the array stride is a power of two is now done with a shift rather than a multiply.

This PR has two changes. Are they related? If not, can they be split into two separate commits?

The changes were connected.  The main commit was to support 2D array indexing in structures which I had left as a TODO from my previous PR.  I just optimised a special case of this specific  indexing when the array stride was a power of 2 and a shift seemed to be better than a multiply.
I have no benchmarks to justify this guess, and it is far from performance critical code, so I have removed the shift substitution.

If Yosys optimization passes would not take care of a multiply-by-power-of-2 I would consider it a bug.


[Aio] Correct type annotation of grpc.aio.ServicerContext.abort

Related https://github.com/grpc/grpc/issues/24942
grpc.aio.ServicerContext.abort accepts 3 arguments, code, details, trailing metadata. And only the first argument code is positional without default value. This is how grpc.aio.ServicerContext is implemented in Cython and tested in unit tests. But I overlooked this issue when I created the abstract class.

Known Failure: https://github.com/grpc/grpc/pull/24947


feat(define): prevent assignment

Description
Fixes #5269
Fixes #4271
Fixes #5278
Fixes #4887

Prevent define replacement if has trailing assignments
Additional context

Should this be a new config option? It could be breaking, but in rare edge cases.

What is the purpose of this pull request?

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

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

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

Im not sure we should do this. The users in the linked issues should use a constant name tgat doesnt conflict with their code. With this change, the local variable isnt replaced but their use later in the scope is replaced, so potentially there could be inconsistencies because of this. IMO if wexare not going to do proper AST replacement, we should keep define simple and continue to ask users to choose non-conflicting names

Wouldn't the inconsistency be intentional? I believe this is how @rollup/plugin-replace works too according to its docs example, and preventAssignment is true by default. FWIW the regex used is the same as well. I think it's still a fair change to guard users from the odd error message, and in most case the user would know the risk of using define and its inconsistency.

Thanks for the pointers @bluwy, looks like Rollup didn't change it right away to prevent a breaking change then. I would vote to add this without an option as you did in this PR to align with future versions of the official rollup define plugin (I don't think we should have the complexity of options on our side). I'll bring this PR to the next team meeting so we can discuss it there

We talked in the last team meeting about this, let's merge it in the 2.8 beta so the ecosystem has a bit of time to test it

Perhaps the import/export variables and file names should also be ignored?

@btea would you create a new feature request or PR to discuss further changes?

There is one more question, I don’t know if my understanding is correct.
According to the current modification, the variable declaration will be prevented from being replaced, but the use of the variable does not prevent the replacement, which will cause the packaged code to get the wrong value of the variable.

@btea would you create a new feature request or PR to discuss further changes?

Maybe I should create a new feature to discuss it?

There is one more question, I don’t know if my understanding is correct.
According to the current modification, the variable declaration will be prevented from being replaced, but the use of the variable does not prevent the replacement, which will cause the packaged code to get the wrong value of the variable.

If you mean for code like this:
const __PROD__ = false

console.log(__PROD__) // if we replace __PROD__ as true, this will log true, instead of the from the variable -- false

I think this behaviour would be intentional from the user point of view, as from the linked issues.

Perhaps the import/export variables and file names should also be ignored?

This sounds like a fair thing to support too, I think we should create a new feature issue for it.

I think this behaviour would be intentional from the user point of view, as from the linked issues.

Yeah, you are right.

This sounds like a fair thing to support too, I think we should create a new feature issue for it.

ok.


Double JOIN with order

I guess the example below will speak on it's own :
named_scope :select_with_authors,
:joins => "LEFT OUTER JOIN authors ON authors.id = posts.author_id"

Post.select_with_authors.descend_by_author_name
ActiveRecord::StatementInvalid: Mysql::Error: Not unique table/alias: 'authors': SELECT posts.* FROM posts INNER JOIN authors ON authors.id = posts.author_id LEFT OUTER JOIN authors ON authors.id = posts.author_id  ORDER BY authors.name DESC

Please let me know if you like further details.

Just ran into this today with the INNER JOIN for :include'd has_one association and LEFT OUTER JOIN for search conditions on the same has_one association.  MySQL throws a "Not unique table/alias" error.


What are the minimum version of compilers for coda?

Documentation request:
This impacts changes that I'd like send to coda.  Does coda really need to support compilers older than 8 years (e.g. if it's Visual Studio >= 2012 or newer, gcc >= ?, clang >= ?, mingw >= ?) that changes what the code has to do.
e.g. for my local build (with bazel and clang/llvm), I have to remove a swath of coda.h and replace it with just #include <stdint.h> (without a surrounding #ifdef HAVE_STDINT_H).  I'm happy to share that change if it doesn't break the requirements of coda.  e.g. In my local code, I can assume that stdint.h works and I'm wondering your assumptions.
And does the code have to be C89 or can C99 features be used?  I presume you don't support and C11 features.

https://en.wikipedia.org/wiki/List_of_compilers#C_compilers
https://en.wikipedia.org/wiki/Microsoft_Visual_Studio#History
http://mingw-w64.org/doku.php/versions
https://gcc.gnu.org/releases.html
https://en.wikipedia.org/wiki/Clang

Closing this issue as there is no immediate action to take.


Implement new UVH5 data format

This PR adds a new "UVH5" data format into pyuvdata. The format is saved as an HDF5 data type, and is a straightforward serialization of a UVData object. Tests show that the UVH5 read and write methods are able to round-trip MIRIAD and UVFITS files, including current HERA data. Future plans include adding support for partially reading files and using the built-in compression filters (in particular, the bitshuffle filter for visibility data), but this PR adds the basic functionality. Any feedback or comments are welcome!

Looking good @plaplant. The travis build got wedged so I restarted it. It looks like you haven't added anything to the tutorial or readme. Are you planning to on this PR or do you not want to advertise the functionality yet?

I think I’d rather wait ~a week to update the docs and tutorial. We might have to update the installation instructions to make bitshuffle available to the users, but I’d like to have this core functionality in place so Jack and I can iterate.

Ok, then once this is merged, can you make an issue to track that? Also any todos in the code should be issues as well.

Sure, happy to make some issues.


What if the version in package.json has a prefix like v1.6.1?

Hi guys,
What if the version in package.json has a prefix like v1.6.1?
When I try the gulp-bump it says that this is invalid semver .. but actually it is so important to support prefixes?
Any workaround?
Thanks.

any clue?

 Please be respectful as I (or the community) will answer in due time.
I have added support for modifying versions with a prefix, available in published version 2.5.0

@stevelacy Respectful!!!!!! What I have said that makes you so angry!


fix: respondNoContent() returns Kint script in development mode

Description
Fixes #7346 about API\ResponseTrait::respondNoContent() returns Kint script in development mode.
Checklist:

[x] Securely signed commits
[ ] Component(s) with PHPDoc blocks, only if necessary or adds value
[ ] Unit testing, with >80% coverage
[ ] User guide updated
[x] Conforms to style guide

Thank you.
You need GPG sign all your commits.
See https://github.com/codeigniter4/CodeIgniter4/blob/develop/contributing/pull_request.md#signing
Can you write test code to prove the bug fix?
https://github.com/codeigniter4/CodeIgniter4/blob/develop/contributing/pull_request.md#unit-testing

We do not use git merge. Please use git rebase to update your PR branch.
See https://github.com/codeigniter4/CodeIgniter4/blob/develop/contributing/workflow.md#updating-your-branch
And please sign all your commits.
See https://github.com/codeigniter4/CodeIgniter4/blob/develop/contributing/workflow.md#gpg-signing-old-commits

is it correct now?

Good! Thank you.

I add content type checking on unit test, to make sure it returns non-HTML format,
public function testNoContent()
{
    $controller = $this->makeController();

    $this->invoke($controller, 'respondNoContent', ['']);

    $this->assertStringStartsWith('application/json', $this->response->getHeaderLine('Content-Type'));
    $this->assertSame('No Content', $this->response->getReason());
    $this->assertSame(204, $this->response->getStatusCode());
}

because in Toolbar.php it checks the type
// Non-HTML formats should not include the debugbar
// then we send headers saying where to find the debug data
 // for this response
 if ($request->isAJAX() || strpos($format, 'html') === false) {
    $response->setHeader('Debugbar-Time', "{$time}")
         ->setHeader('Debugbar-Link', site_url("?debugbar_time={$time}"));

    return;
}

Thank you!


ISSUE-60 support password flow

#60
support all scenarios that azure knows: still the user is responsible to act with care :lock:

This is mainly an enabler to bring @ivy-lmu 's team-notification into the market. Without a technical auth-flow, we will not be able to achieve it.

A general enablement of this flow is ok for me. But I need to dive deeper in this topic to see if the use case is valid that the Resource Owner Password Flow is really needed. Maybe the Intention of Microsoft is to use other APIs to achieve the same things (e.g. A Bot?)
P.S. without a real integration test we don't know if this is working. but maybe you add one, where you use this feature?!

yes, sure Microsoft will preferably works with a bot in such scenarios. And this will indeed be a good long term solution. What I can say for today is, that there are many APIs, that only work with a personal user. /chat:oneToOne is one of them. And yes, I plan to apply this flow there: when implementing Teams notifications.
Another interesting point that also should force us to allow this flow, is that azure-ad can be used as identity-provider, despite you may finally use the authenticated session to work on your custom app. I've had this scenario on Monday at the customer site. See also the related #62

Thanks for this addition 👍


JNI appender

Initial version of the JNI appender.
For a quick test, run:
GEN=ninja BUILD_JDBC=1 make && java -cp build/release/tools/jdbc/duckdb_jdbc.jar org.duckdb.test.TestDuckDBJDBC

Known limitations:

No datetime support.

Looks very good, the CodeFactor complaints are false positives. Thanks!

Looks very good, the CodeFactor complaints are false positives. Thanks!


Added download profolio link

hi sir ,
updated the project page. now we can scroll and  see all the images of the project.
new updates like download pdf and project page are merged.
thank you,
apsar

From: Suvel Rathneswar @.>
Sent: Sunday, March 24, 2024 5:35 AM
To: Apsar7004/Blossom_Flower_landscaping @.>
Cc: Subscribed @.***>
Subject: Re: [Apsar7004/Blossom_Flower_landscaping] Added download profolio link (PR #5)
Closed #5https://github.com/Apsar7004/Blossom_Flower_landscaping/pull/5.
—
Reply to this email directly, view it on GitHubhttps://github.com/Apsar7004/Blossom_Flower_landscaping/pull/5#event-12225828846, or unsubscribehttps://github.com/notifications/unsubscribe-auth/A3U5H4OTZUCQKSVUOKMNYHTYZZQZPAVCNFSM6AAAAABFFL5CS6VHI2DSMVQWIX3LMV45UABCJFZXG5LFIV3GK3TUJZXXI2LGNFRWC5DJN5XDWMJSGIZDKOBSHA4DINQ.
You are receiving this because you are subscribed to this thread.Message ID: @.***>


Fix #39

NOTE: The code for .map_glyph is duplicated, which isn't great. I can't create a ReadArray<T> from a Vec<T>, so I can only either duplicate the code or use a macro.

I'm not sure I really want is_mark to be public again. Ideally it would remain an implementation detail. What was your use case for making it public? The pattern I'm using in the code I'm working on at the moment is along these lines:
        let position = match info.mark_placement {
            MarkPlacement::None => {
                match info.placement {
                    Placement::None => (x, y),
                    Placement::Distance(dx, dy) => (x + dx, y + dy),
                    Placement::Anchor(_, _) => (x, y), // FIXME: position anchors for cursive
                };
            }
            MarkPlacement::MarkAnchor(base_index, x, y) => {
                unimplemented!()
            }
            MarkPlacement::MarkOverprint(base_index) => {
                positions[base_index];
            }
        };

Did you have a particular need for making the various types Copy? Typically I reserve Copy for smaller types (that fit in a register or two) to avoid surprise memcpys.
Here are the sizes (in bytes) of some of the types: HeadTable = 56, TableRecord = 16 HheaTable = 22, MaxpTable = 30, MaxpVersion1SubTable = 26

I'm okay with not implementing Copy, but I need them to be at least Clone, because my implementation doesn't store the entire font, it only stores a few tables. So I can either parse the tables again or I could simply clone them from the font provider.
My usecase for is_mark is that I want to switch the positioning between MarkPlacement and Placement. I can make it a function that returns true on MarkPlacement::None, it's just that I wanted my code to build yesterday evening

My usecase for is_mark is that I want to switch the positioning between MarkPlacement and Placement.

So after going through the process of adding positioning info to the allsorts-tools shape command I don't think it's quite this simple: if a mark is attached to a base glyph with Placement::Distance then the mark also needs to be adjusted by that distance:
https://github.com/yeslogic/allsorts-tools/pull/14/files#diff-f686acba59f5e44ba9e3177784a8c42d45806e266474c5540305fd33dad01f40R64-R70

I merged a number of these changes manually. I had to drop the derived Debug and PartialEq impls on the cmap types as these don't work in Rust 1.38.0, which is the minimum version we support. Are these required for your use case?

No, not required, I just added it because it does make things easier for testing. Clone is the most important trait.
Thanks for mergin.


Add [Pocket option ] #exchange GB5ALF…VCFE7G

Address: GB5ALFSUU7TBUC3CNBXIFDAWWVY4OVG7I3PUJHULMW4YTNCCKDVCFE7G
Tags: exchange
Title: **Pocket option **
Requested by @photowilll

Hello @photowill,
Please feel free to reopen this issue with any new information you have to provide.
Looking forward to the future with you,
Rob


Daisy improved corrections to the effective potential?

It seems that the generic_potential.py module does not include the daisy improved corrections. Are there any way to improve this? Or this package actually have this?

You should be able to include thermal mass corrections in the boson_massSq method which amounts to the same thing (I think, it's been a long while since I worked on this). See e.g. https://arxiv.org/pdf/1204.5464.pdf eqs 12-14. In general, you can change the potential function to be whatever you wish. You can completely replace Vtot(X, T) to include different corrections or exotic functional forms so long as you keep the same call signature and the resulting function is continuously differentiable.


docs: add some guidance to configure RTL mode

Documentation Request
There is no information about how to configure RTL mode in the docs. However, by searching the issues I have found out some information about RTL support. (e.g. #386 and a demo site)

FluentDesignTheme has a Direction property which when is set to LocalizationDirection.RightToLeft has no impact.
Based on this comment, in the aforementioned issue, I have done this:

Added a simple js file containing a simple dir configuration:

export function switchDirection(dir) {
    document.dir = dir;
}

Added these initializations to OnAfterRenderAsync of the main layout of the app:

    protected override async Task OnAfterRenderAsync(bool firstRender)
    {
        if(firstRender)
        {
            _jsModule = await JSRuntime.InvokeAsync<IJSObjectReference>("import",
                 "./_content/[corresponding assembly]/Layouts/MainLayout.razor.js");

            var dir = LocalizationDirection.rtl;
            GlobalState.SetDirection(dir);

            await Direction.SetValueFor(container, dir.ToAttributeValue());
            await _jsModule!.InvokeVoidAsync("switchDirection", dir.ToString());

            StateHasChanged();
        }
    }

But the result is a rtl layout with the exactly same issue reported in #386. So there should be a better documentation on this feature which I know implementing it have taken lots of time and effort.
v4.7.2
net8.0

That is a very old demo site you are looking at. Hence, why we say on the first line:

The site at this URL is no longer being updated. Visit https://www.fluentui-blazor.net/ for an up-to-date version.

We should indeed update the docs on that but too much to do in too little time...until that time, you can use the source of the demo site here in the repo (in the examples folder) as an example implementation.

That is a very old demo site you are looking at.

I knew it was a very old demo site. I had to refer to it because the main documentation has no mention of this mode (I should have said "an old demo site".)
I have no problem with contributing to this part of documentation, but I could not find the correct way to configure it.
I inferred from marking this issue as "completed" that you are not interested in my contribution.

I inferred from marking this issue as "completed" that you are not interested in my contribution.

We absolutely are always interested in contributions!
But it was not clear to me that you want to do that from the first post. I closed it because it is on our ToDo list and no further action was needed on this for now. I'll reopen and label it as 'community contribution'. Looking forward to it!

@mobinseven are you still planning to contribute this?
If not, that is totally ok too, but then I'll close the issue to keep expectations straight.

Of course. I have managed to reach a minimal configuration to correctly setup RTL mode at startup.
@* MainLayout.razor *@

@using Microsoft.FluentUI.AspNetCore.Components.DesignTokens
@inject Direction DirectionToken
@inherits LayoutComponentBase
...
@Body
...
<FluentDesignTheme Direction="@Direction" />
@code {
    LocalizationDirection Direction { get; set; }
    protected override async Task OnAfterRenderAsync(bool f)
    {
        await base.OnAfterRenderAsync(f);
        if(!f)
            return;
        await DirectionToken.WithDefault("rtl");
        Direction = LocalizationDirection.RightToLeft;
        StateHasChanged();
    }
}

Compontents like FluentTreeView have issues when just direction is switched (check FluentTreeView on demo site when "Right to Left" setting is on.)
Is it ok to add this snippet to docs?

Yes, please. Let's add a RTL section in the https://www.fluentui-blazor.net/CodeSetup page


Server switch malfunction on Windows 11

Ever since I upgraded to Windows 11, the server switches stop working & clicking on the server icons doesn't do anything...

I've tried

unloading plugins
reinstalling multiple times...
Seems to be a Win11 compatibility issue...
Consequently, I've had to disable BetterDC...

To replicate :

Just wait for 2-3 mins after Discord's launch in Win11
Restarting Dc fixes the issue for like 30 seconds after which it goes back to the inactive state

This is the wrong repo, and even then this isn't a BD issue. This is an issue with DevilBro's plugins.

@christiankek My bad!
Thanks for informing ... Lemme see, if not will open in actual repo