Mysterious lack of compilation error

Hi,
I'm a relatively new Flutter developer, who came from regular, native Android apps where strongly statically typed languages like Kotlin athe the first class citizens.
I was executing tasks from the official (?) codelab when I found an interesting piece of code which compiles with no single error or even warning but unfortunately  fails at runtime.
Take a look at the following method from the second part of the above codelab:
 void _pushSaved() {
    Navigator.of(context).push(
      MaterialPageRoute<void>(
        builder: (BuildContext context) {
          final Iterable<ListTile> tiles = _saved.map(
            (WordPair pair) {
              return ListTile(
                title: Text(
                  pair.asPascalCase,
                  style: _biggerFont,
                ),
              );
            },
          );
          final List<Widget> divided = ListTile.divideTiles(
            context: context,
            tiles: tiles,
          ).toList();
        },
      ),
    );
  }

This method is basically called on some kind of user action and it should open a new screen by pushing new MaterialPageRoute.
What I found interesting is the way MaterialPageRoute's constructor is being invoked:
MaterialPageRoute<void>(
   builder: (BuildContext context) {
       ...
   }
)

As far as I understand, builder parameter is type of WidgetBuilder which is actually a typedef of a function which has to return a Widget:
typedef WidgetBuilder = Widget Function(BuildContext context);

Surprisingly, implementing this function without returning any Widget still compiles but when I try to run the app and push this new route I get this runtime crash:
I/flutter (28374): ══╡ EXCEPTION CAUGHT BY WIDGETS LIBRARY ╞═══════════════════════════════════════════════════════════
I/flutter (28374): The following assertion was thrown building Builder(dirty, dependencies:
I/flutter (28374): [_LocalizationsScope-[GlobalKey#0b08b], _InheritedTheme]):
I/flutter (28374): The builder for route "null" returned null.
I/flutter (28374): Route builders must never return null.
I/flutter (28374):
I/flutter (28374): When the exception was thrown, this was the stack:
I/flutter (28374): #0      MaterialPageRoute.buildPage.<anonymous closure> (package:flutter/src/material/page.dart:90:9)
I/flutter (28374): #1      MaterialPageRoute.buildPage (package:flutter/src/material/page.dart:96:6)
I/flutter (28374): #2      _ModalScopeState.build.<anonymous closure> (package:flutter/src/widgets/routes.dart:666:43)
I/flutter (28374): #3      Builder.build (package:flutter/src/widgets/basic.dart:6427:41)
I/flutter (28374): #4      StatelessElement.build (package:flutter/src/widgets/framework.dart:3974:28)

I'm using latest (as of 1st July 2019) recommended SDK: flutter_macos_v1.5.4-hotfix.2-stable.zip. I found that in the master branch there's new parameter in analysis_options.yaml:
missing_return: warning

Changing it to error indeed causes IDE to show the problem but it's just a static analysis, not a compiler error as this code still builds successfully by running flutter run:

I'm just wondering, wether is it a Dart compiler or any other tooling issue or am I missing something which is most likely more Dart language specific thing than the Flutter framework itself?
Thanks in advance for any comments.

This is indeed a Dart language behavior, not something specific to Flutter. See https://dart.dev/guides/language/language-tour#return-values

Fine, I get it. I can't understand though how function can be declared as returning something Widget in this case) and does not return it when implemented as lambda. Either way, thanks for clarification. Will find some resources about that.

I can't understand though how function can be declared as returning something (Widget in this case) and do not return it when implemented as lambda.

I linked to the explanation in my comment above: if you don't explicitly return something, a return null is implicitly added.

Ok, i think I get it after reading it for the second time.
It's more or less like in Java where public String foo() { return null; } is a valid code whereas in Dart return statement can be ommited and the code behaves the same.
When I try to explicitly return a value but of different type, like:
  return "foo";

compiler fails, as expected:
lib/main.dart:64:18: Error: A value of type 'String' can't be assigned to a variable of type 'Widget'.

Thanks for your help. I found it a little bit tricky and consider it as error-prone, but that's just how language works and static analysis can definitely help here.


n8n-nodes-netsuite becomes unrecognized everytime I restart n8n

The error:
The following error occurred on workflow activation:
Unrecognized node type: n8n-nodes-netsuite.n8n-nodes-netsuite.netsuite".

I discovered that it will only have this issue if I use postgresql as backend db. SQLite db did not have this issue.
So I compared data in sqlite and postgresql. Below is the table installed_nodes.
SQLite

name
type
latestVersion
package

NetSuite
n8n-nodes-netsuite.netsuite
1
n8n-nodes-netsuite

Postgresql

name
type
latestVersion
package

NetSuite
n8n-nodes-netsuite.n8n-nodes-netsuite.netsuite
1
n8n-nodes-netsuite

The value in column type are different in the two databases. I hope this helps in troubleshooting. When I update postgresql value to n8n-nodes-netsuite.netsuite, this issue does not happen anymore.
I don't know if this is n8n issue or this node.


Infra flags for fulcio / rekor / oidc values

Implements flags for fulco, rekor and OIDC values to allow a user to set their own custom values.
The defaults fall to the public good service

Looks like you might need to plumb the flag through to the e2e tests to point at localhost


Contact

Hey!
Pretty nice project you are working on!
I would like to chat with you a bit for one thing, can you drop me an email at<EMAIL_ADDRESS>?
(Sorry to open an issue here, didn't know how else to contact you)

Thank you so much for your interest 🙏.
Here's my discord id: watchdogs132#8237.  I think it would be better to chat there than by mail. Although if you prefer the latter then sure I will drop a mail. Thanks!


[Config] Simplify RelayMiner config

Objective
Simplify the RelayMiner's relay_miner_config.yaml, providing operators with a more intuitive config file.
Origin Document
Private net external testers expressed confusion w.r.t. how the RelayMiners are configured.

Goals

Have a more intuitive relay_miner_config.yaml reflecting the RelayMiner's setup
Clear-up RelayMiner's confusion w.r.t. the proxies section and how it's linked to the suppliers section.

Deliverables

[ ] A signle PR changing the RelayMiner configuration parser to handle the following structure:
https://gist.github.com/okdas/c20a2c0d778f4c291c8388a386b52d20

Non-goals / Non-deliverables

Change how the RelayMiner sets up its servers
Change the structure resulting from parsing the config file

General deliverables

[ ] Comments: Add/update TODOs and comments alongside the source code so it is easier to follow.
[ ] Testing: Add new tests (unit and/or E2E) to the test suite.
[ ] Documentation: Update architectural or development READMEs; use mermaid diagrams where appropriate.

Creator: @red-0ne
Co-Owners: @okdas

Closed as per #477


how should look upload.php?

help me, please

the code will come tomorrow, ok

Why so long?

https://github.com/CreativeDream/php-uploader

Thanks! I wish you success!


Crashes with standard equality check

I applied this change:
diff --git a/test/closed_intervals_test.exs b/test/closed_intervals_test.exs
index 36f1c90..9348dbb 100644
--- a/test/closed_intervals_test.exs
+++ b/test/closed_intervals_test.exs
@@ -50,7 +50,7 @@ defmodule ClosedIntervalsTest do
     end

     test "get_interval" do
-      tree = from([1, 2, 3, 4])
+      tree = from([1, 2, 3, 4], eq: &==/2)
       assert {1, 2} == get_interval(tree, 2)
       assert {1, 2} == get_interval(tree, 1.5)
       assert {2, 3} == get_interval(tree, 2.5)

It crashes:
$ mix test test/closed_intervals_test.exs:52
Excluding tags: [:test]
Including tags: [line: "52"]

  1) test default order get_interval (ClosedIntervalsTest)
     test/closed_intervals_test.exs:52
     ** (CaseClauseError) no case clause matching: [{1, 2}, {2, 3}]
     code: assert {1, 2} == get_interval(tree, 2)
     stacktrace:
       (closed_intervals 0.5.0) lib/closed_intervals.ex:166: ClosedIntervals.get_interval/2
       test/closed_intervals_test.exs:54: (test)

Finished in 0.1 seconds (0.00s async, 0.1s sync)
11 doctests, 2 properties, 6 tests, 1 failure, 18 excluded

Randomized with seed 659730

I find that surprising since this is the most straightforward equality function I can think of. Found while working at a PR for #15

Interesting. This crashes because the interval for  2 is not unique. Maybe this should be signaled instead of crashing.


Heap Corruption everytime esp_http_read is called multiple times (IDFGH-4391)

Hello,
I am trying to read data from a server that sends non chunked response of the size of ~800kb. I am trying to read this as below. However, there is always heap corruption on the second read/while loop iteration.
Code:

while (total_read_bytes_tts <= content_length_tts) {
ESP_LOGI(PTAG, "setting bytes to 0");
memset(buffer_tts, 0, MAX_HTTP_RECV_BUFFER_TTS + 1);
ESP_LOGI(PTAG, "reading more bytes");
read_bytes_tts = esp_http_client_read(client_tts, buffer_tts, MAX_HTTP_RECV_BUFFER_TTS);
if (read_bytes_tts <= 0) {
ESP_LOGI(PTAG, "ERROR READING TTS DATA");
esp_periph_set_destroy(set);
free(buffer_tts);
return 0;
}
total_read_bytes_tts += read_bytes_tts;
ESP_LOGI(PTAG, "total bytes read: %d", total_read_bytes_tts);
//ESP_LOGI(PTAG, "bytes read: %s", buf);
//  free(buffer_tts);
}

and the error:

CORRUPT HEAP: multi_heap.c:432 detected at 0x3ffcf81c
abort() was called at PC 0x40094ccb on core 0
0x40094ccb: multi_heap_assert at C:/esp/esp-idf/components/heap/multi_heap_platform.h:54
(inlined by) multi_heap_malloc_impl at C:/esp/esp-idf/components/heap/multi_heap.c:432
ELF file SHA256: 17b375fb87930e74b4fd0c78c242cf8b44b294063da5fd8271ffaef6f0aa5438
Backtrace: 0x40091d75:0x3ffbe770 0x400920ed:0x3ffbe790 0x40094ccb:0x3ffbe7b0 0x4009507f:0x3ffbe7d0 0x40082565:0x3ffbe7f0 0x40082595:0x3ffbe810 0x40098340:0x3ffbe830 0x40098364:0x3ffbe850 0x400d5dca:0x3ffbe870 0x40135a85:0x3ffbe8e0 0x400f29cf:0x3ffbe900 0x40193791:0x3ffbe930 0x400f2a42:0x3ffbe950 0x4019374d:0x3ffbe970 0x400ddfcd:0x3ffbe990 0x400d7bde:0x3ffbe9b0 0x400d801b:0x3ffbeac0 0x400d34de:0x3ffbeae0 0x40095229:0x3ffbeb00
0x40091d75: invoke_abort at C:/esp/esp-idf/components/esp32/panic.c:155
0x400920ed: abort at C:/esp/esp-idf/components/esp32/panic.c:172
0x40094ccb: multi_heap_assert at C:/esp/esp-idf/components/heap/multi_heap_platform.h:54
(inlined by) multi_heap_malloc_impl at C:/esp/esp-idf/components/heap/multi_heap.c:432
0x4009507f: multi_heap_malloc at C:/esp/esp-idf/components/heap/multi_heap_poisoning.c:191
0x40082565: heap_caps_malloc at C:/esp/esp-idf/components/heap/heap_caps.c:111
0x40082595: heap_caps_malloc_default at C:/esp/esp-idf/components/heap/heap_caps.c:140
0x40098340: _calloc_r at C:/esp/esp-idf/components/newlib/heap.c:73
0x40098364: calloc at C:/esp/esp-idf/components/newlib/heap.c:37
0x400d5dca: esp_vfs_select at C:/esp/esp-idf/components/vfs/vfs.c:875
0x40135a85: select at C:/esp/esp-idf/components/newlib/select.c:62
0x400f29cf: tcp_poll_read at C:/esp/esp-idf/components/tcp_transport/transport_tcp.c:123
0x40193791: esp_transport_poll_read at C:/esp/esp-idf/components/tcp_transport/transport.c:198
0x400f2a42: tcp_read at C:/esp/esp-idf/components/tcp_transport/transport_tcp.c:105
0x4019374d: esp_transport_read at C:/esp/esp-idf/components/tcp_transport/transport.c:182
0x400ddfcd: esp_http_client_read at C:/esp/esp-idf/components/esp_http_client/esp_http_client.c:830
0x400d7bde: wav_http_post at c:\esp\esp-adf\examples\speech_recognition\asr-updated-v2\build/../main/srcs/wav_http_post.c:348 (discriminator 9)
0x400d801b: app_main at c:\esp\esp-adf\examples\speech_recognition\asr-updated-v2\build/../main/srcs/example_asr_main.c:270 (discriminator 9)
0x400d34de: main_task at C:/esp/esp-idf/components/esp32/cpu_start.c:554
0x40095229: vPortTaskWrapper at C:/esp/esp-idf/components/freertos/port.c:143

There are large chunks of information missing from your issue, please first look at the bug report template and provide all essential information (idf & tool versions, etc). You also provided a very limited snippet of your code, without how you actually allocated the buffer, how large it is, how you established connection, any log output outside of the error, and other maybe seemingly-unimportant details. I can confirm that for me, esp_http_read currently works correctly on the master version of ESP-IDF, as I do a very similar thing in my OTA code.
As a side note, please use the code formatting option when posting snppets, and not the quote formatting. There is a large difference in readability.
Insert code option (using: ``` newline code newline ```)
while (total_read_bytes_tts <= content_length_tts) {

Insert quote option (using: > quote)

while (total_read_bytes_tts <= content_length_tts) {

@rjam-xav can you please provide some small test code which I can try at my end and reproduce the issue ?
The missing information suggested by @istokm will indeed help to reproduce and debug the issue.
If the problem is already resolved on your end can you please close the issue ?

@rjam-xav can you please provide some small test code which I can try at my end and reproduce the issue ?
The missing information suggested by @istokm will indeed help to reproduce and debug the issue.
If the problem is already resolved on your end can you please close the issue ?

Closing due to lack of update. Please feel free to reopen with more information as requested.

Closing due to lack of update. Please feel free to reopen with more information as requested.


Fix Servfail on malformed query

Modifying dns.msg directly seems to produce broken DNS query, causing SERVFAIL from server.  I modified to use dns.Msg API , and it's fixed.
I put the before BEFORE and AFTER below. Also added a benchmark to make sure using API did not regress performance.
Known Bugs

pRecurse is not being set properly. will fix before merging

AFTER
SERVFAIL count = 0
# this branch
$ go run . -s <IP_ADDRESS> www.ucla.edu
Benchmarking <IP_ADDRESS>:53 via udp with 1 concurrent requests

Total requests:  1 of 1 (100.0%)
DNS success codes:      1

DNS response codes
        NOERROR:        1

Test with NC
 nc -lup 5353 | hexdump -c
0000000      M 001  \0  \0 001  \0  \0  \0  \0  \0  \0 004   u   c   l

0000010   a 003   e   d   u  \0  \0 001  \0 001
000001a

BEFORE
SERVFAIL count = 1
tonymet@DESKTOP-51A0S7U:~/go/src/github.com/redsift/dnstrace$ git checkout master
Switched to branch 'master'
$ go run . -s <IP_ADDRESS> www.ucla.edu
Benchmarking <IP_ADDRESS>:53 via udp with 1 concurrent requests
Total requests:  1 of 1 (100.0%)
DNS success codes:      0

DNS response codes
        SERVFAIL:       1

TEST Query with NC
nc -lup 5353 | hexdump -c
0000000   ] 020  \0  \0  \0 001  \0  \0  \0  \0  \0  \0 004   u   c   l

0000010   a 003   e   d   u  \0  \0 001  \0 001
000001a

Bench mark "raw" (BEFORE) vs "wrapped" (AFTER)
raw = 933ns/op & 3 allocs
wrapped = 10759 ns/op & 1 alloc

tonymet@DESKTOP-51A0S7U:~/go/src/github.com/redsift/dnstrace$ go test -test.bench  .
goos: linux
goarch: amd64
pkg: github.com/redsift/dnstrace
cpu: Intel(R) Core(TM) i5-8500T CPU @ 2.10GHz
BenchmarkMessageRaw-6             110208             10759 ns/op            5376 B/op          1 allocs/op
BenchmarkMsgWrapped-6            1303633               933.0 ns/op            28 B/op          3 allocs/op
PASS
ok      github.com/redsift/dnstrace     3.454s

@csucu can you see if you observe SERVFAIL and if this patch fixes it? if it does I can clean up for merge.

bump

Been away on vacation, will take a look this week

I was able to repro on a common dns server <IP_ADDRESS>
 go run . -s <IP_ADDRESS> -c 2 -n 10 www.ucla.edu
Benchmarking <IP_ADDRESS>:53 via udp with 2 concurrent requests

Total requests:	 20 of 20 (100.0%)
DNS success codes:	0

DNS response codes
	SERVFAIL:	20

Time taken for tests:	 151.803024ms
Questions per second:	 131.7

DNS timings, 20 datapoints
	 min:		 13.631488ms
	 mean:		 14.942208ms
	 [+/-sd]:	 597.78µs
	 max:		 16.252927ms

DNS distribution, 20 datapoints

Recursion is false by default, i get no errors when running with -r flag. i.e go run . -r -s <IP_ADDRESS> www.ucla.edu

ok thanks for explaining. i was able to get this behavior with dig as well.

From the user's point of view, I think recurse=true would be a better default (e.g. it's the default with dig).
From developer's point of view, perf seems better with this patch.

What are your thoughts on both


code does not seem to build with brotli? v1.0.2

Error: Command failed: bro -q 11 --input ...
 at checkExecSyncError (child_process.js:601:13)
    at execSync (child_process.js:641:13)
    at brotliFile (/.../dev/preact-hn/scripts/minify.js:32:3)
    at optimizeFileDelivery (/.../dev/preact-hn/scripts/minify.js:45:7)
    at /.../dev/preact-hn/scripts/minify.js:59:5
    at Object.<anonymous> (/.../dev/preact-hn/scripts/minify.js:67:3)
    at Module._compile (module.js:635:30)
    at Object.Module._extensions..js (module.js:646:10)
    at Module.load (module.js:554:32)
    at tryModuleLoad (module.js:497:12)

Hmm, seems to be failing on brotli encoding.
Can you verify that you can encode arbitrary files? bro -q 11 --input ${filePath} --output ${filePath + '.br'}
If this fails it indicates an issue with the installation of brotli.

We can use WASM implementation instead of external dependency https://github.com/dfrankland/wasm-brotli


Pimple

@evrifaessa @fordnox I have this error that I have encountered and would like to have some assistance in resolving. syntax error, unexpected namespaced name "implements\IteratorAggregate", expecting "{"

@evrifaessa @fordnox  Closing branch due to fix in pull request #863


[EM] Add GPU version of the external memory QDM.

The second part of https://github.com/dmlc/xgboost/pull/10682 .
This PR adds basic GPU support for the  external memory QDM, some features still need to be worked on:
TO-DO:

Interpolation between CPU and GPU.
Exporting data to CSR.
Expose the features to the interface.

In addition, we will change the GPU updater to support batch-based training in the coming PRs. I haven't decided whether we should support the page concatenation in the future. It will depend on the difficulty of supporting two batch-based training paradigms.

cc @rongou .


build: run tests against latest Microsoft Edge version

Updates the Microsoft Edge version that is used by BrowserStack. We recently
hit a lot of flakes with getComputedStyle which sometimes didn't return the
resolved value for requested CSS properties in the grid-list tests.
I tried looking for bug reports in the Edge issue tracker, also tried reproducing the grid-list test flakes locally.. but wasn't able to reproduce these. It could be a broken version in BrowserStack that causes
the flaky value resolution w/ getComputedStyle. I didn't find anything odd in the tests itself as these execute properly in other browsers and the grid tile elements are rendered (added to DOM)

Closing as this unveiled permanent failures within the select. It's not in scope for this PR to fix/investigate these. cc. @crisbeto in case you are interested (see CI job)


some small fixes to grammar and spelling

I have made some small fixes to spelling and grammar across the whole codebase. I think these changes are rather uncontroversial.
I also added a clarifying sentence in the mushroom tutorial, maybe that needs to be checked for contents.

LGTM!


'The preLaunchTask 'xxxx' cannot be tracked.' exception after installing the October 1.18 release

VSCode Version: 1.18.0
OS Version: Win10

Steps to Reproduce:

Use a task to run a dotnet command to build the project as a preLaunchTask in launch.json.
Debug the project.

Reproduces without extensions: Yes
In launch.json ...
{
    "name": ".NET Core Launch (web)",
    "type": "coreclr",
    "request": "launch",
    "preLaunchTask": "build",
    "program": "${workspaceRoot}/bin/Debug/netcoreapp2.0/ChangeTokenSample.dll",
    "args": [],
    "cwd": "${workspaceRoot}",
    ...

build task [I'm using an older format (e.g., taskName over label) due to some requirements here. tasks.json version: 0.1.0] ...
{
    "taskName": "build",
    "suppressTaskName": true,
    "args" : ["dotnet", "build", "--configuration", "Debug"],
    "showOutput": "always",
    "isBuildCommand": true,
    "problemMatcher": "$msCompile"
},

Starting with the 1.18.0 release, when I run the debugger, it throws this exception ...
The preLaunchTask 'build' cannot be tracked.

If that's normal now (i.e., not a bug), is there any way to disable it from throwing? The debugger still works fine, and the build task works fine all by itself, but I must click past this exception with the Debug Anyway button in order to get the debug session running. I'm not concerned whether or not the pre-launch task can be tracked or not (at least I'm not concerned yet :smile: lol).

I am having this issue as well. Started happening after the upgrade.

I'm in the process now of testing with a 2.0.0 version of tasks.json. Maybe my use of an ancient version is hurting me. lol

Yes ... that seems to have done the trick. I've upgraded my old tasks.json to 2.0 ...
{
    "type": "shell",
    "taskName": "build",
    "label": "build",
    "command" : "dotnet build --configuration Debug",
    "group": {
        "kind": "build",
        "isDefault": true
    },
    "problemMatcher": "$msCompile"
},

... and that silenced the exception.
Although it's obsolete, I must leave the old taskName in there for now. I have an old extension that still uses it.
I'll leave this open for a sec so that the team has a chance to say why this came up tho for version 0.1.0 of the tasks.json file.

@dbaeumer please advise

I have the same problems after upgrade to 1.18.
My tasks.json:
{
  "version": "2.0.0",
  "tasks": [
    {
      "label": "Launch Function App",
      "identifier": "launchFunctionApp",
      "type": "shell",
      "command": "func host start --script-root target/azure-functions/test-1510276830503/",
      "isBackground": true,
      "presentation": {
        "reveal": "always"
      },
      "problemMatcher": [
        {
          "owner": "azureFunctions",
          "pattern": [
            {
              "regexp": "\\b\\B",
              "file": 1,
              "location": 2,
              "message": 3
            }
          ],
          "background": {
            "activeOnStart": true,
            "beginsPattern": "^.*Stopping host.*",
            "endsPattern": "^.*Job host started.*"
          }
        }
      ],
      "dependsOn": [
        "Maven Clean Package"
      ]
    },
    {
      "label": "Maven Clean Package",
      "type": "shell",
      "command": "mvn clean package",
      "isBackground": true
    }
  ]
}

I tried to reproduce this with a simple dotnet console application and was not able to using the following files
tasks.json
{
	"version": "0.1.0",
	"tasks": [
		{
			"taskName": "build",
			"suppressTaskName": true,
			"command": "dotnet",
			"args": [
				"build",
				"--configuration",
				"Debug"
			],
			"showOutput": "always",
			"isBuildCommand": true,
			"problemMatcher": "$msCompile"
		}
	]
}

launch.json
{
   // Use IntelliSense to find out which attributes exist for C# debugging
   // Use hover for the description of the existing attributes
   // For further information visit https://github.com/OmniSharp/omnisharp-vscode/blob/master/debugger-launchjson.md
   "version": "0.2.0",
   "configurations": [
        {
            "name": ".NET Core Launch (console)",
            "type": "coreclr",
            "request": "launch",
            "preLaunchTask": "build",
            // If you have changed target frameworks, make sure to update the program path.
            "program": "${workspaceFolder}/bin/Debug/netcoreapp2.0/37997.dll",
            "args": [],
            "cwd": "${workspaceFolder}",
            // For more information about the 'console' field, see https://github.com/OmniSharp/omnisharp-vscode/blob/master/debugger-launchjson.md#console-terminal-window
            "console": "internalConsole",
            "stopAtEntry": false,
            "internalConsoleOptions": "openOnSessionStart"
        },
        {
            "name": ".NET Core Attach",
            "type": "coreclr",
            "request": "attach",
            "processId": "${command:pickProcess}"
        }
    ]
}

I generated the example using dotnet new console.

So tasks 0.1.0 seems to send the right events.
@guardrex can you provide a more complete example so that we can investigate. Could it be that the tasks.json specifies isWatching / isBackground also the dotnet command is not actually running in background?
@jdneo example I think is different. It uses 2.0.0 and I guess that debugging never receives an activate event. But without a complete example it is hard to say why this is happening.
@isidorn may be we should introduce a special event to make this less dependent on a correct configuration.

@dbaeumer Sure thing. I'm slammed this afternoon, but I'll get that for you by the end of the day.

@dbaeumer Yes, it still repros with the Change Tokens sample I was working on. If I click to Debug Anyway, it seems to run fine. My fix is simply to take my task.json files to 2.0.0, so I'm not blocked.
https://github.com/guardrex/ChangeTokenSample

+1 having this issue. This is my tasks.json:
{
    "version": "0.1.0",
    "command": "dotnet",
    "isShellCommand": true,
    "args": [],
    "tasks": [
    	{
            "taskName": "build",
	    	"command": "${workspaceRoot}/build.sh",
	    	"isShellCommand": true,
	    	"isBuildCommand": true
        },
    	{
    		"taskName": "transpile",
    		"command": "tsc",
    		"args": ["-p", "./wwwroot/js/tsconfig.json", "--outDir", "./wwwroot/js/"],
    		"isShellCommand": true,
    		"isTestCommand": true
    	}
    ]
}

Hi @dbaeumer, thanks for replying. Here is my launch.json:
{
  "version": "0.2.0",
  "configurations": [
    {
      "name": "Attach to Azure Functions",
      "type": "java",
      "request": "attach",
      "hostName": "localhost",
      "port": 5005,
      "preLaunchTask": "launchFunctionApp"
    }
  ]
}

and task.json:
{
  "version": "2.0.0",
  "tasks": [
    {
      "label": "Launch Function App",
      "identifier": "launchFunctionApp",
      "type": "shell",
      "command": "func host start --script-root target/azure-functions/test-1510276830503/",
      "isBackground": true,
      "presentation": {
        "reveal": "always"
      },
      "problemMatcher": [
        {
          "owner": "azureFunctions",
          "pattern": [
            {
              "regexp": "\\b\\B",
              "file": 1,
              "location": 2,
              "message": 3
            }
          ],
          "background": {
            "activeOnStart": true,
            "beginsPattern": "^.*Stopping host.*",
            "endsPattern": "^.*Job host started.*"
          }
        }
      ],
      "dependsOn": "mavenCleanPackage"
    },
    {
      "label": "Maven Clean Package",
      "identifier": "mavenCleanPackage",
      "type": "shell",
      "command": "mvn",
      "args": [
        "clean",
        "package"
      ],
      "isBackground": true
    }
  ]
}

@guardrex interestingly I was not able to reproduce this using the change token sample. When you see that happen does it take a long time until the task starts. Debug waits 10 seconds for the start to happen and if it doesn't signals this to the user.

OK. I debugged this a little and the reason is that node emitters are deferring events.  Especially when the started task takes a while to spawn the child process since this is async.
To make sure we are not hit by this event firing needs to happen sync which I actually hate to do.
@isidorn, @weinand may be we need to thing about a different strategy about detecting a not completed start.

@dbaeumer are you saying that the start event is delayed and debug's timeout of 10 seconds is not sufficient?

Yes, it looks like this. I debugged through the example I got from @guardrex and all events are fired correctly, but the node event emitter delays them in a deferred queue. Right after that we do some child process spawning.
I would assume that 10 seconds should be enough. So may be you could look at the debug code as well to see if there is something that could cause some problems.

When you see that happen does it take a long time until the task starts.

Yes, I repeated the debugging startup several times and timed it, and the message pops up right at ~10 seconds every time.

If you still need a repro, I can reproduce this over at TypeScript with a launch.json of
{
    // Use IntelliSense to learn about possible Node.js debug attributes.
    // Hover to view descriptions of existing attributes.
    // For more information, visit: https://go.microsoft.com/fwlink/?linkid=830387
    "version": "0.2.0",
    "configurations": [
        {
            "type": "node",
            "request": "launch",
            "name": "Mocha Tests (currently opened test)",
            "program": "${workspaceRoot}/node_modules/mocha/bin/_mocha",
            "args": [
                "-u",
                "tdd",
                "--timeout",
                "2000000",
                "--colors",
                "built/local/run.js",
                "-f",
                "${fileBasenameNoExtension}"
            ],
            "env": {
                "NODE_ENV": "testing"
            },
            "stopOnEntry": false,
            "sourceMaps": true,
            "console": "integratedTerminal",
            "preLaunchTask": "tests"
        },
        {
            "type": "node",
            "request": "attach",
            "name": "Attach to Process",
            "processId": "${command:PickProcess}",
            "port": 9229
        },
        {
            "type": "node",
            "request": "launch",
            "name": "Launch Program",
            "program": "${workspaceRoot}\\lib\\typescript.js",
            "outFiles": [
                "${workspaceRoot}/out/**/*.js"
            ]
        }
    ]
}

while attempting to run the Mocha Tests (currently opened test) task with no built target yet (so the preLaunchTask takes time to run).
The tasks.json reads:
// Available variables which can be used inside of strings.
// ${workspaceRoot}: the root folder of the team
// ${file}: the current opened file
// ${fileBasename}: the current opened file's basename
// ${fileDirname}: the current opened file's dirname
// ${fileExtname}: the current opened file's extension
// ${cwd}: the current working directory of the spawned process
{
    "version": "0.1.0",
    "command": "gulp",
    "isShellCommand": true,
    "showOutput": "silent",
    "tasks": [
        {
            "taskName": "local",
            "isBuildCommand": true,
            "showOutput": "silent",
            "problemMatcher": [
                "$tsc"
            ]
        },
        {
            "taskName": "tests",
            "showOutput": "silent",
            "problemMatcher": [
                "$tsc"
            ]
        }
    ]
}

Worked fine prior to this release.

Just to clarify some things: we wait 10 seconds for the task to report it started not for the task to finish. This basically means  this event, which we sent out before we actually start the task using either a terminal in task v2 or a child process in v0.1.0 is not being delivered to the debug service in 10 seconds. The only difference are background tasks. For those we fire the event when finding the begin pattern in the output. But neither the example from @guardrex nor from @weswigham uses background tasks.

@dbaeumer Thanks Dirk for continuing the investigation. My situation has cleared up thus far by upgrading my files to v2.0.0, but it looks like some cats are running into this even with v2.0.0 files. I will say as a funny side note that I'm on ...

i7-3770K @ 3.50 GHz

Time for an upgrade to an i9?? lol :smile: ... I was looking for an excuse to build a new PC. Let me know.  lol :grinning:

The same is in PowerShell repo. I tried to switch to v2 and it works.

Continued debugging this and the events are NOT deferred. This was a source map problem :-(. After debugging this in plan JS all events where sent synch.
However looking at the debug code I noticed the following problems:

the task is not marked as taskStarted if the start fails (the error case)
WinJS promises that are resolved executed sync on then callbacks.

For the debug code that mean that the code here:
			const promise = this.taskService.getActiveTasks().then(tasks => {
				if (tasks.filter(t => t._id === task._id).length) {
					// task is already running - nothing to do.
					return TPromise.as(null);
				}

				const taskPromise = this.taskService.run(task);
				if (task.isBackground) {
					return new TPromise((c, e) => this.toDispose.push(this.taskService.addOneTimeListener(TaskServiceEvents.Inactive, () => c(null))));
				}

				return taskPromise;
			});

Could execute full sync when the task system is full resolve (I cash promises heavily and reuse their result if not invalidated)
If this happens the TaskServiceEvents.Active listener is hook to late and might miss the event. If the task then needs more than 10 seconds to complete the message is shown.
@isidorn can you please have a look.
Besides the above I still the code might have the potential to leak listeners as well. If you miss the event then the listener only gets disposed if the debug service gets disposed. So IMO the code should keep the disposable locally and dispose it when the tasks either finished or errors.

@isidorn See the attached screen shot which shows the sequence in the debugger:

Since this issue affects users of the "preLaunchTask" attribute across all debuggers, I think a fix should go into the recovery release.

@dbaeumer thanks for this great investigation.
Following your advice I have pushed https://github.com/Microsoft/vscode/commit/304e00115bd0515d64a78e8e58660a85c489a5b8
@dbaeumer Can you please code review?
I have checked and everything seems to work fine on my machine. Though please note that I was never unable to reproduce the issue described in the first comment.
As for the listeners leak this is fixed via 2432b746f4 however that I will not chery pick on release since this is not critical, thus the seperate commit.

@isidorn the change looks good to me. However I think in the error case what could happen is the following:

the run fails early before the event is sent
the timer is still installed
taskStarted is not set to true
the user sees an error that the task failed
10 seconds later he gets an error saying that the task can't be tracked.

May be not critical for a recovery build

@dbaeumer thanks for the review.
However I think your observation can not happen, because if the task failed before the event is sent then the promise will already complete with the error case here
The timeout will after 10s try to copmlete the promise with the error case, but that already happened and this will not have any effect.
Even if I am wrong I think this is not critical for the recovery build, thus I am cherry picking my commit on top of the release branch.
Thanks!

@isidorn your analysis is correct. The second resolve of the error promise will be a noop.

It would be great if someone from the users could verify the fix solves the issue for them. Here are the links to vscode builds which containt the fix
[win64(]https://az764295.vo.msecnd.net/insider/3a7d517f65212de615f469587c2ace53275ce96e/VSCodeSetup-x64-1.19.0-insider.exe)
win32
mac
If somebody is on ubuntu let me know what version of code you need and I will provide a link

It (win32) works for me - many thanks!

@iSazonov great, thanks for verifying. This will be in stable in the next couple of days

Is there any workaround for this for the time being?
Is a fix "a long way off" or "coming in the next few days/weeks"?

Thanks!

use vscode insiders
next few days


Doesn't work with http.FileServerFS

Title says it all. It doesn't work with http.FileServerFS
Note that this is said by the docs:

The files provided by fsys must implement io.Seeker.

I guess this filesystem doesn't.

It seems afero's memfs is better for that, asit implements everything
https://pkg.go.dev/github.com/spf13/afero#section-readme


Update category of 'iOS 10 Day by Day'

Update category of 'iOS 10 Day by Day' from 'Tutorials and Keynotes' to 'News, Blogs and more', to match category with 'iOS9-day-by-day' and 'iOS8-day-by-day'

          1 Error

:no_entry_sign:
Found 3 link issues

Link issues by awesome_bot

Line
Status
Link

976
301
https://moltin.com/ios-ecommerce-sdk redirects tohttp://moltin.com/ios-ecommerce-sdk/

2083
301
https://buddybuild.com/ redirects tohttps://www.buddybuild.com/

2320
Error
http://www.swiftplayhouse.com/  Net::ReadTimeout

  Generated by :no_entry_sign: danger


Update status.py

Pull Request
Description
Please delete the italicised instruction text!
Please include a summary of the change and which issue is fixed.
Please also include relevant motivation and context. List any dependencies that are required for this change.
Fixes #
How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce.
Please also list any relevant details for your test configuration

[ ] Yes

If your changes affect data processing, have you plotted any changes? i.e. have you done a quick sanity check?

[ ] Yes

Checklist:

[ ] My code follows OCF's coding style guidelines
[ ] I have performed a self-review of my own code
[ ] I have made corresponding changes to the documentation
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] I have checked my code and corrected any misspellings

Codecov Report

Merging #109 (58adf46) into main (dda4e9a) will increase coverage by 0.12%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main     #109      +/-   ##
==========================================
+ Coverage   98.70%   98.82%   +0.12%
==========================================
  Files           9       11       +2
  Lines         309      341      +32
==========================================
+ Hits          305      337      +32
  Misses          4        4

Impacted Files
Coverage Δ

src/gsp.py
100.00% <ø> (ø)

src/database.py
94.23% <100.00%> (+0.48%)
:arrow_up:

src/main.py
97.05% <100.00%> (+0.18%)
:arrow_up:

src/status.py
100.00% <100.00%> (ø)

src/tests/test_gsp.py
100.00% <100.00%> (ø)

src/tests/test_status.py
100.00% <100.00%> (ø)

:mega: Codecov can now indicate which changes are the most critical in Pull Requests. Learn more


Yaml/toml/json builders.

Implement a crate that reads a configuration file and creates a drain defined in it. Box<Drain> will be used all over the place, which is supported and perfectly fine.

Maybe write at least a proof of concept before freezing things in 1.0 ...

Will do after 1.0.

Prototyping...

https://github.com/slog-rs/config in progress...

@dpc Perhaps we could collaborate here? I'm writing a general configuration library to be used by frameworks and libraries.
The idea is a library (such as slog-rs) just uses it config::get("output.stderr.type") and an application or framework will define where the configuration comes from.
config::merge(config::File::with_name("Settings")).unwrap();

https://github.com/mehcode/config-rs

It is definitely something to consider. I will try to investigate more when I had some spare time.

@mehcode , I did some longer investigation.

You could make first Source implementation a  &str/String. Then File can be built on top of it, and people can generally fetch String from anywhere they want. I wonder if it could even be impl Source for std::fs::File. Generally in slog-config I'd like to retain the ability to feed it &str, that's why I'm thinking about it.
There are quite a few features used. Right now slog-config it's serde that requires nightly and that should be fixed when procedural macros land in stable (hopefully soon).
I don't understand how sequences work. slog-config config file can have multiple outpu instances, each describing different logging destination.

Anyway, since I released slog-config 0.1, I'm closing this. It should be reasonably easy to add more stuff to it, and there was a PR already, so I think the task is done ATM.


Work with Google to publish documentation to cloud.google.com/docs

As we work through the rollout of InfluxDB Cloud v2, we will deploy on GCP Marketplace.  As part of our relationship with Google, we are required to publish our documentation into their site.
We will need to determine:

format for the content (output from Hugo?)
publishing process and frequency (limits?, controls?)

We will need to work with @gunnaraasen very closely on this effort.

@gunnaraasen checking back on/doing research to see if we need any updates at this point


typings: change any to unknown

There is some inconsistency on the EventHandlers interface, hoping we can use void instead since there is no reason to return an object on these events anyways.

Big fan of unknown. Not a fan of void. When they return void, the code in end users events become much worse. For example:
// Will error with void
messageCreate: (message) => {
  if (!message.guildId) return await bot.helpers.sendTextMessage("I only work in guilds.");
}

// void requires it to be written like this
messageCreate: (message) => {
  if (!message.guildId) {
    await bot.helpers.sendTextMessage("I only work in guilds.");
    return;
  }
}

I just find the code much uglier with void. Unknown was good choice!


Enable local authentication by default for auth generator

Closes https://github.com/feathersjs/feathers-docs/issues/612

:shipit:


Suggestion: If parentheses are transformed in pairs, move the input position inside the parentheses.

When outputting parentheses in pairs, most users type characters inside the parentheses.
I think it would be convenient if mozc moved the input position to the inside of the parentheses when the parentheses were output in pairs, like the auto-completion of half-width parentheses in VSCode.
Useful if the user wants to pre-output just the parenthesis pair to avoid forgetting to close it.
There are two ideas that can be combined.
Idea 1. The input position moves to the middle of the parentheses when the user converts the parentheses in pairs alone. "かっこ"→"「|」","（）"→"【|】"
Idea 2. If the user converts with "かっこあいう", it will be converted to be sandwiched like "「あいう」".
(In the case of Idea 2, the user can convert "（）あいう" to "「あいう」","（あいう）","【あいう】",etc.)

Hi mikunimaru,
Indeed, https://github.com/google/mozc/commit/e8cf4194d8b8c78d10ff61941d881c54a85da7af is the commit for that functionality. We need to update the client side too.
Thanks,


Sourcemaps for legacy JS scripts have absolute path, instead of relative

Versions
angular/cli: 1.4.9
node: 6.11.3
os: win32 x64
@angular/animations: 4.4.6
@angular/common: 4.4.6
@angular/compiler: 4.4.6
@angular/core: 4.4.6
@angular/forms: 4.4.6
@angular/http: 4.4.6
@angular/platform-browser: 4.4.6
@angular/platform-browser-dynamic: 4.4.6
@angular/router: 4.4.6
@angular/cli: 1.4.9
@angular/compiler-cli: 4.4.6
@angular/language-service: 4.4.6
typescript: 2.3.4

Repro steps

Add legacy js scripts to angular-cli.json scripts array
Build via "ng build --prod --sm

here is my angular-cli.json:
],
      "scripts": [
        "App.js",
        "modules/module1.js",
        "modules/module2.js"
        ....
]

Observed behavior
Sourcemaps generated for legacy js scripts (scripts.bundle.js) contain relative path, instead js.map looks like this:
{"version":3,"sources":["C:\\Users\\ibalan\\Documents\\Projects\\myproject\\src\\App.js"}

Desired behavior
Sourcemaps should have relative paths, is there is switch or option in a config for this?
When I build with --dev - sourcemap for scripts is not created at all

This issue should be resolved... please update to the latest version of the CLI (currently: 1.6.0).

Unfortunately, upgrading to 1.6 did not resolve the issue. I still get absolute paths.

Guys, any news on this?

I think this was fixed by webpack 4, as I cannot reproduce this locally with latest master.


Standard ouput

Implement console output than output in a file
Remove \n when single output types executed

One problem - output.txt now begins with a blank line. That doesn't bother me, but it is non-standard and SAS proc import refuses to read the file.
Originally posted by @feenberg in https://github.com/PolicyEngine/policyengine-taxsim/issues/141#issuecomment-2543934761

On Tue, 17 Dec 2024, Al Noman wrote:

Implement console output than output in a file

Need input from console also.
Dan

Remove \n when single output types executed

Yes.

Also, the line message "Output saved to outputtxt" should be suppressed if i/o is standard in/out.  I undersand it is sent to standard error, but I redirect standard error to standard out so that I get error messages back to the web browser as it makes testing much easier for the web application. Otherwise you have to look in the error_log which is less friendly.


Add validation to check for duplicate usernames

As title says we currently don't check if a user inputted a username to register that already exists. Similar to the pipeline we use to let the user they inputted the incorrect username or password on login, we should implement some user feedback for inputting a duplicate username

implemented


Add documentation about using Negroni simply as an http.Handler

This approach is preferred as it leads to much more flexibility with
negronis usage over just using negroni.Run() which is just a light
wrapper around http.ListenAndServe().
Supercedes #106
Supercedes #107

cc/ @meatballhat

YAAAAS


Adding CI publish workflow

This adds CI to publish the vsix extension to the vscode marketplace

I tested all of this in act locally

I made a pre-release targeting the commit hash of my branch:
https://marketplace.visualstudio.com/items?itemName=ChiaNetwork.chialisp-lsp-client
Note that it will only show pre-release if viewing from the actual VSCode App:

This does not show up in VSCodium by default - https://github.com/VSCodium/vscodium/blob/master/DOCS.md#how-to-use-the-vs-code-marketplace
so we might want to investigate publishing this to open-vsx as well. I did verify that you can download the .vsix file and install it in VSCodium.
I'll add uploading of the artifact to the release page in a separate PR.

clvm_tools_rs is now in a publishable state.
I added a build of it in here in its wasm personality, which is pulled into runner via npm link.
When we pop clvm_tools_rs into npm, we can remove the ad-hoc build and add clvm_tools_rs to its dependencies properly.
I am unsure whether we want to release here or wait until the language server branch is merged, so being conservative on that.
In any case, while heavy handed, vsce package is fixed in here, assuming everything runs to completion (excluding missing system dependencies etc that i might have missed).


[Windows] Support msix

The current version 0.2.2 already supports msix.
https://distributor.leanflutter.org/docs/makers/msix


Antigen 'apply' takes 5 seconds

Hey, I've been using antigen for the last year or so and it's been amazing.
However I just purchased Kaspersky Internet Security for my mac during black Friday and now running a simple antigen apply with no bundles or themes takes over 5 seconds. My normal .zshrc with plugins takes upwards of 20-25 seconds to load now.
I realize that this is obviously because of Kaspersky, but I wanted to know what's the best way to debug this. Does antigen have a verbose mode? How can I profile it to see what's taking so long?
What's weird is that a 'blank' antigen apply takes so long, if I'm not loading any plugins or themes then what is it doing?

Hey there @orf, unfortunately Antigen has no verbose mode. What I do to debug is either activating zprof or simply perform a binary search/git bisect depending on the issue. Take a look here.
Regarding antigen apply, what it does is compinit. Check if performing the following lines does cause an unusual delay:
autoload -U compinit
compinit -i -d $HOME/.compdump

After a bit of hacky profiling with adding echo 'something $(date +%d.%m.%y-%H:%M:%S)' all around antigen.zsh it seems the culprit is source $_ZCACHE_PAYLOAD_PATH. This takes over 12 seconds when kaspersky is installed.
-zcache-unhook-antigen also takes 4 or 5 seconds as well.
I'm going to open a support request with Kaspersky to see if they have anything to say about this, but is there any way to avoid those calls? I think I know the answer: no, because they seem to be an amalgamated form of all my plugins?

I don't know how Kaspersky works but it could hook source/eval/sed/$() functions/builtins.
You can disable Antigen's cache (and those calls) with:
export _ANTIGEN_CACHE_ENABLED=false

I'll close this for now as it's not related to Antigen. Do come back with news if you have any.
Cheers.


Added spaces before links

Added spaces before links - this is an issue in almost all of the VBA documentation. Looks like when it was converted, links lost the preceding space.
Same issue with italics under syntax - it is missing a space in almost all the documentation.
Is there anyway you could run a quick regex replace on everything?
/[^\s][/\ [/
Would work for links.

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

VBA/Language-Reference-VBA/articles/resume-statement.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

VBA/Language-Reference-VBA/articles/resume-statement.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

Falthazar is completely right that a missing space before a link is a common formatting problem that could be fixed with a global search-and-replace.
Here are other common formatting problems I've seen:

Add a space before links, if missing.
Add a space between an italic mark and a bold mark.
Add a blank line between single-spaced lines.
Change bold Syntax and Remarks to headings.

See further corrections to the Resume statement in https://github.com/OfficeDev/VBA-content/pull/652

All content in this repository has been migrated to a new repository (https://github.com/MicrosoftDocs/VBA-Docs). The changes in this PR have been applied in the new repository, so this PR is being closed. The changes will be published to the live docs soon. Thank you for your contribution!


itemReorder not working with itemSliding on iOS

Ionic version:  (check one with "x")
[x] 2.x
I'm submitting a ...  (check one with "x")
[x] bug report
Current behavior:
The combination of itemReorder and itemSliding doesn't seem to work on iOS. The HTML below won't properly reorder the items on the GUI on iOS. By "reorder the items on the GUI", I mean that the selected item can be moved, but no other items shift as you move the selected item:

Am I doing somthing wrong? If I do the same sort of thing with a simple ion-item (non-sliding), reordering works fine on iOS. If I use the sliding item in this manner on non-iOS, it works fine.
Expected behavior:
Other items shift as you move the selected item.
Steps to reproduce:

Create a list with reorder attribute set to true
The list should consists of ion-item elements which are wrapped with ion-item-sliding
Run the app on iOS and try to reorder items

Related code:
<ion-list reorder="true" (ionItemReorder)="reorderItems($event)">
    <ion-item-sliding *ngFor="let ski of skis">
      <ion-item>
        <ion-checkbox color="primary" item-left checked="false" (ionChange)="toggleCheckbox(ski.id)"></ion-checkbox>
        <ion-label>

{{ski.name}}

{{ski.description}}

        </ion-label>
      </ion-item>
      <ion-item-options>
          <button ion-button color="favorite" (click)="goToUpdatePage(ski)">Bearbeiten</button>
          <button ion-button color="danger" (click)="showRemoveAlert(ski)">Löschen</button>
      </ion-item-options>
    </ion-item-sliding>
</ion-list>

Other information:
There exists already a forum entry for this issue: https://forum.ionicframework.com/t/itemreorder-not-working-with-itemsliding-on-ios/64385
Cordova CLI: 6.4.0
Ionic Framework Version: 2.0.0-rc.3
Ionic CLI Version: 2.1.7
Ionic App Lib Version: 2.1.4
Ionic App Scripts Version: 0.0.45
ios-deploy version: 1.9.0
ios-sim version: 5.0.8
OS: macOS Sierra
Node Version: v6.6.0
Xcode version: Xcode 8.1 Build version 8T61a

@manucorporat Just saw that RC4 was just released. Unfortunately I can still see this error. Are you intending to fix this in a future release?

Problem still exists in current stable ionic 2.0. Will this be fixed in a future release? In Browser and on Android, the combination is working, but not on iOS (device and simulator).

Any update on this issue? Is there a recommended workaround?

i am also got that error . if any update ,just remind me

Same here!

This works for me. Does it work for you when you test it on your desktop browser?

any fix?


Reflect gray/green difference in light theme

-
old
new

dark

light

@mkondratek pls remove the setup script changes from this PR, I'm fix that on a separate PR


"Some header menu buttons on the homepage are unresponsive when accessed via a mobile phone."

Found a minor bug in NavbarMobile.tsx. Fixed.


GUI - Add header to price details drawer

We would like to display some details for the selected charge price.

Tasks

[ ] Use watt-drawer-actions to add some buttons (should be disabled for now)
[ ] Include time metadata next to buttons (if possible using standard watt components)
[ ] Display header with type, name, owner and id? next to actions
[ ] Create expandable description area (what should go here?)
[ ] Add "table" view for the remaining metadata

Acceptance Criteria

[ ] As a GUI user, I would like to be able to see detailed information in the drawer
[ ] As a GUI user, I would like to be able to expand a panel for a detailed description

Greyed out text is out of scope i.e. "pris opdateret, stamdata opdateret"

We have agreed with @IreneDataHub that it is more valuable to show the validity dates (from and to) instead of the "Pris" and "Gyldighed" showed in the screenshot from Figma.

Test 14. October 2022
Environment T-001
Danish language = OK

English translate missing for tax, transparent invoiving


Add support for per-field visibility for FeatureCollectionDataAdapter

Currently, FeatureCollectionDataAdapter only supports per-feature visibility.  Combining multiple visibility configuration within a single table namespace will cause a whole host of issues.  We need to implement true support for per-field visibility.

FeatureCollectionDataAdapter removed in 0.9.0


Tracking issue for filter expressions

Documentation: https://nexte.st/book/filter-expressions.html
TODO:

[x] #159
[x] #160
[x] #29

cc @Guiguiprim

I have such a test:
crypto-markets::bitmex aaaaa
crypto-markets::bitmex bbbbb
crypto-markets::bitmex ccccc
crypto-markets::others bbbbb
crypto-markets::dont_exclude aaaaa

How can I exclude the bitmex tests while still including others and dont_exclude? I tried package(/bitmex/), and it said:
$ cargo nextest run -E 'package(/bitmex/)'

  error: operator didn't match any packages
   ╭────
 1 │ package(/bitmex/)
   ·         ────┬───
   ·             ╰── no packages matched this
   ╰────

error: failed to parse filter expression

While test(/bitmex/) can only match:
crypto-markets::bitmex bitmex_aaaaa
crypto-markets::bitmex bitmex_bbbbb
crypto-markets::bitmex bitmex_ccccc
crypto-markets::others bitmex_bbbbb
crypto-markets::dont_exclude bitmex_aaaaa

Thanks for pointing this out! The next version of nextest (hopefully done in a couple of days) will add a new binary predicate for this purpose.

Filter expressions have been released and are now part of cargo-nextest 0.9.25.


More key commands?

Hi there :)
I was wondering about the ability to have the same shred handling key commands in miniAudicle, in this extension.
Reckon it should be doable, right?

I haven't used miniAudicle, so I'm not familiar with this functionality. Could you provide more information?

Closing issue due to no response.


Fix update_by_query request

as titled.

Confirmed after looking at the ES code: there's no support for arrays there.

Ack, I'll dig how to resolve this.

Opened https://github.com/elastic/elasticsearch/pull/77700 to fix the test upstream.


Feature Request : Ability to force h.264 insted of vp9 on HTML5 videos.

Currently VP9 videos stutter and/or freeze for me on all my machines (using Waterfox). So it would be really nice to have functionality of https://github.com/erkserkserks/h264ify built into this addon, should probably be helpful for bunch of other people too.

Afaik waterfox is offered mp4 by default if it is following the current Firefox progress. You will only be offered vp9 (webm format) if you have the last field marked in blue here: https://www.youtube.com/html5
If you do have it in blue then try disabling webm via the about:config page (hopefully it is the same as with Firefox) and look for media.mediasource.webm.enabled, change the value to false.

is google really serving VP9?
I tried youtube-dl on couple of videos and almost everywhere it looks like this:
140          m4a        audio only DASH audio  130k , m4a_dash
141          m4a        audio only DASH audio  256k , m4a_dash
160          mp4        256x144    DASH video  114k , 15fps, vi
133          mp4        426x240    DASH video  276k , 30fps, vi
134          mp4        640x360    DASH video  619k , 30fps, vi
135          mp4        854x480    DASH video 1114k , 30fps, vi
136          mp4        1280x720   DASH video 2224k , 30fps, vi
137          mp4        1920x1080  DASH video 4623k , 30fps, vi
17           3gp        176x144
36           3gp        320x240
5            flv        400x240
43           webm       640x360
18           mp4        640x360
22           mp4        1280x720   (best)
only mp4 for hires versions. Is there some secret handshake only Chrome is able to do to force VP9?

VP9-encoded videos are a fairly new feature on YouTube. It's very possible that only a small handful of new uploads take advantage of it. Examples of format codes that use VP9 video in a WebM container are 315, 272, 313, 308, 271, 303, 302. AFAIK, all of those are DASH streams.

Thank you. I manually downloaded vp9 and h264 versions and compared couple of frames. Sure file size is 33% better, but quality suffers :/, and still no combined video+audio 1080 vp9 link. Back to streaming 720p h264 for me :/

wait a minute - how does HTML5 player fetch video stream? does it use separate video and audio streams and muxes them on the client side? or is there another list of streams not exposed in dash?

@raszpl It uses individual audio and video streams which play at the same time via the MSE specification. You can say that they "mux" it on the client side, but it's more like they synch together inside a common interface which can contain multiple sources.

Based on what I've seen, the new WebM/VP9 streams are handled with DASH.

@Yonezpt    media.mediasource.webm.enabled    worked like a charm, though I would probably still prefer to have a check box in addon setting, just in case.
Google does not seem to serve VP9 on all videos, might be related to who had 60fps beta and who got it on rollout. As an example DF was in the beta : http://www.youtube.com/watch?v=adUqCT08Yq8 This has VP9. As for LC I have no clue about the beta status, but there are no VP9 for me https://www.youtube.com/watch?v=Far891oj5u0
Also I'd say quality of VP9 aint up to snuff (at least for non filmed content), at least at the bitrate Youtube uses for 1080p. Also while VP9 might be hardware accelerated on a lot of things in theory, it's not on my work Nvidia GTX 750 machine and home Haswell notebook (looking at CPU usage compared to h.264).
Also I've only checked 60fps videos, since they are the main interest of moving fully to HTML5 (part from flash freezing from time to time).
Also AFAIK all 1080p is DASH only nowdays.

any news on this? I'd be curious to see the answer myself

@allbetsareoff29 It hasn't even been a full day since this request was made. Please be more patient.

                                                                                My apologies. I didn't check the time stamp of your post. Please let me know if I can help in any way. I'll gladly test any solutions that you guys come up with                                                                                Sent from my smartphone. Please excuse any typos                                                                                From: ErdeFBSent: Wednesday, April 8, 2015 2:20 AMTo: YePpHa/YouTubeCenterReply To: YePpHa/YouTubeCenterCc: allbetsareoff29Subject: Re: [YouTubeCenter] Feature Request : Ability to force h.264 insted of vp9 on HTML5 videos. (#1627)@allbetsareoff29 It hasn't even been a full day since this request was made. Please be more patient.

—Reply to this email directly or view it on GitHub.

Just to update, have seen some similar questions on reddit, so I would imagine percentage of people that know of media.mediasource.webm.enabled is relatively small. So I'd still say having this as an option in the addon is great. And the help link that you access with ? can describe that you can do the after-mentioned tweak, so awareness would be tiny bit better with time.


Support data-=@expression properly and remove the data_ workaround

This requires changes in Razor.
Assigning to @rynowak, but it's not urgent and I know it won't be done for 0.1.0.

When this is done, we should be able to revert the implementation code in c16263fb13cc3781b08aad957a3f64bd23851be0 (but not all its tests, as some of those are still applicable).

@SteveSandersonMS @danroth27 - this support lands in 15.7 preview 4 which has been released. Do we want to require 15.7 preview 4 for 0.3.0 and then we can address this issue? Seems worth it to me 😀

If preview 4 means we can remove the workaround, I'd be in favour of doing that.
It doesn't strictly mean we'd have to require p4 though, right? It just means that if you don't have p4 then you will encounter a bug with data-* (like you already do today), but everything else still works OK.

We already tell people to install the latest VS2017 15.7 preview. You can't really go back and install an older preview. In general folks on an older preview should upgrade.

It doesn't strictly mean we'd have to require p4 though, right? I

preview5 is out now 😆 - I think people will want p4 anyway because it has the fix for that nasty format-on-paste issue.

Some bad news on this. This fix was not present in preview2 of Razor, so we can't use it yet. This would work in VS, but fail at the command line.
See history here: https://github.com/aspnet/Razor/commits/release/2.1
We need 4cd24a2b087ea2ec0ff7cc687cb2ed88080c8e7f
But 2.1.0-preview2-final is b1007744b00ce094af080602ee26d3887a786d56


sessionid fails

Hi,
request.COOKIES.get('sessionid', 'nosessionid')
fails if SESSION_COOKIE_NAME is set to any other value than 'sessionid'. In our project, for example,
SESSION_COOKIE_NAME = "_sid"
so user session won't pass to PDF version and it fails.

whops! Thanks for reporting this, seems this must be a configurable setting.
Thanks!
2015-03-10 7:07 GMT-03:00 softshape<EMAIL_ADDRESS>
Hi,
request.COOKIES.get('sessionid', 'nosessionid')

fails if SESSION_COOKIE_NAME is set to any other value than
'sessionid'. In our project, for example,

SESSION_COOKIE_NAME = "_sid"
so user session won't pass to PDF version and it fails.
—
Reply to this email directly or view it on GitHub
https://github.com/BananaDesk/django-phantom-pdf/issues/9.

--
Broken code @ https://github.com/edvm
<EMAIL_ADDRESS>

Just change 'sessionid' to settings.SESSION_COOKIE_NAME, i.e. -
request.COOKIES.get(settings.SESSION_COOKIE_NAME, 'nosessionid')

Hey @softshape - can you send a PR?
On Tue, Mar 10, 2015 at 11:01 AM softshape<EMAIL_ADDRESS>wrote:

Just change 'sessionid' to settings.SESSION_COOKIE_NAME, i.e. -
request.COOKIES.get(settings.SESSION_COOKIE_NAME, 'nosessionid')
—
Reply to this email directly or view it on GitHub
https://github.com/BananaDesk/django-phantom-pdf/issues/9#issuecomment-78058367
.

No idea how to do this. Just waiting @x-ip to fix it :)
BTW @x-ip please write us when an update is available via pip. I don't want to edit dist-packages manually, really.

@tzenderman btw, I made a PR.

Thanks, I'll check this out today.
On Tue, May 5, 2015 at 7:48 AM Tobias Birmili<EMAIL_ADDRESS>wrote:

@tzenderman https://github.com/tzenderman btw, I made a PR.
—
Reply to this email directly or view it on GitHub
https://github.com/BananaDesk/django-phantom-pdf/issues/9#issuecomment-99031091
.

@softshape: This should be fixed with @toabi 's PR which was just merged into the master branch. Still need to do a Pypi release though...


Web - Login - User gets 'Magic Code' instead of 'Abracadabra, you are signed in' message

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:
First scenario:

Open staging.new.expensify.com and continue with the log in flow with any account
Navigate to the email inbox and look for the magic link email
Change the link to staging if needed
Open a new tab and paste the link in the URL
Verify an "Abracadabra, you are signed in!" page is displayed

Second scenario

Log out of NewDot and request a new magic link
Navigate to email and locate the magic link - change to staging if needed
Open an new tab and navigate to the staging link
Verify the 2FA required page is displayed. Should have a message similar to "Please enter the two-factor authentication code where you are trying to sign in."
Navigate back to the original tab - Keep the second tab open
Verify the log in flow is now asking the user to enter the 2FA code
Enter the 2FA code
Verify the user can log in
Navigate to the secondary tab
Verify the "Abracadabra, you're logged in" page is displayed

Expected Result:
"Abracadabra, you are signed in!" page is displayed.
Actual Result:
User gets 'Magic Code' instead of 'Abracadabra, you are signed in' message
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android / native
[ ] Android / Chrome
[ ] iOS / native
[ ] iOS / Safari
[x] MacOS / Chrome / Safari
[ ] MacOS / Desktop

Version Number: <IP_ADDRESS>
Reproducible in staging?: Yes
Reproducible in production?: No
If this was caught during regression testing, add the test name, ID and link from TestRail:
First scenario https://expensify.testrail.io/index.php?/tests/view/3456321
Second scenario https://expensify.testrail.io/index.php?/tests/view/3456319
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://github.com/Expensify/App/assets/93399543/ebd7f58b-6efc-455a-bc15-fdf1275d049e
https://github.com/Expensify/App/assets/93399543/dc89506c-97a7-4100-93d3-8c6f180ce49a
Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal Team
Slack conversation:
View all open jobs on GitHub

@isagoico

:wave: Friendly reminder that deploy blockers are time-sensitive ⏱ issues! Check out the open StagingDeployCash deploy checklist to see the list of PRs included in this release, then work quickly to do one of the following:

Identify the pull request that introduced this issue and revert it.
Find someone who can quickly fix the issue.
Fix the issue yourself.

There's a weird behavior on Staging surrounding the Magic link. A few hours ago this was working without issues, video below:
https://github.com/Expensify/App/assets/44479856/7faa45b2-7cd3-4ce5-bee7-914ee15cc2de
Now, the "abracadabra" page is not being displayed consistently on staging nor production. Will investigate further on the issue🫠 @roryabraham I think that we should be good with removing the deploy blocker label on this one.

Not a blocker for the NewDot deploy since this is also happening on Prod.

Making this a Daily again

PR ready for review: https://github.com/Expensify/App/pull/19468

Hey @roryabraham would you be able to check on this GH to see if this will be fixed with this implementation?
https://github.com/Expensify/App/issues/19860

Oh, so this is done / should've been closed already. I'm going to close this but then will check on that other one.


Do we want be stricter on flake8 (PEP8) violations on new pull requests?

OPTIONAL: .travis.yml currently runs flake8 in two passes.  The first pass looks for a handful of crucial errors and fails the build on these.  The second treats all remaining issues only as warnings.  The output looks like this:

https://travis-ci.org/Fiware/dataModels/jobs/282605654#L1559

If there are any issues such as unused imports, long lines, etc. that we want future PRs to fix then we could move those issues from the second pass to the first pass and fail the build on those violations.
Some projects like CPython and Django treat all issues as unacceptable but others projects do not care about style issues.

let's keep as it is for the moment with regards to flake8.
I would force that Travis checks that PEP8 rules are met.

These two statements seem to contradict each other.  The first statement says we should not be strict on PEP8 violations and the second says that we should be more strict on PEP8 violations (the flake8 Exxx issues).

oh, then I'm totally in favour of forcing compliance with PEP8 rules
I'm afraid we will be failing, would you be able to provide a pull request which solves the issue over all .py files? I guess it is a matter of running autopep8, isn't it?

code landed
do we need to modify the travis.yml recipe as well?


Definitions of Evaluation Results

Hello, I've evaluated both Second and MVXNet on some adverse weather conditions. However, I'm struggling to understand the format of the results.  My problems are as follow:

I understand that<EMAIL_ADDRESS>means IOU thresholds of 70%, 50%, and 25%, respectively; meaning overlaps have to meet these thresholds to count as true positives then why is it that the 70% is labeled as easy not hard when it should be harder to achieve IOU of 70% then 50% and 50% than 25%?

The results are divided into 4 categories: BBox, Bev, 3d, and aos.  Some of these are familiar while others are not. BBox, I believe means the accuracy of estimating 2D boxes from the 3D boxes (please do correct me if I'm wrong). Bev means bird's eye view, though I'm not sure what it means in the context of object detection. 3d I believe means the accuracy of estimating 3D orientation but isn't that measured by aos?

Finally, there are 2 sets of results for each class, "strict" and "loose". These terms are unfamiliar to me.

A detailed response would be greatly appreciated.

Please refer to #185 and the definition given by the official website.

Thank you @Tai-Wang for your response. However, it's still not explained what "strict" and "loose" metrics means. Please explain.

“strict” refers to more strict criteria for positive predictions. For example, IOU>0.7 is strict for car while 0.5 is the loose one. For cyclist and pedestrian, IOU>0.5 is strict and 0.25 is loose. You can observe these evaluation messages more clearly in the print results when training/evaluating a specific model.


nixos/stage-1: check secret paths before copying

Motivation for this change
Fixes #84976.
This is a workaround for #85000.
Things done

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

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[x] 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
[x] Fits CONTRIBUTING.md.

Technically the assertion message is a little inaccurate; "${someDerivation}/..." is valid too. But I couldn't think of a better way to word it and that's pretty marginal.

cc @sjau, can you check if the error message is better with this patch?

@GrahamcOfBorg test initrd-network-ssh

Cherry-picked onto release-20.03 branch: https://github.com/NixOS/nixpkgs/commit/f2d4179af38cd74ac1f985fd34b34435d1dcb662


Hive Testing: eth_feeHistory

[ ] fee-history

This is blocked since it's failing on multiple clients, see Geth for example: https://hivetests2.ethdevops.io/suite.html?suiteid=1726047969-3fb9e072eed7a0f13a9b8d9f3ca1b8d9.json&suitename=rpc-compat#test-27
It has been broken since this change: https://github.com/ethereum/execution-apis/commit/4b225e0d273e92982b2c539d63eaaa756c5285a4


JgiMg and JgiMt: Add rule that requires a dna_cont_well when the dna_cont_type is "plate"

But not vice versa

try a simpler rule first?

when the dna_cont_type= plate, then the dna_volume must be less than 1 (mL?)
when the dna_cont_type = plate, then the dna_volume must be greater than 1

[{"preconditions":{"slot_conditions":{"dna_cont_type":{"equals_string":"plate"}}},"postconditions":{"slot_conditions":{"dna_volume":{"maximum_value":1}}}},{"preconditions":{"slot_conditions":{"dna_cont_type":{"equals_string":"tube"}}},"postconditions":{"slot_conditions":{"dna_volume":{"minimum_value":1}}}}]

@pkalita-lbl will be working on the emission of rules by gen-jsonschema

Duplicate https://github.com/microbiomedata/nmdc-schema/issues/1433
Closing in favor of the issue that includes DNA/RNA plate and other slot dependancies.


Improve Workspaces docs page

What does this PR try to resolve?
from https://github.com/rust-lang/cargo/pull/13725，make commits more reasonable.
Additional information
r? @weihanglo

I am going to close this in favor of https://github.com/rust-lang/cargo/pull/13794. That one has a more complete view of the difference between root package and virtual workspace, with less redundancy.
Thank you for discussing with us and finding out the missing part of the doc :)


Improved Typescript Definitions

HI @Myzel394 ,

The new definitions is giving this error

E:/npm/react-ts-app/src/App.tsx
TypeScript error in E:/npm/react-ts-app/src/App.tsx(23,12):
JSX element type 'Alert' does not have any construct or call signatures.  TS2604

    21 |           <Switch />
    22 |
  > 23 |           <Alert isVisible={true}/>
       |            ^

Can you please check in your system why its happening.
2. we added space in the definitions for better code readibility.
Eg:
title ?: string;

Since from the following Stackoverflow Answer
1 Answer
You should use React.FunctionComponent: Rename React's SFC to FunctionalComponent

This PR renames React.SFC and React.StatelessComponent to React.FunctionComponent, while introducing deprecated aliases for the old names.

So your example would become:
const example: React.FunctionComponent<IExample> = ({propsType}) => ();
or

const example: React.FC<IExample> = ({propsType}) => ();

From the above following stackoverflow answer for arrow functions we now using React.FC or React.FunctionComponent


Fix typo

This checklist is used to make sure that common guidelines for a pull request are followed.
General Guidelines

[ ] The PR has modified HISTORY.rst describing any customer-facing, functional changes. Note that this does not include changes only to help content. (see Modifying change log).

Command Guidelines

[ ] Each command and parameter has a meaningful description.
[ ] Each new command has a test.

(see Authoring Command Modules)

View a preview at https://prompt.ws/r/Azure/azure-cli/5723
This is an experimental preview for @microsoft users.

Because this is the first change to the profile module post-release, it needs a version bump.

This is why we need #5955...

Closing this as it needs to be rebased in favor of https://github.com/Azure/azure-cli/pull/6151


chore(changelog): add v2.5.0 changelog

List of the PRs added in changelog:

#14
#22

Codecov Report

Merging #26 (c0f9ee5) into develop (b732c7d) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff            @@
##           develop      #26   +/-   ##
========================================
  Coverage    34.60%   34.60%
========================================
  Files           27       27
  Lines         2124     2124
========================================
  Hits           735      735
  Misses        1352     1352
  Partials        37       37

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 b732c7d...c0f9ee5. Read the comment docs.

Codecov Report

Merging #26 (c0f9ee5) into develop (b732c7d) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff            @@
##           develop      #26   +/-   ##
========================================
  Coverage    34.60%   34.60%
========================================
  Files           27       27
  Lines         2124     2124
========================================
  Hits           735      735
  Misses        1352     1352
  Partials        37       37

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 b732c7d...c0f9ee5. Read the comment docs.


[WIP] This PR is used to fix spelling errors in doc repo

fixed:
content/en/docs/concepts/containers/
content/en/docs/concepts/services-networking/

A bit confused: cncf's cla service is not working properly? I have signed the cla about two years before:

/signed cla

/check-cla

/approve

@kbarnard10 , please help to check the modification of "content/en/blog" part, thanks!

@steveperry-53 conflict resolved, please check it, thanks.

/lgtm


how to use navmesh data that export from unity

my game is develop with unity engine, and the server side use recastnavigation for AI navigation.
unity provide an API UnityEngine.AI.NavMesh.CalculateTriangulation to export navmesh data, so I wonder is there any way to create a dtNavMesh object by these triangle soup?

you can export unity scene to obj file for recastnavigation to build navmesh.for example,you can use ExportSceneToObj to export obj file.

thanks for reply,I tried to export my scene to obj file and build navmesh in recastnavigation demo,but the results are somewhat different from unity export data(use same build param).
I think there's different build algorithm between unity and recastnavigation.So after exporting navmesh unity, I export the data via the UnityEngine.AI.NavMesh.CalculateTriangulation API,then convert to a dtNavMesh structure that can be used directly by recastnavigation.
I have implemented a rough version(Navigator), and I will improve it when I have time.

I want to export navigation and offMeshLInk，can you help me? thx.


DO NOT MERGE: Start vendoring portability dependencies to not have dependency conflicts

Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

It will help us expedite review of your Pull Request if you tag someone (e.g. @username) to look at it.

Run Java PreCommit

Run Java PreCommit

Run Go PreCommit

Run Go PreCommit

Run Go PreCommit

R: @swegner

@kennknowles It isn't a separate package yet, its just being shaded into the model jars since one of the issues that I didn't figure out was how to get the generated code to use vendored classes.
As a side note, I was hoping that you would use this as an example in the PR that you created that vendored guava and I would be able to re-use what you created.

This change is blocked on deletion of the POMs as this will break the Maven build.

Above comments are mostly about documentation. No other major concerns from me (LGTM)

We have turned on autoformatting of the codebase, which causes small conflicts across the board. You can probably safely rebase and just keep your changes. Like this:
$ git rebase
... see some conflicts
$ git diff
... confirmed that the conflicts are just autoformatting
... so we can just keep our changes are do our own autoformat
$ git checkout --theirs --
$ git add -u
$ git rebase --continue
$ ./gradlew spotlessJavaApply

Please ping me if you run into any difficulty.


Gallery .jpg files don't populate on createdb, link to thumbnail wrong

Django 1.10, Mezzanine 4.2.0
Running an initial createdb creates the Gallery demo page, but the actual .jpg files aren't getting out of the zip files and the reference to the image thumbnail HTML doesn't reference the correct directory.
python3 manage.py createdb --noinput

The thumbnails appear to be there e.g. ./static/media/uploads/gallery/.thumbnails/Warsaw, Poland.jpg exists, but the thumbnail doesn't show in the gallery, and the link itself 404's.
Attempting to edit the post to in the back-end also produces an error and the link can't be modified.
It looks like there is supposed to be a set of full-size files in the 'gallery' and the thumbnails are to be used in the gallery page backend.
Here's a snippet of the HTML:

BaseGallery's save() method is supposed to extract the files from the zip file, but they're not there.
The .thumbnails directory is there, but the HTML incorrectly specifies the path to the thumbnail file in the:
<a class="thumbnail"...

hunk of HTML (it references the full-sized file instead).  Even if the full sized file were there, shoehorning it into the preview image size is the wrong way to do it so that seems like a different bug.

You don't have Pillow correctly installed.


Package Import is not working from neither way "CommonJS" or "Module"

Does this issue occur when all extensions are disabled?: Yes/No

VS Code Version: 1.88
OS Version: MacOS 14.4.1 | Macbook Pro (M3 Pro)

Problem
Just installed node on my system, but when i start importing packages in nodejs in vscode. VSCode intellisense didn't show require keyword and after typing complete line manually
const firebase = requrire("firebase-admin");

expected behaviour should be firebase.collection in intellisense dialog but in actual it shows import firebase in that.
Due to which i have reinstalled node and nvm so many times that issues may be in nodejs environment and also reconfigured paths .zshrc file but didn't work.
After doing all things, i thought i should downgrade vs code version 1.86.2 (universal). After downgrading, this issue resolved

We need a concrete way to repro this (example repo would be best). Thanks!

Upgrading to new version of VS Code, issue persist again and attaching screenshots

Require Keyword is not coming in code completion suggestion

import firebase coming in suggestion, after writing complete import statement

No further suggestions coming

Using this latest version

If i used this version as below, all suggestions are coming as expected

Suggestions

We need a concrete way to repro this (example repo would be best). Thanks!


Add goss boss command

Haha

LOL, thanks for the laugh!
@ryansydnor when are you gonna come through with the Goss swag :)


Send mesage to external upstream without httpclient

Hi Igor,
Hope you're doing well.
i use SQ now it's really amazing. Thanks
How  can i send a message to an external upstream server without a client connect.
For example i use SQ to connect to a upstream server.  In specs of this server.. at every connection the upstream server must send to me (here SQ) a 0800 message and i should response automaticaly after make some verification by 0810
Actually when i receive the request 0800 from upstream , SQ log error as ISO host send data with no client.
The concerned upstream server connection specifications describe:

Socket must send 0800 to upstream server
upstream server response 0810 and automatically send 0800 with an additionnal data (DE48)
SQ must verify the value of the additionnal data (DE48) and send a 0810 as response with responseCode 0 and must send automatically new 0800
After this exchange done correctly , the upstream server it ready to receive and authorize transactions request from SQ client
Can you Assist please

You can specify this under queueMessageKeyFields (default: '11,41'), so you can remove 41 while starting up SQ as below
Example -
node socketQueue.js --upstreamHost=X.X.X.X --upstreamPort=54321 --listenPort=2100 --listenHttpPort=2110 --vv --logFile=log.txt --queueMessageKeyFields=false
Or, Alternatively you could start SQ specifying queueLockTerminal=false so that SQ queues doesn't lock to a terminal.
Example -
node socketQueue.js --upstreamHost=X.X.X.X --upstreamPort=54321 --listenPort=2100 --listenHttpPort=2110 --vv --logFile=log.txt --queueLockTerminal=false

Hi shoaibjdev ,
Thanks for your revert.
About the TID i remove from parameter.js and i just let DE11 (Stan) , waiting to create a logic to ignore this DE for network messages.
My real request it's looking for kind of logic that's automatically send a 0800 request when SQ connected to the upstream and some others message for authentication with the upstream server befor Http client or any SQ client make payment auth request

Hi!
There is an "Upstream Echo" event: https://github.com/juks/SocketQueue/blob/1b172de810bb928516b511b4fe81b0ae7e0751ac/lib/iso8583/lib/logic/smartVista.js#L103
It is called every time upstream send a 0800 packet to SQ: https://github.com/juks/SocketQueue/blob/1b172de810bb928516b511b4fe81b0ae7e0751ac/lib/socketServer/lib/upstream.js#L253
The thing is you can always tell if the 0800 message is addressed to SQ since it is not a response but a request. How would you like to know if 0810 message does not belong to any client?
Do you mean SQ should own it's own terminal number?
There is a method that is called right after SQ connects to an upstream: https://github.com/juks/SocketQueue/blob/1b172de810bb928516b511b4fe81b0ae7e0751ac/lib/socketServer/lib/upstream.js#L123.
You can add your own code that sends a message to the upstream. But still the question will be: how to process the upstream response in a proper way

To send a 0800 packet to an upstream every 5 minutes you need to set up a javascript timer.

Get it.
below code:
var echoTestTimout = 300000;
 if (typeof parent.logic.replyUpstreamEcho === 'function') {
           var rp = parent.logic.replyUpstreamEcho(p);
            setTimeout(function() {
                 dd('Sent echo test request to Upstream ..../5min' , 'verbose');
            this.write(rp.getMessage({
              staticHeader: global.c['useStaticHeader'],
              lengthHeader: global.c['useLengthHeader']
            }));
            }.bind(this)
              , echoTestTimout);```

Thanks for your help. Really appreciate


Update QuestariaAereospace.netkan

Adds a dependency on patch manager, which was recently added to Questaria Aerospace

The heck did I mess up here?

It's something with the swinfo.json file, haven't taken a look at it yet...
1780 [1] FATAL CKAN.NetKAN.Program (null) - Object reference not set to an instance of an object.
1847 [1] FATAL CKAN.NetKAN.Program (null) -    at CKAN.NetKAN.Transformers.SpaceWarpInfoTransformer.<Transform>d__3.MoveNext()
   at System.Linq.Enumerable.<SelectManyIterator>d__17`2.MoveNext()
   at System.Linq.Buffer`1..ctor(IEnumerable`1 source)
   at System.Linq.Enumerable.ToArray[TSource](IEnumerable`1 source)
   at CKAN.NetKAN.Transformers.NetkanTransformer.Transform(Metadata metadata, TransformOptions opts)
   at CKAN.NetKAN.Processors.Inflator.<>c__DisplayClass1_0.<Inflate>b__0(Metadata netkan)
   at System.Linq.Enumerable.<SelectManyIterator>d__17`2.MoveNext()
   at System.Linq.Lookup`2.Create[TSource](IEnumerable`1 source, Func`2 keySelector, Func`2 elementSelector, IEqualityComparer`1 comparer)
   at System.Linq.GroupedEnumerable`3.GetEnumerator()
   at System.Linq.Enumerable.WhereSelectEnumerableIterator`2.MoveNext()
   at System.Collections.Generic.List`1..ctor(IEnumerable`1 collection)
   at System.Linq.Enumerable.ToList[TSource](IEnumerable`1 source)
   at CKAN.NetKAN.Processors.Inflator.Inflate(String filename, Metadata[] netkans, TransformOptions opts)
   at CKAN.NetKAN.Program.Main(String[] args)

Whatever that error is, it's already live in production without these changes.

For reference here is the swinfo.json from the latest download
{
  "spec": "2.0",
  "mod_id": "QuestariaAerospace",
  "name": "Questaria Aerospace",
  "author": "IsaQuest",
  "description": "Parts for Exploration",
  "source": "",
  "version": "1.1.1",
  "dependencies": [
    {
      "id": "com.github.x606.spacewarp",
      "version": {
        "min": "1.7.0",
        "max": "*"
      }
    },
		{
      "id":  "PatchManager",
      "version": {
        "min": "0.7.2",
        "max": "*"
      }
    },
    {
      "id": "lfo",
      "version": {
        "min": "1.0.0",
        "max": "*"
      }
    }
  ],
  "conflicts": []
}

Wait, hmm, I edited it out cuz it looked like a formating error, but the indentation before the { before "id": "PatchManager" is 2 tabs not spaces

Plus, I see that KSP2 version info is missing, whether that has anything to do with the error or not

Plus, I see that KSP2 version info is missing, whether that has anything to do with the error or not

Yes, I think SpaceWarpInfoTransformer is throwing here because ksp2_version is null:
                    if (GameVersion.TryParse(swinfo.ksp2_version.min, out GameVersion minVer)
                        || GameVersion.TryParse(swinfo.ksp2_version.max, out maxVer))

So, I know why that likely wasn't set
It's a bug that I fixed today with KSP2 Unity Tools

Is ksp2_version intended to be required in the swinfo spec? I don't think that was something I considered explicitly when writing the above code. If it's supposed to be required, we can add an explicit check and clear error message for it. Otherwise we can make that line more forgiving of nulls.

[JsonProperty("ksp2_version", Required = Required.DisallowNull)]
public SupportedVersionsInfo SupportedKsp2Versions { get; internal set; } = new()
{
    Min = "*",
    Max = "*"
};

It's defined as being allowed to be missing, or to have a non-null value, with the default being unbounded compatibility

Alright issue has been fixed mod-side, will merge now


Build ERROR - Undefined Driver_I2C0

I have been able to run "layer list" and "convert" commands for the example projects.
I am focused on the frdmk22f_a8974 builds.  I have that board plugged in.
I get following error when I build any of the cprj files.
Any suggestions on what step I am missing?
I look up and Driver_I2C0 is defined in frdm_k22f.h as "extern ARM_DRIVER_I2C Driver_I2C0;"
I run the following:   cbuild fxls8974cf_interrupt.debug+frdmk22f_a8974.cprj
Succesfully links.  Then fails with following:
FAILED: C:/GIT/NXP_Sensor_SDK/examples/issdk/sensors/fxls8974cf/interrupt/out/fxls8974cf_interrupt/frdmk22f_a8974/debug/fxls8974cf_interrupt.debug+frdmk22f_a8974.axf
cmd.exe /C "cd . && C:\Keil_v5\ARM\ARMCLANG\bin\armlink.exe --cpu=Cortex-M4 --scatter="C:/GIT/NXP_Sensor_SDK/boards/frdmk22f/RTE/Device/MK22FN512VLH12/MK22FN512xxx12_flash.scf"  --entry=Reset_Handler --strict --info summarysizes --remove --keep=(.crp) --keep=(.FlashConfig) --diag_suppress 6314,6319    --via=CMakeFiles\fxls8974cf_interrupt.debug+frdmk22f_a8974.rsp -o C:\GIT\NXP_Sensor_SDK\examples\issdk\sensors\fxls8974cf\interrupt\out\fxls8974cf_interrupt\frdmk22f_a8974\debug\fxls8974cf_interrupt.debug+frdmk22f_a8974.axf  --list=C:\GIT\NXP_Sensor_SDK\examples\issdk\sensors\fxls8974cf\interrupt\out\fxls8974cf_interrupt\frdmk22f_a8974\debug\fxls8974cf_interrupt.debug+frdmk22f_a8974.axf.map && cmd.exe /C "cd /D C:\GIT\NXP_Sensor_SDK\examples\issdk\sensors\fxls8974cf\interrupt\tmp\fxls8974cf_interrupt\frdmk22f_a8974\debug && c:\Keil_v5\ARM\ARMCLANG\bin\fromelf.exe --i32combined --output C:/GIT/NXP_Sensor_SDK/examples/issdk/sensors/fxls8974cf/interrupt/out/fxls8974cf_interrupt/frdmk22f_a8974/debug/fxls8974cf_interrupt.debug+frdmk22f_a8974.hex C:/GIT/NXP_Sensor_SDK/examples/issdk/sensors/fxls8974cf/interrupt/out/fxls8974cf_interrupt/frdmk22f_a8974/debug/fxls8974cf_interrupt.debug+frdmk22f_a8974.axf && cd /D C:\GIT\NXP_Sensor_SDK\examples\issdk\sensors\fxls8974cf\interrupt\tmp\fxls8974cf_interrupt\frdmk22f_a8974\debug && c:\Keil_v5\ARM\ARMCLANG\bin\fromelf.exe --bin --output C:/GIT/NXP_Sensor_SDK/examples/issdk/sensors/fxls8974cf/interrupt/out/fxls8974cf_interrupt/frdmk22f_a8974/debug/fxls8974cf_interrupt.debug+frdmk22f_a8974.bin C:/GIT/NXP_Sensor_SDK/examples/issdk/sensors/fxls8974cf/interrupt/out/fxls8974cf_interrupt/frdmk22f_a8974/debug/fxls8974cf_interrupt.debug+frdmk22f_a8974.axf""
Warning: L9931W: Your license for feature mdk_pro_compiler5 will expire in 29 days
Error: L6218E: Undefined symbol Driver_I2C0 (referred from fxls8974cf_interrupt.o).

Using compiler pointed at Keil_v5.  Does that need to change to get the v6.18 or later?
-- The ASM compiler identification is ARMClang
-- Found assembler: c:/Keil_v5/ARM/ARMCLANG/bin/armclang.exe

I tried to open the .cprj output in my Keil uVision5 tools.  It loads, but has 12 errors with missing files.
All files are shown in Project Explorer and Manage project items.  However, seems environment is not set correctly when imported.
I will wait until you have time to review after holiday.
Thanks for support.

It should build out of the box.

Succesfully links. Then fails with following:

Not sure what fails when link is successful. It looks as linking fails.
The missing Driver_I2C0 could indicate the wrong RTE_Device.h being used. That could be the case with older CMSIS-Toolbox. Are you using the latest version 1.4.0? Are you modifying the source code?
Please check the following:

clean checkout of the repo ('main' or 'develop' branch)
using CMSIS-Toolbox 1.4.0
using Arm Compiler 6.18 or later (I'm using 6.19)
using CMake 3.18.0 or higher (I'm using 3.24.1)
using Ninja 1.10.0 or higher (I'm using 1.11.0)
using all CMSIS-Packs specified in the .csolution.yml

Run:
cd examples\issdk\sensors\fxls8974cf\interrupt
csolution convert fxls8974cf_interrupt.csolution.yml
cbuild fxls8974cf_interrupt.debug+frdmk22f_a8974.cprj

There should not be any errors during csolution convert or cbuild.
If it still fails, we could check the generated .cprj and also .cbuild.yml files.
The .cprj can be also imported in Keil uVision. I believe that the latest uVision from MDK 5,38 is required to build the project successfully. I'm using MDK 5.38 and the imported project builds without errors.

Error was caused by environment variables being reset to older values within my Git-Bash terminal.
Once I updated to correct settings... worked out of the box.

A note to self to recall FIX:  Also need to fix RTE_device.h where RTE_I2C4 was not defined.


Bridge request for Mediapart blogs club

Bridge request

General information

Host URI for the bridge (i.e. https://github.com):

https://blogs.mediapart.fr/48H or https://blogs.mediapart.fr/

Which information would you like to see?

Blogs news titles

How should the information be displayed/formatted?

Which of the following parameters do you expect?

[X] Title
[X] URI (link to the original article)
[ ] Author
[ ] Timestamp
[X] Content (the content of the article)
[ ] Enclosures (pictures, videos, etc...)
[ ] Categories (categories, tags, etc...)

Options

[ ] Limit number of returned items

Default limit: 5

[ ] Load full articles

Cache articles (articles are stored in a local cache on first request): yes
Cache timeout (max = 24 hours): 24 hours

[X] Balance requests (RSS-Bridge uses cached versions to reduce bandwith usage)

Timeout (default = 5 minutes, max = 24 hours): 5 minutes

There is already MediapartBlogsBridge. Have you tried it?

ha sorry, no I never try that, how to do that ?

If you are instance admin, add MediapartBlogsBridge to whitelist.
If are not, find instance that has this bridge enabled and working: https://github.com/RSS-Bridge/rss-bridge/wiki/Public-hosts

For example : On https://feeds.proxeuse.com/ he said me enter "Blog slug", and I try to click on Atom, give me an download link, but I can't have the like of the rss generator, for copy past on my rss reader. I don't have link for follow all 48hours blogs list of Mediapart.

Ping @somini as maintainer of this bridge.

https://blogs.mediapart.fr/48H is some kind of aggregation. The MediapartBlogsBridge only works for individual blogs.
For example, for the first example on that link:

https://blogs.mediapart.fr/jean-vincot use jean-vincot as slug.

Supporting that 48H aggregator is a feature request.
Sorry about the delay when responding to this.


Antigen test profile creation: Add textContentType to text fields

Description
This PR adds textContentType to the text fields used to create the antigen test profile.
Adding a textContentType attribute allows iOS to suggest input for fields like first name or last name. This allows for a quicker profile creation. See screenshots for a demonstration of the use in practice.
The API is available for iOS 10+ so compatibility is not an issue.
Link to Jira
N/A
Screenshots

@toto Thank you for contributing.