MatDialog scroll entire dialog

Bug, feature request, or proposal:
feature request
What is the expected behavior?
An ability to scroll the entire dialog and not just the content. Meaning - the dialog should have the full height of its content and scrolling will happen "outside" the dialog. The dialog would expand beyond the boundaries of the view.
What is the current behavior?
Scrolling can only happen inside the dialog. This creates a confusing UX where users don't have an indication that there is more content under scroll.
What are the steps to reproduce?
Providing a StackBlitz reproduction is the best way to share your issue.
StackBlitz starter: https://goo.gl/wwnhMV
What is the use-case or motivation for changing an existing behavior?
Which versions of Angular, Material, OS, TypeScript, browsers are affected?
Is there anything else we should know?

I don't think this really makes sense for the dialog. The dialog is supposed to be a modal popup that blocks interaction with the rest of the page, including scrolling. You might be able to make something for your use case using the CDK overlay which the dialog is based on


changing database password of env and fixing react

here's a modification in env and database connection with some modification in react

I appreciate your work, and glad to have an other contributor


✨[feature Request]: Signup page UI does not matches with the Login page UI

Is there an existing issue for this?

[X] I have searched the existing issues

Feature Description

Signup page UI does not matches with the login page UI
@Anjaliavv51 assign this to me i want to fix that
Use Case
Similar UI makes the user feel connected with the content
Benefits
No response
Add ScreenShots
No response
Priority
High
Record

[X] I have read the Contributing Guidelines
[X] I'm a GSSOC'24EXTD contributor
[X] I have starred the repository

please assign me this @Anjaliavv51!!

Here I am with the blend of designing and technecial skills. this will be a great apportunity to contribute as to this task. being a UI/UX specialist and a Coder. I am contribute in this.

@vaiidehisb then please create an issue of your own.

@amitroy-thedev if u r active please create PR and mention me

@Anjaliavv51 okay


Button "Invisible" loses accent color when using a leadingVisual

Description
Button variant "invisible" loses accent color when using a leadingVisual.
https://github.com/primer/react/assets/980622/46ca0aa4-459e-4e16-8138-946879a46bf2
In Figma the component doesn't lose the accent color but the icon doesn't have the accent color applied when added.

Steps to reproduce
Go to Storybook
Version
Latest
Browser
Safari

Hey,
I can work on this, could you please assign it this to me ?

Was taking a look at this issue, and I'm a bit unsure of what the expected color should be 🤔 I took a look at the most recent version of PRC (pre-release), and I see that the default color for "invisible" without the leading visual has changed from the screenshot. I think this was part of https://github.com/primer/react/pull/4940, so I'm not sure if this is the expectation or not for invisible. If it is, then I think we can close this issue.
@langermank, you might have more context than I do 🙇

Screen capture of "invisible" variant in Storybook

@TylerJDev thanks for tagging me! Yes, the original design for the invisible variant was never intended to retain the accent blue color if visuals were present. Therefore, it was only ever blue if no visuals were present.
We recently made a change to make the invisible button the default text color in all scenarios. We found that in most cases the blue color was being overridden by consumers, and it was difficult to use the variant with an inconsistent text color.
As far as I'm concerned there are no current color bugs with Button 😀


Revert "Buffered/Non-buffered read by configuration entry"

Reverts zjf2012/daos#13
@gczsjdy, your UTs are failed.

Thanks @zjf2012 , this assertion failure is due to I resolved the last minor conflict in browser, but didn't test locally.


Unit test framework for C++ timing of generic operators and activation improvement.

Description
NOTE: This also contains changes from PR 'optimize_basic_omp':
https://github.com/apache/incubator-mxnet/pull/7854
Unit test framework for C++ timing of generic operators.
Activation operator converted to Kernel from MShadow.  Performance improves (see below), especially in backward pass.
OLD
Timing: 50 iterations of 10 calls, shape = (1,1,28,28)
Fully connected:  Timing [Forward] 56.215 ms, avg: 0.11243 ms X 500 passes
Fully connected:  Timing [Backward] 69.322 ms, avg: 0.138644 ms X 500 passes
Timing: 50 iterations of 10 calls, shape = (1,3,28,28)
Fully connected:  Timing [Forward] 24.187 ms, avg: 0.048374 ms X 500 passes
Fully connected:  Timing [Backward] 33.798 ms, avg: 0.067596 ms X 500 passes
Timing: 50 iterations of 10 calls, shape = (50,1,18,32)
Fully connected:  Timing [Forward] 98.219 ms, avg: 0.196438 ms X 500 passes
Fully connected:  Timing [Backward] 35.933 ms, avg: 0.071866 ms X 500 passes
Timing: 50 iterations of 10 calls, shape = (50,3,18,32)
Fully connected:  Timing [Forward] 346.737 ms, avg: 0.693474 ms X 500 passes
Fully connected:  Timing [Backward] 60.141 ms, avg: 0.120282 ms X 500 passes
Timing: 50 iterations of 10 calls, shape = (20,3,128,128)
Fully connected:  Timing [Forward] 3607.84 ms, avg: 7.21567 ms X 500 passes
Fully connected:  Timing [Backward] 387.725 ms, avg: 0.77545 ms X 500 passes
NEW
Timing: 50 iterations of 10 calls, shape = (1,1,28,28)
Fully connected:  Timing [Forward] 44.111 ms, avg: 0.088222 ms X 500 passes
Fully connected:  Timing [Backward] 0.84 ms, avg: 0.00168 ms X 500 passes
Timing: 50 iterations of 10 calls, shape = (1,3,28,28)
Fully connected:  Timing [Forward] 16.093 ms, avg: 0.032186 ms X 500 passes
Fully connected:  Timing [Backward] 1.419 ms, avg: 0.002838 ms X 500 passes
Timing: 50 iterations of 10 calls, shape = (50,1,18,32)
Fully connected:  Timing [Forward] 137.882 ms, avg: 0.275764 ms X 500 passes
Fully connected:  Timing [Backward] 38.945 ms, avg: 0.07789 ms X 500 passes
Timing: 50 iterations of 10 calls, shape = (50,3,18,32)
Fully connected:  Timing [Forward] 340.161 ms, avg: 0.680322 ms X 500 passes
Fully connected:  Timing [Backward] 68.256 ms, avg: 0.136512 ms X 500 passes
Timing: 50 iterations of 10 calls, shape = (20,3,128,128)
Fully connected:  Timing [Forward] 3465.03 ms, avg: 6.93005 ms X 500 passes
Fully connected:  Timing [Backward] 322.912 ms, avg: 0.645824 ms X 500 passes
Checklist
Essentials

[x] Passed code style checking (make lint)
[x] Changes are complete (i.e. I finished coding on this PR)
[x] All changes have test coverage
[ ] For user-facing API changes, API doc string has been updated.
[ ] To my best knowledge, examples are either not affected by this change, or have been fixed to be compatible with this change

Changes

[ ] Feature1, tests, (and when applicable, API doc)
[ ] Feature2, tests, (and when applicable, API doc)

Comments

If this change is a backward incompatible change, why must this change be made.
Intersting edge cases to note here

Too large, will wait for other PR to complete


WIP: Improve ChefSpec tests

NOTE: This is currently a WIP, therefore won't be feature complete, or ready for merge. However, this will be complete this evening (UTC)
TODOs:

[ ] Test ::default
[x] Test ::package
[ ] Test ::source
[ ] Ensure CI passes
[ ] Rebase for DCO
[ ] Rebase to amend commit descriptions (for extra clarity)

Description
This PR will improve the amount of ChefSpec (unit) tests in the transmission cookbook. It will ensure that in the future, any changes to recipes (that are not updated in tests) will be caught when the tests fail.
Issues Resolved
This closes #20.
Check List

[ ] All tests pass. See https://github.com/chef-cookbooks/community_cookbook_documentation/blob/master/TESTING.MD
[ ] New functionality includes testing.
[ ] New functionality has been documented in the README if applicable
[ ] All commits have been signed for the Developer Certificate of Origin. See https://github.com/chef-cookbooks/community_cookbook_documentation/blob/master/CONTRIBUTING.MD

Closing this out since we never received the DCO sign off


Client never receives authorization tokens.

"The application was successfully granted access. Please wait for the client to retrieve the authorization tokens."
Firefox just sits there and my terminal just sits there waiting for the tokens. So I  removed the default directory with
rm -r ~/.gdfuse/default
And trying again.  Same result.
After removing Firefox, Chrome opens and google-drive-ocamlfuse is able to access tokens correctly.
Unfortunately, nothing is logged while it sits there and does nothing so I don't have anything to give you.
uname -a
Linux cthulhu 4.2.5-1-ARCH #1 SMP PREEMPT Tue Oct 27 08:13:28 CET 2015 x86_64 GNU/Linux
google-drive-ocamlfuse -version
google-drive-ocamlfuse, version 0.5.18
firefox --version
Mozilla Firefox 42.0
google-chrome-stable -version
Google Chrome 46.0.2490.80
xdg-open --version
xdg-open 1.1.0 rc3

If you enable debug logging with -debug, you should see the requests to obtain the token in ~/.gdfuse/default/curl.log. If there is any error, it should be logged there too (or in ~/.gdfuse/default/gdfuse.log).

Okay I'll try that when I get home.
On Wed, Nov 4, 2015 at 2:01 PM, Alessandro Strada<EMAIL_ADDRESS>wrote:

If you enable debug logging with -debug, you should see the requests to
obtain the token in ~/.gdfuse/default/curl.log. If there is any error, it
should be logged there too (or in ~/.gdfuse/default/gdfuse.log).
—
Reply to this email directly or view it on GitHub
https://github.com/astrada/google-drive-ocamlfuse/issues/142#issuecomment-153831308
.

--
Thank you,
Ryan Whited


How to detect syslog via tcp connection loss

If the tcp connection associated with syslog hook goes down, what is the best way to detect this?  Ideally, I'd like to trap the connection loss via a callback, close/destroy the existing hook and recreate a new hook.  Please suggest.

You can make a custom hook that will reconnect/panic/etc if the connection dies.

@sksastry Hitting the same problem. Did you implement a reconnection tool? If so how?


Fix #42: Parse for TAB and CR

Fix #42: Replace all TABS with spaces before processing

@FlipWarthog can you check this before I merge it?

It is on my list for tomorrow. I didn't get a chance to today.

Okay @melloware @jbrower2 I tested with the sample in the ticket and intellisense was flaky, then I loaded these changes and it seems to be working more consistently. I think we're good to merge this


[AES:DV] adding documentation and code for AES DV

I have added the code and documentation for aes verification.
currently only a non DPI wakeup_test is committed until I get dpi working correcty.
please review code and testplan

@sriyerg @imphil
the checks fail due to user name, but I did change the commit name and push somehow the name does not change in my PR how do I fix that?

(I did follow up with Rasmus via email. This PR needs to be closed and a fresh one needs to be opened unfortunately.)


issues with importing deck

Hello,
I am facing issues importing decks through the vocablist and api tabs. I am pretty sure I am facing the same problems as described here. https://github.com/Raffaelbdl/kuma-browser/issues/8
Is there any chance you can add a rate limit throttle? I don't care how long it would take to import, nor performance, as long as it finishes. Even a simple sleep would work here lol.
Anki 24.06.3 (d678e393)  (ao)
Python 3.9.18 Qt 6.6.2 PyQt 6.6.1
Platform: Windows-10-10.0.22631

Traceback (most recent call last):
  File "urllib3.connectionpool", line 466, in _make_request
  File "urllib3.connectionpool", line 1095, in _validate_conn
  File "urllib3.connection", line 652, in connect
  File "urllib3.connection", line 805, in _ssl_wrap_socket_and_match_hostname
  File "urllib3.util.ssl_", line 465, in ssl_wrap_socket
  File "urllib3.util.ssl_", line 509, in _ssl_wrap_socket_impl
  File "ssl", line 501, in wrap_socket
  File "ssl", line 1074, in _create
  File "ssl", line 1343, in do_handshake
ssl.SSLEOFError: EOF occurred in violation of protocol (_ssl.c:1129)

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "urllib3.connectionpool", line 789, in urlopen
  File "urllib3.connectionpool", line 490, in _make_request
urllib3.exceptions.SSLError: EOF occurred in violation of protocol (_ssl.c:1129)

The above exception was the direct cause of the following exception:

Traceback (most recent call last):
  File "requests.adapters", line 589, in send
  File "urllib3.connectionpool", line 843, in urlopen
  File "urllib3.util.retry", line 519, in increment
urllib3.exceptions.MaxRetryError: HTTPSConnectionPool(host='jpdb.io', port=443): Max retries exceeded with url: /api/v1/deck/list-vocabulary (Caused by SSLError(SSLEOFError(8, 'EOF occurred in violation of protocol (_ssl.c:1129)')))

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "path_to_anki\Anki2\addons21\kuma\jpdb_api.py", line 322, in generate_or_update
    notes = api.vocabulary_list(int(deck_id))
  File "path_to_anki\Anki2\addons21\kuma\jpdb_api.py", line 40, in vocabulary_list
    response = requests.post(url, json=payload, headers=headers)
  File "requests.api", line 115, in post
  File "requests.api", line 59, in request
  File "requests.sessions", line 589, in request
  File "requests.sessions", line 703, in send
  File "requests.adapters", line 620, in send
requests.exceptions.SSLError: HTTPSConnectionPool(host='jpdb.io', port=443): Max retries exceeded with url: /api/v1/deck/list-vocabulary (Caused by SSLError(SSLEOFError(8, 'EOF occurred in violation of protocol (_ssl.c:1129)')))

===Add-ons (active)===
(add-on provided name [Add-on folder, installed at, version, is config changed])
'' ['kuma', 0, 'None', '']
JPDB Anki Importer ['541896873', 2023-03-01T22:01, 'None', '']

===IDs of active AnkiWeb add-ons===
541896873

===Add-ons (inactive)===
(add-on provided name [Add-on folder, installed at, version, is config changed])

Hi ! Thank you for testing the addon !
I am not sure what is going in your case. Contrary to the original method, this one uses a single API call to collect all the notes in the vocabulary list, which is the line that failed. I cannot limit the rate for this one. The API was in fact used to go around #8 problem.
I will add a simple line in the json next to the jpdb token that you will be able to change manually to control the original method's rate. If the original methods works for you after that, I will push it to master.

I have added a straightforward sleep like you suggested, could you try installing the addon from this branch ?
On my side, I also have a lot of difficulty using the original method without a VPN. (in fact I still cannot access the website without one right now :laughing: )

PR works great for a moderately large deck ~10 000. Query itself took only maybe a minute with note generation still running. I assume I can change the sleep_time val in vl.json if I need a longer time in the future?

uses a single API call

Ahh I see what happened. I had used the original method > ip banned > api method but too late. Assumed it was using the same query method and didn't want to risk running it again after the unban.
Thanks for the quick PR and addon work!

Indeed that seems quite plausible !
The sleep time is the one set in vl.json. I will set it back to 0 and add a section to the README about this issue as it is very common. Thank you for raising it :hugs:


Website appears fubar

I get "an unrelated error has occurred. Reload". Page displays "header" and "footer", but the body just says "loading"

Additional detail from console logs:
Microsoft.AspNetCore.Components.WebAssembly.Rendering.WebAssemblyRenderer[100]
Unhandled exception rendering component: No item factory specified for 'SpiritAspect'
System.Exception: No item factory specified for 'SpiritAspect'
at SiRandomizer.Data.OverallConfiguration.TakeSettingsFrom[SpiritAspect,SpiritAspect](IComponentCollection1 destination, IComponentCollection1 source, ILogger logger, Func2 itemFactory)    at SiRandomizer.Data.OverallConfiguration.TakeSettingsFrom[SpiritAspect](IComponentCollection1 destination, IComponentCollection1 source, ILogger logger)    at SiRandomizer.Data.OverallConfiguration.TakeSettingsFrom[Spirit,SpiritAspect](IComponentCollection1 destination, IComponentCollection1 source, ILogger logger, Func2 itemFactory)
at SiRandomizer.Data.OverallConfiguration.TakeSettingsFrom(OverallConfiguration other, ILogger logger)
at SiRandomizer.Services.PresetService.InitialisePresetsAsync()
at SiRandomizer.Pages.Index.OnInitializedAsync()
at Microsoft.AspNetCore.Components.ComponentBase.RunInitAndSetParametersAsync()
at Microsoft.AspNetCore.Components.RenderTree.Renderer.GetErrorHandledTask(Task , ComponentState )

Looks like it is an issue involving cached data in local storage. After the redeployment, the cached info for Lure no longers plays nice with the updated Spirit Aspect. Clearing my local storage resolved the issue on my end (i'd expect that clearing cached data would have the same effect, but can't verify).

Apologies, I pushed a change out without trying it first.
I've reverted now, so everything should be running fine.
Should be able to get this sorted properly in the next day or two.

FYI Steve, everything did work as expected once I cleared my browsing history/cached data in my browser.


Warnings in example event_viewer

druid 65eb306018cf0990cce4305952924c2303ed6af9
The event_viewer example throws a bunch of warnings when pressing a keyboard key with the text area focused (top left) while scrolling down with the mouse wheel the list of events (since it's not auto-scrolling, I wanted to see them anyway as I type).
cargo run --example event_viewer

results in many:
WARN  [druid::core] WidgetId(56) received an event (MouseMove(MouseEvent { pos: (329.0, 883.0), window_pos: (420.0, 511.0), buttons: MouseButtons(00000), mods: Modifiers(NUM_LOCK), count: 0, focus: false, button: None, wheel_delta: Vec2 { x: 0.0, y: 0.0 } })) without having been laid out. This likely indicates a missed call to set_layout_rect.

I just downloaded druid 10 min ago so have no idea nor more context to provide. Hopefully someone with more knowledge can repro. :)

Just tried to reproduce this, and I can confirm this still occurs.


DDN-219 Oracle DBName differs from username when username not provided

Did a bit of a refactor then merged, thanks! :-)


quick fix #1002

bmorel sent me this patch, on my end it fixes:

https://github.com/DaemonEngine/Daemon/issues/1002

Quick deepl translation of his words:

it fixes the crashes in theory. But it doesn't solve the bugs, and it's a very naive patch.
in concrete terms, drawSurface_t is the equivalent of: memset( &drawSurface, 0, sizeof( drawSurface_t ) );
i think grangermaze triggers this because of the intensive use of func_misc which appear and disappear quickly
possibly linked to the fact that portals are used to see them

I have not reviewed the code, only tested that it builds and fixes the bug.

I wonder why we're getting null entities in the first place

I wonder why we're getting null entities in the first place

Yes.
I'm closing this PR in favor of #1004 anyway:

https://github.com/DaemonEngine/Daemon/pull/1004


Simplifying PieceLocations

Piece Locations is now a structure of [u8; 32], with each square (consisting of a Piece and a Color) represented as 4 bits.
This makes it very compact, but may provide unnecessary overhead due to needing to mask and shift values. A possible solution is accept wasted space, and use an array of [u8; 64] to represent each square

#18


[Misc] Take user preference in attention selector

The current selection logic on NVIDIA GPUs is a bit weird on NVIDIA GPUs. Specially, it checks whether the model can use FlashAttn, and directly falls back to xFormers if not, without considering consider user preference (i.e., environment variable VLLM_ATTENTION_BACKEND). User specified backend can only be used  when FlashAttn is valid for the model and installed in the environment.
This PR refactors the logic to consider user preference first.
In addition, this PR also fixes the issue that eager mode is not enforced when FlashInfer is used. This was introduced by #4353 but somehow removed by recent refactoring. Since the model runner refactoring is still ongoing, this PR simply refines the WARNING message to ask users manually specify --enforce-eager.
cc @LiuXiaoxuanPKU @rkooo567 @WoosukKwon
BEFORE SUBMITTING, PLEASE READ THE CHECKLIST BELOW AND FILL IN THE DESCRIPTION ABOVE

 PR Checklist (Click to Expand)
Thank you for your contribution to vLLM! Before submitting the pull request, please ensure the PR meets the following criteria. This helps vLLM maintain the code quality and improve the efficiency of the review process.
PR Title and Classification
Only specific types of PRs will be reviewed. The PR title is prefixed appropriately to indicate the type of change. Please use one of the following:

[Bugfix] for bug fixes.
[CI/Build] for build or continuous integration improvements.
[Doc] for documentation fixes and improvements.
[Model] for adding a new model or improving an existing model. Model name should appear in the title.
[Frontend] For changes on the vLLM frontend (e.g., OpenAI API server, LLM class, etc.)
[Kernel] for changes affecting CUDA kernels or other compute kernels.
[Core] for changes in the core vLLM logic (e.g., LLMEngine, AsyncLLMEngine, Scheduler, etc.)
[Hardware][Vendor] for hardware-specific changes. Vendor name should appear in the prefix (e.g., [Hardware][AMD]).
[Misc] for PRs that do not fit the above categories. Please use this sparingly.

Note: If the PR spans more than one category, please include all relevant prefixes.
Code Quality
The PR need to meet the following code quality standards:

We adhere to Google Python style guide and Google C++ style guide.
Pass all linter checks. Please use format.sh to format your code.
The code need to be well-documented to ensure future contributors can easily understand the code.
Include sufficient tests to ensure the project to stay correct and robust. This includes both unit tests and integration tests.
Please add documentation to docs/source/ if the PR modifies the user-facing behaviors of vLLM. It helps vLLM user understand and utilize the new features or changes.

Notes for Large Changes
Please keep the changes as concise as possible. For major architectural changes (>500 LOC excluding kernel/data/config/test), we would expect a GitHub issue (RFC) discussing the technical design and justification. Otherwise, we will tag it with rfc-required and might not go through the PR.
What to Expect for the Reviews
The goal of the vLLM team is to be a transparent reviewing machine. We would like to make the review process transparent and efficient and make sure no contributor feel confused or frustrated. However, the vLLM team is small, so we need to prioritize some PRs over others. Here is what you can expect from the review process:

 After the PR is submitted, the PR will be assigned to a reviewer. Every reviewer will pick up the PRs based on their expertise and availability.
 After the PR is assigned, the reviewer will provide status update every 2-3 days. If the PR is not reviewed within 7 days, please feel free to ping the reviewer or the vLLM team.
 After the review, the reviewer will put an  action-required label on the PR if there are changes required. The contributor should address the comments and ping the reviewer to re-review the PR.
 Please respond to all comments within a reasonable time frame. If a comment isn't clear or you disagree with a suggestion, feel free to ask for clarification or discuss the suggestion.

Thank You
 Finally, thank you for taking the time to read these guidelines and for your interest in contributing to vLLM. Your contributions make vLLM a great tool for everyone!

Rebase according to #4944.
cc @rkooo567 @Isotr0py @mgoin

As we now still maintain a clean logic separation between which_attn_to_use and get_attn_backend, I promoted which_attn_to_use to a public method and added a test for it, so that we don't have to import other backends. @rkooo567 @mgoin PTAL

awesome tests!


Neofetch exits with error code 1

Description
Neofetch exits with an error code of 1, and it shows up as a problem in my screenshots! It didn't do this a bit ago.

[x] Does this issue still occur in the master branch? (Required if issue)

Neofetch version
Neofetch 7.1.0
Screenshot
P10K shows an error after running Neofetch

Don't think it's a config problem, as it does it even when I override it.

Config file
Here
Verbose log
Here

Check it: https://github.com/dylanaraps/neofetch/issues/2209

Yup, that's it. No errors with neofetch --disable cols.

When editing my config to place colours higher up, it just stops abruptly.
Config:

Output:

I can reproduce this issue, and calling --disable cols also stops it from exiting with 1

Which version of bash are you using? Can I see /usr/bin/env bash --version?

Returns
GNU bash, version 5.2.2(1)-release (x86_64-suse-linux)
Copyright (C) 2022 Free Software Foundation, Inc.
License GPLv3+: GNU GPL version 3 or later <http://gnu.org/licenses/gpl.html>

This is free software; you are free to change and redistribute it.
There is NO WARRANTY, to the extent permitted by law.

The column cutoff issue on Bash ≥5.2 is fixed in HyFetch 1.4.3 / neowofetch 7.3.3 by the pull request https://github.com/hykilpikonna/hyfetch/pull/24.

HyFetch is a fork of neofetch with LGBTQ pride flags, but the repo also maintains an updated version of the original neofetch, addressing many pull requests that are not merged in the original repo.
Read the "Running Updated Original Neofetch" section for more info!

fork of neofetch with LGBTQ pride flags, but the repo also maintains an updated version of the original neofetch, addressing many pull requests that are not merged in the original repo.
Sounds good, is this neofetch codebase dead?

Sounds good, is this neofetch codebase dead?

It probably is... this repo hasn't been updated for almost a year.


Create a long-term support/maintenance plan

As stated in #105 we should determine a long-term maintenance plan. This should be documented in our contribution guidelines defined in #103.

Closing due to issue age.


📝 Mutate tiledbsoma store

This is great!
The only thing that needs to be prettified is to hide the output here:


[PBI] Add pub sub service as sub-repo in Chariott

Description
Add a publish/subscribe service (as a repo) to the Chariott project. This will add a service that registers with Chariott to provide publish and subscribe functionality and dynamic topic management capabilities.
Acceptance criteria

[ ] Service provides publish/subscribe functionality
[ ] Easily integrates with Chariott service

The repo has now been added to the Chariott project here: Agemo


docs: Python Tutorial: Dependencies, PyPI, Order, Grammar

I was searching the documentation for a way to install optional dependencies from PyPI using pixi and got helped on the Discord.
I figured the Python tutorial is the most straightforward place to add this, while the functionality is still being developed.
I have taken this chance to slightly adjust the flow of the tutorial and used Grammarly Premium to work through the grammar and sprinkle in some commas.

He @JesperDramsch, Thanks for helping us improve the tutorial. I gave it another pass. The amount of line changes in my commit have to do with that I like the "single sentence per line" principle. Which wasn't used in the tutorial yet.

Thanks for the follow up @ruben-arts those single-line changes make absolute sense to me. I wasn't sure if I should change them all, as that would've been an artificially huge PR from a newcomer, so glad you took those on.

LGTM and @ruben-arts will merge!

Thanks everyone!


spring handson 2 done

Metric
Value

alert_status
:white_check_mark: OK

bugs
0

code_smells
0

reliability_rating
A (0 Bugs)Note: A being best, E being worst

security_rating
A (0 Vulnerabilities)Note: A being best, E being worst

vulnerabilities
0


set command is used instead of setx one in legacy code analysis setup

set command is used instead of setx one in legacy code analysis setup:
set EnableLegacyCodeAnalysis = true

But the right command is:
setx EnableLegacyCodeAnalysis true

I've checked it out - it works properly.

@alvinseville7cf -- Alvin, thank you for your feedback and fix.


Firefox: Pass page wait for tests

related to #113

@kblok what happened to the CI? works for me locally

@Meir017 I stopped it to rebuild it. It seems we have a recurrent fail.


[Bug] Artisan migrate appears to work when an SQLite database doesn't exist without error

In an environment where the database being used is SQLite, if you attempt to run artisan migrate where the .sqlite file for the database does not exist, it will loop through all the migrations and display that a complete migration was successful.
Simply creating a blank file, touch development.sqlite and then running artisan migrate means the migration actually works.
I think that an error should be given if the database file does not exist so that the user is not given the illusion that something is happening when it is not.

2 years later and issuing a migration still doesn't create the SQLite file? What am I missing here?

@TeckniX :+1:

@TeckniX :anguished:

This is fixed. We're not creating the file. The error was we weren't throwing an exception if it didn't exist, and that's fixed now.

i downloaded laravel yesterday and i didn't get any error

I get the error still when trying to connect to a sql lite file that doesn't exist - I would like to have it created as I am using forge and want less configuration as possible.


(6.4.x) RHBPMS-678: parameters of conditions can not be null more

I don't know whether we want this in the 6.4.x branch - RHBPMS-678 is a regression for v6.2.0, but it's not a blocker. What does one of the gatekeepers think?

I understand, I am fine if the PR will be not merged into 6.4.x it is not critical bug.

In case we want to include this PR into 6.4.x I amended this PR according to comments on master.

PR created for 6.5.x.


nuxt: Unexpected token <

Hi @tchret, could you detail the error a little more, please?

[vue-router] Failed to resolve async component default: SyntaxError: Unexpected token <
[vue-router] uncaught error during route navigation:
/Users/olivier/projects/dzango.com/www/node_modules/vue-country-flag/CountryFlag.vue:1

SyntaxError: Unexpected token <
at Module._compile (internal/modules/cjs/loader.js:721:23)
at Object.Module._extensions..js (internal/modules/cjs/loader.js:787:10)
at Module.load (internal/modules/cjs/loader.js:653:32)
at tryModuleLoad (internal/modules/cjs/loader.js:593:12)
at Function.Module._load (internal/modules/cjs/loader.js:585:3)
at Module.require (internal/modules/cjs/loader.js:690:17)
at require (internal/modules/cjs/helpers.js:25:18)
at r (/Users/olivier/projects/dzango.com/www/node_modules/vue-server-renderer/build.js:8360:16)
at Object. (server-bundle.js:2316:18)
at webpack_require (server-bundle.js:27:30)
at Object. (0.server-bundle.js:546:75)
at webpack_require (server-bundle.js:27:30)
at Object. (0.server-bundle.js:537:191)
at webpack_require (server-bundle.js:27:30)
at Object. (0.server-bundle.js:1545:96)
at webpack_require (server-bundle.js:27:30)
error during render : /
/Users/olivier/projects/dzango.com/www/node_modules/vue-country-flag/CountryFlag.vue:1

SyntaxError: Unexpected token <
at Module._compile (internal/modules/cjs/loader.js:721:23)
at Object.Module._extensions..js (internal/modules/cjs/loader.js:787:10)
at Module.load (internal/modules/cjs/loader.js:653:32)
at tryModuleLoad (internal/modules/cjs/loader.js:593:12)
at Function.Module._load (internal/modules/cjs/loader.js:585:3)
at Module.require (internal/modules/cjs/loader.js:690:17)
at require (internal/modules/cjs/helpers.js:25:18)
at r (/Users/olivier/projects/dzango.com/www/node_modules/vue-server-renderer/build.js:8360:16)
at Object. (webpack:/external "vue-country-flag":1:0)
at webpack_require (webpack:/webpack/bootstrap cfa7b7c94d8b68de6b4d:25:0)
at Object. (0.server-bundle.js:546:75)
at webpack_require (webpack:/webpack/bootstrap cfa7b7c94d8b68de6b4d:25:0)
at Object. (src/modules/home/_components/Contact.vue?c0b3:1:0)
at webpack_require (webpack:/webpack/bootstrap cfa7b7c94d8b68de6b4d:25:0)
at Object. (src/modules/home/_components/Contact.vue:1:0)
at webpack_require (webpack:/webpack/bootstrap cfa7b7c94d8b68de6b4d:25:0)

@opichon thanks for the log, I'll give it a look.
Just to confirm my understanding here: did you notice the issue while building with Webpack and this component embedded in as dependency?

The component was embedded inside another Vue component
import VueCountryFlag from 'vue-country-flag`

export default {

  components: {
    CountryFlag
  }
}

(approximately only, since I've now removed the component).
The app is built with Webpack.

@opichon I am trying to reproduce your problem, but at the moment, even using Webpack, in my samples the component renders properly.
Do you have any environment with your Webpack configuration setup? That would be helpful to further investigate the problem.
Thank you!

 Thank you 😄

@tchret @opichon an updated version of the component is available on NPM. It is a built version of the component as a minified SFC available in three different flavours:

UNPKG version (see<EMAIL_ADDRESS>ES module
SSR

Please, have a look. This might resolve this issue too.

Just in case you wanted to know, this works for me in Nuxt. :)


White dot in the center

There will be a white dot in the center of the screen when using
java -jar path-to/sikulixapi.jar -p
on Windows 10.
This may cause OCR failure.
Not tested with pattern recognition. But I think it's best to remove it.

Should I put the issues here or in https://github.com/RaiMan/SikuliX1/issues ？

LOL, never seen this before: it is the background positioner for the popup and becomes visible on non-white backgrounds if you move the popup out of way.
This anyways is only a helper for now, to get a chance to stop the server.
I think I will try to change this to the usage of a hot-key (probably alt-shift-c)

... and yes: everything related to real Python usage should be reported here for now.
Be aware: very early stage, very experimental.

I have removed the popup and added the possibility to stop the server with the stop hotkey.

Very Cool Project! Thank you!


Request to publish the Source

Hello how are you? Can I contact you from here?

I would appreciate it if you would reply when you see this message.

Hello how are you? Can I contact you from here?
I would appreciate it if you would reply when you see this message.

Hey, I'm doing fine. Thanks. How can I help you?

Do you have Discord?

Do you have Discord?

Sure, but I'd first like to know what this is about. Unfortunately I am a little short on time, so I can not join any other projects currently.

I'll tell you right away then.

I just discovered your software and it works fine, but it has some minor issues. I think your adjustments according to my country are wrong, I am from Turkey. If you don't mind, could you share the source codes with me? I am also a software developer and according to my country, your software does not work properly.

I am having problems with the letters İ, Ü, Ö, Ğ, Ç, which belong to my country.

Oh alright. Thanks for reporting the issue. Honestly did not even know that the software still works, neither that someone uses it. If I still have the source code, I ll see if I can push an update or share it directly. If I remember correctly due to it being one of my first creations, the code quality was rather bad, which was why I pushed an exe in the first place ^^.  I'll give it a look over the weekend.

It would be better for me if you share it, I want to adapt it directly to myself, I want to change a few more things.

It was a little difficult to discover your software, people cannot find it by searching this way. I would also like to inform you that my goal is definitely not to make a profit, but to adapt and use it according to my own needs.

It would be better for me if you share it, I want to adapt it directly to myself, I want to change a few more things.

Alright, if I still have it, I'll clean it up a little, not to embarrass myself too much, and push it. Move the exes to the release tab, as it should be. Then you can adjust the source as you wish and build yourself. Is this okay with you?

Never think of it as embarrassing, sir, even if you write "hello world" there, this is still an effort.

I want to fix one more thing. Is something like this possible? While writing the text, it creates backspace and corrects it, but I want it to pass without correcting it after I write it wrong.

I want it to pass by typing the wrong text without pressing the backspace button.

I want to fix one more thing. Is something like this possible? While writing the text, it creates backspace and corrects it, but I want it to pass without correcting it after I write it wrong.

I think it does not always correct the words. If I remember correctly, it should sometimes just mistype without correction. Although this may be rare.

Yes, but it's hard to time it during video evidence.

I don't want any corrections.

Just write it wrong and skip it.

Ah, okay, when I find the source, you should definitely be able to adjust to your liking. But you need to give me some time. I need to go off now. I'll get back to you within the weekend.

Today is Saturday, tomorrow is Sunday. Will you be able to return within these 2 days?

Today is Saturday, tomorrow is Sunday. Will you be able to return within these 2 days?

yes

I don't recommend that we talk here. Would it be okay for you to talk via discord? 10FF officials are constantly checking.

how are you sir?

I don't recommend that we talk here. Would it be okay for you to talk via discord? 10FF officials are constantly checking.

ah I do not think they monitor this project ^^ I'd say they do not care, after all, they have not fixed their website in years...
Also, I found the code, it did not need too much refactoring. The main reason for uploading an exe seems to have been obfuscation, but that really does not seem to be necessary.

how are you sir?

Thanks, good, and you? The sources are online, as well as requirements.txt and build instructions. Enjoy yourself :)

If you make drastical improvements and do not want to keep them private, consider making a PR :^)

Thank you but I couldn't see the source file.

I see now that you opened a new folder and put it there, thank you very much sir.

I want to say it again. My goal is definitely not to make a profit, but to use it for my own benefit.

I want to say it again. My goal is definitely not to make a profit [...]

Okay, honestly, I think there is not much profit to be made, especially with the code in here. As far as I am aware, this can be achieved much easier, without the need for extra UI and stuff, by simply injecting JS into the browser via the dev console.

[...] use it for my own benefit
GL with that! :)

I consider this issue closed

I have reviewed your software and thank you very much for your extensive efforts. Where can I cancel this correction part? If you just tell me this, I will pray for you.

I have reviewed your software and thank you very much for your extensive efforts. Where can I cancel this correction part? If you just tell me this, I will pray for you.

should be around here
https://github.com/Zylence/10-fast-fingers-typing-bot-lightning_fingers/blob/ad89ba3f8439de0c3f653c9a7f3adcffa9ef45d9/main/main.py#L379

Thank you very much, I offer my respects. Have a nice day, good work, sir.


IP limited token

Feature Description
Some users use some token in a special machine with a special IP and they don't want the token be used in another machine.
So a token could add a IP range limitation.
Screenshots
No response

Great, and Thank you !


Tools and Components-as-Tools do not express their specific functionality

Feature Request
The Tool Name and Description that a Tool or a Component-as-a-Tool expresses to the agent should be what is on the Vertex (UI component), and not require editing the Source Code to update this.
Motivation
A tool needs to present its name and description to the Agent , but when using Tools or Components as tools, no amount of editing on the Vertex or the Code itself seems to make a difference in the description being presented. This does vary on a component-by-component basis; the Calculator Tool for example is hard-coded:
    def build_tool(self) -> Tool:
        return StructuredTool.from_function(
            name="calculator",
            description="Evaluate basic arithmetic expressions. Input should be a string containing the expression.",
            func=self._eval_expr_with_error,
            args_schema=self.CalculatorToolSchema,
        )

other components use self.name and self.description. None I have found use the Vertex.
Component as Tool Description problem.json illustrates the problem. I have 3 different calculator tools provided, one the default, another where I edited the Vertex, and the third where I edited the name/description at the top.
The tools parameter is logged, and all three tools are identical:
'tools': HandleInput(trace_as_metadata=True, is_list=True, field_type=<FieldTypes.OTHER: 'other'>, required=False, placeholder='', show=True, name='tools',
value=[
   StructuredTool(name='calculator', description='Evaluate basic arithmetic expressions. Input should be a string containing the expression.', args_schema=<class 'langflow.utils.validate.CalculatorToolComponent.CalculatorToolSchema'>, func=<bound method CalculatorToolComponent._eval_expr_with_error of <langflow.utils.validate.CalculatorToolComponent object at 0x000001B7B934F980>>),
   StructuredTool(name='calculator', description='Evaluate basic arithmetic expressions. Input should be a string containing the expression.', args_schema=<class 'langflow.utils.validate.CalculatorToolComponent.CalculatorToolSchema'>, func=<bound method CalculatorToolComponent._eval_expr_with_error of <langflow.utils.validate.CalculatorToolComponent object at 0x000001B7B9251580>>),
   StructuredTool(name='calculator', description='Evaluate basic arithmetic expressions. Input should be a string containing the expression.', args_schema=<class 'langflow.utils.validate.CalculatorToolComponent.CalculatorToolSchema'>, func=<bound method CalculatorToolComponent._eval_expr_with_error of <langflow.utils.validate.CalculatorToolComponent object at 0x000001B7B93A1E20>>)
],
display_name='Tools', advanced=False, input_types=['Tool', 'BaseTool', 'StructuredTool'], dynamic=False, info='These are the tools that the agent can use to help with tasks.', real_time_refresh=None, refresh_button=None, refresh_button_text=None, title_case=False),

Your Contribution
See  https://github.com/langflow-ai/langflow/pull/4949 that will facilitate this better, and https://github.com/langflow-ai/langflow/pull/4924 is an example of a tool that is able to update its name dynamically based on the Vertex.

Another variation of this problem is "Component as Tool" - there's NO way to edit the tool metadata, so if you wanted to use an API Request to wrap around a microservice, you have to build an entirely new custom component, rather than just click on the Vertex and edit the name and description.


Navigation in finder window randomly covered

Describe the bug
Sometimes when I open the finder the navigation panel is covered by the preview:

Steps to reproduce
No procedure as the problem seems to appear randomly. It even happens if I open the finder from the same method a couple of times.
Expected behavior
The navigation panel should be visible. Here is the finder opened on the exact same method as above:

Neovim version (nvim -v)
0.9.0 commit 040f14598
lspsaga commit
4f07545
Terminal name/version
Gnome Terminal 3.44.0

see version3


Traces-analytics bug fix

Description
added MDS id to the flyouts
Issues Resolved
[List any issues this PR will resolve]
Check List

[ ] New functionality includes testing.

[ ] All tests pass, including unit test, integration test and doctest

[ ] New functionality has been documented.

[ ] New functionality has javadoc added
[ ] New functionality has user manual doc added

[ ] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Linter has not been updated as there is another PR from @ps48 ti fix the traces linter issues


hack/terraform-quickstart: initial commit

This adds a basic way to use terraform to setup AWS compute along with the
quickstart scripts to initiate a cluster. This is designed for forming
the basis for test setup.

Addressed feedback.  Number of worker nodes is now input variable.

Thanks for the review!


tasks

SourceCode/2024/May 2024/pavan/Task 5/project.json
Score: 2/5 - Here's a breakdown of the issues to address before submitting your pull request:

[ProjectSettings/AutomationHub URL]: Add this missing information using Project Settings. (Score: 1/1)
[Main.xaml] Activity usage improvement: Replace WriteLine with tracepoints during debug and consider removing if not needed. Also, evaluate the necessity of a Log Message. (Score: 1/1)
Code cleanup - Duplicate activity names: Ensure each activity has a unique name that reflects its function for better understanding. (Score: 1/2)
[Activity Flowchart title] and [Activity Assign title]: Consider adding more descriptive titles to make the code easier to read. (Score: 0.5/1)
Usage of flowcharts in secondary layers: Limit the use of flowcharts only to top-level activities for maintainability. (Score: 0.5/1)

Total Score: 3/5

@pavansaiuppala
Hi Pavan,
You raised a PR in GitHub randomly. You need to mention the PR number in the Excel tracker within the respective task. Only then can we review your code and update the points in the tracker.


Update package-lock.json

I'm submitting a ...

[x] bug report
[ ] feature request
[ ] other (Please do not submit support requests here (below))

The package-lock.json file was created with an old version of npm. Should fix that (update to lockfile version v2).

package-lock.json regenerated in PR https://github.com/h5bp/create-html5-boilerplate/pull/283


Variable substitution in image tag

Hi Kurt!
I've create a Scala.js sample application using your plugin: https://gitlab.com/bullbytes/scala-js-example
When running the app using docker-compose up, I'd like to use an environment variable to define which tag of the image to use (latest, staging, or production).
The default tag should be latest.
Thus, I added this to my docker-compose.yml, using Docker's variable substitution with default values:
image: registry.gitlab.com/bullbytes/scala-js-example:${APP_DOCKER_IMAGE_TAG:-latest}

While this works using docker-compose up, I run into an error when executing dockerComposeUp in SBT:
Error parsing reference: "registry.gitlab.com/bullbytes/scala-js-example:${APP_DOCKER_IMAGE_TAG:-latest}" is not a valid repository/tag: invalid reference format

Here's my docker-compose.yml.
Thanks!
P.S.: If you have any feedback for my example app (e.g., my build.sbt), I'd be more than glad to here it. 😊

Hi @mb720! Very cool project.
So the way that environment variable substitution works with the sbt-docker-compose plugin is that you need to define the environment values (or a task that calculates the environment variable values) in your build.sbt file. The plug-in then substitutes these values in the compose file before passing it on to docker to process. For reference here is the PR that added this feature.
There is also an example project basic-variable-substitution showing how to use the variable substitution feature.
There are two build.sbt settings that can be used to configure environment values in the docker compose file:
variablesForSubstitution =: // A Map[String,String] of variables to substitute in your docker-compose file. These are substituted substituted by the plugin and not using environment variables.
variablesForSubstitutionTask =: // An sbt task that returns a Map[String,String] of variables to substitute in your docker-compose file. These are substituted by the plugin and not using environment variables.
That being said, looking at your example docker compose file, it looks like Docker introduced a new feature starting in the 2.1 version of the compose file format where you can provide a -default value to the environment variable definition:${VARIABLE:-default}
The current sbt-docker-compose plugin code does not handle this -default value setting at the moment and would have to be updated to support it. For now, you would have to use the standard ${VARIABLE} format and perform any default value logic in your build.sbt. It should be possible to make a code update to the plugin so that it knows how to read the new -default value when it exists and use it if nothing was explicitly provided for substitution. I'll have to look into adding this in the future.
I hope this helps!

@kurtkopchik Handling of the variable substitution default value is something I am as well looking forward to.

@dpfeiffer & @mb720 - I made an update to the plugin that allows it to now handle docker-compose environment variables that are defined using the ${VARIABLE:-default} format introduced in version 2.1.
If you do not define the environment variable value in the settings variablesForSubstitution or variablesForSubstitutionTask then the default value will be used.
This change has been published in 1.0.22-SNAPSHOT
resolvers += "Sonatype Snapshots" at "https://oss.sonatype.org/content/repositories/snapshots/"
addSbtPlugin("com.tapad" % "sbt-docker-compose" % "1.0.22-SNAPSHOT")

Let me know if you find any issues with it before I merge it to master. Apologies for the delay.

@kurtkopchik I will try today

@dpfeiffer - Thank you for verifying! The fix for this issue is now included in an official release:
addSbtPlugin("com.tapad" % "sbt-docker-compose" % "1.0.22")


Make (pasted) tables adapt/automatically adjust to the page width. Columns: relative not absolute.

Describe the feature you'd like
After pasting (the raw content from) an Excel table with many columns I realized that BookStack doesn't adjust the table width (and its contained cells) to the white page width, but instead the table simply flows over the page border:

Instead a table always should have a maximum width (same as text on page), and any column added should always be relative to that table and the other column). Text size should be independently from the cell size adjustable (line break vs. small enough to fit), whereas the overall text size scales with the table size).
The final table should behave more like an image, which means it should adjust in size, like in this screenshot (where the table actually is an image, not HTML code):

I can imagine that this is a complex problem to solve. Perhaps there is a trick to not display an actual HTML table, but instead render a PNG from the inline created table on-the-fly and display that image. (Downside would be that the table’s content would not be directly copyable).

https://github.com/BookStackApp/BookStack/issues/1387


Escape multiple variables by reference

I understand that auto-escaping isn't feasible, but I believe we could still improve things in this area by simply offering a method to mass-escape multiple variables at once. For example, we could then have this at the top of a template:
$this->escapeMultiple($foo, $bar, $baz);

And that would allow the actual template code to be much cleaner:

<?=$foo?>

<?=$bar?>

<?=$baz?>

This way the template author still has to think about escaping, but they only have to do it once at the top. If they've escaped some variables and not others, this is nicely self-documenting, as they can glance back up to the top if they need to check whether a particular variable has been escaped already. It's not quite as slick as full-blown auto-escaping, but this feels like it would be a middle ground that's better than the current requirement of manually escaping every single time, without running into the problems associated with auto-escaping.
I tried to write an extension to add this (which was going to support both strings and arrays of strings), but discovered it seems to be impossible to add as an extension given the current architecture, because template functions cannot receive data by reference.
Any chance of this being added as an official feature? If not, any chance of making it possible for template functions to get data by reference, so I can at least finish my extension?

Hey @cdCorey, I'm thinking on this one, and will get back to you! 👍

Thanks for replying. I'm using a non-extension version of this in a couple templates right now, and it seems to work well for its intended purpose of escaping string variables and arrays filled with strings. It would definitely be more convenient to use as an extension, though.
A slightly more in-depth approach to this – and maybe this is adding more weight than desired, I'll let you be the judge of that – would be to offer the option of registering a template's variables at the top. That would, again, be self-documenting as to what can be expected in the template, but the main uses that spring to mind for me would be to allow variables to be flagged for auto-escaping, or flagged as required (in which case a template might throw an error if the variable is missing). This would also be a good place to set defaults for optional variables.
Right now, I find myself writing code to do a lot of that at the top of each template, and that works fine. But maybe there would be benefit to adding a more structured way of handling that (if nothing else, it would encourage template authors to stick that stuff at the top, which I at least feel is a better place for it than mixed into the template).

@cdCorey, You could also just escape a set of variables before you render the data.
<?php

$engine->render('template', escapeData([
  'field' => 'a'
]));

escapeData is a made up function, but i'm sure you can find a package or create it yourself.
Would something like that not work?

In general this type of function scares me a little since the escaping doesn't happen at the exact point of output. I think this could lead to security issues. For example, maybe you add some extra output down the road, but you forget to the update the escapeMultiple() call earlier.
I would go with @ragboyjr's suggestion and create a helper function for this. I would have thought this would be possible as an extension as well, since all extensions have access to the current template via the public $template property. Using that you could modify (escape) the data of that template, and save it back to the template itself.
Not tested at all, but it would look something like this:
class EscapeMultiple implements ExtensionInterface
{
    public $template;

    public function register(Engine $engine)
    {
        $engine->registerFunction('escapeMultiple', array($this, 'escapeMultiple'));
    }

    public function escapeMultiple(...$vars)
    {
        // escape the vars

        $this->template->data($vars);
    }
}

@reinink, I have to admit I'm a bit frustrated by the contradiction in your latest response. It's fine that you disagree with my suggestion, but if that's because you think moving escaping away from output is a bad idea, how can you then turn around and say moving it even further away from output is a good idea?
While I appreciate @ragboyjr's suggestion, I do think it's off the mark. Not only does escaping when the template is called further separate escaping from output, but it requires repetition of escape code every time the template is used, and breaks separation of concerns because now every piece of code that calls the template also has to know precisely how the template is going to use its input—and as a result, any change to how the template uses its input has the potential to break and require updates to every single piece of code that calls the template. Even in a hypothetical best case scenario where the template's internal logic never changes, requiring every developer to know and remember exactly which variables to escape every single time they call every single template seems far more prone to cause security issues due to human error. It's just not a good solution.
I agree with the current approach to escaping insofar as that I absolutely think it belongs in the template. This keeps things DRY, maintains single responsibility, keeps escaping packaged with output, etc.
Anyway, a custom extension is a perfectly acceptable solution. My only real issue was that, as far as I could tell at the time, it didn't seem to be possible to get this working as an extension. However, I may not have been aware there was a template->data() function to let me overwrite the existing template variables. If that's the correct way to do it, that should solve my issue. I'll give it another shot when I get a chance, and if I get an extension to do this working I'll link it here for anyone else who wants it. Thank you for the assistance.

First off, sorry you're frustrated. That was certainly not my intention. We're doing our best.
My point isn't where the escaping logic is kept, it's about where it's called. Sorry if I didn't make that clearer. I think that if you're using native PHP templates you should be doing the escaping at the exact point of output, and really to every single variable that you output regardless of whether it's considered "safe" or not.
I've spent more hours than I'd like to admit trying to improve escaping in Plates, and this is where I've landed. There are limits on what can be done with native PHP when it comes to escaping. This is actually a big reason why I recommend developers use a library that supports automatic escaping, such as Twig or Blade. (I understand that might seem odd coming from the developer of a native PHP templating library.)
Creating an extension to do this should be pretty straight forward. Let us know if you run into any issues with that!

@reinink, it was just a bit confusing that you expressed support for @ragboyjr's suggestion when that directly contradicted the reason you said you disagreed with my suggestion. Your response doesn't really address that, but it does reiterate the same position you've expressed consistently everywhere else, so I have to assume there was just some kind of miscommunication. Happens all the time on the Internet, so no worries.
I'm not sure how escaping "every single variable that you output regardless of whether it's considered 'safe' or not" would work, though. I often need to pass HTML content to templates. That can't be escaped without breaking the markup, or am I missing something?

I often need to pass HTML content to templates. That can't be escaped without breaking the markup, or am I missing something?

Correct, you wouldn't escape those. I mean all variables, except those that must be outputted raw.

@reinink, the call to $this->template->data($vars) doesn't appear to be working. I can echo properly escaped $vars within my extension right up to the line where I push it to the template, so I know the rest is working, but when I output the variables in an actual template, they're not coming out escaped. I looked at the code for data() and it certainly looks like what you suggested should work, though. Pretty weird. Any thoughts you'd be willing to share?
`class EscapeMultiple implements ExtensionInterface {
public $template;

public function register(Engine $engine) {
	$engine->registerFunction('escapeMultiple', [$this, 'escape_multiple']);
}

/**
 * Escapes multiple string variables or arrays of strings by reference.
 *
 * This is the public API call. It tests provided data, and if any non-string data
 * is received, throws an exception without making changes. If all data passes, it
 * loops through all strings and replaces them with escaped strings.
 *
 * @param    string|string[] $vars... Any number of string variables or arrays of strings to be escaped.
 * @return   void
 */
public function escape_multiple(...$vars) {
	if ( !$this->are_all_strings($vars) ) {
		throw new Exception('escape_multiple() received inescapable data!');
	}

echo 'UNESCAPED: ';
print_r($vars);
$vars = $this->escape_confirmed_strings($vars);
echo 'AFTER ESCAPING: ';
print_r($vars);
$this->template->data($vars);
} // escape_multiple()
/**
 * Escapes pre-tested array of strings and/or arrays of strings by reference.
 *
 * @param    string[] $vars Array of strings and/or arrays of strings to be escaped.
 * @return   void
 */
private function escape_confirmed_strings($vars) {
	foreach ($vars as &$v) {
		if ( is_array($v) ) {
			// Escape each array element
			$v = $this->escape_confirmed_strings($v);
		} else {
			$v = $this->escape_confirmed_string($v);
		}
	} // foreach

	return $vars;
} // escape_confirmed_strings()

/**
 * Escapes one pre-tested string variable by reference.
 *
 * @param    string $var One string variable to be escaped.
 * @return   void
 */
private function escape_confirmed_string($var) {
	$var = $this->template->escape($var);
	echo "ESCAPE: $var\n\n";
	return $var;
} // escape_confirmed_string()

/**
 * Confirms all non-empty values in array are either strings or arrays of strings.
 *
 * If ANY non-string/non-string-array value is found, test immediately fails.
 *
 * @param    string $var A variable to test.
 * @return   bool Indicates whether all values are strings.
 */
public function are_all_strings(array $vars) {
	foreach ($vars as $v) {
		if ( !empty($v) ) {
			if ( is_array($v) ) {
				// Check each array element
				$valid = $this->are_all_strings($v);

				if (!$valid) return false;
			}
			elseif ( !is_string($v) ) {
				return false;
			}
		} // if !empty
	} // foreach

	// If we got here, everything passed!
	return true;
} // are_all_strings()

} // class
`

@cdCorey it looks like you are calling data() wrong. data expects an associative array of varname => value. You are passing in an array of values, not an associative array of names to values. So those values will NOT be escaped. ALSO, i'm not sure that calling data will also extract the variables properly.
$template->data(['container_classes' => $container_classes]).
You'd have to accept them by reference because you're just escaping new values, but not actually escaping the variables inside of the template.
Right now, you are accepting copies of the values you provided.
And i'm not even sure you could properly capture variables by reference

This is different than your current approach, but maybe you could try injecting a function into the data like so.
<?php

$template->data(['e' => function($v) use ($template) {
  return $template->escape($v);
});

Then in your template, you can simply escape via <?=$e($var)?>
This is a departure from what your are trying to do, but it could definitely be implemented and might be just enough for what you need.

@ragboyjr, if you look at the title of this issue, you'll note my request in the first place was to make it possible to pass the template variables to an extension by reference, because that seems to be impossible with the current architecture. I only tried the data() function at the recommendation of @reinink, but it appears his suggested code won't work because the template's data array is extracted before the extension can be called. Bottom line, making changes to the template's variables from within an extension is simply not supported by Plates's current architecture.
Since my original request has already been rejected and closed, it doesn't look like that limitation is going to change, so I'll just have to keep using an external function to do this. I think it would be cleaner as a Plates extension, but c'est la vie. Thanks to both of you for taking the time to offer some suggestions.

@cdCorey, another thought...
You could also just create your own function to escape multiple and inject into your templates..... It doesn't need to be apart of the plates template class.
<?php

$escapeMulti = function(&...$vars) {
     foreach ($vars as &$var) {
        $var = htmlentities($var);
    }
};
$engine->addData(['escapeMulti' => $escapeMulti]);

In the template
<?php $escapeMulti($var1, $var2) ?>


PyTCPEncoder can fail if the user provided function returns a string with a unicode character

In the apply function of the PyTCPEncoder if the user provided encode function returns a string with a unicode character the @PyString_AsString function can raise an error and cause the application to crash with a call to Fail().
Should we explicity tell users that unicode characters need to be encoded in the encode function in order to prevent this crash?
Should we do additional encoding on our end prior to sending the message out to the sink?

This issue is, in part, a Python string encoding issue.  I think a documenting the problem would be good. We can't, however, do "additional encoding" on our part. It might not be the correct encoding.

@SeanTAllen ok, I'll update this issue to indicate that it should be documented

In 0.5.4 we allow encoders to return byte arrays or unicode strings.


[Feature] Show / hide menu entries based on user permissions

Is your feature request related to a problem? Please describe.
One common requirement in production are different user roles, with granular access rights, for example:

Admin User (CRUD on everything, can access and modify everything)
Extended User (CRUD on specific index, can create, open, edit, and delete dashboards, and has additional permissions e.g. to access notebooks or alerts)
User (CRUD on specific index, can create, open, edit, and delete dashboards)
Limited User (Read Only on specific index, can only view dashboards)

Currently there is only the option to either show all menu entries or restrict them to only show Dashboards with the kibana_read_only role.
These are suited for the Admin User and the Limited User use case.
All other users see the full menu and after they click on a menu item without having the neccessary permissions they are presented with an empty list and different red error messages.
Especially for user that are new to opensearch and need to get used to it without being overhelmed, this behaviour is off-puting gives a "buggy" first impression.
Describe the solution you'd like
It would be beneficial for users to only see the menu entries, that are available to them.
Describe alternatives you've considered
A whitelist to define a subset of the menu entries to be shown, optionally configurable for each role.
Additional context
Related thread from opensearch forum
Full Menu:

Example Error Messages:

Read Only Menu:

was there any progress concerning this feature? It would be very good to a proper UI permission system working. It is very confusing from an user persepective that you get error messages while accessing feature where you don't have permissions.

We're working on workspace feature which allows workspace admin to config which plugins will be available in the workspace, so once in the workspace, user only see plugins they choose. However to support show/hide menu entries based on user permission is still under investigation.

Linking/closing in favor of the conversation around workspaces


Use local inbox folder

After configuring nmail, I assume it accesses my mails on the server. If it's right, it probably can't access mails stored locally. Did I miss some options ?
I use neomutt at the moment and my workflow is the following:

retrieve and store locally all mails with fdm
compose a new mail with neomutt
send mail with msmtp.

So the mail client don't need to access the server. It's done by fdm and msmtp. My mail client, it would be nmail, must be able to read my mails locally, that's all. Is it possible ? If no, do you plan to implement it ?

Hi @Vouivre - nmail only supports retrieving/sending email via network protocols (IMAP and SMTP). It does support disk caching of messages retrieved from an IMAP server to improve performance and reduce network usage.
But nmail does not support reading local emails stored by other applications such as fdm, and there is currently no plan to add support for it. I will update the README to clarify this.
One of the main motivations for creating nmail in the first place was to create a single easy-to-use and easy-to-configure console-based email program.
If you like the UI of nmail it might be worth checking out alpine (the UI of nmail was largely inspired by alpine and its predecessor pine). I believe alpine supports local mail storage.


Azure mssql database and elasticpool support.

Change description
Azure mssql database and elasticpool support.
Type of change

[ ] Bug fix (fixes an issue)
[ ] New feature (adds functionality)

Related issues

Fix #1

Checklists
Development

[ ] Pull requests may not be submitted to the master branch (use develop instead) - or they will be closed.
[ ] Lint rules pass locally
[ ] Application changes have been tested thoroughly
[ ] Automated tests covering modified code pass

Security

[ ] Security impact of change has been considered
[ ] Code follows company security practices and guidelines

Code review

[ ] Pull request has a descriptive title and context useful to a reviewer.
[ ] "Ready for review" label attached and reviewers assigned
[ ] Changes have been reviewed by at least one other contributor
[ ] Pull request linked to task tracker where applicable

Task linked: DUPLO-3516 Azure - Elastic pool creation

Task linked: DUPLO-3521 Azure - SQL Database create


SparkConf() not resolved

I must be doing something wrong here.
I've community Cassandra 3.7, Spark 1.6.2
Here is what I got:
ubuntu@:~/spark-1.6.2-bin-hadoop2.6$ ./bin/pyspark --packages TargetHolding/pyspark-cassandra:0.3.5
Python 2.7.6 (default, Jun 22 2015, 17:58:13)
[GCC 4.8.2] on linux2
Type "help", "copyright", "credits" or "license" for more information.
Ivy Default Cache set to: /home/ubuntu/.ivy2/cache
The jars for the packages stored in: /home/ubuntu/.ivy2/jars
:: loading settings :: url = jar:file:/home/ubuntu/spark-1.6.2-bin-hadoop2.6/lib/spark-assembly-1.6.2-hadoop2.6.0.jar!/org/apache/ivy/core/settings/ivysettings.xml
TargetHolding#pyspark-cassandra added as a dependency
:: resolving dependencies :: org.apache.spark#spark-submit-parent;1.0
confs: [default]
found TargetHolding#pyspark-cassandra;0.3.5 in spark-packages
found com.datastax.spark#spark-cassandra-connector-java_2.10;1.6.0-M1 in central
found com.datastax.spark#spark-cassandra-connector_2.10;1.6.0-M1 in central
found org.apache.cassandra#cassandra-clientutil;3.0.2 in central
found com.datastax.cassandra#cassandra-driver-core;3.0.0 in central
found io.netty#netty-handler;4.0.33.Final in central
found io.netty#netty-buffer;4.0.33.Final in central
found io.netty#netty-common;4.0.33.Final in central
found io.netty#netty-transport;4.0.33.Final in central
found io.netty#netty-codec;4.0.33.Final in central
found io.dropwizard.metrics#metrics-core;3.1.2 in central
found org.slf4j#slf4j-api;1.7.7 in central
found org.apache.commons#commons-lang3;3.3.2 in central
found com.google.guava#guava;16.0.1 in central
found org.joda#joda-convert;1.2 in central
found joda-time#joda-time;2.3 in central
found com.twitter#jsr166e;1.1.0 in central
found org.scala-lang#scala-reflect;2.10.5 in central
:: resolution report :: resolve 484ms :: artifacts dl 10ms
:: modules in use:
TargetHolding#pyspark-cassandra;0.3.5 from spark-packages in [default]
com.datastax.cassandra#cassandra-driver-core;3.0.0 from central in [default]
com.datastax.spark#spark-cassandra-connector-java_2.10;1.6.0-M1 from central in [default]
com.datastax.spark#spark-cassandra-connector_2.10;1.6.0-M1 from central in [default]
com.google.guava#guava;16.0.1 from central in [default]
com.twitter#jsr166e;1.1.0 from central in [default]
io.dropwizard.metrics#metrics-core;3.1.2 from central in [default]
io.netty#netty-buffer;4.0.33.Final from central in [default]
io.netty#netty-codec;4.0.33.Final from central in [default]
io.netty#netty-common;4.0.33.Final from central in [default]
io.netty#netty-handler;4.0.33.Final from central in [default]
io.netty#netty-transport;4.0.33.Final from central in [default]
joda-time#joda-time;2.3 from central in [default]
org.apache.cassandra#cassandra-clientutil;3.0.2 from central in [default]
org.apache.commons#commons-lang3;3.3.2 from central in [default]
org.joda#joda-convert;1.2 from central in [default]
org.scala-lang#scala-reflect;2.10.5 from central in [default]
org.slf4j#slf4j-api;1.7.7 from central in [default]
---------------------------------------------------------------------
|                  |            modules            ||   artifacts   |
|       conf       | number| search|dwnlded|evicted|| number|dwnlded|
---------------------------------------------------------------------
|      default     |   18  |   0   |   0   |   0   ||   18  |   0   |
---------------------------------------------------------------------
:: retrieving :: org.apache.spark#spark-submit-parent
confs: [default]
0 artifacts copied, 18 already retrieved (0kB/13ms)
16/07/30 01:47:07 WARN NativeCodeLoader: Unable to load native-hadoop library for your platform... using builtin-java classes where applicable
Welcome to
____              __
/ /  ___ / /
\ / _ / _ `/ __/  '/
/ / ._/_,// //_\   version 1.6.2
//
Using Python version 2.7.6 (default, Jun 22 2015 17:58:13)
SparkContext available as sc, HiveContext available as sqlContext.

sc
<pyspark.context.SparkContext object at 0x7f2456894910>
import pyspark_cassandra
conf = SparkConf()
... .setAppName("PySpark Cassandra Test")
... .setMaster("spark://<IP_ADDRESS>:7077")
... .set("spark.cassandra.connection.host", "<IP_ADDRESS>")
Traceback (most recent call last):
File "", line 1, in
NameError: name 'SparkConf' is not defined

did you import SparkConf?


kpm add additional nuget sources

My company has its own nuget sources, how can I add so kpm uses these as well?

Use NuGet.Config

Yep, found this out and it works. Sorry for rogue post here


JSON API to get a screenshot and/or current dominant color.

Feature request

What problem does this feature solve?
I am trying to get and download a screen capture of what is processed by hyperHDR. I want this to create a dominant color sensor on Home Assistant.
Another aproach could be get an JSON API to expose the current dominant color value.
What does the proposed API look like?
What i am looking for is an JSON API that return the the current image being processed by HyperHDR (Like the Live "Screenshot" feature of the GUI.)
How should this be implemented in your opinion?
n/a
Are you willing to work on this yourself?
Should learn C/C++.

Hi
How often do you plan to execute it? I'm afraid this is the perfect backdoor to kill HyperHDR/RPi with overload as such a potential command is not guarded and averaging over the whole image (or returning the image as a string/base64 encoded in the JSON result) is very time consuming? Alternatively, the average color of the LEDs for the selected instance?

Hi Awawa, thanks for your fast response.
I want to execute this just on certain media_player state changes (Ex: to playing, to pause and track changed) using the built in home assistant color quantizer.
Also like the idea to get the average color of leds for a specific instance instance. It should works well and it is cheaper for the cpu cause i may create a instance with just a few leds to make the math easier.

Implemented in https://github.com/awawa-dev/HyperHDR/pull/611


feat(api): Adds unstable_pages module to JS API

What this PR solves / how to test:
This PR refactors the existing Pages publish code into its own exportable function and exports it in the wrangler API under the unstable_pages object. This will allow folks to run the publish function in a scriptable JS environment and will get the entire deployment response as a variable.
Associated docs issues/PR:

https://github.com/cloudflare/wrangler2/issues/2437

Author has included the following, where applicable:

[ ] Tests
[x] Changeset

Reviewer has performed the following, where applicable:

[x] Checked for inclusion of relevant tests
[x] Checked for inclusion of a relevant changeset
[x] Checked for creation of associated docs updates
[ ] Manually pulled down the changes and spot-tested

I will pull this down and test it tomorrow, unless someone else gets to it first. At a quick glance it seems like just turning the current code programmatic and moving it under the /api which as long as the tests are passing should be a confident refactor.


Sync Status: Update UI

Modify the sync status/history UI to better reflect the latests designs. Also refactors our approach to passing data over to the list adapter slightly, and adds some new drawable resources.

@gino-m FYI - but I think we still need a few tweaks on the UI itself?

See #876

I think we should keep the entire "History and sync status" if that's what's in the nav bar. In very small screens the header may break, so I'd also recommend we shorten this title if we can. Maybe just "history and sync"?

I think we should keep the entire "History and sync status" if that's what's in the nav bar. In very small screens the header may break, so I'd also recommend we shorten this title if we can. Maybe just "history and sync"?

Good point. @amegantz Can we just say "Sync status" in the screen title and side menu? History is implied.


Update module github.com/hetznercloud/hcloud-go/v2 to v2.11.0

This PR contains the following updates:

Package
Type
Update
Change

github.com/hetznercloud/hcloud-go/v2
require
minor
v2.10.2 -> v2.11.0

Release Notes

hetznercloud/hcloud-go (github.com/hetznercloud/hcloud-go/v2)
v2.11.0
Compare Source
Features

add truncated exponential backoff with full jitter (#459) (fd1f46c)
allow configuring retry options (#488) (2db9575)
exp: add sliceutil package (#489) (f4ad6bc)
exp: rename *utils package to *util (#487) (19da475)
respect cancelled contexts during retry sleep (#470) (756f605)
retry requests when the api gateway errors (#470) (756f605)
retry requests when the network timed out (#470) (756f605)
retry requests when the rate limit was reached (#470) (756f605)

Bug Fixes

exp: set capacity for each batch (#490) (57f53c1)

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Enabled.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about this update again.

[ ] If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot.

ℹ Artifact update notice
File name: go.mod
In order to perform the update(s) described in the table above, Renovate ran the go get command, which resulted in the following additional change(s):

2 additional dependencies were updated

Details:

Package
Change

golang.org/x/net
v0.26.0 -> v0.27.0

golang.org/x/sys
v0.21.0 -> v0.22.0


Duplicated cache due to concurrency

Misc
When running multiple runs at the same time, the cache is duplicated due to concurrency, even with the same key and for the same branch:

Version: cache@v4
Step:
    - name: Cache
      uses: actions/cache@v4
      id: cache
      with:
        key: venv-${{ runner.os }}-${{ runner.arch }}-pip-${{ hashFiles(format('acceptance/{0}', inputs.file)) }}
        path: ${{ steps.cache-dir.outputs.dir }}
        restore-keys: ${{ runner.os }}-${{ runner.arch }}-pip-

I am facing same issue with the following configuration
  uses: actions/cache@v4
        id: node_modules
        with:
          path: |
            **/node_modules
          key: ${{ runner.os }}-node_modules-${{ hashFiles('**/yarn.lock') }}-${{ steps.node.outputs.version }}

I am facing same issue with the following configuration
  uses: actions/cache@v4
        id: node_modules
        with:
          path: |
            **/node_modules
          key: ${{ runner.os }}-node_modules-${{ hashFiles('**/yarn.lock') }}-${{ steps.node.outputs.version }}

You have one for each ref. I think that this is the expected result. The problem is when a cache is duplicated for the same hash and the same ref.

I am facing same issue with the following configuration
  uses: actions/cache@v4
        id: node_modules
        with:
          path: |
            **/node_modules
          key: ${{ runner.os }}-node_modules-${{ hashFiles('**/yarn.lock') }}-${{ steps.node.outputs.version }}

You have one for each ref. I think that this is the expected result. The problem is when a cache is duplicated for the same hash and the same ref.

Ah sorry you are right, I was expecting the case to be used if the key is same regardless of the ref

More examples:


Batchnorm forward and backward optimizations

Batchnorm forward and backward optimizations
Performance improvements are available in the attached xls sheet..
BN_configs_opt_bwd.xlsx
BN_configs_opt_fwd.xlsx

@muralinr Looks like for small H*W cases the routing change to variant 0 works out much better.
Can you check where this config lands?

MIOpenDriver   bnormfp16 -m 1 -s 1 -r 1 -n 256 -c 256 -H 56 -W 56 -V 0 -F 1 -b 0 -i 100 -t 1   -w 0
1.785267
1.960407
-0.17514
-9.8%

@muralinr Looks like for small H*W cases the routing change to variant 0 works out much better.
Can you check where this config lands?
MIOpenDriver bnormfp16 -m 1 -s 1 -r 1 -n 256 -c 256 -H 56 -W 56 -V 0 -F 1 -b 0 -i 100 -t 1 -w 0	1.785267	1.960407	-0.17514	-9.8%

yes Daniel.  Variant#0 works out much better for small H*W cases.  Above config uses variant#1. Negative performance drop is due to jitter.  My changes would not impact varaint#1.  I ran it again and see it better performance of 1.73 ms/1.77 ms instead of 1.96 ms.
root@miopen908-1:/testx/MIOpen/build# ./bin/MIOpenDriver bnormfp16 -m 1 -s 1 -r 1 -n 256 -c 256 -H 56 -W 56 -V 0 -F 1 -b 0 -i 100 -t 1 -w 0
MIOpenDriver bnormfp16 -m 1 -s 1 -r 1 -n 256 -c 256 -H 56 -W 56 -V 0 -F 1 -b 0 -i 100 -t 1 -w 0
GPU Kernel Min Time Forward Batch Normalization Elapsed: 1.724973 ms
GPU Kernel Avg Time Forward Batch Normalization Elapsed: 1.737368 ms, for 99 iterations.
stats: bnormf, 0, 411042816, 411042816, 0, 714.868288, 1.724973
root@miopen908-1:/testx/MIOpen/build# ./bin/MIOpenDriver bnormfp16 -m 1 -s 1 -r 1 -n 256 -c 256 -H 56 -W 56 -V 0 -F 1 -b 0 -i 100 -t 1 -w 0
MIOpenDriver bnormfp16 -m 1 -s 1 -r 1 -n 256 -c 256 -H 56 -W 56 -V 0 -F 1 -b 0 -i 100 -t 1 -w 0
GPU Kernel Min Time Forward Batch Normalization Elapsed: 1.765760 ms
GPU Kernel Avg Time Forward Batch Normalization Elapsed: 1.774880 ms, for 99 iterations.
stats: bnormf, 0, 411042816, 411042816, 0, 698.355648, 1.765760


Restore editor screenshot.

Fixes https://github.com/godotengine/godot/issues/45898
Fixes https://github.com/godotengine/godot/issues/44811

Does this still require salvaging? Taking a screenshot works flawlessly to me.
Indeed, https://github.com/godotengine/godot/issues/44811 was fixed by https://github.com/godotengine/godot/pull/54427.