Triton Server Example does not work

Hi,
I run into this issue when following the triton inference tutorial with vllm here:
https://github.com/triton-inference-server/tutorials/tree/main/Quick_Deploy/vLLM
I'm running on a AWS Ubuntu machine with this default image on an A10 GPU.
UNAVAILABLE: Internal: AssertionError: vLLM Triton backend must be configured to use decoupled model transaction policy
Thank you

Ah on further digging:
https://github.com/triton-inference-server/vllm_backend/pull/20
It's because this autocomplete isn't actually being called so config.pbtxt is not set up correctly.
So using the old config.pbtxt seems to have fixed this

The auto-complete feature is currently available only on triton's main branch that tracks development to a future release. The auto-complete functionality will be available with 23.11 release which is scheduled to be released by end of the month.
Thanks for your patience.

The auto-complete feature is currently available only on triton's main branch that tracks development to a future release. The auto-complete functionality will be available with 23.11 release which is scheduled to be released by end of the month. Thanks for your patience.

Thanks @tanmayv25 for your work. It is truly amazing and blazingly fast! Is it normal that all Mistral-based models when ran on Trinton-AWQ server, run out of memory? It has happened in all my experiments but not for any Llama-based models.

@guialfaro053 Which GPU are you using? What is gpu_memory_utilization setting in your model.json?
We have been able to run Mistral models within Triton + vLLM backend.

@tanmayv25 I am using an RTX4090 24gb and my gpu_memory_utilization is 0.5. But even if i change that parameter to 0.9 or 0.1, the same allocation error occurs so maybe my installation process is wrong? I am using the 23.10 image and then I update I pip install vllm==0.2.2 for Mistral support.

@tanmayv25 Yes, outside Triton, I have no issues.

That is directly in a python script?

I basically just run the server in the entrypoints/openai directory and everything is okay, but when I follow the Triton-vLLM backend tutorial I get OOM.


Use ruff for formatting the code

This gives us a bit of alignment with BCI-dockerfile-generator. Like if both would be part of the same project. sort of.

this breaks the isort part of it, which in ruff is in linting, so we need to switch linting as well.


How to pass GEOMETRYCOLLECtion from sql

I am working on Leaflet-WKT.js as provided on Github
https://github.com/arthur-e/Wicket/blob/master/wicket-leaflet.js
I am trying to pass geometrycollection WKT MultiPolygon collection.
Could anyone please help me to resolve this issue?
Attachment
https://drive.google.com/file/d/1wke9XbsTYYb7OQEW8Y4blJvIGbW5LgkG/view?usp=drivesdk
Console log
The geometrycollection WKT type is not yet supported.

This repo is the rust programming language crate for parsing WKTs; not wicket.   Your error seems to be from wicket.js;  please report and inquire it in that repo.


Add options to ignore specific tags

http://<url>/<org>/<repo>?nl=dependencies

The key could be something like nl. Will be additive, as in even if there is a label that the user has specified, if it has a label specified via nl we drop the item.
Current usecase is to avoid dependabot pull requests.

Maybe even -l instead of nl for key.


Performance regressions

I was recently profiling energy usage and have noticed that there are some significant performance regressions in super_editor.
A while ago https://github.com/superlistapp/super_editor/pull/285 got merged to fix blinking cursor performance, but later it was replaced by BlinkController, which uses a Ticker. Unlike custom timer Ticker on desktop runs at constant 60 fps (each tick requesting new frame), causing unnecessary repaints given that cursor blinking interval is much longer.
This is further exacerbated by the fact that super editor forces repaint even if cursor is not visible. This is caused by DocumentCaretOverlay calling startBlinking on BlinkController unconditionally in initState. Regardless of whether there is any selection or whether the editor is focused).
Initially I assumed that removing startBlinking from initState would fix this, but that's not the case because DocumentCaretOverlay state gets torn down and re-inflated every time content layer gets rebuilt (i.e. on selection change). Which seems like a strange behavior caused by ContentLayers. Every time layers are rebuilt, the child elements are inflated, which throws away the previous element state recreating the element tree from scratch. This seems at odds with how rest of Flutter works, where instead of always calling inflate, the element would normally call updateChild which allows preserving state of elements in tree if widgets are compatible.

Cc @matthew-carroll

@knopp would you be interested in pairing on this? The ContentLayers stuff is pretty hairy. It's definitely trying to do stuff that Flutter doesn't want to do, but that's because, in my opinion, Flutter is missing some important capabilities around build invalidation.
Here's an issue I filed with Flutter: https://github.com/flutter/flutter/issues/123305
I've found it pretty difficult to track what I'm supposed to do from an Element and RenderObject perspective when implementing non-trivial multi-child behavior. Maybe you can help me find places for improvements in ContentLayers?

Also @knopp part of this rebuild behavior is probably on purpose. The Element getting recreated might not be intentional, but rebuilding the layers whenever the content changes is intentional. The purpose of the content layers is to size and position widgets based on the content layout. For example, a caret positioned within some text. If the text layout changes, then the caret position should also change, which requires a rebuild of the caret widget.

I tried an adjustment here: https://github.com/superlistapp/super_editor/pull/1239
I added a test that expects the overlay and underlay Element to remain the same across a rebuild.
Unfortunately, a few caret position tests broke.


Autonumber tasks and/or expose already added ones

Tasks don't seem to show up if I don't add an "index" value, but the tool itself could add an index and thus aleviate the need to account for the tasks added and coming up with a new index. A function exposing alrady added taks (bars) would also be apreciated and in turn this could be used to come up with a new index. Thanks.

I originally had the indexes autoincrement, but the way of specifying indexes allows you to set up tasks out of order (you might want something to be the 2nd listed task, but in your workflow, it is instantiated later, for some reason). If we had it auto-increment, we'd have to decide whether to autoincrement from the last index, or auto-fill any missing rows. I supposed we could just append to the end.
Adding a way to get tasks based on indices and/or indexes based on task name is no problem.

Implemented in latest commit: f6b1d377bbd576045426387040d4025a05723005
Published on npm as v3.2.0


A slight refactor

Jesus work on your code lol, this is bad, so many weird variable names, squished code and in general messiness, please don't make me suffer

Bahahha I was actually hoping to fix up the code myself after my surgery, thanks for this though! There were some things in here I didn’t catch right away, thanks for the contribution!


Adding license and auto-formating

I suggest adding a license file, and also maybe a .luacheckrc or stylua.toml file to keep the source code format consistent (I think you can have luacheck or stylua run in Github action on push). This will make it easier for potential future contributors to scan through the code base.
Thanks for this amazing plugin btw.

I want to re-distribute a fork of this plugin but I'm not sure if I'm legally allowed to, because there isn't a license. Could you add a license please?

I want to re-distribute a fork of this plugin but I'm not sure if I'm legally allowed to, because there isn't a license. Could you add a license please?

Are you seriously think about that in regards to some minor plugin for some text editor? Wow, maybe I should take my work more seriously.. I mean, I never thought this would be helpful or even of interest to anyone
I added a license, no problem, you may re-distribute whatever derivatives of this code any way you like

Thanks!


Add support for pausing the application

The shell will not be interactive while it's paused, but at least it will not close the connection.
Also the buildozer.spec has been fixed for newer buildozer versions.

This is fixed in the latest changes, thanks!


Update Config “general”

Automatically generated by Netlify CMS

👷 Deploy Preview for superb-sawine-40c5c7 processing.

Name
Link

🔨 Latest commit
94e0cc859521164d515080a6a639e628ad7f98b6

🔍 Latest deploy log
https://app.netlify.com/sites/superb-sawine-40c5c7/deploys/6294688a6a3a2b000964f35e


Initial Contour Controller Implementation

Moves main() to cmd/operator pkg. cmd/operator/start.go : Exposes a flag to set the Contour image used by the operator.
Singularizes the controller pkg.
controller/contour/controller.go: Initial controller implementation that watches the contour CRD and a) finalizes the resource and b) creates the contour namespace if it does not exist.
util/retryableerror/retryableerror.go: Utility function for retrying reconciliation errors.
util/slice/slice.go: Utility function to check if a given slice of strings contains the provided string.

Signed-off-by: Daneyon Hansen<EMAIL_ADDRESS>

/assign @jpeach
/cc @youngnick @Miciah

@jpeach I believe commit e83f7b1 addresses all your feedback except https://github.com/projectcontour/contour-operator/pull/10#discussion_r480759628. I've never implemented subtests like you mention. Do you have a Contour example to share or would you mind implementing after this merges so I can learn?


Skip option for preinstall hook

Cases like https://github.com/prisma/prisma/issues/2041 show that we need a way for people to skip the preinstall if it somehow breaks on their system. Super special case, but there is no other way for them to fix this (besides manually creating a folder... meh).
Can one actually give input to preinstall hooks? If not, probably and ENV var is the easiest way.

This is also important for the weird issue we see with yarn deleting the engine sometimes.

That is not connected, that is run via postinstall and needs to be fixed properly.


Add FullHeightBox-component

Do we want components like these or is there another way to achieve simplifying full-height boxes? Or is the current way good enough?

:+1:
I'd add page into the name, as this can be used at page level only. (Or do we have a mor specific name for this?)
PageFullHeight

PageFullHeight
Since it is a Mui-Box I would keep Box in the name. But another solution just came into my mind, we could simply add a fullheight-prop to MainContent instead creating an additional component. How about that?

Since it is a Mui-Box I would keep Box in the name. But another solution just came into my mind, we could simply add a fullheight-prop to MainContent instead of creating an additional component. How about that?

I'd also prefer the MainContent prop over a separate component, maybe it even makes sense to always make MainContent span over the full height by default, with min-height.

height: calc(100vh - var(--comet-admin-master-layout-content-top-spacing))

As @johnnyomair mentioned internally using this height will not work on pages that render a Toolbar above the MainContent, as the Toolbar-height is not accounted for here.
We could probably change the way we use the Toolbar by always rendering it inside MainContent and add custom styling for Toolbar inside MainContent to prevent spacing around the Toolbar.


Allowed domains

This is a very useful application. Thank you.
For my use case I want to only have the proxy work for a list of allowed domains. Reason being I have work to do on each site to make sure all assets are sourced/served securely. If the proxy runs for all, I'm no better off until this work is done since browsers will show insecure warnings.
I looked through the flags, and I can exclude domains, but not allow. Given the number of sites, passing as an exclude argument is difficult, so I'm proposing to implement a look up, possibly from a YAML file, so that the proxy works only for the allowed domains in this file.
Some of the low level socket stuff in your application, I'm not familiar with, but I would like to implement this feature, so my question is, where would be a good place to branch to a YAML file lookup, so that lets-proxy does it's thing for allowed domains contained inside the file, but terminates its connection for anything where https is not to be used, or best case, does not even establish a connection, so the default browser behaviour is maintained.
Russian via Google translate.
Это очень полезное приложение. Спасибо.
Для моего случая использования я хочу иметь только прокси работу для списка разрешенных доменов. Причина в том, у меня есть работа, чтобы сделать на каждом сайте, чтобы убедиться, что все активы соерсед / служили надежно. Если прокси-сервер работает для всех, я не лучше, пока эта работа не будет сделано, так как браузеры будут показывать небезопасные предупреждения.
Я посмотрел через флаги, и я могу исключить домены, но не позволяют. Учитывая количество сайтов, передавая в качестве аргумента исключения сложно, поэтому я предлагаю реализовать смотреть вверх, возможно, из файла YAML, так что прокси-сервер работает только для разрешенных сайтов в этом файле.
Некоторые вещи низкого уровня сокета в вашем приложении, я не знаком, но я хотел бы реализовать эту функцию, поэтому мой вопрос, где будет хорошее место, чтобы расшириться к YAML поиска по файлу, так что lets- прокси делает это вещь для разрешенных доменов, содержащихся в файле, но прекращает свое соединение для чего-нибудь, где HTTPS не будет использоваться, лучшем случае даже не установить соединение, так что поведение браузера по умолчанию поддерживается.

Now that I've got logging running on a server handling TLS for a domain I think the entry point for a lookup to allowed domains is just after this line in main.go:638...
serverName := tlsConn.ConnectionState().ServerName

Would you agree and does such a feature (and my approach) sit with your plans for lets-proxy app.

It is very late to check domain.
TLS connection (and check certificate) completed in line 628:
err = tlsConn.Handshake()
Now domains checking in block:
	for _, checkDomain := range domainsToObtain {
		for _, re := range nonCertDomainsRegexps {
			if re.MatchString(domain) {
				logrus.Debugf("Ignore obtain cert for domain %v by regexp '%v'", DomainPresent(domain), re.String())
				continue forRegexpCheckDomain
			}
		}
		allowedByRegexp = append(allowedByRegexp, checkDomain)
	}
	logrus.Debugf("Allowed domains by regexp for '%v': '%v'", domainsToObtain, allowedByRegexp)

I have no decision about format of config file.
I can do whiltelist as command line argument separated by comma - similar to other options.
command line on 64bit linux near 2Megabytes
xargs --show-limits

line "Maximum length of command we could actually use ..."
2Mb allow to list very many domains. It is not very comfortable to read, but sometime later I will create common way to handle arguments parameters and config file.

Thanks for directing me to the earlier code, I will review it.
I considered a file useful since it could be appended without restarting - but I think I am being naive with this approach? The whitelist needs to be in memory whether from a file or command line argument, otherwise performance is not optimal. Given this, maybe command line argument is sufficient.

Perfomance is not problem in obtain certificate:
this code work only when need new certificate it is very rare (few thouthands per day on large shared hosting server).
The problem is save config simple for use.
Hmm. I think white list in separate file may be useful after allow config file too.
I will do it in 1-2 weeks.

Ok, I'll follow the issue. I just found --loglevel debug would have pointed me to the earlier access point for the domain - so sorry to trouble you with that one - I was using --loglevel info

I've had a go at this, and have code which seems to do the job.  To check domain early and avoid the Let's Encrypt certificate request process, I chose to wrap the getCertificate function with my own middleware style function.
In main.go:609
// handle ssl
tlsConfig := tls.Config{
	GetCertificate: getCertificateMiddleware,
}

I've added a new file whitelist.go
package main

import (
	"crypto/tls"
	"errors"
	"io/ioutil"
	"strings"
)

// queryCache exists so we don't need to repeat check whitelist.yaml lookups for domains
// we've seen
var queryCache = make(map[string]bool)

// checkWhitelist looks up serverName in queryCache map to determine whether to handle it
// if domain is not present, looks up domain in whitelist.yaml and updates queryCache if found
func checkWhitelist(serverName string) bool {
	whitelist := false
	// Check the cache
	if val, ok := queryCache[serverName]; ok {
		whitelist = val
	} else {
		// No entry in cache, check whitelist.yaml
		domains, err := ioutil.ReadFile("whitelist.yaml")
		if err != nil {
			return whitelist
		}

		checkDomain := strings.Replace(strings.ToLower(serverName), "www.", "", -1)
		found := strings.Contains(string(domains), checkDomain)
		whitelist = found
		if found == true {
			// Update queryCache
			queryCache[serverName] = true
		}

	}
	return whitelist
}

// getCertificateMiddleware wraps getCertificate allowing for early check of domain against whitelist
// before letsEncrypt certificate request process commences
func getCertificateMiddleware(clientHello *tls.ClientHelloInfo) (cert *tls.Certificate, err error) {
	serverName := clientHello.ServerName
	whitelist := checkWhitelist(serverName)
	if whitelist != false {
		cert, err = certificateGet(clientHello)
	} else {
		cert = &tls.Certificate{}
		err = errors.New("Domain not whitelisted")
	}
	return cert, err
}

As a result, the connection is terminated with in Chrome with ERR_SSL_PROTOCOL_ERROR which for my purposes will suffice.
I will use this in the interim, and follow this issue. You are of course welcome to use any and all of this code :) but in any event,  I would value your feedback on the approach I've taken here.

Hello, thanks to talk.
I think wrapper don't need:
certificates check in code above work before certificate request too: first check and request cert only if domain allowed.
I think your variant may be rewrite simpler - add checkWhiltelist in common check loop and add regexp .* as deny regexp.
	for _, checkDomain := range domainsToObtain {
		if !checkWhitelist(checkDomain) { // <-- added if block
			for _, re := range nonCertDomainsRegexps {
				if re.MatchString(domain) {
					logrus.Debugf("Ignore obtain cert for domain %v by regexp '%v'", DomainPresent(domain), re.String())
					continue forRegexpCheckDomain
				}
			}
		}
		allowedByRegexp = append(allowedByRegexp, checkDomain)
	}

it allow to work common cache of bad domains, it will response with server error for non whitelist domains - as your code. It doesn't need additional check in whitelist with every request: for good domains with certificate (most queries) it will get certificate from cache without additional checks.

Ah ok, I hadn't appreciated that. Let me look at this some more, and thanks for the feedback

Hello. Should either of these add that regex to the nonCertDomains?
./lets-proxy --test true --non-cert-domains ".*"

./lets-proxy --test true --non-cert-domains "/.*/"

I'm seeing an empty slice at the point the test is made? I think I'm not passing in correct format.

Now It need first variant:
|--non-cert-domains ".*"|
16.02.2017 15:47, Ollie Phillips пишет:

Hello. Should either of these add that regex to the |nonCertDomains|?
|./lets-proxy --test true --non-cert-domains "." |
|./lets-proxy --test true --non-cert-domains "/./" |
I'm seeing an empty slice at the point the test is made? I think I'm
not passing in correct format.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/rekby/lets-proxy/issues/23#issuecomment-280321420,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AFKRhd8P73AuZbLj5LXCbdqM0rLSnHFhks5rdEVbgaJpZM4L9wR_.

--
Timofey Koolin

Try branch whitelist

Appreciate your work on this. For me It does not seem to work.
I plan to look at this tomorrow since it is likely minor, but briefly requests are falling through the whitelist check despite not being domains included in whitelist.txt. I am correct in thinking the command line string to start whitelisting from a file is just this one?
./lets-proxy --whitelist-domains-file "whitelist.txt"

Using --loglevel "debug" I don't see the logrus messages related to the whitelisting procedure that I would have expected if those code blocks where being run, nor a logged error if I try pass a file to the argument which does not exist.
I spotted the typo 'whilelist' in the command line flag and have amended in my local version.

--whitelist-domains-file work only for domains, which was rejected by --non-cert-domains
try:
./lets-proxy --whitelist-domains-file "whitelist.txt" --non-cert-domains=".*"

it is optimizations for avoid read file if domain allowed without whitelist, becouse file may be very large.

Got it, thanks, I'll give it another shot

Yes, that's working perfectly with the additional flag

I was about to update the flag help to reflect the requirement to pass the --non-cert-domains flag also, but maybe I can instead add code which detects the --whitelist-domains-file flag and adds the .* regex to the nonCertDomainsRegexps slice? It would simplify the UI and still be inkeeping with the optimizations?

Auto add .* to non-cert-domains is bad. It can contain other regexps, for example - exclude temporary domains of customers and whitelist for own technically domains of hosting company.
Detect and add .* to non-cert only if non-cert is empty - i don't like too. Becouse it deny work whitelist without regexp reject.Whitelist work can be widely, then now. For example if feauture proxy can skip ip check for whitelist - in this scenario need use whitelist without reject other domains.
But add description to help text will good.
-------- Исходное сообщение --------От: Ollie Phillips<EMAIL_ADDRESS>Дата: 16.02.2017  20:50  (GMT+03:00) Кому: rekby/lets-proxy<EMAIL_ADDRESS>Копия: Timofey Koolin<EMAIL_ADDRESS>Comment<EMAIL_ADDRESS>Тема: Re: [rekby/lets-proxy] Allowed domains (#23)
I was about to update the flag help to reflect the requirement to pass the --non-cert-domains flag also, but maybe I can instead add code which detects the --whitelist-domains-file flag and adds the .* regex to the nonCertDomainsRegexps slice? It would simplify the UI and still be inkeeping with the optimizations?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or mute the thread.
{"api_version":"1.0","publisher":{"api_key":"05dde50f1d1a384dd78767c55493e4bb","name":"GitHub"},"entity":{"external_key":"github/rekby/lets-proxy","title":"rekby/lets-proxy","subtitle":"GitHub repository","main_image_url":"https://cloud.githubusercontent.com/assets/143418/17495839/a5054eac-5d88-11e6-95fc-7290892c7bb5.png","avatar_image_url":"https://cloud.githubusercontent.com/assets/143418/15842166/7c72db34-2c0b-11e6-9aed-b52498112777.png","action":{"name":"Open in GitHub","url":"https://github.com/rekby/lets-proxy"}},"updates":{"snippets":[{"icon":"PERSON","message":"@olliephillips in #23: I was about to update the flag help to reflect the requirement to pass the --non-cert-domains flag also, but maybe I can instead add code which detects the --whitelist-domains-file flag and adds the .* regex to the nonCertDomainsRegexps slice? It would simplify the UI and still be inkeeping with the optimizations?"}],"action":{"name":"View Issue","url":"https://github.com/rekby/lets-proxy/issues/23#issuecomment-280405578"}}}

Maybe others will use --whitelist-domains-file or maybe they won't. I hope the feature will be useful to others. But, either way I don't want to intrude on the flow of your app without full understanding of it, and admittedly, my specific use case - so I'm very happy to update the flag help string.
Thanks for your attention to this issue, I'll send a PR with the help additions once I've merged your latest to avoid conflicts.


RFE: Support for encrypt at REST within the clients.

It's fairly typical for storage systems to provide an encrypt at REST to ensure both the key's data + channel are encrypted.
I don't think you can store encoded keys.

Is this similar to #7250? The idea is to wrap the client KV and Txn implementations to encrypt data going into etcd and decrypt it coming out.

yup.. I'll close this one in favor for that one.


Running phpunit tests from PhpStorm tooling breaks if mutagen is enabled

Is there an existing issue for this?

[X] I have searched the existing issues

Run a Diagnostic and Paste Link Here
ddev debug test
chrisweber@MacBook-Pro ddev-nomutagen % ddev debug test
Running bash [-c /var/folders/cl/7ss_0jzj2q14mxk7qys2qj980000gp/T/test_ddev.sh]
======= Existing project config =========
These config files were loaded for project ddev-nomutagen: [/Users/chrisweber/Sites/ddev-nomutagen/.ddev/config.yaml /Users/chrisweber/Sites/ddev-nomutagen/.ddev/config.selenium-standalone-chrome.yaml]
name: ddev-nomutagen
type: drupal10
docroot: web
php_version: 8.1
webserver_type: nginx-fpm
webimage: drud/ddev-webserver:v1.21.4
router_http_port: 80
router_https_port: 443
database: {mariadb 10.4}
mailhog_port: 8025
mailhog_https_port: 8026
phpmyadmin_port: 8036
phpmyadmin_https_port: 8037
project_tld: ddev.site
use_dns_when_possible: true
composer_version: 2
nodejs_version: 16
default_container_timeout: 120
======= Creating dummy project named  tryddevproject-16641 in ../tryddevproject-16641 =========
OS Information: Darwin MacBook-Pro 22.3.0 Darwin Kernel Version 22.3.0: Mon Jan 30 20:38:37 PST 2023; root:xnu-8792.81.3~2/RELEASE_ARM64_T6000 arm64
ProductName:		macOS
ProductVersion:		13.2.1
BuildVersion:		22D68
User information: uid=502(chrisweber) gid=20(staff) groups=20(staff),12(everyone),61(localaccounts),79(_appserverusr),80(admin),81(_appserveradm),98(_lpadmin),702(com.apple.sharepoint.group.2),33(_appstore),100(_lpoperator),204(_developer),250(_analyticsusers),395(com.apple.access_ftp),398(com.apple.access_screensharing),399(com.apple.access_ssh),400(com.apple.access_remote_ae),701(com.apple.sharepoint.group.1)
DDEV version:  ITEM             VALUE
 DDEV version     v1.21.4
 architecture     arm64
 db               drud/ddev-dbserver-mariadb-10.4:v1.21.4
 dba              phpmyadmin:5
 ddev-ssh-agent   drud/ddev-ssh-agent:v1.21.4
 docker           20.10.23
 docker-compose   v2.14.0
 docker-platform  docker-desktop
 mutagen          0.16.0
 os               darwin
 router           drud/ddev-router:v1.21.4
 web              drud/ddev-webserver:v1.21.4
PROXY settings: HTTP_PROXY='' HTTPS_PROXY='' http_proxy='' NO_PROXY=''
======= DDEV global info =========
Global configuration:
instrumentation-opt-in=true
omit-containers=[]
mutagen-enabled=false
nfs-mount-enabled=false
router-bind-all-interfaces=false
internet-detection-timeout-ms=3000
disable-http2=false
use-letsencrypt=false
letsencrypt-email=
table-style=default
simple-formatting=false
auto-restart-containers=false
use-hardened-images=false
fail-on-hook-fail=false
required-docker-compose-version=
use-docker-compose-from-path=false
project-tld=
xdebug-ide-location=
no-bind-mounts=false
use-traefik=false

======= DOCKER info =========
docker location: lrwxr-xr-x  1 root  wheel  54 Dec  8 13:09 /usr/local/bin/docker -> /Applications/Docker.app/Contents/Resources/bin/docker
Docker Desktop Version: Docker Desktop for Mac 4.17.0 build 99724
docker version:
Client:
 Cloud integration: v1.0.31
 Version:           20.10.23
 API version:       1.41
 Go version:        go1.18.10
 Git commit:        7155243
 Built:             Thu Jan 19 17:35:19 2023
 OS/Arch:           darwin/arm64
 Context:           default
 Experimental:      true

Server: Docker Desktop 4.17.0 (99724)
 Engine:
  Version:          20.10.23
  API version:      1.41 (minimum version 1.12)
  Go version:       go1.18.10
  Git commit:       6051f14
  Built:            Thu Jan 19 17:31:28 2023
  OS/Arch:          linux/arm64
  Experimental:     false
 containerd:
  Version:          1.6.18
  GitCommit:        2456e983eb9e37e47538f59ea18f2043c9a73640
 runc:
  Version:          1.1.4
  GitCommit:        v1.1.4-0-g5fd4c4d
 docker-init:
  Version:          0.19.0
  GitCommit:        de40ad0
DOCKER_DEFAULT_PLATFORM=notset
======= Mutagen Info =========
Mutagen is installed in ddev, version=0.16.0
--------------------------------------------------------------------------------
Name: core-dev
Identifier: sync_pNkRwebP3T5tVjBivnFWUdTBZAyNxQ5Id9VsSLV33wL
Labels:
	com.ddev.volume-signature: var-run-1677778329
Configuration:
	Synchronization mode: Two Way Resolved
	Maximum allowed entry count: Default (2⁶⁴−1)
	Maximum staging file size: Default (18 EB)
	Symbolic link mode: POSIX Raw
	Ignore VCS mode: Default (Propagate)
	Ignores:
		/.git
		/.tarballs
		/.ddev/db_snapshots
		/.ddev/.importdb*
		.DS_Store
		.idea
		/web/sites/default/files
	Permissions mode: Default (Portable)
Alpha:
	URL: /Users/chrisweber/Sites/core-dev
	Configuration:
		Watch mode: Default (Portable)
		Watch polling interval: Default (10 seconds)
		Probe mode: Default (Probe)
		Scan mode: Default (Accelerated)
		Stage mode: Neighboring
		File mode: Default (0600)
		Directory mode: Default (0700)
		Default file/directory owner: Default
		Default file/directory group: Default
	Connected: No
Beta:
	URL: docker://ddev-core-dev-web/var/www/html
		DOCKER_HOST=unix:///var/run/docker.sock
	Configuration:
		Watch mode: Default (Portable)
		Watch polling interval: Default (10 seconds)
		Probe mode: Default (Probe)
		Scan mode: Default (Accelerated)
		Stage mode: Neighboring
		File mode: Default (0600)
		Directory mode: Default (0700)
		Default file/directory owner: Default
		Default file/directory group: Default
	Connected: No
Status: [Paused]
--------------------------------------------------------------------------------
Name: ddev-mutagen
Identifier: sync_M4cHZaIiqNFrwRkYc0IVw4YcdNyexYdYdUcMwNPnL6A
Labels:
	com.ddev.volume-signature: Users-chrisweber-colima-default-1677786106
Configuration:
	Synchronization mode: Two Way Resolved
	Maximum allowed entry count: Default (2⁶⁴−1)
	Maximum staging file size: Default (18 EB)
	Symbolic link mode: POSIX Raw
	Ignore VCS mode: Default (Propagate)
	Ignores:
		/.git
		/.tarballs
		/.ddev/db_snapshots
		/.ddev/.importdb*
		.DS_Store
		.idea
		/web/sites/default/files
	Permissions mode: Default (Portable)
Alpha:
	URL: /Users/chrisweber/Sites/ddev-mutagen
	Configuration:
		Watch mode: Default (Portable)
		Watch polling interval: Default (10 seconds)
		Probe mode: Default (Probe)
		Scan mode: Default (Accelerated)
		Stage mode: Neighboring
		File mode: Default (0600)
		Directory mode: Default (0700)
		Default file/directory owner: Default
		Default file/directory group: Default
	Connected: Yes
Beta:
	URL: docker://ddev-ddev-mutagen-web/var/www/html
		DOCKER_HOST=unix:///Users/chrisweber/.colima/default/docker.sock
	Configuration:
		Watch mode: Default (Portable)
		Watch polling interval: Default (10 seconds)
		Probe mode: Default (Probe)
		Scan mode: Default (Accelerated)
		Stage mode: Neighboring
		File mode: Default (0600)
		Directory mode: Default (0700)
		Default file/directory owner: Default
		Default file/directory group: Default
	Connected: No
Last error: beta polling error: unable to receive poll response: unable to read message length: unexpected EOF
Status: Connecting to beta
--------------------------------------------------------------------------------
Name: ddev-nomutagen
Identifier: sync_kdkVUfgta0aBSr4DOAaRkqYeDFmm0maB2IJ4RVwE47W
Labels:
	com.ddev.volume-signature: var-run-1677791725
Configuration:
	Synchronization mode: Two Way Resolved
	Maximum allowed entry count: Default (2⁶⁴−1)
	Maximum staging file size: Default (18 EB)
	Symbolic link mode: POSIX Raw
	Ignore VCS mode: Default (Propagate)
	Ignores:
		/.git
		/.tarballs
		/.ddev/db_snapshots
		/.ddev/.importdb*
		.DS_Store
		.idea
		/web/sites/default/files
	Permissions mode: Default (Portable)
Alpha:
	URL: /Users/chrisweber/Sites/ddev-nomutagen
	Configuration:
		Watch mode: Default (Portable)
		Watch polling interval: Default (10 seconds)
		Probe mode: Default (Probe)
		Scan mode: Default (Accelerated)
		Stage mode: Neighboring
		File mode: Default (0600)
		Directory mode: Default (0700)
		Default file/directory owner: Default
		Default file/directory group: Default
	Connected: Yes
	Synchronizable contents:
		6499 directories
		25372 files (126 MB)
		4 symbolic links
Beta:
	URL: docker://ddev-ddev-nomutagen-web/var/www/html
		DOCKER_HOST=unix:///var/run/docker.sock
	Configuration:
		Watch mode: Default (Portable)
		Watch polling interval: Default (10 seconds)
		Probe mode: Default (Probe)
		Scan mode: Default (Accelerated)
		Stage mode: Neighboring
		File mode: Default (0600)
		Directory mode: Default (0700)
		Default file/directory owner: Default
		Default file/directory group: Default
	Connected: Yes
	Synchronizable contents:
		6499 directories
		25372 files (126 MB)
		4 symbolic links
Status: Watching for changes
--------------------------------------------------------------------------------
======= Docker Info =========
Docker platform: docker-desktop
Using docker context: default (unix:///var/run/docker.sock)
docker-compose: v2.14.0
Using DOCKER_HOST=unix:///var/run/docker.sock
Docker version: 20.10.23
Able to run simple container that mounts a volume.
Able to use internet inside container.
Docker disk space:
Filesystem                Size      Used Available Use% Mounted on
overlay                  58.4G     13.7G     41.6G  25% /

Container ddev-ddev-nomutagen-db  Removed
Container ddev-ddev-nomutagen-dba  Removed
Container ddev-ddev-nomutagen-web  Removed
Container ddev-ddev-nomutagen-selenium-chrome  Removed
Network ddev-ddev-nomutagen_default  Removed
Project ddev-nomutagen has been stopped.
Stopped mutagen daemon
The ddev-ssh-agent container has been removed. When you start it again you will have to use 'ddev auth ssh' to provide key authentication again.
Existing docker containers:
CONTAINER ID   IMAGE                      COMMAND                  CREATED        STATUS                      PORTS     NAMES
46736414225d   docksal/varnish:6.6-3.1    "docker-entrypoint.s…"   29 hours ago   Exited (0) 26 hours ago               demo_varnish_1
d86e6faa7bdc   docksal/nginx:1.20-1.2     "docker-entrypoint.s…"   29 hours ago   Exited (0) 26 hours ago               demo_web_1
053f23225fe7   demo_cli                   "/opt/startup.sh sup…"   29 hours ago   Exited (0) 26 hours ago               demo_cli_1
40ba7f3c3186   b9af201457e0               "/bin/sh"                29 hours ago   Created                               phpstorm_helpers_PS-223.8617.59
3bbc4961fb6a   docksal/solr:1.0-solr3     "java -jar start.jar"    29 hours ago   Exited (137) 26 hours ago             demo_solr_1
a416b548745e   docksal/mariadb:10.4-1.3   "docker-entrypoint.s…"   29 hours ago   Exited (0) 26 hours ago               demo_db_1
0141f92dad8b   wodby/redis:5.0            "/docker-entrypoint.…"   29 hours ago   Exited (0) 26 hours ago               demo_redis_1
6bc888e99297   docksal/ssh-agent:1.4      "docker-entrypoint.s…"   7 days ago     Exited (2) 26 hours ago               docksal-ssh-agent
5ca1cc835e7c   docksal/dns:1.2            "docker-entrypoint.s…"   7 days ago     Exited (0) 26 hours ago               docksal-dns
7157df8b8a96   docksal/vhost-proxy:1.8    "docker-entrypoint.s…"   7 days ago     Exited (0) 26 hours ago               docksal-vhost-proxy
Creating a new ddev project config in the current directory (/Users/chrisweber/Sites/tryddevproject-16641)
Once completed, your configuration will be written to /Users/chrisweber/Sites/tryddevproject-16641/.ddev/config.yaml

Configuring unrecognized codebase as project type 'php' at /Users/chrisweber/Sites/tryddevproject-16641/web
Configuration complete. You may now run 'ddev start'.
Network ddev_default created
Starting tryddevproject-16641...
Container ddev-ssh-agent  Started
ssh-agent container is running: If you want to add authentication to the ssh-agent container, run 'ddev auth ssh' to enable your keys.
Network ddev-tryddevproject-16641_default  Created
Container ddev-tryddevproject-16641-web  Started
Container ddev-tryddevproject-16641-db  Started
Container ddev-tryddevproject-16641-dba  Started
Container ddev-router  Started
Successfully started tryddevproject-16641
Project can be reached at https://tryddevproject-16641.ddev.site https://<IP_ADDRESS>:51837
======== Curl of site from inside container:
HTTP/1.1 200 OK
Server: nginx
Date: Thu, 02 Mar 2023 21:25:54 GMT
Content-Type: text/html; charset=UTF-8
Connection: keep-alive
Vary: Accept-Encoding

======== curl -I of http://tryddevproject-16641.ddev.site from outside:
HTTP/1.1 200 OK
Server: nginx/1.20.1
Date: Thu, 02 Mar 2023 21:25:54 GMT
Content-Type: text/html; charset=UTF-8
Connection: keep-alive
Vary: Accept-Encoding

======== full curl of http://tryddevproject-16641.ddev.site from outside:
Success accessing database... db via TCP/IP
ddev is working. You will want to delete this project with 'ddev delete -Oy tryddevproject-16641'
======== Project ownership on host:
drwxr-xr-x  4 chrisweber  staff  128 Mar  2 15:25 ../tryddevproject-16641
======== Project ownership in container:
drwxr-xr-x 4 chrisweber dialout 128 Mar  2 21:25 /var/www/html
======== In-container filesystem:
Filesystem     Type          1K-blocks      Used Available Use% Mounted on
grpcfuse       fuse.grpcfuse 971350180 237141232 734208948  25% /var/www/html
======== curl again of tryddevproject-16641 from host:
Success accessing database... db via TCP/IP
ddev is working. You will want to delete this project with 'ddev delete -Oy tryddevproject-16641'
Thanks for running the diagnostic. It was successful.
Please provide the output of this script in a new gist at gist.github.com
Running ddev launch in 5 seconds
If you're brave and you have jq you can delete all tryddevproject instances with this one-liner:
    ddev delete -Oy $(ddev list -j |jq -r .raw[].name | grep tryddevproject)
In the future ddev debug test will also provide this option.

Please delete this project after debugging with 'ddev delete -Oy tryddevproject-16641'

Expected Behavior
As a developer, I want to run + debug tests using PhpStorm's test runner.
Actual Behavior
I am able to run tests via the command line, but I can't run tests using PhpStorm's test runner when mutagen is enabled.
If I disable mutagen, then I can run and debug tests using PhpStorm's test runner.
Steps To Reproduce
DDEV + Mutagen caused PhpStorm errors when running tests
Steps to Reproduce
1. Use CMS Quickstart to build + install PHPUnit
Using Docker Desktop:
mkdir my-drupal10-site
cd my-drupal10-site
ddev config --project-type=drupal10 --docroot=web --create-docroot

# Enable Mutagen:  This part introduces the bug.
ddev config --mutagen-enabled=true

ddev start
ddev composer create drupal/recommended-project
ddev composer require drupal/core-dev --dev --update-with-all-dependencies
ddev composer require drush/drush
ddev drush site:install --account-name=admin --account-pass=admin -y

2. Run a basic Unit Test | this should work.
ddev exec phpunit -c /var/www/html/web/core/phpunit.xml.dist /var/www/html/web/core/modules/system/tests/src/Unit/Routing/AdminRouteSubscriberTest.php

OR like this:
ddev exec -d /var/www/html/web "../vendor/bin/phpunit -v -c ./core/phpunit.xml.dist ./core/modules/system/tests/src/Unit/Routing/AdminRouteSubscriberTest.php"

3. Run the same tests in PhpStorm
Navigate to core/modules/system/tests/src/Unit/Routing/AdminRouteSubscriberTest.php and run the test by click on the test icon next to either a method or class.
The error you will receive will be: note I replaced my user directory with <user-dir>
Test framework quit unexpectedly

[docker-compose://[/Users/<user-dir>/Sites/ddev-mutagen/.ddev/.ddev-docker-compose-full.yaml]:web/]:php8.1 vendor/bin/phpunit --configuration /var/www/html/web/core/phpunit.xml.dist --filter Drupal\\Tests\\system\\Unit\\Routing\\AdminRouteSubscriberTest --test-suffix AdminRouteSubscriberTest.php /Users/<user-dir>/Sites/ddev-mutagen/web/core/modules/system/tests/src/Unit/Routing --teamcity
Testing started at 8:44 AM ...
PHPUnit 9.6.4 by Sebastian Bergmann and contributors.

Cannot open file "/Users/<user-dir>/Sites/ddev-mutagen/web/core/modules/system/tests/src/Unit/Routing".

Process finished with exit code 1

4. Disable Mutagen
ddev config --mutagen-enabled=false
ddev start

5. Run the same tests in PhpStorm, again
You can run the test on the command line, it will continue to work.
ddev exec phpunit -c /var/www/html/web/core/phpunit.xml.dist /var/www/html/web/core/modules/system/tests/src/Unit/Routing/AdminRouteSubscriberTest.php

OR like this:
ddev exec -d /var/www/html/web "../vendor/bin/phpunit -v -c ./core/phpunit.xml.dist ./core/modules/system/tests/src/Unit/Routing/AdminRouteSubscriberTest.php"

Now, navigate to core/modules/system/tests/src/Unit/Routing/AdminRouteSubscriberTest.php and run the test by click on the test icon next to either a method that isn't setup or the class.
Running tests in PhpStorm should work now.**
Anything else?
What we're likely seeing here is an error not with DDEV but with PhpStorm.  Still, I thought it prudent to create a bug report here to refer to the bug report I'll make with them.
I've created a full repo of my code that demonstrates this issue here: https://github.com/cosmicdreams/ddev-mutagen-phpstorm-error
Related PhpStorm error report: https://youtrack.jetbrains.com/issue/WI-71654/DDEV-Using-Mutagen-breaks-the-ability-to-run-tests-in-PhpStorm-not-the-command-line

I see what's going on. PhpStorm is suggesting the wrong path mappings when mutagen is enabled.
Change the path mappings from /var/tmp/html to /var/www/html.
<Project root>→/var/tmp/html; /Users/rfay/Library/Application Support/JetBrains/PhpSto...

That did indeed fix my issue.  This is such a huge win for those of us that use PhpStorm.  These tests are running VERY fast now.

Now that we see this what are the Next Steps?
DDev documentation fix: https://github.com/ddev/ddev/pull/4693
PhpStorm bug tracking
File an issue with the DDEV PhpStorm plugin: https://github.com/php-perfect/ddev-intellij-plugin/issues/166

Replied on the PhpStorm issue; he'll need more help to understand. Hopefully my reply will help.
The key in the plugin issue is whether that's even in scope for the plugin, whether it manages the default mappings at all.

It seems like everything is in its proper place now. Can't remember if the mapping problem is already in docs or if it's in your PR now.

I hadn't noticed their reply.  I added testing steps in the form of sample tests to run.

Make sure you "watch" the bug at https://youtrack.jetbrains.com/issue/WI-71654/DDEV-Using-Mutagen-breaks-the-ability-to-run-tests-in-PhpStorm-not-the-command-line - requires a login via github or otherwise.

Actually... I didn't get an email notification of your post, even though I'm watching. They may have something broken.

One approach here that may help with this is to remove the internally-used /var/tmp/html mount (for other reasons) and maybe then PhpStorm won't be confused. I may well work on that. The internal reasons for its existence are complex, and finding a more sustainable technique is appealing.

This has been solved in DDEV v1.22.0-beta*, and has been dealt with in the PHPStorm extension as well. Would very much appreciate your confirmation @cosmicdreams ! Closing for now.


Bump "Creator" above "Provider" on search results

This is currently at the bottom:

https://github.com/UWM-Libraries/GeoDiscovery/commit/383348c3538f834823952fe3210a69289b298311


Adding Parquet writer test for flushPolicyFactory and encoding options

Resolves https://github.com/facebookincubator/velox/issues/9496.

@svm1 can you help share what is your plan for this PR? Thx


[zh]fix namespaces-walkthrough title format error

Related issue:  #32030
Change '理解默认名字空间' to second level title as same as the en

/lgtm
Thanks for catch!

/lgtm

you may need to force push and re-trigger the pipeline

/approve


fix: dockerfile.backend build process fixed so pycharm test running works

…
Signed-off-by: Trent Smith<EMAIL_ADDRESS>
#TICKET_NUMBER

Reviewers
Functional: @danieljhegeman
Readability:

Changes

modify dockerfile.backend build process fixed so pycharm test running works.

QA steps (optional)
Notes for Reviewer

Interesting - why was this failing with Pycharm?

Something about not being able to find "./container_init.sh"

hold on maybe this does work fine. I was running this with the redesigned branch it was failing, but it seems to be working now on main.

Ah yes you might've needed to rebase. It's possible you were working off of 1220be39 on main but you needed my subsequent commit 8e9dc8ed which followed a few hours later to fix exactly that error - not able to find container_init.sh
My fault on that. Unlucky on the timing. 🙋♂ 😕 😞

change not needed


[Question] Is there a way to convert an observable to a normal React state?

Hi Jay,
As I mentioned in the title, sometimes I need to pass arguments to other hooks. If we have an API like convertToState(ob$) or ob$.toState(), it's really useful instead of wrapping the whole component with observer.
Thank you.

You can use useSelector to get the value to pass to other hooks, and you can pass obs$.set as the set function.
Making a hook to wrap that up nicely could be interesting... Would anyone else want that?


Bulk Add of Devices Into Traccar

Is there any way to add a large number of GPS devices/identfiers into the Traccar system at one time, or is it necessary to enter these one by one via the web interface? I have a Traccar server that listens to Calamp devices, and I changed from the H2 database to a mysql database. Everything works/tracks like it should for the devices I've entered manually. However, for the devices I've tried to add by simply pasting them into the device table of the mysql database, they do not show up or collect the GPS data at all. Any help or information one way or the other would be appreciated.

There are two options:

Directly add devices to the database. Note that you need to link them to your user to be able to see devices in the web console.
Use web API to add devices.

Thank You, this worked great. Have a great day!


Add golangci-lint

Current situation
No static linter currently in the pipeline.
Proposal
Use golangci-lint https://golangci-lint.run/

Part of #8


Unable to clone properly.

Hi Nice project. I wanted to clone the repo on my machine and I got this error. So not sure if it is my machine or a general issue?
Using GitHub Desktop for Windows, and Visual Studio 2022 - same issue.
It may be the trailing space in "iOS Browser UI Kit " in the path
Cloning into 'c:\dev\projects\GitHub\Responsivize'...
remote: Enumerating objects: 1836, done.
remote: Counting objects: 100% (84/84), done.
remote: Compressing objects: 100% (68/68), done.
remote: Total 1836 (delta 43), reused 40 (delta 16), pack-reused 1752
Receiving objects: 100% (1836/1836), 93.99 MiB | 15.95 MiB/s, done.
Resolving deltas: 100% (442/442), done.
error: invalid path 'assets/apple/iOS Browser UI Kit /.DS_Store'
fatal: unable to checkout working tree
warning: Clone succeeded, but checkout failed.
You can inspect what was checked out with 'git status'
and retry with 'git restore --source=HEAD :/'

@iisfaq, I'm not too sure why this is happening. It clones fine on my macbook:
❯ git clone https://github.com/virejdasani/Responsivize.git
Cloning into 'Responsivize'...
remote: Enumerating objects: 1836, done.
remote: Counting objects: 100% (84/84), done.
remote: Compressing objects: 100% (68/68), done.
remote: Total 1836 (delta 43), reused 40 (delta 16), pack-reused 1752
Receiving objects: 100% (1836/1836), 93.99 MiB | 16.70 MiB/s, done.
Resolving deltas: 100% (442/442), done.
Updating files: 100% (980/980), done.

However, I have gone ahead and removed the .DS_Store from the iOS Browser UI Kit, and fixed the trailing spaces. This should be resolved now. Can you go ahead and check if it works now?

Confirmed all working now  - thanks - cool project.


pre-commit: PR114689

Link: https://github.com/llvm/llvm-project/pull/114689
Requested by: @goldsteinn

/add-label regression,reviewed


Add link to finding aid

refs #736

Coverage decreased (-3.9%) to 96.104% when pulling b7170dbc924e5b41aa0dcfaf68bc6311731082e9 on 736-view-in-finding-aid into 5bb9e9e02878cc992b685c31e7bc4a3502510838 on master.


Thumbs do not display if output folder path has spaces

If you have C:\some folder with spaces\hub.vha2 the thumbnails will not show up 😓
Whether it's %20 or   nothing seems to work -- unsure how to fix at this time 😓

can I contribute to this issue?

Hello @ARSHIL1804 thank you for the offer!
Please see if you can replicate the bug first, and then I hope you can figure out a way to fix it 😅
I'm happy to help as much as I can (but I'll be traveling for about 2 weeks starting tomorrow so I might not respond for a while).

I looked into it. I can give you a guarantee that it is not because of any folder name issue. For me, it was replicated when I created a new hub. But when I reopen the same hub one of the video's thumbs was not visible so I changed the thumb path to the visible one's path and it worked. So my conclusion is it is not because of any path issue. Thanks for giving me a chance.


Initializing newly created tenant database

After successfully creating a new customer, hostname and website, I am struggling to get initialize the newly created tenant database.
The tenant database gets created with a corresponding db user with all db permissions set correctly.
All migrations are run and the tables get created.
But the table initialization code only works on windows and not ubuntu.
The tenant database gets initialized on windows and not ubuntu.
By initialized - all lookup tables are seeded, the default user is inserted  and user preference record is inserted.
WORKS on
windows 7 or 10.
mysql 5.7.9 (going to try and update windows to mysql 5.7.21)
apache 2.4.33
php 7.1.15
FAILS on
ubuntu 17.10
mysql 5.7.21
apache 2.4.33
PHP 7.1.16
Questions:
Does the tenant db connection get automatically set after creating a new tenant database? Appears to on windows setup.
The migrations work on both
If not, how can I force the tenant connection to the new tenant database?
Not sure how to handle - or if I'm supposed to.
public function __construct(
Config $config,
PasswordGenerator $passwordGenerator,
DatabaseManager $db,
Kernel $artisan
) {
$this->config = $config;
$this->passwordGenerator = $passwordGenerator;
$this->db = $db;
$this->artisan = $artisan;
    $this->enforceDefaultConnection();
}

When a browser hits a subdomain.domain.com it correctly sets the tenant db.

I can't really make sense of your issue as to what the exact problem is aside from Windows identifying the tenant after it being created whereas Linux does not?

Does the tenant db connection get automatically set after creating a new tenant database? Appears to on windows setup.

No it won't, unless you hit the hostname of a tenant after creating.. To know for sure I'd like to know how you create tenants, is it done from a form or in terminal?

I updated windows mysql database to 5.7.21
Everything still works on windows.
On ubuntu 17.10, the database seeds and any tenant table inserts right after the newly created tenant db and user do NOT persist.
The same code persists to the windows tenant db.
WORKS on
windows 7 or 10.
mysql 5.7.21
apache 2.4.33
php 7.1.15
FAILS on
ubuntu 17.10
mysql 5.7.21
apache 2.4.33
PHP 7.1.16

Let's stick to #429 👍


chore(ci): enable Ruff rule ERA001 and fix errors

Issue number: #2531
Summary
Changes
Enabling Ruff rule https://beta.ruff.rs/docs/rules/commented-out-code/.
User experience

Please share what the user experience looks like before and after this change

Checklist
If your change doesn't seem to apply, please leave them unchecked.

[x] Meet tenets criteria
[x] I have performed a self-review of this change
[x] Changes have been tested
[ ] Changes are documented
[x] PR title follows conventional commit semantics

Is this a breaking change?
RFC issue number:
Checklist:

[ ] Migration process documented
[ ] Implement warnings (if it can live side by side)

Acknowledgment
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.
Disclaimer: We value your time and bandwidth. As such, any pull requests created on non-triaged issues might not be successful.

Great work!


Load CQMs saved as files with load(..) function

Codecov Report

Merging #843 (46009f9) into main (4fd10b0) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##             main     #843   +/-   ##
=======================================
  Coverage   88.56%   88.56%
=======================================
  Files          71       71
  Lines        6068     6069    +1
=======================================
+ Hits         5374     5375    +1
  Misses        694      694

Impacted Files
Coverage Δ

dimod/constrained.py
88.39% <100.00%> (+0.06%)
:arrow_up:

Continue to review full report at Codecov.

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


Add new method for calulating splits without needing to return an array of splits.

Returning an array of Money might not always be the best solution when all you care about it to mutate and transform the returned Array. I'm proposing we add a new method that returns the splits information so we can do arithmetics on the information without requiring a new Array. Sometimes returning an array works, but some other times, it causes a lot of stress on the RAM with a very high num parameter.

@bdewater ping :)


Your bundle is locked to passgen (1.0.2), but that version could not be found in any of the sources listed in your Gemfile.

Seems like passgen has been deleted from RubyGems.
I have repushed version to peatio-passgen. https://rubygems.org/gems/peatio-passgen

It was yanked from RubyGems because of the following issue: https://github.com/cryptice/Passgen/issues/2


ADA Symbol overlapping text on main table

Hey Nathan, would you mind providing a few more details on this issue?
I'm just curious which browser you're using, and at what width your screen is?  I'm having a tricky time reproducing this issue on my end. thanks!

Sure thing.
It's Chrome, in the most recent update as far as I can tell.
Not sure how to gauge the exact dimensions of the browser, but it's full maximized on a 1080x1920 display.


move folder

👷 Deploy Preview for fancy-puffpuff-df775f processing.

Name
Link

🔨 Latest commit
a424237f790f561ecb6dcb009d826671f58eed8c

🔍 Latest deploy log
https://app.netlify.com/sites/fancy-puffpuff-df775f/deploys/64cedb5421bd220007855f26


feat: support conflux blockchain ide

The Conflux plugin for Ethereum's Remix IDE. It support for deploy and interacting on a Conflux network.
Signed-off-by: Deshi Xiao<EMAIL_ADDRESS>

thanks. please check this issue: https://github.com/conflux-dao-toc/conflux-remix-plugin/issues/1

thanks. please check this issue: conflux-dao-toc/conflux-remix-plugin#1

fix it. please double check it again.

If I connect I still get
js-conflux-sdk.umd.min.js:1 Uncaught (in promise) Error: Invalid parameters: address
ok, so maybe not a problem, then I deploy a contract
TypeError: t.constructor.apply is not a function
at Compiler.tsx:166
When use at address it is fine.

If I connect I still get
js-conflux-sdk.umd.min.js:1 Uncaught (in promise) Error: Invalid parameters: address
ok, so maybe not a problem, then I deploy a contract
TypeError: t.constructor.apply is not a function
at Compiler.tsx:166
When use at address it is fine.

Oops, Could you please give me your testing address and your testing contract, let me evaluate the root cause?

My conflux address: cfx:aasrcpfkxx221y7uuc0575xjc99cputbjpa83c95dt
I use the 1_Storage.sol, the first one in the example contracts of remix


The GRPC target is not implemented on the server

I am trying to create a web data stream using the following code:
client.createWebDataStream({
            parent: property[0].name,
            webDataStream: {
                displayName: ${name}DS,
                defaultUri: ${url},
                type: "WEB_DATA_STREAM",
            },
        })

Below is the error I am getting:
{
  "status": 200,
  "message": "Failed to create data stream for property TestApp: 12 UNIMPLEMENTED: The GRPC target is not implemented on the server, host: analyticsadmin.googleapis.com, method: /google.analytics.admin.v1alpha.AnalyticsAdminService/CreateWebDataStream.",
  "property": {
    "code": 12,
    "details": "The GRPC target is not implemented on the server, host: analyticsadmin.googleapis.com, method: /google.analytics.admin.v1alpha.AnalyticsAdminService/CreateWebDataStream.",
    "metadata": {
      "content-type": [
        "application/grpc"
      ],
      "grpc-server-stats-bin": [
        {
          "type": "Buffer",
          "data": [
            0,
            0,
            13,
            126,
            4,
            0,
            0,
            0,
            0,
            0
          ]
        }
      ],
      "content-length": [
        "0"
      ],
      "date": [
        "Mon, 07 Mar 2022 18:29:13 GMT"
      ],
      "alt-svc": [
        "h3=":443"; ma=2592000,h3-29=":443"; ma=2592000,h3-Q050=":443"; ma=2592000,h3-Q046=":443"; ma=2592000,h3-Q043=":443"; ma=2592000,quic=":443"; ma=2592000; v="46,43""
      ]
    }
  }
}

Should be fixed by 3.2.0, please let us know if there are any continued issues:
https://github.com/googleapis/nodejs-analytics-admin/pull/175


When terminal is focused, selecting test in editor won't focus the editor

VSCode Version: 1.2.1
OS Version: OS X

Steps to Reproduce:

Open terminal
Open a text editor as well
Execute a command in terminal
Using mouse make a selection in editor

Editor is not focused. You have to click on editor again to focus in it

I cannot reproduce, @mohsen1 any more details? video / screenshot maybe?

It's actually not just by selecting. Clicking on the editor won't switch the focus either.
I use Fish if that matters.
here is a screencapture:
https://gfycat.com/ImpeccableFluidLabradorretriever

@Tyriar can you reproduce?

Duplicate of https://github.com/Microsoft/vscode/issues/7684, fixed in Insiders


Vigilant feature marks cryptographically verified signatures (whose GPG key's email has been verified by github) as unverified.

It says cryptographically unverified, when it in fact is cryptographically verifiable/verified.
I'd suggest Github changes this scenario to partially verified at the very least (as opposed to "WARNING! WARNING! UNVERIFIED!", but ideally I'd like to be allowed to set whatever author email I want in my git commits, without that impeding on the veracity of my GPG signatures.
My use-case is to not spread an email I care about more than I have to. Enough trawlers already - few if any care to parse GPG keys and extract (it seems).
9/10 times (in my personal experience) they just trawl github commit logs with a webscraping service instead.
Those who think that the emails must match (that the git commit/tag author email gives any security at all) have a nasty surprise-in-waiting.

Why:
Closes https://github.com/github/docs/issues/8173.

What's being changed:

Check off the following:

[x] I have reviewed my changes in staging (look for the latest deployment event in your pull request's timeline, then click View deployment).
[x] For content changes, I have completed the self-review checklist.

Writer impact (This section is for GitHub staff members only):

[ ] This pull request impacts the contribution experience

[ ] I have added the 'writer impact' label
[ ] I have added a description and/or a video demo of the changes below (e.g. a "before and after video")

@x10an14 Thanks so much for opening a PR! I'll get this triaged for review :zap:

the claim is valid

? Are you referring to the gpg signature?

I'm sorry, but I don't think you've solved the core of the issue here. Is this perhaps not the right place to come with this customer feedback?
Users will look at "Vigilant" and see the "verified/unverified" tags - and act/react on a basis if something seems "verified and safe" vs "unverified and unsafe".
And only the fewest users will care to git clone the commits themselves, and double-check the GPG commit signature themselves - when and if your "Vigilant" system marks a commit as "unverified".
In essence, I get the UX that you're hiding/diminishing the value of a "cryptographically verifiable signature" in your efforts to promote the Github's vendor-specific "Vigilant".
My commits (pushed to github) will still be marked as "unverified", and that's doing me a disservice since I've gone through the trouble of learning GPG, handling my keys securely with smart-cards, and signing my commits so people can trust that this is me.
Not some random rootkit on my machine or random mischievous agent on the internet.
I don't mind Github wanting to promote/make use of "Vigilant", but why does it need to react when these two emails don't match? Given that the cryptographic signature is valid???

Re-reading your edits to the proposed changes, I see that I actually am fulfilling all requirements. Yet my commits get marked "Unverified".
Just see here.

By default commits and tags are marked "Verified" if they are signed with a GPG or S/MIME key that was successfully verified. If a commit or tag has a signature that can't be verified by {% data variables.product.prodname_dotcom %}, we mark the commit or tag "Unverified." In all other cases no verification status is displayed.

It is cryptographically verifiable.

You should only enable vigilant mode if you sign all of your commits and tags and use an email address that is verified for your account on {% data variables.product.product_name %} as your committer email address.

But both my committer (AND GPG email) are verified with you (read: Github) - so why are my commits marked as unverified then?
The only thing I can see (but there might be something I don't see) is that you (for some unfathomable reason to me) compare the committer email with the emails listed in the GPG key.

@x10an14 This is great feedback to give to https://support.github.com/contact/feedback :yellow_heart:


Revert "Update Dataflow worker harness container image version."

Reverts apache/beam#13699. The new image causes performance regressions.

Do you know why it is failing tests?

Do you know why it is failing tests?

Do you know why it is failing tests?

Java_Examples_Dataflow: Cancel request is committed for workflow job: 2021-01-11_14_35_06-18160383623200316800.
I'm not sure where a cancel request came from.
Java_Examples_Dataflow_Java11:
INFO: Job 2021-01-11_14_31_10-9493314082335955186 finished with status DONE. Jan 11, 2021 10:34:37 PM org.apache.beam.runners.dataflow.TestDataflowRunner run WARNING: Dataflow job 2021-01-11_14_31_10-9493314082335955186 did not output a success or failure metric. Jan 11, 2021 10:34:37 PM org.apache.beam.runners.dataflow.DataflowPipelineJob logTerminalState INFO: Job 2021-01-11_14_31_10-9493314082335955186 finished with status DONE.
However according to the UI the job status is 'Succeeded".

Do you know why it is failing tests?

Java_Examples_Dataflow: Cancel request is committed for workflow job: 2021-01-11_14_35_06-18160383623200316800.
I'm not sure where a cancel request came from.
Java_Examples_Dataflow_Java11:
INFO: Job 2021-01-11_14_31_10-9493314082335955186 finished with status DONE. Jan 11, 2021 10:34:37 PM org.apache.beam.runners.dataflow.TestDataflowRunner run WARNING: Dataflow job 2021-01-11_14_31_10-9493314082335955186 did not output a success or failure metric. Jan 11, 2021 10:34:37 PM org.apache.beam.runners.dataflow.DataflowPipelineJob logTerminalState INFO: Job 2021-01-11_14_31_10-9493314082335955186 finished with status DONE.
However according to the UI the job status is 'Succeeded".

If you think it is a flake, please file a jira and rerun the tests.

If you think it is a flake, please file a jira and rerun the tests.

Created https://issues.apache.org/jira/browse/BEAM-11621.

Created https://issues.apache.org/jira/browse/BEAM-11621.

Run Java_Examples_Dataflow PreCommit

Run Java_Examples_Dataflow PreCommit

Run Java_Examples_Dataflow PreCommit

Run Java_Examples_Dataflow PreCommit

Run Java_Examples_Dataflow_Java11 PreCommit

Run Java_Examples_Dataflow_Java11 PreCommit

Tests pass, it was a flake. Can you merge this please?

Tests pass, it was a flake. Can you merge this please?

LGTM. Thank you Tyson.

LGTM. Thank you Tyson.


Add "anicent" feature

This allows the project to compile for 1.36.0, the version stabilized alloc.
This would close #2.
Not sure if this is the best feature name ...

See https://github.com/bbqsrc/bare-io/issues/2#issuecomment-743734678.


[stable/velero] Configure Velero plugins with ConfigMaps

What this PR does / why we need it:
My use case is to change the restic restore image with a configmap as documented here: https://velero.io/docs/v1.0.0/restic/
I've taken a generic approach that can be used for other Velero plugins instead of a dedicated configmap for the restic restore image. Give me feedback if it's not a good idea.
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[x] DCO signed
[x] Chart Version bumped
[x] Variables are documented in the README.md
[x] Title of the PR starts with chart name (e.g. [stable/chart])

/assign @hectorj2f

/ok-to-test

/lgtm

as helm deployment name is prefixed with the release name, velero plugin add fails because it tries to find the (release-less) deployment by name velero..
can you demonstrate how one would use this configmap to install say the portworx-velero plugin?

@aleks-mariusz the configmap is not for installing plugins but for configuring them. In the documentation only the embedded restic plugin seems to use that kind on configmap.
It seems there is no mechanism yet in the chart to do the "velero plugin add" command that install plugins.

@aleks-mariusz after reading the whole values.yaml, you certainly can install the plugin by configuring initContainers


does not use python3 when both python2 & 3 are installed

I have python 3.7 installed. I am using a mac. The default installation of python 3 for mac users leaves python 2 installed because python 3 is not backwards compatible: https://stackoverflow.com/questions/5846167/how-to-change-default-python-version
However rma does not look for the python 3 libraries it needs, instead it uses the default ones and fails as follows:
rma --help
Traceback (most recent call last):
  File "/usr/local/bin/rma", line 6, in <module>
    from rma.cli.rma_cli import main
  File "/usr/local/lib/python2.7/site-packages/rma/__init__.py", line 1, in <module>
    from rma.application import RmaApplication
  File "/usr/local/lib/python2.7/site-packages/rma/application.py", line 145
    self.reporter.print(str_res)
                      ^
SyntaxError: invalid syntax

I do not use python regularly as a programming language, so I am unfamiliar with how the python community typically resolves this issue. I am trying to use rma because it provides a unique feature set. It would be helpful if rma either did the right thing in this situation or there was a paragraph in the README explaining how to resolve this.

Make sure you used pip3 to install rma. You may need to do sudo pip uninstall rma, and then sudo pip3 install rma.

ok that resolved the issue, closing this.


Refactor exec command

To be merged after https://github.com/kurtosis-tech/kurtosis/pull/700 is released

@victorcolombo i guess you'll have to fix this https://github.com/kurtosis-tech/eth2-package/issues/50 too


six- and seven-split layouts have two splits of the same size, one mini one plain

We should be consistent for a given size of split, no?

Oh, we already have an issue for this #6915****


changing bounds on base in .cabal to work with ghc 7.10

ghc 7.10 uses base 4.8, so I had to update the base dep. bounds. Works for me with ghc 7.8 and 7.10.
Is there are reason why you required a particular major version of th-lift before?
Another thing I had to do to get HaPy working with ghc 7.10 is issue the following install command:
cabal install --enable-shared --enable-executable-dynamic
Haven't thought about why this is necessary for me.

If it works with the relaxed bounds then this LGTM.  Thanks for the patch!
I required that particular major version of th-lift solely defensively.  It's not entirely clear to me what best practices are here (since to my understanding it's all a bit of mess), so relaxing that bound seems reasonable.
With regards to needing --enable-executable-dynamic, I'm not certain why that would be necessary.  Are you building as an executable instead of as a library?

Upon further consideration, I think upper bounds are probably a good idea (and hackage requires an upper bound for base to accept package uploads), so I'm adding back in relaxed upper bounds.  This shouldn't cause any problems for GHC 7.10, though, because base 4.8 is now accepted.

Thanks!


InvalidEndpointAssociationName error when creating dataCollectionRuleAssociations and specifying dataCollectionEndpointId

Describe the bug
I am creating an dataCollectionRuleAssociations using terraforms resource 'azurerm_resource_group_template_deployment' and the ARM template that is described here.
When I don't specify the dataCollectionEndpointId, it will be empty and the association with the data collection endpoint is not created. But I do want to specify this, to make the association with the data collection enpoint as well. But when I do that, the deployment fails with the following error message:
{"code":"DeploymentFailed","message":"At least one resource deployment operation failed. Please list deployment operations for details. Please see https://aka.ms/DeployOperations for usage details.","details":[{"code":"InvalidEndpointAssociationName","message":"An association of data collection endpoint must be named 'configurationAccessEndpoint'."}]}
To Reproduce
Steps to reproduce the behavior:

Create an data collection rule and a data collection endpoint
Create a virtual machine
Create a dataCollectionRuleAssociation with the ARM template that is described here
See error

Expected behavior
This ARM template deployment should create the association with the data collection rule and the data collection endpoint.
Screenshots
Not applicable
Client [Terraform, API)
Additional context
My code:
Terraform resource:
resource "azurerm_resource_group_template_deployment" "data_collection_rule_association" {   name                         = "data_collection_rule_association_${var.vm_name}"   resource_group_name          = var.rg_name   deployment_mode              = "Incremental"   template_content             = file("${path.module}/arm_templates/dataCollectionRuleAssociation.json")   parameters_content           = jsonencode({     "vmName"                   = { value = var.vm_name }     "associationName"          = { value = "data_collection_rule_association_${var.vm_name}" }     "dataCollectionRuleId"     = { value = var.data_collection_rule_id }     "dataCollectionEndpointId" = { value = var.data_collection_endpoint_id }   }) }
ARM Template:
{   "$schema": "https://schema.management.azure.com/schemas/2019-04-01/deploymentTemplate.json#",   "contentVersion": "<IP_ADDRESS>",   "parameters": {     "vmName": {       "type": "string",       "metadata": {         "description": "The name of the virtual machine."       }     },     "associationName": {       "type": "string",       "metadata": {         "description": "The name of the association."       }     },     "dataCollectionRuleId": {       "type": "string",       "metadata": {         "description": "The resource ID of the data collection rule."       }     },     "dataCollectionEndpointId": {       "type": "string",       "metadata": {         "description": "The resource ID of the data collection endpoint."       }     }   },   "resources": [     {       "type": "Microsoft.Insights/dataCollectionRuleAssociations",       "apiVersion": "2021-09-01-preview",       "scope": "[format('Microsoft.Compute/virtualMachines/{0}', parameters('vmName'))]",       "name": "[parameters('associationName')]",       "properties": {         "description": "Association of data collection rule. Deleting this association will break the data collection for this virtual machine.",         "dataCollectionRuleId": "[parameters('dataCollectionRuleId')]",         "dataCollectionEndpointId": "[parameters('dataCollectionEndpointId')]"       }     }   ] }

+1

Hi Sander I fixed this by literally naming the association 'configurationAccessEndpoint' for the endpoint association. I also had to create a separate association for the Data Collection Rule association.
Hope this helps


NFT: split parametrisation of Address/Token

At the moment there is no way to find out on chain what the correct CurrencySymbol is for the correct parametrisation of an address. Up until now we have been using workarounds but they have not been adequate solutions. To generalise and create a solution we should do the following:
Parametrise the Validator Script with the Unique Token, and parametrise the Minting Policy with the Address of the Script:
ValidatorScript :: UniqueToken -> Datum -> Redeemer -> ScriptContext
MintingScript   :: Address ValidatorScript -> Redeemer -> ScriptContext

Now at initialisation, we proceed as always to create a list head by placing the UniqueToken and the Head List Token in the Head of the list. When operating on the list, the transaction must make proof of the List Head. The Validator Script can now ensure that the correct List Token is minted, by comparing it to the other Token present in the Head (the List Token).  The Minting script has access to its own CurrencySymbol, and can also only mint tokens only for the Validator Script Address.

[ ] done in commit c3ef7ca3fb546265d6bfefcda06d3cd0ef111069 for the Gov.
[ ] Nft


update blossom-ci action repo addr [skip ci]

Signed-off-by: Peixin Li<EMAIL_ADDRESS>the blossom-action has been moved to NVIDIA org.
Tested in my forked repo, I will make a following PR to double check after merged

build


SigmoidCrossEntropy without reduction (v2)

SigmoidCrossEntropy without reduction. This PR is related to #2558.

LGTM, I am running GPU tests.

Sorry for not responding so long. GPU test failed due to the error that does not relevant to this PR. Could you merge the latest _v2 branch?

I merged _v2.

Following the tentative conclusion I wrote here, I close this PR. Thank you.


Base search in relation fields on the formatted value equality

In form, add formatted value of a field to searched fields

Available on v3.2.6 🚀


Reversed ReorderableListView does not allow for items to be put last

Steps to Reproduce

Create app with ReorderableListView and reverse is true: http://dartpad.dartlang.org/bddc60353fa0c519d9afac08c1ca033b
Try to drag an Item 1 below Item 4, it does not work:

Logs

[✓] Flutter (Channel master, 1.19.0-6.0.pre.42, on Mac OS X 10.15.5 19F101, locale en-DE)
    • Flutter version 1.19.0-6.0.pre.42 at /Users/perc/development/flutter
    • Framework revision d9653445f4 (14 hours ago), 2020-06-09 18:43:03 -0400
    • Engine revision e8c13aa012
    • Dart version 2.9.0 (build 2.9.0-14.0.dev 5c1376615e)

[✓] Android toolchain - develop for Android devices (Android SDK version 29.0.3)
    • Android SDK at /Users/perc/Library/Android/sdk
    • Platform android-stable, build-tools 29.0.3
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_212-release-1586-b4-5784211)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 11.3.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 11.3.1, Build version 11C505
    • CocoaPods version 1.8.4

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio
    • Android Studio at /Applications/Android Studio with Blaze.app/Contents
    • Flutter plugin version 46.0.2
    • Dart plugin version 193.7361
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6222593)

[✓] Android Studio (version 3.6)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin version 45.1.1
    • Dart plugin version 192.7761
    • Java version OpenJDK Runtime Environment (build 1.8.0_212-release-1586-b4-5784211)

[!] VS Code (version 1.45.1)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    ✗ Flutter extension not installed; install from
      https://marketplace.visualstudio.com/items?itemName=Dart-Code.flutter

[✓] Connected device (4 available)
    • iPhone 11 Pro Max • 81158E5C-C27D-49D1-9FD9-55EE1011FEBF • ios            • com.apple.CoreSimulator.SimRuntime.iOS-13-3 (simulator)
    • macOS             • macOS                                • darwin-x64     • Mac OS X 10.15.5 19F101
    • Web Server        • web-server                           • web-javascript • Flutter Tools
    • Chrome            • chrome                               • web-javascript • Google Chrome 83.0.4103.97

This issue has been fixed in the latest stable version
Video
https://user-images.githubusercontent.com/50555895/171165674-12f67c4b-7399-4a4b-8fe6-6733c5ecb6a0.mov

Code sample
import 'package:flutter/material.dart';

void main() => runApp(const OnReorderDemo());

class OnReorderDemo extends StatelessWidget {
  const OnReorderDemo({Key? key}) : super(key: key);

  @override
  Widget build(BuildContext context) {
    return const MaterialApp(
      title: 'Reversed ReorderableListView',
      home: MyReorderableList(),
    );
  }
}

class MyReorderableList extends StatefulWidget {
  const MyReorderableList({Key? key}) : super(key: key);

  @override
  State<MyReorderableList> createState() => _MyReorderableListState();
}

class _MyReorderableListState extends State<MyReorderableList> {
  List<Widget> listItems = ['Item 1', 'Item 2', 'Item 3', 'Item 4']
      .map((title) => ListTile(
            title: Text(title),
            key: Key(title),
          ))
      .toList();

  void onReorder(int oldIndex, int newIndex) {
    print('oldIndex: $oldIndex, newIndex: $newIndex');
  }

  @override
  Widget build(BuildContext context) {
    return Scaffold(
      appBar: AppBar(
        title: const Text('Reversed ReorderableListView'),
      ),
      body: SafeArea(
        child: Padding(
          padding: const EdgeInsets.only(bottom: 20),
          child: ReorderableListView(
            reverse: true,
            onReorder: onReorder,
            children: listItems,
          ),
        ),
      ),
    );
  }
}

flutter doctor -v (Stable)
[✓] Flutter (Channel stable, 3.0.1, on macOS 12.4 21F79 darwin-arm, locale en-US)
    • Flutter version 3.0.1 at /Users/nabeelparkar/fvm/versions/stable
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision fb57da5f94 (11 days ago), 2022-05-19 15:50:29 -0700
    • Engine revision caaafc5604
    • Dart version 2.17.1
    • DevTools version 2.12.2

[✓] Android toolchain - develop for Android devices (Android SDK version 32.1.0-rc1)
    • Android SDK at /Users/nabeelparkar/Library/Android/sdk/
    • Platform android-32, build-tools 32.1.0-rc1
    • ANDROID_SDK_ROOT = /Users/nabeelparkar/Library/Android/sdk/
    • Java binary at: /Applications/Android Studio.app/Contents/jre/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 11.0.12+0-b1504.28-7817840)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 13.4)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • CocoaPods version 1.11.3

[✓] Chrome - develop for the web
    • CHROME_EXECUTABLE = /Applications/Brave Browser.app/Contents/MacOS/Brave Browser

[✓] Android Studio (version 2021.2)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.12+0-b1504.28-7817840)

[✓] IntelliJ IDEA Ultimate Edition (version 2022.1.1)
    • IntelliJ at /Applications/IntelliJ IDEA.app
    • Flutter plugin version 67.1.4
    • Dart plugin version 221.5591.58

[✓] VS Code (version 1.67.2)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.40.0

[✓] Connected device (2 available)
    • macOS (desktop) • macos  • darwin-arm64   • macOS 12.4 21F79 darwin-arm
    • Chrome (web)    • chrome • web-javascript • Brave Browser <IP_ADDRESS>

[✓] HTTP Host Availability
    • All required HTTP hosts are available

• No issues found!

flutter doctor -v (Master)
[✓] Flutter (Channel master, 3.1.0-0.0.pre.1014, on macOS 12.4 21F79 darwin-arm, locale en-US)
    • Flutter version 3.1.0-0.0.pre.1014 at /Users/nabeelparkar/fvm/versions/master
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 08e61c3be2 (2 hours ago), 2022-05-31 00:18:09 -0400
    • Engine revision 693f95391e
    • Dart version 2.18.0 (build 2.18.0-152.0.dev)
    • DevTools version 2.13.1

[✓] Android toolchain - develop for Android devices (Android SDK version 32.1.0-rc1)
    • Android SDK at /Users/nabeelparkar/Library/Android/sdk/
    • Platform android-32, build-tools 32.1.0-rc1
    • ANDROID_SDK_ROOT = /Users/nabeelparkar/Library/Android/sdk/
    • Java binary at: /Applications/Android Studio.app/Contents/jre/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 11.0.12+0-b1504.28-7817840)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 13.4)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • CocoaPods version 1.11.3

[✓] Chrome - develop for the web
    • CHROME_EXECUTABLE = /Applications/Brave Browser.app/Contents/MacOS/Brave Browser

[✓] Android Studio (version 2021.2)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.12+0-b1504.28-7817840)

[✓] IntelliJ IDEA Ultimate Edition (version 2022.1.1)
    • IntelliJ at /Applications/IntelliJ IDEA.app
    • Flutter plugin version 67.1.4
    • Dart plugin version 221.5591.58

[✓] VS Code (version 1.67.2)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.40.0

[✓] Connected device (2 available)
    • macOS (desktop) • macos  • darwin-arm64   • macOS 12.4 21F79 darwin-arm
    • Chrome (web)    • chrome • web-javascript • Brave Browser <IP_ADDRESS>

[✓] HTTP Host Availability
    • All required HTTP hosts are available

• No issues found!


Add/remove of targets when editing an External Service is not working.

Server version - V0.23.0-rc5
Steps to reproduce the problem:
Create External Service with 2 targets.
Activate service.
Edit Service and add a new target.
The newly added target is not show as part of the External service.
Edit Service and remove an existing target.
Removed  target is still shown as part of the External service.

https://github.com/rancherio/cattle/pull/566

Is this just an UI part now that Alena has merged her PR?

Server version - V0.23.0-rc6 also has this problem,

ok..reassigning back to Alena.

With V0.23.0-rc6:
When i edit an external service , i see UI sending the address - "<IP_ADDRESS>.7" that was added:
{id: "1s6", type: "externalService", name: "Extserv1", state: "active", accountId: "1a5",…}
accountId: "1a5"
consumedServicesUpdated: 0
created: "2015-06-06T05:20:12Z"
createdTS:<PHONE_NUMBER>000
description: "Extserv1"
environmentId: "1e1"
externalIpAddresses: ["<IP_ADDRESS>", "<IP_ADDRESS>", "<IP_ADDRESS>.7"]
headers: null
id: "1s6"
kind: "externalService"
name: "Extserv1"
removeTime: null
removed: null
state: "active"
transitioning: "no"
transitioningMessage: null
transitioningProgress: null
type: "externalService"
uuid: "7e805185-e57e-4396-bf9f-f81e69dcdd58"
But DB does not have this:
ysql> select * from service_expose_map where service_id=6;
+----+------+------------------+--------------------------------------+-------------+--------+---------------------+---------+-------------+------+------------+-------------+------------+---------------+
| id | name | kind             | uuid                                 | description | state  | created             | removed | remove_time | data | service_id | instance_id | account_id | ip_address    |
+----+------+------------------+--------------------------------------+-------------+--------+---------------------+---------+-------------+------+------------+-------------+------------+---------------+
| 12 | NULL | serviceExposeMap | 36850740-34c1-476a-83b0-fc590faddd99 | NULL        | active | 2015-06-06 05:20:17 | NULL    | NULL        | {}   |          6 |        NULL |          5 | <IP_ADDRESS> |
| 13 | NULL | serviceExposeMap | 2ef0836e-e4f5-4572-a353-7991cd198b44 | NULL        | active | 2015-06-06 05:20:17 | NULL    | NULL        | {}   |          6 |        NULL |          5 | <IP_ADDRESS>  |
+----+------+------------------+--------------------------------------+-------------+--------+---------------------+---------+-------------+------+------------+-------------+------------+---------------+
2 rows in set (0.00 sec)

The fix is not a part of RC6; its a part of master - got merged late at night

Tested on lastet master build.
Adding/removing of targets when editing an External Service is working as expected.


Feature/ancient debris miner

I'm not sure if ancient debris not coming from the dimensional mineshaft was a conscious design choice, but this PR adds it at a 0.2% chance.

unsure how my pr got some of your changes lol - gonna reopen with a cleaner one.


Move Accelerate backend elsewhere

Codecov Report

Merging #110 (0551420) into main (e802121) will decrease coverage by 0.59%.
The diff coverage is n/a.

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

@@            Coverage Diff             @@
##             main     #110      +/-   ##
==========================================
- Coverage   95.74%   95.16%   -0.59%
==========================================
  Files          55       39      -16
  Lines        3951     3039     -912
==========================================
- Hits         3783     2892     -891
+ Misses        168      147      -21

Files Changed
Coverage Δ

RTNeural/Layer.h
83.33% <ø> (ø)

RTNeural/ModelT.h
91.93% <ø> (-0.17%)
:arrow_down:

RTNeural/activation/activation.h
93.20% <ø> (ø)

RTNeural/batchnorm/batchnorm.tpp
100.00% <ø> (ø)

RTNeural/batchnorm/batchnorm2d.tpp
100.00% <ø> (ø)

RTNeural/common.h
94.11% <ø> (-0.54%)
:arrow_down:

RTNeural/conv1d/conv1d.h
95.34% <ø> (ø)

RTNeural/conv1d/conv1d.tpp
100.00% <ø> (ø)

RTNeural/conv2d/conv2d.tpp
100.00% <ø> (ø)

RTNeural/dense/dense.h
96.29% <ø> (ø)

... and 4 more

... and 16 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Redesign Subpass/Pipeline Distinction

It seems from the API that Anvil has a few strange interactions between Subpasses and Pipelines.
For example, the Anvil::RenderPass::add_subpass() method takes pointers to all the shader stages, even though this is a property of a Pipeline. If you don't pass in a PipelineID, it will create one for you with these shader stages, but if not then it will ignore these arguments.
The subpass also stores a pointer to a single pipeline that can be set and retrieved with get_subpass_graphics_pipeline_id() and set_subpass_graphics_pipeline_id(). But a subpass can and often will have more than one pipeline associated with it, since this is how different materials are usually handled. So I don't know that it makes sense for there to be a 1-to-1 mapping like this.
It seems like there may also be a circular dependency in the API - If I try to create a pipeline through the GraphicsPipelineManager, it requires the ID of the subpass I want to create it for.
BUT if I want to create a Subpass via RenderPass::add_subpass(), it requires a PipelineID if I don't want to just create the default.
So it seems like the only choice the API gives is to use the default pipeline that is created automatically by add_subpass()
I think probably the best course of action overall is to just leave it to the user of the API to create their own pipelines manually. Even when I was first using the API, it seemed like somewhat of an counter-intuitive side-effect that the add_subpass() method went into the GraphicsPipelineManager and added a GraphicsPipeline on my behalf.

Fixed internally - subpasses are no longer associated with specific graphics pipelines. Sorry about this!
The fix will be coming in the first week of January's drop.


DC Mission: Events to Business Actions Use Case

Dynamically react to changing business events in your supply chain (mission)

Thanks Mahesh!
We have just re-structured our repository (see https://github.com/SAP-samples/btp-terraform-samples/pull/53 ) and will have to move your TF script into that structure after the PR is reviewed and approved by @lechnerc77 .
Thanks a lot for the contribution!!

@mahesh0431 @rui1610 The PR with the updated structure is merged and available in main

Just checked the code, and I think we need to change some code before merging it to main. Will leave some comments.

Please go through the comments I've added and address them properly.

Hi @rui1610, I've made the changes to follow the new structure. Please review. I also want to thank @lechnerc77 for helping me with my queries :)


Globally disable without config?

The mere presence of this gem preventing emails from going out seems like a dangerous approach. Why require configuration to be both in the Gemfile and the initializer?

If you don't want mail safe to do anything, don't require it.

Bundler is pretty much the de-facto standard, and you can't exclude a named group as far as I know. Requiring a gem that has potentially dangerous side-effects might be something worth mulling over...

Bundler is pretty much the de-facto standard, and you can't exclude a named group as far as I know. Requiring a gem that has potentially dangerous side-effects might be something worth mulling over...

I agree. That's why you shouldn't be auto-requiring any gems.
There are three solutions you can use with bundler:

Use Bundler.setup rather than Bundler.require.  setup sets up your load path but does not auto-require gems. I think it's the best way to use bundler anyway...I really dislike bundler's auto-requiring functionality.  It's a great way to get an app with really slow boot times.
Use Bundler.require(:group1, :group2), and put put mail_safe in one of the other groups.
add :require => nil to the gem declaration in your Gemfile.


feat: add bom.definitions

This might also be considered as a part of #696 since the bom.definitions is introduced in v1.6.
We're currently using bom.definitions to indicate that the given SBOM aligns with certain standards. It'd be nice to have it supported in the library.

@hakandilek ,
You know your use cases best, and you could provide examples and tests.
Would you be interested in contributing this feature?


Add GitHub Actions for CI

Setup to deploy features from https://www.hydroshare.org/resource/495b65e56e994289baaa5feeb401358e/

@webb-ben  this failed, i thinkg the GCP cloud storage acrtion id deprecated


Support 0-dim params in distributions

This PR uses #5075.

Jenkins, test this please.

Jenkins CI test (for commit 7508c34c2877b33ba0c94ad46b9024cc4a15df99) succeeded without errors!

According to @toslunar, this feature does not require #5075 (which was reverted in #5087.)


glog: update to version 0.3.5-p1

Testing:

https://ci.appveyor.com/project/ingenue/hunter/build/1.0.2298
https://travis-ci.org/ingenue/hunter/builds/270771404?utm_source=github_status&utm_medium=notification

three Apple builds are failing. They all have been terminated

Testing

Some windows tests failed.

three Apple builds are failing. They all have been terminated

It's unit-tests, nothing bad.

all windows tests below vs-14 2015 are failing with an error like this
C:\projects\hunter\_testing\Hunter\_Base\50c7ee6\0cc91fa\7ededb5\Build\glog\Source\src\logging.cc(1154) : error C2483: 'thread_msg_data' : object with constructor or destructor cannot be declared 'thread'

corresponting source code
https://github.com/hunter-packages/glog/blob/hunter_v0.3.5/src/logging.cc#L1154
// Static log data space to avoid alloc failures in a LOG(FATAL)
//
// Since multiple threads may call LOG(FATAL), and we want to preserve
// the data from the first call, we allocate two sets of space.  One
// for exclusive use by the first thread, and one for shared use by
// all other threads.
static Mutex fatal_msg_lock;
static CrashReason crash_reason;
static bool fatal_msg_exclusive = true;
static LogMessage::LogMessageData fatal_msg_data_exclusive;
static LogMessage::LogMessageData fatal_msg_data_shared;

#ifdef GLOG_THREAD_LOCAL_STORAGE
// Static thread-local log data space to use, because typically at most one
// LogMessageData object exists (in this case glog makes zero heap memory
// allocations).
static GLOG_THREAD_LOCAL_STORAGE bool thread_data_available = true;
static GLOG_THREAD_LOCAL_STORAGE LogMessage::LogMessageData thread_msg_data;
#endif // defined(GLOG_THREAD_LOCAL_STORAGE)

is thread maybe a new feature? I don't know enough to fix this

is thread maybe a new feature? I don't know enough to fix this

I guess it's some new C++11 feature which is implemented in vs-14-2015 and not available for versions before. Feel free to exclude.

PR excluding: https://github.com/ingenue/hunter/pull/121

Testing:

https://ci.appveyor.com/project/ingenue/hunter/build/1.0.2314
https://travis-ci.org/ingenue/hunter/builds/270827299?utm_source=github_status&utm_medium=notification

Uploading:

https://ci.appveyor.com/project/ingenue/hunter/build/1.0.2317
https://travis-ci.org/ingenue/hunter/builds/271127417?utm_source=github_status&utm_medium=notification

Released: https://github.com/ruslo/hunter/releases/tag/v0.19.91

https://github.com/google/glog/blob/40fc21467abb4516976ea2aa586bd206d0b86d20/CMakeLists.txt#L195-L218
C++11 supports thread_local storage, and various compilers before this have supported approximations of it in the past.  glog does try to check_cxx_source_compiles() to find backups:
check_cxx_source_compiles ("
__declspec(thread) int tls;
int main() { }
" HAVE_MSVC_TLS)

check_cxx_source_compiles ("
thread_local int tls;
int main() { }
" HAVE_CXX11_TLS)

check_cxx_source_compiles ("
__attribute__((thread)) int tls;
int main() { }
" HAVE_CYGWIN_TLS)

if (WITH_TLS)
  if (HAVE_CYGWIN_TLS)
    set (GLOG_THREAD_LOCAL_STORAGE "__attribute__((thread))")
  elseif (HAVE_MSVC_TLS)
    set (GLOG_THREAD_LOCAL_STORAGE "__declspec(thread)")
  elseif (HAVE_CXX11_TLS)
    set (GLOG_THREAD_LOCAL_STORAGE thread_local)
  endif (HAVE_CYGWIN_TLS)
endif (WITH_TLS)

glog checks availablility, but why does it fail on older vs compilers? Are the flag-tests wrong, are the flags not properly used in code or do older vs-compilers just don't support TLS?


Explore whether we must update "scorer" interface for multi-level learning

There are techniques which are explicitly designed for learning from data produced at multiple levels of theory. We should ensure that these are possible to implement within our Scorer interface.

Yes, we can, and I think the way to do it is to still have a single target outcome (e.g., the highest level of theory) but allow the model to use lower levels of theory as input for training and inference.

We will need to include >1 recipe for the inputs and the outputs to do delta learning


[common-ui] Add common classes for Recycler from production projects (module 'recycler')

Try to use "payloads" in the adapter
    override fun onBindViewHolder(
        holder: BindingViewHolder,
        position: Int,
        payloads: MutableList<Any>
    ) {
        super.onBindViewHolder(holder, position, payloads)
    }

Article https://medium.com/livefront/recyclerview-trick-selectively-bind-viewholders-with-payloads-4b28e3d2cce8


Fix build on wasm with serialization feature

serde only implements Serialize for OsString on unix and windows, so this didn't compile previously.
This PR just does the most minimal change of disabling the Serialize impl on wasm.

Thanks for the comment and the PR description! Might also be helpful to have a comment on the annotation saying why we are doing this, for future readers. :)

I've added a comment :)

Thanks!


Tags starting with numbers causes syntax error

Creating rules with tags that start with a numerical value fails YARA syntax checks and causes an error. While not common, there are situations where this functionality would be useful.
An example rule:
rule tag_number_test : malware 123rat
{
    strings:
        $ = "foo"

    condition:
        all of them
}

Running this rule results in the following error:

$ yara -r test.yara samples/
test.yara(1): error: syntax error, unexpected NUMBER, expecting '{'

Confirmed that this error occurs in YARA 1.7.2, 3.4, 3.5.

This is intentional. Tags must be valid identifiers and identifiers can not start with numbers. They follow the same rules for identifiers in C.

As @wxsBSD said this is intentional. Tokens starting with digits are treated as numbers, if we accept identifiers starting with numbers we can not distinguish if 100KB is an identifier or a number (like in filesize > 100KB)


Add Oracle metadata connector test

I'm adding a test case requested in the comment in #347.

@kpiotrowski The main point is to add the unit test for the addTasksTo method. I'll leave it up to you to decide if it is faster to merge this PR and then modify the test later or close this PR and add a test in #358.