android build crashes with flutter 3.24.0

With flutter 3.24.0 "flutter build apk" crashes with every flutter application using simple_barcode_scanner, even with newly created projects (flutter create ), as soon as simple_barcode_scanner is in pubspec.yaml.
Error message:

FAILURE: Build failed with an exception.

What went wrong:
Execution failed for task ':flutter_barcode_scanner:verifyReleaseResources'.

A failure occurred while executing com.android.build.gradle.tasks.VerifyLibraryResourcesTask$Action
Android resource linking failed
ERROR:<project_path>\build\flutter_barcode_scanner\intermediates\merged_res\release\values\values.xml:6135: AAPT: error: resource android:attr/lStar not found

Environment:

flutter doctor -v
[√] Flutter (Channel stable, 3.24.0, on Microsoft Windows [Version 10.0.22631.3880], locale de-DE)
• Flutter version 3.24.0 on channel stable at C:\prog\flutter
• Upstream repository https://github.com/flutter/flutter.git
• Framework revision 80c2e84975 (10 days ago), 2024-07-30 23:06:49 +0700
• Engine revision b8800d88be
• Dart version 3.5.0
• DevTools version 2.37.2
[√] Windows Version (Installed version of Windows is version 10 or higher)
[√] Android toolchain - develop for Android devices (Android SDK version 30.0.3)
• Android SDK at C:\Users\Admin\AppData\Local\Android\Sdk
• Platform android-34, build-tools 30.0.3
• ANDROID_SDK_ROOT = C:\Users\Admin\AppData\Local\Android\Sdk
• Java binary at: C:\prog\Android\Android Studio\jbr\bin\java
• Java version OpenJDK Runtime Environment (build 17.0.10+0--11609105)
• All Android licenses accepted.
[√] Chrome - develop for the web
• CHROME_EXECUTABLE = C:\prog\PortableApps\GoogleChromePortable\GoogleChromePortable.exe
[√] Visual Studio - develop Windows apps (Visual Studio Community 2022 17.8.6)
• Visual Studio at C:\Program Files\Microsoft Visual Studio\2022\Community
• Visual Studio Community 2022 version 17.8.34525.116
• Windows 10 SDK version 10.0.19041.0
[√] Android Studio (version 2024.1)
• Android Studio at C:\prog\Android\Android Studio
• Flutter plugin can be installed from:
https://plugins.jetbrains.com/plugin/9212-flutter
• Dart plugin can be installed from:
https://plugins.jetbrains.com/plugin/6351-dart
• Java version OpenJDK Runtime Environment (build 17.0.10+0--11609105)
[√] IntelliJ IDEA Community Edition (version 2021.3)
• IntelliJ at C:\prog\JetBrains\IntelliJ IDEA Community Edition 2020.3.3
• Flutter plugin can be installed from:
https://plugins.jetbrains.com/plugin/9212-flutter
• Dart plugin can be installed from:
https://plugins.jetbrains.com/plugin/6351-dart
[√] Connected device (3 available)
• Windows (desktop) • windows • windows-x64    • Microsoft Windows [Version 10.0.22631.3880]
• Chrome (web)      • chrome  • web-javascript • Google Chrome 127.0.6533.89
• Edge (web)        • edge    • web-javascript • Microsoft Edge 127.0.2651.86
[√] Network resources
• All expected network resources are available.
• No issues found!

Probably connected with https://github.com/flutter/flutter/issues/153281 - the problem in 3.24.0 seems to affect multiple plugins. It is recommended there to upgrade compileSdkVersion to 34, as this has resolved the issue in https://github.com/pr0gramista/charset_converter/issues/40#issuecomment-2282932050.

this plugin is a wrapper of flutter_barcode_scanner, which was last updated a 3 years ago :cry:  this works for my build :pray:

Yea, Issue caused by flutter_barcode_scanner, planning to upgrade soon. closing the the issue now feel free to reopen if issue cause from simple_barcode_scanner


allow empty files and ensure that we fail on invalid files

Signed-off-by: John Cornish<EMAIL_ADDRESS>Resolves issue #75.
Tested the new workflow with a built binary:
→ credhub interpolate --file minimal-yaml.yml
null

→ echo $?
0

→ credhub interpolate --file invalid-yaml.yml
yaml: invalid map key: ...

→ echo $?
1

→ credhub interpolate --file empty-yaml.yml

→ echo $?
0

Release https://github.com/cloudfoundry-incubator/credhub-cli/releases/tag/2.5.0 has a fix for this. Sorry again about the miscommunication.


add link to navbar

Issue
https://github.com/subeshb1/developer-community-stats/issues/87
Description
I added a link to the Github icon in the navbar that takes the user to the developer-community-stats repository.
Checklist

[ ] No merge conflicts

Great @ellu-laitinen


Add support for Monolog 3

Thanks @bshaffer! Any idea when a new release will be published?

This is now released in v2.12.5


[FEATURE] list of activities as switches

Not all activities are Tv related, I have a “Bluetooth Speaker” activity, which doesn’t fit into the Tv accessory
A list of activities to be skipped by the Tv accessory and published as switches would be a nice add on

Hi,
You can unselect the input in properties of tv accessory to disable your Bluetooth input .
You can also add a second sub platform (1.0.0) or a second platorm (0.6.X) that behaves a switch .
I will think of an option to hide some activities on switch mode however :)


If "TEST" or "ATTEMPT" button is pressed, its tooltip does not disappear

The tooltip from TEST button is still present while the cursor is hovering over ATTEMPT button:

Fixed

I've just noticed that the same thing happens with the DISCUSS and VIEW SOLUTIONS buttons too.
The remaining RESET, SKIP, UNLOCK SOLUTIONS work fine.

Doesn't those go to discussion page and solution pages?

Yes, but you can middle-click those and the tooltip will remain, which is probably not desired.

This is because the element has :focus state. Most of the new tooltip libraries seems to work this way for accessibility.
The problem is that the tooltip should be shown when focused using a keyboard, but not after middle clicking.

The remaining RESET, SKIP, UNLOCK SOLUTIONS work fine.

These can't be middle clicked because there's no href. I can change DISCUSS/VIEW SOLUTIONS to be the same, but that will prevent opening the discussion page in new tab.
I've been gradually updating the frontend codebase to be more maintainable, so I'll keep this in mind as a low priority enhancement.


Create Collection takes 30-45 seconds & sometimes times out

Describe the bug
This issue is reproducible on both https://sandbox.dspace.org (running pre-8.0) and https://demo.dspace.org (running 7.x). When attempting to create a new Collection, the request always takes a long time (at least 30-45 seconds) and sometimes times out with a 504 Gateway Timeout error.
This issue only occurs for creation of Collections.  Creating Communities or Items seems uneffected.
To Reproduce
Steps to reproduce the behavior:

Login as an Admin to either Sandbox or Demo
In your browser, open up your DevTools -> Network tab (useful to watch the create step)
In the Admin menu, choose "New -> Collection".  Select any Community as the parent
Enter a title and click "Save".  The page will appear to not respond.  Eventually either a 504 or 200 will return, but only after about 30-45 seconds.

In your DevTools -> Network tab you'll see that a POST request was made to /server/api/core/collections?parent=[community-uuid].  It will remain "Pending" for the entire 30-45 seconds.
On Sandbox, the request will sometimes return as successful (200 OK), but only after about 30-45 seconds.
On Demo, the request almost always times out (504 error returned).  However, the Collection will have been created successfully behind the scenes. So, if you search for the collection after seeing a 504 error, you will find that it exists.
In the backend dspace.log file, the POST request appears to succeed almost immediately.  It's unclear why it stays "Pending" for so long, and whether the bug is in the frontend or backend.

Expected behavior
Creating a Collection obviously should be as fast as creating a Community.
Related work
It's unclear what caused this bug, but the bug exists in both dspace-7_x and main.  So, that implies it was caused by a bug fix which was applied to both branches.

Hi @tdonohue, reading the description of the issue, I think it could be related to a consumer configured to reload the submission configurations when a collection is edited/created. See:
https://github.com/DSpace/DSpace/blob/main/dspace/config/dspace.cfg#L833-L835
I think that this consumer could be refactored to avoid reload the configurations multiple times during the creation of a collection, moving the reload process to the end() function.
I am not able to reproduce it locally, but I suppose that it happens because in demo and sandbox reloading the submission configurations takes more time.
I will send a PR with the proposed changes to see if it could solve or mitigate the issue.

@toniprieto : That would be wonderful!  I had actually just realized myself that the likely cause is the SubmissionConfigConsumer, but I had not determined how to fix it.  If you could create a quick PR to help mitigate the issues, that would be very much appreciated!  I'd ensure it gets reviewed/tested quickly.

NOTE: After applying the fix in #9462, the "Create Collection" step is much faster... though it's still slightly slower than I'd like to see.  It now takes ~6 seconds instead of ~30-45 seconds.
We may need to consider ways to speed this up further...or possibly run the submission config "reload" behind the scenes (instead of waiting on it to complete before returning).  But, for now, the significant performance issue is fixed.

There is an issue https://github.com/DSpace/DSpace/issues/9402 that is addressed with: https://github.com/DSpace/DSpace/pull/9415 . This PR fixes an issue that limit you to a maximum of 10 collections per entity type (default solr rows limit). But that one isn't in DS source yet.
By choosing Solr as a solution for enabling you to set a form based on the defined entity types at collections,
we intended to speed results. Even with https://github.com/DSpace/DSpace/pull/9415 30-45 seconds (considering we are using solr) it seems too much time. Perhaps you have a big repository (with a large number of collections) with a large number of different entities types, that could explain.
Also, does the repository changes the configuration?
event.consumer.submissionconfig.filters = Collection+Modify_Metadata

Currently, with our local migrated repositories we didn't verified this issue. With our local repositories it takes about 15 sec to create a new collection and 5 sec to effectively edit it. Perhaps the issue could also be with policy creation.

A better implementation to the one I did with forms loading, with some associated effort, would be to also consider an alternative way how submission forms are made available.  Currently, it's based on the configured collections, specifically on the collection's Handle, but, perhaps it makes sense to do a runtime validation based on the entity type for the provided collection.

@paulo-graca : I can verify the behavior here was on sandbox.dspace.org and demo.dspace.org.  Those sites both use default settings in dspace.cfg with regards to the event.consumer.submissionconfig.filters.
I've not yet narrowed down why this submissionconfig consumer is adding so much time to the Collection creation process, but it is down to ~6 seconds now.  That's a big improvement over 30+ seconds, but I'd still like to see if we can make it faster (as Communities are created much more quickly, for example).
I'll take a closer look at https://github.com/DSpace/DSpace/pull/9415 as well to see if there are improvements there...but at a glance, I'm not sure whether that will impact performance.  It does seem like an important bug fix though which we should get into 8.0 and 7.x

@paulo-graca I think it's a good idea to implement what you suggested in your last comment. In version 8.x, a change that allows configuring forms at the community level has been included (DSpace/DSpace#9259). With this change, the function that returns the Item Submission process used by a collection receives a collection as a parameter (previously it received a handle). See:
https://github.com/DSpace/DSpace/blob/main/dspace-api/src/main/java/org/dspace/app/util/SubmissionConfigReader.java#L228
Having the collection makes it simpler to retrieve the entity type and read a map with the relation between entityTypes and item submission process that could be built during the initial load. This way, there would be no need to perform a submission config reload when a collection is edited/created and the issue DSpace/DSpace#9402 should also be resolved.

@paulo-graca @tdonohue I've had time these days to implement the approach described in the last comment, and I've sent a PR: DSpace/DSpace#9478 The implementation is very similar to DSpace/DSpace#9259
Could you take a look to see if this new approach is appropriate?


Vite is Underestimated

I think Vite is going to prove to be the biggest idea for web development in 2020.
It isn't about speed and feeds either. I've felt for a long time that the one thing that really SUCKED about JS was the bundlers and build tooling.
Its importance has been overlooked also...  Vue et al mostly solved the reactivity issues years ago. But meanwhile, we've been dealing with 30s startup times for Webpack and 5 minute builds every time we want to deploy! Not to mention the multi-day debugging and impossible to understand errors.
Vite solves that elegantly. Finally.
I know you get this many places, but Evan much respect for your ability to innovate man!

Thanks!

Closing to keep issues clean.


No devics are displayed in the map if they are home

Home Assistant release (hass --version):
0.14.2
Python release (python3 --version):
Python 3.4.3
Component/platform:
http - mapview
Description of problem:
I have thought that the device map did not work because I just looked at it when all devices was home.
Now when I tried to resolve the issue I looked at the code and saw that it's not displaying the devices if they have the state home. I suggest that we remove that condition so that others don't think it's broken as I did just because the devices are at the home zone.

This is by design.
We currently do not group multiple people/icons in one place. As it would soon become crowded and difficult to distinguish where home was, it was chosen to hide people that are home.

Could this be represented with with either a number of tracked devices or something to indicate that the device/devices hasn't disappeared?
I ran in to the same problem as @stefan-jonasson now when I'm redoing my setup from scratch.


v1.2.16 update README

Coverage remained the same at 100.0% when pulling 618fef2b62a3a1cb905106b6ea34587e259b80a7 on dev into ea81075295d34198eed9d16f3339b351861d5231 on master.

Coverage remained the same at 100.0% when pulling 618fef2b62a3a1cb905106b6ea34587e259b80a7 on dev into ea81075295d34198eed9d16f3339b351861d5231 on master.

Coverage remained the same at 100.0% when pulling 618fef2b62a3a1cb905106b6ea34587e259b80a7 on dev into ea81075295d34198eed9d16f3339b351861d5231 on master.

Coverage remained the same at 100.0% when pulling 618fef2b62a3a1cb905106b6ea34587e259b80a7 on dev into ea81075295d34198eed9d16f3339b351861d5231 on master.


[REQ] When opening a link exit the app when navigating back

Describe the problem that needs solving
A clear description of what the problem is. E.g. I'm always frustrated when [...]
When opening a Twitter link externally, it opens the app and shows the content, but instead of exiting it goes to the main feed.
Describe the solution you'd like
A clear description of what you want to happen.
Don't go to the main feed and exit.

Agreed. It is annoying to have to use the back button twice to exit, especially when opening the link from a website or app like Reddit containing multiple Twitter links in the same post.
If some users prefer going back to the feed with the first back button, it may be better to have an option to toggle that in app settings.

This has now been fixed in aa54d3a19e27b010593b4798354382340f5a67ca, and will be going out in the next release.


Simplify incrementFileName

Fixes #55128

This is some great work. I love it, especailly the tests!
I was thinking if the differentiation between file and folder behavior is really necessery. But it seems that all native explorer have different behavior, so let's keep it like that. The corner case being I have a folder called "hello.t" -> "hello.t copy" instead of "hello copy.t". So let's keep the differentiation I think.
If you want you can address my two minor comments. I would merge this either way.
Thanks a lot!

@isidorn updated the PR, now I'm using extname and basename

Great work, merging in.
Thanks a lot!


Show logs in futon

Uses the couch_log read/1  and if a "last read" is provided, it automatically returns log entries after last_read log entry.

@sivsushruth Thanks for the PR! Though could you rebase your PR again latest master? The current one use an old version which breaks the CI :)
About the patch by itself, my only concern right now is to handle the case where you rotate the logs. This should be detected. This may be done by hooking to the events generated by lager. Also maybe the plugin can use websockets? It should be easy using lager. To read old logs people could use the trick you're using.
See for example:
https://github.com/ivanos/lager_websocket
This one is working with cowboy but can probably be adapted to the websockets system in mochiweb. If needed we can bump the mochiweb version we use in barrel.
Thoughts?

I am checking the links you provided. The polling method is usually a
fallback. For realtime logs, we could definitely work websockets.
So one way to implement, as you said, would be to hook to the function
generating logs and then creating a wrapper around it to stream logs onto
the client.
Let me know what you think.
Can this PR be left as a work in progress ?
On Sat, Mar 19, 2016 at 2:52 PM, Benoit Chesneau<EMAIL_ADDRESS>wrote:

@sivsushruth https://github.com/sivsushruth Thanks for the PR! Though
could you rebase your PR again latest master? The current one use an old
version which breaks the CI :)
About the patch by itself, my only concern right now is to handle the case
where you rotate the logs. This should be detected. This may be done by
hooking to the events generated by lager. Also maybe the plugin can use
websockets? It should be easy using lager. To read old logs people could
use the trick you're using.
See for example:
https://github.com/ivanos/lager_websocket
This one is working with cowboy but can probably be adapted to the websockets
system
https://github.com/mochi/mochiweb/blob/master/src/mochiweb_websocket.erl
in mochiweb. If needed we can bump the mochiweb version we use in barrel.
Thoughts?
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
https://github.com/barrel-db/barrel-platform/pull/29#issuecomment-198672489

@sivsushruth i wasn't planning to close the PR, this is definitely a work in progress :)
The way lagger_websocket is working, is registering to the event handler of lager. It's "just" another sink you configure like the file handler.

Will reopen with PR along with new branch


I never get from server field __createdAt

The field __createdAt is not contained in the responses I get from the server while synchronizing a table. Does this happen on purpose?

For mobile apps __createdAt has been renamed to simply createdAt.  It's possible that there is stripping behavior going on, is this at the sdk level or in the http responses?  What backend are you using, node or .net?

It is in the http responses. I am using .net backend.

Hi @icsd, I took a look & this is the consistent behavior across platforms.  If you would like to access createdAt/updatedAt/deleted columns on the client you can add those to the table definition and they should be populated.


Using latest DNS SD entry if there are duplicates

The DNSServiceResolve function in Darwin seems to cache DNS SD entries. It returns them in sequential order and the SDK holds a map of the entries keyed off the interface ID. This change updates it so that we always use the latest entry returned by the callback.

You're likely correct about the os-wide cache, it looks like killing the app doesn't always resolve the issue (neither does uninstalling it then re-installing it). So there's some TTL there that it probably hits at some point when it will remove it.
The way i'm validating the most current advertisement is that i'm using a separate application that listens for DNS broadcasts and i can see the currently broadcasted TXN record.

I'm closing this PR, looks like our advertiser was not setting the cache-flush bit properly. iOS seems to respect this bit OK afterward and no longer sends multiple entries of the DNS-SD broadcasts and only sends the latest one. There's still a change i need to make for the sample app to update it correctly but i'll push out a separate PR for that.

Thanks for fixing that. Can you say what software that was?

Thanks for fixing that. Can you say what software that was?

The software producing the advertisement was a device running the v1.0.0 version of the Matter SDK. We'll have to try to get it more updated.


[WIP] Add npipe support

Revives https://github.com/portainer/portainer/pull/459/files
Fixes: https://github.com/portainer/portainer/issues/1179

I hadn't noticed @StefanScherer had also started this work here: https://github.com/portainer/portainer/compare/develop...StefanScherer:npipe_support
The current code is not working. It seems like the named-pipe dial doesn't work correctly. I've checked that the right named pipe url is fed in.
Debugging ideas appreciated.

Ah, using @StefanScherer's changes to use winio makes it work for me too.

Ok, great ! Please have a look at my comments, there are a few things I'd like to change.

@deviantony @StefanScherer ready for review

I compiled the PR and tested it on Windows.
For my tests I don't use a mapped volume, so I have to enter the admin password. After logging in for the first time I see this dialog to select where to connect.

Then I select local and see the new message with the -v option for Linux and Windows.

Then I sometimes saw error messages that the local connection couldn't be established, so I pressed again.
In this case only the dashboard shows an error:

It seems that the dashboard directly communicates with the local connection?
When I open the endpoints I see the primary (from the -H option) and the connection I had to select before seeing the dashboard.

So maybe the named pipe is not available directly after starting the container and portainer.exe is accessing it too early? I'll try more in a different Azure VM.

@StefanScherer

I compiled the PR and tested it on Windows.
For my tests I don't use a mapped volume, so I have to enter the admin password. After logging in for the first time I see this dialog to select where to connect.

This is strange, if you used the -H field on the CLI, you should not see the endpoint init. view. I'll have a look at this.
We also need to ensure that the npipe error cannot be reproduced.

rebased, @StefanScherer feel free to give a whirl.
Still need to do the API-related changes

Thanks @friism I recompiled commit f0f4815. I also created a fresh Azure VM with 16278 running in a nested Hyper-V VM and Docker 17.09.0-ce-rc1 installed. Then I built the portainer image with this little Dockerfile
FROM microsoft/nanoserver-insider
COPY dist /
COPY dist/portainer-windows-amd64 /portainer.exe
ADD https://master.dockerproject.org/windows/x86_64/docker.exe /windows/system32/docker.exe
VOLUME C:\\data
EXPOSE 9000
ENTRYPOINT [ "/portainer.exe" ]

So I have the Docker CLI as well inside to check if named pipe works (docker exec containerid docker version).
I still see some strange effects. I tried a screen capture, but of course in the first recording everything worked fine :-) But here is another one:

But here is a working demo:

Haha, we have containers, so I can run portainer more than once. I had reproduced the error in one container, then started another portainer instance with mapped port 9001, so I could look inside through the working instance.
In the errornous container I could run the docker version command, so the named pipe is also accessible.
I also tried to open a shell through the working copy of portainer, but then got an error in the first one
2017/09/11 09:42:31 Starting Portainer 1.14.0 on :9000
2017/09/11 09:43:15 http error: User not found (code=404)
2017/09/11 09:44:13 error during hijack: dial npipe: unknown network npipe

So there is some work to do for the terminal execution. I tried of course only a cmd.exe as powershell.exe is not available in microsoft/nanoserver-insider base image.

Thanks for the details @StefanScherer !
About the terminal execution, I think that your error is raised by https://github.com/portainer/portainer/blob/develop/api/http/handler/websocket.go#L125
This file should probably be adapted to use Winio as well.
The inconsistent communication error with the named pipe are strange tho.

Thanks @deviantony that did it. I had to change the websocket.go as follows, ping @friism
diff --git a/api/http/handler/websocket.go b/api/http/handler/websocket.go
index dbc4fd9..ecd16d2 100644
--- a/api/http/handler/websocket.go
+++ b/api/http/handler/websocket.go
@@ -16,6 +16,7 @@ import (
        "time"

        "github.com/gorilla/mux"
+       "github.com/Microsoft/go-winio"
        "github.com/portainer/portainer"
        "github.com/portainer/portainer/crypto"
        "golang.org/x/net/websocket"
@@ -65,7 +66,7 @@ func (handler *WebSocketHandler) webSocketDockerExec(ws *websocket.Conn) {
        var host string
        if endpointURL.Scheme == "tcp" {
                host = endpointURL.Host
-       } else if endpointURL.Scheme == "unix" {
+       } else if endpointURL.Scheme == "unix" || endpointURL.Scheme == "npipe" {
                host = endpointURL.Path
        }

@@ -122,7 +123,11 @@ func hijack(addr, scheme, method, path string, tlsConfig *tls.Config, setRawTerm
        )

        if tlsConfig == nil {
-               dial, dialErr = net.Dial(scheme, addr)
+               if scheme == "npipe" {
+                       dial, dialErr = winio.DialPipe(addr, nil)
+               } else {
+                       dial, dialErr = net.Dial(scheme, addr)
+               }
        } else {
                dial, dialErr = tls.Dial(scheme, addr, tlsConfig)
        }

I've pushed the hijack fix to https://github.com/portainer/portainer/commit/9d5fe59eadf3733ebef6c7ac54b96012c7164170

@friism
Conflicts have been introduced with the recent changes, mind doing a rebase ?

I can take a look tmrw Friday

If we need better UX whether mapping named pipe is supported, this version check in upstream repo might help:
	if len(mps) > 0 && system.GetOSVersion().Build < 16210 { // replace with Win10 RS3 build number at RTM
		return errors.New("named pipe mounts are not supported on this version of Windows")

@deviantony @StefanScherer ready for review.
@StefanScherer if you want, feel free to add the more detailed versioncheck as follow-up

@friism I'm testing now. Also looking into the problem with the red popups that the named pipe couldn't be opened.

I often can reproduce the problem opening the named pipe. I found out that the frontend makes two endpoint requests in parallel: /api/endpoints/1/docker/info and /api/endpoints/1/docker/version, one of them often fails in my environment:

Seems like the createFile() in https://github.com/Microsoft/go-winio/blob/78439966b38d69bf38227fbf57ac8a6fee70f69a/pipe.go#L146 returns "open //./pipe/docker_engine: The system cannot find the file specified."
I've added a simple retry around winio.DialPipe()
func newNamedPipeTransport(namedPipePath string) *http.Transport {
	return &http.Transport{
		Dial: func(proto, addr string) (conn net.Conn, err error) {
			fmt.Println("dialing named pipe", namedPipePath)
			for {
				conn, err = winio.DialPipe(namedPipePath, nil)
				if err != nil {
					fmt.Println("error dialing pipe", err)
					time.Sleep(10 * time.Millisecond)
				} else {
					fmt.Println("pipe dialed")
					break
				}
			}
			return conn, err
		},
	}
}

and after opening the Web UI the console output of portainer shows sometimes it does a retry:
dialing named pipe //./pipe/docker_engine
dialing named pipe //./pipe/docker_engine
error dialing pipe open //./pipe/docker_engine: The system cannot find the file specified.
pipe dialed
dialing named pipe //./pipe/docker_engine
pipe dialed
dialing named pipe //./pipe/docker_engine
pipe dialed
dialing named pipe //./pipe/docker_engine
error dialing pipe open //./pipe/docker_engine: The system cannot find the file specified.
dialing named pipe //./pipe/docker_engine
error dialing pipe open //./pipe/docker_engine: The system cannot find the file specified.
pipe dialed
pipe dialed
pipe dialed
I also tried the timeout parameter that is nil, but this also doesn't help. The pipe isn't busy, it just cannot be found so the winio function returns. Maybe this should be fixed in go-winio, but for now this makes Portainer work much more stable with the named pipes.

@friism Added a retry (without the Println shown above) with https://github.com/portainer/portainer/commit/2263c5590af15bc0fc150c548856b78d846ba479 (branch https://github.com/StefanScherer/portainer/tree/pr1186-4) which makes Portainer work in my Insider VM's.

@StefanScherer
I'm not really keen on adding the retry workaround, the named pipe support feature should be integrated only if it is stable.

The pipe isn't busy, it just cannot be found so the winio function returns. Maybe this should be fixed in go-winio, but for now this makes Portainer work much more stable with the named pipes.

I'd say the way to go would be to report that issue first in the go-winio repository and discuss it with the maintainers. Would one of you be able to file an issue there?

@deviantony Agreed. I've opened https://github.com/Microsoft/go-winio/issues/67 as the internal call createFile returns that the pipe is busy, but the waitNamedPipe just does not wait. If that would work we can specify a timeout instead of nil or leave it at the default timeout.

Great, let's wait for an update on this issue.

My current status on this:

Windows 10 1709 is available, even Enterprise Eval ISO, but you cannot use named pipe mapping with HyperV containers, so this is not usable for Windows 10 users with D4W or native dockerd.exe running.
Windows Server 1709 ISO available with MSDN subscription, named pipe mapping works with the standard non-HyperV-containers.

Thanks for the update @StefanScherer
Do you know if named pipe mapping will be available to Window 10 users ?

@deviantony I don‘t know if this is on the roadmap. Maybe we see progress in the next Insider builds.

Does anyone have a container pushed to Docker Hub that I could try?

@PatrickLang I have pushed a new image to stefanscherer/portainer:1709 with the binaries from my previous stefanscherer/portainer:insider image. The corresponding Dockerfile can be found here.
docker run -v //./pipe/docker_engine://./pipe/docker_engine -p 9000:9000 -d stefanscherer/portainer:1709

Enter a password for user admin and then try to use the local connection. Unfortunately I always get an error "Fatal: Invalid request data format"
This portainer.exe binary worked with the insider builds as far as I can remember.
I also have added a docker CLI into the 1709 image, so you can check the named pipe with
docker exec $(docker ps -ql) docker version

Ok, let's get back on this.
I've fix a compilation issue related to duplicate code (probably a merge gone wrong).

Enter a password for user admin and then try to use the local connection. Unfortunately I always get an error "Fatal: Invalid request data format"

I believe I just fixed this.
As we don't have any input yet for https://github.com/Microsoft/go-winio/issues/67, I've also pushed the workaround that @StefanScherer introduced via https://github.com/portainer/portainer/commit/2263c5590af15bc0fc150c548856b78d846ba479
@PatrickLang
I've just created a new image for this based on our latest branch, it is available via portainer/portainer:pr1186-windows-amd64. Could you give it a try ?
Worth noting that this PR cannot be merged in this state as it will not compile on Linux (platform dependent code should be isolated).

@deviantony Thanks for providing the test image. It is based on microsoft/nanoserver:10.0.14393.1770, but I have replaced the base image with microsoft/nanoserver:1709 and pushed to stefanscherer/portainer:pr1186-1709.
It works for me with
docker run -v //./pipe/docker_engine://./pipe/docker_engine -p 9000:9000 -d stefanscherer/portainer:pr1186-1709

ping @PatrickLang

@StefanScherer can I use the latest version of microsoft/nanoserver image?
Also, could any of you confirm that the container console feature is working?

@deviantony If you are running Windows Server 2016 (10.0.14393.x) then you cannot run the microsoft/nanoserver:1709 container.
To be able to run this you need Windows Server 1709 (10.0.16299.x). The kernel version of host and container must match. Only with Server 1709 + HyperV feature enable you are also able to run older 10.0.14393 images with docker run --isolation hyperv ....
And yes, the console (cmd.exe into the portainer container as nano has no powershell.exe) works:

Only the xterm.js (right?) shows some window shade effect, but this also happens in older versions.
BTW: The next PowersShell 6 beta 9 renames powershell.exe to pwsh.exe.

Thanks! Giving it a look

Here's what I'm getting right now. I had to update to the latest docker/dockerd.exe from master.dockerproject.org first.
Process isolation (default on Windows Server 1709) - works
with --isolation=hyperv - verified both on Windows Server version 1709 and Windows 10 version 1709
 Connect Failure
open //./pipe/docker_engine: message readmode pipes not supported

Is that consistent with what you're seeing @friism and @StefanScherer ?

@PatrickLang I also saw message readmode pipes not supported as I accidentally called portainer with --isolation=hyperv.

If you are running Windows Server 2016 (10.0.14393.x) then you cannot run the microsoft/nanoserver:1709 container.

To be able to run this you need Windows Server 1709 (10.0.16299.x). The kernel version of host and container must match. Only with Server 1709 + HyperV feature enable you are also able to run older 10.0.14393 images with docker run --isolation hyperv ....

So, let's say I would like this to be part of our next Portainer release, what information message would I need to show to the users? Can I still use microsoft/nanoserver:latest or is microsoft/nanoserver:1709 mandatory?
If I understand correctly, this feature woud only be available for Windows Server 1709 (10.0.16299.x) users?

@deviantony Yes, microsoft/nanoserver:1709  is mandatory at the moment and will only work on Windows Server 1709 without the HyperV isolation mode.

Please ensure that you have started the Portainer container with the following Docker flag -v /var/run/docker.sock:/var/run/docker.sock (Linux) or -v \.\pipe\docker_engine:\.\pipe\docker_engine (Native Windows containers) in order to connect to the local Docker environment.

Maybe this:
Please ensure that you have started the Portainer container with the following Docker flag -v /var/run/docker.sock:/var/run/docker.sock (Linux) or -v \.\pipe\docker_engine:\.\pipe\docker_engine (Native Windows Server 1709 containers) in order to connect to the local Docker environment.
Maybe there will be a fix for HyperV isolation in the future, then Windows 10 users could use that.
I've used the manifest-tool today to provide multi-os-version manifest list to support both Windows versions. This works fine, so the docker engine pulls the image with the matching os.version number.
I can provide you more details after some further tests how to provide two Windows variants for the next release.

@StefanScherer would there be any con to use microsoft/nanoserver:1709 instead of microsoft/nanoserver:latest ? I'd like to avoid having too many images to maintain. Will latest point to 1709 at some point?
We could also merge this, or keep it open, until it can affect a wider userbase.

@deviantony yes, if you only use microsoft/nanoserver:1709 image then Windows Server 2016 LTS users that don't update to the Semi-annual channel cannot run Portainer, even with HyperV isolation it's not possible to run newer container images on older Windows Docker host.
I was able to produce a manifest list for multiple Windows OS versions today with manifest-tool push from-spec with a manifest YAML file. I guess the manifest-tool push from-args won't help us here.
Creating a second Windows image for 1709 would work even on an older Windows Docker host. I did that eg. for a curl image with a manifest.yml and these steps in AppVeyor.
At least having this PR merged and available with only one Windows image would be better than nothing. But as portainer/portainer was the first best use-case for multi-arch as we just can docker run portainer/portainer without thinking of any platform, it would be consequent to add 1709 support as well :-)
I'll explain my steps in a blog post soon to get more feedback on that approach.

Well, there is still some work to do before this PR can be merged but I agree with you, it'd be neat to have this feature implemented even if it will be only available to a specific set of users.
Ping me when your blog post is ready, I'm keen to add 1709 support :)

@deviantony https://stefanscherer.github.io/poc-build-images-for-1709-without-1709/ 🎃

We still have no answer in https://github.com/Microsoft/go-winio/issues/67
This PR waits for the refactoring to make it work for Linux again, @friism ping me if I should take care of the remaining steps.

that's a great goal!
For windows socket I found a little interesting article：
https://blogs.msdn.microsoft.com/commandline/2017/12/19/af_unix-comes-to-windows/
https://blogs.msdn.microsoft.com/commandline/2018/02/07/windowswsl-interop-with-af_unix/

There is no AF_UNIX support in Docker engine on Windows yet. See https://github.com/moby/moby/issues/36442 for first ideas. Also Golang has to support it first.

npipe support is very theoretical at the moment.

The Docker 17.10.0-preview3 should no longer be used due to security reasons.
The current Docker EE 17.06.2-ee-10 does not have npipe support.
The Docker for Windows Stable and Edge binaries have npipe support, eg. D4W Edge with 18.05.0-ce binaries.

I'm using the dockerd.exe from D4W Edge in a Windows Server 1803 on Azure.  With this setup I can run docker run -v //./pipe/docker_engine://./pipe/docker_engine stefanscherer/docker-cli-windows docker version which works fine.
But on Windows 10 there is only hyperv isolation mode and this is not supported for npipe mounts.

@friism, @StefanScherer 18.03.1-ee-1is out so can we now get forward with this one?

@olljanat named pipes into process isolated containers will work with 18.03.1. use with hyper-v isolated containers requires PRs that landed too late to make it into 18.03.1-ee-1:

Microsoft/go-winio#82
moby/moby#37364
docker/cli#1157

See https://github.com/docker/cli/pull/1157#issuecomment-401489729

@carlfischer1 based on my experience I would not recommend Hyper-V isolation mode for anyone anyway because of much bigger overhead and worse performance.
So can we get this one merged so we can start using it on environments which are using process isolation mode?

@olljanat The implementation of this POC relied on the microsoft/go-winio library. We've spotted an issue during our tests which has still not been fixed: https://github.com/Microsoft/go-winio/issues/67
I believe that we should wait for an update on this topic before going further.

@deviantony ok. then lets continue with non-containerized Windows agent on #1929
This is very critical feature for companies like us who are running mixed Linux + Windows swarms on production.

Closed in favor of #2018


chore: fixed name of starknet.js target

Name of the target was @typechain/starknet.js but should be @typechain/starknet

Same PR as https://github.com/dethcrypto/TypeChain/pull/748


remove contribute from readme

remove contribute from readme

Thanks for your PR, I'll close this one as PR #82 will cover this issue.


Notebooks: Separate ContentManager from NotebookManager

Now that it's possible to have multiple NotebookManagers (e.g. with SQL and Jupyter, for example), this now means that there are multiple (local) ContentManagers. We really should have a local ContentManager that's separate from the NotebookManager.

While Jupyter implementation was removed still define it in sqlops.proposed. Need to delete / solve refactoring.


Fail to Install pnpm on GitHub Actions with ERR_PNPM_FETCH_404

My config
name: Deploy Blog

on:
  push:
    branches:
      - main

jobs:
  deploy-gh-pages:
    runs-on: ubuntu-latest
    steps:
      - name: Checkout
        uses: actions/checkout@v3
        with:
          fetch-depth: 0

      - name: Install pnpm
        uses: pnpm/action-setup@v2 # I try both v2 or v2.2.2
        with:
          version: 7
          run_install: true

# ... other action

Error Message:
Run<EMAIL_ADDRESS>Running self-installer...
Installation Completed!

Running pnpm recursive install...
  Lockfile is up to date, resolution step is skipped
  Progress: resolved 1, reused 0, downloaded 0, added 0
  .                                        | +575 ++++++++++++++++++++++++++++++++
  Packages are hard linked from the content-addressable store to the virtual store.
    Content-addressable store is at: /home/runner/.local/share/pnpm/store/v3
    Virtual store is at:             node_modules/.pnpm
  Progress: resolved 575, reused 0, downloaded 21, added 19
   WARN  GET https://registry.npmjs.org/<EMAIL_ADDRESS>error (ERR_PNPM_FETCH_404). Will retry in 10 seconds. 2 retries left.
  Progress: resolved 575, reused 0, downloaded 140, added 137
  Progress: resolved 575, reused 0, downloaded 187, added 184
  Progress: resolved 575, reused 0, downloaded 275, added 274
  Progress: resolved 575, reused 0, downloaded 342, added 339
   WARN  GET https://registry.npmjs.org/concat-map/download/concat-map-0.0.1.tgz error (ERR_PNPM_FETCH_404). Will retry in 10 seconds. 2 retries left.
  Progress: resolved 575, reused 0, downloaded 481, added 475
  Progress: resolved 575, reused 0, downloaded 5[6](https://github.com/plylrnsdy/plylrnsdy.github.io_source/actions/runs/3065146727/jobs/4948951570#step:3:7)1, added 556
  Progress: resolved 575, reused 0, downloaded 572, added 572
  Progress: resolved 575, reused 0, downloaded 573, added 572
  Progress: resolved 575, reused 0, downloaded 573, added 573
   WARN  GET https://registry.npmjs.org/<EMAIL_ADDRESS>error (ERR_PNPM_FETCH_404). Will retry in 1 minute. 1 retries left.
   WARN  GET https://registry.npmjs.org/concat-map/download/concat-map-0.0.1.tgz error (ERR_PNPM_FETCH_[40](https://github.com/plylrnsdy/plylrnsdy.github.io_source/actions/runs/3065146727/jobs/4948951570#step:3:42)4). Will retry in 1 minute. 1 retries left.
  undefined
   ERR_PNPM_FETCH_404  GET https://registry.npmjs.org/<EMAIL_ADDRESS>Not Found - 404

  No authorization header was set for the request.
Error: Command pnpm recursive install (cwd: undefined) exits with status 1

I try to open these links. They actual do not exist.
https://registry.npmjs.org/<EMAIL_ADDRESS>https://registry.npmjs.org/concat-map/download/concat-map-0.0.1.tgz

Can you try separating this block:
      - name: Install pnpm
        uses: pnpm/action-setup@v2 # I try both v2 or v2.2.2
        with:
          version: 7
          run_install: true

into:
      - name: Install pnpm
        uses: pnpm/action-setup@v2 # I try both v2 or v2.2.2
        with:
          version: 7
          run_install: false

      - name: Install dependencies
        run: pnpm install --recursive

and see if it's the same result?

I try remove run_install: true
name: Deploy Blog

on:
  push:
    branches:
      - main

jobs:
  deploy-gh-pages:
    runs-on: ubuntu-latest
    steps:
      - name: Checkout
        uses: actions/checkout@v3
        with:
          fetch-depth: 0

      - name: Install pnpm
        uses: pnpm/action-setup@v2
        with:
          version: 7

      - name: Setup Node.js
        uses: actions/setup-node@v3
        with:
          node-version: 16
          cache: pnpm

      - name: Install dependencies
        run: pnpm install --frozen-lockfile

# ... other action

Now it can run into Install dependencies, but it throws same error:
Run pnpm install --frozen-lockfile
Lockfile is up to date, resolution step is skipped
Progress: resolved 1, reused 0, downloaded 0, added 0
Packages: +575
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Packages are hard linked from the content-addressable store to the virtual store.
  Content-addressable store is at: /home/runner/.local/share/pnpm/store/v3
  Virtual store is at:             node_modules/.pnpm
Progress: resolved 575, reused 0, downloaded 10, added 9
 WARN  GET https://registry.npmjs.org/<EMAIL_ADDRESS>error (ERR_PNPM_FETCH_404). Will retry in 10 seconds. 2 retries left.
Progress: resolved 575, reused 0, downloaded 53, added 50
Progress: resolved 575, reused 0, downloaded 12[6](https://github.com/plylrnsdy/plylrnsdy.github.io_source/actions/runs/3065265181/jobs/4949190343#step:5:7), added 122
Progress: resolved 5[7](https://github.com/plylrnsdy/plylrnsdy.github.io_source/actions/runs/3065265181/jobs/4949190343#step:5:8)5, reused 0, downloaded 167, added 165
Progress: resolved 575, reused 0, downloaded 1[8](https://github.com/plylrnsdy/plylrnsdy.github.io_source/actions/runs/3065265181/jobs/4949190343#step:5:9)4, added 183
Progress: resolved 575, reused 0, downloaded 244, added 242
Progress: resolved 575, reused 0, downloaded 278, added 276
Progress: resolved 575, reused 0, downloaded 318, added 318
Progress: resolved 575, reused 0, downloaded 422, added 420
 WARN  GET https://registry.npmjs.org/concat-map/download/concat-map-0.0.1.tgz error (ERR_PNPM_FETCH_404). Will retry in [10](https://github.com/plylrnsdy/plylrnsdy.github.io_source/actions/runs/3065265181/jobs/4949190343#step:5:11) seconds. 2 retries left.
Progress: resolved 575, reused 0, downloaded 506, added 504
Progress: resolved 575, reused 0, downloaded 564, added 564
Progress: resolved 575, reused 0, downloaded 571, added 571
 WARN  GET https://registry.npmjs.org/<EMAIL_ADDRESS>error (ERR_PNPM_FETCH_404). Will retry in 1 minute. 1 retries left.
Progress: resolved 575, reused 0, downloaded 572, added 572
Progress: resolved 575, reused 0, downloaded 573, added 573
 WARN  GET https://registry.npmjs.org/concat-map/download/concat-map-0.0.1.tgz error (ERR_PNPM_FETCH_404). Will retry in 1 minute. 1 retries left.
 ERR_PNPM_FETCH_404  GET https://registry.npmjs.org/<EMAIL_ADDRESS>Not Found - 404

No authorization header was set for the request.
Error: Process completed with exit code 1.

Then it's not the fault of pnpm/action-setup. It's either a pnpm bug, a registry bug, or a package.json bug.

Thanks.
This issue is related to pnpm#3744.
Because I use third party's registry globally, I need to setup same registry on GitHub Actions.
name: Deploy Blog

on:
  push:
    branches:
      - main

jobs:
  deploy-gh-pages:
    runs-on: ubuntu-latest
    steps:
      - name: Checkout
        uses: actions/checkout@v3
        with:
          fetch-depth: 0

      - name: Install pnpm
        uses: pnpm/action-setup@v2
        with:
          version: 7

      - name: Setup Node.js
        uses: actions/setup-node@v3
        with:
          node-version: 16
          cache: pnpm

      - name: Use taobao registry
        run: npm config set registry https://registry.npmmirror.com/

      - name: Install dependencies
        run: pnpm install --frozen-lockfile

# ... other action


How to know all starting points exactly on map?

I need to know the starting points near traffic lights (for example) or near cross-roads. Is there a map as to where exactly each point reflects to each position in the map?

I am interested in this too.

I couldn't find a certain way of knowing all the start points visually for each map. So I had to:

Make carla start a new episode randomly.

Print this number out.

Make this number your fixed number from now on instead of starting randomly.

In the simlation itself, you can zoom out until you can see where the car is exactly in the map.

As per traffic, it turns out that carla only returns ALL the states of all traffic lights in the whole map in each single frame. No specific traffic light that's ahead of you is given. So it's pretty hard to know the excat state of the traffic light ahead of you (the car)

@LoayMAmin why not just open the Carla project in the UE4 Editor and mark the coordinates of all the player start points and see if a traffic light is near it.
Alternatively, map all the player start points to the lane information map and just see its distance from a T intersection. (Traffic lights are only at the intersection)

Hello. This was fixed is 0.8.1
Please view http://carla.org/2018/04/05/release-0.8.1/


Hash battle tracks alongside the race tracks

This also always hashes tracks, even if not My Stuff replaced, to check for possible bad dumps or bad disks.

How long does this take on console?

I haven't checked, but possibly a significant amount of time. Having just thought about it, I could remove the disk hash patch if I implement more logic down the road for #884.

Just checked and it takes an extra 30 seconds on boot, which is unacceptable, so I have removed that section of the PR.


Renaming .kitchen.docker.yml to .kitchen.dokken.yml

PLEASE STOP RENAMING THIS to .kitchen.docker.yml.

They are two completely different plugins. Having a naming mismatch creates confusion.

From the kitchen-dokken README under USAGE:

Add the following to your ~/.bash_profile
  export KITCHEN_YAML=.kitchen.yml
  export KITCHEN_LOCAL_YAML=.kitchen.dokken.yml

With the above settings, if you cd into a cookbook with a .kitchen.dokkken.yml, it will automatically pick up and use the dokken config. Otherwise it defaults to Vagarnt.

Doing "cd mysql ; kitchen verify smoke -c" explodes laptops under Vagrant.

Signed-off-by: Sean OMeara<EMAIL_ADDRESS>

After discussions with folks in the cookbook-engineering channel we decided on standardizing on the .kitchen.DRIVER.yml configuration. There were conflicts on your PR so I went ahead and just updated the changes. Thanks @someara.


Proposal: MediaPlayerElement enhancement

Bring native MediaPlayerElement UI to Xbox.
For more details, please refer to Bring native MediaPlayerElement UI to Xbox

Keep the playback rate.
Currently, MediaPlayerElement reset the playback rate when playing a new media, even I set the rate manually. This means developers can't set the rate, this is annoying when playing a series drama. Users want 2x rate, they have to set every episode.......
This code doesn't work.

ViuMediaPlayer.Source = mediaPlaybackItem;
mediaPlayer.PlaybackSession.PlaybackRate = 2.0;

Add debug panel and relative APIs.
Currently, Apple TV player supports debug panel like YouTube's stats for nerds. This debug panel is critically important for developers to check the network status.

More.
The above is what I'm facing now, but I think there is more to improve. Cause the UWP user voice forum is closed, I can't find any more info.

This means developers can't set the rate

You need to set playback rate after media opened, I think.
At least it worked for me for a while ago.

This means developers can't set the rate

Hi, I tried the code, If I use a static speed, it works.
mediaPlayer.PlaybackSession.PlaybackRate = 2.0;
But I have to store the rate in a global param. Then the rate always is reset to 1.0 when changing the source.
private async void MediaPlayer_MediaOpened(MediaPlayer sender, object args)
{
if(videoType == VideoType.Video)
mediaPlayer.PlaybackSession.PlaybackRate = Globals.playbackRate;
}
    //PlaybackRate will always be set to 1.0 when changing the source
    private void PlaybackSession_PlaybackRateChanged(MediaPlaybackSession sender, object args)
    {
        Globals.playbackRate = mediaPlayer.PlaybackSession.PlaybackRate;
    }

@hupo376787 can you check playback session state in the PlaybackSession_PlaybackRateChanged?

@hupo376787 can you check playback session state in the PlaybackSession_PlaybackRateChanged?

Thanks, @maxkatz6 . I can keep the playback rate now by checking MediaPlaybackState.
        private void PlaybackSession_PlaybackRateChanged(MediaPlaybackSession sender, object args)
        {
            Debug.WriteLine(mediaPlayer.PlaybackSession.PlaybackState);
            if(mediaPlayer.PlaybackSession.PlaybackState != MediaPlaybackState.None)
                Globals.playbackRate = mediaPlayer.PlaybackSession.PlaybackRate;
        }

@Austin-Lamb or @codendone I'm struggling to remember where this should get pathed, I think reach is correct?


SurveyActivity java.lang.NoSuchMethodError

Hi ,
The in-app notification is crashing on android 4.4 and lower
See the Stackstrace,
thanks
app_issue_1423_crash_58B6E7D4000300013F853D9EE4768F20_754b11427af04fdb8a31b4466dac6314_0_v1.txt

Hi @amensia, thank you very much for the crash report. I've fixed this issue and made a new release that is now available (4.9.7). Take care!

Awesome, thank you ;)

Hi @patedit .Still have some issue in 4.9.7.  In-App Message can't remain on screen, it close itself automatically very soon. But I do not see any error message in Android monitor. :(

Hi @HsuHsiaoHauan! Can you open a new issue and describe what happens since your issue is not related to this ticket? Thank you!


Travis build on JDK 6+7+8

Configures Travis to build on Oracle JDK 7+8 and OpenJDK 6+7.
Also enables migration to Travis' container-based infrastructure: http://docs.travis-ci.com/user/migrating-from-legacy/

Apparently build is broken on JDK 6:
[ERROR] Failed to execute goal org.apache.maven.plugins:maven-compiler-plugin:3.1:compile (default-compile) on project spring-oauth2-tests-common: Fatal error compiling: invalid target release: 1.7 -> [Help 1]

It should be fine to use the default JDK (which I think is openjdk8). Not sure why we get so many travis fails though.

Travis doesn't have OpenJDK 8: https://docs.travis-ci.com/user/languages/java#Testing-Against-Multiple-JDKs

Fine. My comment still stands (the default ought to be fine).


[DOCFIX] Getting started cluster doc improvements

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/20269/
Test PASSed.


Fixes problem with chicken-and-env for react-native-config.

Finishes hooking up react-native-config to Android and temporarily switch to a branch which doesn't die when building projects missing an .env file.
https://github.com/luggit/react-native-config/pull/31

Self Merged.  I did however, review it in a british accent.  So pretty much good to go.

as long as it wasn't tea time, you're good.


Now that we have custom color, can we remove ice from trees?

Just set the foliage color to the same color as the ice.

Done


_build_tree: Sort the triangles along the split dimension

Is it a bug? The comment says "Sort the triangles according to their center along the split dimension" but it actually sorts them based on vertex 0?
https://github.com/InteractiveComputerGraphics/TriangleMeshDistance/blob/f66ad12cc0d201d299c76e460f5a355e768bf6dc/TriangleMeshDistance/include/tmd/TriangleMeshDistance.h#L488-L494

Hi. That was simply an old comment that was never corrected. I updated it now. Thanks for pointing that up! :)


Solo.getText(String) takes regexp but does literal comparison

From<EMAIL_ADDRESS>on May 22, 2012 21:46:35
What steps will reproduce the problem? 1. call Solo.getText("Hello (World)")  => this will throw an exception since the string is no valid regexp (since inside Getter.getView calls waiter.waitForText which expects a regexp)
2. however, in Getter.getView a few lines down robotium performs a literal comparison
public \<T extends TextView> T getView(Class<T> classToFilterBy, String text, boolean onlyVisible) {

=> REGEX		waiter.waitForText(text, 0, 10000, false, onlyVisible);
ArrayList views = viewFetcher.getCurrentViews(classToFilterBy);
if(onlyVisible)
views =	RobotiumUtils.removeInvisibleViews(views);
T viewToReturn = null;
for(T view: views){
=> LITERAL		if(view.getText().toString().equals(text))
viewToReturn = view;
} What is the expected output? What do you see instead? I think that the Robotium API needs to make it very clear which api uses regex and which uses literal. Right now it seems to be a mixed bag. Ideally all text methods should be able to provide both versions of the api. Wherever a regex or matcher is passed in, then a regex comparison is made, whenever a String is passed in, a literal comparison is made. What version of the product are you using? On what operating system? robotium 3.2.1 Please provide any additional information below.
Original issue: http://code.google.com/p/robotium/issues/detail?id=263

From<EMAIL_ADDRESS>on May 22, 2012 23:21:36
Thanks for reporting this. I agree, the best solution is to allow both inputs. Would it be possible for you to send a patch through github?

From<EMAIL_ADDRESS>on May 23, 2012 05:03:30
I don't have a patch and I am not sure whether I will have the time to create one (as it probably would mean that approx half of the solo methods need to be rewritten. :(

From<EMAIL_ADDRESS>on May 23, 2012 05:13:50
I will take care of it. Thanks again for reporting it.
Status: Accepted
Owner<EMAIL_ADDRESS>

From<EMAIL_ADDRESS>on May 28, 2012 01:13:18
Just suffered from the same issue, can't click on a button with the string "+"

From<EMAIL_ADDRESS>on May 28, 2012 01:15:52
If you use "\" before the text then special characters will not be taken into account.

From<EMAIL_ADDRESS>on May 28, 2012 01:42:53
Thanks it works!

From<EMAIL_ADDRESS>on June 11, 2012 09:10:51
This has been partly fixed in Robotium 3.3.
Status: Verified


[ST] VCX transaction 32hs

Describe the bug
A transaction of 20k VCX from Optimism to Ethereum has been processing for 1d8h. https://connextscan.io/tx/0x48d959f77ed71cb0dbf99eb4157156332e8e0d1cafef12c67715733e49fa6983
Why is it taking that long? We have another user sending 27k right now (https://connextscan.io/tx/0x9166a5339fc07bbcbfc53e87de83990e96f41080fb339e840312dca7510df84e), his transaction has been processing for 5h and he's worried that it might not complete just like the 20k one.
Where did the user initiate this transfer?
Vaultcraft's UI
Link to ticket in intercom
https://app.intercom.com/a/inbox/qge2vd8z/inbox/shared/all/conversation/1482?view=List

Another transaction: https://connextscan.io/tx/0x42277fc667904a9a6a0ae8875df5d39ce0924b733503a130a4782a10b63a8dc2


update the flutter create template files

update the flutter create template iml files to the latest that the Dart plugin generates (for context, https://github.com/flutter/flutter/issues/12639#issuecomment-338049759)

@cbracken


asd

asd

Closing this PR. This seems like a rebase of some sort. If you have a change to propose, feel free to open another PR with a description of what you have in mind and a git commit that you authored.


Replace cookie consent with micro:bit version

This is common to all Foundation apps.
I've slimmed down the CookieManager to just be responsible for cookie read/write rather than consent.
I've deleted the cookie UI we no longer use. Although likely to cause conflicts in future, these will at least be easy to address by retaining the delete. Otherwise I had to retain stub functionality in CookieManager for the unused UI.
All non-analytics scenarios are likely to be classed as necessary/essential so this seems fairly straightforward.

This seems to be working nicely, it also fixes a problem we were having with cookie dialog flash on page reload 😄
Is there any more work that needs doing on this before it can come out of draft and we can get it merged?


TileSource with EPSG:3395 (WGS 84 / World Mercator)

Hello. Thanks for your work!
How can I add tiles in EPSG:3395 (WGS 84 / World Mercator) and use it in TileSource?
Which direction do you recommend to look? Do I need my own TileProvider or something more?
I tried this solution with TilesOverlay (I updated it to new version) but it has some issues with caching and displaing.
Issue Type
[x] Question
Version
6.1.6

@DaniilPavlenko I don't known that much about projections and I cannot read cyrillic scripts, but perhaps #971 and subclassing TileSystem might be helpful (we use TileSystemWebMercator).

osmdroid only uses EPSG:3857. So you\ll need a way to convert whatever you have into EPSG:3857.  One solution is to use geoserver with gdal. I'm pretty sure it can convert it.
Then you can either wire up osmdroid's wms library, or use mobac to create an offline tile store from geoserver

just create Proj4TileSystem extends TileSystem class with public static final Proj4TileSystem YANDEX; and init this in static block:
static{
YANDEX = new Proj4TileSystem(
"EPSG:3395",
"+proj=merc +lon_0=0 +k=1 +x_0=0 +y_0=0 +datum=WGS84 +units=m +no_defs",
new BoundingBox(85.05112877980659, 180, -85.05112877980659, -180),
new RectF(-20037508, -20037508, 20037508, 20037508 ));
MERCATOR = new Proj4TileSystem(
"EPSG:3857",
"+proj=merc +a=6378137 +b=6378137 +lat_ts=0.0 +lon_0=0.0 +x_0=0.0 +y_0=0 +k=1.0 +units=m
+nadgrids=@null +wktext +no_defs",
new BoundingBox(85.05112877980659, 180, -85.05112877980659, -180),
new RectF(-20037508, -20037508, 20037508, 20037508));
}
then use MapView.setTileSystem(Proj4TileSystem.MERCATOR) for example on OpenStreetMap and MapView.setTileSystem(Proj4TileSystem.YANDEX) when switching on Yandex

@MaximKrasikov Thanks, your answer very help


FE: Fixing the style of certificate text

Should fix the issue with certificate not having the correct format when displayed

Is this fixing a specific issue? If so, could you please link it to this PR?


GSD Request

--- GSD JSON ---
{
  "vendor_name": "Linux",
  "product_name": "Kernel",
  "product_version": "versions from v4.0 to before v4.19.224",
  "vulnerability_type": "unspecified",
  "affected_component": "unspecified",
  "attack_vector": "unspecified",
  "impact": "unspecified",
  "credit": "",
  "references": [
    "https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git/commit/?id=5e33f6fdf735cda1d4580fe6f1878da05718fe73",
    "https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git/commit/?id=33f6a0cbb7772146e1c11f38028fffbfed14728b"
  ],
  "extended_references": [
    {
      "type": "commit",
      "value": "5e33f6fdf735cda1d4580fe6f1878da05718fe73",
      "note": "introduced"
    },
    {
      "type": "commit",
      "value": "33f6a0cbb7772146e1c11f38028fffbfed14728b",
      "note": "fixed"
    }
  ],
  "reporter": "joshbressers",
  "reporter_id": 1692786,
  "notes": "",
  "description": "usb: gadget: f_fs: Clear ffs_eventfd in ffs_data_clear.\n\nThis is an automated ID intended to aid in discovery of potential security vulnerabilities. The actual impact and attack plausibility have not yet been proven.\nThis ID is fixed in Linux Kernel version v4.19.224 by commit 33f6a0cbb7772146e1c11f38028fffbfed14728b, it was introduced in version v4.0 by commit 5e33f6fdf735cda1d4580fe6f1878da05718fe73. For more details please see the references link."
}
--- GSD JSON ---

This issue has been assigned GSD-2022-1000023


Add tests for video component

What is the current behaviour?
Video component works but there are no tests.
What is the expected behaviour?
Component should have tests.

I would be happy to help here, @pascalwengerter.

Thanks a million @MarcoBomfim - happy coding and let us know if you run into troubles ;)

@pascalwengerter, I'm having trouble getting a new project up and running to test out some of the matestack's components before testing them out, can you let me know whether the documentation on the site is up to date? Or if I'm perhaps missing something?
I'm following the steps after running a rails new command.

@pascalwengerter Checking up on the image_spec assertion actually helped me out getting passed the issue I was having. Although, I'm also on a different machine, so I can't confirm if anything else was of help.

Closed via #259 thanks to @MarcoBomfim :rocket:


Android no value for key

Hi this behaves differently on Android to iOS.
On iOS if there is no value when fetching a key null is returned to the success callback, on android the errorCallback is called
line 145 :
if (sharedPrefs.contains(key)) {
....
line 177 :
} else {
//          Log.d("", "no value");
callbackContext.error(0);
//            callbackContext.sendPluginResult(new PluginResult ());
}
To rectify and fit my needs I changed this in the else
line 177 :
} else {
//          Log.d("", "no value");
callbackContext.success(returnVal);
}
as returnVal is declared as null on line 144.
In my personal opinion I think its better for them both to return null to the success callback rather than the call the error callback

fixed with ff52c73503331a90fe34e0b62fc7498bfe2f23f6


about the generator

currently generator works only on windows, i can't make it work on other OSes.
But i could write a php script that would work on all OSes, windows, mac, linux, but you would have to have a php installed to run it.
Or we could set up a travis CI to run the php script on incoming change that would generate those files zip them and put them into release.

I vote for Travis CI, one script to rule them all :)

my vote also goes to Travis, would love to learn more about it!

+1 for travis + script or wercker + script.

Can you guys help me out with this? Check the develop branch for current progress.

you don't neet phpunit, there are no unit tests in your project, in your .travis.yml "script" line remove phpunit, and leave only one php version, as i don't think we have a need to run this script on different php versions to check compatibility:
php:
  - '5.6'
script: generator.php

unless you want to make sure that generate.php runs on all php versions :)

@Gamesh thanks for the small fix'ups I changed the the travis CI file but it's still failing :(
/home/travis/build.sh: line 45: generator.php: command not found

try adding php:
script: php generator.php

if it does not work it might be because travis is not building in the root dir of the probject.

in combination with chmodding the .htaccess to 777 made the generator work, next step is to add a nginx version: Build #25 was fixed. thanks so much @Gamesh!!

added the creation of a nignx referral-spam.conf file to the generator! Could somebody please look at the generator file?  And please tell me what's next :+1:

@Stevie-Ray looks good. Take a look here:
https://docs.travis-ci.com/user/deployment/releases/

@Gamesh This works only if a tag is applied to a commit, see: #37 I'm not really sure if it works..

@Stevie-Ray yes this method probably will work and will build on every commit. Usually people commit and repo owner does the merges/pull requests and then when he decides that this is release ready tags current version and the release is built. In your case i agree maybe a per commit build would be better.
the only way to see is to try it out :)

Hi @Gamesh, the deployment with tags is working, the only problem is that it doesn't do anything yet, see: #51 and the travis file for reference, can you see what the problem is?

hmm, but it published the files to the releases, what do you expect it to do ?
https://github.com/Stevie-Ray/apache-nginx-referral-spam-blacklist/releases
by the way the global secure token should not be visible, put it in the environment variables.
and add ls -aR so the .htaccess would be visible.
also maybe we should zip both .conf and .htaccess and deploy the zip?

@Gamesh in my opinion would be awesome if it could continuously update the two files in the repo, so in my case I can just use git pull and restart my nginx server to make it effective.
As it is now I have to download always the latest conf file and place if manually somewhere. Would it be possible to make travis commit the output on the master repo?

isn't the secure token encrypted? I prefer updating the files in the repo (to the master branch), not as a release just like @julianxhokaxhiu mentioned :) anyway thanks for your help guys!

@Stevie-Ray I prefer the master branch too. I'm not really a big fan of releases, at least, not for this project :)

i thought that the whole point of having travis was to run maybe some checks, sort the domains.txt or do other stuff like validations of some kind and then release daily or on the need basis and maybe provide an auto update script for people that would always download the latest version from releases using github api.

sorting the domains.txt would be nice. It would be awesome if you guys could help me with the travis file.

I'm gonna have some Christmas holidays in the next days so I'll try to take a look at it :)

I added a sort function for the domains in run.php file in the develop branch (the code is still a bit messy, with more code duplication). But, it works! I already wish you a merry Christmas ;-)


Client undefined on first page load

Hey,
I'm having an issue with routing on a page refresh because the client in AuthenticationProperties seems to be undefined and therefore, the authentication status is not correctly retrieved. Here is the error:

After a few seconds, authentication is established and if I now click on any router-link within my vue app, it just works fine.

I've developed a workaround by checking if client is defined within my AuthenticationGuard:
// If the Auth0Plugin has loaded already, check the authentication state
  if (AuthenticationProperties.client && !AuthenticationState.loading) {
    return guardAction()
  }

  watch(AuthenticationState, async () => {
    if (AuthenticationProperties.client && !AuthenticationState.loading) {
      console.log(AuthenticationState)
      guardAction()
    }
  })

Hey @larsschellhas, did you try the beta version yet?

@jnt0r I haven't had the chance to try it, yet. Hopefully on the weekend! But thank you for providing a potential fix! 😊

To be honest, @jnt0r, I can't really get it to work right now with the beta version and the promise.
I'm gonna keep my workaround for now as it actually works quite stable :)
But thank you for your efforts!

Would be happy to make it work for you too.


Pods of one app from two clusters are registered in one pilot service entry

Describe the bug
We've successfully deployed the Single control plane topology with two clusters.
Some of our apps are deployed simultaneously in these clusters, this results in this problem:
Pods of one app from two clusters are registered in one pilot service entry as below, only half requests on the service are successful.
  {
   "service-key": "k8s-demo.stage.svc.cluster.local|http",
   "hosts": [
    {
     "ip_address": "<IP_ADDRESS>",
     "port": 7001
    },
    {
     "ip_address": "<IP_ADDRESS>",
     "port": 7001
    },
    {
     "ip_address": "<IP_ADDRESS>",
     "port": 7001
    },
    {
     "ip_address": "<IP_ADDRESS>",
     "port": 7001
    }
   ]
  }

Expected behavior
Pods of one app from two clusters are registered in different pilot services entries, all requests on the services should be successful.
Steps to reproduce the bug
Deploy an app simultaneously in two clusters, visit the k8s service of the app from an injected pod.
Version
➜  ~ istioctl version
Version: 1.0.5
GitRevision: c1707e45e71c75d74bf3a5dec8c7086f32f32fad
User: root@6f6ea1061f2b
Hub: docker.io/istio
GolangVersion: go1.10.4
BuildStatus: Clean
➜  ~ kubectl version
Client Version: version.Info{Major:"1", Minor:"11", GitVersion:"v1.11.0", GitCommit:"91e7b4fd31fcd3d5f436da26c980becec37ceefe", GitTreeState:"clean", BuildDate:"2018-06-27T22:29:25Z", GoVersion:"go1.10.3", Compiler:"gc", Platform:"darwin/amd64"}
Server Version: version.Info{Major:"1", Minor:"11", GitVersion:"v1.11.2", GitCommit:"bb9ffb1654d4a729bb4cec18ff088eacc153c239", GitTreeState:"clean", BuildDate:"2018-08-07T23:08:19Z", GoVersion:"go1.10.3", Compiler:"gc", Platform:"linux/amd64"}

Installation
Alibaba Cloud -> Istio helm chart
Environment
Alibaba Cloud
Cluster state
No need to provide

I tried to differentiate the service-key with k8s service port name, like:
port name of app svc from cluster1: http-cluster1
port name of app svc from cluster2: http-cluster2
Then only one pilot service entry is created:
  {
   "service-key": "k8s-demo.stage.svc.cluster.local|http-cluster1",
   "hosts": [
    {
     "ip_address": "<IP_ADDRESS>",
     "port": 7001
    },
    {
     "ip_address": "<IP_ADDRESS>",
     "port": 7001
    }
   ]
  }

Pods of one app from two clusters are registered in one pilot service entry as below, only half requests on the service are successful.

I see your two clusters are not of flat network.
I think this is expected, app from cluster one can not request the other cluster pod directly.

MeshNetworks may help

/cc @ymesika

I tried to differentiate the service-key with k8s service port name, like:
port name of app svc from cluster1: http-cluster1
port name of app svc from cluster2: http-cluster2

Then only one pilot service entry is created
This is also expected, if services from 2 clusters with same name, istio assumes they are one same service. So the service port from the first cluster will be used.

@hzxuzhonghu the Istio version is 1.0.5 which doesn't have the Split Horizon feature.
@xiaolanz This is normal that you see endpoints from both clusters as the instances of the same service. This is because then Envoy can (by default) round robin between local and remote endpoints.
Your issue is that in cases when the remote endpoints (IPs) are picked up it fails to connect to.
This can either be that the endpoints on your cluster2 are not accessible from cluster1 or, if you have auth on, Citadels on both clusters aren't sharing the same root CA.
If there is no flat network between the two clusters then we have solutions for this in v1.1 that redirects traffic through gateways instead of directly.

This is also expected, if services from 2 clusters with same name, istio assumes they are one same service. So the service port from the first cluster will be used.
@hzxuzhonghu So two k8s services with the same name forms one istio service, I see istio generates multi internal services for multi ports of one same k8s service. These situations are all normal, right?

@ymesika Actually we have the flat network ready. My latest test result is as below, that all four pods are accessible via the istio service.

So two k8s services with the same name forms one istio service, I see istio generates multi internal services for multi ports of one same k8s service

I am not sure what do you mean by this internal services

Pods of one app from two clusters are registered in different istio services

I think they belong to a same service as showed in above graph

I am not sure what do you mean by this internal services

I mean istio services got with {pilot}:8080/v1/registration

I think they belong to a same service as showed in above graph.

This is the current situation, that services with same name are merged. It's not what we expect.
We expect that requests are routed to pods from local cluster.

We expect that requests are routed to pods from local cluster

So I think locality based LB feature may satisfy you.

So I think locality based LB feature may satisfy you.

It produces more complexity. We want it getting done with a ready made option. Does istio have this option to disable services merging?
If not, we might distinguish services by adding a postfix to the namespaces, as the last choice.

No, pilot higher level does not care about cluster. Also one cluster may across many regions., so route to instances in same cluster is not reasonable.

we solved this by distinguish namespaces.


Allow configuring Jinja with StrictUndefined

Discussed in https://github.com/orgs/copier-org/discussions/1512

Originally posted by coretl February  9, 2024
I've been bitten a couple of times by this workflow:

Change the name of a variable in copier.yml
Forget to rename some of the references to that variable in my template
End up with that variable being rendered blank in a text file somewhere when using the template

It would appear that Jinja does this by default, and the way it suggests to raise an error is by supplying undefined=StrictUndefined in its Environment. I tried this in copier.yml:
_envops:
    undefined: StrictUndefined

But it fails because it can't turn the string into a callable:
  File "/venv/lib/python3.11/site-packages/copier/main.py", line 1024, in run_copy
    with Worker(src_path=src_path, dst_path=Path(dst_path), **kwargs) as worker:
  File "/venv/lib/python3.11/site-packages/copier/main.py", line 205, in __exit__
    raise value
  File "/venv/lib/python3.11/site-packages/copier/main.py", line 1025, in run_copy
    worker.run_copy()
  File "/venv/lib/python3.11/site-packages/copier/main.py", line 745, in run_copy
    self._ask()
  File "/venv/lib/python3.11/site-packages/copier/main.py", line 421, in _ask
    jinja_env=self.jinja_env,
              ^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.11/functools.py", line 1001, in __get__
    val = self.func(instance)
          ^^^^^^^^^^^^^^^^^^^
  File "/venv/lib/python3.11/site-packages/copier/main.py", line 500, in jinja_env
    env = SandboxedEnvironment(
          ^^^^^^^^^^^^^^^^^^^^^
  File "/venv/lib/python3.11/site-packages/jinja2/sandbox.py", line 253, in __init__
    super().__init__(*args, **kwargs)
  File "/venv/lib/python3.11/site-packages/jinja2/environment.py", line 366, in __init__
    _environment_config_check(self)
  File "/venv/lib/python3.11/site-packages/jinja2/environment.py", line 126, in _environment_config_check
    assert issubclass(
           ^^^^^^^^^^^
TypeError: issubclass() arg 1 must be a class

Is there a way to make the copier copy command fail if you use an undefined name in a template?
Related to #313 but with a different aim in mind

Would you like a PR for this?

Of course.

I'll start working on this now.

@yajo Would it be acceptable to enable StrictUndefined by default? If so, I'll create a PR.

Makes sense to me, although that'd be a breaking change. WDYT @sisp?

Let's start by adding the option, and later the default.


Manifest was depending on undefined behavior

(find-class x) where x isn't a symbol is undefined behavior.
It broke manifest for clasp - a new Common Lisp implementation.

This is issue #10, which I had actually fixed in my fork ages ago, but somehow never pushed a PR for. In any case, I endorse this PR … a mere 5 years later.


influxd not binding to all required ports on restart

Yesterday we restarted the influxd service on one of our InfluxDB boxes but when it came back up it only bound to port 8088 (meta) instead of all the configured ports; which should be:

8083 (admin)
8086 (http)
25826 (collectd)
2003 (graphite)

This was on InfluxDB 0.9.5 on CentOS 7.
The logs had the following content (nothing immediately helpful):
[wal] 2016/03/01 17:15:59 WAL writing to /var/lib/influxdb/wal/_internal/monitor/124
[wal] 2016/03/01 17:15:59 WAL starting with 30720 ready series size, 0.50 compaction threshold, and 52428800 partition size threshold
[wal] 2016/03/01 17:15:59 WAL writing to /var/lib/influxdb/wal/collectd/default/103
[wal] 2016/03/01 17:15:59 WAL starting with 30720 ready series size, 0.50 compaction threshold, and 52428800 partition size threshold
[wal] 2016/03/01 17:15:59 WAL writing to /var/lib/influxdb/wal/collectd/default/112
[wal] 2016/03/01 17:15:59 WAL starting with 30720 ready series size, 0.50 compaction threshold, and 52428800 partition size threshold
[wal] 2016/03/01 17:15:59 WAL writing to /var/lib/influxdb/wal/collectd/default/121
[wal] 2016/03/01 17:15:59 reading WAL files to flush to index
[wal] 2016/03/01 17:15:59 writing 3438 series from WAL file /var/lib/influxdb/wal/collectd/default/121/01.012614.wal to index
[tcp] 2016/03/01 17:16:01 tcp.Mux: handler not registered: 71

I killed off the process, ensure it wasn't running, checked nothing was bound to any of the ports InfluxDB needed and started it again - but we got the same issue.
I decided to start via the CLI to see if we got some info that wasn't logged (I accidently did this as root instead of the influxdb user):
# /usr/bin/influxd -config /etc/influxdb/influxdb.conf

 8888888           .d888<PHONE_NUMBER>b.  888888b.
   888            d88P"  888                   888  "Y88b 888  "88b
   888            888    888                   888    888 888  .88P
   888   88888b.  888888 888 888  888 888  888<PHONE_NUMBER>888K.
   888   888 "88b 888    888 888  888  Y8bd8P' 888    888 888  "Y88b
   888   888  888 888    888 888  888   X88K   888    888 888    888
   888   888  888 888    888 Y88b 888 .d8""8b. 888  .d88P 888   d88P
 8888888 888  888 888    888  "Y88888<PHONE_NUMBER>888P"  8888888P"

2016/03/01 17:24:45 InfluxDB starting, version 0.9.5, branch 0.9.5, commit 770b0ecf9f1f61a9d3a97136d5b6d7dd21afea1e, built 2015-11-20T15:30:37+0000
2016/03/01 17:24:45 Go version go1.4.2, GOMAXPROCS set to 2
2016/03/01 17:24:45 Using configuration at: /etc/influxdb/influxdb.conf
[metastore] 2016/03/01 17:24:45 Using data dir: /var/lib/influxdb/meta
[metastore] 2016/03/01 17:24:45 Skipping cluster join: already member of cluster: nodeId=1 raftEnabled=true peers=[localhost:8088]
[metastore] 2016/03/01 17:24:45 Node at localhost:8088 [Follower]
[metastore] 2016/03/01 17:24:46 Node at localhost:8088 [Leader]. peers=[localhost:8088]
[metastore] 2016/03/01 17:24:46 spun up monitoring for 1
[store] 2016/03/01 17:24:46 Using data dir: /var/lib/influxdb/data
[metastore] 2016/03/01 17:24:46 Updated node id=1 hostname=localhost:8088
[wal] 2016/03/01 17:24:46 WAL starting with 30720 ready series size, 0.50 compaction threshold, and 52428800 partition size threshold
[wal] 2016/03/01 17:24:46 WAL writing to /var/lib/influxdb/wal/collectd/default/103
[wal] 2016/03/01 17:24:46 WAL starting with 30720 ready series size, 0.50 compaction threshold, and 52428800 partition size threshold
[wal] 2016/03/01 17:24:46 WAL writing to /var/lib/influxdb/wal/collectd/default/112
[wal] 2016/03/01 17:24:47 WAL starting with 30720 ready series size, 0.50 compaction threshold, and 52428800 partition size threshold
[wal] 2016/03/01 17:24:47 WAL writing to /var/lib/influxdb/wal/collectd/default/121
[wal] 2016/03/01 17:24:47 reading WAL files to flush to index
[wal] 2016/03/01 17:24:47 writing 3438 series from WAL file /var/lib/influxdb/wal/collectd/default/121/01.012614.wal to index
^C

No error, it just seems to simply not start anything other than the meta part of InfluxDB.
At this point I was at a loss so I updated to the latest release (0.10.1) and restarted influxd.
Same issue but with one minor difference, we now have influxd listening on an address/port we don't have configured:
tcp        0      0 <IP_ADDRESS>:8091          <IP_ADDRESS>:*               LISTEN      54657/influxd

So the above InfluxDB instance seems to be working now, so I would assume a VERY long start-up time.
Looking at Grafana it looks like it took ~2 hours to start up and start recording data sent to it again (we have collectd pumping data in roughly once every second).

@kemra102 Can you tell us a little bit more about the data you had stored on disk when it took so long to start up?

Very little, we're still finding our feet with InfluxDB.
There was just shy of 80GB  (collected over a period of about 2.5 months) of data total. It is mostly stats from collectd running across our estate and talking to Influx's fake collectd server. We also had Icinga forwarding information it collected to Influx's fake Graphite server backend.
We haven't done anything with the format of that data before it was sent or after it was received by Influx (other than graphing it in Grafana), so it should be in the native format one would expect from those components.

@kemra102 regarding 8091 on 0.10: that port is used by a meta node to bootstrap communication with other nodes in a cluster. It's currently needed even for single node clusters.

Issue no longer relevant to current InfluxDB version


PoELadder Does Not Work with Streaming (OBS Studio)

There seems to be something abnormal you are doing which makes it so that we can't stream this. Would love to be able to.
If i use the ubiquitous OBS Window Capture, if PoELadder is started (STARTED/stop), it doesn't appear in the list of windows available to add. Perhaps related to what you do that makes it not appear in the alt+tab list. But perhaps not.
If PoELadder isn't started (start/STOPPED), i'm able to capture the window, but the PoELadder window appears solid black. So it seems clear something non-standard is being used here as well, even with just the PoELadder program simply opened.
If i click START in the latter scenario, the window promptly disappears from the stream.

There are a couple of factors contributing to this issue.
I'm not able to make the window visible to OBS when the tracker is started without causing the taskbar to be always on top. (See this microsoft community thread) To address this, I'm launching the window in toolbar mode, which has the unfortunate side-effect of making the window invisible to obs (and likely many other programs as well).
Second, OBS cannot display chrome windows that have hardware acceleration enabled. I cannot disable hardware acceleration and also have a transparent window as far as I'm aware, which causes the solid black appearance of the window in OBS.
In the meantime, you could always use the display capture source, and crop the window (alt+drag the scene edges) as a work around. If Microsoft manages to fix the issue I referred to above, I'll explore some more options related to the second.
Sorry I can't give a better answer to this right now!


Allow Attach and Clear Script on multiple nodes

See https://github.com/godotengine/godot/issues/17089
Not sure if this makes sense, but an experiment to allow Attach Script and Clear Script on multi selections:

I moved the menu items lower than the single selection ones, since I think the multi selection add and clear variants can be dangerous and should not be topmost.
Clearing clears scripts on all selected nodes.
Attaching scripts takes the presets from the first selected script, but if you create or load a script in the create/load dialog, it then gets attached to all selected nodes - you need to know what you're doing obviously (in terms of class hierarchies).
I didn't touch the attach/clear script button, which still only operates on single selections.

Added undo/redo support, so Clear and Attach Script are now undoable.

Does this really have a use case? I expect that if you need to add several nodes with the same script, you'll likely make them a scene and instance them, instead of doing it all manually.
The multiedit for clearing scripts might be slightly more useful, but even then, I'm not sure when one would use that.
Adding undoredo is nice though.

From a UX standpoint, couldn't this actually be harmful? If you end up doing this often, it could mean you're not using Godot's node system (and instancing) as it is intended?
I.e. whenever you have multiple of something and you want to clear or attach the same script, shouldn't those things really have been instances of a scene where you do so?

@mhilbrunner There are border cases, where having instances is not an option, at least with the current way the editors work; for example, if you want to use and edit various Polygon2Ds that are all customized using a script, you wouldn't want to hide them in an instance, since you can then no longer edit their vertices. I cannot think of another object type where this would be the case though.

@poke1024 Fair enough, didn't think of that :)

It doesn't make the code too complex and the added UndoRedo is nice, so why not :)

@poke1024 but you can still customize their vertices and then just run a script that loops through them in _ready

@girng Yes, I had a specific use case with a tool script in mind (I have one project where I use a custom polygon workflow in the editor that reshapes polygons into a game asset).


fix https://github.com/mattn/anko/issues/267

Codecov Report

Merging #270 into master will increase coverage by 0.04%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #270      +/-   ##
==========================================
+ Coverage   82.11%   82.15%   +0.04%
==========================================
  Files           8        8
  Lines        1895     1872      -23
==========================================
- Hits         1556     1538      -18
+ Misses        254      249       -5
  Partials       85       85

Impacted Files
Coverage Δ

vm/env.go
100% <ø> (ø)
:arrow_up:

vm/vmStmt.go
80.7% <ø> (+0.76%)
: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 6707570...2388851. Read the comment docs.

LGTM

Thank you


can FIO be run on windows ?

we would want to test FIO on windows, perfkit supporting that?

@ehankland
@noahl
Not as of yet.  You could implement it pretty quickly by dupping the Windows ntttcp benchmark and going from there.
I think however Noah has this on his list of things to do.

I've seen some work that on this.
https://github.com/GoogleCloudPlatform/PerfKitBenchmarker/tree/master/perfkitbenchmarker/windows_benchmarks

I'm working on this right now. My branch is https://github.com/GoogleCloudPlatform/PerfKitBenchmarker/tree/fio-on-windows.


Add OrganisationType for "Court"

As per Pivotal story #92038070, adds a new organisation type for "Court", and also ensures organisations of this type don't appear on GOV.UK anywhere. Future work will add pages for this type, and also index them and add them to content store/registry in the correct place.
Possible contentious point: adds a non_governmental flag for Organisation types. It seems this would be useful for any number of [qua]ngos that might be added to the site in the future.

Would you be able to explain what the definition of non_governmental will be please? In my mind there are other organisation types which should also be marked as non_governmental like Public Corporations which are also a type of organisation.

@edds that's a good point. In this case, it's basically "excluded from the organisations list". The case for this is that we really don't want anyone to infer that Courts are part of the Government.
Perhaps there needs to be different terminology for this. exclude_from_government instead? exclude_from_organisation_namespace? Or perhaps an explicit namespace option where you specify organisations or courts or similar?

Is there a reason why these are a type of Organisation and not an entirely separate model?

@boffbowsh I like the namespace idea - I think in the future having the ability to filter the organisations list would be useful. We'd just have to make sure that courts never leak into the government API.

The hierarchical model of Organisations seemed like a good fit, as they will need to be e.g. "administered by HM Courts and Tribunals Service", and also we will in future have things like the Administrative Court which is part of Queen's Bench Division, which is part of the High Court, which is administered by HMCTS. Plus we then get all the admin/publishing/indexing etc functionality for free

So @bradleywright I'll add namespace data to Organisation Types, and then filter on that as government or courts? I think organisations is actually too broad, in retrospect.

The problem with using the word 'government' is that there are organisations which will specially say they are not government (like for example Ofcom or Planning Inspectorate). There really needs to be a very clear definition of how you are expecting that namespace to be used (preferably within a nomenclature section of the project Readme).

Maybe default and courts for now, and then we can make them more specific in the future as required?

Yeah, I'm not so sure about the namespace thing. To keep things simple for the time being and do just enough to give us what we want, I reckon a excluding_courts scope would be enough.

I'm happy with that, @tekin. Any others have thoughts?

I am a little bit concerned that we're adding these as a type of Organisation. Although we do get all the admin interface, etc, for free, my concern is that they're different enough from the other types of organisations that we'll to end up with lots of conditional logic to restrict/add the court-specific features. Sounds like they'll live under a different path ("/government/courts" not "/government/organisations") and will also be a different format in the new Publishing 2.0 stack ("court" not "organisation"). Feels like there's the potential to create the same sorts of problems that we've encountered because of the overloading of the Edition model. Thoughts @heathd?

Reading the background stories in pivotal, it feels more the case that they should be modelled as a separate model rather than a type of Organisation. They don't need "contacts" (that's what the , "features", we don't want documents tagged to them, nor ministers. They don't have any feeds or email alerts and their corporate information won't be the same as for organisations.

I'm struggling to form a view on this, there's so much context to load. I can see both sides of the argument. I agree that there's a balance to be struck which depends on how well courts fit with the organisation model.
From what I can see there has been a fair bit of background research gone into evaluating different possible approaches. Was the stand-alone model approach considered and rejected?

So I think this work is lightweight enough as it is to carry on adding it to the Organisation model. Changing the scopes to excluding_courts and courts_only is a good alternative to correctly defining the non-governmental language and namespaces.
There will be a new CourtsController and associated templates. When Organisations are ported to a new publishing model, it will require a relatively small amount of additional effort to port Courts as well.
This seems to be the most pragmatic solution based on HMCTS' requirements.

Happy for the ultimate decision to fall with your team who will have the full context around the feature and also whitehall's history. I just wanted to ask the question as it does feel like it has the potential to tie us in knots in the future by overloading the Organisation model, especially if a "court" is relatively simple and doesn't need the behaviour of an organisation.

This is now ready for review again, with the removal of "non-governmental" and scoping based just on whether it's a Court or not.

:+1: Looks good


feat(prettier): keep empty lines in SwitchStatement

Current dependencies on/for this PR:

main

PR #1531

PR #1532   👈

This stack of pull requests is managed by Graphite.

Merge activity

Nov 24, 4:05 AM: @Boshen started a stack merge that includes this pull request via Graphite.


feat(axis): Allow hiding tick notches and styling tick texts

Moved the text tick labeler to be configurable and added param
to hide notches.

Great! More customization is definitely desired -- hoping to have some sort of a unified way of customizing them in the future but this fits in nicely right now.


[Tree] Add support to nested sets and closure tree strategies

Now tree behavior support only materialized path.

I'm going through the old issues/PRs and closing most of them so we can focus on now.
This one is missing any feedback or reasoning so I cannot deduce anything from it.
Thanks for you though, but closing for reasons above.
If the bug still remains, please send failing test case to verify it. Only with test we'll be albe to help.


Editor window is scaled using nearest neighbour when moving window to lower DPI monitor

Tested versions

4.2.1
4.3.dev4
4.3.dev5

System information
MacOS Sonoma 14.3.1, Macbook Pro M2 Max
Issue description
I have an ultra-wide 3440x1440 monitor connected to my M2 Max Mac Pro. When I move the editor window to the ultra wide which has a lower DPI it resizes the window using nearest neighbour which aliases horribly. This behaviour does not occur in 4.2.1. I've attached a video to demonstrate.
https://github.com/godotengine/godot/assets/7454145/2a8249a6-e397-4fdc-96d8-3e6461d48727
Steps to reproduce

Connect a monitor with a lower DPI than the primary.
Move the editor window between the two monitors.

Minimal reproduction project (MRP)
N/A

This is still present in 4.3.dev6

Thanks for the report.
This sounds similar to other existing issues but I couldn't find them with a quick search, maybe @bruvzg would know more.

Thanks for the report! Consolidating in https://github.com/godotengine/godot/issues/89019.


bruno: 1.18.0 -> 1.18.1

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for bruno is: Open-source IDE For exploring and testing APIs.
meta.homepage for bruno is: https://www.usebruno.com
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 1.18.1 in filename of file in /nix/store/69zjxamk2jylxp7fhgzlsbsjm9m40h2q-bruno-1.18.1

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

2 total rebuild path(s)

2 package rebuild(s)

First fifty rebuilds by attrpath

bruno

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/69zjxamk2jylxp7fhgzlsbsjm9m40h2q-bruno-1.18.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 bruno https://github.com/r-ryantm/nixpkgs/archive/997fb6d35169552a2f0e4f4df11fa2875396b040.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/997fb6d35169552a2f0e4f4df11fa2875396b040#bruno

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/69zjxamk2jylxp7fhgzlsbsjm9m40h2q-bruno-1.18.1
ls -la /nix/store/69zjxamk2jylxp7fhgzlsbsjm9m40h2q-bruno-1.18.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 --extra-nixpkgs-config '{ allowInsecurePredicate = x: true; }' run on x86_64-linux 1

1 package built:

bruno

Maintainer pings
cc @gepbird @kashw2 @lucasew @mattpolzin @water-sucks for testing.

@NixOS/nixpkgs-merge-bot merge