Possible conflict between azure-cli package and fence-agents-azure-arm package

This is autogenerated. Please review and update as needed.
Describe the bug
when installing the fence-agents-azure-arm package to configure azure fence agent on RHEL cluster as per the document:
https://docs.microsoft.com/en-us/azure/virtual-machines/workloads/sap/high-availability-guide-rhel-pacemaker#cluster-installation
and then trying to install the azure cli on the same machine, the azure cli commands is failing , a below is an example "az login".
the installtion of fence-agents-azure-arm package is installing the python3-azure-sdk package, but I am not sure how this would affect the use of the azure cli.
if the python3-azure-sdk package is removed, the azure cli commands works fine.
note : trying the older versions of azure-cli would not help too.
would you please help us to understand how they are correlated ? and if there is any solution to have both of them on the same machine ?
Command Name
az login
Errors:
The command failed with an unexpected error. Here is the traceback:
__init__() got an unexpected keyword argument 'user_agent'
Traceback (most recent call last):
  File "/lib64/az/lib/python3.6/site-packages/knack/cli.py", line 231, in invoke
    cmd_result = self.invocation.execute(args)
  File "/lib64/az/lib/python3.6/site-packages/azure/cli/core/commands/__init__.py", line 658, in execute
    raise ex
  File "/lib64/az/lib/python3.6/site-packages/azure/cli/core/commands/__init__.py", line 721, in _run_jobs_serially
    results.append(self._run_job(expanded_arg, cmd_copy))
  File "/lib64/az/lib/python3.6/site-packages/azure/cli/core/commands/__init__.py", line 692, in _run_job
    result = cmd_copy(params)
  File "/lib64/az/lib/python3.6/site-packages/azure/cli/core/commands/__init__.py", line 328, in __call__
    return self.handler(*args, **kwargs)
  File "/lib64/az/lib/python3.6/site-packages/azure/cli/core/commands/command_operation.py", line 121, in handler
    return op(**command_args)
  File "/lib64/az/lib/python3.6/site-packages/azure/cli/command_modules/profile/custom.py", line 158, in login
    use_cert_sn_issuer=use_cert_sn_issuer)
  File "/lib64/az/lib/python3.6/site-packages/azure/cli/core/_profile.py", line 175, in login
    subscriptions = subscription_finder.find_using_specific_tenant(tenant, credential)
  File "/lib64/az/lib/python3.6/site-packages/azure/cli/core/_profile.py", line 815, in find_using_specific_tenant
    client = self._create_subscription_client(credential)
  File "/lib64/az/lib/python3.6/site-packages/azure/cli/core/_profile.py", line 839, in _create_subscription_client
    **client_kwargs)
TypeError: __init__() got an unexpected keyword argument 'user_agent'

To Reproduce:
Steps to reproduce the behavior. Note that argument values have been redacted, as they may contain sensitive information.
it can be tested also on centos 8 machines with HA repos enabled, and azure cli repos enabled.

Put any pre-requisite steps here...
yum install fence-agents-azure-arm
yum install azure-cli
az login --service-principal --allow-no-subscriptions -u {} -p {} -t {}

Expected Behavior
to be able to login successfully
Environment Summary
Linux-4.18.0-269.el8.x86_64-x86_64-with-centos-8, CentOS Stream 8
Python 3.6.8
Installer: RPM

azure-cli 2.31.0

Additional Context

@jiasli for awareness

@imabedalghafer, I am not familiar with fence-agents-azure-arm, but I can't seem to install them:
docker run -it --rm centos

[root@8f0358034b9c /]# yum install python3-azure-sdk
Failed to set locale, defaulting to C.UTF-8
CentOS Linux 8 - AppStream                                                              179 kB/s | 8.4 MB     00:47
CentOS Linux 8 - BaseOS                                                                 743 kB/s | 3.6 MB     00:05
CentOS Linux 8 - Extras                                                                  12 kB/s |  10 kB     00:00
No match for argument: python3-azure-sdk
Error: Unable to find a match: python3-azure-sdk
...
[root@8f0358034b9c /]# yum install -y pcs pacemaker fence-agents-azure-arm nmap-ncat
Failed to set locale, defaulting to C.UTF-8
Last metadata expiration check: 0:00:25 ago on Fri Dec 24 02:50:10 2021.
No match for argument: pcs
No match for argument: pacemaker
No match for argument: fence-agents-azure-arm
Error: Unable to find a match: pcs pacemaker fence-agents-azure-arm

Do I need to add some repo first?
Azure CLI has an entry script /usr/bin/az:
# command -v az
/usr/bin/az

# cat /usr/bin/az
#!/usr/bin/env bash
bin_dir=`cd "$(dirname "$BASH_SOURCE[0]")"; pwd`
python_cmd=python3
if command -v python3.6 &>/dev/null; then python_cmd=python3.6; fi
AZ_INSTALLER=RPM PYTHONPATH="$bin_dir/../lib64/az/lib/python3.6/site-packages" $python_cmd -sm azure.cli "$@"

azure-mgmt-resource SDK is installed in a virtual environment. That's why the entry script /usr/bin/az has
PYTHONPATH="$bin_dir/../lib64/az/lib/python3.6/site-packages"

Perhaps you can share the result of
# PYTHONPATH="$bin_dir/../lib64/az/lib/python3.6/site-packages" python3.6 -m pip list -v
...
azure-mgmt-resource                     20.0.0    /usr/lib64/az/lib/python3.6/site-packages pip

@jiasli Thank you for your reply.
I do apologize I forget to mention, that yes, you would need to enable the high availability repository of Centos , which can be found under:/etc/yum.repos.d/CentOS-Stream-HighAvailability.repo
you would need to edit the file and enable the repository , or it can be done via below:
sed -i 's/enabled=0/enabled=1/g' /etc/yum.repos.d/CentOS-Stream-HighAvailability.repo

The output for the above command (the verbose did not return for me the full path for that package) :
# PYTHONPATH="$bin_dir/../lib64/az/lib/python3.6/site-packages" python3.6 -m pip list --format columns -vvv | grep azure-mgmt-resource
azure-mgmt-resource                              2.1.0
azure-mgmt-resourcegraph                         1.0.0

Please let me know if any further information is needed.

For the provided result
The version of azure-mgmt-resource is 2.1.0 which is different from what azure-cli requires - 20.0.0. Also, the installation location is missing from the result of pip list command. We need that information to identify where azure-mgmt-resource 2.1.0 is installed.
It is possible PYTHONPATH is not set correctly, so the system pip is invoked. Could you try
# ls /lib64/az/lib/python3.6/site-packages

It should show azure-cli's lib.
Repro
There doesn't seem to be a CentOS Stream official image, so I tried the official centos image and also reproduced:
> docker run -it --rm centos

# sed -i 's/enabled=0/enabled=1/g' /etc/yum.repos.d/CentOS-Linux-HighAvailability.repo
# yum install -y pcs pacemaker fence-agents-azure-arm nmap-ncat

# rpm --import https://packages.microsoft.com/keys/microsoft.asc
# echo -e "[azure-cli]
name=Azure CLI
baseurl=https://packages.microsoft.com/yumrepos/azure-cli
enabled=1
gpgcheck=1
gpgkey=https://packages.microsoft.com/keys/microsoft.asc" | tee /etc/yum.repos.d/azure-cli.repo
# yum install azure-cli

# az login
TypeError: __init__() got an unexpected keyword argument 'user_agent'

Root cause
Because fence-agents-azure-arm installs azure-mgmt-resource 2.1.0 which is under a regular packages azure-mgmt due to the /usr/lib/python3.6/site-packages/azure/mgmt/__init__.py file, it takes precedence over azure-mgmt-resource 20.0.0 which is under a namespace package azure-mgmt installed at /usr/lib64/az/lib/python3.6/site-packages.
# PYTHONPATH="/lib64/az/lib/python3.6/site-packages" python3.6 -m pip list --format columns -vvv | grep azure-mgmt-resource
azure-mgmt-resource                              20.0.0      /usr/lib64/az/lib/python3.6/site-packages pip
azure-mgmt-resourcegraph                         1.0.0       /usr/lib/python3.6/site-packages

# PYTHONPATH="/lib64/az/lib/python3.6/site-packages" python3.6 -c "import azure.mgmt.resource; print(azure.mgmt.resource.__path__)"
['/usr/lib/python3.6/site-packages/azure/mgmt/resource']

# PYTHONPATH="/lib64/az/lib/python3.6/site-packages" python3.6 -c "import azure.mgmt; print(azure.mgmt)"
<module 'azure.mgmt' from '/usr/lib/python3.6/site-packages/azure/mgmt/__init__.py'>

# cat /usr/lib/python3.6/site-packages/azure/mgmt/resource/version.py
VERSION = "2.1.0"

As az is invoked using the system python, include-system-site-packages actually doesn't work here:
# cat /lib64/az/pyvenv.cfg
home = /usr/bin
include-system-site-packages = false
version = 3.6.8

We've seen similar issue before: https://github.com/Azure/azure-cli/issues/19027
Possible solution

As a solution, we can consider using the python in the virtual env to launch azure.cli, so that include-system-site-packages is honored.
You may also consider reporting this issue to fence-agents-azure-arm since it is using a very old and deprecated azure-mgmt-resource 2.1.0.

Workaround
Current as a workaround, you may use pip to install Azure CLI to make it isolated from the system python lib:
python3 -m venv az-env
. az-env/bin/activate
pip install --upgrade pip
pip install azure-cli

@jiasli
Thank you so much for your help and analysis.
Regarding the first solution :

As a solution, we can consider using the python in the virtual env to launch azure.cli, so that include-system-site-packages is honored.

would you please elaborate more about the solution (as far as I understood, it would be something similar to the workaround) ?
Also regarding the workaround, would it be possible to update the az script as of this scenario to automatically activate the virtual environment  and start using the az command from there ?
Regards,
Ibrahim Abedalghafer

@jiasli
What I meat for the update of the az command to activate the virtual environment in the az command is something like below:
# vi /bin/az
--
#!/usr/bin/env bash
#bin_dir=`cd "$(dirname "$BASH_SOURCE[0]")"; pwd`
#python_cmd=python3
#if command -v python3.6 &>/dev/null; then python_cmd=python3.6; fi
#AZ_INSTALLER=RPM PYTHONPATH="$bin_dir/../lib64/az/lib/python3.6/site-packages" $python_cmd -sm azure.cli "$@"
## Adding the below as a workaround for issue :  20820 ####
source /root/az-env/bin/activate
python -sm azure.cli "$@"

I have tested it and it is working , and just want to confirm if it can be considered as a valid workaround to use.

would you please elaborate more about the solution (as far as I understood, it would be something similar to the workaround) ?

Ah. "We" here means us, Microsoft. If you check our DEB packages, it actually contains its own python. I am not really sure why the same is not done for the RPM package (it was developed by the previous team years ago), but I will definitely work on this in the future.
Actually, you don't need to activate the venv for az to work. You can simply launch az in the venv:
docker run -it --rm --workdir /root python bash

python -m venv az-env
az-env/bin/pip install azure-cli
az-env/bin/az --version

You may also add az-env/bin/ to PATH so that az is available everywhere:
export PATH="/root/az-env/bin:$PATH"
az --version

@jiasli
Thank you so much for your help on the case.
The tests now are working fine with the provided workaround.
please feel free to close the issue or if you would like to keep it open for future reference it is up to you 😊


modified the signupvalidation so that a symbol must be included in the password field

now a symbol must be included in password field

Fixes: https://github.com/Peacexoom/AnimeHub/issues/43

Thank you @Rockleemode for your contribution.


Update React 18 post for beta

Adds a note to the top of the page linking to the 18 Beta announcement.

Intentionally fully bold? Otherwise you need a linebreak.


Improve RequestsFetcher cleanup mechanisms

Description or feature request:
Implement RequestsFetcher as a context manager and/or provide public cleanup method to be called by its users (for ex. fetcher.close() )
Current behavior:
RequestsFetcher internlly tracks all exit paths to release allocated resources which is error prone.
Expected behavior:
Reliably release all resources allocated by RequestsFetcher.

I think fetch() being a CM might make sense (if it makes the client code nicer) but I don't believe it makes it any easier to handle cleanup...
The only resource (that requires handling)  in RequestsFetcher is the requests.Response: this is tricky because we return a generator function that owns the Response but otherwise I think there's nothing that requires attention.
fetcher.close() could not solve any issues I believe: there is no "context" in fetcher itself that is problematic. fetch() does have to handle the Response but that can't be fixed with more methods as once we return the generator that owns the Reponse, it is no longer something we should or can worry about.

Reviving this issue after having a look at RequestsFetcher after a while.
Implementing fetch() as a context manager makes the default TUF implementation look better because now response is closed in a single place. For example:
requests_fetcher.py:
    @contextmanager
    def fetch(self, url, required_length):
        ...
        try:
            response.raise_for_status()
            yield self._chunks(response, required_length)

        except requests.HTTPError as e:
            status = e.response.status_code
            raise exceptions.FetcherHTTPError(str(e), status)

        finally:
            response.close()

download.py:
-            chunks = self._fetcher.fetch(url, required_length)
-            for chunk in chunks:
-                temp_file.write(chunk)
-                number_of_bytes_received += len(chunk)
+            with self._fetcher.fetch(url, required_length) as chunks:
+                for chunk in chunks:
+                    temp_file.write(chunk)
+                    number_of_bytes_received += len(chunk)
+

But this also means adding a requirement to FetcherInterface.fetch() to be a context manager and I can't decide if it is adding enough value?

@sechkova do you think this is still an active issue -- I think moving the download methods into Fetcher have solved the issues around this? I don't see our code having trouble with unclosed things currently

Yeah, with download_file() being context manager and RequestsFetcher correctly cleaning up sockets, I think this is solved. Closing it.


"Ignoring" someone is too strong

I see that to Ignore someone means turning off their

video
audio
chat messages

That may be OK for an annoying person in a public chat, but our use case is cancelling echos when sitting in the same room. For that, you should definitely see the chat messages (with links to issue trackers) and possibly also the video (if the other party is sitting back to back with you).

Canceling echoes without ignoring is already possible with right click on the participant thumbnail -> mute. That will mute that participant only locally.

On the other hand, the ignore button has proven to be useful to reset the connection with a peer without quiting the room. If you are not getting audio, video or data from some concrete participant, you can ignore him and stop ignoring him immediately afterwards. That resets the three channels and has become the main use case for the ignore button. :wink:

I'm cleaning up the list of issues. As explained in the previous comment, browsers already allow "ignoring" the audio, so closing this.


Not Working With Magisk 20.3

Magisk Fully Updated and Working

Energized Loads Fine at First

Fails & Hangs

Magisk binary is com.txQQ345io4oFgPML which is randomized. Was working fine before, but maybe now it's not?

Getting the same error with Energized 4.9 and Magisk ce7cb1ee (20306)

Same here in Samsung Galaxy On7 running Android 6.0.1 with Magisk 20.4. Please look into this issue.

pretty clear this is a wont fix. I've since moved on.

Moved on to what? 🤔

AdAway.


Adding Title, Description, and UA Code.

This adds key features to the jekyll config.

pinging github/pages

Adding cla: yes label, as the PR author is a googler at the time of this PR was created.


x/tools/cmd/goimports: incorrectly walks GOPATH nested within the current module

What version of Go are you using (go version)?

$ go version
go version go1.12beta1 darwin/amd64

Does this issue reproduce with the latest release?
Yes the issue is reproducible with the latest version of go (go1.12beta1)
What operating system and processor architecture are you using (go env)?
go env Output
$ go env
GOARCH="amd64"
GOBIN=""
GOCACHE="/Users/subhash_sharma/Library/Caches/go-build"
GOEXE=""
GOFLAGS=""
GOHOSTARCH="amd64"
GOHOSTOS="darwin"
GOOS="darwin"
GOPATH="/Users/subhash_sharma/go"
GOPROXY=""
GORACE=""
GOROOT="/Users/subhash_sharma/.goversions/go1.12beta1"
GOTMPDIR=""
GOTOOLDIR="/Users/subhash_sharma/.goversions/go1.12beta1/pkg/tool/darwin_amd64"
GCCGO="gccgo"
CC="clang"
CXX="clang++"
CGO_ENABLED="1"
GOMOD="/Users/subhash_sharma/subash/go.mod"
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -m64 -pthread -fno-caret-diagnostics -Qunused-arguments -fmessage-length=0 -fdebug-prefix-map=/var/folders/w0/d6l19r7j3dbg7fq56_nn4l340000gp/T/go-build083506298=/tmp/go-build -gno-record-
gcc-switches -fno-common"

What did you do?
The following testscript reproduces the issue:
# goimportsbug.txt

go mod download<EMAIL_ADDRESS>go mod download
goimports -l main.go

-- go.mod --
module mod

require fruit.com v1.0.0

replace fruit.com => github.com/user/banana v1.0.0

-- main.go --
package main

import (
    "fmt"

    // missing import for core below
)

func main() {
    fmt.Println(core.Orange)
}
-- core/core.go --
package core

type T struct{}
var Orange T

# mod/fruit.com_v1.0.0.txt

-- .mod --
module fruit.com

-- .info --
{"Version":"v1.0.0","Time":"2018-10-22T18:45:39Z"}

-- go.mod --
module fruit.com

-- fruit/fruit.go --
package fruit

const Apple = "apple"
-- coretest/coretest.go --
// package coretest becomes a candidate for the missing
// core import in main above
package coretest

const Mandarin = "mandarin"

# mod/github.com_user_banana_v1.0.0.txt

-- .mod --
module fruit.com

-- .info --
{"Version":"v1.0.0","Time":"2018-10-22T18:45:39Z"}

-- go.mod --
module fruit.com

-- fruit/fruit.go --
package fruit

const Apple = "apple"
-- coretest/coretest.go --
// package coretest becomes a candidate for the missing
// core import in main above
package coretest

const Mandarin = "mandarin"

What did you expect to see?
Output:
main.go

because main.go is missing an import to satisfy the qualified identifier core..
What did you see instead?
go [list -e -json -compiled -test=false -export=false -deps=false -find=true -- ./core<EMAIL_ADDRESS><EMAIL_ADDRESS>.]: exit
status 1: go: directory<EMAIL_ADDRESS>outside available modules

Note that in this test, our GOPATH happens to be nested within the main module; GOPATH=$WORK/gopath and GOMOD=$WORK/go.mod.
goimports seems to unconditionally walk all subdirectories of the main module, including the nested GOPATH (which includes the pkg/mod subdirectory). This appears to be wrong; we should probably prune the GOPATH subdirectory tree for this phase of the walk.
Note the error triggered by this unconditional walking is not a goimports bug per se, but rather this issue
A clear workaround for this rather obscure edge case is to either not nest your GOPATH or prefix the nested GOPATH directory name with a hiding character (_ or .).
cc @heschik @bradfitz @ianthehat @myitcv

You could also put a .goimportsignore config file in the GOPATH root.
Thanks for the thorough bug report. I'm on the fence about fixing this. I guess I understand why, pre-modules, it might make sense to check in GOPATH to accomplish basically the same thing as /vendor. But what's the point with modules? Why not just leave GOPATH outside the repository?

@subash-a and I work together so I'll chime in here too

Why not just leave GOPATH outside the repository?

Yes, totally. As @subash-a pointed out above, this is one solution, as is prefixing the name of the directory with a hiding character.
We raised this issue more out of completeness than anything. @subash-a come across this whilst tracking down https://github.com/golang/go/issues/29550.

We raised this issue more out of completeness than anything.

Okay, thanks. I take that as meaning that this won't cause you serious trouble. Unless you think this is a common setup, I'm inclined to call this a one-off and close the issue. Comment if you disagree, of course.
If someone else is reading this because they had the same problem, please leave a note here so we can reconsider fixing it. Thanks.


Update zio-aws-dynamodb, zio-aws-netty to <IP_ADDRESS>

Updates

dev.zio:zio-aws-dynamodb
dev.zio:zio-aws-netty

from <IP_ADDRESS> to <IP_ADDRESS>.
GitHub Release Notes - 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!

Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "dev.zio" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "30 days" },
  dependency = { groupId = "dev.zio" }
}]

labels: library-update, commit-count:1

Superseded by #453.


fix: support for OpenResty 1.11.2.*

This PR cleans up and replaces #16

need to make sure we capture the use case of compiling / building to one location with the intention of packaging and installing to another.
Specifically
            --with-lua=$OPENRESTY_INSTALL/luajit \
            --with-lua-include=$OPENRESTY_INSTALL/luajit/include/luajit-2.1

needs to reference the final end result but during compilation / buildling the location also has to exist. I don't like and hope we can find a better solution then the one where I double make install'd

Test scenario:
LUAROCKS_PREFIX=/usr/local \
LUAROCKS_DESTDIR=/tmp/build \
OPENRESTY_PREFIX=/usr/local/openresty \
OPENRESTY_DESTDIR=/tmp/build \
OPENSSL_PREFIX=/usr/local/kong \
OPENSSL_DESTDIR=/tmp/build \
/tmp/kong-ngx-build -p /tmp/build/usr/local \
--openresty $RESTY_VERSION \
--openssl $RESTY_OPENSSL_VERSION \
--luarocks $RESTY_LUAROCKS_VERSION \
--pcre $RESTY_PCRE_VERSION \
--work /work
cp -R /tmp/build/* /

Result:
luarocks config

   LUA = "/tmp/build/usr/local/openresty/luajit/bin/luajit",
   LUALIB = "libluajit-5.1.a",
   LUA_BINDIR = "/tmp/build/usr/local/openresty/luajit/bin",
   LUA_DIR = "/tmp/build/usr/local/openresty/luajit",
   LUA_INCDIR = "/tmp/build/usr/local/openresty/luajit/include/luajit-2.1",
   LUA_LIBDIR = "/tmp/build/usr/local/openresty/luajit/lib",

Well, fixing this use-case is outside the scope of this PR (which already extended quite beyond its original one...) I will be merging this for now :)


File name of scss output

Hi,
File name of scss output should starting by '_' like: _icon.svg.scss.
Because when lib-sass compile these files, it'll create : icon.svg.css, icon.png.css and icon.fallback.css in dist folder.
I think users can import these files as a dependencies in there main.scss like so:
@import "icon.svg";
What do you think?

Hm, sounds like a plan. Will push this to a future 0.4.* version so that it doesn't break any setups using 0.3.*. Thanks for spotting this!

This would be incredibly helpful to me. Has there been any progress on it?

Agreed.
Needs a '_', it does compile correctly by using @import but duplicates output.

Any news on this? Issue seems to be closed, but unfortunately 0.4.0 does not prepend filenames with _ (_icons.svg.scss).

Any news on this?

I'd like to see some movement as well. At least give the option to prepend with an _ even if it's not the default.

Fixed/added to v0.6.0. Scss is written to "underscore files" and css to regular icons.css files. Needed to replace the gulp-sass plugin with a customised one, as "underscore files" are treated as partial files and are not processed by gulp-sass.


Enhancement proposal: progress bar in silx view whel loading volumes or stacks

Hi Silx,
I've begun using Silx in BL09 (Soft X-Ray Tomography Beamline) at ALBA. In the past we were using hdfview, but we are going to debian9, and hdfview has a bug not corrected in debain9 which does not allow to see the content of hdf5 files. First of all, thanks for this great application of Silx Viewer.
I have observed that when loading some volumes or stacks of images this is taking some time. Maybe a progress bar during the loading of big hdf5 files would be a good option. Just an idea as a possible enhancement proposal.
Cheers,

It's an interesting idea. @t20100 want to do it for the iso surface viewer. For other parts it can be complicate to implement as python is not well designed to use threads.

I will take a look. Maybe something can be done easily.


prevent proovread from skipping stubby reads

Hello,
Is there an option, when running proovread to keep all the long reads including the stubby ones (<400bp)?
$ proovread -l Contigs-.fasta --bam all_sort.bam -p proovread_test -t 40 --coverage 30 --keep-temporary --ignore-sr-length
Reads < 2* min_sr_lentgh are discarded
I've tried modifying the parameter in the file ($sr_length) without success.
Thanks
Sabrina

Hi Sabrina,
I've just uploaded a modified version, which should give you better control over the minimum long read length. You can clone it from this branch: https://github.com/BioInf-Wuerzburg/proovread/tree/feature/stubby-reads
To run proovread and keep long reads of any length, create a config file (my.cfg) with the following content:
'lr-min-length' => 0

Run proovread  with the config
proovread -c my.cfg -l Contigs-.fasta --bam all_sort.bam -p proovread_test -t 40 --coverage 30 --keep-temporary --ignore-sr-length

This should overwrite the default behaviour with the 2*min_sr_length. I haven't had time to test this thoroughly, so let me know if it works or if you run into unexpected behaviour.
Cheers
Thomas

Did you get the chance to give this a try? And did you run into any problems?


Minimap integration

Initial integration of minimap in our project. Changes were needed internal to the minimap library, so the project was pulled into our lib folder.
Pull requests into Flux require the following. Submitter and reviewer should :white_check_mark: when done. For items that are not-applicable, note it's not-applicable (N/A) and :white_check_mark:.
Submitter:
Running the Application

[x] Manually tested in Chrome
[x] Manually tested in IE

Documentation

[x] This pull request describes why these changes were made
[x] Recognizes any potential shortcomings/bugs in the description
Cheat sheet is updated
Demo script is updated
Documentation on Wiki has been updated
[x] Additional technical components and libraries are documented and added to wiki as needed

NoteParser

Note parser has been updated if structured phrases change

Code Quality

[x] 4-space indents - convert any tabs to spaces
[x] Use public class field syntax for binding functions - ex. handleChange = () => { ... };
[x] Code is commented

Tests

[x] Existing tests passed
Added UI tests for slim mode
Added UI tests for full mode
Added backend tests for fluxNotes code
Added note parser examples to test new functionality

Reviewer 1: Name

[ ] Code is maintainable and reusable, reuses existing code and infrastructure where appropriate, and accomplishes the task’s purpose
[ ] The tests appropriately test the new code, including edge cases
[ ] You have tried to break the code

Reviewer 2: Name

[ ] Code is maintainable and reusable, reuses existing code and infrastructure where appropriate, and accomplishes the task’s purpose
[ ] The tests appropriately test the new code, including edge cases
[ ] You have tried to break the code

I'm reviewing now


#759 add changelog

Description
Adds changelog for #759
Sorry, forgot to add this!

We have been referring to pull requests rather than issues in the changelog

I'm happy to change this, but note that this particular breaking change was documented in the issue, not the PR.

oh right, good point, leave as is then. Thanks!


Move rnd_chars to utils

✓ Checklist:

[x] This PR has been added to CHANGELOG.md (at the top of the list);
[x] Tests for the changes have been added (for bug fixes / features).

Codecov Report

Merging #79 (7b3885b) into main (d65c0b3) will not change coverage.
The diff coverage is 89.28%.

:exclamation: Current head 7b3885b differs from pull request most recent head df75886. Consider uploading reports for the commit df75886 to get more accurate results

@@           Coverage Diff           @@
##             main      #79   +/-   ##
=======================================
  Coverage   79.32%   79.32%
=======================================
  Files          47       47
  Lines        2786     2786
=======================================
  Hits         2210     2210
  Misses        576      576

Impacted Files
Coverage Δ

datanymizer_engine/src/lib.rs
100.00% <ø> (ø)

datanymizer_engine/src/transformers/mod.rs
100.00% <ø> (ø)

...atanymizer_engine/src/transformers/token/base64.rs
93.33% <50.00%> (ø)

...nymizer_engine/src/transformers/token/base64url.rs
93.33% <50.00%> (ø)

datanymizer_engine/src/transformers/token/hex.rs
96.77% <50.00%> (ø)

datanymizer_engine/src/utils.rs
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

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


Broken link

(https://go.microsoft.com/fwlink/p/?LinkId=525629)
The VS Web Essentials link is a dead link

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

ID: eea369bc-1eca-8279-fd5d-8615d4b8c4f0
Version Independent ID: 9891ab3e-82dc-4823-34b3-9c893471d24f
Content: Minification and bundling in SharePoint Online - Microsoft 365 Enterprise
Content Source: microsoft-365/enterprise/minification-and-bundling-in-sharepoint-online.md
Service: o365-administration
GitHub Login: @kelleyvice-msft
Microsoft Alias: kvice

@yogkumgit still pr #8104 is not merged.

@RAJU2529 Thank you very much for the contribution and updating the documentation with PR. @GitWesDone Hope this update is helpful for you. Thanks for taking out some time to open the issue. Appreciate and encourage you to do the same in future also.


 Actually I didn't wrote this on purpose. As I said in the readme file, this is forked from another project. By forked I mean stripped.
  I may update this lib or not. That's depends on am I in a good mood or not. I do it because I want to, not to stay in the game.
 Such a tool.


Small ui fixes

statisic view with data
before:

after:

webservice table icon column
before:

after:

was even visible on the screenshot compare 🙈


docs page generating high CPU usage

the docs page generating high CPU usage and even  freeze the site for few seconds
https://www.prisma.io/docs/
on the console, the following error message appeared
https://www.prisma.io/docs/page-data/reference/tools-and-interfaces/prisma-client/api/page-data.json net::ERR_ABORTED 404
Regards
Norbert

Exact same here.
Site freezes and then crashes few seconds after.

Sorry about this, we are working on it.

This should now be fixed. Seems a configuration change on Algolia, our search provider, lead to a pretty confusing regression. We will investigate next week how that could happen and how we can avoid that in the future.


characters lost from value when doing table storage roundtrip

When round-tripping certain values to table storage and back the value is mangled. In some cases new line characters  appear to be stripped on the way back. When querying the row from .NET I can see the original line breaks. So it appears there is no corruption, but rather just mangling somewhere in the node sdk. It doesn't happen for all values.
This popped up when storing a JavaScript function body in an azure table. Somehow this value (and any other similar snippet) is affected.
IN:
for ( var i in [1,2,3] ) {
 console.log( i );
}

OUT:
for ( var i in [1,2,3] ) { console.log( i );
}

This is indeed a dup. @einaros has just did a fix for issue #27 which once merged should fix this one as well.


Move Money Dialog CSS needs updating

From forum user leinhto:
Not sure it was reported but the latest Toolkit version 0.3.6 has a bug. The search category list when moving money is floating now so it hovers over the list blocking the items. Before it stickied to the top of the drop down. I am using Chrome.

I just started getting into nYNAB and this extension. Really cool! Let me know if the fix could use more testing.

@ctlevi, welcome!
If you want to contribute more I'll add you to the team on Trello so you can claim cards, update status, etc.

Definitely @christophertate on trello (I believe? A bit of a trello noob)

Added you to both the Roadmap and the Team Discussions board. To move stuff drag it. To add new stuff type and press enter.
We have someone from the Forum community managing feature requests, so I try to channel everything through her so she knows about all the features going in. Her name on the forums and Trello is @bluebird8203. Welcome!


Vulnerability on react-scripts 4.0.3

I am using react-scripts 4.0.3 on my project and the dependabot of GitHub found the following vuln:
Dependabot cannot update is-svg to a non-vulnerable version
The latest possible version that can be installed is 3.0.0 because of the following conflicting dependency:

<EMAIL_ADDRESS>requires<EMAIL_ADDRESS>via a transitive dependency on<EMAIL_ADDRESS>The earliest fixed version is 4.2.2.

View logs or learn more about troubleshooting Dependabot errors.

Are you going to fix that?

Duplicate of https://github.com/facebook/create-react-app/issues/10762

Hey, any idea when this will be addressed?

There is no vulnerability here, so it will not be addressed.
https://github.com/facebook/create-react-app/issues/11174


Fix download links for macos and linux builds

Update wrong macos and linux download links.
Fix #4

Thanks! This fix was rolled into https://github.com/apple/pkl/pull/21


Embedded loops

malloc() in embedded loops do not get counted for the correct number of times. Need to determine if a loop is embedded. Then get the number of iterations of the parent loop and multiply it by the current loops iteration count.

Wasn't too complicated of an issue to fix. Just wrote one utility function that runs through the array of loop info's and determines which loop are embedded based on their start and end indices and correctly determines the number of iterations based on its parent loop.


Fix failing tests

If you run gulp build, gulp regenerate, then npm test some of the tests will fail.
These should be fixed or at least disabled.

Is this in autorest.nodejs or autorest.typescript ?

This change is happening in autorest.nodejs. That's what we talked about, right?

This test failure is in autorest.nodejs.


opencagegeo should use callback on err

Fix for: https://github.com/nchaulet/node-geocoder/issues/145

:+1:


Action failure: ValueError: Length mismatch: Expected axis has 10 elements, new values have 9 elements

https://github.com/pombase/allele_qc/actions/runs/11357468279
  File "/opt/hostedtoolcache/Python/3.9.20/x64/lib/python3.9/site-packages/pandas/core/internals/managers.py", line 228, in set_axis
    self._validate_set_axis(axis, new_labels)
  File "/opt/hostedtoolcache/Python/3.9.20/x64/lib/python3.9/site-packages/pandas/core/internals/base.py", line 70, in _validate_set_axis
    raise ValueError(
ValueError: Length mismatch: Expected axis has 10 elements, new values have 9 elements
Error: Process completed with exit code 1.

The processing failed because the modifications export file now has an extra column "assigned_by":

pombase/pombase-chado#1232

I've fixed the Python code.


Update create-a-project.md

Sorry for recreating this, had problems on my forked repo 😢

Closing in favour of #329


ATN providers no longer throw an exception if cannot extract header

Signed-off-by: Tomas Langer<EMAIL_ADDRESS>

Resolves #666

Rebased, added a small perf. optimization


Updates based on PSRule v2.3.0-B0130 fixes

Description

Updates as discussed based on the latest fixes and interface improvements.

This checklist is used to make sure that common guidelines for a pull request are followed.

[x] I have read the contribution guidelines.
[x] The pull request does not introduce breaking changes.

General Guidelines

[x] Title of the pull request is clear and informative.
[x] Description of the pull request is clear and informative.
[ ] I have added myself to the 'assignees'.
[ ] I have added 'linked issues' if relevant.

Testing Guidelines

[ ] Pull request includes test coverage for the included changes.

@VeraBE Updates for your branch.


Feature Request: shortcut options

As a developer I want to have the ability to treat some options as aliases of others, similar to command line arguments where -q could be equivalent to --quiet
Specifically in scenic the goal is to allow developers to specify the translate (as one example) as either :t or :translate:
https://github.com/boydm/scenic/blob/b2e57b50737443d8fb7a6d7ef2e88ef704cf42c2/lib/scenic/primitive/transform/translate.ex#L21-L29
This is currently implemented using :rename_to which is now deprecated in 0.4:
https://github.com/boydm/scenic/blob/b2e57b50737443d8fb7a6d7ef2e88ef704cf42c2/lib/scenic/primitive/transform/transform.ex#L61-L68
It would be nice to be able to do this without having to post-process the output of nimble_options (one specific downside to this is having to customize the output of NimbleOptions.docs/2).
This is effectively a request to include :other_names mentioned in https://github.com/dashbitco/nimble_options/issues/68#issuecomment-953619287 although I think :aliases would be a better name for it.

@axelson I think aliases (or shortcuts) are much more common in CLI tools arguments rather than options like the ones NimbleOptions is targeting. As you said, doing a pre-pass on the options to rename them before feeding them into nimble_options should be straightforward and, as you mentioned, you can customize the doc: option to mention aliases.
You could even do all of this in a programmatic way by preprocessing the schema too:
schema_with_aliases =
  Enum.map(schema, fn {key, schema} ->
    case Keyword.fetch(schema, :aliases) do
      {:ok, aliases} ->
        doc = "Aliased to: #{inspect(aliases)}"
        Keyword.update(schema, :doc, " #{doc}", &(&1 <> " #{doc}")
  end)

or something along those lines. You get the idea. Thoughts?

Yeah, I think a pre-pass is the way to go too. I understand why it exists on Scenic and agree with it, but we likely don’t want to encourage APIs to have aliases. It is rather the exception than the rule.


DWOC doesn't respect SSH key passphrase when push changes from workspace

Description
Related changes:

https://github.com/devfile/devworkspace-operator/pull/1291,
https://github.com/eclipse-che/che-dashboard/pull/1157

How To Reproduce
Prerequisites:
a) create SSH key with passphrase, using the command ssh-keygen -b 2048 -t rsa -N "test"
b) add it to the GitHub.com
Steps to reproduce:

Install Eclipse Che using User Dashboard https://quay.io/repository/eclipse/che-dashboard:pr-1157 with support of SSH key with passphrase
Open User Dashboard and add SSH key to the User Preferences.
Go to GitHub.com and get SSH link to any private repo using.
Go to "Eclipse Che -> Create workspace" page and create workspace using SSH link to the private repo.
Wait on workspace to be created, started, and project to be cloned.
Make changes in the project, commit them and push.
Go to workspace Terminal and try to push changes from there.

Let Dev Spaces keep ssh passphrase for git.webm
Expected behavior
Changes are pushed without errors or asking for passphrase.
Actual behavior
screencast-bpconcjcammlapcogcnnelfmaeghhagj-2024.08.06-14_39_32.webm
At the step 6
there was an error Git: Warning: Permanently added the ECDSA host key for IP address '<IP_ADDRESS>' to the list of known hosts. when tried to push changes to the repo using Source Control view of VS Code Editor.
git-error log:
> git pull --tags origin main
Warning: Permanently added the ECDSA host key for IP address '<IP_ADDRESS>' to the list of known hosts.
<EMAIL_ADDRESS>Permission denied (publickey).
fatal: Could not read from remote repository.

Please make sure you have the correct access rights
and the repository exists.

At the step 7
there was a request to enter passphrase for key '/etc/ssh/dwo_ssh_key":

@AObuchow I believe that the problem is in the DWO side indeed. From the recording we see that the git push command called from terminal invokes the default ssh-askpass tool. Looks like we need to propagate the ssh-askpass.sh script to the workspace container as well.

I believe the issue with git pushing from terminal prompting the user for the passphrase is because we do not have the required ssh-add environment variables injected into the workspace containers (only the project clone container has them).
We could either add the DISPLAY &  SSH_ASKPASS environment variables to all devworkspace containers, or try to add it only to the tooling container, similar to how we infer the tooling container component when determining the image for the persistent-home-init-container.
@vinokurig @dkwon17 any thoughts?

@AObuchow I believe that the problem is in the DWO side indeed. From the recording we see that the git push command called from terminal invokes the default ssh-askpass tool. Looks like we need to propagate the ssh-askpass.sh script to the workspace container as well.

My thoughts exactly :) I was just about to finish writing my comment when you posted your comment.

@vinokurig, here are some thoughts on where in the codebase we could make the required changes:

We could inject the required DISPLAY & SSH_ASKPASS environment variables to the workspace containers in /pkg/library/env/workspaceenv.go. A new function (something like addSSHEnvironmentVariables()) could be made to inject these environment variables, or we could potentially use the commonEnvironmentVariables() function.
The code related to injecting ssh-askpass.sh into the workspace containers could probably be added in its own file in /pkg/provision/, e.g. /pkg/provision/ssh/. Alternatively, it could go into /pkg/provision/workspace/.

You will probably have to call your function that injects ssh-askpass.sh in the main reconcile function (unless there's a better place for this). We did something similar for injecting service account tokens into the workspace.

I believe you'll have to create a volume and volumeMount for mounting the ssh-askpass.sh script, and then create a configmap on the cluster which mounts into the volume and contains the ssh-askpass.sh contents as its data.
Here's an example of creating a volume and volumeMount for a configmap. Your implementation will be much simpler however: we need to create a simple volume that mounts at a specific path /usr/local/bin/ssh-askpass.sh and add it to the workspace's pod additions (like we do for the serviceAccountToken volumes)
Here's an example of creating a configmap that has some data (again, this example is more complex than the current case but should give some ideas).

Lastly, something that slipped my mind during the review of your original PR for supporting ssh passphrases: instead of adding the ssh-add environment variables directly into the project clone image, we could add them into the container's environment variables. An argument against this approach however, is that users who provide their own project clone container image might not want these environment variables set for them.
I don't think we need to do anything about this, but I wanted to mention this here incase the topic ever comes up.

moving to the next release for 3.17 - https://issues.redhat.com/browse/CRW-6614?focusedId=25289708&page=com.atlassian.jira.plugin.system.issuetabpanels%3Acomment-tabpanel#commen[…]289708


Working without declaring DbSet in DbContext

Good morning,
is it possible to use the library without declaring DbSet properties on DbContext?
Somethink like following:
public class DemoContext : DbContext {

  public DemoContext(DbContextOptions options) : base(options)
  {
  }

  protected override void OnModelCreating(ModelBuilder builder) {
    builder.ApplyConfigurationsFromAssembly(typeof(DemoContext).Assembly);
  }
}

At the moment I'm getting exception:
EntityGraphQL.Compiler.EntityGraphQLCompilerException: 'Field 'xxx' not found on type 'Query''
Thank you!

Hi Stefano,
No you can not. EntityGraphQL works by building dotnet expressions. E.g.
{
SomeEntities { id }
}
Will compile to (ctx) => ctx.SomeEntities.Select(e => id = e.Id)
Where ctx will be the instance of your DbContext. If SomeEntities is not a
property or field, EntityGraphQL cannot build the expression.
I.e. EntityGraphQL does not do anything with SQL - it only builds
expressions and lets EF handle the rest.
Cheers
Luke
On Thu, Aug 3, 2023 at 5:07 PM Stefano Vuerich @.***>
wrote:

Good morning,
is it possible to use the library without declaring DbSet properties on
DbContext?
Somethink like following:
public class DemoContext : DbContext {
public DemoContext(DbContextOptions options) : base(options)
{
}
protected override void OnModelCreating(ModelBuilder builder) {
builder.ApplyConfigurationsFromAssembly(typeof(DemoContext).Assembly);
}
}
At the moment I'm getting exception:
EntityGraphQL.Compiler.EntityGraphQLCompilerException: 'Field 'xxx' not
found on type 'Query''
Thank you!
—
Reply to this email directly, view it on GitHub
https://github.com/EntityGraphQL/EntityGraphQL/issues/312, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AABGHUJ4GCXWW52XJHG4C7DXTNE2HANCNFSM6AAAAAA3CJ4HYE
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

Hi Luke,
I manage my problem using these classes:
public class GraphQLSchemaBuilder<T> where T : class
{
    public static void ConfigureSchema(SchemaProvider<GraphQLDbContext> schema)
    {
        var instance = Activator.CreateInstance<T>();
        var name = instance.GetType().Name;

        if (!schema.HasType(instance.GetType().Name))
        {
            schema.AddType<T>(name).AddAllFields();
        }

        schema.UpdateQuery(queryType =>
        {
            queryType.AddField(name, db => db.Set<T>(), $"List of {name}");
        });

        System.IO.File.WriteAllText("schema.graphql", schema.ToGraphQLSchemaString());
    }
}

Then I can call the ConfigureSchema method using refletion:
builder.ConfigureServices((context, services) =>
{
    services.AddDbContext<GraphQLDbContext>(/*config the db context*/);

    services.AddGraphQLSchema<GraphQLDbContext>(options =>
    {
        options.ConfigureSchema = schemaBuilder =>
        {
            var myModels = Assembly.GetEntryAssembly().GetTypes().Where(/* logic to get only models i want to manage through graphql */)

            myModels .ForEach(m=>
            {
                Type generic = typeof(GraphQLSchemaBuilder<>);
                var model = Activator.CreateInstance(m);
                Type[] typeArgs = { model.GetType() };
                Type constructed = generic.MakeGenericType(typeArgs);
                var method = constructed.GetMethod("ConfigureSchema");
                method.Invoke(Activator.CreateInstance(constructed), new object[] { schemaBuilder });
            });
        };
    });
});

@lukemurray What you think about this? I tried and it is working, do you see any downside?
Thanks

OK. I didn't realise you just wanted to use ctx.Set<MyEntity>() as the root of the expression.
Yes that will work and should be no issues. It is the same as manually adding them all but using reflection to auto add the ones you want if you don't want them defined at the DbContext or manually write out each one.


Adding custom dataloaders, based on recipes

💡 Your Question
Hi, I have a question about integrating user-made recipes into dataloaders. Basically, I want to make something like:
@register_dataloader("my_train_dataloader")
def my_train_dataloader(dataset_params: Dict = None, dataloader_params: Dict = None) -> DataLoader:
    return get_data_loader(
        config_name="my_dataset_dataset_params",
        dataset_cls=YoloDarknetFormatDetectionDataset,
        train=True,
        dataset_params=dataset_params,
        dataloader_params=dataloader_params,
    )

I have a recipes folder with dataset_params subfolder, the same format as used for pre-defined recipes. How can I configure my dataloaders to access my recipes, not pre-defined ones? I want for them to be usable inside notebooks, so @hydra.main is not accessible for launching.
Thanks!
Versions
No response

@ShpihanVlad if you are using coded training, you can simply instantiate the dataloaders as with any regular Pytorch training.
Then pass them through train_dataloader and val_dataloader arguments in Trainer.train(). No need to define configuration files for them.


A proper fix for v1.0.2 on real hardware & PS3 | xdelta patch addendum

Here's a temporary fix for the freezes on real hardware (also mentioned in https://github.com/wagrenier/Zero2Undub/issues/12), as the currently available "Safe Undub" option for the real hardware compatibility results in a partial undub.
Changes:

Corrected the total ISO size in the ISO ToC
Corrected the IMG_BD.BIN size in the ISO ToC
Placed the dummy files (shrunken down to 2 kb) after the IMG_BD.BIN like in the original image so that IMG_BD.BIN doesn't overlap with them; ofc also reflected the dummy files' new offsets and sizes in the ISO ToC
Appended back the original image's end padding and the EoF sector to the end of the image

This turns the ISO "legit," and just that seems to resolve the freezes that were occurring on PS2 & PS3.
Fatal Frame II Undub v1.0.2 fix.zip
Apply to the v1.0.2 undubbed ISO (CRC32: 67F461BD /
MD5: 8B33F35838CE97E73D311E14B1A3385B)

I believe as of now my patch already does steps 3) and 4). My undub most likely crashes because I do not correctly adjust the file sizes for the ISO. @swosho
I'll look at the offsets of an ISO to achieve that and push that into a branch. Would you be able to test the patch?

I believe as of now my patch already does steps 3) and 4)

If by that you mean v1.0.2, then that's not quite the case. IMG_BD.BIN's appended data at 0xC23CB000 just overwrites everything starting from there, which is: the dummy files, the end padding / EoF postgap, and the EoF sector. I guess the stuff that the patcher currently overwrites was meant to be shifted forward, right?

Would you be able to test the patch?

The undub with the fix applied is confirmed to run issue-free on an actual PS2, I think there's no need for further testing if you can simply compare the upcoming patch version against this temporary addendum.
Here's the same fix as in the OP but with the dummy files' sizes kept intact so it would be identical to an ISO with UDF ToC stuff sorted and all the end data shifted forward instead being of overwritten: Fatal Frame II Undub v1.0.2 fix-norip.zip

If it helps, I'm testing this on real hardware right now, it's already looking good as the first freeze I would have on real hardware (the first cutscene after entering the first building) no longer happens, I'm gonna play a bit more but I'm likely to use the fixed patch to pick up my savefile in Chapter 5!
Great work fixing this

Thanks to both for the information with that I'll work on the fix and it should be out today or tomorrow for FF2 and maybe next week for FF1

I am done writing the code needed for this fix, I'll be testing it tomorrow.

I need to make further adjustments so next week I'll have the new version released. @swosho that tool did you use to change the file sizes? My current patches does not read files so the process of manually patch the iso is very tedious. I think my best solution would be rewriting the patcher

I did those edits plainly in a hex editor, since there were so few values to correct after all ¯_(ツ)_/¯
You'd normally parse and fix the ISO9660 header, and then do the same for the UDF descriptors following it... but in this case, I think it's not too much of a "hack" to just hardcode the offsets for the needed edits and write predetermined values there.

Yeah I'll try to find an ISO9660 parser for C++ or C#, would most likely mean a rewrite tho, which will take a bit more time

@swosho @wagrenier you guys ROCKS!


Fix generation of "schema" section for a "body" parameter to contain correct "type" & "format"

I ran into a problem where the Swagger2 JSON document generated from a SpringBoot application with the following API method:
  @RequestMapping(...)
  @ResponseStatus(...)
  public void setThreshold(..., @RequestBody int threshold) { ... }

would include the following for the @RequestBody parameter
{
  "in": "body",
  "name": "threshold",
  "description": "threshold",
  "required": true,
  "schema": {
    "type": "int"
  }
}

This JSON has "int" for the "type" property, when it should be "integer", with "format": "int32".
This pull request updates ParameterMapper to ensure that it uses io.swagger.models.properties.Property for the "schema" "type" and "format" information, similar to what is done for collections and maps in that class.

Thank you for catching that! and for the pr! :beers:


Certbot crashes with error "exec /bin/start_cert.sh: exec format error"

joplin-vieweb-joplin-vieweb-certbot-1  | exec /bin/start_cert.sh: exec format error
Running in DigitalOcean, using Docker CE.

Hello.
I think it's because my certbot image is only built for those arch: linux/amd64,linux/arm64
Could you say which arch are you using on DigitalOcean ?

Interesting... "dpkg --print-architecture" returns "amd64".

OK... so it's... something else ;-)
However, even if it doesn't solve directly your question, but on this page: https://www.digitalocean.com/pricing/app-platform I read: "checkmarkAutomatic HTTPS".
So you may not need certbot ? neither joplin-vieweb-nginx ?

No news... I close this issue. Feel free to open it again if needed.


Install Error

Hi, I have deleted and installed papers multiple times and I have this error when running the installation cell
Installing Stable Diffusion Web UI Forge...
error: pathspec '29be1da' did not match any file(s) known to git

Fixed in forge notebook version 2024.07.31.
Sorry for the inconvenience. This is a mistake I made in the previous update.


Any plans to decouple exalibrain from Obsidian?

Are there any thoughts or plans to decouple Exalibrain from Obsidian? so that it can work with other markdown editors such VSCode or Emacs? Thank you in advance!

This is absolutely my "plan". ExcaliBrain was built as a proof-of-concept solution, it just works too well (is not uncomfortable enough) to be of sufficient motivation to get on with building a standalone package that would perform the same feature but without the dependency on Excalidraw and Obsidian. I don't have a timeline though

It would be the first open source local-first generic white board. I imagine it would be wildly popular, especially if one could use their favourite editor.


Update RazorPagesMovieContext.cs

@guardrex  anything else I need to delete to fix RP intro? You can clean up after I merge this.

I'm doing it now. Best to wait .... could create a merge :boom: if we both make changes.


doxity develop port issues

It keeps saying "Something is already running at port 8000" Which is not true.
Then it prompts, "Would you like to run the app at another port instead? [Y/n]"
Entering "Y" does nothing.

Hm, thanks for this. A workaround is to use npm run start inside the ./scripts/doxity dir.

@hitchcott
In ./scripts/doxity running npm start and npm run start give me a missing script: start error.
What are the commands that are meant to be run in this script, so I can do them manually?


Need to install libraries and headers

I need to compile another project that heavily relies on the header files and such that from this project. When building, I run into all kinds of dependency errors such as this one:
src/protocol.c:6:10: fatal error: opendnp3/app/APDURequest.h: No such file or directory
 #include <opendnp3/app/APDURequest.h>

Any chance I might get an install-libs target like e2fsprogs has for this purpose?

There's an install target here:
https://github.com/dnp3/opendnp3/blob/release/cpp/lib/CMakeLists.txt#L689
make install should work

Oh sweet, thanks!
I did run into a problem building, though. Apparently libasio-dev isn't happy.
I'm on Ubuntu 18.04.
/usr/local/bin/cmake -S/home/wnilges/Documents/opendnp3/cpp/lib -B/home/wnilges/Documents/opendnp3/cpp/lib/build --che
ck-build-system CMakeFiles/Makefile.cmake 0
/usr/local/bin/cmake -E cmake_progress_start /home/wnilges/Documents/opendnp3/cpp/lib/build/CMakeFiles /home/wnilges/D
ocuments/opendnp3/cpp/lib/build//CMakeFiles/progress.marks
make -s -f CMakeFiles/Makefile2 all
[  0%] Building CXX object CMakeFiles/opendnp3.dir/src/DNP3Manager.o
In file included from /home/wnilges/Documents/opendnp3/cpp/lib/src/DNP3Manager.cpp:22:0:
/home/wnilges/Documents/opendnp3/cpp/lib/src/DNP3ManagerImpl.h:110:33: error: ‘io_context’ is not a member of ‘asio’
     const std::shared_ptr<asio::io_context> io;
                                 ^~~~~~~~~~
/home/wnilges/Documents/opendnp3/cpp/lib/src/DNP3ManagerImpl.h:110:33: note: suggested alternative: ‘connect’
     const std::shared_ptr<asio::io_context> io;
                                 ^~~~~~~~~~
                                 connect
/home/wnilges/Documents/opendnp3/cpp/lib/src/DNP3ManagerImpl.h:110:33: error: ‘io_context’ is not a member of ‘asio’
/home/wnilges/Documents/opendnp3/cpp/lib/src/DNP3ManagerImpl.h:110:33: note: suggested alternative: ‘connect’
     const std::shared_ptr<asio::io_context> io;
                                 ^~~~~~~~~~
                                 connect
/home/wnilges/Documents/opendnp3/cpp/lib/src/DNP3ManagerImpl.h:110:43: error: template argument 1 is invalid
     const std::shared_ptr<asio::io_context> io;
                                           ^
CMakeFiles/opendnp3.dir/build.make:95: recipe for target 'CMakeFiles/opendnp3.dir/src/DNP3Manager.o' failed
make[2]: *** [CMakeFiles/opendnp3.dir/src/DNP3Manager.o] Error 1
CMakeFiles/Makefile2:95: recipe for target 'CMakeFiles/opendnp3.dir/all' failed
make[1]: *** [CMakeFiles/opendnp3.dir/all] Error 2
Makefile:149: recipe for target 'all' failed
make: *** [all] Error 2

This might be relevant: https://stackoverflow.com/questions/47895520/io-context-error-in-boost-library

Use the provided version via cmake fetch:
https://github.com/dnp3/opendnp3/blob/release/deps/asio.cmake#L3
Older version from the package manager will not work.

I can't seem to find another header file.
In file included from /home/wnilges/Documents/opendnp3/cpp/lib/src/channel/IOHandler.h:23:0,
                 from /home/wnilges/Documents/opendnp3/cpp/lib/src/channel/DNP3Channel.h:24,
                 from /home/wnilges/Documents/opendnp3/cpp/lib/src/DNP3ManagerImpl.cpp:31:
/home/wnilges/Documents/opendnp3/cpp/lib/src/channel/IAsyncChannel.h:27:10: fatal error: ser4cpp/container/SequenceTyp
es.h: No such file or directory
 #include <ser4cpp/container/SequenceTypes.h>
          ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
compilation terminated.

I checked the linked version of ser4cpp and the newest one, but I can't seem to figure out where this header file is.

https://github.com/automatak/ser4cpp/blob/master/src/ser4cpp/container/SequenceTypes.h
I'm guessing that there is perhaps some problem during cmake initialization with fetching dependencies.
When you first run cmake, it should go and download a number of dependencies. For example, the referenced commit of ser4cpp should downloaded into <build>/_deps/ser4cpp-src/.
After running cmake you should be able to  ls .\_deps\ser4cpp-src\ and see the ser4cpp src and test dirs.

No dice, that directory doesn't appear to exist.

What version of CMake are you running? OpenDNP3 requires version 3.11 or higher. Ubuntu 18.04 is shipped with CMake 3.10 by default.

I installed 3.18. I actually managed to get it to compile. I just installed ser4cpp manually. Now I'm having different dependency issues :call_me_hand:
And this time, I legitimately have no idea whose fault that is.
src/protocol.c:4:10: fatal error: openpal/util/ToHex.h: No such file or directory
 #include <openpal/util/ToHex.h>
          ^~~~~~~~~~~~~~~~~~~~~~
compilation terminated.
Makefile:20: recipe for target 'build/protocol.o' failed
make: *** [build/protocol.o] Error 1

That being said, I think my dependency issues for opendnp3 are solved. If someone has any advice for openpal, I'd appreciate it, but I guess that's not exactly on topic :sweat_smile: Thanks, y'all!

openpal refers to code using the 2.x release branch. The OpenDNP3 version you are using is a 3.x version, you shouldn't use this include anymore. In 3.x, every header is in opendnp3, no more openpal, asiopal, asiodnp3 and friends.
The dependency issue is weird though... CMake should download everything needed for you right in the build directory without requiring any action from your end.

Yes, you will need opendnp3 to be installed. All the dependencies (like ser4cpp and asio) are used internally to build the library, but they are not needed by your client code once opendnp3 is compiled and installed.
Another option is to use the release-2.x branch of OpenDNP3 if you don't want to upgrade the code handed to you.

For posterity's sake, I think I'm going to have to emulate the environment with which this app was built, then, when I get that working, I'll have to do some major refactoring. Will keep you posted.

Trying to build release-2.x branch, I'm hitting this error about asio.

The 2.x release uses the standalone asio 1.12.2 (not the one included in Boost). You should probably rely on the git submodule instead of the system dependency. Documentation of the 2.x is still available here: https://dnp3.github.io/docs/guide/2.3.0/build/cmake/#locating-asio

Sweet, got my project to compile. Hopefully that's the end of that. Thanks for all the help, guys!


:robot: triggering CI on branch 'release-next' after synching from upstream/main

Codecov Report

Merging #632 (5dfc2db) into release-next (5dfc2db) will not change coverage.
The diff coverage is n/a.

:exclamation: Current head 5dfc2db differs from pull request most recent head 0555c45. Consider uploading reports for the commit 0555c45 to get more accurate results

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
@@              Coverage Diff              @@
##           release-next     #632   +/-   ##
=============================================
  Coverage         49.05%   49.05%
=============================================
  Files                96       96
  Lines             13100    13100
=============================================
  Hits               6426     6426
  Misses             6039     6039
  Partials            635      635

OCF Webhook is merging this PR


/Savetocloud 404 5532-59ms

Hm...
After a certain period of time playing the game on the browser, namely 4 minutes 30 seconds, the "cloudsave plugin" no longer works.

On the server is it giving a 404 or 403 error code?


Add source-git-branch exception-to pegasus frontend

Likely what's the maintainer @mmatyas wish to (https://github.com/flathub/org.pegasus_frontend.Pegasus/issues/44).

No. commit+branch mean the build WILL fail as soon as the branch HEAD changes. There is no going around.
Flathub is NOT for nightly builds.

There is commit and a branch, currently:
 - type: git
   url: https://github.com/mmatyas/pegasus-frontend.git
   branch: master
   commit: cc2497abf7ebb12e0cbdb57a16addaf62d6f50bd

The goal is to update the commit to the latest one in master, which is the latest release.

I guess an other release cycle will have to be found.

Yes please, one that doesn't involve updating daily to a new commit or tag or whatever.
If you want to do that, either update it with a custom workflow once a week like I mentioned in the original issue above or you are free to host it externally.
Doing nightlies is not sustainable for the infra.

Once a week sounds good, let's do like RPCS3. To have this https://github.com/flathub/org.pegasus_frontend.Pegasus/pull/49 work, an exception will still be needed though?
Or, we'll have to make a custom script for that purpose :thinking:

If I understand correctly the issue, this is more about the frequency of builds than the nature of the release?
For example, a stable release per day is more problematic than a nightly build per month.

Frequency is a problem for the infra as well as having software that is not "stable". End users expect software from Flathub to be stable, Nightly belongs in upstream CI.

@mmatyas do you confirm that the latest commit in master is considered a stable release and not a nightly one? Otherwise, I guess an other release cycle will have to be found.

The latest stable release is the last commit with an annotated Git tag (see https://github.com/mmatyas/pegasus-frontend/releases). I assume this would require a daily-running script, but if it's too much trouble, I'm fine with a weekly workflow too.
I also had good experience with externally hosting an auto-updated Flatpak release; we can set one up for nightlies if necessary.

If you switch to those weekly_ tags, exception is not needed anymore.

What do we do then @mmatyas, have a weekly cron taking the latest commit with the continuous tag?

Yes, that sounds good to me.


🐛 BUG: "On this page" section of website is not scrollable

Quick checklist

[X] I am using the latest version of Snowpack and all plugins.

What package manager are you using?
yarn
What operating system are you using?
macOS
Describe the bug
** On this page ** section at https://www.snowpack.dev/reference/configuration is not scrollable and half of the content is not visible.
Browser used: Chrome
Steps to reproduce

Go to https://www.snowpack.dev/reference/configuration#devoptions
Scroll to the bottom of the page

Link to minimal reproducible example (optional)
No response

I suggest to make it scrollable like on the left side of the website, by adding
     height: calc(100vh - #{$nav-height + $algolia-height});    overflow-y: auto;
to .subnav
on Subnav.astro

Same problem, please fix

КАЗАКТАР АЛГААА!


[API] Expose 'connected' and 'authenticated' status

Is this a request for a new feature or a bug report?
Enhancement
General Description:

Expand the ._connected variable to either a combined state object or just add an ._authenticated variable.
Consider dropped the underscore as these are intended to be publicly visible.

yes pls.
problem still exists.
i have to use protected _connected to check if it is already connected or not.
why is this prop protected? reason?
maybe a getter would be enough: get isConnected: () => this._connected

Hi, I would like to have this property too. Checking if the socket is connected or not is very useful.
By default a websocket expose those properties, why protect them ?
@haganbmj the issue is opened since 2017 and the feature wasn't planned for the V1 ?

Is this a thing yet? It's vital for my application.


Simply use setlocale to initialize the locale

There is a bit more logic involved despite inspecting the LC_* variables:
If locale is an empty string, "", each part of the locale that should be modified is set according to the  environment  variables.   The
       details are implementation-dependent.  For glibc, first (regardless of category), the environment variable LC_ALL is inspected, next the
       environment variable with the same name as the category (see the table above), and finally the environment  variable  LANG.   The  first
       existing  environment  variable  is  used.   If  its value is not a valid locale specification, the locale is unchanged, and setlocale()
       returns NULL.

For instance, I have only LANG set, which should also be taken into account but currently is not.

Thanks @avdv, this makes a lot of sense, and honestly should have been there from the beginning.


Issue#14 - flags improvements

Just replacing the flags lib by a new one, POSIX/GNU compliant.
https://github.com/dachad/check-max-tcp-connections/issues/14
Abbreviations added, and delay becomes "sleep" to prevent it conflicting with debug (I've not renamed debug to verbose, as in the future verbose may mean extended stats)

Please, review/integrate after https://github.com/dachad/check-max-tcp-connections/pull/12 , as its a branch for that one


Stripe Connect Token Request

Hello,
I was wondering if there is a call to get the token for Stripe Standalone tokens  https://stripe.com/docs/connect/standalone-accounts#token-request
I would love to contribute if it doesn't exist but checking to make sure if I am wrong.

@vkoppaka doesn't look like it's already implemented, please go ahead.
I would also try researching this repository
@jaymedavis has managed a thorough implementation of a here at https://github.com/jaymedavis/stripe.net


[Example] Python-Pandas

See https://drive.google.com/drive/u/1/folders/1WMba3yWHJ8a2ywLb-y4oEV6frF1DBEzS

@philwinder - I can work on this one.  Not sure I have permission to assign it to myself


Fix typos

I fixed 2 typos in the comments

Fixes #

 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.:x: xanderwessels sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.


Number of deleted entries on del()

In the documentation for del() it says "...Returns the number of deleted entries..."
Will this always be 0 or 1?

If the key argument is a string, yes.
If it is an array of string keys, then it will return the number of matching elements that were removed.

That's right. You'll get the count of deleted elements.
see: https://github.com/tcs-de/nodecache#delete-multiple-keys-mdel


Need spec on conformal coating

Migrated from https://github.com/jadonk/beaglebone-blue-jadonk/issues/38
Originally created by @jadonk on Mon, 07 Nov 2016 19:18:32 GMT

Migrated from https://github.com/jadonk/beaglebone-blue-jadonk/issues/38#issuecomment-262286824
Originally created by @jadonk on Tue, 22 Nov 2016 16:19:13 GMT

Graphic added... https://github.com/jadonk/beaglebone-blue/commit/432ebf36c79731519eb745985146eb871d6f98e4
Now we just need to add related information to the README.md in the MFG directory and we can close this issue.

Migrated from https://github.com/jadonk/beaglebone-blue-jadonk/issues/38#issuecomment-259023488
Originally created by @StrawsonDesign on Tue, 08 Nov 2016 02:01:09 GMT

MG chemicals 419D Acrylic Conformal Coating was my original suggestion to Embest. However they used Humiseal 1B73 which is an almost identical product and seems to have worked well on our prototypes.

Migrated from https://github.com/jadonk/beaglebone-blue-jadonk/issues/38#issuecomment-260761635
Originally created by @jadonk on Tue, 15 Nov 2016 20:42:52 GMT

@StrawsonDesign What areas on the A2 layout? Can you mark-up the board PDF?

I believe we should remove the conformal coat on rev C and focus on I/O protection.


Update react-dom/server

Add streaming API.
Also added the version string.

Could you link to some sort of docs that verify this is correct?

https://reactjs.org/docs/react-dom-server.html#rendertonodestream

I don't see any reference to the version string, can we confirm in exists or remove it from the PR?


Proposal: Add custom activeClassName matcher to Link

Sometimes links should appear active even when they are not the current route. An example for this is nested navigation, where the top-navigation should be highlighted when the user is on a sub-route.
Example:
Help            (/help)
Help -> General (/help/general)
Help -> Contact (/help/contact)

For the imagined application, the link to /help should be active, even if the user is on /help/general or /help/contact.
I propose adding a property to Link called activeMatcher which is a function. It gets called with the current path (or maybe location?). If it returns true, the value stored in activeClassName will get appended. If activeMatcher is set, it overwrites the default router.isActive(to, query) behavior.
Example usage:
// in render() {..} of a component

return (
  <Link
    to="/dashboard/help"
    className="iconlink"
    activeClassName="-active"

    // overwrite default activeClass matcher behavior
    activeMatcher={(path) => /^\/help/.test(path)}
  >
    Help
  </Link>
)

I currently have got this as its own component. I could send a PR to add the functionality to the Link component itself.
Do you think this should be part of react-router?

Just have a parent route that simply renders the child route handler and
you'll have it.
On Sat, Jun 20, 2015 at 12:43 PM Dominik Ferber<EMAIL_ADDRESS>wrote:

Sometimes links should appear active even when they are not the current
route. An example for this is nested navigation, where the top-navigation
should be highlighted when the user is on a sub-route.
Example:
Help            (/help)
Help -> General (/help/general)
Help -> Contact (/help/contact)
For the imagined application, the link to /help should be active, even if
the user is on /help/general or /help/contact.
I propose adding a property to Link called activeMatcher which is a
function. It gets called with the current path (or maybe location?). If it
returns true, the value stored in activeClassName will get appended. If
activeMatcher is set, it overwrites the default router.isActive(to, query)
behavior.
Example usage:
// in render() {..} of a component
const isHelpActive = (path) => /^/help/.test(path)return (
// overwrite default activeClass matcher behavior
activeMatcher={isHelpActive}

Help

)
I currently have got this as its own component. I could send a PR to add
it to the Link component itself.
Do you think this should be part of react-router?
—
Reply to this email directly or view it on GitHub
https://github.com/rackt/react-router/issues/1374.

I knew there was an easier way. Awesome, thank you :clap:
I initially had my routes like this, that's why it wouldn't work. I'm using v1.0.0-beta2.
<Route path="help" component={Help}>
  <Route path="general" component={General}/>
  <Route path="contact" component={Contact}/>
</Route>

Now I have them like this and it works like you said.
<Route component={Help}>
  <Route path="help/general" component={General}/>
  <Route path="help/contact" component={Contact}/>
</Route>

Leaving it here for anyone running into the same issue.
I started using the Router today and it has been great so far!


Create version tags according to npmjs.com

Hello,
could you please create version tags same as you have on npmjs.com?
Right now last tag here is 0.21.0 but latest version is 0.23.1
It is a little confusing.
Thank you

Create 0.23.1 release tag


Anthropic tool support

Langchain is complaining about the pace where we add a system message with the date time. Appartently the system message must be the first message. For OpenAI agents this is not the case, so might be anthropic specific. Anycase, I'm thinking of just appending "the current date is {current_date}" to the prompt

I think this is Anthropic specific #211

Ah, excellent!

In a future PR we can update the tool to use our ScheduledMessages


Trigger notice if keyfile is missing a project ID

Closes #171.

cc @tmatsuo

I think this fell off the map because CI wasn't passing. Is it something we can get patched up and rebased with the new repo structure?

There were some issues with getting the tests passing. I'll need to circle back and reacquaint myself with it again, then I can give a firmer answer.

@jdpedrie do you think you'll have a chance to look at this again this week?

I'll reacquaint myself with it!

@dwsupplee this is ready for review.
@tmatsuo @bshaffer I'd appreciate input on potential edge cases with various types of key files where this change might pose an issue.


MakeCrvCoincidenceClusterMCAssns_module.cc belongs in Offline

This module should be part of Offline.  Alternatively, the Assns could be integrated with CrvCoincidenceClusterMatchMC

This was already done in #92 and Mu2e/Offline#1047. Closing issue now


<EMAIL_ADDRESS>+ Next.js

I use turso in my Next.js app (14.1.4). Config based on https://docs.turso.tech/sdk/ts/guides/nextjs
On version 0.6.0 everything is OK.
After updating to version 0.6.1, I'm getting Error: Unexpected end of JSON input
There's no info about 0.6.1 version in the Changelog
Console
Uncaught SyntaxError: Unexpected end of JSON input
    at JSON.parse (<anonymous>)
    at parseJSONFromBytes (node:internal/deps/undici/undici:5584:19)
    at successSteps (node:internal/deps/undici/undici:5555:27)
    at fullyReadBody (node:internal/deps/undici/undici:1665:9)
    at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
    at async specConsumeBody (node:internal/deps/undici/undici:5564:7)
    at async getIsSchemaDatabase (/Users/userdave/Workspace/xyz/node_modules/.pnpm/@libsql+client@0.6.1/node_modules/@libsql/client/lib-cjs/migrations.js:48:19)
    at async HttpClient.getIsSchemaDatabase (/Users/userdave/Workspace/xyz/node_modules/.pnpm/@libsql+client@0.6.1/node_modules/@libsql/client/lib-cjs/http.js:79:38)
    at async HttpClient.execute (/Users/userdave/Workspace/xyz/node_modules/.pnpm/@libsql+client@0.6.1/node_modules/@libsql/client/lib-cjs/http.js:101:38)
    at async Turso (webpack-internal:///(rsc)/./src/app/turso/page.tsx:18:22)

Node/Next output
result:  Response {
  [Symbol(realm)]: null,
  [Symbol(state)]: {
    aborted: false,
    rangeRequested: false,
    timingAllowPassed: true,
    requestIncludesCredentials: true,
    type: 'default',
    status: 404,
    timingInfo: {
      startTime: 11683.973666,
      redirectStartTime: 0,
      redirectEndTime: 0,
      postRedirectStartTime: 11683.973666,
      finalServiceWorkerStartTime: 0,
      finalNetworkResponseStartTime: 0,
      finalNetworkRequestStartTime: 0,
      endTime: 0,
      encodedBodySize: 0,
      decodedBodySize: 0,
      finalConnectionTimingInfo: null
    },
    cacheState: '',
    statusText: 'Not Found',
    headersList: HeadersList {
      cookies: null,
      [Symbol(headers map)]: [Map],
      [Symbol(headers map sorted)]: null
    },
    urlList: [ URL {} ],
    body: { stream: undefined, length: undefined, source: undefined }
  },
  [Symbol(headers)]: HeadersList {
    cookies: null,
    [Symbol(headers map)]: Map(5) {
      'date' => [Object],
      'fly-request-id' => [Object],
      'server' => [Object],
      'via' => [Object],
      'content-length' => [Object]
    },
    [Symbol(headers map sorted)]: null
  }
}
SyntaxError: Unexpected end of JSON input
    at JSON.parse (<anonymous>)
    at parseJSONFromBytes (node:internal/deps/undici/undici:5584:19)
    at successSteps (node:internal/deps/undici/undici:5555:27)
    at fullyReadBody (node:internal/deps/undici/undici:1665:9)
    at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
    at async specConsumeBody (node:internal/deps/undici/undici:5564:7)
    at async getIsSchemaDatabase (/Users/userdave/Workspace/xyz/node_modules/.pnpm/@libsql+client@0.6.1/node_modules/@libsql/client/lib-cjs/migrations.js:48:19)
    at async HttpClient.getIsSchemaDatabase (/Users/userdave/Workspace/xyz/node_modules/.pnpm/@libsql+client@0.6.1/node_modules/@libsql/client/lib-cjs/http.js:79:38)
 ⨯ unhandledRejection: SyntaxError: Unexpected end of JSON input
    at JSON.parse (<anonymous>)
    at parseJSONFromBytes (node:internal/deps/undici/undici:5584:19)
    at successSteps (node:internal/deps/undici/undici:5555:27)
    at fullyReadBody (node:internal/deps/undici/undici:1665:9)
    at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
    at async specConsumeBody (node:internal/deps/undici/undici:5564:7)
    at async getIsSchemaDatabase (/Users/userdave/Workspace/xyz/node_modules/.pnpm/@libsql+client@0.6.1/node_modules/@libsql/client/lib-cjs/migrations.js:48:19)
    at async HttpClient.getIsSchemaDatabase (/Users/userdave/Workspace/xyz/node_modules/.pnpm/@libsql+client@0.6.1/node_modules/@libsql/client/lib-cjs/http.js:79:38)

Same behavior on<EMAIL_ADDRESS>(latest)

same for drizzle usage only https://github.com/drizzle-team/drizzle-orm/issues/2417

Why was there such a breaking change from 6.0.0 to 6.0.1?

Why was there such a breaking change from 6.0.0 to 6.0.1? @penberg

Hey @GustavoOS, sorry for that. This was a bug in the latest version and we release a fix for it. You can upgrade to 0.6.2 and this should not happen anymore.
Let me know if you have any other issues and I'll be happy to help! 😄

Fixed in 0.6.2. Thanks @giovannibenussi


.mxml syntax

add mxml to xml syntax: http://help.adobe.com/en_US/flex/using/WS2db454920e96a9e51e63e3d11c0bf5f65f-7ff1.html

This issue was moved to atom/language-xml#21


GitLab does not pass gitlab:check

Issue description
The NixOS GitLab service does not pass GitLab's gitlab:check application check rule. This rule must pass to receive support from GitLab.
Steps to reproduce
Run:
nix run nixpkgs.gitlab -c sudo -u gitlab  gitlab-rake gitlab:check SANITIZE=true

A number of failures will be reported. These include:

Numerous issues of the form:

3/2 ... wrong or missing hooks
  Try fixing it:
  sudo -u gitlab -H /nix/store/qx227lr5gya6f79v85fpbn2n98kqj2ah-gitlab-shell-8.4.1/bin/create-hooks /var/gitlab/state/repositories
  Check the hooks_path in config/gitlab.yml
  Check your gitlab-shell installation
  For more information see:
  doc/install/installation.md in section "GitLab Shell"
  Please fix the error above and rerun the checks.

Running the indicated command does not fix the issue.

This:

Uploads directory tmp has correct permissions? ... no
  Try fixing it:
  sudo chown -R gitlab /var/gitlab/state/uploads
  sudo find /var/gitlab/state/uploads -type f -exec chmod 0644 {} \;
  sudo find /var/gitlab/state/uploads -type d -not -path /var/gitlab/state/uploads -exec chmod 0700 {} \;
  For more information see:
  doc/install/installation.md in section "GitLab"
  Please fix the error above and rerun the checks.
Init script exists? ... no
  Try fixing it:
  Install the init script
  For more information see:
  doc/install/installation.md in section "Install Init Script"
  Please fix the error above and rerun the checks.
Init script up-to-date? ... can't check because of previous errors

Redis version >= 2.8.0? ... no
Ruby version >= 2.3.5 ? ... no

Technical details

system: "aarch64-linux"
host os: Linux 4.14.76, NixOS, 19.03pre162412.6449501902c (Koi)
multi-user?: yes
sandbox: yes
version: nix-env (Nix) 2.1.3
channels(root): "nixos-19.03pre162412.6449501902c"
nixpkgs: /nix/var/nix/profiles/per-user/root/channels/nixos

CC @fpletz @globin @krav

I've been mostly able to reproduce this, but not seeing any errors in the form of "wrong or missing hooks".
Hooks on my system is a symlink to the nix store, so I guess this is a case similar to authorized_keys where upgrading and garbage collection leads to a dead symlink.
The uploads permissions seems easy to fix, same with upgrading Redis and Ruby. Init scripts we don't have AFAIK, unless it refers to the systemd unit and the pre-start script?
I get an additional error, not sure what the "default" SSH configuration is:
Git user has default SSH configuration? ... no
  Try fixing it:
  mkdir ~/gitlab-check-backup-1545132645
  sudo mv /var/gitlab/state/home/.ssh/authorized_keys.bak2 ~/gitlab-check-backup-1545132645
  sudo mv /var/gitlab/state/home/.ssh/authorized_keys.bak ~/gitlab-check-backup-1545132645
  For more information see:
  doc/ssh/README.md in section "SSH on the GitLab server"

Gitlab probably doesn't officially support NixOS, even if we would pass this check.
If you really want to go down that path, best way forward might be to reach out to them, and see what their requirements for getting support on NixOS would be - there's not much we can change downstream at this time.


Warning: Promise.defer is deprecated and will be removed in a future version. Use new Promise instead.

When writing and reading files, the technique used for Promises seems to be deprecated and this might break excel js in future updates.

+1

+1

+1

+1

maybe we use anothor promise lib?

use Q promise?

Thanks for the heads-up. Will look into it

Would appreciate it...

Promise.defer() is not part of the Promise API anymore:
https://developer.mozilla.org/en-US/docs/Mozilla/JavaScript_code_modules/Promise.jsm/Deferred
Instead of doing:
var deferred = Promise.defer();
doSomething(function cb(err, res) {
    if (err) deferred.reject(err); else deferred.resolve(res);
});
return deferred.promise;

You should do:
return new Promise(function(resolve, reject) {
    doSomething(function cb(err, res) {
        if (err) reject(err); else resolve(res);
    });
});

Would be great if you could fix this :)

I'm currently working on adding defined names.
After that I'd like to ES6 it up - and yes, change to a new promise lib!

Great!

+1

+1

Merged and Published!


when writing bash scripts, VSCode doesn't recognize that ;& is a valid operator to end a case in a switch/case statement

Type: Bug
example in bash 4.0 or later, im in a WSL workspace aswell:
case $1 in
  build_all)
    make all
    # ;& is a fall through operator that should be recognized by VSCode as a
    # valid operator but it is whining at me saying there is wrong syntax
    ;&
  debug)
    make all debug=yes
    ;;
  *)
    make all
    ;;
esac

VS Code version: Code 1.80.0 (660393deaaa6d1996740ff4880f1bad43768c814, 2023-07-04T15:06:02.407Z)
OS version: Windows_NT x64 10.0.19044
Modes:
Remote OS version: Linux x64 <IP_ADDRESS>-microsoft-standard-WSL2
Remote OS version: Linux x64 <IP_ADDRESS>-microsoft-standard-WSL2
Remote OS version: Linux x64 3.10.0-327.el7.x86_64

System Info

Item
Value

CPUs
13th Gen Intel(R) Core(TM) i7-13700H (20 x 2918)

GPU Status
2d_canvas: enabledcanvas_oop_rasterization: disabled_offdirect_rendering_display_compositor: disabled_off_okgpu_compositing: enabledmultiple_raster_threads: enabled_onopengl: enabled_onrasterization: enabledraw_draw: disabled_off_okvideo_decode: enabledvideo_encode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabledwebgpu: enabled

Load (avg)
undefined

Memory (System)
31.69GB (16.15GB free)

Process Argv
--crash-reporter-id fca7cecf-7642-41ac-9135-435a175e7188

Screen Reader
no

VM
0%

Item
Value

Remote
WSL: Ubuntu

OS
Linux x64 <IP_ADDRESS>-microsoft-standard-WSL2

CPUs
13th Gen Intel(R) Core(TM) i7-13700H (20 x 2918)

Memory (System)
24.81GB (24.02GB free)

VM
0%

Item
Value

Remote
WSL: Ubuntu

OS
Linux x64 <IP_ADDRESS>-microsoft-standard-WSL2

CPUs
13th Gen Intel(R) Core(TM) i7-13700H (20 x 2918)

Memory (System)
24.81GB (24.02GB free)

VM
0%

Item
Value

Remote
SSH: watson.rockwellcollins.com

OS
Linux x64 3.10.0-327.el7.x86_64

CPUs
Intel(R) Xeon(R) CPU E5-1620 0 @ 3.60GHz (8 x 1199)

Memory (System)
3.63GB (2.80GB free)

VM
0%

Extensions (20)

Extension
Author (truncated)
Version

remotehub
Git
0.60.0

gitpod-desktop
git
0.0.136

remote-containers
ms-
0.299.0

remote-ssh
ms-
0.102.0

remote-ssh-edit
ms-
0.86.0

remote-wsl
ms-
0.80.2

vscode-remote-extensionpack
ms-
0.24.0

azure-repos
ms-
0.36.0

remote-explorer
ms-
0.4.1

remote-repositories
ms-
0.38.0

remote-server
ms-
1.3.0

svn-scm
joh
2.17.0

csharp
ms-
1.26.0

python
ms-
2023.12.0

vscode-pylance
ms-
2023.7.10

cmake-tools
ms-
1.14.34

cpptools
ms-
1.16.3

cpptools-extension-pack
ms-
1.3.0

makefile-tools
ms-
0.7.0

cmake
twx
0.0.17

(2 theme extensions excluded)

A/B Experiments
vsliv368:30146709
vsreu685:30147344
python383:30185418
vspor879:30202332
vspor708:30202333
vspor363:30204092
vslsvsres303:30308271
vserr242:30382549
pythontb:30283811
vsjup518:30340749
pythonptprofiler:30281270
vshan820:30294714
vstes263cf:30335440
vscorecescf:30445987
vscod805cf:30301675
binariesv615:30325510
bridge0708:30335490
bridge0723:30353136
vsaa593:30376534
pythonvs932:30410667
vscaac:30438847
vsclangdf:30486550
c4g48928:30535728
dsvsc012:30540252
pynewext54:30695312
azure-dev_surveyone:30548225
vsccc:30610678
2e4cg342:30602488
pyind779:30671433
f6dab269:30613381
pythonsymbol12:30671437
showlangstatbar:30737416
vsctsb:30748421
pythonms35:30701012
03d35959:30757346
pythonfmttext:30731395
pythoncmv:30756943
fixshowwlkth:30771522
hideindicator:30785051
pythongtdpath:30769146
i26e3531:30780429
pythonnosmt12:30779714
pythonidxpt:30784022
pythonnocebcf:30776496
h7j2d465:30786200

@jeff-hykin

If you see an error, that would be from an extension. VS Code doesn't do any validation on bash

What extension would that be if you know? I basically only have microsoft extensions installed and none of them are specifically for bash so if you know which one handles bash validation I'd like to reach out to them to see if they could do a quick fix for it.

@roblourens I think he means the syntax highlighting

;& is not recognized the same as ;;
debug) should be the same colour as build_all)
@emp4506
prob best to open an issue in https://github.com/jeff-hykin/better-shell-syntax

Ok I opened a ticket there, hopefully it get's fixed soon as it seems to be a minor detail :)

https://github.com/jeff-hykin/better-shell-syntax/issues/63


add simple test for thread registration

Tests were missing in #450 and I just found them again in my local repo

Thank you!


Config changes to reflect the default branch from master to main. Please merge to complete the default branch renaming job.

Auto branch renaming content.

Docs Build status updates of commit 5204044:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

.openpublishing.publish.config.json
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


TurtleSerializer baseUrl and prefix mappings effect

Say I create a RDF graph.
I'm trying to save the graph in a file and I want to set the baseUrl and prefix mappings.
However, the following won't set the baseUrl and will only put the default prefix mappings (not the ones I specified).
main :: IO ()
main = do
  let prefixMappings = ...
  let graph = ...
  let baseUri = Just $ "http://localhost:8080/resource"
  withFile "..." WriteMode (\h -> hWriteRdf (TurtleSerializer baseUri prefixMappings) h graph)

Seems like the only way to set the baseUrl and prefix mapping is through mkRdf or addPrefixMappings. If that's the case, what is the point of TurtleSerializer parameters ?

Actually, the 2nd parameter of TurtleSerializer correctly adds the specified prefixes in the saved file, but it also adds the defaults prefixes from Data.RDF.Namespace.standard_ns_mappings

Actually, the 2nd parameter of TurtleSerializer correctly adds the specified prefixes in the saved file, but it also adds the defaults prefixes from Data.RDF.Namespace.standard_ns_mappings

@koslambrou could you put together a small, compilable, example that demonstrates this? Thanks.

Here:
test :: IO ()
test = do
  let emptyRdf = RDF.empty :: RDF RDF.TList
  let mappings = RDF.PrefixMappings $ Map.fromList [ ("mo", "http://purl.org/ontology/mo/") ]
  let baseUrl = "http://example.org/resource"
  RDF.writeRdf (TurtleSerializer Nothing mappings) emptyRdf

Expected output:
@base <http://example.com/resource> .
@prefix mo: <http://purl.org/ontology/mo/> .

Actual output:
@prefix dc: <http://purl.org/dc/elements/1.1/> .
@prefix dct: <http://purl.org/dc/terms/> .
@prefix ex: <http://www.example.org/> .
@prefix ex2: <http://www2.example.org/> .
@prefix foaf: <http://xmlns.com/foaf/0.1/> .
@prefix mo: <http://purl.org/ontology/mo/> .
@prefix owl: <http://www.w3.org/2002/07/owl#> .
@prefix rdf: <http://www.w3.org/1999/02/22-rdf-syntax-ns#> .
@prefix rdfs: <http://www.w3.org/2000/01/rdf-schema#> .
@prefix skos: <http://www.w3.org/2004/02/skos/core#> .
@prefix xsd: <http://www.w3.org/2001/XMLSchema#> .

The actual output in my last comment is not on the latest commit.
Here's the actual output:
@prefix mo: <http://purl.org/ontology/mo/> .

There's just the @base missing.

Because in your code:
let baseUrl = "http://example.org/resource"
RDF.writeRdf (TurtleSerializer Nothing mappings) emptyRdf

You're not using baseUrl in the TurtleSerializer.

Oh right!
run :: RIO App ()
run = do
  let emptyRdf = RDF.empty :: RDF RDF.TList
  let mappings = RDF.PrefixMappings $ Map.fromList [ ("mo", "http://purl.org/ontology/mo/") ]
  let baseUrl = Just "http://example.org/resource"
  liftIO $ RDF.writeRdf (TurtleSerializer baseUrl mappings) emptyRdf

The actual output is still:
@prefix mo: <http://purl.org/ontology/mo/> .

Ah, thanks.
The optional argument representing the base URL in the Turtle parser is to use for resolving relative URLs in the document. This may be overridden in the document itself using the @base directive.
However in the Turtle serialisation, the optional base URL is used when serialising all URI nodes in the outputted RDF, i.e. normalising prefix:foo to be http://example.com/foo if the prefix namespace is http://example.com. That is, I'm not sure there's a need to include the @base in the serialised namespaces since no URIs in the outputted RDF are relative, they are all absolute.

The following Turtle output contains a relative URI, no?
@base <http://example.com/resource> .
@prefix rdf: <http://www.w3.org/1999/02/22-rdf-syntax-ns#> .
@prefix rdfs: <http://www.w3.org/2000/01/rdf-schema#> .

</Movie> a rdfs:Class .

Isn't /Movie considered a relative URI since it should be resolved to http://example.com/resource/Movie with the base URI ?
Also, I don't understand the use case behind the optional base URL in the Turtle serialisation. Why would we want the convert a single prefix (e.g. prefix:foo) to absolute form (e.g. http://example.com/foo). Hope I make sense.


Have a mechanism to manually run automerge checks for PRs opened by non-authorised users/bots

Currently Automerge checks are run only for PRs opened by a limited list of authorised bots.  There are some users who host their code on services different from github.com and gitlab.com for whom using JuliaRegistrator or the JuliaHub services isn't an option, however this means that

they need to manually open PRs to this repository (presumably they're using LocalRegistry.jl which at least makes this process simpler)
when they open the PR to General, Automerge doesn't run. While this is reasonable because Automerge is meant to be run only for PRs proposing new versions or new packages, arbitrary PRs shouldn't trigger automerge checks, this makes life for the registry maintainers harder

I think we should be able to have a mechanism (comment-based?  adding a label?) to let the repo maintainers trigger Automerge checks for PRs that they evaluate are safe candidates for Automerge checks.  One challenge is that the result of the automerge check should be invalidated if the PR is later modified (e.g. by pushing a new commit), to prevent malicious action.

safe candidates for Automerge checks

Safety isn't that hard for Automerge to assess itself. It could e.g. require that some key information is given in the PR body, then call RegistryTools itself to see if it gets the same file changes as in the PR.
Another question is whether the PR author should be allowed to make registrations for the package. That could e.g. be solved by having a file in the package repo listing approved users.

One challenge is that the result of the automerge check should be invalidated if the PR is later modified (e.g. by pushing a new commit), to prevent malicious action.

AutoMerge has two "components": the PR job that runs on the PR and decides whether the PR "passes AutoMerge" or "fails AutoMerge" (but doesn't actually merge anything), and the cron job that goes through the list of PRs and does the merges. The way that the two components communicate with each other is via the automerge/decision commit status. The PR job creates the automerge/decision commit status, and then the cron job reads the value of the automerge/decision commit status.
 So if a new commit is pushed, the old automerge/decision commit status will be associated with the old commit, not the new commit (if I'm remembering everything correctly), and thus the new commit won't be able to get automerged.
So, I think the existing decision of AutoMerge already takes care of this aspect of the OP.

safe candidates for Automerge checks

Safety isn't that hard for Automerge to assess itself.

The registry maintainer needs to manually assess safety themselves prior to triggering AutoMerge.
Otherwise, someone can just open a PR modify the manifest files in .ci/ to use a malicious fork of RegistryCI.jl. The registry maintainer triggers AutoMerge, the AutoMerge GitHub Actions job runs, it runs the code in the malicious fork of RegistryCI.jl, and then (because the AutoMerge GitHub Action job runs with a GITHUB_TOKEN that has write access to the General registry), the malicious codebase steals the GITHUB_TOKEN, and the attacker has compromised the General registry.

Good points. Somehow it seems like you would like to run the CI checks from a different repository, to separate the testing infrastructure from the registry content. I have no idea what token nightmares that would lead to.


Fix a bug for 64-bit 2D Morton codes

Introduced in #635.
We do (1 << 31). The problem is that MAX_INT (which is<PHONE_NUMBER>) is one smaller than that, which leads to overflow.
NVCC correctly warns about that. But we don't catch Nvidia warnings in CI (see #272), so we missed this.
Not sure why none of the host compilers produced a warning.
This is the only place where it is necessary, all other places shift by less than 31.
This bug potentially leads to construction of a wrong hierarchy.

Can you think of a test that would catch that issue?

So far, no. In fact, unsigned int N = 1 << 31; and unsigned int N = 1u << 31; produce the same value on CPU. However, I had a severe performance degradation when running 2D hacked code on a real problem. I think it may only manifest on a GPU.

OK, after careful examination, this is not a bug. It works correctly despite doing a weird thing. The slowdown I observed on GPU was actually caused by a different issue.

Traversal algorithms improvements, only warnings from SYCL container build.


Add button on the landing page

create a button before the `join the community
make sure both the button aligned nicely
add the docs link (https://www.openprivacytech.org/docs/intro)
to the button

i would love to be assigned this issue please

Hi @raji2004 - Awesome!
assigning you to this issue.