Add pkg-config to buildtool dependencies

pkg-config is required to find GTS dependency needed to build the graphics component

I believe this is due to a bug in https://github.com/gazebo-tooling/gz_vendor/blob/main/create_gz_vendor_pkg/templates/package.xml.jinja in that it doesn't include buildtool dependencies when generating the package.xml file. I have a potential fix in mind, but I won't be able to get a PR out till after next week due to ROSCon.


Pickle error from Trainer.fit when using MLFlowLogger and distributed data parallel without SLURM

🐛 Bug
Trainer.fit fails with a pickle error when the logger is MLFlowLogger, and distributed_backend='ddp' on GPUs but without SLURM.
To Reproduce
Steps to reproduce the behavior:

Instantiate MLFlowLogger in Pytorch <IP_ADDRESS> with Pytorch 1.3.1 and MLFlow 1.4.0. The execution environment has environment variables MLFLOW_TRACKING_URI, and also MLFLOW_TRACKING_USERNAME and MLFLOW_TRACKING_PASSWORD to connect to the MLflow tracking server with HTTP Basic Authentication. The MLflow tracking server is also v1.4.0.
Instantiate Trainer with MLFlowLogger instance as logger, distributed_backend='ddp' and with the gpus parameter on a machine with NVIDIA GPUs but without SLURM.
Run Trainer.fit

From the error output, it looks like multiprocessing is attempting to pickle the nested function in MLflow function _get_rest_store:
ayla.khan@gpu12:~/photosynthetic$ python test_mlflow.py
Traceback (most recent call last):
  File "test_mlflow.py", line 71, in <module>
    trainer.fit(model)
  File "/mnt/unihome/home/CORP/ayla.khan/miniconda2/envs/photosynthetic/lib/python3.6/site-packages/pytorch_lightning/trainer/trainer.py", line 343, in fit
    mp.spawn(self.ddp_train, nprocs=self.num_gpus, args=(model,))
  File "/mnt/unihome/home/CORP/ayla.khan/miniconda2/envs/photosynthetic/lib/python3.6/site-packages/torch/multiprocessing/spawn.py", line 162, in spawn
    process.start()
  File "/mnt/unihome/home/CORP/ayla.khan/miniconda2/envs/photosynthetic/lib/python3.6/multiprocessing/process.py", line 105, in start
    self._popen = self._Popen(self)
  File "/mnt/unihome/home/CORP/ayla.khan/miniconda2/envs/photosynthetic/lib/python3.6/multiprocessing/context.py", line 284, in _Popen
    return Popen(process_obj)
  File "/mnt/unihome/home/CORP/ayla.khan/miniconda2/envs/photosynthetic/lib/python3.6/multiprocessing/popen_spawn_posix.py", line 32, in __init__
    super().__init__(process_obj)
  File "/mnt/unihome/home/CORP/ayla.khan/miniconda2/envs/photosynthetic/lib/python3.6/multiprocessing/popen_fork.py", line 19, in __init__
    self._launch(process_obj)
  File "/mnt/unihome/home/CORP/ayla.khan/miniconda2/envs/photosynthetic/lib/python3.6/multiprocessing/popen_spawn_posix.py", line 47, in _launch
    reduction.dump(process_obj, fp)
  File "/mnt/unihome/home/CORP/ayla.khan/miniconda2/envs/photosynthetic/lib/python3.6/multiprocessing/reduction.py", line 60, in dump
    ForkingPickler(file, protocol).dump(obj)
AttributeError: Can't pickle local object '_get_rest_store.<locals>.get_default_host_creds'

Code sample
Sample code tested with a very simple test model (gist):
test_hparams = Namespace()
model = XORGateModel(test_hparams)

logger = MLFlowLogger(experiment_name='test_lightning_logger',
                                          tracking_uri=os.environ['MLFLOW_TRACKING_URI'])
trainer = pl.Trainer(logger=logger, distributed_backend='ddp', gpus='-1')
trainer.fit(model)

Expected behavior
Trainer.fit runs without error.
Environment
(photosynthetic) ayla.khan@gpu12:~/photosynthetic$ python collect_env.py
Collecting environment information...
 We have a test that is supposed to prevent these problems from sneaking back in, but apparently it's not doing it's job.

I imagine everyone is busy with the build failures - but for the record, I am  having a similar problem. Essentially, I cannot get a logger to work using ddp. It's gving me one of those days when I wonder why I ever wanted to write software ;)
This is Ubuntu 18.04.2LTS, on a 14 core, 7 gpu machine. Python 3.6.8, pytorch 1.3.1, pytorch-lightning <IP_ADDRESS>, Tensorboard 2.1.0. Everything else standard except pillow isis 6.2.2 due to known bug in 7.0.
I am working with a tried and true model and hyperparameters. The model and logging work fine as cpu, gpu, or dp - and ddp if I don't log. But not ddp with logging.
I have tried to get around this several ways: passing a custom logger, not using the logger created by Trainer(), etc. Thet either fail when called from one of the new processes, with an attribute error in Tensorboard TTDummyFileWriter.get_logdir(), or they fail with a pickle error about thread.locks when being copied to a new process
I will detail these in a bug report if you think they are NOT due to the recent build issues.
But thought you'd want to know ...
s

@dbczumar,  @smurching?

@neggert is this fixed now?

Can this issue be re-opened? I'm currently working with Pytorch-Lightning==0.7.6 and am getting an identical pickle issue when using DDP with the MLFLowLogger.
Reproducing
Using the script the OP gave led to some other errors (mostly to do with lightning version differences), so a new gist to reproduce in Pytorch-Lightning 0.7.6 can be found here.
This is easily reproducible in other projects as well.
Error Output
/home/user/anaconda3/envs/pytorch/lib/python3.6/site-packages/pytorch_lightning/utilities/distributed.py:23: DeprecationWarning: `logging` package has been renamed to `loggers` since v0.7.0 The deprecated package name will be removed in v0.9.0.
  warnings.warn(*args, **kwargs)
/home/user/anaconda3/envs/pytorch/lib/python3.6/site-packages/pytorch_lightning/utilities/distributed.py:23: DeprecationWarning: `mlflow_logger` module has been renamed to `mlflow` since v0.6.0. The deprecated module name will be removed in v0.8.0.
  warnings.warn(*args, **kwargs)
/home/user/anaconda3/envs/pytorch/lib/python3.6/site-packages/pytorch_lightning/utilities/distributed.py:23: DeprecationWarning: `data_loader` decorator deprecated in v0.7.0. Will be removed v0.9.0
  warnings.warn(*args, **kwargs)
GPU available: True, used: True
No environment variable for node rank defined. Set as 0.
CUDA_VISIBLE_DEVICES: [0,1,2,3]
Traceback (most recent call last):
  File "mlflow_test.py", line 65, in <module>
    trainer.fit(model)
  File "/home/user/anaconda3/envs/pytorch/lib/python3.6/site-packages/pytorch_lightning/trainer/trainer.py", line 844, in fit
    mp.spawn(self.ddp_train, nprocs=self.num_processes, args=(model,))
  File "/home/user/anaconda3/envs/pytorch/lib/python3.6/site-packages/torch/multiprocessing/spawn.py", line 200, in spawn
    return start_processes(fn, args, nprocs, join, daemon, start_method='spawn')
  File "/home/user/anaconda3/envs/pytorch/lib/python3.6/site-packages/torch/multiprocessing/spawn.py", line 149, in start_processes
    process.start()
  File "/home/user/anaconda3/envs/pytorch/lib/python3.6/multiprocessing/process.py", line 105, in start
    self._popen = self._Popen(self)
  File "/home/user/anaconda3/envs/pytorch/lib/python3.6/multiprocessing/context.py", line 284, in _Popen
    return Popen(process_obj)
  File "/home/user/anaconda3/envs/pytorch/lib/python3.6/multiprocessing/popen_spawn_posix.py", line 32, in __init__
    super().__init__(process_obj)
  File "/home/user/anaconda3/envs/pytorch/lib/python3.6/multiprocessing/popen_fork.py", line 19, in __init__
    self._launch(process_obj)
  File "/home/user/anaconda3/envs/pytorch/lib/python3.6/multiprocessing/popen_spawn_posix.py", line 47, in _launch
    reduction.dump(process_obj, fp)
  File "/home/user/anaconda3/envs/pytorch/lib/python3.6/multiprocessing/reduction.py", line 60, in dump
    ForkingPickler(file, protocol).dump(obj)
AttributeError: Can't pickle local object '_get_rest_store.<locals>.get_default_host_creds'

Environment
* CUDA:
	- GPU:
		- GeForce GTX 1080 Ti
		- GeForce GTX 1080 Ti
		- GeForce GTX 1080 Ti
		- GeForce GTX 1080 Ti
	- available:         True
	- version:           10.2
* Packages:
	- numpy:             1.18.5
	- pyTorch_debug:     False
	- pyTorch_version:   1.5.0
	- pytorch-lightning: 0.7.6
	- tensorboard:       2.2.2
	- tqdm:              4.46.1
* System:
	- OS:                Linux
	- architecture:
		- 64bit
		-
	- processor:         x86_64
	- python:            3.6.8
	- version:           #102-Ubuntu SMP Mon May 11 10:07:26 UTC 2020

To add to this, it appears to be a greater issue with MLFLow and how their tracking utilities are coded. They use a higher order function that causes issues with pickling in torches DDP backend. I've created an issue on MLFLow git, and submitted a PR to remedy the problem.
In the interim, feel free to implement the fix described in the issue in the MLFlow git as a temporary fix until/if they review/merge mine

Following up on this: The pickling fix was merged into the master branch of MLFlow a couple days ago (see the bug mention above). Training using DDP is now functional on MLFLow versions installed from master, but it may take them some time to release the fix to PyPi

Running into this same issue as are a few others here:
https://github.com/minimaxir/aitextgen/issues/135


Archive Profile: Change permissions should be unset

Under “Change Permissions”, “is active” should switch to “no”; any custom permissions should be removed, GitHub login should get disabled

Not every user has access to AMY through email login. If profile archival disables github login, they will not be able to log in back.

If profile archival disables github login, they will not be able to log in back.

@pbanaszkiewicz, yes that is intended. As they asked for their profile to be archived, they should not be able to log into AMY.

Is active is unset already with https://github.com/carpentries/amy/pull/1905


Files overwritten on mount

When starting the container, if you mount in custom config or even client.json, it's blindly being processed and replaced with the template.

If you want to override the configs I would recommend mounting your custom configs over the templates.

The problem is, the mount is done prior to running the bin/start so they are always overwritten. I'm currently resorting to building an image layer over yours that removes the process_template() steps but would really appreciate a way to just use config files in a host mount volume instead of them being generated with env vars.

My recommendation was to mount /etc/sensu/templates

Ah right, my bad!

So fully what is happening is all files in https://github.com/sstarcher/docker-sensu/tree/master/templates are processed.  If you have a file that overlaps with those files it will get overwritten.  If you would like to not use the built in templates you can mount over /etc/sensu/templates
If you want to provide extra templates you can mount and add those to
CONFIG_DIR=/etc/sensu/conf.d
CHECK_DIR=/etc/sensu/check.d
HANDLERS_DIR=/etc/sensu/handlers

Any files in the above directories that end in .tmpl will be processed and added correctly.


Unable to find a specification for TwilioChatClient

@joliveros
[!] Unable to find a specification for `TwilioChatClient (~> 0.16.0)`
[!] Unable to find a specification for `TwilioAccessManager (~> 0.1.1)`

Hi @otymartin,
It seems you may have a problem with your pods repo. I would try either one of these solutions:
pod repo remove master
pod setup
pod install

Or:
sudo rm -fr ~/.cocoapods/repos/master
pod setup
pod install

whats the difference between TwilioChatClient, TwilioAccessManager and TwilioCommon, TwilioIPMessagingClient?

You can see more info regarding the refactor of the SDK here:
https://www.twilio.com/docs/api/chat/guides/migration-guide-ios-0160

@joliveros  i tried both methods but nothing works
It only worked when I specified the source at the top of my podfile, how come?
source 'https://github.com/CocoaPods/Specs'
source 'https://github.com/twilio/cocoapod-specs'

That's definitely a sign that something is wrong with cocoapods setup. I would clear my config and install cocoapods again:
One way to clear caches:
https://gist.github.com/mbinna/4202236
Re-install cocoapods:
gem install cocoapods

@otymartin Those source lines are necessary and currently in the Podfile. If you don't specify a source, cocoapods will try to use https://github.com/twilio/cocoapod-specs. But as we need to specify Twilio's source, you also need to explicitly tell cocoapods to search in https://github.com/twilio/cocoapod-specs


Add overrides to python rosdeps

Please add the following dependency to the rosdep database.
Package name:
overrides
Package Upstream Source:
https://github.com/mkorpela/overrides
Purpose of using this:
This dependency makes it easier and cleaner to manage docstrings when using class inheritance in Python3.
Links to Distribution Packages

pip: https://pypi.org

https://pypi.org/project/overrides

Debian: https://packages.debian.org/

Use pip package linked above. Not available in the Debian repositories.

Ubuntu: https://packages.ubuntu.com/

Use pip package linked above. Not available in the Ubuntu repositories.

Fedora: https://packages.fedoraproject.org/

Use pip package linked above. Not available in the Fedora repositories.

Arch: https://www.archlinux.org/packages/
-https://archlinux.org/packages/extra/any/python-overrides/
Gentoo: https://packages.gentoo.org/

Not available

macOS: https://formulae.brew.sh/

Use pip package linked above. Not available in the Homebrew repositories.

Alpine: https://pkgs.alpinelinux.org/packages

Not available

NixOS/nixpkgs: https://search.nixos.org/packages

Not available

openSUSE: https://software.opensuse.org/package/

Not available

rhel: https://rhel.pkgs.org/

Not available

Done. The reason I originally kept it with a non-pip ID is because the arch package is non-pip. However, given that that is a less commonly used OS, I understand the logic in keeping pip in the ID.

Done. The reason I originally kept it with a non-pip ID is because the arch package is non-pip. However, given that that is a less commonly used OS, I understand the logic in keeping pip in the ID.

Ah, sorry, I totally missed that it was available as a package in arch.  Sorry about that.
At this point, we can either:

Go back to the non-pip key, given that it does have at least one OS where it is packaged.
Remove the arch entry, and leave it as a pip key.

I'm fine with either one, I'll leave it up to you.

Done, I reverted back to the non-pip key.


Feature Request: Task to cancel a build

Note
Issues in this repo are for tracking bugs, feature requests and questions for the tasks in this repo
For a list:
https://github.com/Microsoft/azure-pipelines-tasks/tree/master/Tasks
If you have an issue or request for the Azure Pipelines service, use developer community instead:
https://developercommunity.visualstudio.com/spaces/21/index.html )
Required Information
Entering this information will route you directly to the right team and expedite traction.
Question, Bug, or Feature?
Type: Feature
Enter Task Name:   NYI
list here (V# not needed):
https://github.com/Microsoft/azure-pipelines-tasks/tree/master/Tasks
Environment

Server - Azure Pipelines or TFS on-premises?

If using TFS on-premises, provide the version:

If using Azure Pipelines, provide the account name, team project name, build definition name/build number:

Agent - Hosted or Private:

If using Hosted agent, provide agent queue name: Ubuntu 16

If using private agent, provide the OS of the machine running the agent and the agent version:

Issue Description
This request comes from a recent change where some agent variables are becoming read-only, thus removing the ability to cancel a build (not fail, but cancel).
See my initial question on the matter here: https://github.com/microsoft/azure-pipelines-yaml/issues/439
My goal is to run various checks and depending on the result cancel the build because I don't need to run the rest of the tasks.
My main use-case is to check for files changes in various folder and cancel the build to save time/resources/agents.
Currently, my solution is to execute the following
  echo "##vso[task.setvariable variable=agent.jobstatus;]canceled"
  echo "##vso[task.complete result=Canceled;]DONE"

However, these variables will become read-only soon.
What I'm asking is a way, any way really, to cancel a build after running some code/script/task.
The simplest would be a task that cancels simply cancels the build and I could control execution with a custom condition using a variable that I would set myself.
Having official support to cancel a build would also make it much easier for people to find the mechanism to do so.
I know I searched for years, while I was working at Microsoft, for a way to cancel a build until I found this solution.
Ideally, those variables wouldn't become read-only and I wouldn't have to change anything, but I figured this request was out-of-scope for this repo.

### Task logs
[Enable debug logging and please provide the zip file containing all the logs for a speedy resolution]
Troubleshooting
Checkout how to troubleshoot failures and collect debug logs: https://docs.microsoft.com/en-us/vsts/build-release/actions/troubleshooting
Error logs
[Insert error from the logs here for a quick overview]

Hey @Cellule - I followed up in https://github.com/microsoft/azure-pipelines-yaml/issues/439
I'm going to close here since if we do something like this, it almost certainly wouldn't be a custom task - we'd probably be more likely to expose a primitive on the agent (similar to what you're doing with echo "##vso[task.setvariable variable=agent.jobstatus;]canceled").
I'm not sure we should be doing this, but https://github.com/microsoft/azure-pipelines-yaml/issues/439 is probably the right thing for that discussion


Update sql-query.js

Fixing errors and making so this should be now be properly runnable.
Only question is whether the URL is too long.

Thanks, should be good!


Make features with similar names distinctive

Description
We have two features with similar names: conversation-filters and conversation-activity-filter. It's not clear from the names they do very different things, and I think it's best to distinguish them via a different name.
I suppose rename conversation-filters to more-conversation-filters, since that's what the feature does actually. It's not adding a whole feature of conversation filters.
Screenshot
No response
Example URL
https://github.com/refined-github/refined-github/issues

The same could also applies to quick-review and quick-review-buttons. There could be more.

The same could also applies to quick-review and quick-review-buttons. There could be more.

I was gonna open an issue for this 😃

Maybe also hide-tips and hide-obvious-tooltips. I know they are not the same but still.

They could even be merged

Side note: while we're at it, quick-pr-diff-options should probably be renamed to quick-diff-options, since it also runs on commit and compare pages.

شرح
ما دو ویژگی با نام های مشابه داریم: conversation-filtersو conversation-activity-filter. از نام‌ها مشخص نیست که آنها کارهای بسیار متفاوتی انجام می‌دهند، و به نظر من بهتر است آنها را از طریق یک نام متفاوت تشخیص دهیم.
من فکر می کنم نام را conversation-filtersبه تغییر دهید more-conversation-filters، زیرا این همان کاری است که ویژگی در واقع انجام می دهد. این یک ویژگی کامل از فیلترهای مکالمه را اضافه نمی کند.
اسکرین شات
بدون پاسخ
URL مثال
https://github.com/refined-github/refined-github/issues

I am an unemployed Iranian youth and I have no involvement in foreign policy. Why should I earn money and receive bitcoins and other digital currencies !?

Any more suggestions


chromedp.Navigate never finishes or returns

What versions are you running?
$ go list -m github.com/chromedp/chromedp
github.com/chromedp/chromedp v0.9.5
$ google-chrome --version
Version 121.0.6167.160 (Official Build) (arm64)
$ go version
go version go1.20.5 darwin/arm64

What did you do? Include clear steps.

package main

import (
	"context"
	"fmt"
	"os"
	"time"

	"github.com/chromedp/chromedp"
)

func TakeScreenShot(ctx context.Context, url string) ([]byte, error) {
	context, cancel := chromedp.NewContext(ctx)
	defer cancel()

	var filebyte []byte
	if err := chromedp.Run(context, chromedp.Tasks{
		chromedp.Navigate(url),
		chromedp.Sleep(1 * time.Second),
		chromedp.CaptureScreenshot(&filebyte),
	}); err != nil {
		return nil, err
	}
	return filebyte, nil
}

func main() {
	url := "https://ord.zuexeuz.net/preview/f59f1a5cf10c8b081f503b397e8e3813822c3ace68b7b601fea6ae3e8253c2fai0" // DOES NOT WORK
	// url := "https://ord.zuexeuz.net/preview/821dcd5da03e0a6771ecac114ba1054690183f42b72ae8303e84260a654356c7i0" // WORKS
	ctx := context.TODO()

	data, err := TakeScreenShot(ctx, url)
	if err != nil {
		panic(err)
	}

	defer ctx.Done()

	pngFile, err := os.Create("./shot.png")
	if err != nil {
		panic(err)
	}

	defer pngFile.Close()

	pngFile.Write(data)
	fmt.Println("screen shot taken")
}

What did you expect to see?
I expect the simple code to give me a screenshot. The the un-commented url doesn't work. chromedp.Navigate seems to never return.
What did you see instead?
I would expect some error. The commented url works correctly.
Ive tried this on my local Mac machine as well as a ubuntu server. Both with same results. Im really curious how I can debug this.  The url that is not working returns html which has a canvas element. Im not sure if that maybe the issue.

@ismyhc I just ran the code and it works as expected. The screenshot did take a "long time" (roughly ~30 seconds) to return though. Not sure exactly why this is the case, but it's likely due to the complexity of this script/canvas drawing and not having access to a hardware renderer. Attached is a cleaned up script, the log output, and the actual captured screenshot.
out.txt

ken@ken-desktop:~/b$ cat main.go
package main

import (
	"context"
	"log"
	"os"
	"time"

	"github.com/chromedp/chromedp"
)

func TakeScreenShot(ctx context.Context, url string) ([]byte, error) {
	ctx, cancel := chromedp.NewContext(ctx,
		chromedp.WithDebugf(log.Printf),
	)
	defer cancel()
	var buf []byte
	if err := chromedp.Run(ctx,
		chromedp.Navigate(url),
		chromedp.Sleep(1*time.Second),
		chromedp.CaptureScreenshot(&buf),
	); err != nil {
		return nil, err
	}
	return buf, nil
}

func main() {
	url := "https://ord.zuexeuz.net/preview/f59f1a5cf10c8b081f503b397e8e3813822c3ace68b7b601fea6ae3e8253c2fai0" // DOES NOT WORK
	// url := "https://ord.zuexeuz.net/preview/821dcd5da03e0a6771ecac114ba1054690183f42b72ae8303e84260a654356c7i0" // WORKS
	data, err := TakeScreenShot(context.Background(), url)
	if err != nil {
		panic(err)
	}
	if err := os.WriteFile("shot.png", data, 0o644); err != nil {
		panic(err)
	}
}
ken@ken-desktop:~/b$

Oops, I did want to share what I used to run the script, which I wrapped with time:
$ time go run main.go &> out.txt

real	0m31.050s
user	0m0.611s
sys	0m0.359s

@ismyhc If you're new to using chromedp, I would suggest checking out the examples in the chromedp/examples repository. Most common things people are looking to do with Chrome via chromedp are demonstrated there.

@ismyhc I just ran the code and it works as expected. The screenshot did take a "long time" (roughly ~30 seconds) to return though. Not sure exactly why this is the case, but it's likely due to the complexity of this script/canvas drawing and not having access to a hardware renderer. Attached is a cleaned up script, the log output, and the actual captured screenshot. out.txt
ken@ken-desktop:~/b$ cat main.go
package main

import (
	"context"
	"log"
	"os"
	"time"

	"github.com/chromedp/chromedp"
)

func TakeScreenShot(ctx context.Context, url string) ([]byte, error) {
	ctx, cancel := chromedp.NewContext(ctx,
		chromedp.WithDebugf(log.Printf),
	)
	defer cancel()
	var buf []byte
	if err := chromedp.Run(ctx,
		chromedp.Navigate(url),
		chromedp.Sleep(1*time.Second),
		chromedp.CaptureScreenshot(&buf),
	); err != nil {
		return nil, err
	}
	return buf, nil
}

func main() {
	url := "https://ord.zuexeuz.net/preview/f59f1a5cf10c8b081f503b397e8e3813822c3ace68b7b601fea6ae3e8253c2fai0" // DOES NOT WORK
	// url := "https://ord.zuexeuz.net/preview/821dcd5da03e0a6771ecac114ba1054690183f42b72ae8303e84260a654356c7i0" // WORKS
	data, err := TakeScreenShot(context.Background(), url)
	if err != nil {
		panic(err)
	}
	if err := os.WriteFile("shot.png", data, 0o644); err != nil {
		panic(err)
	}
}
ken@ken-desktop:~/b$

Ive tried running this exact code on my Mac and it never returns anything. Ive waited unto 5 min. Im running on Mac M1 Max, latest osx version. Standard chrome installation, latest version.
Not sure the issue could be. I understand this probably isn't an issue. with chromedp, rather my configuration or setup.

@ismyhc I just ran the code and it works as expected. The screenshot did take a "long time" (roughly ~30 seconds) to return though. Not sure exactly why this is the case, but it's likely due to the complexity of this script/canvas drawing and not having access to a hardware renderer. Attached is a cleaned up script, the log output, and the actual captured screenshot. out.txt
ken@ken-desktop:~/b$ cat main.go
package main

import (
	"context"
	"log"
	"os"
	"time"

	"github.com/chromedp/chromedp"
)

func TakeScreenShot(ctx context.Context, url string) ([]byte, error) {
	ctx, cancel := chromedp.NewContext(ctx,
		chromedp.WithDebugf(log.Printf),
	)
	defer cancel()
	var buf []byte
	if err := chromedp.Run(ctx,
		chromedp.Navigate(url),
		chromedp.Sleep(1*time.Second),
		chromedp.CaptureScreenshot(&buf),
	); err != nil {
		return nil, err
	}
	return buf, nil
}

func main() {
	url := "https://ord.zuexeuz.net/preview/f59f1a5cf10c8b081f503b397e8e3813822c3ace68b7b601fea6ae3e8253c2fai0" // DOES NOT WORK
	// url := "https://ord.zuexeuz.net/preview/821dcd5da03e0a6771ecac114ba1054690183f42b72ae8303e84260a654356c7i0" // WORKS
	data, err := TakeScreenShot(context.Background(), url)
	if err != nil {
		panic(err)
	}
	if err := os.WriteFile("shot.png", data, 0o644); err != nil {
		panic(err)
	}
}
ken@ken-desktop:~/b$

Ive tried running this exact code on my Mac and it never returns anything. Ive waited unto 5 min. Im running on Mac M1 Max, latest osx version. Standard chrome installation, latest version.
Not sure the issue could be. I understand this probably isn't an issue. with chromedp, rather my configuration or setup.

@kenshaw actually, it did finally return after 5.5 min. Do you happen to know of a solution that would handle these types faster? I would assume since I'm running locally it would use gpu if that's an issue.
I have a lot of these I need to chew through. :)


Default to unidoc for scaladoc generation.

Since sbt doc doesn't generate scaladoc for subprojects, switch to sbt unidoc so we generate scaladoc for coreMacros and chiselFrontend.
Disambiguate DontCare reference in initializeSingletons() avoiding compilation error during unidoc generation.
Use DontCare on outs not ins in InvalidateAPISpec.
Add additional InvalidateAPISpec tests demonstrating selective override of Strict/NotStrict behavior.

I suppose this really should be broken up into two commits:

one for the unidoc change
one for the InvalidateAPISpec changes.

Seems like a good idea.


Enhanced Recurring Donations Intro

https://powerofus.force.com/s/article/NPSP-Enhanced-RD-Upgrade-Guide
https://docs.google.com/document/d/1UDVpemfkOtIo5WBYZjK4xgUGy7xjMpUi3YJkpyBABLg/edit

@coriobriensfdo  this is ready for you

@barbarachristensen I reviewed and added some comments. Back to you! :)

@Billflorio ready for you or Rebecca to take on


"UserWarning: Converting sparse IndexedSlices to a dense Tensor of unknown shape." for TFHub BERT encoder

Hi,
I'm using a saved-down BERT encoder (https://tfhub.dev/tensorflow/small_bert/bert_en_uncased_L-2_H-128_A-2/2) inside a TimeDistributed Keras layer and it keeps throwing me the following error out of gradient_tape:
UserWarning: Converting sparse IndexedSlices to a dense Tensor of unknown shape. This may consume a large amount of memory.
Is it anything I can do with it as a user? Or I correctly assume that's on the side of the saved-down BERT model?
Thanks in advance!

@HannaLochOlszewska , please provide a snippet of your code to help us understand the part where you are facing this warning.
It is a warning than an actual error.If it is not killing your training/retraining,it is safe to ignore it.
To suppress warnings, you might try out :-
a) use the below code before your tensorflow imports -
import warnings
warnings.filterwarnings('ignore')

or
b)
import logging
tf.get_logger().setLevel(logging.ERROR)

@HannaLochOlszewska  , please update on your latest status.

@HannaLochOlszewska , closing this as there has been no response for sometime.Please feel free to reopen if required.


Whats with the new callback in ReactCBLite.init?

I just hit the error with a new required parameter for the init method. Whats the callback for?

Apologies, I didn't mean to leave that in - thought I had reverted the change,  I'll fix it up today.  (I found some docs that showed a 3rd parameter but they weren't correct)


Vertical Writing Mode?

 We'd welcome a PR for this functionality.

We won't support this directly, but multi-line labels are now supported (will be available in the next release). If you want vertical labels you can set the labels like:
labels.labels.map(function(label) {
  return label.split('');
});

Sorry Etimberg,
where to put your code?
labels.labels.map(function(label) {
  return label.split('');
});

Can you do a simple jsfiddle example?
Thanks


Added support for injecting custom generic class objects via Depends().

Added support for injecting custom generic class objects via Depends().
how it worked before
how it works now
The Issue:
Using custom generic classes via Depends() led to an unpredictable result. The fastAPI  could not parse the initializer signature correctly due to the way generic classes work in Python.
Why is it important?
The ability to use generic classes via Depends() is expected behavior for a typed program. It allows you to make more stable program using DI and avoid writing duplicate code.
Using generics types makes the code more flexible, safer, and more concise.
What's changed?
dependencies/utils.py module was changed.

Added function 'is_generic_type':
– to check whether the object is generic type.

Added function ‘substitute_generic_type’:
– to substitute a real annotate in the signature.

Changed function ‘get_typed_annotation’:
- added optional attribute ‘typevars’;
- if 'typevars' is received, the function calls ‘substitute_generic_type’;

Changed function ‘get_typed_signature:
- it calls ‘is_generic_type ’ function. If the function returns False, it works as before. Otherwise, the typevars and types mapping dict is created. This dict pass to 'get_typed_annotation'.

Testing:
The changes do not affect the existing implementation of the fastAPI. It only add new functionality.
All basic tests have been completed, and new ones have been added (tests/test_dependency_generic_class.py) that check work with custom generic types via Depends().
It has been tested with Python 3.7-3.10.

@YYKlimenko can you please review why the test cases are failing

@YYKlimenko can you please review why the test cases are failing

I found a silly mistake: two files were mixed up. I did a new commit.


Unable to open .drawio files from OneDrive web app.

Preflight Checklist

[x] I agree to follow the Code of Conduct that this project adheres to.
[x] I have searched the issue tracker for a feature request that matches the one I want to file, without success.

Describe the bug
Can't open .drawio files from my OneDrive web app (onedrive.live.com).
To Reproduce
Steps to reproduce the behavior:

From 'onedrive.live.com', try to open any .drawio file.
Only gives the option to Download, not open the file, see screenshot.

Expected behavior
I am in the process of switching from Google One to Microsoft 365 Personal.
Opening a .drawio file in Google drive gives a preview and the option to open it in diaggrams.net in a new browser tab (both Chrome and Edge).
It seems that .drawio is not among the file types that can be previewed in OneDrive, so a preview is out and that's ok, if not ideal.
But is there any way to at least open the file in diagrams.net, like from Google Drive?
If not, this gets quite problematic. Downloading a separate file locally, editing it in the draw io win app, then having to manually replace the original file in OneDrive is not a reasonable solution.
Using only the draw io Win app to open files in my local OneDrive folder is not really a solution either, since I want to be able to link to/share/integrate with docs etc. I also prefer working in my browser, because of the auto-save, ability to Bookmark and group with other tabs by projects etc.
Is there any way to make this work? I really love the app and would hate to have to look for alternatives for this reason alone.
Screenshots
If applicable, add screenshots to help explain your problem.

draw.io version (In the Help->About menu of the draw.io editor):

draw.io version 23.1.5 Win app, but this is really about diagrams.net <IP_ADDRESS> (?)

Desktop (please complete the following information):

OS: Windows 10
Chrome 122.0.6261.95 or Edge 122.0.2365.66 - no difference

I tested the problem in incognito/private mode with all browser extensions switched off, write "yes" below:
Yes
Additional context
If I try sharing a diagram, similar to issue #3758 I also only get the "Download" option after being redirected to OneDrive, not "open" as the OP of that issue.

OneDrive doesn't support opening third-party apps in the way Google Drive does. The only option is to open within app.diagrams.net.

The workaround I found for this is:

open the draw.io stored in Onedrive/SharePoint in app.diagram.net from app.diagram.net
right click on the page tab on the bottom and choose Link....
Create a view and/or Edit link.
Click new/create link in Onedrive/SharePoint in the same folder or other folder where you saved your draw.io file and paste the the created link from step 3.
now you can single click from OneDrive/Sharepoint to open a draw.io file in the browser(app.diagram.net) and also share this link.

Thanks! Ended up going back to gdrive for various reasons, but appreciate
you taking the time to get back to me on this!
All the best!
On Fri, 7 Jun 2024 at 09:53, Florisz @.***> wrote:

The workaround I found for this is:

open the draw.io stored in Onedrive/SharePoint in app.diagram.net
from app.diagram.net
right click on the page tab on the bottom and choose Link....
Create a view and/or Edit link.
Click new/create link in Onedrive/SharePoint in the same folder or
other folder where you saved your draw.io file and paste the the
created link from step 3.
now you can single click from OneDrive/Sharepoint to open a draw.io
file in the browser(app.diagram.net) and also share this link.

—
Reply to this email directly, view it on GitHub
https://github.com/jgraph/drawio/issues/4228#issuecomment-2154293055,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AZAZVECW74P36PENLB6B543ZGFRIHAVCNFSM6AAAAABI6E66CGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNJUGI4TGMBVGU
.
You are receiving this because you authored the thread.Message ID:
@.***>


Update router5 7.0 -> 8.0

https://router5.js.org/migration/migrating-from-6.x-to-7.x

(Yes that link actually points to the 7 -> 8 docs - not exactly confidence inspiring...)

ah -- well at least if those are the docs for 7-8 that's something.  I had assumed they were the 6-7 docs

I discussed this with Leo -- let's continue to hold off here.

Unfortunately does not look like anything has improved here - project still appears to be effectively abandoned, docs still in misleading state described above.  I'm going to close the ticket as we don't have any pressing need to take an update and it's not clear that the 8.x release is fully baked - and if there are any problems, does not look like they are likely to be fixed.


Version published on crates.io does not match what's on GitHub

The title says everything. According to the crates.io page, the latest version of clickhouse is 0.10.0. As I'm writing this issue, the version of clickhouse from this repository's Cargo.toml is 0.9.3.
Thank you for your work on this crate!

Uh, my mistake, sorry!


Line in chart not showing

On OS X El Capitan version 10.11.5 (15F34)
node --version v6.0.0
nam --v v6.2.2
react-native-cli: 1.0.0
react-native: 0.28.0
watchman --v 4.5.0
xcode 7.3.1
I have this code using react-native-chart:
const styles = StyleSheet.create({
  parent: {
    flex: 1,
    backgroundColor: colors.MAIN_BACKGROUND_COLOR
  },
  container: {
    flex: 1,
    alignItems: 'center',
    justifyContent: 'center',
  },
  chart: {
    width: 300,
    height: 300
  }
});

progression = [[0,1], [1,3], [3,7], [4,9], [5, -10]];
    return (
      <View style={styles.parent}>
        <View style={styles.container}>
          <Chart
            style={styles.chart}
            data={progression}
            type="line"
            />
        </View>
      </View>);

The problem is that the axis are displayed fine but the line with the progression is not displayed.
If I change the type to "bar" or "pie"  works fine, but not for "line".

Still having this problem.
I'm using react-native-chart version 1.0.8-beta.
When I have line type this is the result of the chart (no line showing):

But if I change to bar type, this is the result:

+1, i'm having the same issue.

Hey Laura,
After looking at other open issues I found something that works!
      <Chart
        style={styles.chart}
        data={data}
        verticalGridStep={5}
        showDataPoint={true}
        type="line"
        />

Looks like showDataPoint is false by default, if you set it to true explicitly it'll render on iOS.
It'll be a good workaround for now

I'll be on a plane tomorrow, so I'll have time to investigate these issues. Thanks.

@anthonyalayo it's working with showDataPoint={true} as you said so thank you very much.
@tomauty thank you for your work in this project

As a generic FYI — I am no longer able to maintain this library. I recommend checking out victory-native as it's much more maintained.


Generated configs for LB services does not include information about target's host/port/path.

Server version - V0.27.0-rc1
Create LB services with targets that have host/port/path.
Use "View Config" option to get the docker-compose.yml and rancher-compose.yml for the services.
rancher-compose.
Generated docker-compose.yml/rancher-compose.yml does not have information about the target's host/port/path.

Tested with server version - v0.30.0-rc2
Generated configs for LB services includes information about target's host/port/path
labels:
io.rancher.loadbalancer.target.web2: host1/service1=80
io.rancher.loadbalancer.target.web3: host3:7777
io.rancher.loadbalancer.target.web1: host2/service2


Add documentation for use of env vars for API key and URL

Add doc strings for how the client uses environment variables as fallback for the API key and URL parameters.

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

@mjul Thank you!


stdin cannot be read

I tried to run my stdin reading program yesterday with miri 0.1.0 (10419b3 2020-05-11) and rustc 1.45.0-nightly (a74d1862d 2020-05-14) but upon reading the stdin Ok(0) seems to be always returned at least on Linux.
Example:
use std::io::Read;

fn main() {
    let mut bytes = [0u8; 8192];
    let total = std::io::stdin().lock().read(&mut bytes[..]).unwrap();
    println!("{}", total);
}

With stable echo foo | cargo run prints 4.
With miri echo foo | cargo +nightly miri run -- -Zmiri-disable-isolation prints 0. Same for reading in a loop.
Looking at src/shims/foreign_items/posix.rs and src/shims/fs.rs I can't at least see any relevant TODO's or other notes which would suggest this should not work.
Could this be because src/bin/cargo-miri.rs launches a subprocess and somehow the stdin does not get inherited?

Reading from stdin is indeed not implemented. So what I would have expected is that it returns an error, saying that the FD does not exist...

Yeah we are definitely returning -1 from that one. But somehow libstd swallows the error?

Yeah libstd contains explicit code to translate "ebadf" errors for stdin/stdout/stderr into 0 reads -- probably to work in environments where the handle is for some reason not available. (This is the Maybe wrapper defined in io/stdio.rs.)

Ok good that I filed this issue then, I had misunderstood and was hoping this could be an E-easy issue I could snipe :) I'll update when I've looked more on how this could be implemented.

Hm, I'd still call this easy but yeah it requires digging into the shims code a bit more. ;) Probably it is time to handle FDs 0-2 in shims/fs.rs instead of having that code separately.
And what you said about cargo wrapping Miri causing problems might still be true, but we'll only know after the driver can handle stdin.^^

I improved the error message in https://github.com/rust-lang/miri/pull/1435.

I am interested in picking this issue up.
@RalfJung Just to clarify, we want to start supporting reads on stdin, right?

@samrat yes. The starting point for that would be here:
https://github.com/rust-lang/miri/blob/b334004607f1cfb5754d9b0a007114748f272444/src/shims/foreign_items/posix.rs#L69

Oh, I just realized that @koivunej had mentioned that he was interested in picking this up-- didn't mean to step on their toes like this.
@koivunej Assuming you're still interested in the issue, please it away! I look forward to checking out the solution you come up with :)

@samrat I don't expect to have time until the weekend so you can go ahead. Given the starting point linked above I'd guess reading should be implemented similar to stdout and stderr writing few lines below.

I look forward to checking out the solution you come up with :)

Likewise :)

I'd guess reading should be implemented similar to stdout and stderr writing few lines below.

Yes I think that would be a good start.
Longer-term, I'd like to move the handling of "special" FDs to shims/fs.rs together with the other FDs, with some kind of abstraction for "either a normal FD or stdout/stdin/stderr" -- then we could even support dup or close on the "special" FDs. But that does not have to happen in the same PR.


Bugfix/module imports

PR Checklist
This PR fulfills the following requirements:

[x] The commit message follows our guidelines: Contributing guidelines
[ ] Tests for the changes have been added (for bug fixes / features)
[x] Docs have been added / updated (for bug fixes / features)
[ ] A changelog entry was added

PR Type
What kind of change does this PR introduce?

[x] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[x] Build related changes
[ ] CI related changes
[ ] Documentation content changes
[ ] angular.io application / infrastructure changes
[ ] Other... Please describe:

What is the current behavior?

Issue Number: #38 #27 #22 #19 #17
What is the new behavior?

updated schematics, examples now work and are available as a separate command (for existing packages)
updated angular packages to latest patch
add angular packages as peer dependencies
fixed components not resolved in example modules in styleguide
fix table exports, aot error + add basic example
cleanup dependencies in packages

Does this PR introduce a breaking change?

[ ] Yes
[x] No

Other information
Resolved issues
Fixes: #38 #27 #19 #17

#22 will be fixed in a subbranch of this one. PR coming up.


Fix connection deprecated function logic

When 2 functions share the same Path, we need to separate their deprecation status
One could be deprecated and not the other

✅ No public API change.


Match fabric-network API to v1.4 as closely as possible

Remove Gateway.client
Remove Gateway.identityContext
Rename TransactionOptions to DefaultEventHandlerOptions
Rename transaction option to eventHandlerOptions
Rename QueryOptions to DefaultQueryHandlerOptions
Rename query option to queryHandlerOptions
Remove 'maxAge' from DiscoveryOptions (not used)

/azp run


Errors Here

Hi there currently getting this error on Windows 11 - I have python in my path ENVIRONMENT variables and I also put in pyvenv to install a version for this.  Any help would be amazing as I am excited to use this, thank you!
WARNING: There was an error checking the latest version of pip.
(venv) PS C:\Projects\whisper-diarization> python diarize.py -a session-05-short.m4a --language en
[NeMo W 2024-10-17 18:02:14 nemo_logging:349] C:\Projects\whisper-diarization\venv\lib\site-packages\pyannote\audio\core\io.py:43: UserWarning: torchaudio._backend.set_audio_backend has been deprecated. With dispatcher enabled, this function is no-op. You can remove the function call.
      torchaudio.set_audio_backend("soundfile")

[NeMo W 2024-10-17 18:02:14 nemo_logging:349] C:\Projects\whisper-diarization\venv\lib\site-packages\pyannote\audio\pipelines\speaker_verification.py:43: UserWarning: torchaudio._backend.get_audio_backend has been deprecated. With dispatcher enabled, this function is no-op. You can remove the function call.
      backend = torchaudio.get_audio_backend()

[NeMo W 2024-10-17 18:02:15 nemo_logging:349] C:\Program Files\Python310\lib\inspect.py:869: UserWarning: Module 'speechbrain.pretrained' was deprecated, redirecting to 'speechbrain.inference'. Please update your script. This is a change from SpeechBrain 1.0. See: https://github.com/speechbrain/speechbrain/releases/tag/v1.0.0
      if ismodule(module) and hasattr(module, '__file__'):

[NeMo W 2024-10-17 18:02:15 nemo_logging:349] C:\Projects\whisper-diarization\venv\lib\site-packages\pyannote\audio\pipelines\speaker_verification.py:45: UserWarning: Module 'speechbrain.pretrained' was deprecated, redirecting to 'speechbrain.inference'. Please update your script. This is a change from SpeechBrain 1.0. See: https://github.com/speechbrain/speechbrain/releases/tag/v1.0.0
      from speechbrain.pretrained import (

[NeMo W 2024-10-17 18:02:15 nemo_logging:349] C:\Projects\whisper-diarization\venv\lib\site-packages\pyannote\audio\pipelines\speaker_verification.py:53: UserWarning: torchaudio._backend.set_audio_backend has been deprecated. With dispatcher enabled, this function is no-op. You can remove the function call.
      torchaudio.set_audio_backend(backend)

[NeMo W 2024-10-17 18:02:15 nemo_logging:349] C:\Projects\whisper-diarization\venv\lib\site-packages\pyannote\audio\tasks\segmentation\mixins.py:37: UserWarning: `torchaudio.backend.common.AudioMetaData` has been moved to `torchaudio.AudioMetaData`. Please update the import path.
      from torchaudio.backend.common import AudioMetaData

Python was not found; run without arguments to install from the Microsoft Store, or disable this shortcut from Settings > Manage App Execution Aliases.
WARNING:root:Source splitting failed, using original audio file. Use --no-stem argument to disable it.
Lightning automatically upgraded your loaded checkpoint from v1.5.4 to v2.4.0. To apply the upgrade to your files permanently, run `python -m pytorch_lightning.utilities.upgrade_checkpoint C:\Users\andrewbeniston\.cache\torch\whisperx-vad-segmentation.bin`
Model was trained with pyannote.audio 0.0.1, yours is 3.1.1. Bad things might happen unless you revert pyannote.audio to 0.x.
Model was trained with torch 1.10.0+cu102, yours is 2.5.0+cpu. Bad things might happen unless you revert torch to 1.x.

can you try running this simple script?
import os
os.system('python3 --version')


[fix-#12875]command line that  -Dmaven.test.skip=true  have no effect

command line that  -Dmaven.test.skip=true  have no effect
Purpose of the pull request
Brief change log
command line that  -Dmaven.test.skip=true  have no effect
Verify this pull request
This pull request is code cleanup without any test coverage.

Codecov Report

Merging #12876 (f433cdc) into dev (87a88e3) will decrease coverage by 0.00%.
The diff coverage is n/a.

:exclamation: Current head f433cdc differs from pull request most recent head 1debacc. Consider uploading reports for the commit 1debacc to get more accurate results

@@             Coverage Diff              @@
##                dev   #12876      +/-   ##
============================================
- Coverage     39.27%   39.27%   -0.01%
  Complexity     4231     4231
============================================
  Files          1051     1051
  Lines         39849    39817      -32
  Branches       4585     4567      -18
============================================
- Hits          15652    15637      -15
+ Misses        22422    22410      -12
+ Partials       1775     1770       -5

Impacted Files
Coverage Δ

...r/plugin/registry/zookeeper/ZookeeperRegistry.java
43.54% <0.00%> (-6.46%)
:arrow_down:

...hinscheduler/plugin/alert/script/ScriptSender.java
70.58% <0.00%> (-1.64%)
:arrow_down:

...r/plugin/task/sqoop/parameter/SqoopParameters.java
51.19% <0.00%> (-1.20%)
:arrow_down:

...cheduler/plugin/alert/dingtalk/DingTalkSender.java
34.13% <0.00%> (-0.78%)
:arrow_down:

...rver/master/runner/task/BlockingTaskProcessor.java
75.86% <0.00%> (-0.55%)
:arrow_down:

...inscheduler/plugin/registry/etcd/EtcdRegistry.java
50.69% <0.00%> (-0.35%)
:arrow_down:

.../dolphinscheduler/plugin/task/datax/DataxTask.java
36.62% <0.00%> (-0.26%)
:arrow_down:

.../dolphinscheduler/plugin/task/spark/SparkTask.java
72.56% <0.00%> (-0.25%)
:arrow_down:

...dolphinscheduler/service/storage/impl/S3Utils.java
0.00% <0.00%> (ø)

...eduler/api/dto/workflow/WorkflowCreateRequest.java
100.00% <0.00%> (ø)

... and 17 more

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Make VPC database resources to be conditionally added only if there is a database needed in the network

Ticket
N/A
Changes

What was added, updated, or removed in this PR.

Set database subnets and subnet group to be conditionally added only if there is a database specified in the network

Context for reviewers

Testing instructions, background context, more in-depth details of the implementation, and anything else you'd like to call out or ask reviewers.

Right now, the first time you configure the network and run make infra-update-network NETWORK_NAME=<NETWORK_NAME>, the database subnets and subnet group are created even if there are no databases specified in the network.
This change makes creating those resources conditional based on whether or not there databases are needed.
I think it's better to make this conditional to reduce the number of unnecessary resources created. In general, unnecessary resources add additional cost and management overhead (i.e. it was confusing to me to see database subnets being created for an application that had no database).
Testing

Provide evidence that the code works as expected. Explain what was done for testing and the results of the test plan. Include screenshots, GIF demos, shell commands or output to help show the changes working as expected. ProTip: you can drag and drop or paste images into this textbox.

Testing performed by deploying this branch to my local version of https://github.com/navapbc/platform-test and then deploying necessary resources (account, network) to my AWS account. This screenshot shows:

on the top left, has_database is set to false in /infra/app/app-config/main.tf
on the bottom left, the changes in this PR
on the right, the deployed dev network which does not have any database VPC endpoints

left some nits. also, i didn't see any evidence of testing in the PR, which i'd ideally like to see before approving

Adding now and will re-ping you when I'm ready for re-review. Thanks!

Rollout note: One thing to remember to do after merging, is that on platform-test-nextjs you'll want to run terraform apply in the network layer, since the network layer doesn't get auto-applied as part of CD. I imagine apply would end up removing the subnets and subnet group.

Thank you for the tip! I would have missed that.

Rollout note: One thing to remember to do after merging, is that on platform-test-nextjs you'll want to run terraform apply in the network layer, since the network layer doesn't get auto-applied as part of CD. I imagine apply would end up removing the subnets and subnet group.

Thank you for the tip! I would have missed that.

Following up to say that this is complete!


Size in TextLayoutInfo does not correspond to actual Text2d dimensions with non-one scale factor

Bevy version
main
Relevant system information
AdapterInfo { name: "Apple M1 Max", vendor: 0, device: 0, device_type: IntegratedGpu, driver: "", backend: Metal }
SystemInfo { os: "MacOS 13.1 ", kernel: "22.2.0", cpu: "Apple M1 Max", core_count: "10", memory: "64.0 GiB" }

(Scale factor 2.0)
What you did
Spawn a Text2dBundle with a SpriteBundle in the background, sized according to TextLayoutInfo.size.
use bevy::{prelude::*, text::TextLayoutInfo};

fn main() {
    App::new()
        .add_plugins(DefaultPlugins)
        .add_startup_system(setup)
        .add_system(resize)
        .run();
}

fn resize(
    mut sprite_query: Query<&mut Sprite>,
    info_query: Query<&TextLayoutInfo, Changed<TextLayoutInfo>>,
) {
    let mut sprite = sprite_query.single_mut();

    for info in &info_query {
        sprite.custom_size = Some(Vec2::splat(4.) + info.size);
    }
}

fn setup(mut commands: Commands, asset_server: Res<AssetServer>) {
    commands.spawn(Camera2dBundle::default());

    commands.spawn(Text2dBundle {
        text: Text::from_section(
            "text2dtext2dtext2d",
            TextStyle {
                font: asset_server.load("fonts/FiraSans-Bold.ttf"),
                font_size: 60.0,
                color: Color::WHITE,
            },
        ),
        transform: Transform::from_xyz(0., 0., 1.),
        ..default()
    });

    commands.spawn(SpriteBundle {
        sprite: Sprite {
            color: Color::BLACK,
            ..default()
        },
        ..default()
    });
}

What went wrong
I expected a black box underneath my text, with 4px of padding.
Instead, the black box is 2x the size of the text with an additional 4px of padding.

Additional information
I had previously used Text2dSize for this purpose which worked fine, but it was removed in #6807.
I can/will work around this by querying for the scale factor and dividing, but it seems like a hack.

This is my fault, I should have considered this when I made this change.
I think this points to a place the code can be simplified and there isn't any need for TextInfo to be a scaled value. It's just inverted again in the extraction function anyway.
I'll just make a short PR to fix it now.

#7794


Repo name

Should we rename this repository to just yak since the architecture has changed to a server/client combined system?

Done.


Release v0.11.1

And it's released! Now to figure out how to better automate this part of the process....
Thanks for putting this together @Unix-Code.


System.Runtime.InteropServices.RuntimeInformation

Hi,
When I try to instal your nuget I got this error:
Could not install package 'System.Runtime.InteropServices.RuntimeInformation 4.0.0'. You are trying to install this package into a project that targets '.NETPortable,Version=v4.5,Profile=Profile111', but the package does not contain any assembly references or content files that are compatible with that framework. For more information, contact the package author

Please read the readme file.  v5 is .net standard, not PCL.  If you want PCL, you can install v4.x from nuget which supports PCL profile 259.


feat!: generic syscall handlers

Syscall handlers with limited lifetime and are !Send can still be very useful.
We can support them (in addition to 'static + Send + Sync handlers) by making the syscall handler type a generic type parameter. Unlike previous ckb-vm versions, we are not adding a lifetime parameter to the machine types.
I've also implemented Syscall for () (for when there's no syscalls) and &mut impl SupportMachine -> Result closures. Vec<BoxedSyscalls> is now handled by impl Syscall for Vec<BoxedSyscalls>.

See #355 instead.


Custom Codeblock "linenos" start

Hi Sir,
{% highlight python linenos %} is the valid syntax and the numbering for each code block starts from 1. Is there anyway I give a custom line number to start with ??
...... Thanks in advance.

@dAmnation69 Don't think so. This is a question is better for the Jekyll maintainers as the {% highlight %} tag is part of their project. I'd search around their issues or raise your own.


[FEATURE] Get rid of the dependency on OpenSearch core

Is your feature request related to a problem?
The opensearch-java library uses RestClient from OpenSearch core, bringing in the kitchen sink and causing problems such as #156.
What solution would you like?
Get rid of the dependency.

The core problem I see is that this dependency couples Java clients to the server implementation. The Java version does not impact Python or JS clients for example. But, the update the JDK 11 in core forced an update on clients. OpenSearch core should be able to update JDK versions based on the needs of system administrators and developers, not based on the needs of clients. Similarly, client developers should be able to update their JDK versions based on their needs and the needs of more complicated projects.
The low-level REST client from OpenSearch core is mostly a wrapper around Apache HTTP. It adds some logic that is relevant for communicating with an OpenSearch cluster in general, such as load balancing, error/warning handling, node selection controls.
I see a few options:

Remove the dependency from opensearch-java entirely.
Invert the dependencies. That is, make OpenSearch depend on opensearch-java. I think this approach requires that opensearch-java continue to export the low-level REST client as its own library so that core does not need the high-level APIs.

Option 1)
Upsides:

Fixes the core issue.
Does not create a dependency between core and the client.

Some downsides:

We duplicate some of the same logic in both projects.
Client developers must start a migration path

Option 2)
Upsides:

Fixes the core issue.
This could produce the same library in Maven Central that client developers use. Thus, client developers can continue to use the same setup they have been using.
No code duplication.

Some downsides:

OpenSearch core will not depend on opensearch-java

I agree with @dlvenable statement that

The low-level REST client from OpenSearch core is mostly a wrapper around Apache HTTP. It adds some logic that is relevant for communicating with an OpenSearch cluster in general, such as load balancing, error/warning handling, node selection controls.

It indeed does not depend on anything from the core. I would like to suggest a third option:
3) Move off RestClient to separate repo (+ test module as well), for the reasons it is not dependent nor core nor opensearch-java
Pros:

not attached to core or opensearch-java
changed very rarely
own release cycle
no lockstep update for core and opensearch-java (each can do that at own pace)
Cons:
may need core and opensearch-java releases in case of bugfixes

The option 1) is also on the table but I would certainly be against of option 2): the core should have no deps on opensearch-java art any time (my opinion).

I'm good with Option 3 as well.
To clarify, my option 2 was really the same from a module perspective. I was proposing that opensearch-java would produce two jars. They would be developed in the same Git repo and released together. From a module perspective core would not really be affected since it would have the same dependency chain.

I have a few other thoughts on just removing the dependency.
Clarity - We can make a clearer story for client developers. Right now, there are three Java clients. In my opinion, this is a confusing aspect of OpenSearch for end-users, especially client developers. I also experienced something similar when I was using spring-data-elasticsearch and ES 7. I was unsure what client I should be using. If we just remove this dependency, then we have a single client to offer to client developers. This should reduce friction for getting started.
Client Configuration - Using the opensearch-java client has one main transport - RestClientTransport (there is a really nice AWS one, but that is only for some situations). By dropping the RestClient, opensearch-java could instead offer options to use either Apache HTTP Client or Netty. This can help reduce dependencies for clients. Say the client is already using Netty, they can remove an unnecessary Apache dependency and vice-versa.

Thanks @dlvenable, I think there is an agreement to recommend opensearch-java as the preferred option [1]. The RestClient is used in many plugins internally (in core) and very likely outside as well (fe security), so I think it still make sense to reuse it (but that is arguable for sure).

By dropping the RestClient, opensearch-java could instead offer options to use either Apache HTTP Client or Netty.

You can do that now with OpenSearchTransport, one of them happens to be RestClientTransport based on RestClient as you mentioned, so nothing is preventing to offer other options (if it makes sense).
[1] https://github.com/opensearch-project/OpenSearch/issues/4256#issuecomment-1241339946

I think we derailed a bit from the subject (but in a good sense), just to sum up the on the RestClientTransport, since it is relevant: the RestClient is at 95% wrapper around Apache HttpClient 5 (or 4 for 2.x) but there are quite a few things that needed to be implemented to make Apache HttpClient 5 work (there is no way around it). This part I would like to avoid duplicating and we will have to even if we rebrand RestClientTransport and ApacheClientTransport and drop RestClient dependency.

Why should the java client be any different from JS or Python client? It shouldn't. If code duplication is to be had, so what? Either way, the long term plan to avoid code duplication is https://github.com/opensearch-project/opensearch-clients/issues/19 which clarifies the relationship between clients and server.
I would merge a PR that removes the dependency by duplicating whatever code needs to be duplicated and bumping the major version and creating an UPGRADING.md that explains what to do. Hint ... hint :)

Why should the java client be any different from JS or Python client? It shouldn't. If code duplication is to be had, so what?

I think the main difference is that the core uses  java client internally (in plugins / modules / tests fe).

I would merge a PR that removes the dependency by duplicating whatever code needs to be duplicated and bumping the major version and creating an UPGRADING.md that explains what to do. It could also restore support for JDK 1.8. Hint ... hint :)

I will prototype that

Why should the java client be any different from JS or Python client? It shouldn't.

Agreed

If code duplication is to be had, so what?

It would be nice to avoid it. But, I think clarity to the end-users is more valuable. So I think code duplication is a good trade-off here.

I would merge a PR that removes the dependency by duplicating whatever code needs to be duplicated and bumping the major version and creating an UPGRADING.md that explains what to do. It could also restore support for JDK 1.8. Hint ... hint :)

I think a migration path could be helpful there. That is opensearch-java 2.x adds the duplicated code as a new transport. But it retains the old dependency as an optional transport. Then in the major version 3.x there is no option to use the old dependency.

@dlvenable what is a scenario in which someone benefits from keeping the old dependency as an optional transport?

@wbeckler , I only think it is useful for a smoother migration.
Minor release (say 2.2) - Adds the new transport and deprecates the old
Major release (say 3.0) - Removes the old transport.
A client developer who uses the library has an opportunity in 2.2 to make these changes before they are removed. They see the deprecation and they have some time to migrate. The alternative is that they want to update to 3.0 and suddenly they have to make the change then.

Why does removing the dependency affect a user of opensearch-java when we're duplicating the old transport? Is there a use of the client that depends on the source of the transport dependency? I'm asking because I'd like to find a way to remove the core dependency and avoid the major version bump.

+1 @wbeckler if there's no end-user change we can avoid the major version increment, it looks like we're not even changing the namespace

Why does removing the dependency affect a user of opensearch-java when we're duplicating the old transport?

@dblock @wbeckler this is a tricky question: we could duplicate old transport as-is, which basically means we would have at least 2 RestClient implementations, but we technically do not need to do that (see fe https://github.com/opensearch-project/opensearch-java/pull/281) and probably should not do that.

Is there a use of the client that depends on the source of the transport dependency?

We could keep the possibility to use RestClient from core (and not breaking the existing users) by making the rest-client dependency optional in 3.0.0 and still supporting it but not requiring.

I would pick restoring JDK 8 support over not doing a major version increment. I say @reta you should decide what's best and let us contribute thoughts. I think I'd merge any ready version of #281, including with breaking changes (and a major version increment).

I also opened https://github.com/opensearch-project/OpenSearch/issues/5424. Would love your comments @reta and @dlvenable!

I also opened opensearch-project/OpenSearch#5424. Would love your comments @reta and @dlvenable!

It makes sense to me: if we drop dependency on core, the plugins may migrate to opensearch-java as well, as such RestClient could be dropped at some point. Regarding the suggested plan, it might be a bit aggressive (add a note https://github.com/opensearch-project/OpenSearch/issues/5424#issuecomment-1334412578)


ARROW-10503: Uriparser will not compile using Intel compiler

At some point, Intel changed __force_inline to __forceinline. Arrow does not compile without this. Changes the header file to redefine the macro from __force_inline to __forceinline. Source: here

Patch for upstream: https://github.com/uriparser/uriparser/pull/93

Thank you @jensenrichardson !


[KED-2927] Fix versioning tracking datasets

Description
The tracking datasets have been setup to be versioned by default. This was done by setting the version to Version(None, None) in the __init__ of each dataset.
However, we found out that the version timestamp given to these datasets is different from the regularly versioned datasets and thus also different from the session ID. This is problematic because for the experiment tracking feature we need to be able to group all data used and generated during a specific kedro run.
Development notes
The reason the version timestamp on a tracking dataset is different from the other versioned datasets is because of the following:

When the catalog gets registered through the registration hook, DataCatalog.from_catalog() gets called.
This then sets a save_version 💾
Then AbstractDataSet.from_config() is called to generate a proper dataset object from the catalog configuration in parse_dataset_definition.
If we are dealing with a regularly versioned dataset, which has been enabled for versioning that entry would look like this in the catalog config:

my_file:
     type: pandas.CSVDataSet
     filepath: /Users/blabla/file.csv
     versioned: True

parse_dataset_definition will check whether the versioned key is found in the config and if that's the case it will then set the timestamp for the load and save version as Version(load_version, save_version) with the save_version coming from step 2 💾
However, if this key does not exist which is the case for the tracking datasets the version will remain unset.
🧪  For the tracking dataset the version will only be resolved when save is called and the savepath is determined.
This will trigger resolve_save_version and ultimately _fetch_latest_save_version gets called which creates a new save timestamp.
This timestamp is generated later on in the process and thus results in the timestamp on a tracking dataset being different from regularly versioned datasets.

Solution
The proposed solution here is to have an additional check in parse_dataset_definition for tracking datasets that will then set the timestamp for the load and save version as Version(load_version, save_version) with the save_version coming from step 2 💾   just like for the regular versioned datasets.
Checklist

[X] Read the contributing guidelines
[ ] Opened this PR as a 'Draft Pull Request' if it is work-in-progress
[ ] Updated the documentation to reflect the code changes
[x] Added a description of this change and added my name to the list of supporting contributions in the RELEASE.md file
[X] Added tests to cover my changes

Notice

[X] I acknowledge and agree that, by checking this box and clicking "Submit Pull Request":

I submit this contribution under the Apache 2.0 license and represent that I am entitled to do so on behalf of myself, my employer, or relevant third parties, as applicable.

I certify that (a) this contribution is my original creation and / or (b) to the extent it is not my original creation, I am authorised to submit this contribution on behalf of the original creator(s) or their licensees.

I certify that the use of this contribution as authorised by the Apache 2.0 license does not violate the intellectual property rights of anyone else.

@MerelTheisenQB Hey, this is the POC that illustrated the idea @lorenabalan mentioned: https://github.com/quantumblacklabs/kedro/pull/1019. The idea is we can set a versioned property on the dataset class and just grab it if it's available.


Adjustments related to key indexes

After the user logs in and the client connects, the client should always invoke client.syncAllKeyIndexes() by default. Otherwise it's going to create confusion for new users when they run ldpos-commander on a new machine for the first time.

We can remove all the indexes commands from ldpos-commander. The level of control that they provide is not necessary for this use case. Synching the key index automatically is guaranteed to be secure because client.syncAllKeyIndexes() verifies the index automatically.

@jondubois added in commit b23a10c6080a90604df40e4b2566b090f70c54a1


Postgres: Collections list page in admin UI cannot fetch data in prod but can in local

Link to reproduction
No response
Describe the Bug
I have three collections - media, questions and users.
My questions collection is flat and has a blocks field that has 3 blocks only (media upload, richtext and text block for video url)
I have also turned on drafts and versioning on questions collection and use the postgres adapater.
On local everything works perfectly but as soon as I deploy to prod and visit the admin UI I see the following:
The API routes work ok however.
"error: column _questions_v__blocks_contentBlock._uuid does not exist"

To Reproduce
import { CollectionConfig } from "payload/types";
import { Media } from "../blocks/Media";
import { Content } from "../blocks/Content";
import { Video } from "../blocks/Video";
import { slugField } from "../fields/slug";
import { populateAuthors } from "../hooks/populateAuthors";
import { populatePublishedDate } from "../hooks/populatePublishedDate";
import { docOwner } from "../fields/docOwner";

const Questions: CollectionConfig = {
  slug: "questions",
  hooks: {
    afterRead: [populateAuthors],
    beforeChange: [populatePublishedDate],
  },
  versions: {
    drafts: true,
  },
  admin: {
    useAsTitle: "question",
    defaultColumns: ["question", "icon", "difficulty", "category"],
  },
  fields: [
    {
      name: "icon",
      label: "Icon Number",
      type: "number",
    },
    {
      name: "question",
      label: "Question",
      type: "text",
      required: true,
    },
    {
      name: "difficulty",
      label: "Difficulty",
      type: "select",
      admin: {
        position: "sidebar",
      },
      options: [
        { label: "🟢 Easy", value: "Easy" },
        { label: "🟡 Medium", value: "Medium" },
        { label: "🔴 Hard", value: "Hard" },
      ],
      required: true,
    },
    {
      name: "category",
      label: "Category",
      type: "select",
      admin: {
        position: "sidebar",
      },
      options: [
        {
          label: "Option 1",
          value: "option1",
        },
      ....
      ],
      required: true,
    },
    {
      name: "publishedOn",
      admin: {
        date: {
          pickerAppearance: "dayAndTime",
        },
        position: "sidebar",
      },
      hooks: {
        beforeChange: [
          ({ siblingData, value }) => {
            if (siblingData._status === "published" && !value) {
              return new Date();
            }
            return value;
          },
        ],
      },
      type: "date",
    },
    ...docOwner(),
    slugField("question"),
    {
      name: "criteria",
      label: "Criteria",
      type: "textarea",
      required: true,
    },
    {
      name: "supportingInfo",
      label: "Supplementary Information",
      type: "blocks",
      blocks: [Content, Media, Video],
    },
  ],
};

export default Questions;

Content Block
import { Block } from "payload/types";

export const Content: Block = {
  fields: [
    {
      name: "content",
      label: "Content",
      type: "richText",
    },
  ],
  slug: "contentBlock",
};

Media Block
import { Block } from "payload/types";

export const Media: Block = {
  fields: [
    {
      name: "media",
      relationTo: "media",
      required: true,
      type: "upload",
    },
  ],
  slug: "mediaBlock",
};

Video Block
import { Block } from "payload/types";

export const Video: Block = {
  slug: "videoBlock",
  fields: [
    {
      name: "title",
      label: "Title",
      type: "text",
      required: true,
    },
    {
      name: "url",
      label: "URL",
      type: "text",
    },
  ],
};

Steps to reproduce:

Yarn dev locally to migrate prod db
Deploy to prod
Access the admin UI on prod and locally. The local will load the collection list page but prod will show you this

Payload Version
^2.0.0
Adapters and Plugins
"@google-cloud/storage": "^7.3.0",     "@payloadcms/bundler-webpack": "^1.0.0",     "@payloadcms/db-postgres": "^0.x",     "@payloadcms/plugin-cloud": "^2.0.0",     "@payloadcms/plugin-cloud-storage": "^1.1.1",     "@payloadcms/richtext-slate": "^1.0.0",     "cross-env": "^7.0.3",     "dotenv": "^8.2.0",     "express": "^4.17.1",     "payload": "^2.0.0"

@Ali-Parandeh What version of payload shows up in your lock file?

@Ali-Parandeh Can you try running payload migrate:create on your prod instance?

@JarrodMFlesch Sorry I was running against deadlines so I dropped the Postgres adapter in favour of mongoose adapter and switched to mongodb instead. It is all now working as expected.

The postgres adapter is not stable enough for schema changes and can run locally but not in prod sometimes. Payload migrate create might have worked not sure

"@google-cloud/storage": "^7.3.0",     "@payloadcms/bundler-webpack": "^1.0.0",     "@payloadcms/db-postgres": "^0.x",     "@payloadcms/plugin-cloud": "^2.0.0",     "@payloadcms/plugin-cloud-storage": "^1.1.1",     "@payloadcms/richtext-slate": "^1.0.0",     "cross-env": "^7.0.3",     "dotenv": "^8.2.0",     "express": "^4.17.1",     "payload": "^2.0.0"

@Ali-Parandeh where are you deploying this at? Payload Cloud?

This is on Google cloud run on Google cloud as a docker container

@Ali-Parandeh I was able to load this locally also, I think the migrations might have worked. I am going to close this for now since you switched from the beta db-postgres adapter to the stable db-mongodb adapter for now. If you run into this in the future feel free to open a new issue. Thanks!

Hey all, I am pretty confident that this is related to your database and production, not having the required fields as indicated by your schema. The UUID field is for version enabled collections and came with a postgres bug fix a couple months ago. Try and run migrations against prod to see if that fixes it.


gitignore: clean up

This removes several outdated or unnecessary ignore patterns.
Especially those that match random words such as 'local' or 'core',
which repeatedly caused weird behavior that's hard to debug, e.g.
invisble vendored files.
I'd even argue for removing all the editor files as these patterns belong into a users own .gitignore – we cannot add ignores for all possible editors.

:+1:


SARAALERT-521: Implement writing of Lab Results in the API

Description
Jira Ticket: SARAALERT-521
This PR adds support for creating and updating (POST, PUT, PATCH) Lab Results via the API using a FHIR Observation Resource. It also adds support for writing Patients alongside their Lab Results in one single atomic transaction, via a FHIR Bundle.
Important Changes
api_controller.rb

Add the ability to update a Lab via the API
Add the ability to create a Lab via the API
Add a transaction method which can be used to create a Bundle of Patients and Observations. The Observations must point to the Patients in the same Bundle, so you can't add a bunch of Observations for Patients that already exist. This should make it possible to enroll a Patient alongside a positive lab into the Isolation workflow, without a SOD. The transaction should work atomically, so if any resource has an issue, nothing is persisted.
Some refactoring of how Patients are created and saved so we could do it in multiple places.

fhir_helper.rb

Update the laboratory_as_fhir method to correspond with how we want to represent Labs now. In theory this is a breaking change, but currently no one is reading labs via the API.
Add a laboratory_from_fhir method.
Add helper functions for reading and generating a Bundle.

isolation_symptom_onset_validator.rb

Add new validator to run in the :api context which replaces the previous validation requiring SOD in Isolation. Now if a record is in Isolation, either SOD must be present, or the Patient must have a positive Lab Result with a specimen collection date.

laboratory.rb

Add methods for converting between FHIR codes and Sara Alert values, and back again.

patient.rb

Update validation to use the new validator in isolation_symptom_onset_validator.rb

Testing

Here is an example of a Bundle that you can use as a starter for testing

Notes

This PR may be dependent on https://github.com/SaraAlert/SaraAlert/pull/861. I actually don't see why they would be dependent, but I think it is still best to wait to merge this until after #861 to make sure this works well with that.
If it would be easier to review, I can split this up into two PRs, one to add the writing Lab Results, and one to add the transactions. It wouldn't be too hard, the commits are independent enough.
I still need to document this, but it's quite a bit to document, so and I think the code can be reviewed while I document.
As @tstrass has pointed out before, the ApiController needs a refactor, it was tempting to do it here, but I'd rather do it separately so that this PR isn't impossible to review.

Checklists
Submitter:

[x] This PR describes why these changes were made.
[x] This PR is into the correct branch.
[x] This PR includes the correct JIRA ticket reference.
[x] Code diff has been reviewed (it does not contain: additional white space, not applicable code changes, debug statements, etc.)
[x] If UI changes have been made, Chrome Dev Tools Lighthouse accessibility test has been executed to ensure no 508 issues were introduced.
[x] Tests are included and test edge cases
[x] Tests have been run locally and pass (remember to update Gemfile when applicable)
[x] Test fixtures updated and documented as necessary

@tstrass

[ ] Code is maintainable and reusable, reuses existing code and infrastructure where appropriate, and accomplishes the task’s purpose
[x] The tests appropriately test the new code, including edge cases
[ ] You have tried to break the code
[ ] If applicable, you have tested changes against a large database, and considered possible performance regressions

@holmesie

[ ] 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
[ ] If applicable, you have tested changes against a large database, and considered possible performance regressions

@timwongj

[x] Code is maintainable and reusable, reuses existing code and infrastructure where appropriate, and accomplishes the task’s purpose
[x] The tests appropriately test the new code, including edge cases
[x] You have tried to break the code
[x] If applicable, you have tested changes against a large database, and considered possible performance regressions

Don't merge until https://github.com/SaraAlert/SaraAlert/pull/861 is merged

Don't merge until #861 is merged

Since #861 is merged now, I've just rebased and this should be good to go.


Invisible text when using the dark theme

The Endgame settings tab does not completely conform to the dark theme and you end up with white text on a white backdrop.  Also the other tabs have bits which are black that should be white considering they are on a dark grey backdrop.

I'll see what I can do with this, soon

fixed with cfc469f5


feat: SPT-420 - Add base-path input parameter

We are working on allowing to have auto versioned libraries on a repo (for example a library repo).
In order to have a version out of root folder we added the base-path input parameter to indicate where need to be placed the action artifacts.

Execution sample whiteout base-path parameter

From version-and-release job on shared-oauht-lib-dotnet workflow line 91
commit generated 3ffc0138a3d2d7ad9974152bf1ba17dbce8e94fb

Execution sample with base-path parameter

From version-pack-and-push-package action on package-version-and-release-notes step line 46

commit generated 091da15f73078ac4376af43b3ffa7dc951cbfe34

@emartinezohpen you have the commit links swapped, the one in the without base-path is the one with base-path, and the other way around

fixed  commit links swapped

already implemented with a different approach


Commands seem to be modifying a different Routing table than the AZ Portal.

When you go into a virtual hub routing information the routes and tables shown there are different from the ones shown and returned by this command.
az network vhub route-table list
This tables don't seem to affect the branches or the Vnets they are attached too.
Is this a preview command for future functionality that is not yet implemented in the portal?

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

ID: 6b1cb32b-6079-1c88-b236-6c0cd3e45ed9
Version Independent ID: 2864c2f3-4739-729a-5827-ac3bb5e65f9e
Content: az network vhub route-table
Content Source: latest/docs-ref-autogen/ext/virtual-wan/network/vhub/route-table.yml
GitHub Login: @rloutlaw
Microsoft Alias: routlaw

@MyronFanQiu Please take a look.

@annerajb Hello. currently, routes and route-table are different commands right now. Based on my knowledge, it should call route-v2. I'm not sure about the portal. @dedhar Could you please take a look at this issue and answer the question?

@annerajb Hi. Do you still have question about these commands? I'm not sure about portal, you can try original command az network vhub route to see whether it modify the same route table with Azure Portal or not. But from my perspective view, it should be different.

I ended up opening a support request and finding out a different command
yes.
On Tue, Mar 3, 2020, 2:31 AM MyronFanQiu<EMAIL_ADDRESS>wrote:

@annerajb https://github.com/annerajb Hi. Do you still have question
about these commands? I'm not sure about portal, you can try original
command az network vhub route to see whether it modify the same route
table with Azure Portal or not. But from my perspective view, it should be
different.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/Azure/azure-cli-extensions/issues/1169?email_source=notifications&email_token=AAGXQXSVPMRRMJ3LXFMA5NTRFSW45A5CNFSM4KBW4ODKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOENSMXHQ#issuecomment-593808286,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAGXQXSQ75SK4ZUBYXQJYLLRFSW45ANCNFSM4KBW4ODA
.

@annerajb Good to know. 😄


Publish

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

File
Status
Preview URL
Details

docs/concepts/privacy-and-security.md
:white_check_mark:Succeeded
View

docs/develop/create-sso-office-add-ins-aspnet.md
:white_check_mark:Succeeded
View

docs/develop/create-sso-office-add-ins-nodejs.md
:white_check_mark:Succeeded
View

docs/outlook/privacy-and-security.md
:white_check_mark:Succeeded
View

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 in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel


DRAFT JAMES-2586 Implement PostgresQuotaLimitDAO And PerUserMaxQuotaManager

10:09:35,701 [ERROR] Failed to execute goal org.apache.maven.plugins:maven-checkstyle-plugin:3.3.0:check (check-style) on project apache-james-backends-cassandra: You have 2 Checkstyle violations. -> [Help 1]


Hidden charakters in Obsidian in downloaded .md file

Hello
The download works fine. I can open the .md file in Atom an there is nothing wrong with it. When it is opened in Obsidian the Preview-Mode works absoluty fine, but when I switch to the Edit-Mode suddenly there are everywhere small red dots. When I am hovering over them \uad appears. Can anyone help me?
macOS 10.15.7
Firefox 85.0
Markdown 2.4.1
Obsidian v0.10.13
Atom 1.54.0

Does this happen with every site you clip, or just a few? Could you provide me a link to a site that is producing this behaviour?
I think I have an idea what the issue might be, but want to check out an example so I know any potential fixes work

I clipped some more and it just happens to this one
https://epaper.zeit.de/article/3a9b64701a8198e8fc07d86aabc5cdba858311babe8471f76f8207e13bdf90be
Articles from the normal homepage are working fine (e.g. https://www.zeit.de/gesellschaft/2021-02/klassismus-soziale-gruppen-soziologie-literatur-gesellschaft)

The Add-on can not pull the data of this site www.dezeen.com e.g. https://www.dezeen.com/2018/07/04/personal-data-could-be-a-vital-tool-in-designing-our-cities/


Several questions regarding CCF estimation results

Hi,
I’ve been benchmarking SVClone on our in-house cell line purity ladders, where each tumor sample is an in-silico mix of tumor and normal (i.e., known purity/ploidy). I’ve been reviewing the results and have a few questions.
For SVs I’m using breakpoints that were called by two or more callers (svaba/manta/lumpy), and for SNVs I’m using calls that were called by two or more callers (mutect2/strelka2/lancet). For CNVs I used ASCAT, run with known purities/ploidies.
1. Oscillating CCF estimates
Looking at how individual variant CCF estimates change with purity, I noticed an oscillating pattern in a subset of variants. This seems to occur for variants with CCF estimates farther away for the cluster mean. I’m seeing this for both SVs and SNVs, visualized below. Each line traces a given variant call’s CCF estimate across each purity. For SVs, I’m using the CCF estimate from breakend 1. All CCFs are capped at 1.

Have you seen this before? Is this just an artifact of the clustering algorithm?
2. Collapse of clusters
For the same cell line, the number of clusters as estimated by SVClone collapses from 4 clusters to 1 halfway down the purity ladder. While I assume you’d expect a diminishing ability to resolve clusters as purity decreases, this sudden decrease suggests that something else is going wrong. The boxplot below shows CCFs for each purity level, split by cluster (not capped at 1 here).

My first thought was that this was simply due to a decrease in variant calls at lower purity levels. While you do see some variant dropout, it doesn’t seem to be enough to be the cause of the issue. Below is a plot similar to those in (1), but including variants that were only called in a subset of purities.

Have you encountered something like this before? Do you have any idea what could be happening here?
3. Inconsistent cluster assignments
Variants will change cluster assignment between purities, even when the same number of clusters is present in both purity levels. Below is a similar plot to the line plot in (2), except cluster mean CCF is used instead of individual breakend CCF. The numbers in the plot represent the number of variants present at a given cluster/purity combination. The collapse in clusters mentioned in (2) is visible here as well. Is this expected?

Thanks for your continued help.
-Will

Re 1: I'm guessing something going on with the actual read counts and copy number.  Cluster assignments mainly affect instance CCFs via multiplicity calls. The ups and downs seem to be 1 multiplicity difference.
Re 2: For collapse of clusters, I think this could have something to do with variant read counts being too high in those low purity samples. You can see instance CCFs as high as 5 in the first box and strong super clonal clusters in 1st four boxes. We dont expect a gradual drop of number of clusters when purity is low. During fitting, maximum allowed cluster centre is fixed at 1. If there are a lot of instance variants having CCF larger than 1, then at some point the fitting will have difficulty in calling clusters < 1. This is an intended effect to spot out problematic samples.
R3: Generally speaking, changing assignment could happen. Another measurement we use for sanity is instance CCF * multiplicity. In datasets where you know the truth, a good agreement usually indicates the fitting is working as intended.
Finally, you trace plots show a trend of more lower instance CCF estimates as purity increases, which fits our expectation.
Hope this helps.


New format

No idea if you still develop this, but there's a new encryption scheme for header format 2. You'll need this:
	(hdr, length, key) = struct.unpack("!4sii", content[:4] + "\x00" + content[4:7] + "\x00\x00\x00" + content[7:8])
	print "Hdr: %s\nLength: %d\nKey: %d" % (hdr, length, key)

From here https://github.com/smx-smx/asuswrt-rt/blob/master/apps/public/nvram/nvram.c , starts line 96.

And this for the encryption
header = struct.pack("!4s3s1s", "HDR2", struct.pack("!i", length)[-3:], struct.pack("!i", key)[-1:])

Thanks for the information.
This was more or less a one-off tool I put on GitHub just in case I need it again. I don't plan to fix it myself, but if you send me a pull request with a link to an example romfile, I'll be happy to merge it.


[STS] fix for sts StagefrightTest failures

13 testcases are passed out of 19 failures
with this changes.
Tracked-On: OAM-102797
Signed-off-by: Vasoya,Nikhilx<EMAIL_ADDRESS>

@TianmiChen<EMAIL_ADDRESS>kindly review the PR.


Video buffer requied

when video player is playing rtsp stream.
I think, there is no video buffer. so sometimes the video was delayed.
but when I play with VLC Player, there is no delayed, it seems to be buffer exists.
VLC player has a minimum buffer about 1second. I think.
so if you can add video buffer on your lib,
It will prevent the video from being delayed.

Hi,
Do you mean stuttering issues?
l plan to add some sample buffering logic.

yes, right.

ok, thank you for your feedback

@BogdanovKirill
There is a need for sample buffer but please don't force the need for them. I would somehow integrate it as an argument when creating the class.
Right now it works amazing without sample buffers and basically realtime on many of my cameras. Only a few it's having trouble without sample buffers.
I'd hate to gain more latency to the already working cameras if it's not needed. I know this would require a little more infra structuring out but would definitely solve some of the issues other libraries have with live playing,

@BogdanovKirill

Do you mean stuttering issues?
l plan to add some sample buffering logic.

Any progress on buffer? I think I will need it and plan to implement some simple solution it in a next few weeks. But may be you already have some beta version and it doesn't have sense for me to start?


LE Audio support in bluez-alsa

Feature description

Now I try add LE audio function in my buildroot system. I now use the pipewire, but I think it is too hard to use. Can bluealsa support LE Audio? Just like how it support the A2DP. Thank you for your great work.

Can bluealsa support LE Audio? Just like how it support the A2DP.

As for now, BlueALSA does not support LE Audio. I've been thinking about that, but there are other things to do and LE Audio is very low priority for me right now (it would be another big feature which will require maintenance, at least at the very early stage of integration with BlueALSA). But please stay tuned, one day it will for sure be added to this project.

@teddyxiong53 , May I know if you have any analysis on LE Audio development?
One general question, To make LE audio work, Do we need to update the kernel version?
Is there any kernel/driver requirement here?


Consume(out Message<TKey, TValue> message, int millisecondsTimeout) returns only a single message

Description
Consume(out Message<TKey, TValue> message, int millisecondsTimeout) returns only a single message. Is there any other method which we could use for consuming multiple messages() from a kafka topic
How to reproduce
Use Consume(out Message<TKey, TValue> message, int millisecondsTimeout) to consume messages from kafka. You end up getting only a single message. I am consuming from a topic which has billions of messages. What is the recommendation here.
Checklist
Please provide the following information:

[0.11.3 ] Confluent.Kafka nuget version:
[ ] Apache Kafka version:
[ ] Client configuration:
[ Windows 10] Operating system:
[ ] Provide logs (with "debug" : "..." as necessary in configuration)
[ ] Provide broker log excerpts
[ ] Critical issue

no, but note that behind the scenes, data is fetched from kafka in batches, it is just delivered to you via the api one message at a time.


test_linger[2-1-True]

https://travis-ci.org/opensistemas-hub/osbrain/jobs/341786321
https://travis-ci.org/opensistemas-hub/osbrain/jobs/341672572
https://travis-ci.org/Peque/osbrain/jobs/341815179
https://travis-ci.org/Peque/osbrain/jobs/342288465

I had to expand the timeout for these tests because they were more frequently failing on Windows. Let's see if that helps, I'll check back on this when the windows compatibility PR is merged.

Closing in favor of #252.

Reopening: https://travis-ci.org/opensistemas-hub/osbrain/jobs/347174040


Bp 2076

I found I cannot force-force PR opened by mergify. So I reopened this to subsume #2090

@sequencer I don't know why its failing, it seems like it's a DirectMissingImplementation error? https://github.com/lightbend/mima/blob/b20f56d9af7543ecbf7c78fe997ca0cf4e3b2a69/core/src/main/scala/com/typesafe/tools/mima/lib/analyze/method/MethodChecker.scala#L105
Does this need to be back ported?


scale_y_log10: hidden segments

Look at the following example:
from lets_plot import *
LetsPlot.setup_html()
n = 10
x = list(range(n))
y = [2**d for d in range(n)]
ggplot({'x': x, 'y': y}) + \
    geom_segment(aes(x='x', y=[0]*n, xend='x', yend='y')) + \
    geom_point(aes('x', 'y')) + \
    scale_y_log10()

It produces an image without segments:

Segments appear if we remove scale_y_log10() layer:

I suspect we loose segments because their lower Y (=0) becomes -Infinity on log10 scale.

Later I reverted the 'fix': excluded 0.0 from log10 domain. So, the code here will produce plot without segments again(.
The workaround would be:
a) define segment 'y' as constant: geom_segment(aes(x='x', xend='x', yend='y'), y=0)
b) make 'y'-s not 0 but just small: geom_segment(aes(x='x', y=[1e-3]*n, xend='x', yend='y'))


Fix #492: Add the editor.scrollBar.visible setting to control visibility of buffer scrollbar

Issue: There is no way to disable the buffer scroll bar, which some users may find distracting or unnecessary.
Fix: Add a setting for editor.scrollBar.visible that can be modified in the config.js:

Fixes #492


add the 'drag' or 'drop' event to the 'ion-item' in ionic4

Support Question
Please do not submit support requests or "How to" questions here. Instead, please use one of these channels: https://forum.ionicframework.com/ or http://ionicworldwide.herokuapp.com/

i used the dragdrop directive inject the :
<ion-item (click)="onClick(image)" lines="none" class="list-item" *ngFor="let image of imageArray" dragdrop>
DragDropDirective:
ngOnInit() {
this.el.nativeElement.addEventListener('drag', function(e) {
console.log(e);
});
}
why the can't fire the 'drag' event?


Better Plugin Documentation

Howdy, Would you add more documentation as to how to add a trumbowyg specific plugin. It seems heavily read existing plugins and hope for the best.
(maybe even give examples of building one using the concepts for an existing plugin)

I work on writing more doc asap, but that was not easy before v2 to make a plugin, I have completly rewrote plugins implementation.
So now, I can work on a clean doc.
This issue is a duplicate of #205 which have the exact same title, so I close this issue.


first draft support functions

Part of #215 and #226
Overview
Resolve #226 by implementing the necessary convenience functions.
Discussion
The full context for this work is outlined in #215.
Testing
I ran the new code in succession to my seedling end-to-end run, and it produced identical results.

:books: Documentation preview :books:: https://garden-ai--227.org.readthedocs.build/en/227/

Codecov Report

Merging #227 (cfb0f5b) into main (5e9f23e) will decrease coverage by 0.05%.
The diff coverage is 17.07%.

: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             @@
##             main     #227      +/-   ##
==========================================
- Coverage   79.60%   79.56%   -0.05%
==========================================
  Files          46       47       +1
  Lines        2913     3205     +292
==========================================
+ Hits         2319     2550     +231
- Misses        594      655      +61

Files Changed
Coverage Δ

garden_ai/client.py
62.98% <17.07%> (-3.87%)
:arrow_down:

... and 3 files with indirect coverage changes