Reusing empty test in default filter.

Improving default by reusing empty test.
Following logic https://github.com/twigphp/Twig/blob/55704ce1d29f184374f77b9542baaba58d4d30f2/src/Extension/CoreExtension.php#L810

Please include a test demonstrating the bug this fixes.

I'll need take a look when when running mocha everything is fine, using twig on web I have different results.

Please include a test demonstrating the bug this fixes.

It's not longer an issue.

@willrowe seems that we have misunderstanding. this PR is still relevant about improving default filter.
but

It's not longer an issue.

was about {{params}} being false instead of array

Is this fixing some kind of inconsistency? Please explain more about what it is improving.

Inconsistency 1: default filter in php has default value (empty), while default filter in js requires an argument
Inconsistency 2: default filter in js has it's own logic of detecting empty value, which is not the same as empty test, which does not detect empty array/or object


runtime,sync: SIGSEGV in runtime.checkptrBase via (*Pool).pin

2021-05-04T20:50:35-d19e549/freebsd-amd64-race
fatal error: unexpected signal during runtime execution
[signal SIGSEGV: segmentation violation code=0x1 addr=0x0 pc=0x45a399]

goroutine 26 [running]:
runtime.throw({0x7d61ce, 0x2a})
	/tmp/workdir/go/src/runtime/panic.go:1198 +0x74 fp=0xc000061418 sp=0xc0000613e8 pc=0x47e9b4
runtime.sigpanic()
	/tmp/workdir/go/src/runtime/signal_unix.go:719 +0x4a5 fp=0xc000061478 sp=0xc000061418 pc=0x498545
runtime.spanOf(...)
	/tmp/workdir/go/src/runtime/mheap.go:652
runtime.findObject(0xc000234080, 0x0, 0x0)
	/tmp/workdir/go/src/runtime/mbitmap.go:385 +0x39 fp=0xc0000614b0 sp=0xc000061478 pc=0x45a399
runtime.checkptrBase(0xc000234080)
	/tmp/workdir/go/src/runtime/checkptr.go:68 +0x65 fp=0xc0000614f0 sp=0xc0000614b0 pc=0x44b805
runtime.checkptrArithmetic(0xc000234080, {0xc000061560, 0x1, 0x1})
	/tmp/workdir/go/src/runtime/checkptr.go:32 +0x45 fp=0xc000061520 sp=0xc0000614f0 pc=0x44b705
sync.indexLocal(...)
	/tmp/workdir/go/src/sync/pool.go:277
sync.(*Pool).pin(0x999e60)
	/tmp/workdir/go/src/sync/pool.go:204 +0x91 fp=0xc000061578 sp=0xc000061520 pc=0x4c5f91
sync.(*Pool).Get(0x999e60)
	/tmp/workdir/go/src/sync/pool.go:128 +0x45 fp=0xc0000615d0 sp=0xc000061578 pc=0x4c5905
fmt.newPrinter()
	/tmp/workdir/go/src/fmt/print.go:137 +0x3f fp=0xc000061608 sp=0xc0000615d0 pc=0x53435f
fmt.Sprintf({0x7c9cc7, 0x5}, {0xc0000616a8, 0x1, 0x1})
	/tmp/workdir/go/src/fmt/print.go:218 +0x34 fp=0xc000061660 sp=0xc000061608 pc=0x534dd4
testing.fmtDuration(0x6e8dfe)
	/tmp/workdir/go/src/testing/testing.go:628 +0xd8 fp=0xc0000616c8 sp=0xc000061660 pc=0x55f998
testing.tRunner.func1.2({0x78bec0, 0x999090})
	/tmp/workdir/go/src/testing/testing.go:1186 +0x236 fp=0xc0000617b0 sp=0xc0000616c8 pc=0x5629b6
testing.tRunner.func1(0xc0001fbba0)
	/tmp/workdir/go/src/testing/testing.go:1195 +0x41f fp=0xc000061938 sp=0xc0000617b0 pc=0x5622df
runtime.call16(0x0, 0x7dbc80, 0xc000061fa0, 0x8, 0x8, 0x8, 0xc000061988)
	/tmp/workdir/go/src/runtime/asm_amd64.s:625 +0x49 fp=0xc000061958 sp=0xc000061938 pc=0x4b4589
panic({0x78bec0, 0x999090})
	/tmp/workdir/go/src/runtime/panic.go:1052 +0x2fe fp=0xc000061a30 sp=0xc000061958 pc=0x47e35e
runtime.panicmem(...)
	/tmp/workdir/go/src/runtime/panic.go:221
runtime.sigpanic()
	/tmp/workdir/go/src/runtime/signal_unix.go:735 +0x40e fp=0xc000061a90 sp=0xc000061a30 pc=0x4984ae
runtime.spanOf(...)
	/tmp/workdir/go/src/runtime/mheap.go:652
runtime.findObject(0x9a36e0, 0x0, 0x0)
	/tmp/workdir/go/src/runtime/mbitmap.go:385 +0x39 fp=0xc000061ac8 sp=0xc000061a90 pc=0x45a399
runtime.checkptrBase(0x9a36e0)
	/tmp/workdir/go/src/runtime/checkptr.go:68 +0x65 fp=0xc000061b08 sp=0xc000061ac8 pc=0x44b805
runtime.checkptrAlignment(0x9a36e0, 0x79d9c0, 0x1)
	/tmp/workdir/go/src/runtime/checkptr.go:19 +0x6c fp=0xc000061b38 sp=0xc000061b08 pc=0x44b62c
sync/atomic.(*Value).Load(0x9a36e0)
	/tmp/workdir/go/src/sync/atomic/value.go:29 +0x5c fp=0xc000061b70 sp=0xc000061b38 pc=0x4b9c3c
internal/testlog.Logger(...)
	/tmp/workdir/go/src/internal/testlog/log.go:43
internal/testlog.Getenv({0x7da73f, 0x5})
	/tmp/workdir/go/src/internal/testlog/log.go:52 +0x3f fp=0xc000061ba0 sp=0xc000061b70 pc=0x4ec8df
os.Getenv({0x7da73f, 0x5})
	/tmp/workdir/go/src/os/env.go:102 +0x47 fp=0xc000061be8 sp=0xc000061ba0 pc=0x4f1de7
internal/testenv.GoToolPath({0x836d38, 0xc0001fbba0})
	/tmp/workdir/go/src/internal/testenv/testenv.go:92 +0x1cc fp=0xc000061c68 sp=0xc000061be8 pc=0x75e10c
cmd/vet_test.vetCmd(0xc0001fbba0, {0x7cf0f8, 0x16}, {0x7ca8ef, 0x8})
	/tmp/workdir/go/src/cmd/vet/vet_test.go:72 +0x65 fp=0xc000061d58 sp=0xc000061c68 pc=0x75ef05
cmd/vet_test.TestVet.func1(0xc0001fbba0)
	/tmp/workdir/go/src/cmd/vet/vet_test.go:113 +0xfe fp=0xc000061ed0 sp=0xc000061d58 pc=0x75f43e
testing.tRunner(0xc0001fbba0, 0xc0001be7b0)
	/tmp/workdir/go/src/testing/testing.go:1242 +0x22e fp=0xc000061fd0 sp=0xc000061ed0 pc=0x561d8e
runtime.goexit()
	/tmp/workdir/go/src/runtime/asm_amd64.s:1581 +0x1 fp=0xc000061fd8 sp=0xc000061fd0 pc=0x4b62e1
created by testing.(*T).Run
	/tmp/workdir/go/src/testing/testing.go:1289 +0x78e

Marking as release-blocker until we better understand whether this is a 1.17 regression.
(Certainly there have been many runtime changes this cycle; it's not clear to me whether or how they interact with sync.Pool.)
CC @mknyszek

Weekly check-in: this needs to be investigated before beta 1.

There's actually two spanOf panics in the stack trace. Originally sync.(*Value).Load panics due to spanOf (within checkptr); and then the panic handler ends up calling sync.(*Pool).Get, which panics again in spanOf.
This makes me think this is a runtime issue, that the spans table has been corrupted somehow. But I'm sure @mknyszek has a better idea what might be going wrong here.

These spanOf failures are interesting. They both fall on accessing the L2 map, that as the comment in spanOf indicates, should always be non-nil on amd64. Based on the panic, though, it does appear to be nil. I don't think it's a compiler issue because although spanOf is inlined, it's inlined into findObject. If that were broken somehow, we'd be seeing much bigger, louder failures. I'm inclined to think there is some degree of memory corruption happening here as @mdempsky suggests, and somehow a value is zero is written into mheap_.arenas. However, unless we have more data, I'm not entirely sure how to proceed.
Side-note: technically, the L2 is not actually guaranteed to be non-nil by any part of the initialization. Theoretically checkptr could fail in this way if there are no arenas created whatsoever. However, I will note that any stack allocation forces an arena to be created, and we always allocate at least one goroutine stack during initialization (not to mention the heap allocations we do before the GC is turned on), so in practice it is effectively guaranteed. Perhaps we should make that a more explicit part of the initialization procedure.

I started looking at dashboard error logs containing "SIGSEGV" and I think we have broader memory corruption issues on freebsd/amd64. The other related failures appear to be #46103 and #46182.
@bcmills do you mind if I close those and merge them all into a general "FreeBSD memory corruption" issue?

i can't reproduce under 12.2-RELEASE or 14-CURRENT.  is this running on one of the 12.2 builders?  does this error occur test run?

@ayang64 The failures are varied and rare, but all appear to be some form of memory corruption (at least to my eyes).
I suspect that reproducing this error specifically is going to be difficult or impossible. I think if we want to try to nail this we just need to stress running all.bash on a whole bunch of FreeBSD machines and grab a core dump when things go wrong.

@mknyszek, if we believe that the underlying problem is “undiagnosed memory corruption” I'm fine with deduping all of the issues we suspect share that cause to a single issue. (We can always open new issues if there are still crashes after the suspected root cause is diagnosed and fixed.)

Closing as a duplicate of #46272.


Fix #47 by Adding support for feature branches at install time

Use additional env variables through install.{repo|priv}
Pass these variables through su or export

This is a potential incompatible change with current install.priv usage.
Existing users will have to check the value of their WODPROVBRANCH in order to use the correct one.


Remove examples/switch

Describe your changes
Remove examples/switch
This example works only with older version of transformers library and there has been a few security issues identified in that version. Also, the originating repository, MOE and moe_examples, are not maintained anymore.
Checklist before requesting a review

[ ] Add unit tests for this change.
[ ] Make sure all tests can pass.
[ ] Update documents if necessary.
[ ] Lint and apply fixes to your code by running lintrunner -a
[ ] Is this a user-facing change? If yes, give a description of this change to be included in the release notes.

(Optional) Issue link

It seems you need exclude the NOTICE.txt file? https://github.com/reviewdog/action-misspell?tab=readme-ov-file#exclude

FYI https://github.com/microsoft/Olive/pull/955


Incorrect IsCheckedChanged events triggered in ListBox with CheckBox controls when hidden by GridSplitter

Describe the bug
When using a GridSplitter to hide a ListBox containing CheckBox controls in Avalonia, incorrect IsCheckedChanged events are generated for all items except the first one in the ListBox. This leads to unexpected behavior and incorrect event handling.
To Reproduce
Steps to reproduce the behavior:

Open the application containing the GridSplitter and ListBox with CheckBox controls:
CheckBoxBug.zip
Use the GridSplitter to hide the ListBox.
Observe that incorrect IsCheckedChanged events are triggered for all items except the first one.

Expected behavior
The IsCheckedChanged events should only be triggered when the actual state of the CheckBox controls is changed. Hiding the ListBox using the GridSplitter should not generate these events.
Screenshots
none
Desktop (please complete the following information):

OS: Windows 10
Version: Avalonia 11.0.2

Additional context
This issue might lead to unexpected behavior in applications that rely on accurate IsCheckedChanged events for CheckBox controls within a ListBox. Careful handling and validation may be needed to avoid potential problems caused by these false events.

I don't think you can use IsCheckedChanged like this with ListBox virtualization. The best thing to do is have an IsSelected property on your view model and update that through other means. Otherwise, with recycling and virtualization it's going to be difficult I think.

If I try the sample with a non-virtualized ItemPanel, it does not trigger the event as expected. So this behavior is by design in case of using virtualization.
<ListBox Grid.Column="2" Name="listBox" ItemsSource="{Binding CbItems}">
        <ListBox.ItemTemplate>
            <DataTemplate>
                <StackPanel Orientation="Horizontal">
                    <CheckBox IsChecked="{Binding IsChecked}" IsCheckedChanged="IsCheckedChangedEvent" />
                    <TextBlock Text="{Binding Name}" />
                </StackPanel>
            </DataTemplate>
        </ListBox.ItemTemplate>

!       <!-- DO NOT USE THIS APPROACH IF YOU EXPECT MORE THAN A FEW ITEMS! -->
!       <!-- Performance will decrease noticeable -->
+       <ListBox.ItemsPanel>
+           <ItemsPanelTemplate>
+               <StackPanel />
+           </ItemsPanelTemplate>
+       </ListBox.ItemsPanel>
    </ListBox>


testing CI for PR

No need to merge, just a test

Hello! I'm the build bot for the Mono project.
I need approval from a Mono team member to build this pull request. A team member should reply with "approve" to approve a build of this pull request, "whitelist" to whitelist this and all future pull requests from this contributor, or "build" to explicitly request a build, even if one has already been done.
Contributors can ignore this message.


[fix] fixed header responsiveness

Fixed the header responsiveness

i'm working on the other parts as well, need some time to complete it

i'm working on making the other parts responsive as well, need some time to complete it

hey, a friend of mine is also working on responsiveness, so it might result in conflicts, so thank you for your contributions, but this can result into conflicts.

hey, a friend of mine is also working on responsiveness, so it might result in conflicts, so thank you for your contributions, but this can result into conflicts.

Oh alright :(

hey, a friend of mine is also working on responsiveness, so it might result in conflicts, so thank you for your contributions, but this can result into conflicts.

Oh alright :(

ah, closing the PR wasn't needed, but nvm, sorry

ah, closing the PR wasn't needed, but nvm, sorry

no problem :D


add gridmap-ros

This adds dependency to gridmap-ros package for costmap_generator (https://github.com/tier4/Pilot.Auto/pull/93)

CI should pass when https://github.com/ANYbotics/grid_map/pull/270 is merged

@mitsudome-r perhaps we could temporarily point to your fork instead of upstream, and then update the build_depends.repos file when your PR is merged.


feat(client): hide the outline instead of removing it

Checklist:

[x] I have read and followed the contribution guidelines.
[x] I have read and followed the how to open a pull request guide.
[x] My pull request targets the main branch of freeCodeCamp.
[x] I have tested these changes either locally on my machine, or GitPod.

In high contrast mode, which used for accessibility, outline is needed to provide contrast, this aims to show it in high contrast mode

We are going to be able to get rid of most of this outline related CSS once PR #49579 is merged. So it might be best to hold off on this until that happens.

When you have free time, can you take another look?

@Sboonny, I can't seem to run this. I'm getting the following:
Your pnpm version is incompatible with "/home/bbsmooth/freecodecamp/freeCodeCamp".
Expected version: 7
Got: 8.1.0

@bbsmooth you should be able to test this now. I just did a quick rebase for this PR.


s.replace is not a function

Hi. I think problem is with using v-text, does this problem happens with original (unmodified) error template too? And also what if we change v-text to v-html ?

Solved, thank you very much

Hi it seems my problem after change v-text to v-html does not solved.


Consistent primary verb

We should be consistent about which verb we suggest to the user that they are performing:

search the UKHPI
browse the UKHPI
explore the UKHPI

Some history: in the old HPI app we started off with 'explore', but we were asked by LR comms to change it to 'search' on the landing page, on the basis (as far as I can recall) that searching is a more familiar action to users. The current landing page copy follows that previous tradition. Stephanie has suggested that explore or browse might be a better fit with the actual task.

I've deployed an update which removes the primary verb from the main app page - it was a bit unnecessary anyway. This gave the opportunity to simplify the heading a bit.

Comment from Stephanie on the latest change:

Hi Ian, it looks much better and more consistent,

@ijdickinson looks good to me and I'm happy to close unless you are planning any further updates on this?

@marklunn I'm not planning any changes at the moment, but I'd like Liz to comment on Stephanie's suggestions and say that she's happy with where we are (or not!)

Liz has updated - she is happy


Addition of dependabot.yml file

We need to add dependabot.yml file as it helps us to keep our dependencies up to date. Every day, it checks your dependency files for outdated requirements and opens individual PRs for any it finds.
@avinashkranjan  please assign me this issue under gssoc'23.

We need to add dependabot.yml file as it helps us to keep our dependencies up to date. Every day, it checks your dependency files for outdated requirements and opens individual PRs for any it finds.
@avinashkranjan please assign me this issue under gssoc'23.

Go Ahead @DivyanshiNarang


Step 4: Label Generation reg_FSROI_Jan2014.sh

I try to run you preprocessing steps for PPMI dataset. And faced with errors on the  Label Generation step 4.
Could you tell me at what step and how do you get brainmask.mgz, rawavg.mgz, aparc+aseg.mgz files for your script reg_FSROI_Jan2014.sh ? Did you use dt-recon ?

These files should be included with Freesurfer and you should get it from about lines 13~17 in the reg_FSROI_Jan2014.sh

Sorry for the confusion. You are right. As far as I know, these .mgz files seem to be auto generated from the process you convet raw MRI to .nii file. I think recon-all will do the job.


Feature Request

We have been using this gem in our project and its working perfect !!  Thanks .
As of now we can send the message to only one channel which is configured in env file SLACK_CHANNEL . But it will be nice if we can select a channel while sending the slack mail. We have a requirement where we need to send a message to different channel depending on the type of operation.Is it possible to implement it ?

Sure! I'll take a look this week

Ok, support for multiple channels is in. Keep in mind that each channel has it's own incoming web hook url. You can now declare it in the mailer:
def notification
  mail(:from =><EMAIL_ADDRESS>
       :to =><EMAIL_ADDRESS>
       :subject => 'Subject',
       :channel => '#yourChanel'
       :webhook => 'https://your-webhook-url'
       :delivery_method => :slack)
end

The readme is updated as well. Enjoy :)

Thanks a lot. Will test it and let you know.


Deomonstrate error with modern versiom of Hits

This shows off the selection error when trying to work with catagory axes and slicing.

THis is a partial fix for #7


quickfix for directives_ordering

Follow-up from: https://github.com/dart-lang/linter/issues/26.

Ping? Why was this closed?


[Feature request] Add a hook to allow user to manipulate model init kwargs

The current from_pretrained implementation in ModelHubMixin reads in config.json (if available), and adds it as a kwarg named 'config' in the model init kwargs. This is in line with transformers style, more or less.
I'd rather explicitly write config kwargs in my model's __init__. To do that, we have to unpack the config dict instead of having it be a kwarg. The solution right now is to basically copy paste 99% of ModelHubMixin.from_pretrained to update one line of code.

What do you think about adding a simple hook that wraps this line and allows users to manipulate model_kwargs if they'd like by overriding?
https://github.com/huggingface/huggingface_hub/blob/379414dade73e0816a996d83ead7e4ffd6986a53/src/huggingface_hub/hub_mixin.py#L196

TL;DR: Sounds good to me, do you want to give it a try?

Longer-term I think we should consider making ModelHubMixin more agnostic and require the user to define a _save_pretrained  and a _from_pretrained  method for it to work.
And then we would implement a TorchModelHubMixin , ScikitLearnModelHubMixin , etc. that are framework-specific and slightly more opinionated (and then in your case you can either tweak TorchModelHubMixin, or just inherit from the general ModelHubMixin and implement your logic the way you want)

Sure, I'll give it a go!


No option to backspace

There is no option to backspace. You can't remove single digits in the entry widget. There is option to only clear the widget and you have to type everything back again.
Possible solution:

Add another button for backspace.
OR
Make it such that on clicking the "C" button once, removes a single digit from the entry widget and on long press, clears the entire widget.

Will try to add this, thanks!

Where would you want to put the button?

Maybe near the entry widget? That is, on it's right side.

@RivanParmar How do you think if I remove the 'M' button to darkdetect to free up space for the backspace button?

Yes, it would be great to do something like that, however place the backspace button on the right side and not the left as it would look good as well as be easier to click.

Ok, thank you!

https://user-images.githubusercontent.com/86362423/156884893-87e059f9-e80d-4005-a42e-ffd1f6c9ed68.mp4
Works like a charm! Thanks for giving me suggestion! @RivanParmar

https://user-images.githubusercontent.com/86362423/156884893-87e059f9-e80d-4005-a42e-ffd1f6c9ed68.mp4
Works like a charm! Thanks for giving me suggestion! @RivanParmar

Yes, it is looking awesome!

Coming soon with beta 6 or maybe version 1!


chore: cherry pick https://github.com/aws-amplify/amplify-codegen-ui/pull/297 into main

Cherry picking into main to include a bug-fix for a variant issue.
Co-authored-by: Alexander Harris<EMAIL_ADDRESS>

Codecov Report

Merging #304 (1589139) into main (8b02361) will increase coverage by 0.17%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main     #304      +/-   ##
==========================================
+ Coverage   89.48%   89.66%   +0.17%
==========================================
  Files          34       34
  Lines        1217     1238      +21
  Branches      273      278       +5
==========================================
+ Hits         1089     1110      +21
  Misses        125      125
  Partials        3        3

Impacted Files
Coverage Δ

...gen-ui-react/lib/react-studio-template-renderer.ts
91.21% <100.00%> (+0.38%)
:arrow_up:

packages/codegen-ui/lib/studio-node.ts
96.29% <100.00%> (+1.29%)
:arrow_up:

Continue to review full report at Codecov.

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


Ensure old format of path mappings are supported with the new debugger

It seems the old format isn't supported with the new debugger, https://github.com/Microsoft/vscode-python/issues/2354#issuecomment-411776315

I don't quite follow what you're worried about with this. What "path mappings" are you talking about?

Mapping of the remote to local paths during remote debugging

I just tested this locally by having two copies of a Django app on my machine and using pathMappings let me set a breakpoint in code and have it trip.

And now I tested it by just stripping away the "pathMapping" parts and it still works.
@DonJayamanne if there is some other way I'm supposed to test this working then let me know, but it seems to be fine by me. VS Code does flag "localRoot" and "remoteRoot" as un-allowed properties, but otherwise things seem to work.

Also verified in Windows -> Linux debugging. Worked just fine with both the old mapping method as well as the new.
The old one shows up with green squiggles though, perhaps giving the user some feedback that they are doing things slightly incorrectly, even though it works:


⚠ Bofrost.lu has degraded performance

In cbad18f, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 1883 ms

Resolved: Bofrost.lu performance has improved in 44108a2 after 14 minutes.


bamstats report

The latest update of the conda environment has completely ruined the CGAT report fo Jupyter notebook and Rmarkdown. The reports need to be refactored to reflect the changes

Fixed with  #395


Resolve #19130

Q
A

Is bugfix?
✔

New feature?
❌

Breaks BC?
❌

Fixed issues
#19130

The PR resolves issue #19130, the DbSession's regenerateID should follow Session's regenerateID signature to do nothing if no active sessionID exists.

Are you sure it fixes https://github.com/yiisoft/yii2/issues/19130? Because the code in question was added about 8 years ago.

Yes, from the first analyze of #19130 session write by passes writeSession.
Moreover, the DbSession extends MultiFieldSession which then extends Session and the regenerateID function signature mandates it will do nothing if a session is not active (can not be found from storage)

so the branch with else to create session (id) is not correct.
The code was added 8 years ago but there is very little chance to get to this else branch, for instance the according session record is missing somehow. In this case, the user / account session should be established in proper way like redirect to login page and the session record can be opened (with success login).
Finally, when removed the else branch, there is no fail unit test happen :D

OK. Please add a line for CHANGELOG. Thanks.

Sure, will do shortly. Thanks a lot

👍


Multilingual support

Are you planning on adding multilingual support to zerostatic hugo themes?
Love your themes.

No this was not a high priority, sorry :(

Ok


Update investigation.md

Updated the correct property value per API call to State

@JohanFreelancer9 Please copy review

Issue#https://github.com/MicrosoftDocs/microsoft-365-docs/issues/5878


Azure CLI Docs are v1 specific

It appears that the Azure cli docs are Azure CLI v1 specific as Azure CLI v2 uses az instead of azure as the keyword.  It also doesn't have an az site.  The docs here should be specific by Azure CLI version.

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

ID: 46653d6e-d057-e698-ef94-32d48e12d39f
Version Independent ID: 74beaf3c-60f9-006d-110a-fbe2348d4797
Content
Content Source
Service: app-service

It actually appears that the Streaming with Azure CLI are for v2 and the downloading with Azure CLI are v1.

Thanks for pointing out. I will work to fix the command in the document.

Tagging @ahmedelnably from the PM team


Consider how to approach CodeableConcept.text

When we denormalize CodeableConcepts, how should we approach text?
Scenarios

Coding.display is present but no Concept.text
Coding.display and Concept.text both present
No Coding.display but Concept.text is present
No Codings at all, but Concept.text is present

1 & 2 are probably easy - just use the Coding.display value as the denormalized display.
3 probably doesn't happen that often. But could.
4 happens in Cerner for inline Medications, Device.type, and probably elsewhere. It's not clear yet if EHRs generally offer a dropdown for that kind of situation and/or allow freeform text entry. In which case, the number of unique text strings is huge (a consideration for using this string in a CUBE).
Use cases

What to show in the dashboard as part of the CUBE?
What to offer to studies for inspection? (i.e. do we want to allow display and/or text as columns for studies to try to parse?)

Current status is that we ignore text right now and always use display


'ascii' codec can't encode character u'\u011f' in position

What would you like to do? 12
Who? XXXXXX
Getting followers: 100%|..| 55360/55513 [02:29<00:00, 222.38it/s]2017-04-23 12:57:53,569 - WARNING - Request return 400 error!
Getting [ XXXX ] followers:   0%|
                                                                                | 0/55360 [00:00<?, ?it/s]
2017-04-23 12:57:54,001 - WARNING - User ozturktugba_ not write because: 'ascii' codec can't encode character u'\u011f' in position 2: ordinal not in range(128)

Having trouble with saving followers to file. w examples/multi_script_CLI.py command: 12

Try to launch with python 3


API_KEY environment variable not working in Docker image due to outdated build - Suggest adding GitHub Actions for automated releases

I have deployed this service as:
faster-whisper:
    image: fedirz/faster-whisper-server:latest-cuda
    container_name: faster-whisper
    restart: unless-stopped
    env_file: /path/to/your/.env
    volumes:
      - /path/to/your/cache:/root/.cache/huggingface
    deploy:
      resources:
        reservations:
          devices:
            - driver: nvidia
              count: all
              capabilities: [gpu]
    networks:
      traefik_network:
        ipv4_address: 172.x.x.x
    labels:
      - "traefik.enable=true"
      # Playground router
      - "traefik.http.routers.whisper-ui.rule=Host(`your-domain.example`) && PathPrefix(`/`)"
      - "traefik.http.routers.whisper-ui.entrypoints=websecure"
      - "traefik.http.routers.whisper-ui.tls.certresolver=myresolver"
      - "traefik.http.routers.whisper-ui.middlewares=bouncer@docker,security@docker,ipallowlist@docker"
      # API router
      - "traefik.http.routers.whisper-api.rule=Host(`your-domain.example`) && PathPrefix(`/v1`)"
      - "traefik.http.routers.whisper-api.entrypoints=websecure"
      - "traefik.http.routers.whisper-api.tls.certresolver=myresolver"
      - "traefik.http.routers.whisper-api.middlewares=bouncer@docker,security@docker"
      # IP allowlist middleware
      - "traefik.http.middlewares.ipallowlist.ipallowlist.sourcerange=<IP_ADDRESS>/24"
      # Service
      - "traefik.http.services.whisper.loadbalancer.server.port=8000"

# API Configuration
API_KEY=sk-xxxxxxxxxxxxxxxxxxxxxxxxxxxxx
LOG_LEVEL=debug
OPENAI_API_KEY="xxxxxxxxxxxxx"
OPENAI_BASE_URL=http://<IP_ADDRESS>:8000/v1/

# Server Settings
UVICORN_HOST=<IP_ADDRESS>
UVICORN_PORT=8000

# CORS Settings
ALLOW_ORIGINS='["https://your-domain.example"]'

# UI Settings
ENABLE_UI=true

# Whisper Model Configuration
WHISPER__MODEL=organization/model-name-v3
WHISPER__INFERENCE_DEVICE=cuda
WHISPER__DEVICE_INDEX=[0,1]
WHISPER__COMPUTE_TYPE=float16
WHISPER__CPU_THREADS=8
WHISPER__NUM_WORKERS=4
WHISPER__TTL=600

# Language Settings
DEFAULT_LANGUAGE=en
DEFAULT_RESPONSE_FORMAT=json

# Model Preloading
PRELOAD_MODELS='["organization/model-name-v3"]'

# Performance-Optimized Timing Settings
MAX_NO_DATA_SECONDS=1.5
MIN_DURATION=0.8
WORD_TIMESTAMP_ERROR_MARGIN=0.15
MAX_INACTIVITY_SECONDS=3.0
INACTIVITY_WINDOW_SECONDS=6.0

Which is working well. I noticed that the API_KEY environment variable is not being respected or used. After inspecting the container's source, I noticed that the Docker Hub images need to be updated with the latest commits. This means that important changes, like the API_KEY functionality from c6b0d43, aren't available in the published Docker images.
To solve this and improve the experience in deployment, I'd suggest adding a GitHub Actions workflow to automatically build and publish Docker images on new releases. Here's a proposed workflow file:
name: Build and Push Docker Images

on:
  release:
    types: [published]
  workflow_dispatch:

jobs:
  docker:
    runs-on: ubuntu-latest
    steps:
      - name: Checkout
        uses: actions/checkout@v4

      - name: Set up QEMU
        uses: docker/setup-qemu-action@v3

      - name: Set up Docker Buildx
        uses: docker/setup-buildx-action@v3

      - name: Login to Docker Hub
        uses: docker/login-action@v3
        with:
          username: ${{ secrets.DOCKERHUB_USERNAME }}
          password: ${{ secrets.DOCKERHUB_TOKEN }}

      - name: Build and push CPU image
        uses: docker/build-push-action@v5
        with:
          context: .
          push: true
          tags: |
            fedirz/faster-whisper-server:latest
            fedirz/faster-whisper-server:${{ github.event.release.tag_name }}

      - name: Build and push CUDA image
        uses: docker/build-push-action@v5
        with:
          context: .
          file: Dockerfile.cuda
          push: true
          tags: |
            fedirz/faster-whisper-server:latest-cuda
            fedirz/faster-whisper-server:${{ github.event.release.tag_name }}-cuda

This workflow would:

Automatically trigger when a new release is created
Build both CPU and CUDA versions of the image
Tag them with both latest and the release version
Push them to Docker Hub

To implement this, you would need to:

Add this file as .github/workflows/docker-publish.yml
Set up DOCKERHUB_USERNAME and DOCKERHUB_TOKEN secrets in the repository settings
Create a new release to trigger the workflow

This would ensure that users always have access to the latest features and fixes when using the Docker images.

Please update the whisper server image in hub.docker.com . The image that is there does not support working with API_KEY. The new image is needed for deployment in runpod.io . Thank you.

@fedirz, could you update this on Docker Hub? I would love to deploy with the latest functionality and keep my containers updated with your latest official releases on docker. Please advise.


Verify contracts

also find the way how we can verify it from the inteface

useful https://github.com/sourcifyeth/remix-sourcify


Too Complex Starting Point

Looks like the starting point for slim application has become complex recently,
When someone does composer create-project, I have to delete a lot of files and folders before starting on development.
can it be simplified to the minimum bare bones, as it is for 2.x

Hi, I found this and maybe it's useful for you too:
$ composer create-project akrabat/slim4-empty my-new-project

Check it out:
https://akrabat.com/slim4-empty-minimal-slim-4-starting-point/

I ABSOLUTELY agree!!!!!!!!! It is utterly infuriating how God Damned complex it is to JUST get up and running. I've spent the past 5+ hours studying Apache documentation to figure out what a "Virtual Host Document Root" is and how to set it from my web host, and still haven't figured it out 😠
I already have to memorize

thousands of configs for my jobs' IIS server, SQL Server, and internal systems,
an overwhelming amount of functions in PHP, Node.js, T-SQL, Excel, xBase, PowerShell,
the never ending amount of frameworks/libraries/modules/extensions in each language,
and now I have to learn how to set the document root from a virtual host to use the slim4 skeleton... like WTF?!?

My human brain is just a balloon that's about to explode!!! But to anyone else who may be feeling my frustration, here is the pertinent docs (but I can't help you with your web host/cloud vendor)

Directive Dictionary
REQUEST_FILENAME
Virtual Host
RewriteRule directive
RewriteBase directive, RewriteCond is right below it
DocumentRoot

... It's a good thing reading for HOUR AND HOURS on end is a TON of fun! Good luck fellow PHP programmers who aren't that familiar with the underlying Apache .htaccess & httpd.conf stuff

@adriansuter also has an excellent skeleton located here:
https://github.com/adriansuter/Slim4-Skeleton


Sky with "thunder" weather condition pops in instantly

If you have a sky layer that's set to show during thunder, it won't fade in gradually, rather it will instantly pop in when the weather has fully transitioned to thunder. Using commands to set the weather, you can see how it takes a couple of seconds while the weather is changing, and then the sky pops in. Then if you set to rain weather, the thunder sky will instantly disappear.
https://user-images.githubusercontent.com/96586974/216759243-d192e8b0-7ee3-4c24-83e6-b27977bbe590.mp4

Upon thorough investigation, it has been determined that the issue at hand is not related to the fade system but rather to the multiply blend. The alpha channel does not seem to be functioning correctly, despite the code suggesting otherwise. Further investigation is needed to determine the root cause of this issue.


Update Zoom plugin instructions

Zoom has made changes in how they create webhooks, so the Zoom plugin guide needs updated.
From Zoom support:

The username and password fields are for webhook apps setup using the V1 webhook. The new marketplace webhooks don't have that option. You can recreate the webhook using the marketplace to circumvent entering the credentials into those fields.

I'd suggest waiting till Feb 11th, because then Zoom will deprecate the old developer portal and move to the new site.
After that new screenshots can be made for the docs because right now the dev portal is full of deprecation warnings and in the set up process you're switching between old and new site all the time.

This can be closed now I guess. @amyblais @jasonblais


Enhancement: Enable method_separation fixer

This PR

[x] enables the method_separation fixer
[x] runs make cs

💁♂ For reference, see https://github.com/FriendsOfPHP/PHP-CS-Fixer/tree/v2.8.0#usage:

method_separation [@Symfony]
Methods must be separated with one blank line.

Thank you, @chartjes!


consideration for stack size for newer versions after 2.0.1

hello, I have been using scene manager 2.0.1 for Godot 3.x game that will make it to android game market as a puzzle. I have read all the work that you have put into this, I must say I was amazed by this addon. (it is the best tool out there in godot plugin market)
one thing I realized is, the stack size keeps growing unless I reset the scene, in later versions of 2.0.1, have you considered adding a stack cleaning/pruning default limit? like 50-100 default and after that many transitions, the stack does remove on the [0] element like a buffer size.

Hello,
Thanks for your kind words. I'm happy that this addon is actually useful and been used in a practical project that will be released to public.
I'm actually trying to improve it and add more features for godot4 at the time and support load screens in future releases too. (not too soon but I wish it happens sooner)
About the question, If I want to be honest, not really.
Is there a need for it?
I mean in terms of space allocation, _stack is just an array of strings. Even if it goes high up to 50-100 or even 10000 elements(which is a super rare condition), it doesn't really take a lot of space.
Plus if I add this behavior and programmer doesn't know or doesn't want it, like you said, after 50-100 reset is gonna automatically happen anyway and that's gonna cause the programmer some confusion or unwanted behavior.
I get that it's a rare condition but... My image of scene_manager is that I got to give all handles to programmer's hand and avoid side effects that programmer does not know about them or cause unwanted behavior.
So that's why I don't see this behavior needed.
What is your vision in this exactly?
Why this is needed?
Does it cause performance problems in your project or...?

my assumption was if someone holds the app on pause and doesn't close it forever, it can go over limits, I do agree with side effect side because all the other scene managers have too many side effects, it should be very visible and obvious to the user like putting stack max size to here on the menu and making it overrideable. It doesn't cause a performance problem, it may cause a memory overflow if the player keeps the app running in the background all the time.

I can make it a feature and by calling a function like set_stack_limit(input: int) -> void, there can be a limit which the programmer can set.
And input can have three different values:

input = -1 which means no limit. (by default)
input = 0 which means do not stack scenes at all.
input > 0 which means a limit for _stack and let's not have a reset functionality on this but instead I think this is a better functionality to have the limit and when the limit reaches, pop one item from the end(which is an old data) and then push one in and like that, we don't have sudden loosing of data anymore.

Is it interesting?

Added and tested it all, but the function name is set_back_limit instead of set_stack_limit and works exactly as I mentioned in previous comment and inside README.md.
For godot3:
https://github.com/maktoobgar/scene_manager/releases/tag/2.1.0
For godot4:
https://github.com/maktoobgar/scene_manager/releases/tag/3.2.0
If you had there was any other ideas or found any bug, let me know.
And happy new year. 🥳


worker process exited on signal 11

Hi All!
From time to time I see this error in error.log
2014/09/15 11:34:24 [alert] 27#0: worker process 28 exited on signal 11
2014/09/15 11:44:28 [alert] 27#0: worker process 21539 exited on signal 11
2014/09/15 12:34:48 [alert] 27#0: worker process 9652 exited on signal 11
2014/09/15 12:44:53 [alert] 27#0: worker process 15638 exited on signal 11
2014/09/15 12:49:20 [alert] 27#0: worker process 32588 exited on signal 11
2014/09/15 12:53:49 [alert] 27#0: worker process 4974 exited on signal 11
2014/09/15 13:12:06 [alert] 27#0: worker process 10578 exited on signal 11
2014/09/15 13:28:17 [alert] 27#0: worker process 2807 exited on signal 11
2014/09/15 13:35:06 [alert] 27#0: worker process 30753 exited on signal 11
I don't have to much experience to debug such problem, what I tried was to attache strace to the worker process, the last few lines from the log:
kill(26701, SIGKILL)                    = 0
--- SIGCHLD {si_signo=SIGCHLD, si_code=CLD_KILLED, si_pid=26701, si_status=SIGKILL, si_utime=1, si_stime=0} ---
gettimeofday({1410787697, 694141}, NULL) = 0
wait4(-1, [{WIFSIGNALED(s) && WTERMSIG(s) == SIGKILL}], WNOHANG, NULL) = 26701
write(4, "2014/09/15 13:28:17 [alert] 2807"..., 77) = 77
wait4(-1, 0x7fff044965fc, WNOHANG, NULL) = 0
rt_sigreturn()                          = 0
sendto(63, "\2\0\0\0\0\0\6\4\0\0\0\0\0\1\0\0\0\1", 18, 0, NULL, 0) = -1 EPIPE (Broken pipe)
--- SIGPIPE {si_signo=SIGPIPE, si_code=SI_USER, si_pid=2807, si_uid=65534} ---
close(16)                               = 0
close(63)                               = 0
--- SIGSEGV {si_signo=SIGSEGV, si_code=SEGV_MAPERR, si_addr=0x10} ---
+++ killed by SIGSEGV +++
Can someone help to identify the root case of this problem ?
Thank you for your time.
Regards
Gabor

Hi,
I have came same error in nginx 1.10 version.
worker process 8912 exited on signal 11
Can you explain where is exact problem.

Still there today on Nginx 1.11.1

having this issue in Nginx 1.13.0

try add
ssl_session_cache    shared:SSL:10m;
in your nginx.conf and reload nginx


Big accuracy drop after simplify

Hi guys, congrats for the work, nice library.
I am trying to use Simplify with ResNet on CIFAR-100 dataset. I use prune.ln_structured method  from Pytorch with amount value is 0.8 and then apply simplify and do fine-tune of the model. As a result I have got the inference time speed up by two times. But I have big accuracy drop: 75% accuracy before prune.ln_structured + simplify + fine-tune and 65% after prune.ln_structured + simplify + fine-tune. Is it expected result? Have you checked accuracy before and after simplify in your cases?

@AndRayt Hi, thanks for your interest. Could you provide a sample code to reproduce the issue?

Hi, thank you for your answer.
I  attached an ipynb file with code.
Here I am training the ResNet-50 model from scratch for 15 epochs and the accuracy is 36% (to make it easier for you to reproduce. Increasing the number of epochs improves accuracy up to 98%). Then I apply the prune.ln_structured method with amount=0.8 and simplify this model. As a result I have good inference time speed up and accuracy is only 1%. Then I fine-tune the model for 8 epochs (half of the number of epochs that was spent on training before) and get accuracy is only 25.7%. It is clear that in this case we could continue to train until we get 36% accuracy again, as after 15 epochs of training from scratch. But if, for example, we take ResNet-18 also on CIFAR-100, then we have an accuracy of 75% after 300 epochs and fine-tune during even the same 300 epochs does not help to achieve 75% (or even 72%) accuracy again. Instead, I have an accuracy of 10% lower after fine-tune.
Please help me to understand this issue. I hope I have described the problem with accuracy drop in sufficient detail.
https://colab.research.google.com/drive/1uuUxNEuNv9yG46eVQdviSY3ud5ETtY7r?usp=sharing

Hi! Did you compare the accuracy of the pruned model before and after applying simplify?

Hi! Do you mean that this is a problem of the torch.prune.ln_structured method as one of the ways to zero the weights of the model? As I know, structured pruning is performed in two stages: 1.Zeroing the weights of the model, 2. Trimming the previously zeroed weights.
And therefore, in my opinion, it is necessary to evaluate the accuracy of the model, exactly like the inference time, before zeroing the weights + trimming weights and after that.
Or is the simplify library designed exclusively for the inference time speed up? Can you suggest another effective way to zero the weights instead of the prune.ln_structured method before applying simplify in this case? Because an accuracy drop of more than 2-3% is an unsatisfactory  result.

Yes, the aim of Simplify is only to reduce the running time for inference on a pruned model. Simplify should not alter accuracy whatsoever, so it is weird if that happens. Could you check if the accuracy drop is given by torch.prune.ln_structured or simplify? If the cause is ln_structured, you could maybe try with a more advanced pruning scheme

Hi! Yes, we have a significant accuracy drop after ln_structured method due to the zeroing of big part (0.8) of the weights, but we also have an accuracy drop after Simplify, because we remove the zeroed weights and leave only part of the model weights, so the accuracy drops to almost zero.
May be do you know any ways for zeroing weights is better than ln_structured?
Your library helped a lot to improve the inference time and I would like to solve the issue now with zeroing the weights (what to use instead of ln_structured?) and get a good final result. Unfortunately, current accuracy drop is very big.

Sorry for the delay. It seems then that there might be a bug with the current Simplify implementation. Under no circumstances the simplify procedure should change the model output. We need to investigate this deeper, thanks for the issue.

What version of torch, torchvision and simplify are you using?


chore: bdd-nh-1592502148 to 0.0.1

chore: Promote bdd-nh-1592502148 to version 0.0.1

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approvers:
If they are not already assigned, you can assign the PR to them by writing /assign  in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


Track allowed requests for state/warp sync

Alternative to https://github.com/paritytech/substrate/pull/10842

bot merge


Type error when running convergence_and_basins_of_attraction

Describe the bug
When running convergence_and_basins_of_attraction with any mapper, I get an Inexact error: Int64. I believe this is due to the array iterations within convergence_and_basins_of_attraction being initialized with type Int while it is intended to store Float times. Also, basins should be initialized with type Int so there are no issues when using the entries as keys to attractors right?
Minimal Working Example
using Pkg; Pkg.add("DynamicalSystems")
Pkg.add("Attractors")
Pkg.instantiate()

using DynamicalSystems
using Attractors

function fold_nf_rule!(du, u, p, t)
    du[1] = -u[1]^2 + p[1]
    return nothing
end

u0 = [10.0]
p0 = [1.0]

fold_nf_ds = CoupledODEs(fold_nf_rule!, u0, p0)

grid = (range(-2, 2; length = 41),)

mapper = AttractorsViaRecurrences(fold_nf_ds, grid; sparse = false)

basins, attractors, convergence = convergence_and_basins_of_attraction(mapper, grid; show_progress = true)

Package versions
Status `~/.julia/environments/v1.10/Manifest.toml`
  [f3fd9213] Attractors v1.21.0
  [61744808] DynamicalSystems v3.3.22

Hi! I will have a look at this.

Looks like a bug. I will fix it.

Thanks!


telegram: Bad Request: can't parse entities: Unsupported start tag "" at byte offset

Hello,
I have got this error
ts=2022-06-23T08:16:30.131Z caller=notify.go:732 level=warn component=dispatcher receiver=telegram integration=telegram[0] msg="Notify attempt failed, will retry later" attempts=1 err="telegram: Bad Request: can't parse entities: Unsupported start tag \"\" at byte offset 265 (400)"

global:
  http_config:
    follow_redirects: true
  resolve_timeout: 1m
  telegram_api_url: https://api.telegram.org
inhibit_rules:
- equal:
  - alertname
  - cluster
  - service
  source_matchers:
  - severity="critical"
  target_matchers:
  - severity="warning"
receivers:
- name: telegram
  telegram_configs:
  - api_url: https://api.telegram.org
    bot_token: 23825:AA
    chat_id: 217****
    disable_notifications: false
    http_config:
      follow_redirects: true
    parse_mode: HTML
    send_resolved: true
route:
  group_by:
  - alertname
  - cluster
  - service
  - alerttype
  - job
  group_interval: 5m
  group_wait: 30s
  receiver: telegram
  repeat_interval: 1m
  routes:
  - matchers:
    - severity=~"warning|critical"
    receiver: telegram

I am getting a similar error
ts=2022-06-25T23:47:56.079Z caller=notify.go:732 level=warn component=dispatcher receiver=telegram integration=telegram[0] msg="Notify attempt failed, will retry later" attempts=1 err="telegram: Bad Request: can't parse entities: Character '-' is reserved and must be escaped with the preceding '\\' (400)"

with this receiver config:
      - name: telegram
        telegram_configs:
          - api_url: https://api.telegram.org
            bot_token: '{{ telegram.bot_token }}'
            chat_id: {{ telegram.chat_id }}
            parse_mode: MarkdownV2
            send_resolved: true

Seems like the message is not properly escaped.

In another script I am using I escaped it using this
      escape() {
        sed -E 's/(\\|_|\*|\[|]|\(|\)|`|~|>|#|\+|-|=|\||\{|}|\.|!)/\\\1/g'
      }

Thanks for reporting. 👍
Closing as duplicate of https://github.com/prometheus/alertmanager/issues/2960.
A fix has been sent in https://github.com/prometheus/alertmanager/pull/2879.


Pseudononymize current value with a custom provider

I have a table/column that I want to pseudonymize where it is not sufficient to work with the LITERAL type (my database does not support what I want to do).
I created a custom provider as follows:
from faker.providers import BaseProvider

class MyProvider(BaseProvider):
    def mypseudo(self):
        return originalValue.replace("a", "b")

which I can use in my strategy file just fine:
providers:
  - providers.myprovider.MyProvider

tables:
  TABLE:
    columns:
      COLUMN:
        type: mypseudo

Describe the solution you'd like
I want to have a way to give the current column value to my mypseudo to perform "things" on it.
Describe alternatives you've considered
I tried using fake_args as written in the docs - but didn't manage to get the current value "injected" into my method.
E.g. something like
...
      COLUMN:
        type: mypseudo
        fake_args:
          current: (SELECT COLUMN)

Did not work.
Am I missing something or is this feature not available yet?

Hi,
LITERAL is the right fit for you here. It's the only mechanism for doing row-level operations.
It's a little counter-intuitive, but the fake_update type and providers in pynonymizer is pre-seeded and that means it doesn't have access to the row data when it replaces the value.
[from README.md]

Pynonymizer's main data replacement mechanism fake_update is a random selection from a small pool of data (--seed-rows controls the available Faker data). This process is chosen for compatibility and speed of operation, but does not guarantee uniqueness

Can you add any info here on what you're trying to do to the column and what database you're using? It's hard to know how likely or unlikely your use-case is and what to recommend.
from your example (although i suspect there might be more to it!)
text replacement is possible in all 3 dbs:

mysql REPLACE function foo: ( REPLACE(foo, 'a', 'b') )
postgres regexp_replace function foo: ( regexp_replace(foo, 'a', 'b') )
mssql REPLACE function foo: ( REPLACE(foo, 'a', 'b') )

Hi,
LITERAL is the right fit for you here. It's the only mechanism for doing row-level operations. It's a little counter-intuitive, but the fake_update type and providers in pynonymizer is pre-seeded and that means it doesn't have access to the row data when it replaces the value.
[from README.md]

Pynonymizer's main data replacement mechanism fake_update is a random selection from a small pool of data (--seed-rows controls the available Faker data). This process is chosen for compatibility and speed of operation, but does not guarantee uniqueness

Hi, ok that's what I was afraid of.

Can you add any info here on what you're trying to do to the column and what database you're using? It's hard to know how likely or unlikely your use-case is and what to recommend.
from your example (although i suspect there might be more to it!) text replacement is possible in all 3 dbs:

mysql REPLACE function foo: ( REPLACE(foo, 'a', 'b') )
postgres regexp_replace function foo: ( regexp_replace(foo, 'a', 'b') )
mssql REPLACE function foo: ( REPLACE(foo, 'a', 'b') )

Well I am using (unfortunately) mysql 5.7 where there is no regex_replace function (and I don't have the possibility to create any custom functions or similar).
My concrete usecase is:
given: key1="10.0|%"; mail="email1<EMAIL_ADDRESS>another_property="false";

here I want to replace the content in the "mail ''key''".
Thanks.

The first example will preserve other keys and leave the rest of the string
alone. Edit the sqlfiddle with more test cases and see!
On Tue, 5 Sep 2023, 17:21 dosera, @.***> wrote:

I agree, you definitely can't do a regexp replace on a string in 5.7.
(edit: to clarify, I dont know of a nice way to do any kind of pattern
match in 5.7. it's a nightmare)
But here's some alternative ideas that might help:

If the application supports other key names, Rename the existing
key1 with something that wont be used, and concat your own value:
( CONCAT('key1="something"; ', REPLACE(foo, 'key1=', 'unusedkey1=')) )
Example http://sqlfiddle.com/#!9/a0b124/3
Do you need to preserve all the other values? you could generate the
whole value with no other info: e.g. literal with some random text function (
CONCAT('key1="', LEFT(MD5(UUID()), 8), '"; mail="a_predictable_value";') )
Example http://sqlfiddle.com/#!9/a0b124/12

Thank your for your help & ideas.
Unfortunately both won't work as the values after mail are relevant as
well. Furthermore the order is not specified as well as if mail is even
part of my string.
—
Reply to this email directly, view it on GitHub
https://github.com/rwnx/pynonymizer/issues/130#issuecomment-1706945798,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABQJ3WEW4V2Z4ZMMR4GPEO3XY5GPNANCNFSM6AAAAAA4LQAHL4
.
You are receiving this because you commented.Message ID:
@.***>


#6876 Remote SQLite driver

Great stuff. I'm excited to try this out

related issues:
https://github.com/dbeaver/dbeaver-ee/issues/1970
https://github.com/dbeaver/dbeaver-ee/issues/1971
https://github.com/dbeaver/dbeaver-ee/issues/1972


Make achievements more prominent?

If I understand correctly, the only way to reach achievements is to click the small "user" icon at the right of the user name?
I believe most people may not find it. How about showing the level of the person instead of that icon?
For instance (with "level 5" being underlined to show it is clickable):
JoeFromHCMC - level 5

or maybe:
JoeFromHCMC
level 5

Thanks @nicolas-raoul! I will send a PR to change the icon.

I agree, IMO the "person" icon next to the username on the nav bar looks like it would lead to a user profile, not an Achievements screen. How about a trophy icon?

It looks OK to me on the right side, but I could go either way I guess.

I have no problem on either side :) , I will post screenshots of  both cases

Thanks @tanvidadu . I'm actually voting for icon on the right (with some padding added). But let's see what others say, haha.

Personally I prefer the screenshot on the left (meaning the trophy is on the right)
A space would be welcome before the trophy though.


Log 2 Scaling for Violin PLot

Fixes #852.  I think if we want some sort of slider, we should make a separate issue.

Can you make this a coordination type so that it can be initialized from the view config?
Maybe a new type called geneExpressionTransform which takes null or string value 'log1p'?
https://github.com/vitessce/vitessce/blob/master/src/app/state/coordination.js#L34 Then in the future it would be easy to use this setting in other components of Vitessce
And is there a reason for log2? It probably does not make a big difference but it looks like ScanPy log1p uses natural log by default https://scanpy.readthedocs.io/en/stable/api/scanpy.pp.log1p.html and also Seurat https://github.com/satijalab/seurat/blob/9843b843ed0c3429d86203011fda00badeb29c2e/src/data_manipulation.cpp#L121 http://www.cplusplus.com/reference/cmath/log1p/

Oh I see you removed the request sorry if you are still working on this

No @keller-mark I was just waiting to make sure tests passed.  That's great feedback.  I'll do that and re-request.


Create regularbracketexpression.java

This question belongs to Codeforces Educational Round 114.
The question can be found here as well.
https://codeforces.com/contest/1574/problem/A
Fixes #2431

DS ALGO PROJECTS ARE NOT COUNTED FOR HACKTOBERFEST21 so i will close this PR


hardcoded username in layouts/yazelix.kdl

firstly thank you for creating such a wonderful work.
your username is hardcoded in layouts/yazelix.kdl (link), making not able to install successfully just by following instructions.

Yeah, noticed this as well. Easy change would be to just refer to ~/.config/... as you've done elsewhere.

oh wow, I should've seen that, hahaha, my bad
thank you, wonderful people for this very important fix!

ive pushed the fix to main. Solved by nyawox and orual


Why did the scale factor change?

Feature request  / 功能建议
Excellent work! I have a few questions about the parameter details. The VAE used for both the 5B model and the 2B model remains unchanged, but why did the scale factor change from 1.15 to 0.7?

Motivation / 动机
Having doubts about the “scale factor” parameter
Your contribution / 您的贡献
None

Frankly speaking, we think scale factor is more like an empirical parameter, although it was originally proposed to normalize the variance of latent.
Essentially, the scale factor adjusts the relative degree of the video and the noise. So we change it empirically.

Is the selection criterion related to the distribution of the training data, or what does the empirical selection depend on?

Depends only on the observed generation results. And we find that small changes do not have a big impact

The difference between 1.15 and 0.7 is not that big. The original intention of scale_factor is to change the standard deviation of latent to 1, so it is 1/std. The average std of latent generated by our vae is around 1.15. We fluctuate the scale_factor based on experience.
As for the SVD of only 0.18, it is because they use a different vae than us.

Thanks for letting me know!
As you said, controlling scale factor wouldn't be an impact (or even worse)

Thank you for your answer, I think I get it now.


SIMULATE does not support GIVEN < and > predicates

simulate apogee_km, perigee_km from satellites_cc given apogee_km < 50000 limit 1000
BQLParseError: Parse error: syntax error near <
This is more of a feature request. I tried this conditional query before I tried apogee_km = 40000 as it seemed more natural given the context---I wanted to view the learned joint distribution of two variables within a given range to make more efficient use of samples.
Also, I'm guessing crosscat does not support this?

related https://github.com/probcomp/bdbcontrib/issues/32

Also, I'm guessing crosscat does not support this?

There exists get_predictive_cdf in ContinuousComponentModel that is not used anywhere in the crosscat pipeline, and it is not exposed through the famous sample_utils (the query) interface.

For arbitrary predicates, rejection sampling is required by simulating followed by filtering using WHERE.


organizeImports: Avoid using full FindAllReferences

For imports, all we really need is getPossibleSymbolReferencePositions followed by checking for exact symbol match. FindAllReferences would look for related symbols, but we don't actually want those in this case.

FYI @mhegazy


Add DisclaimerPage model

Add a Disclaimer and DisclaimerTranslation model which should be similar to Page and PageTranslation.

I made a test by creating to test-models one "Oberklasse" and one "Unterklasse" where Unterklasse inherits from Oberklasse.
When I query all Oberklasse-instance I get all instances, no matter which class was chosen on creation.
(When I query all Unterklasse-instances I only get the instances which were explicity created as Unterklasse-Instances, of course)
So, to keep the behaviour that "Page.objects.all()" only returns the "content pages" (not the disclaimer page) we would need something like a "GeneralPage" which is inherited from Page. Analog thing with PageTranslation.
Under these circumstances I find it more sensible to create a DisclaimerTranslation-model with the attributes region, language and text with region and language as composite primary key (or as constraint unique_together).
To handle the feedback, we can add a flag is_desclaimer_feedback to the region-feedback-model, which defaults to False.

Does the disclaimer page and its translations have the same translation workflow like the normal pages? Do we also keep older versions?
So do we need the attributes, currently_in_translation version, minor_edit-attributes for a disclaimer page?

Oh sorry, it seems that I should have done better research before proposing this.
I found a few ways how to achieve the desired behaviour with a custom model manager, but this is kind of hacky and not worth the drawbacks I think.
So yes, probably it's best to define these as standalone classes... if we use an abstract base class for the content in the future, we can make our optimizations there.

One other option would be to define an AbstractBasePage which is the parent model of Page and Disclaimer and an AbstractBasePageTranslation which is the parent model of PageTranslation and DisclaimerTranslation.
Maybe, this can also be implemented together with #320 Refactor translation models.


Breaks internet connection with Shadowrocket (iOS VPN app)

@ViRb3 commented on Tue Dec 07 2021

Steps to reproduce

Install Shadowrocket iOS app on a M1 Mac
Connect to a VPN server using the app

Expected behavior
Internet works.
Actual behavior
Internet does not work unless you disable AdGuard.
ping still works, but TCP seem to fail. For example, ssh errors, and HTTP requests result in:
$ curl -v google.com
*   Trying <IP_ADDRESS>:80...
* Connected to google.com (<IP_ADDRESS>) port 80 (#0)
> GET / HTTP/1.1
> Host: google.com
> User-Agent: curl/7.77.0
> Accept: */*
>
* Empty reply from server
* Closing connection 0
curl: (52) Empty reply from server

I suspect this may be a more general issue with iOS VPN apps in general, but I have not found a good way to test that.
Your environment
macOS 12.0.1
MacBook Pro M1 Max
AdGuard <IP_ADDRESS>1
Shadowrocket  2.1.92 (1451)

@Chinaski1 commented on Tue Dec 14 2021
Sorry for the late reply.
The thing is that at the moment there is no support for compatibility with iOS VPN applications.
We will work in this direction in the near future.
As a workaround, you can try to configure upstream proxy in the AdGuard advanced settings section provided that the VPN can operate in proxy mode.

This is most probably due to a bug in macOS 12.0.1 where connections bypassed by the transparent proxy break. Works fine on macOS 12.4 beta. Should also work on the most recent release macOS version.

I just tested macOS 12.3.1  with Shadowrocket 2.2.4 and AdGuard <IP_ADDRESS>4, and unfortunately, internet access is lost as soon as both are turned on. If I disable AdGuard, internet connectivity is immediately restored. It doesn't matter in what order I turn them on, as soon as the second one is activated, internet is lost. Each one separately works great.

Regarding @Chinaski1's suggestion, there does not appear to be a way to tell Shadowrocket to "proxy" through AdGuard.

@ViRb3 Hi! Some questions to troubleshoot:

What kind of proxy do you select in Shadowrocket?
Is the proxy in the local network or in the Internet?
Have you tried resetting AdGuard's settings?
Also: you should not add Shadowrocket to AdGuard's list of filtered apps.

Good news, it seems to actually work. The problem was that Safari sometimes needs to be restarted after enabling the VPN. Ping from terminal, even with hostname, always works though, so I think everything is good!


Add simple EU case input data to benchmark folder

Pull request details
Describe the changes made in this pull request
Add simple EU case data to the benchmark folder
List of related issues or pull requests
Concerns #112
Collaboration confirmation
As a contributor I confirm

[x] I read and followed the instructions in README.dev.md
[x] The documentation is up to date with the changes introduced in this Pull Request (or NA)
[x] Tests are passing
[x] Lint is passing

Thank you


Update cutesdr to 1.19

After making all changes to the cask:

[x] brew cask audit --download {{cask_file}} is error-free.
[x] brew cask style --fix {{cask_file}} left no offenses.
[x] The commit message includes the cask’s name and version.

Thank you for the contribution. It was merged directly as fb7e5d9aed5f82229a94588575df661b5652ce6e. Your contribution is still credited to you.


Quic fixes

:pray: thanks @kpp! Included with https://github.com/mxinden/libp2p-perf/commit/2d462123a5a8611064503a996f9d3629dc781406.


Option to show stack trace when an error occurs

Synopsis
The ability to see the stack trace on exceptions thrown by the script would help with future development.
Expected Behaviour
Calling the script with an option similar to ./podcast-parakeet -v url would then print all stack traces underneath the current error message Unable to parse: {podcast link}.
Current Behaviour
Errors in parsing or retrieving a podcast feed are only indicated with a brief, nondescript error message, Unable to parse: {podcast link}.

Resolved with #12


Build pipeline

In this build the configuration pipeline and template to to reuse the task in other pipeline

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.:x: rahultiwari95 sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.


Add constexpr for event and state

Fixes fuchsia-statically-constructed-objects warning in clang-tidy when doing something like this:
constexpr auto my_state = boost::sml::event<class MyState>;
using namespace boost::sml;
constexpr auto my_event = "event1"_e;

Tested on clang and gcc only.

Nice one @ukreator. Thank you!

hm, msvc 2015 doesn't like this:
const auto x = state<class CC>;
when state is constexpr

Thanks, @ukreator


Docs

@FavourTy kindly create an issue stating what you want to do, we will check and assign it to you then you can send in a PR .


Search results API

Blacklight provides us with a default search api, i.e. /catalog.json which is good. It needs a little TLC if we want to continue to use it, or something else if we decide to go another directly.
TODOs

if we do decide to use it, we should revert/change 3aedb309002963e1b0b5a5776c861c788c6317ff and do a better job of it
don't use Solr-based records for search results?

There is an "API" for the moment, when there's a need a re-address this, we can re-ticket it.

There is an "API" for the moment, when there's a need a re-address this, we can re-ticket it.


RTE: Move soft-hyphen functionality to @comet/admin

Soft Hyphen is moved from cms-admin to admin-rte.
Cursor Behaviour:

https://github.com/vivid-planet/comet/assets/122883866/91c55619-955a-4293-8095-2bb2c8f57a02
https://github.com/vivid-planet/comet/assets/122883866/668e13d7-8a36-4911-a9af-9a1ef47234db
Known issue: the icon is not selectable with text highlighting.

Storybook Video:

https://github.com/vivid-planet/comet/assets/122883866/a648604a-ab0b-4ec5-944f-4f36ac837cfe

Admin Video:

https://github.com/vivid-planet/comet/assets/122883866/d0f1a961-7263-44d4-9879-0ba869e1c1a7

I added a change set. Is it written in the style you had in mind?


Add strict option

See #329. @muzimuzhi

With what we already have

In v3.3.4, cache is also updated when a package is updated. This is also the case when tlmgr itself is updated with update-all-packages: false.
_https://github.com/teatimeguest/setup-texlive-action/issues/329#issuecomment-2501659242_

setting strict: true won't change the cache update behavior, but only make the new cache use new cache key (and perhaps differs in whether the old cache is removed)?

If update-all-package: true, then yes. But if not,

This also seems not ideal. As scheme and collection names are both accepted, when a new package is added to a scheme or collection, then it will never be installed when cache is hit? (https://github.com/teatimeguest/setup-texlive-action/issues/329#issuecomment-2496574569)

this still remains.
Would the change in v3.3.4 be sufficient?

On rethinking, I now think it all depends on what the expected installation outcome is after a cache is restored.

With cache: false, all schemes/collections/packages are installed from scratch, thus all latest versions are installed.
With cache: true and when no cache is restored (due to no existing or no matching caches), it falls back to the first case.
With cache: true and when a cache is indeed restored,

3.1 if update-all-package: true is set, then the resulting installation is the same as previous cases;
3.2 if update-all-package: false is set (which is the default in v3.3.4), then the installation would finish with historic list of packages (ignoring auto-remove and auto-install ones, see doc of tlmgr update) and their historic versions.

Therefore it seems to me, update-all-package: true should be set by default.
(Lacking of lock file does change the installation staggery and complicate the situations...)

Therefore it seems to me, update-all-package: true should be set by default.

I understand that this is ideal for package/class maintainers. But for book/paper authors,

3.2 if update-all-package: false is set (which is the default in v3.3.4), then the installation would finish with historic list of packages [...] and their historic versions.

I still think this is rather expected, more preferable behavior; It is just as packages are not updated in a local environment unless one consciously does tlmgr update.
In my experience, editorial processes and/or submission systems often require an older version of TeX Live. For example, if you look at the section "How do I identify which submission system I’m using?" in the FAQs on this page of Springer Nature, you will find that TeX Live 2017 is still in use. (This is the main reason why the action supports the installation of older versions.)
Rarely is the version to be used explicitly stated in such a way. Sometimes editors later complain that they cannot compile. Thus, in order to avoid problems, it is unfortunately an effective strategy to avoid updating packages and to avoid using new features as much as possible.
In such a situation, it would be more beneficial to be able to check the compiled results quickly than to have packages updated. This is my concern with defaulting to update.

However, active development can keep a cache alive for an unintentionally long period of time without packages being updated. This can indeed be considered a problem.
I think a possible solution would be to update packages (and cache if appropriate) when the cache age exceeds a certain value (e.g., 7 days) even if update-all-packages: false. This would not prevent either quick cache restoration or regular package updates.
Please let me know if you have any thoughts on this.

For book/paper authors, maybe docker image is the best choice to get a version-locked full LaTeX installation (exceptions are the historic yearly versions of TeX Live). For example islandoftex provides weekly snapshots of TeX Live images.
Ideally the cache mechanism should be transparent to users. That is, whether a cache is restored or not should only affect the installation time, but not what will be installed. update-all-packages: false breaks this thus it becomes users' responsibility to be aware of cache restoration.
As caches without access will expire after 7 days, a rather short duration, I'm afraid update-all-packages: false won't always help.

I see what's the key of my concern: cache is far from a persistent storage which makes the behavior of setting update-all-packages: false not reliable. As a cache expires in 7 days and an artifact expires in at most 90 days, the only persistent storage is the release asset.

For book/paper authors, maybe docker image is the best choice to get a version-locked full LaTeX installation (exceptions are the historic yearly versions of TeX Live). For example islandoftex provides weekly snapshots of TeX Live images.

I understand the point, but you can get by with what is described in https://github.com/teatimeguest/setup-texlive-action/issues/289#issuecomment-1987086225 on the other hand (from my perspective) running update-all-packages: false should do exactly what the key says, i.e. DO NOT UPDATE ANYTHING, no matter the amount of time (7 days or 90 days).

I understand the point, but you can get by with what is described in #289 (comment)

Great! I didn't know there are daily archives of texlive (or once read but then totally forgot  it).

on the other hand (from my perspective) running update-all-packages: false should do exactly what the key says, i.e. DO NOT UPDATE ANYTHING, no matter the amount of time (7 days or 90 days).

@pablgonz Sounds like a complain about bad naming :), as it has been documented as "Update all TeX packages when cache restored." from day one (see 029f57a53e4930a373ae0491dfda9ab8982066bc).


Toggling view should retail the filter query

When pressing f to toggle full-screen, retain the filter string (query) so that the filtering will remain there upon returning to the standard view.
The lack of carrying the query makes the experience painful when dealing with many stories.
Also, restoring the scroll position in the stories container will improve UX.
Ps: apologies for ignoring the issue template.

fixed, scheduled for a release

@tajo  Any ETA on the new release?

by the end of year, going to a bigger release with React 19 and Vite v6

Great release, @tajo :)
Is there any chance of an option to preserve the query in the URL (e.g., q=query) so that it's shareable and so that refreshing the page won't lose it?

@phaistonian yea, that would be a nice feature if anyone wants to implement it


How get access to the CSSOM?

Hi
I am loving this project so far! I'm trying to get information about the CSSOM for a parsed document and I'm wondering how do I even get to it? I couldn't find any documentation related to it.

Hi @alexanderuv thanks for the question - the CSSOM is pretty much divided into the following areas:

Style property for inline styles
On link and style DOM elements via Sheet property
To get the computed style via the window (getComputedStyle)

Hope that helps!
Remark: The CSSOM is not very extensive and should vastly improve once the new release incl. AngleSharp.Css is out. Unfortunately, I did not have much time "recently", which is why this is postponed indefinitely... The implementation of getComputedStyle is "pragmatic" / "experimental", i.e., very simple.


Overview header font size

Describe the bug
The font size for the overview header doesn't match the header for the details pages. Should also hide the info icon for an empty state (i.e., when there are no providers).
To Reproduce
Steps to reproduce the behavior:

Go to 'Overview'
See error

Expected behavior
All headers should be the same font size.
Screenshots
Overview:

Ocp details:

Fixed by https://github.com/project-koku/koku-ui/pull/800