Filters on "id: ID" field not working for mongo db

I have a use case where I want to perform a not in filter:
query {
  findVolunteers(
    filter: { not: { id: { in: ["5ef626c190701a3b90ebb24f", "5ef9c631da553f262c7a95dc"] } } }
  ) {
    items {
      id
    }
    count
  }
}

Which should return all volunteers whose id is in not equal to the one passed in the array.
But this returns
{
  "data": {
    "findVolunteers": {
      "items": [
        {
          "id": "5ef6262d90701a3b90ebb227"
        },
        {
          "id": "5ef6266890701a3b90ebb237"
        },
        {
          "id": "5ef6269490701a3b90ebb245"
        },
        {
          "id": "5ef626c190701a3b90ebb24f"
        },
        {
          "id": "5ef626e690701a3b90ebb25b"
        },
        {
          "id": "5ef6271a90701a3b90ebb26a"
        },
        {
          "id": "5ef9c631da553f262c7a95dc"
        }
      ],
      "count": 7
    }
  }
}

With the 5ef626c190701a3b90ebb24f and 5ef9c631da553f262c7a95dc present while they should have been filtered out.
/cc @craicoverflow @wtrocki

I guess it comes from the transformation of id to _id.
Because even the eq is failing.
query {
   all: findVolunteers {
    items {
      id
    }
    count
  }
  in: findVolunteers(
    filter: { id: {
      in:  ["5ef626c190701a3b90ebb24f", "5ef9c631da553f262c7a95dc"]
    }}
  ) {
    items {
      id
    }
    count
  }

  eqFirst: findVolunteers(
    filter: { id: {
      eq:  "5ef626c190701a3b90ebb24f"
    }}
  ) {
    items {
      id
    }
    count
  }

  eqSecond: findVolunteers(
    filter: { id: {
      eq:  "5ef9c631da553f262c7a95dc"
    }}
  ) {
    items {
      id
    }
    count
  }
}

gives
{
  "data": {
    "all": {
      "items": [
        {
          "id": "5ef6262d90701a3b90ebb227"
        },
        {
          "id": "5ef6266890701a3b90ebb237"
        },
        {
          "id": "5ef6269490701a3b90ebb245"
        },
        {
          "id": "5ef626c190701a3b90ebb24f"
        },
        {
          "id": "5ef626e690701a3b90ebb25b"
        },
        {
          "id": "5ef6271a90701a3b90ebb26a"
        },
        {
          "id": "5ef9c631da553f262c7a95dc"
        }
      ],
      "count": 7
    },
    "in": {
      "items": [],
      "count": 0
    },
    "eqFirst": {
      "items": [],
      "count": 0
    },
    "eqSecond": {
      "items": [],
      "count": 0
    }
  }
}

In what we were trying to achieve at OVP we have this workaround:
https://github.com/aerogear/OpenVolunteerPlatform/blob/master/platform/server/src/resolvers/custom-resolvers.ts#L157
The actual value is build here:
Using https://github.com/aerogear/OpenVolunteerPlatform/blob/master/platform/server/src/resolvers/custom-resolvers.ts#L25
Example query is here. https://github.com/aerogear/OpenVolunteerPlatform/blob/master/platform/server/src/resolvers/custom-resolvers.ts#L30

@craicoverflow @wtrocki renamed this issue to capture what is really not working.
see the above workaround used to get it working in OVP.

For filtering on primary key to work seamless in MongoDB we need to either:

Do an update right after insertion in the MongoDBDataProvider to make sure that the "id" is set - https://github.com/aerogear/graphback/blob/master/packages/graphback-runtime-mongodb/src/MongoDBDataProvider.ts#L48. The advantage of this is that the id field will always be set and we can filter against it. The disadvantage of this solution is an extra dangling "id" field in the document. Also, having to perform two operations just for the creation. I've tried this by manually editing the MongoDBDataProvider to add the update line and this works.

Always requiring it to be _id: ObjectID, and automatically providing the ObjectID scalar for it. I've done this in my test by manually changing the model and creating the ObjectId resolver. And it works but seems like a lot of work for the user, ideally we should be creating the resolver ourselves and have it as part of the resolvers created by our schema.  There is a very good discussion about doing this here https://github.com/aerogear/graphback/issues/1547.

And it works but seems like a lot of work for the user, ideally we should be creating the resolver ourselves and have it as part of the resolvers created by our schema.

Yes. That should be the thing.
There are couple big changes for this:

Embedded scalar support
MongoDB working only with _id

All of those are 0.15.0 targets.
My proposal will be to move this to 0.15.0 target as it is not possible to fix that without significant amount of work or hacks that will be invalidated in 0.15.0.
Target for us would be to not let community to wait so long for 0.15.0 so we can document this as well known limitation (link this particular bug)

And it works but seems like a lot of work for the user, ideally we should be creating the resolver ourselves and have it as part of the resolvers created by our schema.

Yes. That should be the thing.
There are couple big changes to resolve this problem:

Embedded scalar support
MongoDB working only with _id

Yes, this will require solving this issue https://github.com/aerogear/graphback/issues/1547 at least.

All of those are 0.15.0 targets.
My proposal will be to move this to 0.15.0 target as it is not possible to fix that without significant amount of work or hacks that will be invalidated in 0.15.0.

Target for us would be to not let community to wait so long for 0.15.0 so we can document this as well known limitation (link this particular bug)

The proposal makes sense and I was thinking the same.


cli/image: add a clear-versioned subcommand

This introduces a torcx image clear-versioned subcommand, which takes
care of garbage-collectiong images from versioned stores.
It operates either on a single image name or on whole stores, taking
a list of versions to keep.

/cc @squeed for review

A fresh pair of eyeballs is always appreciated :smile:. Your comments were on spot, I amended this to take care of those. It should be hopefully ready to merge, PTAL.

LGTM!


Add refactoring features like change in file, module and variable names

Environment data

VS Code version: XXX
Extension version (available under the Extensions sidebar): XXX
OS and version: XXX
Python version (& distribution if applicable, e.g. Anaconda): XXX
Type of virtual environment used (N/A | venv | virtualenv | conda | ...): XXX
Relevant/affected Python packages and their versions: XXX

Expected behaviour
XXX
Actual behaviour
XXX
Steps to reproduce:

XXX

Logs
Output for Python in the Output panel (View→Output, change the drop-down the upper-right of the Output panel to Python)
XXX

Output from Console under the Developer Tools panel (toggle Developer Tools on under Help; turn on source maps to make any tracebacks be useful by running Enable source map support for extension debugging)
XXX

Just to explicitly list things here:

We support variable renaming already (F2)
Renaming modules/files, no, but that may be https://github.com/microsoft/pylance-release/issues/127.

I think I'm going to close this in favor of #127 (as noted above); we're working on it.

@jakebailey FWIW I'm pretty sure #127 won't solve the thing I need right now (that PyCharm does). That is breaking the stuff (functions, consts, classes, etc) up in some file into a bunch of smaller files and renaming the references.
It's pretty similar conceptually to rename symbol, but since each other file in the workspace has some reference to the current module it's mostly about easily updating those.
Since #127 operates on a file level I don't think it will help with actual refactoring.


Small performance improvement in topics diff

This pull requests slightly improves the performance of diff-ing the already created topics topicMetadata, with the list of topics that the user want's to create topics.
On average a in b should be faster than b.indexOf(a).

Hi, the in operation will check inherited properties in a given object, I don't think it's faster than the indexOf of Array.

I agree @haio indexOf should be faster, at least this test on my browser tells so:
https://jsperf.com/regex-vs-indexof-vs-in

I think in op for object created with Object.create(null) will be faster than everything.

... Because calls are expensive ops, even if they are optimized coz functions are in core of V8. And you do 2 CallFunction ops. here. While making an object w/ missing prototype would make in quite fast.

Seems like a good change but could we include some benchmarks to verify?


How to add L2 regularization to the loss

How can add L2 regularization and optimize with regularization gradient?

You can use Optimizer.weight_decay for the L2 regularization.
http://docs.chainer.org/en/latest/reference/core/optimizer.html#chainer.Optimizer.weight_decay
BTW, would you put a question into the official forum from the next time? Thanks.
https://groups.google.com/forum/?hl=ja#!forum/chainer


update-binfmts

after trying to install eclipse'
reciveing this error:
root@ELDADH-VAIO:~# dpkg --configure -a
Setting up ca-certificates (20160104ubuntu1) ...
Setting up openjdk-8-jre-headless:amd64 (8u91-b14-0ubuntu4~16.04.1) ...
update-binfmts: warning: binfmt_misc module seemed to be loaded, but no /proc/sys/fs/binfmt_misc directory!  Giving up.
Help please :-)

Thanks for reporting the issue.  This is the same as https://github.com/Microsoft/BashOnWindows/issues/668 so closing it out as a duplicate.
We will be tracking\prioritizing which features to light up next by looking at the feedback provided at the wpdev portal. We appreciate your feedback and please make sure to provide it through the wpdev portal as well.
https://wpdev.uservoice.com/forums/266908-command-prompt-console-bash-on-ubuntu-on-windo


clevertap

Cleavertap is a tracking platforma that tracks users by across devices and services by  fingerperinting them

Thank you Ahsan @ahsandar!
Merging.


Maybe the channel is already ready when we call getstatus

When we use plaintext, it is common that we get notified for CONNECTING but when we call GetStatus the channel status is turned to READY already. This caused test to be flaky.

LGTM


Select Canonical URLs for manual pages

We started adding canonical URLs in the header of pages for the built-ins, std and third party and it has helped search engines find the "correct" version of something.  The Google Search Console for deno.land though indicates that manual of the manual pages are falling through the cracks because of the lack of a canonical.
Something like the logic for built-ins, std and third party should work for the manual.

This has been addressed.


Set up the development environment

As a [role]
I need [function]
So that [benefit]
Details and Assumptions
* [document what you know]

Acceptance Criteria
```gherkin
Given [some context]
When [certain action is taken]
Then [the outcome of action is observed]
```


Add tests about fix option

Codecov Report

Merging #15 (f11d6df) into main (91ec933) will increase coverage by 1.18%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##               main      #15      +/-   ##
============================================
+ Coverage     87.89%   89.08%   +1.18%
  Complexity      400      400
============================================
  Files            24       24
  Lines          1099     1099
============================================
+ Hits            966      979      +13
+ Misses          133      120      -13

Impacted Files
Coverage Δ

src/Runner/Linter.php
69.56% <0.00%> (+14.49%)
:arrow_up:

src/File/Finder.php
94.44% <0.00%> (+16.66%)
:arrow_up:

Continue to review full report at Codecov.

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


Allow for a identifiying class option within the container adapter or whatever

I added my own CSS theme for some of my selects but since the default attachment of the container is on body end it is hard to style the results in there.
Would be good if an identifying class could be added so it could be picked out.

If you set a custom theme, a class will be appended to the containers wrapping both the selection and the dropdown called select2-container--[theme] (select2-container--default for the default) which can be used for styling.
If you look at the first line of the theme files, you'll see that is how we are namespacing themes.

Awesome, I saw that but was unsure if it would fully fit the job


Redirection URL path prefix

Redirection URL path

Summary
I use traefik as a reverse proxy. My different services are differentiated by path.

my.domain.com/service1
my.domain.com/service2
my.domain.com/service3

I want service 1 to use keycloak authentication so I used the following:
redirection-url: http://my.domain.com/service1
secure-cookie: false

When I visit http://my.domain.com/service1 it redirects to http://my.domain.com/oauth/authorize?state=(...). However it should redirect to http://my.domain.com/service1/oauth/authorize?state=(...)

Environment

Docker image: dcm4che/louketo-proxy:1.0.0

Expected Results

Get redirected to http://my.domain.com/service1/oauth/authorize?state=(...)
Actual Results

Got redirected to http://my.domain.com/oauth/authorize?state=(...)
Additional Information
Due to reasons I'm forced to use paths. Only my.domain.com is available on one port.
Everything is actually https but traefik offloads tls so the app only sees http.
I can strip the /service1 prefix with traefik once traefik has determined which service this request belongs so so the actual requested endpoints will match up but the redirect needs to be correct.

I have the same problem with the /oauth/authorize with Traefik with your setup.
Unfortunately, the /oauth/authorize won't use the Redirection URL to redirect it. The source code for this is specified below
`
// WithOAuthURI returns the oauth uri
func (r *Config) WithOAuthURI(uri string) string {
	if r.BaseURI != "" {
		return fmt.Sprintf("%s/%s/%s", r.BaseURI, r.OAuthURI, uri)
	}

	return fmt.Sprintf("%s/%s", r.OAuthURI, uri)
}

The only way to make it to redirect correctly is to use the base-uri variable. In my case, I have to design a custom variable for the Traefik because the base-uri will also affect how cookies will be stored.

@johnhieu what do you mean when you say "I have to design a custom variable for the Traefik"?
I think i have a related problem:
environment k8s/ingress nginx that rewrites path: /sample(/|$)(.*) to nginx.ingress.kubernetes.io/rewrite-target: /$2

with  --base-uri=/sample i get into a redirect loop (i guess this has to do with the cookies)
without  --base-uri it redirects without the /sample (i do have my nginx controller set x-forwarded-prefix, but its ignored)

How did you get it to work?
tx.,

@johnhieu what do you mean when you say "I have to design a custom variable for the Traefik"?
I think i have a related problem:
environment k8s/ingress nginx that rewrites path: /sample(/|$)(.*) to nginx.ingress.kubernetes.io/rewrite-target: /$2

with  --base-uri=/sample i get into a redirect loop (i guess this has to do with the cookies)
without  --base-uri it redirects without the /sample (i do have my nginx controller set x-forwarded-prefix, but its ignored)

How did you get it to work?
tx.,

Thanks for asking and sorry for the delayed response. I am not at good at explaining things but I try my best to explain. When I mentioned the custom variable, the variable is added in this configuration (link).
What I did was to modify these files:
doc.go - Added  TraefikAPIURI variable
        BaseURI string `json:"base-uri" yaml:"base-uri" usage:"common prefix for all URIs" env:"BASE_URI"`
	// OAuthURI is the uri for the oauth endpoints for the proxy
	OAuthURI string `json:"oauth-uri" yaml:"oauth-uri" usage:"the uri for proxy oauth endpoints" env:"OAUTH_URI"`
	// TraefikProxyPath is the uri for the authorize endpoint since /oauth/authorize does not include Traefik path as a base address
	TraefikAPIURI string `json:"traefik-api-uri" yaml:"traefik-api-uri" usage:"the uri for /oauth/authorize endpoints" env:"TRAEFIK_API_URI"`
	// Scopes is a list of scope we should request
	Scopes []string `json:"scopes" yaml:"scopes" usage:"list of scopes requested when authenticating the user"`
	// Upstream is the upstream endpoint i.e whom were proxying to

handlers.go - added a custom code to use the variable specified in the above configuration inside oauthCallbackHandler function (should be at line 104)
	if r.config.TraefikAPIURI != "" {
		redirectURI = fmt.Sprintf("/%s%s", r.config.TraefikAPIURI, redirectURI)
	}

	r.redirectToURL(redirectURI, w, req, http.StatusSeeOther)

With this variable, I can add the path prefix and make sure Keycloak will add that path prefix when redirecting back to the /oauth/authorize URL, as described by the OP's expectation for the URL.

Thanks, i get it. Thanks for your explanation :)
For now i am trying a different path (istio with oauth2-proxy). I might abandon that path and return to louketo-proxy. It seems however this project is EOL. This means i will have to move at one time anyway.  (FYI oauth2-proxy as a sidecar proxy, was also not able to work with a path prefix. Hope to get more luck by combining Envoy and oauth2-proxy.
Can't understand why this has to be soo difficult :(
Thanks again

It's unfortunate to see that the project is EOL. I am still watching the thread to see anyone stepping up to maintain this project. There are people talking about continuing the project so let's hope that this project will be on-going.
I have tried other repo like oauth2-proxy, etc but none of them seems to work well with a path prefix, only Keycloak Gatekeeper seems to support but there are limitations as well. That's why I had to design a solution and fork the project to our private server.
Hopefully, someone will take over the project.
Cheers, and you're welcome.


taxonomy base fix

when using taxonomy base from ENV, do not override with the default value from get_params
also place taxonomy path inside of this ENV check. If a user provides a global cfg with ANY of these paths, models, etc we should not override w/ the defaults

Checklist:

[ ] Commit Message Formatting: Commit titles and messages follow guidelines in the
conventional commits.
[ ] Changelog updated with breaking and/or notable changes for the next minor release.
[ ] Documentation has been updated, if necessary.
[ ] Unit tests have been added, if necessary.
[ ] Integration tests have been added, if necessary.

@mergifyio rebase


WIP: Topic/heightmap cost - [opened]

In GitLab by @cmastalli on Sep 30, 2019, 18:20
Merges topic/heightmap-cost -> devel
Added friction cones in the WF added heightMap Class added CostModelHeightMapDistance for obstacle avoidance

In GitLab by @cmastalli on Sep 30, 2019, 18:21
mentioned in merge request !219

In GitLab by @cmastalli on Sep 30, 2019, 18:22
@mfocchi I rebase your branch with respect to the latest devel branch.
I will start to add comments about this PR.

In GitLab by @cmastalli on Sep 30, 2019, 18:24
Why we need this? We already have a display callback.
Please remove this callback. We are managing threads at the c++ level.

In GitLab by @cmastalli on Sep 30, 2019, 18:24
same for this. Please remove this class from the PR.

In GitLab by @cmastalli on Sep 30, 2019, 18:26
You're creating memory in Python. Please do like this:
data.Lxx[:, :] = np.dot(data.Rx.T, data.Rx)

In GitLab by @cmastalli on Sep 30, 2019, 18:26
Remove commented lines and add the following:
data.Lxu[:, :] = np.dot(data.Rx.T, data.Ru)

In GitLab by @cmastalli on Sep 30, 2019, 18:27
same here, don't allocate memory. This might also break the code.
data.Luu[:, :] = np.dot(data.Ru.T, data.Ru)

In GitLab by @cmastalli on Sep 30, 2019, 18:28
calc also returns the next state
return data.xnext, data.cost

In GitLab by @cmastalli on Sep 30, 2019, 18:28
calcDiff also returns the next state
return data.xnext, data.cost

In GitLab by @cmastalli on Sep 30, 2019, 18:32
We don't use models to store data. This is very bad. Please do as follows:

Create function to set heightmap, first-point and grip resolution.

def setHeighmap(self, data, heightmap, firstPoint, gridResolution):
// Store this information inside data. For that you need to create a specialized data class
.
.

Use data to retrieve information in all the functions (getIndex, getHeight, getHeightGradient
Embedded this class inside the cost one (i.e. CostModelHeightMapDistance)

In GitLab by @cmastalli on Sep 30, 2019, 18:34
Pass the data, i.e.
def getIndex(data, xy):

In GitLab by @cmastalli on Sep 30, 2019, 18:35
same here, use data

In GitLab by @cmastalli on Sep 30, 2019, 18:35
same here, use data

In GitLab by @cmastalli on Sep 30, 2019, 18:35
Remove heightMap from here. As mentioned above, everything should be contained in data class

In GitLab by @cmastalli on Sep 30, 2019, 18:39
You create memory, always access to the block memory using [:, :]
data.residuals[:] = p[2] - (self.heightMap.getHeight(p[:2]) + self.clearance)

In GitLab by @cmastalli on Sep 30, 2019, 18:41
Add the data here, as mentioned!

In GitLab by @cmastalli on Sep 30, 2019, 18:42
Remove this, keep the contact object. This is done in data.f = data.contact.f. Again keep the code as before!

In GitLab by @cmastalli on Sep 30, 2019, 18:45
Why do we need to rotate?
@nmansard what is your thought?

In GitLab by @cmastalli on Sep 30, 2019, 18:46
The code is not passing since changes in the force. You need to return them as mentioned

In GitLab by @mfocchi on Nov 8, 2019, 15:05

we need to rotate cause it is easier for the user to express the cones in an inertial frame rather than in the local frame,
the height map  is tipically given in an inertial frame

In GitLab by @jcarpent on Nov 8, 2019, 15:10
I do not think it is changing in terms of memory allocation

In GitLab by @cmastalli on Nov 8, 2019, 15:21
This is an old comments, it was not using eigenpy just numpy.

In GitLab by @cmastalli on Nov 8, 2019, 15:26
These are Jacobians with higher dimensionality of friction-cone vertices.
I think it could be faster to rotate the vertices of the friction-cone.
What do you think?

In GitLab by @cmastalli on Nov 8, 2019, 15:28
@mfocchi - after discussing with @gfadini, we have agreed to start only with the height-map.
We have an ongoing discussion about forces in the riot chatroom (@proyan and I), about software API.
First, to integrate this nice stuff, we need to agree on this matter.
We also want to do it in such a way that can be penalized the CoP for both cases: humanoids and quadrupeds.
These cost functions are important for us too, we need to deploy an MPC controller with ANYmal (@wxmerkt and I).

In GitLab by @mfocchi on Nov 8, 2019, 15:31
"I think it could be faster to rotate the vertices of the friction-cone."
it is doable, we just wanted to modify the existing code as little as possible. Verify with a unit test that it gives the same result

In GitLab by @cmastalli on Nov 8, 2019, 15:55
Yes, I understand.
We could start as it is, but again, first we need to agree with the API. Once this is done, you and I should work on it :smiley:


Modify allow-single-line value

The rule allow-single-line must be set on true instead of false, since our style guide allows this format.
👋  @jvnni, @pmontesano, are you okay with this?

@battaglr esta ok para mi!

Resolved by<PHONE_NUMBER>25c6676708d41fd4fbcf54ae835e94.


Add anti alias support for linear Canvasitem draw primitives

(This PR is an follow-up for https://github.com/godotengine/godot/pull/84472, kept as draft for now. see https://github.com/godotengine/godot/pull/84472#issuecomment-1793736388)
Add anti alias to the following canvasitem draw primitives

unfilled circle and rect
multiline and dashed_line

func _draw():
	draw_circle(Vector2(100,100), 100, Color.BROWN, false, 10)
	draw_circle(Vector2(100,400), 100, Color.BROWN, false, 10, true)

	draw_rect(Rect2(400, 0, 200, 200), Color.AQUA, false, 10)
	draw_rect(Rect2(400, 300, 200, 200), Color.AQUA, false, 10, true)

	draw_dashed_line(Vector2(700, 0), Vector2(900, 200), Color.CADET_BLUE, 10, 20, true)
	draw_dashed_line(Vector2(700, 50), Vector2(900, 250), Color.CADET_BLUE, 10, 20, true, true)

Could you show how antialiased multiline with transparency looks?

@MewPurPur  The multiline drawing behaviour should be identical to the original line. Here are some examples.

Any updates on this?

It's intended as a follow-up to a PR that hasn't been merged yet, so see:

#84472

@xiongyaohua Now that https://github.com/godotengine/godot/pull/84472 has been merged, you should be able to rebase this PR onto the latest master branch.

@Calinou will do in the next week.

Now that https://github.com/godotengine/godot/pull/84472 has been merged, you should be able to rebase this PR onto the latest master branch.

I'm going to close this PR, as all codes are incorporated into the following PR https://github.com/godotengine/godot/pull/84523.  Review can be done over there.


time: truncated times don't imply truncated time differences

An interesting ramification of the new monotonic time issue, perhaps not solvable.
I just wrote some code that looked a bit like this:
 t0 := time.Now().Truncate(time.Millisecond)
 something()
 t1 := time.Now().Truncate(time.Millisecond)
 fmt.Println(t1.Sub(t0))

and expected it to print a duration that's a whole number of milliseconds but it doesn't, because of the variable skew between the monotonic and wall clock times.
https://play.golang.org/p/pn2AWxrRU0
go version devel +5f374ea Mon Feb 6 22:45:49 2017 +0000 linux/amd64

Maybe Truncate should also remove the monotonic bit in its returned time? Or keep it but truncate its monotonic value? (which would be a unique case, though)
@rogpeppe, was that hypothetical or did it appear in real code?

Well, it was real code that I just wrote and expected to work, not code that I wrote to demonstrate a problem I thought might exist.
If I did it, I bet others have done it too.

I was on the fence about whether this was a good idea. I did it this way to see what would break. Please keep reporting instances of this kind of thing. Probably we'll want to make Truncate and Round clear the monotonic reading for Go 1.9, but let's keep gathering data.
I wonder if @rogpeppe's example means that Duration should also have Round and Truncate. Then you could use time.Since(t0).Truncate(time.Millisecond), although I typically use fmt.Printf("%.3fs", time.Since(t0).Seconds()), which is nicer anyway.

I would've used Duration.Truncate it it had existed.
The not-so-nice thing about fmt.Printf("%.3fs") is that it doesn't print so nicely for minutes and hours.

The printing is a great argument for Duration.Truncate. We should do that regardless. If someone wants to file an issue and/or send a CL, that's great.

I looked briefly and non-scientifically at calls to Now().Round/Truncate in go-corpus, and it does look like many of them would be broken by preserving the monotonic clock as we do today. I didn't check them originally because the original proposal was for them to strip the reading, along with In, Local, and UTC. @rogpeppe, you were the one who talked me into not stripping for In, Local, and UTC, and I added Round, Truncate to the list too. I should check on In, Local, UTC too. The argument for stripping in those cases is that you're in some sense explicitly asking to work in wall times.

See #18996 for Duration.Truncate.

In, Local and UTC change the wall clock time independent of the monononic time, so I think they're OK. Truncate and Round make an adjustment that is dependent on the absolute value of the time, which is an argument for making them strip monotonic time, I think.

Another case of code broken by Truncate preserving the monotonic clock:
https://github.com/youtube/vitess/blob/master/go/vt/throttler/interval_history.go#L41
The full context is spread around a few other files, but the short description is that this code is mapping times into windows and assumes that all times truncated to a given window are equal.

In case it's helpful, see https://github.com/google/go-github/pull/568 for a real world case test that fails on Go master in google/go-github repository. The change in that PR makes it pass.
Note that the original code uses != on time.Time values, rather than time.Equal, for doing comparisons. From package time documentation, using == operator on time.Time values seems to be discouraged, but not deemed illegal (from my reading of docs in 1.8 and latest master).

I'll send a CL for Round, Truncate; we can leave this issue open as a reminder to decide about In, Local, UTC.

I wanted to revisit one phrase in the time documentation after 6c85fb08c27c0346125567573e0f0a2b8124bc3b.

If time values returned from time.Now and time values constructed by other means (for example, by time.Parse or time.Unix) are meant to compare equal when used as map keys, the times returned by time.Now must have the monotonic clock reading stripped, by setting t = t.AddDate(0, 0, 0).

After 6c85fb08c27c0346125567573e0f0a2b8124bc3b, there are 3 ways to strip monotonic clock reading:

by setting t = t.AddDate(0, 0, 0).
by setting t = t.Round(0).
by setting t = t.Truncate(0).

I wanted to ask if suggesting t = t.AddDate(0, 0, 0) as a way of stripping monotonic clock reading is still the best option out of the three, after 6c85fb08c27c0346125567573e0f0a2b8124bc3b.
I think it's fine, but just wanted to ask.

@rsc, decision?

In Google's code base, fixing UTC and Local to strip the monotonic reading would fix about half as many tests as fixing Truncate / Round did (but different ones). It seems worth doing. I will send a CL.


Crash after closing dialog (sometimes)

The menu transitioning needs to be safer.

 Make menu transitioning safer


Python 3 incompatible

Currently the script is incompatible to python 3. It throws following error:
  File "dnstwist.py", line 97
    print 'dnstwist (' + __version__ + ') by<EMAIL_ADDRESS>                     ^
SyntaxError: invalid syntax

I use
Python 3.4.2 (v3.4.2:ab2c023a9432, Oct  6 2014, 22:16:31) [MSC v.1600 64 bit (AMD64)] on win32

Adding parenthesis to all prints that do not have them solves the issue. I don't know if it's backwards compatible that way, but I whink python 3 should be preferred over python 2.

Hi. Thank you for pointing this out. It is fixed now.
$ python3 dnstwist.py
dnstwist (20150615) by<EMAIL_ADDRESS>Usage: dnstwist.py <domain>

Nice, thank you for your fast response :)


Can't submit stats for leaderboard

Our team has been using the xbox-live-unity-plugin for UWP builds of our game.  Everything is going well, but we cannot for the life of us figure out how to properly submit stats.  It seems like it would be intuitive, but we've had smart people working on it for over a week and we can't seem to solve this simple problem.
We've found several web pages that seem to explain the process such as:
https://docs.microsoft.com/en-us/windows/uwp/xbox-live/get-started-with-creators/add-stats-and-leaderboards-in-unity
… but all of the pages we've found have parts that are unclear or out of date.
We've been experimenting non-stop, but we haven't successfully saved a stat even once.  At this point we're making projects that consist of a single scene that does nothing but try to set a stat with no luck.  If anyone can provide us with some tips on how to submit a stat, either through code or using the prefabs in the xbox-live-unity-plugin, we'd greatly appreciate it.
We set up our stat (Top Score) in the Developer Center.  We have sign in working properly so we're acquiring a user.  We have an Xbox Live Services prefab in our first scene.
We've tried dropping an IntegerStat prefab into our scene and manipulating it.  That appears to only affect the variable locally.
We've tried writing a C# script that does nothing but set a score to 20 (just as a test).  It basically calls  XboxLive.Instance.StatsManager.SetStatisticIntegerData(user, "1", 20) directly after setting user to be the current XboxLiveUser.
We've tried a lot.  Just not the right thing, apparently.  Any suggestions?

Here is a video detailing how to set stats.
If this doesn't work or you're still having issues feel free to reopen this.

Thanks for sharing that video (and for making it).  It was very well made.  Unfortunately we've watched it many times and I still cannot get Stats to save.  I've gone so far as to make a minimal sample project that does nothing but attempt to save and retrieve a single stat.  I'm hoping that if I share the code from that here someone will be able to see what we're missing.
There are two scenes in the project.  The first one just signs the player in and then loads the second scene.  We know that part is working properly because the PlayerAuthentication prefab displays the correct Gamertag.  The second scene attempts runs through three things.  It attempts to retrieve a stat, then save a stat, then save a stat using an alternate method.  Evey time we run it the "retrieve stat" part cannot find a stat.  Here is the code.
`using Microsoft.Xbox.Services;
using Microsoft.Xbox.Services.Client;
using Microsoft.Xbox.Services.Statistics.Manager;
using System.Collections;
using UnityEngine;
using UnityEngine.UI;
public class MainTestScript : MonoBehaviour {
public Text message;
public IntegerStat Stat;
XboxLiveUser user;
StatisticValue statValue;
bool started = false;

private void Start()
{
    StartCoroutine("RunScript");
}

IEnumerator RunScript()
{
    yield return new WaitForSeconds(5f);
    GetUser();
    yield return new WaitForSeconds(10f);
    GetStat();
    yield return new WaitForSeconds(10f);
    SetStat();
    yield return new WaitForSeconds(50f);
    SetStatAnotherWay();
    yield return null;
}

void GetUser()
{
    user = SignInManager.Instance.GetPlayer(1);
    message.text += "\n" + "User = " +  user.Gamertag.ToString();
}

void GetStat()
{
    try
    {
        statValue = XboxLive.Instance.StatsManager.GetStatistic(user, "test");
    }
    catch
    {
        message.text += "\n" + "Unable to get stat from Xbox Live";
    }
    if (statValue != null)
    {
        message.text += "\n" + "Pulled score of " +  statValue.AsInteger.ToString() + " from Xbox Live.";
    }
}

void SetStat()
{
    Stat.Value = 321;
    StatsManagerComponent.Instance.RequestFlushToService(user, false);
    message.text += "\n" + "Set IntegerStat to 321 and Flushed";
}

void SetStatAnotherWay()
{
    XboxLive.Instance.StatsManager.SetStatisticIntegerData(user, "test", 123);
    message.text += "\n" + "Set stat to 123 directly and Flushed";
}

}
`
We have a PlayerAuthentication Prefab in this scene that correctly displays the gamertag of the signed in user.  We have a text field called message (obvious by the code snippet) that correctly displays the gamertag as well.  Either we are not correctly saving the stat, or we are not correctly loading the stat.  Since we also have also tried using the Leaderboard prefab in another scene (hooked up to a correctly filled out IntegerStat prefab) and it shows as blank we're guessing it's the saving part we aren't getting.
Here are some things we have already thought of:
We configured the stat with ID test (all lowercase) in Dev Center and pressed pressed Test (multiple times all with a Success message).
Our IntergerStat prefab has an ID of test (all lowercase).  We triple checked.
Our project is successfully Associated with our Game through Visual Studio.
We've tried this on PC and Xbox One.  We've tried it with Debug projects as well as fully packaged projects.
Any ideas?

The code looks right. While the GetStatistic method is great for testing, you shouldn't use it as your production way of retrieving stat info. Featured Stats is not meant to be the source of truth and it is up to the game to ensure that the values that Featured Stats have are correct.
Some additional things you can check:

Ensure the user is added to StatsManager
Ensure that StatsManager.DoWork() method is being called regularly and that the LocalUserAdded event has been returned
Check that StatsManager.DoWork() returns a StatUpdateComplete event

In addition, can you capture a fiddler trace of retrieving/setting the stat?

Thanks rakhad.
Thank you very much for your reply and for all your help.  We truly appreciate it.
I would like to check on the three things you suggested and then capture a fiddler trace, but I don't have the slightest idea what any of what you said means.  I'm trying to comprehend it, but to me it's like watching the last 5 minutes of an episode of Star Trek: Voyager where they shout out a bunch of technobabble that magically solves all their problems.  Here is my interpretation of what I just read.

You're saying an XboxLiveUser has to be added to something called a StatsManager?  Is this something I was supposed to me doing in code?  I have no idea what this means let alone how to verify it's happening.  I would google it, but I already read through every piece of documentation on the Xbox Live Unity Prefab and Xbox Leaderboards/Stats many times and I didn't see it explained anywhere so there's literally nothing I can do.

I have no idea what the StatsManager.DoWork() method is, if I'm supposed to be calling it myself somehow, or how I would verify it's working.  Also, there's a LocalUserAdded event?

3, Again, no idea what StatsManager.DoWork() even is.

There is absolutely no chance of any of us learning what Fiddler is in order to figure out why Stats/Leaderboards aren't working.

At this point our goal is simple.  There is clearly correct code to "try" to save and load stats.  We just want to know what it is.  If you're saying our code looks like it should have worked, then I guess that means we already have it.  If we're already trying the correct code and it isn't working then we'll simply have to conclude that Xbox Leaderboards and Stats through the Prefab are not working correctly and give up on putting them in our game for now.  If anyone sees a flaw in our code and would suggest trying something different please let us know.
Again, sincere thank you to rakhad and to anyone else who decides to contribute a suggestion.

Hi Andy.
I have one idea what might be causing your problem. The Xbox Live Sandboxes are absurdly unreliable. It could be that your script would work fine in a published game but is failing in the test environment. I don't see a good way to test that theory. I suppose you could finish your game using the save code you showed here and hope for the best but that doesn't seem like a good idea.  I would tell you what I think of your code but to be honest I have never been able to get leaderboards working on Xbox either. When I saw your post I tried again over the weekend and had no luck. The only feedback I can give you is that I tried your exact script you posted and it didn't work for me either.
I do have one question for Ramsey Khadder if he sees this again. Of the two save stat methods Andy posted which one looks correct? SetStat() or SetStatAnotherWay()? I'll probably keep experimenting and trying to get it to work but knowing which one of those looks correct to you would be a big help. Also, is that method the ONLY code that needs to be called in order to save a stat or is there a missing step?  If I can find a solution during my testing I'll post it here for Andy and anyone else having similar issues.

Update:  We haven't been able to get stats to save but we can confirm a few things.  First, the problem is not related to the sandbox environment.  We added a High Score stat to a shipped game and it will not save there either.  Here is the code we tried.
  stat.Value = score;         XboxLive.Instance.StatsManager.SetStatisticIntegerData(user, "score", score);         StatsManagerComponent.Instance.RequestFlushToService(user, true);
That is the actual code we run to try to save the stat.  We haven't included any other code since it is not relevant.  Through other code and testing we are certain of a few things:
The variable "Stat" has been correctly linked to an IntegerStat prefab.
A use is definitely signed in.
the variable "score" is definitely higher than zero in all of our tests.
The only logical conclusion is that setting an IntergerStat to a value and then calling   StatsManagerComponent.Instance.RequestFlushToService(user, true);
is clearly not the correct way to try to save a stat using the Xbox Live Unity Plugin.  There is something missing.  It is a shame that there are no instructions provided on how to use the plugin as some simple instructions would have eliminated this problem.  Instead we are left with only a monolithic example to try to reverse engineer if we want to try to decipher what code we're supposed to call in order to actually use the prefabs in the plugin, then do a series of trial and error research to test out theories.
If anyone from Microsoft sees this, how about just posting the actual line or two of code that we're supposed to be calling to save a stat using your plugin so that we, Andy, and anyone else who finds this thread can actually use it?
If my tone sounds harsh, I apologize, but I am very frustrated by this.  I've sunk a ridiculous amount of hours into trying to get this to work.  For comparison's sake, this took us 5 minutes to get up and running on every other console we've developed for, and we develop for all of them.

Sorry for the delay.
Something interesting seems to be going on as you are calling the right methods. I will do a deep dive and report my findings by Monday. Thank you for your patience.

I worked on this over the weekend, but the only thing I have to report that could be any useful is that I tried to get it to work with a much older version of the Xbox Live Unity Plugin that I still have and I couldn't get it to work there either.  Then again that version was from the launch of the Creator's Program so it was so wildly different I might not have been trying the correct things.
I know for sure that it's a stat saving issue not a stat recall issue because the game "Space Command" which is in the store tries the code I've posted to save at multiple points in the game.  It's been selling well but zero people have a stat to report on their game hubs and the leaderboard is empty.
At least I have that game tracking people's personal high scores in local memory, so if I ever get this working I can add proper leaderboards.

How do you handle sign in? Are you using the PlayerAuthentication prefab, using the SignInManager, or writing your own code?

We have Stats and Leaderboards working now thanks to your example.  We even updated one of our published games and it has working Leaderboards now (thank you SOO much).  We plan on walking back some of the changes and testing at each point to find exactly what the problem was and then posting that result here (even if the thread is closed) to help the person who originally opened this thread and anyone who finds it in the future.  Since we haven't heard from the original poster in 10 days it might be worth closing the thread if he doesn't post something soon.

Glad to hear its working.
By default stats are written to the service in 5 minute intervals to reduce load on the service. This might have been why stats weren't showing up. Flushing with a low priority takes all the pending stat writes and attempts to write them to the service on a 30 second interval. Flushing with a high priority takes all the pending stat writes and writes them immediately; however, flushing with high priority can only be called once every 30 seconds. You should flush with low priority when you finish a level and you should flush with high priority when the game is closing. I've included a few examples below.
Context

Stats manager is initialized, the default 5 minute timer begins now, the priority 30 second time begins now
Level starts
30 seconds go by
Level finishes
Write stat to IntegerStat.Value
Next level starts
1 minute go by
Level finishes
Write stat to IntegerStat.Value
45 seconds go by

At this point we have 2 pending stat writes and 2 minutes and 15 seconds have elapsed
Default Behavior

There is 2 minutes and 45 seconds left on the timer before stats manager automatically uploads the pending 2 stat writes. If you attempt to get the stat value or get a leaderboard during that 2 minutes and 45 seconds then the value returned by the service will not reflect the pending writes.

Low Priority Flush

Lets say we make a low priority flush after step 10.
All pending stat writes are moved from the default queue to the priority queue. There is 15 seconds left on the priority timer before stats manager automatically uploads the pending 2 priority stat writes. If you attempt to get the stat value or get a leaderboard during that 15 seconds then the value returned by the service will not reflect the pending writes.

High Priority Flush

Lets say we are using the default behavior and our 2 pending stat writes are on the default queue. Lets also say the user decides to close the game and Unity gives us the opportunity to handle the game exiting. In the Unity callback we would call a high priority flush.
Stat manager will attempt to immediately write all pending stat writes to the service assuming there has not been another high priority flush in the last 30 seconds. In this case there hasn't been and the stat writes are immediately written to the service.
Note if there has been a high priority flush in the last 30 seconds then the pending stat writes will be moved to the priority queue and will have to wait until the next time the 30 second priority timer completes.

More detail can be found in the stat manager docs.


Background Job

Find out why on Android Google stock, after some days the widget stops updating at the right frequence

Wait some days, then execute
adb shell dumpsys jobscheduler | grep "renshuu"
to check


"Add special" for the opening hours editor

Special intervals for the editor: PH, Tu[-1], Dec 24 etc.

Done in 0.7.


turn on prefer-const

per https://google.github.io/styleguide/jsguide.html#features-use-const-and-let

@chrisjshull can you sign the CLA so this can be merged? If not please let me know and I'll add this myself.


please reopen: google_travel _time options: mode can't take the state from an " input_select" or sensor as argument

please continue the recently closed  https://github.com/home-assistant/home-assistant/issues/11606
would be really helpful if it became possible to select travel mode through an input select or automation in HA.
  travel_mode:
    icon: mdi:bike
    name: Travel mode
    options:
      - driving
      - bicycling
      - walking
      - transit
    initial: bicycling

I have reopened the original, so will close this one ...


refactor(frontend): Introduce new size for Logo component

Motivation
In PR #3028 we are going to use an additional size for Logo, that is 42px.
So, we "shift" all the sizes so that the new mapping is
const sizes = {
  xs: '22px',
  sm: '36px',
  md: '42px',
  lg: '52px',
  xl: '64px'
};

[OISY-69]


TeachOpenCADD as Jupyter Book?

Explore Jupyter Book to render TeachOpenCADD Jupyter Notebooks:

https://blog.jupyter.org/announcing-the-new-jupyter-book-cbf7aa8bc72e
https://jupyterbook.org/intro.html
https://jupyterbook.org/start/build.html

Also, maybe use this Scientific Python introduction for talktorial 017 (rendered with Jupyter Book)?
https://python-programming.quantecon.org/index_toc.html

Worth a deeper look in the future, for sure. Do you think it's something that'd replace our current plans or just an addition on top of the current efforts?

I am closing this issue now, since we are not planning to set up a Jupyter Book in the foreseeable future (we have the website instead).


use of undeclared crate or module tonic_build

error[E0433]: failed to resolve: use of undeclared crate or module `tonic_build`
 --> build.rs:2:5
  |
2 |     tonic_build::compile_protos("proto/main.proto")
  |     ^^^^^^^^^^^ use of undeclared crate or module `tonic_build`

Cargo.toml
[dependencies]

tonic = "^0.5"
prost = "^0.8"
futures-core = "0.3"
tokio = { version = "^1.6", features = ["net", "signal", "io-std", "io-util", "rt-multi-thread", "macros", "sync", "time"]}
tokio-stream = "^0.1"
async-stream = "^0.3"

[dev-dependencies]
http = "^0.2"
fake = "^2.4"
tonic-build = { version = "^0.5", features = ["prost", "compression"] }

it is supposed to be [build-dependencies]

Well i know its closed topic a long time ago but, i had the same issues with newer versions (0.12.1) but i managed to fix it by adding tonic-build for both [dependencies] and [build-dependencies].


Question and other marks are not displayed

Are you experiencing an issue with...
shields.io
🐞 Description
I am trying to pass in the image text a question mark.  "?"
Obviously it is not shown because it is used to define parameters in the URL get
They don't work either:
HTML Entity:	? ? ?
UTF-8 Encoding:	0x3F
UTF-16 Encoding:	0x003F
🔗 Link to the badge
https://img.shields.io/badge/SERVER-PRIVATE?-1?style=for-the-badge&logo=&logoColor=white&color=000
https://img.shields.io/badge/SERVER-PRIVATE0x003F-1?style=for-the-badge&logo=&logoColor=white&color=000
💡 Possible Solution
Integrate in your server a function that detects an especial code for some symbols that url cant have.
For example:  +q+ for question mark
Love and thanks  <3

You need to use URL encoding: %3F
example: https://img.shields.io/badge/questionmark-%3F-blue


Avoid/optimize packages.Load

For parsing AST, we use lib called "packages", is go stdlib from x
Currently this lib requre all go installation in host, so it platform dependant thing.
And most horrible, we need to include all GO setup inside docker release images.
In this ticket, we need at least remove all not needed files from go18-buster image and reduce linter size
and at most, replace packages.Load for something else.
read more:

https://github.com/golang/mock/pull/420
https://github.com/vektra/mockery/issues/258

I haven't really dug deep into go-arch-lint's source code, but it seems to me, that linting dependencies could be done solely by parsing import statements in a package, which in itself is a rather trivial thing to do. Perhaps, we could replace packages with custom written parser specifically for import sections?

For linting imports, we don't have to use 'packages' at all, that's how it been before in old versions.
But linter with the deepScan flag enabled, reads all GO code, not just imports.
This required to check for dependency injection, function calls, etc.


docs: Init Algolia Docsearch

:tada: This PR is included in version 0.3.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


qbs: 1.23.0 -> 1.23.1

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/metapackage/qbs/versions.
meta.description for qbs is: "A tool that helps simplify the build process for developing projects across multiple platforms"
meta.homepage for qbs is: "https://wiki.qt.io/Qbs"
meta.changelog for qbs is: ""
Updates performed

Version update

To inspect upstream changes

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 1.23.1 with grep in /nix/store/gd7cajsvsifg10c0f6vs91lgwl6iwrl4-qbs-1.23.1
found 1.23.1 in filename of file in /nix/store/gd7cajsvsifg10c0f6vs91lgwl6iwrl4-qbs-1.23.1

Rebuild report (if merged into master) (click to expand)

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
qbs

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/gd7cajsvsifg10c0f6vs91lgwl6iwrl4-qbs-1.23.1 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A qbs https://github.com/r-ryantm/nixpkgs/archive/66233b81188e9eb2b4541b11ba149837032aa630.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/gd7cajsvsifg10c0f6vs91lgwl6iwrl4-qbs-1.23.1
ls -la /nix/store/gd7cajsvsifg10c0f6vs91lgwl6iwrl4-qbs-1.23.1/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

qbs

Maintainer pings
cc @expipiplus1 for testing.

Result of nixpkgs-review pr 187025 run on x86_64-linux 1

1 package built:

qbs


EF6.4, CodeFirst 1.6, no such a table

var sqliteConnectionInitializer = new SqliteCreateDatabaseIfNotExists(modelBuilder);
Database.SetInitializer(sqliteConnectionInitializer);
modelBuilder.Conventions.Remove();
modelBuilder.Conventions.Remove();
modelBuilder.Conventions.Remove();
modelBuilder.Configurations.AddFromAssembly(typeof(SQLiteDBContext).Assembly);
How to solve it？

Please give me a code sample

Or, DatabaseExists is not supported by the provider.
public class Configuration : DbMigrationsConfiguration<SQLiteDBContext>
    {
        public Configuration ()
        {
            AutomaticMigrationsEnabled = true;
            AutomaticMigrationDataLossAllowed = true;
            SetSqlGenerator("System.Data.SQLite", new SQLiteMigrationSqlGenerator());
        }

        protected override void Seed(SQLiteDBContext context)
        {

        }
    }

Database.SetInitializer(new MigrateDatabaseToLatestVersion<SQLiteDBContext, Configuration>());

The code sample is located here: https://github.com/msallin/SQLiteCodeFirst/tree/master/SQLite.CodeFirst.Console
If it doesn't work, upload a repro (as small/striped down as possible) and upload it, so that I can have a look at it.

Closed due to inactivity.


[Feature Request] You can add a callback parameter to the onClick callback of the Notification component to identify who I clicked?

Existing Component
是
Component Name
Notification
Description
I found the onclick event that i can not get the parameter to  identify who I clicked when use the Notification component , You can add?

you can add a closure in the callback function to get the identifier.


Display the OOS stories in their correct column

What
The physical board, didn't really have the cards in review, waiting for slots to free up. This change will make a similar appearance.
Personally, I don't really like it, but that's the way it use to be.
How to review
You can:

Wait for the moment, when there will be real out of slot stories
Mess with pivotal, to achieve the desired state
EASIEST/PREFERED: Set the REVIEW_SLOTS_LIMIT and APPROVE_SLOTS_LIMIT to 1 before launching the app.

Fire the app, and see if it suits our needs.
Who can review
Not @paroxp

it works, but THB, the result is less clear. Current form gives very nice overview of which story is in which state. Now review stories are in doing and review column and approval are in review and approval column.
Given that we already have logic to show story counts in red if we have more of them than slots, i'd call this functionality superfluous. It makes the app worse for me.


fix(button-group): update border from 2px to 1px

Checklist

[ ] Fix linting errors
[ ] Tests have been added / updated (or snapshots)

Change information

:tada: This PR is included in version 1.0.0-alpha.20 :tada:
The release is available on:

npm package (@alpha dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 1.1.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Add clock interface

2nd commit needs reviewing: I ran into a weird issue, where lifecycling failed due to asking for observations at epoch (1970-01-01T00:00:00Z) for dynamic claims instruments. This was due to the condition in this line not holding true for freshly initialized dynamic claims instruments. For those, after the change to put observations (and everything else, i.p. dateToDateClockTime) to midnight instead of noon, acquisition time == today upon first lifecycling, therefore it entered the lifecycle function and requested the observations for epoch.
I think this just happened to work before, because we (accidentally?) mapped acquisition time to epoch noon (due to dateToDateClockTime impl), and today date was just epoch. Therefore, the condition acqisitionTime > today held true (by accident).
I'm not sure the fix I did (hardcoding acquisition time for those instruments to epoch + 1 instead of epoch) is the right / best one, it needs discussing.

@georg-da I implemented the change we discussed on the whiteboard.
I realised there are a couple of interfaces which are undocumented (my bad), will fix in a separate PR

Seeing the various removals of the dependency on Data I think this was a good change.


judges-action#191: add review_comments to Fbe::FakeOctokit

@yegor256 please merge this pr

@rultor release, tag is 0.0.65

@rultor release, tag is 0.0.65

@yegor256 OK, I will release it now. Please check the progress here

@rultor release, tag is 0.0.65

@yegor256 Done! FYI, the full log is here (took me 21min)

@Suban05 Great job on the review! 🌟 You've earned +4 points: base +4, -10 for limited comments, +10 bonus. Your dedication is appreciated! Current balance: 388 points. Keep up the excellent work!

@Yegorov Hey there! 👋 Thanks for your code contribution! You've earned +4 points this time. Here's the breakdown: +4 as a base, -4 because there were only 2 comments (we love feedback!), and +4 to keep you motivated. Remember, more hits-of-code and active code reviews can boost your points next time. Keep the contributions coming! Your current balance is +367. Looking forward to your next awesome contribution! 💻🚀


What happened to the web pages - https://www.ospray.org/ospray_studio/

This site had some very good still images and panoramas of the Bentley car show room.  The metallic materials used on the cars are superior to what Cadalog can render in our new SketchUp plug-in called PodiumxRT.  I would really like access to those images and panoramas.

Sorry, that was a glitch when we migrated repositories. Studio's website is available again.

Thank you
On Mon, Apr 29, 2024 at 5:03 AM Johannes Günther @.***>
wrote:

Sorry, that was a glitch when we migrated repositories. Studio's website
is available again.
—
Reply to this email directly, view it on GitHub
https://github.com/RenderKit/ospray-studio/issues/31#issuecomment-2082549499,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAYGNWJUHZ7GCVZZUN45KZDY7YZH5AVCNFSM6AAAAABG5DDPNGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDAOBSGU2DSNBZHE
.
You are receiving this because you authored the thread.Message ID:
@.***>


Add method to access a sub-layer in Sequential.

Trivial addition to Sequential that allows obtaining a reference to a sublayer.
I used an index type of i64 because that's what len() currently returns, I'm not sure why it is using an i64 and not an usize?

Closing this: I think sequential is intended as a trivial forward-pass only container.


Feature request: includeFor integration

It is useful to have an integration only apply for a specific service.
We have this in Ruby like so: https://github.com/smithy-lang/smithy-ruby/blob/779dd349560371b525758a45e83e21673e67753a/codegen/smithy-ruby-codegen/src/main/java/software/amazon/smithy/ruby/codegen/RubyIntegration.java#L45
Kotlin also has this: https://github.com/smithy-lang/smithy-kotlin/blob/main/codegen/smithy-kotlin-codegen/src/main/kotlin/software/amazon/smithy/kotlin/codegen/integration/KotlinIntegration.kt#L60
I'm unsure of the recommended method arguments for this, but the need is for integrations to only apply in some cases, such as s3, or if a service has a particular trait. Such a method (I propose includeFor) should run before preprocessModel and other integration hooks - it should be the very first thing checked IMO.
Currently we run into issues using preprocessModel, which runs before our hook to includeFor, so each preprocessModel method has to check and return the model if certain shapes aren't present or if the service model isn't what the integration is targeting.

There's not really a super clean and robust way of offering this generically. The most flexible way would be to add something like this to the interface:
default boolean enabled() {
    return true;
}

Then we could skip over any that return false, and you'd be able to change that whenever you like based on whatever reason. Alternatively (or additionally) we could provide some base classes that have methods for you to implement on configuration or when the model's ready. They'd be some really odd interfaces though. A wrapping class might be better.

what happens if another integration changes the model such that it would change whether you run the rest of the integration or not?

I was suggesting that all integrations run includeFor, and then whatever ones were selected, then run preprocessModel on all of those? (not includeFor then preprocess for each integration). Would that work?

There's not really a super clean and robust way of offering this generically.
It would be nice to inspect the model and/or the service shape, so we can check "Does this apply to S3?" or some other service. Our interface is a boolean.

then whatever ones were selected, then run preprocessModel on all of those

There is still the problem of preprocess changing the model such that a given integration is now valid or is now no longer valid. For example, imagine you have an integration that generates a protocol implementation for restXml. What happens when another integration removes that protocol from the service during preprocessModel? In the best case it would check before doing anything else and raise a meaningful error or otherwise disable itself. In this case there's no meaningful benefit to having includeFor. More likely it would unexpectedly fail when trying to access some property on the trait, raising an error that is likely hard for a user to understand. But it's also possible it just generates an implementation anyway, which puts the generated code in an invalid state.
So you can't provide the model before preprocess is called. You can provide the settings, which don't change and which do include the service id, but there's plenty of reason to need the final model (like requiring the presence of a protocol trait).
We could provide hooks at both points in time, one with just settings and one with full context. Ideally in some way where the post-preprocess method forbids use of preprocess. The question is how much foot-gunning do we want to accept.

I understand your concern. I don't really have a preference, but there is a valid need for integrations to only apply to a single service, as Ruby and Kotlin (and others?) have done. I still think it's reasonable to have includeFor run for all integrations and where true, only apply those to the service before any pre processing, even if pre processing would otherwise make it valid in that case.


Change mermaid output from svg (bug) to pdf

Continuation of https://github.com/pandoc/lua-filters/pull/266
svg output does not work for graphs so the mermaid filter has been changed to produce pdf output instead.

It's surprising that this would be necessary, I'll take a closer look.
The old code should be removed instead of commented out. We'll also have either install Inkscape for the Mermaid CI tests, or switch to LaTeX as the output format for those tests.

Sorry for old code. I just forgot to remove it.
For the svg problem... I thought it was related to https://github.com/mermaid-js/mermaid-cli/issues/112.
But I am not sure anymore...
I tested with this file :
test.md
Document is compiled to pdf with :
pandoc --lua-filter diagram_without_my_PR.lua test.md -t latex -o test.pdf
In the PDF result values 1 and 2 do not appear in the nodes.
But with :
pandoc --lua-filter diagram_with_my_PR.lua test.md -t latex -o test.pdf
Then, values 1 and 2 appear.
BUT... ( :) )
If I manually compile the mermaid graph to svg (mmdc --input graph.mmd --output graph.svg)
graph TD
      A["1"]
      B["2"]
      A --> B

The resulting graph.svg seems OK in firefox (1 and 2 printed inside nodes).
Then I do :
---
title: Sample file 2
---

![](graph.svg)

and compile this file with pandoc... and numbers have disappeared. So may be the problem is related to the way pandoc processes svg...

I checked and indeed, this seems to be a PDF-to-SVG  conversion issue. I'll modify the code and pass an indicator to the generator that indicates whether is should prefer PDF over SVG. That should fix it.

I didn't like the PDF output produced by mermaid, so the filter opts for PNG output when PDF/LaTeX is the targeted output format.

You didn't like ? It seemed to me that images were exactly the same, except that you can not zoom PNG indefinitely and you wll have to finely tune resolution to get a high quality printed document without generating 100Meg pngs. Just to be sure, can you explain what's wrong with PDF output ?

pdf-mermaid.pdf
png-mermaid.pdf

I see now that you included the -f flag in the PR. That should fix it.

Oh yes. Please, add the -f flag and keep pdf output that is better quality than png. I hope the issue with svg (mermaid or pandoc...) will be addressed soon and that it will be safe for your filter to fall back to svg everywhere. Thank you.

Already done :slightly_smiling_face:


Trait Namespace do not get Imported at the top

Traits do not get imported at the top while typing and hitting enter.
Even when, at the top you try to write the use SomeTrait it won't give you any hint or Trait Namespace.

fixed in v0.8.2


Add DMP editing and extra details to review_detail_sidebar

Addresses #312 and #325. Creates an UpdateView for the Data Management Plan, only accesible by the secretary. Additionally, this branch shows the applicant's name and supervisor in the review_detail_sidebar.

@miggol I updated the branch according to your excellent feedback. Sorry for all the sloppiness, I had some serious Friday-brain.


System detecting wrong end date time auctions

I have just deployed the ebay-snipe-server, and all the auctions I am watching are returning something like that:
{"ended":true,"endingAt":"1970-01-01T00:00:00.000Z",...}
So I cannot place a snipe b/c it says auction has ended.

@iwalucas I have just tried the server and I had no problem with the /auction/<id> endpoint. Can you tell me one of the IDs that is failing for you and any custom setting you may have in your application.conf?

@ruippeixotog here is one I just picked up:<PHONE_NUMBER>35
there are no custom settings

That's strange... I have just tried getting that auction and I got the right date:
{
    "ended": false,
    "endingAt": "2018-04-23T17:21:19.000+01:00",
    "location": "Riga, default, Latvia",
    "bidCount": 0,
    "shippingCost": "GBP 5.99",
    "currentBid": null,
    "id": "292496978835",
    "buyNowPrice": "GBP 7.49",
    "defaultCurrency": "GBP",
    "title": "SI TECH LP Scuba Hose BCD Inflator Dry Suit Standart Fitting Dive 80cm/31.5inch  | eBay",
    "seller": {
        "id": "to_deal",
        "feedback": 1411,
        "positivePercentage": 100
    },
    "thumbnailUrl": "https://i.ebayimg.com/images/i/292496978835-0-1/s-l1000.jpg"
}

It seems that the date is the only field with wrong information. Are you running this on Windows? What's the locale of your computer?

I am running on a Mac, with GMT-3 timezone

Ok, I'm running on Mac too. The fact that you mentioned "LetÃ´nia" while I had "Latvia" at the item location points to a difference in localization on the eBay webpage, which can cause this issue if the date is in another place or in another format. It also makes this hard for me to reproduce it... Can you tell me the output of locale in your shell?

sure, here it is:
LANG="en_US.UTF-8"
LC_COLLATE="en_US.UTF-8"
LC_CTYPE="en_US.UTF-8"
LC_MESSAGES="en_US.UTF-8"
LC_MONETARY="en_US.UTF-8"
LC_NUMERIC="en_US.UTF-8"
LC_TIME="en_US.UTF-8"
LC_ALL=
2018-03-30 7:56 GMT-03:00 Rui Gonçalves<EMAIL_ADDRESS>
Ok, I'm running on Mac too. The fact that you mentioned "LetÃ´nia" while I
had "Latvia" at the item location points to a difference in localization on
the eBay webpage, which can cause this issue if the date is in another
place or in another format. It also makes this hard for me to reproduce
it... Can you tell me the output of locale in your shell?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/ruippeixotog/ebay-snipe-server/issues/7#issuecomment-377501673,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ADfEf00adqlCIJEbu6sv-5FYR7XRLmWKks5tjg97gaJpZM4Syk8X
.

--
Lucas

I have the same error:
root@server:~# http localhost:3647/auction/202601112701
HTTP/1.1 200 OK
Content-Length: 439
Content-Type: application/json
Date: Mon, 18 Feb 2019 17:38:10 GMT
Server: akka-http/10.0.9

{
    "bidCount": 0,
    "buyNowPrice": null,
    "currentBid": "EUR 600.0",
    "defaultCurrency": "EUR",
    "ended": true,
    "endingAt": "1970-01-01T00:00:00.000Z",
    "id": "202601112701",
    "location": "Cassino, Италия",
    "seller": {
        "feedback": 44,
        "id": "ice_plug",
        "positivePercentage": 100.0
    },
    "shippingCost": "EUR 39.0",
    "thumbnailUrl": "https://i.ebayimg.com/images/i/202601112701-0-1/s-l1000.jpg",
    "title": "APPLE MACBOOK PRO 13\" RETINA DISPLAY 2015 RAM 8GB SSD 128GB   | eBay"
}

Hi,
I am encountering the same issue when requesting auctions from my local eBay (ebay.nl). The date is correctly returned when getting the same auction via ebay.com.
Like in the US, the ending date is mentioned also mentioned in the .vi-tm-left element on the Dutch version. But query("ending-at") keeps returning a None when logging the output in EbayClient.scala.
I don't think I'm at the point where date formatting matter, but I have changed it in my ebay-nl.conf to match the Dutch date formats:
auction-info {
  attributes {
    ending-at = { query: ".vi-tm-left, .lable:contains(\"Afgelopen:\") + .u-flL", date-formats: ["(dd MMM yyyy HH:mm:ss zzz)", "dd-MMM HH:mm"] }
  }
}

The locale of the environment the server is running in is set to nl_NL.UTF-8.
The other data is found and returned just fine. Any idea what could cause only .vi-tm-left to not be found?

I still have no idea what causes the issue, but I have learned two things:

My changes to auction-info.attributes in my ebay-nl.conf were not overriding those of common.conf. So for now I'm editing common.conf directly.
If I leave out the date-formats attribute in the configuration, query("ending-at") does find the element. But queryType("ending-at") then fails of course.

Hi @Terr! It sounds plausible that ebay.nl CSS queries might have flaws, I didn't test with any website other than ebay.com.
Do you mind sending me your current ebay-nl.conf and one or more auctions that are currently failing? Once this is fixed I can include the .nl config in the repo, if you want.

Hi, thanks for the reply!
I have pasted my ebay-nl.conf and common.conf in this Gist.
I had to modify common.conf because I failed to override the auction-info.attributes via ebay-nl.conf but maybe I did it wrong. Should it be possible to override those attributes in a site-specific config file?
The new date format pattern is correct I think, but because that's something that I needed to change I might have made a mistake there.
Here are two random auctions that I tried just now and have a few days remaining:

https://www.ebay.nl/itm/Lot-Of-10-Used-Micro-SD-Memory-Cards/233202703952?hash=item364bf6de50:g:lywAAOSwTmxcuSu~ (ID:<PHONE_NUMBER>52)
https://www.ebay.nl/itm/Aviator-folding-sunglasses-With-case/183773933879?hash=item2ac9c7b137:g:~XgAAOSw02Fciq3b (ID:<PHONE_NUMBER>79)

I did not translate all the text patterns in ebay-nl.conf yet, but the ones that I did do are enough to log in and retrieve auctions. After I've come across the other texts that need translation I would be happy to share it in a pull request :)

Hi @Terr! Sorry for taking this long to reply. I finally got the chance to look at your config, but the problem seems to be that the date pattern in auction-info.attributes.ending-at.date-formats does not accept dutch month names like "mei".
Unfortunately scala-scraper, the library that extracts content from HTML in the server, does not support localization currently. I just opened https://github.com/ruippeixotog/scala-scraper/issues/65, feel free to contribute if you find the time for it 🙂

Closing this issue because I'm not officially supporting ebay.nl (even though I'd like to have a config for it here) and I don't have any reproducible example of this failing in ebay.com.

Thanks for taking the time to investigate.
I'm not sure I can contribute code to scala-scraper but I've subscribed to the issue, in case I can help with testing it.
Should I get a complete and working config for ebay.nl together in the future I will be sure to create a merge request :)


[BUG] Downloading just doesn't work every time.

Describe the bug
Some mods (in my experience, ModMenu and LibZoomer) cannot be updated with this tool. I've successfully updated MouseWheelie so I know it's not completely busted.
To Reproduce
Steps to reproduce the behavior:

While running an outdated ModMenu (I am on modmenu-2.0.5.jar)
Running open ModMenu
Click on the update icon
Attempt to update ModMenu
See error

Expected behavior
The mod updates silently and the menu is closed.
** Please complete the following information:**

OS: Pop!_OS
Minecraft Version: 1.17.1
ModManager Version: 1.0.2+1.17-alpha

** Logs **
Error while executing task for modmenu
java.lang.NullPointerException: Cannot invoke "String.equals(Object)" because the return value of "xyz.deathsgun.modmanager.model.ReducedModMetadata.getName()" is null
	at xyz.deathsgun.modmanager.util.FabricMods.getJarFromModContainer(FabricMods.java:57) ~[modmanager-1.0.2+1.17-alpha.jar:?]
	at xyz.deathsgun.modmanager.tasks.ModUpdateTask.execute(ModUpdateTask.java:38) ~[modmanager-1.0.2+1.17-alpha.jar:?]
	at xyz.deathsgun.modmanager.api.manipulation.ManipulationTask.executeTask(ManipulationTask.java:47) [modmanager-1.0.2+1.17-alpha.jar:?]
	at xyz.deathsgun.modmanager.services.ManipulationService.run(ManipulationService.java:49) [modmanager-1.0.2+1.17-alpha.jar:?]

v1.1.0+1.17-alpha should fix this as it's a complete rewrite of ModManager. I will provide a test version tomorrow.

Here you go:
modmanager-1.1.0+1.17-alpha.jar.zip


GDM fixes

Motivation for this change
GDM is currently failing in multiple ways:

nixos-rebuild switch can break X11
fighting for tty1 with getty
nvidia permission issues

This merges all of the changes in #29576 and #25621 with some changes.

start GDM on tty7 so it doesn't fight with tty1 unnecessarily
disable the GDM wayland mode by default
fixes the nvidia permission issues

Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option build-use-sandbox in nix.conf on non-NixOS)
Built on platform(s)

[ ] NixOS
[ ] macOS
[ ] Linux

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Fits CONTRIBUTING.md.

we are still trying to fix the gnome3-gdm tests

@zimbatm you might want to be explicit about true and false because toString behaves as follows:
nix-repl> toString true
"1"

i.e.
[daemon]
-      WaylandEnable=${toString cfg.gdm.wayland}
+      WaylandEnable=${if cfg.gdm.wayland then "true" else "false"}
       ${optionalString cfg.gdm.autoLogin.enable (

Any idea why it seems to think I'm an owner here?

I don't know, when I created the issue, a bunch of people were automatically added as reviewers

Probably because the base branch was incorrect the first time, GitHub though you're proposing to submit every single commit from 17.09 in master.

anyone with a nvidia card wants to test this?

Should be fine. Thanks!


No man pages in wayshot-bin AUR Package

Steps to reproduce:
Install wayshot-bin 1.2.2-2 from the AUR
$ pacman -Ql wayshot-bin
wayshot-bin /usr/
wayshot-bin /usr/bin/
wayshot-bin /usr/bin/wayshot

Installing wayshot-git <IP_ADDRESS>.g538caac-1 doesn't have the same issue.
$ pacman -Ql wayshot-git
wayshot-git /usr/
wayshot-git /usr/bin/
wayshot-git /usr/bin/wayshot
wayshot-git /usr/share/
wayshot-git /usr/share/man/
wayshot-git /usr/share/man/man1/
wayshot-git /usr/share/man/man1/wayshot.1.gz
wayshot-git /usr/share/man/man7/
wayshot-git /usr/share/man/man7/wayshot.7.gz

https://github.com/waycrate/wayshot/commit/43fae61e239d75b8381f1e07e09e40dc2103cc8e


A service failed in the 'onPrepare'

Someone can help me please?
CONSOLE LOG
A service failed in the 'onPrepare' hook
Error: Unable to connect to selenium
    at hasStarted (/usr/local/lib/node_modules/wdio-selenium-standalone-service/node_modules/selenium-standalone/lib/check-started.js:17:10)
    at Timer.listOnTimeout (timers.js:92:15)

Continue...

ERROR: unknown error: Chrome failed to start: crashed
 (Driver info: chromedriver=2.33.506106, platform=Mac OS X 10.13.1 x86_64) (WARNING: The server did not provide any stacktrace information)
Command duration or timeout: 60.45 seconds
Build info: version: '3.6.0', revision: '6fbf3ec767', time: '2017-09-27T16:15:40.131Z'
System info: host: 'macmini3.local', ip: '192.168.xx.xx', os.name: 'Mac OS X', os.arch: 'x86_64', os.version: '10.13.1', java.version: '1.8.0_151'
Driver info: driver.version: unknown
chrome
    at new RuntimeError (/usr/local/lib/node_modules/webdriverio/build/lib/utils/ErrorHandler.js:144:12)
    at Request._callback (/usr/local/lib/node_modules/webdriverio/build/lib/utils/RequestHandler.js:313:39)
    at Request.self.callback (/usr/local/lib/node_modules/webdriverio/node_modules/request/request.js:186:22)
    at emitTwo (events.js:87:13)
    at Request.emit (events.js:172:7)
    at Request.<anonymous> (/usr/local/lib/node_modules/webdriverio/node_modules/request/request.js:1163:10)
    at emitOne (events.js:77:13)
    at Request.emit (events.js:169:7)
    at IncomingMessage.<anonymous> (/usr/local/lib/node_modules/webdriverio/node_modules/request/request.js:1085:12)
    at IncomingMessage.g (events.js:260:16)

CONFIG FILE
exports.config = {

    //
    // ==================
    // Specify Test Files
    // ==================
    // Define which test specs should run. The pattern is relative to the directory
    // from which `wdio` was called. Notice that, if you are calling `wdio` from an
    // NPM script (see https://docs.npmjs.com/cli/run-script) then the current working
    // directory is where your package.json resides, so `wdio` will be called from there.
    //
    specs: [
        './run/**/*.js'
    ],
    // Patterns to exclude.
    exclude: [
        // 'path/to/excluded/files'
    ],
    //
    // ============
    // Capabilities
    // ============
    // Define your capabilities here. WebdriverIO can run multiple capabilities at the same
    // time. Depending on the number of capabilities, WebdriverIO launches several test
    // sessions. Within your capabilities you can overwrite the spec and exclude options in
    // order to group specific specs to a specific capability.
    //
    // First, you can define how many instances should be started at the same time. Let's
    // say you have 3 different capabilities (Chrome, Firefox, and Safari) and you have
    // set maxInstances to 1; wdio will spawn 3 processes. Therefore, if you have 10 spec
    // files and you set maxInstances to 10, all spec files will get tested at the same time
    // and 30 processes will get spawned. The property handles how many capabilities
    // from the same test should run tests.
    //
    maxInstances: 10,
    //
    // If you have trouble getting all important capabilities together, check out the
    // Sauce Labs platform configurator - a great tool to configure your capabilities:
    // https://docs.saucelabs.com/reference/platforms-configurator
    //
    capabilities: [{
        // maxInstances can get overwritten per capability. So if you have an in-house Selenium
        // grid with only 5 firefox instance available you can make sure that not more than
        // 5 instance gets started at a time.
        maxInstances: 1,
        //
        browserName: 'chrome'
    }],
    //
    // ===================
    // Test Configurations
    // ===================
    // Define all options that are relevant for the WebdriverIO instance here
    //
    // By default WebdriverIO commands are executed in a synchronous way using
    // the wdio-sync package. If you still want to run your tests in an async way
    // e.g. using promises you can set the sync option to false.
    sync: true,
    //
    // Level of logging verbosity: silent | verbose | command | data | result | error
    logLevel: 'verbose',
    //
    // Enables colors for log output.
    coloredLogs: true,
    //
    // Saves a screenshot to a given path if a command fails.
    screenshotPath: './errorShots/',
    //
    // Set a base URL in order to shorten url command calls. If your url parameter starts
    // with "/", then the base url gets prepended.

    baseUrl: 'http://localhost:8181',

    //
    // Default timeout for all waitFor* commands.
    waitforTimeout: 10000,
    //
    // Default timeout in milliseconds for request
    // if Selenium Grid doesn't send response
    connectionRetryTimeout: 90000,
    //
    // Default request retries count
    connectionRetryCount: 3,
    //
    // Initialize the browser instance with a WebdriverIO plugin. The object should have the
    // plugin name as key and the desired plugin options as properties. Make sure you have
    // the plugin installed before running any tests. The following plugins are currently
    // available:
    // WebdriverCSS: https://github.com/webdriverio/webdrivercss
    // WebdriverRTC: https://github.com/webdriverio/webdriverrtc
    // Browserevent: https://github.com/webdriverio/browserevent
    // plugins: {
    //     webdrivercss: {
    //         screenshotRoot: 'my-shots',
    //         failedComparisonsRoot: 'diffs',
    //         misMatchTolerance: 0.05,
    //         screenWidth: [320,480,640,1024]
    //     },
    //     webdriverrtc: {},
    //     browserevent: {}
    // },
    //
    // Test runner services
    // Services take over a specific job you don't want to take care of. They enhance
    // your test setup with almost no effort. Unlike plugins, they don't add new
    // commands. Instead, they hook themselves up into the test process.
    services: ['selenium-standalone'],
    //
    // Framework you want to run your specs with.
    // The following are supported: Mocha, Jasmine, and Cucumber
    // see also: http://webdriver.io/guide/testrunner/frameworks.html
    //
    // Make sure you have the wdio adapter package for the specific framework installed
    // before running any tests.
    framework: 'mocha',
    //
    // Test reporter for stdout.
    // The only one supported by default is 'dot'
    // see also: http://webdriver.io/guide/testrunner/reporters.html
    // reporters: ['dot'],
    //
    // Options to be passed to Mocha.
    // See the full list at http://mochajs.org/
    mochaOpts: {
        ui: 'bdd',
        timeout: 99999999
    },
    //
    // =====
    // Hooks
    // =====
    // WebdriverIO provides several hooks you can use to interfere with the test process in order to enhance
    // it and to build services around it. You can either apply a single function or an array of
    // methods to it. If one of them returns with a promise, WebdriverIO will wait until that promise got
    // resolved to continue.
    //
    // Gets executed once before all workers get launched.
    // onPrepare: function (config, capabilities) {
    // },
    //
    // Gets executed before test execution begins. At this point you can access all global
    // variables, such as `browser`. It is the perfect place to define custom commands.
    // before: function (capabilities, specs) {
    // },
    //
    // Hook that gets executed before the suite starts
    // beforeSuite: function (suite) {
    // },
    //
    // Hook that gets executed _before_ a hook within the suite starts (e.g. runs before calling
    // beforeEach in Mocha)
    // beforeHook: function () {
    // },
    //
    // Hook that gets executed _after_ a hook within the suite starts (e.g. runs after calling
    // afterEach in Mocha)
    // afterHook: function () {
    // },
    //
    // Function to be executed before a test (in Mocha/Jasmine) or a step (in Cucumber) starts.
    // beforeTest: function (test) {
    // },
    //
    // Runs before a WebdriverIO command gets executed.
    // beforeCommand: function (commandName, args) {
    // },
    //
    // Runs after a WebdriverIO command gets executed
    // afterCommand: function (commandName, args, result, error) {
    // },
    //
    // Function to be executed after a test (in Mocha/Jasmine) or a step (in Cucumber) starts.
    // afterTest: function (test) {
    // },
    //
    // Hook that gets executed after the suite has ended
    // afterSuite: function (suite) {
    // },
    //
    // Gets executed after all tests are done. You still have access to all global variables from
    // the test.
    // after: function (result, capabilities, specs) {
    // },
    //
    // Gets executed after all workers got shut down and the process is about to exit. It is not
    // possible to defer the end of the process using a promise.
    // onComplete: function(exitCode) {
    // }
}

Chrome failed to start: crashed

This seems to be an compatibility problem and nothing related to this service. Please join our   for support like this.


Reference timeline events broken

Idk what is happening. I swear if GitHub broke the GraphQL endpoint again I’m gonna lose it...

Bug Report Dump (Auto-generated)

Version 1.12.0 (1183)
Device: iPhone X (iOS 11.1.1)
TestFlight: false

Working now


Spring-boot2 collection needs to be update to be same as in 0.3.3 release

The spring-boot2 stack in 0.4.0 needs to be updated to match the level that has been recently updated in 0.3.3, ie to use RHR base images.
The latest appsody build process may not be able to be included in the 0.4.0 release, so any potential stack chnages to use templating etc will not be able to be used within the stack. We therefore need the spring-boot2 stack setup the same as in the 0.3.3 release. We can then modify the stack once the build process has been modified.

@groeges did you assign the right Andrew?


213 conf persistence

Description

Configuration persistence. If Connector is restarted for any reason, it will reload the currencies configuration from a backup file.
Fixes #213  (issue)
Dependencies (if any)

Type of change

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

How Has This Been Tested?

Nodechain running for eth in testnet network
python3 nodechain.py -t eth -n testnet -p 80 -sp 443 -b /path/to/blockchains/testnet start

Getting default configuration
curl --location --request POST 'http://localhost:80/admin/getcoin' \
--header 'x-api-key: API_KEY_VALUE' \
--header 'Content-Type: application/json' \
--data-raw '{
    "coin": "eth",
    "network": "testnet"
}'

{
    "success": true,
    "message": "Configuration for testnet network for currency eth retrieved successfully",
    "coin": "eth",
    "network": "testnet",
    "config": {
       "rpcEndpoint": "http://ethereumgo-testnet:8545",
       "wsEndpoint": "ws://ethereumgo-testnet:8546",
       "indexerEndpoint": "http://postgrest-testnet:3000"
    }
}

Config for eth in testnet network updated successfully
curl --location --request POST 'http://localhost:80/admin/updatecoin' \
--header 'x-api-key: API_KEY_VALUE' \
--header 'Content-Type: application/json' \
--data-raw '{
    "coin": "eth",
    "network": "testnet",
    "config": {
        "rpcEndpoint": "http://another-ethereumgo-testnet:8545",
        "wsEndpoint": "ws://another-ethereumgo-testnet:8546",
        "indexerEndpoint": "http://another-postgrest-testnet:3000"
    }
}'

 {
    "success": true,
    "message": "Configuration for testnet network for currency eth updated successfully"
}

Connector is restarted
docker restart connector_connector_1

Getting config for eth in testnet network
curl --location --request POST 'http://localhost:80/admin/getcoin' \
--header 'x-api-key: API_KEY_VALUE' \
--header 'Content-Type: application/json' \
--data-raw '{
    "coin": "eth",
    "network": "testnet"
}'

{
    "success": true,
    "message": "Configuration for testnet network for currency eth retrieved successfully",
    "coin": "eth",
    "network": "testnet",
    "config": {
        "rpcEndpoint": "http://another-ethereumgo-testnet:8545",
        "wsEndpoint": "ws://another-ethereumgo-testnet:8546",
        "indexerEndpoint": "http://another-postgrest-testnet:3000"
    }
}

Test Configuration:

Operating system (output of sw_vers): macOS 12.5
Kernel version (output of uname -sr): Darwin 21.6.0
Architecture (output of uname -m): x86_6

Good practices to consider

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[ ] Any dependent changes have been merged and published in downstream modules

Functions and files shouldn't be called 'backup' because Nodechain is not backing up anything. Instead, it is persisting the configuration. So, we should replace 'backup' word in the code with 'persistent configuration'. The file should be named something like 'config.json'.
Docker-composes should be configured with the configuration file inside a volume.


add composer_packages + fixes + upgrade default versions

Add composer_packages variable to install some composer global packages (those packages will be automatically autoloaded in php.ini using auto_prepend_file).
By default, the symfony/var-dumper package is installed, in order to be able to use dump() function everywhere (except php -r "dump(...);") in the provisioned machine.
Fix php7.1 "main" installation
PHP default version is now 7.0
NodeJS default version is now 6.x
Documentation for some other variables

to v1.4.0


Inline label editing click and save functionality

Expected Behaviour
When editing an inline form label clicking a point in the string should move the cursor to that point. Hitting enter should save the authored changes.
Actual Behaviour
When editing an inline form label clicking a point in the string ends editing. Hitting enter ends editing and removes the changes.
Steps to Reproduce

Create a text box field

Editing issue

Select the field and click the edit button
Click anywhere in the label to try to move the cursor
Editing ends

Saving issue

Select the field and click the edit button
Alter the text of the label and hit enter
Editing ends and changes are reverted

AEM Version (mention the exact version in case of cloud SDK)
2023.12.14697.20231215T125030Z-231200
AEM Forms Version
2<IP_ADDRESS>-231200

@mnahmias Yes, we're aware of this issue, but it hasn't been prioritized yet as there are alternative solutions available to accomplish the same use-case


Extension Manager requires double-click to open preview

Azure Data Studio Version: 1.18.0

Steps to Reproduce:

Open Extension Manager and select an extension

Expected: Preview page is shown in document area
Actual: Now a double-click on the extension is required to open preview.  This is a regression from April and doesn't match current vscode behavior.

Note that this issue is also impacting the Git change navigation experience.  Both issues are fixed by #10473.

Fix has been merged to release/1.18


Fix: Move test into appropriate namespace

This PR

[x] moves a test into the appropriate namespace

Thank you, @chartjes!


Allow HostSNI values to match on a wildcard *. subdomain

What does this PR do?
Allow HostSNI values to match on a wildcard *. subdomain
Motivation
This was needed to allow any subdomains of a given domain to
be passed through to a single service.
More

[x] Added/updated tests
[ ] Added/updated documentation

Additional Notes

For some reason, after following the docs, I couldn't get the tests to pass on my dev machine; I am not sure why but the Docker tests would fail, so my added test code hasn't been checked locally. If I have understood the test framework correctly then I expect they should pass though, and I have manually run this build and checked the feature works as expected/intended.

Hello @richbayliss,
Thanks for your interest in Traefik and your pull request!
The implementation you have made does not conform to what we have implemented for the HeadersRegexp and HostRegexp rules. To be consistent a new rule should be implemented which supports more complex use cases.
As explained in https://github.com/traefik/traefik/pull/6003#issuecomment-640494203, we think that the mux syntax for regex is not perfect. For a new regex matcher we think it's not a good idea to use the mux syntax, but if we introduce a new syntax, we need to handle it in both TCP and HTTP regex matches.
That's why we think it's better to talk about this before in #7266.
Therefore, we will decline your pull request accordingly.


Manual regression tests: Fix plot labels

Complete this sentence
THIS PULL REQUEST IS READY TO MERGE
Feature or improvement description
The baseline and local results were mislabeled on the plots generated by manualRegressionTest.py.
I also updated the deprecated syntax of the legend text.
Related issue, if one exists
none.
Impacted areas of the software
Regression test plots

Thanks for catching and fixing that, @bjonkman.
@rhammond2 - you might want to verify this bug is fixed in your upcoming work on the regression test framework.


[BUG] TableExt 30106 Shpfy Sales Shipment Header - Field 30102 "Shpfy Order No."

Hi,
Can you please check if this field ID is an error or not?
`tableextension 30106 "Shpfy Sales Shipment Header" extends "Sales Shipment Header"
{
fields
{
field(30100; "Shpfy Order Id"; BigInteger)
{
Caption = 'Shopify Order Id';
DataClassification = CustomerContent;
Editable = false;
}
    field(**30102**; "Shpfy Order No."; Code[50])
    {
        Caption = 'Shopify Order No.';
        DataClassification = CustomerContent;
        Editable = false;
    }
    field(30103; "Shpfy Fulfillment Id"; BigInteger)
    {
        Caption = 'Shopify Fulfillment Id';
        DataClassification = CustomerContent;
        Editable = false;
    }
}

}`
Into the table extension Sales Header and Sales Invoice Header, the field ID is 30101.
When posting the Shipment, the Shpfy Order No. don't follow with the transferfield.
To solve this issue temporary, I just add an event subscribber into Codeunit80.

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.
Build ID: 10459.

Availability update: We will publish a fix for this issue in the next update for release 22.
Build ID to track: 55336.

We will fix this for now by adding a subscriber to post shipment (similar to the current mitigation). hopefully we will be able to change field ids at a later state so we can correct this on a non breaking way


Use function callback on titleTag string replacement to avoid escaping dollar signs

Addresses this issue https://github.com/nfl/react-helmet/issues/164

 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.