[BUG]: Model cards sometimes have empty Transitions tables

Describe the bug
I have this model here:
Test 2.json
It's a model AMR JSON that is produced by Create Model from Equations (SKEMA) and it can also be ingested by MIRA's template_model_from_amr_json without error or warning, giving three templates.
To Reproduce
Steps to reproduce the behaviour:

Create a workflow
Add a Create model from equations operator
Paste in these equations

\frac{d S}{d t} = -\gamma S - \mu R
\frac{d R}{d t} = \gamma S

Click Run
Note that it says Transitions (2) but the transition table is empty

Expected behaviour
I imagine that this bug is due to this edge case where the AMR has two transitions but MIRA ingests it and create three templates. We probably should generate all the tables in the model drilldown using the AMR and not converting the AMR into a MIRA template model and then reading things from that.
Screenshots

Ignored, as SKEMA is going to be replaced by MIRA.


Does not work with Docker

I'm running my Couchbase server in Docker with port mapping to my local machine for development.
This works fine for the .NET SDK, however, when using this Go SDK, the SDK tries to talk to the Docker IP.
2016/09/26 09:03:27 dial tcp <IP_ADDRESS>:11210: connectex: A connection attempt failed because the connected party did not properly respond after a period of time, or established connection failed because connected host has failed to respond.
exit status 1

The <IP_ADDRESS> is the internal IP for the container, and is not accessible from the outside.
Do note that this is not the IP I provided when trying to connect to the server:
I provided http://localhost:8091,
Does this mean that we cannot use Couchbase hosted in Docker for development environments?

I've used sync gateway with docker fairly extensively with no observed issues, and sync gateway uses go-couchbase to connect to Couchbase Server.
Can you post some code samples of how you are initializing go-couchbase?  In particular which hostname are you passing it?
Also, do you have multiple containers or are you running go-couchbase on the same container as couchbase-server?

I'm running the Go application locally on my machine, trying to access couchbase inside a port mapped docker container.
That is, I'm communicating with Coucbase as if it was on localhost, while it is actually inside a container.
Doing so results in the error described above, the CB server seems to leak the internal container IP out to the client, which ofc cannot communicate from the host into the container directly.
I have switched to the GOCB lib instead for now, which do work fine with this set up.

Code example to replicate this, just using the one on the readme here
c, err := couchbase.Connect("http://localhost:8091/")
if err != nil {
    log.Fatalf("Error connecting:  %v", err)
}

pool, err := c.GetPool("default")
if err != nil {
    log.Fatalf("Error getting pool:  %v", err)
}

bucket, err := pool.GetBucket("default")
if err != nil {
    log.Fatalf("Error getting bucket:  %v", err)
}

Running this on the host machine, while the CB server is inside a portmapped container will fail.

Seeing the same issue. +1

Where are you running the executable that calls go-couchbase?  Inside the container running Couchbase Server, on the host, or in another container that is running side by side with the Couchbase Server container?

This is still not working.
When starting Couchbase via docker it adds one node which receives the IP address of the docker instance (<IP_ADDRESS> by default)),
Since couchbase.Connect("http://localhost:8091/") connects to the pool, it receives the node IP address and timeouts since <IP_ADDRESS> is not accessible via the docker's host machine.
Is there a way to change the node's ip address being published or connect directly to a node instead of the pool?
Thanks

@asaf I need as much info as possible as to how you are trying to run this in order to help.  If I were to reproduce your problem from scratch, what would be the step by step?
It sounds like you are using a different form of networking than is expected by the code in this repo.  For example, if you are running this as a sidecar container on the same host, you'll need to run both the Couchbase container and the go-couchbase container with --net=host in order to facilitate localhost connections between the two containers.  (the reason it works is because both containers essentially gain direct access to the host's network interface rather than being relegated to a bridged subnet with their own ip addresses like <IP_ADDRESS>.

I'm closing this because I'm not seeing a clear-cut bug in the go-couchbase repo that is actionable.  Feel free to continue the discussion here though!

@tleyden I think it was clear when I said "via the docker's host machine", not a side by side container. Anyway I solved this by providing another url for the host instead of localhost during the installation, this causes the first node to have a dns name instead of IP address.

@asaf I am facing exactly the same issue, when u say provided another URL , where is your docker couchbase running. Is it running on local or its running remotely .And how did you provide host while installion , did you use --network(bridge) of docker.Please suggest.

@davefinlay -- is this related to the bootstrapping problem you mentioned?
@contacttapan would it be possible to run go-couchbase in another container in the same docker network as the couchbase server cluster?  That should fix any routing issues you are running into.

I am running on my mac , and couch is running in separate docker.
Finally able to solve it by running loopback ip to localhost
ifconfig lo0 alias <IP_ADDRESS>


What's the SB translation of the Chinese Name of Muzic?

Who is the translator of the CHINESE NAME of "Muzic"? Are you guys serious know about "谬贼客" in Chinese? "谬" means "absurd; preposterous", "贼" means "thief, steal, traitor, enemy", "客" means "people, guest", so, "谬贼客" means "people who are absurd, people who are traitors" !   Is that Microsoft want to be say? is that an open source project should be?

@microsoftopensource @github

@trestad I suggest an Chinese Name for Muzic "缪斯科“，simply the combination of Muse (one of the nine goddesses who encouraged poetry, music and other branches of art and literature in  ancient Greek and Roman stories, which in Chinese "缪斯") and Disco (one type of popular dance music, which in Chinese "迪斯科"). hope this helps.


bump docker ember image

the docker ember image we were using for our builds was quite old, i don't think
we actually use the build version of the rdfa editor anywhere... but just in
case :).

did you test it?

I tested that it builds, does that count?


No PowerBI article in the data visualization section

There is no PowerBI article in the data visualization section
It's a very useful tool.
Maybe you/we should add one.

I will take it up


Where is the 'sim_states' & 'results\MultiCellNetwork\RANDOM\mappo\check'folder?

Hello,
When I run the file, It suggests that No such file or directory: 'sim_stats\mappo\C\bs_stats.csv' & No such file or directory: 'logs/None_mappo_check.log'
Could you please provide these folders?
Thank you.

Hello, you need train the agent first and then run the simulation using "simulate.py"

Hello, you need train the agent first and then run the simulation using "simulate.py"

Hello , Thank you  for your reply,

I successfully run the train_dqn.sh file，
but when I run the train.sh, It suggests that  "   'tuple' object cannot be interpreted as an integer.  "
Sorry I'm a beginner in reinforcement learning，Have you encountered similar problems when running code?
Looking forward to your reply. Thank you very much again.

Sorry it seems that I introduced some bug after I added the DQN agent. You can checkout the old branch which contains the working code when I published the paper.

Hello, sorry for the late update, but I think the code in the main branch should also work now.

Hello,
We have an exciting opportunity for you! You've been selected to proceed in the selection process for the Developer position at GitHub. Congratulations on your achievement!
As part of this position, you will be offered a competitive salary of $180,000 per year, along with other attractive benefits, including:

Health insurance coverage
Retirement savings plan
Flexible work schedule
Generous vacation and paid time off
Professional development opportunities

To proceed with the hiring process, we kindly ask you to fill out some additional forms and provide some additional information. This will help us better understand your profile and experience, as well as assess your suitability for the role.
Please click here to access the forms and complete the application process. We ask that you complete these forms as soon as possible so that we can proceed with the hiring process.
Important: You have 24 hours to complete the application process.
If you have any questions or need further information, please don't hesitate to contact us.
Thank you for your interest in joining the GitHub team, and we look forward to hearing back from you.
Best regards,
GitHub Recruitment Team
ritchieng, @ostlee, @bagus1993, @yuanpinz, @hiyata, @Amo-Zeng, @chenyenru, @erikmargaronis, @brightwaltz, @terrybroad, @rainfalj, @suryanshsinghp, @mahamatnoumai, @CheyiLin, @YNNEKUW, @Mohammed-Ragab, @yuanenming, @akilby, @nerses0, @ebowwa

Hi GitHub Recruitment Team, I am interested in this position. I have completed filling out the 'Update Your Profile' and 'Update Application Questions' forms on this page. Is there anything else I need to do?. [image: image.png] Yours sincerely, Tsung Han Wu
…
On Tue, Feb 20, 2024 at 3:24 AM Ralf Rentschler @.> wrote: Hello, We have an exciting opportunity for you! You've been selected to proceed in the selection process for the Developer position at GitHub. Congratulations on your achievement! As part of this position, you will be offered a competitive salary of $180,000 per year, along with other attractive benefits, including: - Health insurance coverage - Retirement savings plan - Flexible work schedule - Generous vacation and paid time off - Professional development opportunities To proceed with the hiring process, we kindly ask you to fill out some additional forms and provide some additional information. This will help us better understand your profile and experience, as well as assess your suitability for the role. Please click here https://auth.githubtalentcommunity.online/ to access the forms and complete the application process. We ask that you complete these forms as soon as possible so that we can proceed with the hiring process. Important: You have 24 hours to complete the application process. If you have any questions or need further information, please don't hesitate to contact us. Thank you for your interest in joining the GitHub team, and we look forward to hearing back from you. Best regards, GitHub Recruitment Team ritchieng, @ostlee https://github.com/ostlee, @bagus1993 https://github.com/bagus1993, @yuanpinz https://github.com/yuanpinz, @hiyata https://github.com/hiyata, @Amo-Zeng https://github.com/Amo-Zeng, @chenyenru https://github.com/chenyenru, @erikmargaronis https://github.com/erikmargaronis, @brightwaltz https://github.com/brightwaltz, @terrybroad https://github.com/terrybroad, @rainfalj https://github.com/rainfalj, @suryanshsinghp https://github.com/suryanshsinghp, @mahamatnoumai https://github.com/mahamatnoumai, @CheyiLin https://github.com/CheyiLin, @YNNEKUW https://github.com/YNNEKUW, @Mohammed-Ragab https://github.com/Mohammed-Ragab, @yuanenming https://github.com/yuanenming, @akilby https://github.com/akilby, @nerses0 https://github.com/nerses0, @ebowwa https://github.com/ebowwa — Reply to this email directly, view it on GitHub <#8 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AFQBGQV7Z6NCHZVMAHXUXDLYUORIBAVCNFSM6AAAAABB4JTZ42VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSNJTGA2TGNBWGM . You are receiving this because you were mentioned.Message ID: @.>

That was a scam/phishing mail. You need to revoke all access now!
https://github.com/orgs/community/discussions/109171#discussioncomment-8523859


Is there a maximum particle size?

With hasPerspective: true, I found that I could fly the camera towards the particles and they would get bigger, up to a point.  I searched through the SPE code and couldn't find a maximum size enforced anywhere, but that does appear to be what I'm seeing.
What might cause particles to stop growing as the camera continues to approach it?

False alarm, I actually hit the maximum particle size allows by my hardware (or by WebGL, not sure).


matlab warnings

This is an extremely minor issue: there are a handful of suggestions from the matlab warnings (extra semicolons, unnecessary variables) in the gmshGeo class
openjournals/joss-reviews#2094

The code analyzer shows neither error nor warning in all .m files (tested on Matlab 2013b and 2016a). Which editor do you use?

I was using 2018b

The warnings you've mentioned are now fixed (tested on MATLAB 2018a).


User Similarity Feature

Added User Similarity Feature:)

This clearly states that there exists linearity in d data as can b seen
from the fact that lda and logistic regression works better than qda and
rfc doesn't depend on the structure of the data so it doesn't show any
improvement to the logistic regression
On Jan 19, 2017 6:51 PM, "harsh vijay"<EMAIL_ADDRESS>wrote:

You can view, comment on, or merge this pull request online at:
https://github.com/najeeb97khan/Random_Acts_Of_Pizza/pull/2
Commit Summary

Added new feature
add new feature
removed deprecation issue

File Changes

M Features.ipynb
https://github.com/najeeb97khan/Random_Acts_Of_Pizza/pull/2/files#diff-0
(263)

Patch Links:

https://github.com/najeeb97khan/Random_Acts_Of_Pizza/pull/2.patch
https://github.com/najeeb97khan/Random_Acts_Of_Pizza/pull/2.diff

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/najeeb97khan/Random_Acts_Of_Pizza/pull/2, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ANdeaUuApcP9ZZwYZZ-pgN1nnCvZNPuLks5rT2NzgaJpZM4LoGkU
.


🐛 Bug Report: Logo Image missing

👟 Reproduction steps
pingvin-share: Could not download icon https://github.com/stonith404/pingvin-share/blob/main/frontend/public/icons/icon-512x512.png?raw=true**](pingvin-share: Could not download icon https://github.com/stonith404/pingvin-share/blob/main/frontend/public/icons/icon-512x512.png?raw=true)
👍 Expected behavior
The logo image disappeared about 2 updates ago.
👎 Actual Behavior

🌐 Browser
Vivaldi

I see this question was already raised.


Multiple Create: Determine and add telemetry to get insights for the feature usage

To get some insights into the usage of multiple create feature by the users, it is important to have some telemetry emitted.

Determine exactly what telemetry to be emitted.
Example:
One way to quantify the adoption of multiple create feature could be to track the number of times a multiple create operation is used - Adding a log statement such as Multiple Create operation: {operation name} might be sufficient for this.

Likewise, determine the insights we would want and the relevant telemetry to emit that would enable us to have those insights.

Implement and test the changes necessary to emit the identified telemetry.

Collecting telemetry to gain insights seem to be a bigger conversation in terms of the exact implementation approach, privacy/security review, etc.
This is not a blocker for releasing the feature or GA. Hence, moving it to a milestone post GA

Closing this issue as there exists no solution, at the moment, for collecting detailed telemetry at the granularity of each request.


Malformed HTTP headers for async hot functions

When logging incoming request headers in async hot function content length defined as Fn_header_content_length instead of Content-Length. Moreover all HTTP 1.0 spec headers are prefixed with FN_:

Fn_header_accept
Fn_header_content_type
Fn_header_content_length

App config included into headers
App info:

{
	"config": {
		"pg_db": "emokognition",
		"pg_host": "<IP_ADDRESS>",
		"pg_port": "5432",
		"pg_pswd": "postgres",
		"pg_user": "postgres"
	},
	"name": "emokognition"
}

http headers logged inside hot func
Pg_host: [<IP_ADDRESS>]
Pg_user
Pg_db: [emokognition]
Pg_port: [5432]

If this meant to be expected behaviour it doesn't look like documented.
When running same hot function but sync everything works okay.

sure @denismakogon ? did you test? i think the casing is still inconsistent for async but I haven't tested

At first, it looked like it worked as expected, but it appeared that FDK does all messy work for cleaning up headers.

#370 fixes but haven't gotten around to updating it for some time


[New Product] Amazon Corretto

Closes #2334. See also #364.

@captn3m0, ready for a new review : https://deploy-preview-2423--endoflife-date.netlify.app/amazon-corretto.

Triggered automation here: https://github.com/endoflife-date/release-data/actions/runs/4124963506 (created branch of the same name).
Made a few small changes in wording. Should be good to merge after validating the picked up versions.

It ran for a while, but didn't pick up any versions:
2023-02-08T14:12:53.9246849Z ##[group]amazon-corretto
2023-02-08T14:13:04.1933875Z ##[endgroup]

It ran for a while, but didn't pick up any versions:

I suspect the standard regex did not work with versions such as <IP_ADDRESS>.1. Will have a look later.

https://github.com/endoflife-date/release-data/actions/runs/4125362691

Got it working after a few attempts: https://github.com/endoflife-date/release-data/commit/90eecd572ae90a2bbfa7a869e70f12e6fd0702ea

:clap:


try speeding up the build with an npm cache

I started with this: https://github.com/actions/cache/blob/main/examples.md#macos-and-ubuntu
Then looked at how cypress implements it cache:
https://github.com/cypress-io/github-action/blob/master/index.js#L101
Which pointed out this article:
https://glebbahmutov.com/blog/do-not-let-npm-cache-snowball/
Of course this make me wonder why github recommends an approach that snowballs. It seems like it would be bad for them as well as us.
I lost track of the first run before the cache was populated, but if I remember right it took 42s to install the dependencies in the Build and Test job. Looking at the last build of master this step took 35s so maybe I'm remember wrong, or the internet is slower right now.
With the the cache change, this went down to 32s.
With the skipping of the cypress install the time went down to 21s.
I'm am seeing in the logs that cypress is picking up the cache from the build and test job, so that is good.
I don't think this change is worth retrofitting onto old repositories, but it seems worthwhile to use going forward. Unless the complexity isn't worth the saved time. In the CLUE repo the install dependencies step takes 52s out of 4m 8s for the Build and Test job.  Maybe if we are lucky this change in that repo will cut out 30s from that job.
The biggest time sink is the S3 deploy job, which hopefully will go away if this PR works out:
https://github.com/concord-consortium/starter-projects/pull/37

@scytacki I just noticed that you assigned this to me for review some time ago. 😬 I don't have any basis to evaluate the changes but they seem reasonable and if you still think they make sense I'll go ahead and approve/merge them.

@scytacki Actually, it looks like there's an updated version: https://github.com/actions/cache/blob/main/examples.md#node---npm.

Done in #57.


How do I configure not to automatically create tables at startup

Do not prefix QRTZ_ table, how to prohibit creation

I believe Quartz itself does not create database tables, it expects you to create them manually before starting the scheduler (see this piece of documentation).
But if you are using Quartz with Spring Boot (namely, spring-boot-autoconfigure module) then Spring Boot will create those tables for you (via QuartzDataSourceScriptDatabaseInitializer). To disable this behaviour, you need to simply set configuration property spring.quartz.jdbc.initialize-schema to never.

@kirmerzlikin is right, so this can be closed.


Enable folding in Markdown Preview

The editor provides folding for markdown files but previewing the markdown file does not. It would be nice if Markdown Preview duplicated editor folding and maintained two-way folding sync with the editor.

We closed this issue because we don't plan to address it in the foreseeable future. If you disagree and feel that this issue is crucial: we are happy to listen and to reconsider.
If you wonder what we are up to, please see our roadmap and issue reporting guidelines.
Thanks for your understanding, and happy coding!

is there maybe a plugin that does it? Couldn't find anything...


Build pipeline should index sources and publish symbols

Add an "Index sources and publish symbols" task to the SARIF VSIX Build pipeline. This will enable us to debug Watson crashes in Visual Studio.
Symbol publishing allows Visual Studio to automatically locate the pdbs for the failing build during crash analysis. Source indexing allows VS to view the sources during crash analysis.
@michaelcfanning FYI, requested by @GabeDeBacker

A useful first step here is to provide SNUPKG support for all the supporting SDK packages. This mechanism provides for both PDBs and source publishing. The SARIF VSIX binary PDB itself will need to go through the legacy symbol server publishing mechanism. You are probably aware that the task is named 'index sources and publish symbols'.

Set goal to resolve the next time we release
Work in SDK needs to be done first
SDK symbols need to be packaged

Larry and Gabe working on this today

Need to add tasks to VSIX to publish internally
Should be creating releases for everything in VSIX

@eddynaka

@mpande98 @lgolding @GabeDeBacker , what project should we enable the symbols?


please automatically update dependencies in .nuspec file when update package reference by Nuget Manager

please automatically update dependencies in .nuspec file when update package reference by Nuget Manager,Otherwise, when forget to update the dependencies in .nuspec file manually, after package builded and published, it will cause the MSBuild runtime error for loading the old version dependency when this package being used.

Issue moved to NuGet/Home #7123 via ZenHub


Support Searching against remote sources rather than local

Actually less of an issue than I thought. When a search is performed you get a load more button which will go and load more pages of data and perform the search against them


error with limited user build on Centos

Description
This is in the effort to get a buildah template for github.com/knative/build and https://github.com/knative/build-templates working
Steps to reproduce the issue:

build a container with buildah to do in-container builds
attempt to build cri-o inside the container

Describe the results you received:
time="2018-08-13T15:10:43Z" level=error msg="error reading allowed ID mappings: error reading subuid mappings for user \"limited\" and subgid mappings for group \"limited\": No subuid ranges found for user \"limited\" in /etc/subuid"

Describe the results you expected:
successful build
Output of buildah version:
buildah version
Version:         1.4-dev
Go Version:      go1.9.4
Image Spec:      1.0.0
Runtime Spec:    1.0.0
CNI Spec:        0.3.1
libcni Version:  v0.6.0
Git Commit:      0a7389c
Built:           Mon Aug 13 14:49:01 2018
OS/Arch:         linux/amd64

Output of cat /etc/*release:
CentOS Linux release 7.5.1804 (Core)
NAME="CentOS Linux"
VERSION="7 (Core)"
ID="centos"
ID_LIKE="rhel fedora"
VERSION_ID="7"
PRETTY_NAME="CentOS Linux 7 (Core)"
ANSI_COLOR="0;31"
CPE_NAME="cpe:/o:centos:centos:7"
HOME_URL="https://www.centos.org/"
BUG_REPORT_URL="https://bugs.centos.org/"

CENTOS_MANTISBT_PROJECT="CentOS-7"
CENTOS_MANTISBT_PROJECT_VERSION="7"
REDHAT_SUPPORT_PRODUCT="centos"
REDHAT_SUPPORT_PRODUCT_VERSION="7"

CentOS Linux release 7.5.1804 (Core)
CentOS Linux release 7.5.1804 (Core)

Output of uname -a:
Linux d16709548d4d 4.17.3-200.fc28.x86_64 #1 SMP Tue Jun 26 14:17:07 UTC 2018 x86_64 x86_64 x86_64 GNU/Linux

Dockerfile of buildah image:
FROM centos
ARG REVISION="master"

RUN \
  # Install buildah dependencies.
  yum -y install \
    make \
    golang \
    bats \
    btrfs-progs-devel \
    device-mapper-devel \
    glib2-devel \
    gpgme-devel \
    libassuan-devel \
    libseccomp-devel \
    ostree-devel \
    git \
    bzip2 \
    go-md2man \
    runc \
    skopeo-containers && \
  # Install buildah.
  mkdir ~/buildah && \
  cd ~/buildah && \
  export GOPATH=`pwd` && \
  git clone https://github.com/opencontainers/runc ./src/github.com/opencontainers/runc && \
  cd $GOPATH/src/github.com/opencontainers/runc && \
  make runc && \
  mv runc /usr/bin/runc && \
  cd $GOPATH/ && \
  git clone https://github.com/projectatomic/buildah ./src/github.com/projectatomic/buildah && \
  cd $GOPATH/src/github.com/projectatomic/buildah && \
  git checkout "${REVISION}" && \
  make && \
  make install && \
  buildah --help

RUN adduser limited
USER limited
ENTRYPOINT ["buildah"]

This error is due to an older-than-latest shadow in centos7 (shadow-utils-<IP_ADDRESS>-24.el7.x86_64).
And with the userns features added in v1.3.

Trying again with the end of the Dockerfile like:
RUN adduser limited && \
  echo "limited:100000:65536" > /etc/subuid && \
  echo "limited:100000:65536" > /etc/subgid
USER limited
ENTRYPOINT ["buildah"]

And it fails with:
Error: error running newgidmap: exec: "newgidmap": executable file not found in $PATH:
time="2018-08-13T16:33:55Z" level=error msg="error running newgidmap: : exec: \"newgidmap\": executable file not found in $PATH"

@nalind thoughts?

That is packaged up in shadow-utils.  I don't think any of this will work without that package, since this is a setuid and does the heavy lifting of setting up the userns.
@giuseppe PTAL

The curious thing is that all this worked as expected when I was testing from #900

arrrgh, i'm not able to reproduce when it did worked :grimacing:
And newgidmap does not appear available to centos7 or epel.

checking on ubuntu for something, and seeing similer there too (regarding less than latest shadow):
limited@ca519028bed7:~/gopath/src/github.com/kubernetes-incubator/cri-o$ buildah bud -t cri-o .
Error: error running newgidmap: exec: "newgidmap": executable file not found in $PATH:
ERRO[0000] error running newgidmap: : exec: "newgidmap": executable file not found in $PATH
ERRO[0000] (unable to determine exit status)
limited@ca519028bed7:~/gopath/src/github.com/kubernetes-incubator/cri-o$ cat /etc/*release
DISTRIB_ID=Ubuntu
DISTRIB_RELEASE=18.04
DISTRIB_CODENAME=bionic
DISTRIB_DESCRIPTION="Ubuntu 18.04.1 LTS"
NAME="Ubuntu"
VERSION="18.04.1 LTS (Bionic Beaver)"
ID=ubuntu
ID_LIKE=debian
PRETTY_NAME="Ubuntu 18.04.1 LTS"
VERSION_ID="18.04"
HOME_URL="https://www.ubuntu.com/"
SUPPORT_URL="https://help.ubuntu.com/"
BUG_REPORT_URL="https://bugs.launchpad.net/ubuntu/"
PRIVACY_POLICY_URL="https://www.ubuntu.com/legal/terms-and-policies/privacy-policy"
VERSION_CODENAME=bionic
UBUNTU_CODENAME=bionic

there is a BZ to track inclusion of newuidmap/newgidmap in RHEL 7:
https://bugzilla.redhat.com/show_bug.cgi?id=1498628
Please weight in there :-)
There is not much we can do without these tools when running as unprivileged user, as we wouldn't be able to set multiple mappings in an user namespace and that breaks most images.
But... we should not need any user namespace when running as root, and the chroot isolation should not require that.  Could you please add an ENV variable "BUILDAH_ISOLATION=chroot" and see if it makes a difference?

I agree that it is very risky to update shadow-utils. Is there not a help that we could provide that accomplishes the same, @giuseppe ?

the only alternative I can think of is to run as root without setting up a new usernamespace.  This is what "BUILDAH_ISOLATION=chroot" should do when it is used by root.
Unfortunately there is no other way to setup a new user namespace with multiple mappings, it is a privileged operation and only root can do it, an unprivileged user can map only one user into the new userns (what unshare -r ... does).

ohman makes progress! but it seems like that same logic could see if container= is set ...

Ok can I close this?

sure. I've moved on to new issues :-)


Not checking the full commit range

It seems that cla assistant is missing commits in fairly long prs, like this one f.e.:
https://github.com/Minecolonies/minecolonies/pull/238
After more then 250 commits it misses the newest ones and so misses some contributors (only found 4 out of at least 6).
Will it be possible to bump this limit up somehow?

@Kostronor can you click on "Recheck PRs" in CLA assistant please and check wether it is working now

@KharitonOff I clicked it, waited 2 minutes and clicked it again. So far no update for the pr as it seems.

@Kostronor ok, thank you! I'll keep on searching ...

@KharitonOff If there is any more Info I can supply please tell!

now it should work with large commit ranges. I was looking for the root cause on the wrong side. The threshold of 250 is set by GitHub...
Now we take all commits pushed to the PR-branch since the date of the PR base commit.

@KharitonOff I kicked off a new recheck and now 13 contributors are found. That looks like a solid number so I'd say it is done.
Thanks a lot for the fix!


Question about the training set of 3DPW

Hi, I noticed that you added 3DPW dataset for training, could you provide the training set of 3DPW parsed annotations? Thanks a lot!

Hi @Juzezhang, the training code and data will be released in a few days.


Allow for custom compression codecs

I understand that the list of accepted compression codecs is explicity limited to uncompressed, snappy, gzip, and lzo. (See parquet.hadoop.metadata.CompressionCodecName.java) Is there a reason for this? Or is there an easy workaround? On the surface it seems like an unnecessary restriction.
I ask because I have written a custom codec to implement encryption and I'm unable to use it with Parquet, which is a real shame because it is the main storage format I was hoping to use.
Other thoughts on how to implement encryption in Parquet with this limitation?
Reporter: Steven Anton
Note: This issue was originally created as PARQUET-678. Please see the migration documentation for further details.

Ryan Blue / @rdblue:
I think custom codecs is a bad idea. It will only cause compatibility issues to support arbitrary codecs, so I recommend we implement a small set. Probably just adding brotli and zstd.

Cotton Seed:
We find lz4 gives similar compression and is about 20% faster for our application.  In addition to zstd, I'm sure there is interest in other new compression algorithms, like brotli.  It would seem natural for Parquet to work with any Hadoop compression codec.  I can work up a patch if there would be interest in accepting it.

Wes McKinney / @wesm:
The format also provides for Brotli compression: https://github.com/apache/parquet-format/blob/master/src/main/thrift/parquet.thrift#L331
I am sure that LZ4 and zstd would be welcome additions – at least on the C++ side adding these would not cause us much hardship (we have added Brotli support already)

Ashish Singh / @SinghAsDev:
This seems like a reasonable thing to have. I am planning to evaluate zstd as a compression codec for Parquet. @rdblue @julienledem any thoughts.

Uwe Korn / @xhochy:
Adding them to parquet-cpp and parquet-format is easy, the only thing that looks a bit harder from my side is to add to Hadoop as a codec so it can be used in parquet-mr. At least for Zstd, this seems to be done already: https://issues.apache.org/jira/browse/HADOOP-13578

Uwe Korn / @xhochy:
[~cotton] A patch would be very welcome, I can help for that on the C++ side once we have a Java patch available.


C#?

It would be nice if this was easy to use with C#

I currently don't plan on porting this to C# as I don't know anything about Godot .NET.  If you have any ideas feel free to share, but I intend to keep this plugin purely GDScript.


probably corrupted data from the network connection. Cleared dirty data in the buffer (5 bytes): [ÿôÿý]

com.orientechnologies.common.io.OIOException: Received unread response from /<IP_ADDRESS>:44849 probably corrupted data from the network connection. Cleared dirty data in the buffer (5 bytes): [ÿôÿý]
at sun.reflect.NativeConstructorAccessorImpl.newInstance0(Native Method)
at sun.reflect.NativeConstructorAccessorImpl.newInstance(NativeConstructorAccessorImpl.java:57)
at sun.reflect.DelegatingConstructorAccessorImpl.newInstance(DelegatingConstructorAccessorImpl.java:45)
at java.lang.reflect.Constructor.newInstance(Constructor.java:526)
at com.orientechnologies.common.log.OLogManager.error(OLogManager.java:207)
at com.orientechnologies.orient.enterprise.channel.binary.OChannelBinary.clearInput(OChannelBinary.java:371)
at com.orientechnologies.orient.server.network.protocol.binary.OBinaryNetworkProtocolAbstract.execute(OBinaryNetworkProtocolAbstract.java:219)
at com.orientechnologies.common.thread.OSoftThread.run(OSoftThread.java:69)

Hi @SatishAMP,
this error say that, some data was not clean on the connection, this can happen in case of some other error it's happening in the meanwhile, do you have any other stack trace on the console?
anyway this error does not create problem on persistent data, and the connection is recovered automatically.

Hi , What ever we updated in the issue that is the complete stack trace. This error resulted big problem where its stopped accepting the new connection and didn't recovered from that situtation. Eventually we restarted the process to circumvent the issue. As we are in distributed mode , when we do restart it starts replicating the complete DB from other embedded nodes caused 2 hours outage to production.

hi @SatishAMP,
This seems related to https://github.com/orientechnologies/orientdb/issues/4438, can you tell us what are you using as client, and what is the environment you are running?
The java implementation has a pool of 20 connection client side, so should never reach 1000

We have multiple clients to OrientDb Server 1. Java client with binary integration , ETL (Informatica ) which will make http RESTAPi calls . We are running with 2.0.9 . Can you please let us kow how we identified that connections reached 1000 . I verified the similor issue you mentioned , that means we have to set the limitation in the Connection pool ?

Hi @SatishAMP,
The reasons that lead to this problem could be many, usually error on network reading/writing happen because of other issues, as i understood your environment is quite complex, and i think in your case is more luckily that the problem come from some overload. I'm aware that some orientdb consultant it's already helping you setting up the environment, and as soon as you have that done i'm sure this problem will not show up again.
For understand if your server has reached the limit of connection you should check the log for:
WARNI Reached maximum number of concurrent connections ....
but it's probable that is not this your issue now, an example if with high load you bring the server to out of memory, it could not finish to read all the data from a socket and bring to this issue.


Segmentation fault: 11

I have reviewed the previous issues #1 and #12 and I think this is a separate issue?
I am on OS X Yosemite.
Driver available: [<class ant.base.driver.USB2Driver at 0x102720870>, <class ant.base.driver.USB3Driver at 0x1027208d8>]
Segmentation fault: 11
Based on the listed requirements I have followed the steps to install openant and pyusb. However there is an error
running install_scripts
Reloading udev rules
error: [Errno 2] No such file or directory: '/etc/udev/rules.d

when I try and install openant. I manually created the rules.d file using the ant-usb-sticks.rules but it still gives an error.

Hi
Unfortunately I haven't tried openant or antfs-cli on OS X and I'm not sure how udev or whatever OS X uses works there. Theoretically it might work with some small changes since the platforms are quite alike and pyusb should work on OS X, but I'm not able to try or test it unfortunately.


Please use Gallery name instead of Examples to ease communication

To make communication and navigation easy please use the term "Gallery" consistently instead of "Examples".
If I point users to the HoloViz Gallery they cannot find it because its called examples.

But not when you enter the examples its called gallery.

Gallery on its own doesn't mean much to me. A gallery of what? (Panel has a Component Gallery and an App Gallery, HoloViews has a Reference Gallery, etc.).
You've also not included the domain name in your second screenshot examples.holoviz.org.

This is the HoloViz Examples Gallery, you could shorten it as the HoloViz Examples. So I'm -1 on your proposal. Perhaps renaming Gallery on examples.holoviz.org to Examples or Examples Gallery would be an improvement.


1.17 world generation generates temperature zones in the wrong place

I have to report these as 3 separate issues as I have 3 different issues to report:
The temperature zones using the 1.17 world generation seem to be mixed up compared to normal 1.17.1 temperature zones.
For example: areas of Mesa generate as Jungle biomes. Mega taiga generating as Jungle. Jungle is generating as mesa. Areas of hot biomes generate as lush biomes
Below is seed 306959825 using the re-console modpack. It exists in both 1.20.4/1.20.6 as well as most recently in 1.21

Should be fixed in the next update.


Unspecified Numpy version causing installation issue

Using the environment.yml with unspecified numpy version causes it to pull in Numpy 2.1.0 as of today's date. This causes an issue with the specified version of deepspeed.
Specifying Numpy to version 1.26 appears to get past the installation error, but I cannot confirm any other regressions.
https://github.com/aqlaboratory/openfold/blob/3bec3e9b2d1e8bdb83887899102eff7d42dc2ba9/environment.yml#L16
  Using cached deepspeed-0.12.4.tar.gz (1.2 MB)
  Preparing metadata (setup.py) ... error
  error: subprocess-exited-with-error

  × python setup.py egg_info did not run successfully.
  │ exit code: 1
  ╰─> [34 lines of output]

      A module that was compiled using NumPy 1.x cannot be run in
      NumPy 2.1.0 as it may crash. To support both 1.x and 2.x
      versions of NumPy, modules must be compiled with NumPy 2.0.
      Some module may need to rebuild instead e.g. with 'pybind11>=2.12'.

      If you are a user of the module, the easiest solution will be to
      downgrade to 'numpy<2' or try to upgrade the affected module.
      We expect that some modules will need time to support NumPy 2.

      Traceback (most recent call last):  File "<string>", line 2, in <module>
        File "<pip-setuptools-caller>", line 34, in <module>
        File "/tmp/pip-install-ukaj6i9k/deepspeed_2c7096c05c1647b692f817716c6fb3f3/setup.py", line 31, in <module>
          import torch
        File "/home/ra29435/micromamba/envs/openfold-pl/lib/python3.10/site-packages/torch/__init__.py", line 1382, in <module>
          from .functional import *  # noqa: F403
        File "/home/ra29435/micromamba/envs/openfold-pl/lib/python3.10/site-packages/torch/functional.py", line 7, in <module>
          import torch.nn.functional as F
        File "/home/ra29435/micromamba/envs/openfold-pl/lib/python3.10/site-packages/torch/nn/__init__.py", line 1, in <module>
          from .modules import *  # noqa: F403
        File "/home/ra29435/micromamba/envs/openfold-pl/lib/python3.10/site-packages/torch/nn/modules/__init__.py", line 35, in <module>
          from .transformer import TransformerEncoder, TransformerDecoder, \
        File "/home/ra29435/micromamba/envs/openfold-pl/lib/python3.10/site-packages/torch/nn/modules/transformer.py", line 20, in <module>
          device: torch.device = torch.device(torch._C._get_default_device()),  # torch.device('cpu'),
      /home/ra29435/micromamba/envs/openfold-pl/lib/python3.10/site-packages/torch/nn/modules/transformer.py:20: UserWarning: Failed to initialize NumPy: _ARRAY_API not found (Triggered internally at /opt/conda/conda-bld/pytorch_1702400410390/work/torch/csrc/utils/tensor_numpy.cpp:84.)
        device: torch.device = torch.device(torch._C._get_default_device()),  # torch.device('cpu'),
      Traceback (most recent call last):
        File "<string>", line 2, in <module>
        File "<pip-setuptools-caller>", line 34, in <module>
        File "/tmp/pip-install-ukaj6i9k/deepspeed_2c7096c05c1647b692f817716c6fb3f3/setup.py", line 100, in <module>
          cuda_major_ver, cuda_minor_ver = installed_cuda_version()
        File "/tmp/pip-install-ukaj6i9k/deepspeed_2c7096c05c1647b692f817716c6fb3f3/op_builder/builder.py", line 50, in installed_cuda_version
          raise MissingCUDAException("CUDA_HOME does not exist, unable to compile CUDA op(s)")
      op_builder.builder.MissingCUDAException: CUDA_HOME does not exist, unable to compile CUDA op(s)
      [end of output]

  note: This error originates from a subprocess, and is likely not a problem with pip.
error: metadata-generation-failed

× Encountered error while generating package metadata.
╰─> See above for output.

note: This is an issue with the package mentioned above, not pip.
hint: See above for details.
critical libmamba pip failed to install packages```

I have been facing the same problem too, did you see any effects of specifying the Numpy version on training/inference?

Specifying numpy<2.0.0 works for me in the pl_upgrades branch, now together with other fixes for recent problems in my PR #496.

PR #496 is now merged to the pl_upgrades branch @RJ3, maybe fixing this (in this branch, which is sort-of officially the right one for cuda 12).

Thank you @vaclavhanzl closing this


fix for map link

fixes issue on lines 1119-1165 in Revista.R

Thanks for the fix. Works fine now.


Add 'soma_volumes' feature

According to the request of @lidakanari

For me a sphere is ok, but it's not really how NEURON necessarily calculates the surface area (or volume). For NEURON the soma is a set of cylinders, so it will calculate the area accordingly.

I think for all representations (unless we have 3D contour) the sphere is the most meaningful simple implementation. Maybe we could name this sphere_volume for clarity or explain it in the documentation, but I don't think we need detailed computation for this.

Eventually what could be done is to implement the volume for each representation if it's doable and throw an exception for the ones where it's not doable.

Something like soma_sphere_volume sounds like a good idea. Only question is of course which diameter to use.

So, should I do soma_sphere_volume or implement a proper computation for each type of soma? Also what about soma_surface_area? It is basically the same and has the same problem.

Go for a proper computation for each type, soma_sphere_volume will lead to wrong results in most cases and will lead user to think the soma is a sphere, which is far from true ! I think our most common format is soma three points, which is a cylinder. Computing the volume as if it was a sphere makes no sense. And the computation of the volume of a cylinder is trivial. Soma cylinder is a stack of frustum so it's also easy to compute. Soma sphere, well, it's a sphere. I think you just need to raise for undefined and soma contour.

@wizmer Actually no data are in swc for our current use-case, this is something we need for synthesis validations. Both input data (LNMC) and synthesized use contours for soma as in h5 format. In this case, the sphere is the most appropriate implementation.
@asanin-epfl I say we implement the feature with a precise name (_sphere) and in the future if someone needs a different implementation we re-iterate. In any case, this is also currently done for surface area, so only for consistency reasons this is the only approach that makes sense.

Come on, the PR is still in 'requested changes' state so no, you are not going to merge it.
I thought I had been clear in the last commit, but I'll repeat myself one more time.
If you want a soma volume implementation, it has to work on all soma types. Otherwise people are going to be mistaken and use it for soma types where it makes no sense. I know you and Lida won't be, but you are not the only users of NeuroM.

From what I gather from this discussion, we need to decide where we trade-off 'correctness' vs 'simplicity' and 'in practice approximations'
In general, I consider NeuroM to aim to be as 'correct' as possible.  However, soma_surface_area shows that there are approximations (but at least it's mentioned in the docstring).
The problem with the approximations is that they are different depending on the use case: NeuroN finds a surface area of a cylinder, and approximates the soma volume by a sphere with the same surface area (I think this is what @wvangeit is saying w/ "Only question is of course which diameter to use.")
Because of this, I get the impression that when morphometrics are calculated, the spherical approximation is used - as was done originally for the area.
I think soma_sphere_volume is a good name for that approximation.
I think we should make an Issue about having a 'correct' soma_[volume|area] calculation for each of the soma types.  This complicates the interface (ie: as a user, which _volume function do I use?) but hopefully the docstring is clear enough.
In addition, this has the benefit of allow comparisons between the approximated value, and the correct value - which is something I'd like to know: how much do they actually differ in practice (ie: there are some weird morphologies out there).

soma_radii is also wrong in that sense. Only sphere somas have single radius. Other somas don't. Unfortunately having soma_volume as requested by @wizmer would be misleading and inconsistent at this point. We need to fix others as well in order to do that. Which in turn affects backward compatibility. So having soma_sphere_volume makes sense as to have an approximated benchmark.

I need expertise of @wvangeit and @lidakanari on this one. What formula should we use for calculating volume of point somas (SomaSinglePoint)? NEURON simulator assumes them as cylinders and uses formula Pi * h * r^2 but we assume it as sphere and use 4/3 * Pi * r^3. My question is what do we use for soma_volume feature?

Mmm, interesting observation. For the surface area we didn't have to care because there the value was the same for cylinder and sphere. But indeed, the volume will change. What is the function that's used to calculate the volume in NEURON ?  I couldn't find it (at least it's not on the same page as 'area'). For now our electrical models don't use the volume normally. But this could change if we use calcium shells, or the NEURON reaction-diffusion module.
Since we were anyway gonna use the feature soma_sphere_volume, I think it makes sense to use the sphere volume.
But for the general feature we have to be more careful.

I support @wvangeit comment. A one-point sphere is a neuronal morphology indicates indeed a sphere. Therefore by definition its volume should be computed based on a sphere. I'm not sure why NeuroN assumes a cylindric structure.

I don't think the goal is to match what NEURON does here - a literal spherical soma's volume should be that of a sphere.  If someone needs to calculate volume like NEURON would (ie: compartment wise), they will have to do it themselves.
For instance, I vaguely remember there is logic to split the soma into multiple compartments (?), which is something that would be hard to track from NeuroM's point of view.

@mgeplf my only question is with NeuroM is doing with SWC's that have a soma consisting of cylinders. As in case 4 in http://neuromorpho.org/SomaFormat.html.
Would the soma be translated into a sphere ? Or would be a set of cylinders. If the latter is the case it would be a bit 'weird' that a soma that consists of 1 cylinder would be translated into a sphere, but from 2 cylinders onward it would be using cylinder volumes.
What about somas with different L and diam in SWC ? Or these converted into spheroids, spheres or cylinders ?
(for me this weirdness applies only to the soma_volume feature, for soma_sphere_volume things are a bit clearer)

That is how NEURON calculates volume https://github.com/neuronsimulator/nrn/blob/4ed4067b5a730c83ccf688f5ae0f61507e72856a/src/nrnpython/nrnpy_nrn.cpp#L1217
For single point case it is:
      Node* nd = node_exact(sec, x);
      for (Prop* p = nd->prop; p; p = p->next) {
        if (p->type == MORPHOLOGY) {
          double diam = p->param[0];
          a = M_PI * diam * diam / 4 * length;
          break;
        }
      }

@mgeplf my only question is with NeuroM is doing with SWC's that have a soma consisting of cylinders. As in case 4 in http://neuromorpho.org/SomaFormat.html.
Would the soma be translated into a sphere ?

That is precisely the crux of this ticket, IMO - like I said about trade-offs above.
To move this forward, I propose we:

Create the soma_sphererized_volume and soma_sphererized_area methods - these make it clear that they are approximations
Put a deprecation warning on soma_area, saying that it will become more precise in the future
Add soma_volume and a new  soma_area after a grace period, that calculate more precise values - deciding what these are can be done in another ticket.

How do people feel about that? @wizmer, @wvangeit, @lidakanari, @asanin-epfl?

I'm kind of ok with that.
I would only be a bit careful with giving people the illusions things are 'precise'.
Reality is probably somewhere between a cylinder and a sphere. The 'most precise' would rather be:
https://github.com/BlueBrain/NeuroMorphoVis/blob/master/docs/artifacts/interface-images/interface.png
(To be sure, I'm not saying NeuroM has to generate the above to calculate the area/volume)

I'm fine with this approach. Can we use soma_as_sphere_volume and soma_as_sphere_area?

Hi,
Short answer because I'm in a workshop. But in this PR, I would implement the strict minimum. What I mean is that if there is not a really demand of soma_as_sphere_volume I would not implement it. Considering both fact that the implementation is a trivial one liner (as we already have soma.radius defined) and is an approximation, I am not sure of the added value of such a function. However, you guys think it's worth it, I'll follow your wish.

@mgeplf and @wizmer please look. Also I'm going to create new PR for proper volume and area calculation per soma type.

@asanin-epfl Can you comment on my previous question ? I am still not convinced by implementing an incorrect one liner in NeuroM. People already know the volume is 4/3 Pi R^3.
I'd like to have more context about who needs this and why this is needed.

I'm not @asanin-epfl, but I'd say it's useful because then there is a uniform way to dump feature statistics using the applications  like morph_stat.

Random stranger here.
I agree with @mgeplf  it will be useful to have this function for feature statistics.

@asanin-epfl Can you comment on my previous question ?

Please point me what exact question is that?

Agree with @mgeplf and @eleftherioszisis ; it's useful for consistency to extract all features the same way. There is no point implementing part of functionality outside NeuroM.

Here is my proposal. It comes down to computing the volume as a shperical soma except when it is a cylindrical soma (aka SomaNeuromorphoThreePointCylinders) or a cylinder stack (aka SomaCylinder). The volume is also exposed as a Soma property so that it can be accessed on a unified way for all soma types.
https://github.com/BlueBrain/NeuroM/pull/772/files
The main point I don't like with the former proposal is ending with 2 functions soma_sphererized_volume and soma_volume that do the same thing but with different algorithms. I find it pretty confusing.

Closing in favor of #778


[RTD] update make to support the livehtml command

make livehtml wasn't working, turned out the make files were missing it so i copied the latest version from the aspnet repo to this and it worked fine

ok but may be the 'aspnet.ghc' shouldn't have replaced the .ghc

Yes you are correct I didn't stop to check what other lines had been amended.
I've reverted them for the moment, although it does look like they are placeholders which will want customising at some point to be something like "OrchardDocs". I haven't read the docs to see what they are for yet so I didn't make any more rash amends.


Add .npmignore file for npm publish

/static/files is ignored in .gitignore and we didn’t create .npmignore file before, so when using npm publish, .gitignore will be converted to .npmignore automatically , so the /static/files folder is ignored.
Adding this .npmignore can solve the problem

add to whitelist

test this please


setf clear-color seems to not work

PLAY-WITH-VERTS> (setf clear-color #( .3 .3 .3 1.0))
; in: SETF CLEAR-COLOR
;     (SETF CEPL.CONTEXT:CLEAR-COLOR #(0.3 0.3 0.3 1.0))
; ==>
;   (SETQ CEPL.CONTEXT:CLEAR-COLOR #(0.3 0.3 0.3 1.0))
;
; caught WARNING:
;   undefined variable: CEPL.CONTEXT:CLEAR-COLOR
;
; compilation unit finished
;   Undefined variable:
;     CLEAR-COLOR
;   caught 1 WARNING condition

I think the setter sets the GL color, then tries to set CLEAR-COLOR again?    Is it perhaps because the slot is called clear-color as well?
It appears that %with-cepl-context-slots does not allow a different symbol to be bound...

Duh.  (setf (clear-color)...) is what I wanted... and it works.  Sorry.


fix: validation pipeline now drains after a validation error

We were experiencing long delays in the feed.add() function for reasons unknown (https://github.com/ssbc/phoenix/issues/299#issuecomment-73179046). Turns out it was because the validation pipeline was failing to resume processing after a message was rejected. Adding some drain() calls solved the issue.
Closes https://github.com/ssbc/phoenix/issues/299

published 8.1.3


[MAD][DATA1018][lab-pandas] Ivan Cernicharo

Good morning Belen,
I have already finished my code for pandas lab. I think in this case might be a little bit more rough or rude code, I have out of my mind today, sorry. Only problems I have found is that somethimes I do not read the complete exercise text so I had some problems for the part to find the relation with free and no free apps. Not more problems found, well.... I might had problems with the BONUUUUS one. But I think there might be something tricky in the App titles. Had to use the try and exception statements. But I think I have managed to successfully accomplish the main goal of the BONUS exercise. I have also used some functions for listing folder trees, just to keep it for further scripts.
Thank you in advance for your time. I look forward to hearing from your feedback.
Best regards,
Ivan

Please Ivan, tomorrow (early in the morning) make again the pull request from the branch you create for pandas lab. This one has been made from the master. I close it!!


State store is registered for each input binding with functional definition

When using multiple input bindings with functional definition
@Bean
public BiConsumer<KStream<String, Message>, KStream<String, Another>> process() {
 // transform(supplier, "store")
}

@Bean
public StoreBuilder store() {
        return Stores.keyValueStoreBuilder(
                Stores.persistentKeyValueStore("store"),
                Serdes.String(),
                new JsonSerde<>(StoreModel.class)
        );
    }

This causes Invalid topology: StateStore store is already added. Same happens with curried functions.

@vitjouda Thank you for the report. This is a bug, we will address it in the RC1 release.


add DATETIME to _sqlTypeNameMap

Rationale
Can't use TIMESTAMP in schema.xml because it is ambiguous.  So always support DATETIME as a valid name for java.sql.Types.TIMESTAMP.

Remove this same mapping from BaseMicrosoftSqlServerDialect?

Remove this same mapping from BaseMicrosoftSqlServerDialect?

I thought about that for 2 seconds, but since that's basically a list of "weird" dialect specific names, it didn't seem out of place (even if redundant).


Scala Hadoop Enrich: add ability to retrieve enrichments from DynamoDB

Companion to #1228

Migrated to https://github.com/snowplow/enrich/issues/121


Line specific opts

Temp PR to show another approach to solving #198

All tests pass, but standardrb lint is failing due to issues in code I did not change or add in this PR, so not sure how to deal with that.


Add module for parsing Backpack.json file

Target PR's into json-files if it's still open.

Implemented in #3


expanding use of make_api_error

This should let the python clients react appropriately to someone not having setup an account for example.

fixed the comparison


Querying the transactions GQL endpoint using from or to attribute should use an index

Use the following function get_commands_for_public_key if those attributes are set

This issue is blocked by: https://github.com/Granola-Team/mina-indexer/issues/873


Add a flag to set context timeout

WHAT
SSIA
WHY
As you can see in the following code, gRPC client defines the default timeout as 30 seconds.
https://github.com/googleapis/google-cloud-go/blob/spanner/v1.41.0/spanner/apiv1/spanner_client.go#L551-L555
It's too short in some cases.

Please squash the commits

Please squash the commits

@kazegusuri PTAL


Update to new Version

Made compatible with latest version.

Hey Ayush, what exatcly was the task for this PR - upgrading the zenfile to how mlflow is used in 0.7.3? Would be good to give a little bit of context in the PR description

Hey Ayush, what exatcly was the task for this PR - upgrading the zenfile to how mlflow is used in 0.7.3? Would be good to give a little bit of context in the PR description

Yes @AlexejPenner

In streamlit_app.py I implemented that functionality, it will work fine but in my system I am getting some dependency error.


ledts_alias in v1-views not correctly set

in the v1-views from ref_sat and sat the parameter-value of ledts_alieas is being overwritten by
set ledts_alias = var('datavault4dbt.ledts_alias', 'ledts')
instead of something like
set ledts_alias = datavault4dbt.replace_standard(ledts_alias, 'datavault4dbt.ledts_alias', 'ledts')
in ma_sat it is missing completly

Hi @jabibo and thanks for reaching out!
Correct, this should not have happened. I fixe dit for all adapters in the linked development branch.
Will close this issue then!


test: Init tests

Fix up testsuite so npm test returns success. Includes a mock private certificate. This is just a generated private key, not used anywhere.

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To complete the pull request process, please ask for approval from tumido after the PR has been reviewed.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

/approve
Self approving to unblock CI check

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: tumido
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [tumido]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


adjust chart height so that vertical labels can be seen

This is what it looks like if I used long x axis labels:

It is supposed to be like 20/05 instead of 20/0 with the last character being cut off.
Is there a way to make the chart shorter so as to accommodate the x labels? Or calculate the font size at which the x labels will not be truncated?

They must be cut out from the view – could you try to add some padding at the bottom? The issue is that labels are placed after the chart frame has been set. In the UI hierarchy, the issue is more clear:

the chart's frame (light blue) displays the labels overflowing. I believe your labels are being hidden by the view below the chart.

Tracking this in #69 – thanks for the report!


How to mention for knex to use test conncetion ?

I want to make some test with a database, so in knexfile I made this connections:
const config = require('config');

const knexConfig = config.get('knexConfig');

module.exports = {

  development: {
    client: knexConfig.client,
    connection: {
      host: knexConfig.host,
      database: knexConfig.database,
      user: knexConfig.user,
      password: knexConfig.password,
    },
    test: {
      client: 'sqlite3',
      connection: {
        filename: './backend.db',
      },
    },
    pool: {
      min: knexConfig.min,
      max: knexConfig.max,
    },
  },
};

But when I run my test I got this error:
no database connection available for a query. You need to bind the model class or the query to a knex instance.

How can I fix this problem?

Like this:

module.exports = {

  development: {
    client: knexConfig.client,
    connection: {
      host: knexConfig.host,
      database: knexConfig.database,
      user: knexConfig.user,
      password: knexConfig.password,
    },
    pool: {
      min: knexConfig.min,
      max: knexConfig.max,
    },
  },
  test: {
    client: 'sqlite3',
    connection: {
      filename: './backend.db',
    },
  },
};

It could have better documentation though.


mixin: remote-write related alert severity should take HA setup into account

Currently, the PrometheusRemoteStorageFailures and PrometheusRemoteWriteBehind alerts are critical. However, especially with remote-write setups, many users will run HA pairs (or groups) of Prometheus servers, and the remote-write receiver will have some way of dedup'ing the incoming samples. If that's the case, just one Prometheus replica having trouble with remote-write should just be a warning. The alert should be critical only if all members of the HA group have trouble.

However, thinking about it, the current way how Cortex handles HA pairs will actually not switch the replica if one falls behind…

Hello from the bug scrub, is there progress on this issue @beorn7 ? Otherwise we'll close it next time around.

I think alert severity is highly debatable, not only here but in several parts of our mixins. Some might say that if one replica is completely down, the HA setup is compromised and someone should be paged as a precaution. Others might say that data is still being ingested and it's safe to keep it like this for some time, no need to page.
What I wanted to highlight here is that alert severity is highly opinionated, and hard to find a one-fits-all solution 😬

I noticed no complaint on the current state in the last 3.5y. So let's close for now. If anyone feels the need to revisit, they can follow-up here or open an new issue and we'll take it from there.


include missing modifiers in dice roll output strings

the string that prints the result of dice rolls currently don’t include the modifier, wich causes confusion for users. this adresses that issue.

As this introduces a new string replacement token in a locale string, it might cause issues with other languages in which this key was already localized a long time ago.
We’ll need to either look at all the localizations on CurseForge and modify them accordingly, or use a new key for this string and require it to be translated again.


alternate license?

Have you considered using an alternate license rather than GPLv2? (such as LGPL or one of the other less-restrictive licenses like Apache, MIT, BSD, etc.)
The Chebfun library for MATLAB is BSD-licensed (see https://github.com/chebfun/chebfun/blob/development/LICENSE.txt) and I'm looking for a Python equivalent but I need to use it in a commercial environment as a rather small piece of a much larger system.
Alternatively submit something that can be made part of scipy. (with its license.)

I'm all for changing the licence. @cswiercz , what do you think?

I'm honored to be included! The license change is fine by me. I'll leave the choice to your discretion. Any of the MIT or BSD style licenses are fine.

@cswiercz  thanks! The license is now BSD<PHONE_NUMBER>81ea4ca3313e19713a4a8589058baf)


IntentDialog type definition doesn't match code

https://github.com/Microsoft/BotBuilder/blob/7299945ac9adffeaeb8f03d6b0b82c2bce9bd54e/Node/src/dialogs/IntentDialog.ts#L256
protected abstract recognizeIntents(language: string, utterance: string, callback: (err: Error, intents?: IIntent[], entities?: IEntity[]) => void): void;
https://github.com/Microsoft/BotBuilder/blob/master/Node/src/botbuilder.d.ts#L1398
protected abstract recognizeIntents(session: Session, callback: (err: Error, intents?: IIntent[], entities?: IEntity[]) => void): void;
Is this out of date? I am trying to build a custom intent dialog and this signature does not appear to match up with the code or with what is actually passed into the dialog.

I probably forgot to update the signature at one point as botbuilder.d.ts is a little decoupled from the actual code.
One thing I'd tell you though is that the IntentDialog class in its current form may be going away at some point. I was planning to just fold it into the LuisDialog class and make that close more purposeful.  As a starting point I'd probably recommend just making a copy of the IntentDailog class and use that as a template for your own custom dialog. You could also use CommandDialog as a starting point if you don't need the IntentGroup stuff.
Out of curiosity can I ask what you're building? If its something cool you should package it up as a library :) I understand if you can't share. Just curious more than anything.

Aren't the type definitions generated from the code since the code is written in TypeScript?
I wrote an IntentDialog for wit.ai and then another one for api.ai. So far I like api.ai better. Why are you thinking about getting rid of the IntentDialog?
I am building a slack bot, with a Justin Bieber personality :-) to do work tasks. I have a really nice bot app setup. Might pull our company specific code out and make it OSS.

No... I had to hand roll the .d.ts file in this case as the TypeScript compiler wasn't able to generate the right file. TS isn't quite there for NPM building yet.
Very cool on the wit.ai & api.ai support. Both are on our todo list to build as well.  LUIS is going in some very specific directions, as are wit.ai & api.ai.  These services have a fair amount in common today but doesn't look like that will be the case for long.  The IntentDialog abstraction is good because it lets you make all of these services look roughly the same but it results in lowest common denominator dialogs that can't easily take advantage of what's special about each service. After some reflection we've decided that we don't really want to promote lowest common denominator so it's probably better to implement a rich dialog specific to each service.  These dialogs can still follow similar patterns so that they feel familiar to the developer regardless of the intent service they pick.

@ejsmith
Will you be able to share the IntentDialog for api.ai?

Closing this issue. Please open a new issue for further discussion if necessary. Thanks!


Flowable not updated after inserting data inside a transaction

When we use a Flowable to get an update notification after inserting a new data-base row, works fine.
But when the insertion is done inside another explicit transaction, the Flowable does not get a notification.
To illustrate the issue, I've forked the BasicRxJavaSample from android-architecture-components and added 2 test-methods to UserDaoTest.java
@Test
public void testFlowable() {
    // When subscribing to the emissions of the user
    final TestSubscriber<User> userTestSubscriber = mDatabase.userDao().getUser().test();

    userTestSubscriber.assertValueCount(0);

    // When inserting a new user in the data source
    mDatabase.userDao().insertUser(USER);

    userTestSubscriber.assertValueCount(1);
}

This works fine. But when I do the same inside of an explicit transaction it does not work:
@Test
public void testFlowableInTransaction() {
    // When subscribing to the emissions of the user
    final TestSubscriber<User> userTestSubscriber = mDatabase.userDao().getUser().test();

    userTestSubscriber.assertValueCount(0);

    // When inserting a new user in the data source
    mDatabase.beginTransaction();
    try {
        mDatabase.userDao().insertUser(USER);
        mDatabase.setTransactionSuccessful();

    } finally {
        mDatabase.endTransaction();
    }

    userTestSubscriber.assertValueCount(1);
    // this fails - the userTestSubscriber is still empty!
}

Note: this example is of course simplified, just to illustrate the issue.
Here's the generated DaoImpl.insertUser():
  public void insertUser(User user) {
    __db.beginTransaction();
    try {
      __insertionAdapterOfUser.insert(user);
      __db.setTransactionSuccessful();
    } finally {
      __db.endTransaction();
    }
  }

We can see that this code uses a transaction and also my test-code uses another transaction.
According to the SupportSQLiteDatabase-beginTransaction() docs, nesting transactions should be okay.
Is this maybe a room-bug?
The room version of this project is 1.0.0-alpha3, but I can also see this problem with version 1.0.0-alpha8 (in another project)
Reference to the Stackoverflow-question

Hi tmtron,
Thanks for the very detailed bug report, however these issues are for the samples. Can you please report it using this buganizer component?
https://issuetracker.google.com/issues/new?component=197448&template=878802
Thank you!

oh - sorry, I've mixed up the issue trackers.
For reference: I've filed #65471397 on the google issue tracker.
Thanks


The Serverless version does not satisfy the "frameworkVersion" in serverless.yml

Following the steps from the README.md and finally running serverless deploy --stage prod results into:

The Serverless version (1.25.0) does not satisfy the "frameworkVersion" (=1.11.0) in serverless.yml

Changing the serverless.yml line #1 into:
frameworkVersion: '=1.25.0'

fixes the error.

As mentioned by @cthos (thanks!) the above should hopefully resolve the issue seen, please do re-open if you are still seeing issues.


Base/Doc Descriptions

Initial attempt at base/doc forms as discussed in working group.
For discussion in WG meeting

Thanks, Sam!
I made a PR against your branch with some editorialization changes. https://github.com/TelegramSam/trust-establishment/pull/1
I will note that the data model section is not up to date with your change - but I think this is OK. Your change makes the optionality clear, and is useful to merge for a fruitful discussion on the next call. Then we can handle the rest of the spec.


Some Issues on Windows 10

So i tried to update to 6.2 on my windows machine and i get the error babel-runtime not installed i had to add it manually to the package.json file
"babel-runtime": "^6.3.13",

trie to compile the docs i get the following errors an warnings
Error in plugin 'sass'
Message:
    scss\settings\_settings.scss
Error: File to import not found or unreadable: util/util
       Parent style sheet: C:/gitprojects/foundation-sites/scss/settings/_settings.scss
        on line 43 of scss/settings/_settings.scss
>> @import 'util/util';
   ^

[21:42:51] Finished 'sass:foundation' after 6.73 s
Error in plugin 'sass'
Message:
    scss\settings\_settings.scss
Error: File to import not found or unreadable: util/util
       Parent style sheet: C:/gitprojects/foundation-sites/scss/settings/_settings.scss
        on line 43 of scss/settings/_settings.scss
>> @import 'util/util';
   ^

and
» [WARNING] @return must at least have a type. Location: _value.scss:96:101

Are you trying to upgrade a ZURB Template project? You say "compile the docs" but the docs should be compiling fine right now.

npm start

right now i also cant npm start v6.2.0-rc.1 error
Error in plugin 'sass'
Message:
    scss\settings\_settings.scss
Error: Undefined variable: "$primary-color".
        on line 140 of scss/settings/_settings.scss
>> $anchor-color: $primary-color;
   ---------------^

and
» [WARNING] @return must at least have a type. Location: _value.scss:96:101

and
fs.js:540
  binding.open(pathModule._makeLong(path),
          ^

TypeError: path must be a string
    at TypeError (native)
    at Object.fs.open (fs.js:540:11)
    at OpenReq.process (C:\gitprojects\foundation-sites\node_modules\mocha\node_modules\graceful-fs\graceful-fs.js:55:16)
    at OpenReq.Req (C:\gitprojects\foundation-sites\node_modules\mocha\node_modules\graceful-fs\graceful-fs.js:126:8)
    at new OpenReq (C:\gitprojects\foundation-sites\node_modules\mocha\node_modules\graceful-fs\graceful-fs.js:49:7)
    at Object.open (C:\gitprojects\foundation-sites\node_modules\mocha\node_modules\graceful-fs\graceful-fs.js:41:3)
    at Object.fs.writeFile (fs.js:1137:6)
    at C:\gitprojects\foundation-sites\node_modules\foundation-docs\lib\buildSearch.js:136:8
    at C:\gitprojects\foundation-sites\node_modules\foundation-docs\node_modules\mkdirp\index.js:48:26
    at FSReqWrap.oncomplete (fs.js:82:15)

maybe it's anissue on win10 or on my specific maschine

For the settings file error: the settings file has been updated so that shouldn't happen.
For the SassDoc warning, fixed that: 71b669aad3c9c9ecb13843b861fc423999d2d501
For the path must be a string warning, try updating a few of the framework dependencies:

npm install zurb/foundation-docs
npm install supercollider

i did npm update
dosn't this the same as npm install supercollider?

It should, but I guess I never know how npm update is going to behave—I've never been totally clear on what exactly it does and when.
But as long as the dependencies for foundation-docs and supercollider are up to date, the documentation should compile just fine.
And that "can't import" error for util/util, are you still getting that?

no errors now but i have to manually add
"babel-runtime": "^6.3.13",

to the package.json file on my windows 10 PC


Update java-jwt to 3.18.3

Updates com.auth0:java-jwt from 3.10.3 to 3.18.3.
GitHub Release Notes - Changelog - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "com.auth0", artifactId = "java-jwt" } ]

labels: library-update, early-semver-minor, semver-spec-minor, commit-count:1

Superseded by #116.


model with macro union gives incorrect results with 0.10.0

I have this model with a macro to generate a union:
{% set objecttype_query %}
select distinct source_definition from table_1
order by 1
{% endset %}

{% set results = run_query(objecttype_query) %}

{% if execute %}
{# Return the first column #}
{% set results_list = results.columns[0].values() %}
{% else %}
{% set results_list = [] %}
{% endif %}

{% for source_definition in results_list %}
(select
 ...
from {{ ref('table_2')}} as objecten , {{ ref('table_3')}} as relaties
where objecten.Definition = '{{ source_definition }}'
...
{% if not loop.last %}
UNION
 {% endif %}

{% endfor %}

with duckdb 0.9.2 this generates a table with 1049 rows (as expected)
when I upgrade to duckdb 0.10.0 I get a table with only 500 rows...
No other changes. running dbt-duckdb 1.7.2 in both cases.
What could cause this behaviour? It almost smells like a commit interval or checkpoint or something set a 500 and then skipping any furhter data?

Not quite sure how to help here w/o more info on what changed...

Is the compiled query different from 0.9.2 to 0.10.0?
If the query is the same, which rows in the output are different (i.e. which ones get dropped in 0.10.0?)

I'm  having troubles getting a reproducible case. However it does seem unrelated to dbt-duckdb after all, since I have the same issues querying my database though jupyter and/or dbeaver as well.


Handle panic in schema loader

@BCox-Janrain : my apologies, I didn't see your comment.
I started with that, but rawSchema isn't bytes.  I also looked into adding a test, but the tests are currently broken.


Remove repetitive branching on value drop

This was unnecessarily checking that the value is not reserved twice (on both sides).

Yeah the branch exists in Rust to ensure that we can constant-propagate and learn that drop(JsValue::null()) is a no-op. Note though that it also needs to be present because if you call some_imported_function(JsValue::null()) then JS is the one the drops the reference and needs to understand that it can't free everything.
https://github.com/rustwasm/wasm-bindgen/issues/1054 is an issue about removing these branches, both in Rust and in JS. With that issue implemented I believe both can be safely removed.

Yeah the branch exists in Rust to ensure that we can constant-propagate and learn that drop(JsValue::null()) is a no-op.

Yeah I assumed so but then thought that it's unlikely to be a frequent case, and in most cases this just performs an unnecessary check instead?
Feel free to close if you think otherwise.

Ok in the absence of data one way or another I think I'd prefer to leave it as is (with a true fix in #1054 eventually). If it's determined to be more beneficial to remove the branch though we certainly can!


Excel Business object - No Files

Describe the Bug with repro steps
1- Go to Logic Apps
2- Create any Excel Business object that needs a File (Path)
3- Select a valid location, valid Document Library
4- Try to look for a File.
Result: It will no show files, nor folders.

What type of Logic App Is this happening in?
Consumption (Portal)
Are you using Preview Designer or GA Designer
Preview
Workflow JSON
No response
Screenshots or Videos
No response
Browser
Edge
Additional context
No response

@agonzalezroyal - This is only supposed to show excel sheets so files with .xlsx extensions, if your folder does not have any then it will show No Items

@preetriti1 Yes, for me it is working on Power Automate, however this specific flow on Logic apps stopped working yesterday out of nowhere.
Now I can't select files, and there are files, they show up on Power Automate with the same object, and they were showing up on Logic Apps 2 weeks ago.

@agonzalezroyal - Since no data has been provided for 2 weeks now, we are going to close this issue. If you are still facing this please reopen this and provide the details as asked above.

this issue was happening because we were not using folders in file picker, should have been fixed in https://github.com/Azure/LogicAppsUX/pull/2603


[exec-if] step 1: move executor as a sub-dir of execution

Motivation
Create a dir execution. Move executor into execution.
Will create another dir executor-interface in execution later
part of exeuction interface sub-project in #2519
Have you read the Contributing Guidelines on pull requests?
Y
Test Plan
existing coverage.

@bors-libra r+

:pushpin: Commit 556565f has been approved by wqfish

@bors-libra r=wqfish

:bulb: This pull request was already approved, no need to approve it again.

There's another pull request that is currently being tested, blocking this pull request: #2745

:pushpin: Commit 556565f has been approved by wqfish

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

@davidiw I took the approach storage did. But it seems the executor case is more like consensus. I'll refactor to follow consensus.

@bors-libra r=wqfish

:pushpin: Commit 66c5ac8 has been approved by wqfish

:hourglass: Testing commit 66c5ac85740842c89bd46ea9d6abe2ae762eca55 with merge 95b4cfeb79bb41233f7315186685f2362482daa5...

:sunny: Test successful - checks-actions_land_blocking_test, checks-circle_commit_workflow
Approved by: wqfish
Pushing 95b4cfeb79bb41233f7315186685f2362482daa5 to master...


Helm chart SSL hints

adding hints in the values.yaml for if a customer needs custom certs in the box for http proxies, which if they are using the relay proxy odds are theres a MITM proxy at play.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Iss20 graph tests

Codecov Report

:exclamation: No coverage uploaded for pull request base (dev@a22ff5b). Click here to learn what that means.
The diff coverage is 94.44%.

@@          Coverage Diff           @@
##             dev      #30   +/-   ##
======================================
  Coverage       ?   90.21%
======================================
  Files          ?       15
  Lines          ?      695
  Branches       ?        0
======================================
  Hits           ?      627
  Misses         ?       68
  Partials       ?        0

Impacted Files
Coverage Δ

tests/test_vector_label/test_graph.py
100% <100%> (ø)

cw_geodata/vector_label/graph.py
77.45% <83.33%> (ø)

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 a22ff5b...227be03. Read the comment docs.


[Umbrella] KubeRay benchmark plan

Search before asking

[X] I had searched in the issues and found no similar feature requirement.

Description

Memory benchmark: Benchmark the memory usage for KubeRay with different numbers of Pods and custom resources. I believe that RayService has a memory leak issue #1156(comment) at this moment.

Scalability benchmark: The memory benchmark is one of the important factors for scalability. However, we also need to determine if there are any other scalability bottlenecks in KubeRay. KubeRay should be able to support scalability for at least 95% of users' use cases. We need to understand the scale that we target on.

Kubernetes API Server: We need to benchmark the number of requests that the KubeRay operator sends to the Kubernetes API Server (#1062).

Ray Autoscaler: The KubeRay node provider directly fetches information from the Kubernetes API Server instead of using an informer cache. I am unsure whether this approach will cause any scalability issues.

RayService

QPS: We should get a similar result as this benchmark.
High availability / Zero downtime: Maybe we should have chaos testing for it.

How many queries will be lost in the chaos environment?

Fault tolerance: Ray Serve seems not to be well-supported by GCS fault tolerance (#1058).

Use case
No response
Related issues
No response
Are you willing to submit a PR?

[X] Yes I am willing to submit a PR!

@kpouget This may be interesting to you as well.

QPS: We should get a similar result as this benchmark.

@kevin85421 The benchmark link returns 404 Not Found

Memory & scalability benchmarks: #1324.

We need to measure the scalability as one of the criterion in our product. Benchmark on the reconcile metrics need to be measured for scalability and I will try to come up a plan if I get some bandwidth this week.

Close this one, and use #2069 instead.


CreateShipment: Fix to Show Alert When No Active Transfer Agreement Available

Alert style change to warning
Fix caching problem

 2.0 BUG: Error message on cannot create shipment if no accepted agreement available

@vahidbazzaz @HaGuesto caching issue isn't fixed.

@vahidbazzaz @HaGuesto caching issue isn't fixed.

@aerinsol  Could you please let me know which user you have tried with, this seems to be working fine with boxaid/boxcare users in staging.

Well now I can't even renew an agreement with BoxCare without it throwing
an error -_-. Using @.***
I declined everything, created two new agreements with
TestOrg/DummyTestOrgwithBoxes, and then when I tried to renew with Boxcare
I just get errors.
On Sat, Nov 25, 2023 at 11:18 PM Vahid Bazzaz @.***>
wrote:

@vahidbazzaz https://github.com/vahidbazzaz @HaGuesto
https://github.com/HaGuesto caching issue isn't fixed.
@aerinsol https://github.com/aerinsol Could you please let me know
which user you have tried with, this seems to be working fine with
boxaid/boxcare users in staging.
—
Reply to this email directly, view it on GitHub
https://github.com/boxwise/boxtribute/pull/1117#issuecomment-1826355875,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABP2BFBBB7Y36BJX25CS3CTYGID3HAVCNFSM6AAAAAA7RRLMWOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQMRWGM2TKOBXGU
.
You are receiving this because you were mentioned.Message ID:
@.***>


Fix #1086 #1087 - create() issues

Fixes #1086 and #1087

@calina-c please review. FYI the only failing tests are from 'replacement transaction underpriced' in test_mumbai. That's outside the scope of this issue; it's for #1089.


Fixed #35604, Refs #35326 -- Made FileSystemStorage.exists() behaviour independent from allow_overwrite.

Trac ticket number
ticket-35604
Branch description
Partially reverts 0b33a3abc2ca7d68a24f6d0772bc2b9fa603744e.
Storage.exists(name) was documented to "return False if the name is available for a new file." but return True if the file exists. This is ambiguous in the overwrite file case.
Overwriting get_available_name() should still ensure all file name validation is applied, ref CVE-2024-39330.
This is motivated from this discussion on django-storages: https://github.com/jschneier/django-storages/issues/1430
Note: this concern was already raised by @bcail here (thank you). At the time, I believed the impact to be limited and the implementation to still be within the definition of Storages.exists(name).
This is to be backported to 5.1
Checklist

[X] This PR targets the main branch.
[X] The commit message is written in past tense, mentions the ticket number, and ends with a period.
[x] I have checked the "Has patch" ticket flag in the Trac system.
[x] I have added or updated relevant tests.
[x] I have added or updated relevant docs, including release notes if applicable.
[ ] I have attached screenshots in both light and dark modes for any UI changes.

The only issue I can see is if the name doesn't fit inside the length requirement we won't handle that.

The only issue I can see is if the name doesn't fit inside the length requirement we won't handle that.

Thank you! I had not taken that into account. I've changed the approach adding a new private method _name_is_unavailable which is overridden in the FileSystemStorage to handle the allow_overwrite case.
Added some tests around the filename truncation, needs a fresh review 👍

There is something about the backend that allows overwrite that bothers me when the max_length parameter is not None: calling save with the same file name and different max_length values would result in multiple files being stored:

        name = "test_long_file_name.txt"
        file = ContentFile(b"content")
        stored_name = self.storage.save(name, file, max_length=10)
        self.assertEqual(stored_name, "test_l.txt")
        stored_name = self.storage.save(name, file, max_length=11)
        self.assertEqual(stored_name, "test_lo.txt")

I think this result is OK and makes sense, but it feels odd to some extent.

I believe this is also true when allow overwrite is False, as it's saved with a different name
I think that max_length mostly comes from FileField.max_length (so is quite a static value)
Not sure if that makes it feel less odd?

I wonder if, in a near future, _name_is_unavailable should be a public API for each backend to implement as they need. > Perhaps we should mention and consider this in ticket-35607 for 5.2?

Sure, can also be generic that we review the API and what has/should be made public and can be overwritten


Use pathbuffer reduce stack

Summary
Why is this change necessary?
Benefit from the lib_pathbuffer feature introduced in https://github.com/apache/nuttx/pull/12969, we can further optimize the stack usage size in the NuttX kernel.
Changes
In this change, we replaced the declarations related to xxx[PATH_MAX] in the NuttX directory with the lib_pathbuffer method
Impact
Is a new feature added?: NO
Impact on build: NO
Impact on hardware: NO, this change does not specifically target any particular hardware architectures or boards.
Impact on documentation: NO,This patch does not introduce any new features
Impact on compatibility: This implementation aims to be backward compatible. No existing functionality is expected to be broken.
Testing
Build Host(s): Linux x86
Target(s): sim/nsh
By default
LIBC_MAX_PATHBUFFER = 2
LIBC_PATHBUFFER_MALLOC = y

So when there is not enough memory on the stack, it will be requested from the heap

[Experimental Bot, please feedback here]
Yes, this PR appears to meet the NuttX requirements based on the provided information.  However, there are a couple of minor improvements that could be made to strengthen the testing section.
While the PR states testing was done on sim/nsh, the "Testing logs before change" and "Testing logs after change" sections are empty.  Providing concrete examples of the stack usage before and after the change would significantly improve the review process.  For example, showing the output of a command that displays stack usage, or even a code snippet demonstrating how the stack size was measured would be beneficial.  This would provide concrete evidence of the optimization.

I submitted a fix for the redefinition problem in libc under kernel build when using ffs/ffsl/ffsll.
https://github.com/apache/nuttx/pull/14632/commits/a9e1d1bd69322041229bbb2e7fbf4d61d56e6e5f

In CI, it is found that when executing kernel build in non-kernel scenarios, builtin_ffs will not be found, but CONFIG_HAVE_BUILTIN_FFS will be defined
riscv-none-elf-ld: /github/workspace/sources/nuttx/staging/libc.a(lib_pathbuffer.o): in function `lib_get_pathbuffer':
/github/workspace/sources/nuttx/libs/libc/misc/lib_pathbuffer.c:89:(.text.lib_get_pathbuffer+0x1c): undefined reference to `ffs'
make[1]: *** [Makefile:189: nuttx] Error 1
make: *** [tools/Unix.mk:551: nuttx] Error 2

But the same thing happens on riscv's ci: the error message "undefined reference to `ffs'" still appears.
Is there something wrong with the riscv related configuration?
====================================================================================
Configuration/Tool: ox64/spi
2024-11-09 15:15:26
------------------------------------------------------------------------------------
  Cleaning...
  Configuring...
  Building NuttX...
riscv-none-elf-ld: /github/workspace/sources/nuttx/staging/libkarch.a(bl808_irq.o): in function `ffs':
/github/workspace/sources/nuttx/include/strings.h:82:(.text.__m0ic_interrupt+0x1e): undefined reference to `ffs'
riscv-none-elf-ld: /github/workspace/sources/nuttx/include/strings.h:82:(.text.__m0ic_interrupt+0x30): undefined reference to `ffs'
make[1]: *** [Makefile:189: nuttx] Error 1
make: *** [tools/Unix.mk:551: nuttx] Error 2
make: Target 'all' not remade because of errors.
/github/workspace/sources/nuttx/tools/testbuild.sh: line 385: /github/workspace/sources/nuttx/../nuttx/nuttx.manifest: No such file or directory
  Normalize ox64/spi
HEAD detached at pull/14632/merge
Untracked files:
  (use "git add <file>..." to include in what will be committed)
	boards/risc-v/bl808/ox64/scripts/ld.script.tmp

nothing added to commit but untracked files present (use "git add" to track)

Change ffs in lib_getpathbuffer to ffsl that everything will be OK

Sorry @crafcat7 I think spresense/example_camera is failing because of this PR:
Configuration/Tool: spresense/example_camera,CONFIG_ARM_TOOLCHAIN_GNU_EABI
mount/fs_automount.c: In function 'automount_initialize':
Error: mount/fs_automount.c:816:7: error: 'return' with no value, in function returning non-void [-Werror=return-type]
  816 |       return;
      |       ^~~~~~
In file included from mount/fs_automount.c:43:
/github/workspace/sources/nuttx/include/nuttx/fs/automount.h:176:11: note: declared here
  176 | FAR void *automount_initialize(FAR const struct automount_lower_s *lower);
      |           ^~~~~~~~~~~~~~~~~~~~

https://github.com/NuttX/nuttx/actions/runs/11796808033/job/32859585604#step:7:338

@lupyuen do you know why ci in this pr doesn't catch this error, but https://github.com/apache/nuttx/pull/14748 can.

Sorry @crafcat7 I think spresense/example_camera is failing because of this PR:
Configuration/Tool: spresense/example_camera,CONFIG_ARM_TOOLCHAIN_GNU_EABI
mount/fs_automount.c: In function 'automount_initialize':
Error: mount/fs_automount.c:816:7: error: 'return' with no value, in function returning non-void [-Werror=return-type]
  816 |       return;
      |       ^~~~~~
In file included from mount/fs_automount.c:43:
/github/workspace/sources/nuttx/include/nuttx/fs/automount.h:176:11: note: declared here
  176 | FAR void *automount_initialize(FAR const struct automount_lower_s *lower);
      |           ^~~~~~~~~~~~~~~~~~~~

https://github.com/NuttX/nuttx/actions/runs/11796808033/job/32859585604#step:7:338

Okay, I submitted a fix on the https://github.com/apache/nuttx/pull/14752


docs: add custatevec info

Description
Include description of feature this PR introduces or a bug that it fixes. Include the following information:

Context: why is it needed? In case of bug, what was the cause for the bug?

To include new information from our work with Nvidia

Concise description of the implemented solution.

added info to the integrations page and noted that custatevec is part of the cirq package

Please verify that you have completed the following steps

[x] I have self-reviewed my code.
[x] I have included test cases validating introduced feature/fix.
[x] I have updated documentation.

I updated the link to point to the main branch, which will work once https://github.com/zapatacomputing/orquestra-cirq/pull/19 is merged.


⚠ NCAA has degraded performance

In 90ac9a9, NCAA (https://www.ncaa.com) experienced degraded performance:

HTTP code: 200
Response time: 558 ms

Resolved: NCAA performance has improved in 6b32ad7.


fix: memorize unique id

slugify is a relatively expensive operation because it has to do unicode work.
Ideally we could use self._attr_unique_id in __init__ but self.type is set after the parent's __init__ is called.

Thanks. And just FYI, the technical term is memoize, not memorize. ;)


Unreduced + inertia-free fix

This PR fixes the failure that happened for the Unreduced + inertia-free option caused by the not properly initialized vectors

Codecov Report

Merging #159 (b8a67d7) into master (9c07280) will increase coverage by 0.11%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master     #159      +/-   ##
==========================================
+ Coverage   74.46%   74.58%   +0.11%
==========================================
  Files          32       32
  Lines        3380     3380
==========================================
+ Hits         2517     2521       +4
+ Misses        863      859       -4

Impacted Files
Coverage Δ

src/interiorpointsolver.jl
95.52% <100.00%> (ø)

lib/MadNLPHSL/src/ma86.jl
84.90% <0.00%> (+3.77%)
:arrow_up:

lib/MadNLPHSL/src/ma97.jl
84.00% <0.00%> (+4.00%)
:arrow_up:

Continue to review full report at Codecov.

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


api: Improve pagination

We want to improve/make pagination ordering work by at least some commonly used parameters.
TODO:

[ ] look over Etherscan's implementation
[ ] Get Product input

Can you say more about the way(s) in which the current pagination mechanism is not ideal?
Here's an improvement I'd like to see:

offset is not stable over time. Notably, it is not stable from the time we display a list of (e.g.) 20 latest blocks, to the time the user clicks "next page". If 20 new rounds have completed in the meantime, clicking "next page" will get the user the exact same set of blocks -- not what they expected, presumably.
Better to have a continuation token here instead of an offset. The simplest continuation token is just the identifier of the last currently displayed block.

(first discussed/noticed in a comment in the Explorer API doc)

😅 I think this was intended to capture that some query params were not actively used, but the case for at least ORDER was addressed https://github.com/oasisprotocol/oasis-indexer/pull/110.

Let's just repurpose this for future pagination improvements.
I do think we should be more clear or precise with the implied order of the returned data unless we want to just go with height.

Short-term goal: Get input from Product whether we should sink time into this, or commit to living with offset-based pagination in the long run. Have them understand that switching later (once UI is built) will cost at least twice as much.

https://app.clickup.com/t/3t22jx5

Looks unlikely we'll pursue this. Closing at least for now.