[Snyk] Security upgrade CrispyWaffle from 8.2.149 to 8.2.185

User description

Snyk has created this PR to fix 1 vulnerabilities in the nuget dependencies of this project.
Snyk changed the following file(s):

Src/VTEX.Health/VTEX.Health.csproj

Vulnerabilities that will be fixed with an upgrade:

Issue
Score

Inefficient Algorithmic Complexity SNYK-DOTNET-SYSTEMTEXTJSON-8168848
  721

[!IMPORTANT]

Check the changes in this PR to ensure they won't cause issues with your project.
Max score is 1000. Note that the real score may have changed since the PR was raised.
This PR was automatically created by Snyk using the credentials of a real user.

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open fix PRs.
For more information:
🧐 View latest project report
📜 Customise PR templates
🛠 Adjust project settings
📚 Read about Snyk's upgrade logic

Learn how to fix vulnerabilities with free interactive lessons:
🦉 Learn about vulnerability in an interactive lesson of Snyk Learn.

Description

Upgraded CrispyWaffle package to address security vulnerabilities.
This upgrade resolves the issue identified as SNYK-DOTNET-SYSTEMTEXTJSON-8168848 with a score of 721.

Changes walkthrough 📝
Relevant filesDependencies

VTEX.Health.csprojUpgrade CrispyWaffle package to fix vulnerabilities

Src/VTEX.Health/VTEX.Health.csproj
Upgraded CrispyWaffle package from version 8.2.149 to 8.2.185.

+1/-1

💡 Penify usage:
Comment /help on the PR to get a list of all available Penify tools and their descriptions

:white_check_mark: Build VTEX-SDK-dotnet 2.3.1142 completed (commit https://github.com/guibranco/VTEX-SDK-dotnet/commit/757ffc5faa by @gstraccini[bot])


Adding -DkimSelector parameter for more flexibility

Adding -DkimSelector parameter for more flexibility: https://www.reddit.com/r/PowerShell/comments/occgr2/powershell_script_for_checking_spf_dkim_and_dmarc/h3xlify?utm_source=share&utm_medium=web2x&context=3

Added in version 1.4


refactor: use swagger for table model in search

Summary of Changes
Use swagger for table model in search to make it consistent.
Tests
What tests did you add or modify and why? If no tests were added or modified, explain why. Remove this line
Documentation
What documentation did you add or modify and why? Add any relevant links then remove this line
CheckList
Make sure you have checked all steps below to ensure a timely review.

[ ] PR title addresses the issue accurately and concisely. Example: "Updates the version of Flask to v1.0.2"

In case you are adding a dependency, check if the license complies with the ASF 3rd Party License Policy.

[ ] PR includes a summary of changes.
[ ] PR adds unit tests, updates existing unit tests, OR documents why no test additions or modifications are needed.
[ ] In case of new functionality, my PR adds documentation that describes how to use it.

All the public functions and the classes in the PR contain docstrings that explain what it does

[ ] PR passes make test

Codecov Report

Merging #110 into master will increase coverage by 0.08%.
The diff coverage is 84.61%.

@@            Coverage Diff             @@
##           master     #110      +/-   ##
==========================================
+ Coverage   75.20%   75.28%   +0.08%
==========================================
  Files          17       17
  Lines         613      615       +2
  Branches       76       77       +1
==========================================
+ Hits          461      463       +2
  Misses        127      127
  Partials       25       25

Impacted Files
Coverage Δ

search_service/api/table.py
100.00% <ø> (ø)

search_service/proxy/base.py
68.18% <ø> (ø)

search_service/proxy/atlas.py
49.03% <33.33%> (ø)

search_service/models/table.py
100.00% <100.00%> (ø)

search_service/proxy/elasticsearch.py
76.57% <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 e02a563...9f80c3c. Read the comment docs.


Re-introduce _tag field and deprecate it

New Pull Request Checklist

[x] I have read and understood the CONTRIBUTING guide

[x] I have read the Documentation

[x] I have searched for a similar pull request in the project and found none

[x] I have updated this branch with the latest master to avoid conflicts (via merge from master or rebase)

[x] I have added the required tests to prove the fix/feature I am adding

[x] I have updated the documentation (if necessary)

[x] I have run the tests and they pass

[x] I have run the lint and it passes (pod lib lint)

This merge request fixes / refers to the following issues: #1177 #1223
Pull Request Description
In #1177 the tag property was deprecated and superseded by representedObject. However, the underlying _tag field was removed completely instead of simply deprecated. This caused the change to become a breaking change.
This re-introduces the underlying _tag field and also marks it as deprecated.

Note that I've also already increased the version to 3.7.2, so that we can quickly release a follow-up version here to fix the breakage.

@sushichop Thanks!

@ffried It's my pleasure🙂


unresolved import iced::Alignment

Was trying to run the example code "Counter"
[rust-analyzer unresolved-import] [E] unresolved import
   ───────────────────────────────────────────────────────
   [rustc E0432] [E] unresolved import `iced::Alignment`
   no `Alignment` in the root

error[E0432]: unresolved import `iced::Alignment`
 --> src/main.rs:2:13
  |
2 |     button, Alignment, Button, Column, Element, Sandbox, Settings, Text,
  |             ^^^^^^^^^ no `Alignment` in the root

For more information about this error, try `rustc --explain E0432`.

@tarkah Thanks, problem solved.


updated ledger, wallet, etc. in save schema howto

I didn't understand signoff.  Let me submit the PR again


[CI] Security: Upgrade to docker image to Alpine 3.15

Codecov Report

Merging #4838 (c9127de) into master (48d07dd) will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff           @@
##           master   #4838   +/-   ##
======================================
  Coverage    9.31%   9.31%
======================================
  Files          61      61
  Lines        5151    5151
======================================
  Hits          480     480
  Misses       4553    4553
  Partials      118     118

Flag
Coverage Δ

gointegrationtests
9.31% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out 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 1aa0dc1...c9127de. Read the comment docs.

The golang unit tests have failed on two separate runs of the workflows for this PR.

@Revolyssup @navendu-pottekkat @piyushsingariya do you know what is causing this?

@Revolyssup @navendu-pottekkat @piyushsingariya do you know what is causing this?

@Revolyssup @navendu-pottekkat @piyushsingariya do you know what is causing this?

Has been fixed on the master branch

@Revolyssup @navendu-pottekkat @piyushsingariya do you know what is causing this?

Has been fixed on the master branch

Thank you, @Revolyssup. Branch sync'ed with master.

Thank you, @Revolyssup. Branch sync'ed with master.

All green lights 🟢

All green lights 🟢


ASP.NET 5 support

ASP.NET 5 includes substantial changes that the BugSnag notifier will need to be modified to support, including:

all MVC-related types have moved to Microsoft.AspNet namespace; the exception handler filter will need to be modified to cater for these.
ConfigurationSettingsHandlers are no longer used (no web.config, ideally), so setup/config should be done via an IApplicationBuilder extension method.
it would be nice to also have an ILogger implementation and notify at certain log levels - not sure if this behaviour is in line with that provided on other platforms.

This will most likely need to be provided via a separate NuGet package; it may be a battle to try and support different MVC versions in the one assembly.

Thanks @samritchie! :) This is a great list to consider for ASP.NET 5 support. We've got these on our list as well for an eventual update, but we are always accepting PRs in the meantime.

Any update on this since .Net Core RTM?

@samritchie @jaredcnance I have just pushed v2 (alpha) of our .net notifier which support ASP.NET Core to Nuget. It is an alpha release right now but we would love to have any feedback on it https://www.nuget.org/packages/Bugsnag.AspNet.Core/


HIVE-25813: CTLT query on JDBCStorageHandler fails with Null exception

What changes were proposed in this pull request?
Fixed the null pointer exception for CTLT operation.

Why are the changes needed?
Otherwise, CTLT operation based on JDBC storage handler is failing

Does this PR introduce any user-facing change?
No

How was this patch tested?
Local machine, Remote cluster

Can we add qtest to validate the same?


SAG Mill recipes sent through IMC don't get their energy cost set properly.

No matter what value I pass it seems to default to 400RF.
Here's my code that is sending the message: https://github.com/ganymedes01/Another-One-Bites-the-Dust/blob/mc1.7/src/main/java/ganymedes01/aobd/recipes/modules/EnderIOModule.java

Nvm, my fault for passing it a float instead of an int.


@MainActor isolation on expect functions

All the expect functions are @MainActor isolated and the async expression closures as well. I'm not sure, if this is always needed. The evaluate function uses an internal actor to protect the isFulfilled state. If I want to test some expression that has to run on the MainActor this is useful.
But for code that doesn't have to be isolated this requires MainActor isolation code to be added. I don't want to add this to the call site, if I don't actually need the isolation for data safety.
I think the async expression closures should also be @Sendable ?
I forked the code and I'm experimenting with non isolated expect function overloads. Unfortunately this requires different names it seems. Apparently Swift doesn't support function overloading with and without @MainActor isolation.

I think annotating the async closures with @MainActor makes them Sendable. Agree that it would be nice to skip adding the @MainActor on the client side. Would it work to remove the MainActor as default and just mark the closures as sendable?

Yes, I think it should be possible to just mark the closure as required to be @Sendable. I can take a look, when I have time.


ThemeVariant property definitions to ThemeVariantScope + fix DataGrid theme variant switch

What does the pull request do?
Fixes: #10292 - Popup wasn't properly inheriting theme variant from the placement target
Fixes: #10201 - Workaround for #10345, default value of actual theme now is "null" instead of "Light" (correct value is inherited)
Fixes: #9410 - Fluent DataGrid theme wasn't using theme dictionaries
Fixes: #10319 - Some Simple control themes that were ported from Fluent had the same problem as DataGrid, but instead I simplified them to be in-line with other simple control themes.
Fixes: #10309 - Invalid property name registration was breaking bindings.

You can test this PR using the following package version. 11.0.999-cibuild0029386-beta. (feed url: https://pkgs.dev.azure.com/AvaloniaUI/AvaloniaUI/_packaging/avalonia-all/nuget/v3/index.json) [PRBUILDID]

You can test this PR using the following package version. 11.0.999-cibuild0029389-beta. (feed url: https://pkgs.dev.azure.com/AvaloniaUI/AvaloniaUI/_packaging/avalonia-all/nuget/v3/index.json) [PRBUILDID]

I never did get through the original PR. Just noticed this though:
/// <summary>
/// Specifies a UI theme variant that should be used for the
/// </summary>
[TypeConverter(typeof(ThemeVariantTypeConverter))]
public sealed record ThemeVariant

Missing comments. Just was never completed.
Is record really the best to use here?

I know it had many uses now but it was originally designed to represent a row in a database table and give boilerplate implementations for equals/etc based on values. It doesn't look quite right in this context. record class would look a bit better but I suppose that's preference.
You don't actually take advantage of any record features from what I see. You re-implemented Equals and GetHashCode anyway.
You DID NOT add new implementations for the equality operators... so == (implemented by the compiler) will be different than .Equals() which is bad practice.

Anyway, just added this here since you are making some fixes. I can delete and create a new issue if you want.

Missing comments. Just was never completed.

Thanks, will fix that later.

record class would look a bit better but I suppose that's preference.

"record class" and "record" is the same thing.

You DID NOT add new implementations for the equality operators

No, record will use user defined Equals method. I did double check this.

You don't actually take advantage of any record features from what I see

Tbh, initially when I wrote this class, there was more benefits of using record. But then I changed much.
Still there are some benefits while no disadvantages.
See sharplab output for this type.
Differences:

record automatically implements IEquatable
record automatically defienes == and != operators
record automatically overrides Equals(object) method and calls user defined Equals(ThemeVariant)
record defines Clone for variant with {} syntax, which is rather useless as there are no publicly settable properties
It also has defined PrintMembers method, which seems to be leftover from original generated ToString, which is replaced by user-written one. But since it's a private trimmable method, that's not a problem.

@amwx I haven't checked what's wrong again with #10110, but by any chance could it be the same as https://github.com/AvaloniaUI/Avalonia/issues/10201? And possibly fixed in this PR.

You can test this PR using the following package version. 11.0.999-cibuild0030054-beta. (feed url: https://pkgs.dev.azure.com/AvaloniaUI/AvaloniaUI/_packaging/avalonia-all/nuget/v3/index.json) [PRBUILDID]

@amwx I haven't checked what's wrong again with #10110, but by any chance could it be the same as #10201? And possibly fixed in this PR.

Just tested with this branch and HighContrast theme seems to be working correctly now (both on load and switching)

You can test this PR using the following package version. 11.0.999-cibuild0030086-beta. (feed url: https://pkgs.dev.azure.com/AvaloniaUI/AvaloniaUI/_packaging/avalonia-all/nuget/v3/index.json) [PRBUILDID]

You can test this PR using the following package version. 11.0.999-cibuild0030339-beta. (feed url: https://pkgs.dev.azure.com/AvaloniaUI/AvaloniaUI/_packaging/avalonia-all/nuget/v3/index.json) [PRBUILDID]


UserAvatar: Ignore color inversion on iOS.

ed69e1ec0 inadvertently removed this when we switched away from
ImageBackground, but it seems like something that we should include,
since avatars will look better non-inverted, and inverting arbitrary
images isn't likely to be very useful for contrast/accessibility
reasons.
Fixes: #4765

Looks good, thanks, merged!


Add support for multiple connection hops in tx chan-open-ack

Part of: #3940
Summary
tx chan-open-ack

[x] Replace call to tx_chan_cmd! macro with an inline function on TxChanOpenAckCmd
[x] Add build_multihop_chan_open_ack() to method build_chan_open_ack_and_send() to enable the construction of MsgChannelOpenAck messages when multiple connection hops have been specified.
[x] Modify validated_expected_channel() to support channel ends with multiple connection hops on the destination chain.

Tests

[x] Manual testing for single-hop channel handshake
[x] Manual testing for multihop channel handshake with 2 connection hops
[x] Manual testing for multihop channel handshake with 2+ connection hops

For Admin Use

[ ] Not duplicate issue
[ ] Appropriate labels applied
[ ] Appropriate milestone (priority) applied
[ ] Appropriate contributors tagged
[x] Contributor assigned/self-assigned

Done in https://github.com/informalsystems/hermes/pull/4050


Cache Control for 404 response

Environment
Cloudflare Workers (i think all envs have same logic)
Reproduction
Try to access localhost:3000/_nuxt/test.css and see Cache Control headers.
Describe the bug
I think nitro shouldnt cache 404's errors by default or atleast with different rules. For now we are facing issue when user visited page after deploy and 404 response was cached in browser.
Additional context
No response
Logs
No response

Hi. Can you please share nuxt/nitro versions?

It's latest 2.2.3

@pi0 here is example, https://test-docus.vercel.app/_nuxt/test.css.
As you see cache-control exists here, while response status is 404
age: 3
cache-control: public, max-age=2592000, immutable
content-encoding: br
content-type: text/html;charset=utf-8
date: Wed, 01 Mar 2023 13:11:00 GMT
server: Vercel

Hi. I'm not sure if the deployment (cannot see source of open it anymore) were depending on cache headers generated by vercel edge or nitro so please feel free to open new issue with reproduction if facing similar issues.


How to fill a form inside an iframe with Puppeteer-Sharp

WebUrl: http://w8r.fut168.com/
SearchCode:11188
I have tried to check the frames with page.Frames, but since the iframes have no name and id, difficult to find the correct frame I am looking for.

Did you try to find it with the URL?

Did you try to find it with the URL?

Yeah, But Every time you open a web page, the iframe URL is different.

Maybe using a Contains? There is not much Puppeteer can do...


Safari test should work with signing certificate

Previously, there was no signing certificate set in the Xcode project, but it's annoying for me to change it to the correct certificate each time. It's better to force xcodebuild command to run without a certificate instead.
https://github.com/refined-github/refined-github/runs/4524894735?check_suite_focus=true
https://github.com/refined-github/refined-github/blob/a79f49bdb7e571bd8402f3cddba8154220bd0e0c/package.json#L13

I have disabled the test until this is resolved: https://github.com/refined-github/refined-github/commit/a79f49bdb7e571bd8402f3cddba8154220bd0e0c#diff-faff1af3d8ff408964a57b2e475f69a6b7c7b71c9978cccc8f471798caac2c88R79

I have a solution for this that goes through a local ENV file. This allows you to set your own certificate without setting it for everyone (the change on main broke CI as well as my build)


[Issue]: docs improvement for conversation Pattern

Describe the issue

guys, please specify like summary_args={"summar_prompt": "Your prompt here"}
because line in image shows like we need to do
initiate_chat(
summay_prompt =
)
i needed to look into source code

    @staticmethod
    def _reflection_with_llm_as_summary(sender, recipient, summary_args):
        prompt = summary_args.get("summary_prompt")
        prompt = ConversableAgent.DEFAULT_SUMMARY_PROMPT if prompt is None else prompt
        if not isinstance(prompt, str):
            raise ValueError("The summary_prompt must be a string.")

Steps to reproduce
No response
Screenshots and logs
No response
Additional Information
No response

You are welcome to submit a PR to fix it.

sure


Port svmlight and domainadaptation svm examples

there is some sort of UB with the svm example, I get different results sometimes on my machine too

the typemap from jblas matrices to sgvectors is broken


Test The Trained Model error

hi , im on this step get this
Total progress100% 20/20 [00:03<00:00,  5.72it/s]
Error completing request
Arguments: (0, '', '', 'None', 'None', None, None, None, None, None, 0, 20, 0, 4, 0, 1, False, False, 1, 1, 7, 0.75, -1.0, -1.0, 0, 0, 0, False, 512, 512, 0, False, 32, 0, '', '', 0, '\nCFG Scale should be 2 or lower.\n\n', True, True, '', '', True, 50, True, 1, 0, False, 4, 1, 'Recommended settings: Sampling Steps: 80-100, Sampler: Euler a, Denoising strength: 0.8', 128, 8, ['left', 'right', 'up', 'down'], 1, 0.05, 128, 4, 0, ['left', 'right', 'up', 'down'], False, False, False, False, '', 'Will upscale the image by the selected scale factor; use width and height sliders to set tile size', 64, 0, 2, 1, '', 0, '', True, False, False) {}
Traceback (most recent call last):
File "/content/gdrive/MyDrive/sd/stable-diffusion-webui/modules/call_queue.py", line 45, in f
res = list(func(*args, **kwargs))
File "/content/gdrive/MyDrive/sd/stable-diffusion-webui/modules/call_queue.py", line 28, in f
res = func(*args, **kwargs)
File "/content/gdrive/MyDrive/sd/stable-diffusion-webui/modules/img2img.py", line 152, in img2img
processed = process_images(p)
File "/content/gdrive/MyDrive/sd/stable-diffusion-webui/modules/processing.py", line 464, in process_images
res = process_images_inner(p)
File "/content/gdrive/MyDrive/sd/stable-diffusion-webui/modules/processing.py", line 532, in process_images_inner
p.init(p.all_prompts, p.all_seeds, p.all_subseeds)
File "/content/gdrive/MyDrive/sd/stable-diffusion-webui/modules/processing.py", line 832, in init
image = img.convert("RGB")
AttributeError: 'NoneType' object has no attribute 'convert'
Downloading: "https://storage.googleapis.com/sfr-vision-language-research/BLIP/models/model_base_caption_capfilt_large.pth" to /content/gdrive/MyDrive/sd/stable-diffusion-webui/models/BLIP/model_base_caption_capfilt_large.pth
100% 855M/855M [00:38<00:00, 23.3MB/s]
Downloading100% 232k/232k [00:00<00:00, 263kB/s]
Downloading100% 28.0/28.0 [00:00<00:00, 23.5kB/s]
Downloading100% 570/570 [00:00<00:00, 361kB/s]
load checkpoint from /content/gdrive/MyDrive/sd/stable-diffusion-webui/models/BLIP/model_base_caption_capfilt_large.pth
100%|███████████████████████████████████████| 890M/890M [00:14<00:00, 65.2MiB/s]
Error interrogating
Traceback (most recent call last):
File "/content/gdrive/MyDrive/sd/stable-diffusion-webui/modules/interrogate.py", line 148, in interrogate
caption = self.generate_caption(pil_image)
File "/content/gdrive/MyDrive/sd/stable-diffusion-webui/modules/interrogate.py", line 126, in generate_caption
gpu_image = transforms.Compose([
File "/usr/local/lib/python3.8/dist-packages/torchvision/transforms/transforms.py", line 95, in call
img = t(img)
File "/usr/local/lib/python3.8/dist-packages/torch/nn/modules/module.py", line 1190, in _call_impl
return forward_call(*input, **kwargs)
File "/usr/local/lib/python3.8/dist-packages/torchvision/transforms/transforms.py", line 346, in forward
return F.resize(img, self.size, self.interpolation, self.max_size, self.antialias)
File "/usr/local/lib/python3.8/dist-packages/torchvision/transforms/functional.py", line 462, in resize
_, image_height, image_width = get_dimensions(img)
File "/usr/local/lib/python3.8/dist-packages/torchvision/transforms/functional.py", line 75, in get_dimensions
return F_pil.get_dimensions(img)
File "/usr/local/lib/python3.8/dist-packages/torchvision/transforms/functional_pil.py", line 33, in get_dimensions
raise TypeError(f"Unexpected type {type(img)}")
TypeError: Unexpected type <class 'NoneType'>
Traceback (most recent call last):
File "/content/gdrive/MyDrive/sd/stable-diffusion-webui/modules/interrogate.py", line 148, in interrogate
caption = self.generate_caption(pil_image)
File "/content/gdrive/MyDrive/sd/stable-diffusion-webui/modules/interrogate.py", line 126, in generate_caption
gpu_image = transforms.Compose([
File "/usr/local/lib/python3.8/dist-packages/torchvision/transforms/transforms.py", line 95, in call
img = t(img)
File "/usr/local/lib/python3.8/dist-packages/torch/nn/modules/module.py", line 1190, in _call_impl
return forward_call(*input, **kwargs)
File "/usr/local/lib/python3.8/dist-packages/torchvision/transforms/transforms.py", line 346, in forward
return F.resize(img, self.size, self.interpolation, self.max_size, self.antialias)
File "/usr/local/lib/python3.8/dist-packages/torchvision/transforms/functional.py", line 462, in resize
_, image_height, image_width = get_dimensions(img)
File "/usr/local/lib/python3.8/dist-packages/torchvision/transforms/functional.py", line 75, in get_dimensions
return F_pil.get_dimensions(img)
File "/usr/local/lib/python3.8/dist-packages/torchvision/transforms/functional_pil.py", line 33, in get_dimensions
raise TypeError(f"Unexpected type {type(img)}")
TypeError: Unexpected type <class 'NoneType'>
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "/usr/local/lib/python3.8/dist-packages/gradio/routes.py", line 284, in run_predict
output = await app.blocks.process_api(
File "/usr/local/lib/python3.8/dist-packages/gradio/blocks.py", line 982, in process_api
result = await self.call_function(fn_index, inputs, iterator)
File "/usr/local/lib/python3.8/dist-packages/gradio/blocks.py", line 824, in call_function
prediction = await anyio.to_thread.run_sync(
File "/usr/local/lib/python3.8/dist-packages/anyio/to_thread.py", line 31, in run_sync
return await get_asynclib().run_sync_in_worker_thread(
File "/usr/local/lib/python3.8/dist-packages/anyio/_backends/_asyncio.py", line 937, in run_sync_in_worker_thread
return await future
File "/usr/local/lib/python3.8/dist-packages/anyio/_backends/_asyncio.py", line 867, in run
result = context.run(func, *args)
File "/content/gdrive/MyDrive/sd/stable-diffusion-webui/modules/ui.py", line 269, in interrogate
prompt = shared.interrogator.interrogate(image)
File "/content/gdrive/MyDrive/sd/stable-diffusion-webui/modules/interrogate.py", line 177, in interrogate
res += ""
TypeError: unsupported operand type(s) for +=: 'NoneType' and 'str'

i think you are right


Will negative samples involve the input sentence?

Hi, thanks for sharing the great work!
I am wondering that if the negative samples are randomly sample from the training data, will it possibly involves the input sentence?

The training data for each batch is randomly selected, and the queue is updated in the order of selection first-in-first-out, but this does not guarantee that there are duplicate entries in the data to interfere with this. The data we use is consistent with the previous work.

Thanks for your instruction!
And if it is in the first iteration of training, the queue will be randomly initialized with the size of 128 negative samples, right? In this way, will the negative samples possibly contain the training input and couse the duplication?

The initial 128 negative samples are randomly generated values rather than randomly selected negative pieces, which does not lead to duplicating negative examples. In addition, these random values are quickly replaced and are used only once during the entire training.

Thanks for your explanation, can you show me where the code that inital 128 negative samples is?

Hi, please refer here:
https://github.com/xbdxwyh/mocose/blob/45cbc8105253b969541e8a4ae2e27753d98f9966/mocose.py#L309


Fix ibrowse version to 4.0.0

Fixes myfreeweb/httpotion#124
It seems ibrowse 4.0.1 has the problem.
So I fix ibrowse version to 4.0.0 in mix.exs.
This will remove after bugfix.

Is the bug reported to ibrowse?

No, not yet.


mGBA Pokemon Fire Red not saving

Self explanitory. Pokemon Red on mGBA does not save, instead it gets infinitely stuck on a page with "Save Failed. Checking the backup memory... Please wait. 'Time required: about 1 minute'"

Can you send a screenshot?

That's very strange, I've never had this issue before. Does this happen all the time?

This is definitely not Red, but FireRed, for the GBA, right? I could test that one myself and try to recreate it, if nothing else.

Just tried both revisions of the cart: neither have problems saving in the first room. Where does this problem occur at? Do your cartridge dumps match either of these SHA-1's?
FireRed Rev0: 41cb23d8dccc8ebd7c649cd8fbb58eeace6e2fdc
FireRed Rev1: dd5945db9b930750cb39d00c84da8571feebf417
If you don't know how to generate an SHA-1, just use this. If your roms don't match either of these, your file is bad.

Ah, looks like my file is bad, the checksum is c89b51766c8a8b23332c27dc45a270cea912bc21

Redump your cart, or "employ other methods". You should know where to look; I am not going to help with the latter.


Rename matter type names to align with Apply

This renames the two matter type names to be consistent with their naming in Apply (as suggested by the now deleted TODO comments).
Link to story

This may not be as straight forward as it seems. I think it will impact Apply as we expect certain values there too.
I think it may be why I left a comment rather than just changing it 🤔
I would check with Dave to make sure he's happy with the changes too 👍

Closing because I merged #851 and there is a JIRA ticket that captures the original "TODO"


Fix merge conflict introduced in #147

This fixes a bad merge introduced in #147

pre-commit.ci autofix


Useless (?) separator on site header

On the left side of the login dropdown menu there is a separator pipe (see below) which was introduced in #292, but in the current layout looks like a UI bug in my opinion. Does it serve any purpose anymore?
https://github.com/NetrunnerDB/netrunnerdb/blob/6108ec616537f20abcf7fe9504f1cc55bf28aed8/app/Resources/views/layout.html.twig#L152

i'm going to remove this as part of an upcoming set of changes that will require some navbar changes.
AFAICT, this extra LI doesn't make a difference on the page.  If you see some funky behavior / layout caused by this, please let us know with a screenshot and some browser/platform info.
Thanks!

Obsolete.


Add multi session support

Fix: #35
Change cert_chain_buffer to array to store 0-7 slot_id cert_chain;
Add multi session support by inputting multi session key;
The follow test has passed:

Signed-off-by: Wenxing Hou<EMAIL_ADDRESS>

Please update all checks to follow the global rule:

The first step is to dump all information, if possible. No check is required before dump.
Then do the check before access the data.


SCIM2 Group's member listing via /Users endpoint returns a full list of members causing performance issues.

Description
SCIM2 Groups endpoint doesn't support member pagination while listing the groups. Therefore, we use the /Users endpoint as below.
GET https://localhost:9443/scim2/Users?filter=groups+eq+admin&startIndex=0&limit=2
Even this endpoint doesn't honour pagination parameters if the filter contains groups. This again returns a full list of members causing performance issues if the given groups filter contains large amount of users.
Improvement
Introduce a configurable hard limit to throttle the number of members returned in the GET https://localhost:9443/scim2/Users?filter=groups+eq+admin request. (Not pagination, just a hard limit to throttle the number of users)
Additional context
WSO2is-.5.10.0
WUM - 2021 February

This issue is being closed due to extended inactivity. Please feel free to reopen it if further attention is needed. Thank you for helping us keep the issue list relevant and focused!


Remove version: CrystalDewWorld.CrystalDiskInfo.KureiKeiEdition version 8.17.5

[Automated] It returns code over 400 in all urls
Microsoft Reviewers: Open in CodeFlow

URL: https://osdn.net/frs/redir.php?m=ipconnect&f=crystaldiskinfo%2F77593%2FCrystalDiskInfo8_17_5KureiKei.exe
Response Error: The underlying connection was closed: Could not establish trust relationship for the SSL/TLS secure channel.
(Automated message - build 923)


Update Pages “about”

Automatically generated by Netlify CMS

👷 Deploy Preview for frolicking-faloodeh-5fb180 processing.

Name
Link

🔨 Latest commit
9559b9a113a250f24a70be41d3a78c9eab7e3f18

🔍 Latest deploy log
https://app.netlify.com/sites/frolicking-faloodeh-5fb180/deploys/6291dae4016de800081e676a


What are the labels for contrasting losses?

Hello authors, thank you very much for your excellent work.
I am trying to train Grounding Dino, but have some questions about the contrast loss mentioned in section 3.5 of the text.
My understanding is that "predicted_objects" is the output of the decoder, and "language tokens" is the "encoded_text" in text_dict, but I haven't figured out what the ground truth is supposed to be, can you help me with this?

https://github.com/IDEA-Research/GroundingDINO/issues/228


Can not shrink the group

Can not shrink the group when i scroll.

I also encountered this kind of situation，the group that Every time come in is open by default


🐛 [cloud_firestore] Cloud Firestore on web fires two times with wrong data in first snapshot

Bug report
Describe the bug
I have made a plugin to make firestore coming data paginate nicely. It was running fine before, but right now there some issue with cloud firestore on web.
How my package works:

It sets a query with a limit provided by the user.
Then snapshots are being listened to.
If the number of docs read are less than query limit, pagination is ended and all listeners are cancelled.
After the users scrolls to the bottom of list the limit to the query is increased.
Then the previous subscription to snapshots is cancelled.
And repeats from step 2

The package is working fine on android but on the web when query limit is increased, the snapshot listener fires two times with

docs with size of previous limit
docs with size of new limit

As on web it fires two times, due to the first snapshot with old size limit, pagination is ended due to step 3 of how package works.
Steps to reproduce
Steps to reproduce the behavior:

Go to package repo
Clone the project
Create a firebase demo project and set firestore rules to all true
Add firebase project credentials in the main function inside example\lib\main.dart
Run the app in browser and click button Firestore Pagination Example
Add around 25 messages and then go back
Again open the Firestore Pagination Example and scrolling up
You'll see only the last 12 messages are loaded (as default limit setted is 6)

Do the same on android and you'll see all messages are loaded.
Expected behavior
All the messages should load in the web app just like it does in the android app.
Meaning the query should not fire twice with 1 snapshot having the wrong data.
Sample project
https://github.com/OutdatedGuy/firebase_pagination
Firestore Example App Code: example\lib\firestore_pagination_example.dart
Firestore Pagination Plugin Code: lib\src\firestore_pagination.dart

Additional context
Example App Demo Video

Android
Web

This is how the example app is supposed to work on web:

This is with Realtime Database of firebase and not the Firestore

https://user-images.githubusercontent.com/74326345/192826480-2f895ac9-5adc-40e4-8937-51d543c2dd8b.mp4

Flutter doctor
Run flutter doctor and paste the output below:
Click To Expand
Doctor summary (to see all details, run flutter doctor -v):
[√] Flutter (Channel stable, 3.3.2, on Microsoft Windows [Version 10.0.22622.601], locale en-IN)
Checking Android licenses is taking an unexpectedly long time...[√] Android toolchain - develop for Android devices (Android SDK version 32.1.0-rc1)
[√] Chrome - develop for the web
[√] Visual Studio - develop for Windows (Visual Studio Community 2022 17.2.3)
[√] Android Studio (version 2021.3)
[√] VS Code (version 1.71.2)
[√] Connected device (3 available)
[√] HTTP Host Availability

• No issues found!

Flutter dependencies
Run flutter pub deps -- --style=compact and paste the output below:
Click To Expand
Dart SDK 2.18.1
Flutter SDK 3.3.2
firebase_pagination 1.1.3

dependencies:
- cloud_firestore 3.4.7 [cloud_firestore_platform_interface cloud_firestore_web collection firebase_core firebase_core_platform_interface flutter meta]
- firebase_database 9.1.4 [firebase_core firebase_core_platform_interface firebase_database_platform_interface firebase_database_web flutter]
- flutter 0.0.0 [characters collection material_color_utilities meta vector_math sky_engine]

dev dependencies:
- flutter_test 0.0.0 [flutter test_api path fake_async clock stack_trace vector_math async boolean_selector characters collection matcher material_color_utilities meta source_span stream_channel string_scanner term_glyph]
- very_good_analysis 3.0.2

transitive dependencies:
- async 2.9.0 [collection meta]
- boolean_selector 2.1.0 [source_span string_scanner]
- characters 1.2.1
- clock 1.1.1
- cloud_firestore_platform_interface 5.7.4 [collection firebase_core flutter meta plugin_platform_interface]
- cloud_firestore_web 2.8.7 [cloud_firestore_platform_interface collection firebase_core firebase_core_web flutter flutter_web_plugins js]
- collection 1.16.0
- fake_async 1.3.1 [clock collection]
- firebase_core 1.22.0 [firebase_core_platform_interface firebase_core_web flutter meta]
- firebase_core_platform_interface 4.5.1 [collection flutter flutter_test meta plugin_platform_interface]
- firebase_core_web 1.7.2 [firebase_core_platform_interface flutter flutter_web_plugins js meta]
- firebase_database_platform_interface 0.2.2+4 [collection firebase_core flutter meta plugin_platform_interface]
- firebase_database_web 0.2.1+6 [firebase_core firebase_core_web firebase_database_platform_interface flutter flutter_web_plugins js]
- flutter_web_plugins 0.0.0 [flutter js characters collection material_color_utilities meta vector_math]
- js 0.6.4
- matcher 0.12.12 [stack_trace]
- material_color_utilities 0.1.5
- meta 1.8.0
- path 1.8.2
- plugin_platform_interface 2.1.2 [meta]
- sky_engine 0.0.99
- source_span 1.9.0 [collection path term_glyph]
- stack_trace 1.10.0 [path]
- stream_channel 2.1.0 [async]
- string_scanner 1.1.1 [source_span]
- term_glyph 1.2.1
- test_api 0.4.12 [async boolean_selector collection meta source_span stack_trace stream_channel string_scanner term_glyph matcher]
- vector_math 2.1.2

Thanks for the detailed report and code sample. I cloned the project, added sample firebase project to it, ran it on web and added test data and observed that not all data is retrieved:
https://user-images.githubusercontent.com/67046386/192976870-4969e7ef-99bd-4c00-8d31-5863b244a824.mov
In backend, I see the data I added is created, for example:

8. You'll see only the last 12 messages are loaded (as default limit setted is 6)

Can you tell us where and how you have set it rule ?

I would also suggest to try out a community plugin https://pub.dev/packages/paginate_firestore and see if using it, you get same behavior or not.
Please note that, this isn't an endorsement of the plugin, but just to check if the same behavior occurs with external library as well, or is it just specific to cloud_firestore.

Can you tell us where and how you have set it rule ?

For example app, the limit is setted here: example\lib\firestore_pagination_example.dart:36:15 (go to this location in the repo)
And as for the plugin itself, its here: lib\src\firestore_pagination.dart:183:3 (the function to add more data and set query)

Thanks for the update. Based on my verification https://github.com/firebase/flutterfire/issues/9630#issuecomment-1261926374, and observing same behavior as reported, keeping this issue open and labeling.

Yo @darshankawar, any updates here?

Hey @OutdatedGuy, could you please provide a small repro of the query you are using that differs between web and android? Happy to look into this issue without the third party dependency, firebase pagination. Thanks.

Yo @russellwheatley, the issue is so old I forgot where the problem exactly occurs. I seem to have fixed my personal issue with some hack.
Also,

Happy to look into this issue without the third party dependency, firebase pagination

firebase_pagination is my own package which is having this problem, not a 3rd Party Package.
But anyways, as I have found a hack to fix my issue, you can go ahead and close this issue.

Thanks for the update, @OutdatedGuy 🙏


Expose Request Method in requestConfig for Middleware

Description:
I would like to propose an enhancement to the fal.config method where a requestMiddleware function can be further configured. Currently, the requestConfig object only exposes the headers and URL information. However, it would be beneficial if the requestConfig also exposed the HTTP request method.
Use Case:
In our use case, we need to supply an API auth token to the FAL proxy server through the middleware. Here’s a basic example of how we configure the client:
fal.config({
  proxyUrl: "http://localhost:8080/api/fal/proxy",
  requestMiddleware: (requestConfig) => {
    return Promise.resolve({
      ...requestConfig,
      headers: {
        ...requestConfig.headers,
        "x-my-bearer": `Bearer ${bearer}`,
        "x-my-content-id": contentId,
      },
    });
  },
});

This middleware adds authentication tokens to the request headers. However, to enhance security, it would be ideal to omit sending tokens in GET requests, lowering exposure.
Proposed Enhancement:
Consider exposing the request method in the requestConfig object passed to the requestMiddleware. This additional information would allow developers to conditionally include or exclude sensitive headers based on the request method, reducing the risk of exposing secrets in scenarios where they are not needed.
Example Usage:
fal.config({
  proxyUrl: "http://localhost:8080/api/fal/proxy",
  requestMiddleware: (requestConfig) => {
    if (requestConfig.method === 'POST') {
      return Promise.resolve({
        ...requestConfig,
        headers: {
          ...requestConfig.headers,
          "x-my-bearer": `Bearer ${bearer}`,
          "x-my-content-id": contentId,
        },
      });
    } else {
      return Promise.resolve(requestConfig);
    }
  },
});

Hey @fco-fbatch, thanks for the thoughtful feature request. I opened a PR for this: #87


feat(besu): block validator endpoint

Resolve #955

@AzaharaC Are you still working on this?

@AzaharaC No worries, the Corda 5.x work that you are doing is higher priority than this so we can leave this until after that too.

@AzaharaC Closing this down for now just to tidy up the PR queue a bit. We can re-open it anytime later.


CA injection via cert-manager in ClusterSecretStore do not work

Describe the bug
I tried to setup ClusterSecretStore after deployment of the self signed certificate chain and the helm installation. I tried to inject the ca instead of using the caBundle field statically.
apiVersion: external-secrets.io/v1beta1
kind: ClusterSecretStore
metadata:
  name: bitwarden-secrets-manager
  annotations:
    cert-manager.io/inject-ca-from: external-secrets/bitwarden-tls-certs

But it tells me caBundle is a required field.
Full message from reconciliation:
ClusterSecretStore/external-secrets/bitwarden-secrets-manager dry-run failed (Invalid): ClusterSecretStore.external-secrets.io "bitwarden-secrets-manager" is invalid: spec.provider.bitwardensecretsmanager.caBundle: Required value

To Reproduce
Steps to reproduce the behavior:
My deployment is made with flux. My yaml code can be seen here:
https://github.com/volschin/home-ops/blob/main/kubernetes/apps/external-secrets/external-secrets/stores/bitwarden-secrets/clustersecretstore.yaml
Expected behavior
It should be possible to inject the ca from the generated certificate in the ClusterSecretStore.
** Additional **
In my connected research I found this linkerd issue an pr from some years ago that goes in a same direction an perhaps helps.
https://github.com/linkerd/linkerd2/issues/7353

Oh interesting. I don't know of that approach. :) I'll take a look on how to implement that.

Hmmm. Not entirely sure how that would work since external secrets isn't necessarily aware of the service. It's handling it as a generic something that has an api.
If it would start checking any certificates it would impact the setup of the whole project too.
It's not impossible I just think it's not necessarily worth the effort. 🤣 but maybe I'm wrong. 😀

Hmmm. Then again we are talking about secrets here that potentially could destroy a system. So scratch what I said. Setting up mTLS would be nice.

Btw I don't think this ca injector annotation would work here since the caBundle field isn't in the spec. It's under spec provider bitwardenprovider auth. The thing we configure is neither a webhook nor an api service. So I'm not sure how this would work.

That said... eso does have it's own injector. I wonder if I could implement something there. I will have to check that out.

Hm, this is most likely not going to happen as it requires a bunch of rewrites to our injector and the recommended approach is to either point to a secret or use a bundled certificate. I will add the secret approach as well.

Meaning it will get something like this:
// Used to provide custom certificate authority (CA) certificates
// for a secret store. The CAProvider points to a Secret or ConfigMap resource
// that contains a PEM-encoded certificate.
type CAProvider struct {
	// The type of provider to use such as "Secret", or "ConfigMap".
	// +kubebuilder:validation:Enum="Secret";"ConfigMap"
	Type CAProviderType `json:"type"`

	// The name of the object located at the provider type.
	Name string `json:"name"`

	// The key where the CA certificate can be found in the Secret or ConfigMap.
	// +kubebuilder:validation:Optional
	Key string `json:"key,omitempty"`

	// The namespace the Provider type is in.
	// Can only be defined when used in a ClusterSecretStore.
	// +optional
	Namespace *string `json:"namespace,omitempty"`
}

So the source can be a secret or a configmap too.

https://github.com/external-secrets/external-secrets/issues/3676

https://github.com/external-secrets/external-secrets/pull/3699

I've just hit into this one as well as the caProvider option (which is in the docs) is not in the CRD https://github.com/external-secrets/external-secrets/blob/4f62fb396397bc4973f54dce1744e8de39900a9c/apis/externalsecrets/v1beta1/secretsstore_bitwarden_types.go#L20 also lacking the ref in the upstream api docs as well https://external-secrets.io/latest/api/spec/#external-secrets.io/v1beta1.CAProvider
I have resorted to (for now, for testing) copying the CA bundle from the CertManager output secret and adding in into the caBundle string as a bypass, though this is not ideal as any changes upstream (CertManager) could result in a new CA being required and then its invalidated.
I can see you have an MR thats open which looks at a glace to be addressing this, however :)

This will have to be released first in ESO to take effect. So keep an eye out for 0.10.1 for ESO.


test: cloudflare key value store for persisted documents in Hive Cloud

Background
R2 is too slow for the initial (uncached) request.
Description
Let's see if Cloudflare KV is a good alternative for cloud users.
Checklist

[ ] Input validation
[ ] Output encoding
[ ] Authentication management
[ ] Session management
[ ] Access control
[ ] Cryptographic practices
[ ] Error handling and logging
[ ] Data protection
[ ] Communication security
[ ] System configuration
[ ] Database security
[ ] File management
[ ] Memory management
[ ] Testing

closing as r2 is good enough


merging still frames to a video

I have a bunch of bmp files that I'm trying to convert into a 30 fps video, one image per frame. This is what I'm trying...
ffmpeg('000000.bmp')
    .input('000001.bmp')
    .input('000002.bmp')
    .on('error', function(err) {
        console.log('An error occurred: ' + err.message);
    })
    .on('end', function() {
        console.log('Merging finished!');
    })
    .fps(30)
    .videoCodec('libx264')
    .mergeToFile('out.mp4', 'tmp');

This completes without error, but I can't play the output file in any players.
I'm on Windows 10, with the latest ffmpeg in my path variable. Thanks for any tips!

Also... is this creating a command line to ffmpeg.exe in the end? If so, is it going to fail when I have thousands of frames to process just because the command gets too long?

Yes, this is the goal of fluent-ffmpeg. It will probably fail given Windows has stupildy low limitations on command line lengths. Use the %d notation in this case (see ffmpeg docs).
Also you don't want to use mergeToFile(). This is used to concatenate videos.

Thanks for the reply... this seemed to work for something that I could play in VLC.
ffmpeg('%06d.bmp')
    .save('out.mp4'));


⚠ SceneTime has degraded performance

In 286aa4c, SceneTime ($ST) experienced degraded performance:

HTTP code: 200
Response time: 10928 ms

Resolved: SceneTime performance has improved in 959a9a9.


Adding git pre commit hook for formatting staged changes

This hook will run on every commit to run prettier on the staged changes only (not whole files). The prettier configuration file used is https://github.com/Azure/azure-sdk-for-js/blob/master/sdk/.prettierrc.json. More details about the hook can be found here.
To install the hook, run rush update.

@mikeharder Thanks for your feedback. However, I am trying to avoid spamming all package.json files with the same script that is not intended to be executed manually anyway.

@mikeharder Thanks for your feedback. However, I am trying to avoid spamming all package.json files with the same script that is not intended to be executed manually anyway.

I don't think adding something to every package.json file is a bad thing.  Our package.json files already have many of the same dev dependencies and script commands, and rush makes it easy to keep the versions aligned across packages and updated regularly.
Other than needing to change every package.json file, what are the pros and cons of enabling this tool via package.json as opposed to the rush pre-commit hooks?  Would it be valuable to give devs the ability to run this tool manually via npm run precise-commits, say if they wanted to review the prettier changes before committing?
We should use the option we think gives the best overall developer experience, ignoring whether we need to modify every package.json or not.

If we decide there is no value in adding this command to every package.json, then I agree this PR aligns with the guidance at https://rushjs.io/pages/maintainer/git_hooks/.

what are the pros and cons of enabling this tool via package.json as opposed to the rush pre-commit hooks?

@mikeharder Using rush to manage the hook ensures that the hook will always be installed in the right place .git/hooks/pre-commit once we run rush update, a common operation in our workflow. On the other side, to have the same behavior without rush's help, we will need an extra package (husky) to install the hook using npm. Otherwise, expressiveness of the hook scripts are the same for both approaches.

Would it be valuable to give devs the ability to run this tool manually via npm run precise-commits, say if they wanted to review the prettier changes before committing?

I do not think so. In fact, the hook will not commit the formatting changes and will leave it up to you to decide if you want to commit them. With this workflow, I can not think of a scenario where you would want to run the formatter manually. Ideally, prettier is already running in your editor, so this is more of a soft alarm against accidental bad formatting. Eventually, I would like to add to our CI a formatting check similar to the one described here.

We should use the option we think gives the best overall developer experience, ignoring whether we need to modify every package.json or not.

Duplication generally just does not sit right with me but this is a good and important point.

After today's offline discussion, we will wait to get acknowledgement from our partners. @ramya-rao-a would you please ask them to put a review/acknowledgement?
For the sake of completeness, I will outline below the assumptions I made and how this hook affects our workflow.
Assumptions:
1- It is very likely that prettier already runs in your editor using the configuration files located in the sdk directory.
2- You do not mind waiting anywhere between well less than a second and a few seconds when you issue a git commit to run the hook script first before your commit is actually performed. This waiting time depends for the most part on how many files have been changed. So for the typical case of a few files, the waiting time is less than a second.
3- You could be interested in looking at the formatting changes before actually commit them.
4- You want to move fast so you do not want to get your commit aborted for any reason.
5- It does not matter how many commits you might need to make because they will always get squashed at the time of merging with master.
Description:
This is a pre commit git hook that runs when you issue a git commit. The script calls precise-commits which in turn calls prettier on the changed parts of the changed files and will write the result to disk, if any, without staging or committing them. Consequently, the commit operation will go through and all your original staged changed will only be part of the commit. After the commit, if there are formatting changes, they will remain unstaged so you can inspect, stage, and commit them.
Of course, not everyone will like the assumptions I made and another hook behavior might suit them better. So to get an idea about the consensus, please upvote this comment if this works for you. If not, please post another comment with your assumptions and a description of the behavior you like more.

cc @southpolesteve, @zfoster, @XiaoningLiu

That's a great step! Please make sure win/linux/mac can work with this. Looks good.

I verified that it works on linux (including wsl) and in git bash on windows.

I just noticed that the precise-commits repo is stall for two years now. I reached out to the repo owners asking them what is their plan for this project. In the mean time, I propose to keep using it as long as it works which is compatible with assumptions 1 and 6 here.

prettier has been ran across our packages (see #10269, #10273, #10272, #10271, #10270, #10268, #10267, #10266, #10265, #10264, #10263, #10262, #10261), so this hook should work on new changes only, without introducing any noise. Furthermore, merging this PR clears the path to merge #10145. Now that everyone is onboard with this, I will go ahead and merge it.


Exclude debian dir

Summary of changes
In a Debian package build, there is going to be a top-level debian directory. Exclude this from namespace_package searches by default.
Pull Request Checklist

[ ] Changes have tests
[x] News fragment added in newsfragments/.
(See documentation for details)

It's hard to imagine that these 3.12 test failures are related to the branch's changes

It's hard to imagine that these 3.12 test failures are related to the branch's changes

Yep, reproduced on main, locally.

I will assume that everyone maintaining python-debian is happy to specify packages = manually and opt-out on auto-discover (just documenting it here, following up the conversation on Discord). Please let me know if this assumption is not correct.

Yeah, that's understood, and not an issue at all.

(I think) I fixed the errors on main and rebase the PR.


Releases page redirects to 404

https://asciidocsy.netlify.app/releases redirects to 404 page

I don't think this URL (/releases) is supposed to be linked to from anywhere. How did you get that URL?
The history page is at https://asciidocsy.netlify.app/docs/theme/history, But I know you already saw that.
I'm going to try to get some more stuff squared away today. Definitely tell me if you find any other bugs Sean.

Top right corner of https://asciidocsy.netlify.app/, next to About is Releases

Gah!  Will fix.


Rebase and correct styles on #3763

:pushpin: Commit b61bd78 has been approved by indirect

@homu r+

:hourglass: Testing commit b61bd78 with merge eb462b6...

:broken_heart: Test failed - status

@indirect failures look legit?

They do. I'll try to get to the bottom of it this weekend (unless someone else fancies it). Any clues on what could be the significant difference in older versions of RubyGems?

Closing in favor of #3763.


Simulation output path handling

run_simulation_study.py currently writes a csv with the call .write_csv("out/sim_study.csv").
We should (1) check if this directory exists before trying to write to it and (2) create the path in a system-agnostic way (presumably with os.path.join()?).

the pretty way is:
from pathlib import Path
out_path = Path("/path/to/whatever") # you need to discover this in some way
whatever.write_csv(out_path / "sim_study.csv")


Google is Evil

ReCaptcha is proof.

"Don't do evil" okay so remove all traces of Recaptcha from the net? Awful lot of evil gone all at once real fast.

The fact that this issue has been open for 8 months suggests that either:

nobody at Google care about community feedback enough to triage new issues; or,
somebody at Google agrees with you

The fact that this issue has been open for 8 months suggests that either:

nobody at Google care about community feedback enough to triage new issues; or,
somebody at Google agrees with you

I can only agree


Optional argument mostRecent for 2D Checkers

Boolean argument (default false) to select the most recent verb of each component axis instead of their sum. Useful for individuated directional keys (eg. WASD, arrows keys, JoyCon D-Buttons) as well as alternate binds where directional input may benefit from taking the last-pressed value while multiple sources are held.
When true, holding left, then depressing and releasing right will give "left right left" instead of "left none left".

👍

Done!

thanks ! stuff seems broken on 2022.500.0.88, feel free to ping here if you update to a later beta otherwise i'll rollback and author a fix in a little while

Looks like we're on the same beta now but still broken/needs review

#334


Create client/runner that supports auto-updating the vault client

The vault client on spacewalk would have to manually update the new releases that we would be rolling out. In order to avoid this and save vault client effort, we want to implement an auto-updater mechanism that automatically updates the vault client for new releases.
TODO
We will try to adopt this implementation.
Since we don't have a client runner for the 'oracle' or 'faucet, the runner will only have to support the ClientType vault for now.
This ticket is related to https://github.com/pendulum-chain/tasks/issues/46.

Hey team! Please add your planning poker estimate with Zenhub @adelarja @b-yap @ebma @gianfra-t @TorstenStueber


type for persons

persons working in the project schould have special type (create view)

they are now personel


Fix link for "CSS style management"

Found a 404 on https://templ.guide/syntax-and-usage/attributes/ so here's the fix 🙌

Thanks!


Document release process

Create documentation for the release process of Taipy and it's subpackages.

Duplicate of taipy-enterprise-#363.


Doc preview url in PRs not working for new toc entries

Bug Report
Per versioning design, in a PR when I want to preview doc changes/addition:
I select the preview link, and change the url from /docs/v0.11/ to /docs/edge/
This works ok, but if I select a new topic from the toc, the link defaults to docs/v0.11/edge/
related to: #3906
Expected Behavior

Steps to Reproduce the Bug

Screenshots or additional information and context

Environment Details

Build version (tanzu version):
Deployment (Managed/Unmanaged cluster):
Infrastructure Provider (Docker/AWS/Azure/vSphere):
Operating System (client):

Diagnostics and log bundle

@kcoriordan @jonasrosland we discussed in retro today that we'll assign this to ya'll for v0.12.0.
If it turns out something core eng should work on, just let us know and we'll reassign.
Appreciate it!

I'll spend some time on this today and tomorrow.


Rename finishedPlantingTime to plantingCompletedTime

We decided to use the terms "completed planting" and "planting completed" instead
of "finished planting" in the product. Rename the representation of this concept
in the database and the API to match the terminology in the product.

Current dependencies on/for this PR:

main

PR #1203

PR #1205

PR #1206   👈

This comment was auto-generated by Graphite.

@sgrimm started a stack merge that includes this pull request via Graphite.

Graphite rebased this pull request as part of a merge.

@sgrimm merged this pull request with Graphite.


Support encoding non-RGBA pixel formats to WebP

Whilst using TIPXWebPEncoder, I noticed that some UIImages will have their red and blue channels swapped in the resulting WebP image data. This is due to the underlying pixel data being in BGRA format as opposed to RGBA.
To fix this, I've added in some code to convert images to standard RGBA format using Accelerate.framework's vImage functions before importing it into the WebP.

OK, change looks good.  We'll have to go through hardening time before merging, but full credit will be attributed once this lands.  If anything comes up, I'll return to this PR for follow up.

change has been merged via v2.4.4 release


fix parsing problems, allow invocation and support name resolution of trait constraints

All the following samples work now (for type checking and codegen - execution not yet tested)
let legacyConcat1 (x: string) (y: string) = x ^ y
let legacyConcat2 (x: string) (y: string) = x ^y
let legacyConcat3 (x: string) (y: string) = x^ y
let legacyConcat4 (x: string) (y: string) = x^y

type IAdditionOperator<'T> =
    static abstract op_Addition: 'T * 'T -> 'T

type C(c: int) =
    member _.Value = c
    interface IAdditionOperator<C> with
        static member op_Addition(x, y) = C(x.Value + y.Value)

let f<'T when 'T :> IAdditionOperator<'T>>(x: 'T, y: 'T) =
    'T.op_Addition(x, y)

let f2<'T when 'T :> IAdditionOperator<'T>>(x: 'T, y: 'T) =
    'T.(+)(x, y)

if f<C>(C(3), C(4)).Value <> 7 then
    failwith "incorrect value"

let inline f3<^T when ^T :> IAdditionOperator<^T>>(x: ^T, y: ^T) =
    ^T.op_Addition(x,y)

let inline f4<^T when ^T : (static member (+): ^T * ^T -> ^T)>(x: ^T, y: ^T) =
    ^T.op_Addition(x,y)

let inline f5<^T when ^T : (static member (+): ^T * ^T -> ^T)>(x: ^T, y: ^T) =
    ^T.(+)(x,y)

let inline f6<^T when ^T : (static member (+): ^T * ^T -> ^T)>(x: ^T, y: ^T) =
    x + y

This one doesn't (using IWSAMs to satisfy SRTP) and is what I'll work on next.
let inline f7<'T when 'T :> IAdditionOperator<'T>>(x: 'T, y: 'T) =
    x + y

Includes a bug fix for SRTP generics when rigid type parameters reduce multi-support to single-support SRTP, e.g.
let inline f6<^T when ^T : (static member (+): ^T * ^T -> ^T)>(x: ^T, y: ^T) =
    x + y

Properties need testing
Adhoc tests are in tests/adhoc.fsx for now, we can move them to a better place

@vzarytovskii I'll actually push straight to your branch to get Proto build testing.  I can take you through some of the changes


Document string DSL functions

Linked issue(s)
KOL-7582
What change does this PR introduce and why?
Document string DSL functions (and examples):

lower
upper
ltrim
replace
substr

Corresponding impl PR: https://github.com/kolenaIO/shipyard/pull/2507
Please check if the PR fulfills these requirements

[x] Include reference to internal ticket and/or GitHub issue "Fixes #NNNN" (if applicable)
[ ] Relevant tests for the changes have been added
[x] Relevant docs have been added / updated

Can't say it's terribly pretty with the optional parameters:

cc @mkaramlou thoughts?


Add note about trailing slash in report path

Is your feature request related to a problem? Please describe:

The only significant issue I had when setting up the plugin was that the plugin defaulted to a path of /data/report while the gateway itself defaulted to a path of /data/report/, and so no data went through.
Describe the solution you'd like:

It might be helpful to add a note in the README that the presence/absence of a trailing slash is significant.
Describe alternatives you've considered:

Alternately, modify the data listening server so that the trailing slash isn't significant.
Additional context:

Thanks for making the plugin.

Thanks @ilg that's a good suggestion to note in the README


Person Risk Profile for "Person whose uCOVID budget is X%"

At the moment, it's unclear how to calculate microcovids for interacting with someone who's following a given microcovid budget. Such a person doesn't neatly fit into any of the pre-baked "Person Risk" profiles.

+1


How to run this project if I have downloaded your pre-processed data?

python train.py -task ext -mode train -bert_data_path BERT_DATA_PATH -ext_dropout 0.1 -model_path MODEL_PATH -lr 2e-3 -visible_gpus 0,1,2 -report_every 50 -save_checkpoint_steps 1000 -batch_size 3000 -train_steps 50000 -accum_count 2 -log_file ../logs/ext_bert_cnndm -use_interval true -warmup_steps 10000 -max_pos 512
Traceback (most recent call last):
File "train.py", line 10, in
from train_abstractive import validate_abs, train_abs, baseline, test_abs, test_text_abs
File "/home/aditya64/text summerizer/PreSumm-master/src/train_abstractive.py", line 15, in
from pytorch_transformers import BertTokenizer
ModuleNotFoundError: No module named 'pytorch_transformers'

pip install pytorch_transformers
Check the dependencies in the README and install it.


Use geerlingguy/ubuntu1404 box for VMware Fusion

I have added VMware Fusion support in #35, but I'm currently using the rather beefy (non-minimal) base box chef/ubuntu-14.04 instead of geerlingguy/ubuntu1404, since I don't yet have the base box Packerized for Fusion.
I'll have to do that as part of the https://github.com/geerlingguy/packer-ubuntu-1404 project...
Until that's done, NFS mounts won't work out of the box with chef (mounting a volume that way times out after a few minutes).

I finally finished building a box for VMWare, and I'm testing it locally now. I'll just need to update the project's README once this is done, and then that's it!

I'm testing everything now... here are some preliminary benchmarks with the exact same configuration on VMware vs VirtualBox:

Hypervisor
Req/s

VMware
17.19

VirtualBox
TODO

VMware
 01:31 PM:~ $ wrk -t4 -c24 -d60 http://drupaltest.dev/
Running 1m test @ http://drupaltest.dev/
  4 threads and 24 connections
  Thread Stats   Avg      Stdev     Max   +/- Stdev
    Latency     1.40s   179.83ms   2.28s    82.50%
    Req/Sec     3.91      0.40     5.00     87.50%
  1034 requests in 1.00m, 9.72MB read
Requests/sec:     17.19
Transfer/sec:    165.58KB

VirtualBox
TODO

...and apparently installing VMware tools makes a substantial performance difference :)


vpx-corvette - Corvette (Midway 1994)

@lilalien Cheers. Both Corvette and Car Hop are now updated. 👍


E2E integration test with Jest: no way to manually stop http server

Is there an existing issue for this?

[X] I have searched the existing issues

Current behavior
I was dealing with this for a few days and I think there is an issue with this. I was following your guid for testing (E2E testing with supertest) and I notice that the example provided initialise the underline server with listen but await appInstance.getHttpServer() return an object on which is not possible to call .stop() to stop web server. This means that basically the test leave the server running after test is completed and jest won't terminate.
Here is my code:
import "reflect-metadata";

import { AppFactory, ServerEnvironment, StatusCode } from "@expressots/core";
import { container } from "../../../src/app.container";
import { App } from "../../../src/providers/app/app.provider";
import express from "express";
import request from "supertest";

describe("health check api", () => {
    let server:express.Application;

    beforeAll(async () => {
        await startHttpServerOnRandomPort();
    });

    afterAll(async () => {
        //await server.close() we cannot close the server due to framework limitation?
    });

    async function startHttpServerOnRandomPort() {
        const appInstance = await AppFactory.create(container, App);
        await appInstance.listen(0, ServerEnvironment.Development); //we create a server every time on random port, it could be changed when we can manually stop the server (see afterAll)
        server = await appInstance.getHttpServer();
    }

    describe("Should call health check endpoint and return 200 OK", () => {
        test("GET /health", async () => {

            const expectedResponseBody = {
                health: "OK",
            };

            const response = await request(server).get("/health");

            expect(response.status).toBe(StatusCode.OK);
            expect(response.body).toEqual(expectedResponseBody);
        });
    });
});

Any idea what's wrong here?
Steps to reproduce
No response
Expected behavior
Expected: a way to manually stop()/cleanup server in afterAll method:
    afterAll(async () => {         await server.close()      });
Package version
1.2.0
Node.js version
No response
In which operating systems have you tested?

[X] macOS
[ ] Windows
[X] Linux

Other
Test Suites: 1 passed, 1 total
Tests:       1 passed, 1 total
Snapshots:   0 total
Time:        0.8 s, estimated 1 s
Ran all test suites.
Jest did not exit one second after the test run has completed.
'This usually means that there are asynchronous operations that weren't stopped in your tests. Consider running Jest with --detectOpenHandles to troubleshoot this issue.

ah, I am very glad to hear that!
I tried the other route you suggested already but it was looking like a workaround and there were other issues with that.
One thing also to consider in my opinion is the ability to start the server on a random available port, usually if you pass '0' as port the server starts on a random available port that is very handy for e2e testing. I am not sure if nodejs allow this or not but I think this would be a nice addition to what you already have there.
Looking forward to try 3.0.0 and nice job for the framework, it is very cool imo!

ah, I am very glad to hear that!
I tried the other route you suggested already but it was looking like a workaround and there were other issues with that.
One thing also to consider in my opinion is the ability to start the server on a random available port, usually if you pass '0' as port the server starts on a random available port that is very handy for e2e testing. I am not sure if nodejs allow this or not but I think this would be a nice addition to what you already have there. In my previous example there I wanted to do that but in reality what's happening there is that if I pass 0 the server starts on port 3000 as default...
Looking forward to try 3.0.0 and nice job for the framework, it is very cool imo!

Thank you for your kind words! Lots of inspirations from SpringBoot, .Net and some others.
This is a good idea and easy to implement. I might implement this to incorporate in the v3.0.0.
Currently, we offer the method close() from the adapter-express directly, as more requests are coming from devs to have access to the http server underneath, what I might do is:
interface IHttpServer {
   close(enableLog?: boolean): Promise<void>;
   get Port(): number;
}

app.listen(): Promise<IHttpServer> I might return the server on app.listen() method, so I can create an interface to tailor what I want to allow the dev to have access to, to avoid breaking the application flow, and any new request I can just add to that interface.
It will looks like this:
app.listen("your port");
app.listen(); // assign random port
app.listen(0); // assign random port

const server = app.listen(0);

server.close();
server.Port

Yes but app.listen atm is also starting the server on the specific port.
Ideally I'd like to do this:
 const appInstance = await AppFactory.create(container, App);
 await appInstance.listen(0, ServerEnvironment.Development); // pass 0 to listen on random port

and later on:
appInstance.stop()
Then if we already have
 appInstance.getHttpServer()
this method should actually return the underline httpServer.
This is was also what I was expecting by appInstance.getHttpServer() (to return the underline webserver object) but in reality is returning something else so also the doc was kind of misleading.
Not sure if that makes sense

Hi @edesdan npm i<EMAIL_ADDRESS>is out, try it!
It resolves the 2 issues referred in this ticket:
1- returning the underline http server which you can call close() method
import request from "supertest";

import { AppFactory, StatusCode } from "@expressots/core";
import { IWebServerBuilder } from "@expressots/shared";

import { Server } from "http";
import { App } from "../src/app";

describe("AppController", () => {
    let server: Server;
    let webServerBuilder: IWebServerBuilder;

    beforeAll(async () => {
        webServerBuilder = await AppFactory.create(App);
        const app = await webServerBuilder.listen(3000);
        server = await app.getHttpServer();
    });

    afterAll(async () => {
        await server.close();
    });

    it("returns a valid AppResponse", async () => {
        return request(server)
            .get("/v1")
            .expect(StatusCode.OK)
            .expect("Hello from ExpressoTS!");
    });
});

2 - Now you can pass 0 in the listen() method and the server will assign any available port in your OS randomly.
AppFactory.create(App).then((app) =>
    app.listen(0, {
        appName: env.App.appName,
        appVersion: env.App.appVersion,
    }),
);

Oh, that's nice, thanks for the quick turn around!
I am having some issues upgrading to 3.x now, is there like a migration guide from 2.x? It looks like there are few breakable/structural changes..

ah nice! I think I managed to upgrade but I am left with the error above? Can you help me? Am I doing something wrong?
I am following the guide but I get this:
src/app.ts:11:15 - error TS2416: Property 'globalConfiguration' in type 'App' is not assignable to the same property in base type 'AppExpress'.
  Type '() => void | Promise<void>' is not assignable to type '() => Promise<void>'.
    Type 'void | Promise<void>' is not assignable to type 'Promise<void>'.
      Type 'void' is not assignable to type 'Promise<void>'.

11     protected globalConfiguration(): void | Promise<void> {

ah nice! I think I managed to upgrade but I am left with the error above. Can you help me? Am I doing something wrong?
I am following the guide but I get this error:
src/app.ts:11:15 - error TS2416: Property 'globalConfiguration' in type 'App' is not assignable to the same property in base type 'AppExpress'.
  Type '() => void | Promise<void>' is not assignable to type '() => Promise<void>'.
    Type 'void | Promise<void>' is not assignable to type 'Promise<void>'.
      Type 'void' is not assignable to type 'Promise<void>'.

11     protected globalConfiguration(): void | Promise<void> {

All life cycle hooks in the app.ts file are now async, please change the globalConfiguraion, configureServices, postServerInitialization and serverShutdown to async and Promise as follow:
import { AppExpress } from "@expressots/adapter-express";
import { AppContainer, Env } from "@expressots/core";
import { AppModule } from<EMAIL_ADDRESS>
export class App extends AppExpress {
    private config: AppContainer = this.configContainer([AppModule]);

    async globalConfiguration(): Promise<void> {}

    async configureServices(): Promise<void> {}

    async postServerInitialization(): Promise<void> {}

    async serverShutdown(): Promise<void> {}

Here is the comprehensive guide to migrate from 2.x to 3.x : https://doc.expresso-ts.com/docs/prologue/upgrade_guide
Version 3.0 is coming out tomorrow if all tests running tonight are green. We will be announcing in our discord and LinkedIn soon.

ok that's very good news. I managed to upgrade but I've got 2 issues now:

listening on random port doesn't seems to work:

 async function startHttpServer() {

        webServerBuilder = await AppFactory.create(App);
        const app = await webServerBuilder.listen(0, {
            appName: env.Application.APP_NAME,
            appVersion: env.Application.APP_VERSION,
        });

        server = await app.getHttpServer();
    }

I get this:

running npm run dev gives me following error on cli:

npm run dev
(node:32061) ExperimentalWarning: CommonJS module /opt/homebrew/lib/node_modules/npm/node_modules/debug/src/node.js is loading ES Module /opt/homebrew/lib/node_modules/npm/node_modules/supports-color/index.js using require().
Support for loading ES Module in require() is an experimental feature and might change at any time
(Use `node --trace-warnings ...` to show where the warning was created)

><EMAIL_ADDRESS>dev
> expressots dev

🐎 Expressots

/bin/sh: tsx: command not found
/Users/daniele.desanctis/repositories/travelport/trip-manager-html-to-pdf/node_modules/@expressots/cli/bin/commands/project.commands.js:137
                reject(new Error(`Command failed with code ${code}`));
                       ^
Error: Command failed with code 127
    at ChildProcess.<anonymous> (/Users/daniele.desanctis/repositories/travelport/trip-manager-html-to-pdf/node_modules/@expressots/cli/bin/commands/project.commands.js:137:24)
    at ChildProcess.emit (node:events:507:28)
    at ChildProcess.emit (node:domain:489:12)
    at maybeClose (node:internal/child_process:1101:16)
    at Process.ChildProcess._handle.onexit (node:internal/child_process:305:5)

Few things for you to confirm:
1 - make sure you have install the cli<EMAIL_ADDRESS>in the npm global
npm i<EMAIL_ADDRESS>-g

2 - Make sure all deps are correct
"dependencies": {
    "@expressots/adapter-express": "3.0.0-beta.4.2",
    "@expressots/core": "3.0.0-beta.4",
    "@expressots/shared": "3.0.0-beta.4"
  },
  "devDependencies": {
    "@expressots/cli": "3.0.0-beta.4",
    "@types/express": "5.0.0",
    "@types/jest": "29.5.14",
    "@types/node": "20.12.7",
    "@typescript-eslint/eslint-plugin": "8.0.0",
    "@typescript-eslint/parser": "8.0.0",
    "eslint": "8.57.0",
    "eslint-config-prettier": "9.0.0",
    "eslint-plugin-prettier": "5.0.0",
    "jest": "29.7.0",
    "prettier": "3.2.5",
    "supertest": "^7.0.0",
    "ts-jest": "29.1.2",
    "tsconfig-paths": "4.2.0",
    "tsx": "4.19.2",
    "typescript": "5.1.3"
  }

3 - make sure expressots.config is correct configured for the new version
import { ExpressoConfig, Pattern } from "@expressots/shared";

const config: ExpressoConfig = {
    entryPoint: "main",
    sourceRoot: "src",
    scaffoldPattern: Pattern.KEBAB_CASE,
    opinionated: true,
};

export default config;

4 - to avoid any conflict with previous installation, as you migrated your project, make sure you:
4.1 - remove node_modules and package-lock.json
rm node_modules
rm package-lock.json

This will remove any cache and clean your npm install locally
5 - now make sure you install everything :
npm i

6 - execute your command npm run dev
Obs: make sure that your "devDependencies": { "@expressots/cli": "3.0.0-beta.4" } is there. This is the most common mistake on migrating.
Other than that, read carefully the migration doc and you should be good to go.

ok so I followed your guide and the issue with server running on random port was solved by adding
    "@expressots/adapter-express": "3.0.0-beta.4.2"
previously I only had 3.0.0-beta.4. The nice thing is that is now working. Thanks!
The issue running  npm run dev is still there. I cannot run the app atm.

I found out that I need this to be installed otherwise I can't run npm run dev:
npm install -g tsx
but I am not sure why as before I didn't need to do that I think. Shouldn't this been installed as dependency somehow?

tsx is used for executing the application. It is listed in the previous message as a dev dependency of your project. Its also part of the migration doc. You don't need to install globally if you don't want to.
I am closing this ticket as the initial issue related to E2E testing was resolved. Any other new problem please open a new ticket.
Thank you @edesdan


Add auto merge

Add auto merge of PR then the label is "Ready to merge" and the dir format has no problem.
I'm quite busy I would appreciate if someone can do this

Don't worry I'm here daily to help. If there's any issue I will write you directly

This issue is not a problem again.


What about always transform no matter "serve" or "build"

Describe the bug
Thanks your work, it is a helpful package
Does this plugin only work under "build"? It maybe useful under development, Or we can add a option, to let user whether always transform the import declaration
Libarary Info(please complete the following information):
Libarary: [name]

Version [e.g. 22]

To Reproduce
Steps to reproduce the behavior:
Paste your code here or provide a minimal reproduce repo link.
Expected behavior
A clear and concise description of what you expected to happen.
** Error message**
Paste the error message here.
Additional context
Add any other context about the problem here.

You can just comment out the code for this plugin.
Another way is using the process.env.NODE_ENV
plugins: [
    react(),
    process.env.NODE_ENV === 'production' ? vitePluginImp() : null
  ],

return {
  code: `${importStr};${viteConfig.command === 'serve' ? code : codeRemoveOriginImport}`,
  map: isSourcemap ? sourcemap : null
}

Let's clear what I mean. In this code, I want to it return codeRemoveOriginImport no matter what the viteConfig.command is. so I can get better experience under local development both serve and build
And the code like this
return {
  code: `${importStr};${(options.always || viteConfig.command === 'serve') ? code : codeRemoveOriginImport}`,
  map: isSourcemap ? sourcemap : null
}

In order to be compatible with previous versions, this ability needs to be enabled by the developer
Now you can open this in V2.4.0
vitePluginImp({
   removeOriginImport: true
}),


initial memberlist.Transport implementation allowing us to use an upgrader for ListenTCP/ListenUDP

amended. also updated it to use the new slog/handlers/discard instead of slog/noop @karasz


iOS: iPad layout issues on v9

See https://forum.newsblur.com/t/newsblur-9-not-working-on-latest-ipad-pro/7759.
I believe we fixed one of these layout issues but it's possible the others remain.

I can't reproduce these with the latest code, in iOS 12 or 13, so were probably fixed by my changes in #1216. I'll leave this issue open for now; let me know if anyone can reproduce them with the 9.1 code (or after that is released). Steps to reproduce would be very helpful.

I've seen it when returning to the iPad app from another app. Try switching to the out-of-app Safari and returning and you should be guaranteed to see it.


nginx window server version. hls cleanup Delefile() fail.

I have set up a new streaming system using nginx hls in window server 2008 R2, also another computer for streaming 24hrX7.
But after may be a day or two
there would be some error on the error log and the video cant see.
4522#4384: hls: clean up deletefile() fail on" dir"( 5: Access is denied)
sometime also with the error msg with : *3635 hls: error creating fragment file (5. access is denied)
the streaming work good at the beginning, but after some days this error occur with no reguler time. i need to restart the server then everything fine..
how  can i solve it to run 24X7 without restart..
the config is
live on;
hls onl
hls path "dir"
hls_fragment 1s;
wait_key on;
allow pubish all;
allow play all;

@waitakzz, what version were you using?
@FTABoy, how were you able to use nginx 1.12.1 ?? Did you compile it yourself for Windows?


Add PageExtender

Project URL
https://github.com/fphilipe/PageExtender.app
Category
Extension
Description
This adds PageExtender to the list.
Checklist

[x] Edit applications.json instead of README.md.
[x] Only one project/change is in this pull request
[x] Screenshots(s) added if any
[x] Has a commit from less than 2 years ago
[x] Has a clear README in English

@fphilipe Thank you! 👍


Fix ACM debug collection

Some labels had not been updated to match current labeling. The Hub cluster had also been missed for log collection. Both of these can make E2E/canary triaging a bit more straightforward.

/cherry-pick release-2.8

/cherry-pick release-2.7

/cherry-pick release-2.6


'loader not exported from pixi.js' compile error with react.js using create-react-app

Expected Behavior
Used
import * as PIXI from "pixi.js";
to import Pixi in a react.js project set up with create react. When using the loader I expect sprite to load and trigger the callback.
Current Behavior
When using the loader (testing using npm start) I get the following compile error
./src/canv.js
Attempted import error: 'loader' is not exported from 'pixi.js' (imported as 'PIXI').
Possible Solution
Steps to Reproduce
Environment

pixi.js version: 5.0.2
React version: 16.8.6
project set up with create-react-app

PIXI.loader is deprecated, Use app.loader or let loader = new PIXI.Loader

Perfect - thanks!


[TACHYON-1118] Add HiBench to deploy module

https://tachyon.atlassian.net/browse/TACHYON-1118

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/Tachyon-Pull-Request-Builder/5857/
Test PASSed.


Getting Error, TAG not found. When site settings don't require tags for question creation.

Describe the bug

Same error will be received irrespective of this settings

To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior
You should be able to post your question without any tags. When settings are configured.
Screenshots
If applicable, add screenshots or video to help explain your problem.
Platform

Device: [e.g. Desktop, Mobile]
OS: [e.g. macOS]
Browser and version: [e.g. Chrome, Safari]
Version: [e.g. v1.2.0]

Oh, then it's a strong design bug. After your text I went back and understood what it is.
Actually, this part of the setting page has three sections. which I totally didn't understand until I read your message.

I always thought based on UI it has two sections that's what UI layout says to the users. something like this

maybe a line break after section would help in understanding. @fenbox

Thank you, and we will try to improve.

I tried to make some changes, maybe this way won't be confusing.?

it's still the same. Notice the repeated UI portion (it's a toggle button & multi select input) its repeated, it makes interpretation to user that there are two sections and that's what defines section from UI perspective.
Does this have 3 or 4 sections? if four then I would suggest changing order,
suggested order as an example

Restrict write
set required tags
recommended tags
reserved tags

so that UI doesn't give false intentions
if its three sections
I think just having a line after each section would be sufficient.

It's four sections.

One issue I could see is for item 2. The name/title says "Set Required Tags" but its description says "set recommend tags as required tags". Title and description are not matching.
I would suggest this order:

Reserved Tags
Recommended Tags
Set Recommended tags as Required (title and description should match for this item)
Answer write


Write section about Blahut-Arimoto sampling

We would like to address the sampling properly in a follow-up paper (for all cases, not just the one-step Blahut-Arimoto) and think that it's better to leave it out for now.


Bower package should not use *.min version

According to the bower spec (https://github.com/bower/bower.json-spec#main), the non-minified files should only be used in Main.
Using the .min version in through bower can cause problems in some build processes (ie, grunt)
Just wanted to help and point that out. Thanks!

:+1:

It should include both.  I got an error when I tried to add a second Uglify task to the gruntfile but it should be trivial for someone who is familiar with Grunt.

https://github.com/HubSpot/offline/pull/148 should have fixed this issue


Red Hat support

I added support for finding distros with Red Hat in the name, and then displaying a logo for it.

Hey there @ImNtReal! Thanks for the contribution and sorry I ended up forgetting that original issue https://github.com/nidnogg/zeitfetch/issues/8 you put up.
From what I skimmed, this looks good to me so far. I'll run these flows and review/merge this in later today.

@nldnogg, No worries, at all. Github isn't in my normal day job, so sometimes it takes me a while to circle back to things.

@ImNtReal I've merged this to master and I'll roll this along an update that lets you toggle the logo, make the whole output minimal (just render the logo + user prompt vertically), or make it logo-only as well.
Should be releasing later today on brew/cargo!


Upload configuration in monitor configuration shows error but uploads the file successfully.

When the user goes to the Monitor configuration page and tries to upload the configuration, the upload succeeds but an error message is shown.
How to reproduce the issue.

Go to Monitor > Configurations page.
Click on upload for any of the services. For ex prometheus.
Upload the file. An http error response is shown in the notification.
On checking the server location the new file is updated.

@skdwriting Needs to be fixed.


Person name parsing errors

I found some parsing errors in about 100 person names—they contain page numbers, university names,  etc. Full list with examples with curesponding URIs you can in this gist.
10 14 pages
16 7 klaus miesenberger pages
3 fraiture france monique noirhomme toulouse universit

Btw, we can't check URIs that contain spaces in LDF app: example.
I think it's the issue either with .hdt file or with LDF server itself. Maybe will check later.


Add postgres restore function

Had to disable random ordering of tests because no amount of cleanup of SQLAlchemy makes TestDatabase::test_get_session work after other tests that use SQLAlchemy!

Pull Request Test Coverage Report for Build<PHONE_NUMBER>8
Details

55 of 56 (98.21%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.8%) to 95.172%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/hdx/database/init.py
20
21
95.24%

Totals

Change from base Build<PHONE_NUMBER>:
0.8%

Covered Lines:
276

Relevant Lines:
290

💛  - Coveralls


Add preferred citation for Reg Stats webpage

Something along the lines of:

The George Washington University Regulatory Studies Center, "CHART OR DATASET TITLE," retrieved ACCESS DATE, go.gwu.edu/regstudies.

Or we could use the direct link to Reg Stats or ask Nate to make a shortcut for Reg Stats (e.g., go.gwu.edu/regstats)

Just to add to this issue and remind ourselves:
We need to revise the intro on the Reg Stats webpage with the suggested citation and link to this Github repo.
I'll draft a revised intro and send it for review.

I suggest we modify the introduction text on the Reg Stats page to make it more focused and accurate. Below is the revised text. Please review and feel free to make any changes. @mfebrizio @haysarah
This page provides accessible charts and supporting data to better understand patterns and trends in federal rulemaking activities across presidential administrations, across agencies, and over time. The policy team at the Regulatory Studies Center updates these charts on a regular basis as new data are available.
The data and charts on this page have been featured in a number of publications. Please include an appropriate citation if you would like to use these files in your publication or presentation.
Suggested citation:

The George Washington University Regulatory Studies Center, "CHART OR DATASET TITLE," retrieved ACCESS DATE, go.gwu.edu/regstudies.

All data and charts are also accessible on Github.
If you would like to see us include other data or have questions about the information presented here, please email<EMAIL_ADDRESS>

Minor changes suggested:

patterns and trends in federal rulemaking activities across presidential administrations, across among agencies, and over time.

The source code for all data and charts are also accessible on GitHub.

Otherwise, this looks excellent! Much clearer.

One small suggestion from me as well (to be a little more concise):

This page provides accessible charts and supporting data to better understand highlight patterns and trends

And quick grammar fix in Mark's suggested edit:

The source code for all data and charts are is also accessible on GitHub.

Otherwise looks great - thanks for drafting that, Zoey!

Thank you both @mfebrizio @haysarah ! I will send the revised language to Nate.

Thanks, Zoey! If it would help on his end, I can update the website text if
he approves it.
Mark Febrizio
Senior Policy Analyst | Regulatory Studies Center
https://regulatorystudies.columbian.gwu.edu/
The George Washington University
@.***
On Mon, May 13, 2024 at 14:52 Zoey X. @.***> wrote:

Thank you both @mfebrizio https://github.com/mfebrizio @haysarah
https://github.com/haysarah ! I will send the revised language to Nate.
—
Reply to this email directly, view it on GitHub
https://github.com/regulatorystudies/Reg-Stats/issues/47#issuecomment-2108574326,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/APDQHOKAP2N7SVXOYBPGTATZCEDYFAVCNFSM6AAAAABFOZAF5KVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCMBYGU3TIMZSGY
.
You are receiving this because you were mentioned.Message ID:
@.***>


Add NIST University, Berhampur

Please add NIST University, Berhampur to the repo. NIST is a Technical Institute which offers courses like BTECH, MTECH, MSC etc

@CyKrome Thank you for your patience while waiting for a review and processing your request. Pull request merged. Please start applying for JetBrains educational subscription at https://www.jetbrains.com/shop/eform/students in an hour, to allow the changes to propagate through JetBrains systems.


content/index-privacy-statement.md

https://github.com/sponsors/community?account=Greadcosmo&sort_by=MOST_USED

Why:
Closes [issue link]

What's being changed:

Check off the following:

[ ] I have reviewed my changes in staging (look for "Automatically generated comment" and click Modified to view your latest changes).
[ ] For content changes, I have completed the self-review checklist.

Writer impact (This section is for GitHub staff members only):

[ ] This pull request impacts the contribution experience

[ ] I have added the 'writer impact' label
[ ] I have added a description and/or a video demo of the changes below (e.g. a "before and after video")

@azirulisnani #12421 https://github.com/sponsors/community?account=Greadcosmo&sort_by=MOST_USED


flex-xs-column doesn't seem to work

Things to check

[X] I have searched the existing issues for this bug
[X] To rule out a caching problem I made sure the bug also happens in an incognito tab

Bug type
Other
Component name
No response
What happened?
I apply flex-xs-column to div and expect that children elemnts will be aligned in column when page loaded on the mobile but it doesn't happen
Expected behavior
Children elemnts of of flexbox should be aligned in column when rendered on mobile screen
Reproduction link
https://try.mudblazor.com/snippet/wuGIkiOXJxdeWzwn
Reproduction steps

Create div
Add some elemnts inside it
Add to the class attribute of this div - d-flex flex-xs-column

Relevant log output
No response
Version (bug)
7.4.0
Version (working)
No response
What browsers are you seeing the problem on?
Edge
On which operating systems are you experiencing the issue?
Windows
Pull Request

[ ] I would like to do a Pull Request

Code of Conduct

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

You should be able to use flex-column. See: https://github.com/MudBlazor/MudBlazor/pull/9622#issuecomment-2286229734

Oh got it. Thanks a lot


Busy-wait spinner disappears too soon

Originally reported by @msorens
Overview
Video
It turns out that if you have a lot of things (tokens, users, teams, etc.) then the busy-wait spinner disappears before the data on the page renders--sometimes a long time before.
I first noticed this in acceptance testing on https://a2-local-inplace-upgrade-acceptance.cd.chef.co/, which had on the order of a hundred tokens. The delay between the spinner disappearing and the tokens displaying was not long, but noticeable, from a half-second to perhaps 2 seconds.
I then set about to reproduce this on my own box, first creating a token generator:
a2TokenFactory() {
  default_token_mode=create
  default_count=5
  token_mode=${1:-$default_token_mode}
  count=${2:-$default_count}
  id_prefix=test-token-1111

  if [[ $token_mode == create ]]; then
    echo "Creating $count tokens..."
    for (( i = 1; i <= $count; i++ ))
    do
      jo -p id="$id_prefix-$i" name="$id_prefix $i" active=true | curl -sSkH "api-token: $TOK" $TARGET_HOST/apis/iam/v2beta/tokens -X POST --data @- > /dev/null
    done
  elif [[ $token_mode == delete ]]; then
    echo "Removing $count tokens..."
    for (( i = 1; i <= $count; i++ ))
    do
      curl -sSkH "api-token: $TOK" $TARGET_HOST/apis/iam/v2beta/tokens/$id_prefix-$i -X DELETE > /dev/null
    done
  else
    echo 'usage: a2TokenFactory ( create | delete ) ( count )'
    echo 'default: a2TokenFactory create 5'
  fi
}

I then created 500 tokens...
a2TokenFactory create 500

and artificially slowed down my network using Chrome dev tools.
The first time I clicked on tokens the gap was just a few seconds. But then clicking away, e.g. to Teams, then back to Tokens, the delay was, well.... watch the video:
So the issue is that we are storing in ngrx state (a) the status that data is available at the same time (indeed, in the same call) that (b) the data itself. The problem is that the page rendering starts to react to both at the same time and, naturally, it is much quicker to react to a single boolean state value than it is to react to a large array of objects that need to be rendered in a table.
So I explored this a bit on StackOverflow: Finer control over angular rendering and was advised of a hook I was looking for that might help... alas, tried it briefly, but no joy!
By the way, all those tokens can be cleaned up with:
a2TokenFactory delete 500

Meta: what a great issue. :100:

A few thoughts here:

Our token APIs for listing tokens are an all-or-nothing affair: they don't support pagination. If there was a way to request as many tokens as you want to render only, the delay would probably be less noticable.

On a related note, at some point, the "return all tokens at once" will hit GRPC's message limit of 4Mb. It could be that this never happens in the real world, but it's a theoretical possibility.

Another speed-related matter is that all token-related read requests (Get and List) are fulfilled by asking the database. authn-service, which offers the API to manage tokens, could cache them in memory and serve the requests from there. I'm not sure how this would impact this issue, though. However, since for API requests (data ingestion), the same database roundtrip happens, it could be worthwhile to look into this. We know that pg can become a bottleneck for giant deployments, I think.

On your first point, @srenatus, yes for pagination! That would completely alleviate this issue.
On your second point--I'm sure you know but for others following along just want to be explicit--back-end speed performance is important, but unrelated. It would not affect this particular issue one way or the other.

We're also seeing the spinner go away too soon (if it appears at all) when attempting to add users on a team. The page flickers while it loads a list of users, modifies and sorts those users. We should have a spinner up while that happens.

Step 1 when for when we start the first sprint on Monday will be to investigate each of the cases above and gather more information about where we need to make the changes.