recipe_autoassess_stratosphere.yml not working

esmvalcore._task.DiagnosticError: Diagnostic script autoassess/autoassess_area_base.py failed with return code 1. See the log in /mnt/lustre02/work/bd0854/b309057/ESMValTool/v2_output/recipe_autoassess_stratosphere_20191030_095038/run/aa_strato/autoassess_strato_test_2/log.txt

The logfile contains the following traceback:
Traceback (most recent call last):
  File "/mnt/lustre01/pf/b/b309057/ESMValTool/public/esmvaltool/diag_scripts/autoassess/autoassess_area_base.py", line 440, in <module>
    run_area(config)
  File "/mnt/lustre01/pf/b/b309057/ESMValTool/public/esmvaltool/diag_scripts/autoassess/autoassess_area_base.py", line 413, in run_area
    metrics = metric_function(run_obj)
  File "/mnt/lustre01/pf/b/b309057/ESMValTool/public/esmvaltool/diag_scripts/autoassess/stratosphere/strat_metrics_1.py", line 604, in mainfunc
    qbo_metrics(run, ucube, metrics)
  File "/mnt/lustre01/pf/b/b309057/ESMValTool/public/esmvaltool/diag_scripts/autoassess/stratosphere/strat_metrics_1.py", line 337, in qbo_metrics
    (period, amp_west, amp_east) = calc_qbo_index(qbo30)
  File "/mnt/lustre01/pf/b/b309057/ESMValTool/public/esmvaltool/diag_scripts/autoassess/stratosphere/strat_metrics_1.py", line 162, in calc_qbo_index
    if indiciesdown[0] < indiciesup[0]:
IndexError: list index out of range

Cheers @mattiarighi, I'll have a looksee today :beer:
Dr Valeriu Predoi.
Computational scientist
NCAS-CMS
University of Reading
Department of Meteorology
Reading RG6 6BB
United Kingdom
On Wed, 30 Oct 2019, 09:57 Mattia Righi<EMAIL_ADDRESS>wrote:

Assigned #1414 https://github.com/ESMValGroup/ESMValTool/issues/1414 to
@valeriupredoi https://github.com/valeriupredoi.
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
https://github.com/ESMValGroup/ESMValTool/issues/1414?email_source=notifications&email_token=AG5EFI7E7V425ZQHWLCIISDQRFLAPA5CNFSM4JGWMUU2YY3PNVWWK3TUL52HS4DFWZEXG43VMVCXMZLOORHG65DJMZUWGYLUNFXW5KTDN5WW2ZLOORPWSZGOUQ5F3VY#event-2755288535,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AG5EFI3IF4IS5A6IBFA6UYDQRFLAPANCNFSM4JGWMUUQ
.


onIndexChange not called when i swipe to page at index 0

render() {
if (!this.state.navigation) return Loading;
return (
  <View style={{ flex: 1, backgroundColor: 'black' }}>
    {!this.state.isExpanded && this.renderToolBar()}
    <TabViewAnimated
      style={AppStyles.flex1}
      renderScene={this.renderScene}
      renderHeader={this.renderHeader}
      navigationState={this.state.navigation}
      onIndexChange={this.handleIndexChange}
    />
    {this.renderInfoView()}
  </View>
);

}
handleIndexChange = index => {
console.log(index change ${index})
this.setState({
selectedIndex: index,
})
}
handleIndexChange not called when i swipe to page at index 0. Please help me. Many thanks !

I can't reproduce that issue. Can you provide a minimal/small example that illustrates the bug?

@kpsroka The example is for the swipe event, not clicking, did you try specific swiping?

@djalmaaraujo Yes, I tried swiping. I used the example app from react-native-tab-view (running on Expo), which uses TabViewAnimated in a similar way to what @mtr1012 described. I suppose there's some interaction between TabViewAnimated and the parent View that's causing the problem, but without a piece of code that I can reproduce the issue with, I can't help.

I have the same problem, onIndexChange not called at index 0.

I am also facing this issue. onIndexChange gets called for second and third tab but not for the first tab
<TabView           navigationState={this.state}           renderScene={this.renderScene}           onIndexChange={this.onChangeTab}           renderTabBar={(props) => (             <TabBar               {...props}               indicatorStyle={{ backgroundColor: '#6BAC1B' }}               style={{ backgroundColor: 'white', height: 40 }}               renderLabel={this.renderLabel}               indicatorStyle={{ backgroundColor: '#6BAC1B' }}             />           )}         />


#94: implement stelem.i1 - stelem.r8 MSIL opcodes + #95 fix numerical conversion of float types

A bit of refactoring as well, around the prepare method, to introduce a similar utility function as can be found around the ldelem implementation.
Also fix numerical conversion of float types.

Thank you send fixing PR, I will reveiew it within 2 or 3 days, plz wait.

@cyborgyn @pengzeshan I added test code for stelem and conv opcodes. Please notify me when verify on your environment.

Thank you for the addition! A very nice set of new tests! All green on my side 1492/1492. :) Took a while to run all of them, though... :)

Finally testing now, and will merge detect no problems on any test.


WonderTheme doesn t recompose when darkTheme change

Describe the bug
When I try to dynamically switch between light and dark themes using WonderTheme in my app, the theme's visual elements don't update correctly. The text changes as expected, but the theme colors (containers and text) do not recompose accordingly. This works perfectly with AppTheme, but not with WonderTheme.
To Reproduce
Implement a compose app using WonderTheme and a variable isDarkTheme (as a mutableStateOf to produce recomposition).
Update the value of isDarkTheme via an event (e.g., by clicking a button).
Notice that while the text changes correctly, the theme colors do not update.
A snippet for reproduce :
var isDarkTheme by remember { mutableStateOf(true) }
WonderTheme(darkTheme = isDarkTheme) {
    Column(modifier = Modifier.background(VitaminPlay.colors.containerNeutral)) {
        Text(
            text = "Current theme: ${if (isDarkTheme) "Dark" else "Light"}",
            color = VitaminPlay.colors.contentNeutral,
            modifier = Modifier
                .padding(16.dp)
                .background(VitaminPlay.colors.containerNeutral)
        )
        Text(
            text = "DARK",
            Modifier
                .background(Color.Red)
                .clickable { isDarkTheme = true })
        Text(
            text = "LIGHT",
            Modifier
                .background(Color.Red)
                .clickable { isDarkTheme = false })
    }
}

Expected behavior
WonderTheme should switch DARK <-> LIGHT when switching between dark and light themes

Thanks @dgaillard2 for your issue, We'll have a look a this soon. I just transfer this issue to the right repository and close this one :)


New lesson: big picture

This is a tracking issue for making a new (short) lesson which summarieses the big picture.  It will follow the backwards lesson design.  The original post text will be edited with the latest design, so you won't get many notifications.  Anyone who can edit this text, feel free to do so directly.
Brainstorming: This is a "first lesson" which motivates people that our tools are important.  It could be given at the start of a CR workshop, but also independently to motivate research to take a pause and learn some stuff.  Target duration: something that can be presented in 15 minutes, with links to more advanced stuff and possibly optional parts that can be used to make the talk 30-60 minutes and include some content.
Learner profiles:

Alpha is starting off at a coderefinery workshop.  They are already coming, but don't quite know what to expect and not fully sure why they should care and how they will use the lessons.
Bravo is a new researcher at an "introduction to research" type seminar.  They are very excited, but don't know where to start.  They think it's all about science and probably don't realize that computing skills are very important.
Charlie is a new-ish researcher in a group.  They've been doing computational research for a while, but haven't picked up on these skills yet.  They are at a group workshop/hackathon and will listen to an inspiring presentation about why they should take a pause and build up their background knowledge some.

Summative assessment:

Why are good programming tools important to computational researchers?
What are the typical barriers to adopting good tools?
What do the following terms mean: version control, automated testing, modularity, etc.

For each, what benefits does it give?
For each, what are the different levels of adoption (e.g. reasonable levels for self-use, small teams, community)

What tools should you focus on for your work?

Formative assessments :

TBD, seems to roughly be in same order as summative assessment.

Outline (slides/sections):

Hook anecdote: have you ever spent weeks traking down the smallest bug?  Or had to spend longer to re-run your code for the revisions than for the original paper?
What is scicomp and how does it differ from (normal programming) and (the science)
Code maturity pyramid: you can have some messy code, but it needs to build on better code
What does good research code look like?

VC
tests
documented
etc, one slide per topic.  Each slide lists: what it is, why it's important, when to use it, link to more info.

This is now 3 years old. These are good ideas but maybe we already have such an overview? If not, should we drop it and admit that we don't have time for it right now?

This could have some potential if we reformat the workshop to be more spread in time.

Issue is now 5 years old and our lessons have evolved. We now also have CodeRefinery in half a day, the mini-workshop, and presentations on this topic. I will close this since I believe we have this. What we have is not perfect but maybe we evolve what we have? Please reopen if you disagree and would like to follow-up on this more.


New version: Sundials_jll v6.7.0+0

Autogenerated JLL package registration

Registering JLL package Sundials_jll.jl
Repository: https://github.com/JuliaBinaryWrappers/Sundials_jll.jl
Version: v6.7.0+0
Commit: c9be19e45cb3af4f4a5ef4f160c6e41ba9d7f484
Revision on Yggdrasil: https://github.com/JuliaPackaging/Yggdrasil/commit/7ca1142489cd4bb1cf745645c8a8c09104919056
Created by: @staticfloat

[noblock]
Looks like there is a merge conflict.
@staticfloat  Can you re-trigger a registration in Ygg? This will fix the merge conflict.

[noblock]
It's not so easy: https://buildkite.com/julialang/yggdrasil/builds/8277#018d8ac9-3c6f-42aa-9668-e945d5931a7f

[noblock]
Can you delete the old release assets?

That seems quite drastic, why not just resolve the conflicts here?


Add shields/badges to README.md

Shields or badges allow you to show the state of a project to users. Sometimes they also show how much effort and dedication are put into the project. For now, we want to show the python version (3.9 and above), license (MIT), project version, and commit activity.

@huenique please assign this issue to me, I wanna work on this.

@avinash-tiwari sure, go ahead!


TypeError: Unexpected keyword argument 'message_content'

When using keyword arguments like this:
aadiscordbot.tasks.send_channel_message_by_discord_id.delay(
    channel_id=rss_feed.discord_channel.channel,
    message_content=discord_message,
    embed=False,
)

A TypeError exception is raised:
Traceback (most recent call last):
  File "/mnt/sda1/Development/Python/AllianceAuth/allianceauth-discordbot/aadiscordbot/bot_tasks.py", line 38, in run_tasks
    await task(bot, *args, **kwargs)

Despite message_content being the right keyword.
Further investigation resulted in the following:

In all tasks (aadiscordbot/tasks.py) message_content is used as keyword
In the called functions (aadiscordot/bot_tasks.py) message is used as keyword for all API functions

Yep, looks like that was all shifted to message not message_content but those tasks fake-tasks werent updated
the send_message() helper function is passing the right values and is suggested way of queueing messages to be send now.
I'll have a look at fixing those fake-tasks, but the argument keyword will need to change to message , breaking any use of them, though they havent worked anyway for 13 months since the rewrite and send_message() was introduced

Ah, I see.
In that case, ignore this one :-)


Linux x265 support?

I've read the other ticket regarding supporting x265 on linux. However I came across another build that does manage to support x265 on linux.
https://github.com/macchrome/linchrome/releases/tag/v103.0.5060.68-r1002911-portable-ungoogled-Lin64
it requires the --enable-clear-hevc-for-testing command line argument, but it does work.
I've also tried the same command line argument with thorium without success.
Just thought this might be an interesting development.

@lovelytwo I can confirm this. Tried that build, and tried my latest build, with both cmdline patches applied. Ungoogled h265 worked, in thorium it didn't.
I can't find any h265 patches in ungoogled chromium.
Perhaps this has something to do with my vaapi/vdpau patches? I got them from debian, so im going to run debians latest chromium and see if h265 works in it or not.

@lovelytwo Debians Chromium doesn't work with h265.
So it looks like I can either have it play h265 content, or have hardware acceleration for other content on more systems.

@lovelytwo Adding to BUGS.md

@lovelytwo Turns out Thorium DOES support H.265, but only if you have hardware acceleration. I.E. your OS, GPU, and graphics driver all have to support it. Ungoogled Chromium is using a patch to enable software HEVC decoding. I will look into it.


Fix Text not appearing on the image

when the preview got moved around, the text from the input fields also stopped appearing. this needs to be fixed

Download
https://www.dropbox.com/scl/fi/ku9a1wblqyb84rb8ekase/fix.zip?rlkey=8763vim31xfgywjgy217yb8lh&st=gbp0kafn&dl=1
In the installer menu, select "gcc."

Download
https://www.dropbox.com/scl/fi/ku9a1wblqyb84rb8ekase/fix.zip?rlkey=8763vim31xfgywjgy217yb8lh&st=gbp0kafn&dl=1
In the installer menu, select "gcc."

Fixed.
Issue was caused by text locations being pixel based. when image was resized to be smaller, text would appear off canvas since pixel size was now too large for image.
Refactored the code so that pixel location is now expressed as a fraction (percentage) of the canvas size which can then be converted back to the pixel locations based on the image width and height. This will allow the text stay in its correct location with any possible sizing of the canvas. This could potentially break if the aspect ratio changes, if so the text would need to be stretched and in that scenario it would need to be fixed to regular aspect ratio anyways since that wouldn't look proper for the invite card


When failing to read a file say which one.

I've run into the problem a couple times when doing mass conversions where a file will have something wrong in the frontmatter and I'll get an error, but don't know which file has the problem.
This ensures that if reading of a file fails you know which file it failed on.

@ianstormtaylor  This should be ready to land. Let me know if there is anything else that needs to be fixed.

@ianstormtaylor  This should be ready to land. Let me know if there is anything else that needs to be fixed.

Ah shoot sorry I missed the followup. Thanks for all the help!


[#177] Handle text children that are of type number, boolean, etc

Fix the issue where text children that are not of type string did not get mapped to TextVNode.
Context
See Issue #177.

Thanks for your contribution 👏🏼 This one makes a lot of sense.

You're welcome, Angel.


[android] drawer menu is not closed on back button

Open drawer menu
Press the back button
Is:
Either the close app dialog appears if there is no stack or there is a fragment transition under the drawer with drawer menu getting out of sync
Expected:
Hide drawer menu

Thank you for the initiative!


fix(config): remove network_tokenization_service env from docker_compose.toml

…ompose.toml
Type of Change

[x] Bugfix
[ ] New feature
[ ] Enhancement
[ ] Refactoring
[ ] Dependency updates
[ ] Documentation
[ ] CI/CD

Description

remove network_tokenization_service env from docker_compose.toml
Additional Changes

[ ] This PR modifies the API contract
[ ] This PR modifies the database schema
[ ] This PR modifies application configuration/environment variables

Motivation and Context

How did you test it?

Checklist

[x] I formatted the code cargo +nightly fmt --all
[ ] I addressed lints thrown by cargo clippy
[ ] I reviewed the submitted code
[ ] I added unit tests for my changes where possible

Closing this pr, since this issue has been fixed in this pr - #5953

Closing this pr, since this issue has been fixed in this pr - https://github.com/juspay/hyperswitch/pull/5953


Notepad Cell

The ability to define a cell as a free text notepad to write notes. It may support Markdown but doesn't need the whole WYSIWYG support and overhead.

I'm reluctant to store text in a format that can only be read by GM Screen. I can see allowing an "editable" journal entry toggle though. The issues with that are the obvious "tinyMCE does not like small spaces."

Understood, but it would be most useful to DM to have a quick, expendable notepad with fast writing notes or conditions or things that can't be supported by mods or systems. It doesn't need the overhead of TinyMCE, and it can be an editable notepad that writes every X seconds into a Journal.

That's fair. I'd say this is a reasonable Dyanamic Cell request. Truly the plan for Dynamic Cells is for them to basically just be Applications rendered without the popout styles inside a grid cell. So in theory any "Dynamic Cell" can survive on its own without GM screen.

 I am confident in saying they will never be a feature of GM Screen.


Can't start wsl from run window

Windows Version
Microsoft Windows [Version 10.0.19045.2846]
WSL Version
<IP_ADDRESS>
Are you using WSL 1 or WSL 2?

[x] WSL 2
[ ] WSL 1

Kernel Version
<IP_ADDRESS>
Distro Version
Ubuntu 20.04
Other Software
No response
Repro Steps
open Run window (win+r)
type wsl, press enter
Expected Behavior
Launch wsl successfully
I was messing around with conhost.exe (mainly security tab to transfer ownership to myself but i've since returned it to TrustedInstaller)
Would appreciate any tips on how to debug this issue
WSL starts fine from cmd
Actual Behavior
got error (Application was unable to start correctly (0xc0000022))
Diagnostic Logs
No response

/logs

WslLogs-2023-04-26_00-25-54.zip

Did you let the tracing collect logs while reproducing the issue? I don't see much in there.

Yes, the issue is that wsl doesn't start at all when trying to start from Run window and gives 0xc0000022 error

Every other app I tried works fine

I didn't read this fully. If you changed the app execution alias permissions and haven't restored them completely it is likely to not work.

The error you're getting is the NTSTATUS constant for access denied. I'm guessing that when playing around with the ACLs you forgot to re-add yourself, or Authenticated Users etc, with read/execute permissions. Because when you made yourself owner, you most likely broke the inheritance, meaning when you gave up ownership you may not have any permissions at all :)

Thanks for the insight, I will try again with permissions!

Also is there a way what file exactly gives access denied error? that would help a lot

It's possible that it gets recorded in the application, system or security logs that you can view in Event Viewer, but it's far from certain.
The fastest way to repair this would probably be to start an elevated cmd.exe (if you can :P) and run icacls.exe c:\windows\system32 /reset /t /c /l, assuming you have your installation there.

just tried that, didn't work

How did it fail? Were you able to launch an elevated console or did you get access denied? Did you get cmd.exe running as admin, but got access denied when attempting to reset permissions? Did everything appear to work, only to not have any effect?

I fixed it, apparently i also messed up WindowsApp permissions.
I returned them to normal and everything works fine now
Thanks for the help!


Consider unsealing the Point class

Edit: I guess I should say "convert from struct to class". My editor said it was "sealed" but I checked the code and it's a struct. Maybe you have compelling perf reasons to keep it a struct.

I was very surprised that I couldn't subclass the Point class. I know I can use composition instead. I'm using an ECS pattern where components implement an IComponent interface (for serialization). Some of the components are just points with a different name. LocationComponent and DestinationComponent for example. It would be a lot cleaner for them to inherit point so I can do things like subtract them.

var delta = location - destination
entity.AddComponent(delta);

V.S.

var delta = location.point - destination.point
var deltaComponent = new DeltaComponent(){point = delta};
entity.AddComponent(deltaComponent)

Thank you for the feedback. It was originally a class but got converted to a struct in this commit:
https://github.com/FaronBracy/RogueSharp/commit/87b1568c09005580cb4be9bbbd67f380b11e0822#diff-336d9b5ef45c6e307febd568f0845ed0
People generally prefer it a struct for "performance" reasons, but in any of my performance profiling I haven't seen a big difference either way. I feel like it's a situation where it won't ever please everyone. I'll need to think about and research it some more before changing it back.
In the meantime could a potential workaround be to override the - operator on your component classes that are composed of points?

If it's not a problem in the profiler, it's not a problem! Anyone who says differently is guilty of premature optimization IMO. In the mean time I wrote a wrapper class so I can make things points and hand them to RogueSharp and back easily. It's not so much that I needed the - operator as I wanted to be able to interchange the points easily.

I would much rather prefer an interface which has X and Y. This can then be used by Point, Actor, Cell, Behaviour, Item, etc, etc.
It would also be the input to all these methods that takes int x, int y. It greatly improves the extensibility and also gets rid of a lot of code.

I suppose a record or a ValueTuple could be used, too?


Gas limit estimation on xDAI is not working

Description
Part of #2998
Implement a workaround for the problem with gas estimations for xDAI.
This error affects to all operations in xDAO, both for Dapps using Wallet Connect, and Gnosis Safe Apps.
More context on the issue: https://gnosisinc.slack.com/archives/CQB729LLF/p1635940014445400

🚨 NOTE: This issue is very important to us, so it is a blocker to launch the Gnosis Safe in xDAI network. If Gnosis Safe team solves this issue, we could release as planned next week this new integration. If not, we would need to invest some time in disabling Gnosis Safe for xDAI for CowSwap users.

Additionally, this is somewhat related to #3002
Environment

Network: xDAI

Steps to reproduce

Connect to xDAI
Open SushiSwap App
Try to wrap ETH, when the last signer tries to execute the transaction, they will see a message saying something like: "this transaction will revert, please reject it to save some gas costs". However, if you try to manually edit the gas limit, then it will say it doesn't revert anymore.

Expected result
I believe the issue is because Ethermine has issues estimating the required gas (https://github.com/NethermindEth/nethermind/issues/3577), however I believe while they don't fix the issue, the web could try to overestimate the gas if the web observes that it reverts, and see if it doesn't revert with that overestimation.
This work-around could be reverted after Ethermine solution, and be even be only triggered in xDAI (so its more scoped).
Obtained result
Users think the transaction will revert, and they are encouraged to reject the transactions.
Screenshots

Tried a few tx and they worked with the gas estimated by default:
Tx builder
Contract interaction
Removing/adding owner
Send funds
Create a spending limit
Issue:
The safe creation message shows that you will need a certain amount of funds to create the safe, but in MM is like 1/3 of what it was shown:

Fixed ✅
Apparently, it's a long-standing bug: we weren't passing the estimated gas price to the wallet on Safe Creation. We didn't notice before because the wallet's own and our estimations were similar.

Looks good now.

Nice! is there an URL where i can test this?
Is there an approximated date to release this to production? Just to coordinate on our end, we will need to re-enable xDAI on our end when that happens.

Nice! is there an URL where i can test this? Is there an approximated date to release this to production? Just to coordinate on our end, we will need to re-enable xDAI on our end when that happens.

https://safe-team.dev.gnosisdev.com/app

@anxolin we're planning to go live with it on Dec 1st.

Okay, cool, we went then live with xDAI disabled for Gnosis Safe, only Rinkeby and Mainnet are available: https://cowswap.exchange/#/
I'll set the date in the calendar then to enable xDAI. Let us know if there's change of plans. Thanks @katspaugh!


Fix example commands in HelloWorld.html

Fixed the mv commands that was listed as HelloWorld.hpp-template and HelloWorld.cpp-template. The actual files generated were HelloWorld.template.hpp  and HelloWorld.template.cpp.
Changed the example where it uses the private keyword in HelloWorld.hpp to uppercase to make it match what is in the generated code.

Sorry for the late review, thank you for making this change!


ms5611: cope with early Pixracer prototypes

ms5611 was on sensors bus in earlier prototypes

What's the reason for giving the MS5611 a different device name in this case?

Should we maybe phase these early prototypes out instead? They also have no support for FRAM.

There were not many of these in the wild, so let's not confuse people. Please phase them out.

I think that makes more sense, closing.


Cannot use load_from_checkpoint in an offline environment

Motivation
Due to the policy of my company, I cannot access to an online server (e.g., huggingface).
So I tried to use load_from_checkpoint function after I manually downloaded wmt22-comet-da checkpoint.
However, it makes an SSL error "SSLError: HTTPSConnectionPool(host='huggingface.co', port=443): Max retries exceeded".
This error is raised because of missing local_files_only param to each model's from_pretrained method.
🚀 Feature
I think COMET needs to support local_files_only parameter to load_from_checkpoint function not to raise http connection error.
Below is my current solution to the problem.

Add local_files_only param to load_from_checkpoint.
Pass it to str2model value classes' load_from_checkpoint(https://github.com/Unbabel/COMET/blob/master/comet/models/__init__.py#L88)

it automatically passes the param to model instantiation through LightningModule's load_from_checkpoint using kwargs.

Add local_files_only param to CometModel.__init__() (https://github.com/Unbabel/COMET/blob/master/comet/models/base.py#L94)
Pass it to str2encoder value classes from_pretrained (https://github.com/Unbabel/COMET/blob/master/comet/models/base.py#L119)
Add local_files_only param to each encoder classes' __init__ such as

self.model = XLMRobertaModel(
    XLMRobertaConfig.from_pretrained(pretrained_model, local_files_only=local_files_only),
    add_pooling_layer=False,
)

Can I make a PR?
Please let me know if there is better way.

yep, this seems good to me!

I made a PR for this issue, please check it :)
Additionally, I think it is also needed to add cache_dir param too. (If you thinks it's okay, I will make another PR for this)

Hi @zzaebok thanks for the PR!
I am on vocations and going to ACL next week so it might take me some time but I will look at it ASAP.

Hi @zzaebok thanks for the PR!
I am on vocations and going to ACL next week so it might take me some time but I will look at it ASAP.

any updates?

i need this update as well :)

do i simply place the downloaded (eg. xlm-roberta-large) and place it on the parent folder?

do i simply place the downloaded (eg. xlm-roberta-large) and place it on the parent folder?

I modified the local comet configs hparam.yaml, where there is a path for the infoxlm ckpt path (pretrained_model). Then switch on the reload_hparams during comet loading:
 comet_scorer = comet.load_from_checkpoint(args.comet_ckpt, reload_hparams=True)

@vergilus appreciate your feedback 👍   working now :)


Android: Get a wrong event from onForegroundEvent

Thanks for a great library., it is very helpful for us! But I found a problem, can you look into it?
Problem
Get a wrong event from onForegroundEvent
Environment
Only on android
Steps to reproduce

Launch application
Kill application from task history
Receive push "A" and display notification
Launch application from app icon(not from push)
Kill application from task history
Receive push "B" and display notification(Both "A" and "B" are exist in notification drawer)
Tap notification "A"
-> The argument from onForegroundEvent is the event of "B"

Cause
Pending intent has been overwritten, because unique id was same.
uniequeId for getActivity always starts with 0 after each app launched.
https://github.com/invertase/notifee/blob/58dfa45848f492110f0a22e699f6df0d829ca403/android/src/main/java/app/notifee/core/NotificationPendingIntent.java#L32
So uniequeId in steps to reproduce: 3 is 0. And uniequeId in steps to reproduce: 6 is also 0. As a result, the PendingIntent of "A" will be overwritten by PendingIntent of "B".
Solution
I have two solutions. Both are not perfect, but better than now.
A. Use random initial value
https://github.com/invertase/notifee/blob/58dfa45848f492110f0a22e699f6df0d829ca403/android/src/main/java/app/notifee/core/NotificationPendingIntent.java#L32
private static final AtomicInteger uniqueIds = new AtomicInteger((new Random()).nextInt());

B. Use hash for uniequeInt
https://github.com/invertase/notifee/blob/58dfa45848f492110f0a22e699f6df0d829ca403/android/src/main/java/app/notifee/core/NotificationPendingIntent.java#L69
int uniqueInt = notificationId;

From the user's perspective
I think this problem occurs in the below use case from the user's perspective.

User starts some game which uses a lot of memory
Android kills our application
User Receives our push "A", and our application launch and display push "A"
User keeps playing game which uses a lot of memory
Android kills our application
User Receives our push "B", and our application launch and display push "B"
-> PendingIntent of "A" is overwritten by "B"

Additional information
Perhaps https://github.com/invertase/notifee/issues/637 is a same problem

Please feel free to ask me, if you'd like to know any other information. Thanks!

@helenaford
Let me know your opinion?

should be fixed in the next release


Keep updated npm dist-tags

Bug, feature request, or proposal:
Proposal
What is the expected behavior?
Keep updated npm dist-tags.
Currently would be:
{
  "dist-tags": {
    "experimental": "5.0.0-rc.1",
    "latest": "5.0.0-rc.1",
    "next": "5.0.0-rc.1"
  }
}

What is the current behavior?
{
  "dist-tags": {
    "experimental": "2.0.0-alpha.11-2",
    "latest": "5.0.0-rc.1",
    "next": "5.0.0-rc0"
  }
}

Running npm update, 5.0.0-rc0 is installed instead of 5.0.0-rc.1.
Which versions of Angular, Material, OS, TypeScript, browsers are affected?
@angular/material 5.0.0-rc.1

Deleted the next and experimental tags


strong mode and hot reload

Some currently accepted reloads might violate strong mode heap safety guarantees, e.g.
class A {
  String a;
  A(this.a);
}

final a = new A("10");

String bar(String b) => a + b;

can be reloaded as
class A {
  int a;
  A(this.a);
}

final a = new A(10);

int bar(int b) => a + b;

after which we have an instance of A in the heap which contains String in field a but should contain an int.
We need to

[ ] Classify potential violations of heap safety
[ ] Cover these violations with tests to illustrate an issue
[ ] Decide which ones are accepted (because they are common) and which ones should be rejected.
[ ] Devise strategy for reconciling these violations with strong mode based optimizations (either by disabling optimizations or augmenting them to cope)

/cc @a-siva @rmacnak-google @alexmarkov

So I understand better - why does having stack frames which refer to old-structured objects violate strong mode?

after which we have an instance of A in the heap which contains String in field a but should contain an int.

Isn't the premise of hot-reload not to update live objects, but only update class definitions so that only objects created from that point on will have updated structure?

So I understand better - why does having stack frames which refer to old-structured objects violate strong mode?

Actually just having old stack frames violates strong mode - imagine we do:
void foo() {
  bar("10");
  bar("10");
}

void bar(String x) {
  print(x.length);
  debugger();  // *
}

and then we reload it while it is paused on * at the first invocation with
void foo() {
  bar(10);
  bar(10);
}

void bar(int x) {
  print(x + 1);
  debugger();  // *
}

and resume then we will end up invoking bar(...) with a string - which should never happen in the correctly typed strong mode program.

but only update class definitions so that only objects created from that point on will have updated structure?

Yep, but that means that live objects might become inconsistent with their definitions (e.g. field types don't match actual field values).
The more I think about it the more convinced I am that we are just forced to disable all static strong mode based optimizations when hot reload is possible.

Actually just having old stack frames violates strong mode - imagine we do...then we will end up invoking bar(...) with a string

Nice example. Somehow I don't see this as a violation of a strong mode. Yes, making such change would result in runtime exception, but that is as expected considering live frame and change in method signature that user requested.

The more I think about it the more convinced I am that we are just forced to disable all static strong mode based optimizations when hot reload is possible.

Yes, it seems rather hard to identify and rollback (all transitive) optimizations that would be invalidated by given hot reload.
Arguably, though, without those optimization, isn't performance of the "debug"(jit) app is similar to dart 1 checked mode performance(what Flutter used to run in, for example).

Yes, making such change would result in runtime exception, but that is as expected considering live frame and change in method signature that user requested.

Yes, but well typed static invocations are not expected to cause runtime exceptions in strong mode under normal conditions. (/cc @leafpetersen )
But of course hot reload is not a normal condition.

Arguably, though, without those optimization, isn't performance of the "debug"(jit) app is similar to dart 1 checked mode performance(what Flutter used to run in, for example).

Close, but probably worse - because Dart 1 did not reify generic methods meaning that type tests were missing in those places. Also there were no implicit down casts which exist to guarantee static type safety.
I guess we need to start by adding a flag that turns on to true when hot reload is possible and otherwise turned off.
Then we can go around and disable optimizations that require global well typedness.
We might also need to start benchmarking this mode on golem - like we did with checked mode.
My concern here is that this significantly increases complexity of our optimizations because we know need to account for this very carefully in our mental models of the code.

guess we need to start by adding a flag that turns on to true when hot reload is possible and otherwise turned off.

There is Isolate::CanReload() https://github.com/dart-lang/sdk/blob/master/runtime/vm/isolate.cc#L1236

Isolate::CanReload() does not actually give us anything useful - it will usually be true even when you are running a batch mode dart2js.

Not sure I follow - you can connect to dart2js running in batch mode via observatory and request hot reload.

As a developer who writes Dart-VM based tools I am always very surprised to find out my code is running slower/less optimally in order to support either Observatory, Hot Reload, or other debug-only features. I don't know if we need an explicit dart --debug, but maybe we do?

This issue has been a P1 since 2018, I feel it should be a P2 as we evaluate further and fix the issues.

I believe this was largely addressed by @rmacnak-google

https://github.com/dart-lang/sdk/commit/a8b3773054c283ac9057522c491a1f9f7a4fe68e#diff-1d5b5a2785a3bb4ca902e997a42a3d93 by adding runtime type checks when needed.


Version of numpy to use the library

Thank you so much for your excellent work! I would like to use nlp library in my project. While importing nlp, I am receiving the following error AttributeError: module 'numpy.random' has no attribute 'Generator' Numpy version in my project is 1.16.0. May I learn which numpy version is used for the nlp library.
Thanks in advance.

Seems like this method was added in 1.17. I'll add a requirement on this.

Thank you so much. After upgrading the numpy library, it worked.


Improved size calculation method

Switch case is not good

I don't think this is really necessary. Gonna close to clean up old PRs. Sorry :/


Handle Promises as output properties

When a promise is returned from a reactive function callback, the output property should be

immediately set to a special value ReactiveModel.PENDING
set to the resolved value when the Promise is resolved.

Wondering if you have thought about how multiple promises will apply in such a case? Specifically, should the promise resolutions be applied in FIFO request-order or FIFO resolution-order?

Interesting thoughts, thank you for your input.
I haven't yet tackled the promises implementation, but intuitively I would think it makes the most sense to apply the resolutions in FIFO request order, not FIFO resolution-order. Consider the case of changing the path for a CSV file several times. The last file name set should be the one loaded by the promise, even if a request for a previous file name takes longer to load.
I'm not sure about the caching issue, because digest() is synchronous. The only way to morph the values in the dependency graph is to make a change to it inside a reactive function (by calling a setter function). Users of the library should not do this, but there is no strict enforcement that disallows it. Perhaps rather than caching, setter functions could throw errors when invoked within reactive functions.

Agreed that FIFO request order makes most intuitive sense. If that is the case, should the first request be prevented from continuing on resolution?
On the caching, consider this case:

reactiveModel.promiseGeneratingEvent('some arguments')
This initiates a promise and is in ReactiveModel.PENDING state.
So the dependents have not been exercised yet.

  ... changes some dependency
 })```

Promise resolves, will have to re-evaluate the graph (since it's a new event), but the graph is not what it was when the Promise was evoked.

Admittedly this is not "normal" usage, but I can imagine using dynamic dependencies for various reasons (some complex forms of A/B testing might be one).
A possible approach:
If the react() caches the results of the graph, perhaps on a per node basis (only the outputs have to be cached?). Much like immutable.js, these can be stored by version. When the promise is evoked, one can store a version# of the cache, register a pending request.
Until all pending promises are resolved, the cache keeps the older versions around. The last version dependent promise cleansup after itself - possibly resetting the version number if that is safe.

Indeed, if any input properties change and a promise is pending, its eventual resolution should be ignored. A new promise should be created, and only the resolution of the newly created promise should have an effect.
Ah I see now what you mean - if the structure of the dependency graph is mutated before a promise is resolved, what happens? I was thinking that in the case of promises, a call to digest() will initiate the promise, and set the output property of its corresponding reactive function to ReactiveModel.PENDING. Any reactive functions that depend on the output value would be invoked at this time, with the value ReactiveModel.PENDING (so they would need to check for that case). This would be the end of a complete digest cycle.
After the promise resolves, the output property of its corresponding reactive function would be set, and this would trigger a completely new digest cycle to occur. Because this is a separate digest cycle, independent from the first one that initiated the promise, there should be no problems caused by mutating the dependency graph between the two digest cycles. The two digests would occur on the two different graph structures, independently.
So, my intuition is that it should be fine to mutate the dependency graph (add or remove reactive functions) while promises are pending, because the promise resolution would trigger another digest cycle that is independent from the first one. However this is a great edge case that you've identified, it would be great to have some unit tests for this once promises are handled.
Keep in mind all of this is just theoretical, I'm not actually very confident that I know what the correct behavior should be - maybe the output property should not be set to ReactiveModel.PENDING, because that would make the code that depends on the output property need to check for that value specifically, which may be a hassle. I'm not sure.
Thanks again for your input on this. I hope I can find time soon to implement the promises case. If you'd like to take a stab at it, your efforts would be much appreciated.

You may want to take a look at https://github.com/ZJONSSON/clues by @ZJONSSON , which is very similar to reactive-model and works with promises.

Added first pass at promises handling in 37b9b35883d529f408a3180f31988971b4a3e884

Dang! you beat me to the my POC.
Even so, here it is my POC, using promises without library modifications!
var Model = require('./reactive-model.js');
var model = Model()
var Promise = require('bluebird')

function asyncFunction(model, evt, ms) {
  var promise = Promise.resolve();
  ms = ms || 1000;
  console.log('Waiting for ' + ms + ' ms');
  setTimeout(function() {
    promise.then(function() {
      model[evt](42);
    });
  }, ms);
  // we could return a promise here, but there should be no consumer.
}

model.react({
  invokePromise:[ 'start', function(delay) {
    // This is the first half of the promise invocation.
    // Ensure no one depends on this reactive-property.
    console.log('promise invoked at ' + (new Date()))
    // Notice, we've introduced a pseudo-reactive-property - 'resolution'
    // Instead of depending on 'start', depend on 'resolution'
    asyncFunction(model, 'resolution', delay);
  }],
  postPromise: ['resolution', function(resolution) {
    // When done right, it works like a charm!
    console.log('Blocked issue can now be worked on ' + (new Date()) + ' ' + resolution)
  }]
});

model.start(5000);

It is equivalent to the ReactiveModel.PENDING state, without needing the actual state.
The only programming care needed is to ensure that no-one depends on 'start'.

That's a very nice implementation!

Thanks! And thank you for your POC, it's an interesting approach.
The implementation is just a start, the following issues still need to be addressed:

FIFO request-order - The previous promise needs to be somehow cancelled, or its resolution ignored, when it is in progress and a new input value comes along. It should be straightforward to write a unit test that fails without this behavior.
Any code that uses a reactive model that uses promises has no way of knowing whether there is a promise in progress. The would be useful if one wanted to make, for example, a "loading" spinning icon appear while data is being fetched. This was the original idea behind ReactiveModel.PENDING, but now I'm not so sure the best solution is to set a special value on the output property, because it would clutter up code with lines like if(data !== ReactiveModel.PENDING){ ... }. Perhaps there could be a separate API for reporting promises in progress, something like ReactiveModel.onPromisesPending(callback(boolean somethingIsPending){}).
I think there is a subtle bug whereby reactive functions downstream of the output value of a promise are still getting executed, even if their input values have not changed. This is another thing that should be possible to test.

I had another idea about this - it would be cool if the promises could have messages to display to the user while they are pending. This way, a list of transient messages like "fetching data" or "loading plugin" could be shown to the user, similar to the message that appears in the lower left corner of Chrome when requests are pending.

Great idea. To some degree, the intermediate message is what my implementation was attempting. Although it was brittle in requiring code hygiene.
Doing it in the library gives you an option to set both on/off state from the same event. Will have to provide for proper name-spacing to handle multiple simultaneous promises. Perhaps something like evt:promise-pending with a value true or false.

Not sure if working with Promises is worth adding back in given the current async format.
model({
  b: [function (a, done){
    setTimeout(function (){
      done(a + 1);
    }, 500);
  }, "a"]
});

Feel free to reopen if you want this.


Ensure that the favorite country is only suggested, not added automatically

Ensure that the favorite country is only suggested, not added automatically.
Otherwise, when people travel, or edit product from other countries, we risk entering bad data in the DB.

I'll work on it.

I tried to investigate this bug but can't "reproduce" it easily. How could we test it ?


[BENCHMARK] Up to 1,100% faster than ratelimit.js

:thumbsup:
Rough Specification
As someone with obviously a ton of experience with rate limiting, would be happy to hear your thoughts on the abstraction (if you have the time, of course).
I've been doing really extensive testing for load-testing etc on it and hope to put this into production where it would def be tested against some serious load!
More tests and examples of implementation (including the aggregated result) are in the specification above.
tl;dr
Just want to say thank you! as this provides a huge performance improvement over what is pretty much the only other established method I have found ON TOP OF actually implementing GCRA rather than a simple "how many requests were made in the last n seconds.
Stress Test Results
Calls Setup, Starting Requests
Running Tests for:  cell
[ 'user', '1', 'trade' ]
[ 'user', '2', 'trade' ]
[ 'user', '3', 'trade' ]
[ 'user', '4', 'trade' ]
[ 'user', '5', 'trade' ]
[ 'user', '6', 'trade' ]
[ 'user', '7', 'trade' ]
Running Tests for:  rl
[ '1', 60, 'user:1:trade' ]
[ '2', 60, 'user:2:trade' ]
[ '3', 60, 'user:3:trade' ]
[ '4', 60, 'user:4:trade' ]
[ '5', 60, 'user:5:trade' ]
[ '6', 60, 'user:6:trade' ]
[ '7', 60, 'user:7:trade' ]

    --- Test Results ---

    Total Iterations: 5000 * 7 Tests (35,000 iterations each)

    rl:
      Total Duration: 343693971.74181193
      Average: 9819.82776405177
      Max: 14198.909738004208
      Min: 5541.352702006698

    cell:
      Total Duration: 26785993.573875546
      Average:<PHONE_NUMBER>107299
      Max: 1454.7679300010204
      Min:<PHONE_NUMBER>9902725

    Diff: cell is 1183% faster

Quick Test Results (consistent against around 500 runs of the test)
// RUN FOR CELL ONLY (Dont Require rl or include in scope at all)
Calls Setup, Starting Requests
Running Tests for:  cell
[ 'user', '1', 'trade' ]
[ 'user', '2', 'trade' ]
[ 'user', '3', 'trade' ]
[ 'user', '4', 'trade' ]
[ 'user', '5', 'trade' ]
[ 'user', '6', 'trade' ]
[ 'user', '7', 'trade' ]

    --- Test Results ---

    Total Iterations: 10 * 7 Tests (70 iterations each)

    cell:
      Total Duration:<PHONE_NUMBER>3997478
      Average: 3.3139302277139255
      Max: 4.6971050053834915
      Min: 2.4761980026960373

// RUN FOR RL ONLY (Dont Require cell or include in scope at all)
Calls Setup, Starting Requests
Running Tests for:  rl
[ '1', 60, 'user:1:trade' ]
[ '2', 60, 'user:2:trade' ]
[ '3', 60, 'user:3:trade' ]
[ '4', 60, 'user:4:trade' ]
[ '5', 60, 'user:5:trade' ]
[ '6', 60, 'user:6:trade' ]
[ '7', 60, 'user:7:trade' ]

    --- Test Results ---

    Total Iterations: 10 * 7 Tests (70 iterations each)

    rl:
      Total Duration: 2249.6415529847145
      Average: 32.13773647121021
      Max: 36.03293499350548
      Min: 28.242240011692047

Implementation / Benchmark
So this is definitely not even a fair benchmark considering the module runs as a native module and I implemented lua logic to handle the situation - but my goal was to implement a multi-bucket rate limiter using this as a base.
Team is super spooked by the "note on stability" part though and wants to go with RateLimit.js  anyway which is completely client side :-\
Essentially I take a limits.yaml and use it to build a rate limiting bucket:
schema:
  # user:
  user:
    children:
      # if no others match
      "*":
        # limits against user:${username}
        limits:
          - 15
          - 30
          - 60
        children:
          # limits against user:${username}:trade
          trade:
            limits:
              - 5
              - 10
              - 15

Then I receive a "bucket key" which is an array of arguments ("user", "myuserid", "trade") and run against each step when limits is found, returning the results so that we can easily and efficiently implement multi-tiered rate limits against user actions with as few requests as possible.
So essentially when I call ("user", "myuserid", "trade") it is running
CL.THROTTLE user:myuserid 15 30 60
CL.THROTTLE user:myuserid:trade 5 10 15

--[[
  Summary:
    Takes the generated lua limits table (from limits.yaml which must be compiled
    whenever we need to change it) and iterates the request, returning the results
    of all matching rate limits in the requested path.
]]
local LimitsSchema = {
  ["user"] = {
    ["children"] = {["*"] = {["limits"] = {15, 30, 60}, ["children"] = {["trade"] = {["limits"] = {5, 10, 15}}}}}
  }
}

local Response, LimitsTable, CurrentPath = {}, {}, {}
local Complete = 1
local child

local CheckLimit = function(bucket, args)
  return redis.call("CL.THROTTLE", bucket, unpack(args))
end

for k, v in ipairs(KEYS) do
  if LimitsSchema[v] then
    child = LimitsSchema[v]
  elseif LimitsSchema["*"] then
    child = LimitsSchema["*"]
  else
    Complete = 0
    break
  end

  table.insert(CurrentPath, v)

  if child["limits"] then
    LimitsTable[table.concat(CurrentPath, ":")] = child["limits"]
  end
  LimitsSchema = child["children"]
end

for k, v in pairs(LimitsTable) do
  table.insert(Response, CheckLimit(k, v))
end

if Complete == 0 then
  return redis.error_reply("Invalid Path at: " .. table.concat(CurrentPath, ":"))
end

return Response

Lua Performance: In an initial check with the lua performance when running the benchmark, looks like the latency of the request being made averaged at around 0.50 milliseconds per call... pretty awesome!

Then utilizing ioredis to handle the script
// static setup that shouldnt count towards perf since it is
// done one time per server instance
const Redis = require("ioredis");
const fs = require("fs");
const path = require("path");
const redis = new Redis();

const cmd = {
  lua: fs.readFileSync(
    path.resolve(__dirname, "..", "lua", "traverse.lua")
  )
};

redis.defineCommand("limiter", cmd);

module.exports = redis;

const { performance } = require("perf_hooks");
const redis = require("./cell-setup");

function checkPath(...path) {
  return redis.limiter(path.length, ...path);
}

async function request(...args) {
  const startTime = performance.now();
  await checkPath(...args);
  return performance.now() - startTime;
}

module.exports = {
  request
};

Very cool @bradennapier! Thanks for the detailed writeup.

Team is super spooked by the "note on stability" part though and wants to go with RateLimit.js anyway which is completely client side :-\

I put this note there mostly because I'm not actively using it myself. If I can get a few confirmed users that are using it regularly with good success, I'll remove it (I think some already are, but it's difficult to tell unless they're advertising themselves).

@bradennapier That's great!!
Thanks for remembering to report back in. As suggested above, I removed the note on unknown stability in the README. See 456f0b5.


GC and Health checks fail, maybe caused by bug in git migration?

Gitea version (or commit ref): 1.14.3
Git version: 2.20.1 (server sided)
Operating system: Debian 10.10
"Your gitea build"
systemd
Linux version 4.19.0-17-amd64
Database (use [x]):

[ ] PostgreSQL
[X] MySQL
[ ] MSSQL
[ ] SQLite

Can you reproduce the bug at https://try.gitea.io:

[ ] Yes (provide example URL)
[ ] No
[X] Cannot (Admin required)

Log gist:
2021/07/15 17:49:02 .../repository/check.go:39:func1() [W] Failed to health check repository (&{8812 320 ORG  name DESC  1 MIGRATED_FROM_LINK develop 2 0 0 1 0 1 0 0 0 0 0 0 0 0 0 true false false false  0 map[] map[] []  false 0  false 0<PHONE_NUMBER>   true false [] default 8812-099d026b04c807c40b9c6236915c97e5<PHONE_NUMBER><PHONE_NUMBER>}): context deadline exceeded
2021/07/15 17:49:02 models/admin.go:53:createNotice() [I] [SQL] INSERT INTO notice (type,description,created_unix) VALUES (?,?,?) [1 Failed to health check repository (ORG/REPO): context deadline exceeded<PHONE_NUMBER>] - 1.809991ms
2021/07/15 17:49:02 .../repository/check.go:39:func1() [W] Failed to health check repository (&{8813 320 ORG  repo REPO DESC  0  master 2 0 0 0 0 0 0 0 0 0 0 0 0 0 0 true false false false  0 map[] map[] []  false 0  false 0<PHONE_NUMBER>7   true false [] default<PHONE_NUMBER><PHONE_NUMBER>}): context deadline exceeded
2021/07/15 17:49:02 models/admin.go:53:createNotice() [I] [SQL] INSERT INTO notice (type,description,created_unix) VALUES (?,?,?) [1 Failed to health check repository (ORG/REPO): context deadline exceeded<PHONE_NUMBER>] - 1.297265ms
2021/07/15 17:49:02 models/admin.go:53:createNotice() [I] [SQL] INSERT INTO notice (type,description,created_unix) VALUES (?,?,?) [2 Task: Health check all repositories started by gerdel has finished<PHONE_NUMBER>] - 1.38977ms

.ini:
[migrations]
ALLOW_LOCALNETWORKS = true
[git.timeout]
MIGRATE = 1800
GC = 1800
[cron.git_gc_repos]
ENABLED = true
RUN_AT_START = true
TIMEOUT = 1800
[cron.repo_health_check]
TIMEOUT = 1800
Description
After setting up a fresh gitea server, I migrated several hundred repos into it. Afterwards I took one of these repos and migrated it to LFS, via git lfs migrate and pushed it into a newly created repo of the gitea server.
After starting a health check, the health check fails within a few seconds for >1000 repos with the following message: "context deadline exceeded"
After starting a garbage collection, the garbage fails and stops with the first repo, also with the following message: "context deadline exceeded"
After investigating the logfile, I saw that every error included the repo url from the migration source.
May this be caused by the migration process or are my repos corrupted?

context deadline exceeded - means that git fsck etc took too long.
Change these settings:
https://docs.gitea.io/en-us/config-cheat-sheet/#git---timeout-settings-gittimeout

Do you mean this part? ;;(For GC)
[git.timeout]
GC = 1800
[cron.repo_health_check] ;;(For fsck)
TIMEOUT = 1800
These were the only options I could find.
If yes these are included already, also it seems the default timeout is less than 1 second (because it fails directly), what seems weird to me.

[cron.repo_health_check] ;;(For fsck)
TIMEOUT = 1800s

Ahh well I didn't saw the 's'..
Sorry about that.
I will try it out by tomorrow, give you feedback and close the issue if it's resolved.
Thank you for your time in advance.

Ahh well I didn't saw the 's'..

There's an inconsistency in the git.timeout configuration vs. the cron configuration. It's hard to fix though because the git.timeout although the configuration options should be using the duration notation changing it would be a seriously breaking configuration change.

Just my own 2 cents:
What about the idea of killing the process if the config has not parsable content;
This way the user is directly informed about his mistake, because there is no gitea and the log / console's last line would be like:
"Invalid value found for XXX" (Did you forget the unit of time?) For example?

it is parsable.
Just that the 1800 was interpreted as 1800 ms not seconds


RR-395 - Added PrisonerSearchSummary

This PR adds the location field to PrisonerSummary because we can populate that field from the Prisoner record that we get back from prisoner-search-api
It also adds PrisonerSearchSummary which extends PrisonerSummary and adds fields that will be populated from calls made to the PLP and CIAG APIs.

LGTM


Refactoring query functions to make better use of SqlPersistT.

N/A
Overview

Refactoring our use of SqlBackend.
There are lots of places in this code where we run in a ReaderT  SqlBackend monad and pass a SqlBackend around manually.
I've neatened this up as much as I can by removing the manual
backend-passing, and hoisting the runQuery backend call into Servant's
hoistServer mechanism.
The end result is, IMHO, much cleaner.
Comments

liftIO change pushed, and I've amended the PR title.

bors r+


Failure in linking xtensor-blas with lapack/blas?

I'm trying to compile a program that uses xtensor-blas, when I encounter a linker error. The error message is
...
testTensor.o: In function `int cxxlapack::getrf<int>(int, int, double*, int, int*)':
/usr/local/include/xflens/cxxlapack/interface/getrf.tcc:76: undefined reference to `dgetrf_'
collect2: error: ld returned 1 exit status
...

where testTensor.o is calling auto d = xt::linalg::det(a);.
I'm using lapack-3.10.1 and blas-3.10.0, and I have supplied the -lblas -llapack -DHAVE_CBLAS=1 tag to g++, as suggested by the documention. I've also tried using the alternative tag -DXTENSOR_USE_FLENS_BLAS, which results in the same error message.
Any help or suggestion is hugely appreciated. Thanks!

I'm trying to compile a program that uses xtensor-blas, when I encounter a linker error. The error message is
...
testTensor.o: In function `int cxxlapack::getrf<int>(int, int, double*, int, int*)':
/usr/local/include/xflens/cxxlapack/interface/getrf.tcc:76: undefined reference to `dgetrf_'
collect2: error: ld returned 1 exit status
...

where testTensor.o is calling auto d = xt::linalg::det(a);. I'm using lapack-3.10.1 and blas-3.10.0, and I have supplied the -lblas -llapack -DHAVE_CBLAS=1 tag to g++, as suggested by the documention. I've also tried using the alternative tag -DXTENSOR_USE_FLENS_BLAS, which results in the same error message.
Any help or suggestion is hugely appreciated. Thanks!

Do u resolve this problem? I have the same one.

@willamhou In my case, I'm on Ubuntu, and I have to also supply -lgfortran after -lblas -llapack as these two libraries depends on it.
But when switching to OSX, it seems like the opposite is the case.

@willamhou In my case (I'm on Ubuntu), I have to also supply -lgfortran after -lblas -llapack as these two libraries depends on it. But when switching to OSX, it seems like -lgfortran should NOT be supplied.

Thanks u for your reply. I solve this problem, add "-lcblas -lblas -llapack `-lgfortran" to complie the binary.

@willamhou In my case (I'm on Ubuntu), I have to also supply -lgfortran after -lblas -llapack as these two libraries depends on it. But when switching to OSX, it seems like -lgfortran should NOT be supplied.

hanks u for your reply. I solve this problem, by adding "-lcblas -lblas -llapack `-lgfortran" to complie the binary.


Investigate alexjs for catching insensitive language

Slack: https://prisma-company.slack.com/archives/CD2HW8X2A/p1589789400418400?thread_ts=1589787541.416600&cid=CD2HW8X2A
https://alexjs.com/

Example
https://github.com/facebook/react-native-website/blob/master/.alexrc.js
The rules explanation
https://github.com/retextjs/retext-equality/blob/master/rules.md

You can run it like this npx alex

Some additional context that might be interesting: There recently were a few Twitter threads were some people expressed unhappiness with the broad strokes that alexjs uses - thing like asian being classified as possibly "profane" and the list of words just being taken from some source and not being tailored or customized. (Can't find an easy link to these discussions though - sorry)
Should be fine from our side though as long as we are aware of this.

The stuff @janpio is calling attention to is https://wooorm.com/blog/alex/ (my side, linked at the bottom is that twitter thread).

Thanks @wooorm for the link!
I guess where alex could be useful here is to get a sense of what the current state is. Because it's a linter it can read all the text at once. Then we can decide what to update or not and ignore some rules.
So maybe more a one time use of the tool.


Axis label multiple levels for more dimension

Task:
If there are multiple dimensions on axis we should show the labels for all dimensions instead of selecting one with labelLevel parameter
This can be replicated by following the plots in the sorting tutorial notebook.
As you follow along, the second to last plot looks like this:

and the final plot looks like:

Here - I think ideally, the final plot would have multi-tiered x axis labels, as by default it is impossible to know what each of the Genre bars within a Kind represent.

There should be two different types of multi level axis labels:

nested:

A   B   C   A   B   C
    X           Y

flattened:

X,A X,B X,C Y,A Y,B Y,C


SocialUserValueObjectTestを作成

IssuesNumber

#125

Coverage increased (+1.3%) to 6.753% when pulling 8ca1ef9b6cc1f521b948bb7c4d87f45aa9485bba on 66 into 9944716e3e36349beb991c8e019cf35ea62a1c58 on develop.


Proper documentation for cstor pool deletion using director.

I need to automate cstor pool deletion test case for that I need proper documentation including information about POST request.

Still waiting for @shovanmaity to get the API docs to start with testcases.

Delete SPC
URL -
/v3/groups/<groupId>/cstorpooloperations
METHOD -
POST
BODY -
{
  "clusterId": "1c1122",
  "input": "{"name":"<name>","kind":"StoragePoolClaim","version":"v1alpha1"}",
  "kind": "DeleteCStorPoolCluster"
}

NOTE - input in the request body is a JSON string not a JSON object.

Delete v1alpha1 CSPC
URL -
/v3/groups/<groupId>/cstorpooloperations
METHOD -
POST
BODY -
{
  "clusterId": "1c1122",
  "input": "{"name":"<name>","kind":"CStorPoolCluster","version":"v1alpha1"}",
  "kind": "DeleteCStorPoolCluster"
}

NOTE - input in the request body is a JSON string not a JSON object.

Delete v1 CSPC
URL -
/v3/groups/<groupId>/cstorpooloperations
METHOD -
POST
BODY -
{
  "clusterId": "1c1122",
  "input": "{"name":"<name>","kind":"CStorPoolCluster","version":"v1"}",
  "kind": "DeleteCStorPoolCluster"
}

NOTE - input in the request body is a JSON string not a JSON object.

Info provided by shovan


Added 'Last Updated' text to addon details (bug 988078)

This should be pretty self-explanatory.

@magopian Good to know! Fixed.

Thanks!
r+


YAML: No support for using a config variable for endpoint

Reproduction steps
Summary:
we tried to make endpoint as a configurable parameter we are seeing a build error attached diff link for the changes made in YAML file please please let us know if i am doing it in wrong way.
Diff link: https://github.com/project-chip/connectedhomeip/compare/master...manjunath-grl:connectedhomeip:endpoint_Config_BuildError
Bug prevalence

GitHub hash of the SDK that was being used
dc66d4efbd95c55c6564c2432f4929cb5a46ba24
Platform
other
Platform Version(s)
No response
Anything else?
Execution logs for reference:

Endpoint_Error.zip

@vivien-apple we still need this support to add 2 endpoints in the same script


Add regex param to HeaderMatchesChecker rule in the default config.

I added a missing regex parameter from the HeaderMatchesChecker rule in the default config.
This fixes #321.

Released in https://github.com/beautiful-scala/scalastyle/releases/tag/v1.4.0.


Update http4s-blaze-client, http4s-circe to 0.21.2

Updates

org.http4s:http4s-blaze-client
org.http4s:http4s-circe

from 0.21.1 to 0.21.2.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.http4s" } ]

labels: library-update, semver-patch

Codecov Report

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

@@           Coverage Diff           @@
##           master     #387   +/-   ##
=======================================
  Coverage   78.71%   78.71%
=======================================
  Files          23       23
  Lines         531      531
  Branches        2        2
=======================================
  Hits          418      418
  Misses        113      113

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 05a164c...7e6bcc8. Read the comment docs.


Packaged release 0.2.9

Codecov Report

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

@@            Coverage Diff            @@
##           master   #3396      +/-   ##
=========================================
- Coverage    70.7%   70.7%   -0.01%
=========================================
  Files         393     393
  Lines       25472   25472
  Branches     3884    3884
=========================================
- Hits        18010   18009       -1
- Misses       6319    6320       +1
  Partials     1143    1143

Impacted Files
Coverage Δ

...li-rdbms/azure/cli/command_modules/rdbms/custom.py
63.93% <0%> (-1.64%)
:arrow_down:

Continue to review full report at Codecov.

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


[FluentPanelSplitter] New Splitter component (Experimental)

Panel Splitter (Experimental)

⚠ This component is for evaluation purposes only and is subject to change or removal in future updates. Suppress this diagnostic to proceed. To use it, you need to add the following line to the top of your razor file @{ #pragma warning disable FluentPanelSplitter }

The Panel Splitter splits the page into sections and allows the user to control the page layout.
You can include as many Pane as you like in a container. By default, panels are resizable, but the Resizable property can be used to block this functionality. The initial Size is in percent or pixels and has a Min and Max value. A Panel can be fully collapsed by setting the Collapsible property to True.
These CSS variables are predefined with these values, but can be overwritten (using Style property
    .fluent-panel-splitter {
        /* Resize Icon colors */
        --fluent-panel-splitter-color: var(--neutral-stroke-strong-rest);
        --fluent-panel-splitter-color-active: var(--neutral-stroke-strong-hover);

        /* Bar colors */
        --fluent-panel-splitter-background-color: var(--neutral-stroke-rest);
        --fluent-panel-splitter-background-color-active: var(--neutral-stroke-hover);
        --fluent-panel-splitter-hover-opacity: 0.8;

        /* Bar size */
        --fluent-panel-splitter-bar-size: 8px;
    }

Why have we created this new component when FluentSplitter already exists?
We want to generalize this component by making it easier to have several (>2) panels per container. We've also used CSS variables to simplify the customization of colors and sizes.
Give us your feedback

Example

<FluentPanelSplitter Height="150px">
    <FluentPanelSplitterPane Size="20%" Min="50px" Max="70%">
        Left Menu
    </FluentPanelSplitterPane>
    <FluentPanelSplitterPane Size="50%">
        <FluentPanelSplitter Orientation="Orientation.Vertical">
            <FluentPanelSplitterPane Collapsible="true">
                Main Content
            </FluentPanelSplitterPane>
            <FluentPanelSplitterPane Collapsible="true">
                Console log
            </FluentPanelSplitterPane>
        </FluentPanelSplitter>
    </FluentPanelSplitterPane>
    <FluentPanelSplitterPane Size="30%">
        Properties
    </FluentPanelSplitterPane>
</FluentPanelSplitter>

Unit Tests

BarSize should be used when calculating the size of the panes. If I have two panes and set them both to 50%, they do not have the same size because BarSize is not taken into account.

BarSize should be used when calculating the size of the panes. If I have two panes and set their Size both to 50%, they do not have the same size because BarSize is not taken into account.

I've updated the code so that the proportions between the panels are correct. The problem wasn't removing the BarSize, but setting the correct flex value in the CSS: flex: 0 0 auto; => flex: 0 1 auto;.
In this way, the panel size is automatically calculated by Flex.

⚠ Following an internal discussion, we probably renamed this component to FluentMultiSplitter. To give a more distinctive name (vs FluentSplitter) and improve clarity of use.


typo erros in below section

Below command has typo's. Please correct it.
Copy the appropriate admin.kubeconfig, kube-controller-manager and kube-scheduler kubeconfig files to each controller instance:
for instance in master-1 master-2; do
scp admin.kubeconfig kube-controller-manager.kubeconfig kube-scheduler.kubeconfig ${instance}:~/
done
Correct command.
for instance in master-1 master-2; do
  scp admin.kubeconfig kube-controller-manager  kube-scheduler  ${instance}:~/
done

Hi @saikantgajula
We have now merged a major change to bring this to v1.24.
Many issues have been addressed as part of the upgrade. Please try it now and feel free to raise further issues.
Thanks.


Update codebase to support v6

v6 was published a couple of days ago
https://github.com/nestjs/throttler/releases/tag/v6.0.0

Released v0.5.0 on NPM with this.

Released v0.5.0 on NPM with this.


Can't build master

$ sbt assembly
Getting org.scala-sbt sbt 0.13.12 ...
downloading https://repo.typesafe.com/typesafe/ivy-releases/org.scala-sbt/sbt/0.13.12/jars/sbt.jar ...
	[SUCCESSFUL ] org.scala-sbt#sbt;0.13.12!sbt.jar (1245ms)
[...]
	5 artifacts copied, 0 already retrieved (24494kB/23ms)
/home/olivier/src/shard/build.sbt:22: error: not found: value assembly
test in assembly := {}
        ^
[error] Type error in expression
Project loading failed: (r)etry, (q)uit, (l)ast, or (i)gnore? r

$ java -version
openjdk version "1.8.0_102"
OpenJDK Runtime Environment (build 1.8.0_102-b14)
OpenJDK 64-Bit Server VM (build 25.102-b14, mixed mode)
Using Arch linux.
Jar works fine.

Ah. my .gitignore was too restrictive and hid the required project/assembly.sbt file. Pull the latest commit, should work now. I guess you are the first one to try and build it :)

I can build master branch now. Thanks!


Expose metrics for webhook and ca-injector

Is your feature request related to a problem? Please describe.
Currently, for the cert-manager-webhook and cert-manager-cainjector there is no visibility in how the these 2 components are working compared to the controller besides kube metrics. So we are unable to know if they are having any problems or not and know its availability which we can use for SLOs.
Describe the solution you'd like
Have some minimum metrics exposed e.g.
webhook - http_requests (status code, method, path)
ca-injector - injection_status (success, fail etc..), errors
Describe alternatives you've considered
Using kube metrics, but this is very broad and doesn't provide much details on its behavior
Additional context
Similar to how other kube components in the industry exposes metrics for it self for measuring
Environment details (remove if not applicable):

Kubernetes version: 1.25+
Cloud-provider/provisioner: eks
cert-manager version: v1.14.2,
Install method: helm

/kind feature

I will start by adding the standard go runtime metrics to those two components for consistency with the controller:

https://github.com/cert-manager/cert-manager/pull/6966

@nitinatgh
A pre-release is now available containing the fix for this issue. Please test and feedback if you have time.

https://github.com/cert-manager/cert-manager/releases/tag/v1.16.0-alpha.0

A beta-release is now available which contains the fix for this issue. Please test and feedback if you have time.

https://github.com/cert-manager/cert-manager/releases/tag/v1.16.0-beta.0


Doc edits

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

/do-doc-build


[Refactor] ng-lib installation logs

🐞 Bug report
Description
When the project started, a support was added to Angular v8 with a dedicated ng-lib-v8.
The package is added in relation with the Angular version of the project in which Scully is added :
if (+ngCoreVersionTag.search(/(\^8|~8)/g) === 0) {
    context.logger.info('Install ng-lib for Angular v8');
    addPackageToPackageJson(tree, '@scullyio/ng-lib-v8', `${scullyComponentVersion}`);
  } else {
    context.logger.info('Install ng-lib for Angular v9');
    addPackageToPackageJson(tree, '@scullyio/ng-lib', `${scullyComponentVersion}`);
  }

But as i just added scully on an Angular 10 project, it feels misleading to see the for Angular v9 as Scully supports Angular 10.

🔬 Minimal Reproduction
run ng add @scullyio/init on a project
look at the terminal logs
💻Your Environment
Angular Version:

Angular CLI: 10.0.8
Node: 12.18.2
OS: darwin x64

Angular: 10.1.2
... animations, common, compiler, compiler-cli, core, forms
... language-service, platform-browser, platform-browser-dynamic
... router
Ivy Workspace: Yes

Package                           Version
-----------------------------------------------------------
@angular-devkit/architect         0.1000.8
@angular-devkit/build-angular     0.1000.8
@angular-devkit/build-optimizer   0.1000.8
@angular-devkit/build-webpack     0.1000.8
@angular-devkit/core              10.0.8
@angular-devkit/schematics        10.0.8
@angular/cli                      10.0.8
@ngtools/webpack                  10.0.8
@schematics/angular               10.0.8
@schematics/update                0.1000.8
rxjs                              6.5.5
typescript                        3.9.7
webpack                           4.43.0

Scully Version:

"@scullyio/init": "^1.0.0",
    "@scullyio/ng-lib": "^1.0.0-beta.1",
    "@scullyio/scully": "^1.0.0-beta.5",

🔥 Exception or Error

if you are fine with removing the 'for Angular v9' part, i can make a PR.


HTTP verb rules in router are NOT case-insensitive

The docs http://www.codeigniter.com/user_guide/general/routing.html?highlight=rest#using-http-verbs-in-routes says that HTTP verb rules are case-insensitive  and proceeds to give these examples.
$route['products/(:num)']['DELETE'] = 'product/delete/$1';
$route['products']['put'] = 'product/insert'

But, when I try to use capitalized HTTP verb such as POST etc,  404 error is shown.
Looking through function _parse_routes() in system/core/Router.php.
$http_verb = isset($_SERVER['REQUEST_METHOD']) ? strtolower($_SERVER['REQUEST_METHOD']) : 'cli';
....
elseif (is_array($this->routes[$uri]) && isset($this->routes[$uri][$http_verb]))

HTTP verbs from the $_SERVER is changed into lowercase before the $this->routes array is checked.
Hence, only lower case HTTP verbs seems to be working, contradicting with the docs.

Thanks @narfbg for the quick fix. :+1:


[Gemfile] Lock the config gem

The config gem had a recent change to it that was recently released which broke our build:

master
kasparov
jansa

So for now, lock the version to a known working version for us.


https problem

When the application page is loaded via https, google stuff needs also be loaded from https, otherwise the browsers won't load it because of mixed content prohibition.
Please change the line in loadScripts to
Loader.defaultUrl = location.protocol.replace('file', 'https')+'//maps.google.com/maps/api/js?';
and the line in markerclusterer to
MarkerClusterer.IMAGE_PATH = location.protocol.replace('file', 'https')+"//google-maps-utility-library-v3.googlecode.com/svn/trunk/markerclustererplus/images/m";

i believe this was fixed

Yes, indeed, this is fixed, sorry, forgot to close the issue when I noticed this.


bug: Clicking on timestamp from comments redirects user to other page

Describe the bug
When the user clicks on any timestamp present in comments, it navigates to other page.
To Reproduce
Steps to reproduce the behavior:

Add a comment with timestamp (Eg: 10:01)
Click on the timestamp
It navigates to other page(Home page)

Expected behavior
When a user clicks on timestamp, it should seek the video to that time.
Browser:

Chrome

@hkirat , This issue is fixed in PR #464 .


📝 Update includes in ./docs/zh/docs/advanced/sub-applications.md

Thanks for the effort! :cake:
This was handled in another PR, so I'll close this one now. :coffee:


disallow folders with just one file

For me, having only a single file sitting in directory is not nice.
For example in this repo: https://github.com/benawad/butler-cli/tree/9ad6e4c11ee096314402060f541047253336c5e1/src/index/formatFileStructure/syncFileSystem/fixImports
I would prefer if butler places the makeImportPath.ts into syncFileSystem folder next to fixImports.ts.
If you could add this disallow-leaf-folders-with-one-file config that would be just great! WDYT?

I wouldn't recommend doing that in a "managed by hand" fractal structure since it involve to move makeImportPath.ts to a new folder if you want to add another deps tofixImports.ts. But because this is now managed by Butler, he could move makeImportPath.ts in a new fixImports/ folder for us. So yeah, why not!

Anyone having a short name idea for this option?
I though about singleFile/--single--file but it's not that self explanatory...

I think --move-only-child or something similar is quite explanatory.

--rename-only-child is even better I think.


Add advanced options to set manager, role, department, and location

[x] Retrieve available departments from Ramp, disable external departments (Foundation Group), allow selecting any other department, default to Programs
[x] Retrieve available locations from Ramp, allow selecting any location, default to Atlanta
[x] Hardcoded roles list (BUSINESS_BOOKKEEPER, BUSINESS_USER)
[x] Retrieve available users from Ramp, disable external users (Foundation Group), allow selecting any other user, default to the one matching the Apiary manager

Completed in c7b56ac3948b9c30a8a41ab9c474b5c068e88de8


Incompatibility with spatialdata objects on plotting features by using sq.pl.spatial_scatter

Description
I would like to visualize two features (total_counts and n_genes_by_counts) in my spatialdata objects by using sq.pl.spatial_scatter. I was able to generate those figures when I loaded my visium data by using sq.read.visium but it is not clear how to do it with my Xenium data in SpatialData format. Any thoughts or suggestions? Thank you.
Minimal reproducible example
Here is my code:
sq.pl.spatial_scatter(sdata_list[0]["table"], color=["total_counts", "n_genes_by_counts"], ncols=2, img=False)
Traceback
Cell In[40], line 1
----> 1 sq.pl.spatial_scatter(sdata_list[0]["table"],
2 color=["total_counts", "n_genes_by_counts"], ncols=2, img=False)
File ~/.local/lib/python3.9/site-packages/squidpy/pl/_spatial.py:420, in spatial_scatter(adata, shape, **kwargs)
379 @d.dedent
380 @_wrap_signature
381 def spatial_scatter(
(...)
384 **kwargs: Any,
385 ) -> Axes | Sequence[Axes] | None:
386 """
387 Plot spatial omics data with data overlayed on top.
388
(...)
418 %(spatial_plot.returns)s
419 """
--> 420 return _spatial_plot(adata, shape=shape, seg=None, seg_key=None, **kwargs)
File ~/.local/lib/python3.9/site-packages/squidpy/pl/_spatial.py:182, in _spatial_plot(adata, shape, color, groups, library_id, library_key, spatial_key, img, img_res_key, img_alpha, img_cmap, img_channel, seg, seg_key, seg_cell_id, seg_contourpx, seg_outline, use_raw, layer, alt_var, size, size_key, scale_factor, crop_coord, cmap, palette, alpha, norm, na_color, connectivity_key, edges_width, edges_color, library_first, frameon, wspace, hspace, ncols, outline, outline_color, outline_width, legend_loc, legend_fontsize, legend_fontweight, legend_fontoutline, legend_na, colorbar, scalebar_dx, scalebar_units, title, axis_label, fig, ax, return_ax, figsize, dpi, save, scalebar_kwargs, edges_kwargs, **kwargs)
168 edges_kwargs = dict(edges_kwargs)
170 color_params = _prepare_args_plot(
171 adata=adata,
172 shape=shape,
(...)
179 palette=palette,
180 )
--> 182 spatial_params = _image_spatial_attrs(
183 adata=adata,
184 shape=shape,
185 spatial_key=spatial_key,
186 library_id=library_id,
187 library_key=library_key,
188 img=img,
189 img_res_key=img_res_key,
190 img_channel=img_channel,
191 seg=seg,
192 seg_key=seg_key,
193 cell_id_key=seg_cell_id,
194 scale_factor=scale_factor,
195 size=size,
196 size_key=size_key,
197 img_cmap=img_cmap,
198 )
200 coords, crops = _set_coords_crops(
201 adata=adata,
202 spatial_params=spatial_params,
203 spatial_key=spatial_key,
204 crop_coord=crop_coord,
205 )
207 fig_params, cmap_params, scalebar_params, kwargs = _prepare_params_plot(
208 color_params=color_params,
209 spatial_params=spatial_params,
(...)
228 **kwargs,
229 )
File ~/.local/lib/python3.9/site-packages/squidpy/pl/_spatial_utils.py:289, in _image_spatial_attrs(adata, shape, spatial_key, library_id, library_key, img, img_res_key, img_channel, seg, seg_key, cell_id_key, scale_factor, size, size_key, img_cmap)
286 return False
287 return img is True or len(img) # type: ignore
--> 289 library_id = _get_library_id(
290 adata=adata,
291 shape=shape,
292 spatial_key=spatial_key,
293 library_id=library_id,
294 library_key=library_key,
295 )
296 if len(library_id) > 1 and library_key is None:
297 raise ValueError(
298 f"Found library_id: {library_id} but no library_key was specified. Please specify library_key."
299 )
File ~/.local/lib/python3.9/site-packages/squidpy/pl/_spatial_utils.py:128, in _get_library_id(adata, shape, spatial_key, library_id, library_key)
120 def _get_library_id(
121 adata: AnnData,
122 shape: _AvailShapes | None,
(...)
125 library_key: str | None = None,
126 ) -> Sequence[str]:
127 if shape is not None:
--> 128 library_id = Key.uns.library_id(adata, spatial_key, library_id, return_all=True)
129 if library_id is None:
130 raise ValueError(f"Could not fetch library_id, check that spatial_key: {spatial_key} is correct.")
File ~/.local/lib/python3.9/site-packages/squidpy/_constants/_pkg_constants.py:151, in Key.uns.library_id(cls, adata, spatial_key, library_id, return_all)
143 https://github.com/classmethod
144 def library_id(
145 cls,
(...)
149 return_all: bool = False,
150 ) -> Sequence[str] | str | None:
--> 151 library_id = cls._sort_haystack(adata, spatial_key, library_id, sub_key=None)
152 if return_all or library_id is None:
153 return library_id
File ~/.local/lib/python3.9/site-packages/squidpy/_constants/_pkg_constants.py:182, in Key.uns._sort_haystack(cls, adata, spatial_key, library_id, sub_key)
173 https://github.com/classmethod
174 def _sort_haystack(
175 cls,
(...)
179 sub_key: str | None = None,
180 ) -> Sequence[str] | None:
181 if spatial_key not in adata.uns:
--> 182 raise KeyError(f"Spatial key {spatial_key!r} not found in adata.uns.")
183 haystack = list(adata.uns[spatial_key])
184 if library_id is not None:
KeyError: "Spatial key 'spatial' not found in adata.uns."

hi @joonlee3 , so the plotting functions in squidpy don't work with spatialdata. In this specific case, they don't work for anndata either, because the anndata stored in spatialdata does not have all info required for the plotting function to work. In this specific case, if you would have set shape=None then it would have worked, as spatial_scatter would have defaulted to scatterpplot and not visium circles. However, I would suggest to use spatialdata-plot for plotting spatialdata and not squidpy


Web - Does no show any member on invite member to policy

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:

Go to Settings > Workspaces
Click on New workspace button
Go to Settings > Workspaces > Members
Click on invite button
Type email with capital latter and which is not present in list
Click on Next button
Click on invite button which redirects user to Members screen

Expected Result:
Should show members which is part of policy
Actual Result:
Does no show any member
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

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

Version Number: 1.3.33-4
Reproducible in staging?: y
Reproducible in production?: y
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://github.com/Expensify/App/assets/93399543/9c7565fc-f743-412f-bfb4-30b07b373cfc
Expensify/Expensify Issue URL:
Issue reported by: @gadhiyamanan
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1687943695819349
View all open jobs on GitHub

Linking similar issue https://github.com/Expensify/App/issues/21804

@kbecciv - you're right - i think this is essentially a dupe so closing in favour of https://github.com/Expensify/App/issues/21804.


Use the safe load method in the YAML loader to avoid a warning

The YAML loader is currently using the yaml.load() method without defining a loader, which is deprecated and causes a warning to appear:
YAMLLoadWarning: calling yaml.load() without Loader=... is deprecated, as the default Loader is unsafe. Please read https://msg.pyyaml.org/load for full details.
  obj = self.yaml.load(f) or {}

This patch updates the call to use yaml.safe_load() which is a shortcut method to use the SafeLoader class.
Fixes #517

Thanks.


feat: Allow for multiple MessageBus subscriptions

PR Checklist
Please check if your PR fulfills the following requirements:

[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)

PR Type
What kind of change does this PR introduce?

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

What is the current behavior?
Profiles use old SubscribeTopic for a single topic
Issue Number: #152
What is the new behavior?
Profiles use new SubscribeTopics allowing for single or multiple topics
Does this PR introduce a breaking change?

[X] Yes
[ ] No

Are there any new imports or modules? If so, what are they used for and why?
No
Are there any specific instructions or things that should be known prior to reviewing?
Other information

This PR is in draft as it is dependent on PR https://github.com/edgexfoundry/app-functions-sdk-go/pull/625 to be merged first

This PR is in draft as it is dependent on PR https://github.com/edgexfoundry/app-functions-sdk-go/pull/625 to be merged first


Document ScatterPlotList

ScatterPlotList requires demo and better documentation
https://scottplot.net/faq/live-data/
Related #1751

https://scottplot.net/cookbook/4.1/category/plottable-scatter-plot-list/
https://scottplot.net/faq/live-data/#growing-data-with-scatterplotlist


Add documentation for xslURL in sitemap package

Description
Add documentation for the new xslURL in @astrojs/sitemap.

Hello! Thank you for opening your first PR to Astro’s Docs! 🎉
Here’s what will happen next:

Our GitHub bots will run to check your changes.
If they spot any broken links you will see some error messages on this PR.
Don’t hesitate to ask any questions if you’re not sure what these mean!

In a few minutes, you’ll be able to see a preview of your changes on Vercel 🥳

One or more of our maintainers will take a look and may ask you to make changes.
We try to be responsive, but don’t worry if this takes a few days.

Lunaria Status Overview
🌕 This pull request will trigger status changes.

Learn more
By default, every PR changing files present in the Lunaria configuration's files property will be considered and trigger status changes accordingly.
You can change this by adding one of the keywords present in the ignoreKeywords property in your Lunaria configuration file in the PR's title (ignoring all files) or by including a tracker directive in the merged commit's description.

Tracked Files

Locale
File
Note

en
guides/integrations-guide/sitemap.mdx
Source changed, localizations will be marked as outdated.

Warnings reference

Icon
Description

🔄
The source for this localization has been updated since the creation of this pull request, make sure all changes in the source have been applied.

Hi @sondr3 ! Can you please fill out the PR template and add a description so that I know what this is referring to? Thank you!

Well, this is odd! @sondr3 I know I made an edit here originally, but it's probably not saved on GitHub. Is there any chance my commit still exists in your history somewhere?
I can re-edit to conform to our standards, but if I already did this once, it would be handy if that could just be applied here!

I checked on my machine and I don't have anything besides my own commit sadly 😢

@sondr3 No worries! Thank you for checking! I'll make some tiny edits today to just conform to our style, but wanted to wait just in case there was a history of what I'd already done!

Looks like this was released!


docker-machine env is slow

I could not find an open ticket regarding this problem, but I guess I'm not the only one having this problem?
$ uname -a
Darwin 14.5.0 Darwin
Kernel Version 14.5.0: Wed Jul 29 02:26:53 PDT 2015;
root:xnu-2782.40.9~1/RELEASE_X86_64 x86_64

$ time docker-machine env dev >/dev/null

real	0m0.254s
user	0m0.217s

.. so having eval $(docker-machine env dev) in .bash_profile is no fun.
Workaround 1: Hard code the exports
$ docker-machine env dev >> ~/.bash_profile

Workaround 2: Very dirty lazy-loading fix
# Lazy load docker-machine env on first docker run
alias docker='docker-machine start dev>/dev/null && eval $(docker-machine env dev) && unalias docker && docker'

It's harder than it looks to make it fast, since you have to actually lookup the IP address of the VM in question (and regenerate certs if they have been invalidated).  If you're willing to accept the consistency tradeoff, you might be able to make your own version using the values "cached" by inspect, but I can't guarantee that will work forever.

I'm going to close this as I can't think of a way to improve the current method without sacrificing consistency, but feel free to open a new issue with suggestions on how the method might be improved, e.g. docker-machine env --cached dev to use "cached" values.

I'm not that familiar with docker-machine, so it's hard to come up with a suggestion. I just wanted to report the issue so it could be discussed :)
I'm using the vmwarefusion driver btw.

Thanks for reporting, and noted!

I ended up with this, it runs in 0.040 for me.
: ${DOCKER_MACHINE_NAME="my_vm_name"}

docker-machine inspect ${DOCKER_MACHINE_NAME} --format \
"export DOCKER_HOST=tcp://{{ .Driver.StorePath }}:2376
export DOCKER_TLS_VERIFY=1
export DOCKER_CERT_PATH={{ .Driver.StorePath }}/${DOCKER_MACHINE_NAME}
export DOCKER_MACHINE_NAME=${DOCKER_MACHINE_NAME}"


getProperty omits functions

Steps to reproduce
Tell us about your environment:

Puppeteer version: 1.19.0
Platform / OS version: Win 10
URLs (if applicable): n/a
Node.js version: 10.15.3 + @stencil/core 1.8.1

What steps will reproduce the problem?
Add a property that includes a method to an element and then call getProperty, the result will include everything except the method:
Please include code that reproduces the issue.
const myObj = {
  propOne: 'abc',
  methodOne: () => console.log('hello')
}

myElement.setProperty('someProp', myObj)

What is the expected result?
await myElement.getProperty('someProp') should return:
{
  propOne: 'abc',
  methodOne: () => console.log('hello')
}

What happens instead?
instead, await myElement.getProperty('someProp') returns:
{
  propOne: 'abc'
  // methodOne is missing!
}

I bet getProperty uses the the structured clone algorithm. I understand if this is a no-fix issue. But it does prevent me from using e2e test code that works fine in the browser directly. That is, myElement['someProp'] in the browser returns the correct result. So I'm not sure how I should design my stencil component differently to accomodate the default e2e test browser (puppeteer).
In my case, the presence of a function causes different DOM elements to be rendered and the elements in the DOM are referenced in my tests.

I, ended up wrapping the function in an object and I now check to make sure the object is truthy rather than the function. This is still a shallow way to make it pass that, at some point, would get in the way of a true e2e test.. But for my purposes, the scope provided by the object is ultimately where I saw my code going anyway to incorporate more functionality on my roadmap.
I am curious though if there is a solution to this issue.


Modloader gives error when loading the same mod twice.

I get an error with the modloader, i try to load the same mod in different files. i am not so sure how good the idea is in general but it seems the modloader does not play well with it: i thing you should check if  loadableMods.Add(vertex.name, vertex); contains the mod already or dont do it with .Add the output is:
[= INFO =] Loading RadioActiveMod, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=null
[= INFO =] Loading RadioActiveMod, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=null
[= INFO =] Loading RadioActiveMod, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=null
[= INFO =] Loading ModHelper, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=null
ArgumentException: element already exists
at System.Collections.Generic.SortedList2[System.String,ModLoader.DependencyGraph+Vertex].PutImpl (System.String key, ModLoader.Vertex value, Boolean overwrite) [0x00000] in :0
at System.Collections.Generic.SortedList2[System.String,ModLoader.DependencyGraph+Vertex].Add (System.String key, ModLoader.Vertex value) [0x00000] in :0
at ModLoader.DependencyGraph.TopologicalSort () [0x00000] in :0
at ModLoader.ModLoader.Start () [0x00000] in :0
at ModLoader.Activator.Activate () [0x00000] in :0
at (wrapper managed-to-native) System.Reflection.MonoMethod:InternalInvoke (object,object[],System.Exception&)
at System.Reflection.MonoMethod.Invoke (System.Object obj, BindingFlags invokeAttr, System.Reflection.Binder binder, System.Object[] parameters, System.Globalization.CultureInfo culture) [0x00000] in :0
Rethrow as TargetInvocationException: Exception has been thrown by the target of an invocation.
at System.Reflection.MonoMethod.Invoke (System.Object obj, BindingFlags invokeAttr, System.Reflection.Binder binder, System.Object[] parameters, System.Globalization.CultureInfo culture) [0x00000] in :0
at System.Reflection.MethodBase.Invoke (System.Object obj, System.Object[] parameters) [0x00000] in :0
at LaunchInitializer.Update () [0x00000] in :0

Nevermind, i made up my mind i will now do it with only one modfile.

Ok


containerapp add preview flag for springboot

This checklist is used to make sure that common guidelines for a pull request are followed.

For new extensions:

[ ] My extension description/summary conforms to the Extension Summary Guidelines.

About Extension Publish
There is a pipeline to automatically build, upload and publish extension wheels.
Once your pull request is merged into main branch, a new pull request will be created to update src/index.json automatically.
You only need to update the version information in file setup.py and historical information in file HISTORY.rst in your PR but do not modify src/index.json.

Thank you for your contribution! We will review the pull request and get back to you soon.

Hi @zhoxing-ms
This pr has been review by hong and zunli. Could you please help to merge this small pr?


logical group top n: prune_col may transform logical group top n into logical top n

Describe the bug
prune_col will generate logical top n here:
https://github.com/risingwavelabs/risingwave/blob/1c3cce7e43de9d5d2f96062f6043c944ddc9b659/src/frontend/src/optimizer/plan_node/logical_topn.rs#L321
in spite of the fact that self may be a logical group top n
To Reproduce
No response
Expected behavior
prune_col will should generate logical group top n for logical group top n.
Additional context
No response

fixed in #6595


Contents deleted after moving the column

Describe the bug
When I use the embed function to insert images, moving that column causes all the contents within the column to be cleared.
Steps to reproduce
moving the embed column -> all contents in the column  has been cleared.
Expected behavior
Just move the column
Are you using the mobile app?
No
Obsidian debug info

Relevant log output
No response

@h-ruii Hi, I have not been able to reproduce the bug. Can you attach a more complete reproduction case?


SCP: Use last block hash in computeHashNode

The random seed is not stable and changes as new pre-images get externalized,
so it could be used to influence a node, while the previous block hash is stable.

What does stable mean in this context?

It has the potential to seemingly randomly change during a round as new pre-images get externalized.

What does stable mean in this context?

During the nominations 'computeHashNode' is called as part of something called 'leader selection', and the return of this function influences who will be the leader at a particular round.
This method was not stable, because new preimages could have been received during the nomination, and this method's return value was depending on the new preimages received.

Thank you for the explanation!