[FEATURE-REQUEST] support JSX/TSX

hey, I don't know if nativescript-vue v2 support jsx syntax . but can this version consider support it ?

Vue 3 does support it - so it should work - but we may need jsx typings generated, like react-nativescript and svelte-native do.

@rigor789 I am new to typescript . but how can I help to make support for this feature ?

Can we code on nativescript vue 3 without jsx language and pude .vue ?

any progress on nativescript vue 3? i mean thee are several months from the last commit =(

@nour-dev not sure, as I haven't used jsx before, and don't really know what would be needed. I know it's working in svelte/react nativescript, and they had to generate some specific typings to do that, but haven't looked at their types etc.
@eeerrrttty yep, old way with .vue files etc is working with vue3.
@radudiaconu0 (off-topic) progress has been slow, as I frankly didn't have time/motivation to do it.

Ah ok. I want to wait t'ill final release. I have some big project that I want to do and I want to do it in native script vue 3:)

@rigor789 i love nativescript vue bro ;)

@rigor789 please continue the project. i really love this project ;)

I would already like to start trying it out until its stable or final version. How to do ?


Update suspension_type

Design
Additions

For Assay
MUST Use

mCT-seq [EFO:0030060]
"cell" or "nucleus"

MERFISH [EFO:0008992]
"na"

methylation profiling by high throughput sequencing [EFO:0002761] and its descendants
"nucleus"

ScaleBio single cell RNA sequencing [EFO:0022490]
"cell" or "nucleus"

Deletions

For Assay
MUST Use

CITE-seq [EFO:0009294] and its descendants
"cell"

smFISH [EFO:0009918] and its descendants
"na"

snmC-seq [EFO:0008939]
"nucleus"

snmC-seq2 [EFO:0030027]
"nucleus"

spatial proteomics [EFO:0700000] and its descendants
"na"

Notes
Additional assays can be added to suspension_type.
Types from @jychien:

~mCT-seq: “cell” or “nucleus”~
~ScaleBio: “cell” or “nucleus”~
~methylation profiling by high throughput sequencing: The current children are both “nucleus”. Methylation is typically done on nucleus, but I am guessing we’ll need to revisit if there are new decedents in the future.~

Note from @jahilton:
With the updated submission policy, our only smFISH Datasets (which are private) will be removed, and no more will be accepted. So the smFISH and its descendants row can be simplified to MERFISH EFO:0008992
That assay list review...

~remove "CITE-seq and its descendants"~
~replace "smFISH and its descendants" with "MERFISH EFO:0008992"~
~remove "snmC-seq"~
~replace snmC-seq2 with "methylation profiling by high throughput sequencing [EFO:0002761] and its descendants"~

~we need to cover snm3C-seq as all our existing "methylation profiling..." datasets will be revised to that~

~remove spatial proteomics and its descendants~

The list of assays must also be reviewed to eliminate cases that are not supported such as cite-seq.

That assay list review...

remove "CITE-seq and its descendants"
replace "smFISH and its descendants" with "MERFISH EFO:0008992"
remove "snmC-seq"
replace snmC-seq2 with "methylation profiling by high throughput sequencing [EFO:0002761] and its descendants"

we need to cover snm3C-seq as all our existing "methylation profiling..." datasets will be revised to that

remove spatial proteomics and its descendants

Show the math.


maxProfit() missing 1 required positional argument: 'prices'

------------------------------ Best Time to Buy and Sell Stock IV/best_buy_sell.py
Traceback (most recent call last):
File "/Users/hughbrown/workspace/Solutions-To-Problems/LeetCode/Python Solutions/Best Time to Buy and Sell Stock IV/best_buy_sell.py", line 40, in
Solve = Sol.maxProfit([2,4,1])  # [2,4,1] -> 2
TypeError: maxProfit() missing 1 required positional argument: 'prices'

Fixed it. Should work.


Fix dleger logging issue

Fixes #5604

Codecov Report

Merging #5607 (485461f) into develop (08dbe7e) will increase coverage by 0.02%.
The diff coverage is 9.09%.

@@              Coverage Diff              @@
##             develop    #5607      +/-   ##
=============================================
+ Coverage      42.37%   42.40%   +0.02%
- Complexity      7929     7935       +6
=============================================
  Files           1022     1022
  Lines          71254    71261       +7
  Branches        9399     9399
=============================================
+ Hits           30196    30220      +24
+ Misses         37230    37209      -21
- Partials        3828     3832       +4

Impacted Files
Coverage Δ

...broker/processor/AbstractSendMessageProcessor.java
33.43% <ø> (ø)

...ocketmq/broker/processor/PullMessageProcessor.java
27.72% <ø> (ø)

...ocketmq/broker/processor/SendMessageProcessor.java
37.39% <ø> (ø)

.../apache/rocketmq/common/AbortProcessException.java
0.00% <ø> (ø)

.../java/org/apache/rocketmq/common/BrokerConfig.java
28.03% <ø> (-0.12%)
:arrow_down:

...rocketmq/remoting/netty/NettyRemotingAbstract.java
51.92% <9.09%> (+1.92%)
:arrow_up:

...e/rocketmq/store/ha/autoswitch/EpochFileCache.java
78.47% <0.00%> (-2.78%)
:arrow_down:

.../apache/rocketmq/controller/ControllerManager.java
73.11% <0.00%> (-2.16%)
:arrow_down:

...apache/rocketmq/store/ha/GroupTransferService.java
92.30% <0.00%> (-1.29%)
:arrow_down:

... and 18 more

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


The filters should work properly if the user is trying to filter the products.

Steps To reproduce--
1. log in as an admin.
2. Click on products inside the catalog
3. Try to filter the products by order id
Expected result: The filters should work properly if the user is trying to filter the products.
Actual result: The filters are not working properly if the user is trying to filter the products. Please find the below screen recording Here

To avoid so much work duplicacy i am linking this issue to new datagrid. Will fix it all these issues when new datagrid code implemented all over the project.
Link: https://github.com/bagisto/bagisto/issues/5292


[MicroK8s] On "Access Kubeflow" access URL is still correct?

On "Access Kubeflow"
Following document is still correct or it is specific to some version?

Point browser to either: - http://” Your kubeflow VM IP”:“Ambassador PORT”

http://localhost:" Ambassador PORT”

https://github.com/kubeflow/website/blob/8d1235ef6c312105f2fb0a999e29c4756c3e089f/content/docs/started/workstation/getting-started-multipass.md

It is also helpful to write AWS case. (not local server)

Thanks for reporting this @sakaia! I'm closing this issue as a duplicate of https://github.com/kubeflow/website/issues/1113
/close


⚠ Bofrost.at has degraded performance

In e5ccdb3, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1146 ms

Resolved: Bofrost.at performance has improved in f985953 after 6 minutes.


Best pattern to check file content?

Hi @Acconut ,
I see in the FAQ https://tus.io/faq.html#how-can-i-get-the-file-name-or-file-type-for-an-upload a possible way to know about the file content type could to be told by the frontend.
But from a security perspective the frontend could put in metada "hey, here an image", and finally uploads an executable.
With that in mind, what do you recommend to prevent users uploading something wrong? I would like to restrict files they upload.
From what I understand it should be after a success upload since we need to have file content to check its true metadata/content. Maybe it could be something we deal with the PR https://github.com/tus/tusd/pull/373 ?
Otherwise if that's not in a blocking hook, the only solution I see is to consider all tusd uploads as unsafe, and thanks to "post-finished" asynchronous hook, I check their type/content and if ok, I move them elsewhere in the storage I use (another location, Google bucket...)?
What do you think :) ?
Thank you,

the only solution I see is to consider all tusd uploads as unsafe

I would always recommend that mindset, regardless of which user input you are dealing with. File uploads are always potentially harmful, so you can never trust them fully :)

I check their type/content and if ok, I move them elsewhere in the storage I use (another location, Google bucket...)?

Exactly, that's the best solution: Add a check after the upload is complete and only using the file if the check passed.

Thank you for the answer!
I need to figure out the proper way to make this check async while preventing other microservices to start using it.
In case the user submits the profile form with the tusd generated URL for its avatar, the ProfileService can FileService to "commit" this file and get the final image URL (after being moved to the public folder)... but it needs to wait for the check to complete so:

In case the check is long to process (or long queue of jobs), if ProfileService needs to wait, the frontend will also takes time to get an answer
Otherwise, I could make the "commit" call giving in response the final URL image and on during that the FileService pushs a message in an event bus, this message as soon as consumed tries to check the file and move it (but not sure it's fast enough if the frontend needs to use it just after the form submit).

PR https://github.com/tus/tusd/pull/382 may be interesting for you. It adds the pre-finish hook which is a synchronous version of post-finish and is executed after the file has been received but before a response is sent to the client. Maybe that is a good spot to do you validation?


Windows: .atom installs in multiple user dir

Closing #204 in favor of this one.
For some windows machines that have multiple users, the IDE will sometimes split install the .atom in two different user dir.
I have 3 theories on what could cause this.

The IDE is installing alphabetically
Multiple users can be signed in (multiple sessions) at a time and the IDE installs by first login
IDE favors installing in admin dir

Still investigating.

we're no longer installing any files in the .atom directory so this shouldn't be an issue any more


[6.0.1] Enable the sendingArgsAndResults experimental feature

Explanation: SE-0430 was accepted for Swift 6.0 but the corresponding experimental feature sendingArgsAndResults in swift-syntax was not enabled with the proposal’s acceptance. Because of this, SwiftParser cannot parse eg. func whatever(_ foo: sending Foo)
Scope: Parsing of sending parameters in SwiftParser
Risk: Fairly low, this is enabling a feature flag.
Testing: Ran test suite
Issue: https://github.com/swiftlang/swift-syntax/issues/2836 / rdar://135336930
Original PR: https://github.com/swiftlang/swift-syntax/pull/2835
Reviewer: @bnbarham @rintaro @hamishknight

@swift-ci Please test


Brightness slider not showing black (v1.0.5)

Please complete the following information:

Library Version [e.g. v1.0.5]
Affected Device(s) [e.g. Pixel 6 Android 13]

Describe the Bug:
Brightness slider is white on left side
Expected Behavior:
The brightness slider should be black on the left side, instead it's white.
Here's a screenshot displaying the issue

@alfietapping I submitted a PR for this just now. While we wait for the maintainer to take a look at it, you can work around the problem by giving your BrightnessSlider a black background. The default borderRadius is 6.dp in the code, so that's what I used here to make it blend in seamlessly. If you override that value, make sure to change it in the Modifier too:
BrightnessSlider(
    modifier = Modifier
        .fillMaxWidth()
        .background(color = Color.Black, shape = RoundedCornerShape(6.dp))
        .height(35.dp),
    controller = controller,
)

@moertel thanks for this, much appreciated!

@moertel @alfietapping Thanks for addressing this issue guys! I checked the PR, and it looks great to me. The next version will be released with additional bug fixes.

Thanks a lot for the super quick release, @skydoves! I can confirm it's working normally again in 1.0.7 🚀


util: Add LOOP macro for loop with incremented arguments

Added tests for UTIL_LISTIFY and corrected test name.

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@7f78415). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master   #13174   +/-   ##
=========================================
  Coverage          ?   48.97%
=========================================
  Files             ?      317
  Lines             ?    46586
  Branches          ?    10751
=========================================
  Hits              ?    22816
  Misses            ?    19218
  Partials          ?     4552

Impacted Files
Coverage Δ

include/misc/util.h
50% <ø> (ø)

Continue to review full report at Codecov.

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

there is UTIL_LISTIFY macro. It looks that it's doing the same.

An AFAICT equally[0] powerful mechanism is already present.
To reduce the amount of infrastructure, I believe it would be better to re-use what we have.
4dc9f86
[0] Actually, more flexible, as it doesn't require ';' to separate each element and is therefore more re-usable than LOOP.

Thanks for notice, I will try to use UTIL_LISTIFY, could not understand what it does by description.

could not understand what it does by description.

Please do post a PR if you can find a better description.

I started to use UTIL_LISTIFY but it requires to add  ; and I have warnings

but it requires to add ;``
Yes it does, but this is an acceptable price to pay to not have to maintain two macros that nearly do the same thing.
LOOP()
LOOP_WITHOUT_SEMICOLON()

What kind of warnings?

What kind of warnings?
Check warnings in this PR.

Check warnings in this PR.

Not sure how, "Details" doesn't give any more details.
Anyhow, these macro's often violate the static analyzer's rules, I believe a maintainer usually ignores the rule and merges it anyway.

This error seems reasonable to me:
-:13: ERROR:MULTISTATEMENT_MACRO_USE_DO_WHILE: Macros with multiple statements should be enclosed in a do - while loop
do { i += x; } while(0);

This error seems reasonable to me:
-:13: ERROR:MULTISTATEMENT_MACRO_USE_DO_WHILE: Macros with multiple statements should be enclosed in a do - while loop
do { i += x; } while(0);

-:10: WARNING:SINGLE_STATEMENT_DO_WHILE_MACRO: Single statement macros should not use a do {} while (0) loop
#10: FILE: tests/misc/util/src/main.c:63:
+#define INC(x, _) do { i += x; } while (0);

Can't catch a break ...
I don't know how to resolve the warnings.

@SebastianBoe

Not sure how, "Details" doesn't give any more details.

find first comment by zephyrbot, browse through the history of the comment.


CIF-2085: upgrade to wcm core components 2.18

Description

Update WCM Core Components to 2.18
Update Graphql Client to 1.7.7
Fix button v2 to use the now linkURL property when passing the link down to the delegate
Fix compilation of breaking API change in `HtmlPageItem``

Related Issue
CIF-2085
Motivation and Context

How Has This Been Tested?

Screenshots (if appropriate):
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[X] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[X] I have signed the Adobe Open Source CLA.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[X] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes and the overall coverage did not decrease.
[X] All unit tests pass on CircleCi.
[X] I ran all tests locally and they pass.

In https://github.com/adobe/aem-core-cif-components/issues/850 also the resource-types are mentioned to be upgraded. will this be part of the PR?

No, we will do that in separate PRs as we will not just update existing components but rather will create new versions of the existing ones.

Makes sense, thanks 👍


No space between the words

Hi there,
I met a problem. There is no space between the words when I use your extension.

Please fix it. Thanks.

Hi! Could you please send me a public link where I can reproduce this?
Thanks!

Hi! Could you please send me a public link where I can reproduce this?
Thanks!

https://edu.google.com/

Thanks! I will look into that. As a temporary solution, you can disable the extension on https://edu.google.com/ until the problem is fixed.

Thanks! I will look into that. As a temporary solution, you can disable the extension on https://edu.google.com/ until the problem is fixed.

Okay, Thanks a lot!  By the way, how can I lookup the words in video? It's highlighted but cannot popup.

Yes, sometimes the extension highlights words in very strange places - in such cases popup may not work. The modern HTML markup can be super weird. As in the previous case, you can just disable the extension on this site or even switch the extension to "Highlight only favorite sites" mode.


Failed to find deployment (url) in (user)

Hi! I'm deploying an organization project using this action.
I used project linking to get my vercel-project-id and my vercel-org-id
The deploy works perfectly, the URL I'm getting is working, but I get the following error:
Failed to find deployment (url) in (my-personal-account)

I think it may be because I'm using an account token for an organization project, but I can't find where to get an organization token. Is this possible?
Thanks

Just found https://github.com/amondnet/vercel-action/issues/4


OC-185 ~ Updated poms

This pull request makes the build fail.

SonarQube analysis reported 3 issues
Note: The following issues were found on lines that were not modified in the pull request. Because these issues can't be reported as line comments, they are summarized here:

 pom.xml#L596: Make this line start at column 9.
 pom.xml#L597: Make this line start at column 9.
 pom.xml#L598: Make this line start at column 9.

All is well.


Changed KDA type from long to double

I assume there was some changes recently in OpenDota API and this change is needed for Api Client to work

Thanks, I have updated the package


DeepGOPlus code vs DeepGOWeb server inconsistencies

I've been comparing the results generated from the code in this repository and the results generated from the DeepGoWeb server and I noticed some inconsistencies in regards to the GO classes and scores computed. I believe I am getting these inconsistencies due to an incorrect alpha value. I'm including a Jupyter notebook detailing a few test runs with different alpha values and their Jaccard similarity score as well as the Dockerfile I used to create the environment to run the DeepGOPlus code. I also attached below a table of some alpha values I tried out and was wondering if you could provide the ones you use in the DeepGoWeb server for version 1.0.1. I also created a PR request with a potential fix for the alpha values elaborating on the issue and the code I changed.

Dockerfile.txt
DeepGOPlus Alpha Experimentation.pdf
@glatard

Hi,
I think you are comparing different models here. Version 1.0.1 and 1.0.5 use different versions of UniprotKB to train and evaluate the models and they also have different alphas

Thanks for the reply, I had tested data version 1.0.1 as recommended, but still encountered the same issue but with decreased Jaccard similarity scores between the code and server results. Could there be differences in the code between versions?

Hi! I have also encountered the same problem. I used version 1.0.13 in DeepGOWeb and the GO dataset of data-1.0.13.tar.gz in DeepGOPlus, and found that the results of the two were almost not completely similar. At the same threshold, the results of DeepGOPlus are generally lower than those of DeepGOWeb. I would like to know if there have been changes to the web version of the algorithm, or if there are any other reasons. If someone already knows the reason, I would be happy to know and would greatly appreciate the answer.

I have the same problem now. The same data version runs with the same data on the Web version and locally, and the results are almost completely inconsistent.
Can someone tell me why?
Best wish.

Thank you for noticing this issue. There was indeed an issue regarding the alpha values and also a discrepancy between experimental and propagated annotations. We have updated the DeepGOWeb server and release a new DeepGOPlus package (v.1.0.2).


update timeout to additional_config

connect_to_custom() and connect_to_local() still had the old timeout argument and description. I replaced it with the new additional_config to match their input variables (similar to connect_to_wcs() and connect_to_embedded())

Hey @SavvasMohito, thanks for catching this documentation issue! Can you target the main branch of the repo now that we have officially launched v4? Your PR will make it into the first patch release then!

Hi @tsmith023, sure! Check #840. I didn't realise v4 is getting released officially. Great news!


Backup-DbaDatabase, style fixes

Type of Change

[x] Bug fix (non-breaking change)
[ ] New feature (non-breaking change, adds functionality)
[ ] Breaking change (effects multiple commands or functionality)
[x] Ran manual Pester test and has passed (`.\tests\manual.pester.ps1)
[ ] Pester test is included

Purpose
style fixes

no cmdlets were harmed in the process
(do Backup)

@niphlod - i'll wait until these past the tests!

passes checks 👍


Address Book selection drop down menu has invisible text with Blue Ocean Gradient theme for Thunderbird

Describe the problem and steps to reproduce it:
Thunderbird v 91.2.1 using Blue Ocean Gradient theme by Antonio Souza - Click on Write menu icon to open compose message window.  With multiple address books, click on the address book drop down arrow to select the desired address book.
(Please include as many details as possible.)
What happened?
Address Book selection drop down opens with black background and black text.
What did you expect to happen?
Black background and white text
Anything else we should know?
(Please include a link to the page, screenshots and any relevant files.)
All other drop downs and right click menus are fine, and the theme works fine in this version of t-bird other than this issue.

This is an issue with the theme, not the website, contact the author.


bug fix 16604

SUMMARY

The Copy to Clipboard button does not copy the column names, but only the data below.
ADDITIONAL INFORMATION

[x ] Has associated issue: https://github.com/apache/superset/issues/16604
[ ] Required feature flags:
[ ] Changes UI
[ ] Includes DB Migration (follow approval process in SIP-59)

[ ] Migration is atomic, supports rollback & is backwards-compatible
[ ] Confirm DB migration upgrade and downgrade tested
[ ] Runtime estimates and downtime expectations provided

[ ] Introduces new feature or API
[ ] Removes existing feature or API

Was this superseded by https://github.com/apache/superset/pull/23720 ?


GH1: Add option to specify version

This PR addresses #1, by adding support for specifying / pinning the version of Cake installed using cake-version property, example:
    uses: ecampidoglio/cake-action@master
    with:
      cake-version: 0.34.1
      target: Version-Check-Task

This PR adds a bit more, please let me know if you want separate PRs/more focused PR, have separated in multiple commits though.

Adjusted tests to support Windows path if running on Windows
Execute tests on both Windows and Linux
Build addin for integration tests (otherwise previous built version is used)
Add unit and integration tests for cake-version
Only do coveralls if not PR i.e. running on master branch (build fails on PR / forks otherwise)

@devlead Thank you for implementing this!
The Windows-related changes I can pretty much merge as-is; as for the cake-version implementation, I do have a bit of feedback on that.
Would you mind submitting 8205ae6 97dfdfa and 4dad707 448b24a as two separate PRs?

@ecampidoglio sure I can do that and then rebase this PR once 8205ae6 97dfdfa is in.

@ecampidoglio #4 submitted with 8205ae6 97dfdfa

@ecampidoglio PR rebased against latest master.

@ecampidoglio so I've redone most if this PR

.github\workflows\tests.yml now only has additional steps not jobs
gracefully uninstalls and installs if version differs
moved some cake specific code in installLocalCakeTool to a more .NET tool generic approach in installLocalTool.


Make 'Observe' method conditional on having all the references resolved

Signed-off-by: soorena776<EMAIL_ADDRESS>
Description of your changes

Fixes #62
Checklist

I have:

[x] Run make reviewable to ensure this PR is ready for review.
[x] Ensured this PR contains a neat, self documenting set of commits.
[ ] Updated any relevant documentation, examples, or release notes.
[ ] Updated the RBAC permissions in clusterrole.yaml to include any new types.

I think the issue here is a more general one; we shouldn't set the finalizer unless we are sure the resource actually exists, either creating or in any form. See https://github.com/crossplaneio/crossplane-runtime/issues/63

@muvaf I saw #63 and I agree. This fix however is focused on where ReferenceResolver should be placed in the Reconcile loop, as it currently always exits the loop early if a reference is not resolved

// If the references are not resolved the resource cannot exist, assuming
// that the reference fields are immutable.

I am not sure about both of the statements above hold true in our use cases.
On another note, I'm a bit hesitant to make a change on managed reconciler this late to the 0.4 release. How about moving reference resolve block to after if meta.WasDeleted(managed) block to solve this specific issue if it's a really pressing one?

I think we should avoid fixing this for 0.4, which we plan to ship today. My rationale is:

Fixing it would require last minute changes to the managed resource reconciler, which is used by well over half of our controllers these days. The blast radius of unintended effects on 0.4 would be large.
Fixing the managed resource reconciler won't fix the issue for the Kubernetes cluster, bucket, and Azure Redis controllers (assuming they are also affected), which do not use the managed resource reconciler.
I believe it's possible to work around the issue (albeit manually) by editing it to either remove the finalizer, or remove the offending references.
I'm not certain the change proposed by this PR is the best path forward.

I suggest we consider the bug raised by #62 together with that raised by #63 holistically and see if we can come up with a bug fix for 0.4.1.

We ended up taking a different path for this in https://github.com/crossplaneio/crossplane-runtime/pull/70, so I'm going to close this PR. I appreciate the effort coming up with a quick fix, even though we did not run with it for 0.4!


Transition group event hooks

Version
2.3.3
Reproduction link
https://jsfiddle.net/vtoy09cy/2/
Steps to reproduce
Create a transition-group with two or more uniquely keyed transition elements. Attach event listeners to transition hooks and log out the number of times that hook is called. Will equal the number of transition elements.
What is expected?
Each transition element should fire their respective event hooks internally, but the group should emit the event hooks once each transition element has fired that hook. I expect the group to aggregate these events.
What is actually happening?
When using the transition-group component, each uniquely keyed transition element will fire all the transition event hooks discretely rather than in a group.

Is this the desired functionality? A technical limitation?

This is intended behavior. If the hook is used to manipulate the DOM elements then it has to be called for every single item respectively.

Thanks for the response! Any best practice tips for managing the events as an aggregate? A good ol' counter system is all I got, but I don't love the idea of manually syncing the number of transition elements with my counter logic.

Perhaps using namespaced keys and filtering through vm.$children and checking for a match with the vm.$vnode.key? Although you can have regular HTML elements as children, so that would have to be checked as well.


Proposal: Port plugin to AndroidComponentsExtension

I took some time to debug #349 with AGP 7.3.x and I think the root of the problem is that AGP now ignores the addition of the source set. It seems one can set it fine, but it is just silently ignored. I verified this by setting some break points inside the AGP code during the merge resources task and the easylauncher source set is not there. Any other attempts to fix this failed.
I then stumbled over the AndroidComponentsExtension (https://developer.android.com/reference/tools/gradle-api/4.2/com/android/build/api/extension/AndroidComponentsExtension) which provides an API for what easylauncher actually wants to achieve: create a new source set, use a task to generate some new assets and hook task execution and source set merging into the build process.
This required some refactorings (summarized below), but now the implementation no longer relies on implicit assumptions (such as the name of the merge resources task). As far as I can tell, there are only two regressions for which I have not yet found a solution:

the project requires setting android.disableResourceValidation=true
the API does not allow to query the isDebuggable flag

Following is a brief summary of changes:

Move most parts of the processing into the task execution (instead of at configuration time)

Manifest parsing
Icon file gathering

Port to AndroidComponentsExtension API

Entry point: variant.artifact(...)
This tells AGP to execute this task which will generate some resources. The path handling and the task dependencies are automatically taken care of

The sample projects now work again with AGP 7.3.x. Was hoping to get some feedback on this approach and especially the two remaining issues before continuing.

Tremendous! 🎉 This addresses a long overdue migration I wanted to do, but waited until https://issuetracker.google.com/issues/197121905 and https://issuetracker.google.com/issues/170650362 are somehow addressed. If that fixes the linked issue, then even better 🚀 🚀 🚀
I opened a https://github.com/usefulness/easylauncher-gradle-plugin/pull/380 where I'll try to do a follow-up and implement missing stuff.
Regarding:

the project requires setting android.disableResourceValidation=true

I think that's fine 🤷 In the issue report, Google folks acknowledged it's an invalid behaviour, and mentioned this is a workaround, so I hope it'll be eventually addressed on AGP side. If I'm not mistaken, it's already fixed in latest AGP 8.0.0 alphas 👀

the API does not allow to query the isDebuggable flag

I was able to come up with a workaround, which heavily relies on current AGP implementation:
internal val Variant.isDebuggable
    get() = when (this) {
        is AnalyticsEnabledApplicationVariant -> delegate.isDebuggable
        is ComponentCreationConfig -> debuggable
        else -> false
    }

if that breaks, I'll figure something else. Although I'd expected a P1 issue will eventually be addressed at some point 🤷
Thanks again! Great idea on moving work into task execution rather than doing things in configuration time 🚀

Merged in https://github.com/usefulness/easylauncher-gradle-plugin/pull/380, released in 6.0.0

You're welcome :)
Thank you for cleaning up and fixing the open issues! Glad to know that the current workarounds are known AGP bugs.


Update default version to 1.4.0

Older versions have problems with antiquated dependencies.

@ablbot rebase


When use aliyun redis cluster, raise  all the keys that the script uses should be passed using the KEYS arrayr

-ERR bad lua script for redis cluster, all the keys that the script uses should be passed using the KEYS array
link

@tianlu1677 I need more context to understand what is the problem. A sample app that reproduces the problem would be welcome


Nextjs 14 css problem

I upgraded to nextjs14 and I faced some strange problems. Let me show you the screenshots:

Do you have any suggestion? Thanks.

I solved the problem. In app/layout.tsx file remove the <link id="theme-css".... /> tag, put that line to top of the file: import "primereact/resources/themes/lara-light-cyan/theme.css";

But with this method I can't change theme from the right side config panel.


Remove ping from the logger

This both reduces a roundtrip and also keeps the logger alive if ping() fails.
We still need to make the logger return a no-op object if it cannot connect to braintrust at all.
Fixes BRA-527

exciting! Shall we remove user_id from the javascript SDK logging as well?

Mind taking a quick look? Just removed it from the JS side too


adding test to ensure spinner stop was called during init

adds test to fix #1095

Coverage remained the same at 76.307% when pulling 4f58cad543925d1401fb34577d563c94678cf682 on ducks:add-spinner-stop-tests-1095 into<PHONE_NUMBER>190d5433712236583656f4e9d45f7c on tessel:master.


[feat]: Support for Vinxi applications

Feature description
I would like Shadcn CLI to support Vinxi applications which under the hood uses ViteJS.
Currently on a Vinxi setup, shadcn's CLI complains that the framework is not supported, I'm assuming it does that because in Vinxi codebase, the vite.config.ts is usually named app.config.ts.
I was able to solve this by renaming app.config.ts -> vite.config.ts but I would like the CLI to understand that app.config.ts is a ViteJS app too.
Affected component/components
No response
Additional Context
Additional details here...
Before submitting

[X] I've made research efforts and searched the documentation
[X] I've searched for existing issues and PRs

I can make a PR for this if needed, let me know.


How to setup Google cloud storage as Backup Target

I think the backup target section in the docs doesn't really document the GCS setup in entirety.
I'm able to set the correct URI using AWS_ENDPOINTS: <base_64 for https://storage.googleapis.com>.
But I still get the error
invalid header field value \"AWS4-HMAC-SHA256 Credential=\\x18\\xe3\\x86\\x18\\xe4J\\xd9\\x05\\x01\\xe9\\x056 \\xcd\\xd5\\x01s\\x87/20210128/asia-south1/s3/aws4_request

Environment:

Longhorn version: 1.0.2
Kubernetes version: 1.19
Underlying Infrastructure (e.g. on AWS/GCE, EKS/GKE, VMWare/KVM, Baremetal): Digital Ocean

full error message
error listing backups: error listing backup volumes: Failed to execute: /var/lib/longhorn/engine-binaries/longhornio-longhorn-engine-v1.0.2/longhorn [backup ls --volume-only s3://ahins@asia-south1/], output failed to list objects with param: { Bucket: "ahins", Delimiter: "/", Prefix: "/" } response: { } error: AWS Error: RequestError send request failed Get https://storage.googleapis.com/ahins?delimiter=%!F(MISSING)&prefix=%!F(MISSING): net/http: invalid header field value "AWS4-HMAC-SHA256 Credential=\x18\xe3\x86\x18\xe4J\xd9\x05\x01\xe9\x056 \xcd\xd5\x01s\x87/20210128/asia-south1/s3/aws4_request, SignedHeaders=host;x-amz-content-sha256;x-amz-date, Signature=d30219f6e2a2b27081dc51a0ca0365bb5d6a2ea3b3c7c9f25faea7c75b211215" for key Authorization , stderr, time="2021-01-28T20:31:10Z" level=error msg="Fail to list s3: failed to list objects with param: {\n Bucket: \"ahins\",\n Delimiter: \"/\",\n Prefix: \"/\"\n} response: {\n\n} error: AWS Error: RequestError send request failed Get https://storage.googleapis.com/ahins?delimiter=%!F(MISSING)&prefix=%!F(MISSING): net/http: invalid header field value \"AWS4-HMAC-SHA256 Credential=\\x18\\xe3\\x86\\x18\\xe4J\\xd9\\x05\\x01\\xe9\\x056 \\xcd\\xd5\\x01s\\x87/20210128/asia-south1/s3/aws4_request, SignedHeaders=host;x-amz-content-sha256;x-amz-date, Signature=d30219f6e2a2b27081dc51a0ca0365bb5d6a2ea3b3c7c9f25faea7c75b211215\" for key Authorization\n" pkg=s3 time="2021-01-28T20:31:10Z" level=error msg="failed to list objects with param: {\n Bucket: \"ahins\",\n Delimiter: \"/\",\n Prefix: \"/\"\n} response: {\n\n} error: AWS Error: RequestError send request failed Get https://storage.googleapis.com/ahins?delimiter=%!F(MISSING)&prefix=%!F(MISSING): net/http: invalid header field value \"AWS4-HMAC-SHA256 Credential=\\x18\\xe3\\x86\\x18\\xe4J\\xd9\\x05\\x01\\xe9\\x056 \\xcd\\xd5\\x01s\\x87/20210128/asia-south1/s3/aws4_request, SignedHeaders=host;x-amz-content-sha256;x-amz-date, Signature=d30219f6e2a2b27081dc51a0ca0365bb5d6a2ea3b3c7c9f25faea7c75b211215\" for key Authorization\n" , error exit status 1

@ishankhare07
This error is due to an extra new line when generating your base64 encoding for AWS_ACCESS_KEY_ID
Please re-generate  using the following command: echo -n YOUR AWS ACCESS KEY ID  | base64

@meldafrawi I can confirm this doesn't solve the issue. Also the question was about compatibility with Google cloud storage. Can some light be shed on what goes in the fields for AWS_ACCESS_KEY_ID and AWS_SECRET_ACCESS_KEY from GCP?
I am getting the above error even for AWS s3

@ishankhare07 I hit the same error. I finally find out it is only because of my k8s secret yaml (AWS_ACCESS_KEY_ID and AWS_SECRET_ACCESS_KEY) not in in base64 format. after input as base64 then resolved.

It would be great if the documentation provided a setup guide for GCP, as an alternative to AWS S3.
https://staging--longhornio.netlify.app/docs/0.8.1/snapshots-and-backups/backup-and-restore/set-backup-target/


Adding min istio version for new features to avoid running on older revisions

Please provide a description of this PR:
We are adding min istio version for tests related to below PRs as this functionalities were not there in previous revisions :-
a. gRPC fault
injection(https://github.com/istio/istio/pull/39295)
b. Ignore port number in domain
matching(https://github.com/istio/istio/pull/40475)
c. Tunneling outbound traffic :- new tunnel field got
introduced(https://github.com/istio/istio/pull/37968)
d. Fix consistent hash based on source IP for TCP
proxy(https://github.com/istio/istio/pull/38438)
e. Traffic policy load balancer API
changes(https://github.com/istio/istio/pull/39742)
To help us figure out who should review this PR, please put an X in all the areas that this PR affects.

[ ] Ambient
[ ] Configuration Infrastructure
[ ] Docs
[ ] Installation
[ ] Networking
[ ] Performance and Scalability
[ ] Policies and Telemetry
[ ] Security
[X ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure

Please check any characteristics that apply to this pull request.

[X ] Does not have any user-facing changes. This may include CLI changes, API changes, behavior changes, performance improvements, etc.

Hi @hemendrateli. Thanks for your PR.
I'm waiting for a istio member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.
Once the patch is verified, the new status will be reflected by the ok-to-test label.
I understand the commands that are listed here.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

Not sure i understand why you are testing older istio version with test cases in master

/cc @stewartbutler

@hzxuzhonghu This is to test migration use case where multiple versions are installed and workloads communicate across revisions.

/ok-to-test

@hemendrateli: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

release-notes_istio
ab05e308140f797d071624cff152c832e7e52607
link
true
/test release-notes

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/cherrypick release-1.15

@stewartbutler: new pull request created: #40957

In response to this:

/cherrypick release-1.15

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.


Running on Raspbian fails with 'Failed to Launch Browser Process!'

Running nestAuth u=xxx p=xxx fails with the above error.

Fixed in latest merge into Main

Fixed in latest merge into Main


Re-designing Footer

Contact us footer of the project to be Re-designed.

Hey, can I work on this issue under gssoc'22? please assign me.

Can I get the specifications what else to be added in the footer.

@himanshujaidka Can I work on this issue??


SQLGetData incorrectly set strLen to SQL_NO_TOTAL

Describe the bug
SQLGetData is setting strLen to SQL_NO_TOTAL when it finishes reading data or when there's no data. This causes PBI and Excel to fail to load data through the driver.
Per ODBC specs, it should not do this; SQL_NO_TOTAL is only set when the driver could not determine the data size.

See part 7 here.

Expected Behavior
strLen is set to SQL_NO_TOTAL only when the data size could not be determine.
Current Behavior
SQLGetData set strLen to SQL_NO_TOTAL when data is empty or after it finishes reading data.
Reproduction Steps
Execute an empty query SELECT '' then use SQLGetData to get the column 1 data. The result should be successful and strLen = -4 (SQL_NO_TOTAL)
Possible Solution
This should return 0 instead
Additional Information/Context
No response
Timestream ODBC Driver Log
No response
Timestream ODBC Driver version
2.1.1
Host Operating System and version
Windows 11, OSX 14

Thank you for creating the issue. Closing as PR-22 has the fix for the bug.


docs: added new readme badge

This PR adds a new README badge

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 82.14%. Comparing base (8e67800) to head (2b70169).
Report is 7 commits behind head on main.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files

@@            Coverage Diff             @@
##             main     #123      +/-   ##
==========================================
+ Coverage   81.87%   82.14%   +0.27%
==========================================
  Files           4        4
  Lines         651      661      +10
==========================================
+ Hits          533      543      +10
+ Misses         91       90       -1
- Partials       27       28       +1

see 1 file with indirect coverage changes


Reduce indentation in the database/tables tree (or a setting to do so)

On a laptop screen the space is very restrained. I would like to make the indentation 2x smaller and give space to more useful things:

Please take a look at this feature request and vote 👍 to promote it if you want it to be released: https://github.com/dbeaver/dbeaver/issues/34961#issuecomment-2247307048
It pursues the same goal.
The idea is to avoid two indentation levels in a tree to release some additional space and avoid additional mouse clicks when navigating the tree.

Additional way to release more space could be to decrease the width of the margins highlighted yellow on the image below:

Thank you for idea. Maybe it makes sense to implement some compact mode for small screens.

We want to know how many users need this feature. So, for anyone who want it be implemented, please vote for the issue with 👍 under the issue description.

This is an inefficient way to undersatnd user feedback. Only a small percent of users would bother to go to github AND find this thread AND vote. Don't waste managerial time on this burocracy, just follow common sense, which is:

people use laptops
laptop screens are smaller than desktop screens
dbeaver lacks compact view

This should be enough to decide to make it possible to display the tree in a more compact way.
Otherwise you'll end up spending more time managing and researching than just doing it and releasing. Come on!


Not working

It is not working instead of blocking the ads i am permanently stuck on the disable your adblocker page no matter how many times i reload no many how many times i open a new tab, try different videos, etc I do not have any other addblockers running.
Also another issue i noticed before it completely broke is sometimes when i would click on the video it would just be a white screen no video would play and i would have to reload the page
This a really great add blocker please fix it
:)

After update the script Ads started. I'm Use Opera Browser full updated.

Try updating your browser, it worked for me.

Up to date Chrome - still same effect. Doesn't work since this morning at all

I'm Notice when we use if we use YouTube Miniplayer then ads started. otherwise work well.

No, it doesn't work at all since last update.

3.1 is broken on firefox!

Downgrade to 2.9. version (Should Work!) To Do so (For Google Chrome):

Uninstall tampermonkey extension.
Close web browser and re-open.
Install tampermonkey extension - https://chromewebstore.google.com/detail/tampermonkey/dhdgffkkebhmkfjojejmpbldmpobfkfo?hl=lv&utm_source=ext_sidebar
Install script - https://github.com/TheRealJoelmatic/RemoveAdblockThing/releases
Do Not! I repeat! Do Not! UPDATE! If it asks for update just cancel it!

WOXYLV

Fixed in 3.2


Attribute application@appComponentFactory value=(android.support.v4.app.CoreComponentFactory) from [com.android.support:support-compat:28.0.0] AndroidManifest.xml:22:18-91   	is also present at [androidx.core:core:1.0.1] AndroidManifest.xml:22:18-86 value=(androidx.core.app.CoreComponentFactory).   	Suggestion: add 'tools:replace="android:appComponentFactory"' to  element at AndroidManifest.xml:12:5-125:19 to override.

Attribute application@appComponentFactory value=(android.support.v4.app.CoreComponentFactory) from [com.android.support:support-compat:28.0.0] AndroidManifest.xml:22:18-91
is also present at [androidx.core:core:1.0.1] AndroidManifest.xml:22:18-86 value=(androidx.core.app.CoreComponentFactory).
Suggestion: add 'tools:replace="android:appComponentFactory"' to  element at AndroidManifest.xml:12:5-125:19 to override.

This is not very useful. Are you using appcompat? you should use androidx instead.

Thanks for your fast response any way I found another smilers library
that's worked well
On Sun, Mar 3, 2019, 11:39 PM Pierfrancesco Soffritti <
<EMAIL_ADDRESS>wrote:

[image: Boxbe] https://www.boxbe.com/overview This message is eligible
for Automatic Cleanup<EMAIL_ADDRESS>Add cleanup rule
https://www.boxbe.com/popup?url=https%3A%2F%2Fwww.boxbe.com%2Fcleanup%3Fkey%3DmveFqf84mXqPOPyMO2H5LuhkMzadnkDPQppW6lRbpP4%253D%26token%3D1DGtU%252BA6m1o4ggr1VxUFcobfuBrh1dCVRSPZb3J%252F2ZBnC45dsYej%252Fs1sLuzK2WtiLnvLc3nSTxPvsrlRTRxXgzlS%252BXH7O723gBJNc6Zno%252FE5ixx2ynM%252Fcdr6ZngSRpTLxB8fy3nW9XxbQF4NCf4t4Q%253D%253D&tc_serial=48440001965&tc_rand=1736993515&utm_source=stf&utm_medium=email&utm_campaign=ANNO_CLEANUP_ADD&utm_content=001
| More info
http://blog.boxbe.com/general/boxbe-automatic-cleanup?tc_serial=48440001965&tc_rand=1736993515&utm_source=stf&utm_medium=email&utm_campaign=ANNO_CLEANUP_ADD&utm_content=001
This is not very useful. Are you using appcompat? you should use androidx
instead.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/PierfrancescoSoffritti/android-youtube-player/issues/318#issuecomment-469047071,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ANsZPcUZLT8Szul0AA4dwqQxAx5n33ydks5vTAyrgaJpZM4bbIPy
.


nbdev_template repo creation ci fail

Hi,
when clicking on nbdev_template link from nbdev tutorial, I can Create a new repo  but then the ci pipeline start and fail automatically (without changing anything in the code)
 Successfully installed MarkupSafe-1.1.1 Send2Trash-1.5.0 attrs-19.3.0 backcall-0.2.0 bleach-3.1.5 decorator-4.4.2 defusedxml-0.6.0 entrypoints-0.3 fastscript-0.1.4 importlib-metadata-1.6.1 ipykernel-5.3.0 ipython-7.15.0 ipython-genutils-0.2.0 ipywidgets-7.5.1 jedi-0.17.1 jinja2-2.11.2 jsonschema-3.2.0 jupyter-1.0.0 jupyter-client-6.1.3 jupyter-console-6.1.0 jupyter-core-4.6.3 mistune-0.8.4 nbconvert-5.6.1 nbdev-0.2.18 nbformat-5.0.7 notebook-6.0.3 packaging-20.4 pandocfilters-1.4.2 parso-0.7.0 pexpect-4.8.0 pickleshare-0.7.5 prometheus-client-0.8.0 prompt-toolkit-3.0.5 ptyprocess-0.6.0 pygments-2.6.1 pyparsing-2.4.7 pyrsistent-0.16.0 python-dateutil-2.8.1 pyyaml-5.3.1 pyzmq-19.0.1 qtconsole-4.7.5 qtpy-1.9.0 six-1.15.0 terminado-0.8.3 testpath-0.4.4 tornado-6.0.4 traitlets-4.3.3 wcwidth-0.2.4 webencodings-0.5.1 widgetsnbextension-3.5.1 zipp-3.1.0 124 Obtaining file:///home/runner/work/nbdev_test/nbdev_test 125     ERROR: Command errored out with exit status 1: 126      command: /opt/hostedtoolcache/Python/3.6.10/x64/bin/python -c 'import sys, setuptools, tokenize; sys.argv[0] = '"'"'/home/runner/work/nbdev_test/nbdev_test/setup.py'"'"'; __file__='"'"'/home/runner/work/nbdev_test/nbdev_test/setup.py'"'"';f=getattr(tokenize, '"'"'open'"'"', open)(__file__);code=f.read().replace('"'"'\r\n'"'"', '"'"'\n'"'"');f.close();exec(compile(code, __file__, '"'"'exec'"'"'))' egg_info --egg-base /tmp/pip-pip-egg-info-uuq9d1po 127          cwd: /home/runner/work/nbdev_test/nbdev_test/ 128     Complete output (5 lines): 129     Traceback (most recent call last): 130       File "<string>", line 1, in <module> 131       File "/home/runner/work/nbdev_test/nbdev_test/setup.py", line 13, in <module> 132         for o in expected: assert o in cfg, "missing expected setting: {}".format(o) 133     AssertionError: missing expected setting: description 134     ---------------------------------------- 135 ERROR: Command errored out with exit status 1: python setup.py egg_info Check the logs for full command output. 136 ##[error]Process completed with exit code 1.
I did the same process with the command line and the pushing a new repo but same error

"without changing anything in the code" is the problem (o: there are a few things you need to do before a new project will build - if you follow https://nbdev.fast.ai/tutorial/ up to "Commit to Github", you'll have a working project

Thanks a lot, it works now, I'll close the issue.


backend and creating routes

Created the authentication for the login and register page. created the database and node.js

Lgtm


DB Backup doesn't clear the backup folder

This leads to obsolete collections re-appearing in the DB

Also restores are not clean, because collections which are not restored are not dropped


[BEAM-4454] Support Avro POJO objects

Allow schema inference from POJOs marked up with avro tags (e.g. AvroName, AvroIgnore, AvroSchema).
Note: enum types are still not supported
R: @kanterov

LGTM 👍 Great, the code became much cleaner after refactoring!


[proxy] weave status should report on proxy status

so what should it report? just that it's running, and whether --with-dns or --with-ipam are in effect.


[Idea] Draw from existing HTML code and HTML (.html) files

Hey there Andy,
This is just a semi-random idea.
In the README you show this example:

And the corresponding glimmer code:
require 'glimmer-dsl-web'

include Glimmer

div('#application') {
  label(class: 'greeting') {
    'Hello, World!'
  }
}

What if the HTML code would already exist, and glimmer would parse
that code (as a String), and then automatically create / build the
corresponding glimmer code? Optionally to also output this and
allow people to re-use it as-is.
This could allow for the functionality to, for instance, download
a .html file, and have glimmer parse it into "glimmer elements"
so to speak.
It could also allow to work with "glimmer components" and re-use
them easily.
I guess you may ask "what is the use case and rationale for this"?
Well, it would allow for a bit more flexibility in using of the
glimmer suite, which may be useful. It may also make a transition
into glimmer easier. And people could embed and mix HTML code
with "glimmer code".
Now that may be a bit messy (PHP code serves as negative example
perhaps), but ultimately the flexibility gain may be useful, so
that may be a trade off simply.
An additional idea could be to actually write purely in HTML and
CSS and then just build glimmer apps "for free", without having
to learn or transition into the glimmer DSL (which may take a
while for people to learn; not everyone may want to learn ruby
either, for instance).
Anyway, this is just an idea. Please feel free to proceed as
always in any way you see fit. :)
PS: Actually, as a "spin off" idea to this, if glimmer were to
be able to parse HTML/CSS, and if it were to obtain a full
(and flexible) specification, people could actually tap into
it and have glimmer act as the "translator" between different
toolkits AND DSLs, even across different programming languages.
And then perhaps become a bit more like rack (a ground for
others to build upon) and perhaps also rubocop (e. g. having
code elements that can yield glimmer code, or other code,
automatically, such as via the autocorrect option for rubocop,
kind of like a "glimmer AI", aka "Hello Sir, today I shall
take your HTML/CSS/JavaScript code and automatically translate
it into ruby + glimmre for you - and then also automatically
compile a statically compiled variant via graalVM". Ok a bit
of wishful thinking here, but it's just an idea after all.

The whole point of using the Glimmer DSL though is to NOT write any HTML or CSS code and to be able to mix imperative logic with declarative structure in one language (without opening scriptlets like in PHP/ERB or using awkward technologies like JSX). People don't need Glimmer to use HTML/CSS. They already are using it, so there is no point in supporting that. If people were to add Glimmer, it would be to use the Glimmer DSL, which provides leaps and bounds of productivity over HTML/CSS.
That said, there is merit to the idea of auto-converting plain HTML into Glimmer DSL to enable web developers to reuse old web app HTML from Ruby by first converting into Glimmer DSL code and then adding Ruby logic to it.
I will keep this issue open till I have implemented an inverse HTML to Glimmer DSL converter in the future.

The implementation of an automatic HTML to Glimmer Converter has been handled in the newly released Glimmer DSL for XML 1.4.0.
An HTML to Glimmer Converter has been included in it that depends on the nokogiri Ruby gem:
https://github.com/AndyObtiva/glimmer-dsl-xml?tab=readme-ov-file#html-to-glimmer-converter
So, I am closing this issue as it has been fully handled.
Here is the converter documentation from the Glimmer DSL for XML project readme:
HTML to Glimmer Converter
The Ruby gem includes a HTML to Glimmer converter (html_to_glimmer) to automatically convert legacy HTML code into Glimmer DSL syntax.
Prerequisite: the nokogiri Ruby gem. If not already installed, run gem install nokogiri before using html_to_glimmer.
Script:
bin/html_to_glimmer
Usage:
html_to_glimmer path_to_html_file

Example:
Suppose we have an HTML file called input.html:

Welcome

It is good to have you in our platform!

    <form action="/owner" method="post">
      <input type="text" value="you" />
    </form>

We can run this command:
html_to_glimmer input.html

Printout:
Converting from HTML syntax to Glimmer DSL Ruby syntax for input file: input.html
Converted output file: input.html.glimmer.rb

Output file (input.html.glimmer.rb) is a runnable Ruby file containing Glimmer DSL for XML & HTML syntax:
require 'glimmer-dsl-xml'

include Glimmer

html_document = xml {
  html(style: 'max-height: 100%') {
    body(style: "max-height: 100%; font: 'Times New Roman', Arial;") {
      h1(id: 'top-header', class: 'header', 'data-owner': 'John "Bonham" Doe') {
        "Welcome"
      }
      p {
        span {
          "It is good to have "
        }
        strong {
          "you"
        }
        span {
          " in our "
        }
        strong {
          em {
            "platform"
          }
        }
        span {
          "!"
        }
      }
      form(action: '/owner', method: 'post') {
        input(type: 'text', value: 'you')
      }
    }
  }
}

puts html_document.to_s


Add open source license for VisualizeWifi

@hsiehbingting , please add an open source license for your project. If you are not sure how to do it, see https://github.com/solvcon/modmesh.

Please provide update once you made progress related to an issue.


Resolve RS1031 errors

Encountered the following errors when building with the latest roslyn-analyzers version.
error RS1031: The diagnostic title should not contain a period, nor any line return character, nor any leading or trailing whitespaces

Thank you!


[BUG] Angular v. 10 - Cannot use namespace 'CSS' as a type.

🐞 Bug report
Description
When building my project under Angular version I get the following error:
ERROR in node_modules/@ng-web-apis/common/tokens/css.d.ts:4:14 - error TS2709: Cannot use namespace 'CSS' as a type.

    4         CSS: CSS;
                   ~~~
    node_modules/@ng-web-apis/common/tokens/css.d.ts:7:42 - error TS2709: Cannot use namespace 'CSS' as a type.

    7 export declare const CSS: `InjectionToken<CSS>;

Reproduction
Build project under Angular version 10
Expected behavior
No error should be shown
Versions

OS: Linux
Browser: Chrome
Angular 10

Additional context

It's an unexpected issue with TypeScript 3.9+. We will release a fix, in a meantime you can upvote this: https://github.com/microsoft/TypeScript/issues/39273

Hi! Thank you for this issue
We released a tmp fix for the problem in https://github.com/ng-web-apis/common/pull/6.
Please, update to a package version v1.4.1 to use it with Angular 10


Add Option for Remove Local Echo

Hi,
thank you for this great project.

So my question is if an option can be added to remove local echoes.
Thanks!

Hi, would you be able to share more infomation? Because "local echo" is usually an option provided by a serial terminal tool, however modpoll doesn't have "local echo" feature builtin.

Hi,
i am using modpoll in modbus RTU.
since my new usb-rs485 converter (FTDI FT232BL Chipset) echoes every sent byte back to its TX, modpoll gets checksum errors
Some modbus master simulators like in my screenshot do removing local echo.

Close it due to no activity for a while.


Background mode for location update is working only for 3-4 minutes in android 9.0(P) version

I am using background mode plugin for location tracking even when the app is in background. I need to update my driver location periodically in our server. I can update my driver location in some versions of android. I have tested in android 5.0 Samsung, android 6.0 moto, Redmi 8.1, Redmi 9.0 versions. It's not working in Android 9.0 after 3-4 minutes. It's showing background tracking notification and it's sending the location to the server. After 3-4 minutes it stopped working.

having the same issue here

could you save the geolocation in the background ??? he only leaves me once later he doesn't execute it again.

Hi guys,
I have the same problème the app work in backgroun for 4-5 minutes then stop working.
is it because of my Android version (I am using Android 6.0.1).
please if there is anyone that this plugin is working fine with him let me know what android version you are using.
Thank you

Same issue on Android 9.0
3-4 minutes and then it stops

I am facing the same porblem, tested on Androi 7, 9 and 10.
Has anyone found a solution in the meanwhile?


updateLatency can't detect failure

I am using go-redis's redis cluster client. And this is my client configuration:
rdb = redis.NewClusterClient(&redis.ClusterOptions{
Addrs: redisAddresses,
ReadOnly: true,
RouteByLatency: true,
ReadTimeout: time.Duration(readTimeout) * time.Millisecond,
WriteTimeout: time.Duration(writeTimeout) * time.Millisecond,
})
So i enabled RouteByLatency, i expect this feature will route my request to the redis node with best network environment. Also if the master/slave failed, i expect it will route my request to the healthy node in the same shard immediately. But during the test, if i killed my redis master node, it will take around 15-30 sec for the go redis client to recover. So my question is, what actually happen if one of redis node failed, and RouteByLatency can't detect that? So i looked into the code:
func (n *clusterNode) updateLatency() {
const numProbe = 10
var dur uint64
for i := 0; i < numProbe; i++ {
	time.Sleep(time.Duration(10+rand.Intn(10)) * time.Millisecond)

	start := time.Now()
	n.Client.Ping(context.TODO())
	dur += uint64(time.Since(start) / time.Microsecond)
}

latency := float64(dur) / float64(numProbe)
atomic.StoreUint32(&n.latency, uint32(latency+0.5))

}
Seems like n.Client.Ping(context.TODO()) it's waiting forever without a timeout? So if the redis node is failed, this function call will block until it recover?

updateLatency relies on ReadTimeout to not block forever. And it just measures latency without trying to detect unavailable nodes. That is Redis Cluster responsibility, not go-redis.

But i do want to achieve this fast detection and switch to a healthy node feature. And i have already set my ReadTimeout as 100 millisecond. So i guess if one of the node failed, this n.Client.Ping() will return in 100 millisecond, and record a 100 millisecond as latency for current node, which is very bad. Then it will automatically switch to another node in the same shard? But in reality, we didn't find such pattern.

As I said it is not go-redis responsibility to detect bad nodes. go-redis can only retry the command on other node which AFAIK it already does.

So you mean if go-redis failed to fetch result from current node, it will automatically retry the command on other node? I am wondering where is the logic? We didn't find it.

See https://github.com/go-redis/redis/blob/master/cluster.go#L771-L850

Thanks!


Biodiversity5 WIP

This is a WIP, do not merge even if tests pass.

Codecov Report

Merging #1982 (8022530) into development (d8c3f6d) will decrease coverage by 17.21%.
The diff coverage is n/a.

@@               Coverage Diff                @@
##           development    #1982       +/-   ##
================================================
- Coverage        88.74%   71.52%   -17.22%
================================================
  Files             1785     1388      -397
  Lines            61206    38443    -22763
================================================
- Hits             54316    27497    -26819
- Misses            6890    10946     +4056

Impacted Files
Coverage Δ

spec/support/concerns/random_token_methods.rb
2.63% <0.00%> (-97.37%)
:arrow_down:

spec/support/concerns/request/api.rb
11.11% <0.00%> (-88.89%)
:arrow_down:

app/models/identifier/global/issn.rb
14.28% <0.00%> (-85.72%)
:arrow_down:

app/models/identifier/global/isbn.rb
8.33% <0.00%> (-85.42%)
:arrow_down:

lib/dwca/generator.rb
20.00% <0.00%> (-80.00%)
:arrow_down:

app/models/identifier/global/orcid.rb
22.22% <0.00%> (-77.78%)
:arrow_down:

app/models/identifier/global/lccn.rb
11.53% <0.00%> (-76.93%)
:arrow_down:

lib/batch_load/import/otus.rb
24.24% <0.00%> (-75.76%)
:arrow_down:

app/models/geographic_item/geometry_collection.rb
20.58% <0.00%> (-73.53%)
:arrow_down:

...odels/identifier/global/gen_bank_accession_code.rb
27.27% <0.00%> (-72.73%)
:arrow_down:

... and 829 more

Continue to review full report at Codecov.

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

Codecov Report

Merging #1982 (8022530) into development (d8c3f6d) will decrease coverage by 17.21%.
The diff coverage is n/a.

@@               Coverage Diff                @@
##           development    #1982       +/-   ##
================================================
- Coverage        88.74%   71.52%   -17.22%
================================================
  Files             1785     1388      -397
  Lines            61206    38443    -22763
================================================
- Hits             54316    27497    -26819
- Misses            6890    10946     +4056

Impacted Files
Coverage Δ

spec/support/concerns/random_token_methods.rb
2.63% <0.00%> (-97.37%)
:arrow_down:

spec/support/concerns/request/api.rb
11.11% <0.00%> (-88.89%)
:arrow_down:

app/models/identifier/global/issn.rb
14.28% <0.00%> (-85.72%)
:arrow_down:

app/models/identifier/global/isbn.rb
8.33% <0.00%> (-85.42%)
:arrow_down:

lib/dwca/generator.rb
20.00% <0.00%> (-80.00%)
:arrow_down:

app/models/identifier/global/orcid.rb
22.22% <0.00%> (-77.78%)
:arrow_down:

app/models/identifier/global/lccn.rb
11.53% <0.00%> (-76.93%)
:arrow_down:

lib/batch_load/import/otus.rb
24.24% <0.00%> (-75.76%)
:arrow_down:

app/models/geographic_item/geometry_collection.rb
20.58% <0.00%> (-73.53%)
:arrow_down:

...odels/identifier/global/gen_bank_accession_code.rb
27.27% <0.00%> (-72.73%)
:arrow_down:

... and 829 more

Continue to review full report at Codecov.

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


Can't build in XCode 6.1.1 (iOS 8.1)

I'm try to build this project in XCode 6.1.1 but i got many issue. Please check it.
Thanks.

Unfortunately, MarqueeLabel-Swift is very out of date. I added a note to the README about the status, until I get around to updating things. Meanwhile, please use the Objective-C version of MarqueeLabel, or help out by submitting fixes!


Update version and CHANGELOG for new release: v0.0.3

Signed-off-by: Alexander Xydes<EMAIL_ADDRESS>

Codecov Report

Merging #15 (67f570a) into main (d1fff87) will not change coverage.
The diff coverage is n/a.

:exclamation: Current head 67f570a differs from pull request most recent head c036671. Consider uploading reports for the commit c036671 to get more accurate results

@@            Coverage Diff            @@
##              main       #15   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            4         4
  Lines          289       289
=========================================
  Hits           289       289

Continue to review full report at Codecov.

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


[SEARCH INPUT TEXT]: THE TEXT REMAINS EVEN AFTER RERENDER

Give a brief about the bug ✍
The input text that is expected to be cleared or updated during a rerendering process remains visible and unchanged.This is not advisable as it would cause some disturbance in searching.
What is the expected behavior? 🤔
The expected behaviour is after a rerender the text in the search input shoud reset to empty or its default placeholder.
Provide step by step information reproduce the bug 📄
I plan on solving the issue by providing new logic for the text input change on rerender by either creating a function to change text.
Select program in which you are contributing
GSSoC24
Code of Conduct

[X] I follow CONTRIBUTING GUIDELINE of this project.

Hey @laxmanp090404 !
Thank you for raising an issue 💗
You can self assign the issue by commenting /assign in comment 😀
Make sure you follow CODE OF CONDUCT and CONTRIBUTING GUIDELINES 🚀
Don’t Forget to ⭐ our GameZone🎮
Make sure you join our Discord🕹

Hey @laxmanp090404! 🙂
It seems like you are not following proper guidelines !! 👀
Read documentation properly !!🙏
If you have any queries reach out to us on Discord🕹 .

Hey @laxmanp090404 ! Thank you so much for your raising the issue💗
It’s all yours, you can come anytime again and make some contributions! 🚀
Alone, we can do little, but together we can do so much! 😇


node.js - span navigation links

verify that the user can navigate between spans, not in the current document.
this behaviour should be enabled after the cache implementation

Implemented in #244


Feature: add option to include splash damage as affix

Link to get info from:
https://d2mods.info/forum/viewtopic.php?f=122&t=65367&sid=f6383286386c122a085c0a261fdf2a46&start=25
Basics can get done, adding few lines in txt files.

So in d2mm they done pretty same that in this topic, here the actual files (contents just appended)
https://github.com/tlentz/d2modmaker/tree/0588eb03e47f6260786fcd41b6ea4dc6892e970d/assets/splash
Tricky part is adding a localization though.
I guess I can just avoid this in D2 legacy mode for now.

Probably can be done as an extension with new mod merge features. If yes, I will close that as "won't fix"

Won't do that. Current features are sufficient.


Accepting friend request

I'm not sure how to fix this but I'm getting
TypeError: Cannot read property 'PendingInvitee' of undefined
steamFriends.on('friend', function(steamID, relationship) {
if (relationship == steamFriends.EFriendRelationship.PendingInvitee) {
console.log ('[' + steamID + '] Accepted friend request');
steamFriends.addFriend(steamID);
}
else if (relationship == steamFriends.EFriendRelationship.None) {
console.log('[' + steamID + '] Un-friended');
}
});

There's no "PendingInvitee" here.

Thanks, I was following the example in SteamFriends

Thanks for reminding, fixed in c3fca6a24d413eba78daa85c8c4bfc5c4dec8596.

I'm still getting the same error
TypeError: Cannot read property 'RequestRecipient' of undefined

Enums are on Steam, not on a SteamFriends instance. Enums.


Cannot connect to testnet

My testnet wallet is failing to connect to the testnet. Is the DNS seed tseed.paycoin.com down maybe?

Okay so this is only for new wallets trying to connect to the network. A wallet that has previously been on the network seems to be fine presumably because it is sourcing a list of peers from peers.dat rather than the DNS seed.
ping @HyperCommNet is this something you are able to get back online?

Okay so this is only for new wallets trying to connect to the network. A wallet that has previously been on the network seems to be fine presumably because it is sourcing a list of peers from peers.dat rather than the DNS seed.
ping @HyperCommNet is this something you are able to get back online?

Got same problem, obviously. Will try adding manually.

I believe this is resolved.

Looks good.

I don't think this fixed, or the DNS seed is offline again. I can't sync a new testnet wallet

Is is this testnet DNS seed connected in anyway to the testnet ledger, coincidentally also down...
@ligerzero459 here's some IPs to use in the meantime:
addnode=<IP_ADDRESS>
addnode=<IP_ADDRESS>
addnode=<IP_ADDRESS>

Yes I believe the seed is connected to the ledger. It certainly was working when I connected a new node without a peers.dat file but I haven't checked in the last 24 hours.

Basically we need a new DNS seed for the testnet.

Who was running the one we had, are they leaving it offline?
I am happy to contribute one but I will need to look into how to create one... is BIND9 what is being used?

@mitchellcash I believe yoshi was running the other and yes he's unable to restart it at this time. I'm not sure what you mean by BIND9 without googling it but there's really nothing complex about setting up a seednode.
The following is a copy of the xpydev mainnet seednode conf (with rpc info redacted), obviously a testnet node would also need testnet=1
daemon=1
dbcache=100
rpcuser=...
rpcpassword=...
maxconnections=25

Beyond that you'd need to decide the best way to start the node on bootup for your particular system/configuration (this obviously varies)

This doesn't seem like a proper DNS seeder though right? In any event I'll set this up tomorrow using your configuration :)

Okay I have a node running at testnet-seed.paycoin.mitchellcash.com using the above instructions.


Some fixes for the latest version of home-assistant

Added python virtual env
Updated mqtt to specific version 1
added init: false to fix startup error

Thanks so much!

No problem! My kids are happily using it as we speak, it was exactly what I was looking for


Make migrations python3 friendly

Under python3 makemigrations complains that request_profiler has
changes in models which are not reflected in its migrations.
It's caused by b'' strings.

Pushed to PyPI as v0.12.2


[FormKit Schemas] Conditional Options in Select Inputs don't work with Value/Label Arrays

Reproduction
https://formkit.link/45372b58b9085a12485dfbb868c06c18
Describe the bug
Description
FormKit Schemas and conditional options in select inputs don't seem to be parsing the options correclty if provided as value/label arrays.
It works when I am using conditional objects as string arrays, but not when I use them as array of value/label objects.
Working:
https://formkit.link/ed394dc69094cc223179abdef1a6ed92
Not working:
https://formkit.link/45372b58b9085a12485dfbb868c06c18
As discussed with @justin-schroeder this looks like its somehow interpreting it incorrectly:
https://discord.com/channels/793529058377072650/940340536747896842/995048809941966858

Workaround
This seems to work for now:
https://formkit.link/97175d1b55f32fd1a937cc09b9aaaf48
Environment
• Version Beta 8

This is fixed in the latest @next release (will be released in beta.10)


🐛 cel-go: update to pull in data race fix

Summary
cel-go: update to pull in data race fix
Related issue(s)
Fixes #2548

#2550 tracks going back to upstream

/approve

/lgtm


Bezier Animation to Cubic Spline Animation conversion error.

When animating an object in blender the user has the option to select an object's key frames in the Graph Editor, Right-Click to open the f-curve context menu, then set the interpolation mode to Bezier which will make the animation key frames CUBICSPLINE in the gltf file. I've been successful in reproducing the LINEAR interpolation mode in a model viewer I work on but I've run into a bug while trying to reproduce the CUBICSPLINE interpolation mode.
The bug is that CUBICSPLINE includes 3 vectors per key frame, 1 vector is simply the LINEAR interpolation vector, but the 2 supplementary vectors are always identical and are calculated using the left control point. The scale of the right control point has no effect on the exported gltf file even though scaling the right control point definitely has an effect on the animation.
I've wrote a simple python script to parses two gltf files. One that has a right control point scaled by 1.0 and another gltf that has the same right control point scaled by 0.1. The python script will find all the differences. It shows that there are no differences in the buffer of the gltf file. A repository containing the python file illustrating the problem along with the 2 blender and 2 gltf files can be found here (https://github.com/tbutynAvalon/gltf_cubic_spline_bug)
To Reproduce
Steps to reproduce the behavior:

Clone the repository on the page: https://github.com/tbutynAvalon/gltf_cubic_spline_bug
Open the 2 blender files and playback the animation to observe the difference in animation.
Run the python script and observe that the gltf files are the same.
(optional). Set a break point after buffer_views0_1 has been populated and observe that
buffer_views0_1[0] == buffer_views0_1[2]
buffer_views0_1[3] == buffer_views0_1[5]
buffer_views0_1[6] == buffer_views0_1[8]
where i is a multiple of 3:
buffer_views0_1[i] == buffer_views0_1[i+2]
Change the right control point in the blender file and export to get more interesting values for this optional step.

Expected behavior
I think what should happen is buffer_views0_1[i] != buffer_views0_1[i+2] and buffer_views0_1[i+2] is supposed to describe the right control point or some converted representation of the right control point.
Screenshots
N/A
.blend file/ .gltf
repository: https://github.com/tbutynAvalon/gltf_cubic_spline_bug
Version

OS: Windows
Blender Version 2.80 (2019-03-18)

Additional context
N/A

You are using a version that is 7 month old.
Please test again with recent build. (Official 2.80, or better, 2.81 on builder.blender.org

I recreated this bug in Blender 2.81 Beta and Blender 2.82 Alpha. I updated the files in the repository to contain the blender 2.81 versions on the .blender and .gltf files. A .gltf file has diff between 2.80 and 2.81 but the two separate files within a version remain identical.

This is copy-paste error
https://github.com/KhronosGroup/glTF-Blender-IO/blob/1f350035f1b219e27038547062ad3f06175f6f9c/addons/io_scene_gltf2/blender/exp/gltf2_blender_gather_animation_sampler_keyframes.py#L112-L114

Please do not fix it in master, I will manage to get in 2.81

Thank you! I appreciate the effort you all put into making blender better.


Add 20 papers on the Efficient and scalable vision section

Papers list retrieved from the official program. Github and arxiv links were verified.

Hi @pvtien96, 👋
Great job! All additions to the repo have been merged.


Mutex, timed_mutex, and call_once are not fully standard-compliant

Certain parts of the existing classes are not fully standard compliant. This comment lists them, lest we forget.

Classes mutex and timed_mutex do not satisfy the StandardLayoutType concept.

Adding the variable mOwnerThread violates the requirement for the inheritance tree to have only one class that defines non-static members.
Compliance is practical, but possibly undesired.

Recursively locking a non-recursive mutex is undefined behavior; recursive_mutex is a fully-compliant mutex class.
Compliance would either remove error-checking code or bloat the recursive mutex classes.

Function call_once does not actually call invoke.

Compliance may be impractical.

The implementer states that invoke is not defined in all cases.

According to cppreference
com, std::invoke is C++17, and std::call_once is C++11. If this is correct, the C++11 standard cannot require the implementation to use std::invoke

The cppreference.com site is somewhat inconsistent there. The call_once documentation indicates that invoke ought to be used before C++17, despite invoke being a C++17 feature. I think the intent is "call as if by invoke", which is still not quite satisfied by the current implementation.
I will adjust the issue accordingly.


Ability to edit the categories in the database

Add the functionality to edit the categories in the database like renaming categories or adding new ones.

I do not see this ability at all, where/how can you edit the categories?


UAN 2.5.5

Summary and Scope
[2.5.5] - 2022-10-05

Disable cray-ifconf
Fix edge case in uan_interfaces

Issues and Related PRs

Resolves CASMUSER-3101 CASMUSER-3102

Testing
tyr CAN
surtur CHN
shandy empty "child" cabinent
Risks and Mitigations
Disabling cray-ifconf reverts UAN to the previous working uan_interface role.
CASMUSER-3101 covers an SLS json structure edge case
Pull Request Checklist

[x] Version number(s) incremented, if applicable
[x] Copyrights updated
[x] License file intact
[x] Target branch correct
[x] CHANGELOG.md updated
[x] Testing is appropriate and complete, if applicable
[ ] HPC Product Announcement prepared, if applicable

Approved. Merge and build when ready.


When creating a new rule, the Similar-Rules feature is not working.

Always finds one rule and its always the same rule.

Fixed in e153ba38328a7c8b02d1bddf0fb80a41c6312cb0


Attach disks to Ceph servers

Hey folks!
Loving this project, may end up using it as my own ceph vagrant playground.
I saw that in your README you noted you haven't been able to figure out how to attach disks to each Vagrant host. I may have a patch that does that, but I'm not sure if you guys were looking for something more portable.
This may resolve https://github.com/carmstrong/multinode-ceph-vagrant/issues/15

@jeffWelling Does this work even though we are enumerating several VMs in the file? I think this was the issue I ran into with #15.
I'd rather get #15 working, since that Vagrant plugin should more gracefully handle the createhd and storageattach.


Remove 'tif' and 'tiff' from VIEWABLE_EXTS in pdsfile.py

Fixes https://github.com/SETI/pds-webserver/issues/70
Removed 'tif' and 'tiff' from the VIEWABLE_EXTS

Move https://github.com/SETI/pds-webtools/pull/86


Allow unknown fields in the old-ver IstioOperator during istioctl upgrade

Fix: #26475
[ ] Configuration Infrastructure
[ ] Docs
[x] Installation
[ ] Networking
[ ] Performance and Scalability
[ ] Policies and Telemetry
[ ] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure
Pull Request Attributes
Please check any characteristics that apply to this pull request.
[ ] Does not have any changes that may affect Istio users.

Skipping CI for Draft Pull Request.
If you want CI signal for your change, please convert it to an actual PR.
You can still manually trigger a test run with /test all

/test all

@elfinhe: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

release-notes_istio
40a10038157c369e4b5255e4d3a6004230ba59c6
link
/test release-notes_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

Below is the output:
$ istioctldev upgrade --set tag=1.7.0-rc.1 --set hub=docker.io/istio
Control Plane - ingressgateway pod - istio-ingressgateway-f9c86f54c-jfhrh - version: 1.6.5
Control Plane - pilot pod - istiod-7697c784c4-s48c6 - version: 1.6.5

2020-08-14T04:11:21.224503Z	warn	found 4 CRD of unsupported v1alpha1 security policy: [clusterrbacconfigs.rbac.istio.io rbacconfigs.rbac.istio.io servicerolebindings.rbac.istio.io serviceroles.rbac.istio.io]. The v1alpha1 security policy is no longer supported starting 1.6. It's strongly recommended to delete the CRD of the v1alpha1 security policy to avoid applying any of the v1alpha1 security policy in the unsupported version
Upgrade version check passed: 1.6.5 -> 1.7.0.

Upgrade check: Warning!!! The following IOPS will be changed as part of upgrade. Please double check they are correct:
addonComponents:
  grafana: map[enabled:false k8s:map[replicaCount:1]] ->
  kiali: map[enabled:false k8s:map[replicaCount:1]] ->
  prometheus: map[enabled:true k8s:map[replicaCount:1]] ->
  tracing: map[enabled:false] ->
components:
  egressGateways:
    '[#0]':
      k8s:
        service:
          ports:
            '[#0]':
              targetPort: -> 8080
            '[#1]':
              targetPort: -> 8443
installPackagePath: /tmp/istio-install-packages/istio-1.6.5/manifests ->
meshConfig:
  enablePrometheusMerge: false -> true
values:
  base:
    enableCRDTemplates: -> false
  gateways:
    istio-ingressgateway:
      meshExpansionPorts:
        '[0->?]': map[name:tcp-pilot-grpc-tls port:15011 targetPort:15011] ->
        '[2->?]': map[name:tcp-citadel-grpc-tls port:8060 targetPort:8060] ->
  global:
    istiod:
      enabled: true ->
    proxy:
      envoyStatsd: map[enabled:false host:<nil> port:<nil>] ->
    proxy_init:
      resources:
        limits:
          cpu: 100m -> 2000m
          memory: 50Mi -> 1024Mi
  grafana:
    image:
      tag: 6.7.4 -> 7.0.5
  kiali:
    tag: v1.18 -> v1.22
  prometheus:
    tag: v2.15.1 -> v2.19.2
  telemetry:
    v2:
      metadataExchange:
        wasmEnabled: -> false
      prometheus:
        wasmEnabled: -> false
  tracing:
    jaeger:
      tag: 1.16 -> 1.18

Confirm to proceed [y/N]? y
2020-08-14T04:11:28.648111Z	info	proto: tag has too few fields: "-"
✔ Istio core installed
✔ Istiod installed
✔ Ingress gateways installed
✔ Installation complete                                                                                ..........
Control Plane - ingressgateway pod - istio-ingressgateway-f9c86f54c-jfhrh - version: 1.6.5 does not match the target version 1.7.0
..........
Upgrade rollout completed. All Istio control plane pods are running on the target version.

Control Plane - ingressgateway pod - istio-ingressgateway-56f4746747-4pgb2 - version: 1.7.0
Control Plane - ingressgateway pod - istio-ingressgateway-56f4746747-hnhdq - version: 1.7.0
Control Plane - pilot pod - istiod-ddffdc7d7-jt9lx - version: 1.7.0

Success. Now the Istio control plane is running at version 1.7.0.

To upgrade the Istio data plane, you will need to re-inject it.
If you’re using automatic sidecar injection, you can upgrade the sidecar by doing a rolling update for all the pods:
    kubectl rollout restart deployment --namespace <namespace with auto injection>
If you’re using manual injection, you can upgrade the sidecar by executing:
    kubectl apply -f < (istioctl kube-inject -f <original application deployment yaml>)

/retest

I think it would be better to include the removed fields in the output warning as well, not just values update. But approve it now as it solves the unmarshal issue and unblocks the release

Agreed on @richardwxn 's comment.  I assume this would ignore if someone uses citadel:enabled:true without warning.

I hear 2 questions here.

If users explicitly specify citadel:enabled:true, will istioctl ignore it without warning?

The answer is no, because allowUnknownFields only applies to the version before upgrade, i.e., 1.6.x. When processing 1.7.x upgrade/install, unknownFields is not allowed, and an error will show up.

Can we include warning for removed fields?

I digged into the code, found this deprecation messages, then I thought we can Citadel here as well.
However, that is not feasible, because Citadel's protobuf is already deleted from the protobuf in 1.7. The deprecation message mechanism will not be able to find Citadel in v1alpha1.IstioOperatorSpec.

Today when users enter citadel:enabled:true, istioctl will return this error:
Error: failed to generate Istio configs from file [], error: could not unmarshal merged YAML: unknown field "citadel" in v1alpha1.IstioComponentSetSpec


Type aliases

type IntMap = Map(Int, Int)

@lpil would I try on this one if not done yet?
I'm pretty interested on your project, so hope to join on.

Hi @djKooks ! Contributions are very welcome :)
Do you have an idea of how you would implement this feature within the typer?

@lpil thanks! I'll check and let you know.


ExternalName services showing all pods

Steps to reproduce:

Create an ExternalName service in the cluster, e.g.:

apiVersion: v1
kind: Service
metadata:
  name: some-external-service
  namespace: default
spec:
  type: ExternalName
  externalName: example.com

Look in Capacitor UI

Expected behaviour: No pods are shown.
Actual behaviour: All pods in cluster are shown.

Dupe #113  (I searched, I swear)


Investigate alternatives to Google Analytics

Problem

Currently we have Google Analytics set up but disabled for the frontend. GA requires careful configuration to avoid PII collection, requires a GDPR notice and strategy/cookie consent configuration. It also has massive numbers of features we will likely never use. WordPress and Openverse users may also have questions about Google's commitment to user privacy that could get in the way of folks trusting Openverse. It is also unused by WordPress.org and I don't think we should be the ones to introduce it there.
Description

Let's look into alternatives like https://www.goatcounter.com/ or https://plausible.io/.
Alternatives

Get rid of these types of things altogether. Do we need this for any specific reason? Are there alternatives that wouldn't require integrating with a 3rd party service (or self-hosting yet another service)?
Implementation

[ ] 🙋 I would be interested in implementing this feature.

This is a great idea, just noting that wordpress.org definitely uses Google analytics

Investigating further and GoatCounter could even be used to gather API usage data through a django middleware.


ISPN-1854 Get server mbeans documented and included in RHQ plugin xml

https://issues.jboss.org/browse/ISPN-1854
5.1.x branch: t_1854_5

integrated.


Ensure Java 17 and 21 compatibility

This PR ensure Avro Xml Mapper is compatible with Java 17 and 21

@PaulSec @sebastienviale QG is failing. We should deal with it in another PR, what do you think?


CASMINST-5352: Create troubleshooting page for unused drives on storage NCNs

Description
After deploying NCNs during a fresh install, a set of health checks are performed. Prior to csm-1.3, this included a manual check of the OSDs on utility storage nodes. In csm-1.3, this was replaced by a Goss test. However, the manual procedure (and associated remediation steps) should still be available in the documentation as a troubleshooting page. This PR creates that page, and links to it from the appropriate step during the install.
The content of the page is essentially copied from the 1.2 docs, fixing only linting-type issues.
Checklist Before Merging

[X] If I added any command snippets, the steps they belong to follow the prompt conventions (see example).
[X] If I added a new directory, I also updated .github/CODEOWNERS with the corresponding team in Cray-HPE.
[X] My commits or Pull-Request Title contain my JIRA information, or I do not have a JIRA.

/backport main


Reporting of Errors (ValiError Format compared with ZodError Format)

We currently use zod and wanted to give this library a try to utilize the tree-shakeability features of it.
In comparsion valibot seems to be less expressive compared to zod when dealing with parsing errors . Are there plans to change/improve that?
Zod parse error:
ZodError: [
  {
    "code": "invalid_string",
    "validation": "datetime",
    "message": "Invalid datetime",
    "path": [
      "timestamp"
    ]
  }
]

Valibot parse error:
ValiError: Invalid date-time

When parsing, valibot should throw a ValiError that contains an array of Issue if it encounters an error.
From the docs:

The parse method will throw a ValiError if the input does not match the schema.

type Issue = {
  reason: IssueReason;
  validation: string;
  origin: IssueOrigin;
  message: string;
  input: any;
  path?: PathItem[];
  issues?: Issues;
  abortEarly?: boolean;
  abortPipeEarly?: boolean;
  skipPipe?: boolean;
}

Printing error.issues you should find the kinda result you're expecting :)

Please take a look at the following guide and contact me if you need further help. I am happy to assist you. https://valibot.dev/guides/issues/

Thanks for the super fast response, appreciate that!
Yeah I saw that the inside object is pretty identical to the structure, I like that :)
For some validation test scenarios we like to use matchininlinesnapshot and there it was quiet a difference (see below).
  it("should fail with invalid timestamp", async () => {
    const event = {
      timestamp: "2020-01-01T00:00:00.000",
    };

    await expect(valibotHandler(event)).rejects.toThrowErrorMatchingInlineSnapshot(
      `"Invalid timestamp"`
    );
  });

  it("should fail with invalid event", async () => {
    const event = {
      timestamp: "2020-01-01T00:00:00.000",
    };
    await expect(zodHandler(event)).rejects.toThrowErrorMatchingInlineSnapshot(`
      "[
        {
          "code": "invalid_string",
          "validation": "datetime",
          "message": "Invalid datetime",
          "path": [
            "timestamp"
          ]
        }
      ]"
    `);
  });

But an unhandled ValiError during runtime looks pretty similar to the similar serialized structure, so I guess I'll close the issue. Thanks!


Self enforcement doesn't work

In https://github.com/althea-mesh/althea_rs/commit/e3bd4dcbb22e0ed580b2d13d86d892febc0cc84a I tried to have devices limit themselves to the free tier when they had a low balance.
I thought limiting the wg_exit tunnel with tbf would be sufficient but it turns out that doesn't do what I expect and somehow traffic can traverse that tunnel without being subject to that rule.

I've realized that my approach here is entirely wrong thanks to @jtremback  I've realized that we can't rate limit incoming (or download) traffic using our current method and tc

after further discussion with @jtremback  it seems the best approach is to limit upload on the local side and use the low balance warning parameter to tell the exit to limit download speed exit side. This will limit usage during low balance periods voluntarily and achieve our most important goal of allowing for clients downstream of relays to be immune to a relay that's both not depositing and heavily using themselves.


[ar] Add Arabic technique glossary

Adding Arabic technique glossary to unify and avoid using different translation for the same English word.
Unified Kubernetes translation based on the discussion : https://github.com/kubernetes/website/issues/45078

LGTM for Arabic, thank you @mboukhalfa

Thanks @mboukhalfa. It may be more convenient and better Git hygiene to move updates to the existing docs to a new PR.

LGTM for Arabic!
Thank you fro the efforts @mboukhalfa

LGTM for Arabic!
Thank you fro the efforts @mboukhalfa

The idea is to unify the words once this PR gets merged so we do not have merged content conflicting with the technical glossary

@mboukhalfa I understand your sentiment to make sure the development branch is "clean" in this way, but I would like to respectfully rebut, for several reasons:

dev-1.29-ar.1 is a development, unpublished branch. Since its content is not visible publicly, we can afford to have merged work in progress.
As more and more documents get added, it will become impractical to combine all updates to terminology localization in this one PR. What I fear is that way, this PR will not merge until after everything else is done, which I do not think is the intuitive order of operations.
Repeating the above, I fear that we regress into a "waterfall" model where all PRs are "pending" because we are waiting for "perfectly consistent" PRs. This will slow down progress and defeats the purpose of having a development branch and a Git history.
On that topic, as an aside, I believe that this project is set up with the ability to "squash" commits when PRs are merged, so I believe that we can indeed create incremental commits within PRs, saving the "history" of the work. As @seifrajhi pointed out in #45047, in reference to #40184, force-pushes break Netlify's rendering behavior.

The content of this PR (minus the last set of changes) looks great to me, and I really appreciate your effort. I am encouraging to merge it ASAP so that we can see real, even if incremental, progress.

Also, for reference, this PR pours into the effort: #44682.

/lgtm

/assign @seifrajhi
Will you please take a look and lgtm?

/lgtm

/lgtm


Storage expansion

Q
A

issue or Enhancement
issue

Changelog updated?
no

License
MIT

What's in this PR?
Explain what the changes in this PR do.
Checklist

[ ] Updated CHANGELOG.md to describe BC breaks / deprecations | new feature | bugfix

I'm getting the following error when checking out the StorageExpansion branch (and using composer update / composer install):
`In DBALException.php line 131:
An exception occurred while executing 'SELECT count(id) FROM ?' with params ["plenty_backlog"]:
SQLSTATE[42000]: Syntax error or access violation: 1064 You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near ''plenty_backlog'' at line 1
In Connection.php line 847:
SQLSTATE[42000]: Syntax error or access violation: 1064 You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near ''plenty_backlog'' at line 1
`

@florianliebig thanks for testing! i have pushed a fix for this problem :)

Is there any smart way to add name2 and name3 to existing products? I did a new transfer but these new attributes are not in the item set. Or is it necessary to delete products and do a full new sync?  Maybe also a point for documentation.

@florianliebig after installing the new code a full sync (plentyconnector:process product --all) should be all you need to do. I was not able to reproduce your problem on my installation :(

@florianliebig i think i found a potential error. Could you recheck on your system?

@jochenmanz It works now on the existing installation thank you very much!


[testhelpers] Add helpers for Galera (create, delete, get)

Looks good. I'm just wondering where will we use these. Currently service operators only depend on the k8s Service object (see CreateDBService) to exists during envtest testing.

I plan to use it in the openstack-operator where we create the db instances

Looks good. I'm just wondering where will we use these. Currently service operators only depend on the k8s Service object (see CreateDBService) to exists during envtest testing.

I plan to use it in the openstack-operator where we create the db instances

Ohh, good point. I will be happy to review envtest coverage for openstack-operator!


Update README.md

Add: schedule variable, as it is a feature most people are gonna want to customize
📃 Description
Most people are going to want to customize how often a speedtest is triggered. For several days I considered forking  this and implementing it myself, until i found out it was already possible. The primary focus of this change is to easily show newcomers that it is indeed possible.
🪵 Changelog
✏ Changed

Added SPEEDTEST_SCHEDULE to README.MD (defaults to 1 hour).

Just my 2 cents;
Not sure if using the hour mark as a good idea. There have been multiple issues with reports that tests on the hour mark like 9:00, 10:00 etc give less results then a few minutes later due to congestion on network or Speedtest Server.  Just to not cause more of these reports ourself ;)

That is a good point. I suppose it doesn't really matter what it is set to originally, personally i trigger a speedtest every 15 minutes.

Yeah honestly. I just wanted to have this added to the readme so that new users are aware that this is customizable. I don't know how many days/weeks i spent troubleshooting this, at some point i considered triggering the speedtest myself and just appending the data to database, but thankfully i found that environment variable hehe.

Just my 2 cents; Not sure if using the hour mark as a good idea. There have been multiple issues with reports that tests on the hour mark like 9:00, 10:00 etc give less results then a few minutes later due to congestion on network or Speedtest Server. Just to not cause more of these reports ourself ;)

How about we can default something like SPEEDTEST_SCHEDULE=5,15,25,35,45,55 * * * *
While this version can be slightly harder to understand to some, this way we can avoid the full hour mark. Sure would've been nice to avoid those extra reports.

@Kristoffeh I'm happy to add the option in but can you comment the line out so it's "disabled" by default with no schedule suggestion?