Is it possible to extend the tailwind theme with the twindv1 plugin?

I tried adding the following to the twind.config.ts file:
import { defineConfig, Preset } from "https://esm.sh/<EMAIL_ADDRESS>import presetTailwind from "https://esm.sh/<EMAIL_ADDRESS>import presetAutoprefix from "https://esm.sh/<EMAIL_ADDRESS>
export default {
  ...defineConfig({
    presets: [presetTailwind() as Preset, presetAutoprefix()],
    theme: {
      extend: {
        colors: {
          primary: '#4fa846'
        }
      }
    }
  }),
  selfURL: import.meta.url,
};

Is this the correct way to extend the tailwind theme using the twindv1 plugin?
It doesn't seem to work for me and I get a type error in the main.ts file when trying to run it:
import { start } from "$fresh/server.ts";
import manifest from "./fresh.gen.ts";

import twindPlugin from "$fresh/plugins/twindv1.ts";
import twindConfig from "./twind.config.ts";

await start(manifest, { plugins: [twindPlugin(twindConfig)] });

Argument of type '{ selfURL: string; darkMode?: DarkModeConfig; darkColor?: DarkColor<BaseTheme & { extend: never; }> | undefined; theme: ThemeConfig<BaseTheme & { extend: never; }>; ... 6 more ...; finalize: Finalize<...>[]; }' is not assignable to parameter of type 'Options'.
  Types of property 'darkColor' are incompatible.
    Type 'DarkColor<BaseTheme & { extend: never; }> | undefined' is not assignable to type 'DarkColor<BaseTheme> | undefined'.
      Type 'DarkColor<BaseTheme & { extend: never; }>' is not assignable to type 'DarkColor<BaseTheme>'.
        Types of parameters 'context' and 'context' are incompatible.
          Type 'Context<BaseTheme>' is not assignable to type 'Context<BaseTheme & { extend: never; }>'.
            The types returned by 'theme()' are incompatible between these types.
              Type 'BaseTheme' is not assignable to type 'BaseTheme & { extend: never; }'.deno-ts(2345)

It seems like they have some typing issues all over the place. If you force it like this:
    theme: {
      extend: {
        colors: {
          primary: "#4fa846",
        },
      },
    } as ThemeConfig,

then the error goes away.
The theme property of a TwindUserConfig is typed like this:
theme?: Theme | ThemeConfig<BaseTheme & ExtractThemes<Theme, Presets>>;

and a ThemeConfig is:
declare type ThemeConfig<Theme extends BaseTheme = BaseTheme> = PartialTheme<Theme> & {
    extend?: PartialTheme<Theme>;
};

Let me know if this helps.


claim about image sha256 not true from docker v1.12

1.我的运行场景:
2.我的需求的具体情况（feature request || bug）:
3.重现该问题的步骤:
1.
2.
3.
4.描述您获得的实际情况:
The README.md states:
When the Docker image is pushed to Registry, a hash will be generated but the value will not change with image content. We calculate SHA256 hash of the tar file content saved with docker save, which only represents the certain image.
I think this statement is not true anymore for docker version from 1.12.
Since docker v1.12 the sha256 returned is the sha256 sum over the configuration object which contains the sha256 sums of each layer.
Layers are validated with downloaded.
5.描述您预期的情况:
Only the sha256 digest of the images needs to be linked to the image tag on the block chain.
Further more only a sha256 digest over the image digest and the address and the image tag need to be stored on the block chain.
If a verifier then wants to verify if the image is ok he can take the publisher address, image tag and the image digest and calculate the sha256 over that and look on the blockchain if it exists.
6.您认为的其他有价值的信息:

ping @Revolution1 @jjyr @hainesc


[bug]: Duplicate Neon-Gradient-Card installation component

Describe the bug
Visual website bug such that when you try to get the manual installation for 'Neon-Gradient-Card' on the website, it will give you two of the same component making it somewhat confusing. Visual bug, nothing functional. Minor.
Affected component/components
Neon-Gradient-Card
How to reproduce
Go to https://nyxbui.design/docs/components/neon-gradient-card.
Scroll to installation
Click 'Manual'
Click 'Expand'
if you scroll long enough you'll see a duplicate installation component on the website.
Codesandbox/StackBlitz link
https://nyxbui.design/docs/components/neon-gradient-card
Logs
N/A

System Info
Dell XPS 13 9310 - Windows 10 - Brave Browser Version 1.73.89 Chromium: 131.0.6778.69 (Official Build) (64-bit)

Before submitting

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

Thank you for reporting this visual duplication issue with the Neon-Gradient-Card manual installation section. I'm happy to inform you that the bug has been fixed - there's no longer a duplicate component showing up in the documentation. Your detailed bug report helped us identify and resolve the problem quickly. Thanks for helping improve the documentation! 🙌

Glad to help :).


fix example typos and relative paths

Description
Just typo and path fixes for the examples files
Checklist

[X] Code compiles correctly
[ ] Created tests which fail without the change (if possible)
[X] All tests passing
[ ] Extended the README / documentation, if necessary
[ ] Added myself / the copyright holder to the AUTHORS file

Coverage remained the same at 95.627% when pulling 65a6a2f9376974793eb2e929758f2f81649c3229 on mandeldl:master into bf03b6bed331319c2c7f3a2e327df9bdeac0f44c on lapwinglabs:master.


Catchup not working

Whenever I try to play a catchup programme it doesn't play properly. Sometimes only sound with black screen and sometimes no sound no video. Already tried relogin, cache clear but not working.

7 Days Catchup is working fine. No issues seen

Catchup is showing but it is not playing. Is it playing on yours.?

@Alexios775 Catchup is showing but it is not playing. Is it playing on yours.?

Yes, Playing Fine.
7 Days Catchup is playing great on the channels where it is available both in HLS stream and MPD (MPEG-DASH) stream. I have checked it today (9 Nov, 2023) on FireTV Stick.


The "Home" background video is not displayed in mobile browsers.

Description:
When the user mobile browsers the background video is not displayed.
-Env:
Crafter Hotel_Info Blueprint
Iphone 7 IOS 10.3 Safari
Android 7.1
Google Chrome 55.0.2883.79
Firefox 54.0
Also happens in:
Device: MacBook Pro
Safari Version 8.0.5 (10600.5.17)
REPRODUCIBILITY LEVEL:
Always
STEPS:
Steps

Open the Crafter studio with the Hotel_Info site.
Go to the “Home” section.
Review the section.

CURRENT RESULT:
The application is not displaying the background video of the “Home” section in mobile browsers.
EXPECTED RESULT:
The application should display the background video of the “Home” section in mobile browsers.

Known issue as specified in the readme file

Bug fixed.


background

Tried everything that i know and i dont release how can we run the server in background?  with screen works but then we have one more process in bg.
Thanks in advanced

It shouldn't be any different than any other program. Can you provide more details about what you're trying?
Some users contributed systemd files a while back which may be useful to you:
https://github.com/boringproxy/boringproxy/tree/master/systemd

Closing for now feel free to reopen if you have further questions.


Adding support for Tuple<T1, T2, ...> as option/value parsing target

I want to be able to parse this:
$ app --webserver localhost 8080

to:
class Options {
  [Option] public Tuple<string, int> WebServer { get; set; }
}

Opinions? cc/ @mizipzor @nemec @gimmemoore

This seems to go against the standard. To have it accepted you would have to quote the "localhost 8080" part since it contains a space. But it just looks weird. Although in this specific case, an uri, the solution is obvious since it has an accepted syntax localhost:8080. Off the bat, I cannot come up with a case where the above makes sense.

@mizipzor, I'm not agree, why one have to quote input?
In this case just think a tuple like a heterogeneous typed sequence. As long as T1, T2, Tx... types of Tuple<...> match input, the parser will collect values, convert them and create appropriate Tuple instance.
I think it's good also for forcing an order without effort. E.g. 8080 before localhost will generate a conversion error.
Probably will complicate the parsing kernel, but I thik it could be useful for some scenarios.
Obviously it's a feature that must be used correctly like in my sample.
E.g.: if I want capture exactly 10 int I'll use an [Option(Min=10, Max=10)] mapped to IEnumerable<int> not a Tuple<int, int, int...> (also if it works the same); but this is a design problem of the developer and doesn't concern us (I think).

My only argument is that its simply not how the original getopt works, to have spaces you either quote them or escape them. As long as that is still required I have nothing against adding support for tuples. People know getopt and having the library behave like getopt is a feature (and a good one at that). If the library works "like getopt with a few tweaks", it is not as attractive.

The only benefit I can see here is the ability to specify multiple tuple options in a single argument string, otherwise there doesn't seem to be much benefit over using a Verb with ValueAttributes: $ app webserver localhost 8080
Additionally, this syntax maps a single OptionAttribute to multiple parsed values (T1...TN) which makes configuration of each inner option more difficult. For example, it would be nice if your sample webserver defaulted to 8080 if a port wasn't provided but I'm not sure how that could be specified.
Maybe one compromise could be to let the OptionAttribute be tagged on custom classes (like Verbs) but that contain only ValueAttributes (or create a new MultiValueOptionAttribute), which could allow you to re-use the parsing logic that already exists for Values.
Another option, slightly more verbose, could be to add the ability to define "option groups" that require all options in the group to be defined or none at all:
$ app --server localhost --port 8080  # allowed
$ app --server localhost              # fail, missing --port
$ app --port 8080                     # fail, missing --server

Personally, I agree with @mizipzor that keeping consistency with getopt is a very good thing (especially now that C# is becoming more cross-platform) and for this particular example adding a colon separator (localhost:8080) is the more accepted option.

@nemec, thanks for comments!
Personally I see a Tuple<T1, T2, ...> as compound data type. I don't see it as something against getopt()...
A true thing instead is that two Value attribute can mimic this behaviour in some way.
Any other comment?

No more comments, I've said my part and still hold the same opinion.


Improvement | Text visibility

Welcome to Gani Tobacco
Choose Life, Not Tobacco: Ignite Hope, Not Cancer.

To enhance readability and ensure your message is effectively communicated, it's essential to select an appropriate color scheme for the text on your homepage. By choosing colors that contrast well with the background, you can significantly improve readability and make your message more impactful.

I can work on it if you assign me!

I can work on it if you assign me!

Hi Hassan, you can work, fork the repo and do. Also create PR so I can merger your code as well.


postContent.matchAll(...) is not a function or its return value is not iterable

I get the error postContent.matchAll(...) is not a function or its return value is not iterable. Looking at the XML, there's only two posts and neither have images. (Most of the XML is pages rather than posts.)
const matches = [...postContent.matchAll(/<img[^>]*src="(.+?\.(?:gif|jpe?g|png))"[^>]*>/gi)];

Are you using Node.js v12.14 or later?

Are you using Node.js v12.14 or later?

Hi @lonekorean, No, I agree, that's the problem: there's no matchAll in Debian Stable. Sorry for the noise.

Hi @lonekorean, No, I agree, that's the problem: there's no matchAll in Debian Stable. Sorry for the noise.

@RalphCorderoy no worries. I should probably add a check in code and give a more helpful error message. :)

@RalphCorderoy no worries. I should probably add a check in code and give a more helpful error message. :)


PPP changes

Pull Request Test Coverage Report for Build 447

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

Totals

Change from base Build 446:
0.0%

Covered Lines:
1859

Relevant Lines:
3349

💛  - Coveralls


Does this support windows 10?

because I get an error in this section
><EMAIL_ADDRESS>dev
> nodemon ./bin/index.js

[nodemon] 3.0.1
[nodemon] to restart at any time, enter `rs`
[nodemon] watching path(s): *.*
[nodemon] watching extensions: js,mjs,cjs,json
[nodemon] starting `node ./bin/index.js`
fs.js:47
} = primordials;
    ^

ReferenceError: primordials is not defined
    at fs.js:47:5
    at req_ (C:\node\oimi-live-main\node_modules\natives\index.js:143:24)
    at Object.req [as require] (C:\node\oimi-live-main\node_modules\natives\index.js:55:10)
    at Object.<anonymous> (C:\node\oimi-live-main\node_modules\vinyl-fs\node_modules\graceful-fs\fs.js:1:37)
    at Module._compile (node:internal/modules/cjs/loader:1105:14)
    at Object.Module._extensions..js (node:internal/modules/cjs/loader:1159:10)
    at Module.load (node:internal/modules/cjs/loader:981:32)
    at Function.Module._load (node:internal/modules/cjs/loader:822:12)
    at Module.require (node:internal/modules/cjs/loader:1005:19)
    at require (node:internal/modules/cjs/helpers:102:18)
[nodemon] app crashed - waiting for file changes before starting...

Of course， it’s support. This error mean you node version is higher. You can try to change your node version to 16 below
---- Replied Message ----
| From | Maldini @.> |
| Date | 03/17/2024 22:49 |
| To | helson-lin/oimi-live @.> |
| Cc | Subscribed @.***> |
| Subject | [helson-lin/oimi-live] Does this support windows 10? (Issue #1) |
because I get an error in this section

@.*** dev
nodemon ./bin/index.js

[nodemon] 3.0.1
[nodemon] to restart at any time, enter rs
[nodemon] watching path(s): .
[nodemon] watching extensions: js,mjs,cjs,json
[nodemon] starting node ./bin/index.js
fs.js:47
} = primordials;
^
ReferenceError: primordials is not defined
at fs.js:47:5
at req_ (C:\node\oimi-live-main\node_modules\natives\index.js:143:24)
at Object.req [as require] (C:\node\oimi-live-main\node_modules\natives\index.js:55:10)
at Object. (C:\node\oimi-live-main\node_modules\vinyl-fs\node_modules\graceful-fs\fs.js:1:37)
at Module._compile (node:internal/modules/cjs/loader:1105:14)
at Object.Module._extensions..js (node:internal/modules/cjs/loader:1159:10)
at Module.load (node:internal/modules/cjs/loader:981:32)
at Function.Module._load (node:internal/modules/cjs/loader:822:12)
at Module.require (node:internal/modules/cjs/loader:1005:19)
at require (node:internal/modules/cjs/helpers:102:18)
[nodemon] app crashed - waiting for file changes before starting...
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.Message ID: @.***>


Daedalus unacceptable bug

Hi,
Using Daedalus version 5.2.0
Today, after a 1 ada transaction to myself, to empty the rewards, Daedalus displayed an erroneous ADA balance, adding twice the stake rewards to the addr1 balance.
There was no way to clear this error, except b y quitting Daedalus and starting it up again.
In parallel, I checked my balance on other client wallets, suc as Ledger live, which displayed the correct balance.
So I guess what is wrong is that Daedalus does not wait the end of the transaction and/or the correct sync with the blockchain to display the account balance.
This needs to be adressed as a priority as it destroys any credibility in Daedalus being a serious and stable wallet client.

@HuguesD Talk to Ledger Support on the Ledger Live support so that your issue can be looked into.
Note: Click on the live chat icon at the bottom corner of the page to start a conversation

@HuguesD Talk to Ledger Support on the Ledger Live support so that your issue can be looked into.
Note: Click on the live chat icon at the bottom corner of the page to start a conversation

The  mentioned bug is a daedalus bug and has nothing to do whatsoever with Ledger....


[Aladdin Parser] Parse aladdin.py into commands _help.py

Can you write some description about the whole process and how the file will be used?


Process handle not refreshed and becomes stale

Greaka at 6/25/2020 6:24:12 AM +00:00

okay, regarding <#534492012263636992>:
the main window had a faulty handle, i assumed that bhud catched the login handle and didnt update when the game started
so my temporary fix was changing public IntPtr Gw2WindowHandle { get; private set; } to
        public IntPtr Gw2WindowHandle
        {
            get
            {
                this.Gw2Process.Refresh();
                return this.Gw2Process.MainWindowHandle;
            }
        }

it refreshes the cached process object before accessing the windowhandle
before merging this into the dev branch, we should proabably think about a better way as there are 8 references, sometimes in controls that use this handle

Source discussion from the #💢help channel: https://discordapp.com/channels/531175899588984842/534492012263636992/725566715466874892
Logs:
blishhud.20200624-211059.log

21:23:45.7555|WARN|Blish_HUD.WindowUtil|GetClientRect failed with error code 1400.

REF: https://discordapp.com/channels/531175899588984842/536970543736291346/725597194371203092

It is not slow enough to be a major concern for us. For now, it would probably be fine if we put it in the update loop. I just want to avoid refreshing it multiple times per update. The Refresh on error sounds good too.


hotfix -- Synapse's CREATE VIEW refuses database name; cleanup

Synapse's CREATE VIEW refuses database name
standardize on {{ from_relation.include(database=False) }} rather than {{ from_relation.schema }}.{{ from_relation.identifier }}

Awesome! Thanks!
Just commenting that I just came across this exact issue. Reinstalled from the repo and can confirm this fixes the issue on a separate side.
You have to uninstall as this still has last release tag.
python -m pip install git+https://github.com/swanderz/dbt-synapse --upgrade
BTW @swanderz  are you thinking of enabling issues on this repo?

wow! a user who isn't me or @NandanHegde15? 😆 I didn't even know I had the option to enable issues. I assumed I couldn't because it is a fork... just did now.
I haven't yet pushed a new release to PyPI, so you'll have to install from GitHub for the time being. I'm working on getting the adapter passing all tests in https://github.com/fishtown-analytics/dbt-adapter-tests. I should be done in a few days, after which I'll publish a new package version to PyPI.
Please let us know of any other issues you're having and we can try and get any fixes included in the next release.

Awesome... Yeh, I'm just making the switch over from the mikalene dbt-sqlserver stuff atm.
Really appreciate the clarity of the README so far. I'd just got authenticated when I had the view issue so I was very glad to find this hotfix!

Thanks, no more problems yet.
Will let you know if any come up as i use more dbt feature, but looking
great so far
On Sat, 26 Sep 2020, 00:05 Anders<EMAIL_ADDRESS>wrote:

@alittlesliceoftom https://github.com/alittlesliceoftom just cut a new
release and it's on PyPI. LMK if you find any other issues.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/swanderz/dbt-synapse/pull/14#issuecomment-699204591,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACVRBJJ7MVE2JCH53EAYP5TSHUO35ANCNFSM4RXU7KEA
.


[WIP] feat: add cargo-husky

Add cargo-husky
Changes suggested by cargo clippy

Pull Request type

Please check the type of change your PR introduces:

[ ] Bugfix
[ ] Feature
[x] Code style update (formatting, renaming)
[x] Refactoring (no functional changes, no API changes)
[ ] Build-related changes
[ ] Documentation content changes
[ ] Testing
[ ] Other (please describe):

What is the current behavior?

Issue Number: N/A
What is the new behavior?

Does this introduce a breaking change?

[ ] Yes
[x] No

Other information

I added cargo-husky to the project and applied the changes suggested by cargo clippy. I'm not sure if there was any discussion about where to place the husky configuration, but I left it in the beerus-core module. If you think this pull request is unnecessary, feel free to close it. What do @danilowhk  and @drspacemn  think about this? Thankss :)


feature: doctest support

Did you check the tree-sitter docs?

[x] I have read all the tree-sitter docs if it relates to using the parser

Is your feature request related to a problem? Please describe.
Doctests are typically embedded inside docstrings in Python code, e.g.
def factorial(n):
    """Return the factorial of n, an exact integer >= 0.

    >>> [factorial(n) for n in range(6)]
    [1, 1, 2, 6, 24, 120]
    """

Today, the grammar does not parse docstrings and is oblivious to the doctests potentially lurking inside.
Describe the solution you'd like
The grammar should parse these doctests so that we can have proper editing support.
Describe alternatives you've considered
No alternatives have been considered.
Additional context
Some more discussion on the topic:

https://github.com/microsoft/pylance-release/discussions/4196
https://www.reddit.com/r/neovim/comments/zbmqcc/is_it_possible_to_have_rust_doc_test_comments/

This would be better suited for injections, so you can then filter out the actual docstrings with queries, and inject "pydoc" into them. Take a look at jsdoc or luadoc for inspirations if you'd like to embark on that! But nothing is actionable here for that, so I'll close this out

Documentation on language injections: https://tree-sitter.github.io/tree-sitter/syntax-highlighting#language-injection.


feat(AlgebraicGeometry): pre-requisites of description of topological space of pullback

Prerequisites for #17767

bors r+


Cannot promote card from forecast when given the option to seize relic.

As title; when you meld a card from hand, if that meld gives you the option to seize a relic, the "choose card to promote" step is skipped.

This was a really good find! Thanks.


Update Tinkerbell chart with new image configuration

This PR updates the EKS-A owned Tinkerbell chart to be compatible with the images updated in https://github.com/aws/eks-anywhere-build-tooling/pull/1671.
The images specified in the chart are overwritten by EKS-A CLI on deployment with values from the Bundle so their value is irrelevant here.

you have to update the GIT_TAG to the chart version here main/projects/tinkerbell/tinkerbell-chart/helm/sedfile.template and here main/projects/tinkerbell/tinkerbell-chart/GIT_TAG

Good catch. Thank you.


Can't add augmentation to speech recognition.

Can't add augmentation to speech recognition. Docs is very confusing.
What is your question?
What have you tried?
Looked at this example but it didn't work.
Show an example of working code for adding augmentation in transform.
What's your environment?

lightning-flash==0.6.0

My example for augmentation.
`
from flash.core.data.io.input_transform import InputTransform
from dataclasses import dataclass
@dataclass
class BaseSpeechInputTransform(InputTransform):
# custom transforms.
augmentation = Compose([
    Resampler(resample_rate=16000),
    PreEmphasisFilter(),
    AddGaussianNoise(min_amplitude=0.001, max_amplitude=0.02, p=0.3),
    Mp3Compression(backend="pydub", p=0.3),
        ])

def sample_augmentator(self, sample, resample_rate=16000):
    sampling_rate = sample['metadata']['sampling_rate']
    sample['input'] = self.augmentation(sample['input'], sampling_rate)
    sample['metadata']['sampling_rate'] = resample_rate
    return sample

def per_sample_transform(self):
    return self.sample_augmentator

`

Nice! @dayvagrant this is awesome!!! Would you be interested in opening a PR to add this example to our speech recognition docs? We went through a refactor recently which is why the old example doesn't work. Would be very interested to hear your thoughts on the new API if there's anything you'd like to share :smiley:


[Safe Apps] Compound app - Icon fails to load

There is no steps, After a while it stops loading the icon. The Icon won't display properly in the compound app tab or the transactions.

https://gnosis-safe.io/app/#/safes/0xB7526726EE1031AF7d700B0351EE5a20dd910177/transactions

Could you add your browser version? As cors is enforced by the browser this is important.

Chrome v 81.0.4044.122 for Win10 64bits

@francovenica  is this still happening?

@francovenica did you experience this again?

No, I tried again and I had no issues.


Added groupId as org.opensearch.plugin in pluginZip publication

Signed-off-by: prudhvigodithi<EMAIL_ADDRESS>Description
Added groupId as org.opensearch.plugin in pluginZip publication
Issues Resolved
https://github.com/opensearch-project/job-scheduler/issues/225
Check List

[ ] New functionality includes testing.

[ ] All tests pass

[ ] New functionality has been documented.

[ ] New functionality has javadoc added

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

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

Codecov Report

Merging #251 (5823b52) into main (b6e4661) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##               main     #251   +/-   ##
=========================================
  Coverage     53.19%   53.19%
  Complexity       65       65
=========================================
  Files             8        8
  Lines           438      438
  Branches         50       50
=========================================
  Hits            233      233
  Misses          186      186
  Partials         19       19

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

Sure @saratvemulapalli  I do see the error > Task :integTest FAILED which is not related to this change, @joshpalis can you please confirm that?

@prudhvigodithi Integration and BWC tests have passed locally for me, might just be a flaky test but re-running the failed jobs have also failed. I shall spend some time digging into this.

@prudhvigodithi @joshpalis this commit on OpenSearch could have broken the changes[1].
Looks like org.apache.http was updated to a newer version which is why its failing.
[1]https://github.com/opensearch-project/OpenSearch/pull/4459/files

I was able to re-produce the problem on local box.
/gradlew :opensearch-job-scheduler-sample-extension:compileTestJava
=======================================
OpenSearch Build Hamster says Hello!
  Gradle Version        : 7.5.1
  OS Info               : Linux 5.4.0-1037-aws (amd64)
  JDK Version           : 14 (Private Build JDK)
  JAVA_HOME             : /usr/lib/jvm/java-14-openjdk-amd64
  Random Testing Seed   : 990C0B537CE645E9
  In FIPS 140 mode      : false
=======================================

> Task :opensearch-job-scheduler-sample-extension:compileTestJava
/home/ubuntu/prudhvi-groupId-js/sample-extension-plugin/src/test/java/org/opensearch/jobscheduler/sampleextension/SampleExtensionIntegTestCase.java:8: error: package org.apache.http does not exist
import org.apache.http.Header;
                      ^
/home/ubuntu/prudhvi-groupId-js/sample-extension-plugin/src/test/java/org/opensearch/jobscheduler/sampleextension/SampleExtensionIntegTestCase.java:9: error: package org.apache.http does not exist
import org.apache.http.HttpEntity;
                      ^
/home/ubuntu/prudhvi-groupId-js/sample-extension-plugin/src/test/java/org/opensearch/jobscheduler/sampleextension/SampleExtensionIntegTestCase.java:10: error: package org.apache.http.entity does not exist
import org.apache.http.entity.ContentType;
                             ^
/home/ubuntu/prudhvi-groupId-js/sample-extension-plugin/src/test/java/org/opensearch/jobscheduler/sampleextension/SampleExtensionIntegTestCase.java:11: error: package org.apache.http.entity does not exist
import org.apache.http.entity.StringEntity;
                             ^
/home/ubuntu/prudhvi-groupId-js/sample-extension-plugin/src/test/java/org/opensearch/jobscheduler/sampleextension/SampleExtensionIntegTestCase.java:84: error: cannot find symbol
                                   HttpEntity entity, Header... headers) throws IOException {
                                   ^
  symbol:   class HttpEntity
  location: class SampleExtensionIntegTestCase
/home/ubuntu/prudhvi-groupId-js/sample-extension-plugin/src/test/java/org/opensearch/jobscheduler/sampleextension/SampleExtensionIntegTestCase.java:84: error: cannot find symbol
                                   HttpEntity entity, Header... headers) throws IOException {
                                                      ^
  symbol:   class Header
  location: class SampleExtensionIntegTestCase
/home/ubuntu/prudhvi-groupId-js/sample-extension-plugin/src/test/java/org/opensearch/jobscheduler/sampleextension/SampleExtensionIntegTestCase.java:62: error: cannot find symbol
                new StringEntity(jobParameter, ContentType.APPLICATION_JSON));
                    ^
  symbol:   class StringEntity
  location: class SampleExtensionIntegTestCase
/home/ubuntu/prudhvi-groupId-js/sample-extension-plugin/src/test/java/org/opensearch/jobscheduler/sampleextension/SampleExtensionIntegTestCase.java:62: error: cannot find symbol
                new StringEntity(jobParameter, ContentType.APPLICATION_JSON));
                                               ^
  symbol:   variable ContentType
  location: class SampleExtensionIntegTestCase
/home/ubuntu/prudhvi-groupId-js/sample-extension-plugin/src/test/java/org/opensearch/jobscheduler/sampleextension/SampleExtensionIntegTestCase.java:89: error: cannot find symbol
        for (Header header: headers) {
             ^
  symbol:   class Header
  location: class SampleExtensionIntegTestCase
/home/ubuntu/prudhvi-groupId-js/sample-extension-plugin/src/test/java/org/opensearch/jobscheduler/sampleextension/SampleExtensionIntegTestCase.java:165: error: cannot find symbol
                Collections.emptyMap(), new StringEntity(entity, ContentType.APPLICATION_JSON));
                                            ^
  symbol:   class StringEntity
  location: class SampleExtensionIntegTestCase
/home/ubuntu/prudhvi-groupId-js/sample-extension-plugin/src/test/java/org/opensearch/jobscheduler/sampleextension/SampleExtensionIntegTestCase.java:165: error: cannot find symbol
                Collections.emptyMap(), new StringEntity(entity, ContentType.APPLICATION_JSON));
                                                                 ^
  symbol:   variable ContentType
  location: class SampleExtensionIntegTestCase
/home/ubuntu/prudhvi-groupId-js/sample-extension-plugin/src/test/java/org/opensearch/jobscheduler/sampleextension/SampleExtensionIntegTestCase.java:242: error: cannot find symbol
                Collections.emptyMap(), new StringEntity(entity, ContentType.APPLICATION_JSON));
                                            ^
  symbol:   class StringEntity
  location: class SampleExtensionIntegTestCase
/home/ubuntu/prudhvi-groupId-js/sample-extension-plugin/src/test/java/org/opensearch/jobscheduler/sampleextension/SampleExtensionIntegTestCase.java:242: error: cannot find symbol
                Collections.emptyMap(), new StringEntity(entity, ContentType.APPLICATION_JSON));
                                                                 ^
  symbol:   variable ContentType
  location: class SampleExtensionIntegTestCase
/home/ubuntu/prudhvi-groupId-js/sample-extension-plugin/src/test/java/org/opensearch/jobscheduler/sampleextension/SampleExtensionIntegTestCase.java:265: error: cannot find symbol
                Collections.emptyMap(), new StringEntity(entity, ContentType.APPLICATION_JSON));
                                            ^
  symbol:   class StringEntity
  location: class SampleExtensionIntegTestCase
/home/ubuntu/prudhvi-groupId-js/sample-extension-plugin/src/test/java/org/opensearch/jobscheduler/sampleextension/SampleExtensionIntegTestCase.java:265: error: cannot find symbol
                Collections.emptyMap(), new StringEntity(entity, ContentType.APPLICATION_JSON));
                                                                 ^
  symbol:   variable ContentType
  location: class SampleExtensionIntegTestCase
15 errors

> Task :opensearch-job-scheduler-sample-extension:compileTestJava FAILED

FAILURE: Build failed with an exception.

* What went wrong:
Execution failed for task ':opensearch-job-scheduler-sample-extension:compileTestJava'.
> Compilation failed; see the compiler error output for details.

@prudhvigodithi I will raise a PR later today to modify the org.apache.http imports. Once this change is merged, please rebase your fork with JS main and the failing tests should be resolved.

@prudhvigodithi could you rebase your changes with main, this should get CIs to pass.

Hey @saratvemulapalli and @joshpalis  and I just did 👍


UpdateAccessRule: Should it report success when configuration values are provided which aren't supported in a PATCH update

Confirmation

[X] My issue isn't already found on the issue tracker.
[X] I have replicated my issue using the latest version of the library and it is still present.

cloudflare-go version
All
Go environment
All
Expected output
An error when passing the following to "UpdateAccessRule" because the Cloudflare API doesn't allow configuration blocks to change you can only update the mode or notes on an access rule via the API.
 curl -X PATCH "https://api.cloudflare.com/client/v4/zones/$ZONE_ID/firewall/access_rules/rules/$RULE_ID" \
-H "X-Auth-Email: $EMAIL_ADDRESS" \
-H "X-Auth-Key: $AUTH_KEY"  \
-H "Content-Type: application/json" \
--data '{"mode": "whitelist","configuration": {"target": "ip","value": "<IP_ADDRESS>"},"notes": "Test"}'

Actual output
The API reported a success even though it ignored the configuration block completely:
{
  "result": {
    "id": "<REDACTED>",
    "paused": false,
    "modified_on": "2021-08-11T05:17:34.736351Z",
    "allowed_modes": [
      "whitelist",
      "block",
      "challenge",
      "js_challenge",
      "managed_challenge"
    ],
    "mode": "whitelist",
    "notes": "Test",
    "configuration": {
      "target": "ip",
      "value": "<IP_ADDRESS>"
    },
    "scope": {
      "id": "<REDACTED>",
      "name": "example.com",
      "type": "zone"
    },
    "created_on": "2021-08-11T05:17:34.769917109Z"
  },
  "success": true,
  "errors": [],
  "messages": []
}

Code demonstrating the issue
See above examples of calls against the API.
Steps to reproduce
See above example of call against the API for access rule patch.
References
No current open issues or PRs. I'd like to understand if the Cloudflare API is WAI here.... as this makes the Cloudflare terraform provider code for access rule management observe a bug relating to access rule updating.
As the API never updates the configuration block but returns a success. The Terraform provider thinks it was successful only to pick up the same resource change next time a plan/apply is run.... rinse and repeat.
I'll open a terraform provider issue once we confirm how the API should behaving here. As I know how to fix the provider (well mostly) but if the API is doing the wrong thing here then I'd rather Cloudflare fix the API first and then we can fix the provider handling.

this behaviour is a little odd, but expected. per the documentation the only parts that are updatable are mode and notes. other than that, you need to recreate the rule.
in terraform-provider-cloudflare, we trigger new resource creation when any other resources are changed.

@jacobbednarz do you want me to open an issue in the terraform-provider queue then?
As we're using v3.8.0 of the provider and it does not trigger new resource creation of access rules. It is trying to update them in place and never does so continuously tries to diff/apply because the API ignores the configuration block.
This is why I started the discussion here. If you think this doesn't need to be updated in the API then I'll move this to the terraform provider as an issue there.

if the issue is in the provider, that is the best place to start the conversation. I would recommend ensuring you're on the latest version before submitting an issue and if it is still an issue, then follow the issue template to get it triaged.

a brief skim of the CHANGELOG shows that v3.15.0 may have already addressed this and you just need to upgrade to fix this issue - https://github.com/cloudflare/terraform-provider-cloudflare/blob/master/CHANGELOG.md#3150-may-18th-2022

Legend thanks for taking a look. I looked at the provider code and didn't see force new yesterday.. hmm. Anywhose... will play around with it and open an issue if required.
Thanks for your help!


Vs Code extension not launching on macos

Environment information
Installed via npm locally on project

What happened?
After launching vs code, extension throw an error:
`
Biome binary found at /Users/crusader/data/projects/dayhub/dayhub-interface/node_modules/@biomejs/cli-darwin-arm64/biome
Copying binary to temporary folder: file:///Users/crusader/Library/Application%20Support/Code/User/workspaceStorage/cef63b6f70a39189be82687e55bf0d5a/biomejs.biome/biome
Executing Biome from: /Users/crusader/Library/Application Support/Code/User/workspaceStorage/cef63b6f70a39189be82687e55bf0d5a/biomejs.biome/biome
[cli-stderr] data 522
[cli-stderr] data 22
[cli-stderr] data 191
[cli-stderr] data 40
[cli-stderr] end
[cli-stdout] end
[cli-stderr] finish
[cli-stdout] finish
[cli-stdout] close
[cli-stderr] close
[cli] exit 1
[cli] close 1
[Error - 22:48:57] Biome client: couldn't create connection to server.
Error: Command "/Users/crusader/Library/Application Support/Code/User/workspaceStorage/cef63b6f70a39189be82687e55bf0d5a/biomejs.biome/biome __print_socket" exited with code 1
Output:
Biome encountered an unexpected error
This is a bug in Biome, not an error in your code, and we would appreciate it if you could report it to https://github.com/biomejs/biome/issues/ along with the following information to help us fixing the issue:
Source Location: crates/biome_cli/src/commands/daemon.rs:216:10
Thread Name: main
Message: Failed to start the logger for the daemon.: InitError { context: "failed to create log directory", source: Os { code: 13, kind: PermissionDenied, message: "Permission denied" } }
internalError/io ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
× Errors occurred while executing I/O operations.
Caused by:
  the server process exited before the connection could be established

at getSocket (/Users/crusader/.vscode/extensions/biomejs.biome-2.3.0/out/main.js:36539:11)
at processTicksAndRejections (node:internal/process/task_queues:95:5)
at createMessageTransports (/Users/crusader/.vscode/extensions/biomejs.biome-2.3.0/out/main.js:36553:16)
at LanguageClient2.createConnection (/Users/crusader/.vscode/extensions/biomejs.biome-2.3.0/out/main.js:15548:28)
at LanguageClient2.start (/Users/crusader/.vscode/extensions/biomejs.biome-2.3.0/out/main.js:15124:30)
at activate (/Users/crusader/.vscode/extensions/biomejs.biome-2.3.0/out/main.js:36294:3)
at m.n (/Users/crusader/data/soft/Visual Studio Code.app/Contents/Resources/app/out/vs/workbench/api/node/extensionHostProcess.js:151:6409)
at m (/Users/crusader/data/soft/Visual Studio Code.app/Contents/Resources/app/out/vs/workbench/api/node/extensionHostProcess.js:151:6372)
at m.l (/Users/crusader/data/soft/Visual Studio Code.app/Contents/Resources/app/out/vs/workbench/api/node/extensionHostProcess.js:151:5829)

`
Expected result
Extension launched
Code of Conduct

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

When the Biome server starts, it automatically creates a folder for storing its logs. By default that's where the folder is: https://biomejs.dev/guides/editors/create-a-extension/#daemon-logs
It seems it isn't possible. You could try to set the environment variable differently: https://biomejs.dev/reference/environment-variables/#biome_log_path

Thanks for the info. The issue was resolved by setting correct permissions for /Users/<UserName>/Library/Caches/dev.biomejs.biome


⚠ Fortnite has degraded performance

In 8b24ecb, Fortnite (https://fortnite.com) experienced degraded performance:

HTTP code: 200
Response time: 1464 ms

Resolved: Fortnite performance has improved in de26d37.


Better: Remove legacy field from interventions schema. RD-20480

Only schema changes so no need to update postman file.

LGTM. We may not need to wait for the code deploy day to merge this one, do we?

LGTM. We may not need to wait for the code deploy day to merge this one, do we?

https://developers.ringcentral.com/engage/digital/api-reference/Interventions/getIntervention the field is not even in the doc yet, shouldn't we at least wait for it to appear there?

mmh, but this breaking change is supposed to be deployed on January 12. If we wait for the previous doc commit to be deployed and then merge this current one, it would be deployed probably after Jan 12.
Can you coordinate this with Anastasia please?

mmh, but this breaking change is supposed to be deployed on January 12. If we wait for the previous doc commit to be deployed and then merge this current one, it would be deployed probably after Jan 12.
Can you coordinate this with Anastasia please?

I checked with her, she asked to merge today


If no filtering attributes, throws null exception

Even with the null exception check... Maybe an issue with the setter in serialisation?
Should be heavily covered in test suite

This is covered in a unit test and doesn't throw. Need to test and replicate with original manifest


Add tag for the fixed revision

I suggest adding 2.0.1 tag (2.0 to note the breaking change originally introduced in satori 1.2 version, and 0.1 to note the security patch).

I have forked this from satori/go.uuid to create https://github.com/satori/go.uuid/pull/75
I'm not commited to maintain this fork. You should instead look at alternatives such as github.com/google/uuid or https://github.com/kevinburke/go.uuid for alternatives!

To throw our name in the ring: https://github.com/gofrs/uuid -- it's forked from the same repo.


6pm a course list for current quarter

Display the course list on the home page for the current quarter. Have the link that the link to the detail page.

Retrieve a list of courses from CourseController and search for courses offered in W21(this is supposed to be 20211, but just for demonstration purposes, the name of the current quarter is hard-coded as W21 in the controller.)
Display the course information (course Id, course name, and current quarter name)
Link the name of the courses to another page to display specific information.
the link should include the course id for future use.
for any courses in the database that is not offered in the current quarter (W21) will not be displayed on the Home page.

If this PR is not ready for review (it looks like you're still writing tests), please convert this PR to a draft! :)

At this moment, this PR is still failing tests on CI.   Please investigate---if you can't get the tests to pass, close this PR and make a draft PR until you are ready to merge.

I resolved a series of merge conflicts for you.

Regarding functionality: your PR description states that "any courses in the database that is not offered in the current quarter (W21) will not be displayed on the Home page", but I am still able to see courses from other quarters on the homepage. See below:

@andrewhlu ... I think the PR description is out of date.   Originally the PR had a hard coded quarter, but another team added the "quarter filter" feature where the admin can set the quarter filter (i.e. the app's concept of the "current quarter")... it's one of the features on the Admin menu.
When I looked at the code a couple of days ago, those feature were in conflict, but 6pm-a seems to have resolved the conflict between the two features.    What I see is that the quarter filter seems to be working as intended now.

I edited the PR description to be more in line with what the code does now, which is what we want the code to do:

40 pts


Multistage graph ( shortest path between source and destination)

Description
Description
Multistage graph is a directed graph in which vertices can be divided in sets ( stages) such that vertices on same stage do not have edges between them and can't have a edge from present stage to previous stage . And in this given a source and destination for which shortest path is to be find.
Domain
Algorithm
Type of Contribution
Addition
Code of Conduct

[X] I follow Contributing Guidelines & Code of conduct of LearnCPP.

/assign


Hide Logout button

Hi,
I often accidentally logged out from the app whenever I want to refresh the messeges. Could you put the "Logout" button away from the main focus? The current placement is really close to the "Reload" button and it's also top above the other buttons. Which will lead to accidental logout. In addition to that, it gives impression that logout is the main feature of the menu.

I think logging out it's the least user want to do in an app. So following common UX conventional, putting logout button away from the main focus is a good idea. Like any other apps do.
Thanks in advance. I use your app a lot because MIUI has problem with default Whatsapp app.

Also may add an option to use CSS to hide the logout button when WhatsApp is loading.

Hi, thanks for your feedback.
I'll move the Logout button further down in the next release and also add an "are you sure" popup to prevent unwanted logouts


clean up env vars

.env: reorganize env vars according to their domain.
env.sh: remove jobs-list frontend and api specific env var from workflow env scripts.

OK


Fix the country table not displaying numbers in the correct column

Fixes #27
Description
In the getCountry.js file remove the unwanted countryInfo property causing the issue #27 from the api response data by destructuring the response api.data object into countryInfo and restData (representing the rest of the object data minus countryInfo), then passing the filtered restData to the let data = Object.values(restData) line.

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

Thank you but I had already fixed it via filter across the files.
Update the CLI. It's been fixed already.
Go ahead and run this:
npm i -g corona-cli


Documentation error in "Hacktoberfest.md"

simple Documentation error in Hacktober file
still no one noticed, Those who are serious about Hacktoberfest will only notice I guess!!  😉

I hope you try another PR in future.
I suggest you try a code or documentation PR.

I hope you try another PR in future.
I suggest you try a code or documentation PR.

@dartpain
I'm very new to open source, it's been two months, I'm hard coding and this is my first time registering for Hacktoberfest,
If you would be kind enough to put this PR as a hacktoberfest accepted.. it will be very motivating and helpful for me..
Or you can suggest me or assign me something to contribute to Hacktoberfest...
Thank you..

I hope you try another PR in future.
I suggest you try a code or documentation PR.

@dartpain
I'm very new to open source, it's been two months, I'm hard coding and this is my first time registering for Hacktoberfest,
If you would be kind enough to put this PR as a hacktoberfest accepted.. it will be very motivating and helpful for me..
Or you can suggest me or assign me something to contribute to Hacktoberfest...
Thank you..

@dartpain
This small help would motivate me a lot...

I hope you try another PR in future.
I suggest you try a code or documentation PR.

@dartpain
Please can you put this PR as approved or Hacktoberfest-accepted that would be very helpful and kind of you

@holopin-bot @negativenagesh Thank you!


bounds values set wrongly after calling .select on Image object

When .select method is called on Image object it looks like bounds of newly created image are set to the input of .select method. I found that in following case:
import numpy as np
import holoviews
from holoviews import Image, Points
holoviews.notebook_extension()

side = 5
region=(-side,-side,side,side)

def coords(bounds,samples):
    l,b,r,t=bounds
    hc=side/samples
    return np.meshgrid(np.linspace(l+hc,r-hc,samples),
                       np.linspace(b+hc,t-hc,samples))

x,y = coords(region, 5)
z = np.sin(x**2+y**2)

img1 = Image(z, group="Function", label="Sine", bounds=region)
img2 = img1.clone(extents=(-2, -2, 2, 2))
img3 = img1.select(x=(-2,2), y=(-2,2))

>>> print(img1[0.95, 0], img2[0.95, 0], img3[0.95, 0])
0.0 0.0 -0.756802495308

%%opts Image [situate_axes=True] {+axiswise}
img1*Points([(0.9,0)]) + img2*Points([(0.9,0)]) + img3*Points([(0.9,0)])

it leads to confusion when accessing value at the same coordinates of the initial and new object.
Maybe bounds of a new object should be selected to the edge of edge pixels that got selected?

Thanks @RafalSkolasinski. This a pretty major and annoying bug and will have to be addressed.

I've assigned the milestone to 1.6.0 as I agree this is an important bug to address. I would consider moving it to a minor,  post 1.5 release if we decide we are going to have one of those (seems likely!).

Hmm... maybe it would be good idea to make
[situate_axes=True]

a default setting?

This turned out to be a pretty important bug which actually caused some distortions in a plot in my thesis. I've finally figured it out in #994, which will hopefully make it into the 1.7 release.

This has now finally been finally been fixed in #994, which was just merged.
>>> print(img1[0.95, 0], img2[0.95, 0], img3[0.95, 0])
(0.0, 0.0, 0.0)

%%opts Image [situate_axes=True] {+axiswise}
img1*Points([(0.9,0)]) + img2*Points([(0.9,0)]) + img3*Points([(0.9,0)])


Accessibility: The questions and their response buttons should be grouped with  and

Actual
A screen reader user with tab navigation just hears the response values such as “Yes” or “No” without hearing the question.  A screen reader user has to use arrow keys to read the corresponding question which is not a desired experience.
Expected
Use  and  to associate/bind questions and their corresponding response buttons.
WCAG Violation
Info and Relationships 1.3.1
https://www.w3.org/WAI/WCAG21/Understanding/info-and-relationships.html
Parsing 4.1.1
https://www.w3.org/TR/UNDERSTANDING-WCAG20/ensure-compat-parses.html

These are accessibility issues of v3 of Web Chat, which has been deprecated and we no longer support unless there are security risks. For accessible Web Chat, please migrate to v4.

This is fixed in #3076.


Add support for Function Call tag

Chages in this PR:

Loads default GTM container in Android if present. This behavior is currently disabled by always passing -1 to loadContainerPreferFresh method.
Adds support to register GTM Function Call tag handlers in React Native environment. (closes #256)

Thanks for the PR! I'll review this as soon as I can.

The above means that RNFunctionCallTagHandler probably isn't needed either. You could probably get away with doing the following when registering a tag function:
[self.container registerFunctionCallTagHandler:self forTag:functionName];
``
`

Would not need to keep reference to `handlers` either since the `RCTGoogleTagManagerBridge` class is the only handler

Thanks man. I'll see what I can do.

Great work! Thanks for the changes. I will release this as a minor version after testing it some more.


Terminal links that contain a space break the markdown link handling

@eamodio On stable when I try typing git log, the underline only shows up under log.

@eamodio On stable when I try typing git log, the underline only shows up under log.

@rzhao271 you would need to create a link provider extension that creates a link out of "git log". I can verify this one.

@rzhao271 you would need to create a link provider extension that creates a link out of "git log". I can verify this one.


Run "make generate"

Two PRs collided, this makes the generated protobuf code consistent.

Oops, thanks for fixing this!  LGTM!


Spaces indentation improvement - align indenting left/right to rounded indentation level

Hello,
I'd like to propose change to current indentation mechanics in GDScript editor. There are some cases that are causing issues for me, consider following situations where indent is set to 4 spaces:

As you can notice I accidentally hit space on line 15 which gives this line total indent of 5 spaces while the rest of code is indented to 4. Okey, I hit backspace (or shitf+tab) to delete this space - but I actually end up with just one space like this:

Obviously i'd expect deletion to go stop at 4 spaces to align to closest full indentation level - this is the way it works in most code editors (for instance in sublime).
The same goes for indenting - if I have cursor set after one space hitting tab editor should move cursor to next full indentation level of 4 - now it just adds four spaces giving line 5 spaces of indentation.
Another thing that could be changed is clicking backspace where there are less spaces than indentation level for instance:

I'd expect whole indentation level removed here whereas spaces are removed one by one (this is special case due to ifs logic in code_editor.cpp code).
To sum up the changes I propose are:

Hitting tab would move cursor to next full indentation level instead of adding fixed number of spaces
Hitting backspace would remove spaces up to next full indentation level
Hitting shift+tab would remove spaces up to next full indentation level
Hitting backspace when there are less spaces than indentation size would remove spaces up to beginning of the line

Since there is lot of going on I decided to create Issue to ask about your opinion but I have PR work in progress - let me know if this feature is wanted and I'll provide code for it.
Ps.: There seems to also be separate code branch for indenting when selection is not empty, this needs to be adjusted accordingly too

Fixed by #28811.


ARTEMIS-1537 broker was less strict while reloading configuration

Thanks for your comment @jdanekrh

@stanlyDoge good work :) +1, will look to merge later today if no one else does before me.

@stanlyDoge all merged, thanks for this.

wow!
such fix!
thanks much!


Make local registry and nginx optional

This change makes registry certificate hosting URL and tink-worker image name:tag configurable.
It also adds a check for whether registry certificate is required and allow for absolute URIs
to be specified for action images in workflow template.

If you are using a registry with a self-signed certificate, you can specify
a custom URL of where the certificate is hosted. On boots, use extra-kernel-args
command-line parameter to specify a kernel argument with key 'registry_cert_url'
and value set to the URL location for registry CA certificate. Boots will pass this
as a kernel command-line parameter to Hook

In order to specify a custom absolute URI name:tag for tink-worker image instead of
defaulting to the hardcoded default of 'tink-worker:latest', use the
extra-kernel-args command-line parameter on Boots to specify a kernel argument with
key 'tink_worker_image' and value set to the absolute URI of tink-worker image.
Boots passes this to hook as a kernel command-line parameter

If you are using a public registry with trusted CA certificate, bootkit and
tink-docker do not need to download and setup registry certificate. Use
extra-kernel-args command-line parameter on Boots to set a kernel parameter with
key 'registry_cert_required' and value set to 'true' or 'false' to indicate this.
Since we are making local registry optional, we need to make the certificate
download and setup optional

Currently the tink-worker code always prepends the specified docker_registry to
the action images specified in the tinkerbell action template. The newly added
use_absolute_action_image_uri allows specifying the full URI for the action image
in the template and prevents the hardcoded prepending of the registry to these.
This allows action images to potentially reside in different registries. Pass this
kernel as the key=value kernel parameter from Boots to Hook as explained above. Value
would be set to 'true' or 'false'

Description

Why is this needed

Fixes: #
How Has This Been Tested?

How are existing users impacted? What migration steps/scripts do we need?

Checklist:
I have:

[ ] updated the documentation and/or roadmap (if required)
[ ] added unit or e2e tests
[ ] provided instructions on how to upgrade

This has gone stale and i believe all of this functionality now exists. Please re-open if you feel otherwise or would like to revisit this. Thank you!


Bad response from protect when not installed

I have a UDM Pro and a UNVR.  I have two separate configurations for each appliance,  Those configurations are pointed at the appropriate IP for that appliance. The config for the UNVR is going to "/proxy/protect/api/bootstrap" which is working fine.  I have 3 nodes using my config node for the UDM Pro that pointed to "/proxy/network/api/s/default/stat/alluser", "/api/ws/system", and "/proxy/network/wss/s/default/events".  For some reason the config node for my UDM Pro is throwing the following two errors:
"Bad response from: https://UDMPRO_IP/proxy/protect/api/bootstrap"  followed by "Endpoint not found: AxiosError: Request failed with status code 404"
The weird thing is that the protect endpoint is in use on my config node with the UNVR IP address on it, but it's the network config node that's pointed at the UDM Pro IP that's throwing the error from the other configuration's endpoint.  This just started about a week ago, and the UDM Pro doesn't even have protect installed.  Makes no sense as to why that config node is throwing an error for an endpoint used on a different config node, especially when the other config node is working fine.

I have the same setup - this error started when the new protect node was added because we started to grab the bootstrap in order to populate the cameras list. That list is populated at each "login" but there is no way (yet) to disable it for the UniFi OS instance without protect.
@marcus-j-davies I believe you should be seeing this as well now that you've put your Protect over to a cloud key gen 2 plus. If you've disabled Protect on your Dream Router...
The easiest fix might be to add a check box in the controller config to say whether this controller has Protect?
Maybe we change to show 2 options - "console with protect" and "console without protect" in this menu:

Just need some way to say "there's no Protect" so we don't keep trying to grab that bootstrap forever...

Yeah - we need a way to detect that there is no bootstrap to grab, ill play with it this weekend

Looking over things very quickly...
The websocket /api/ws/system should exist for any controller.
Updates come in very often of type DEVICE_STATE_CHANGED which have the following JSON key payload.apps.controllers[1].isRunning

I would think that we need the protect instance to exist there - and to be running - if we expect a cams bootstrap will exist.
Watching that  will even show whether the protect instance is actually running (not restarting or stopped) when you try to pull the bootstrap? There could be something there?

Well what doesn't make sense is that the error is for "/proxy/protect/api/bootstrap" even though I'm not calling it.  Even if I disable the one calling "/api/ws/system" it still happens.  This just started a couple weeks ago, I've been using this exact flow for months with no changes and it never happened until a couple weeks ago.

As already mentioned by @crxporter
"we started to grab the bootstrap in order to populate the cameras list" - therefore it's us, not you.
It will be fixed this weekend if i get time.

This just started a couple weeks ago

It would have started when you updated the plugin to version 0.9.0 - the endpoint is being called in the background by the unifi-access-controller-node node and is be used to populate the cameras list on controllers that have Protect running. We didn't (yet) consider the case of controllers that don't have protect running...
I noticed this error a while ago and mentioned it to Marcus, but we didn't deal with it yet.
A quick fix if you don't want to see the errors while it's being fixed - just install and start the Protect app on your UDM. No need to add any cameras, just having the app running will make the error go away.

Will that not mess with protect on the other system?  I migrated protect from my UDM Pro to UNVR so I had it installed previously...

They act fully independent of each other. Having a protect instance on the udm won't do anything to the running system on the unvr.
The udm protect won't even see the cameras since they're already adopted. If you click into it you'll just the splash screen of how to get started or something

This should be fixed in V1.0.0


Android sdk app crashes after integrating Firebase with Google maps

--------- beginning of crash
04-21 17:23:24.682 2887-2887/com.example.kshitija.googlemaps E/AndroidRuntime: FATAL EXCEPTION: main
Process: com.example.kshitija.googlemaps, PID: 2887
java.lang.NoSuchMethodError: No virtual method zzTt()Z in class Lcom/google/firebase/FirebaseApp; or its super classes (declaration of 'com.google.firebase.FirebaseApp' appears in /data/app/com.example.kshitija.googlemaps-1/split_lib_dependencies_apk.apk:classes13.dex)
at com.google.firebase.database.FirebaseDatabase.getInstance(Unknown Source)
at com.google.firebase.database.FirebaseDatabase.getInstance(Unknown Source)
at com.example.kshitija.googlemaps.MapsActivity.(MapsActivity.java:22)
at java.lang.reflect.Constructor.newInstance(Native Method)
at java.lang.Class.newInstance(Class.java:1572)
at android.app.Instrumentation.newActivity(Instrumentation.java:1065)
at android.app.ActivityThread.performLaunchActivity(ActivityThread.java:2199)
at android.app.ActivityThread.handleLaunchActivity(ActivityThread.java:2360)
at android.app.ActivityThread.access$800(ActivityThread.java:144)
at android.app.ActivityThread$H.handleMessage(ActivityThread.java:1278)
at android.os.Handler.dispatchMessage(Handler.java:102)
at android.os.Looper.loop(Looper.java:135)
at android.app.ActivityThread.main(ActivityThread.java:5221)
at java.lang.reflect.Method.invoke(Native Method)
at java.lang.reflect.Method.invoke(Method.java:372)
at com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run(ZygoteInit.java:899)
at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:694)
04-21 17:23:32.362 2887-2887/? I/Process: Sending signal. PID: 2887 SIG: 9

Probably the problem is a distinct google-services version.
Check in your project/node_modules/react-native-maps/android/build.gradle the version of play-services. You must use the same version in all components.

Hello @KarlosQ  I found the solution. The problem was I had to import existing Google project (in my case google maps) into firebase through firebase console. I was running 2 different projects which is why the app was crashing even though I had packages and SHA1 key stored for both.

@kshitijakarkar Then close this issue?

@henrikra Yes. That was the solution in my case.


[GHSA-m2h2-264f-f486] angular vulnerable to regular expression denial of service (ReDoS)

Updates

Affected products

Comments
The issue is fixed in HeroDevs version 1.9.7.

Hey @lennin-cp, similar to https://github.com/github/advisory-database/pull/5075 the version your suggesting doesn't seem to exist https://www.npmjs.com/package/angular/v/1.9.7

@darakian, thank you for the follow-up on this. The fix is not in a community version but a paid version offered by HeroDevs, see https://docs.herodevs.com/angularjs/faqs/fixed-vuln

same as with https://github.com/github/advisory-database/pull/5075
our scope is strictly limited to public packages on public registries.
https://github.com/github/advisory-database?tab=readme-ov-file#supported-ecosystems
We cannot inspect nor recommend private packages.


[Bug]: Only choose from the specific zone in function canWriteForNode

Contact Details
No response
Is there an existing issue for this?

[X] I have searched all the existing issues

Priority
low (Default)
Environment
- CubeFS version:3.4.0

Current Behavior
After a change in the zone topology, there is an issue where you cannot specify to create data partitions or metadata partitions on idle nodes within a specific zone.
Expected Behavior
After a change in the zone topology, it should be possible to select idle nodes within a specified zone to create data partitions or metadata partitions.
Steps To Reproduce
No response

CubeFS Log
No response
Anything else? (Additional Context)
No response

done


[Question] correct fetch-depth for pull_request

Hi there, love this action!
I'm having trouble understanding the correct setting for fetch-depth for a pull_request -- I would prefer 2 instead of 0, since my repo is huge, but does that mean if I push multiple commits to remote to an open PR, it will only check the latest commit?
For example:

main branch at commit 1
open PR with commit 2
action checks changed files between commits 2 and 1
push commits 3, 4, 5
action checks changed files between commits 5 and 4 (?)

How do I configure it to check commit 5 against commit 2?
Edit: actually, just tested this out and it does seem to work as I expect -- is it checking against main branch at commit 1 always? If so, awesome!

Hi @mariaines, in regard to your question
main (1) gets compared to the HEAD of the PR (5) using the merge base (2) i.e 2 -> 5 (retrieves all changes)
Using:
name: CI

on:
  pull_request:
    branches:
      - main

  job2:
    runs-on: ubuntu-latest  # windows-latest | macos-latest
    name: Test changed-files
    steps:
      - uses: actions/checkout@v3
        with:
          fetch-depth: 0 # You can use 0 for all history, 1 for the current commit and 2 for the current and preceding commits.
          #  The changed-files action pulls an initial history of the PR if 1 or 2 is used before performing the diff you can also control the history pulled by the action using the `fetch_depth` input directly passed to the changed-files action.

      - name: Get changed files
        id: changed-files
        uses: tj-actions/changed-files@v37

Say you pushed 3-5 and 5-7 to compare 7 to the last pushed remote commit 5, you'll need to use the same setup as above with a slight difference
... # Same as above
      - name: Get changed files
        id: changed-files
        uses: tj-actions/changed-files@v37
        with:
           since_last_remote_commit: true # This compares 5 -> 7

For more usage-related questions you can also take a look at previous discussion here and feel free to open a new discussion you have any more questions.

Hi @jackton1 ,
sorry to introduce myself into this topic but i'm currently using your github actions and actually i'm stuck in a similar use-case.
Actually what i'm trying to do is to resolve this situation:

open new feature branch fb1
commit_1 -> adding file_a and file_b
push
execution_1 of the workflow (that include list-changes action) -> fails due to an error in file_a
commit_2 -> fix file_a
push
exection_2 of the workflow

There is any chance to retrieve with the action in the execution_2 both file_a and file_b?
It is possible to specify the depth in terms of how many commits i want to observe?

name: CI

on:
  pull_request:
    branches:
      - main

  job2:
    runs-on: ubuntu-latest  # windows-latest | macos-latest
    name: Test changed-files
    steps:
      - uses: actions/checkout@v3
        with:
          fetch-depth: 0 # You can use 0 for all history, 1 for the current commit and 2 for the current and preceding commits.

      - name: Get changed files
        id: changed-files
        uses: tj-actions/changed-files@v37

@vinceRicchiutiPrima  Based on the previous answer, this workflow would list all changed filenames file_a and file_b. You can control the start and end commit by passing in the base_sha and sha inputs.
For example:
... # Same as above
      - name: Get changed files
        id: changed-files
        uses: tj-actions/changed-files@v37
        with:
           base_sha: "HEAD~8"   # Compare the 8th parent of the current commit
           sha: "HEAD"  # Until the current commit. Note this is not needed as this is assumed by default

Hi @jackton1 thank you for your hint!!!

What if I have a stack of diffs on a PR, and this commit stack is ever growing as I continue to work. Can I derive the original commit hash of the target branch?

@brian-fogg-fileread That’s the default behaviour and you won’t have to change anything the last commit of base branch e.g (main or any other branch) would always be compare to the latest commit of your PR branch for pull_request events.


Don't sleep if ProtectedQueue.localQueue is not empty. (#6234)

Within process_event(), if the Event can not be locked , it will be pushed into ProtectedQueue.localQueue for retry in awhile.
We missed the checked on ProtectedQueue.localQueue before to calculate the sleep time.
As a result, these retrying Event may wait up to 60 ms before they can be processed.

[approve ci autest clang-analyzer]

[approve ci autest]

[approve ci clang-analyzer]

Cherry-picked to v9.0.x branch.

Cherry-picked to 8.1.x


SnakeCase Json and PascalCase C#

I'm from the AWS .NET team that does a lot of the .NET Lambda tooling. I wanted to compare this library with the JSON.NET one we support. I saw you had an example code in your README but I can't get it to work because the JSON AWS services send uses snake case which doesn't match the Pascal Case all of the .NET properties are using when deserializing.
Am I missing a setting somewhere to make that work?

You can specify the case using resolver passed to deserialization method or specified as default.
E.g.:
JsonSerializer.Deserialize<T>(Data, StandardResolver.ExcludeNullSnakeCase);

or you can specify default resolver to use
JsonSerializer.SetDefaultResolver(StandardResolver.ExcludeNullSnakeCase);
You can read more about resolvers in this portion of README
https://github.com/neuecc/Utf8Json#resolver


chore: increase version

This change is

Reviewed 2 of 2 files at r1.
Review status: all files reviewed at latest revision, all discussions resolved, some commit checks failed.

Comments from Reviewable

Coverage remained the same at 91.771% when pulling 8034fe70f2e337a09ad1b3d52a86bce944eed8e4 on v2.21.2 into 8edb1f9808d5492e5ffdfea3db12d09724dd1ce1 on master.

@homu r+

:pushpin: Commit 8034fe7 has been approved by kbknapp

:zap: Test exempted - status


yarn install --frozen-lockfile shows a warning: The engine "vscode" appears to be invalid.

Describe the bug
During install of dependencies I get a warning about a transitive dependency being wrong:
yarn install --frozen-lockfile
yarn install v1.22.21
warning ../package.json: No license field
[1/4] 🔍  Resolving packages...
[2/4] 🚚  Fetching packages...
warning<EMAIL_ADDRESS>The engine "vscode" appears to be invalid.
[3/4] 🔗  Linking dependencies...
[4/4] 🔨  Building fresh packages...
✨  Done in 4.01s.

Upon using the yarn why command this dependency is linked to vite-plugin-checker:
yarn why vscode-languageclient
yarn why v1.22.21
warning ../package.json: No license field
[1/4] 🤔  Why do we have the module "vscode-languageclient"...?
[2/4] 🚚  Initialising dependency graph...
[3/4] 🔍  Finding dependency...
[4/4] 🚡  Calculating file sizes...
=> Found<EMAIL_ADDRESS>info Reasons this module exists
   - "vite-plugin-checker" depends on it
   - Hoisted from "vite-plugin-checker#vscode-languageclient"
info Disk size without dependencies: "944KB"
info Disk size with unique dependencies: "1.43MB"
info Disk size with transitive dependencies: "2.14MB"
info Number of shared dependencies: 5
✨  Done in 0.16s.

Reproduction
Going into my repository I execute:

git clean -dxf
yarn install --frozen-lockfile

Expected behavior
I am expecting to not get a warning about transitive dependencies brought in by vite-plugin-checker
yarn install --frozen-lockfile
yarn install v1.22.21
warning ../package.json: No license field
[1/4] 🔍  Resolving packages...
[2/4] 🚚  Fetching packages...
[3/4] 🔗  Linking dependencies...
[4/4] 🔨  Building fresh packages...
✨  Done in 4.01s.

System Info
System:
    OS: macOS 12.0.1
    CPU: (10) arm64 Apple M1 Max
    Memory: 73.06 MB / 64.00 GB
    Shell: 5.8 - /bin/zsh
  Binaries:
    Node: 18.18.2 - ~/.nvm/versions/node/v18.18.2/bin/node
    Yarn: 1.22.21 - ~/.nvm/versions/node/v18.18.2/bin/yarn
    npm: 9.8.1 - ~/.nvm/versions/node/v18.18.2/bin/npm
  Browsers:
    Chrome: 120.0.6099.129
    Safari: 15.1
  npmPackages:
    vite-plugin-checker: ^0.6.2 => 0.6.2

Additional context
This is not a blocker assuming that it doesn't indicate a more serious issue. I noticed this: https://github.com/microsoft/vscode/issues/91009
One poster commented:

In this case it's useless warning and can be ignored.
Unfortunately - only way to hide error specifically for vscode - only all
errors about engine yarn install --ignore-engines

Perhaps this error can be ignored? If so this issue can be closed.
Validations

[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.

+1 Experiencing the same issue.


Make devops component seperate from ks-installer

This pull request aims to make devops component separate from ks-installer. However, there are two steps we need to do:

Remove devops component from ks-installer(current PR is doing the right thing)

At the same time, we are doing some works for kubesphere apiserver, and the upstream branch is https://github.com/linuxsuren/kubesphere/tree/remove-devops-ctrl. When this branch is merged into kubesphere/kubesphere, this pull request can be ready to merge.
What have I done

Remove devops CRDs
Remove devops roles files, including s2i and jenkins
Use ks-devops helm chart to install devops component
Use ks-devops-plugin helm chart to install devops plugin.

Things to do

[x] Recover SonarQube configuration
[ ] Invert devops.enable config
[x] Package helm charts
[ ] Keep image pull policy IfNotPresent
[ ] Stick image tag with sha256

Step to test
kubectl apply -f https://raw.githubusercontent.com/JohnNiang/ks-installer/refactor/remove-devops-crd/deploy/kubesphere-installer.yaml
kubectl apply -f https://raw.githubusercontent.com/JohnNiang/ks-installer/refactor/remove-devops-crd/deploy/cluster-configuration.yaml

or replace ks-installer image: johnniang/ks-installer:remove-devops-comp.
/area devops
/kind optimization

Unable to synchronize the credentials created on the kubesphere interface to jenkins


Setup Project with styleguide

Use styleguide based setup as reccomended by klappy in this article

Tested the style guide setup


Fix async producer typo

s/confured/enabled

I think it was probably meant to be 'configured' originally, but 'enabled' works fine :)
Thanks!


Fixed "Issue" #1

Refactored the HomePage into a ListWidget.
Set body to a list view.

Note that it has not been styled.

Just quickly skimming over the code, it looks pretty good! When I get the chance to sit down at my computer and take a deeper look I'll do a full review. That might be this weekend or early next week.

@tycebrown I have completed my first review of this pull request. The functionality works great, the code just needs some cleanup.

Please address each of the items I mentioned above.

Respond with "done" or something similar to each item when they have been completed. Please do not click the "Resolve conversation" button,
I will click it after I have confirmed that you have applied the necessary fix.

When you have addressed all the items, please remove the "changes requested" label, and leave a comment asking me to look over this again.

Done and ready!

Three style-related things to address, and then we'll be done here!

Committed & pushed your style changes.


v-text on custom component with slot work in SPA but not in SSR

Environment
nuxt3 lates
linux
Reproduction
https://github.com/max5432112345/nuxt-error-2/
Describe the bug
If use v-text on custom component with slot when in SPA text is displayed in page but in SSR not displayed (hot reload only sometime make it to display but after refresh page disappear).
Index.vue
<template>

    <CustomText v-text="'test'" />
     <CustomText> TEST </CustomText>

</template>

CustomText.vue

<template>

    <slot name="default" />

</template>

Additional context
No response
Logs
No response

As comments are deleted here - linking the related issue in vue core https://github.com/vuejs/core/issues/6435

let's track upstream then 🙏


Godot 4.2.2 support?

Feature description
4.2.2 is giving way more fps and stability in VR then 4.2. But when I try to run this code snippet:
	GDOculusPlatform.initialize_android_async("314159265358979").then(func(_initialization_resp):
		print("Oculus Platform initialized!")
	).error(func(initialization_err):
		print("Oculus Platform initialization error: ", initialization_err)
	)

It says, Cannot call method 'then' on a null value. I also tried it with my own app ID.

Hi, I just tested in 4.2.2-stable and was able to initialize correctly.
Did you enable the plugin in Project Settings -> Plugins -> Godot Oculus Platform ?

Im on 4.2.2 stable also. Im also on ubuntu. I did enable it but it didnt work, so I cloned it again and followed the build steps, it built successfully and I import the demo godot project and it has an error "Parser Error: Identifier "GDOculusPlatform" not declared in the current scope." :(((

That is weird... I would recommend sticking with the files from the latest release, since they at least are recognized inside the editor.
Keep in mind that this plugin only works on android, when deploying the game to the headset. When you run from Windows/Linux/MacOS you should get an error saying "Godot Oculus Platform only works with the Meta Quest (android).", thought from looking at the code, it seems like I missed a part of the code and that's why you get the "Cannot call method 'then' on a null value" message.

Okay that makes sense I was just running it on my computer to test if it worked or not


fix: fix imports to use file type

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


feat: get stream for local and http resources

Codecov Report
Attention: Patch coverage is 71.01449% with 20 lines in your changes missing coverage. Please review.

Project coverage is 77.25%. Comparing base (5061b52) to head (449cba6).
Report is 1 commits behind head on main.

Files with missing lines
Patch %
Lines

...c/Packata.Core/Serialization/Json/PathConverter.cs
51.85%
11 Missing and 2 partials :warning:

src/Packata.Core/PathHandling/FileSystem.cs
0.00%
2 Missing :warning:

src/Packata.Core/PathHandling/HttpPath.cs
84.61%
2 Missing :warning:

src/Packata.Core/PathHandling/LocalPath.cs
84.61%
2 Missing :warning:

.../Serialization/Json/DataPackagePropertyResolver.cs
85.71%
0 Missing and 1 partial :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #12      +/-   ##
==========================================
- Coverage   86.54%   77.25%   -9.30%
==========================================
  Files          14       19       +5
  Lines         171      233      +62
  Branches       29       44      +15
==========================================
+ Hits          148      180      +32
- Misses         20       48      +28
- Partials        3        5       +2

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


test(v2): fix Jest tests on Windows

Motivation
I want to fix the Test Cases for Windows OS.
Have you read the Contributing Guidelines on pull requests?
Yes.
Test Plan
I have run test on

Windows Locally - yarn test, yarn start, yarn start:v1, yarn prettier && yarn lint
Linux on GitPod -  yarn test, yarn start, yarn start:v1, yarn prettier && yarn lint

Related PRs
#3481
TODOs - packages to be fixed

[x] docusaurus-plugin-client-redirects
[ ] docusaurus-plugin-content-docs
[ ] docusaurus-plugin-content-blog
[ ] docusaurus-plugin-content-pages
[ ] docusaurus-mdx-loader
[x] docusaurus-utils
[ ] docusaurus-1.x
[x] Add window test configuration in nodejs-windows.yml

@slorber I think it made sense to merge PR #3481 to see successful progress in this PR (CI will not fails).

@lex111 I don't think it's a good idea because this would lead to master failing on windows CI and all created PRs after the merge would also fail on windows and let contributors think they did something wrong. Also we are not sure yet how easy it is to fix all the windows tests

However we should be able to easily track progress in this PR.
maybe @skr571999 you can enable windows tests here so that we'll see this PR failing with the number of failing tests decreasing as your work progress?

Yeah ok, @slorber

In the next commit I will add that.

@slorber agree, but it's better to close PR #3481, there's no point in that.

Hi, there
Any idea why it is it failing the yarn build:v2

On my local system it is building fine

Hi @skr571999
No idea sorry 😅 maybe try to revert some of the changes until it passes?

Hi @slorber @lex111
Getting mdx-loader/transformImage Test case fail on GitHub Action due to extra ../ in the Received Path

But it is running fine on my machine
My Machine

Window 10 Home - Version(2004)

Not getting How to check on my local system that it is fine or not.

Thanks for reporting @skr571999 , don't know why it only works on your computer but not CI.
Looks to me there are quite a few snapshot errors due to snapshots here: https://github.com/facebook/docusaurus/pull/3540/checks?check_run_id=1229879405
As you modify both the tests and the implementation at the same time, it's hard to tell if it can't lead to regressions.
Can you comment on your PRs about why you choose to modify each line to help me understand? Particularly the "cleanPath", that looks surprising to me that we need this line

Hi @slorber
I have added the comment for review(contain the description about cleanPath function),

As you modify both the tests and the implementation at the same time, it's hard to tell if it can't lead to regressions.

I have revert back the old commit, because that was also having something similar problem, so i thought to try this one and look if it is happening here or not.
Can u have a look and suggest what I can do because I am not able to test the changes Locally as they are passed locally and are failing here 😂

😅 I am not sure but as in GitHub Action we are having Windows Server OS  and in my local system I am having Window 10 Home, is this can be the cause of the different Output of the Test Cases??

Hi,
As I don't even have windows right now it's hard for me to review this PR.
I don't now either why it works locally and fails on the CI and what to do about it.
Maybe we should put this PR in stand-by until I have Windows and can provide more help?

closed in favor of https://github.com/facebook/docusaurus/pull/3959
the new pr is based on your initial commits so you'll be credited for your work

fixed in https://github.com/facebook/docusaurus/pull/3959


Close tab with ctrl+w c

This would be a nice complement to existing ctrl+s and ctrl+c.
This extension is already doing a very impressive job. Thank you so much!

Contributions welcome if i won't do this next days, there could be some troubles because <C-c> isn't being sent to neovim and intercepted by vscode. However configuring Ctrl+w Ctrl+c as separate keybinding in package.json should help in this case. Once the keystroke is being sent to neovim it should be trivial to bind it and call vscode command to close active tab here

I guess it was already implemented, but a typo slipped through? See https://github.com/asvetliakov/vscode-neovim/pull/180.

Yes, mistype. But didn't you ask about <C-w> <C-c> ( the PR fixes <C-w> c )? (just checked neovim docs - <C-w> <C-c> cancels the command, so you had probably mean <C-w> c)

I meant asking for <C-w> c but did in fact write <C-w> <C-c> 😀

Oh yes, then fixed. Thank you!


Nested at-rule

Was working on some styled-components stuff, and I think I've found a bug.
Here's the code I'm running (just to get a sense of how the plugins work atm) using v3.2.8:
import Stylis from 'stylis'

const stylis = new Stylis({
  global: false,
  cascade: true,
  keyframe: false,
  prefix: true,
  compress: false,
  semicolon: true,
})

const plugin_calls = []
stylis.use((context, content, selectors, parent, line, column, length) => {
  plugin_calls.push({ context, content, selectors, parent, line, column, length })
})
stylis('.foo', `
  bar: baz;
  @media screen and (min-width: 600px) {
    yeah: nah;
    @supports (lol) {
      too: right;
    }
  }
`)
console.log(JSON.stringify(plugin_calls, null, 2))

[
  {
    "context": -1,
    "content": "\n  bar: baz;\n  @media screen and (min-width: 600px) { \n    yeah: nah;\n    @supports (lol) {\n      too: right;\n    }\n  }\n",
    "selectors": [
      ".foo"
    ],
    "parent": [
      ".foo"
    ],
    "line": 1,
    "column": 1,
    "length": 0
  },
  {
    "context": 1,
    "content": "bar:baz",
    "selectors": [
      ".foo"
    ],
    "parent": [],
    "line": 2,
    "column": 11,
    "length": 0
  },
  {
    "context": 1,
    "content": "yeah:nah",
    "selectors": [
      ".foo"
    ],
    "parent": [
      ".foo"
    ],
    "line": 4,
    "column": 14,
    "length": 0
  },
  {
    "context": 1,
    "content": "too:right",
    "selectors": [
      ".foo"
    ],
    "parent": [
      ".foo"
    ],
    "line": 6,
    "column": 17,
    "length": 0
  },
  {
    "context": 2,
    "content": "too:right;",
    "selectors": [
      ".foo"
    ],
    "parent": [
      ".foo"
    ],
    "line": 7,
    "column": 5,
    "length": 10
  },
  {
    "context": 3,
    "content": ".foo{too:right;}",
    "selectors": [
      "@supports (lol)"
    ],
    "parent": [
      ".foo"
    ],
    "line": 7,
    "column": 5,
    "length": 16
  },
  {
    "context": 2,
    "content": "yeah:nah;",
    "selectors": [
      ".foo"
    ],
    "parent": [
      ".foo"
    ],
    "line": 8,
    "column": 3,
    "length": 9
  },
  {
    "context": 3,
    "content": ".foo{yeah:nah;}@supports (lol){.foo{too:right;}}",
    "selectors": [
      "@media screen and (min-width:600px)"
    ],
    "parent": [
      ".foo"
    ],
    "line": 8,
    "column": 3,
    "length": 48
  },
  {
    "context": 2,
    "content": "bar:baz;",
    "selectors": [
      ".foo"
    ],
    "parent": [],
    "line": 9,
    "column": 1,
    "length": 8
  },
  {
    "context": -2,
    "content": ".foo{bar:baz;}@media screen and (min-width:600px){.foo{yeah:nah;}@supports (lol){.foo{too:right;}}}",
    "selectors": [
      ".foo"
    ],
    "parent": [
      ".foo"
    ],
    "line": 9,
    "column": 1,
    "length": 99
  }
]

To my mind, the first "context": 3, callback (for "content": ".foo{too:right;}",) should have a parent of [".foo", "@media screen and (min-width:600px)"], not just .foo. I'm guessing somewhere there's an expectation that at-rules can only be one level deep, which isn't the case (@media and @supports work just fine nested, at least).
I'm trying to build a simple plugin that can do the sort of thing described here: https://github.com/threepointone/glam/issues/5 to make stylis' output insertRule-able.
Would there be a better way to approach this?

Created this plugin that generates an AST https://github.com/thysultan/stylis.js/blob/master/plugins/ast/, maybe that can also help.
Can't remember the reason i implemented it like that and if that was intended or not but will look into it.

I think the reason it's like that is because nested @media queries aren't represented as ancestors in the sense of a selector relationship i.e #a > #b of the nesting they appear in. This only reflects on selectors blocks that can be concat-ed to make a valid selector.
#a { #b {} } -> #a #b {}

#a { @media { #b } } -> @media { #a #b {} }

Though i'm open to improving this heuristic.

This is also a problem for a nested @-rule in @page which is needed for CSS Generated Content for Paged Media Module
Input:
@page {
    @bottom-right {
        content: 'Page ' counter(page) ' of ' counter(pages);
        margin-right: 1cm;
    }
}

Output:
@page@bottom-right {
	content: 'Page ' counter(page) ' of ' counter(pages);
	margin-right: 1cm;
}

Expected output is the same as the input.

@k15a Though unrelated the fix for that looks identical to the fix for #71 – will publish a patch for that.

@k15a 3.4.2 patches the nested @page issue.

@thysultan this is fixed now right?


feat: poc using style package in big-number [wip]

💔 Breaking Changes
🏆 Enhancements
This gets BigNumber referencing the superset theme.
📜 Documentation
🐛 Bug Fix
🏠 Internal

Is this still WIP or ready for review?

I think this was more a poc than a wip, actually. Closing.


Cache ~/.m2/repository to speed up CI speed

Description

Upgrade Notes
Does this PR prevent a zero down-time upgrade? (Assume upgrade order: Controller, Broker, Server, Minion)

[ ] Yes (Please label as backward-incompat, and complete the section below on Release Notes)

Does this PR fix a zero-downtime upgrade introduced earlier?

[ ] Yes (Please label this as backward-incompat, and complete the section below on Release Notes)

Does this PR otherwise need attention when creating release notes? Things to consider:

New configuration options
Deprecation of configurations
Signature changes to public methods/interfaces
New plugins added or old plugins removed

[ ] Yes (Please label this PR as release-notes and complete the section on Release Notes)

Release Notes

Documentation

Codecov Report

Merging #7408 (264496e) into master (6294d1b) will decrease coverage by 2.23%.
The diff coverage is n/a.

:exclamation: Current head 264496e differs from pull request most recent head 570d2eb. Consider uploading reports for the commit 570d2eb to get more accurate results

@@             Coverage Diff              @@
##             master    #7408      +/-   ##
============================================
- Coverage     71.89%   69.66%   -2.24%
+ Complexity     3430     3251     -179
============================================
  Files          1517     1123     -394
  Lines         75039    52948   -22091
  Branches      10921     7970    -2951
============================================
- Hits          53950    36885   -17065
+ Misses        17457    13437    -4020
+ Partials       3632     2626    -1006

Flag
Coverage Δ

integration1
?

integration2
?

unittests1
69.66% <ø> (-0.04%)
:arrow_down:

unittests2
?

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

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

Seems no significant changes


Multi versions

PR checklist

[x] This comment contains a description of changes (with reason).
[ ] If you've fixed a bug or added code that should be tested, add tests!

[ ] If you've added a new tool - have you followed the pipeline conventions in the contribution docs
[ ] If necessary, also make a PR on the nf-core/rnaseq branch on the nf-core/test-datasets repository.

[ ] Make sure your code lints (nf-core lint).
[ ] Ensure the test suite passes (nextflow run . -profile test,docker).
[ ] Usage Documentation in docs/usage.md is updated.
[ ] Output Documentation in docs/output.md is updated.
[ ] CHANGELOG.md is updated.
[ ] README.md is updated (including new tool citations and authors/contributors).

Apply https://github.com/nf-core/tools/pull/1247/files to the rnaseq pipeline to check if anything breaks.

This is what the multiqc table looks like now... not sure if this is what we want or if we easily can get a better layout

Blocked by https://github.com/ewels/MultiQC/issues/1515

MultiQC bug fixed by @ErikDanielsson and merged ✅
Suggestion on Slack from @mahesh-panchal about the table:

I think an ok format would be a three column table with Module, software, version.

Can also dump custom HTML if you would prefer. Shout if there's anything I can help with 👍🏻

Ok, I think this is ready.
@mahesh-panchal, will you update the tools PR?
I can make an exemplary PR to fastqc before applying the changes to the modules repository.

Yes, will do.

OK. I think I ported all the relevant parts.

Officially implemented this along with other updates in https://github.com/nf-core/rnaseq/pull/704


Hotfix Add null check before using .Bind(serviceHub) to prevent crash on using .Include in ParseQueries (anonym access)

Improved fix for this PR
https://github.com/parse-community/Parse-SDK-dotNET/pull/388

Could you please add a test for this bug?

Hi @mtrezza ,
not 100% sure how I can design a valid test for this problem.
So here some pseudo code to demonstrate the issue. may u can help to find a way to integrate this test?
//Prepare
ParseObject blog = new ParseObject("Blog");
blog["title"] = "My blog";
ParseObject comment = new ParseObject("Comment");
comment["Blog"] = blog;
await comment.SaveAsync();

//Test
var query = ParseClient.Instance.GetQuery<Comment>().Include("Blog");
var results = await query.FindAsync(CancellationToken.None);
List<Comment> items = results.ToList();

Problem:
The var results will be ok, but when casting to the var items list it throws an exception

@Werkezeugs I looked at the tests and they are not very extensive, no e2e and a lot of mocking is missing, like obj saving. If you cannot add a test and cannot add the mocks, then we can merge without a test.


Member settings in labs

Added Labs flag to enable members
Added subscription settings for members configuration

Coverage decreased (-0.4%) to 70.561% when pulling 8b340a68bd48864d193db58e8b657f79b9b61b5a on members-labs into 3a53a3041f3ac7f5ec090aa65b19f4fc220b476f on master.


Search index crawl delay on full index rebuild

Is your feature request related to a problem? Please describe.
When rebuilding the index from the admin the index builds without seemingly any delay between indexed items. When having thousands of pages this basically means when we crawl the pages by doing http request we would be making a lot of requests to the site increasing the performance requirements.
Describe the solution you'd like
A setting to set the index delay item per index item when doing a rebuild of the full index. For example in the appsettings having
{
...
     "IndexItemDelay": 2000
...
}

will have the indexing a page every 2 seconds.
Describe alternatives you've considered
Another approach is to have this index respect the Crawl-delay value in the robots.txt. We can set the useragent of the crawler it would make sense to have it read the robots.txt and delay the crawling accordingly.
User-agent: SearchCrawler
Crawl-delay: 10

Additional context
Add any other context or screenshots about the feature request here.

@bobharing do you have any thoughts about this being a global application setting managed in code (appsettings.json) vs per-index configuration managed in the administration UI?

@seangwright Well that depends on what the vision would be.
Having it as part of the index setup in the admin will mean that we want to have this control per index. I can imagine some use cases have large indexes and others smaller (and less resource intensive depending on the pages it crawls). So you might want to be more granular on the delay per index. Plus you get the advantage of being able to modify this delay without having to do a redeployment (or modifying the appsettings.json directly which is not possible on SaaS environments)
If we want to apply just a singular blanket rule, then I would prefer to have it applied in appsettings.json instead of course.

Hello, optional IndexItemDelay has been added to address this request in the release v4.1.1


altimeter: use system variables to calibrate altimeter.

Use system variables to calibrate altimeter. That means that calibration works for other applications, too, and persists between altimeter starts.
Immediately show altitude after calibration changes, otherwise it is hard to set the calibration as the actual values lag behind reality.

Cool, thanks!


feature: recursive DataFrame.derive()

class Foo(pt.Model):
    bar: int = pt.Field(derived_from="foo")
    quad_bar: int = pt.Field(derived_from=2 * pl.col("double_bar"))
    double_bar: int = pt.Field(derived_from=2 * pl.col("bar"))

will derive double_bar first, then quad_bar, automatically detecting upstream dependencies and ensuring that execution order is correct. The data frame will be returned in the order specified by the model (quad_bar comes before double_bar).
Added new test_recursive_dependencies. Tests will pass for both polars==0.18 and polars==0.19 after merging #24.

I really like the idea and understand the need for this. Any chance you'd like to rebase this on latest main branch?

I really like the idea and understand the need for this. Any chance you'd like to rebase this on latest main branch?

Got it in there on the pydantic v2 refactor! See test_recursive_derive: https://github.com/JakobGM/patito/blob/89e59f313ec2452fe76a1e72aff2e157f6fac298/tests/test_polars.py#L284
Will close this.


Hand-drawn lines can overlap text, become less readable.

a->b: This is difficult to read
a->b: This line is significantly longer than the one which appears above.

As sequence text length increases, the maximum vertical range for the swoop in hand drawn lines also increases.
At the lengths above, that line can run right through the line text. At even longer lengths, the line can appear above it's associated text.

In Raphael.fn.wobble, try a lower divisor; e.g.,
var wobble = Math.sqrt( (x2 - x1) * (x2 - x1) + (y2 - y1) * (y2 - y1)) / 30;

instead of
var wobble = Math.sqrt( (x2 - x1) * (x2 - x1) + (y2 - y1) * (y2 - y1)) / 15;

Thanks for the feedback. I've actually reworked the wobble a little, capping how much it can wobble. Going over text should no longer be a problem:


Verified VAA (Verifiable Action Approval) structure

https://github.com/zk-Lokomotive/zk-lokomotive-move/commit/2e3bea61232be1407f3cebf3363ed3b834eab524?diff=split&w=0


Is there any way to convert "yolov4-tiny-3l.weights"?

Hi, guys.
I used to detect objects with "yolov4.tflite(608x608)" on Android, but it was very slow.
So I tried to convert and use "yolov4-tiny-3l.tflite(608x608, for small object)" and I got problems about detection results.
There are no detection results with "yolov4-tiny-3l.tflite" on Android.
(Before conversion, I checked the detection results of "yolov4-tiny-3l.weights" on windows 10)
I think that the conversion code is not for "yolov4-tiny-3l.weights".
Is there any way to convert "yolov4-tiny-3l.weights" to ".pb" and "tflite" files?
Please help me and give some tips.
Thanks.

I'm facing the same problem, have you solved it?