make test failing

What did you do?
On HEAD of master:
GMCQUILLAN-2722:prometheus-operator gmcquillan$ git log -n 1
commit 20f1929124c23a47478e487e79ca35c858e173ea (HEAD -> master, origin/master, origin/HEAD)
Author: eedugon<EMAIL_ADDRESS>Date:   Wed Aug 2 12:09:44 2017 +0200

    Hints and notes added to handle grafana dashboards (#534)

Using latest stable golang:
GMCQUILLAN-2722:prometheus-operator gmcquillan$ go version
go version go1.8.3 darwin/amd64

Here's What I see
GMCQUILLAN-2722:prometheus-operator gmcquillan$ make test
# github.com/gmcquillan/prometheus-operator/pkg/migrator
pkg/migrator/migrator.go:73: cannot use client.Discovery() (type "github.com/gmcquillan/prometheus-operator/vendor/k8s.io/client-go/discovery".DiscoveryInterface) as type "github.com/coreos/prometheus-operator/vendor/k8s.io/client-go/discovery".DiscoveryInterface in argument to k8sutil.GetMinorVersion:
	"github.com/gmcquillan/prometheus-operator/vendor/k8s.io/client-go/discovery".DiscoveryInterface does not implement "github.com/coreos/prometheus-operator/vendor/k8s.io/client-go/discovery".DiscoveryInterface (wrong type for OpenAPISchema method)
		have OpenAPISchema() (*"github.com/gmcquillan/prometheus-operator/vendor/github.com/go-openapi/spec".Swagger, error)
		want OpenAPISchema() (*"github.com/coreos/prometheus-operator/vendor/github.com/go-openapi/spec".Swagger, error)
# github.com/gmcquillan/prometheus-operator/pkg/prometheus
pkg/prometheus/operator.go:101: cannot use &conf.TLSConfig (type *"github.com/gmcquillan/prometheus-operator/vendor/k8s.io/client-go/rest".TLSClientConfig) as type *"github.com/coreos/prometheus-operator/vendor/k8s.io/client-go/rest".TLSClientConfig in argument to k8sutil.NewClusterConfig
pkg/prometheus/operator.go:105: cannot use cfg (type *"github.com/coreos/prometheus-operator/vendor/k8s.io/client-go/rest".Config) as type *"github.com/gmcquillan/prometheus-operator/vendor/k8s.io/client-go/rest".Config in argument to kubernetes.NewForConfig
pkg/prometheus/operator.go:115: cannot use cfg (type *"github.com/coreos/prometheus-operator/vendor/k8s.io/client-go/rest".Config) as type *"github.com/gmcquillan/prometheus-operator/vendor/k8s.io/client-go/rest".Config in argument to clientset.NewForConfig
pkg/prometheus/operator.go:149: cannot use mclient.Prometheuses("github.com/gmcquillan/prometheus-operator/vendor/k8s.io/client-go/pkg/api".NamespaceAll).List (type func("github.com/coreos/prometheus-operator/vendor/k8s.io/apimachinery/pkg/apis/meta/v1".ListOptions) ("github.com/coreos/prometheus-operator/vendor/k8s.io/apimachinery/pkg/runtime".Object, error)) as type cache.ListFunc in field value
pkg/prometheus/operator.go:150: cannot use mclient.Prometheuses("github.com/gmcquillan/prometheus-operator/vendor/k8s.io/client-go/pkg/api".NamespaceAll).Watch (type func("github.com/coreos/prometheus-operator/vendor/k8s.io/apimachinery/pkg/apis/meta/v1".ListOptions) ("github.com/coreos/prometheus-operator/vendor/k8s.io/apimachinery/pkg/watch".Interface, error)) as type cache.WatchFunc in field value
pkg/prometheus/operator.go:152: cannot use "github.com/coreos/prometheus-operator/pkg/client/monitoring/v1alpha1".Prometheus literal (type *"github.com/coreos/prometheus-operator/pkg/client/monitoring/v1alpha1".Prometheus) as type "github.com/gmcquillan/prometheus-operator/vendor/k8s.io/apimachinery/pkg/runtime".Object in argument to cache.NewSharedIndexInformer:
	*"github.com/coreos/prometheus-operator/pkg/client/monitoring/v1alpha1".Prometheus does not implement "github.com/gmcquillan/prometheus-operator/vendor/k8s.io/apimachinery/pkg/runtime".Object (wrong type for GetObjectKind method)
		have GetObjectKind() "github.com/coreos/prometheus-operator/vendor/k8s.io/apimachinery/pkg/runtime/schema".ObjectKind
		want GetObjectKind() "github.com/gmcquillan/prometheus-operator/vendor/k8s.io/apimachinery/pkg/runtime/schema".ObjectKind
pkg/prometheus/operator.go:162: cannot use mclient.ServiceMonitors("github.com/gmcquillan/prometheus-operator/vendor/k8s.io/client-go/pkg/api".NamespaceAll).List (type func("github.com/coreos/prometheus-operator/vendor/k8s.io/apimachinery/pkg/apis/meta/v1".ListOptions) ("github.com/coreos/prometheus-operator/vendor/k8s.io/apimachinery/pkg/runtime".Object, error)) as type cache.ListFunc in field value
pkg/prometheus/operator.go:163: cannot use mclient.ServiceMonitors("github.com/gmcquillan/prometheus-operator/vendor/k8s.io/client-go/pkg/api".NamespaceAll).Watch (type func("github.com/coreos/prometheus-operator/vendor/k8s.io/apimachinery/pkg/apis/meta/v1".ListOptions) ("github.com/coreos/prometheus-operator/vendor/k8s.io/apimachinery/pkg/watch".Interface, error)) as type cache.WatchFunc in field value
pkg/prometheus/operator.go:165: cannot use "github.com/coreos/prometheus-operator/pkg/client/monitoring/v1alpha1".ServiceMonitor literal (type *"github.com/coreos/prometheus-operator/pkg/client/monitoring/v1alpha1".ServiceMonitor) as type "github.com/gmcquillan/prometheus-operator/vendor/k8s.io/apimachinery/pkg/runtime".Object in argument to cache.NewSharedIndexInformer:
	*"github.com/coreos/prometheus-operator/pkg/client/monitoring/v1alpha1".ServiceMonitor does not implement "github.com/gmcquillan/prometheus-operator/vendor/k8s.io/apimachinery/pkg/runtime".Object (wrong type for GetObjectKind method)
		have GetObjectKind() "github.com/coreos/prometheus-operator/vendor/k8s.io/apimachinery/pkg/runtime/schema".ObjectKind
		want GetObjectKind() "github.com/gmcquillan/prometheus-operator/vendor/k8s.io/apimachinery/pkg/runtime/schema".ObjectKind
pkg/prometheus/operator.go:234: cannot use c.kclient.Discovery() (type "github.com/gmcquillan/prometheus-operator/vendor/k8s.io/client-go/discovery".DiscoveryInterface) as type "github.com/coreos/prometheus-operator/vendor/k8s.io/client-go/discovery".DiscoveryInterface in argument to k8sutil.GetMinorVersion:
	"github.com/gmcquillan/prometheus-operator/vendor/k8s.io/client-go/discovery".DiscoveryInterface does not implement "github.com/coreos/prometheus-operator/vendor/k8s.io/client-go/discovery".DiscoveryInterface (wrong type for OpenAPISchema method)
		have OpenAPISchema() (*"github.com/gmcquillan/prometheus-operator/vendor/github.com/go-openapi/spec".Swagger, error)
		want OpenAPISchema() (*"github.com/coreos/prometheus-operator/vendor/github.com/go-openapi/spec".Swagger, error)
pkg/prometheus/operator.go:234: too many errors
# github.com/gmcquillan/prometheus-operator/pkg/api
pkg/api/api.go:43: cannot use cfg (type *"github.com/coreos/prometheus-operator/vendor/k8s.io/client-go/rest".Config) as type *"github.com/gmcquillan/prometheus-operator/vendor/k8s.io/client-go/rest".Config in argument to "github.com/gmcquillan/prometheus-operator/vendor/k8s.io/client-go/kubernetes".NewForConfig
pkg/api/api.go:99: cannot use "github.com/gmcquillan/prometheus-operator/vendor/k8s.io/apimachinery/pkg/apis/meta/v1".GetOptions literal (type "github.com/gmcquillan/prometheus-operator/vendor/k8s.io/apimachinery/pkg/apis/meta/v1".GetOptions) as type "github.com/coreos/prometheus-operator/vendor/k8s.io/apimachinery/pkg/apis/meta/v1".GetOptions in argument to api.mclient.Prometheuses(or.namespace).Get
pkg/api/api.go:108: cannot use api.kclient (type *"github.com/gmcquillan/prometheus-operator/vendor/k8s.io/client-go/kubernetes".Clientset) as type "github.com/coreos/prometheus-operator/vendor/k8s.io/client-go/kubernetes".Interface in argument to "github.com/coreos/prometheus-operator/pkg/prometheus".PrometheusStatus:
	*"github.com/gmcquillan/prometheus-operator/vendor/k8s.io/client-go/kubernetes".Clientset does not implement "github.com/coreos/prometheus-operator/vendor/k8s.io/client-go/kubernetes".Interface (wrong type for Admissionregistration method)
		have Admissionregistration() "github.com/gmcquillan/prometheus-operator/vendor/k8s.io/client-go/kubernetes/typed/admissionregistration/v1alpha1".AdmissionregistrationV1alpha1Interface
		want Admissionregistration() "github.com/coreos/prometheus-operator/vendor/k8s.io/client-go/kubernetes/typed/admissionregistration/v1alpha1".AdmissionregistrationV1alpha1Interface
# github.com/gmcquillan/prometheus-operator/pkg/alertmanager
pkg/alertmanager/operator.go:83: cannot use cfg (type *"github.com/coreos/prometheus-operator/vendor/k8s.io/client-go/rest".Config) as type *"github.com/gmcquillan/prometheus-operator/vendor/k8s.io/client-go/rest".Config in argument to "github.com/gmcquillan/prometheus-operator/vendor/k8s.io/client-go/kubernetes".NewForConfig
pkg/alertmanager/operator.go:93: cannot use cfg (type *"github.com/coreos/prometheus-operator/vendor/k8s.io/client-go/rest".Config) as type *"github.com/gmcquillan/prometheus-operator/vendor/k8s.io/client-go/rest".Config in argument to "github.com/gmcquillan/prometheus-operator/vendor/k8s.io/apiextensions-apiserver/pkg/client/clientset/clientset".NewForConfig
pkg/alertmanager/operator.go:109: cannot use o.mclient.Alertmanagers("github.com/gmcquillan/prometheus-operator/vendor/k8s.io/client-go/pkg/api".NamespaceAll).List (type func("github.com/coreos/prometheus-operator/vendor/k8s.io/apimachinery/pkg/apis/meta/v1".ListOptions) ("github.com/coreos/prometheus-operator/vendor/k8s.io/apimachinery/pkg/runtime".Object, error)) as type "github.com/gmcquillan/prometheus-operator/vendor/k8s.io/client-go/tools/cache".ListFunc in field value
pkg/alertmanager/operator.go:110: cannot use o.mclient.Alertmanagers("github.com/gmcquillan/prometheus-operator/vendor/k8s.io/client-go/pkg/api".NamespaceAll).Watch (type func("github.com/coreos/prometheus-operator/vendor/k8s.io/apimachinery/pkg/apis/meta/v1".ListOptions) ("github.com/coreos/prometheus-operator/vendor/k8s.io/apimachinery/pkg/watch".Interface, error)) as type "github.com/gmcquillan/prometheus-operator/vendor/k8s.io/client-go/tools/cache".WatchFunc in field value
pkg/alertmanager/operator.go:112: cannot use "github.com/coreos/prometheus-operator/pkg/client/monitoring/v1alpha1".Alertmanager literal (type *"github.com/coreos/prometheus-operator/pkg/client/monitoring/v1alpha1".Alertmanager) as type "github.com/gmcquillan/prometheus-operator/vendor/k8s.io/apimachinery/pkg/runtime".Object in argument to "github.com/gmcquillan/prometheus-operator/vendor/k8s.io/client-go/tools/cache".NewSharedIndexInformer:
	*"github.com/coreos/prometheus-operator/pkg/client/monitoring/v1alpha1".Alertmanager does not implement "github.com/gmcquillan/prometheus-operator/vendor/k8s.io/apimachinery/pkg/runtime".Object (wrong type for GetObjectKind method)
		have GetObjectKind() "github.com/coreos/prometheus-operator/vendor/k8s.io/apimachinery/pkg/runtime/schema".ObjectKind
		want GetObjectKind() "github.com/gmcquillan/prometheus-operator/vendor/k8s.io/apimachinery/pkg/runtime/schema".ObjectKind
pkg/alertmanager/operator.go:150: cannot use c.kclient.Discovery() (type "github.com/gmcquillan/prometheus-operator/vendor/k8s.io/client-go/discovery".DiscoveryInterface) as type "github.com/coreos/prometheus-operator/vendor/k8s.io/client-go/discovery".DiscoveryInterface in argument to k8sutil.GetMinorVersion:
	"github.com/gmcquillan/prometheus-operator/vendor/k8s.io/client-go/discovery".DiscoveryInterface does not implement "github.com/coreos/prometheus-operator/vendor/k8s.io/client-go/discovery".DiscoveryInterface (wrong type for OpenAPISchema method)
		have OpenAPISchema() (*"github.com/gmcquillan/prometheus-operator/vendor/github.com/go-openapi/spec".Swagger, error)
		want OpenAPISchema() (*"github.com/coreos/prometheus-operator/vendor/github.com/go-openapi/spec".Swagger, error)
pkg/alertmanager/operator.go:388: cannot use svcClient (type "github.com/gmcquillan/prometheus-operator/vendor/k8s.io/client-go/kubernetes/typed/core/v1".ServiceInterface) as type "github.com/coreos/prometheus-operator/vendor/k8s.io/client-go/kubernetes/typed/core/v1".ServiceInterface in argument to k8sutil.CreateOrUpdateService:
	"github.com/gmcquillan/prometheus-operator/vendor/k8s.io/client-go/kubernetes/typed/core/v1".ServiceInterface does not implement "github.com/coreos/prometheus-operator/vendor/k8s.io/client-go/kubernetes/typed/core/v1".ServiceInterface (wrong type for Create method)
		have Create(*"github.com/gmcquillan/prometheus-operator/vendor/k8s.io/client-go/pkg/api/v1".Service) (*"github.com/gmcquillan/prometheus-operator/vendor/k8s.io/client-go/pkg/api/v1".Service, error)
		want Create(*"github.com/coreos/prometheus-operator/vendor/k8s.io/client-go/pkg/api/v1".Service) (*"github.com/coreos/prometheus-operator/vendor/k8s.io/client-go/pkg/api/v1".Service, error)
pkg/alertmanager/operator.go:388: cannot use makeStatefulSetService(am) (type *"github.com/gmcquillan/prometheus-operator/vendor/k8s.io/client-go/pkg/api/v1".Service) as type *"github.com/coreos/prometheus-operator/vendor/k8s.io/client-go/pkg/api/v1".Service in argument to k8sutil.CreateOrUpdateService
pkg/alertmanager/operator.go:487: cannot use pod (type "github.com/gmcquillan/prometheus-operator/vendor/k8s.io/client-go/pkg/api/v1".Pod) as type "github.com/coreos/prometheus-operator/vendor/k8s.io/client-go/pkg/api/v1".Pod in argument to k8sutil.PodRunningAndReady
pkg/alertmanager/operator.go:592: cannot use c.kclient.CoreV1().RESTClient() (type "github.com/gmcquillan/prometheus-operator/vendor/k8s.io/client-go/rest".Interface) as type "github.com/coreos/prometheus-operator/vendor/k8s.io/client-go/rest".Interface in argument to k8sutil.WaitForCRDReady:
	"github.com/gmcquillan/prometheus-operator/vendor/k8s.io/client-go/rest".Interface does not implement "github.com/coreos/prometheus-operator/vendor/k8s.io/client-go/rest".Interface (wrong type for APIVersion method)
		have APIVersion() "github.com/gmcquillan/prometheus-operator/vendor/k8s.io/apimachinery/pkg/runtime/schema".GroupVersion
		want APIVersion() "github.com/coreos/prometheus-operator/vendor/k8s.io/apimachinery/pkg/runtime/schema".GroupVersion
pkg/alertmanager/operator.go:592: too many errors
# github.com/gmcquillan/prometheus-operator/cmd/operator
cmd/operator/main.go:101: cannot use r (type *"github.com/gmcquillan/prometheus-operator/vendor/github.com/prometheus/client_golang/prometheus".Registry) as type "github.com/coreos/prometheus-operator/vendor/github.com/prometheus/client_golang/prometheus".Registerer in argument to po.RegisterMetrics:
	*"github.com/gmcquillan/prometheus-operator/vendor/github.com/prometheus/client_golang/prometheus".Registry does not implement "github.com/coreos/prometheus-operator/vendor/github.com/prometheus/client_golang/prometheus".Registerer (wrong type for MustRegister method)
		have MustRegister(..."github.com/gmcquillan/prometheus-operator/vendor/github.com/prometheus/client_golang/prometheus".Collector)
		want MustRegister(..."github.com/coreos/prometheus-operator/vendor/github.com/prometheus/client_golang/prometheus".Collector)
cmd/operator/main.go:102: cannot use r (type *"github.com/gmcquillan/prometheus-operator/vendor/github.com/prometheus/client_golang/prometheus".Registry) as type "github.com/coreos/prometheus-operator/vendor/github.com/prometheus/client_golang/prometheus".Registerer in argument to ao.RegisterMetrics:
	*"github.com/gmcquillan/prometheus-operator/vendor/github.com/prometheus/client_golang/prometheus".Registry does not implement "github.com/coreos/prometheus-operator/vendor/github.com/prometheus/client_golang/prometheus".Registerer (wrong type for MustRegister method)
		have MustRegister(..."github.com/gmcquillan/prometheus-operator/vendor/github.com/prometheus/client_golang/prometheus".Collector)
		want MustRegister(..."github.com/coreos/prometheus-operator/vendor/github.com/prometheus/client_golang/prometheus".Collector)
make: *** [test] Error 2

Now that I look at it it seems to be the difference between running tests on a fork of the project vs. the main project. Any ideas for this?
(Note how the paths for imports are slightly different due to the forking: s/coreos/gmcquillan/)

Okay -- confirmed.
Tests only work if you:

git remote add <remote> <forked url>
git pull <remote> <branch>
make test

I'd argue that this is mainly golang quirky-ness, I'd recommend even for a fork to do development in $GOPATH/src/github.com/coreos/prometheus-operator. Do you want to add that to the dev guide?

Yeah, that's exactly what I was thinking -- just wanted to confirm that's the best idea. Marking as closed.


[identity] fix scope assignment logic

Packages impacted by this PR
@azure/identity
Issues associated with this PR
Describe the problem that is addressed by this PR
scopes assignment in managed identity leads to splitting of a single string into characters' array due to the previous logic. Even though functionally, it works as expected within msal, this is incorrect. The single string shouldn't split and just be there as an entry in the array.
Provide a list of related PRs (if any)
https://github.com/Azure/azure-sdk-for-js/pull/24103#issuecomment-1338220243
Command used to generate this PR:**(Applicable only to SDK release request PRs)
Checklists

[x] Added impacted package name to the issue description
[ ] Does this PR needs any fixes in the SDK Generator?** (If so, create an Issue in the Autorest/typescript repository and link it here)
[ ] Added a changelog (if necessary)

API change check
API changes are not detected in this pull request.


add one more embassy in the wild example

Want this?
It is a hobby project, so likely interesting only to beginners,

@lulf cool, I am happy :-) Thx!
Merge queue is stuck on that transient Bad Request thing, I have seen in another of my PRs here, though :-)


SAK-28996 Don't log error when a user can't be found.

The deleting of users is expected to happen and chat shouldn't fill the logs with errors when it comes across a message posted from a user who no longer exists.

+1


remove (some) Trello references

Since we've moved away from Trello, removing some of the documentation from the guidebook.  One thing to note, there are a lot of references in the guidebook to Trello boards still.  I don't know where they've moved to to update them

📚 Documentation preview 📚: https://civicactions-handbook--1465.org.readthedocs.build/en/1465/

Looks like the docs build didn't get kicked off on read the docs side. Might need to toggle the PR (close and re-open) to re-run all the checks and build.

Trello is likely being replaced with Jira and Confluence, but for now... thanks for removing links to Trello.


Added Kahlan in Testing

I would like to suggest to add the Kahlan library in your repository.
Kahlan is a "RSpec/JSpec-like" testing libraries (like Peridot or Pho) but with a wider scope. It's a full featured framework which bundles together all tools to speed up test developpement.
Main features are:

Simple API
Small code base (~10 times smaller than PHPUnit)
Fast Code Coverage metrics (xdebug required) (built-in exporter for Coveralls, Code Climate, Scrutinizer)
Handy stubbing system (no more mockery or prophecy required)
Set stubs on your class methods directly (i.e allows dynamic mocking)
Ability to Monkey Patch your code (i.e. allows replacement of core functions/classes on the fly without any PECL extension required)
Check called methods on your class/instances directly
Built-in Reporters/Exporters (Terminal, Coveralls, Code Climate, Scrutinizer, Clover)
Extensible, customizable workflow

:+1:

:+1:

+1

Not sure if this would change your mind, but Kahlan is now present in http://www.phptherightway.com/#testing and still the only one with Monkey Patching capabilities https://twitter.com/jeffrey_way/status/651051685197053952

:+1:

Hi @ziadoz, Kahlan has now 175 stars, so either merge or close this PR please.
Thank you.

@josegonzalez :+1:


English translation of the CurseForge description

Since English isn't your first language, I thought I'd give you a quick tip to help your mod present itself better to English speakers.
The description would be best translated as:

Bored by finding your diamonds legitimately? This mod is for you!

This will be much more readable to English speakers and will help your mod stand out better in the listing.

Thanks, it's edited and will be changed in future mod descriptions.


Cache data on the server with manually created caches

This is a breakout from #570 and replaces makeServerQueryCache in that PR. More background can be found there.
Before this PR, queries wont ever get cached in a server environment to avoid leaking sensitive information between requests (#70). This PR makes it so any cache that has been manually created via makeQueryCache will cache data, even on the server.
This change wont improve SSR (much) by itself but is a prerequisite. I'm pretty sure this PR is the only breaking change needed and that de/rehydration can be achieved without breaking changes.
This also adds some extra guards for scheduleStale and scheduleGarbage, since I don't believe these should ever be called on the server. Lifetime on the server should be manually managed (usually lifetime should be a single request).
I saw defaultConfig was added to makeQueryCache. I needed an extra frozen-option that I don't think makes sense as a cache config, but rather an option to makeQueryCache itself, so I took the liberty of changing the signature into an options-object instead if that's ok?
I'm not sure if the frozen-option should be documented and public or not and it would also be trivial to add .freeze() and .unfreeze()-methods, but I'm not sure if I see any use-cases for exposing this?
Just a heads up: I'm AFK for about a week from tomorrow, so while I can read and respond, actually fixing comments will take some time. 😄  I'll tackle next steps after that.

Alright, the next branch is pretty darn stable. If you could would you mind resolving any conflicts and updating any uses of makeQueryCache in the repo to take it into account?

NM, I got it merged :)

:tada: This PR is included in version 2.0.0-next.1 :tada:
The release is available on:

npm package (@next dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Definition list produced by option directive styled incorrectly

Problem
#838 fixed a bunch of issues about definition list (dl) with Sphinx 2, the simple definition list, option list, glossary terms are all styled correctly now, but the dl produced by the option directive seems still incorrect.

It should be styled as same as option list.
Environment Info

Python Version: 3.9.1
Sphinx Version: 3.5.1
RTD Theme Version: 0.5.1

By option do you mean .. option::?

@Blendify, exactly.

Technically, .. option:: is part of the reStructuredText domain which is different than an option list so it makes to style similar to the rest of the API domains. Maybe here we should make an expectation and style them differently. Something like this perhaps?

It looks better than the one I posted.


Zephir slow on modifying big array

I have the following code to benchmark zephir:
utils/example.zep
namespace Utils;

class Example {
public static function run() {
    array array3 = [];
    int i = 0;
    while (i < 1000000) {
        let array3[i] = i*2;
        let i++;
    }
    var start;
    let start = microtime(true);
    int length3;
    let length3 = count(array3);
    let i = 0;
    while (i < length3) {
        let array3[i] += 1;
        let i++;
    }
    echo (microtime(true) - start) . PHP_EOL;
}
}

zephir build && php -d extension=utils.so -r 'echo Utils\Example::run();'
php code:
<?php
function test(){
$array = array();
for ($i=0; $i<1000000; $i++) $array[] = $i*2;

$start = microtime(true);
$count = count($array);
for ($key=0; $key<$count; $key++) $array[$key]++;
echo microtime(true)-$start . PHP_EOL;
}
test();

Result is almost same as PHP code:
0.1663990020752
c++ is:
0.01149
c++ (-O3) is:
0.00101
hhvm is:
0.05
any ideas to improve my code?
zephir 0.7.1b
php 5.5.9
hhvm 3.8.1

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/25293970-zephir-slow-on-modifying-big-array?utm_campaign=plugin&utm_content=tracker%2F280146&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F280146&utm_medium=issues&utm_source=github).

Zephir does not have specialized structured for arrays different than the same structures for arrays provided by PHP so it's likely the same speed. Probably when PHP7 support is available it would improve.

Closed since that is unpreventable with the current array implementation.


Add helm chart

@rmstoi Is there a way to avoid duplicating the CRDs?

@rmstoi Is there a way to avoid duplicating the CRDs?

implemented make helm to copy them


Add support for display mode switching

In addition to standard FULLSCREEN support, we should add proper support for display mode switching. This may work better on some systems, and definitely will work better if using a lower resolution than the desktop.
Previously, this was blocked by no API for getting the available display modes for the monitor, but this was done months ago, so lets put this back on the wishlist :smile:

https://github.com/openfl/lime/commit/81c4926802f0be15b646aecb6af89dd5d7065b00


Save State Patch 3

Add a callback to storageAvaliable check to invoke LRU eviction if storage is available but full.
Add a clear local cache button to user preferences or as a menu option of hamburger menu (in the case of existing neuroglancer it will be a button), to empty out all local states manually.

Related:
Most used code paths interpret a full localStorage currently as "Local Storage not supported". That causes for example the "View states" button, that I would like to use to clear the storage, to return an exception "Cannot access saved states"...


x/vuln/client: revisit Client interface

Currently Client is defined as:
type Client interface {
	GetByModule(context.Context, string) ([]*osv.Entry, error)
	GetByID(context.Context, string) (*osv.Entry, error)
	GetByAlias(context.Context, string) ([]*osv.Entry, error)
	ListIDs(context.Context) ([]string, error)
	LastModifiedTime(context.Context) (time.Time, error)
}

Placing Get in getters is not common in Go code, how about renaming Get* methods to EntriesByModule?
As the database grows, ListIDs response size will keep growing. Current size of this response from vuln.go.dev is ~3KB, so that may not be too concerning. However, the response itself does not contain sufficient info so I guess most clients will have to call GetByID for each id every time after ListIDs. (See the example usage in x/pkgsite) However, most entries will remain unchanged even when the database is updated (indicated by the LastModifiedTime change). Why don't we make ListIDs return the last modified timestamp for each osv.Entry too? (Similar to  vulnerability database $base/index.json endpoint provides (i.e. (module path, timestamp)-pairs).

Another feature request from @alandonovan is the ability to batch lookups (e.g. GetByModules that takes a batch of module names).

Duplicate of an internal issue.


[Possible Bug(s)] Searches Failing to Load and Batch Downloads Not Completing

Running Grabber v7.3.2 on Windows 10 on PC
Issue 1: Searches Failing to Load (Possible bug?)
Issue Description:
Occasionally, when I hit the "ok" button to perform a search for a booru and hit "ok" again before it's able to finish loading everything, the results completely disappear for the booru. It's not that it shows "booru_name: no search results"; it just doesn't display anything.
Image of Issue:
Here's an example of what it looks like after hitting "ok" to search using safebooru and then hitting "ok" again before it's able to finish loading:

Replicating Issue:
I can sometimes replicate it by clicking "ok" and then clicking "ok" again before it finishes loading, but sometimes I have to click the "ok" button several times before it will trigger the issue. However, I think that in the cases where I have to click more than twice, it's because it's loading too quickly.
Troubleshooting:
So far I've tried using different builds of Grabber and the same issue occurs. The only way that I can "fix" the error is by restarting the application, entirely.
Log File for Example of Issue:
[21:17:37.742][Info] Loading results...
[21:17:37.744][Info] [safebooru.org][Html] Loading page https://safebooru.org/index.php?page=post&s=list&tags=kono_subarashii_sekai_ni_shukufuku_wo!&pid=0
[21:17:37.744][Info] [safebooru.org][Xml] Loading page https://safebooru.org/index.php?page=dapi&s=post&q=index&limit=100&pid=0&tags=kono_subarashii_sekai_ni_shukufuku_wo!
[21:17:38.779][Info] [safebooru.org][Xml] Receiving page https://safebooru.org/index.php?page=dapi&s=post&q=index&limit=100&pid=0&tags=kono_subarashii_sekai_ni_shukufuku_wo!
[21:17:38.850][Info] [safebooru.org][Xml] Parsed page https://safebooru.org/index.php?page=dapi&s=post&q=index&limit=100&pid=0&tags=kono_subarashii_sekai_ni_shukufuku_wo!: 100 images (100), 0 tags (676), 3702 total (3702), 38 pages (38)
[21:17:39.286][Info] [safebooru.org][Html] Receiving page https://safebooru.org/index.php?page=post&s=list&tags=kono_subarashii_sekai_ni_shukufuku_wo!&pid=0
[21:17:39.307][Info] [safebooru.org][Html] Parsed page https://safebooru.org/index.php?page=post&s=list&tags=kono_subarashii_sekai_ni_shukufuku_wo!&pid=0: 40 images (40), 54 tags (54), -1 total (3696), 88 pages (88)
[21:17:46.524][Info] Loading results...
[21:17:46.533][Info] [safebooru.org][Html] Loading page https://safebooru.org/index.php?page=post&s=list&tags=kono_subarashii_sekai_ni_shukufuku_wo!&pid=0
[21:17:46.533][Info] [safebooru.org][Xml] Loading page https://safebooru.org/index.php?page=dapi&s=post&q=index&limit=100&pid=0&tags=kono_subarashii_sekai_ni_shukufuku_wo!
[21:17:47.295][Info] [safebooru.org][Html] Receiving page https://safebooru.org/index.php?page=post&s=list&tags=kono_subarashii_sekai_ni_shukufuku_wo!&pid=0
[21:17:47.316][Info] [safebooru.org][Html] Parsed page https://safebooru.org/index.php?page=post&s=list&tags=kono_subarashii_sekai_ni_shukufuku_wo!&pid=0: 40 images (40), 54 tags (54), -1 total (3696), 88 pages (88)
[21:17:47.807][Info] Loading results...
[21:17:47.814][Info] [safebooru.org][Xml] Receiving page https://safebooru.org/index.php?page=dapi&s=post&q=index&limit=100&pid=0&tags=kono_subarashii_sekai_ni_shukufuku_wo!
[21:17:47.815][Warning] [Qt][default] QIODevice::read (QNetworkReplyHttpImpl): device not open
[21:17:47.815][Warning] [safebooru.org] Loading using Xml failed.

Issue 2: Batch Downloads Not Completing (Maximum Download limit?)
Issue Description:
Batch downloads are occasionally failing to complete and stopping about half-way through downloading.
Image of Issue:
It looks sort of like this when it happens (obviously it's not paused when it does it, though):

Replicating Issue:
I've been unable to determine precisely what is triggering it in order to replicate the issue, but it generally happens after a lot of batch downloads. As a very, very rough estimate, the issue occurs after downloading perhaps somewhere around ~2000-3000+ images total.
Troubleshooting:
I've tried closing the batch download window and starting the batch download again, but it doesn't download anything. I've also tried multiple versions of Grabber builds and the same issue persists. The only way that I've been able to fix it is by closing and restarting the entire application.
Issue 3: Images Failing to Load (Maximum Download limit?)
Issue Description:
Images in the image viewer are occasionally failing to load completely. They remain blurry as if they're trying to load, but fail to do so.
Image of Issue:
Here's an example of what the issue looks like when it happens:

Replicating Issue:
So far, I've been unable to determine what triggers it in order to replicate the issue, but it generally happens after viewing a lot of images or downloading a lot of images.
Troubleshooting:
The only way I can fix it is by closing and restarting the application, entirely. I've tried multiple versions of Grabber and the issue persists. Is there some sort of maximum download limit option that I'm missing that's causing both issue 2 and 3?
Thanks a lot for any assistance!

I've definitely experienced similar issues regarding having too many groups cued up causing a soft-crash. When it occurs, the only thing that seems to alleviate the issue is to close and boot-up the program again - which clears the saved groups.
Also, in the case where the image no longer loads when previewing it, I would double-check to see if the image exists on the website you're pulling from. I've had some experiences where the image grabber is able to find images that no-longer exist on the site.

Thanks for the response!
Closing it and rebooting it is the only way I've been able to fix it.
Yup, they all exist. When it no longer loads, I always try clicking on a lot of other images to check and it still doesn't manage to load.
I've tried fiddling with clicking on the program to get it to resume downloading when it "pauses", but it doesn't seem to get it to fire back up, again, unfortunately.

That's unfortunate to hear about the inability to resume downloads. I've been trying to ensure I'm not downloading more than 2000 or 3000 items at a time, or space them out if I can. I wonder if it's possible that this program takes a lot more computing or network power than it leads on.

That's entirely possible. It just gets tedious having to restart the program every few batch downloads or so.

That's unfortunate to hear about the inability to resume downloads. I've been trying to ensure I'm not downloading more than 2000 or 3000 items at a time, or space them out if I can. I wonder if it's possible that this program takes a lot more computing or network power than it leads on.

Encountered the bug again, not exactly sure why. But some photos as proof for it essentially being unable to access the internet after something triggers.

As you can see, there are other programs running and accessing the internet without an issue, but the grabber just suddenly disconnects itself.

A quick fix I actually found was going into settings and allowing the program to restore previous session and go to my AppData/local files for grabber and click on the "restore.igl" file. Surprisingly, it keeps all tabs open, all downloads queued, and seems to fix the issue with connectivity.
The restore last session on start  thing may not be necessary. But I think until the issue is pinned down, just refer to the restore.igl


Looking to Contribute

I'm interested in contributing to this project, how can I help?

I find this project on youtube. It's a very interesting idea！
And I imagined what it may look like in the end. Here are some of my thoughts.
Perhaps it is better to define instruments and songs separately. The definition of instruments should be defined before composition. And songs (or patterns) are created according to the definition of musical instruments.
In this case, users may have a more intuitive experience of some instruments that will be directly provided by plaintext-daw in the future.

Hey! Thanks, I'm really excited that you want to contribute!
I'm still just working my way through some very basic functionality, trying to make weekly progress. What are your interests? Maybe you can help with design and ideas. As things get larger, maybe you can be in charge of a specific subsystem if you want. Do you have experience with GUIs? That could be very helpful.
Would you be interested in collaborating on a future video? We could do some pair-programming or some kind of podcast discussing features. We have a PageKey Discord, we could chat/screenshare/talk there: https://discord.gg/5m5yFgDPF5
I like the idea of eventually defining instruments and songs separately. It would be cool if instruments could be defined in their own Git repo - to load an instrument, just provide a git@github.com:whatever/piano.git url or something like that.

Closing this out - you're a contributor! Looking forward to working with you more, thanks


Restore wallet via the GUI

bitcoin-qt has the Backup Wallet option (File -> Backup Wallet...).
But apparently there is no way to restore the wallet from a backup file via the GUI.
The only way to do this is to replace the wallet.dat of a newly created wallet with the backup file, which can hardly be considered an intuitive way.
Maybe it would be interesting to add a Restore Wallet From Backup option in bitcoin-qt.

This could be a good followup to https://github.com/bitcoin/bitcoin/pull/22541


Added composer.json and main.php

There is a banner "This package is not auto-updated. Please set up the GitHub Hook for Packagist so that it gets updated whenever you push!"
What do you think?

Yeah, you should set up it in the project's settings. Something like this:
https://www.w3resource.com/php/composer/how-to-auto-update-composer-packages-with-webhooks.php
So when you push a new version, it will automatically be synced and updated in packagist.org


Trigger options similar to UltiSnips

Hi!
I was wondering if like UltiSnips, nvim-snippy has options for triggers. For example, in UltiSnips
snippet ss1 "Subsection*" A
\subsection*{$1}

$0
endsnippet

would mean that typing ss1 would instantly place the template without pressing <Tab> (A in the first line means automatic). Is this sort of thing possible with nvim-snippy?

Hi, thanks for trying out Snippy.

I was wondering if like UltiSnips, nvim-snippy has options for triggers.

There is no support for UltiSnips-style auto-trigger at the moment. It sounds like a nice feature to have and also not too difficult to implement. I might try to implement it in the future if someone else doesn't in the meantime.

@dcampos Thank you! Unfortunately, my workflow requires auto-triggers along with other options that come with UltiSnips, my main reason for switching being responsiveness/speed purposes. Hopefully, the feature gets implemented soon into snippy. :-)

Hi!
What about the other trigger options? Namely the options for when to expand:
b for beginning of the line,
w on word boundary,
i inside word, (this one especially)
r expand on a regexp (probably not simple and probably not necessary at first).
This is currently the only thing holding me back from using snippy over ultisnips, so I would really appreciate those being implemented!

@AurelWeinhold Hi! I wasn't aware of options other than A and b. Thanks!
Although, I highly doubt these trigger options are implemented in snippy. @dcampos would know better.

The autotrigger feature has been implemented.


TypeError: this.import is not a function

I tried using the master branch but I'm getting the following error.
this.import is not a function
TypeError: this.import is not a function
    at CoreObject.module.exports.included (/Users/Chaitanya/Developer/Harvard/dotplot/node_modules/emberx-file-input/index.js:9:16)
    at CoreObject.superWrapper [as included] (/Users/Chaitanya/Developer/Harvard/dotplot/node_modules/ember-cli/node_modules/core-object/lib/assign-properties.js:32:18)
    at EmberApp.<anonymous> (/Users/Chaitanya/Developer/Harvard/dotplot/node_modules/ember-cli/lib/broccoli/ember-app.js:435:15)
    at Array.filter (native)
    at EmberApp._notifyAddonIncluded (/Users/Chaitanya/Developer/Harvard/dotplot/node_modules/ember-cli/lib/broccoli/ember-app.js:430:45)
    at new EmberApp (/Users/Chaitanya/Developer/Harvard/dotplot/node_modules/ember-cli/lib/broccoli/ember-app.js:109:8)
    at module.exports (/Users/Chaitanya/Developer/Harvard/dotplot/ember-cli-build.js:6:15)
    at CoreObject.module.exports.Task.extend.setupBroccoliBuilder (/Users/Chaitanya/Developer/Harvard/dotplot/node_modules/ember-cli/lib/models/builder.js:54:19)
    at CoreObject.module.exports.Task.extend.init (/Users/Chaitanya/Developer/Harvard/dotplot/node_modules/ember-cli/lib/models/builder.js:89:10)
    at CoreObject.superWrapper [as init] (/Users/Chaitanya/Developer/Harvard/dotplot/node_modules/ember-cli/node_modules/core-object/lib/assign-properties.js:32:18)

The one on npm is still giving me deprication warning:
DEPRECATION: Overriding init without calling this._super is deprecated. Please call `this._super.init && this._super.init.apply(this, arguments);` addon: `ember-cli-htmlbars`
    at Function.Addon.lookup (/Users/Chaitanya/Developer/Harvard/dotplot/node_modules/ember-cli/lib/models/addon.js:896:27)

Hmm, I'm going to guess it's because the addon that's doing the import polyfill needs to be moved into dependencies rather than devDependencies: https://github.com/thefrontside/emberx-file-input/blob/master/package.json#L27
Tomorrow is my open source day so I'll get things patched up :)
Thanks for opening an issue!

Hmm I'd like to get @xcambar's feedback on this. According to this discussion we shouldn't need change anything within our addon. The consuming app, if not on the latest ember-cli, would use this addon.
This means we should remove ember-cli-import-polyfill.

There's been a lot of noise/misunderstanding around the polyfill for which I'm accountable for. But please don't forget because of that that this.import is definitively the way to go. This is how asset imports should be done in addons from now on. To sum it up:

The polyfill is for the host app, it's not useful anywhere else (my bad)
Checking whether the host app uses the new import API is the responsibility of the addon  developer, unfortunately it does not come at no cost (though the total cost is 3 lines of code so...).

For the addons that want to support the whole range of ember-cli versions, here's a snippet for the included hook:
included: function(parent) {
  var import;
  if (this && typeof this.import === 'function') {
    import = this.import.bind(this);
  } else {
    import = parent.import.bind(app);
  }

  // Do your imports here
  import('path/to/my/assets/foo.js');
  import('path/to/my/assets/bar.js');
}

There's a (more general) discussion going on here: https://github.com/ember-cli/rfcs/pull/63


MM-19922 Add E2E tests for Mark as Unread

Based off https://github.com/mattermost/mattermost-webapp/pull/4225 since that fixes some of what we're testing here. The changes specific to this PR are here: https://github.com/mattermost/mattermost-webapp/commit/3517e32d7e6643cfde2d567905bdf8e74481c436
This adds tests to cover:

Marking a channel as unread by alt-clicking on a post
Marking a channel as unread from the post menu
Marking a channel as unread from another device
Ensuring that the channel is correctly read or unread when switching to or away from it

Ticket Link
https://mattermost.atlassian.net/browse/MM-19922

Removing QA label, Jelena on the feature branch spinmint tested

@saturninoabril Requesting your review here as a dev since you know how the E2E tests work

@hmhealey Feel free to merge anytime.

/update-branch

Will try to auto merge this PR

Pull Request successfully merged
SHA: a09d96c21d13cec9f66c28f7ced3228fc0140f7e


[Bug fix] A lax search for presence of binding during save page to match client algorithm to reduce page save error

Squash merging forcibly. All the server side tests have passed. Need to cherry pick to master.


Any possible alternatives to hipBLASLt?

Hi,
I'm really excited about this awesome project, however if I understand correctly only people with gfx90a and gfx94x based hardware can actually use it? Is far as I can tell ZLUDA is linking vs hipBLASLt, which only supports the above mentioned cards, everyone else is out of luck.
I am not an expert on the ROCm ecosystem yet, so perhaps you could provide some insight - are there alternative libraries which would work on other hardware and which could be used instead? How much of a hard requirement is hipBLASLt (say, if there was an option to make it a compile time option)?
Kind regards,
Jin

Not sure what you mean. For building ZLUDA you don't need even need a GPU. During run time if an application does not use cuBLASLt then hipBLASLt does not get loaded.
As for hipBLASLt alternatives, I don't know

Hmm, I was running the cargo build as described in the docs, it seemed to build some native (-sys) stuff and threw a link error, because it did not find hipBLASLt on my system, which was correct as I did not have it installed. I assumed that it's a must have dependency, hence my question, but I think I understand better now - what is actually being loaded and used will be decided during runtime, depending on the application.
Thank you for the clarification!
As for hipBLASLt alternatives - I guess I need to do my own research then, I am one of the unlucky MI25 gfx9000 owners.


Wrong labe on enter key when "enterKeyType" is default

Single line TextInput with default type should read "Ok" on enter key instead of "Return".

"Ok" is not available on iOS. We leave "Return" as it's default for the platform.


Improve PlotJugger installation

Checklist

[X] I've read the contribution guidelines.
[X] I've searched other issues and no duplicate issues were found.
[X] I've agreed with the maintainers that I can plan this task.

Description
PlotJugger installation was added in #243.
But it has a problem, it can't be used if it's not released.
So if we add the source in autoware.repos, it will still cause an error.
Since it's low-priority for Galactic, it's okay if we work on it after we move to Humble.
Purpose
To prevent an error when it's not released.
Possible approaches
It can be resolved by adding exec_depend to some packages as explained in https://github.com/orgs/autowarefoundation/discussions/235#discussioncomment-2638505.
There are some choices for where to place the depends.

autoware_launch
autoware_common/dev_tools (A new package)
etc.

Considering many packages depend on autoware_common, is autoware_launch better?
Definition of done

[ ] Where to place the depends is agreed.
[ ] The depends are added.
[ ] The behavior is confirmed.

I'll add an exec_depend after #300 is merged.


[EASI-3100] Update Model Categories

EASI-3100
Changes and Description

Update model categories to match CMS spec
Add additional categories

How to test this change

PR Author Review Checklist

[ ] Met the ticket's acceptance criteria, or will meet them in a subsequent PR.
[ ] Added or updated tests for backend resolvers or other functions as needed.
[ ] Added or updated client tests for new components, parent components, functions, or e2e tests as necessary.
[ ] Updated the Postman Collection if necessary.

PR Reviewer Guidelines

It's best to pull the branch locally and test it, rather than just looking at the code online!
Check that all code is adequately covered by tests - if it isn't feel free to suggest the addition of tests.
Always make comments, even if it's minor, or if you don't understand why something was done.

@OddTomBrooks @ClayBenson94 Check Migrations was failing because it detected a duplicate file. V102.... I renamed the file in commit.
Is that right? And if is there anythign else to do to address that failing test?

Noticed that the model categories on the home page are not rendering


Add inherently.xyz

This PR is:

[X] Adding a new domain
[ ] Updating existing domain
[ ] Changing domain name
[ ] Removing existing domain from list
[ ] Website code changes (darktheme.club site)
[ ] Other not listed

[X] The domain is in the correct alphabetical order
[X] The following information is filled identical to the data file

I confirm that I have read the FAQ section, particularly the two red items around minimal pages and inappropriate content and I attest that this site is neither of these things.

[X] Check to confirm

- domain: inherently.xyz
  url: https://inherently.xyz/
  method: mediaquery
  last_checked: 2022-07-17

I'm sorry about that merge conflict. 😕 Do you mind resolving it?

I think it should be fine now, let me know if any further changes are needed

Looks good to me. Thank you!


Error in ELECTRON_ASAR

I get this error in Windows x64 in the ELECTRON_ASAR.js line 173:
Uncaught Error: %1 is not a valid Win32 application.
\?(path to my app)\node_modules\spellchecker\build\Release\spellchecker.node
This same code is working correctly in macOS (latest version). Any guess to what might be happening?

Getting the  same error on  my machine too.

Any updates on this issue?  I'm getting the same exact error on windows.

@neurosnap Try installing this first https://github.com/felixrieseberg/windows-build-tools and then do npm install, worked for me.

Yes, for anyone else looking at this, each electron target must be built on their target platform.  Windows must be built on windows, linux on linux, and mac on mac.

Looks like this has been resolved.


update discover UI.

svg to ttf.
create custom web font icon.
add web fonts to iso and android.
Complete the discover component layout.

Coverage increased (+5.1%) to 58.571% when pulling 8eb82b87bb2bb964d379f16d76a70a45548b326b on travelgeezer:discover into 766fc855a7f4fda4e6806a42420b13113d209890 on phodal:master.


Vibration occurs when autocomplete items are selected.

VSCode Version: Code 1.11.0 (9122ec4b1068bd391e41974f46a4b54c1b9c90f9, 2017-04-05T20:52:01.201Z)
OS Version: Darwin x64 14.5.0
Extensions:

Extension
Author
Version

quick-open-file
AlanWalk
0.1.0

cpptools
ms-vscode
0.10.5

Steps to Reproduce:
Please see this video: http://take.ms/J7Gka

This looks like a duplicate of #23840


New version: ApproxFun v0.13.11

Registering package: ApproxFun
Repository: https://github.com/JuliaApproximation/ApproxFun.jl
Created by: @jishnub
Version: v0.13.11
Commit: 55653f5bb1d401206912e4a7e65b9a485e82d423
Reviewed by: @jishnub
Reference: https://github.com/JuliaApproximation/ApproxFun.jl/commit/55653f5bb1d401206912e4a7e65b9a485e82d423#commitcomment-85495557
Description: Julia package for function approximation

don't merge, version number skipped

This should have been superceded by https://github.com/JuliaRegistries/General/pull/69296, but I wonder if I had triggered the bot too quickly


ci: add CODEOWNERS file

:tada: This PR is included in version 1.3.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


How can I change the database that the application uses?

I wanted to change the database that my application uses, instead of using MySQL data it should use SQLite data.
I've tried this, but it does not work:
      Tpch1gApplication app = new Tpch1gApplicationBuilder()
                .withPassword("root")
                .withLogging(ApplicationBuilder.LogType.STREAM_OPTIMIZER)
                .withLogging(ApplicationBuilder.LogType.STREAM)
                .withLogging(ApplicationBuilder.LogType.APPLICATION_BUILDER)
                .withSkipCheckDatabaseConnectivity()
                .withDbms(NationManager.IDENTIFIER, dbms -> dbms.mutator().setTypeName("SQLite"))
                .build();

I have this error:
Exception in thread "main" com.speedment.runtime.core.exception.SpeedmentException: java.sql.SQLException: [SQLITE_ERROR] SQL error or missing database (no such table: nation)
	at com.speedment.runtime.core.internal.db.AbstractDbmsOperationHandler.executeQuery(AbstractDbmsOperationHandler.java:113)
	at com.speedment.runtime.core.internal.component.sql.SqlStreamSupplierImpl.executeAndGetLong(SqlStreamSupplierImpl.java:210)
	at com.speedment.runtime.core.internal.component.sql.override.optimized.util.CountUtil.countHelper(CountUtil.java:65)
	at com.speedment.runtime.core.internal.component.sql.override.optimized.reference.OptimizedCountTerminator.apply(OptimizedCountTerminator.java:45)
	at com.speedment.runtime.core.internal.manager.sql.SqlStreamTerminator.count(SqlStreamTerminator.java:156)
	at com.speedment.runtime.core.internal.stream.builder.ReferenceStreamBuilder.lambda$count$11(ReferenceStreamBuilder.java:333)
	at com.speedment.runtime.core.internal.stream.builder.AbstractStreamBuilder.finallyCloseLong(AbstractStreamBuilder.java:141)
	at com.speedment.runtime.core.internal.stream.builder.ReferenceStreamBuilder.count(ReferenceStreamBuilder.java:333)
	at Main.main(Main.java:20)

@solangepaz I think you are on the right track with this solution. For SQLite, you need to set the file name of the database. Unfortunately, the method for doing this is missing in the DbmsMutator. I have filed a separate issue (#731) for this. As a workaround until next version, you can do:
      Tpch1gApplication app = new Tpch1gApplicationBuilder()
                .withPassword("root")
                .withLogging(ApplicationBuilder.LogType.STREAM_OPTIMIZER)
                .withLogging(ApplicationBuilder.LogType.STREAM)
                .withLogging(ApplicationBuilder.LogType.APPLICATION_BUILDER)
                .withSkipCheckDatabaseConnectivity()
                .withDbms(NationManager.IDENTIFIER, dbms -> dbms.mutator().setTypeName("SQLite"))
                .withDbms(FilmManager.IDENTIFIER, d -> d.mutator().put(Dbms.LOCAL_PATH, "your-file-path"))
                .build();

(I assume that you have created the table nation in SQLite)

Thank you, this works. And can not I use the mutator without identifying the table?
Something like:
.withDbms (dbms-> dbms.mutator (). setTypeName ("SQLite"))

In the general case, there could be more than one DBMS. So that is the reason you have to give an identifier.


Use client-go/1.5 everywhere

Instead of using unversioned internal kubernetes client we will
use versioned public client go from 2.0.0 branch.
All changes in this pr are related to incompatibilities between
previous and current APIs.
Also it allows us to remove kubernetes from dependencies

This change is

Reviewed 2 of 32 files at r1, 38 of 38 files at r2.
Review status: all files reviewed at latest revision, 5 unresolved discussions.

glide.lock, line 243 at r1 (raw file):

  - 1.5/tools/metrics
  - 1.5/transport
- name: k8s.io/kubernetes

Should it still be here?

Makefile, line 12 at r2 (raw file):

test: vendor glide.lock Makefile
	go list ./... | grep -v /vendor/ | xargs go test -v

Why are you adding -v here? If the test fails it will print out its output, otherwise  IMO it's enough to report that tests passed.

mocks/petsets.go, line 29 at r2 (raw file):

)

type petSetClient struct {

Shouldn't it be changed to statefulSetClient?

resources/petset.go, line 17 at r2 (raw file):

package resources

import (

What about changing this file name to statefulset.go?

resources/petset_test.go, line 17 at r2 (raw file):

package resources

import (

Change filename?

Comments from Reviewable

Nice!

Reviewed 33 of 42 files at r2, 10 of 10 files at r3.
Review status: all files reviewed at latest revision, 5 unresolved discussions.

Comments from Reviewable

Reviewed 10 of 10 files at r3.
Review status: all files reviewed at latest revision, 4 unresolved discussions.

Comments from Reviewable


Avoid unwanted project persistence.

Session storage lasts for the lifetime of the tab, so we need to make sure subsequent migration URLs and iframe configurations work.
Motivated by issues with "Open in Classroom" and "Open in Python" on microbit.org.

I can switch between the two code snippets by navigating back/forward in this tab from the URLs you've provided (Ie this appears to fix the issue for me, here)


[5.5] Remove the between operator in basic where clause

It's currently not working and generating wrong sql:
\App\User::where('created_at', 'between', ['2017-01-01', '2017-01-30'])->get();

Will generate
select * from users where created_at between 2017-01-01

Should it not just be fixed?

Like whereIn I believe whereBetween should be used instead.

In Illuminate\Database\Query\Builder::where something like
if ($operator == 'between') {
    return $this->whereBetween($column, $value, $boolean);
}

would be useful in case of array of wheres
Model::where([
    ['active', 1],
    ['created_at', 'between', ['2017-01-01', '2017-01-07']],
]);

@aleksandrlat I'm ok with both ways, @taylorotwell what do you think?

I kind of agree people should just use whereBetween?

@themsaid this broke my code. Took me a few hours to track it down :cry:
I have a join with a BETWEEN where it checks if a given column's value is between two other columns:
$join->on('b.lft', 'BETWEEN', DB::raw('a.lft AND a.rgt'))

This is used to query a nested set table, and worked perfectly for years. This change broke it :disappointed:


Refine session related code: rename it, add tests, fix bugs, update docs

Description

Got rid of RigAuth and moved the Session module as well as the authorization-check modules under the RIG namespace.
Fix: A connection is now associated to its session right after the connection is established, given the request carries a JWT in its authorization header.
Added unit tests for the session module.

What to look out for
Dear reviewer, I want you to

gain knowledge - I think it's important for you to know about the change.
help to find bugs.

Have fun!

Running smoke tests and getting logs below, can you confirm?
╓─────[ Running tests, plaintext Kafka connection ]─────╖

Starting zookeeper ... done
Starting rest-api  ... done
Starting kafka     ... done
08:09:37.074 module=Rig.Config [error] Could not resolve HTTPS_CERTFILE: "cert/selfsigned.pem"
08:09:37.092 module=Rig.Config [error] Could not resolve HTTPS_KEYFILE: "cert/selfsigned_key.des3.pem"


Vickel: Watch cake-ideas.com

Vickel requests the watch of the watch_keyword cake-ideas\.com. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

cake-ideas\.com has been seen in 2 true positives, 0 false positives, and 0 NAAs.

Approved by Jeff Schaller in Charcoal HQ


Improve readability of methods on PyRef

For some reason it only occurred to me just now that we don't need the arbitrary-self-types rust feature to make the receiver be self. So now it is.
This also adds a PyStringRef alias, since PyRef<PyString> is a mouthful (and feels more natural given we have PyObjectRef).
(What really nice is that this doesn't just make our rust more idiomatic, it's also congruous with python self.)

Codecov Report

Merging #610 into master will decrease coverage by <.01%.
The diff coverage is 69.04%.

@@            Coverage Diff             @@
##           master     #610      +/-   ##
==========================================
- Coverage   40.86%   40.86%   -0.01%
==========================================
  Files          75       75
  Lines       17104    17087      -17
  Branches     4502     4499       -3
==========================================
- Hits         6990     6983       -7
+ Misses       8123     8114       -9
+ Partials     1991     1990       -1

Impacted Files
Coverage Δ

vm/src/obj/objstr.rs
34.34% <69.04%> (-0.12%)
:arrow_down:

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 06e17af...c68dbcc. Read the comment docs.


Added type casting for limiting min/max

Both 0 and "0" are valid json integers when representing min/max.
I added a cast to the limit to ensure proper comparison.

Could you please add a reference which backs your statement?
To me it seems like you'd like to have it this way, but the spec clearly says it has to be a JSON number.

I'm sorry - I think I pinned the blame on json-schema when it really belongs with raml-mocker.  I'm going to close this as the spec is pretty clear it needs to be an integer.


Move to yarn 3

📦 Published PR as canary version: 1.3.3--canary.69.6753c6f.0

:sparkles: Test out this PR locally via:
npm install<EMAIL_ADDRESS># or
yarn add<EMAIL_ADDRESS>

Sorry, to comment here, but I figured I would ask this before opening an issue.
Was prettier intended to be a production dependency?

Hey @TyIsI
Yes it was. For the auto-configuration code mods it runs prettier afterward to protect a users formatting


If user doesn't log out the uuid is kept in the db, but not used

This is a problem, because infinite uuid's could be stored.
It could be fixed by adding a maximum usable time to logged in user uuids.
Or if a user logs in and it already has a uuid created(because he/she didn't log out), use that uuid

To solve this I would add the time when a UUID is created to the DB so it would look like this: user + SEPCHAR + user_uuid + creation_time.
Then, every time the user does something with Hermes (send a message, get messages...) I would change the creation time to the current time.

That's a good solution


Deal with HRIT Full Disks / EMWIN from GOES-16

Quick fixes by adding the handlers for EMWIN files and Full Disks of GOES-16. The replay file of Monday (27th march) Tests  is here: http://www.teske.net.br/lucas/demuxdump-1490627438.bin

Done!


Add Clojure support.

Needs tests, haven't looked well into how you guys do those yet.
Anyone who's inspired to write them: feel free to do so.

See CONTRIBUTING.md


Cannot run examples

After I installed package and start a very simple script, import error suddenly happened, and I don't know how other people can smoothly pass it. I use typescript 4.5.0, ts-node 8.0.0。
This is my code
import { wormhole } from "@wormhole-foundation/sdk";
import evm from "@wormhole-foundation/sdk/evm";

async function main() {
  const wh = await wormhole("Testnet", [evm]);
  console.log(wh);

  // console.log(await evm());
}

main();

The error is:
Failed to load required packages TypeError: Cannot convert undefined or null to object
    at Function.entries (<anonymous>)
    at loadProtocols (/Users/jiyuan.zheng/projects/chain-build/packages/test/node_modules/@wormhole-foundation/sdk/src/index.ts:88:18)
    at /Users/jiyuan.zheng/projects/chain-build/packages/test/node_modules/@wormhole-foundation/sdk/src/index.ts:64:56
    at Array.map (<anonymous>)
    at loadPlatforms (/Users/jiyuan.zheng/projects/chain-build/packages/test/node_modules/@wormhole-foundation/sdk/src/index.ts:64:33)
    at async wormhole (/Users/jiyuan.zheng/projects/chain-build/packages/test/node_modules/@wormhole-foundation/sdk/src/index.ts:102:14)
    at async main (/Users/jiyuan.zheng/projects/chain-build/packages/test/scripts/wormhole.ts:5:14)
Failed to load required packages TypeError: Cannot convert undefined or null to object
    at Function.entries (<anonymous>)
    at loadProtocols (/Users/jiyuan.zheng/projects/chain-build/packages/test/node_modules/@wormhole-foundation/sdk/src/index.ts:88:18)
    at /Users/jiyuan.zheng/projects/chain-build/packages/test/node_modules/@wormhole-foundation/sdk/src/index.ts:64:56
    at Array.map (<anonymous>)
    at loadPlatforms (/Users/jiyuan.zheng/projects/chain-build/packages/test/node_modules/@wormhole-foundation/sdk/src/index.ts:64:33)
    at async wormhole (/Users/jiyuan.zheng/projects/chain-build/packages/test/node_modules/@wormhole-foundation/sdk/src/index.ts:102:14)
    at async main (/Users/jiyuan.zheng/projects/chain-build/packages/test/scripts/wormhole.ts:5:14)
TypeError: Cannot convert undefined or null to object
    at Function.entries (<anonymous>)
    at loadProtocols (/Users/jiyuan.zheng/projects/chain-build/packages/test/node_modules/@wormhole-foundation/sdk/src/index.ts:88:18)
    at /Users/jiyuan.zheng/projects/chain-build/packages/test/node_modules/@wormhole-foundation/sdk/src/index.ts:64:56
    at Array.map (<anonymous>)
    at loadPlatforms (/Users/jiyuan.zheng/projects/chain-build/packages/test/node_modules/@wormhole-foundation/sdk/src/index.ts:64:33)
    at async wormhole (/Users/jiyuan.zheng/projects/chain-build/packages/test/node_modules/@wormhole-foundation/sdk/src/index.ts:102:14)
    at async main (/Users/jiyuan.zheng/projects/chain-build/packages/test/scripts/wormhole.ts:5:14)

This is due to evm is an async function, but the result still is a object that have a key default. so right know I only have to rewrite node-modules code to pass this, can anyone help me how to fix it?

I think the problem probably is here: sdk/src/evm.ts, need to export two default:
(await import("./platforms/evm.js")).default.default

@MarkJiYuan what does this line mean? Where is that applied? Could you provide a code example? We are having exactly the same issue with the solana environment.

bump typescript to > 5.0
prefer tsx to ts-node

If those don't fix it please lmk


Compile Postgres and Extension with -Werror

I've noticed, that we started to get a compile warning again:
/Users/runner/work/neon/neon//pgxn/neon/libpagestore.c:430:15: warning: incompatible function pointer types assigning to 'dbsize_hook_type' (aka 'long (*)(unsigned int)') from 'const int64 (Oid)' (aka 'const long (unsigned int)') [-Wincompatible-function-pointer-types]
                dbsize_hook = zenith_dbsize;
                            ^ ~~~~~~~~~~~~~
1 warning generated.

https://github.com/neondatabase/neon/runs/8225115360?check_suite_focus=true#step:11:4621
This was fixed in https://github.com/neondatabase/postgres/pull/189, but accidentally brought back by the next PR: https://github.com/neondatabase/postgres/pull/188. I don't really know if these changes could affect us in a bad way.
I think it would be nice to catch such kinds of warnings on CI, so I'm adding -Werror to Makefiles for the extension.
Initially, I wanted to pass -Werror as a flag on CI, but apparently my make-fu is not strong enough for this, so suggestions on how to do it (and not break the build) are welcome. On the other hand, currently, we don't have any warnings, so having this in makefiles by default should help us to catch them earlier during the local development.

Putting this in PG_CFLAGS in the top-level Makefile might be even better. That would also catch warnnigs/errors coming from the changes we've made to vendor/postgres.

Added to PG_CFLAGS, and passed to extension explicitly via CFLAGS.

Interesting, adding -Werror results in configure to fail with Cannot find a working 64-bit integer type.. And googling this particular error message gets a lot of Postgres-related questions (mostly from psql-hackers and StackOverflow): https://www.google.com/search?q=Cannot+find+a+working+64-bit+integer+type

Interesting, adding -Werror results in configure to fail with Cannot find a working 64-bit integer type.

Usually this has to do with the order of such flags, i.e.:
int main(void) {
    int a; // -Werror=unused-variable
    return 0;
}

gcc test.c -Wall                    # OK
gcc test.c -Wall -Werror            # FAIL
gcc test.c -Wall -Werror -Wno-error # OK, the latter takes precedence
gcc test.c -Wall -Wno-error -Werror # FAIL, the latter takes precedence

Configure's tests essentially just try to compile the code in order to check whether a feature is available, so this flag might hinder them. You might want to take a look at configure.log, it should contain the exact reason.

Interesting, adding -Werror results in configure to fail with Cannot find a working 64-bit integer type.

Usually this has to do with the order of such flags, i.e.:

Thanks for the explanations!
Postgres doc recommends setting -Werror flag via COPT:

An important example is that gcc's -Werror option cannot be included in the CFLAGS passed to configure, because it will break many of configure's built-in tests. To add such flags, include them in the COPT environment variable while running make.

So, I've changed the approach: I'm setting COPT=-Werror in codestyle.yml and passing COPT to the extensions CFLAGS`


Add node-ray-cli docs

This PR adds documentation for bash/cli debugging with permafrost-dev/node-ray-cli.  It is based on the Ray integration package node-ray.
Specifically, it adds pages for:

installation
configuration
usage
command reference

Thanks!


Cow: Watch nanda.ca

Cow requests the watch of the watch_keyword nanda\.ca. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

nanda\.ca has been seen in 1 true positive, 0 false positives, and 0 NAAs.

Approved by Jeff Schaller in Charcoal HQ


ModelParallel not working

Hi,
I am running the AlexNet cudnn training example on Ubuntu 14.04 and getting this error
after mean image is computed.
torch/fbcunn/examples/imagenet/models/alexnet_cudnn.lua:8: attempt to call field 'ModelParallel' (a nil value)
stack traceback:
.../torch/fbcunn/examples/imagenet/models/alexnet_cudnn.lua:8: in function 'createModel'
/home/omkar/torch/fbcunn/examples/imagenet/model.lua:25: in main chunk
[C]: in function 'dofile'
main.lua:29: in main chunk
[C]: in function 'dofile'
...mkar/torch/install/lib/luarocks/rocks/trepl/scm-1/bin/th:131: in main chunk
[C]: at 0x00406170

fixed now in master. there was a missing "require"


Question: Always add to Target Dependencies

I have a project where we want to add linked or embedded binaries always also to the Target Dependencies of the target to make them declared as specific as possible. On the scheme level we have find/build explicit dependencies turned off.
However, most of the time targets are only added to "linked" or "embedded" dependencies. Is there a way to force targets to always be added to the "target dependencies"?

Specifically, this happens with SPM package dependencies. If we define the SPM dependency using:
      - package: SPMPackage
        product: SPMPackageProduct
        link: false

then it shows up in the Xcode Target Dependencies section properly. However, if we instead do:
      - package: SPMPackage
        product: SPMPackageProduct
        link: true

then it shows up only in Link Binary With Libraries. embed: true / embed: false does not seem to make a difference in the outcome.
We want a way of having it show up under both Target Dependencies and Link Binary With Libraries. We've identified a workaround of defining the dependency twice, once with link: true and a second time with link: false, but it would be better to only need to define the dependency once.


Cranelift: The clif-util run command encountered a segmentation fault

I constructed a clif file.
multi_func15.zip
I used the following command to run it.
clif-util run -v multi_func15.clif

But I got the following error:
Segmentation fault (core dumped)

The test probably reads or writes to memory it shouldn't. In any case please reduce your test cases rather than making others try to extract the issue from a ton of clif ir.

The test probably reads or writes to memory it shouldn't. In any case please reduce your test cases rather than making others try to extract the issue from a ton of clif ir.

Ok, I will try to reduce the size of the test case.

@bjorn3 @cfallin  Hi, I've extracted the function where caused the segmentation fault to reduce this test case.
test optimize
set opt_level=speed
set enable_pinned_reg=true
set preserve_frame_pointers=true
target s390x
target aarch64
target riscv64
target x86_64

function %main(i64, i32) -> i64, f32 fast {
    ss0 = explicit_slot 8
    ss1 = explicit_slot 8
    sig0 = (i32, i64) -> f64, i32 fast

    const0 = 0x74b82aa0f192f5649620f8c17a3f6964
    const1 = 0xa2aa8b9ee040395c

block0(v2: i64, v3: i32):
    v4 = iconst.i8 92
    v5 = iconst.i16 1372
    v6 = iconst.i32 -493943460
    v7 = iconst.i64 0x78be_9a03_e28f_055c
    v8 = f16const 0x0.ef4p-14
    v9 = f32const 0x1.be8908p-3
    v10 = f64const 0x1.ceaf296167eb8p-1
    v11 = vconst.i8x16 const0
    v12 = vconst.i16x8 const0
    v13 = vconst.f16x8 const0
    v14 = vconst.i32x4 const0
    v15 = vconst.f32x4 const0
    v16 = vconst.i64x2 const0
    v17 = vconst.f64x2 const0
    v18 = vconst.i8x8 const1
    jump block2

block1:
    v31 = stack_addr.i32 ss0
    v32 = uload32 v31+4
    v33 = icmp.i16 slt v5, v5  ; v5 = 1372, v5 = 1372
    v34 = stack_addr.i32 ss0
    v35 = load.i64x2 v34+3
    v36 = uextend.i64 v27
    v37 = f64const 0x1.b67f3058b67e0p-2
    return v36, v9  ; v9 = 0x1.be8908p-3

block2:
    brif.i32 v6, block3, block6  ; v6 = -493943460

block3:
    v19 = rotr_imm.i16 v5, -17672  ; v5 = 1372
    v20 = urem.i64 v7, v7  ; v7 = 0x78be_9a03_e28f_055c, v7 = 0x78be_9a03_e28f_055c
    v21 = band.f16 v8, v8  ; v8 = 0x0.ef4p-14, v8 = 0x0.ef4p-14
    jump block5(v6, v4)  ; v6 = -493943460, v4 = 92

block5(v60: i32, v61: i8):
    v22 = stack_addr.i32 ss0
    v23 = load.i32 v22+1
    v24 = stack_addr.i32 ss1
    v25 = load.f16 v60+2
    v26 = f32const 0x1.a28e48p-2
    v27 = ishl_imm v60, -1794784619
    v28, v29 = umul_overflow.i8 v4, v4  ; v4 = 92, v4 = 92
    v30 = stack_addr.i32 ss0
    istore16.i32 v6, v60  ; v6 = -493943460
    jump block1

block6:
    v38 = stack_addr.i32 ss0
    v39 = load.i64 v38+4
    v40 = ineg.i64x2 v16  ; v16 = const0
    v41 = iconst.i32 -794804681
    v42 = popcnt.i16 v5  ; v5 = 1372
    jump block7

block7:
    v43 = udiv.i16 v5, v42  ; v5 = 1372
    v44 = srem.i64 v39, v2
    jump block9

block9:
    v45 = swizzle v11, v11  ; v11 = const0, v11 = const0
    v46 = stack_addr.i32 ss0
    v47 = load.i8 v46+5
    brif.i32 v3, block10, block11

block10:
    v48 = trunc.f32x4 v15  ; v15 = const0
    v49 = stack_addr.i32 ss0
    v50 = load.f64 v49+4
    v51 = stack_addr.i32 ss1
    v52 = atomic_load.i64 v51
    v53 = stack_addr.i32 ss1
    v54 = sload8.i16 v53+1
    v55 = stack_addr.i32 ss1
    v56 = sload8.i16 v55+1
    v57 = urem.i32 v41, v3  ; v41 = -794804681

    v63 = f64const 0x1.ceaf296167eb8p-1
    v64 = iconst.i32 100

    jump block12(v52)

block11:
    jump block12(v39)

block12(v62: i64):
    v58 = fvpromote_low v15  ; v15 = const0
    v59 = stack_addr.i32 ss0
    atomic_store.i64 v39, v59
    jump block5(v41, v47)  ; v41 = -794804681
}

; print: %main(-1, -1725923514)

After my analysis, the segmentation fault may occur in the following code block. I think it is due to memory out of bounds for the v25 = load.f16 v60+2 instruction.
block5(v60: i32, v61: i8):
    v22 = stack_addr.i32 ss0
    v23 = load.i32 v22+1
    v24 = stack_addr.i32 ss1
    v25 = load.f16 v60+2
    v26 = f32const 0x1.a28e48p-2
    v27 = ishl_imm v60, -1794784619
    v28, v29 = umul_overflow.i8 v4, v4  ; v4 = 92, v4 = 92
    v30 = stack_addr.i32 ss0
    istore16.i32 v6, v60  ; v6 = -493943460
    jump block1

And the following instruction with segmentation fault might be the assembler instruction corresponding to v25 = load.f16 v60+2.

Hi @abc767234318 -- to be blunt, bug reports with this much un-minimized detail and this little analysis are not very useful to us. You are likely generating this from fuzzing or some other random testing strategy (yes?) and submitting any crashes you find. That takes very little effort. The hard part is determining whether the issue is "real" -- that is, due to a bug in Cranelift, rather than your testing infrastructure or your assumptions.
When we get a flood of reports like this (7 from you in the past 6 days, by my count) it is basically performing a human denial-of-service attack on our project. We are interested in bugs, of course, but the social contract of open-source is that you need to participate fairly and do your share of the work.
Could you please, for this and any future bug reports you submit, add analysis that indicates why you believe the program should not behave the way it does? Here, for example, could you indicate why you believe the load should not be out of bounds, what the value of the address operand is, whether the code runs successfully on other ISAs or the CLIF interpreter if applicable, and any other questions you can think of to help reduce the issue?

Hi @abc767234318 -- to be blunt, bug reports with this much un-minimized detail and this little analysis are not very useful to us. You are likely generating this from fuzzing or some other random testing strategy (yes?) and submitting any crashes you find. That takes very little effort. The hard part is determining whether the issue is "real" -- that is, due to a bug in Cranelift, rather than your testing infrastructure or your assumptions.
When we get a flood of reports like this (7 from you in the past 6 days, by my count) it is basically performing a human denial-of-service attack on our project. We are interested in bugs, of course, but the social contract of open-source is that you need to participate fairly and do your share of the work.
Could you please, for this and any future bug reports you submit, add analysis that indicates why you believe the program should not behave the way it does? Here, for example, could you indicate why you believe the load should not be out of bounds, what the value of the address operand is, whether the code runs successfully on other ISAs or the CLIF interpreter if applicable, and any other questions you can think of to help reduce the issue?

I apologize for not providing sufficient detail and analysis in my reports. I appreciate the importance of thorough investigations in identifying real issues, especially in an open-source project. I want to clarify that my reports are generated from fuzzing and random testing strategies, as you suspected, and I understand that this approach may not always yield the most actionable insights. My current understanding of cranelift compiler and its internals is limited, which has contributed to the lack of detailed analysis in my reports.
I will strive to include more context and reasoning in my reports. I’ll make sure to analyze the behavior of the program more thoroughly, including addressing the specific points you mentioned.
Thank you for your patience, and I appreciate your guidance as I work to improve my contributions.

Not directly applicable to clif test cases, but you may find https://docs.wasmtime.dev/contributing-reducing-test-cases.html enlightening.


Fix ErrorProne warnings in :model-core

Reviewing cheatsheet
Before merging the PR, comments starting with

❌ ❓must be fixed
🤔 💅 should be fixed
💭 may be fixed
🎉 celebrate happy things

@bot-gradle test this

I've triggered the following builds for you. Click here to see all build failures.

PullRequestFeedback build


Address finder doesn't show addresses with single number

Can't get a screenshot sorry.
When 1 Brown street entered, only 11 brown street appears etc

1 brown street does not exist
1 broadwater does .


Happa on gorgoth loads clusters.cluster.x-k8s.io/v1alpha3 and fails

Seen on gorgoth (KVM). The problem doesn't occur on gopher.

I did a bit of investigating: for KVM installations, users logged in via SSO (i.e. GS staff) use the MAPI for organizations management. The MAPI UI for organizations is loaded as a result. However, cluster management with the MAPI is not supported, and any subsequent requests to list clusters for that UI (via the MAPI) should fail.
I think we're actually seeing the expected result on gorgoth. On other KVM installations, the request doesn't fail, but the list of clusters returned is empty.
Currently, this only affects GS staff on KVM, as customers there are not logged in via SSO.

What I don't understand: why is there a difference between the installations? An empty list sounds like the desired response. A 404 error does not.

I’m not sure why there is a difference between installations. I saw this change, and from my understanding that meant Cluster CRs are not ensured for KVM installations any more, but maybe that was the wrong conclusion?


project vanished from hacs

System Health details
System Information

version
core-2024.5.5

installation_type
Home Assistant OS

dev
false

hassio
true

docker
true

user
root

virtualenv
false

python_version
3.12.2

os_name
Linux

os_version
6.6.28-haos-raspi

arch
aarch64

timezone
Europe/London

config_dir
/config

Home Assistant Community Store

GitHub API
ok

GitHub Content
ok

GitHub Web
ok

GitHub API Calls Remaining
4852

Installed Version
1.34.0

Stage
running

Available Repositories
1394

Downloaded Repositories
20

Home Assistant Cloud

logged_in
false

can_reach_cert_server
ok

can_reach_cloud_auth
ok

can_reach_cloud
ok

Home Assistant Supervisor

host_os
Home Assistant OS 12.3

update_channel
stable

supervisor_version
supervisor-2024.05.1

agent_version
1.6.0

docker_version
25.0.5

disk_total
57.3 GB

disk_used
24.9 GB

healthy
true

supported
true

board
rpi4-64

supervisor_api
ok

version_api
ok

installed_addons
Duck DNS (1.17.0), Mosquitto broker (6.4.1), WireGuard (0.10.2), Advanced SSH & Web Terminal (18.0.0), File editor (5.8.0), ESPHome (2024.5.4), Log Viewer (0.17.0), Zigbee2MQTT (1.38.0-1), Whisper (2.1.0), Piper (1.5.0), Portainer (2.20.1), HassOS I2C Configurator (0.14), I2C OLED Display (1.0.5), pigpio (1.5.2), Epic Games Free (debian-2024-04-28), go2rtc (1.9.2), Speedtest (1.6.2), MQTT Explorer (browser-1.0.3), Neolink-latest (0.0.1), Network UPS Tools (0.13.0), Samba share (12.3.1), MeshCentral (1.1.22)

Dashboards

dashboards
5

resources
8

views
19

mode
storage

Recorder

oldest_recorder_run
24 May 2024 at 20:24

current_recorder_run
2 June 2024 at 22:44

estimated_db_size
653.79 MiB

database_engine
sqlite

database_version
3.44.2

Checklist

[X] I have enabled debug logging for my installation.
[X] I have filled out the issue template to the best of my ability.
[X] This issue only contains 1 issue (if you have multiple issues, open one issue for each issue).
[X] This issue is not a duplicate issue of currently previous issues..

Describe the issue
this project seems to have vanished from hacs?
Reproduction steps

try searching for prject in hacs

...
Debug logs
no debug logs

Diagnostics dump
No response

Any update on this? I too cannot find rpi_gpio_pwm in HACS.

Well, originally, it was listed in the HACS, but it's not longer there and still isn't today?

The only way to get it to appear is to add your repo as a repository in HACS for it to then appear

I can agree - this doesn't show up, but pressing the 3 dots in the top right of HACS, and adding the repo as an integration did the trick for me (Repo: https://github.com/RedMeKool/HA-Raspberry-pi-GPIO-PWM)
Not sure why this doesn't appear in HACS natively, I'm not sure how that works.


🍒 Cherry pick PR #18619 to staging 🍒

🍒 Cherry pick https://github.com/Expensify/App/pull/18619 to staging 🍒

🚀 Cherry-picked to staging by https://github.com/OSBotify in version: 1.3.11-5 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

@Expensify/applauseleads please QA this PR and check it off on the deploy checklist if it passes.

🚀 Deployed to production by https://github.com/roryabraham in version: 1.3.12-0 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

🚀 Deployed to production by https://github.com/AndrewGable in version: 1.3.28-5 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅


5991: Adds shortcodes to missions and responses

woo!

The failing spec is the SMS decoding ones again -- it seems to be something going on in the environment on travis because I can't reproduce those failures locally using the same seed.

Sonofagun!
On Fri, Mar 17, 2017 at 2:48 PM, Melody<EMAIL_ADDRESS>wrote:

The failing spec is the SMS decoding ones again -- it seems to be
something going on in the environment on travis because I can't reproduce
those failures locally using the same seed.
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
https://github.com/thecartercenter/elmo/pull/300#issuecomment-287440461,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAq3cNAuiVMhDGwM2Hz8tUgvn0R8VARBks5rmtVigaJpZM4Md7mD
.

--
Tom Smyth
Worker-Owner, Sassafras Tech Collective
Specializing in innovative, usable tech for social change
sassafras.coop · @sassafrastech


iOS bug

I got app crash  on iPhone 14 Pro Max when test on real device
*** Terminating app due to uncaught exception 'NSRangeException', reason: 'Cannot remove an observer <real_volume.VolumeListener 0x282dd85c0> for the key path "outputVolume" from <AVAudioSession 0x283a4d290> because it is not registered as an observer.'
*** First throw call stack:
(0x1983cc870 0x1906cbc00 0x19736eba8 0x19736e8e8 0x10ba02a24 0x10ba02338 0x10ba023e4 0x110af4e98 0x110558e08 0x10dd84f50 0x10dd86b34 0x10dd96fec 0x10dd96ba0 0x19831720c 0x198313f18 0x198313668 0x1db6de5ec 0x19a7302b4 0x19a72f8f0 0x1042b0f78 0x1bad66dcc)
libc++abi: terminating due to uncaught exception of type NSException
flutter doctor
Doctor summary (to see all details, run flutter doctor -v):
[✓] Flutter (Channel stable, 3.13.9, on macOS 14.1.1 23B81 darwin-x64, locale en-EG)
[✓] Android toolchain - develop for Android devices (Android SDK version 34.0.0)
[✓] Xcode - develop for iOS and macOS (Xcode 15.0.1)
[✓] Chrome - develop for the web
[✓] Android Studio (version 2022.3)
[✓] VS Code (version 1.84.2)
[✓] Connected device (3 available)
[✓] Network resources
• No issues found!

@heshesh2010 Thanks for the feedback I'll have a look at it asap.

@heshesh2010  Unfortunately I wasn't able to reproduce this error. Can you please provide a code example on how you use the library?

Closing due to inactivity

Any updates? I started getting this error.
Using 1.0.8

Any updates ?

@Roee-Tsur  @shaharhn  Can you test with the latest version 1.0.9?

I am getting this error still. on 1.0.9 version.


Update wal.md

update exemplar record type

@songjiayang You need to signoff your commit for the DCO to pass.

DCO

Thx, have done.


update README - fix links to license file

Fixes
Checklist

[ ] I have made a material change to the repo (functionality, testing, spelling, grammar)
[ ] I have read the [Contribution Guide] and my PR follows them.
[ ] I updated my branch with the master branch.
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] I have added necessary documentation about the functionality in the appropriate .md file
[ ] I have added in line documentation to the code I modified

Short description of what this PR does:

fix incorrect links to license file

If you have questions, please send an email to Sendgrid, or file a Github Issue in this repository.

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

@krlevkirill
We have not been able to merge your Pull Request, but because you are awesome - we wanted to make sure you could still get a SendGrid Hacktoberfest shirt.
Please go fill out our swag form before Nov 5th and we will send the shirt! (We know that you might have tried this before and it didn’t work, sorry about that!)
You have till Nov 5th to fill out this form in order to get the Hacktoberfest shirt!
Thank you for contributing during Hacktoberfest! We hope to see you in the repos soon! Just so you know, we always give away a SendGrid shirt for your first ever non-Hacktoberfest PR that gets merged.

Hello @krlevkirill,Thanks again for the PR! We appreciate your contribution and look forward to                                              continued collaboration. Thanks!                                             Team SendGrid DX

If you are interested, we would love the opportunity to talk to you about Hacktoberfest and our API libraries.
Our agenda would be: Explore what you liked and is there anything we can do to improve?
You can grab a time on my calendar that works for you and we can have a chat on Google Hangout or Skype. If you prefer, you can email me using my GitHub username at my company’s domain.
Thank you so much,
Matt Bernier - @mbernier - SendGrid Developer Experience Product Manager
Elmer Thomas - @thinkingserious - SendGrid Developer Experience Engineer
@SendGrid


DRAFT: CGroup OS Structure and MemoryCurrent property

This pull request didn't trigger Jenkins as its author isn't in the whitelist.
An organization member must perform one of the following:

To have this branch tested by Jenkins, use the "ok to test" command.
To have a one time test done, use the "test this please" command.

Those commands are simple Github comments of the format: "jenkins: COMMAND"

Aborting to work based off of https://github.com/lxc/lxd/pull/6581


xtensa: Fix allocation of FPU registers in exception context

Summary
This PR intends to fix the allocation of FPU (and other coprocessors) registers in exception context.
Impact
Fix FPU Test in ostest for Xtensa chips.
Solved memory corruption reported in #6914:
user_main: FPU test
Starting task FPU#1
fpu_test: Started task FPU#1 at PID=4
FPU#1: pass 1
Starting task FPU#2
fpu_test: Started task FPU#2 at PID=5
FPU#2: pass 1
FPU#1: pass 2
FPU#2: pass 2
FPU#1: pass 3
FPU#2: pass 3
FPU#1: pass 4
FPU#2: pass 4
FPU#1: pass 5
FPU#2: pass 5
FPU#1: pass 6
FPU#2: pass 6
FPU#1: pass 7
FPU#2: pass 7
FPU#1: pass 8
FPU#2: pass 8
FPU#1: pass 9
FPU#2: pass 9
FPU#1: pass 10
FPU#2: pass 10
FPU#1: pass 11
FPU#2: pass 11
FPU#1: pass 12
FPU#2: pass 12
FPU#1: pass 13
FPU#2: pass 13
FPU#1: pass 14
FPU#2: pass 14
FPU#1: pass 15
FPU#2: pass 15
FPU#1: pass 16
FPU#2: pass 16
FPU#1: Succeeded
FPU#2: Succeeded
fpu_test: Returning

Testing
Successful execution of ostest in the following targets:

esp32-devkitc:nsh
esp32-devkitc:smp
esp32s3-devkit:nsh (CONFIG_XTENSA_CP_INITSET=0x001 and CONFIG_XTENSA_CP_INITSET=0x009)
esp32s3-devkit:smp (CONFIG_XTENSA_CP_INITSET=0x001 and CONFIG_XTENSA_CP_INITSET=0x009)

@no1wudi Please, take a look these patches.


Fix error in flow types

Previously type Options had excludeActivityTypes property, but in RCTActionSheetManager.m and docs this property is named excludedActivityTypes.

Want to fix error in flow types
Test Plan
Actually, it seems there aren't any need for test plan for this PR. Just fix a mistyping error.

@facebook-github-bot shipit


Thrillseekr patch 2

Updated Readme.md
Update Readme.md with badges.
Proposed Changes
Added informative badges related to repository and tech stack.
Check List (Check all the applicable boxes)

[x] New Feature(s) added
[x] Documentation changed
[x] My code follows the code style of this project.

Screenshots (if any)
NA
Additional Information
NA

Updated Readme.md
Update Readme.md with badges.
Proposed Changes
Added informative badges related to repository and tech stack.
Check List (Check all the applicable boxes)

[x] New Feature(s) added
[x] Documentation changed
[x] My code follows the code style of this project.

Screenshots (if any)

Additional Information
NA

Checks are failing because someone from team will review it.
Please do the needful to merge this PR.
Below is the screenshot of check fail.

A few stacks are missing please add these as well(Solidity, MetaMask, Auth0, ThirdWeb).

Sure


add gitattributes file to force CRLF line endings for dos.stl

The most recent release falls short of 100% test coverage, because dos.stl doesn't have CRLF line endings. This should fix the problem.

Thank you :)
Sometimes git is a bit too smart ;)


Replace fromNow-util with Intl.RelativeTimeFormat

We have fromNow which is obsolete because we can use Intl.RelativeTimeFormat, https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Intl/RelativeTimeFormat/RelativeTimeFormat#basic_format_usage

@jrieken Are you suggesting that fromNow, would internally use the new APIs? Because looking at the format API there doesn't look like there is a way to format a date automatically based on the distance -- as you have to supply a particular unit.

Yes, I'd say it is simple and much less code to pick a unit and then use the built-in API

Well the built-in API doesn't support the shortened strings we use in the Timeline view -- so we could use the built-ins for other usages just not the Timeline view column.

Well the built-in API doesn't support the shortened strings we use in the Timeline view

Can you explain that further?

We use abbreviated strings in the Timeline view -- hr = hour, mo = month, wk = week, etc, but AFAICT there is no abbreviated format with the built-ins.

Like so? (running short and/or narrow)

const rtf = new Intl.RelativeTimeFormat("en", {
    localeMatcher: "best fit", // other values: "lookup"
    numeric: "always", // other values: "auto"
    style: "long", // other values: "short" or "narrow"
});

Kind of, but we don't add the . nor the ago


Update website docs now the help repo has issue templates

https://github.com/jazzband-roadies/help/pull/197 has been merged, which closed https://github.com/jazzband-roadies/help/issues/188.
That said, we should update the website to tell users to select the correct template, rather than asking to apply labels.

(I will do this, feel free to assign me!)

Thanks @jstockwin!


Read/Write Channel Group Name not propagated in Unflatten Asynchronous Execution Units

Describe the bug
Read/Write Channel Group Name needs to be bundled when unflattening asynchronous execution units.
To Reproduce
Steps to reproduce the behavior:

Go to Unflatten Asynchronous Execution Units.vi
See issue in that Read/Write Channel Group Name element is not propagated.

Expected behavior
Pass all elements of compiled asynchronous execution units.
Screenshots

Desktop (please complete the following information):

OS: Windows
Version: 10 Enterprise

Additional context
linked to Issues #118, #117

@douglasnorman


fix(connlib): Persist logging guard for the duration of the session

This allows the file logger to write events as they're emitted so that we (attempt to) capture everything for the lifetime of the session.
 I was thinking about maybe logging it in json format instead, wdyt?

@conectado TunnelStats is pretty huge and not well formatted for JSON logs. I was thinking about maybe logging it in json format instead, wdyt?

I'd suggest to instead use structured logs and avoid debug-printing big structs. See https://docs.rs/tracing/latest/tracing/index.html#recording-fields for details.

@thomaseizinger I think I've addressed your feedback.

@conectado TunnelStats is pretty huge and not well formatted for JSON logs. I was thinking about maybe logging it in json format instead, wdyt?

I'd suggest to instead use structured logs and avoid debug-printing big structs. See https://docs.rs/tracing/latest/tracing/index.html#recording-fields for details.

I'm in favor of this. Then, @jamilbk we can enable json print using tracing-subscriber's JSON layer


Accordian: use hidden-until-found internally to support browser native "Find in Page"

Feature Type
Enhancement
Feature Description

Reference: https://developer.chrome.com/docs/css-ui/hidden-until-found
Caveat: This is for collapsible sections that exist in the DOM, so not applicable for sections hidden with ngIf/@if

Current Behavior:
Try searching for the term "open source" on this page: https://angularprimitives.com/primitives/accordion.
Expected Behavior:
Browser-native search should work for hidden/collapsed sections.

Thank you very much for the suggestion! I actually came across this same article a week or two ago myself. I haven't implemented it yet because the feature is still marked as experimental and has limited browser support:

https://caniuse.com/?search=hidden%3Duntil-found
But I will leave this issue open as this is definitely something I would like to do as soon as possible. Great suggestion, thank you!

Yeah, that makes sense! I saw that it has been implemented in React Aria as a part of the Disclosure component so I thought it would be great to include here as well. Plus, it is a progressive enhancement and doesn't impact browsers that don't support it.

Ah interesting, if that's the case then we could probably add this for browsers that support it! Thank you 😊


fix: quote object mock key if it is an identifier name

Status
READY
Description
If the OpenAPI specification uses properties with special characters like @, the generated mocks are invalid JS since the object key is not a valid identifier name.
In this case, the key needs to be quoted.
Related PRs
N/A
Todos

[x] Tests
[ ] Documentation
[ ] Changelog Entry (unreleased)

Thanks @alanpoulain  🎉


Missing returned field in README: copyright?

When reading through the docs and looking at the examples, it seems like the Copyright field isn't listed as something that can be returned? Are there any more fields that haven't been added?
Another short question, is this the API that is being served currently from https://api.nasa.gov/planetary/apod? If yes, then how come many of the fields aren't documented on https://api.nasa.gov/?

The same goes for ServiceVersion.


LC Mapper needs to reconcile earlier/later established forms

they're often, if not always, bnodes.

an example: http://id.loc.gov/authorities/subjects/sh85147710

closed by #150


Trigger load

Lint test files
Add triggerLoad function ref #42

Misses tests. A working example is available on the demo.
Uses the matches API.

What do you think @ApoorvSaxena ?

also, please fix the conflicts

Done.

Very cool! Could the npm package be updated as well?

@lonepalmus sorry for the delay, have updated the package version

Cool! and thanks for a great library!


About: Alphabetize Board Report Types + 2 other updates

It's been a bit since we reviewed the About Us page, and there a a couple of areas I'd like to update! It's been getting some attention internally because the Report Type definitions are really useful, so I hope this updates make them even more:

Remove the description of "Types of Board Reports" (highlighted in the snip below) - the current text is a little off and I think the section speaks for itself.

Alphabetize the Board Report Types. It looks like they're organized by meta-type, but I think alphabetic organization would be easier to use for most users.

Change the name of what is currently the last item in the list, "Board Box" to "Board Box / Board Correspondence" and update the definition to read: “Formal information communication from Metro staff to the Board, which does not require Board action. Historically, these items were called "Board Box"(es), because a physical box containing each month's items was delivered to the Board for review. Because these items are now delivered digitally, their "Board Report Type" designation is "Board Correspondence" so the nature of these items is more clear to the public. We are in the process of importing Board Boxes to this website; in the meantime, please access Board Box/Correspondence items through the [Archive Search](https://boardagendas.metro.net/archive-search. You can also access a directory of Board Correspondence from 2015 to present, by year, here: http://boardarchives.metro.net/BoardBox/"

@shrayshray Deployed to the live site


Implement StreamBuilder in the Widget Catalog

StreamBuilder Widget needs to be added to the Widget Catalog. Make sure to include the description of the widget in the Description tab and add an argument to include link to the official docs of the widget (Refer to any example in homepage.dart for better understanding. Also add an argument for category of the widget and give it a category accordingly. Showcase proper implementation of the widget and feel free to have a creative approach.
Please attach a screen recording of the same once done in your PR.

@AryanKhubchandani Pls assign this issue to me.

@AryanKhubchandani Pls assign this issue to me.

Sure, good luck!