why etcd or zk different from consul？

https://docs.docker.com/swarm/discovery/
Start the Swarm manager on any machine or your laptop.
Etcd:
swarm manage -H tcp://<swarm_ip:swarm_port> etcd://<etcd_addr1>,<etcd_addr2>/
Consul:
swarm manage -H tcp://<swarm_ip:swarm_port> consul://<consul_addr>/
ZooKeeper:
swarm manage -H tcp://<swarm_ip:swarm_port> zk://<zookeeper_addr1>,<zookeeper_addr2>/
as the above, etcd have addr1 and addr2,zk also have addr1 and addr2. consul have only addr.
If the only one consul_addr failed, how to implement the HA of swarm manage.
Can consul have several addrs？

The consul client only allows for a single IP-address, see https://github.com/docker/swarm/pull/583 and https://github.com/hashicorp/consul/issues/849
I'll close this issue because this is not a bug report; please keep in mind that the GitHub issue tracker is for reporting bugs and feature requests, and not a general support forum. For other questions, use

https://forums.docker.com
the docker community slack channel https://blog.docker.com/2016/11/introducing-docker-community-directory-docker-community-slack/

Feel free to comment though, after I closed this issue

Also; the external key/value stores are only needed for "classic" Swarm (https://github.com/docker/swarm). The services feature ("swarm mode") introduced in docker 1.12 no longer needs an external key/value store, so may be of interest to you; https://docs.docker.com/engine/swarm/


Bug in WhisperResult.force_order

I have received the following error message :
File "/usr/local/lib/python3.8/dist-packages/stable_whisper/result.py", line 485, in {}init{}
  self.force_order()
File "/usr/local/lib/python3.8/dist-packages/stable_whisper/result.py", line 529, in force_order
  ts.end = ts.start if i == len(timestamps) else timestamps[i+1].start
IndexError: list index out of range

This error depens on icorrect timestamp indexing
See the following PR: https://github.com/jianfch/stable-ts/pull/270

Thanks for the PR. Merged in 0430a31dc8fe05f4bc4a3cf39c28074d7afc79dd.


Does not respond to verification request of unspecified type

Describe the bug
When another Matrix client starts a verification request, pantalaimon does not answer back if the request didn't specify a particular verification type, such as emoji comparison. Only when a verification request specifies a type does pantalaimon respond.
Even then, sometimes you have to restart the verification for it to work all the way through.
To Reproduce
Steps to reproduce the behavior:

Sign in to a Matrix account with Element Web (or a self-hosted version of it) or Element Android (or one of its forks)
Sign in to the same account with pantalaimon
In Element, click/tap the prompt to verify the pantalaimon session
See that pantaliamon never responds with a DBus notification, and that Element waits forever
Cancel the verification request from Element, and use other menus to start an emoji verification request
See that pantalimon responds, and accept the verification request
Potentially see that pantalaimon never shows the emoji to verify (but Element does). Repeat from step 5 if necessary

Expected behavior
pantaliamon should respond to a generic verification request. Perhaps pan needs to answer with which verification types it supports.
Desktop (please complete the following information):

OS: Firefox 32
Browser: Firefox 81

That's a long standing issue where Pantalaimon doesn't support the new cross signing style verification flows. We are blocked behind nio getting cross signing support which in turn is blocked behind the rust-sdk crypto layer getting on par with the nio one.
We are now pretty close to being on par but we're still talking about a month at least untill I integrate it into nio properly.

Any update on this?


New Terraform version has been released

@Andrew.Copeland has made some changes to our terraform provider in order to support the new version of terraform that came out.
The objective of this card is :

To ensure all changes are in
To release the binaries of the provider

Also see
https://github.com/cyberark/terraform-provider-conjur/issues/26

Waiting on this as some of the other providers used already upgraded and Conjur has been the only one which is not compatible (so enforcing baseline version :( ).
Cheers.

Closed with release v0.3.0: https://github.com/cyberark/terraform-provider-conjur/releases/tag/v0.3.0.


Refactor endianness, byte swaps, and indexers to be more standardized.

I'm trying to understand why:

Only the ubuntu tests are failing.
Only the ubuntu tests are run on all python versions.
Why are the pre-commit caches skipped when running these tests.

I'm trying to understand why:

Only the ubuntu tests are failing.
Only the ubuntu tests are run on all python versions.
Why are the pre-commit caches skipped when running these tests.

Tests are resolved now.
Running all Pythons on ubuntu but not others is deliberate, no need to run all versions on all OS. We need to test for OS and python compatibilty on one OS.
not sure


#2ewen7y Include source to nef generation

Summary or solution description
Included source field to NeoMetadata.
How to Reproduce
https://github.com/CityOfZion/neo3-boa/blob/e5137e4be022e9021025ba162172122cd4766b29/boa3_test/test_sc/metadata_test/MetadataInfoSource.py#L9-L13
Tests
https://github.com/CityOfZion/neo3-boa/blob/e5137e4be022e9021025ba162172122cd4766b29/boa3_test/tests/compiler_tests/test_metadata.py#L453-L462
Platform:

OS: Windows 10 x64
Python version: Python 3.7+

Task linked: CU-2ewen7y Include source to nef generation

Task linked: CU-2y8grne Invalid contract is generated when using classes and setting _deploy method in the same smart contract


add on onKeyPress event

thanku very much sir i learnt a lot from you.....

ok sir ...sorry for disturbing you !!


51 refactor aggregates part of the documentation

I made some changes to the Aggregates field in the python's documentation
Splitting the main points and adding some examples to make it clear

Is the introduction not redundant with the "Object types" section ?


Regression in BM_PathVolatilityTracker

See https://flutter-engine-perf.skia.org/e/?begin=1618949947&end=1619040543&keys=X48f6fd7f6cb56e212be6c28a24afacda&num_commits=50&request_type=1&xbaroffset=22269.
Looks like it started with commit https://github.com/flutter/engine/pull/25689, a Skia roll.

Resolving; this only affects tests.


(puppet-ccs_hcu) add apache license to repo

Ticket of reference: https://rubinobs.atlassian.net/browse/IT-5292

@glennmorris Are you OK with this?

@glennmorris Are you OK with this?

OK with me.
(The metadata file already says that this is the license.)


npm ERR! cb() never called!

Seemingly  out of nowhere npm install starts throwing the npm ERR! cb() never called error, for some packages (angular-cli and autoprefixer-core) but not all (bower and yeoman install fine for example). The versions of npm and node don't seem to matter.

Operating system and version:
Ubuntu 16.10

nvm debug output:

npm install --verbose
...
# this mostly happens after this, though caniuse-db installs just fine, if I install it specifically
# via npm install caniuse-db
npm verb afterAdd /home/johannes/.npm/caniuse-db/1.0.30000602/package/package.json written
npm ERR! cb() never called!

nvm ls output:

->       v5.9.1
         v6.9.1
         v7.0.0
         system
default -> 5.9.1 (-> v5.9.1)
node -> stable (-> v7.0.0) (default)
stable -> 7.0 (-> v7.0.0) (default)
iojs -> N/A (default)
lts/* -> lts/boron (-> v6.9.1)
lts/argon -> v4.6.1 (-> N/A)
lts/boron -> v6.9.1

How did you install nvm? (e.g. install script in readme, homebrew):
Install script.

This is issue is related: https://github.com/angular/angular-cli/issues/3771#issuecomment-269619094

Oh. Sorry, wrong repository for this issue.

np, this is an issue for npm itself.


Moving all ipcRenderer calls out of jsx components to be shared

Very nice! replacing IPCrenderer with comm looks very nice. I also really appreciate the use of moment and the general code clean-up.


Research and test of possible presence sensors

GPTree should start a conversation when a human shows up before the tree or passes the tree.
Requirements:

2 to 8 m detections distance
3.3 V operation
some RasPi code would be nice

Hardware
Should be a radar sensor working on high temperature environment and sunlight, see: https://www.youtube.com/watch?v=PNbAM9IhfBE (at 11:14)
A good overview gives Andreas Spiess in
Radar Switches: Vergleich und Tests
https://www.youtube.com/watch?v=9WiJJgIi3W0&t=14s
Only the HFS-DC06 in this video is working at 3.3 V! All other radar modules need 5V. But also some 5 V sensors like the RCWL-0516 work at 3.3 V logic level, see https://randomnerdtutorials.com/esp32-rcwl-0516-arduino/ .
This video is around 6 years old, so it may be good to have a look on an update:
Radar Sensors from $3 to over $100: Which one is Best?
https://www.youtube.com/watch?v=s-GzUTyIH9c
With a overview table under https://drive.google.com/drive/folders/1Tslck6CkSz_NcNoW2aIw0u0UAqh7oL49
Software

Only the HFS-DC06 in this video is working at 3.3 V! All other radar modules need 5V. But also some 5 V sensors like the RCWL-0516 work at 3.3 V logic level, see https://randomnerdtutorials.com/esp32-rcwl-0516-arduino/ .

I ordered this two HFS-DC06 5.8GHz Radar Sensor Modules https://www.amazon.de/dp/B081J44XCK and tried it on an ESP32. Unfortuatelly is not reliable working at 3.3 V, it is better with 5 V so I assume that at least this module needs 5 V for stable working. Sensor voltage level seems to be Vin level and is not limited -- at least not to 3.3 V.

Hint from Effective Ways To Detect People With Common Sensors https://www.youtube.com/watch?v=LNeSz5hfTLI Sensors like the mmWave Radar https://www.dfrobot.com/product-2282.html from DFRobot do not only detect movement but also presence. So also peple who stay on the tree can detected. And you can detect the absence of a person "Oh, you're leaving, what a pitty, I'd like to keep talk with you!" But 10x more expensive!

Today I tested the RCWL-0516 with a RasPi and ESP32 and it works good. I tested it with 3.3 V and 5 V Vin and the module is working on both voltage levels, while digital output is still 3.3 V level! Data sheet says 4 V recommended at least.
As with many motion sensors, there is a delay of approx. 2-3 s if the sensor has detected "no movement", so it stays 2-3 s off even if there is movement before the sensor. And it seems to be a bit less sesitive than a PIR sensor; very slow movements don't trigger it.
Detection range (with 3.3 V) is about 4-5 m, but I have to test it more detailed in the field (not in the office).
OT: The pin with "3V3" lable on the module is juste a constant 3V3 source and does not change by movement or not, so not needed for us.


React 0.14

Merge master into react-0.14
Updated to latest beta version
Use react-dom for rendering html
Fix warnings in examples

Awesome stuff, thanks! Hopefully we are right around the corner for React 0.14 and can officially release :-)

Great - can you publish this as a beta version to npm?

You know how that works? Trying to find some documentation. Thought I could just checkout the branch and publish it with --tag beta?

Yup, just use npm publish --tag beta and it won't be installed by default.

Hm, for some reason it states that I can not publish over the previous version. Maybe I still have to give it a new version number, but using --tag beta it will not be installed by default?

Yeah, change the version number to 0.14.1-beta1 or something. Unfortunate version number coincidence with react 0.14 :)

ah, yes, there we go:
npm install formsy-react@beta :-)

nice - thanks!

Would be great if you could confirm that everything works as expected, and especially if not :-)

Not working - fixed in https://github.com/christianalfoni/formsy-react/pull/193


fix: replace custom padding with standard bootstrap

This change has already been made in https://github.com/appfolio/react-gears/commit/55d359848e02ce11a4c9277d75846a34eabb4115


config: reinstate #8478 (unification of delta and SotW xDS), reverted by #8939

Delta and state-of-the-world gRPC xDS now share the same GrpcMux and
Subscription implementations. These are the implementations added in #7293 for
delta. These implementations are a rewrite/extensive refactoring of the old
GrpcMuxImpl (deleted by this PR) and associated logic.
Risk Level: high (rewrite/large refactor of gRPC xDS)
Testing: added sotw_subscription_state_test, added regression test for #8927

There was a minor merge conflict with #8886 to resolve. While doing that I noticed that some comments in new_grpc_mux_impl.cc wrongly referred to "DeltaDiscoveryRequest" when they should say a generic "discovery request". I can undo that if you'd prefer it to be a pure revert revert. Other than the new_grpc_mux_impl.cc cleanup and the WellKnownGrpcStatus merge, this should be identical to #8478.

@fredlas can you check CI?
@kyessenov once we get CI passing do you mind giving this branch a quick smoke test however you tested master last time? Thank you.
/wait

Sure, waiting for the CI to pass.

Done, updated some more GrpcStatuses. I'm also seeing that any _speed_test test is failing to build, due to a problem when trying to link with libpthread. I'm not sure if that's a newly uncovered quirk with just my own machine's environment, or a recently introduced actual breakage that hasn't yet been caught and fixed, but it's hard to imagine something in these changes causing it (and not causing it before). I guess we'll see once the tests get there.

@kyessenov looks good for a quick smoke test. Thank you! cc @wgallagher
/wait-any

I ran go-control-plane CI test, and it passes fine now. Thanks.

Awesome, thanks. I'm relieved it got back in so quickly!

@fredlas @mattklein123 hmm so apparently this change causes some of our instances to get stuck on startup -- unclear if it's CDS or SDS or something in XDS but something with XDS broke which prevents some of our binary/config configurations to start... Any ideas before I start carefully reading this PR?
cc: @fishcakez

@rgs1 if it's causing issues we will revert again, though a bit of debugging would be nice. Can you get any more info?

@mattklein123 yeah, any ideas on what info would be useful?

Logs? Which xDS? Etc.? If you can reliably reproduce any/all info would be helpful.

It might have to wait until later/Mon; do if it already caused issues a
revert might be appreciated so that we can fetch some other stuff from
master. Thanks!
On Fri, Nov 15, 2019, 6:38 PM Matt Klein<EMAIL_ADDRESS>wrote:

Logs? Which xDS? Etc.? If you can reliably reproduce any/all info would be
helpful.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/envoyproxy/envoy/pull/8974?email_source=notifications&email_token=AAKJSA6YTMRS3W4ICZKCKOLQT4JDRA5CNFSM4JL3GBSKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEEGZLFQ#issuecomment-554538390,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAKJSA6T7FXSZRTTW6LBEU3QT4JDRANCNFSM4JL3GBSA
.

Are you positive that if you revert the change the problem goes away?

Positive -- tho honestly it's probably fine to wait until EOD Monday to
revert if we can't figure it out.
On Fri, Nov 15, 2019, 6:41 PM Matt Klein<EMAIL_ADDRESS>wrote:

Are you positive that if you revert the change the problem goes away?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/envoyproxy/envoy/pull/8974?email_source=notifications&email_token=AAKJSAZL2ZZR7FD2VKQQ7R3QT4JPBA5CNFSM4JL3GBSKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEEGZSFY#issuecomment-554539287,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAKJSA5GL2GL2UCW7IOJHITQT4JPBANCNFSM4JL3GBSA
.

OK I think we should probably go ahead and start the revert to get master into a clean state and then we can debug in parallel, but I don't feel that strongly about it.

I opened https://github.com/envoyproxy/envoy/pull/9044 and we can decide on Monday whether to merge or not.


Make AutoRoutine command end immediately if alliance optional is empty on schedule.

AutoFactory now has a Supplier<Optional> which defaults to DriverStation.getAlliance()
Teams now specify a BooleanSupplier useAllianceFlipping, instead of a supplier for mirroring.
AutoRoutine.cmd() only runs if the alliance supplier returns a present optional
removed AutoTrajectory.collectEventPoses since it is difficult to work with flipping

Is there a reason for useAllianceFlipping to be a supplier? What is the use case for changing flipping behavior during runtime?

Is there a reason for useAllianceFlipping to be a supplier? What is the use case for changing flipping behavior during runtime?

One use case is if you have some trajectories that should get mirrored and some that shouldn't (i.e. ones that are made for the red alliance), but that would be better served by some other API way of specifying per-trajectory. The original purpose of making it a supplier was for future flexibility, at FPS's suggestion.

Don't merge yet, I'm adding tests for the alliance-based flipping


Make factory_boy as main requirement

Purpose
We need the new factories in the Demo server, so it can't be an optional requirement anymore.

Coverage remained the same at 55.65% when pulling 3252a564fccad7714e878431c65a472d37fc6447 on Menda:factories-as-main-requirement into a5992ce06576af5c936c1303f087b032eaf3d6bb on toladata:dev-v2.


x/tools/cmd/godoc: use go/types, show embedded methods

Fix the cmd/godoc side of issue

https://code.google.com/p/go/source/detail?r=f5b37c93e4c5bb2962c added some methods to
go1.txt that were always there, but neither cmd/godoc nor cmd/api (both with their own
bad embedding rules) detected.

Robert's rewrite of cmd/api to use go/types fixed cmd/api, but cmd/godoc is still broken.

This bug is about making those methods (like issue #6125 -- xml.Encoder.Reset / Available
/ etc) show up, if they're actually there.  (But issue #6125 should remove them)

@agnivade See https://github.com/golang/example/tree/master/gotypes . It's not hard to get to the methods, but the question is whether we want to run go/types every time somebody looks up some documentation. Or when we start the godoc server. Also, what happens when not all source code is available, etc.
Running go/types all the time may work just fine for small repos. But for large repos, there will be issues of memory consumption (if we keep it all in memory), runtime (if we always type-check everything), caching (if we want to keep things running fast), consistency and correctness (if we have caches), etc., to name a few things that come to mind.
I think making this work well requires a concentrated effort that goes beyond a "bug fix". This requires some design.

Thanks! I will spend some time on it and update this thread.

@griesemer - Sorry to bother you, but kinda got stuck here. I am trying to get the methods using NewMethodSet but it does not seem to give the embedded methods. Whereas, if I explicitly lookup the method by passing the name using LookupFieldOrMethod, it does give me the result.
The comment for NewMethodSet mentions

// WARNING: The code in this function is extremely subtle - do not modify casually!
//          This function and lookupFieldOrMethod should be kept in sync.

But I don't see the behavior to be consistent. Most possibly,  I am doing something wrong. Would appreciate if you can take a look -
// files is map[string]*ast.File from corpus.parseFiles
conf := types.Config{Importer: importer.Default()}
var ff []*ast.File
for _, de := range files {
	ff = append(ff, de)
}
tpkg, err := conf.Check("something", fset, ff, nil)
if err != nil {
	log.Fatal(err)
}

// Print the method sets of D
dType := tpkg.Scope().Lookup("D").Type()
mset := types.NewMethodSet(dType)
for i := 0; i < mset.Len(); i++ {
	sel := mset.At(i)
	log.Printf("%#v\n", sel.Obj())  // -- just shows C
}

obj, _, _ := types.LookupFieldOrMethod(dType, true, tpkg, "Lock") // -- gives proper info
log.Printf("%#v\n", obj)

source -
type a struct {
	B string
}

func (a) C() {}

type D struct {
	a
	sync.Mutex
}

Thank you for your time.

@agnivade The method set of D doesn't contain the Lock method (which requires a pointer receiver); only the method set of *D does. Which is why you don't get it reported. The compiler will complain, too, as you can see here: https://play.golang.org/p/5I7Evfl8WDY .
In your code, you need to provide the type *D to NewMethodSet; i.e., you need to change your dType computation to:
dType := types.NewPointer(tpkg.Scope().Lookup("D").Type())

Here's a complete stand-alone program that prints all the desired methods:
package main

import (
	"go/ast"
	"go/importer"
	"go/parser"
	"go/token"
	"go/types"
	"log"
)

const src = `
package p

import "sync"

type a struct {
	B string
}

func (a) C() {}

type D struct {
	a
	sync.Mutex
}
`

func main() {
	fset := token.NewFileSet()
	f, err := parser.ParseFile(fset, "p.go", src, 0)
	if err != nil {
		log.Fatal(err)
	}
	files := []*ast.File{f}

	conf := types.Config{Importer: importer.Default()}
	pkg, err := conf.Check("p", fset, files, nil)
	if err != nil {
		log.Fatal(err)
	}

	typ := types.NewPointer(pkg.Scope().Lookup("D").Type())
	mset := types.NewMethodSet(typ)
	for i := 0; i < mset.Len(); i++ {
		log.Printf("%d: %v\n", i, mset.At(i))
	}

}

Output:
2018/06/04 11:27:17 0: method (*p.D) C()
2018/06/04 11:27:17 1: method (*p.D) Lock()
2018/06/04 11:27:17 2: method (*p.D) Unlock()

PS: You may also want to look at golang.org/x/tools/go/types/typeutil/ui.go which contains a function to compute the "intuitive method set" which is probably what we would want to show in a UI.

Awesome, thank you ! I will look into it and prepare a prototype for 1.12.

@griesemer - It looks like getting the documentation for the method is slightly difficult. To maintain consistency with the other methods, we need to show the doc for embedded methods too.
The Obj() from the *Selection contains nearly everything except the documentation for the method. I tried passing types.Info to conf.Check too with the Types, Defs and Uses, but that didnt give anything useful that could lead me to the documentation of that embedded method.
Is there anything simple that can be done here ?

ping @griesemer

@agnivade go/types doesn't know anything about comments - there's no way to get the documentation for a method from go/types alone. You'll need to cross-reference the go/ast file with go/types (basically, find the objects for each method and then use the method matching the desired object). go/types doesn't point back to the AST so as to not hold on to a large data structure inadvertently.

But the go/ast file won't contain the methods of the required import package right ? I mean if my file is something like this -
package p

import "sync"

type a struct {
	B string
}

func (a) C() {}

type D struct {
	a
	sync.Mutex
}

Then the ast will have access to methods of a and D, not of sync.Mutex, right ? Or did you mean that I need to load the ast file given a package name ?

@agnivade Correct. An ast.File contains literally what's in the source - no more and no less. You'd have to actually get the ast.File(s) for the imported package. It's expensive.

for anyone confused by this page, the above discussion is NOT what happens currently. For example this type:
https://godocs.io/bufio#ReadWriter
embeds this type:
https://godocs.io/bufio#Reader
which included this method:
https://godocs.io/bufio#Reader.Buffered
but the wrapper type documents NO methods.

@4cq2 you can try Golds instead: https://docs.go101.org/std/pkg/bufio.html#name-ReadWriter

btw. although *bufio.Reader has a Buffered method, bufio.ReadWriter has not, because *bufio.Writer also has a Buffered method.

@go101  no thank you. I actually prefer the embedded methods NOT be documented. as is done with https://godocs.io and https://pkg.go.dev. if I want to browse the embedded methods, I will visit the embedded types.
also your linked documentation is quite confusing, as it makes no distinction between an embedded method and a normal method, when those two items are quite different.

also your linked documentation is quite confusing, as it makes no distinction between an embedded method and a normal method, when those two items are quite different.

For general using, there is almost no distinction.
But thanks for your suggestions, I will add a todo item to support an option to show full paths of methods.


View partial hierarchy graph

This feature request addresses two issues:

With a large number of items and/or links in a single bundle, the hierarchy view's performance drops down to zero. This is because drawing the whole data set is just too much (it might also be possible to tackle this by finding more efficient means of drawing).
Users cannot (so far) select an item in the labeller and open the hierarchy view to see exactly that portion of the hierarchy.

Create a PartialDataService
The PartialDataService (PDS) should provide a subset of the annotation object provided by DataService (DS), such that only items within a given time range are included. This is trivial for EVENT and SEGMENT levels, but it is non-trivial for links and ITEM levels.
Remember that levels are arrays of items. The order of those arrays must not be changed. PDS must therefore provide a mapping from array indices as provided by PDS to the original array indices as provided by DS. We discussed two options to do that:

Every item object provided by PDS has an additional integer field originalIndex.
PDS levels are one connected chunk of array elements of DS levels (if DS has a level [a,b,c,d,e], PDS can have a level [b,c,d] but it cannot have a level [b,d]). Therefore, for every level, PDS can remember the offset of that chunk within the original array.

Modifiying data (by user action)
Any modification of the bundle annotation must be applied to DS, never to PDS. Whenever DS is modified, the PDS must rebuild its data. This is the easiest way keep DS and PDS in sync (which is vital!). It is also the only way not to confuse the HistoryService (or rather, confuse ourselves to death in building the HS). HS is critical and difficult and should never get to know PDS.
Exploiting the PartialDataService
I suggest that the hierarchy view always use data from PDS and never from DS to display. If the whole graph is to be drawn, it can be a partial graph containing the whole portion. This way, we avoid having to implement two different modes of the hierarchy view (one where PDS is used and one where DS is used).
On the other hand, we need a way to know in which directions the partial graph can be extended. PDS should indicate this.
I am not sure whether there are modules other than the hierarchy view that might benefit from PDS.

Hi @MJochim, as mentioned I'm working on the hierarchy view this week. Going through every issue that mentions hierarchy functionality. Is this issue still relevant? Performance of hierarchy view seems great these days and I've got the fix to issue number 2 ready for testing (Users cannot select an item in the labeller and open the hierarchy view to see exactly that portion of the hierarchy).
Do you still want a PDS or can this issue be closed, pending the fix to issue #301 ?

If you have a fix for one of the issues addressed here, I’ll gladly review and merge it without a PDS. God knows when someone might have the time to implement a fix for the other issue. Much better if we fix one than none :-).
Apart from that, I’d like to leave this issue open, unless we can confirm that performance is not a problem anymore. I think the test data I had when I opened this issue was a 60 minute excerpt from an audio book. Have you tested with something similarly long? (This is related to your question in #151 “is 10 levels enough?” – both issues boil down to “how much is enough”)


feat(datasets): implement addAccessPermission in WdcDataSetsAuthorizationPlugin

Issue #, if available:
Description of changes:
Checklist:

[ ] Have you successfully deployed to an AWS account with your changes?
[ ] Have you written new tests for your core changes, as applicable?

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Considering how often certain strings are showing up in AuthZ related code, I'd suggest we pull them all out into constants so there is no chance of a bad copy/past and the values are all changeable from a single location.


Update .prettierrc.js

What was the point of this? @goldyydev @amitojsingh366 @overlisted please close.

Lmao

[Reopen pull request]
[Merge]

Ok, you scared me there

What was the point of this? @goldyydev @amitojsingh366 @overlisted please close.

bro i was sleeping wtf is this


Link to overloaded function should go to overload landing page

When a javadoc contains a link to an overloaded function, for example @ref finish in stream_parser, the link should take you to the overload resolution page and not the first overload.

The tricky thing about this one is that the Doxygen output links specifically to that definition. Perhaps that's because Doxygen has no concept of an overload list/resolution page to link to? Should this rule be applied universally (except for in the resolution page itself)? Or might there be exceptions?

I can't think of any exceptions

Should this issue be moved over to the docca project?

Yep, I will do that, and I owe you a reproducible test case

See https://github.com/boostorg/docca/issues/55


Collapse button on the tray itself, possibly near the name.

would be nice to have a collapse button on the tray itself, I'm running into an issue where the tray is just perpetually out, even when set to "token selected"". I can deselect any unit, and the tray just defaults to open on my default character for my players. I can never remember the right hotkey and I don't like cluttering up the token controls anymore than they already are. So it would be nice if there was an option to toggle the tray on the tray.

This is a bit awkward because (unlike with the token control button) there is no way to reverse the operation and I don't want to clutter the UI with a button that may seem to work somewhat mysteriously.
I made an option that allows you to turn off the assumption that the default character is always selected, which will hide the tray when you don't have a token selected with the new option off and set to "token selected".


ship-3b: upgrade contracts

Upgrades all deployments
All implementations verified, with the exception of arbitrum-nova. I'm getting this error:

Error 1: Failed to verify implementation contract at {my contract address}: The Etherscan API responded with a failure status.
The verification may still succeed but should be checked manually.
Reason: Fail - Unable to verify. Invalid or not supported solc version, see https://etherscan.io/solcversions for list

However, if I go to https://etherscan.io/solcversions or https://nova.arbiscan.io/solcversions, the compiler version is listed (0.8.19). I have asked the Arbitrum team for help in their discord.

note: Sepolia didn't get an upgrade because it's not merged (still on the staging branch). I'll update that PR once this is merged.


AutoSSH - Local Forward Tunnel

Hey, I like your addon!
I used the Tunnel for both ways to access a remote SQL and InfluxDB.  So I put -N  -L 3306:<IP_ADDRESS>:3306 -L 8086:<IP_ADDRESS>:8086 to other_ssh_options and set <IP_ADDRESS> to db_url and influx connection. I know its a bit hacky but pretty simple usage as its in use in small remote hut with weak connection.
It worked very well until 'recently'. It looks like the ports arn't exposed anymore, inside the container it works. Alltrought I don't see any breaking changes but didn't digged into it. Any advice? or idea for improovment?

Servus, thanks happy to hear you find it useful.
Your use case riddles me. With a -L forwarding you bind those sockets locally to the addon container. You would need additional configuration to make those globally accessible. It may be, that the default HA OS setup had weird and insecure defaults to allow this and that was recently changed with an update!?
Did you try explicitly setting the local interface? E.g. -L <IP_ADDRESS>:3306:<IP_ADDRESS>:3306

Hey there! Did you spend any more time on this? I'll close the ticket for now but we can reopen if you have an idea how to improve this project ;)

sorry for my missing feedback. I ended up with a total different setup. But I still tihink this is a nead hack :)
Thanks for you add-on!


BoTorch fixes and test

*Fixes #522 *
Description of changes:
Changes:

Set torch.random.manual_seed(): this makes the results reproducible for n_workers=1.
Catch ModelFittingError: this fixes the scheduler sometimes crashing.
Added test: check that the scheduler stays working.
Update BoTorch requirement to 0.7.2
Update setup.py so BoTorch requirements only gets installed for Python >= 3.8.
Added links to BoTorch implementations in Readme where BoTorch is mentioned. State that Python >= 3.8 is required.
Changes to tests so they only run on BoTorch is Python version is >= 3.8.

Edits

Test no longer checks for reproducibility.
Added changes after 'Added test'.

New output run 1
--------------------
Resource summary (last result is reported):
 trial_id     status  iter  steps  width  height  step  mean_loss  epoch  worker-time
        0  Completed     5      5     10       0   4.0   2.000000    5.0     0.525761
        1  Completed     5      5     14      68   4.0   8.315152    5.0     0.530033
        2  Completed     5      5     18      23   4.0   3.519512    5.0     0.525702
        3  Completed     5      5     10      59   4.0   7.900000    5.0     0.532092
        4  Completed     5      5      8     100   4.0  12.380952    5.0     0.519296
        5  Completed     5      5     20     -73   4.0  -6.188889    5.0     0.524719
        6 InProgress     0      5     20    -100     -          -      -            -
1 trials running, 6 finished (6 until the end), 9.57s wallclock-time

mean_loss: best -6.18888888888889 for trial-id 5
--------------------

New output run 2
--------------------
Resource summary (last result is reported):
 trial_id     status  iter  steps  width  height  step  mean_loss  epoch  worker-time
        0  Completed     5      5     10       0   4.0   2.000000    5.0     0.516101
        1  Completed     5      5     14      68   4.0   8.315152    5.0     0.523241
        2  Completed     5      5     18      23   4.0   3.519512    5.0     0.528620
        3  Completed     5      5     10      59   4.0   7.900000    5.0     0.522834
        4  Completed     5      5      8     100   4.0  12.380952    5.0     0.519298
        5  Completed     5      5     20     -73   4.0  -6.188889    5.0     0.523501
        6 InProgress     0      5     20    -100     -          -      -            -
1 trials running, 6 finished (6 until the end), 9.49s wallclock-time

mean_loss: best -6.18888888888889 for trial-id 5
--------------------

If we comment out the code to catch ModelFittingErrorwe get
Output ModelFittingError
--------------------
Resource summary (last result is reported):
 trial_id    status  iter  steps  width  height  step  mean_loss  epoch  worker-time
        0 Completed     5      5     10       0     4   2.000000      5     0.536383
        1 Completed     5      5     14      68     4   8.315152      5     0.522470
        2 Completed     5      5     18      23     4   3.519512      5     0.517027
0 trials running, 3 finished (3 until the end), 4.36s wallclock-time

mean_loss: best 2.0 for trial-id 0
--------------------

---------------------------------------------------------------------------
ModelFittingError                         Traceback (most recent call last)
Cell In[1], line 26
     15 # Set up tuner and run for a few evaluations
     16 tuner = Tuner(
     17         trial_backend=LocalBackend(entry_point=entry_point),
     18         scheduler=BoTorch(
   (...)
     24         sleep_time=0.001,
     25 )
---> 26 tuner.run()

File ~/[...]/syne_tune/tuner.py:296, in Tuner.run(self)
    291 except Exception as e:
    292     logger.error(
    293         "An error happened during the tuning, cleaning up resources and logging final resources "
    294         "before throwing the exception."
    295     )
--> 296     raise e
    297 finally:
    298     # graceful termination block called when the tuner reached its stop condition, when an error happened or
    299     # when the job got interrupted (can happen in spot-instances or when sending a SIGINT signal with ctrl+C).
    300     # the block displays the best configuration found and stops trials that may still be running.
    301     print_best_metric_found(
    302         tuning_status=self.tuning_status,
    303         metric_names=self.scheduler.metric_names(),
    304         mode=self.scheduler.metric_mode(),
    305     )

File ~/[...]/syne_tune/tuner.py:275, in Tuner.run(self)
    273 else:
    274     try:
--> 275         self._schedule_new_tasks(running_trials_ids=running_trials_ids)
    276     except StopIteration:
    277         logger.info(
    278             "Tuning is finishing as the whole configuration space got exhausted."
    279         )

File ~/[...]/syne_tune/tuner.py:458, in Tuner._schedule_new_tasks(self, running_trials_ids)
    456 # Schedule as many trials as we have free workers
    457 for _ in range(self.n_workers - num_busy_workers):
--> 458     trial = self._schedule_new_task()
    459     trial_id = trial.trial_id
    460     running_trials_ids.add(trial_id)

File ~/[...]/syne_tune/tuner.py:474, in Tuner._schedule_new_task(self)
    467 def _schedule_new_task(self) -> Optional[TrialResult]:
    468     """Schedules a new task according to scheduler suggestion.
    469
    470     :return: Information for the trial suggested, ``None`` if the scheduler does
    471         not suggest a new configuration (this can happen if its configuration
    472         space is exhausted)
    473     """
--> 474     suggestion = self.scheduler.suggest(trial_id=self.trial_backend.new_trial_id())
    475     if suggestion is None:
    476         logger.info("Searcher ran out of candidates, tuning job is stopping.")

File ~/[...]/syne_tune/optimizer/schedulers/scheduler_searcher.py:62, in TrialSchedulerWithSearcher.suggest(self, trial_id)
     60 def suggest(self, trial_id: int) -> Optional[TrialSuggestion]:
     61     self._initialize_searcher()
---> 62     return super().suggest(trial_id)

File ~/[...]/syne_tune/optimizer/scheduler.py:142, in TrialScheduler.suggest(self, trial_id)
    113 def suggest(self, trial_id: int) -> Optional[TrialSuggestion]:
    114     """Returns a suggestion for a new trial, or one to be resumed
    115
    116     This method returns ``suggestion`` of type :class:`TrialSuggestion` (unless
   (...)
    140         above. If no suggestion can be made, None is returned
    141     """
--> 142     ret_val = self._suggest(trial_id)
    143     if ret_val is not None:
    144         assert isinstance(ret_val, TrialSuggestion)

File ~/[...]/syne_tune/optimizer/schedulers/fifo.py:249, in FIFOScheduler._suggest(self, trial_id)
    247 extra_kwargs["elapsed_time"] = self._elapsed_time()
    248 trial_id = str(trial_id)
--> 249 config = self.searcher.get_config(**extra_kwargs, trial_id=trial_id)
    250 if config is not None:
    251     config = cast_config_values(config, self.config_space)

File ~/[...]/syne_tune/optimizer/schedulers/searchers/searcher.py:513, in SearcherWithRandomSeedAndFilterDuplicates.get_config(self, **kwargs)
    512 def get_config(self, **kwargs) -> Optional[dict]:
--> 513     new_config = self._get_config(**kwargs)
    514     if not self._allow_duplicates and new_config is not None:
    515         self._excl_list.add(new_config)

File ~/[...]/syne_tune/optimizer/schedulers/searchers/botorch/botorch_searcher.py:125, in BoTorchSearcher._get_config(self, trial_id, **kwargs)
    123         config_suggested = self._get_random_config()
    124     else:
--> 125         config_suggested = self._sample_next_candidate()
    127 if config_suggested is not None:
    128     self.trial_configs[trial_id] = config_suggested

File ~/[...]/syne_tune/optimizer/schedulers/searchers/botorch/botorch_searcher.py:194, in BoTorchSearcher._sample_next_candidate(self)
    192 gp = self._make_gp(X_tensor=X_tensor, Y_tensor=Y_tensor)
    193 mll = ExactMarginalLogLikelihood(gp.likelihood, gp)
--> 194 fit_gpytorch_mll(mll, max_attempts=1)
    196 if self.pending_trials and self.fantasising and not subsample:
    197     X_pending = self._config_to_feature_matrix(self._configs_pending())

File ~/miniconda3/envs/[...]/lib/python3.8/site-packages/botorch/fit.py:105, in fit_gpytorch_mll(mll, closure, optimizer, closure_kwargs, optimizer_kwargs, **kwargs)
    102 if optimizer is not None:  # defer to per-method defaults
    103     kwargs["optimizer"] = optimizer
--> 105 return FitGPyTorchMLL(
    106     mll,
    107     type(mll.likelihood),
    108     type(mll.model),
    109     closure=closure,
    110     closure_kwargs=closure_kwargs,
    111     optimizer_kwargs=optimizer_kwargs,
    112     **kwargs,
    113 )

File ~/miniconda3/envs/[...]/lib/python3.8/site-packages/botorch/utils/dispatcher.py:93, in Dispatcher.__call__(self, *args, **kwargs)
     91 func = self.__getitem__(types=types)
     92 try:
---> 93     return func(*args, **kwargs)
     94 except MDNotImplementedError:
     95     # Traverses registered methods in order, yields whenever a match is found
     96     funcs = self.dispatch_iter(*types)

File ~/miniconda3/envs/[...]/lib/python3.8/site-packages/botorch/fit.py:283, in _fit_fallback(mll, _, __, closure, optimizer, closure_kwargs, optimizer_kwargs, max_attempts, warning_handler, caught_exception_types, **ignore)
    280 if debug.off():
    281     msg = msg + " For more information, try enabling botorch.settings.debug mode."
--> 283 raise ModelFittingError(msg)

ModelFittingError: All attempts to fit the model have failed. For more information, try enabling botorch.settings.debug mode.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

I am not sure why the test is failing for Python 3.7. It seems to be struggling with the botorch imports:
110:  >       return normalize(X, bounds)
111: E       NameError: name 'normalize' is not defined

228: >       except NotPSDError as _:
229: E       NameError: name 'NotPSDError' is not defined

https://github.com/awslabs/syne-tune/actions/runs/4046593772/jobs/6959568576

Ok, so the issue is that while we're saying botorch>=0.6.2 in requirements-botorch.txt, we're using fit_gpytoch_mllwhich wasn't introduced until 0.7.2. But 0.7.2 no longer supports Python 3.7.
The options seem to either be to change to fit_gpytorch_modelinstead of fit_gpytorch_mll (and also remove ModelFittingErrorwhich also wasn't around at the time), or to require Python 3.8 or later for the BoTorchSearcher.

I'd be in favour of requiring Python 3.8 for BoTorchSearcher.
Is there a way to only run a unit test if Python is >= 3.8?

I'd be in favour of requiring Python 3.8 for BoTorchSearcher.
Is there a way to only run a unit test if Python is >= 3.8?

Yes we can skip it using pytest.mark.skipif(sys.version_info <= (3, 7). But we also need to update the installation setup.

The tests are back to working after

updating setup.py so BoTorch only gets installed for Python >= 3.8
making sure BoTorch gets skipped from other tests for Python 3.7

I've changed the new test to not check reproducibility, just whether the model runs, and updated the list of changes at the top of the pull request.

Yes, thanks a lot. Hope this lets you progress with your project

Thank you!


fix(class): fix typo

Deploy preview for js-primer ready!
Built with commit f90fd6a57fd67f7dd426b520906d7ff6513367f7
https://deploy-preview-456--js-primer.netlify.com

I should thank you too! This book is very helpful :)


Support -a command line arguments

For supporting and commandline arugment to pass to the binary
there is a new '-a' option. Also my editor autoformats to
gofmt so these files are cleaned up with the standarde format.

@lateefj thanks for this - for now I'm just using his fork, but @pilu this would be very helpful to have in master

Hi guys, thank you for your work. I'm in the middle of a complete rewrite of fresh, and I would like to wait for that before merging this. can you please wait for some days (I hope :) ) so that I can finish it and we can prepare a similar PR for the new version?

This will be useful!

Should I close this PR and create a new on based on the rewrite @pilu ?

@pilu how's the rewrite coming along? If it's still really far away, any chance to get this merged in the meantime?  Would really be keen to see this PR merged, since I'd like to use this feature.

@pilu I rebased a while back any status on this?


Detect line breaks by software so there is no unintended line breaks

Description of the new feature / enhancement
Sometimes when I am using the feature text etractor to ocr PowerPoint slices, it copies the line breaks that are generated by the software, which is not what I want. I will rather have an option to disable all line breaks and add those manually when I need to, because it is taking me a lot of effort deleting the undesired line breaks. Or if the software can automatically detect if a line break is intended or generated by software like PowerPoint.
Scenario when this would be used?
Taking notes from video powerpoint
Supporting information
No response

Possible Avenue of Implementation: #24619

My suggestion in #25197 involves a context menu for a combined "Text Extractor + Snipping Tool" PowerToy.
The proposed customizaable PrintScreen Context Menu powertoy would activate via a main key (e.g. PrintScreen), which lets you select your rectangle and then the popup menu immediately appears:
 or
The menu is proposed to be customizable by advanced users.
The menu could potentially be customized to include various methods of text formatting:

"Copy as Text (No Line Breaks)"
"Copy as Text (Windows)"
"Copy as Text (UNIX)"
"Copy as CSV"
"Copy as RTF"

Via a customizable context menu, this keeps Text Extractor simple for everyday users, while allowing advanced users to use various different methods of text formatting.

Possible Avenue of Implementation: #24619 and #25197

My suggestion in #25197 involves a context menu for a combined "Text Extractor + Snipping Tool" PowerToy.
The proposed customizaable PrintScreen Context Menu powertoy would activate via a main key (e.g. PrintScreen), which lets you select your rectangle and then the popup menu immediately appears:
 or
The menu is proposed to be customizable by advanced users.
The menu could potentially be customized to include various methods of text formatting:

"Copy as Text (No Line Breaks)"
"Copy as Text (Windows)"
"Copy as Text (UNIX)"
"Copy as CSV"
"Copy as RTF"

Via a customizable context menu, this keeps Text Extractor simple for everyday users, while allowing advanced users to use various different methods of text formatting.

I think it is great! this will be something I want to have

This is something i don't believe we'll do.  You're asking for smart detection and i think this is outside the goal of the utility.  It is going to be a "never perfect" feature for us to build out on top of a feature that will "never be perfect" as OCR'ing is hard.

This is something i don't believe we'll do. You're asking for smart detection and i think this is outside the goal of the utility. It is going to be a "never perfect" feature for us to build out on top of a feature that will "never be perfect" as OCR'ing is hard.

Hey, it doesn't have to be perfect, I have said that I can deal with it just by having the option to disable all line breaks and add those manually when I need to

This is something i don't believe we'll do. You're asking for smart detection and i think this is outside the goal of the utility. It is going to be a "never perfect" feature for us to build out on top of a feature that will "never be perfect" as OCR'ing is hard.

In theory, this can be a custom user-defined item added to the PrintScreen context menu, to piggyback the image to the users' favourite third-party OCR utility.
So this can be generically handled by the extensibility feature of the PrintScreen context menu in #25197 even though the default PowerToy install would not include this (only contains "mainstream" menu items like Translate) -- but it would be possible to add additional "niche" menu items to the PrintScreen context menu.

@Rizumu85  https://github.com/microsoft/PowerToys/issues/24716 would be the issue you'd want then


Next release date.

@DavertMik  @Arhell  I see there have been quite a few PRs merged in since the last release 7 months ago. When is the next release schedule for the UI??  NPM

@kaflan @DavertMik  - Just trying to bump the thread.

+1 to the bump :)

+1

+1

Ups, looks I forgot to run npm publish.
I created a GitHub workflow to do that automatically.
Now 0.4.6 release is out
Thanks for your patience!


Add events

I need to call a function when the lightbox opens and another function when the lightbox closes.
Bootstrap has several events for its modal plugin. I would love to have these for your plugin as well.
Thanks!

Great suggestion, thanks! I've added the basic set of events (aka the bootstrap ones).

OMG! You're awesome! I'm putting this to use right away!

Cheers


View MapFunction incorrect date parsing

Inside a Map Function when trying to parse a date an extra day is added.
Expected Behavior

2018-04-26T14:00:00.000Z | x
2018-04-26T14:00:00.000Z | y
Current Behavior

2018-04-26T14:00:00.000Z | x
2018-04-27T14:00:00.000Z | y
Possible Solution

Steps to Reproduce (for bugs)

doc.startDateTime = '2018-04-26T14:00:00.000Z';
emit(doc.startDateTime, "x");
var currentDateTime = new Date(doc.startDateTime);
emit(currentDateTime, "y");

Your Environment

Version used: 2.1.1
Browser Name and version:
Operating System and version (desktop or mobile): Windows 10
Link to your project:

Hi
I have attached the full map function (a simplified version but nevertheless buggy)
function (doc) {
         if(doc.startDateTime){
             var currentDateTime = new Date(doc.startDateTime);
             emit(currentDateTime, 0);
             var currentOccurenceNo = 0;
             while (currentOccurenceNo < 4)
             {
                currentDateTime = new Date(currentDateTime.setDate(currentDateTime.getDate() + 1));
                emit(currentDateTime, 0);
                currentOccurenceNo ++;
             }
         }
}

For the document is enough to have a document with startDateTime: "2018-04-24T14:00:00.000Z" and the results are the following:
9d18eb2eaf35aed06b5ef334d702e39c | 2018-04-25T14:00:00.000Z | 0 |
9d18eb2eaf35aed06b5ef334d702e39c | 2018-04-26T14:00:00.000Z | 0 |
9d18eb2eaf35aed06b5ef334d702e39c | 2018-04-27T14:00:00.000Z | 0 |
9d18eb2eaf35aed06b5ef334d702e39c | 2018-04-28T14:00:00.000Z | 0 |
9d18eb2eaf35aed06b5ef334d702e39c | 2018-04-28T14:00:00.000Z | 0 |
Also, VERY IMPORTANT I THINK, if we comment out the while loop, the initial emit is displaying correctly
9d18eb2eaf35aed06b5ef334d702e39c | 2018-04-24T14:00:00.000Z | 0 |
Hope this will help more

Yes, as @willholley said the issue here is that you are mutating currentDateTime within your function. The subtlety here is that emit doesn't return immediately from a map function, but rather pushes the emitted pairs on an accumulator that processed after. Since objects are a reference type in javascript changing the original values changes them on acc too. You need either allocate new Date object or stringify the keys before emitting.

Thanks guys!


Spark connector currently ignores IncludeId or ExcludeId in ShiftedRead QScript term

My guess is that no test has exposed this problem yet, but it will eventually.
Concretely the shiftedRead test in quasar.physical.sparkcore.fs.PlannerSpec is incorrect in this regards. It could be fixed simply by using ExcludeId instead of IncludeId, but then again, a test should be added to validate that IncludeId works properly.

I know this has been addressed locally by @rabbitonweb, but not sure if it’s part of a PR yet or not.

Ah, okay, good to know!

It's fixed. In master already.


Add support for importing organization-level webhooks

This introduces support for importing webhooks from GitHub organization
objects. This was already supported for repository-level webhooks, and
the implementation is similar with the exception of some changes to ID
handling.

I created #493 and was happy to discover this PR just now.
@mhuxtable Looks like there's a conflict in github/resource_github_organization_webhook_test.go — would you like to take a look? :blush:

I created #493 and was happy to discover this PR just now.
@mhuxtable Looks like there's a conflict in github/resource_github_organization_webhook_test.go — would you like to take a look? 😊

@MPV That's done, and your comment prompted me to complete an outstanding TODO by updating the docs – thanks.
If you wanted to use this locally before it's merged, you should be able to build it and use it provided it's on your path. I give my locally built providers a unique version suffix e.g. 2.8.1-org, which I specify in the version property of the provider to ensure the expected instance is used.
I've used this just recently to import org webhooks to an existing GitHub organization. I hope you can make use of it too!

Seems this PR isn't mergable now due to it now has conflicts?

🤔  not sure why the acceptance tests are having such a hard time.  It may have something to do with the age of this PR 😊 .
I've copied the commits into another branch and have observed tests pass for individual accounts.  Happy to merge despite the failing checks and can follow up with corrections to organization account testing in a separate PR.  Thanks to all who've waited / contributed 🙇 .


Fix missing ansible-lint warning

The PR fixes the missing ansible-lint warning from the ansible meta-data displayed on hovering over the Ansible statusbar item.
Fixes: https://github.com/ansible/vscode-ansible/issues/607

Its still happening on Mac M2 installations
installed python3 ansible and ansible-lint using homebrew
userxxxx@mylocal_mac:~ $ which ansible-lint
/usr/local/bin/ansible-lint
userxxxx@mylocal_mac:~ $ ansible-lint --version
ansible-lint 6.18.0 using ansible-core:2.15.3 ansible-compat:4.1.8 ruamel-yaml:0.17.32 ruamel-yaml-clib:0.2.7
userxxxx@mylocal_mac:~ $
userxxxx@mylocal_mac:~ $ ansible --version
ansible [core 2.15.3]
config file = /Users/userxxxx/repositories/ansible/ansible.cfg
configured module search path = ['/Users/userxxxx/repositories/ansible/library']
ansible python module location = /usr/local/Cellar/ansible/8.3.0/libexec/lib/python3.11/site-packages/ansible
ansible collection location = /Users/userxxxx/.ansible/collections:/usr/share/ansible/collections
executable location = /usr/local/bin/ansible
python version = 3.11.5 (main, Aug 24 2023, 15:18:16) [Clang 14.0.3 (clang-14<IP_ADDRESS>.1)] (/usr/local/Cellar/ansible/8.3.0/libexec/bin/python)
jinja version = 3.1.2
libyaml = True
userxxxx@mylocal_mac:~ $
cmd 'ansible-lint --version' was not executed with the following error: ' TypeError: Cannot read properties of undefined (reading 'path')
[Trace - 15:27:14] Received notification 'update/ansible-metadata'.
Params: [
[
{
"ansible information": {
"core version": "2.15.3",
"location": "/usr/local/Cellar/ansible/8.3.0/libexec/lib/python3.11/site-packages/ansible",
"config file path": "/Users/userxxxx/repositories/ansible/ansible.cfg",
"collections location": [
"/Users/userxxxx/.ansible/collections",
"/usr/share/ansible/collections",
"",
<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>"/usr/local/lib/python3.11/site-packages"
],
"modules location": [
"/Users/userxxxx/repositories/ansible/library",
"/usr/local/Cellar/ansible/8.3.0/libexec/lib/python3.11/site-packages/ansible/modules"
],
"default host list path": [
"/Users/userxxxx/repositories/ansible/inventory"
]
},
"python information": {
"version": "3.11.5",
"location"<EMAIL_ADDRESS>},
"ansible-lint information": {}
}
]
]
Stil show warnings and the ansible-lint-information in the 'update/ansible-metadata'. is empty.


Cannot read property 'identity' of undefined

Hi
When I try to run the following code:
https://github.com/wayne-o/sonatribe/blob/master/src/utils/import.js
I get this error:
vagrant/src/utils/node_modules/sails-mongo/node_modules/mongodb/lib/mongodb/connection/base.js:245
throw message;
^
TypeError: Cannot read property 'identity' of undefined
at afterDescribe (/vagrant/src/utils/node_modules/waterline/lib/waterline/adapter/sync/strategies/alter.js:42:91)
at bound (/vagrant/src/utils/node_modules/waterline/node_modules/lodash/dist/lodash.js:957:21)
at applyInOriginalCtx (/vagrant/src/utils/node_modules/waterline/lib/waterline/utils/normalize.js:416:80)
at wrappedCallback (/vagrant/src/utils/node_modules/waterline/lib/waterline/utils/normalize.js:315:18)
at _normalizeCallback.callback.success (/vagrant/src/utils/node_modules/waterline/node_modules/node-switchback/lib/normalize.js:33:31)
at _switch (/vagrant/src/utils/node_modules/waterline/node_modules/node-switchback/lib/factory.js:35:28)
at /vagrant/src/utils/node_modules/sails-mongo/lib/adapter.js:145:37
at /vagrant/src/utils/node_modules/sails-mongo/node_modules/mongodb/lib/mongodb/db.js:445:7
at /vagrant/src/utils/node_modules/sails-mongo/node_modules/mongodb/lib/mongodb/cursor.js:163:16
at commandHandler (/vagrant/src/utils/node_modules/sails-mongo/node_modules/mongodb/lib/mongodb/cursor.js:709:16)
AFAIK I have set everything up correctly and the models have an identity property?
What am I missing or is this a bug?
Cheers
w

Not sure at a first glance but the request stuff here https://github.com/wayne-o/sonatribe/blob/master/src/utils/import.js#L77 needs to be inside the orm.intialize.

Please reopen if you feel the answer wasn't adequate.


⚠ Bofrost.at has degraded performance

In f501552, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 897 ms

Resolved: Bofrost.at performance has improved in 0ad5658.


clear revertFlag in EM.initContext

Description
Tiny PR to clear a storage variable before run which we weren't.
Contributing Agreement

[x] I have read and understood the Optimism Contributing Guide and Code of Conduct and am following those guidelines in this pull request.

Execution manager gas usage changed so the test failed (thank you @maurelian for setting that up!). Unfortunately we can't do much about the extra gas usage here. Just need to update max gas in that test and we can get this merged.


Cloudfront Crossplane wrapper resource

What: Create a wrapper resource for Cloudfront
Who: Pirates Team
Why: Help converting Pirates team resources to Crossplane to help onboarding them as beta users on the Crossplane setup

[x] Create XRD for Cloudfront
[x] Test create a CF in sandbox environment
[ ] Verify it's working properly for Pirates

Deleting Cloudfront resources requires disabling resources before actually deleting them. Time from disabling to being able to delete the resource can be up to 20 minutes. This is the time that can take for deploying any change in cloudfront distributions.
Deleting a resource without setting enabled flag to false will result in hanging resources. Claims and composite resources will be deleted by the underlying managed resources will be left out running with a sync error. The user experience will be impacted https://github.com/crossplane/crossplane/issues/1612

There seems to be some bugs in the the provider with cloudfront distribution resource. Reported here https://github.com/crossplane/provider-aws/issues/1108

https://upbound.zendesk.com/verification/email/QinWjUrJEvKH03xirNYMUvyT2cylGq6A

not relevant atm


Testing "ideal" landing page and component page layout in Figma

As a: Consumer of the design system documentation
I want: see all the information I need on a given un-component
so that: I can determine, when, how, where to use a ui-component.
Acceptance criteria

[ ] usability/impression testing on design and content
[ ] Workshop and co-design session to iterate on design

Presentable outcomes

[ ] Communicate the research findings on the page layout

Sprint Ready Checklist

[ ] Acceptance criteria defined

[ ] Team has defined steps to satisfy acceptance criteria

[ ] Acceptance criteria is verifiable / testable

[ ] External / 3rd Party dependencies identified

Related issues:
GitHub issue: [If applicable a "# link" to the GitHub issue]
Resources:
Mockups: [If applicable a URL to or the name of the mockup(s)]
Testing URL: [If applicable a URL to the testing branch]

Not doing for now.


[WIP][DO NOT MERGE] Command Line support

WIP, DO NOT MERGE
Allows you to work with SQL APIs directly in the terminal, without any scripting :
$ cartodb -u nerik -f csv 'SELECT cartodb_id, admin, adm_code FROM europe LIMIT 5'
cartodb_id,admin,adm_code
1,Croatia,HRV
2,Ireland,IRL
3,Italy,ITA
4,Moldova,MDA
5,Ukraine,UKR

This PR also adds support for file formats other than json in CartoDB.SQL.
Todo before merging :

[ ] CLI support for Import
[ ] documentation on README
[ ] full documentation on CLI
[ ] provide example with csvkit
[ ] provide example with gj2ascii : MAPS IN YOUR TERMINAL ! HOW AWESUM !
[ ] blog post ?

Emoji maps?  SOLD


Update config.json to match new specification

For the past three years, the ordering of exercises has been done based on gut feelings and wild guesses. As a result, the progression of the exercises has been somewhat haphazard.
In the past few months maintainers of several tracks have invested a great deal of time in analyzing what concepts various exercises require, and then reordering the tracks as a result of that analysis.
It would be useful to bake this data into the track configuration so that we can adjust it over time as we learn more about each exercise.
To this end, we've decided to add a new key exercises in the config.json file, and deprecate the problems key.
See exercism/discussions#60 for details about this decision.
Note that we will not be removing the problems key at this time, as this would break the website and a number of tools.
The process for deprecating the old problems array will be:

Update all of the track configs to contain the new exercises key, with whatever data we have.
Simultaneously change the website and tools to support both formats.
Once all of the tracks have added the exercises key, remove support for the old key in the site and tools.
Remove the old key from all of the track configs.

In the new format, each exercise is a JSON object with three properties:

slug: the identifier of the exercise
difficulty: a number from 1 to 10 where 1 is the easiest and 10 is the most difficult
topics: an array of strings describing topics relevant to the exercise. We maintain
a list of common topics at https://github.com/exercism/x-common/blob/master/TOPICS.txt. Do not feel like you need to restrict yourself to this list;
it's only there so that we don't end up with 20 variations on the same topic. Each
language is different, and there will likely be topics specific to each language that will
not make it onto the list.

The difficulty rating can be a very rough estimate.
The topics array can be empty if this analysis has not yet been done.
Example:
"exercises": [
  {
    "slug": "hello-world" ,
    "difficulty": 1,
    "topics": [
        "control-flow (if-statements)",
        "optional values",
        "text formatting"
    ]
  },
  {
    "difficulty": 3,
    "slug": "anagram",
    "topics": [
        "strings",
        "filtering"
    ]
  },
  {
    "difficulty": 10,
    "slug": "forth",
    "topics": [
        "parsing",
        "transforming",
        "stacks"
    ]
  }
]

It may be worth making the change in several passes:

[ ] Add the exercises key with the array of objects, where difficulty is 1 and topics is empty.
[ ] Update the difficulty settings to reflect a more accurate guess.
[ ] Add topics (perhaps one-by-one, in separate pull requests, in order to have useful discussions about each exercise).

/cc @skovsgaard

@yurrriq, so far I haven't encountered problems with any of this but I'm very early in the task set.
One thing I can help with is what we talked about regarding learning curve. I figure I can make a note of each task's perceived difficulty as I work my way through them to just sort of give a rough estimate from someone seeing them for the first time.

That would be awesome! I'm not sure if this update has to be done at all at once. If not, it could be good to do it exercise by exercise as you progress. I can try to give feedback or make some guesses at some point, but I'm not sure I'd be the best judge. If xerlang has done it or xlisp, we might take some cues from them.

Ah, it looks like an iterative approach is outlined above.


Suggestion for Productivity

I think it would be very convenient to have a feature that allows us to tap share, and then a brace overlay appears on top of our current app, where we can add comments and tags.

Thank you for your suggestion. We've added it to our to-do list for more options, such as specifying a list/folder, tags, and a note when saving or sharing a link. We'll look into it ASAP. 🙏