Use variable length string for passwordSubstitute

The current implementation is painfully redundant. For example in the sample app, one could just simply use ••••• is a secret number as app:pref_summaryHasText without using String.format to insert a fixed String.

The substitute text's length is intentionally fix so that no one can guess the actual password by looking at the number of substitute characters when the user scrolls through the list.

I know. And you still have the option to do that after this change.

I understand that but IMO the direction is wrong. The default should be the fixed length thing and the variable length would have to be enabled by an explicit switch. This needs to be done in this fashion since most users won't care about the security aspects and dangers of having a hidden password displayed with its actual length.

Even in theoretical security, the length of password isn't considered secret in most settings (for example, see the security definition for encryption scheme). It's not really a big deal IMO.

The length technically limits the "guess space". In most cases this would mean something like if the length is 4, it's probably the year of birth. Webpages usually do the same: showing a fixed length hidden pass instead of the actual length when asking the user to review the entered data before submitting it. This is because the users do not care about the password directly since they only see asterisks (or other non-letter chars) which has no meaning other than to display the field as "there was a password here". I think showing the actual length of the entered value does not add any value to the user but might reveal the pass to the attacker by just seeing the size of it. The math also supports this: testing all combinations starting with 1 letter through 6 (just an example) takes significantly more time than testing for only 6-long passwords. So either add an argument / call that explicitly enables this less-secure behavior or it won't be merged.

Talking is cheap.
Assuming alphabet size sigma, testing all combinations starting with 1 to n takes:
sigma + sigma^2 + sigma^3 + ... + sigma^n = (sigma^n - 1) * sigma / (sigma - 1)
which has entropy n*log(sigma) - O(1/sigma).
Testing all combinations with length n takes:
sigma^n
which has entropy n*log(sigma).
You see the difference in entropy is just at most a constant if n is sufficiently large. If n is small, the user should blame himself for using a weak password.

Still, it won't get merged until the default behavior is changed. If it's not a big deal to display the actual length then it shouldn't be a big deal to not display it.

In plain English:

If the password is weak, a brute-force attacker can attack it without length information;
If the password is strong, giving a brute-force attacker the length information doesn't make him suddenly able to attack something he can't attack before.

For the latter comment, I agree. From the security perspective, it really makes no difference. But there are two drawbacks for the current approach:

passwordSubstitute is painfully redundant;
The user interface isn't intuitive to user either.

Therefore I suggest either:

Merge this change or
Don't display a fake 5-digit password at all, use an empty string instead.

I think a fix library should not mess with this, and it does not have an agreed behavior as well. Simply let develeopers extend EditTextPreference and override getSummary() to obtain the behavior they see fit will be good enough. Or at least, please leave the default behavior unchanged and offer this new functionality in a different subclass.

Superseded by the pref_summaryPasswordSubstituteLength attribute which can set either a fixed length or set the behavior to dynamic length of the substitute password in the summary. Also, the EditTextPreference has been reverted to its original state and the new behavior is implemented by AutoSummaryEditTextPreference available in the same package starting from v<IP_ADDRESS>.

Sounds good. Why do we make that optional though? We can insert % by using %%.

Also standard ListPreference uses this behavior so I think it wouldn't cause too much harm.

Why do you want to insert %?
I personally do not like the ListPreference-like summary handling because there's no way to display a summary when there's no value set. Using a different attribute is not a big burden yet it keeps the possibility to choose either this way or the ListPreference one.


Sync two files with upstream

Update traffic management, change translations of some sentences
Delete some blocks
Fill a link anchor
Redirect some references to Chinese content

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

Needs approval from an approver in each of these files:

content_zh/OWNERS

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


Documentation and manifest mismatch

Regarding this page: https://projectcalico.docs.tigera.io/reference/installation/api
It says, regarding the APIServer: "At most one instance of this resource is supported. It must be named “tigera-secure”."
However on this page: https://projectcalico.docs.tigera.io/getting-started/kubernetes/self-managed-onprem/onpremises
Step 2 says to get https://raw.githubusercontent.com/projectcalico/calico/v3.24.1/manifests/custom-resources.yaml
It's a little vague, but then it says "If you wish to customize the Calico install, customize the downloaded custom-resources.yaml manifest locally."
The custom-resources.yaml has:
# This section configures the Calico API server.
# For more information, see: https://projectcalico.docs.tigera.io/master/reference/installation/api#operator.tigera.io/v1.APIServer
apiVersion: operator.tigera.io/v1
kind: APIServer
metadata:
  name: default
spec: {}

Notice the APIServer name is at variance with the reference guidance. So either the manifest is wrong, or the instructions are wrong or  (this would be odd) the instructions are right and you want to make people patch the manifest...
Thanks.

Duplicate of #6730

Upon closer inspection this isn't actually a duplicate.

This looks like a documentation bug. Looking at the code it seems that tigera-secure is what the APIServer used to be called, and when we transitioned to instead naming it default we simply forgot to update this reference doc you've linked.

This should fix it: https://github.com/tigera/operator/pull/2184


Master to Live

Docs Build status updates of commit 9acb958:
:clock10: Full build: the average full build time is 17 min(s) 36 sec(s), this is based on the last 1000 build(s) for this repository.

Docs Build status updates of commit 9acb958:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

exchange/docs-conceptual/app-only-auth-powershell-v2.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/docs-conceptual/connect-to-exchange-online-powershell.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/docs-conceptual/exchange-online-powershell-v2.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/Get-MailboxAuditBypassAssociation.md
:white_check_mark:Succeeded
View (exchange-ps)

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel


ci: fix coveralls parallelism

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 84.316%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
2059

Relevant Lines:
2442

💛  - Coveralls


[FEATURE] Show Less Data use  pagination or use react-infinite-scroll-component package

Description
Here we can show the limited data let's say 15-20 we can use pagination or use react-infinite-scroll-component package
Screenshots

Additional information
No response

Thank you for raising the issue. There is already an issue for search pagination already, so I will close this issue as it is a duplicate https://github.com/EddieHubCommunity/LinkFree/issues/7599


[KOGITO-4940] move KieBase creation in generated code into a static f…

…ield

The Linux check has failed. Please check the logs.

The Linux check has failed. Please check the logs.

The Linux check has failed. Please check the logs.

The Linux check has failed. Please check the logs.

Jenkins run native

The Linux check has failed. Please check the logs.

The Native check has failed. Please check the logs.

jenkins retest this

Jenkins run native

The Native check has failed. Please check the logs.

The Linux check has failed. Please check the logs.


Queue using circular array

🚀 Queue implementation using circular array in C++.
I want to implement Queue and its features like enqueue, dequeue, isEmpty, isFull etc. using Circular array in C++.
Have you read the Contributing Guidelines on Pull Requests?
YES
Motivation
I want to do it under GSSoc'21.
Pitch
This will help the user to know the implementation of queue using limited sized circular array, how classes are implemented and features of Queue.

@HarshCasper


Labels "opaque" blending does not persist after click to canvas

🐛 Bug
The labels "opaque" blending mode does not stay opaque after the user clicks on the 3D canvas.
To Reproduce
Steps to reproduce the behavior:

Set up some example data with labels

import napari
from skimage.filters import gaussian, threshold_otsu
from skimage.measure import label

# Open demo data
viewer = napari.Viewer()
viewer.open_sample('napari', 'cells3d')

# Create labels
nuclei = viewer.layers[-1].data
smoothed = gaussian(nuclei, sigma=2)
thresh = threshold_otsu(smoothed)
mask = smoothed > thresh
labels = label(mask)
viewer.add_labels(labels)

# Switch to 3d view mode and turn on opaque blending mode
viewer.dims.ndisplay = 3
viewer.layers[-1].blending = 'opaque'

Click on the napari canvas and observe the change to the label blending. The labels display becomes slightly more transparent.

Here's a gif, switching between the view before & after clicking the canvas:

Expected behavior

I expected that clicking on the canvas should not affect the transparency/display of a labels layer.
Environment

Please copy and paste the information at napari info option in help menubar here:

napari: 0.4.16
Platform: macOS-12.3.1-arm64-arm-64bit
System: MacOS 12.3.1
Python: 3.9.13 | packaged by conda-forge | (main, May 27 2022, 17:00:33) [Clang 13.0.1 ]
Qt: 5.15.2
PyQt5: 5.15.4
NumPy: 1.22.4
SciPy: 1.9.1
Dask: 2022.9.1
VisPy: 0.10.0
OpenGL:

GL version: 2.1 Metal - 76.3
MAX_TEXTURE_SIZE: 16384

Screens:

screen 1: resolution 1440x900, scale 2.0
screen 2: resolution 1920x1080, scale 1.0

Plugins:

console: 0.0.6

empanada-napari: 0.2.3

napari-clusters-plotter: 0.5.2

napari-nD-annotator: 0.0.8

napari-svg: 0.1.6

napari-tools-menu: 0.1.17

napari_skimage_regionprops1: 0.5.5

napari_skimage_regionprops2: 0.5.5

scikit-image: 0.4.16

zarpaint: 0.1.1.dev27+gfc48307

Any other relevant information:

If you query the blending mode using viewer.layers[-1].blending it still reports using the "opaque" blending mode, even after the display change caused by clicking on the canvas.
The change in the labels blending mode looks similar to, but is not quite the same as the "additive" blending mode.
Additional context

Could you chcek if it still happens in napari 0.4.17rc?

I can reproduce this in 0.4.16, but on my end it's fixed in 0.4.17.
@GenevieveBuckley Can you double check with the new live release, 0.4.17?

It doesn't seem possible to upgrade napari in the environment I'm actually using (too many conflicts), but I've just created a new empty test environment and can confirm that the "opaque" setting is actually opaque in 0.4.17, unlike the screenshots above from 0.4.16


samples: net: http_server: Send final chunk

The final ending chunk was not sent after the request was served.
This caused the remote client to constantly send the same request
to the server.
Fixes #6356
Signed-off-by: Jukka Rissanen<EMAIL_ADDRESS>

Codecov Report

Merging #6786 into master will increase coverage by <.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #6786      +/-   ##
==========================================
+ Coverage   54.64%   54.64%   +<.01%
==========================================
  Files         447      447
  Lines       42587    42587
  Branches     8040     8040
==========================================
+ Hits        23272    23273       +1
+ Misses      16084    16083       -1
  Partials     3231     3231

Impacted Files
Coverage Δ

kernel/posix/pthread.c
70% <0%> (+0.5%)
:arrow_up:

Continue to review full report at Codecov.

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

@jukkar this fails in a different way.  Running:
wget -nv -O - http://<IP_ADDRESS>/

Zephyr HTTP server sample

It works!

but wget never completes.
tcpdump shows:
21:21:39.731378 IP phoenix.57690 > <IP_ADDRESS>.http: Flags [S], seq<PHONE_NUMBER>, win 29200, options [mss 1460,sackOK,TS val<PHONE_NUMBER> ecr 0,nop,wscale 7], length 0
21:21:39.742491 IP <IP_ADDRESS>.http > phoenix.57690: Flags [S.], seq<PHONE_NUMBER>, ack<PHONE_NUMBER>, win 1280, length 0
21:21:39.742538 IP phoenix.57690 > <IP_ADDRESS>.http: Flags [.], ack 1, win 29200, length 0
21:21:39.742681 IP phoenix.57690 > <IP_ADDRESS>.http: Flags [P.], seq 1:137, ack 1, win 29200, length 136: HTTP: GET / HTTP/1.1
21:21:39.863181 IP <IP_ADDRESS>.http > phoenix.57690: Flags [P.], seq 1:250, ack 1, win 1280, length 249: HTTP: HTTP/1.1 200 OK
21:21:39.863201 IP phoenix.57690 > <IP_ADDRESS>.http: Flags [.], ack 250, win 30016, length 0
21:21:39.863332 IP <IP_ADDRESS>.http > phoenix.57690: Flags [F.], seq 250, ack 1, win 1280, length 0
21:21:39.863446 IP phoenix.57690 > <IP_ADDRESS>.http: Flags [F.], seq 137, ack 251, win 30016, length 0
21:21:39.879806 IP <IP_ADDRESS>.http > phoenix.57690: Flags [.], ack 137, win 1280, length 0
21:21:39.879924 IP <IP_ADDRESS>.http > phoenix.57690: Flags [.], ack 138, win 1280, length 0
21:21:40.864037 IP phoenix.57692 > <IP_ADDRESS>.http: Flags [S], seq<PHONE_NUMBER>, win 29200, options [mss 1460,sackOK,TS val<PHONE_NUMBER> ecr 0,nop,wscale 7], length 0
21:21:40.875199 IP <IP_ADDRESS>.http > phoenix.57692: Flags [S.], seq<PHONE_NUMBER>, ack<PHONE_NUMBER>, win 1280, length 0
21:21:40.875279 IP phoenix.57692 > <IP_ADDRESS>.http: Flags [.], ack 1, win 29200, length 0
21:21:40.875365 IP phoenix.57692 > <IP_ADDRESS>.http: Flags [P.], seq 1:156, ack 1, win 29200, length 155: HTTP: GET / HTTP/1.1
21:21:40.996030 IP <IP_ADDRESS>.http > phoenix.57692: Flags [P.], seq 1:250, ack 1, win 1280, length 249: HTTP: HTTP/1.1 200 OK
21:21:40.996057 IP phoenix.57692 > <IP_ADDRESS>.http: Flags [.], ack 250, win 30016, length 0
21:21:40.996198 IP <IP_ADDRESS>.http > phoenix.57692: Flags [F.], seq 250, ack 1, win 1280, length 0
21:21:40.996236 IP phoenix.57692 > <IP_ADDRESS>.http: Flags [F.], seq 156, ack 251, win 30016, length 0
21:21:41.012635 IP <IP_ADDRESS>.http > phoenix.57692: Flags [.], ack 156, win 1280, length 0
21:21:41.012764 IP <IP_ADDRESS>.http > phoenix.57692: Flags [.], ack 157, win 1280, length 0

but wget never completes.

I do not understand what you mean by this. When I try this, wget quits just fine.
wget -nv -O - http://<IP_ADDRESS>

    <meta charset="UTF-8">
    <title>Zephyr HTTP server sample</title>

It works!

2018-03-27 14:06:26 URL:http://<IP_ADDRESS>/ [170] -> "-" [1]

Lets apply this. If there are additional issues we can fix them later.


Question: What are bookmarks?

When updating to version 2.7.0, my application broke because Contracts\FormatterInterface was changed. Of course, I would love to have the newest version of this package, so I want to update my implementation of the interface to this new signature.
The change is that FormatBoltResult has a BookmarkHolder $holder and I do not know what to do with it. Can I just ignore it?

I will answer the question what are bookmarks.
Bookmarks are used when Neo4j is running in cluster (multiple instances). When you execute query (cluster choose one instance to run it) and immediatelly execute another query while the cluster decide to use different instance, you can hit a problem when data was not synchronized (yet) between instances in cluster. From outside it looks like the data from first query was not processed. Bookmark solve this problem because with bookmark you are refering to changes executed in first query.
You can imagine bookmarks as git commit hash, it's like referring to moment in history.

I checked the readme and code about Formatters and it seems is up to you if you use it.
If you look at BasicFormatter the BookmarkHolder is not used.
https://github.com/neo4j-php/neo4j-php-client/blob/439a59282f9fcfc6799b24fe617bf1ebf716ccc0/src/Formatter/BasicFormatter.php#L59-L70
If you want to use it you can get some inspiration in OGMFormatter where is applied with finished callback.
https://github.com/neo4j-php/neo4j-php-client/blob/439a59282f9fcfc6799b24fe617bf1ebf716ccc0/src/Formatter/OGMFormatter.php#L78-L94
Propably @transistive will know more (because he is author) and you can wait for his answer. ;)

Thank you for your response, I now know what to do!

Good stuff! The Basic Formatter should also do something with the bookmarks. Looks like an oversight from my side! I'll create an issue
@wgevaert, I'm thinking about redesigning the formatters in the next major release, as the parameters are getting a bit out of control. I'll create an issue for it later this week, so you can join the discussion if you want


Ctrl+click randomly stops working

Issue Type: Bug
I am working on a C/C++ project. After my project window is open for some time ctrl+click no longer goes to any variable/function definition, instead a blue "loading" bar appears on top of the editor forever. Closing the window and opening the project in a new one solves it. Sorry, I know this is really hard to reproduce but maybe you have some idea why it might happen..
VS Code version: Code 1.23.1 (d0182c3417d225529c6d5ad24b7572815d0de9ac, 2018-05-10T17:11:17.614Z)
OS version: Windows_NT x64 10.0.17134

System Info

Item
Value

CPUs
Intel(R) Core(TM) i5-6400 CPU @ 2.70GHz (4 x 2712)

GPU Status
2d_canvas: enabledflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onnative_gpu_memory_buffers: disabled_softwarerasterization: disabled_softwarevideo_decode: enabledvideo_encode: enabledvpx_decode: unavailable_softwarewebgl: enabledwebgl2: enabled

Memory (System)
7.87GB (2.20GB free)

Process Argv
C:\Program Files\Microsoft VS Code\Code.exe

Screen Reader
no

VM
0%

Extensions (2)

Extension
Author (truncated)
Version

language-x86-64-assembly
13x
2.2.11

cpptools
ms-
0.17.3

https://github.com/Microsoft/vscode-cpptools


fix: add missing container tags

I just built & published my image with the tool and noticed there are some differences to startingpoint with the generated tags

There's a commit hash tag, that's cool

It shouldn't need the -latest suffix, though. And another tag had latest as a prefix, hmm.

Major version tags are missing (38, 39)

Also version-date and version-commit

Latest date tags are named differently 20240122 (startingpoint) vs latest-20240122 (bluebuild)

Do we not want to include latest to just be more verbose? I always side with explicit over implicit

What branch do you build on? Cause I hardcoded it to look for the live branch while I was getting it thrown together for GitHub.

Major version tags are missing (38, 39)
OH! I think it's because I'm using the version that's defined in the recipe yaml. I guess we would need to add logic to pull info from the base image to determine the proper fedora version huh?

This seems to be tied to https://github.com/blue-build/github-action/issues/2

Major version tags are missing (38, 39)

OH! I think it's because I'm using the version that's defined in the recipe yaml. I guess we would need to add logic to pull info from the base image to determine the proper fedora version huh?

Yeah, the info should be in /etc/os-release, but I'm not sure how that translates to other distros.

What branch do you build on? Cause I hardcoded it to look for the live branch while I was getting it thrown together for GitHub.

I think for new templates we want to lose the live/template branch structure, since the new reusable github action will handle all the updates in features. I'm not sure where you hardcoded this,

Ok, I found the part where live is hardcoded. Didn't know the tags were generated in template.rs. I guess you should change it to support either live or main branch to support both the old and new plans.

So with Gitlab CI, they have an env var to tell your pipeline what branch is the default branch. If I can find that var for Github, that would be a good replacement

GitHub allows for environment variables. Go to settings, secrets/keys and it should be an environment option

Forgot to respond to this

Do we not want to include latest to just be more verbose? I always side with explicit over implicit.

I was just thinking abt how startingpoint does things and keeping parity. Another argument is: latest should always mean latest version and date-latest might be contradicting. And of course the tags always pull the latest image with that tag, that's the point. :39 is the latest version of 39, so why wouldn't :20240101 be the latest version from that date? If one wants, they could specify a different version though, like :38-20240101. But the point of :latest is to roll over to the next version automatically, so I don't see the point of :date-latest.

GitHub allows for environment variables. Go to settings, secrets/keys and it should be an environment option

I was talking about the built in variables set by GitHub actions. I can look it up later

From conversations in the Discord:

check if branch is live or main
put image_version on the end of tags
Add a timestamp tag
Detect actual OS version and use that in place of recipe's image_version

What should the the timestamp be formatted as?

What should the the timestamp be formatted as?

Same as now, YYYYMMDD.


[BiliIntl/bilibili.tv] ASS subtitles unable to download for anime, and additional encoding error

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
Indonesia, Malaysia, Vietnam, Thailand
Provide a description that is worded well enough to be understood
When user added --sub-langs all --sub-format 'ass,srt' --embed-subs parameter to yt-dlp and the media uses ASS subtitle, it seems that yt-dlp unable to save the file as the extractor assumes the subtitle were in raw JSON format.

For additional context, I've also tried to change --sub-format from ass/srt, but seems failed to do.
Then, for some older titles, it seems that the extractor also tried to decode UTF8-with-BOM, which made the subtitle extraction failed too.
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
[debug] Command-line config: ['-vU', '-f', 'bv*[height=1080]+ba,codec:hev1', '--cookies', '~\\Downloads\\bilibili.tv_cookies.txt', '--sub-langs', 'all', '--sub-format', 'ass,srt', '--embed-subs', '--embed-chapters', '--embed-metadata', '--merge-output-format', 'mkv', '--ffmpeg-location', 'C:\\Users\\nattadasu\\AppData\\Local\\Programs\\youtube-dl-gui\\resources\\app.asar.unpacked\\binaries\\', 'https://www.bilibili.tv/id/play/2098140']
[debug] Encodings: locale cp65001, fs utf-8, pref cp65001, out utf-8, error utf-8, screen utf-8
[debug] yt-dlp version<EMAIL_ADDRESS>from yt-dlp/yt-dlp [f10589e34] (pip)
[debug] Python 3.11.8 (CPython AMD64 64bit) - Windows-10-10.0.22631-SP0 (OpenSSL 3.0.13 30 Jan 2024)
[debug] exe versions: ffmpeg 4.4.1-full_build-www.gyan.dev (setts), ffprobe 4.4.1-full_build-www.gyan.dev
[debug] Optional libraries: Cryptodome-3.19.0, brotli-1.1.0, certifi-2023.11.17, mutagen-1.47.0, requests-2.31.0, sqlite3-3.43.1, urllib3-2.1.0, websockets-12.0
[debug] Proxy map: {}
[debug] Request Handlers: urllib, requests, websockets
[debug] Loaded 1798 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)
[biliIntl:series] Extracting URL: https://www.bilibili.tv/id/play/2098140
[biliIntl:series] 2098140: Downloading JSON metadata
[download] Downloading playlist: The Foolish Angel Dances with the Devil
[biliIntl:series] 2098140: Downloading JSON metadata
[biliIntl:series] Playlist The Foolish Angel Dances with the Devil: Downloading 7 items of 7
[download] Downloading item 1 of 7
[BiliIntl] Extracting URL: https://www.bilibili.tv/en/play/2098140/13083766#__youtubedl_smuggle=%7B%22title%22%3A+%22E1+-+Dance+with+the+devil%22%2C+%22thumbnail%22%3A+%22https%3A%2F%2Fpic.bstarstatic.com%2Fogv%2Ff4a8ac4dd3486ed3db01064e6d1352e472c8d23c.png%22%2C+%22episode_number%22%3A+1%7D
[BiliIntl] 13083766: Downloading JSON metadata
[BiliIntl] 13083766: Downloading video formats
[BiliIntl] 13083766: Downloading subtitles list
[BiliIntl] 13083766: Downloading subtitles for English
WARNING: [BiliIntl] 13083766: Failed to parse JSON: Unexpected UTF-8 BOM (decode using utf-8-sig): line 1 column 1 (char 0)
[BiliIntl] 13083766: Downloading subtitles for ภาษาไทย
WARNING: [BiliIntl] 13083766: Failed to parse JSON: Unexpected UTF-8 BOM (decode using utf-8-sig): line 1 column 1 (char 0)
[BiliIntl] 13083766: Downloading subtitles for Bahasa Indonesia
WARNING: [BiliIntl] 13083766: Failed to parse JSON: Unexpected UTF-8 BOM (decode using utf-8-sig): line 1 column 1 (char 0)
[BiliIntl] 13083766: Downloading subtitles for 中文（简体）
[info] 13083766: Downloading subtitles: zh-Hans
WARNING: No subtitle format found matching "ass,srt" for language zh-Hans, using srt
[debug] Formats sorted by: hasvid, ie_pref, lang, quality, res, fps, hdr:12(7), vcodec:vp9.2(10), channels, acodec, size, br, asr, proto, vext, aext, hasaud, source, id
[info] 13083766: Downloading 1 format(s): 16+2
[info] Writing video subtitles to: E1 - Dance with the devil [13083766].zh-Hans.srt
[debug] Invoking http downloader on "https://upos-bstar1-mirrorakam.akamaized.net/iupxcodeboss/08/wj/n240105er73lt8bhzmy8g30hky2ewj08-1-261220110000.m4s?e=ig8euxZM2rNcNbdlhoNvNC8BqJIzNbfqXBvEqxTEto8BTrNvN0GvT90W5JZMkX_YN0MvXg8gNEV4NC8xNEV4N03eN0B5tZlqNxTEto8BTrNvNeZVuJ10Kj_g2UB02J0mN0B5tZlqNCNEto8BTrNvNC7MTX502C8f2jmMQJ6mqF2fka1mqx6gqj0eN0B599M=&uipk=5&nbs=1&deadline=1708950152&gen=playurlv2&os=akam&oi=3035704899&trid=be3d85beb65e48e0aa160dbabbb7d051i&mid=1625976544&platform=pc&upsig=0eaaf03a9dd9d9f9598c9aca78d81c0a&uparams=e,uipk,nbs,deadline,gen,os,oi,trid,mid,platform&hdnts=exp=1708950152~hmac=eabad0289668a2a9925d295bba6e72584b9179df3e083c478c4b18de5d7dcfaa&bvc=vod&nettype=0&orderid=0,2&logo=00000000&f=i_0_0"
[debug] File locking is not supported. Proceeding without locking
[download] Destination: E1 - Dance with the devil [13083766].f16.mp4
[download] 100% of  187.27MiB in 00:01:50 at 1.70MiB/s
[debug] Invoking http downloader on "https://upos-bstar1-mirrorakam.akamaized.net/iupxcodeboss/08/wj/n240105er73lt8bhzmy8g30hky2ewj08-1-2d1301000023.m4s?e=ig8euxZM2rNcNbdlhoNvNC8BqJIzNbfqXBvEqxTEto8BTrNvN0GvT90W5JZMkX_YN0MvXg8gNEV4NC8xNEV4N03eN0B5tZlqNxTEto8BTrNvNeZVuJ10Kj_g2UB02J0mN0B5tZlqNCNEto8BTrNvNC7MTX502C8f2jmMQJ6mqF2fka1mqx6gqj0eN0B599M=&uipk=5&nbs=1&deadline=1708950152&gen=playurlv2&os=akam&oi=3035704899&trid=a5dbc4d1045a4c14b0c05c35721a9313i&mid=1625976544&platform=pc&upsig=7d5b509e70708584f71728d5b1fbdc54&uparams=e,uipk,nbs,deadline,gen,os,oi,trid,mid,platform&hdnts=exp=1708950152~hmac=712bc3e582f3169595471795446034def746a1039881c1324510a608d9265bfc&bvc=vod&nettype=0&orderid=0,2&logo=00000000&f=i_0_0"
[download] Destination: E1 - Dance with the devil [13083766].f2.mp4
[download] 100% of   33.58MiB in 00:00:18 at 1.86MiB/s
[Merger] Merging formats into "E1 - Dance with the devil [13083766].mkv"
[debug] ffmpeg command line: "C:\Users\nattadasu\AppData\Local\Programs\youtube-dl-gui\resources\app.asar.unpacked\binaries\ffmpeg" -y -loglevel "repeat+info" -i "file:E1 - Dance with the devil [13083766].f16.mp4" -i "file:E1 - Dance with the devil [13083766].f2.mp4" -c copy -map "0:v:0" -map "1:a:0" -movflags "+faststart" "file:E1 - Dance with the devil [13083766].temp.mkv"
Deleting original file E1 - Dance with the devil [13083766].f2.mp4 (pass -k to keep)
Deleting original file E1 - Dance with the devil [13083766].f16.mp4 (pass -k to keep)
[EmbedSubtitle] Embedding subtitles in "E1 - Dance with the devil [13083766].mkv"
[debug] ffmpeg command line: "C:\Users\nattadasu\AppData\Local\Programs\youtube-dl-gui\resources\app.asar.unpacked\binaries\ffmpeg" -y -loglevel "repeat+info" -i "file:E1 - Dance with the devil [13083766].mkv" -i "file:E1 - Dance with the devil [13083766].zh-Hans.srt" -map 0 -dn -ignore_unknown -c copy -map "-0:s" -map "1:0" "-metadata:s:s:0" "language=zho" -movflags "+faststart" "file:E1 - Dance with the devil [13083766].temp.mkv"
Deleting original file E1 - Dance with the devil [13083766].zh-Hans.srt (pass -k to keep)
[Metadata] Adding metadata to "E1 - Dance with the devil [13083766].mkv"
[debug] ffmpeg command line: "C:\Users\nattadasu\AppData\Local\Programs\youtube-dl-gui\resources\app.asar.unpacked\binaries\ffmpeg" -y -loglevel "repeat+info" -i "file:E1 - Dance with the devil [13083766].mkv" -i "file:E1 - Dance with the devil [13083766].meta" -map 0 -dn -ignore_unknown -c copy -map_metadata 1 -write_id3v1 1 -metadata "title=E1 - Dance with the devil" -metadata "purl=https://www.bilibili.tv/en/play/2098140/13083766#__youtubedl_smuggle=%7B%22title%22%3A+%22E1+-+Dance+with+the+devil%22%2C+%22thumbnail%22%3A+%22https%3A%2F%2Fpic.bstarstatic.com%2Fogv%2Ff4a8ac4dd3486ed3db01064e6d1352e472c8d23c.png%22%2C+%22episode_number%22%3A+1%7D" -metadata "comment=https://www.bilibili.tv/en/play/2098140/13083766#__youtubedl_smuggle=%7B%22title%22%3A+%22E1+-+Dance+with+the+devil%22%2C+%22thumbnail%22%3A+%22https%3A%2F%2Fpic.bstarstatic.com%2Fogv%2Ff4a8ac4dd3486ed3db01064e6d1352e472c8d23c.png%22%2C+%22episode_number%22%3A+1%7D" -metadata "episode_id=Episode 1" -metadata "episode_sort=1" -movflags "+faststart" "file:E1 - Dance with the devil [13083766].temp.mkv"
[download] Downloading item 2 of 7
[BiliIntl] Extracting URL: https://www.bilibili.tv/en/play/2098140/13083848#__youtubedl_smuggle=%7B%22title%22%3A+%22E2+-+Lalapalooza%22%2C+%22thumbnail%22%3A+%22https%3A%2F%2Fpic.bstarstatic.com%2Fogv%2Fc2053524d016f1ed5f4eccbd7b906d88c29e8e85.png%22%2C+%22episode_number%22%3A+2%7D
[BiliIntl] 13083848: Downloading JSON metadata
[BiliIntl] 13083848: Downloading video formats
[BiliIntl] 13083848: Downloading subtitles list
[BiliIntl] 13083848: Downloading subtitles for English
WARNING: [BiliIntl] Unable to download subtitles: HTTPSConnectionPool(host='s.bstarstatic.com', port=443): Read timed out. (read timeout=20.0)
[BiliIntl] 13083848: Downloading subtitles for ภาษาไทย
WARNING: [BiliIntl] 13083848: Failed to parse JSON: Unexpected UTF-8 BOM (decode using utf-8-sig): line 1 column 1 (char 0)
[BiliIntl] 13083848: Downloading subtitles for Bahasa Indonesia
WARNING: [BiliIntl] 13083848: Failed to parse JSON: Unexpected UTF-8 BOM (decode using utf-8-sig): line 1 column 1 (char 0)
[BiliIntl] 13083848: Downloading subtitles for 中文（简体）
[info] 13083848: Downloading subtitles: zh-Hans
WARNING: No subtitle format found matching "ass,srt" for language zh-Hans, using srt
[debug] Formats sorted by: hasvid, ie_pref, lang, quality, res, fps, hdr:12(7), vcodec:vp9.2(10), channels, acodec, size, br, asr, proto, vext, aext, hasaud, source, id
[info] 13083848: Downloading 1 format(s): 16+2
[info] Writing video subtitles to: E2 - Lalapalooza [13083848].zh-Hans.srt
[debug] Invoking http downloader on "https://upos-bstar1-mirrorakam.akamaized.net/iupxcodeboss/br/r0/n240115erw26xppx4ufmf1wihf6xr0br-1-261220110000.m4s?e=ig8euxZM2rNcNbdlhoNvNC8BqJIzNbfqXBvEqxTEto8BTrNvN0GvT90W5JZMkX_YN0MvXg8gNEV4NC8xNEV4N03eN0B5tZlqNxTEto8BTrNvNeZVuJ10Kj_g2UB02J0mN0B5tZlqNCNEto8BTrNvNC7MTX502C8f2jmMQJ6mqF2fka1mqx6gqj0eN0B599M=&uipk=5&nbs=1&deadline=1708950295&gen=playurlv2&os=akam&oi=3035704899&trid=dce740e837c2410494f43c5ac4f1c3a9i&mid=1625976544&platform=pc&upsig=9b3e87710a5bf867d7028799a605b9d5&uparams=e,uipk,nbs,deadline,gen,os,oi,trid,mid,platform&hdnts=exp=1708950295~hmac=9aa11a242f1d101a6542e6bbdf6307e2b2dc68f032b9f9632fb43814d044a983&bvc=vod&nettype=0&orderid=0,2&logo=00000000&f=i_0_0"
[download] Destination: E2 - Lalapalooza [13083848].f16.mp4
[download] 100% of  188.15MiB in 00:01:37 at 1.94MiB/s
[debug] Invoking http downloader on "https://upos-bstar1-mirrorakam.akamaized.net/iupxcodeboss/br/r0/n240115erw26xppx4ufmf1wihf6xr0br-1-2d1301000023.m4s?e=ig8euxZM2rNcNbdlhoNvNC8BqJIzNbfqXBvEqxTEto8BTrNvN0GvT90W5JZMkX_YN0MvXg8gNEV4NC8xNEV4N03eN0B5tZlqNxTEto8BTrNvNeZVuJ10Kj_g2UB02J0mN0B5tZlqNCNEto8BTrNvNC7MTX502C8f2jmMQJ6mqF2fka1mqx6gqj0eN0B599M=&uipk=5&nbs=1&deadline=1708950295&gen=playurlv2&os=akam&oi=3035704899&trid=a0bf82baf8414b6aa2582cc19c9db21di&mid=1625976544&platform=pc&upsig=59cb0bc2e4320b552c56d8d409e99948&uparams=e,uipk,nbs,deadline,gen,os,oi,trid,mid,platform&hdnts=exp=1708950295~hmac=8ef06999952dcb57ca46c56d6146f92e4a6bbc55f98142dc975b8c8caccb8d54&bvc=vod&nettype=0&orderid=0,2&logo=00000000&f=i_0_0"
[download] Destination: E2 - Lalapalooza [13083848].f2.mp4
[download] 100% of   32.46MiB in 00:00:17 at 1.85MiB/s
[Merger] Merging formats into "E2 - Lalapalooza [13083848].mkv"
[debug] ffmpeg command line: "C:\Users\nattadasu\AppData\Local\Programs\youtube-dl-gui\resources\app.asar.unpacked\binaries\ffmpeg" -y -loglevel "repeat+info" -i "file:E2 - Lalapalooza [13083848].f16.mp4" -i "file:E2 - Lalapalooza [13083848].f2.mp4" -c copy -map "0:v:0" -map "1:a:0" -movflags "+faststart" "file:E2 - Lalapalooza [13083848].temp.mkv"
Deleting original file E2 - Lalapalooza [13083848].f2.mp4 (pass -k to keep)
Deleting original file E2 - Lalapalooza [13083848].f16.mp4 (pass -k to keep)
[EmbedSubtitle] Embedding subtitles in "E2 - Lalapalooza [13083848].mkv"
[debug] ffmpeg command line: "C:\Users\nattadasu\AppData\Local\Programs\youtube-dl-gui\resources\app.asar.unpacked\binaries\ffmpeg" -y -loglevel "repeat+info" -i "file:E2 - Lalapalooza [13083848].mkv" -i "file:E2 - Lalapalooza [13083848].zh-Hans.srt" -map 0 -dn -ignore_unknown -c copy -map "-0:s" -map "1:0" "-metadata:s:s:0" "language=zho" -movflags "+faststart" "file:E2 - Lalapalooza [13083848].temp.mkv"
Deleting original file E2 - Lalapalooza [13083848].zh-Hans.srt (pass -k to keep)
[Metadata] Adding metadata to "E2 - Lalapalooza [13083848].mkv"
[debug] ffmpeg command line: "C:\Users\nattadasu\AppData\Local\Programs\youtube-dl-gui\resources\app.asar.unpacked\binaries\ffmpeg" -y -loglevel "repeat+info" -i "file:E2 - Lalapalooza [13083848].mkv" -i "file:E2 - Lalapalooza [13083848].meta" -map 0 -dn -ignore_unknown -c copy -map_metadata 1 -write_id3v1 1 -metadata "title=E2 - Lalapalooza" -metadata "purl=https://www.bilibili.tv/en/play/2098140/13083848#__youtubedl_smuggle=%7B%22title%22%3A+%22E2+-+Lalapalooza%22%2C+%22thumbnail%22%3A+%22https%3A%2F%2Fpic.bstarstatic.com%2Fogv%2Fc2053524d016f1ed5f4eccbd7b906d88c29e8e85.png%22%2C+%22episode_number%22%3A+2%7D" -metadata "comment=https://www.bilibili.tv/en/play/2098140/13083848#__youtubedl_smuggle=%7B%22title%22%3A+%22E2+-+Lalapalooza%22%2C+%22thumbnail%22%3A+%22https%3A%2F%2Fpic.bstarstatic.com%2Fogv%2Fc2053524d016f1ed5f4eccbd7b906d88c29e8e85.png%22%2C+%22episode_number%22%3A+2%7D" -metadata "episode_id=Episode 2" -metadata "episode_sort=2" -movflags "+faststart" "file:E2 - Lalapalooza [13083848].temp.mkv"

...

What does yt-dlp show with --list-subs?

This was addressed in https://github.com/yt-dlp/yt-dlp/commit/cf6413e840476c15e5b166dc2f7cc2a90a4a9aad. Update to a current nightly/master build.


plugins_graphdriver.md not exist and delete the reference

plugins_graphdriver.md not exist and delete the reference.

@lixiaobing10051267 That last commit doesn't look quite right as it reverts the previous one instead of linking to the path that @thaJeztah suggested. Also, please squash.

This PR is confused, I will close it and start another PR, sorry.


Test new Feature of LSP library

Refs: https://github.com/microsoft/vscode-languageserver-node/issues/599

[x] anyOS: @aeschli

Complexity: 3

There are new next version for the LSP client (6.2.0-next.1) and server (6.2.0-next.1) side adding the following features:

insert / replace edit for completion items
complex code for diagnostics (e.g. to support documentation URIs)
tag support for SymbolInformation

Please review the features and the new API that goes along with it.
In addition a cancel strategy got added to jsonrpc to allow for a different cancel strategy than sending a $/cancel notification. This strategy can for example used to implement file based cancellation for servers that run in a single thread environment.
Please test the new cancel strategy by implementing a custom cancel strategy. The following test case can be used as a blue print https://github.com/microsoft/vscode-languageserver-node/blob/master/jsonrpc/src/test/customCancellationStrategy.ts#L1

Closing item.


Pass in query parameters to mfa bypass evaluator rest endpoint

I was looking to use the bypass evaluator feature for the multifactor authentication. While I did successfully configure CAS to perform GET request on the configured endpoint (behaved as documented), the query parameters did not seem to be passed along in the request.
This is a trivial fix to include the query params in the http request, so that the behavior aligns with the documentation (https://apereo.github.io/cas/6.5.x/mfa/Configuring-Multifactor-Authentication-Bypass.html#bypass-via-rest):

Parameter
Description

principal
The identifier of the authenticated principal.

provider
The identifier of the multifactor authentication provider.

service
The identifier of the registered service in the registry, if any.

(I didn't include any test cases to verify the behavior, since I'm not very familiar with the spring framework)

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


Improve the use of Hadoop's DistributedCache

Update DistributedCacheHelper to both add a cached file to a MapReduce
job and to retrieve an InputStream from the DistributedCache (or HDFS,
if the item was not successfully cached).
Ensure uniqueness in the local cached file names and that the files are
read from the correct filesystem.
Add some tests to TokenFileIT
Specify the charset explicitly when java.util.Scanner is used (most
occurrences were related to code using the DistributedCache, but also
fixed two trivial occurrences in shell-related code).

This supersedes #1112 (it incorporates @jzgithub1 's changes from that issue, but makes further alterations) and should wrap up #1052 completely (or at least enough for 2.0).


Do Rollback

A rollback function would be handy to allow end-users to revert back to previous version.

Great suggestion! This is something we've considered but not yet started on. If you are interested in putting together a pull request on this, please feel free. Otherwise we may build it at some point, not sure when.


Animation detach on scene change.

Tested versions
Godot 4.4.dev3 - Compatibility renderer
System information
Android 14
Issue description
animation get detach from mesh when changeing scene. It also happening when changeing animation player property. But the animation is working well when playing in game.

Steps to reproduce
Added .glb model animation.
Made it local.
Renamed a animation.
Changed the scene.

Please upload an MRP to make this easier to test and fix

Please upload an MRP to make this easier to test and fix

Sure. Here is the project.
first-person-controller.zip


Unexpected error when running blockstack price

Seeing error when running blockstack price, also fail to load user profile.
MacBook-Gro:Downloads jobs$ blockstack lookup muneeb.id
Failed to load user profile
MacBook-Gro:Downloads jobs$ blockstack price google.id
{
    "error": "Unexpected error:\nTraceback (most recent call last):\n  File \"/usr/local/bin/blockstack\", line 101, in <module>\n    result = run_cli()\n  File \"/Library/Python/2.7/site-packages/blockstack_client/cli.py\", line 292, in run_cli\n    result = method( args, config_path=config_path )\n  File \"/Library/Python/2.7/site-packages/blockstack_client/actions.py\", line 498, in cli_price\n    fees = get_total_registration_fees( fqu, payment_privkey_info, owner_privkey_info, proxy=proxy, config_path=config_path, payment_address=payment_address )\n  File \"/Library/Python/2.7/site-packages/blockstack_client/actions.py\", line 271, in get_total_registration_fees\n    preorder_tx_fee = estimate_preorder_tx_fee( name, data['satoshis'], payment_address, utxo_client, owner_privkey_params=get_privkey_info_params(owner_privkey_info), config_path=config_path, include_dust=True )\n  File \"/Library/Python/2.7/site-packages/blockstack_client/backend/nameops.py\", line 119, in estimate_preorder_tx_fee\n    unsigned_tx = preorder_tx( name, payment_addr, fake_owner_address, name_cost, fake_consensus_hash, utxo_client )\n  File \"/Library/Python/2.7/site-packages/blockstack_client/tx.py\", line 40, in preorder_tx\n    inputs, outputs = tx_preorder( *args, **kw )\n  File \"/Library/Python/2.7/site-packages/blockstack_client/operations/preorder.py\", line 128, in make_transaction\n    inputs = tx_get_unspents( payment_addr, blockchain_client )\n  File \"/Library/Python/2.7/site-packages/blockstack_client/scripts.py\", line 476, in tx_get_unspents\n    data = pybitcoin.get_unspents(address, utxo_client)\n  File \"/Library/Python/2.7/site-packages/pybitcoin/transactions/network.py\", line 36, in get_unspents\n    return blockcypher.get_unspents(address, blockchain_client)\n  File \"/Library/Python/2.7/site-packages/pybitcoin/services/blockcypher.py\", line 57, in get_unspents\n    r = requests.get(url)\n  File \"/Library/Python/2.7/site-packages/requests/api.py\", line 70, in get\n    return request('get', url, params=params, **kwargs)\n  File \"/Library/Python/2.7/site-packages/requests/api.py\", line 56, in request\n    return session.request(method=method, url=url, **kwargs)\n  File \"/Library/Python/2.7/site-packages/requests/sessions.py\", line 488, in request\n    resp = self.send(prep, **send_kwargs)\n  File \"/Library/Python/2.7/site-packages/requests/sessions.py\", line 609, in send\n    r = adapter.send(request, **kwargs)\n  File \"/Library/Python/2.7/site-packages/requests/adapters.py\", line 423, in send\n    timeout=timeout\n  File \"/Library/Python/2.7/site-packages/requests/packages/urllib3/connectionpool.py\", line 594, in urlopen\n    chunked=chunked)\n  File \"/Library/Python/2.7/site-packages/requests/packages/urllib3/connectionpool.py\", line 350, in _make_request\n    self._validate_conn(conn)\n  File \"/Library/Python/2.7/site-packages/requests/packages/urllib3/connectionpool.py\", line 835, in _validate_conn\n    conn.connect()\n  File \"/Library/Python/2.7/site-packages/requests/packages/urllib3/connection.py\", line 330, in connect\n    cert = self.sock.getpeercert()\n  File \"/Library/Python/2.7/site-packages/requests/packages/urllib3/contrib/pyopenssl.py\", line 324, in getpeercert\n    'subjectAltName': get_subj_alt_name(x509)\n  File \"/Library/Python/2.7/site-packages/requests/packages/urllib3/contrib/pyopenssl.py\", line 166, in get_subj_alt_name\n    cert = _Certificate(openssl_backend, peer_cert._x509)\nAttributeError: 'X509' object has no attribute '_x509'\n",
    "help": "Try getting latest version of CLI with \"sudo pip install blockstack --upgrade\""
}

Hey @DSofter, thanks for reporting!
It looks like your pyopenssl package may be out of date.  Can you try upgrading it with pip install --upgrade pyopenssl?

Closing this issue for inactivity.  I'm guessing that it's working with the latest pyopenssl?  If not, please re-open in the blockstack-core issues.  Thanks!


Transfer to audiojs?

Hi @juliangruber!
Thanks for the foundational package, we heavily use it in @audiojs.
Just a humble suggestion − since we are using it in so many dependent packages, and since you are the member of @audiojs − is there a sense to transfer the package there?
We will set greenkeeper and API docs for that. Of course you will be the owner of the package. The name perfectly fits to the naming of audio-* packages.
Cheers

makes very much sense! great suggestion, and moved :)


Fix Objective C warnings

Fixed the following warnings:

conflicting return type in implementation of 'setNodeId:': 'void *' vs 'void'
extra tokens at end of #endif directive
unannotated fall-through between switch labels
designated initializer should only invoke a designated initializer on 'super'
convenience initializer should not invoke an initializer on 'super'
hiding of overloaded virtual functions

Thanks for addressing these warnings!

Codecov Report

Merging #507 into master will increase coverage by <.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #507      +/-   ##
==========================================
+ Coverage   54.16%   54.16%   +<.01%
==========================================
  Files         329      329
  Lines       57161    57161
==========================================
+ Hits        30962    30964       +2
+ Misses      26199    26197       -2

Impacted Files
Coverage Δ

src/lib/profiles/security/WeaveCert.cpp
88.96% <0%> (+0.45%)
:arrow_up:

Continue to review full report at Codecov.

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


Ranobes provider not working

Tried downloading 5 books from ranobes, all of them stuck at downloading first chapter. Other sources work fine, but ranobes has the freshest chapters and chillest novels.

iirc they have a lot of antibot stuff, so it is recommended to stream read on that provider

I see. I use moonreader for TTS so that's not an option for me, ended up using WebToEpub and transfering the files my tablet.


Issue with longest prefix matching in graphite templates with filters

I'm using influxdb_0.10.1-1_amd64 and working with multiple filters and templates and seeing unexpected behavior with 'longest prefix' matching
I have these templates:
templates = [
"abc.*.worker.*.*.*.count .host.type.service.metricType.measurement.field match=1",
"abc.*.worker.* .host.type.service.metricType.measurement match=2",
"abc .host.measurement* match=3",
".host.measurement* match=default",
]
and see the following matching behavior for these cases:

abc.hostA.worker.watching.NORMALIZED.dispatch-time.count   -> match=1
abc.hostA.worker.watching.NORMALIZED.dispatch-time.total   -> match=1
abc.hostA.worker.bookmarks.COUNTER.bookmarks-messages  -> match=default

For case 3, I expect match=2, since that's the longest match. If I remove template with match=1, then I see case 3 match as expected.
Am I missing something in the way my templates are defined?
Thanks,

Kiran

Will follow this up in the forum first.


PCLib Failed to initiate. Your server's version (v1_14_R1) is not supported

Describe the bug
In the spigotmc repository, you say, that it supports 1.14.4, but it doesnt Load
NPCLib usage option
3
NPCLib JAR name
npclib-api-2.0.1-SNAPSHOT.jar
Server version
This server is running CraftBukkit version git-Spigot-798ea6a-44d675a (MC: 1.14.4) (Implementing API version 1.14.4-R0.1-SNAPSHOT)

Hi there, thanks for opening this issue.
Are you shading npclib-api or npclib-plugin in your pom file? When shading NPCLib it's important to shade the npclib-plugin artifact instead of the API, as this will include the necessary NMS classes.

Hi. thank you, now i shaded the npclib-plugin instead of the api, and it works.

👍


fixed issues 20171 only for android, ios not tested

Description
Fixed "showDatePicker A RenderFlex overflowed by 5.0 pixels on the bottom of  DatePickerHeader widget"
Tests passed on android for Chinese localization, not test on IOS
Related Issues
issues 20171
Checklist
Before you create this PR confirm that it meets all requirements listed below by checking the relevant checkboxes ([x]). This will ensure a smooth and quick review process.

[ ] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[ ] My PR includes tests for all changed/updated/fixed behaviors (See Test Coverage).
[x] All existing and new tests are passing.
[ ] I updated/added relevant documentation (doc comments with ///).
[x] The analyzer (flutter analyze --flutter-repo) does not report any problems on my PR.
[ ] I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
[ ] I signed the CLA.
[ ] I am willing to follow-up on review comments in a timely manner.

Breaking Change
Does your PR require Flutter developers to manually update their apps to accommodate your change?

[ ] Yes, this is a breaking change (Please read Handling breaking changes).
[x] No, this is not a breaking change.

Unfortunately, in its current state we cannot review this PR.  You'll need to sign the CLA (see instructions in the comment by @googlebot).
I'm also not sure this is really the right fix - it will work a little better for some text scale factors and locales, but it doesn't really help make the date picker fully accessible.  Let's discuss this some more in the issue.


feat: Page tagging

Users want to add tags to pages so that data from RUM events can be separated into different groups for analysis, metric aggregation, etc. Currently, users are unable to add tags to pages. This feature will allow them to add tags and will store the page tag data in the event metadata. Users can pass in the page tag data:
(1) when manually recording page view events using recordPageView()
(2) by manually setting the page tag data using setCustomAttributes()

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Could we separate the page tagging change from the custom attributes change?
Specifically, I'm not sure we want to add setCustomAttributes with this change. While it is clear how the application will add custom page attributes (i.e., using recordPageView + PageAttributes), we'll need to think about how the application will add custom session attributes (i.e., using the config object or using some addSessionAttribute function).

Looks good to me! Let's wait until the event schema is deployed to merge.


Use spread when converting completion entry to protocol format

This avoids a problem that's happened in the past where we add a new property to the internal completion entry type, but forget to set it when converting it to the protocol completion entry format.

Majority of test changes are due to different order of property hasAction in the test logs. Other is that now we have isFromUncheckedFile present where we weren't setting it before, e.g. in baseline tests/baselines/reference/tsserver/fourslashServer/completionEntryDetailAcrossFiles01.js

@typescript-bot perf test this
@typescript-bot user test tsserver
@typescript-bot test tsserver top400


add SetRequest for Context

Codecov Report

Merging #53 into master will decrease coverage by 0.06%.
The diff coverage is 0%.

@@            Coverage Diff             @@
##           master      #53      +/-   ##
==========================================
- Coverage   68.92%   68.86%   -0.07%
==========================================
  Files          18       18
  Lines        2140     2142       +2
==========================================
  Hits         1475     1475
- Misses        511      513       +2
  Partials      154      154

Impacted Files
Coverage Δ

context.go
76.2% <0%> (-0.62%)
:arrow_down:

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 692f3aa...c0e1987. Read the comment docs.


sqlite 3.11.0

Failures are the same known link/unlink CI dance, and some contamination from the prior Science build which is nice. Merging.

Hey Dom, does this test failure for 3.12.0 look like the same link/unlink CI problems? Can't tell if this is the existing issue, or a remnant of the brew doctor related failures we were seeing yesterday and I thought were now fixed.
https://github.com/Homebrew/homebrew-core/pull/11
https://gist.github.com/apjanke/1e8362b367f5d72fb0f5823f38c7956b


Support @ special character

Why '@' character excluded from special characters ?

Wasn't intentional - Toss a PR my way to implement!

Version 1.0.8 published - thanks @sachindongare029!


Test - Please accept!

Bugreport:
Bug reported by Monkeyyy11
Is it a bugreport for the website or for the bot?: bot
What is the title of your bugreport?: Test - Please accept!
How can you reproduce the bug? (Explain in steps): Test - Please accept!
Which result would normally have to be?: Test - Please accept!
What happens if you do the action now?: Test - Please accept!
ReportID: 59

➤ LenoxBot commented:
Approve:
test
Nickkel

➤ LenoxBot commented:
Approve:

Denarioyt

➤ LenoxBot commented:
Approve:
test
Fabiooo


haskell.packages.ghc921: use random_1_2_1 (and related fixes)

Motivation for this change
@sternenseemann As promised, random ;)
QuickCheck and hspec were blocked by this, so this should fix a lot of packages.
I've removed the dontChecks that were directly blocked by random, but it looks like it might be possible to remove more by adding some jailbreaks.
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Github does not support dependent PRs, so I don't know where to put potentially unrelated but minor changes to the package set.
So here is a jailbreak for network :smile:

Using nixpkgs/haskell-updates with flakes, I am hitting this error. Anyone else has seen this?
λ ❯ nix build .#921/stc
warning: Git tree '/home/smunix/Programming/compilers/stc' is dirty
error: attribute 'random_1_2_1' missing

       at /nix/store/ibzaj9sgbph29yfh37q6vfiv3mbm5ml8-source/pkgs/development/haskell-modules/configuration-ghc-9.2.x.nix:202:22:

          201|   # The test suite indirectly depends on random, which leads to infinite recursion
          202|   random = dontCheck super.random_1_2_1;
             |                      ^
          203|

This is known and expected as we are currently doing major work on the package set. Don't use the haskell-updates for any purpose other than nixpkgs development/maintenance work. It is specifically intended as a place where we can have a completely broken package set in order for it to be stabilized and merged into master which is what you should track instead (in most cases it's probably even better to track an appropriate channel, e.g. nixpkgs-unstable or nixos-unstable).


Use .ts extension when creating Playwright config for TypeScript projects

Describe the bug
I've seen a few reports of this. It appears to have been a known issue when the feature was added: https://github.com/sveltejs/kit/pull/4056#discussion_r813166060
I saw that someone fixed it by prefixing with some environment variable. Unforunately, I can't find what that was at the moment
Reproduction
Create TypeScript + Playwright project from wizard
Logs
No response
System Info
latest

Severity
serious, but I can work around it
Additional Information
No response

"test": "PW_EXPERIMENTAL_TS_ESM=1 playwright test"

example implementation: https://codeberg.org/vhs/svelte-headlessui-starter

I believe you're looking for:
"test": "PW_EXPERIMENTAL_TS_ESM=1 playwright test"

example implementation: https://github.com/vhscom/svelte-headlessui-starter

Last update I saw for ESM in TS for Playwrite dated Jan 11: https://github.com/microsoft/playwright/issues/7121#issuecomment-1009140849. I'd expect the ENV var to simply become optional later, but since this flag was recently introduced I haven't dug in deeper yet.

I opened this some time ago and I'm waiting for the next version. After this TS code is usable. https://github.com/microsoft/playwright/issues/12307

I believe you're looking for:
"test": "PW_EXPERIMENTAL_TS_ESM=1 playwright test"

example implementation: https://github.com/vhscom/svelte-headlessui-starter

Testing this on the latest versions appears to result in the following issue:
Cannot find package 'url.js' imported from<EMAIL_ADDRESS>
(Full build logs for context)
(And a reproduction of the issue)

@EdricChan03 the url.js appears to have started with https://github.com/sveltejs/kit/blob/master/packages/kit/CHANGELOG.md#100-next292

The url.js appears to have started with 1.0.0-next.292 when using PW_EXPERIMENTAL_TS_ESM flag for tests.

Testing this on the latest versions appears to result in the following issue:
Cannot find package 'url.js' imported from<EMAIL_ADDRESS>

Appears to have started with 1.0.0-next.292 when using PW_EXPERIMENTAL_TS_ESM flag for tests with TypeScript. Backing down to<EMAIL_ADDRESS>and tests are functional again. So something in the 291-292 changeset is likely conflicting with the experimental TS ESM flag for Playwright.

Testing this on the latest versions appears to result in the following issue:
Cannot find package 'url.js' imported from<EMAIL_ADDRESS>

Appears to have started with 1.0.0-next.292 when using PW_EXPERIMENTAL_TS_ESM flag for tests with TypeScript. Backing down to<EMAIL_ADDRESS>and tests are functional again. So something in the 291-292 changeset is conflicting with the experimental TS ESM flag for Playwright in the current stable release of playwright. As a result, if using<EMAIL_ADDRESS>or above, consider installing playwright version 1.20.0 (currently tagged beta) or above, or try the latest nightly release with @playwright/test@next.
You're using experimental features, so I'll assume you know how to upgrade and downgrade your version of Node.
If the browser window stops opening and tests fail as a result, reinstall playwright with npx playwright install.

Works  again with the latest version of Playwright (1.20), which has been released just now.

PW_EXPERIMENTAL_TS_ESM had no effect for me. I had to rename the .js file to .ts as suggested in https://github.com/sveltejs/kit/issues/4341

PW_EXPERIMENTAL_TS_ESM=1 is required for the Playwright config to be written in TypeScript as of v1.20.

PW_EXPERIMENTAL_TS_ESM=1 is required for the Playwright config to be written in TypeScript as of v1.20. Reference implementation. OP asked about an environment variable. Were you looking for something else?

With the latest release of SvelteKit, when you setup a new Playwright+TypeScript project it is now working without needing any further changes
However, if I rename playwright.config.js to playwright.config.ts it does not work even when adding PW_EXPERIMENTAL_TS_ESM=1. Newly created projects are using Playwright 1.20.0

I just now ran a completely new installation of sveltekit, did an npm install and then npm test and got this error. Adding "test": "PW_EXPERIMENTAL_TS_ESM=1 playwright test" did not work. (on mac m1).

Use 1.20.0, they are fixing bugs with 1.20.1.

Unfortunately:
I'm having this issue using both 1.20.1 and next:

Only a VSCode issue. The tests are working.
If I use ./constants (without .ts) I get error in pw at runtime: [ERR_MODULE_NOT_FOUND] using PW_EXPERIMENTAL_TS_ESM=1 playwright test.

I think this can be closed now? https://github.com/sveltejs/kit/pull/4479

I'm using SvelteKit with PW 1.20.2 with no problem and Typescript. I think we can close.


Add region and mapping for REMIND 3.0

This PR implements the discussion from #208 about REMIND 3.0 regions as separate files keeping the existing model-mapping for REMIND 2.1 unchanged (to not alter behavior for existing model submissions to the IIASA database infrastructure).
We will also add guidance on region-naming conventions, see https://github.com/IAMconsortium/nomenclature/issues/142

Permissions to upload to the ECEMF-internal Scenario Explorer instance granted to all REMIND modelers.


Enforce session timeout

[#173359395]
I've added a new utility called IdleDetector. It's based on the existing library, although it was a bit dead (updated 2-3 years ago) and written in JS. I quickly rewrote it to TS as I was hoping I could support iframes. Finally, it turned out I can't really do it. I've left this copy here, as it's in TS now and I've added some tests that the original version didn't have.
If the user interacts only with iframes, we might trigger the warning about being idle. I can't really see a solution for that. But we don't log out user automatically. He'll still have 5 minutes to get back to the activity. Given that the initial timeout is 20 minutes, I think it's very unlikely to happen. And even if it does, there's still an easy way to continue work.

@scytacki, we discussed that a bit, you might be interested in the final error UI.
If you have some suggestions re wording, I'm happy to update it. It hasn't been designed by Kiley or Michael, I just based it on Michael's generic error UI.

👍 This looks good! I think some of the wording could be a bit clearer for our users, but I'm guessing much of this came from LARA so we want to leave as is.

Actually, this one doesn't come from LARA. I think I just googled for similar dialogs. If you have some suggestions (as native speakers), I'm happy to edit. :)

👍 This looks good! I think some of the wording could be a bit clearer for our users, but I'm guessing much of this came from LARA so we want to leave as is.

Actually, this one doesn't come from LARA. I think I just googled for similar dialogs. If you have some suggestions (as native speakers), I'm happy to edit. :)

@pjanik it would be good if this event of "continuing the session" is logged. I didn't check your changes carefully, but I didn't see it being logged to both GA and our log system. I think the AP has a way to log events to both systems with one function, but I might be wrong.

Done.

I'd also note that the code coverage is a little lower than the target. You can see the details here: https://codecov.io/gh/concord-consortium/activity-player/pull/122/diff
Perhaps it would be possible to use cypress's clock function to make a cypress test which brings up the dialog and exercises it: https://docs.cypress.io/api/commands/clock.html#Yields
Another question is how this handles anonymous users. It seems that case should be handled differently. For example they often will not be launched from the portal, so telling the user to go back to the portal would be wrong. In the anonymous case the user can just reload the page and the run_key in the URL will continue to work and they could see their data.
I'd guess the easiest approach is to disable the idle timer when anonymous.
Or we could show a different dialog that basically says, "You've been idle for too long. Click continue below to keep working". So that way there'd be kind of a similar experience between the auth'd and anonymous and we can know in the log messages whether the student is actually paying attention.
I don't think this case is a priority, so doing whichever is easier seems best. Perhaps the anonymous dialog will be harder, but make it easier to get more complete code coverage. In that case it might be worth it.

I think mostly app.tsx changes are untested, new components and utils should be. Generally, I'd say there's some growing tech debt in AP as its core parts, which require some more effort and planning in terms of testing, are not really covered yet - app.tsx logic, managed interactive, iframe runtime. I guess it'd be nice to work on that separately from feature stories at this point. I'm mostly talking about unit testing. Cypress will cover some of that, although it doesn't seem we have a pattern to test the launching from (fake) Portal. So, it might be difficult to test some features when we have separate code paths for logged in and anonymous student.
Re anonymous case and different dialog - I thought it'd be still useful to have the same error with the large username ("Anonymous") and the button to log in as a different user. If many students share the same machine, it might help to avoid completing the activity as anonymous by accident. Although, when two or more students share one machine, I guess they will be all either logged in or anonymous. Note that error messages (network, auth) designed by Kiley and Michael have "Click here" link that also redirects to Portal and defaults to learn.concord.org for anonymous users (where portal info it's not available). If we're changing it here, should we change these generic errors too?
This new idle warning shouldn't have any counter, just a button to continue?

@pjanik it would be good if this event of "continuing the session" is logged. I didn't check your changes carefully, but I didn't see it being logged to both GA and our log system. I think the AP has a way to log events to both systems with one function, but I might be wrong.

Done.

I'd also note that the code coverage is a little lower than the target. You can see the details here: https://codecov.io/gh/concord-consortium/activity-player/pull/122/diff
Perhaps it would be possible to use cypress's clock function to make a cypress test which brings up the dialog and exercises it: https://docs.cypress.io/api/commands/clock.html#Yields
Another question is how this handles anonymous users. It seems that case should be handled differently. For example they often will not be launched from the portal, so telling the user to go back to the portal would be wrong. In the anonymous case the user can just reload the page and the run_key in the URL will continue to work and they could see their data.
I'd guess the easiest approach is to disable the idle timer when anonymous.
Or we could show a different dialog that basically says, "You've been idle for too long. Click continue below to keep working". So that way there'd be kind of a similar experience between the auth'd and anonymous and we can know in the log messages whether the student is actually paying attention.
I don't think this case is a priority, so doing whichever is easier seems best. Perhaps the anonymous dialog will be harder, but make it easier to get more complete code coverage. In that case it might be worth it.

I think mostly app.tsx changes are untested, new components and utils should be. Generally, I'd say there's some growing tech debt in AP as its core parts, which require some more effort and planning in terms of testing, are not really covered yet - app.tsx logic, managed interactive, iframe runtime. I guess it'd be nice to work on that separately from feature stories at this point. I'm mostly talking about unit testing. Cypress will cover some of that, although it doesn't seem we have a pattern to test the launching from (fake) Portal. So, it might be difficult to test some features when we have separate code paths for logged in and anonymous student.
Re anonymous case and different dialog - I thought it'd be still useful to have the same error with the large username ("Anonymous") and the button to log in as a different user. If many students share the same machine, it might help to avoid completing the activity as anonymous by accident. Although, when two or more students share one machine, I guess they will be all either logged in or anonymous. Note that error messages (network, auth) designed by Kiley and Michael have "Click here" link that also redirects to Portal and defaults to learn.concord.org for anonymous users (where portal info it's not available). If we're changing it here, should we change these generic errors too?
This new idle warning shouldn't have any counter, just a button to continue?

OK, to move forward:

I've added anonymous variant of the idle warning following @scytacki suggestions:

I've added Cypress tests. Finally, I've managed to use cy.clock and cy.tick. The key was the correct order of Cypress commands. When AP downloads sample activity, it actually uses a timer too (to fake network requests). The code execution was getting stuck there because of the cy.clock. So, that's why I had to call cy.tick(1000) between cy.visit and cy.get(activity-summary).

To test logged in user variant, I've added __cypressLoggedIn URL param. Not pretty, but I think works fine for this use case. When we setup Cypress to handle URLs with some fake token=, this param should be removed.

OK, to move forward:

I've added anonymous variant of the idle warning following @scytacki suggestions:

I've added Cypress tests. Finally, I've managed to use cy.clock and cy.tick. The key was the correct order of Cypress commands. When AP downloads sample activity, it actually uses a timer too (to fake network requests). The code execution was getting stuck there because of the cy.clock. So, that's why I had to call cy.tick(1000) between cy.visit and cy.get(activity-summary).

To test logged in user variant, I've added __cypressLoggedIn URL param. Not pretty, but I think works fine for this use case. When we setup Cypress to handle URLs with some fake token=, this param should be removed.

@pjanik looks good, but there is a failing Jest test.

@pjanik looks good, but there is a failing Jest test.


Integrate joint limits interface into drivetrain

enforce joint velocity and effort limits for drivetrain

Should this be done? considering the roboteq already has limits imposed in its profile.xml file?

I think limits are unneeded for the drivetrain  motors/joints. Limits on the motors angular limits are in the roboteq profile and limits on the rover's linear/angular speed are on the drivetrain controller.


Question about the min-height: 100vh;

Hi Andy,
Your modern CSS Reset is awesome. We needed a new approach on the whole reset thing and you did a great job.
Anyway I may have an issue / question about the min-height: 100vh; you put on the body element.
I was reading a few days ago this article that we should not use this on mobile because of address bar and all. Here you have an other article about that too.
That's not an issue per say, I'm curious on your thoughts about this subject.
Thank you for your time.
Cheers! ✌

Howdy!
I put this rule in because the probability that your body will be shorter than the viewport is pretty minuscule. I guess if your thing is going to be the exact height of the viewport, remove the rule from the reset 👍


Add support Swift2.0 and XCode 7

Thanks for the PR!

not at all :) I have ideas how we can  improve your idea :)


Welcoming event

Adds a welcoming event, through summoning of a character named Paradox Game Converters Group (English because that's our main working language, for Catholic I had no idea)
Adds an (untested) effect to turn pregnant women at start up actually pregnant
Removes game start flags for achievements and historical 9th century settings

SIDENOTE: Welcoming event does not appear in Observer mode

(Character's appearance is random)

Wrong repo, my bad


Add some basic intro text to playground

I might end up moving more markdown / documentation in to the playground, but this is a good start.

Codecov Report

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

@@           Coverage Diff           @@
##           master     #206   +/-   ##
=======================================
  Coverage   26.54%   26.54%
=======================================
  Files          15       15
  Lines        2238     2238
=======================================
  Hits          594      594
  Misses       1644     1644

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 1ba2630...a9cf9bf. Read the comment docs.


Allow collection of tags from ContainerImage again

We need to move perf_tag method from CiMixing to ArTaggable. That is because in case of ContainerImage we collect tags, however we don't collect C&U for the image -> ContainerImage does not participate in CiMixin.
This bug was introduced in #11761 and reported by @zeari. Thanks! And sorry for the inconvenience.
@miq-bot add_label bug, metrics, provider/containers
@miq-bot assign @gtanzillo

@miq-bot add_label providers/containers

👍 LGTM

Euwe Backport details:
$ git log -1
commit 8fb76215abdb47f0ab0012aaeb152b9d5b5f7b54
Author: Gregg Tanzillo<EMAIL_ADDRESS>Date:   Tue Oct 18 17:44:39 2016 -0400

    Merge pull request #11909 from isimluk/fix-broken-cu-for-containers

    Allow collection of tags from ContainerImage again
    (cherry picked from commit 094689c55d196d8fc78d984cf133e5cacb8d2faa)


Windows auto-update doesn't clean up extensions directory from old extensions

Updating to the latest will keep the old bundled extensions and add the new ones because they have changed names which will cause duplicate extensions running at the same time.
Steps to reproduce:
Update to the latest with the app update.
Go to %AppData%
Checkout node_modules and see there are still the old extensions and new extensions (survey, license etc)

This is what I see after update to 2022.9.2806.35:

Interesting! Have you auto-updated from 5.x or had a fresh install after the new build system (I guess after we moved the extensions to own repos and renamed them)?
My node_modules looks like this:

See there's for example lens-license and lens-license-extension

Ah, I probably have done a clean install on windows since Lens 6 was released. So if I install the last 5.x version and autoupdate I should see this? And it only happens on windows?

When I ran the 5.4 installer (overtop of my 6.1 installation) I saw the double extensions (e.g. lens-telemetry and telemetry-lens-extension).
But when I let it update to 6.1 there was only telemetry-lens-extension.
Also, when I ran the 6.1 installer (overtop of my 5.4 installation) there was only telemetry-lens-extension

I only see the double extensions when running an old installer overtop of a newer installation. This is for an installation for all users.

Interesting because when I auto-updated from 5.3 to 6.1, the old ones were left there. Maybe it happens only for some users. I'm running Windows 11.

Confirmed it happens for single user installs on Windows, when a 5.x version is updated to a 6.x version, the old extensions remain.


Stuck in one lane with no enemies.

I've just noticed I was in lane 2 with 0 enemies when there were enemies in lane 1 and lane 3.

I guess it decides that because of some cool abilities being used on the lane, not sure if this needs to be fixed at all.

But it might prevent some lanes to be cleared fast if others will be stuck in some empty lane. :p

See #38

I have Autocannon, they are probably saying that their script isn't switching lanes at all, mine switches them perfectly (because of Autocannon I guess), it's just it's possible to get stuck on a lane with 0 enemies, when I choose other lanes with enemies I get thrown back to the empty lane.

This doesn't happen 100% of the time, I spotted this just once. I assume it is either because of some sweet abilities that the script likes in that specific lane or some voodoo magic.

I've had this happen, for me, I just needed to add autocannon and the lane switching worked.

I have a problem similar to this. It isn't switching lanes at all, it acts as if lane switching is manual and just stays in whatever lane I left it in and doesn't correct itself when I switch lanes. it does auto click but nothing else. I have put it in manually into both firefox and chrome and it is still doing the same thing. It was working fine yesterday but isn't today. I'm not sure what to do.

ecomett, for you I would suggest to look at the above comment asking if you've tried purchasing autocannon.
For ProfessorKaos64, autocannon helps when no switching happens at all. I have 10 autocannons and the lanes do switch, but sometimes, as described above, they will stick to the empty one even if I try to manually change it the script will return me to the empty one.

It works now, thank you.

Merged autocannon fix, you no longer need to buy an autocannon.


Datepicker does not support language locale for numbers

Ionic version:  (check one with "x")
[ ] 1.x
[X] 2.x
I'm submitting a ...  (check one with "x")
[ ] bug report
[X] feature request
[ ] support request => Please do not submit support requests here, use one of these channels: https://forum.ionicframework.com/ or http://ionicworldwide.herokuapp.com/
Current behavior:

Locale is interpreted as a string/array of values for day and month names. But, locale is just not confined to the day and month names. It covers the numbers also. One suggestion given is to use moment.js to implement locale support.
See the example below. If I want to display the calendar in arabic (leave out rtl for the moment), even if I supply the locale strings for the month and weekdays, the numbers will still showup with locale=en.
If I generate numbers and pass it as string, all the validations inside the utility for date time validations will fail since everything is numeric.
So, the current implementation of the datepicker fails even in browsers which have implemented i10n and i18n.
    var date = new Date()
    console.log(date.toLocaleString('ar'));

Expected behavior:

Accept an optional language input and try to display all the calendar information in the specified language locale. If it is not specified default it to current logic. If both language (optional input) and locale names for months and dayofWeek (optional now) provided, ignore language input and proceed with the current logic.

Another assumption that datepicker has is, it assumes the calendar is always English. In Arabian countries, the calendar is mostly hijri (Islamic calendar). For i10n compatibilty, I believe moment.js needs to be integrated since there is no way for the developer to manipulate the numerals that appear on the calendar as per locale.


Ghost text Project

Related Issue
Added a new project, Ghost text.
Closes:  #1130
Describe the changes you've made
Spooky text hover effect project.
Describe if there is any unusual behaviour of your code(Write NA if there isn't)
NA
Checklist:

[x ] My code follows the style guidelines of this project.
[x ] I have performed a self-review of my own code.
[ x] I have commented my code, particularly in hard-to-understand areas.
[ ] I have made corresponding changes to the documentation.
[x

] New and existing unit tests pass locally with my changes.

Screenshots
Put any screenshot(s) of the project here.

Good Work.

Thanks for this awesome contribution. :star_struck:
Keep Contributing.

I've fixed the script.js and index.html conflict, it arised coz of you created branch from some other branch other than synced main branch.
Always create branch from the main branch, it will not bring any unknown commits.

I've fixed the script.js and index.html conflict, it arised coz of you created branch from some other branch other than synced main branch.
Always create branch from the main branch, it will not bring any unknown commits.

Thankyou so much! Will take care in future :)


Elementwise, parallelizable mutation?

I happen to be working on a GA problem with a rather costly mutation. Unlike the problem definition, I don't see any parallelizable mutation implementation. Is there any plan or interest in making mutation element-wise and parallelizing? I'll be happy to contribute. Since I get the reference to problem in the mutation, I guess I can use the elementwise_runner of the problem instance.

Just curious, what is the mutation you have implemented?
Most mutations I have implemented are written already vectorized (because the input is always the whole population).
If that is not possible, then using threads is a good option! You might also want to check out joblib which has a nice API for parallelization as well.


Enable quicktype extension to run on VS Code on the web

:wave: I'm Isidor, a PM from the VS Code team, and we recently announced VS Code for the Web at https://vscode.dev/, which provides a free, zero-install Microsoft Visual Studio Code experience running entirely in your browser.  You can learn more here.
We'd like to ensure quicktype can run in the web, and we have a guide for enabling extensions for the web here. As a couple of highlights:

In VS Code for the Web, both the UI and extension host run inside the browser.
A web extension is structured like a regular extension, but with a different main file: it's defined by the browser property
Access to workspace files needs to go through the VS Code file system API accessible at vscode.workspace.fs
There are currently three ways to test a web extension

Please let us know if you have questions or feedback on enabling your extension for the web - our team would love to learn from your experiences and collaborate. Thank you!

Hi @schani! I was wondering if you might have thoughts on this? We're happy to discuss if you have any questions or feedback. Thank you!

@isidorn @bamurtaugh Apologies for the late reply, and thank you for the pointers!
I'm afraid we don't currently have the time to do this.  We do have an intern starting in two months at glideapps.com though - maybe he'll be up for doing this.
We also accept PRs ;-)


isSafeDevice logic is confusing

hi, i just find out that the logic for isSafeDevice is absolutely confusing
return isJailBroken || !isRealDevice || canMockLocation;
the docs indicates that :
// Check if device violates any of the above
but the variable name is isSafeDevice
I suggest to swap either the variable name or the return logic  @ufukhawk

safe_device: ^1.1.5


Issue 7 - remove dependence on proxying requests

This PR made it easier for the client app to talk to the API.

update RxJS
update NestJS
Added a variable for API host

@farhadmak Could you check out this PR and verify it's working for local dev?

Btw, closes #7

can confirm this works on Ubuntu WSL.

Perfect. Merging in.


Added Luminous library

Project URL
https://github.com/andrealufino/Luminous
Description
Luminous is a big library that gives you a lot of information about the system. You can retrieve more than 40 kind of different information. Luminous provides also some handy methods (email check for example) and an interface to handle better some common variables used in the projects (color palette or server url).
Why it should be included to awesome-ios
Because people has to know that there is also this library that can help them to boost their develop time and get lot of information with just one line of code.
Checklist

[x] Only one project/change is in this pull request
[x] Addition in chronological order (bottom of category)
[x] Supports iOS 8 or later
[x] Supports Swift 3
[x] Has a commit from less than 2 years ago
[x] Has a clear README in English

Nice lib, @andrealufino. Your library is soo complete! 👍

          1 Error

  :no_entry_sign:
  Found 2 link issues

Link issues by awesome_bot

Line
Status
Link

1351
301
https://github.com/jaydee3/JDFlipNumberView redirects tohttps://github.com/calimarkus/JDFlipNumberView

1447
301
https://github.com/jaydee3/JDStatusBarNotification redirects tohttps://github.com/calimarkus/JDStatusBarNotification

  Generated by :no_entry_sign: danger

Thank you!


feat(in-app swaps): connect swap screen to cloud function

Description
Connect swap screen to cloud function and ensure details are correct:

add error banner / clear exchange rate when api request fails
only query the api at a debounced time interval
do not query api when the swap amount is 0
handle exchange rates from the api, for when changing swap amounts and swap tokens
handle a visual loading state while fetching new rates

There are 2 not so elegant things in this implementation (open for suggestions on how to improve these)

android and ios TextInput don't behave the same, so to handle long values (e.g. long swap values on small devices) there is an elaborate hack involving the selection in SwapAmountInput.tsx - i've tried to keep this localised, so that if we don't need it anymore then it's straightforward to remove
changing swap amount (independent value) -> should get new quote / exchange rate -> should change swap amount (dependent value). this loop makes it difficult to avoid a duplicate request - i've added a ref variable to prevent this in useSwapQuote.ts, again not great but i've tried to keep it contained so that we can change it.

https://user-images.githubusercontent.com/20150449/182576818-4721dad6-13f8-42d5-938d-b02dff4d61da.mp4
Other changes
N/A
Tested
A truckload of unit tests + much manual testing on android / ios
How others should test

on change "from" input => the "to" value should change and the exchange rate should be updated
on change "to" input => the "from" value should change and the exchange rate should be updated
on press "max" => the "from" value should be the max balance of that token for the user, and the "to" value + exchange rate should be updated
removing any value (by clicking on the cross icon inside the "to" or "from" input) should clear values for both lines
on change "from" token => the "from" value should remain, and the "to" value + exchange rate should be updated
on change "to" token => the "to" value should remain, and the "from" value + exchange rate should be updated

Related issues

Fixes RET-269

Backwards compatibility
Yes

@kathaypacific one thing that stood out to me in a quick test was if I change to To token the amount in the from token updates. I think this is a bit strange as if I wanted to swap 10 Celo for something i'd be nice to be able to go through my different options seeing the result of the swap for 10 Celo.

@kathaypacific one thing that stood out to me in a quick test was if I change to To token the amount in the from token updates. I think this is a bit strange as if I wanted to swap 10 Celo for something i'd be nice to be able to go through my different options seeing the result of the swap for 10 Celo.

oh interesting, that makes sense. i actually put in a line of code specifically to make this happen, i thought it made sense but maybe not. i removed it, i guess the question still remains what should happen when you change the value of the "to" field, and then change the "to" token? would you expect the "to" or the "from" value to change in this case? i've got a meeting with Nitya tonight, will also check with her :)  thanks!

when you change the value of the "to" field, and then change the "to" token?

I would expect the 'from' field to update in this case. Maybe we could keep track of the last input selected? This could be a follow up PR if desired.

Default to the 'from' field as last selected.
If last selected is 'from': Changes to 'from' input, 'from' token and 'to' token - change the value in the 'to' input.
If last selected is 'to': Changes 'to' input and 'to' token - changes the value in the 'from' input.

when you change the value of the "to" field, and then change the "to" token?

I would expect the 'from' field to update in this case. Maybe we could keep track of the last input selected? This could be a follow up PR if desired.

Default to the 'from' field as last selected.
If last selected is 'from': Changes to 'from' input, 'from' token and 'to' token - change the value in the 'to' input.
If last selected is 'to': Changes 'to' input and 'to' token - changes the value in the 'from' input.

i think that is the behaviour that is in this PR :D so we are aligned!

Hey @MuckT as discussed in weeky call this is still behind feature flag / still in implementation phase, please let us know when it needed to be tested
Cc: @kathaypacific
Thanks..!


Update meeting-immersive-spaces.md

This PR is a fix to a specific typo as mentioned in the issue MicrosoftDocs/OfficeDocs-SkypeForBusiness-pr#13644

@typride
PR has been copyedited and is ready for final review, could you please check and merge? Thanks!

@JohanFreelancer9 @scanum  Whats the status of this?


Accessing component's internal state within asyncValidate

I've got a custom-ish select component that's implemented as two dropdowns – the first being option groups, the second being the selected option group's dependents. This component is implemented using internal state to hold the option group value, which allows the second select to display the appropriate options.
The asyncValidate function seems to run as soon as pristine === false, however I'd like to check the component's internal state to to evaluate if state.dependentId has a value before validating.
Is there a way to access the component's internal state within the asyncValidate function?

The answer was to take that piece of internal state and send it up to redux's global state using the redux-form onChange event. Hopefully someone else trying to manage a component's internal state will find this helpful.
<StatefulSelectComponent
  onChange={ (event) => {
    this.props.fields[field_name].onChange(event.target.value);
    const dependentId = options.reduce((acc, option) => {return option[1] === ~~event.target.value ? option[2] : acc;}, '');
    this.props.setInputDependent(dependentId);
  }}
  {...this.props} />

Thanks for posting your solution. :+1:


Search results numbered throughout types

Currently you search and get hits "0-4" in every type. This requires then to specify the type when wanting to play something. Better would be to number them throughout types and then "play 7" would play that item, regardless of the type:
Tracks:
1.
2.
3.
4.
Albums:
5.
6.
7.
Playlists:
8.
9.
10.

This seems to be the most common mistake (people typing play 0) and would be mitigated by this change.

@baschny ja, good idea

@baschny this is fixed now...hopefully


Continuing development on the main repo

Hey folks, sorry for being AWOL.
I'd added you to both GitHub and PyPI, but didn't know there were more things that I had to update, like enabling merge commits on the repo.
I never wanted to abandon the project, my intention was to continue working on it when I'd dealt with some other things in my life.
I've synced the repo with all of your changes. The only thing I added on top was swapping poetry with uv because I've been working with it and I'm amazed at how fast it is.
I've also released a new version of https://pypi.org/project/excalibur-py with some bugfixes, and pushed linux and macos wheels for https://pypi.org/project/pdftopng from python 3.9 to 3.13, I still need to push the windows wheels.
I'll release a new camelot-py version by the end of this week based on all the changes I've synced back from this repo.
I request you to come back and work on it with me 🙏🏽 I really want to figure out a sustainable way to take this project forward.
Sorry again for my hiatus.

cc: @MartinThoma @foarsitter @bosd

I'm just discovering that I had to add you all to the repo too, I thought adding you to the organization would've given you all permissions 😭

@vinayak-mehta Thanks, I received an invite to contibute over at the other repo.
Sadly, I don't have enough permissions (yet) to do any kind of maintenance task.
(Acces to repo secrets, actions) Which is needed otherwise, the project can go into a locked state, as happened beofre when the key person is having a break.
You have to be an owner of an organization to have these powers..
Github permissions / user levels is really a "thing".
Recently one of the organizations I started was taken over by an dictator. As a fellow owner, he had the powers to kick all other owners out. So with these powers come great responsibility...
https://docs.github.com/en/organizations/managing-peoples-access-to-your-organization-with-roles/roles-in-an-organization#permissions-for-organization-roles
Having that said. I still think there should be multiple owners of an organization. Life happens, people may take a break for a while.
This goes for the "main" github repo. As well as the other tools. (Codecov, Pypi, readthedocs)

The only thing I added on top was swapping poetry with uv because I've been working with it and I'm amazed at how fast it is.

@vinayak-mehta Cool :sunglasses: , I have been starting to use (playing) with UV as well.
I've created a cookiecutter based on the one used here, where poetry is replaced with uv.
Maybe you can use/contribute to it: https://github.com/bosd/cookiecutter-uv-hypermodern-python
https://cookiecutter-uv-hypermodern-python.readthedocs.io/en/latest/

Can you give me your PyPI, codecov, and readthedocs usernames and emails so I can add you there too?
Let me read up on github permissions too to see what's the best way to go about it. Till then I'll make sure I'm not a blocker for any actions, I plan to spend some time every week working on this.
Looks like you all are based in Europe, are you planning to go to FOSDEM in February and would you like to meet in person?

I'm super happy to hear that camelot will be revived :tada: Thank you @vinayak-mehta :pray:

Once @bosd has the necessary permissions to do merges on Github and releases on pypi, I'd suggest that I archive pypdf_table_extraction. It' better to have the development continue at the original project.

@vinayak-mehta You don't need to add me to anything; I don't give pypdf and pdfly (and some others) enough attention so I should not add something on top :sweat_smile:


Screen readers are not able to read the PIN input screen.

When rereading the PIN, the screen reader reads the PIN as a single number (ex. 1 0 0 0 0 1 will read one hundred thousand and one). It should read each individual digit.

Put spaces in-between each number
Strip out spacing before submitting

Ref: #551

This happens because the splitting of digits is just astetics. Under the hood its just a text field that splits characters up for display. To fix this, we will need to find a better replacement component, write our own, or contribute a fix back to the main project. Regardless of the choice its going to be some effort.

We're going to test to see if we can add spaces to the digits in the underlying text field (and strip them out) to force the screen reader to recite each digit separately.


TS module error when trying to deploy

Hello,
On v0.14.0, when trying to deploy the whole monorepo, I have the error:
    ✖  nx run backend-orchestrator:deploy
       ><EMAIL_ADDRESS>deploy ........./swarmion/backend/orchestrator
       > serverless deploy

       Environment: linux, node 16.17.0, framework 3.25.0 (local), plugin 6.2.2, SDK 4.3.2
       Docs:        docs.serverless.com
       Support:     forum.serverless.com
       Bugs:        github.com/serverless/serverless/issues

       Error:
       Cannot load "serverless.ts": Initialization error: /home/fargito/Documents/theodo/swarmion/backend/orchestrator/serverless.ts:1
       import { listDeploymentsContract, onDeploymentRequestedContract, requestSyncDeploymentContract, } from '@swarmion/orchestrator-contracts';
       ^^^^^^

I believe the TS "module" setting is incorrect for Serverless services. It is set to esnext and should be node16.
However, I agree that esnext should be the value for frontend services.
I think we should set "module": "node16" in the root tsconfig and override it only in frontend packages in the examples.
@adriencaccia WDYT?

Thank you for the issue!
Going further than what you pointed out, now all serverless cli commands will not work since they are using ts-node under the hood, which uses the tsconfig.json file.
Since this is an implementation detail of the serverless framework and how it uses ts-node, I would recommend to add it only in the serverless services. Like this:
{
  ...
  "ts-node": {
+   "compilerOptions": {
+     "module": "node16"
+   },
    "files": true
}

WDYT?


Not working properly on iOS devices

It does not properly focuses the next field on typing nor properly focuses the previous field on deletation on iOS devices. See the attached screenshot of my implementation for six digit code.

What length value have you passed. Might be the limit.

It works perfectly fine for android devices.

I noticed that ios doesn't count the hex character \uFEFF in the string. For example, when we type the character 1 into the field, the plugin concatenates with \uFEFF. On android this string is size 2. On ios it is size 1.

yes even am facing same problem in ios