[Component Proposal] Bubble chart card

Description
A clear and concise description of what you are requesting and why.
The carbon data viz design is underway for bubble charts, it is however looking likely that it will follow the design precedent set by the scatter plot, which will make our current bubble map design a deviation.
EDS image:

EDS: https://ibm.invisionapp.com/share/U2NYW8D7DFV#/screens/319100262
Carbon: https://www.carbondesignsystem.com/data-visualization/basic-charts#bubble-chart
Name of Pattern or Asset
Bubble chart
Proposal

The style for bubbles (Carbon: Low opacity fill, and a border. EDS: Moderate opacity fill, no border).
Recommendation: Revert to Carbon style as this is consistent across chart types, without strong rationale for the deviation.

Additional context
This work should be done with care to involve the Carbon charts team to ensure we don't align to 'old' Carbon charts designs, and need to update again after.

closing. Implementation will come from Carbon charts. We can reopen if a team prioritizes this chart type.


[ICON] Calendar with Download Arrow

Icon Request

Icon name: calendar-download
Use case: Download button for a calendatr
Screenshots of similar icons:

This is what I am currently using for this purpose, made by adding a downwards arrow to calendar-minus.
Feel free to add this icon to the repo

Upvote & Fund

We're using Polar.sh so you can upvote and help fund this issue.
We receive the funding once the issue is completed & confirmed by you.
Thank you in advance for helping prioritize & fund our backlog.

released in v7.9.0


Installing llm-mistral but not setting an api key breaks llm models list

Installing llm-mistral but not setting an api key for it breaks llm models list. Not sure if this is due to how mistral works. Or if this should be reported to the llm repository. A workaround could be to ignore mistral when no api key set and just issue a warning for this when listing models.
❯ llm keys
No keys found

❯ llm plugins
[]

❯ llm models
OpenAI Chat: gpt-3.5-turbo (aliases: 3.5, chatgpt)
OpenAI Chat: gpt-3.5-turbo-16k (aliases: chatgpt-16k, 3.5-16k)
OpenAI Chat: gpt-4 (aliases: 4, gpt4)
OpenAI Chat: gpt-4-32k (aliases: 4-32k)
OpenAI Chat: gpt-4-1106-preview
OpenAI Chat: gpt-4-0125-preview
OpenAI Chat: gpt-4-turbo-preview (aliases: gpt-4-turbo, 4-turbo, 4t)
OpenAI Completion: gpt-3.5-turbo-instruct (aliases: 3.5-instruct, chatgpt-instruct)

❯ llm install llm-mistral
Collecting llm-mistral
...
...
Using cached llm_mistral-0.3-py3-none-any.whl (9.5 kB)
Installing collected packages: llm-mistral
Successfully installed llm-mistral-0.3

❯ llm models
Error: You must set the 'mistral' key or the LLM_MISTRAL_KEY environment variable.

Thanks, yeah this is a bug.

I tested this manually by deleting ~/Library/Application\ Support/io.datasette.llm/mistral_models.json and then removing my mistral key from the keys.json file.

🙏 Thank you.


readme.md needs to be updated.

please update the readme.md, I tried to open some links in the documents. But it directs to an unexisting website/url.

Hi @fakecore , readme is used for generate docs for docs.io.
Unfortunately I didn't find ways to let readme compatible with both docs.io and GitHub.
you may refer to https://docs.rs/cfg-rs/latest/cfg_rs/ to get the correct links.

ok. Thank you for your reply.


Error deploying Cloud Run with gcp exporter

Hi,
I've been trying to deploy a cloud run instance with the gcp exporter but I'm getting the following issue:
[io.ope.sdk.tra.exp.SimpleSpanProcessor] (vert.x-eventloop-thread-0) Exporter threw an Exception: com.google.api.gax.rpc.UnauthenticatedException: io.grpc.StatusRuntimeException: UNAUTHENTICATED: Failed computing credential metadata
These are the packages I've been using:
  <dependencies>
    <dependency>
      <groupId>io.quarkus</groupId>
      <artifactId>quarkus-resteasy-reactive</artifactId>
    </dependency>
    <dependency>
      <groupId>io.quarkus</groupId>
      <artifactId>quarkus-rest-client-reactive-jackson</artifactId>
    </dependency>
    <dependency>
      <groupId>io.quarkiverse.opentelemetry.exporter</groupId>
      <artifactId>quarkus-opentelemetry-exporter-gcp</artifactId>
      <version>2.0.0.Final</version>
    </dependency>
    <dependency>
      <groupId>io.quarkus</groupId>
      <artifactId>quarkus-container-image-jib</artifactId>
    </dependency>
  </dependencies>

And I've defined the following parameters:
quarkus.opentelemetry.tracer.exporter.gcp.enabled = true
quarkus.opentelemetry.tracer.exporter.gcp.projectid = "test"
quarkus.opentelemetry.tracer.exporter.gcp.cloudrun = true
quarkus.opentelemetry.tracer.exporter.otlp.enabled = false

Is there any dependency package I have to add or any parameter that is missing.

Hello,
I've noticed that the GCP's Project ID set to test. Do you really have such project?
If you are deploying your code directly to GCP, you can skip this property. It's only useful for local testing. The project name is set by CloudRun infra.
Hope this helps

Hello,
I've noticed that the GCP's Project ID set to test. Do you really have such project?
If you are deploying your code directly to GCP, you can skip this property. It's only useful for local testing. The project name is set by CloudRun infra.
Hope this helps

Hi @t0xa , Yes. I've deployed it in GCP directly. I've also tried without this parameter and I'm also getting the same issue. Any idea what might be happening? Is there a specific quarkus version that I must use to make it work properly?

Hi @marcusricardoaguiar
I've just tried to create sample project via the https://code.quarkus.io/ and deployed it to new project in GCP. It works as expected.

What are the quarkus versions you are using?
Are you using any other GCP libraries in the pom?
Does the Cloud Run service account in the IAM section have custom roles/permissions, or is it just an Editor.

Hi @marcusricardoaguiar
I've just tried to create sample project via the https://code.quarkus.io/ and deployed it to new project in GCP. It works as expected.

What are the quarkus versions you are using?
Are you using any other GCP libraries in the pom?
Does the Cloud Run service account in the IAM section have custom roles/permissions, or is it just an Editor.

Hi @t0xa , I've just tried using the sample project you've shared and it still shows the issue. Just to clarify, I was able to deploy the application, but when I try to hit /hello endpoint it shows the warning error when trying to export the traces:

Don't you see any of these issues on your logs? Thanks!

Hi,
Could you please check you IAM settings. What are the permissions for<EMAIL_ADDRESS>principal?
Do you have billing connected and enabled for your project?
How are you building the docker image? Quarkus maven plugin?

One of the differences I see, is that you are building native image. Could you please try to build standard image?

One of the differences I see, is that you are building native image. Could you please try to build standard image?

Hi @t0xa, I was able to make it work just changing the mvn command to build/push the image:
 mvn package \
 	-DskipTests \
 	-Dquarkus.native.remote-container-build=true \
 	-Dquarkus.container-image.build=true \
 	-Dquarkus.container-image.registry="****" \
 	-Dquarkus.container-image.group="*****" \
 	-Dquarkus.container-image.tag=v1 \
 	-Dquarkus.container-image.name=hello-world \
 	-Dquarkus.container-image.push=true

Thank you so much!!!

No problem, but it looks like the native image doesn't work :(
I will try.

No problem, but it looks like the native image doesn't work :(
I will try to reproduce and fix this.

Hi @t0xa , do you have news about this issue? Let me know if you need help testing this one. Thanks!

We have a fix for the native image but we are stuck in a GraalVM bug now.


created Gesture Volume Control

Gesture Control to change the volume of a computer using opencv and python

@Pranshu-jais please share the results like @aviralgarg05 he did in #64

@Pranshu-jais attach this video in readme file like how results will looks like...after that will merge

i have already attached this video in readme file

@Pranshu-jais good


New Release 0.44.11?

It would be nice to release a new version. Because it would contain the capabilities support 6e214b261604c4ab1ffc244272443a587bb59927 and other security features.

I would enjoy a new 0.44.11 release for the seedrng entropy fixes, please?
bump.

As an Alpine Linux developer, I’m also eagerly awaiting the new release with the capabilities support.

Thanks


[Distributed] Tensor Parallel RMSNorm

Implement tensor-parallel RMS Norm for activations that are distributed along the hidden dimension.
Looks like we might want to do something like this for the RMS Norms on Q and K for the new OLMo model (#10503)

Currently running into the following issue with the unit test - If anyone has any tips here LMK
E           ValueError: Package size (864.37MiB) exceeds the maximum size of 500.00MiB. You can exclude large files using the 'excludes' option to the runtime_env or provide a remote URI of a zip file using protocols such as 's3://', 'https://' and so on, refer to https://docs.ray.io/en/latest/ray-core/handling-dependencies.html#api-reference.


helmfile: 0.167.1 -> 0.168.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/helmfile/helmfile/releases.
meta.description for helmfile is: Declarative spec for deploying Helm charts
meta.homepage for helmfile is: https://helmfile.readthedocs.io/
Updates performed

Golang update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 0.168.0 with grep in /nix/store/vc0pnvvrxgg8as95cjcc60j9m8rddvgl-helmfile-0.168.0
found 0.168.0 in filename of file in /nix/store/vc0pnvvrxgg8as95cjcc60j9m8rddvgl-helmfile-0.168.0

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

3 total rebuild path(s)

3 package rebuild(s)

First fifty rebuilds by attrpath

helmfile
helmfile-wrapped

Instructions to test this update (click to expand)

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

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

Or:
nix build github:r-ryantm/nixpkgs/ee32ff5edeb635e12407e62852182522487c7d98#helmfile

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

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

2 packages built:

helmfile
helmfile-wrapped

Maintainer pings
cc @kliu128 @yurrriq for testing.

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

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

lgtm on x86_64-linux


Add testing for different scenarios for basic zone implementation

Coverage increased (+2.5%) to 87.951% when pulling bd28bfd9095fa85489255c7ad724d32144481b6c on basic/testing into 1ff9a9a05e3296e9814e11068c6476ce833242a3 on master.

Coverage increased (+2.9%) to 88.293% when pulling 1e077948d4994dd9f32c2de775eafdaf8ca338fe on basic/testing into 1ff9a9a05e3296e9814e11068c6476ce833242a3 on master.


[flutter_tools] forward flutter format to dart format and deprecate

Description
re-implement support for flutter format by passing all arguments directly to dart format. Applies a deprecation warning for flutter format

FYI @devoncarew , thanks for the idea of just passing the arguments through - it works like a charm. Not sure if we should start including the warning now, but I would be nice to get users over sooner rather than later


Consider simplifying roadmap page

In the roadmap we currently have:

A list of "critical O/RM features" from the backlog

These are things we think we need before we can say EF Core is the recommended version of EF for everyone. Until we implement these features, although EF Core will be a valid option for many applications (especially on platforms like .NET Core where EF 6 does not work), for many applications the lack of these features will make EF 6 a better choice.

A list of "high priority features" from the backlog

There are many features on our backlog and this is by no means an exhaustive list. These features are high priority but we think EF Core would be a compelling release for the vast majority of applications without them:

Things we are actually intending to do in the next release

It seems that the difference between (1) and (2) may have become ambiguous, and as for everything that appears to be set in stone, I also think we should question it and review if they are all still valuable to have listed here :smile:
Perhaps we could merge the two, and possibly leave some of them to become regular backlog items, unless we believe there is value keeping them here.
Then it would be nice to find a way to structure things so that what we are doing in the next release and what we consider critical could be visualized together rather than having to look manually compare two different lists.

Also, update it to be consistent with issues we have "propose close" closed.


Don't set as default Dask cluster

Changing the default Dask cluster can have unwanted and unexpected side
effects for users.
Contributor Checklist:

[x] I have added myself to the creators.json file

Thanks, merged!


New custom markup

hello, I want to create an action that allows me to directly insert an embedded video, not thumnail , or html with a specific class.

Take a look at the previewRender option. It'll let you create your own custom rendering engine.

Thank colleague I will review , if you can help me with an example would be better for since I'm a rookie.

There are multiple examples in the README.


backend: patron_exists implemented

implemented patron exists
removed unused imports
fixed a bug with undefined params
closes #323

Tests fail also on travis as they do locally.
tests/api/conftest.py:144:
DetachedInstanceError: Instance <Transaction at 0x7fcfeeb34b10> is not bound to a Session; attribute refresh operation cannot proceed (Background on this error at: http://sqlalche.me/e/bhk3)
I took the initiative and removed unused imports from test_mails.py, it seems if I revert them the tests are passing, @ntarocco @zzacharo @equadon @vladbmihaescu anyone any thoughts or ideas? I don't think its all the dependencies but there is some magic going on there, I think the import of current_app is affecting it, will investigate a bit to see whats going on.


[FEATURE] User changelog entry for Profile progress

Description
Now we have a profile progress, we should let the users now about it in the user changelog page
https://github.com/EddieHubCommunity/LinkFree/blob/main/pages/changelog.js

Screenshots
No response
Additional information
No response

@eddiejaoude can i take this issue?


We should use strings for card values.

In this challenge, we're using the face values of cards which can't all be numerical. e.g. "J", "Q", "K", "A". We're not performing calculations in this challenge and we haven't learned string to integer conversion at this point. We have learned that case statements use strict equality. We should only be testing string values.  The automated tests use a mixture of string and integer values. The only version of the code that will pass uses integers for the numbered cards and strings for the face cards. Alternatively, a single line about the parseInt() function would ease confusion.
Challenge Counting Cards has an issue.
User Agent is: Mozilla/5.0 (Windows NT 10.0; WOW64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/54.0.2840.99 Safari/537.36.
Please describe how to reproduce this issue, and include links to screenshots if possible.
My code:

var count = 0;

function cc(card) {
  // Only change code below this line
  switch(card) {
    case 2:
    case 3:
    case 4:
    case 5:
    case 6:
      count++;
      break;
    case 10:
    case "J":
    case "Q":
    case "K":
    case "A":
      count--;
      break;
    default:
      break;
  }

  var decision;

  if (count > 0) {
    decision = count + " Bet";
  } else {
    decision = count + " Hold";
  }
  return decision;
  // Only change code above this line
}

// Add/remove calls to test your function.
// Note: Only the last will display
cc(10); cc("J"); cc("Q"); cc("K"); cc("A");

@VAggrippino thanks for the issue.

...we haven't learned string to integer conversion at this point.

This isn't too much of an issue because JavaScript is dynamically untyped language. That is why you can code 3 + " Bet" and the interpreter will not yell at you for putting together different types (i.e. numeric plus string).
I understand your concern for consistency in dealing with just strings, rather than two data types, but I don't think it is too much of a concern for this challenge.
The only clarifying change to the instructions could be to add in that the input parameter card can be a numeric or string.
cc/ @FreeCodeCamp/moderators

First-timers only
The code for this challenge is here.
The changes needs are:
You will write a card counting function. It will receive a card parameter , which can be a number or a string, and increment or decrement the global count variable according to the card's value (see table).
Following CONTRIBUTING.md will help you get set up and make your first PR to FreeCodeCamp :+1:
If you get stuck, reach out to us in Contributors chat room
Good Luck!

I initially quoted all my case values and didn't understand why I didn't pass until I looked more closely at the test function calls. I thought I should mention it just in case someone else approached the challenge from the same perspective as me.
Thank you for your response and your time. I haven't had many positive experiences with either support or bug reporting on the Internet and I wasn't really expecting any response. I'm pleasantly surprised.
@Bouncey Thank you for the encouragement, but I can't risk a pull request. Due to severe mental health / memory issues and occasional seizures I may not be able to be accountable for any changes I make because I'm likely to forget everything. For me, FCC isn't a means to learning skills for a career. It's just exercise for my failing mind.

@VAggrippino I was opening this issue up for anyone to create a PR. We would be grateful for any attempt to fix this, but if you are not up to it, someone else will create a PR. No pressure :+1:
Happy Coding!


JFYI: Outdated Dockerfile at docker hub

Hello!
You got outdated dockerfile at docker hub.
https://hub.docker.com/r/prom/alertmanager/~/dockerfile/
JFYI, feel free to close this issue.

It is a dupe of https://github.com/docker/hub-feedback/issues/1847. Unfortunately we got no update from the project :disappointed:


meshery-app-mesh/app-mesh/ all the files are named as Istio

In meshery-app-mesh "meshery-app-mesh/app-mesh/" folder all the files are named as Istio. it should be changed

@leecalcote sir, should i do the required changes here??

@ashiscs that would be wonderful, yes.


Error initializing X-Ray, one of the required parameters was blank.

Hey there, I was trying to generate an XRAY for one of my books and I got the following error:

An error occurred while building the X-Ray:
Error initializing X-Ray, one of the required parameters was blank.
at XRayBuilder.Core.XRay.XRay..ctor(String shelfari, String db, String guid, String asin, ISecondarySource dataSource)
at XRayBuilder.Core.XRay.Logic.XRayService.d__5.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
at System.Runtime.CompilerServices.ConfiguredTaskAwaitable`1.ConfiguredTaskAwaiter.GetResult()
at XRayBuilderGUI.UI.frmMain.<btnBuild_Run>d__35.MoveNext()

I think the problem is, that for some reason the xray builder is not generating a database name:

Extracting metadata...
Got metadata!
Database Name:
UniqueID:<PHONE_NUMBER>
ASIN: B003P8Q5QC
Book's Goodreads URL: https://www.goodreads.com/book/show/10371118
Attempting to build X-Ray...

Is there some way around this?
Thanks!

The database name comes from the book, it's not generated by the app. Weird that isn't blank but I suppose it shouldn't actually be required anymore anyway. Give this release a try and see if it works:
https://github.com/Ephemerality/xray-builder.gui/releases/tag/2.1.126

I'm still getting the following error:

Extracting metadata...
Got metadata!
Database Name:
UniqueID:<PHONE_NUMBER>
ASIN: B003P8Q5QC
Book's Goodreads URL: https://www.goodreads.com/book/show/10371118
Attempting to build X-Ray...
Loading terms from file...
An error occurred while building the X-Ray:
Error processing XML file: There is an error in XML document (2414, 1176).
If the error contains a (#, #), the first number is the line the error occurred on.
at XRayBuilder.Core.Libraries.Serialization.Xml.Util.XmlUtil.DeserializeFile[T](String filePath)
at XRayBuilder.Core.DataSources.Secondary.SecondarySourceFile.GetTermsAsync(String xmlFile, String asin, String tld, Boolean includeTopics, IProgressBar progress, CancellationToken cancellationToken)
at XRayBuilder.Core.XRay.Logic.XRayService.d__5.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
at System.Runtime.CompilerServices.ConfiguredTaskAwaitable`1.ConfiguredTaskAwaiter.GetResult()
at XRayBuilderGUI.UI.frmMain.<btnBuild_Run>d__32.MoveNext()

But that seems to be due to my XML file, generating XRAYs with Goodreads and Roentgen seems to be working now.

Yeah seems like there's something wrong with your XML on line 2414. Does it look like it's formatted correctly?

Yup, fixed it and it works fine now, thanks!


update masks.html & css

Submission Checklist
Required

[ ] The pull request title clearly contains the name of the sheet I am editing.
[ ] The pull request title clearly states the type of change I am submitting (New Sheet/New Feature/Bugfix/etc.).
[ ] The pull request makes changes to files in only one sub-folder.
[ ] The pull request does not contain changes to any json files in the translations folder (translation.json is permitted)

Changes / Description
Change color of the roll message
directly in the html with a div

Character Sheet Info Roll20 Internal Use only.


Ensure mounting of XXX filesystems is disabled check maybe wrong

Question
Hi,
In section_1/cis_1.1/cis_<IP_ADDRESS>_7.yml file all modprobe command contains jffs instead of proper module name.
  cramfs:
    title: <IP_ADDRESS> | Ensure mounting of cramfs filesystems is disabled
    exit-status: 0
    exec: if [ `lsmod | grep -c cramfs` != 0 ]; then  "modprobe -n -v jffs | grep -E '(cramfs|install)'"; else echo OK; fi

  freevxfs:
    title: <IP_ADDRESS> | Ensure mounting of freevxfs filesystems is disabled
    exit-status: 0
    exec: if [ `lsmod | grep -c freevxfs` != 0 ]; then  "modprobe -n -v jffs | grep -E '(freevxfs|install)'"; else echo OK; fi

...
and other

I have opened only question instead of bug ticket, because I'm not entirely sure is this a bug or not.
Environment (please complete the following information):

goss version: v0.3.18
OS version: Ubuntu 20.04

hi @arvatoth-vcc
Thank you for taking the time to feedback. Its only by feedback we can continue to improve. Apoloigies for the delay on this one client requirements take priorities. This is now incorporated into a new PR that we hope to release asap.
Hoping in the next couple of days to get this released.
best regards
uk-bolly


Support for Gradle's lazy configuration API

Currently the plugin is using the legacy "eager" task creation API. While this works, it has a number of drawbacks:

tasks are created and configured even if they won't be called
the Artifactory plugin tasks are expensive to create (they, for example, read all system properties at configuration time)
it's a prerequisite for this plugin to support Gradle's configuration cache

Expected behavior
The plugin should use the configuration avoidance API and lazy configuration API.
An unintented consequence of using the eager creation API is that any build which uses the Artifactory plugin will not be compatible with the configuration cache: even if you only execute test, for example, the Artifactory plugin will create a number of tasks and via the build info extractor plugin, read system properties in an unsafe manner.

To give some context, here are some of the issues reported by gradle when trying to use configuration cache

./gradlew --configuration-cache help

- Plugin class 'org.jfrog.gradle.plugin.artifactory.ArtifactoryPlugin': registration of listener on 'Gradle.addListener' is unsupported
  See https://docs.gradle.org/7.1.1/userguide/configuration_cache.html#config_cache:requirements:build_listeners
- Plugin class 'org.jfrog.gradle.plugin.artifactory.ArtifactoryPlugin': read system property 'sun.java.command'
  See https://docs.gradle.org/7.1.1/userguide/configuration_cache.html#config_cache:requirements:undeclared_sys_prop_read
- Plugin class 'org.jfrog.gradle.plugin.artifactory.ArtifactoryPlugin': read system property 'gopherProxySet'
  See https://docs.gradle.org/7.1.1/userguide/configuration_cache.html#config_cache:requirements:undeclared_sys_prop_read
- Plugin class 'org.jfrog.gradle.plugin.artifactory.ArtifactoryPlugin': read system property 'awt.toolkit'
  See https://docs.gradle.org/7.1.1/userguide/configuration_cache.html#config_cache:requirements:undeclared_sys_prop_read
- Plugin class 'org.jfrog.gradle.plugin.artifactory.ArtifactoryPlugin': read system property 'sun.cpu.isalist'
  See https://docs.gradle.org/7.1.1/userguide/configuration_cache.html#config_cache:requirements:undeclared_sys_prop_read
- Plugin class 'org.jfrog.gradle.plugin.artifactory.ArtifactoryPlugin': read system property 'sun.jnu.encoding'
  See https://docs.gradle.org/7.1.1/userguide/configuration_cache.html#config_cache:requirements:undeclared_sys_prop_read
- Plugin class 'org.jfrog.gradle.plugin.artifactory.ArtifactoryPlugin': read system property 'sun.arch.data.model'
  See https://docs.gradle.org/7.1.1/userguide/configuration_cache.html#config_cache:requirements:undeclared_sys_prop_read
- Plugin class 'org.jfrog.gradle.plugin.artifactory.ArtifactoryPlugin': read system property 'user.variant'
  See https://docs.gradle.org/7.1.1/userguide/configuration_cache.html#config_cache:requirements:undeclared_sys_prop_read
- Plugin class 'org.jfrog.gradle.plugin.artifactory.ArtifactoryPlugin': read system property 'user.timezone'
  See https://docs.gradle.org/7.1.1/userguide/configuration_cache.html#config_cache:requirements:undeclared_sys_prop_read
- Plugin class 'org.jfrog.gradle.plugin.artifactory.ArtifactoryPlugin': read system property 'user.country'
  See https://docs.gradle.org/7.1.1/userguide/configuration_cache.html#config_cache:requirements:undeclared_sys_prop_read
- Plugin class 'org.jfrog.gradle.plugin.artifactory.ArtifactoryPlugin': read system property 'sun.java.launcher'
  See https://docs.gradle.org/7.1.1/userguide/configuration_cache.html#config_cache:requirements:undeclared_sys_prop_read
- Plugin class 'org.jfrog.gradle.plugin.artifactory.ArtifactoryPlugin': read system property 'sun.boot.library.path'
  See https://docs.gradle.org/7.1.1/userguide/configuration_cache.html#config_cache:requirements:undeclared_sys_prop_read
- Plugin class 'org.jfrog.gradle.plugin.artifactory.ArtifactoryPlugin': read system property 'http.nonProxyHosts'
  See https://docs.gradle.org/7.1.1/userguide/configuration_cache.html#config_cache:requirements:undeclared_sys_prop_read
- Plugin class 'org.jfrog.gradle.plugin.artifactory.ArtifactoryPlugin': read system property 'jdk.debug'
  See https://docs.gradle.org/7.1.1/userguide/configuration_cache.html#config_cache:requirements:undeclared_sys_prop_read
- Plugin class 'org.jfrog.gradle.plugin.artifactory.ArtifactoryPlugin': read system property 'sun.cpu.endian'
  See https://docs.gradle.org/7.1.1/userguide/configuration_cache.html#config_cache:requirements:undeclared_sys_prop_read

It'd be a great save of build time for many developers if JFrog fixed this ❤

Why it is labeled as "gradle bug" instead of "used deprecated gradle api, required migration to new api"?

Any plans on fixing this?

We had to stop using the JFrog plugin for all the reasons noted by the OP. In large multi-project builds, the overheads became too much.

Same for us. The migration was super smooth, less than 1h of work.

Same, we've since migrated to the maven-publish plugin directly.

any hints or nuances on migration? Or are the main docs straight forward enough?

Dear @melix, @Maragues, @andreya108, @rpavliuk, @dmeibusch, and @Sineaggi,
Thank you for using the Gradle Artifactory plugin. We would like to inform you that we are currently in the process of releasing version 5 of the plugin. This major update involves several changes, including the translation of all Groovy code to Java, migration of the code to a new GitHub repository, and handling deprecations to facilitate maintenance of the plugin.
Once we have completed version 5, our next task is to add support for Gradle Lazy configuration and Configuration cache.
We apologize for the long wait and promise to keep you informed of any further developments.

@melix, @Maragues, @andreya108, @rpavliuk, @dmeibusch, and @Sineaggi
Gradle Artifactory plugin 5.1.0 is released.
In this version, we added support for task configuration avoidance. See https://github.com/jfrog/artifactory-gradle-plugin/pull/37.
We'd appreciate your feedback on that.

@yahavi Thanks a lot! We were waiting for this moment 🙂


Use CreateCa

Hello,
we need to add CA to use AAR
oapi-cli-x86_64.AppImage CreateCa --CaPem "$(cat ca-osc.pem)"
it doesn't seem to work

the issue should be fix with this PR: https://github.com/outscale/oapi-cli/pull/14
could you tets using the appimage generate here https://github.com/outscale/oapi-cli/actions/runs/4477789074
named oapi-cli-linux

should be fix, you can re-open the issue if it's not

OK with osc-cli --profile eu-west-2 api CreateCa --Description "My company CA222" --CaPem="$(awk -v quote="'" 'BEGIN {printf quote} {ORS="\\n"} END {printf quote}; 1' $HOME/.osc_sdk/cert/ca-osc.pem)"
{
"ResponseContext": {
"RequestId": "eb9c477a-25c7-4f91-896b-78e650d78af0"
},
"Ca": {
"Description": "My company CA222",
"CaId": "ca-ed5374b6fc",
"CaFingerprint": "fb36c30b340248eb401d54"
}
}
NOT OK with ./oapi-cli-v2.AppImage CreateCa --CaPem="$(awk -v quote="'" 'BEGIN {printf quote} {ORS="\\n"} END {printf quote}; 1' $HOME/.osc_sdk/cert/ca-osc.pem)" or ./oapi-cli-v2.AppImage CreateCa --CaPem="$(awk -v quote="'" 'BEGIN {printf quote} {ORS="\\n"} END {printf quote}; 1' $HOME/.osc_sdk/cert/ca-osc.pem)"
... kLCiPYXNq1otW9912dS3o/FwN75XlZXLPIuFLa9F5CTcDheSr5OQDJKA6tsj7b8f
IJ4p5Msi61fTi81E+35rRWVO6+YN3xZHRzdxZ2unZMx/
-----END CERTIFICATE----- is not a valide argument for 'CreateCa'


iOS build issue on latest version - v0.10.0

Version 0.9.0 was working just fine, the issue appeared in the last one.
React Native: 0.68.0
I checked the Swift version used on the project's build settings and it is Swift 5

The last two lines disappeared after I remove the node modules and pods, then reinstall them and clean the build folder using Xcode, but the Cannot find type 'Enumerable' in scope it's still there.

@alantoa any idea? I saw you changed to Enumerable in #12

I think we may need to upgrade expo modules

A bit confused though, @alantoa said it was working for him. I wonder if we could get a repro made?

@nandorojo yeah, I made the change because EnumArgument was deprecated in expo-modules-core  v1.0.x, the source code can be found here.
@juanchoperezj  so I guess you are using expo-modules-core version less than v1, right? if so, you need to upgrade it for it to work!

Thanks for the response @alantoa @nandorojo
My bad, I was using Expo 45, and now it works fine!
But I do not see two lines on the toast on iOS. Text is: In order to use the app, you must have internet connection.
Toast config:
  title: 'No internet connection',
  message: 'In order to use the app, you must have internet connection.',
  preset: 'error',
  duration: 5,
}

I believe that's a limitation of SPIndicator: https://github.com/ivanvorobei/SPIndicator/blob/main/Sources/SPIndicator/SPIndicatorView.swift#L155

This isn't necessarily bad: Toasts should be very short and easy-to-read.

Agree! I just wanted to bring it out, I'll close the issue! Thanks for the help!


Support deploying to multiple networks

Replaced by https://github.com/joinzien/scarce-editions/tree/edge


Bug: value of 0 for min throws an error

This check cannot rely on the falsy value:
https://github.com/ashnur/random-number/blob/master/index.js#L34
As it means that it uses options.min instead of the 0 passed in as min. I.e. if I create a generator with
var gen  = rn.generator( { min: 0, integer: true } );

and then try
var randomInt = gen( 0, 3 );

It throws this error:
node_modules/random-number/index.js:16
    if ( max < min ) throw new Error('invalid options, max must be >= min')

Thanks, will look into it.

Turns out I had it fixed locally a long time ago, I just completely forgot to push it :(


Packages from appstream did not migrate properly to ol8_appstream

``While updating a Centos-8 system to Oracle Linux 8 with a running docker container I have encountered an issue where a few packages did not switch from appstream to ol8_appstream .
# yum list installed | grep appstream | grep -v ol8
container-selinux.noarch             2:2.167.0-1.module_el8.5.0+911+f19012f9       @appstream
fuse-overlayfs.x86_64                1.7.1-1.module_el8.5.0+890+6b136101           @appstream
libslirp.x86_64                      4.4.0-1.module_el8.5.0+890+6b136101           @appstream
qemu-guest-agent.x86_64              15:4.2.0-59.module_el8.5.0+1063+c9b9feff.1    @appstream
slirp4netns.x86_64                   1.1.8-1.module_el8.5.0+890+6b136101           @appstream
#

A dnf repoquery on any of these packages results in nothing returned
# dnf repoquery container-selinux.noarch fuse-overlayfs.x86_64 libslirp.x86_64
Last metadata expiration check: 0:05:04 ago on Mon May  9 16:45:10 2022.
#

I was able to verify the packages exist in: https://yum.oracle.com/repo/OracleLinux/OL8/appstream/x86_64/index.html
I can query other packages just not these.
There are no excludes defined in the system
# pwd
/etc
# grep -r excludes
lvm/lvm.conf:   # identifies and excludes LVs.
telegraf/telegraf.conf:#   # excludes = [ "/dev/pass6" ]
telegraf/telegraf.conf:#   ## and all found will be included except for the excluded in excludes.
#

This issues does not occur when I migrate a system that does not have docker-ce installed.
The system is running but I will not be able to update these packages in the future.
Without uninstalling docker-ce and it's dependencies prior to updating to OL8, how can I get these packages to be associates with ol8_appstream

OK, transferred this issue to the appropriate repo.

The packages you list are part of the container-tools module stream in OL8. You'll probably need to do something like:
dnf module remove <module that currently provides these packages>
dnf module reset <module that currently provides these packages>
dnf module install container-tools

I tried your suggestion and receive this error message:
# dnf module install container-tools
Last metadata expiration check: 1:30:13 ago on Tue May 10 13:02:37 2022.
Installing module 'container-tools:rhel8' from Fail-Safe repository @modulefailsafe is not allowed
No default profiles for module container-tools:rhel8. Available profiles: common
Error: Installing module from Fail-Safe repository is not allowed

All of the modules I listed are in  ol8_appstream (x86-64) You can search for them here: https://yum.oracle.com/repo/OracleLinux/OL8/appstream/x86_64/index.html
Also on the systems that I have been testing migration on WITHOUT docker-ce installed I can query those packages:
# dnf repoquery fuse-overlayfs
Last metadata expiration check: 0:54:18 ago on Tue May 10 13:41:32 2022.
fuse-overlayfs-0:0.3-2.module+el8+5201+6423ecab.x86_64
fuse-overlayfs-0:0.3-2.module+el8.0.0+5215+77f672ad.x86_64
fuse-overlayfs-0:0.3-2.module+el8.0.0+5336+de35ffea.x86_64
fuse-overlayfs-0:0.3-2.module+el8.0.0.z+5239+bd2c200f.x86_64
fuse-overlayfs-0:0.4.1-1.module+el8.1.0+5440+994fc847.x86_64
fuse-overlayfs-0:0.7.2-1.module+el8.1.1+5502+fbec5cc6.x86_64
fuse-overlayfs-0:0.7.2-5.module+el8.1.1+5573+1c3f6079.x86_64
fuse-overlayfs-0:0.7.2-5.module+el8.2.0+5584+b2b2d3fb.x86_64
fuse-overlayfs-0:0.7.2-5.module+el8.2.0+7615+180dc822.x86_64
fuse-overlayfs-0:1.0.0-2.module+el8.2.1+7658+86e51d52.x86_64
fuse-overlayfs-0:1.1.2-3.module+el8.3.0+7866+f387f528.x86_64
fuse-overlayfs-0:1.3.0-2.module+el8.3.1+9659+c1901784.x86_64
fuse-overlayfs-0:1.4.0-2.module+el8.4.0+20157+b6591bfb.x86_64
fuse-overlayfs-0:1.4.0-3.module+el8.4.0+20195+0a4a4953.x86_64
fuse-overlayfs-0:1.6-1.module+el8.4.0+20289+730b73cc.x86_64
fuse-overlayfs-0:1.7.1-1.module+el8.5.0+20416+d687fed7.x86_64
fuse-overlayfs-0:1.8-1.module+el8.5.0+20494+0311868c.x86_64

Can you disable this repository? Fail-Safe repository @modulefailsafe

No, there is no repository with that name:
# pwd
/etc/yum.repos.d
# grep fail *repo
#

what's the output of:
dnf module list --installed
dnf module list --enabled
dnf repolist

I'd like @Djelibeybi's input on this.

This is one of the reasons we document the requirement to remove non-base RPMs prior to migration. The simplest fix is to rollback, remove docker-ce (and its dependencies) before migration and then run the script.

Removing all the software off a system to switch to your OS, then re-deploy any software above the OS does not make any sense. At that point it would be less work to just spin up a new VM on OL7/8 and redeploy the applications.
I will report back to my management and let them decide if they want to move ahead with oracle Linux.

The problem is that we can't control the dependencies that are pulled in by non-base packages. This is particularly problematic with CentOS because their versioning deviated from RHEL's standard.
This is a problem for all migration tools. The AlmaLinux Elevate package has a similar restriction: https://wiki.almalinux.org/elevate/ELevate-frequent-issues.html#known-issues

I understand, but I would make it much more clear that you cannot support Centos to Oracle migrations with anything but a base system. I would suggest adding the text of your last reply to the README file for people looking to migrate.
My management was hoping to avoid Centos Stream, which has migration issues also. We will have to look at a rolling replacement strategy if they want to migrate to Oracle Linux.

There is no need to continue this issue.
Thanks for the explaniation

You'll need a rolling replacement strategy to move to any other drop-in replacement distro, not just Oracle Linux, because they have the same constraints that we do.


Meta node only fixes

This fixes some issues that come up with creating meta-only nodes.

Monitor service defaults to storing statistics in the _internal database.  A meta-only node can't store anything itself so for now that config option has to be disabled.  The server will report that as a configuration error for now.
A data node was always getting created in the meta store regardless of whether the role of the node was actually a data node.  We now create the appropriate nodes based on the services that are enabled.
The join config file option was incorrectly exposed under the [meta] section as a string slice.  This has been revert and it's now a top-level string that matches the env var and -join flag (e.g.  join = 'host1:8091,host2:9091").

Fixes #5754

LGTM :+1:


NAS cmdlets

Description
Updates and adds cmdlets for NAS fileset support:
Get-RubrikFileset (updated)
Get-RubrikFilesetTemplate (updated)
Get-RubrikNASShare (new)
New-RubrikFileset (new)
New-RubrikFilesetTemplate (new)
New-RubrikNASShare (new)
Remove-RubrikNASShare (new)
Set-RubrikNASShare (new)
Related Issue
#190
Motivation and Context
Cmdlets did not support management of NAS shares or the creation of new filesets.
How Has This Been Tested?
@jasonburrell  tested with a POC customer along with internal validation/testing.
Types of changes
What types of changes does your code introduce? Put an x in all the boxes that apply:

[ ] Bug fix (non-breaking change which fixes an issue)
[X] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:
Go over all the following points, and put an x in all the boxes that apply. If you're unsure about any of these, don't hesitate to ask. We're here to help!

[X] My code follows the code style of this project.
[X] My change requires a change to the documentation.
[X] I have updated the documentation accordingly.
[X] I have read the CONTRIBUTION document.
[ ] I have updated the CHANGELOG file accordingly for the version that this merge modifies.
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.

:shipit:


restore argsz for DMA map/unmap

Signed-off-by: Thanos Makatos<EMAIL_ADDRESS>

This change breaks compatibility with QEMU; before merging we need to update the relevant QEMU code.

This PR should also contain the relevant doc changes where argsz is part of the request which must conform to the following:

Message sizes
Some requests have an argsz field. In a request, it defines the maximum expected reply payload size, which should be at least the size of the fixed reply payload headers defined here. For messages that don't have a reply payload, it defines the size of the incoming request (not including the standard header); otherwise it's not related to the request message size.
In a reply, the server sets argsz field to the size needed for a full payload size. This may be less than the requested maximum size. This may be larger than the requested maximum size: in that case, the full payload is not included in the reply, but the argsz field in the reply indicates the needed size, allowing a client to allocate a larger buffer for holding the reply before trying again.

We should restore the original vfio_iommu_type1_dma_map / vfio_iommu_type1_dma_unmap according to https://github.com/oracle/qemu/commit/9baf27a8fb99392a4ff85071670cd40cec1ef4af.

@jlevon I've rewritten DMA map/unmap to match VFIO's, please take another look.

@jlevon I've addressed your comments, please take another look.

@jlevon I just realized that my implementation is wrong: if VFIO_DMA_UNMAP_FLAG_GET_DIRTY_BITMAP is set but argsz is too small then we don't return the bitmap but proceed with unmapping the DMA region. The client can't retry the request since the DMA region will have been removed, so there's no way to retrieve the dirty page bitmap. I'll check to see what VFIO does in this case.

Good point, I hadn't thought this through properly. We can't return a payload and an error, so I think we should go back to your earlier version that requires a valid argsz after all. Sorry for asking you to do that :)

Good point, I hadn't thought this through properly. We can't return a payload and an error, so I think we should go back to your earlier version that requires a valid argsz after all. Sorry for asking you to do that :)

no problem


gMSA is officially supported now

Hi Doc Maintainers,
On this page it is stated that gMSA is in the Alpha test of Kubernetes, but it is now officially in v1.18 stable. So please update the docs with that 😉
The link in the article actually refers to the correct page with:
FEATURE STATE: Kubernetes v1.18 [stable]
Friendly Regards
Stefan B. Christensen
[Enter feedback here]

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

ID: 79d3cfcb-855f-3389-17db-44e5d97e1059
Version Independent ID: 7556c803-8842-9990-a9dd-8757a81c1cbd
Content: Orchestrate containers with a gMSA
Content Source: virtualization/windowscontainers/manage-containers/gmsa-orchestrate-containers.md
Service: windows-containers
Product: windows-containers
GitHub Login: @rpsqrd

/cc @immuzz and @weijuans-msft

We hugely appreciate your contribution, feedback, and support to our technical documentation.
We’ve phased out GitHub Issues on this repository as the feedback mechanism for content and have replaced it with a new feedback system. For more information see, Content User Feedback. As part of this rollout, we are migrating your issue where our team will use your comments to improve the documentation experience. Unfortunately, this does mean you won't be able to access this issue after the migration is complete.
All issues will be migrated by Thursday 29th August 2024.
Once again, thank you for your valuable feedback, we hope you’ll continue to contribute using our new feedback system.


Add missing space in confirm dialog

Before:

After:

Obviously no need for the PR reward for this, just bothered me for a while :P
Edit:
Or ist this even wanted? Just saw that there's also no space for other currencies in the client... well if that's the case, it can be closed obviously.

@j-a-m-l
I know what you mean.
We both agree that something has to be done, but probably not in this PR.
If you prefer the currency format setting together with the date that's also fine by me.
=> Maybe we can close this and create an issue instead for this?

@Nasicus I can't reply right now about it, but I'll do it ASAP

I think that we could close this issue, although it has been a good discussion that would be useful for the next wallet version.


tutorial data and script are outdated

the example data are outdated. For example, https://github.com/openforcefield/openff-evaluator/blob/1e1a2d06e5a01d0f3262518a648996314fdb8276/openff/evaluator/data/tutorials/tutorial02/estimated_data_set.json#L241 here openff.evaluator.utils.statistics no longer exists
https://github.com/openforcefield/openff-evaluator/blob/main/docs/tutorials/tutorial04.ipynb this script uses some old api of openff-evaluator and openff-toolkit and no longer works.

I believe this is closed with #499 and a release, please re-open if it hasn't been resolved


DynamoDB Filters with map objects

Hi,
I connected an AWS dynamoDB and I try to filter map objects but I get an error message  "ExpressionAttributeNames contains invalid key: Syntax error; key: "#environment.item" (Service: DynamoDb, Status Code: 400, Request ID: TVDIN0SGF7JSFBR1HAA3006BI3VV4KQNSO5AEMVJF66Q9ASUAAJG)".
You can see on screenshot DBeaver 1 the filter and on screensho DBeaver 2 the error.
May you help me solve this error ?
Thanks

thanks for the report

Fixed.
Now you can use maps (aka nested attributes) in both table filters and SQL queries.
E.g.
SELECT *
FROM Movies WHERE info.rating>8;

verified


HWY-275: Do not schedule same vertex for synchronization more than once.

For some reason dev was broken and clippy started failing for me on totally unrelated changes. Fixed in c7b0a0d (#1305)

This seems to prevent later senders (i.e. alternative sources for a vertex) from being added to vertices_to_be_added.
In general: Currently, if Alice and Bob send us a vertex with a dependency, we'd request the dependency from both, so if Alice doesn't answer, we receive it from Bob.
With this change, if Alice sent it first, would we ignore Bob? Would be be stuck now if Alice doesn't answer, because we'd never request it from Bob?

Ah, you're right! I don't know what I was thinking.


Package Version update

#516  Issue number
Relevant issue number

Resolves #

Please check the following

[ ] Do the tests still pass? (see Run the Tests)
[ ] Is the code formatted properly? (see Linting (Formatting))
For New Features:

[ ] Have tests been added to cover any new features or fixes?
[ ] Has the documentation been updated accordingly?

Please describe additional details for testing this change

@jerryh-jr thank you it will be good to update. There are three other files that specify node versions which should be updated:

.nvmrc
.github/workflows/npm-run-script.yml
.github/workflows/deploy-storybook.yml

Can you update these to match the other two, fix the merge conflict in pacakge-lock.json, and use npm run version:patch to bump the app version?

Hey @itsalaidbacklife  Will work on this now. Sorry for that.

Closing in lieu of https://github.com/cuttle-cards/cuttle/pull/659


Desktop Application not Client

https://treehouse.spideroak.com/pandora/wiki/Marketing/StyleGuide says "Application" but we still use "client" in every language in both mobile apps as of blue 3.3.0 rc1
AFAIK this is only relevant in the first login with a given account Zero-Knowledge warning

Added the new "translations" label because any changes to text content will require translations.


Can I use umami with Electron app?

You can simply add as site but I would like to use umami with an electron app, does it possible?
Thanks

Yes, Electron should be able to run any web app.

I need help, umami needs a website or domain thing. I don't have an electron app based on website (basically a note taking app), so how can I use it then?
Thanks

I haven't tried it, but I think simply adding the tracking code to your electron app should work. What error are you seeing?

From my quick testing Umami works in an Electron app but would need some customization to get page views working, as it just reports on main_window views:


Close #801 - peak detector min datapoints

Closes -- https://github.com/cantino/huginn/issues/801
Testing -- ran rspec
About --
Adds an option min_events to the peak detector agent. The
agent will start looking for peaks only after min number of
events are accumulated.

Thank you, @dsander. I've added a db migration, similar to the one you mentioned.
I've tested the db migrating forwards/backwards on a local MySQL, observing the min_events option being accordingly added/removed.

Thanks @aeter!

Thanks, looks good!


Jenkins: Enable jemalloc for benchmark build on Jenkins

This uses the new CMake flag to dynamically link jemalloc only for the benchmark build on Jenkins. Assuming the numbers make sense and it appears to be working correctly, we'll update the nightly build pipeline and reset the benchmark lookback next week.

Select highlights from the benchmark suite:
pass 1.187e+06 9.007e+05 10     +31    history  10   GarbageCollectorBenchmark    UnlinkTime
pass 5.874e+06 3.398e+06 10     +72    history  10   GarbageCollectorBenchmark    ReclaimTime
pass 6.562e+04 5.401e+04 10     +21    history  10   TPCCBenchmark                ScaleFactor4WithLogging
pass 6.794e+04 5.716e+04 10     +18    history  10   TPCCBenchmark                ScaleFactor4WithoutLogging

The UnlinkTime benchmark doesn't make sense to me and makes me wonder if the data structures used to store the txns are internally using malloc and free frequently. They're forward_lists, which would make sense.


yt search fails due to argcomplete version conflict

Problem
I updated pip (8.1.0) then installed yotta (0.14.2) on a fresh Ubuntu 15 machine. When I run yt search it errors out due to version conflicts with argcomplete
yt search something
Fatal Exception, yotta=unknown
Traceback (most recent call last):
  File "/usr/local/bin/yt", line 4, in <module>
    yotta.main()
  File "/usr/local/lib/python2.7/dist-packages/yotta/main.py", line 201, in main
    status = args.command(args, following_args)
  File "/usr/local/lib/python2.7/dist-packages/yotta/search.py", line 125, in execCommand
    for result in itertools.islice(registry_access.search(query=args.query, keywords=args.kw, registry=args.registry), args.limit):
  File "/usr/local/lib/python2.7/dist-packages/yotta/lib/registry_access.py", line 683, in search
    headers = _headersForRegistry(registry)
  File "/usr/local/lib/python2.7/dist-packages/yotta/lib/registry_access.py", line 414, in _headersForRegistry
    'X-Yotta-Client-Version': _getYottaVersion(),
  File "/usr/local/lib/python2.7/dist-packages/yotta/lib/registry_access.py", line 397, in _getYottaVersion
    _yotta_version = pkg_resources.require("yotta")[0].version
  File "/usr/local/lib/python2.7/dist-packages/pkg_resources/__init__.py", line 943, in require
    needed = self.resolve(parse_requirements(requirements))
  File "/usr/local/lib/python2.7/dist-packages/pkg_resources/__init__.py", line 835, in resolve
    raise VersionConflict(dist, req).with_context(dependent_req)
pkg_resources.ContextualVersionConflict: (argcomplete 1.1.0 (/usr/local/lib/python2.7/dist-packages), Requirement.parse('argcomplete<1.0,>=0.8.0'), set(['yotta']))

Solution
Need to update / fix yotta so it can use newer versions of the argcomplete package as that is the new default for newer systems.
Alternatively we could force all users to install yotta in a VE, but that seems impractical due to the scope of yotta.

Thanks for reporting – do you know if things work OK with argcomplete 1.1.0 if you relax the restriction in setup.py?

I've also had quite a few issues with updating yotta, caused by problems similar to this one. Since pip seems to have problems with updating dependencies, I'm wondering if yotta shouldn't pin dependency number (argcomplete==1.1.0 and so on) instead of specifying a range of dependencies.

We don't pin them, we specify compatible ranges. In this case the range is argcomplete<1.0,>=0.8.0
Please report any issues like this, so we can keep the ranges up to date!

We don't pin them, we specify compatible ranges

I know, and I'm saying that maybe we shouldn't. Not necessarily because of the way we specify dependency ranges, but because lots of times pip behaves unpredictably when I do pip install -U yotta (one or two times it behaved so unpredictably that I had to reinstall Python completely on my Windows box). Also, if you pin versions, it's much easier for people to reproduce the same yotta version you published. Updating the pinned numbers can happen when a new yotta version is released, for example. Just my £0.02.

Ah double negative :)
Pinning exact versions will mean requiring installation in a virtual env, otherwise conflicts are inevitable. While reproduction of the exact version is a nice idea, I think it is less important than making sure people get the latest bugfixes (and potentially security updates) from dependencies when they install.
Basically the same reasoning that applies to dependency versions within yotta itself.
I can't recall an instance where an updated dependency broke yotta by breaking functionality* (only cases like this were we should have been more relaxed in the versions we allow updates to ;)

well not quite true, there was one case where a dependency update broke a test case in the CI (the Travis CI will use the latest matching versions of dependencies on each build)

You're right, I should've mentioned venvs earlier. I think that's a good idea.
I'm not talking about updates that broke yotta, I'm talking about updates that brought pip and hence Python. That actually happened :)

@autopulated I have not tested further

should be fixed by #743


Support multiple users within Bailo

Known issues:

Users come up as 'datasets' on the model view page.
No migration script for MongoDB
No migration script for existing Docker deployments

Now just need a Docker migration script.


[DOC] Fixed outdated programming-guide link

The link was not working.

Can one of the admins verify this patch?

The Spark docs are designed to be viewed via the rendered docs at spark.apache.org, not through GitHub, so the current link is correct.

Ok, no wonder I saw more invalid links. Thanks though.


More up to date Go?

The canonical version of Go (as in, not gccgo) on godbolt is a little out of date.  Go is currently on version 1.9, and has undergone significant changes since 1.4 - in particular, a new SSA back-end has been added as of 1.7, so the differences should be significant.

Ey! Thanks for this! Go really did need some attention at CE.
I've been looking into it and there's a good reason for 1.4 to be the latest version here. Since 1.5, Go moved to a unified tool instead of the old g4 g6 etc, which makes it harder for us to implement it
But! That's not saying it's not possible to implement. We'll just need more time! For the time being tho, I'll add the latest gccgo version (The one that comes with GCC 7.2.0, which if https://golang.org/doc/install/gccgo#Releases is accurate, should have a Go 1.8 implementation) in the hopes that that's ok/enough while we can figure out a way to implement the newer versions
I can't however give you an ETA, sorry!

Hi there.
Thank you for your response.  Sorry for the long delay in responding, but
it is possible to make modern Go dump assembly.
https://golang.org/cmd/compile/
Are there other things necessary for your tool?  I ask this from a place of
ignorance.  If there are missing pieces, I'm sure the Go developers might
consider your requests, as your tool is incredibly awesome.
The problem with GCCGo is that in my experience it doesn't seem to get a
lot of use in the greater Go community, and the output of the two compilers
aren't comparable.
-Alex
On Sun, Sep 24, 2017 at 4:18 PM, Rubén<EMAIL_ADDRESS>wrote:

Ey! Thanks for this! Go really did need some attention at CE.
I've been looking into it and there's a good reason for 1.4 to be the
latest version here. Since 1.5, Go moved to a unified tool instead of the
old g4 g6 etc, which makes it harder for us to implement it
But! That's not saying it's not possible to implement. We'll just need
more time! For the time being tho, I'll add the latest gccgo version (The
one that comes with GCC 7.2.0, which if https://golang.org/doc/
install/gccgo#Releases is accurate, should have a Go 1.8 implementation)
in the hopes that that's ok/enough while we can figure out a way to
implement the newer versions
I can't however give you an ETA, sorry!
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/mattgodbolt/compiler-explorer/issues/536#issuecomment-331737384,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AABcC_IOgBaKcrISZl2UeQv1NhLp534_ks5slrkMgaJpZM4Phppc
.

That looks about right. CE expects to be able to run a single command and have it "just work" to generate ASM. As long as we can run go tool compile ... with a different go command for each version of Go, we're in business. Last time I looked I had to define a bunch of environment variables per Go version to point at different roots, and that made it less trivial to add. Is that still the case?

At least in my quick testing (Go 1.7.3 installed from the website on Linux), you do not need environment variables for getting a source listing for an arbitrary random .go file.
go tool compile -S -S test.go
"".main t=1 size=179 args=0x0 locals=0x58
        0x0000 00000 (test.go:5)        TEXT    "".main(SB), $88-0
        0x0000 00000 (test.go:5)        MOVQ    (TLS), CX
... elided ...

Go's assembler output is kind of odd, and is not a 1:1 representation of the underlying instructions for a particular architecture, but it is what Go developers expect to see.  The assembler is documented in more detail here, if you're curious about its idiosyncrasies or need some guidance.
Since Go 1.5, there are environment variables that you can set to output assembly for different platforms and environments.  go tool dist list lists them all, and you can set GOARCH and GOOS.  However, I've only messed around with this a little bit, and I think one must ensure that the standard library exists for that architecture/environment combination before using the compile tool.  I will update this issue if I have more insight later.

As far as multiple Go versions are concerned, I found this post that recommended compiling Go from source.  The directions don't look too terribly difficult to follow, and when you're done, you don't even need to change GOROOT or anything similar.

Uhg. Sorry for taking so damn long
Here are a few interesting points:
Turns out GoLang includes pkg/tool/linux_amd64/compile which lets you directly compile code (As far as I can tell!) so there's no reason to call go with different env variables.
Between 1.4.1 and 1.8.x, the assembly output changed a bit so we needed a new filter to remove boring parts of the outputted asm
Now, the new compilers are: 1.7.2, 1.8.5, 1.9.2, and in the future, it will be much easier to add newer ones.
I've called them "x86-64 GoLang 1.X.X" for the lack of a better name. I can however change this, so if you think a better naming for them could work, please don't hesitate in suggesting them!
I want to thank you for pushing us towards better Go support on CE, and for being able to wait almost 2 months without a response
:)
PD: I'm closing the issue but feel free to answer here if you have any more questions!

Pushing to beta right now so you can take a look

( https://go.godbolt.org/beta ... https://godbolt.org/g/hWmKzB for example)

Wow, this looks pretty spiffy.  Thanks for taking the time to implement this.
Unfortunately, it appears that I can't import the stdlib for some reason.
package main

import (
	"fmt"
)

func main() {
	fmt.Println("Hello, playground")
}

Trying to run this on godbolt results in "can't find import fmt" errors.

As for naming, it appears that the golang website uses the term gc to refer to the canonical compiler in several locations, as opposed to gccgo.  It calls the architecture (GOARCH) 386 or amd64 depending on if it's x86 or x86-64.  It also calls the operating system (GOOS) linux.
I would change the compiler name at the very least.  Architecture - it's up to you.

@AlexMax thanks for checking it out. Re standard libraries, seems like it's the problem 6g has had: #69 . I'm not knowledgeable enough in how go works to start fixing this, any advice appreciated.
With respect to naming, we'd be happy to rename to gc to make it more consistent with the naming convention. Thanks for your input!

So, we are going places!

Needs some bugfixing before I can push it, but we're closer than ever!

Ok, so I've hit a roadblock with 1.8.and 1.7. With the newer 1.9, go is able to guess the GOROOT variable automatically (https://golang.org/doc/go1.9#goroot) which makes it possible for us to use it at all.
Older versions need either a GOROOT env variable (Which is out of the question for us) OR to be built from source, where you can set a custom GOROOT_FINAL variable that wil be backed into the binaries (As the article you linked mentions)
So I guess next step here is to build 1.7 and 1.8 from source, but as we do support the newer version, I think we can safely mark this issue as completed, and create a new one to track there the building from source.
We do have one problem tho. 1.4 can either:

Stay like it's been until now: Libraries won't work (We can't set env variables with mroe than 1 version of go installed!)
Remove it and have old links use the 1.9.2 version
Remove it, break old links -> Not gonna happen. Url should live forever!

If anyone has any idea, please let me know

We can try and supply a GOROOT when we execute the compiler. The environment is already modified to include some GCC stuff like LIBRARY_PATH for multiarch support...so it's possible!


[Boost-regex] Build error "stdio.h:83:8: error: 'FILE' does not name a type"

Operating system
MacOS Sonoma 14.6.1 - Intel-based Mac only
Compiler
gcc 12.4.0 (Homebrew)
Steps to reproduce the behavior
cmake .

Failure logs
Installing 40/64<EMAIL_ADDRESS>Building<EMAIL_ADDRESS>/Users/alpha/vcpkg/buildtrees/versioning_/versions/boost-regex/6f5d77fa3adaafae00b56ca0072d56ef4c4836a4: info: installing overlay port from here
-- Downloading https://github.com/boostorg/regex/archive/boost-1.85.0.tar.gz -> boostorg-regex-boost-1.85.0.tar.gz...
-- Extracting source /Users/alpha/vcpkg/downloads/boostorg-regex-boost-1.85.0.tar.gz
-- Applying patch compat.diff
-- Using source at /Users/alpha/vcpkg/buildtrees/boost-regex/src/ost-1.85.0-afd9b213e1.clean
-- Configuring x64-osx
-- Building x64-osx-dbg
CMake Error at scripts/cmake/vcpkg_execute_build_process.cmake:134 (message):
Command failed: /usr/local/Cellar/cmake/3.30.3/bin/cmake --build . --config Debug --target install -- -v -j3
Working Directory: /Users/alpha/vcpkg/buildtrees/boost-regex/x64-osx-dbg
See logs for more information:
/Users/alpha/vcpkg/buildtrees/boost-regex/install-x64-osx-dbg-out.log
Call Stack (most recent call first):
/Users/alpha/code/UnmanagedInfra/.m/vcpkg_installed/x64-osx/share/vcpkg-cmake/vcpkg_cmake_build.cmake:74 (vcpkg_execute_build_process)
/Users/alpha/code/UnmanagedInfra/.m/vcpkg_installed/x64-osx/share/vcpkg-cmake/vcpkg_cmake_install.cmake:16 (vcpkg_cmake_build)
/Users/alpha/code/UnmanagedInfra/.m/vcpkg_installed/x64-osx/share/vcpkg-boost/boost-install.cmake:65 (vcpkg_cmake_install)
buildtrees/versioning_/versions/boost-regex/6f5d77fa3adaafae00b56ca0072d56ef4c4836a4/portfile.cmake:15 (boost_configure_and_install)
scripts/ports.cmake:192 (include)
install-x64-osx-dbg-out.log
Additional context
Same project & compiler configurations works on Arm-based Mac

In file included from /Library/Developer/CommandLineTools/SDKs/MacOSX15.0.sdk/usr/include/Availability.h:178,
                 from /Library/Developer/CommandLineTools/SDKs/MacOSX15.0.sdk/usr/include/_stdlib.h:61,
                 from /Library/Developer/CommandLineTools/SDKs/MacOSX15.0.sdk/usr/include/stdlib.h:58,
                 from /usr/local/Cellar/gcc@12/12.4.0/include/c++/12/cstdlib:75,
                 from /usr/local/Cellar/gcc@12/12.4.0/include/c++/12/stdlib.h:36,
                 from /Library/Developer/CommandLineTools/SDKs/MacOSX15.0.sdk/usr/include/assert.h:44:
/Library/Developer/CommandLineTools/SDKs/MacOSX15.0.sdk/usr/include/_wchar.h:215:47: error: attributes after parenthesized initializer ignored [-fpermissive]
  215 | wchar_t *fgetwln(FILE * __restrict, size_t *) __OSX_AVAILABLE_STARTING(__MAC_10_7, __IPHONE_4_3);
      |                                               ^~~~~~~~~~~~~~~~~~~~~~~~
/Library/Developer/CommandLineTools/SDKs/MacOSX15.0.sdk/usr/include/_wchar.h:215:47: error: expression list treated as compound expression in initializer [-fpermissive]
In file included from /usr/local/Cellar/gcc@12/12.4.0/include/c++/12/ext/string_conversions.h:43,
                 from /usr/local/Cellar/gcc@12/12.4.0/include/c++/12/bits/basic_string.h:3972,
                 from /usr/local/Cellar/gcc@12/12.4.0/include/c++/12/string:53:
/usr/local/Cellar/gcc@12/12.4.0/include/c++/12/cstdio:98:11: error: 'FILE' has not been declared in '::'
   98 |   using ::FILE;
      |           ^~~~

Any vcpkg file involved at all? Maybe a bug in homebrew's gcc.
Not vcpkg's supported x64-osx (Apple clang).

The issue is related to MacOSX SDK 15 - it won't replicate for SDK 13.3 . It is not related just to Vcpkg, @MonicaLiu0311 I do apologize for wrong addressing.
I'm not sure it happens for clang as well, I tried only on gcc@12.
My Arm-based mac was not updated for 15, and since by default cmake takes the last one, I was able to see that issue only on Intel-based mac that was updated.


Add Flush method in the CommandTextCache, and exposed the class.

The code changes has been checked-in to the master branch and will be available on version > v1.9.2.


Read digital channels from rhd

Hi, I’m using IntanIO to read form rhd files, which works well but does not read digital channels. Am I missing something or is this still to-be-implemented?
Just as a reference, Intantech provides python scripts that read those files and extract basically everything in nice dictionaries: http://intantech.com/downloads.html?tabSelect=Software

Hi.  I am also interested in this feature if it becomes available.  Thanks!

Hi all.
I am the autor of this IO but I never use it.
Reading this
https://github.com/NeuralEnsemble/python-neo/blob/master/neo/rawio/intanrawio.py#L522
suggest that it is implemented at header parsing level but thoses channel are not exposed to
_get_analogsignal_chunk and not expose in the header['signal_channels']
If someone have time to do it it would be great.
Do you plan to have expose thoeses channels as analogsignal (with 0 and 1) or via events (that would need a full read of the entire file) ?

For me, either an analogsignal or events would be fine.  If I had it as as analogsignal, most likely I would then convert it into an events, but this should be straightforward.

Some progress towards this in #1249

@CodyCBakerPhD , @luiztauffer , #1249 is now merged. What are the next steps here?

@luiztauffer what project did you need this for? (If you can remember) - this issue can likely be closed
A new release of neo would be great @samuelgarcia
I suppose confirmation that it works in SI to close https://github.com/SpikeInterface/spikeinterface/issues/1876 is the small 'next step'
Otherwise, @weiglszonja has a current project that needs this so they will be making the data interface on NeuroConv: https://github.com/catalystneuro/neuroconv/issues/519


Enable clippy::cargo and/or unused_crate_dependencies lints for every workspace

As the title says, it would be nice to have these lints enabled for every workspace
more context : https://github.com/ethereum/trin/pull/575#discussion_r1140504504
UNUSED_CRATE_DEPENDENCIES seems to have some peculiar edge cases

I tried to enable clippy::cargo or unused_crate_dependencies for all workspaces. But in some cases, both linters have false positives, and this cause a bunch of lint errors in Trin.

clippy::cargo

multiple_crate_versions is one of lint in clippy::cargo. It has a known issue for errors caused by the dependencies themselves.
Like in Trin, reth-primitives has a dependency on<EMAIL_ADDRESS>but eth_trie has a dependency on hashbrown @0.3.1, then the lint will issue multiple versions for dependency hashbrown.
Related Issue#9756 on clippy.

unused_crate_dependencies

As docs said, it has a known issue for multiple create in a cargo package.
Like in the Trin package (trin/src), the package has a dependency on anyhow. anyhow is used in bin/purge_db.rs but not in lib.rs, then the linter will incorrectly issue anyhow is unused in the lib.rs.
Related Issue#95513 on rust.

@e3243eric Thanks for looking into this! Yeah, I've played around with these and was never able to find a stable configuration, I ran into a lot of the errors you're describing. So it's nice to get some confirmation. I wasn't aware of the multiple_crate_versions lint, and it would be so nice to get something like that in...
If I have a pr in which I suspect that there are some unused crate dependencies, what I'll do is plug in #[warn(unused_crate_dependencies)] lint in the lib.rs of the specific workspace. Then I'll run the linter locally. And it's usually pretty good about telling me if there are any unused dependencies. Of course, there are also some false positives, but I just ignore those.
My question is... do you have any ideas / could you look into an "optional" lint configuration? In an ideal world, it would be a lint I would run locally (probably just before merging a pr), but it wouldn't be included in our ci build (so we don't have to worry about the issues breaking the build). This "optional" lint configuration would lint for unused_crate_dependencies & multiple_crate_version. Tbh, I'm not so concerned about verify the clippy::cargo lint, but if it can be included as well, that would be nice.
Honestly, it might even be best to just add a command to the Makefile that will lint specifically for the above mentioned lints. Then this is something that could easily be run locally and wouldn't affect our ci builds


feat: Name test callbacks for profiling

🚀 Feature Proposal
Assign the test name to the test callback if the test callback is not named.
Naively add before https://github.com/facebook/jest/blob/eff3eaaad2bde223365da6522cfac7960267fb46/packages/jest-circus/src/index.ts#L182
let actualName = testName;
if (fn.name === "") {
    Object.defineProperty(fn, 'name', {
        get() {
            return actualName
        },
        set(name) {
            actualName = name
        }
    })
}

Motivation
The passed callback to e.g. test('my test description', () => { /* the callback */ }) is hard to find when profiling tests. Right now we have to explicitly name them via test('my test description', function mySlowTest() { /* the callback */ }).
Example
@kentcdodds encountered the same issue and made a video explaining the issue: https://www.youtube.com/watch?v=RB2g-o39upo.
Pitch
Reduces barrier to entry into profiling jest tests. While I would consider naming functions a tool you should know about for profiling, I would always prefer to not change code when profiling/debugging.

Comment

Bump

Bump

Yeah, I don't think this would hurt anything - happy to take a PR 👍


Color functions and mixins

Adds brightness function: Different than Sass's "lightness," see the test for an explanation
Includes a variety of other color functions that need to be documented/included in tests, like color-difference, safe-saturate (will not saturate black/white to red as the default behavior does), luminosity, contrast-ratio, etc.
Includes a few text effects, though these will be moved
Tests included

Another great PR.
Just put the code into a shorter filename and I'll merge this.

Well done !


Limit amount of unique URL hashes collected and display that in the report

Limit the amount of unique url hashes collected to prevent memory from overflowing. Introduce a property to limit or relax that settings. Optionally consider using a soft reference approach so that data can be released if under memory pressure. In that case, the stats should express something like "N/A unique urls" or 3711 unique urls (some data dropped).
The memory increase as part of #117 should help to avoid this situation most of the time. Only tight memory settings mostly cause this overflow.

Check HyperLogLog usage for that. Might give what we need: https://github.com/aggregateknowledge/java-hll

Using the HLL changes the correctness of the distinct count for large distinct counts over 100k a little. Tests have shows deviations of up to 0.5%, up to 1% for well over 1 million distinct values. All smaller values are correct and comparable to the previous system of int-hashcode storage and counting.
We save a lot of memory and the processing speed increases because the HLL structure is more memory-efficient.
Old way of doing things

HLL version

We also show the distinct count now directly on the request tab and removed the 1/d value instead.

Another implementation for later evaluation is https://github.com/prasanthj/hyperloglog


Updated MongoDB Operator to 0.6

This updates the ClusterServiceVersion file for the MongoDB operator to latest release, just pushed to RedHat's Container Catalog.
This latest version of the MongoDB Operator has been rebuilt on top of the operator-sdk.

/approve

/ok-to-test

/skip gitlab-ci/pipeline
/lgtm


Updating curry definition/example

This is more complex a definition, but I think it's more accurate.

I was able to understand better though. +1


500 ERROR is occurred again on Method

Hi,
I find out 500 Error is occurred again after I input your company info and companies info.
It was disappeared when I followed below step.
php artisan db:seed --class=PermissionTableSeeder
php artisan db:seed --class=CreateAdminUserSeeder
instead of
php artisan db:seed
during first dev install.
The Laravel log is attached
laravel.log

Damn, you are a good tester!
Do you know how it came about? did you record any data?
Could you capture the data in the tables for me?
or you just run the commands and error?

Hi, I tested from git download and followed all install way.
https://github.com/SMEWebify/WebErpMesv2/wiki/Installation-Steps-(for-dev)

Input your company info
Add company info
Add Lead info

Then, Method is displaying without 500 Error.
It seams I input something wrong during I cannot find how to input Company address and contact name.
Anyhow I am OK now!!
Thank you!!

ok, I found the problem, dbseed insert user id data from I don't know where even though there is no user in the table
is now fix


Capture the throughput and compare it to expectations in the subscriber example app

Could we also capture the throughput and compare it to expectations? We have seen that older phones also publish less messages in general (assumed to be due to older CPU, older Android versions...). It would be good to surface a throughput rating.
I am thinking something like 60 messages per minute could be 5/5 or 5 stars and anything lower would degrate the score.
Originally posted by @rdeoliv in https://github.com/ably/ablyassettrackingandroid/issues/533#issuecomment986709907

The corresponding Jira issue is https://ably.atlassian.net/browse/SDK-1424


10-12 Channel Continuous with interrupts.

Hi Rob, all your work on this is much appreciated! By far the most comprehensive implementation for these devices!
I am trying to sample 10-12 channels (3x ADS1115) as quickly as possible. Using 860sps, 6.144 FSR, 5v supply, Arduino Mega2560 (I think I should be able to use 3 interrupt pins - 2, 3, and 18 - per #14 ). My current setup is based off of the ADS_continuous_8_channel.ino example, I simply copied/pasted to create ADS_3. Currently, the code causes a hang with 3 ADCs, I believe due to arbitration issues on the I2C bus. With the third ADC commented out, it works as expected, because it is pretty much identical to the example. The attached code has 3 channels on ADS_1 and ADS_2, and 4 channels on ADS_3. For this reason, I set the address of ADS_3 to 0x48, in hopes that it would win arbitration to make up for the slightly slower sampling (3 vs 4 channels). I was using 3 channels on the other 2 ADCs in an attempt to take advantage of some advantages discussed in #49 - the hope was to sample 10 channels with maximum SPS.
In terms of solutions, the following comment from #53  seems like it may have potential: "What might be an option is to combine async requesting a conversion and use interrupts to trigger the reading of the result. Polling a flag is faster than requesting status over I2C."
I have also played around with the ADS_async_8_channel.ino example (and 16channel variant), #24 . These examples (of course) work flawlessly for my application- but my initial goal was to reach faster sampling rates by modifying the 8channel_continuous code instead.
Should I stick with the async implementation? Is this the fastest performance possible with 3x ADS1115? Thank you!
Code:

uint32_t starttime1;
uint32_t endtime1;
uint32_t starttime2;
uint32_t endtime2;
uint32_t starttime3;
uint32_t endtime3;
uint32_t times[10] = { 0, 0, 0, 0, 0, 0, 0, 0, 0, 0 };

int SPS1 = 0;
int SPS2 = 0;
int SPS3 = 0;
uint32_t lastTime1 = 0;
uint32_t lastTime2 = 0;
uint32_t lastTime3 = 0;

#include "ADS1X15.h"

//  adjust addresses if needed
ADS1115 ADS_1(0x49); //VDD 1,2,3
ADS1115 ADS_2(0x4B); //SCL 4,5,6
ADS1115 ADS_3(0x48); //GND 7,8,9,10 - lowest because 4 channels, will win arbitration

//  two interrupt flags
volatile bool RDY_1 = false;
volatile bool RDY_2 = false;
volatile bool RDY_3 = false;

uint8_t channel_1 = 0;  //  channel from device 1
uint8_t channel_2 = 0;  //  channel from device 2
uint8_t channel_3 = 0;  //  channel from device 2

//  array to hold the data.
int16_t val[10] = { 0, 0, 0, 0, 0, 0, 0, 0, 0, 0 };

void setup() {

  Serial.begin(115200);
  Serial.println(__FILE__);
  Serial.print("ADS1X15_LIB_VERSION: ");
  Serial.println(ADS1X15_LIB_VERSION);

  Wire.begin();

  //  SETUP FIRST ADS1115
  ADS_1.begin();
  ADS_1.setGain(0);      //  6.144 volt
  ADS_1.setDataRate(7);  //  0 = slow   4 = medium   7 = fast
  ADS_1.setWireClock(400000);

  //  SET ALERT RDY PIN
  ADS_1.setComparatorThresholdHigh(0x8000);
  ADS_1.setComparatorThresholdLow(0x0000);
  ADS_1.setComparatorQueConvert(0);

  //  SET INTERRUPT HANDLER TO CATCH CONVERSION READY
  pinMode(18, INPUT_PULLUP); //ADC1 whjte wire
  attachInterrupt(digitalPinToInterrupt(18), adsReady_1, RISING);

  ADS_1.setMode(0);             //  continuous mode
  ADS_1.requestADC(channel_1);  //  trigger first read

  //  SETUP SECOND ADS1115
  ADS_2.begin();
  ADS_2.setGain(0);  //  6.144 volt
  ADS_2.setDataRate(7);
  ADS_2.setWireClock(400000);

  //  SET ALERT RDY PIN
  ADS_2.setComparatorThresholdHigh(0x8000);
  ADS_2.setComparatorThresholdLow(0x0000);
  ADS_2.setComparatorQueConvert(0);

  //  SET INTERRUPT HANDLER TO CATCH CONVERSION READY
  pinMode(2, INPUT_PULLUP); //ADC2 yellow wire
  attachInterrupt(digitalPinToInterrupt(2), adsReady_2, RISING);

  ADS_2.setMode(0);             //  continuous mode
  ADS_2.requestADC(channel_2);  //  trigger first read

 //  SETUP THIRD ADS1115
 ADS_3.begin();
 ADS_3.setGain(0);      //  6.144 volt
 ADS_3.setDataRate(7);  //  0 = slow   4 = medium   7 = fast
 ADS_3.setWireClock(400000);

 //  SET ALERT RDY PIN
 ADS_3.setComparatorThresholdHigh(0x8000);
 ADS_3.setComparatorThresholdLow(0x0000);
 ADS_3.setComparatorQueConvert(0);

 //  SET INTERRUPT HANDLER TO CATCH CONVERSION READY
 pinMode(3, INPUT_PULLUP); //ADC3 purple wire
 attachInterrupt(digitalPinToInterrupt(3), adsReady_3, RISING);

 ADS_3.setMode(0);             //  continuous mode
 ADS_3.requestADC(channel_3);  //  trigger first read
}

void loop() {
  handleConversion();

  //mine
  String serial_out = "";
  String time_out = "";

  Serial.println("DEBUG: Entering Loop");
  for (int i = 0; i < 10; i++) {
    serial_out = serial_out + '\t' + val[i];
    time_out = 't' + time_out + '\t' + times[i];
    //Serial.print(val[i]);
    //Serial.print('\t');
    handleConversion();
  }
  //Serial.println();

  //Mine
  Serial.println(serial_out);
  Serial.println(time_out);

}

//  catch interrupt and set flag device 1
void adsReady_1() {
  RDY_1 = true;
}

//  catch interrupt and set flag device 1
void adsReady_2() {
  RDY_2 = true;
}

void adsReady_3() {
  RDY_3 = true;
}

//  handle conversions that are ready
bool handleConversion() {
  bool rv = false;
  if (RDY_1) {
    //  save the last value
    SPS1++;
    val[channel_1] = ADS_1.getValue();

    endtime1 = micros();
    times[channel_1] = endtime1 - starttime1;

    //  request next channel
    channel_1++;
    if (channel_1 >= 3) channel_1 = 0;

    starttime1 = micros();

    ADS_1.requestADC(channel_1);
    RDY_1 = false;
    rv = true;

    if (SPS1 % 200 == 0) {
      for (int i = 0; i < 3; i++) {
        Serial.print('\t');
        Serial.print(val[i]);
      }
      Serial.println();
    }

    if (millis() - lastTime1 >= 1000) {
      lastTime1 = millis();
      Serial.print("SPS1: ");
      Serial.println(SPS1);
      SPS1 = 0;
    }

  }
  if (RDY_2) {
    SPS2++;
    //  save the last value
    val[3 + channel_2] = ADS_2.getValue();

    endtime2 = micros();
    times[3 + channel_2] = endtime2 - starttime2;

    //  request next channel
    channel_2++;
    if (channel_2 >= 3) channel_2 = 0;

    starttime2 = micros();

    ADS_2.requestADC(channel_2);
    RDY_2 = false;
    rv = true;

    if (SPS2 % 200 == 0) {
      for (int i = 3; i < 6; i++) {
        Serial.print('\t');
        Serial.print(val[i]);
      }
      Serial.println();
    }
    if (millis() - lastTime2 >= 1000) {
      lastTime2 = millis();
      Serial.print("SPS2: ");
      Serial.println(SPS2);
      SPS2 = 0;
    }

  }
 if (RDY_3) {
   SPS3++;
   //  save the last value
   val[6 + channel_3] = ADS_3.getValue();

   endtime3 = micros();
   times[6 + channel_3] = endtime3 - starttime3;

   //  request next channel
   channel_3++;
   if (channel_3 >= 4) channel_2 = 0;

   starttime3 = micros();

   ADS_3.requestADC(channel_3);
   RDY_3 = false;
   rv = true;

   if (SPS3 % 200 == 0) {
     for (int i = 6; i < 10; i++) {
       Serial.print('\t');
       Serial.print(val[i]);
     }
     Serial.println();
   }
   if (millis() - lastTime3 >= 1000) {
     lastTime3 = millis();
     Serial.print("SPS3: ");
     Serial.println(SPS3);
     SPS3 = 0;
   }

 }

  return rv;
}

A first look through the code does not reveal any big improvements.
You could comment the print statements in loop() as those are printed under interrupt which may slow down things a bit.
Minimize the other print statements and use Serial.begin(500000); would save some clock cycles.
Furthermore you could skip one call to microseconds(). would maybe add 1 sample per second.
  if (RDY_2) {
    SPS2++;
    //  save the last value
    val[3 + channel_2] = ADS_2.getValue();

    endtime2 = micros();
    times[3 + channel_2] = endtime2 - starttime2;

    //  request next channel
    channel_2++;
    if (channel_2 >= 3) channel_2 = 0;

    starttime2 = endtime2;  //  <<<<<<<<<<<<
    ADS_2.requestADC(channel_2);

removing the times capture would save some clock cycles too.
you could make uint8_t channel_2 an int as that is the default datatype.
Don't know if the compiler already optimizes this (gains might be a few micros.)
So you might gain a few cycles but I expect no substantial increase in SPS.

Okay, thanks for taking a look! Besides SPS, My more significant issue is that it does not currently function with 3 ADCs, the sketch hangs after 2-3 readings. I believe the code is missing safety protections, but I'm not exactly sure how to implement them.
I was also hoping you could provide a bit more detail on how your "continuous" examples are different from the "async" examples, I don't think I fully understand.

Still needing help to make async safe continuous interrupts.

As your question is not solved yet I reopened the issue, so I see it on my open issues list.

hi, i did some speed testing in the past, on a adafruit feather adalogger M0 a SAMD21 chip running at 48 MHz
at 400 kHz:
requesting a sample takes around: 120 us
and reading a sample takes round: 150 us
so together around 270 us
however on an atmega board running at 16 MHz I guess it adds around 60-80 us for a total of around 340 us
i was able to run the ADS1115 up to 1.2 MHz (short lines, 1.5 k pullups, may vary from device to device)
but I would no recommend to go higher than 600 - 800 kHz (ideally test the limit and reduce by a factor of 1.5 or 2)
on an atmega I think above around 600 - 800 kHz one gets diminishing returns anyways
@600 kHz (on an M0):
request: 80 us
reading: 100 us
so a total of 180 us which on a atmega might end up at give or take 250 us

@lynx985
Thanks for these performance figures. Insightful
If time permits and I can find a free MEGA I will do actual timing measurements this week.

Could do a simple test on an UNO, which is not a MEGA but same clock speed and same ARCH
Performance is better than I expected, see table below.

clock
requestADC ()
getValue()
sum
notes

100
444
564
1008

200
252
324
576

400
156
196
352

600
124
156
280
max advised

800
112
140
252

1000
56
lock / crash

Code snippet
  start = micros();
  ADS.requestADC(0);
  stop = micros();
  Serial.println(stop - start);
  delay(100);
  while (ADS.isBusy());
  start = micros();
  val_0 = ADS.getValue();
  stop = micros();
  Serial.println(stop - start);
  delay(100);

@marcus-4
Have you tried the proposed improvements?

minimize what is printed (e.g nly SPS count)
increase Serial baudrate to 250000
increase I2C clock to 500k or 600k

As there are no new insights here I close the issue.
Feel free to reopen if needed.