Blood Magic Living Armor (Boots and Leggings) can't be retrieved from Grid

I put all my living armor from Blood Magic in refined storage and when I went to retrieve them I could only get the chestplate and helmet out and the boots and leggings do not allow me to remove them.  I tried dragging, shift clicking, right clicking, searching first and then manually finding and nothing allows them to be removed.
Lastly, I tried an exporter with and without a conduit filter to attempt to extract them into a chest and it wouldn't let go of them that way either.

Hi, which version are you on?

I am running FTB Unstable which is currently on MC 1.10.0 which FTB team bundles RefinedStorage 0.8.5.

Is the issue still present in 0.8.6?

Yes it is still present. I can't retrieve my Blood Orb from BloodMagic, also my crystal from DeepResonance.
But I really like the new textures and blocks, especially the new solderer.

In what storage medium are you storing your blood orb? Storage disks in disk drive?

I have a 64k storage disk.

But I used 0.8.5 and upgraded with the same save to 0.8.6. So this might be why. I'll start a new save with 0.8.6 from start and report if it still happens.

I am using 2 16k disks on a drive array.  I'll update this weekend to 0.8.6 as well and see if that helps.

Ok, I managed to reproduce the issue in a fresh save with 0.8.6.
Blood Orb worked for me this time but the armour (Helmet) still got stock and DeepResonance Crystal also got stuck in the 64k storage disk.

Did you get the helmet straight out of the creative inventory or did you use it first?

I was in a survival world and got the items with commands. I used the blood orb a few times and the helmet I put straight in the strorage.
But I just tested it with a used Helmet, still can't get it out of the grid.

All 3 are stuck.

My items, boots and leggings, were obtained legit, if it helps.  I just tried on 0.8.7 and the issue of the living armor still is in 0.8.7 and also all my deep resonance crystals are stuck in the crafting grid/storage and can't be retrieved as well.
FWIW, I am also playing Arisen 2 which is on 1.9.4 and so the Refined Storage 0.7.19 version does not have this issue with the crystal I just found.  I can get it out no problem but I haven't started blood magic yet so can't comment on living armor being retrievable yet.

I just noticed this issue on our world also, a tad frustrating to have my crystals locked up in limbo.

I upgraded to 0.8.9 and it didn't help get the items out but I also couldn't reproduce with either "given" items, crafted from scratch, damaged a bit items or anything else I tried.  I only spawned in crystals so I am not sure if crafting them in the crystal grower would make a difference.
I wonder if it has to do with when they are imported?  I used an importer and pulled from all my chests and that is the only thing I can think of?!?
Is there something those of us who have stuck items can do to help determine what is blocking it?  Can you make a special build with debugging info pasted into chat when we try to drag or click an item out?

I put my crystals in through the terminal. I did not import them or anything.

I'll have another go at this issue tomorrow, and if I can't figure it out I'll give you a debug build.

Here are the relevant lines, when i tried to get the blood orb and a deepresonance crystal:
[11:54:39] [Server thread/INFO] [STDOUT]: [refinedstorage.apiimpl.network.GridHandler:onExtract:25]: Trying to extract item 1xitem.BloodMagic.orb.@0 (grid extract flags = 4) [11:54:39] [Server thread/INFO] [STDOUT]: [refinedstorage.RefinedStorageUtils:compareStack:109]: Comparing 1xitem.BloodMagic.orb.@0 with 1xitem.BloodMagic.orb.@0 (flags = 3) [11:54:39] [Server thread/INFO] [STDOUT]: [refinedstorage.RefinedStorageUtils:compareStack:138]: NBT is not equal, l = null, r = {} [11:54:39] [Server thread/INFO] [STDOUT]: [refinedstorage.apiimpl.network.GridHandler:onExtract:30]: Extraction failed: item was null [11:54:50] [Server thread/INFO] [STDOUT]: [refinedstorage.apiimpl.network.GridHandler:onExtract:25]: Trying to extract item 1xtile.deepresonance.resonating_crystal@0 (grid extract flags = 4) [11:54:50] [Server thread/INFO] [STDOUT]: [refinedstorage.RefinedStorageUtils:compareStack:109]: Comparing 1xtile.deepresonance.resonating_crystal@0 with 1xtile.deepresonance.resonating_crystal@0 (flags = 3) [11:54:50] [Server thread/INFO] [STDOUT]: [refinedstorage.RefinedStorageUtils:compareStack:138]: NBT is not equal, l = {owner:"",efficiency:1.4112228f,strength:0.4626714f,glowing:0b,purity:6.228219f,infused:0,power:0.0f,version:2b}, r = {efficiency:1.4112228f,strength:0.4626714f,glowing:0b,purity:6.228219f,infused:0,power:0.0f,version:2b} [11:54:50] [Server thread/INFO] [STDOUT]: [refinedstorage.apiimpl.network.GridHandler:onExtract:30]: Extraction failed: item was null
Here as .txt files:
Relevant Lines.txt
fml-client-latest.txt

Okay, can you try this one for me?
refinedstorage-debug2.zip

Similar output for me with an "item was null" error:
[07:58:57] [Server thread/INFO] [STDOUT/]: [refinedstorage.apiimpl.network.GridHandler:onExtract:25]: Trying to extract item 1xtile.deepresonance.resonating_crystal@0 (grid extract flags = 0)
[07:58:57] [Server thread/INFO] [STDOUT/]: [refinedstorage.RefinedStorageUtils:compareStack:109]: Comparing 1xtile.deepresonance.resonating_crystal@0 with 1xtile.deepresonance.resonating_crystal@0 (flags = 3)
[07:58:57] [Server thread/INFO] [STDOUT/]: [refinedstorage.RefinedStorageUtils:compareStack:138]: NBT is not equal, l = {owner:"",efficiency:1.3283716f,strength:1.8075614f,glowing:0b,purity:5.908609f,infused:0,power:2.1873345f,version:2b}, r = {efficiency:1.3283716f,strength:1.8075614f,glowing:0b,purity:5.908609f,infused:0,power:2.1873345f,version:2b}
[07:58:57] [Server thread/INFO] [STDOUT/]: [refinedstorage.RefinedStorageUtils:compareStack:109]: Comparing 1xtile.deepresonance.resonating_crystal@0 with 1xtile.deepresonance.resonating_crystal@0 (flags = 3)
[07:58:57] [Server thread/INFO] [STDOUT/]: [refinedstorage.RefinedStorageUtils:compareStack:138]: NBT is not equal, l = {owner:"",efficiency:2.795206f,strength:1.4542f,glowing:0b,purity:12.726061f,infused:0,power:7.8774333f,version:2b}, r = {efficiency:1.3283716f,strength:1.8075614f,glowing:0b,purity:5.908609f,infused:0,power:2.1873345f,version:2b}
[07:58:57] [Server thread/INFO] [STDOUT/]: [refinedstorage.RefinedStorageUtils:compareStack:109]: Comparing 1xtile.deepresonance.resonating_crystal@0 with 1xtile.deepresonance.resonating_crystal@0 (flags = 3)
[07:58:57] [Server thread/INFO] [STDOUT/]: [refinedstorage.RefinedStorageUtils:compareStack:138]: NBT is not equal, l = {owner:"",efficiency:1.440359f,strength:2.3209314f,glowing:0b,purity:5.343514f,infused:0,power:22.007769f,version:2b}, r = {efficiency:1.3283716f,strength:1.8075614f,glowing:0b,purity:5.908609f,infused:0,power:2.1873345f,version:2b}
[07:58:57] [Server thread/INFO] [STDOUT/]: [refinedstorage.RefinedStorageUtils:compareStack:109]: Comparing 1xtile.deepresonance.resonating_crystal@0 with 1xtile.deepresonance.resonating_crystal@0 (flags = 3)
[07:58:57] [Server thread/INFO] [STDOUT/]: [refinedstorage.RefinedStorageUtils:compareStack:138]: NBT is not equal, l = {owner:"",efficiency:0.94835144f,strength:1.2934377f,glowing:0b,purity:11.857521f,infused:0,power:50.59206f,version:2b}, r = {efficiency:1.3283716f,strength:1.8075614f,glowing:0b,purity:5.908609f,infused:0,power:2.1873345f,version:2b}
[07:58:57] [Server thread/INFO] [STDOUT/]: [refinedstorage.RefinedStorageUtils:compareStack:109]: Comparing 1xtile.deepresonance.resonating_crystal@0 with 1xtile.deepresonance.resonating_crystal@0 (flags = 3)
[07:58:57] [Server thread/INFO] [STDOUT/]: [refinedstorage.RefinedStorageUtils:compareStack:138]: NBT is not equal, l = {owner:"",efficiency:0.10747299f,strength:1.4189719f,glowing:0b,purity:6.330573f,infused:0,power:38.98236f,version:2b}, r = {efficiency:1.3283716f,strength:1.8075614f,glowing:0b,purity:5.908609f,infused:0,power:2.1873345f,version:2b}
[07:58:57] [Server thread/INFO] [STDOUT/]: [refinedstorage.RefinedStorageUtils:compareStack:109]: Comparing 1xtile.deepresonance.resonating_crystal@0 with 1xtile.deepresonance.resonating_crystal@0 (flags = 3)
[07:58:57] [Server thread/INFO] [STDOUT/]: [refinedstorage.RefinedStorageUtils:compareStack:138]: NBT is not equal, l = {owner:"",efficiency:1.0621457f,strength:1.0508118f,glowing:0b,purity:5.7464576f,infused:0,power:28.505196f,version:2b}, r = {efficiency:1.3283716f,strength:1.8075614f,glowing:0b,purity:5.908609f,infused:0,power:2.1873345f,version:2b}
[07:58:57] [Server thread/INFO] [STDOUT/]: [refinedstorage.RefinedStorageUtils:compareStack:109]: Comparing 1xtile.deepresonance.resonating_crystal@0 with 1xtile.deepresonance.resonating_crystal@0 (flags = 3)
[07:58:57] [Server thread/INFO] [STDOUT/]: [refinedstorage.RefinedStorageUtils:compareStack:138]: NBT is not equal, l = {owner:"",efficiency:0.26320454f,strength:2.5847518f,glowing:0b,purity:5.3464346f,infused:0,power:18.713894f,version:2b}, r = {efficiency:1.3283716f,strength:1.8075614f,glowing:0b,purity:5.908609f,infused:0,power:2.1873345f,version:2b}
[07:58:57] [Server thread/INFO] [STDOUT/]: [refinedstorage.apiimpl.network.GridHandler:onExtract:30]: Extraction failed: item was null

fml-client-latest.txt

Can you also try extracting something else than the crystals? Like the botania stuff for example

Sure thing:  Here you go, debug 2, living armor that is stuck, click and drag plus shift click attempts:
[08:17:29] [Server thread/INFO] [STDOUT/]: [refinedstorage.apiimpl.network.GridHandler:onExtract:25]: Trying to extract item 1xitem.BloodMagic.livingArmour.@110 (grid extract flags = 0)
[08:17:29] [Server thread/INFO] [STDOUT/]: [refinedstorage.RefinedStorageUtils:compareStack:109]: Comparing 1xitem.BloodMagic.livingArmour.@110 with 1xitem.BloodMagic.livingArmour.@110 (flags = 3)
[08:17:29] [Server thread/INFO] [STDOUT/]: [refinedstorage.RefinedStorageUtils:compareStack:138]: NBT is not equal, l = null, r = {}
[08:17:29] [Server thread/INFO] [STDOUT/]: [refinedstorage.apiimpl.network.GridHandler:onExtract:30]: Extraction failed: item was null
[08:17:30] [Server thread/INFO] [STDOUT/]: [refinedstorage.apiimpl.network.GridHandler:onExtract:25]: Trying to extract item 1xitem.BloodMagic.livingArmour.@110 (grid extract flags = 0)
[08:17:30] [Server thread/INFO] [STDOUT/]: [refinedstorage.RefinedStorageUtils:compareStack:109]: Comparing 1xitem.BloodMagic.livingArmour.@110 with 1xitem.BloodMagic.livingArmour.@110 (flags = 3)
[08:17:30] [Server thread/INFO] [STDOUT/]: [refinedstorage.RefinedStorageUtils:compareStack:138]: NBT is not equal, l = null, r = {}
[08:17:30] [Server thread/INFO] [STDOUT/]: [refinedstorage.apiimpl.network.GridHandler:onExtract:30]: Extraction failed: item was null
[08:17:30] [Server thread/INFO] [STDOUT/]: [refinedstorage.apiimpl.network.GridHandler:onExtract:25]: Trying to extract item 1xitem.BloodMagic.livingArmour.@110 (grid extract flags = 0)
[08:17:30] [Server thread/INFO] [STDOUT/]: [refinedstorage.RefinedStorageUtils:compareStack:109]: Comparing 1xitem.BloodMagic.livingArmour.@110 with 1xitem.BloodMagic.livingArmour.@110 (flags = 3)
[08:17:30] [Server thread/INFO] [STDOUT/]: [refinedstorage.RefinedStorageUtils:compareStack:138]: NBT is not equal, l = null, r = {}
[08:17:30] [Server thread/INFO] [STDOUT/]: [refinedstorage.apiimpl.network.GridHandler:onExtract:30]: Extraction failed: item was null
[08:17:31] [Server thread/INFO] [STDOUT/]: [refinedstorage.apiimpl.network.GridHandler:onExtract:25]: Trying to extract item 1xitem.BloodMagic.livingArmour.@110 (grid extract flags = 4)
[08:17:31] [Server thread/INFO] [STDOUT/]: [refinedstorage.RefinedStorageUtils:compareStack:109]: Comparing 1xitem.BloodMagic.livingArmour.@110 with 1xitem.BloodMagic.livingArmour.@110 (flags = 3)
[08:17:31] [Server thread/INFO] [STDOUT/]: [refinedstorage.RefinedStorageUtils:compareStack:138]: NBT is not equal, l = null, r = {}
[08:17:31] [Server thread/INFO] [STDOUT/]: [refinedstorage.apiimpl.network.GridHandler:onExtract:30]: Extraction failed: item was null
[08:17:32] [Server thread/INFO] [STDOUT/]: [refinedstorage.apiimpl.network.GridHandler:onExtract:25]: Trying to extract item 1xitem.BloodMagic.livingArmour.@110 (grid extract flags = 4)
[08:17:32] [Server thread/INFO] [STDOUT/]: [refinedstorage.RefinedStorageUtils:compareStack:109]: Comparing 1xitem.BloodMagic.livingArmour.@110 with 1xitem.BloodMagic.livingArmour.@110 (flags = 3)
[08:17:32] [Server thread/INFO] [STDOUT/]: [refinedstorage.RefinedStorageUtils:compareStack:138]: NBT is not equal, l = null, r = {}
[08:17:32] [Server thread/INFO] [STDOUT/]: [refinedstorage.apiimpl.network.GridHandler:onExtract:30]: Extraction failed: item was null

fml-client-latest.txt

it seems same thing happens with some other mod items (namely ic2 energy crystals and deep resonance crystals) where they get stuck in the machine unable to extract them... oddly it doesnt happen every single time.

Hmm, are you able to export stuck items WITHOUT comparing on NBT and damage?

Oooh, that worked!  Crafting grid to cables to exporter to chest with given empty crystal and my enchanted work & damaged living armor used as filter and it pulled them right out.  I thought I had tried this before as I already had an exporter crafted but I am several versions newer since when I would have tried in the past.
Here are the logs in case it helps troubleshoot:
fml-client-latest.txt

It looks like the majority of the issue is trying to compare an empty NBT tag to a null one.  For pretty much all intents, these two conditions are equal, but I don't think the standard comparator treats them as such.  As for the owner="" vs not present tag on crystals, that's probably something that should be brought to McJty's attention.

@tahg I could probably add a an additional check to make null and size=0 equal to each other, but I want to find the real cause of getting null tags. Oh well,  I suppose I could add the check for now.

It starts out null. It generally becomes not null when some mod pre-emptively creates it, and then conditionally decides to add a tag or not, in my experience.  If creating and deserialization of an item stack always created the tag, I don't think it'd be an issue.

yeah, that is true. that still leaves me wondering about that owner tag. As as soon as you push to a grid it takes a copy of the stack, so taking out should work. When a grid is opened it sends the stack to the client, and the client sends the wanted stack back in a packet when trying to pull. Maybe there is some kind of different between the client and server tag?

I suspect this code is responsible: https://github.com/McJty/DeepResonance/blob/master/src/main/java/mcjty/deepresonance/blocks/crystals/ResonatingCrystalBlock.java#L117
As for null vs empty, that's probably just an issue with the way stacks are (de)serialized over the network

aha! Good find.
Well, this is the new NBT compare I wrote:
    public static boolean compareNbt(ItemStack left, ItemStack right) {
        if (!ItemStack.areItemStackTagsEqual(left, right)) {
            if (left.hasTagCompound() && !right.hasTagCompound() && left.getTagCompound().hasNoTags()) {
                return true;
            } else if (!left.hasTagCompound() && right.hasTagCompound() && right.getTagCompound().hasNoTags()) {
                return true;
            }

            return false;
        }

        return true;
    }

Kinda verbose, but clear.

For the owner tag issue, I can probably maintain 2 copies of every itemstack, one that is rendered, and one that is kept in memory for sending to the server.

Do you have sorting options or such that prevents you from just sending back an index of the item you picked?

Yup, sorting and search bar. I could send over a ID as well, but that would mask issues that could occur in the future.

Okay, I just published v0.8.10. Please try it out and let me know if it works now

Yes it works for me.
I can retrieve all stuck items on all of my saves.
So DeepResonance and BloodMagic are working for me now. Thanks for your work.

With pleasure!


Add community health files

Add code of conduct
Switch to Apache license and use CLA. The CLA is referenced in a PR template so that every new PR that gets opened will, by default, include a note agreeing to the CLA. I also noted in the contribution guidelines that opening a PR constitutes acceptance of the CLA.
Add contributing guidelines
Rearrange the README

@U8NWXD -- I'm redoing much of the readme in #144, including all of @1fish2's suggestions.


Performance issues

Pushing 605 files for a total of 6 MB took about 16 minutes from a modern SSD over a 100+ mbps university Internet connection. Is this expected?
I was attempting to migrate from Dropbox using this tool. I have hundreds of thousands of small files totaling about 8GB. Is this a feasible use case?
This looks like a great tool and don't want to appear rude. I just want to know if I'm doing it wrong.

No worries, you aren't being rude, this is a legitimate concern. To be honest I think upload speed is relative. From my university I am able to push about 2GB in about 19 minutes. Your specs seem even more impressive. To start ruling out cases, you can  try uploading a big file and see that the upload speed should be good. I suspect it has to do with setting up the respective hierarchies, hence queueing /scheduling might be the bottle neck. Also might you be uploading *.wav files?
Thank you for reporting this. Resolving this surely will improve user experience.

I have pushed another folder containing about 70 files for 750 MB and that took 120 sec. This seems great for few big files, but  very slow for numerous small files. The big files were PDFs, the small ones Matlab source files.

So there you go, it is with the scheduling. I actually cut out a whole lot of parallelism to ensure correctness first. This is was lacking before and causing crashes due to resource exhaustion. Offline, I have been working on getting parallelism in and I have a couple of prototypes. However, I just need to do some heavy testing.

Got it. Thanks. Looking forward to it.
On Mar 3, 2015 4:06 PM, "Emmanuel Odeke"<EMAIL_ADDRESS>wrote:

So there you go, it is with the scheduling. I actually cut out a whole lot
of parallelism to ensure correctness first. This is was lacking before and
causing crashes due to resource exhaustion. Offline, I have been working on
getting parallelism in and I have a couple of prototypes. However, I just
need to do some heavy testing.
—
Reply to this email directly or view it on GitHub
https://github.com/odeke-em/drive/issues/77#issuecomment-77034479.

Watch this space: I'll ask for your help in testing it out, if you don't mind.

Hello folks,
Thank you very much for your patience. I got time this weekend to take a look at how to resolve this and I have spun up a work in progress PR #210 to address it. Please take a look at it if interested.

Please get the latest from master. Thank you very much for reporting this issue and for the discussions everyone, plus the patience. Closing it. Please re-open if it persists.

I am having this issue with a bunch of source trees. I have been uploading a lot of stuff lately, could google be throttleing me?

Hey @gauteh please take a look at https://github.com/odeke-em/drive/issues/480


How to pass parameter with ui-sref between deeply nested objects?

Hi,
I got problem with how i pass parameter with ui-sref. This is post i have already put it in stackoverflow.
Hope any of you, can help me.
http://stackoverflow.com/questions/30186531/how-to-pass-parameter-with-ui-sref-between-deeply-nested-objects
Thank you.

closing as duplicate of #1952


Catchable fatal error: Object of class RemoteWebElement could not be converted to string in C:\path\file.php on line 130

129  foreach ( $els as $key => $el ) {
130  $txt = $el.getText('');
131  }```

The code above produces the error message:
Catchable fatal error: Object of class RemoteWebElement could not be converted to string in C:\path\file.php on line 129

I can work around the error by adding the following code to RemoteWebElement.php
```public function __toString() {
  return $this->getText();
}```

php-webdriver
selenium-server-standalone-2.41.0.jar
Windows 7 SP1
Firefox 28
xampp includes
Apache/2.4.7 (Win32) OpenSSL/1.0.1e PHP/5.5.9

$buttonNext2	 =  $driver->findElement(WebDriverBy::tagName("body"));
//array_push($URLSArray,$URLPARSE1);
//echo  $googlemapsresult = $buttonNext2->getAttribute('innerHTML');
echo "buttonNext2->getAttribute('innerHTML";
$googlemapsresult = $buttonNext2->getAttribute('innerHTML');

innerHTML is not a DOM element attribute, you'll need to use an executeScript() call to get what you need here.

You can do this, for example
$ul = $driver->findElement(WebDriverBy::xpath('.//ul[@id="<put the id here>"]'));
    $lis = $ul->getAttribute('innerHTML');


Convert binstub conflict deprecation to a warning

What was the end-user problem that led to this PR?
The problem was that this specific deprecation I don't think it makes sense as a deprecation.
What was your diagnosis of the problem?
My diagnosis was that nothing is changing in bundler so that this needs to be a deprecation. This is just a potentially undesired situation that can be caused by gems shipping executables conflicting with the executables of other gems, and that can be fixed by using project specific binstubs.
What is your fix for the problem, implemented in this PR?
My fix is to change this deprecation message to a warning, and to get it tested.
Why did you choose this fix out of the possible options?
I chose this fix because it makes sense to me. With this, we can start printing this warning now without thinking of a deprecation path because I don't think we're changing anything here, really.

I noticed that this warning is actually hardly ever useful, because the only case it was detecting wouldn't cause any problems because even if it was loading an incorrect binstub, the gem finally loaded is the one the user would expect.
However, the situation were multiple conflicting executables were being used in the same app was going unnoticed. I'll try to fix this.

Ok. I'm quite satisfied with this PR now. This is what it does:

Simplify the find_spec_for_exe to assume exec_name is always passed. The previous version was defaulting to Gem::Specification.default_executable (an executable named just like the gem), but that method has been long deprecated in rubygems. This changes goes together with https://github.com/rubygems/rubygems/pull/2706.

Convert the previous deprecation to a warning as explained in the PR body.

Add a new warning about conflicting executables with the same name in the same application. The warning text I added is (for example, taking from the new spec):

The rackup executable in the fake gem is being loaded, but it's also present in other gems (rack). If you meant to run the executable for another gem, make sure you use a project specific binstub (bundle binstub <gem_name>). If you plan to actually use both conflicting executables, generate binstubs for both and disambiguate their names.

As a note, I had to change a previously existing spec, but in my opinion the spec I changed didn't make any sense. It's this one:
https://github.com/bundler/bundler/blob/53989a9812338b804f2c951b6077dfe296d72a99/spec/install/binstubs_spec.rb#L23-L42
The situation described in the test is pretty much undefined. There's no information to know which executable is the good one. As a matter of fact, I run the spec several times under bundler 2 and the results as per which binstub is loaded were undefined, I guess it comes down to which gem is downloaded and installed in the last place.

Finally got this one green!


Reconsider deprecation of $environment parameter in run()

Hello everyone,
I've reached a stage in my Castor usage where I need to refactor and centralize the logic that builds my commands. After some consideration, I've settled on the following architecture:
tools/castor/src/docker.php
/**
 * Build a docker-compose command.
 *
 * @param array<int|string|\Stringable>|string $command
 * @param array<string, int|string|\Stringable> $environment
 *
 * @return array{command: list<int|string|\Stringable>, environment: array<string, int|string|\Stringable>}
 */
function dcp(string|array $command, array $environment = []): array
{
    $command = \is_array($command) ? $command : explode(' ', $command);

    return ['command' => ['docker', 'compose', ...$command], 'environment' => $environment];
}

/**
 * @param array<string, int|string|\Stringable> $environment
 *
 * @return array{command: list<int|string|\Stringable>, environment: array<string, int|string|\Stringable>}
 */
function dcp_start(bool $forceRecreate = false, array $environment = []): array
{
    $args = 'up -d --remove-orphans';
    if ($forceRecreate) {
        $args .= ' --force-recreate';
    }

    return dcp($args, $environment);
}

/**
 * @return array{command: list<int|string|\Stringable>, environment: array<string, int|string|\Stringable>}
 */
function dcp_down(): array
{
    return dcp('down');
}

castor.php
require_once __DIR__ . '/tools/castor/src/docker.php';

#[AsTask(name: 'start', description: 'Start the Docker containers')]
function task_start(): void
{
    run(...dcp_start());
}

#[AsTask(name: 'restart', description: 'Start the Docker containers, ensuring all necessary services are running, and install PHP dependencies using Composer.')]
function task_restart(): void
{
    run(...dcp_start(true));
}

#[AsTask(name: 'down', description: 'Stop and remove all running Docker containers, cleaning up the environment.', aliases: ['stop'])]
function task_down(): void
{
    run(...dcp_down());
}

However, this implementation throws a deprecation notice since it relies on the now-deprecated array $environment parameter in run(), requiring the use of run(context: context()->withEnvironment()). This means I can no longer do a simple run(...['command' => 'my-command', 'environment' => ['FOO' => 'bar']]).
In my setup, returning both the command and the associated environment variables feels more intuitive, as it mirrors the way we typically structure commands in the command line (e.g., FOO=bar my-command-line). Forcing part of my command into run(command: 'my-command-line') and the other into a less readable run(context: context()->withEnvironment(['FOO' => 'bar'])) feels somewhat counterintuitive and rigid.
Would it be possible to reconsider the deprecation and retain the environment parameter as a DX improvement? This approach would essentially act as a proxy to context()->withEnvironment() without much downside IMO.

What about adding your own run command, that does what our run command require?

This is indeed what I did in the meantime. Here’s the workaround I implemented to bypass the deprecation notices for now:
namespace app {
    /**
     * @param string|array<string|\Stringable|int>}|\Castor\CommandBuilder\CommandBuilderInterface $command
     * @param array<string, string|\Stringable|int>}|null $environment
     * @param callable(string, string, \Symfony\Component\Process\Process): void|null $callback
     */
    function run(string|array|CommandBuilderInterface $command, ?array $environment = null, ?string $workingDirectory = null, ?bool $tty = null, ?bool $pty = null, ?float $timeout = null, ?bool $quiet = null, ?bool $allowFailure = null, ?bool $notify = null, ?callable $callback = null, ?Context $context = null, ?string $path = null): Process
    {
        return \Castor\run(
            command: $command,
            workingDirectory: $workingDirectory,
            tty: $tty,
            pty: $pty,
            timeout: $timeout,
            quiet: $quiet,
            allowFailure: $allowFailure,
            notify: $notify,
            callback: $callback,
            context: $context?->withEnvironment($environment ?? []) ?? \Castor\context()->withEnvironment($environment ?? []),
            path: $path
        );
    }

    /**
     * @param string|array<string|\Stringable|int>} $command
     * @param array<string, string|\Stringable|int>}|null $environment
     */
    function capture(string|array $command, ?array $environment = null, ?string $workingDirectory = null, ?float $timeout = null, ?bool $allowFailure = null, ?string $onFailure = null, ?Context $context = null, ?string $path = null): string
    {
        return \Castor\capture(
            command: $command,
            workingDirectory: $workingDirectory,
            timeout: $timeout,
            allowFailure: $allowFailure,
            onFailure: $onFailure,
            context: $context?->withEnvironment($environment ?? []) ?? \Castor\context()->withEnvironment($environment ?? []),
            path: $path
        );
    }
    /**
     * @param string|array<string|\Stringable|int>} $command
     * @param array<string, string|\Stringable|int>}|null $environment
     */
    function exit_code(string|array $command, ?array $environment = null, ?string $workingDirectory = null, ?float $timeout = null, ?bool $quiet = null, ?Context $context = null, ?string $path = null): int
    {
        return \Castor\exit_code(
            command: $command,
            workingDirectory: $workingDirectory,
            timeout: $timeout,
            quiet: $quiet,
            context: $context?->withEnvironment($environment ?? []) ?? \Castor\context()->withEnvironment($environment ?? []),
            path: $path
        );
    }
}
}

That said this approach does introduce additional maintenance overhead, especially considering Castor’s fast-paced development and lack of a stable version. I fear each update could break my
I suspect other users particularly newcomers to Castor may face similar challenges so addressing this at the core level could enhance usability and make Castor more approachable IMO and that's why I created this issue. But it's only my opinion 😉

Weel, we removed all argument because of maintenance. Now it's simpler, there is not a ton of argument to handle. Future addition will be easy (only in the context)
I'm sorry for the deprecation, but castor is quite stable now. I think we won't change the signature anymore.

Anyway, did you see https://castor.jolicode.com/going-further/extending-castor/command-builder/ ?
I think you could leverage this to abstract your docker layer.

Finally, Why don't you do:
function task_start(): void
{
    $start = dcp_start();
    run($start['command'], context: context()->withEnvironment($start['environment]));
}

Thank you for pointing that out! The command builder approach is indeed exactly what I need for my advanced use case and provides the clarity and readability of run(dcp()) I want when working with the task file. Not sure how I've missed it 🤦 Appreciate the guidance!


Allow to configure __.settings.classes

I'm using the lib to make a shopping cart panel, but I needed to scroll only a specific container inside the panel, leaving the header and footer fixed, so I've changed the library to accept customized classes for the __.settings.class.
If this PR is approved, I'd be glad to update the documentation. If not, how would you recommend me to implement this use case?

Perfect!🍺  Thank you for your contribution!


Missing .editorconfig

While not causing any issues, it looks like you could have included a .editorconfig with your repository.
Usually helps if people contribute to your code.
https://github.com/DeSinc/SallyBot/blob/c87b5b529268293152269e9ca1057248bd0ff8b5/sallybot.csproj#L12

not sure where this file is, I have one that's in my build in the debug folder but don't think it's the one you need

@DeSinc so ..\.editorconfig would mean that it's in the folder above your SallyBot directory - could you check there?

Ah here it is
[*.cs]
CS4014: Because this call is not awaited, execution of the current method continues before the call is completed
dotnet_diagnostic.CS4014.severity = silent
IDE0028: Simplify collection initialization
dotnet_style_collection_initializer = true
Where would I upload it in the github since it's meant to be in the folder above?


segfault in openssl_probe call from native-tls on recent alpine

I'm seeing this debugging an issue with a DNS server I'm working on. The issue is here: https://github.com/zerotier/zeronsd/issues/106
I'm throwing it over the fence for educated eyeballs, but will debug and report back when I know more about what's causing it. I'm fairly certain something messy in the environment is causing the issue.
Hope this is useful, if you don't know what's going on or don't want to investigate, I will report back in a day or two.

Sorry I don't know much about how this crate could cause a segfault myself :(

yeah, kind of jumped the gun here. sorry.


Add a layer to show where on the map problems are occurring, either i…

…ndividually or as a heatmap. #600
This was super quick to crank out by just copying what we've already got for the population map. It also seems pretty decent at quickly pinpointing actual problematic areas:

Video demo (gif too big, still having trouble encoding to mp4 and github accepting it): https://www.dropbox.com/s/ctjzh1073qb5o4m/problem_demo.gif?dl=0

Also asking for @Robinlovelace's thoughts on this dataviz, as it's usable for historic collision data, and I'm sure you've encountered plenty of tools that show things like this before. Async review fine; fast iteration, not long reviews, are my vibe

Oh yeah, and the layer isn't intended to show change in problems due to edits, just the current state of things. And I'll add a time filter in the future, to quickly see if a problem affects an area only part of the day.

I switched to show the heatmap by default and use purple, not red dots.
Also added a time slider filter, even though actually using the sliders is clunky -- no smoothly dragging, only clicking to set a new state. Will keep iterating going forward.

Do you think it would make sense to show some kind of icon on the screen while zoomed in when this layer is present? Ideally something distinct per problem and/or with a distinct tooltip on hover.

Yuwen actually brought this up after the latest UX study. Most layers don't show anything zoomed in, only unzoomed or on the minimap. If we decide this is actually a good thing and not just kind of accidental, then we may try automatically unzooming when you open a layer while zoomed in. I'm not sure if it'll be distracting / unexpected, or cause problems for people who want to return the camera to where it was.
But to your point here, we could add more detail and allow tooltips here. The implementation will need to aggregate problems per point, since everything right now maps to the center of a lane or intersection. The tooltip could show the count broken down by problem type and maybe even some histogram of when the problems occurred through time, and even give you a way to time-warp and sample a few of the problems.

Will take a look. I'm just rebuilding the latest version on this branch to give it a go with
git checkout safety_layer
Branch 'safety_layer' set up to track remote branch 'safety_layer' from 'origin'.
Switched to a new branch 'safety_layer'

abstreet on  safety_layer [?] via 𝗥 v1.50.0
➜ git pull
Already up-to-date.

abstreet on  safety_layer [?] via 𝗥 v1.50.0 took 3s
➜ cargo build

Will give it a go later today at a time when we're both awake (probably after 5pm UK time).


REST service to create a new Rasa story

AS AN Architect
I WANT to have a REST service to create new Rasa stories
SO THAT there wouldn't be duplicate services for it
Acceptance Criteria

Temporary list of endpoints - https://github.com/buerokratt/Buerokratt-onboarding/issues/8
Proto to add new Rasa stories

[ ] Use POST endpoint /rasa/stories/add to create a new form
[ ] Input provided as valid JSON, converted to YAML by #45
[ ] Creating a new story is forbidden if #112 responds other than HTTP 200
[ ] Apply changes on Rasa stories.yml

Examples
Input stories.yml before adding a new story
version: "3.0"
stories:
- story: andmekaitse_küsimused
  steps:
  - intent: andmekaitse_küsimused
  - action: utter_andmekaitse_küsimused

- story: test
  steps:
  - intent: test
  - action: utter_test
  - checkpoint: check_stuff

Request to add a story named "tervitus" with intents "greeing_ee" and "greeing_en" and action "utter_laters", the resulting stories.yml would look like
version: "3.0"
stories:
- story: andmekaitse_küsimused
  steps:
  - intent: andmekaitse_küsimused
  - action: utter_andmekaitse_küsimused

- story: test
  steps:
  - intent: test
  - action: utter_test
  - checkpoint: check_stuff

- story: tervitus
  steps:
  - intent: greeing_ee
  - intent: greeing_en
  - action: utter_laters

sample input for /rasa/stories/add
{
  "story": "tervitamine",
  "steps": [
    {
      "intent": "greeing_ee"
    },
    {
      "intent": "greeing_en"
    },
    {
      "action": "utter_laters"
    }
  ]
}

@RaulAltmae please link this issue with an appropriate pull request.

BUG - When creating a new Rasa story. Unable to save, after clicking "Salvesta/Save" it gives error code 409.

It is possible to create a new story, when trying to "Change/Muuda" the title and then saving, it creates the story.

Short summary: When editing the title and deleting letters one by one then when the last letter is deleted the whole selection is reset
Expected result: you just enter the new title
Real result: The title selection is reset
How to reproduce: login and go to https://admin.dev.buerokratt.ee/training/training/stories . then try to add a story and then try to edit the title.
Pic:

BUG - Occasionally i am able to create new Rasa stories, yet the second time it is not possible.

@KasparMinn please link PR

Short summary: When you add three slots to the condition and then delete the middle one, then the 2 slots down the middle merge.
Expected result: The middle one should just be deleted
Real result: You have like 2,5 slots
How to reproduce: go to stories, add 3 slots, then delete the mid one.
Pics:

PR: https://github.com/buerokratt/Training-Module/pull/396

After the GUI fix, task still needs a bit of work to adjust some Story-Rule YML writing logic, new PR soon.

Important notice: For stories and rules generation, there is currently in use an alternate datamapper script called (DSL/Datamapper/js/convert/jsonToYamlStories) as the Stories & Rules YAML files are both holding a more unique structure that consists of several JSON elements. Those elements get mixed up during the object transfer in Ruuter and the file is generated in the wrong way, therefore a more manual solution was temporarily needed to make everything work.

PR to add some elements to YAML generation: https://github.com/buerokratt/Training-Module/pull/398

Short summary: same bug as in the rules (https://github.com/buerokratt/Training-Module/issues/111#issuecomment-1966428009). Basically the flow of the story is different from the one that was created. Conditions and slots are gone and others are shown as response.
Pic:

PR: https://github.com/buerokratt/Training-Module/pull/405

Short summary: Can't add a story as I get an error. No matter which elements I try I seem to get an error.
Pic:

According to @KasparMinn this task was previously moved to "ATR" due to deployment errors. No development done or needed.

Gives an 500 error code when I try to make a new story.

error message:
{
"error": "ScriptingException: Error executing script: JSON.parse(storiesData).response.body.stories"
}

Still the same issue as stated in my earlier comment.

Error executing DSL: rasa/stories/add
Caused by: ee.buerokratt.ruuter.service.exception.StepExecutionException: Error executing: mergeStories
Caused by: ee.buerokratt.ruuter.helper.exception.ScriptEvaluationException: Error executing script: JSON.parse(storiesData).response.body.stories


ConnectivityChanged not fired when switching between cellular and wifi

Please take a moment to fill out the following (change to preview to check or place x in []):
This is a

[ x] Bug
[ ] Feature Request

Which plugin does this impact:

[ ] Battery
[ x] Connectivity
[ ] Contacts
[ ] DeviceInfo
[ ] ExternalMaps
[ ] Geolocator
[ ] Media
[ ] Permissions
[ ] Settings
[ ] Text To Speech
[ ] Vibrate
Other:

Version Number of Plugin: 2.1.2
Device Tested On: Nexus 5 Android 6.0.1
Simulator Tested On: (no cellular connection possible on simulators)
Expected Behavior
You said that this event is fired when connectivity is gained, changed or loosed.
When switching between cellular and wifi connection, this event should be fired and the event args could eventually contains the ConnectionTypes added or removed.
Actual Behavior
Event is only fired when connectivity is gained or loosed.
Steps to reproduce the Behavior
Deploy a test application to a real device without any connection.
Activate cellular connection => event fired
Activate wifi connection => event not fired (but if we check the ConnectionTypes, Cellular has been replaced by Wifi)
Desactivate wifi connection => event not fired (but if we check the ConnectionTypes Wifi has been replaced by Cellular)
Desactivate cellular connection => event fired
Thanks in advance ;)

Any workarounds for this?

Duplicate of: https://github.com/jamesmontemagno/ConnectivityPlugin/issues/3


Bring back keyboard shortcut for Fullscreen mode

🚀 Feature
Bring back keyboard shortcut for Fullscreen mode.
Press 'F' and you enter fullscreen. 'ESC' or 'F' again to leave fullscreen.
Motivation
It's the best feature. So good.
Pitch
I just explained that this is the best feature.
People tell me they love this feature --  it is true.
Alternatives
None.
Additional context
Fullscreen is all there is to see.

Ah I had not realized this had disappeared but I will look into it (I have an idea why). Maybe it's also time to add ctrl+W, ctrl+Q to close / quit? Need to think about where these live.  My thoughts are on Window right now.

I think it disappeared in a very early refactor. I agree that shortcuts should go on the window, (w in ctrl-w is for window), but f is possibly for the viewer.

fixed by #472


Merge build-to-app into master

Why are these changes needed?

Related issue number

Checks

[ ] I've run scripts/format.sh to lint the changes in this PR.
[ ] I've included any doc changes needed for https://docs.ray.io/en/master/.
[ ] I've made sure the tests are passing. Note that there might be a few flaky tests, see the recent failures at https://flakey-tests.ray.io/
Testing Strategy

[ ] Unit tests
[ ] Release tests
[ ] This PR is not tested :(

Current dependencies on/for this PR:

master

PR #53   (this PR)

This comment was auto-generated by Graphite.


Merge conflicts block the 'Mark as Manually Merged' operation

Description
When a PR shows this message at the bottom:

This pull request has changes conflicting with the target branch.

it is no longer possible to mark the PR as 'manually merged'. This is the same for when the conflict checking is in progress, in such cases it's also impossible to just mark things as merged. Since conflict checking can take a lot of time, this is often a blocking issue for the Blender project.
Gitea Version
https://github.com/blender/gitea/tree/blender-merged tracking Gitea main at ce5d711
Can you reproduce the bug on the Gitea demo site?
No
Log Gist
No response
Screenshots
No response
Git Version
No response
Operating System
No response
How are you running Gitea?
Running our own branch.
Database
PostgreSQL

it is no longer possible to mark the PR as 'manually merged'.

You think it's an regression? We did alter the PR merge box recently, so I suspect it came from that work.

I'm not sure whether it's a regression or not. In any case I feel that it would be prudent to always have the option to close a PR as 'manually merged', regardless of any background processing that might be going on.

I guess it is a regression, I have left comments there

On my side, I can see the "manual merge" form.
Do you have a reproducible example?


Add possibility to bundle node module

Our reusable webpack configurations should respect dependencies and peerDependencies in a way that dependencies are bundled and peerDependencies not.

Closed by #7545


Fix a recent regression in package identity

At this point (until we…change the semantics), an explicitly provided name in a package dependency is not used as the identity.  It is validated against what the package manifest claims, but does not currently constitute the identity.
re: https://github.com/apple/swift-package-manager/pull/2997
rdar://problem/70764221

@swift-ci please smoke test

Merging, since this is a needed regression fix.  The tests pass and this is restoring previous semantics.


java.lang.NoClassDefFoundError: com.orm.SugarDb

I get this error on android devices running 4.4, there's no problems with 5.0+. Do you have any idea what the cause may be? It's pointing to the line super.onCreate() in the app class.
java.lang.NoClassDefFoundError: com.orm.SugarDb
  at com.orm.Database.<init>(Database.java:12)
  at com.orm.SugarApp.onCreate(SugarApp.java:11)
  at com.myapp.MyApp.onCreate(MyApp.java:15)
  at android.app.Instrumentation.callApplicationOnCreate(Instrumentation.java:1013)
  at android.app.ActivityThread.handleBindApplication(ActivityThread.java:4729)
  at android.app.ActivityThread.access$1600(ActivityThread.java:174)
  at android.app.ActivityThread$H.handleMessage(ActivityThread.java:1367)
  at android.os.Handler.dispatchMessage(Handler.java:102)
  at android.os.Looper.loop(Looper.java:146)
  at android.app.ActivityThread.main(ActivityThread.java:5593)
  at java.lang.reflect.Method.invokeNative(Native Method)
  at java.lang.reflect.Method.invoke(Method.java:515)
  at com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run(ZygoteInit.java:1283)
  at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:1099)
  at dalvik.system.NativeStart.main(Native Method)

Here's is the app class.
public class MyApp extends SugarApp {
  @Override
  public void onCreate() {
    super.onCreate();

    // configure Flurry
    FlurryAgent.setLogEnabled(false);

    // init Flurry
    FlurryAgent.init(this, VERSION.FLURRY);

    // Adjust SDK
    AdjustConfig config = new AdjustConfig(this, VERSION.ADJUST, VERSION.ADJUST_ENV);
    Adjust.onCreate(config);
  }
}

And the gradle import - we're using 1.3
compile 'com.github.satyan:sugar:1.3'

I believe this error was caused by a number of things, for one I had a custom application class that extended SugarApp, while I actually needed it to extend MultiDexApplication.
There is a way to init the SugarContext without extending the SugarApp, but that is not possible with just the gradle imports. My solution was to clone the repository and add the library to my own app. Then following this pattern to init SugarORM.
public class ClientApp extends Application {
    @Override
    public void onCreate() {
        super.onCreate();
        SugarContext.init(this);
    }

    @Override
    public void onTerminate() {
        super.onTerminate();
        SugarContext.terminate();
    }
}

I haven't managed to get this tested properly yet, but the initial tests on emulators running API 19 seem positive.
So, in the end it was not an error with the library it self - it was a problem with the gradle versions not being up to date with the source code.

hi ,
i'm also facing the same problem , i think the problem with the module set up config. file ,
Any Help ??


Add auto gpu layer count detection

Currently, we have to provide the right amount of gpu layer count for each model individually.
It would be nice, if the best possible amount of gpu layer count can be computed or estimated automagically dependent of user's gpu capability.

Some related upstream discussion:

https://github.com/ggerganov/llama.cpp/issues/3719

I'm going to close this one as wontfix for now. Sorry :(
Implementing it in LLamaSharp itself isn't possible. We would need changes upstream in llama.cpp to calculate the number. Of course if that does ever get implemented in llama.cpp then LLamaSharp will use it!
@ChezzPlaya if you're still interested in this feature please open a feature request on llama.cpp (and link it here). That's the first step required!


TTS + ParallelWaveGAN progress

If you don't mind, I like to share my progress with PWGAN with TTS.
Here is the first try results:
https://soundcloud.com/user-565970875/sets/ljspeech_tacotron_5233_paralle
Results are not better than what we have with WaveRNN, I should say it is much faster.
There is a hissing noise in the backgroung. If you have any idea to get rid of this, please let me know.
The only difference in training (I guess) I don't apply mean-normalization to melspectrograms and I normalize to -4,4 range.

Hi @erogol. Thank you for sharing your progress!
We also try to combine our TTS with Parallel WaveGAN.
You can check our results at google drive.
https://drive.google.com/open?id=1HvB0_LDf1PVinJdehiuCt5gWmXGguqtx
(Maybe phn_train_no_dev_pytorch_train_pytorch_tacotron2.v3 is nice to compare with yours.)
The basic quality is almost the same as yours, but in ours samples, there is no or less hissing noise than yours.
How did you perform normalization?
Did you do generate -> denormalize -> normalize with our stats -> synthesis?

I use the same normalization scheme for TTS and PWGAN which is
self.min_level_db = -100
sellf.max_norm = 4
S_norm = ((S - self.min_level_db) / - self.min_level_db)
S_norm = ((2 * self.max_norm) * S_norm) - self.max_norm
S_norm = np.clip(S_norm, -self.max_norm, self.max_norm)

Did you also observe hissing without mean normalization if you've tried?

I see.
I’ve never tried without mean var normalization.
How about the analysis-synthesis sample?
Is there also hissing noise?
Yes, the samples are generated by ESPnet-TTS.

I came up with an idea.
In the case of WaveGlow, changing the standard deviation of input noise can reduce the high-frequency noise.
It is worthwhile to try to change it in the inference.

@kan-bayashi could you tell me the intuition behind mean var normalization in this case. Or is it just the way in general GANs are trained?
What is analysis-synthesis? Is it just testing the trained model with the training specs?
Yeah makes sense. I'll try the std trick and let you know the result. Thx for the suggestion!

Analysis-synthesis means synthesis with natural (groundtruth) features.
I've never compared among normalization methods.
Actually, the reason why I used mean var normalization is that I am from ASR.
I followed the manner in ASR, so I have no technical reason... Sorry...

Alas, std trick does not work. The lower causes voice loss and the higher makes the noise worse. I'll try mean var normalization and see if it makes the trick.

Thanks for your report, @erogol.
I also tried to combine r = 2 or r = 3 tacotron2 / Transformer.
It seems working without hissing noise.
(You can try in Colab of Japanese demo.)
I'm looking forward to seeing the difference between normalization.

I should also say, more training helps for reducing the noise.

You mean more iterations of Parallel WaveGAN improve the quality?

@kan-bayashi yes more iteration with PWGAN improved the quality.
Then we have only two possible reasons.

normalization
tacotron vs tacotron2

@kan-bayashi are the examples on gdrive with tacotron2.v3 trained with the conf train_pytorch_tacotron2.v3.yaml?

@erogol Yes, the model is based on that config.
https://github.com/espnet/espnet/blob/master/egs/ljspeech/tts1/conf/tuning/train_pytorch_tacotron2.v3.yaml
I compared forward attention with transition agent and location-sensitive + guided attention.
The MOS and CER are almost the same on LJSpeech dataset.

@erogol I confirmed that hiss noise is due to mean var normalization, I am using ESPnet FastSpeech architecture and trained with my own preprocessing with -4,4 normalization, I also get noisy hiss like constant sound with GL vocoder and with WaveRNN both. Though I haven't trained WaveRNN with GTA generated by FastSpeech I just used pretrained LJSpeech.

@rishikksh20 as far as I understand, you actually did not try ParallelWaveGAN but WaveRNN, is this right? Did you also observe better results with mean var normalization for the same models above?

I forgot to trim silences. And retraining it with silence trimmed, made it better. There is still a slight background noise but, it is there in the original recordings too.

@erogol Do you mean that the reason for hissing noise is caused by the silence part? If so, the difference of the normalization methods did not affect the quality, right?

@kan-bayashi I guess yes, but before being sure, I need to run a model with normalization too. But if I compare the latest mozilla-tts results with the espnet examples above, they are on par.

Actually I realized training more exesperated the results and made the noise more apperant. I'll try mean-var normalization for the next time.

The problem above looks like all with silence trimming. After that, things started to work well.
I can also tell preemphasis helps the model to converge more stable.


stdlib: Fix more typed throws and non-copyable generics condfails

There were two categories of problem for older compilers that consume the stdlib .swiftinterface:

The case .some(borrowing x) syntax isn't accepted; use _borrowing
Various functions that addopted ~Copyable generic constraints conflict with the @usableFromInline ABI stubs that were left in to preserve compatibility when the functions are printed with ~Copyable constraints stripped. The stubs need to have different names to get out of the way.

Fixes fallout from https://github.com/apple/swift/pull/73045.
Resolves rdar://127132742

@swift-ci please test

@swift-ci please test Windows


ASIM Network Session schema parser with its sample and test data for SentinelOne

Required items, please complete
Change(s):

Added ASIM Network Session schema parser for SentinelOne based on API Data

Reason for Change(s):

Initial version ASIM Network Session Parser

Version Updated:

Initial version

Testing Completed:

Yes

Checked that the validations are passing and have addressed any issues that are present:

Yes

Hello @jayeshprajapaticrest, thank you for raising Pull Request with us! We will review it internally and get back to you by shortly.

ETA for review is 18th Aug

@vakohl we have mapped the inspection fields below in the NetworkSession parser, which we found from SentinelOne Alerts logs.
ThreatConfidence
ThreatOriginalConfidence


Fix null cursor field in RPC

resolves #200

Thanks!

This changes the attribute name from cursor to cursors. Is this intentional?

😞

but I like the name change :smile:

let's see what dad says

:) :) :)
"cursor" just because it's shorter and less disruptive, though I do like "cursors" as well.


unable to run elasticsearch 5.0 due to bootstrap checks

Elasticsearch 5.0 performs a series of checks at bootstrap to prevent common long running issues.  One of these long running issues is insufficient vm.max_map_count.  The system default is ~65k and ES 5.0 expects a minimum of 256k.
This article:  https://www.novell.com/support/kb/doc.php?id=7000830 suggests there is no effect for kernels that don't consume the extra mappings  and that if a process does consume them there are some small but expected slowdowns but nothing machine threatening.
The only downside I've found so far is a mention that setting this value high can cause kernel memory pressure but again, nothing machine threatening.
As I poke about the internet I'm seeing this come up in regards to a lot of databases.
I would like to increase this value for all cluster nodes to the ES required minimum:  262144

Got around this by creating a init container to do the job.  Its hacky but it means we don't need to expose kernel params through k2 yet.
https://github.com/samsung-cnct/set_max_map_count


Wrong button text color in symptom flow (EXPOSUREAPP-4207)

Description
Text colors in symptom flow are fixed
Steps to reproduce
Scan a positive QR code. Give consent and procced to symtoms from thank you screen

Please rebase one 1.11.x

Tested on Samsung S8:

Probably need to adjust calendar header too.

Adapted day and night mode for consistency and high contrast


enable posting to gaia hub with multi-sig JWTs

In order to upload a profile that belongs to a blockstack id that is backed by a multi-sig address gaia should accept writes that are authenticated via multi-party signature.
I'm not sure this formulation is as precise as I can make it. It is an issue that came up in a conversation with @jcnelson, hopefully he can clarify.

That's a good description, @bodymindarts, thanks.
I think the plan to move forward for this would be to:

Add support in our JWT library (https://github.com/blockstack/jsontokens-js) for multi-sig
Make the changes required in the verifier on the gaia hub to support that.

Though I would be interested in understanding the use cases for this. There's a lot going on I think beyond just supporting this on the gaia hub --- pushing data to the hub from a multi-sig address seems like a pretty hard usability problem to me (different public key holders would have to do independent signatures of the JWT and then one person who holds that JWT would then need to push the data) -- but the JWT used in gaia auth is also generic -- so it would allow a push to any path for that address. So we would probably want to add support for a "path" specifier in the JWT claim.
There's some other dependent libraries that would probably need to be updated to support this as well (blockstack.js's sign+verify in the file routines, probably the putFile would need to accept an optional JWT).

The idea we discussed is to update Gaia's write authentication to require multiple signatures over the challenge text.  This would be enforced by ensuring that the set of public keys provided alongside the signatures continue to hash to the multisig address.
The motivation for this is that in Misthos, there exists the notion of a "shared" record (i.e. a venture) that multiple different people have to sign to consider it valid.  An alternative approach to this that would work today is to have the requisite set of people each sign a shared record, and replicate their signatures to their respective Gaia hubs.  Then, alice.id could check that bob.id and carol.id signed the venture she wrote and shared with them by checking their Gaia hubs for their signatures over the shared venture.

Then, alice.id could check that bob.id and carol.id signed the venture she wrote and shared with them by checking their Gaia hubs for their signatures over the shared venture.

Irrespective of the work (which is significant) required to implement the alternative, the above approach actually sounds better -- this allows, say, alice.id to re-key and still participate in the venture, but also it actually matches the model proposed for Misthos more closely.
Before we commit to trying to solve this problem through a bunch of changes in the developer tooling, we should understand what's involved solving this problem with a multi-sig name owner, the changes required are:

signature verifications in blockstack.js must be changed to enable multisig verification (that's us explicitly writing new signature verification scheme, btw, not a standardized approach)
what does encryption now look like in blockstack.js?
implement specific multi-sig address checks for P2SH, P2WH, P2SHWH style multisig addresses in jsontokens
add support for the above jsontokens implementations into gaia hubs.

That seems like a lot of things to change (with a lot of custom cryptographic code) and a lot of new interfaces to commit to for a use-case which it's not obvious is best solved by this approach.

I understand that this requires a lot of code (which I am happy to contribute). And I am open to alternative implementation approaches, however I haven't yet seen (or perhaps understood) a suggestion that offers 3rd parties a publicly discoverable, independently verifiable and always consistent way to discover data about a venture that does not necessitate the revealing of the ids of participants.

As far as I undestood this forum post multi-sig was on the roadmap anyhow.

Okay, I think I see now.
So -- the idea here would be:

A name, say aaron-co.venture is owned by a multi-sig address <multi-sig-address>
aaron-co.venture has an app-specific address <multi-sig-app-address>, which is used to construct and store a gaia hub URL in the apps field of aaron-co.venture's profile.
participants in the venture construct authentication tokens for writing to the above gaia hub (this should probably be a path-specific token)
participants collaborate to write a signed message to a path 'foo.json' in the above gaia hub using that authentication token.
if I want to read from that venture as another user, I can do a simple getFile('foo.json', { decrypt: false, user: 'aaron-co.venture' }) and I have some way over verifying a signature on that file.

The (non-anonymous) alternative would be:

A name, say aaron-co.venture is owned by a multi-sig address <multi-sig-address>
aaron-co.venture stores a Gaia URL in its apps field
That gaia URL has just one required file (participants.json)
Participants write to foo.json within their own gaia hub.
If I want to read from that venture as another user, I do getFile('participants.json', { user: 'aaron-co.venture' }) and then iterate through the participants, performing getFile on foo.json for each.

I think these have non-trivial trade-offs. The first approach will be much faster for readers, but requires a lot of coordination -- in particular, when it comes to constructing the authentication token, the participants would have to each independently perform their signatures and then secretly communicate it between the participants, because those signatures are being used for authentication. And there's a question about how long that authentication should be valid, and which client actually performs the write. The second approach has an admittedly slow read path and doesn't allow for more anonymous participation, but allows for independent writes.

The first point is certainly a valid question, ideally if all files being uploaded are additionally signed by a quorum of participants the time of validity isn't a huge security issue, as non-signed files would be rejected by clients reading them anyhow.

True, though the authentication token could be used to delete the file.

As to the 2nd approach, what do you mean with ... but allows for independent writes.? The setup which includes setting participants.json would require just as much coordination to upload that file, or?

Yes, it requires just as much coordination to upload that file, but wouldn't require that coordination on every write to other files.

True, though the authentication token could be used to delete (write an empty buffer to) the file.

hmm... this is a tough one. Would it be possible to scope the token to permissible actions? Eg. write but not overwrite, or something like that?

the more think about it the more I like the benefits of the approach with participants.json. Still need to solve the same problems though to get that one persisted. Would it make a difference if the participants got persisted directly into the profile.json instead of another file references from the profile?

True, though the authentication token could be used to delete (write an empty buffer to) the file.

Perhaps authentication could be scoped not only to a /path but also a hash of a file to change / upload.

the more think about it the more I like the benefits of the approach with participants.json. Still need to solve the same problems though to get that one persisted. Would it make a difference if the participants got persisted directly into the profile.json instead of another file references from the profile?

Wouldn't make too much of a difference, though it would require a custom lookup -- basically instead of a getFile you'd call lookupProfile and then fetch the participants entry out of there.

Perhaps authentication could be scoped not only to a /path but also a hash of a file to change / upload.

That could work -- though right now, the gaia hub doesn't actually need to read the uploaded file before passing it to the backend (which improves performance a lot). Alternatively, I suppose the authentication could provide write but not a delete (once that op is supported), and then the higher level signature checks on the data would fail if say, one of the participants tried to write empty data.

signature verifications in blockstack.js must be changed to enable multisig verification (that's us explicitly writing new signature verification scheme, btw, not a standardized approach)
what does encryption now look like in blockstack.js?
implement specific multi-sig address checks for P2SH, P2WH, P2SHWH style multisig addresses in jsontokens
add support for the above jsontokens implementations into gaia hubs.

regarding 2: the whole point of this use case is to make data publicly available, so no need for encryption. Its not really possible to encrypt here anyhow since there is no pub / private key pair.

Doesn't 3 cover most of 1 + 4? As in: verification is all in the library and blockstack.js + gaia just call the library?

Doesn't 3 cover most of 1 + 4? As in: verification is all in the library and blockstack.js + gaia just call the library?

blockstack.js's signature verification doesn't use jsontokens because we wanted the property that if the file was unencrypted, that the raw contents of the file remained the same, and the signature was stored in a metadaa file. So we'd have to port that regime over to the blockstack.js library.

A point that is still an issue for me:
- A name, say aaron-co.venture is owned by a multi-sig address <multi-sig-address>
- aaron-co.venture has an app-specific address <multi-sig-app-address>, which is used to construct and store a gaia hub URL in the apps field of aaron-co.venture's profile.

Is it a must that we have an apps field? I was thinking the owner address could be the same gaia hub URL. Again this use case is for public-only data so no need to segregate the storage space of the apps.
I was thinking the claim field of a multi-sig address could look like:
{
  "@type": "Venture",
  "@context": "http://schema.org",
  "pubKeys": [<pubKey1>,<pubKey2>],
  "nSignersMin": 2,
  "multiSigType": "witness",
  "partnerIds": [<blockstackId1>,<blockstackId2>],
  "gaiaURL": "https://gaia.blockstack.org/hub/<ownerAddress>/",
  }

Or something like that. pubKeys would probably also show up outside the claim field under issuer and subject as it is now.

Is it a must that we have an apps field? I was thinking the owner address could be the same gaia hub URL. Again this use case is for public-only data so no need to segregate the storage space of the apps.

Yeah, that would work -- though some of the functions in blockstack.js may behave kind of strangely, because they assume that they are writing on behalf of a specific application (so they look for that apps entry in the profile).
So in this case, the participants.json file would be uploaded to the owner's gaia address -- meaning a multi-sig address, right?

@kantai Is this issue currently awaiting further discussion before its precise specification is clear?

No, I think a spec for multi-sig auth in Gaia would be fairly clear.

Closing this issue due to lack of activity. Please feel free to comment if this should be re-opened.


open app automatically when killed/closed with notification-data fcm

I want to open app(ionic5) is killed/closed with notification-data.
json for api fcm-google is below:
{
"to": "ca2KNRcxSz2waPUzY1SogM:APA91bH1hopX7x55urxeubs4Y0UQvYAoQtMaciLymjSxN0ht9AshWwXo4fkXpKmASHjyCG2Q8B8pw0X_0PtgfKByWVRPV2ARVQ85OWK8q06XZkYxAeLhIZvpISab9CemfU47CW5IagVE",
"data": {
    "force-start": "1",
    "forceStart":"1",
    "no-cache": "1",
    "body": "Test",
    "title": "Test",
    "sound": "default",
    "icon": "default",
    "badge": 56
},
"priority": "high"
}

I use forceStart in json but not work.
where is the problem from ?

No one to guide?

There is not sufficient information in your issue to diagnose any specific problem - you didn't even say which platform this relates to (iOS or Android).
However, based on your reference to force-start I presume you are referring to what is not a standard Firebase feature but is specific to phonegap-plugin-push which uses it to trigger an intent if the app is not already running on Android.
This plugin does not yet contain such equivalent functionality, however this is an open issue (#235) to integrate the AutoStarter library for Android to resolve this.


Cardborg Helmet now hides identity and ingestion.

About the PR

Cardborg Helmet now hides identity and ingestion.
Resolves #14436
Media

[X] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Changelog

:cl: Whisper

add: Cardborg Helmet now hides identity.
add: Cardborg Helmet now blocks ingestion.

2 line pr :+1:


linker fails to compile

@pierremoreau since your last commit the linker doesn't compile here
/home/airlied/devel/spirv-tools/source/link/linker.cpp: In lambda function:
/home/airlied/devel/spirv-tools/source/link/linker.cpp:177:55: error: conversion from ‘size_t’ {aka ‘long unsigned int’} to ‘uint32_t’ {aka ‘unsigned int’} may change value [-Werror=conversion]
177 |       insn->ForEachId([&id_bound](uint32_t* id) { *id += id_bound; });
|
looks like a size mismatch, did you just build on 32-bit?

I am building on 64-bit and did not get that warning with clang, but I can see that several CI tests failed. Does the above commit fixes it? I will set up a build with gcc and see if I can catch it there.

The current HEAD is indeed failing with gcc, and builds fine with the fix.


⚠ gorzdrav.org has degraded performance

In ad03f84, gorzdrav.org (https://gorzdrav.org) experienced degraded performance:

HTTP code: 200
Response time: 3176 ms

Resolved: gorzdrav.org performance has improved in eff8636 after 5 minutes.


[DEVX-1295] Sign windows binary

Proposed changes
Sign windows Binary before archiving it.
Types of changes

[ ] Bugfix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation Update (if none of the other choices apply)

Checklist

[ ] Lint and unit tests pass locally with my changes
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] I have added necessary documentation (if appropriate)
[ ] I have updated the json schema (if appropriate)

Further comments
Before merging, this require to generate a certificate/key pair and set pipeline secrets.

Not required yet.
Will be implemented if becomes needed.


Update FAQ.md about time to make iOS and Web stable.

Issue https://github.com/JetBrains/compose-multiplatform/issues/3711

Soon (™)?

The FAQ was rewritten here


Disabled snapshots still throws error if wrong datatype in period fields

Version - 2.2 master 11/13
Steps to reproduce (least amount of steps as possible):

Create a RKE cluster
Open Advanced Cluster Options
put in abc in snapshot creation period and snapshot retention period
Disable snapshots
Click create

Result:  Error "Snapshot Period Error:  You must designate a time value and SI time unit.  You entered abc".  Should ignore fields if you have disabled snapshots and not give error.

Version - 2.1.2-rc12 - verified fixed, leaving open for master.

Version - master 11/14
Verified fixed


Wolves occasionally stop pathing to players right next to them

Traced this issue to path finding inconsistencies:
Creating a PathEntity uses floored boundingbox minima to determine from which coordinate to which coordinate a path should be constructed, that means that it grabs the coordinates of the North-West most point of the Entity hitbox.
If i would had to guess this is done to get predictable pathfinding for bigger entities?
while processing, the path info uses the horizontal center of the entity and compares it to the center of a block (example 50.5D, 64..5D, 50.5D) to see if it reached a path point.
since changing pathfinding could pose major side effects I thought I'd rather make a report instead of push through changes..
solution one:
PathEntity creation could be changed to be determined from the center of Entities instead of their Northwest corner. There is a bit of offsetting happening for small entities (PathFinder: 509) and I find it hard to determine how this would affect it
solution two:
Could change the processing to check the northwest corners instead of the horizontal centers. the method used for this (PathNavigate: 303 getEntityPosition() seems to only be used by this process, or point the process to a new method getEntityPathPosition() or something like that, so that EntityPosition doesn't become a weird confusing name
I'm not sure Either one of the solutions is the way to go here, or we should let it be

On another note, with two small hitboxes. even pathing to the right coordinate could mean the hitboxes don't touch in very rare cases

Anecdotally (with the bug) this means wolves will wait to attack you in a bedroll until the morning.  Have watched several times a wolf, even mad wolf, get into my base and look at me until morning.

Bumping this as it's still present in the latest CE 3.0 Snapshot (2.1), specifically was able to reproduce the behaviour stated by LannyRipple here.


"Snort" source link out of date

Mac$ brew install snort
==> Downloading https://www.snort.org/downloads/snort/snort-<IP_ADDRESS>.tar.gz
curl: (22) The requested URL returned error: 404 Not Found
Error: Failed to download resource "snort"
Download failed: https://www.snort.org/downloads/snort/snort-<IP_ADDRESS>.tar.gz
The new path is:
https://www.snort.org/downloads/snort/snort-<IP_ADDRESS>.tar.gz
Can I customize the install process with new version number?
Regards

Can I customize the install process with new version number?

Afraid not. I've updated the upstream link though at Homebrew so brew install snort will succeed now.


move nodeInterfae to k8sasssit - [closed]

In GitLab by @luotian-github on May 15, 2018, 03:09
Merges updatek8s18 -> updatek8s18
move nodeInterfae to k8sasssit

In GitLab by @wknet123 on Nov 13, 2019, 10:54
closed


M46015: Fixing typo appes/apples

Translator has reported possible source content issue.
Description: appes should be apples

Hi @amynic , @revodavid, can you please approve and merge this PR?
Thanks in advance.

Thanks @v-maudel for spotting and correcting this for us - this makes the content better for everyone ⭐⭐

Thanks to you @amynic


Patch to add front() and back(), as per issue #2123.

Adds the C++11 functions front() and back() to std::string, as well as unittests for functionality.

Thanks!
The test is failing on travis due to missing C++11 support. You can fix that by moving the test into a new test file and adding the tag cpp11 at the top.

Also, this doesn't handle the const string case correctly. I guess Cython might end up generating incorrect code in this case, e.g. when it needs to assign the result to a temporary variable or when type inference comes into play.

@scoder I thought Cython generally doesn't handle const? I can always add both const and non-const overloads.

@scoder I've fixed my pull request, however, due to the current format of the element access methods in std::string, I decided not to const-qualify the front() and back() methods.
Currently, my diff looks something like this:
        char& at(size_t)
        char& operator[](size_t)
+        char& front()
+        char& back()

After your comments, I attempted to create const-qualified versions of all member functions, as follows:
        char& at(size_t)
        const char& at(size_t) const
        char& operator[](size_t)
        const char& operator[](size_t) const
        char& front()
        const char& front() const
        char& back()
        const char& back() const

However, as I expected, with my current version of Cython, attempting to compile such code culminates in a warning of:
=== Got: ===
64:16: Indexing 'string' not supported for index type 'long'
64:30: ambiguous overloaded method

@scoder Any updates about this pull request?

Looks good to me and fixes a problem.


Fixate type of list in a program

Currently, a type of a list changes in a program when updated.
x = [1, 2, 3] # x : [int, int, int]
x.append(4)   # x : int list

(cf. https://github.com/pfnet-research/chainer-compiler/compare/master...momohatt:list-invariant?expand=1#diff-5c356c79d973908cc78cef75c3d6ed41L526-L527 )
However, because Python lists are mutable, such changes to types can ruin the soundness when alias analysis is not present.
This PR fixes this problem by fixating the type of lists in a program as follows.
x = [1, 2, 3] # x : int list <- not '[int, int, int]'
x.append(4)   # x : int list

y = [1,2]      # y : optional(int) list
y.append(None) # y : optional(int) list

/test


build quantas container

Hi @danielnachun I am trying to build quantas container to streamline analysis, for now I just need to add quantas and samtools into it, please let me know if there is any problem! Thanks!

Looks good to me!


Modify the salt master's log config file name

In the salt master's config folder, the file name "log-level-debug.d" change to "log-level-debug.conf" as it's a configure file.

@k8s-bot test this

LGTM

Shippable appears to be a flake.


After introducing dependencies, iOS starts to report an error

After the introduction of dependencies, nothing has changed, and an error is reported when iOS starts
Flutter version 3.3.5
Audioplayers Version 1.1.1
Device iPhone7  iOS14.7.1
OC selected by iOS language when creating Flutter project
Launching lib/main.dart on 988的iPhone in debug mode...
Automatically signing iOS for device deployment using specified development team in Xcode project: T6924M2DC5
Running pod install...
Running Xcode build...
Xcode build done.                                           34.2s
Installing and launching...
(lldb) dyld: Library not loaded: @rpath/libswiftCore.dylib
  Referenced from: /private/var/containers/Bundle/Application/B0310FBD-0ACA-43BD-9E6D-79D3D324B4EE/Runner.app/Runner
  Reason: image not found
* thread #1, stop reason = signal SIGABRT
    frame #0: 0x0000000102cb6b64 dyld`__abort_with_payload + 8
dyld`:
->  0x102cb6b64 <+8>:  b.lo   0x102cb6b80               ; <+36>
    0x102cb6b68 <+12>: stp    x29, x30, [sp, #-0x10]!
    0x102cb6b6c <+16>: mov    x29, sp
    0x102cb6b70 <+20>: bl     0x102cb5168               ; cerror_nocancel
Target 0: (Runner) stopped.
Error launching application on 988的iPhone.

Just found a solution. The official will see if there is a better solution
My solution is to add these two paths to the "Runpath Search Paths" of the iOS project
/usr/bin/swift
/usr/lib/swift

As shown in the figure

Can you compare with our example?
https://github.com/bluefireteam/audioplayers/tree/main/packages/audioplayers/example/ios
Does the error also occur on a fresh project?


fix(deps): remove unused deps and add build check

Superseded by https://github.com/fooker/photonic/pull/29


Add tests for bugfixes in #453

Adds tests for two of the bugfixes shipped out in pull request #453.
Specifically,

checks that the correct supplier_id is added to the audit event created on a generic framework update
makes sure that the wrong FrameworkAgreement will not be updated

In the latter case, the problem was that the previous code would look for any FrameworkAgreement belonging to a supplier and update it.  This meant that if

a framework agreement existed for a supplier
a framework agreement update came in for a different framework
the framework agreement for the first framework would always be returned and updated.  A new framework agreement would never be created.

This test verifies that even if a framework agreement for another framework exists, a new framework agreement is always created.

💀

After talking with @idavidmcdonald, it sounds like both of these tests are no longer needed.

The first tests tests the second of the two audit events that are being created in the update_supplier_framework_details route, but apparently this route is being hacked up over the next two days.
The second test makes sure that the correct framework is returned in the same route.  However, since we're moving towards a world where a SupplierFramework has multiple FrameworkAgreements, the logic this test relies on is also being cut up and redistributed.


Sudden crash of files application

Description
I am running this on windows 10 laptop with high specs still it usually just crashes by itself when i need to use it,

here as you can see clearly why this is happening is not mentioned, I kept it as default application still its no good for me. It usually crashes itself in 2-3minutes.
Steps To Reproduce
No response
Expected behavior
It is supposed to run without crashing for however long its opened. Crashing of an app, means what to say....
Files Version
Version: <IP_ADDRESS> OS Version: 10.0.19043.1586
Windows Version
Windows 10 version 21H2
Relevant Assets/Logs

You will need to upload the logs files directly to github. You can get them in files settings > about > open log folder.


Is there any inputListener

i have to call ok button on 6th number entered?

There's a large tutorial for a reason 😛

Thanks i did it... Awesome library bro


Outdated docs mention http_header_filter.

Parts of the config documentation still mention the old http_header_filter option that is deprecated. The docs should only show how to use the new http_filter, which supports both header_filter and path_filter.
I've spotted this in the docs for IncomingFileConfig, and in LayerConfig.
To fix this, change the docs to show the newest config, re-build the schema, run medschool and publish the generated docs to mirrord.dev.
Side question: For how long are keeping both versions? What's our policy for removing deprecated stuff?

closes by https://github.com/metalbear-co/mirrord.dev/pull/210


Upgrading Igor-oes from OES-1.30.X to OES-1.33.X

https://devopsmx.atlassian.net/browse/OP-22085 : ParentJira
https://devopsmx.atlassian.net/browse/OP-22196 : JIra :
Testing :
Compile/Build got success, All TestCases passed, No new TestCases failed.
igor-monitor-plugins: Results: SUCCESS (6 tests, 6 successes, 0 failures, 0 skipped)
igor-core: Results: SUCCESS (43 tests, 43 successes, 0 failures, 0 skipped)
igor-monitor-artifactory: Results: SUCCESS (9 tests, 9 successes, 0 failures, 0 skipped)
igor-monitor-travis: Results: SUCCESS (106 tests, 106 successes, 0 failures, 0 skipped)
igor-web: Results: SUCCESS (350 tests, 350 successes, 0 failures, 0 skipped)
Deployed docker image in dev-spin-cve-cluster cvetarget ns
UseCase tested : Configured jenkins in cvetarget ns and tested the jenkins pipeline stage. working as expected.
verified igor logs, looks clean
Note :
All commits covered - verified from sheet.
fiatVersion=1-0-SNAPSHOT & korkVersion=1.33.2-nemesis-SNAPSHOT

Verified few commits from sheet - LGTM
Verified .gradle files - LGTM (just check few comments)


[Run] Fix environment helper for nested environment variables

Summary of the Pull Request
There is a bug in the environment code somewhere that causes wrong results on environment updates if there are nested variables.
This PR implements a fix: We call the update code twice.

PR Checklist

[x] Closes: #26864
[x] Communication: I've discussed this with core contributors already. If work hasn't been agreed, this work might be rejected
[x] Tests: Added/updated and all pass
[ ] Localization: All end user facing strings can be localized
[ ] Dev docs: Added/updated
[ ] New binaries: Added on the required places

[ ] JSON for signing for new binaries
[ ] WXS for installer for new binaries and localization folder
[ ] YML for CI pipeline for new test projects
[ ] YML for signed pipeline

[ ] Documentation updated: If checked, please file a pull request on our docs repo and link it here: #xxx

Detailed Description of the Pull Request / Additional comments

Validation Steps Performed
Build code local and test the behavior and variable update of nested variables.

PR is ready for review

@crutkas
Can we get this fix in for 0.71.0? It is important to have correct environment variables in app opened from PT Run.

Why do 2 calls fix the issue? If there is a problem with EnvironmentUpdate code, I guess we should focus on fixing it there

Because the problem is not in our code. I tried with extra code for replacement of variables in strings (Environment.ExpandEnvironmentStrings()) and with Thread.Sleep() before reading the "new" environment. Both didn't fix the problem. The only thing that helps is executing it twice. For me it looks like a bug in .Net or in Windows.

Why do 2 calls fix the issue? If there is a problem with EnvironmentUpdate code, I guess we should focus on fixing it there

Because the problem is not in our code. I tried with extra code for replacement of variables in strings (Environment.ExpandEnvironmentStrings()) and with Thread.Sleep() before reading the "new" environment. Both didn't fix the problem. The only thing that helps is executing it twice. For me it looks like a bug in .Net or in Windows.

I see. It does fix the problem, even though it is weird :) I saw in the comments that there is similar behavior so Env Var dialog as well, so it makes sense.


comments in transcripts

It might be worth allowing comments in plans and/or transcripts. Both human written (for marking up tests in a corpus) and also machine-written (eg. we might allow grammar productions to carry comments that accumulate as an explanation / commentary for a plan).

fixed in b77ebc9c0362d7fa3e1f50c0668ceb36d2217e79


[Integrated Tests] Invoke lit.py using python3

Also provides an escape hatch to invoke with the system default (just launching lit.py as a standalone script executable)
LLVM is transitioning its script infrastructure to python3 and we may run into issues if we continue using the system-default python2 on some platforms.
Resolves rdar://75848017

@swift-ci please test

@swift-ci please test

@swift-ci please test


Printing out the event name that triggers the Action

Current dependencies on/for this PR:

main

PR #394   👈

This comment was auto-generated by Graphite.


Probable bug: can't enable Share Lending in auto investing accounts if you didn't check the box during the initial account application

Under the "More" section of an auto investing account, there should be a section and slider to enable/disable Share Lending. If, during the application process, however, you did not check the box to approve Share Lending, then you can never activate this feature later. That's the bug. There should be a way to activate it after opening the account.
I just called SoFi and the rep said it was active on my account, and would probably show that button in the next 24 hours or so, but it's not showing up now. I need to confirm this bug. If this is not the case that it shows up later, then the bug is confirmed and I need to screenshot the whole account opening process and everything too, to show how to replicate it.
 This would be really dumb.

On a computer, under a Joint Automated investing account, I can click More --> Share lending at the left, and here's what I see: an option to turn it on and off:

On my account I Just opened, I can see this too! So, it's not a bug, I think.

But, there is a different problem: the $6 I put in there over the last 2 weeks hasn't resulted in any automated trades to buy into the stock market. Why? It should have.

I called SoFi, and spoke to Joe. He had to turn on a "Rebalance" feature. He thinks he may have overlooked it since he had to manually approve the opening of my account for me before after verifying proof of address documents.
This "rebalance" feature needs to be turned on for automatic rebalancing ETF purchases based on your risk selection in these automated accounts. So, now that it's on, I should see my $6 get automatically purchased into the appropriate "aggressive" strategy ETFs over the next 1 to 2 business days (ex: by Monday or Tuesday, since today is Friday). So, that's done.

Closing this issue as NOT A BUG regarding the Share Lending thing above.


IGNITE-19021 Support the directory deployment

https://issues.apache.org/jira/browse/IGNITE-19021

@valepakh I've launched the CLI in the project's root directory and run:
unit deploy --path ./examples --version 1.0.0 my.unit
Then I listed the work/deployment/my.unit/1.0.0 directory and I could not see the nested content of the exampled directory. I think we have to support it and it would be nice to add some tests for nested directories support.


Multistep Form - Logic Issue

The multistep from is displaying "How quickly do you need your one time shredding service?" on step 4. Regardless of the service selected on step one. This question should only appear if the user selects one time shredding service. Please ensure this is fixed across all variation of the multistep form (modal, inline, campaign, etc.)

Example page: https://main--shredit--stericycle.aem.live/en-us (click Get A Quote in header)

@colinvlasak

One time shredding service and  Hard drive destruction has the same value "Purge" is this correct?
We use input values to hide and show, so the rule will execute for both the options.

@ravise5 the way it is mapped on our current site it is directly dependent on the servicetyp1 based on name not the value.
Example form: https://author-p29452-e94470.adobeaemcloud.com/ui#/aem/aem/experience-fragments.html/content/experience-fragments/shred-it/us/en/site/multistep-inline-form

Technically for the value both pass through as purge but one time flags the service as paper while hard drive flags as hard drive.

The "How quickly do you need your one time shredding service?" on step 4 should only apply to users who select the One-time shredding option
@rwcsrichardspencer may have more details on how this works.

@colinvlasak This is now done for all forms

@ravise5 looks good!


11k sat transaction sent/received - zero balance, no channel created

From first Phoenix wallet I made 11k sat transfer to another, freshly created Phoenix wallet.
Transaction was sent properly, and properly received 10k sat on the other end. But available balance is still 0 and no channel in Settings -> Channels List. Fresh Phoenix wallet looks like this:
https://imgur.com/a/pdVRXSg
What to do now?

Sorry about that, a regression on our node. Should be fixed now if you restart Phoenix

restart or reinstall? I forced switch-off the application and start - and there is no effect of it...

Yeah, you're right, this is just fixing future channels. This is going to take a bit longer stay tuned

Ok, should be good now.

yes, fixed! thank you
btw, what if I would send 10k? 9k on the other end is enough?
(if not - that in fact 11k sat requirement - should be more highlighted, imo)

The reaon for this bug is that we accidentally pushed an update to the node that required Phoenix to also be updated. It should have been done the other way: first Phoenix then our node.

btw, what if I would send 10k? 9k on the other end is enough?
(if not - that in fact 11k sat requirement - should be more highlighted, imo)

You should be able to send 10k, Phoenix will receive 9k. Just tried on testnet

works with 10k/9k on mainnet, too
(wen iOS? ;)


Always call benchmark script with absolute path + actually test this

This wasn't caught previously, because all tests used absolute paths in the config.
See https://github.com/spacetelescope/asv/commit/7564627ba4af275d0d753fefd22e5fe80a3c75e5#commitcomment-12233571

Thanks.  Makes sense.


Replace Tippy with Radix tooltip

We should refactor our tooltips and migrate to use Radix tooltip

Yesssss!

https://www.figma.com/file/2MLHeIeL6XPVi3Tc2DfFCr/❖-Leather-–-Design-System?type=design&node-id=11733-62874&mode=design&t=3Isi5zWhyxxcp1jr-4

@edgarkhanzadian I believe this ticket is currently in progress in development?


Dynamic output filename template

Checklist

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

Description

WRITE DESCRIPTION HERE
Currently, if the variable value of the output template does not exist, it is replaced with the NA value. Instead, it would be nice to have the ability to take additional candidate groups dynamically and first set the output template in the order of correspondence.
For example, the user sets multiple -o options in the config file and takes the setting that NA does not occur in the order listed.

hey, I know that is really old, is there an option for that by now?

hey, I know that is really old, is there an option for that by now?

Not exactly the same as what I said, but yt-dlp supports similar options
https://github.com/yt-dlp/yt-dlp#output-template-examples


JointClientKit API: custom plugin/display paths

The current search path list in the plugin host API searches paths that are relative to the executable API. However, this is not always the location where plugins are located. For instance, in an Android application, the executable path will be /system/bin/app_process32, and the plugins (for JointClientKit) will need to be copied from the app bundle to its /data/data/com.packagename/files directory for it to have access to them.
This affects any search paths in the server or plugin APIs when using the JointClientKit - for instance, when explicitly specifying a display config file, or when auto-loading plugins or loading a specific named plugin via the JointClientKit APIs. There needs to be a way to customize the search paths.
You could just have one API call to override the root executable directory. This option might make it easier to catch all the places where we generate search paths without making a separate API for each search path (display json vs plugins, et. al.). The developer would only need to recreate the typical osvr_server file layout at the that directory. It might be easier to implement as well, requiring only that we centralize any code that looks for the "executable directory" to call an internal utility function that applies the override, if present.
Alternatively, you could have APIs for each search path set (plugins, files referenced in the server config json, etc...) to add one or more custom search paths for each separately, but this would be more complicated to implement and to use. It might be more flexible, however.

The referenced pull request adds the current working directory to the search path. Might be a good short term solution for Android. Doesn't solve the problem for files referenced from the config json.


fix: removed unused files

Removed unused files as per comments for issue #47971
Checklist:

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the main branch of freeCodeCamp.
[x] I have tested these changes either locally on my machine, or GitPod.

Ref #47971

Hey @anubhav823
Thank you for opening this pull request.
This is a standard message notifying you that we've reviewed your pull request and have decided not to merge it. We would welcome future pull requests from you.
Thank you and happy coding.

Hey  @Sboonny
Can you please let me know what I am missing in my pull request so that I can correct it?

no worries, we are using these files. so we closed this pr

@Sboonny Thank you for reopening the PR!

No worries, sorry for closing it

Hi @ojeytonwilliams
Yes, I am working on the other tasks of the issue. As the ask requires changes to several files, I thought it would be easier to fix some issues at a go.

@anubhav823 perfect, thanks!  I've updated your original comment on this PR so that it no longer closes the issue, but multiple PRs are very welcome.

Hcyfh

@ojeytonwilliams I see some files mentioned under Unused Exports section such as-
Block src/templates/Introduction/components/block.tsx
Should I remove the entire Block class from the file?

Okay. I will do that.
Also, is it possible for you to merge the PR in its current state?

We'll need to QA it first, but definitely. Please feel free to create a second PR if you'd like to keep working on this.

Okay.
Thank you!

The tests caught a few issues, I'm afraid. It's worth running npm lint before committing to make 100% sure you've not removed something that we need.
For example, universal-nav exports the class twice, but it is the default export that's used, so that cannot be removed.

I will keep that in mind.

I will keep that in mind.

You will also need to update the PR to change that

@ojeytonwilliams @ieahleen can someone help me out? I am unable to understand why the Lint (16.x) build is not getting triggered.

The tests caught a few issues, I'm afraid. It's worth running npm lint before committing to make 100% sure you've not removed something that we need.
For example, universal-nav exports the class twice, but it is the default export that's used, so that cannot be removed.

I have fixed the build.

Thanks @anubhav823, it's looking good.  All being well I should be able to get this reviewed later this week.

@ojeytonwilliams Thank you! Meanwhile, I will continue to work on the other files of the issue.

Thanks again for helping out with this.
FYI, https://github.com/freeCodeCamp/freeCodeCamp/pull/48019 already seems to cover the remaining issues (though there are some failing tests), so more work may not be needed.

@ojeytonwilliams Thank you! Hope to contribute more to freeCodeCamp!

 Thanks again for the help.