DB::afterCommit callbacks aren't run in tests when RefreshDatabase trait is used

Laravel Version: v8.21.0
PHP Version: 7.4.11
Database Driver & Version: psql (PostgreSQL) 12.4 (Ubuntu 12.4-1.pgdg20.04+1) - It doesn't depend on DB driver.

Description:
DB::afterCommit callbacks aren't run when RefreshDatabase trait is used because ManagesTransactions trait only commits when all transactions are executed, but RefreshDatabase uses a transaction. Then the callbacks aren't called.
Steps To Reproduce:
I've created a reproduction repo: https://github.com/mariomka/laravel-after-commit-repro
There are two tests, one with RefreshDatabase (it fails) and one without RefreshDatabase (it passes): https://github.com/mariomka/laravel-after-commit-repro/tree/master/tests
Both tests run the same "action": https://github.com/mariomka/laravel-after-commit-repro/blob/master/app/AfterCommitAction.php
Step by step reproduction:
git clone git@github.com:mariomka/laravel-after-commit-repro.git
cd laravel-after-commit-repro
cp .env.example .env
composer install
php vendor/bin/homestead make
# Enable postgresql in Homestead.yaml
vagrant up
vagrant ssh
cd code
php vendor/bin/phpunit

I've installed your repo locally and both tests pass for me. Can you first please try one of the support channels below? If you can actually identify this as a bug, feel free to report back and I'll gladly help you out and re-open this issue.

Laracasts Forums
Laravel.io Forums
StackOverflow
Discord
Larachat
IRC

Thanks!

I've installed your repo locally and both tests pass for me. Can you first please try one of the support channels below? If you can actually identify this as a bug, feel free to report back and I'll gladly help you out and re-open this issue.

Laracasts Forums
Laravel.io Forums
StackOverflow
Discord
Larachat
IRC

Thanks!

@driesvints have you used MySQL or Postgres driver? with SQLite runs because I think it doesn't run transactions

@driesvints have you used MySQL or Postgres driver? with SQLite runs because I think it doesn't run transactions

Ah, you said:

It doesn't depend on DB driver.

So I tried with SQLite indeed.
But still, can you maybe try a support channel first? DB::afterCommit isn't documented atm and I suspect it might just be a misplacement. I'm not 100% sure myself.

Ah, you said:

It doesn't depend on DB driver.

So I tried with SQLite indeed.
But still, can you maybe try a support channel first? DB::afterCommit isn't documented atm and I suspect it might just be a misplacement. I'm not 100% sure myself.

@driesvints  I'm Sorry, I didn't think about SQLite.
I'm sure that this is not intended behavior, it works perfectly outside the tests. The issue only occurs in tests.
Checkout this fragment: https://github.com/laravel/framework/blob/374e9565b2718c549f329af5c137e84a7d753a28/src/Illuminate/Database/Concerns/ManagesTransactions.php#L43

It only commits when there is only one transaction, if I run the action directly without tests it runs because there is only one transaction but for testing, I need to use RefreshDatabase and it uses a transaction.
There is the line where DatabaseTransactionsManager runs callbacks when a transaction is committed: https://github.com/laravel/framework/blob/374e9565b2718c549f329af5c137e84a7d753a28/src/Illuminate/Database/DatabaseTransactionsManager.php#L59

@driesvints  I'm Sorry, I didn't think about SQLite.
I'm sure that this is not intended behavior, it works perfectly outside the tests. The issue only occurs in tests.
Checkout this fragment: https://github.com/laravel/framework/blob/374e9565b2718c549f329af5c137e84a7d753a28/src/Illuminate/Database/Concerns/ManagesTransactions.php#L43

It only commits when there is only one transaction, if I run the action directly without tests it runs because there is only one transaction but for testing, I need to use RefreshDatabase and it uses a transaction.
There is the line where DatabaseTransactionsManager runs callbacks when a transaction is committed: https://github.com/laravel/framework/blob/374e9565b2718c549f329af5c137e84a7d753a28/src/Illuminate/Database/DatabaseTransactionsManager.php#L59

@mariomka can you maybe try a support channel first? If you can confirm the bug feel free to report back.

@mariomka can you maybe try a support channel first? If you can confirm the bug feel free to report back.

But still, can you maybe try a support channel first? DB::afterCommit isn't documented atm and I suspect it might just be a misplacement. I'm not 100% sure myself.

@driesvints Thank you for your time but I don't know what you want me to ask the community. Seeing some comments on #35373 (https://github.com/laravel/framework/pull/35373#issuecomment-734333318, https://github.com/laravel/framework/pull/35373#issuecomment-734396793, https://github.com/laravel/framework/pull/35373#issuecomment-734730916) DB::afterCommit should be inside a transaction, then when the transaction is committed callback is called. In fact, it works fine. However, it is untestable when the test needs the RefreshDatabase trait (common when you are using a DB in tests).

But still, can you maybe try a support channel first? DB::afterCommit isn't documented atm and I suspect it might just be a misplacement. I'm not 100% sure myself.

@driesvints Thank you for your time but I don't know what you want me to ask the community. Seeing some comments on #35373 (https://github.com/laravel/framework/pull/35373#issuecomment-734333318, https://github.com/laravel/framework/pull/35373#issuecomment-734396793, https://github.com/laravel/framework/pull/35373#issuecomment-734730916) DB::afterCommit should be inside a transaction, then when the transaction is committed callback is called. In fact, it works fine. However, it is untestable when the test needs the RefreshDatabase trait (common when you are using a DB in tests).

Wait.
Doesn't it support nested transactions?
Apologies, because I use https://github.com/fntneves/laravel-transactional-events and the transaction opened via the trait (to wrap a test in transactions) does not count as influencing "transactionable ware" code triggered in nested transactions.
(btw I use this together with Postgres with great success).
It would be irritating to me to test a part of an application, which makes use of such a feature and then not knowing whether it was supposed to be executed or not just because the test is wrapped in a transaction. 🤔

@mariomka I have the same issue when using DB::transaction, but not when handling the transaction manually with DB::beginTransaction() and DB::commit(). Do you have the answer for this?

anyone find a solution for this? i am trying to test a job that creates some models in an DB transaction... and i am trying to test that some jobs are dispatched in the model observer... we are using afterCommit = true in the observer... and they are not being fired

@simonmaass
I just found this thread, and I didn't see an answer either. I went ahead and solved it myself. For everyone else coming here in the future, here's how I resolved this issue.
The way Laravel handles the "after commit" functionality is by the new DatabaseTransactionsManager introduced in Laravel 8.x. All forms of "after commit" logic funnel into the addCallback method:
public function addCallback($callback)
{
    if ($current = $this->transactions->last()) {
        return $current->addCallback($callback);
    }

    call_user_func($callback);
}

As seen above, if the connection currently has a transaction, all of the callbacks are bound to the ending of that transaction (which in the case of testing, has the scenario where it may never commit).
I overrode the DatabaseTransactionsManager like so:
<?php

namespace App\Providers;

use App\Testing\DatabaseTransactionsManager;
use Illuminate\Support\ServiceProvider;

class TestServiceProvider extends ServiceProvider
{
    /**
     * Register the service provider.
     *
     * @return void
     */
    public function register()
    {
        if (! $this->app->runningUnitTests()) {
            return;
        }

        $this->registerDatabaseTransactionsManager();
    }

    /**
     * Registers the database transaction manager.
     */
    protected function registerDatabaseTransactionsManager()
    {
        // Our automated tests wrap everything inside of a database
        // transactions, which means the "afterCommit" behavior
        // is never invoked. We'll need to tweak this a bit.

        $this->app->singleton('db.transactions', function () {
            return new DatabaseTransactionsManager;
        });
    }
}

This binds my own DatabaseTransactionsManager in place of the original one. Here's what that looks like:
<?php

namespace App\Testing;

use Illuminate\Database\DatabaseTransactionsManager as Manager;

class DatabaseTransactionsManager extends Manager
{
    /**
     * The baseline transaction depth.
     */
    protected $baselineDepth = 0;

    /**
     * Register a transaction callback.
     *
     * @param  callable  $callback
     *
     * @return void
     */
    public function addCallback($callback)
    {
        if ($this->transactions->count() > $this->baselineDepth) {
            return parent::addCallback($callback);
        }

        call_user_func($callback);
    }

    /**
     * Updates the baseline transaction depth to the current transaction depth.
     *
     * @return $this
     */
    public function captureTransactionDepth()
    {
        $this->baselineDepth = $this->transactions->count();

        return $this;
    }
}

The new manager allows you to capture a "baseline depth", of which, upon reaching it, all callbacks are immediately fired, rather than deferred to a transaction committal.
Now you just need to capture the transaction depth in the correct place, which can be done inside of your test:
public function setUp(): void
{
    parent::setUp();

    $this->app->make('db.transactions')->captureTransactionDepth();
}

And with that all in place, any "after commit" calls on top of the refresh transactions will be called immediately. If you have a transaction within your test, the callback will be deferred until the end of that specific transaction, but will still be called at some point in your test.

@tylernathanreed your solution only work if only one transaction is perform in your application code. I implemented your solution, and with following code in my controller :
                DB::transaction(function () {
                   DB::afterCommit(function () { dump('This is executed'); });
                });
                DB::transaction(function () {});
                DB::transaction(function () {});
                DB::transaction(function () {
                    DB::afterCommit(function () { dd('This is not'); });
                });

it doesn't dd()
Because, each time you start a transaction with DB::transaction(function () {}); it increment the $this->transactions->count().
But, there is no code that can decrement it. Transaction cleaning is done when calling commit() on the DatabaseTransactionsManager which is only called when the very first level transaction is committed. (Check code here : https://github.com/laravel/framework/blob/b9203fca96960ef9cd8860cb4ec99d1279353a8d/src/Illuminate/Database/Concerns/ManagesTransactions.php#L48-L52) (That transaction in our concerns would be the one that hold the database refresh. or here https://github.com/laravel/framework/blob/b9203fca96960ef9cd8860cb4ec99d1279353a8d/src/Illuminate/Database/Concerns/ManagesTransactions.php#L194-L198

@driesvints Can you reconsider re-opening this issue ? It looks we are several people (including me) experiencing this issue with non-sqlite dd driver.

Sure. Appreciating PR's to solve this.

+1 subscribing for the solution.
For now this workaround works:
public bool $afterCommit = true;

public function __construct()
{
        if (app()->runningUnitTests()) {
            $this->afterCommit = false;
        }
}

Potential fix https://github.com/laravel/framework/pull/41782
Would appreciate feedback / testing.

I truly appreciate the kind words @shadoWalker89 but we're not going to change the way we work, sorry.

I'm sorry @BrandonSurowiec but 8.x is closed for bug fixes. You should upgrade as soon as you can.


Recommend adding a link to the OSCON EU video

I'd recommend adding a link to the OSCON EU video where Doug announces the Seif project, so that people coming here via seif.place can get an overview of what the vision for the project is.

Video link on the Wiki.


skip_misc extras

Compile & check before you accept the pull.
This might not work.

No idea why that is conflicting, but the only change you should need would be commits 4d0bacf and a850f24

radar works with these commit

So we only need 4d0bacf and a850f24?
Are 8122a9f 0c238c1 f651b6a just optimizations or was that just part of the troubleshooting process?
Thanks!

So we only need 4d0bacf and a850f24?
Are 8122a9f 0c238c1 f651b6a just optimizations or was that just part of the troubleshooting process?
Thanks!

I applied this change manually in 2118feb4357f73cb2477dbb34563015ba8776d77. Thanks a lot!

I applied this change manually in 2118feb4357f73cb2477dbb34563015ba8776d77. Thanks a lot! Let me know if there are any more issues!


Flowing text out of div in subcircuit

Fixes https://github.com/CircuitVerse/CircuitVerse/issues/899
Describe the changes you have made in this pr -
Prevented the overflowing of div when the circuit name is long in subcircuit
Screenshots of the changes (If any) -
Before:

After:

Coverage increased (+0.08%) to 84.347% when pulling 59ad5c39b5d311a1f184a40f9824f0035c4cdb6d on yash15601:flowing-out-div-subcircuit into 16e1b86d3474f30c47f84465750245d45c88fabc on CircuitVerse:master.

Working on part one @Shivansh2407

@tachyons Review Please

@satu0king Please review

@tachyons thank you


What is the best way to override ENV variables set by modules?

I'm trying to override HISTSIZE but it doesn't seem to be working, I tried in zprofile, zshenv & zshrc and it's always is set to 10000 which comes from here
So what is the best way to do this?

Modify it in place or at the bottom of zshrc.


fix: DH-16865: Fix spinner on empty partitions list

Closes #1904

This PR should fix the issue with partitioned tables with no partitions in DHE. But I think I found an edge case with widgets of type VariableType.PARTITIONEDTABLE without partitions in DHC, where a null returned from this.partitionedTable.getTable(partitions) sends IrisGrid into an infinite initState loop.

Closing PR since this bug has been resolved in https://github.com/deephaven/web-client-ui/pull/2110


Update rook version to get OSD placement fixes

We update to a version of rook that has placement fixes for OSD.
Also, we now use the ceph pacific latest images and the stretch cluster feature has some fixes which aren't available in the previous image we were using. This means that we will have to live with a moving target of ceph daemon.

@jarrpa Please take a look.

/cherrypick release-4.7


Gradle 8.1.1: Resolution of the configuration ... was attempted from a context different than the project context

I get lots and lots of message of this type for varying configurations:
Resolution of the configuration :jsource:osis:runtime:server:spotbugsPlugins was attempted from a context different than the project context. Have a look at the documentation to understand why this is a problem and how it can be resolved. This behavior has been deprecated. This will fail with an error in Gradle 9.0. See https://docs.gradle.org/8.1.1/userguide/viewing_debugging_dependencies.html#sub:resolving-unsafe-configuration-resolution-errors for more details
After a little drilldown I realize that this is probably not caused by the spotbugs plugin but the dependencycheck plugin, because the message is created FOR a spotbugs configuration, but also when executing just the dependencyCheckAggregate Task.

Could you share a mcve project to reproduce this issue? Thanks for your report!

I could not create a MCVE without dependency check. I am absolutely sure now, that depdenceny check is the culprit!
However, while trying to create the MCVE I stumbled about this:
plugins {
  id 'com.github.spotbugs' version '5.0.14'
}

apply plugin: 'java'

repositories {
  mavenCentral()
}

spotbugsMain {
/* Generates a warning:

The org.gradle.util.ClosureBackedAction type has been deprecated. This is scheduled to be removed in Gradle 9.0. Consult the upgrading guide for further information: https://docs.gradle.org/8.2.1/userguide/upgrading_version_7.html#org_gradle_util_reports_deprecations
        at build_a3rrfjmmvkuarj48u4gn03nvt$_run_closure2.doCall(C:\Users\MatthiasKraaz\temp\spotbugs_mcve\build.gradle:12)
        (Run with --stacktrace to get the full stack trace of this deprecation warning.)
*/
  reports {
    xml.enabled = true
  }
/* Generates no warning:

  configure(reports) {
    xml.enabled = true
  }
*/
}

Not sure if this concerns you somehow. If it does, I can supply a MCVE for this...

The org.gradle.util.ClosureBackedAction type has been deprecated

It is a known issue, #616, waiting for action on the Gradle side.
I think it is not related to the problem you met firstly. 🤔


[DO NOT MERGE OR CLOSE][fix][docs] Update typo of GitHub

Signed-off-by: Max Xu<EMAIL_ADDRESS>
(If this PR fixes a github issue, please add Fixes #<xyz>.)
Fixes #
(or if this PR is one task of a github issue, please add Master Issue: #<xyz> to link to the master issue.)
Master Issue: #
Motivation
Explain here the context, and why you're making that change. What is the problem you're trying to solve.
Modifications
Describe the modifications you've done.
Verifying this change

[ ] Make sure that the change passes the CI checks.

(Please pick either of the following options)
This change is a trivial rework / code cleanup without any test coverage.
(or)
This change is already covered by existing tests, such as (please describe tests).
(or)
This change added tests and can be verified as follows:
(example:)

Added integration tests for end-to-end deployment with large payloads (10MB)
Extended integration test for recovery after broker failure

Does this pull request potentially affect one of the following parts:
If yes was chosen, please highlight the changes

Dependencies (does it add or upgrade a dependency): (yes / no)
The public API: (yes / no)
The schema: (yes / no / don't know)
The default values of configurations: (yes / no)
The wire protocol: (yes / no)
The rest endpoints: (yes / no)
The admin cli options: (yes / no)
Anything that affects deployment: (yes / no / don't know)

Documentation
Check the box below or label this PR directly.
Need to update docs?

[ ] doc-required
(Your PR needs to update docs and you will update later)

[ ] doc-not-needed
(Please explain why)

[ ] doc
(Your PR contains doc changes)

[x] doc-complete
(Docs have been already added)

I think that the community language should be describing the whole Pulsar community as people on the Mailing List, GitHub, and Slack - everyone in the community can engage in conversation in all of these locations and should be encouraged to do so.

Close this DocBot demo PR.


[BUG] - Health glitching mid-fight

What happened?
As far as i can remember, it only happened on the hunting road, but the monster's hp/total hp would be around 2 digits lower than it's suppose to be, or it would just be 0/1 HP
What version of the program are you using?
0.22.0
What version of the game are you using?
Non High-Grade Edition

ok so turns out it also happens outside of the road


Add isTagDirty();

tag(true) doesn't work for me because of --abbrev=0.
This pr adds a function which returns if the tag is dirty.

What do you think?

Looks good; I'll have some time tomorrow to run through some local tests and merge. Thanks for the PR!

Thanks for your feedback 👍

@kurttheviking could you already test this?

@jantimon Can you review my pull request in the meantime? :)

@kurttheviking could you please test and merge this merge request?

Wow thanks 👍


Manage syslog

https://docs.opnsense.org/development/api/core/syslog.html

Feature added in https://github.com/andeman/opn-cli/pull/25


Add multi gpus capacity for transcription

This PR:

Remove bad async design and blocking operations.
Allow multi-GPUs for transcription

WARNING: NEEDS TO BE DONE

We should handle multi-GPU cases when we have to load a specific whisper model for extra languages. I suggest introducing a sort of index for that will load the model and unload the model only for 1 GPU, and not all the GPUs...
Nemo is the bottleneck atm and it should also use an index to load multiple diarization models

I implemented a GPU index for each service based on the GPUService class that handles the acquisition and the release of the GPU (to avoid 2 jobs trying to use the same GPU at the exact same moment).
I'm actually facing a weird issue with ctranslate2 inference: RuntimeError: CUDA: invalid arguments


package 'data_service' not found and Arduino issue

Hi,
I just finished assembling the nice car following your guide. It is awesome! Thank you!
When I try to test the car, at the section where I am supposed to type

rosrun data_service service.py

It shows that

[rospack] Error: package 'data_service' not found

Do you know what is the issue here?
I tried to search online and it seems that it is with version issue.
If so I have confronted similar issues when connecting to Arduino I thinks. When I type

rosrun rosserial_python serial_node.py

It shows the following after a while

[ERROR] [WallTime:<PHONE_NUMBER>.700273] Unable to sync with device; possible link problem or link software version mismatch such as hydro rosserial_python with groovy Arduino

Did I do anything wrong? Or can you tell me what is the issue here? I have done a git pull before I did all above.
By the way, when ever I open a terminal it always shows

bash: /home/odroid/barc/workspace/devel/setup.bash: No such file or directory

I can find that bash file anywhere.
Thank you in advance!

Hi,
I just updated the rebuild_system script. Can you pull from master?
then open another terminal and run the command rebuild_system
it should rebuild all the packages, then you should be able to find data_service

Hi Jon,
Thank you for your help!
 I did try this. I `git pull` under ~/barc folder and then do a `rebuild_system`  right after opening the second terminal. I think now it can find data_service. Also, the message

bash: /home/odroid/barc/workspace/devel/setup.bash: No such file or directory

disappear and now every time I open a terminal, it is normal.
But I got some messages saying that it

Traceback (most recent call last):
File "/home/odroid/barc/workspace/src/data_service/scripts/service.py", line 28, in
from data_service.srv import *
ImportError: No module named srv

Here is a screenshot

In the meantime I still have the Arduino communication issue, the same message still pops up.When I type

rosrun rosserial_python serial_node.py

It shows the following after a while

[ERROR] [WallTime:<PHONE_NUMBER>.700273] Unable to sync with device; possible link problem or link software version mismatch such as hydro rosserial_python with groovy Arduino

Maybe this is still a version issue?
Thank you in advance!

hmmm,

are you running this code on the master branch? is the branch up-to-date

git branch
git checkout master
git pull

I'm not sure why you have that issue
Are you using the odroid image from the link we provided in the documentation?
On your machine, make the packages are under the workspace/src folder (e.g. barc, barc_gui, data_service, etc.)
Then try running catkin_make or rebulid_system again

I've seen that error in the past, try disconnectinf the arduino from the RC, and try running it with only the odroid and arduino connect, maybe one of the ports is causing issues. Or perhaps something is connected incorrectly
Also, perhaps the arduino is damaged (we also had that happen in the past)

Hi Jon,
Thank you for the reply.
For 1:
I think I tried a couple of times. I pulled and updated the folder and I checked the files. Under workspace/src, I can find everything you mentioned. The image I have is for sure the one I got from the website. As the assembly instruction suggested, I did the check the MD5 hash value to make sure the file I downloaded was correct.
I did what you suggested. This time when I rebuild, there is a long process, which is different from what I see before. This time I can proceed to the third step, as you can see, when I do

bash ~/barc/scripts/register_cloud.sh

Something else is missing. I am not sure what does that mean.
I also try to proceed to the next few step, it seems that the GUI I open shows nothing, an empty window. I think this may due to the failure in the previous step.

for the Arduino unit. I also tried a couple of times on different port. I also tried on my other computers to make sure the Arduino is not damaged. So far I have the difficulty on other machines as well so I am buying another one and give it another try.

Thank you any way!

it seems python does not have the boto3 package
open a terminal, and try pip install boto3 (actually, do this for any package that is missing)
let me know if that resolves the issue

Hi Jon,
Thank you very much! I think it is working now! I can see the screen as the assembly instruction. One quick question I have is for the cloud registration, do I have to do this every time I run? Or I just need to do it once for one day run?
For the arduino part. I have two Arduino nano now, both of them works since I can upload several examples in the library, and they all can be uploaded to both Arduino.  I tried only connecting the arduino to the odroid and unplug all the port with camera, ultrasound etc. The same error still appear for both issues.

Do you think it could be a version issue? If it was a port issue, can I write some code and try to detect it?
By the way, in the arduino schematics
There are two ports I am not sure where to connect, the ESC D10 and Servo for D11. I can only find the throttle and steering from the RC receiver. This is not mentioned in assembly instruction.
And I think there might be something missing in the files I should have still. When I test the camera, I am not sure whether I launch the right camera.launch file. What I run was the sensor test and the barc_cam.
I got a warning from both the GUI and terminal.

I also run the arduino test from here and I got the same error message in the GUI, which I think is due to the same issue.
Do you think this could be due to the rqt_launch issue mentioned at the end of assembly instruction?
Thank you in advance for suggestion/solutions/comments of any kind!

I think on a potential issue is that I don't have the head file "EnableInterrupt.h" in the library, which is called by the arduino ino file .
In the arduino markdown it is written that

Encoders now use software interrupts (formerly hardware interrupts. Install the necessary library by running source ~/barc/scripts/install_enable_interrupt.sh

I tried but I don't have this "install_enable_interrupt.sh" in scripts either. I  git pull before my action and I cannot find this file still.

Actually, sorry for the confusion, I realize we included code to install that script inside the install_apps.sh script.
If you open the file and look inside, you should see the commands inside there
can you try running
source ~/barc/scripts/install_apps.sh
that should install the enable interrupts library for the arduino
D7 and D8 are for replaying remote control commands (steering and throttle) to the Arduino.
This is useful, for example, if you are manually controlling the RC with a remote control, but you want to modify the user inputs.
We transmit the user's input control signals, captured by the receiver unit, to arduino ports D7 and D8.
I haven't developed any programs for this yet, but it should be useful functionality
I'll add some pictures to the documentation
Also, without trying to run a program, can you run the following
roscore
rosrun rosserial_python serial_node.py

This should run only the ROS serial communication node with between the odroid and the arduino.
Do you still get the same output?


vanishPlayer API does not work

java.lang.NullPointerException: Cannot invoke "String.length()" because "teamName" is null
at net.william278.velocitab.packet.UpdateTeamsPacket.removeTeam(UpdateTeamsPacket.java:120)
at net.william278.velocitab.packet.ScoreboardManager.vanishPlayer(ScoreboardManager.java:89)
at net.william278.velocitab.vanish.VanishManager.lambda$vanishPlayer$0(VanishManager.java:63)
at java.base/java.util.Optional.ifPresent(Optional.java:178)
at net.william278.velocitab.vanish.VanishManager.vanishPlayer(VanishManager.java:63)
at net.william278.velocitab.api.VelocitabAPI.vanishPlayer(VelocitabAPI.java:154)
       @Subscribe
public void onJoin(ServerPostConnectEvent event) {
    	plugin.getVelocitabAPI().vanishPlayer(event.getPlayer());
}

Hi, maybe this is a syncronization issue as you are vanishing the player before he can handled by velocitab. I'm adding an event you can listen to.

@TwinkleToes777
Hi, could you try the latest build of velocitab with this new api version. Use the new PlayerAddedToTabEvent to listen when a player is loaded and then use the vanish api.

how to get player in this new event? I tried event.getPlayer() and event.getPlayer().getPlayer(), all throw compile errors

Could you send the stack trace pls and the code you are using.

Could you send the stack trace pls and the code you are using.

There is no stacktrace as the code does not compile, it throws compile-time error: "no symbol found" on event.getPlayer() inside PlayerAddedToTabEvent, the event itself compiles correctly, just event.getPlayer() does not

Try with player(), check rhe docs
here

@alexdev03 oh, I didn't notice that. I'd prefer us to not use lombok in this class and add a getPlayer method

@TwinkleToes777 have you tried, using the latest API snapshot, calling #getPlayer()? you'll need to install the latest alpha on your proxy from modrinth. let me know.

@TwinkleToes777 have you tried, using the latest API snapshot, calling #getTabPlayer()? you'll need to install the latest alpha on your proxy from modrinth. let me know.

No error, no vanish.

@TwinkleToes777
In order to vanish a player you need to provide a vanish integration. 1.3 otherwise velocitab will use the default one.
Vanish Integration is a system to provide velocitab a way to check if a player can see another player.


Implement GetBufferData() in SetupBones Task.

fix for morph.
Implementing these changes will prevent nullrefs the GLTFSceneImporter when importing certain models.
To Test:

download my test model:  https://drive.google.com/file/d/1ot3huGT49jf4q2C-9FnSpsUECGBlD-sn/view?usp=sharing

Try to import the model in master, it should give a nullreff.

Try to import the model after you applied my changes, should work.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


add a new parameter for unobserved magnitudes

There's already a parameter here that converts a non-detection sentinel value into the 99 value that BPZ expects. This adds a new parameter unobserved_val that similarly converts a sentinel for undetected objects to the -99 value that it uses.

Maybe we could do this in a more generic way?

Do you mean at a higher level somewhere, above BPZ? Or something else?


Contribute button on footer needs to point to the guide

This is a...

[ ] Feature Request
[x] Bug Report

Problem:
The contribute button on the footer points to the main k/k repo instead of a landing page for contributing.
Proposed Solution:
It should point to https://git.k8s.io/community/contributors/guide.
Page to Update:
https://kubernetes.io/...
all - the button is located in the footer.html
I'm sending in the fix/PR shortly.

@parispittman The issue is fixed with your change, could you close it, please?

/assign

/close
Closing the issue, it's fixed


Linter setting doesn't work with the newest linter update

I get this error
[Linter] Invalid Linter received
These issues were encountered while registering a Linter named 'Unknown'
  • Linter.name must be a string

Seems like a relatively easy fix.

Uh... I'm tempted to say it's a linter package issue. First it added name
field, now it's apparently mandatory, all without changing protocol version
despite breaking changes. However, I'm in the process of rewriting and
rewiring things, which should eventually fix this as well. I'll try to get
it out faster. In the meantime, you could complain to @steelbrain if you're
so inclined :)
чт, 16 мар. 2017 г., 12:31 Luka Horvat<EMAIL_ADDRESS>I get this error
[Linter] Invalid Linter received
These issues were encountered while registering a Linter named 'Unknown'
• Linter.name must be a string
Seems like a relatively easy fix.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/atom-haskell/haskell-ghc-mod/issues/208, or mute the
thread
https://github.com/notifications/unsubscribe-auth/AG8EZr7IG_FN1GFgSsuUoY8Del4WAfmYks5rmQFzgaJpZM4MfCaD
.

Hi @lierdakil
Linter didn't break it while in v1, we bumped major so it was supposed to have some incompatibilities. Anyways instead of forcing the providers, I'll try to make linter as compatible as possible
I've opened https://github.com/steelbrain/linter/pull/1371 to fix this

Strangely, I've got, instead of an error, this when using other providers that didn't set name:

That's the expected behavior, it has to fill in something for the provider, so it sets unknown for those who don't specify it

@steelbrain, while you bumped package version, service version is still 1.0.0, so the general expectation is that it should behave the same =\

Although Linter has had two bugfix updates and is now on 2.1.2, the problem still persists. :-/

@quchen, are you referring to the error or to the "unknown" thingy?

Anyway, since planned updates take way longer than expected, I've released a patch that should fix this on our end.

Using Linter 2.1.2:

Updated to the just released new haskell-ghc-mod 1.19.8, I now get

Well, that's different =\
I'm not sure where the problem lies exactly, but there are a few cases where result might not be an Array (or rather, Promise<Array>), but rather undefined. Here they are:

h-g-m didn't start internal process tracking. Not very likely, unless package was deactivated due to update or manually, and for some reason Linter didn't dispose of it in time.
Neither onChangeCheck/Lint and onSaveCheck/Lint are enabled in h-g-m settings.
Text editor is empty. There is nothing to check/lint.

Since everything seems to work more or less as expected on my end, I would guess that one of the three above is going on for you. If it's not (2) or (3), then we have a problem. Otherwise, well, (2) should be easy to fix yourself, and (3) is kind of a weird outlier, but it's an easy fix for me.
Soo... does any of this sound at least remotely probable?

Well, anyway, I pushed another patch that should work as a band-aid to this particular problem. Let me know if it works for you.

Thanks, now it works as intended! I did indeed have all linters disabled (i.e. HLint wasn’t called because it was buggy recently, ignoring the HLint.hs configuration file). Since this is a valid use case I think it should not throw an error.
Anyway, thanks for the fix! I hope it’s a band aid you can remove in the future again, and not something we’re stuck with forever.

Well, the intention is to move Linter support to ide-haskell, so that not only h-g-m can use Linter as frontend, but also ide-haskell-cabal and ide-haskell-repl. And since h-g-m is severely crippled without ide-haskell, well, it's a bit of a requirement anyway. But that particular change, as well as some other, are currently work-in-progress, and will remain so for a bit longer (probably till after Easter). I'll try to make the eventual transition as effortless for the end-user as possible.


[Feature Request]: improve book review web app

Is there an existing issue for this?

[X] I have searched the existing issues

Feature Description
Proposed Features:

Hover Effect: Add an interactive hover effect to enhance the user experience. For example, subtle animations when users hover over book reviews or buttons.
Likes/Shares Icons: Include "Like" and "Share" icons under each book review to promote user engagement and sharing.
Fix Titles: Address inconsistencies in the titles of book reviews to ensure clarity and uniformity across the platform.
Improve About Us Section: Redesign the "About Us" section to better reflect the platform’s mission, vision, and team details in a visually appealing manner.

Use Case
These updates aim to make the platform more user-friendly, visually engaging, and interactive while maintaining a professional and clean design.
Benefits
No response
Add ScreenShots/Videos
No response
Priority
High
Record

[X] I have read the Contributing Guidelines
[ ] I'm a IEEE-IGDTUW Open Source Week 2024 contributor
[X] I have starred the repository

@SanikaBhalerao1345 u are assigned , u can start working

@apu52 pls check i have raised a PR


Select Dropdown color impacted by Operating system.

Select drop-down on hover takes default color as Operating system.
In simple Form if we select dropdown on hover with mouse it shows Orange color on Ubuntu and Blue for IOS and Windows
select dropdown on hover should show the according to style sheet applied
Images shows the Orange color on Ubuntu and Blue on IOS and Windows

@HarishRMaryada this is not related to redux-form, this is the behaviour of the os/browser. Look Chromium on Ubuntu.

Thanks,@andrewluca. The color issue is with the select tag itself,it behave differently in different OS.


Make REPL mode

Hi! Disclaimer: I have not yet used this package, and discovered it yesterday.
I was just thinking that for this package, a REPL mode would be fantastic! It could save the user from typing ai"" in every call, display the current model in the promt, and other goodies. Also, ai! could be called automatically on subsequent inputs, until the user exits and reenters the REPL mode, or types end (mimicing the julia syntax to end a block), or something else.
It might even be very easy to make, using ReplMaker!

Perhaps this could be helpful: https://github.com/ThatcherC/ReplGPT.jl?
Or were you thinking about something else?

Oh, that is really cool! Yes, that is what I was imagining. Is that package and this very different in their goals? Why are they separate packages?

It's a different scope (single chat vs. llm primitives for various workloads)

I'll close this one for now as a "will not implement", since the package scope is different. PromptingTools.jl is better suited as building blocks for a REPL mode, and one such REPL already exists.
Feel free to open again if anyone disagrees!

That sounds reasonable, thanks!


Configuration fixes

See issue nodejs/node#34765
This fixes https://github.com/okonet/lint-staged/issues/1054

Ping @okonet yet another issue not caught by tests because of Babel transpilation. 😅

I also pushed https://github.com/okonet/lint-staged/pull/1055/commits/b00ca79eacba15bcf6e9b58849f6e930490184b2 which fixes https://github.com/okonet/lint-staged/issues/1056 and https://github.com/okonet/lint-staged/issues/1057

Ping @okonet let's get these unfortunate bugs fixed!


Add SEIR model template

Thank you for your great and interesting work!
I prepared a custom template of the SEIR model, a differential equation model for predicting infectious diseases.
For more details, please visit my blog post, where I wrote about an experiment using this template.
Note
To ensure this template code runs correctly, the line https://github.com/SakanaAI/AI-Scientist/blob/8383c8a7d835a8b93a08dec50a93cd745242c1e1/ai_scientist/perform_experiments.py#L64
in the run_experiment() function needs to be changed to
results = {k: v for k, v in results.items() if k != "solution"}

Would you be able to add a dummy means field for your new variable so we don't have to make a change that could be hard to maintain in the future? Thanks so much!

Thank you for reviewing!
I added a dummy means and a dummy dataset name. We can run this template without any modification now.


 Detailed function allocation isn’t regulated.

INFO accepted.

Modify WARN to INFO in mtask8 artifact.

… A.6.2.d
d. Info if any individual required monitor, except Continuous Component Monitoring (CCM) is supported by more than one OBD ECU.


Merge pull request #1 from dotnet/master

Send Pull Request

Appears to have been opened in error.


overlap fails when other pyrange is empty

Hi,
I ran into this problem when finding what ranges from one set overlap ranges in several other sets. Here is a trivial test case:
pr1 = pr.PyRanges(pd.DataFrame({
    'Chromosome': ['1', '1'],
    'Start': [243, 645],
    'End': [267, 689],
}))
pr2 = pr.PyRanges(pd.DataFrame({
    'Chromosome': ['2', '2'],
    'Start': [243, 645],
    'End': [267, 689],
}))
pr3 = pr.PyRanges(pd.DataFrame({
    'Chromosome': ['3', '3'],
    'Start': [243, 645],
    'End': [267, 689],
}))

# overlap of pr1 and pr2; should be empty
s1 = pr1
s2 = pr2
res1 = s1.overlap(s2, strandedness=None, how='first')

# overlap of first comparison with pr3; should also be empty
s1 = res1
s2 = pr2
res2 = s1.overlap(s2, strandedness=None, how='first')

Which generates this error:
---------------------------------------------------------------------------
IndexError                                Traceback (most recent call last)
<ipython-input-158-047be8f5ac7d> in <module>
     17 s1 = res1
     18 s2 = pr2
---> 19 res2 = s1.overlap(s2, strandedness=None, how='first')
     20 len(res2.df)

~/jupyter-venv/lib/python3.7/site-packages/pyranges/pyranges.py in overlap(self, other, **kwargs)
    423         kwargs = fill_kwargs(kwargs)
    424
--> 425         dfs = pyrange_apply(_overlap, self, other, **kwargs)
    426
    427         return PyRanges(dfs)

~/jupyter-venv/lib/python3.7/site-packages/pyranges/multithreaded.py in pyrange_apply(function, self, other, **kwargs)
    215     else:
    216
--> 217         if self.stranded and not other.stranded:
    218
    219             for (c, s), df in items:

~/jupyter-venv/lib/python3.7/site-packages/pyranges/pyranges.py in stranded(self)
    686     def stranded(self):
    687         # print(self.keys())
--> 688         return len(list(self.keys())[0]) == 2
    689
    690     @property

IndexError: list index out of range

The immediate problem is since res1 is empty it doesn't have anything with which to populate the dataframe dictionary dfs. So when checking if either set is stranded it fails because self.keys() is empty
    @property
    def stranded(self):
        return len(list(self.keys())[0]) == 2

Not sure if this is the best fix, but stranded(self) should probably first check if len(self.keys()) > 0 or something. Alternately, maybe just save a lot of computational work and check in overlap if either PyRanges is empty and if so return an empty PyRanges object.

Wow, thanks for the detective work. I should think up some hypothesis tests that do three or more operations in a row. I wouldn’t need to check the result really, just that no exception is thrown :)

Dunno how much you care about internals, but: Pyranges has a function dispatcher which is the same for all binary functions. It takes care of the strand logic, multithreading and so on. That is actually the place where this is happening, before the overlap function is dispatched. Pyranges is extremely thoroughly tested with automatically generated tests, but it is always tested with pyranges of at least one row. I’ll create some tests where zero row pyranges are allowed also :)

Btw, this is syntax you can use to create an empty gr:
gr_empty = pr.PyRanges(pd.DataFrame(columns="Chromosome Start End".split()))
Will try to fix this tonight after sending in the paper. This raises the question whether empty pyranges should be considered stranded or not 🤔

Awesome! Glad to help! I'm very excited about this project. It's always the case that users almost immediately find cases that break your painstakingly polished code.
That is shorter than what I'd been using:
pr.PyRanges(pd.DataFrame({
            'Chromosome': [],
            'Start': [],
            'End': [],
        }))

It seems like the problem with mixing stranded and non-stranded features together stems from the difficulty in combining DataFrames with optional 'Strand' columns. Maybe the 'Strand' columns should be required and a third symbol (like * or .) used to indicate it's lack of application to that feature. They could be automatically added and set to the no strand symbol if missing. What do you think? I'm not sure of all the implications here.
Also, as part what I was working on I created a fairly clean solution to check for ranges from one set that overlap ranges in all members of a list of other ranges. Would you be open to accepting a pull-request for code that adds this and similar functionality?

I'm hesitant to add code and want to keep the core library fairly slim. It is better to create additional libraries I think. I have plenty of useful functionality that I have kept out of pyranges, but which I will release as other packages eventually. Tidy-functions, GenomicFeatures-functions and so on :)
Does your function do something like the following?
pr.concat([query_range.overlap(subject_range) for subject_range in subject_ranges])

If a PyRange is empty, it is not technically correct to call it stranded or unstranded. The main problem is that most functionality should work with empty PyRanges, i.e. empty_pyrange.intersect(other_pyrange, strandedness="same") should return a valid result, not err with a cannot perform stranded operations on unstranded PyRanges-error :)
Will think more about strandedness, I guess that is a discussion for a different thread.

Sounds good.
So if I have a list of PyRanges: [A, B, C, D] I needed a list of all the ranges in A that overlapped a range in ALL of B, C, D. This is the code I have:
from functools import reduce

def multioverlap(iv1, iv2):
    return iv1.overlap(iv2, strandedness=None, how='first')

# assume A, B, C, D are existing PyRanges objects
interval_list = [A, B, C, D]
ovlp = reduce(multioverlap, interval_list)
# ovlp now has ranges from A which overlap ranges in all of B, C, D

Then this is what you want:
pr.concat([A.overlap(subject_range, how="first") for gr in [B, C, D]])

The pr.concat reduces all the pyranges to one :) Thanks for explaining.

Wouldn't that give all the ranges in A that overlap ANY range from B, C, D ?

Ah, sorry, yes ignore. I am too tired. I love that you can do that so tersely in Python :) But because of that, I feel that it does not need to be a part of the library. But it would be good to include in a tutorial/book on PyRanges.
One-liner:
reduce(lambda iv1, iv2: iv1.overlap(iv2, strandedness=None, how='first'), interval_list)

No worries, I know the feeling! Thanks for your work on this, I look forward to using it extensively!

This is fixed in 0.0.19. Thanks for reporting. There are other bugs that happen when you chain a lot of data, working on fixing/testing this with https://github.com/biocore-ntnu/pyranges/blob/master/tests/test_do_not_error.py
Will continue after the weekend.


Allow minimal distance-based edges in a habitat graph

Description: Currently graph edges are only added to the graph when two polygons overlap or touch. For habitats there will almost always be a certain distance a species is able to travel across land that is not part of the habitat. Therefore we want to be able to add edges between polygons if they are within a certain maximum travel distance.
Potential solution: One way would be to add an edge to all polygons within a certain range (e.g. 10m) in the initial creation of the (parent-)graph and add an edge-attribute of min_distance between two polygons. Then when we add habitat subgraphs later we could easily filter the edges such that their min_distance is withing the max_travel_distance.
Further notes:
The create_habitat method of the GeoGraph class might take the following arguments

name (e.g. "eagle habitat")
list of landcover classes that are part of the habitat (e.g. ["pine forests", ...])
max_travel_distance: maximum travel distance outside habitat (e.g. 50m)
list of landcover classes that are uncrossable/impassable (e.g. ["motorway",...])

The RTree implementation returns a list of polygons which are within the rectangular bounding box of the passed Geometry. Therefore the best algorithm for this may be:
for each polygon:
    define a rectangular polygon object from the bounding box of the original polygon by adding on the max_travel_distance
    query the rtree with this object to get a list of candidate polygons
    for each candidate polygon:
        check if the min_distance between the candidate and the original polygon is within max_travel_distance
        if yes, add an edge with min_distance

Note that we don't have to do this on graph creation, but we can do it at any time as long as we have the RTree and a way to iterate over polygons/nodes (maybe we should add the RTree as a class attribute). After looking into the networkx docs, I think we will actually need to create a separate graph object for each habitat (correct me if I'm wrong), so the create_habitat function might duplicate the graph, add these edges in, then add the graph object for that habitat into a class attribute dictionary.

That pseudocode looks good! The idea with doing adding the min_distance attribute at graph creation was to reuse the min_distance computations (because I think those will be the most computationally expensive part, although I haven't tested this). I agree that doing all the computations at the graph creation stage might be limiting (as it would set a max value for max_travel_distance for all subsequent habitats), maybe there is a way we can save the computations just between habitat creations? (so that when we create Habitat B after Habitat A, all min_distance computations from Habitat A can be reused. I might also be overthinking it though because min_distance not that computationally expensive after all.)

I expect that calculating the distance will be the most expensive part of this - so the best caching solution I can see would be to store all min_distance's calculated as edge attributes, then when we create Habitat B we can reuse the ones that we have and calculate ones that we don't.

Here's an alternative method based on what we know now.
for each polygon:
    define a new_polygon by using buffer(max_travel_distance)
    query the rtree with this object to get a list of candidate polygons
    for each candidate polygon:
        check if the new_polygon intersects with the candidate polygon
        if yes, add an edge (maybe add min_distance as an edge attribute)

This should be significantly faster since we only ever calculate the distance between nodes that are actually part of the habitat.

Closed by #31


Skeleton for BaseContract

Still not working completely, but the base should be correct.

We don't need this anymore. I PRed an updated BaseContract. Thanks for sending this in though, George!

thanks to both


Separate binary content from the provisioning framework

Fixed by https://github.com/lorengordon/SystemPrep/commit/dc77d0d8648296a8c0063ee6d880c5049ac81f84


dockerfiles for android build corrupted

@mpl we spoke in person. please see this issue in @wasabeef docker project for android  https://github.com/wasabeef/dockerfiles/issues/1

@cliveb here's the situation: the wasabeef/android Dockerfile (https://github.com/wasabeef/dockerfiles/blob/master/android/Dockerfile) indeed tries to fetch some archives (like the apache ant one) that do not exist anymore.
Which means if you tried to build the image yourself with docker build you wouldn't be able to. Or the day the maintainer removes the image from his disk and wants to rebuild it, he'll be boned too.
However, the image on the docker registry itself is still there, and very much usable. I was able to 'docker pull wasabeef/android' from two different computers without a problem.
I suspect the docker registry had some problems at the moment you tried. Can you retry to 'docker pull wasabeef/android' and confirm that it works for you now?
Building the "camlistore/android" image is another, different, story. It has other problems, which I'm looking at atm.

@mpl success. thanks for working through the camlistore/android.
Restored session: Sat May 14 03:05:14 PDT 2016
clivebs-bigMac:cliveb seebe$ wget
https://github.com/wasabeef/dockerfiles/blob/master/android/Dockerfile
--2016-05-14 15:01:00--
https://github.com/wasabeef/dockerfiles/blob/master/android/Dockerfile
Resolving github.com... <IP_ADDRESS>
Connecting to github.com|<IP_ADDRESS>|:443... connected.
HTTP request sent, awaiting response... 200 OK
Length: unspecified [text/html]
Saving to: 'Dockerfile'
Dockerfile                        [ <=>
]  47.64K   244KB/s    in 0.2s
2016-05-14 15:01:00 (244 KB/s) - 'Dockerfile' saved [48781]
On Fri, May 13, 2016 at 11:24 AM, mpl<EMAIL_ADDRESS>wrote:

@cliveb https://github.com/cliveb here's the situation: the
wasabeef/android Dockerfile (
https://github.com/wasabeef/dockerfiles/blob/master/android/Dockerfile)
indeed tries to fetch some archives (like the apache ant one) that do not
exist anymore.
Which means if you tried to build the image yourself with docker build you
wouldn't be able to. Or the day the maintainer removes the image from his
disk and wants to rebuild it, he'll be boned too.
However, the image on the docker registry itself is still there, and very
much usable. I was able to 'docker pull wasabeef/android' from two
different computers without a problem.
I suspect the docker registry had some problems at the moment you tried.
Can you retry to 'docker pull wasabeef/android' and confirm that it works
for you now?
Building the "camlistore/android" image is another, different, story. It
has other problems, which I'm looking at atm.
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
https://github.com/camlistore/camlistore/issues/731#issuecomment-219122175


[Updated]Migrate to null safety

I migrated this plugin to null safety.
I have already checked the code can run on my application.
This is my first time pull request, Is there any questions, please connect me.

@ShaunRain any update on this?

please update it on pub.dev, we can use it like this:-
flutter_tindercard:
git:
url: https://github.com/ShaunRain/flutter_tindercard.git
but update on pub.dev will be helpful for beginners.


Handling updated pip freeze output

The pip freeze command output has been updated to say 'pip freeze' instead of 'pip --freeze'.
We switch from a string match to a regex so that we can handle both the old and new styles.
I was very interested to see the cram tests as I've never heard of it before. I've done my best to include tests which cover the case we're interested in. As pip-dump calls pip freeze internally it is quite hard to 'mock' it out as such so we have two tests which install different versions of pip before repeating the pattern.
Hope it is acceptable,
Michael

Ah, I removed argparse because it is installed on my system and I cannot get my python/virtualenv setup to ignore my system modules so the tests working working for me. I'll update my new tests to work on travis.
The others were failing anyway though I will see if they can be trivially updated.

I have done my best. Feel free to ignore the last two commits as they aren't strictly related but they do attempt to fix the other test failures.

Thanks for the hard work, @michaeljones!

No trouble. Thanks for writing the tool, I've found it very useful!


And then you write:
"Replace "<your_CN_value>" with "mytestcert""
But <your_CN_value> is not mentioned in any other place.
In addition, it would be interesting for the reader to know how to configure the https certificate using SF native configuration like:

And get the value to the code and kerstel.
Thanks,
Avner

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

ID: 3ac102a5-e9ad-0046-f217-446f1ad263c5
Version Independent ID: a47523e2-0c30-dee4-0037-5f58836fa37f
Content: Add an HTTPS endpoint using Kestrel - Azure Service Fabric
Content Source: articles/service-fabric/service-fabric-tutorial-dotnet-app-enable-https-endpoint.md
Service: service-fabric
GitHub Login: @sukanyamsft
Microsoft Alias: sukanyas

@avnerms Thanks for the feedback! I have assigned the issue to content author to check and update the document as appropriate.We will update you accordingly.

@sukanyamsft Can you please check and add your comments on this doc update request as applicable

Thanks for your dedication to our documentation. Unfortunately, at this time we have been unable to review your issue in a timely manner and we sincerely apologize for the delayed response. Though the scope of our feedback channel here on GitHub covers specific documentation fixes, we can help redirect you to the right support channel to get an answer to your question: [reference to alternative product/service or community endpoint]. Because this issue does not relate to documentation, we are closing this issue. #please-close


Change the RootSync helm.namespace field default value

Change the helm.namespace default value from "config-management-system"
to "default" to be consistent with other tools: Helm, Kustomize etc.

@xinnywinne , please rebase your PR.

It seems that this duplicates https://github.com/GoogleContainerTools/kpt-config-sync/pull/137.

This PR cherrypicks #137 to the v1.13 branch.

This PR cherrypicks #137 to the v1.13 branch.

Ahh. I see. I didn't notice that.

/lgtm

It seems that this duplicates #137.

Thanks Haiyan for your review, just see your comment :)

/lgtm


Revert "Bump node-red from 0.17.5 to 1.0.4"

Reverts NoeSamaille/node-red-contrib-activedirectory#2

Not relevant anymore


Confusing if you try to log in while logged in

What I did

Log out of OSF
Open two tabs at the login page
Log in on one tab
Log into a different account on the second tab

What I would expect
There would be an error that says, "You are already logged in as <username>. Please log out first."
What happened
It took me to the dashboard for the already logged in user
Alternate solution
It logs out the initial user and logs in the second user.
This happens on staging and production. It's not a security issue as near as I can tell, because you don't have any access you wouldn't have had by just clicking on the home link, but it can certainly be confusing.

@huangginny @JeffSpies any updates on this? I didn't see any PRs mentioning this issue.

@HarryRybacki @huangginny had submitted #1015 and referenced this, but I am uncertain whether all questions were resolved. I can help you followup.

@HarryRybacki I think my pr is still in the old osf repository, not in the osf.io one :8ball:  I'll check it out

Thanks!
On Mon, Feb 23, 2015 at 9:16 AM, Ginny<EMAIL_ADDRESS>wrote:

@HarryRybacki https://github.com/HarryRybacki I think my pr is still in
the old osf repository, not in the osf.io one [image: :8ball:] I'll check
it out
—
Reply to this email directly or view it on GitHub
https://github.com/CenterForOpenScience/osf.io/issues/982#issuecomment-75547302
.

--
Harry Rybacki - Developer, Center for Open Science
Research: http://scinet.osf.io
Blog: http://rybacki.io
[image: http://github.com/harryrybacki]
http://github.com/HarryRybacki [image:
https://twitter.com/HarryRybacki] https://twitter.com/HarryRybacki [image:
http://www.linkedin.com/in/HarryRybacki/]
http://www.linkedin.com/in/HarryRybacki/ [image:
https://plus.google.com/+HarryRybacki]
https://plus.google.com/+HarryRybacki

@HarryRybacki CenterForOpenScience/osf#1015

Recreated Ginny's changes. Logs in second person after logging out 1st. Should I submit a PR? Any more comments?

Updated #1015 @Ghalko ? Yes, looks like consensus and other practice says go with   'Logs in second person after logging out 1st.'

@brianjgeiger Does this issue still exist with the latest integration of SSO/CAS?

Yes, it works with the alternate solution given above. Much better.


'Model' tab of App Settings is not correct anymore

In chapter 3 of https://docs.mendix.com/refguide/date-and-time-handling, the last paragraph speaks of the Model tab under App Settings.
The setting that is talked about (the default timezone) can be found on the Runtime tab instead.

Hi Martin,
Thanks very much for pointing out this error. I have updated the documentation in this commit: https://github.com/mendix/docs/commit/5efca7990ac5504d1a186154604b4569d25cfec1.
Thanks for taking the time to point out the error and submitting an issue. It is great to have support in keeping the documentation up-to-date and correct.
Yours
Mark


How to change colors of the header footer and buttons

I could n't figure out it checked the code but couldn't figure it out

pretty much every element type has https://github.com/goolord/alpha-nvim/blob/main/doc/alpha.txt#L78 and buttons have https://github.com/goolord/alpha-nvim/blob/main/doc/alpha.txt#L82
so changing the header for example would be dashboard.section.header.opts.hl = "Red"

could you give me an example of adding colors in that group     hl = "hl_group" | {{"hl_group", 0, -1}}


[YUNIKORN-2548] Potential deadlock during concurrent bottom-up/top-down queue traversal

What is this PR for?
Avoid potential deadlock while removing queues during config refresh. MarkQueueForRemoval() performs a top-down traversal of the hierarchy, while resource increment/decrement does the opposite.
What type of PR is it?

[x] - Bug Fix
[ ] - Improvement
[ ] - Feature
[ ] - Documentation
[ ] - Hot Fix
[ ] - Refactoring

Todos

[ ] - Task

What is the Jira issue?
https://issues.apache.org/jira/browse/YUNIKORN-2548
How should this be tested?
Run test from smoke_test.go with the deadlock detector enabled.
Screenshots (if appropriate)
Questions:

[ ] - The licenses files need update.
[ ] - There is breaking changes for older versions.
[ ] - It needs documentation.

Codecov Report
Attention: Patch coverage is 94.11765% with 1 lines in your changes are missing coverage. Please review.

Project coverage is 77.01%. Comparing base (f0fe932) to head (1e65bec).

Files
Patch %
Lines

pkg/scheduler/objects/queue.go
94.11%
0 Missing and 1 partial :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #839      +/-   ##
==========================================
- Coverage   77.01%   77.01%   -0.01%
==========================================
  Files          97       97
  Lines       12043    12050       +7
==========================================
+ Hits         9275     9280       +5
- Misses       2432     2433       +1
- Partials      336      337       +1

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

/recheck


Replace null with empty array []

Many projects use sindresorhus's eslint-plugin-unicorn. It's recommended config includes the no-null rule. Here is the reasoning behind not using null 1, 2, 3.
I suppose it would make sense to replace it with an empty array [].

@Kamahl19 I guess this is a more subjective topic than an objective one. Even in the discussion you've shared, there are a lot of arguments for both sides. I've also worked on some codebases that used the unicorn plugin and disabled many of the proposed rules (and that happens with almost every ESLint plugin), so it's pretty hard to make decisions based on someone's opinion.
MDN's definition of null says:

The null value represents the intentional absence of any object value. It is one of JavaScript's primitive values and is treated as falsy for boolean operations.

Which is exactly the point of declaring null when you don't want a queryKey to add properties to the inferred value: we are saying "this value exists, but it's empty". That's why I implemented with null and not undefined.
The reason an empty array [] was not used, is because the type system inference would suffer from it and it would become more complex to handle allowed values for queryKeys (all arrays are tuples in order to be able to infer the values inside each index ~ [string, number, string] vs (string | number)[] ~ so, if we pass an empty array, there's a conflict where Typescript might think you're missing at least one value in order for the value to be the expected tuple).
Every decision comes with a tradeoff, I think it's less burden to disable an ESLint rule for a specific file than to have to refactor the type system

@lukemorales thanks for the reply. Your reasoning is sound.


Inappropriate highlighting of the previous line when the last line of a text file is added or removed

Say i have two text files which are identical, except that one has a line at the end that the other does not. If i diff these, then the diff shows the added or removed line, but it also highlights the previous line, even though it's unchanged.

This is using a freshly compiled Difftastic 0.45.0 on Ubuntu 18.04.6.

Thank you!


Added track_dependency

I understand that there is work under way to move to OpenCensus, which may then cover this use case.
In the meantime, I've added this method which should be in line with the current design.

Travis failed. Have I merged incorrectly? Can you please update PR?
Yes, directionally we will be moving to Open Census. We may need to keep API layer that simply allows to create and send data types. As Application Insights has more data types than Census.

The tests were passing locally, I swear :see_no_evil:
The type hints were the culprit, apologies. Didn't realise that the lib targets 2.7 and 3.4

@SergeyKanzhelev do you have a planned release date for these changes? We're currently using a patched TelemetryClient in our projects, but being able to just use the vanilla one would make our lives considerably easier.

@dstockhammer published. Let me know if you have any troubles with it


bugfix: e.screenPosition does not take scrollOffset into account

Hi,
We are using a div as a parent container for the director class. And when the div is positioned at for example top: 1000px, and the user need to scroll down to start playing the game, the e.screenPostion does not take scrollOffset into account. So, all the buttons and sprites do not respond to the click event because of the hitTest method using e.screenPosition.
I've found a fix for this @ http://www.quirksmode.org/js/events_properties.html and added it to events/eventdispatcher.js.
starting at line 119:
else {
// screenPosition does not take scrollOffset into account!
//ee.screenPosition = new goog.math.Coordinate(e.clientX, e.clientY);
var posx = 0;
var posy = 0;
if (!e) var e = window.event;
if (e.pageX || e.pageY) 	{
posx = e.pageX;
posy = e.pageY;
}
else if (e.clientX || e.clientY) 	{
posx = e.clientX + document.body.scrollLeft
+ document.documentElement.scrollLeft;
posy = e.clientY + document.body.scrollTop
+ document.documentElement.scrollTop;
}
ee.screenPosition = new goog.math.Coordinate(posx, posy);
doBreak = 1;
}
Is there a way to overwrite this function in our application? Or should this be added to the limejs core?
Thank you & keep up the good work!
Wouter

Thanks for reporting.
I included it in https://github.com/digitalfruit/limejs/commit/42f3c57c2f1858755175d4d908193a8c998f43b9


Given an infinity input, ConvLSTM2D will output a normal value only when the cols shape is 1

Describe the problem.
Hi, I was using the latest Keras and TensorFlow to construct a model with ConvLSTM2D layer. However, I find that, when I set the shape in the cols dimension to be 1, ConvLSTM2D will output a normal value even when the input is infinity. But when I set the cols dimension > 1, ConvLSTM2D will output nan instead.
Describe the expected behavior.
I am not sure why such inconsistent behavior occurs when I set the cols to be 1 or not 1. I guess ConvLSTM2D should behave the same?
To reproduce:
import keras
import numpy as np
x = keras.layers.Input((3,3,3,1))
model = keras.models.Model(x, keras.layers.ConvLSTM2D(10, 1)(x))
input = np.random.rand(1,3,3,3,1)*(np.Inf)
model.predict(input)

output:
array([[[[ 0.       ,  0.9950547,  0.       , -0.       , -0.       ,
           0.       ,  0.       ,  0.       ,  0.       , -0.       ],
         [ 0.       ,  0.9950547,  0.       , -0.       , -0.       ,
           0.       ,  0.       ,  0.       ,  0.       , -0.       ],
         [ 0.       ,  0.9950547,  0.       , -0.       , -0.       ,
           0.       ,  0.       ,  0.       ,  0.       , -0.       ]],

        [[ 0.       ,  0.9950547,  0.       , -0.       , -0.       ,
           0.       ,  0.       ,  0.       ,  0.       , -0.       ],
         [ 0.       ,  0.9950547,  0.       , -0.       , -0.       ,
           0.       ,  0.       ,  0.       ,  0.       , -0.       ],
         [ 0.       ,  0.9950547,  0.       , -0.       , -0.       ,
           0.       ,  0.       ,  0.       ,  0.       , -0.       ]],

        [[ 0.       ,  0.9950547,  0.       , -0.       , -0.       ,
           0.       ,  0.       ,  0.       ,  0.       , -0.       ],
         [ 0.       ,  0.9950547,  0.       , -0.       , -0.       ,
           0.       ,  0.       ,  0.       ,  0.       , -0.       ],
         [ 0.       ,  0.9950547,  0.       , -0.       , -0.       ,
           0.       ,  0.       ,  0.       ,  0.       , -0.       ]]]],
      dtype=float32)

@maybeLee I tried to reproduce the issue with the provided code  on colab using TF v2.8.0 and faced a different outcome as below .Could you please find the gist here and confirm the same?
array([[[[ 0.       ,  0.       ,  0.       ,  0.9950547,  0.9950547,
          -0.       ,  0.9950547, -0.       ,  0.       ,  0.       ],
         [ 0.       ,  0.       ,  0.       ,  0.9950547,  0.9950547,
          -0.       ,  0.9950547, -0.       ,  0.       ,  0.       ],
         [ 0.       ,  0.       ,  0.       ,  0.9950547,  0.9950547,
          -0.       ,  0.9950547, -0.       ,  0.       ,  0.       ]],

        [[ 0.       ,  0.       ,  0.       ,  0.9950547,  0.9950547,
          -0.       ,  0.9950547, -0.       ,  0.       ,  0.       ],
         [ 0.       ,  0.       ,  0.       ,  0.9950547,  0.9950547,
          -0.       ,  0.9950547, -0.       ,  0.       ,  0.       ],
         [ 0.       ,  0.       ,  0.       ,  0.9950547,  0.9950547,
          -0.       ,  0.9950547, -0.       ,  0.       ,  0.       ]],

        [[ 0.       ,  0.       ,  0.       ,  0.9950547,  0.9950547,
          -0.       ,  0.9950547, -0.       ,  0.       ,  0.       ],
         [ 0.       ,  0.       ,  0.       ,  0.9950547,  0.9950547,
          -0.       ,  0.9950547, -0.       ,  0.       ,  0.       ],
         [ 0.       ,  0.       ,  0.       ,  0.9950547,  0.9950547,
          -0.       ,  0.9950547, -0.       ,  0.       ,  0.       ]]]],
      dtype=float32)

Thanks!

Thanks, I confirm the issue. I find the model prediction result differs each time, I guess it is caused by the random initialization of ConvLSTM2D layer.
However, the symptoms are consistent (i.e., no NaN but normal output)

I observed the similar behavior in both Functional API and Sequential model.
When kernel_size is 1 it predicts random value given infinity input.
When kernel_size > 1 the outcome changes to nan given infinity input.
Attaching gist here for reference.

Triage notes: Unclear to me what's the reason here, probably related to some subtle implementation detail in keras/tf stack. Will find an assignee in triage meeting (low priority)


Add Package data model

Letter Actions should involve Packages rather than letters.
Packages contain 0 or more letters and/or 0 or more gifts.

We are simplifying the modelling of actions so this is no longer needed.


Long replies breaks the app

If the app gives a lengthy reply, the dialogue goes full screen, but is not scrollable nor is the browser zoomable.
I've tried using some tailwinds overflow-auto and overflow-scrolls. but doesn't seem to change anything.
Surely this has been experienced by other users when lengthy replies are given >?

Good catch @musashi-garami. Would love a PR to help resolve!

Just made my first ever pull request, not sure if I did it right. I forked this repo and changed the code in my forked version. (I have a private repo with many changes that's not forked, was made with the auto deploy function of this (the deploy to vercel process).
I changed the code to fix this, made a pull request for myself. Accepted, then made a pull request for this ?
If I did something wrong, please let me know.. But exciting doing first pull request!

PR looks great @musashi-garami. Thanks for contributing!


pull content from downstream back to upstream

It happens quite often that dist-git branch is updated async. We should take those changes back to source git (or even upstream repo, when a specfile changes)

This pull request solves this issue. #145

This is already solved by #145


[DOCS] Correct user id description

What part(s) of the docs would you like to see updated?
This information (taken from here) is not entirely correct:
User ID
The user_id is the user's public key ...

It should be:
User ID
The user_id is derived from user's public key ...

For the derivation we are using the Hash Sha512.

Fixed in #7


Additional aliases for the same hash type

Currently, to refer to ripemd-160, the option for it is --ripemd160. While it is a little long, OpenSSL does have this set to --rmd160.
Another example is b2sum, which can be made into an --b2b512 alias instead of --blake2b512.
Proposition:

Add --rmd160 for --ripemd160.
Add --b2s256 for --blake2s256.
Add --b2b512 for --blake2b512.
Add --mm3a for --murmur3a, maybe.
Add --mm3c for --murmur3c, maybe.
Add --mm3f for --murmur3f, maybe.

ddgst 3.0 gained --sha3 for SHA-3-256, --b2 for BLAKE2b512, and --rmd160 for RIPEMD-160.


⚠ Bofrost.at has degraded performance

In 563a709, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 2217 ms

Resolved: Bofrost.at performance has improved in fa6e83c after 29 minutes.


Generic and robust UserPreferences

Separated UserPreferences from Settings and made user-specific overrides easy and generic
What current issue(s) does this address?, or what feature is it adding?
Preferences were mangled with Settings, not easily extensible and had bugs with missing prefs json file.
How did you solve this problem?

Using defaults, only overrides are stored in JSON file
Graceful fallbacks
Easily extensible for other preferences
Separate from Settings
Common fixed filenames are defined in Settings
100% test coverage

How did you make sure your solution works?
Tests, both unit and manual
Did you add any tests?
Yes
Did you check your pycodestyle?
yes
Are there any special changes in the code that we should be aware of?
No. Just that new user preferences can easily be included into UserPreferences.py
Are you making a PR to a feature branch or development rather than master?
development

Coverage increased (+0.3%) to 66.367% when pulling 92f8aa62180fbb1576d11a95435b964b597e7cd4 on metachris:preferences into bc761328bf0ec39903c433faa99920bfc2765427 on CityOfZion:development.

This is great, if there was an emoji for user preferences I would use it!


run the tasks asynchronous

I want to run background tasks at the same time, option BACKGROUND_TASK_RUN_ASYNC = True but i am having a problem and i can not run it . Can you help me?

Could you provide more information?
Is the process running?

@philippeowagner Hey there phillip, thanks for this wonderful lib.
I have an issue with the BACKGROUND_TASK_RUN_ASYNC.
I did a test with it set to false in the first go, which of course executed the tasks linearly.
I changed it to be True and it still executed the two tasks linearly one after another instead of doing it async.
Django version: 3.2.6 (latest)
python version: 3.8
dj-vbg-tasks version: 1.2.5 (latest)
the task is a simple function which just print the argument passed to it.
both tasks are scheduled for almost the same time with a couple second's difference
Happy to provide more info if needed.
Any help is appreciated :)


VectorLayer prerender event is not being fired

While migrating from v5.2.0 to 6.1.1 I noticed that the prerender event does not get fired for VectorLayers, while precompose used to work fine. CodeSandbox with simple example: https://codesandbox.io/s/vectorlayertest-32l2p

The pre-/postrender events are only fired if the source is not empty. The renderFrame function will return early in that case.
https://github.com/openlayers/openlayers/blob/c1c256040ea9bac4e42af383ea78682ab38e181e/src/ol/renderer/canvas/VectorLayer.js#L137-L142
The events would be fired after this:
https://github.com/openlayers/openlayers/blob/c1c256040ea9bac4e42af383ea78682ab38e181e/src/ol/renderer/canvas/VectorLayer.js#L157


Possibility to have permanent tokens

Now, when setting token_lifetime to nil on initializer, it doesn't validate the exp claim anymore.

Thank you @gustavovnicius! I should have done this a long time ago. Great PR 👍

Thanks for the feedbacks @nsarno. I was a bit sleepy while doing the PR :sleepy:.
Already solved the conflicts that came from #51 will update it on #62 too


tests/fakedroid: emulate a nix-on-droid-like env for testing

Run "on-device-tests" in CI. It sure won't catch some of the weirder bugs, but it'll catch the simpler regressions.
Had to dump everything it builds to cachix, lest it takes an hour+ to run. Which is a bit weird given that on my laptop it runs comparable to my phone, I guess github actions is slower than both.
Funny how we take Ubuntu with Nix and emulate what the android app does to then run Nix inside it to test a project that brings Linux/Nix experience to... a fake not-android Linux/Nix setup.
@Gerschtli, I've tried to annotate the script, hope it's more readable now, suggestions are welcome.

I think the script is good, I just see a lot of duplication of some parts (e.g. proot mounts, symlinks/executables, login-inner mv). Hopefully at some point, we can do some nixification of this script to reduce this duplication, but for now totally fine. As long as we have a working setup to test on our computers (which is really incredible!).

From the reviewers POV it would help a lot if you would push separate commits instead of overwriting the same commit again. After the PR review is finished you can rebase your commits. That way I would actually be able to see, what you changed :)

OK, I'll try to remember this isn't GitLab where it's not a problem.

Tested locally, runs perfectly fine. Please add /.fakedroid to .gitignore :)


[Suggestion] Biome Grouping

To prevent overflow of biomes listed in unregistered pokemon gui, especially when biome mods are added such as BYG or BOP, maybe a config option to group the biomes into biome groups would make sense. Similar to what's in the betterspawning config, as an example, everything under mountainous will just be displayed as mountainous biomes then they can click on it to get the real list via /wiki command
"mountainous": [
  "minecraft:mountains",
  "minecraft:mountain_edge",
  "minecraft:wooded_mountains",
  "minecraft:gravelly_mountains",
  "minecraft:modified_gravelly_mountains",
  "pixelmon:ultra_rocky",
  "pixelmon:ultra_crater",
  "pixelmon:ultra_cliff",
  "biomesoplenty:alps",
  "biomesoplenty:alps_foothills",
  "biomesoplenty:highland",
  "biomesoplenty:highland_crag",
  "biomesoplenty:highland_moor",
  "biomeoplenty:jade_cliffs",
  "byg:alpine_foothills",
  "byg:alps"

Perhaps worth re-visiting in 1.20.1 with biome tags? Thoughts? @SnowBlitzz

Perhaps worth re-visiting in 1.20.1 with biome tags? Thoughts? @SnowBlitzz

yeah sounds good. I think there can even be some way to make it "expand" by if they need the full biome list


Coarser grained tasks for forked SBT mode, fixing failure reporting

Fixes #185
Tested manually in the Scala build:
[success] Total time: 4 s, completed 13/03/2018 1:12:27 PM
> scalacheck/testOnly TreeMapTest
[info] Compiling 1 Scala source to /Users/jz/code/scala/target/scalacheck/test-classes...
[info] + TreeMap.foreach/iterator consistency: OK, passed 100 tests.
[info] + TreeMap.worst-case tree height is iterable: OK, passed 100 tests.
[info] + TreeMap.sorted: OK, passed 100 tests.
[info] + TreeMap.contains all: OK, passed 100 tests.
[info] + TreeMap.size: OK, passed 100 tests.
[info] + TreeMap.toSeq: OK, passed 100 tests.
[info] + TreeMap.head: OK, passed 100 tests.
[info] + TreeMap.last: OK, passed 100 tests.
[info] + TreeMap.head/tail identity: OK, passed 100 tests.
[info] + TreeMap.init/last identity: OK, passed 100 tests.
[info] + TreeMap.take: OK, passed 100 tests.
[info] + TreeMap.drop: OK, passed 100 tests.
[info] + TreeMap.take/drop identity: OK, passed 100 tests.
[info] + TreeMap.splitAt: OK, passed 100 tests.
[info] + TreeMap.slice: OK, passed 100 tests.
[info] + TreeMap.takeWhile: OK, passed 100 tests.
[info] + TreeMap.dropWhile: OK, passed 100 tests.
[info] + TreeMap.span identity: OK, passed 100 tests.
[info] + TreeMap.from is inclusive: OK, passed 100 tests.
[info] + TreeMap.to is inclusive: OK, passed 100 tests.
[info] ! TreeMap.faily: Falsified after 1 passed tests.
[info] > ARG_0: 0
[info] > ARG_0_ORIGINAL: -1814962128
[info] + TreeMap.until is exclusive: OK, passed 100 tests.
[info] + TreeMap.remove single: OK, passed 100 tests.
[info] + TreeMap.remove all: OK, passed 100 tests.
[error] Failed: Total 24, Failed 1, Errors 0, Passed 23
[error] Failed tests:
[error]     TreeMapTest
[error] (scalacheck/test:testOnly) sbt.TestsFailedException: Tests unsuccessful
[error] Total time: 4 s, completed 13/03/2018 1:12:35 PM

Previously:
> scalacheck/testOnly TreeMapTest
Picked up _JAVA_OPTIONS: -Xdebug -Xrunjdwp:transport=dt_socket,server=y,suspend=y,address=5006
Listening for transport dt_socket at address: 5006
[info] + TreeMap.splitAt: OK, passed 100 tests.
[info] + TreeMap.init/last identity: OK, passed 100 tests.
[info] ! TreeMap.faily: Falsified after 1 passed tests.
[info] > ARG_0: 0
[info] + TreeMap.span identity: OK, passed 100 tests.
[info] + TreeMap.remove all: OK, passed 100 tests.
[info] + TreeMap.contains all: OK, passed 100 tests.
[info] + TreeMap.takeWhile: OK, passed 100 tests.
[info] + TreeMap.until is exclusive: OK, passed 100 tests.
[info] + TreeMap.dropWhile: OK, passed 100 tests.
[info] + TreeMap.drop: OK, passed 100 tests.
[info] + TreeMap.take: OK, passed 100 tests.
[info] + TreeMap.worst-case tree height is iterable: OK, passed 100 tests.
[info] + TreeMap.foreach/iterator consistency: OK, passed 100 tests.
[info] + TreeMap.sorted: OK, passed 100 tests.
[info] + TreeMap.from is inclusive: OK, passed 100 tests.
[info] + TreeMap.to is inclusive: OK, passed 100 tests.
[info] + TreeMap.head: OK, passed 100 tests.
[info] + TreeMap.remove single: OK, passed 100 tests.
[info] + TreeMap.slice: OK, passed 100 tests.
[info] + TreeMap.size: OK, passed 100 tests.
[info] + TreeMap.last: OK, passed 100 tests.
[info] + TreeMap.head/tail identity: OK, passed 100 tests.
[info] + TreeMap.toSeq: OK, passed 100 tests.
[info] + TreeMap.take/drop identity: OK, passed 100 tests.
[info] Passed: Total 1, Failed 0, Errors 0, Passed 1
[success] Total time: 5 s, completed 13/03/2018 10:13:31 AM

I don't see an obvious way to update SBT to make things work without this change.
I first tried changing the server (that receives the events) to merge them:
⚡ git diff head
diff --git a/main/actions/src/main/scala/sbt/ForkTests.scala b/main/actions/src/main/scala/sbt/ForkTests.scala
index 4791a081b..a554f3b13 100644
--- a/main/actions/src/main/scala/sbt/ForkTests.scala
+++ b/main/actions/src/main/scala/sbt/ForkTests.scala
@@ -137,7 +137,12 @@ private final class React(is: ObjectInputStream, os: ObjectOutputStream, log: Lo
       val event = TestEvent(tEvents)
       listeners.foreach(_ testEvent event)
       val suiteResult = SuiteResult(tEvents)
-      results += group -> suiteResult
+      results.get(group) match {
+        case Some(sr: SuiteResult) => results(group) = sr + SuiteResult(tEvents)
+        case _ => results(group) = suiteResult
+      }
+      // Don't want to defer this until the end of the run! Maybe the underlying framework
+      // really needs to manage a buffer of these events until the group is complete.
       listeners.foreach(_ endGroup (group, suiteResult.result))
       react()
   }

But we end up calling endGroup more than once for a given name which is likely to mess up reporters.
So it seems the right place to fix this is in ForkMain, which would be changed to have the same task structure as non-forked tests
But I can't really understand why the non-forked version works, either. 😕

I've created a ticket for the SBT behaviour in https://github.com/sbt/sbt/issues/4010, which is based on a standalone reproduction using a toy test framework modelled after the structure used here in Scalacheck. I hope we can merge this workaround and get a new Scalacheck release out in the meantime.


Cannot run local dev server in windows

Current behavior
I'm setting up ariakit locally on my machine
Steps to reproduce the bug

Cloned repo
Ran npm install
Ran npm run dev but getting this error

> dev
> npm run dev -w website

><EMAIL_ADDRESS>predev
> npm run pages

><EMAIL_ADDRESS>pages
> node ../../scripts/pages

node:fs:1651
  handleErrorFromBinding(ctx);
  ^

Error: EPERM: operation not permitted, symlink '..\.pages\guide' -> 'D:\CAM Files\Javascript\ariakit\packages\website\pages\guide'
    at Object.symlinkSync (node:fs:1651:3)
    at writeSymlinks (D:\CAM Files\Javascript\ariakit\scripts\pages\utils.js:461:6)
    at D:\CAM Files\Javascript\ariakit\scripts\pages\index.js:82:3 {
  errno: -4048,
  syscall: 'symlink',
  code: 'EPERM',
  path: '..\\.pages\\guide',
  dest: 'D:\\CAM Files\\Javascript\\ariakit\\packages\\website\\pages\\guide'
}
npm ERR! Lifecycle script `dev` failed with error:
npm ERR! Error: command failed
npm ERR!   in workspace<EMAIL_ADDRESS>npm ERR!   at location: D:\CAM Files\Javascript\ariakit\packages\website

System Specs:

Windows 11 64bit
Node: 16.13.0 (Currently using 64-bit executable)

Expected behavior
Should run without errors
Possible solutions
No response

Oh oops! Thanks I missed it.


G UI new puzzle from base

Finished new puzzle from base command and Added Documentation for it

Common Mitchell slay


Compile Time issues on Windows because of same filename.

Compile Time issues on Windows because of same filename(case Insensitivity).
https://github.com/quarkiverse/quarkus-authzed-client/blob/main/grpc/client/src/main/java/com/authzed/api/v1alpha1/WatchResourcesService.java
https://github.com/quarkiverse/quarkus-authzed-client/blob/main/grpc/client/src/main/java/com/authzed/api/v1alpha1/WatchresourcesService.java

This code is generate, os I am not sure if there is much we can do about it.


Implement versioned transaction

Implement transaction V2 & CU requests.

Hi, is there any ETA on supporting versioned transactions?

Anyone please? Let me kindly ask, is this project still maintained?