Bundled installs fail with coreutils cp

Describe the bug
If coreutils cp has been set up as the preferred command for cp, asdf installs fail.  This maybe is the problem of the user who chose to do this, but possibly there's an agnostic use of cp that could work for both cases, or the script added in #21 could try to detect which cp is in use.
Steps to reproduce
❯ which -a cp
/usr/local/opt/coreutils/libexec/gnubin/cp
/bin/cp

❯ asdf install awscli 2.11.18
asdf-awscli: Expected /Users/me/.asdf/installs/awscli/2.11.18/bin/aws to be executable.
asdf-awscli: An error ocurred while installing awscli 2.11.18.

❯ export PATH=/bin:$PATH
❯ which -a cp
/bin/cp
/usr/local/opt/coreutils/libexec/gnubin/cp
/bin/cp

❯ asdf install awscli 2.11.18
mkdir: /Users/me/.asdf/downloads/awscli/2.11.18: File exists
asdf-awscli: asdf-awscli 2.11.18 installation was successful!

Expected behavior
Expected this to work.
Additional context
The issue is with different behaviour of cp -a.  With coreutils, the directory b/ is preserved in the destination when doing a cp -a a/b/ c, with the system cp on MacOS, it is not.  Example.
❯ ls -1 bar/foo/test.txt
bar/foo/test.txt

❯ mkdir baz quux
❯ which -a cp
/usr/local/opt/coreutils/libexec/gnubin/cp
/bin/cp

❯ cp -a bar/foo/ baz
❯ /bin/cp -a bar/foo/ quux

❯ ls -1 baz/
foo
❯ ls -1 baz/foo/
test.txt

❯ ls -1 quux/
test.txt

ah yikes.
Could you post the results of cp --version or the very last line of man cp? That'll help narrow down exactly which version of cp we need to guard against.
I mostly use cp -a as shorthand instead of cp -r, so the short term fix can probably swap them out. I suspect this is because of a very old version of coreutils and we should also tag https://github.com/asdf-vm/asdf/issues/1435 with what we find out :)

^^ Ditto. @kr3cj and I are on the same version. Here's the last line of manpage (/opt/homebrew/opt/coreutils/share/man/man1/gcp.1):
GNU coreutils 9.3          April 2023           CP(1)

The manpage has this to say about -a:
       -a, --archive
              same as -dR --preserve=all
       -d     same as --no-dereference --preserve=links
       -R, -r, --recursive
              copy directories recursively
       --preserve[=ATTR_LIST]
              preserve the specified attributes

I suspect this is because of a very old version of coreutils

Here is the relevant parts of the brew installation info for coreutils:
❯ brew info coreutils
==> coreutils: stable 9.3 (bottled), HEAD
GNU File, Shell, and Text utilities
https://www.gnu.org/software/coreutils
Conflicts with:
  aardvark_shell_utils (because both install `realpath` binaries)
  b2sum (because both install `b2sum` binaries)
  ganglia (because both install `gstat` binaries)
  gdu (because both install `gdu` binaries)
  gfold (because both install `gfold` binaries)
  idutils (because both install `gid` and `gid.1`)
  md5sha1sum (because both install `md5sum` and `sha1sum` binaries)
  truncate (because both install `truncate` binaries)
  uutils-coreutils (because coreutils and uutils-coreutils install the same binaries)
/opt/homebrew/Cellar/coreutils/9.3 (476 files, 13.4MB) *
  Poured from bottle using the formulae.brew.sh API on 2023-04-20 at 16:49:06
From: https://github.com/Homebrew/homebrew-core/blob/HEAD/Formula/coreutils.rb
License: GPL-3.0-or-later

I'm on the latest available version.

Can you try with asdf plugin update awscli macos-coreutils-pr-27? That switches from cp -a to cp -r.
The coreutils changelog looks like it has some macos specific implementation (e.g., https://github.com/coreutils/coreutils/blob/master/NEWS#L407-L410) but I'm not sure exactly where the difference is.

I tried the new branch but it didn't work. I dug in a bit deeper and found the real problem!
It's the symlink that's no good...
I added ls -l  and file on ${install_path}/aws:
@@ -1,4 +1,5 @@
 #!/usr/bin/env bash
+set -x

 set -euo pipefail

@@ -112,6 +113,8 @@ install_v2_macos_bundled_installer() {
        cp -r "${download_path}/tmp-awscliv2/aws-cli.pkg/Payload/aws-cli/" "${install_path}"
        ln -snf "${install_path}/aws" "${install_path}/bin/aws"
        ln -snf "${install_path}/aws_completer" "${install_path}/bin/aws_completer"
+       ls -l "${install_path}/bin/aws"
+       file "${install_path}/bin/aws"
        rm -rf "${download_path}/tmp-awscliv2"
 }

With this we can see:
+ file /Users/matt.klich/.local/share/rtx/installs/awscli/2.11.25/bin/aws
/Users/matt.klich/.local/share/rtx/installs/awscli/2.11.25/bin/aws: broken symbolic link to /Users/matt.klich/.local/share/rtx/installs/awscli/2.11.25/aws

It looks like they ditched the bin dir:
❯ ls -l source/aws-cli.pkg/Payload/aws-cli/
Permissions Size User       Group Date Modified    Name
drwxr-xr-x     - matt.klich staff 2023-06-02 11:19  awscli/
drwxr-xr-x     - matt.klich staff 2023-06-02 11:19  cryptography/
drwxr-xr-x     - matt.klich staff 2023-06-02 11:19  docutils/
drwxr-xr-x     - matt.klich staff 2023-06-02 11:19  lib-dynload/
.rwxr-xr-x  1.8M matt.klich staff 2023-06-02 11:19  _awscrt.cpython-311-darwin.so*
.rwxr-xr-x  225k matt.klich staff 2023-06-02 11:19  _cffi_backend.cpython-311-darwin.so*
.rwxr-xr-x  421k matt.klich staff 2023-06-02 11:19  _ruamel_yaml.cpython-311-darwin.so*
.rwxr-xr-x  6.7M matt.klich staff 2023-06-02 11:19  aws*
.rwxr-xr-x  6.7M matt.klich staff 2023-06-02 11:19  aws_completer*
.rwxr-xr-x  1.8M matt.klich staff 2023-06-02 11:19  base_library.zip*
.rwxr-xr-x  2.9M matt.klich staff 2023-06-02 11:19  libcrypto.1.1.dylib*
.rwxr-xr-x  594k matt.klich staff 2023-06-02 11:19  libssl.1.1.dylib*
.rwxr-xr-x  7.2M matt.klich staff 2023-06-02 11:19  Python*

Below are some logs with more context...
+ major_version=2
+ [[ 2 = \1 ]]
+ [[ 2 = \2 ]]
+ [[ Darwin = \L\i\n\u\x ]]
+ [[ Darwin = \D\a\r\w\i\n ]]
+ install_v2_macos_bundled_installer /Users/matt.klich/.local/share/rtx/downloads/awscli/2.11.25 /Users/matt.klich/.local/share/rtx/installs/awscli/2.11.25
+ local download_path install_path
+ download_path=/Users/matt.klich/.local/share/rtx/downloads/awscli/2.11.25
+ install_path=/Users/matt.klich/.local/share/rtx/installs/awscli/2.11.25
+ mkdir -p /Users/matt.klich/.local/share/rtx/installs/awscli/2.11.25/bin
+ pkgutil --expand-full /Users/matt.klich/.local/share/rtx/downloads/awscli/2.11.25/AWSCLIV2.pkg /Users/matt.klich/.local/share/rtx/downloads/awscli/2.11.25/tmp-awscliv2
+ cp -r /Users/matt.klich/.local/share/rtx/downloads/awscli/2.11.25/tmp-awscliv2/aws-cli.pkg/Payload/aws-cli/ /Users/matt.klich/.local/share/rtx/installs/awscli/2.11.25
+ ln -snf /Users/matt.klich/.local/share/rtx/installs/awscli/2.11.25/aws /Users/matt.klich/.local/share/rtx/installs/awscli/2.11.25/bin/aws
+ ln -snf /Users/matt.klich/.local/share/rtx/installs/awscli/2.11.25/aws_completer /Users/matt.klich/.local/share/rtx/installs/awscli/2.11.25/bin/aws_completer
+ ls -l /Users/matt.klich/.local/share/rtx/installs/awscli/2.11.25/bin/aws
lrwxr-xr-x 1 matt.klich staff 62 2023-06-07 11:05 /Users/matt.klich/.local/share/rtx/installs/awscli/2.11.25/bin/aws -> /Users/matt.klich/.local/share/rtx/installs/awscli/2.11.25/aws
+ file /Users/matt.klich/.local/share/rtx/installs/awscli/2.11.25/bin/aws
/Users/matt.klich/.local/share/rtx/installs/awscli/2.11.25/bin/aws: broken symbolic link to /Users/matt.klich/.local/share/rtx/installs/awscli/2.11.25/aws
+ rm -rf /Users/matt.klich/.local/share/rtx/downloads/awscli/2.11.25/tmp-awscliv2
+ local tool_cmd
++ echo 'aws --version'
++ cut '-d ' -f1
+ tool_cmd=aws
+ test -x /Users/matt.klich/.local/share/rtx/installs/awscli/2.11.25/bin/aws
+ fail 'Expected /Users/matt.klich/.local/share/rtx/installs/awscli/2.11.25/bin/aws to be executable.'
+ echo -e 'asdf-awscli: Expected /Users/matt.klich/.local/share/rtx/installs/awscli/2.11.25/bin/aws to be executable.'
asdf-awscli: Expected /Users/matt.klich/.local/share/rtx/installs/awscli/2.11.25/bin/aws to be executable.
+ exit 1
+ rm -rf /Users/matt.klich/.local/share/rtx/installs/awscli/2.11.25

I didn't have time to do much more during the day. But I've been looking tonight. bisect narrowed it down to #21 being when the install on (at least ARM) Mac started failing.
I did reproduce the original report which is that cp from coreutils fails while the default OSX cp works. I even figured out why (ugh, gross).
But, that said, this isn't the only tool that I've used (or built) that has issues with one over the other. It might be best to force the plugin to use the OSX default cp... Doing so would avoid this particular issue and future potential gotchas. Up to you of course.
It's worth pointing out that coreutils comes with lots of "duplicate" binaries. Any of which may act differently than the system version.
❯ ls -1 /opt/homebrew/opt/coreutils/libexec/gnubin | wc -l
106

So, why does gnu / coreutils cp fail? Because it behaves differently depending on whether the destination directory exists when it is run.
# let's make sure we start clean
❯ rm -rf gnu-dir-exists gnu-dir-does-not-exist

# let's force a directory to exist
❯ mkdir gnu-dir-exists

# now copy files to it
❯ /opt/homebrew/opt/coreutils/libexec/gnubin/cp -a foo/aws-cli.pkg/Payload/aws-cli/ gnu-dir-exists

# now, copy the same files using the same args to a directory that _does not_ exist
❯ /opt/homebrew/opt/coreutils/libexec/gnubin/cp -a foo/aws-cli.pkg/Payload/aws-cli/ gnu-dir-does-not-exist

# we can see these two copies result in _DIFFERENT_ results!!!
❯ ls -l gnu-dir-exists gnu-dir-does-not-exist
gnu-dir-exists:
Permissions Size User       Group Date Modified    Git Name
drwxr-xr-x     - matt.klich staff 2023-06-02 11:19  -N  aws-cli/

gnu-dir-does-not-exist:
Permissions Size User       Group Date Modified    Git Name
drwxr-xr-x     - matt.klich staff 2023-06-02 11:19  -N  awscli/
drwxr-xr-x     - matt.klich staff 2023-06-02 11:19  -N  cryptography/
drwxr-xr-x     - matt.klich staff 2023-06-02 11:19  -N  docutils/
drwxr-xr-x     - matt.klich staff 2023-06-02 11:19  -N  lib-dynload/
.rwxr-xr-x  1.8M matt.klich staff 2023-06-02 11:19  -N  _awscrt.cpython-311-darwin.so*
.rwxr-xr-x  225k matt.klich staff 2023-06-02 11:19  -N  _cffi_backend.cpython-311-darwin.so*
.rwxr-xr-x  421k matt.klich staff 2023-06-02 11:19  -N  _ruamel_yaml.cpython-311-darwin.so*
.rwxr-xr-x  6.7M matt.klich staff 2023-06-02 11:19  -N  aws*
.rwxr-xr-x  6.7M matt.klich staff 2023-06-02 11:19  -N  aws_completer*
.rwxr-xr-x  1.8M matt.klich staff 2023-06-02 11:19  -N  base_library.zip*
.rwxr-xr-x  2.9M matt.klich staff 2023-06-02 11:19  -N  libcrypto.1.1.dylib*
.rwxr-xr-x  594k matt.klich staff 2023-06-02 11:19  -N  libssl.1.1.dylib*
.rwxr-xr-x  7.2M matt.klich staff 2023-06-02 11:19  -N  Python*

That is unexpected behavior. Seems like a simple fix would be to just replace cp with /bin/cp when uname is Darwin

I disagree with the notion that it's unexpected behavior. The docs for macOS's /bin/cp are quite clear - it's not unexpected so much as unstandard behavior.
Given that macos has been steadily removing their BSD internals, I'm very uncomfortable hardcoding behavior to their system /bin/cp, when that may drastically change with macOS15. I have zero visibility into when or why macos will touch /bin/cp and that's not a topic I want to tackle.
If that's not something you/your company are in favor of, I hope this is easy enough to fork against and I welcome PRs that make that easier going forward! Avoiding darwin-specific behavior is going to be my favored response given how weird macOS is lately and the prohibitive cost of testing the various flavors. It is extraordinarily frustrating having to deal with edge cases from a platform I can't test without spending $50 dollars a day on a test instance with AWS because I don't happen to own a macbook.
I'm happy to expand my dev/testing if you and/or your company is willing to sponsor this repo.
There's also no guarantee that this behavior won't hit Linux-based OSes (both Fedora and Debian upstream have coreutils 9, and while the current tests don't show it, I wouldn't be surprised if a similar error showed up next year on a CentOS/RockyLinux/Ubuntu). One of the most compelling arguments to me on the main asdf repo for POSIX compatibility is that it's just wildly hard to try to maintain compatibility across consumer systems that have no incentive to maintain that same compatibility and it's a folly to try, so the only sensible action is to target the POSIX standard. That macOS explicitly breaks POSIX compatibility because /bin/cp has been frozen since 2003 isn't reasonable for me to defend against.
The most recent GH-Action runs suggest that coreutils-9 isn't the actual cause, so versionlocking wouldn't help either.

To be clear, I'm still working on this and hope to have a solution soon

Given that macos has been steadily removing their BSD internals, I'm very uncomfortable hardcoding behavior to their system /bin/cp, when that may drastically change with macOS15. I have zero visibility into when or why macos will touch /bin/cp and that's not a topic I want to tackle.

Fair point. I suggested the opposite because I've been bitten by a lack of consistency with regard to Mac users having coreutils installed. Personally, I prefer the coreutils versions for the very reasons you state but, as they aren't always available, I've chosen to use the Mac versions.

Avoiding darwin-specific behavior is going to be my favored response given how weird macOS is lately

Thanks for sharing your thoughts on what you'd prefer in the repository. Knowing this can help ease contributions.

What about mv instead of cp?
The following works locally (tested both with and without coreutils):
diff --git a/bin/install b/bin/install
index 3966c98..956aed4 100755
--- a/bin/install
+++ b/bin/install
@@ -109,9 +109,9 @@ install_v2_macos_bundled_installer() {

        mkdir -p "${install_path}/bin"
        pkgutil --expand-full "${download_path}/AWSCLIV2.pkg" "${download_path}/tmp-awscliv2"
-       cp -a "${download_path}/tmp-awscliv2/aws-cli.pkg/Payload/aws-cli/" "${install_path}"
-       ln -snf "${install_path}/aws" "${install_path}/bin/aws"
-       ln -snf "${install_path}/aws_completer" "${install_path}/bin/aws_completer"
+       mv "${download_path}/tmp-awscliv2/aws-cli.pkg/Payload/aws-cli" "${install_path}"
+       ln -snf "${install_path}/aws-cli/aws" "${install_path}/bin/aws"
+       ln -snf "${install_path}/aws-cli/aws_completer" "${install_path}/bin/aws_completer"
        rm -rf "${download_path}/tmp-awscliv2"
 }

#29 has working test coverage for this now, and 1.0.1 has been released with the update


Wrong resource file encoding when using proguard obfuscation option -adaptresourcefilecontents

after obfuscation resource file(s) become corrupted (UTF-8 encoding is lost)
OS name: "windows 11"
Java 8
Maven 3.9.7
plugin configuration:
proguard.conf
proguard.conf:
-dontshrink
-dontoptimize
-adaptresourcefilecontents META-INF/**.tld
All my resource files are correct with UTF-8 encoding.
Need to pass JVM argument -Dfile.encoding=UTF-8 to the forked java process. (same as passing maxMemory)
I checked directly executing proguard.jar. With argument -Dfile.encoding=UTF-8 resource files are good. Without argument -Dfile.encoding=UTF-8 I also get corrupted resource files.
P.S. other maven plugins (surefire plugin, failsafe plugin) have argLine argument for passing JVM options to the forked java process

Arguments are passed here https://github.com/wvengen/proguard-maven-plugin/blob/master/src/main/java/com/github/wvengen/maven/proguard/ProGuardMojo.java#L1037 It should be relatively easy to pass file.encoding on

Arguments are passed here https://github.com/wvengen/proguard-maven-plugin/blob/master/src/main/java/com/github/wvengen/maven/proguard/ProGuardMojo.java#L1037 It should be relatively easy to pass file.encoding on

These are main class arguments? Not the jvm args for java fork.
I get proguard.ParseException: Unknown option
https://github.com/Guardsquare/proguard/blob/master/base/src/main/java/proguard/ConfigurationParser.java#L169

Yes, you cannot pass it via that directly, but using https://ant.apache.org/manual/api/org/apache/tools/ant/taskdefs/Java.html#createJvmarg()
But it should be done in similar place.


Is it the design or in the 'clients' section the quotes are overlapping with the text(quotes)

@surajmandalcell Hey there. Can you provide a screenshot of the issue?

Sure!

There you go!

@surajmandalcell as you can see in the design (picture), it is intended this way.

Hum....okey thanks. Ill close this issue now then


Use relative imports inside integrations

Description:
Use relative imports inside integrations
Related issue (if applicable): fixes https://github.com/home-assistant/architecture/issues/180
Used this script with some manual work:
import pathlib

path = pathlib.Path('./homeassistant/components/')

for fil in path.glob('*/*.py'):
    domain = fil.parent.name
    to_find = f"homeassistant.components.{domain}"
    to_replace = "."

    with open(fil) as fp:
        content = fp.read()

    with open(fil, 'w') as fp:
        fp.write(content.replace(to_find, to_replace))

:shipit:


Add support for Laravel 7

I also updated the pipeline to be faster and be able to support Laravel 7.


fix: pool_metadata proto capitalization

Related PRs:

https://github.com/neutron-org/neutron-integration-tests/pull/288
https://github.com/neutron-org/neutron-dev-contracts/pull/45
https://github.com/neutron-org/neutron-sdk/pull/139
https://github.com/neutron-org/neutronjsplus/pull/32

https://github.com/neutron-org/neutron-tests/actions/runs/7631859355

https://github.com/neutron-org/neutron-tests/actions/runs/7631859355

tests failed
you need

regenerate https://github.com/neutron-org/neutron-sdk/tree/main/packages/neutron-sdk/src/proto_types, ask @sotnikov-s
update neutron-dev-contracts with new neutron-sdk https://github.com/neutron-org/neutron-dev-contracts/blob/main/Cargo.toml#L16

This PR is API breaking and will be merged in the next breaking changes release

has conflicts

@jcompagni10 please fix conflicts here

https://github.com/neutron-org/neutron-tests/actions/runs/8850785311


Unable to play a particular video

I am trying to play below video, the video just keeps on loading
https://www.youtube.com/embed/heJu7r3JqNc?enablejsapi=1

Idk, hard to tell from your description.


Write on boarding text for "Calculate clusters of points" analysis

{Calculating point clusters} has finished.
{A new column has} been added to your dataset.

{cluster_no}
{A numeric identifier for each cluster in your dataset. Values start at 1 and go up to the number of clusters you requested (or max rows if less)}

{Point clustering is uses k-means to calculate a predefined number of clusters from your data.}
{link to documentation}

Since we'll know the number of clusters that the user requested, what about writing this instead:
A numeric identifier for each cluster in your dataset. Values start at 1 and go up to the number of clusters you requested ({N}) or max rows if less

love it

On a second thought, this sounds much better :)
A numeric identifier for each cluster in your dataset. Values start at 1 and go up to the {N}  clusters you requested (or max rows if less)

yeah, really good!
On Tue, Jul 26, 2016 at 9:22 AM Javier Arce<EMAIL_ADDRESS>wrote:

On a second thought, this sounds much better :)
A numeric identifier for each cluster in your dataset. Values start at 1
and go up to the {N} clusters you requested (or max rows if less)
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
https://github.com/CartoDB/cartodb/issues/9031#issuecomment-235265225,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAWmU7e0GTyjo3SGfg_1fTxDjF4BFYtTks5qZgn_gaJpZM4JRAHo
.

--
Andrew W. Hill, Chief Science Officer
www.carto.com | @andrewxhill https://twitter.com/andrewxhill
Carto—Predict through Location
201 Moore Street, Brooklyn, New York, 11206, USA


get rid of '_source' structure in out_memory.py

It's unnecessary, and was only ever included to be similar to the elastic output.

done


EntityFramework beta5 / NuGet Public

Hi Devs,
Do you know when beta5 will make its way to the public NuGet feed? I don't want to depend on nightly builds (beta6+ stuff), and from what I can see only beta4 has reached the public NuGet feed
https://github.com/aspnet/EntityFramework/releases/tag/7.0.0-beta5
https://www.nuget.org/packages/EntityFramework/7.0.0-beta4
Unfortunately beta4 still has a dependency on Microsoft.Framework.DependencyInjection.Interfaces which has since been renamed to Abstractions so I can't mix framework beta5 bits with EF7 beta4.

Beta5 is on NuGet.org, but the EntityFramework package is no longer in use, reference the provider package that you want to use, for example
EntityFramework.SqlServer -Pre

https://www.nuget.org/packages/EntityFramework.SqlServer/7.0.0-beta5

@ErikEJ Thanks for this information, makes sense! Most of my framework components are being designed as provider-agnostic, so I can take dependencies on sub-packages like EntityFramework.Core etc, and just have the host dependant on the provider-specific package.
Thanks for your help!

Is the beta5 available for sqlite on NuGet? I can only find a beta1 version.
Thanks!

Beta5 is not available on Nuget yet. We hope it will be soon. Until then, you can try using the nightly builds of SQLite.

Just to clarify - Beta 5 in general is available but we did not publish the SQLite provider as part of Beta 5. Our plan is to ship it as part of Beta 6 which will release shortly.

Thanks guys, can't wait to try it out.


deps; use oas-schema-walker

The copy in /lib had no changes of its own (but was missing one hardening change from upstream).
Tests pass, be interesting to see if this affects the coverage.

Merged in #96


Add a version safeguard?

Currently, unknown value  (e.g. LAYER_UNKNOWN) is the last number.
It will create a versioning problem.
If we add a new layer (say foo_layer). It will have a number of old LAYER_UNKNOWN and will be interpreted differently in an older version.
But in older version will not have foo_layer, should this be alright?
Instead we put a version in our model format to mark which version is competible?

:octocat: cibot: Thank you for posting issue #640. The person in charge will reply soon.

yes, unknown must be set to a large number like 9999 .

As we are going toward the plugin-able architecture, this is no longer needed

As we are going toward the plugin-able architecture, this is no longer needed


Support JS compilation

In the future, there will be an opportunity to compile this library to Javascript. This would allow it to be used in React and React Native.
This issue tries to keep track of the current known hurdles which will prevent us from targeting JS at the moment:

PolymorphicSerializer does not yet work for JS
The library currently relies on reflection here and there which is JVM-specific.

Although polymorph serialization does not yet work, I found a workaround to at least allow compiling a multiplatform project with common serialization code, in which the JVM target can use serialization, but the JS target does not (it is likely that the same serialization code will support JS serialization in the future). This works by using expect and actual (Kotlin multiplatform feature), and mapping the the kotlinx.serialization.Serializable annotation to a dummy annotation on the JS target. Yet, again, I had to jump through an additional hurdle to get this to work (possibly due to current Kotlin typealias limitations).
I am now working towards making all tests between both target platforms shared (in common). Since the JavaScript target does not support serialization quite a few unit tests will fail. Ignoring tests in specific platforms is not yet possible, but it should be possible to use the expect and actual trick again to work around this.

Project is now multiplatform: https://github.com/cph-cachet/carp.protocols.core/commit/a689c82a301cc599441f042b50344619f96a1629


Condense required rbac roles for keda-operator

A clear and concise description of what the bug is.
Expected Behavior
https://github.com/kedacore/keda/blob/master/deploy/10-cluster_role.yaml
clusterrole should only list the required roles keda-operator needs to operate
Actual Behavior
keda-operator requires full control of pods or secrets etc.
Specifications

**KEDA Version: v.1.5.0
**Platform & Version: N/A
**Kubernetes Version: v1.17.5
**Scaler(s): N/A

The following rbac roles seems enough
- apiGroups:
  - ""
  resources:
  - pods
  - services
  - services/finalizers
  - endpoints
  - events
  - configmaps
  - namespaces
  - external
  verbs:
  - get
  - list
- apiGroups:
  - ""
  resources:
  - configmaps
  verbs:
  - '*'

Thanks, good catch. The verbs for the pods/secrets etc should contain watch as well.

#fixed by #1024


Cannot find namespace 'firebase'

Hi what do think about
ERROR in C:/Users/MAREUIL/backend/firecms/firebase-cms-master/src/app/storefront-components/product/product.component.ts (22,20): Cannot find namespace 'firebase'.
ERROR in C:/Users/MAREUIL/backend/firecms/firebase-cms-master/src/app/storefront-components/cart/cart.component.ts (20,20): Cannot find namespace 'firebase'.
ERROR in C:/Users/MAREUIL/backend/firecms/firebase-cms-master/src/app/storefront-components/cart-icon/cart-icon.component.ts (14,20): Cannot find namespace 'firebase'.
Best regards

@antonyfrance  Please try to add a firebase.d.ts reference on to typings.d.ts.
Just add the following line into the src/typings.d.ts file in your angular-cli project.
///


Add Chili sddm theme

Description of changes
Adds the chili sddm theme
There were two previous pull requests for this theme:

#59016 for the non-plasma version
#70495 for the plasma version

This PR adds the non-plasma version of the theme
Because most people want to customize the them with their own backgrounds, etc., I added
a parameter themeConfig, that users can set by overriding the package.
In my opinion there should be a sddm module option for this, but as that doesn't exist right now, I think this is
a good compromise.

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[x] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[x] Tested basic functionality of all binary files (usually in ./result/bin/)
23.05 Release Notes (or backporting 22.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Also I'm not sure but maybe there is a better way to have the theme know about qtgraphicaleffects than echoing it into nix-support/propagated-user-env-packages.

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-ready-for-review/3032/2384


What is the use of fastapi.params

In chapter 7, the author has written from fastapi.params import Depends
https://github.com/PacktPublishing/Building-Data-Science-Applications-with-FastAPI/blob/main/chapter7/chapter7_api_key_header.py#L2
I don't find any use in it. Can @frankie567 clarify this? Maybe, I am wrong.

Good catch, thanks! It's a remainder of a wrong automatic import. I'll fix it right away!


Local JSON themes don't work for regular players (only trusted & above)

In GitLab by @n3dst4 on Aug 14, 2022, 10:49

I'm closing this on the grounds that I cannot see any way to avoid it. It's not a core feature, and should not be a deal breaker for anyone.


Spec contains incomplete wording

Section statements.bind says:

The method used to execute a parameterized Statement an IllegalStateException if a value is not supplied for a bind marker.

That should rather be:

The execute method validates a parameterized Statement and throws an IllegalStateException if no value is supplied for a bind marker.

(with some wording polishing applied)
Blob interface has no class-level documentation.

See also GoogleCloudPlatform/cloud-spanner-r2dbc#74.


wymeditor/skins/refinery.css.scss suggestion

not having height specified (as below), allows the :rows option in <%= f.text_area part, :rows => 5, :class => 'wymeditor widest ' -%>
to have effect. otherwise,  the height: 400px; is CSS dominant and cannot be overridden (or I couldn't figure out how) even with another height value.
textarea.wymeditor, .wym_skin_refinery .wym_iframe iframe {
/* height: 400px; */
}
this is at line 78 in the file.
:-)

Yes, I find that frustrating too. Would you like to open a pull request? Is there some way we can make the text areas default to 400px high but be able to be changed?

I don't mind doing pull request w/o testing ;-)
Yes, I suggest having the height either part of "widest" or as its own
class (say, "highest"), so that it could at least be removed from the
invocation line in view w/o affecting other wymeditor settings.
Invocation example:
<%= f.text_area part, :rows => 5, :class => 'wymeditor widest highest' -%>
if you like this idea, or adding it to widest, then I'll make those
changes. If added to widest, would probably have the least impact to
existing code.
On 02/15/2012 09:12 PM, Philip Arndt wrote:

Yes, I find that frustrating too. Would you like to open a pull request? Is there some way we can make the text areas default to 400px high but be able to be changed?

Reply to this email directly or view it on GitHub:
https://github.com/resolve/refinerycms/issues/1299#issuecomment-3995204

I think that having the rule:
textarea.wymeditor {
  height: 400px;
}

This could work fine.. but what do you think?

It feels like the same difference .. altho I don't know off-hand if the
iframe height was the thing that was affecting it not being overridden
downstream, or not. I could test that out after I take a break :-)
On 02/15/2012 09:25 PM, Philip Arndt wrote:

I think that having the rule:
textarea.wymeditor {
   height: 400px;
}

This could work fine.. but what do you think?

Reply to this email directly or view it on GitHub:
https://github.com/resolve/refinerycms/issues/1299#issuecomment-3995296

Thanks Daudi, I appreciate it :-)

This, and various other similar variations, did not work (the css
selectors don't affect the actual wymeditor area).
But I have found a way to do a downstream CSS override that does work:
textarea.wymeditor, .wym_skin_refinery .wym_iframe iframe {

   height: auto;

}

It turns out that that exact selector string is required, no other
variation has effect. Unfortunately, this is very non-obvious to even
the most earnest observer wishing to override.
Sooooo, at this point, there's still no changes to justify a pull request.
On 02/15/2012 09:25 PM, Philip Arndt wrote:

I think that having the rule:
textarea.wymeditor {
   height: 400px;
}

This could work fine.. but what do you think?

Reply to this email directly or view it on GitHub:
https://github.com/resolve/refinerycms/issues/1299#issuecomment-3995296

If you ever figure it out, feel free to open a pull request :-) thank you!

Thanks Phil .. it seems as though my answer to your suggestion got lost;
I'll repeat it here.
textarea.wymeditor {
height: 400px;
}
and various other similar variations, did not work (the css selectors
don't affect the actual wymeditor area).
But I have found a way to do a downstream CSS override that does work:
textarea.wymeditor, .wym_skin_refinery .wym_iframe iframe {

   height: auto;

}

It turns out that that exact selector string is required, no other
variation has effect. Unfortunately, this is very non-obvious to even
the most earnest observer wishing to override.
Sooooo, at this point, there's still no changes to justify a pull request.
I think it's ok to close this out.
On 02/20/2012 09:43 PM, Philip Arndt wrote:

If you ever figure it out, feel free to open a pull request :-) thank you!

Reply to this email directly or view it on GitHub:
https://github.com/resolve/refinerycms/issues/1299#issuecomment-4068664

Ah okay thanks for the update :-)


Syntax highlighting broken in yaml

I love this syntax theme and want to keep using it, but it's not integrating well with yaml and Dockerfiles
In a yaml file, keys and values have the same syntax highlighting and makes it difficult to read. Same with Dockerfiles.
For example, these keys and values have the same color (green):
config.yml
version: 2.0
jobs:
  build:
    working_directory: ~/app
    docker:
      - image: circleci/ruby:2.3.3-node-browsers
        environment:
          RAILS_ENV: test
          RACK_ENV: test
          PGHOST: <IP_ADDRESS>
      - image: circleci/postgres:9.6.2-alpine
        environment:
          POSTGRES_USER: circleci
          POSTGRES_PASSWORD: ''
    parallelism: 5
    steps:
      - checkout

      - restore_cache:
          name: Restore bundle cache
          keys:
            - app-bundle-{{ checksum "Gemfile.lock" }}
            - app-bundle

Here's a sample Dockerfile (must install Docker support package):
FROM busybox
ENV foo /bar
WORKDIR ${foo}   # WORKDIR /bar
ADD . $foo       # ADD . /bar
COPY \$foo /quux # COPY $foo /quux

Thanks for the feedback. I will look into that and make some changes. :)

@laurawadden Thanks again for you suggestion. I finally had the time to tweak the colors a bit. :)


channeldb: inject now function [wip, no review]

Just saw this. Now that we have clock.Clock you might as well use that instead of the now closure.


Add missing import statements for python 3.9 and improve FPS.

Looks like this still needs to be merged @lcrh & @hmoraldo

Looks like this still needs to be merged @lcrh & @hmoraldo

Yes, I'm waiting for the tests to finish


When you update a monitor, an exception can occur when the related adapter executes

Describe the issue
I was testing monitoring using Eclipse. I replaced the configuration of ibis4example by my own example configuration. That configuration defined a monitor and it had an adapter with which you could raise or clear the monitor. Initially, the adapter had no threshold and no period. Then I went to the console and updated the monitor to give it a non-zero period and a non-zero threshold. Then an exception occurred when I ran the adapter in test-a-pipeline.
Reporter
Martijn Dirkse

To Reproduce
Steps to reproduce the behavior:

Replace Configuration.xml of ibis4example with the configuration shown below.
Go to test-a-pipeline and run adapter TestMonitoring a few times.
Go to "Monitoring" in the console and update "myMonitor" to give it a non-zero threshold and a non-zero period.
Go to test-a-pipeline again and issue non-valid XML to adapter "TestMonitoring".

Here is the example configuration I applied:
<Configuration name="Ibis4Example"
  xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
  xsi:noNamespaceSchemaLocation="./FrankConfig-strict.xsd">
        <!--
        <jmsRealms>
              <jmsRealm realmName="jdbc" datasourceName="${jdbc.datasource.default}"/>
        </jmsRealms>
        -->
        <monitoring>
            <destination className="nl.nn.adapterframework.monitoring.GalmMonitorAdapter" name="GALM" />
            <monitor name="myMonitor" destinations="GALM">
                <trigger className="nl.nn.adapterframework.monitoring.Alarm" eventCode="Invalid XML: parser error" severity="WARNING" />
                <trigger className="nl.nn.adapterframework.monitoring.Clearing" eventCode="valid XML" severity="WARNING"/>
            </monitor>
        </monitoring>
        <Adapter name="TestMonitoring">
            <Receiver name="TestMonitoringReceiver">
                <JavaListener name="myListener" serviceName="myListener" />
            </Receiver>
            <Pipeline firstPipe="myPipe">
                <Exit state="success" path="EXIT" />
                <Exit state="error" path="ERROR_EXIT" />
                <XmlWellFormedCheckerPipe name="myPipe">
                    <Forward name="success" path="EXIT" />
                    <Forward name="failure" path="ERROR_EXIT" />
                </XmlWellFormedCheckerPipe>
            </Pipeline>
        </Adapter>
</Configuration>

Screenshots
Here is the exception that occurs:
...
Caused by: java.lang.NullPointerException
        at nl.nn.adapterframework.monitoring.Trigger.cleanUpEvents(Trigger.java:136)
        at nl.nn.adapterframework.monitoring.Trigger.evaluateEvent(Trigger.java:118)
        at nl.nn.adapterframework.monitoring.Trigger.evaluateEvent(Trigger.java:100)
        at nl.nn.adapterframework.monitoring.Trigger.onApplicationEvent(Trigger.java:92)
        ...

Environment
This happened when I was working on PR #2113. I tested by tweaking ibis4example.

Additional Environment

n/a
Chrome

Additional Context
Add any other context about the problem here. (f.e. ladybug report / test adapter with larva test)

I think the issue is that variable Trigger.eventDates is only initialized during configure(). It seems that this variable is not initialized to an empty list when a monitor is updated. Then a null pointer exception occurs when the list is accessed.


Implement SecondaryMap on sparse containers.

As mentioned in #51, we should implement the trait for HashSet/HashMap/BTreeSet/BTreeMap

#62 added a HashSet impl, so I'm closing this PR.
We can reopen it if we have some usecase for other container types.


[.NET] Action Fallback card doesn't render properly

Platform
What platform is your issue or question related to? (Delete other platforms).

.NET WPF

Issue
The samples\v1.2\Elements\ActionFallback.json card does not render correctly on .NET WPF visualizer. The fallback actions are not being rendered.
UWP results [correct]:

.NET WPF results [missing fallback actions]:

@almedina-ms is this something you can take a stab at ?

@shalinijoshi19 yes, I can take a stab at this


Inferentia lab in AIML module

What this PR does / why we need it:
Create the lab for Inferentia in the AIML module.
Quality checks

[x] My content adheres to the style guidelines
[x] I ran make test or make e2e-test and it was successful

Close and re-create a new PR to enable the new netlify build automation


Add test data product related to job posting

Added data definition for job posting.
For future development you might want to consider moving common parameters and schemas (x-consent-token, validationError, etc...) to single place, so you can simply reference them in the open api spec file instead of duplicating code.

Regarding the suggestion to avoid duplicating code for x-consent-token, validationError, etc..., the recommended way to avoid a lot fo duplication and boilerplate is to build the definition using python/pydantic classes, that get conveniently converted to OpenAPI spec. The learning curve for that should be pretty minimal and you don't need to care about the deep and nested structure of the OpenAPI spec JSON files. You can check how easily those can be built by looking at example within the /src directory of this repo. Personally I find the OpenAPI JSON files more or less unreadable without any tools whereas the python/pydantic versions are really easy to view and understand. Of course it's a matter of taste.


Support for custom OpenAI-Compatible backends

What's the problem?
Usage of local large language models through LocalAI, LMStudio and so on, all of these are providing OpenAI-compatible API but application need to expose a setting to change baseurl.

Yes, AzureOpenAI support, please.

We've open-sourced the OpenAI API logic (https://github.com/BloopAI/bloop/tree/oss/server/bleep/src/llm). You can now build and run bloop with a custom OpenAI API key (see: https://github.com/BloopAI/bloop?tab=readme-ov-file#building-from-source).
Feel free to open a PR adding support for Azure OpenAI 😀


Rework dtypes.py module and add more tests for it

Codecov Report

Merging #366 into master will increase coverage by 0.48%.
The diff coverage is 98.8%.

@@            Coverage Diff             @@
##           master     #366      +/-   ##
==========================================
+ Coverage    94.6%   95.09%   +0.48%
==========================================
  Files          32       32
  Lines        9725     9707      -18
==========================================
+ Hits         9200     9230      +30
+ Misses        525      477      -48


Implement OpenSearch ("make the search-field usable as browser-search")

title says it all
i want to add caniuse.com as search engine in chrome

push for opensearch B-)

+1

Vote!

If it makes it easier, the following should work:
<?xml version="1.0" encoding="UTF-8"?>
<OpenSearchDescription xmlns="http://a9.com/-/spec/opensearch/1.1/" xmlns:moz="http://www.mozilla.org/2006/browser/search/">
  <ShortName>caniuse</ShortName>
  <LongName>Can I Use...</LongName>
  <Description>Search for feature support across browsers on caniuse.com</Description>

  <InputEncoding>UTF-8</InputEncoding>

  <Image width="16" height="16" type="image/x-icon">http://caniuse.com/favicon.ico</Image>
  <Image width="128" height="128" type="image/png">http://caniuse.com/img/favicon-128.png</Image>

  <Url type="text/html" method="get" template="http://caniuse.com/#search={searchTerms}"/>
  <Url type="application/opensearchdescription+xml" rel="self" template="http://caniuse.com/search.xml" /> <!-- or wherever you put it -->

  <moz:SearchForm>http://caniuse.com/</moz:SearchForm>
</OpenSearchDescription>

push again, see @Tigt's comment :)

Now implemented! Sorry for the delay, and many thanks to @tigt for making it easy to add. :)

Thank you very much :)

Great, but perhaps not quite finished. In chrome most search inputs will have "Add Search Engine" in their right click menu, which is really convenient. To get that to work it seems that a search input needs to be inside a form element having the attribute action="/search", or something along the lines of "/search". It varies. If you check out the form above, for instance, you'll see action="/Fyrd/caniuse/search" (and if you backspace the [This Repository] qualifier it'll change to action="/search")

@makoConstruct Firefox has similar functionality (Context menu > "Add a keyword for this search"), which I can activate by adding an empty, attribute-less <form> element around the search input.

@Fyrd I can't find it on caniuse, can you paste the link?
Thanks.

@Fyrd
Ohh wait, this issue is about adding it to the site...
I'm asking adding which browsers support it... :smile:
Couldn't find any info about it.

@gdoron OpenSearch is not part of HTML/CSS standards

@inoyakaigor ohh that makes sense.
Anyway, since I spammed this thread already :smile:  , do you know which browsers support it beside Chrome?

@gdoron Firefox, IE, Safari. About mobile browsers I have no information


test of conflicts

Hello @mikelove,
I played around with building a tidyomics repository taking tidyverse repository as a template.
Look what I get when I load the repo XD
Isn't wonderful?! It tells us how far are we from building a consistent ecosystem.
> library(tidyomics)
── Attaching core tidyomics packages ─────────────────────────────────────────────────────────────────────────────────── tidyomics 0.1.0 ──
✔ dplyr                    1.1.2      ✔ tidyr                    1.3.0
✔ ggplot2                  3.4.2      ✔ tidyseurat               0.6.0
✔ nullranges               1.4.0      ✔ tidySingleCellExperiment 1.11.2
✔ plyranges                1.18.0     ✔ tidySummarizedExperiment 1.9.6
✔ tidybulk                 1.11.3
── Conflicts ───────────────────────────────────────────────────────────────────────────────────────────────────── tidyomics_conflicts() ──
✖ dplyr::between()    masks plyranges::between()
✖ dplyr::bind_cols()  masks tidybulk::bind_cols(), tidyseurat::bind_cols(), tidySingleCellExperiment::bind_cols(), ttservice::bind_cols(), tidySummarizedExperiment::bind_cols()
✖ dplyr::bind_rows()  masks tidybulk::bind_rows(), tidyseurat::bind_rows(), tidySingleCellExperiment::bind_rows(), ttservice::bind_rows(), tidySummarizedExperiment::bind_rows()
✖ dplyr::count()      masks tidyseurat::count(), tidySingleCellExperiment::count(), tidySummarizedExperiment::count(), matrixStats::count()
✖ dplyr::filter()     masks tidybulk::filter(), plyranges::filter(), tidyseurat::filter(), tidySingleCellExperiment::filter(), tidySummarizedExperiment::filter(), stats::filter()
✖ dplyr::n()          masks plyranges::n()
✖ dplyr::n_distinct() masks plyranges::n_distinct()
✖ dplyr::rename()     masks tidybulk::rename(), tidyseurat::rename(), tidySingleCellExperiment::rename(), tidySummarizedExperiment::rename(), S4Vectors::rename()
✖ dplyr::slice()      masks plyranges::slice(), tidyseurat::slice(), tidySingleCellExperiment::slice(), tidySummarizedExperiment::slice(), IRanges::slice()
ℹ Use the conflicted package to force all conflicts to become errors

It's a bit strange that I developed ttservice exactly to avoid conflicts (I only declare generics in ttservice), but it detects conflicts nonetheless. I have to make some tests.

Hey @stemangiola , I'm interested in working on this, I've got some experience with R package development, though it was not specifically using tidyverse. Any tips to get started would be awesome!

Hey @stemangiola , I'm interested in working on this, I've got some experience with R package development, though it was not specifically using tidyverse. Any tips to get started would be awesome!

Amazing! So the work that @HelenaLC, @william-hutchison @aliosmancetin, and @GrootJ are doing should solve all conflicts for transcriptomic packages.
plyranges needs some work in this direction; you can ask @mikelove for more details (feel free to tag Stuart Lee and Mike Lawrence). The issue might be that some methods of plyranges do not use the tidyverse generics but build a new one.

Hi @stemangiola thank you so much. I installed and loaded tidyomics and see the conflict message.
I guess to resolve the conflicts for plyranges, we can specify functions with the package name?
I looked into one or two but found some were created in plyranges, some are the same as those in say dplyr. Not sure I need to update the plyranges package or just within tidyomics (& maybe some other related packages..), what would you suggest to do?

I looked into one or two but found some were created in plyranges, some are the same as those in say dplyr.

I think a good example is how we deal in tidySingleCellExperiment.
We would need to fork and do a PR for plyranges.  What do you think about this challenge @lawremi @sa-lee @mikelove

@mikelove @william-hutchison  Currently, if we import tidyverse after tidyomics,  we get these conflicts
5 for plyranges
3 for tidytranscriptomics
the rest is for IRanges, Biobase, and S4Vectors, for which obviously we cannot do much.
✖ lubridate::%within%()    masks IRanges::%within%()
✖ plyranges::between()     masks dplyr::between()
✖ tidybulk::bind_cols()    masks ttservice::bind_cols(), dplyr::bind_cols()
✖ ttservice::bind_rows()   masks dplyr::bind_rows()
✖ IRanges::collapse()      masks dplyr::collapse()
✖ Biobase::combine()       masks BiocGenerics::combine(), dplyr::combine()
✖ matrixStats::count()     masks dplyr::count()
✖ IRanges::desc()          masks dplyr::desc()
✖ S4Vectors::expand()      masks tidyr::expand()
✖ plyranges::filter()      masks tidybulk::filter(), dplyr::filter(), stats::filter()
✖ S4Vectors::first()       masks dplyr::first()
✖ dplyr::lag()             masks stats::lag()
✖ plyranges::n()           masks dplyr::n()
✖ plyranges::n_distinct()  masks dplyr::n_distinct()
✖ BiocGenerics::Position() masks ggplot2::Position(), base::Position()
✖ purrr::reduce()          masks GenomicRanges::reduce(), IRanges::reduce()
✖ tidybulk::rename()       masks S4Vectors::rename(), dplyr::rename()
✖ lubridate::second()      masks S4Vectors::second()
✖ lubridate::second<-()    masks S4Vectors::second<-()
✖ plyranges::slice()       masks IRanges::slice(), dplyr::slice()

@mikelove, any news from plyranges maintenance?

I don't think they will be able to fix this easily.
One option would be to call plyranges a key tidyomics package, but not loaded with library(tidyomics).

Would importing a tidyomics-flavoured fork of plyranges a feasible option? I'm not sure what the opinion of the original developers and the community would be.


Publish page does not preserve plugin order

Add three link plugins to a placeholder in the following order and with the following names:

Link 1
Link 2
Link 3

Now publish your changes to create the 'live' version. Now go back to draft mode and drag Link 3 so it comes after Link 2. Both structure and content mode show the plugins now in the correct 1,2,3 order. Click publish and note that the live version still has the plugins in the original order 1,3,2.

I should be clear that my pull request is against the develop branch. Your pull request #3709 is against the support/3.0.x branch which doesn't appear to do the same plugin position wackiness that was introduced after the django-treebeard change. I'm not clear on what the bug is you are fixing in your pull request, but agree that the code is confusing & possibly buggy.

@yakky I finally added a proper test case after all this time! If you temporarily revert 41d988e2df09da21a4567d3d612238fc30d4cba2 you'll see the tests fail due to the scrambled ordering on the live page and that with the offending lines removed, the ordering is preserved. I've punted on the question of refactoring the plugin position changing code for now.

Anything else needed (aside from free time!) to get this merged? Let me know.

@jsma I should be able to review the PR in the next few days.
Thanks for your efforts


Feature/update vertical domain

Description
Makes the Y domain a bit bigger so that top items don't get cropped.
Issues Resolved
Closes #334
Check List

[ ] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Codecov Report

:exclamation: No coverage uploaded for pull request base (main@b41857f). Click here to learn what that means.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #372   +/-   ##
=======================================
  Coverage        ?   15.66%
=======================================
  Files           ?      109
  Lines           ?     2898
  Branches        ?      479
=======================================
  Hits            ?      454
  Misses          ?     2403
  Partials        ?       41

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

As mentioned in this post, there is an issue with the windows runner that likely causing the windows cypress tests to timeout.


Circular package ref in 4.12 prevents you from being able to update from 4.11 packages

Github issues should be used for bugs and feature requests. Use Stack Overflow for general "how-to" questions.
Version
4.12.2
Describe the bug
I tried updating my projects packages from 4.11 to 4.12 but it fails because of a dependency error.  I tracked it down to the fact that Bot.Builder.AI.Luis now has a reference on Bot.Builder.DIalogs.Adaptive.Declarative.  The issue is that package has a reference on Bot.Builder.Dialogs.Adaptive which has a reference on Bot.Builder.AI.Luis.
I was able to work around the issue by manually updating all of the package versions in the csproj file to "4.12.2".

I think this is a composer runtime issue, and would be fixed with:
https://github.com/microsoft/BotFramework-Composer/pull/6506/

Closing this as it is being addressed by Composer.


Unable to install Windows CPC 1.16 Server 2016

While attempting to install CPC 1.16 on Windows Server 2016 the service fails to start after verification of the device with the appropriate Google Account and pressing enter when prompted for an auto-assigning e-mail address as one is not needed.
As a side note upon rollback of the installer the Cloud Print Connector service is not cleaned up properly and is left on the system.

Set the stuck Cloud Print Connector service to disabled and restart the server
Manually remove the entry by opening regedit and navigate to the following key:
HKEY_LOCAL_MACHINE\SYSTEM\CurrentControlSet\Services
Find the Cloud Print Connector Key and delete it
Restart the system to refresh the services list correctly

Errors?

I'm also having this problem. I'm not sure if a Windows Update patch caused this problem, or if something changed with Google's API, but the service seems to hang every time the system attempts to start. I was able to manually run the the command, but my printers were still not appearing on my Chrome devices. I've contacted Google about this to see if I get anything from them, but they haven't been able to shed any light on it, either. I'm hoping that the developer is aware of what the problem might be, and hopefully working on a fix. This service seems to be the only thing that is working for our organization, but something has caused it to malfunction.


Got Tuple index out of range issue when calling manual_forecast() for rnn

Hi Michael,
I got the IndexError: tuple index out of range when calling manual forecast() for rnn. Below are the details:
Error
IndexError                                Traceback (most recent call last)
Cell In[18], line 2
1 f.set_estimator('rnn')
----> 2 f.manual_forecast(
3     lags = 24,
4     layers_struct=[
5         ('LSTM',{'units':10,'activation':'tanh'}),
6         ('LSTM',{'units':10,'activation':'tanh'}),
7         ('LSTM',{'units':10,'activation':'tanh'}),
8     ],
9     optimizer = 'Adam',
10     epochs = 15,
11     plot_loss = True,
12     validation_split=0.2,
13     call_me = 'rnn_tanh_activation',
14 )
735 self.call_me = self.estimator if call_me is None else call_me
737 if test_again and self.test_length > 0 and self.estimator != 'combo':

--> 738     self.test(
739         **kwargs,
740         dynamic_testing=dynamic_testing,
741         call_me=call_me,
742     )
743 elif bank_history:
744     if self.call_me not in self.history.keys():
1319 f1.eval_cis(False)
1320 f1.actuals = actuals
-> 1321 f1.manual_forecast(
1322     dynamic_testing = dynamic_testing,
1323     test_again = False,
1324     call_me = call_me,
1325     **kwargs,
1326 )
1327 fcst = f1.forecast
1329 if not already_existed:
744     if self.call_me not in self.history.keys():
745         self.history[self.call_me] = {}
747 preds = (
748     self._forecast_sklearn(
749         fcster=self.estimator,
750         dynamic_testing=dynamic_testing,
751         **kwargs,
752     )
753     if self.estimator in self.sklearn_estimators

--> 754     else getattr(self, f"forecast{self.estimator}")(
755         dynamic_testing=dynamic_testing, **kwargs
756     )
757 )
758 self.forecast = preds
759 if bank_history:
 12 with warnings.catch_warnings(record=True) as warn_list:
 13     warnings.simplefilter("always")

---> 14     result = func(*args, **kwargs)
15     for warn in warn_list:
16         logging.warning(warn.message)
978 X = X.reshape(X.shape[0], X.shape[1], 1)
979 fut = fut.reshape(fut.shape[0], fut.shape[1], 1)
--> 980 model = get_compiled_model(y)
981 hist = model.fit(X, y, **kwargs)
982 fcst = model.predict(fut)
850                 kv[1]["return_sequences"] = (
851                     layers_struct[i + 1][0] != "Dense"
852                 )
854         model.add(layer(**kv[1]))

--> 855 model.add(Dense(y.shape[1]))  # output layer
857 # compile model
858 model.compile(optimizer=local_optimizer, loss=loss)
IndexError: tuple index out of range
Code:
f.set_estimator('rnn')
f.manual_forecast(
lags = 24,
layers_struct=[
('LSTM',{'units':10,'activation':'tanh'}),
('LSTM',{'units':10,'activation':'tanh'}),
('LSTM',{'units':10,'activation':'tanh'}),
],
optimizer = 'Adam',
epochs = 15,
plot_loss = True,
validation_split=0.2,
call_me = 'rnn_tanh_activation',
)
Code from:
https://towardsdatascience.com/stacking-time-series-models-to-improve-accuracy-7977c6667d29
Could you please help? Thank you so much!
Thank you,
Lily

Thank you for raising the issue! Give me a little bit to look into it.

Are you able to tell me the following:

number of observations in the time series
f.test_length
len(f.future_dates)

I'm thinking you are setting too many observations aside in the test set for the given number of lags. For the RNN model, it cuts several observations from any series to create the lagged features necessary to perform a direct forecast. A way to fix this is to either use fewer test-set observations or fewer lags in the model (f.manual_forecast(lags=...)).
In my next update, I will make the error that gets raised in this situation more descriptive.

Hello,
Yes, there are not enough observations to use a forecast length of 12, test length of 21, and 24 lags. You need at least number of lags + number of forecast steps + 1 observations for that model to work. If you upgrade the package, the error that gets raised in this situation should be more descriptive now.


chore(ci): Use codecov for test coverage.

Codecov Report

:exclamation: No coverage uploaded for pull request base (main@210cb6b). Click here to learn what that means.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #45   +/-   ##
=======================================
  Coverage        ?   83.05%
  Complexity      ?      492
=======================================
  Files           ?       44
  Lines           ?     1523
  Branches        ?      138
=======================================
  Hits            ?     1265
  Misses          ?      211
  Partials        ?       47

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 210cb6b...2786e53. Read the comment docs.


publish<EMAIL_ADDRESS>

Requested by: @chadwhitacre
Merge target: (default)
Quick links:

View changes
View check runs

Assign the accepted label to this issue to approve the release.
Leave the #retract comment under this issue to retract the release (original issuer only).
Targets

[ ] github

Targets marked with a checkbox have already been executed.  Administrators can manually tick a checkbox to force craft to skip it.

tests failed in self hosted due to flaky error, rerunning


feat: remove fidelity promises.

connects to #51
There was one part were i needed to do a little refactoring to get the tests to pass,  which was how we handled errors in the event of a timeout.   https://github.com/bucharest-gold/opossum/compare/master...lholmquist:51-remove-fidelty?expand=1#diff-dbc2c6ba82726efeb035efaaac7c38acR278 and https://github.com/bucharest-gold/opossum/compare/master...lholmquist:51-remove-fidelty?expand=1#diff-dbc2c6ba82726efeb035efaaac7c38acL356
I also removed the option to specify a different Promise library, not sure if that was part of this though

Coverage decreased (-0.01%) to 99.18% when pulling 05b3ac9bcc4d8b9baf6f34a5f68b579018492573 on lholmquist:51-remove-fidelty into f54822e2eaf9b8224b45c372766f7979029ca9e0 on bucharest-gold:master.

@lance @helio-frota thoughts?  i would like to merge this before we merge #55 since i'll probably have some conflicts while merging

i would like to merge this before we merge #55 since i'll probably have some conflicts while merging

makes sense. :+1:

@lholmquist I reviewed a while ago - LGTM

Coverage decreased (-0.01%) to 99.18% when pulling 3f5827a982b52263c5a0e0fdfb1c943fccfa21cf on lholmquist:51-remove-fidelty into f54822e2eaf9b8224b45c372766f7979029ca9e0 on bucharest-gold:master.


Adaptive JSON object size estimator for binary printers

Circe's reuseWriters printing option is great and we need something similar for binary printing. One idea is to predict the size of the printing object so we can allocate an ultimate destination buffer in one shot (no need to copy-and-grow iterations).
Netty uses this AdaptiveRecvByteBufAllocator to achieve the very same goal. We can use it for inspiration.
I'd be happy to give it a try.

Sure, sounds good to me!


The URL path is not valid - The resource 'resetRatingsRequests' does not exist

New Issue Checklist

[x] Updated fastlane to the latest version
[x] I read the Contribution Guidelines
[x] I read docs.fastlane.tools
[x] I searched for existing GitHub issues

Issue Description

When running "fastlane deliver" with the "reset_ratings" field in the Deliverfile set to true, we're getting the error below (see complete output). We've traced the error to this API call but are unsure of how to fix this issue.
@joshdholtz Could you please take a look? Thanks!
Command executed

fastlane deliver -f --verbose --api_key_path ****
Complete output when running fastlane, including the stack trace and command used

 /Users/****/.rvm/gems/ruby-2.7.0/gems/fastlane-2.191.0/spaceship/lib/spaceship/connect_api/api_client.rb:199:in `handle_response': \e[31m[!] The URL path is not valid - The resource 'resetRatingsRequests' does not exist\e[0m (Spaceship::UnexpectedResponse)
	from /Users/****/.rvm/gems/ruby-2.7.0/gems/fastlane-2.191.0/spaceship/lib/spaceship/connect_api/api_client.rb:124:in `post'
	from /Users/****/.rvm/gems/ruby-2.7.0/gems/fastlane-2.191.0/spaceship/lib/spaceship/connect_api/tunes/tunes.rb:951:in `post_reset_ratings_request'
	from /Users/****/.rvm/rubies/ruby-2.7.0/lib/ruby/2.7.0/forwardable.rb:235:in `post_reset_ratings_request'
	from /Users/****/.rvm/gems/ruby-2.7.0/gems/fastlane-2.191.0/spaceship/lib/spaceship/connect_api/models/app_store_version.rb:236:in `create_reset_ratings_request'
	from /Users/****/.rvm/gems/ruby-2.7.0/gems/fastlane-2.191.0/deliver/lib/deliver/upload_metadata.rb:335:in `upload'
	from /Users/****/.rvm/gems/ruby-2.7.0/gems/fastlane-2.191.0/deliver/lib/deliver/runner.rb:146:in `upload_metadata'
	from /Users/****/.rvm/gems/ruby-2.7.0/gems/fastlane-2.191.0/deliver/lib/deliver/runner.rb:55:in `run'
	from /Users/****/.rvm/gems/ruby-2.7.0/gems/fastlane-2.191.0/deliver/lib/deliver/commands_generator.rb:82:in `block (2 levels) in run'
	from /Users/****/.rvm/gems/ruby-2.7.0/gems/commander-4.6.0/lib/commander/command.rb:187:in `call'
	from /Users/****/.rvm/gems/ruby-2.7.0/gems/commander-4.6.0/lib/commander/command.rb:157:in `run'
	from /Users/****/.rvm/gems/ruby-2.7.0/gems/commander-4.6.0/lib/commander/runner.rb:444:in `run_active_command'
	from /Users/****/.rvm/gems/ruby-2.7.0/gems/fastlane-2.191.0/fastlane_core/lib/fastlane_core/ui/fastlane_runner.rb:76:in `run!'
	from /Users/****/.rvm/gems/ruby-2.7.0/gems/commander-4.6.0/lib/commander/delegates.rb:18:in `run!'
	from /Users/****/.rvm/gems/ruby-2.7.0/gems/fastlane-2.191.0/deliver/lib/deliver/commands_generator.rb:187:in `run'
	from /Users/****/.rvm/gems/ruby-2.7.0/gems/fastlane-2.191.0/deliver/lib/deliver/commands_generator.rb:18:in `start'
	from /Users/****/.rvm/gems/ruby-2.7.0/gems/fastlane-2.191.0/fastlane/lib/fastlane/cli_tools_distributor.rb:114:in `take_off'
	from /Users/****/.rvm/gems/ruby-2.7.0/gems/fastlane-2.191.0/bin/fastlane:23:in `'
	from /Users/****/.rvm/gems/ruby-2.7.0/bin/fastlane:23:in `load'
	from /Users/****/.rvm/gems/ruby-2.7.0/bin/fastlane:23:in `'
	from /Users/****/.rvm/gems/ruby-2.7.0/bin/ruby_executable_hooks:22:in `eval'
	from /Users/****/.rvm/gems/ruby-2.7.0/bin/ruby_executable_hooks:22:in `'

Environment

 ✅ fastlane environment ✅
Stack

Key
Value

OS
10.15.1

Ruby
2.7.0

Bundler?
true

Git
git version 2.21.1 (Apple Git-122.3)

Installation Source
~/.rvm/gems/ruby-2.7.0/bin/fastlane

Host
Mac OS X 10.15.1 (19B88)

Ruby Lib Dir
~/.rvm/rubies/ruby-2.7.0/lib

OpenSSL Version
OpenSSL 1.1.1g  21 Apr 2020

Is contained
false

Is homebrew
false

Is installed via Fabric.app
false

Xcode Path
/Applications/Xcode.app/Contents/Developer/

Xcode Version
11.3.1

Swift Version
5.1.3

System Locale

Variable
Value

LANG
en_US.UTF-8
✅

LC_ALL
en_US.UTF-8
✅

LANGUAGE

fastlane files:
`./fastlane/Fastfile`
lane :privacy do
  upload_app_privacy_details_to_app_store(
    username: "xxxx",
    team_name: "xxxx",
    app_identifier: "xxxx",
    json_path: "fastlane/app_privacy_details.json"
  )
end

No Appfile found
fastlane gems

Gem
Version
Update-Status

fastlane
2.191.0
✅ Up-To-Date

Loaded fastlane plugins:
No plugins Loaded
Loaded gems

Gem
Version

did_you_mean
1.5.0

executable-hooks
1.6.1

bundler-unload
1.0.2

rubygems-bundler
1.4.5

bundler
2.2.19

rake
13.0.6

CFPropertyList
3.0.3

public_suffix
4.0.6

addressable
2.8.0

artifactory
3.0.15

atomos
0.1.3

aws-eventstream
1.1.1

aws-partitions
1.484.0

aws-sigv4
1.2.4

jmespath
1.4.0

aws-sdk-core
3.119.0

aws-sdk-kms
1.46.0

aws-sdk-s3
1.98.0

babosa
1.0.4

claide
1.0.3

colored
1.2

colored2
3.1.2

highline
2.0.3

commander
4.6.0

declarative
0.0.20

digest-crc
0.6.4

unf_ext
<IP_ADDRESS>

unf
0.1.4

domain_name
0.5.20190701

dotenv
2.7.6

emoji_regex
3.2.2

excon
0.85.0

faraday-em_http
1.0.0

faraday-em_synchrony
1.0.0

faraday-excon
1.1.0

faraday-httpclient
1.0.1

faraday-net_http
1.0.1

faraday-net_http_persistent
1.2.0

faraday-patron
1.0.0

faraday-rack
1.0.0

multipart-post
2.0.0

ruby2_keywords
0.0.5

faraday
1.6.0

http-cookie
1.0.4

faraday-cookie_jar
0.0.7

faraday_middleware
1.1.0

fastimage
2.2.4

gh_inspector
1.1.3

jwt
2.2.3

memoist
0.16.2

multi_json
1.15.0

os
1.1.1

signet
0.15.0

googleauth
0.17.0

httpclient
2.8.3

mini_mime
1.1.0

trailblazer-option
0.1.1

uber
0.1.0

representable
3.1.1

retriable
3.1.2

rexml
3.2.5

webrick
1.7.0

google-apis-core
0.4.1

google-apis-androidpublisher_v3
0.10.0

google-apis-playcustomapp_v1
0.5.0

google-apis-iamcredentials_v1
0.6.0

google-apis-storage_v1
0.6.0

google-cloud-env
1.5.0

google-cloud-errors
1.1.0

google-cloud-core
1.6.0

google-cloud-storage
1.34.1

json
2.5.1

mini_magick
4.11.0

naturally
2.2.1

plist
3.6.0

rubyzip
2.3.2

security
0.1.3

simctl
1.6.8

terminal-notifier
2.0.0

unicode-display_width
1.7.0

terminal-table
1.8.0

tty-screen
0.8.1

tty-cursor
0.7.1

tty-spinner
0.9.3

word_wrap
1.0.0

nanaimo
0.3.0

xcodeproj
1.20.0

rouge
2.0.7

xcpretty
0.3.0

xcpretty-travis-formatter
1.0.1

generated on: 2021-08-18

Seen this error as well.

Facing this error too.

Seeing the same issue as well.

Hi @joshdholtz,
Have you had a chance to look into this issue?
Thanks!

@Leonwu21 Hello! I have not yet 😣 I've been traveling to speak at a conference but it's on my list!

Hi @joshdholtz,
I am part of Leon's team as well. Did you get a chance to look into this item?
Thanks!

@Krishsidu Thanks for the ping! Will bump up in my queue 💪


WIP: Add comment on commit support

Target #4898
Todo list:

[x] Adjust comment table to add repo_id and allow issue_id zero and added some indexes
[x] Allow submit comment on a commit without treepath and line, and list all these comments above the form
[ ] Allow submit comment with treepath and line on a commit and list them
[ ] Adjust notification list for comment on commit
[ ] API support
[ ] Docs update
[ ] Search comments on commits

Can the Gitea devs please put more focus into this? My project really needs this awesome feature.
I don't mean to come off as rude, sorry I just returned from India and I'm about to head off to bed.

closed as so many files conflicted.


Return index (indices) of the index of the words of numbers

I want to start by saying that this is really a helpful library.
Now, this library helps to convert words for numbers to numeric and it's great. However, thinking of an instance where I want to convert all the words for numbers in a string (e.g. in a proper sentence - These are the four items I like the most) and still have the string as a result (These are the 4 items I like the most), if this library also returns the index (indices) of the words in the original string or just return the original sentence after replacing the words for numbers with the numbers will help. Thanks.
Please suggest.

Thanks for the library. I agree with above. If I do
>>> from word2number import w2n
>>> w2n.word_to_num("1 tsp five-spice powder")
5

it would be useful to know that the number was only a subset of the string. Providing a mechanism to know the indexes of the match would provide that


DeepgramApiError with None message

What is the current behavior?
When I get a 400 error from the API, the error raised by the SDK contains no context or error message, other than the status code.
Steps to reproduce
Here's a unit test. I got the error payload from the 400 example on this page: https://developers.deepgram.com/reference/listen-file
import json
import unittest
from unittest.mock import patch, Mock

from deepgram import DeepgramClient, PrerecordedOptions
from deepgram.clients.errors import DeepgramApiError
from httpx import HTTPStatusError

class TestDeepgramTranscriber(unittest.IsolatedAsyncioTestCase):
    @patch('httpx.Client.request', new_callable=Mock)
    async def test_transcribe_url_no_retry_example(self, mock_post):
        api_key = 'your_api_key'
        deepgram_client = DeepgramClient(api_key)

        mock_response = Mock()
        mock_response.status_code = 400
        mock_response.text = json.dumps(
            {
                "err_code": "Bad Request",
                "err_msg": "Content-type was application/json, but we could not process the JSON payload.",
                "request_id": "4399e2a5-2b29-4e1e-9456-79043cc88d0a"
            }
        )

        mock_response.raise_for_status = Mock(side_effect=HTTPStatusError("message", request={}, response=mock_response))

        mock_post.return_value = mock_response

        audio_url = 'https://example.com/audio.mp3'
        source = {'url': audio_url}
        options = PrerecordedOptions(
            punctuate=False,
            model="nhtsa",
            version="v1_dc",
        )
        with self.assertRaises(DeepgramApiError) as context:
            await deepgram_client.listen.asyncprerecorded.v("1").transcribe_url(source, options)

        # Assert that the request was not retried and the result is as expected
        mock_post.assert_called_with(
            'POST',
            'https://api.deepgram.com/v1/listen?model=nhtsa&punctuate=false&version=v1_dc',
            headers={'Accept': 'application/json', 'Authorization': 'Token your_api_key'},
            json={'url': 'https://example.com/audio.mp3'}
        )

        self.assertEqual(context.exception.message, 'Content-type was application/json, but we could not process the JSON payload.')

Expected behavior
I'd like the see the error message that the API returns, and the request ID, in case I need to reach out to support.
Please tell us about your environment

Operating System/Version: Linux
Language: Python

Other information
I think the error message key should actually be err_msg instead of message on this line
https://github.com/deepgram/deepgram-python-sdk/blob/5a29caedd097bdffe7e8d03536ff119ffd29bfca/deepgram/clients/abstract_sync_client.py#L113
Here's a full stacktrace that I got in my production environment:
[ERROR] DeepgramApiError: DeepgramApiError: None (Status: 400)
Traceback (most recent call last):
  File "/var/task/app.py", line 330, in lambda_handler
    asyncio.run(transcribe_pipeline())
  File "/var/lang/lib/python3.11/asyncio/runners.py", line 190, in run
    return runner.run(main)
  File "/var/lang/lib/python3.11/asyncio/runners.py", line 118, in run
    return self._loop.run_until_complete(task)
  File "/var/lang/lib/python3.11/asyncio/base_events.py", line 653, in run_until_complete
    return future.result()
  File "/var/task/app.py", line 282, in transcribe_pipeline
    transcript = await deepgram_client.transcribe_url(audio_url)
  File "/var/task/datadrivenstreets/retry.py", line 23, in wrapper
    raise e
  File "/var/task/datadrivenstreets/retry.py", line 19, in wrapper
    result = await func(*args, **kwargs)
  File "/var/task/datadrivenstreets/clients/deepgram.py", line 24, in transcribe_url
    .transcribe_url(source, options, timeout=Timeout(10.0, read=20.0))
  File "/var/lang/lib/python3.11/site-packages/deepgram/clients/prerecorded/v1/client.py", line 77, in transcribe_url
    result = self.post(
  File "/var/lang/lib/python3.11/site-packages/deepgram/clients/abstract_sync_client.py", line 49, in post
    return self._handle_request(
  File "/var/lang/lib/python3.11/site-packages/deepgram/clients/abstract_sync_client.py", line 112, in _handle_request
    raise DeepgramApiError(

This PR should be merged.
BTW @jonahweissman if you at all want to contribute those unit tests, that would be awesome!

should be available in https://github.com/deepgram/deepgram-python-sdk/releases/tag/v3.0.2


Fix VEP PDB display

Since this is specific to this tool, I think a workaround is the best option for now. It defers handling of versioned stable IDs to the API, which is (I think) the right thing to do.
See ticket https://www.ebi.ac.uk/panda/jira/browse/ENSWEB-6409

@ens-ap5 the code change looks good but i couldn't test it since there was no sandbox URL. I couldn't even see the button that was mentionned in the jira ticket on the live site. Just wanted to see how this is happening.

We did discuss this in a code review, and decided it was OK to merge it as long as we test it once the tools test is available.


Include timestamp in the .Alert.Vars object

I wanted to include a timestamp in the HTTP Post notification, and was not sure the best way to do that. After playing around for a bit I settled on at {{.State.Touched.Format "15:04:05UTC"}} which generates the correct timestamp in my case (a log alert) but am pretty sure the Touched value could be modified and represent something other than the alert timestamp.
Is there a better way to create a timestamp in the alert template? If not can we add .Alert.Vars.Time like is exposed in the Unknown alert template.

What exactly should this timestamp represent? I think this needs to be better defined.

For example of could be the timestamp of when the alert first triggered (since being closed). It could be the time of the last severity increase....

There already is some "history" exposed by the .State.History and .State.Actions objects. I believe Touched represents the last time the state was updated. Not sure how valuable a "first triggered" would be unless it is reset after an alert is closed. The only only one I currently wanted was time.Now().UTC() from when the alert was triggered.

Can you define what you mean exactly by triggered?

Or actually, what is the problem you trying to solve?

"I wanted to include a timestamp in the HTTP Post notification"

Right, we can include all sorts timestamps. I could make one that just returns this constant string everytime "Tue Mar 31 16:20:38 UTC 2015" which would be obviously useless. What I'm try to get from you is what exactly is being stamped. What is the event you would like the timestamp to represent?

Assuming I was posting to something that forwarded via SMS text message (which can have a large delay), some indicator of what time the event actually occurred. Even on our chat system I can see that the Touched timestamp is more than 10s older than when the post was created in chat.


DynamicLibrary.open error: Invalid argument(s): Failed to load dynamic library (126)

Hey, I want to ask why I still report 126 errors when compiling 64 bit DLL with vs2012
c代码
`#include  <windows.h>
int add(int a, int b){
return a + b;
}
void showBox(){
MessageBox(NULL,"Hello Dart","Title",MB_OK);
}dart代码
import 'dart:ffi' as ffi;
typedef NativeAddSign = ffi.Int32 Function(ffi.Int32, ffi.Int32);
typedef DartAddSign = int Function(int, int);
void main(List args) {
ffi.DynamicLibrary dl = ffi.DynamicLibrary.open("./full_all/test.dll");
var add = dl.lookupFunction<NativeAddSign, DartAddSign>("add");
print(add(8, 9));
}
`

Hi @xz-zone
The goal of this Github repo is not to give assistance on personal code or the whole package/plugin project,
but to improve flutter reporting bugs or with feature requests.
If you encounter behavior that you believe could be a bug, please provide a completed and minimal reproducible code sample that represents the problem. Also, please provide the output of flutter doctor -v as well.
Instead, if you think that the issue might depend on your implementation, please see flutter's community for resources and ask questions like this. You may also get some help if you post it on Stack Overflow and flutterhelp.
Thank you!

Hi @xz-zone The goal of this Github repo is not to give assistance on personal code or the whole package/plugin project, but to improve flutter reporting bugs or with feature requests.
If you encounter behavior that you believe could be a bug, please provide a completed and minimal reproducible code sample that represents the problem. Also, please provide the output of flutter doctor -v as well.
Instead, if you think that the issue might depend on your implementation, please see flutter's community for resources and ask questions like this. You may also get some help if you post it on Stack Overflow and flutterhelp.
Thank you!

Sorry, which section do I need to submit to ask questions

@xz-zone
Please provide the information indicated below:

A completed and minimal reproducible code sample
The output of flutter doctor -v
Reproduction steps, Expected results, Actual results
Which environment did this issue happen in? (Android/iOS/Web/MacOS/Windows/Linux)

@xz-zone 请提供以下信息：

一个完整且最小的可重现代码示例
的输出flutter doctor -v
再现步骤、预期结果、实际结果
这个问题是在什么环境下发生的？(Android/iOS/Web/MacOS/Windows/Linux)

Operating system:  Windows
Dart environment:  dart SDK version: 2.17.5 (stable) (Tue Jun 21 11:05:10 2022+0200) on "windows_x64"
Compiling DLL environment:  vs2012
Error reporting log:
Unhandled exception:
Invalid argument(s): Failed to load dynamic library './full_all/Dll164.dll': error code 126
#0      _open (dart:ffi-patch/ffi_dynamic_library_patch.dart:12:43)
#1      new DynamicLibrary.open (dart:ffi-patch/ffi_dynamic_library_patch.dart:23:12)
#2      main (file:///c:/Users/Administrator/Desktop/dart/newuser.dart:27:40)
#3      _delayEntrypointInvocation. (dart:isolate-patch/isolate_patch.dart:295:32)
#4      _RawReceivePortImpl._handleMessage (dart:isolate-patch/isolate_patch.dart:192:12)

@xz-zone 请提供以下信息：

一个完整且最小的可重现代码示例
的输出flutter doctor -v
再现步骤、预期结果、实际结果
这个问题是在什么环境下发生的？(Android/iOS/Web/MacOS/Windows/Linux)

Operating system:  Windows
Dart environment:  dart SDK version: 2.17.5 (stable) (Tue Jun 21 11:05:10 2022+0200) on "windows_x64"
Compiling DLL environment:  vs2012
Error reporting log:
Unhandled exception:
Invalid argument(s): Failed to load dynamic library './full_all/Dll164.dll': error code 126
#0      _open (dart:ffi-patch/ffi_dynamic_library_patch.dart:12:43)
#1      new DynamicLibrary.open (dart:ffi-patch/ffi_dynamic_library_patch.dart:23:12)
#2      main (file:///c:/Users/Administrator/Desktop/dart/newuser.dart:27:40)
#3      _delayEntrypointInvocation. (dart:isolate-patch/isolate_patch.dart:295:32)
#4      _RawReceivePortImpl._handleMessage (dart:isolate-patch/isolate_patch.dart:192:12)

@xz-zone Please provide the information indicated below:

A completed and minimal reproducible code sample
The output of flutter doctor -v
Reproduction steps, Expected results, Actual results
Which environment did this issue happen in? (Android/iOS/Web/MacOS/Windows/Linux)

Operating system:  Windows
Dart environment:  dart SDK version: 2.17.5 (stable) (Tue Jun 21 11:05:10 2022+0200) on "windows_x64"
Compiling DLL environment:  vs2012
Error reporting log:
Unhandled exception:
Invalid argument(s): Failed to load dynamic library './full_all/Dll164.dll': error code 126
#0      _open (dart:ffi-patch/ffi_dynamic_library_patch.dart:12:43)
#1      new DynamicLibrary.open (dart:ffi-patch/ffi_dynamic_library_patch.dart:23:12)
#2      main (file:///c:/Users/Administrator/Desktop/dart/newuser.dart:27:40)
#3      _delayEntrypointInvocation. (dart:isolate-patch/isolate_patch.dart:295:32)
#4      _RawReceivePortImpl._handleMessage (dart:isolate-patch/isolate_patch.dart:192:12)

@xz-zone
I see there's an issue similar to the case you described #58079 and look at https://github.com/flutter/flutter/issues/58079#issuecomment-634547807 that provides a workaround for this. Let's read all the comments there that might be helpful to you.
Also see https://github.com/dart-lang/sdk/issues/38450#issuecomment-644722545 that explained error codes:

Error code 126 could be due to a wrong path, try using backslashes in the path.

@xz-zone 我看到有一个与您描述的案例类似的问题#58079并查看#58079（评论），它为此提供了解决方法。让我们阅读那里可能对您有帮助的所有评论。 另请参阅解释错误代码的 dart-lang/sdk#38450（评论） ：

错误代码 126 可能是由于路径错误，请尝试在路径中使用反斜杠。

The same problem

@xz-zone I see there's an issue similar to the case you described #58079 and look at #58079 (comment) that provides a workaround for this. Let's read all the comments there that might be helpful to you. Also see dart-lang/sdk#38450 (comment) that explained error codes:

Error code 126 could be due to a wrong path, try using backslashes in the path.

The same problem, the following figure is the result I detected with sigcheck

Hi @xz-zone
It seems this issue will have the best support in dart-lang/sdk. Please open the issue in the dedicated repository.
If you believe it could be a Flutter bug, please provide the information indicated below:

A completed and minimal reproducible code sample
The output of flutter doctor -v
Reproduction steps, Expected results, Actual results

Thank you!

I don't understand

Hi @xz-zone It seems this issue will have the best support in dart-lang/sdk. Please open the issue in the dedicated repository.
If you believe it could be a Flutter bug, please provide the information indicated below:

A completed and minimal reproducible code sample
The output of flutter doctor -v
Reproduction steps, Expected results, Actual results

Thank you!

I don't understand

Hi @xz-zone
Which part don't you understand?

It seems this issue will have the best support in dart-lang/sdk. Please open the issue in the dedicated repository.

As the explanation on the https://github.com/flutter/flutter/issues/108344#issuecomment-1195371087, the similar issue has been solved in https://github.com/dart-lang/sdk/issues/38450. So I recommend you file the new issue in the dedicated repository to have better support there.

If you believe it could be a Flutter bug, please provide the information indicated below:
A completed and minimal reproducible code sample
The output of flutter doctor -v
Reproduction steps, Expected results, Actual results

Flutter bug here means that the bug belongs to Flutter framework/plugins/packages.
Hope the above explanations help you understand better.

Hi @xz-zone Which part don't you understand?

It seems this issue will have the best support in dart-lang/sdk. Please open the issue in the dedicated repository.

As the explanation on the #108344 (comment), the similar issue has been solved in dart-lang/sdk#38450. So I recommend you file the new issue in the dedicated repository to have better support there.

If you believe it could be a Flutter bug, please provide the information indicated below:
A completed and minimal reproducible code sample
The output of flutter doctor -v
Reproduction steps, Expected results, Actual results

Flutter bug here means that the bug belongs to Flutter framework/plugins/packages.
Hope the above explanations help you understand better.

Do you mean I resubmit a new question？

@xz-zone

Do you mean I resubmit a new question？

Yes, please file a new issue in the dart-lang/sdk.
Closing from now. If you disagree, please write in the comments and I will reopen it.


[Documentation] Accepted values for system

Issue description
Still new to nix, so forgive me if this is documented somewhere already.
The system parameter is used in a couple of places, but I had to hunt down for the possible values it accepts. I believe all the possible values are these: https://github.com/NixOS/nix/blob/4ae6e849015fb269414c1544e815aba75085cf7b/release.nix#L4
It would be nice to include these possible values in the Nix Pills section that talks about the parameter.

It's sort of documented here:
https://github.com/NixOS/nixpkgs/blob/master/pkgs/top-level/impure.nix#L15-L19
But it definitely should be available in one of the manuals.


Feat/ Docs for v0.1.0 release (MVP)

Update on the docs with the latest updates about the Incubation DAO program with Axolotarium DAO

Closes #16


Is this project still maintained?

There hasn't been any activity on this repo or the related Melody repo for 2 years. Personally, I've had to stop using this plugin because of issue #51, and I imagine others are in the same boat.
Unfortunately, this has become the de facto plugin for formatting Twig files with Prettier, so if it's been abandoned, it would be good to know, so people can stop creating issues/PRs and start looking for or working on alternatives.

Hi @twbartel, would you mind taking a look at this question? I am hoping for a release with the nandroid security issue patched. Issue #51 would be a very welcome addition as well.

bump @twbartel @urbantrout @rjanjic @mrkanister
This plugin could use some love to make it an essential pick for every project with twig. As it stands a lot of core twig functionality is still missing


Vertically align the asset download counts in releases

https://github.com/electron/electron/releases
Vertically align them on the icon. And with some more right margin. Would make it look cleaner. I think we could use CSS grid for this.

// @dotconnor

+1 And also put count and filesize in different columns.

Could someone in the right direction for using css grid? From my understanding css grid only has parent-child relationships, and the given layout is:
<container>
	<row>
		<item>
	</row>
	<row>
		<item>
	</row>
</container>

@jerone Still need the downloads to be left aligned, something you can only do with a table/grid. (I think?)

@dotconnor commented on Jun 8, 2019, 12:18 PM GMT+2:

@jerone Still need the downloads to be left aligned, something you can only do with a table/grid. (I think?)

Don't think left align the downloads makes it visually better;

.py-1 small {
    min-width: 60px;
    display: inline-block;

    &:nth-child(2) {
        text-align: right;
    }
}

.release .Box-body small:nth-child(2) {
	min-width: 60px;
	display: inline-block;
	text-align: right;
}

and flex aligning the children


can I use django-import-export import fields different with the export fields?

I found with django-import-export, the import fields and export fields is the same, but I want to set different fields when importing and exporting, I don't know whether django-import-export support that?

Off course, use 2 resources.

hi @bmihelac , can I ask a simple example 😊 , I am not actually understand the documentation. 😢

This would use BookResource for import and ExportBookResouce for export.
class BookAdmin(ImportExportModelAdmin):
    resource_class = BookResource

    def get_export_resource_class(self):
        """
        Returns ResourceClass to use for export.
        """
        return ExportBookResource

Have you tried overriding Resource.get_export_fields()? https://github.com/django-import-export/django-import-export/blob/master/import_export/resources.py#L597
We use that to change both what fields are visible on the import screen, as well as what fields get exported.


allow parallel

Coverage remained the same at 84.815% when pulling 440e1e0aab231a796056a06a93f11103bc39e976 on allow-parallel into be6ae03373b4200c7a7b4571c4f109c0f90e2aff on master.


Complete type hints and annotation comments

Changed log

It's related to issue #63.
I use the PHPDoc style and internal PHP type hints to complete the type hints for all codes in src folder.

I'm not sure that these type hints or annotation comments are set the correct type hints/comments.
@emsifa, please check that and write down some comments/request changes.
Thanks.

Wow thank you @peter279k . You're so fast.  :+1:
I will check it, or maybe I will directly update it in your branch.

I think it's good for now. I will merge it.
Thank you all. :smiley:

@emsifa, thank you for your commits and concern. I appreciate your help :).


Question about Mixamo dataset

Hi, great works! I am curious about the dataset setting: are all of Mixamo animations manually designed or are they also created by some kind of motion retargeting? It raises my doubts when some characters in Mixamo, e.g., Sword Woman, do not behave naturally in certain motions. If the Mixamo animations are not designed by human artists, then why can we take them as ground truth (please feel free to correct me)? Thanks.

Hi, great works! I am curious about the dataset setting: are all of Mixamo animations manually designed or are they also created by some kind of motion retargeting? It raises my doubts when some characters in Mixamo, e.g., Sword Woman, do not behave naturally in certain motions. If the Mixamo animations are not designed by human artists, then why can we take them as ground truth (please feel free to correct me)? Thanks.

I have the same questions as you! I found that there existing model intersections for some motions of a few characters like "Mousey"

ground truth is whatever you decide it is, no?

ground truth is whatever you decide it is, no?

Yes, but what I concerned about is that: if the motion data of different characters are not created manually but in an automatic motion retargeting methods, then what our deep learning models learns is nothing more than this "automatic retargting method".  Then the motivation of using deep learning methodology is worth to thinking about.
But, I still think this project is worth to exploring!

@bmahlbrand Ground truth upper bounds a learning-based method's performance. @crissallan Good to hear a similar concern! And surely, this is an interesting project. I am just not quite sure about how Mixamo animations are created and why they can be taken as supervision signals. @PeizhuoLi @kfiraberman  Please kindly have a look! Thank you in advance.

Great question. As far as I know, motion retargeting procedure within Mixamo is automatic for the "intra-structural" examples (same skeletal structure). However, since there is no existing fully-automated way to perform cross-structural retargeting,
at least one "cross-structural" example from each motion should be generated manually (or semi-manually; IK + manual corrections for the missing correspondences), then this example can be retargeted to the other skeletons with a similar structure.
I agree with the claim that training on examples that were automatically generated is equivalent to train a network to approximate an existing retargeting algorithm which sounds useless in some aspects (but maybe some benefits can be gained, like better processing times). However, since there is no existing "cross-structural" retargeting algorithm, in our case the network learns a new "algorithm".
In addition, remember that the framework is general. It was trained on Mixamo only for evaluation purposes - there was no better (publicly available) way to quantitatively evaluate the performance.

In addition, remember that the framework is general. It was trained on Mixamo only for evaluation purposes - there was no better (publicly available) way to quantitatively evaluate the performance.

To be clear, this is what I was driving at. Not that it will somehow result in a realistic manifold by training on unnatural motion.

Thank you so much for your reply @kfiraberman, that clearly clarifies my concern!


Load Testing

Started performance loading this library in the last few days and I ran into an issue where I could only get it to perform around 3.3 requests per second.
Had to add System.Net.ServicePointManager.DefaultConnectionLimit = 1000; to my app startup to fix this issue.

I think you can achieve the same by providing that in the ProxyOptions, i.e:
app.RunProxy(new ProxyOptions {     Scheme = "https",     Host = "example.com",     Port = 443,     BackChannelMessageHandler = new HttpClientHandler { MaxConnectionsPerServer = 1000 } });

Interesting, thanks for your input.  We've ran all out tests etc via the my suggested fix above but if I have time I'll check it out.  Hopefully it helps someone else.


Cannot find available options for the icon attribute in cards

Is your feature request related to a problem? Please describe
When creating various type of cards often there's a parameter for icon which accepts a string text. But developers cannot find any reference on the available options/names for the icons.
ui.stat("User", q.user.user.name, icon='contact'),

Describe the solution you'd like
With some offline discussions I could find that WAVE is supporting fonts available in https://uifabricicons.azurewebsites.net/.
Better if we can provide this link in WAVE docs for developer reference.
Describe alternatives you've considered
May be we can also introduce an enum  instead of string for icon. But since we don't have control over the transformation of the text into icons, better solution would be a link to https://uifabricicons.azurewebsites.net/ on the docs in relevant places.

Thanks @rasika! Agree. This is already handled by the upcoming docs update in the next release. In addition to that, the Pycharm plugin we will release together with 0.20.0 will provide you with autocomplete for all the icon attributes so the whole dev experience should be much smoother.

Awesome, thanks for the update @mturoci. Any link for the fix?
rel: Pycharm plugin, where can I find Pycharm plugin code? Any ETA on 0.20.0 ?

Any link for the fix?

https://github.com/h2oai/wave/tree/master/website/components

where can I find Pycharm plugin code?

https://github.com/h2oai/wave/tree/master/tools/intellij-plugin

Any ETA on 0.20.0

Once https://github.com/h2oai/wave/issues/59 is resolved.

Thanks found it https://github.com/h2oai/wave/search?q=https%3A%2F%2Fuifabricicons.azurewebsites.net%2F


[Look&Feel] use semantic header with correct size for page, modal and flyout

Description
Use H1s on main pages and H2s on secondary experiences (modals/flyouts). These should be under  to make sure they’re the correct size.
[Done] Need to reproduce the behavior of alerting visualization flyout, and make change accordingly

Issues Resolved
[List any issues this PR will resolve]
Check List

[ ] New functionality includes testing.

[ ] All tests pass

[ ] New functionality has been documented.

[ ] New functionality has javadoc added

[ ] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Also, can you address the failing checks

Also, can you address the failing checks

@virajsanghvi Failed CI is not related to code change. The same cypress test also failed on the HEAD commit of main branch https://github.com/opensearch-project/alerting-dashboards-plugin/commits/main/

let's make sure e2e passes before merging

let's make sure e2e passes before merging

@riysaxen-amzn The failed test is not related to my change, could you help with re-run? I don't have permission


The module name does not match the repo name

The module name is onesignal-go-client in the go.mod but the repo name is onesignal-go-api. This causes an error
<EMAIL_ADDRESS>parsing go.mod:
        module declares its path as: github.com/OneSignal/onesignal-go-client
                but was required as: github.com/OneSignal/onesignal-go-api

I was able to get it installed by doing go get github.com/OneSignal/onesignal-go-client. I'm guessing the repo was renamed. If this was intentional then update the instructions here - https://onesignal.com/blog/our-go-client-sdk-is-now-available/

Howdy @akeemphilbert !
You are exactly right. Thanks for pointing this out. We will get it fixed asap.

This is now resolved. Thanks


#21 Add functionality to display edges in neuron populations with mis…

Add a filter to show and hide paths with missing nodes and complete nodes. The mechanism is just the same as SCKAN filter.

@napakalas thanks for the PR, but I'm developing generic methods to control feature visibility and styling based on arbitrary properties. The intent is to replace all specific filters (path type, Sckan validity, taxon, node completeness, etc) and any future ones for new properties with a general mechanism (which will also handle 3D paths), so I'm closing this.


Enable to publish multi-arch operator image

Is your feature request related to a problem? Please describe.
Currently, an operator image is available only for x86_64 at https://quay.io/repository/confidential-containers/operator.
I would be good to have it available for other architectures (aarch64, ppc64le and s390x) with the same tag.
Describe the solution you'd like

Turn on a docker buildx functionality on the GHA workflow.

$ # install qemu-user-static
$ docker run --rm --privileged multiarch/qemu-user-static --reset -p yes
$ # create/use a new builder for multi-arch support
$ docker buildx create --driver-opt network=host --use --name container-builder
$ docker buildx inspect --bootstrap
$ docker buildx use container-builder
$ # enable buildx
$ export DOCKER_BUILDKIT=1

Change a base image from bitnami/golang to golang:1.18 (one compatible with all architectures)
Make a dedicated script for multi-arch image build (or within the GHA workflow) with a tag ending with a architecture name
Make Dockerfile configurable with an argument named DOCKER_ARCHES which is used for GOARCH in go build.
Consolidate all architecture specific docker images into one tag (via docker manifest)

Additional context
Multi-arch support for payload image (e.g. quay.io/confidential-containers/runtime-payload) will be handled in the CCv0 branch of the kata-containers project (https://github.com/kata-containers/kata-containers/issues/5660). The Ansible test will be followed after this issue.
I look forward to feedback from you.
cc: @fidencio @stevenhorsman @hbrueckner

I'd be happy to see this happening!


hideWhenNoData option does not hide the visualization when no data found

It should be possible to put the following in a viz and it will hide if no data is present.
    "containerOptions": {
        "visibility": {
            "hideWhenNoData": true
        }
    },
    ```

This currently does not work.

It looks like this is because we are using the "Viewer" layours, which have bug that prevents this working.
Switching to the non-"View" mode seems to fix.


remove demonic gorilla plugin

While other things are up to interpretation jagex were very clear they do not want this.

I still think this is up to interpretation. They only said that the client cannot tell you what to pray. What about a simpler plugin that at least tells you how many attacks have happened before the last prayer switch? This would at least ease you from counting attacks while still being compliant with the newspost.
I don't think we necessarily have to scrap the plugin altogether as a consequence.

abex did it