[PublishTestResultsV2] Add option to fail task if no test result files are found

Task name: PublishTestResultsV2
Description:
Adds an option that lets the user configure the task to fail in case no
test result files have been found. This serves as a safeguard to make
sure that the test-run task and this task has been configured properly
to find the test result files.
We are defaulting to not failing the task on missing files, to keep current behavior.
Documentation changes required: Y
I guess this option should be documented.
Added unit tests: N
Not sure how this should be tested, as the "Warning" functionality doesn't seem to be tested either?
Attached related issue: Y
Solves microsoft/azure-pipelines-tasks#13275
Checklist:

[ ] Task version was bumped - please check instruction how to do it
[ ] Checked that applied changes work as expected

I have no idea if this works, or how to test it. I have ran the build and existing tests for the task, but haven't manually tested the task anywhere (and have no idea how anything here works...).

Soooo is this going somewhere?

Why is that a problem to review this nice improvement for 7 months !?

@shailesh-sk could you do a review on this PR?

@shailesh-sk Could you do a review on this PR.
Or maybe some of the more active contributers could do a review?
Perhaps @anatolybolshakov or @nadesu @v-jkarri
Merging this would allow closing https://github.com/microsoft/azure-pipelines-tasks/issues/13275 and https://github.com/microsoft/azure-pipelines-tasks/issues/13275

reping here since it's a really really useful feature! could anyone go through the code review?

Could we merge this essential feature any time soon?

@the-rule I can't really do anything about it, its up for the maintainers to merge it.
To me its infuriating that such a simple change, a feature that exists in pretty much all CI systems, has not been merged yet, and has been actively ignored for over a year.

@shailesh-sk ping-ping

To me its infuriating that such a simple change, a feature that exists in pretty much all CI systems, has not been merged yet, and has been actively ignored for over a year.

Totally agree, to me this change is not even a feature but a bug fix!

@shailesh-sk I would completely agree with @the-rule !

Maybe anyone else who can review this PR? @DenisRumyantsev?
Very strange that such a small improvement is stuck on review.

@navin22 can you maybe help? This is such a small PR with incredibly added value...

@navin22 or @rasunkar is there no one to be found?

Beep boop

/azp run

/azp run

Merged in Pull request: https://github.com/microsoft/azure-pipelines-tasks/pull/18931
This would be available in version 2.228.0


feat: Update install command and font.

This uses Consolas to override the @, {, }, (, and ) glyphs in Space Mono because they look kind of weird.
I also updated the install command on the homepage and disabled font ligatures in code examples.
Why Consolas? I tried a bunch of different fonts and this was the only one where the weight of the characters looked similar to Space Mono and didn't alter the vertical spacing too much.
This affects:

The installation command on the homepage
Code examples in blog posts
Playground

This does not affect documentation. We will have to make a similar change in the eslint repo for that.
Fixes #281
Closes #254

Flickers appear on initial load and on playground input. They first become Space Mono and then Consolas.

Does this happen again when you close the tab and revisit the website in a new tab?

Yes, it can be steadily reproduced.

I think we can probably fix this by prefetching the font file. I’ll look into that.

Okay, I think this fixes the problem. I'm not seeing it locally, but please verify on the deploy preview.
And thanks @kecrily for catching this!

Looks like something went wrong during the rebase, there are a lot of changes now.

Hmm, yes it did. Very strange. I'll take a look.

Okay, got it straightened out.


Allow multiple examples at the item level to allow for reciprocal tests

First up, thanks for creating this great tool!
I have a Postman collection that has the following basic product collection:

GET /products
GET /product/:id
POST /products

At the moment, I believe postman-to-openapi uses the examples to create the schema. The issue is that for the POST endpoint, I have two versions for the 400 and 200 use cases with separate tests and examples. Unfortunately, you can't assign tests to examples so this is the only way to ensure that both response codes are tested (e.g. using Newman CLI).
See this example collection to demonstrate the point:
{
	"info": {
		"name": "Example Products API",
		"schema": "https://schema.getpostman.com/json/collection/v2.1.0/collection.json"
	},
	"item": [
		{
			"name": "POST /products/ (200)",
			"event": [
				{
					"listen": "test",
					"script": {
						"exec": [
							"pm.test(\"Status code is 200\", function () {",
							"    pm.response.to.have.status(200);",
							"});",
							""
						],
						"type": "text/javascript"
					}
				}
			],
			"request": {
				"method": "POST",
				"header": [
					{
						"key": "Content-Type",
						"type": "text",
						"value": "application/json"
					}
				],
				"body": {
					"mode": "raw",
					"raw": "{\n    \"id\": \"09\",\n    \"name\": \"Gem Visa\",\n    \"type\": \"CREDIT_CARD\",\n    \"price\": 99.99,\n    \"version\": \"v1\"\n}"
				},
				"url": {
					"raw": "{{host}}/products",
					"host": [
						"{{host}}"
					],
					"path": [
						"products"
					]
				},
				"description": "Create a product"
			},
			"response": [
				{
					"name": "POST /products/ (200)",
					"originalRequest": {
						"method": "POST",
						"header": [
							{
								"key": "Content-Type",
								"type": "text",
								"value": "application/json"
							}
						],
						"body": {
							"mode": "raw",
							"raw": "{\n    \"id\": \"09\",\n    \"name\": \"Gem Visa\",\n    \"type\": \"CREDIT_CARD\",\n    \"price\": 99.99,\n    \"version\": \"v1\"\n}"
						},
						"url": {
							"raw": "{{host}}/products",
							"host": [
								"{{host}}"
							],
							"path": [
								"products"
							]
						}
					},
					"status": "OK",
					"code": 200,
					"_postman_previewlanguage": "json",
					"header": [
						{
							"key": "X-Powered-By",
							"value": "Express"
						},
						{
							"key": "Access-Control-Allow-Origin",
							"value": "*"
						},
						{
							"key": "Content-Type",
							"value": "application/json; charset=utf-8"
						},
						{
							"key": "Content-Length",
							"value": "79"
						},
						{
							"key": "ETag",
							"value": "W/\"4f-7upA8VUHobjcwMU2JZU+mGYVfEo\""
						},
						{
							"key": "Date",
							"value": "Sun, 09 May 2021 12:29:53 GMT"
						},
						{
							"key": "Connection",
							"value": "keep-alive"
						}
					],
					"cookie": [],
					"body": "{\n    \"id\": \"09\",\n    \"name\": \"Gem Visa\",\n    \"type\": \"CREDIT_CARD\",\n    \"price\": 99.99,\n    \"version\": \"v1\"\n}"
				}
			]
		},
		{
			"name": "POST /products/ (400)",
			"event": [
				{
					"listen": "test",
					"script": {
						"exec": [
							"pm.test(\"Status code is 400\", function () {",
							"    pm.response.to.have.status(400);",
							"});",
							""
						],
						"type": "text/javascript"
					}
				}
			],
			"request": {
				"method": "POST",
				"header": [
					{
						"key": "Content-Type",
						"type": "text",
						"value": "application/json"
					}
				],
				"body": {
					"mode": "raw",
					"raw": "{\n\n}"
				},
				"url": {
					"raw": "{{host}}/products",
					"host": [
						"{{host}}"
					],
					"path": [
						"products"
					]
				},
				"description": "Create a product"
			},
			"response": [
				{
					"name": "POST /products/ (400)",
					"originalRequest": {
						"method": "POST",
						"header": [
							{
								"key": "Content-Type",
								"type": "text",
								"value": "application/json"
							}
						],
						"body": {
							"mode": "raw",
							"raw": "{\n\n}"
						},
						"url": {
							"raw": "{{host}}/products",
							"host": [
								"{{host}}"
							],
							"path": [
								"products"
							]
						}
					},
					"status": "Bad Request",
					"code": 400,
					"_postman_previewlanguage": "json",
					"header": [
						{
							"key": "X-Powered-By",
							"value": "Express"
						},
						{
							"key": "Access-Control-Allow-Origin",
							"value": "*"
						},
						{
							"key": "Content-Type",
							"value": "application/json; charset=utf-8"
						},
						{
							"key": "Content-Length",
							"value": "29"
						},
						{
							"key": "ETag",
							"value": "W/\"1d-pnhbRSD4NZML3cnaJuyottC+RiE\""
						},
						{
							"key": "Date",
							"value": "Sun, 09 May 2021 12:53:30 GMT"
						},
						{
							"key": "Connection",
							"value": "keep-alive"
						}
					],
					"cookie": [],
					"body": "{\n    \"message\": \"invalid product\"\n}"
				}
			]
		}
	]
}

It will generate the following OAS (note the response body for 200 and 400 is missing):
openapi: 3.0.0
info:
  title: Example Products API
  version: 1.0.0
servers:
  - url: undefined://{{host}}
paths:
  /products:
    post:
      tags:
        - default
      summary: POST /products/ (200)
      description: Create a product
      requestBody:
        content:
          application/json:
            schema:
              type: object
              example:
                id: '09'
                name: Gem Visa
                type: CREDIT_CARD
                price: 99.99
                version: v1
      parameters:
        - name: Content-Type
          in: header
          schema:
            type: string
          example: application/json
      responses:
        '200':
          description: Successful response
          content:
            application/json: {}

As I started debugging, I realised that whilst it would correctly determine the request body, I could never get it to create a response body. Looking into the test examples in the repository, I can't see any that have example response bodies - are these supported?

Hi @mefellows ,
Yea, you are right, the use of postman example  is still not supported by the library sorry, I have this in mind for schema see #22 but could be also useful for response examples in OpenAPi. I will keep this issue for that task.

Added support for postman example parsing in version 1.3.0

Awesome - thanks for that, I'll check it out next week!


Layers support

I propose a layers feature that would allow to display multiple tile maps at the same time. Higher layers would use images with alpha so that the lower layers are still visible. Individual layer opacity would also be configurable.
Use cases

Normal tile layer for a world map + weather tile layer with clouds that renders on top
Ground level tile layer + layer for every architectural level on top
Server-side data overlays over the base map
Animated switching of map types/styles (add both maps as 2 layers, animate alpha, remove old layer)

Usage
From the API perspective, this could be a version/replacement of TileStreamProvider that returns multiple tiles (List<InputStream>). There would also need to be a way to control a layers' alpha; this could be either defining the layers and their alpha separately, or as part of the TileStreamProvider.
There is also a case to be made that this is something the client code could do, and not a library feature: blend the multiple tiles yourself and provide the composite tile to TileStreamProvider. The issue with that with the current API is that TileStreamProvider wants an InputStream, which is difficult to work with if the client code was to generate tiles dynamically. Presumably the tiles were already read from whatever InputStreams and blended into the final Bitmap, so TileStreamProvider would need to work with Bitmaps, not InputStreams.
It would definitely be nicer as a proper library feature though.
For reference, here is the feature in Google Maps SDK: https://developers.google.com/maps/documentation/android-sdk/tileoverlay
In OsmDroid: https://osmdroid.github.io/osmdroid/Map-Sources.html (Can I use more than one tile source at a time? section)
What do you think?

I have implemented layering in my trekking app, using a custom TileStreamProvider. Have a look at https://github.com/peterLaurence/TrekMe/blob/master/app/src/main/java/com/peterlaurence/trekme/core/providers/stream/TileStreamProviderOverlay.kt
In this implementation, the tile from the primary layer is fetched, then loaded into a Bitmapn itself drawn into a Canvas. For each overlay layers, bitmaps are drawn onto that bitmap. Then an InputStream is made from the resulting bitmap (don't pay too much attention to the custom TileResult type -- it's just a wrapper).
Before continuing the discussion, let me know what you think of this approach.

That's what I envisioned with the client-side implementation, great to see a working example! But it seems to have the problem I talked about with TileStreamProvider requiring an InputStream. You have to compress the Bitmap to a PNG, so then the library has to decode it back again. Surely that's not great for performance?

Well, if that's not done client side, the exact same code will be done by the library. Unless I am missing something.
The fact that the TileStreamProvider returns an InputStream isn't a problem here. If it returned a Bitmap (which means that it would already be allocated), another Bitmap instance has to be allocated anyway to draw it onto the previous one.
Again, unless I am missing something.
BTW, there's a good reason why TileStreamProvider returns an InputStream and not a Bitmap. TileCompose internally use Bitmap pooling to optimize allocations. That implfies that Bitmap allocations aren't user managed.
Which.. bring me to the conclusion that some library optimizations could be done to avoid that. Yes.

The Bitmap allocations must happen either way, yes, but what I mean is that after you have the final Bitmap, you are compressing it to PNG and then the library has to decodeStream it back into a Bitmap. I think that compression-decompression step is unnecessary.

Oh yes the compressing thing is propably not necessary. I don't remember why it's there.

FYI : https://www.youtube.com/watch?v=_ioFW3cyRV0
I believe the library could do that no only for a single layer, but potentially with multiple TileStreamProvider.

Although in case of implementing layers in the library, I was thinking of calling the drawing code for every layer, not creating a composite Bitmap and then drawing it. So draw both Bitmaps into the final Canvas, without an intermediate per-tile Canvas .

Although in case of implementing layers in the library, I was thinking of calling the drawing code for every layer, not creating a composite Bitmap and then drawing it. So draw both Bitmaps into the final Canvas, without an intermediate per-tile Canvas .

Not only that, but also reuse as many as Bitmap instances as necessary.

Also that, yes!

Alright. I'll have stab at this one. It will impact the core of the library.

I remember why there's a PNG compression. The resulting stream isn't used for MapCompose, but for making PNGs image files.
Those files (tiles) are later read to show the map on screen, so all the layering thing is already done because every tiles are made of several blended images. On the good side, it's impossible to be more performant. The downside is that it's not dynamic. It's impossible to dynamically change the opacity of a layer.
So that's the approach I had in my app.

I see. For my use case I'd definitely need per-layer opacity control. I want the user to be able to control the layers they enable and their opacity (think current and historical map layer, with user being able to control which one they see and how much, to compare them). I don't have caching implemented yet, but I'd cache the tile files separately, so I would also be able to avoid the additional compression step when showing the map, if the library allowed it.

That's clear now. A library support is necessary when things need to be dynamic.

I've started thinking about it, and realized it'll require major changes to the core. However, I'd like to finish other work before diving into this. Can it wait a few weeks?

I'm sure I can reorganize my to do list to work on other app features than the one I need the layers for. : ) Wouldn't want to put any pressure on your voluntary work, the library is a huge help to me as it is!

I was commenting on commits to catch them before they appear on master, ideally I'd comment on a PR, if there is going to be one.

No problem! For the moment it's still WIP (sometimes, I revert something I did before). It's a bit too soon for a global code review, but thanks for your early comments. It helps to know your expectations early.

The plan is to finish the APIs, make a dedicated demo, and let you know when you can evaluate this layer branch. Once I get you feedback, I'll make necessary changes (if any). Then, I'll create a v2 branch from master and make breaking changes like the suspending TileStreamProvider, update compose to 1.1.0 (beta), etc.
I believe layers will land in 2.0.0

The layers can now be tested. For the moment, don't use addLayer api (I haven't tested it yet). The others worked per my tests.
For each individual issues that may (ok.. will) arise, we will create separate issues. For example, I'm expecting that you get the blank screen again. However, for this one, I have a fix candidate very similar to what you've done on master (it's a shared flow, but a replay of 1). We'll see.
For all other issues, it's import to try as hard as we can to reproduce on the demo.

The first issue, like you said, is the blank screen. This is related to scale. No tiles are drawn when scale >= 1. If I initialize the map with scale < 1 it works from the start, but as soon as it scaled in past 1 tiles disappear.
Can you reproduce this one or should I try to investigate / reproduce on the demo app?

I can't reproduce the scale issue in the demo. In the demo, it scales up to 2f (the max scale by default). Even increasing the max scale to 4, it works in the demo.

Ok I'm creating an issue.

Here's my suggestion for the layers API changes we discussed earlier.
Instead of having to specify an id when adding a layer, the id is created by the library using an UUID. The created id is returned by the API (let's say addLayer).
This id is important to keep, because it's necessary to use other APIs such as moveLayerUp, moveLayerDown, etc.
The consequence of this, which isn't a big deal but worth to notice since it's a breaking change : it's no longer required to pass a TileStreamProvider to the MapState constructor (because the user may need to know the id of this implicitly created layer).
That way, there's concept of "primary layer". Everything is a layer.
The library will apply default rules like :

The lowest layer has an opacity of 1 and cannot be changed dynamically
Removing layers defaults to removing all layers except the lowest one

Note that any layer can become the lowest one dynamically.
A new API would be handy: replaceLayer(id). Which would be the equivalent of the current setPrimaryLayer (if of course the id corresponds to the lowest layer).

I like this much more!

This is done. I'll now create a separate testing app.


Picking audio changes extension to mpeg

Whenever I pick an audio using AudioPicker, The file extension becomes .mpeg even though the original file is in .mp3 format

I have the same problems :(

I am having the same issue.
Device: Pixel 3a
OS: Android 10 (Q)


Getting Started

Hey Id love to give this a try but im having trouble using the accessibilityImage .
when I use SnapshotTesting's assert:
 func testExample() throws {
        assertSnapshot(of: ContentView(), as: .image)
}

that takes a snapshot.
But when I try to use the accessibilityImage strategy :
func testExample() throws {
        assertSnapshot(of: ContentView(), as: .accessibilityImage)

    }

I get compile error:
"Type of expression is ambiguous without a type annotation"
is this trying to use SnapshotTesting's assert  rather than AccessibilitySnapshot's?

Hey @LostStudent, that error means the compiler can't resolve the method signature. The assertSnapshot method is defined by SnapshotTesting, while the accessibilityImage strategy is defined by AccessibilitySnapshot.

How are you installing the frameworks? (What package manager, and what configuration options within that package manager)
Are you importing both frameworks in this test class file? What is the import statement for AccessibilitySnapshot?

ah adding a hosting controller works.
func testExample() throws {

        let host = UIHostingController(rootView: ContentView())
        host.view.frame = .init(x: 0, y: 0, width: 100, height: 100)
        assertSnapshot(
            matching: host,
            as: .accessibilityImage(showActivationPoints: .always,
                                    drawHierarchyInKeyWindow: true),
            named: "shot"
        )

    }

should the previous usage have worked?
My test project imports AccessibilitySnapshot using swift package manager in xcode.
AccessibilitySnapshot is added as a dependency of the test target
the imports in my test file are:
import XCTest
import SnapshotTesting
@testable import accessSnaps
import SwiftUI
import AccessibilitySnapshot

Ahh, that's expected. Try updating to version 0.7. Prior versions didn't support snapshotting SwiftUI views directly, requiring a hosting controller instead.


feat: add more entry to open terminal

Types

[x] 🎉 New Features

Background or solution
Close: #664
Changelog
add more entry to open terminal

Codecov Report

Merging #686 (4d8f7e3) into main (78dedf2) will decrease coverage by 0.00%.
The diff coverage is 77.77%.

@@            Coverage Diff             @@
##             main     #686      +/-   ##
==========================================
- Coverage   58.63%   58.63%   -0.01%
==========================================
  Files        1209     1209
  Lines       74734    74804      +70
  Branches    15505    15529      +24
==========================================
+ Hits        43823    43864      +41
- Misses      28174    28203      +29
  Partials     2737     2737

Impacted Files
Coverage Δ

packages/i18n/src/common/en-US.lang.ts
100.00% <ø> (ø)

packages/i18n/src/common/zh-CN.lang.ts
100.00% <ø> (ø)

...-next/src/browser/contribution/terminal.command.ts
0.00% <0.00%> (ø)

...xt/src/browser/contribution/terminal.keybinding.ts
0.00% <0.00%> (ø)

packages/core-browser/src/common/common.command.ts
100.00% <100.00%> (ø)

packages/core-browser/src/style/icon/icon.ts
100.00% <100.00%> (ø)

...s/extension/src/browser/vscode/builtin-commands.ts
100.00% <100.00%> (ø)

packages/terminal-next/src/common/commands.ts
100.00% <100.00%> (ø)

packages/addons/src/index.ts
0.00% <0.00%> (-100.00%)
:arrow_down:

packages/extension/src/node/index.ts
0.00% <0.00%> (-57.90%)
:arrow_down:

... and 51 more

Continue to review full report at Codecov.

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


⚠ PrivateHD has degraded performance

In 6164db7, PrivateHD ($PHD) experienced degraded performance:

HTTP code: 200
Response time: 2339 ms

Resolved: PrivateHD performance has improved in b044841.


IPv4 addresse of new interface is added to the DNS A record of OPNsense FQDN

Describe the bug
When I create a new VLAN interface and select it in Unbound for listenig for DNS querys, the IPv4 address of the new interface is added to the DNS A record of the OPNsense FQDN.
Therefore the access to the OPNsense WebGUI over the FQDN timed out sometimes, because Unbound delivere the IP address of the new interface on which the WebGUI is not listening.
Already existing VLAN interfaces are not affected.
To Reproduce
Steps to reproduce the behavior:

Create a new VLAN interface
Select it in Unbound to listening for DNS querys
Do a nslookup for the FQDN of the OPNsense appliance

Expected behavior
The OPNsense FQDN should only point to one IP address.
Screenshots
NSLOOKUP for OPNsense FQDN
C:\Users\xxx>nslookup jaswall.mgmt.home.arpa
Server:  JasWall.jasnet.home.arpa
Address:  <IP_ADDRESS>

Name:    jaswall.mgmt.home.arpa
Addresses:  <IP_ADDRESS>
          <IP_ADDRESS>
C:\Users\xxx>

<IP_ADDRESS> (re1) is the correct address for my OPNsense.
<IP_ADDRESS> (re1.20) is the new VLAN interface.
Relevant log files
Not sure which log files could help you.
Additional context
I had an similar issue with OpenVPN last year (see https://github.com/opnsense/core/issues/2828). Maybe this helps you.
Environment
OPNsense 19.1.9-amd64
Installed on self-configured hardware (Intel(R) Celeron(R) CPU J1900 @ 1.99GHz (4 cores))

Technically this is more a feature request than a bug, all interface addresses are recorded in the local dns, which is correct, but might not suite your use-case. A solution could be to provide the option to
disable these automatic interface rules so you can add the one you need manually.
Logically, it should need an option and an additional check around here https://github.com/opnsense/core/blob/31a81c51a6d76b44439acf4fdcb186075cab0e79/src/etc/inc/plugins.inc.d/unbound.inc#L527

That means I should use the IP address instead of the FQDN to access the WebGui? That's OK for me, but I'm interested if this is the recommend way.
Also why only new adapter register in DNS for the OPNsense FQDN? I've two other VLAN interfaces and one WAN interface which are not listed when I resolve the FQDN via nslookup.
Thank you for your support!

If I'm not mistaken, all non local (and non ovpn) interfaces are registered in /var/unbound/host_entries.conf. What I was suggesting is not switching to ip, but adding an option to stop automatic generation so you can add the entry you actually want to use (instead of registering all).
Thinking about this a bit more, you can also add another (static) host entry, which you then use to access the webgui. This would be the easiest option and doesn't require any changes to the system.

Thinking about this a bit more, you can also add another (static) host entry, which you then use to access the webgui. This would be the easiest option and doesn't require any changes to the system.

Yep, that's a cool idea :)
But the option to choose which address will registered for the OPNsense FQDN would be also nice.

you should already be able add a manual entry (different hostname) in Services -> Unbound DNS -> Overrides

I run to the same issue as well.
I hostname of opnsense is "fw1.internal.domain.com".
On management interface I activated snmp and added it to the monitoring server.
A few minutes later the devices was down and I realized that the problem is the same: fw1.internal.domain.com points to all interface ip's exept the WAN.
Problem is:

I've created a override in unbound for fw1.internal.domain.com and there are still all IPs in DNS.
I've also selected to only activate the "WebGUI" on management interface. So there should be no reason to add the other IPs to DNS I think.

Is there any workaround for this exept to use another DNS name for FW?

This issue has been automatically timed-out (after 180 days of inactivity).
For more information about the policies for this repository,
please read https://github.com/opnsense/core/blob/master/CONTRIBUTING.md for further details.
If someone wants to step up and work on this issue,
just let us know, so we can reopen the issue and assign an owner to it.


nest: 3.4 -> 3.5

Semi-automatic update generated by nixpkgs-update tools.
meta.description for nest is: NEST is a command line tool for simulating neural networks
meta.homepage for nest is: https://www.nest-simulator.org/
meta.changelog for nest is: https://github.com/nest/nest-simulator/releases/tag/v3.5
Updates performed

Version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 3.5 with grep in /nix/store/jh0j81rk8j1dk69z4wq5qkmns241nf4v-nest-3.5
found 3.5 in filename of file in /nix/store/jh0j81rk8j1dk69z4wq5qkmns241nf4v-nest-3.5

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

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
nest

Instructions to test this update (click to expand)

Build yourself:
nix-build -A nest https://github.com/jiegec/nixpkgs/archive/fa8067ad4f69e54492d7bd86830a2970712633fb.tar.gz

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

Pre-merge build results
NixPkgs review skipped

Maintainer pings
cc @jiegec @CyborgPotato for testing.

Superseded by #242302


Create a channel for dotnet/tye on Discord DotNetEvolution

What should we add or change to make your life better?
is there any chance to see a tye channel on DotNetEvolution Discord ?
quote from gitter:

The fancy URL for the new discord is https://aka.ms/dotnet-discord

Why is this important to you?
Sometimes I have the feeling that asking a question might not be asked on github or some question about an ongoing review
I'm not sure it's actually possible, just curious
eg: there's an #AspNet channel but I'm not sure it's being used

ping @migueldeicaza

Done ;)


[Fix #3248] Support 'ruby-' prefix in .ruby-version

[x] Wrote good commit messages.
[x] Commit message starts with [Fix #issue-number] (if the related issue exists).
[x] Used the same coding conventions as the rest of the project.
[x] Feature branch is up-to-date with master (if not - rebase it).
[x] Squashed related commits together.
[x] Added tests.
[x] Added an entry to the Changelog if the new code introduces user-observable changes. See changelog entry format.
[x] All tests are passing.
[x] The new code doesn't generate RuboCop offenses.
[x] The PR relates to only one subject with a clear title
and description in grammatically correct, complete sentences.

I’m having the same problem this is intended to solve, but I’m using JRuby, so my prefix in .ruby-version is jruby. It doesn’t seem like this change will fix the problem for JRuby users.

What's the right thing to do for JRuby with regard to .ruby-version? Is it to ignore .ruby-version and always use TargetRubyVersion from the configuration?

@aviflax The problem isn’t that the prefix is jruby – the regex will match that just fine. I guess the problem rather is that the jruby version numbers don’t match the MRI version numbers. If you’re using the most recent version of jruby, RuboCop will now set target_ruby_version as 9.1, while its KNOWN_RUBIES are limited to [1.9, 2.0, 2.1, 2.2, 2.3]. That may be a problem indeed.

What's the right thing to do for JRuby with regard to .ruby-version? Is it to ignore .ruby-version and always use TargetRubyVersion from the configuration?

I think we should switch the precedence, so .ruby-version only gets read if TargetRubyVersion isn’t configured.

@bquorning yes, good point. My file’s current value is jruby-<IP_ADDRESS> which is perfectly valid for rbenv.

I think we should switch the precedence, so .ruby-version only gets read if TargetRubyVersion isn’t configured.

+1 makes sense to me.
But also, IIRC until this change, Rubocop probably had some kind of sensible default if TargetRubyVersion isn’t configured? So it seems extreme for Rubocop to just die if it doesn’t understand the value in .ruby-version. I’d think it’d be better for it to emit a warning and just continue with a default setting.

I opened https://github.com/bbatsov/rubocop/pull/3261 if that helps to resolve the situation.


Hsbc hk fixes and package

Hi Olivier, as discussed.  Now logs on to HSBC HK without a problem and made a proper pip installable package.  Good to test somewhere else though

There are a lot of things I like in your code!
However, changes are easier to deal with when they are incremental.
Since you made a single pull request, I need to cover everything before I can accept all.
Also, I would reject some commits.
Give me some time to cover the whole code and to learn some parts I am missing.


Remove dynamic map names in Cilium agent

This is a major prerequisite for https://github.com/cilium/cilium/issues/29216 and will allow us to pre-build an ELF containing all of the agent's map definitions.
This can then be used to pull map BTF out of at runtime, so agent-managed maps can have value pretty-printing and use new BPF features that require map BTF. (notably spinlocks and timers)
Also contributes towards https://github.com/cilium/cilium/issues/30894.

Many of Cilium's maps have a definition like this:
#ifdef CONFIG_MAP
struct {
	__uint(type, BPF_MAP_TYPE_ARRAY);
	__type(key, __u32);
	__type(value, __u64);
	__uint(pinning, LIBBPF_PIN_BY_NAME);
	__uint(max_entries, CONFIG_MAP_SIZE);
} CONFIG_MAP __section_maps_btf;
#endif

and are accessed like this:
time = map_lookup_elem(&CONFIG_MAP, &index);

and require matching agent code like this:
cDefinesMap["CONFIG_MAP"] = configmap.MapName
cDefinesMap["CONFIG_MAP_SIZE"] = fmt.Sprintf("%d", configmap.MaxEntries)

This issue is for making the names of all map definitions static, removing the cDefinesMap entries, removing the xyz.MapName variables and fixing up bpf C call sites with direct references. Bonus points for inlining maxentries if those cannot change (e.g. `configmap.MaxEntries is a const...).

I assume that this is not considering policymaps, since those are currently per-endpoint?

@joestringer I glossed over that in the description, but yes, this needs to be done for per-endpoint maps as well. We're no longer bound by the ELF rewrite hack, so we don't need the cilium_calls_65535 templating mechanism anymore.
In C code, the symbol can simply be named cilium_calls, and the loader needs to alter CollectionSpec.Maps["cilium_calls"].Name = "cilium_calls_00123" for specific endpoints. This behaviour is already in place today, albeit cilium_calls_65535 -> cilium_calls_00123 instead of cilium_calls -> cilium_calls_00123 in the future.

The proposal makes sense to me. I think the only danger is it could feel a bit magic in that all code references will refer to a single map name / reference but in practice the per-endpoint ones will be distinct instances, but I'm sure we'll figure out a reasonable representation of these maps.

 Everything refers to CALLS_MAP and one needs to jump through multiple layers of indirection to find the real map name.


CLI commands should be verbs/actions

Current pythainlp commands are ['corpus', 'tokenization', 'soundex', 'tagging'], which are

not verbs
not consistent with module names (.corpus, .tokenize, .transliterate, .tag)
(see: https://github.com/PyThaiNLP/pythainlp/blob/dev/pythainlp/cli/init.py )

Verb commands are more intuitive (and many time also shorter).
Example of verb commands:

git add
git commit
git push
git pull

We should try to use verb commands when possible.
Work in progress in https://github.com/PyThaiNLP/pythainlp/tree/verb-commands
Todo:

thainlp data purge to remove all downloaded datasets
Add benchmark command: https://github.com/PyThaiNLP/pythainlp/blob/dev/bin/word-tokenization-benchmark
Write testcases (maybe using subprocess)
Update document at https://github.com/PyThaiNLP/pythainlp/blob/verb-commands/docs/notes/command_line.rst
Record usage demo with https://asciinema.org

DRAFT DESIGN for soundex

thainlp soundex -h - help
thainlp soundex TEXT - get soundex of TEXT, using default engine
thainlp soundex -e ENGINE TEXT - get soundex of TEXT, using ENGINE engine

DRAFT DESIGN for data

thainlp data catalog - print dataset/corpus available for download (download the list from internet)
thainlp data info DATASET_NAME - show information about DATASET
thainlp data download DATASET_NAME - download DATASET
thainlp data remove DATASET_NAME - remove DATASET
thainlp data path - print path for local dataset
thainlp data local - show dataset already downloaded/available locally NOT IMPLEMENTED YET

DRAFT DESIGN for tokenize
(NOT YET IMPLEMENTED, comments are very welcome)

thainlp tokenize TEXT - tokenize TEXT at word level (default)
thainlp tokenize -worda ALGO -s : TEXT - tokenize TEXT at word level using ALGO as algorithm, and using : as seperator (-word argument is assumed if -worda exists)
thainlp tokenize -sub -word -sent TEXT - tokenize TEXT at subword, word, and sentence levels

should be very straightforward to implement, but the problem is more of the output representation like which character to use for separator for each level?

thainlp tokenize --infile INPUT_FILENAME - tokenize text from INPUT_FILE and print output to stdout
thainlp tokenize --infile INPUT_FILE --outfile OUTPUT_FILE

DRAFT DESIGN for tag
(NOT YET IMPLEMENTED, comments are very welcome)

thainlp tag -pos TEXT
thainlp tag -pos --infile INPUT_FILE --outfile OUTPUT_FILE

We also have a benchmark command here:
https://github.com/PyThaiNLP/pythainlp/blob/dev/bin/word-tokenization-benchmark.
For data command, we might also have purge to remove all downloaded datasets.

One idea. We might use https://asciinema.org to advertise + show how to leverage this functionality.

I guess we can close this issue now?

I think we've implemented most of the stuffs we can do for now. Let's file another issue for any enhancement.


fix(model-training): revert to python3.8 for s3 scripts

The image used for s3 step still uses python 3.8 and needs to be rebuilt before switching to python 3.10

Closing this PR to use 3.10 image instead.


Post-usability testing fix: include images of playbills on 'about' page

Talking about it later, realised that we could use image of row of thumbnails if including full text views would compete with text we want people to read.

This is done.


Zombie process is not reaped

When process become zombie consuming almost 100% of CPU core kernel should reape it (as I read, sorry if I don't use correct terms), but it sticks in the OS even if it's container was stopped.
I am not sure if it is bug in Docker, boot2docker or Linux kernel, I hope you will send me to the right direction how to solve it.
Steps to reproduce are in: docker-solr/docker-solr#4

I read it could be caused by bug in kernel. And boot2docker does not have the newest kernet. Is there some "command" how to upgrade kernel, so I can try if it was not solved in newer releases?

Could be related to https://github.com/boot2docker/boot2docker/issues/1099 and https://github.com/docker/docker/issues/18180 ?

Yes it could - Java program crashes and becomes zombie.
Jan Pobořil
2016-01-07 14:52 GMT+01:00 Sebastiaan van Stijn<EMAIL_ADDRESS>
Could be related to #1099
https://github.com/boot2docker/boot2docker/issues/1099 and
docker/docker#18180 https://github.com/docker/docker/issues/18180 ?
—
Reply to this email directly or view it on GitHub
https://github.com/boot2docker/boot2docker/issues/1094#issuecomment-169669284
.

v1.10.0 should have fixed this. I suggest closing this issue.


fix: update regex and test case for organisation website

Description

Website regex has missed the / at the end of websites. So adding that.

Fixes #8970
Type of change

Bug fix (non-breaking change which fixes an issue)

How Has This Been Tested?

WorkspaceServiceTest$validUpdateWorkspaceValidWebsite

Checklist:

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

/ok-to-test sha=8f998c9


pulseaudio-dlna: unstable-2017-11-01 -> unstable-2019-02-09

Motivation for this change
Fixes https://github.com/NixOS/nixpkgs/issues/108394
Part of https://github.com/NixOS/nixpkgs/issues/101964
Things done

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

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

WIP - the build fails with:
ERROR: Could not find a version that satisfies the requirement dbus-python (from pulseaudio-dlna)
ERROR: No matching distribution found for dbus-python

Let's close this in favor of https://github.com/NixOS/nixpkgs/pull/146915.


PostgreSQL Procedures cause an error during clean

Which version and edition of Flyway are you using?
6.0.8
Community Edition
If this is not the latest version, can you reproduce the issue with the latest one as well?
(Many bugs are fixed in newer releases and upgrading will often resolve the issue)
Which client are you using? (Command-line, Java API, Maven plugin, Gradle plugin)
Command-line
Which database are you using (type & version)?
PostgreSQL 12.1
Which operating system are you using?
macOS 10.15.2
What did you do?
(Please include the content causing the issue, any relevant configuration settings, the SQL statement that failed (if relevant) and the command you ran.)
I attempt to clean the database using flyway clean.  My database has a few procedures, which seem to be causing the issues.  Here is one of the procedures
CREATE PROCEDURE add_theme(name VARCHAR)
  LANGUAGE plpgsql
AS
$$
BEGIN
  INSERT INTO theme (theme_name)
  VALUES (name);
END;
$$;

What did you expect to see?
I expect to see a successful exit after the clean
What did you see instead?
ERROR:
Unable to clean schema "public"
-------------------------------
SQL State  : 42809
Error Code : 0
Message    : ERROR: public.add_theme(character varying) is not a function

It seems that the procedures are causing the issues.  Everything seems to work fine with functions.  If I remove them by hand, flyway is then able to perform the clean.

@beedaan A fix for this issue was released in Flyway 6.1.1, so I recommend you upgrade to the latest version.
Duplicate of https://github.com/flyway/flyway/issues/2578.


Add support for publishing ASP.NET Core apps to Windows VMs in GCE

See this link for more information on what is required to publishing ASP.NET Core apps to IIS on a Windows VM.

This is awesome, great work Jim!


Cordova CLI not detected with ionic info

I installed cordova using sudo npm install -g cordova and can see that its installed:
sudo npm -g list | grep cordova
├─┬<EMAIL_ADDRESS>│ ├─┬<EMAIL_ADDRESS>│ │ ├──<EMAIL_ADDRESS>│ │ ├─┬<EMAIL_ADDRESS>│ │ ├─┬<EMAIL_ADDRESS>│ │ ├──<EMAIL_ADDRESS>│ │ ├─┬<EMAIL_ADDRESS>│ │ ├─┬<EMAIL_ADDRESS>│ │ │ ├──<EMAIL_ADDRESS>│ │ │ ├─┬<EMAIL_ADDRESS>│ │ │ ├──<EMAIL_ADDRESS>│ │ │ ├─┬<EMAIL_ADDRESS>
But, ionic info shows that Cordova CLI is not installed.
$ sudo ionic info
Your system information:

Cordova CLI: Not installed
Ionic Version: 2.0.0-alpha.53
Ionic CLI Version: 2.0.0-beta.17
Ionic App Lib Version: 2.0.0-beta.8
ios-deploy version: Not installed
ios-sim version: 5.0.6
OS: Mac OS X El Capitan
Node Version: v5.3.0
Xcode version: Xcode 6.4 Build version 6E35b

However, if I install<EMAIL_ADDRESS>then ionic info identifies that its installed. Strange!
I tried uninstalling and reinstalling ionic<EMAIL_ADDRESS>but still it shows the same error. Am I missing something? Is there any way to let ionic know the path of Cordova? I want to use newer versions of cordova.
Thanks!

Thanks @tlancina . Let me know if you need any other debug info from me. I m still stuck with this :(

This issue was moved to driftyco/ionic-cli#756


Pelican UUID Update

added pelican egg and ptero egg:
https://github.com/pelican-eggs/voice/pull/5


Runtime corpus test now uses more interesting snapshot times

I also made the avalanche pretty printing a bit more compact / easier to read.
! @tranma

! @amosr seeing as you're here

pretty good 👍


CriteriaBuilder orderBy generates strange query

CriteriaBuilder orderBy generates strange query

blaze-persistence.version:
I'm using the CriteriaBuilder method orderBy like in the docs:
https://persistence.blazebit.com/documentation/1.2/core/manual/en_US/#anchor-order-by-clause
CriteriaBuilder<Tuple> cb = cbf.create(em, Tuple.class)
    .from(Cat.class)
    .select("age")
    .select("id")
    .orderByAsc("age");

and expected to receive a sql like in the docs, but the sql comes with a case when, that I don't understand from where it comes.
 The problem is that MySQL does not support the NULLS FIRST/LAST clause which is why the expression you see is rendered. If the attribute is non-nullable, it will not render this emulation.

If you have any further questions, don't hesitate to create a new issue or join us on Slack
PS: The queries in the docs are usually JPQL or what we call JPQL.next


[DUMMY]

:clipboard: Add associated issues, tickets, docs URL here.
Overview
Describe what your Pull Request is about in a few sentences.
Changes
Describe your changes and implementation choices. More details make PRs easier to review.

Change 1
Change 2
...

Testing
Describe how to test your new feature/bug fix and if possible, a step by step guide on how to demo this.

Coverage remained the same at ?% when pulling 9c12e2c80d3aef23f883f74754dc8dfa1e02788a on code-owners-test-4 into 0645ec7669e07f6f4c1b8543213d27d0b03f9956 on code-owners.


added results.compressed = false

Thanks, BTW you can continue to commit to your branch after you have done a PR to make alterations, there is no need to create a new PR


[Feature] Integrating Apache Kyuubi service. #423

Purpose of the pull request

Brief change log

Verify this pull request

This pull request is code cleanup without any test coverage.
(or)
This pull request is already covered by existing tests, such as (please describe tests).
(or)
This change added tests and can be verified as follows:

(or)
If your pull request contains incompatible changes, you should also pull request the documentation to https://github.com/datasophon/datasophon-website

Translate to English: The monitoring part is currently missing, and I will handle it later.

The monitoring part is currently missing, and I will handle it later.


Remove non-gender-neutral language

Thanks Andrés. No result shows up if you search for a specific pronoun, so it's probably not indexed and might be difficult to search systematically. It's a living document, so we should just patch them whenever we spot them.


Suggester creation via portal

There is documentation for creating suggesters via Rest and C# code, but nothing via the portal.  Is this an oversight or is this ability not available through the portal?  You might want to document, if they aren't available through the portal, that this deficiency exists.

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 0c80cb97-e34a-1c2d-7e51-8104bf5079c8
Version Independent ID: 56a3fd9d-dd1b-3271-c208-5967b95f6514
Content: Create a suggester - Azure Cognitive Search
Content Source: articles/search/index-add-suggesters.md
Service: cognitive-search
GitHub Login: @HeidiSteen
Microsoft Alias: heidist

@JHUMikeSchnell  It's an oversight!  Thanks for catching it -- I'll try to fix it today or tomorrow.  In the portal, when you use Add Index to create the index, there is a text box where you can specify a suggester name.  In the Field definition section, there is a column/checkbox for "Suggester" where you select specific string fields for which you want to enable autocomplete or suggestions.

@HeidiSteen Thanks Heidi for your quick response.
@JHUMikeSchnell
We will now proceed to close this thread. If there are further questions regarding this matter, please respond here and @YutongTie-MSFT and we will gladly continue the discussion.


Fixing of 3 (6318, 6345, 6319) defects and few additional issues

This not all defects fixed from list that you assigned to me, but fixed staff is enought critical.
Please consider this pull request for now. I will proceed to work on other defects after nearest vocation.

Oleg please updated and then merge this branch

i merged it with master


Improve Sign-In, Sign-Out, Register Pages

Sign-out page

Should redirect to Landing page after a certain amount of time.
Improve design (if necessary)

Landing page

Handle signing up for the website to add to the list of users
After user signs up, redirect to user's home page.
Improve design (if necessary)

Sign-in Page

Allow to sign in with either email or username
Add link to landing page if user does not have an account

Do not add as  but next to login button

Improve design (if necessary)


Enable osx-arm64 / linux-{aarch64, ppc64le} builds, modernize recipe, add myself to maintainers

Checklist

[x] Used a personal fork of the feedstock to propose changes
[x] Bumped the build number (if the version is unchanged)
[ ] Reset the build number to 0 (if the version changed)
[x] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[ ] Ensured the license file is being packaged.

package inspection shows that building like that only includes bin/gitui ... so rm $PREFIX/.crates2.json + rm $PREFIX/.crates.toml are not required anymore.
closes https://github.com/conda-forge/gitui-feedstock/issues/12

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe/meta.yaml) and found it was in an excellent condition.

@TheBlackSheep3 Ready for review / merge. Thanks!


Last BootDPRO Build does not work

Describe the bug
Executing BootDPRO.cmd fail to generate a Dolphin image.
To Reproduce
Steps to reproduce the behavior:

Clone Dolphin repo
Fetch VM
Execute BootDPRO.cmd
See error

Expected behavior
Create a clean Dolphin image
Screenshots
If applicable, add screenshots to help explain your problem.
Please complete the following information):

OS: Windows 8
Dolphi 7.1.4
Additional context
Loading source package 'Development System' from: C:\Users\Usuario\Documents\Git
Hub\Dolphin\Core\Object Arts\Dolphin\IDE\Base\Development System.pax
Recompiling references to AbstractLOGFONT (size 28)...
Recompiling references to LOGFONTW (size 92)...
----------------------------------------Unhandled exception - a MessageNotUnders
tood('Array does not understand #negated')

Array(Object)>>doesNotUnderstand:
Font>>pointSize:
Font class>>name:pointSize:
SmalltalkWorkspace class>>initialize
SmalltalkWorkspace class(Class)>>initializeAfterLoad
[] in Package>>initializeClasses
Array(ArrayedCollection)>>do:
Package>>initializeClasses
Package>>initializeGlobals
[] in Package>>loadPAX
BlockClosure>>ifCurtailed:
BlockClosure>>ensure:
Package>>loadPAX
Package>>load
PackageManager>>basicInstall:
[] in PackageManager>>install:
BlockClosure>>ifCurtailed:
[] in PackageManager>>install:
OrderedCollection>>do:
[] in PackageManager>>install:
BlockClosure>>ifCurtailed:
BlockClosure>>ensure:
PackageManager>>install:
[] in DolphinProduct>>basicLoad
ExceptionHandler(ExceptionHandlerAbstract)>>markAndTry
[] in ExceptionHandler(ExceptionHandlerAbstract)>>try:
BlockClosure>>ifCurtailed:
BlockClosure>>ensure:
ExceptionHandler(ExceptionHandlerAbstract)>>try:
BlockClosure>>on:do:
[] in DolphinProduct>>basicLoad
PluggableLookupTable(LookupTable)>>at:ifAbsent:
PackageManager>>packageNamed:ifNone:
[] in DolphinProduct>>basicLoad
OrderedCollection>>do:
DolphinProfessional(DolphinProduct)>>basicLoad
DolphinProfessional(DolphinProduct)>>loadShrinkAndCompress
DolphinProfessional(DolphinProduct)>>boot
DolphinProfessional class(DolphinProduct class)>>boot
[] in UndefinedObject>>{unbound}doIt
ExceptionHandler(ExceptionHandlerAbstract)>>markAndTry
[] in ExceptionHandler(ExceptionHandlerAbstract)>>try:
BlockClosure>>ifCurtailed:
BlockClosure>>ensure:
ExceptionHandler(ExceptionHandlerAbstract)>>try:
BlockClosure>>on:do:
UndefinedObject>>{unbound}doIt
Compiler class>>evaluate:for:evaluationPools:logged:ifFail:
Compiler class>>evaluate:for:evaluationPools:logged:
Compiler class>>evaluate:for:logged:
[] in ChunkSourceFiler>>{unbound}fileIn
ExceptionHandler(ExceptionHandlerAbstract)>>{unbound}markAndTry
[] in ExceptionHandler(ExceptionHandlerAbstract)>>{unbound}try:
BlockClosure>>{unbound}ifCurtailed:
BlockClosure>>{unbound}ensure:
ExceptionHandler(ExceptionHandlerAbstract)>>{unbound}try:
BlockClosure>>{unbound}on:do:
ChunkSourceFiler>>{unbound}fileIn
SourceManager>>{unbound}fileInFrom:normalizeLineEndings:
[] in SourceManager>>{unbound}fileIn:normalizeLineEndings:
BlockClosure>>{unbound}ifCurtailed:
BlockClosure>>{unbound}ensure:
SourceManager>>{unbound}fileIn:normalizeLineEndings:
SourceManager>>{unbound}fileIn:
BootSessionManager>>{unbound}main
BootSessionManager(SessionManager)>>{unbound}mainLoopStarted
[] in BootSessionManager(SessionManager)>>{unbound}forkMain
InputState>>{unbound}loopWhile:
InputState>>{unbound}mainLoop
[] in InputState>>{unbound}forkMain
ExceptionHandler(ExceptionHandlerAbstract)>>{unbound}markAndTry
[] in ExceptionHandler(ExceptionHandlerAbstract)>>{unbound}try:
BlockClosure>>{unbound}ifCurtailed:
BlockClosure>>{unbound}ensure:
ExceptionHandler(ExceptionHandlerAbstract)>>{unbound}try:
BlockClosure>>{unbound}on:do:
[] in BlockClosure>>{unbound}newProcess:
Shutting down boot session
Boot failed, Code=-1

If you are fetching VM 7.1.4, then your repo is out of date (see https://github.com/dolphinsmalltalk/Dolphin/tags). Have you fetched the tags?

Thanks Blair,
FetchVM.cmd v7.1.6 fix the problem FetchVM.cmd without argument fail. This can be closed !
It is explained here:
https://github.com/dolphinsmalltalk/Dolphin/issues/757#issuecomment-487681393

If you have the tags locally and have checked out (or from) master, then FetchVM will download the latest tagged version - you do not need to specify it. Remember that if you are working from a fork, then if you update it by fetching and pushing to it, git does not push tags by default. Depending on how you cloned/ fetched, you may not get them either. Try running git describe to see what you have. From the current state of master it should say v7.1.6-4-g4dc63497


Deploy

🚢

@l2succes should we hold off on deploying to production until that collect page fix is in?

This deploy seems like a huge one, but most of the changes have been made by prettier automatically and are just syntax changes.


License file is missing

This looks like a nice library. Could you please add a license file or statement indicating what the licensing of this code is. For example, the RSQL project on which this is based is MIT License.
Thanks.

Thanks for pointing that out. I will look into it.


Allow keyboard modifiers for mouse events

I would like the implement complex mouse interaction using keyboards modifiers such as alt, ctrl, shift and space and even possible combinations. As sidenote, defining constants for mouse buttons and keyboards modifiers would be nice. It wloud the look like this:
cornerstoneTools.pan.activate(element, cornerstoneTools.MOUSE_LEFT, cornerstoneTools.KB_SPACE);
If that is fine, I may get on it soon.

Revisiting this, I'm not entirely sold on complex bindings such as this. This is something that is easier to add now more than ever, but when would we want left to be one tool and left + ctrl to be another? The example that comes to mind:

freehand
freehand sculptor

Mostly because they manipulate the same data. But you could argue that these might make more sense as a single tool, and you use the modifier internally to determine behavior?
Still want one tool for left click, and the other for right? Use two mouseButtonMasks, but then use the tool's options/config to determine behavior on click.
I'm open to make the the EventDispatchers more complex and for adding support for bindings like this, but I think it's at least worth a discussion? What would help sell me are real world examples where support for this would make DX much easier/simplified, or scenarios that can only be accommodated by making this change.

Im curious where this ended up? In my experience with medical image viewers, shift + click seems like a great way to enable additional tools without collisions. E.g., panning using shift + left click allows it to always be on and quickly accessible. It seems like a great option to have.

Like @gattia I am wanting to implement a shift + click to pan.  This is important for me because the single click will be occupied by a modified freehandroi tool.  Does anyone have any examples of doing such keybinding with cornerstone?
@dannyrb
As a work around, you say "One way to get around this would be creating a tool that is activated for a binding, and then using a key modifier within the tool to allow other tools to handle the event."
Could you by chance provide an example of a simple tool doing this? I am not particularly familiar with the way cornerstonetools handles events.
Also, you mention key modifiers for active tools; is there any example or documentation for how that is done? I imagine that will be useful for me soon.
Thanks.

For what it’s worth, I ended up making it a toggle. Hit space bar or something and it toggles between pan & window/level for mouse dragging. Not the same as we’re used to, but it worked fine.

@gattia
Thanks, that got me to thinking that I could activate or disable tools depending on whether I am holding down shift or not.  That has worked for me. When shift is down, tool A is active and tool B is disabled.  When shift is not down, tool A is disabled and B is active.     So Shift + Click works as expected for panning.


Submit and others buttons labels[Feature Request]

Hi,
How can I change  Submit or Cancel buttons labels to another language?

You can override the form action buttons using pywebio.input.actions()

That's great, thank you!


Issues with page_template and creating a *.html-file from any directory

Hi guys. I was about to run following command (I've added PotreeConverter to my $PATH already) in ~/pclouds/las:
PotreeConverter test.las -o testdir -p testpage
While the process seems to output the files for the potree viewer, I don't get any html-page displayed. It works when I do it the way MadhavEsDios did in this: https://github.com/potree/PotreeConverter/issues/314 but on the long run, I'll need to run it from multiple folders.
I've tried copying the resources folder, the contents of the resources folder and basically every other combination to ~/pclouds and ~/pclouds/las to no avail.
What do I have to do to make PotreeConverter produce an html file when ran from any working directory?
Thanks in advance!

Having the same issue, cannot find any solution on the web either

Had the same issue. Solution: PotreeConverter [-input] -o [-outputDir] -p someName. The outputDir should for example "/home/kristofer/dev/workspaces/PotreeConverter/master/PotreeConverter/resources/page_template/ " or additionaly ".../page_template/pointclouds"


Add rotated orpheus hand site banner

It was requested in https://github.com/hackclub/hackclub/issues/712

🙌


Enhance collision support + serialization + parallelization

Exploit functors of hpp-fcl.

I think the code is explicit enough to see how it works.
You are also sufficiently expert to run the bench by yourself, I have no such time on my side.


Docs: where to search for logs especially issued queries to postgresql?

For the embedded PostgreSQL instance, look in the EdgeDB data directory, which is a regular PostgreSQL data directory.  You can configure the underlying Postgres instance logging by editing postgresql.conf, as usual.
Future versions of EdgeDB will feature a more elaborate logging system.

Thank you


Outdated data from some leagues

good morning, I don't know why if I upload the Chinese league data (which on https://www.football-data.co.uk/china.php are correctly updated until today) instead on the program the matches stop on December 31, 2022

@kochlisGit same error with Brazilian league, matches on the schedule go up to 13/11/2022

need help on this too, for Brazilian leagues, on fixtures import has an error, for Jan 01


Awake: Provide different notification area icon with steam to let us now that "keep awake" is active

Description of the new feature / enhancement
Whenever one of the "keep awake" or "keep screen on" feature is active, show a coffee icon with steam coming out of it so we know at a glance that the computer is being kept awake.
Scenario when this would be used?

If we want to know if it's on we can just look at the notification area instead of popping up the menu.
Without seeking the information, we could notice that it's being kept awake just by seeing it in the notification area, this could be useful if we're not aware that this is on and we might not be wanting to keep the computer awake.

Supporting information
Steam over a cup of coffee lets us know that it's full so it's pretty intuitive to deduct that this means the app is active.

/dup #11996


Block events generated even when there are no policies applied in DVWA app

Bug Report
Network (ICMP) related audit events are generated when there are no policies enforced at all on dvwa(Damn Vulnerable Web Application) workload.
This is observed with AppArmor based enforcer only. My assumption is that the default apparmor profile that is created by kubearmor has some rule that denies the ICMP events. But it not known why the same is not true when wordpress-mysql app is used.
General Information

Environment description (GKE, VM-Kubeadm, vagrant-dev-env, minikube, microk8s, ...)
k8s - any env

Kernel version (run uname -a)
Enforcer should be AppArmor. Check using karmor probe.

To Reproduce

Setup k8s cluster
Install dvwa app
a. git clone https://github.com/cytopia/docker-dvwa.git
b. kubectl apply -f docker-dvwa/k8s/
c. You should now see two pods dvwa-web-xxx and dvwa-mysql-xxx running in default namespace.
Run karmor log --json in another terminal to observe logs.
kubectl exec -it dvwa-web-xxx -- sh -c 'ping localhost'

At this stage, you should see Block events reported in the karmor log screen.

Block events
{
  "Timestamp":<PHONE_NUMBER>,
  "UpdatedTime": "2022-11-25T12:53:47.102712Z",
  "ClusterName": "default",
  "HostName": "aks-agentpool-41853272-vmss000000",
  "NamespaceName": "default",
  "PodName": "dvwa-web-68d58bc4b9-68ckw",
  "Labels": "app=dvwa-web,tier=frontend",
  "ContainerID": "e58ce6fae940212abe16dec80557f9f8849a5a9b27e744815baad5bf402aa296",
  "ContainerName": "dvwa",
  "ContainerImage": "docker.io/cytopia/dvwa:php-8.1@sha256:ae5ff249472f01a476a352eafd442b8c3687e7d3e2e5e55ee0140d9c5b871324",
  "HostPPID": 1001314,
  "HostPID": 1001320,
  "PPID": 102,
  "PID": 108,
  "ParentProcessName": "/bin/dash",
  "ProcessName": "/bin/ping",
  "PolicyName": "DefaultPosture",
  "Type": "MatchedPolicy",
  "Source": "/bin/ping localhost",
  "Operation": "Network",
  "Resource": "domain=AF_INET type=SOCK_DGRAM protocol=ICMP",
  "Data": "syscall=SYS_SOCKET",
  "Enforcer": "AppArmor",
  "Action": "Block",
  "Result": "Permission denied"
}
{
  "Timestamp":<PHONE_NUMBER>,
  "UpdatedTime": "2022-11-25T12:53:47.102716Z",
  "ClusterName": "default",
  "HostName": "aks-agentpool-41853272-vmss000000",
  "NamespaceName": "default",
  "PodName": "dvwa-web-68d58bc4b9-68ckw",
  "Labels": "app=dvwa-web,tier=frontend",
  "ContainerID": "e58ce6fae940212abe16dec80557f9f8849a5a9b27e744815baad5bf402aa296",
  "ContainerName": "dvwa",
  "ContainerImage": "docker.io/cytopia/dvwa:php-8.1@sha256:ae5ff249472f01a476a352eafd442b8c3687e7d3e2e5e55ee0140d9c5b871324",
  "HostPPID": 1001314,
  "HostPID": 1001320,
  "PPID": 102,
  "PID": 108,
  "ParentProcessName": "/bin/dash",
  "ProcessName": "/bin/ping",
  "PolicyName": "DefaultPosture",
  "Type": "MatchedPolicy",
  "Source": "/bin/ping localhost",
  "Operation": "Network",
  "Resource": "domain=AF_INET6 type=SOCK_DGRAM protocol=ICMPv6",
  "Data": "syscall=SYS_SOCKET",
  "Enforcer": "AppArmor",
  "Action": "Block",
  "Result": "Permission denied"
}

Expected behavior
There should be no block/audit events.
karmor sysdump
karmor-sysdump-Fri Nov 25 18_31_40 IST 2022.zip

Sure

This is not caused by kubearmor.
when using a diffrent ping binary in the pod, no logs are generated. the error is shown because some implimentations fails to aquire the apropriate privs before creating the socket


fix: more accurate DOM parsing for NPMJS which resolves navigation during brower history buttons

Fix for #104

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

Chrome Web Store
v2.13.1
GitHub release

Your semantic-release bot :package::rocket:


dev-9150 Fix double data requests for charts and maps

When charts or maps are embedded like <CdcChart configUrl=, they make one data request on page load, but when they're embedded like <CdcChart config=, they make two. Since they're embedded with configUrl= when viewing charts locally but with config= in WCMS, we only see the issue on live charts.
This useEffect (which is only added when loading the chart with a configObj) is what's causing the double-request. I think the original intent of the useEffect was for dashboard filtering, but it's a little hard to tell if this change will have other side effects.
To test that this fixes the issue, chart/src/index.jsx can be changed to:
import React from 'react'
import ReactDOM from 'react-dom/client'

import CdcChart from './CdcChart'

import 'react-tooltip/dist/react-tooltip.css'

let isEditor = window.location.href.includes('editor=true')
let isDebug = window.location.href.includes('debug=true')

let domContainer = document.getElementsByClassName('react-container')[0]

let configUrl = domContainer.attributes['data-config'].value;
let config = (await (await fetch(configUrl)).json());

ReactDOM.createRoot(domContainer).render(
  <CdcChart config={config} isEditor={isEditor} isDebug={isDebug} />
  //<CdcChart configUrl={configUrl} isEditor={isEditor} isDebug={isDebug} />
)

@jayb can you add clearer testing instructions? something like this:
Example Test Case:
Dropdown List Sorting

	Scenario: The User has logged into the application and the application has fully loaded.

	1) When the user clicks on the first tab.
	2) Then the user scrolls to the section on the page with the first dropdown.
	3) Then the user opens the dropdown.

	Expected: The user should see a dropdown list with items in alphabetical order.

@jayb @Mgetz10 should this PR be closed?


Update ds4drv.conf

Changed file that lists event codes
https://github.com/torvalds/linux/blob/master/include/uapi/linux/input-event-codes.h

Thanks.


Rails/SaveBang doesn't work well with postfix if

Expected behavior
./x.rb :
def x
  instance.save if @flag
end

$ bundle exec rubocop x.rb --only Rails/SaveBang should be pass.
Actual behavior
./x.rb :
def x
  instance.save if @flag
end

$ bundle exec rubocop x.rb --only Rails/SaveBang
Inspecting 1 file
C

Offenses:

x.rb:2:12: C: [Correctable] Rails/SaveBang: Use save! instead of save if the return value is not checked.
  instance.save if @flag
           ^^^^

1 file inspected, 1 offense detected, 1 offense autocorrectable

note:
def x
  @flag && instance.save
end

is ok.
Steps to reproduce the problem
see above
RuboCop version
Include the output of rubocop -V or bundle exec rubocop -V if using Bundler. Here's an example:
You can see extension cop versions (e.g. rubocop-rails, rubocop-performance, and others) output by rubocop -V,
include them as well. Here's an example:
$ bundle exec rubocop -V
1.58.0 (using Parser <IP_ADDRESS>, rubocop-ast 1.30.0, running on ruby 3.2.2) [arm64-darwin22]
  - rubocop-rails 2.22.2

👍 Agreeing with the Proposed Solution: #1212
I've reviewed the suggested solution:
def x
  @flag && instance.save
end

I agree that this alternative is a suitable and clean approach. It addresses the reported issue and aligns with best practices. If no one has objections, I believe we can consider this as a valid resolution.
Looking forward to any further discussions or feedback from the community!
🚀 Thanks!


Use ActionSet in Tearsheet

Contributes to #555
Use the ActionSet in the Tearsheet and replace the buttons prop with array of shape, as in SidePanel.
What did you change?
Tearsheet
How did you test and verify your work?
ran tests and storybook

NB once approved this PR should NOT be merged until #583 has been merged, as it assumes Storybook 6.2 behaviour.


Path Issue to JSPM packages when using buildSFX

For some reason (and it may just be my lack of understanding) when using buildSFX it will only work if I use 'app' as the path to my main module file (app.js), if I try to set the path to be ''./app/assets/src/js/app' it gives me the error (Error loading "app/assets/src/js/app/app" at file:/app/assets/src/js/app/app.js).
I got around this by setting the baseURL to be '/app/assets/src/js/app', but now I must move my jspm_packages into that folder for it to work. Is there a way I can keep my jspm_packages folder in the root, whilst successfully referencing './app/assets/src/js/app/' for my main module?
// load SystemJS config from file
var builder = new Builder();
builder.loadConfig('./config.js')
.then(function() {
    // Change baseURL to match the file system
    builder.config({ baseURL: path.resolve('./app/assets/src/js/app/') });

    // Build a self-executing bundle (ie. Has SystemJS built in and auto-imports the 'app' module)
    return builder.buildSFX('app', './app/assets/dest/js/app.js', { minify: false, sourceMaps: true});
    }).catch(function(err) {
    console.error(err);
});

Currently relative path builds are not supported, although it's coming in the next release very soon.
So yes, you have to use a path relative to the baseURL, and if the baseURL is not low enough, lower it to include the jspm_packages.
The restriction of having jspm_packages within the baseURL though is something that won't apply in the next release as well.

Thanks for getting back to me, so is moving my JSPM Packages the only solution for now? If I lower the base to include the JSPM packages I will no longer be able to reference the build? Or is there an additional config setting I am missing where I can define the build source folder?
I don't suppose you have a rough ETA for the next release? Thanks for all the hard work on this by the way, it's incredible that we are able to do these things already!!

I'd suggest setting the baseURL to the lowest reference point - app/assets, which contains jspm_packages, and then bundling your app as jspm bundle src/js/app.

Note a key point here is that the baseURL needs to match between client and server as a reference point. (/app/assets in client and public/app/assets on the server say). You can rerun the prompts with jspm init -p.

I played around and was able to fix it using this:
// load SystemJS config from file
var builder = new Builder();
builder.loadConfig('./config.js')
.then(function() {
    // Change baseURL to match the file system
    builder.config({ baseURL: path.resolve('./') });

    // Build a self-executing bundle (ie. Has SystemJS built in and auto-imports the 'app' module)
    return builder.buildSFX('app/assets/src/js/app/app', './app/assets/dest/js/app.js', { minify: false, sourceMaps: true});
    }).catch(function(err) {
    console.error(err);
});

Thanks for the help :-)

Glad to hear it's working.

when I try this I get this error:
TypeError: baseURL should only be configured once and must be configured first.
I want to have different baseURLs for SystemJS-Builder and SystemJS-Runtime. Is this possible?

@Jandalf you can set the baseURL when constructing the builder via -
new Builder('path/to/baseURL', 'path/to/config.js')

And it will ignore the config.js baseURL so there will be no conflict.


Mention CORS in Dynamic Imports

CC: @petele, @mathiasbynens

:tada: This has been pushed live to https://developers.google.com/web/


Add: Disconnect button to each new user connection

Adding a disconnect button will allow users/clients to disconnect the peer dataConnection. When a clients are disconnected they can no longer participate in group chat or video conference

.

This addition will be useful in video conference feature, to end the video conference or to exit any user in the conference. Good job @ditml


Command Not found

Hi After Installing HHVM
I run This Command sudo /usr/share/hhvm/install_fastcgi.sh
But it showing me   command not found
How to solve This

Sorry, this repo is no longer updated/supported. It's very possible HHVM has changed. Closing and archiving the repo. Good luck!


routinator currently broken?

$ cargo install --force --git https://github.com/NLnetLabs/routinator.git

results in tons of errors:
   Compiling mio v0.6.16
   Compiling clap v2.32.0
error[E0658]: `crate` in paths is experimental (see issue #45477)
  --> /home/job/.cargo/registry/src/github.com-1ecc6299db9ec823/ring-0.13.5/src/bssl.rs:15:5
   |
15 | use crate::{c, error};
   |     ^^^^^

error[E0658]: `crate` in paths is experimental (see issue #45477)
  --> /home/job/.cargo/registry/src/github.com-1ecc6299db9ec823/ring-0.13.5/src/aead/mod.rs:28:5
   |
28 | use crate::{constant_time, error, init, poly1305, polyfill};
   |     ^^^^^

error[E0658]: `crate` in paths is experimental (see issue #45477)
   --> /home/job/.cargo/registry/src/github.com-1ecc6299db9ec823/ring-0.13.5/src/polyfill.rs:128:13
    |
128 |         use crate::error;
    |             ^^^^^
    |
   ::: /home/job/.cargo/registry/src/github.com-1ecc6299db9ec823/ring-0.13.5/src/aead/mod.rs:120:17
    |
120 |     let nonce = slice_as_array_ref!(nonce, NONCE_LEN)?;
    |                 ------------------------------------- in this macro invocation

This looks like ring (the crate that does the crypto) now requires Rust 1.30. Can you update your Rust? Or do we need to work around it?

I've upgraded to rust 1.30


add validation for dimensions

fixes #645

@hamilton tagging you for review on this one re #645

Thank you @bcolloran , the feedback is much improved on the config surface.  I did notice, however, that the dashboard is still presenting the missing dimension with "no available values". I think we should probably remove leaderboards and dimensions that are not valid. It is not clear if all the values are null or the dimension is not available at the dashboard level.

@magorlick -- i think that goes beyond the scope of #645, can we merge this as an incremental improvement and file those additional observations as a new issue? If so, would you mind clicking the "approve" button on this one so that we can get it in? Thanks :-)


Unschedule command

Add the unschedule command, as per issue https://github.com/globocom/azkaban-cli/issues/8
This command calls the unschedule request from the Azkaban API. The API accepts a scheduleId parameter, whereas the CLI accepts the project and flow names. Within the call (transparent to the user), two GET requests are made to "translate" the project and flow names into the required scheduleId parameter.

Nice, thanks for the contribution, Andre! I'm merging it


e2e: Bail early on ProvisionStopped

e2e was detecting ClusterDeployment condition ProvisionFailed to decide
whether to give up early. ProvisionStopped is appropriate also (or
perhaps instead?) so add it.

/test e2e

/lgtm

/lgtm

/retest

/hold
retest after https://github.com/openshift/hive/pull/1802 lands

/hold cancel
/retest


poor results even with very close strings and low threshold

Hi,
After setting up the plugin (version 1.10.4) on strapi (version 4.5.1) I can't seem to get good results even after lowering the threshold a lot.
I currently have 11 entries in the database with the word "abridor" in them, like "Abridor amarelo", "chaveiro abridor", "abridor adesivado", etc...
But when I try to search for "abrdor" or "abredor", I get zero results.
Some variations return some results but still very few, like "abridos" gets 1 result.
Here's my config:
"fuzzy-search": {
    enabled: true,
    config: {
      contentTypes: [
        {
          uid: "api::material.material",
          modelName: "material",
          transliterate: false,
          fuzzysortOptions: {
            characterLimit: 500,
            threshold: -8000,
            limit: 9999,
            keys: [
              {
                name: "nome",
                weight: 100,
              }
            ],
          },
        },
      ],
    },
  },

Any ideas?
Thank you!
Vítor

Hey @vitormanfredini ,
everything seems to be set up correctly on your end.
In some cases the algorithm (fuzzysort) will behave a bit strange, when it comes to getting good results for typos. Between fuse and fuzzysort I found the latter to produce more reliable results though, especially when searching through longer strings, where I found fuse to return a lot of false positives.
Sorry I can't be much of help. I think that if typo tolerance (and having sophisticated configuration options) something like meilisearch may be the more suitable option for your use case.


Update README.md

Copyright 2015 -> 2016

👍

@anned20 , thank you. See you again in 6 months time!? 😁

I guess :smile_cat:

I've often wondered about the date in a (C) message. Seems it should be the oldest date!
http://www.legalcorner.net/articletemplate1.asp?name=copyright_art_003
https://www.plagiarismtoday.com/2011/11/08/how-to-write-a-copyright-notice/
http://www.clickandcopyright.com/copyright-resources/copyright-symbol-usage.aspx
I read these and see that the copyright should be from the initial publication date, which is 2012
https://github.com/robmorgan/phinx/commit/38190e029e67af993a48ddb4e2e71c8739b4b634

Also no idea what happens when collaborators and contributors ass their pennies worth.

@anned20 @rquadling I don't really care too much to be honest... more or less I'd rather take a simple commit from somebody. Even if its a simple typo fix, date etc. That way people learn the whole PR, Github workflow. It's also how I personally got started in open-source ;)


Basic functionality roadmap

If you want to contribute to hematite_server this roadmap might help you! Order is relevant.

[x] Login (online-mode=false). Done in #40! :tada:
[ ] Implement concept of world. Dimensions ticking in sync! Example: http://is.gd/oGH1Yr (this is example code, a real version wouldn't clone every world on every tick ofc)
[ ] Keep track of entity positions. Just the player's position with some simple checks (moving too fast, send Disconnect packet) should be enough to get going. Example: HashMap<String, Position> which will come handy when we start adding more entities.
[ ] A working chunk loader. Once we know where the player is we can load and unload his/her surroundings.
[ ] Chat. Basically just echoing back whatever the player writes. This would allow people to start working on commands.
[ ] Time handling. Having /time working would be great, also time being updated accordingly every now and then just to keep players in sync.
[ ] Basic inventory handling. Moving items around in their inventory, including bottom bar.
[ ] Digging mechanics. This involves actually breaking blocks and spawning entities so the player can pick them up. Quite a milestone.
[ ] Dropping items. We already had the inventory working and know how to spawn entities, should be easy.
[ ] Illumination. If we start breaking blocks, we should update lighting accordingly?

I suggest adding/agreeing upon an 'api' to allow concurrent development.
Such as defining what arguments the chunk-loader will take, what it's responsibilities are, and what it returns.
Also so no part's design depends on another to be functioning

@RichardlL There a ton of components to be defined, being so few people actually developing I think the best workflow is submitting proposals (pseudocode or working code), discussing them until quorum is met and merging. Otherwise we'll waste too much time. Maybe it's not the best idea.
/cc @fenhl

I'd like to get started on world management.
I would like to implement it based on my proposal in #97, henceforth named Allocater / Worker as requested by @fenhl #100,
So implement

1.9 world format
Requests based on channels, while abstracted by regular functions for ease of use
public function chunk_send()
Possibly Incorporating hematite base's chunk loader

public funtion ``chunk_send``` to take

A slice of Tuples, each a (i32,i32) x z coordinate for chunks,
A mutable reference to a TcpStream
Returning Error if unable to generate map or unable to send.

@toqueteos Bump

Hello, I'm interested in getting invovled! On this list what isn't being tackled by anyone yet?

@JamesLaverack I think everything is free but the first two and chat. /cc @fenhl

I'm not currently working on anything on this list.

@toqueteos Who's working on chat? Is there a branch for it somewhere?

@JamesLaverack https://github.com/PistonDevelopers/hematite_server/pull/106 contains some changes to the Chat type. There's also some related issues opened. Knowing that you can also work on Chat without problems.


test: add integration tests for Upgradable

Adds integration tests and removes the corresponding unit tests. The background and motivation are described in https://github.com/aurora-is-near/near-plugins/pull/51, which does the same for Pausable.
Expands test coverage compared to previous unit tests.


[Suggestion] Provide sample integration with firestore

I want to suggest providing some sample integration with firestore + ngrx, maybe on some branch. I think it's quite popular BaaS and some people will try to use it, especially for some MVP concepts. And from what I see integrating this with ngrx is not very trivial so this would help a lot to see your sample implementation :)

All contributions are very welcome and appreciated! They can be used as a basis to community effort to improve and enhance!

Has there been any further discussion/thought on this? My initial that is to create a simple Firebase backend for the Todo list. I think though that you probably wouldn't want to open that up (e.g. you'd probably want working auth stuff first). @tomastrajan would you be open to using firebase auth in this project?

@ZackDeRose definitely, auth stuff is extremely basic, basically does nothing just demonstrates auth guard but there is no handling of the auth besides setting a boolean flag. Please, feel free to propose solution.

This is out of scope for the starter. (Old issue cleanup)


Update dng.cpp

Added support for monochromatic, because currently it doesn't work, as described in this forum post https://forums.raspberrypi.com/viewtopic.php?t=343824

I'm closing this on account that this should already be supported.


Advice for Travis-CI to handle the rebuild

I think some repositories are set up to have continuous integration build the spec. I haven't gotten my own proposals set up this way, but it seems to work well for the main spec. This way, you can even merge PRs from other contributors without having to run a rebuild step yourself as a maintainer. It'd be great if the setup for this could be ported to this repository for others to use.

Having travis do the rebuild greatly messes up the git history by adding commits every time a master build completes; I'm working on a git hook as a way for committers to easily update the build process, as well as a CI step to fail if the build process isn't run.

Travis doing the rebuild works great if you confine it to the gh-pages branch.

The process is a bit much for most people though. Even if this repo sets up a lot of it, each repo-creator would need to do these two steps, including getting the Travis CLI, generating a SSH key, adding it to GitHub as a deploy key, and editing .travis.yml to add the encryption label. I'm not sure I'd want to inflict that on people, although for me it's pretty painless these days.

That's fair; I think if travis was doing the build, it would make sense to use gh-pages instead of master (whereas using master is way easier when humans do the build).
I agree as well that that process is intimidatingly complex.

This is now handled with github actions, and should also include PR branches from (properly updated) forks.


Resolve compatibility issues after new ddr release

After ddr release there were some compatibility issues. This PR is resolving them.

PR-Codex overview
This PR focuses on updating dependencies and improving file path handling.
Detailed summary

Updated the version of the ddr artifact from 0.0.10 to 0.0.11 in the pom.xml file.
Modified the return type of the constructInPath and constructOutPath functions in several files from String to Path.
Imported the Path class in several files.
Replaced the usage of deprecated methods with alternative methods in the UnitTestBase class.
Updated the return type of the constructOutPath function in the UnitTestBase class from String to Path.
Updated the return type of the constructOutPath function in the AtomsBase class from String to Path.
Imported the getAttrContent function from the org.objectionary.ddr.util package in several files.
Removed unused imports in several files.

The following files were skipped due to too many changes: src/test/kotlin/org/objectionary/aoi/unit/atoms/AtomsBase.kt, src/main/kotlin/org/objectionary/aoi/process/AtomsProcessor.kt, src/main/kotlin/org/objectionary/aoi/launch/AoiWorkflow.kt, src/test/kotlin/org/objectionary/aoi/integration/IntegrationTestBase.kt, src/main/kotlin/org/objectionary/aoi/process/InstanceUsageProcessor.kt

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

@Graur could you please check?

@rultor merge

@rultor merge

@yegor256 OK, I'll try to merge now. You can check the progress of the merge here

@rultor merge

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


Issue for reconnect when stop using ctrl + c

Issue template

Hardware description:
Olimex STM32-E407

RTOS:
freertos

Installation type:
micro_ros_agent

Version or commit hash:
foxy

Steps to reproduce the issue

Expected behavior
Can be executed multiple times using “ ros2 run micro_ros_agent micro_ros_agent serial --dev /dev/ttyUSB0 -v6 “ when  stop using ctrc + c
Actual behavior
At present, using micros can connect to the board through “ ros2 run micro_ros_agent micro_ros_agent serial --dev /dev/ttyUSB0 -v6“, but if I stop using ctrc + c, I can't connect successfully if I want to execute it again.
Additional information
I can only connect normally after running the command “ ros2 run micro_ros_agent micro_ros_agent serial --dev /dev/ttyUSB0 -v6“ after executing “ros2 run micro_ros_setup flash_firmware.sh“  every time

You should be running flash_firmware.sh once.
Then, you need to have an open agent before the micro-ROS code starts to run. So when using the int32_example, you have to reset your board if you close the agent.
Have I understood well your question?

Examples are as follows:

Power on the board
Run the agent on the computer through the command "ros2 run micro_ros_agent micro_ros_agent serial --dev /dev/ttyUSB0 -v6"
It can be connected to the board ports normally and work normally
Turn off the agent on the computer, and press ctrl + c
Run the agent again on the computer, "ros2 run micro_ros_agent micro_ros_agent serial --dev /dev/ttyUSB0 -v6"
Stay in
[1605573834.109076] info | TermiosAgentLinux.cpp | init | running... | fd: 3
[1605573834.109228] info | Root.cpp | set_verbose_level | logger setup | verbose_level: 6
I tried many times and it was the same.
Re-power on my board or flash it again to work normally again.

Obviously, none of the examples takes care of handling the reconnection between client and agent. So if you destroy the agent, the client will not create the entities again since the examples are not like this.
You can find a reconnection example here.

@pablogs9  thank you ， i have a look !

Closing since not an issue, we can continue talking here if you have any other doubts.


可不可以统一管理快捷键!!!

可不可以统一管理快捷键!!!(比如在当前快捷键列表的基础上,增加双击修改,或者取消功能)
各种插件的快捷键,互相覆盖;很乱....
造成各种冲突....下一个版本可以考虑解决这个问题么...
非常感谢!!!!

If you can, could you please rephrase your issue into English?
As far as I know, none of the maintainers or devs of this repository know how to read this and shouldn't be asked to translate it themselves.

This is what the automatic Chrome translation came up with:

Can !!! unified management shortcuts (such as on the basis of the current list of shortcuts on the increase, double-click to modify, or cancel function)
Shortcuts various plug-ins, each covering; mess ....
Cause conflicts .... next version could be considered to solve this problem what ...
Thank you very much!!!!

In which case, I suspect it is a duplicate of https://github.com/atom/atom-keymap/issues/82

Good guess.

Astonished that there's an issue written in Chinese here. I hope this will not lead to bad impression on us Chinese developers. And I suggest a new title manage shortcuts in a unified way.

So the basic gist here is that he wants a better system to manage key bindings and add/modify/remove them since they currently can conflict with each other, etc.
Then he asks if the next version of Atom can fix this.
And finally, thanks very much.
Therefore I think this is more of a duplicate of atom/settings-view#163 than atom/atom-keymap#82.

I hope this will not lead to bad impression on us Chinese developers.

It definitely will not lead to a bad impression. It just makes things a little more time consuming :grinning:

Thanks, @tcbbd and @50Wliu.

thankyou very much，I haven't write english ，because my english  very poor…so sorry😣

No worries @crperlin!
Since this is a duplicate I'm going to close this out in favor of https://github.com/atom/settings-view/issues/163.


Improve ToXXXTimeTransformer

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 100.00%. Comparing base (2b3e13a) to head (45929e5).

Additional details and impacted files
@@            Coverage Diff            @@
##              main       #72   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           28        28
  Lines          582       572   -10
  Branches        29        27    -2
=========================================
- Hits           582       572   -10

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


added hdwallet docs

Please check that I have correctly written the docs for the hd wallet api.
@MrNaif2018

Thank you very much for your PR and sorry for delay!


Add border radius to minimap

This makes the minimap fit in more with the rest of iD by having a small border radius

sure, why not. :wink:


Implement VJPs for gamma PDF

This PR implements VJPs for the gamma PDF w.r.t. its argument and shape parameter (x and a in SciPy, respectively).

Just added the VJP for the gamma CDF w.r.t. its argument only. As far as I can tell, the derivative w.r.t. the shape parameter is non-trivial.

Thanks!


[Documentation] HDB and XS artifacts feature parity

Describe the solution you'd like
In xsk.io I'd like to see a matrix describing the HDB artifacts (.hdb*) and XS Classic artifacts (.xs*). Which ones are supported, which ones are not, if not - one sentence summary would be nice.
Each artifact should have its own page with more detailed documentation what is its comparison to the HANA
For each artifact describe the behavior, limitations in comparison to HANA in intuitive way. Color coding on top would be nice way to easily digest the information - big green letters if it's 1:1 working, yellow if there's some (small) difference in behavior, red if incompatible or major difference in behavior

fixed with #1049


community curation feed corrupted (BLOCKER)

has somehow picked up the text
Recent community curation
Browse PomBase's inventory of conserved proteins with no known biological role
Oh and the research spotlight  is showing the CC feed.....
off by one error ;)

Could you try reloading?  There was a temporary problem when I added the new spotlight item.

fine now


docs: Bootstrap coda website

Bootstraps a Sphinx website for CODA and populates it with some initial content.
DOCPR-1167

LGTM!


Is webscoket multithreaded

I am sorry if the answer is obvious but I am really new to this.
I want to know the behaviour of onMessage function. if multiple clients sends message simultaneously will each client spawn a new websocket thread?
Thank You

Yup multithreaded

To put this more formally, each client is connected on a different thread as of now (but this may be subject to change), be it plain HTTP or more advanced sub-protocols like websockets (currently the only sub-protocol we're shipping, but there were people with their eyes set on SSE here). This means care must be taken when accessing centralized data and code.


If I call getProducts, getSubscription, there is an error.

Current Version : "react-native-iap": "12.14.1"
This error is not found in the previous version of "react-native-iap": "12.12.2". The error appears only in IOS.

Step:

call getProducts
call getProducts or getSubscriptions
error

I can make a code that calls getProducts only once, but I have to call getSubscription as well.

I'm facing the same error after updating React Native from 0.71.13 to 0.74.2.
react-native-iap version (12.13.0) and its implementation remained unchanged. Everything was working fine before updating React Native.

@arthurgeron Could you kindly follow up this issue? Looks like related to #2777 2518

I'm having the same issue using the latest version of iap and react-native

In order to display the price information of the subscription page first, I called getSubscriptions after the app started, and called getSubscriptions again to update the data when entering the premium page. However, the same problem occurred as long as it was called twice. There was no problem if it was called only once.

2. getSubscriptions

Have you waited for the resolution of the first Promise before calling getSubscriptions or getProducts again?

I just deleted this parts. But It's not good method.
//reject ("E_CANCELED", "Previous request was canceled deue to a new request", nil) in node_modules/react-native-iap/ios/LatestPromiseKeeper.swift is deleted and used.
There seems to be no problem with other movements.

That only leaves the previous promise hanging infinitely, the error is on purpose. You're not meant to make these calls in parallel, a new call will override the data from the previous invocation.

@hyochan ,
We have a few options to consider:

Allow parallel calls: This could lead to multiple renders and potential issues, such as an older promise resolving after a more recent one.
Resolve canceled promises: Instead of rejecting them, we could resolve them since they don't return data anyway. However, this might be confusing in some scenarios as it results in a falsy promise resolution and could be considered an anti-pattern.
Improve documentation: We can enhance the documentation to clarify this behavior.

If you agree with the changes I've described in #2604, which includes rejecting canceled promises, I suggest we focus on updating the documentation. Changing coding patterns and behaviors solely due to user confusion might not be practical and could introduce more bugs.

There's indeed a bug, I'm working on a fix!


Improve PHPUnit fixtures

Changed log

According to the official PHPUnit doc, it should be protected function setUp(): void.

I don't have that checked box on the GitHub organization. I'm going to close this PR and fork this repository on my GitHub user repository.


chore: release 0.19.1

release notes

Codecov Report

Merging #156 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #156   +/-   ##
=======================================
  Coverage   97.67%   97.67%
=======================================
  Files          15       15
  Lines        2879     2879
  Branches      176      176
=======================================
  Hits         2812     2812
  Misses         41       41
  Partials       26       26

Continue to review full report at Codecov.

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