Use Patterns on AcmDonutChart

Signed-off-by: Chunxi Luo<EMAIL_ADDRESS>https://github.com/stolostron/backlog/issues/26194

/retest

/retest

/cc @KevinFCormier

/cc @heyethankim

/cc @KevinFCormier @heyethankim
Let me know any feedback

@ChunxiAlexLuo
As I noted in the issue, the intention was to enhance the experience including color blind people. But as you can see below, it doesn't solve the issue and made challenging even for non-color blind users. Let me discuss with @imjoyjean and get back to you.
Here is the link you could try out the color blindness (choose Red-blind or Green-blind) :
https://www.color-blindness.com/coblis-color-blindness-simulator/

@ChunxiAlexLuo As I noted in the issue, the intention was to enhance the experience for color blind people. But as you can see below, it doesn't solve the issue and made challenging even for non-color blind users. Let me discuss with @imjoyjean and get back to you.
Here is the link you could try out the color blindness (choose Red-blind or Green-blind) : https://www.color-blindness.com/coblis-color-blindness-simulator/

The above is the default Pattern stylish, let me know if there is any specific UI design on this.

/hold

https://github.com/stolostron/backlog/issues/26194#issuecomment-1317334745
Close as we decide to use other color options rather than Patterns


Scene transitions (pushing popping multiple scenes at once)

What's a sensible way of handling transitions?
I added a FadeIn/FadeOut Scene, the problem is that once you pop the scene, the scene underneath is drawn for a frame before that is popped, so a fade to black would suddenly flicker once before being popped off.
I solved this by adding Push/PopMultiple but I'm unsure whether this is a good idea? see https://github.com/ggez/ggez-goodies/compare/master...joetsoi:push-multiple

Heh, now that I think back, I had this exact problem and I think I resolved it by having my FadeOut scene be a Replace and actually consuming the old scene and drawing it as necessary.  So you'd have:
[Combat]
// Replace a fadeout, the fadeout gets ownership of the old scene that got popped,
// and draws it on its own
[FadeOut (containing old Combat scene)]
// Fadeout replaces itself with main menu
[MainMenu]

Pushing and popping multiple scenes might honestly be simpler.  XD

Finally found some time for this. I've run into a problem that I'm calling update https://github.com/ggez/ggez-goodies/blob/master/src/scene.rs#L32
so if I try to emit a replace
fn update(&mut self, mut game: &mut Game, ctx: &mut Context) -> FSceneSwitch {
     // if self.done {
     SceneSwitch::Replace(self)
}

which isn't possible, since I have a mutable reference to self. So I could clone the scene, but this seems quite wasteful, especially if there is a lot of state in the scene.
The other way I looked into would be to Pop, but there's nowhere to consume that popped scene as it's just discarded https://github.com/ggez/ggez-goodies/blob/master/src/scene.rs#L133
This isn't a problem for a fade in, as you're fading into a new scene.
I was also worried about the static lifetimein https://github.com/ggez/ggez-goodies/blob/master/src/scene.rs#L52 , but that might be completely unfounded


Hint for codemod should highlight command in backticks and/or colored

Right now we print a hint that people can update their code using the codemod:

I suggest to either:

add backticks to the suggested command `npx @prisma/codemods update-2.12 ./`
color the suggested command so that it stands out a bit more, e.g. in green

We could also consider doing both.

We already use colors, so for some reason the color is not visible in your case.
We can just add backticks additionally to make it work in non-color environments.
Action items:

[ ] Put the npx command in new line - easier to copy
[ ] Add backticks


cant's connect  SQL Server

[SqlException (0x80131904): 在与 SQL Server 建立连接时出现与网络相关的或特定于实例的错误。未找到或无法访问服务器。请验证实例名称是否正确并且 SQL Server 已配置为允许远程连接。 (provider: Named Pipes Provider, error: 40 - 无法打开到 SQL Server 的连接)]

Have you checked your connection string?
BTW, please post your error message in English, so @YodasMyDad can understand your question.
Thanks

Thanks,I am in China，Haha，I'll be careful next time

Named Pipes Provider, error: 40  normally means your application can not connect to your database.  Review your connectionstring.

@wendy2017666 Hi boy,Did you solved the problem of SqlConnection ?Could you tell me?I meet the same.


End of support is the same as the release date...

Release date for 0.8.0 = 6/24/2021
End of support date for 0.8.0 = 6/24/2021
Surely not?

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

ID: 67d1132c-974a-7dbc-6c46-2ebbc47e9944
Version Independent ID: 732091d2-ebcd-dc09-7ed1-ac339ecf834d
Content: Windows App SDK release channels - Windows apps
Content Source: hub/apps/windows-app-sdk/release-channels.md
Product: reunion
Technology: get-started
GitHub Login: @zaryaf
Microsoft Alias: zafaraj

Thank you for reporting this. The correct end of support date for that release is 6/24/2022. We have updated the article.


Initialize signer lazily

By initializing the signer lazily, only when used the first time, the check for missing secret_key and principal_id is also postponed until an actual API call to Azure is made.
This makes it possible to load the Rails environment without setting the Azure Storage credentials in environments that do not need to connect to Azure. For example, a build step that precompiles assets needs to load the environments, but usually does not need Azure Storage. Before this change, one would still have to set some (possibly fake) secret_key or principal_id in such build step, while with this change those can be omitted in any environment in which no real calls to the Azure API will be made.

The last two weeks were pretty buzy, I'll review soon.

Initially, I was loading on startup intentionally so that no Rails server could start and pass a health check (potentially killing the previous deployment) and then fail on uploads.
Though I didn't consider asset precompilation. I think a lazy option would make sense.


Fix alignment of fiat value tooltip icon in Account details

I will wait for Don's review in this case.


Design a new banner for the repo README

Feature request-
New banner needs to be designed for the project. Current banner has our older logo thus the need to change it.

Note:

Do mention if your are working under GSSoC'21.
If you want to work on an issue, you should check if it has already been assigned to anyone. If the issue is free you can comment /assign to get the issue assigned to you.
If you are raising a new issue and want to work on it then also you should comment /assign under the issue to get it auto assigned.
Please refrain from adding labels to your issue/pull-request on your own. It is the job of the Project Admin and the Mentors to review your issue/pull-request and add labels accordingly.
According to the level of the issue, respective time will be allotted under which the task needs to be completed/resolved by the assigned person. If the issue is not resolved within the given time, it will be assigned to the next person. In case of any discrepancies, the decision of the maintainer will be final.

level - 0: 2 days
level - 1: 3 days
level - 2: 4 days
level - 3: 5 days

/assign

@Pranav016 Please take a look at this and let me know if yo like it or should I try something else

Something else

Do you have something in mind ?

Do you have something in mind ?

Something similar to our current banner..
Current banner is very stylish.

Do you have something in mind ?

Something similar to our current banner..
Current banner is very stylish.

Okay. I'll try something different then.

would like to work on this issue.

@Pranav016 What about this ?

Something like this? will try some more

Something like this? will try some more

Nope

@Pranav016 What about this ?

Better but the components are not blended, I can still see all the boundaries of the pictures that don't match the background..

@Pranav016 What about this ?

Better but the components are not blended, I can still see all the boundaries of the pictures that don't match the background..

The gray background is not a part of the banner. You are talking about the mongoose one right ?

This?

@aanchalrakheja  This one looks really nice, let's go with this one..

Ok. Will continue with it.

Can you please guide me a bit about how to go about updating it.


Podcasts: add 240, add new podcatchers

Added a commit to disable to localizations navigation links (similar to topics pages). Can revisit if there is demand for translations related to the podcast.

Skimmed the transcript and quickly checked the diff.  Didn't see any problems.

I listened to the Spaces a bit reading the transcript in parallel, then realized that my podcast catcher already had the actual episode. Switching over, my experience was hugely improved! :grin:
I noticed a few discrepancies, and slight misunderstandings of the transcriptionist. A couple of these actually change the meaning, the rest are just nits.

Took transcription suggestions from @Xekyo review, thanks! Squashed. 🚀


Opensearch serverless image

Is your feature request related to a problem? Please describe
It is handy to use the Opensearch docker image for local testing during development to get quick feedback if features will work against AWS Opensearch.
AWS Opensearch serverless recently became available.
It has a restricted set of api operations and plugins when compared to regular opensearch.
Using the (non-serverless) Opensearch docker image will not flush out usages of unsupported operations while in dev.
Describe the solution you'd like
A docker image that has the same limitations as AWS Opensearch Serverless.
Describe alternatives you've considered
I looked around to see if AWS provide a docker image for serverless, but could not find one.
Apologies if this request doesn't really belong here (since it is not directly affiliated with AWS).
But still, any advice would be appreciated 🙏 :)
Additional context
-

I think an in image that only includes Serverless-compatible APIs is not a bad idea, however it might be tricky to implement. Serverless, for example, doesn't have the concept of nodes, or has SigV4. I would open a feature request with AWS support either way.

I think an in image that only includes Serverless-compatible APIs is not a bad idea, however it might be tricky to implement. Serverless, for example, doesn't have the concept of nodes, or has SigV4. I would open a feature request with AWS support either way.

Right! Restricting specific APIs should come from underlying software. Docker is one of the distribution created using the product (tarball today). In order to create such image basic requirement would be to have a opensearch-serverless as a separate product.
Moving this request to OpenSearch repository to see if they have any future plans for this.
Thanks!
cc: @peterzhuamazon  @bbarani @CEHENKLE


Here is no description how to deal with disk security issues!

A simple example: I moved data HDD from one system (Win2012 R2) to another (Win 2016).
But now programs does not work because some disk permissions do not match 'some tricky conditions'.
 So, I did not configured anything special. All folders structure were created from Windows Explorer.

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

ID: a1523bcb-77d6-5e4c-bdd7-e71598ef43ef
Version Independent ID: 9dbb2dcb-eaac-18d4-b324-f0130ba7a1da
Content: Move disks to another computer
Content Source: WindowsServerDocs/storage/disk-management/move-disks-to-another-computer.md
Product: windows-server-threshold
Technology: storage
GitHub Login: @JasonGerend
Microsoft Alias: jgerend

Thanks for your dedication to our documentation. If we have been unable to review your issue in a timely manner, we sincerely apologize for the delayed response. We have captured the informtion you have provided for review. We are closing this issue for now, but if you feel that it's still a concern, please respond and let us know. If you determine another possible update to our documentation, please don't hesitate to reach out again. #please-close


Add fixture: Varytec Hero Spot Wash 80 2in1 RGBW+W

With this, I am adding the moving head with spot and wash function, including the related Gobos, where only slightly similar ones existed but did not fully match. These gobos are in fact also used in other moving heads (such as in the Showtec Shark series), but did not yet exist in the gobos library yet.
The manual can be found here: https://images.static-thomann.de/pics/atg/atgdata/document/manual/451570_c_451570_v2_en_online.pdf
Do we want to provide PNG files next to all SVG files to make them better identifiable in Windows (Explorer gallery view)? It could be accomplished with a bash script:
for i in resources/gobos/*/*.svg; do if [ ! -f $(basename "$i" .svg).png ]; then echo "Missing; $i"; inkscape -z -w 64 -h 64 "$i" -e $(basename "$i" .svg).png; fi; done

Hello @mcallegari, the build failure seems unrelated as the stdlib.h is not found. Can you re-trigger the build or still merge or do I need to push some unrelated change?

Seems like Appveyor updated gcc (v. 11) and Qt is no longer compatible with it. Nice.
You don't need to do anything. That's another thing to fix on me.

Hi, sorry, I just created a conflict with this.
I also realized you were trying to push PNG gobos... Please don't. They're not compatible with QLC+ 5.
All of the Hero Spot Wash 80 gobos are already in the library (I just checked)
Please use the existing ones, thanks

As for SVG previews:
https://joshuawinn.com/how-to-preview-svg-files-in-windows-10-file-explorer-without-3rd-party-extensions/

This should be it
 <Channel Name="Gobo">
  <Group Byte="0">Gobo</Group>
  <Capability Min="0" Max="10" Preset="GoboMacro" Res1="Others/open.svg">Open</Capability>
  <Capability Min="11" Max="20" Preset="GoboMacro" Res1="Robe/gobo00025.svg">Gobo 1</Capability>
  <Capability Min="21" Max="30" Preset="GoboMacro" Res1="SGM/gobo00008.svg">Gobo 2</Capability>
  <Capability Min="31" Max="40" Preset="GoboMacro" Res1="Others/gobo00108.svg">Gobo 3</Capability>
  <Capability Min="41" Max="50" Preset="GoboMacro" Res1="Others/gobo00112.svg">Gobo 4</Capability>
  <Capability Min="51" Max="60" Preset="GoboMacro" Res1="Chauvet/gobo00034.svg">Gobo 5</Capability>
  <Capability Min="61" Max="70" Preset="GoboMacro" Res1="Others/gobo00125.svg">Gobo 6</Capability>
  <Capability Min="71" Max="80" Preset="GoboMacro" Res1="Others/gobo00126.svg">Gobo 7</Capability>
  <Capability Min="81" Max="90" Preset="GoboShakeMacro" Res1="Robe/gobo00025.svg">Gobo 1 Shake</Capability>
  <Capability Min="91" Max="100" Preset="GoboShakeMacro" Res1="SGM/gobo00008.svg">Gobo 2 Shake</Capability>
  <Capability Min="101" Max="110" Preset="GoboShakeMacro" Res1="Others/gobo00108.svg">Gobo 3 Shake</Capability>
  <Capability Min="111" Max="120" Preset="GoboShakeMacro" Res1="Others/gobo00112.svg">Gobo 4 Shake</Capability>
  <Capability Min="121" Max="130" Preset="GoboShakeMacro" Res1="Chauvet/gobo00034.svg">Gobo 5 Shake</Capability>
  <Capability Min="131" Max="140" Preset="GoboShakeMacro" Res1="Others/gobo00125.svg">Gobo 6 Shake</Capability>
  <Capability Min="141" Max="150" Preset="GoboShakeMacro" Res1="Others/gobo00126.svg">Gobo 7 Shake</Capability>
  <Capability Min="151" Max="255" Preset="RotationClockwiseFastToSlow">Gobo Rotation</Capability>
 </Channel>

Thanks, I will fix these issues in a week.

Hello @mcallegari
I changed the gobos as per your proposal and removed the extra files.
Please also check the (usefulness of) the added line in resources/fixtures/FixturesMap.xml as I am not completely sure if that should be there or if the entry in the fixture file itself would be sufficient. Should I remove the line or should it stay?

I was also wondering if the spot and wash would have to be represented in two heads, but it seems, putting them into one head works for me.

Thanks for removing the leftover channels with your review. I was blind by looking at it too long and forgot to remove them at the end.


config broken?

I was trying to configure the plugin for the first time and kept on getting this error:
Failed to run config for iron.nvim
.../xxx/.local/share/nvim/lazy/iron.nvim/lua/iron/core.lua:458: attempt to call field 'nvim__set_hl_ns' (a nil value)
stacktrace:

/iron.nvim/lua/iron/core.lua:458 in setup
nvim/lua/plugins/iron.lua:14 in config

I think you have any extra '_' in the name of the function vim_set_hl_ns. When I edit it, it seems to work.

You can try the latest master branch.

I think that's an old API, from when it was experimental. I can try to have a look, though I'm out on vacation. I'll try to get back at this later today or during the week.


[Feature]: Datasource image should be bundled

Is there an existing issue for this?

[X] I have searched the existing issues

Description
When running appsmith in docker on premise the svg's for datasource are hardcoded to lad from assets.appsmith.com and thus not displayed
Steps To Reproduce
Run docker on premise and check datasources
Public Sample App
No response
Version
selfhosted - 1.7

@pkrolkgp First of all, please accept our sincere apologies for getting to this late. Can you please add more details on the problem statement you are looking to solve for, we might have a workaround for you in store.


Ability to get a list of keyspaces and keys from datacake_eventual_consistency

The underlying Storage trait requires the get_keyspace_list and the iter_metadata methods.  However, these methods are not exposed to an application using datacake_eventual_consistency.
Is this an intentional design choice, am I missing the intended way to access the list of keyspaces and keys from the underlying storage, or is this something that needs to be built out?

My intent is provide to new public methods in the eventual consistency layer to allow applications using the layer to discover the available keyspaces and keys.
After reviewing the associated underlying code, it seemed most prudent not to expose an iterator of the metadata.
After reviewing other implementations of similar systems, the typical approach is to return a collection of the keys.

Is this an intentional design choice, am I missing the intended way to access the list of keyspaces and keys from the underlying storage, or is this something that needs to be built out?

The methods are more for the internals of datacake rather than for a publically accessible API. They're there so datacake can recreate its internal memory state.
The only reason why I'm somewhat hesitant to expose these methods is because they can't give you the 'full' list potentially, they can only give you the state that it's aware of locally rather than across the whole cluster.
Another thing to mention (maybe it's worth improving the docs on;) is that keyspaces should be fairly static rather than dynamically adding them. Although it's not necessarily bad, you can't remove keyspaces once they're made, and past a certain point, you probably limit your performance quite heavily (In the event of there being millions of keyspaces).
For example in lnx there are effectively 4 static keyspaces:

metadata/config
schemas/settings
schemas/schema
data/doc-blocks

Now in reality there are technically more of these keyspaces because each keyspace is effectively one concurrent lane, so if you have say 12 keyspaces, you can do up to 12 operations concurrently if you split the data across them.
So in lnx's case it has 128 data/doc-block keyspaces to maximise its replication speed, but these are fixed and aren't added dynamically because technically you can end up in a situation where the number of keyspaces just increases forever.
I may be able to improve this in later versions if the state is completely empty, we can probably remove the keyspace from the internal state. But this is quite rare in normal situations.


Gas estimate is too high

When sending a transaction from the frontend, the recommended gas fees are ~3 times higher than the MetaMask market estimate.

@alainncls
Hi there,
I attempted to reproduce your issue and was unable to do so.
We use GitHub issues as a place to track bugs and other development-related issues. If your issue persists, please write us back using our “Chat Support” icon located on our Help Center
You can include a link to this issue in the message content.
Alternatively, you can also search for an answer in our help documentation or get help from other users on our community forums.
The issue here will be closed.
Thanks!


Pairing stops after step 1 on iOS 10.3 beta

I have been using HAP-Nodejs on a Raspberry Pi successfully for several months now. I recently tried to add a custom camera to my setup but was unable to complete pairing with the device. Using debug I found that step 1 of the pairing process would show up in the output but after entering the pin code it never proceeded, eventually it would time out. I tried updating all of the modules, rebuilding, and even renaming the device, all in vain. As a last resort I deleted my persist cache and removed all devices from the Home app, but now I can't finish pairing with the accessories I previously had working fine. I have even reverted back to only the accessories before this whole problem began with no effect. The only thing I can think of that could be causing the issue is that I am using the iOS 10.3 public beta released last week and that it has somehow broken compatibility.

I just tried with 10.3 and it's working properly. Please make sure your accessory is returning correct values.

And I also e works perfectly with iOS 10.3 beta 1

i tested hap-nodejs on multiple devices. however it looks like sometimes a iphone/hap server combo can be stuck. i cannot discover one device on my iphone (deleted all data i could on iphone and hap server), but the same device can be discovered on another iphone. sounds like a ios bug, where some stalled data exists. someone else discovered a similar issue?


Roll Flutter Engine from 32c468507b32 to d7dbe5bf37f4 (5 revisions)

https://github.com/flutter/engine/compare/32c468507b32...d7dbe5bf37f4
2023-01-04<EMAIL_ADDRESS>Roll Skia from 9423a8a0fc2d to 60e4a4a27375 (5 revisions) (flutter/engine#38633)
2023-01-04<EMAIL_ADDRESS>Update FlutterPlatformNodeDelegate (flutter/engine#38615)
2023-01-04<EMAIL_ADDRESS>Roll Skia from 15d36b15bca1 to 9423a8a0fc2d (37 revisions) (flutter/engine#38631)
2023-01-04<EMAIL_ADDRESS>Roll Dart SDK from f6dcb8b0b5d3 to 0b064bc49557 (1 revision) (flutter/engine#38630)
2023-01-04<EMAIL_ADDRESS>Roll Dart SDK from 617e70c95f5b to f6dcb8b0b5d3 (7 revisions) (flutter/engine#38626)
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose
To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

Commit queue failed; closing this roll.


Under the light theme, some toolbars are transparent.

IDE:CLion 2023.2.2
Theme: sky

Hi. This look like a color bug with new UI, because plugin didn't change icons color in themes.
I saw similar ticket on youtrack: https://youtrack.jetbrains.com/issue/IDEA-307187/EJB-Icon-stays-gray-when-highlighted-new-UI Could you switch to "Intelij Light" and check if icon is same gray color.

Hi. This look like a color bug with new UI, because plugin didn't change icons color in themes.
Could you switch to "Intelij Light" and check if icon is same gray color.

Okay, thank you.

I check this issue with default light theme has same problem, the icon color is hard to see. I will report this on youtrack later.

This issue is fixed in latest version. Please try after upgrade :)

This issue is fixed in latest plugin version. Please try after upgrade plugin. :)


Fix dynamic connector extensions & update internal testing

Fix dynamic connector extensions management in making 'parameters' optional despite the spec which says it's required
Update internal tests
Add Guardian files
Update .gitignore file
Fix BuildLocalPackages.cmd script

✅ No public API change.

✅ No public API change.

✅ No public API change.


heap and memory question ?

-official esp8266 RTOS sdk use heap_5.c memory management for freertos. esp-open-rtos too or another model ?

"malloc" and "free"  will allocate memory from the task heap (inside) ? or oustide of the task ?
if a task "malloc" memory, other task can manipulate this memory ?
i got a situation : one task "malloc" memory, send memory ptr with a queue to another task and this task after treatments "free" this memory. the memory is "free" truly ?

I got some memory problem.

if i do this:
#define "MY TEXT HERE"
and use it in my function, the text is on ROM memory or RAM ?
when i do this:
const char* text = "MY TEXT HERE"
The text is in ROM ?
I call multiple function with text and think the compiler will stock these text in ROM. its good ?
if not, can i force put them in ROM ?

#define configTOTAL_HEAP_SIZE		( ( size_t ) ( 32 * 1024 ) )
Max heap can be change ? i remenber, that an update of linker from @ourairquality  save a lot of RAM. (Maybe not the same RAM)

Something interesting, the official RTOS SDK got ~70KB heap
SDK version:1.5.0-dev(a8c4880)
Heap :69984

I think that espressif use Instruction RAM/FLASH cache ram. get more heap.
esp8266 memory map
When we look last commit of them:

Change cache size from 32KB to 16KB, reverse this 16KB region as heap;
Someone know how to disable cache for this region: 0x40108000 ? if we can do this,  it will allow 16KB heap more. (if we not change other things).

After compare DPORT register for both SDK:
esp-open-rtos:
DPORT SPI_CACHE_RAM         3FF00024      0000001A
esp8266 rtos sdk:
DPORT SPI_CACHE_RAM         3FF00024      0000000A
Maybe we just need to setup this register to get more 16Kb more heap.

There are some clues in the include files:
/* Details for SPI_CACHE_RAM register */
#define DPORT_SPI_CACHE_RAM_BANK1  BIT(3)
#define DPORT_SPI_CACHE_RAM_BANK0  BIT(4)

So bit 4 has changed to be zero, and the include file suggests this disables bank 0 of the spi cache ram. Bank zero is probably 0x40108000 to 0x4010c000. They use the FreeRTOS heap5 allocator which can handle separate regions, and it seems to use a region that starts at the end of the used iram (and so also uses the remaining iram) and runs to 0x4010c000 (the end of bank 0).
The comments suggest some limitations using this cache ram, that it can not be a TX buffer, or task stack. The TX buffer limitation might be a hardware issue. Not sure why the task stack could not use this area.
Perhaps another option would be to use it as iram, to have the linker use 0x40100000 to 0x4010c000 for iram, to be able to link some code there if an app really needed that for speed etc.
Perhaps Cache_Read_Enable sets up this register, and we would need a custom version.

After do some test, i valid:  Cache_Read_Enable setup the cache register. Unfortunately, this function must initialize or  set other parts that I do not know.
They use heap5 and get more heap like 72Kb setting. They use more ram region to get this value.(where ?) . But if the RAM/CACHE is following the 32K ram region, We need really to change heap system ? or can we just increase heap size. (and linker update )

i found this
cache enable C version

After replace  Cache_Read_Enable function by the C version above and replace
Cache_Read_Enable(0, 0, 1);  by   Cache_Read_Enable(0, 0, 0); everywhere.
I got a succes !
DPORT SPI_CACHE_RAM 3FF00024 0000000A

Perhaps it is not necessary to replace Cache_Read_Enable rather as you did just replace the third argument with 0, and perhaps that could be done in core/spiflash-cache-enable.S by changing the a4 value passed to the ROM function Cache_Read_Enable to zero.

When i test the original function, put the third arg at 0 didn't change.
	/* map the first megabyte of flash */
	movi a2, 0
	movi a3, 0
	movi a4, 0

I clean and rebuild. But no effect.

The attached patch appears adequate here. With this it was possible to use 0x40108000 to 0x40108000 as scratch ram in a simple test.
patch.txt

However only aligned 32 bit memory read operations appear to work in this region. Even aligned 8 and 16 bit write operations fault. There does appear to be code in the exception handlers to deal with this, but it appears to only implement the read opcodes. So do we need to extend that exception handler to deal with store opcodes too, or is there some hardware workaround? If only 32 bit operators are efficient in this region then it might have an even bigger performance hit.

Perhaps it's connected as instruction RAM (which only can be accessed by 32-bit store/loads)? And when it's used as cache, an external cache controller is used (otherwise it's hard to imagine how can it work both as SPI cache and RAM)

After test, Heap is on "dram0" size. dram  : 81920 . After printf "sdk_system_print_meminfo" (it is buggy, i will fix it). i got this information
data  : 0x3ffe8000 ~ 0x3ffe8964, len: 2404
rodata: 0x3ffe8968 ~ 0x3ffe9258, len: 2288
bss   : 0x3ffe9258 ~ 0x3fff1500, len: 33448
heap  : 0x3fff1500 ~ 0x3fffc000, len: 43776
//
i guess espressif use heap_5 to add iram1:
iram1 : 32768   =>Instruction RAM (40100000h - 40108000h)
?Seg0 : 16384  =>RAM/FLASH CACHE (40108000h - 4010C000h)
//Limitation of iram is do not allocate task and TX buffer. (So we can use it for queue,buffers,stream...)

RTOS ESPRESSIF SDK information:
data  : 0x3ffe8000 ~ 0x3ffe8850, len: 2128
rodata: 0x3ffe8850 ~ 0x3ffe8d44, len: 1268
bss   : 0x3ffe8d48 ~ 0x3ffef990, len: 27720
heap  : 0x3ffef990 ~ 0x40000000, len: 67184
interesting map of heap. They use additionnal 4K from this segment
ETS system data RAM (0x3FFFC000 - 0x40000000)
i think they use only this part of iram as heap too:
?Seg0 : 16384 =>RAM/FLASH CACHE (40108000h - 4010C000h)

@ourairquality Ty for your patch, it is work well.
was looking to add last 4K Dram segment.
we need change ld script
dram0_0_seg : org = 0x3FFE8000, len = 0x14000
to
dram0_0_seg : org = 0x3FFE8000, len = 0x18000
I dont know what is the process to allocate memory. The change above don't change the size of heap. I guess it is "malloc" from libc that allocate memory  and know the limit  and we need recompile libc ?
pvPortMalloc = malloc;
vPortFree = free;

Much of that top 4k is still used, it is just not made available to the linker to use. The malloc area starts from the _heap_start defined by the linker and ending at the xPortSupervisorStackPointer defined at startup which is in that top 4k, so malloc can use what is not otherwise used.
Not sure what is above that. Tried writing over the area above the xPortSupervisorStackPointer and checking if it changes and it does not appear to be used.
Assuming it is the startup stack, then once scheduling is started it is probably no longer needed and perhaps the available heap could be increased to 0x40000000 rather that just the SP when scheduling was started?
Would anyone know what that top area of RAM is used for and if we could malloc over it?

Here's a quick patch to make all of the top area of RAM available for allocation once scheduling has started. This assumes that the area was all the start up stack and is no longer needed - an assumption not checked!
heap-end-patch.txt

From RTOS sdk:
 NEW VERSION: 1.3.0
17. Modify "ld" files, "dram0 len" should be 0x18000 in RTOS SDK.

Why they said "should be" ?
Your changes got effect. I do a program to malloc memory template until memory is full. and write a template into this memory. I got no crash. Maybe is a good news .
But total heap is only increase of 1000?
Before:
MAIN: Heap: 40588
heap : 0x3fff1520 ~ 0x3fffc000, len: 43744

Now:
MAIN: Heap: 41868
heap  : 0x3fff1520 ~ 0x40000000, len: 60128

Its maybe normal. I didnt know well how it is work.

the first heap is value from xPortGetFreeHeapSize()
the second heap value from sdk_system_print_meminfo() (I got a fixed one, the original is buggy, miss &)

I found three interesting article about memory and FreeRTOS / Newlib.
FreeRTOS heap 5
FreeRTOS newlib
newlib integration FreeRTOS
the esp-open-rtos project use the memory scheme of newlib ? f it is the case, why do not use heap_5 from Freertos ?
I use PR #519 and #523 in my project. I go not memory problem for now.

I recently update esp-open-rtos with lock implementation than use "configUSE_RECURSIVE_MUTEXES". This implementation use 1k heap.
lock functions are used by printf and others functions that i don't know.
Can we get a preprocessor option to disable recursive mutexes (swap with old system ?) without cause troubles and  disable functions usage like printf.


[docs] update README to fix docker run command flag order

fixes: #1924

@PeterDaveHello, your thoughts?

LGTM, that was a mistake.


Support the LBA 2 data in DOS format (like version from GoG.com)

I managed to unpack the LBA2.GOG file. It's not hard - just run the dosbox and mount the folder containing LBA2.GOG e.g. to C: drive (my paths examples are for MacOS).
mount C "/Users/foo/myPathToFolderWithLBA2Files"
After that mount the image to D drive
c:
imgmount d "LBA2.GOG" -t iso
d:
dir

Then you will see all the files. You need the whole LBA2 folder. To copy it recursively to C: drive where your folder is mounted you need to use some tool or script, as MS-DOS copy doesn't support recursive copy of the folders. I have used Volkov Commander from dosbox. You can grab it from here: https://archive.org/details/TheVolkovCommander_1020
So now this LBA2 folder is ready to be placed as www/data
However, there are many issues as e.g. VOX files are not in compatible name and format. All my VOX files were named FR_000.VOX etc, while game expects FR_000_AAC.VOX, etc.
Just renaming them doesn't help - the script doesn't reconize the format, so I had to download original files from the demo web resources. The same will happen with SAMPLES_AAC.HQR that in DOS version are just SAMPLES.HQR.
After that, and after downloading some mp4 files, the game started to work for me.
So yes, you can mostly use DOS version from GoG, but there will be some issues with some of the files. The engine is not ready to support the DOS version of LBA 2 yet, and that can be an issue for people like me that own GoG version. Maybe a separate issue should be created for this.
Originally posted by @pleasenophp in https://github.com/agrande/lba2remake/issues/4#issuecomment-594886730

I'm not sure what you call the DOS format, but if I recall correctly the GOG version uses a bin/cue image like described in the original issue #4. I'm using the GOG version myself.
The VOX files are renamed because we don't have a way of decoding and playing the original audio format used in the game, so we need to extract all the HQR entries (VOX files are HQR files) convert them to a format we can use (AAC) and repackage them.
We have done it manually for our local versions and for the demo server, but the whole point of issue #4 is to find a way of doing it in code, so future contributors don't need to download it from the demo server. And it would be best if we could do this in a cross-platform way, without requiring external tools.
Feel free to tell me if I misunderstood something here. Otherwise I'll close this issue as it's basically a duplicate of #4.

The VOX files are renamed because we don't have a way of decoding and playing the original audio format used in the game, so we need to extract all the HQR entries (VOX files are HQR files) convert them to a format we can use (AAC) and repackage them.

Ah, I didn't see anything about this step in the instructions on how to run the game locally. Is there any place where I can read it?
What about the SAMPLES.HQR? I also had to download SAMPLES_AAC.HQR
I don't know which other files I might need in future, as I only played a few maps. But it would be nice if you can give the full instructions of what files need to be modified / repacked in the GoG version. I will try to make a script that does it automatically.

Ok, I'll add a list of the files and what I and xesf did to convert them in the documentation.


Feature request add Soviet Linux support

I am the creator and principal dev of Soviet Linux , and we would really like to have neofetch showing our logo.


RADIUS support

Elevator pitch
A RADIUS server authenticates a user base on username and password. Normally they look up the username in a database or authenticate against something like LDAP/Active Directory.
What if the RADIUS server instead looked at the username to see if it contained an LNURLw or cashu token and approved or rejected the request based on the result of claiming the payment. The user could even come back with the spent LNURLw or cashu and use that as their username for future interactions.
Buzzwords: Payment instead of identity. Payment as identity. Just in time account creation
CLI DEMO
#install a radius client to debug and test
apt install -y freeradius-utils

#this password is used by the service that wants to authenticate a client
password_to_use_radius_server=tollgate

#we don't really care about the clients password, but for now this needs to be set to tollgate for the demo to work
client_password=tollgate

client_username_reject=PlzSirNeedWifi
radtest ${client_username_reject} ${client_password} radius.dns4sats.xyz 1812 ${password_to_use_radius_server}

Sent Access-Request Id 188 from <IP_ADDRESS>:40145 to <IP_ADDRESS>:1812 length 84
User-Name = "PlzSirNeedWifi"
User-Password = "tollgate"
NAS-IP-Address = <IP_ADDRESS>
NAS-Port = 1812
Message-Authenticator = 0x00
Cleartext-Password = "tollgate"
Received Access-Reject Id 188 from <IP_ADDRESS>:1812 to <IP_ADDRESS>:40145 length 20
(0) -: Expected Access-Accept got Access-Reject

client_username_accept=LNURL1DP68GURN8GHJ7ER9D4HJUMRWVF5HGUEWVDHK6TMHD96XSERJV9MJ7CTSDYHHVVF0D3H82UNV9AF5CNTD2939ZAZ4XDA9YUJ5XSE85AZ4FFQ5VT68FPUXYEN2FD6YCU690PQ4QVMD092KX3M02CWEGJK5
radtest ${client_username_accept} ${client_password} radius.dns4sats.xyz 1812 ${password_to_use_radius_server}

Sent Access-Request Id 25 from <IP_ADDRESS>:44068 to <IP_ADDRESS>:1812 length 228
User-Name = "LNURL1DP68GURN8GHJ7ER9D4HJUMRWVF5HGUEWVDHK6TMHD96XSERJV9MJ7CTSDYHHVVF0D3H82UNV9AF5CNTD2939ZAZ4XDA9YUJ5XSE85AZ4FFQ5VT68FPUXYEN2FD6YCU690PQ4QVMD092KX3M02CWEGJK5"
User-Password = "tollgate"
NAS-IP-Address = <IP_ADDRESS>
NAS-Port = 1812
Message-Authenticator = 0x00
Cleartext-Password = "tollgate"
Received Access-Accept Id 25 from <IP_ADDRESS>:1812 to <IP_ADDRESS>:44068 length 20

DEMO on wireless router (OpenWRT or any other router that supports WPA enterprise
Note that hostapd-mini and wpad-mini don't support WPA enterprise. You will need to opkg remove them and install the full version. Make sure to do this while connected to the router over a cable and not wifi or you will lock yourself out.
You should also be able to use any other non-openwrt access point that supports RADIUS authentification.
OpenWRT settings:
config wifi-iface
    option device 'radio0'
    option mode 'ap'
    option ssid 'TollGateRADIUS'
    option encryption 'wpa2'
    option auth_server 'radius.dns4sats.xyz'
    option auth_port '1812'
    option auth_secret 'tollgate'

Now you should be able to connect to the access point, enter LNURL or cashu as the username and connect to the network.
DEMO: OpenVPN
OpenVPN also supports RADIUS as the auth backend. Just point the OpenVPN server at the RADIUS server and the RADIUS server will approve or reject the connection depending on if it was able to claim the payment
DEMO: 802.1X: Port-Based Network Access Control demo
Many managed switches and routers support 802.1X, including OpenWRT. Computers too (Ubuntu, Windows, OsX)
The implications are that a computer could authenticate to an OpenWRT router or off the shelf managed switch and provide an LNURLw or cashu token as its username. The switch/router would forward the request to the RADIUS server which would claim the payment and notify the switch/router if it should turn the ethernet port on or off. No extra software is required on the computer or switch, all of the payment logic is done my the RADIUS server. It should work using the radius.dns4sats.xyz demo but has not yet been tested
DEMO: Logging in to Windows or Linux computers
Not tested, but should be totally doable. Could be used by a service like hashbang.sh
ssh<EMAIL_ADDRESS>
There is a project to configure PAM to use a RADIUS server, but this has not been tested yet
DEMO: Accessing a mobile network phone network using cashu/LNURLw
A smart mobile phone operator could look for client devices that provide cashu or LNURLw as their username when trying to connect to the mobile phone data network. Both iOS and Android allow you to specify a username and a password when connecting to the APN. The mobile phone operator could try to claim the username as a payment and then make a decision about how much data to provide.
This is a really big deal when roaming in foreign countries. Normally the roaming Telco provider would have to bill the home Telco provider for X gigabytes of usage. The home provider will likely charge the end user a lot more (sometimes many times more) then what the roaming provider is paid. If the roaming provider instead is able to receive payment directly from the end user, it could make more money then if it bills the home Telco provider. It is also able to collect the payment in realtime (actually up front, before providing service).
Feel free to add more use cases of RADIUS where payment like this could be usefull.

How to configure the radius server
Full writeup coming real soon (tm), but here are some notes:
This issue implements several configuration updates and enhancements in FreeRADIUS to improve security, add validation, and customize authentication. Below are the diffs explained and organized by file.

Added a universal client configuration for "all_clients" in clients.conf
+client all_clients {
+    ipaddr     = <IP_ADDRESS>/0
+    secret     = tollgate
+    shortname  = all
+}

Changed default EAP type to PEAP in mods-available/eap
-		default_eap_type = md5
+#		default_eap_type = md5
+                default_eap_type = peap
+

Added a new module validate_username in mods-available/validate_username to validate usernames using a shell script
+exec validate_username {
+    wait = yes
+    program = "/usr/local/bin/validate_username.sh %{User-Name}"
+    input_pairs = request
+    output_pairs = reply
+    shell_escape = yes
+}

Added user authorization entries in mods-config/files/authorize
+a   Cleartext-Password := "a"
+
+DEFAULT   Cleartext-Password := "tollgate"
+

Enabled validate_username module by creating a symlink in mods-enabled/validate_username
/etc/freeradius/3.0/mods-available/validate_username

Enabled logging for authentication attempts in radiusd.conf
-	auth = no
+	auth = yes

-	auth_badpass = no
-	auth_goodpass = no
+	auth_badpass = yes
+	auth_goodpass = yes

Integrated validate_username module in sites-available/default for authorization
+    validate_username
+if (validate_username == fail) {
+    reject
+}

Integrated validate_username module in sites-available/inner-tunnel for inner-tunnel authorization
+	    validate_username
+if (validate_username == fail) {
+    reject
+}

And finally, the logic that actually would claim the payment. For now it is not claiming any money, just validating that it looks like a string that might represent an actual payment using grep:
/usr/local/bin/validate_username.sh
USERNAME=${1}

# Ensure USERNAME contains only allowed characters
#if [[ ! "${USERNAME}" =~ ^[a-zA-Z0-9_-]+$ ]]; then
#    exit 1
#fi

#todo: also allow this kind of links:
#https://demo.lnbits.com/withdraw/api/v1/lnurl/cb/HWgBJw4kjfrPXjWctfxj7M

# Check if USERNAME contains 'cashu' or 'lnurlw'
#if [[ "${USERNAME}" == *"cashu"* ]]; then
if echo "${USERNAME}" | grep -qi "cashu"; then

    # For 'cashu' users
   echo "${USERNAME}" >> /tmp/cashu
    exit 0
#    echo "User-Category := 'cashu_user'"
#    echo "Max-Data-Rate := '30Mbps'"
#elif [[ "${USERNAME}" == *"lnurl"* ]]; then
elif echo "${USERNAME}" | grep -qi "lnurl"; then

    # For 'lnurlw' users
   echo "${USERNAME}" >> /tmp/lnurl
    exit 0
#    echo "User-Category := 'lnurlw_user'"
#    echo "Max-Data-Rate := '20Mbps'"
else
    # Username does not contain 'cashu' or 'lnurlw', reject authentication
    exit 1
fi

# Log the USERNAME to /tmp/log
echo "${USERNAME}" >> /tmp/log

exit 0

Thanks @Amperstrand , I'll render it as HTML and add it to the docs. Hoping to reproduce what you achieved sooner rather than later...

I linked the tutorial in with the rest of the website.. Thanks!
https://npub1suw0zfxerywd4zku4gjsjde22zhzye9dl2hsll6s3z2qap75p78s66lkhp.nsite.orangesync.tech/payments_in_radius_server.html
https://gitworkshop.dev/r/naddr1qvzqqqrhnypzqcnlucwjsudxcw0e48ghzzapkyq3udm93lkh7jp8sr6j4s8jjqx2qy88wumn8ghj7mn0wvhxcmmv9uqqs4r0d3kygmmrwvqpy6x0/proposals/note1qm33ncdahrdjayuu2vteep7ud20fulyu6n99vzv2ue75fwnl2gnqsemjus


RLE Support

Hi - I just implemented a DICOM RLE decoder in Rust which you can find here: https://github.com/chafey/dicomrle-rs
I wanted to let you know in case you want to use it for adding support for RLE decoding to this library.  I would also appreciate any feedback as I am new to Rust.  Thanks!

Nice project, thank you for sharing!
While it is not quite the time to think about a possible integration (it would be best to establish an image abstraction API first), one thing to keep an eye on is whether you can take advantage of the dicom-core and dicom-encoding crates for common DICOM structures and algorithms, rather than making them from scratch. In the event that something does not fit or is missing, then that could become a source of discussion and mutual improvement.

I don't see anything in dicom-core or dicom-encoding that will help with the DICOM RLE Decoder.  The DICOM RLE algorithm is quite simple and self contained, it doesn't have any dependencies on anything else in DICOM.  I'll go ahead and close this issue for now as there doesn't seem to be interest in integrating.


docs: update linting/prettier instruction

Added a small change to docs in response to #918 and formatted this time in response to #1767 ...oops

nice addition - thx 🤗


got an unexpected keyword argument 'language'"

Search before asking

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

Operating system information
Linux
Python version information
3.10
DB-GPT version
main
Related scenes

[X] Chat Data
[X] Chat Excel
[X] Chat DB
[X] Chat Knowledge
[X] Model Management
[X] Dashboard
[ ] Plugins

Installation Information

[X] Installation From Source

[ ] Docker Installation

[ ] Docker Compose Installation

[ ] Cluster Installation

[ ] AutoDL Image

[ ] Other

Device information
macos
Models information
LLM：gpt-3.5
What happened
File "/Users/raoyunfei/worker/DB-GPT/dbgpt/util/dbgpts/loader.py", line 451, in load_package
logger.warning(f"Load dbgpts package error: {e}", e)
Message: "Load dbgpts package error: SummaryAction.init() got an unexpected keyword argument 'language'"
Arguments: (TypeError("SummaryAction.init() got an unexpected keyword argument 'language'"),)
What you expected to happen
No error message, and the default Chinese reply
How to reproduce
Restart on startup
Additional context
No response
Are you willing to submit PR?

[ ] Yes I am willing to submit a PR!

hi @shanmu-raoyunfei ,can you reproduce how you operate?

hi @shanmu-raoyunfei ,can you reproduce how you operate?

I met the same problem.
on the Web app, I chose any one agent or workflow from menu "DBGPTS Community" and tried installed. There's success tip from the web.
But on the terminal, the exception happened and there's no just installedd the workflow app on the "AWEL flow" page.


asn: 0.76.1 -> 0.77.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/nitefood/asn/releases.
meta.description for asn is: OSINT command line tool for investigating network data
meta.homepage for asn is: https://github.com/nitefood/asn
meta.changelog for asn is: https://github.com/nitefood/asn/releases/tag/v0.77.0
Updates performed

Version update

To inspect upstream changes
Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 0.77.0 with grep in /nix/store/502s8920904gjayqla4sh336wd0z7x8p-asn-0.77.0
found 0.77.0 in filename of file in /nix/store/502s8920904gjayqla4sh336wd0z7x8p-asn-0.77.0

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

2 total rebuild path(s)

2 package rebuild(s)

First fifty rebuilds by attrpath

asn

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/502s8920904gjayqla4sh336wd0z7x8p-asn-0.77.0 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

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

Or:
nix build github:r-ryantm/nixpkgs/9891f68ac358fbf11facaca26b16af8d653488df#asn

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/502s8920904gjayqla4sh336wd0z7x8p-asn-0.77.0
ls -la /nix/store/502s8920904gjayqla4sh336wd0z7x8p-asn-0.77.0/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

asn

Maintainer pings
cc @devhell for testing.

@NixOS/nixpkgs-merge-bot merge


Please help: PANIC: unprotected error in call to Lua API (application.lua:1: unexpected symbol near ''ï'')

Please forgive me if this is an easy issue that I am just to much newb to know about it or figure it out.  I have googled the "unexpected symbol" error and found it suggested that it could be an issue with hidden characters due to encoding.  I copied your raw application.lua file and pasted it into visual basic and saved it encoded with UTF-8 without signature as application.lua.  Sadly, this made no difference...  About the 3rd time formatting and re-loading the .lua files it I got a the same error but with a "?" instead of the 'ï'.  Please help.  I'd really like to get this working.
Please see error in action below.
NodeMCU custom build by frightanic.com
branch: <IP_ADDRESS>-final
commit: 1885a30bd99aec3384**********************
SSL: true
modules: cjson,file,gpio,http,net,node,tmr,uart,wifi,tls
build 	built on: 2017-03-23 20:41
powered by Lua 5.1.4 on SDK <IP_ADDRESS>(39cb9a32)
Connecting to WiFi access point...

Waiting for IP address...
Waiting for IP address...
Waiting for IP address...
WiFi connection established, IP address: <IP_ADDRESS>
You have 3 seconds to abort
Waiting...
Running application...
PANIC: unprotected error in call to Lua API (application.lua:1: unexpected symbol near '?')

ets Jan  8 2013,rst cause:2, boot mode:(3,6)
load 0x40100000, len 24580, room 16
tail 4
chksum 0xaf
load 0x3ffe8000, len 2260, room 4
tail 0
chksum 0xce
load 0x3ffe88d4, len 136, room 8
tail 0
chksum 0x52
csum 0x52
Œãäƒoì›rƒûg|ìdldl`Œâs’l„lÄþ
Application.txt

I have not seen this problem before, sorry. Why do you need to copy/paste the file? Why not just upload the files directly?


New version of live-share plugin is much slower than previous versions

Product and Version [VS/VSCode]: VSCode
OS Version [macOS/Windows]: Description: Ubuntu 18.04.1 LTS
Live Share Extension Version: v0.3.1120
Target Platform or Language [e.g. Node.js]: Python 3
Previous versions of the Live Share Plugin connected almost instantly to clients, and now with recent updates it hangs and takes a very long time to connect.

Closing as a duplicate of #1552.


Additional FAQ Q

Under the pricing FAQ, can we include additional cost such as vNet peering egress and ingress charges that would apply?
How are "connect groups" (non vNet peering connections) charged? Same as ingress and egress?

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

ID: a27d6ac3-cc1d-46de-a865-dd519042b90f
Version Independent ID: 2be3da6a-3a56-3bcc-45fc-9c2c0ebd7f00
Content: Frequently asked questions about Azure Virtual Network Manager
Content Source: articles/virtual-network-manager/faq.md
Service: virtual-network-manager
GitHub Login: @mbender-ms
Microsoft Alias: mbender

@bbabcock1990
Thanks for your feedback! We will investigate and update as appropriate.

@bbabcock1990 , May I know which pricing FAQ you are referring to?
The Virtual network peering charges are already available in the pricing page as below:
https://azure.microsoft.com/en-gb/pricing/details/virtual-network/

Ingress is the Inbound data transfer mentioned above.
Egress is the Outbound data transfer mentioned above.
Additionally, in the same pricing page, there is a FAQ, which has the below information:
https://azure.microsoft.com/en-gb/pricing/details/virtual-network/#faq

So, could you please provide more details regarding your ask?
Also, what are you referring as "connect groups (non vNet peering connections) "?
There is no resource in Azure called connect groups.
It is either Vnet peering, Vnet-to-Vnet connections or Site-to-Site connections.
If you are referring to "Vnet-to-Vnet connections", then the pricing depends on the VPN gateways used + data transfer, which you can find the VPN gateway pricing page:
https://azure.microsoft.com/en-gb/pricing/details/vpn-gateway/
Price of the VPN gateway depending upon the SKU which you deploy
+
The data transfer charges as below:

From the VPN gateway pricing FAQ, you can find the below additional info:

So, request you to share additional details on your ask for further discussion on this issue.

Hi,
This is Virtual Network Manager service.
https://learn.microsoft.com/en-us/azure/virtual-network-manager/faq#whats-the-cost-of-using-azure-virtual-network-manager
There are concepts within this service that establish vNet peerings and/or "connected groups".
I think it should be called out that ontop of this service, you still pay for vNet peering ingress and egress charges.
For Connect group connections, are these treated the same cost wise as vNet peering?

@bbabcock1990 , thank you for the update.
Azure Virtual Network Manager pricing page already mentions about the Vnet peering cost:

Refer: https://azure.microsoft.com/en-us/pricing/details/virtual-network-manager/
But there is no mention of the pricing for Connected Group.
So, we have assigned this to the content owner/author for review.
@mbender-ms , could you please review this and share your inputs?

Adding @andreamichaelmsft for visibility.
Will review with team and update docs as needed.
Thanks for the feedback @bbabcock1990!

@bbabcock1990 The same pricing structure of VNet peering is used for all ingress/egress traffic in a mesh topology (which creates a connected group) and Hub-and-spoke topology. All topologies are created through a Connectivity configuration in AVNM, thus what the note on the pricing page refers to.
Updating the FAQ to refer to the pricing page.
#Please-close

https://github.com/MicrosoftDocs/azure-docs-pr/pull/247206

https://github.com/MicrosoftDocs/azure-docs/issues/112870


Angle bracket typecasts cause many unrelated errors (since 0.40.0)

Since Volar 0.40.0 I see hundreds of errors across all my .vue component files, a selection including:
Code images...

Corresponding errors...

Have had to roll back to 0.39.5 for now, where everything is fine (ie. no errors).

I also have this issue, and found it may be related to syntax that uses angled brackets such as a typecast:
(Difficulty)[difficulty]
Removing the angled brackets stopped a lot of these errors showing.

Thanks @timcodesdotnet that seems to have narrowed down to a specific issue. Removing angle bracket typecasts does seem to ripple through and fix all the other (seemingly unrelated) issues.
That's a workaround but this is still an issue as angle bracket casts are supported in TS and haven't been deprecated afaik. Running vue-tsc shows no error with casts so Volar shouldn't either.

#1706 same issue

Installing any version before prior to 0.40.0 fixed the problem for me.
On the "Uninstall" button, click the dropdown arrow and choose "Install another version". Then choose 0.39.5

duplicate of https://github.com/johnsoncodehk/volar/issues/1706


Update data-analytics.md fix capabilities table to have references

Adds links to each bucket like it is on this page: https://handbook.sourcegraph.com/departments/data-analytics/

not needed since bizops pages are going to be deprecated


add padded options for season and episode numbers

adds support for {padded_episode_number} and {padded_season_number}

Will add the documentation part of it later today, if you guys agree with this as an option.

What about anime that have more than 99 episodes?

What about anime that have more than 99 episodes?

That's where a template/templating engine comes into use.

What about anime that have more than 99 episodes?

That's where a template/templating engine comes into use.

As per purely 99 episode an higher, anything over ep 99 will stop being padded in any way. i.e.
s1e9 ends up as s01e09, but s123e123 ends up as s13e123, subsequently all episodes and seasons with <2 digits will end up getting a 0 as padding.

Yeah, I am aware of that. I just wanted to say I personally think in this case this:
S1E001
...
S1E123

would make more sense than this:
S1E01
...
S1E123

Probably also has to do with personal preferences.

What about anime that have more than 99 episodes?

That's where a template/templating engine comes into use.

As per purely 99 episode an higher, anything over ep 99 will stop being padded in any way. i.e. s1e9 ends up as s01e09, but s123e123 ends up as s123e123, subsequently all episodes and seasons with <2 digits will end up getting a 0 as padding.

I'm agreeing with this. Given that the old (golang) versions are all using the 0<2 padding too and most anime doesn't have 100+ episodes this is the most useful implementation.
Padding depending of the total amount of episodes would also be an option but to archive this way too much request must be made (since cr doesn't provide a number of the total episode amount themself) so this isn't really worth it

I totally agree that this would probably be the best and easiest solution. Just wanted to point out that it could happen that an anime has more than 99 episodes, it's fairly unlikely but not impossible.

I pushed my fork to current master, will redo the PR.


where is the page.Find tested? (or: examples needed)

I only see a lot of tests that are written with the other 2 packages (ginkgo & gomega) but no separate agouti only tests. I was looking for a way to use find, and since the documentation is not that clear about the usage, I thought to look inside the tests, but I don't seem to find it.
I was trying to see if there is a string inside a page:
	someVar := page.Find("ERR_NAME_NOT_RESOLVED")
	fmt.Println(someVar)

but that didn't do the trick. Find only reacts on selectors.
So basically to check if an URL is a good one, I use this:
	someVar := page.Find(".error-code")
	someCount, err := someVar.Count()
	if err == nil {
		if someCount > 0 {
			text, err := someVar.Text()
			if err == nil {
				if strings.Contains(text, "ERR_NAME_NOT_RESOLVED") {
					fmt.Println("URL cannot be found")
				}
			}
		}
	}

Isn't there a better way to achieve this?
I think a string compare to find out an url isn't a good one is a bit buggy, so I'm looking for advice here.

The tests you are looking for are here. They are linked to in the README.
Determining if the page you navigated to belongs to an unregistered domain will vary significantly between WebDrivers and Browsers. I would not recommend using a WebDriver for this task.
The page.Find() method only takes CSS selectors. Additional selection methods are documented.
I recommend indenting your error cases in the code you've provided, instead of indenting the expected flow.

Additionally, there is an example of Agouti used by itself on agouti.org here.

So I overlooked the:
	loginURL, err := page.URL()
	if err != nil {
		panic(err)
	}

but this doesn't do it for me either.
isn't there any way in finding out the url isn't a good one?

The test doesn't show any page.find() that is tested unless you mean you just test the result.
I'm used to testing a functionality by only testing that part (without wrappers to other packages)
so a test of page.Find() would test an empty string, a non-existing string, an 1 time existing string, and a multiple existing string.
Perhaps even (since it should be a selector), every variant of a selector.
But hey, call me careful.
I'm just trying to see what the package can do for me without bothering you, but somehow, I don't understand it's inner workings right now. Glad you help me (or kick me) in the right direction.

The relevant unit tests for the FindX methods are here:
https://github.com/sclevine/agouti/blob/master/selectable_test.go
https://github.com/sclevine/agouti/blob/master/api/element_test.go#L50
https://github.com/sclevine/agouti/blob/master/api/session_test.go#L39
Relevant integration tests are here:
https://github.com/sclevine/agouti/blob/master/internal/integration/selection_test.go
Agouti is a fairly thin wrapper on top of the w3c WebDriver spec. It wouldn't make sense for Agouti to exhaustively re-test every combination of selector values, given that the various WebDrivers already contain these type of tests. We do thoroughly test Agouti's interface with the WebDrivers.
I think you're looking for functionality that is already available in the standard library:
https://golang.org/pkg/net/#LookupHost

Thanks for trying to help me, I fixed the url-not-found in about the same way you mentioned.


Indonesian added

This is resolving issue #5.

@wangzhipeng-vicky please check if the Indonesian translation is correct to resolve #5.

@wangzhipeng-vicky please check if the Indonesian translation is correct to resolve #5.

I'm using version 1.2.0, it doesn't seem to work,

@wangzhipeng-vicky that wasn't my intention. I asked you to review the code change to validate if the Indonesian translation is correct before I merge this into main and release v1.3.

@wangzhipeng-vicky that wasn't my intention. I asked you to review the code change to validate if the Indonesian translation is correct before I merge this into main and release v1.3.

I took a look. It's what I need


[FormsyText] New implementation to allow initialization through this.refs.form.reset

Based on https://github.com/christianalfoni/formsy-react/wiki/validate-on-blur
Hi @mbrookes
Apologies for the bad jsx file. I indeed copied it from index.js instead of doing it the other way.
I have compiled jsx on https://babeljs.io/repl
This time I have also created a demo video for you to check out the behaviour of TextField.
Do let me know if it fixes https://github.com/mbrookes/formsy-material-ui/issues/6 or not.

https://youtu.be/bs56hnZozZU
Note : When ever an error comes in the video without any reason that means I have clicked enter key.
Thanks

Are you editing the code directly on github? - not really a good idea for code changes, as errors are likely (like yesterday's ;) It also means I can't merge from your fork to test locally.
Use babel <infile> -o <outfile>, as per the prepublish hook in package.json.
Please refer to the discussion in #6. Enter key support is useful, but doesn't solve the fundamental problem with onBlur. (I'm not asking you to solve it with this PR, just saying this doesn't entirely :smile:)

Nope. But I am working on your github repo directly by creating new branches, and then making pull request with master on github.
I know about babel  -o . Will keep that in mind. But I think https://babeljs.io/repl also does the same thing.
Assuming you are talking about below problem from https://github.com/mbrookes/formsy-material-ui/issues/6
"If a user fills out a textfield as the last required input, with onBlur the submit button isn't enabled unless the user clicks away from the field. With onChange this problem would likely be resolved."

In my implementation as soon as input field becomes valid the submit button gets enabled. (during onChange itself) but error comes only if onBlur or onEnterKey press which feels very natural.
Please let me know if this pr is ready for merge or do you want me to improve something :)

Nope. But I am working on your github repo directly by creating new branches, and then making pull request with master on github.

Ah, sorry - missed that. I'd prefer if you edit your own fork and PR from there.

I think https://babeljs.io/repl also does the same thing.

Yes, but copying and pasting is error prone, as you discovered. :wink: Not sure why you would use that when you can run babel locally in your branch.

In my implementation as soon as input field becomes valid the submit button gets enabled.

Yes, that sounds ideal. It wasn't obvious from the video.
Testing your branch now.

Merged and published. Thanks!! :ok:


first round of styling changes

Pre-Flight checklist

[x] Screenshots and design review for any changes that affect layout or styling

[x] Desktop screenshots
[x] Mobile width screenshots
[x] Tag @ferdi or @pdpinch for review

[x] Testing

[x] Code is tested
[x] Changes have been manually tested

What are the relevant tickets?
closes #115
What's this PR do?
This brings the site more in line with the mockups. It doesn't implement every single thing in he mockups (notably the different header style shown therein on the site pages) but it gets most of the site in shape.

background colors
text styling (Abdul recommended going with Google Roboto for our basic font)
button styling
adding content cards to wrap forms and so on

and a few more random enhancements besides.
How should this be manually tested?
check out the mocks, and then click around the site. Make sure that no functionality has been compromised anywhere. Then check that all butons are switched over, and that all the gray elements have been eliminated.
Screenshots (if appropriate)

Codecov Report

Merging #126 (53d1e32) into master (5628ec4) will decrease coverage by 3.06%.
The diff coverage is n/a.

:exclamation: Current head 53d1e32 differs from pull request most recent head b63152f. Consider uploading reports for the commit b63152f to get more accurate results

@@            Coverage Diff             @@
##           master     #126      +/-   ##
==========================================
- Coverage   87.69%   84.62%   -3.07%
==========================================
  Files          89       41      -48
  Lines        2169     1197     -972
  Branches      319      136     -183
==========================================
- Hits         1902     1013     -889
+ Misses        250      168      -82
+ Partials       17       16       -1

Impacted Files
Coverage Δ

static/js/components/MarkdownEditor.tsx

static/js/query-configs/utils.ts

static/js/util/incrementer.ts

static/js/lib/ckeditor/turndown.ts

static/js/lib/ckeditor/plugins/test_util.ts

static/js/util/integration_test_helper.tsx

static/js/mocks/styleMock.js

static/js/store/configureStore.ts

static/js/constants.ts

static/js/test_util.ts

... and 21 more

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 5628ec4...b63152f. Read the comment docs.


Uniqueness validator explicitly uses case sensitivity

Fixes Rail 6.1 deprecation warnings
DEPRECATION WARNING: Uniqueness validator will no longer enforce case sensitive comparison in Rails 6.1. To continue case sensitive comparison on the :slug attribute in StepByStepPage model, pass `case_sensitive: true` option explicitly to the uniqueness validator

DEPRECATION WARNING: Uniqueness validator will no longer enforce case sensitive comparison in Rails 6.1. To continue case sensitive comparison on the :slug attribute in Tag model, pass `case_sensitive: true` option explicitly to the uniqueness validator

DEPRECATION WARNING: Uniqueness validator will no longer enforce case sensitive comparison in Rails 6.1. To continue case sensitive comparison on the :base_path attribute in SecondaryContentLink model, pass `case_sensitive: true` option explicitly to the uniqueness validator

I think it's correct that they are all validated with case sensitivity.

@benthorner Thanks Ben, will set as false and merge.


Left alignment assumed in node labels

When using the option nodedimensionsincludelabels, there is an implicit assumption that the node will be to the left of the label, and that the label is larger than the width of the node.
If, for example, the label is horizontally centered in the node, we need to change the getPos function to add (nodeWidth/2) to the first step where p.x is being updated.
Otherwise, if the label is long, it will intersect with other nodes.
I believe the same is true for p.y. we need to add nodeHeight/2 in the first time it's updated, otherwise tall labels will intersect other nodes.

As you may be already aware from your research:  Unless the layout algorithm itself can be aware of label orientations, the best general solution is to notify the layout that additional width and height should be used.  This simply tells the layout to give a node extra space for long labels.  This works well when labels are centred, which is often the case.
If you have specific suggestions for customising the official Klay algorithm , you're welcome to comment there.  If you have suggestions for better integrations with existing Klay algorithm options, you're welcome to make a PR here.
Thanks for your feedback


Table of contents links

I have some markup
1. [Top Level](#Top-Level)
   1. [Windows](#Windows)
   2. [OSX/Linux](#OSX/Linux)
   3. [Android](#Android)

## Top Level

## Windows

## OSX/Linux

## Android

In markup the table of conents links to the headers.  The HTML generated creates the right links
Windows

but the headings aren't anchored.  If they were

Windows

It would work.  Is there a way of doing this?

you would need to add an id to each one of the headers in order for the correct links to be generated.
I replaced my heading node in the renderers prop with HeadingRenderer
import React from 'react';

const flatten = (text: string, child) => {
  return typeof child === 'string'
    ? text + child
    : React.Children.toArray(child.props.children).reduce(flatten, text);
};

/**
 * HeadingRenderer is a custom renderer
 * It parses the heading and attaches an id to it to be used as an anchor
 */
const HeadingRenderer = props => {
  const children = React.Children.toArray(props.children);
  const text = children.reduce(flatten, '');
  const slug = text.toLowerCase().replace(/\W/g, '-');
  return React.createElement('h' + props.level, { id: slug }, props.children);
};

export default HeadingRenderer;

So I should use
<ReactMarkdown
  source={markdown}
  escapeHtml={false}
  renderers={{'heading',HeadingRederer}}
/>

So I should use
<ReactMarkdown
  source={markdown}
  escapeHtml={false}
  renderers={{'heading',HeadingRenderer}}
/>

No, it would be renderers={{ heading: HeadingRenderer}}

Thank you very much - had to remove toLowerCase() on line 16, but after doing that it works perfectly

@davidsteed another approach could be to use a plugin to automatically add anchors to headings, like https://github.com/remarkjs/remark-slug
or even generate the entire table of contents automatically with a plugin using: https://github.com/remarkjs/remark-toc

you would need to add an id to each one of the headers in order for the correct links to be generated.
I replaced my heading node in the renderers prop with HeadingRenderer
import React from 'react';

const flatten = (text: string, child) => {
  return typeof child === 'string'
    ? text + child
    : React.Children.toArray(child.props.children).reduce(flatten, text);
};

/**
 * HeadingRenderer is a custom renderer
 * It parses the heading and attaches an id to it to be used as an anchor
 */
const HeadingRenderer = props => {
  const children = React.Children.toArray(props.children);
  const text = children.reduce(flatten, '');
  const slug = text.toLowerCase().replace(/\W/g, '-');
  return React.createElement('h' + props.level, { id: slug }, props.children);
};

export default HeadingRenderer;

Thank you very much!


[BUG] Named export 'SyncOutlined' not found.

Describe the bug
When building our app, we get this error:
Named export 'SyncOutlined' not found. The requested module '@ant-design/icons' is a CommonJS module, which may not support all module.exports as named exports.
CommonJS modules can always be imported via the default export, for example using:

import pkg from '@ant-design/icons';

Steps To Reproduce
Install the latest refine packages and run the app.
Expected behavior
No error is given when building
Packages
"@refinedev/antd": "^5.38.0",
"@refinedev/cli": "^2.16.30",
"@refinedev/core": "^4.49.1",
"@refinedev/devtools": "1.2.0",
"@refinedev/inferencer": "^4.6.1",
"@refinedev/kbar": "^1.3.9",
"@refinedev/nextjs-router": "^6.0.3",
"@refinedev/simple-rest": "^5.0.5",
Additional Context
No response

Hey @bombillazo, sorry for the issue 🙏 We had this release yesterday and it caused some errors with the type declarations in our packages. A new release is just on its way which addressess the type declaration issues. I'll ping here when the release is done and let's see if it fixes the issue you're having as well 🤔

Ok thanks, looking much forward to it.

Now a patch is released for all packages. For @refinedev/antd the version is 5.38.1. Can you check if this fixes the issue? :pray:

on it

Hey, I still get the error...
SyntaxError: Named export 'SyncOutlined' not found. The requested module '@ant-design/icons' is a CommonJS module, which may not support all module.exports as named exports.
CommonJS modules can always be imported via the default export, for example using:

import pkg from '@ant-design/icons';
const {value:o,configurable:!0});import et from"react";import{Form: ot}from"antd";import{useForm: rt}from"sunflower-antd";import{flattenObjectKeys: tt,propertyPathToArray: Xo}from"@refinedev/core";import{useForm: st,useWarnAboutChange: nt,pickNotDeprecated: Jo,useTranslate: at,useRefineContext: it}from"@refinedev/core";

@bombillazo, I've found an issue related to this in @ant-design/icons repo. Please check https://github.com/ant-design/ant-design-icons/issues/605. Can you try downgrading @ant-design/icons to 5.1.4? If it's not installed can you try installing it with the specified version?
Please let us know if this resolves the issue 🙏

I tried it but it doesnt fix the issue, currently, I've reverted to using "@refinedev/antd": "5.37.6" and it works

Can you also check for<EMAIL_ADDRESS><EMAIL_ADDRESS>had this version pinned down and the latest version removed that version lock 🤔

I fixed the version to 5.0.1 but I still get the error when using<EMAIL_ADDRESS>

I see 5.38.1 still not locking antd 5.0.1 in the package.lock
  "node_modules/@refinedev/antd": {
      "version": "5.38.1",
      "resolved"<EMAIL_ADDRESS>      "integrity": "sha512-CFonTqYHV6wTEq3ofoFQ0XODW1DMcpuCP9ApPjPYQZ8ORY0pcudPf22L3RTeeyHodabdsSJ+C3bqeVLyg9+4Zw==",
      "dependencies": {
        "@ant-design/icons": "^5.0.1",
        "@ant-design/pro-layout": "7.17.12",
        "@refinedev/ui-types": "^1.22.7",
        "@tanstack/react-query": "^4.10.1",
        "antd": "5.16.5",
        "dayjs": "^1.10.7",
        "react-markdown": "^6.0.1",
        "remark-gfm": "^1.0.0",
        "sunflower-antd": "1.0.0-beta.3",
        "tslib": "^2.6.2"
      },

I manually locked it to 5.0.1 in the package.lock file and reinstalled deps, ran it with 5.38.1 but the error persists.

The current combination that works for me is this:
 "@ant-design/icons": "^5.3.7",
 "@refinedev/antd": "5.37.6",

Thank you for your help investigating the issue @bombillazo. I think we might need to re-apply the version pinning here for the @ant-design/icons. Do you want to open up the PR for this? While the PR might take some to be released, you may also check for options like package.json#overrides (or `#resolutions for yarn) but I'm not sure if this is going to work 😅

I tried that but it messes up my stylings for some reason. Ill keep using "@refinedev/antd": "5.37.6" for now.

I would love to help with the PR but I'm not sure where the change is required in the repo, Im not familiar with the refine repo.

Check out this PR https://github.com/refinedev/refine/pull/5908, it looks like there are a lot of changes but most of them are from /examples using @ant-design/icons. We need to apply the same version pin to them as well but related changes in packages are at /packages/antd, /packages/inferencer and /packages/live-previews. You can check our Contributing guide to get start 🚀

Hey @bombillazo, can you confirm that this issue is resolved with the latest releases? 🙏

Closing this as completed with https://github.com/refinedev/refine/pull/5934

Hey, seems like its not giving any more issues, confirmed it today. Thanks!


Problem with Template constraints

Hello,
Im using play 2.6.5 and deadbolt 2.6.1 java.
Im trying to add @subjectPresentOr() {}{} but dont work.
im add the import
@import be.objectify.deadbolt.java.views.html.di.{subjectPresent, subjectPresentOr}
but all time say me
Compilation error
not found: value subjectPresentOr
https://deadbolt-java.readme.io/docs/template-constraints
What are doing wrong?
Thanks.

I need to update the documentation, thanks for bringing this to my attention.
In the meantime, this should work
@this(subjectPresentOr: be.objectify.deadbolt.java.views.html.di.subjectPresentOr)

because constraints are now injected into templates (Play 2.6 removed the global state that templates previously relied on).  You can see it used in a test here

Ok, I try but another error,
non-static method render(be.objectify.deadbolt.java.DeadboltHandler,play.data.Form<dto.security.UserDto>) cannot be referenced from a static context
I think it was because i use @this to inyect.
@this(subjectNotPresent: be.objectify.deadbolt.java.views.html.di.subjectNotPresent)
@(deadboltHandler: be.objectify.deadbolt.java.DeadboltHandler, form: play.data.Form[dto.security.UserDto])
In the Controller, I use inyector,
public Result loginForm () {
return ok(views.html.login.render(handler, formFactory.form(UserDto.class)));
}
And inyect the handler in contructor,
@Inject
public UsersController(UserDao userDao, FormFactory formFactory, @HandlerQualifiers.MainHandler final DeadboltHandler handler) {
this.userDao = userDao;
this.formFactory = formFactory;
this.handler = handler;
}
If I delete @this, this error dissapear, but dont work.
Thanks.

@this(...) is used inside Deadbolt's templates because they have required Deadbolt components injected into them.
I think the problem is this:
non-static method  xxx referenced from a static context
I think what's happening, without seeing your code, is that you're not injecting your own components into your controllers.  Take a look at this guide to see how to do this.

Yes, this is the problem... I forgot inyect the view. Now work perfect.
Thank you so much.

Thanks - I'll update the docs as soon as I have chance.

I don't see this in the doc, so for future reference you probably want
@this(subjectPresent: be.objectify.deadbolt.java.views.html.di.subjectPresent, subjectPresentOr: be.objectify.deadbolt.java.views.html.di.subjectPresentOr)
This shows how to have two items, as opposed to just one. From here on (three, four items) it's self-explanatory.
Actually, it's interesting how so often documentation shows how to do something with ONE item, or with TWO items, but never both, when really both are needed. But that's just a pet peeve of mine.


Spi support

Rebased. Leave the remaining fix to me.🤗

Changes (mainly in https://github.com/hpmicro-rs/hpm-hal/pull/3/commits/d4cb3e6124c7a1feb54b554d5824a69990e92241)

Define some enums in metapac crate
rename addr_width data_width to _phrase for improved clarity and consistency.
Remove unused enums
Incorporated the SpiMode enum and the MODE_0 constant from the embedded-hal crate, aligning the implementation with established standards.
Group timing config into Timings. The field enum types might be stripped out as well(in future refactor)

@HaoboGu
Final changes

CS pins are bonded to the QSPI driver, not the ordinary SPI driver(leaving it a SpiBus)
transfer and transfer_inplace are fixed, leaving some minor optimization undone
The bug for tx is located: Due to the L1 cache being enabled, blocking_write returns very early before fifo finishes writing. So it must wait for fifo empty(I choose spiactive for better busy/idle wait)
Freq calc is changed, if a specific freq is not possible, a lower one(high clk_div) will be selected


os-freeradius - Bug report: 'collections.OrderedDict object' has no attribute 'ldap'

Important notices
Before you add a new report, we ask you kindly to acknowledge the following:

[X] I have read the contributing guide lines at https://github.com/opnsense/plugins/blob/master/CONTRIBUTING.md
[X] I have searched the existing issues, open and closed, and I'm convinced that mine is new.
[X] The title contains the plugin to which this issue belongs

Describe the bug
Disclaimer... First time setting up opnsense, so if this is user error please feel free to close it.
When setting up freeradius and not using the ldap option, The following error seems to be generated.
[6ccdc393-f3d4-4e0c-bcfb-3dfeaf5e9d6f] Inline action failed with OPNsense/Freeradius OPNsense/Freeradius/sites-enabled-inner-tunnel 'collections.OrderedDict object' has no attribute 'ldap' at Traceback (most recent call last): File "/usr/local/opnsense/service/modules/template.py", line 267, in _generate content = j2_page.render(cnf_data) File "/usr/local/lib/python3.8/site-packages/jinja2/environment.py", line 1304, in render self.environment.handle_exception() File "/usr/local/lib/python3.8/site-packages/jinja2/environment.py", line 925, in handle_exception raise rewrite_traceback_stack(source=source) File "/usr/local/opnsense/service/modules/../templates/OPNsense/Freeradius/sites-enabled-inner-tunnel", line 158, in top-level template code {% if helpers.exists('OPNsense.freeradius.ldap.innertunnel') and OPNsense.freeradius.general.ldap.innertunnel == '1' %} File "/usr/local/lib/python3.8/site-packages/jinja2/environment.py", line 474, in getattr return getattr(obj, attribute) jinja2.exceptions.UndefinedError: 'collections.OrderedDict object' has no attribute 'ldap' During handling of the above exception, another exception occurred: Traceback (most recent call last): File "/usr/local/opnsense/service/modules/processhandler.py", line 506, in execute return ph_inline_actions.execute(self, inline_act_parameters) File "/usr/local/opnsense/service/modules/ph_inline_actions.py", line 51, in execute filenames = tmpl.generate(parameters) File "/usr/local/opnsense/service/modules/template.py", line 344, in generate raise render_exception File "/usr/local/opnsense/service/modules/template.py", line 335, in generate for filename in self._generate(template_name, create_directory): File "/usr/local/opnsense/service/modules/template.py", line 270, in _generate raise Exception("%s %s %s" % (module_name, template_filename, render_exception)) Exception: OPNsense/Freeradius OPNsense/Freeradius/sites-enabled-inner-tunnel
It looks like some UI items make it into files like: cat /usr/local/etc/raddb/clients.conf but others like are empty cat /usr/local/etc/raddb/users
To Reproduce
Set up a basic free radius implementation w/ no LDAP options selected. See screenshots.
Expected behavior
I expect freeradius config files to be written on save.

Relevant log files
See Above.
Additional context
I posted about this in the Reddit Channel, someone else said they experienced to as well which leads me to hope it's a bug and not user error. https://www.reddit.com/r/opnsense/comments/t7mcau/running_in_to_error_inline_action_failed_with/
Environment
os-freeradius 1.9.19
opnsense version ; Version 22.1.2_1; Architecture amd64; Flavour OpenSSL; Commit 80eff6d1e
Intel(R) Celeron(R) CPU J3160 @ 1.60GHz (4 cores, 4 threads)

Please check If this fixes it
https://github.com/opnsense/plugins/pull/2881#issuecomment-1060478797

Thank you @mimugmail That fixed it!
Shoot, I don't know how I missed when I searched earlier 🤦🏻♂


ENYO-5426: Fix the GridListImageItem's image height

Checklist

[X] I have read and understand the contribution guide

[ ] A CHANGELOG entry is included

[ ] At least one test case is included for this feature or bug fix

[X] Documentation was added or is not needed

[ ] This is an API breaking change

Issue Resolved / Feature Added

If using only a caption or a subCaption, there is the gap between its text and the GridListImageItem border.
.caption and .subCaption css classes are not needed any more.

Resolution

Increase the image height if using only a caption or a subCaption.
Remove useless css classes.

Additional Considerations

Add CSS styles for subCaption that we used before. (I don't know the exact history for the styles, but we defined the styles for subCaption before and it is not applied to the subCaption now.)

Links
ENYO-5426
Comments
The target branch for this PR is not develop but feature/ENYO-5426-same-height. So I did not update CHANGELOG.md and no test case is needed.

I created the new PR #1872 instead of this PR.


Creating exception_handlers on the main @app causes "RuntimeError: Caught handled exception, but response already started."

Writing this up for the benefit of others that might stumble into the same thing.
When using versionizer and adding an exception_handler to the main app, something like this -
@app.exception_handler(AwesomeException)
async def awesome_exception_handler(request: Request, exc: AwesomeException):
    ...

You'll discover that those exceptions do not get caught as expected; instead you see -

a http response at the client
the Exception you are expecting to catch then get caught after the response has already been sent
a second Exception will be raised complaining that a response has already been sent

There is a FastAPI ticket that has been converted into a discussion on this here -

https://github.com/tiangolo/fastapi/discussions/9098

The root cause of the issue is that the exception_handler is being added at the root-application not the sub-application mount that versionizer uses under the covers to mount each version - you can see this here
The way to handle this then is to insert exception_handlers into the versionize() construct.
versions = versionize(
    app=app,
    prefix_format=f"/v{{major}}.{{minor}}",
    sorted_routes=True,
    exception_handlers=get_exception_handlers()
)

def get_exception_handlers():
    async def awesome_exception_handler(request: Request, exc: AwesomeException):
        return JSONResponse(
            status_code=exc.status_code,
            content={"detail": exc.detail},
        )

    return { AwesomeException: awesome_exception_handler }

This will cause the AwesomeException handler to be added to each sub-application mount that versionizer then uses.

Ticket is a solution to probably common pitfall - closing


Move extended email to new format for DSL v1.43

In Jenkins 1.43, the DSL format for extended email changed.
The CommonUtils.addExtendedEmail function was updated to reflect the new format.
Side Effects:

the triggers parameter was renamed to triggerList
the trigger values now need to be lowercase (e.g. 'failure' instead of 'Failure')

Change Description: https://github.com/jenkinsci/job-dsl-plugin/wiki/Migration#extended-email
API Docs: https://jenkinsci.github.io/job-dsl-plugin/#method/javaposse.jobdsl.dsl.helpers.publisher.PublisherContext.extendedEmail

We should hold off merging this until all affected teams are aware of the change.

Good idea @m3brown !


pythonPackages.elasticsearch-curator: 5.5.1 -> 5.5.2

Motivation for this change
Fix build of elasticsearch-curator.
cc @basvandijk
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[X] Fits CONTRIBUTING.md.

As for certifi, the only change was https://github.com/certifi/python-certifi/commit/bf1224e31f12c9bde8d83ac5c8d2c2360ad5640b (removed Elektronik Sertifika Hizmet from trusted). I'm suprised it causes so much rebuilds, according to ofborg, but I assume most of those are Python-only?

@danbst Please squash the last commit into the appropriate one and format your commit messages according to the contributing guidelines, i.e. as the title of this PR.
@GrahamcOfBorg build python2.pkgs.certifi python3.pkgs.certifi python2.pkgs.elasticsearch-curator python3.pkgs.elasticsearch-curator

Success on aarch64-linux (full log)
Attempted: python2.pkgs.certifi, python3.pkgs.certifi, python2.pkgs.elasticsearch-curator, python3.pkgs.elasticsearch-curator
Partial log (click to expand)
TOTAL                                  3086   1093    65%
----------------------------------------------------------------------
Ran 508 tests in 234.783s

OK (SKIP=1)
Shutting down....
/nix/store/ixpkgqg4pqgwg20mjp3g68jp0j5g8bfb-python2.7-certifi-2018.4.16
/nix/store/3imni9l57hfsv202w2dg5lb6r2llq3jk-python3.6-certifi-2018.4.16
/nix/store/7zid2zq68bhvbm0hiiqasdi80r26bypp-python2.7-elasticsearch-curator-5.5.2
/nix/store/my686p9wa8sawijxrsj85almrjhs93jx-python3.6-elasticsearch-curator-5.5.2

Success on x86_64-linux (full log)
Attempted: python2.pkgs.certifi, python3.pkgs.certifi, python2.pkgs.elasticsearch-curator, python3.pkgs.elasticsearch-curator
Partial log (click to expand)
curator/validators/config_file.py         4      4     0%
curator/validators/filters.py            32     10    69%
curator/validators/options.py            13     13     0%
curator/validators/schemacheck.py        41      8    80%
---------------------------------------------------------
TOTAL                                  3086   1095    65%
----------------------------------------------------------------------
Ran 508 tests in 150.806s

OK (SKIP=1)

Also rebase to staging.

Staging already has elasticsearch-curator: 5.5.4. We should include the certifi bump in the next package set update however.

Oh, indeed! And it fails to build exactly because certifi is not bumped
https://hydra.nixos.org/build/75612592/nixlog/1/tail
unpacking sources
unpacking source archive /nix/store/83lvqn2nn8n5il6k8h6jm4d4kh9a9r45-elasticsearch-curator-5.5.4.tar.gz
source root is elasticsearch-curator-5.5.4
setting SOURCE_DATE_EPOCH to timestamp<PHONE_NUMBER> of file elasticsearch-curator-5.5.4/setup.cfg
patching sources
configuring
building
Download error on https://pypi.org/simple/certifi/: [Errno -2] Name or service not known -- Some packages may not be found!
Download error on https://pypi.org/simple/: [Errno -2] Name or service not known -- Some packages may not be found!
No local packages or working download links found for certifi>=2018.4.16
Traceback (most recent call last):
  File "nix_run_setup", line 8, in <module>
    exec(compile(getattr(tokenize, 'open', open)(__file__).read().replace('\\r\\n', '\\n'), __file__, 'exec'))
  File "setup.py", line 173, in <module>
    tests_require = ["mock", "nose", "coverage", "nosexcover"]
  File "/nix/store/x9yd5nwcqndjmqzpysfhl426wwpzbqz9-python2.7-bootstrapped-pip-10.0.1/lib/python2.7/site-packages/setuptools/__init__.py", line 128, in setup
    _install_setup_requires(attrs)
  File "/nix/store/x9yd5nwcqndjmqzpysfhl426wwpzbqz9-python2.7-bootstrapped-pip-10.0.1/lib/python2.7/site-packages/setuptools/__init__.py", line 123, in _install_setup_requires
    dist.fetch_build_eggs(dist.setup_requires)
  File "/nix/store/x9yd5nwcqndjmqzpysfhl426wwpzbqz9-python2.7-bootstrapped-pip-10.0.1/lib/python2.7/site-packages/setuptools/dist.py", line 514, in fetch_build_eggs
    replace_conflicting=True,
  File "/nix/store/x9yd5nwcqndjmqzpysfhl426wwpzbqz9-python2.7-bootstrapped-pip-10.0.1/lib/python2.7/site-packages/pkg_resources/__init__.py", line 770, in resolve
    replace_conflicting=replace_conflicting
  File "/nix/store/x9yd5nwcqndjmqzpysfhl426wwpzbqz9-python2.7-bootstrapped-pip-10.0.1/lib/python2.7/site-packages/pkg_resources/__init__.py", line 1053, in best_match
    return self.obtain(req, installer)
  File "/nix/store/x9yd5nwcqndjmqzpysfhl426wwpzbqz9-python2.7-bootstrapped-pip-10.0.1/lib/python2.7/site-packages/pkg_resources/__init__.py", line 1065, in obtain
    return installer(requirement)
  File "/nix/store/x9yd5nwcqndjmqzpysfhl426wwpzbqz9-python2.7-bootstrapped-pip-10.0.1/lib/python2.7/site-packages/setuptools/dist.py", line 581, in fetch_build_egg
    return cmd.easy_install(req)
  File "/nix/store/x9yd5nwcqndjmqzpysfhl426wwpzbqz9-python2.7-bootstrapped-pip-10.0.1/lib/python2.7/site-packages/setuptools/command/easy_install.py", line 667, in easy_install
    raise DistutilsError(msg)
distutils.errors.DistutilsError: Could not find suitable distribution for Requirement.parse('certifi>=2018.4.16')
builder for '/nix/store/a3hqi1dfjwnrwi3mpyiyklvvihqaxicg-python2.7-elasticsearch-curator-5.5.4.drv' failed with exit code 1

rebased over staging and dropped curator update - staging has newer one

Also, @FRidh is that a direct requirement to have base16 encoded sha256 hash?

@dotlambda done!

Commit message still not formatted correctly :(

@dotlambda aye, sorry, forgot the colon. Do we have automated tools to process/aggregate commit messages?


Line chart dev

Removed interactivity on the small more-info version of the line graphs that was causing scrollbars issues

Coverage remained the same at 64.89% when pulling c75e145151cfcc005ef361a8b6dad02277d8be20 on jamespcole:line-chart-dev into 992a9265927edd3547b1a0761c7a7fe8381ac8db on balloob:dev.


[Fixes #7658] FileStreamResultExecutor fails to Dispose FileStream

Note: This issue still requires approval

@mkArtakMSFT We have following guards for preventing this
https://github.com/aspnet/Mvc/blob/dev/src/Microsoft.AspNetCore.Mvc.Core/FileStreamResult.cs#L44
https://github.com/aspnet/Mvc/blob/dev/src/Microsoft.AspNetCore.Mvc.Core/FileStreamResult.cs#L60

https://github.com/aspnet/Mvc/commit/b56436b5c607e08ada5fccabfc71c475f8fbbbea


Why input can't get focus

      <button id="button">Get a random number</button>

    <input type="text" />

*, *:before, *:after {
  box-sizing: inherit;
  margin: 0;
  padding: 0;
  position: relative;

  /* Prevent the content from being selectionable */
  -webkit-user-select: none;
  user-select: none;
}

input, textarea {
  -webkit-user-select: auto;
  user-select: auto;
}

OS:mac OS Catalina 10.15
Sketch:53

you solve this problem?

you solve this problem?

By upgrading the version of sketch

I  use Sketch latest version, still  input can't get focus


Add return typehint to getIterator methods for php 8.1

Hello,
Thanks for this useful project :+1:
Just a little contribution to avoid deprecation notices on php 8.1:
PHPArkitect 0.2.6

Config file: tests/phparkitect.php

Deprecated: Return type of Arkitect\ClassSet::getIterator() should either be compatible with IteratorAggregate::getIterator(): Traversable, or the #[\ReturnTypeWillChange] attribute should be used to temporarily suppress the notice in /srv/vendor/phparkitect/phparkitect/src/ClassSet.php on line 39

Deprecated: Return type of Arkitect\Rules\Violations::getIterator() should either be compatible with IteratorAggregate::getIterator(): Traversable, or the #[\ReturnTypeWillChange] attribute should be used to temporarily suppress the notice in /srv/vendor/phparkitect/phparkitect/src/Rules/Violations.php on line 34

Deprecated: Return type of Arkitect\Rules\ParsingErrors::getIterator() should either be compatible with IteratorAggregate::getIterator(): Traversable, or the #[\ReturnTypeWillChange] attribute should be used to temporarily suppress the notice in /srv/vendor/phparkitect/phparkitect/src/Rules/ParsingErrors.php on line 34
analyze class set /srv/tests/../src

 155/155 [============================] 100%

NO VIOLATIONS DETECTED!

Codecov Report

Merging #212 (5bead95) into main (b6ee900) will not change coverage.
The diff coverage is 33.33%.

@@            Coverage Diff            @@
##               main     #212   +/-   ##
=========================================
  Coverage     92.32%   92.32%
  Complexity      293      293
=========================================
  Files            51       51
  Lines           756      756
=========================================
  Hits            698      698
  Misses           58       58

Impacted Files
Coverage Δ

src/Rules/ParsingErrors.php
85.71% <0.00%> (ø)

src/Rules/Violations.php
83.33% <0.00%> (ø)

src/ClassSet.php
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

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

Thanks @stephpy !