Update user guide

[x] Rework glossary table.
[x] Move Ui.png back into docs/images to adhere to course requirements.
[x] Rework introductory paragraph into bullet points for better readability.
[x] Update quick start instructions for better a11y to include users who are not as technologically inclined.

Codecov Report
Base: 62.69% // Head: 62.69% // No change to project coverage :thumbsup:

Coverage data is based on head (69c8095) compared to base (53051b8).
Patch has no changes to coverable lines.

Additional details and impacted files
@@            Coverage Diff            @@
##             master     #184   +/-   ##
=========================================
  Coverage     62.69%   62.69%
  Complexity      666      666
=========================================
  Files           118      118
  Lines          2356     2356
  Branches        295      295
=========================================
  Hits           1477     1477
  Misses          769      769
  Partials        110      110

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


[ITensors] [BUG] autodifferentation with dag function of complex ITensors

Description of bug
Autodifferentiation using dag function with A+iX produces different results than by simply breaking up the inner products.
Define $f(X) = tr((A+iX)^\dagger (A+iX))$. Then $f'(X)$ produces different results depending on whether one takes (dag(A+iX)*(A+iX))[] or (dag(A)*A)[]-i(dag(X)*A)[]+i(dag(A)*X)[]+(dag(X)*X)[], where A and X are ITensors.
Minimal code demonstrating the bug or unexpected behavior
Minimal runnable code
Here, we have four different functions: fun1, fun2, fun3, and fun4.
fun1 and fun2 returns the trace of $(A+iX)^\dagger (A+iX)$. The former simply breaks down the trace whereas the latter defines $G = A+iX$ and takes the trace of $G^\dagger G$.
fun3 and fun4 returns the trace of $(A+X)^\dagger (A+X)$. The former simply breaks down the trace whereas the latter defines $G = A+X$ and takes the trace of $G^\dagger G$.
using ITensors
using LinearAlgebra
using Zygote
using Random

Random.seed!(1234)

L = 4
sites = siteinds("S=1/2",L)
x = randomMPS(sites)
y = randomMPS(sites)
A = outer(x',y)
A = prod(A)

x = randomMPS(sites)
y = randomMPS(sites)
X = outer(x',y)
X = prod(X)

function fun1(X::ITensor, A::ITensor)
    trSA = (dag(A)*A)[]-1.0im*(dag(X)*A)[]+1.0im*(dag(A)*X)[]+(dag(X)*X)[]
    trSA = real.(trSA)
    return trSA
end

function fun2(X::ITensor, A::ITensor)
    G = A + (1.0im)*X
    trSA = (dag(G)*G)[]
    trSA = real.(trSA)
    return trSA
end

function fun3(X::ITensor, A::ITensor)
    trSA = (dag(A)*A)[]+(dag(X)*A)[]+(dag(A)*X)[]+(dag(X)*X)[]
    trSA = real.(trSA)
    return trSA
end

function fun4(X::ITensor, A::ITensor)
    G = A + X
    trSA = (dag(G)*G)[]
    trSA = real.(trSA)
    return trSA
end

fun1wrap(X) = fun1(X,A)
fun2wrap(X) = fun2(X,A)
fun3wrap(X) = fun3(X,A)
fun4wrap(X) = fun4(X,A)
∂fun1(X) = fun1wrap'(X)
∂fun2(X) = fun2wrap'(X)
∂fun3(X) = fun3wrap'(X)
∂fun4(X) = fun4wrap'(X)

@show fun1wrap(X), fun2wrap(X)

∂tr1 = ∂fun1(X)
∂tr2 = ∂fun2(X)
@show norm(∂tr1-∂tr2)
@show tr(∂tr1), tr(∂tr2)

@show fun3wrap(X), fun4wrap(X)

∂tr3 = ∂fun3(X)
∂tr4 = ∂fun4(X)
@show norm(∂tr3-∂tr4)
@show tr(∂tr3), tr(∂tr4)

Expected output or behavior
We would expect that fun1(X) = fun2(X) and fun3(X) = fun4(X). Further, we would expect that fun1'(X) = fun2'(X) and fun3'(X) = fun4'(X), where $A$ is fixed and the derivatives are taken with respect to $X$.
Actual output or behavior
As expected, fun1(X) = fun2(X) and fun3(X) = fun4(X). However, we see that fun1'(X) != fun2'(X) but fun3'(X) = fun4'(X). Furthermore, the real parts of the trace of fun1'(X) and fun2'(X) are identical but the imaginary parts are not: namely, the imaginary part of fun1'(X) is zero while that of fun2'(X) is non-zero. On the other hand, fun3'(X) = fun4'(X),
(Disclaimer: fun'(X)'s were compared by taking the norm of the difference)
Output of minimal runnable code
(fun1wrap(X), fun2wrap(X)) = (2.000000000000001, 2.0000000000000004)
norm(∂tr1 - ∂tr2) = 2.0000000000000004
(tr(∂tr1), tr(∂tr2)) = (0.44181916988683434, 0.44181916988683434 - 1.0717199952164762im)
(fun3wrap(X), fun4wrap(X)) = (2.01198154936903, 2.01198154936903)
norm(∂tr3 - ∂tr4) = 1.9211915651343907e-16
(tr(∂tr3), tr(∂tr4)) = (1.5135391651033105, 1.5135391651033105)

Version information

Output from versioninfo():

julia> versioninfo()
Julia Version 1.7.3
Commit 742b9abb4d (2022-05-06 12:58 UTC)
Platform Info:
  OS: Windows (x86_64-w64-mingw32)
  CPU: AMD Ryzen 9 4900HS with Radeon Graphics
  WORD_SIZE: 64
  LIBM: libopenlibm
  LLVM: libLLVM-12.0.1 (ORCJIT, znver2)

Output from using Pkg; Pkg.status("ITensors"):

julia> using Pkg; Pkg.status("ITensors")
      Status `C:\Users\hjkqu\.julia\environments\v1.7\Project.toml`
  [9136182c] ITensors v0.3.15

Fixed by https://github.com/ITensor/ITensors.jl/commit/b66d1b7c1f0b67332d07d7b539c6916dbe6d673c.


Transform decorators containing private field references into a 'static {}` block

This is a WIP investigation into potential compiler performance regressions that could arise from conditionally moving decorator evaluation into a static {} block when a decorator contains a private field reference.

@typescript-bot perf test

#50074 seems like the better performing approach, so I will use that implementation instead.


Start using new eval history table

This PR wires the eval history service into the evaluation engine. It is gated behind a feature flag while we experiment with it. A follow on PR will add logging for remediation and alerting.
Relates to #3556
Summary
Provide a brief overview of the changes and the issue being addressed.
Explain the rationale and any background necessary for understanding the changes.
List dependencies required by this change, if any.
Fixes #(related issue)
Change Type
Mark the type of change your PR introduces:

[ ] Bug fix (resolves an issue without affecting existing features)
[x] Feature (adds new functionality without breaking changes)
[ ] Breaking change (may impact existing functionalities or require documentation updates)
[ ] Documentation (updates or additions to documentation)
[ ] Refactoring or test improvements (no bug fixes or new functionality)

Testing
Outline how the changes were tested, including steps to reproduce and any relevant configurations.
Attach screenshots if helpful.
Review Checklist:

[x] Reviewed my own code for quality and clarity.
[x] Added comments to complex or tricky code sections.
[ ] Updated any affected documentation.
[x] Included tests that validate the fix or feature.
[ ] Checked that related changes are merged.

coverage: 52.815% (+0.08%) from 52.736%
when pulling ab94888ca8ee760716d4d212c21143fabc44a39e on wire-up-history-logs
into b5398b905cadcb3998b8a44ce651915d45911245 on main.

coverage: 52.805% (+0.06%) from 52.746%
when pulling 3bb152d42749d48f2563300f4730f9e3e811690e on wire-up-history-logs
into 66c47f5e86ad108acf27505a27e114d3191b28df on main.

coverage: 52.495% (+0.06%) from 52.432%
when pulling b6d11cb18dee6c2c7bb6e6321a050705f5122a3e on wire-up-history-logs
into 98373adb14347e11bdb703df365347de476339cd on main.

coverage: 52.486% (+0.05%) from 52.432%
when pulling a1d2120b9c01f9a6584a52d071432e819b845551 on wire-up-history-logs
into 98373adb14347e11bdb703df365347de476339cd on main.


Adds check before dismissing the reminder of other user

Please take a look.

Codecov Report

Merging #207 (914d4fd) into 2021.6.x (744bb70) will increase coverage by 0.00%.
The diff coverage is 44.44%.

@@            Coverage Diff            @@
##           2021.6.x     #207   +/-   ##
=========================================
  Coverage     39.64%   39.65%
=========================================
  Files           624      624
  Lines         61611    61645   +34
=========================================
+ Hits          24426    24443   +17
- Misses        34381    34396   +15
- Partials       2804     2806    +2

Impacted Files
Coverage Δ

system/service/reminder_actions.gen.go
10.37% <39.28%> (+2.79%)
:arrow_up:

system/service/reminder.go
60.37% <62.50%> (+0.24%)
:arrow_up:

pkg/handle/handle.go
78.57% <0.00%> (-7.15%)
:arrow_down:

system/rest/handlers/reminder.go
63.33% <0.00%> (+3.33%)
: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 744bb70...914d4fd. Read the comment docs.


Add function comments

Add JsDoc comments to Frontend components

i want to deal with this issue.
I am interested in contributing to the project you are developing.
Do you still need help?

Hi Sophia,
that's great. If you want we can schedule a meeting and we can tell you a little bit more about the project. We are building a new version on the v2 branch and definitly will be happy about help.
Best regards,
Anett


Box Folder Download Overwrite/Ignore

When you using the box folders:download option there is no option of ignore existing files or overwrite in the target.  This means that we cannot download as it fails as soon as a folder already exists.
Would it be possible to define a 'skip' or 'overwrite' when doing a folder download?

@jacquesvandermerwe Can you elaborate more on the use case for this?  If a user wants to overwrite any existing files anyway, they should be able to delete the folder they already have and re-download — that would have the same effect.  Is there a compelling case for the skip behavior?

@mattwiller This is an action that is done for a LOT of content. The box cli is used in combination with Upload and Download actions. Only new data is synced down and the other source uploads new content which is then synced back up to Box. So the CLI is being used to create a stable state. With the amount content it will cause excessive price spikes on the platform API call count and bandwidth costs. So if the content already exists it should not be overwritten but skipped.

@jacquesvandermerwe Makes sense to me!  I'll add this to our backlog.

This issue was closed by mistake.
There is already a corresponding Pulse request #39125995. We'll review this request and add to the roadmap in a few quarters.

Hello, just wondering if there is any update on this topic? Thx!

Hi @plmeta
This change isn't planned for the next release (Aug-Nov 2023), however we'll evaluate it for the next release cycle. Team is currently busy with making significant enhancement to underlying library we use and after that we will evaluate our roadmap and consider this change.

Hi @plmeta
We have just implemented this feature and it should be included in the next release.

Is it possible to also implement this feature to box folders:update? I encountered this issue when the uploading process got interrupted and I am not allowed to continue the process  as it fails as soon as a folder already exists.


Locale-dependent test

Hello! This DateMessageParser test is locale-dependent, and fails in non-English locales (which is how I discovered it!).
What would the best solution for this issue? Adding a new locale parameter (e.g. message.timestamp.locale) or just removing the test?

On Mon, Feb 23, 2015 at 1:44 PM, Alessandro Andrioni <
<EMAIL_ADDRESS>wrote:

Hello! This DateMessageParser test
https://github.com/pinterest/secor/blob/master/src/test/java/com/pinterest/secor/parser/DateMessageParserTest.java#L53-L55
is locale-dependent, and fails in non-English locales (which is how I
discovered it!).
What would the best solution for this issue? Adding a new locale parameter
(e.g. message.timestamp.locale) or just removing the test?
—
Reply to this email directly or view it on GitHub
https://github.com/pinterest/secor/issues/78.

@zago would you mind taking a look at this test?


[Screengrab] Make sure you've used Screengrab.screenshot() on API < 30

New Issue Checklist

[x] Updated fastlane to the latest version
[x] I read the Contribution Guidelines
[x] I read docs.fastlane.tools
[x] I searched for existing GitHub issues

Issue Description

Hello,
fastlane screengrab doesn't work below API 30
I use Screengrab.screenshot("home") in my UI test.
Command executed

On API 29 or below
fastlane screengrab
With the exact same code, on a device with API 30, it works
Complete output when running fastlane, including the stack trace and command used

 +-----------------------------+--------------------------------------------------------------------------------------------+
|                                              Summary for screengrab 2.186.0                                              |
+-----------------------------+--------------------------------------------------------------------------------------------+
| android_home                | /Users/alizee.camarasa/Library/Android/sdk                                                 |
| locales                     | ["fr-FR", "fr-BE"]                                                                         |
| clear_previous_screenshots  | true                                                                                |
| output_directory            | fastlane/metadata/android                                                                  |
| skip_open_summary           | false                                                                                |
| app_package_name            | com.backmarket.troy.debug                                                                  |
| tests_package_name          | com.backmarket.troy.debug.test                                                             |
| use_tests_in_classes        | ["com.backmarket.screenshots.tests.ScreenshotsTest"]                                       |
| test_instrumentation_runner | androidx.test.runner.AndroidJUnitRunner                                                    |
| ending_locale               | en-US                                                                                |
| use_adb_root                | false                                                                                |
| app_apk_path                | app/build/outputs/apk/developProd/screenshots/app-develop-prod-screenshots.apk             |
| tests_apk_path              | app/build/outputs/apk/androidTest/developProd/debug/app-develop-prod-debug-androidTest.apk |
| device_type                 | phone                                                                                |
| exit_on_test_failure        | false                                                                                |
| reinstall_app               | true                                                                                |
| use_timestamp_suffix        | true                                                                                |
+-----------------------------+--------------------------------------------------------------------------------------------+
[11:03:21]: Clearing phoneScreenshots within fastlane/metadata/android
[11:03:21]: $ /Users/alizee.camarasa/Library/Android/sdk/platform-tools/adb -s emulator-5554 shell echo $EXTERNAL_STORAGE
[11:03:21]: ▸ /sdcard
[11:03:21]: Cleaning screenshots on device
ls: /sdcard/com.backmarket.troy.debug/screengrab/fr-FR/images/screenshots: No such file or directory
[11:03:21]: Exit status: 1
ls: /sdcard/com.backmarket.troy.debug/screengrab/fr-BE/images/screenshots: No such file or directory
[11:03:21]: Exit status: 1
ls: /data/user/0/com.backmarket.troy.debug/files/com.backmarket.troy.debug/screengrab/fr-FR/images/screenshots: No such file or directory
[11:03:21]: Exit status: 1
ls: /data/data/com.backmarket.troy.debug/files/com.backmarket.troy.debug/screengrab/fr-FR/images/screenshots: No such file or directory
[11:03:21]: Exit status: 1
ls: /data/data/com.backmarket.troy.debug/app_screengrab/fr-FR/images/screenshots: No such file or directory
[11:03:21]: Exit status: 1
ls: /data/data/com.backmarket.troy.debug/screengrab/fr-FR/images/screenshots: No such file or directory
[11:03:21]: Exit status: 1
ls: /data/user/0/com.backmarket.troy.debug/files/com.backmarket.troy.debug/screengrab/fr-BE/images/screenshots: No such file or directory
[11:03:21]: Exit status: 1
ls: /data/data/com.backmarket.troy.debug/files/com.backmarket.troy.debug/screengrab/fr-BE/images/screenshots: No such file or directory
[11:03:21]: Exit status: 1
ls: /data/data/com.backmarket.troy.debug/app_screengrab/fr-BE/images/screenshots: No such file or directory
[11:03:21]: Exit status: 1
ls: /data/data/com.backmarket.troy.debug/screengrab/fr-BE/images/screenshots: No such file or directory
[11:03:21]: Exit status: 1
[11:03:21]: Validating app APK
[11:03:21]: $ /Users/alizee.camarasa/Library/Android/sdk/build-tools/30.0.3/aapt dump permissions app/build/outputs/apk/developProd/screenshots/app-develop-prod-screenshots.apk
[11:03:21]: ▸ package: com.backmarket.troy.debug
[11:03:21]: ▸ uses-permission: name='android.permission.DISABLE_KEYGUARD'
[11:03:21]: ▸ uses-permission: name='android.permission.WAKE_LOCK'
[11:03:21]: ▸ uses-permission: name='android.permission.READ_EXTERNAL_STORAGE'
[11:03:21]: ▸ uses-permission: name='android.permission.WRITE_EXTERNAL_STORAGE'
[11:03:21]: ▸ uses-permission: name='android.permission.CHANGE_CONFIGURATION'
[11:03:21]: ▸ uses-permission: name='android.permission.VIBRATE'
[11:03:21]: ▸ uses-permission: name='android.permission.USE_FINGERPRINT'
[11:03:21]: ▸ uses-permission: name='android.permission.USE_BIOMETRIC'
[11:03:21]: ▸ uses-permission: name='android.permission.INTERNET'
[11:03:21]: ▸ uses-permission: name='android.permission.ACCESS_NETWORK_STATE'
[11:03:21]: ▸ uses-permission: name='android.permission.CAMERA'
[11:03:21]: ▸ uses-permission: name='android.permission.ACCESS_WIFI_STATE'
[11:03:21]: ▸ uses-permission: name='android.permission.BLUETOOTH'
[11:03:21]: ▸ uses-permission: name='android.permission.BLUETOOTH_ADMIN'
[11:03:21]: ▸ uses-permission: name='android.permission.CHANGE_NETWORK_STATE'
[11:03:21]: ▸ uses-permission: name='android.permission.CHANGE_WIFI_STATE'
[11:03:21]: ▸ uses-permission: name='android.permission.FLASHLIGHT'
[11:03:21]: ▸ uses-permission: name='android.permission.RECORD_AUDIO'
[11:03:21]: ▸ uses-permission: name='android.permission.WRITE_SETTINGS'
[11:03:21]: ▸ uses-permission: name='android.permission.ACCESS_COARSE_LOCATION'
[11:03:21]: ▸ uses-permission: name='android.permission.ACCESS_FINE_LOCATION'
[11:03:21]: ▸ uses-permission: name='android.permission.SYSTEM_ALERT_WINDOW'
[11:03:21]: ▸ uses-permission: name='android.permission.RECEIVE_BOOT_COMPLETED'
[11:03:21]: ▸ uses-permission: name='android.permission.FOREGROUND_SERVICE'
[11:03:21]: ▸ uses-permission: name='com.google.android.c2dm.permission.RECEIVE'
[11:03:21]: ▸ permission: com.backmarket.troy.debug.batch.permission.INTERNAL_BROADCAST
[11:03:21]: ▸ uses-permission: name='com.backmarket.troy.debug.batch.permission.INTERNAL_BROADCAST'
[11:03:21]: ▸ uses-permission: name='com.google.android.finsky.permission.BIND_GET_INSTALL_REFERRER_SERVICE'
[11:03:21]: $ /Users/alizee.camarasa/Library/Android/sdk/platform-tools/adb -s emulator-5554 shell pm list packages
[11:03:21]: ▸ package:com.android.cts.priv.ctsshim
[11:03:21]: ▸ package:com.google.android.youtube
[11:03:21]: ▸ package:com.android.internal.display.cutout.emulation.corner
[11:03:21]: ▸ package:com.google.android.ext.services
[11:03:21]: ▸ package:com.android.internal.display.cutout.emulation.double
[11:03:21]: ▸ package:com.android.providers.telephony
[11:03:21]: ▸ package:com.google.android.googlequicksearchbox
[11:03:21]: ▸ package:com.android.providers.calendar
[11:03:21]: ▸ package:com.android.providers.media
[11:03:21]: ▸ package:com.google.android.onetimeinitializer
[11:03:21]: ▸ package:com.google.android.ext.shared
[11:03:21]: ▸ package:com.android.documentsui
[11:03:21]: ▸ package:com.android.externalstorage
[11:03:21]: ▸ package:com.android.htmlviewer
[11:03:21]: ▸ package:com.android.companiondevicemanager
[11:03:21]: ▸ package:com.android.mms.service
[11:03:21]: ▸ package:com.android.providers.downloads
[11:03:21]: ▸ package:com.google.android.apps.messaging
[11:03:21]: ▸ package:com.google.android.configupdater
[11:03:21]: ▸ package:com.backmarket.troy.debug.test
[11:03:21]: ▸ package:com.android.defcontainer
[11:03:21]: ▸ package:com.android.providers.downloads.ui
[11:03:21]: ▸ package:com.android.vending
[11:03:21]: ▸ package:com.android.pacprocessor
[11:03:21]: ▸ package:com.android.simappdialog
[11:03:21]: ▸ package:com.android.internal.display.cutout.emulation.tall
[11:03:21]: ▸ package:com.android.certinstaller
[11:03:21]: ▸ package:com.android.carrierconfig
[11:03:21]: ▸ package:android
[11:03:21]: ▸ package:com.android.contacts
[11:03:21]: ▸ package:com.android.camera2
[11:03:21]: ▸ package:com.google.android.apps.inputmethod.hindi
[11:03:21]: ▸ package:com.android.egg
[11:03:21]: ▸ package:com.android.mtp
[11:03:21]: ▸ package:com.android.nfc
[11:03:21]: ▸ package:com.android.backupconfirm
[11:03:21]: ▸ package:com.google.android.deskclock
[11:03:21]: ▸ package:com.android.statementservice
[11:03:21]: ▸ package:com.google.android.gm
[11:03:21]: ▸ package:com.android.settings.intelligence
[11:03:21]: ▸ package:com.android.systemui.theme.dark
[11:03:21]: ▸ package:com.google.android.setupwizard
[11:03:21]: ▸ package:com.android.providers.settings
[11:03:21]: ▸ package:com.android.sharedstoragebackup
[11:03:21]: ▸ package:com.google.android.music
[11:03:21]: ▸ package:com.android.printspooler
[11:03:21]: ▸ package:com.android.dreams.basic
[11:03:21]: ▸ package:com.android.se
[11:03:21]: ▸ package:com.android.inputdevices
[11:03:21]: ▸ package:com.google.android.dialer
[11:03:21]: ▸ package:com.android.bips
[11:03:21]: ▸ package:com.android.musicfx
[11:03:21]: ▸ package:com.google.android.apps.docs
[11:03:21]: ▸ package:com.google.android.apps.maps
[11:03:21]: ▸ package:com.backmarket.design.showcase
[11:03:21]: ▸ package:com.android.internal.display.cutout.emulation.emu01
[11:03:21]: ▸ package:com.android.cellbroadcastreceiver
[11:03:21]: ▸ package:com.google.android.webview
[11:03:21]: ▸ package:com.android.server.telecom
[11:03:21]: ▸ package:com.google.android.syncadapters.contacts
[11:03:21]: ▸ package:com.backmarket.features.account.test
[11:03:21]: ▸ package:com.android.keychain
[11:03:21]: ▸ package:com.android.chrome
[11:03:21]: ▸ package:com.google.android.packageinstaller
[11:03:21]: ▸ package:com.google.android.gms
[11:03:21]: ▸ package:com.google.android.gsf
[11:03:21]: ▸ package:com.google.android.ims
[11:03:21]: ▸ package:com.google.android.tts
[11:03:21]: ▸ package:com.ustwo.lwp
[11:03:21]: ▸ package:com.android.calllogbackup
[11:03:21]: ▸ package:com.google.android.partnersetup
[11:03:21]: ▸ package:com.google.android.apps.wallpaper.nexus
[11:03:21]: ▸ package:com.google.android.videos
[11:03:21]: ▸ package:com.google.android.apps.nexuslauncher
[11:03:21]: ▸ package:com.android.carrierdefaultapp
[11:03:21]: ▸ package:com.backmarket.troy.debug
[11:03:21]: ▸ package:com.android.proxyhandler
[11:03:21]: ▸ package:com.breel.geswallpapers
[11:03:21]: ▸ package:org.chromium.webview_shell
[11:03:21]: ▸ package:com.google.android.feedback
[11:03:21]: ▸ package:com.google.android.printservice.recommendation
[11:03:21]: ▸ package:com.google.android.apps.photos
[11:03:21]: ▸ package:com.google.android.calendar
[11:03:21]: ▸ package:com.android.managedprovisioning
[11:03:21]: ▸ package:com.android.providers.partnerbookmarks
[11:03:21]: ▸ package:com.android.smspush
[11:03:21]: ▸ package:com.android.wallpaper.livepicker
[11:03:21]: ▸ package:com.google.android.sdksetup
[11:03:21]: ▸ package:com.google.android.backuptransport
[11:03:21]: ▸ package:com.android.storagemanager
[11:03:21]: ▸ package:com.android.bookmarkprovider
[11:03:21]: ▸ package:com.android.settings
[11:03:21]: ▸ package:com.google.android.inputmethod.pinyin
[11:03:21]: ▸ package:com.android.calculator2
[11:03:21]: ▸ package:com.android.cts.ctsshim
[11:03:21]: ▸ package:com.android.vpndialogs
[11:03:21]: ▸ package:com.google.android.apps.wallpaper
[11:03:21]: ▸ package:com.android.phone
[11:03:21]: ▸ package:com.android.shell
[11:03:21]: ▸ package:com.android.wallpaperbackup
[11:03:21]: ▸ package:com.android.providers.blockednumber
[11:03:21]: ▸ package:com.android.providers.userdictionary
[11:03:21]: ▸ package:com.android.emergency
[11:03:21]: ▸ package:com.android.location.fused
[11:03:21]: ▸ package:com.android.systemui
[11:03:21]: ▸ package:com.android.bluetoothmidiservice
[11:03:21]: ▸ package:com.android.traceur
[11:03:21]: ▸ package:com.android.bluetooth
[11:03:21]: ▸ package:com.android.providers.contacts
[11:03:21]: ▸ package:com.android.captiveportallogin
[11:03:21]: ▸ package:com.google.android.inputmethod.latin
[11:03:21]: ▸ package:com.google.android.apps.restore
[11:03:21]: Uninstalling app APK
[11:03:21]: $ /Users/alizee.camarasa/Library/Android/sdk/platform-tools/adb -s emulator-5554 uninstall com.backmarket.troy.debug
[11:03:21]: ▸ Success
[11:03:21]: Uninstalling tests APK
[11:03:21]: $ /Users/alizee.camarasa/Library/Android/sdk/platform-tools/adb -s emulator-5554 uninstall com.backmarket.troy.debug.test
[11:03:21]: ▸ Success
[11:03:21]: Installing app APK
[11:03:21]: $ /Users/alizee.camarasa/Library/Android/sdk/platform-tools/adb -s emulator-5554 install -t -r app/build/outputs/apk/developProd/screenshots/app-develop-prod-screenshots.apk
[11:03:21]: ▸ Performing Streamed Install
[11:03:24]: ▸ Success
[11:03:24]: Installing tests APK
[11:03:24]: $ /Users/alizee.camarasa/Library/Android/sdk/platform-tools/adb -s emulator-5554 install -t -r app/build/outputs/apk/androidTest/developProd/debug/app-develop-prod-debug-androidTest.apk
[11:03:24]: ▸ Performing Streamed Install
[11:03:25]: ▸ Success
[11:03:25]: Granting the permission necessary to change locales on the device
[11:03:25]: $ /Users/alizee.camarasa/Library/Android/sdk/platform-tools/adb -s emulator-5554 shell pm grant com.backmarket.troy.debug android.permission.CHANGE_CONFIGURATION
[11:03:25]: $ /Users/alizee.camarasa/Library/Android/sdk/platform-tools/adb -s emulator-5554 shell getprop ro.build.version.sdk
[11:03:25]: ▸ 28
[11:03:25]: Granting the permissions necessary to access device external storage
[11:03:25]: $ /Users/alizee.camarasa/Library/Android/sdk/platform-tools/adb -s emulator-5554 shell pm grant com.backmarket.troy.debug android.permission.WRITE_EXTERNAL_STORAGE
[11:03:25]: $ /Users/alizee.camarasa/Library/Android/sdk/platform-tools/adb -s emulator-5554 shell pm grant com.backmarket.troy.debug android.permission.READ_EXTERNAL_STORAGE
[11:03:25]: $ /Users/alizee.camarasa/Library/Android/sdk/platform-tools/adb -s emulator-5554 shell getprop ro.build.version.sdk
[11:03:25]: ▸ 28
[11:03:25]: $ /Users/alizee.camarasa/Library/Android/sdk/build-tools/30.0.3/aapt dump badging app/build/outputs/apk/developProd/screenshots/app-develop-prod-screenshots.apk
[11:03:25]: ▸ package: name='com.backmarket.troy.debug' versionCode='181' versionName='3.17.1-screenshots-prod' compileSdkVersion='30' compileSdkVersionCodename='11'
[11:03:25]: ▸ sdkVersion:'23'
[11:03:25]: ▸ targetSdkVersion:'30'
[11:03:25]: ▸ uses-permission: name='android.permission.DISABLE_KEYGUARD'
[11:03:25]: ▸ uses-permission: name='android.permission.WAKE_LOCK'
[11:03:25]: ▸ uses-permission: name='android.permission.READ_EXTERNAL_STORAGE'
[11:03:25]: ▸ uses-permission: name='android.permission.WRITE_EXTERNAL_STORAGE'
[11:03:25]: ▸ uses-permission: name='android.permission.CHANGE_CONFIGURATION'
[11:03:25]: ▸ uses-permission: name='android.permission.VIBRATE'
[11:03:25]: ▸ uses-permission: name='android.permission.USE_FINGERPRINT'
[11:03:25]: ▸ uses-permission: name='android.permission.USE_BIOMETRIC'
[11:03:25]: ▸ uses-permission: name='android.permission.INTERNET'
[11:03:25]: ▸ uses-permission: name='android.permission.ACCESS_NETWORK_STATE'
[11:03:25]: ▸ uses-permission: name='android.permission.CAMERA'
[11:03:25]: ▸ uses-permission: name='android.permission.ACCESS_WIFI_STATE'
[11:03:25]: ▸ uses-permission: name='android.permission.BLUETOOTH'
[11:03:25]: ▸ uses-permission: name='android.permission.BLUETOOTH_ADMIN'
[11:03:25]: ▸ uses-permission: name='android.permission.CHANGE_NETWORK_STATE'
[11:03:25]: ▸ uses-permission: name='android.permission.CHANGE_WIFI_STATE'
[11:03:25]: ▸ uses-permission: name='android.permission.FLASHLIGHT'
[11:03:25]: ▸ uses-permission: name='android.permission.RECORD_AUDIO'
[11:03:25]: ▸ uses-permission: name='android.permission.WRITE_SETTINGS'
[11:03:25]: ▸ uses-permission: name='android.permission.ACCESS_COARSE_LOCATION'
[11:03:25]: ▸ uses-permission: name='android.permission.ACCESS_FINE_LOCATION'
[11:03:25]: ▸ uses-permission: name='android.permission.SYSTEM_ALERT_WINDOW'
[11:03:25]: ▸ uses-permission: name='android.permission.RECEIVE_BOOT_COMPLETED'
[11:03:25]: ▸ uses-permission: name='android.permission.FOREGROUND_SERVICE'
[11:03:25]: ▸ uses-permission: name='com.google.android.c2dm.permission.RECEIVE'
[11:03:25]: ▸ uses-permission: name='com.backmarket.troy.debug.batch.permission.INTERNAL_BROADCAST'
[11:03:25]: ▸ uses-permission: name='com.google.android.finsky.permission.BIND_GET_INSTALL_REFERRER_SERVICE'
[11:03:25]: ▸ application-label:'RootBeerLib'
[11:03:25]: ▸ application-label-de:'RootBeerLib'
[11:03:25]: ▸ application-label-de-AT:'RootBeerLib'
[11:03:25]: ▸ application-label-en:'RootBeerLib'
[11:03:25]: ▸ application-label-en-GB:'RootBeerLib'
[11:03:25]: ▸ application-label-en-IE:'RootBeerLib'
[11:03:25]: ▸ application-label-es:'RootBeerLib'
[11:03:25]: ▸ application-label-fi:'RootBeerLib'
[11:03:25]: ▸ application-label-fr:'RootBeerLib'
[11:03:25]: ▸ application-label-fr-BE:'RootBeerLib'
[11:03:25]: ▸ application-label-it:'RootBeerLib'
[11:03:25]: ▸ application-label-nl:'RootBeerLib'
[11:03:25]: ▸ application-label-pt:'RootBeerLib'
[11:03:25]: ▸ application-icon-120:'res/mipmap-anydpi-v26/ic_launcher.xml'
[11:03:25]: ▸ application-icon-160:'res/mipmap-anydpi-v26/ic_launcher.xml'
[11:03:25]: ▸ application-icon-240:'res/mipmap-anydpi-v26/ic_launcher.xml'
[11:03:25]: ▸ application-icon-320:'res/mipmap-anydpi-v26/ic_launcher.xml'
[11:03:25]: ▸ application-icon-480:'res/mipmap-anydpi-v26/ic_launcher.xml'
[11:03:25]: ▸ application-icon-640:'res/mipmap-anydpi-v26/ic_launcher.xml'
[11:03:25]: ▸ application-icon-65534:'res/mipmap-anydpi-v26/ic_launcher.xml'
[11:03:25]: ▸ application: label='RootBeerLib' icon='res/mipmap-anydpi-v26/ic_launcher.xml'
[11:03:25]: ▸ application-debuggable
[11:03:25]: ▸ launchable-activity: name='com.backmarket.debug.ui.DebugActivity'  label='Troy: Let's debug!' icon='res/mipmap-anydpi-v26/ic_launcher_debug.xml'
[11:03:25]: ▸ uses-library-not-required:'org.apache.http.legacy'
[11:03:25]: ▸ feature-group: label=''
[11:03:25]: ▸   uses-gl-es: '0x20000'
[11:03:25]: ▸   uses-feature-not-required: name='android.hardware.bluetooth'
[11:03:25]: ▸   uses-feature-not-required: name='android.hardware.camera'
[11:03:25]: ▸   uses-feature-not-required: name='android.hardware.camera.autofocus'
[11:03:25]: ▸   uses-feature-not-required: name='android.hardware.fingerprint'
[11:03:25]: ▸   uses-feature-not-required: name='android.hardware.microphone'
[11:03:25]: ▸   uses-feature-not-required: name='android.hardware.sensor.accelerometer'
[11:03:25]: ▸   uses-feature-not-required: name='android.hardware.sensor.ambient_temperature'
[11:03:25]: ▸   uses-feature-not-required: name='android.hardware.sensor.barometer'
[11:03:25]: ▸   uses-feature-not-required: name='android.hardware.sensor.proximity'
[11:03:25]: ▸   uses-feature-not-required: name='android.hardware.sensor.relative_humidity'
[11:03:25]: ▸   uses-feature: name='android.hardware.faketouch'
[11:03:25]: ▸   uses-implied-feature: name='android.hardware.faketouch' reason='default feature for all apps'
[11:03:25]: ▸   uses-feature: name='android.hardware.location'
[11:03:25]: ▸   uses-implied-feature: name='android.hardware.location' reason='requested android.permission.ACCESS_COARSE_LOCATION permission, and requested android.permission.ACCESS_FINE_LOCATION permission'
[11:03:25]: ▸   uses-feature: name='android.hardware.screen.portrait'
[11:03:25]: ▸   uses-implied-feature: name='android.hardware.screen.portrait' reason='one or more activities have specified a portrait orientation'
[11:03:25]: ▸   uses-feature: name='android.hardware.wifi'
[11:03:25]: ▸   uses-implied-feature: name='android.hardware.wifi' reason='requested android.permission.ACCESS_WIFI_STATE permission, and requested android.permission.CHANGE_WIFI_STATE permission'
[11:03:25]: ▸ main
[11:03:25]: ▸ other-activities
[11:03:25]: ▸ other-receivers
[11:03:25]: ▸ other-services
[11:03:25]: ▸ supports-screens: 'small' 'normal' 'large' 'xlarge'
[11:03:25]: ▸ supports-any-density: 'true'
[11:03:25]: ▸ locales: '--_--' 'de' 'de-AT' 'en' 'en-GB' 'en-IE' 'es' 'fi' 'fr' 'fr-BE' 'it' 'nl' 'pt'
[11:03:25]: ▸ densities: '120' '160' '240' '320' '480' '640' '65534'
[11:03:25]: ▸ native-code: 'arm64-v8a' 'armeabi-v7a' 'x86' 'x86_64'
[11:03:25]: Running tests for locale: fr-FR
[11:03:25]: $ /Users/alizee.camarasa/Library/Android/sdk/platform-tools/adb -s emulator-5554 shell am instrument --no-window-animation -w
-e testLocale fr_FR
-e endingLocale en_US
-e appendTimestamp true
-e class com.backmarket.screenshots.tests.ScreenshotsTest
com.backmarket.troy.debug.test/androidx.test.runner.AndroidJUnitRunner
[11:03:40]: ▸ com.backmarket.screenshots.tests.ScreenshotsTest:INSTRUMENTATION_RESULT: shortMsg=Process crashed.
[11:03:40]: ▸ INSTRUMENTATION_CODE: 0
[11:03:40]: Pulling captured screenshots for locale fr-FR from the device
ls: /sdcard/com.backmarket.troy.debug/screengrab/fr-FR/images/screenshots: No such file or directory
[11:03:40]: Exit status: 1
ls: /sdcard/com.backmarket.troy.debug/screengrab/fr-BE/images/screenshots: No such file or directory
[11:03:40]: Exit status: 1
ls: /data/user/0/com.backmarket.troy.debug/files/com.backmarket.troy.debug/screengrab/fr-FR/images/screenshots: No such file or directory
[11:03:40]: Exit status: 1
ls: /data/data/com.backmarket.troy.debug/files/com.backmarket.troy.debug/screengrab/fr-FR/images/screenshots: No such file or directory
[11:03:40]: Exit status: 1
ls: /data/data/com.backmarket.troy.debug/app_screengrab/fr-FR/images/screenshots: No such file or directory
[11:03:40]: Exit status: 1
ls: /data/data/com.backmarket.troy.debug/screengrab/fr-FR/images/screenshots: No such file or directory
[11:03:40]: Exit status: 1
ls: /data/user/0/com.backmarket.troy.debug/files/com.backmarket.troy.debug/screengrab/fr-BE/images/screenshots: No such file or directory
[11:03:40]: Exit status: 1
ls: /data/data/com.backmarket.troy.debug/files/com.backmarket.troy.debug/screengrab/fr-BE/images/screenshots: No such file or directory
[11:03:41]: Exit status: 1
ls: /data/data/com.backmarket.troy.debug/app_screengrab/fr-BE/images/screenshots: No such file or directory
[11:03:41]: Exit status: 1
ls: /data/data/com.backmarket.troy.debug/screengrab/fr-BE/images/screenshots: No such file or directory
[11:03:41]: Exit status: 1
[11:03:41]: Make sure you've used Screengrab.screenshot() in your tests and that your expected tests are being run.
+------------------+---------------------+
|              Lane Context              |
+------------------+---------------------+
| DEFAULT_PLATFORM | android             |
| PLATFORM_NAME    |                     |
| LANE_NAME        | generateScreenshots |
+------------------+---------------------+
[11:03:41]: No screenshots were detected 📷❌
+------+------------------------------------+-------------+
|                    fastlane summary                     |
+------+------------------------------------+-------------+
| Step | Action                             | Time (in s) |
+------+------------------------------------+-------------+
| 1    | default_platform                   | 0           |
| 2    | Switch to installTestBuild lane    | 0           |
| 3    | installDevelopProdScreenshots      | 10          |
| 4    | installDevelopProdDebugAndroidTest | 6           |
| 💥   | screengrab                         | 20          |
+------+------------------------------------+-------------+
[11:03:41]: fastlane finished with errors
[!] No screenshots were detected 📷❌

Environment

 🚫 fastlane environment 🚫
Stack

Key
Value

OS
10.15.7

Ruby
2.7.3

Bundler?
false

Git
git version 2.24.3 (Apple Git-128)

Installation Source
/usr/local/Cellar/fastlane/2.186.0/libexec/bin/fastlane

Host
Mac OS X 10.15.7 (19H1217)

Ruby Lib Dir
<EMAIL_ADDRESS>

OpenSSL Version
OpenSSL 1.1.1k  25 Mar 2021

Is contained
false

Is homebrew
true

Is installed via Fabric.app
false

Xcode Path
/Library/Developer/CommandLineTools/

Xcode Version

Swift Version
5.3.2

System Locale

Error

No Locale with UTF8 found 🚫

fastlane files:
`./fastlane/Fastfile`
# This file contains the fastlane.tools configuration
# You can find the documentation at https://docs.fastlane.tools
#
# For a list of all available actions, check out
#
#     https://docs.fastlane.tools/actions
#
# For a list of all available plugins, check out
#
#     https://docs.fastlane.tools/plugins/available-plugins
#

# Uncomment the line if you want fastlane to automatically update itself
# update_fastlane

default_platform(:android)

lane :installTestBuild do
    gradle(
        task: "installDevelopProdScreenshots"
    )
    gradle(
        task: "installDevelopProdDebugAndroidTest"
    )
end

lane :generateScreenshots do
    #installTestBuild
    screengrab
    frame_screenshots(use_platform: "ANDROID")
end

`./fastlane/Appfile`
json_key_file("") # Path to the json secret file - Follow https://docs.fastlane.tools/actions/supply/#setup to get one
package_name("com.backmarket") # e.g. com.krausefx.app

fastlane gems

Gem
Version
Update-Status

fastlane
2.186.0
✅ Up-To-Date

Loaded fastlane plugins:
No plugins Loaded
Loaded gems

Gem
Version

did_you_mean
1.4.0

atomos
0.1.3

CFPropertyList
3.0.3

claide
1.0.3

colored2
3.1.2

nanaimo
0.3.0

xcodeproj
1.19.0

rouge
2.0.7

xcpretty
0.3.0

terminal-notifier
2.0.0

unicode-display_width
1.7.0

terminal-table
1.8.0

plist
3.6.0

public_suffix
4.0.6

addressable
2.7.0

multipart-post
2.0.0

word_wrap
1.0.0

tty-screen
0.8.1

tty-cursor
0.7.1

tty-spinner
0.9.3

artifactory
3.0.15

babosa
1.0.4

colored
1.2

highline
2.0.3

commander
4.6.0

excon
0.82.0

faraday-em_http
1.0.0

faraday-em_synchrony
1.0.0

faraday-excon
1.1.0

faraday-net_http
1.0.1

faraday-net_http_persistent
1.1.0

ruby2_keywords
0.0.4

faraday
1.4.2

unf_ext
<IP_ADDRESS>

unf
0.1.4

domain_name
0.5.20190701

http-cookie
1.0.4

faraday-cookie_jar
0.0.7

faraday_middleware
1.0.0

fastimage
2.2.4

gh_inspector
1.1.3

json
2.3.0

mini_magick
4.11.0

naturally
2.2.1

rubyzip
2.3.0

security
0.1.3

xcpretty-travis-formatter
1.0.1

dotenv
2.7.6

bundler
2.2.15

simctl
1.6.8

jwt
2.2.3

uber
0.1.0

declarative
0.0.20

trailblazer-option
0.1.1

representable
3.1.1

retriable
3.1.2

mini_mime
1.1.0

multi_json
1.15.0

signet
0.15.0

memoist
0.16.2

os
1.1.1

googleauth
0.16.2

httpclient
2.8.3

rexml
<IP_ADDRESS>

webrick
1.6.1

google-apis-core
0.3.0

google-apis-playcustomapp_v1
0.3.0

google-apis-androidpublisher_v3
0.6.0

google-cloud-env
1.5.0

google-cloud-errors
1.1.0

google-cloud-core
1.6.0

google-apis-iamcredentials_v1
0.4.0

google-apis-storage_v1
0.4.0

rake
13.0.3

digest-crc
0.6.3

google-cloud-storage
1.32.0

emoji_regex
3.2.2

jmespath
1.4.0

aws-partitions
1.470.0

aws-eventstream
1.1.1

aws-sigv4
1.2.3

aws-sdk-core
3.114.3

aws-sdk-kms
1.44.0

aws-sdk-s3
1.96.1

uri
0.10.0

forwardable
1.3.1

logger
1.4.2

cgi
0.1.0

date
3.0.0

timeout
0.1.0

stringio
0.1.0

ipaddr
1.2.2

openssl
2.1.2

zlib
1.1.0

ostruct
0.2.0

strscan
1.0.3

io-console
0.5.6

delegate
0.1.0

fileutils
1.4.1

singleton
0.1.0

open3
0.1.0

yaml
0.1.0

psych
3.1.0

mutex_m
0.1.0

generated on: 2021-06-28

My bad it works now, sorry for the disturbance


Change NettyRoutingFilter to use Webflux for obtaining the HttpClient

Is your feature request related to a problem? Please describe.
I would like to override the NettyRoutingFilter's getHttpClient method but need to use WebFlux in my override.   Attempting to use WebFlux in the getHttpClient creates an issue if you attempt to block while on a reactor thread.
Describe the solution you'd like
I would like to have a method that returns a Mono instead of HttpClient.

What does "Attempting to use WebFlux" mean?

What does "Attempting to use WebFlux" mean?

We have a Webflux Mono that is used to generate the HttpClient. The current getHttpClient method requires us to block in order to return the client. But blocking at this point causes an exception.

Closing in favor of #2993


testPacketContentAssemblerFlowsAndFiles.sh failed due to missing file

21 Jun 2018 06:31:20.351+0000 [17538] INFO #splapptrc,J[0],P[0],PacketStream,PacketFileSource M[PacketStream.cpp:processPCAPfile:171]  - opening PCAP file '/mnt/scratch/data.garanti-bank/Ayca_Test_wireshark.pcap'
21 Jun 2018 06:31:20.354+0000 [17538] ERROR #splapptrc,J[0],P[0],PacketStream,#splapptrc,J[0],P[0],PacketStream,spl_operator M[PEImpl.cpp:handleOperatorFailure:661]  - CDISR5030E: An exception occurred during the execution of the PacketStream operator. The exception is: error opening PCAP file '/mnt/scratch/data.garanti-bank/Ayca_Test_wireshark.pcap', /mnt/scratch/data.garanti-bank/Ayca_Test_wireshark.pcap: No such file or directory

test script must not reference files that are not part of test environment


In the MySQL optimized query split SELECT / UPDATE

Given the following index on the delayed_jobs table:
CREATE INDEX delayed_job_priority ON delayed_jobs(priority,run_at);
MySQL will ignore this index, and does not support hints when doing an
UPDATE SET ... LIMIT 1; Without an index, the ordering is done via a
filesort, and can be problematic when the delayed_jobs table
becomes large (>10,000 records). This results in an increasingly long
system lock wait time to reserve the job.
By separating the SELECT from the UPDATE, we can take advantage of an
INDEX while fetching the row to lock for a reservation.
More information on SELECT ... FOR UPDATE here:
http://dev.mysql.com/doc/refman/5.7/en/innodb-locking-reads.html
More information on the issue at:
https://mrfrosti.com/2016/06/06/delayed-job-performance-tuning/

This is basically what Delayed::Backend::ActiveRecord.configuration.reserve_sql_strategy = :default_sql does. This strategy increases contention while running more workers. We introduced the setting to allow people to chose based on how they run things. Different people have different requirements and there is no solution that works best for all.

@albus522 Thanks for the quick response. Interesting that the :default_sql strategy increases worker contention. How many workers are typically running when contention begins to be an issue? To clarify, the :optimize_sql better lends itself to smaller table sizes and more workers, while :default_sql is better for larger table sizes with less workers?

One more question about the worker contention - is this applicable even with the index on delayed_jobs(priority,run_at)? Seems like the :default_sql strategy would benefit from this where the :optimize_sql strategy could not?

Yes, generally optimize is better for most scenarios except super large jobs tables, which is most users, and default is the old style which has its own quirks including increasing worker contention. The worker contention comes from the locking, no other worker can do anything until the lock is released. The default strategy tries to partially offset this by not locking but returning a set of potential jobs then the worker tries to lock each until one succeeds, but if enough workers are active you end up with them all fighting to get jobs instead of working on jobs.
Large jobs tables are usually, but not always, a result of creating A LOT of fast jobs the workers are just expected to slam through fast. This is NOT what DJ is optimized for. DJ is optimized for running jobs that take too long to simply run inline, for which the runtime is way more than the lock time even if the lock is a little slower.


Error Loading BLKINDEX.dat

Experiencing a "error loading blkindex.dat"
Currently using OSX Sierra and previously had access the wallet on Jan 24th. Updated to the 1/30 dmg file and still no dice.

you just need to go to your verge data directory ... delete out the blkindex.dat , i think there's more than 1 ., maybe like blkindex001.dat  and blkindex002.dat ... you can delete those.
Go to verge-blockchain.com and download / unzip the latest bootstrap to your data directory
https://verge-blockchain.com/
Just be very careful in your data directory.  do NOT delete your wallet.dat ... and you should have backups of your wallet.dat if you don't already .. you need to make a backup and copy it to a jump drive or something :smile:

check out #642  as well.  Different issue but exactly the same resolution steps apply for you.

Let me do some tinkering and I will report back. I know I have tried downloading the March 23rd and 24th blockchain files and replacing them. That is what lead to the first error message.
Since my post I reinstalled verge qt. Deleted my old verge data folder and put in my wallet.dat file. It’s been naturally syncing since but I imagine this will take a very very long time.

Cool -- in the future you don't need to really do a 'reinstall' of verge qt... it can't hurt... but the data directory is going to be the problem 99% of the time.
I'm glad you are smart and know how to do the wallet.dat
I would close out of Verge-QT ... download the blockchain ... unzip it to your data directory -- and then re-open verge-qt.
There's absolutely no harm in doing it your way though and letting it sync naturally.  As you said -- it just might take a day or two.  It should work fine if you have the time to wait !

While I wait to redownload march 24th, I wanted to add that my folder does not contain a
blkindex001.dat but only a blk0001.dat

Ya sorry I'm not in front of my wallet at the moment... blk0001.dat and blk0002.dat is right sorry !

wouldn't hurt to delete your peers.dat as well after you unzip.

Did everything like we discussed except for deleting PEERS.DAT file. Reached 100% and received error loading blkindex.dat

Can I close out Verge, delete the peers.dat file and try that?

Update. Tried the deletion of the peers.dat file and still receiving the blkindex error once it hits 100%

just to make sure we are on the same page here ... this is what you are doing:
step 1)  Delete everything in data directory (except wallet.dat)
step 2)  download server3_QT-Wallet_v4.x_Verge-Blockchain_2018-March-24.zip from verge-blockchain.com
step 3)  unzip server3_QT-Wallet_v4.x_Verge-Blockchain_2018-March-24.zip into your data directory
step 4)  copy wallet.dat back to data directory
Step 5)  Run verge-qt
regarding step 3... in that file , it has these files in it right:
blkindex.dat
blk0001.dat
blk0002.dat ?
There should be a db.log or debug.log in your verge data directory, could look through those and/or post here.

and to clarify -- you said this which has me a little confused:

Reached 100% and received error loading blkindex.dat

What do you mean by this?  How did you "reach 100%?"  While Verge-qt was open and running?  and While it was open and running did that error pop up ?  Or did you:
Opened verge-qt -> got to 100% -> closed verge-qt -> opened verge-qt -> now got the error

Only other possibilities I can think of at this time:
~The current bootstrap might be corrupt -- wait until next version is uploaded later today / tomorrow and try that
~your virus scan or another app is locking down blkindex.dat
I'd hate to suggest you let it sync for the next day or 2 from 0 blocks and then end up with the same error.

unfortunately #610 and #95 are similar issues,  but neither user ever got back to us and posted their solution.
justin did suggest in one of them to let it sync from block 0

I did see this on reddit as well, looks intense:
https://www.reddit.com/r/vergecurrency/comments/7sbext/verge_qt_wallet_help_w_mac_os_high_sierra/

There is not a blkindex.dat files or a 0002.dat files. Just a blk0001.dat file, database,peers,readme,txlevel

You are correct, I'm home now and just checked.  From the reddit link i posted -- copied and pasted into a better format... you might want to delete blkindex.dat (only that one) , and then try these steps...
VergeQt 4.0.1 fork for stealth / Mac OS High Sierra Instructions if not working.
1.Backup your wallet.dat file to a safe place, found in: go,library,application support,Verge.
2.Now go to Finder bottom left corner, type terminal, open terminal.
3.copy & paste: xcode-select —install
4.press enter
5.install this then back onto terminal
6.copy & paste: ruby -e "$(curl -fsSL https://raw.githubusercontent.com/Homebrew/install/master/install)"
7.press enter
8.press enter again
9.Type in password used for admin or to log in laptop, as you type you won’t see it.
10.press enter
11.copy & paste: brew doctor
12.press enter
13.copy & paste: brew install boost
14.press enter
15.now go to open 4.0.1 fork for stealth allow & open.

There is not a blkindex.dat in the folder either. is that normal?

it probably tries to create, but you get the 'error loading blkindex.dat' and it goes away.
in addition to those steps -- if you wanted to -- when you open verge-qt -- watch your data directory and see if blkindex.dat is created and grows in size,, up to the point of the error .. and then disappears.
I still think the steps above will resolve you though.

All those things already said they were installed (I remember doing this a few months back). Would my logs help

Definitely will help !

debug.log

db.log is empty and has 0 size

Btw there doesn't seem to be blkindex.dat at any point in the folder

LoadBlockIndex(): hashBestChain=000000000000b61f87b61723ac774dc6cddda3a022ad7c59d5ea5698ad47c5a6  height=1977491  trust=1977492  date=03/23/18 23:59:32
LoadBlockIndex(): synchronized checkpoint 00000fc63692467faeb20cdb3b53200dc601d75bdfa1001463304cc790d77278
Verifying last 2500 blocks at level 1
ERROR: CBlock::ReadFromDisk() : OpenBlockFile failed
ERROR: LoadBlockIndex() : block.ReadFromDisk failed

Saw this on a bitcoin github a few years back .. something about file descriptors .. i'm not a Mac guru so
https://github.com/bitcoin/bitcoin/issues/2712
saw this post with some suggestions on increasing file descriptors
https://superuser.com/questions/302754/increase-the-maximum-number-of-open-file-descriptors-in-snow-leopard
will keep looking

have also seen posts about Anti-Virus being enabled causes problems with the I/O ... did you try disabling anti-virus temporarily ?

saw this as well:
https://cryptocurrencytalk.com/topic/27625-cant-sync-mac-wallet-for-3-weeks-now/
you can start with post 15 and then go to page 2.. but coincidentally he refers to the same article i posted:
https://superuser.com/questions/302754/increase-the-maximum-number-of-open-file-descriptors-in-snow-leopard

keep in mind that all QT Wallets are basically based on the same code -- so when doing google searches -- don't feel restricted to typing "verge qt".  You can google for the problem with "bitcoin qt" and find exact same resolutions.

I don't believe I have anti-virus. I did just disable my icloud. hopefully that helps or fixes.

Also I should have Verge QT stealth transactions version installed correct upload jan 30. there is also a jan 8th version that is under 4.0.1.
Trying to see which one to be working on..

Seems to be hitting a dead end and naturally letting it sync seems like it’ll take a lot more than 2-3 days

Is there any tips or tricks to sync faster? Currently 13 active connections

Sadly no ... just have to suffer through it.  the "trick" is to use a bootstrap to catch up .. but we can't use that trick :(

Hi, I have exactly the same problem. I hadn’t opened the wallet for over a month and last night it was taking an age to update so thought I would just just use the latest bootstrap files.
Now I get the exact same error also I know another person who yesterday did the same thing and also having the exact same issue. Are the downloaded files must be corrupt?
Looks like several of us have the same problems. Just for you info.
I have done it before without any problems.

Hi @jrsikors, I've been having the same problem but with windows 10.
Im really new to all this but this is what happened.
I downloaded the wallet 4.0.1 from the Verge website, when I opened it it was out of sync and had so many blocks to load (at this point i still didn't know i could download the blockchain) anyways i waited nearly 2 days for all the blocks to download and with the wallet open everytime it was getting close to finishing it went back to around 70%.
While this was happening I tried to send all my XVG to this unsynced wallet to an address i generated from there (not sure whether this is ok or not ...)
I started searching online and saw i could download the blockchain, so i downloaded "Wallet_v4.x_Verge-Blockchain_2018-March-25" (copied my wallet to safe location because i thought there should be the coins i previously sent) unzipped it in the APPDATA then i try to open the Wallet APP but once it finished loading the blockchain index it gives me "error loading blkindex.dat".
I tried doing this various times also deleting the Verge folder in the APPDATA and tried again with the 26 March Blockchain. I tried to unzip the blockchain inside the folder and overwrite the files already there and also tried deleting all the files in the folder and just unzipping the blockchain in there. I tried to disable my antivirus. Attached below the log:
debug.log
As i've said im very new to all this and not very good with computers so I might have done something stupid along the way, but i looked at multiple videos of people doing this operation and everything went all right

So other people have started reporting same problems as well -- with the march 26th bootstrap.
I'm currently testing with the march 28th bootstrap.. but will be a while.
I still think you should continue your sync from scratch -- it might get sync'd quicker than a potential bootstrap issue is identified and resolved.
Sorry man -- was unfortunate you were the first to run into problems !

@pthai112
#657
@jetpippo
#658
consolidating blkindex.dat problems to one issue for easier updates/tracking.
I have notified relevant people to let them know of a potential problem with the bootstrap.
The only workaround is to download the blockchain from scratch (0 blocks) ... it might take a couple days to sync, which we know is brutal.  There's no ETA on when a bootstrap issue will be identified and resolved.  In other words, it might be quicker to take 2-3 days to sync instead of waiting for a bootstrap.

When messing around with your data directory -- remember to back up your wallet.dat to a safe place .. jump drive ideally, somewhere off computer.
To get a "new" data directory, the safest thing is to go rename the old data directory.  IE:
C:\Users\YOU\AppData\Roaming\Verge
Rename the Verge folder to VergeOld
then start verge-qt again -- and it will create you a brand new data directory.  remember to copy your wallet.dat back to the new data directory eventually.
If you need immediate access to your funds before the QT fully syncs, the best thing to do is those steps ... get your wallet.dat in there... open verge-qt :

go to debug consoole
type walletpassphrase 'YOUR PASSPHRASE HERE BETWEEN SINGLE QUOTES' 600
dumpprivkey YOURVERGEADDRESS

Now you can take that private key and import it into any other verge wallet like Electrum.

per @swat69 -- he identified issues with recent bootstraps.  He will work to resolve.  No ETA though, stand by !

new file blk0002.dat missing in zip, fixed.
new up-to-date bootstrap 28th ready for download now

Thanks for the quick turnaround @swat69 .
Everyone should go try out the new download at:
https://verge-blockchain.com/
Again -- make sure you back up wallet.dat
Although probably not necessary .. I would strongly advise clearing out your data directory first (except wallet.dat).
The safe way is to rename verge data folder to verge_old
But the quicker way is to:
-delete EVERYTHING in your Verge DataDirectory EXCEPT WALLET.DAT (default is c:\users\YOU\appdata\roaming\verge).
-download bootstrap from verge-blockchain.com
-unzip
-run verge-qt

@pthai112 see previous post from me

now i have Timed out waiting for onion hostname. I have disabled firewall and run qt as admin, nothing helps.

definitely a different issue -- are you running any other applications using Tor?
Does removing the blkdata1.dat and blkdata002.dat make your client not time out ?

If you still have your old data directory -- or it's in the recycle bin -- you could always try copying the old
DATADIR\tor folder into the new datadir , and overwrite/replace the new tor folder files.
If the tor folder doesn't exist, even better

I don´t know why but after several tries, it worked. Strange behaviour, I tried it 5 times and everytime the error comes up. Between I also cleaned everything with cc cleaner, restarted my laptop and the error comes again. After the 6th try, it worked now and my wallet synced. I have nothing changed between the tries...strange. Thanks anyway

I’ll try when I get home. Currently at 30-40 percent (synced)

no prob @omdespers  , glad it worked.   It's not entirely surprising /alarming.  I was actually going to suggest you try to run it like 10x .. and figured it would eventually find something.  Tor can be wonky like that

Guys you rock!
Confirmed working for me now. Thanks for your support and quick turnaround!

@jrsikors @swat69
Dear Jeff and Ben,
I just woke up downloaded the new bootstrap, put my wallet in the APPDATA and everything loaded perfectly after I waited some 10 minutes. Thank you so much for your help have an awesome day!

Just downloaded the Blockchain for today. (btw it seems to mislabeled as March 29th..). Anyhow..
WORKS LIKE A CHARM!!!! Thanks guys!

@jrsikors
Hi Jeff. I just download and updated QT-Wallet_v4.x_Verge-Blockchain_2018-March-29.zip and result is good. However my coin don't show, I also resys 2 times but result is same. I checked and found coin in verge block chain explorer . How to display it on Verge wallet software?

Are you sure you got the right wallet.dat in there?
go to the help -> debug console
type:
scanforalltxns
just that, no number after.  It will take about 15-20 minutes to re-scan.  Don't close client it will finish.

@jrsikors
Hi.
Thank your for your guide. Now coin show on wallet.
So problem:
1.QT-Wallet_v4.x_Verge-Blockchain_2018-March-29.zip to update fully sysr
2.Type: scanforalltxns to show coin.

going to close this one up -- sounds like almost everyone is good with the latest bootstrap.
It probably updated from 28th to 29th ... website says its updates every 00:00 with a new copy.. so i think we are good there.
Please feel free to post a new issue if anyone is still having problems with anything


fixed invalid table name that caused bm25_refresh to fail

#13

Can you fix the DCO according to the instruction?

Can you fix the DCO according to the instruction?

Done


Update scripts

sorry wrong repo :)
closing
sorry, anyway there some updates witch can be helpful

Thank you for your contribution.
Actually, I have a plan to update this since there's some improvement of the workflows and documentation but I didn't have time to update 🥲


Update dependencies

Updates all dependencies except ahash since no-one else seems to be using the latest version

No one's updated ahash cause this never got fixed lol: https://github.com/tkaitchuck/aHash/issues/95


update JQuery version to 3.5.1

I don't think you tested this change since you created this commit using the GitHub web interface (and didn't even paste the full file, since the first line with the version number hasn't change)...

I also find a difference between the jquery in this patch and 3.5.1 downloaded from jquery itself (not sure where as it is hard to parse, but the diff engine suggests they aren't the same). I've therefore opened #1806. Thanks for this update and the issue.


Travis targets are outdated

NodeJS <10 have reached EOL and 10 is already to be scheduled to be EOL in April 2021.
Travis testing only covers 4, 6 & 8 at the moment, so it might be time to update this to ensure nothing breaks on newer versions of node (testing on older versions doesn't have to be removed, it might actually be good).

Well.. That's just dandy.


Widget property fix

Converted Scale, Scrollbar and ScrolledWindow to use new property system as per Issue #47 and Issue #49.

Rebased off current master HEAD.


Issue with stan math functions

I was double checking the code and I only found this issue because my local machine has a newer version of this vs the production machine I'm using at work.
I've isolated the issue and it's very perplexing to me as this should be the same. However, the answer is wrong.
// currently we have
// L is lower-triangular
// J = (L^-1)^T
// x = (J * J^T) * g0 = (L^-1)^T * (L^-1) * g0 = G^-1 * g0
auto L = stan::math::cholesky_decompose(G);
auto J = stan::math::transpose(stan::math::mdivide_left_tri_low(L));
auto x = multiply(stan::math::multiply_lower_tri_self_transpose(J), g0);

// vs just computing the inverse
// auto x = multiply(inverse(G), g0);

I've attached an example where the output should be 0.7, 0.3 and I've confirmed this with the R package quadprogpp and the old version using multiply(inverse(G), g0);. The "new" wrong version gives 1 in variable places for this test.
It's easy for us to revert back to using the inverse solution but I hope this isn't a bigger bug in Stan math so I'm bringing it up here.
K <- 8
J <- 20
true_v <- c(0.2, 0.5, 0.3, rep(0, K - 3))
true_w <- c(0.7, 0.3, rep(0, J - 2))
X0 <- matrix(rnorm(K * J, 0, sd = 1), J, K)
X1 <- apply(X0, 2, function(x) t(true_w) %*% (x ))
beta <- true_v * 10 + rnorm(K, 0, 0.1)
Y0 <- X0 %*% beta + rnorm(20, 0, 0.1)
Y1 <- beta %*% X1 + rnorm(1, 0, 0.1)
data =  list(
  K = K,
  J = J,
  X1 = matrix(X1, K, 1),
  X0 = t(X0),
  Y0 = as.vector(Y0),
  Y1 = c(Y1),
  CE = t(t(rep(1, J))),
  ce0 = array(-1),
  CI =  cbind(diag(J), -diag(J)),
  ci0 =c(rep(0, J), rep(1, J)),
  v_prior = abs(beta)
)
K <- K
J <- J
X1 <- matrix(X1, K, 1)
X_qp <- t(X0)
Y0 <- as.vector(Y0)
Y1 <- c(Y1)
CE <- t(t(rep(1, J)))
ce0 <- array(-1)
CI <-  cbind(diag(J), -diag(J))
ci0 <-c(rep(0, J), rep(1, J))
v_prior <- abs(beta)
G <- t(X_qp) %*% diag(v_prior/sum(v_prior)) %*% X_qp
g0 <- -1 * c(t(X1) %*% diag(v_prior/sum(v_prior)) %*% X_qp )
library(cmdstanr)
set_cmdstan_path("~/cmdstan")
set.seed(12345)
mod <- cmdstan_model("~/cmdstan/test_for_rok.stan")
# mod <- cmdstan_model("~/cmdstan/quadprog_test.stan")

fit <- mod$sample(
  data = data,
  seed = 987083,
  chains = 2,
  parallel_chains = 2,
  iter_warmup = 200,
  iter_sampling = 200,
  max_treedepth = 10,
  adapt_delta = 0.8
)

test_for_rok.stan.zip

This would be better to check directly in C++. Can you give me an example of G and g0 for which this does not match? I can then make simple unit tests.

You can use from the R script above. We don't even need the v thing.
G <- t(X_qp) %*% X_qp
g0 <- -1 * c(t(X1) %*% X_qp

I pushed updates that fixed issues but I couldn't do what I originally wanted which should be mathematically correct.
I wanted to use the Cholesky factor L and take the inverse then do multiply lower tri self transpose to get the inverse of G. In the C++ code it is
auto L = stan::math::cholesky_decompose(G);
auto J = stan::math::transpose(stan::math::mdivide_left_tri_low(L));
auto x = multiply(stan::math::multiply_lower_tri_self_transpose(J), g0)

this does not work! Even though it should, see
PD_mat <-  matrix(
              c(1.00, 0.25, 0.10, 0.40,
              0.25, 1.00, 0.80, 0.00,
              0.10, 0.80, 1.00, -0.05,
              0.40, 0.00, -0.05,  1.00),
              nrow = 4, ncol = 4, byrow = T)
J <- chol(G_test) # R returns upper tri so don't need transpose like in Eigen
all.equal(solve(G_test), tcrossprod(solve(J)))
[1] TRUE

Here's a nice testing script.
K <- 8
J <- 20
true_w <- c(0.7, 0.3, rep(0, J - 2))
X0 <- matrix(rnorm(K * J, 0, sd = 1), J, K)
X1 <- apply(X0, 2, function(x) t(true_w) %*% (x ))
beta <- true_v * 10 + rnorm(K, 0, 0.1)
Y0 <- X0 %*% beta + rnorm(20, 0, 0.1)
Y1 <- beta %*% X1 + rnorm(1, 0, 0.1)

K <- K
J <- J
X1 <- matrix(X1, K, 1)
X_qp <- t(X0)
Y0 <- as.vector(Y0)
Y1 <- c(Y1)
CE <- t(t(rep(1, J)))
ce0 <- array(-1)
CI <-  cbind(diag(J), -diag(J))
ci0 <-c(rep(0, J), rep(1, J))

G <- t(X_qp) %*% X_qp
g0 <- -1 * c(t(X1) %*% X_qp )

data =  list(
  K = K,
  J = J,
  CE = CE,
  ce0 =  ce0,
  CI =  CI,
  ci0 = ci0,
  G = G + diag(.000001, J),
  g0 = g0
)
library(cmdstanr)
set_cmdstan_path("solve_qp/cmdstan")
set.seed(12345)
mod <- cmdstan_model("solve_qp/test_for_rok.stan", force_recompile = TRUE)
fit <- mod$sample(
  data = data,
  chains = 2,
  parallel_chains = 2,
  iter_warmup = 200,
  iter_sampling = 200,
  max_treedepth = 10,
  adapt_delta = 0.8
)

fit$summary("w_optim")$mean
fit$summary("w_chol")$mean
# should return 0.7 0.3 0.0 0.0 0.0 0.0 0.0 0.0 0.0 0.0 0.0 0.0 0.0 0.0 0.0 0.0 0.0 0.0 0.0 0.0

test_for_rok.stan
data {
    int<lower=1> K;             // num predictors
    int<lower=0> J;             // num countries
    matrix[J, 1] CE;            // CE for predictor country
    vector[1] ce0;              // CE for target country
    matrix[J, 2 * J] CI;        // CI for predictor country
    vector[2 * J] ci0;          // CI for target country
    matrix[J, J] G;
    vector[J] g0;
}
parameters {
    real<lower=0> sigma;
}
transformed parameters {
    vector[J] w_optim = round(100 * solve_quadprog(G, g0, CE, ce0, CI, ci0))/100;
    vector[J] w_chol = round(100 * solve_quadprog_chol(cholesky_decompose(G), g0, CE, ce0, CI, ci0))/100;
}
model {
  sigma ~ std_normal();
}


Fix for max_concurrent_streams issue - Call mark_stream_closed before…

Fix for max_concurrent_streams issue - Call mark_stream_closed before…

NOT READY FOR REVIEW YET

Corrupt JSON data (indicates timeout or crash):
    bm_fullstack_trickle.BM_PumpUnbalancedUnary_Trickle_256k_256k_4M.opt.new: 4
    bm_fullstack_trickle.BM_PumpUnbalancedUnary_Trickle_16M_16M_4M.opt.new: 4
    bm_fullstack_trickle.BM_PumpUnbalancedUnary_Trickle_256k_256k_64M.opt.new: 4
    bm_fullstack_trickle.BM_PumpUnbalancedUnary_Trickle_256k_256k_256k.opt.new: 4
    bm_fullstack_trickle.BM_PumpUnbalancedUnary_Trickle_1_256k_64M.opt.new: 4
    bm_fullstack_trickle.BM_PumpUnbalancedUnary_Trickle_1_256k_4M.opt.new: 4
    bm_fullstack_trickle.BM_PumpUnbalancedUnary_Trickle_1_16M_4M.opt.new: 4
    bm_fullstack_trickle.BM_PumpUnbalancedUnary_Trickle_16M_16M_256k.opt.new: 4
    bm_fullstack_trickle.BM_PumpUnbalancedUnary_Trickle_1_16M_256k.opt.new: 4
    bm_fullstack_trickle.BM_PumpUnbalancedUnary_Trickle_1_256k_16k.opt.new: 4
    bm_fullstack_trickle.BM_PumpUnbalancedUnary_Trickle_256k_256k_16k.opt.new: 4
    bm_fullstack_trickle.BM_PumpUnbalancedUnary_Trickle_1_16M_64M.opt.new: 4
    bm_fullstack_trickle.BM_PumpUnbalancedUnary_Trickle_1_256k_256k.opt.new: 4
    bm_fullstack_trickle.BM_PumpUnbalancedUnary_Trickle_16M_16M_64M.opt.new: 4

[trickle] No significant performance differences

[trickle] No significant performance differences

[microbenchmarks] Performance differences noted:
Benchmark                                                    cpu_time    real_time
-----------------------------------------------------------  ----------  -----------
BM_HpackParserParseHeader<NonIndexedBinaryElem<100, false>>  +4%         +5%

[trickle] No significant performance differences

[microbenchmarks] No significant performance differences

Jenkins: test this please

Known Issues
Basic Tests C & C++ Linux [opt] : #11737
Basic Tests Linux (opt) : #11737
Cloud To Prod Interop : #10763
Msan C (Internal CI) : #12307
UBsan C (Internal CI) : #11528

[trickle] No significant performance differences

[microbenchmarks] No significant performance differences

The change is much cleaner and smaller now. Earlier, I had added the mark_stream_closed much earlier than necessary which was causing the fail_pending_writes to result in issues. At the current placement of mark_stream_closed in writing.c, there is no need for changing mark_stream_closed or bring out fail_pending_writes :)

Much cleaner, nice. re-LGTM

Jenkins: retest this please

[trickle] No significant performance differences

[trickle] No significant performance differences

Corrupt JSON data (indicates timeout or crash):
    bm_fullstack_streaming_ping_pong.BM_StreamingPingPongWithCoalescingApi_InProcessCHTTP2_NoOpMutator_NoOpMutator__2M_2_0.opt.old: 1
    bm_fullstack_streaming_ping_pong.BM_StreamingPingPongWithCoalescingApi_InProcessCHTTP2_NoOpMutator_NoOpMutator__2M_1_0.counters.old: 1

[microbenchmarks] No significant performance differences

Known issues : #12527, #11055, #11737, #12307


fix: Lambda requires extra line before End Of File

fixes #96228
It seems that when parsing code GDScriptTokinazer was generating 2xToken::DEDENT at end instead of Token::DEDENT + TOKEN::TK_EOF. This was because lambdas have their own indent stack (indent_stack_stack) and it moved to default indent_stack after lambda parse, inside  pop_expression_indented_block method. As a result, this caused problems.
Meanwhile, indent_stack is clear in check_indent() method, when end of file reached:
https://github.com/godotengine/godot/blob/6395450b10d73bc3515763c7bbd1b2f5b7425d10/modules/gdscript/gdscript_tokenizer.cpp#L1173-L1178
I would have liked to reset indent_stack_stack here as well, but pop_expression_indented_block has error about empty stack.
https://github.com/godotengine/godot/blob/6395450b10d73bc3515763c7bbd1b2f5b7425d10/modules/gdscript/gdscript_tokenizer.cpp#L295-L299
So I had do it there.

I added an extra commit by removing clear and replacing it with pop_back. This is because in the process I came across a bug in the solution, when another lambda is called inside one lambda and the bracket in the expression is not closed and not extra line before EOF:
func test(function1: Callable) -> void:
	function1.call()

func _ready():
	test(func(): test(func(): print("hello"))

If lambdas are called within each other, the unclosed parenthesis at the end of the expression, in addition to the expected Expected closing ')' after calling arguments error, will also report an empty lambda stack error from pop_expression_indented_block:
https://github.com/godotengine/godot/blob/6395450b10d73bc3515763c7bbd1b2f5b7425d10/modules/gdscript/gdscript_tokenizer.cpp#L295-L299

Commits needs to be squashed (see https://docs.godotengine.org/en/latest/contributing/workflow/pr_workflow.html#modifying-a-pull-request)

@Chaosus fixed.
But, PR don't pass static checks with file formatting. It require newline at end of file, but this issue (#96228) about miss newline at end of file. And that's we need for regression. What to do?

But, PR don't pass static checks with file formatting. It require newline at end of file, but this issue (https://github.com/godotengine/godot/issues/96228) about miss newline at end of file. And that's we need for regression. What to do?

If it's a test for error, you need to create it at tests/scripts/runtime/errors (look on other tests in that category as a reference).

If it's a test for error, you need to create it at tests/scripts/runtime/errors (look on other tests in that category as a reference).

No, this is not a error test. There shouldn't be any errors right now. But the style checker wants a empty line at end of the GDScript file. And if I add that line, the test is meaningless. Because it worked before with an empty line too.
Issue #96228 about GDScript file without an empty line after lambda expression.

I understood. Maybe the other developers knows how to fix it: cc @AThousandShips, cc @akien-mga. If it's impossible, then a test should probably just be removed.

You need to add an exception for this file in the modules/gdscript/tests/scripts/.editorconfig file. I also suggest renaming the file as I put it in this diff to be a bit clearer / more concise:
diff --git a/modules/gdscript/tests/scripts/.editorconfig b/modules/gdscript/tests/scripts/.editorconfig
index 3a877f36f0..d4f466328f 100644
--- a/modules/gdscript/tests/scripts/.editorconfig
+++ b/modules/gdscript/tests/scripts/.editorconfig
@@ -1,6 +1,9 @@
 # Some tests handle invalid syntax deliberately; exclude relevant attributes.
 # See also the `file-format` section in `.pre-commit-config.yaml`.

+[parser/features/lambda_at_eof_without_newline.gd]
+insert_final_newline = false
+
 [parser/features/mixed_indentation_on_blank_lines.gd]
 trim_trailing_whitespace = false

@akien-mga I did this, but Style Checker is still not happy. So additionally I add test script to exclusions for file-format in pre-commit config.

Why close?

@Zireael07 oops, it's my bad. I trying pull and merge new commits to my repository. A little confused)

I changed the tests and I think it need re-review @akien-mga

test_gdscript.cpp. I needed to run tests for the GDScriptTokenizerBuffer and use .gdc files to do so. However, the following code was problematic and caused parsing errors:

https://github.com/godotengine/godot/blob/0f95e9f8e665cedaa5aace5f6f86de2b4b5a8c60/modules/gdscript/tests/test_gdscript.cpp#L306-L310
ERROR: Condition "total_len > 0" is true. Returning: ERR_INVALID_DATA
All because of the addition of the last element. So I removed it. And it work fine. After that, added a branching and overloaded method for test_parse to start binary parsing.

gdscript_test_runner.cpp. I needed to generate a .gdc file for the tests, so I added a tool for that. I don't know if it will be needed. But I needed it :)


Make authenticated requests to backstage backend from third party

Backstage documentation explains how two plugins can easily communicate with each other, but is there a way I can call backstage backend APIs from an external service to programmatically access my data in Backstage?
For e.g. using the API http://localhost:7007/api/catalog/entities?filter=kind=API to list all the APIs.

Sure. These are just regular REST APIs with no particular magic. In particular the catalog api is distinctly meant to be usable by other services or consumers.

@freben But what about authentication for the same? Surely the deployed the instances which a organisation is using won't have open access for these APIs, right?

@DefCon-007 We'll get proper docs for this in place, but for now you'll want to replicate this: https://github.com/backstage/backstage/blob/9370194ce9e52b8abeeda06539b67bb34b323c54/packages/backend-common/src/tokens/ServerTokenManager.ts#L85
So create a JWT with the payload {"sub":"backstage-server"} and sign it with the shared symmetric (oct) key that you configure in the Backstage backend.
Bit of a warning that the combination of backend authentication and external requests is still unstable and that we're likely to change the token format, for example adding an expiration, so you'll need to keep an eye out for that in the changelog.


Add enhanced X509 Certificate/Private RSA Key validation to Traffic Ops (AddSSLKeys Endpoint)

Which issue is fixed by this PR? If not related to an existing issue, what does this PR do?
Partially Fixes #3267
All imported x509 certificates / private key pairs will be validated with the following criteria:

All PEM encoded strings contained in the CRT and KEY fields must be decodable to DER binary format via pem.Decode()
A single PEM encoded private RSA key must be decodable from KEY payload.  KEY format must be PKCS #1 or PKCS #8
The import server X509 certificate in the CRT payload is checked for the appropriate Public Key Algorithm (x509.RSA is only supported currently)
Private RSA Key modulus (value N) is compared against the x509 cert RSA Public Key modulus value N.  If both values do not match, x509 certificate and key are rejected with httpStatus 4XX.
Original certificate Chain validation functionality has been left in-place.
Ability to import x509v3 Certificates without SKI/AKI has been left in-place.

Which TC components are affected by this PR?

[ ] Documentation
[ ] Grove
[ ] Traffic Analytics
[ ] Traffic Monitor
[X] Traffic Ops
[ ] Traffic Ops ORT
[ ] Traffic Portal
[ ] Traffic Router
[ ] Traffic Stats
[ ] Traffic Vault
[ ] Other _________

What is the best way to verify this PR? Please include manual steps or automated tests.
(If no tests are part of this PR, please provide explanation as to why no tests are included.)
Tests to perform for validation:

Import x509 certificate with mismatched RSA Private Key

Expected Result: x509 certificate and key are rejected and not saved in traffic vault.

Import x509 certificate with encrypted RSA Private Key

Expected Result: x509 certificate and key are rejected and not saved in traffic vault.

Attempt to import ECC x509 certificate and ECDSA private key

Expected Result: x509 certificate and key are rejected and . not saved in traffic vault.

Import self-signed certificate for a delivery service with matching Private RSA key.

Optional: Startup CiaB with this commit cherry-picked in.

Run the unit tests in deliveryservice/keys_test.go

=== RUN   TestDecodePrivateKey_PKCS8_RSA_2048
--- PASS: TestDecodePrivateKey_PKCS8_RSA_2048 (0.00s)
=== RUN   TestDecodePrivateKey_PKCS1_RSA_2048
--- PASS: TestDecodePrivateKey_PKCS1_RSA_2048 (0.00s)
=== RUN   TestDecodePrivateKey_RSA_Encrypted_And_BadData
--- PASS: TestDecodePrivateKey_RSA_Encrypted_And_BadData (0.00s)
=== RUN   TestVerifyAndEncodeCertificate_BadData
--- PASS: TestVerifyAndEncodeCertificate_BadData (0.00s)
=== RUN   TestVerifyAndEncodeCertificate_SelfSigned_NO_SKIAKI_CertKeyPair
--- PASS: TestVerifyAndEncodeCertificate_SelfSigned_NO_SKIAKI_CertKeyPair (0.15s)
=== RUN   TestVerifyAndEncodeCertificate_SelfSignedCertKeyPair
--- PASS: TestVerifyAndEncodeCertificate_SelfSignedCertKeyPair (0.00s)
=== RUN   TestVerifyAndEncodeCertificate_SelfSignedCertKeyPair_MisMatched_PrivateKey
--- PASS: TestVerifyAndEncodeCertificate_SelfSignedCertKeyPair_MisMatched_PrivateKey (0.00s)
=== RUN   TestVerifyAndEncodeCertificate_CASignedCertKeyPair
--- PASS: TestVerifyAndEncodeCertificate_CASignedCertKeyPair (0.00s)
=== RUN   TestVerifyAndEncodeCertificate_CASigned_NO_SKIAKI_CertKeyPair
--- PASS: TestVerifyAndEncodeCertificate_CASigned_NO_SKIAKI_CertKeyPair (0.00s)
=== RUN   TestVerifyAndEncodeCertificate_CASigned_NO_SKIAKI_CertKeyPair_With_RootCA
--- PASS: TestVerifyAndEncodeCertificate_CASigned_NO_SKIAKI_CertKeyPair_With_RootCA (0.00s)
=== RUN   TestVerifyAndEncodeCertificate_CASignedCertKeyPair_With_RootCA
--- PASS: TestVerifyAndEncodeCertificate_CASignedCertKeyPair_With_RootCA (0.00s)
=== RUN   TestVerifyAndEncodeCertificate_ECC_CertificateKeyPair
--- PASS: TestVerifyAndEncodeCertificate_ECC_CertificateKeyPair (0.00s)
PASS
ok  	command-line-arguments	(cached)

Check all that apply

[ ] This PR includes tests
[ ] This PR includes documentation updates
[ ] This PR includes an update to CHANGELOG.md
[ ] This PR includes all required license headers
[ ] This PR includes a database migration (ensure that migration sequence is correct)
[ ] This PR fixes a serious security flaw. Read more: www.apache.org/security

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/trafficcontrol-PR/3306/
Test PASSed.

If this PR only partially fixes that issue, you should change "Fixes" to e.g. "Addressed" - As-is, GitHub's going to auto-close the issue when this gets merged.

Thanks for reviewing this PR @ocket8888 @rob05c. I'm working on getting these requested changes in today.

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/trafficcontrol-PR/3326/
Test FAILed.

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/trafficcontrol-PR/3328/
Test FAILed.

I'm adding some extra logic at the end of the certificate unit tests that check for correct number of certs that should be in the cert chain.  The unit tests  that test verification of x509v3 certificates that contain Subject/Authority Key identifiers are not output the correct number of certs in the bundle.

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/trafficcontrol-PR/3329/
Test PASSed.

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/trafficcontrol-PR/3330/
Test PASSed.

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/trafficcontrol-PR/3333/
Test FAILed.

I have forced pushed a branch that combines PR #3382 with PR #3417 and adds ECDSA support for DNS delivery services.
TODO:

Add in the SQL query that limits ECDSA certs to DNS delivery services only.
Fix the unit tests for the ECC certificate/key pair import use case.
Fix up the docs to be 'go doc' format.

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/trafficcontrol-PR/3423/
Test PASSed.

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/trafficcontrol-PR/3424/
Test PASSed.

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/trafficcontrol-PR/3425/
Test PASSed.

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/trafficcontrol-PR/3426/
Test PASSed.

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/trafficcontrol-PR/3427/
Test PASSed.

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/trafficcontrol-PR/3428/
Test PASSed.

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/trafficcontrol-PR/3429/
Test PASSed.

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/trafficcontrol-PR/3430/
Test PASSed.

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/trafficcontrol-PR/3431/
Test PASSed.

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/trafficcontrol-PR/3432/
Test PASSed.

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/trafficcontrol-PR/3433/
Test PASSed.

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/trafficcontrol-PR/3434/
Test PASSed.

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/trafficcontrol-PR/3435/
Test PASSed.

Latest branch provides the following changes:

Merge in PR #3417
Adds ECDSA support for only DNS* delivery service types.
Verification that both RSA and ECDSA private keys match the corresponding submitted x509 certificate. RSA verification ensures the public modulus N value matches the modulus N value in the x509 certificate. ECDSA verification ensures the curve set {name, public-X, public-Y} curve values are equal to those in the x509 certificate.
There are 27 unit tests that verify both x509v3 and x509v1 certificate/key combinations (look at keys_test.go). Yes I'm aware the documentation is lacking. There is a fair amount of comments, but the documentation is lacking. It doesn't make sense for me to write a ton of docs if this PR is rejected.  I can update the docs once the PR is deemed viable.
Verification that ECDSA keys are only permitted on DNS* delivery services explicitly.
Reject all x509 + DSA certificates because no modern HTTPS client supports it

@rawlinp I pushed four commits that resolve all comments you raised from yesterday's review.

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/trafficcontrol-PR/3444/
Test PASSed.


napi: add async API

https://github.com/thepassle/module-graph/pull/16/files#diff-e727e4bdf3657fd1d798edcd6b099d6e092f8573cba266154583a746bba0f346R175
Made it slower

https://github.com/oxc-project/oxc-resolver/pull/191


Weekday in "Custom Date"

Requirements

[X] I have updated Mushroom to the latest available version
[X] I did a search to see if there is a similar issue or if a pull request is open.

Is your feature request related to a problem?
I would like to see only the weekday instad of dd.mm.yyy, because is too long
Describe the solution you'd like
I think that the custom date format have to accept also ddd for the weekday
Describe alternatives you've considered
No response
Additional context
No response

Please implement this.
I've never seen someone restrict a date formatter in such a way. Please allow all possible date formatter options that are supported by DateTime :)

Solved in #405


Add prettier

This PR adds prettier along with supporting changes to make this integration automatic. This includes creating a pre-commit task and vscode integration.
Closes #3

The following command was ran to add the prettier pre-commit hook via husky: npx mrm@2 lint-staged. This command also generated several of the files changed in this commit.


add custom data to an element

Hi, pretty package you have here!
I am looking to use this flowchart editor as a 'workflow' editor for BPMN type of workflow for just the basics.
Therefore it would be nice to be able to add custom data to connections and elements,
When the generated flowchart could be loaded into the flutter flowbuilder dart package, we could use this flowchart package as a business process modelling tool.
any thoughts about this?
Regards,
Hans

Hi @hansbak,
I think this is feasible. I think we discussed a similar question in the #13 issue. But I don't know if I could do this shortly.
Thanks for your feedback! We could leave this issue open to remember later.

no need can add a selection in the external menus.


set self.worker_memory to None when appropriate

We currently have a misconfigured SLURM setup where we can't specify the memory.
Therefore using dask_jobqueue always leads to a Memory specification can not be satisfied error, even when memory=None. This PR fixes that problem.

Thanks @basnijholt - this seems fine to me. Any objections from others?

I'm going to go ahead and merge.  Thanks @basnijholt !
I'm fine with ternary if.  It looks like we don't yet include flake8 in our test suite.  I'm inclined to handle this more generally by adding a test to the CI matrix rather than troubling people on each PR.


Reduce target framework version

At the moment we target, I think, .net 45. I want to reduce the target framework version to .net 35.
Reducing the target framework version will increase the availability of the library.

IsEmail uses EmailAddressAttribute from System.ComponentModel.DataAnnotations which was added in .net 40. The IsEmail methods needs to be refactored before reducing the target framework version.


python38Packages.dash-html-components: 1.1.3 -> 1.1.4

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for python38Packages.dash-html-components is: "HTML components for Dash"
meta.homepage for python38Packages.dash-html-components is: "https://dash.plot.ly/dash-html-components"
meta.changelog for python38Packages.dash-html-components is: ""
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS

Warning: a test defined in passthru.tests did not pass

0 of 0 passed binary check by having a zero exit code.

0 of 0 passed binary check by having the new version present in output.

found 1.1.4 with grep in /nix/store/mmlqm01nn7k93a6501lnyq6i90223fm4-python3.8-dash_html_components-1.1.4

found 1.1.4 in filename of file in /nix/store/mmlqm01nn7k93a6501lnyq6i90223fm4-python3.8-dash_html_components-1.1.4

directory tree listing: https://gist.github.com/54aeb4053875f1a93b8ab71cf1644df7

du listing: https://gist.github.com/a2de2b94fddd8180432457b4f30a249b

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

16 total rebuild path(s)

4 package rebuild(s)

4 x86_64-linux rebuild(s)
4 i686-linux rebuild(s)
4 x86_64-darwin rebuild(s)
4 aarch64-linux rebuild(s)

First fifty rebuilds by attrpath
python38Packages.dash
python38Packages.dash-html-components
python39Packages.dash
python39Packages.dash-html-components

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/mmlqm01nn7k93a6501lnyq6i90223fm4-python3.8-dash_html_components-1.1.4 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A python38Packages.dash-html-components https://github.com/r-ryantm/nixpkgs/archive/7d0232a51f57f327276af78bc9da3a05482f5b4a.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/mmlqm01nn7k93a6501lnyq6i90223fm4-python3.8-dash_html_components-1.1.4
ls -la /nix/store/mmlqm01nn7k93a6501lnyq6i90223fm4-python3.8-dash_html_components-1.1.4/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
nixpkgs-review took longer than 45m and timed out

Maintainer pings
cc @antoinerg for testing.

Result of nixpkgs-review pr 133810 at 7d0232a5 run on x86_64-linux 1

2 packages failed to build:

python38Packages.dash
python39Packages.dash

2 packages built successfully:

python38Packages.dash-html-components
python39Packages.dash-html-components

1 suggestion:

warning: python-include-tests
Consider adding a checkPhase for tests, or if not feasible, pythonImportsCheck.
Near pkgs/development/python-modules/dash-html-components/default.nix:19:0:
   |
19 |     description = "HTML components for Dash";
   | ^

Note that build failures may predate this PR, and could be nondeterministic or hardware dependent.
Please exercise your independent judgement. Does something look off? Please file an issue or reach out on IRC.

#133866


chef_config really should take a hash of options rather than a string

This PR make it optionally do so.  Even worked once for me!

This is now significantly more tested, huzzah


htarg and ftarg MUST be dicts

Coverage decreased (-0.002%) to 99.933% when pulling 6cbfeb921fddd941a7db9e5cec0f92bd7ee233c4 on hankel-fourier into 7a9437ca893b272cf58ac1eff20a420a8f213a6d on v2-dev.

Coverage decreased (-0.002%) to 99.933% when pulling 6cbfeb921fddd941a7db9e5cec0f92bd7ee233c4 on hankel-fourier into 7a9437ca893b272cf58ac1eff20a420a8f213a6d on v2-dev.


Add the dustmaps package to the build environment

We don't have dustmaps as part of the distributed system yet, but metric-pipeline-tasks depends on it.  This adds that package and downloads the necessary data.

jenkins-dm-jobs is a bit of an outlier.  Until we have a good way for Groovy developers to test their code (or avoid having Groovy developers at all), we'll have to let this repo be a bit more "cowboy".  I'd still like u/ branches, though, to be somewhat consistent with other repos and processes.


MockRasterCache lacks plumbing to determine if layer subtrees were properly rendered

While fixing https://github.com/flutter/engine/pull/22674 I was looking into writing a more explicit test where we could EXPECT that the proper leaf nodes were rendered into the cache. Unfortunately, MockRasterCache plugs into the RasterCache machinery at such a high level that we bypass the code that was making a mistake here.
I started reimplementing MockRasterCache to have finer grained hooks into the caching process, but I ended up with dozens of lines of code that would refactor the way we cache images. The task was large enough that I thought that it should be done as a separate set of changes.
The goal would be for MockRasterCache to substitute caching surfaces with working SkCanvas implementations that would record what was rendered to the cache and be able to report back on that afterwards for EXPECT testing in a unit test.

The raster cache is a legacy feature at this point (no plans on adding it to impeller) so I'm going to close this.


Added initialPosition and launchMode to settings schema

Summary of the Pull Request
Added initialPosition and launchMode to settings json schema and md file

References

PR Checklist

[x] Closes #3290
[x] CLA signed. If not, go over here and sign the CLA
[ ] Tests added/passed
[ ] Requires documentation to be updated
[x] I've discussed this with core contributors already. If not checked, I'm ready to accept this work might be rejected in favor of a different grand plan. Issue number where discussion took place: #xxx

Detailed Description of the Pull Request / Additional comments

Validation Steps Performed

I know the regex for the coordinates is wrong, currently working to figure it out.


WIP: removing httpretty

I've added travis, coverage, fixed the setup and renamed the folder.
I'll be adding commits to remove httpretty, but wanted to check that travis installs correctly.

What's the plan for this and #4 & #5?

Tests here are gonna fail cause I did not completely remove httpretty. I did multiple PR to keep the size of the PR small, so review this one, then #4 and in #5 all tests should pass. I should've done the PR to a different branch (not master) but since I'm expecting to get this merged fairly fast I don't think it is an issue. What do you think?

Got it, thanks. Looks good.


support admin api operations

Summary
In addition to just supporting code instrumentation, it would be useful to be able to configure Sentry:

projects CRUD
teams CRUD
client key CRUD
alerts CRUD

Motivation
We are setting up IaC where each microservice gets its own Sentry project. We have some internal code to support CRUD operations, but it would be nice to have upstream support.
Additional Context
We have some pre-existing code we'd be willing to contribute. Is this something you would be willing to accept?

What are Admin UI operations?

I'm talking about the API to configure Sentry
https://docs.sentry.io/api/

Ahh, I see. This is something outside of the scope of the SDK. Sorry.

Why are you opposed to adding support? I'm not asking you to do it, I'm offering to contribute


PR: Improve documentation for "Sharing Data Between Workers" section

Motivation/Description of the PR

Improves documentation of "Sharing Data Between Workers" in docs/parallel.md

Applicable helpers:

[ ] WebDriver
[ ] Puppeteer
[ ] Nightmare
[ ] REST
[ ] FileHelper
[ ] Appium
[ ] Protractor
[ ] TestCafe
[ ] Playwright

Applicable plugins:

[ ] allure
[ ] autoDelay
[ ] autoLogin
[ ] customLocator
[ ] pauseOnFail
[ ] puppeteerCoverage
[ ] retryFailedStep
[ ] screenshotOnFail
[ ] selenoid
[ ] stepByStepReport
[ ] wdio

Type of change

[ ] :fire: Breaking changes
[ ] :rocket: New functionality
[ ] :bug: Bug fix
[ ] :clipboard: Documentation changes/updates
[ ] :hotsprings: Hot fix
[x] :hammer: Markdown files fix - not related to source code
[ ] :nail_care: Polish code

Checklist:

[ ] Tests have been added
[x] Documentation has been added (Run npm run docs)
[ ] Lint checking (Run npm run lint)
[ ] Local tests are passed (Run npm test)

Removed local: true option from default example, because the headline is "Sharing Data Between Workers" and it is confusing that the example will only work within same worker. That took 1-2 hours from my life to realize this. I added the description of the opt-in feature {local: true } to the bottom of the chapter as an extra.
Fixed a bug in line 224. From my point of view it will only work if you share with object that includes key of object you want to share.
Some grammar changes


Question about FQDN vs CNAME fields in Dangling DNS script

Hi everyone,
At my company we've been reading up on Azure's recommendations for how to detect dangling DNS records, and came across this repository. I'm curious if there's a specific reason why the FQDN value is being assigned to the CNAME field for a given record set. Here is one example, but it occurs in multiple places, including in the final logic for determining if a given record is dangling or not.
Thanks for your time!

Hello @Gleland
Thank you for the feedback. We are looking into this issue and we will provide a feedback shortly

@gleland The script scans only CNAME records in the zone. The CNAME field for a records is where the target for the CNAME record is populated. This is why that field is being read everywhere in the code.

@abhijeetgaiha Thanks for the quick response! I'm not sure I understand:

If the CNAME record is the field being checked, why does the script rename it to FQDN in a fwe places?
Shouldn't the script actually be checking for the FQDN resource itself (as mentioned in Azure's dangling record documentation)?

The script is doing exactly what you describe.
Like I said, the CNAME field is where the target of the CNAME record is stored, which would be the FQDN of the resource.

Ah! @abhijeetgaiha Thank you! I understand now. The canonical name is indeed what the script is referencing when checking for resources. Thank you for clarifying this! For any future readers, these two pages helped me with some more background reading:

https://en.wikipedia.org/wiki/CNAME_record#Details
https://datatracker.ietf.org/doc/html/rfc2181#section-10.1.1

I'll close this, thank you for your help!

@gleland Glad to help!


VisualStudio 2019 2022 plugin

Great extension for Vs-code, is it possible or difficult to port this to visual studio 2019 2022.
Thank you in advance

@harrism76 I'm guessing it can easily be re-written as a Visual Studio extension. Calling an external program (cppinsights) and opening the diff-view should be easy. But it would be a re-write rather than a direct port.