Add option import_dangerfile(url:) to use with custom repositories and servers

Use Case
We are importing Dangerfile from our self-hosted Bitbucket repository, with custom url:
https://bitbucket.our-company.com/projects/PROJECT-NAME/repos/our-danger-rules/browse/Dangerfile?raw
Currently it's supported with import_plugin but not with import_dangerfile.
This PR adds url option to import_dangerfile.

Cool, yeah, this looks good to me - nice work on the docs 👍🏻


listenActiveWindow sends notification upon mouse move in current active window

listenActiveWindow should only send a notification when the focused window changes, not when your mouse moves within the active window.
https://user-images.githubusercontent.com/83777472/117330911-2d01fe00-ae64-11eb-8597-76508aef1793.mov

@kdouglass-oliveai - No this is not intended behavior - I will bring this one back to the team and create a new defect for this.
Thanks for reporting!

@kdouglass-oliveai , after a lot of internal debate we've decided not to suppress these events b/c there may be use cases for accessing it. Instead, we'll add some documentation that explains that some apps will generate multiple windows, and that Loop authors should consider the windows changed to/from.

Will be fixed in #167

Closing with documentation update


Remove new file menu item

I also fixed some comments and moved some duplicate code into functions.

Thanks!


Generating “Biochemical activity” annotations for BIOGRID and using biochemical activity for MF has_substrate annotations

This is low priority but we could generate BioGRID annotations for the GO MF with substrate data,
and use the BioGRID biochemical activity interactions to inform out MF_> has_direct input annotations.
Need to think about this more but wanted to remember so opening a ticket now
Original comment by: ValWood

We need to look at this at some point.
I think it seems off that we display Biogrid "biochemical activity" annotations in the physical interaction section, but we don't allow their curation.
I think maybe we should not display them (but we should do a check to make sure we aren't missing valid modifications).

We need to look at this at some point.
I think it seems off that we display Biogrid "biochemical activity" annotations in the physical interaction section, but we don't allow their curation.
I think maybe we should not display them (but we should do a check to make sure we aren't missing valid modifications).


WAD Library is not being recognized in RIDE

I installed and configured WinAppDriver on my PC:

Download Windows Application Driver installer
Run the installer on a Windows 10 machine
Run WinAppDriver.exe from the installation directory

After the installation (pip install robotframework-wadlibrary) I tried to add WAD library in RIDE (IDE for Robot Framework), but seems is not been recognized.

HI,
Is this how the library look like in Ride?

pip will install wadlibrary into your \Python...\Lib\site-packages in my case C:\Python37\Lib\site-packages\WADLibrary
and this site-packages  should be in path ! - of course.
is this help?
Best Regards,
Robert


Multi-platform build failing for ruby:2.7.3-alpine3.13

The base image we're using doesn't have all the platforms we need.
The root cause is https://github.com/docker-library/ruby/issues/308.
There are a few options available:

Wait for a fix. I'd rather not do this, because that issue is still open after almost a year.
Move to a different platform (Ubuntu?) for the base image.
Update to a Ruby 3.x Alpine image.

Anything else?

We got only #34 to request support besides x86_64, so I'll remove the other platforms for now, as those should be available


Disable usersnap for mobile visitors

It's not obvious what is is and I don't think there's a way to customize it.

@madprime do we want to keep usersnap around at all? It seems to me that in the last two months only one person - @danamlews - reported something through it, and she has plenty of ways to get in touch with us already 😂
For many other users it's just confusing (from my non-scientific study done by shoulder surfing ppl who encounter the website for the first time)


Failure to verify macos_user resource with new ChefDK: "found more than one group with the same name"

2018-03-15T00:09:55.9858670Z      found more than one group with the same name, please use `groups` resource```

Duplicate of #64


Getting more than 100 tracks via getPlaylist

Hi there,
Apologies if this isn't the correct place to ask.
I'm trying to get all track from a playlist and the playlist has more than 100 tracks within. Using the code below, I am able to get the first 100 tracks, but despite my efforts I am unable to figure out how to get the next 100 (or increase the limit). Could you tell me how to get all tracks, even if there are, say, thousands?
public void TracksFromSpotifyPlaylist(final String playlistID) {

	api.setAccessToken(mToken);
	SpotifyService spotify = api.getService();

	spotify.getPlaylist(mUserID, playlistID, new Callback<Playlist>() {

		@Override
		public void success(Playlist playlist, retrofit.client.Response response) {

			QueueManager.spotifyPlaylistTracks.clear();

			//Only gets the first 100:
			for (PlaylistTrack track : playlist.tracks.items) {
				MusicTrack newTrack = new MusicTrack(track);
				QueueManager.spotifyPlaylistTracks.add(newTrack);
			}
		}

		@Override
		public void failure(RetrofitError error) {
			Log.i("Error", error.toString());
		}
	});
}

+1

@TalonGrayson
If you use the
getPlaylist(String trackIds, Map<String,Object> options, retrofit.Callback<Tracks> callback)

call instead, you can feed in options with a map. You can edit your code like this (probably)
Map<String, Object> options = new Map<String, Object>();
options.put(SpotifyService.OFFSET, 0)
options.put(SpotifyService.LIMIT, 2000)
spotifyService.getPlaylist(userId, playlistId, options, new Callback<Playlist>() {
.......

And get litterally (two) tousands of tracks. Probably a good idea to limit it to do several calls and not set the limit too high.

Hi there,
Just wanted to reply to say thank you for the info. I haven't had a chance to try this out yet as I have barely even been at my PC over the last couple of weeks.
I will report back once I have tried out and hopefully we can close this :)

If you'd like to fetch all tracks in the playlist you should use getPlaylistTracks method that allows to
request specific subset of tracks using offset and limit parameters.
Unfortunately setting limit to 2000 won't work because max size of the page is 100 (defined in the endpoint docs).
The response of this call is a paging object that has the total field that will tell you how many tracks the playlist has as well as offset and limit fields that will tell you where in the track set you are.
So for a playlists with 268 tracks a simplified example could look like this:
Map<String, Object> options = new Map<String, Object>();
options.put(SpotifyService.LIMIT, 100);

// get tracks 0-99
options.put(SpotifyService.OFFSET, 0);
Pager<PlaylistTrack> response = spotifyService.getPlaylistTracks(userId, playlistId, options);

// get tracks 100-199
options.put(SpotifyService.OFFSET, 100);
Pager<PlaylistTrack> response = spotifyService.getPlaylistTracks(userId, playlistId, options);

// get tracks 200-268
options.put(SpotifyService.OFFSET, 200);
Pager<PlaylistTrack> response = spotifyService.getPlaylistTracks(userId, playlistId, options);

Where you can keep querying as long as offset + limit < total


FileSystemException: FileSystemException: Directory listing failed

I'm going to embed a new flutter app to an exsiting app, https://github.com/flutter/flutter/wiki/Add-Flutter-to-existing-apps
the last step，something looks wrong, this is crash log
macOS Mojave 10.14 Beta (18A336e)
Flutter crash report; please file at https://github.com/flutter/flutter/issues.
command
flutter run --use-application-binary /Users/parkinwu/outofwork/flutter/embedder/build/ios/Debug-iphoneos/embedder.app -d iPhone X
exception
FileSystemException: FileSystemException: Directory listing failed, path = '/var/folders/gh/9m3fw8sx7jv9wglkjdpyx1bc0000gn/T/flutter_app_iXRxJ3/Payload/' (OS Error: No such file or directory, errno = 2)
#0      _Directory._fillWithDirectoryListing (dart:io-patch/directory_patch.dart:37)
#1      _Directory.listSync (dart:io/directory_impl.dart:250)
#2      LocalFileSystemEntity&ForwardingDirectory.listSync (package:file/src/forwarding/forwarding_directory.dart:45)
#3      new IOSApp.fromIpa (package:flutter_tools/src/application_package.dart:158)
#4      getApplicationPackageForPlatform (package:flutter_tools/src/application_package.dart:277)
<asynchronous suspension>
#5      FlutterDevice.runHot (package:flutter_tools/src/resident_runner.dart:269)
<asynchronous suspension>
#6      HotRunner.run (package:flutter_tools/src/run_hot.dart:216)
<asynchronous suspension>
#7      RunCommand.runCommand (package:flutter_tools/src/commands/run.dart:395)
<asynchronous suspension>
#8      FlutterCommand.verifyThenRunCommand (package:flutter_tools/src/runner/flutter_command.dart:344)
<asynchronous suspension>
#9      FlutterCommand.run.<anonymous closure> (package:flutter_tools/src/runner/flutter_command.dart:279)
<asynchronous suspension>
#10     AppContext.run.<anonymous closure> (package:flutter_tools/src/base/context.dart:142)
<asynchronous suspension>
#11     _rootRun (dart:async/zone.dart:1126)
#12     _CustomZone.run (dart:async/zone.dart:1023)
#13     runZoned (dart:async/zone.dart:1501)
#14     AppContext.run (package:flutter_tools/src/base/context.dart:141)
<asynchronous suspension>
#15     FlutterCommand.run (package:flutter_tools/src/runner/flutter_command.dart:270)
#16     CommandRunner.runCommand (package:args/command_runner.dart:194)
<asynchronous suspension>
#17     FlutterCommandRunner.runCommand.<anonymous closure> (package:flutter_tools/src/runner/flutter_command_runner.dart:309)
<asynchronous suspension>
#18     AppContext.run.<anonymous closure> (package:flutter_tools/src/base/context.dart:142)
<asynchronous suspension>
#19     _rootRun (dart:async/zone.dart:1126)
#20     _CustomZone.run (dart:async/zone.dart:1023)
#21     runZoned (dart:async/zone.dart:1501)
#22     AppContext.run (package:flutter_tools/src/base/context.dart:141)
<asynchronous suspension>
#23     FlutterCommandRunner.runCommand (package:flutter_tools/src/runner/flutter_command_runner.dart:265)
<asynchronous suspension>
#24     CommandRunner.run.<anonymous closure> (package:args/command_runner.dart:109)
#25     new Future.sync (dart:async/future.dart:222)
#26     CommandRunner.run (package:args/command_runner.dart:109)
#27     FlutterCommandRunner.run (package:flutter_tools/src/runner/flutter_command_runner.dart:174)
#28     run.<anonymous closure> (package:flutter_tools/runner.dart:59)
<asynchronous suspension>
#29     AppContext.run.<anonymous closure> (package:flutter_tools/src/base/context.dart:142)
<asynchronous suspension>
#30     _rootRun (dart:async/zone.dart:1126)
#31     _CustomZone.run (dart:async/zone.dart:1023)
#32     runZoned (dart:async/zone.dart:1501)
#33     AppContext.run (package:flutter_tools/src/base/context.dart:141)
<asynchronous suspension>
#34     runInContext (package:flutter_tools/src/context_runner.dart:43)
<asynchronous suspension>
#35     run (package:flutter_tools/runner.dart:50)
#36     main (package:flutter_tools/executable.dart:49)
<asynchronous suspension>
#37     main (file:///b/build/slave/Mac_Flutter_Packaging/build/archive/flutter/packages/flutter_tools/bin/flutter_tools.dart:8)
#38     _startIsolate.<anonymous closure> (dart:isolate-patch/dart:isolate/isolate_patch.dart:277)
#39     _RawReceivePortImpl._handleMessage (dart:isolate-patch/dart:isolate/isolate_patch.dart:165)

flutter doctor
[✓] Flutter (Channel beta, v0.5.1, on Mac OS X 10.14 18A336e, locale zh-Hans-CN)
    • Flutter version 0.5.1 at /Users/parkinwu/Documents/flutter
    • Framework revision c7ea3ca377 (7 weeks ago), 2018-05-29 21:07:33 +0200
    • Engine revision 1ed25ca7b7
    • Dart version 2.0.0-dev.58.0.flutter-f981f09760

[✗] Android toolchain - develop for Android devices
    ✗ Unable to locate Android SDK.
      Install Android Studio from: https://developer.android.com/studio/index.html
      On first launch it will assist you in installing the Android SDK components.
      (or visit https://flutter.io/setup/#android-setup for detailed instructions).
      If Android SDK has been installed to a custom location, set $ANDROID_HOME to that location.

[✓] iOS toolchain - develop for iOS devices (Xcode 10.0)
    • Xcode at /Applications/Xcode-beta.app/Contents/Developer
    • Xcode 10.0, Build version 10L201y
    • ios-deploy 1.9.2
    • CocoaPods version 1.5.3

[✗] Android Studio (not installed)
    • Android Studio not found; download from https://developer.android.com/studio/index.html
      (or visit https://flutter.io/setup/#android-setup for detailed instructions).

[✓] IntelliJ IDEA Community Edition (version 2018.1.6)
    • IntelliJ at /Applications/IntelliJ IDEA CE.app
    • Flutter plugin version 26.0.2
    • Dart plugin version 181.4892.1

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

[✓] Connected devices (2 available)
    • iPhone 6s • CA2E2711-70E8-4EF0-9F5C-EE0622D00472 • ios • com.apple.CoreSimulator.SimRuntime.iOS-11-4 (simulator)
    • iPhone X  • B6D3CAA2-F32E-4BE0-B256-FC81DF09FD77 • ios • com.apple.CoreSimulator.SimRuntime.iOS-11-4 (simulator)

! Doctor found issues in 3 categories.

me too

Could be a dup of #21297

Is this only on Android or only iOS or both?
Is this only in release or also in debug mode?

me too
@zoechi android and iOS and every time in release mode, in debug mode only sometimes

@ParkinWu @BigAlKo would either of you mind trying this again to see if the problem still exists?
Also, would you please provide the relevant logs from both the iOS and Android attempts?
Additionally, please describe the general structure of the app that you're adding to.  Is the project configuration customized at all?  Does the initial launching ViewController/Activity contain Flutter content? etc.
I've been using the add2app instructions for Android for a couple months and have never had any issue like this in debug or release mode.

We can't reproduce this.  If you're still seeing this, please reopen this with more context, as @matthew-carroll  mentioned above.  Thanks!


Allow removing the squirrel.

Pre-flight checklist

[X] I have read the contribution documentation for this project.
[X] I agree to follow the code of conduct that this project uses.
[X] I have searched the issue tracker for a feature request that matches the one I want to file, without success.

Problem description
Allow to remove the squirrel.

@electron-forge/maker-squirrel (makers new MakerSquirrel({}))
'electron-squirrel-startup'

Currently build is not possible anymore.
electron-forge make

Error:
An unhandled rejection has occurred inside Forge:
Error: Could not find any make targets configured for the "win32" platform.

Proposed solution
Allow removing the squirrel.
Additional information
Maybe I do not fully understand what Squirrel.Windows is. docs.

To make proper distributables, please use the Make command.

Maybe explain it a bit better. I wonder why is there an additional exe file for squirrel in the build. I would prefer to have a simple portable app without setup. Or a simple lightweight setup. And why if-require(squirrel-startup): app.quit()? Is make only for squirrel.windows? Or what does it else? What does the squirrel exe? Is there an alternative setup system? In best case for portable variant without installing.

Hi @twoco,
Thanks for the question. I think this could be explained better within the documentation, but there are a few parts to this question:

Allow to remove the squirrel.

Building a Squirrel.Windows target is not mandatory, but removing it outright means that you don't have any build targets left for "win32", which will be the default target platform if you're using a Windows machine.

Is there an alternative setup system?

There are first-party installers for WiX MSI and AppX (Windows Store) as well.

I wonder why is there an additional exe file for squirrel in the build. I would prefer to have a simple portable app without setup.

What's your use case for the single-file portable exe? Squirrel.Windows effectively acts as a one-click installer and shouldn't require any user intervention (electron-squirrel-startup should handle most of the user setup for you).

Ok, thanks for info. I just wondered if "make" is for production builds and "package" development. I would also suggest to change the template about that (prod / dev) e.g. don't "openDevTools" and hide menu strip in prod mode. I'm new in Electron and sure if there is a production mode. If I understand correctly. The "package" command creates the production app and "make" just wraps it in a setup?


Suggestion :  Validator on skill fields (max 5)

Place a validator on the fields in the Skills (max of 5 in book, but able to place any amount we want)

No max apparently

Minimum does default to 0 in the calculation, but the negative value is kept in the field.

You're quite right. This dates back to when I switched the inputs from number type to text (as the numbers were a bit twitchy and buggy and made scrolling difficult etc).. meant to add data validation to make up for the lack of min-max functionality but completely forgot about it.
Good spot!

Pushed a fix to the dev branch for this.  Without modifying the rendering code added a onchange event handler to check value before saving and resetting to default value.


Cannot parse dict comprehension with negative number

Operating system: MacOS 10.12.6
Python version: 3.6
Black version: 18.3a3
Does also happen on master: yes
{a: b * -2 for a, b in dictionary.items()}
becomes
{a: b * -2for a, b in dictionary.items()}
which, surprisingly to me, doesn't break, but is clearly wrong.
{a: b * 2 for a, b in dictionary.items()}
is not affected by this issue (black leaves it alone.)

Thanks for your report! This is now fixed. ✨ 🍰 ✨


Document CSS selectors when using shallow mounting

I was a little confused by this today, and realised that selectors don't apply to rendering of child components. As far as I can see, this isn't currently documented in the selectors docs.
I propose adding a note to the docs about this. Alternatively, we could add the note to .find on https://github.com/airbnb/enzyme/blob/master/docs/api/ShallowWrapper/find.md instead. Let me know what you think.

We can add this if other contributors feel it necessary, but this is kind of the sole purpose of shallow rendering - to render shallowly, ie, not to render child components.

hmm yes, I agree with @ljharb this is the whole point of the shallow renderer, if this is confusing we should instead better explain the difference between shallow and mount

I agree, if anything we should clarify shallow rendering elsewhere, or link to resources that explain it. I don't think the selectors API doc is the place to do that, but we appreciate the PR @Wilfred!


Publish on your homepage

Please consider publishing this repo on your github homepage too, since it's a quite usefule repo I think.

Fixed https://github.com/roflmaostc/felixwechsler.science/commits/master


Helidon Web Starter generates only Helidon SE projects

https://helidon.io/starter/3.0.1?flavor=mp&step=1&app-type=custom
generates Helidon SE although MP is selected!

We track issues related to the website internally.
Seems to be a UI bug. If you see this issue again, copy the download link and make sure it matches the selected options.
Note the URL below (/api/starter)
curl -o /tmp/project.zip "https://helidon.io/api/starter/3.0.1/generate?flavor=mp&app-type=custom" && \
unzip -p /tmp/project.zip myproject/pom.xml | \
grep '<artifactId>helidon-' | head -1


If Workspaces window is narrow, buttons become misaligned

Need to minwidth name field

Not anymore now that URL is gone


The profile output for awk is hard to parse

The output of profile is a timeline along the y axis with a field per process in the job along the x axis.  If a process is not active at a given time then nothing is printed (empty field).  For csv we thus get yyyy-mm-hh hh:mm,,,x,y,,, and so on.    As a special case, awk interprets a field separator of a single space as "any string of whitespace" (https://www.gnu.org/software/gawk/manual/gawk.html#Whitespace-Normally-Separates-Fields).  Therefore, for awk output in the profile we should probably print something, eg, '.' would be fine.

Isn't this a problem for awk output in every case?  I guess very few fields can have blank values.


UI: Enhance Users’ Page with Table View and Pagination

💡 Summary
The current Users’ page lacks a user-friendly interface. By implementing the following enhancements, we can significantly improve the user experience:

Table View: Display all Users in a tabular format, similar to the new Organizations lists.
Pagination: Enable pagination to efficiently handle large user lists.
Search Functionality: Add the ability to search the user list.
Alphabetical Selection: Allow users to select users alphabetically.
Move “Create User” Functionality: Relocate the “Create User” functionality to the top of the page for better visibility.

This Issue proceeds https://github.com/cisagov/crossfeed/issues/2549 due to a change in repositories.
Implementation notes

Mimic all UI List features from the Organizations Lists.

Acceptance criteria
How do we know when this work is done?

[ ] Commit to Integration Branch
[ ] Demonstrate a Response Design
[ ] Include all MUI breakpoints
[ ] Include Screenshots Of Updated Views
[ ] Commit to Staging
[ ] Create Functional Tests

https://github.com/cisagov/crossfeed/issues/2549


Make types accessible for other modules

This makes it possible to easily mock the client. At the moment that is a bit cumbersome as the types are internal.
I'm working to support gandi as a provider in https://github.com/kubernetes-sigs/external-dns. As some functions return the StandardResponse defined in internal I had to copy those structures to be able to mock the LiveDNS client.
See: https://github.com/packi/external-dns/blob/gandi-provider/provider/gandi/client.go
I don't know if this is the golang way to do things but I vaguely remember seeing other modules do something similar.

Original author here....
I don't use this lib currently, so I'm not confident in accepting the PR without checking with recent contributors.
I think these types must be outside of internal if they are returned by functions outside of internal...
@thommay ?

@robbyoconnor you are also working on types on #22, could you please take a look on this one?


client: Add DNSServers field to ContainerSpec

This is used to allow created containers to use a specific set of DNS servers, rather than relying on the hosts's /etc/resolv.conf.  Useful in configurations where the host DNS server is inappropriate (e.g. localhost without being exposed on other interfaces).
See also cloudfoundry-incubator/garden-linux#60.
This will have matching PRs in guardian and guardian-release, followed by a second PR in the garden family to implement the same.

Looking again, this change isn't actually needed :confused: (The ones for guardian and guardian-release are, though)
Sorry about that, closing.


combine c2 and communication namespaces

Does it make sense to combine the c2 and communication namespaces so they show up under one common parent?
c2 - commands that may be issued by a controller, such as interactive shell or file transfer
communication - HTTP, TCP, etc.
IMO c2 falls under communication.
This is for example relevant when filtering by namespace.

I think moving c2 under communication makes sense. The c2 commands are essentially another layer of protocol, just like IP or TCP.
With only two c2 categories I don't think this is a big change that we should over-think.


After standard deploy, shipyard can't communicate with rethinkdb

Is this caused by pulling in the latest version of 'rethinkdb' instead of a known-good-version? Is there anything I can do about it? Was that image launched as-is or are there specific command parameters I have to pass? Does anyone know the last-known good version?
Basically, I'd like to be able to piece together a docker command to get a known-good-version of 'rethinkdb' running in the stack. Is it just:
$> docker run \
    -ti \
    -d \
    --restart=always \
    --name shipyard-rethinkdb \
    rethinkdb:known-good-version

Installation logs:
kelly@server:~$ docker -v
Docker version 1.11.1, build 5604cbe
kelly@server:~$ docker ps -a
CONTAINER ID        IMAGE               COMMAND             CREATED             STATUS              PORTS               NAMES
kelly@server:~$ curl -sSL https://shipyard-project.com/deploy | bash -s
Deploying Shipyard
 -> Starting Database
 -> Starting Discovery
 -> Starting Cert Volume
 -> Starting Proxy
 -> Starting Swarm Manager
 -> Starting Swarm Agent
 -> Starting Controller
Waiting for Shipyard on <IP_ADDRESS>:8080
.....................................................^C
kelly@server:~$ docker ps -a
CONTAINER ID        IMAGE                          COMMAND                  CREATED             STATUS                          PORTS                                            NAMES
e78e27b45353        shipyard/shipyard:latest       "/bin/controller --de"   2 minutes ago       Restarting (1) 30 seconds ago   <IP_ADDRESS>:8080->8080/tcp                           shipyard-controller
41776e754216        swarm:latest                   "/swarm j --addr 192."   2 minutes ago       Up 2 minutes                    2375/tcp                                         shipyard-swarm-agent
5c8d4c1c4a02        swarm:latest                   "/swarm m --replicati"   2 minutes ago       Up 2 minutes                    2375/tcp                                         shipyard-swarm-manager
903bb236ec33        shipyard/docker-proxy:latest   "/usr/local/bin/run"     2 minutes ago       Up 2 minutes                    <IP_ADDRESS>:2375->2375/tcp                           shipyard-proxy
8d2e06081458        alpine                         "sh"                     2 minutes ago       Up 2 minutes                                                                     shipyard-certs
e1696d4d2274        microbox/etcd:latest           "/bin/etcd -addr 192."   2 minutes ago       Up 2 minutes                    <IP_ADDRESS>:4001->4001/tcp, <IP_ADDRESS>:7001->7001/tcp   shipyard-discovery
bc1a4698ebc7        rethinkdb                      "rethinkdb --bind all"   2 minutes ago       Up 2 minutes                    8080/tcp, 28015/tcp, 29015/tcp                   shipyard-rethinkdb
kelly@server:~$ docker logs e78e27b45353
INFO[0000] shipyard version 3.0.4
DEBU[0000] setting up rethinkdb session
WARN[0000] Error creating connection: gorethink: Received an unsupported protocol version. This port is for RethinkDB queries. Does your client driver version not match the server?

FATA[0000] no connections were made when creating the session
INFO[0000] shipyard version 3.0.4
DEBU[0000] setting up rethinkdb session
WARN[0000] Error creating connection: gorethink: Received an unsupported protocol version. This port is for RethinkDB queries. Does your client driver version not match the server?

FATA[0000] no connections were made when creating the session

Upon further looking, it might not be the rethinkdb version.
kelly@server:~$ docker images
REPOSITORY                TAG                 IMAGE ID            CREATED             SIZE
shipyard/shipyard         latest              ba426f0944bc        5 days ago          58.92 MB
alpine                    latest              14f89d0e6257        3 months ago        4.79 MB
rethinkdb                 latest              7d4194e13b7d        3 months ago        181.8 MB
shipyard/docker-proxy     latest              cfee14e5d6f2        4 months ago        9.464 MB
swarm                     latest              32d67c5a4211        4 months ago        17.15 MB
microbox/etcd             latest              6aef84b9ec5a        9 months ago        17.86 MB

Only the shipyard/shipyard image seems to have been updated with the most recent deploy. All the other images are the same as the previous (working) deploy.

There's something wrong with 3.0.4.
Logs of 3.0.3 working:
kelly@server:~$ curl -sSL https://shipyard-project.com/deploy | IMAGE=shipyard/shipyard:3.0.3 bash -s
Deploying Shipyard
 -> Starting Database
 -> Starting Discovery
 -> Starting Cert Volume
 -> Starting Proxy
 -> Starting Swarm Manager
 -> Starting Swarm Agent
 -> Starting Controller
Waiting for Shipyard on <IP_ADDRESS>:8080
..............
Shipyard available at http://<IP_ADDRESS>:8080
Username: admin Password: shipyard

I went to examine the code changes between 3.0.3 and 3.0.4. Unfortunately, there are no tagged releases for those. I don't really feel like examining the 70 to master since 3.0.0. Sorry I can't be of more help.

Hi, I make updgrade and unable to get shipyard web part running anymore?
FATA[0000] no connections were made when creating the session
INFO[0000] shipyard version 3.0.5
DEBU[0000] setting up rethinkdb session
WARN[0000] Error creating connection: gorethink: Received an unsupported protocol version. This port is for RethinkDB queries. Does your client driver version not match the server?
FATA[0000] no connections were made when creating the session
I also make tryed fully to remove and redeploy but error is same.

I do have the same problem.

I had the same issue. To solve it, I removed all shipyard and rethinkdb containers and images. After that i made a clean install with the automated deployment method.


chore(install): move E2E cleanup to its own workflow to run separately

@NRhzhao and @mbruzina if you could double check this PR that would be great. It would make our PR and release a bit faster.

Everything looks fine, but I am not too familiar with github action, so not sure if this can be tested?


Maximum Gap Problem

Is your feature request related to a problem? Please describe.
The question is :
Given an integer array nums, return the maximum difference between two
successive elements in its sorted form. If the array contains less than two elements,
return 0.
You must write an algorithm that runs in linear time and uses linear extra space.

Here the point to be noted is You must write an algorithm that runs in linear time and uses linear extra space
Describe the solution you'd like
The Approach is to use Radix Sort
Algorithm for Radix Sort

The idea of Radix Sort is to do digit by digit sort starting from least significant
digit to most significant digit. Radix sort uses counting sort as a subroutine to sort.

1. radixSort(arr)
2. max = largest element in the given array
3. d = number of digits in the largest element (or, max)
4. Now, create d buckets of size 0 - 9
5. for i -> 0 to d
6. sort the array elements using counting sort (or any stable sort) according to
the digits at the ith place

we are going to use the Counting Sort algorithm as a subroutine within the Radix Sort implementation.
Radix Sort is a non-comparative sorting algorithm that sorts numbers by their digits. It performs counting sort for each digit position from the least significant digit to the most significant digit. In each iteration, the counting sort is applied to sort the numbers based on the current digit.

Please don't raise issues to solve problems from LeetCode, GFG or HackerRank


Unify lodash imports

[x] kCore
[ ] kTeam
[ ] kNotify
[x] kMap
[ ] kEvent
[ ] kApp

Cleaned callback function call in order to avoid using '_'


Consider adding wireless labprint connection

Solving this may borrow heavily from this: https://github.com/kylelaker/useful_scripts/blob/master/labprintconfig
But also depend on package installation for driver support.

Requires cups-client and gtk3-print-backends

This should do it for the BW printer. For the color printer, just replace "BW" with "Color".
    lpadmin -p "JMU-LabPrintBW" -E\
        -v<EMAIL_ADDRESS>        -L "JMU Labs & Libraries"\
        -m "drv:///sample.drv/generic.ppd""\
        -o "Duplex=DuplexNoTumble"\
        -o "Option1=True"

CUPS should come with the generic postscript driver by default, so that should be fine. cups-client is necessary for lpadmin and gtk3-print-backends will be necessary have the printers available in applications like Firefox. I'm not sure of the two -o options are actually necessary, but they work for me without an issue and seem to get duplex printing working (which probably isn't a bad default).
Since Ansible will be running as root, permissions shouldn't be an issue but a user will need to be in a group in the SystemGroup line in /etc/cups/cups-file.conf to modify the CUPS config (but that is not necessary to use the printers).
I have run this several times on a machine where the printers were already added and I did not have an issue with there being duplicate printers.

gtk3-print-backends appears to be an Arch-ism, as libprintbackend is in the libgtk-3-0 Ubuntu package.

Fixed by c76a9ad6223fed1e9c3de1528d6d559da12f1dc7 and a9cb76fcb6ffa66c896e4cc5676be560f92e6289


Use of this library with Angular

I wish use this components with Angular for improve the performance of components, I try implement this with ngReact but I not found a way to do this. Perhaps I not have skills to do this. Can tell me if someone from development team did something similar?

If you are asking how to put some react goodness into your angular code, here is a way to do it:
const React = require('react');
const FeedsContainer = require('./FeedsContainer');

const $inject = [];
const feeds = function () {
  const link = ($scope, $element) => {
    const node = $element.get(0);
    $scope.$watch('feeds', f =>
      React.render(<FeedsContainer isLoading={!f.length} feeds={f} />, node));
    $scope.$on('$destroy', () => React.unmountComponentAtNode(node));
  };

  return {
    scope: {
      feeds: '='
    },
    restrict: 'E',
    link: link
  };
};
feeds.$inject = $inject;

module.exports = feeds;

Nice, thanks @nkbt :+1:


Create a React editor for HTML content

Use HtmlJsonEditor as example.

The json editor is located at site/node_modules/reactlet-html-json-edit

I created a component for html content input. It is called reactlet-html-web-edit, located at:
./site/node_modules/reactlet-html-web-edit
This widget will be selected for display when field is text/html. The logic is in the common.js
    } else if (modelField.type === 'text') {
        if (modelField.subtype && modelField.subtype.type === 'html') {
            formField = {
                type: 'HtmlWebEdit',
                data: {
                    label: fieldName,
                    key: fieldName,
                    value: fieldValue
                }
            };


Port 80 error when hub and device nodes placed in more than one flow

SUMMARY
The node runs just fine, but only when in a single flow. If I try to use the hub and the device node in another flow, I receive the error message:
Error: listen EADDRINUSE: address already in use :::80"
ECHO DEVICES

Echo Plus 2nd generation
-->

MODULE VERSION

0.1.10

CONFIGURATION

[{"id":"e520dde6.25263","type":"tab","label":"Flow 1","disabled":false,"info":""},{"id":"6b313860.4bb1e8","type":"amazon-echo-device","z":"e520dde6.25263","name":"Office TV","topic":"","x":380,"y":280,"wires":[[]]},{"id":"43d82c91.26dd14","type":"amazon-echo-hub","z":"e520dde6.25263","port":"80","processinput":0,"discovery":true,"x":250,"y":360,"wires":[["6b313860.4bb1e8"]]}]

[{"id":"60dc4b86.f42664","type":"tab","label":"Flow 2","disabled":false,"info":""},{"id":"5a2fc4e6.3358cc","type":"amazon-echo-device","z":"60dc4b86.f42664","name":"Kitchen Light","topic":"","x":270,"y":280,"wires":[[]]},{"id":"7ddca4c8.d8a3fc","type":"amazon-echo-hub","z":"60dc4b86.f42664","port":"80","processinput":0,"discovery":true,"x":130,"y":360,"wires":[["5a2fc4e6.3358cc"]]}]

CONSOLE OUTPUT

**Error: listen EADDRINUSE: address already in use :::80"**

STEPS TO REPRODUCE

Place the hub and device nodes in more than 1 flow
EXPECTED RESULTS

No errors and the node works in as many flows as it's placed in.
ACTUAL RESULTS
Error: listen EADDRINUSE: address already in use :::80"

#datech friend, I think it's better you improve your troubleshooter to avoid such daily issues (

Sorry, I don't understand your response? I don't see guidance on how to resolve this issue.

#datech friend, I think it's better you improve your troubleshooter to avoid such daily issues (

Sorry, I don't understand your response? I don't see guidance on how to resolve this issue.

There is a parallel topic with similar situation. We would need a detailed troubleshooter to avoid same questions

There is a parallel topic with similar situation. We would need a detailed troubleshooter to avoid same questions

Ah ok. That seemed slightly different to my problem. With that problem, the node would not work at all. Mine works, but only when in 1 flow.
Maybe I'm incorrect?
How can I help with troubleshooting? Happy to do so!

Duplicate to #141

Oh thanks! Somehow I missed this issue! I will give this a go!
Thanks!


Allowing nested username (e.g. /rhat/smahajan/) in docker tag command

Closed #11115 because of a bad rebase.
Opened a new one here.

please next time force push your branch instead of opening a new PR
On Mon, Mar 2, 2015 at 1:09 PM, Shishir<EMAIL_ADDRESS>wrote:

Closed #11115 https://github.com/docker/docker/pull/11115 because of a
bad rebase.
Opened a new one here.
You can view, comment on, or merge this pull request online at:
https://github.com/docker/docker/pull/11116
Commit Summary

Allowing nested username (e.g. /rhat/smahajan/) in docker tag command

File Changes

M registry/config.go
https://github.com/docker/docker/pull/11116/files#diff-0 (17)

Patch Links:

https://github.com/docker/docker/pull/11116.patch
https://github.com/docker/docker/pull/11116.diff

—
Reply to this email directly or view it on GitHub
https://github.com/docker/docker/pull/11116.

@jfrazelle noted :)

ping @dmcgowan @dmp42 this will work right also @kencochrane for hub
Also @tiborvass do you think this will effect anything with namespacing

@jfrazelle I'm afraid this one will take some time to review, and it is not only because of code review.
I'm putting this in design review.
@shishir-a412ed
Some questions are:

if the hub does not support it currently, will it?
is there a usecase for this without the hub supporting it?
If so, would it be okay to accept naming images like that, and erroring out on push to a docker registry until the registry supports it?

@shishir-a412ed can you add some test cases?

The v1 registry has no support for nested usernames.  Currently this is updating an already messy implementation verifying the namespace.  This code will be refactored for 1.7 when the code is split out into distribution. I am marking it as 1.7.0, we might close it with an alternative implementation later which only works for v2 registries and enforces v1 namespace rules.

@dmcgowan apart from not wanting to change messy code that is going to be refactored, can you comment on the design itself? Do you agree? + https://github.com/docker/docker/pull/11116#issuecomment-77172790
@ncdc thanks for helping with the review! Please remember that this is in design review phase :) It's in order to avoid unnecessary back-and-forths between contributors and maintainers.

This should be handled through the validation of names via the docker/distribution imports. While the library support is fresh, the validation and regular expressions are solid and should be used here.
Note that we are going away from a limited <namespace>/<name> view, so I am unsure what problem this specific PR is addressing. To clarify, the weak notion of naming the path components should be dropped.
It would be helpful if you could outline the desired behavior clearly in the the description, including examples of what was valid/invalid before and after this change. From the looks of it, we may be already supporting what you're looking for.

@stevvooe
Example: registry.access.redhat.com:5000/rhat/dockeruser/smahajan/busybox:latest
In this example: namespace (username) is rhat/dockeruser/smahajan
Before this change:
This is an invalid namespace (username) if I try to run the docker tag command.
We want to allow this to have fully qualified usernames. This change fixes that.
If this is already being supported, let me know and I ll test it.

@shishir-a412ed I would think this should already be supported, if not already. I think the confusion I have with this PR is that it makes the slash (/) a valid namespace component character, which it absolutely is not.
Check out https://github.com/docker/distribution/blob/master/registry/api/v2/names.go for "the future of naming". We are moving towards that layout as much as possible and the tags command should probably reflect that.
Let me know what your findings are.

This implementation should match the rules defined in the distribution package, look at the link's Andy sent.
As to the comment from @stevvooe, it would be better if the name was split apart and each component individually checked.  The way it is coded now names such as "docker/-bad/docker" will be considered valid.

This change uses #11478 . Waiting for it to be merged.

@shishir-a412ed please close this PR then, and reopen it once the dependent PR is merged.

Yes, I'm sorry but there is just to much to keep track of to afford dealing with PR that need to wait on other PR.

Since https://github.com/docker/docker/pull/11478 has been merged, should we reopen this pull request.

@rhatdan this is superseded by #13070

Thanks, tough to follow the bouncing pull requests.

@rhatdan apparently github doesn't allow to reopen the PR, if we force push on a closed PR.
and a force push is sometimes needed to rebase. So had to open a new PR.
https://github.com/isaacs/github/issues/361


Json Codec: Read missing field as empty collection

Currently, if a field that is a collection is missing, that will lead to a decoding error.
Either by default or configurable, if a field that is a collection is missing, we should just return an empty collection.

/bounty $75

/attempt #664


show canvas on retina in a correct way

This is a split of #117. More details there.

Looks good! Let's wait untill tests passed and I'll merge it then.

@kottenator We are good to go.


feat: bulild vsphere template in air gapped environment

What problem does this PR solve?:

create vsphere image template for RHEL 7.9 and 8.4 in air gapped environments

Which issue(s) does this PR fix?:

https://jira.d2iq.com/browse/D2IQ-84576

Output:
Following template was created in airgapped VM network.
The packer job to create image in template was run from a bastion VM on the same airgapped network as the VM.
--> vsphere-clone: konvoy-ova-vsphere-rhel-79-1.21.6-1645637142
--> vsphere-clone: konvoy-ova-vsphere-rhel-79-1.21.6-1645637142

Special notes for your reviewer:

issues and workarounds:

Issue:  The cloud-init datasource for vmware guestinfo was merged in Aug 2021 (https://github.com/canonical/cloud-init/pull/953) and only available in cloud-init > 21.3 . The cloud-init rpms that are available in Centos7/RHEL repos is > 19.4-7.

workaround: upstream project downloads and runs the installation script from the archived repo (https://raw.githubusercontent.com/vmware/cloud-init-vmware-guestinfo/v1.4.0/install.sh) to install the vmware datasource. similar workaround is done to support this in airgapped mode where we copy the datasource module from ansible host to the guest and then run the script.

issue: The vmware cloudinit datasource module requires 8.1>pip<21.6 for Centos7/RHEL7. The pip-8.1.2 is default version. The latest pip version breaks on the Centos7/RHEL7

workaround:  Download fix latest pip version 20.3.4 that is < 21.6  for offline builds

issue: unable to run packer job for airgapped environment from laptop or teamcity job as the VMs created in airgapped environments are not accessible from outside.

workaround: Created a bastion node manually that has two VM networks: "public" and "airgapped".  copy the repository to the bastion host using public IP. then run packer to create image in "airgapped" environment by overriding networking for the target VM to "airgapped"
enhancements: Create a terraform script to create bastion host with two network interfaces. copy konvoy-image binary and ansible/packer files to the bastion host and run packer remotely with ssh. (similar to e2e tests in konvoy2)
should we embed ansible playbooks in the konvoy-binary too?

@jkoelker I had to change dependency order of kubeadm role and packages role since kubeadm installs latest version of kubectl rpm which is not compatible with kubeadm. This needs to fixed upstream, but as a stop gap I had to provide fix ordering between these two roles. Please let me know if this hack is fine.

@jkoelker I had to change dependency order of kubeadm role and packages role since kubeadm installs latest version of kubectl rpm which is not compatible with kubeadm. This needs to fixed upstream, but as a stop gap I had to provide fix ordering between these two roles. Please let me know if this hack is fine.

:sad-panda:, yep that should be fine, the list-images.yaml playbook only needs kubeadm and containerd roles, so changing the order on the provision.yaml playbook should work no problems.


fix #14 mixed bug in mixed xml and text content

@dylang Do you still maintain this package? I need this changes, too. Please, merge it and publish a new package version.


anyhow::Errorを抱えたエラー型は==ではなくassert_matches!で比較する

内容
https://github.com/VOICEVOX/voicevox_core/pull/440#issuecomment-1471561912
関連 Issue
Closes #440.
その他

いやassert_matches!を使うのではなく、pretty_assertions::Comparisonだけ使ってpretty版assert_matches!的なものを作った方がいいかも

assert_matches!ベースだとpretty_assertionsが使えないため、失敗したときのメッセージがこんな感じになるので #440 の方がいいかもしれません。
failures:

---- engine::open_jtalk::tests::extract_fullcontext_works::case_2 stdout ----
-------------- TEST START --------------
thread 'engine::open_jtalk::tests::extract_fullcontext_works::case_2' panicked at 'assertion failed: `Ok(["xx^xx-sil+k=o/A:xx+xx+xx/B:xx-xx_xx/C:xx_xx+xx/D:09+xx_xx/E:xx_xx!xx_xx-xx/F:xx_xx#xx_xx@xx_xx|xx_xx/G:5_5%0_xx_xx/H:xx_xx/I:xx-xx@xx+xx&xx-xx|xx+xx/J:1_5/K:2+2-9", "xx^sil-k+o=N/A:-4+1+5/B:xx-xx_xx/C:09_xx+xx/D:09+xx_xx/E:xx_xx!xx_xx-xx/F:5_5#0_xx@1_1|1_5/G:4_1%0_xx_0/H:xx_xx/I:1-5@1+2&1-2|1+9/J:1_4/K:2+2-9", "sil^k-o+N=n/A:-4+1+5/B:xx-xx_xx/C:09_xx+xx/D:09+xx_xx/E:xx_xx!xx_xx-xx/F:5_5#0_xx@1_1|1_5/G:4_1%0_xx_0/H:xx_xx/I:1-5@1+2&1-2|1+9/J:1_4/K:2+2-9", "k^o-N+n=i/A:-3+2+4/B:xx-xx_xx/C:09_xx+xx/D:09+xx_xx/E:xx_xx!xx_xx-xx/F:5_5#0_xx@1_1|1_5/G:4_1%0_xx_0/H:xx_xx/I:1-5@1+2&1-2|1+9/J:1_4/K:2+2-9", "o^N-n+i=ch/A:-2+3+3/B:xx-xx_xx/C:09_xx+xx/D:09+xx_xx/E:xx_xx!xx_xx-xx/F:5_5#0_xx@1_1|1_5/G:4_1%0_xx_0/H:xx_xx/I:1-5@1+2&1-2|1+9/J:1_4/K:2+2-9", "N^n-i+ch=i/A:-2+3+3/B:xx-xx_xx/C:09_xx+xx/D:09+xx_xx/E:xx_xx!xx_xx-xx/F:5_5#0_xx@1_1|1_5/G:4_1%0_xx_0/H:xx_xx/I:1-5@1+2&1-2|1+9/J:1_4/K:2+2-9", "n^i-ch+i=w/A:-1+4+2/B:xx-xx_xx/C:09_xx+xx/D:09+xx_xx/E:xx_xx!xx_xx-xx/F:5_5#0_xx@1_1|1_5/G:4_1%0_xx_0/H:xx_xx/I:1-5@1+2&1-2|1+9/J:1_4/K:2+2-9", "i^ch-i+w=a/A:-1+4+2/B:xx-xx_xx/C:09_xx+xx/D:09+xx_xx/E:xx_xx!xx_xx-xx/F:5_5#0_xx@1_1|1_5/G:4_1%0_xx_0/H:xx_xx/I:1-5@1+2&1-2|1+9/J:1_4/K:2+2-9", "ch^i-w+a=pau/A:0+5+1/B:xx-xx_xx/C:09_xx+xx/D:09+xx_xx/E:xx_xx!xx_xx-xx/F:5_5#0_xx@1_1|1_5/G:4_1%0_xx_0/H:xx_xx/I:1-5@1+2&1-2|1+9/J:1_4/K:2+2-9", "i^w-a+pau=h/A:0+5+1/B:xx-xx_xx/C:09_xx+xx/D:09+xx_xx/E:xx_xx!xx_xx-xx/F:5_5#0_xx@1_1|1_5/G:4_1%0_xx_0/H:xx_xx/I:1-5@1+2&1-2|1+9/J:1_4/K:2+2-9", "w^a-pau+h=i/A:xx+xx+xx/B:09-xx_xx/C:xx_xx+xx/D:09+xx_xx/E:5_5!0_xx-xx/F:xx_xx#xx_xx@xx_xx|xx_xx/G:4_1%0_xx_xx/H:1_5/I:xx-xx@xx+xx&xx-xx|xx+xx/J:1_4/K:2+2-9", "a^pau-h+i=h/A:0+1+4/B:09-xx_xx/C:09_xx+xx/D:22+xx_xx/E:5_5!0_xx-0/F:4_1#0_xx@1_1|1_4/G:xx_xx%xx_xx_xx/H:1_5/I:1-4@2+1&2-1|6+4/J:xx_xx/K:2+2-9", "pau^h-i+h=o/A:0+1+4/B:09-xx_xx/C:09_xx+xx/D:22+xx_xx/E:5_5!0_xx-0/F:4_1#0_xx@1_1|1_4/G:xx_xx%xx_xx_xx/H:1_5/I:1-4@2+1&2-1|6+4/J:xx_xx/K:2+2-9", "h^i-h+o=d/A:1+2+3/B:09-xx_xx/C:22_xx+xx/D:10+7_2/E:5_5!0_xx-0/F:4_1#0_xx@1_1|1_4/G:xx_xx%xx_xx_xx/H:1_5/I:1-4@2+1&2-1|6+4/J:xx_xx/K:2+2-9", "i^h-o+d=e/A:1+2+3/B:09-xx_xx/C:22_xx+xx/D:10+7_2/E:5_5!0_xx-0/F:4_1#0_xx@1_1|1_4/G:xx_xx%xx_xx_xx/H:1_5/I:1-4@2+1&2-1|6+4/J:xx_xx/K:2+2-9", "h^o-d+e=s/A:2+3+2/B:22-xx_xx/C:10_7+2/D:xx+xx_xx/E:5_5!0_xx-0/F:4_1#0_xx@1_1|1_4/G:xx_xx%xx_xx_xx/H:1_5/I:1-4@2+1&2-1|6+4/J:xx_xx/K:2+2-9", "o^d-e+s=U/A:2+3+2/B:22-xx_xx/C:10_7+2/D:xx+xx_xx/E:5_5!0_xx-0/F:4_1#0_xx@1_1|1_4/G:xx_xx%xx_xx_xx/H:1_5/I:1-4@2+1&2-1|6+4/J:xx_xx/K:2+2-9", "d^e-s+U=sil/A:3+4+1/B:22-xx_xx/C:10_7+2/D:xx+xx_xx/E:5_5!0_xx-0/F:4_1#0_xx@1_1|1_4/G:xx_xx%xx_xx_xx/H:1_5/I:1-4@2+1&2-1|6+4/J:xx_xx/K:2+2-9", "e^s-U+sil=xx/A:3+4+1/B:22-xx_xx/C:10_7+2/D:xx+xx_xx/E:5_5!0_xx-0/F:4_1#0_xx@1_1|1_4/G:xx_xx%xx_xx_xx/H:1_5/I:1-4@2+1&2-1|6+4/J:xx_xx/K:2+2-9", "s^U-sil+xx=xx/A:xx+xx+xx/B:10-7_2/C:xx_xx+xx/D:xx+xx_xx/E:4_1!0_xx-xx/F:xx_xx#xx_xx@xx_xx|xx_xx/G:xx_xx%xx_xx_xx/H:1_4/I:xx-xx@xx+xx&xx-xx|xx+xx/J:xx_xx/K:2+2-9"])` does not match `Ok(c) if c != testdata_hello_hiho()`', crates/voicevox_core/src/engine/open_jtalk.rs:196:38
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace


Export isOpen() and metrics

We have a need to check if a circuit is open.  Is there an easy way to do that?  Also, we want to send some status to our metrics server and the only way I can think of is to hack the HTTP streamer.  An example here (to stdout):
// This is  used to output the Hystrix stream to stdout and only used for debugging
// circuit stats
type outputResponseStdout struct {
	HeaderMap http.Header
}

func (o *outputResponseStdout) Header() http.Header {
	m := o.HeaderMap
	if m == nil {
		m = make(http.Header)
		o.HeaderMap = m
	}
	return m
}

func (o *outputResponseStdout) Write(buf []byte) (int, error) {
	return os.Stdout.Write(buf)
}

func (o *outputResponseStdout) WriteHeader(c int) {
	fmt.Println("HTTP code: ", c)
}

func OutputHystrixEvents() {
	s := hystrix.NewStreamHandler()
	s.Start()
	rh := &outputResponseStdout{}
	req, err := http.NewRequest("GET", "", nil)
	if err != nil {
		return
	}
	s.ServeHTTP(rh, req)
}

@crxpandion is working on a patch introducing non-dashboard metrics (i.e. statsd)

Yeah I am working on introducing a metrics interface that will allow pluggable aggregators.

#25

statsd metrics are now possible in master, and there is a framework for your own implementation for custom solutions.


[release-v1.13] Remove assumption on default namespace for EventType reference (#7724)

Backport of https://github.com/knative/eventing/pull/7724

/retest

/retest-required

/retest

/override ci/prow/411-test-e2e-aws-411
as the 4.11 jobs have been removed (see https://github.com/openshift-knative/hack/pull/126)

same here
/override ci/prow/411-test-encryption-auth-e2e-aws-411

/retest

/retest

/retest

/test 412-test-reconciler-aws-412

/unhold


[BACKPORT-2.1][SPARK-19512][SQL] codegen for compare structs fails #16852

What changes were proposed in this pull request?
Set currentVars to null in GenerateOrdering.genComparisons before genCode is called. genCode ignores INPUT_ROW if currentVars is not null and in genComparisons we want it to use INPUT_ROW.
How was this patch tested?
Added test with 2 queries in WholeStageCodegenSuite

Test build #72658 has started for PR 16875 at commit f155856.

LGTM

Test build #72658 has finished for PR 16875 at commit f155856.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

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/SparkPullRequestBuilder/72658/
Test PASSed.

@bogdanrdc can you close this? It won't auto close because it is not merged in master.


Specify authentication fields in the documentation

I have an application using email and password for authentication, and I've had a few problems trying to integrate Cake3-CookieAuth. After some investigation, I have found out that you need to tell CookieAuth which fields are in use:
$this->loadComponent('Auth', [
    'authenticate' => [
        'Form' => [
            'fields' => ['username' => 'email']
        ],
        'Xety/Cake3CookieAuth.Cookie' => [
            'fields' => ['username' => 'email'] // this is what I was missing
        ]
    ],
    ...
]);

I think that this should be explained in the README...

👍 Was missing the same thing

The fields name are the default one used by CakePHP itself. But i will add it to the Readme.

This doesn't appear to have been added to the readme.

@PhantomWatson I added it in this commit : https://github.com/Xety/Cake3-CookieAuth/commit/b1f899eba842e43584caee2c873ec852bfa180fa#diff-04c6e90faac2675aa89e2176d2eec7d8R31. You can find it in the Configuration part : https://github.com/Xety/Cake3-CookieAuth#configuration
I don't want to set all the configuration options in the Readme, due to the change that Cake can do. So i prefer to redirect people to the officiel Cake doc.

Ohhh, I understand now. Thanks!


Remove empty unreferenced file unshimmed-spark311.txt

Signed-off-by: Gera Shegalov<EMAIL_ADDRESS>

build


Ensure Travis builds run in containers

Travis CI now supports containers as the default build infrastructure. For
builds running outside of containers, a warning is created for each build.
We can upgrade to the container-based infrastructure by adding this line to
the .travis.yml. The new infrastructure provides us with other build
benefits, too:
http://blog.travis-ci.com/2014-12-17-faster-builds-with-container-based-infrastructure/

We can also enable dependency caching, however as this build completed rather quickly, I didn't bother. My main concern is that it will cache old versions of Gems, if we're not pinning the version.


SvelteKit 2.0 breaking "tabulator-tables" import statement

Describe the bug
Importing the package "tabulator-tables" throws an error saying that it's a CJS module and it should be imported in a specific way. On SvelteKit 1 this importation is working fine as is but the issue started to happen when I upgraded to SvelteKit 2. This could also potentially be an issue with Vite but so far this issue is only happening for SvelteKit 2 and not on other frontend frameworks.
Here's the import code
import { TabulatorFull as Tabulator } from 'tabulator-tables';

This is currently my workaround for this issue
import * as Tabulator_Import from 'tabulator-tables';
const { TabulatorFull: Tabulator } = Tabulator_Import;

Here's the bug report that was opened on tabulator-tables. The author stated that it should work properly as is and it's weird that this is happening specifically only on SvelteKit.
https://github.com/olifolkerd/tabulator/issues/4378
Here's the output of the issue

Reproduction
Here's a repo with the minimum reproducible issue:
https://github.com/thisjt/tabulator-import-issue

clone the repo
npm install
npm run dev
navigate to the dev environment on the browser (http://localhost:5173)

Logs
No response
System Info
(GitHub Codespaces)
  System:
    OS: Linux 6.5 Ubuntu 20.04.6 LTS (Focal Fossa)
    CPU: (2) x64 AMD EPYC 7763 64-Core Processor
    Memory: 5.76 GB / 7.74 GB
    Container: Yes
    Shell: 5.0.17 - /bin/bash
  Binaries:
    Node: 20.12.1 - ~/nvm/current/bin/node
    Yarn: 1.22.19 - /usr/bin/yarn
    npm: 10.5.0 - ~/nvm/current/bin/npm
    pnpm: 8.15.6 - ~/nvm/current/bin/pnpm
  npmPackages:
    svelte: ^4.2.7 => 4.2.15

Severity
annoyance

the package in question does not have "type": "module" in its package.json so .js files in it are considered to be commonjs. But /dist/js/tabulator_esm.js  is esm. It should either have a .mjs extension or the package needs to change to "type":"module" and commonjs exports need a .cjs extension.
see<EMAIL_ADDRESS>and also https://antfu.me/posts/publish-esm-and-cjs
opening your reproduction in stackblitz shows a different error https://stackblitz.com/github/thisjt/tabulator-import-issue?file=README.md  (looks like building an error stacktrace broke)
This is very unlikely to be caused by sveltekit.  On the offchance it is, please provide a reproduction that shows tabulator-tables 6.0 working in sveltekit@1, not working in sveltekit@2 and also working in a generic vite 5 js project.
sveltekit is an esm first framework, and personally i would recommend using dependencies that also share this spirit.

I have inform the author with your suggestions. I'm currently waiting for his reply. As of now, I have replicated the package working fine as is in<EMAIL_ADDRESS>I'm not sure if I'm doing the barebones vite@5 right but the importing is also working fine there. For sveltekit@2 I'm getting the same error as what you saw when trying to import the package so not sure what's up with that.
<EMAIL_ADDRESS>barebones repro https://stackblitz.com/edit/vitejs-vite-pqilhv
Sveltekit@2 barebones repro https://stackblitz.com/edit/vitejs-vite-dp8aka (broken with unrelated error)
Vite@5 barebones repro https://stackblitz.com/edit/vitejs-vite-cdmwht

@thisjt please check Vite 5 with the vanilla SSR template from https://github.com/bluwy/create-vite-extra
Some issues with vite only show up in SSR which is not in the default template

We can close this issue now. This has been fixed as per PR https://github.com/olifolkerd/tabulator/pull/4490 from the package tabulator-tables. Thanks all!


choices are not enforced in form

The choices on the unit type are not enforced in the frontend:

User can input any arbitrary value

Fixed in 1b8d43b4e5cd4ad3c16de028233e36cb291b2685


"Psr-http-message-bridge" package was not required?

Laravel Version: 5.5.22
PHP Version:
Database Driver & Version:

Description:
This is not a problem encountered in use, but to see the code is by accident, using the package "Psr-http-message-bridge" but not required.
Steps To Reproduce:
The file path is /usr/local/var/www/laravel_pro/vendor/laravel/framework/src/Illuminate/Routing/Router.php
public static function toResponse($request, $response)
    {
        if ($response instanceof Responsable) {
            $response = $response->toResponse($request);
        }

        if ($response instanceof PsrResponseInterface) {
            $response = (new HttpFoundationFactory)->createResponse($response);
        } elseif (! $response instanceof SymfonyResponse &&
                   ($response instanceof Arrayable ||
                    $response instanceof Jsonable ||
                    $response instanceof ArrayObject ||
                    $response instanceof JsonSerializable ||
                    is_array($response))) {
            $response = new JsonResponse($response);
        } elseif (! $response instanceof SymfonyResponse) {
            $response = new Response($response);
        }

        if ($response->getStatusCode() === Response::HTTP_NOT_MODIFIED) {
            $response->setNotModified();
        }

        return $response->prepare($request);
    }

$response = (new HttpFoundationFactory)->createResponse($response);
The class HttpFoundationFactory belongs to Psr-http-message-bridge this package, but not required.
thanks!

According to the documentation, if you would like to use the psr7 standard for requests/responses...

you will first need to install a few libraries. Laravel uses the Symfony HTTP Message Bridge component to convert typical Laravel requests and responses into PSR-7 compatible implementations:

composer require symfony/psr-http-message-bridge
composer require zendframework/zend-diactoros

so if you are indeed using the psr7 requests/responses, the appropriate libraries will be installed when needed.

Composer will also tell you this when you run it. Look at the suggested package output.

thanks


修正特征提取，补齐不足batch的数据

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

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

pr和commit描述都改成英文吧

pr和commit描述都改成英文吧


Access Permission disappear on book after using the API

Describe the Bug
To edit pages on Bookstack, some people in our team use the web interface, while others store their content on git and use a Python tool to upload it to Bookstack. In the latter case, this has caused some permission issues, with permissions on the book disappearing (but not on the pages or chapters). The joint_permissions table is then empty for the targeted books. Manually creating permissions in this table, or using php artisan bookstack:regenerate-permissions fixes the issue. This has happened three times yet, the first time on January 23rd. It had never happened before we’ve used 23.12. However we are increasingly using Bookstack and the issue could have been present before.
It seems similar to https://github.com/BookStackApp/BookStack/issues/4791, but on the book, and not on the page.
Steps to Reproduce
I am not yet sure how the bug is caused exactly. What I can say is that it has always happened while using the API.
Expected Behaviour
The book should remain accessible.
Screenshots or Additional Context
No response
Browser Details
No response
Exact BookStack Version
v23.12

Hi @matthias4217, Could you confirm specifically what API endpoints are being used, and the order they're used in?
Or (if public) link to the tool and advise which methods/actions of the tool are being used?
Just want to gain an idea of exactly what endpoints are used.

We have developed two tools around Bookstack. Both are Python apps, in private repositories.

Bookstack Carpenter manages user permissions and enforcing an architecture on Bookstack (creating some books depending on some external condition for instance). It uses get, put and post from requests.
Gitplo uploads contents from a git repository to a book on Bookstack. It uses the Session object from requests.

The error has been detected using Gitplo on the first call to the Bookstack API, a GET on the books endpoint. I suppose the issue began before that. The three impacted books are edited using Gitplo, and only two of them are managed by Bookstack Carpenter, so I assume it’s a call from Gitplo that is responsible for the issue. These books are notably not modified through the web interface.
Gitplo uses these API endpoints :

Attachments : GET, LIST, DELETE, CREATE
Books : GET, LIST
Chapters : GET, LIST, DELETE, CREATE, UPDATE
Image-gallery : GET, LIST, DELETE, CREATE
Page : GET, LIST, DELETE, CREATE, UPDATE

Notably, it doesn’t modify books nor does it use the Content-permissions endpoint, and apart for three times where this error has happened (on two Bookstack instances in total), our worflow works fine.

Thanks for the extra info @matthias4217.
Can you detail how you're running BookStack? (hosting method, OS, webserver).

We run Bookstack on Openshift, and we build our Docker images ourselves (with php:8.1-fpm-alpine and nginx:1.25-alpine). We have slightly modified the code to show resource ids on pages (using this merge request : https://github.com/BookStackApp/BookStack/pull/4314/files) and tot display the book parent shelves (from this issue : https://github.com/BookStackApp/BookStack/issues/4547#issuecomment-1726420230).

Report from a user on discord of similar, running on Ubuntu 18.04 server, with MariaDB 10.1.48 database, Apache2, & PHP8.1-CGI. Reference

We have a similar problem with missing page permission. We are still trying to reproduce, but our assumption is that calling PUT /api/pages/{id} for multiple pages with changed priority in parallel was the root cause (we tried to sort the book).
After php artisan bookstack:regenerate-permissions the pages reappeared.
System info: BookStack 24.10 on Ubuntu 24.04.1 with MariaDB 10.11.8 and libapache2-mod-php8.3

I was able to reproduce the issue and I think I found the issue. When multiple API requests are running simultaneously there is a race condition. The permissions are first deleted and then recreated:
https://github.com/BookStackApp/BookStack/blob/ac27e1893388c39f2b43c92fb91a0fca3d6ae7b3/app/Permissions/JointPermissionBuilder.php#L146-L151
Delete and Create are each wrapped in transaction:
https://github.com/BookStackApp/BookStack/blob/ac27e1893388c39f2b43c92fb91a0fca3d6ae7b3/app/Permissions/JointPermissionBuilder.php#L163
https://github.com/BookStackApp/BookStack/blob/ac27e1893388c39f2b43c92fb91a0fca3d6ae7b3/app/Permissions/JointPermissionBuilder.php#L228
But two parallel update requests for different pages within the same book may interleave and fail with missing permissions for the second page.


directionPositionList null

Hi, i was using this lib for a few days now and i realized that i can't draw routes anymore, after debugging i noticed that directionPositionList is empty. Any help please

I too can't seem to draw routes anymore and I have this error
 E/Volley: [155] hj.a: Unexpected response code 400 for https://clients4.google.com/glm/mmap/api D/Volley: [154] hj.a: HTTP response for request=<[ ] https://clients4.google.com/glm/mmap/api 0x99e6744e NORMAL 95> [lifetime=12072], [size=145], [rc=400], [retryCount=1] D/Volley: [1] gu.b: 12074 ms: [ ] https://clients4.google.com/glm/mmap/api 0x99e6744e NORMAL 95

I solved my problem by changing the Google ServerKey i was using, check
that out
2016-07-02 16:02 GMT+00:00 Ivan Alberto<EMAIL_ADDRESS>
I too can't seem to draw routes anymore and I have this error
E/Volley: [155] hj.a: Unexpected response code 400 for
https://clients4.google.com/glm/mmap/api
D/Volley: [154] hj.a: HTTP response for request=<[ ]
https://clients4.google.com/glm/mmap/api 0x99e6744e NORMAL 95>
[lifetime=12072], [size=145], [rc=400], [retryCount=1]
D/Volley: [1] gu.b: 12074 ms: [ ] https://clients4.google.com/glm/mmap/api
0x99e6744e NORMAL 95
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/akexorcist/Android-GoogleDirectionLibrary/issues/17#issuecomment-230108837,
or mute the thread
https://github.com/notifications/unsubscribe/AQlY9bdT8U0zjPY2U_LOqn1EXy25H8jYks5qRot8gaJpZM4I-CFI
.


Does not work with Ghoulcraft's CIT

Just simply doesnt let Ghoulcraft's CIT appear.

After much consideration and a suggestion that was given in #9, I've decided to allow broken paths and block it behind both a config option that defaults to not allow broken paths as well as mark the packs incompatible in the resourcepack selection screen.
This has been implemented in v0.5.0.
Still this is very experimental and no bugs that derive from the config option being enabled will be resolved.


Update LSTMBlockCell, LSTMBlockFusedCell to use LSTMStateTuple as state

Fixes #6582

Can one of the admins verify this patch?

@ebrevdo any interest?

Reading my codes written in dawn again, I found LSTMBlockFusedCell should not be fixed! I will revert it back.

@jihunchoi just to make sure, this PR is blocked on #7387 ?

No, I mentioned this PR wrongly in #7387 but the message does not disappear after I deleted the mention...

It's okay, just wanted to understand the status -- so this is ready to review again?

Yes, the mention can be safely ignored.

Can you resolve merge conflicts?

Was there a conflict? It was able to be automatically merged with tensorflow/master branch.
Anyway, I updated it to follow the latest master branch and it seems there exists no conflict.

thanks!  please ensure all tests pass.

@tensorflow-jenkins test this please


Remember editor window mode, screen, size and position on restart

Second attempt at a change to allow godot editor to remember the last size, screen, position and window mode across restarts.
When working on a godot project on a high res screen or with an external text editor, people might prefer to not have the editor maximized on screen.
The previous behaviour was for the editor window to start maximized, and when windowed it would resize to a very small size.
I made a previous attempt here: https://github.com/godotengine/godot/pull/76076
However, this version has some nice improvements:

It runs earlier in the process, which means the window starts at the right size and does not change after the editor has loaded;
Is per-project, so people can setup different work environments per project (which is also the behaviour for layouts)

The project setting defaults to false and is hidden under the advanced tab. This can be changed depending of what people decide the best "default behaviour" would be

Instead of adding a new method, this can be done in _save_docks() (which contrary to its name, saves not only docks).

Thank you @KoBeWi for the feedback. Your suggested change has the additional side-effect of saving the window position when docks are changed, which I see as a bonus (if the editor crashes or is otherwise killed, it would still remember the position). I have updated the code

@spanzeri I did something similar in #62378 for docks, here: https://github.com/godotengine/godot/pull/62378/files#diff-b88f0bde58b731e8dd353707c8011fb7a1e452f0db2c9f15f4b7905d60e5e52eR5230-R5265
You might want to take a look at it, especially the fallbacks.

@trollodel first of all, I love your PR. I would love to have the script editor detachable. It is probably a good idea to add some checks on reload, as soon as I have some time I will improve the restore code to be a bit smarter

@trollodel I had a look and I am conflicted about making adjustments.
The settings are read before both the display server and the editor node are created.
The editor_node is aware of the minimum size for a working editor and fixes the windows size during construction.
The position and screen are ensured by the display_driver implementations that all have some platform specific logic to deal with off-screen windows (or in some cases, like android, they ignore the settings altogether).
Your changelist was different on this aspect because it deals with windows created after the first one, which at the very least need a valid screen index and a valid size.
For this reason I think it would be better to not modify the initial window after the display server is created to not cause unexpected behaviours if either the editor_node or display server implementation is changed.
If you can think of a specific case where this breaks that I did not think of, I am happy to discuss it :)

Didn't know about the checks done in display servers.
In any case, I wouldn't rely on display server and its fallback behavior. For example, I would center the window in the screen if the saved rect goes off-screen.
However, I agree that the context in #62378 is different, so feel free to take whatever you think is useful and leave the rest.

Implemented logic to re-fit the editor window to the screen usable rect or their position has changed since the last editor cfg was saved

I'll see if I can make it happen

Do it please, it's so annoying that it maximizes the window every time, like leave it alone

@nbooba Please don't bump issues without contributing significant new information. Use the :+1: reaction button on the first post instead.

I have finally spent some time looking at this PR again. Apologies for the long break, but work left me with a lot less spare time as of late.
I have changed the saving code to not override the size when the window is something other than windowed.
Now, this is not a perfect solution.
If one closes the window maximized, the next run of the editor, the window will remember the last non maximized size.
However, if the size is changed, the window maximized, then the editor gets closed, the last non-maximised size is not saved.
I have tried adding the WM_RESIZE notification, but that seems to trigger on the window, but not on the editor node.
I would prefer to not further scatter this code on a whole bunch of different places.
If anyone with a better knowledge of the engine/editor than me has an idea how to further improve this code (i.e. capture window events and store the last non-maximised size every time it changes) I am open to suggestions :)

Any improvement on this feature ?

Discussion around this PR has stagnated a bit and the code is fairly out of date at this point. I'd need to find the time to merge with latest with master and see if I can improve it enough to get it over the line.
if there is interest, I could certainly try to find the time to do that

if there is interest

Let me just say, every time my project (window) loads, I think of you ;)

A lot of godot users will thank you, including me :)

Please don't bump without contributing significant new information. Use the :+1: reaction button on the first post instead. Feedback is appreciated but this doesn't really add anything and it just pings a lot of people needlessly

New and improved (hopefully) version of this PR.
I made a few changes to address some of the standing issues/concerns.

The editor node now detects window size changes and can remember the last state (maximized, full-screen or windowed) even if the editor is closed while minimised;
I moved the code within the trace events so it is benchmarked with the other startup code;
I moved the setting so it is now a editor config rather than a project config (note that position, size, mode and screen are still saved per project so people can have different editor config for different projects)

This is on by default in the current version, but if we'd rather have it off  I am happy to flip the condition.
I have also retested the ability of the window to recenter/resize if it starts off-screen (for example in case of a monitor being unplugged) and that seems to be working fine.

This works correctly now, but there are some weird things in the implementation. has_valid_last_windowed_size has effectively no effect. Also is there a reason why you connect to size_changed signal instead of just fetching values when saving layout? This seems to have to do something with minimized window, but it's implemented incorrectly and does nothing. If editor is closed with minimized window, IMO it's fine if it's stored as fullscreen.
Or you can save the window as minimized, so it opens minimized 😏

Thank you @KoBeWi  for the review.
I have removed the empty line and removed the last window size as you suggested, because was indeed not needed.
However, the signal and bool were working as intended.
They are there- as you guessed - to deal with the case when the editor window is closed while minimised.
With the current implementation (which is hopefully a bit clearer), we keep track if the window was windowed before being minimised. On the next startup, we can restore the window to the last size and windowed mode.
I could save maximised in that case, but imho remembering the last user config is a more pleasant behaviour than maximising.
I could also store the window mode as minimised and start the editor minimised on restart, but that would just be evil :)
I have run a few tests again and the bool is changing as expected, but if there is something you see and I am missing feel free to let me know.
As a side node, I'd be preferable to not have to connect the signal and rely on notification, but unfortunately WM events seems to only fire for windows and viewports and when I added a printf to _notification there was nothing coming through for minimise/restore event on the editor node.
But if there is a better way to handle those cases that I don't know about, I'd be happy to make another change

Window position/size is only remembered if you press Ctrl + S before exiting the editor.
Window position is restored correctly, but size is not.

That wasn't the case when I tested 🤔

@Calinou that is very surprising indeed. The function that saves position and size is called by the same function that stores layout, and always on close (without the need for saving). Also, the size and position are stored at the same time.
Is it possible you had some command line overrides?  My CL intentionally does nothing if you specify a command line argument as I assume you know what you want and you don't want me to mess with it.
If you don't, then I need to figure out why you are seeing that issue (I'm also on linux nowadays, but wayland and not X11)

I have rebased with latest master and run the change again.
On linux + wayland I can confirm it saves both position as size as expected (without the need for saving).
Regarding save on Ctrl+C, that is a bit more complicated. I am not sure if we have some signal handler, but it would need to happen on a OS specific manner and I am not even sure the window would still be available at that point.
However, I do feel like there is less of an expectation for the window size and position to be remembered on those circumstances (crash, sigabort/sigkill, terminate from a debug session etc).
I know Unreal for example also does not save window position and size on those cases

Is it possible you had some command line overrides?

No, I started the editor from the command line (by specifying the path to the project.godot file) but that's about it.

Would you be able to open editor_layout.cfg from the cache directory of your project (path_to_project/.godot/editor/editor_layout.cfg) and see what's inside there? Should look something like this

I also just realized the save function is called on resize (due to the layout being saved on resize). Maybe it should also call (at least the new function) on move, so even if the editor crashes it will have saved both correctly if you moved/resized

Will this get merged soon? My Godot always opens in fullscreen no matter how I close it and it's so annoying :)

Thanks for the great usability improvement, and for sticking with the long review process!

Really glad to see this change has been merged, and looking forward to it making its way into release.
Is there any possibility that the size and position could also be added as command line options, perhaps?  Assuming you have cracked the nut of being able to restore the window based on a configuration / project file, perhaps there could also be a command-line override?
Reason for this is wanting to "force" particular sizes and positions for new projects.  Unless your change already accounts for that, and I just misunderstood?

@som45oul command line option for size and other display properties have been present for a while. See here: [https://docs.godotengine.org/en/stable/tutorials/editor/command_line_tutorial.html#command-line-reference](Command Line Reference)

Thanks @spanzeri. I believe I read somewhere these options had been deprecated and no longer work. I tested this previously and found that to be the case, but I will check again. Appreciate the reply.

Hey, was this merge reversed at somepoint? I see the changes were merged directly into the master branch. But building the master branch doesn't have an option in the project settings and insepcting EditorSettings.xml also shows no such setting

The setting is now part of interface/editor/editor_screen as "Auto (Remembers Last Position)".

Excellent, found it. Thanks


Use license-header binary from bufbuild/buf

In my opinion, using the Go binary does not make sense in an idiomatic JS repo for a few reasons:

they don't typically use makefiles (generally regarded as an anti-pattern in the JS community)
Go shouldn't be a requirement
there is no npm package published for the Go binary

One option is to use https://www.npmjs.com/package/eslint-plugin-notice to keep the headers up to date for us, like this https://github.com/bufbuild/connect-query/pull/16
I propose following a similar paradigm for connect-web. wdyt @timostamm?

I propose following a similar paradigm for connect-web. wdyt @timostamm?

The license-header binary can update .proto files, and it can update generated code. We do both in connect-web and protobuf-es. eslint cannot do that.
I welcome a proposal that takes care of license headers with idiomatic tools, but it needs to include a way forward for the use cases that would cease to work.

The command-line tool license-header happens to be written in Go, just like other tools happen to be written in Rust, C++, etc. Command-line tools have no specific language that they should be written in.

I'm with @fyockm on this one, we've already got the eslint toolchain enforcing formatting, might as well let it enforce licensing as well. I noticed the old PR (#16) required changing format of the license so I suggest an alternative PR (https://github.com/connectrpc/connect-query-es/pull/124) that does not require changes to the license format and still uses eslint as a standard to supporting fixing and validation.

As I said earlier:

The license-header binary can update .proto files, and it can update generated code. We do both in connect-web and protobuf-es. eslint cannot do that.

I would much prefer we had a single way of enforcing license headers that works everywhere. We are actually not using the eslint toolchain to enforce formatting (see prettier), and the npm package addresses all of Drews concerns. It's already used here: https://github.com/bufbuild/jest-environment-jsdom

Oh man, i totally missed that comment. Yeah, generated code makes sense. Will adjust.

@paul-sachs, https://github.com/bufbuild/tools/pull/6 would give you the option to run license-header in the lint:code script:
https://github.com/connectrpc/connect-query-es/blob/635971e2cda5227dcfcf07eeaa7160ffe2c3ee48/package.json#L14-L15

@timostamm yeah, I'm just trying to figure out how best to customize the ignore paths. Passing in individual ignores for each is a little clunky. I might try to implement something in license-header that obeys the local .gitignore file. But for now, i'll work around it.

Ah, derp, the tool already does that. Awesome!


Improve documentation of get_resource() endpoint (#5900)

Pull request opened by the merge tool on behalf of #5900

Pull request rejected by merge tool. The tests for this branch did not succeed.

Merged into branches iso5 in 1c85f34da8bdde4b5af72dbc303042800af7dd2c


additional lookup options

might be worth doing something like this as well:
http://dns.squish.net/traverses/b1b3de535805c3962ccfa2584a389748/detail
or, a single-lookup page, which allows lookups of other data too?

This repository has been archived, either due to lack of time or no maintainers. If you are wondering why, please see "A note on archived repositories" documented on my profile. If this repository is un-archived in the future, this issue may be re-opened.


Add histogram toggle and toggle button component

Fixes: https://issues.redhat.com/browse/LOG-3212
This PR also adds the histogram date controls to the detail view so users can select a time range for specific resource logs
https://user-images.githubusercontent.com/5461414/198305746-061090ed-a29d-4fec-ad8d-1bcae8f0bc1e.mov

/lgtm


feat: add support for bitcoin_get_utxos to ic-mgmt

Motivation
Add support for bitcoin_get_utxos features to ic-mgmt library.
Documentation
https://internetcomputer.org/docs/current/references/ic-interface-spec#ic-bitcoin_get_utxos
Changes

expose bitcoin_get_utxos and bitcoin_get_utxos_query

Yes, couldn't develop it without. I should have mentioned it, sorry.


Enlarged text and changed colors on navbar

I made the colors for the navbar under our control, and made the text bigger, for usability.

Sorry for the delay in approving these, I won't do this again

Okay, I am using this PR to fix what went wrong with the navbar.

lit

Alright, I am merging this now.  After this, all changes get merged to develop, not master.  Master gets moved when you finish the auto-deployment doop.


Cannot parse 'Port' as int: strconv.ParseInt in K3s

Describe the bug
When running the container with minimal configuration the following error appears in Kubernetes (K3s):
* cannot parse 'Port' as int: strconv.ParseInt: parsing "tcp://<IP_ADDRESS>:5230": invalid syntax
panic: runtime error: invalid memory address or nil pointer dereference
[signal SIGSEGV: segmentation violation code=0x1 addr=0x30 pc=0xd7adfd]

Steps to reproduce
Deploy the app to K3s.
Screenshots or additional context
No response

cannot parse 'Port' as int: strconv.ParseInt: parsing "tcp://<IP_ADDRESS>:5230": invalid syntax panic: runtime error: invalid memory address or nil pointer dereference

It looks like you passed an invalid port parameter.

Same problem. I have try to change the service name to "memos-web", but it still doesn't work.

Just hit the same issue in my Kubernetes cluster. It's a bit unfortunate that Kubernetes default injected Environment Variable collide with memos environment variable.  I'd really appreciate it if the environment variable could be renamed in memos. Maybe name it MEMOS_HTTP_PORT instead?
Funny enough, I hit the same issue with Trillium as you can see here: https://github.com/zadam/trilium/issues/3177
A workaround would be to manually set MEMOS_PORT as an environment variable in your deployment. This will overwrite the default injected environment variables created by Kubernetes.

Changing deployment name didn't work for me, only changing Docker tag to :0.13.0 worked
For anyone who's coming here:
kubectl get deployment memos -o yaml > deployment.yaml -n memos

Edit :latest -> :0.13.0
kubectl apply -f deployment.yaml

But really the port variable should be changed since it's conflicting

Changing deployment name did the trick for me.

Similarly with this issue mattermost/mattermost-helm#228 (comment)
with the latest tag (0.14.2 at 23-07-26 utc), after I setting env in deployment like below, it works now:
Method 1: edit deployment, add
          env:
            - name: MEMOS_PORT
              value: "5230"

or:
Method 2: using cli to set env kubectl set env deployment/focalboard MEMOS_PORT=5230

it works for me.


chore: update version of the versions plugin

fixes: #400

@krishnapaparaju this is needed for the Spring Boot configmap boosters to work in OSIO. Can you have it merged ASAP?

Thanks!


Someone mentioned on Mastodon that some of the numbers here don't pass the smell test...

For example https://routes.tracestrack.com/ie/hiking/all/ lists 2 routes, and https://routes.tracestrack.com/gb/hiking/all/ lists 137.  When I run a query on a database with IE and GB in it I get 1614:
gis=> select count(*)  from planet_osm_line where highway = 'ldpnwn' and ref != '';
count
1614
(1 row)
The database population lua code is at https://github.com/SomeoneElseOSM/SomeoneElse-style/blob/master/style.lua#L9908 if you're curious, and it also includes IoM, Jersey, Guernsey etc., and also some semicolon-separated networks like "lwn;lcn;lhn".  That doesn't account for the different between 139 and 1614 though?
Are you perhaps missing some, or am I misunderstanding what you're listing?

Hi! I think there is some misunderstanding indeed. The project lists route relations rather than plain ways. So the table should be planet_osm_rels instead of planet_osm_line. The SQL that generates the hiking json list is this: https://github.com/tracestrack/osm-routes-index/blob/main/generator/dump_hiking_routes.sql
As you can see, it queries relations where route is either foot' or 'hike.

The project lists route relations rather than plain ways

Sorry - I should have made it clearer.  My "1614" number above is route relations.

hike

Not sure if that's a typo, but https://taginfo.openstreetmap.org/keys/route#values suggests "hiking"

Ah! Right🙈 not hike but hiking! I'll update the query and hopefully it'll match your numbers!

hopefully it'll match your numbers!

It'll be closer - overpass suggests 146 rather than 137 (subtracting the 2 in Ireland.
You might want to consider your assumption that "all trails have a ref" as well...

There are now 2890 at https://routes.tracestrack.com/gb/hiking/all/ - much more plausible!


plugins/edx-app-gradle-plugin not found in root project

Hello everyone!
I'm struggling with a compilation issue on Android Studio.
I am using the latest version 3.6.1.
Could you please advise?
Thanks!

Are you using the latest master branch?

Also, try doing this:

Delete all the files and folder inside buildSrc folder except the following 3 files:
settings.gradle
build.gradle
.gitignore

and then Rebuild the project

The issue seems to be resolved.
Plus we are using the current latest version Android Studio Chipmunk | 2021.2.1.


fixes broken Dockerfile

debian:stretch has been archived: https://lists.debian.org/debian-devel-announce/2023/03/msg00006.html
I'm applying this fix: https://unix.stackexchange.com/a/743865

We could probably also just update the debian version, but merging this to fix it for now.


Support for additional attributes

It would be nice, to have additional parameters for Add-IntuneWin32App command, so the following attributes can be submitted:

Category
Information URL
Display app in company portal
Privacy URL
Developer
Owner
Comments

Will be added to 1.2.0. Thanks for reminding me about those.


Mysterious 2x perf regression on GEMM

With no code or hardware change at all, after month there is a 2x perf regression, OpenBLAS also is a bit slower (with no package update):
A matrix shape: (M: 1920, N: 1920)
B matrix shape: (M: 1920, N: 1920)
Output shape: (M: 1920, N: 1920)
Required number of operations: 14155.776 millions
Required bytes:                   29.491 MB
Arithmetic intensity:            480.000 FLOP/byte
Theoretical peak single-core:    224.000 GFLOP/s
Theoretical peak multi:         4032.000 GFLOP/s
Make sure to not bench Apple Accelerate or the default Linux BLAS.

 (MKL-DNN is linked to Intel OpenMP)

And changing the order can slow down OpenBLAS as well
$  nim cpp -r -d:release -d:openmp -d:danger --outdir:build benchmarks/gemm/gemm_bench_float32.nim
Hint: used config file '/home/beta/.choosenim/toolchains/nim-1.0.2/config/nim.cfg' [Conf]
Hint: used config file '/home/beta/Programming/Nim/laser/nim.cfg' [Conf]
Hint: operation successful (340 lines compiled; 0.025 sec total; 5.754MiB peakmem; Dangerous Release Build) [SuccessX]
Hint: /home/beta/Programming/Nim/laser/build/gemm_bench_float32  [Exec]

A matrix shape: (M: 1920, N: 1920)
B matrix shape: (M: 1920, N: 1920)
Output shape: (M: 1920, N: 1920)
Required number of operations: 14155.776 millions
Required bytes:                   29.491 MB
Arithmetic intensity:            480.000 FLOP/byte
Theoretical peak single-core:    224.000 GFLOP/s
Theoretical peak multi:         4032.000 GFLOP/s
Make sure to not bench Apple Accelerate or the default Linux BLAS.

Laser production implementation
Collected 10 samples in 0.071 seconds
Average time: 6.416 ms
Stddev  time: 1.526 ms
Min     time: 5.861 ms
Max     time: 10.753 ms
Perf:         2206.263 GFLOP/s

OpenBLAS benchmark
Collected 10 samples in 0.151 seconds
Average time: 14.448 ms
Stddev  time: 10.255 ms
Min     time: 9.415 ms
Max     time: 37.410 ms
Perf:         979.779 GFLOP/s


TrackMate 3 should ship branch features natively.

In TrackMate 2 you can only get features for spots, edges and tracks.
But a track is a connected component of a graph (a sub-graph). For instance, a cell followed up through all its divisions makes a single track. End users desire features to be calculated for branches, because they typically match a biological object (e.g. a daughter cell before it divides).
For this, we need a way to access, store and compute branches, which is another issue.

Fixed with #160


If the page param is invalid, default it

If the page param is invalid, default it instead of passing on the bad value.
A spambot might send page=abc to your pages, and we should handle that better.  Even page=0 will fail server-side.

This validation should be done on the user's side.


Added support for get time API.

Ready for Review

@omab can you please review this when you have a moment?

Requested Changes are Completed. Ready for Review

@sankarsubramanian I've approved the PR.
About the print statements, while this is correct:
print("foo" + "bar")

the preferred way is to use the format method:
print("foo {0}".format("bar"))


Custom ES config

Support ES basic auth
Support ES https protocol
Support ES custom port

Hey this is very nice, thanks!


Minor javascript typo

Unless I'm sorely mistaken, there are a few colons missing from this example code block... :-)

Added another minor grammar typo.

Thanks :)


chore: bdd-sbhg-1587451650 to 0.0.1

chore: Promote bdd-sbhg-1587451650 to version 0.0.1

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approvers:
If they are not already assigned, you can assign the PR to them by writing /assign  in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


PotreeConvert to convert abnormal color

Hello,
I am a software developer from China and have encountered some problems in using the Potree framework.
We use FARO's scanning device to shoot and use Faro Sence to export files in .xyz format.
We use PotreeConvert to convert .xyz to a bin file.
The conversion commands are as follows:
PotreeConverter.exe D:\faro\wuxu\SCENE_Scan_087.xyz -o C:\faro\PotreeConverted -p leida --overwrite --material RGB
But when loading, the color is abnormal.
We have not found a solution on [github.](
Please inform us about the treatment, thank you.

Same problem here. Did you find a solution?
Thank you


have addSignalFunction support tuples or lists

The snippet below fails as it requires a single float as output:
def rpyFunction(msg):
quat = quat_to_euler([msg.pose.rotation.w, msg.pose.rotation.x, msg.pose.rotation.y, msg.pose.rotation.z])
rpy = (quat)
return msg.utime, rpy
addSignalFunction('EST_ROBOT_STATE', rpyFunction)

We could definitely do that.  Note, you can also do something like this:
addSignalFunction('EST_ROBOT_STATE', lambda x: rpyFunction(x)[0], label='roll')
addSignalFunction('EST_ROBOT_STATE', lambda x: rpyFunction(x)[0], label='pitch')
addSignalFunction('EST_ROBOT_STATE', lambda x: rpyFunction(x)[0], label='yaw')
This might work fine for most purposes, but it will be slightly less efficient since it calls the quat_to_euler conversion three times.
We could definitely add new support for multi component signals, and that might introduce some nice features and generalities.  You could then add support for applying common functions to the signals, conversions that are commonly used like quat-to-rpy, magnitude, component filters, etc.

Note, you will need to pull signal-scope master in order to get the "label" keyword argument that I used in my previous reply.

Oh wait, the example I posted won't quite work, don't try it.  I'll post a working version in a minute.

Yeah it would have to be another function wrapper like:

def extractComponent(func, idx):
    def f(msg):
        t, v = func(msg)
        return t, v[idx]
    return f

def rpyFunction(msg):
    quat = quat_to_euler([msg.pose.rotation.w, msg.pose.rotation.x, msg.pose.rotation.y, msg.pose.rotation.z])
    rpy = (quat)
    return msg.utime, rpy

addSignalFunction('EST_ROBOT_STATE', extractComponent(rpyFunction, 0), label='roll')
addSignalFunction('EST_ROBOT_STATE', extractComponent(rpyFunction, 1), label='pitch')
addSignalFunction('EST_ROBOT_STATE', extractComponent(rpyFunction, 2), label='yaw')


[TLUL agent] Support to drop VALID without READY

According to TL spec, we need to have these 2 supports, which are not implemented in our agent

VALID can't be dropped even READY is low (I'm already working on this)
sender can change the contents of transition when it's not accepted, and we only consider receiver accept the item when both VALID and READY are high.

Note that a sender may raise valid and then lower it on the following cycle, even if the message was
not accepted on the previous cycle. For example, the sender might have some other higher priority
task to perform on the following cycle, instead of trying to send the rejected message again.
Furthermore, the sender may change the contents of the control and data signals when a message
was not accepted.

If ready is LOW, the receiver must not process the beat and the sender must not consider the
beat processed.

Related to #3354

@weicaiyang can we close this one? it looks like the enhancement is already merged.

@weicaiyang can we close this one? it looks like the enhancement is already merged.

the item 1 is done. I'm abort to submit another PR for item 2. Will close it once that PR is merged.

thanks!
On Thu, Sep 24, 2020 at 1:55 PM weicaiyang<EMAIL_ADDRESS>wrote:

@weicaiyang https://github.com/weicaiyang can we close this one? it
looks like the enhancement is already merged.
the item 1 is done. I'm abort to submit another PR for item 2. Will close
it once that PR is merged.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/lowRISC/opentitan/issues/3383#issuecomment-698583468,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAH2RSWI6AFAOVKYE5W235TSHOW3DANCNFSM4RAMDPYQ
.

close via #3478 and #3585