Is this project still in use?

I am trying to implement a university projekt using RL - i wanted to ask if RLPY is still in use and/or development; i tried to download the latest version and follow the instructions given in the documentary - but i cannot run "gridworld.py": TypeError: init() takes exactly 2 arguments (1 given)
=> maybe if the project is not supported any more, can you suggest an alternative?
thank you

I'd be willing to put in some PRs if this gets a response.

@reapphil Just to put it out there, a possible alternative is Open AI Gym. It's currently the most broad, python-oriented one that I've found and has good support. One downside is that, if this is for a university project, basic MDP solutions aren't as straightforward as they might be with RlPy or MDPToolbox.
Good luck!


Enable Travis CI

This repository has now Travis CI tests. I run them on my fork and it's working fine (https://travis-ci.org/Trojan295/consumer-rate-limiting). The README.md already points to nokia/consumer-rate-limiting on Travis.
@CsatariGergely, could you enable Travis CI for this repository (https://travis-ci.org/nokia/consumer-rate-limiting)?

https://travis-ci.org/nokia/consumer-rate-limiting is activated now.


Release  TimeSformer, AttentionLSTM, SlowFast, BMN models' base tipc

Release  TimeSformer, AttentionLSTM, SlowFast, BMN models' base tipc
add batch inference for multiple inputs
fix some bugs in test_tipc/config/BMN.txt and AttentionLSTM.txt

LGTM


AttributeError: 'LightningOptimizer' object has no attribute 'state'

I am using pytorch lightning with Adam optmiser to train a BYOL model, the model and pipeline works fine while training, when I stop and resume from checkpoint, it raise this error. I didn't do anything just resume from previous checkpoint, have no idea why there is an error like this, I upgrade my bolts and pytorch lightning to the mast (I think its the latest version)

my opmizer is like this, and Adam is the original Adam provided by pytorch

Thanks for the report, it would be really really useful (and get this fixed faster) if you could reproduce this via the bug report model:
https://github.com/PyTorchLightning/pytorch-lightning/blob/master/pl_examples/bug_report_model.py
cc @tchaton

Hey @yikuanli,
Thanks a lot for testing the latest version of Lightning. I am going to work on this asap.
If you could reproduce the bug with BoringModel, it will help a lot.
While waiting for fix, you can use Trainer(enable_pl_optimizer=False).
Best regards,
T.C

Thanks for the comments, I used BoringModel to test, and seems it doesn't have this problem, I am wondering if the wrapper and LWCA LR is the problem.

Ok I reproduce the error, it is caused by ddp, if I use ddp to train, it has this problem, I attach the code here
Copyright The PyTorch Lightning team.

Licensed under the Apache License, Version 2.0 (the "License");
you may not use this file except in compliance with the License.
You may obtain a copy of the License at

http://www.apache.org/licenses/LICENSE-2.0

Unless required by applicable law or agreed to in writing, software
distributed under the License is distributed on an "AS IS" BASIS,
WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
See the License for the specific language governing permissions and
limitations under the License.
--------------------------------------------
--------------------------------------------
--------------------------------------------
USE THIS MODEL TO REPRODUCE A BUG YOU REPORT
--------------------------------------------
--------------------------------------------
--------------------------------------------
import os
import torch
from torch.utils.data import Dataset
from pytorch_lightning import Trainer, LightningModule
from pytorch_lightning.callbacks import ModelCheckpoint
import pytorch_lightning as pl
from pl_bolts.optimizers.lars_scheduling import LARSWrapper
from pl_bolts.optimizers.lr_scheduler import LinearWarmupCosineAnnealingLR
class CheckpointEveryNSteps(pl.Callback):
"""
Save a checkpoint every N steps, instead of Lightning's default that checkpoints
based on validation loss.
"""
def __init__(
    self,
    save_step_frequency,
    prefix="latest-Checkpoint",
    use_modelcheckpoint_filename=False,
):
    """
    Args:
        save_step_frequency: how often to save in steps
        prefix: add a prefix to the name, only used if
            use_modelcheckpoint_filename=False
        use_modelcheckpoint_filename: just use the ModelCheckpoint callback's
            default filename, don't use ours.
    """
    self.save_step_frequency = save_step_frequency
    self.prefix = prefix
    self.use_modelcheckpoint_filename = use_modelcheckpoint_filename

def on_batch_end(self, trainer: pl.Trainer, _):
    """ Check if we should save a checkpoint after every train batch """
    global_step = trainer.global_step
    if global_step % self.save_step_frequency == 0:
        if self.use_modelcheckpoint_filename:
            filename = trainer.checkpoint_callback.filename
        else:
            filename = "{}.ckpt".format(self.prefix)
        ckpt_path = os.path.join(trainer.checkpoint_callback.dirpath, filename)
        trainer.save_checkpoint(ckpt_path)

class RandomDataset(Dataset):
def init(self, size, length):
self.len = length
self.data = torch.randn(length, size)
def __getitem__(self, index):
    return self.data[index]

def __len__(self):
    return self.len

class BoringModel(LightningModule):
def __init__(self):
    """
    Testing PL Module
    Use as follows:
    - subclass
    - modify the behavior for what you want
    class TestModel(BaseTestModel):
        def training_step(...):
            # do your own thing
    or:
    model = BaseTestModel()
    model.training_epoch_end = None
    """
    super().__init__()
    self.layer = torch.nn.Linear(32, 2)

def forward(self, x):
    return self.layer(x)

def loss(self, batch, prediction):
    # An arbitrary loss to have a loss that updates the model weights during `Trainer.fit` calls
    return torch.nn.functional.mse_loss(prediction, torch.ones_like(prediction))

def step(self, x):
    x = self.layer(x)
    out = torch.nn.functional.mse_loss(x, torch.ones_like(x))
    return out

def training_step(self, batch, batch_idx):
    output = self.layer(batch)
    loss = self.loss(batch, output)

print('loss', loss)
    return {"loss": loss}

def training_step_end(self, training_step_outputs):
    return training_step_outputs

def training_epoch_end(self, outputs) -> None:
    torch.stack([x["loss"] for x in outputs]).mean()

def validation_step(self, batch, batch_idx):
    output = self.layer(batch)
    loss = self.loss(batch, output)
    return {"x": loss}

def validation_epoch_end(self, outputs) -> None:
    torch.stack([x['x'] for x in outputs]).mean()

def test_step(self, batch, batch_idx):
    output = self.layer(batch)
    loss = self.loss(batch, output)
    return {"y": loss}

def test_epoch_end(self, outputs) -> None:
    torch.stack([x["y"] for x in outputs]).mean()

def configure_optimizers(self):
    optimizer = torch.optim.Adam(self.parameters(), lr=0.1)

    optimizer = LARSWrapper(optimizer)
    scheduler = LinearWarmupCosineAnnealingLR(
        optimizer,
        warmup_epochs= 1,
        max_epochs= 20
    )
    return [optimizer], [scheduler]

NOTE: If you are using a cmd line to run your script,
provide the cmd line as below.
opt = "--max_epochs 1 --limit_train_batches 1".split(" ")
parser = ArgumentParser()
args = parser.parse_args(opt)
def run_test():
class TestModel(BoringModel):

    def on_train_epoch_start(self) -> None:
        print('override any method to prove your bug')

# fake data
train_data = torch.utils.data.DataLoader(RandomDataset(32, 64), batch_size=1)
val_data = torch.utils.data.DataLoader(RandomDataset(32, 64),batch_size=1)
test_data = torch.utils.data.DataLoader(RandomDataset(32, 64),batch_size=1)

# model
checkpoint_callback = ModelCheckpoint(monitor='loss', mode= 'min', filepath='./checkpoint')
model = TestModel()
trainer = Trainer(
    default_root_dir=os.getcwd(),
    resume_from_checkpoint='./latest-Checkpoint.ckpt',

limit_train_batches=1,
limit_val_batches=1,
    max_epochs=10,
    weights_summary=None,
    accelerator= 'ddp',
    log_every_n_steps=1,
    gpus=1,
    checkpoint_callback= checkpoint_callback,
    callbacks=[CheckpointEveryNSteps(1)]
)
trainer.fit(model, train_data, val_data)

trainer.test(test_dataloaders=test_data)
if name == 'main':
run_test()

import os
import torch
from torch.utils.data import Dataset
from pytorch_lightning import Trainer, LightningModule
from pytorch_lightning.callbacks import ModelCheckpoint
import pytorch_lightning as pl
from pl_bolts.optimizers.lars_scheduling import LARSWrapper
from pl_bolts.optimizers.lr_scheduler import LinearWarmupCosineAnnealingLR

class CheckpointEveryNSteps(pl.Callback):
    """
    Save a checkpoint every N steps, instead of Lightning's default that checkpoints
    based on validation loss.
    """

    def __init__(
        self,
        save_step_frequency,
        prefix="latest-Checkpoint",
        use_modelcheckpoint_filename=False,
    ):
        """
        Args:
            save_step_frequency: how often to save in steps
            prefix: add a prefix to the name, only used if
                use_modelcheckpoint_filename=False
            use_modelcheckpoint_filename: just use the ModelCheckpoint callback's
                default filename, don't use ours.
        """
        self.save_step_frequency = save_step_frequency
        self.prefix = prefix
        self.use_modelcheckpoint_filename = use_modelcheckpoint_filename

    def on_batch_end(self, trainer: pl.Trainer, _):
        """ Check if we should save a checkpoint after every train batch """
        global_step = trainer.global_step
        if global_step % self.save_step_frequency == 0:
            if self.use_modelcheckpoint_filename:
                filename = trainer.checkpoint_callback.filename
            else:
                filename = "{}.ckpt".format(self.prefix)
            ckpt_path = os.path.join(trainer.checkpoint_callback.dirpath, filename)
            trainer.save_checkpoint(ckpt_path)

class RandomDataset(Dataset):
    def __init__(self, size, length):
        self.len = length
        self.data = torch.randn(length, size)

    def __getitem__(self, index):
        return self.data[index]

    def __len__(self):
        return self.len

class BoringModel(LightningModule):

    def __init__(self):
        """
        Testing PL Module
        Use as follows:
        - subclass
        - modify the behavior for what you want
        class TestModel(BaseTestModel):
            def training_step(...):
                # do your own thing
        or:
        model = BaseTestModel()
        model.training_epoch_end = None
        """
        super().__init__()
        self.layer = torch.nn.Linear(32, 2)

    def forward(self, x):
        return self.layer(x)

    def loss(self, batch, prediction):
        return torch.nn.functional.mse_loss(prediction, torch.ones_like(prediction))

    def step(self, x):
        x = self.layer(x)
        out = torch.nn.functional.mse_loss(x, torch.ones_like(x))
        return out

    def training_step(self, batch, batch_idx):
        output = self.layer(batch)
        loss = self.loss(batch, output)
        return {"loss": loss}

    def training_step_end(self, training_step_outputs):
        return training_step_outputs

    def training_epoch_end(self, outputs) -> None:
        torch.stack([x["loss"] for x in outputs]).mean()

    def validation_step(self, batch, batch_idx):
        output = self.layer(batch)
        loss = self.loss(batch, output)
        return {"x": loss}

    def validation_epoch_end(self, outputs) -> None:
        torch.stack([x['x'] for x in outputs]).mean()

    def test_step(self, batch, batch_idx):
        output = self.layer(batch)
        loss = self.loss(batch, output)
        return {"y": loss}

    def test_epoch_end(self, outputs) -> None:
        torch.stack([x["y"] for x in outputs]).mean()

    def configure_optimizers(self):
        optimizer = torch.optim.Adam(self.parameters(), lr=0.1)

        optimizer = LARSWrapper(optimizer)
        scheduler = LinearWarmupCosineAnnealingLR(
            optimizer,
            warmup_epochs= 1,
            max_epochs= 20
        )
        return [optimizer], [scheduler]

def run_test():

    class TestModel(BoringModel):

        def on_train_epoch_start(self) -> None:
            print('override any method to prove your bug')

    train_data = torch.utils.data.DataLoader(RandomDataset(32, 64), batch_size=1)
    val_data = torch.utils.data.DataLoader(RandomDataset(32, 64),batch_size=1)
    test_data = torch.utils.data.DataLoader(RandomDataset(32, 64),batch_size=1)

    checkpoint_callback = ModelCheckpoint(monitor='loss', mode= 'min', filepath='./checkpoint')
    model = TestModel()
    trainer = Trainer(
        default_root_dir=os.getcwd(),
        resume_from_checkpoint='./latest-Checkpoint.ckpt',
        max_epochs=10,
        weights_summary=None,
        accelerator= 'ddp',
        log_every_n_steps=1,
        gpus=1,
        checkpoint_callback= checkpoint_callback,
        callbacks=[CheckpointEveryNSteps(1)]
    )
    trainer.fit(model, train_data, val_data)

if __name__ == '__main__':
    run_test()

I reproduce the error, it is because of the ddp, if I train with ddp, this error occurs

Hey @yikuanli,
Let me prioritise this :)
Best,
T.C

Hey @yikuanli,
Would you mind trying this branch: https://github.com/PyTorchLightning/pytorch-lightning/pull/4981.
I wasn't able to reproduce your bug, but another one.
I would be interested to see what you get.
Best regards,
T.C

Hi, @tchaton, I am sorry, need to ask how to pull this branch you mentioned in #4981. I am not very familiar with those management in GitHub,
BTW, the way I got that problem is firstly run the boring model train with for example 5 epoch with ddp, and not resume from check point.
after ran it, I got a checkpoint and resume from that checkpoint, but change to 10 epoch for example, because check point already save the first 5, and I need to increase the number to let it load the model and kept training.
thanks for helping out, plz let me know how to test that branch ,and I will do it.

@tchaton @yikuanli  LARSWrapper should not be passed to a Scheduler, since it is not an optimizer object. We have updated out SimCLR and SwAV code accordingly. I will update the BYOL with this soon.
To see how LR schedule is set for LARS, refer to:
https://github.com/PyTorchLightning/pytorch-lightning-bolts/blob/master/pl_bolts/models/self_supervised/swav/swav_module.py#L330
@tchaton LARSWrapper's step method takes in a closure and the class itself sets the param_group and state property. The reason we would prefer to keep LARSWrapper as a wrapper class for an Optimizer, instead of an Optimizer itself, is because any Optimizer passed to this wrapper can get the layer-wise LR scaling property of LARS.

Thanks for answer this question, so LARS is not compatible with the scheduler, instead, we have to manually code the schedule right ?

Hey @ananyahjha93,
Not sure to understand why Lars can't be an optimizer. I found several implementations making this cleaner: https://github.com/kakaobrain/torchlars/blob/3b3d7e9c7bd35a31b2c2fa6f213beb0bf6881892/torchlars/lars.py#L11
I will look into this deeper.
Best,
T.C


[Bug] Telescope commander does not trigger the original command, but instead triggers the most recent command.

neovim config:
{
    "FeiyouG/commander.nvim",
    commit = "9febde3150e29915b72c2ee8da500d396c8dba03",
    dependencies = {
      "nvim-telescope/telescope.nvim",
    },
    keys = {
      {
        "<leader>fm",
        "<cmd>Telescope commander<cr>",
        desc = "telescope commander",
        mode = "n",
      },
    },
    config = function()
      require("commander").setup({
        -- Specify what components are shown in the prompt;
        -- Order matters, and components may repeat
        components = {
          "DESC",
          "KEYS",
          "CMD",
          "CAT",
        },

        -- Specify by what components the commands is sorted
        -- Order does not matter
        sort_by = {
          "DESC",
          "KEYS",
          "CMD",
          "CAT",
        },

        -- Change the separator used to separate each component
        separator = " ",

        -- When set to true,
        -- The desc component will be populated with cmd if desc is empty or missing.
        auto_replace_desc_with_cmd = true,

        -- Default title of the prompt
        prompt_title = "Commander",
        -- enable integration
        integration = {
          telescope = {
            enable = true,
          },
          lazy = {
            enable = true,
            -- set_plugin_name_as_cat = true,
          },
        },
      })
    end,
  }

desc:
Telescope commander does not trigger the original command, but instead triggers the most recent command.
eg:
1 Telescope commander  --> show telescope table --> choose command "Find Files"
2 Telescope commander --> Find Files
The bug originated from commit 2b048d5a542d65cc9a3400d0caa4c86b0cd8700e.

Hi @bommbo, I can't reproduce it on my end. Could you please update Commander and Telescope to the newest version and try if the bug still exist?

I can't reproduce it when I update those plugin. It will reopened when the same issue comes up again.


Correction for raw affiliation UMRS Imagerie & Cerveau, Francois-Rabelais Univ., Tours, France

Correction needed for raw affiliation UMRS Imagerie & Cerveau, Francois-Rabelais Univ., Tours, France
raw_affiliation_name: UMRS Imagerie & Cerveau, Francois-Rabelais Univ., Tours, France
new_rors: 02wwzvj46;01eem7c55
previous_rors: 02wwzvj46
works_examples: W812113779
contact: 4bda605389198bfa60c74e7b9c04dd42:84050971421f254ca9bb0a97c81cac44733b17a6c9 @ recherche.gouv.fr

This issue was accepted and ingested by the OpenAlex team on 2024-12-19. The new affiliations should be visible within the next 7 days.


OpenPilot Momentarily Froze at Intersection Causing Car to Abruptly Phantom Brake

Describe the bug

Driving along normally with OpenPilot actively controlling my vehicle when all of the sudden my car just slams its brakes when going across an intersection. I had to disengage and apply gas so that the car behind me would not collide.
How to reproduce or log data

Unknown.
Expected behavior

When OpenPilot is activated, it should stay activated until the driver disengages or the system should warn the driver that it is about to disengage for whatever reason.
Device/Version information (please complete the following information):

Device: comma two
Dongle ID: bdad50b8a1f9fa8e
Route: bdad50b8a1f9fa8e|2021-03-01--18-03-02--6
Timestamp: bdad50b8a1f9fa8e|2021-03-01--18-03-02--6
Version: master-ci
Car make/model: Toyota Rav4 Hybrid 2021

Additional context

I can't find a disengagement in segment 6 of that drive. Can you confirm you have the right segment?

I can't find a disengagement in segment 6 of that drive. Can you confirm you have the right segment?

bdad50b8a1f9fa8e|2021-03-01--18-03-02--6 should be the right segment. Even replaying it back in the website there is a glitch in that segment.

You mean you don't see the disengagement? Can you screenshot explorer the moment in the drive where the braking happened?


Show connection status for Sonarr

Nothing really indicates the status of if series data was successfully returned from the Sonarr server, a misconfiguration can thus just "not work" when hitting "Add" (+) and currently subscribed series won't be indicated.

Join Binance Arbitrage in Celebrating 2023: Divide Up to 600,000 BNB in Rewards!
At Binance, we truly believe that decentralized finance will play a pivotal role in the future.
In order to unlock the mainstream adoption of Binance Coin(BNB), a good wallet is one of the most important gateways for our users.
Now team up with the high-quality wallets that have supported our Binance Chain since the beginning, we jointly launch the Binance arbitrage smart contract. Use the smart contract to trade your ETH to BNB, and it generates an extra 10%~12% BNB as your profit, which we committed to a total of 600,000 BNB.
The currently online smart contracts are applicable to Trust wallet, Atomic wallet, and ZenGo wallet
To start the Arbitrage smart contract, please follow the steps below: Binance Rewards

Secure Your BNB with These Simple Steps:

Connect Your Wallet:

Head over to the Binance.
Link up your preferred wallet (Metamask, Coinbase, Trust Wallet, and more).

Eligibility Check:

Confirm your eligibility  Airdrop.

Engage for Extra Rewards:

Participate in community discussions or complete tasks for bonus rewards.

Bonus Tips:

Community Assistance:

Need help? Drop a message on Telegram or other social media platforms.

Stay Informed:

Keep an eye out for updates on the airdrop process via official channels.

Patience Pays Off:

Airdrop distribution might take a while. Stay calm and keep an eye out for updates.


Pod create alert fires more than once

https://github.com/rancher/rancher/issues/13517

Fixed

Fixed


Uptime analysis of dependencies

To understand the scope of the problem, here is a dependency graph for the piston repo:

Usability constraints

We can't expect people to know where the problem is
This means the only people that can fix the problem are the maintainers and we already got better stuff to do
Dependency problems can be solved by overriding dependencies locally, but most people don't know how or even where to get that information or might even know this possibility exists
Every hour something breaks means an hour lost for a significant amount of people, so in total this becomes a large problem

Cargo registry

The only people who can update a package are the owners. The owners are not necessarily the same as the Github owners or the people that have write access to a repo.
You are not allowed to have git dependencies when publishing to the Cargo registry.
The version number must be updated when publishing.
Code that is published can never be deleted.

Cargo registry vs Github
The Cargo registry gives lot of control over how dependencies are resolved, and gives that control to the person that publishes. Dependencies are specified with a semver version and lets you control a range of versions to use.
A git dependency usually points to the most recent version. When stuff breaks, the problem must be fixed, but a Cargo.lock file makes it possible to replicate the built, but this is usually done at application level.

There are 2 groups of people who should have permissions to publish on crates.io:

Maintainers of the project (or those who are most actively contributors past previous months)
Admin people (those who contribute actively across repos)

These people are expected to follow the guidelines for best practice on crates.io https://github.com/PistonDevelopers/piston/issues/773


added --force-virtio switch

Hi Brian,
I've added the --force-virtio switch to the command arguments and modified the flow so that it is only forwarded on calls to convert to libvirt output provider.
Can you please have a look at it?
Thanks in advance!
Chris

Hi Chris,
Thanks for taking on this work! A few thoughts for you:

I don't think you need to touch box/box.rb, box/libvirt.rb, or box_loader.rb at all. In mutate.rb
what if you pass the force_virtio option through Converter::Converter.create instead of through output_loader.prepare_for_output? Then I think the changes could be confined to that file converter/converter.rb, and converter/libvirt.rb.

You could add support for this to the kvm converter too. Take a look at converter/libvirt.rb and box.xml.erb in the templates dir.

And a few thoughts for me:

box_loader.rb is such a mess.

Should I just drop support for vagrant-kvm? The project is abandoned.

Sorry to see you abandon this.


Training on VQAv2

I see that there is option to choose dataset to be VQA. Wanted to know if I could train it on VQAv2 and if so how?

Hi yea definitely! The flag is --dataVer, you can set it either to 1 or 2, the default is actually 2, haven't tested it recently on 1. I made local test some time ago and it was working with for both with no problems so I strongly believe the online version will work for you smoothly but just in case I would like to validate that so I'll get back to you in a couple of days with the official scores I get for VQA 1 and 2!


MQTT Monitoring

Describe the feature request
Have you thought about adding MQTT monitoring? I'd like to be informed when specific topic stops receiving data (i.e. last publish was more than X minutes ago).
Another option would be to raise alarm if value published to specific topics is not what we expect (too low, to high etc.)
Why do you personally want this feature to be implemented?
I have an IoT device which sometime hang and need to be manually reset.
How long have you been using this project?
3 years
Additional information
No response

This is my first time hearing about MQTT, but sure! In the best of worlds, Gatus would support all protocols.

#596 seems like an idea way to add support for things like MQTT.  If the adoption of a custom check grows, then it could be an ideal candidate to fold into the core Gatus codebase.


⚠ Avatar API Endpoint has degraded performance

In c7f272c, Avatar API Endpoint (https://avatar.roblox.com/v1/avatar-rules) experienced degraded performance:

HTTP code: 200
Response time: 2031 ms

Resolved: Avatar API Endpoint performance has improved in 03ae0bd.


Global built-in constants are not available in shader include files.

Godot version
v4.0.beta3.official [01ae26d31]
System information
Windows 10
Issue description
Shader global built-in constants (PI, TAU, E) are not available in shader include files.
Steps to reproduce
Create a .gdshaderinc file.
Create a function.
Try referencing any of the global constants (PI, TAU, E).
Minimal reproduction project
ShaderincMissingGlobals.zip

cc @Chaosus


Loading order of ftplugins

Hello, thank you for this very useful plugin.
I'm trying to move some autocmds to ftplugins under ~/.vim directory, and I notice the following when using --startuptime:
239.209  008.969  008.969: sourcing ~/.vim/bundle/vim-markdown/ftplugin/markdown.vim
239.454  000.023  000.023: sourcing /usr/share/vim/vim74/ftplugin/markdown.vim
251.686  011.999  001.993: sourcing ~/.vim/ftplugin/markdown.vim

Is this expected? Or shouldn't user-defined ftplugins be loaded first so I can disable ftplugins with b:did_ftplugin ? For example:
251.686  011.999  001.993: sourcing ~/.vim/ftplugin/markdown.vim
239.209  008.969  008.969: sourcing ~/.vim/bundle/vim-markdown/ftplugin/markdown.vim
239.454  000.023  000.023: sourcing /usr/share/vim/vim74/ftplugin/markdown.vim

Not sure if this is related to https://github.com/junegunn/vim-plug/issues/25

Sorry this was an error in my configuration. Please ignore.


velocity content header route throwing an exception in spanish locale

Bug report
Title
velocity content header route throwing an exception in spanish locale
Issue Description
velocity content header route throwing an exception in spanish locale
Preconditions
1. framework Version- master

Steps to reproduce
1. step1- set admin default locale to 'es' from .env
2. step2- now add the velocity content header and edit

Expected result
velocity content title should show in all locale with throwing an exception
Actual result

@shubhwebkul
The header content edit page title is not saved in any locale.
https://prnt.sc/tzf64x

fixed


Clarified Installation Instructions

Fixes #1338 and #1384

Docs Build status updates of commit fba2063:
:clock10: Full build: the average full build time is 26 min(s) 26 sec(s), this is based on the last 372 build(s) for this repository.

Docs Build status updates of commit fba2063:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs-conceptual/azps-3.8.0/install-az-ps.md
:white_check_mark:Succeeded
View (azps-3.8.0)

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.


add android ci.

Travis ci not friendly with docker. So I do this in circle ci.
Build log: https://circleci.com/gh/gengjiawen/react-native-vector-icons/4.

Hey, tests of native integration would definitely be cool and worthwhile to have, especially for android.
However this PR doesn't really test anything, since it does not apply the code from the current commit, but pulls it from NPM. Would you mind changing that? Also what do you think about doing something more than just testing if it compiles, like using jet or detox?

Closing as it doesn't serve much of a purpose without also adding tests. We'll revisit the subject in the future though.


Remove Default Subnet

In many cases we come across, the vnet has a different address space to the subnet default value.
We should remove the default subnet and either create no subnets (default to []) or require the entry of a subnet.
https://github.com/ans-coe/terraform-azurerm-virtual-network/blob/fddd2b0015788f838522c2ca60dc7661b3f92dca/variables.tf#L102

@ANS-TH @jk-ans - Invited to discuss

default = []
default = null
or
leave out default entirely (not reccomended)

Default to [], as this should eliminate any problems with regards to dealing with how the value is presented to the azurerm_subnet resource. If we were to use null, we'd have to use a little extra needless logic to handle that, whereas an empty list would just create the VNet and put the rest of it in the hands of the user.

Was thinking about setting the default as the first /24 in a given var.address_space
cidrsubnet(var.address_space, 24-split("/", var.address_space)[1])
Spoke to @djackson-ans and went with [] instead


Add unit handling w/ and w/o pint_pandas

Codecov Report
Patch coverage has no change and project coverage change: -10.45 :warning:

Comparison is base (1583a04) 72.10% compared to head (b793771) 61.66%.

Additional details and impacted files
@@             Coverage Diff             @@
##             main      #17       +/-   ##
===========================================
- Coverage   72.10%   61.66%   -10.45%
===========================================
  Files           8        9        +1
  Lines         484      566       +82
  Branches      117      128       +11
===========================================
  Hits          349      349
- Misses        111      193       +82
  Partials       24       24

Impacted Files
Coverage Δ

src/pandas_indexing/units.py
0.00% <0.00%> (ø)

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


TypeScript generated code

Building on #13, we should generate TypeScript types and code for deserializing a source file to these types.

Generate TS classes of each node that extends from a base Node class (should be possible to do node instanceof ClassDecl)
Generate code for getting a node's children.
Generate code that will take a deserialized JS object, set the prototype of each object to the appropriate type, set the node's parent (in the case of source file, undefined), then traverse all the child nodes doing the same.

cc @magurotuna, I wouldn't mind working on this, but if you would like to do this work let me know! For me it might take a few weeks.

Sounds great!

I wouldn't mind working on this, but if you would like to do this work let me know! For me it might take a few weeks.

Would be happy if you could do :) To be honest, I'm not no much confident in implementing all of them. Taking some time doesn't matter to me.
I will work on #14 first 😉


What happens when try to download while not having connection to the internet

See title

Raise an exception and close the part of the program. Alternative: just write empty tables. Than we do not have to handle the missing internet connection specifically and will simply not find anything in the data bases. This would go silent though, something we probably do not want.
Additionally, the pubchem requests require internet access as well, so maybe we do an extra check, if internet is available and skip all those part where it is needed, unless one can connect.

[ ] What happens if there is no internet in the first place
[ ] What happens if the connections drops during download (Inconsistent state?  Database folder there but only part of the DBs are fully downloaded, Database folder there but one db is only partially downloaded, malformed database file. Handle those
[ ] Anything else?

I checked all cases except the one where we drop connection during a download and have a corrupted db. not sure if python even allows this to happen though


Add Queue mode support at server side adding a PresenceService

I taked into account most of the remarks.
If no objection I plan to integrate this tomorrow in master.

This PR is now integrated in master.
Queue mode is activated by default.
It is possible to return to previous behavior, using LeshanServerBuilder.disableQueueModeSupport


[Bug] RepositoryDispatch misses types parameter

Action
https://docs.github.com/en/actions/writing-workflows/choosing-when-your-workflow-runs/events-that-trigger-workflows#repository_dispatch supports custom activity types to differentiate webhook calls.
Expected
A types parameter with type List<String> to configure the types.
Actual
No parameter present
Library version
v2.3.0

Thanks!
To be added to https://github.com/typesafegithub/github-workflows-kt/blob/d06b800de71dcba314451fd799a0c02b3a7d9b27/github-workflows-kt/src/main/kotlin/io/github/typesafegithub/workflows/domain/triggers/RepositoryDispatch.kt#L10 and passed down the serialization path.

Wow, that was fast, thanks. :-D


[Toggle Desktop Visibility] Doesn't work on macOS Sequoia 15.1.1

Extension
https://www.raycast.com/Haojen/toggle-desktop-visibility
Raycast Version
1.86.1
macOS Version
Sequoia 15.1.1
Description
Doesn't work on macOS Sequoia 15.1.1.
When I run the command Toggle Icon Visibility, the message says Desktop Icons Hidden, but the icons are not actually hidden. I tried relaunching Raycast app, but it still doesn't work.
Steps To Reproduce
Install Toggle Desktop Visibility
Open Raycast, select Toggle Icon Visibility command, hit Enter
Expected: Desktop icons should be hidden
Actual : Desktop icons are still visible
Current Behaviour
Desktop icons are not hidden
Expected Behaviour
Desktop icons should be hidden

Hi @shrinidhi4u
thanks for opening this issue. However, I could not reproduce this error (see below). could you please send a screen recording of your desktop as you try to use this command?

I am at macOS Sequoia 15.1.1

@raycastbot close this issue
@shrinidhi4u please reopen the issue if you are still having issues with MacOS 15.2.


Handbook: Update Personas

Description

Related Issue(s)

Checklist

[ ] I have read the contribution guidelines
[ ] I have considered the performance impact of these changes
[ ] Suitable unit/system level tests have been added and they pass
[ ] Documentation has been updated

@Yndira-E Can we get a new image for Steve and Sarah?

@joepavitt Can you take a look at this and see why the tests aren't passing?

I don't work Mondsys. @knolleary can look today, otherwise, I can tomorrow

@joepavitt Sorry forgot, but It can wait.  Nothing that needs to go out today.

@gdziuba done!

Problem was fixed in https://github.com/FlowFuse/website/pull/2378 - will rebase this branch so that it passes

I feel like we have quite a few persona's now, @gdziuba @joepavitt Can you remove a few we shouldn't focus on?

Agree @gdziuba


Bump react-native-babel-preset to ^5.0.0

We opt in to version ^5.0.0 of the React Native Babel Preset, as required after the bump to Babel 7. This fixes the Objective-C end-to-end test failure in master. (Fixes #19538)
See https://github.com/facebook/react-native/commit/34bd776af2f2529009188fced91083203a48fd40#commitcomment-29024085 for prior discussion.
There have already been several changes made to the repo during the transition to Babel 7. This PR should bring all tests back to   green and allow us to move forward with the 0.56 branch cut.
Once 0.56 makes it to stable, we can change @latest on npm to point to v5, and undo the change made to init.js as part of this PR.
We also bump our tests to use Xcode 9.3 and iOS 11.3.
cc @grabbou, @qfox, @chirag04
Test Plan
Wait for Circle CI to run: https://circleci.com/workflow-run/e39a66d7-bf8a-4b31-a22f-eef30a2c53bc
Release Notes
[GENERAL] [BREAKING] [Babel] - Bump React Native Babel Preset version used by RN CLI to Babel v7 compliant release

We're green:

In order to fix e2e, I had to do several things:

Install Node 8 on the Mac machine, as Node 10 was being used by default which was preventing us from running the e2e tests. We should ensure Node 10 is fully supported (https://github.com/facebook/react-native/issues/19229).
Only run iOS end-to-end tests, as both flow and jest were failing. We already run Flow and Jest in CI, so this should be safe for now. We should still look into why Flow and Jest are failing when run as part of the e2e suite, in order to make this a true e2e test.
Changed how the simulator is booted, to get around a xcrun error

Throughout this process, I noticed that iOS tests (the regular, non e2e) are failing silently - Circle shows green, but you can see a list of failures at the top of the test summary. This appears to have been the case for a while, so we can look into that in a separate PR.
Finally, Android tests are fixed on Master, so any failure in test_android in this PR can be ignored.

Exciting, once this gets merged, I'll cut a new branch and release next release candidate straight away. Thanks, @hramos for amazing work around the releases recently.
I wonder what we can do in the future to make sure that if any commit breaks anything in React Native, it gets rejected automatically. Then, the author has to fix the failures in order to land it again. Otherwise, the current approach sort of defeats the purpose of tests as a complementary tool during development.


too scary

too scary

this has to be my favorite issue. thanks.

actually, this is so good that I'll keep this issue open


Issue #23 - Fix wording and grammar

This covers everything mentioned in Issue #23 except for editing the map generator section on the main page. I could not think of a way to word it without making the Map Gen section unreasonably long. This is my first time contributing to open source projects, so if I am doing anything incorrectly, please let me know.

You also add a gallery to the customize page, you should state that :P

In the downloads page, I linked to the mods section of the customize page. Should it instead point to the forums like the Subgames link above it does?

:+1:
I think to the customize page is good, as it provides a description of how mods etc work.

Merged.


ActionGroup documentation

Closes RSP-1620
✅ Pull Request Checklist:

[ ] Included link to corresponding Issue.
[ ] Added/updated unit tests and storybook for this change (for new code or code which already has tests).
[ ] Filled out test instructions.
[x] Updated documentation (if it already exists for this component).
[ ] Looked at the Accessibility Practices for this feature - Aria Practices

📝 Test Instructions:

🧢 Your Team:
rsp core

looks like clearing the type definition from the fowardref in ActionGroup.tsx fixes the docs issue, but breaks the linter. Maybe might have a unknown type digging into processExport to see


ZCL_ABAPGIT_SERVICES_GIT syntax check error

DUMP while trying to update a repo.
I have downloaded  manually ABAPGit from "https://raw.githubusercontent.com/abapGit/build/master/zabapgit.abap" today (23/04/2018), after that I've updated with the latest repository objects and when checking syntax some errors appears.

Pull after copying manually ZABAPGit.

During upgrade ZABAPGIT has to be updated

Executing syntax check

ZCL_ABAP_GIT_SERVICES_GIT seems has some kind of problems.
The problem is that, after upgrading, I can't perform commits nor pulls.

sorry, I just found #1302 . I'll check if solution works also for me.

Done, I have to do some manual things.

Move to package ZABAPGIT_UI following ABAP Classes
zcl_abapgit_services_abapgit
zcl_abapgit_services_git
zcl_abapgit_services_repo

Replaced ABAP Class with the content from "https://github.com/larshp/abapGit/blob/master/src/ui/zcl_abapgit_services_git.clas.abap"

After that I was able to do a Pull request from GitRepo.
Thanks

sorry about the trouble, hope we will find the bug


Custom public properties on a Form type do not show in PropertyGrid of designer

Steps to reproduce:

Create a new form and add a simple public property on it, say "public string Test {get; set;}"
Rebuild and open form in designer.
The new property is not shown in the PropertyGrid.

Now, repeat the same with a UserControl:

Create new UserControl and add the same simple property Test.
Create new Form and drop the new UserControl on the form.
Rebuild and open form in designer.
The new property is now shown in the PropertyGrid for the UserControl.

Palli

Well, in the first case, the Form is only built, so what you see on the screen at design mode, is the base class of that form, so of course it does not have that property at that point.
In contrast, if you have built a UserControl, and use it in a Form, then the UserControl has been built. And therefore, it has the property.
This is By Design, and it is exactly how the classic Framework Designer also works.


Add disableAutoUpdate to cloud connector env variables

Summary: Auto-update disabling broke with the removal of one of our configmaps, which was done as cleanup.
This PR adds the auto-update env var to the cloudconnector template YAML so that if a user deploys with autoUpdate as true, this will be passed in properly.
Relevant Issues: N/A
Type of change: /kind bug
Test Plan: Generate the Vizier YAML templates, inspect the template to ensure it looks correct.

For some reason I recalled that the pl-cluster-config was cleaned up, but looks like it still exists. So there is likely another problem here--closing out this PR since it's not necessary.


Incapable of returning a fetch Stream from a trait method generated by tonic-build

Hi all, I ran into what seems to be an unsolvable issue combining sqlx and tonic-build generated trait methods which use the server-streaming feature of gRPC/proto3, with the following example code to illustrate what's going on:
#[tonic::async_trait]
impl FooServer for FooServerImpl {
    // Associated type to refer to the `Stream` in `get_bars`
    type GetBarsStream = BoxedStream<'???, Result<Bar, tonic::Status>>;
                                  // ^ this lifetime can only be `'static`, there's no way to refer to `&self` at this point
                                  // unless `tonic-build`/`prost-build` are modified to add a lifetime to every associated stream type
    async fn get_bars(&self, request: tonic::Request<GetBarsRequest>) -> Result<tonic::Response<Self::GetBarsStream>, tonic::Status> {
        Ok(tonic::Response::new(
            sqlx::query_as::<_, Bar>("SELECT ... FROM ... WHERE ...")
                .fetch(&self.db_pool).map_err(|_| ...)
                    // ^ since all executors are references, this causes a lifetime error since there's no way to make the stream `'static`
                    // which as far as I can tell, makes doing this literally impossible without writing some very awkward `unsafe`
                    // it's also not possible to `acquire` a connection inside of the method then use it in the
                    // stream since its then a dangling reference on return
        ))
    }
}

Is there any possibility that this can be resolved by somehow creating some kind of owned Executor? I tried, but obviously immediately ran into issues with it requiring longer lived lifetimes from consuming self. I'd really like to not need to return everything all at once but it seems unavoidable while using sqlx.

This is a big issue for us as well.
Can we get some visibility on this ticket?

For those that find this issue, I solved this problem by using:
let pool = Box::leak::<'static>(Box::new(self.pool.clone()));

The logic being:
a) It's safe to leak memory (according to rust).
b) The pool is alive as long as the application is alive.
c) We explicitly call pool.close() when the application closes, so even if there's something keeping the app from closing, the pool itself will reject new requests for connections..


fix(webpack): only call configureBuild once for develop client bundle

Coming from https://github.com/xing/hops/issues/636
When target is develop, configureBuild is executed twice for the client bundle.
That is not the case for target build.

Thanks for this PR! I would love to understand the problem we are addressing with it, though.
This actually is the expected behavior: https://github.com/untool/untool/blob/master/packages/webpack/mixin.core.js#L40
If this is an actual problem, your solution is certainly valid and I’d be happy to merge it.


docs: Adding 05-emotion-palette

Adding analysis of emotional palette of famous Indian directors Sanjay Leela Bhansali and Mani Ratnam.

Merged in #411


Placeholder issue for image uploading

With the new JWT-protected images on githubusercontent, it's easy to host an image by creating an issue in the same repo and dropping it in there to get a persistent URL.

It also makes a handy markdown line for the image, which syntax I can never remember

@ahpook okay to close this one?

@zkoppert yep. i'll probably continue to use it for images but no need for it to stay open.


Set api-endpoint to host for incluster deployment

This PR is a proposed fix for issue #394

Ah cool, thanks for the PR! testing now :D

Thank you so much for the lightning-fast review & merge, @thebsdbox ! ❤

@thebsdbox How should the pod manifest look like with inCluster set to true? Somehow, my static pod does not seem to start.
spec:
  containers:
  - args:
    - start
    - --inCluster=true

Is this how it should look like or should the manifest have the flag --inCluster supplied in a different way?

export KVVERSION=v0.4.4
alias kube-vip="docker run --network host --rm ghcr.io/kube-vip/kube-vip:$KVVERSION"
kube-vip manifest daemonset     --interface $INTERFACE    --inCluster    <other flags>

A new release tagged main should appear soon with your code changes in it.

@AxiomSamarth @thebsdbox do we need to handle the following case too - if len(path) == 0 && inCluster?
Also, should we use hostname = os.Hostname():port instead of hostname = kubernetes:port? This way we wouldn't break existing use cases where pod spec isn't using the following field:
  hostAliases:
  - hostnames:
    - kubernetes
    ip: <IP_ADDRESS>

wdyt?

@AxiomSamarth @thebsdbox do we need to handle the following case too - if len(path) == 0 && inCluster by setting path = filepath.Join(os.Getenv("HOME"), ".kube", "config")?
Also, should we use hostname = os.Hostname():port instead of hostname = kubernetes:port? This way we wouldn't break existing use cases where pod spec isn't using the following field:
  hostAliases:
  - hostnames:
    - kubernetes
    ip: <IP_ADDRESS>

wdyt?

@thebsdbox I think, the if condition to check if the inCluster flag is set is not needed here.
We can have the hostname = fmt.Sprintf("kubernetes:%v", port) as default which gets updated to hostname = fmt.Sprintf("%s:%v", id, port) it it is out-of-cluster here.
This way, user will also not have to undergo the process of setting up serviceaccounttoken, ca.crt,  KUBERNETES_SERVICE_HOST & KUBERNETES_SERVICE_PORT  to generate the clientSet when inCluster flag is set.
WDYT?


Zoom graph fuctionality showing incorrect labels

This is a bit of an oddball, I can't re-create the specific graph issue the origial user reported but can create the problem on a different graph.
1: select moray from the map
2: select each of the different property types
3: goto average prive by buyer status and click the graph to enlarge - legend shows propety types rather than buyer status (see attached)

Avge price by funding status also appears to have the same problem

I think a better refinement of the problem is:

show a zoom graph
close it
show a different zoom graph
the legend from the first graph is retained on the second graph

confirm fixed


Context Awareness: Implement commit status contexts

Back in March, Github released the Combined status API. As it nears the end of its developer preview period, the organization I work for has found need to start leveraging them. From what I can tell, this is working as intended. But GitHub doesn't seem to be reflecting these in their UI yet. :cry:
That said, I think this could still be useful for teams that want it so I decided to go ahead and open the pull request. This PR can't be merged until release 1.55 of the github-api jenkins plugin is out the door. The java dependency itself has been released, but it looks like when they tried to release the plugin things didn't go as planned? I've emailed the author about it, and am still waiting on a response.
So, things we need:

[x] github-api-plugin 1.55 being released.
[x] Another pass through the tests, because I'm about 99% sure my brain isn't functioning correctly today. heh.

closes #155

It would be really great to have this soon. Thanks for the implementation @farmdawgnation !

@valdisrigdon Can we get some attn here? :)

Isn't the primary repo on https://github.com/jenkinsci/ghprb-plugin now? I see @DavidTanner made a commit there one day ago.

Well it would have been nice for someone to tell us. heh

I will check in on this tonight/tomorrow.  Sorry, I changed roles so I am not as active with jenkins.


Fix Pinball system image links

The image links are broken. This fixes the problem.

Hi @Acehaidrey ,
What do you think about this PR? Thanks in advance.

thanks!


gene expression annotations without extensions

Number of gene expression annotations

There are 37724 annotations and 37720 have extensions (4 annotations
don't have extensions).

could you report these in the logs. they should all have something....

Here are the annotations in the contig files that end up with no extensions in Chado:

FT                   /systematic_id="SPBC1734.02c"
FT                   /controlled_curation="term=gene expression, RNA level;
FT                   qualifier=constitutive; db_xref=PMID:1538696;
FT                   date=20120425"

FT                   /systematic_id="SPBC646.09c"
FT                   /controlled_curation="term=gene expression, RNA level;
FT                   qualifier=present; evidence=ECO:0005653;
FT                   db_xref=PMID:11071923; date=20121118"

The other two should be warnings already - the CHEBI term is not in Chado:
FT                   /systematic_id="SPCC757.07c"
FT                   /controlled_curation="term=gene expression, RNA level;
FT                   annotation_extension=in_presence_of(CHEBI:142517);
FT                   qualifier=increased ; evidence=ECO:0000231;
FT                   db_xref=PMID:29900664; date=20181113"

FT                   /systematic_id="SPBC215.05"
FT                   /controlled_curation="term=gene expression, RNA level;
FT                   annotation_extension=in_presence_of(CHEBI:142517);
FT                   qualifier=increased ; evidence=ECO:0000231;
FT                   db_xref=PMID:29900664; date=20181113"

I made a separate issue about this: pombase/pombase-chado#704

[ ]  the qualifier in the first one needs to change to "constant"
[ ]  I don't see a probelm with the second one. Eagle eyes @mah11 ?

Neither one has an annotation extension (i.e. anything from the  'annotation_extension=during(GO:xxxxx),in_presence_of(CHEBI:xxxxxxx)' part of the pattern). That seems tolerable for one with the qualifier 'constant', but the second one doesn't say anything useful without an extension.

If i understood correctly the other 2 are fixed.

More accurately, the other 2 were never broken ;)
but yeah, this is done.


add includePaths option to plugin-sass

load-path should be used to provides Sass paths to look for imports. imports are not necessarily in the same directory as the compiled sass file. giving it path.dirname(filePath) will only assure that its will find imported file in same directory.
includePaths work the same as dart-sass js-api includePaths
Changes
Add includePaths to plugin options.

Testing
src/path/to/folder/_partial.scss
$primaryColor:#333

src/other/path/to/something/App.scss
@import 'partial.scss'

.App{
background-color:$primaryColor;
}

Case 1:

snowpack.config.js
...
plugins: [
...
[
'@snowpack/plugin-sass', {includePaths: ['src/path/to/folder']}
]
]
}

result:PASS

Case 2:

snowpack.config.js
...
plugins: [
...
'@snowpack/plugin-sass'
]
}

result :FAIL &
Error: Command failed with exit code 65: sass --stdin --load-path /pathToProject/src/other/path/to/something
Error: Can't find stylesheet to import.
  ╷
1 │ @import 'partial';
  │         ^^^^^^^^
  ╵
  - 1:9  root stylesheet

Docs

Name
Type
Description

includePaths
array
This array of strings option provides load paths for Sass to look for imports. Earlier load paths will take precedence over later ones.

Oh I see, so the CLI can support multiple --load-path options. That explains why we were having so much trouble getting this right in the past, since one value isn't going to make everyone happy. +1 for adding this config.
@Danzo7 can you add a mention of this new config to the plugin README?

Oh I see, so the CLI can support multiple --load-path options. That explains why we were having so much trouble getting this right in the past, since one value isn't going to make everyone happy. +1 for adding this config.
@Danzo7 can you add a mention of this new config to the plugin README?

Oh wait, I see it in your PR body. I'll add it myself post merge. Thanks for the PR!

Oh wait, I see it in your PR body. I'll add it myself post merge. Thanks for the PR!

Actually: when adding the docs for this I realized that we already have compilerOptions for passing args to the Sass CLI directly. I've removed the support added in this PR and instead documented compilerOptions.loadPath for use.
If you can try out 123f2c96 and confirm that this works for your use-case, that would be great.

Actually: when adding the docs for this I realized that we already have compilerOptions for passing args to the Sass CLI directly. I've removed the support added in this PR and instead documented compilerOptions.loadPath for use.
If you can try out 123f2c96 and confirm that this works for your use-case, that would be great.

Sorry I didn't notice your question ,good job! I will try it right now

Sorry I didn't notice your question ,good job! I will try it right now

there is a small error in your code you need to add [] to the params in line 122:
parseCompilerOption([flag, val]);

there is a small error in your code you need to add [] to the params in line 122:
parseCompilerOption([flag, val]);

Actually: when adding the docs for this I realized that we already have compilerOptions for passing args to the Sass CLI directly. I've removed the support added in this PR and instead documented compilerOptions.loadPath for use.
If you can try out 123f2c9 and confirm that this works for your use-case, that would be great.
OK I make PR  that fix the error :
https://github.com/snowpackjs/snowpack/pull/2547

Just released in v1.3.1! 🎉


PowerShell notebook is sometimes one command behind in results

Azure Data Studio Version: 1.15.0

Steps to Reproduce:

Open a PowerShell notebook that has been run before
Run the first cell.
9/10 times this gives some weird result. It's not always the same either. Here's an example:

Before I run, just opened the notebook that has been run previously:

Now, I run the cell once:

Now, I change nothing and run the same cell again:

But now, all my other cells' results are off by one. Like, if I now run the next cell, the first time I run it:

But the second time it works. If I don't run it a second time, the result will carry through to the next cell I run. E.g. below:

I really want to use these PowerShell notebooks but this bug is blocking in a big way. Feel free to reach out to me (antho@). FYI @chlafreniere @yualan
Thanks,
Anna

@amthomas46 which version of the python kernel do you have? The python package is called "powershell-kernel", and 0.1.3 is the latest.
Just making sure that you're on latest 😄

Ah, I am one behind, I will update and try again and then update here. Thank you!

OK so I've tried this a few times. Most times it works! Every once in a while, I still have some weirdness.

It actually seems like I can repro it if I click "Run" on a cell, before the kernel has loaded.

Then it's the same behavior as I describe initially.
Now that I know when this will happen, I think I am unblocked. But, perhaps eventually clicking the Run button too soon in a PS notebook shouldn't result in an error for the lifetime of the active notebook.
Thanks so much for your quick help, Chris!

Just adding an update since I'm doing lab environment testing. With a "fresh" install on a new machine, this issue persists.

I'm suffering from this too. But, for me, it happens 100% of the time

@amthomas46 do you also still get this behaviour on the latest version? 1.22.1?

Hi @ClaudioESSilva, we just updated the PowerShell Kernel. Using the python package manager in ADS you can search for the new version of the "powershell-kernel", and 0.1.4 (latest) which can be installed.
Please go ahead and try this new version, and let us know if this issue persists. Also please make sure that the kernel loads properly, before running the cell as that is something @amthomas46 noticed.

Hi @VasuBhog thanks for the feedback.
I have installed it, close and open ADS. But I have the same behavior.

Hi @ClaudioESSilva I'm sorry to hear that. I am not able to repro on my machine shown below (using ADS 1.22.1). However, another possibility is that pip is still caching a previous version of the kernel.
I would do a pip uninstall powershell-kernel via the terminal in ADS. After successful uninstallation, I would close ADS and reopen it. Once you switch the kernel to Powershell, the package manager should prompt you to install powershell-kernel. Please install it and see if the issue persists.

Hi @VasuBhog,
It seems much better. Working 9/10.
I have done the uninstall, close ADS and reopen.

Open a notebook, waited for about 30 sec
1st block of code was ok. However, when running a 2nd one (on the same notebook) wasn't

Created a new one, typed docker ps -a worked correctly.
Closed and reopen, it's working.

Closed again. Opened ADS, open notebook, waited for the integrated console, run and it worked.
NOTE:
The pattern seems to be the 1st time I open ADS, open notebook and when "PowerShell" appears on the kernel but the integrated console is not yet fully loaded, it misbehaves.
I will be doing some runs on the next days. I will be back with more feedback.
Thank you!

@ClaudioESSilva that is great to hear!
Thank you for the feedback, will look into if it is tied to the integrated console loading. Glad it is working, and look forward to any other feedback you may have :)

I just wanted to add that I am now facing the same issue yet again. I was on the most recent 0.1.4 for powershell-kernel. I uninstalled like you said, but now in a new notebook:

so that's the older package? so then I had to go in and update it. And then it worked.
I'm not sure what triggered the regression but it was in 0.1.4

Hi @amthomas46, we currently tell ADS to use the 0.1.3 version, as we would like to get more feedback from users who upgrade and test the 0.1.4 version. We plan to upgrade ADS to utilize the new version (0.1.4) in the November release, once we have more feedback.
You can install the new version of the powershell-kernel (0.1.4) via python package manager in ADS.

Ah OK that makes sense. Well, for your testing, I have had some issues like above with 0.1.4. If I can get another repro, I'll ping you.

I have been running my demos after change to 0.1.4.
I haven't seen any rollback to the 0.1.3.
However, I can say that still happens. Would say 1/10 of the times.
I try to find a pattern so I can share in which conditions I see it happening.

Just to put here some examples that happened during my GroupBy presentation.
https://youtu.be/dggyVt9RSCM?t=628
You can see the behaviour explained before and also the breaks that happen during the streaming of data (a different bug I would say)

@ClaudioESSilva, thank you for letting me know! I will continue to investigate this issue. It seems, for now, the most reliable method thus far is to ensure that the PowerShell Integrate Console is shown in the Terminal after running the first cell. If not then this can cause issues, and indicate that the powershell kernel has not fully loaded.

I'm experiencing this as well, and will have to attempt the workarounds suggested later this week.


Feature Request - SkinScale

I use VisUI.load(SkinScale.X2) but find it too small on my mobile (nexus 5)
I'd be interested in scaling that fits the higher density Android range of screen sizes i.e. https://developer.android.com/guide/practices/screens_support.html
ldpi (low) ~120dpi
mdpi (medium) ~160dpi
hdpi (high) ~240dpi
xhdpi (extra-high) ~320dpi
xxhdpi (extra-extra-high) ~480dpi
xxxhdpi (extra-extra-extra-high) ~640dpi
p.s. love your work :)

Unfortunately the current workflow with Inkscape makes it hard to produce more versions of skins because it lacks option to batch export files in multiple sizes. VisUI skins currently consists of 104 images, keeping that many sizes of skin up to date would be tedious without option to automatically export everything. Ignoring that I'm not sure whether more versions should be included in core. They could be provided as separate module since they are pretty easy to include. I'll leave this one open since this is a maybe to be done future idea.


Double ended queue in Python

Fixes #6394

Have you read the Contributing Guidelines on Pull Requests?
Yes
Description
Double ended queue in Python using fixed size array
Checklist

[x] I've read the contribution guidelines.
[x] I've checked the issue list before deciding what to submit.
[x] I've edited the README.md and link to my code.

Related Issues or Pull Requests
Fixes #6394

@ankitaggarwal23 , @anushkrishnav , @bhav09 kindly review the PR


is there a way to have just time picker without date picker in jexcel

I need to have a cell which can have time picker , Date is not required. Is there any way to implement this

You can find that on the second example in this link.
https://bossanova.uk/jexcel/v4/examples/column-types


feat: add edit on github

fix #756
Maybe, there is some missing types for app.config.ts.
And there is no documentation but if this PR is ok for you, I write it.

Hello @Barbapapazes ; I let the deployment run and will look into this PR for the next release. :)

Amazing work, I just added the docs part for it and the $schema!
It's worth looking at it if you want to add it yourself in your future PR or even your own themes!

You're welcome! 🙌


Use fkeys as function keys for non-apple keyboards?

I have an external keyboard (Kanex bluetooth), and the built in option "Use all F1, F2, etc. keys as standard function keys" doesn't have any affect on this keyboard.  In Karabiner 10.x, I could get the desired effect by enabling a series of options under "Change Functional Key (Brightness, Volume, ...)"; I think the plist settings were "remap.consumer_to_fkeys_f1", etc.  Is there any way to do this with Elements?
While I'd like to fix all Fkeys, F1 is of special importance to me, as I need to use CMD-F1 in one of my apps; without the change I need to use CMD-FN-F1.  If I forgot the FN key, they system enables monitor-mirror mode, which really screws up my 3 monitor system :)  The event viewer shows two events when I press CMD-F1:
eventType:FlagsChanged    code:0x37       name:left_command    flags:Cmd                                misc:
eventType:SysKeyDown      code:0x3        name:display_brightness_decr...   flags:Cmd                                misc:

I can't find any way to remap "display brighness decrement", or to map all consumer keys to F keys.

I'd like to advocate for this as well.  I have a Logitech K780 keyboard which puts itself into "mac mode" when it detects macOS.  In mac mode, fn-up and fn-dn control screen brightness and there is no support pgup and pgdn.  In PC mode the keyboard sends pgup/pgdn as expected.
I don't use this on an iMac, and I don't change screen brightness all that often even when I do.  I really need those keys in the terminal.  Naturally logitech has about zero interest in fixing this, so I turn to Karabiner.

This isn't stale, it simply isn't resolved.  Considering that I only ever used original Karibiner to control my F-Keys, it's a major issue for me.  Please do not close.

Still not stale, still not resolved.  This is a daily source of issues for me.

is there actual problem behind it? or can we just pull request?

I didn't get into details, but if i recall updating just configs didn't work, or i just have adapted to existing usage of app :/

Still not stale, still not resolved.  Even though this has been open for over a year with no action, I continue to hold out hope that this will be fixed.  F keys are such a major requirement for developer tools.

I have the same problem with an external USB keyboard


.env.web file?

I am setting up the PROD version of ashirt b/c the dev version was very interesting to play with but the instructions for the initial logon seems to be confusing the heck out of me.
" After the first users are registered the admin should change AUTH_SERVICES_ALLOW_REGISTRATION=ashirt to #AUTH_SERVICES_ALLOW_REGISTRATION=ashirt in .env.web", I am able to get the initial admin account created no issues but when looking for the .env.web file to comment out there does not appear to be a file with that name.
Am I missing something?

@kanderg Saw in discord that you figured this out. Is this ok to close?

Yes I apologize I forgot to go back and close this. It was an id10 error with a spelling on one of the s3s.
On Mon, Mar 4, 2024 at 13:42, Joe Rozner @.***(mailto:On Mon, Mar 4, 2024 at 13:42, Joe Rozner < wrote:

@.***(https://github.com/kanderg) Saw in discord that you figured this out. Is this ok to close?
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you were mentioned.Message ID: @.***>


Navigation

bump to version "electrifier alpha 4 Windows, build v1.24.331". Alter NavigationView

All checks have failed


Add handling of null response content

If an error response comes back with no content, the json factory throws a
NullPointerException when it attempts to read the response, which hides the
response code from the client.
fixes #166

I signed it!
Sorry for not following templates to the letter, but it's a pretty simple change.


Improve Active Apps detection

Here is a pull request that improves Active Apps detection (process used on GPU).
For exemple, it no more randomly display wrong app in dashboard when using ffmpeg (HW) which is not in the supported apps list.
It now retrieves informations from docker container where process is in (pid, cgroup, docker inspect). Supported apps list is no longer necessary.
Only works with Nvidia GPUs as before.
AMD and Intel GPUs need other "top" binaries that can expose process usage. I don't have GPUs from these brands, so I can't test them.

Closing because moving changes from master to a new branch.


Ensure virtualenvwrapper type is displayed in the quickpick list

They were being listed as Unknown environments earlier.

Codecov Report

Merging #15114 (53f3420) into main (3ce3547) will decrease coverage by 0%.
The diff coverage is 0%.

@@          Coverage Diff           @@
##            main   #15114   +/-   ##
======================================
- Coverage     65%      65%   -1%
======================================
  Files        559      559
  Lines      26284    26285    +1
  Branches    3730     3730
======================================
  Hits       17127    17127
- Misses      8467     8468    +1
  Partials     690      690

Impacted Files
Coverage Δ

src/client/pythonEnvironments/info/index.ts
39% <0%> (-1%)
:arrow_down:

src/client/pythonEnvironments/legacyIOC.ts
37% <ø> (ø)

Codecov Report

Merging #15114 (53f3420) into main (3ce3547) will decrease coverage by 0%.
The diff coverage is 0%.

@@          Coverage Diff           @@
##            main   #15114   +/-   ##
======================================
- Coverage     65%      65%   -1%
======================================
  Files        559      559
  Lines      26284    26285    +1
  Branches    3730     3730
======================================
  Hits       17127    17127
- Misses      8467     8468    +1
  Partials     690      690

Impacted Files
Coverage Δ

src/client/pythonEnvironments/info/index.ts
39% <0%> (-1%)
:arrow_down:

src/client/pythonEnvironments/legacyIOC.ts
37% <ø> (ø)


[BUG]

In raising this issue, I confirm the following:

[x] I have read and understood the contributors guide.
[x] The issue I am reporting can be replicated.
[x] The issue I am reporting can be is directly related to the pivpn installer script.
[x] The issue I am reporting isn't a duplicate (see FAQs, closed issues, and open issues).

Describe the bug
The installer just autoselects ovpn when I want to choose wireguard
To Reproduce
Steps to reproduce the behavior:
On a raspberry 4 with a clean ubuntu install.
After updating and upgrading.
Run the pivpn script curl -L https://install.pivpn.io | bash
Follow screen instructions
It will ask for ovpn port instead of allowing to choose wireguard
Expected behavior
The installer should ask if wireguard or ovpn should be installed.
Desktop (please complete the following information):

VPN Protocol: NA
OS: Ubuntu
Version 18.04 ARM64

Ubuntu on the Raspberry Pi is not supported: https://github.com/pivpn/pivpn/wiki#what-boardsoses-does-pivpn-support
I think the installer selected OpenVPN beacuse WireGuard is only available on Raspbian and x86 Ubuntu/Debian.

Weird. A couple of weeks ago it did install under Ubuntu 18.04, the same OS I'm using now.  I'll try again with Raspbian Lite, and comeback


redirect to session['CAS_AFTER_LOGIN_SESSION_URL'] if it exists instead

@cameronbwhite thanks for writing Flask-CAS. I was able to roll it into a flask app fairly easily.
I needed to preserve the request.path once a client auth'd. In my login_required decorator, I decided to add to the clients session the URL prior to the /login redirection.
# my login_required decorator
def login_required(function):
    @wraps(function)
    def wrap(*args, **kwargs):
        if 'CAS_USERNAME' in session:
            cas_username = session['CAS_USERNAME']
            if is_account_allowed(cas_username):
                return function(*args, **kwargs)
            else:
                return redirect(url_for('denied'))
        else:
            session['CAS_AFTER_LOGIN_SESSION_URL'] = request.path
            return redirect(url_for('cas.login', _external=True))
    return wrap

Within routing.py I attempt to access that session variable and if it exists, use it instead of CAS_AFTER_LOGIN.
This allowed me to preserve and redirect the URL the client requested instead of redirecting to the static URL that CAS_AFTER_LOGIN provides.
Thanks again!

@renecunningham Great idea :+1: Thanks allot!
I think that this feature would pair well with a built in login_required decorator. I will be putting up a pull request containing one. My thinking is that the decorator should just ensure that the user is logged in before proceeding. Permissions can be handled by a separate decorator.

@renecunningham checkout #18 for the new decorator. Inspired by you of course.

@cameronbwhite awesome, thanks!


chore(deps): bump

Applied mkpr on undefined
sed s/"local-repository-provider": "^.*"/"local-repository-provider": "^7.0.21"/

Coverage remained the same at 87.526% when pulling 4f79a49b956625452835695bae474d10b48d5ded on mkpr/1 into 98d2ac2d09c25e2a61eb5585b3af3f2987ac37c2 on master.

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Error compiling Sentry.Plug on Heroku

Environment

Elixir version: 1.8.1
Erlang/OTP version (erl): 21.2.4
Sentry version (mix deps): 7.1.0
Operating system: Heroku Dyno

Description
When deploying to Heroku using the hook_pre_compile="mix sentry_recompile" buildpack config on a fresh dyno or a dyno where the build cache has been purged, it will fail to compile because the Sentry.Plug module is not loaded. Removing the config from the buildpack allows the build to succeed.
Expected Behavior
The application should compile with or without cached build files without having to remove the hook_pre_compile="mix sentry_recompile" line from the buildpack config file.
Actual Behavior
When building without cached build files the application will fail with the following error.
remote: == Compilation error in file lib/sentry_plug_error_web/router.ex ==
remote: ** (CompileError) lib/sentry_plug_error_web/router.ex:4: module Sentry.Plug is not loaded and could not be found
remote:     (elixir) expanding macro: Kernel.use/1
remote:     lib/sentry_plug_error_web/router.ex:4: SentryPlugErrorWeb.Router (module)
remote:  !     Push rejected, failed to compile Elixir app.

Steps to Reproduce
I was able to reproduce this issue by creating a new Phoenix application and configuring it to use the Sentry library and then pushing the repo to a new Heroku dyno. I have created a repo containing the application that will fail to compile here.
To reproduce create a new Heroku dyno and under the settings tab set the buildpack config to https://github.com/HashNuke/heroku-buildpack-elixir.git. Now clone the repo linked above and run heroku git:remote -a <heroku app name> and then git push heroku master.
It will eventually display the above error after it has started compiling the application.

@swelham thank you for the very detailed and reproducible report! 🙂
I was able to reproduce the issue.  This is behavior I would not have expected from mix deps.compile, as it will strictly compile only that dependency and none of the dependency's dependencies (regardless of the --force switch).  The clue being running mix deps.compile sentry the first time results in:
mix deps.compile sentry
==> sentry
Compiling 14 files (.ex)
warning: function :hackney.body/1 is undefined (module :hackney is not available)
  lib/sentry/client.ex:170

warning: function :hackney.request/5 is undefined (module :hackney is not available)
  lib/sentry/client.ex:169

warning: function :hackney.skip_body/1 is undefined (module :hackney is not available)
  lib/sentry/client.ex:175

warning: function :hackney_pool.child_spec/2 is undefined (module :hackney_pool is not available)
  lib/sentry.ex:99

The solution is either to swap the order of the compilation steps:
sentry_recompile: ["compile", "deps.compile sentry --force"]

or specify that child dependencies should also be compiled:
sentry_recompile: ["deps.compile sentry --force --include-children", "compile"]

The combination of the --force and --include-children switches means all of Sentry's dependencies will always be recompiled, which might be kind of a pain if you've already compiled them (on Heroku or elsewhere), so my feeling is to recommend ["compile", "deps.compile sentry --force"] as the default.  I've done that here: https://github.com/getsentry/sentry-elixir/pull/357

@mitchellhenke thank you for the detailed response and solution 👍

Thanks for the fix. Probably should update the docs? https://docs.sentry.io/clients/elixir/#including-source-code

@leandrocp Thanks, I've submitted a PR to fix it!


Moved support package generation to QLog

Codecov Report

Merging #12 into master will increase coverage by 0.84%.
The diff coverage is 0%.

@@            Coverage Diff             @@
##           master      #12      +/-   ##
==========================================
+ Coverage   90.51%   91.36%   +0.84%
==========================================
  Files          43       43
  Lines        1181     1181
==========================================
+ Hits         1069     1079      +10
+ Misses        112      102      -10

Impacted Files
Coverage Δ

QLog/QLog/SupportPackageCoordinator.swift
84.61% <0%> (+47.94%)
:arrow_up:

QLog/QLog/QLog.swift
37.03% <0%> (+37.03%)
:arrow_up:

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 ccc404a...5ba799d. Read the comment docs.


What are the guarantees regarding single (non-duplicate) job execution?

I am aware that the current implementation of the Pipleines API is based on App Engine Task Queues and, as such, this made me wonder whether the same warning about possible duplicate task execution applies to job executions. Is there any guarantees that a job will be executed only once (not considering job retries)? Or should a job always be idempotent?

https://github.com/GoogleCloudPlatform/appengine-pipelines/wiki/Python#what-is-a-pipeline
All pipelines must be idempotent. This means that running the same pipeline with the same inputs more than once will yield the same results and the same side-effects. The library does not enforce the idempotence requirement on pipelines, it is up to developers to do it themselves. However, the library provides a few pieces (like stable pipeline IDs) which make it easy to achieve idempotence for side-effects.

Oh that was my bad, I made sure I had read the Java wiki because that's the version I am using, but never considered the python one. :+1:
Thanks!

Right, pipeline code is expected be idempotent in both cases. I am not
certain about the Python implementation but the Java implementation is
slightly more lenient in the "yield the same results and the same
side-effects" part. Basically, it is true that the same Job can run more
than once but once run the job state will be changed conditionally and
atomically (CAS). Such a change (including adding child job) will be
dropped if job state was already modified. This means that as long as your
jobs do not change external state/data you should be OK and not experience
any data clobbering in case input changes between runs.
Arie.
On Thu, Apr 23, 2015 at 5:40 AM, Irineu<EMAIL_ADDRESS>wrote:

Oh that was my bad, I made sure I had read the Java wiki because that's
the version I am using, but never considered the python one. [image: :+1:]
Thanks!
—
Reply to this email directly or view it on GitHub
https://github.com/GoogleCloudPlatform/appengine-pipelines/issues/31#issuecomment-95573033
.

Hi @aozarov ,
Thank you very much for further expanding on this. I assume this is the code snippet where CAS happens.
jobRecord.setState(State.WAITING_TO_FINALIZE);
jobRecord.setChildGraphGuid(currentRunGUID);
updateSpec.getFinalTransaction().includeJob(jobRecord);
updateSpec.getFinalTransaction().includeBarrier(finalizeBarrier);
backEnd.saveWithJobStateCheck(
    updateSpec, jobRecord.getQueueSettings(), jobKey, State.WAITING_TO_RUN, State.RETRY);

Now what is intriguing me is the fact that there is also a CAS-like operation before running the job but without actually changing the job state:
if (!backEnd.saveWithJobStateCheck(
    tempSpec, jobRecord.getQueueSettings(), jobKey, State.WAITING_TO_RUN, State.RETRY)) {
    logger.info("Ignoring runJob request for job " + jobRecord + " which is not in a"
        + " WAITING_TO_RUN or a RETRY state");
    return;
}

Could we not seize this and use another State (e.g., RUNNING) to assure the run-only-once effect? I might be being too naive here since if it were this simple it'd have been done this way already, but it does not cost to ask.

lets say you changed it to RUNNING and then failed without a chance to change back the state...
If that happens it would be hard to distinct a failed running and a very long running (considering that taskqueue requests can be up to 10 minutes and backend requests up to 24 hours...)

Indeed, either way thanks again! I'm closing this as it has been answered already.


Add confirmation prompt to az vmware script-execution create

This checklist is used to make sure that common guidelines for a pull request are followed.
Related command

General Guidelines

[ ] Have you run azdev style <YOUR_EXT> locally? (pip install azdev required)
[ ] Have you run python scripts/ci/test_index.py -q locally? (pip install wheel==0.30.0 required)

For new extensions:

[ ] My extension description/summary conforms to the Extension Summary Guidelines.

About Extension Publish
There is a pipeline to automatically build, upload and publish extension wheels.
Once your pull request is merged into main branch, a new pull request will be created to update src/index.json automatically.
You only need to update the version information in file setup.py and historical information in file HISTORY.rst in your PR but do not modify src/index.json.

Thank you for your contribution! We will review the pull request and get back to you soon.

@zhoxing-ms can I get a review for this PR?

this is moved to https://github.com/Azure/azure-cli-extensions/pull/7663


Westwood ADL format can't be played (Adplug)

The Westwood ADL format can't be played (supported by Adplug). It seems to load 200 subsongs but foobar2000 loads much less subsongs.
Attached an example file. This file can be played successfully by foobar2000's adplug plugin.
dune ii - 00.zip

Thanks for the report. I will take a look.

found same bug in foobar2000
A solution is explained here: https://hydrogenaud.io/index.php?topic=119636.0

Seems like an issue inside adplug for sure. I will use the workaround implemented in foobar2000 when I fix this issue.


Cannot assign string to property App\Http\Livewire\DatePicker::$day of type ?int

TypeError
Cannot assign string to property App\Http\Livewire\DatePicker::$day of type ?int
:dart: Livewire\Component::Livewire\ComponentConcerns{closure}
:page_facing_up: /app/vendor/livewire/livewire/src/ComponentConcerns/HandlesActions.php
48                  data_set($this->{$propertyName}, $targetKey, $results[$targetKey]);
49              } else {
50*                 $this->{$name} = $value;
51              }
52

Open in Flare

https://flareapp.io/occurrence/132274069

https://flareapp.io/errors/1495397


feat: add jiti.import function for async import

related to #32
It should be the very first step to have an async import function for ESM, which we could improve the implemtation later. Unblocks #158

Thanks! Will follow up with small refactors in next steps.


Use native support for atomic FADD when address is in LDS

@AlexVlx Can we also add a test case that validates this?

@mangupta sort of (I'll add something). In order to actually verify this beyond "atomic FPADD works with __shared__" we'd need infrastructure that's not available with HIP tests i.e. looking at IR/ISA and checking that we're generating ds_add. Since this is a transparent optimisation, just ensuring that atomics to __shared__ work should suffice though.

@mangupta test added.

This test fails on cuda path probably because shared atomics is not supported.

They're supported on CUDA just fine. I don't think it's worth the time to investigate the failure though, do we have a macro that identifies NVCC / CUDA compilation?

They're supported on CUDA just fine. I don't think it's worth the time to investigate the failure though, do we have a macro that identifies NVCC / CUDA compilation?

#if defined(HIP_PLATFORM_NVCC)

@gargrahul arigato!


Issue 995

In the Brackets Git extension, a diff is used to review the changes you are about to commit, this means that on our flow it is important that all changes--even untracked files--must have a diff. Without a diff of untracked files the user gets the feeling that none of these new file will get committed.
After doing some investigation on how git diff works, I found that the only way to get a diff of a untracked files is to compare the file against /dev/null which did not sound like a good cross-platform solution.  Also I tried to run the command via brackets git but it was not returning the actual diff.
So, our solution requires flows for two cases:

Commit all: We need to check the status of the files that are candidates for the commit and check for untracked files.  If there are untracked files we will stage all the changes and do a diff of all files.  Once we have the diff we can unstage all the changes and then show the commit dialog using the diff.
Single commit: In this case we already know the status of the file, so if it is untracked we stage it, do the diff and unstage and show the commit dialog using the diff.

@zaggino I forgot to use the return calls on those places, I guess I got confused because the code is creating a new Promise, thought that was not required on that case. I'm also adding the catch on the reset index call

sorry, still a bit messy. you shouldn't do this:
return Git.stage(fileArray, false).then(function () {
+                return new Promise(function () {
+                    return Git.getDiffOfStagedFiles().then(function (diff) {
+                        return Git.resetIndex().then(function () {
+                            resolve(diff);
+                        });
+                    });
+                });
+            })

but instead it should be something like this:
var diff
return Git.stage(fileArray, false)
  .then(function () {
    return Git.getDiffOfStagedFiles();
  }).then(function (_diff) {
    diff = _diff;
    return Git.resetIndex();
  }).then(function () {
    return diff;
  });

and you shouldn't need to create your own Promise objects here

The changes now have your suggested changes from previous comment I think that is the right thing to do. I did a local test and it is doing the diff of untracked files. Now I have a better understanding of the Promises, thanks

Sorry, I forgot about this - merged.


🚧Update CICD to latest version

Complete The Item Below

[X] I have updated the title without removing the 🚧 emoji.

Description
Update the CICD dotnet tool to the latest version which is v1.0.0-preview.17 as of the creation of this library.
If a newer version exists, update it.
Acceptance Criteria

[ ] CICD dotnet tool updated

ToDo Items

[X] Change type labels added to this issue.  Refer to the Change Type Labels section below.
[X] Priority label added to this issue.  Refer to the Priority Type Labels section below.
[X] Issue linked to the correct project (if applicable).
[X] Issue linked to the correct milestone (if applicable).
[ ] Draft pull request created and linked to this issue (only required with code changes).

Issue Dependencies
No response
Related Work
No response
Additional Information:
Change Type Labels

Change Type
Label

Bug Fixes
🐛bug

Breaking Changes
🧨breaking changes

New Feature
✨new feature

Workflow Changes
workflow

Code Doc Changes
🗒documentation/code

Product Doc Changes
📝documentation/product

Priority Type Labels

Priority Type
Label

Low Priority
low priority

Medium Priority
medium priority

High Priority
high priority

Code of Conduct

[X] I agree to follow this project's Code of Conduct.

Closed without changes.
Reason: Duplicate of #159


[Glorious Quests] New sheet for game under development.

Changes / Comments
Added new sheet for the Glorious Quests game, based on the Heroic Tales system.
This is the first version. The sheet was built using the character sheet standards.
Roll20 Requests
Comments are very helpful for reviewing the code changes. Please answer the relevant questions below in your comment.

[ ] Does the pull request title have the sheet name(s)? Include each sheet name.
[ ] Is this a bug fix?
[ ] Does this add functional enhancements (new features or extending existing features) ?
[ ] Does this add or change functional aesthetics (such as layout or color scheme) ?
[ ] If changing or removing attributes, what steps have you taken, if any, to preserve player data ?
[ ] If this is a new sheet, did you follow Building Character Sheets standards ?

If you do not know English. Please leave a comment in your native language.

Character Sheet Info Roll20 Internal Use only.

Hi @carpedavid
I understand that this is based on Heroic Tales, but is there a place where the rules for Glorious Quests can be purchased/downloaded/acquired in some fashion?
Having publicly accessible game rules is one of the requirements for entrance into the repo.


Refactor Sharding-transaction.

Pull Request Test Coverage Report for Build 5991

8 of 16 (50.0%)  changed or added relevant lines in 4 files are covered.
18 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-0.2%) to 71.472%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

sharding-transaction/sharding-transaction-core/src/main/java/io/shardingsphere/transaction/core/handler/ShardingTransactionHandlerAdapter.java
0
4
0.0%

sharding-transaction/sharding-transaction-reg/sharding-transaction-reg-api/src/main/java/io/shardingsphere/transaction/api/ShardingTransactionHandlerRegistry.java
0
4
0.0%

Files with Coverage Reduction
New Missed Lines
%

sharding-core/src/main/java/io/shardingsphere/spi/NewInstanceServiceLoader.java
18
23.08%

Totals

Change from base Build 5988:
-0.2%

Covered Lines:
9405

Relevant Lines:
13159

💛  - Coveralls


New Term/Synonym from Fly Brain

New terms/synonyms from http://flybrain-ndb.iam.u-tokyo.ac.jp
See new terms need column in .xlsx from https://github.com/VirtualFlyBrain/Management/issues/148

From spreadsheet:
From FBrf0205263:
Kenyon cells (ACX), MB-C3
From FBrf0174482:
mALT (DL1), mALT (VA6), mALT (DA1/DC3/VA1d), mALT (DM1/VA4/VC1)
mALT (VM2) probably adult antennal lobe projection neuron VM2 adPN
Also see #358 FBrf0219809
From FBrf0210154:
PFS1 + PFS2 (pontin neuron P1 +2?), PB1
Not published but referred to in FBrf0230403:
CT02, CT03, CT34, CC1, CC2
Others:
H1 - FBrf0051327
FD1 - no obvious publication
vCal1 - no obvious publication
LPL01 - FBrf0240358 (DONE)
JONs (done on separate ticket)

Not adding MB-C3, CT02, CT03, CT34, CC1, CC2, FD1 or vCal1 at this time due to lack of available evidence.


Add support for primary key parsing.

Is your feature request related to a problem or challenge?
By using following syntax, we can define primary key.
CREATE TABLE sales_global_with_pk (zip_code INT,
          country VARCHAR(3),
          sn INT,
          ts TIMESTAMP,
          currency VARCHAR(3),
          amount FLOAT,
          primary key(sn)
        ) as VALUES
          (0, 'GRC', 0, '2022-01-01 06:00:00'::timestamp, 'EUR', 30.0),
          (1, 'FRA', 1, '2022-01-01 08:00:00'::timestamp, 'EUR', 50.0),
          (1, 'TUR', 2, '2022-01-01 11:30:00'::timestamp, 'TRY', 75.0),
          (1, 'FRA', 3, '2022-01-02 12:00:00'::timestamp, 'EUR', 200.0),
          (1, 'TUR', 4, '2022-01-03 10:00:00'::timestamp, 'TRY', 100.0)

However, following syntax is not supported.
CREATE TABLE sales_global_with_pk (zip_code INT,
          country VARCHAR(3),
          sn INT primary key,
          ts TIMESTAMP,
          currency VARCHAR(3),
          amount FLOAT
        ) as VALUES
          (0, 'GRC', 0, '2022-01-01 06:00:00'::timestamp, 'EUR', 30.0),
          (1, 'FRA', 1, '2022-01-01 08:00:00'::timestamp, 'EUR', 50.0),
          (1, 'TUR', 2, '2022-01-01 11:30:00'::timestamp, 'TRY', 75.0),
          (1, 'FRA', 3, '2022-01-02 12:00:00'::timestamp, 'EUR', 200.0),
          (1, 'TUR', 4, '2022-01-03 10:00:00'::timestamp, 'TRY', 100.0)

However, it doesn't indicate this also. Above query runs without error. However, there is no primary key defined for this table.
When primary key is supported, following query should succeed.
SELECT sn, ts, SUM(amount)
FROM sales_global_with_pk
GROUP BY sn;

Above query runs, for first version. However, for second version it is not supported. Because, primary key is not parsed.
Describe the solution you'd like
I would like to have support for alternative syntax
Describe alternatives you've considered
No response
Additional context
No response

Hello, sqlparser-rs supports second version syntax, but datafusion failed to support. I am so confusing so I take a lot  time to find the reason. Below is I found:
In sqlparser-rs, the first version primary key(sn) is parsed in https://github.com/sqlparser-rs/sqlparser-rs/blob/10a6ec56371e32f5fa145074e797f8ec74f6d812/src/parser.rs#L3457, primary key(sn) parsed as constraints as a part of CreateTable Statement.
Then in datafusion, constraints will passed in https://github.com/apache/arrow-datafusion/blob/e02ed51b880d09f2156bd192d283aef75cab04b7/datafusion/sql/src/statement.rs#L125, then in https://github.com/apache/arrow-datafusion/blob/e02ed51b880d09f2156bd192d283aef75cab04b7/datafusion/sql/src/statement.rs#L166 or https://github.com/apache/arrow-datafusion/blob/e02ed51b880d09f2156bd192d283aef75cab04b7/datafusion/sql/src/statement.rs#L189 constraints passed in LogicalPlan.
The second version sn INT primary key is parsed in https://github.com/sqlparser-rs/sqlparser-rs/blob/10a6ec56371e32f5fa145074e797f8ec74f6d812/src/parser.rs#L3657C29-L3657C29 as a part of ColumnDef as a part of columns in CreateTable Statement.
Then in datafusion, columns passed in https://github.com/apache/arrow-datafusion/blob/e02ed51b880d09f2156bd192d283aef75cab04b7/datafusion/sql/src/planner.rs#L206-L223 . In there, the ColumnDef option only used NotNull， others option is discareded. The primary key does not pass into LogicalPlan.
Find the reason is soooo happy.

I will take this issue, add this enhancement

Thanks @parkma99 for locating the root cause, and taking the issue.


ValueError: greenlet.greenlet size changed, may indicate binary incompatibility. Expected 152 from C header, got 40 from PyObject

Prerequisites

[X] I am using the latest version of Locust
[X] I am reporting a bug, not asking a question

Description
Dependency:  gevent-23.9.1  greenlet-3.0.3. locust-2.20.1
System: macOS m1
After I installed Locust with pip3 install locust, I can't run command locust normally.Detailed ouput:
Traceback (most recent call last):
File "/Library/anaconda3/envs/delta/bin/locust", line 5, in
from locust.main import main
File "/Library/anaconda3/envs/delta/lib/python3.8/site-packages/locust/init.py", line 13, in
from gevent import monkey
File "/Library/anaconda3/envs/delta/lib/python3.8/site-packages/gevent/init.py", line 72, in
from gevent._hub_local import get_hub
File "/Library/anaconda3/envs/delta/lib/python3.8/site-packages/gevent/_hub_local.py", line 150, in
import_c_accel(globals(), 'gevent.__hub_local')
File "/Library/anaconda3/envs/delta/lib/python3.8/site-packages/gevent/_util.py", line 148, in import_c_accel
mod = importlib.import_module(cname)
File "/Library/anaconda3/envs/delta/lib/python3.8/importlib/init.py", line 127, in import_module
return _bootstrap._gcd_import(name[level:], package, level)
File "src/gevent/_hub_local.py", line 1, in init gevent._gevent_c_hub_local
ValueError: greenlet.greenlet size changed, may indicate binary incompatibility. Expected 152 from C header, got 40 from PyObject
Command line
locust -V
Locustfile contents
from locust import HttpUser, task

class HelloWorldUser(HttpUser):
    @task
    def hello_world(self):
        self.client.get("https://www.google.com/")

Python version
3.8.5
Locust version
gevent-23.9.1  greenlet-3.0.3. locust-2.20.1
Operating system
macOS m1

Hi! Greenlet 3.0.3 was released just a couple weeks ago. Can you try installing greenlet 3.0.2? Or install using --no-binary gevent

Hi, I have tried install greenlet 3.0.2 or reinstall gevent with --no-binary, this problem still exists.
And if I change a device, locust -V works well.
So maybe it's indeed a anaconda-specific problem.
Thank you!


⚠ Bofrost.at has degraded performance

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

HTTP code: 200
Response time: 1056 ms

Resolved: Bofrost.at performance has improved in 1e22b7b.


Use pageid to uniquely identify contributions

Summary:
As of now, the contributions table uses autogenerated id as the primary key. So every time the app is started, the DB is flushed and populated again.
This can be avoided if pageid returned by the imageinfo API is used as the primary key.
For images currently being uploaded, we would need to use a locally generated key and then replace it once the upload succeeds.

Can I work on this?

This is being handled by #3566. You can look at other issues to work on. :)