AND and OR in IF statement

It seems that AND and OR is not supported now.
We need this feature to make our condition statements much simpler.
if (a && b)
if (a && b && c ...)
if (a || b)
if (a || b || c ...)

若「甲」且「乙」者。
若「甲」且「乙」且「丙」者。
若「甲」或「乙」者。
若「甲」或「乙」或「丙」者。

Mixing AND and OR can be considered later.
related:  #365 Integration suggestion for else-if statement

Hi @cuixiping , thanks for bringing this up.
AND|OR can already be percolated with
夫「甲」「乙」中有陽乎。

and
夫「甲」「乙」中無陰乎。

They can be then immediately followed by 若其者 to use in the if statement, which seems convenient enough.
Since wenyan-lang does not (and will not) have brackets, ambiguity will be introduced if multiple AND|OR are put in the if statement. One possibility is that we allow only EITHER && OR || in a single if statement. Feel free to discuss :)

One possibility is that we allow only EITHER && OR || in a single if statement.

I strongly agree this.
since 夫「甲」「乙」中有陽乎/無陰乎 only accept two identifiers, we can not do
若夫「甲」大於一。「甲」小於十。中無陰乎者。

We need
若「甲」大於一且「甲」小於十者。


AttributeError: module 'torch' has no attribute 'concat'

When running "inference_example", the cell that suppose to return LCA coefficients results in error "AttributeError: module 'torch' has no attribute 'concat'". It has likely to do with PyTorch version on my side since they have the page for this method (https://pytorch.org/docs/stable/generated/torch.concat.html) but it might tell us the minimal version to specify on the dependencies list
UPD: Before I had version 1.9.0. The error is given after installing 1.11.0

this is weird because the torch.concat() doc page is supposedly for 1.11.0.
just curious, does torch.cat() work?

I have not documented it properly but as far as I could recall - yes torch.cat() worked.


update_dart_sdk.ps1 not digitally signed on Windows

Is there an existing issue for this?

[X] I have searched the existing issues
[X] I have read the guide to filing a bug

Steps to reproduce

Change the script execution policy using Group Policy (requires Windows Pro)

Execute any flutter commands (e.g., flutter doctor)

The previous issues were closed due to inactivity:

#50934
#39692

Expected results
The PowerShell script should be signed to work on Windows machines with policies restricting executing signed scripts only
Actual results
The script flutter\bin\internal\update_dart_sdk.ps1 is not digitally signed. Upon executing Flutter commands like flutter doctor or flutter upgrade the error would show
Code sample
Code sample
flutter doctor

Screenshots or Video

Screenshots / Video demonstration
[Upload media here]

Logs
Logs
& : File C:\flutter\bin\internal\update_dart_sdk.ps1 cannot be loaded. The file
C:\flutter\bin\internal\update_dart_sdk.ps1 is not digitally signed. You cannot run this script on the
current system. For more information about running scripts and setting execution policy, see about_Execution_Policies
at https:/go.microsoft.com/fwlink/?LinkID=135170.
At line:1 char:81
+ ... sdk.ps1'; & 'C:\flutter\bin\internal\update_dart_sdk.ps1' ...
+                 ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : SecurityError: (:) [], PSSecurityException
    + FullyQualifiedErrorId : UnauthorizedAccess

Flutter Doctor output
Doctor output
> flutter doctor
Checking Dart SDK version...
Downloading Dart SDK from Flutter engine b20183e04096094bcc37d9cde2a4b96f5cc684cf...
& : File C:\flutter\bin\internal\update_dart_sdk.ps1 cannot be loaded. The file
C:\flutter\bin\internal\update_dart_sdk.ps1 is not digitally signed. You cannot run this script on the
current system. For more information about running scripts and setting execution policy, see about_Execution_Policies
at https:/go.microsoft.com/fwlink/?LinkID=135170.
At line:1 char:81
+ ... sdk.ps1'; & 'C:\flutter\bin\internal\update_dart_sdk.ps1' ...
+                 ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : SecurityError: (:) [], PSSecurityException
    + FullyQualifiedErrorId : UnauthorizedAccess
Building flutter tool...
Running pub upgrade...
Resolving dependencies... (3.0s)
Got dependencies.
Doctor summary (to see all details, run flutter doctor -v):
[✓] Flutter (Channel stable, 3.13.2, on Microsoft Windows [Version 10.0.22621.2134], locale en-US)
[✓] Windows Version (Installed version of Windows is version 10 or higher)
[✓] Android toolchain - develop for Android devices (Android SDK version 33.0.2)
[✓] Chrome - develop for the web
[✓] Visual Studio - develop Windows apps (Visual Studio Community 2022 17.7.2)
[✓] Android Studio (version 2022.1)
[✓] IntelliJ IDEA Ultimate Edition (version 2023.1)
[✓] VS Code (version 1.81.1)
[✓] Connected device (1 available)
[✓] Network resources

• No issues found!

Thanks for the report. Currently I don't have Windows Pro edition of the machine to replicate / verify this, but based on the report, keeping it open for team's input / attention and tracking.

cc @loic-sharma

cc @godofredoc long-term we may want to consider doing Windows codesigning


remove duplicate code

Closes #44
@thomas-muench Thanks for pointing out this code duplication. I don't think executing the statement twice is necessary. Let's see if the tests are still green.
[Edit: The tests are green. Feel free to merge this PR whenever you like.]

Thanks for looking into this!


Polish log

What's the purpose of this PR
XXXXX
Which issue(s) this PR fixes:
Fixes #
Brief changelog
XXXXX
Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Read the Contributing Guide before making this pull request.
[ ] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
[ ] Write necessary unit tests to verify the code.
[ ] Run mvn clean test to make sure this pull request doesn't break anything.
[ ] Update the CHANGES log.

Codecov Report

Merging #3882 (75ff290) into master (2c9b4dc) will increase coverage by 0.01%.
The diff coverage is 33.33%.

@@            Coverage Diff            @@
##             master    #3882   +/-   ##
=========================================
  Coverage     50.13%   50.14%
- Complexity     2477     2478    +1
=========================================
  Files           485      485
  Lines         14983    14984    +1
  Branches       1532     1532
=========================================
+ Hits           7511     7513    +2
+ Misses         6939     6938    -1
  Partials        533      533

Impacted Files
Coverage Δ

...ip/framework/apollo/mockserver/EmbeddedApollo.java
92.77% <0.00%> (ø)

...rk/apollo/spring/property/SpringValueRegistry.java
83.78% <50.00%> (+0.45%)
:arrow_up:

.../apollo/internals/RemoteConfigLongPollService.java
77.71% <0.00%> (+0.60%)
: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 2c9b4dc...75ff290. Read the comment docs.


Feature/toast invalid value options 666

Description
[Describe what this change achieves]
The below features are implemented in these changes:

Default axes selected on Data config value options
Error toast If the user selected any invalid or empty axis
Error toast if the user trying to save the visualization, If f user selected any invalid or empty axis

Added these features for existing as well as new charts including Gauge, Histogram, and Treemap.
Issues Resolved
[List any issues this PR will resolve]
#666
Check List

[ ] New functionality includes an Error toast message on invalid value options selection while Apply and Save click, snapshot test cases, cypress test cases.

[ ] All tests pass, including unit test, and integration test, except a few test cases are failing at my end which are not related to this functionality.

[ ] Commits are signed per the DCO using --signoff

The below listed files changes are respective to this PR and rest are showing after rebasing the main to the feature branch:
.../.cypress/integration/event_analytics.spec.js
.../common/constants/shared.ts
.../common/types/explorer.ts
.../public/components/custom_panels/helpers/tests/snapshots/utils.test.tsx.snap
.../public/components/event_analytics/explorer/explorer.tsx
.../public/components/event_analytics/explorer/visualizations/config_panel/tests/snapshots/config_panel.test.tsx.snap
.../public/components/event_analytics/explorer/visualizations/config_panel/tests/config_panel.test.tsx
.../public/components/event_analytics/explorer/visualizations/config_panel/config_panel.tsx
.../public/components/event_analytics/explorer/visualizations/index.tsx
.../public/components/visualizations/charts/helpers/viz_types.ts
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Adding @spattnaik @abasatwar for reference.

As there are few main changes included after rebasing main, so closing this and raising a new PR with respective changes.


Too many requests

Description
When opening multiple applications one at a time, I get too many requests on the oauth2 flow and the error Internal error in some tabs
Reproduction steps
Click on the applications one after another
Expected behavior
All the tabs should have a loaded application
Screenshots/Videos
Too many errors

Internal error

Tested on
Mozilla/5.0 (X11; Ubuntu; Linux x86_64; rv:109.0) Gecko/20100101 Firefox/111.0
Version
2023.3.0-dev.5

this issue will be addressed in https://github.com/cortezaproject/corteza/issues/493

The issue was fixed


Better treebank detokenizer for punctuations

Originally, before this PR, the punctuations simply did the reverse of the tokenization regexes and that's results in erroneous left/right shifts.
It's because during tokenization it's safe to add more spaces but during detokenization, simply undoing the padding doesn't really help.
Before this PR:
>>> from nltk.tokenize.treebank import TreebankWordDetokenizer
>>> twd = TreebankWordDetokenizer()
>>> toks = ['hello', ',', 'i', 'ca', "n't", 'feel', 'my', 'feet', '!', 'Help', '!', '!']
>>> twd.detokenize(toks)
"hello,i can't feel my feet!Help!!"

After:
>>> from nltk.tokenize.treebank import TreebankWordDetokenizer
>>> toks = ['hello', ',', 'i', 'can't", 'feel', 'my', 'feet', '!', 'Help', '!', '!']
>>> twd = TreebankWordDetokenizer()
>>> twd.detokenize(toks)
"hello, i can't feel my feet! Help!!"

>>> toks = ['hello', ',', 'i', 'ca', "n't", 'feel', ';', 'my', 'feet', '!', 'Help', '!', '!', 'He', 'said', ':', 'Help', ',', 'help', '?', '!']
>>> twd.detokenize(toks)
"hello, i can't feel; my feet! Help!! He said: Help, help?!"

[CI: retestt]

Thanks @alvations


Intellisense support for Angularjs 1.4.5

Can you please add support for Angularjs 1.4.5 ?

This is definitely a need, but unfortunately is not a high priority for me right now. If you experiment with the debug logging and can determine where Intellisense is breaking down, please leave your notes. Or better yet, create a pull request with a fix and I will incorporate it.

That would be great. Will try that. +1

Is there any way to debug angular.intellisense.js?

There is not a way to do debugging proper, but you can enable logging. Try adding the following to the top of the JavaScript file you hope to get Intellisense for.
_$AngularJS_VisualStudio_Intellisense.setLogLevelVerbose();

I'll try thnx. I am using angular.intelisense.js working on angular projects and still have some problems with intellisense. I would like to help but i have no idea for now how to help you )


Add tag inputs for endpoints

It would be nice to be able to add additional tags to endpoint resources (ie. those created from enable_s3_endpoint, enable_kms_endpoint, enable_dynamodb_endpoint, etc.)

Hey @Velgus there is the vpc_endpoint_tags variable to add tags to additional tags to all vpc endpoints.

Thanks for the info. That definitely helps for the specific needs I have.
You may close the issue if that is considered sufficient for most people, but there may still be some value for being able to tag the different endpoint types on a more granular level.

Thanks @Velgus, Closing this for now. When a specific use case is raises it can be discussed.


Show swap usage

Suggest adding the ability to show swap used / available (percentage) similar to memory & disk.  Should include flag for swap percentage similar to memory and disk.
While most modern systems have enough memory to make swapping relatively rare, embedded devices (e.g. routers running entware) generally need to use swap for extended functions.

https://github.com/dylanaraps/neofetch/issues/1261 -> seems not possible to display correctly, is it? htop works fine but neofetch gets it wrong if I use /dev/partition.


(note: scheme tableau not found, using s2color)

Hi,
When I was trying to use white_tableau scheme in Stata17, it reported that
(note: scheme tableau not found, using s2color)
Do you know how to solve the issue?
Thanks!

Checking!

Dear @gen-li, I doubled checked with a bunch of people using Stata 17 and the schemes are working fine. Please reinstall them. Also see if you can physically see in the the C:/ado/personal/s/ folder. Also make sure the names are correctly specified. If it still doesn't work send me a screenshot of the dofile please!


👍 RuntimeWarning: Couldn't find ffmpeg or avconv

RuntimeWarning: Couldn't find ffmpeg or avconv - defaulting to ffmpeg, but may not work
warn("Couldn't find ffmpeg or avconv - defaulting to ffmpeg, but may not work", RuntimeWarning)
Installation on WIN11 and vs code.
Solved the issue by installing additionally :
pip install ffmpeg-downloader
ffdl install --add-path
****Closed and opened vs code and worked as a charm.

Solved by above solution.


Native binaries in Extensions missing +x

Describe the bug
When installing an extension that contains native binaries, the native binaries sometimes are missing the execute permission. This makes analysis steps that try to run them fail.
I've had this issue before (https://github.com/mborgerson/ghidra-xbe/issues/27), but it seemed to not appear anymore, making me think it was a one off fluke, but I can now reproduce it again.
To Reproduce
Steps to reproduce the behavior:

Download and install the extension linked below
Check the permissions for ghidra-xbe/os/linux64/XbSymbolDatabaseTool. +x will be missing.
Optionally, download the xbe in https://github.com/mborgerson/ghidra-xbe/tree/master/tests/xbefiles and try to analyze it in Ghidra. The symbol analyzer step should fail.

Expected behavior
ghidra-xbe/os/linux64/XbSymbolDatabaseTool should have the execute permission, and running the symbol detection step on an xbe file should succeed.
Screenshots

Attachments
ghidra_10.0.3_PUBLIC_20210924_ghidra-xbe.zip
log.txt (log file containing extensions installation and usage, the other analysis steps failing is unimportant)
Environment (please complete the following information):

OS: Arch Linux
Java Version: OpenJDK 16.0.2
Ghidra Version: 10.0.3
Ghidra Origin: Arch Linux repositories

Additional context

Faced the same issue with Ghidra 10.0.3 on macOS 11.4.
After doing some research, I figured out that there's nothing wrong with Ghidra - it preserves file system permissions while unpacking the extension.
To verify that, I did the following:

Unpacked the ghidra_10.0.3_PUBLIC_20210924_ghidra-xbe.zip file with unzip;
Gave XbSymbolDatabaseTool permission for executing with chmod +x;
Repacked the extension with zip -r.

After installing the modified extension, the aforementioned binary had all the required permissions.

@agarmash you seem to be correct, the change you submitted to ghidra-xbe fixes the issue. I suspected Ghidra might preserve the permissions, but I must've made a mistake during testing, because the zip file didn't contain the execute permission flag when I set it manually on the filesystem and rebuilt.
I'm also not sure why this issue didn't occur every time, but since it's fixed now and doesn't appear to be a Ghidra bug, I think I can close this.


Add tests for database sources

Motivation
Add tests to check that database sources for SQLi vulnerability is supported.

Changes

New test created
yaml files updated

Workflow

⚠ Create your PR as draft ⚠
Work on you PR until the CI passes (if something not related to your task is failing, you can ignore it)
Mark it as ready for review

Test logic is modified? -> Get a review from RFC owner. We're working on refining the codeowners file quickly.
Framework is modified, or non obvious usage of it -> get a review from R&P team

:rocket: Once your PR is reviewed, you can merge it!
🛟 #apm-shared-testing 🛟
Reviewer checklist

[ ] If PR title starts with [<language>], double-check that only <language> is impacted by the change
[ ] No system-tests internal is modified. Otherwise, I have the approval from R&P team
[ ] CI is green, or failing jobs are not related to this change (and you are 100% sure about this statement)
[ ] A docker base image is modified?

[ ] the relevant build-XXX-image label is present

[ ] A scenario is added (or removed)?

[ ] Get a review from R&P team

Failing java test seems unrelated with the PR.


add distinct to manifest_components where to prevent duplicates from …

…released_components filter

@RedHatProductSecurity/corgi-devs please review. I noticed duplicates with the next released_components filter. Caused by having multiple relations of type ERRATA for a single build.


Fix parallel execution in internal back-office

need to add Dependencies.toml.template file changes

Fix https://github.com/wso2/apk/issues/456


Add @IBInspectable @IBDesignable support to TypographyLabel

Intro
As part of adding IB support to our Typography UI elements, it would be nice to (1) be able to see the control rendered in IB, (2) be able to set the typography
Discussion
Unfortunately, @IBInspectable only works with fixed types such as Int, CGFloat, UIColor, CGRect, etc. It cannot work for our struct Typography. However, most of the properties of Typography correspond to these types, so let's try to forward them. e.g.
@IBInspectable fontSize: CGFloat {
    get { typography.fontSize }
    set { typography = typography.fontSize(newValue) }
}

This is a bit exploratory, so want to try it only for the label for now. If it works and seems valuable, we can extend to button, text field, and text view.
Task

Add @IBDesignable support to TypographyLabel and make sure it renders properly in Interface Builder
Add @IBInspectable support for the following properties (fetch value from typography property, on set create a new typography from the existing one but mutating the specified property)

familyName
fontWeight (raw value as CGFloat)
fontSize
lineHeight
letterSpacing

AC

Task (above) completed
0 SwiftLint violations
compiles with 0 warnings for iOS and tvOS
unit tests pass for iOS and tvOS
100% documentation coverage as per Jazzy

Blocked By

Issue #36
Issue #39
Issue #41
Issue #42
Issue #43

Due to longstanding bugs in Xcode (since 2020), IBDesignable does not work for view code contained in Swift Packages. Because YMatterType is of course a Swift Package, I fear that these components will never correctly render in Interface Builder. If a future version of Xcode ever fixes this (still broken in Xcode 14.1 rc2 as of late October 2022), then we can revisit this work.


Could you please provide the LM ?

I've attached a complete copy of the LM and other training configuration. Note that this is normally generated for a specific reciter by the script mentioned in the README and, hence, will require some filename changes to work with your own data (just find/replace Abdul_Basit_Mujawwad_128kbps as appropriate):
etc.zip

https://media.tenor.co/images/3d6ef5c0cacab962cd9db2e309114a7e/raw
doesn't really point to the script. Could you please upload it again?

The linked image is meant to hint at the reasons that the script is, as described, "unpublishable.'


Publish azure storage cpp client library v6.1.0

Publish azure storage cpp client library v6.1.0

Hi @EmmaZhu, could you please also bump the version in CONTROL file?

Thanks for the PR!


Patch for /trunk/core/src/main/java/org/powermock/core/MockGateway.java

From<EMAIL_ADDRESS>on July 25, 2012 11:14:10
Fix for issue #51 .
Attachment: MockGateway.java.patch
Original issue: http://code.google.com/p/powermock/issues/detail?id=400

From<EMAIL_ADDRESS>on August 04, 2012 10:53:50
Status: Duplicate
Mergedinto: 51


[Feature] [helper]-[SymbolDraw]?

What problem does this feature solve?
src/chart/helper/SymbolDraw.ts
What does the proposed API look like?
0

261a96af5a155c1cd765b3ea6b1cd3c166bfea99


[Snyk] Fix for 2 vulnerabilities

This PR was automatically created by Snyk using the credentials of a real user.Snyk has created this PR to fix one or more vulnerable packages in the `pip` dependencies of this project.
Changes included in this PR

Changes to the following files to upgrade the vulnerable dependencies to a fixed version:

requirements/docs.txt

⚠ Warning
sphinx 5.3.0 has requirement docutils<0.20,>=0.14, but you have docutils 0.20.1.

Vulnerabilities that will be fixed
By pinning:

Severity
Priority Score (*)
Issue
Upgrade
Breaking Change
Exploit Maturity

704/1000  Why? Has a fix available, CVSS 9.8
Improper Following of a Certificate's Chain of Trust SNYK-PYTHON-CERTIFI-5805047
certifi:  2023.5.7 -> 2023.7.22
No
No Known Exploit

581/1000  Why? Recently disclosed, Has a fix available, CVSS 5.9
Information Exposure Through Sent Data SNYK-PYTHON-URLLIB3-5926907
urllib3:  2.0.3 -> 2.0.6
No
No Known Exploit

(*) Note that the real score may have changed since the PR was raised.
Some vulnerabilities couldn't be fully fixed and so Snyk will still find them when the project is tested again. This may be because the vulnerability existed within more than one direct dependency, but not all of the affected dependencies could be upgraded.
Check the changes in this PR to ensure they won't cause issues with your project.

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open fix PRs.
For more information:
🧐 View latest project report
🛠 Adjust project settings
📚 Read more about Snyk's upgrade and patch logic

Learn how to fix vulnerabilities with free interactive lessons:
🦉 Learn about vulnerability in an interactive lesson of Snyk Learn.

Checkmarx One – Scan Summary & Details – 3f34eeb7-7697-475e-bfd6-bd14727abf2f
No New Or Fixed Issues Found


Imartin/separate rewards from project

📲 What

Project query obtains now very minimum information about rewards,(ideally in the future will not query anything around rewards, but for now some details to pre-load quickly the rewards carousel is required).
New query that will fetch the rewards + simple shipping rules + reward Items has being created, this new query is called within RewardsSelectionViewModel.
Crash fixed related to changing the selected location for a pledge.

🤔 Why
Project query has modified to obtain all simple shipping rules, for all rewards, (Globally shipping rewards then will hold a collection of 246 shipping rules each). Project (as any other data model within the app) can be parzelized to be serialized/deserialized from Intents, but unfortunately we have hit again the transactionTooLarge exception due to Project being too big (we need to find other ways to persist information within the app, passing the hole object from screen to screen via intent to avoid extra network calls has clear limitations).
👀 See
|  |  |
📋 QA
Instructions for anyone to be able to QA this work.
Story 📖
MBL-1877
MBL-1893

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
Attention: Patch coverage is 68.88889% with 14 lines in your changes missing coverage. Please review.

Project coverage is 68.99%. Comparing base (f7f29a1) to head (e4a78e5).

Files with missing lines
Patch %
Lines

...iewmodels/projectpage/RewardsSelectionViewModel.kt
78.94%
1 Missing and 3 partials :warning:

...kstarter/viewmodels/projectpage/AddOnsViewModel.kt
57.14%
0 Missing and 3 partials :warning:

...ter/viewmodels/usecases/GetShippingRulesUseCase.kt
78.57%
0 Missing and 3 partials :warning:

...arter/services/transformers/GraphQLTransformers.kt
0.00%
2 Missing :warning:

...kickstarter/libs/utils/extensions/PledgeDataExt.kt
0.00%
0 Missing and 1 partial :warning:

...ewmodels/projectpage/CrowdfundCheckoutViewModel.kt
50.00%
0 Missing and 1 partial :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@             Coverage Diff              @@
##             master    #2173      +/-   ##
============================================
+ Coverage     68.97%   68.99%   +0.01%
- Complexity     2162     2163       +1
============================================
  Files           346      346
  Lines         22845    22848       +3
  Branches       3346     3343       -3
============================================
+ Hits          15758    15764       +6
+ Misses         5288     5286       -2
+ Partials       1799     1798       -1

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

🚨 Try these New Features:

Flaky Tests Detection - Detect and resolve failed and flaky tests


SetDownstreamData/SetUpstreamData support

Do you think of adding support for SetDownstreamData/SetUpstreamData like in the C++ SDK (https://github.com/proxy-wasm/proxy-wasm-cpp-sdk/blob/master/proxy_wasm_api.h#L900)
Something like SetHttpRequestBody but for data?

done in #156. Thx!


S3Client::GeneratePresignedUrl return invalid url when key contains spaces

Issue #, if available:
The issue is here: https://github.com/aws/aws-sdk-cpp/issues/2396
Description of changes:
Change urlencode twice to urlencode once.
Check all that applies:

[ *] Did a review by yourself.
[ *] Added proper tests to cover this PR. (If tests are not applicable, explain.)
[ *] Checked if this PR is a breaking (APIs have been changed) change.
[ *] Checked if this PR will not introduce cross-platform inconsistent behavior.
[ *] Checked if this PR would require a ReadMe/Wiki update.

Check which platforms you have built SDK on to verify the correctness of this PR.

[ *] Linux
[ ] Windows
[ ] Android
[ ] MacOS
[ ] IOS
[ ] Other Platforms

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Hey just checking in, please squash your commits and we're good to merge

Hey, just finished, please check it

counds good, running one last test, then will give approval


Hive takes a long time to Sync

added a device that had way more files in its Hive; had to force quit the app (with keep me logged in selected) to get the device to show up in mobile app - extra Hive files have not appeared after 1) force quit 2) manual rescan of desktop app 3)log out and back in of mobile app.
Those Hive files show in Hive when navigated to via the device itself on mobile app, though.
about 20 mins later signed back in via mobile and Hive showed the extra files.

verified that it happens in web too - again only in that Hive tab that is the default opening screen in web and mobile; if you go to the device and select its Hive the file shows up relatively quickly


Update attack-surface-reduction-rules-reference.md

Corrected PowerShell command for setting a rule in Warn mode.  As it was written, it would produce an error if run, both in PowerShell and in a cmd window.
If invoked from a cmd window, a working command would be
powershell -command "Add-MpPreference -AttackSurfaceReductionRules_Ids 56a863a9-875e-4185-98a7-b882c64b5ce5 -AttackSurfaceReductionRules_Actions Warn"

Learn Build status updates of commit 1fd666f:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

microsoft-365/security/defender-endpoint/attack-surface-reduction-rules-reference.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel


refactor: bot code structure

tyyyy, ended up refactoring the refactor a bit but this is MUCH cleaner


urh: 2.9.6 -> 2.9.8

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for urh is: Universal Radio Hacker: investigate wireless protocols like a boss
meta.homepage for urh is: https://github.com/jopohl/urh
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 2.9.8 with grep in /nix/store/w395dz6kfpzpc6vnihxa7462x3ryff92-urh-2.9.8
found 2.9.8 in filename of file in /nix/store/w395dz6kfpzpc6vnihxa7462x3ryff92-urh-2.9.8

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

2 total rebuild path(s)

2 package rebuild(s)

First fifty rebuilds by attrpath

urh

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/w395dz6kfpzpc6vnihxa7462x3ryff92-urh-2.9.8 \
  --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 urh https://github.com/r-ryantm/nixpkgs/archive/416ba4b43fc26a0830badec209b9cfb6bb6e7761.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/416ba4b43fc26a0830badec209b9cfb6bb6e7761#urh

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/w395dz6kfpzpc6vnihxa7462x3ryff92-urh-2.9.8
ls -la /nix/store/w395dz6kfpzpc6vnihxa7462x3ryff92-urh-2.9.8/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 result
Generated using nixpkgs-review.
Command: nixpkgs-review

x86_64-linux

:white_check_mark: 2 packages built:

urh
urh.dist

Maintainer pings
cc @fpletz for testing.

[!TIP]
As a maintainer, if your package is located under pkgs/by-name/*, you can comment @NixOS/nixpkgs-merge-bot merge to automatically merge this update using the nixpkgs-merge-bot.

Add a :+1: reaction to pull requests you find important.

Done in https://github.com/NixOS/nixpkgs/pull/357930


use_scm_version = True

https://github.com/pypa/setuptools_scm

Considered that, but my concerns are about tarballs downloaded from github.

I'd disallow tarballs with git repo, regarding GitHub Releases, it is possible to make setuptools-scm generate a python module containing version assignment. If one downloads some invalid version, raise friendly exception explaining what is wrong and how to do it right.

JFYI, there's actually an additional distribution solving (2): https://pypi.python.org/pypi/setuptools_scm_git_archive


Official solutions folder for premium users

Issue title is pretty self-explanatory but basically I just want official solutions available as a sub-folder of the problem folder with each solution having its own file. Would be great to have corresponding READMEs containing detailed explanations for official solutions but I'm fine with just copy pasting the code solutions

I don't have access to Leetcode premium so I can't design for the latter part of your question. The idea of having a solution as part of the folder submission seems interesting and I can work on that after my finals this week.

Ah gotcha. I guess you can add the top most voted submission in Discussion tab in the solution subfolder and adding a link to the post as a comment in the solution file would be good enough.

The point of leethub is to offer a lightweight portfolio. i can see the value your idea brings, but it'd increase network delay. LeetHub prefaces on optimization. Currently, all network requests take a max. of ~400ms. Increasing it can cause delays on clients' devices, especially since Leethub doesn't use a server to cache requests.

The point of leethub is to offer a lightweight portfolio. i can see the value your idea brings, but it'd increase network delay. LeetHub prefaces on optimization. Currently, all network requests take a max. of ~400ms. Increasing it can cause delays on clients' devices, especially since Leethub doesn't use a server to cache requests.


[beta] Backport #6771

r? @alexcrichton
(rust_highfive has picked a reviewer for you, use r? to override)

:warning: Warning :warning:

Pull requests are usually filed against the master branch for this repo, but this one is against rust-1.34.0. Please double check that you specified the right target!

@bors: r+

:pushpin: Commit<PHONE_NUMBER>538a4db7680c187372d6dda93525fd has been approved by alexcrichton

:hourglass: Testing commit<PHONE_NUMBER>538a4db7680c187372d6dda93525fd with merge d3c0198a3e3386d96c19d7f887eff5b2e02c0c66...

I think this will fail for multiple reasons. @sgrif can you cherry-pick some other PRs to get the 1.34 branch working?  I think it will need #6807. It looks like there is another unused item in src/cargo/util/paths.rs (build with the 2019-04-01 nightly, with build --features=deny-warnings --tests).
Also, it looks like the minimal versions CI entries are failing.  Just delete the entire entry from .travis.yml and appveyor.yml.
Let me know if you want me to do it or have any questions.

Cherry picked #6807 and deleted the minimal version entries from travis and appveyor

Fixed the errors from cargo +nightly-2019-04-01 build --features=deny-warnings --tests

Thanks!  Sorry, the beta branches always seem to have problems since they are so rarely built.
@bors r=alexcrichton

:pushpin: Commit 067c199b5733ddc12f549c52bfd6a7390b4a3396 has been approved by alexcrichton

:hourglass: Testing commit 067c199b5733ddc12f549c52bfd6a7390b4a3396 with merge 6789d8a0a54a96d95365c4e1fb01d47a5eed9937...

No worries. This is mostly on me for letting this sit for so long.

:sunny: Test successful - checks-travis, status-appveyor
Approved by: alexcrichton
Pushing 6789d8a0a54a96d95365c4e1fb01d47a5eed9937 to rust-1.34.0...


runtime: get cgo tests working on android/iOS

cl/110400043 disables some tests because of the lack of compiler on android. Find a way
to get them working again.

For iOS, this is very close to working. The sticking point is TestCthread:

thread #6: tid = 0x8de56, 0x040ac240 gotest`/Users/crawshaw/ios/go/misc/cgo/test.Add + 124, stop reason = EXC_BAD_ACCESS (code=1, address=0x0)

frame #0: 0x040ac240 gotest/Users/crawshaw/ios/go/misc/cgo/test.Add + 124 frame #1: 0x04059aa8 gotestruntime.call16 + 112
frame #2: 0x04009efc gotestruntime.cgocallbackg1 + 204 frame #3: 0x04009e10 gotestruntime.cgocallbackg + 128
frame #4: 0x0405ba50 gotestruntime.cgocallback_gofunc + 108 frame #5: 0x0405c404 gotestruntime.goexit + 4

This is a slightly gnarlier version of #10043. In this instance, our EXC_BAD_ACCESS handler isn't be activated because the cgo callback is being run on a foreign thread. For foreign threads we have never had an opportunity to grab the mach_task_port.
One potential solution is to steal the pthread_create symbol and register the task port then.

The cgo test is probably the most important test that should be run on iOS.
(If I have to choose a subset of test on iOS, I'd choose runtime, sync, cgo,
os, math, net, and net/http to always run, and happily ignore all the
others.)
That said, I don't like the idea of overriding pthread_create even though
I'm
the one that proposed the same idea to implement cgo on OpenBSD and it's
how cgo works on OpenBSD despite the fact that the OS doesn't natively
support TLS.
The reason is that it's probably too risky to override such fundamental
functions
when the binary is subject to review by the App store. In fact, I think
we'd better
remove the task port code as well when building the release binary. (add a
TODO for cmd/gomobile?)
There is a way to install the task_port for foreign threads, because that
test
want to test that recover on foreign threads can still recover from SIGSEGV,
we can do the task_port installation at cgocallback time when we detected
that we're being called on a foreign thread.

Many tests now run, but there's still a few more that would be nice. Punting what's left to 1.6.


day/time in taskbar gets unbalanced contrast

Starting light theme I cannot see the date/time in taskbar anymore (black letters on dark blue background), when its colour follows other themes like Bing Desktop, see screenshot attached.

@dark0dev I think this is more a Windows issue. Which version of Windows do you have?

Of course latest Win 10 1909 (Build 18363.752) 64-bit. Light settings might collide with Bing settings by Dynamic Theme App or system wide settings. Interestingly the option to unset accent color of task bar and system tray is greyed out, see screenshot.

I have same Problem.
Letters in date/Time and the Icon of Microsoft Store Keep dark instead of become white again

I've noticed this problem as well in Microsoft Store and it happened after some Windows Updates for me. Changing the light/dark app theme from the Windows Settings yields the same result. Closing this as this is a Windows UI bug.


chore: Update with changes up to (including) d8f852600306aca92e6db329dea24e667551f0e8

Description
This is a preparation for merging the branch to main
This PR contains all the changes up to d8f852600306aca92e6db329dea24e667551f0e8
Changes proposed in this pull request:

...
...
...

Related issue(s)

Closing because this PR already contains all these changes: https://github.com/kyma-project/lifecycle-manager/pull/1892


RS metadata fast follows

Design or Research
Fast follow up from story #690.
Tasks

[ ] Schedule follow up meeting with Kalish to work on integration test - scheduling for 6/5/24
[x] Add a gradle task that sets the ETOR_TI_baseurl - https://github.com/CDCgov/prime-reportstream/pull/14608
[x] Add some info about how to add URL to .env.local (needed for docker-compose) - added to the TI readme
[ ] Make database integration test

Additional Context
Add any other context or screenshots about the work here.
Notes from Sylvie

add gradle task (possibly as a fast follow) that sets the environment variable - we need this for our own purposes ASAP
maybe add some info about how to add URL to .env.local (needed for docker-compose)
schedule follow up meeting with Kalish to work on integration test

Error definitions needed.

We are running into an issue that we believe resides in part of the RS testing code.  We have reached out to one of their engineers to ask if they can take a look at the issue since we believe it's something specific to their framework that is causing an issue.


[Vital Module]All the fields of this service must be compulsory

[Actual Results]: The system is not blocking the user to go to the summay page after not entering any infomation in the vital service.
[Expected results]: All fields of the vital service must be Required.
[Steps to reproduce]: Reception module and vital module-> Patient dashboard ->Services tab-> Vital service
FREQUENCIA CARDIACA (5087)
RESPIRATORY RATE (5242)
TEMPERATURE (C) (5088)
WEIGHT (KG) (5089)
ALTURA (CM) (5090)
SYSTOLIC BLOOD PRESSURE (5085)
DIASTOLIC BLOOD PRESSURE (5086)

@ningosi. Can you please make this fields compulsory?

 Metadata

@Benilde  I think this is solved, any other queries we can re-open or create a new one

There is no field set as mandatory in this form, it should allow you to see the confirmation page so that you know that your form is blank, but it shouldn't allow to save.

@valvijo Like this!?

@valvijo Or you meant, "No field should be mandatory in the form, the next button should take the user to the confirmation page and block the save"!?

:confused::confused: I don't think this is going to work @larslemos, you don't expect to find all the triage equipment available all the time. Also mentioned in the last MoH discussion.
 I'm using my localhost. Since the beginning of the development of vital module stakeholders has informed that vital information should be required. We had a discussion about the availability of the equipment, but reported that this information is necessary for the analysis of the patient's condition. I would propose to discuss this topic at TWG.


Display's Backlight control and keyboard does not work

The keyboard's button for Display and Keyboard does not work properly.

It requires:
$ sudo pacman -S brightnessctl


Initial draft of Batched DoFn user guide

Fixes #23908
GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI.

Run Website PreCommit

R: @pcoet (for language/readability)
R: @robertwb (for technical accuracy)
Preview is available here: http://apache-beam-website-pull-requests.storage.googleapis.com/23909/documentation/programming-guide/index.html#batched-dofns

Thanks for the feedback @pcoet and @pabloem :)


SSL with DNS challenge fails due to propagation but ee thinks it renewed

Due to cloudflare we are using the DNS challenge method in order to renew the letsencrypt certificate of the site.
Even though the renewal has failed due to DNS records not being propagated, easyengine thinks that the SSL certificate has been renewed causing issues later on.
Issues such as not renewing the certificate when needed so the certificate expires. Not allowing to force renew so we have to delete the site and re-upload it.
All the issues cost money because of the time spend.
`ee site ssl-renew www.example.com
Starting SSL cert renewal
Loading current certificate for www.example.com
Loading current certificate for www.example.com
Created DNS record: _acme-challenge.example.com. with value 4l5Sq1y5bUzcDcmcIPTOj6aP1zzUOQYoSjR7CT_0pmA.
Created DNS record: _acme-challenge.www.example.com. with value zGav80pE1TgvEKR94QSemDq7wFA2qz1LnutSzdBNDaE.
Waiting for DNS entry propagation.
Starting SSL verification.
Warning: The dns entries have not yet propogated. Manually check:
host -t TXT _acme-challenge.www.example.com
Before retrying ee site ssl www.example.com
Warning: Check logs and retry ee site ssl-verify www.example.com once the issue is resolved.
Starting site's services.
Success: SSL renewal completed.`

A recent incident. Where one of my client changed dns to their sites. As ee renews all sites in a single command this broken ssl for all other sites. here is how:

ee starts renewing one after another.
if dns is changed at any moment. le just times out and thorws error. This exists the renew process. leaving all others sites below un-renewed.
this keeps happening and eventually ssl for all sites below is expired and sites broken.

This needs to be fixed. Possible solution: Check DNS before renew.


Last updated shows two different values...

Last updated shows different values over the tab vs map. Looks like an inconsistency.

@mananthprabhu Hey! The right one is the last updated time for the current state (Maharashtra in this case), while the left one is the overall last updated time. Closing


feat: add request header to show url passing to service

What does this PR do?
In the replace path and replace path regex middlewares we defined a new header that contains the X-Service-Path.
Motivation
For debugging purpose is really helpful to see if your expected path to service is correct.
More

[x] Added/updated tests

Hello @VladislavKravitski4888,
We'd like to better understand your needs and motivation.
Could you associate it with an issue to help us evaluate the use case, please?
This will also allow other users to upvote and expand on their needs too.
Thanks in advance.

Hello @nmengin,
e.g. we tried to use the replacePathRegex middleware like regex: “^/whoami/(.*)“, replacement: "/whoami/bli/bla/blub/$1“. As we could see, there was a header named X-Replaced-Path containing original request url but no request header to show what url would go to a service. For consistent reasons it’s for our aim better to see which both urls coming at a service were. It were fine for us to see both information in logging but it is not the case at the moment. In the log files you can only see the original request path but no url going to a service. Like … "RequestMethod":"GET","RequestPath":"/whoami/bla","RequestPort":"8443","RequestProtocol":"HTTP/1.1","RequestScheme":"http","RetryAttempts":0,"RouterName":"my-router@file","ServiceAddr":"apigateway_demo:8080","ServiceName":"service-whoami@file","ServiceURL":{"Scheme":"http","Opaque":"","User":null,"Host":"apigateway_demo:8080","Path":"","RawPath":"","OmitHost":false,"ForceQuery":false,"RawQuery":"","Fragment":"","RawFragment“:““},…

Hello @VladislavKravitski4888,
Thanks for the contribution!
We discussed the opportunity of the changes proposed in this PR, and we don't think that Traefik should add a header for debugging purposes only.
For this reason, we are declining this PR. Nonetheless, if another need for this is raised in the future, we are open to reconsidering it.
As @nmengin mentioned in his previous comment, we could also consider adding a DEBUG log for that purpose.
Additionally, we think that your use case fits with the plugin mechanism, which you could leverage to build your own plugin to add the header containing the service path to the request.


Mis Aligned ENV config for Speech Voice

Please provide us with the following information:

This issue is for a: (mark with an x)
- [x] bug report -> please search issues before submitting
- [ ] feature request
- [ ] documentation issue or request
- [ ] regression (a behavior that used to work and stopped in a new release)

Minimal steps to reproduce

The tutorial at https://github.com/Azure-Samples/azure-search-openai-demo/blob/0946893fe904cab1e89de2a38c4421e38d508608/docs/deploy_features.md?plain=1#L198 is telling for config env var AZURE_SPEECH_SERVICE_VOICE , but in the code, it tried to load AZURE_SPEECH_VOICE  https://github.com/Azure-Samples/azure-search-openai-demo/blob/0946893fe904cab1e89de2a38c4421e38d508608/app/backend/app.py#L447, which is then used at https://github.com/Azure-Samples/azure-search-openai-demo/blob/0946893fe904cab1e89de2a38c4421e38d508608/app/backend/app.py#L570
Any log messages given by the failure

Expected/desired behavior

The env variable name should be kept consistent.

OS and Version?

Windows 7, 8 or 10. Linux (which distribution). macOS (Yosemite? El Capitan? Sierra?)

azd version?

run azd version and copy paste here.

Versions

Mention any other details that might be useful

Thanks! We'll be in touch soon.

I've sent a PR addressing this, thanks for letting us know!


Documentation on the webhooks status

In the "Implementing a webhook on the SaaS service" you give two examples of webhooks (One with Success status and the other with InProgress status) but there is no documentation about the status. I have two questions:

What is the difference between a webhook received with the status InProgress and the status Success? What does this status represent?

Should they be processed in the same way by the ISV Service?

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

ID: fe8750da-cb47-7396-8d1b-0abff019a0be
Version Independent ID: b19143ed-64c0-1554-5ac9-610513382609
Content: SaaS fulfillment APIs v2 in Microsoft commercial marketplace
Content Source: articles/marketplace/partner-center-portal/pc-saas-fulfillment-api-v2.md
Service: marketplace
Sub-service: partnercenter-marketplace-publisher
GitHub Login: @mingshen-ms
Microsoft Alias: mingshen

Thanks for bringing this to our attention. Your feedback has been shared with the content owner for further review.

#please-close Thank you for the feedback @mneLSI. We have included this feedback in changes to the documentation that we will be publishing in the future.


Tools: Consolidate -Force and -Revert on Remove-Migration

Should we change the way Remove-Migration works?
Questions:

When should it check if the migration is applied?

What should it do if it can't?
What should it do if it is? (e.g. warn, error)

When should it revert the migration?

Design notes
We'll re-purpose -Force to revert the migration if applied. We'll remove the -Revert option introduced in 2.1.0-preview1.
Remove-Migration will...

Check if the migration is applied

On error, error with: Can't connect. Use -Force to remove. If applied elsewhere, revert it first.
If applied, error with: Migration applied. Use -Fore to revert and remove it. If applied elsewhere, revert it everywhere.
Otherwise, remove it.

When -Force is specified...

Check if the migration is applied

On error, warn: Can't connect.
If applied, revert it.

Remove it.


Add comments in file.cr

I've added comments in file.cr for self.dirname(filename), self.basename(filename) and self.basename(filename, suffix). Hope it's helpful :)

Thanks!


Create custom code components for Power Platform

Adding support for the component framework can help avoid technical debt and maintenance as this will be common for multiple Power Platform elements, as opposed to traditional components which is only useable in Canvas Apps.
Component framework
Development
Resources
https://learn.microsoft.com/en-us/power-platform/developer/cli/reference/pcf
https://github.com/microsoft/powercat-code-components
https://github.com/microsoft/PowerApps-Samples
https://pcf.gallery/
Environment
EDS Dev
Deployment

We should perhaps put the PCF components in the EDSPowerPlatform
repository and using actions in this repository for building and deploying components.

Users
Important to remember that the users of PCF components are the citizen developers and not their end users, this is covered by the core component. We should try our best to adhere to conventions used by Microsoft. e.g. the order of parameters.
Tasks

[ ] Create PCF components

See list under components for priority

[ ] CI/CD for code-components

essentially we should deploy this to any environment in our tenant where code components are enabled. for now we stick to deploying these to the EDS Dev environment

Components
Start out with components cannot be replicated by configuring native controls (e.g. labels)

Prioritize 2.0 components, or components not expected to change much in 2.0.

[ ] Button (2.0)
[ ] ~Datepicker~
[x] Table
[ ] Sidebar
[x] Snackbar
[x] Chip
[ ] ...

Created a workaround for working with pcf components i devcontainer documented here
Registered an issue to find a better solution: https://github.com/microsoft/powerplatform-vscode/issues/573

Need to clarify strategy for EDS Components and Template apps.

We should plan a small workshop with a couple users to align our priorities of both which components and how we define the component properties etc. I have a couple of candidates in mind for this workshop and will set it up.

I believe the actual bottleneck is mapping properties that are suitable with Power Apps Studio conventions and best practices. Whether or not 2.0 is done is not as important as initially thought, instead we should rely on whatever is available in the npm library.

Removed from task:
Prioritise 2.0 components, or components not expected to change much in 2.0.

Autocomplete is added to the list after button.

Button component fixes: https://github.com/equinor/EDSPowerPlatform/commit/661a583c536bb833f629e4c4b3450a0232098e32
Next: Find best practice for setting a default size of a code component when it is added to a canvas in power apps studio.


create list of Florida Legislators with their ccid

legislator's name, district, party, role, ccid, vote on SB 300 (hearing was today, 4/3/23)
ccid formula: 12 + 000 (number of district) + U (for Senate)
example - ccid: 12030U

SB 300 - data - Sheet1.csv

thank you kind sir

can you add first name and last name fields?

Senator Torres abstained from voting. So it would be "na" instead of "m"

SB 300 - data - Sheet1.csv

SB 300 - data - Sheet1.csv

thank you this works!


Adding epoch to Ampyear

This should not be merged! It's me testing how this works and we should not merge until https://github.com/KSP-CKAN/CKAN-netkan/pull/60 is merged so we get a consistent naming schema.
resolves https://github.com/KSP-CKAN/CKAN/issues/931

#rebuild


Added WithDisableCircuitBreaker and WithDisableBusyLoopBreaker options

Added WithDisableCircuitBreaker and WithDisableBusyLoopBreaker options. These are variants of the now deprecated DisableCircuitBreaker
and DisableBusyLoopBreaker options. They provide a booling parameter which is more convenient for usage with
code generation and for shimming with configuration.

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
Attention: Patch coverage is 83.33333% with 1 line in your changes missing coverage. Please review.

Project coverage is 89.46%. Comparing base (ef8e6bc) to head (358b7b8).

Files with missing lines
Patch %
Lines

workoption.go
83.33%
1 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #16      +/-   ##
==========================================
- Coverage   89.75%   89.46%   -0.30%
==========================================
  Files          18       18
  Lines        1571     1575       +4
==========================================
- Hits         1410     1409       -1
- Misses        104      107       +3
- Partials       57       59       +2

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


Handles interfaces implementing another in AutoConfiguredNSubstitute

This PR fixes #415.

Thank you for your contribution! It's now live as AutoFixture.AutoNSubstitute 3.30.8.


Support for cli extension framework

Can the extension framework [like hooks] be added along with the rest and the cli interface? Something like Recap as a microkernel with a plugin-style system. One possible extension is generating a static website from the Recap metadata [like dbt docs], which can be reused by many.

This is a great idea. It should be doable. I need to do a bit of research to figure out the right way to do this in Python. IIRC, dbt has a similar style. It would be nice if there was some open source library that made extension management easy.

So, I sniffed around dbt a bit. They have adapters, which are much more limited in scope than the extensions we're talking about. They also have packages, which looks more like what I was thinking. But packages seem to be dbt projects rather than raw Python code.
What I really want is just a Python library that someone can write and publish. If the library has classes that implement Recap abstract classes, then the library's functionality automatically become available in Recap. To make Recap aware, it'd either have to crawl all packages in PYTHONPATH, or developers would have to update a config variable to tell Recap what's available.
Some relevant links:

https://hub.getdbt.com - Where packages are published.
https://docs.getdbt.com/guides/legacy/building-packages - How to write a dbt package.

I'm starting to poke around Python plugin libraries, which seem useful.  I'm not sure how much value it adds over implementing my own:

https://packaging.python.org/en/latest/guides/creating-and-discovering-plugins/
https://alysivji.github.io/simple-plugin-system.html

I'm in favor of Implementing my own plugin architecture, and following the packaging.python.org guide. It seems fairly straight forward. The main thing is making sure I have the right abstractions in Recap, something I'm still iterating on.

The airflow plugin architecture is also an interesting approach, though they do not support cli extension, but does API extension.
https://airflow.apache.org/docs/apache-airflow/stable/plugins.html

I sniffed around the Airflow plugin system a bit. It looks to me like it's essentially just doing option (3) in the packaging.python.org guide--setuptools entrypoints. This was already the direction I was leaning, so Airflow's usage further validates my inclination. :)

Added @ananthdurai! Thanks for the idea. See:
https://github.com/recap-cloud/recap/pull/52
For some docs.


fix typo in error message (tiktok -> tiktoken)

I thought this was unintentional

I believe OpenAI's staffs love tiktok.

Thanks, this fix was present in the internal version of the codebase and has been synced


Comment character in text columns causes parsing error

I have some CSV's that look like this:
# Created: 11 Oct 2017 19:33:22 UTC
# Data Period: 2017-06-01 - 2017-09-30
station,station_name,lat,lon,day,doy,gdd_50_86,gdd_40_86,high,low,precip
WI4195,KEWAUNEE,44.4500,-87.5000,2017/06/01,152,11,18.5,72,45,0.0
WI1205,BURLINGTON,42.6500,-88.2500,2017/06/01,152,10.5,17,71,43,0.0
WI3038,GENOA DAM 8,43.5667,-91.2167,2017/06/01,152,12.5,21.5,75,48,0.0
WI3244,GRANTSBURG,45.7667,-92.6833,2017/06/01,152,11,16,72,36,0.0

which I'm able to successfully parse using this configuration:
options = {
    comment: '#',
    rowDelimiter: ['\r','\n'],
    columns: true,
    skip_empty_lines: true,
  };

  let rawData = fs.readFileSync(filePath);
  let parsedRows = parse(rawData, options);

Unfortunately, there are some records in the data set with # in the station_name column, like so:
WI0892,BLANCHARDVILLE #2,42.8000,-89.8500,2017/06/01,152,11,19,72,46,0.0

When I attempt to parse a file with records like these I get an error along the lines of Error: Number of columns on line 9 does not match header, where the line number given is nowhere near the offending record.
It's fine if the module can't reasonably parse these records with the given configuration, it should treat content after a # as a comment. It'd be great if I could just pass in a regex to check for the start of a line, or just a boolean setting to only tolerate comments on their own lines, but for now I can just remove the comments from my documents before processing.
That being said, the random line number in the error message suggests that the parser is getting confused and trying to process data after encountering the offending line (for one document it threw an error on line 113, when the first record with a # in it is on line 59), so I'm guessing there are some issues with this section of the code worth investigating.
Thanks.

It's fine if the module can't reasonably parse these records with the given configuration
It's a feature, we treat comment at the begining or at the end of a line (like in javascript for exemple). In a perfect world, your cell will be quoted, but I undestand that we dont always choose how we receive the csv dataset.
It'd be great if I could just pass in a regex
I dont think it's possible, we don't have notion of line against which we could apply the regex, the reason is that line feed are valid chars if present inside a quoted cell. I'll double check. What we could do is creating a new option such as "dont_treat_comment_unless_at_the_beginning_of_a_line"
the random line number in the error message suggests that the parser is getting confused
Please isolate a simple example illustrating the issue. I'll give it a try as well

Here's a zip file with a minimal example.
Archive.zip
unzip
npm install
run node parseExample.js
should see error:
/Users/{username}/tmp/node_modules/csv-parse/lib/sync.js:28
    throw err;
    ^

Error: Number of columns on line 57 does not match header

(Note the error is thrown on line 57 of the document, the first non-comment # appears on line 57 in the example document.  A minor difference in this case, but in some of my documents the error was thrown dozens of lines after the #.)
delete first 2 lines from the top of the included csv
delete comment: '#', from the parseConfig object in the JS file
run node parseExample.js
should see this printed to console:
parsedRows[0]:  { station: 'WI4195',
  station_name: 'KEWAUNEE',
  lat: '44.4500',
  lon: '-87.5000',
  day: '2017/06/01',
  doy: '152',
  gdd_50_86: '11',
  gdd_40_86: '18.5',
  high: '72',
  low: '45',
  precip: '0.0' }

indicating that the document was parsed successfully (to prove that the document is well formed aside from the comment string issues).
I've realized that for my uses, all my 'real' comments start with # , and all the records I've seen so far use #2 or something along those lines, so I can simply use #  as the comment string for now.

Closing due to lack of activity.


JPA dependency and automatic Bean mapping

I am wondering if there is any way JPA dependency can be avoided and instead resultset can be directly mapped to POJO just like Spring JDBC template. In my opinion it'll be one less dependency to worry about.

why not use SpringJDBC then? JPA annotations give additional metadata like tables or columns names etc, also there should be at least some clue on which columns makes primary id for select or delete.
anyway, if the only thing you need is map result set to pojo, i'd suggest to make a pull request, see Introspected class where the Column annotation is searched.


Error while creating summary: Error: Trades not found

Note: this is the technical bug tracker, please use other platforms for getting support and starting a (non technical) discussion. See the getting help page for details.
I'm submitting a ...
[x ] bug report
[ ] question about the decisions made in the repository
Action taken (what you did)
Running Gekko in pm2 with newest commit (e12eea0279b2c16b2d3e308c09d77e7b24b2d95e)
Expected result (what you hoped would happen)
Summary to be posted. Right now it just returned a tradeErrored event.
Actual result (unexpected outcome)
Summary failed due to 'trades not found' .  At least order was filled.
Other information (e.g. detailed explanation, stacktraces, related issues, suggestions how to fix, links for us to have context, eg. stackoverflow, etc)
Debug log:
Current market: bull
Percent change predicted: 0.5231228061487263
Current price: 9.809
Last buy price: 0
Current position: none
2018-08-22 17:59:10 (INFO):     Trader Received advice to go long. Buying  BNB
2018-08-22 17:59:10 (DEBUG):    Creating order to buy <censored amount> BNB
2018-08-22 17:59:16 (DEBUG):    [ORDER] statusChange: SUBMITTED
2018-08-22 17:59:16 (DEBUG):    [ORDER] statusChange: OPEN
2018-08-22 17:59:18 (INFO):     [ORDER] partial buy fill, total filled: 183.46
2018-08-22 17:59:18 (DEBUG):    [ORDER] statusChange: FILLED

2018-08-22 18:00:09 (DEBUG):                                    [EVENT tradeErrored]
 { id: 'trade-1',
  adviceId: 'advice-1',
  date: moment("2018-08-22T17:59:20.764"),
  reason: 'Trades not found' }

pm2 Error log:
  2018-08-22 17:59:20 (ERROR):    Error while creating summary: Error: Trades not found
      at get (/home/chris/gekko-neural-USDTpair/exchange/wrappers/binance.js:364:23)
      at checkFn (/home/chris/gekko-neural-USDTpair/exchange/exchangeUtils.js:23:16)
      at /home/chris/gekko-neural-USDTpair/exchange/wrappers/binance.js:129:12
      at Request.request [as _callback] (/home/chris/gekko-neural-USDTpair/exchange/node_modules/binance/lib/rest.js:100:21)
      at Request.self.callback (/home/chris/gekko-neural-USDTpair/exchange/node_modules/request/request.js:185:22)
      at emitTwo (events.js:126:13)
      at Request.emit (events.js:214:7)
      at Request.<anonymous> (/home/chris/gekko-neural-USDTpair/exchange/node_modules/request/request.js:1157:10)
      at emitOne (events.js:116:13)
      at Request.emit (events.js:211:7)
      at IncomingMessage.<anonymous> (/home/chris/gekko-neural-USDTpair/exchange/node_modules/request/request.js:1079:12)
      at Object.onceWrapper (events.js:313:30)
      at emitNone (events.js:111:20)
      at IncomingMessage.emit (events.js:208:7)
      at endReadableNT (_stream_readable.js:1064:12)
      at args.(anonymous function) (/usr/lib/node_modules/pm2/node_modules/event-loop-inspector/index.js:133:29)
      at _combinedTickCallback (internal/process/next_tick.js:138:11)
      at process._tickDomainCallback (internal/process/next_tick.js:218:9)

This is actually the same issue as #2465 so closing this as a dup. Replying there as well.


chore: add security policy

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

GitHub release
npm package (@latest dist-tag)

Your semantic-release bot :package::rocket:


docs: update syzbot.md

Add in some information about fix bisection in the documentation.

Codecov Report

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

@@           Coverage Diff           @@
##           master    #1375   +/-   ##
=======================================
  Coverage   57.82%   57.82%
=======================================
  Files         131      131
  Lines       24773    24773
=======================================
  Hits        14324    14324
  Misses       9779     9779
  Partials      670      670

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 526709f...e79059a. Read the comment docs.

Hi @dvyukov, I was wondering if you had a chance to review this PR. Thanks.

LGTM


An headless approach to this Library

Feature Request
Will it be more effective if this WebChat SDK just export Hooks or Redux Dispatches to the logic and all UI Components be created and customized by the developer? Is really a pain to make this WebChat looks good, using CSS is weird because overwrites leads to break in new releases as discussed the classes and html markup changes a lot and the code and stylesheet looks so dirty, the JSS approach leads to limited styling.
I know how important accessibility is and all the efforts from the core members of this project is doing but I think an headless approach will be better, providing logic via hooks and the style leaves to the developer using this lib to build, even handling accessibility.
I love how powerfull is the use of custom components, but I think the standard chat isnt customizible enough, even the sendinputBox is really dificult and weird to style and to add custom features to it.
Keep the great work on this SDK, this is just an idea that I think will make this lib even more powerful.

Thanks for your appreciations.
We refactored headless business logics into the botframework-webchat-api package (which our web UI is based on). It should be good for React Native but we haven't tested it yet.
You will need to bring-your-own UI. Top of my head, there are a few complicated logics that is not included in the api package:

Speech recognition and synthesis
Activity grouping

Grouping messages based on who send it, and when they send it

Some UI related logics

E.g. no "send button click" hook, because "send button" is UI, not headless

You are right. Accessibility is very difficult to get it right. We keep learning and rewriting our DOM tree to make accessibility experience better, but IMO, we are not there yet. Lucky we have good set of E2E visual regression tests, so we are quite confident about our quality when rewriting DOM tree.
CSS is a difficult topic. There are few things in my head that need more thinking/experiments:

Publish an official list of CSS class names that are good for extensibility (namely, "CSS insertion points")

While adding features or making accessibility experience great, inevitably, we need to modify the DOM tree

Instead of style options, use CSS vars (need to figure out a story for IE11 or browser which doesn't support it yet)

Few more things in my head that need more thinking/experiments too:

Ability to customize send box

We have an input modality work item, which devs can set to use only speech or text. And while we work on that, we will make it customizable... 1P = 3P

Move away or hide our Redux store... it should be internal

A lot of devs want to receive trigger when an activity arrives, doing it with Redux is easy, doing it with hooks are not

Move away from Observable (which DirectLineJS use). Instead, you can provide an activity array and let Web Chat render it out. Will be easier for you to use Web Chat on other chat platforms or rendering static transcript
Refactor out the middleware architecture into a separate package, and do some clean ups

The middleware architecture is great, so we want to refactor it out to make it even cleaner
Some middleware has poor API signature, cleaning them up is not trivial as a lot of devs relies on them

More samples or put them on CodePen/CodeSandBox

Reasons we didn't go CodePen/CodeSandBox: we can't integrate them with our CI/CD to programmatically publish samples there
Or build a Jupyter notebook like playground, so you can test your code side-by-side with UI


feat(downstate): docs + implementation for example components

Description
This one was a bit of a doozy in that I tried a lot of different approaches. TLDR, I landed on down state needing to be implemented in each component that requires it in S2. The short story for that is that each component is slightly different, and an overall custom property for the transform value wasn't an option because custom props have to be defined before they're used (which doesn't play nicely with our dynamic width and height).
More details:
I started with looking at a purely class based implementation of down state and ran into complications. For components like Button where the whole component is interactable, class based works fine because you can apply the class to the root/parent of the component. But with a more complex component like Checkbox, you have to apply the styles to the box itself, but the active state that we're targeting happens at the root/parent level.
So then I experimented with this mix of utility class and placeholder approaches, but it felt a bit ick to keep both because it could lead to confusion for users. We also ran into issues with a placeholder approach when working on tooltip after which we landed on not using the placeholders at all. At this point I'm thinking that users shouldn't really need utility classes since this feature should be shipped with the components that need it anyway, and placeholders that rely on custom props haven't worked well in the past.
Then Josh suggested that I try creating a custom token for the transform value, to keep the code DRY. Custom properties have to be defined before they're used, so I had to define --spectrum-downstate-width and --spectrum-downstate-height as 0px each in custom-vars.css for the transform value (perspective(max(var(--spectrum-downstate-height), var(--spectrum-downstate-width) * var(--spectrum-component-size-width-ratio-down))) translateZ(var(--spectrum-component-size-difference-down));) to be considered defined by the browser. The problem is that even when I was updating those width/height custom props using the style tag, the global custom token wasn't picking up those new values, it would always use 0px. So this option was also, unfortunately, ruled out.
And that brings us to the solution detailed in the most recent commit of this PR, which is to implement this transform within the components that need the feature in S2. Button and checkbox are the ones I included in this work to demo because Lynn asked to see those as examples in our Storybook (she also asked us to show Select Box since it's like 200x200px, but this is an S2 component we haven't built yet so I'm waiting to hear back from Lynn on if there is a large size S1 component we can use as an example in the meantime). As we pick up cards to migrate components to S2, adding down state to those that require it should be included in the work on those cards. The documentation provided in the docs site as part of this PR is intended to help guide that work.
How and where has this been tested?
Please tag yourself on the tests you've marked complete to confirm the tests have been run by someone other than the author.
Validation steps
Disclaimer: this code will look a lot more like the Corner Rounding setup once we've established that pattern

[ ] On the down state docs site, do the docs make sense? Think of it from your perspective as a spectrum-css dev as far as implementing this later on in components, but also consider a user's perspective. Is anything missing?
[ ] In the docs site, down state has a page and shows a checkbox and button with down state implemented
[ ] In storybook, down state has a page under "concepts" at the bottom of the side nav (open to ideas for what this kinda section should be named) and shows a checkbox and button with down state implemented

Regression testing
Validate:

The documentation pages for at least two other components are still loading, including:

[ ] The pages render correctly, are accessible, and are responsive.

If components have been modified, VRTs have been run on this branch:

[ ] VRTs have been run and looked at.
[ ] Any VRT changes have been accepted (by reviewer and/or PR author), or there are no changes.

Screenshots

To-do list

[ ] I have read the contribution guidelines.
[ ] I have updated relevant storybook stories and templates.
[ ] I have tested these changes in Windows High Contrast mode.
[ ] If my change impacts other components, I have tested to make sure they don't break.
[ ] If my change impacts documentation, I have updated the documentation accordingly.
[ ] ✨ This pull request is ready to merge. ✨

I don't necessarily thing this is something that should be addressed but I did notice on Safari and Windows(via assistive labs) that the checkbox active state squares out the checkbox borders while other browsers do not.
Safari Active

Should we also think and test this concept in mobile devices too with different themes?

@Rajdeepc Good questions!

Yes, :active is the way we can simulate a "down state" in CSS. From the design docs: down state is the concept to make a component look "pressed in" when activated by a user. We're using that phrase "down state" to align with design.
We don't have a :pressed pseudo class available in CSS, so we're using the :active CSS pseudo class. It should be equivalent to the idea of a pressed state, but let me know if that's not the case with web components

I agree, we should do some testing on mobile as well once the team agrees on the approach.
Let me know if this answers your questions 🙂

should the down state have a prefers-reduced-motion alternative?

There isn't anything for this in the design docs so I reached out to PJ on Slack to ask 👍

After talking with Damian and PJ in Slack, this down state motion isn't jarring or problematic from an accessibility perspective, so we don't need a reduced motion alternative.

@Rajdeepc Good questions!

Yes, :active is the way we can simulate a "down state" in CSS. From the design docs: down state is the concept to make a component look "pressed in" when activated by a user. We're using that phrase "down state" to align with design.
We don't have a :pressed pseudo class available in CSS, so we're using the :active CSS pseudo class. It should be equivalent to the idea of a pressed state, but let me know if that's not the case with web components

I agree, we should do some testing on mobile as well once the team agrees on the approach.
Let me know if this answers your questions 🙂

Sure thanks. This make sense. Also curious to know how are you shipping javascript? We haven't had a chance to discuss this but if you had any discussion do let me know. Since we are building a logic to tackle this from SWC side we would love to see an example.

Question about components in which button and checkbox are a dependency (alert banner, card). I'm assuming we don't want down state to be included in those components until they're migrated to S2, yes? Either way, this feature is probably a breaking change, so for the components that use button and checkbox, we just won't update their dependencies to this feature version until migration to S2, is that correct?

Question about components in which button and checkbox are a dependency (alert banner, card). I'm assuming we don't want down state to be included in those components until they're migrated to S2, yes? Either way, this feature is probably a breaking change, so for the components that use button and checkbox, we just won't update their dependencies to this feature version until migration to S2, is that correct?

Great question about the semantics of how we'd release this! I'm thinking about it like this:

Anything migrated to S2 will be a breaking change for that initial migration bit. (meaning, the component points to the S2 tokens rather than the S1 tokens)
This PR is adding a feature, so normally, I'd say it's a minor bump (at least), BUT, I think the S2 migration bit from my first point overrides that; it'll be a major.

As for components that use button and checkbox as dependencies: yeah, let's just wait for those for now.
Can you update the package.json for Button and Checkbox to 14.0.0-next.2?


On-call shows individual that is N levels down from the first level of the escalation policy

Describe the bug
When migrating from the old version of the plugin (from @backstage/plugin-pagerduty to @pagerduty/backstage-plugin and friends), the list of "On Call" individuals is incorrect. Only a single entry is shown and that individual is not the first level of on call in the relevant escalation policy (at least in the applications I've spot checked after the upgrade in my local environment).
To Reproduce
Steps to reproduce the behavior:

Go to Backstage entity screen with a pagerduty.com/integration-key annotation
Select tab with PagerDuty card
Note the names listed in the "ON CALL" section
Click the "ESCALATION POLICY" button
Compare list of names shown on the pagerduty side (they're different)

Expected behavior
List of individuals that are on call should match
Screenshots
Here you can see that the individual on the fifth level of the escalation policy is the only entry shown:

With the old version of the plugin, we correctly see all levels of the escalation policy:

Desktop (please complete the following information):

OS: Ubuntu
Browser Chrome
Version 121

Additional context
I have tried using the API token-based auth (which we were using with the old version of the plugin) as well as the OAuth version and am observing identical results

This actually appears to be intentionally coded this way over in the backend plugin: https://github.com/PagerDuty/backstage-plugin-backend/blob/main/src/apis/pagerduty.ts#L392-L394
Apologies for not posting in the correct repo. Can you help me understand the intent behind filtering the list of oncalls based on the first one returned?

Hello @brianphillips!
Thanks for opening this issue.
The old version of the plugin owned by Backstage was showing all users from all escalation levels under the on-call section. After collecting feedback from several plugin users this didn't seem to align with their expectation as all levels after level 1 are not necessarily on-call. They are escalation paths.
Still, the result on your first screenshot would only be correct according to our logic if Doug was the engineer currently on-call for Ops OCC on-call schedule. Can you confirm this?

By the way, one thing I'm considering to do to support both behaviours our users are requesting is to have a plugin flag that allows admins to define wether you just show the engineers on-call for escalation level 1 (default) or all escalation levels. Would that be a possible solution for your use case?


Add option to configure server host

Please select if your request is either something new or an enhancement

[x] Enhancement of an existing Feature.
[ ] Request of a new feature.

Please select the area your request applies to. (Multiple selections are Possible. You can leave blank if you're not sure.)

[x] Workspace - VSCode workspace, vaults, Intellisense/autocomplete, Dendron settings
[ ] Lookup - Dendron's Lookup Command
[ ] Views - Dendron Preview, Tree View, Side Panels in the UI
[ ] Schema - Dendron Schemas
[ ] Pod - Data import from / export to Dendron
[ ] Publish - External Site Publish
[ ] Markdown - Markdown features, such as syntax support and features

Is your feature request related to a problem? Please describe
Services like gitpod are running inside of a browser under a unique url, but the extension is sending requests to localhost. Thus, the extension is not fully functional in gitpod and problems like the preview getting stuck on loading screen occur.
Describe the solution you'd like
An option to set the host for the background server would probably solve the problem.
But for automation purposes, it would also be helpful to make this setting available to the shell.
Maybe this can be done by environment variables or by calling a command, which is provided in the setting (like gp url for gitpod, which returns the workspace url)
Describe alternatives you've considered
I have tried redirecting the requests and spoofing localhost, but none of those options worked.
Additional context
You can load the workspace yourself with this link. You only need to register on gitpod.io, the first 50 hours are free.
Nevertheless, here are some screenshots:
Screenshot of gitpod showing the problem as described above

Output of the gp url <port> command, please note that the url is prefixed by the port

Proof, that the server is working, if localhost is exchanged by the workspace url

@mfillmer - thanks for the interesting suggestion.  I personally haven't tried gitpod before - but it's cool that it can support spawning of the host server process, as your last screenshot proves.
We can look into this one - since it appears that the server is running, it seems like exposing a simply config setting for the host root address should do the trick.

Cool, thank you! Let me know if you need any more info.

I think this is a brilliant suggestion! If we could get Dendron working in GitPod, we'd have a web-client for free! @kevinslin

This issue is featured in this week's round of CROP.

Was this ever implemented? My company doesn't allow me to access the server via http://localhost, but rather http://machine-name-on-network and I cannot use Dendron as-is.


Distribution build uses a regeneratorRuntime function that doesn't exist

Hi,
It looks like you're using regeneratorRuntime in the compiled useApiRequest.js, probably via compilation from Babel.
From dist/ApiRequest/useApiRequest.js
function _loadData() {
    _loadData = _asyncToGenerator(regeneratorRuntime.mark(function _callee() {
      var res, _data;

      return regeneratorRuntime.wrap(function _callee$(_context) {
...

But regeneratorRuntime itself is not defined nor imported.
I think that you need to update your .babelrc to include an extra babel plugin

I've opened a pr here @idanlo
#8

Thanks. I'll merge that

@samjt  I released a new version. Tell me if that works for you

@idanlo Yes, works great now! Thanks for the quick release.


Replace dyn Error with anyhow in the actor interface.

Issue summary

Replace Result<T, Box<dyn Error>> with anyhow::Result<T> in vm/actor_interface/*.
Update the error handling of all functions that call into the actor interface.
Use best judgement to limit the scope of this ticket. Not all of forest has to be updated at once; fixing the low-hanging fruit in vm/actor_interface would go a long way in making our work easier in the future.

Other information and links

This issue is a subset of #1458.

Done.


Improvement set grpc channel's keepAliveTime and remove proxy …

…on InterruptedExcep… (#37304)
Proposed changes

set grpc channel's keepAliveTime
remove proxy on InterruptedException/TimeoutException to avoid channel unavailable

pick from #37304
Proposed changes
Issue Number: close #xxx

run buildall


Detect Camera Position in the AR environment relative to the marker

Do you want to request a feature or report a bug?
Feature

What is the current behavior?
Currently there is no way to calculate the camera's coordinate relative to the marker
If the current behavior is a bug, please provide the steps to reproduce.
I am using a-camera-static that makes the center of my marker as the coordinate (0,0,0). Now if i place a 3d entity and set its position to (0,0,0). its going to load over the marker. But what if i want to place the object near the device through which the user is looking at the AR experience.

What is the expected behavior?
The expected behaviour would be to calculate the coordinate of the camera relative to the marker based on its position. And an update method that updates the coordinate when the user moves the camera. Once we know the coordinates of the camera we can position objects relative to the camera's position
If this is a feature request, what is motivation or use case for changing the behavior?
For eg :- i want to make a shooting game in which my monster is on the marker and i want to shoot bullets at it using my phone/device from different angles. If there is any way to achieve this please let me know.
Please mention other relevant information such as the browser version, Operating System and Device Name
Device Name : One Plus 6

Can't you give your camera an id, e.g. camera and then read its WorldPosition with document.getElementById('camera').object3D.getWorldPosition()? There is a look-at component with an example at https://github.com/supermedium/superframe/blob/master/components/look-at/examples/basic/index.html that has an object looking at the camera. Maybe this works in AR as well.

@aj019 were you able to achieve what you mentioned above ?
We are trying to create a mobile based R platform that can be used to visualize MRI slices. For that, we need to calculate distance of phone from marker so that we can display the appropriate slice accordingly. We needed some help in that matter. Did the solution given by @janpio work?

Closing this issue due to dismission of this repository. New repository of AR.js will be up at https://github.com/AR-js-org/AR.js/
Feel free to re-open the issue there if still valid.


Blocks with fields of type date and number are empty when fetched via graphql

Bug Report
When I create a block that contains a field of type number or date, it is showing up fine in the admin UI, but when fetched in the client via graphql it's returned as an empty object.
Admin fetch response
[
    {
        "value": 10,
        "id": "63e9dfb38b69cd009592db2e",
        "blockName": "Test",
        "blockType": "numberInput"
    }
]

Graphql response
specifications: [
    {}
]

Field definition:
export const NumberSpecification: Block = {
  slug: 'numberInput',
  fields: [
    {
      name: 'value',
      type: 'number',
    }
  ]
};

Other Details
Payload 1.6.9

Hey @itzaks, this appears to be working properly. It likely has to do with querying properly via GraphQL.
Here is what it looks like to query block data:


The closed channels list, should return who initiated the channel-close operation close_initiator

Background
The closed channels list, should return who initiated the channel-close operation close_initiator
Expected behaviour
show "close_initiator": true/false, like "initiator": true for channels
Actual behaviour
attribute not present

Closing as duplicate of https://github.com/lightningnetwork/lnd/issues/2594.

@wpaulino I don't think this is a duplicate.  #2594 asks closedchannels to include the channel initiator; this issue is asking closedchannels to include information about which party initiated the close.

This would only be useful in the co-op close case as you can already determine which party force closed through the close type field.


Problems with audio: how to debug?

I'm unable to make the audio generation work properly, and can't understand how to debug. Installed TeleFrame on a Raspberry using the provided script, auto-run with PM2. Is any log file available in this configuration?

You can run pm2 logs TeleFrame to see the logs.

Thanks, and in case someone else follows this discussion (with limited knowledge of PM2 as myself): if you installed TeleFrame with the provided script on a Raspberry the logs are reached with pm2 logs TeleFrame as pi user, not root or sudo.

Could you fix your audio problems?

Not yet I'm trying now

...ok solved was as simple as an audio peripheral not being recognized/used by Alsa.
How to debug on a Raspberry, TeleFrame installed with the script:

look at TeleFrame logs by pm2 logs TeleFrame (as pi user) while recording  an audio message (reveals problems with Alsa)
look at system logs by sudo tail -f /var/log/syslog while unplugging/plugging an audio peripheral on USB (reveals problems recognizing the peripheral)
google "raspberry usb audio ..." and the shown error

In my case in the end I solved by modifying two lines on /usr/share/alsa/alsa.conf:
defaults.ctl.card 1
defaults.pcm.card 1


Fix Tooltip problem

Helper hUse Custom Functions in General Function has a tooltip issue, the viewer always shows the right output while the tooltip does not when switching distance metric

Add this one to be checked with elias at the next ocasion


support scala 2.12, fix some tests who's messages are no longer the same

PS, thanks for the project Martin :)

Thanks for the updates. It's been a while since i've worked on this project and didn't realise things were out of date. Glad you find it useful!!

Hey guys. Thanks for all the hard work. Is a 2.12-compatible version of the lib released anywhere?
I tried to find scala-datatable_2.11 in https://repo.maven.apache.org/maven2/com/github/martincooper/, but it looks like it hasn't been published :(.
Would you mind publishing it there when you get the chance?

Hey. I've just published the updated version (0.8.0) to maven.
https://repo.maven.apache.org/maven2/com/github/martincooper/scala-datatable_2.12/
Enjoy :smile:


feature: allow interaction handlers of any observable type

Fixes #1260.

Coverage increased (+0.01%) to 65.574% when pulling 50fe50420d68a4f4cf2363728ee5cf51b61321c1 on kentcb:Issue1260 into baab571678939b9ec83f3e83744c97870643d98d on reactiveui:develop.

@ghuntley it's not a breaking change in the sense that if people were accidentally calling the non-observable version because their observable was not Unit, this change will actually help them.

@ghuntley not sure which docs you mean? Nothing has changed from a consumer point of view apart from the API being more forgiving of the observable type. But the docs don't talk about that any way.

Apologies, an assumption on my part that documentation would need updating. If they don't, they don't. I've started thinking about making this a standard question as merging pull requests - does the doc need updating and if so please do it. I'm keen to merge the documentation repo back into the main RxUI repo at some stage this year so documentation gets versioned along side the code.


Problems to resolve some artifacts in maven repo

Some scala or sbt related projects have scalaVersion or sbtVersion in their module name. Which confuse the coursier resolver.
Like the following artifact.
https://repo1.maven.org/maven2/net/virtual-void/sbt-dependency-graph_2.10_0.13/0.8.0/sbt-dependency-graph-0.8.0.pom

When I try to use coursier to resolve it like net.virtual-void:sbt-dependency-graph_2.10_0.13:0.8.0. Coursier tries to reach https://repo1.maven.org/maven2/net/virtual-void/sbt-dependency-graph_2.10_0.13/0.8.0/sbt-dependency-graph_2.10_0.13-0.8.0.pom which does not exists. It seems that coursier thinks sbt-dependency-graph_2.10_0.13 is the artifact's name, but the maven repo thinks sbt-dependency-graph is the actual name.
What's the workaround of this problem. Please.

The workaround is two fold:

2.10 and 0.13 are attributes (in sbt, addSbtPlugin adds these attributes to dependencies), that should be passed to coursier like net.virtual-void:sbt-dependency-graph;scalaVersion=2.10;sbtVersion=0.13:0.8.0 (from the command line, it is better to put this between quotes),
for the directory name to be appended the _2.10_0.13 suffix, the --sbt-plugin-hack option must be passed to coursier.

So the following works:
$ ./coursier fetch \
    "net.virtual-void:sbt-dependency-graph;scalaVersion=2.10;sbtVersion=0.13:0.8.0" \
    --sbt-plugin-hack

I didn't see this sbt-plugin-hack thing in the README. Maybe it should be updated.
Thank you for the workaround and quickly response.

I have another issue. I try to update a project's dependencies. But I got coursier.ResolutionException: x not found error. The artifact coursier complains not found is actually exists. I double checked. And I try to monitoring the network activity. coursier does not try to access the repository server when I run update job.

I found a walk around of this problem. After hours searching on the google, somebody mentioned that coursier tries to download artifact's jar archive and sha1 hash in parallel, for some reason the sha1 is not corresponding to the jar archive. So, delete the artifact in the cache and re-try again, then coursier could resolve that artifact correctly.


pub/media folder already exists

I have noticed that in Mac, the pub/media folder already exists and cannot be deleted.
But in Ubuntu, that folder is not present by default while creating a warden environment.
What might be the issue with Mac?

This is a difference due to how volumes are handled on Darwin hosts. On Linux, everything in /var/www/html is mounted into the container. On Mac /var/www/html is a docker volume synchronized with the host via Mutagen, and /var/www/html/pub/media is mounted from the host.


Feature authentication

What was Done

Slicing UI for Sign In Page
Integrate Sign In with Firebase Auth
Adjust Button for Sign In

Block

Wait endpoint from backend for full integration

aman si, walaupun sebenernya dummy data di /app/auth/signin/page.tsx mendingan dipindah ke /data/dummy.ts, cuma yaa fine lah


System: Create abstraction layer for entity creation

When there will be more endpoints for creation, try to create an abstraction layer.

Somehow done!


DEV 10348.1  adds env in /etc/shippable/admiral.env to decide which version should be installed on master and workers

https://github.com/Shippable/pm/issues/10348

closing, this will be done in different issue