[tools] flutter_migrate wrapper command

@GaryQian isn't contributing to Flutter any more, if anyone would like to pick this up please don't hesitate to do so (maybe coordinate on the #hackers-android channel on Discord first!).


Upload percentage

Please add an option to show upload percentage

https://github.com/out386/aria-telegram-mirror-bot/commit/9d2e29ad44e0ed2a8d61be7699685c10f9fc251d


remove some internal API

remove some internal API

this API is in draft and is subject to change.
semver major because it technically removes functionality

contains whitespace per #120

please just review https://github.com/IBM-Cloud/gp-js-client/pull/119/commits/20d0071547df1c8482ad5bba56aa51fbb527ab6a — the rest is whitespace


DesktopBridgeToUWP

Hi,
Thank you for your sample code , is there any code/knowledge on how to do DesktopBridge on javascript version? My code is on hosted web app and want to call Desktop App to get data.
Right now what I do is set appService to be StartPage="js/appservice.js"
and after UWP ( index.html ) call to Desktop via launchFullTrustProcessForCurrentAppAsync it open WPF App , after that WPF app call back on AppServiceConnection and open the connection to UWP App on StartPage ( appservice.js ) but my deadend here since my index.html cannot get anything from appservice.js , even in appservice.js show messagebox that come from WPF it still not show.
Did I missing some concept or how I can make it work
Thank you,
Tour

Hi Tour,
Thanks for reporting the issue.
One thing you can try is adding the following to the manifest:

UWP.dll

We will work to add a javascript sample in future.
Thanks,
Vladimir


Improve configuration errors and nest iohk-monitoring config under Logging key

Now shows exactly which config values are missing (and the corresponding CLI option if it exists) and which fields couldn't be parsed in the config file.
An example:
Configuration values were either not passed via the CLI or missing from the configuration file configuration/morpho-configuration.yaml:
- Field StableLedgerDepth missing
- Failed to parse field PrometheusPort: parsing Int failed, expected Number, but encountered String
- Field TopologyFile missing
  and CLI option --topology not passed
- Unrecognized field PowBlockFetchInterval

Whereas previously you'd get:
morpho-checkpoint-node: user error (key "StableLedgerDepth" not found)
# (adding the missing field)

morpho-checkpoint-node: user error (parsing Int failed, expected Number, but encountered String)
# (figuring out that the error is for the PrometheusPort field, then fixing it)

morpho-checkpoint-node: user error (key "TopologyFile" not found)

# (no error at all showing that PowBlockFetchInterval isn't a recognized field)

In addition, the iohk-monitoring configuration is now nested under the Logging key of the config file. This includes settings like minSeverity and setupScribes. The somewhat full list of options can be seen in https://github.com/input-output-hk/iohk-monitoring-framework/blob/master/iohk-monitoring/test/config.yaml
The reason this is done is because mixing two different configuration file parsers into the same namespace in the same file gets messy and confusing.

There can be conflicts between fields. If iohk-monitoring suddenly introduces a field that morpho already uses, the value for morpho will be used for iohk-monitoring as well (which probably wouldn't make sense)
The naming of options is inconsistent (iohk-monitoring uses lowercase, morpho uppercase)
We can't provide error messages for unknown fields, as they might be used by the other config file parser

These things aren't a problem anymore now.
This relies on the changes in #25

Related to https://github.com/input-output-hk/iohk-monitoring-framework/pull/616, which makes sure that with this change, some implicit Representation conversions are still done.


Move ignite-vector-icons to new repo

Let's promote this package to its repo.

Also, lets make sure we release it on npm before removing it from the ignite repo.

The repo getting lighter and lighter, Steve. This is a good sign.


[BIOMAGE-1638] Parametrise the Github org name in default-config.js in the API and make it's value conditional

Description

Details
URL to issue
https://biomage.atlassian.net/jira/software/c/projects/BIOMAGE/boards/1?modal=detail&selectedIssue=BIOMAGE-1638
Link to staging deployment URL (or set N/A)

Links to any PRs or resources related to this PR

Integration test branch
master

Merge checklist
Your changes will be ready for merging after all of the steps below have been completed.

Code updates
Have best practices and ongoing refactors being observed in this PR

[ ] Migrated any selector / reducer used to the new format.

Manual/unit testing

[ ] Tested changes using InfraMock locally or no tests required for change, e.g. Kubernetes chart updates.
[ ] Validated that current unit tests for code work as expected and are sufficient for code coverage or no unit tests required for change, e.g. documentation update.
[ ] Unit tests written or no unit tests required for change, e.g. documentation update.

Integration testing
You must check the box below to run integration tests on the latest commit on your PR branch.
Integration tests have to pass before the PR can be merged. Without checking the box, your PR
will not pass the required status checks for merging.

[ ] Started end-to-end tests on the latest commit.

Documentation updates

[ ] Relevant Github READMEs updated or no GitHub README updates required.
[ ] Relevant Wiki pages created/updated or no Wiki updates required.

Optional

[ ] Staging environment is unstaged before merging.
[ ] Photo of a cute animal attached to this PR.

@StefanBabukov can you also not forget to do a release on this one: if you forget to release this, as soon as we do the rename of the Github org, we will be unable to launch workers and pipelines in production, since biomage-ltd won't exist.


bug: Horizontal scroll slider in the wrappid-app homepage.

Describe the bug
There is a horizontal slider in the wrappid-app homepage .
Expected behavior
The page should not have a horizontal slider by default.
Solution

Remove the default LayoutsAppDiv height and width.

As of now closing this issue


Lower PCIDevice-specific operations to PCIDevice class

The tt_SiliconDevice class contains data structures that are indexed by the so-called "logical" id, yet some of these structures represent things that conceptually do not exist for remote chips.
From this standpoint, these structures would make more sense if they were indexed by "device id" (not to be confused with PCIe device id).  However, a better design would move these concepts out of the tt_SiliconDevice class and into PCIDevice.

I'm going through some rewrite... here are the items that make sense to be moved:

bar io: bar_write32 and bar_read32
iatu specific code: iatu_configure_peer_region and init_pcie_iatus and code from destructor
hugepages: lots of code, init_hugepage, also destruction. Also moves out the communication with the kernel driver.
arc_msg: There is also remote_arc_msg. Some part should live in some lower part of the stack, and some part has to be part of the Chip abstraction.

@broskoTT situation here?

@broskoTT situation here?

In progress.
#253 resolved hugepages part.
The other 3 items still remain.


Terminal cursor shape is changed and not reset when exit REPL

Describe the bug?
Compiled erg wiht full-repl features. Enter the REPL and exit, the cursor shape is changed to Line type.
Reproducible code
Change the cursor type in the terminal to block type. Enter the REPL and then exit.
Expected result
Not changed.
Actual result
Chagned.
Additional context
I think this line is unneeded.
https://github.com/erg-lang/erg/blob/a6db8b94bdc7ec83f1e6fd7fe3fef6280975718a/crates/erg_common/stdin.rs#L106
Erg version
main branch
Python version
None
OS
None

@C-BJ What was your intention in making the shape of the cursor a line at this point?
Are there any problems with removing it?

And, I see that SetCursorShape has been removed in the latest crossterm (v0.26.1).

In v0.26.1, SetCursorStyle has replaced SetCursorShape.

@C-BJ What was your intention in making the shape of the cursor a line at this point?
https://github.com/erg-lang/erg/blob/a6db8b94bdc7ec83f1e6fd7fe3fef6280975718a/crates/erg_common/stdin.rs#L106
Are there any problems with removing it?

There should be no problem deleting it, I think I've forgotten the reason for doing so in the first place.


Reference to file different from check existence

This line is different from line checking if file exists. Is it right?
https://github.com/Lith/strapi-provider-upload-google-cloud-storage/blob/d1177c2ee0f9c6d4d37703ac6c46fbc550e0c498/lib/index.js#L115
https://github.com/Lith/strapi-provider-upload-google-cloud-storage/blob/d1177c2ee0f9c6d4d37703ac6c46fbc550e0c498/lib/index.js#L126

Hi @dgmike :)
It's a good question !
When we create a new file, call my-awesome-file.jpg , the file name contains my-awesome-file.jpg.
So we need to add the filePath in order to have the full url.
But, if the file already exist and we need to remove it, we need to give only the fileName.
Not sure if it's clear, but on Google Storage the "upload" and "delete" defer a little bit.
Thanks

I see...


Docker cgroupdriver configuration is misleading

The Kubelet use by default "cgroupfs" on Debian/Ubuntu/OpenSUSE and "systemd" on Fedora/CentOS/RHEL so we should make it clear in the installation documentation.

This change is

@gooodmorningopenstack  please resolve the conflict :)

@xiangpengzhao done

/lgtm

@gooodmorningopenstack It still shows that This branch cannot be rebased due to conflicts. Please check it :)
And the commit Merge branch 'master' into patch-1 seems a bit strange :)


[Feature] Added Dockerfile which takes base Python image, installed pipx & fabric

What this Pull Request (PR) does
Introduces a Dockerfile configured to set up environment for working with Fabric.

Installs, git, build-essential, ffmpeg.
Installs and Congifures pipx
Clones fabric repo
Sets working dir to /app
Set entry point to /usr/bin/bash/

Note:
This Dockerfile is not hardened for security purposes. If you mount this container to your file system you expose yourself highly to the container. Any users should probably made aware of this before suggesting it's use.
Feedback and suggestions for improvements and hardening are welcome! 🐱
Related issues
#25
closes #275

This is great! And I suggest something like this:
FROM python:3.12.2-slim

# Install required packages
RUN apt-get update \
    && apt-get install -y --no-install-recommends \
        git \
        build-essential \
        ffmpeg \
        sudo \
    && rm -rf /var/lib/apt/lists/* \
    && apt-get clean

# Create a non-root user
RUN useradd --create-home appuser \
    && echo "appuser ALL=(ALL) NOPASSWD: ALL" > /etc/sudoers.d/appuser \
    && chmod 0440 /etc/sudoers.d/appuser
USER appuser

# Set up work directory
WORKDIR /home/appuser/app

# Install pipx
RUN python3 -m pip install --upgrade pip \
    && python3 -m pip install --user pipx \
    && python3 -m pipx ensurepath

# Clone the repository and install its dependencies
RUN git clone https://github.com/danielmiessler/fabric.git \
    && python3 -m pipx install ./fabric

# Set file permissions
RUN chmod -R 755 /home/appuser/app

# Use a least privileged user
USER appuser

# Set the entrypoint
ENTRYPOINT ["/usr/bin/bash"]

This is a bit better, security-wise, and you can mount your ~/.config/fabric inside the container to use your API keys.
docker run --rm -i -t -v C:\Users\kayvan\.config\fabric\:/home/appuser/.config/fabric fabric:latest

appuser@07eba7feb23e:~/app$ ls -la ~/.config/fabric/
total 12
drwxrwxrwx 1 root root 4096 Apr  6 20:34 .
drwxr-xr-x 3 root root 4096 Apr  8 06:57 ..
-rwxrwxrwx 1 root root  250 Mar 30 23:58 .env
-rwxrwxrwx 1 root root 4970 Apr  6 20:34 fabric-bootstrap.inc
drwxrwxrwx 1 root root 4096 Apr  6 20:34 patterns

appuser@07eba7feb23e:~/app$ fabric --listmodels
GPT Models:
gpt-3.5-turbo
gpt-3.5-turbo-0125
gpt-3.5-turbo-0301
gpt-3.5-turbo-0613
gpt-3.5-turbo-1106
gpt-3.5-turbo-16k
gpt-3.5-turbo-16k-0613
gpt-3.5-turbo-instruct
gpt-3.5-turbo-instruct-0914
gpt-4
gpt-4-0125-preview
gpt-4-0613
gpt-4-1106-preview
gpt-4-1106-vision-preview
gpt-4-turbo-preview
gpt-4-vision-preview

Local Models:

Claude Models:
claude-3-opus-20240229
claude-3-sonnet-20240229
claude-3-haiku-20240307
claude-2.1

Or even use local OLLAMA models running on the host like this:
appuser@07eba7feb23e:~/app$ export OLLAMA_HOST=host.docker.internal
appuser@07eba7feb23e:~/app$ fabric --listmodels
GPT Models:
gpt-3.5-turbo
gpt-3.5-turbo-0125
gpt-3.5-turbo-0301
gpt-3.5-turbo-0613
gpt-3.5-turbo-1106
gpt-3.5-turbo-16k
gpt-3.5-turbo-16k-0613
gpt-3.5-turbo-instruct
gpt-3.5-turbo-instruct-0914
gpt-4
gpt-4-0125-preview
gpt-4-0613
gpt-4-1106-preview
gpt-4-1106-vision-preview
gpt-4-turbo-preview
gpt-4-vision-preview

Local Models:
codellama:13b
codellama:latest
dolphincoder:latest
llama2:13b
llama2:latest
mistral:latest
starcoder2:15b

Claude Models:
claude-3-opus-20240229
claude-3-sonnet-20240229
claude-3-haiku-20240307
claude-2.1

Awesome feedback! @ksylvan I've added in the recommended changes. The mounting of the config file is also very clean.
Thanks!

nice work!  I was about to try to dockerise fabric and you've already done it :)

Thanks! @gilesheron
Side note for everyone that comes across this PR: I'll update the fork occasionally with the latest updates, but the dockerfile itself will clone from the main repo. So you won't have to worry about updating there. (If fabric moves to 3.13 support I'll make sure to update dockerfile to the latest python base image).

This is amazing work but we're migrating to go before too long, so it won't be needed. But THANK YOU.

Sounds good! Looking forward to the migration! 😄

@7kevin49 This is great thanks for this, I was trying to do this myself until I found out your message, one question and sorry if this is a dumb question I'm new in all this, I got my container set up and all looks good but the container do not keep running, is there anything I need to do in order for the container to stay running?

@7kevin49 This is great thanks for this, I was trying to do this myself until I found out your message, one question and sorry if this is a dumb question I'm new in all this, I got my container set up and all looks good but the container do not keep running, is there anything I need to do in order for the container to stay running?

@Ishidad fabric isn't the kind of command you keep running usually, so I don't understand your question. The idea of the container is to set up another easy packaging of the whole stack and make it accessible to more people.
What is it you're trying to do with the container?

Thanks for your reply, I friend help me understand what you mentioned, my idea was to connect to the container to run the prompts as I want to have it in a server and not in my local machine

This is amazing work but we're migrating to go before too long, so it won't be needed. But THANK YOU.

Out of curiosity why would migrating to Go require closing and not merging a working Dockerfile for the project as-is? As an engineer, I'd rather run containerized services than to install global packages for individual projects on bare metal that cause "works on my machine" type problems elsewhere.

Is there a practical need for user shenanigans @ksylvan suggested? Why should be care about permissions inside docker container that has a specific purpose?

they have switched to go. is this going to be updated?
think there should be a .env file to set the following:
local_ollama_server_address: {localhost:11434}
local_default_ollama_model: {llama3.1}
openai_key:
claude_key:

This is amazing work but we're migrating to go before too long, so it won't be needed. But THANK YOU.

sorry but containerising this project is important. I deploy my ai-stack which consists on the following services all in one docker network and docker-compose. Otherwise I will be dealing with dependency hell forever:
ollama
open-webui
whisper
stable diffusion
searxng
fabric (yet to be deployed using docker)

Thanks for your reply, I friend help me understand what you mentioned, my idea was to connect to the container to run the prompts as I want to have it in a server and not in my local machine

use docker run -d which is to run the container as a daemon in the background and you can connect to it as and when required

Is there a practical need for user shenanigans @ksylvan suggested? Why should we care about permissions inside docker container that has a specific purpose?

It's just generally good practice. Some rules around docker require non-root be used for running the applications as well.

This is amazing work but we're migrating to go before too long, so it won't be needed. But THANK YOU.

sorry but containerising this project is important. I deploy my ai-stack which consists on the following services all in one docker network and docker-compose. Otherwise I will be dealing with dependency hell forever:

ollama

open-webui

whisper

stable diffusion

searxng

fabric (yet to be deployed using docker)

You can change to a different commit within the Dockerfile after cloning the repo. Just be aware you'll be running outdated/unmaintained code.

You can change to a different commit within the Dockerfile after cloning the repo. Just be aware you'll be running outdated/unmaintained code.

are you planning to dockerise the new go version by any chance? @7kevin49 ??

I can give it a shot later today. Looks like it should be rather straight forward.

I can give it a shot later today. Looks like it should be rather straight forward.

That would be great, thanks!

I created one already...
There is a PR here (https://github.com/danielmiessler/fabric/pull/845)


Junit reporter doesn't include project root in output

Describe the bug
As originally reported in #2566, but not quite resolved in #256 as anticipated, in a monorepo / multi-workspace setup, the Junit reporter does not include the project root in its output (see example below)
Here, this testsuite is in... well... damn, I don't know which project it's in, because the testsuite.name attribute is relative.  It should be something like packages/my-package/tests/index.test.ts.
<?xml version="1.0" encoding="UTF-8" ?>
<testsuites name="vitest tests" tests="67" failures="0" errors="0" time="1.636">
    <testsuite name="test/index.test.ts" timestamp="2024-01-31T22:24:19.411Z" hostname="MacBook-Pro-M1.local" tests="1" failures="0" errors="0" skipped="0" time="0.016">

Reproduction
(no repro -- reporting a previously closed bug)
System Info
System:
    OS: macOS 13.6.3
    CPU: (10) arm64 Apple M1 Pro
    Memory: 213.03 MB / 32.00 GB
    Shell: 5.9 - /bin/zsh
  Binaries:
    Node: 18.16.0 - ~/.asdf/installs/nodejs/18.16.0/bin/node
    Yarn: 1.22.19 - /opt/homebrew/bin/yarn
    npm: 9.5.1 - ~/.asdf/plugins/nodejs/shims/npm
    Watchman: 2<IP_ADDRESS> - /opt/homebrew/bin/watchman
  Browsers:
    Chrome: 121.0.6167.85
    Firefox: 122.0
    Safari: 16.6
  npmPackages:
    @vitest/coverage-v8: ^1.2.2 => 1.2.2
    vitest: ^1.2.2 => 1.2.2

Used Package Manager
npm
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Check that this is a concrete bug. For Q&A open a GitHub Discussion or join our Discord Chat Server.
[X] The provided reproduction is a minimal reproducible example of the bug.

I see what you mean. I guess it should be relative to the root config instead of a project one.

If it's any help, I'm also using vitest-sonar-reporter, and that one uses the root properly.


"as explained in the readme.md. There is significant benefit which is outlined in the the readme"

We read the following comment In: "common/TokenService.ashx.cs"
For the Web SDK to authorize correctly, you still must assign Azure role based access control for the managed identity as explained in the readme.md. There is significant benefit which is outlined in the the readme.
The readme file does not have any explanation on how to set this up.
What Readme file are we referring to here ?

Thanks for highlighting this. The authentication code comes from a separate project here: https://github.com/Azure-Samples/Azure-Maps-AzureAD-Samples
In particular, the client grant method: https://github.com/Azure-Samples/Azure-Maps-AzureAD-Samples/tree/master/src/ClientGrant/AzureMapsAzureAdDaemon
I'll update the comment in the code sample service.


Base64 encode the API key

Fixes bug where the API key was not base64 encoded as required by the API. Previously users had to provide their API key as a base64 encoded string for the client to work.

💯


Declutter kwargs in convert_cli

In #404, we used a workaroung to add config_file to the kwargs (it is an input to thie `convert_cli? function).
if config_file:
    kwargs["config_file"] = config_file

Actually, this whole discussion is part of a bigger problem, namely that we call
data = data_reader().read(  # type: ignore[operator]
        template=Template(template), file_paths=input_file, **kwargs
    )

in the convert function. But, most of our reader don't actually accept kwargs, so this will always be a bit unstable. We should fix the kwargs downstream passing and general argument handling.

Relevant discussion here: https://github.com/FAIRmat-NFDI/pynxtools/pull/404#discussion_r1726687480


[CWS] fix tag resolver usage in windows functional tests

What does this PR do?
This PR fixes an issue with the tag resolver in CWS functional tests for windows. On linux functional tests were already using a fake tag resolver but not on windows. When https://github.com/DataDog/datadog-agent/commit/2786875f1a233bf397f2590e83edde45ee429bc5 was merged, since the remote tagger was already disabled before, we no ended up with a nil pointer dereference.
This PR simply re-uses what is done on linux, but on windows this time.
This PR also removes the flake marker and mark the job as "allowed_to_fail". The goal here is to have the job run, collect information on flakyness across a few days, and then remove the "allowed_to_fail"
Motivation
Describe how to test/QA your changes
This code is covered by tests (that are failing, which is why this PR exists in the first place), and it mostly touching test code.
Possible Drawbacks / Trade-offs
Additional Notes

/merge


Install protoc in tools/bin dir

How to categorize this PR?

/area dev-productivity robustness
/kind enhancement
What this PR does / why we need it:
Now, protoc is installed ad-hoc in tools/bin dir.
Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:
Release note:

NONE

/test all


SignalInfo in new AudioMetaData

Hi,
We're using sox_io backend for torchaudio. New version of the AudioMetaData class returned by the info method doesn't have all the data I need for analysis.
 info method in the deprecated sox backend used to return sox_signalinfo_t and sox_encodinginfo_t that had much more detailed information about encoding, bit depth etc.
What should I use to get the info about audio file's encoding, bit depth and other details?

(since this is part of the BC-breaking change discussed in #903, cc @mthrok)

Hi @RimaOlson
Currently there is no way to get the other information using "sox_io" backend.
We made this decision because, exposing the internal of libsox makes it very difficult to maintain the backends across the different OS.
Having said that, could you give information of what type of attributes you are using as a form of feature request?
From there I can come up with a way to make it available again.

@mthrok we use following attributes to ensure proper encoding of audio files before feeding them into our models:
sox_signalinfo_t.length
sox_encodinginfo_t.encoding
sox_encodinginfo_t.bits_per_sample

@mthrok we use following attributes to ensure proper encoding of audio files before feeding them into our models:
sox_signalinfo_t.length
sox_encodinginfo_t.encoding
sox_encodinginfo_t.bits_per_sample

Thanks @RimaOlson

Could you tell, which value do you use for encoding?
For the available values of encoding, you can see https://fossies.org/dox/sox-14.4.2/sox_8h.html#ab0a659b0258d1989c751ba5415e6a4cd

length is the number of samples (across channels) so you should be able to get the number with num_channels * num_frames. Let me know if this is not the case.

encoding-> 1 (SOX_ENCODING_SIGN2) and encoding-> 9 (SOX_ENCODING_ULAW )
So num_frames is the same as sox_signalinfo_t.length for mono files?

@RimaOlson

encoding-> 1 (SOX_ENCODING_SIGN2) and encoding-> 9 (SOX_ENCODING_ULAW )

Thanks

So num_frames is the same as sox_signalinfo_t.length for mono files?

Yes.

@RimaOlson One more question about encoding. What formats (extensions) does your application expect?
I think there are formats other than WAV that are mapped to SOX_ENCODING_SIGN2, and would like to get a clearer vision if possible.

@mthrok we use wav files

Note to self: this is the corresponding encoding + bit depth part in PySoundFile
https://github.com/bastibe/python-soundfile/blob/744efb4b01abc72498a96b09115b42a4cabd85e4/soundfile.py#L38-L94

Note2: It is ideal that the mapping we create from sox_encoding_t to our custom type is generic enough to handle ffmpeg's ffmpeg's AVCodecID enum.

Note2: It is ideal that the mapping we create from sox_encoding_t to our custom type is generic enough to handle ffmpeg's ffmpeg's AVCodecID enum.

Hi @RimaOlson
Can you illustrate what your if statement looks like when you use sox_encodinginfo_t.encoding?
I am wondering what the new data structure should look like. Strongly typed ENUM type or just string would do.

Hi @RimaOlson
Can you illustrate what your if statement looks like when you use sox_encodinginfo_t.encoding?
I am wondering what the new data structure should look like. Strongly typed ENUM type or just string would do.

@RimaOlson

@mthrok it will be something like this:
   switcher = {
        0: "SOX_ENCODING_UNKNOWN ",
        1: "SOX_ENCODING_SIGN2",
        2: "SOX_ENCODING_UNSIGNED",
    …
    }

Thanks. Suppose, the type of format does not seem to matter. I will most likely pass as string.

QQ: does torchaudio.load automatically transcode to linear by default?

I am not an audio expert so I am not sure if I understand the question correctly, but assuming "transcode to linear" means something like Linear PCM, I think the answer is yes.
libsox internally decode audio samples into 32bit signed integer [doc], which is what torchaudio.load is returning (with or without normalization to float32).

@RimaOlson

@mthrok it will be something like this:
   switcher = {
        0: "SOX_ENCODING_UNKNOWN ",
        1: "SOX_ENCODING_SIGN2",
        2: "SOX_ENCODING_UNSIGNED",
    …
    }

Thanks. Suppose, the type of format does not seem to matter. I will most likely pass as string.

QQ: does torchaudio.load automatically transcode to linear by default?

I am not an audio expert so I am not sure if I understand the question correctly, but assuming "transcode to linear" means something like Linear PCM, I think the answer is yes.
libsox internally decode audio samples into 32bit signed integer [doc], which is what torchaudio.load is returning (with or without normalization to float32).

@mthrok thank you.
Yes, I did mean transcoding to Linear PCM from the original encoding. I'll take a look at libsox

@mthrok thank you.
Yes, I did mean transcoding to Linear PCM from the original encoding. I'll take a look at libsox

@RimaOlson
FYI: For bit depth, we are adding bits_per_sample attribute in #1177.

@RimaOlson
FYI: For bit depth, we are adding bits_per_sample attribute in #1177.

@mthrok Sounds good, thank you.

@mthrok Sounds good, thank you.

Hi @RimaOlson
Here is the mapping I am thinking to add. Let me know if this might not work for you.

SoX Representation
torchaudio Representation

SOX_ENCODING_UNKNOWN
"UNKNOWN"

SOX_ENCODING_SIGN2
"PCM_S"

SOX_ENCODING_UNSIGNED
"PCM_U"

SOX_ENCODING_FLOAT
"PCM_F"

SOX_ENCODING_ULAW
"ULAW"

SOX_ENCODING_ALAW
"ALAW"

SOX_ENCODING_MP3
"MP3"

SOX_ENCODING_VORBIS
"VORBIS"

SOX_ENCODING_AMR_WB
"AMR_WB"

SOX_ENCODING_AMR_NB
"AMR_NB"

SOX_ENCODING_OPUS
"OPUS"

Hi @RimaOlson
Here is the mapping I am thinking to add. Let me know if this might not work for you.

SoX Representation
torchaudio Representation

SOX_ENCODING_UNKNOWN
"UNKNOWN"

SOX_ENCODING_SIGN2
"PCM_S"

SOX_ENCODING_UNSIGNED
"PCM_U"

SOX_ENCODING_FLOAT
"PCM_F"

SOX_ENCODING_ULAW
"ULAW"

SOX_ENCODING_ALAW
"ALAW"

SOX_ENCODING_MP3
"MP3"

SOX_ENCODING_VORBIS
"VORBIS"

SOX_ENCODING_AMR_WB
"AMR_WB"

SOX_ENCODING_AMR_NB
"AMR_NB"

SOX_ENCODING_OPUS
"OPUS"

@mthrok above list looks good, thank you.

@mthrok above list looks good, thank you.

@RimaOlson I have merged #1206, which adds encoding attribute to AudioMetaData class. This will become available in tomorrow's nightly build. Please try it out and let me know if this works.

@RimaOlson I have merged #1206, which adds encoding attribute to AudioMetaData class. This will become available in tomorrow's nightly build. Please try it out and let me know if this works.

@RimaOlson Did you have time to try it out? Let me know if it is not working as you expected.

Thank you @mthrok , I checked out the nightly build. Tested the pcm and mulaw, they look good. Still trying to test OPUS and MP3. The issue might that it's more of a sox issue than torchaidio.

Actually I looked at the headers with ffmpeg, it looks the same as in AudioMetaData.encoding. When can it be merged into master?

Hi @RimaOlson
It's been merged to master and it will be released as part of 0.8 soon. We expect the release to happen early March.

Closing the issue. If you find something, please open a a new one. Thanks for the feedback!

Thanks a lot @mthrok !


Branch State Testing

Codecov Report

Merging #326 (d94efb2) into master (d7653dd) will increase coverage by 0.50%.
The diff coverage is 28.57%.

@@             Coverage Diff              @@
##             master     #326      +/-   ##
============================================
+ Coverage     39.21%   39.71%   +0.50%
- Complexity      394      416      +22
============================================
  Files           114      114
  Lines          2229     2241      +12
  Branches        203      206       +3
============================================
+ Hits            874      890      +16
+ Misses         1249     1239      -10
- Partials        106      112       +6

Impacted Files
Coverage Δ
Complexity Δ

...rc/main/java/seedu/address/state/StateManager.java
91.93% <0.00%> (+15.26%)
31.00 <4.00> (+14.00)

.../address/state/budgetindex/BudgetIndexManager.java
53.33% <ø> (+22.08%)
8.00 <0.00> (+2.00)

...tate/expenditureindex/ExpenditureIndexManager.java
46.66% <30.00%> (+46.66%)
7.00 <5.00> (+7.00)

...rc/main/java/seedu/address/model/ModelManager.java
39.23% <50.00%> (-2.80%)
28.00 <1.00> (-1.00)

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 d7653dd...d94efb2. Read the comment docs.


Icon family in variables

There is a problem with variables, if i change icon family to FontAwesome it will still read default value which is Ionicons.
property name:   iconFamily

I had a similar issue (#493). Have you tried upgrading to 2.0.5?

Update it, works now.


.eslintignore not working

Hi,
I have this version of VSCode 1.35.1, for macOS, this is my project structure:
.eslintrc
.eslintignore
js/
   ...
   app.js

I want ignore app.js and i have write this in ignore file:
js/app.js

But Eslint continue a give me error on that file and lint it, I can't understand where is the problem, any help?

Is it ignored when running ESLint from the terminal. If yes, can you please provide me with a GitHub repository I can clone that demos what you see.
Also note the the .eslintignore file must be in the current working directory to work. It is not honor up the parent file chain. This is ESLint npm behaviour.


Update schema.graphql

@mendesfabio @markusbkoch Added descriptions to schema.  Once completed this will help finalize PR - Revised Subgraph Documents

Thanks @PaulieB14 . The pattern Fabio linked to helps explorers like thegraph's playground display the documentation in their UI - see eg https://thegraph.com/hosted-service/subgraph/balancer-labs/balancer-gauges

@mendesfabio @markusbkoch Thanks - makes sense. I used the # only because a few were already listed this way but probably should have used " ".  Wasn't fun but I got it done. lol.  Let me know if this is all good or needs revisions.  Thanks for working w/ me on it

@mendesfabio @markusbkoch please let me know if I can do anything to help get this PR completed

@mendesfabio @markusbkoch Any way this or https://github.com/balancer-labs/docs-developers/pull/5 would be looked at?  Please let me know if you need anything to complete.


Exclude playground files from precommit

This is not useful
/home/runner/work/cursorless/cursorless/data/playground/typescript.ts
  1:10  warning  'helloWorld' is defined but never used. Allowed unused vars must match /^_/u      @typescript-eslint/no-unused-vars
  2:9   warning  'value' is assigned a value but never used. Allowed unused vars must match /^_/u  @typescript-eslint/no-unused-vars

Yeah was thinking a global exclude may be in order. A couple of the checks are still useful (eg no big files, no symlinks, etc), but prob easiest to just have global exclude

Sounds good


Feature socialmedia

Coverage decreased (-0.9%) to 83.878% when pulling b7a38052b6f9a9e84b3c99d428f387fdf2424a4d on feature-socialmedia into 4697c96e72e032fe9b18eb39b85049cb49190961 on develop.


unit test from BYS

Hi @bey62672, could you run npm run lint and fix the errors?

@bey62672 Looks good.  Please merge.


New URLs for listing pickups

We need two URLs, one to fetch the "completed" pickups and the other one to fetch the "on going" and "Waiting for Review" pickups

We also need a way to filter this listings. Let's say to do the easy way:
pickups?skip=10&list10 could bring the 10-20 list and so on.

Yes, that would be perfect, I already have some code that manipulates that functionality in iOS

Sooooooooooo...Is it up already? =)

Hey, guys, I'm on it, but I have some doubts:
On the model there is no field to hold these "completed", "on going" and "waiting for review" statuses.
Will we create this field? If so, who will change this status, in a business point of view?
I also saw that there is no review on the model... what  "waiting for review" means?
Can you clarify these points for me?
Thanks!

Good question, I guess we can keep the field with the default value of "on going ".
This one was created to separate the pickups, all should come with "waiting for review " then a staff will change it to "om going ".

Hey great to see you here!! Let me clear out some stuff.
1- yes, those fields have to be created for the pickup, we don't have them yet.
2- the status will be changed by the user's actions. The pickup starts as "on going", it is changed to "waiting for review" after the binner says it has completed the pickup. Finally the status changes to "Completed" after the user reviews how was his experience for the pickup.
3 - waiting for review means that the user has to review the pickup for the status to be changed to completed. There should be a rating structure inside the pickup, it should contain a string for comments and a 1-10 score of how good was the pickup.
Hope I could help! If you have any other questions just ask away!

@matheusruschel ,
Just for check:

Finally the status changes to "Completed" after the user reviews how was his experience for the pickup.

Only the requester can review the pickup, right? I'm going to include this check when a review is sent, ok?

Guys, I've done this task.
Just a brief explanation:
PATCH /api/v1.0/pickups/{_id}/done
Puts a pickup as done, status is set to 'waiting review'
POST /api/v1.0/pickups/{_id}/review
Place a review for a pickup and status is set to 'completed'
GET /api/v1.0/pickups/completed
List last 6 months user's pickups completed
GET /api/v1.0/pickups/ongoing
List last 6 months user's pickups on going
GET /api/v1.0/pickups/waitingreview
List last 6 months user's pickups waiting for review
If we need some change, please, let me know.

Yes, you're correct about the requester being the only one able to review the pickup.
Awesome work man!! In this rhythm we will be over pretty soon!
On Oct 6, 2016, 2:48 PM -0300, Romulo Soeiro<EMAIL_ADDRESS>wrote:

Guys, I've done this task.
Just a brief explanation:
PATCH /api/v1.0/pickups/{_id}/done
Puts a pickup as done, status is set to 'waiting review'
POST /api/v1.0/pickups/{_id}/review
Place a review for a pickup and status is set to 'completed'
GET /api/v1.0/pickups/completed
List last 6 months user's pickups completed
GET /api/v1.0/pickups/ongoing
List last 6 months user's pickups on going
GET /api/v1.0/pickups/waitingreview
List last 6 months user's pickups waiting for review
If we need some change, please, let me know.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub (https://github.com/VanHack/binners-project-api/issues/48#issuecomment-252037564), or mute the thread (https://github.com/notifications/unsubscribe-auth/AFzLK1QLVBYcl_8AmUMK_1FdG_tmWDpTks5qxTRzgaJpZM4JhvK1).

Hey we need another url, could you build them please?
We need something to fetch ongoing pickups and wating for review pickups at the same time since they're gonna be displayed at the same page, can we get them ordered by time(most recent to oldest)?
GET /api/v1.0/pickups/waitingreview=30&ongoing=30
Can we make something like that?
Thanks you rock =)

Forgot something. We also need a url to Cancel the pickup, and 3 other urls

change the pickup time
change the pickup day
change the pickup address (includes coordinates and address)

Thanks!

We need something to fetch ongoing pickups and wating for review pickups at the same time since they're gonna be displayed at the same page, can we get them ordered by time(most recent to oldest)?

@matheusruschel can you perform two requests, like below, and order the results on the client?
GET /api/v1.0/pickups/waitingreview?limit=30
GET /api/v1.0/pickups/ongoing?limit=30
This approach let the API a little bit more generic and plain, delegating to client perform specific view operations. Of course, if you say that you really need that another approach I can do it.

Forgot something. We also need a url to Cancel the pickup, and 3 other urls

Please, use:
PATCH /api/v1.0/pickups/{_id}/canceled to change the status for canceled.

change the pickup time
change the pickup day
change the pickup address (includes coordinates and address)

And, use:
PUT /api/v1.0/pickups/{_id} to update the pickup. Observe that the model schema (refer to API doc) only allow changing the fields that you asked, ok?
I'm going to submit the PR.
Thanks by let me participate and help!! :)

Ok! Thanks @soeirosantos, you rock!

@matheusruschel, I've created the below endpoint.
Example:
/api/v1.0/pickups/status-tracking?status=ongoing&status=canceled&limit=30
Values allowed for status: ongoing, waitingreview, completed and canceled.
I think this endpoint deprecates the specifics endpoints for each status. If so, we can remove them in future.
Please, referer the API doc for details.B

Thank you man! Yes it does, sorry for wasting your time. Anyway, we're close to the end now I guess
On Oct 26, 2016, 11:44 PM -0200, Romulo Soeiro<EMAIL_ADDRESS>wrote:

@matheusruschel (https://github.com/matheusruschel), I've created the below endpoint.
Example:
/api/v1.0/pickups/status-tracking?status=ongoing&status=canceled&limit=30
Values allowed for status: ongoing, waitingreview, completed and canceled.
I think this endpoint deprecates the specifics endpoints for each status. If so, we can remove them in future.
Please, referer the API doc for details.B
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub (https://github.com/VanHack/binners-project-api/issues/48#issuecomment-256524803), or mute the thread (https://github.com/notifications/unsubscribe-auth/AFzLK9kisrL9kEuhDCCF5LYnLGjuDARSks5q4AH5gaJpZM4JhvK1).


Trouble mocking a generic method on a class

I have this class I need to mock, the method is giving me problems - how do I set up findThing<T>()?
export interface ISomething { aValue: string; }
export class SomeClass implements ISomething { aValue: string; }

// These classes are unimportant, the method mock is the goal...
export class ParentClass {
    private _obj = new SomeClass();

    findThing<T>(): Observable<T> {
        return of(this._obj as unknown as T);
    }
}

Here's the code I tried:
   const theMock = mock(ParentClass);
   when(theMock.findThing<ISomething>()).thenReturn(of(new SomeClass()));

   const theMockInstance = instance(theMock);
   const result = theMockInstance.findThing();

I get this response:
   TypeError: theMockInstance.findThing is not a function
If someone can confirm this is a problem/bug/missing feature that would be helpful.

I'm also experiencing the same issue.


do not popup autosuggest window when there is only one suggestion and editor.inlineSuggest.enabled and editor.suggest.showSuggestionPreview are both true

Requirement:
Reason:
A popup window is distracting. In this case it adds absolutely no value. It rather distracts (sometimes even forces me to press escape to dismiss it.)
This will improve my code writing flow.
I think in general, most developers 'in the flow' will like this feature.

closing as /duplicate of https://github.com/microsoft/vscode/issues/126375


Adding CSS to improve some alignment

Description
Adding CSS to improve

Alignment of Light/Dark Mode switch button with other navigation buttons.
Alignment of light and dark mode icon with the blue ball inside the Light/Dark Mode switch button

Type of change
Please delete options that are not relevant.

[x] User Interface

Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] My changes generate no new warnings

ScreenShots (if any)
Earlier:

Final:

@harkiratsm please review it.
Thanks :)


HTTP profile incorrectly added to TCP listener with SOURCE_IP persistence

@jlongstaf @zancas
What issues does this address?
Fixes #660
What's this change do?
Added logic in listener_service.py to only remove fastl4 profile if listener
is TCP and persistence does not equal SOURCE_IP.
Where should the reviewer start?
Any background context?
When adding a pool with SOURCE_IP persistence, if the associated
listener has TCP protocol, the fastL4 profile is removed and http and
oneconnect profiles are added. These profiles are not required for
source_addr persistence profiles and should not be added to the virtual
server. Note, http profiles are required for HTTP and APP_COOKIE
persistence, and are also added for listeners with HTTP, HTTPS, or
TERMINATED_HTTPS protocol types.
Added a functional test in test_session_persistence.py test module to
ensure the correct profiles are added when a pool is attached to a
listener with session persistence defined.

Yes, they will. TCP does not always equal fastl4 however. If a one connect profile is added, the virtual is then changed from fastl4 to a standard virtual.


CARDS-2435: Use the new parent pom as the base pom

I'm confused about where everything that was removed from the root pom went, please clarify.

I'm confused about where everything that was removed from the root pom went, please clarify.

Here: https://github.com/data-team-uhn/cards-parent

I'm confused about where everything that was removed from the root pom went, please clarify.

Here: https://github.com/data-team-uhn/cards-parent

What's the motivation behind having this single pom file in a separate repo from the cards platform? Won't it be more cumbersome to develop features that require adding a new dependency?


fixed multiple bugs in amcnetworks.py extractor

Please follow the guide below

You will be asked some questions, please read them carefully and answer honestly
Put an x into all the boxes [ ] relevant to your pull request (like that [x])
Use Preview tab to see how your pull request will actually look like

Before submitting a pull request make sure you have:

[x] At least skimmed through adding new extractor tutorial and youtube-dl coding conventions sections
[x] Searched the bugtracker for similar pull requests
[x] Checked the code with flake8

In order to be accepted and merged into youtube-dl each piece of code must be in public domain or released under Unlicense. Check one of the following options:

[x] I am the original author of this code and I am willing to release it under Unlicense
[ ] I am not the original author of this code but it is in public domain or released under Unlicense (provide reliable evidence)

What is the purpose of your pull request?

[x] Bug fix
[ ] Improvement
[ ] New extractor
[ ] New feature

Description of your pull request and other information
Some videos couldn't be downloaded because of Fairplay encryption. Alternate video manifests are available that don't use Fairplay.
Additional thumbnail images were added, often larger than the single 1920x1080 image identified by the previous version of the extractor.
"season_number" and "series" were being added to "title" for some reason. That behavior was removed.
"series" now gets displayed as "NA" rather than "_" when its value is empty.
To do: Add fallbacks.

Closes https://github.com/yt-dlp/yt-dlp/issues/177


Add KubeCon EU 2024 Details to Website

PR to add details about the upcoming KubeCon EU 2024 in Paris including relevant talks and links to talks. Information about a meetup is still TBD.

we can keep the events for file for Kubecon EU 2023


Advancement loading errors SF-3355

Mods (50): Minecraft, Minecraft Coder Pack, Forge Mod Loader, Minecraft Forge, SpongeAPI, SpongeForge, AI Improvements, AbyssalCraft, AbyssalCraft Integration, AppleCore, AppleSkin, Applied Energistics 2, BetterBedrockGen, Biomes O' Plenty, CoFH Core, CoFH World, CodeChicken Lib, CosmeticArmorReworked, Diet Hopper, FoamFix, FoamFixCore, InstantUnify, Inventory Tweaks, JourneyMap, Just Enough Items, Just Enough Pattern Banners, LagGoggles, LegibleNei, Mantle, Not Enough Items, NuclearCraft, Pam's Cookables, Pam's HarvestCraft, Quick Leaf Decay, Reborn Core, Redstone Flux, Roguelike Dungeons, Simply Jetpacks 2, Tech Reborn, The Betweenlands, Thermal Dynamics, Thermal Expansion, Thermal Foundation, Tinker's JEI, Tinkers' Construct, UniDict, WAIM, Waila, WanionLib, What Are We Looking At

Plugins (10): Minecraft, Minecraft Coder Pack, SpongeAPI, SpongeForge, GriefPrevention, LuckPerms, MagiBridge, Nucleus, Total Economy, UltimateChat

SpongeForge
    Minecraft: 1.12.2
    SpongeAPI: 7.1.0-SNAPSHOT-51caddf
    SpongeForge: 1.12.2-2705-7.1.0-BETA-3355
Minecraft Forge: <IP_ADDRESS>39

https://gist.github.com/XakepSDK/249e9ca265f5c6b5c1ccb069ea6d4aa2
Should be fixed by https://github.com/SpongePowered/SpongeCommon/pull/1999, but seems to be not fixed.

Looks like there is a mod removing recipes without removing their unlock advancements.

You're right. Caused by pams harvestcraft.

mod bug that happens with just forge


A question to your ContentfulApi

First off, nice pair coding with Jason on Youtube! Love the content...
I would like to know in case you have a lot of different api calls to CF, do you still put them into that one file? Or would you split them into different files and scope them a little bit more? I like the way you did it and I was doing it almost the same in a project I am a part of but my Lead Dev was asking me if we scale, if this would be good to put all those calls into a single file?! I think it depends... centralising all API Calls to CF in a single file kinda makes sense! What do you think?

Hi @gutisalex!
Thanks for your message.
I usually split the API calls out, actually, by content type. You can see how I do it in my current personal website here. Notice how each file is named after the content type it is referencing:

In this starter project I didn't want to be too opinionated, but when files get too big, I usually recommend splitting out the code into a system that makes sense for the project and the developers working on it.
Hope that helps!
(and I really need to update the dependencies on this project, so thanks for the reminder!)


src/debug.h: incorrect FSF address

Would be great to get header with license updated in that file.

Should be fixed now. Thanks!


15 (not expected 16) plumedSerpent audio files

@davidjamesbeck just did a fresh pull on the master branch - you have been busy!
There's a small discrepancy in test_AudioExtractor.py:
--- test_extract_plumedSerpent
Traceback (most recent call last):
  File "test_AudioExtractor.py", line 114, in <module>
    runTests()
  File "test_AudioExtractor.py", line 17, in runTests
    test_extract_plumedSerpent()
  File "test_AudioExtractor.py", line 110, in test_extract_plumedSerpent
    assert(len(fileList) == 16)

For what it's worth, I see 15 .wav files in that directory.

There are 16 in the directory in my repo because I ran webapp2 on that data and it copies the full recording into the audio folder. I think I added the /audio/ directories to .gitignore while I was working on some very large sound files and had slower bandwidth this summer in Duncan. Those files are not in use anymore, so we can either remove the /audio/ folders from .gitignore or manually add the file. I guess if the GitHub repo is going to be accessed, we might think about removing the audio files—they aren’t all public, I’ve just been using them because they are handy on my computer.
David

On Nov 17, 2019, at 9:46 AM, Paul Shannon<EMAIL_ADDRESS>wrote:
@davidjamesbeck https://github.com/davidjamesbeck just did a fresh pull on the master branch - you have been busy!
There's a small discrepancy in test_AudioExtractor.py:
--- test_extract_plumedSerpent
Traceback (most recent call last):
File "test_AudioExtractor.py", line 114, in
runTests()
File "test_AudioExtractor.py", line 17, in runTests
test_extract_plumedSerpent()
File "test_AudioExtractor.py", line 110, in test_extract_plumedSerpent
assert(len(fileList) == 16)
For what it's worth, I see 15 .wav files in that directory.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/paul-shannon/slexil/issues/52?email_source=notifications&email_token=AKN4HBS6GYKKK6QPYQALIRLQUFYOXA5CNFSM4JOKVKTKYY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4HZ3ZDMA, or unsubscribe https://github.com/notifications/unsubscribe-auth/AKN4HBSTCQEY3HQ7PONNPMLQUFYOXANCNFSM4JOKVKTA.

Thanks, @davidjamesbeck.  Any way for me to help?  I pretty often do this myself - that is, see my tests pass on my local machine, and then a colleague reports they fail after a pull.
After I get all of your updates I want to propose two very small additions to the standard html output file:

A usually empty div, “annotation” added to the end of each line, after the translation element.
My ad hoc annotation-popup javascript can write into that div.  Being normally empty and hidden,
I don’t think it will get in the way.

An html comment in the header - also invisible. This provides a bit of text for me to replace
with a few script and style tags which I need to create an annotated version of a page.

It could look like this:

And then could be programmatically transformed, out of a makefile, to

What do you think?

Paul

On Nov 18, 2019, at 6:49 AM, David Beck<EMAIL_ADDRESS>wrote:
There are 16 in the directory in my repo because I ran webapp2 on that data and it copies the full recording into the audio folder. I think I added the /audio/ directories to .gitignore while I was working on some very large sound files and had slower bandwidth this summer in Duncan. Those files are not in use anymore, so we can either remove the /audio/ folders from .gitignore or manually add the file. I guess if the GitHub repo is going to be accessed, we might think about removing the audio files—they aren’t all public, I’ve just been using them because they are handy on my computer.
David

On Nov 17, 2019, at 9:46 AM, Paul Shannon<EMAIL_ADDRESS>wrote:
@davidjamesbeck https://github.com/davidjamesbeck just did a fresh pull on the master branch - you have been busy!
There's a small discrepancy in test_AudioExtractor.py:
--- test_extract_plumedSerpent
Traceback (most recent call last):
File "test_AudioExtractor.py", line 114, in
runTests()
File "test_AudioExtractor.py", line 17, in runTests
test_extract_plumedSerpent()
File "test_AudioExtractor.py", line 110, in test_extract_plumedSerpent
assert(len(fileList) == 16)
For what it's worth, I see 15 .wav files in that directory.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/paul-shannon/slexil/issues/52?email_source=notifications&email_token=AKN4HBS6GYKKK6QPYQALIRLQUFYOXA5CNFSM4JOKVKTKYY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4HZ3ZDMA, or unsubscribe https://github.com/notifications/unsubscribe-auth/AKN4HBSTCQEY3HQ7PONNPMLQUFYOXANCNFSM4JOKVKTA.

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

Hi, Paul
Why don’t I try the easy way for the moment and make sure that git is updating the /audio/ folders. At some point we’ll have to fix the tests not to use other people’s data, or at least not other people’s recordings. I can probably just use one recording of length >= the longest current .wav file we use for all the tests. They will pass even if the sounds we hear don’t match the transcriptions (we’re beyond worrying about that level of undetected error, I think). Unless, we can protect some parts of the GitHub repository? Then we can let people see the code but not the tests and testData?
I like both of those tweaks. Adding a findable annotation element to each line has lots of practical uses for others as well. The add-on to the header is harmless enough, and someone might find it useful someday.
Cheers,
David

On Nov 18, 2019, at 8:31 AM, Paul Shannon<EMAIL_ADDRESS>wrote:
Thanks, @davidjamesbeck. Any way for me to help? I pretty often do this myself - that is, see my tests pass on my local machine, and then a colleague reports they fail after a pull.
After I get all of your updates I want to propose two very small additions to the standard html output file:

A usually empty div, “annotation” added to the end of each line, after the translation element.
My ad hoc annotation-popup javascript can write into that div. Being normally empty and hidden,
I don’t think it will get in the way.

An html comment in the header - also invisible. This provides a bit of text for me to replace
with a few script and style tags which I need to create an annotated version of a page.

It could look like this:

And then could be programmatically transformed, out of a makefile, to

What do you think?

Paul

On Nov 18, 2019, at 6:49 AM, David Beck<EMAIL_ADDRESS>wrote:
There are 16 in the directory in my repo because I ran webapp2 on that data and it copies the full recording into the audio folder. I think I added the /audio/ directories to .gitignore while I was working on some very large sound files and had slower bandwidth this summer in Duncan. Those files are not in use anymore, so we can either remove the /audio/ folders from .gitignore or manually add the file. I guess if the GitHub repo is going to be accessed, we might think about removing the audio files—they aren’t all public, I’ve just been using them because they are handy on my computer.
David

On Nov 17, 2019, at 9:46 AM, Paul Shannon<EMAIL_ADDRESS>wrote:
@davidjamesbeck https://github.com/davidjamesbeck just did a fresh pull on the master branch - you have been busy!
There's a small discrepancy in test_AudioExtractor.py:
--- test_extract_plumedSerpent
Traceback (most recent call last):
File "test_AudioExtractor.py", line 114, in
runTests()
File "test_AudioExtractor.py", line 17, in runTests
test_extract_plumedSerpent()
File "test_AudioExtractor.py", line 110, in test_extract_plumedSerpent
assert(len(fileList) == 16)
For what it's worth, I see 15 .wav files in that directory.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/paul-shannon/slexil/issues/52?email_source=notifications&email_token=AKN4HBS6GYKKK6QPYQALIRLQUFYOXA5CNFSM4JOKVKTKYY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4HZ3ZDMA, or unsubscribe https://github.com/notifications/unsubscribe-auth/AKN4HBSTCQEY3HQ7PONNPMLQUFYOXANCNFSM4JOKVKTA.

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/paul-shannon/slexil/issues/52?email_source=notifications&email_token=AKN4HBUDXDO44PB66AMCVRTQUKYMDA5CNFSM4JOKVKTKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEEK2OLI#issuecomment-555067181, or unsubscribe https://github.com/notifications/unsubscribe-auth/AKN4HBQFSLVWHLBM3CWUUCDQUKYMDANCNFSM4JOKVKTA.


Add Bootstrap role to the README

And some other minor improvements.
Signed-off-by: Ricardo Marques<EMAIL_ADDRESS>

you're switched the order of things in the png! That was irritating!

you're switched the order of things in the png! That was irritating!
I saw the diagram as an architectural thing, where the bases (ceph-salt) are below the things running on them (cephadm) instead of a flow-chart.

@sebastian-philipp ceph-salt is now on the bottom again.


SegmentControl gets Disposed in TabbedPage

Hello,
first of all, Great Plugin!
I have an issue on Android with the newest Plugin version 2.0.0. and Xamarin Version <IP_ADDRESS>4934.
I'm using a TabbedPage and in this TabbedPage is a Tab with a SegmentControl. When navigating
the tabs, everything works fine, but when i'm rotating the device to use it in Landscape i get
Unhandled Exception:
System.ObjectDisposedException: Cannot access a disposed object.
Object name: 'Plugin.Segmented.Control.Droid.SegmentedControlRenderer'.
It throws at OnSizeAllocated, see Exception Output:
https://pastebin.com/HMnpGepD

Thank you.
Should be fixed in version 2.0.1 that is tested and build for Xamarin.Forms Version <IP_ADDRESS>7436.
Could you please confirm that this solves your issue?

No, sorry still persists.

I'm unable to reproduce. Does this only happen when SegmentControl is on the TappedPage?

Yes, only on the TabbedPage and only on Android. On iOS it works perfectly, maybe because on iOS
the Pages on the TabbedPage get cached.

Thank you

Seems others are having similar challenges with a different control: Object Disposed Exception. Crash. Android only

I just upgraded to Xamarin Forms 3.0. You may want to test that version instead.

Looks like the same problem, it also crashes in OnLayout. I will try it with Forms 3.0.

There was a bug for Android in NuGet v3.0.0. Please test with NuGet v3.0.1.

Bug still persists with Xamarin.Forms 3.0 and NuGet 3.0.1. Tomorrow i can try to fix it.

I tested it with my Project and found the Source of the Problem. The problem is that
the SegmentControl gets disposed before the NativveControl is disposed, hence it can't dispose a disposed object.
I can't test it anymore because i'm on vacation for the next week, but take a look at the Dispose method. For me it works if
base.Dispose(disposing) gets called before the nativeControl gets Disposed.

That could be it.
I've added a version that checked for null before trying to dispose, which I should probably have done from the beginning anyhow.
Let me know if NuGet 3.0.2 fixes the issue.

Sorry for the late reply.
NuGet 3.0.2 still craches when layouting, just tested it.
I fixed it by calling base.Dispose(disposing) first in the SegmentControlRenderer class.
        protected override void Dispose(bool disposing)
        {

            try
            {
                base.Dispose(disposing);
            }
            catch (Exception)
            {
                return;
            }
            if (_nativeControl != null)
            {
                _nativeControl.Dispose();
                _nativeControl = null;
                _v = null;
            }

            if (Element != null)
                Element.SizeChanged -= Element_SizeChanged;
        }

Thank you.
I tried that too. Unfortunately, changing the order will result in an Exception when removing the control.

System.ObjectDisposedException: Cannot access a disposed object. Object name: Android.Widget.RadioGroup.

Could you try if this works:
`        protected override void Dispose(bool disposing)
{
try
{
if (_nativeControl != null)
{
_nativeControl.CheckedChange -= NativeControl_ValueChanged;
_nativeControl?.Dispose();
_nativeControl = null;
_nativeRadioButtonControl = null;
}
            if (Element != null)
                Element.SizeChanged -= Element_SizeChanged;
        }
        catch (ObjectDisposedException)
        {
        }

        try
        {
            base.Dispose(disposing);
        }
        catch (Exception)
        {
            return;
        }
    }`

unfortunatley no, _nativeRadioButtonControl doesn't exist in my Project?

No problem, but the App is still crashing in OnLayout.
If i comment this line out though:
//_nativeControl?.Dispose();
it isn't crashing anymore. But i guess i can't remove it ?

My concern is that omitting this line will leave the RadioGroup dangling, hence creating a potential memory leak in some scenarios.
It is really wierd that the try/catch doesn't catch the attempt to dispose an object that have already been disposed. It is also wierd that this only seems to be an issue with Android.
I'm running out of ideas, but please let me know if you come up with a new idea or a solution that does not introduce some other issue or side-effect.

I did some playing around with this bug and would be curious if this fixes your issue.
Calling base.Dispose disposes the _nativeControl. Set it to null after calling base.Dispose so it possibly can be GC'ed sooner.
See this discussion from stackoverflow.com
protected override void Dispose(bool disposing)
{
    if (_nativeControl != null)
    {
        _nativeControl.CheckedChange -= NativeControl_ValueChanged;
    }
    if (_v != null)
    {
        _v.Dispose();
        _v = null;
    }

    if (Element != null)
        Element.SizeChanged -= Element_SizeChanged;

    base.Dispose(disposing);
    _nativeControl = null;
}

Great work @rjantz3!
I will accept PR #25 and release an updated NuGet.
@bluekuen could you please test if this fixes your issue, as I'm not able to reproduce the issue you've reported.

Really Great work @rjantz3 !
I just tested and it really did fix the issue !
Thank you for your help @rjantz3 and @1iveowl !

@1iveowl I did not include this code in my PR #25.


Show data label below the chart and include Legend Keys

Chart.js can do like this?

Show data label below the chart and include Legend Keys or any alternative solution that can tell which label is of bar chart?
thank you

No, this is not currently supported. The best way to do it right now would be to generate an HTML table yourself outside of the chart.
The best way to add support for this if you wanted to do it in a general reusable way for all charts would be to create a plugin


Add the previous version of Dockerfile back for A100 & T4

Fix issue30

thx for the reminder. instructions added in README file. please check it out. @chmstimoteo


Using fillField with values other than strings

Using fillField() with values other than strings, e.g: ints, leads to pretty hard to trace error, I think the bundle should either implicitly handle that or throw a more informative Exception.

@thedamnedrhino Can you put a simple code to reproduce and the output you get?.

Yes, I'll do that in the coming days. From what I remember phantomjs tries to call substr on the passed value, but values of a type other than string don't implement that.

I think i know where the problem is, i have to fix the setValue function of the Mink Driver interface is causing problem in other cases not only in yours.

Just had this happen with ints. The code currently converts bool to string, see https://github.com/jcalderonzumba/MinkPhantomJSDriver/blob/9407a5c0504edf6efd84587aaacadd002e80f48e/src/FormManipulationTrait.php#L34

@mglaman I have reviewed your code, looks ok but i made only one small change request

Closing as it was solved in PR merged.


JLink start command changes current directory

The program should run from the directory of the user, not from the JRI directory.
When I start the program, for example in /home/me, I expect the user.dir to be /home/me, but it changes to the JRI directory, such as /home/me/my-app-jri.
This causes issues with any use of relative directories in the application.

@tomas-langer As I thought, I did do this deliberately: it is a requirement to make CDS work.
The CDS archive is a cache, so the JVM has to validate it against the classpath. Unfortunately, it is not very flexible when it comes to file system paths, the best you can do is ensure that they are relative to the JRI directory, hence the cd in the start script. Without that, CDS will fail:
        Error occurred during initialization of VM
        An error has occurred while processing the shared archive file.
        Unable to use shared archive.
        shared class paths mismatch (hint: enable -Xlog:class+path=info to diagnose the failure)

We can avoid the cd when CDS is disabled, though this could be a confusing/problematic difference in behavior. I will experiment with this approach.


Add path groups

Initial implementation of #26 as discussed in #35.
After thinking about how a standard node and relationship are returned, this initial implementation mirrors that for paths.
For example:
MATCH (A)-[R]->(B)
RETURN A, R, B

Returns:
(A node id, R relationship attributes, B node id)

With this change:
MATCH P = (A)-[R]->(B)
RETURN P

Returns:
[node id1, {relationship attributes} node id2]
The plumbing is now in place to return the data in whatever format we think makes sense.
I think I modified the lark spec to make the path_clause optional. The tests pass.

This is awesome!! Ready to merge?

Ready from my end.
I was hoping to get this into a new grandcypher release and require that version or greater when using it with txtai.
This feature is going to enable this functionality: https://github.com/neuml/txtai/issues/540

Groovy — will merge now!

Sure. I can push another commit with a change log update for 0.6.0. I'll also update setup.py.

Just pushed the updated CHANGELOG and bumped version in setup.py


light latex knight (PLAYER FORM) transforming into centaur crashes the game

latest.log
when attempting to transform into a light latex centaur as a light latex knight via killing EG a villager the game will crash
has been tested with no other mods

extra note the game does actualy save upon crashing so re-entering the world will show the result of the centaur form


Support for running Powershell on top of mono

Add support to run powershell with mono, instead of .net core. Mono can run in a broader set of scenarios and platforms, net core doesnt not allow to run powershell on old systems like Centos6 per example.
As mono and .net core has common APIs and both will support netstandard 2.0 it should make powershell independant of running .net core or mono.

If anyone decides to try this, be aware that I am aware of at least one feature that Mono provides an api for, but it silently does nothing.
PowerShell classes rely on dynamic assemblies that can be garbage collected. When I last investigated, Mono had the appropriate api (so PowerShell could run), but the implementation never collected the assemblies, so using PowerShell classes could result in a permanent memory leak.

Thanks @lzybkr! I was talking in the mono gitter channel about this and they told me to open a bug with the APIs that are not correctly implemented and they will review. In the last versions they have have replaced lot of their original code/api by the open-source microsoft .net references code so it is getting better and better.
Could you provide more details about bad implemented APIs and problems to file a bug for mono? or steps on how can I can test/found thems?

I believe this line was the problem, and more specifically, RunAndCollect did not enable garbage collection.

@lzybkr thanks, i was discussing this on mono channel, and it is right and it can take a them a while to implement but as it is just a memory leak, for tasks like running a script it is not a big problem as running from bash "mono powershell.exe myscript.ps1" it will free all memory once the script is finish. I guess this could be more a problem if you open a powershell.exe shell for a long time or script that are running for days.
Are there another issues? If there arent, how could i have a try at this and try run some powershell code on mono?
I have opened the bug for mono here:
https://bugzilla.xamarin.com/show_bug.cgi?id=46569

any update on this? can anyone provide a package that i could try to run on mono and find issues/leaks?

I'm not aware of anyone working on this, and the issue is Up-for-Grabs, so if you want to see it happen, maybe you can contribute.
I will say that at some point in the next month or two, I think we'll be moving from project.json build files to msbuild now that dotnet supports it, so it might be better to wait for that work to finish before starting on mono support.

Recently I took a look at the source code, the libraries still compiles and works if re-targeted to netstandard2.0 (Modify PowerShell.Common.props while keep powershell program itself netcoreapp2.0)
One problem is System.Management.Automation relies heavily on AssemblyLoadContext (mostly in ClrFacade and CorePsAssemblyLoadContext), which is not available in any versions of .NET Standard or .NET Framework. System.Runtime.Loader package reference has to be added manually to get System.Management.Automation compiled under netstandard2.0, but this package is just a stub so it will fail at runtime if mono/.net framework tries to load it.
Luckily there is AppDomain in .NET Standard 2.0 which may provide sufficient API to replace AssemblyLoadContext, but rewriting these classes is certainly required and I don't know if PowerShell Core team will do this or not. (Didn't see any plan to make System.Management.Automation a netstandard2.0 library in https://github.com/PowerShell/PowerShell/issues/3046)

I guess it is as simple with .Net 5.0 as creating new package.
/cc @TravisEz13 @SteveL-MSFT

This should be much easier now Mono now implements the CLR memory model: https://github.com/mono/mono/pull/17136

I guess it is as simple with .NET 5.0 as creating new package.

Like that is simple. 🤣

https://github.com/Pash-Project/Pash
We can revive this project

@iSazonov Please could you reopen this issue.


RegisterAndNotify limit 20 bytes on Android

Please check all of the platforms you are having the issue on (if platform is not listed, it is not supported)

[] iOS
[x] Android
[] tvOS
[] macOS
[] .NET Standard
UWP IS NOT SUPPORTED DURING THE BETA PERIOD - ONLY PRs ARE WELCOME FOR NOW

Version of OS(s) listed above with issue
7.1.1
Version of Library
5.2.2
Expected Behaviour
RegisterAndNotify contains full data buffer of more than 20 bytes.
Basically when i step through the RegisterAndNotify, the data is 20 bytes, when on iOS it is limited to 158 bytes.  My data that I have incoming is slightly more than 20 bytes but under the 158 byte limit.  I am just trying to figure out how I can make it either send more bytes or make it so that I can read the next set of bytes within that same notification?
Actual Behavior
RegisterAndNotify limits to 20 bytes rather than constructing buffer of more than 20 bytes.  I'd like to know how I can structure to read the next 20 bytes if needed.
Code of Conduct

No Library Update Requests
No Timeline Requests
Please read through the documentation and FAQs
Please be respectful of my time. I put a lot of effort into my libraries.  I try to support the issue requests within a day.  I'm not psychic, so please take the time to fill out the above form as complete as possible

This is a general limit of BLE for the most part.  You can use the adapter.RequestMtu which can give you up to 512 bytes on a limited set of devices.  You'll need to read up on the devices/OS versions that will allow you to do this.  I'm afraid that question goes beyond this library though.
Best of luck.

I guess basically what i'll need to do is not use notify unless to let the device know that there it needs to do a read or something.  This is what I feared.
Also I cannot upgrade to 5.3.1 was there a reason for removing ScanListen?

Because it was painful to maintain so it won't back.

Okay thanks that makes sense, so is there a way to read the next set of bytes on notify or will I need to do some sort of notify where it reaches back and read the set of data I need to read up to a certain amount?
Something like:
Incoming Notify -> Total Bytes = 1294 bytes, do a read based on mtu and then keep reading until you have all the buffer?  Or is there something like
notify's data = 20 bytes.  Read the next 20 bytes until you have reached all the buffer of data?  Probably not going to work that way with the current library setup?

Nevermind I see what your saying about RequestMtu.  Thanks!

So I attempted to change the Mtu and it worked however it still only brought back 20 bytes of data.
https://socialdrip.s3.amazonaws.com/ble.png

As I said, you'll need to read up on the device & OS support!  It isn't a guarantee that you can go beyond 20 bytes.  The sending device may only be sending 20 bytes at a time anyhow.  iOS sometimes does internal buffering to make you believe it at a larger MTU.

Gotcha, just what I feared.  Thanks


After login on whatsapp gets an error 'Unable to locate element'

Hi, I install the script on my MacOs  High Sierra, when I run the script and scan the qr code, when I see the chat list press 'y' at the 'question phone connected?' at the console and gets the error:
'Message: no such element: Unable to locate element: {"method":"xpath","selector":"//*[@id="main"]/header//div[contains(@class, "chat-main")]"}
(Session info: chrome=64.0.3282.140)
(Driver info: chromedriver=2.35.528157 (4429ca2590d6988c0745c24c8858745aaaec01ef),platform=Mac OS X 10.13.3 x86_64)
    Chromedriver Error. Read the above error (if any), then
    Check if installed chromedriver version is compatible with installed Chrome vesion.'

Any ideas?
Thank you

+1, similar issue here in Ubuntu 16.04. As Firefox has released new versions, do you plan to release a new version compatible with Firefox?

+1, Same issue

This issue isn't Chromedriver related, it's due to some code changes in the webversion of WhatsApp itself. I'll have a patch ready today. I must add that I am a brand new Github user, so am I supposed to create a fork if I want to patch a project of which I am not the author?

Unfortunately in the same boat - new user myself.
I assumed that it was due to DOM changes as I was trying out the xpath
selectors manually and obviously they had no match.
I was in the process of manually specifying them(to which end I managed to
send messages but not receive them - but I was just fiddling during down
time, plus I'm not familiar with python).
Sorry I can't be of any further help! Best of luck :)
On 22 Feb 2018 15:40, "PiPauwels"<EMAIL_ADDRESS>wrote:
This issue isn't Chromedriver related, it's due to some code changes in the
webversion of WhatsApp itself. I'll have a patch ready today. I must add
that I am a brand new Github user, so am I supposed to create a fork if I
want to patch a project of which I am not the author?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/zvovov/whatsapp-web/issues/13#issuecomment-367720769,
or mute the thread
https://github.com/notifications/unsubscribe-auth/Ad44ijhfAdVjCgOrIfLW0AtTTZymZpgHks5tXYpVgaJpZM4R_wVp
.

Update: the most recent version of my fork is now able to receive messages, as well as send them: https://github.com/PiPauwels/whatsapp-web
Since the receiving mechanism was based on an attribute "data-id", which has been omitted during recent DOM-changes, the new mechanism is now based on the name of te sender and the contents of the message (if both differ from the previous last printed message, new messages are printed to the screen). I've also incorporated the timestamp and sender name present in the current DOM, which is -IMHO- more useful than printing the group name in front of every message.
There remains one problem, which has something to do with another reported issue: https://github.com/zvovov/whatsapp-web/issues/4. At random times (not yet sure what causes the exception) that same exception is thrown, after which receiving messages is nog longer possible, but sending them keeps working. I will investigate this up next.
Planning on incorporating images & emoji once I get that bug out of the way.

Should be fixed now.
@PiPauwels brilliant work!


authenticator.xml problem

Your Environment

Plugin version: latest
Platform: Android
Device OS version: 11
Device manufacturer / model: Motorola
XCode version:
Capacitor info (npx cap doctor)
Plugin config provided to #ready:

INSERT_YOUR_CODE_HERE

Expected Behavior
The project build in Android Studio should be successful.
Actual Behavior
The project build in Android Sudio fails with this error:
ERROR: app/android/capacitor-cordova-android-plugins/build/intermediates/packaged_res/debug/xml/authenticator.xml:7: AAPT: error: resource string/plugin_bgloc_account_type (aka io.ionic.starter:string/plugin_bgloc_account_type) not found.
I have the exact same project on another machine thats builds without errors.
On this machine the above mentioned authenticator.xml file doesnt even exist.
Steps to Reproduce

Build project with  capacitor-background-geolocation plugin in Android Studio

Context

Debug logs

Logs
PASTE_YOUR_LOGS_HERE

Show me your package.json.

package.json:
{
  "name": "app",
  "version": "0.0.1",
  "author": "app",
  "homepage": "app",
  "scripts": {
    "ng": "ng",
    "start": "ng serve",
    "build": "ng build",
    "test": "ng test",
    "lint": "ng lint",
    "e2e": "ng e2e"
  },
  "private": true,
  "dependencies": {
    "@angular/common": "~12.1.1",
    "@angular/core": "~12.1.1",
    "@angular/forms": "~12.1.1",
    "@angular/platform-browser": "~12.1.1",
    "@angular/platform-browser-dynamic": "~12.1.1",
    "@angular/router": "~12.1.1",
    "@capacitor/android": "3.2.2",
    "@capacitor/app": "1.0.3",
    "@capacitor/core": "3.2.2",
    "@capacitor/haptics": "1.0.3",
    "@capacitor/ios": "3.2.4",
    "@capacitor/keyboard": "1.0.3",
    "@capacitor/status-bar": "1.0.3",
    "@ionic-native/geolocation": "^5.36.0",
    "@ionic/angular": "^5.5.2",
    "@transistorsoft/capacitor-background-fetch": "^0.0.6",
    "@transistorsoft/capacitor-background-geolocation": "^4.3.0",
    "leaflet": "^1.7.1",
    "rxjs": "~6.6.0",
    "tslib": "^2.2.0",
    "zone.js": "~0.11.4"
  },
  "devDependencies": {
    "@angular-devkit/build-angular": "~12.1.1",
    "@angular-eslint/builder": "~12.0.0",
    "@angular-eslint/eslint-plugin": "~12.0.0",
    "@angular-eslint/eslint-plugin-template": "~12.0.0",
    "@angular-eslint/template-parser": "~12.0.0",
    "@angular/cli": "~12.1.1",
    "@angular/compiler": "~12.1.1",
    "@angular/compiler-cli": "~12.1.1",
    "@angular/language-service": "~12.0.1",
    "@capacitor/cli": "3.2.2",
    "@ionic/angular-toolkit": "^4.0.0",
    "@types/jasmine": "~3.6.0",
    "@types/jasminewd2": "~2.0.3",
    "@types/node": "^12.11.1",
    "@typescript-eslint/eslint-plugin": "4.16.1",
    "@typescript-eslint/parser": "4.16.1",
    "eslint": "^7.6.0",
    "eslint-plugin-import": "2.22.1",
    "eslint-plugin-jsdoc": "30.7.6",
    "eslint-plugin-prefer-arrow": "1.2.2",
    "jasmine-core": "~3.8.0",
    "jasmine-spec-reporter": "~5.0.0",
    "karma": "~6.3.2",
    "karma-chrome-launcher": "~3.1.0",
    "karma-coverage": "~2.0.3",
    "karma-coverage-istanbul-reporter": "~3.0.2",
    "karma-jasmine": "~4.0.0",
    "karma-jasmine-html-reporter": "^1.5.0",
    "protractor": "~7.0.0",
    "ts-node": "~8.3.0",
    "typescript": "~4.2.4"
  },
  "description": "An Ionic project"
}

My plugins do not contain any reference to an "Authenticator.xml*.
But Mauron85's old plugin does
You've got something installed from Mauron85 causing that problem.


feat(sozo): add support for v3 transactions

not necessarily blocked, but katana is yet to fully support v3 transactions due to no STRK token declared ref #2405

@lambda-0x since @kariy merged the STRK token, when core will be merged let's rebase this one to unlock v3.
As @kariy mentioned if by default we could have sozo choosing the token that has balance, it could be great for UX. But we should keep a simple way with a flag to actually chose the token to use.

@lambda-0x if you still want to tackle this, with the rc.0 should be much simpler for the invoker. Just about how the token fee is carried by the transaction config.

Simplified in #2650.


SOF multicore: fix multicore booting on cAVS 1.8

cAVS 1.8 was missing two crucial macro definitions in its platform.h header, which prevented secondary cores from booting. Besides ARCH_HAS_COHERENCE has to be selected for all cAVS architectures, it was only present in 1.5 so far.

Concerning the CI failures: one of them I don't understand, the other one should be fixed now


Mixed licensing

Hi,
Looking over the src folder I noticed that the code for Android and iOS is licensed as GPL2, however the LICENSE file in the root is MIT. Is that intentional?
Regards
Phil

No, it's not, thanks for pointing that out. At an early stage I considered GPL but stuck with MIT and didn't update the license headers. I'll have this updated in no time, thanks!


prepend of www in URL seems to break auth

Load a full URL like: http://www.arenaxr.org including the www. and then if you go to login or directly to a page it internally has a 400 error.  It seems like without the www the same flows / pages are ok.

Hm, I have questions. I'll check in the morning.

For now, we've added redirection from www.arenaxr.org to arenaxr.org in nginx.


How to finetune a pretrained model with modified units file

How to finetune a pretrained model with modified units file ?

Reset output layers related to the number of units, such as CTC and decoder output, and keep the left unchanged.


braille dump instead of xxxxxxxx?

"Braille dump" is a way of representing non-alphanumeric characters using distinct Unicode symbols. See this for example and rationale: https://justine.lol/braille/
I'm not able to provide a pull request at the moment, but if you're interested in implementing this feature, you're welcome to copy the charset from my JS implementation: https://github.com/hackbg/toolbox/blob/main/dump/dump.ts#L10 :slightly_smiling_face:

That does look cool, and I am curious about it 🧐.
I don't think I would make it a default, but we can probably add a command switch for it

I have just added a braille mode, which can be activated with -b or --braille.
Your JS implementation was very useful. Thanks ! 🙂
I have just changed a 2 characters that were confusing as they were very similar to the cursor.
It's on the main branch if you want to try it.
It will be in the 0.8 release, but there is quite a bit of work I plan to do before the next release.

I have also added -B as an option to do braille dump for values 0x80 and above.
I am pondering making it the default, since it does give a bit more information, and isn't too distracting like the emojis of the values below 0x10.


Improve support for searching with relative paths in a monorepo

For very large monorepos, it's not feasible to give VS Code the monorepo root (e.g. checkout/mycompanyname) since it's infeasible to have VS Code search the entire monorepo - it's too big.
So instead, folks create workspaces containing multiple relatively small folders inside the monorepo (e.g. checkout/mycompanyname/mobile/iphone/MyProject and checkout/mycompanyname/mobile/iphone/shared/SharedFramework). This allows VS Code to easily find files inside of these folders since VS Code will index the small folders without issue, although I've opened https://github.com/microsoft/vscode/pull/150450 to improve this.
However, a problem occurs now when a user tries to jump to a file using a relative path from the base of the monorepo, e.g. mobile/iphone/shared/UIKitAdditions/UIView+Accessibility.swift since it doesn't exist in the VS Code project.  VS Code only includes relative path results from the workspace folders here, but this doesn't work for this monorepo relative path since VS Code doesn't know about the monorepo root.
Ideally, we can provide VS Code with a path to the monorepo root (checkout/mycompanyname in this example), and it can use this to resolve relative paths above without needing to index the entire monorepo. Additionally, it might be useful to use the parent dir name (checkout in this example) as the folder name for the monorepo in case people have multiple named checkouts. This would ideally simplify the rendered item path in the UI (e.g. checkout/relative/path vs. checkout/mycompanyname/relative/path vs. /some/path/to/the/checkout/mycompanyname/relative/path), just like is done for files in workspace folders.

This is related to https://github.com/microsoft/vscode/issues/73524 as well I think. If that API was available then it might be possible to work around this in an extension.

We've discussed how to make this scenario better. A lot of people use multiroot workspaces for working on a subset of a monorepo. Probably instead of a multiroot workspace you want to open the monorepo with only a subset of folders included, and the rest ignored. You can sort of do this with files.exclude but here is an issue for something better: https://github.com/microsoft/vscode/issues/41860. I think that is the right solution, not searching paths outside of the workspace folders in a multiroot workspace, I will close this as a duplicate of that issue


[FEATURE] - Added bills download to mintapi cli

Added a constants key for mint bills, a cli option --bills to download bill data from mint, and a call to mint.get_bills if that option is set.

Nice, great addition, thanks! Was this manually tested/verified?

You bet, and thanks for maintaining this!  I have not tested it extensively beyond my own mint account, but the cli does return the correct array of bills for my account.

Looks like the lint check had failed, possibly due to line endings or whitespace.  I didn't have the action enabled on my fork.  I re-formatted with black and re-committed to this pull request if you'd like to re-run the checks on this end.

Thanks! For some reason GH shows the whole cli.py file as being rewritten. Is it possible spacing was changed, or changed to tabs, or similar?

Looks like black changed all the line endings from unix to windows, but somehow only on that file!  I reverted to fix the line endings, and re-committed.  The linter also broke up a few help message string constants, but I left those in case that's preferrable.

Sure thing.  I agree - I prefer not to have extraneous edits in a commit.  On my local repository, the black action now fails this commit because it still wants to break up those strings, but I'd assume it would continue to pass here.

Thanks @BillJr99 ! Perhaps we need to have a contributor guide that specifies the version/arguments for black so it all aligns. Sorry for the troubles!


DEV 8587.9 Reduce the time for which Admiral checks whether Secrets service is up from 180 to 30 seconds, if Admiral is installing Secrets on the same machine as Admiral.

https://github.com/Shippable/pm/issues/8587

this is done, closing


initial tests for LaserPulse class

See example code here,
https://github.com/radiasoft/rslaser/blob/master/examples/notebooks/diagnostics/TransverseEnvelope.ipynb

instantiate LaserPulse instance with zero chirp (i.e. d_lambda = 0), central wavelength lambda0=1e-6 meters and with 3 slices;  query the wavelength (i.e. _lambda0) associated with each slice; it should be the same as for the pulse as a whole
do the same with d_lambda = lambda0/10; in this case, each slice should have a different wavelength, but probably won't because of a bug

After a couple small changes to the LaserPulse class, the first test passes. However, the second test fails. Is each slice's lambda0 supposed to be incremented or decremented by d_lambda during instantiation?

The code snippet above will result in each LaserPulseSlice instance having a photon energy that is inconsistent with the specified wavelength, that's getting passed in via the **_k argument.
I think all default values should be removed from the LaserPulseSlice constructor and the **_k argument should not be passed through from the LaserPulse() constructor.
This will result in the need for more syntax regarding management of arguments.

Perhaps a couple of tests for LaserPulseSlice() should be implemented before resolving the present issue.
Also, the PKDict object could be required to take photon energy (aka phE) rather than wavelength (aka lambda0).
Conversion back and forth between phE and lambda0 should be a shared function.

Given our agreement to use phE as the fundamental quantity and lambda as the derived quantity (which should not be added to the PKDict parameters), the d_lambda variable should also be abandoned and users should be required to specify _chirp.


The list of nodes in the streaming graph UI should be left aligned.

https://github.com/risingwavelabs/risingwave/blame/c8cdb9f59ff12554085d4261d4d2e6a617854196/dashboard/pages/streaming_graph.tsx#L100

instead of center aligned(current)

Thanks for your feedback. #14356 should already fix that!


Grid Proxy Client missing rentStatus in the returned details of node.

There should be parameter rentStatus to show if its free, yours, or taken to specify the button in the didicated nodes .
As in old dashboard

It should be returned here for every node

I found solution for this condition is to check the rentedByTwinId with my twin Id and it's working fine as the old dashboard.


Testing with React-Intl

Hi. I'm trying to use snapshot testing for components that use React-Intl (https://github.com/yahoo/react-intl). I tried following a similar approach to how we'd set it up using Enzyme (https://github.com/yahoo/react-intl/wiki/Testing-with-React-Intl#enzyme):
import React from 'react';
jest.mock('react-dom');
import renderer from 'react-test-renderer';
import { IntlProvider } from 'react-intl';

const messages = require('../../build/lang/en-US.json');
const intlProvider = new IntlProvider({ locale: 'en', messages }, {});
const { intl } = intlProvider.getChildContext();

function nodeWithIntlProp(node) {
  return React.cloneElement(node, { intl });
}

const rendererWithIntl = (node) => {
  return renderer.create(nodeWithIntlProp(node), { context: { intl } });
};

export {
  rendererWithIntl,
};

and then my tests:
import React from 'react';
jest.mock('react-dom');
import { rendererWithIntl } from 'utils/tests';
import Item from './Item';

describe('<Item /> - snapshot', () => {
  it('renders the default Item', () => {
    const component = rendererWithIntl(
      <Item
        items={ [{ id: '1', text: '123' }] }
        selectItem={ () => {} }
      />
    );
    const tree = component.toJSON();
    expect(tree).toMatchSnapshot();
  });
});

it keeps throwing an error saying  Invariant Violation: [React Intl] Could not find required intl object. <IntlProvider> needs to exist in the component ancestry.. So it looks like I'm not defining the intl context properly using renderer.create. Wondering if you have any tips for how to create the intl context using react-test-renderer's renderer?

i don't think react-test-renderer can accept the context argument.
i think for the snapshot testing it's better to look at this or this example.
hey @ericf can you look at this?
we should write a doc for jest if we can get a working example :)

Just to confirm, so you're suggesting to use getRenderedOutput to do snapshot testing?
I currently got the below to work, but I'm not sure how I feel about it:
import React from 'react';
jest.mock('react-dom');
import { createRenderer } from 'react-addons-test-utils';
import { IntlProvider } from 'react-intl';

const messages = require('../../build/lang/en-US.json');
const intlProvider = new IntlProvider({ locale: 'en', messages }, {});
const { intl } = intlProvider.getChildContext();

import Item from './Item';

describe('<Item /> - snapshot', () => {
  const renderer = createRenderer();

  it('renders the default Item', () => {
    const component = renderer.render(
      <Item.WrappedComponent
        items={ [{ id: '1', text: '123' }] }
        selectItem={ () => {} }
        intl={ intl }
      />
    );

    const tree = renderer.getRenderOutput();

    expect(tree).toMatchSnapshot();
  });
});

…sorry for the delay, just realized I was pinged on this.
@iam-peekay I looked at react-test-renderer and it doesn't support passing context as @dmitriiabramov said. The reason for the example I wrote up in the docs about manually creating an IntlProvider and calling its getChildContext() method was to get the context.intl without adding another level to tree — which doesn't work out for shallow renderer. But react-test-renderer supports rending deep trees, so it's actually simpler. You should be able to update this code:
const rendererWithIntl = (node) => {
  return renderer.create(nodeWithIntlProp(node), { context: { intl } });
};

…to this:
const rendererWithIntl = (node) => {
  return renderer.create(<IntlProvider locale='en' messages={messages}>{node}</IntlProvider>);
};

@ericf thank you!! This works :)


[19.03 backport] Fix panic in drivers/overlay/encryption.go

backport of https://github.com/docker/libnetwork/pull/2462
Issue - "index out of range" panic in drivers/overlay/encryption.go:539
due to a mismatch in indices between curKeys and spis due to
case where updateKeys might bail out due to an error and
not update the spis
Fix - Reconfigure keys when there is a key update failure
Signed-off-by: Arko Dasgupta<EMAIL_ADDRESS>(cherry picked from commit 4420ee92f5b3b951f98a36b2bc8144a19b560a22)
Signed-off-by: Sebastiaan van Stijn<EMAIL_ADDRESS>

@arkodg @selansen ptal


R1.10 rocm post-PR#70 updates

In this PR we add the post-PR#70 ROCm updates to the r1.10-rocm release branch

Let me merge this PR so CQE can start to test.


Build(deps-dev): Bump @types/node from 17.0.18 to 17.0.19 (#2831)

Pull request opened by the merge tool on behalf of #2831

Merged in a1d6fcecd44b40e5a8f49a992f52dc7389fc2d87