Clarification on RWKV not supporting "Training Parallelization" designation on ARXIV paper ?

This is pertaining to the paper published here: https://arxiv.org/abs/2307.08621
Unfortunately as I could not find contact information directly. We would like to ask for clarification on what does it mean that RWKV does not "training parallization"?

Page 6

This has cause quite a number of confusion within the community, as RWKV was designed to be able to take advantage of multiple GPUs and be trained in parallel. Across multiple data samples concurrently. Additionally, due to the cascading nature of RWKV state with tokens, it is able to rapidly scale up in a cascading pattern in training in parallel across multiple tokens ( see digram here: https://wiki.rwkv.com/advance/architecture.html#how-does-rwkv-differ-from-classic-rnn )
If this was a mistake in the paper, the RWKV community would like it to be fixed, otherwise we would like a footnote / clarification on what did it mean when we do not "training parallization"
Hoping that someone from the authoring team can be tagged.

It's not complicated. In RWKV-4, the sequence modeling is implemented by recurrent patterns in CUDA code which can't utilize full SM Kernel and Tensor Cores. Even if the community claims it has potential on parallel-scan, there is no implementation when we publish our paper.
As for the cascading nature, every decoder has this property. But it is a question of whether it is friendly to hardware. If your community accelerates the training by that, it's better to show the code and evaluation.
Nevertheless, even though RWKV doesn't utilize parallel thoroughly, it isn't a question to you.  "Training Parallelization" is a fact, but it doesn't have an influence on your real training process, which is still not slow I believe.

Thanks for the clarification - are you referring to the RWKV-v4 or the RWKV-v4neo code in blinks main repo here? https://github.com/BlinkDL/RWKV-LM
The former (v4) is a reference implementation, the later (v4neo) is the recommended trainer, that has been better optimised to better make use of the GPU. There is no architecture changes between the two (they use the same models).
The v4neo trainer has been available since sept 2022
Currently, with the right configuration of batch size, and sufficient training context size (eg. 4k/8k), it is possible to configure RWKV to keep the GPU under sustained loads - the following is an example from a recent training run i have done across 8 GPUs.

( the dips in between are checkpoints. The usage graph should be similar to other transformer trainers )
Admittingly - getting the configuration right to fully make use of all the available GPU, is really "not obvious" in the default RWKV trainer. And thats our fault in the RWKV community for not having clearer and better documentation for this process.
We can also backdate the trainer code to sometime before the paper, and assist with the required configurations if needed.

The reason, I am seeking clarification, is since the paper was preprinted on 17 July, we been getting questions on the RWKV discord every few days by a reader of the paper. Asking what does it mean when RWKV does not support "Training Parallelization"

If the definition, is defined as the ability to train across multiple GPUs and make use of all them (similar to how huggingface phrases it, or how most of the community understands it, or in your reply "friendly to hardware"), we would like to assist in getting the configuration right for your replication attempts if needed. Be it for single node or multiple nodes.
If the definition, is something else which we might fall short on (as its not obvious to me currently). A footnote would be great in describing that definition.

Example:

While RWKV does not support <___ Training Parallelization definition ___> , it is still able to heavily utilise multiple gpu's in parallel across multiple nodes with the right configuration.

PS: I am not the author of RWKV nor the RWKV paper, i am simply a volunteer on the discord, who is getting abit tired of explaining that "yes we support multiple GPU training" + "I do not know what the author of the paper mean, that we do not support Training Parallelization, please ask them instead"

We don't mention anything about multiple gpu's parallelizations...so the misunderstanding on the discord is not what we hope for.
The point we emphasize is the parallelization along the sequence, which we write in the paper " RWKV replaces AFT’s position embeddings with exponential decay and runs the models recurrently for training and inference". When the sequence is longer, the GPU utilization may be bounded.
I don't doubt the throughput of RWKV. I also reproduce it in our pipeline. In the paper, we aim to talk more academically, where the parallelization should be both in sequence and in channel. For that, I read the RWKV-4-neo code, it seems still recurrent in sequence dimension, which is not sequence parallelization and doesn't use more advanced hardware acceleration like Tensor Core.
I personally recommend considering our paper's claim from a theoretical perspective, where the current model runs well but may be bounded in the future. As for other misunderstandings, I believe it's not a big problem.

Thanks for clarifying,
That it is not about throughput.
But it is regarding the need to compute the previous token state, to compute the next token state.
In this regard, there is no dispute on my side on this. As you are right in your understanding, and that this is a fundamental design characteristic of RWKV.
A few of the original RWKV paper authors, actually had pointed out that this could have been the interperation. But were not certain. Due to lack of a definition of the term in the v1 paper.
Part of the confusion by the public, might be simple due to the fact that a google search on "Training Parallelization" leads to articles like the following

https://betterprogramming.pub/parallel-and-distributed-training-in-deep-learning-for-beginners-part-1-introduction-612a4534a117
https://arxiv.org/pdf/1907.13257.pdf
https://www.researchgate.net/figure/Different-Training-Parallelization-Strategies_fig2_334821612

Ideally, there should be a definition / clarification to avoid further confusion.
But the above explanation is sufficient for me. As I can now form a proper answer accordingly.
I hope you understand, that I did not want to be mis-explaining what the term meant, and misrepresenting your claims.
Either way, perhaps we should view the growing interest by very random folks (and their misunderstanding of the paper), as a good sign that there is growing interest to a new alternative to transformers. Be it "Retentive Network" or "RWKV"
After all we need more, not less alternatives =)

We don't mention anything about multiple gpu's parallelizations...so the misunderstanding on the discord is not what we hope for.

I now understand your intention, but was deeply confused by your paper because I've never seen anyone use language like that before. Given that it's already causing widespread confusion and misunderstanding, do you think you could change the language to something else?

We don't mention anything about multiple gpu's parallelizations...so the misunderstanding on the discord is not what we hope for.

I now understand your intention, but was deeply confused by your paper because I've never seen anyone use language like that before. Given that it's already causing widespread confusion and misunderstanding, do you think you could change the language to something else?

Don't worry, you understand that now, and we will give a more detailed explanation in the next version of our paper.

We don't mention anything about multiple gpu's parallelizations...so the misunderstanding on the discord is not what we hope for.

I now understand your intention, but was deeply confused by your paper because I've never seen anyone use language like that before. Given that it's already causing widespread confusion and misunderstanding, do you think you could change the language to something else?

Don't worry, you understand that now, and we will give a more detailed explanation in the next version of our paper.

I was wondering when this "next version of our paper" was expected to come out. The most recent version was released the same day as your comment and seems to still have the misleading language. On a practical front, this use of language has continued to cause confusion and we have even seen people claim exactly what you are professing not to claim and cite your paper as the source.


commands

hello, my installation was all ok without errors, but i can run the command /run for other commands i get a error "no command is running" i have tryied the commands cancel ende kill , and restart server but no lucky... any ideas??
thanks in advanced

that's weird, could you post a screenshot?

The shell bot has been moved to its own repository
https://github.com/botgram/shell-bot
The blog post has also been updated since installation is much simpler now.
If you're still having problems with it, open an issue or comment on the blog.


backup: Use buffered channels to collect backup status

What does this PR change? What problem does it solve?
While working on #3955 I've noticed that the block profile of go reported that the two FileSaver were in total blocked for 0.8 seconds while reporting progress. For comparison, the whole backup run took less than 4 seconds. Thus, add a small buffer to the channels used to report status information to prevent blocking. This also has the benefit of reducing the stress on the golang runtime scheduler.
As the status information is purely informational, it doesn't matter that the status reporting shutdown is somewhat racy and could miss a few final updates.
Click on the image below, to see the full block profile (look for FileSaver calls).

Was the change previously discussed in an issue or on the forum?
No.
Checklist

[x] I have read the contribution guidelines.
[x] I have enabled maintainer edits.
[ ] I have added tests for all code changes.
[ ] I have added documentation for relevant changes (in the manual).
[ ] There's a new file in changelog/unreleased/ that describes the changes for our users (see template).
[x] I have run gofmt on the code in all commits.
[x] All commit messages are formatted in the same style as the other commits in the repo.
[x] I'm done! This pull request is ready for review.

LGTM as a stopgap, but I do think this code needs an overhaul. A mutex on the Progress struct would make it much easier to reason about and probably equally fast. E.g., unless I'm mistaken, there's a race condition between
https://github.com/restic/restic/blob/1a6160d15233fb2dd8f4655d9f46beb39378fdf5/internal/ui/backup/progress.go#L132
and
https://github.com/restic/restic/blob/1a6160d15233fb2dd8f4655d9f46beb39378fdf5/internal/ui/backup/progress.go#L297

In fact, one of the reasons why the progress reporter spends so much time in scheduling is probably the way its main loop is structured. It has a time.Ticker, but instead of using that to determine when to print an update, it checks time.Since(lastUpdate) every time a message comes in.

I have a cleaned up version in a local branch. It needs some finishing touches.

unless I'm mistaken, there's a race condition between

It would be a race condition, if ReportTotal were used by something else than the Scanner. As the scanner tells Progress to close the channel once it's finished and afterwards never calls ReportTotal again, this should be fine.

The sending thread will block, unless Progress.Run is waiting to receive data. Thus, if Run were fast enough this could work without blocking the goroutine sending the backup progress. But that is apparently not the case. By buffering the progress reporting channels, it should be possible to avoid this most of the time.

but I do think this code needs an overhaul.

I agree, but I've decided to not change this myself to not distract me from finishing some other PRs ;-)

I have a cleaned up version in a local branch. It needs some finishing touches.

Would that replace this PR? Or should I merge this one first?

Would that replace this PR? Or should I merge this one first?

I suggest you merge this one first.

LGTM.


Add some functions to PCF8523 (Alarm)

Thank you for creating a pull request to contribute to Adafruit's GitHub code!
Before you open the request please review the following guidelines and tips to
help it be more easily integrated:

Describe the scope of your change--i.e. what the change does and what parts
of the code were modified.  This will help us understand any risks of integrating
the code.
This PR just add few functions to PCF8523, and add Alarm function

Describe any known limitations with your change.  For example if the change
doesn't apply to a supported platform of the library please mention it.
It should work straight everywhere

Please run any tests or examples that can exercise your modified code.  We
strive to not break users of the code and running tests/examples helps with this
process.

It is straight forward
Thank you again for contributing!  We will try to test and integrate the change
as soon as we can, but be aware we have many GitHub repositories to manage and
can't immediately respond to every request.  There is no need to bump or check in
on a pull request (it will clutter the discussion of the request).
Also don't be worried if the request is closed or not integrated--sometimes the
priorities of Adafruit's GitHub code (education, ease of use) might not match the
priorities of the pull request.  Don't fret, the open source community thrives on
forks and GitHub makes it easy to keep your changes in a forked repo.
After reviewing the guidelines above you can delete this text from the pull request.
Let me know if you need any modification
Best regards,
Aurelien BOUIN

Thank you @edgar-bonet for your time
I have updated the PR following your advice

Great!
LGTM, but note that clang-format is being picky about your formatting, hence the CI failure. You may want to
clang-format -i src/*

or, alternatively, patch the output of the clang-format test failure to git apply.

Thanks again @edgar-bonet
clang-format applied


Add nonprofit link to navbar

Feature Request
https://techqueria.org/resources/nonprofits should be added to dropdown navbar
Any additional context?
See #323 for more context

I'd love to assist on this and a few other issues you have posted, if no first timers claim them.
Thanks!

@teisenhower Definitely, feel free to tackle it and let me know if you have any questions!

Great! Thank you! I will make a PR soon.

@teisenhower Turns out it was added in a previous PR and I just didn't notice it - good to go now!


[PORT] fix for multi user scenario

Port this change from botbuilder-dotnet/master branch:
https://github.com/microsoft/botbuilder-dotnet/pull/3118

They are not thread-safe when multiple users call these functions.
Also we must use services.AddSingleton<IBot, TestBot>() for the state of RandomSelector._rand

Closing.  No Adaptive support in Python yet.


RDM-4426 Complex Element only on list screen - Import Validation

JIRA link (if applicable)
https://tools.hmcts.net/jira/browse/RDM-4426
Change description
Validation for ListElementCode in:

SearchInputFields
SearchResultFields
WorkBasketInputFields
WorkBasketResultFields

Does this PR introduce a breaking change? (check one with "x")
[ ] Yes
[X ] No

This PR will have some code conflicts with this PR. So watch out for changes if it's merged before.

Codecov Report

Merging #379 into master will increase coverage by 0.16%.
The diff coverage is 97.53%.

@@            Coverage Diff             @@
##           master     #379      +/-   ##
==========================================
+ Coverage    89.6%   89.77%   +0.16%
==========================================
  Files         332      335       +3
  Lines        6226     6289      +63
  Branches      415      412       -3
==========================================
+ Hits         5579     5646      +67
+ Misses        564      559       -5
- Partials       83       84       +1

Impacted Files
Coverage Δ

...eld/EventCaseFieldDisplayContextValidatorImpl.java
88.88% <ø> (ø)
:arrow_up:

...tore/domain/service/EntityToResponseDTOMapper.java
100% <ø> (ø)
:arrow_up:

...nition/store/domain/service/LayoutServiceImpl.java
100% <100%> (ø)
:arrow_up:

...lidation/genericlayout/GenericLayoutValidator.java
100% <100%> (ø)

...enericlayout/GenericLayoutEntityValidatorImpl.java
100% <100%> (+21.42%)
:arrow_up:

...ield/EventCaseFieldShowConditionValidatorImpl.java
100% <100%> (ø)
:arrow_up:

...nition/store/excel/parser/GenericLayoutParser.java
92.85% <100%> (+0.12%)
:arrow_up:

...n/displaygroup/PageShowConditionValidatorImpl.java
100% <100%> (ø)
:arrow_up:

...ition/store/repository/model/SearchInputField.java
100% <100%> (ø)
:arrow_up:

...finition/store/repository/CaseFieldEntityUtil.java
92.3% <100%> (+0.3%)
:arrow_up:

... and 13 more

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 8baba39...ed7904c. Read the comment docs.


Top-level "Available options" move to the bottom of the help text when using group_help

Consider the command chaining example from the docs:
use bpaf::{construct, long, positional, short, OptionParser, Parser};

#[derive(Debug, Clone)]
pub struct Options {
    premium: bool,
    commands: Vec<Cmd>,
}

#[derive(Debug, Clone)]
// shape of the variants doesn't really matter, let's use all of them :)
enum Cmd {
    Eat(String),
    Drink { coffee: bool },
    Sleep { time: usize },
}

fn cmd() -> impl Parser<Cmd> {
    let eat = positional::<String>("FOOD")
        .to_options()
        .descr("Performs eating action")
        .command("eat")
        .adjacent()
        .map(Cmd::Eat);

    let coffee = long("coffee")
        .help("Are you going to drink coffee?")
        .switch();
    let drink = construct!(Cmd::Drink { coffee })
        .to_options()
        .descr("Performs drinking action")
        .command("drink")
        .adjacent();

    let time = long("time").argument::<usize>("HOURS");
    let sleep = construct!(Cmd::Sleep { time })
        .to_options()
        .descr("Performs taking a nap action")
        .command("sleep")
        .adjacent();

    construct!([eat, drink, sleep])
}

pub fn options() -> OptionParser<Options> {
    let premium = short('p')
        .long("premium")
        .help("Opt in for premium serivces")
        .switch();
    let commands = cmd().many();
    construct!(Options { premium, commands }).to_options()
}

fn main() {
    println!("{:?}", options().run())
}

It yields the following help string:
Usage: bug [-p] [COMMAND ...]...

Available options:
    -p, --premium  Opt in for premium serivces
    -h, --help     Prints help information

Available commands:
    eat            Performs eating action
    drink          Performs drinking action
    sleep          Performs taking a nap action

Now, add a group help to the command parser:
-    construct!([eat, drink, sleep])
+    construct!([eat, drink, sleep]).group_help("My Commands")

The help string is now changed to the following:
Usage: bug [-p] [COMMAND ...]...

My Commands
    eat            Performs eating action
    drink          Performs drinking action
    sleep          Performs taking a nap action

Available options:
    -p, --premium  Opt in for premium serivces
    -h, --help     Prints help information

In particular, notice how the "Available options" section is now at the bottom. This could be annoying if the total number of commands is large.
Reproduced with bpaf 0.9.11

This is the current behavior, I have some ideas to how to make this more flexible but nothing concrete yet.


current working directory not in sys.path causing "module not found" errors

I just found out about luigi this afternoon and started teaching myself how to use it.
I set up an environment in /home/john/Practice/luigi/.  I created a hello_luigi.py based on an example in the documentation in /home/john/Practice/luigi While in /home/john/Practice/luigi I ran the command luigi --local-scheduler --module hello_luigi MyTask --x 1 --y 2 and got a "hello_luigi" module not found error.  I modified  /home/john/Practice/luigi/bin/luigi to print the contents of sys.path and found out that whereas the bin directory was on the path it's parent (and my current working directory) /home/john/Practice/luigi was not.  By adding a symbolic in bin using the command ln -s ../hello_luigi.py hello_luigi.py I could get it to run.  This experiment with the symbolic link led the way to my workaround which is to add the following lines to  /home/john/Practice/luigi/bin/luigi just under if __name__ == '__main__
    cwd = getcwd()
    if cwd not in sys.path:
        sys.path.insert(0, cwd)

I see a bin/luigi file in the source code, but it differs from what I'm seeing.  I'm guessing this file is altered by the setup process.  The need for this hack was unexpected.  Skimming through the documentation I didn't see anything as to how a luigi project directory should be set up (i.e. where do the modules with my tasks go?).  Is there some luigi.cfg option that fixes this problem?

This has nothing to do with luigi itself: when it sees '--module hello_luigi' it (roughly saying) tries to import it by means of standard python import system. It's just as if you wrote from hello_luigi import MyTask in your arbitrary script , so the hello_luigi module must be visible to Python interpreter. The list of locations where Python looks for packages is governed by sys.path list which can be configured via PYTHONPATH environment variable. Thus, doing
export PYTHONPATH="$PYTHONPATH:<directory where hello_luigi.py resides>" before runng scripts must help.

I think we need to include instructions about modifying PYTHONPATH right after the pip install command.
[Some musings on technological work-arounds]
I've written Python programs for years and I don't think I've modified PYTHONPATH more than once or twice, so this caught me surprise.  I'm comparing this to some other packages that I've used (pytest, Django, Jupyter, pyflakes).  Django is probably the best parallel in that you have a main script that comes with the package which runs your code.  It side-steps the PYTHONPATH issue by dropping a script (manage.py) in your code directory when you run it's django-admin script to start the project.  Perhaps a more descriptive "You need to add your directory full of tasks to the PYTHONPATH" error message.
I'm aware of the practice of separating out my source code from the virtualenv.  To just crack open a package and try some "hello world" examples I usually don't bother to create the 2 separate directories.

Modifying PYTHONPATH seems a bad idea to introduce python2/3 incompatibility. If you check
python3 -c "import sys;print(sys.path)" you would see an empty string included [''",... ]. Setting luigi to behave differently from python3 -c would violate the minimal surprise principle
###['', '/usr/lib/python35.zip', '/usr/lib/python3.5', '/usr/lib/python3.5/plat-x86_64-linux-gnu', '/usr/lib/python3.5/lib-dynload', '/home/shouldsee/.local/lib/python3.5/site-packages', '/home/shouldsee/.local/lib/python3.5/site-packages/smop-0.32_7_gcce8558-py3.5.egg', '/home/shouldsee/.local/lib/python3.5/site-packages/simplesbml-1.1-py3.5.egg', '/home/shouldsee/.local/lib/python3.5/site-packages/pip-20.1-py3.5.egg', '/usr/local/lib/python3.5/dist-packages', '/usr/lib/python3/dist-packages']

I am making a monkepatch that could be tested.


Leagues with more than 50 players

Hi,
First of all, good work on the API, it works really well!
I've read through your documentation and unless I'm missing something there doesn't seem to be a way to retrieve a complete list of players in a given league if the league contains more than 50 players.
This seems to be the standard page-size FPL uses in their own league views, but is there a possibility for you to extend the API to also accept pageNr (or some other way of getting more than 50 players) by any chance?
Thanks in advance :)

Hi,
Thanks for this.
I have released new version 2.2.0 that now takes pageNumber as an additional param.
https://github.com/tgreyuk/fpl-api-node/wiki#findLeagueStandings
The returned data structure contains a 'has_next' flag.
https://github.com/tgreyuk/fpl-api-node/wiki#type_LeagueStandings
Let me know if you require anything else.
Thanks.....


adding Excel tutorials

:warning: Validation status: warnings

File
Status
Preview URL
Details

docs/tutorials/excel-tutorial-create-chart.md
:warning:Warning

Details

docs/tutorials/excel-tutorial-create-table.md
:warning:Warning

Details

docs/tutorials/excel-tutorial-filter-and-sort-table.md
:warning:Warning

Details

docs/tutorials/excel-tutorial-freeze-header.md
:warning:Warning

Details

docs/tutorials/excel-tutorial-protect-worksheet.md
:warning:Warning

Details

docs/images/excel-tutorial-create-chart.png
:white_check_mark:Succeeded

docs/images/excel-tutorial-create-table.png
:white_check_mark:Succeeded

docs/images/excel-tutorial-filter-and-sort-table.png
:white_check_mark:Succeeded

docs/images/excel-tutorial-freeze-header.png
:white_check_mark:Succeeded

docs/images/excel-tutorial-ribbon-with-protection-on.png
:white_check_mark:Succeeded

docs/tutorials/excel-tutorial-create-chart.md

Line 93: [Warning] Invalid file link:(~/tutorials/excel-add-ins-get-started-jquery.md).

docs/tutorials/excel-tutorial-create-table.md

Line 12: [Warning] Invalid file link:(~/tutorials/excel-add-ins-get-started-jquery.md).
Line 131: [Warning] Invalid file link:(~/tutorials/excel-add-ins-get-started-jquery.md).

docs/tutorials/excel-tutorial-filter-and-sort-table.md

Line 126: [Warning] Invalid file link:(~/tutorials/excel-add-ins-get-started-jquery.md).

docs/tutorials/excel-tutorial-freeze-header.md

Line 68: [Warning] Invalid file link:(~/tutorials/excel-add-ins-get-started-jquery.md).

docs/tutorials/excel-tutorial-protect-worksheet.md

Line 224: [Warning] Invalid file link:(~/tutorials/excel-add-ins-get-started-jquery.md).
Line 234: [Warning] Invalid file link:(~/tutorials/excel-add-ins-core-concepts.md).
Line 237: [Warning] Invalid file link:(~/../reference/excel/excel-add-ins-reference-overview.md).

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

:warning: Validation status: warnings

File
Status
Preview URL
Details

docs/tutorials/excel-tutorial-protect-worksheet.md
:warning:Warning

Details

docs/images/excel-tutorial-create-chart.png
:white_check_mark:Succeeded

docs/images/excel-tutorial-create-table.png
:white_check_mark:Succeeded

docs/images/excel-tutorial-filter-and-sort-table.png
:white_check_mark:Succeeded

docs/images/excel-tutorial-freeze-header.png
:white_check_mark:Succeeded

docs/images/excel-tutorial-ribbon-with-protection-on.png
:white_check_mark:Succeeded

docs/tutorials/excel-tutorial-create-chart.md
:white_check_mark:Succeeded

docs/tutorials/excel-tutorial-create-table.md
:white_check_mark:Succeeded

docs/tutorials/excel-tutorial-filter-and-sort-table.md
:white_check_mark:Succeeded

docs/tutorials/excel-tutorial-freeze-header.md
:white_check_mark:Succeeded

docs/tutorials/excel-tutorial-protect-worksheet.md

Line 237: [Warning] Invalid file link:(~/../reference/excel/excel-add-ins-reference-overview.md).

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


Polish zh version of pod-topology-spread-constraints.md

Fix some format issues.

/assign @xichengliudui

/lgtm


Amazon EC2 - Adding option to inject EC2 user data at launch

Signed-off-by: Eric Nordlund<EMAIL_ADDRESS>

@nrdlngr thanks for the contribution however we are really trying to have an opinionated control over the provisioning.  since not all providers support this we don't want to have different experiences across the drivers.  thanks!

Thanks for your contribution! I agree with @ehazlett on closing this -- To elaborate a little further on my reasoning:

The EC2 driver already has a lot stuff in it and we'd still really like to do a rewrite using the Stripe API bindings (now official) instead of our homebrew solution.  It moved really quickly in the early days and we're still struggling to clean it up.
Before we go throwing things like custom user data into the mix, we need to get comfortable with the matrix we already have (OS x Provider) and provide the most uniform experience possible.  Before we allow flexibility in terms of provisioning, we have to decide what we are and are not willing to accept and define a real provisioning model, e.g. Do we support cloud-init only?  Do we allow users to run bash scripts at various points in provisioning?  Do we bootstrap Compose automatically?  There's a lot to consider.

It's not "no forever", but we need a definition of behavior which is relatively consistent across drivers before letting such a feature in.

Any chance this decision could be revisited? https://github.com/docker/machine/pull/2141 was merged so this would actually bring consistency.

I'd certainly review a pull request.  It will need to be different than this code probably since the EC2 driver has been re-written to use the official Go bindings for AWS.


unable to get content for MessageUnsupported

Question is how get content for MessageUnsupported (I use latest build of tdlib - 1.8.20).
Once it was received (via UpdateMessageContent) after GetChatHistory call, but at next program launch when I call GetChatHistory, it receives the same MessageUnsupported again, but no any UpdateMessageContent.
I know, tdlib utilizes local DB, maybe updated content of the message is stored in that DB ?
I tried GetMessageLocally and ViewMessages (with different sources) calls, but still not result (no content, and no updates).

Similar issues here.
Also found out that all quoted messages, or similar messages carrying any feature introduced by latest release, will all be categorized into messageUnsupported. Updating TDLib to latest version does not help.
Does TDLib plan to support those new features in subsequent releases?

The answer is here - https://github.com/tdlib/td/issues/2657#issuecomment-1789638698

@Vadis00 Thanks!

After release 1.8.21, messages are being received OK, no MessageUnsupported


Event trigger

This PR allows to register a callback on 2 events.

changes on the header checkbox (on-header-checkbox-changed)
uncheck a row (on-uncheck)

I'm not sure what's the proper way to do it (and what would be the correct names to use), but it's usefull if you want to do ajax calls when you click on somewhere.

Thanks for the PR!
A uncheck event is a tad odd, what are you trying to accomplish maybe there is a different way?

I'm doing ajax calls on checkboxes changes (because the pager doesn't work well yet with the multiselect).

I believe the checkbox event fires on any change, why not just use that?

The "on-select" event only fires on select. I couldn't find an "on-change" event or something like that. But maybe I didn't understand what was going on.
I couldn't find an event for the header checkbox either.

Sorry, been busy on other projects. I'll look into this in the next week.

I'm closing it because I have the on header checbox change in that one.
Also there are other pr/issue for this problem.


[BUG] - Dropdown Menu in Navbar changed position when scroll

Describe the bug
When I use Dropdown Component with User Component inside.
Dropdown Menu is shown when clicked but its position didn't fix when scrolling
Your Example Website or App
https://codesandbox.io/s/8isf4c?file=/App.js&from-sandpack=true
Steps to Reproduce the Bug or Issue
Scrolling
Expected behavior
Expected like this:

Screenshots or Videos

Operating System Version
Windows
Browser
Chrome

Hey,
Why you closed this one? I encountered the same issue with an fixed Navbar from NextUI.
Did you find a work around? If so i would be glad if you share your approach :-)

Hey,
Why you closed this one? I encountered the same issue with an fixed Navbar from NextUI.
Did you find a work around? If so i would be glad if you share your approach :-)

You just need to pass "position: fixed" inside containerCss in Dropdown.Menu

I tried your code and it worked! I simply have to adjust the top parameter a bit.
Thank you!

Hey please upgrade to V2 since V1 will no receive more updates.


Refresh flake.lock

https://github.com/nickel-lang/organist/pull/122 landed without relevant flake.lock changes. Land these changes and add a check to prevent this from happening in the future.

Here's an example of the check failing without flake.lock change: https://github.com/nickel-lang/organist/actions/runs/6088127317/job/16518145287


Event names with white spaces wont log to analytics

Currently we are unable to log events with the event names containing white spaces, same goes for the keys in the bundle sent. Is there a way to do it or is it a planned feature for the future release?

@varunrramani this is working as intended, no plans to allow spaces that I am aware of.
From: https://firebase.google.com/docs/reference/android/com/google/firebase/analytics/FirebaseAnalytics#logEvent(java.lang.String, android.os.Bundle)
"""
The name of the event. Should contain 1 to 32 alphanumeric characters or underscores. The name must start with an alphabetic character. Some event names are reserved. See FirebaseAnalytics.Event for the list of reserved event names. The "firebase_" prefix is reserved and should not be used. Note that event names are case-sensitive and that logging two events whose names differ only in case will result in two distinct events.
"""


Feature/dark mode

Codecov Report

Merging #242 (fd40c3e) into dev (042764f) will increase coverage by 5.74%.
The diff coverage is 93.29%.

@@            Coverage Diff             @@
##              dev     #242      +/-   ##
==========================================
+ Coverage   89.79%   95.54%   +5.74%
==========================================
  Files        1519     1485      -34
  Lines        6770     6480     -290
  Branches     1024     1050      +26
==========================================
+ Hits         6079     6191     +112
+ Misses        546      137     -409
- Partials      145      152       +7

Impacted Files
Coverage Δ

...eact-magma-dom/src/components/Card/CardHeading.tsx
100.00% <ø> (ø)

...ma-dom/src/components/Dropdown/DropdownContent.tsx
100.00% <ø> (ø)

...s/react-magma-dom/src/components/Heading/index.tsx
100.00% <ø> (ø)

...react-magma-dom/src/components/Paragraph/index.tsx
100.00% <ø> (ø)

...es/react-magma-dom/src/components/Search/index.tsx
94.44% <ø> (ø)

...eact-magma-dom/src/components/Select/ItemsList.tsx
100.00% <ø> (ø)

...-dom/src/components/Select/SelectTriggerButton.tsx
100.00% <ø> (ø)

packages/react-magma-dom/src/i18n/default.ts
100.00% <ø> (ø)

packages/react-magma-icons/src/categories.ts
100.00% <ø> (ø)

...s/react-magma-dom/src/theme/GlobalStyles/index.tsx
50.00% <28.57%> (-50.00%)
:arrow_down:

... and 91 more

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 e343390...fd40c3e. Read the comment docs.

Here are some styling tweaks I'd like for us to try:

Update all borders to rgba(255,255,255, 0.25)


No way of passing meta-data to agm-polygon

Issue description
Based on the document and the code there is no way of passing meta-data to agm-polygon.
This would be essential to know which polygon was clicked.
Even an Id would be great that reflects the creation order.
See related Issue #1627
<agm-polygon *ngFor="let p of polygons; let i = index;" [paths]="p.path" … >
</agm-polygon>

Demo: https://stackblitz.com/edit/angular-google-maps-demo-texamg

a polygon accepts an array of arrays of LatLngs or LatLngLiterals

Also, you can pass the agmpolygon in the template event hook like this: https://stackblitz.com/edit/angular-google-maps-demo-3dhbk6

Thank you very much for your feedback.
Your demo doe not help.
this returns the AppComponent, which I am in t the time of receiving the message in onPolyClick(), and its useless.
Unfortunately, you did not answer the original question:
Can I pass meta-data to agm-polygon, please?

Yes, you're correct, sorry.

I have a set of building polygons within a parcel boundary polyline.
Clicking on a building polygon I need to know which building was clicked on, so I can display the building info.

<agm-polygon *ngFor="let building of buildings; let i = index;"
       [paths]="building.path" (polyClick)="onPolyclick(building)">
</agm-polygon>


Take movable context idea from cpp.react

Description

Currently, contexts aren't movable. But they quite easily can be made one, using cpp.react's an approach for Group. When Context itself is merely a pointer that can be copied, while the only thing that actually matters is reactive_graph.
It was requested on reddit. Not strictly, but it is nice to have feature.
Extra information

µReact version: 0.1.0

Actually it's more than just movable. It is contexts that guarantied outlive nodes that use it. It is more safe and sane using it such way.

Already done in 0.10.0 wip


Grim cannot take a screenshot in sway-1.0 master, GL_INVALID_OPERATION

With the latest master of sway and wlroots, grim fails to take a screenshot, saying "failed to copy output X11-1". I have filed a bug report there (https://github.com/emersion/grim/issues/22), but they claim it is a wlroots issue.
Therefore, I'm posting this bug here. I have attached a logfile. The last entry pops up every time I try to take a screenshot.
 In my defense, I do not control the hardware on this machine and I need cuda support, so I'm kinda stuck with those shitty drivers. Sorry.
sway.log

Can you try to reproduce with rootston? Please enable debug logs and upload the whole file.

I am aware that using nvidia drivers is unsupported and if this bug is caused by that

Since you use the X11 backend, it should work. We don't support nvidia on DRM.

Here's a log. I'm starting rootston just like I normally would sway, open a terminal, try grim test.png, and close the session.
rootston.log


Support Alarming Delay

We need to support one or both of:

alarm on-delay
alarm off-delay

Our current alarm system supports this and uses this apparently (though I'm not sure which one, maybe both).  Delays help minimize nuisance alarms.  An off-delay keeps an alarm active for a time past when the alarm would otherwise no longer be active (setpoint threshold no longer exceeded).  An on-delay prevents an alarm from being active for a time even after it otherwise would have entered the active/alarming/annunciating state (setpoint threshold exceeded).

We've identified at least three implementation options to consider:

The alarm source could handle delays (an EPICS IOC for example)
The client (HMI/GUI) could handle delays
A Stream processing app could handle delays (the filter app could be extended to include timers - or a whole new stream processing app could be created)


Current Version Number still set as 3.4.0. Assumes it needs to update to version v4.0.0-beta.1

I installed the v4.0.0-beta.1 release, but the "About" page shows the version number as <IP_ADDRESS>

When the "Check For Update" button is clicked it states that a new update is available. This is despite the downloaded installer being v4.0.0-beta.1

I'm pretty sure this is the v4.0.0-beta.1 release as:
-it needed .NET 5.0
-it has the features added in that release (Group Packages by publisher, Download with IDM, etc...)

Version 4 is still in beta and I do not have enough time to develop it at the moment


Add workspaces kcpsanity cases

Hi @kasturinarra @wangke19, I added some test cases related to workspaces:

list sub-workspaces under a specific workspace

STEP: 1) Get workspaces before test starts
Current workspace is "root:users:zu:yc:kcp-admin".
Note: 'kubectl ws' now matches 'cd' semantics: go to home workspace. 'kubectl ws -' to go back. 'kubectl ws .' to print current workspace.
STEP: 2) Create five workspaces under the home workspace
STEP: 2.1.0) Go to home workspace
Current workspace is "root:users:zu:yc:kcp-admin".
Note: 'kubectl ws' now matches 'cd' semantics: go to home workspace. 'kubectl ws -' to go back. 'kubectl ws .' to print current workspace.
STEP: 2.2.0) Create workspace under home workspace
Workspace "qetestztwm59vt" (type root:universal) created. Waiting for it to be ready...
Workspace "qetestztwm59vt" (type root:universal) is ready to use.
Current workspace is "root:users:zu:yc:kcp-admin:qetestztwm59vt" (type "root:universal").
.....

STEP: 3) Go to home workspace
Current workspace is "root:users:zu:yc:kcp-admin".
Note: 'kubectl ws' now matches 'cd' semantics: go to home workspace. 'kubectl ws -' to go back. 'kubectl ws .' to print current workspace.
STEP: 4) Check number of workspaces
Current workspace is "root:users:zu:yc:kcp-admin".
Note: 'kubectl ws' now matches 'cd' semantics: go to home workspace. 'kubectl ws -' to go back. 'kubectl ws .' to print current workspace.

passed: (40.9s) 2022-08-05T04:15:50 "[kcp] Kcpsanity Author:zxiao-Medium-2800715-Get all sub-workspaces under current workspace [Suite:openshift/conformance/parallel]"

delete workspace

STEP: 1) Go to home workspace
Current workspace is "root:users:zu:yc:kcp-admin".
Note: 'kubectl ws' now matches 'cd' semantics: go to home workspace. 'kubectl ws -' to go back. 'kubectl ws .' to print current workspace.
STEP: 2) Get current workspace
STEP: 3) Navigate to previous workspace
STEP: 4) Undo navigate action, and return to starting workspace

passed: (8.7s) 2022-08-05T07:08:15 "[kcp] Kcpsanity Author:zxiao-Medium-2800714-Navigate between workspaces using 'cd' semantics [Suite:openshift/conformance/parallel]"

navigate between workspaces using cd semantics

STEP: 1) Create workspace
Current workspace is "root:users:zu:yc:kcp-admin".
Note: 'kubectl ws' now matches 'cd' semantics: go to home workspace. 'kubectl ws -' to go back. 'kubectl ws .' to print current workspace.
Workspace "qetestmuvhldac" (type root:universal) created. Waiting for it to be ready...
Workspace "qetestmuvhldac" (type root:universal) is ready to use.
Current workspace is "root:users:zu:yc:kcp-admin:qetestmuvhldac" (type "root:universal").
STEP: 2) Check if workspace exists
Current workspace is "root:users:zu:yc:kcp-admin".
Note: 'kubectl ws' now matches 'cd' semantics: go to home workspace. 'kubectl ws -' to go back. 'kubectl ws .' to print current workspace.
STEP: 2) Delete workspace
Current workspace is "root:users:zu:yc:kcp-admin".
Note: 'kubectl ws' now matches 'cd' semantics: go to home workspace. 'kubectl ws -' to go back. 'kubectl ws .' to print current workspace.
workspace.tenancy.kcp.dev "qetestmuvhldac" deleted
STEP: 3) Check if workspace is fully deleted

passed: (16.7s) 2022-08-05T07:09:25 "[kcp] Kcpsanity Author:zxiao-Medium-2800716-Delete workspace [Suite:openshift/conformance/parallel]"

All cased tested locally. Because we haven't established a case naming system, I temporarily use our current case ID system. We can change it anytime if we agree on one naming format. Please take a look if you have time. Thanks

For your case, I think it is executed by admin or non-admin user, they are different scenarios. We can describe case title like As an admin, I can ..., As a non-admin, I can ...
For workspace operations, I think we can merge them into one test case.

@zimo-xiao also i see @penghao Wang added some cases, i suggest better to collaborate and work so that we do not duplicate the efforts, thanks !! From next time let us discuss what you would like to start picking up either to automate or to do some manual tests, thanks !!


Support bépo keyboard layout

I’ve just had a look at Doom Emacs and it looks very promising! However, just like #767 and #783, I’m using a non-{AZ,QW}ERT{Y,Z} layout: instead, I’m using the Bépo keyboard layout and I’m used to swapping HJKL and CTSR when using either Vim or Spacemacs, along with some other keybindings (like “é” as a synonym for “w”).
Spacemacs already natively supports such a feature.
For now, I seem to be able to have roughly what I want by creating a bepo.el file in my ~/.doom.d directory like so:
;;; bepo.el -*- lexical-binding: t; -*-

(map! :nvm "h" 'evil-replace
      :nvm "H" 'evil-replace-state
      :nvm "j" 'evil-find-char-to
      :nvm "J" 'evil-find-char-to-backward
      :nvm "k" 'evil-substitute
      :nvm "K" 'evil-change-whole-line
      :nvm "l" 'evil-change
      :nvm "L" 'evil-change-line

      :nvm "c" 'evil-backward-char
      :nvm "C" 'evil-window-top
      :nvm "t" 'evil-next-line
      :nvm "T" 'evil-join
      :nvm "s" 'evil-previous-line
      :nvm "r" 'evil-forward-char
      :nvm "R" 'evil-window-bottom

      :nvm "é" 'evil-forward-word-begin
      :nvm "É" 'evil-forward-WORD-begin

      :map evil-snipe-local-mode-map
      (:nm "k" 'evil-snipe-s
       :nm "K" 'evil-snipe-S
       :nm "s" nil
       :nm "S" nil)
      :map evil-snipe-override-local-mode-map
      (:m "j" 'evil-snipe-t
       :m "J" 'evil-snipe-T
       :m "t" nil
       :m "T" nil))

But I’m not sure if that’s sustainable, since it means that I’d need to swap hjkl and ctsr in any major mode map.
In the end, I’d just like to have (doom! :input bepo) in ~/.doom.d/init.el and have my Bépo bindings ready.

That's definitely not enough, all the evil collection maps, and all packages that have specific evil-* packages, and all other common maps (like mapping é to doom-window-map in normal mode, and possibly map «» and èà (even maybe çê) to avoid chording would be necessary.
The spacemacs layer also remaps avy keys so the targets stay on home row.
I'm slowly creating a module to handle this, and you can steal it if you want, but knowing that Henrik definitely cannot maintain this, I'm waiting to have something better before pushing a PR

Feel free to push PRs and issues if you see anything weird


YOLO NAS export ONNX Dynamic

💡 Your Question
How to export a YOLO NAS model to onnx with dynamic = True?
Regards,
Kevin
Versions
No response

We haven't tested ONNX export with dynamic shapes and model.export() is not designed to support dynamic shapes (At least for current 3.5.0 release).
You can however try exporting a model using torch.onnnx.export and specify dynamic axis during export. I believe this should work, but the model you will obtain will not have the NMS postprocessing.


Mapper is ignored when using custom command

So I have rom-http installed, and I need to create auth. So I've created Login command:
class Login < ROM::Commands::Create[:http]
  relation :session_users
  register_as :login
  result :one

  def execute(login, password)
    response = relation
        .dataset
        .with_base_path('/customer-sessions/login')
        .with_options(request_method: :put, params: {
            login: login,
            password: password
        })
        .response
    [response]
  end
end

A relation:
class SessionUsersRelation < ROM::Relation[:http]
  gateway :volgaspot

  schema(:session_users) do
    attribute :id, Types::Strict::Int
    attribute :login, Types::Strict::String
    attribute :full_name, Types::Strict::String
  end
end

And a repository
class SessionUserRepository < ROM::Repository::Root
  root :session_users

  def login(login:, password:)
    session_users
        .map_with(:session_users_mapper)
        .command(:login)
        .call(login, password)
  end
end

I need to rename a couple of fields and instantiate custom model with the methods I need. For testing purposes, all I've done right now is renaming full_name to name:
class SessionUserMapper < ROM::Mapper
  relation :session_users

  model SessionUser

  register_as :session_users_mapper

  attribute :id
  attribute :login
  attribute :name, from: :full_name
end

Also here's a model with a custom method
class SessionUser < ROM::Struct
  constructor_type :schema

  attribute :id, Types::Strict::Int
  attribute :login, Types::Strict::String
  attribute :name, Types::Strict::String.optional

  def test
    "hello world"
  end
end

So what I'm expecting is that the mapper renames full_name to name and instantiates my custom model. But for some weird reason it ignores my mapper, and creates instance of ROM::Struct::SessionUser with full_name attribute (and of course no test method). And what confuses me the most is that under almost identical conditions (same relations, same mappers) fetching of users does work and renames as expected.

This is more of a rom-core thing than rom-rails specifically.
@solnic can correct me if I'm wrong here, but I believe that commands in general don't respect mappers.  ROM is built off the idea of command - query separation, so a create runs through different paths than a read.  I think there was, at one point, the though of removing the data-return (outside  of PK) for commands, but I'm not sure where that went.
If your HTTP login cycle mimicked a more standard post-and-redirect pattern, where the login POST returns something like an HTTP OK and a location for fetching the user, rather than expecting it to encode the results in that request, it fits better with the underlying expectations.  I understand that's less raw efficiency because two http requests, and also that you may not have any particular control over the service you're interacting with.
It's worth noting that you can call a mapper directly.  the following is not tested,  but should be something like:
  def login(login:, password:)
    (session_users.command(:login) >> SessionUserMapper).call(login, password)
  end

if I know correctly you should use changeset and map data using it

Thanks for response!
@cflipse At first I thought that mappers didn't load somehow and I need additional settings, but now when you've pointed out about the command - query separation I see it makes sense. Oh, and a direct mapping works like a charm, thanks!
@gotar I've used direct mapper call for now, but I'll test a changeset too. Thanks!


website bug.

Moment's website has two times "Japanese" as selection on the landing page at the bottom.

One is Japanese with Latin characters and the other is Japanese with Japanese characters. They should be labeled differently, of course, but it seems OK.

I'm sorry, but it's clearly not Japanese with roman letters... it's not even close to Japanese. Did you actually look at it?
// 5 taun ingkang kepengker
// 4 taun ingkang kepengker
11 jam ingkang kepengker
// wonten ing 13 jam
Please keep the issue open until it gets fixed because this is not Japanese.

ah, you're right, it's Javanese. Ticket is open here: https://github.com/moment/momentjs.com/issues/360


bluetooth: tester: MICP Client tests

Add support for MICP Client tests

@carlescufi I think this can be merged without @sjanc approval (he is on OOO for 2 weeks it seems)


feat: render full error information

This adds all the error information we need to the error info page. It
also introduces styling via Origami and some help text to explain to
engineers what each piece of error information is for.
Feedback
I'm looking for general feedback on:

The approach. We agreed we'd like to avoid a build step or lots
of additional dependencies. Is this approach still what we want
now that we have some code to look at?

Is there any more useful (and easy to get hold of) information
that we might want to display on this page?

Do we want to try and tackle error.stack parsing as part of
this PR? Or should we aim to get this merged and add as a
later feature?

Potential Improvements
We could possibly move each of the renderX methods into their
own files, but this might be overkill. For now I split out the layout
as it's mostly static and made the renderErrorPage method far
harder to understand.
How it looks

With an OperationalError
With a non-operational error

I think these are all great suggestions, and agree we should add more info in a later PR 👍


Fixes for HDFS node expansion on BDC connections

Fixes https://github.com/microsoft/azuredatastudio/issues/13547
The main fix here for the issue is to only connect to the controller if we need to - in most cases this will be unnecessary.
In addition I also

Added some debug logging to determine exact errors if we fail to connect to the token endpoint when determining if the cluster supports kerberos auth
Changed to using getEndPoints to "test" the controller connection since that's available to everyone. The previous getClusterConfig is only available to admin users.

Pull Request Test Coverage Report for Build 541090490

1 of 2 (50.0%)  changed or added relevant lines in 1 file are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage remained the same at 43.538%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

extensions/notebook/src/jupyter/jupyterSessionManager.ts
1
2
50.0%

Files with Coverage Reduction
New Missed Lines
%

extensions/notebook/src/jupyter/jupyterSessionManager.ts
1
72.14%

Totals

Change from base Build 539729717:
0.0%

Covered Lines:
24933

Relevant Lines:
52476

💛  - Coveralls


[BUG] integration test failure at "verify host maintenance mode" step

Describe the bug
The integration test might fail at the following test:
------------------------------
verify host APIs operate via steve API
  verify host maintenance mode
  /go/src/github.com/harvester/harvester/tests/integration/api/host_apis_test.go:42
STEP: enable maintenance mode of the host
STEP: then the node is unschedulable and maintain-status is set
STEP: disable maintenance mode of the host

• Failure [0.040 seconds]
verify host APIs
/go/src/github.com/harvester/harvester/tests/integration/api/host_apis_test.go:18
  operate via steve API
  /go/src/github.com/harvester/harvester/tests/integration/api/host_apis_test.go:20
    verify host maintenance mode [It]
    /go/src/github.com/harvester/harvester/tests/integration/api/host_apis_test.go:42

    failed to post disableMaintenanceMode action, response with 500, Operation cannot be fulfilled on nodes "harvester-control-plane": the object has been modified; please apply your changes to the latest version and try again

    /go/src/github.com/harvester/harvester/vendor/github.com/onsi/ginkgo/ginkgo_dsl.go:463

    Full Stack Trace
    github.com/harvester/harvester/tests/integration/api_test.glob..func1.1.3()
        /go/src/github.com/harvester/harvester/tests/integration/api/host_apis_test.go:64 +0x4d4
    github.com/harvester/harvester/tests/integration/api_test.TestAPI(0xc00148a600)
        /go/src/github.com/harvester/harvester/tests/integration/api/suite_test.go:53 +0xee
    testing.tRunner(0xc00148a600, 0x345f160)
        /usr/local/go/src/testing/testing.go:1193 +0xef
    created by testing.(*T).Run
        /usr/local/go/src/testing/testing.go:1238 +0x2b3
------------------------------

To Reproduce
Steps to reproduce the behavior:

This sometimes happens during the CI tests. Some failed tests:
https://drone-pr.rancher.io/harvester/harvester/1021/1/2
https://drone-pr.rancher.io/harvester/harvester/1039/1/2

Expected behavior
The test should pass.

close in favor of the harvester CI as we didn't encounter a similar issue nowadays.


SECURITY ISSUE - Password in plain text under MiCloud Settings

Describe the bug
Password in plain text under MiCloud Settings under any devices
Expected behavior
Passwords should be hidden

I mean I do not see this is as a security issue as the config is located on your device. I am not aware of any secure way to store passwords in HomeKit, if you have suggestions the I am open for it.


What should I put in Valid redirect uris in console of Keycloak when auhentication is succefull

Hi, I'm new with autentication in keycloak.
Right now I can open the browser to enter the credentials, but when tap on Log in

this page send me to inside of console of keycloak, but I need close the browser and return me to my app.
Maybe some one can explain me how can I do that.
Because I know I can modify the field  valid redirect uris inside of console of keycloak, but I don't sure what path I should put there for return me to my app.

Check the documentation please - Setup Deep Links (iOS)

I need it... please we need more info in docs


I don't program, are there ways I can help?

I don't know how to program but I really appreciate the work that went into this.
This could also be titled, how can I throw money your way?  I know you are a team but if there was something you as a team need I'd be happy to contribute.  We as a server don't have a ton of money but when I come across people trying to make DayZ better, or more reachable for the masses I'd like to help.

Thanks for the offer, I will let you know if I can come up with a way to accept money...
Until then, feel free to donate the money you would have given me to some children's charity
They could use some money way more than I do :)

Thank you. I made a donation to Shoes for the Shoeless.  It's a charity in SE Ohio that provides shoes to children in the area who are below the poverty line.    And included the link to here.  Thanks again!


Repo sync for protected CLA branch

The pull request is created from master637339772240777462 to master to fix git push error for protected CLA branch

@Taojunshen : Thanks for your contribution! The author(s) have been notified to review your proposed change.


Consider different resource management (voice, inventory, other items)

E.g. voice samples could be loaded separately for each scene, and cleared automatically when the scene is removed. To achieve that, a new node type could be introduced, which would be added to each gameplay scene and would hold all voice samples. Global goat_voice would then find that node and read voices from it. This would, however, require a different default voice setting.
For inventory we could add all inventory items to Inventory scene, and then enable them when certain conditions occur. We should confirm first if loading all inventory items at once wouldn't slow down the program too much.
EDIT: this could also solve the problem with orphaned resources.

This goes along perfectly with the main screen plugin: there can be a separate GOAT tab for e.g. managing the inventory, with a 3D preview of each item.


display optional maintainer email

So people can report bugs directly to sysadmin, request bridges, etc

Thanks for merging changes. I'll rename "maintainer" to "administrator" before merging.

I added the necessary changes. Does that work for you?

Yes! I meant to do it myself, but thank you for your diligence. :)

I meant to do it myself, but thank you for your diligence. :)

Sorry for the intrusion, I was feeling bad for requesting too many changes. Next time I'll be more patient!


Fix decimal float parser

Parsing decimal float should cause error if the result is infinite.
Passed new 8 tests.
before
End ".../wain/spec-test/wasm-testsuite/const.wast":
  total: 767, passed: 759, failed: 8, skipped: 0

Results of 76 files:
  total: 19740, passed: 19046, failed: 498, skipped: 196

after
End ".../wain/spec-test/wasm-testsuite/const.wast":
  total: 767, passed: 767, failed: 0, skipped: 0

Results of 76 files:
  total: 19740, passed: 19054, failed: 490, skipped: 196

あーなるほど，確かに fN.const の引数が整数かつ float の範囲を超える時に inf になっちゃいますね…盲点でした

ありがとうございます！


This allows the emoji picker to be shown outside the context of a team

Summary
This allows the emoji picker to be shown outside the context of a team. This is needed for example to show the emoji picker when you are in boards, where the team is not part of the context (for now).
Ticket Link
mattermost/focalboard#1286
Release Note
NONE

Cherry pick is scheduled.


Error occurred when executing LuminaT2ISampler:  index 5 is out of bounds for dimension 0 with size 4

I get this error if the row numbers are not the same:
Error occurred when executing LuminaT2ISampler:
index 5 is out of bounds for dimension 0 with size 4

Can you elaborate? The rows and columns need to form a grid, example from the original demo:

think I get it now, couldn't get the horse though :)


Polling of arma server during session time

We need FABot to announce when the ARMA server goes into slotting mode during our sessions; this means that during the period from 1920h GMT to ~0000h GMT (or between a host using a "!session starting" and a "!session finished" command), the bot has to poll the server regularly (every 30-60 seconds?) and catch transitions into the slotting state and announce those in #general on the discord server.

#38


When logging in for the first time, server crashes with TypeError: Cannot read property 'deleted' of undefined in node_modules/hospitalrun-server-routes/routes/30-auth.js:134

Expected behavior:
Should show logged in state
Actual behavior:
Shows sidebar, but only the HospitalRun header and the cog icon. The login remains
/Users/gregor/Projects/hospitalrun/frontend/node_modules/hospitalrun-server-routes/routes/30-auth.js:134
      if (body.deleted) {
              ^

TypeError: Cannot read property 'deleted' of undefined
    at /Users/gregor/Projects/hospitalrun/frontend/node_modules/hospitalrun-server-routes/routes/30-auth.js:134:15
    at Request._callback (/Users/gregor/Projects/hospitalrun/frontend/node_modules/nano/lib/nano.js:241:7)
    at Request.self.callback (/Users/gregor/Projects/hospitalrun/frontend/node_modules/request/request.js:200:22)
    at emitTwo (events.js:87:13)
    at Request.emit (events.js:172:7)
    at Request.<anonymous> (/Users/gregor/Projects/hospitalrun/frontend/node_modules/request/request.js:1067:10)
    at emitOne (events.js:82:20)
    at Request.emit (events.js:169:7)
    at IncomingMessage.<anonymous> (/Users/gregor/Projects/hospitalrun/frontend/node_modules/request/request.js:988:12)
    at emitNone (events.js:72:20)

Screenshot:

Steps to reproduce:

Open Google Chrome Private mode to make sure you have a clear cache
start server with npm start
Open app at http://localhost:4200
Login with hradmin and test

OS and Browser:

Chorme Version 50.0.2661.102 (64-bit)

Mac 10.11.4

Latest version of master (2adfdea).

node v4.2.2, npm 3.8.6

@gr2m did you run the script ./script/initcouch.sh?
That code could be more bullet proof, but I'm trying to figure out why/how you got that error in the first place.
Were you able to login at some point then?

yes I run it before (with admin username / password). Tried to run it again, got conflict errors as expected:
$ ./script/initcouch.sh admin funkyfresh
{"ok":true}
{"error":"file_exists","reason":"The database could not be created, the file already exists."}
{"ok":true}
{"error":"file_exists","reason":"The database could not be created, the file already exists."}
{"ok":true}
"{couch_httpd_oauth, oauth_authentication_handler}, {couch_httpd_auth, proxy_authentification_handler}, {couch_httpd_auth, cookie_authentication_handler}, {couch_httpd_auth, default_authentication_handler}"
"true"
{"error":"conflict","reason":"Document update conflict."}

@gr2m Were you able to login after the first time?

yes, looks like the POST /_session request goes through. Here are all requests as har: https://gist.github.com/a4cf5f284ad42b11468712abcdcefbde


Gadfly changes Base.range printing

Normally, display(UnitRange[1:3000, 5:6]) prints the ranges as 1:3000, 5:6, but
using Gadfly
display(UnitRange[1:3000, 5:6])

will display all numbers from 1 to 3000.
(note: julia#265 means that you can't get both behaviours in the same session)

I'm not able to reproduce this in Julia 0.4.5 with the latest Gadfly.

Thank you for checking. I investigated further, and it turned out that the problem was in the latest release of Compose.jl, but it has already been fixed on master. There hasn't been a Compose.jl release in five months...

Maybe ping someone who has commit access to the Compose repo and tell them to release the current master? I'm sure there have been a couple bugs squashed since the last release.


#11426: Replace tt_lib in models/experimental/grok,hrnet,lenet,helper…

Issue : https://github.com/tenstorrent/tt-metal/issues/11426 Tracking Issue : https://github.com/tenstorrent/tt-metal/issues/11240 , Master Issue : https://github.com/tenstorrent/tt-metal/issues/10532
Work Done:

Replace tt_lib usage in models/experimental/grok
Replace tt_lib usage in models/experimental/hrnet
Replace tt_lib usage in models/experimental/lenet
Replace tt_lib usage in models/helper_funcs.py

Checklist

[x] All Post commit CI
[ ] Nightly Fast Dispatch Test
[ ] Model regression CI testing passes

Did you run nightly fast dispatch?

Did you run nightly fast dispatch?

yes @tt-rkim . It was Fine.


Buckets don't have dispenser behaviors

Minecraft Version
1.18.2
Fabric/Quilt Version
0.14.9
Fabric/Quilt API Version
0.58.0+1.18.2
Tinkers' Construct Version
1.18.2-3.5.1.DEV.33b3fef
Describe your issue
The buckets that this mod registers do not have a dispenser behavior. When the buckets are inside of a dispenser and it gets powered, they shoot out like other items, instead of placing the fluid like the built-in buckets.
Crash Report
No response
Other mods
The modpack is Cabricality.
Tried reproducing with just Tinkers?
No
Performance Enchancers
Sodium
Searched for known issues?
Searched open issues

Have you checked if this is the same as forge?

Apparently it is, which I find odd because I saw images of it working on an older version of the forge mod, the images found here. I'll close the issue since it's the same as forge, however it would be nice to see support for it anyway.

reopening as this has been fixed in the forge version (commit https://github.com/SlimeKnights/TinkersConstruct/commit/77b1160d6b2b2962dad051f487e02fba18979a41 issue https://github.com/SlimeKnights/TinkersConstruct/issues/4978)

@NyakoFox can you double check this issue the logic in the code seems to checkout not sure if this issue should be closed or not?

This works in the current version so I'm just gonna close this


Refactor: Added Ruff and linted the codebase

Added Ruff to the project with a config of some common linter rules.
This is done in preparation for automated precommit and CI linting, which should make it easier to maintain the code without regressions, especially once more contributors join.
Ruff was selected because it is lightning fast -- meaning it can be configured to run automatically on commit / pre-push without slowing you down.
Linted the codebase, then verified that the notebook still works. I would have verified using the unit tests, but it seems the tests on main are currently failing (see https://github.com/aorwall/moatless-tools/issues/21 ).
Notably:

Using modern syntax for type hints. Optional[], Union[], Type[], List, Dict, Set... can be simplified a lot since we're using Python 3.10 and https://peps.python.org/pep-0585/ and https://peps.python.org/pep-0604/ apply
Print() statements have been replaced with logger.info(), logger.warning() or logger.error()
Import sort order is enforced
open(path, "r") is replaced with open(path) since "r" is the default
The folders ["tests", "evaluations", "notebooks"] are excluded since it made less sense to enforce linting rules there

Looks good to me. And I agree with what needs to be linted. About the tests I answered in #21.  Need to do more work to improve code coverage. The best ROI right now is probably to get the replay trajectory functionality working since that will at least give us regression testing of trajectories we know are correct.


code review for your project

your project was done well and nicely ver clean code but in the read me u should gave credit to your partner as well

I do it by mistake please remove his name and update readme file properly.
On Fri, Apr 28, 2023, 12:13 PM kerlos-bahaa @.***>
wrote:

your project was done well and nicely ver clean code but in the read me u
should gave credit to your partner as well
—
Reply to this email directly, view it on GitHub
https://github.com/Anwaar12/Awesome-books/issues/11, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AQATRWHW7KNVEIV4Q5FAG33XDNU2BANCNFSM6AAAAAAXO23QQI
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>


Feature Request: arbitrary keymapping

Feature Request

Arbitrary keymapping.
Rationale
System-, browser-, extension-, and user-defined key combinations really pile up over time. Type of user who actively goes looking for UX tweaks like what Key-ncognito provides often has several hotkey combos already tied up in customization- and productivity-related software.
Extending the set of available inputs for Key-ncognito's basic key combination (or combination(s), if my previous feature request goes anywhere) would go a long way toward avoiding conflicts, especially with third-parties that love to hog the main modifier keys and don't allow user configuration.

Thanks for your interest and feature request. :)
Currently, ken-ncognito allows for Meta (Alt), Super (Command), Shift and Ctrl to be used in any combination along with a click (modulo System overridden combinations one can test for on the settings page itself). Which other key(s) did you have in mind?

I saw the Linkclump extension you alluded to in #5 and saw that they allow key-mapping with literally any key ([0-9a-z]) pressed on the keyboard during selection as a trigger. Is that what you meant here?
Personally, I use Vimium and, hence, most of the keys on my keyboard already have bound meanings. Hence, I haven't really ever considered using any of those keys as modifiers but I can see merit in the approach.


pr

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.王嘉宁 seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


LUCENE-10379: Count directly into the dense values array in FastTaxonomyFacetCounts#countAll

This change just pulls out part of the optimization done (and subsequently reverted) in LUCENE-10350. The idea is to push only part of the optimization to see if we can isolate the performance regression observed in LUCENE-10350. See LUCENE-10374 for more.

I'm seeing the nice performance improvement locally that @gf2121 and I both observed with LUCENE-10350, so I'm going to go ahead and merge this, which is just a portion of LUCENE-10350 (and hopefully not the portion responsible for the nightly bench regressions).
                            TaskQPS baseline      StdDevQPS candidate      StdDev                Pct diff p-value
       BrowseDayOfYearSSDVFacets       12.73     (15.5%)       12.30     (14.7%)   -3.4% ( -29% -   31%) 0.483
                        Wildcard      103.85      (9.1%)      102.09      (9.3%)   -1.7% ( -18% -   18%) 0.559
     BrowseRandomLabelSSDVFacets        9.73      (7.0%)        9.64      (7.2%)   -0.9% ( -14% -   14%) 0.690
                      OrHighHigh       45.50      (3.8%)       45.11      (3.7%)   -0.9% (  -8% -    6%) 0.466
                       OrHighMed      129.29      (4.3%)      128.54      (4.4%)   -0.6% (  -8% -    8%) 0.672
                    HighSpanNear        1.79      (5.4%)        1.79      (6.1%)   -0.4% ( -11% -   11%) 0.814
             MedIntervalsOrdered       26.95      (5.7%)       26.85      (6.0%)   -0.4% ( -11% -   12%) 0.844
                         LowTerm     2140.59      (3.8%)     2134.06      (3.3%)   -0.3% (  -7% -    7%) 0.786
                 LowSloppyPhrase       12.56      (2.9%)       12.53      (2.4%)   -0.2% (  -5% -    5%) 0.779
            HighIntervalsOrdered        7.50      (5.9%)        7.49      (6.3%)   -0.2% ( -11% -   12%) 0.923
                         Prefix3      239.09      (9.5%)      238.69      (8.9%)   -0.2% ( -16% -   20%) 0.954
                HighSloppyPhrase       27.88      (4.8%)       27.85      (4.1%)   -0.1% (  -8% -    9%) 0.952
             LowIntervalsOrdered       84.79      (4.1%)       84.75      (4.5%)   -0.0% (  -8% -    8%) 0.976
                      TermDTSort       99.64     (16.5%)       99.63     (17.0%)   -0.0% ( -28% -   40%) 0.999
                 MedSloppyPhrase       54.26      (4.8%)       54.30      (3.8%)    0.1% (  -8% -    9%) 0.964
                     AndHighHigh       55.59      (3.5%)       55.66      (4.4%)    0.1% (  -7% -    8%) 0.920
                      AndHighMed      116.21      (3.6%)      116.37      (4.5%)    0.1% (  -7% -    8%) 0.918
                     LowSpanNear       17.01      (3.2%)       17.03      (3.7%)    0.1% (  -6% -    7%) 0.893
                          Fuzzy2       48.58      (1.5%)       48.65      (1.5%)    0.2% (  -2% -    3%) 0.735
                       OrHighLow      263.27      (3.7%)      263.87      (2.9%)    0.2% (  -6% -    7%) 0.829
                          Fuzzy1      109.75      (1.4%)      110.05      (1.6%)    0.3% (  -2% -    3%) 0.566
                       LowPhrase      693.11      (2.6%)      695.08      (2.2%)    0.3% (  -4% -    5%) 0.712
                     MedSpanNear       78.47      (3.4%)       78.78      (4.0%)    0.4% (  -6% -    8%) 0.737
                         Respell       52.36      (1.0%)       52.57      (1.2%)    0.4% (  -1% -    2%) 0.251
                      HighPhrase       44.86      (3.9%)       45.05      (2.8%)    0.4% (  -6% -    7%) 0.693
            MedTermDayTaxoFacets       42.75      (5.5%)       42.93      (4.6%)    0.4% (  -9% -   11%) 0.786
                       MedPhrase      206.29      (2.6%)      207.19      (2.1%)    0.4% (  -4% -    5%) 0.559
          OrHighMedDayTaxoFacets       11.62      (3.6%)       11.67      (4.2%)    0.5% (  -7% -    8%) 0.709
                        PKLookup      169.21      (3.1%)      170.19      (3.1%)    0.6% (  -5% -    7%) 0.558
                          IntNRQ      854.72      (1.8%)      860.51      (1.8%)    0.7% (  -2% -    4%) 0.230
                      AndHighLow     1108.90      (2.8%)     1117.66      (2.9%)    0.8% (  -4% -    6%) 0.387
                         MedTerm     2707.67      (3.8%)     2730.24      (3.5%)    0.8% (  -6% -    8%) 0.472
                   OrNotHighHigh      850.92      (3.5%)      858.09      (3.7%)    0.8% (  -6% -    8%) 0.461
                    OrHighNotLow     1155.56      (4.2%)     1166.24      (3.7%)    0.9% (  -6% -    9%) 0.458
                        HighTerm     1308.41      (4.0%)     1321.26      (3.5%)    1.0% (  -6% -    8%) 0.412
                   OrHighNotHigh     1396.86      (3.2%)     1411.02      (2.0%)    1.0% (  -4% -    6%) 0.228
        AndHighHighDayTaxoFacets        4.67      (4.6%)        4.71      (4.5%)    1.0% (  -7% -   10%) 0.471
                    OrHighNotMed      822.86      (4.1%)      831.64      (3.9%)    1.1% (  -6% -    9%) 0.403
         AndHighMedDayTaxoFacets       86.23      (1.8%)       87.26      (1.8%)    1.2% (  -2% -    4%) 0.038
            HighTermTitleBDVSort       73.64     (18.2%)       74.80     (17.0%)    1.6% ( -28% -   44%) 0.777
                    OrNotHighMed      895.15      (3.5%)      909.34      (3.4%)    1.6% (  -5% -    8%) 0.145
                    OrNotHighLow     1178.85      (1.7%)     1200.20      (2.6%)    1.8% (  -2% -    6%) 0.010
           HighTermDayOfYearSort       82.17     (17.9%)       83.72     (22.0%)    1.9% ( -32% -   50%) 0.766
               HighTermMonthSort       78.32     (18.6%)       79.84     (17.7%)    1.9% ( -28% -   46%) 0.735
           BrowseMonthSSDVFacets       13.03      (8.5%)       13.85     (22.3%)    6.3% ( -22% -   40%) 0.235
       BrowseDayOfYearTaxoFacets       15.09      (6.1%)       20.45     (32.3%)   35.5% (  -2% -   78%) 0.000
     BrowseRandomLabelTaxoFacets       12.82      (5.5%)       17.41     (28.5%)   35.8% (   1% -   73%) 0.000
            BrowseDateTaxoFacets       15.04      (6.1%)       20.50     (32.5%)   36.3% (  -2% -   79%) 0.000
           BrowseMonthTaxoFacets       15.74      (8.1%)       25.59     (48.0%)   62.6% (   6% -  129%) 0.000

This is great! Thanks @gsmiller !

@gf2121 glad it worked out. Now I just wish we could understand what exactly was causing the regression with the other part of the change.


Resolves test failures introduced in rcl.
Before: https://ci.ros2.org/view/nightly/job/nightly_linux_release/1623/testReport/
After: https://ci.ros2.org/job/ci_linux/11705/testReport

This is...unusual. I'll take a look at the failing tests.

Closing this PR as #449 addressed the underlying issue. Thanks for the bump @jacobperron !


can we append files by DiskV

I have a request to persist buffer into local using diskv.
I wants to add a AppendStream method into https://github.com/peterbourgon/diskv/blob/master/diskv.go#L175
Why can't we add such a method before?
Is there any thinking?

diskv so far doesn't treat values as "appendable", and while I guess it could be possible to write an AppendStream method, I don't think it's needed to solve the (linked, Kubernetes) problem that you're apparently trying to solve.


Support evidence.dev as a plugin

Migrated from GitLab: https://gitlab.com/meltano/meltano/-/issues/3341
Originally created by @edgarrmondragon on 2022-03-17 17:02:14

Problem to solve
Evidence advertises as "Business Intelligence for Modern Data Teams" using just SQL and Markdown.
Target audience
Any developers that

want to integrate a BI tool with support for version control and modern front-end dependencies (Svelte) into their stack,

are already using Evidence and other MDS tools (dbt, etc.) but would benefit from managing configuration centrally as part of a Meltano project as well as creating reports in sequence:
meltano run tap-gitlab target-snowflake dbt:run evidence:build

graph LR;
t1[EL] --> t3[Transform];
t3 --> t5[Build Report Site];

Further details

It is npm-based so the container spec might need to be used.

Proposal
YAML Definition (WIP)
plugins:
  utilities:
  - name: evidence
    commands:
      build:
        working_dir: $MELTANO_PROJECT_ROOT/analyses
        command: docker build --name=meltano--evidence .
      init:
        working_dir: $MELTANO_PROJECT_ROOT/analyses
        container_spec:
          image: meltano--evidence
          command: npx degit evidence-dev/template evidence
          volumes:
          - "$MELTANO_PROJECT_ROOT/analyses/:/usr/app/"
      serve:
        container_spec:
          image: meltano--evidence
          command: npm run dev
          volumes:
          - "$MELTANO_PROJECT_ROOT/analyses/evidence:/usr/app/"

What does success look like, and how can we measure that?
Users can use Evidence in their data development workflow.
Regression test

[ ] Write adequate test cases and submit test results
[ ] Test results should be reviewed by a person from the team

Links / references

Hacker News

View 3 previous comments from the original issue on GitLab

https://hub.meltano.com/utilities/evidence


Update cloud-services-how-to-manage.md

Corrected Typos. :)

Hi @eklavyamirani, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!

This seems like a small (but important) contribution, so no contribution license agreement is required at this point. Real humans will now evaluate your PR.

TTYL, AZPRBOT;


can't transfer files from W10 to W7; keep getting "Failed: Unable to write"

tried using nitroshare to transfer files from my W10 Pro PC to 2 different W7 pro PCs with the same result; constant failures with the error status stating "Failed: Unable to write".

@Druout Check both firewalls.


Type of addresses: mix of u32 and usize.

The WRITE_SIZE, ERASE_SIZE etc are usize:
https://github.com/rust-embedded-community/embedded-storage/blob/ef207385baa1a7dd34a6778a8bd0238b42f748e9/embedded-storage-async/src/nor_flash.rs#L7
But the addresses are u32:
https://github.com/rust-embedded-community/embedded-storage/blob/ef207385baa1a7dd34a6778a8bd0238b42f748e9/embedded-storage-async/src/nor_flash.rs#L20
This makes doing math on addresses quite annoying: it needs a lot of casts. I sort of lean towards unifying this on usize, since slice lengths are usize and we can't change that.

Agreed that this should be unified, but to u32:) There are various 16 bit mcu's with more than 64k flash (e.g. MSP430F5529). They need to be able to address their entire internal flash.

The one that should have its type changed is ReadNorFlash::capacity(). It should return u32.

Would it make sense to have an associated size type? A hard u32 rules out applying this to large SD cards, but a hard u64 may be unacceptable overhead for other applications.

large SD cards are not NOR flash though, you don't have to first "erase" a sector to write. You can't write individual words in a sector either. They're best modeled as some "block device" trait where all you do is "read block", "write block". How common is it to have >4GB actual NOR flash devices?

Right; I was mixing in concerns of #23 which would AIU generalize over flash media -- SD cards behave like what is described there to haven an erase size identical to the write size (thus erasure can be implicit and AND writes are just RMW). As long as we don't do these generalizations, u32 iikely suffices.


Update Documentation

Thanks for the great plugin! I'm running Certbot v1.16.0 and it looks like the way to use the plugin as the authenticator has changed at some point – it now needs to be selected using --authenticator dns-cloudns. I also added some basic installation instructions which may help newcomers.

Thanks a ton for your contribution! I'm glad to see that the plugin is useful to someone else :)


merge cline:main to RooVetGit:main

merge cline:main to RooVetGit:main

[!IMPORTANT]
Update README.md to add a careers page link and adjust feature requests link.

Links Update in README.md:

Replaces "Docs" link with "Feature Requests" link.
Adds "We're Hiring!" link pointing to the careers page https://cline.bot/join-us.
Updates contribution section to include a link to the careers page.

This description was created by  for b5d03dd523cd8a802825c724a61853dacf009e2f. It will automatically update as commits are pushed.

Error: RooVeterinaryInc.roo-cline v2.1.18 already exists.
Error: Process completed with exit code 1.

https://github.com/RooVetGit/Roo-Cline/actions/runs/12280324047/job/34266457276
doesn't matter for this pull request

strangely, this pull request merged
but
https://github.com/RooVetGit/Roo-Cline/compare/main...cline%3Acline%3Amain
is still showing differences
and https://github.com/RooVetGit/Roo-Cline/
still says that roo cline is 1 commit behind cline
i don't understand

is it because this merge was completed via a squash commit instead of a merge commit?  otherwise, i don't understand why github and git think that roo cline is 1 commit behind cline.

is it because this merge was completed via a squash commit instead of a merge commit? otherwise, i don't understand why github and git think that roo cline is 1 commit behind cline.

Oh, that could be it - my bad

is it because this merge was completed via a squash commit instead of a merge commit? otherwise, i don't understand why github and git think that roo cline is 1 commit behind cline.

Oh, that could be it - my bad

retry at https://github.com/RooVetGit/Roo-Cline/pull/72#issue-2733465135


Low Memory SegFault in dtls

This is from latest OpenSSL 1.1.1 stable branch.
Testing with forced low memory results in a segmentation fault for failing paths of dtls1_new().
While trying to locate the proper code path for this segfault I instrumented the code to simulate
each failing path in dtls1_new() and if a malloc fails the resultant cleanup triggers a segfault.
Depending on the failing location in dtls1_new() the segfault is one of two, this one:
Program received signal SIGSEGV, Segmentation fault.
DTLS_RECORD_LAYER_clear (rl=rl@entry=0x6cc510) at ssl/record/rec_layer_d1.c:68
68	    while ((item = pqueue_pop(d->unprocessed_rcds.q)) != NULL) {
(gdb) bt
#0  DTLS_RECORD_LAYER_clear (rl=rl@entry=0x6cc510)
    at ssl/record/rec_layer_d1.c:68
#1  0x00007ffff7b6a259 in DTLS_RECORD_LAYER_free (rl=rl@entry=0x6cc510)
    at ssl/record/rec_layer_d1.c:49
#2  0x00007ffff7b67770 in dtls1_free (s=0x6cbcd0) at ssl/d1_lib.c:141
#3  0x00007ffff7b85097 in SSL_free (s=0x6cbcd0) at ssl/ssl_lib.c:1215
#4  0x00007ffff7b863b0 in SSL_new (ctx=ctx@entry=0x6c93f0)
    at ssl/ssl_lib.c:850
#5  0x0000000000442f7e in s_client_main (argc=<optimized out>,
    argv=<optimized out>) at apps/s_client.c:1980
#6  0x00000000004326dd in do_cmd (prog=prog@entry=0x6bbbe0,
    argc=argc@entry=5, argv=argv@entry=0x7fffffffde70) at apps/openssl.c:570
#7  0x000000000041e738 in main (argc=5, argv=0x7fffffffde70)
    at apps/openssl.c:189
(gdb)

or this one:
Program received signal SIGSEGV, Segmentation fault.
0x00007ffff7b676e8 in dtls1_clear_received_buffer (s=s@entry=0x6cbcd0)
    at ssl/d1_lib.c:119
119	    while ((item = pqueue_pop(s->d1->buffered_messages)) != NULL) {
(gdb) bt
#0  0x00007ffff7b676e8 in dtls1_clear_received_buffer (s=s@entry=0x6cbcd0)
    at ssl/d1_lib.c:119
#1  0x00007ffff7b67780 in dtls1_clear_queues (s=0x6cbcd0) at ssl/d1_lib.c:110
#2  dtls1_free (s=0x6cbcd0) at ssl/d1_lib.c:145
#3  0x00007ffff7b85107 in SSL_free (s=0x6cbcd0) at ssl/ssl_lib.c:1215
#4  0x00007ffff7b86420 in SSL_new (ctx=ctx@entry=0x6c93f0)
    at ssl/ssl_lib.c:850
#5  0x0000000000442f7e in s_client_main (argc=<optimized out>,
    argv=<optimized out>) at apps/s_client.c:1980
#6  0x00000000004326dd in do_cmd (prog=prog@entry=0x6bbbe0,
    argc=argc@entry=5, argv=argv@entry=0x7fffffffde70) at apps/openssl.c:570
#7  0x000000000041e738 in main (argc=5, argv=0x7fffffffde70)
    at apps/openssl.c:189
(gdb)

Thanks for your report. Please could you try out the fix in #13655 and let me know if that resolves the issue?
That fix is targeted at the master branch - but it should also apply to 1.1.1.

#13655  fixed all the failing paths, thanks !

#13655 fixed all the failing paths, thanks !

Great. Thanks for testing it.


Descriptions on Transportation Entities

There are no descriptions on any of the Transportation entities. We're very interested in the transportation specific CDM but without a description of the field, it's hard to understand if we're using it correctly.

Hi @kevine323, thanks for your recommendation. Can you check out the schema from here directly to for now? Thanks!

Hi @kevine323 , unfortunately, we don't have better documentation available now.  I would suggest you go through our support team, log a specific request like what information you are looking for, what entities you are interested in.  Our supporting team or PM should be able to provide you some level details for some request.


sanity: Uses k8s_test_harness utils

The k8s_test_harness contains commonly used util functions. This will simplify the tests.

LGTM after conflict is resolved

Done.


Implement Implicit Flow

There is currently no way to get a JWT Token using the implicit flow. Please have a look at my PR implementing it.

Thanks. I just reviewed the PR.

Included in version v0.5.3. Thanks for contributing.


AutoTeleportActionButton: refactoring dependencies.

isReady and the chain of computed values it depends on is implemented in useAutoTeleportTransitionDetails. This refactor makes the code more readable by directly importing into AutoTeleportActionButton its dependencies from their source file.
This refactor didn't follow as stated in #9695. This is because of the explanation stated above as regards isReady, which also applies same to some other dependencies.  Secondly, it's not very clear if isReady can be seperated into its own composable as it depends on the long chain of computed values before it in the useAutoTeleportTransitionDetails file.
I would appreciate review on this PR, if there is a misunderstanding as regards this issue.
Thank you for your contribution to the KodaDot - One Stop Shop for Polkadot NFTs.
👇 __ Let's make a quick check before the contribution.
PR Type

[ ] Bugfix
[ ] Feature
[x] Refactoring

Context

[x] Closes #9695
[ ] Requires deployment <snek/rubick/worker>

Before submitting pull request, please make sure:

[x] My contribution builds clean without any errors or warnings
[ ] I've merged recent default branch -- main and I've no conflicts
[x] I've tried to respect high code quality standards
[x] I've didn't break any original functionality

Optional

[ ] I've tested it at </ksm/collection>
[ ] I've tested PR on mobile
[ ] I've written unit tests 🧪
[ ] I've found edge cases

Did your issue had any of the "$" label on it?

[x] Fill up your DOT address: Payout

it's not very clear if isReady can be seperated into its own composable as it depends on the long chain of computed values before it in the useAutoTeleportTransitionDetails file

this long chain is exactly the issue to be solved
as i said, this is not an easy issue and @hassnian is probably the best person for the task

this long chain is exactly the issue to be solved

Oh, thanks for the clarification.

@daiagi I'm not sure about this, even though code wise this a step in the right direction
if this is merged like it is now , since useAutoTeleportTransitionDetails is now being called in 3 places

AutoTeleportActionButton.vue
useAutoTeleport.ts
useAutoTeleportTransition.ts

this is unnecessarily re-fetching data that we already have, like balances and tx fees(both action and telpeort)
opening the listing cart

interacting with the listing cart , refetches actions tx fees 3 times , isn't this also a step backwards?

wdyt?

can be reopen if internal dev have any response regarding hassnian comment


Removerequest from @graphql-codegen/cli.

Please tell me if I am wrong, but it does not seem to be used.  Additionally, it is deprecated and Yarn gives a warning to that effect when installing @graphql-codegen/cli.

You are right, great catch.
thank you @wtgtybhertgeghgtwtg

You are right, great catch.
thank you @wtgtybhertgeghgtwtg

Thank you for merging.

Thank you for merging.


Feat: Add 'thread' field to messages.

Summary
Parse and use the thread field on messages, as documented here.
Checklist

If code changes were made then they have been tested.
I have updated the documentation to reflect the changes.
This PR adds something new (e.g. new method or parameters).

I have tested it, and the branch this pull request comes from is currently being used in production (and actively using the feature) without issues.

@Artemis21 Any updates?

@Skelmis addressed reviews...

Returns None in a regular channel as well as a thread
with code:
@commands.command()
async def test(self, ctx):
    await ctx.send(str(ctx.message.thread))

@koala9712

Returns None in a regular channel as well as a thread with code:

To get the channel or thread a message was sent in, use Message.channel. This attribute is for determining which if any thread was created from this message, as documented.

This attribute is for determining which if any thread was created from this message, as documented.

So... this should theoretically have...?


Questions about the Relavance Matrix

First of all, thanks for the great work! It's always nice with approaches that are not too complicated to implement but still yield impressive results!
I have a question regarding the final Relevancy matrix. In the code for the different models I see that before the relevancy matrix is sent back, we pick out the scores for the first token and then remove the "self" relevancy score for this token R[0, 1:]. This is done since in this case this token represents the CLS token, right? And the "self" relevancy is removed since we only want to keep the entries for the tokens from the original data, is that correct?

I am using your approach for a GPT model and it has been working well. I initially tested it on a toy problem where the model had to predict whether a certain word was present in the input or not (classification was done on a SEP token at the end of the sequence) and the relevancy score properly highlighted the words if they were present. I am now  trying to generate embeddings for another task by extracting the final SEP token and using it as embedding. I have tried a similar approach to the positive and negative perturbation from your paper and noticed that I can remove up to 90% of the tokens from the input and still get very similar final embeddings. My issue is now how to interpret these tokens that get high relevance scores. For some reason the token at position 0 almost always get the highest score and in this case this token is not the CLS token, instead it can be any token from the vocabulary. So this made me question if there perhaps were any other reasons why R[0, 1:] is performed in your code?

Hi @Paandaman, thanks for your interest in our work!
You are absolutely correct, the removal of relevance score of the first token is since the first token is the CLS which is a dummy token and does not represent real data.
In general, we mention in the paper that since the relevance matrix is initialized with the identity for self attention, the scores of the diagonal tend to have a larger value than the other relevance scores, but in this case it has no effect since we remove R[0,0].
I’m glad to hear our approach proved to be beneficial for GPT! If you’re willing to, I’d be happy to add your code to our repo so other people can use it and get inspired by your work :) (you can open a pull request and I’ll merge your code to our repo).
Regarding your issue, it is interesting, I haven’t seen this phenomenon in any other case.
I have seen that when you use special tokens such as CLS, SEP they sometimes have a higher relevance score than other tokens (perhaps because they appear in every input), but this phenomenon is not one I’m familiar with.
I would suggest to try to limit the context expansion and see what happens as I suggested in this issue regarding CLIP and you can also refer to our colab to see an example of context expansion for CLIP.
I hope this helps, and I hope you’ll share your code for GPT as well :)
Best,
Hila.

Thank you for the informative reply!
That makes a lot of sense, thanks for clarifying it! I would love to share the GPT code but it's currently written together with some  proprietary code so I will have to create a stand-alone version of it first. I will try to do it!
Interesting indeed! I am using a lot of SEP tokens to separate different parts of the text and I have also noticed that these SEP tokens often get high scores. I was thinking that maybe the model is aggregating the information from the sequence of tokens into these SEP tokens in the same way as BERT uses the CLS token, but since there are several SEP tokens maybe it can store different parts of the information in different SEP tokens. If this is the case it might make sense that they get a high score!
Thanks for the suggestion, I tried limiting the context expansion but did not actually see any difference, the first token still gets the highest scores! I will have to try and dig deeper into this to see if it makes sense or not!
Many thanks once again!
Best regards,
Max


Changelog: v3.0.0

Thank you @manics for getting this done!!

For reference, these are the bump PRs in the jupyterhub org:

https://github.com/jupyterhub/docker-image-cleaner/pull/145
https://github.com/jupyterhub/jupyterhub/pull/4639
https://github.com/jupyterhub/configurable-http-proxy/pull/513


branch-3.0 fix test_table_with_buckets due to use old optimizer

Proposed changes
Issue Number: close #xxx

run buildall


MyComplex vs. Complex

Left over from #1122:

But why not use Complex directly like the other packages already do?

Utilize the record Complex (and defined operators) instead of defining own record Modelica.Media.Air.ReferenceMoistAir.Utilities.Ice09_Utilities.Basic.MyComplex and functions MyComplexF.* to avoid redundancy.
Note, that there is Modelica.Media.Air.ReferenceMoistAir.Utilities.Ice09_Utilities.Basic.MyComplexF.'log' but no equivalent function in operator record Complex.

hacktoberfest

Why do you direct this question to Martin and me?
Why don't ask the library officers of Modelica.Media?

Conceptually, this is a trivial fix, but in practice it is quite tedious and it is easy to mess up. Furthermore, I don't see any test model in the MSL to check if the results are the same.
@wischhusen would you be available to take care of this issue (possibly also for the next minor release)?

@wischhusen would you be available to take care of this issue (possibly also for the next minor release)?

@wischhusen Please let me know if you can manage for MSL 4.0.0?

@wischhusen Please let me know if you can manage for MSL 4.0.0?

I prepared #3131 for it.