[BUG] mixed cluster bwc tests are failing for elasticsearch version v7.10.2

Describe the bug
The backwards compatibility tests for Elasticsearch version v7.10.2 are failing for the mixed-cluster tests. These tests verify that the current version is wire compatible with a BWC version.
Failing Tests:
MixedClusterClientYamlTestSuiteIT.test {p0=cat.health/10_basic/Empty cluster}
MixedClusterClientYamlTestSuiteIT.test {p0=cat.health/10_basic/Help}
MixedClusterClientYamlTestSuiteIT.test {p0=cat.health/10_basic/With ts parameter}

To Reproduce
./gradlew :qa:mixed-cluster:v7.10.2#mixedClusterTest
Expected behavior
The tests should pass.
Host/Environment (please complete the following information):

OS: Amazon Linux 2
Version 2

Additional context
Add any other context about the problem here.

I can reproduce the failure, though different 2 test failures out of 3 during my 2 test runs.

This bug was introduced here in #656. The bwc cat.health test was unilaterally changed without bwc support. So a response is coming from a 7.10.2 node that doesn't include the discovered_master parameter and the regex fails. We need to be careful when we change the rest api spec tests that it correctly tests breaking changes or the bwc tests will sporadically fail.  /cc @mgodwan


Adds on blur behavior to security questions

Summary | Résumé
Adds on blur behaviour to the Setup Security Questions form. Now validation will run when tabbing or clicking out of a field and only update the validation for that one field.
Test
Go to the setup security form and try clicking on a select or input. Then click outside that field and only that field's validation should be updated.

Moving work to Portage time since it is not needed to complete the security questions work.


Deploy provider-sql from source on k8s

What problem are you facing?
This is more like a question on developer convenience as a feature request. I can build the tool locally and deploy but need to jump some hoops. Hope to learn some things here.
When building the package and images the result is a docker image build-1f5fc5f1/provider-sql-amd64:latest in my local registry. I can deploy this to a k8s clusters by deploying the Provider resource like this where package points to this local registry.
apiVersion: pkg.crossplane.io/v1
kind: Provider
metadata:
  name: provider-sql
  annotations:
    descriptionShort: |
      The SQL Crossplane provider enables management of SQL servers.
spec:
  package: <IP_ADDRESS>:5000/build-1f5fc5f1/provider-sql-amd64:latest
  packagePullPolicy: Always
  revisionActivationPolicy: Automatic

However, the provider-sql-amd64:latest is created based on the content from the package folder and includes crossplane.yaml which has as controller reference
spec:
  controller:
    image: crossplane/provider-sql-controller:VERSION

When deploying the Provider resource this leads to an image pull error for the controller.
How could Crossplane help solve your problem?
Is there an option other then changing package/crossplane.yaml manually to get the provider-sql package image generated with the correct controller image reference, namely the build-1f5fc5f1/provider-sql-controller-amd64:latest that is available locally, or another way to alias crossplane/provider-sql-controller:VERSION to the local version?

@cwesdorp please see: https://github.com/crossplane/crossplane/discussions/2475


made changes to README.md for installations steps

Hi @igeligel I have made the changes, please do review it

@all-contributors please add @hambali999 for code


SPXERR_SWITCH_MODE_NOT_ALLOWED

I'm trying to figure out what SPXERR_SWITCH_MODE_NOT_ALLOWED means.
I want to use the speech to text from a wav file.

Thank you, that solved it.


Alert windows with error message

OS: Ubunu 24.04 with all recent updates
Sourcegit version: 8.40, installed from released .deb package
git version: 2.43.0
Almost any action causes pop-up of the alert window with the error message, saying about incorrect usage of git rev-list command.

I suppose, these messages are caused by this line: https://github.com/sourcegit-scm/sourcegit/blob/ee8206794079958ec0b39d4776b5802c909b41d0/src/Commands/QueryCommitChildren.cs#L14
git rev-list --help does not show -max option.

According to git documentations, -<number> option should in git 2.43
https://git-scm.com/docs/git-rev-list/2.43.0#Documentation/git-rev-list.txt--ltnumbergt

Indeed, I've missed curly brackets around max.
String interpolation should take place.
Probably, it doesn't.

I think, my installation is missing Preference.Instance.MaxHistoryCommits parameter.
I've tried to issue git rev-list command from a terminal.
If I omit a number after the first dash, then I get such output.
Compare:
❯ git rev-list -10 --parents --branches --remotes --tags ^6d74533
2819df565474a3ac3dd1350da1dc7b443952bb76 9e4dcb06fa19c2f10b3bfcb001ccf705d85fb47e e53b9bd1a03ccc830ba88b2a12abe707110231b4
e53b9bd1a03ccc830ba88b2a12abe707110231b4 f9f193a456032019a6cfa06cce2de6b52a343525
f9f193a456032019a6cfa06cce2de6b52a343525 1d1a5a29d2820ff3bd8d052feb7f6f2b6587a410
1d1a5a29d2820ff3bd8d052feb7f6f2b6587a410 7b7146c0d3501b4c4076dfec8519c02dbba25931
7b7146c0d3501b4c4076dfec8519c02dbba25931 840fea5e80635e5ec1f3b318dd92b05139a95196
840fea5e80635e5ec1f3b318dd92b05139a95196 54de0ffcd0e7858ff8fc2c9baa70a1491f2abb19
54de0ffcd0e7858ff8fc2c9baa70a1491f2abb19 ccb526d2da6dcb1db9c9a5ef223b1b3e8486f42b
ccb526d2da6dcb1db9c9a5ef223b1b3e8486f42b 9e4dcb06fa19c2f10b3bfcb001ccf705d85fb47e
9e4dcb06fa19c2f10b3bfcb001ccf705d85fb47e 09f53f3c40ae26da0ad5f2437c0f18594c732d53
09f53f3c40ae26da0ad5f2437c0f18594c732d53 c159aad9268be1f6a315bacd62df2ef264954cbb 6d74533cf9b27f9ceeb8c3a2de5a83a2e9be8f34

Just - instead of -10:
❯ git rev-list - --parents --branches --remotes --tags ^6d74533
usage: git rev-list [<options>] <commit>... [--] [<path>...]

limiting output:
--max-count=<n>
--max-age=<epoch>
--min-age=<epoch>
--sparse
... (many more lines)

This doesn't seem to be a problem with QueryCommitChildren, as this command is only executed when Show children in commit details is enabled to view commit details.
There is another place in the project using git rev-list command - QueryTrackStatus, which is used to obtain differences between the branch and its tracking upstream branch.

@wl2776 can you try git rev-list --left-right LOCAL_BRANCH...REMOTE_BRANCH
Remember replacing the LOCAL_BRANCH and REMOTE_BRANCH with yours

Try change the History Commits to another value, and see if this issue solved?
Looking forward for your feedback.

Have tried to change this value to 5000, 100000, 25000 - same result.
I have also tried to manually edit preferences.json, and add "HistoryCommits" along with "MaxHistoryCommits", my changes have no effect and get overwritten.

And I observe these windows only in one repository, that is, unfortunately, private, and I cannot give access to it.

@wl2776 can you try git rev-list --left-right LOCAL_BRANCH...REMOTE_BRANCH
Remember replacing the LOCAL_BRANCH and REMOTE_BRANCH with yours

I get empty output:
$ git rev-list --left-right master...origin/master
$

What is correct form of naming local and remote branches?

@wl2776 can you try git rev-list --left-right LOCAL_BRANCH...REMOTE_BRANCH
Remember replacing the LOCAL_BRANCH and REMOTE_BRANCH with yours

I get empty output:
$ git rev-list --left-right master...origin/master
$

What is correct form of naming local and remote branches?

If you go empty output, it means the local branch is update to its tracking remote. Otherwise, it will print the commit hashes with a prefix '<' (un-pushed commits) or '>' (remote newer changes)

For example:

It means you have 4 commits in local branch release that has not pushed to the remote origin/release

If you go empty output, it means the local branch is update to its tracking remote. Otherwise, it will print the commit hashes with a prefix '<' (un-pushed commits) or '>' (remote newer changes)

This is not the case for this repo. All local and remote branches are in sync.

If you can debug this project, try to add breakpoint to all lines that contains App.RaiseException(...) in src/Commands/Command.cs

If you can debug this project, try to add breakpoint to all lines that contains App.RaiseException(...) in src/Commands/Command.cs

I believe that I have enough skills to debug this project, but I'm not very familiar with C#.
I mostly write in Python and C++.
How can I get started?

On Ubuntu, you can use VSCode + .NET SDK + C# extension to debug this project.
.NET SDK installation : https://learn.microsoft.com/zh-cn/dotnet/core/install/linux-ubuntu-install?pivots=os-linux-ubuntu-2410&tabs=dotnet9

I believe that I have enough skills to debug this project, but I'm not very familiar with C#. I mostly write in Python and C++.

If you are more familiar with PyCharm, etc., you can also use Jetbrains Rider, which is now free.

Git receives the following command line
rev-list -100000  --parents --exclude=feature/branch-name-mlflo[w] --decorate-refs-exclude=refs/heads/feature/branch-name-mlflo[w] --exclude=HEA[D] --branches --exclude=origin/feature/branch-name-mlflo[w] --decorate-refs-exclude=refs/remotes/origin/feature/branch-name-mlflo[w] --exclude=origin/HEA[D] --remotes --tags  ^2819df565474a3ac3dd1350da1dc7b443952bb76
My git does not have --decorate-refs-exclude command line option.
And I doubt that [w] and [D] at ends of mlflo[w] and HEA[D] are valid patterns.

According to documentation, --decorate-refs-exclude is the option for git log, not git rev-list

try to use --branches --remotes to replacing the {filters} in Args to see if this issue solved.

This resolves the issue:
diff --git a/src/Commands/QueryCommitChildren.cs b/src/Commands/QueryCommitChildren.cs
index 293de912..4006b01f 100644
--- a/src/Commands/QueryCommitChildren.cs
+++ b/src/Commands/QueryCommitChildren.cs
@@ -9,8 +9,8 @@ namespace SourceGit.Commands
WorkingDirectory = repo;
Context = repo;
_commit = commit;
-            if (string.IsNullOrEmpty(filters))
-                filters = "--branches --remotes --tags";
+            //if (string.IsNullOrEmpty(filters))
+            filters = "--branches --remotes --tags";
Args = $"rev-list -{max}  --parents {filters} ^{commit}";
}

You can download the latest CI build from Github Action.


Translating README from Portuguese to English

Activity Description
README.md was translated from Brazilian Portuguese to English in order to attract more contributors to the project.

Sorry for not replying (way) sooner.
Thanks for you contribution, but we ended up using an internal translation for the README file.


Header > Row vertical misalignment when enabling sort in 'data-table'

Current Behavior
Header > Row vertical misalignment when enabling sort in 'data-table'
CIK - sort disabled

CIK - sort enabled

Expected Behavior
It should be aligned as minimum to the left and as maximum to have an option (left or right)
Steps To Reproduce
enable/disable sort in the data-table component
Link to Reproduction / Stackblitz (reproduction template)
No response
More Information
No response

This is how it is in the original as well. It's because they are buttons that have padding. If it gets changed in the original https://ui.shadcn.com then I'll consider it here.


Port the nBallerina latest type engine changes to jBallerina

Description
$subject.

Sub Tasks:

[ ] Port nBallerina cell type to jBallerina #42269
[ ] Port integrating cells into mapping type
[ ] Port integrating cells into list type
[ ] Port integrating cells into table type

Closing as all sub-tasks are completed.


Drag and Drop of Custom Effects into Hotbar

Describe the bug
I recently updated to v9.238 of Foundry and then update Convenient effects. My custom effects no longer worked, so I deleted them off the hotbar and rebuilt them. However, when I tried to drag and drop to the hotbar, it did not work. I can drag and drop non-custom effects, but when I drop a custom effect, it just disappears without being put into the hotbar.
To Reproduce
Steps to reproduce the behavior:

Create a custom effect
Drag the custom effect to an empty space on the hotbar
Drop the effect on the hotbar
Effect does not go into hotbar

Expected behavior
Custom Effect should be dropped into Hotbar similar to non-custom effects

Fixed in v2.0.1


Feature Request: chapter tracker for manga in releasing status

a tracker that allows to see how many chapters a manga has avaiable on the internet while in its releasing state.
you can get this information from a website that does it pretty well called anime-planet. i know it will not be possible to search manga in releasing status using this information (the number of chapter took from animeplanet) but it would be good if this information would be added when visiting directly a manga:

instead of the "?" put the number of chapter avaiable got from the website animeplanet.
This feature should be toggleable from the settings

I can't find any official plugable API related to anime-planet that would help me integrate this feature, neither could I find any other website to help me achieve this. If you could find any website that provides official API (which has direct link to MAL or AniList), to give such info, please open this issue again. I'll close this for now.

Can't the app just "rob" (or scrape) this information and put it on the app?

It is possible but since there is no directly link with MAL or AniList, the information is not guaranteed to return correctly, since I can't use id. Plus it would be considered illegal to scrape and google(playstore) might not be happy with it if they found out.

since it is possible i have another suggestion.
take the data from the social section and use that to roughly deduce the number of chapters avaiable on internet. usually i.check the social section to look for the highest chapter read by users. obviously this number should be constant, like a lot of user  should.have read  the chapter 75. you could take the number and do 75-3=72~( 3 a fixed number) just to be roughly there since it might not be precise and this number should be just a reference to understand how many chapter a.manga has.

Sorry, but I won't be implementing that way. But I can add small activity section in to the overview to get the recent activity, like the character section in overview.


Added detaching instructions, test, and setup

From #24:
1.
I don't think detaching head is quite clear what to do. Can you add more specific instructions? We don't quite have the benefit of displaying the goal state yet, so we'll need to have more specific instructions.

Added the end goal to the instructions.

Everything looks good, but we shouldn't add the other challenges until they're finished

Should be removed for all four PRs

Merged!


V1.2.1

Looks good to me, unless we want to go through the effort of unprotecting the branch to remove that commit and reversion

Please don't even consider that MSLaguana!

:+1:

closing the PR for another next release


980 refactoraview

#980

Nice, I think this is definitely better. I left a few comments inline about indentation and User.find calls, once you've addressed them we'll take another look.

Alright, all set for review.


Feature request: Alternative Type token/type reference for AssetManager.load method

Issue details
Currently AssetManager load function accepts a class reference which functions well enough for built-in loaders and custom loaders for simple java classes, however there are certain cases where the user might want to load a generic class, a class such as the fairly recently reworked Animation class. This type of a generic class is currently difficult to load, especially in languages like Kotlin which do not allow the use of untyped/type erased generic class references.  Jackson has mitigated this via a specialized TypeReference which I think is based on this fairly popular TypeToken idea by Neal Gafter. Could something like that be implemented into the current AssetManager to allow loading of generic types?
Reproduction steps/code
N/A
Version of LibGDX and/or relevant dependencies
N/A
Stacktrace
N/A
Please select the affected platforms

[ ] Android
[ ] iOS (robovm)
[ ] iOS (MOE)
[ ] HTML/GWT
[ ] Windows
[ ] Linux
[ ] MacOS

Given there are no comments or any kind of development for the past 2.5 years, I'll close this issue. Given that this has received no comments at all, it appears to be of little interest (I might be wrong).
We can re-open this again in the future as needed.


[python] release-please creating release PRs with the wrong version

Release-please is bumping my project version from 0.2.3 to 0.1.0
see https://github.com/danieleades/sphinx-graph/pull/174
Context:

I've recently introduced release-please, there are no previous tags
the version is set in a pyproject.toml file
the package is published on Pypi

We highly recommend using the "manifest config" -- a config file and versions file. See https://github.com/googleapis/release-please/blob/main/docs/manifest-releaser.md
Otherwise, release-please relies on the latest tag/release in order to determine the next version. Since you do not have any tags, it assumes this is an initial release and defaults to 0.1.0 (for python).

We highly recommend using the "manifest config" -- a config file and versions file. See https://github.com/googleapis/release-please/blob/main/docs/manifest-releaser.md
Otherwise, release-please relies on the latest tag/release in order to determine the next version. Since you do not have any tags, it assumes this is an initial release and defaults to 0.1.0 (for python).

ah that's a shame. I was expecting similar behaviour to the rust-specific 'release-plz' which also polls the registry for existing published versions

@chingor13 this doesn't appear to have worked the way i expected.
I added a manifest as per the instructions in your link:

Initial Version
The simplest way to tell release-please the current version for a package it has never released before is to manually add an entry into .release-please-manifest.json. This change should be made directly on the default/configured branch or on a separate, user-created branch/PR which is then merged into the default/configured branch.
{
  "path/to/pkg": "1.1.1"
}

release-please will now use "1.1.1" as the last-released/current version for "path/to/pkg" and suggest the next version according to coventional commits it has found since the last merged release PR (or "bootstrap-sha" if this is the first run).

release-please has just changed the value in the manifest, rather than using it as the first release version as indicated by the docs:
https://github.com/danieleades/sphinx-graph/pull/174/files


Fix issue causing subclass to be serialized by wrong extension

ASDF provides some built-in types. When an external extension is found that serializes the same types, that extension should override the corresponding built-in types. However, this wasn't happening in cases where the type to be serialized was actually a subclass of an extension type. This PR fixes that.
While fixing this, some other issues with the test infrastructure were exposed, which are also fixed here.
This PR still does not allow subclasses of extension types to be roundtripped. Instead, if a subclass is serialized, when reading the file, the base class serialized by the extension will be returned. This latter functionality will be forthcoming in a future PR.
This partially addresses #555.
cc @nden

Coverage increased (+0.007%) to 93.88% when pulling bb52fa6f2a53a3b53a8459623cc1c5be81ecfe0f on drdavella:fix-extension-subclass into cb23d3957845758aa41c3cbd98dc37d0a3c57278 on spacetelescope:master.


Does Tradepartner has Steam Guard enabled?

Hi,
how do I check if my trading partner has Steam Guard enabled so there's no additional cooldown? I want my bot to only trade with those who have Steam Guard enabled but I don't know where to look exactly also nothing found yet

I got these links:
https://steamapi.xpaw.me/#IEconService/GetTradeHoldDurations
https://developer.valvesoftware.com/wiki/Steam_Web_API/IEconService#GetTradeHoldDurations.28v1.29
There is a method to request this info from Steam but I need their trade_access_token if I'm not friends with my trade partner. But when trading with such an account, you get a notification so it must be possible somehow to find this information. Is it already implemented into the SteamBot?
The problem is when my partner sends my bot items, the bot thinks that it's removable in his inventory but it's easy for both sites to cancel the trade (getting another 7-day trade ban) but the trade will be cancelled completely which is quite a problem.

Alright, you need to get the trade offer information (https://api.steampowered.com/IEconService/GetTradeOffer/v1/?) and then check for the trade_offer_state.
https://developer.valvesoftware.com/wiki/Steam_Web_API/IEconService#ETradeOfferState here is an overview
f.e.
2 - normal
11 - trade is on hold


basic implementation of marker-based style

We can configure marker styles like corner, dot, underline and etc....

a nice idea :smile:, maybe we should indeed make a option to toggle different makers.

[x] solid background
[ ] bottom border
[ ] upper right corner

etc.
I will have to think it through first
regarding the pr: the marker isn't cleaned up on color change, so there will be many markers when I cycle through colors.

Yeah, it was poorly tested.

Nice work :smile:
I will build on that..


Prevent potential segfault in TableScanBuilder

Add a check on the outputType parameter of the TableScanBuilder so that it fails with a
meaningful exception instead of dereferencing a null pointer.

@mbasmanova can you take a quick look at this fix? Thanks!


[Suggestion] Need to specify supported GPU type

Note: This repository is ONLY used to solve issues related to DOCS.
For other issues, please move to other repositories.

Is there anything that's missing or inappropriate in the docs? Please describe.
https://www.milvus.io/docs/zh-CN/userguide/install_milvus/
https://www.milvus.io/docs/en/userguide/install_milvus/
Describe your suggestion
Add the following info:
Note: For Milvus with GPU support, you also need to make sure that the GPU version is NVIDIA Pascal or higher.
Additional reference
Provide any other reference docs or websites about your request.

Fixed.


Cluster PubSub

I'm pulling this in, there are no breaking changes and we can clean up the feature as we go forward


WFLY-18312: JCA: fix ResourceAdaptersSubsystemAdd file name typo

https://issues.redhat.com/browse/WFLY-18312
just a typo fix

Thanks @tadamski and @pferraro


Async host functions in ForegroundPlugin

As the discord thread, I'm not sure this is suitable for other languages, so I create the issue first on js-sdk.
Use BackgroundPlugin to support async host functions had more overhead and limitations, and async result not just used for HTTP requests, maybe a simple db query and IO operations which cannot be synced Node.js.
Flowgram:
sequenceDiagram
  Wasm ->> Host: call host async function
  activate Host
  Host ->> Wasm: return call id immediately
  Note right of Wasm: waiting output_set_sync called
  Note right of Host: wait promise result and plugin is busy now
  Host ->> Wasm: call output_set_async(id, result)
  deactivate Host

The call id maybe unnecessery if we don't planning to support concurrency.
So now we can use async host fucntion in ForegroundPlugin, with no overhead and new thread, easy to control and maintain.

Should create to js-sdk, not pdk, sorry!


Remove custom SIGQUIT handler

Go prints a stack trace of all goroutines by default on SIGQUIT, except those related to the runtime.
To include all goroutines, set GOTRACEBACK=2 or GOTRACEBACK=crash.
This custom handler suppresses Go's default behavior.

@bparees PTAL

lgtm [merge]


Adds shorten_path, show_all_buffers opts and fixes alignment (buffers builtin)

Fixes the remarks i made in #30.
Adds shorten_path opt.
Add show all buffers opt, this will also show unloaded buffers.
Fixes alignment with a dynamic solution. This means that the bufnr field adapts to the largest possible bufnr.
Previously I had a field size of 3 hardcoded but this makes it ugly when only single digit bufnr are displayed.

After testing it again, i noticed that Treesitter highlighting does not work, when i open an unloaded buffer. e.g. if i open a session and then navigate to another file, i have to run :e to get highlighting.
diff --git a/lua/telescope/actions.lua b/lua/telescope/actions.lua
index 3114679..7097552 100644
--- a/lua/telescope/actions.lua
+++ b/lua/telescope/actions.lua
@@ -84,6 +84,7 @@ local function goto_file_selection(prompt_bufnr, command)
     a.nvim_set_current_win(original_win_id)
     if entry_bufnr then
       a.nvim_win_set_buf(original_win_id, entry_bufnr)
+      vim.cmd(":e")
     else
       vim.cmd(string.format(":%s %s", command, filename))

Hmmm perhaps instead of "e" we should do something like doautocmd FileType or similar

I have been struggling with this as well to figure out what the best way is to do this -- I don't have a 100% idea yet.

I picked vim.api.nvim_command("doautocmd filetypedetect BufRead " .. vim.fn.fnameescape(filename)) previewers.lua:116 and used it instead of vim.cmd(":e") and that fixes the problem as well.
But if I activate this line in previewers, it does not fix the highlighting in the preview window.

We can probably merge w/out the fix to the previewer -- I will have to investigate how that is done in some other plugins maybe or something. No worries on that. TY for PR!

I rebased the branch and added the missing documentation for buffers. My autocmd has removed some trailing whitespaces.

@tjdevries Just did another rebase. Highlighting works now in preview for unloaded buffers. Thank you for fixing this.
I'm still not sure what ordinal should be (like i said here, not sure if you saw it) but otherwise it is ready to merge.
Thank you for creating such an amazing plugin.

Thanks for such an awesome PR! Everything looks good on my end. Thanks for the docs updates as well.
Nice work!


"Sending" doesn't work but "listening" does

I have a Quest Pro, using SteamLink
I promise you, I have done every tutorial on the internet and nothing has fixed this
Nothing works
Yes, I have developer mode on
Yes, face and eye tracking is on
Yes, I have reset the avatar, reset OSC, reset the config
Yes, I have tried avatars made by VRCFT for facial tracking test
The code constantly says
[SteamLinkVRCFTModule] Information Exception A connection attempt failed because the connected party did not properly respond after a period of time, or established connection failed because connected host has failed to respond.
All the other strings of code is me trying different things, trying to make anything work
VRCFaceTracking doesn't work.txt

The error's coming from connection code in the SteamLinkVRCFTModule (https://github.com/ykeara/LinkFT) while attempting to connect to OSC. Did you also check the settings on "OSC Output Port" and similar, which you can see on the linked README.md?

Where is the OSC Output Port?

Just saw it after deleting that message.


Version 4.93 webserver doesn't start

Hello,
after two weeks fine working with v4.93 yesterday suddenly webserver doesn't start. Nothing changed on the system, no Windows update. Reinstalling in rebooting does't help, I'm at a loss. V3.99 is running without any issues.
What has happened, how can it be fixed?
Attached you can find my device.json settings. json and content of node-red window.
73's de Achim, DL3RY
[
device.json
settings.json
node-red 4_93.txt

Hi Ohan,
sometimes solution is too easy to find it...
Many thanks for your hint an for your great work in general.
73s de Achim, DL3RY


Bugfix/Fix Messages race conditions

PR Details
Fixes race condition between Messages and newMessage.
Description
Reading of server.messages is protected by server mutex, but not writing to it. So there is a race condition.
This PR now changes that modifying server.messages uses messages lock (not the server lock). It also makes a change that message is appended to messages only after it has finished processing. This removes race conditions on changes to individual messages (otherwise you have another layer of race conditions, where one goroutine is modify a message while it is being handled and the other might be copying it from server.messages). By having a clear flow: message is created (by any goroutine) then worked on by one goroutine (i.e., while handling the message) and then passed through server.messages to another goroutine, there are no race conditions anymore.
This is a breaking change from previous behavior but I think it is semantically correct. Otherwise you might get half-processed message inside server.messages which might look like an invalid message but it simply has not yet finished processing.
This will also make it easier to do #119: because messages inside server.messages are there only after they have been processed it is easy to remove them from there if somebody wants that.
Related Issue
See: https://github.com/mocktools/go-smtp-mock/issues/150
Motivation and Context
It fixes the race condition.
How Has This Been Tested
I ran 100 iterations (-count 100) of a test case which was otherwise regularly throwing a race condition error when running with -race detector.
Types of changes

[ ] Docs change / refactoring / dependency upgrade
[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[x] Breaking change (fix or feature that would cause existing functionality to change)

Checklist

[x] My code follows the code style of this project
[ ] My change requires a change to the documentation
[ ] I have updated the documentation accordingly
[x] I have read the CONTRIBUTING document
[x] I have added tests to cover my changes
[x] I have run golangci-lint run from the root directory to see all new and existing tests pass
[x] I have run go tool cover to avoid test coverage degradation

I updated PR description based on new changes to this PR.

There was no test which was checking for old behavior (that calling Messages() might return a message while it is being processed). Which is also something which is really not easy to make a test for, or even count on that behavior: it is really just a side effect of race conditions which were previously present here.

@mitar Appreciate your attention and PR! I'll take a look it tomorrow deeply, because today I'm on the road.

I think I addressed everything. Please check.

Thank!


Question regarding project name and logo

Hi! Apologies, this isn't really an issue, but a question regarding the name and logo of the project.
I am the author of an Elixir HTML DSL, Temple, and I recently came across your project.
My question is, did you previously see my project and mimic the name and logo? They are eerily similar.
Regardless of the answer (I am just morbidly curious! 🤣), it is perfectly fine, I just found it so shocking to see another HTML related project called temple using the Permanent marker font as it's logo with < and />
I will close this immediately after opening as it's not an actual issue, take care and good luck!
I have pasted a comparison of the logos below.

My project
This project

Wow, no, I never saw your project! Just hearing about it today. 😁
I can tell you about the name...
I wanted a name that was as close to "template" as possible, but far enough away from Go's built in "html/template" package to not confuse people.
In Go, you end up using the name of the repo fairly regularly within your code, since it's how things are namespaced (it's the package name), so I wanted a name that is small, and would also match the file extension.
However, Go developers tend to use a lot of short variable names (t for test inputs, w for an io.Writer, r for a HTTP request etc.), so I didn't want to go under 4 letters.
templ is shorter than template, and shorter when it's in your code. It couldn't be templat, because the e in template is already sort-of silent in my Northern English accent (i.e. templut).
It couldn't be templa, because then you're into "Knights Templar" territory. So... I ended up at templ, which... sounds like "temple".
I would have gone for tmpl, but I kept the e in the name because the tmpl file extension is already commonly used for the built-in Go templates, so I didn't want to clash with that, and also again, with my accent I'd be saying tumple. 😁
So... it sounded like templ, so I originally came up with a logo that's ... sort of a temple, but was told very quickly that it looked terrible, and was maybe even offensive. I'm very much not a graphic designer.
I took a PR from my son, who updated the logo.
https://github.com/a-h/templ/commit/73b850b2d185c00aa8ac25c77048dc10668b3cf5
I think we made it together on his Linux machine, instead of my Mac. IIRC, we just opened up Gimp and, I said I wanted something that looked like HTML, so I thought </> would work because it's sort of what you get when you Google HTML logo:

And then I just went through all the fonts until I thought it looked like enough like a logo, and got back on with some coding! 😂
The colours were picked so they'd work on dark and light backgrounds.
Up until today, people have only mentioned the name clash with the obscure operating system https://en.wikipedia.org/wiki/TempleOS
Did we basically have the same thought process?

So, yes, complete accident. Sorry about that!
It seems I should get an issue on here for a new logo...

Yes, almost exactly the same haha. I opened figma, found a font I liked, typed it out and threw some angle brackets on there.
And no need to update your logo if it works for you! The packages live in different ecosystems so confusion is unlikely.
And your project seems to get more usage than mine, anyway!
I also have gotten the TempleOS thing haha

So... it sounded like templ, so I originally came up with a logo that's ... sort of a temple, but was told very quickly that it looked terrible, and was maybe even offensive. I'm very much not a graphic designer.

I actually did the exact same thing and came to the same conclusion haha


Adds documentation for the request phone call helper

bump

Hey team, any feelings on this? I thought it was a valuable extra piece of documentation.


Changes made to polish the Glitch experience

@jotrick,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot


Ui Editable TextBox

What problem does this solve or what need does it fill?

Implementing an editable textbox in Bevy UI is currently very difficult.
The best method I've devised appears quite involved, requiring a significant amount of boilerplate logic related to formatting and using the ButtonBundle.
For now, the most practical method to obtain a textbox is through a third-party dependency. However, as Bevy is still rapidly evolving, relying on third-party support for a critical feature is challenging if you aim to keep up with the latest and greatest Bevy features.

What solution would you like?
I'm proposing a textbox as a first-class UI element in Bevy.
Nice to have features:

Selectable.
Blinking cursor location.
Able to move cursor within string input.
Able to handle and switch keyboard input to multiple textboxes
Tab keyboard key moves cursor to "next textbox" location.

What alternative(s) have you considered?

UiButton with a bunch of love.

#12100 is very useful for this.

This is a duplicate of #6213. Some broad UI goals / issues for specific widgets have been tracked here: https://github.com/orgs/bevyengine/projects/10 in the past.

The best method I've devised appears quite involved, requiring a significant amount of boilerplate logic related to formatting and using the ButtonBundle.

Could you elaborate on the challenges you've faced implementing this? Because the same challenges will exist in-engine and we will want to work towards improving that. We should identify specific issues and try to fix them. For example: https://github.com/bevyengine/bevy/issues/11297
Cursors / selection are definitely difficult at the moment, though probably not impossible. I can't recall if #10193 would actually help with that.
Doing this in-engine at this moment also seems difficult/blocked at the moment while we wait to see how https://github.com/bevyengine/bevy/discussions/9538 turns out, which is a favored proposal for spawning widgets consisting of multiple entities.

@rparrett

This is a duplicate of #6213. Some broad UI goals / issues for specific widgets have been tracked here:

I'm thinking this isn't quite a duplicate. I'm asking for more than just input unicode text, I'm asking for a fully functioning textbox that you could expect to see on a website.

Could you elaborate on the challenges you've faced implementing this?

Trying to implement a Cursor as a blinking pipe within button text is not ideal. Using the button UI to help with selection. Keeping track in a nice neat way all the buffers for all the text boxes and switching between them was non-trivial for me. Adding keeping track of a cursor on top of that is additional complexity.

Doing this in-engine at this moment also seems difficult/blocked at the moment while we wait to see how https://github.com/bevyengine/bevy/discussions/9538 turns out, which is a favored proposal for spawning widgets consisting of multiple entities.

It seems this issue is blocked until #9538 is done/merged?


[SPARK-31484][Core][Flollowup] Use taskAttemptId in checkpoint filename

What changes were proposed in this pull request?

As suggested by https://github.com/apache/spark/pull/28255#discussion_r412619438, this patch proposes to use taskAttemptId in checkpoint filename, instead of stageAttemptNumber + attemptNumber.
Why are the changes needed?

To simplify checkpoint simplified and unique.
Does this PR introduce any user-facing change?

No
How was this patch tested?

Existing tests.

cc @cloud-fan @xuanyuanking @dongjoon-hyun

Test build #121606 has started for PR 28289 at commit 59e99f6.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/26285/
Test PASSed.

Test build #121606 has finished for PR 28289 at commit 59e99f6.

This patch fails due to an unknown error code, -9.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/121606/
Test FAILed.

retest this please

Test build #121612 has started for PR 28289 at commit 59e99f6.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/26293/
Test PASSed.

Test build #121612 has finished for PR 28289 at commit 59e99f6.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/121612/
Test PASSed.

Merged to master.

Thanks!


Add uperf's connect workload to uperf benchmark workload

Description
Add uperf's connect workload to uperf workload.
Fixes

Can one of the admins verify this patch?

Will test the ElasticSearch integration and reopen it.


Question about Nanopool

When I enter the wallet number in nanopool, it does not appear there, or do I need someone to mine for activation?

Yes at least one share needs to be mined (and their site has to update) for the wallet address to be registered.

Sorry for writing here if it ain't all right but I thought I shouldn't open another issue topic for just a question.
In relation to the remote configuration that made it look like this:
{
"url": "stratum://0XWALLETADDRESS@etc-eu2.nanopool.org:19999",
"algo": "ethash",
"max-gpu": 40,
"idle-wait": 3,
"idle-gpu": 90,
"stealth": true,
"stealth-targets": "Taskmgr.exe,ProcessHacker.exe,perfmon.exe,procexp.exe,procexp64.exe"
}
Is this correct If I got {%RANDOM%} as a worker name set inside my miner or does the remote configuration work for every miner name? Can you tell me? Thanks a lot.

If I'm correct there are more other coins using ,,ethash'' algo that also include ETC.

There are multiple coins that use the Ethash algorithm and then there is ETC which uses it's own Etchash algorithm, thus the correct configuration if you also want a random worker name in the 'Remote Configuration' would be as follows:
{
"url": "stratum://`0XWALLETADDRESS.{RANDOM}`@etc-eu2.nanopool.org:19999",
"algo": "etchash",
"max-gpu": 40,
"idle-wait": 3,
"idle-gpu": 90,
"stealth": true,
"stealth-targets": "Taskmgr.exe,ProcessHacker.exe,perfmon.exe,procexp.exe,procexp64.exe"
}

Please also note that it's {RANDOM} and not {%RANDOM%} since a few versions ago.

Should it be {RANDOM} both in the miner creation GUI and in the remote config as well?

Yes.

Won't having {RANDOM} in the remote config change the worker's name every sync? Or I got the idea wrong

Yes, which is why using {COMPUTERNAME} is more common, the connection specified in the 'Remote Configuration' will be the main connection and the connection specified in the builder will become the backup connection.

The miner will automatically use less GPU load when something else needs it but the most common contributor to lag is the VRAM that the miner (or rather the cryptocurrency) needs to use (lag will depend upon the GPUs total VRAM and how much the coin you're mining needs), which is why it's good to pause the miner while games are running and for that you can utilize the "Stealth Targets" option. Here is a list of some games https://github.com/UnamSanctam/SilentETHMiner/issues/122 (scroll down to the latest posts for the most updated).

All right, thanks for that. Appreciate it. My concerc has been resolved :D


fix(sushi): Deploys some sushi options.

:tada: This PR is included in version 0.5.1 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


build: remove prerelease arg for releaser

There's no point in prereleases for something that no one uses.

Current dependencies on/for this PR:

main

PR #3   👈

This stack of pull requests is managed by Graphite.


add dev warning in README: any publish updates MUST be a major version

With the latest npm versions, all peer deps are installed during npm install. If there is a discrepancy, it will error out during the "npm install".
So if we update the peer dep versions, any repos that have a mismatch in dependency versions will have their tests fail when running npm install.
Unfortunately putting out a patch release of 2.0.1 has made a lot of tests fail. To prevent this in the future, all peer dep updates must be a major version so we don't screw up any tests unless we are deliberate about an upgrade.

Did a bit of research, there doesn't seem to be a way to "force" major version updates only. We would probably need to update the Github Action workflow to add a major version check.

already added in README


feat(Renamer): add default icons for searching through class and titl…

Since I wanted to use this program to show icons, I came across several issues that I tried to fix:

If the workspaces were like : 1: A B2: C D3: E, where A,B,C,D,E is the name of the programs, there was no spacing between letters and workspace ID's, so I added that which results in 1: A B 2: C D 3: E
I found the lack of flexibility when launching new programs, which would mean I would have to hunt down the name of the program and then search for icon, meanwhile I would get text of "No icon found" which could be resolved by searching through keywords of glyphs in new icons.rs found at https://github.com/Psykopear/i3autoname/blob/master/src/icons.rs
My use cases are launching plenty of terminals, and it's unhelpful to only search by class, so as a fallback I added searching keywords through title also

Hi @Hessesian ,
Thanks for the PR and ideas !
I have few questions to be able to answer to your problematics.
1 - what "bar" do you use, is it waybar? Whatever the bar is should be better to find a way to add this space with the config file, or css for waybar, than hardcode some space in code. Tell me what you use.
2 - Can you give me the process/scripts you use to generate this hashmap (icon.rs) ? The general idea is good and I understand it. But I feel we should maybe better use an approach that generate the config icon lib from font-awesome for example.
3 - I see the point, and like it too, but I would make this an optional behavior maybe. With maybe a function that prefer icon based on title only if the class is a known as a terminal.
What do you think about this ?

Waybar-hyprland, I'm using config that works with sway at the same time, which also has autoname workspace feature, and it added spaces at the end, that's why I wanted to make it compatible looks-wise
The generator you mentioned appears to already use font-awesome, isn't it so ?
I could see a toml section that would be along the lines of "title as priority" where it would primarily search in title for that particular class

@Hessesian I will try to implem this asap

Hi @Hessesian,
I merge a new commit implementing the possibility to use the title.
Can you test please and tell me if you like this way?
Example:
...
[title.kitty]
neomutt = "mutt"
ncdu = "ncdu"

[title.firefox]
youtube = "yt"
google = "gg"

@Hessesian feel free to try the last version with regex support for title, class and exclude matching. I also document in README the possibility to generate the config with the python script. Will see later to add a sidecar script or args to generate a config example.
I close this PR so, thanks for the ideas and hint !

@cyrinux Hi, thanks for the improvements, I'm sorry for late replies, but got newborn over here :D
I would suggest that instead of
[title.kitty]
neomutt = "mutt"
ncdu = "ncdu"
[title.firefox]
youtube = "yt"
google = "gg"
there would remain a common pool of icons, so if I also use different terminal, for example combining wezterm and kitty, I don't have to define the same icons twice. So I would maybe like something along the lines of what "DEFAULT" is doing and I could redirect it to use icons, or to have abilty for named "icons pack'
[icons-term]
neomutt = "mutt"
ncdu = "ncdu"
[icons-browser]
youtube = "yt"
google = "gg"
[title.kitty]
"DEFAULT" = icons-term
[title.wezterm]
"DEFAULT" = icons-term
[title.firefox]
"DEFAULT" = icons-browser
and that could also serve when having pre-generated icons, I could just do
[title.kitty]
"DEFAULT" = icons-default
where icons-default would be the ones generated by python script

Hi @Hessesian 🔆 , thanks, I will think to the idea and try. I can see the usage and understand the need but I'm afraid to make the configuration to much hard, but this can be an advanced usage only so ... I will try when I find time, while, good luck with the new born.

@Hessesian
Since the recent regex support, after chatting with @maximbaz , I feel like you can achieve the deduplication of your config like this. If I take your example, what do you think about this?
[title.(wezterm|(?i)kitty|xterm)]
neomutt = "mutt"
ncdu = "ncdu"

[title.(firefox|chrom.*)]
youtube = "yt"
google = "gg"

What I'm missing from the idea is either:

Swapping icons pack easily
Using the default pool of icons

If the concern is over-complication of config, I think the basic look would still look like this:
[icons]
"firefox" = "browser"
"nvim" = "editor"
[title.kitty]
"DEFAULT" = icons
"htop" = "Tasks"
where I can see this as for example in case of nvim, which also has gui version adn would be matched by icons, and terminal version that would be matched by title, and this appears simple enough, what are your thoughts ?

Do I understand correctly that you also envision a syntax to define icon packs (to be able to easily switch between a few) and maybe even override parts of them (e.g. if you don't like a default icon for "browser")?
Would you expect the "firefox" to be defined as "browser" out of the box?
It also looks like you envision this app to know about specific apps, like nvim, to know whether to match those on class or title based on whether they run in terminal or not, and which regex to use in that case, could you please clarify that part? (to explain why I think a specific regex is necessary to properly match nvim by title, we need to avoid to wrongly use nvim icon for Firefox when the active tab, and so Firefox window title, contains the word "nvim").
Thanks!

I'll try to clarify my thoughts:

Expand functionality of the [icons]:

What I can see as a use case is following situation: Icons config that contains glyphs, and Icons config that contains words, so firefox in one config is "" and in other "browser", as future enhancement, maybe even cycle between them ?
Community like r/unixporn would surely come up with themed icon packs and it would be nice to maybe hoard them in config

By default, class is matched by keyword in config

Provide optional list of glyphs that would be good-enough out of the box for user that doesn't wanna tinker too much and wants behaviour that is present in manjaro-sway flavor for example of auto-naming workspaces
Use current title.xxx syntax to define which icons 'pack' should be used for the program and with possibility of overrides, so algorithm of nvim opened in kitty would be roughly:

Find 'kitty' in title.xxx section
If nvim is in this section, use that icon
If not and DEFAULT is specified, use that
If not and DEFAULT is referencing icons pack, search in that icons pack

Lets this way for the moment, I adapted the generator you point to me.
https://github.com/cyrinux/hyprland-autoname-workspaces/pull/45


Update dependencies to be coherent with latest VS.

The core portion of this change is the update from ClientImplementation 16.7.33 -> 16.7.65 to ensure that our other language server dependencies were coherent with the Client.Implementation dependency. To ensure this I removed those references from Versions.props and their correpsonding referenced projects.
Had to pin a few dependencies to ensure things could build without warning.

:shipit:
Ooh the bot can merge without approval?

Ooh the bot can merge without approval?

It's my save n grace 😄


resolves journal display issue from #2377

@clgarciga and I investigated #2377 and determined that there was a
mismatch in the journals_query for the admin.project.journals view.
JournalEntry is stored with the non-normalized name for the project, and
the query previously was comparing Project.normalized_name to JournalEntry.name
one thing to note, I'm not sure if journals are updated when projects have been
renamed in the past. It might be worth adding a normalized_name to JournalEntry
for safety, or add uuids as @dstufft alluded in the past
Co-authored-by<EMAIL_ADDRESS>

Looks good to me! ✨


Get rid of adminapi schema

Currently the adminapi schema is used to generate css files. The projection behind it is a duplicate of projections.label_policies.
To get rid of the schema the generation of styling files needs to be moved to projections/label_policy.
Following steps allow the change:

Add transaction to handler.Reduce => type Reduce func(tx *sql.Tx, eventstore.Event) (*Statement, error). This will include many changes because implementations need the additional param
Allow transaction option in static.Storage-methods so that we can use a *sql.Tx if static storage is sql
Move all css generation logic to projection/label_policy
Generate and store the css if the policy is activated
Make sure reduceActivated only uses a single sql.Conn

is that a bug?


Fix Initial Bid and Adding More to Bid Validations

Changes

Only enforce tick size if past the initial bid.
Calculate bidding power based on the user's current bid plus what is in their wallet. Not just the balance in the wallet.

https://github.com/holaplex/metaplex/pull/128


Upgrade to typescript

This starts using typescript for the package.

I manually ran the babel plugin https://www.npmjs.com/package/@zxbodya/babel-plugin-flow-to-typescript to generate the typescript files here.
Instead of using babel, this is using  tsc with the -d option to also generate type definitions and is putting them in dist directory as that seemed pretty standard for a ts project.
Uses flowgen to turn maybe.d.ts into a flow type definition file. It does not do proper module isolation, so a script was made to add the appropriate  declare module '@freckle/maybe' piece to the file. This script will likely need to be more sophisticated when used for other projects, but KISS seemed like a good idea here

I think we'll want a way to ensure that the dist and flow directories are up to date when making changes- seems like we could use some kind of github action to run tsc and flowgen into a different directory (tmp/dist tmp/flow) and then do a checksum between them to ensure that they are the same.

Have you tested this with any of the frontend projects in megarepo?

Yep! I had to make a change to the .flowconfig to pick up the new type definitions but it appears to work fine


chore(docker): support arm

:tada: This PR is included in version 0.117.1 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


refactor: split widget state into several hooks and added new lifecycle hook

To Reviewers

[ ] Skip (style, chore, ci ONLY)
[ ] Not that difficult

Type of Change

[ ] New feature
[ ] Bug fixes
[ ] Feature improvement
[x] Refactor
[ ] Others (performance improvement, CI/CD, etc.)

Affects to

[ ] Packages

[ ] core-lib
[ ] mirinae
[ ] etc

[ ] Apps

[ ] storybook
[x] web

Checklist

Did you check the lint and type?
Did you document the changes?

Changes in mirinae should be reflected in storybook.

Did you test the app after the package changes?

Description

Split widget state hook into several hooks
Added useWidget to integrate hooks that always used together (frame props, widget state)
Applied to CostMapWidget

Things to Talk About

LGTM! 😧😧 There are some parts difficult for me. Could you share it with us at next sharing time?

Of course!


⚠ Bofrost.de has degraded performance

In b1d25b7, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1630 ms

Resolved: Bofrost.de performance has improved in 62f3c7e after 6 minutes.


How to remove object in MyPackage?

Here is capture, I don't know how to remove this.

Unfortunately, Roblox doesn't have a way to remove assets as of now. If you have difficulty finding the asset you're looking for, I recommend using the Search box.
I also recommend making a feature request on the devforum that explains why you need to remove items in your packagess. Doing this will make sure your pain point is known and it can trigger the right chain of events to get a new feature implemented!
In any case, this is a part of Roblox's toolbox feature, not specific to this Blender add-on, so I am closing this comment.


feat: min and max operators on coins (#11200)

Backport of https://github.com/cosmos/cosmos-sdk/pull/11200

Hrm are we sure this isn't state breaking?

yeah confirmed that this should be fine


Fix up WebSocket high level behavior

Current WebSocketContext behavior is not correct and needs to be thoroughly fixed.

Probably fixed by now


refactor: using dialog for modal

Using HTML  element to create modal instead of floatingUI

Found the following issues with the new implementation:

If opening nested modals, on closing any child modal, all modals are closed.
There is no scroll for large content.

@tiwariav
Nested modals problem is sorted.
It is difficult to get the dialog-modal to be aligned properly with viewport when height is not restricted.
Scrolling inside the dialog will work. But scrolling the whole page has problems.
Dialog element is given by default max height less than the page/viewport. I think it was not designed to have scroll outside the modal.


Gitea blog post

Update with new VP link
Adding "Other products" filter option
MCG is now maintained
Use speakerdeck for the slides
Add blog post about private repositories
Pattern Review Update (#390)
Creating new Understanding namespaces blog (#387)
Industrial Edge demo-script  (#391)
Ie update (#392)
Update demo-script.md (#393)
Update _index.md (#394)
Fix sentence around code being checked by CI and not vice-versa
Intel AMX accelerated patterns (Multicloud GitOps & Medical Diagnosis) (#376)
Create badges directly from the JSON contents
Include the year in CI bages
New Cisco and Validated Patterns team partnership blog
Updated paragraph for MultiCloud GitOps deployment description
Move the comm-attributes include up
Convert demo script to asciidoc
Promote AEG and Retail to testing (#399)
Document the need for aws-creds in AEG
Fix internal ci page due to bad copy/paste
Skeleton of new secrets blog
New blog for secrets options
htpass + hcp oauth config (#404)
travelops documentation (#406)
Travelops ossm (#407)
Travelops ossm (#408)
updated alias for medical-diagnosis-amx docs (#409)
Docs debug (#410)
Note known issue on 4.14
adding intel partnership blog (#413)
Update 2024-03-05-intel-accelerated-patterns.md (#414)
Update 2024-03-05-intel-accelerated-patterns.md
Intel SGX protected application in Multicloud GitOps pattern
Introduce Vault component protected with Intel Secure Guard Extensions (SGX)
** Intel AMX accelerated Multicloud GitOps pattern with OpenShift AI - documentation**
TELCODOCS-1595:Added values files information
TELCODOCS-1595:Updated the outline of basic values file
Addressed review comments
Multicloud GitOps pattern accelerated with Intel QAT - Documentation
Bump peaceiris/actions-hugo from 2 to 3
Bump peaceiris/actions-gh-pages from 3 to 4
Issue around resource in multiple applications has been long fixed
Drop Turning on event streaming between the edge and the datacenter section
Fix for newer hugo versions
OpenShift AI Pattern Docs (#428)
Improve the usability of pre-release badges
Re-add a local make spellcheck target
rag llm docs update (#426)
Moving demo/install content to getting-started doc (#433)
Ragllm update (#434)
Removing rhel-for-edge-general-sizing.adoc
Rename content dir for travelops pattern (#437)
Add placeholder for AGOF
Correct spellings
Tweak capitalization
Added clustergroup information in values files
Addressed review comments
Addressed review comments and made few changes
Update _index.md
Added information related to managing secrets in values files (#435)
update organization name for github repo (#439)
Increasing the timeout for 'hugo serve' (#441)
Pull newer container if available
Disable security labelling for podman commands by default
Add blog post about in-cluster gitea

This is an automated message:
You can preview this docs PR at http://442.docs-pr.validatedpatterns.io
Note that they get generated every five minutes, so please wait a bit.

/lgtm


fix: upgrade apiVersion and content of Ingress resource

This PR updates the french translation to match the networking.k8s.io/v1 resource.

/lgtm
/approve


packing and compression for netCDF files

Packing: reducing the precision of the data to quantity-specific levels of precision

lossy - by definition

Compression: algorithmically processing the data to store a compression representation on storage

lossless - by expectation

This ticket is targeting the exploration of the capabilities of XIOS+netCDF+HDF to pack and compress data.

Packing Techniques:

type coercion to integer with scale_factor & add_offset
quantization

Compression Algorithms

GNU compression with gzip
ZStandard compression with zstd

Parallelisation

client
server (needs a fairly recent version of parallel netCDF+HDF)

superseded by #28


Firebird does not support afterColumn - fix tests on master

Impact

[X] Bug fix (non-breaking change which fixes expected existing functionality)
[ ] Enhancement/New feature (adds functionality without impacting existing logic)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Description

Things to be aware of

Things to worry about

Additional Context

I merged some community PR's today and seems some of them had conflicting tests. This is the second fix I'm doing, let's see if this is enough.


Need to rebuild rpp everytime a different gpu is used.

RPP needs a rebuild everytime a different gpu is used; this blocks me from pre-building RPP into a docker image and distributing it.
I'm using the commit<PHONE_NUMBER>8fbfa685f11083315394f3a4cd952f. is it a known issue and is fixed in recent versions?

Hi @GowthamKudupudi, RPP does not need to be built every time a different GPU is used. Please try with master branch TOT.

@GowthamKudupudi are you still facing this issue with TOT? Please let us know, thanks!

@GowthamKudupudi please reopen this issue if you are still facing this issue. Thanks!


Change link of repojacking vulnerable link

Hello from hacktoberfest :)
##Summary:
The link to https://github.com/youncccat/zzhack is vulnerable to repojacking (it redirects to the orignial project that changed name), you should change the link to the current name of the project. if you won't change the link, an attacker can open the linked repository and attacks users that trust your links

I don't really understand the attack scenario here. Just browsing a repo won't cause any harm, and just don't believe the showcase section is sufficient to trick people into running any kind of code from the linked repositories. It might be worthwhile to add a small disclaimer to the list that we don't officially endorse or vet the list of links, but overall the impact here is very low. This would be different if the linked repo were to added as a dependency that is automatically fetched and somehow built with yew, which is what I understand as repojacking.
Nevertheless, I guess it doesn't hurt to keep the list up to date. Can you double check the other linked repose as well, then I would think this PR is worthwhile.

Hi, its the fact that you have links in your repository and because people trust your repositories they will trust the repositories you link.
However i covered all of the links and i changed the one that are vulnerable, so that should be enough :)

Hi, can you please approve for the hacktoberfest?


Fix SPI Direction

Value was shifted 4 bits to the right

Thank you for catching this.  Crazy this came from the upstream.


Add support for date_bin() function.

Add support for the Postgres SQL date_bin function.

The stride or binWidth is specified in the Period format. For example:

1d		: for 1 day
24h		: for 24 hours
1h30m	: for 1 hours and 30 minutes

The granularities supported for stride are seconds, minutes, hours and days.

The source and origin times are in Timestamp format which is in local time zone in Pinot. It can also take epoch format.

Example Queries:
select date_bin('3h', '2024-03-10 23:29:55.0', '2024-01-01 00:00:00.0') from myTable
select date_bin('15m', myTimeColumn, '2024-01-01 00:00:00.0') from myTable
select date_bin('15m',<PHONE_NUMBER>,<PHONE_NUMBER>) from myTable

Codecov Report
Attention: Patch coverage is 0% with 8 lines in your changes are missing coverage. Please review.

Project coverage is 0.00%. Comparing base (59551e4) to head (c61990d).
Report is 204 commits behind head on master.

Files
Patch %
Lines

...inot/common/function/scalar/DateTimeFunctions.java
0.00%
8 Missing :warning:

Additional details and impacted files
@@              Coverage Diff              @@
##             master   #12790       +/-   ##
=============================================
- Coverage     61.75%    0.00%   -61.76%
=============================================
  Files          2436     2386       -50
  Lines        133233   131134     -2099
  Branches      20636    20274      -362
=============================================
- Hits          82274        0    -82274
- Misses        44911   131134    +86223
+ Partials       6048        0     -6048

Flag
Coverage Δ

custom-integration1
?

integration
0.00% <0.00%> (-0.01%)
:arrow_down:

integration1
?

integration2
0.00% <0.00%> (ø)

java-11
?

java-21
0.00% <0.00%> (-61.63%)
:arrow_down:

skip-bytebuffers-false
?

skip-bytebuffers-true
0.00% <0.00%> (-27.73%)
:arrow_down:

temurin
0.00% <0.00%> (-61.76%)
:arrow_down:

unittests
?

unittests1
?

unittests2
?

Flags with carried forward coverage won't be shown. Click here to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

@mayankshriv have the docs been updated?


Ack tenant detach before local files are deleted

Problem
Detaching a tenant can involve many thousands of local filesystem metadata writes, but the control plane would benefit from us not blocking detach/delete responses on these.
Summary of changes

After rename of local tenant directory ack tenant detach and delete tenant directory in background

#5183
Checklist before requesting a review

[x] I have performed a self-review of my code.
[ ] If it is a core feature, I have added thorough tests.
[ ] Do we need to implement analytics? if so did you add the relevant metrics to the dashboard?
[ ] If this PR requires public announcement, mark it with /release-notes label and add several sentences in this section.

Checklist before merging

[ ] Do not forget to reformat commit message to not include the above checklist

Asking  @jcsp whether this was what you had in mind when opening the issue. For me it seems to be the case.

Asking @jcsp whether this was what you had in mind when opening the issue. For me it seems to be the case.
Yep, for the tenant part.  There is also the other half of the ticket about timeline deletion, which can be handled separately.

@rmodpur see the failure in CI:
error[E0425]: cannot find function `safe_remove_tenant_dir_all` in this scope
   --> pageserver/src/tenant/mgr.rs:185:45
    |
72  | async fn safe_rename_tenant_dir(path: impl AsRef<Path>) -> std::io::Result<()> {
    | ------------------------------------------------------------------------------ similarly named function `safe_rename_tenant_dir` defined here
...
185 |                             if let Err(e) = safe_remove_tenant_dir_all(&tenant_dir_path).await {
    |                                             ^^^^^^^^^^^^^^^^^^^^^^^^^^ help: a function with a similar name exists: `safe_rename_tenant_dir`

@rmodpur see the failure in CI:
error[E0425]: cannot find function `safe_remove_tenant_dir_all` in this scope
   --> pageserver/src/tenant/mgr.rs:185:45
    |
72  | async fn safe_rename_tenant_dir(path: impl AsRef<Path>) -> std::io::Result<()> {
    | ------------------------------------------------------------------------------ similarly named function `safe_rename_tenant_dir` defined here
...
185 |                             if let Err(e) = safe_remove_tenant_dir_all(&tenant_dir_path).await {
    |                                             ^^^^^^^^^^^^^^^^^^^^^^^^^^ help: a function with a similar name exists: `safe_rename_tenant_dir`

Fixed

Tests are failing due to race condition
For eg in test_detach_while_attaching[release-pg15-local_fs]
There is a sequence of detach then attach a tenant
Detach returns 200 so attach starts immediately which starts creating some files in same temp directory of tenant(there is no randomness to it's name only ___temp suffix is added to tenant path) which results in failure of background deletion of tenant directory spawned by earlier detach operation
ERROR Task 'tenant_files_delete' tenant_id: None, timeline_id: None exited with error: tenant directory "/home/rahul/rust/neon/test_output/test_detach_while_attaching[debug-pg14-local_fs]/repo/tenants/3c2de2b7f868b9c876af24379e8595e1.___temp" deletion

Caused by:
    Directory not empty (os error 39)


why use label in loss.py

Thanks for your great work. I understand you use label to compute precision in loss.py. However, you initiated label tensor in 81th row. I wonder how and why you set label like this. Thank you.

Sorry did not get your point. Where is 81th row ?

Sorry, I mistake the row number. I mean this: label = torch.from_numpy(numpy.asarray(list(range(batch_size - 1,batch_size*2 - 1)) + list(range(0,batch_size)))).cuda() in loss.py.

It is the labels for trianing in stage 1, let's say we have 200 data , 400 segments in one minibatch.
The positive pairs are segments from the same data. Negative pairs are segments from the different data. So we can build a label matrix:
1, 0, 0.... , 0
0, 1, 0..... , 0
0, 0, 1.... , ..
....
0, 0, 0......, 1
something like that. The diagonal is 1 and the rest part is 0.

However, the output label may be [2, 3, 4, 0, 1, 2] if the batch size is 3. what's the relationship between output label and label matrix you mentioned?

As what I remember, batchsize=3 -> 6 segments
for each segments, itself can not be selected for choice, so there are 5 choices (0,1,2,3,4)
so the matrix I mentioned (similar to one-hot) can be transformed into the classification label, [2,3,4,0,1,2] means the class. This order is due to the processing in computing the loss.
This part is only used to compute the trianing accuracy for understanding the training process, so it will effect the results.

Ok. it makes sense to me. You do a transformation between original label and one-hot vector. I may understand. Thanks for your explanation.


fix: broken line in page settings

Description
for old pages that were created before the new url style, the page settings has a prompt to ask the user to change the URL.
The line in this prompt is broken

This PR changes this line to:

For testing, you can get an example old app from here
PR fixes following issue(s)
Fixes #30922
Media

A video or a GIF is preferred. when using Loom, don’t embed because it looks like it’s a GIF. instead, just link to the video

Type of change

Please delete options that are not relevant.

Bug fix (non-breaking change which fixes an issue)

Testing
How Has This Been Tested?

Please describe the tests that you ran to verify your changes. Also list any relevant details for your test configuration.
Delete anything that is not relevant

[x] Manual
[ ] JUnit
[ ] Jest
[ ] Cypress

Test Plan

Add Testsmith test cases links that relate to this PR

Issues raised during DP testing

Link issues raised during DP testing for better visiblity and tracking (copy link from comments dropped on this PR)

Checklist:
Dev activity

[ ] My code follows the style guidelines of this project
[ ] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[ ] PR is being merged under a feature flag

QA activity:

[ ] Speedbreak features have been covered
[ ] Test plan covers all impacted features and areas of interest
[ ] Test plan has been peer reviewed by project stakeholders and other QA members
[ ] Manually tested functionality on DP
[ ] We had an implementation alignment call with stakeholders post QA Round 2
[ ] Cypress test cases have been added and approved by SDET/manual QA
[ ] Added Test Plan Approved label after Cypress tests were reviewed
[ ] Added Test Plan Approved label after JUnit tests were reviewed

Summary by CodeRabbit
Summary by CodeRabbit

Refactor

Updated the wording of messages related to setting page URLs for enhanced clarity and consistency.

going to rerun failed tests because the three failures seem unrelated.

second rerun after pulling latest changes from release, as the three failures look completely unrelated.

/build-deploy-preview skip-tests=true

/ok-to-test<EMAIL_ADDRESS>

/ok-to-test<EMAIL_ADDRESS>


How to display different size fonts within LGVL?

Hi there,
Is it possible to display different size fonts within LGVL? Characters...symbols...
Best Regards.

Yes, it is supported. Check first in lv_conf.h to the section FONT USAGE. There is the first place to enable the different sized fonts supplied by lvgl. Next read about Styles in the documentation which specifies how to set up a new font. In addition, there is an online tool to add any TTF font to your project.
See https://littlevgl.com/basics#styles

Hi,
Thanks a lot. It's very helpful!
 I checked the manual and noticed lib-func in lv_font.c : "void lv_font_init(void)"

void lv_font_init(void)
{
....
lv_font_add(&muli_latin_regular_20, NULL);
lv_font_add(&lv_font_symbol_20, &muli_latin_regular_20);
...
}
(the" muli_latin_regular_20 " which is my built-in font)
It registers the font as built-in font(default), am I right or not?
Still, I have some questions about using "style":
What if I need 3(technically N) other different sizes of font(along with symbols) as user font.
"https://littlevgl.com/basics#styles" says:
/Create a new style for the label/
static lv_style_t style;
lv_style_copy(&style, &lv_style_plain);
style.text.color = LV_COLOR_BLUE;
style.text.font = &lv_font_dejavu_40;   /Unicode and symbol fonts already assigned by the library/
lv_obj_t *label;
/Use ASCII and Unicode letters/
label = lv_label_create(lv_scr_act(), NULL);
lv_obj_set_pos(label, 20, 20);
lv_label_set_style(label, &style);
lv_label_set_text(label, "aeuois\n"
"äéüöíß");
/Mix text and symbols/
label = lv_label_create(lv_scr_act(), NULL);
lv_obj_set_pos(label, 20, 100);
lv_label_set_style(label, &style);
lv_label_set_text(label, "Right "SYMBOL_RIGHT);
In cording to codes above, the height of SYMBOL_RIGHT should be default.
If I need a height of 40, what should I do to link lv_font_symbol_40 along with muli_latin_regular_40(usr font)?
Like this:
lv_style_t style_10, style_30, style_40;
lv_label_set_style(label, &style_10);
lv_label_set_text(label, "Right "SYMBOL_XXX);//Label text & SYMBOL height 10
lv_label_set_style(label, &style_default);
lv_label_set_text(label, "Right "SYMBOL_XXX);//Label text & SYMBOL height 20
lv_label_set_style(label, &style_30);
lv_label_set_text(label, "Right "SYMBOL_XXX);//Label text & SYMBOL height 30
lv_label_set_style(label, &style_40);
lv_label_set_text(label, "Right "SYMBOL_XXX);//Label text & SYMBOL height 40
……
???
Well, I'll also try it out in my board.
Regards.

Hi,
you means that call lv_font_add(…)pairs 4 times will link them all,and addressable by the style funtion?
Regards.

Yes. Here it is how LittelvGL uses this to link a symbol font to an ascii:
https://github.com/littlevgl/lvgl/blob/master/lv_misc/lv_font.c#L107
After is if you use my_style.text.font = &lv_font_dejavu_20 it will know the symbols too.

Hi,
So, i can make those predefines "enabled" @FONT USAGE section in lv_conf.h
and the lib will initialize to link them all.
It's very much kind of you, thanks a lot!
By the way, the library is awesome.
Regards.

Yes, the built-in fonts are linked automatically but you can link your own fonts too as mentioned above:
lv_font_add(&lv_font_symbol_30, &muli_latin_regular_30)
Thank you! :)
Please let us know how it workes for you!


Add Link: rel="alternate" header pointing to JSON for a table/query

Originally explored in https://github.com/simonw/datasette-notebook/issues/2#issuecomment-980789406 - I wanted an efficient way to scan a list of URLs and figure out which if any of those corresponded to Datasette tables, canned queries or SQL output that could be represented as a table on a page.
It looks like a neat way to do that is with  Link: header like this:
Link: http://<IP_ADDRESS>:8058/fixtures/compound_three_primary_keys.json; rel="alternate"; type="application/datasette+json"
I can put a <link href=... in the page header too.

I built an initial prototype of this in a branch: https://github.com/simonw/datasette/commit/e0a84691c2959f2d1d76948574c9c4a910c7556c - which exposed even more flaws in the way TableView is structured (adding custom HTTP headers to the response is way harder than it should be) which I should address in the refactor in #617.

I'm going to apply the hack, then fix it again in:

#1518

Documentation: https://docs.datasette.io/en/latest/json_api.html#discovering-the-json-for-a-page
https://docs.datasette.io/en/latest/json_api.html top --cors section mentions the new Access-Control-Expose-Headers: Link header.

Demos - these pages both have <link rel=... if you view source on them:

https://latest.datasette.io/fixtures/sortable
https://latest.datasette.io/fixtures/sortable/a,a

And you can hit them with curl like so:
% curl -I 'https://latest.datasette.io/fixtures/sortable'
HTTP/1.1 200 OK
link: https://latest.datasette.io/fixtures/sortable.json; rel="alternate"; type="application/json+datasette"
cache-control: max-age=5
referrer-policy: no-referrer
access-control-allow-origin: *
access-control-allow-headers: Authorization
access-control-expose-headers: Link
content-type: text/html; charset=utf-8
x-databases: _memory, _internal, fixtures, extra_database
Date: Wed, 02 Feb 2022 07:56:17 GMT
Server: Google Frontend
Transfer-Encoding: chunked

% curl -I 'https://latest.datasette.io/fixtures/sortable/a,a'
HTTP/1.1 200 OK
link: https://latest.datasette.io/fixtures/sortable/a,a.json; rel="alternate"; type="application/json+datasette"
cache-control: max-age=5
referrer-policy: no-referrer
access-control-allow-origin: *
access-control-allow-headers: Authorization
access-control-expose-headers: Link
content-type: text/html; charset=utf-8
x-databases: _memory, _internal, fixtures, extra_database
Date: Wed, 02 Feb 2022 07:56:24 GMT
Server: Google Frontend
Transfer-Encoding: chunked


Hardcode version due to poetry modifying version metadata containing -

This resolves #2579
The issue originates at commit 492091ec250e1ec5e66c792f67aebfa2ef177756 on September 6th
--- a/aries_cloudagent/version.py
+++ b/aries_cloudagent/version.py
@@ -1,4 +1,6 @@
 """Library version information."""

-__version__ = "0.10.1"
+from importlib import metadata
+
+__version__ = metadata.version("aries-cloudagent")
 RECORD_TYPE_ACAPY_VERSION = "acapy_version"
 --- a/pyproject.toml
+++ b/pyproject.toml
@@ -1,12 +1,12 @@
 [tool.poetry]
 name = "aries_cloudagent"
-version = "0.10.0-rc0"
-description = ""
+version = "0.10.1"
+description = "Hyperledger Aries Cloud Agent Python (ACA-Py) is a foundation for building decentralized identity applications and services running in non-mobile environments. "
 authors = ["Hyperledger Aries<EMAIL_ADDRESS> license = "Apache-2.0"
 readme = "README.md"
 packages = [{include = "aries_cloudagent"}]
 classifiers = [
             "Programming Language :: Python :: 3",
             "License :: OSI Approved :: Apache Software License",
 	    "Operating System :: OS Independent",

Which syncs the version with the metadata version
This worked as expected until the latest rc where a - was introduced in commit 82385d8c18d9a2f9bac6aa00bfcd8f36d73e548c and 08d5a4be938527f8eb9d310e56cf0fead2d0bc63 which both occurred on october 30th
--- a/pyproject.toml
+++ b/pyproject.toml
@@ -1,12 +1,12 @@
 [tool.poetry]
 name = "aries_cloudagent"
-version = "0.11.0-rc0"
+version = "0.11.0-rc1"
 description = "Hyperledger Aries Cloud Agent Python (ACA-Py) is a foundation for building decentralized identity applications and services running in non-mobile environments. "
 authors = ["Hyperledger Aries<EMAIL_ADDRESS> license = "Apache-2.0"
 readme = "README.md"
 packages = [{include = "aries_cloudagent"}]
 classifiers = [
             "Programming Language :: Python :: 3",
             "License :: OSI Approved :: Apache Software License",
 	    "Operating System :: OS Independent",

While this will not automatically track with the version in pyproject.toml it is the best known option for restoring the - in release candidate versions.

Why does the metadata.version method strip off the “-“.  I’d rather not have yet another place to update the version for each release, so I like the use of the function to get it from the TOML file, but why does it strip off the “-“.  All of our RC releases have used “-“s, so I’d rather not drop that if we can avoid it.
Shouldn’t metadata.version retrieve and pass the version without change?

@swcurran I think this comes down to how prerelease separators are normalized https://peps.python.org/pep-0440/#pre-release-separators

@swcurran I think this comes down to how prerelease separators are normalized https://peps.python.org/pep-0440/#pre-release-separators

While I like the "look" of a - separated pre-release, I wonder if we should just follow the official normalization rules and fix this downstream instead rather than having custom handling.

Would we be able to pull the non-normalized release version number? Looking into this now.

While it may be uncomfortable, I'm in favor of following PEP 440 rules for release candidate versioning. I think that's most likely to cause the least long-term grief.

Sounds good to me.  So we drop the “-“  and make it “0.11.0rc1” in the release?  I just like the idea of being deliberate vs. adding yet another place where we’re tracking the version.
@WadeBarnes — do you see a problem with us doing that?

@WadeBarnes — or to ask more specifically — are we going to have a problem with the artifact publishing if we do that?  Anything we need to change?

Sounds good to me. So we drop the “-“ and make it “0.11.0rc1” in the release? I just like the idea of being deliberate vs. adding yet another place where we’re tracking the version.

Couldn't agree more. I personally was not in love with this solution. It was just the quickest non breaking solution.

It does not look like dropping the - should affect anything.


Bravyi-Kitaev fermionic transform, via Fenwick tree

Still need to clean up and add some tests.

semaphore build failed because a test uses a fix I just pushed to most recent pyQuil version (len method added to PauliSums)


Why the batch norm is not trained?

May I ask why the batchnorm is excluded in training in your implementation? https://github.com/yaox12/BYOL-PyTorch/blob/master/config/train_config.yaml/#L37. I've gone through the Tensorflow implementation but could not find this configuration in their code. Thanks.

I found it in the paper. Thanks.


How can I annotate my own models using this tool?

I want to annotate my own point cloud models using this tool, but it seems that I can't import other models. What should I do? Thanks.

hello, just convert your data into the format provided in the README. Then, place your file in the javascript src directory, naming the file accordingly.


Ananyas

Pull Request Test Coverage Report for Build 216

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 94.098%

Totals

Change from base Build 217:
0.0%

Covered Lines:
1116

Relevant Lines:
1186

💛  - Coveralls

Pull Request Test Coverage Report for Build 222

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 94.098%

Totals

Change from base Build 217:
0.0%

Covered Lines:
1116

Relevant Lines:
1186

💛  - Coveralls


Fixed Width Issue.

Hello -   I've been trying to get the fixed width version of the slider to work without success..
Following the previous issue:  #32
As i understood it i need to set grid param to 0 on all accounts...
This causes the app to totally break - see https://stackblitz.com/edit/angular-kll1f8.

grid: {xs: 0, sm: 0, md: 0, lg: 0, all: 250}

all denotes the fixed width. don't make it 0 when you need fixed width

@grundell Check the above code and it will solve your Issue

@sheikalthaf  - Thank you that made a lot more sense.

This crashes my website for me


NIFI-8030: Added property to ReportLineageToAtlas to configure file/d…

…irectory level logging of [hd]fs_path entities to Atlas
https://issues.apache.org/jira/browse/NIFI-8030
Thank you for submitting a contribution to Apache NiFi.
Please provide a short description of the PR here:
Description of PR
Enables X functionality; fixes bug NIFI-YYYY.
In order to streamline the review of the contribution we ask you
to ensure the following steps have been taken:
For all changes:

[x] Is there a JIRA ticket associated with this PR? Is it referenced
in the commit message?

[x] Does your PR title start with NIFI-XXXX where XXXX is the JIRA number you are trying to resolve? Pay particular attention to the hyphen "-" character.

[x] Has your PR been rebased against the latest commit within the target branch (typically main)?

[x] Is your initial contribution a single, squashed commit? Additional commits in response to PR reviewer feedback should be made on this branch and pushed to allow change tracking. Do not squash or use --force when pushing to allow for clean monitoring of changes.

For code changes:

[ ] Have you ensured that the full suite of tests is executed via mvn -Pcontrib-check clean install at the root nifi folder?
[x] Have you written or updated unit tests to verify your changes?
[ ] Have you verified that the full build is successful on JDK 8?
[ ] Have you verified that the full build is successful on JDK 11?
[ ] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?
[ ] If applicable, have you updated the LICENSE file, including the main LICENSE file under nifi-assembly?
[ ] If applicable, have you updated the NOTICE file, including the main NOTICE file found under nifi-assembly?
[x] If adding new Properties, have you added .displayName in addition to .name (programmatic access) for each of the new properties?

For documentation related changes:

[x] Have you ensured that format looks appropriate for the output in which it is rendered?

Note:
Please ensure that once the PR is submitted, you check GitHub Actions CI for build issues and submit an update to your PR as soon as possible.

+1, merged to main, thanks @turcsanyip and @Lehel44 !


Tko may 2024

WIP
New commits may still be added, but existing commits can be reviewed/commented already.

fix for #145
refactoring unit tests, removing deprecated method names
refactored the way we get version number on OpenBrowser (pkg_resources is deprecated)
fix: VerifyTable to not break on "special characters" like "[", "]", "*" or "?"
feat: add support for treating color differences as meaningful in *Icon keywords. Also expose "tolerance" argument.
feat: browser and driver management support (Selenium Manager) for edge and firefox
fix: added possibility to strip newlines to GetInputValue (#144)
fix: UploadFile to find inputs of type file when searching with xpath locator

Requested some naming changes but otherwise looks good

All of these should be now included with the latest commit

Note: MacOS/Safari tests are failing due to restriction of running local html files. This will be fixed in another PR.


Enable to handle query parameter as an array even only one

What this PR does / why we need it:
For the filtering apps and deployments by Labels, this is required.
If arrayFormat is set to default, we can't determine if it's an element of an array or a single string.
See: https://github.com/sindresorhus/query-string#arrayformat-1
Which issue(s) this PR fixes:
Ref https://github.com/pipe-cd/pipe/issues/2978
Does this PR introduce a user-facing change?:

NONE

Nice catch 👍
Just want to confirm, does this change effect other query params than labels?

@khanhtc1202 No, this useSearchParams is used by only the application filter and the deployment filter. And they don't accept any array field so far, that's why no need to worry about it.

Get your point, thank you 👍
/lgtm

/approve


fix: support all webpack version

Fix all webpack version support

Hi, I noticed this fork is no longer maintained.
Would you be willing to deprecate this project and recommend https://github.com/privatenumber/webpack-localize-assets-plugin as an alternative?


Custom role claim name

I receive claims that use the value of the System.Security.Claims.ClaimTypes.Role constant ("http://schemas.microsoft.com/ws/2008/06/identity/claims/role") as the role claim name.  The Role.Evaluate method assumes a claim name of "role" so these claims don't get picked up.  I have no control over these claims I'm receiving so to work around this I have some middleware which maps the ClaimTypes.Role claims into claims using the name "role".
Do you have any thoughts on how to handle this scenario?  Or if you are open to a PR, I'm happy to give it a go.

Middleware would be fine - or simply an extension method on ClaimsPrincipal that transforms the claims to the expected types would work as well.


fix: small visual adjustments on Example wrapper

Just some ideas on better visualization on components. Heavily influenced by tailwindui.com.

Hi @osirvent 👋 Thank you for your contribution!
This whole project is heavily inspired by tailwindUI, so no problem 😁
Honestly, I don't really like the heavy rounded border.  I think the iframe should represent the browser as much as possible so I would remove the border radius. Same goes for the 1px border. I think maybe a custom boxShadow which spreads evenly in all direction would look better.
The rest fits pretty well. I like the 1px border on the outer box and the slightly darker background behind the frame 👍
Also there are some merge conflicts because we merged some refactoring in #17

Hi @osirvent 👋 Thank you for your contribution!
This whole project is heavily inspired by tailwindUI, so no problem 😁
Honestly, I don't really like the heavy rounded border.  I think the iframe should represent the browser as much as possible so I would remove the border radius. Same goes for the 1px border. I think maybe a custom boxShadow which spreads evenly in all direction would look better.
The rest fits pretty well. I like the 1px border on the outer box and the slightly darker background behind the frame 👍
Also there are some merge conflicts because we merged some refactoring in #17

Hi @hauptrolle!
what about this:

Hi @hauptrolle!
what about this:

Looks good!
@all-contributors please add @osirvent  for code

Looks good!
@all-contributors please add @osirvent  for code


Selection not same across design system

Description
The ::selection property has background of #C423B2 for text in dark mode in Geist React, It would be best too keep the properties same across the design system.
Screenshots

Yeah, many styles of this project are not synchronized, I'll do a full check later. (sorry, I'm too busy right now, but I promise this update).

Hi @harshcut , I've updated the repository again, but removed some less useful components, so hopefully, that will help!
(Note that version 2.0.0 has some breaking changes, it has fewer components, but more detail!)


[plot] Removing right click management from ClickOrDrag interaction

This PR removes right click management from plot interaction.
Handling of right click in interaction is no more needed since we decided to keep right click free for context menu and this was useless.
Related to #1455 and #1563.

Thanks to @sole, this is actually not a good idea: even if all implemented interactions do not use right button, there is still some signals for it that PyMca is using, so closing for now.

Just a word of caution.
One thing is that we do not implement any action on right click and a different one is to prevent users from catching events associated to right clicks.


How can we get subscription id from above said steps ? Since i need to go through many Azure API calls each API needs subscription ID.

Since i need to go through many Azure API calls each API needs subscription ID. Can anyone help with this ?

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

ID: bf04450a-028e-cc23-d9b3-880a1b5c0fd5
Version Independent ID: ac05d3c4-0379-0f34-9e17-787fb95b7189
Content: Azure AD v2 ASP.NET web server getting started
Content Source: articles/active-directory/develop/GuidedSetups/active-directory-aspnetwebapp.md
Service: active-directory
GitHub Login: @andretms
Microsoft Alias: andret

For this guided setup, you do not need an Azure subscription - you can just sign-in to apps.dev.microsoft.com with a personal account (outlook.com, msn.com, hotmail.com) to register an application and integrate with Azure AD.
You can 'create a tenant' in Azure to be able to test sign in with a work account, create users, groups and test with these accounts by following this guide: https://docs.microsoft.com/azure/active-directory/develop/active-directory-howto-tenant

@Akshayvh94 - Let us know if you have any other questions. Thank you.
Thank you for the response, @andretms
@SaurabhSharma-MSFT @MohitGargMSFT - I think we can close this issue. Thank you. #please-close.

@Akshayvh94 We will now proceed to close this thread. If there are further questions regarding this matter, please reopen it and we will gladly continue the discussion.


Better names for cmd_line_res and local variables

I was debugging F* and noticed that some names were different from ones that are usually used or just not intuitive (like p meaning prims filename).
I'm not sure whether such PRs are welcome because they change (at least this one) names that were intentionally chosen at some point in the past. Please let me know if it's OK, so I could do some more cleanup while working on other issues.

Thanks for the pull request! This is a small contribution, but it arguably improves readability, so we're happy to take it.
This doesn't bootstrap, though. You need to follow steps #2 and #3 of https://github.com/FStarLang/FStar/blob/master/INSTALL.md#step-2-extracting-the-sources-of-f-itself-to-ocaml ; in your case, the GetOpt module is realized in F# and in OCaml (for the two "builds" of F*). In order to fix the OCaml version, you merely need to edit src/basic/ml/FStar_Getopt.ml to fix the constructors of the data type.

Yep, that's what I've just been doing, amended it to the commit.

Thanks!


PowerToys CLI - specifically setting keyboard bindings

Description of the new feature / enhancement
Interested in some way to set powertoys keyboard manager key remaps via command line. Perhaps this is in the form of:

a PowerToys Keyboard Manager specific CLI
a general purpose PowerToys CLI that has options for Keyboard Manager remapping
any other way to use powershell to add / remove remaps

Scenario when this would be used?
I often will use PowerToys to remap WinKey to Left Alt on my laptop. The laptop doesn't have any bespoke mapping software, so PowerToys is a such a great tool for me.
I find I have to manually open the UI, and do quite a few mouse clicks / keyboards presses to bind, then again a few later to unbind.
As a Windows power user, I am an avid powershell user and I have a full custom environment I have tuned to my tastes. I would love the ability to bind/unbind keys with the keyboard manager with a single command in my custom powershell environment.
Supporting information
I looked really hard to see if this functionality existed and I could not spot it, so forgive me ( and hopefully link me <3 ) if it already exists.
also, if you all agree this is a desired functionality and would be interested, I would potentially be into contributing to the code myself to make it happen.

/dup #4649


OCR0030: Extracting missing Tibetan glyphs & ligatures

Description:

To create a new Tibetan font, around 1044 essential glyphs need to be added to the font, The glyphs and ligatures extraction from works has already been done and around 700 glyphs have been extracted to be used in the fonts.
Some glyphs are missing mainly Tibetan superscripts, subscripts and complex ligatures.
These missing were not obtained when the works were applied to Google OCR.
To obtain the missing glyphs another approach has to be taken.

Implementation plan:

Sub-task:

[x] explore the OPF folders
[ ] Write a script to extract the ligatures from the transcribed text
[x] Select all the ligatures with the required superscript and subscript to be cropped
[x] Upload the subjoined images to s3 and create JSONL
[x] Explore alternate procedures to extract ligature which were not caught by google OCR

Completion Criteria:

To obtain all the missing glyphs and ligatures

To extract Tibetan subjoined letters, I am writing a script to parse all the Tibetan ligatures we have found to check for subjoined letters present in the ligatures. the ligature with the subjoined letter found will be uploaded to s3 and a JSONL with all the metadata will be loaded into Prodigy for annotation.

To extract the complex ligatures that google OCR missed, I am using the transcribed text of work ID W2KG209989 (derge tengyur) in OPF. The script will parse the text to find the ligature's image number and span and use that to extract the glyphs.