-e equivalent?

Is there a way to get -e-like semantics with turtle? If not, what about something like
shell' :: MonadIO m => Text -> Shell Text -> m ()
shell' cmd input =
    result <- shell cmd input
    case result of
        ExitFailure n -> die (cmd <> ": " <> T.pack n)
        ExitSuccess -> pure ()

This is very similar to #101 but I realized just now that you can do this using the (.||.) operator like this, very similar to Perl/Bash:
shell cmd input .||. die "Error message here"

Would that solve your use case?

It's a bit verbose - in Bash you can just throw a set -e at the top and not have to modify any code to get the added safety. It would be nice to have a similar level of effort put into turtle scripts.
However, nothing is really coming to mind... save for having some Turtle free monad for interpreting ExitCode-returning actions in different ways. Seems a bit heavyweight for just set -e.

You could do this, but then you would no longer be able to run proc/shell commands as ordinary IO commands

Right, every main would have to be wrapped in runTurtle. That sounds worse than your solution above :)

So there is another alternative, which is to do what Python does and throw an exception if the process returns a nonzero exit code and the user has to catch and handle the exception to provide a better error message.

Would that make things like
if [ cmd ]; then
    whatever
end

harder to write? Bash seems to have an advantage here as well - set -e won't affect these actions, only "top level" ones. I think.

You can still catch and handle exceptions, so the if in that example would be catch in Haskell

On the other hand if we stick to explicit error codes + (.||.) + die, then the closest analog of set -e in Haskell would be the -fwarn-unused-do-bind flag + optionally -Werror to warn if you discard an exit code.

Re: handling exceptions, I know - but my point is in Bash you get much nicer syntax.
if [ cmd ]

becomes
success <- catch (cmd >> pure True) (\(_ :: SomeException) -> pure False)

I suppose it's not a totally fair comparison, since [ ] are like little functions :)

Oh, and then all you might need is test :: MonadIO m => m a -> m Bool

I decided to open up a Twitter poll to get a feel for whether users prefer to signal failure via exit code or exceptions: https://twitter.com/GabrielG439/status/662470232083005440

Innocent unless proven guilty vs. Guilty until proven innocent.
Is MaybeT and such out of the question?

I'd probably prefer throwing exceptions over MaybeT/ExceptT since we're already using IO.  Checked exceptions seem like overkill for shell scripting.
I'm generally trying to hew as close as possible to prior art with turtle, but the problem is that prior art is divided on this point: Python uses exceptions and Bash uses return codes.

Using ExceptT with IO is redundant. There are two useful properties of ExceptT:

exceptions are in the type
you can handle them in pure code

You lose both as soon as you start using IO: the code is no longer pure (by definition) and not all exceptions are in the type (and in this case, it's all or nothing: as soon as you get exceptions not reflected in the type, you have no guarantees about exception safety of your code). Having two mechanisms for exceptions is of no benefit.
I'm in favor of unchecked exceptions in this case, but there's a catch (pun intended). We'll need to provide MonadBase and MonadBaseControl instances for Shell, so users can handle exceptions in the Shell monad.

Yeah, I agree that ExceptT without UnexceptionalIO is actually problematic because then you have to deal with both checked and unchecked exceptions in the same code.
I still haven't decided on exit code versus exceptions, but if we did exceptions my preference would be for Shell implementing MonadCatch

So I decided in the short term to go with adding an FAQ section which mentions the cmd .||. die "Message" trick, but I'll leave this open with the goal of adding Exception-based variants in an upcoming Turtle.Process module

In my opinion the lack of safety-by-default is something of a regression compared to Shelly or even bash. It feels wrong that I need to remember to handle errors in a Haskell library.

:+1:
@Gabriel439 mentioned that he might implement parallel APIs for throwing/returning error codes.

Yes, I will add exception-throwing variations of the subprocess runners named procs and shells and change the tutorial to recommend them instead


Fix: 23735 Web - App displays single report in LHN for some time on login with that report link

Details

Fixed Issues

$ https://github.com/Expensify/App/issues/23735
PROPOSAL: https://github.com/Expensify/App/issues/23735#issuecomment-1655501796
Tests

Open the app
Open any report and pin it
Copy the URL
Logout and paste the URL and press enter
Login and observe that app displays report open in step 2 for some time before displaying all the results
Verify that the loaded report and skeletons are shown in LHN until the total reports are loaded

[x] Verify that no errors appear in the JS console

Offline tests

QA Steps

Open the app
Open any report and pin it
Copy the URL
Logout and paste the URL and press enter
Login and observe that app displays report open in step 2 for some time before displaying all the results
Verify that the loaded report and skeletons are shown in LHN until the total reports are loaded

[x] Verify that no errors appear in the JS console

PR Author Checklist

[x] I linked the correct issue in the ### Fixed Issues section above
[x] I wrote clear testing steps that cover the changes made in this PR

[x] I added steps for local testing in the Tests section
[x] I added steps for the expected offline behavior in the Offline steps section
[x] I added steps for Staging and/or Production testing in the QA steps section
[x] I added steps to cover failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[x] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)
[x] I tested this PR with a High Traffic account against the staging or production API to ensure there are no regressions (e.g. long loading states that impact usability).

[x] I included screenshots or videos for tests on all platforms
[x] I ran the tests on all platforms & verified they passed on:

[x] Android / native
[x] Android / Chrome
[x] iOS / native
[x] iOS / Safari
[x] MacOS / Chrome / Safari
[x] MacOS / Desktop

[x] I verified there are no console errors (if there's a console error not related to the PR, report it or open an issue for it to be fixed)
[x] I followed proper code patterns (see Reviewing the code)

[x] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick)
[x] I verified that the left part of a conditional rendering a React component is a boolean and NOT a string, e.g. myBool && <MyComponent />.
[x] I verified that comments were added to code that is not self explanatory
[x] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[x] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method

[x] If any non-english text was added/modified, I verified the translation was requested/reviewed in #expensify-open-source and it was approved by an internal Expensify engineer. Link to Slack message:

[x] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[x] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is approved by marketing by adding the Waiting for Copy label for a copy review on the original GH to get the correct copy.
[x] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[x] I verified the JSDocs style guidelines (in STYLE.md) were followed

[x] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[x] I followed the guidelines as stated in the Review Guidelines
[x] I tested other components that can be impacted by my changes (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar are working as expected)
[x] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[x] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[x] I verified that if a function's arguments changed that all usages have also been updated correctly
[x] If a new component is created I verified that:

[x] A similar component doesn't exist in the codebase
[x] All props are defined accurately and each prop has a /** comment above it */
[x] The file is named correctly
[x] The component has a clear name that is non-ambiguous and the purpose of the component can be inferred from the name alone
[x] The only data being stored in the state is data necessary for rendering and nothing else
[x] If we are not using the full Onyx data that we loaded, I've added the proper selector in order to ensure the component only re-renders when the data it is using changes
[x] For Class Components, any internal methods passed to components event handlers are bound to this properly so there are no scoping issues (i.e. for onClick={this.submit} the method this.submit should be bound to this in the constructor)
[x] Any internal methods bound to this are necessary to be bound (i.e. avoid this.submit = this.submit.bind(this); if this.submit is never passed to a component event handler like onClick)
[x] All JSX used for rendering exists in the render method
[x] The component has the minimum amount of code necessary for its purpose, and it is broken down into smaller components in order to separate concerns and functions

[x] If any new file was added I verified that:

[x] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[x] If a new CSS style is added I verified that:

[x] A similar style doesn't already exist
[x] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(themeColors.componentBG))

[x] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[x] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[x] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[x] If the PR modifies a component or page that can be accessed by a direct deeplink, I verified that the code functions as expected when the deeplink is used - from a logged in and logged out account.
[x] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[x] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.
[x] I have checked off every checkbox in the PR author checklist, including those that don't apply to this PR.

Screenshots/Videos

Web

https://github.com/Expensify/App/assets/113963320/7ca69d42-ebd9-4717-8dc8-679874dbbf31

Mobile Web - Chrome

https://github.com/Expensify/App/assets/113963320/2d4e2401-c122-4af1-ac19-fd8344c17b3b

Mobile Web - Safari

https://github.com/Expensify/App/assets/113963320/5e5c208d-d520-43e9-9267-0f628cfe6f28

Desktop

https://github.com/Expensify/App/assets/113963320/418caa10-ac94-47ad-8e57-f3a23b59a80d

iOS

https://github.com/Expensify/App/assets/113963320/8ec3ec7c-8b8a-4f85-a826-a8e8dc2ceecd

Android

https://github.com/Expensify/App/assets/113963320/5b45e47a-cc7c-490d-a9b5-2d4355d0d5a8

Reviewer Checklist

[ ] I have verified the author checklist is complete (all boxes are checked off).
[ ] I verified the correct issue is linked in the ### Fixed Issues section above
[ ] I verified testing steps are clear and they cover the changes made in this PR

[ ] I verified the steps for local testing are in the Tests section
[ ] I verified the steps for Staging and/or Production testing are in the QA steps section
[ ] I verified the steps cover any possible failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[ ] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)

[ ] I checked that screenshots or videos are included for tests on all platforms
[ ] I included screenshots or videos for tests on all platforms
[ ] I verified tests pass on all platforms & I tested again on:

[ ] Android / native
[ ] Android / Chrome
[ ] iOS / native
[ ] iOS / Safari
[ ] MacOS / Chrome / Safari
[ ] MacOS / Desktop

[ ] If there are any errors in the console that are unrelated to this PR, I either fixed them (preferred) or linked to where I reported them in Slack
[ ] I verified proper code patterns were followed (see Reviewing the code)

[ ] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick).
[ ] I verified that the left part of a conditional rendering a React component is a boolean and NOT a string, e.g. myBool && <MyComponent />.
[ ] I verified that comments were added to code that is not self explanatory
[ ] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[ ] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method
[ ] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[ ] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is approved by marketing by adding the Waiting for Copy label for a copy review on the original GH to get the correct copy.
[ ] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[ ] I verified the JSDocs style guidelines (in STYLE.md) were followed

[ ] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[ ] I verified that this PR follows the guidelines as stated in the Review Guidelines
[ ] I verified other components that can be impacted by these changes have been tested, and I retested again (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar have been tested & I retested again)
[ ] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[ ] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[ ] If a new component is created I verified that:

[ ] A similar component doesn't exist in the codebase
[ ] All props are defined accurately and each prop has a /** comment above it */
[ ] The file is named correctly
[ ] The component has a clear name that is non-ambiguous and the purpose of the component can be inferred from the name alone
[ ] The only data being stored in the state is data necessary for rendering and nothing else
[ ] For Class Components, any internal methods passed to components event handlers are bound to this properly so there are no scoping issues (i.e. for onClick={this.submit} the method this.submit should be bound to this in the constructor)
[ ] Any internal methods bound to this are necessary to be bound (i.e. avoid this.submit = this.submit.bind(this); if this.submit is never passed to a component event handler like onClick)
[ ] All JSX used for rendering exists in the render method
[ ] The component has the minimum amount of code necessary for its purpose, and it is broken down into smaller components in order to separate concerns and functions

[ ] If any new file was added I verified that:

[ ] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[ ] If a new CSS style is added I verified that:

[ ] A similar style doesn't already exist
[ ] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(themeColors.componentBG)

[ ] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[ ] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[ ] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[ ] If the PR modifies a component or page that can be accessed by a direct deeplink, I verified that the code functions as expected when the deeplink is used - from a logged in and logged out account.
[ ] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[ ] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.
[ ] I have checked off every checkbox in the PR reviewer checklist, including those that don't apply to this PR.

Screenshots/Videos

Web

https://github.com/Expensify/App/assets/24796318/6c2367cd-f7e7-4376-8921-7e8770d1ee3e

Mobile Web - Chrome

https://github.com/Expensify/App/assets/24796318/d891edd9-f4be-4559-8d13-d20e6d688b54

Mobile Web - Safari

https://github.com/Expensify/App/assets/24796318/9002e52f-16ee-4f6c-b949-e6510cfb31d3

Desktop
https://github.com/Expensify/App/assets/24796318/aab15e40-5c85-4b28-911f-5f2c36a59892

iOS

https://github.com/Expensify/App/assets/24796318/8940b54f-1e00-4411-9b53-26f49b848812

Android

https://github.com/Expensify/App/assets/24796318/f10b0aea-2886-4f0b-b5a5-2fc6258a6811

:hand: This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release.

🚀 Deployed to staging by https://github.com/aldo-expensify in version: 1.3.54-0 🚀

platform
result

🤖 android 🤖
failure ❌

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
failure ❌

🕸 web 🕸
success ✅

🚀 Deployed to staging by https://github.com/aldo-expensify in version: 1.3.54-0 🚀

platform
result

🤖 android 🤖
failure ❌

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
failure ❌

🕸 web 🕸
success ✅

🚀 Deployed to production by https://github.com/yuwenmemon in version: 1.3.54-13 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

This PR caused a regression for native apps...
To reproduce:

Sign in
Background the app
Bring the app back to foreground
Sidebar will show a loading state while reports load

I think it's this Issue => https://github.com/Expensify/App/issues/24573

@tienifr could you address the regression as PR author please? cc @ArekChr

This case was not handled in this PR.
Issue: Login - LHN does not show skeleton placeholder when the page is refreshed while loading
Steps to reproduce:

Go to staging.new.expensify.com
Log in to HT account.
While the page is loading after login, refresh the page.
LHN does not show skeleton placeholder when the page is refreshed while it is still loading.


Could you have a look at line 91 in svg.resize.js

Line 91:    p: this.transformPoint(event.detail.event.clientX,event.detail.event.clientY),
event.detail.event.clientX  when I change it to be event.detail.x, It works well.

What is the actual error you encountered?


Prep/consider expansion to Contra Costa

BLUF: We would expand to Contra Costa (CC) to learn about application bottlenecks in another county (aka build the funnel) and compare them to SF. Common bottlenecks suggests product/feature opportunities. Disparate bottlenecks suggest there's something important to know but we might not have the hammer (business process interventions, policy interventions, etc.)
Table stakes (aka what we need to move forward)

CC sets up email address to receive Clean apps and clears all incoming cases at least daily
CC covers ~2 days/week for someone to do Clean application tracking
CC specifies basic business process for how Clean applications are going to get Cleared > scheduled > interviewed > approved/denied.
Vakil intros us to ~3 intermediaries who would be active users (probably food bank and other groups currently doing screening + referral or application assistance) so we can talk to them on the phone.

@dave can you review the goals + table stakes above? Does this capture what we want to learn and what we need to say yes?
@RebeccaCoelius can you sanity check the rationale above for expanding clean to Contra Costa? Anything obvious we're not considering?
Notes from first convo with Vakil

Vakil retired as CIO of SF HSA who was our city partner in 2013. He's now doing some consulting work with CC and others to figure out more scalable outreach models.
The first model he's trying in CC is to move the food bank away from direct application assistance and towards supporting others (other CBOs + volunteers) to do application assistance.
CC has ~58% participation now and targeted 75% over next two years, which means adding an ~10-14k cases (depending on churn/drop rate)
CC has 2 units of EWs (~10 total) who do outreach and support CBOs. CBOs call this unit directly to get support on their cases and Clean apps would likely go directly to this group.
There's a "CalFresh Partners" meeting where all county stakeholders come together and share stats...this would be good place to start.

Next steps

Send table stakes to Vakil (once @daguar confirms)
Vakil would secure table stakes and probably pull me into a couple phone calls
Vakil and I probably offer Clean to food bank + folks at the next CalFresh Partners meeting

Overall, :+1:, but a few notes:

I'd like to lay out explicitly (and set timeline for) new features expanding to CC requires before moving ahead with this. At a minimum we're talking about:
New Heroku instance
Some changes to where SF is hard-coded, and abstracting out the case tracking link in email
I think this is a good opportunity to test out what minimal business process change looks like. I say we "let it flow, but instrument" — meaning, let's make the only business change as it relates to actually processing the applications be adding an email address that gets checked and processed, but let's use our internal relationship here to measure where it fails (and fix it for those clients). The goal here: to learn what will go wrong if a county does nothing else besides adding an e-mail address.
I thought CC might use C4 (instead of MyBenefitsCalWin [MBCW]) and so we'd be able to test out how our process works in C4 counties, but it turns out that CC uses MBCW, so not too much new to learn there.
Some other specific comments:

CC covers ~2 days/week for someone to do Clean application tracking

Gut check: is this the level of effort? Feels like it could be high, but I may be wrong and you have a better finger on the pulse.

Vakil intros us to ~3 intermediaries who would be active users (probably food bank and other groups currently doing screening + referral or application assistance) so we can talk to them on the phone.

I think it's worth going out there for a day and meeting with all these folks (assistors + internal agency folks). It's not far and would be just the right thing to do for county # 2.

Some thoughts I put in Slack:
A few of my thoughts re expanding Clean to Contra Costa:
Goals

Main goal is to test our current hypothesized scaling model: specifically, if an agency does nothing more than add an email address accepting password-protected applications, will the existing business processes handle it?

2nd goal is to control for SF-specific variables: we have no clue how much of SF HSA's internal processes (and clients' failure points) are similar to the rest of CA. This is the first step to learning what things are generally how counties do it vs. just how SF does it.

We instrument the biz process both for above learning goals but ALSO to ensure all clients through our system succeed, despite potential failure: that's the main justification for staff time, IMHO.

Arguments for Contra Costa as the 2nd county

To scale responsibly, the 2nd county needs the higher level of buy-in to instrument biz process (to learn about funnel + make sure all eligible approved) —— we have that in Contra Costa but nowhere else yet (except maybe Solano). It's an opportunistic call.

Intuition — SF is more dissimilar to other counties in CA than it is similar. Contra Costa is just straight-up more like the avg county.

CA counties basically have 3 types — urban coastal, rural central CA, and LA. Eventually we want to understand the rural+LA, but for now let's make sure we understand urban coastal.

It's close — so we can have more personal relationship.

In the comment above you state the CC's participation rate is ~58%, with a future target of 75%. In the spreadsheet produced by Diana with ACS data, the adjusted participation rate point estimate is ~85% (SF's is 49% by comparison). Is there something that needs be resolved there, or should we use a particular dataset for these discussions?
I definitely think that Contra Costa serves as a decent proxy for exurban counties, which characterizes a substantial segment of California counties. They are going to have a different ecosystem based on that characterization—different population types, different mobility needs, fewer institutions with well established track records of service delivery and assistance.
I'd also like to get explicit here about what features we think will be required (and timeline for those features, h/t @daguar). It sounds like a lot of that is just going from 1 client to two, abstracting away hard-coded things here and there—and almost nothing user facing.


Can't find Rabbitmq user and password in config file

I have setup Lumify using  RabbitMQ 3.6 , however I can't find any config setting to setup user and password for RabbitMQ. ( I want to use user "test" and password "test")
I am getting following error
Error injecting constructor, com.rabbitmq.client.AuthenticationFailureException: ACCESS_REFUSED - Login was refused using authentication mechanism PLAIN. For details see the broker logfile.
at io.lumify.model.rabbitmq.RabbitMQWorkQueueRepository.(RabbitMQWorkQueueRepository.java:32)
while locating io.lumify.model.rabbitmq.RabbitMQWorkQueueRepository
Nowhere in documentation , I find bypassing authentication , Please help

You'll need to use the command line management tool, rabbitmqadmin, to add the user test. You can use the guest user (password guest) to start out with but there's no user test when installed.
For more info on rabbitmqadmin's you can use the --help argument to see the usage.
rabbitmqadmin add_user test test is probably the command you want.

i actually created a new user test/test , but anyway, guest/guest is working . thanks

"rabbitmqadmin add_user test test " 'rabbitmqadmin' is not recognized as an internal or external command,
operable program or batch file


Added export_avatars_local_helper().

Automated message from Dropbox CLA bot
@showell, it looks like you've already signed the Dropbox CLA. Thanks!

This is for #1567.  I have some questions about the attachment stuff before you push this.

I merged the first 4 commits.

Cool, I merged everything but the last commit.

I'll open a new PR for the attachment stuff and threads and partial.


[fix](cluster key) fix cluster key schema change

What problem does this PR solve?
Issue Number: close #xxx
Related PR: #xxx
Problem Summary:
for schema change: add a key in the tail of all keys, may get:
F20241209 14:52:09.513403 3166619 segment_writer.cpp:1309] Check failed: key.compare(last_key) >= 0 key is not sorted! current key: ^A^B06olTqiL2rEn48nTpmP8, last key: ^B^@^OЭ^BxsmIHehvUnv
*** Check failure stack trace: ***
    @     0x563674f2dcc6  google::LogMessage::SendToLog()
    @     0x563674f2a710  google::LogMessage::Flush()
    @     0x563674f2e509  google::LogMessageFatal::~LogMessageFatal()
    @     0x56366a8a72c0  doris::segment_v2::SegmentWriter::_generate_short_key_index()
    @     0x56366a8a6032  doris::segment_v2::SegmentWriter::append_block()
    @     0x56366a7624c9  doris::SegmentCreator::add_block()
    @     0x56366a731d35  doris::BaseBetaRowsetWriter::add_block()
    @     0x56366a94a233  doris::VSchemaChangeDirectly::_inner_process()
    @     0x56366a95d558  doris::SchemaChange::process()
    @     0x56366a9511e5  doris::SchemaChangeJob::_convert_historical_rowsets()
    @     0x56366a94e5e2  doris::SchemaChangeJob::_do_process_alter_tablet()
    @     0x56366a94cf22  doris::SchemaChangeJob::process_alter_tablet()
    @     0x563669e9e857  doris::alter_tablet_callback()

Release note
None
Check List (For Author)

Test

[ ] Regression test
[ ] Unit Test
[ ] Manual test (add detailed scripts or steps below)
[ ] No need to test or manual test. Explain why:

[ ] This is a refactor/code format and no logic has been changed.
[ ] Previous test can cover this change.
[ ] No code files have been changed.
[ ] Other reason

Behavior changed:

[ ] No.
[ ] Yes.

Does this need documentation?

[ ] No.
[ ] Yes.

Check List (For Reviewer who merge this PR)

[ ] Confirm the release note
[ ] Confirm test cases
[ ] Confirm document
[ ] Add branch pick label

run buildall

run buildall

run buildall

run buildall

run buildall


Updated dpdb (adding Pixel)

Pixel XL is in the database, however Pixel was not covered, and was logging "No DPDB device match".
Also I would like to recommend that https://storage.googleapis.com/cardboard-dpdb/dpdb.json be updated -- as of this pull request it is at least 2 versions behind (let me know if there should be a separate issue for this).

I'll accept this change for now, but keep in mind that DPDB is autogenerated from an internal database so at some point in the future, your change may be reverted.


resources/server/test/test-web-integration.sh: No such file or directory

VSCode Version:
OS Version:

Steps to Reproduce:

Run test of browser. vscode>test>integration>browser
As per readme Run (inside browser) resources/server/test/test-web-integration.sh this script is missing in repo.

Does this issue occur when all extensions are disabled?: Yes

We currently do not allow to run web integration tests from OSS, sorry.

We currently do not allow to run web integration tests from OSS, sorry.


Potion & enchantment support

Description
Recipes do not, by default, support NBT data. Hardcoding support for potion and enchantment recipes would allow them to be transmuted. Enchanted books (and possibly other enchanted items?) will release their enchantments when dissolved, according to enchantment recipes described in JSON. The essentia gained will be multiplied by a factor based on the enchantment's level (1, 2, 4, 7, 11, ... is one possible plan). Potions will also release essentia based on their effects. Enchantments can be applied to books by using a book in place of materia in the Synthesis Table, with an enchanted book or enchantment formula in the target slot. Potion effects can be applied to water bottles by using a water bottle in place of materia in the Synthesis Table, with a potion or potion formula in the target slot.

try looking into NBT crafting for this, and I mean Siphlor's mod called that

I'm aware of this mod, but I think I'm going to take a different approach for gameplay reasons.

Art of Alchemy is now being maintained by a different team in a new repository. Please use https://github.com/Cumulus-Mods/Art-of-Alchemy/issues/4 for further discussion on this issue. Thank you.


Try GaussAdjoint

Following the latest developments and what has been discussed in the review paper, we should try and benchmark the new GaussAdjoint to see if it makes things faster.

This has been added in #151. To be benchmarked accordingly once the new workflow is working with Enzyme.jl.


Optimize layout and add icon

Optimize layout and add icon

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.v_lixin18 seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

Codecov Report

Merging #494 into develop will decrease coverage by 0.00%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##           develop     #494      +/-   ##
===========================================
- Coverage    53.24%   53.23%   -0.01%
===========================================
  Files          257      257
  Lines        23604    23604
===========================================
- Hits         12567    12565       -2
- Misses        9556     9557       +1
- Partials      1481     1482       +1

Impacted Files
Coverage Δ

bfe_util/fixed_pool.go
83.33% <0.00%> (-16.67%)
:arrow_down:

bfe_spdy/frame_read.go
57.35% <0.00%> (-0.76%)
:arrow_down:

bfe_spdy/server_conn.go
78.53% <0.00%> (ø)

bfe_websocket/server_conn.go
74.24% <0.00%> (+1.51%)
:arrow_up:

Continue to review full report at Codecov.

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


Religion DC for movespeed buff increasing indefinitely (causing infinite black screen)

Description of the bug
Not sure if that's a toybox only bug, but someone had a similar issue a week or 2 ago where the religion DC check to (i think) determine the length of the movespeed buff after rest would be attempted until fail by increasing the dc by 5 each time. But because they couldn't fail the check, it increased something like 70 000 times and they were stuck in a infinite black screen because it wouldn't stop going up (Source: https://discord.com/channels/645948717400064030/815735034514112512/1189256931957817384)
Steps To Reproduce
Blup
Expected Behavior
No response
Log Files
No response
Save File
No response
Screenshots
No response
Settings
No response
Toybox Version
1.5.9
Operating System
No response
What game are you using Toybox on?
Pathfinder Wrath of the Righteous
Game Version
No response
Anything else?
No response

Believe this was fixed in 1.5.10a?


Show plots generated with autoviz as part of a marimo notebook does not work

Describe the bug
I am trying to use Autoviz inside of a marino notebook.
The input dataset is the titanic train dataset from kaggle.
This does not raise any errors, however I cannot convince marimo to show the plots in the notebook. It works with Jupyer Notebooks when adding the %matplotlib inline statement.
Any ideas why this happens and how to fix this?
The environment is set up with poetry. If any additional information is required, I am glad to share it. Thank you very much! marimo is an amazing tool!
Environment
{
  "marimo": "0.9.12",
  "OS": "Linux",
  "OS Version": "6.6.54-2-MANJARO",
  "Processor": "",
  "Python Version": "3.12.6",
  "Binaries": {
    "Browser": "--",
    "Node": "v18.20.4"
  },
  "Dependencies": {
    "click": "8.1.7",
    "docutils": "0.21.2",
    "itsdangerous": "2.2.0",
    "jedi": "0.19.1",
    "markdown": "3.7",
    "narwhals": "1.10.0",
    "packaging": "24.1",
    "psutil": "6.0.0",
    "pygments": "2.18.0",
    "pymdown-extensions": "10.11.2",
    "pyyaml": "6.0.2",
    "ruff": "0.7.0",
    "starlette": "0.41.0",
    "tomlkit": "0.13.2",
    "typing-extensions": "4.12.2",
    "uvicorn": "0.32.0",
    "websockets": "12.0"
  },
  "Optional Dependencies": {
    "altair": "5.4.1",
    "pandas": "2.2.2",
    "polars": "1.5.0"
  }
}

Code to reproduce
import marimo

__generated_with = "0.9.12"
app = marimo.App()

@app.cell
def __():
    import marimo as mo
    return (mo,)

@app.cell
def __():
    import pandas as pd
    import polars as pl

    from autoviz.AutoViz_Class import AutoViz_Class
    return AutoViz_Class, pd, pl

@app.cell
def __(pd):
    df = pd.read_csv("data/train.csv")
    return (df,)

@app.cell
def __(df):
    df
    return

@app.cell
def __(AutoViz_Class, df):
    AV = AutoViz_Class()
    AV.AutoViz(df, depVar="Survived", chart_format="png")
    return (AV,)

@app.cell
def __():
    return

if __name__ == "__main__":
    app.run()

I am not sure what the desired output is supposed to look like, but you can convert it to Html to rendering inside marimo.
mo.Html(AV.AutoViz(df, depVar="Survived", chart_format="png").to_html())

Thank you very much. The issue did not solve with mo.Html, I am not sure where the output is captured but it does not end up in the marimo UI. If you are running the same code cells in a jupyter notebook, you can see that the plots appear as part of the jupyter notebook. If it helps, I can also create an share the jupyter notebook.

@johanneswerner, a jupyter notebook would be great. can you also screenshot what you currently see when doing mo.Html(...to_html())?

Since github does not allow me to upload a jupyter notebook file to a comment, here is the content
{
 "cells": [
  {
   "cell_type": "code",
   "execution_count": null,
   "metadata": {},
   "outputs": [],
   "source": [
    "import pandas as pd\n",
    "import polars as pl\n",
    "\n",
    "from autoviz.AutoViz_Class import AutoViz_Class"
   ]
  },
  {
   "cell_type": "code",
   "execution_count": null,
   "metadata": {},
   "outputs": [],
   "source": [
    "%matplotlib inline"
   ]
  },
  {
   "cell_type": "code",
   "execution_count": null,
   "metadata": {},
   "outputs": [],
   "source": [
    "df = pd.read_csv(\"data/train.csv\")\n"
   ]
  },
  {
   "cell_type": "code",
   "execution_count": null,
   "metadata": {},
   "outputs": [],
   "source": [
    "AV = AutoViz_Class()\n",
    "AV.AutoViz(df, depVar=\"Survived\", verbose=2, chart_format=\"png\")"
   ]
  },
  {
   "cell_type": "code",
   "execution_count": null,
   "metadata": {},
   "outputs": [],
   "source": []
  }
 ],
 "metadata": {
  "kernelspec": {
   "display_name": ".venv",
   "language": "python",
   "name": "python3"
  },
  "language_info": {
   "codemirror_mode": {
    "name": "ipython",
    "version": 3
   },
   "file_extension": ".py",
   "mimetype": "text/x-python",
   "name": "python",
   "nbconvert_exporter": "python",
   "pygments_lexer": "ipython3",
   "version": "3.12.6"
  }
 },
 "nbformat": 4,
 "nbformat_minor": 2
}

When I am running the notebook, it produces plots after the last command.
With marimo, it does not show the plots, see screenshot below

and at the end of the textarea

Further information: I am using poetry here and it seems nltk (used for word cloud plots) has problems finding the source path for the nltk data (which ends up in my home folder, not in my environment folder) - but even without nltk, Autoviz should still be able to create plots that are not depending on nltk (which are produced but not showing inline in the plot).


Updated/Changed some strings

Such changes are best done through the transifex service.
https://www.transifex.com/rainloop/rainloop-webmail

Oh, didn't know that, already signed in.


[Feature]  Retries are required for file deletion

Pre-check

[X] I am sure that all the content I provide is in English.

Search before asking

[X] I had searched in the issues and found no similar feature requirement.

Apache Dubbo Component
Java SDK (apache/dubbo)
Descriptions
Retries are really required while deleting any files.
"Files.delete(pathOfFile)" in "FileCacheStore"  may fail due to any of below reasons
"File is in Use", "File Locking", "File System Delays", "Concurrency Issues" etc.
Related issues
No response
Are you willing to submit a pull request to fix on your own?

[X] Yes I am willing to submit a pull request on my own!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

Thanks @AlbumenJ . We do not retry when we get an exception, correct?

In this case, I would prefer use safe delete without any error.


Add AddEmptySlide overloading methods.

:astronaut: Hi @adamshakhabov
In order to give you a proposal for #508 I created this PR.
public interface ISlideLayoutType
{
    string Value { get; }
}

public interface ISlideCollection : IReadOnlyList<ISlide>
{
    ISlide AddEmptySlide(ISlideLayout layout);
    ISlide AddEmptySlide(ISlideLayoutType layoutType);
    ISlide AddEmptySlide(string layoutName);
}

public interface ISlideLayout
{
    ISlideLayoutType Type { get; }
}

:medal_military: Expected behavior of  ISlideCollection .AddEmptySlide(ISlideLayoutType):

Add a new slide from "default" layout: pres.Slides.AddEmptySlide(SCSlideLayoutType.Title)
It find first layout in Master Layout
Copy all placeholders of layout without text content.

:medal_military: Expected behavior of ISlideCollection .AddEmptySlide(string layoutName)

Add a new slide from layout with given name

:medal_military: Expected behavior of SCSlideLayoutType

Initializes a layout type based on type.

I observed that all layouts types described in the specification are not present in powerpoint file, I don't know if this could be problematic someday  :fortune_cookie:
Thanks

PR-Codex overview
This PR introduces a new interface ISlideLayoutType and a new class SCSlideLayoutType that enumerates all possible slide layout types. It also adds functionality to add a new slide based on a layout type or layout name.
Detailed summary

Added ISlideLayoutType interface
Added SCSlideLayoutType class that enumerates all possible slide layout types
Added AddEmptySlide(ISlideLayoutType layoutType) method to ISlideCollection
Added AddEmptySlide(string layoutName) method to ISlideCollection
Added AddEmptySlide(Func<ISlideLayout, bool> query) method to ISlideCollection
Updated ISlideLayout interface to include Type property
Updated SCSlideLayout class to include Type property and GetLayoutType() method

The following files were skipped due to too many changes: src/ShapeCrawler/Enums/SCSlideLayoutType.cs

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

After giving it some thought, I believe we should just provide 9 default layouts, similar to what PowerPoint does:

Title
Title and Content
Section Header
Two Content
Comparison
Title Only
Blank
Content with Caption
Picture with Caption

Both the default and new Slide Masters (ISlideMaster) should already include these layouts (ISlideLayout.Name), or should be able to create them when a user specifies a name.
So, adding the new method ISlideCollection.AddEmptySlide(string layoutName) should be sufficient.

The slide layout have a type property, but the layout name is defined in their CommonSlideData.Name.
The enum class SCSlideLayoutType provides:

Title Title = new("title")
Title and Content TitleAndObject = new("obj");
Section Header SectionHeader = new("secHead");
Two Content TwoObjects = new("twoObj");
Comparison
Title Only TitleOnly = new("titleOnly");
Blank Blank = new("blank");
Content with Caption TitleObjectAndCaption = new("objTx")
Picture with Caption PictureAndCaption = new("picTx");
Custom PictureAndCaption = new("cust");
...

The CommonSlideData.Name is not a constant value, depends on regional culture (maybe), for example in my computer:
Picture with Caption
<p:sldLayout type="picTx" preserve="1">
  <p:cSld name="Imagen con título">
  </p:cSld>
</p:sldLayout>

Both the default and new Slide Masters (ISlideMaster) should already include these layouts (ISlideLayout.Name), or should be able to create them when a user specifies a name.

But on the other hand, presentation created by SCPresentation.Create(); does have just one layout with no Name and Type so it's imposible to create a new layout by any type or name. To solve this problem, maybe we need to apply a "theme" to presentation from a default file (maybe included in the package assets). Basically, copy the slide master in the "source" presentation to the new presentation. Maybe this could be another issue.
Layouts are dependent of the user presentation, and they should know the name of the layout, AddEmptySlide(PictureAndCaption) gives him a shortcut to add an slide without know the display name, but the result could be the expected.
:thinking: As you say, we can remove this overloading and left the SCSlideLayoutType for a other PR (:joystick:"Creates a default slide layout" ISlideMaster.AddLayout(SCSlideLayoutType))
public interface ISlideCollection : IReadOnlyList<ISlide>
{
-  ISlide AddEmptySlide(ISlideLayoutType layoutType);
}

Let me know if you are agree with the proposal to remove the overloading method... have a nice day...
:male_detective:

It's all right! I understand your idea now🙂 Let's keep these overloads.
Could you please add a unit test to verify that the created slide contains placeholders from the layout?

Hi!
I added a generic class for "enum classes", this is more easy for me.
private ISlideLayoutType GetLayoutType()
{
    if (SCSlideLayoutType.TryParse(this.SlideLayoutPart.SlideLayout.Type, out var layoutType))
    {
        return layoutType!;
    }

    return SCSlideLayoutType.Custom;
}

public class SCSlideLayoutType : Enumeration<SCSlideLayoutType>
{
  // ...
}

But I just noticed that there is already exists an enumeration SlideLayoutValues, and I don't know if this is a problem. Enum classes are more easy to me, because I can add more "info" or methods to a value... anyway, I can remove if you want.
A new test Add_add_adds_slide_from_layout was added.
:cowboy_hat_face:


CRI Sbserver: Make PodSandboxStatus friendlier to shim crashes

Currently if you're using the shim-mode sandbox server support, if your shim that's hosting the Sandbox API dies for any reason that wasn't intentional (segfault, oom etc.) PodSandboxStatus is kind of wedged. We can use the fact that if we didn't go through the usual k8s flow of Stop->Remove and we still have an entry in our sandbox store, us not having a shim mapping anymore means this was likely unintentional.

@Iceber  and I are testing on this mode last Friday. runc did not support it yet. This pr makes sense when users try it.


Could you please upload this to PyPi?

Greetings!
This is a nice little module! My team is looking to open source a library that uses this project and it would be great if we could grab it from PyPi.
Ref: http://the-hitchhikers-guide-to-packaging.readthedocs.io/en/latest/quickstart.html
Thanks so much! This saved me a good chunk of time ❤

Sounds doable, @bol I think most of the required package preparation work is already done, pypi requires some kind of registration, since the original repo is under nordnet it would make sense for you or someone at nordnet to open an account. I can of course package and upload it if you want.
@stark525  - No problem! Glad you found it useful :)

Should fit in well with our open source policies. I'm on vacation for
another two weeks and actually didn't bring a computer. I'll take a look
when I get back to civilization.
On Sat, Aug 4, 2018, 21:36 Andreas Calminder<EMAIL_ADDRESS>wrote:

Sounds doable, @bol https://github.com/bol I think most of the required
package preparation work is already done, pypi requires some kind of
registration, since the original repo is under nordnet it would make sense
for you or someone at nordnet to open an account. I can of course package
and upload it if you want.
@stark525 https://github.com/stark525 - No problem! Glad you found it
useful :)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/nordnet/python-freeipa-json/issues/8#issuecomment-410472786,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAEPY81ezuY5r1gLBSrjoQPhtPqeCD58ks5uNffDgaJpZM4VnJyu
.

Hasn't been packaged yet, I probably shouldn't have used "fixes #8" in the PR desc :)

I think maintainer does not support this project.


Question: 40xx visibility errors and the specification

https://github.com/Microsoft/TypeScript/blob/master/doc/spec.md#104-export-declarations
Own example:
module M1 {
    class C {
    }
    export interface F extends C {   // according to the spec, C is not exported and that should be an error
    }
}

Example just from the specification:
interface A { x: string; }

namespace M {
    interface B { x: A; }
    interface C { x: B; }
    export function foo(c: C) {  }  // according to the spec, this should be an error, to reference non-exported interface in exported function parameter
}

But the compiler yields no error in both cases.
Is this specification section obsolete, or is it a compiler bug?

The spec should be more explicit.
the error is generated iff you are trying to emit declarations (--declarations) for your sources. otherwise it is OK.
the rational is, if you are emitting declarations, the emitter needs to write a name for type C. this will result in exposing something that is not originally exposed. so all your exported declarations should not use other declarations of lesser visibility.
if you are not emitting declarations, there is no reason to prohibit you from doing this. C is not visible, so callers of foo, can not reference C by name, but can provide something that is structurally compatible with C.

The spec is now an "archived" artifact and we won't be tracking defects against it anymore.


update terser dependency vulnerability

also fix unit tests with localstack

Thanks for this PR :bow:
It looks all alright but I don't get why you need to rename the hostname, although it also works. Which is also confusing and I guess this is just me not gettign something that docker does ? :thinking:

it was confusing to me too. it is actually a localstack path-style issue where AWS is deprecating path-style and to use virtual hosting in localstack we can't run "localhost:4566" without updating /etc/hosts file. i did some digging and a much shorter path than what I went through is:
https://github.com/localstack/localstack/issues/1792
https://docs.aws.amazon.com/AmazonS3/latest/userguide/VirtualHosting.html
and https://docs.localstack.cloud/aws/s3/ at the bottom.
In my testing, i discovered it handles it for all the localstack services so just changing hostname was easier than making some hack to support s3 alone.


Port request: ctemplate

https://github.com/OlafvdSpek/ctemplate ;)

@OlafvdSpek Initial Port

Thanks a lot!

@ras0219-msft Could you close this?


Cannot toggle breakpoints by clicking in the gutter

Describe the bug
I am aware of the Tips and Tricks to get around the issue but suggest making it narrower
When a breakpoint is added before the gutter coverage appear you cannot remove it by clicking on the breakpoint.  You can also not add breakpoints where the coverage indicators appear in the gutter.  Adding breakpoints using the shortcut works.
To Reproduce
Steps to reproduce the behaviour:

Run the tests to generate the coverage file.
Open a source code file.
Enable the display of code coverage in the gutter (ctrl + shift + 7).
Add a breakpoint by clicking on the gutter where there are colours indicating coverage.

Expected behaviour
A breakpoint should be toggled.
Desktop (please complete the following information):

OS: Linux Mint
Extension Version: 2.6.0
VSCode Version

Version: 1.52.1
Commit: ea3859d4ba2f3e577a159bc91e3074c5d85c0523
Date: 2020-12-16T16:32:10.090Z
Electron: 9.3.5
Chrome: 83.0.4103.122
Node.js: 12.14.1
V8: <IP_ADDRESS>-electron.0
OS: Linux x64 5.8.0-34-generic

Additional context
The problem seems to be that the coloured bars indicating coverage doesn't allow click-through and is thus in the way and intercepts the click.
A possible solution would be to make the bars slightly narrower to allow for clicking next to them.

I realised that it's an issue with VS code

I realised that it's an issue with VS code

@PieterHartzer Thanks for submitting a ticket! I see you figured out the issue, in the meantime, you can find the workaround in the readme under tips and tricks. https://github.com/ryanluker/vscode-coverage-gutters#tips-and-tricks

@PieterHartzer Thanks for submitting a ticket! I see you figured out the issue, in the meantime, you can find the workaround in the readme under tips and tricks. https://github.com/ryanluker/vscode-coverage-gutters#tips-and-tricks


Feature/improve route replacement

Provide a narrative description of what you are trying to accomplish:

[x ] Are you fixing a bug?

[x] Detail how the bug is invoked currently.
\ZF\Apigility\Documentation\Swagger\Service::getRouteWithReplacements was not handling well all the route

[x] Detail the original, incorrect behavior.
The method was not handling route like /user/:user_id

[x] Detail the new, expected behavior.
It now handle it

This repository has been closed and moved to laminas-api-tools/api-tools-documentation-swagger; a new issue has been opened at https://github.com/laminas-api-tools/api-tools-documentation-swagger/issues/3.

This repository has been moved to laminas-api-tools/api-tools-documentation-swagger. If you feel that this patch is still relevant, please re-open against that repository, and reference this issue. To re-open, we suggest the following workflow:

Squash all commits in your branch (git rebase -i origin/{branch})
Make a note of all changed files (`git diff --name-only origin/{branch}...HEAD
Run the laminas/laminas-migration tool on the code.
Clone laminas-api-tools/api-tools-documentation-swagger to another directory.
Copy the files from the second bullet point to the clone of laminas-api-tools/api-tools-documentation-swagger.
In your clone of laminas-api-tools/api-tools-documentation-swagger, commit the files, push to your fork, and open the new PR.
We will be providing tooling via laminas/laminas-migration soon to help  automate the process.


🐞 Revert "fix(pages/Index.vue): fix can't select file in dialog"

Reverts Daniele-rolli/Beaver-Notes#45

@all-contributors please add @bigshans for 🇨🇳 Simplified Chinese and Linux Bugs


Add more fine-grained control of repeating transactions

Hello,
I would like to request a finer control over repeating transactions. For example, I get paid every 2 weeks, but there's no way to set a 2 week recurring payment in Money that I have found. I could split my income into two payments and have it work with Money, but it would be nicer to see it come in on schedule as expected.
Thank you!

Need to create a schedule widget to create a proper schedule for repeating transactions.
Here is the one from GNU Cash:

We won't be able to get this feature ready in time for V2022.11.1. However, definitely will be included in the next release!

Thank you!

https://github.com/nlogozzo/NickvisionMoney/releases/tag/2023.1.0-beta3


adafruit_requests.Session() failed!

I am trying to initialize a request session on my esp32 board. However, it failed, with error message:
File "adafruit_connection_manager.py" line395, in get_connection manager
TypeError: unsupported type for __hash__: socketpool

The version of circuitPython is 8.2.7, my board is ESP32-S2 Reverse TFT Feather. The code I used is attached.
# settings.toml imports
aio_username = os.getenv('AIO_USERNAME')
aio_key = os.getenv('AIO_KEY')
# connect to wifi
wifi.radio.connect(os.getenv('CIRCUITPY_WIFI_SSID'), os.getenv('CIRCUITPY_WIFI_PASSWORD'))
pool = socketpool.SocketPool(wifi.radio)
requests = adafruit_requests.Session(pool, ssl.create_default_context())

Any suggestions?

I can check for backwards compatibility, have you tried using CircuitPython 9.x?

Switching to CircuitPython 9.X solved this issue for me.


semantic image search

Describe the enhancement you'd like
semantic image search
Describe why this will benefit the LibrePhotos
searching images by description
Additional context
openai published CLIP which is trained on Image and text, so it's possible to do zero shot classification or semantic search by vector similarity

I took a look into the code and saw that you already do verctor search for similar images
J think it should be easy replacing the resnet model with clip model and adding text as search input too.
Where to do so ?
Furthermore I'd like to add tesseract ocr, but don't know where's the right place to do so too.

You probably want to change this API Endpoint: https://github.com/LibrePhotos/librephotos/blob/e73c44cc118b3f61459700883d2d1b71308b8ea0/ownphotos/urls.py#L170-L173
Which would be this view: https://github.com/LibrePhotos/librephotos/blob/e73c44cc118b3f61459700883d2d1b71308b8ea0/api/views.py#L202-L223
You probably want to save the encoding / text once and the search of it on demand, right?
You would need to add a couple of new fields here: https://github.com/LibrePhotos/librephotos/blob/dev/api/models/photo.py
You can add it either to this function: https://github.com/LibrePhotos/librephotos/blob/e73c44cc118b3f61459700883d2d1b71308b8ea0/api/models/photo.py#L104-L128
Or you could write your own function and then call it in the scan process here: https://github.com/LibrePhotos/librephotos/blob/e73c44cc118b3f61459700883d2d1b71308b8ea0/api/directory_watcher.py#L95-L103
Do you know how much resources  this algorithm needs and how fast it is?
If it is to ressource heavy, you could also implement this object detection and classification: https://github.com/LibrePhotos/librephotos/issues/54
When we use mobilenet, it should work fast on any machine and we could take the labels to search semantically over it.

https://github.com/UKPLab/sentence-transformers/blob/master/examples/applications/image-search/Image_Search.ipynb
Here is an easy to use example of clip
Locally its pretty fast with in memory search

You probably want to save the encoding / text once and the search of it on demand, right?

There is no text to save
The image gets encoded into vector / tensor, this gets stored in database
While searching the query / text get's encoded as vector too and the similar image vectors get searched
Would be adding tesseract OCR to photocaptions ok ?

Alright, sounds similar to the image_similarity algorithm we use. I think you would then add the clip_encoding as field to photo and calculate them when scanning.
You would need to figure out how to display the result of the query. You have to merge the results of the search over captions with the clip results.
If you want to add Tesseract OCR, I think it would be better to use a new field for the results. For example you could name it tesseract_captions. We already have an issue for that here: https://github.com/LibrePhotos/librephotos/issues/228

I would go by cosine similarity
If bigger than x display before caption results
locally with cosine similarity 0.27 was good score to match exactly the images I wanted to
It's hard for me to understand the structure of code at the moment
If I give you working functions you only have to call to receive OCR texts or vectors, could you add the feature then ?

Wow
I installed right now and it's pretty cool
With CLIP search + OCR I think it's competive to most proprioritary solutions
I like the face recognition solution, to train own modell

You can play around with this demo
https://helios-rkouye.vercel.app/
Pretty cool, powered by CLIP and elasticsearch

I don't have a lot of time at the moment, but you could start with the functions and then I could maybe add it in the future.

We now have that feature :)


Update Readme with command to view key

Thanks for the PR. I'm working on a more complete solution to this. I'll comment here when it is committed.

No worries, i can see the unreleased commit regarding make print-auth-keys in 1.4.3 now. I will close this.


Shortcuts not unregisted on minify event

Hello,
I was having an issue with the electron application Caprine posted here:
https://github.com/sindresorhus/caprine/issues/202
The issue was that f5  and ctrl+shift+c were not being unregistered when I minimized Caprine to the tray, so I could no longer use f5 to refresh pages in my browser; the key was still bound to Caprine.
If I brought Caprine out of the tray or killed the application, I regained control of the f5 key.
I have been told that the issue is likely with this package not unregistering the shortcuts on a minify event.
Is there something I can do to help confirm/fix this?

Thank you for the report @jamesrobb, I will check it soon

I implemented the logic for the minimize/restore and show/hide events in https://github.com/parro-it/electron-localshortcut/commit/fdd0fcde5655168444f1d3c86824c52f0589d537.
I have some doubt on what version should I publish with on npm. This could be considered a breaking change, since some app could be relying on a shortcut being working even with a minimized window.
What do you think?
/cc @sindresorhus

I doubt anyone depends on that, but just do a major release to be safe.

@marshallofsound does this potentially address the concerns you had with this module?

Just published v2.0.0 that implements the logic for the minimize/restore and show/hide events.
@marshallofsound, @zeke, what are these concerns you are talking about? Could I be of some help?

what are these concerns you are talking about?

See https://github.com/electron/electron/pull/8775/files/ba4a2d7c307cc5fe9e57aab898f6b619babc1f81#r103096101

Thank you @zeke, I think these changes could at least alleviate some of the issue he was talking about. But I'm not sure it can solve them completely. I have to test what happens when you register a shortcut already registered as global.

@zeke I tested the behavior of version 2.0.0 in regards of conflicting shortcuts.
Actually, if you register with electron-localshortcut a shortcut that is already registered by the OS or by another app, your shortcut take precedence on the other one.
But only when your window is not minified and has focus. When you minimize your window, the other shortcut is correctly restored.
Instead, if you listen to key event in the DOM, the shortcut of the other app take always precedence, even when your window is focused.
I think sometime this behavior could be desirable and sometime not. Anyway, I saw @MarshallOfSound was suggesting an alternative DOM implementation to solve the problem.
I'm open to implement it if there is interest about...

Thanks for following up @parro-it. What do you think about documenting this specific minimization/focus scenario in the README so folks are less likely to be caught by surprise when using electron-localshortcut?


Added URL Filter Module

Do Not Merge
As discussed in issue #112

Alright the main "removal of query params by rule" part seems like it's done :) I suppose the second big part of this module would be the resolving domains (or whatever identifies the page/site) to the rule sets. Currently it will just remove all query params apart from q on every site.
The original idea of a map of domains to rule sets is nice and simple, but the main challenge should be how to handle different TLD/ccTLDs for the same site. (i.e., both google.com and google.nl should be treated the same). Easiest way is making multiple entries in the map with the same set values for each of these, although it will obviously be more difficult to maintain. Any ideas?
Also I would put the call to normalizeUrl at the start of the function, to make sure to normalize it by removing useless things (in this context) like 'www' and hash fragment before attempting to parse it.

URL objects can return the hostname which seems to be good enough to serve as a key. We can probably extract just the domain part from it i.e. google from google.com can make that as a key.

challenge should be how to handle different TLD/ccTLDs for the same site

I think then it also makes sense to add /maps as a standard blacklist and not google.com/maps.

We can probably extract just the domain part from it i.e. google from google.com can make that as a key.

Any idea how to do this in a generic way? Domain name may work as key for google sites as we can make a fair assumption that there isn't going to be any differences in behaviour across these, but that assumption doesn't hold for general case. Been thinking about this a bit, but can't think of any real solution apart from adding separate entries in the rules collection 😕 must be some better solution...

I think then it also makes sense to add /maps as a standard blacklist and not google.com/maps.

This would block all the google maps sites, but also unintentionally block any site where the string /maps appears anywhere in the URL.  This is also a tricky one for the same reason as above.

I think let's keep it to be the absolute domain at this time. Maybe after some point we can get a dump of all the keys in the dictionary and see if we can find a pattern.
I also think some websites are not consistent across TLDs. So you can probably replace 'co.uk' in amazon.co.uk to 'com' and get the same page on US' site but I wouldn't assume it happens for every such website.
As for Google maps, let's add an absolute if condition in the Blacklist module?

I think let's keep it to be the absolute domain at this time. Maybe after some point we can get a dump of all the keys in the dictionary and see if we can find a pattern.

So you mean adding additional entries for each domain + TLD/ccTLD? I'm happy going that route for now, but you'll need to find a list of all the different google sites and there's probably going to be a lot... here's a non-exhaustive list I just found.
Other way around this would be having the keys as RegExp objects that we can test the current URL on. Big negative with this though is the algorithm's time then becomes linear to the size of the data collection (up from O(1) with a dict lookup), hence no real benefit of using a dictionary-like structure. I think having lots of entries per site is preferable to that in the long run. (maybe it could be reduxed to O(logN) with a sorted collection, assuming RegExp objects are sortable...)

As for Google maps, let's add an absolute if condition in the Blacklist module?

This warrants regex support in the blacklist rather than hardcoding the condition. Current implementation uses String.prototype.includes, but could just as easily work with RegExp.prototype.test with the current blacklist structure without changing existing behaviour. I'll add this on another branch later unless any concerns.

Yeah I meant a new key for every TLD and ccTLD. We can probably avoid this for some very obvious site like Google. But if that doesn't make the data structure consistent, we can have a new key for every ccTLD of Google too.
RegExp doesn't seem a viable option as you said, it just increases the execution time.

@jon is this mergeable now? Right now we don't have rules for specific sites so adding that dictionary structure won't be useful.

Yes this is a required part of this module. Currently if I bring this into the main flows it's going to remove every query param apart from "q" from every URL, which is not what we want. So far we have rules for fb, twitter, and all the different google sites which can all be made as entries (they just happen to be the same rules for now, but we will expand on them over time).
Preferable to put all this entries in a separate file that gets imported by this file and used to construct a Map. Then we have flexibility to move the data to whatever source later without needing to change much

On Nov 5, 2017, at 18:29, Arpit Gogia<EMAIL_ADDRESS>wrote:
@jon is this mergeable now? Right now we don't have rules for specific sites so adding that dictionary structure won't be useful.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or mute the thread.

Another pattern, that I consistently see, mostly on newspages, is this one: ?utm_source=kickbooster&utm_medium=redirect&utm_content=link&utm_campaign=f31ea6

also on amazon.com and all other amazon pages, everything after the ref= is useless to identifying the page.

@arpitgogia @poltak
Just got a hot tip from @bigbluehat on how hypothesis is doing their url identification/cleaning of url
https://github.com/hypothesis/h/blob/baebbd4617a92fd5a6ab91f6317eaa383da01893/h/util/uri.py#L79-L97

Wow amazing! I just tested the history analysis with our current url filter and I could reduce it by whopping 8000 documents!!!


Handle case of not found reset token

Yes, it's a hard issue to solve because it only happens when you run sequentially IT in command line (via Intellij it doesn't happen). It's not the first time I met this issue but I've managed to avoid it until now :)

Solved! I remembered that I just needed to not parallelize the tests :D

I see 👍


enable CloudTrail action

Another stab at #339. This will enable the trail(s) named in the policy, or create then enable them (and a dedicated bucket) if they don’t exist.

actions now require a permissions tuple describing the permissions they need.

This has been updated to support the various options.
Like I mentioned on the call, the coverage didn’t match my expectations. I ran through in the debugger and at least two of the lines that supposedly aren’t getting tested are getting called when tests run, so I’m chalking it up to weirdness in the coverage package and submitting this as is.

Updates based on the last discussion:

Moved the enable trail action to the account resource
Added an explicit call to describe_trails() since we’re now being passed a list of accounts instead of trails
Added DescribeTrails to the list of permissions
moved and updated tests

branch updated and pull request resubmitted in #1098


Polygon label on perimeter of feature

This modification introduces a label placement around the perimeter of a polygon feature. When the user employs this label placement method the code basically treats the outer ring of a polygon feature as a series of line segments and uses most of the polyline logic.
Implemented was a new variation of the rotation angle. The current code calculates it from the segment and uses ATAN. That works for all the calculations except when it comes to the inside/outside of a polygon since it loses the directionality. To account for that a new calculation was introduced with ATAN2 and called theta.
The polygon requires this angle with directionality so that we can determine which side is inside or outside the polygon. The only place theta is used is in the Position method and it doesn't get used for text rotation. All we need theta for is to determine which direction to offset the text relative to the segment. Basically logic to determine if the text is above or below the line.
The DotSpatial.Controls.MapLabelLayer is great code but could use a refactoring at some point. Of note is the multiple calls to the GetSegment method which could be needless overhead. I could envision that DotSpatial could be modified to allow different labeling engines much like a plugin.
For now I think this new label placement method is a good starting point for further work like optimal placement on the perimeter rather than along the first, middle, last, or longest segment. This can be seen in the sample project where the label is always placed on the longest segment. This is only in the UI as the developer can set them as they wish in code. This is due to LabelPlacementMethod and LineLabelPlacementMethod not being mutually exclusive. Perhaps a new SegmentLabelPlacementMethod would resolve this.
Fixes #1211
Checklist

[x] I have included examples or tests
[x] I have updated the change log
[x] I am listed in the CONTRIBUTORS file
[ ] I have cleaned up the commit history (use rebase and squash)

Changes proposed in this pull request:

Introduction of a label placement around the perimeter of a polygon feature

Closed this PR. Will wait until https://github.com/DotSpatial/DotSpatial/pull/1466 gets merged so that I can refactor the logic.


Should editor.suggest.filteredTypes have a deprecated message

VSCode Version: 1.40 Insiders
OS Version: Windows 10

Steps to Reproduce:

Bring over editor.suggest.filteredTypes from 1.39 settings and paste into 1.40 settings.json

editor.suggest.filteredTypes just shows faded with "Unknown Configuration Setting" hover.
Should we add a deprecated message something like "This setting is deprecated, please use 'editor.suggest.filterTypes.*' instead.". It would also be good if there was a warning squiggle in settings.json.

I am afraid that's not possible or at least I don't know how to proceed here... I will check with @sandy081 or @aeschli

Looking at workbench.tree.horizontalScrolling as an example, I think you leave the old setting and add a deprecationMessage.

Thanks @gregvanl

Adding the definition of that property caused a meltdown... The problem is that the old setting is a prefix of the new settings and our config model doesn't like that much.

The plan to rename these settings to editor.suggest.showXYZ. That avoid the conflict and help with clarity


Save elections

For this card.
The save elections form now requires both fields to be filled in before it enables the button to save it, it requires the user to select a date that is in the future, and creates an election in the database by submitting a request to the node backend.  We've also added migrations for the new database that will save dashboard-specific data.

@tie-rack I think we've now addressed everything if you want to take a look again when you get a chance. Thanks!


KubeJS error

When i installed the mod in ATM9 modpack, minecraft runs normally but shows me an error in kubejs mysticalagriculture:bort


proposal: x/exp/slices: add Pop

// Pop removes the elements s[i] from s, returning the modified slice
// and the removed element. Pop panics if s[i] is not a valid slice of s.
func Pop[S ~[]E, E any](s S, i int) (S, E) {
	e := s[i]
	return append(s[:i], s[i+1:]...), e
}

Can you write a doc comment for the proposed function?  Specifically, what happens when the slice is empty.
Seems odd to have Pop without Push.

@ianlancetaylor Like Delete but returns the removed element.

About Push maybe just like this.
// Push appends the element at s[:i], returning the modified slice.
func Push[S ~[]E, E any](s S, i int, e E) S {
	return append(append(s[:i], e), s[i+1:]...)
}

That Push is the same as Insert
Push/Pop seems like a strange name for the given actions, usually they mean operating at the ends of a queue

Maybe change the method name to Remove like Rust's vector?

I agree that Pop is a strange name for it, but Remove for parity with Insert makes sense to me.

What about this?
// Pop removes the element s[i] from s, returning the modified slice
// and the removed element. Pop panics if i is not valid index of s.
func Pop[S ~[]E, E any](s S, i int) (S, E) {
	switch i {
	case 0:
		return s[1:], s[0]
	case len(s) - 1:
		return s[:len(s)-1], s[len(s)-1]
	}
	return append(s[:i], s[i+1:]...), s[i]
}

// Remove removes the elements s[i:j] from s, returning the modified slice
// and the removed elements. Remove panics if s[i:j] is not a valid slice of s.
func Remove[S ~[]E, E any](s S, i, j int) (S, S) {
	var s2 S
	if n := j - i; cap(s) >= len(s)+n {
		// Use the free space to avoid an allocation.
		s2 = s[len(s) : len(s)+n]
	} else {
		s2 = make(S, n)
	}
	copy(s2, s[i:j])
	return append(s[:i], s[j:]...), s2
}

Python has dict.pop which works like this. I don't think this behavior really makes sense for a slice though. It's not a very substantial savings versus just doing:
e := s[i]
s = slices.Delete(s, i, i+1)

It saves you a line, but it adds noise to the docs and open questions about behavior, when really we should be discouraging using O(N) operations more than necessary.

Pop should not be an O(N) operation. Saving s[i] and calling Delete seems fine for this use case. Delete is a bit problematic already; better not to have two.

This proposal has been added to the active column of the proposals project
and will now be reviewed at the weekly proposal review meetings.
— rsc for the proposal review group

Go programs are supposed to handle such conditions explicitly instead of checking preconditions and react on failure to do so.

I agree "Go programs are supposed to handle such conditions explicitly" but I don't see how it follows that being explicit is "instead of checking preconditions". For map, if you couldn't do _, ok := m[k] how would you explicitly check that the map has a key? Comma ok is checking the precondition. For a slice, the equivalent check is if len(s) > i. There doesn't need to be comma ok version of slice access because we have if len(s) > i.
Anyway, Pop is O(N) and really should not be encouraged.

Based on the discussion above, this proposal seems like a likely decline.
— rsc for the proposal review group

No change in consensus, so declined.
— rsc for the proposal review group


hyperlink repomap

When browsing the repomap I often find myself wanting to click on a file
to view it in github. This implements a custom Sphinx role to insert
exactly those hyperlinks. It has two exceptions, if curly braces are
encountered, e.g. file.{c, h} two nodes will be inserted, one for each
file, including a separator between them. If a glob (*) is
encountered, a link to the directory is inserted.  Otherwise the text is
linked verbatim. This also has the nice side-effect of checking that all
files mentioned in the repomap really do exist. And in this case
appveyor.yml seems to no longer exist so it should probably be
removed. Switching to this role for all files in repomap is also done.

I updated the PR to include some more tweaks and remove the no-longer-existing file. It would be nice to have something that checks if all the files in the repository are listed in repomap, but that is something for a rainy day.

Looks good.  Just some very minor issues to fix and this is ready to merge.

@seibert @stuartarchibald thanks for the review! I have addressed all your comments and pushed the fixes.

Fixes for the comments I made look ok, over to you @seibert. Thanks @esc.

Looks fine.  Ready to merge.


Add Icons to Button component

Add startIcon and endIcon to Button component. They are optional props.
startIcon appears at the start of button's label, endIcon appears at the end of button's label.
IconVariant is also supported with startIcon and endIcon

Looks like there's conflicts after my ticket was merged in, I think there's some overlap in our IconVariant enums :)


Troubleshooting CI

Codecov Report

Merging #7017 (ad85d0c) into main (fcdc1e2) will increase coverage by 1%.
The diff coverage is n/a.

@@          Coverage Diff           @@
##            main   #7017    +/-   ##
======================================
+ Coverage     63%     64%    +1%
======================================
  Files        361     361
  Lines      22858   22858
  Branches    3431    3431
======================================
+ Hits       14582   14851   +269
+ Misses      7060    6705   -355
- Partials    1216    1302    +86

Impacted Files
Coverage Δ

src/client/common/cancellation.ts
72% <0%> (-4%)
:arrow_down:

src/client/common/process/pythonDaemonFactory.ts
80% <0%> (-4%)
:arrow_down:

.../datascience/notebook/notebookControllerManager.ts
84% <0%> (-2%)
:arrow_down:

src/client/common/process/baseDaemon.ts
51% <0%> (-1%)
:arrow_down:

...t/datascience/notebook/vscodeNotebookController.ts
79% <0%> (-1%)
:arrow_down:

src/client/common/utils/decorators.ts
77% <0%> (ø)

src/client/datascience/webviews/webviewViewHost.ts
78% <0%> (ø)

.../datascience/variablesView/variableViewProvider.ts
81% <0%> (ø)

src/client/datascience/baseJupyterSession.ts
63% <0%> (+<1%)
:arrow_up:

...rc/client/datascience/jupyter/debuggerVariables.ts
16% <0%> (+<1%)
:arrow_up:

... and 17 more


Can you provide the script file for IDMatcher in id_assigner?

Can you provide the script file for IDMatcher in id_assigner?
KeyError:'IdMatcher is not in the bbox_assigner registry'"'
I had this problem while changing my code, so want your IDMatcher file.
Thank you!

Thanks for your question.
Do you mean this file?
https://github.com/Jingkang50/OpenPSG/blob/4bb2e9316c656e2f170db5de778682089b7aca0f/openpsg/models/relation_heads/approaches/matcher.py#L129

This is it, thank you so much!


WIP: Add initial audittail implementation

audittail is a command line utility that helps one tail audit logs.
It'll create and open the necessary named pipe and ensure to re-open
if there's an error.
Signed-off-by: Juan Antonio Osorio<EMAIL_ADDRESS>

Codecov Report

Merging #7 (2285485) into main (2a9efaa) will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##              main        #7   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            4         7    +3
  Lines           83       149   +66
=========================================
+ Hits            83       149   +66

Flag
Coverage Δ

unittests
100.00% <100.00%> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

writer.go
100.00% <ø> (ø)

cmds/audittail/cmd/common.go
100.00% <100.00%> (ø)

cmds/audittail/cmd/init.go
100.00% <100.00%> (ø)

cmds/audittail/cmd/root.go
100.00% <100.00%> (ø)

ginaudit/mdw.go
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

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


Undo/redo doesn't work when using the inspector plugin

Undo and redo features don't work when items and inventories are edited via the inspector.
Godots UndoRedo helper could be used to manage undo/redo operations in the plugin.

Implemented in https://github.com/peter-kish/gloot/pull/28


Fix/share links with section

This PR adds the section hash to the URL to be consumed by the share button on Agriculture profile page.
Pivotal task

One thing that maybe can be fixed on this PR, the map for "Countries actions on their NDCs" has the share button of the original NDC component, and is copying the embed for that. Let's remove that embed link fro the Agriculture page, please!

@simaob btw. should we add buttons group to Countries’ Actions in their NDCs/Table section?
we don't have a design for that but probably buttons should be there, right?

Hmmm we probably should, but we don't have it here: https://climate-watch.vizzuality.com/ndcs-content/table either... so maybe let's leave it as is for now. It wasn't on the designs before, and they approved it.

thanks :+1:  btw2. I see we have this same case with embed links in Drivers of emissions/Future pathways section

Let's get rid of it there as well! =)

No more embed links ☝


Preview Pane does not automatically refresh on new selection

In the Zotero Beta 5.0.97-beta.46+4cfa23050, the preview pane does not change when selecting a different citation in the Title column. The Preview stays on the same citation, regardless of what other citation one clicks on, until one clicks on and off the Preview pane by selecting another pane such as Notes and the Preview pane again. It would be great if the Preview pane remained linked to the highlighted citation. Thanks.

Not even sure I knew there was a new version coming. I've not looked at that.

I'm running <IP_ADDRESS> (without issues). Not entirely sure how I'd test and install both.

Released a version to fix this issue. Works in 6.0 too.

It's not great in 5.x versions, but 6.0 is good.

Closing this as resolved in Zotero version 6.


passwords visible in preview

Hello,
I've found past issue (https://github.com/StackStorm/st2web/issues/411) that should have solved this, but it seems in 3.8.0 the preview leaks fields marked as secret:

Do I have to configure something in the st2.conf? Or is this by design?
BR,
Filip

This sounds like a bug indeed as secrets should be masked. Thanks for the report.
If someone is interested in contributing, the fix should be done in the st2 core which provides st2web with an API response.

Same issue while checking the past executions in the execution tab.

Same issue while checking the past executions in the execution tab.

can you post example screenshot? I see secrets properly masked in past execution outputs: