[BUG] ...Edit Multiple Flags at Once feature inop on Firefox in macOS 10.14.6

Feature no longer works.  Worked before.
Verified that both Set Flag and Remove Flag still show and can be selected in both Edit pulldown menu and contextual menu, but flags when selected from color choices do not change in xactions, even just one. Verified that Firefox is UpToDate, YNAB was refreshed, Flags can be set for individual xactions both with contextual menu and pulldown Edit menu.  Issue exists both with and without pre-existing flag.
I expected that both contextual menu and Edit pulldown menus would not only show Set Flag and Remove Flag, but those picks would function to add, change or remove flags when the color desired is selected.
Desktop (please complete the following information):

macOS 10.14.6
Firefox latest confirmed in About Firefox
Toolkit Version [e.g. 2.27.0]

Setting Export
Please replace the text below with your exported Toolkit settings. You can export these by going to the Toolkit Options page, click the Import/Export Settings button at the bottom left and copying the text in the modal which appears.
[{"key":"AccountsDisplayDensity","value":"0"},{"key":"AccountsEmphasizedInflows","value":false},{"key":"AccountsEmphasizedOutflows","value":false},{"key":"AccountsStripedRows","value":true},{"key":"AutoDistributeSplits","value":false},{"key":"AutoEnableRunningBalance","value":true},{"key":"AutomaticallyMarkAsCleared","value":false},{"key":"BetterScrollbars","value":"0"},{"key":"BottomNotificationBar","value":false},{"key":"BudgetCategoryFeatures","value":true},{"key":"BudgetProgressBars","value":"0"},{"key":"BudgetQuickSwitch","value":true},{"key":"BudgetSpendingGoal","value":false},{"key":"BulkManagePayees","value":true},{"key":"CalendarFirstDay","value":"0"},{"key":"CategoryActivityCopy","value":true},{"key":"CategoryActivityPopupWidth","value":"0"},{"key":"CategorySoloMode","value":"cat-solo-mode-toggle-all"},{"key":"ChangeEnterBehavior","value":true},{"key":"ChangeMemoEnterBehavior","value":true},{"key":"CheckCreditBalances","value":true},{"key":"CheckNumbers","value":false},{"key":"ClearSelection","value":true},{"key":"CollapseSideMenu","value":false},{"key":"ColourBlindMode","value":false},{"key":"CompactIncomeVsExpense","value":false},{"key":"ConfirmKeyboardCancelationOfTransactionChanges","value":true},{"key":"CreditCardEmoji","value":false},{"key":"CtrlEnterCleared","value":true},{"key":"CurrentMonthIndicator","value":false},{"key":"CustomFlagNames","value":false},{"key":"DateOfMoney","value":true},{"key":"DaysOfBuffering","value":true},{"key":"DaysOfBufferingDate","value":false},{"key":"DaysOfBufferingHistoryLookup","value":"0"},{"key":"DeselectTransactionsOnSave","value":false},{"key":"DisableToolkit","value":false},{"key":"DisplayTargetGoalAmount","value":"0"},{"key":"DisplayTotalMonthlyGoals","value":false},{"key":"DisplayUpcomingAmount","value":false},{"key":"EasyTransactionApproval","value":false},{"key":"EditAccountButton","value":"2"},{"key":"EnlargeCategoriesDropdown","value":true},{"key":"EnterToMove","value":true},{"key":"FilterCategories","value":true},{"key":"GoalIndicator","value":false},{"key":"GoalWarningColor","value":false},{"key":"GoogleFontsSelector","value":"0"},{"key":"HideAccountBalancesType","value":"2"},{"key":"HideAgeOfMoney","value":true},{"key":"HideClosedAccounts","value":true},{"key":"HideHelp","value":false},{"key":"HideReferralBanner","value":true},{"key":"HideTotalAvailable","value":false},{"key":"HighlightNegatives","value":false},{"key":"ImportNotification","value":"0"},{"key":"IncomeFromLastMonth","value":"1"},{"key":"IncomeVsExpenseHoverHighlight","value":true},{"key":"LargerClickableIcons","value":true},{"key":"LinkToInflows","value":true},{"key":"MasterCategoryRowColor","value":false},{"key":"MonthlyNotesPopupWidth","value":"0"},{"key":"NavDisplayDensity","value":"0"},{"key":"NotesAsMarkdown","value":false},{"key":"POSStyleCurrencyEntryMode","value":false},{"key":"Pacing","value":"0"},{"key":"PrintingImprovements","value":true},{"key":"PrivacyMode","value":"0"},{"key":"QuickBudgetWarning","value":true},{"key":"ReconciledTextColor","value":"4"},{"key":"RemovePositiveHighlight","value":false},{"key":"RemoveZeroCategories","value":false},{"key":"ResizeInspector","value":false},{"key":"ResizeSidebar","value":true},{"key":"RightClickToEdit","value":true},{"key":"RowHeight","value":"0"},{"key":"RowsHeight","value":"0"},{"key":"SeamlessBudgetHeader","value":false},{"key":"SetMultipleFlags","value":true},{"key":"ShowCategoryBalance","value":false},{"key":"SpareChange","value":false},{"key":"SplitTransactionAutoAdjust","value":true},{"key":"SquareNegativeMode","value":false},{"key":"StealingFromFuture","value":false},{"key":"StripedBudgetRows","value":false},{"key":"SwapClearedFlagged","value":false},{"key":"TargetBalanceWarning","value":true},{"key":"ToBeBudgetedWarning","value":false},{"key":"ToggleSplits","value":true},{"key":"ToggleTransactionFilters","value":"1"},{"key":"ToolkitReports","value":true},{"key":"ViewZeroAsEmpty","value":false},{"key":"l10n","value":"0"}]

Please back up one step. I dont know where you mean for me to be when I am to tap cmd + shift + i to open. I tried that within YNAB, and within ToolKit for YNAB. Nothing happens.
WAIT — I discovered one more condition necessary for this issue to occur: I was only trying to set multiple flags within Scheduled transactions. The feature works fine on current and past Approved and Unapproved transactions.
The reason I want to change Flag state on Scheduled xactions is for three monthly bills that vary in amount, but show up 2-3 weeks before the EFT.  I have them Scheduled on the email notification date so I can update the amount due and pull date. The emails all come about the same time. After the Amounts and Dates are corrected from this month’s eBill, I want to change their Flags to Blue to designate pending status. After they pop and are approved, I go to next month’s Scheduled xactions to restore Date and Yellow flags.

On Aug 20, 2020, at 10:25, Josh Madewell<EMAIL_ADDRESS>wrote:

@Y-NWG are you getting any errors in the console? (cmd + shift + i to open)
I'm not able to reproduce.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or unsubscribe.

Also experiencing this issue on Chrome in Windows 10. I can confirm that it works in regular transactions but not scheduled transactions. There do not appear to be any related errors in the console.

Having this same issue with MS Edge 86.0.622.38 on Mac OS 10.15.7. Choosing the "Set Flag" option from the Edit menu doesn't do anything -- to singular or multiple transactions.

I have found that it only works on current and past xactions, not Scheduled. Does that help?
Romans 15:7 “...accept one another”

On Dec 13, 2020, at 05:23, bowlingotter<EMAIL_ADDRESS>wrote:

Having this same issue with MS Edge 86.0.622.38 on Mac OS 10.15.7. Choosing the "Set Flag" option from the Edit menu doesn't do anything -- to singular or multiple transactions.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or unsubscribe.

That does, actually - I was trying to flag all scheduled tasks so when they
came current before the bank registered the transactions, I'd be able to
tell at a glance that I initially put them in as scheduled.
On Sun, Dec 13, 2020, 11:34 AM Y-NWG<EMAIL_ADDRESS>wrote:

I have found that it only works on current and past xactions, not
Scheduled. Does that help?
Romans 15:7 “...accept one another”

On Dec 13, 2020, at 05:23, bowlingotter<EMAIL_ADDRESS>wrote:

Having this same issue with MS Edge 86.0.622.38 on Mac OS 10.15.7.
Choosing the "Set Flag" option from the Edit menu doesn't do anything -- to
singular or multiple transactions.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or unsubscribe.

—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/toolkit-for-ynab/toolkit-for-ynab/issues/2125#issuecomment-744033441,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ASDK7W3BDJ7AWICHI53QRXLSUTUKLANCNFSM4QGESBMQ
.

This feature was removed as it's now supported natively by YNAB.


Targeting x86_64-apple-darwin on Linux yields linking errors

Targeting x86_64-apple-darwin on Linux yields linking errors.
source.0.o: file not recognized: File format not recognized
collect2: error: ld returned 1 exit status

To reproduce :
From this Docker container: https://gist.github.com/netgusto/77a8faaa8bf348468c7e6193cba529b6
$ cd /source
$ rustup target add x86_64-apple-darwin
$ cargo init --bin
$ cargo build --target=x86_64-apple-darwin

   Compiling source v0.1.0 (file:///source)
error: linking with `cc` failed: exit code: 1
note: "cc" "-m64" "-L" "/root/.multirust/toolchains/stable-x86_64-unknown-linux-gnu/lib/rustlib/x86_64-apple-darwin/lib" "/source/target/x86_64-apple-darwin/debug/source.0.o" "-o" "/source/target/x86_64-apple-darwin/debug/source" "-Wl,-dead_strip" "-nodefaultlibs" "-L" "/source/target/x86_64-apple-darwin/debug" "-L" "/source/target/x86_64-apple-darwin/debug/deps" "-L" "/root/.multirust/toolchains/stable-x86_64-unknown-linux-gnu/lib/rustlib/x86_64-apple-darwin/lib" "/root/.multirust/toolchains/stable-x86_64-unknown-linux-gnu/lib/rustlib/x86_64-apple-darwin/lib/libstd-4fda350b.rlib" "/root/.multirust/toolchains/stable-x86_64-unknown-linux-gnu/lib/rustlib/x86_64-apple-darwin/lib/libcollections-4fda350b.rlib" "/root/.multirust/toolchains/stable-x86_64-unknown-linux-gnu/lib/rustlib/x86_64-apple-darwin/lib/librustc_unicode-4fda350b.rlib" "/root/.multirust/toolchains/stable-x86_64-unknown-linux-gnu/lib/rustlib/x86_64-apple-darwin/lib/librand-4fda350b.rlib" "/root/.multirust/toolchains/stable-x86_64-unknown-linux-gnu/lib/rustlib/x86_64-apple-darwin/lib/liballoc-4fda350b.rlib" "/root/.multirust/toolchains/stable-x86_64-unknown-linux-gnu/lib/rustlib/x86_64-apple-darwin/lib/liballoc_jemalloc-4fda350b.rlib" "/root/.multirust/toolchains/stable-x86_64-unknown-linux-gnu/lib/rustlib/x86_64-apple-darwin/lib/liblibc-4fda350b.rlib" "/root/.multirust/toolchains/stable-x86_64-unknown-linux-gnu/lib/rustlib/x86_64-apple-darwin/lib/libcore-4fda350b.rlib" "-l" "System" "-l" "pthread" "-l" "c" "-l" "m" "-l" "compiler-rt"
note: /source/target/x86_64-apple-darwin/debug/source.0.o: file not recognized: File format not recognized
collect2: error: ld returned 1 exit status

error: aborting due to previous error

Could not compile `source`.

Environment:
$ uname -a
Linux bd1bb678d87a 4.4.9-moby #1 SMP Tue May 10 10:36:54 UTC 2016 x86_64 x86_64 x86_64 GNU/Linux
$ rustc --version
rustc 1.8.0 (db2939409 2016-04-11)
$ rustup --version
rustup 0.1.12 (c6e430a 2016-05-12)

You'll at the very least need a linker capable of linking Mach-O files. rustup makes it easy to get pre-compiled versions of std for your target platform, but you still need a linker and potentially a C compiler which also target that platform, along with any platform-specific libraries that need to be linked in.

If anyone has a procedure to make it compile, please share it ! Otherwise, I guess this can be closed.

Since there's no practical solution to this on the horizon, and it's similar to other cross-compile scenarios, I'm going to close this. Thanks for the report.

The Go toolchain supports compiling and linking Linux, Mac, and Windows targets from any of these hosts. Could rustup get something similar, to make build bots easier to setup?


Split data then run cell2location

Hello, thank you for this valuable tool. I have a question regarding memory usage in Cell2Location. During the deconvolution step, I'm encountering an 'out of memory' issue with the following code
adata_vis = mod.export_posterior(
    adata_vis, sample_kwargs={'num_samples': 1000, 'batch_size': mod.adata.n_obs, 'use_gpu': True}
)

To address this, I'm considering splitting the data and running Cell2Location for each sub-data. Will the results be the same in this case?

Hi @hahia
If you can train the model on full data, I would strongly recommend continuing to do so, but instead replacing the mod.export_posterior with a more memory-efficient alternative by computing the quantiles directly:
adata_vis = mod.export_posterior(
    adata_vis, use_quantiles=True,
    # choose quantiles
    add_to_obsm=["q05","q50", "q95", "q0001"],
    sample_kwargs={'batch_size': 2500, 'use_gpu': True}
)

note that you would also have to specify mod.plot_QC(summary_name= 'q50') instead of mod.plot_QC(summary_name= 'means') (default).

Hi @vitkl,
I'm having the same error as above, running out of memory when running mod.export_posterior.
I'm trying to run mod.export_posterior by computing quantiles directly, as explained in your reply above, however, I am getting the following error:
TypeError: export_posterior() got an unexpected keyword argument 'use_quantiles'
Is there any way to fix this?
Thank you!
Version of cell2location: 0.1

For the quantiles option you need to install the latest cell2location version from GitHub

Great, thank you for your help, I'll try installing the latest version


Expected prop defaultOptions

Getting warning for,  was created without expected prop 'defaultOptions'.
Could not find info in the docs to remove this.

#7


Varonis saas dev

Required items, please complete
Change(s):

Add new solution that includes connection to Varonis DatAlert service and workbook

Reason for Change(s):

create new data connector that ingest data from Varonis DatAlert service

Version Updated:

Required only for Detections/Analytic Rule templates
See guidance below

Testing Completed:

Need help

Checked that the validations are passing and have addressed any issues that are present:

See guidance below

Guidance <- remove section before submitting

Before submitting this PR please ensure that you have read the following sections and filled out the changes, reason for change and testing complete sections:
Thank you for your contribution to the Microsoft Sentinel Github repo.

Details of the code changes in your submitted PR.  Providing descriptions for pull requests ensures there is context to changes being made and greatly enhances the code review process.  Providing associated Issues that this resolves also easily connects the reason.

Change(s):

Updated syntax for XYZ.yaml

Reason for Change(s):

New schema used for XYZ.yaml
Resolves ISSUE #1234

Version updated:

Yes
Detections/Analytic Rule templates are required to have the version updated

The code should have been tested in a Microsoft Sentinel environment that does not have any custom parsers, functions or tables, so that you validate no incorrect syntax and execution functions properly.  If your submission requires a custom parser or function, it must be submitted with the PR.

Testing Completed:

Yes/No/Need Help

Note: If updating a detection, you must update the version field.

Before the submission has been made, please look at running the KQL and Yaml Validation Checks locally.
https://github.com/Azure/Azure-Sentinel#run-kql-validation-locally

Checked that the validations are passing and have addressed any issues that are present:

Yes/No/Need Help

Note: Let us know if you have tried fixing the validation error and need help.

References:

Guidance for Detection checks
General contribution guidance
PR validation troubleshooting

Hello @vkorenkov-varonis,

Please resolve the validation error for KQL error.
Please add Data connector schema at location .script/utils/schemas. You can refer to other scehmas for more clarification
Add workbook metadata in the file Workbooks/WorkbooksMetadata.json
Also add working images of workbook.
Add input folder alias Data folder and solution metadata which is required to repackage the solution. You can refer to any other solution for more clarification

Hello @vkorenkov-varonis, Please add workbook metadata to the above mentioned file. Also add working images of workbook as well.

@microsoft-github-policy-service agree [company="Varonis"]

@microsoft-github-policy-service agree

Hello @vkorenkov-varonis, Thanks for committing the requested changes. Will take a look at it and come back to you by 18 January, 2024.
Thanks

Thank you for the updates @vkorenkov-varonis, Will check on it once and come back to you by 24 January, 2024.

Working images are function apps output results

@v-prasadboke

Hello @vkorenkov-varonis we are all good with the solution. Just need a few more line to solution description.

Hello @vkorenkov-varonis, Solution description needs a few more line. Please add them.

Hello @vkorenkov-varonis, Solution description needs a few more line. Please add them.

waiting for copywriter

Hello @vkorenkov-varonis, Thanks for raising this PR. This PR will be investigated and will update you about the same before 01 February, 2024


https://serverless.tf/ is down

https://isitdown.vercel.app/?u=https://serverless.tf/

Apparently, DNS changes I did yesterday were wrong 🤔
Huge thanks for telling me! I'll fix it a bit later today (traveling)

Fixed. Thanks again for reporting.
PS: DNSimple UI is faaaar from easy to understand :((


[Python] Port MathSkill from C# to python3

Motivation and Context
Port MathSkill from C# to python3. This is essential for advanced skill like plan
Description

The PR includes two part. The first part is an implementation of MathSkill. The implementation follows C# version.
It does several things:

Parse initial value
Get value from context
Sum or subtract

The unit test covers these scenarios:

Add/subtract value
Error should throw if string is not correct. Not able to parse.

Last part is the init.py and feature matrix.
Unit test pass locally.
Contribution Checklist

[-] The code builds clean without any errors or warnings
[-] The PR follows SK Contribution Guidelines (https://github.com/microsoft/semantic-kernel/blob/main/CONTRIBUTING.md)
[-] The code follows the .NET coding conventions (https://learn.microsoft.com/dotnet/csharp/fundamentals/coding-style/coding-conventions) verified with dotnet format
[-] All unit tests pass, and I have added new tests where possible
[-] I didn't break anyone :smile:

thank you @poweihuang0817 !


fixes #2707 - typo

Curiously browsing the repository, I thought a quick fix for #2707 would be welcome.
Let me know if anything needs to be changed.

Thanks for the pull request, @SphinxKnight!
However, I believe the existing text highlighted as a possible typo in #2707 is correct as written, as the discussion is in the context of GDI support services.
Please see the original issue for additional available resources.

Thanks and sorry. I thought it was a legitimate fix according to a previous sentence in the article which states "Like kernel-mode graphics DLLs, user-mode graphics DLLs must call the graphics DDIs that create or modify a drawing surface. " (https://docs.microsoft.com/en-us/windows-hardware/drivers/print/choosing-user-mode-or-kernel-mode?source=docs#using-the-graphics-ddi-in-user-mode)


Apply formatting on:blur event

When the user clicks away from the field, we should run setFormattedValue() to apply the formatting one last time.

Hi, there! I have a question. When the user clicks away from the field, value doesn't change, correct? So why do we need to call setFormattedValue()?

@gokullan there are some scenarios where formatting is not automatically applied, more specifically:

When pressing Backspace and the formattedValue is ['$', '-$', "-"]
When the decimal point is . and you type 123. but don't enter decimals (same if  , decimal point)

In these cases, if you click away from the input it will just stay unformatted.
It'd be nice if the on:blur event applied the formatting, which for the cases above would end up looking like:

$0.00
$123


Zed shadows (W spell) is broken (they are looking like classic shadows)

i need a new update (and I would also like to thanks the guy who created the skin changer)

#1

Also Galaxy slayer zed skin q flick


GH91: Create ZIP artifact.

Fixes #91

@gep13 do I need to take any further actions in order to get artifact uploaded to appveyor AND to github release?

@mholo65 at the minute, Cake.Recipe only uploads files that are located in the NuGetPackages or ChocolateyPackages folder, to both AppVeyor and GitHub Release.  You could trick it into doing the work by outputting the Zip file into one of these locations, but this is not ideal.  Perhaps, in Cake.Recipe, a generic artifacts folder is required that you can place any files into, and those files will be published on build.  AppVeyor already has this feature built in, and will look for a specific folder, and place them as artifacts on the AppVeyor build, however, this wouldn't help with the publishing to the GitHub Release.


Azure provider.tf generated is missing features block

The provider block for Azure needs to include an empty features block.
Currently:
provider "azurerm" {

  subscription_id = var.subscription_id
  client_id = var.client_id
  client_secret = var.client_secret
  tenant_id = var.tenant_id
}

Should be:
provider "azurerm" {
  features { }

  subscription_id = var.subscription_id
  client_id = var.client_id
  client_secret = var.client_secret
  tenant_id = var.tenant_id
}

This causes terraform to fail when being run on generated output.

Fixed in v2.25.5


[flink-5570] [Table API & SQL]Support register external catalog to table environment

This pr aims to support register external catalog to TableEnvironment.
The pr contains 3 commits, the first and second are about https://issues.apache.org/jira/browse/FLINK-5568, it's content is as same as (https://github.com/apache/flink/pull/3406).
The third one is to support externalCatalog registration. So please focus on the third commit when you review this pr.
The main changes in the second commit including:

add registerExternalCatalog method in TableEnvironment to register external catalog
add scan method in TableEnvironment to scan the table of the external catalog
add test cases for ExternalCatalog, including registration and scan

@fhueske , thanks for the review. I have updated the pr based on your suggestions.

@fhueske @twalthr , thanks for the reviews. I've updated the pr based on your comments.

Merging


First line of of any comment has different font weight

Occurring in Chrome version 126.0.6478.62 on macOS with extension version 1.4.11. From a quick glance at dev tools, it seems that the first line of the comment is inheriting from the html.hnspecial-theme, html.hnspecial-theme body rule. Any subsequent p tag in the comment body is then styled with the html.hnspecial-theme p, html.hnspecial-theme div.comment > span:first-child rule.
Not sure if this is intended behavior. Attached is a screenshot of the issue.

At first, I thought this was due to a change in the extension. But the changelog doesn't indicated as much and it occurred between releases. Here's a slice of DOM, fwiw:

Opened https://github.com/gabrielecirulli/hn-special/pull/134 to resolve it


TASK-13 Remove deprecated code

The task is to remove :

all Serializer related implementation,
SerializerType class,
SerializerFactory.

And finally move into the test classes:

GsonSerializer,
JacksonSerializer,
MoshiSerializer

Development is ready on feature branch.


[Plasma Stick 2040W] add Plasma LEDs link

Add link to other resources section for Plasma LEDs.
Control LEDs attached to a Pimoroni Plasma Stick 2040 W with your web browser.
The Plasma LEDs features a clean web frontend, which allows you to control the LEDs attached to a Pimoroni Plasma Stick 2040 W via a web browser.
It comes with a variety of configurable colors and effects out of the box, and is extendable to additional effects if you have the programming skills.
Some of the effects are based on the awesome example code provided by Pimoroni.

Love this project - thanks very much for adding the link! 🌈


binary with bt classic enabled

Hi everyone,
I am not able to create binary with esp-idf. I need a binary with bt classic enabled. Do anyone has that, if yes, can you please provide me?
and what address I should flash it with esp flash tool?

https://www.espressif.com/zh-hans/support/download/at


Implement gpio:init/1 on esp32

Some GPIO pins require initialization as gpio pins to work, for example GPIO 4 on ESP32C3. This is achieved by calling esp-idf gpio_config. Implement this within gpio:init/1 which is used on rp2040.
These changes are made under both the "Apache 2.0" and the "GNU Lesser General
Public License 2.1 or later" license terms (dual license).
SPDX-License-Identifier: Apache-2.0 OR LGPL-2.1-or-later

Is there any documentation in ESP-IDF about which pins require this special use case? This is new to me, and I can’t find any reference to this being needed for certain pins. We definitely need clear documentation about which pins this is required for to enable them.

This definitely hammers home the need to re-write our GPIO APIs so that they are unified across platforms (and now chipsets within platforms).

Is there any documentation in ESP-IDF about which pins require this special use case? This is new to me, and I can’t find any reference to this being needed for certain pins. We definitely need clear documentation about which pins this is required for to enable them.

The documentation says:

Do not rely on the default configurations values in the Technical Reference Manual, because it may be changed in the bootloader or application startup code before app_main.

https://docs.espressif.com/projects/esp-idf/en/latest/esp32/api-reference/peripherals/gpio.html
Because we didn't call gpio_config, we did rely on previous configuration.
Explicitely, the technical reference manual of esp32c3 (page 169) says that pin 9 of esp32-c3 has function 0 MTMS and function 1 GPIO4, so I'm not surprised I had to call gpio_config. There also are functions to select pin functions, but the API changed between esp idf versions, so I favored gpio_config in gpio:init/1.
https://www.espressif.com/sites/default/files/documentation/esp32-c3_technical_reference_manual_en.pdf
The table seems to document that gpios 4-7, 12-17 and 20-21 would need this. Not all are exposed on popular esp32c3 boards though.


Added set_misc redis2 and ngx_lua modules

Added 3rd party extensions set_misc redis2 and ngx_lua.
Commented out some deprecated directives.
Commented out the 3rd party mp4 (its now baked in)
Commented out 3rd party hashmodule (its now baked in)

Thank you for contributing to Nginx!
Nginx does not use GitHub for code submissions.
Please visit http://nginx.org/en/docs/contributing_changes.html and follow the instructions there to submit changes.


Discussion: Does UWP map control support GeoJson?

Discussion: Does UWP map control support GeoJson?

Does UWP map control support GeoJson? If not when do we expect to have this support added?
Related Links

Bing maps v8 web control already supports it. Link here

@MikeHillberg who can we ask about the map control?

The WinUI team does not own MapControl directly. I received some clarification from the team that does: there is currently no built-in support for GeoJson, this would be a new feature request. In the meantime, you should be able to parse the GeoJson and add MapElements yourself.

The WinUI team does not own MapControl directly. I received some clarification from the team that does: there is currently no built-in support for GeoJson, this would be a new feature request. In the meantime, you should be able to parse the GeoJson and add MapElements yourself.

Thank you @YuliKl
Is there a repo for MapControl where I can open a suggestion to track it?

The WinUI team does not own MapControl directly. I received some clarification from the team that does: there is currently no built-in support for GeoJson, this would be a new feature request. In the meantime, you should be able to parse the GeoJson and add MapElements yourself.

Thank you @YuliKl
Is there a repo for MapControl where I can open a suggestion to track it?

I think Project Reunion is probably the best place


Extend E2 Tokenizer String Escapes

Modifies tokenizer to escape using WireLib.ParseEscapes. This makes writing unicode strings a lot easier in E2.
I noticed compiler rewrite doesn't actually touch this part of the tokenizer so I thought there'd be no harm if I did it.
Demonstration
print("H\101ll\u{6f},\tWo\x72ld!\n\u{2615}\999\xwb\xbw\u123\u{110000}\" \
\n\u{}\u{1234567}\u")

I really would prefer not having a dependency on WireLib inside of the E2 Compiler. Additionally the implementation seems pretty slow, even if this would run only with string escapes, in the future, ideally an editor would use E2's tokenizer for highlighting rather than the garbage it currently has

If you just want \u{123456} support, add a hardcoded pattern case for \\u{(%x*)} and tonumber(slice, 16). \0 can simply be added to the Escapes table.

It's quite fast. It's not just \u support that it adds, but also \xx support

I really would prefer not having a dependency on WireLib inside of the E2 Compiler. Additionally the implementation seems pretty slow, even if this would run only with string escapes, in the future, ideally an editor would use E2's tokenizer for highlighting rather than the garbage it currently has

I could rewrite this to not depend on WireLib but I went for this route because I thought it would be sufficient and simple. I see benefits to both approaches, but I opted for this because it's less, mostly redundant, code.

If you really want \u{123456} support, add a hardcoded pattern case for \\u{(%x*)} and tonumber(slice, 16). \0 can simply be added to the Escapes table.

I think users would appreciate the options of using all valid Lua escape sequences, which is why I made the WireLib function to begin with.

It's not just \u support that it adds, but also \xx support

Imo b'B' is much more readable than "\66".
And I don't think many languages besides lua do this, so I don't really agree with having it in E2.
(JS, Python, C++ parse \66 as 6, Rust, Go, C# all error at compile time)

E2 should not take any syntax or behaviors from Lua

I don't have a problem with that but besides \nnn being decimal instead of octal, I don't see that big of a discrepancy.
Rust and C# don't specify for \nnn escapes, Go requires 3 characters.

You can have invalid hex escape as an error, I only had the other escapes be a warning for backwards compat.
Also you're gonna want to add some tests to ensure the escapes work

You can have invalid hex escape as an error

I didn't think it was worth erroring over.

You can have invalid hex escape as an error

I didn't think it was worth erroring over.

Any potentially undefined / weird behaviors should be avoided and should just error.
I often look at the typescript playground to see if certain stuff errors / what messages they give off.
Invalid hex literals do give errors, so they should with E2 as well.

Is this good now, @Vurv78 ?

Will test in a bit and merge if it works fine for me

Before I do that, would any of you be against \<newline character> interpreting as an empty string?

Before you do that, would any of you be against \<newline character> interpreting as an empty string?
print("Demo \
test")
# Outputs "Demo test" with no break

Why? E2 strings are always multiline, there's no reason for this besides an odd preference for longer code

Why? E2 strings are always multiline, there's no reason for this besides an odd preference for longer code

Just thought it'd be neat.


[Request] Heat and Climate mod Support

Could you add support for Heat and Climate mod ,if you would not mind?
https://www.curseforge.com/minecraft/mc-mods/heat-and-climate

i can look into it

i can look into it

https://www.curseforge.com/minecraft/mc-mods/tinkers-reforged/files/3178347 there you go.

https://www.curseforge.com/minecraft/mc-mods/tinkers-reforged/files/3178347 there you go.


syzkaller, syzbot: attach fsck results to bug reports

The topic has come up multiple times already, last time at the latest LPC (#5331).
We could run fcsk.* on the disk images mounted by the bug reproducer (we already extract them and report them separately) and attach the results to the bug reports:

The overall result: passed/failed.
Detailed fsck output.

FTR: #5518


Upgrade to Dataverse 4.14

Upstream released v4.14. New images needed :smile:
Upstream release docs state no special treatments, so this should be straight forward.

Tagged and done.


WebSocket widget id

Hi i was wondering if the "/clr/overlay/" is sent to the websocket.py manager and if so if it was possible to send to all clients with a specific id?

The reason for this would be so that i can send emotes on to one overlay and other features onto another overlay.

or for a betting system on a different overlay so the streamer can move it on their end and not effect the show emote placement.

Nevermind i worked it out, to do it you must send the widget id to the server via the clr and then add that to the client object and then only send to specific clrs's based on the widget id


Thumbnail for feed is different in the current master build

After switching from the release build of 5.1.3 to building from master, I noticed the thumbnail for the RSS feed https://xkcd.com/rss.xml is different. It no longer uses the website's thumbnail but it uses the first image in the XML in this case:
Old

New

Was this an intentional change? If so, is this logged on a changelog somewhere? The latest changelog I could find was from 5.0.1.

This is intentional.  NNW 6.0 uses higher resolution images for the feed icon if available.  In this case, what you are seeing is the image specified in the open graph image (og:image) from the XKCD homepage.

I'm going to reopen this issue just to fix XKCD.  They seem to be switching the og:image to always match the current comic.  These are almost never legible when turned into an icon.  I'll be adding this to our homePagesWithUglyIcons array so that we don't use og:image when loading XKCD.


Possible border radius for dialogue?

When using the dialogue component, it seems that I'm unable to add a borderRadius to it. Has anyone tried to do this?
Thanks!

You can set the default border radius using MuiThemeProvider:
import YourApp from './YourApp'
import ReactDOM from 'react-dom'
import {createMuiTheme, MuiThemeProvider} from '@material-ui/core/styles'

const theme = createMuiTheme({
    shape: {
        borderRadius: 2
    },
    palette: ...,
    ...
})

ReactDOM.render(
    <MuiThemeProvider theme={theme}>
        <YourApp/>
    </MuiThemeProvider>,
    document.getElementById('root')
)

This will create a global styling applied to all Material-UI components across your application on any level of nesting.
If you don't want other components in your app to be affected, you can always add a className to your Dialog component and set it in your CSS:
.yourClassName {
    border-radius: 2px;
}

<Dialog className="yourClassName">
    ...
</Dialog>

Thank you, that worked great!

I solved it with paperProps property.
<Dialog
  paperProps={{
    style: { borderRadius: 2 }
  }}
  >
  ....
</Dialog>

Here's for the Drawer https://codesandbox.io/s/vmj9q
@nakanori Is there a simpler way like yours?

I solved it with paperProps property.
<Dialog
  paperProps={{
    style: { borderRadius: 2 }
  }}
  >
  ....
</Dialog>

I think Paper P is capital

You can set the default border radius using MuiThemeProvider:
import YourApp from './YourApp'
import ReactDOM from 'react-dom'
import {createMuiTheme, MuiThemeProvider} from '@material-ui/core/styles'

const theme = createMuiTheme({
    shape: {
        borderRadius: 2
    },
    palette: ...,
    ...
})

ReactDOM.render(
    <MuiThemeProvider theme={theme}>
        <YourApp/>
    </MuiThemeProvider>,
    document.getElementById('root')
)

This will create a global styling applied to all Material-UI components across your application on any level of nesting.
If you don't want other components in your app to be affected, you can always add a className to your Dialog component and set it in your CSS:
.yourClassName > div:nth-child(2) {
    border-radius: 2px;
}

<Dialog className="yourClassName" open>
    ...
</Dialog>

How did you come to know that we have to use "shape"?
Is there any documentation to refer?
If yes, please provide a link.
Thank you

I solved it with paperProps property.
<Dialog
  PaperProps={{
    style: { borderRadius: 2 }
  }}
  >
  ....
</Dialog>

Thank you @nakanori

I solved it with paperProps property.
<Dialog
  PaperProps={{
    style: { borderRadius: 2 }
  }}
  >
  ....
</Dialog>

This worked, Thank you so much

I solved it with paperProps property.
<Dialog
  PaperProps={{
    style: { borderRadius: 2 }
  }}
  >
  ....
</Dialog>

Thanks!!


Gesture Recognition

Does the project contain the function of Gesture Recognition?
it is easy to know the hand gesture,  like  one 、two、three、ok....
like this :https://ai.googleblog.com/2019/08/on-device-real-time-hand-tracking-with.html

If such a calculator is not included in mediapipe's repository, no.

see this :https://gist.github.com/TheJLifeX/74958cc59db477a91837244ff598ef4a


[SPARK-45328][DOCS][FOLLOWUP] Update docs for Hive metastore supported versions

What changes were proposed in this pull request?

A minor follow-up of https://github.com/apache/spark/pull/43116
Why are the changes needed?

Correct the docs of spark.sql.hive.metastore.version
Does this PR introduce any user-facing change?

Yes, docs changed.
How was this patch tested?

Review
Was this patch authored or co-authored using generative AI tooling?

No

cc @HyukjinKwon

+1, LGTM. Merging to master.
Thank you, @pan3793 and @yaooqinn @ulysses-you for review.


Convert res to response in APIController tests

Fixes #60
Tests cleanup to use convention in other tests of res to response.
Expected Test Results:
  ․․․․․․․․․․

  6 passing (15s)
  4 failing

👍

Adjusted tests to move to application wide convention of res and req.
  ․․․․․․․․․․

  6 passing (13s)
  4 failing

Great, thanks!


Documents the safeAreaInset prop for <TabBarBottom>

Closes https://github.com/react-navigation/react-navigation.github.io/issues/197

thank you!


Only show indent guides for current block (resolves #561)

This should resolve #561. If you want anything to change I can edit it later today.
One thing to consider is if there should be an option to change the behavior when you are not in any code block. Right now it will not show any indent guides when you are not in a code block, and then only show the blocks indent guides when you are in a code block. I think this could be useful for viewing large files with multiple different blocks, which is why I picked this behavior as the standard.
Also, I am not sure exactly how you would want to write tests for this, but if you point me in the right direction, I can also try to look into that.

I think it makes more sense if this builds on current indent right?

You are right, but I thought some people will want to use this without using current_indent. But I guess I can just make it so that you only do the necessary calculation if either current_indent needs it or this needs it. I will make a new pull request with those features both then.

Sorry, I will reopen this, but I improved it so it doesn't use scope now. It was not a good idea to use scope since it doesn't capture the right thing, but I do think we want to use treesitter for this. Without treesitter, it can be hard to describe the exact block you are in (this is a different thing than what current_indent does, since it doesn't care where the block starts/ends, it just looks at where the indent guides start/end).
All languages I have tested (and I think this should generally be true for treesitter?) has new code blocks as the child of the root, so going up the tree until you see the root should always work (if my assumption is true).

I don't think treesitter is the right tool for this.
This is basically current indent, but walking up all indent levels until you hit the root.
For current indent, you keep a stack of all indents until you hit the cursor, then display the deepest one differently.
For this feature, you would display all indents in the stack, and nothing else.

Ah, I think I misunderstood what you said originally. Keeping a stack similar to current_ident but for current_block like what you described should definitely be possible. I can try to do that later.
I think I will wait until current_indent has been accepted though, so I will know what I can use from that pull request.


Add local storage with full CRUD functionality

Addressing issue #24

After I merged your PR this is what the deployed site showed me. I tried refreshing the page but it didn't work


Have the ability to specify the base URL as a client option

Describe the feature you'd like to request
I'd like to be able to use my local instance of the PokeAPI with this library.
For the app I'm creating, I use the PokeAPI to generate data files during development time. To avoid making lots of calls to the PokeAPI, I host a local instance of it. I would like to use this library with my local instance of the PokeAPI.
Describe the solution you'd like
I believe that adding the base URL as an optional parameter to the ClientArgs interface would be the best solution and, if empty, use BaseURL.REST.
Describe alternatives you've considered
I created a wrapper class around MainClient that replaced the api instance of each client with my own:
import { ClientArgs, MainClient } from "pokenode-ts"
import { setup } from 'axios-cache-adapter';

export class CustomClient extends MainClient {
  constructor(clientArgs?: ClientArgs) {
    super(clientArgs);

    [
      this,
      this.berry,
      this.contest,
      this.encounter,
      this.evolution,
      this.game,
      this.item,
      this.location,
      this.machine,
      this.move,
      this.pokemon,
    ].forEach(_ => _.api = setup({
        baseURL: 'http://localhost/api/v2',
        headers: {
          'Content-Type': 'application/json',
        },
        cache: {
          maxAge: clientArgs?.cacheOptions?.maxAge || 0,
          ...clientArgs?.cacheOptions,
        },
      })
    )
  }
}

If the proposal is accepted, I could code the feature.

Hey @simonorono. LGTM, feel free to contribute to pokenode-ts with this feature.

Released in v1.8.0.


icons: replace angular icons with core ones

We will need to update the website and provide guidelines on how to migrate to cds-icon. I prefer doing that after the code changes are approved.
PR Checklist
Please check if your PR fulfills the following requirements:

[x] Tests for the changes have been added (for bug fixes / features)
[x] Docs have been added / updated (for bug fixes / features)
[ ] If applicable, have a visual design approval

PR Type
What kind of change does this PR introduce?

[ ] Bugfix
[x] Feature
[x] Code style update (formatting, local variables)
[x] Refactoring (no functional changes, no api changes)
[x] Build related changes
[ ] CI related changes
[x] Documentation content changes
[ ] clarity.design website / infrastructure changes
[ ] Other... Please describe:

What is the current behavior?
The clarity angular icons are used everywhere in the components
What is the new behavior?
All angular icons are replaced with the ones provided by core
Does this PR introduce a breaking change?

[ ] Yes
[x] No, maybe 🤷♂

Is this blocked still or just needs to have conflicts resolved?

Is this blocked still or just needs to have conflicts resolved?

Conflicts have to be resolved.


Change Youtube link based on language selection

started in #58 , (en & fr done) other language urls to be updated over time


Responsive Menu not toggling

I have Foundation 6 SASS installed on a clean WP instance on my localhost. One copy i've modified over the last 2 weeks and the responsive menu always worked. A few days it stopped working for some reason, without producing any CSS or JS errors
What i noticed was that the data attribute "data-responsive-toggle" was not triggering the element "site-navigation".
To do a sanity check, i git cloned a fresh copy of this theme and while it did toggle the visiblity in CSS, nothing showed up on screen as far as the responsive menu goes.
Of course it works on the site: https://foundationpress.olefredrik.com/
I'm really stumped why it was working before, and not now - especially with a fresh build of this theme.
Has anyone come across this before?

I haven't pinpointed the exact issue - but have resolved it by updating Foundation core / SASS files to version 6.2.2 rc1. I only picked rc1 because it had numerous bug fixes, a totally blind solution but seems to have addressed the issue i was having.


launcher: Use actual executable path instead of argv[0]

When Jazzer is executed from PATH, argv[0] will just be jazzer, which we can't find jazzer_standalone.jar relative to.
Also fix argv[0] in the case where no shell launcher is generated.
Fixes #641

The code path is also used in all Bazel tests now as we pass the result of the new function to the C++ runfiles library. But I will try to add coverage for PATH invocation specifically.


Sentry stacktraces do not show code

Stacktraces from python - beautiful inline code

Stacktraces from dart - no inline code snippets :(

I understand that python is a super dynamic language and can pick up the code using magic like the inspect module, but is there a way to "give" sentry my dart code too?

fluter doctor -v

[✓] Flutter (Channel stable, v1.17.5, on Mac OS X 10.14.6 18G4032, locale en)
    • Flutter version 1.17.5 at /Users/dev/Tools/flutter
    • Framework revision 8af6b2f038 (6 weeks ago), 2020-06-30 12:53:55 -0700
    • Engine revision ee76268252
    • Dart version 2.8.4

[✓] Android toolchain - develop for Android devices (Android SDK version 29.0.3)
    • Android SDK at /Users/dev/Library/Android/sdk
    • Platform android-29, build-tools 29.0.3
    • Java binary at: /Users/dev/Library/Application Support/JetBrains/Toolbox/apps/AndroidStudio/ch-1/192.6392135/Android
      Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_212-release-1586-b4-5784211)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 11.2.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 11.2.1, Build version 11B500
    • CocoaPods version 1.8.4

[✓] Android Studio (version 3.6)
    • Android Studio at /Users/dev/Library/Application Support/JetBrains/Toolbox/apps/AndroidStudio/ch-1/192.6392135/Android
      Studio.app/Contents
    • Flutter plugin version 45.1.1
    • Dart plugin version 192.8052
    • Java version OpenJDK Runtime Environment (build 1.8.0_212-release-1586-b4-5784211)

[✓] VS Code (version 1.46.1)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.12.1

[✓] Connected device (1 available)
    • POCO F1 • e19ec1f6 • android-arm64 • Android 10 (API 29)

• No issues found!

Hi @devxpy,
Does this sound similar to your question / request ?
Thanks.

Umm, I'm not quite sure whether I understand if https://github.com/flutter/flutter/issues/37875 is about sentry at all.
As far as source maps go, flutter doesn't need those, because there's no transpiling step that I'm aware of.
And the part about "Sometimes sentry doesn't give you something useful because the the stacktrace is only flutter internal libraries" seems only related to exceptions from flutter engine, which dont bubble up to user code.
This issue is strictly about showing source code snippets with stacktraces.

Thanks for fixing my typo :)

We'll investigate if it's possible at all and how we could do this.
One approach is to allow users to run sentry-cli to upload a "source bundle" and on the server we lookup the relevant lines of code on that bundle as the events come in.

@bruno-garcia I've tried uploading dart source files -
sentry-cli releases files "$VERSION" upload ./lib

But the source code still doesn't show in-line with the stack trace on sentry.io

There is not support yet. This will require some work on tooling (get the source code available on the server) and on the backend. We haven't started this work yet.
I've raised an issue in the Sentry Dart/Flutter SDK repository (moved away from Flutter's org):
The issue will be tracked here: https://github.com/getsentry/sentry-dart/issues/132 and for that reason I'll close the issue here.


Create lucky.py

What the program do ?
----print hello world in python
In what programming language it is written?
in python

This pull request do not contains files in specific folders. Please fix it and reopen


API for push mirrors

Implement push mirrors APIs
Fixes: #19401

how does this differ from  #19841

how does this differ from #19841

I didn't notice that one. This pull request could be closed if #19841 gets merged.


added '--rm' to docker command to clean up container after use

I didn't see anything that indicated I needed to manually up the version, but if there is, I'll do that before merging.
This patch adds --rm to the arguments passed to Docker when buf is invoked. This cleans up the list of stopped containers on a system.
For testing, I ran the tests included in ./gradle test and watched as containers were created to run buf and were immediately deleted after they terminated. All tests still passed.

This project is versioned using the semantic versioning plugin.

Published in version 0.3.2.


Clarify setupFilesAfterEnv prerequisite for singleton unit testing guide

In our unit testing guide, we offer a singleton approach to mocking the Prisma Client. This approach only works if you have put the singleton.ts file in the setupFilesAfterEnv option of your jest.config.js.
We make this note under the Add Jest to your project dropdown, but it's easy to miss and the guide will not work without this extra piece of configuration.
We could add another note about setupFilesAfterEnv later down where we mention creating singleton.ts or remove the the Jest setup from the dropdown entirely.
see: https://github.com/prisma/prisma/discussions/12057

TODO:

[ ] Ensure Prerequisites are correct, if any
[ ] Clarify respective prerequisites for each guide
[ ] Clarify shared prerequisites for the guides


InputHint - acceptingInput - for a single message activity turns on microphone, while its said only 'expectingInput' will turn on microphone

Version
Webchat version :  "4.4.2-0" -  npm package
Describe the bug
when a single message activity is sent with inputHint set to acceptingInput, the microphone is turned on automatically on the webchat and listening to user on latest Chrome browser.
Currently only way to keep the microphone off is to set the inputHint to ignoring input.
It is expected that acceptingInput hint should not turn on the microphone.
Expected behavior
For a single message activity,  acceptingInput inputHint should not turn on the microphone.
for multiple consecutive activities the behavior is defined here https://github.com/microsoft/BotFramework-WebChat/issues/2158

@compulim I believe you are currently investigating this problem, is that correct?

@ManjulaMalaiarasan thanks for reporting this issue. However I notice that you are using an older version of Web Chat. Could you test this out using Web Chat's latest, and see if the issue is still being reproduced? Please let us know your findings here. Thank you!

@corinagum Just tested with version 4.5.4. The inputHints work correctly for single and multiple activities and the issue is not reproducible. Thank you. I guess the recommendation would be to update my webchat then.


Added my name to guestbook.md

Simply added my name to the guestbook.md as instructed

Thank you!


add new external posts

Okay but the summary is based on how the article is structured. Did you read that particular article?
On Jun 1, 2023 at 10:42 AM +0100, Bill Mulligan @.***>, wrote:

@xmulligan commented on this pull request.
In src/posts/2023-05-26-cilium-mesh/index.md:

@@ -2,7 +2,7 @@
date: '2023-05-26T15:56:00.000Z'
title: 'Cilium Mesh: A new way to extend Kubernetes benefits across on-premises and cloud networking'
ogImage: ogimage.jpeg
-ogSummary: 'Learn about how Cilium mesh has helped extend the benefits of Kubernetes'
+ogSummary: 'Learn about how Cilium Mesh provides a fresh way to maximize the advantages of Kubernetes, offering new possibilities for extending its benefits'
That's not exactly what Cilium Mesh is, can you reread the the launch blog post and this one to make sure you understand what it is doing?
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.Message ID: @.***>

Yes, I've read the article. It talks about "Kubernetes-style principles powering the network layer has a range of benefits". The benefits aren't from Kubernetes itself, but rather from Cilium which is built in the Kubernetes/cloud native style. It's a subtle difference, but an important one. Cilium providing the networking in a Kubernetes-style way, extends the benefits of cloud native networking outside the Kubernetes world.
The way you currently have it written makes it seem like Cilium Mesh is just extending Kubernetes beyond Kubernetes.


Added Trufflehog

The addition to the gitlab script is only a stand-in until testing architecture is complete.

I probably meant the data-browser codebase. The data-portal didn't contain any secrets.

@GPelayo, please Slack (!) me the commit and location of that secret.


[DEBUG] Only print out diagnostic messages if an environment variable is set

This prevents crashes in test_core.py due to too many diagnostics emitted in https://github.com/llvm/llvm-project/pull/78228 It should also speed up compile times, as we can use multithreading, and avoid handling diagnostic messages.

After some debugging on our side, I am not sure that this patch does what was intended.

If you don't call context->getDiagEngine().registerHandler([](Diagnostic &diag) {, MLIR seems to register one by default that prints to stdout or stderr, which is identical to what we do when we do register a handler, so this has no effect either way.
context->printOpOnDiagnostic(true); is the default, so this has no effect either way.
context->printStackTraceOnDiagnostic(true); is not the default, but setting it doesn't seem to have an effect as far as we can tell.
We observe that the diagnostics printed out when Triton fails seem to be identical with and without this patch (with no envvar set).

Therefore the only effect this patch seems to have is enabling multithreading unless you set the new envvar.
We are happy that the diagnostics are the same with and without this patch, at least in the way we use Triton.  If you want to make a patch to actually print fewer diags, that should probably be disabled by default, and you can enable it on your side.

Thanks for looking into it. That's quite interesting, I just double checked if I missed the printouts in our output after the patch, and I can clearly see them before it, and they disappeared afterwards.
I see two possibilities why that happened: either it's because we are using a newer version of LLVM, and the default MLIR behavior changed (you could try patching in #3149 to try it out), or it has something to do with how the pytest runner works (it's a bit of a mystery to me TBH) - e.g., maybe enabling multithreading again somehow prevents the crash, and pytest only prints out stdout / stderr if the test fails.

According to the latest comments in #3150 it seems that this is doing something even upstream with a newer version of LLVM.


Conversion of intf to int, float and double

Hi AlyShmahell,
I wonder if it is possible to add an interface for converting intf to integer or float number?
Best regards,
Jing

I would like to add some more details for the issue.

Do the conversion

    intf a = 400;
    double v = a;
    double v1 = int(a);

Compile it (with C++11)

Errors are encountered, i.e.,
main.cpp:9:16: error: cannot convert 'intf' to 'double' in initialization
     double v = a;
                ^
main.cpp:10:22: error: invalid cast from type 'intf' to type 'int'
     double v1 = int(a);
                      ^

A potential solution is to add cast-operator, i.e.,
class intf {
     // other member functions
    operator int() const;
    operator double() const;
};

inline intf::operator int() const {
  int v = 0;
  int scale = 1;
  for (auto it = digits.rbegin(); it != digits.rend(); ++it) {
    v += scale * *it;
    scale *= 10;
  }
  return v;
}

inline intf::operator double() const {
  double v = 0;
  double scale = 1.0;
  for (auto it = digits.rbegin(); it != digits.rend(); ++it) {
    v += scale * *it;
    scale *= 10.0;
  }
  return v;
}

@zhongjingjogy
Hello, and thank you for pointing this out.
cast operators have been added, please check example_3.cpp, it also covers exception handling in case intf is of bigger size than int, (last two lines of code).

That is great! I am considering of using intf as the hash value for my application. Thank you so much for your work.
P.S. The indentation of the code is not consistent. I would like to recommend clang-format to you, which is a very nice tool for formatting C++ codes.

@zhongjingjogy
So far I've been using the default A-Style in codeblocks, but I will give clang-format a try during the next update, so thank you.
I'm also going to add primality testing to the library, specifically "Atkin–Morain", if you are interested in that I would appreciate the help 😃


feat!: Move Helm Jmeter Service into keptn-contrib

With this PR we are completing the move of helm-service and jmeter-service into separate their own dedicated repositories in keptn-contrib.
I have removed all code from helm-service and jmeter-service directories. In addition, I have updated both README files with a link to the new repositories.
FYI @Raffy23 @thisthat @oleg-nenashev
Fixes #8502
Please wait until the following PRs are merged

https://github.com/keptn-contrib/helm-service/pull/2
https://github.com/keptn-contrib/jmeter-service/pull/1

pls resolve conflicts, otherwise LGTM


IsNullable == false for nullable reference types

I've just started using the new "nullable reference types" feature of C# 8. It turns out that for a string? the property.Type.IsNullable is false.

@frhagn As the owner of this repo, you have to at least respond with a sign of life. If this project is no longer maintained, then please write that in the readme.


Example

Is there any example available?

Hi, there is a real-world example in the readme. Is that what you're looking for? It's a good starter for every project.

Ah, I was thinking something like this https://github.com/zeit/next.js/tree/master/examples/.
Where they have examples that we can run. We can see how it fits together.
Honestly, I think this plugin has a lot of API. Which is probably good as it gives a lot of flexibility.
On the other hand. It's a bit overwhelming to "Get started" with this plugin.
That's why I think having a simple example project. Which the user can play around with, will help us to get started.
Cool project btw :). Would love to see further development :)

Oh, I see. Since it's using a a CSS framework, I did not setup anything to test with, but I can understand the need.
Actually, I was waiting for this issue to be resolved in order to add an example to play with.
But you're right, I may add a simple Vue application or something to test if tailwind.run don't let me do this soon enough.
Thanks for the kind words :)

The documentation is clearer now. I will soon add some examples with vue-cli so you can clone and build them to try out the plugin.

I added an example. See the readme.


Simplify prugeExpired

It looks like purgeExpired's implementation is just a more complicated was of running HashMap.filter (not . isExpired now), which would likely be much more efficient than repeatedly reading and writing the TVar for each key.

You are correct! I currently don't have time to make any changes, but I'd welcome a PR. otherwise, it might take a while.

I'll see what I can do tonight.


Postgres: deleting last element in text field with hasMany: true not working

Link to reproduction
https://github.com/Janisvb/payload-reproduction/tree/delete-hasmany
Describe the Bug
In a text field with hasMany: true, deleting the last element is not setting the value to [] or null

To Reproduce
const post = await payload.create({ collection: postsSlug, data: { text: ['test'] } })

await payload.update({ collection: postsSlug, id: post.id, data: { text: [] } }) // not working
await payload.update({ collection: postsSlug, id: post.id, data: { text: null } }) // not working

Works as expected with MongoDB.
Payload Version
2.12.1
Adapters and Plugins
db-posgres

This has been fixed in the latest of v3 beta


Does the core safety claim need rephrasing?

From the README:

The core safety claim under this new model is that auditing just the C++ side would be sufficient to catch all problems, i.e. the Rust side can be 100% safe.

I really haven't looked at this in depth so I might be missing something, but it seems to me that given a binding system that exposes unsafe functions to Rust code, the code that uses these unsafe functions has to be audited for correctness.
Some ground assumptions I'm making:

By “problems” you mean “potential sources of undefined behavior.”
“The Rust side can be 100% safe” means that the Rust code can be said to have no undefined behavior by construction, i.e. without any auditing.

OK, sure, but only a subset of C++ functions can be exposed to Rust as safe.  Are you saying CXX doesn't allow other C++ functions (e.g. vector<int>::pop_back()) to be exposed to Rust?  How could it possibly prevent that?

Whether a function is safe to call from Rust is determined by whether the fn is unsafe, not the surrounding block. This function would be unsafe to call from Rust:
extern "C++" {
    unsafe fn pop_back(self: Pin<&mut CxxVector<c_int>>);
}

See https://cxx.rs/extern-c++.html#functions-and-member-functions.

Ok thanks. But what about the other questions?

Are you saying CXX doesn't allow other C++ functions (e.g. vector<int>::pop_back()) to be exposed to Rust? How could it possibly prevent that?

CXX does allow other C++ functions to be exposed to Rust, by writing unsafe fn in the list of functions in the FFI module as in https://github.com/dtolnay/cxx/issues/1364#issuecomment-2243755729. unsafe fn C++ functions can only be called from unsafe Rust code. Safe C++ functions can be called from safe Rust code. CXX does not prevent FFIs from being expressed in terms of unsafe functions. It does prevent the need to do so, by making it possible for real-world elaborate FFIs to be expressed using safe functions.
In general, if you have some arbitrary C++ function to call whose behavior is not safe, you're not going to call it without writing either unsafe Rust code or (unsafe) C++ code. In your example of pop_front the unsafety can go either on the C++ side:
// C++
template <typename T>
bool checked_pop_back(std::vector<T>& vec) {
  if (vec.empty()) {
    return false;
  } else {
    vec.pop_back();
    return true;
  }
}

// Rust
unsafe extern "C++" {
    #[cxx_name = "checked_pop_back"]
    fn checked_pop_back_int(vec: Pin<&mut CxxVector<c_int>>) -> bool;
}

assert!(ffi::checked_pop_back_int(vec));

or the Rust side:
// Rust
extern "C++" {
    unsafe fn pop_back(vec: Pin<&mut CxxVector<c_int>>);
}

fn checked_pop_back(vec: Pin<&mut CxxVector<c_int>>) {
    assert(!vec.is_empty());
    unsafe { ffi::pop_back(vec) }
}

(From experience with >300 CXX-based libraries at Meta, the bindings tend to consist overwhelmingly of safe functions, like this. This makes them accessible to engineers with limited Rust experience. We've had engineers in their first and second week with Rust successfully write bindings that would have been challenging for me to produce without CXX as a Rust expert and C++ "knowledgeable".)

I assumed unsafe extern "C++" {...} meant that the Rust declarations within {...} were treated as unsafe by Rust. Was that wrong?

The assumption is wrong -- unsafe extern "C++" { fn ... } is a function that is safe to call. This is an unsafe thing to claim about a function, hence the unsafe keyword. Inversely extern "C++" { unsafe fn ... } is a function that is unsafe to call, and isn't unsafe to claim that.

It does prevent the need to do so, by making it possible for real-world elaborate FFIs to be expressed using safe functions.

s/real-world/some real-world/ or even s/real-world/many real-world/ if you want to make that claim.

(From experience with >300 CXX-based libraries at Meta, the bindings tend to consist overwhelmingly of safe functions, like this.

Yes, any API using pure value semantics in C++ (to the degree C++ can express it) will always be like that.  It's great that so much of Meta's C++ code is written that way.

This makes them accessible to engineers with limited Rust experience. We've had engineers in their first and second week with Rust successfully write bindings that would have been challenging for me to produce without CXX as a Rust expert and C++ "knowledgeable".)

Not AT ALL questioning the usefulness of this work.  I think it's absolutely fantastic.  However, you do yourself a disservice by not being precise about the safety claims.  It would be easy, for example, for one of those beginning engineers to read your README and reach the wrong conclusions about which things are safe.  Also, someone more experienced like me but with less patience will look at your claim, realize it's impossible as stated, and dismiss CXX prematurely.
Of course some of the Rust code (the code that declares certain things to be safe at the very least!) needs to be vetted.  It would be better to make the claims more accurate, and it would be helpful to characterize the properties of a C++ API that can be declared safe in Rust.  FWIW.


Update iris examples

Hello Victor(@gavv), how things are going?
These are the Iris examples updates from v6 to v11.

Coverage remained the same at 93.955% when pulling d3d9c85ce254fd85f4612826c992e3f3f5cc83d8 on kataras:update-iris into 5699f93360c1692d13d5cc99dd10c8ab3ad2bb7c on gavv:master.

Thanks! It's okay.

Follow-up commits (add build tag to iris_test too; use go mod):

0a11858c2745494e533d4c2d3d5f82afd97b1caf
d7ef0eaac427e0b4682ea37b2e7ad5ad60f20b7a


Add comment to subchannel key creation

https://github.com/grpc/grpc/blob/037d9ab177e4e5ae44b765886f3f2dcf44050413/src/core/ext/filters/client_channel/lb_policy/subchannel_list.h#L492
The arg here doesn't have any filter info.

****************************************************************

libgrpc.so

     VM SIZE        FILE SIZE
 ++++++++++++++  ++++++++++++++

  [ = ]       0        0  [ = ]

****************************************************************

libgrpc++.so

     VM SIZE        FILE SIZE
 ++++++++++++++  ++++++++++++++

  [ = ]       0        0  [ = ]

[trickle] No significant performance differences

Objective-C binary sizes
*****************STATIC******************
  New size                      Old size
 1,980,683      Total (=)      1,980,683

 No significant differences in binary sizes

***************FRAMEWORKS****************
  New size                      Old size
10,811,764      Total (>)     10,811,758

 No significant differences in binary sizes

[microbenchmarks] No significant performance differences


Firing email when schedule task run

Hi Guys
I have a requirement to fire eamil (notification) to the user's email (saved with task) when any schedule task run.
Now I am thinking of saving user's email in the Comment (Description) of the task and as I am calling a .net console application whenever the task run I can grab that user's email from Descirption and sends the email? I want to know are their any other good ways of saving the email with the task so we can fire it from another application which will be running when the schedule task runs
Thanks
Nouman
Originally posted: 2010-09-30T08:35:58

My only other suggestion is to create a customer task handler that is triggered that has the intelligence to pull the email from any location, including the Description or a more secure location. You can find an example project off the home page for this library.
Originally posted: 2010-10-18T15:04:53


Update deps and fix compiling errors.

Thanks @yonas
Let me see if I can migrate the CI over to GH-actions and then we can rebase this onto that CI setup. Bare with me 😄

Actually we'll do this the other way around. Merge this, and then migrate to GH-actions.


8.1.1241 introduces new requirement for count of custom Ex command which breaks old working scripts

Describe the bug
Since 8.1.1241, when a custom command is defined with the -range attribute, a count is tested against a line number in the current buffer (unless -addr= specifies another kind of range).  This breaks old working scripts which provide custom Ex commands with a range of no specific kind.  Now, such commands need to be prefixed with a count matching an existing line in the buffer; or they need -addr=other.  This requirement didn't exist before.
To Reproduce
Run this shell command:
vim -Nu NONE -S <(cat <<'EOF'
    com -range=1 Cmd echo "the command was used with the count: " . <count>
    3Cmd
EOF
)

E16 is raised:
E16: Invalid range:     3Cmd

That's because 3 is tested against a line number.  But there's no line 3 in the current empty buffer.
Expected behavior
No error is raised, because the command was working before 8.1.1241.
Environment

Vim version: 8.2 Included patches: 1-2572
OS: Ubuntu 16.04.7 LTS
Terminal: xterm(366)

Additional context
I suppose it's working as intended.  I think it's documented at :h :command-addr:

Possible values are (second column is the short name used in listing):
-addr=lines                 Range of lines (this is the default for -range)
-addr=arguments       arg   Range for arguments
-addr=buffers         buf   Range for buffers (also not loaded buffers)
-addr=loaded_buffers  load  Range for loaded buffers
-addr=windows         win   Range for windows
-addr=tabs            tab   Range for tab pages
-addr=quickfix        qf    Range for quickfix entries
-addr=other           ?     other kind of range; can use ".", "$" and "%"
as with "lines" (this is the default for
-count)

The solution now is to use the -addr=other attribute:
com -range=1 -addr=other Cmd echo "the command was used with the count: " . <count>
3Cmd

But this has broken old working scripts.  For example vim-scriptease.  According to @tpope, it would still be useful it the old behavior was restored:

it's not too late in terms of the old behavior still being worth restoring. Sometimes I want to be able to do :12345Foo, but also :$Foo! for example, where the interpretation varies based on the presence of ! or some other argument. Also restoring backwards compatibility would be helpful to users of unmaintained plugins.

I must add that the commit which introduced the regression did not change any help file.  So, I'm not sure whether the change was entirely intentional.  I mean, that a range is interpreted as a line number by default is one thing; and it has probably always been the case.  That this line number has to exist is another.

That patch is almost two years old.  I haven't seen complaints about making the range checking stricter.  In general, a range can be expected to be a line range, unless specified otherwise.
Changing it now might also introduce new problems, for commands that omitted "-addr=lines" because it's already the default.
Thus I rather not change this now.


Remove Versioner and PkgCopier

The Versioner and PkgCopier processes are both unnecessary as their function is replicated in MunkiImporter. Also, the presence of PkgCopier results in an addition to the recipe's summary output on each run.
(N.B. I've adjusted some of the indentation, hence the extent of the changes.)

I'll close this now as the PkgCopier bit has been fixed by #11. Removing the Versioner processor isn't critical, although I'm guessing that it's legacy given that the NoMAD pkg is all nice and neat and correctly reports its version to MunkiImporter?
Also, in response to @soberhofer's Slack comment (which I just spotted from last week!), I won't Atom Beautify the code next time I do a PR ;-)


Add terms of use for brand assets

I don't know how to create this, so I'm looking forward to someone to do our terms of use. Thanks

done!


Error connexion with vacuum with python plugin domoticz

Hello can you help me please?
I follow tutorial Xiaomi Mi Robot Vacuum - Domoticz Python plugin
step by step but i have errors when i put command :
sudo ./miio_server.py <IP_ADDRESS> xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx --host <IP_ADDRESS> --port 22222
i have :
pi@raspberrypi:~/domoticz/plugins/xiaomi-mirobot $ sudo ./miio_server.py <IP_ADDRESS> XXXXXXXXXXXXXXXXXXXXXXXX --host <IP_ADDRESS> --port 22222
Traceback (most recent call last):
File "./miio_server.py", line 17, in
from miio import Vacuum, DeviceException
File "././.env/lib/python3.7/site-packages/miio/init.py", line 2, in
from miio.protocol import Message, Utils
File "././.env/lib/python3.7/site-packages/miio/protocol.py", line 155, in
Const(Int16ub, 0x2131),
File "././.env/lib/python3.7/site-packages/construct/core.py", line 2504, in init
super(Const, self).init(subcon)
File "././.env/lib/python3.7/site-packages/construct/core.py", line 670, in init
raise TypeError("subcon should be a Construct field")
TypeError: subcon should be a Construct field
Cand you help me please?
Thanks

Sorry, we don't support the Python plugin here. Do you mind opening the issue in their repo? https://github.com/rytilahti/python-miio

Sorry, we don't support the Python plugin here. Do you mind opening the issue in their repo? https://github.com/rytilahti/python-miio


Rebase rootkey feature branch on latest develop branch

Manual E2E smoke tests for apt pkg update is verified working.
Another iteration will come to fix windows pipeline and e2e test pipeline passing.

closing for now until I can address the windows linker issue and a couple of yocto build compiler issues.


Fix for Bisa/factorio-init#194: Use factorio api for latest-releases

theres 2 methods that do not require the user to install new software nor do they require a large change to the check core.
Method 1: grep + awk
curl -LIs https://www.factorio.com/get-download/stable/headless/linux64 | grep Location | awk -F "?" '{print $1}' | awk -F "/" '{print $NF}' | awk -F "_" '{print $4}' | awk -F ".tar" '{print $1}'
Returns something like this as of today : 1.1.57
Method 2: grep + awk on new API site above
to get experimental
curl -s https://factorio.com/api/latest-releases | awk -F ":" '{print $5 $9}' | awk -F """ '{print $2}'
outputs 1.1.57
stable
curl -s https://factorio.com/api/latest-releases | awk -F ":" '{print $5 $9}' | awk -F """ '{print $6}'
outputs 1.1.57
both methods are simpler and do not force the user to install/rely on more external software

use grep + awk or just awk even and do not require the user to install more software just to handle json
there's 2 methods that do not require the user to install new software nor do they require a large change to the check core.
Method 1: grep + awk
curl -LIs https://www.factorio.com/get-download/stable/headless/linux64 | grep Location | awk -F "?" '{print $1}' | awk -F "/" '{print $NF}' | awk -F "" '{print $4}' | awk -F ".tar" '{print $1}'_
Returns something like this as of today : 1.1.57
to get the file name you can do this curl -LIs https://www.factorio.com/get-download/stable/headless/linux64 | grep Location | awk -F "?" '{print $1}' | awk -F "/" '{print $NF}'
which returns this factorio_headless_x64_1.1.57.tar.xz
Method 2: awk on new API site
to get experimental curl -s https://factorio.com/api/latest-releases | awk -F ":" '{print $5 $9}' | awk -F """ '{print $2}'
outputs 1.1.57
to get stable curl -s https://factorio.com/api/latest-releases | awk -F ":" '{print $5 $9}' | awk -F """ '{print $6}'
outputs 1.1.57
both methods are simpler and do not force the user to install/rely on more external software
for Method 2 this is the sample raw output that awk works on {"experimental":{"alpha":"1.1.57","demo":"1.1.57","headless":"1.1.57"},"stable":{"alpha":"1.1.57","demo":"1.1.57","headless":"1.1.57"}} what its doing is breaking down the response first by : parts and then by a " and then returning the numbered column
the benefit is that any system capable of running a factorio server should also have awk/grep installed by default as they are core POSIX functions

The awk method is not robust to any changes made to the latest releases API.
The advantage of jq (which is by the way super light and you should consider using this for parsing json) is that you rely on key from the json instead of arbitrary number of ":".
Any server capable of running factorio is also capable of having jq installed, since it has no runtime dependencies.

the advantage of awk is still 1 less software to install. There is no need to keep installing more and more packages to do something the os is already capable of doing by default. Less complexity is always better than more.

I don't understand that "fear" to install such a small lib on a system knowing that we install Factorio and everything that goes with it. Using this makes the code more readable and more resistant to changes that the dev. may do in the API.

its not a "fear" its a principle, why install more and more software to do tasks that the currently installed software can already handle just fine.

theres also the issue of slowly creeping into Dependency hell for nothing

"Creeing into dependency hell for nothing"
Ok, then don't install factorio-init (which by the way don't work atm, so nice dependancy) to "just" install a server because it's still one more dependances than just downloading and launching the game.

Just before definetly closing this, i'm curious to know your opinion on this @Bisa.
Thank's in advance !

Thnx for the suggested fix @Legrems - I did however opt to use already existing tools, at some point in the future I may look at using the api, but much like @tmzasz I try to stick to as few "extras" as I can (when I remember to and when I do not whirr off into some random "this is so cool" spree ;) )


Dashr-drug-discovery-cr

Issue for app: #[issue number here]
App pull request

[ ] This is a new app
[x] I am improving an existing app (redesigns/code "makeovers")

About

Playground deployment URL (new version):
Current gallery app URL: (delete this line if inapplicable)
Python app repository link: (delete this line if you are working on a Python app)

Workflow

[x] I have created a branch in the appropriate monorepo, and the
elements necessary for successful deployment are in place.
[ ] If the app is a redesigned and/or restyled version of an
existing gallery app, I've summarized the changes requested in the
appropriate Streambed issue and confirm that they have been applied.
[x] If the app is on the Dash Gallery portal, I have added a link to
the GitHub repository for the source code in the portal description.
[x] If the app is a reimplementation of a Python gallery app for the
DashR gallery, the app in this PR mimics, as closely as possible,
the style and functionality of the existing app.=
[x] I have removed all Google Analytics code from the app's
assets/ folder.

The pre-review review
I have addressed all of the following questions:

[x] Does everything in my code serve some purpose? (I have removed
any dead and/or irrelevant code.)
[x] Does everything in my code have a clear purpose? (My code is
readable and, where it isn't, it has been commented appropriately.)]
[x] Am I reinventing the wheel? (I have used appropriate packages to
lessen the volume of code that needs to be maintained.)

@mleech1,  hi there. I created this branch as part of the re-review app assignment as instructed by @rpkyle.
I listed the changes I made below in detail, hoping that would make it easier to follow and  review. Most of the edits are minor stuff related to formatting and following the naming conventions. However, there are couple of them more important. I made those ones bold. Please let me know if there are any wrong/inappropriate edits. Thank you :)
README.md
1 - Updated original repo link to
https://github.com/plotly/dash-sample-apps/tree/master/apps/dash-drug-discovery
2 - "Install packages (using install.packages('PACKAGE NAME')) that are listed in app.R"
Changed above to init.R installation instructions
3 - Documentation link updated to https://dashr.plot.ly
4 - Moved screenshot to screenshots folder
app.R
1 - library(dashR) -> library(dash)
2 - Removed  header = TRUE, sep = “,”  from read.csv
3 - Reformatted colorscale
4 - Renamed title to appTitle (title is built-in function name)
5 - subtitle1_bold -> subtitle1Bold (google style guide naming convention)
6 - subtitle2_bold -> subtitle2Bold (google style guide naming convention)
7 - three_d -> threeD  (google style guide naming convention)
8 - Reformatted threeD <- plot_ly( ….
9 -  Reformatted histo
10 - scatter_graph -> scatterGraph (google style guide naming convention)
11 - Reformatted scatterGraph
12 - make_dash_table -> MakeDashTable (google style guide naming convention)
13 - ### GRAPH PLOTLY OBJECTS ### -> ### Graph Plotly Objects ###
14 - Paraphrased comment under MakeDashTable
15 - df_subset -> dfSubset (google style guide naming convention)
16 - MakeDashTable -> removed the outer for loop | removed empty initialization table <- list() | Added comment above apply | reformat
17 - Removed line -> drug_names <- unique(df$NAME)
18 - drug_options -> drugOptions
19 - Reformatted drugOptions
20 - drug_name -> drugName
21 - Removed line figure <- threeD #initial figure. Assigned threeD directly to dccGraph figure property
21 - starting_drug -> startingDrug
22 - drug_description -> drugDescription
23 - drug_img -> drugImg
24 - dccGraph( points= list(), id = ‘clickable-graph’) 🔪points= list()
25 - Reformatted htmlA -> chem_name
26 - Reformatted ### CALLBACKS ### -> ### Callbacks ###
27 - Reformatted Change graph based on radio button callback, plot_type -> plotType
28 - Reformatted Description callback
29 - Reformatted Change image on hover callback
30 - Chemical link callback -> description & reformat, variable naming
31 - Chemical name callback -> description & reformat
32 - Container div for graph and drug summaries separated eight columns to four columns similar to python version
33 - Changed app$run_server to
if (!appName == ""){
  app$run_server(host = "<IP_ADDRESS>", port = Sys.getenv('PORT', 8050))
} else {
  app$run_server(debug = TRUE)
}

34 - Hover callbacks were throwing error on debug mode due to not handling null pointing case. Edited the control structure logic
35 - chem_dropdown_value -> chemDropdownValue (Append to table callback)
36 - Centered the chemical image by wrapping it inside a div and setting style = list("text-align" = "center”)
37 - Linted the app, more formatting
Issue: plot_bgcolor not working for threeD?
Future Work: 4 hover callbacks can be merged later when multi output supported
Other file changes
Removed python version files updated the deployment files

As per https://github.com/plotly/streambed/issues/13854, I am closing this as the new deployment process is in place.
If you need any changes from this branch, please recreate your changes against master and create a new pull request. Let me know if you need help and I am happy to jump in.


CommitWithMetadataSource

As a part of work on issue #36 , this PR implements CommitWithMetadataSource.
Pretty simple overload to used CommittingSourceStage with passing function to extract string metadata from consumed message right after this message was consumed.

Will merge once conflicts are fixed

lol I'm sorry @IgorFedchenko - just merged your SourceWithOffsetContext PR and it looks like we have conflicts again

@Aaronontheweb Yup, I was expecting this - you have no choice other that merge one PR and get conflicts in another :) Have resolved it right now

@Aaronontheweb There is this AtMostOnceSource test that is failing from time to time. I thought that this was due to small timeout I set, but now I am debugging it and there is something wrong there. So I will fix it and push commit to make tests pass stable.

I spent some time yesterday on failing AtMostOnceSource test, and still did not find the reason (or, to be more accurate, I did not find a way to fix this) - so created issue #66 for now.
This test is not related to this PR anyway, and was randomly failing during several other PRs. I thought that was a timeout issue, but it is something more complicated. Will return to this later.
So this PR can be merged now.

Sounds good


[Backport][v1.1.4] Instance managers and Pods with attached volumes restarted every hou

Ref https://github.com/longhorn/longhorn/issues/2820

Validation: PASSED
Longhorn Version: v1.1.x
Steps to test:
https://github.com/longhorn/longhorn/issues/2820#issuecomment-1054558178


instanceLimit not respected for compound tasks

Type: Bug
Assume you have 2 tasks and a compound task the runs them in sequence. Setting an instance limit of 1 on the compound task should ensure you never have both of the dependency tasks running at the same time.
{
    "version": "2.0.0",
    "problemMatcher": [],
    "tasks": [
        {
            "label": "clean",
            "type": "shell",
            "command": "Start-Sleep 2",
        },
        {
            "label": "build",
            "type": "shell",
            "command": "Start-Sleep 2.5"
        },
        {
            "label": "rebuild",
            "group": {
                "kind": "build",
                "isDefault": true
            },
            "dependsOn": [
                "clean",
                "build"
            ],
            "dependsOrder": "sequence",
            "runOptions": {
                "instanceLimit": 1,
            },
        }
    ]
}

Give the tasks file above, the following sequence is possible:

User runs rebuild
clean finishes
build begins
User runs rebuild
clean begins
...

Notice clean is able to be started while build is in the middle of executing. 2 instances of rebuild are running simultaneously.
 There's a subtle difference in which task the instance limit is being violated for.
In this issue:
Compound Task     <-- Instance limit is violated here
Dependency Task 1
Dependency Task 2

In the linked issue
Compound Task
Dependency Task   <-- Instance limit is violated here

It seems entirely possible these have the same root cause, so it may in fact be a duplicate, just pointing out the difference in case they are not.


A stubbed _ready does not seem to be called when adding the mocked node to the tree

The following code does not call the stubbed version of _ready, but calls the real one:

When calling _ready manually on the double it does trigger correctly.

_ready, _enter_tree, _exit_tree, _process & _process_delta are special virtual methods. They can't (and probably shouldn't) be dummied/stubbed. The calls to them are done recursively by the engine so I don't have any control over them. I'll add a warning though.

I see. I ran into this when creating a test that uses a secondary object that needed to be added into the tree.
This then automatically triggered the secondary object's ready function with side effects that I'm not interested in for the respective test.
In this case, stubbing would be great, but I understand it's not possible/simple to set up.


Update sizing for graph endpoint

Type of change

[ ] Refactor
[ ] New feature
[ ] Bug fix
[x] Optimization
[ ] Documentation Update

Description
Update the sizing logic

Related Tickets & Documents

Related Issue #
Closes #

Checklist before requesting a review

[ ] I have performed a self-review of my code.
[ ] If it is a core feature, I have added thorough tests.

Testing

Please describe the System Under Test.
Please provide detailed steps to perform tests related to this code change.
How were the fix/results from this change verified? Please provide relevant screenshots or results.

Need to refactor a bit...

I will get to this... Promise :)


use_build_context_synchronously is triggered when using switch statement

Describe the issue
When using BuildContext inside a switch statement, if the first case awaits an async method, the second case triggers the use_build_context_synchronously lint rule. However, this is a false positive.
To Reproduce
import 'package:flutter/material.dart';

Future<void> test(BuildContext context) async {
  if (context.mounted) {
    switch (1) {
      case 1:
        await Future<void>.delayed(Duration.zero);
        break;
      case 2:
        Theme.of(context); // Don't use 'BuildContext's across async gaps.
        break;
    }
  }
}

Expected behavior
use_build_context_synchronously lint works fine.

More than a year leater, still there.


Add Update changelog command

Would be preferable to see some tests that validate the behaviour of the non-command functions in a follow-up at least


Possible to remove matrix multiplication in SANet?

Is it possible somehow to get rid of the two matrix multiplications in SANet.cs?
This is one reason it is really slow.

Hello, ebroglio!
Matrix multiplication is part of any attention module, whether it is self-attention or style-attention. Regarding the speed of work, the problem is most likely directly in the implementation of matrix multiplication. I will check this code and make edits if necessary.
Sincerely,
Gleb Brykin.


[infra] Eliminate MSVS/BuildTools/WinSDK dependencies on windows

Since gclient fetches cipd/windows_sdk, local installations of MSVS/WinSDK do not seem to be needed. Furthermore, relying on both local and cipd-provided tools can lead to incompatibilities and technical debt.
// visual studio, build tools, windows sdk not installed.
// engine directory is empty.
// depot tools and python 3 available.
// powershell:
PS D:\dev\engine> $env:DEPOT_TOOLS_WIN_TOOLCHAIN=0
PS D:\dev\engine> $env:GYP_MSVS_OVERRIDE_PATH="$pwd\src\third_party\windows_sdk"
PS D:\dev\engine> $env:GYP_MSVS_VERSION="2019"
PS D:\dev\engine> $env:WINDOWSSDKDIR="$pwd\src\third_party\windows_sdk\Windows Kits\10"
PS D:\dev\engine> gclient config https://github.com/flutter/engine.git --name=src/flutter --unmanaged
PS D:\dev\engine> gclient sync --shallow --no-history
// apply st.diff.txt to src
PS D:\dev\engine> python .\src\flutter\tools\gn
PS D:\dev\engine> ninja -C .\src\out\host_debug\
🎉

Builds succeed on Windows 10/Windows 11 hosts for win32/UWP targets.
If this approach is accepted in principle, definitive changeset would likely be different and involve cleanups.
st.diff.txt

\cc @cbracken

I'm hitting _SILENCE_CLANG_COROUTINE_MESSAGE issue as well

@safasofuoglu do you happen to have the patch for how to fix the UWP build based on your learnings?

@clarkezone https://github.com/flutter/flutter/issues/91184

This approach has become irrelevant due to the removal of winsdk from DEPS.


Change auto contiguous to always insert contiguous

Why close this out? Guessing rebase and push forward or not needed anymore?


documentation about the networks, especially for cross-cluster

How to categorize this issue?
/area documentation
/kind enhancement
What would you like to be added:
How can I set up CNI parameters, e.g., use calico VXLAN or BGP.
How can I make a shoot cluster routable to another? static routing rules and firewalls specific the infrastructure?
Why is this needed:
We have env-per-cluster pattern, and some services need to be clustered across two clusters, some monitoring tools also need to access pods in another cluster. But some of these are tightly related to CNI and infrastructure. I can do it on my own but gardener does not expose too much of these kind of configurations, or just not documented.

hopefully not auto-closed


More Python 3 fixes, against master

This is just #2251 against master, since the branch it was against has been merged in.
On the other PR, the only test it failed was the CWL conformance test. @DailyDreaming Is that expected to fail?
Also I believe the Jenkins tests are only testing Python 2, right? They still need to be upgraded to also do Python 3.
Closes #2251.

@adamnovak CWL conformance tests have been timing out since they added a 15 minute timeout on the cwltool side.  The tests pass locally and otherwise.  Issue is: #2222 .  It basically just needs to be extended.

Jenkins, test this please.

Also, I want to add tox to jenkins for this, but I leave tomorrow for Portland for a week.  This and the toil move to github.com/DataSphere I felt would be better to attempt when I got back.

I've fixed up a bunch of tests that were still failing in Python 3; I don't know why I didn't catch them before. Some stuff still fails on me when I run it locally under Python 3, and I think some of that is due to me trying to run things on Courtyard with the Docker daemon on a different machine (so bind mounts don't work at all).
I'm running another pass through all the tests now, to make sure that, of the ones I can run properly, none are broken.
I think we still need to enable Python 3 in the CI before we can declare that Python 3 support exists, but I think before we do that we might want to do #2258 so that the appliance can run under Python 3.

Also, #2258 is a good point and I agree definitely needs to be addressed after this.  Once it is, running something a bit beefer like cactus on python 3 would be a good test.

Cactus won't work on python 3 in the near future (though toil supporting it would give some reason to clean it up). We (ab)use a lot of py2-only features right now.

@joelarmstrong Alright, I have some workflows for TOPMed that I can try out.

This is blocked on one caching test with the Google job store failing on Jenkins, which I'm a bit befuddled as to how to debug:
http://jenkins.cgcloud.info/job/toil-pull-requests/4145/testReport/src.toil.test.src.fileStoreTest/CachingFileStoreTestWithGoogleJobStore/testCacheEvictionFailCase/
It looks like we're getting more CacheUnbalancedErrors than the test is expecting, and 3 jobs are failing instead of the expected 1. Is this something to do with chaining and how that affects how failing jobs are counted? Do we just want to remove that check on the number of failed jobs?

Looks like the Ubuntu keyserver was down.
Jenkins, test this please.

OK, now these tests failed, which didn't fail last time the tests ran:
src.toil.test.provisioners.aws.awsProvisionerTest.AWSRestartTest.testAutoScaledCluster
src.toil.test.provisioners.gceProvisionerTest.GCERestartTest.testAutoScaledCluster

It looks like they timed out? @DailyDreaming @joelarmstrong Is this expected behavior?

@adamnovak testCacheEvictionFailCase has been flaky for a while. It's not Google specific.
The others seem like something with Jenkins or one of the make checks. Will test again and check the Jenkins logs:
http://jenkins.cgcloud.info/job/toil-pull-requests/4149/
http://jenkins.cgcloud.info/job/toil-pull-request-integration-tests/2439/
make: *** [jenkins_test_integration] Error 2
Build step 'Execute shell' marked build as failure
$ ssh-agent -k
unset SSH_AUTH_SOCK;
unset SSH_AGENT_PID;
echo Agent pid 2212 killed;
[ssh-agent] Stopped.
Recording test results
Adding one-line test results to commit status...
Setting status of 30296dc0f8248c0d0cb14f9a0d97b6253f1e641a to FAILURE with url http://jenkins.cgcloud.info/job/toil-pull-request-integration-tests/2439/ and message: 'Build finished. 18 tests run, 0 skipped, 2 failed.'
Using context: jenkins-it
Finished: FAILURE

Jenkins, test this please.

I suspect it may be this check and your adding the cached files to the ignored list:
https://github.com/BD2KGenomics/toil/blob/master/Makefile#L278

This looks good to merge.
I think I have a fix for the timeout that makes sense.  I'll post a PR right after this.


NNStreamer module: rename a module of apptest

This commit is to rename the existing module name for nnstreamer apptest.

before:pr-{module-name}.sh

after :pr-{type}-{module-name}.sh

Guide:

First of all, Note that you have to run the below statement in order to check the grammar error of a CI module that your write.
$ bash ./pr-{type}-{module-name}.sh

When you submit a pull request to merge your CI module to TAOS-CI repository, Please note that reviewers check 4 requirements
as follows before merging your pull request.

Maintenance: The module has to be normally executed as a CI component after enabling the module via a configuration file.
Readability: Most of the developer should be able to read source code of the CI module without difficulty.
Execution time: The time required to execute the module should not be long.
Compatibility: The module should be able to run on most Linux distributions as well as Ubuntu.

Signed-off-by: Geunsik Lim<EMAIL_ADDRESS>
[Template] PR Description
In general, github system automatically copies your commit message for your convenience.
Please remove unused part of the template after writing your own PR description with this template.
$ git commit -s filename1 filename2 ... [enter]

Summarize changes in around 50 characters or less

More detailed explanatory text, if necessary. Wrap it to about 72
characters or so. In some contexts, the first line is treated as the
subject of the commit and the rest of the text as the body. The
blank line separating the summary from the body is critical;
various tools like `log`, `shortlog` and `rebase` can get confused
if you run the two together.

Further paragraphs come after blank lines.

**Changes proposed in this PR:**
- Bullet points are okay, too
- Typically a hyphen or asterisk is used for the bullet, preceded
  by a single space, with blank lines in between, but conventions vary here.

Resolves: #123
See also: #456, #789

**Self evaluation:**
1. Build test: [ ]Passed [ ]Failed [*]Skipped
2. Run test: [ ]Passed [ ]Failed [* ]Skipped

**How to evaluate:**
1. Describe how to evaluate in order to be reproduced by reviewer(s).

Add signed-off message automatically by running **$git commit -s ...** command.

$ git push origin <your_branch_name>

:memo: Version: 1.20180820. Thank you for submitting PR #297. Your PR must pass all verificiation processes of cibot before starting a review process from reviewers. If you are new member to join this project, please read manuals in documentation folder and wiki page. In order to monitor a progress status of your PR in more detail, visit http://taos-ci.mooo.com/.

guide: https://github.com/nnsuite/TAOS-CI/blob/master/ci/doc/how-to-use-taos-ci-module.md#requirement-before-submitting-a-pull-request