Integration with bazel-compile-commands-extractor

bazel-compile-commands-extractor can generate compiled_commands.json files. I tried to hot patch the shell binary and add -p $PROJECT_ROOT to the clang-tidy command. However, it seems that still clang-tidy fails to find some headers.
https://github.com/hedronvision/bazel-compile-commands-extractor
The issue seems to be a mismatch between the config and compilation mode. bazel-compile-commands-extractor accepts extra flags like -- --config=debug --compilation_mode=dbg, but bazel_clang_tidy uses fastbuild by default.

I wonder if @hedronvision can help exposing some of the things that bazel_clang_tidy needs. It seems that adding -p project_root doesn't fix the missing header errors.

Hey, @aminya!
My understanding is that clangd embeds clang-tidy these days, providing feedback as you edit if you want. So if you already using clangd and generating a compile_commands.json with our tool, maybe that'd be a good way to get automated style feedback! Would love to hear your experience with that if you end up trying it.
[It looks like the goal of this project is a little different. They're looking to accommodate folks without a compile_commands.json (or clangd) and provide the style feedback as a build-style error. That's great stuff, but you've got the compile_commands.json and clangd already, I think. (All that said, I'm no clang-tidy expert (yet)--nor experienced with this project, so please tell me if I'm wrong or it there's a key integration we should be making.)]
Cheers,
Chris

P.S. For generated files, you'll want to have flags match when you build and when you use the tools! It's definitely a tricky thing to handle well for tool builders.

My issue with clangd's clang-tidy is that it doesn't show all the issues in the workspace to me. I need to open each file to trigger it. It also doesn't seem to work when a function is templated. I have to call that template in my tests to trigger those warnings. That's why running clang-tidy form CLI still shows some warnings that are not shown by clangd.

Oh, got it. Makes sense.
To confirm, is the draw of this tool, then, over just running clang-tidy directly w/ the compile_commands.json, speed, since it takes care of caching via bazel?
Seems like some of the same questions remain, though: Is this tool designed to take a compile_commands.json file if you supply one?

It is not, there's no need for a json file

Hey, Thaler! Great to meet another Bazel toolmaker. Seems like a great thing you've built here.

I realized that with the help of  hedronvision/bazel-compile-commands-extractor, running clang-tidy becomes just a matter of specifying the .cpp files (for example using git ls-files). It works accurately without any missing headers!
bazel run refresh_compile_commands -- --config=debug_asan --compilation_mode=dbg

files=$(git ls-files --exclude-standard | grep -E '\.(cpp|c|cc|cxx)$')
clang-tidy -p ./ $files

It might be worthwhile adding this to the readme files.

@aminya, yeah! That's what I meant. You mean adding it to our README, yeah?
Glad to hear getting the compile commands solves the header things for you.
I assume it doesn't have the same caching as it would under this tool. But fast enough for you?

@aminya, I'm creating an issue to track over at https://github.com/hedronvision/bazel-compile-commands-extractor/issues/52


Issue: 498 meta service long run fix

Problem Desc:
We have a limitation on max chunk size (uint32_t) which is 4GB, which will be carved into portions in blk allocator with each portion maps to tens of thousands of nblks available for each portion. This specific test case specifiy max io size to to limitation that is larger than the portion total nblks and during the test, the io size is randomized and when it comes to larger than the limit, it can't find available allocation in any chunk so return failed.
Fix:

adjust the test case to not exceed limitation
refactor the code a little to get max chunk size from Chunk class.

Test:
Specific failed test (previously failing consistently) can pass with this.

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 67.40%. Comparing base (1a0cef8) to head (fbf6d6a).
Report is 48 commits behind head on master.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@             Coverage Diff             @@
##           master     #523       +/-   ##
===========================================
+ Coverage   56.51%   67.40%   +10.89%
===========================================
  Files         108      109        +1
  Lines       10300    10411      +111
  Branches     1402     1398        -4
===========================================
+ Hits         5821     7018     +1197
+ Misses       3894     2709     -1185
- Partials      585      684       +99

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


Replaced for loop as mentioned in comment

Also contains a commit to get the code coverage back to the old value: added a statement to ASSERT the pointer to default loop returned is the same on the second call to Loop::getDefault.

Coverage increased (+0.2%) to 98.069% when pulling efc86768c9c2c44c78c2768dd5c8386be0a05041 on tusharpm:master into 98d563ad9ec3d44fd4d6f039c11ab0da2897de72 on skypjack:master.


Building failing in arm cortex-m0

Building is failing when CONFIG_NO_OPTIMIZATIONS is enabled. This is the output for ./samples/hello_world/ with bbc-microbit as target.
Memory region         Used Size  Region Size  %age Used
           FLASH:      103636 B       256 KB     39.53%
            SRAM:        9240 B        16 KB     56.40%
        IDT_LIST:         120 B         2 KB      5.86zephyr/arch/arm/core/libarch__arm__core.a(isr_wrapper.S.obj): In function `_idle_state_cleared':
./zephyr/arch/arm/core/isr_wrapper.S:132:(.text._isr_wrapper+0x2c): relocation truncated to fit: R_ARM_THM_JUMP11 against symbol `_IntExit' defined in .text._HandlerModeExit section in zephyr/arch/arm/core/libarch__arm__core.a(exc_exit.S.obj)
collect2: error: ld returned 1 exit status
%
ninja: build stopped: subcommand failed.

Reproduced.
diff --git a/samples/hello_world/CMakeLists.txt b/samples/hello_world/CMakeLists.txt
index 9c3c38b669..ad74422ed8 100644
--- a/samples/hello_world/CMakeLists.txt
+++ b/samples/hello_world/CMakeLists.txt
@@ -1,5 +1,7 @@
 cmake_minimum_required(VERSION 3.8.2)

+set(BOARD nrf51_pca10028)
+
 include($ENV{ZEPHYR_BASE}/cmake/app/boilerplate.cmake NO_POLICY_SCOPE)
 project(hello_world)

diff --git a/samples/hello_world/prj.conf b/samples/hello_world/prj.conf
index b2a4ba5910..8c8a1404c5 100644
--- a/samples/hello_world/prj.conf
+++ b/samples/hello_world/prj.conf
@@ -1 +1,2 @@
 # nothing here
+CONFIG_NO_OPTIMIZATIONS=y

Fixed.
Not sure what kind of consequences this fix has though, it might be breaking things ...
diff --git a/arch/arm/core/isr_wrapper.S b/arch/arm/core/isr_wrapper.S
index 53c24645d7..d3c24d33f1 100644
--- a/arch/arm/core/isr_wrapper.S
+++ b/arch/arm/core/isr_wrapper.S
@@ -133,4 +133,4 @@ _idle_state_cleared:
 #endif /* CONFIG_ARMV6_M_ARMV8_M_BASELINE */

 	/* exception return is done in _IntExit() */
-	b _IntExit
+	bl _IntExit
diff --git a/samples/hello_world/CMakeLists.txt b/samples/hello_world/CMakeLists.txt

The root cause and solution options are described in https://github.com/zephyrproject-rtos/zephyr/commit/17652760001cec19cd25f48fa18d0c973f5ca345#diff-ef6459503728d8b77ea93b33e8de0117
Personally I think that porting to C is the best long-term solution. Not sure what to do short-term ...

Fixed.
Not sure what kind of consequences this fix has though, it might be breaking things ...
diff --git a/arch/arm/core/isr_wrapper.S b/arch/arm/core/isr_wrapper.S
index 53c24645d7..d3c24d33f1 100644
--- a/arch/arm/core/isr_wrapper.S
+++ b/arch/arm/core/isr_wrapper.S
@@ -133,4 +133,4 @@ _idle_state_cleared:
 #endif /* CONFIG_ARMV6_M_ARMV8_M_BASELINE */

 	/* exception return is done in _IntExit() */
-	b _IntExit
+	bl _IntExit
diff --git a/samples/hello_world/CMakeLists.txt b/samples/hello_world/CMakeLists.txt

EDIT: This fix is no good.

Yeah, it override the link register, one change that works is save lr register into other register and restore it in _IntExit. But my re-organize these sections make more sense to me.


TestCase custom beforeRefreshingDatabase is not called

Laravel Version: 9.13.0
PHP Version: 8.1.4
Database Driver & Version: MySQL 5.7

Description:
Following this: https://github.com/laravel/framework/pull/42073 example. The custom beforeRefreshingDatabase is not called when used. I'm not seeing any option to make sure the function is called.
Steps To Reproduce:

Use this testcase

<?php

namespace Tests;

use Illuminate\Contracts\Console\Kernel;
use Illuminate\Foundation\Testing\RefreshDatabase;
use Illuminate\Foundation\Testing\TestCase as BaseTestCase;

class TestCase extends BaseTestCase
{
    use RefreshDatabase;

    protected function beforeRefreshingDatabase()
    {
        dd('hi');
    }

    public function createApplication()
    {
        $app = require __DIR__ . '/../bootstrap/app.php';

        $app->make(Kernel::class)->bootstrap();

        return $app;
    }
}

'hi' is not dumped

Nevermind, in another part it was overwritten


wall doesnt check if cancelled on doafter

Description
lets you rapidly make wall
Reproduction

make girder
start adding steel
move
instantly made

add args.Cancelled check to the doafter handler

also the screwdriver doafter still breaks doors


Why we limit 51 types of resources on the resource type?

Why we limit 51 types of resources on resource health check?
can we add other type of resources?
https://learn.microsoft.com/en-us/azure/service-health/resource-health-checks-resource-types

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

ID: 9b92884e-4bca-47ca-9d8e-4c29c8169b36
Version Independent ID: db3ee172-6788-dc57-c56e-95156023373b
Content: Supported Resource Types through Azure Resource Health - Azure Service Health
Content Source: articles/service-health/resource-health-checks-resource-types.md
Service: service-health
GitHub Login: @rboucher
Microsoft Alias: robb

@shendan19880401
Thanks for your feedback! We will investigate and update as appropriate.

@shendan19880401 - Thanks for bringing this to our attention.
I'm going to assign this to the document author so they can review and update accordingly.

Thanks for your dedication to our documentation. This issue has been moved to an internal work item for triage and prioritization. Thanks in advance for your understanding as we investigate to provide the most accurate documentation updates. #please-close
Internal work item: https://dev.azure.com/msft-skilling/Content/_workitems/edit/271101
Please also note that GitHub documentation issues are being phased out completely. For more information, see: https://aka.ms/ContentUserFeedback.
Our support team is available to assist you if you have any pressing questions or problems that need to be resolved. You can create a support ticket by visiting the following link: https://azure.microsoft.com/support/create-ticket/
You may also ask support questions in our Q&A forums.


Enabling environmental variables from SSH - to Dev

Copy work item to fix CI for Filipe's PR here : https://github.com/Azure/azure-functions-docker/pull/618
From that

The proposed change aims to allow access to the environmental variables of Linux Function App from the SSH console, similar to what is possible on Azure Linux Web Apps.

The changes are based on startup script found on Linux Web Apps https://github.com/Azure-App-Service/ImageBuilder/blob/master/GenerateDockerFiles/dotnetcore/debian-9/init_container.sh

@balag0 - please take a look as well. Thanks @ffarinha-msft for investigation and the PR

Had a chat with @CooperLink few days back on some of the open questions. looks good.


DOC, MAINT: forward port 1.4.0 relnotes

forward port the SciPy 1.4.0 release notes updates following the final release

Merged, thanks @tylerjereddy


Rename scope

What commit message should be if I rename/move file into other scope?

I think you can include 2 scopes EG:
refactor(scope1,scope2): rename

Well, commitizen can't do something like this :(

I think you just need to type scope1,scope2 as your scope

Closing this one due to inactivity :)


/mnt/routes are not loaded upon start

In v3.1, /mnt/routes are not loaded upon start. The same configuration was working on v3.

I'm using docker-compose, and this is the section for logspout:
logspout:
image: gliderlabs/logspout:master
environment:
DEBUG: "true"
volumes:
- /var/run/docker.sock:/var/run/docker.sock
- ./logspout:/mnt/routes
links:
- logstash
And the contents of ./logspout/6898d7162f43.json (which is the only file there):
{
"id": "6898d7162f43",
"filter_name": "apollo",
"adapter": "syslog+tcp",
"address": "logstash:5028"
}

What is the output of docker inspect on that container?

Here it goes:
$ docker inspect apollodocker_logspout_1
[
{
    "Id": "4e602825528dae339b3832827a67b57748376c1719a2e63273a1c65fdc824173",
    "Created": "2016-05-26T10:07:06.736961017Z",
    "Path": "/bin/logspout",
    "Args": [],
    "State": {
        "Status": "exited",
        "Running": false,
        "Paused": false,
        "Restarting": false,
        "OOMKilled": false,
        "Dead": false,
        "Pid": 0,
        "ExitCode": 1,
        "Error": "",
        "StartedAt": "2016-05-26T10:07:07.399453761Z",
        "FinishedAt": "2016-05-26T10:07:07.479579937Z"
    },
    "Image": "081ae548f676f7a32be1777bebd901a5f1073ff2fdcf866e21f644b6fc6af181",
    "ResolvConfPath": "/var/lib/docker/containers/4e602825528dae339b3832827a67b57748376c1719a2e63273a1c65fdc824173/resolv.conf",
    "HostnamePath": "/var/lib/docker/containers/4e602825528dae339b3832827a67b57748376c1719a2e63273a1c65fdc824173/hostname",
    "HostsPath": "/var/lib/docker/containers/4e602825528dae339b3832827a67b57748376c1719a2e63273a1c65fdc824173/hosts",
    "LogPath": "/var/lib/docker/containers/4e602825528dae339b3832827a67b57748376c1719a2e63273a1c65fdc824173/4e602825528dae339b3832827a67b57748376c1719a2e63273a1c65fdc824173-json.log",
    "Name": "/apollodocker_logspout_1",
    "RestartCount": 0,
    "Driver": "devicemapper",
    "ExecDriver": "native-0.2",
    "MountLabel": "",
    "ProcessLabel": "",
    "AppArmorProfile": "",
    "ExecIDs": null,
    "HostConfig": {
        "Binds": [
            "/var/run/docker.sock:/var/run/docker.sock:rw",
            "/home/mhariri/apollo-docker/logspout:/mnt/routes:rw"
        ],
        "ContainerIDFile": "",
        "LxcConf": null,
        "Memory": 0,
        "MemoryReservation": 0,
        "MemorySwap": 0,
        "KernelMemory": 0,
        "CpuShares": 0,
        "CpuPeriod": 0,
        "CpusetCpus": "",
        "CpusetMems": "",
        "CpuQuota": 0,
        "BlkioWeight": 0,
        "OomKillDisable": false,
        "MemorySwappiness": null,
        "Privileged": false,
        "PortBindings": {},
        "Links": [
            "/apollodocker_logstash_1:/apollodocker_logspout_1/apollodocker_logstash_1",
            "/apollodocker_logstash_1:/apollodocker_logspout_1/logstash",
            "/apollodocker_logstash_1:/apollodocker_logspout_1/logstash_1"
        ],
        "PublishAllPorts": false,
        "Dns": null,
        "DnsOptions": null,
        "DnsSearch": null,
        "ExtraHosts": null,
        "VolumesFrom": [],
        "Devices": null,
        "NetworkMode": "default",
        "IpcMode": "",
        "PidMode": "",
        "UTSMode": "",
        "CapAdd": null,
        "CapDrop": null,
        "GroupAdd": null,
        "RestartPolicy": {
            "Name": "",
            "MaximumRetryCount": 0
        },
        "SecurityOpt": null,
        "ReadonlyRootfs": false,
        "Ulimits": null,
        "LogConfig": {
            "Type": "json-file",
            "Config": {}
        },
        "CgroupParent": "",
        "ConsoleSize": [
            0,
            0
        ],
        "VolumeDriver": ""
    },
    "GraphDriver": {
        "Name": "devicemapper",
        "Data": {
            "DeviceId": "2061",
            "DeviceName": "docker-253:0-9437461-4e602825528dae339b3832827a67b57748376c1719a2e63273a1c65fdc824173",
            "DeviceSize": "107374182400"
        }
    },
    "Mounts": [
        {
            "Source": "/home/mhariri/apollo-docker/logspout",
            "Destination": "/mnt/routes",
            "Mode": "rw",
            "RW": true
        },
        {
            "Source": "/var/run/docker.sock",
            "Destination": "/var/run/docker.sock",
            "Mode": "rw",
            "RW": true
        }
    ],
    "Config": {
        "Hostname": "4e602825528d",
        "Domainname": "",
        "User": "",
        "AttachStdin": false,
        "AttachStdout": false,
        "AttachStderr": false,
        "ExposedPorts": {
            "8000/tcp": {}
        },
        "Tty": false,
        "OpenStdin": false,
        "StdinOnce": false,
        "Env": [
            "DEBUG=true",
            "affinity:container==b812dfce2c898218156670bdb1928b57350329c4b2892b399ec707b6a4e5ad5a",
            "PATH=/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin"
        ],
        "Cmd": null,
        "Image": "gliderlabs/logspout:latest",
        "Volumes": {
            "/mnt/routes": {},
            "/var/run/docker.sock": {}
        },
        "WorkingDir": "",
        "Entrypoint": [
            "/bin/logspout"
        ],
        "OnBuild": null,
        "Labels": {
            "com.docker.compose.config-hash": "8674246891a05e880f49395061838a2e2fb96b91ced0f3530b9198725b215ccd",
            "com.docker.compose.container-number": "1",
            "com.docker.compose.oneoff": "False",
            "com.docker.compose.project": "apollodocker",
            "com.docker.compose.service": "logspout",
            "com.docker.compose.version": "1.5.2"
        }
    },
    "NetworkSettings": {
        "Bridge": "",
        "SandboxID": "",
        "HairpinMode": false,
        "LinkLocalIPv6Address": "",
        "LinkLocalIPv6PrefixLen": 0,
        "Ports": null,
        "SandboxKey": "",
        "SecondaryIPAddresses": null,
        "SecondaryIPv6Addresses": null,
        "EndpointID": "",
        "Gateway": "",
        "GlobalIPv6Address": "",
        "GlobalIPv6PrefixLen": 0,
        "IPAddress": "",
        "IPPrefixLen": 0,
        "IPv6Gateway": "",
        "MacAddress": "",
        "Networks": {
            "bridge": {
                "EndpointID": "",
                "Gateway": "",
                "IPAddress": "",
                "IPPrefixLen": 0,
                "IPv6Gateway": "",
                "GlobalIPv6Address": "",
                "GlobalIPv6PrefixLen": 0,
                "MacAddress": ""
            }
        }
    }
}
]```

I just pushed a commit that should fix this. Looks like the bug was introduced in 23d3c9fee8ff470296226c7ae88332ab1ddde5ff

This seems to be fixed. Please comment if not.


[Feature]: Is there any way to get towhee to support batch processing of images

Example:
import towhee

class Resnet50:
    """
    Say something about the ExampleCalass...

    Args:
        args_0 (`type`):
        ...
    """

    def resnet50_extract_feat(self, img_path):
        feat = towhee.glob(img_path) \
                     .image_decode() \
                     .image_embedding.timm(model_name='resnet50') \
                     .tensor_normalize() \
                     .to_list()
        return feat[0]

Towhee support batch interface to run the data in batch, you can refer to this. Is this interface what you want?
BTW, if you want to processing batch images, you can set the img_path as the paths matching a pathname pattern, such as towhee.glon('all_img/*.jpg', 'all_img/*.png'). And set_parallel interface will help to process image in multi parallel.


large file upload

Hi,
i installed everthing like written and it works great. but if i drop a large file  50mb+ the loadingbar is ready before the upload script is ready.. do u know the issue and is there a solution ?

I am not sure about this.


Use TPM engine for PEM files containing private key wrapped by TPM key

If no one objects I'd like to continue the discussion of these patches.
This would be valuable to many people since this ensures the private key cannot be exfiltrated from the machine with which it's bound. This would also benefit users of other engines because they wouldn't need to specify the engine they want to use since specifying the PEM file used by that engine is enough for openssl to know which engine to use.
@levitte could you clarify why this isn't general enough so we can fix it and merge? I can help if the original author is busy.
Thanks!

I'd also like to address Uri's critique in the following message.
He suggests using URI to refer to the private key wrapped by TPM key instead of using PEM files. There seems to be a misunderstanding of how the private key wrapping by TPM works. The key is not stored in the TPM but in the PEM file and encrypted by a key in the TPM which is prohibitively costly for anyone to access without the TPM. So a URI would just be file path and wouldn't tell us which engine to use.

Let's add a bit of context...
At the time of the discussion, there were other related things going on, such as OSSL_STORE, which is a much more generic interface to retrieve diverse objects (such as private keys) than any ENGINE function to load keys ever was. It is part of OpenSSL since 1.1.1, and integrated into the exact same function that the patch you refer to modifies.
What needs to happen is that the OSSL_STORE file loader needs to be modified to recognise the "TSS KEY BLOB" or "TSS2 PRIVATE KEY" PEM types (see http://david.woodhou.se/draft-woodhouse-cert-best-practice.html#rfc.section.4.4 as well as the TPM2 engine) and act accordingly (whatever that means). Also, if there's still an effort for a TPM URI, a corresponding OSSL_STORE loader would be fantastic.
Also, we're currently working on a new plug-in mechanism, called "providers". You might want to look into that going forward.

This feature is intended to allow existing programs that use OpenSSL to use private keys wrapped by tpm with minimal change on their part.
For example with openssh. With this patch we could use this private key wrapped by tpm in place of any old openssh private key and it would work with no changes to openssh code (if the engine is installed).
Can we do the same thing with ossl_store?

Can we do the same thing with ossl_store?

Yes.  Or at least, I assume so.  OSSL_STORE is already integrated into the very function you want to patch, so you have the code path in place already.

This is something that is being already worked on by third party TPM provider. https://github.com/tpm2-software/tpm2-openssl


Unit tests for Row and Column

And fix unit tests after merging two implicitly conflicting pull requests.

[ ] CHANGELOG.md's "Unreleased" section has been updated, if applicable.

These are unit tests from the Compose Foundation adapted for Mosaic, I think they work correctly.
I will fix other issues in the next pull requests.

If they use the math approach for asserting the results then we can keep doing that so they mostly in sync.


Added more annotations for attributes

Fix for: https://github.com/xspec/xspec/issues/113
Add more annotations to defined attributes.

Thanks for this pull request, it's very useful to add more documentation so that it can be visible from Oxygen, it will help developers to use these attributes.
I approve the pull request in principle and suggest further additions following your question on attributes href and select.
I checked the documentation and was able to find references for the attributes href and select so I suggest to add the following:
element import { common-attributes,
    ## The document URI (location) of the imported document.
    attribute href { xs:anyURI } }

node-selection =
    ## The document URI (location) of the imported document.
    attribute href { xs:anyURI }?,
    ## The XPath expression to access the selected node.
    attribute select { xpath }?,
    any-content

value-selection =
    ## The XPath expression to access the selected node.
    attribute select { xpath }

In order to conform your changes with the conventions used in this file, would it be possible to add a space after ## in the documentation you added? For example:
    ## The qualified name of the template which should get called.

Or does this display incorrectly on Oxygen? Sorry for being so pedantic!
If you agree, would you be able to add these changes to your pull request?

Thanks for adding documentation for attributes href and select. As described above, there are two lines that require an additional space after ## in order to make it consistent with the rest of the file. If you add this to your pull request, I'll merge it into the master branch. Sorry again for being so pedantic... ;-)

Thanks for changes, I'll merge the pull request into master.


Refactored GsonCompatibilityMode::updateClassDescriptor()

Refactored GsonCompatibilityMode::updateClassDescriptor() by adding two helper functions, setupBindings() and iterateStrategies(). This reduced the CCN from 16 to 2.
This should resolve #118.

Codecov Report

Merging #131 into lab3_refactoring will increase coverage by 0.36%.
The diff coverage is 75.75%.

@@                 Coverage Diff                 @@
##           lab3_refactoring    #131      +/-   ##
===================================================
+ Coverage             69.84%   70.2%   +0.36%
===================================================
  Files                   107     107
  Lines                  7388    7481      +93
  Branches               1397    1446      +49
===================================================
+ Hits                   5160    5252      +92
- Misses                 1802    1803       +1
  Partials                426     426

Impacted Files
Coverage Δ

...java/com/jsoniter/extra/GsonCompatibilityMode.java
82.86% <75.75%> (+0.18%)
:arrow_up:

src/main/java/com/jsoniter/spi/Config.java
86.09% <0%> (+1.7%)
:arrow_up:

...c/main/java/com/jsoniter/IterImplForStreaming.java
80.04% <0%> (+2.53%)
:arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update b93f87a...0d295f6. Read the comment docs.


[CP] Hot fix package:unified_analytics version to 5.8.0+1

Issue Link
https://github.com/flutter/flutter/issues/143792
Target
stable
PR Link
https://github.com/flutter/flutter/pull/143885
Changelog Description
Bumps the version for analytics to a hotfixed version that fixes flakey bug around parsing session json file
Impacted Users
All flutter users
Impact Description
This error would periodically crash the flutter tool which in turn would crash running apps.
Workaround
No workaround
Risk
medium
Test Coverage
yes
Validation Steps
No response

Released in 3.19.2


Copy GitHub Permalink is missing line info

Creating a multiline GitHub Permalink no longer includes the line information.
The Copy GitHub Permalink as Markdown includes the lines but also creates a surrounding markdown link.
Some time ago this also worked for normal Permalinks.

Using The command from the actions menu CTRL+SHIFT+P works as expected.

Extension version: v0.29.2
VSCode Version: 1.59.1
OS: Windows 10 10.0.19043

Steps to Reproduce:

Mark multiple lines in a github repo file
Use context menu to copy GitHub Permalink

Copy Remote File Url works too.

Thanks for fast fix. ❤


Missing text for talent

Hi, the elementlist talent "Shaper of Desolation" is missing text.
Keep up the good work :) love your application.

This was missing in the data GGG provided. You should be able to fix it by redownloading the skill tree data (Tools ->Program Assets -> Redownload Skill Tree Assets).

Ah okay thanks :)


Full Text Searching use IN BOOLEAN MODE

Full Text Searching
I want to use IN BOOLEAN MODE or IN NATURAL LANGUAGE MODE
any solutions? or change code?

Supported here 9c60c6c82


Mention only works for 'here'

The code formats the string from the mention field into <!string>.  This only works for here.  If you wish to mention a specific user or user group, the message to slack does not function properly, and returns <!jon>.  Maybe add a check for here and format correctly, otherwise send in the @ with a user or group name?  Thanks! This action is great!

Any updates?

Hi @nomeelnoj @antonsnl88, since there is no updates on this report, would you like to check on this alternative Slackify?


Cannot open rJava library using the interactive interpreter

I am using the vscode R editor on a server (via ssh) and I am trying to load the rJava library on a conda environment. Whenever I try to load rJava I get the following error:
Error: package or namespace load failed for ‘rJava’:
 .onLoad failed in loadNamespace() for 'rJava', details:
  call: dyn.load(file, DLLpath = DLLpath, ...)
  error: unable to load shared object '/media/...miniconda3/envs/r4/lib/R/library/rJava/libs/rJava.so':
  libjvm.so: cannot open shared object file: No such file or directory

I have checked that the object '/media/...miniconda3/envs/r4/lib/R/library/rJava/libs/rJava.so' does exist.
Looking online I have found some solutions regarding some environment variables that need to be changed. The usual recommendation is to run on the terminal:
R CMD javareconf -e
LD_LIBRARY_PATH=$LD_LIBRARY_PATH:$JAVA_LD_LIBRARY_PATH

If I run this in the command line directly I can afterwards successfully run Rscript -e "library(rJava)" which uses the terminal environment to load the rJava library "in place".
Looking deeper into it I see that indeed several environment variables have been changed (in the terminal environment). I have tried to change them in the Rinteractive terminal using Sys.setenv() but the error persists.
I'm not sure if the error is on the vscode-R side, but it seems to me that it has something to do with the setting of the java-related environment variables when R interactive is initiated. Any help or suggestions would be highly appreciated.
Environment:

OS: Linux
VSCode Version: 1.81.1
R Version: 4.2.3
vscode-R version: 2.8.1

Additional context
Running R on a conda environment.

Managed to solve it by running R CMD javareconf (without the -e flag)...
In the end it had nothing to do with vscode but leaving it here in case anyone needs it.


How to use custom icons?

I want to test some custom icons for my personal projects. How can I choose my own icons, as I read in the extension description that it supports icon customization.

Please, check the wiki.

I followed the steps given here.
I navigated to C:\Users\Username\AppData\Roaming\Code\User\vsicons-custom-icons, created a new folder named vsicons-custom-icons and placed my icons in that folder.
Please guide me further on how to apply these icons to my files as I didn't understand the further steps given in the documentation.

@deepindersinghbti you don't have to create a folder inside the C:\Users\Username\AppData\Roaming\Code\User\vsicons-custom-icons.
Have you put the icons at the root of the previous path? Do they follow vscode-icons naming convention as stated in the wiki? Have you declared the icons in your vscode settings?

I guess you'll have visited this page of the docs explaining all the process of creating custom icons.
Take a look at this one to learn how to declare your icon customization

I added  the following code to my settings.json file:
  "vsicons.associations.files": [
    {
      "icon": "js",
      "extensions": [".js"],
      "format": ".svg"
    }
  ]

And also added this setting:
  "vsicons.customIconFolderPath": "C:\\Users\\Username\\AppData\\Roaming\\Code\\User\\vsicons-custom-icons",
But it's not working. Please help!

Remove the dots from the extensions and format.

I tried, but it's still not working.


Update TITLE_1_CH_1.md

Editing art XIV
***errata encorregi on comma 3:
"Who is....."

@Mathew-Iron What do you think about this one? I think you've added the key points into the main body outside of this branch, right? @Darjuz96 could you take a look into main to see if we're missing any of your key points?

@Darjuz96 I've added your key points to the main branch and reworded them, so go and see if that works for you :).


I have this problem when switching branches

Description of the bug
I installed it via Homebrew. I have problems switching between branches.

Expected behavior
No response
Actual behavior
No response
Version of rustc
rustc 1.80.0 (051478957 2024-07-21)
Automated bug report
No response
Version of git-branchless
git-branchless-opts 0.9.0
Version of git
git version 2.46.0

Closing as duplicate of https://github.com/arxanas/git-branchless/issues/1388; follow there for updates.


feat: add refresh method

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

0.8.0
GitHub release

Your semantic-release bot :package::rocket:


Re-enable Python 2.6 tests

Tests for Python 2.6 were removed as I didn't have time to find out what was failing in a single test after the last merge:
https://github.com/weso/CWR-DataApi/pull/169
As far as I can tell, it is a problem related to the optional character set field in the transmission header. If it comes empty then the value is rejected.

Python 2.6 is obsolete. These tests won't be re-enabled.


Prompt to update DEVELOPER_DIR for macOS toolchain selection

This updates the toolchain selection workflow to also set the DEVELOPER_DIR environment variable on macOS. This will be done automatically if the user selected an Xcode toolchain. However, they will be prompted to select a developer directory if they select a public Swift toolchain and there is more than one Xcode application installed on their system.
Linux and Windows toolchain selection is unaffected by this change.
Issue: #911

We need to make it clear why you need the DEVELOPER_DIR, as it is not clear to me what the swift downloaded toolchains use from Xcode. @dschaefer2 perhaps you can answer that

We need to make it clear why you need the DEVELOPER_DIR, as it is not clear to me what the swift downloaded toolchains use from Xcode. @dschaefer2 perhaps you can answer that

The macOS SDK? Lots of good stuff in there :).

@swift-server-bot test this please


Export coreml in activity classifier should not return an object

In [10]: model.export_coreml('myModel.mlmodel')
Out[10]: <turicreate._cython.cy_model.UnityModel at 0x151ed1450>

Same thing for OD hmmmm.
Should we fix them together?

Fixed in #2603


ufbx material colors are not converted to sRGB

Tested versions

Reproducible in 4.3 (using ufbx)
Not reproducible in 4.2.x or older

System information
Godot v4.3.dev (655e93d58) - Windows 10.0.19045 - Vulkan (Forward+) - dedicated NVIDIA GeForce RTX 3090 (NVIDIA; <IP_ADDRESS>13) - Intel(R) Core(TM) i9-9900K CPU @ 3.60GHz (16 Threads)
Issue description
Material colors on imported fbx seem to not be converted to srgb.
For example, the RightEar material should be color #e711a5 but is instead imported as #cc0160
Note: I am not talking about vertex colors, which should be tested separately. In this issue, all of the colors on the imported fbx come strictly from materials.
It is important to fix before the release of 4.3, because otherwise we will be forced to break compat in terms of material colors in the future.

Left side: ufbx. Right side: FBX2glTF (matches Blender)
Steps to reproduce

Open the project. fbx_vs_ufbx_test.tscn contains two copies of the identical .fbx file: one is imported with ufbx, and the other with legacy FBX2glTF
The FBX2glTF version has colors which match other programs (blender, unity)
The ufbx version near origin has the incorrect colors, saturated and darker

Minimal reproduction project (MRP)
fbx_material_sorting.zip

The code to apply the material colors is here. https://github.com/godotengine/godot/blob/b2f425fe680d1ed5d5b5fa9ae289ae93fd294607/modules/fbx/fbx_document.cpp#L89C14-L105


Validator compute state root requires full withdrawal list

To compute the state root for Capella blind block, a full withdrawal list will be required in state transaction. Currently, a blind block contains a header that only contains a withdrawal root, not a list. That information is not sufficient to construct state root. One solution is client can utilize get_exepcted_withdrawal from the parent state and use the expected withdrawal and call process_withdrawals. I think we might want to clarify this in the builder specs

You can just check against the withdrawals root of the expected withdrawals:
https://github.com/ethereum/consensus-specs/pull/3092

are we talking about CLs doing what validations they can against a given SignedBuilderBid or are we talking about directions for builders of blocks (and by extension relay validation)?

we already do this in lodestar:
https://github.com/ChainSafe/lodestar/blob/unstable/packages/state-transition/src/block/processWithdrawals.ts#L19:L25

ok I think I'm following -- this is more of an implementation concern than a spec-level issue.
it sounds like clients are using the same pathways to build blocks regardless of the payload being "blinded" or not, and in that case the typical state transition (to calc state roots) would fail bc as written it wants the full withdrawals list.
this code can be modified to just check withdrawals root equivalence which is what it sounds like everyone is doing...
I'll close this issue as it sounds like there is a way forward and I'm not sure this type of thing should live in the builder-specs although my disposition here could change as I separately want to understand better the validations clients do and there could be value in standardizing those... if anything I'll follow up separately and open a new issue if warranted


MudDataGrid/MudTable: Add focus styling for the "rows per page" selector (#9718)

Description
Resolves #9718
Added basic focus styling for the "rows per page" selector on grids and tables.
How Has This Been Tested?
Visually.
Type of Changes

[ ] Bug fix (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 (fix or improvement to the website or code docs)

https://github.com/user-attachments/assets/340e7d28-a124-41fd-b15f-4a95b09e3895
Checklist

[x] The PR is submitted to the correct branch (dev).
[x] My code follows the code style of this project.
[x] I've added relevant tests.

@danielchalmers Fully agree with you! My goal was to have style that fits just the datagrid/table selector just to keep things simple and have the styling match with the button styling, I'm fine with just applying default focus styling for MudSelects instead with Variant => Text and Underline => false.
What do you think?

(I'll need to check for regressions, this implementation adds 2px around the MudSelect input)

I like the outline on focus! Would like to see hover effect: as well

Do you agree it should be mandatory, or the default, for selects to have some kind of effect on focus/hover for the sake of accessibility regardless of variant or underline?

I like the outline on focus! Would like to see hover effect: as well

Do you agree it should be mandatory, or the default, for selects to have some kind of effect on focus/hover for the sake of accessibility regardless of variant or underline?

I think every component that is interactive should have a focus and hover styling (from the few pages I have open, it seems like most components have different focus and hover styles, usually outlines for focus, background-color change for hover). For the Underline variant I think the underline itself already does a good job conveying focus and hover. Our variants already behave like the MUI ones:
https://github.com/user-attachments/assets/2716ed70-f487-4ea2-8665-3c3a541acfd7
The default MUI select implementation has the same issue when using the text & not-underlined variant (no focus style, no hover style): https://stackblitz.com/edit/react-zybax8?file=Demo.tsx,index.tsx

Which do you think makes more sense for us:

outline with default hover colour for hover
background-color with default hover colour (default border radius or squared?)

Ah, thanks for that extra context.
I like your original one the best because it acts like the buttons by not drawing attention to itself by default, but gives feedback on focus and hover, I just don't like diverging the style from other Selects. Ideally all variants would have a focus/hover style like that. I'm curious as to why MUI doesn't do it either. Hope I'm understanding what you're saying

Ah, thanks for that extra context.
I like your original one the best because it acts like the buttons by not drawing attention to itself by default, but gives feedback on focus and hover, I just don't like diverging the style from other Selects. Ideally all variants would have a focus/hover style like that. I'm curious as to why MUI doesn't do it either. Hope I'm understanding what you're saying

Yeah they all have different sorts of focus/hover styling just to match their general look (or are missing / forgotten), we can't necessarily use the same the same style for each of the variants as it would look worse / too cluttered / etc.
Here's an overview of the different variants with/without underline and how they behave:
https://github.com/user-attachments/assets/9f21ab3d-8817-431a-8ae1-0176db4265f7
And here's the datagrid behaviour:
https://github.com/user-attachments/assets/d9730a99-c069-4410-a9c4-780ee3af8619


Foundry v9 compatibility

Hey there,
still using your system and really enjoying it. Bh2e is just amazing.
Now there is a new major version of foundry and I am kinda reluctant to update it, in case this module will not work anymore.
Is there any chance you can have a look at if its compatible?

Hi,
It's on the agenda and I will try and get a look at it towards the end of next week. I've already installed v9 and have given it a quick try to see if I could spot anything obvious (I didn't). I guess I'll have to consider putting together some sort of functional regression document listing things that needs touched upon for each release.
Peter

Great, let me know when you need any help. I will open another feature suggestion to not mix up the issues.

v0.3.1 released so marking this one as closed. If you find any issues in the release please let me know.


Update AI-102

Module: 00
Lab/Demo: 00
Fixes # .
Changes proposed in this pull request:

Graeme,
Here is a screenshot of the resource group page with +Create on it.  I would have to recreate the lab to grab a screenshot of the Connect to Channels.
Jon

From: Graeme Malcolm @.>
Sent: Tuesday, March 23, 2021 2:33 PM
To: MicrosoftLearning/AI-102-AIEngineer @.>
Cc: mctjon @.>; Author @.>
Subject: Re: [MicrosoftLearning/AI-102-AIEngineer] Update AI-102 (#54)
@GraemeMalcolm requested changes on this pull request.
Thanks for submitting!
The typo fixes in the Computer Vision and Form Recognizer labs are fine, but the changes to the Azure Portal UI don't seem to be right - the button to add a resource group when in the Resource Groups blade is +Add, not +Create; and when viewing the Test Bot in Portal blade, the link to the channels page (under Settings in the pane on the left) is "Channels", not "Connect to Channels".
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/MicrosoftLearning/AI-102-AIEngineer/pull/54#pullrequestreview-618940529, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ASYO2MO7XWGTKLRZMKDGK3LTFDNI3ANCNFSM4ZVWI3RQ.

Can't see a screenshot attachment in your message, but are you sure you're not looking at the HOME page rather than the RESOURCE GROUPS page (which is what the instructions assume)?
Screenshots for both UI elements attached

Graeme,
[cid:764f66b6-062f-42d6-8d8b-d29ea2b48ea6]

From: Graeme Malcolm @.>
Sent: Tuesday, March 23, 2021 3:05 PM
To: MicrosoftLearning/AI-102-AIEngineer @.>
Cc: mctjon @.>; Author @.>
Subject: Re: [MicrosoftLearning/AI-102-AIEngineer] Update AI-102 (#54)
Can't see a screenshot attachment in your message, but are you sure you're not looking at the HOME page rather than the RESOURCE GROUPS page (which is what the instructions assume)?
Screenshots for both UI elements attached
[add-rg]https://user-images.githubusercontent.com/11771134/112203592-f338a900-8bcf-11eb-9732-8225f85f23dd.png
[bot-channels]https://user-images.githubusercontent.com/11771134/112203593-f3d13f80-8bcf-11eb-8403-e5d768e29527.png
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/MicrosoftLearning/AI-102-AIEngineer/pull/54#issuecomment-805157911, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ASYO2MM456WSKYE2WT6BIY3TFDQ6PANCNFSM4ZVWI3RQ.

@mctjon
Posting by email doesn't appear to be working (at least, I can't see your comments fully). Any chance you could comment directly in  the pull request (https://github.com/MicrosoftLearning/AI-102-AIEngineer/pull/54)?
Thanks!

So it seems the Azure portal UI is inconsistent - possibly by region.
I'm rewording the instructions to make them more generic.


Model pull error - I/O timeout

What is the issue?
I tried to download llama 2 7b chat 8-bit quantized model, but I am getting the following error

Any suggestion how to resolve this?
OS
Windows
GPU
Nvidia
CPU
Intel
Ollama version
0.1.44

Closing for https://github.com/ollama/ollama/issues/3816


Is it possible to refer one task in other  task

like in java we call one method in other method, is it possible to do same in workflow for tasks

@veveeranishwar can you provide an example?  You can refer to the task's input or output parameters, not sure what you mean by reffering the tasks.

For an example i have 2 task
{
"name": "task1",
"taskReferenceName": "task1",
// this task has decision and sub workflow as well
}
on certain condition I need to execute the same logic which task1 is having  in another task say task 2. I don't want to repeat the same logic in task2 also, instead can I point task2 to execute task1 , is it possible.
here is my task2
{
"name": "task2",
"taskReferenceName": "task2",
//here I have to execute same logic which is in task1, is it possible to execute task1 here  or do I have to repeat same logic which task1 having
}

@veveeranishwar tasks are implemented and executed by workers which is external to Conductor server.  You should be able to organize the code such that both task1 and task2 either shares the same implementation or has other techniques such as common helpers or inheritance used to avoid duplication of logic.

how to do  inheritance  in workflows , can you help me on this

@veveeranishwar Please follow the documentation here to learn about wiring tasks together in your workflow.


Make annotation element types intrinsicially non-nullable.

This is already covered in the
Javadoc,
but the spec didn't cover it until now.

(as noticed in https://github.com/jspecify/jspecify/pull/644#issuecomment-2359409320)


Update config.rst

The current suggested Metadata naming_convention dictionary runs into issues on migrating Boolean columns in SQLite databases as described here (https://github.com/sqlalchemy/sqlalchemy/issues/3345) and here (https://github.com/sqlalchemy/sqlalchemy/issues/4784). If for some reason the original ck value is necessary, it might be useful to have an additional note indicating this as an alternative. Thanks for considering!

In SQLAlchemy 1.4, non-native Boolean and Enum no longer generate constraints by default since it was causing more issues than it was solving. Additionally, both types already took an optional name to be used for the constraint if necessary.

https://docs.sqlalchemy.org/en/14/changelog/migration_14.html#enum-and-boolean-datatypes-no-longer-default-to-create-constraint
https://docs.sqlalchemy.org/en/14/core/type_basics.html#sqlalchemy.types.Boolean

Based on the discussions you linked, it seems more correct to leave this as-is. Projects should use the features mentioned above.


Remove add-celltypes.nf workflow

RIP, former cell typing workflow.

Closed by https://github.com/AlexsLemonade/scpca-nf/pull/577


2017-10-06-WeChat PC Impactor

微信电脑版 HOOK 的一些研究 以及代码分享！
感谢精易论坛坛友(xdssoft,恨不能遗忘,gh0st少主)分享的开源代码参考！

And could you please select a good image for your blog? It would be fantastic if a blog has a beautiful face image.

Just one more thing:   rename 2017-10-06-WeChat-PC-Impactor to 2017-10-06-wechat-pc-impactor
Pay  attention to lowcase, then I will merge this.
Thanks!


Add Jeffrey Fabian to Contributors list

Thanks for making this transition project available!!

I'm quite elated about your pull request. I wanna evolve this project to addresses various problems faced by first-time contributors. I'd love to learn about your journey in open source community, the problems, pain points you had etc.
Could you explain how you felt when you went through the tutorial, made a pull request and learned that I merged it?
We’ve recently added Facebook share and tweet feature to our web app. Could you please go to https://roshanjossey.github.io/first-contributions/#social-share and share your first contribution to open source?
Also, check out projects with easy issues while you’re there.


Added more context to boostrap the configuration

Ha, long time no see @saturnism! Thanks a lot for this improvement 👍

Committed the changes manually and gave you the attribution.


run multiqc as last step

multiqc should run after alignment as it will pick up more metrics for the report

fixed in v2.0.0


ColorDef validation and ColorByName fix

ColorDef.fromString produces black if the input cannot be interpreted as a color.
ColorDef.fromTbgr takes a number and converts it to a uint32 via Uint32Array; fractional parts are truncated, sign bits are flipped, etc.
Added ColorDef.isValidColor to return whether the input (number | string) is a valid color representation, and ColorDef.tryComputeTbgrFromString which - unlike computeTbgrFromString - returns undefined instead of black if the string cannot be interpreted as a color.
In the process of testing this, I discovered that ColorDef.create("aqua") produces black - it fails to find ColorByName.aqua. This is because ColorByName is an enum, ColorByName.cyan has the same integer value as aqua, and enums are wack in TypeScript. Object.entries iterates the values, which omits duplicate values.
Converted ColorByName to an object (via namespace) to eliminate this problem without breaking code that currently does ColorByName.whatever. Code that relies on ColorByName being an enum may break, but enums are wack, so no one should be relying upon that.

Force merging because macOS is hung on ECSql tests again.

@UgniusZidonis in what way is this bug manifesting on 2.19.x?

@pmconne, as I had explained in the Teams channel, there is no way of checking whether an argument for creating a color is valid or not, so with invalid arguments you get a black color. We need that isValidColor function.


Bravo - default controller in a subfolder

This will be a controller found by convention
Make the target link /bravo
We want this handled by a controller inside controllers/bravo. In this case, we want that to be the default controller, and its index method, i.e. controllers/bravo/Welcome::index()
The body of that method should invoke the inherited show method, with the key "2"

Close issue request


Build dockerfiles not updating properly?

As far as I can see the changes in https://github.com/adoptium/infrastructure/pull/2240 have not yet taken effect in the dockerfiles for some reason.
Looking at a recent github test, it looks like https://github.com/adoptium/infrastructure/pull/2223/checks?check_run_id=2849714322 (as an example has the following in it:
 Proceed (y/n)? ERROR: Exception:
Traceback (most recent call last):
  File "/usr/local/lib/python3.6/site-packages/pip/_internal/cli/base_command.py", line 180, in _main
    status = self.run(options, args)
  File "/usr/local/lib/python3.6/site-packages/pip/_internal/commands/uninstall.py", line 86, in run
    auto_confirm=options.yes, verbose=self.verbosity > 0,
  File "/usr/local/lib/python3.6/site-packages/pip/_internal/req/req_install.py", line 672, in uninstall
    uninstalled_pathset.remove(auto_confirm, verbose)
  File "/usr/local/lib/python3.6/site-packages/pip/_internal/req/req_uninstall.py", line 380, in remove
    if auto_confirm or self._allowed_to_proceed(verbose):
  File "/usr/local/lib/python3.6/site-packages/pip/_internal/req/req_uninstall.py", line 423, in _allowed_to_proceed
    return ask('Proceed (y/n)? ', ('y', 'n')) == 'y'
  File "/usr/local/lib/python3.6/site-packages/pip/_internal/utils/misc.py", line 203, in ask
    response = input(message)
EOFError: EOF when reading a line
Loaded plugins: fastestmirror, ovl
Cleaning repos: base centos-sclo-rh centos-sclo-sclo epel extras
              : testing-devtools-2-centos-6 updates

which suggests that it may not have run through to competion, or failed some way. It certainly doesn't appear to have run any of the adoptopenjdk_install invocations. But it seems to be still showing as a green 'pass" which doesn't seem right (if this is expected, we need to document it. If it's not right it needs to properly fail the check in this sutation)

Appears ok - Linux/x64 and Linux/aarch64 are using the boot JDK from the correct location. No idea what was fixed to do this but doesn't seem to have been tracked through this issue


Are you aware of gomail?

Hi!
This looks like a great library. I needed to parse a custom "raw" email, and it worked like a charm. So I would really thank you for it.
However as I can see, it is abandoned. 10 commits and 7 stars for a year old repository is an quite small. On the other hand, there is gomail, the project is well maintained and there is some community built around it. The project is great for building emails. But it lacks the functionality of parsing it.
Are you aware of gomail? If yes, did you think about merging your code into gomail?

I benefit from the package to parse raw email too. Thanks for the work so far!
gomail is kind of messy IMO. It includes both dial/send logic, and the mail construction logic.
I don't find the way to parse multipart/alternative body in the package. I could contribute to make it happen if this repo is still active.
@mohamedattahri @JaTochNietDan

Hi @yichengq, we are using this library for quite a lot of stuff on our own fork so our fork is probably a bit more maintained at this stage.


Is this a fuzzy Neural Network?

Is this a fuzzy neural network implementation using tensorflow???
Working on a similar project so will be very helpful to know.

yes

Thanks.. Just needed a suggestion. Can we apply these fuzzy rules to cnns and rnns too?

https://www.researchgate.net/publication/3855330_Fuzzy_Reasoning_for_the_design_of_CNN-based_image_processing_systems


Add image as a string to publisheddata model

Issue Name
Summary
Steps to Reproduce
Current Behaviour
Expected Behaviour
Extra Details
Here you should include details about the system (if it is unique) and possible information about a fix (feel free to link to code where relevant). Screenshots/GIFs are also fine here.

Image is stored in PublishedData.thumbnail as a base string
https://github.com/SciCatProject/catamel/blob/develop/common/models/published-data.json


[Observation] Refine the macro implementation for Observable

This alters the macro emission for @Observable to:
better handle initialization
produce sensible diagnostics when inferred types are used for properties
produce sensible diagnostics when actors are marked as @Observable
better names for member variables and types to avoid conflicting with users of @Observable

@swift-ci please test

@swift-ci please test


How to use the clear button for a multiline edit text?(Clearable edittext)

Hi..
First of all thanks for this(Clearable Edittext). I have used this in my app and i want to use this for a multiline editext , and in that i want to place the drawable at the end? For a single line Edittext it works fine.

You can specify the number of lines like for a usual TextView. Please use http://stackoverflow.com/questions/tagged/droidparts for questions.


linear_models_issue1

Hi @jstac , adding a file we need for re-linking to this respiratory.

So, this lecture was written a long time ago.  What we should do to update it is to add that code in as a cell and generate the figure, rather than including the figure via a .. figure:: directive.
In other words, we should generate the figure when the lecture code is run, rather than keeping a png file in _static.  Do you know what I mean?  Can you do it?  There's actually a few figures generated from this code at different sample sizes, so you need to do it in a way that minimizes repetition.  I'm happy to discuss.


fixes benchmark CE

mvn clean test -Pbenchmark
[WARNING] COMPILATION WARNING :
[INFO] -------------------------------------------------------------
[WARNING] /home/ruslan/proj/javaslang/javaslang-benchmark/src/test/java/javaslang/benchmark/BenchmarkPerformanceReporter.java:[393,13] found raw type: org.openjdk.jmh.results.Result
  missing type arguments for generic class org.openjdk.jmh.results.Result<T>
[INFO] 1 warning
[INFO] -------------------------------------------------------------
[INFO] -------------------------------------------------------------
[ERROR] COMPILATION ERROR :
[INFO] -------------------------------------------------------------
[ERROR] /home/ruslan/proj/javaslang/javaslang-benchmark/src/test/java/javaslang/benchmark/BenchmarkPerformanceReporter.java: warnings found and -Werror specified
[INFO] 1 error

My environment:
$ mvn -version
Apache Maven 3.3.9
Maven home: /usr/share/maven
Java version: 1.8.0_91, vendor: Oracle Corporation
Java home: /usr/lib/jvm/java-8-oracle/jre
Default locale: en_US, platform encoding: UTF-8
OS name: "linux", version: "4.4.0-22-generic", arch: "amd64", family: "unix"

/cc @paplorinc @eduardmanas

[Current coverage][cc-pull] is 95.54%

Merging [#1349][cc-pull] into [master][cc-base-branch] will not change coverage

2 files (not in diff) in .../main/java/javaslang were modified. more

Misses +1
Partials -1

@@             master      #1349   diff @@
==========================================
  Files            86         86
  Lines          9988       9988
  Methods           0          0
  Messages          0          0
  Branches       1802       1802
==========================================
  Hits           9543       9543
- Misses          318        319     +1
+ Partials        127        126     -1

Powered by Codecov. Last updated by [930163f...b1cd544][cc-compare]
[cc-base-branch]: https://codecov.io/gh/javaslang/javaslang/branch/master?src=pr
[cc-compare]: https://codecov.io/gh/javaslang/javaslang/compare/930163f11b3079449eb5dee1314363b6c3b7710a...b1cd544e24c3c2420df7678700a1066e7bb01d3d
[cc-pull]: https://codecov.io/gh/javaslang/javaslang/pull/1349?src=pr

@ruslansennov Good catch!!!
I remember having fixed that issue, so I have spent a little while scratching my head how on earth is not in master!
As it turns out, I fixed it in the branch of the other pull request https://github.com/javaslang/javaslang/pull/1317

It seems that Daniel's phrase about "the PRs need to be as small as possible" is very actual :smile:

Hi all, let's leave the current PRs as-is. I will finish my review soon. After that a PR should be a small, focused unit of work.


[WFLY-15739] Set java.security.manager system property value to "allow" on JDK18+

https://issues.redhat.com/browse/WFLY-15739
This PR replaces https://github.com/wildfly/wildfly/pull/14938 as the property is activated iff JDK18 and above is detected.

@ropalka is this one ready to review and merge?

Yes it is @darranl .


chore: update node to v18

I believe all of these changes have been integrated into the recently merged https://github.com/graphprotocol/indexer/pull/829.  Closing


3 - Missing RequestServerVersion SOAP header

What steps will reproduce the problem?

FindFolder request with DistinguishedFolderId on publicfoldersroot and Traversal on Shallow.

Expected output: succesful response
Got soap error: The request is valid but does not specify the correct server version in the RequestServerVersion SOAP header. Ensure that the RequestServerVersion SOAP header is set with the correct RequestServerVersionValue.
Quick fix: to NTMLSOAPClient_Exchange::__construct(), I added this:
$soapHeader = new SoapHeader(
    'http://schemas.microsoft.com/exchange/services/2006/types',
    'RequestServerVersion Version="Exchange2007_SP1"'
);
$this->__setSoapHeaders($soapHeader);

Obviously, the Exchange2007_SP1 part should be configurable.
Originally posted by kristof.coomans on Goolge Code: http://code.google.com/p/php-ews/issues/detail?id=3

I find it odd that I haven't run into this issue before. I will verify that this is all that is required and take care of it this weekend.
Originally posted by jamesiarmes on Goolge Code: http://code.google.com/p/php-ews/issues/detail?id=3#c1

Ah, this fix may work with the ExchangeImpersonation, however i'm unsure, how to impliment this on  NTLMSoapClient_Exchange , any thoughts
Originally posted by goresci on Goolge Code: http://code.google.com/p/php-ews/issues/detail?id=3#c1

Hi, I've attached a patch that fixes this issue (we were affected as well.)
Originally posted by jon.kloske.uq on Goolge Code: http://code.google.com/p/php-ews/issues/detail?id=3#c3
File attached: php-ews_issue3_fix.diff

forgot to mention, to use the new code you need to specify the new optional exchange server version parameter to the ExchangewWebServices constructor, like so:
$ews = new ExchangeWebServices('server', 'username', 'password', EWSType_ExchangeVersionType::Exchange2007_SP1);

Originally posted by jon.kloske.uq on Goolge Code: http://code.google.com/p/php-ews/issues/detail?id=3#c4

This has been merged into the master branch. I made some changes to your patch, the biggest change being that I added the constants directly to the ExchangeWebServices class rather than adding a new class. I have also added you to the contributors file, thank you for your contribution.
Originally posted by jamesiarmes on Google Code: http://code.google.com/p/php-ews/issues/detail?id=3#c5


How should we handle CSS properties with custom values?

So far, all single purpose classes that represent CSS properties have been housed inside the 'objects' folder. All of these classes represent CSS properties with fixed values. These classes are completely devoid of any style and are therefore reusable across every project ever.
However, there are also lots of CSS properties that allow for custom values, e.g. margin, padding, border-radius, font-size etc. It is these CSS properties that define the style for our project.
Here are some options:

We could bundle these custom CSS properties into 'objects' along with the other CSS properties.
We could move them to the 'skin' folder?
We could leave these CSS properties inside 'objects', but handle the custom values via global variables elsewhere?

After thinking about this, I actually decided to remove all CS properties which accept custom values from framework. These properties are what gives each project its own unique style, so therefore, you don't want us prescribing these property values for you.
So, all CSS properties which accept custom values should be placed inside the /theme folder. You should do this manually.


issue with pausing the player

An empty player with no playing audio can be unpaused and will play nothing and player.playing will be true
This happens when a track is added to the  queue while the player is not playing. When you resume the player gets stuck at that position since there is no way to find out from the player if it is playing audio or not... Player.playing returns true and player.queue.current returns the track that was added while the player was not playing.

Disable pause when the queue is empty

But what if there is nothing playing but because of the new added track the queue is not empty so you can still unpause... But there is no way to find out if the track has started playing after unpausing from the player itself...

Disable pause when the queue is empty

This wont work if they call Player#pause() after.
I'll make it not let you pause or resume when the queue is empty because there's no point when it's empty.

But what if there is nothing playing but because of the new added track the queue is not empty so you can still unpause... But there is no way to find out if the track has started playing after unpausing from the player itself...

Player#playing returns a boolean value whether the player is pause or not


Base layer is saved twice when switching to a plain base layer and UI shows error message

Context
I was working on adding support for Google Maps Basemaps and I stumbled upon this problem when doing some QA.
Steps to Reproduce

Open a map in BUILDER.
Click on the basemap layer.
Select one of the CARTO basemaps.
Change the basemap to a plain color by clicking on COLOR and selecting a color.

Current Result

Base layer is being saved twice (here and here). The first request is being cancelled, and causes the UI to show an error message temporarily:

Expected result

Base layer should only be changed once (only one network request).
Message should change from "Saving new basemap" to "Basemap saved successfully."

Browser and version
All.

Already fixed! with https://github.com/CartoDB/cartodb/issues/11934

👏 👏 👏


[DO NOT MERGE] More macOS titlebar changes

Makes the following changes:

Do not actually remove the title from the NSWindow, instead hide the title, traffic lights, and set full size content view
Add WindowMac#setTitleVisible(boolean)

Seeing an issue with setTitlebarVisible currently, and waiting on a way to hide traffic lights from #183

I will probably postpone #183 to way later since it’s not that high of the priority. If you can just hide traffic lights really quick without changing their position, just go ahead, I don’t want to be a blocker

I added a temporary way to toggle the traffic lights. I am unsure what is going on with the drawing rect. When setting full size content view the titlebar goes away, but the content is not shifted up to fill the gap until you resize the window manually. Then it seems to work from then on out.
eg. CMD-T and note the gap at the top, then resize and it is fixed. The issue appears on main as well, but it's less obvious (It happens in the Transparent setting).

I suggest we focus on creating new windows, not switching styles on existing one. I think it’s a more realistic use-case (you usually know what type of window you want, instead of deciding on the fly).
For that use-case this PR seems to work good, so let’s merge it.


[SPARK-36737][BUILD][CORE][SQL][SS] Upgrade Apache commons-io to 2.11.0 and revert change of SPARK-36456

What changes were proposed in this pull request?
SPARK-36456 change to use JavaUtils. closeQuietly instead of IOUtils.closeQuietly, but there is slightly different from the 2 methods in default behavior: swallowing IOException is same, but the former logs it as ERROR while the latter doesn't log by default.
Apache commons-io community decided to retain the IOUtils.closeQuietly method in the new version and removed deprecated annotation,  the change has been released in version 2.11.0.
So the change of this pr is to upgrade Apache commons-io to 2.11.0 and revert change of SPARK-36456 to maintain original behavior(don't print error log).
Why are the changes needed?

Upgrade Apache commons-io to 2.11.0 to use non-deprecated closeQuietly API, other changes related to `Apache commons-io are detailed in commons-io/changes-report

Revert change of SPARK-36737 to maintain original IOUtils.closeQuietly API behavior(don't print error log).

Does this PR introduce any user-facing change?
No
How was this patch tested?
Pass the Jenkins or GitHub Action

cc @HeartSaVioR @dongjoon-hyun this pr merge upgrade ·commons-io· and revert change of SPARK-36456 in one,  If need to split it into 2 pr, please let me know.

@HeartSaVioR It seems that there is no transitive dependency,  all maven dependency scope of Apache commons-io are test

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/47696/

Kubernetes integration test status failure
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/47696/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/47696/

Test build #143194 has finished for PR 33977 at commit 90a2544.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/143194/

Do we want to preserve this behavior (of suppressing log message) and rely on commons for something JavaUtils already does ? The intent of closeQuietly is more to ensure close does not throw exceptions - the lack of log message was behavior of commons - or is that a requirement as well ?

I can't say for every places we use, but if I understand correctly, in HDFSBackedStateStoreProvider, we simplified the logic via allowing to close stream twice (with suppression of log message), given the fact that IOException happens with temporary file and we don't hurt anything.
SPARK-36456 seemed to actually deal with deprecation warning message, which means the behavioral difference between IOUtils.closeQuietly and JavaUtils.closeQuietly is an unintentional change. If we identify somewhere we use IOUtils.closeQuietly which is better to log swallowed IOException, we should have a separate PR to change there to JavaUtils.closeQuietly.

IOUtils.closeQuietly can also receive consumer to define the behavior around IOException. The default behavior is no-op (so no log message).

IOUtils.closeQuietly can also receive consumer to define the behavior around IOException. (That said, JavaUtils.closeQuietly can be reimplemented as well leveraging IOUtils.closeQuietly) The default behavior is no-op (so no log message).

Yes, I was just thinking about this. Should we define a consumer for logging, and use IOUtils.closeQuietly(Closeable, Consumer<IOException>)  API where JavaUtils.closeQuietly is used

I like the consumer idea - with a default noop consumer which will ignore the message for the case @HeartSaVioR mentioned (when we truely dont care if exception is thrown).

Yeah that's one of idea on reimplementing JavaUtils.closeQuietly, but I agree it's orthogonal to this PR.

Thanks! Merging to master.

thanks all ~


HTML parsing breaks easily with whitespace

Description
Given this simple HTML:

Heading

Data

MyST fails. Rather than creating a single table, it creates a table for the first line and a tableRow for the second line. MyST attempts to handle this with the reconstructHtml transform - https://github.com/executablebooks/mystmd/blob/main/packages/myst-transforms/src/html.ts#L243 - but that function makes major assumptions about the shape of the HTML which are easily violated.
Proposed solution
We probably need an actual HTML parser in the reconstructHtml transform so we know if the block of HTML is entirely closed or if it still has open elements.

There should be an easy first pass on this that just combines sequential HTML nodes together if they are only separated by whitespace...

I can't actually reproduce this issue in any other case besides blank whitespace interruptions, so maybe we don't need more complicated html parsing... Once #1333 lands, I'll consider this closed - can always re-open if we encounter more issues!

Ok - it's still pretty easy to get html to fail, for example:
blah

doesn't work - it just loses the bold tag. However, fixing that requires a deeper change to the actual html parsing. This issue just handles correctly parsed html that is split across mdast nodes...

The fix to resolve the specific example in the original issue has been released, so I am going to consider this closed.
As my previous comment suggests, there is potentially more work to be done for smarter html parsing, but that can wait until more real examples come up.


[BUG]

Hello, thanks, I found your distro on a page and I'm using it but I can't install it because of the package error when I finish installing grub through livecd .. and I tried to download the current one but it's not available

I would like to have the honor of participating in your project my friend sorry if I write badly I am Brazilian linux user window not desktop thank you

aguardo respostas obrigado

Hello @wagnnerX The new version of Athena OS has been released. It should fix your issue. Try it.

yes it solved for me Athena OS "Parthenos" I did clean installation and
everything working gnome with tiling windows too good
Em seg., 23 de jan. de 2023 às 16:13, D3vil0p3r @.***>
escreveu:

Closed #26 https://github.com/Athena-OS/athena-iso/issues/26 as
completed.
—
Reply to this email directly, view it on GitHub
https://github.com/Athena-OS/athena-iso/issues/26#event-8337440103, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AF7GQI76TIXMJERJVGP7N63WT3J43ANCNFSM6AAAAAAUDYZQRM
.
You are receiving this because you were mentioned.Message ID:
@.***>


Wrong styles type provided by useStyles which has stylesheet with Platfrom.select(...)

Description
The platform-specific style property gets defaulted to ColorValue when we use Platform.select({ android: {...}, ios: {...}, ... }) inside stylesheet.
Steps to reproduce

Create stylesheet using Platform API

const stylesheet = createStyleSheet({
  toastContainer: {
    opacity: 1,
    minHeight: 60,
    ...Platform.select({
      ios: {
        shadowRadius: 12,
        shadowOpacity: 0.1,
        shadowOffset: { width: 0, height: 1 },
        shadowColor: "black",
      },
      android: {
        elevation: 1,
      },
      default: {},
    }),
  },
});

Use the styles in any View component

const { styles, theme } = useStyles(stylesheet);

<View style={styles.toastContainer} />

All the later platform styles type will be defaulted to ColorValue, here elevation type is defaulted to ColorValue.

Snack or a link to a repository (optional)
No response
Unistyles version
2.0.0
React Native version
0.73.2
Platforms
Android, iOS, React Native Web
Engine
Hermes
Architecture
Paper (old)

Thank you for the report. I know where the issue is, it's about Unistyles TypeScript types.
I will patch it as soon as possible, but it will most likely be next week due to my vacation.

What's your TypeScript version? I'm getting different results running your example:

Type:
(property) toastContainer: {
    shadowRadius: number;
    shadowOpacity: number;
    shadowOffset: {
        width: number;
        height: number;
    };
    shadowColor: string;
    elevation?: undefined;
    opacity: number;
    minHeight: number;
} | {
    elevation: number;
    ... 5 more ...;
    minHeight: number;
} | {
    ...;
}

I'm running your example in Unistyles example folder with Unistyles 2.0.2 and TS version set to 5.3.3

@jpudysz I'm not sure if it's my project specific, currently, I'm also using the h Unistyles 2.0.2 and TS version set to 5.3.3.
Here are the dependencies that I'm currently using.

Oh, thanks for the edit!
To be honest, I don't know why you have a different output. It should be exactly the same for everyone.
I will be waiting for your reply about this issue as I'm intrigued 🤔

@jpudysz Finally! After comparing every part with the new project, able to solve the issue. Not sure why it was caused.
I did added the compilerOptions i.e "strict": true, in tsconfig.json ✨ its working.

Thanks for the hint!
If someone will encounter it again I will mention it in the docs 💪


Update cadence version

Description
Update to the cadence version which includes the dependency fix: https://github.com/onflow/cadence/pull/3027
Blocker for https://github.com/onflow/flow-go/pull/5260
Tests are failing due to default contracts (e.g: core-contracts, flow-nft) not being updated yet. These tests should pass once those contracts are updated and the emulator is updated with the latest contracts.
For now, only needs the compilation to work so the flow-go can be updated.
Tests could/should be fixed before merging this PR.

For contributor use:

[x] Targeted PR against master branch
[x] Linked to GitHub issue with discussion and accepted design OR link to spec that describes this work
[x] Code follows the standards mentioned here
[x] Updated relevant documentation
[x] Re-reviewed Files changed in the GitHub PR explorer
[x] Added appropriate labels

@SupunS are the contracts (core-contrcats, nft, ft) currently being updated ? Are there PRs I should track about the updates? This is to unblock an emulator PR similar to yours :)

@tarakby Josh is working on updating the contracts. There is a PR to update flow-go with the latest changes: https://github.com/onflow/flow-go/pull/5301. Not sure if that will fix this issue for sure, but once that PR got merged, I will update this PR and see.

Nice work 👏🏼


Implement Lagrange kernel constraints

Closes: #240
Builds on: #245
Done:

failing test with Air which uses a Lagrange kernel auxiliary column
verifier

Left to do:

[x] Prover populates OodFrame with Lagrange kernel "frame"
[x] Prover evaluates Lagrange kernel transition and boundary constraints
[ ] Cleanup (including revisiting current API)

@irakliyk @Al-Kindi-0 this is ready for review!
The PR has a bigger diff than I originally anticipated. The main reason is that many existing abstractions did not apply to the Lagrange kernel column. For example, I had to create a LagrangeKernelEvaluationFrame, LagrangeKernelBoundaryConstraint, and LagrangeKernelTransitionConstraints, since the existing EvaluationFrame, BoundaryConstraints and TransitionConstraints made assumptions that didn't apply to the Lagrange kernel column. It would be nice to clean this up, but probably in another PR?
Related: #80

@irakliyk ready for another review! I still have the read_lagrange_kernel_frame_into() comment to address - which I agree with and will implement first thing tomorrow.

Added a benchmark that runs the prover on a trace with a Lagrange kernel column, with trace lengths 2^14, 2^15, 2^16. On my Macbook Pro M2, the results were:

trace length $2^{14}$: [426.12 ms 426.70 ms 427.21 ms]
trace length $2^{15}$: [906.85 ms 908.47 ms 910.41 ms]
trace length $2^{16}$: [1.9335 s 1.9348 s 1.9363 s]

@irakliyk this is ready for review!

Also, could you rebase this from the latest main - handling of no_std changed there slight.

I merged main in, but it seems like it made the diff against next bigger. Is this intended?

Regarding benchmarks: what exactly are these measuring (i.e., is it just Lagrange column constraint evaluation - or something else)? If this is just for evaluating constraints, we'll need to dig deeper into how to optimize this as it seems like the impact on prover performance would be quite bad

It is measuring the entire Prover::prove() function - not just the Lagrange constraint evaluation. I did this because the prover API is likely more stable than any internal Lagrange kernel API for constraint evaluation - but I can change it to just benchmark the Lagrange constraint evaluation if that is more useful. I personally used it more as a means of knowing if any given commit did indeed improve general performance, as opposed to knowing the absolute Lagrange kernel constraint evaluation time.

Addressed comments, and further optimized constraint evaluation.
Timing
For a trace length of $2^{20}$, timings are:
trace column building: 84ms
precompute transition divisors time: 11ms
invert transition divisors time: 34ms
invert boundary divisors time: 19ms
evaluate and merge constraints time: 387ms
TOTAL lagrange constraints eval time: 452ms
Benchmarks
Benchmarks are also now significantly faster:

trace length $2^{16}$: [163.01 ms 165.96 ms 168.60 ms]
trace length $2^{20}$: [3.0080 s 3.0399 s 3.0690 s]


my posts removed ??

I was testing instabot on my instagram and for some reason a lot of my posts have been removed ... how do I get them back? And what setting should I make so this does not happen again?

First is, you might want to use a different account or create a new one to test it. I got banned for multiple logins and also all 3 images were all deleted. Make sure not to login multiple times in a row using the script if its not working.

If you can still login, it will show up eventually. Good luck!

my account is disabled. but not account on instabot.
my personal account disabled. i just use multi login in instagram app (personal account + instabot account)

how you got banned? you exceeded limits?

Reopen if you still have not found the solution (don't forget to update the bot).


fix 'kubectl delete'

The command line to remove the load balancer didn't work for me.
The patch corrects it.

This change is

/assign @kunmingg

/ok-to-test

A nice catch. Thanks for the contribution
/lgtm

/approve

/assign


Unable to resolve path to module 'vite-ssr'.

Hello,
I'm using eslint with:
  extends: ['plugin:react/recommended', 'airbnb'],

unfortunately when I try to import 'vite-ssr' I have this error:

ESLint: Unable to resolve path to module 'vite-ssr'.(import/no-unresolved)

If someone has an idea :)

I'm not sure if it's related but vite-ssr is a virtual module, its path doesn't match the actual source code.
It already provides types so I think you can just disable ESLint for that specific import 🤔

I don't know if it's the best solution, but I updated my .eslintrc.js like this:
module.exports = {
 // ...
  settings: {
    'import/core-modules': ['vite-ssr'],
  },
  // ...
}

Thank you for your help.


chore(modstate): Remove IsStopped endpoint from StateModule

@Wondertan is this what you expected?
Closes #2906

@Chirag018 , could you please fix lint errors?

@Chirag018 , could you please fix lint errors?

yeah, will have a look on it

@vgonkivs,  While addressing lint issues, I identified code duplication in the state file. To resolve this, should I refactor the API.Interface struct or make other related changes?
cc: @Wondertan @renaynay

@Chirag018, refactoring that struct should do the thing.
cc @distractedm1nd for confirmation

OK, will wait for @distractedm1nd 's confirmation

Yes, it needs to be removed from the API struct as well as the Module interface - don't forget to remove the implementations as well :)
That should do the trick

@Chirag018 do you still want to finish up this PR? Just add //nolint:dupl to the file that's failing dupl lint

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (33f5cd3) 51.72% compared to head (f20414b) 50.82%.
Report is 21 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2912      +/-   ##
==========================================
- Coverage   51.72%   50.82%   -0.91%
==========================================
  Files         167      168       +1
  Lines       10801    10932     +131
==========================================
- Hits         5587     5556      -31
- Misses       4719     4874     +155
- Partials      495      502       +7

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

@Chirag018 can you rebase main onto here? Or at least adjust the PR so we can update the PR with base branch, then this'll merge. Thanks!


Revise error for incorrect subscript parameters.

We use subscripts for more than just indexes in Swift these days, so the error message needs to be a bit more general.

@swift-ci Please smoke test

@swift-ci Please smoke test

@swift-ci Please smoke test Linux platform

Wait, hang on. "index" is the term for a subscript argument, in addition to "Index" being the term for a Collection concept. If you want to kill the former, at least go with "argument" instead of "value".

"index" is the term for a subscript argument

I don't really agree with this, but I do agree that "argument" is better than "value" here! I'll make that change.

@swift-ci Please smoke test and merge

@swift-ci Please smoke test and merge

@swift-ci Please smoke test Linux platform

@swift-ci Please smoke test Linux platform

@swift-ci Please smoke test Linux platform


Upgrade https-proxy-agent

This PR upgrades the https-proxy-agent dependency.
The old version of https-proxy-agent brings in an old version of agent-base which has several serious issues that has since been fixed. Some of the most important ones can break built in https API for example, see https://github.com/TooTallNate/node-agent-base/pull/47.

@bjornharrtell Thanks, this will be addressed in #3983. Closing as duplicate


Performance overhead estimate?

This library looks very cool!  I was wondering if you have any ballpark estimate of where using regexp-worker becomes "worth it?"
For example, I'm sure it's not worth it if you're running /.*/ on a 10 character string, but is it worth it to run /someword/ on a hundred character string, a thousand character string? Any idea?
Thanks for sharing this cool project!

Great question.
It works best in batch style requests to reduce the call overhead.
The main reason I created this was to address issues where the regexp came from the user causing extreme performance issues and even lockups.
For example, run a regexp against an entire document.


Update readMe and upgrade to new version 3.84

Hi, we tried upgrading the package from 3.7.0 to 3.8.4, but we are facing an issue while accessing the storage account endpoint URL. It just loads black page & doesn't proceed for authentication but when we load from the CDN URL, it loads completely fine and does the authentication. Are we missing any configuration in the app config, or is there a change needed at the code level?


Remove obsolete "build transactions" guide (or link to the new one)

Hello! I'm a new developer in the Cardano ecosystem, and I just spent a few days trying to submit my first transaction. I was following this "Create Simple Transaction" guide mostly word-for-word, and it recommends using cardano-cli transaction build-raw to construct transactions (and cardano-cli transaction calculate-min-fee to compute the fees yourself).
After working through my issues, I learned that this guide was obsolete for a few reasons, and that I should have been following this "Cardano CLI Simple Transactions" guide instead. The old guide is the only one that shows up for me on Google, and I didn't even notice the new one until I was writing this very issue.
I think the old guide should get taken down so other people don't make the same mistake I made.

@CarlosLopezDeLara this appears a consequence of https://github.com/cardano-foundation/developer-portal/pull/1287 - would you like to remove the old page & redirect it to your new one, and/or merge the content somehow? (though I feel sure you've already accommodated everything useful from the old page).
You could follow the redirect syntax in docusaurus.config.js last used extensively in https://github.com/cardano-foundation/developer-portal/pull/1280.

I'll take care of it. Thanks for the report @SupernaviX

Ok, I removed the file in question to see what links would break:
Exhaustive list of all broken links found:

- On source page path = /docs/get-started/secure-workflow:
   -> linking to /docs/get-started/create-simple-transaction
   -> linking to /docs/get-started/create-simple-transaction#determine-the-ttl-time-to-live-for-the-transaction (resolved as: /docs/get-started/create-simple-transaction)
   -> linking to /docs/get-started/create-simple-transaction#check-the-balances (resolved as: /docs/get-started/create-simple-transaction)

After reading the article that links to this, I think it will need some refactoring as well. This to say that this may take a bit longer than expected. I'll start working on it and hopefully have a PR for Monday - Tuesday.

Thank you for the quick response!

If removing the page is difficult, could the old page link to the new one at the top as a short-term workaround?


Zoning E2E Test + E2E Improvements

Description
The primary goal of this PR is to add an E2E test scenario (and robust testing script) that covers the new zoning work. In addition to that, some improvements were added to the E2E test suite.

All custom tests in E2E scenarios are now part of an 'array' of custom tests for their scenario, and can be selected individually. This allows for multiple custom tests to be run per scenario, which was necessary for adding and removing zone labels to k8s nodes. Future scenarios added will need to have their custom tests indented inwards to be part of an array, but legacy support for the previous 'run custom test' E2E step was left intact.
A new way of deploying secrets as a part of E2E tests was introduced, that creates the secret by reading in the template file, editing the string in-memory, and deploying from that string. This was necessary because the previous way templates were used, editing in place, created an impossible scenario to restore from in our testing: if any two template keys had the same value, it was impossible to reverse the string replacement that was done.

GitHub Issues
List the GitHub issues impacted by this PR:

GitHub Issue #

https://github.com/dell/csm/issues/1613

Checklist:

[X] I have performed a self-review of my own code to ensure there are no formatting, vetting, linting, or security issues
[X] I have verified that new and existing unit tests pass locally with my changes
[X] I have not allowed coverage numbers to degenerate
[X] I have maintained at least 80% code coverage
[X] I have commented my code, particularly in hard-to-understand areas
[X] I have made corresponding changes to the documentation
[X] I have added tests that prove my fix is effective or that my feature works
[X] I have maintained backward compatibility
[X] I have executed the relevant end-to-end test scenarios

How Has This Been Tested?
E2E scenario has been run standalone. It passes, though that does not actually test zoning yet, as zoning work is not completed.

Ignore the commit to revert, I went to click 'delete branch' and hit 'revert' instead, which opened a new branch with a revert commit. I've deleted the branch.


Is it possible to prevent cancel unless scrolling inside a container?

I am doing
useEffect(() => {
  scroller.scrollTo('bottom', {
    duration: 80000,
    containerId: 'sidebar',
    smooth: true
  })
}, [])

to start scroll in a container when a page loads, however, any action outside of this container stops the scrolling.
I would like it to continue regardless of what I do outside of it.
Is it possible?
I tried to call scroller.scrollTo again after the scroll was paused (for example if a click changed a state) but it wouldn't resume.

Having this same issue


enumerate_devices for PulseAudio

Method enumerate_devices for PulseAudio follows the pattern of calling async methods and waiting for the results in order to continue (like the rest of the code). What stranges me here is that it does not hold the mutex before async method(s) call and it does not signal from inside the callbacks like purposed in pulse documentations.
http://freedesktop.org/software/pulseaudio/doxygen/threaded_mainloop.html#basic_subsec
To be honest I use it a lot in my examples without any problem so I wonder what I am missing here and if we will be fine since this code is landed in firefox and will be overused.

None of the device enumeration API is used in Firefox yet, so beyond the testing Haakon did and what I noticed in the review during merge, it hasn't had a lot of scrutiny yet... so it's very possible that there are bugs.  Sounds like this needs more investigation.

This issue solved with #71