Could not resolve com.android.tools.build:gradle:3.3.0

Error running Gradle:
ProcessException: Process "C:\Users\user\Desktop\Projects\flutter_wanandroid\android\gradlew.bat" exited abnormally:

Configure project :device_info

Project evaluation failed including an error in afterEvaluate {}. Run with --stacktrace for details of the afterEvaluate {} error.
FAILURE: Build failed with an exception.

What went wrong:
A problem occurred configuring project ':device_info'.

Could not resolve all artifacts for configuration ':device_info:classpath'.
Could not resolve com.android.tools.build:gradle:3.3.0.
Required by:
project :device_info
Could not resolve com.android.tools.build:gradle:3.3.0.
Could not get resource 'https://dl.google.com/dl/android/maven2/com/android/tools/build/gradle/3.3.0/gradle-3.3.0.pom'.
Could not GET 'https://dl.google.com/dl/android/maven2/com/android/tools/build/gradle/3.3.0/gradle-3.3.0.pom'.
sun.security.validator.ValidatorException: PKIX path building failed: sun.security.provider.certpath.SunCertPathBuilderException: unable to find valid certification path to requested target
Could not resolve com.android.tools.build:gradle:3.3.0.
Could not get resource 'https://jcenter.bintray.com/com/android/tools/build/gradle/3.3.0/gradle-3.3.0.pom'.
Could not GET 'https://jcenter.bintray.com/com/android/tools/build/gradle/3.3.0/gradle-3.3.0.pom'.
sun.security.validator.ValidatorException: PKIX path building failed: sun.security.provider.certpath.SunCertPathBuilderException: unable to find valid certification path to requested target

Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output. Run with --scan to get full insights.

像是grade-3.3.0没有下载下来，翻墙试试；
或者暂时把device_info屏蔽了。

试了都不行


Message batching via JSON array

Hi,
we're currently evaluating whether or not it would be feasible to connect multiple OPC UA endpoints (and therefore multiple logical or even physical devices) via a single instance of the opc publisher. Doing so, we ran into the fact that the opc publisher batches messages "manually" using a JSON array of messages - on application layer if you will. In our scenario, this means we might end up with messages from different OPC UA endpoints within the same IoT Hub message. This in turn becomes challenging later on within the processing pipeline, e.g., when joining incoming messages with state from CosmosDB via Azure Functions - we would need to join with multiple documents within a single trigger.
My question is the following: Is this choice intentional? Is there a reason for this design we're not seeing? It seems that the IoT Hub c# SDK offers DeviceClient.SendEventBatchAsync for transport layer optimization, which would potentially avoid this issue.
Also, any idea for a workaround, e.g., disabling batching or splitting messages up within Azure?

Any comments on this?

Sorry for the late response.
The batching was intentional to save cost and done at "app level" since not all transports support it: https://github.com/Azure/azure-iot-sdk-csharp/issues/121
As a workaround i suggest to set the message size parameter and the send interval to 0 via the --ms and --si options.
The batch is sent as a JSON array, so you can split it on the cloud side, but this requires some code on your side.

Thanks for your response! This is valuable insight.
Microsoft advertises this project as part of the reference architecture for Azure IoT. You suggested that the JSON array should be split up on the cloud side. I'd like to discuss how to achieve this best, however, I am not entirely sure that this is the right place to do so. If it's not, can you refer me to a better place?
We evaluated both Azure Stream Analytics (ASA) as well as Azure Functions for the purpose of splitting the array up. For an ASA job, it seems that multiple output "rows" gets batched again into a single Event Hub message again: https://stackoverflow.com/questions/42213418/stream-analytics-to-event-hub-unexpectedly-concatenating-events
Regarding Azure Functions, it seems that output bindings also imply one output event per trigger, i.e., a JSON array: https://docs.microsoft.com/en-us/azure/azure-functions/functions-bindings-event-hubs#output---javascript-example
Any further pointers?


fix: comment highlight when text is deleted

Changelog
🐞 Fix comment highlight when text comment was deleted
An issue was preventing the comments to be highlighted if the text of the comment was deleted.
TextBlock is an empty string when the text was deleted, so there can't be any matches.
The function returns immediately as there is no text to highlight in that case.

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


[Bug] xpath containing single quotes causes "Unexpected token"

Insomnia Version: 5.15.0
Operating System: macOS

Details
Using an XPath filter that looks like this //meta[@name='csrf-token']/@content.  The issue is the single quotes around csrf-token.
That gets rendered into this nunjucks template {% response 'body', 'req_b73584b54f414f888c2d207852ddabbc', '//meta[@name=\'csrf-token\']/@content' %}.  Even though the single quotes look to be escaped, they still cause Insomnia to throw an error.

I have the exact same problem in vesion 5.16.6.
I think that @neontuna described the bug exactly.
If I use double quotes in xpath it also fails because they are not escaped:
{% response 'body', 'req_37c185c43c0c4a308efc142dd3dbfd4d', '//input[@name="_csrf_token"]/@value' %}

Yeah unfortunately this issue got auto closed, I never found a workaround or solution just using a different REST client for testing now.

Hi @gschier, could you please reopen this issue ?

There's a more recent one here: #1030

@verglor
I've found a workaround: use double quotes escaping them manually
"ext_net_href": "{% response 'body', 'req_63e53c90ca7e40bdb711f2ed575ecbb4', '//*[@name=\"Internet_01\"]/@href' %}"

@verglor @neontuna
To define env vars I use the form with xpath filter:

then, after clicking "done", I get "unexpected token" message:

I copy the full json and paste it on a text editor, the text is the following (look at the string "Pilota"):
! unexpected token
{
	"var": "{% response 'body', 'req_63e53c90ca7e40bdb711f2ed575ecbb4', '//*[@name="Pilota"]/@href' %}"
}

I escape manually the double quotes
! manually escaped " -> \"
{
	"var": "{% response 'body', 'req_63e53c90ca7e40bdb711f2ed575ecbb4', '//*[@name=\"Pilota\"]/@href' %}"
}

then copy and past back the json to insomnia, the "unexpected token" message disappears:

I'm not sure is your case, I hope it helps.

Oh, yep, thanks for reminding me! I reopened https://github.com/getinsomnia/insomnia/issues/1030


AM-69: fixed post logout query param issue

ParamUtils class moved to handler core as it is used by AbstractLogoutEndpoint.
Also added test for ParamUtils class which shows the URLs it can compare.

The CI is failing due to useless mock usage

fixes gravitee-io/issues#8610


Restore branch name in clean-conversation-headers

Closes #3808
Test URLs
https://github.com/excalidraw/excalidraw/pull/3230
https://github.com/twbs/bootstrap/pull/33334 (non default branch)
Screenshot

It’s best to find the original PR and test on every situation in that PR, not just 2

It’s best to find the original PR and test on every situation in that PR, not just 2

Sorry I did 2 pr's at once and did not do this one correctly.

Closing this, allowing anyone to take over.


Fix en-US SumOfRawResourceMoreThan TL

Thanks! wanna fix this several times and all ended up forgotten

should have been deployed, thanks again!


AppVeyor build fails due to addition of .NET Core 2.0.2 to build image

AppVeyor updated the Visual Studio 2017 image to include the 2.0.2 SDK on 2017-10-22
https://www.appveyor.com/updates/
This results in the build failing with the following:
The currently installed .NET Command Line Tools is not the required version.
Installed version: 2.0.2
Required version: 2.0.0
Fix steps:
1. Remove the installed version from:
    - on windows '$env:LOCALAPPDATA\Microsoft\dotnet'
    - on macOS and linux '$env:HOME/.dotnet'
2. Run Start-PSBootstrap or Install-Dotnet
3. Start-PSBuild -Clea

Invoke-AppveyorTest
Run CoreCLR tests
CoreCLR pwsh.exe was not built
Invoke-AppveyorFinish
Packaging RID: 'win7-x64'; Packaging Configuration: 'Release'
Please ensure you have run 'Start-PSBuild -Clean -CrossGen -Runtime win7-x64 -Configuration Release'!

example build fail: https://ci.appveyor.com/project/PowerShell/powershell/build/6.0.0-beta.8-6270

cc/ @TravisEz13

Interenting, the image contains 2.0.0 too? #5118 can help.

I think it's because we are doing a -ne check on the version of dotnet:
https://github.com/PowerShell/PowerShell/blob/5b2bbcbf89251734e8557a55c5df40bfb1cba979/build.psm1#L414

we would probably need to fix that and merge #5118

It looks like #5118 was sufficient enough to fix the problem:
https://ci.appveyor.com/project/PowerShell/powershell/build/1.0.6275


Feature request - Allow saving/exporting values of Cbuffers.

Description
Allow us to export or save the cbuffer values to a text file and binary for further analysis.
Steps to reproduce
Debug a pixel shader, you can't save or export the cbuffer values for further analysis.
Environment
 * RenderDoc version: v1.7
 * Operating System: Win10 1809 17763.914
 * Graphics API: D3D11

Constant buffers can be exported from the constant buffer viewer, as opened from the pipeline state, using the 'export to CSV' button. If you need to save directly to binary you can open the buffer in a raw viewer view the resource inspector and export to binary from there.


Feat: download individual weights

Implements the feature to allow models to download individual weights files instead of an entire folder, e.g., YOLOX is now able to download just yolox-tiny.pth (18.9 MB) instead of the entire yolox.zip blob (332.7 MB).
Code changes:

WeightsDownloaderMixin downloads single weights file

blob_file key in model configs changed to contain a .zip file for each model type

_has_weights checks that the weights file path exists (previously dir path)
_has_weights verifies weights file integrity using SHA256 checksum (closes https://github.com/aimakerspace/PeekingDuck-Private/issues/44)

Ground truth checksum list stored as a JSON file on GCP (for now)
For SavedModel weights which stores weights in a folder, recursively hashes the files in the folder. Skips Mac OS files like .DS_Store and __MACOSX to be cross-platform compatible

Add a model_format key to model configs for support of ONNX and TensorRT weights in the future.

Refactored current model weights to either tensorflow or pytorch.

Clean-up/refactor:

Removed .pbtxt and mobilenet 101 files for PoseNet
Change YOLOv4 model sub-dir name to yolo to be consistent like other model nodes
Move "no weights" test to test_weights_downloader.py. No longer need to repeat in every model test

Non-code changes:

Uploaded new weights to GCP according to the following directory structure:

/path/to/GCP/peekingduck/models
+-<model_name_1>
| +-<model_format_1>
| | +-<model_weights_1>.zip
| | +-<model_weights_2>.zip
| | \-...
| |
| +-<model_format_2>
| \-...
|
+-<model_name_2>
\-...

Example:
/path/to/GCP/peekingduck/models
+-csrnet
|  \-tensorflow
|    +-shanghaitech-part-a.zip
|    \-shanghaitech-part-b.zip
+-efficientdet
\-...

Still going through the 40+ changed files, but here's a suggestion which will involve a major rewrite:
For ThresholdCheckerMixin.check_bounds(), consider using mathematical jargon for intervals (c.f. https://en.wikipedia.org/wiki/Interval_(mathematics)).
This will subsume the three arguments value, method, include into one interval string encoded as:
    <value> = <number> | "-inf" | "+inf"
    <left_bracket> = "(" | "["
    <right_bracket> = ")" | "]"
    <interval> = <left_bracket> <value> "," <value> <right_bracket>

where ( | ) denote an open interval, [ | ] denote a closed interval, and a mix of ( | ], [ | ) denote a half-open interval.
Some examples are:
    (0, 1)  :  0 < x < 1
    [0, 1]  :  0 <= x <= 1
    (0, 1]  :  0 < x <= 1
    [0, 1)  :  0 <= x < 1
    [0, +inf) | [0, +inf]  :  x >= 0
    [-inf, 1) | (-inf, 1)  :  x < 1

Can even use inf to replace both -inf and +inf since the meaning can be derived from its left/right positioning within the interval.

Tested weights corruption check working


Previously working teachable.com site now broken.

DO NOT REMOVE OR SKIP THE ISSUE TEMPLATE

[X] I understand that I will be blocked if I intentionally remove or skip any mandatory* field

Checklist

[X] I'm reporting that yt-dlp is broken on a supported site
[X] I've verified that I have updated yt-dlp to nightly or master (update instructions)
[X] I've checked that all provided URLs are playable in a browser with the same IP and same login details
[X] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[X] I've searched known issues and the bugtracker for similar issues including closed ones. DO NOT post duplicates
[X] I've read the guidelines for opening an issue
[X] I've read about sharing account credentials and I'm willing to share it if required

Region
United States
Provide a description that is worded well enough to be understood
Running this command has worked for the past 6-8 months. Now it errors saying that the site is Unsupported.

yt-dlp -vvU --cookies cookies.txt --referer https://player.hotmart.com/ `https://frank-bunn.teachable.com/courses/2442083/lectures/54629763

Site headers confirm the referrer is still https://player.hotmart.com. I updated the site cookies as well.
Provide verbose output that clearly demonstrates the problem

[X] Run your yt-dlp command with -vU flag added (yt-dlp -vU <your command line>)
[X] If using API, add 'verbose': True to YoutubeDL params instead
[X] Copy the WHOLE output (starting with [debug] Command-line config) and insert it below

Complete Verbose Output
yt-dlp -vvU --cookies cookies.txt --referer https://player.hotmart.com/ https://frank-bunn.teachable.com/courses/2442083/lectures/54629763
[debug] Command-line config: ['-vvU', '--cookies', 'cookies.txt', '--referer', 'https://player.hotmart.com/', 'https://frank-bunn.teachable.com/courses/2442083/lectures/54629763']
[debug] Encodings: locale UTF-8, fs utf-8, pref UTF-8, out utf-8, error utf-8, screen utf-8
[debug] yt-dlp version<EMAIL_ADDRESS>from yt-dlp/yt-dlp [12b248ce6] (debian*)
[debug] Python 3.10.12 (CPython x86_64 64bit) - Linux-<IP_ADDRESS>-microsoft-standard-WSL2-x86_64-with-glibc2.35 (OpenSSL 3.0.2 15 Mar 2022, glibc 2.35)
[debug] exe versions: ffmpeg 4.4.2 (setts), ffprobe 4.4.2, rtmpdump 2.4
[debug] Optional libraries: Cryptodome-3.11.0, brotli-1.0.9, certifi-2020.06.20, mutagen-1.45.1, pyxattr-0.7.2, requests-2.31.0, secretstorage-3.3.1, sqlite3-3.37.2, urllib3-2.1.0, websockets-9.1
[debug] Proxy map: {}
[debug] Request Handlers: urllib, requests
[debug] Loaded 1820 extractors
[debug] Fetching release info: https://api.github.com/repos/yt-dlp/yt-dlp/releases/latest
Latest version<EMAIL_ADDRESS>from yt-dlp/yt-dlp
yt-dlp is up to date<EMAIL_ADDRESS>from yt-dlp/yt-dlp)
[generic] Extracting URL: https://frank-bunn.teachable.com/courses/2442083/lectures/54629763
[generic] 54629763: Downloading webpage
WARNING: [generic] Falling back on generic information extractor
[generic] 54629763: Extracting information
[debug] Looking for embeds
ERROR: Unsupported URL: https://frank-bunn.teachable.com/courses/2442083/lectures/54629763
Traceback (most recent call last):
  File "/usr/lib/python3/dist-packages/yt_dlp/YoutubeDL.py", line 1606, in wrapper
    return func(self, *args, **kwargs)
  File "/usr/lib/python3/dist-packages/yt_dlp/YoutubeDL.py", line 1741, in __extract_info
    ie_result = ie.extract(url)
  File "/usr/lib/python3/dist-packages/yt_dlp/extractor/common.py", line 734, in extract
    ie_result = self._real_extract(url)
  File "/usr/lib/python3/dist-packages/yt_dlp/extractor/generic.py", line 2514, in _real_extract
    raise UnsupportedError(url)
yt_dlp.utils.UnsupportedError: Unsupported URL: https://frank-bunn.teachable.com/courses/2442083/lectures/54629763

try prefixing the input URL with teachable:, e.g.
yt-dlp -vU --cookies cookies.txt --referer "https://player.hotmart.com/" "teachable:https://frank-bunn.teachable.com/courses/2442083/lectures/54629763"

Here is the updated debug:
[debug] Command-line config: ['-vU', '--cookies', 'cookies.txt', '--referer', 'https://player.hotmart.com/', 'teachable:https://frank-bunn.teachable.com/courses/2442083/lectures/54530225']
[debug] Encodings: locale UTF-8, fs utf-8, pref UTF-8, out utf-8, error utf-8, screen utf-8
[debug] yt-dlp version<EMAIL_ADDRESS>from yt-dlp/yt-dlp [12b248ce6] (debian*)
[debug] Python 3.10.12 (CPython x86_64 64bit) - Linux-<IP_ADDRESS>-microsoft-standard-WSL2-x86_64-with-glibc2.35 (OpenSSL 3.0.2 15 Mar 2022, glibc 2.35)
[debug] exe versions: ffmpeg 4.4.2 (setts), ffprobe 4.4.2, rtmpdump 2.4
[debug] Optional libraries: Cryptodome-3.11.0, brotli-1.0.9, certifi-2020.06.20, mutagen-1.45.1, pyxattr-0.7.2, requests-2.31.0, secretstorage-3.3.1, sqlite3-3.37.2, urllib3-2.1.0, websockets-9.1
[debug] Proxy map: {}
[debug] Request Handlers: urllib, requests
[debug] Loaded 1820 extractors
[debug] Fetching release info: https://api.github.com/repos/yt-dlp/yt-dlp/releases/latest
Latest version<EMAIL_ADDRESS>from yt-dlp/yt-dlp
yt-dlp is up to date<EMAIL_ADDRESS>from yt-dlp/yt-dlp)
WARNING: The program functionality for this site has been marked as broken, and will probably not work.
[Teachable] Extracting URL: teachable:https://frank-bunn.teachable.com/courses/2442083/lectures/54530225
[Teachable] 54530225: Downloading webpage
ERROR: [Teachable] 54530225: Unable to find video URL; please report this issue on  https://github.com/yt-dlp/yt-dlp/issues?q= , filling out the appropriate issue template. Confirm you are on the latest version using  yt-dlp -U
File "/usr/lib/python3/dist-packages/yt_dlp/extractor/common.py", line 734, in extract
ie_result = self._real_extract(url)
File "/usr/lib/python3/dist-packages/yt_dlp/extractor/teachable.py", line 174, in _real_extract
raise ExtractorError('Unable to find video URL')

Duplicate of #3564
You could try #7650

I switched back to the original script by @Green0Photon in #7650 and it works using "teachable:https://...". I haven't needed this in the past, and it's strange it's not working with yt-dlp natively.
Regards

I switched back to the original script by @Green0Photon in #7650 and it works using "teachable:https://...". I haven't needed this in the past, and it's strange it's not working with yt-dlp natively.
Regards

how exactly does your call of the script look like, I tried it but was not successful. Am I right that I have to export the Cockie information from my browser to a text file first?


Cropping pictures

Hi!! I liked TLPhotoPicker, but I need to be able to crop pictures... Do you have any intention to add this feature?

I think If you want cropped image, you better use other framework than I did.
Like this : https://github.com/TimOliver/TOCropViewController

I need to be able to select multiple pictures and then crop them. Does your framework allow that?

I'm afraid not.


[Site]: Text doesn't align in chinese language

Current State
Description of the current state/issue:
I want set text-align style fix it, but I'm not sure it's compatible with all other translation languages.
The problem of Chinese here is 3-4 bytes, so the width of the text cannot be matched. I want to use css to solve it, but this will lead to a problem that it may not have stable horizontal text spacing (other languages will be Affect)
Link to the page:
https://apisix.apache.org/zh/
Screenshots (if relevant):

Desired State
Description of the change you are proposing:

Would you like to fix this issue?
Yes! Count me in

I will close this issue on May 22 because it is not a good proposal.

I think it's a very good idea.
We can set the style of the Chinese and English fonts separately, which is what I intend to do, but there may be some difficulties.

I think it's a very good idea.
We can set the style of the Chinese and English fonts separately, which is what I intend to do, but there may be some difficulties.

I will close this pr first, and change my mind to solve this problem. Obviously, this style sheet attribute does not apply to all languages.


Eliminate extra space where email is hidden in logged-out profile view

The optional email on a user's profile is leaving extra space when it's not being shown for logged-out users. For example, on https://pypi.org/user/admin:

Logged out
Logged in

Good First Issue: This issue is good for first time contributors. If you've already contributed to Warehouse, work on another issue without this label instead. If there is not a corresponding pull request for this issue, it is up for grabs. For directions for getting set up, see our Getting Started Guide. If you are working on this issue and have questions, feel free to ask them here, #pypa-dev on Freenode, or the pypa-dev mailing list.
Screenshot Required: If your pull request makes a visual change, include a screenshot of your update. This helps our team give you feedback faster.

Hey @di, I tried doing this with css like :blank but was having issues getting it to apply correctly. Another approach is to check request.authenticated_userid, I just need to set uses_session=True on the view_config. What do you think about taking this approach? Do you see any issues?

Yes, we don't want this page to require a session because it would mean it's not cacheable, hence why we're including some parts of it which do require a session (like this email address, which is only displayed for logged-in users) via a client-side include.

Thanks for clarifying. I was eventually able to figure out a way to do it with html and css alone. Not 100% sure if I am making efficient use of the css. Please take a look when you can and let me know what you think.


Cannot validate attributes that are not objects

The following example for validating a non-object field fails because activerecord_json_validator tries to parse every String value as JSON object and defaults to {} if this doesn't work:
class Condition < ActiveRecord::Base
  FIELD_SCHEMA = {
    type: "string",
    enum: ["earnings", "label", "city"]
  }.to_json
  validates :field, json: { schema: FIELD_SCHEMA }
  # ...
end

This also prevents the usage of the fragment option of json-schema in https://github.com/mirego/activerecord_json_validator/issues/30 to validate multiple (String) fields out of a single JSON schema.

The following example for validating a non-object field fails

Can you actually show an example of how Condition#field= fails? What is the argument you’re passing?

I should have been more specific about what happens in that case:
If you assign a new String value to such a string field, the value is replaced by {}.
I created a PR that both shows the problem and has a potential solution: https://github.com/mirego/activerecord_json_validator/pull/32

I’m going to close this issue since #32 and #33 are now closed.


Snap created and ready for publication

Hi,
I've recently created a snap for ttyd and have it working with classic confinement. Meaning it will run "as native" within a snap container for any Linux distro supporting snapd. Learn more about snaps here.
I am happy to maintain the snap for your project, unless you'd prefer maintaining it all "in-house". There's not much anyone on the team would have to do, as the snaps are automatically built and published on the snap store.
As part of the security checks, pedigree, and validation, the upstream team/devs should at least know about it and agree with the publication as a classic snap. Basically, the snap will much like a .deb or .rpm on another distro supporting those packages; except it's distro agnostic.
I hope this comes as a welcome PR. If you'd like to be involved in the convo with the snapcraft team, please see this thread.

Hi @tsl0922  - reaching out again regarding the forum thread I posted above. I'm attempting to publish this snap as a "classic" snap - meaning I need your approval to do so. If you are willing to allow that, do you mind responding here or providing a response in the forum thread?

I can help publishing it to snap store, but it seems the name  ttyd is already registered.

Yes I registered it - created the snap and I'm ready to publish it. Just need your permission. If you'd rather manage it all, you'll need to join the forum thread and open discussions with the snapcraft team.


Load JSON with http service

I have create JSON file and put the valid json structure as per fomly, and load with http service but it is not working, but if i put that same json in component then it is working fine
JSON file
[
{
"fieldGroupClassName": "row",
"fieldGroup": [
{
"key": "firstName",
"type": "input",
"className": "col-md-6",
"templateOptions": {
"type": "text",
"label": "First Name",
"id": "txtFname",
"tabindex": "1"
},
"validators": {
"validation": "Validators.compose([Validators.required, Validators.maxLength(10), Validators.minLength(2)])"
},
"validation": {
"show": true,
"messages": {
"className": "",
"required": "First name is required",
"minlength": "First name must be atleast 2 character long.",
"maxlength": "First name cannot be more than 10 character long."
}
}
}
]
}
]
Diffrence between component and JSON file structure is
In JSON file i have to provide double quotes [" "] to create valid json and in component i have not provide any double quotes [" "]

"validators": {
"validation": "Validators.compose([Validators.required, Validators.maxLength(10), Validators.minLength(2)])"
}

You can't take a JavaScript object that includes function references, escape those references with double quotes and expect things to work.  There are examples in the demo that will jive with fields declared in JSON.

As suggested in example it is showing me an error and i have made entry in JSON as suggested
JSON Entry
{
fieldGroupClassName: 'row',
fieldGroup: [
{
className: 'col-md-6',
type: 'input',
key: 'street',
validators: {
validation: ['required'],
},
validation: {
messages: {
required: 'WOW!! Enter the Street',
},
},
defaultValue: '204 Causley Ave. ',
templateOptions: {
label: 'Street',
placeholder: '604 Causley Ave. ',
description: 'Enter a valid US Address',
}
}
]
}
ERROR
Error: Uncaught (in promise): Error: [Formly Error] There is no validator by the name of "required"
Error: [Formly Error] There is no validator by the name of "required"

You can't reference a validator as a string without defining it in the formly config.  Please refer to the demo.

I have checked provided demo sample but in that sample only required validation is working,
className: 'col-md-6',
type: 'input',
key: 'street',
validators: {
validation: ['required'],
},
but i needs to apply maxlenth, minlenth, pattern and custom validation then it is not working, and please provide my working sample for that with all the validation.

@bhupendrasolanki251185 you must define them in the config section in order to use them in the form, similar to required example


net: lwm2m: replace coap_next_block() w/ coap_update_from_block()

We should call coap_update_from_block() which will determine the minimum
size of the BLOCK1 SIZE between server/client and update the current
offset and total size(if available) accordingly.
This PR shouldn't be conflict w/ #4557

@rbtchc This looks good, but I had to do quite a bit of work to my backend changes forhttps://github.com/zephyrproject-rtos/zephyr/pull/4557.  Can we hold this till after that's merged.  I touched some of this code.

@mike-scott no problem. let's wait.

Rebase on top of the master branch which has  #4557 merged now.
@mike-scott plz let me know if you have any problem w/ it.

LGTM.  Have you run tests to confirm that block size negotiation is working correctly?

@mike-scott Yes, I did. It will now use the minimum block_size between server/client. Originally, it will use whatever we set to "CONFIG_LWM2M_COAP_BLOCK_SIZE"


Angular + Webpack support

@samturrell @limonte . #187
I have created small app with webpack and sweetalert2. I have mentioned the errors near by particular statement in app.js. Here is the repository link https://github.com/kalpitpandit/sweetalert2-with-webpack.
Would really appreciate your input on it.
Thanks.

@limonate Can you please open this issue as sweetalert2 is not working with webpack.

@kalpitpandit looking at your repo, are you saying after running the webpack command you receive the errors in your app.js comments? On my testing environment everything packs fine from your github repo.
Here's my output for comparison:

Hash: a3589b023294334bc34d
Version: webpack 1.13.1
Time: 1400ms
Asset     Size  Chunks             Chunk Names
./build/bundle.js  1.25 MB       0  [emitted]  main
+ 6 hidden modules

@acupajoe No. Bundle is getting build properly. But when you run it in browser, You will see errors in browser console.

On further inspection of the provided example (Thanks for that @kalpitpandit), it looks like it could be plugin related.
Anybody else have any experience with webpack/angular? I'm pretty sure it's a module packaging issue that angular doesn't like though rather than the webpack implementation.
@kalpitpandit - Do you know if angular has any specific module packaging requirements?

May also be angulars internal module loading system expecting an actual ng-module, in which case i'd recommend instantiating it globally or within the module as normal instead of a dependency.

I'm closing this due to inactivity, reopen if needed.

swal2 works perfectly with webpack, I just made the demo: https://github.com/limonte/sweetalert2-webpack-demo

I think it is worth puting in the readme the webpack usage:
require('sweetalert2');
require('sweetalert2/dist/sweetalert2.css');

you also need to import sweetalert2.css to work properly

@sibelius done, thx!

@limonte Thanks a ton for example. I will give a try to it and update here.


Error: 'CocoaPods could not find compatible versions for pod "amplitude-react-native"' on react-native version 0.61.4

Previous issue https://github.com/amplitude/Amplitude-ReactNative/issues/44
Environment:
macOS Catalina: 10.15.4
node: 14.15.5
pod: 1.10.1
Packages:
"@amplitude/react-native": "^2.0.11"
"react-native": "0.61.4"
Reproduce:
Run npm install @amplitude/react-native@latest
Run cd ios && pod install
Expected result:
Command pod install successfully
Result:

I tried run commands from issue https://github.com/amplitude/Amplitude-ReactNative/issues/46 , but it didn't solve my problem.
If I create an empty project via npx react-native init AwesomeProject command (react-native version 0.64.0) everything works:

If I create an empty project via npx react-native init AwesomeProject --version 0.61.4 command  (react-native version 0.61.4) I get an error:

There is no information about this issue in the documentation https://developers.amplitude.com/docs/react-native-setup
Please provide information on what is the minimum version of react-native that can be used.
Thx.

Hi @ALPNP.  We support 0.61 and higher.  We are currently updating the documentation around older versions, thanks for your patience as we get those written.
In the meantime, here's a solution that worked for a user on 0.61.5

fix part one - xcode:

Open your [project-name].xcodeproj file in XCode

Right Click your project name in the file navigator and then click New File, pick swift, it will then prompt you to creating a bridging headers file.  This is necessary to support swift in RN 0.61.5.

Source of this fix: https://stackoverflow.com/a/54586937

fix part two - you will also need to make 4 changes to podfile:

Add use_modular_headers! globally

Disable modular headers for DoubleConversion, Glog and Folly, like this PR commit for my test app.

I'm pretty sure this will fix your problem, if you dont mind just let me know after trying if youre having any issues.

Here is a commit by commit example of getting it to work with 0.61.4: https://github.com/kelsonpw/AmpRN-0.61.4-Example/commits/main

Problem solved. Thanks for the help. Have a nice day.


CSS

CSS
User Story
As a boot camp student
I want the prework notes to be structured on a webpage
So that I can easily find and read the information
Acceptance Criteria
GIVEN a Prework Study Guide website
WHEN I view the study guide
THEN I see a dark blue header and footer, and four boxes with a shadow

CSS is added and HTML file is updated.


Simultaneous log access

Hi.
Can you clarify the statement "Lumberjack assumes that only one process is writing to the output files. Using the same lumberjack configuration from multiple processes on the same machine will result in improper behavior."
Does this mean that only one goroutine within an executable can use log.Write, or that only one executable can write to any particular file?
Thank you.

Lumberjack is goroutine / threadsafe but it's not safe to have two running processes (executables) writing to the same logfile, because tracking the size of the file is done in-memory.

@natefinch Does this mean I can safely have multiple instances of lumberjack. Logger with the same Filename?


Fixes #3: Fix linting issues, add ansible-lint and yamllint configuration.

Closes #3.

Note that this PR adds a GitHub action for linting that will run on any pull request or push to the master branch.

Rebased.

The PR is good. However, we need to verify that Github action is suitable for this project. We will probably want to run the playbook in the CI to verify that it is working as expected in the future. I am not sure that Github action is the best choice. @itwars what do you think ?

@St0rmingBr4in - I was considering also adding a separate workflow (or merging that into this lint.yml workflow and renaming the workflow) with a Molecule configuration that actually runs the playbook against a host, so it could also be used for local development and testing (in addition to CI), but I figured I'd do that separately, since the linting steps at least guarantee some amount of conformity/code structure.

Sure we will do that in another PR.
About CI choices lets wait @itwars input on GitHub action.
On the one hand GitHub action workflows can become ugly fast if we need to overcome some limitations. (Ability to share artifacts between workflows or dependence between workflows for instance)
On the other hand it's nicely integrated with GitHub and we could use some actions from the markeplace if needed.

LGTM


implement invokeLambda step

Please check if the PR fulfills these requirements

[x] The commit message describes your change
[x] Tests for the changes have been added if possible (for bug fixes / features)
[x] Docs have been added / updated (for bug fixes / features)
[x] Changes are mentioned in the changelog (for bug fixes / features)

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)

Add support for invoking Lambdas

Does this PR introduce a breaking change? (What changes might users need to make in their setup due to this PR?)

No

Looks good. Can you please qualify field and method access with this.


Library release removal request

Library name
Arancino
Version to remove
3.1.0

Hi @sergiotomasello. I am confirming your request.
Unlike submissions, this type of request is still handled manually. The Library Manager index maintainer occasionally comes through to clear all the outstanding requests. They will close this issue when it is done. You will get a notification from GitHub at that time.
After that, you can monitor the indexing of the library via its dedicated logs web page.
Information about that here:
https://github.com/arduino/library-registry/blob/main/FAQ.md#can-i-check-on-library-releases-being-added-to-library-manager

Done:
2023/04/19 15:42:51 Loaded 5753 libraries from DB
Removing<EMAIL_ADDRESS>Success!


Add a parameter to allow show custom Text message

Hi!!,
I good idea could be add an extra parameter like 'customMessage' or something like that, to able to add a custom message, by example, a RichText widget to show it
Best regard!!

maybe leave as Widget object parameter to able to put anything, like images, text, etc.,

Hey @luisrodas-digital,
Sure!, added it to our to-do list. we will make the dialogs more customizable in the next updates.
Thanks & regards!

@CodeWithKamal when this will come, we are in 2024

@CodeWithKamal when this will come, we are in 2024

let me update the whole package this weekend.

Now with version 0.1.5, you can change dialog background colour as well as just use PanaraCustomDialog which let's you add your own widgets base/plain dialog.


HAWQ-1227. Fix hawq init fails when user name with a capital.

Add single quote to ROLENAME, so it's capital sensitive.
Add option '-v ON_ERROR_STOP=on' to psql client, so it will fail for each sql queries in the sql file but not only the last query.

LGTM. +1

Last time when I encountered this issue, it was one year ago. It seems I am the only one who likes to use a capitalized user name, :). Glad to hear that this issue is finally resolved. Nice job. +1.
Lirong

CEO is a great capitalized user name.


GC can cause unexpected selector timeout

I was comparing Netty's NioEventLoop to another event loop to see if there were any behavioral differences.  One difference I found was a bug fix that doesn't seem to be in Netty.  It is possible that Netty is not affected.  Consider the following scenario:
A client is connected to a server, and periodically sends keep alives, or health checks.   The server records these health checks, and has a scheduled task that makes sure the client is still connected.  At some point, the server enters a very heavy GC cycle, which may take longer than the keep alive interval.    The relevant code is in NioEventLoop:
                int selectedKeys = selector.select(timeoutMillis);
                selectCnt ++;

                if (selectedKeys != 0 || oldWakenUp || wakenUp.get() || hasTasks() || hasScheduledTasks()) {
                    // - Selected something,
                    // - waken up by user, or
                    // - the task queue has a pending task.
                    // - a scheduled task is ready for processing
                    break;
                }

When the selector times out, it will notice that no keys have been selected, but it does have scheduled tasks.  The scheduled tasks will run, namely the health checker.  Seeing that no health check was received since the last time, it will think the client has gone away, and kill the connection.  However, the client did in fact send a health check; the server would have noticed it on the next select.
The fix in our code looks like this:
        int selected = selector.select(timeoutMs);
        if (selected == 0) {
          selector.selectNow();
        }

In the event that nothing was selected, a last ditch selectNow is made to be sure.  It was decided that the cost of this second select was small, and would happen at most once per second  (Our default timeout, same as Netty).
Before I go further on trying to make a reproducible test case and fix, I wanted to see what others think of this situation?  Keep Alives are a higher level concept, but need a fix at a lower layer.
@normanmaurer @Scottmitch @ejona86  thoughts?

I think it's fine without the "fix"? Whose to say what's happening in the scheduled tasks? Maybe the timeout is to send a watchdog keepalive, and so the extra I/O delays sending even further.
Any time something like keepalives and super-long GC are combined, things will be "interesting" and you'll have to deal with it.
Note that this change would only really help if the scheduled task was the same event loop. While that is likely, it seems strange to special-case.

@zhangkun83  who originally made the fix.

The original fix (more like a hack) on our internal system was deemed necessary because the delay was annoying enough -- it stopped an integration test from starting up. Another factor to consider is that our internal system heavily relies on timely health-checking, which is not the case for gRPC.

Note that PR https://github.com/netty/netty/pull/7042 changes this timeout handling approach for EPOLL.
I agree with @ejona86 that timers + GC (which may block the current thread) may lead to unexpected results.

Agree with @ejona86 here as well. I would suggest we close this issue for now

Let me close this for now... @carl-mastrangelo if you still think we need to do anything please reopen.


 Please, it's the best for you and this world.

Yo @juandavid45
please do this its for your own good

Wtf is wrong with you, also you need the icons for the credits, so this code is literally wrong

that's why i blocked the dummy

I should mention this here.


Query editor crashes when incorrectly creating a function

Environment:
Docker nightly 3-dev7241, Chrome browser
To reproduce:
In the query editor run create function test without trailing semi.
Crashes with:
Something went wrong rendering this view
Traceback:
RangeError: Mark decorations may not be empty
    at n.value (http://localhost:5656/ui/static/js/main.21807f5f.js:2:928075)
    at JL (http://localhost:5656/ui/static/js/main.21807f5f.js:2:1216499)
    at http://localhost:5656/ui/static/js/main.21807f5f.js:2:1221845
    at gs (http://localhost:5656/ui/static/js/main.21807f5f.js:2:316111)
    at Tu (http://localhost:5656/ui/static/js/main.21807f5f.js:2:333229)
    at t.unstable_runWithPriority (http://localhost:5656/ui/static/js/main.21807f5f.js:2:368341)
    at Qi (http://localhost:5656/ui/static/js/main.21807f5f.js:2:275096)
    at Nu (http://localhost:5656/ui/static/js/main.21807f5f.js:2:329964)
    at bu (http://localhost:5656/ui/static/js/main.21807f5f.js:2:326050)
    at http://localhost:5656/ui/static/js/main.21807f5f.js:2:275319```

Just noticed this:
https://github.com/edgedb/edgedb-ui/issues/102
Query Builder crashes with empty schema -- guessing it's related?


Re-enable self-tests on armv7 and x86 self-hosted runners

@klutchell the invariant tag is called X64, not amd64.

We are specifically looking for platform:linux/amd64 which is added at build time. We don't need to use the GitHub automatic labels.

@klutchell found it: https://github.com/organizations/balena-io/settings/actions/runner-groups/3

@klutchell shod be good to go now
The agents were failing self updates due to home directory permissions.

I self-certify!


Various changes (argparse, logging, pathlib)

Hello, here are some changes I would contribute back, if you're interested.
Split into feature-commits (...mostly. Not completely cleanly) for easy cherry-picking.
Note the pathlib change raises py3 requirements to 3.4

I would prefer to stick to Python 3.2 so avoir pathlib. The other changes are OK. Can you adapt the PR ?
What I would really like to have is minimal test coverages with examples and a travis ci setup. That's my only project without one !

Sure, I've rebased the changes.
I'm not really great at writing python unittests, so I'll have to disappoint on that score.
Though I've tested the changes against several gigs of mail archives, and found no regressions here.

Ok then.


Moose 3.0

About the PR
Back by overwhelming popular demand, and after extensive renovations by a dedicated NT crew and long hours invested by multiple teams of elite, wizard-class janitors, Moose Station is ready to be brought back into commission. Featuring a modernized cargo bay and xenoarch lab, along with standards-compliant evacuation and loading docks, this small asteroid station also adds some much needed dormitories and even a state-of-the-art holodeck entertainment room.  Now with almost 2x the maintenance hallways!
Screenshots

Changelog

:cl: checkraze

add: Added a renovated Moose Station

image did not uplolad

image did not uplolad

ok its better now

issues have been addressed and is ready for another review, changing the screenshot once the maprender is finished

I think Emisse's review, emo's review, and my review are all good, and you're a trusted mapper. Thus, playtest time.

Alr well I won't be around for 12 or so ish hours so make sure to leave feedback here or on discord about it

missing fire extinguisher cabinets


Add universal key code constants

This change is

:pushpin: Commit 35312c5 has been approved by jdm

@bors-servo r+

:hourglass: Testing commit 35312c56db192557711e60d92bdc3ac7170b2780 with merge a1f4d07bfabd5b1e36f03b504c461a4b8142d332...

:sunny: Test successful - status-travis
Approved by: jdm
Pushing a1f4d07bfabd5b1e36f03b504c461a4b8142d332 to master...


Unspace not documented

See https://design.perl6.org/S02.html#Unspaces

fixed with https://github.com/perl6/doc/commit/562179787b


Moved to src folder

@scheib, can you write down the benefits of this move? The actual source was already in /app/src/, and Idea and gradle don't add that many files to the root, so there'd be plenty of space to stick a /doc or similar folder next to them.

OK to abandon this if we think it is already tidy enough, the project and
build files seemed noisy at root to me but this isn't a big deal. Can
always move later if there is a need then.
On Mar 4, 2015 3:42 PM, "Jeffrey Yasskin"<EMAIL_ADDRESS>wrote:

@scheib https://github.com/scheib, can you write down the benefits of
this move? The actual source was already in /app/src/, and Idea and gradle
don't add that many files to the root, so there'd be plenty of space to
stick a /doc or similar folder next to them.
—
Reply to this email directly or view it on GitHub
https://github.com/WebBluetoothCG/ble-test-peripheral-android/pull/3#issuecomment-77275674
.


Add a warning that this is a generated file

Added a header in the generated file that states this is actually generated.
This is important in big projects with a lot of files so no one would accidentally edit the generated file.

Great!
I don't have Merge permissions in this repo. Can you merge the PR?

Great!
I don't have Merge permissions in this repo. Can you merge the PR?

Try once more? You should have the permission now :D


Specify presto version by property

If we can specify presto version, it might be better for test. So this pr adds a property for that.

:+1: One thing to note is that the newer versions of Presto include Jersey 2, which can sometimes conflict in weird ways with Dropwizard's use of Jersey 1. Hopefully soon Dropwizard 0.8 will be out and we can upgrade Airpal to Jersey 2, eliminating that class of issues.

Thanks for reminding that!


[11.x] Re-refresh the database if the RefreshDatabase transaction was committed

The RefreshDatabase trait:

Runs migrate:fresh before the first test
Opens a database transaction before each test
Rolls back that transaction after each test

The issue is that it is possible to accidentally commit the transaction started by RefreshDatabase. For example, truncating a table automatically commits any active transactions. If a test commits the transaction, all database records created by that test won't be rolled back, causing each subsequent test to have a dirty database.
This is especially frustrating to debug because the test that commits the transaction passes, but tests after that fail unpredictably depending on the type of assertions they do. The logical place to start debugging the issue is with the failing tests, but those have nothing to do with the problem.
This PR solves this issue by refreshing the database if you (accidentally) commit the database transaction started by the RefreshDatabase trait.
To reproduce this issue, use MySQL, then run the following tests:
class ExampleTest extends TestCase
{
    use RefreshDatabase; // or LazilyRefreshDatabase

    public function test_the_first_test()
    {
        User::truncate();

        User::factory()->create();

        $this->expectNotToPerformAssertions();
    }

    public function test_the_second_test()
    {
        // This test passes if you run it in isolation, but it fails if you run all tests
        $this->assertSame(0, User::count());
    }
}

Is it possible to add a test for this in some way?


Adoption Team model - image resolution

Hi,
would it be possible to publish the Large Scale Corporate Adoption Model image with a higher resolution, please?
Many thanks!
regards,
Liz

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

ID: a3382a91-be9f-bfab-d91f-3197a6655b97
Version Independent ID: 51459c35-8bb3-1aaa-0da0-2128d29251ea
Content: Define outcomes and success for your Microsoft Teams adoption
Content Source: Teams/teams-adoption-define-outcomes.md
Service: msteams
GitHub Login: @karuanag
Microsoft Alias: karuanag

@officedocsbot assign @scanum

Hi @jambirk
Kindly look into this issue. Or suggest us if we need to report this issue to some other team? Thank you.

@jambirk Hi James, any updates on this?

@liesnys Thank you for your feedback, I'm investigating further.

I don't know who would have the original source file. While I agree the image is fuzzy and poor quality, redoing it without that source file might be difficult.  @LanaChin and @karuanag might have the originals. With that, getting a resized image would be possible.

@karuanag @LanaChin Hi! Following up on this issue. Are there any updates on this? Thanks!

@scanum - I've sent you the replacement image in email. Thank you.

@scanum This can be closed, thanks.

@liesnys Thank you for submitting feedback. We understand that this issue has been resolved.
Please feel free to re-open this issue if there is a specific area of the docs that we can improve or make better. Thank you.


fix: add revive and address indent-error-flow issues

Add revive linter and enable indent-error-flow rule only.

Thanks for your contribution! :tada: :+1:


chore(*): s/continuity/fun_prop/

Just in case you didn't see this, I also worked in this direction in the interim: #14009, #13959 and #13994 have some "easy" fixes (just replacements, and some surrounding golfs) and can land already.
Sorry for duplicating work; I would be happy to see either PR land.

#13869 is a small chore necessary for using fun_prop in measure theory; the fun_prop basically approved this, but this needs a maintainer's look.

Closing, we have better progress in other PRs now.


main: Use configuration file instead of flags and simplify app extension

Clair will now use a YAML configuration file instead of command line
arguments as the number of parameters grows.
Also, Clair now exposes a Boot() func that allows everyone to easily
create their own project and load dynamically their own fetchers/updaters.

+7k -4k. Enjoy..

LGTM


updates on header about and footer section

here are my updates on header about and footer section

here is header about and footer section


fix: remove the open classes if no visible reveal exists after destro…

…ying one
Closes https://github.com/zurb/foundation-sites/issues/10626

I guess I oversaw a small bit, will change it later.

Just forgot the this., will add later =)

@DanielRuf Please for each PR:

run tests
start the docs
ensure that no similar change can be made elsewhere
ensure thet the documentation is up-to-date
add comments and references, there's never too much comments ;)

run tests
Might be a good idea to create a precommit hook in general. I'll add it to my todo list.

Will do the rest later today.

@DanielRuf Can you share your Todo list (as issues) ?

We have to check the other failing unit tests on Windows (IE 11 and Firefox 58) as these are different issues.
Who could take a look at them?


KAFKA-18213: Convert CustomQuotaCallbackTest to KRaft

Currently, the test can start up using KRaft, but displays some inconsistent behavior involving whether or not the throttle-time metric shows up on the broker when doing test validation during producing.
Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

Thanks for picking this up @kevin-wu24. Looks like you've got test failures https://github.com/apache/kafka/actions/runs/12264381216/job/34218790498?pr=18126#step:11:1827

Yeah, I'm working on debugging this test currently, since it won't start running in KRaft mode. I'm not really familiar with quotas and I've never seen this "empty serde in RequestContext" error before that is preventing the test from completing an inital CreateTopics request. Any ideas?

I created https://issues.apache.org/jira/browse/KAFKA-18213 to track this tasks as the remaining ZooKeeper tests are blocking the ZooKeeper removal work. I wasn't sure what's your Jira username so I assigned the ticket to myself to avoid anyone else grabbing it, @kevin-wu24 please reassign the ticket to you.

@kevin-wu24 could you please fix the conflicts?

The root cause is KAFKA-18225, which pertains to an unsupported public API in KRaft mode.

I'm not really familiar with quotas and I've never seen this "empty serde in RequestContext" error before that is preventing the test from completing an inital CreateTopics request. Any ideas?

I guess you have fixed it as the PR is using DefaultKafkaPrincipalBuilder now


Workaround to make it compile in WASM

The generated code for tree-sitter-swift is that long that is making the WASM generation to exceed the local limits.
Using wasm-opt -O is improving the generated code avoiding the issue.
https://github.com/ChimeHQ/SwiftTreeSitter/issues/8
This is a workaround, I have already notified the Swift WASM toolchain guys to try to solve that issue.
Cheers

Thank you so much!


Self-referencing typedef causes stack overflow

Example:
SizeRec :: distinct SizeRec

Possibly related to #313

No longer crashes.


PHOENIX-5943 - Consider splitting up SYSTEM.TASK into 2 tables - one …

…for queued jobs and one for task history

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
1m 14s
Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s
No case conflicting files found.

+1 :green_heart:
hbaseanti
0m  0s
Patch does not have any anti-patterns.

+1 :green_heart:
@author
0m  0s
The patch does not contain any @author tags.

+1 :green_heart:
test4tests
0m  0s
The patch appears to include 1 new or modified test files.

_ master Compile Tests _

-1 :x:
mvninstall
3m 58s
root in master failed.

+1 :green_heart:
compile
1m  4s
master passed

+1 :green_heart:
checkstyle
3m 14s
master passed

+1 :green_heart:
javadoc
0m 47s
master passed

+0 :ok:
spotbugs
3m  8s
phoenix-core in master has 962 extant spotbugs warnings.

_ Patch Compile Tests _

-1 :x:
mvninstall
1m  8s
root in the patch failed.

+1 :green_heart:
compile
0m 57s
the patch passed

+1 :green_heart:
javac
0m 57s
the patch passed

-1 :x:
checkstyle
3m 18s
phoenix-core: The patch generated 254 new + 9998 unchanged - 118 fixed = 10252 total (was 10116)

+1 :green_heart:
whitespace
0m  0s
The patch has no whitespace issues.

+1 :green_heart:
javadoc
0m 45s
the patch passed

-1 :x:
spotbugs
3m 24s
phoenix-core generated 6 new + 958 unchanged - 4 fixed = 964 total (was 962)

_ Other Tests _

-1 :x:
unit
168m 13s
phoenix-core in the patch failed.

+1 :green_heart:
asflicense
0m 24s
The patch does not generate ASF License warnings.

194m 18s

Reason
Tests

FindBugs
module:phoenix-core

org.apache.phoenix.jdbc.PhoenixDatabaseMetaData.SYSTEM_TASK_HISTORY_NAME_BYTES is a mutable array  At StatisticsUtil.java: At StatisticsUtil.java:[line 78]

org.apache.phoenix.jdbc.PhoenixDatabaseMetaData.SYSTEM_TASK_QUEUE_NAME_BYTES is a mutable array  At StatisticsUtil.java: At StatisticsUtil.java:[line 77]

org.apache.phoenix.query.ConnectionQueryServicesImpl.createOtherSystemTables(PhoenixConnection) may fail to clean up java.sql.Statement  Obligation to clean up resource created at ConnectionQueryServicesImpl.java:up java.sql.Statement  Obligation to clean up resource created at ConnectionQueryServicesImpl.java:[line 3449] is not discharged

Exception is caught when Exception is not thrown in org.apache.phoenix.query.ConnectionQueryServicesImpl.upgradeSystemTask(PhoenixConnection, Map)  At ConnectionQueryServicesImpl.java:is not thrown in org.apache.phoenix.query.ConnectionQueryServicesImpl.upgradeSystemTask(PhoenixConnection, Map)  At ConnectionQueryServicesImpl.java:[line 4217]

org.apache.phoenix.query.ConnectionlessQueryServicesImpl.init(String, Properties) may fail to clean up java.sql.Statement  Obligation to clean up resource created at ConnectionlessQueryServicesImpl.java:clean up java.sql.Statement  Obligation to clean up resource created at ConnectionlessQueryServicesImpl.java:[line 382] is not discharged

org.apache.phoenix.schema.task.Task.deleteTask(PhoenixConnection, PTable$TaskType, Timestamp, String, String, String, boolean, String) may fail to clean up java.sql.Statement on checked exception  Obligation to clean up resource created at Task.java:String, boolean, String) may fail to clean up java.sql.Statement on checked exception  Obligation to clean up resource created at Task.java:[line 191] is not discharged

Failed junit tests
phoenix.end2end.TableSnapshotReadsMapReduceIT

phoenix.end2end.SystemTablesCreationOnConnectionIT

phoenix.end2end.MigrateSystemTablesToSystemNamespaceIT

phoenix.end2end.UpsertSelectIT

Subsystem
Report/Notes

Docker
ClientAPI=1.40 ServerAPI=1.40 base: https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-970/1/artifact/yetus-general-check/output/Dockerfile

GITHUB PR
https://github.com/apache/phoenix/pull/970

Optional Tests
dupname asflicense javac javadoc unit spotbugs hbaseanti checkstyle compile

uname
Linux 79b808f9bbf9 4.15.0-112-generic #113-Ubuntu SMP Thu Jul 9 23:41:39 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev/phoenix-personality.sh

git revision
master / 5ff3edb

Default Java
Private Build-1.8.0_242-8u242-b08-0ubuntu3~16.04-b08

mvninstall
https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-970/1/artifact/yetus-general-check/output/branch-mvninstall-root.txt

mvninstall
https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-970/1/artifact/yetus-general-check/output/patch-mvninstall-root.txt

checkstyle
https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-970/1/artifact/yetus-general-check/output/diff-checkstyle-phoenix-core.txt

spotbugs
https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-970/1/artifact/yetus-general-check/output/new-spotbugs-phoenix-core.html

unit
https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-970/1/artifact/yetus-general-check/output/patch-unit-phoenix-core.txt

Test Results
https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-970/1/testReport/

Max. process+thread count
6361 (vs. ulimit of 30000)

modules
C: phoenix-core U: phoenix-core

Console output
https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-970/1/console

versions
git=2.7.4 maven=3.3.9 spotbugs=4.1.3

Powered by
Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
1m 26s
Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s
No case conflicting files found.

+1 :green_heart:
hbaseanti
0m  0s
Patch does not have any anti-patterns.

+1 :green_heart:
@author
0m  0s
The patch does not contain any @author tags.

+1 :green_heart:
test4tests
0m  0s
The patch appears to include 1 new or modified test files.

_ master Compile Tests _

+1 :green_heart:
mvninstall
14m 28s
master passed

+1 :green_heart:
compile
1m  2s
master passed

+1 :green_heart:
checkstyle
3m 15s
master passed

+1 :green_heart:
javadoc
0m 47s
master passed

+0 :ok:
spotbugs
3m 11s
phoenix-core in master has 962 extant spotbugs warnings.

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
9m 12s
the patch passed

+1 :green_heart:
compile
0m 56s
the patch passed

+1 :green_heart:
javac
0m 56s
the patch passed

-1 :x:
checkstyle
3m 16s
phoenix-core: The patch generated 182 new + 10076 unchanged - 40 fixed = 10258 total (was 10116)

+1 :green_heart:
whitespace
0m  0s
The patch has no whitespace issues.

+1 :green_heart:
javadoc
0m 47s
the patch passed

-1 :x:
spotbugs
3m 21s
phoenix-core generated 8 new + 958 unchanged - 4 fixed = 966 total (was 962)

_ Other Tests _

-1 :x:
unit
163m 57s
phoenix-core in the patch failed.

+1 :green_heart:
asflicense
0m 30s
The patch does not generate ASF License warnings.

208m 47s

Reason
Tests

FindBugs
module:phoenix-core

Initialization circularity between org.apache.phoenix.jdbc.PhoenixDatabaseMetaData and org.apache.phoenix.query.QueryConstants  At PhoenixDatabaseMetaData.java:org.apache.phoenix.query.QueryConstants  At PhoenixDatabaseMetaData.java:[lines 99-1871]

org.apache.phoenix.jdbc.PhoenixDatabaseMetaData.OLD_SYSTEM_TASK_NAME_BYTES is a mutable array  At PhoenixDatabaseMetaData.java: At PhoenixDatabaseMetaData.java:[line 394]

org.apache.phoenix.jdbc.PhoenixDatabaseMetaData.SYSTEM_TASK_HISTORY_NAME_BYTES is a mutable array  At StatisticsUtil.java: At StatisticsUtil.java:[line 78]

org.apache.phoenix.jdbc.PhoenixDatabaseMetaData.SYSTEM_TASK_QUEUE_NAME_BYTES is a mutable array  At StatisticsUtil.java: At StatisticsUtil.java:[line 77]

org.apache.phoenix.query.ConnectionQueryServicesImpl.createOtherSystemTables(PhoenixConnection) may fail to clean up java.sql.Statement  Obligation to clean up resource created at ConnectionQueryServicesImpl.java:up java.sql.Statement  Obligation to clean up resource created at ConnectionQueryServicesImpl.java:[line 3450] is not discharged

Exception is caught when Exception is not thrown in org.apache.phoenix.query.ConnectionQueryServicesImpl.upgradeSystemTask(PhoenixConnection, Map)  At ConnectionQueryServicesImpl.java:is not thrown in org.apache.phoenix.query.ConnectionQueryServicesImpl.upgradeSystemTask(PhoenixConnection, Map)  At ConnectionQueryServicesImpl.java:[line 4223]

org.apache.phoenix.query.ConnectionlessQueryServicesImpl.init(String, Properties) may fail to clean up java.sql.Statement  Obligation to clean up resource created at ConnectionlessQueryServicesImpl.java:clean up java.sql.Statement  Obligation to clean up resource created at ConnectionlessQueryServicesImpl.java:[line 382] is not discharged

org.apache.phoenix.schema.task.Task.deleteTask(PhoenixConnection, PTable$TaskType, Timestamp, String, String, String, boolean, String) may fail to clean up java.sql.Statement on checked exception  Obligation to clean up resource created at Task.java:String, boolean, String) may fail to clean up java.sql.Statement on checked exception  Obligation to clean up resource created at Task.java:[line 191] is not discharged

Subsystem
Report/Notes

Docker
ClientAPI=1.40 ServerAPI=1.40 base: https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-970/2/artifact/yetus-general-check/output/Dockerfile

GITHUB PR
https://github.com/apache/phoenix/pull/970

Optional Tests
dupname asflicense javac javadoc unit spotbugs hbaseanti checkstyle compile

uname
Linux 7dc99e3fb6bf 4.15.0-112-generic #113-Ubuntu SMP Thu Jul 9 23:41:39 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev/phoenix-personality.sh

git revision
master / 243ac64

Default Java
Private Build-1.8.0_242-8u242-b08-0ubuntu3~16.04-b08

checkstyle
https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-970/2/artifact/yetus-general-check/output/diff-checkstyle-phoenix-core.txt

spotbugs
https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-970/2/artifact/yetus-general-check/output/new-spotbugs-phoenix-core.html

unit
https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-970/2/artifact/yetus-general-check/output/patch-unit-phoenix-core.txt

Test Results
https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-970/2/testReport/

Max. process+thread count
6245 (vs. ulimit of 30000)

modules
C: phoenix-core U: phoenix-core

Console output
https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-970/2/console

versions
git=2.7.4 maven=3.3.9 spotbugs=4.1.3

Powered by
Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.


Add progress callback

Extracting a large .mbtiles file takes time. A callback should be added to enable progress indicator to be shown.

Issue solved in #4. Closing this.
Thanks to you @kengu for the changes.


chore: eslint configuration

add eslint configuration to codebase

Closing as we now have https://eslint.nuxt.com. Would love a PR to update to use that 🙏  (And sorry for the delay merging this one.)

@danielroe sure! that's a great idea (no worries at all)


accessibility of tooltip - WCAG 4.1.3

Users with disabilities have difficulties being aware of changes in the page, they will not read the tooltip as it is right now. The best way to improve this per WCAG criteria 4.1.3 is to add a role="status" to the tooltip-box.
It could be made as a new attribute if you don't always want to have the role.

I created an update of this project found here.
In it, I added role attribute with a default value of status, in this commit.
I also added aria support in this commit.
I build accessibility apps so I do plan to add more in the future. Please let me know if there is more that can be done. Thanks!


Notebook open geotiff

Note: checklist below only relevant if committing to the develop branch:
Proposed changes
Adding notebook on opening Geotiffs and NetCDFs into Xarray.
I was not sure exactly how much to show, so it is just the basics on opening these two file types
Checklist (replace [ ] with [x] to check off)

[x ] Notebook created using the DEA-notebooks template
[ x] Remove any unused Python packages from Load packages
[ x] Remove any unused/empty code cells
[ x] Remove any guidance cells (e.g. General advice)
[x ] Ensure that all code cells follow the PEP8 standard for code. The jupyterlab_code_formatter tool can be used to format code cells to a consistent style: select each code cell, then click Edit and then one of the Apply X Formatter options (YAPF or Black are recommended).
[x ] Include relevant tags in the final notebook cell (refer to the DEA Tags Index, and re-use tags if possible)
[x ] Clear all outputs, run notebook from start to finish, and save the notebook in the state where all cells have been sequentially evaluated
[x ] Test notebook on both the NCI and DEA Sandbox (flag if not working as part of PR and ask for help to solve if needed)
[x ] If applicable, update the Notebook currently compatible with the NCI|DEA Sandbox environment only line below the notebook title to reflect the environments the notebook is compatible with

Yep, something weird happened but it shouldn't be too hard to fix. @Kooie-cate can you try:

Copy the "Open_Geotiff_and_NetCDF_as_xarray.ipynb" somewhere safe (outside the repo)
git checkout --track origin/develop
git pull
`git checkout -b notebook-Open_geotiff_v2 develop
Move the "Open_Geotiff_and_NetCDF_as_xarray.ipynb" back into the Frequently_used_code directory
git commit -am 'Add open geotiff notebook to clean directory'
git push --set-upstream origin notebook-Open_geotiff_v2

That should get the new notebook on a clean branch. You'll then need to do a new pull request (we can close this issue when you do)

I think I got into a big mess with local git, I though I fixed it? but I don't understand how it had so many changes. Thanks @robbibt your instructions to fix it where very clear and easy to follow.


How to use it with webpack postcss loader

Hi,
Could you tell me how to configure webpack's postcss plugin to use postcss-include?
Here is my config:
/*
 * Webpack development server configuration
 *
 * This file is set up for serving the webpak-dev-server, which will watch for changes and recompile as required if
 * the subfolder /webpack-dev-server/ is visited. Visiting the root will not automatically reload.
 */
'use strict';
var webpack = require('webpack');
var webpackPostcssTools = require('webpack-postcss-tools');
var ExtractTextPlugin = require("extract-text-webpack-plugin");
var atcss = require('atcss');

var map = webpackPostcssTools.makeVarMap('src/styles/app.css');

module.exports = {

    output: {
        filename: 'app.js',
        publicPath: '/assets/',
        path: __dirname + '/assets'
    },

    cache: true,
    debug: true,
    devtool: false,
    entry: [
        'webpack/hot/only-dev-server',
        './src/scripts/app.js'
    ],

    stats: {
        colors: true,
        reasons: true
    },

    resolve: {
        packageMains: [
            'webpack',
            'browser',
            'web',
            'browserify',
            ['jam', 'main'],
            'style',
            'main'
        ],
        extensions: ['', '.js'],
        modulesDirectories: ['node_modules', 'bower_components', 'web_modules']
    },
    module: {
        preLoaders: [{
            test: '\\.js$',
            exclude: 'node_modules',
            loader: 'jshint'
        }],
        loaders: [{
            test: /\.js$/,
            loader: 'react-hot!jsx-loader?harmony'
        }, {
            test: /\.styl/,
            loader: 'style-loader!css-loader!stylus-loader'
        }, {
            test: /\.css$/,
            loader: ExtractTextPlugin.extract("style",
                "css?importLoaders=1!postcss"
            )
            //loader: 'style!css?importLoaders=1!postcss'
        }, {
            test: /\.(png|jpg)$/,
            loader: 'url-loader?limit=8192'
        }, {
            test: /\.woff(\?v=[0-9]\.[0-9]\.[0-9])?$/,
            loader: "url-loader?limit=10000&minetype=application/font-woff"
        }, {
            test: /\.(ttf|eot|svg)(\?v=[0-9]\.[0-9]\.[0-9])?$/,
            loader: "file-loader"
        }]
    },

    postcss: [
        webpackPostcssTools.prependTildesToImports,

        require('postcss-custom-properties')({
            variables: map.vars
        }),

        require('postcss-custom-media')({
            extensions: map.media
        }),

        require('postcss-calc'),
        require('postcss-include')
    ],

    plugins: [
        new webpack.HotModuleReplacementPlugin(),
        new webpack.NoErrorsPlugin(),
        new ExtractTextPlugin("app.css", {allChunks: true})
    ]

};

Although it does not throw any error in the result css I can see the definition of base classes and such. Like this:
.base-1 {
    /*
     * @base
     */
    color: red;
}

.base-2 {
    /*
     * @base
     */
    padding: 10px;
}

.foo {
    /*
     * @include .base-1, .base-2
     */
    font-size: 12px;
}

postcss-include must get CSS string as the first argument.
I have not used to use webpack. But i think, you should the below code.
require('postcss-include')(app.css)

It does not work, sorry. I don't know webpack well neither but with a simple console.log() I figured out it gets the css but does not write out anything if I insert a console.log before return root. I have to set up a new debugger then I can tell you more. By the way @include like @extend does delete the @base classes from the source css right? Could it be optional? I don't wanna lose my utility classes 'cause sometimes I need them.

@include like @extend does delete the @base classes from the source css right? Could it be optional? I don't wanna lose my utility classes 'cause sometimes I need them.

OK, I'll take this option into consideration :)
But I think, utility classes should be used when be in development. Using postcss-include, we can run pre-compile code in browsers because the metadata to transform CSS is in comment as annotations. https://github.com/morishitter/atcss#why-annotations

But I think, utility classes should be used when be in development.

I'm learning SUITCSS and it looks to me sometimes there is a place for a uitlity class in production according to SUITCSS style guide.
When to use a utility
Yeah, I've read that part in the atcss documentation but I could not get it, sorry for my dumbness. Why would I run pre-compile code in the browser? And when gets it compiled then? I imagined the way the plugin works is the same as any other css processor. You give a source then it gives an output which gets then served by the server to the client.

In my experience, the products using some utility classes are fragile. Because its name don't have the intent of the design. For example, utility class named as .pdt-10 means only padding-top: 10px. pdt-10 don't have the intent, "Why do you want padding-top: 10px to the contents?". But I think, utility classes are necessary in development. Because using utility classes, we can design in the browser fast.

Why would I run pre-compile code in the browser?

In development I want to use utility classes (the classes that don't named semantic). But in production, semantic class name is better. For example, class="btn-next" is better than class="btn btn-primary btn-large" if the button is to go to next page.

And when gets it compiled then?

Before release in production.

In v0.3.0 this probrem is solved.


ScrollIntoView surprisingly does scrollIntoViewIfNeeded

I spent a little while pondering why cdp.ScrollIntoView was not working as I expected from my knowledge of what it does in JavaScript. The behaviour I saw was that it was not scrolling when I was expecting it to - it was just behaving as though scroll was a no-op. I almost gave up trying to get this to work, believing it to just be broken. But I went hunting:
// ScrollIntoView scrolls the window to the first node matching the selector.
func ScrollIntoView(sel interface{}, opts ...QueryOption) Action {
// ...
        err = EvaluateAsDevTools(fmt.Sprintf(scrollIntoViewJS, nodes[0].FullXPath()), &pos).Do(ctxt, h)
// ...

Turns out is is documented on scrollIntoViewJS:
	// scrollIntoViewJS is a javascript snippet that scrolls the specified node
	// into the window's viewport (if needed), returning the actual window x/y
	// after execution.
	scrollIntoViewJS = `(function(a) {
		a[0].scrollIntoViewIfNeeded(true);
		return [window.scrollX, window.scrollY];
	})($x('%s'))`

I suggest two possible routes forward:

Simply do a[0].scrollIntoView. This should be pretty much backwards compatible, it just means that some things which didn't need the scroll now have the overhead of scrolling. At least bugs like the one I encountered would be averted. Anything which really needs IfNeeded for the performance benefit of avoiding unnecessary scrolls can introduce a new ScrollIntoViewIfNeeded which does it. I can't at this point think of circumstances which would immediately break with the additional scrolling.

If you don't want to change the existing function, at least it should be documented that ScrollIntoView is actually scrollIntoViewIfNeeded on the function. I like this change less, because it's still surprising. I also actually need scrollIntoView, not ...IfNeeded.

My use-case is to take a full-viewport screenshot, but only starting from a specific element. This element happens to be visible in the viewport prior to the scroll request (so ...IfNeeded does not scroll), but I need the elements which are below this element to be scrolled into the viewport too.

Originally we were using scrollIntoView -- it didn't work properly with what we were doing. It's just a javascript function call -- I would suggest writing a similar action to call chromedp.EvaluateAsDevTools()...


Compiling help: ApplicationMain.exe : fatal error LNK1120: 1 unresolved externals

[X] Windows
[ ] Linux
[ ] Mac
when i try to compile the game,  get the error, ApplicationMain.exe : fatal error LNK1120: 1 unresolved externals i have run the setup.bat looked around online and nothing helped
my haxe version is 4.2.5 if that helps

lime build windows -final

still getting the same error

make a clean build using the -clean flag
lime test $target -clean


simdjson in project complains about missing threads

Describe the bug
I installed simdjson (x64) and when I add the provided Cmake config to my CMakeLists.txt
 find_package(simdjson CONFIG REQUIRED)
    # Note: 2 target(s) were omitted.
    target_link_libraries(main PRIVATE simdjson::simdjson simdjson::simdjson-flags simdjson::simdjson-source simdjson::simdjson-headers)

I get the following error:
LINK : fatal error LNK1104: cannot open file 'Threads::Threads.lib'

I reported this in the vcpkg channel and @Neumann-A mentioned that it might be missing find_dependency(Threads) in simdjsonConfig.cmake
Environment

OS: Windows 10 64bit
Compiler:  Microsoft Visual Studio Community 2019 Version 16.7.4

To Reproduce
Steps to reproduce the behavior:

./vcpkg install simdjson (I have set the env var for x64)
Grab the cmake commands
Put in your CMakeLists.txt
Compile
See the LINK : fatal error LNK1104: cannot open file 'Threads::Threads.lib' in msvc ouput pane

Expected behavior
For it to compile without issues
Additional context
Switched to rapidjson so I there is no urgency :relaxed:

Same issue I posted on simdjson repository: https://github.com/simdjson/simdjson/issues/1375

Same issue I posted on simdjson repository: https://github.com/simdjson/simdjson/issues/1375

@AntonioCS @Journeyman-dev ,Today I will submit a solution to pr

@AntonioCS @Journeyman-dev ,Today I will submit a solution to pr

The simdjson library definitively support Microsoft Visual Studio Community 2019 under a 64-bit Windows.
Here is the relevant code in our CMake build...
option(SIMDJSON_ENABLE_THREADS "Link with thread support" ON)
if(SIMDJSON_ENABLE_THREADS)
  set(CMAKE_THREAD_PREFER_PTHREAD TRUE)
  set(THREADS_PREFER_PTHREAD_FLAG TRUE)
  find_package(Threads REQUIRED)
  target_link_libraries(simdjson-flags INTERFACE Threads::Threads)
  target_link_libraries(simdjson-flags INTERFACE ${CMAKE_THREAD_LIBS_INIT})
  target_compile_options(simdjson-flags INTERFACE ${CMAKE_THREAD_LIBS_INIT})
  target_compile_definitions(simdjson-flags INTERFACE SIMDJSON_THREADS_ENABLED=1) # This will be set in the code automatically.
endif()

The simdjson library definitively support Microsoft Visual Studio Community 2019 under a 64-bit Windows.
Here is the relevant code in our CMake build...
option(SIMDJSON_ENABLE_THREADS "Link with thread support" ON)
if(SIMDJSON_ENABLE_THREADS)
  set(CMAKE_THREAD_PREFER_PTHREAD TRUE)
  set(THREADS_PREFER_PTHREAD_FLAG TRUE)
  find_package(Threads REQUIRED)
  target_link_libraries(simdjson-flags INTERFACE Threads::Threads)
  target_link_libraries(simdjson-flags INTERFACE ${CMAKE_THREAD_LIBS_INIT})
  target_compile_options(simdjson-flags INTERFACE ${CMAKE_THREAD_LIBS_INIT})
  target_compile_definitions(simdjson-flags INTERFACE SIMDJSON_THREADS_ENABLED=1) # This will be set in the code automatically.
endif()

Ping @myd7349

Ping @myd7349

@JonLiu1993 Posted an issue downstream regarding the port.
https://github.com/simdjson/simdjson/issues/1383
It seems that provided CMake instructions are incorrect.
The following works for me:
cmake_minimum_required(VERSION 3.0)
project(test)

add_executable(test test.cpp)

find_package(simdjson CONFIG REQUIRED)
find_package(Threads REQUIRED)
target_link_libraries(test PRIVATE simdjson::simdjson)

The provided instructions do not make sense to me. For example, simdjson::simdjson-source does not get installed (nor should it), so I do not see how you could establish a dependency with this target.

@JonLiu1993 Posted an issue downstream regarding the port.
https://github.com/simdjson/simdjson/issues/1383
It seems that provided CMake instructions are incorrect.
The following works for me:
cmake_minimum_required(VERSION 3.0)
project(test)

add_executable(test test.cpp)

find_package(simdjson CONFIG REQUIRED)
find_package(Threads REQUIRED)
target_link_libraries(test PRIVATE simdjson::simdjson)

The provided instructions do not make sense to me. For example, simdjson::simdjson-source does not get installed (nor should it), so I do not see how you could establish a dependency with this target.

@lemire That is an automatically generated usage extracted from the provided targets in the simdjson-config.cmake. As it seems simdjson exports a lot of targets which are unnecessary to be exported or even unuseable if exported. The only reason they are getting exported is because of the misuse of targets in simdjson and cmake giving and error otherwise.

@lemire That is an automatically generated usage extracted from the provided targets in the simdjson-config.cmake. As it seems simdjson exports a lot of targets which are unnecessary to be exported or even unuseable if exported. The only reason they are getting exported is because of the misuse of targets in simdjson and cmake giving and error otherwise.

@Neumann-A Ah. Let us see what we can do about that.

@Neumann-A Ah. Let us see what we can do about that.

I am confident that we will resolve whatever we have to resolve upstream with release 0.8.

I am confident that we will resolve whatever we have to resolve upstream with release 0.8.

I am confident that we will resolve whatever we have to resolve upstream with release 0.8.

thank you for the help

I am confident that we will resolve whatever we have to resolve upstream with release 0.8.

thank you for the help

Release 0.8 is out. It should solve this issue.

Release 0.8 is out. It should solve this issue.

@lemire ,Thank you for your effort. I will update to the latest version for local testing. I will close the issue after success.

@lemire ,Thank you for your effort. I will update to the latest version for local testing. I will close the issue after success.

Solved by #15977


Invoke-DbaLogShipping-SpecificTime fixes #2648

The underlying function for the schedules already supported specific
times but the Invoke-DbaLogShipping didn't.
Added the option to backup, copy and restore schedule parameter.

Tested it myself and with another user.
Command:
Invoke-DbaLogShipping -SourceSqlInstance $SourceSqlInstance -DestinationSqlInstance $DestinationSqlInstance -Database $Database -BackupNetworkPath $BackupNetworkPath -BackupLocalPath $BackupLocalPath -BackupScheduleFrequencyType Daily  -CompressBackup -CopyScheduleFrequencyType Daily -GenerateFullBackup  -RestoreScheduleFrequencyInterval EveryDay  -CopyDestinationFolder $CopyDestinationFolder -RestoreScheduleFrequencyType Daily -RestoreScheduleFrequencySubdayType Time -RestoreScheduleStartTime 230000

DEEEEEEEEEEEEEEEEEEEEEEEEEEEEEEELIghtful!


Emphasize IOptionsMonitor in Options topic + updates

Fixes #9663
Fixes #9662
Internal Review Topic

Emphasizes IOptionsMonitor; downplays IOptions
Sample to 2.2
UE pass
xref API links

Actually looks like this isn't right, OptionsMonitor is a 2.0 thing, and it looks like the rename hit 1.1 and 1.0 as well

I don't think the docs for 1.0 and 1.1 should be updated at all, this should only target 2.0+

@ajcvickers as fyi, we are updating the docs to push IOptionsMonitor instead

Yes, I was going by the API browser as a guide to which interfaces/scenarios were made available for given versions ...
https://docs.microsoft.com/dotnet/api/microsoft.extensions.options.ioptionsmonitor-1
... at the bottom, where Applies to is listed on these.
You're saying that the information is incorrect there. I'll react on the update commit ...
IOptionsMonitor is 2.0+.
I guess that's the only one that didn't pan out.

yes this is confusing

lol :smile: ... and this type of thing could get a contractor into trouble :skull:. lol :smile:
The content is (seemingly) pretty good. I just need to fix up the versioning, so the update shouldn't take long. I hope to have it done by the EOD.

@HaoK I have the feeling that we need to dump the <2.1 coverage (i.e., take the 1.1 topic into a PDF until next June and set the topic up for 2.1 or later. Working with these versioning ranges given the re-implementation work is problematic (e.g., IOptionsMonitor changes at 2.0). We've been taking this approach on a few other topics with similar maintenance issues. Is it ok with you if this topic goes 2.1 or later?

Sure I imagine the most important versions for documentation are the LTS and current versions, so 2.1 and later seems a reasonable place to be.

:tada: Wonderful! ... I :heart: 🔪 those versioning ranges out.
I've sent the PDF off to be posted. As soon as that's online, I'll take this off of WIP. You can look now tho to see if there are any other changes to make.

One minor thing, Post-configuration is supported by everything, its not specific to IOptionsMonitor, its common to all 3 interfaces, just like validation, its something that the IOptionsFactory does automatically which is used by all 3.

@HaoK kk ... the link is live ...
https://webpifeed.blob.core.windows.net/webpifeed/Partners/Options_1.1.pdf
I moved the post-config remark out of the list into its own paragraph. Anything else before I call for docs team review?

@HaoK I think I have that post-configure bit under control. I'll ask Scott to help me put the final touches on this now. Let me know if you have anything else to address.

Cool looks good to me!

@scottaddie Note/reminder on the PDF situation: The 1.1 PDF notice should only appear for <=1.1; but with metadata versioning of >=2.1, it ends in a :boom: :cry:. Therefore, I think the plan is to leave the metadata versioning off until the PDF can be dropped next June, then the topic can get proper versioning. I think that's the plan. Let me know if we're going another way. btw- I'm only doing this on topics where the versioning is horri-terri-nasti-bad 😝. I prefer to leave the 1.x versioning in place, but it's a giant PITA on some of these topics at this point.


Create TelemetryWdmTrait definition

WDM telemetry trait

https://docs.google.com/document/d/1wyonkCJ1XZaPol3hegNKTiovOGKyebbR9y0ISJWutqM/edit?usp=sharing

TelemetryWdmTrait Design and Use.pdf

@pidarped adding to this review ( unable to add as reviewer since I am not owner)

Updated latest version based on review


A small question about ${i}.contig_IS_hits.txt in pseudoR_single-reference.sh (line 116 and line 97)

Hello, Joshua.
I noticed an error in my log file: cut: mapping_files/0TS10.orf_mapping.filtered.sam: No such file or directory (Same error can be found in error log in this issue: https://github.com/joshuakirsch/pseudoR/issues/9). I found it was caused by this command: cut -f3 mapping_files/${i}.orf_mapping.filtered.sam | sort | uniq > mapping_files/${i}.contig_IS_hits.txt (line 116 in pseudoR_single-reference.sh). Specifically, the file ${i}.orf_mapping.filtered.sam was not generated by the previous commands. Besides, the output file ${i}.contig_IS_hits.txt is the same as the output file of above command (line 97 in pseudoR_single-reference.sh).
By checking the following commands, I found the file IS_hits.txt does not seem to be used in subsequent calculations. So it maybe not affect the final results, but I am not sure.
Looking forward to your reply.

Hi,
Sorry about the very very very slow reply. Going back to the codebase and my error messages has also confirmed that this error message should not affect anything else. I will remove that line of code in a future update. Thanks for trying the program and hope your results were informative! Let me know if there is anything else I can help with.


Refactor UTF-8 decoding to enable use of SIMD.

Implements #20.
CHECKLIST

[x] basic SSE2 accelerated decoder (128-bit version; based on BobSteagall's utf_utils)
[ ] AVX decoder (256-bit?)
[ ] AVX2 decoer (256-bit)
[ ] AVX512 decoder?
[x] AVX512-BW accelerated decoder (512-bit version)
[x] only compile SIMD variants if target arch does support it.
[x] default to optimal implementation for target host
[ ] add accelerator::runtime_detect to chose best implementation at runtime (the result should be cached but also the cache should be invalidatable?)
[ ] improve API documentation on members

add accelerator::runtime_detect to chose best implementation at runtime

Can we use some third-party library for this (and in Contour too)?
E.g. https://github.com/google/cpu_features or https://github.com/anrieff/libcpuid.

add accelerator::runtime_detect to chose best implementation at runtime

Can we use some third-party library for this (and in Contour too)?

I think CPU feature testing is so simple and just a few lines of code with regards to what we need. There is no need to pull in another dependency. :)
p.s. i have the code already around.

outdated, and most importantly, superseeded by our new scan API.


Use @visallo/cytoscape fork

[x] joeferner
[ ] mwizeman joeybrk372 jharwig sfeng88

Fixed edge rendering issues with  curve-style=bezier and shape=roundRectangle
CHANGELOG
Fixed: Entities with 2 or more relationships between them would disappear when arranged in certain positions on the graph.

@joeybrk372 Is this temporary? Did we submit a PR?

@joeferner This is a temporary substitution until that rendering fix makes it to a released version. Cytoscape currently has a commit for this fix.


deal with deprecation of x/crypto/openpgp

Description
Per https://github.com/golang/go/issues/44226 golang.org/x/crypto/openpgp is now frozen and deprecated. We should look to switch to a different implementation to ensure we stay up to date as best we can with this technology.

I took another look at the situation and it hasn't really changed.
The library has gotten a new name and some work. But still depends on V3 signatures for signature packet parsing which the protonmail fork doesn't support.
https://github.com/cavaliergopher/rpm/blob/main/signature.go#L75
This is unfortunate as we can't parse ed25519 PGP keys which are getting more popular. Rekor would have problems parsing the kernel.org keyring as an example.

Golang recommends https://github.com/ProtonMail/go-crypto, which seems to be supported. Keybase has a fork too but it hasn't been updated in 2 years.
cc @lukehinds

@haydentherapper The situation hasn't changed around the https://github.com/cavaliergopher/rpm library which uses deprecated SignatureV3 packet, so we can't move to the ProtonMail library unless we also spend time fixing the underlying rpm library.
However I haven't been able to figure out if we can rip out the SignatureV3 packet support from the RPM library, I don't really understand if it's still used in the RPM ecosystem or not.

Spent a little bit of time digging into this issue:

RPM packages use the OpenPGP V3 signature format - See the documentation. While I know nothing about the format, I can't find anything that suggests there's any interest in moving to the V4 format. Maybe someone from RedHat would know more.
https://github.com/ProtonMail/go-crypto is backwards compatible, but unfortunately dropped support for V3 signatures awhile ago (because they're less secure, but I haven't dug into the spec to understand why). I filed https://github.com/ProtonMail/go-crypto/issues/164 asking if they'd be open to adding back support for verification of them. If so, then we'd be able to update the RPM Golang library and switch Rekor's library over too.
If not, then there's really no other options for V3 signature support, besides maintaining a fork of the above library with V3 support patched back in.
We should be able to add in support for V4 signatures and other key types in the pgp PKI type, requested in https://github.com/sigstore/rekor/issues/1438, by using ProtonMail/go-crypto. We can treat V3 as a special case and use the deprecated library. I'd recommend adding a deprecation warning for providing a V3 public key for any non-RPM Rekor type. (Another option would be to have a second PGP PKI type for V4+ keys, but that seemed more confusing to me).

Actually rpm supports V4 signatures for closer to twenty years, that's not an issue at all.
However until recently, a surprising amount of rpm packages has been signed as V3, presumably originally for compatibility but then just forgotten, and --force-v3-sigs becoming a part of a magic incantation folks just copy-paste around without stopping to think about it.
We (rpm upstream) ran into this in 2022 and where just as surprised as anybody, see https://bugzilla.redhat.com/show_bug.cgi?id=2141686#c23 and onwards for the long story.
I pushed for changing to V4 where-ever I could, the Fedora ecosystem is all V4 nowadays and Red Hat signing server switched sometime last year, so any new packages from there will be V4 signed. But of course there will be tonnes of V3 signed content around for years to come, which rpm will need to deal with, I don't know if that's the case for you guys.

Hey, thanks for looking into this! Ideally we would continue to support V3 signatures, but if Red Hat has moved over to V4, maybe now is the time to switch. It's looking to be quite difficult to support V3 anyways given what libraries are actively maintained in Go. There is a fork of protonmail/go-crypto that added back support for v3, though I'm not sure if the fork is trying to concurrently support both v3 and v4 or primarily v3.
We may also recommend to push this down to the client, and rather than deal with rpm support in Rekor, clients should parse rpms themselves and upload the signature as a hashedrekord type. This would be necessary when working with very large rpms anyways.


 },
    }),
  ],
});

How often does this bug happen?
Every time
What is the expected behavior?
these folders shouldn't be created
Swagger/OpenAPI file?
No response
Additional information
No response

Can you provide me a codesandbox? Tried it with your config but cannot reproduce the types folder being created.

right. I guess the folder was there from before I changed the options. sorry for that


Refer a cross-project task as a path rather than as a reference

Related: https://github.com/line/gradle-scripts/pull/61
Motivation:
When configure-on-demand is enabled, inter-project dependencies are
calculated correctly only when task dependency is specified as a task
path string. See:

https://docs.gradle.org/current/userguide/multi_project_builds.html#sec:configuration_on_demand

The task dependencies declared via task path are supported and cause
relevant projects to be configured.
Example: someTask.dependsOn(":someOtherProject:someOtherTask")

Modifications:

Use Task.path when referring to a task in a different project to
call Task.dependsOn()

Result:

Fixes a bug where a certain project is not configured when it has to
be, causing a weird compilation error due to unconfigured dependencies.

Hmm I was using configure on demand in large projects for a long time and never ran into this.
Anyways it seems that Gradle team is basically depreciating the feature because configuration is very fast now anyways
https://guides.gradle.org/migrating-build-logic-from-groovy-to-kotlin/#before_you_start_migrating
While it says shouldn't, I've found that kotlin scripts just don't work with it on at all.
So I'd recommend

Remove configure on demand
Enable parallel which has a bigger impact
Either go through the scripts to make sure they use Gradle's newer lazy APIs (register, named, etc) or migrate to Kotlin where the default DSL is lazy

Yeah they seem to want to remove it but couldn't because at some cases it's useful: https://github.com/gradle/gradle/issues/2868 Let me revisit this PR tomorrow.

Cool - just an anecdote but I've stopped using it in a couple of hugish projects to migrate the scripts to Kotlin and it's been just fine. Gradle's way faster than four years ago :D

Gradle's newer lazy APIs (register, named, etc)

Never knew about them. Do you know any good resources about this? :bow:

Updated to use evaluationDependsOn() instead of error-prone Task.path


Ask for supplementary material

Hello and thank you for your great work. In your original article you said "More details about the training dataset, training protocols, and additional visual results are shown in the supplementary material", where can I download the supplementary material?
I would be grateful for your reply!

where can I download the supplementary material?


merge package from arlac77/npm-package-template

package.json

chore(package<EMAIL_ADDRESS>

Codecov Report

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

@@           Coverage Diff           @@
##           master     #269   +/-   ##
=======================================
  Coverage   71.42%   71.42%
=======================================
  Files           1        1
  Lines          49       49
=======================================
  Hits           35       35
  Misses         14       14

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 ac39a71...114080e. Read the comment docs.

Coverage remained the same at 70.588% when pulling 114080eb80ee56ce5ec323d329aecbbaa16f24ba on template-sync-1 into ac39a71d87d7a08aa7f4e88e8bdf340ac3e7bf3c on master.

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


DOTNET_CLI_UI_LANGUAGE not affecting the resource key TestRunningSummary

Description
When setting the env variable DOTNET_CLI_UI_LANGUAGE the output from dotnet test is translated to the expected language except for the string represented by the TestRunningSummary which remains in English.
For reference, the TestRunningSummary resource I'm refering to is this one (for German): https://github.com/microsoft/vstest/blob/main/src/Microsoft.TestPlatform.Build/Resources/xlf/Resources.de.xlf#L15-L18
Steps to reproduce
Set the environment variable DOTNET_CLI_UI_LANGUAGE to something other than your system setting(I'm running on EN as default). In this example I'm using powershell and setting:
1: $Env:DOTNET_CLI_UI_LANGUAGE="de"
2: Execute the command dotnet test -t -v=q
Expected behavior
Testlauf für "D:\path\to\some\project\nunitNet5\bin\Debug\net5.0\NunitTests.dll" (.NETCoreApp,Version=v5.0)
Microsoft (R) Testausführungs-Befehlszeilentool Version 16.8.0
Copyright (c) Microsoft Corporation. Alle Rechte vorbehalten.

Die folgenden Tests sind verfügbar:

Actual behavior
Test run for D:\path\to\some\project\nunitNet5\bin\Debug\net5.0\NunitTests.dll (.NETCoreApp,Version=v5.0)
Microsoft (R) Testausführungs-Befehlszeilentool Version 16.8.0
Copyright (c) Microsoft Corporation. Alle Rechte vorbehalten.

Die folgenden Tests sind verfügbar:

Diagnostic logs
Environment
Running on windows 10,
dotnet sdk 5.0.100

Hi,
That resource was not localized in version 16.8.0. I see that it works on 16.9 and upwards.

I upgraded my sdk to v5.0.400 which seems to come with version 16.11.0 but the string is still printed in EN (eg disregards the DOTNET_CLI_UI_LANGUAGE setting) or am I missing something else here?
Test run for D:\path\to\some\project\nunitNet5\bin\Debug\net5.0\NunitTests.dll (.NETCoreApp,Version=v5.0)
Microsoft (R) Testausführungs-Befehlszeilentool Version 16.11.0
Copyright (c) Microsoft Corporation. Alle Rechte vorbehalten.
Die folgenden Tests sind verfügbar:

Ah, sorry - I didn't isolate SDK installs. There are two issues here. The first issue was the unlocalized resource, and the second one is that TP didn't respect the value of that variable in our build task. The first issue is fixed in 16.9, and the second one is fixed in 17.0 previews. You can see the correct behavior in net6 preview runner.