Add ResourceCollection support ServletEnvironment

PR to support adding a collection of static resource folders in ServletEnvironment.
Basically this was done to negate the need for consumers to have a dependency on jetty-util library, as they would need to create a ResourceCollection if they have multiple resources for serving static content.

Changes Unknown when pulling b93b72296e9a800ee950c4a8b9dd3557ba83d066 on afrin216:resource into ** on dropwizard:master**.

Changes Unknown when pulling a6b55d1c1dc48646370290a0d1d6c1fa0a04f633 on afrin216:resource into ** on dropwizard:master**.

Looks good! Thanks for the contribution!


Monitor should close what it inherits

In my work I have used a monitored environment which has some resources it needs to close. Using Monitor was the problem as it doesn't call close() on its parent. This fixes it

harry-uglow:monitor_close

@harry-uglow


snapcast update

Description: Update snapcast media player component:

Use latest version of snapcast module
Use stream name instead of path for source select
Fix status typo
Adds config validation

Related issue (if applicable): https://github.com/happyleavesaoc/python-snapcast/issues/1
Checklist:
If code communicates with devices, web services, or a:

[x] Local tests with tox run successfully. Your PR cannot be merged unless tests pass
[x] New dependencies have been added to the REQUIREMENTS variable (example).
[x] New dependencies have been added to requirements_all.txt by running script/gen_requirements_all.py.

:dolphin:


examples: correct link for FlexLine.ipynb

Difference in capitalization results in a 404.

Thanks @DentonGentry


fix place planets in chrome

When the entire rotation is changed, the planets are based on the rotation small to largely offseted


Fix/374 delete deprecated js libraries

Q
A

Branch
master

Bug fix?
no

New feature?
no

BC breaks?
yes

Deprecations?
no

Tests pass?
yes

Fixed issues
chameleon-system/chameleon-system#374

License
MIT

Thanks @pixeljunkie


Explicitly specify both draw syle (line/fill) and color in call.

Needing to call the setColor function to set a color and then call the draw call with the "line" drawstyle is a source of needless extra bugs and complexity. It adds an additional round trip across the CAPI boundary and makes bugs involving forgetting to reset the global state more likely.
Reading the code suggests there is currently no way to provide a material (or otherwise) that specifies color information to a call using "line" drawstyle. Is this correct? Is there a way to do this? Should there be a way to do this?

It's true, right now using a Material instead of a DrawStyle forces the "fill" style.  I think this is because usually materials are/were used for texturing and shading, where having a filled shape with a surface is desired.
Using a Mesh would be a way to work around this and also get better performance, but it is likely going to be cumbersome to set up.  Whether that would be a worthwhile tradeoff depends on the project.
It would be possible to modify the API for primitive function variants that accept materials to also accept a draw style, but this also complicates the bindings a bit and forces people going through the supposedly common path to supply an extra "redundant" argument.  So I'm kind of lukewarm on it, especially when setColor can already be used here.


Sync tags between devices

Before you start...

[X] Have you refreshed the page? You might be using an old version of the website.
[X] Have you checked if someone already requested this feature? If they have, post a comment with your use case instead.

Description
Need to sync tags between different computers.
Alternative solutions
No response
Relevant Assets
No response

Thanks for the feedback, In future please use the Files issue tracker https://github.com/files-community/Files/issues and not the website one for issue about the program.
Tags can already be copied between devices by exporting your settings in Files settings > Advanced > Export and then imported on the other computer in the same page. If you have further questions or issues join the discord https://discord.gg/files


taskcat test run --keep-failed -> ValueError status is not a valid property of <class 'taskcat._cfn.stack.Stack'>

Describe the bug
Ran taskcat test run --keep-failed across (21) AWS Commercial Regions.  It did not delete ANY of the CloudFormation Stacks (both successful & failed).

After reporting on messages come out, see below messages:

[INFO   ] : deleting successful stacks
[ERROR  ] : ValueError status is not a valid property of <class 'taskcat._cfn.stack.Stack'>

To Reproduce
Steps to reproduce the behavior:

Are you testing a QuickStart or Custom template? No
Attach or link a copy of the template if possible (remove any sensitive info)
Provide the parameters that you passed. (remove any sensitive info)
How did you install taskcat? (docker or pip3) pipx
Are you using a profile, an instance role or  access keys to run taskcat? profile
Is your AWS environment configured via aws configure? yes

Expected behavior

I was expecting that it would delete the stacks in all AWS regions where stack deployed successfully, and that it would not delete the stacks in the AWS regions where stack deployment failed.
FYI, it works properly when I do taskcat test run, and then stacks are deleted in all AWS regions (both successful & failed stacks)

**Version (Please make sure you are running the latest version of taskcat)

Taskcat Version: 0.9.23
Python3 Version: 3.8.7

This is something I can help with. Let me do some digging around and I will let you know when I have submitted a fix.

❤

@andrew-glenn Trying to track this down but its a harder than I expected.
TLDR: I found the fix you can skip to the bottom if you want
It starts in _cfn_test.py where we try to delete only the completed stacks. We do that by filtering out stacks using this dictionary {"status": "CREATE_COMPLETE"}. Where the key will be the attribute of the stack as in stack.status and the value is the condition we are looking for.
So the error is actually coming from the filter function. But what I can't figure out is how the stack object doesn't have the status attribute as its clearly defined.
I can only think of two causes. Either a object that wasn't a stack got added to stackers self.stacks or somehow a stack doesn't have the status attribute but neither of these make sense to me?
Okay so I think this is by design. Apparently a property behaves as a Class attribute and not a instance attribute!
You can see it with this little test.
>>> class foo():
...     def __init__(self):
...             self._status = 'test'
...     @property
...     def status(self):
...             return self._status
...
>>> test = foo()
>>> print(test.__dict__)
{'_status': 'test'}
>>> print(foo.__dict__)
{'__module__': '__main__', '__init__': <function foo.__init__ at 0x7fdf8d479598>, 'status': <property object at 0x7fdf8d46cbd8>, '__dict__': <attribute '__dict__' of 'foo' objects>, '__weakref__': <attribute '__weakref__' of 'foo' objects>, '__doc__': None}

I know I have used keep failed and it worked but I don't remember when?
TLDR
Okay I see the problem now. I'm really glad I wrote all this out or I wouldn't have found it lol. I don't know who this guy is but he broke it ;).
I will submit a fix and hopefully a working test case.


Add metal LB config for sandbox environment

we have to be careful not use the IP(s) that kind is using here. but we can do with annotations.
/approve
/lgtm


Might be worth clarifying difference between tenantID and directoryID

Will be worth clarifying difference between tenantID and directoryID, since both these terms are used in the documentation and the azure portal.  Its confusing. Are they the same, are they different. if the same why two terms (is there some back story we need to know about)?

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

ID: 47010f4d-7d5e-c26d-4b9f-066d510c5f35
Version Independent ID: 4a50a101-0899-69f9-0177-6bbeeed27948
Content: Manage Azure portal settings and preferences - Azure portal
Content Source: articles/azure-portal/set-preferences.md
Service: azure-portal
GitHub Login: @JnHs
Microsoft Alias: jenhayes

@dombat
Thanks for your feedback! We will investigate and update as appropriate.

@dombat, thank you for the feedback, You are right - directory and tenant are used interchangeably to represent the same thing. "Dorectory" comes from the on-prem era Active directory, whereas the tenant is the cloud-based solution for same thing. I am assigning this issue to our content team for further review and standardization of the document/portal, as applicable.
@JnHs, could you please review this feedback and share your thoughts as well. Thank you

Hi @dombat - thanks for sharing your feedback. As @AnuragSingh-MSFT notes above, "directory" and "tenant" mean the same thing in this context, with "tenant" being the more current term. We generally use "tenant" (or "tenant ID") in the Azure portal docs, although there are some cases where the portal references "directory". I've added a few clarifications where we mention directories to confirm that this refers to an Azure tenant.
I'll close this issue now, thanks again for taking the time to help us improve the docs!
#please-close


segment generation is slow

Hey! I'm simply amazed by the crazy fast inference time!!!
I have the following code snippet:
start = time.time()
print("Transcribing audio file...")
segments, _ = model.transcribe("samples/sinan_speech_30s_16.wav", word_timestamps=True, beam_size=1)

# print("Detected language '%s' with probability %f" % (info.language, info.language_probability))

print("Transcription took %.2fs" % (time.time() - start))
print('\n')

for segment in segments:
    for word in segment.words:
        # print("[%.2fs -> %.2fs] %s" % (word.start, word.end, word.word))
        print(word.word, end='', flush=True) # same 'speed' with flush=False

For some reason, the loop takes forever... The transcription speed is about 1.39s for a 30s audio clip (just AMAZING?!?!) but the loop takes around 10s
I really appreciate any help!

I am experiencing the same issue. I think it is because segments are generators and you must wait up to finish their processing, isn't it?

Really, is it like that? Is there a way around that? And does that mean that transcription times are incorrect?

When transcribing a 2s audio file with word_timestamps, I get a performance of about 0.4s which is only slightly faster than the openai/whisper model (0.5s). Is that expected

This was already discussed several times. See #67 or #117.


Error while Scan (time.Time -> *sql.RawBytes)

type User struct {
  Preferences *Preferences `sql:"-"`
  UpdatedAt time.Time
}

When I perform ScanAliased(&preferences) after ScanAliased(&user) I get this error right after ScanAliased(&preferences) performed:
sql: Scan error on column index 11: unsupported driver -> Scan pair: time.Time -> *sql.RawBytes
I've quickly changed this line as v = &v to avoid the issue but I'm not sure this is the correct way

What's the definition of Preferences?

Preferences struct is consist of a few string, int64, *string and bool.
Let me give you better view of the issue. I can perform ScanAliased(&user) for select ... from users or ScanAliased(&preferences) for select ... from preferences without any issue.
But when I do ScanAliased(&user) following ScanAliased(&preferences) for select user...., preferences.... from users left join preferences... I get that error in the ScanAliased(&preferences) part. I guess this is because it skips all the user values with &sql.RawBytes{} to get preferences but can't skip mysql datatime with it for UpdatedAt. It works ok when I remove UpdatedAt from User struct. I can prepare a test for this if you want. Thanks!


Large rollbacks are still slow

Large rollbacks like this:
[db-sync-node:Info:65] [2022-01-14 23:44:26.44 UTC] Deleting 63140 blocks up to slot 45964772
[db-sync-node:Info:65] [2022-01-15 00:26:56.43 UTC] Blocks deleted

However, I have found that they can manually be sped up by breaking them into smaller chunks by doing something this multiple times:
delete from block where slot_no = (select slot_no from block where slot_no is not null order by slot_no desc offset 1000 limit 1) ;

We should explore doing something like this from the Haskell code when we are rolling back more than an epoch worth of slots (really only something that is done during debugging).

Something has changed recently. Rollbacks which were relatively quick for a while are slow again :sob: .

Fix by https://github.com/input-output-hk/cardano-db-sync/pull/1022


Fix deployment test in acations

Github actions was broken because of version checking changes. Now fixed.
See: https://docs.google.com/spreadsheets/d/1krJNd-FYvC-iwrBHiZPCX5AIFOW9xEGLWmhxdK8xuTc/edit#gid=0

Thanks for the quick review! merging now!


Do not move the test to child job if parallel list number=1

Close #5375

https://ci.adoptium.net/job/Test_openjdk23_hs_special.system_x86-64_linux/10/

thanks @sophia-guo - we have already branched for Sept release, could this also be cherrypicked to v1.0.3-release branch please.


Rewrite describe and it to use test steps

[x]  https://github.com/udibo/test_suite/issues/1
[ ]  https://github.com/udibo/test_suite/issues/7
[x]  https://github.com/udibo/test_suite/issues/8
[x]  https://github.com/udibo/test_suite/issues/9

[x] Implement only on test steps
[ ] Add test coverage for call signatures that include suite as first argument.
[x] Add test coverage for describe block hooks in callback
[x] Add test coverage for describe block hooks in options
[ ] Add test coverage for error from permissions option on test step
[ ] Add test coverage for error adding global hooks after registering first test
[ ] Add test coverage for error adding hooks after tests start running

[x]  https://github.com/udibo/test_suite/issues/10
[x]  https://github.com/udibo/test_suite/issues/13

[x] Update examples to use describe/it without nesting
[x] Update readme to include the examples
[x] Remove old test_suite.ts file
[x] Add migration path to readme from old test_suite.ts syntax to new describe/it without nesting

I'm going to ship this without each implemented for now. I plan on adding each functions onto describe and it. The call signature for each will be similar, taking a cases property that is an array of data used for creating the tests.


Sidebar overlays footer

1 - Describe the bug
The sidebar list overlays footer when scrolling to the bottom of the page.
2 - Expected behavior of what should have happened (if applicable)
3 - Screenshots (if applicable)

4 - Additional notes

can i work on this issue?

can i take thiis up

Hi @arjuna1729 can you provide more info on this? What browser are you using and what's the screen width that this bug is happening on? Thanks!

I am using Mozilla Firefox. I think that's happening in all browsers because it's the CSS syntactical error.

You are all correct this is due to an increase in the number of items, not a CSS syntactical error. It can be resolved in multiple ways.
@arjuna1729 thank you for reporting the bug. Would you like to work on it or should I assign it to someone else?

can i work on this?

you may go ahead @pinkman7009

Footer has to be changed. Need to fill it up with more content.

thank you @shahednasser . i will start working on this,

@adsingh14 please start a new issue with any idea you have instead of commenting on unrelated issues.

@shahednasser i am not getting this error on chrome and firefox at 100% width screen size. can you tell me where i might be going wrong?

This could be due to different screen sizes. What's your screen width?

@shahednasser my screen size is 15.6 inch which is around 1450 pixels.

That's odd but everyone has seen the bug by now. If you can't reproduce it then maybe we should assign it to someone else. Please let me know.

@shahednasser okay then i guess you should assign it to someone else.

@Dheerajbana do you still want to work on this?

@shahednasser  Ok i will take this up. Was able to reproduce it on firefox with 133%

@shahednasser i have added fix and created a pull request
https://github.com/sButtons/sbuttons/pull/621

This issue is open for assignment again.

I will like to try this

Sure @Goodiec you can work on it.

Can I try this?

Someone is assigned now @Hirak-Pandya

Hi,
I think I was able to fix this issue. Will be very helpful if someone is able to do a code review for the same. :)

Hi @JithinAji someone is working on this. Please look for issues that are unassigned.

sure. 👍  @shahednasser

@shahednasser, the most I could do was make it go behind the footer and that isn't the best solution. I think @JithinAji should try his fix.

Also, I think coming up with another way to show the menu items would be nice as the list is bound to keep increasing

Can I work on this @shahednasser ?

Hi @Caleb335 sure you can! However, we're currently working on a website revamp so we just need a temporary solution. You can try a simple solution such as hiding the sidebar behind the footer for now. Thank you!

Hi @Caleb335 sure you can! However, we're currently working on a website revamp so we just need a temporary solution. You can try a simple solution such as hiding the sidebar behind the footer for now. Thank you!

Alright @shahednasser I'd do just that.

@shahednasser How I did was giving the margin top a negative value whenever the bottom page is reached on scroll.


demo with true magellan-nightwatch page object

Convert all commands and assertions used in demo to customized counterparts to demonstrate page object support of magellan-nightwatch.

:+1:


hardening: use random mac address by default

Replace #207
Changes:

PR to staging instead of live
Commit message now follows guidlines.

Declining due to the prior comments.

Hi @qoijjj thank you for reviewing, but can you please specify which prior comments are you referring to?

Hi @qoijjj thank you for reviewing, but can you please specify which prior comments are you referring to?

The three comments I put on the files

Hi, sorry for the trouble, but I was not able to see the comments, do you mind copying them here?

Hi, sorry for the trouble, but I was not able to see the comments, do you mind copying them here?

I think github wasn't showing them because they were pending. You should be able to see them now :smile:

Thank you! I agree, we need more documentation and could potentially out of scope, as this is more of a privacy hardening than security hardening.

it is harmful as it is not contradictory to security, but it breaks things currently.
a just setting to enable random MAC on all new networks could be handy though, KDE and GNOME at least allow to set it to static manually. That UX is not easy though, so totally opt-in only for now.


Add iree_ref for gemm functionality test

This PR adds utils to generate an iree reference
mmt kernel that can be compiled and run to
obtain a golden output for comparison. Compared
to the torch output, the advantage of this output
is that it can be matched exactly and makes for
an easier comparison target.

This test will fail until https://github.com/iree-org/iree-turbine/pull/101 lands.

Overall looks awesome! just some NITs and Qs then we should be good to go! :)


serde support?

Hello,
I'm using the warc library and need to serialize WARC headers, a feature that is not yet supported/implemented.
Although one can say that serde support is not in the scope of the crate (which I believe is a valid point of view), I think that it would fit the potential usage of warc.
Moreover,  serde could be a feature flag, thus letting the users decide!
I've already implemented a quick way of serializing headers in my fork, and would be glad to participate in implementing the feature in the main repository if needed :)

Thanks @Uinelj. This diff seems pretty tame. I'm going to defer to @jhwgh1968

Hi @Uinelj, if you put back the ToString impl you deleted (which is used a lot), this is completely fine with me. Feel free to submit a PR.
That said, know that I am currently working on an API overhaul (slowly). I would not block you, but the next release (0.3.0) may be a while yet. You would have to base your work on current master until then.

If I'm not mistaken, the Rust docs states that ToString shouldn't be implemented directly, as implementing Display gets us the ToString implementation automagically. However if there's a reason to keep ToString only I'd be completely fine removing my change (and learning a bit more about Rust :))
I'll base my work on current as it is already the case.

If I'm not mistaken, the Rust docs states that ToString shouldn't be implemented directly, as implementing Display gets us the ToString implementation automagically.

I missed that. LGTM then!

Closed by #25


move plugin_management into core

All tests pass for me locally

If we're going the persistent file route, are we sure that we want three separate dummy testing directories, vs. starting with e.g. a dummy library that contains a dummy plugin that can be minimally modified as needed in test setup? It's easier to spot inconsistencies in behavior, etc. with one as opposed to maintaining multiples.

The reason I had three separate repositories was that they all needed slightly different things, so I wanted to avoid a potentially growing mega directory that will have to cater to all sorts of different tests. I'm not sure how you would setup the "base" dummy repository, can you post an example?


fix: ensure ssrRewriteStacktrace doesn't throw

This is a weird edge case so feel free to just close this PR if it doesn't make sense.
Basically right now there's a weird bug in webcontainers where any error during ssr makes the dev server crash. The reason it happens is because ssrRewriteStackTrace it's called twice (for some reason it doesn't happen locally).
This PR fixes this behavior by wrapping ssrRewriteStackTrace in a try/catch and returning the stack as is if it throws.
Again i understand this is a small enough edge case but i tought that given that sometimes ssrRewriteStackTracecan throw would still be a good addition.
Please don't delete this checklist! Before submitting the PR, please make sure you do the following:

[x] It's really useful if your PR references an issue where it is discussed ahead of time. In many cases, features are absent for a reason. For large changes, please create an RFC: https://github.com/sveltejs/rfcs
[x] This message body should clearly illustrate what problems it solves.
[ ] Ideally, include a test that fails without this PR but passes with it.

Tests

[x] Run the tests with pnpm test and lint the project with pnpm lint and pnpm check

Changesets

[ ] If your PR makes a change that should be noted in one or more packages' changelogs, generate a changeset by running pnpm changeset and following the prompts. Changesets that add features should be minor and those that fix bugs should be patch. Please prefix changeset messages with feat:, fix:, or chore:.

This looks like a good change to me, but I'd like to get some more details. How do we reproduce this issue? Any idea why ssrRewriteStackTrace is called twice? (I'm assuming that shouldn't happen). Can you share a stack trace from a crash?

This looks like a good change to me, but I'd like to get some more details. How do we reproduce this issue? Any idea why ssrRewriteStackTrace is called twice? (I'm assuming that shouldn't happen). Can you share a stack trace from a crash?

Unfortunately it's a super weird issue that I was not able to reproduce consistently...i only seen it happen in webcontainers but I while I was implementing an hacky fix for sveltelab (https://github.com/SvelteLab/SvelteLab/pull/278) I discovered that locally it doesn't happen and it only happens on the deployed version on vercel.
I was able to reproduce in stackblitz too tho.

How often does it happen? Can you reproduce it somewhat consistently or is it very rare? I wonder if we should log that an error was encountered rather than silently ignoring it? On the one hand, I don't want to annoy users, but on the other, I don't want to just ignore that something bad is happening and cover up the underlying bug rather than fixing it
@gtm-nayan pointed out the other day that Vite added a check to avoid rewriting stack traces twice in https://github.com/vitejs/vite/commit/feb8ce05bf2c07f9ea1c6facc75248d2c0e995b8. As a result, we may not need fix_stack_trace anymore and could just rely on Vite as we added our code to avoid having stack traces written twice before Vite did

How often does it happen? Can you reproduce it somewhat consistently or is it very rare? I wonder if we should log that an error was encountered rather than silently ignoring it? On the one hand, I don't want to annoy users, but on the other, I don't want to just ignore that something bad is happening and cover up the underlying bug rather than fixing it
@gtm-nayan pointed out the other day that Vite added a check to avoid rewriting stack traces twice in vitejs/vite@feb8ce0. As a result, we may not need fix_stack_trace anymore and could just rely on Vite as we added our code to avoid having stack traces written twice before Vite did

I've also opened an issue on webcontainers with the reproduction. But basically either go on sveltelab or stackblitz and throw an error in the load function.
Here's the issue on webcontainers ( https://github.com/stackblitz/webcontainer-core/issues/1155 )

Maybe the best course of action would be still wrapping the call in a try catch given that it could throw but add the error to the returned stack in the catch?

@paoloricciuti does the error still happen in the latest release since https://github.com/sveltejs/kit/pull/10769 was merged? If so, this PR will need a rebase

@paoloricciuti does the error still happen in the latest release since #10769 was merged? If so, this PR will need a rebase

Uh I missed the linked PR...I have to check, I'll do it in a moment

It seems to work fine on stackblitz ( I can't test on sveltelab because I was temporarily patching sveltekit with this solution to make it work)

You don't need this change now, right?

You don't need this change now, right?

No it seems to have been fixed by #10769


Fix CT drawing tests on ARM

Changes more tests to be more lenient, and changes the templates for the image comparison to work with the Gtest macros
Fixes #1772

This change is

:shipit:


Add flag on application creation to indicate core app or dependency

Unclear what the business value and LOE are.  Any insights you can share, @noelo?

The customer will have applications which are not to be assessed but
are required for dependency management.
The back end is done and just the UI needs to be updated when creating
the applications
On Wed, Jun 6, 2018 at 9:17 PM texasbrok<EMAIL_ADDRESS>wrote:

Unclear what the business value and LOE are. Any insights you can share, @noelo?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

The business value is to allow us to capture app dependency information so we can produce dependency maps, which feeds into the process of recommendations of which apps are easier/harder to migrate.
We need to analyse this further before sizing or beginning work.
ie.

when an app is a dependency, should it be displayed in the "assessments summary" screen?
assuming not to the above, we will need a way to find/edit and remove dependency only apps, so will all apps be displayed in the "customer applications" page?
should we have a column indicating which are dependency only so it's clear why they're not appearing in some places (such as the assessments screen)?


Responsive support?

Is there any way to use this beautiful plugin in responsive way?
I mean, we can change some of its values (e.g. width, height etc.) on $(window).resize();
$(window).resize(function() {
    knob.trigger('configure', {
        width: 'width-value',
        height: 'height-value'
    });
});

And it redraws the canvas element, which is used to render the knob, but it doesn't update the 'width' and 'height' attributes of generated div which contains the canvas and the canvas attributes too.

Up


Allow programmatic configuration of unicast relays.

This change allows users to configure relays from code without having to setenv(GZ_RELAY).
🎉 New feature
Summary
This change allows users to configure relays from code without having to setenv(GZ_RELAY).
Supersedes https://github.com/gazebosim/gz-transport/pull/497
Test it
This can be tested by starting a Node across a boundary where multicast isn't enabled, and setting the relay IP via NodeOptions.
Checklist

[ ] Signed all commits for DCO
[ ] Added tests
[ ] Added example and/or tutorial
[ ] Updated documentation (as needed)
[ ] Updated migration guide (as needed)
[ ] Consider updating Python bindings (if the library has them)
[ ] codecheck passed (See contributing)
[ ] All tests passed (See test coverage)
[ ] While waiting for a review on your PR, please help review another open pull request to support the maintainers

Note to maintainers: Remember to use Squash-Merge and edit the commit message to match the pull request summary while retaining Signed-off-by messages.

Friendly ping @caguero

By making Discovery::AddRelayAddress() public, I think we now have risk for potential race conditions, as this->relayAddrs can be modified from a user thread (via Node) and the discovery thread can read the same variable via SendUnicast().

Should I add a new mutex to guard relayAddrs or just use the existing Discovery::mutex?

By making Discovery::AddRelayAddress() public, I think we now have risk for potential race conditions, as this->relayAddrs can be modified from a user thread (via Node) and the discovery thread can read the same variable via SendUnicast().

Should I add a new mutex to guard relayAddrs or just use the existing Discovery::mutex?

I think you can reuse Discovery::mutex.

By making Discovery::AddRelayAddress() public, I think we now have risk for potential race conditions, as this->relayAddrs can be modified from a user thread (via Node) and the discovery thread can read the same variable via SendUnicast().

Should I add a new mutex to guard relayAddrs or just use the existing Discovery::mutex?

I think you can reuse Discovery::mutex.

Is there a performance concern to worry about re: serializing the user thread + discovery thread in the presence of unicast relays? Every unicast write would need to acquire that mutex.

Thoughts on this test failure https://build.osrfoundation.org/job/gz_transport-pr-win/99/ ? I'm having a hard time figuring out how this PR would trigger behavior change on those tests.

By making Discovery::AddRelayAddress() public, I think we now have risk for potential race conditions, as this->relayAddrs can be modified from a user thread (via Node) and the discovery thread can read the same variable via SendUnicast().

Should I add a new mutex to guard relayAddrs or just use the existing Discovery::mutex?

I think you can reuse Discovery::mutex.

Is there a performance concern to worry about re: serializing the user thread + discovery thread in the presence of unicast relays? Every unicast write would need to acquire that mutex.

By making Discovery::AddRelayAddress() public, I think we now have risk for potential race conditions, as this->relayAddrs can be modified from a user thread (via Node) and the discovery thread can read the same variable via SendUnicast().

Should I add a new mutex to guard relayAddrs or just use the existing Discovery::mutex?

I think you can reuse Discovery::mutex.

Is there a performance concern to worry about re: serializing the user thread + discovery thread in the presence of unicast relays? Every unicast write would need to acquire that mutex.

I'd expect a bit of impact if unicast is used because the mutex will be acquired when calling DispacthDiscoveryMsg(). It shouldn't make a big difference.

Thoughts on this test failure https://build.osrfoundation.org/job/gz_transport-pr-win/99/ ? I'm having a hard time figuring out how this PR would trigger behavior change on those tests.

ReplayTest looks unrelated.

Friendly ping @caguero .

Minor comments.

Done.

Not sure why the ABI checker doesn't catch it but I think that changing the access right (from private to public in our case) in Discovery::AddRelayAddress() breaks ABI.
I think the solution is to create a new public version of the function that calls the existing one.

Let's just target main?

Not sure why the ABI checker doesn't catch it but I think that changing the access right (from private to public in our case) in Discovery::AddRelayAddress() breaks ABI.

I think it's actually fine on Linux, it's only an issue with MSVC where the symbol mangling includes the access rights

@osrf-jenkins run tests

Not sure why this gz_transport-ci-pr_any-jammy-amd64 job is stuck.

Our CI was recently changed so that main branches only running tests on Noble. But the branch protection for gz-transport wasn't updated. Should be good now.

We do need to update the GitHub workflow to use Noble . @caguero would you mind doing that when you get a chance?

We do need to update the GitHub workflow to use Noble . @caguero would you mind doing that when you get a chance?

Thanks @scpeters https://github.com/gazebosim/gz-transport/pull/504


Build says "passed" but batch says "unkown version"; what to do?

Hi,
my project  doesn't show any errors, but the batch () still is not happy. Can't find anything in the docs or the FAQ. What to do?

It seems to look okay on our side: https://readthedocs.org/projects/monk-tf/
-- I'm guessing this is GitHub caching the image.
On Tue, Jan 20, 2015 at 6:14 AM, Erik Bernoth<EMAIL_ADDRESS>wrote:

Hi,
my project https://github.com/DFE/MONK/ doesn't show any errors, but
the batch ([image: Documentation Status]
https://readthedocs.org/projects/monk-tf/?badge=latest) still is not
happy. Can't find anything in the docs
https://docs.readthedocs.org/en/latest/versions.html or the FAQ
https://docs.readthedocs.org/en/latest/faq.html. What to do?
—
Reply to this email directly or view it on GitHub
https://github.com/rtfd/readthedocs.org/issues/1127.

--
Eric Holscher
Maker of the internet residing in Portland, Or
http://ericholscher.com

yeah might be. Getting the green batch in the RTD Profile as well. Then I
simply look again tomorrow if it has changed. Thanks
On Tue, Jan 20, 2015 at 5:59 PM, Eric Holscher<EMAIL_ADDRESS>wrote:

It seems to look okay on our side:
https://readthedocs.org/projects/monk-tf/
-- I'm guessing this is GitHub caching the image.
On Tue, Jan 20, 2015 at 6:14 AM, Erik Bernoth<EMAIL_ADDRESS>wrote:

Hi,
my project https://github.com/DFE/MONK/ doesn't show any errors, but
the batch ([image: Documentation Status]
https://readthedocs.org/projects/monk-tf/?badge=latest) still is not
happy. Can't find anything in the docs
https://docs.readthedocs.org/en/latest/versions.html or the FAQ
https://docs.readthedocs.org/en/latest/faq.html. What to do?
—
Reply to this email directly or view it on GitHub
https://github.com/rtfd/readthedocs.org/issues/1127.

--
Eric Holscher
Maker of the internet residing in Portland, Or
http://ericholscher.com
—
Reply to this email directly or view it on GitHub
https://github.com/rtfd/readthedocs.org/issues/1127#issuecomment-70690010
.

The other badges all updated by now, but as you might still see above the docs link didn't update (although loading the batch directly in the browser shows a green "latest"). Maybe there might be a nearly hidden problem with how the HTTP response explains its caching desires. I'm not sure if you set the batch response headers yourself or if you use an external service for this. But maybe someone can have a look at that other issue and see if an improvement could be implemented.

Hm, seems like the referenced project is removed. For the badge URL we perform a redirect to the shields.io service. The 302 does not contain any caching information.
I guess we could add this to be ~5 Minutes or so.

Oh and we cache the badges up to 30 minutes in the Django level of things, since https://github.com/rtfd/readthedocs.org/commit/bf8931082f29ca1f6f3825b0d3bdc99edbcf537f

Believe this should be fixed, as it was an issue with our caching.


[RESOLVED] Interface eth0(): High error rate (>2 for 5m)

Problem started at 10:06:29 on 2023.05.13
Problem name: Interface eth0(): High error rate (>2 for 5m)
Host: S100-Primario
Severity: Warning
Operational data: errors in: 2, errors out: 0
Original problem ID: 20595
Event details in Zabbix: http://<IP_ADDRESS>/tr_events.php?triggerid=23483&eventid=20595

Problem has been resolved in 6m 0s at 10:12:29 on 2023.05.13
Problem name: Interface eth0(): High error rate (>2 for 5m)
Host: S100-Primario
Severity: Warning
Original problem ID: 20595


Corejs snafu

working as expected ... branched off of other pr without corejs commented out

yes, working as expected on ie 11

yes, working as expected on ie 11


Fix Shipping Bug for Checking Zip Code

Reference No: #30
Implement RegEx for Zip Code Validation in Shipping Page  Form. Only Numeric 6 Digits will be allowed in Zip Code.
File Change

Note -
While I was working on Zipcode, I notice a bug that whenever a user changes phone number in shipping form, a warning error occurs about uncontrolled and controlled values for the phone number section. I am a little bit unaware of this error. I also followed the same steps at the official deployed site, but the error warning did not pop up over there.
Error -

@Shubham29012001 Don't worry about that warning, it's because initially phone_number is null so react treats the input as uncontrolled, and when we change the value of phone_number the input becomes control. That's why React complaints. Ignore that warning 😅
Also, for the regex, move the if condition you implemented, next to the if where we are checking:
if (!postal_code) {
    ....
}

Hey @varunKT001, can you please check now?

@Shubham29012001 Done 👍
Thanks for contributing 🎉


Adding cheatsheets of Git Configuration

What does this PR do?

Add new CheatSheet

Description

Adding more configuration commands in Git Configuration

Can you please add label hacktoberfest and hacktoberfest-approved for me?


Problem with reqirements.txt

Hi I have problems with setting up the book_env.
(book_env) C:\Users\TGrigorov>pip3 install -r %cd%\Documents\Python\Packt\Hands-On-Data-Analysis-with-Pandas-2nd-edition-master\requirements.txt
Collecting git+https://github.com/stefmolin/login-attempt-simulator.git@2nd_edition (from -r C:\Users\TGrigorov\Documents\Python\Packt\Hands-On-Data-Analysis-with-Pandas-2nd-edition-master\requirements.txt (line 15))
Cloning https://github.com/stefmolin/login-attempt-simulator.git (to revision 2nd_edition) to c:\users\tgrigorov\appdata\local\temp\pip-req-build-zgwb1xxn
Running command git clone -q https://github.com/stefmolin/login-attempt-simulator.git 'C:\Users\TGrigorov\AppData\Local\Temp\pip-req-build-zgwb1xxn'
ERROR: Error [WinError 2] Das System kann die angegebene Datei nicht finden while executing command git clone -q https://github.com/stefmolin/login-attempt-simulator.git 'C:\Users\TGrigorov\AppData\Local\Temp\pip-req-build-zgwb1xxn'
ERROR: Cannot find command 'git' - do you have 'git' installed and in your PATH?
I managed to install evrything till "wheel". But "login-attempt-simulator", "ml-utils", "stock-analysis" and "visual-aids" are not installed.

Hi @TsvetalinGr,
The error message is telling you that pip doesn't know where you have installed git. Can you confirm that you have git installed? It can be installed for Windows here.

(book_env) C:\Users\tompe\my_folder>pip3 install -r requirements.txt
Collecting git+https://github.com/stefmolin/login-attempt-simulator.git@2nd_edition (from -r requirements.txt (line 15))
Cloning https://github.com/stefmolin/login-attempt-simulator.git (to revision 2nd_edition) to c:\users\tompe\appdata\local\temp\pip-req-build-vc8wlanj
Running command git clone -q https://github.com/stefmolin/login-attempt-simulator.git 'C:\Users\tompe\AppData\Local\Temp\pip-req-build-vc8wlanj'
ERROR: Error [WinError 2] The system cannot find the file specified while executing command git clone -q https://github.com/stefmolin/login-attempt-simulator.git 'C:\Users\tompe\AppData\Local\Temp\pip-req-build-vc8wlanj'
ERROR: Cannot find command 'git' - do you have 'git' installed and in your PATH?
**Hi Stef
I am getting the same error. I have git installed but how do I let pip know where it is?**

@tompearce - you need to add it to the PATH environment variable. Currently, pip is checking for git.exe in all the locations listed in PATH, but the location for the git executable is not there. I believe during installation of git you have the option to update PATH automatically, but you can do it at any time: find the filepath to git.exe on your local machine (for example, C:\Windows\...\git\bin) and then add it to PATH (instructions here).

@stefmolin -Great thanks. PATH updated and now all installed fine.

@TsvetalinGr - Were you able to resolve your issue?


Redesign the layout of the card

Suggestions:

changing the fonts used in the card.
changing the position of the content
adding a radius to the corners of the card

Hi, I'd like to have a go at this :)


[WIP] iron-0.5 update

This is a work in progress patch for iron-0.5 update. staticfile is working fine but tests are failing due to iron-test crate is not compatible with iron-0.5 yet. I am preparing an update for iron-test crate.

Looks like there is a ongoing PR: reem/iron-test#43

Indeed there is an ongoing PR. :) Thanks for working on this!

I updated tests, it is temporary using git dependency for iron-test, but iron-test stopped working with percent decoded paths: reem/iron-test#44 and that is why decodes_percent_notation test is failing for now.

@onur need any help ?

@ernestas-poskus this PR is actually ready but reem/iron-test#44 needs to be fixed.

This PR is ready if using iron-test from git repository is ok.

Thanks for this! Sorry it's been such work to get this merged. :(


Image from network not rendered

Hi! I need to generate a product grid, like a catalog, by requesting a service which provides me a list of products... Everything is rendered but not the images of the products.
This is what I get...

same issue

Did you guys ever find a fix for this?

you can change image to base64,   html2canvans seems cannot render image cross domain

I've solved this problem by adding "useCORS: true" to html2canvas like this exemple:
html2canvas: {scale: 2, useCORS: true},


How to train using 2D RGB X-ray images

Is it possible to use this framework on 2D RGB X-ray images instead of CT scans?
Here are my data specifications and I want to know if it is sensible and possible to train it using your designs:

Collection of 2D rgb xray images
Lacks camera parameters
Dynamic in nature

Hi, thanks for your interest. Actually, our method is trained on 2D X-ray images instead of CT scans.
If you want to train on your collected 2D RGB x-ray images, the first step is to compute the camera poses for your images. There are two open-source tools for you to use:
DusT3R : https://github.com/naver/dust3r
COLMAP (SfM) : https://github.com/colmap/colmap
Wish this information could help you


fix: re-add CHANGELOG.md

Description
I accidentally deleted the CHANGELOG page in #840 and this PR is adding it again.
Before, I was under the impression, that the remote-content-plugin configured in docusaurus.config.js would add that at build time.
This PR is just for quick fixing. Afterwards, I think we should find a straight forward approach for maintaining the duplicated CHANGELOG.md on the website.
Thanks @evegufy for finding the missing CHANGELOG and letting me know 😁
Pre-review checks
Please ensure to do as many of the following checks as possible, before asking for committer review:

[x] DEPENDENCIES are up-to-date. Dash license tool. Committers can open IP issues for restricted libs.
[x] Copyright and license header are present on all affected files

website build check fails, can you please confirm that that's not related to this change?

It actually is related it seems. I was using the latest CHANGELOG.md from tractus-x-release repo, but this does not seem to be "website ready".
I'll have to check the differences tomorrow.
This to me even more shows, that it is unsustainable to duplicate the CHANGELOG. Maybe we should go for just linking

Hi @evegufy,
I now used the CHANGELOG.md version from our history, so it includes the build fixes.
I also opened a discussion to clarify, how we want to handle/represent the CHANGELOG on our website in general
Discussion: #844
FYI: @Siegfriedk @stephanbcbauer @danielmiehle @maximilianong @FaGru3n


better avatar images for mobile

@keybase/react-hackers

@keybase/react-hackers bump

@keybase/react-hackers bump


use rand_r instead of rand for multithreading purposes

This solves, at least for me, an issue where a lot of time was spent in kernel land and not generating the scene. Performance has increased tremendously due to this change.
NOTE This generates a broken scene.

Issue was resolved in 801c621c7c4ff6a9f8eb1cb7fdc33504a9a6a8b6


Cartographer_ros compiling error

Hi!
I followed the installation guide on google cartographer successfully, but realized the pbstream_trajectories_to_rosbag node is missing.
As such, I replaced the cartographer_ros package from the installation with this repo. However, there seems to be a error compiling with regarding "absl". Following is a snippet of the log:
CMake Error at CMakeLists.txt:57 (find_package):
  By not providing "Findabsl.cmake" in CMAKE_MODULE_PATH this project has
  asked CMake to find a package configuration file provided by "absl", but
  CMake did not find one.

  Could not find a package configuration file provided by "absl" with any of
  the following names:

    abslConfig.cmake
    absl-config.cmake

  Add the installation prefix of "absl" to CMAKE_PREFIX_PATH or set
  "absl_DIR" to a directory containing one of the above files.  If "absl"
  provides a separate development package or SDK, be sure it has been
  installed.

-- Configuring incomplete, errors occurred!
See also "/home/shaw/cartographer_ws/build_isolated/cartographer_ros/CMakeFiles/CMakeOutput.log".
See also "/home/shaw/cartographer_ws/build_isolated/cartographer_ros/CMakeFiles/CMakeError.log".
FAILED: build.ninja
/usr/bin/cmake -H/home/shaw/cartographer_ws/src/cartographer_ros/cartographer_ros -B/home/shaw/cartographer_ws/build_isolated/cartographer_ros
ninja: error: rebuilding 'build.ninja': subcommand failed
<== Failed to process package 'cartographer_ros':
  Command '['/home/shaw/cartographer_ws/install_isolated/env.sh', 'ninja', 'build.ninja']' returned non-zero exit status 1

Or is there a simpler way of me adding the missing node instead of replacing the entire package. Thank you in advance!

Hi @nyangshawbin
I got the same error. I solved it by running install_abseil.sh file in the original cartographer project. It is in the scripts directory.

Hi @ayushgargdroid , yes! That solved it. Thanks so much for your help, really appreciate it!!


Arrow too small

Touch the three bar menu.
We now see a little arrow in the upper right corner of the screen.
The arrow should be bigger, and in the upper left corner.

In fact the problem occurs on a wide variety of pages within this app.


grunt build failing

Hi, awesome project! I'd like to add some more shortcuts but I can't seem to build the project. I ran npm install and then npm install grunt -g but I'm getting this error when trying to build:
[lkysow@Lukes-Macbook learnGitBranching (master)]$ grunt build
Loading "shell.js" tasks and helpers...ERROR
>> TypeError: Cannot read property '_' of undefined
<WARN> Task "shell" not found. Use --force to continue. </WARN>

Aborted due to warnings.

And when run with --force:
lkysow@Lukes-Macbook learnGitBranching (master)]$ grunt  build --force
Loading "shell.js" tasks and helpers...ERROR
>> TypeError: Cannot read property '_' of undefined
<WARN> Task "shell" not found. Used --force, continuing. </WARN>

Done, but with warnings.

If you think the issue I'm seeing is something peculiar to my environment then no worries, I'll continue trying to debug it myself.

Installing v1.3.1 worked for me. I think the newest version is using ES6?


do not use shared billing for guardrails and audit projects - ok for prod/non-prod projects - due to remote org permissions

the guardrails and audit projects will experience IAM permissions issues like creating log sinks due to lack of remote org IAM permissions.  We are ok with the existing billing account user set on the remote org
see #229
https://github.com/GoogleCloudPlatform/pbmm-on-gcp-onboarding/blob/main/environments/common/common.auto.tfvars#L66

working

20240406: Closing issue during retrofit/rebase of this TEF V1 based/modified repo to TEF V4 standards
This issue may participate in the LZ refactor after rebase
Query on all issues related to the older V1 version via the tag
https://github.com/GoogleCloudPlatform/pbmm-on-gcp-onboarding/labels/2024-pre-tef-v4


Gaps in dsm

Even if a correspondingly tile exists for each input layer, they may have no common valid data region.
Should do this union earlier. Possibly want the task to be a class (with run and finalise methods), so that some state can be attached.
Also, need to ensure appropriate flags raised where extra inputs have no data. (e.g. if there is no elevation data for part of the tile, then shadows....) Part of this may be simplified using a datasetpipe decorator..

What should the wofl look like in regions where there is no elevation data?
Some possibilities include:

Conservatively mask it:

as terrain-shadowed
as no-data
as non-contiguous

Optimistically permit it (expecting it to only occur over ocean anyway)

A similar question is, what should happen if PQ is unavailable in some area?
Currently it looks like the data-cube is probably filling out arrays with NaN or zero, for elevation and pixelquality respectively. For PQ this should be equivalent to flagging that all the bad things (cloud, oversaturation, etc) are happening. For elevation... perhaps the tile is being aborted?

We need to be able to process areas of no elevation, otherwise we lose the offshore areas such as the GBR, which is a priority. So - optimistically permit it.
PQ is trickier. I favour having a matched set of NBAR to PQ in order to process a tile. If PQ is absent, then we should not proceed with processing the tile. However is PQ being calculated over the ocean? If not then we still need to process those tiles...


MAS 1.4.12 - Text spacing

No loss of content or functionality should occur if

line height is set to 1.5em
spacing is set to 2 em
letter space is set to .12em
word spacing is set to .16em

Front page download counts overflow the box
GitHub Usage repository information overflows box


Remove payment method color

Hi,
It would have been nice if it could close the color change option according to the payment method.
I do not want to remove it from CSS, because when I update it my changes will be lost.
Sorry for my weak english.

Example;


⚠ NCAA has degraded performance

In 360fd76, NCAA (https://www.ncaa.com) experienced degraded performance:

HTTP code: 200
Response time: 842 ms

Resolved: NCAA performance has improved in 2cbcb2d.


Add support for other LLM services

There's a few competitive services released their API just yet.

Google Bard: https://developers.generativeai.google/products/palm
Antroropic Claude: https://docs.anthropic.com/claude/reference/getting-started-with-the-api

An IDE is needed for Prompt Engineers. Or someone like you will take the initiative, enhance it with a Sublime Text Plugin, and bring this feature to Sublime Text, which would be truly wonderful for all of us. I hope you never lose your motivation for the project, I will become a sponsor as soon as possible - I sincerely thank you for your contribution.

Hi @yaroslavyaroslav @yigitkonur   - I believe we can make this easier
I’m the maintainer of LiteLLM - we allow you to deploy a LLM proxy to call 100+ LLMs in 1 format - PaLM, Bedrock, OpenAI, Anthropic etc https://github.com/BerriAI/litellm/tree/main/openai-proxy.
If this looks useful (we're used in production)- please let me know how we can help.
Usage
PaLM request
curl http://<IP_ADDRESS>:8000/v1/chat/completions \
  -H "Content-Type: application/json" \
  -d '{
     "model": "palm/chat-bison",
     "messages": [{"role": "user", "content": "Say this is a test!"}],
     "temperature": 0.7
   }'

gpt-3.5-turbo request
curl http://<IP_ADDRESS>:8000/v1/chat/completions \
  -H "Content-Type: application/json" \
  -d '{
     "model": "gpt-3.5-turbo",
     "messages": [{"role": "user", "content": "Say this is a test!"}],
     "temperature": 0.7
   }'

claude-2 request
curl http://<IP_ADDRESS>:8000/v1/chat/completions \
  -H "Content-Type: application/json" \
  -d '{
     "model": "claude-2",
     "messages": [{"role": "user", "content": "Say this is a test!"}],
     "temperature": 0.7
   }'

yes the proxy is cross platform
yes it's a proxy server that allows you to call all LLMs in one format. You can choose to deploy it or use it locally

@ishaan-jaff Thanks, I'll consider it in depth when I'll come closer to implementing this one.

It would be cool if this plugin supported Ollama. You can run it locally as a standalone server, and make API calls to it: https://github.com/jmorganca/ollama/blob/main/docs/api.md

Hi @james2doyle litellm already supports ollama

@ishaan-jaff Oh nice. I misunderstood what litellm was, I thought it was a hosted service

no worries - litellm is a python package to call 100+ LLMs in the same I/O format. We also offer a proxy server if you don't want to make code changes to your app

The good news is that PC 4.0.0 released has been just yet, which means soon it'll be possible to add support for a custom python libraries into the package. We're not there yet, coz packagecontrol.io itself still doesn't fully supports 4.0.0 scheme (e.g. arbitrary libraries as dependencies), but I believe that it would take about a month or two to make it happen.

Any news on this? Per now using https://github.com/icebaker/nano-bots-api via https://github.com/icebaker/sublime-nano-bots to talk to Ollama / Mistral locally. Which works, but would like to see if something using this could be better.

Nope unfortunately, every time I tried any local llm as an assistant for my language of interest I noticed a way below gpt4 suggestions quality. More than that the same picture I'm observing with the all competing services as perplexity.
So honestly I have no plans to implement this until things changes.
Peeking at the very last bard/geminy 1m context window though. Maybe it'll worth it.

Any news on this? Per now using https://github.com/icebaker/nano-bots-api via https://github.com/icebaker/sublime-nano-bots to talk to Ollama / Mistral locally. Which works, but would like to see if something using this could be better.

I was able to make Ollama working by some small changes, as Ollama API is compatible with OpenAI API:

instal package manually, so that I could make easily make changes to it
change request in openai_network_client.py, to use self.connection = HTTPClient('localhost:11434') instead of logic present there
added dummy token "ollama-dummy-longer-than-10-characters" (or remove token checking)
change models in assitants:

"assistants": [
	{
		"assistant_role": "Apply the change requested by the user to the code with respect to senior knowledge of programming",
		"chat_model": "codellama",
		"max_tokens": 4000,
		"name": "Replace",
		"prompt_mode": "replace"
	},
	{
		"assistant_role": "Insert code or whatever user will request with the following command instead of placeholder with respect to senior knowledge of programming",
		"chat_model": "codellama",
		"max_tokens": 4000,
		"name": "Insert",
		"prompt_mode": "insert",
		"placeholder": "## placeholder"
	},
	{
		"assistant_role": "Append code or whatever user will request with the following command instead of placeholder with respect to senior knowledge of programming",
		"chat_model": "codellama",
		"max_tokens": 4000,
		"name": "Append",
		"prompt_mode": "append"
	},
]

So it will work nicely, if there would be config options to:

toggle between HTTP and HTTPS
change URL
do not require token, when url is not api.openai.com (or just advice to add some dummy token)

@Aiq0 Confirmed working, thank you!
(Used HTTPConnection, not HTTPClient)

@Aiq0 Confirmed working, thank you! (Used HTTPConnection, not HTTPClient)

You are welcome. (sorry, that was typo)

@rubjo @Aiq0 glad to hear it folks!
It would be just awesome if you'd do an extra mile and perform a PR with such functionality. The network layer code is kinda the least confusing piece through the whole code base, so I believe it could be taken without paying too much effort for that.

I believe not much. Just please try to avoid to overcomplicating things. Like to not present an additional settings if they can be avoided (e.g. I believe that it's perfectly fine to come along with dummy token on the user side for a local models rather than providing a separate toggle for that).
If you're about to add some global settings options, please consider to do so on the first level if it's possible.
A few words dropped in Readme about this new feature would definitely worth it either.


consider adding rewrite rules for docs and latest versions

when promoting a release, symbolic links are automatically added for documentation and for release/latest using https://github.com/nodejs/nodejs-latest-linker:
https://github.com/nodejs/build/blob/0a1b2e9ad35ba7aab215e799529781a3522e00ce/ansible/www-standalone/tools/promote/promote_release.sh#L39
for example:

https://nodejs.org/download/release/latest-v20.x/ is a symlink pointing to https://nodejs.org/download/release/v20.7.0/
https://nodejs.org/download/release/latest/ is a symlink pointing to https://nodejs.org/download/release/v20.7.0/
https://nodejs.org/docs/v20.7.0/ is a symlink pointing to https://nodejs.org/download/release/v20.7.0/docs
https://nodejs.org/docs/latest-v20.x/ is a symlink pointing to https://nodejs.org/download/release/v20.7.0/docs
https://nodejs.org/docs/latest/ is a symlink pointing to https://nodejs.org/download/release/v20.7.0/docs

it might make sense to add these rules directly to the Cloudflare worker to avoid the need to copy duplicate files when promoting a release

As we publish a new version of node, we could call a github action that updates the redirect config?
https://developers.cloudflare.com/rules/url-forwarding/bulk-redirects/reference/json-objects/

we could call a github action that updates the redirect config?

I think either of those approaches are worth a shot and would be a nice way to do it but I'm not sure if they'd work for a worker.
If they don't we might need to try something like a bot or Github action that keeps a file named something like symlinkRedirects.ts up to date with the latest paths described like
export default {
  '/download/release/latest/': '/download/release/v20.7.0',
  '/dist/latest/': '/dist/v20.7.0',
  // ...
}

And then when we get a HTTP GET we can do something like
if (requestUrl.pathname in symlinkRedirects) {
  return Response.redirect(symlinkRedirects[requestUrl.pathname])
}

That would add the need for human action to approve the change, merge it, and then deploy it to prod however.
We might be able to get away with having the redirects in the environment variables of the worker so then the CI would just need to overwrite them via the wrangler cli but not sure if that would work. We would need to make sure to not overwrite the values that Cloudflare has by not setting them in the wrangler.toml

I think either of those approaches are worth a shot and would be a nice way to do it but I'm not sure if they'd work for a worker.

Right - both those approaches (redirect, rewrite) are processed before the request arrives at the worker. They are both more performant and invokes less worker cpu time.

Right - both those approaches (redirect, rewrite) are processed before the request arrives at the worker

In that case then yeah I think that's the way to go.
Also apologies for the edit, was on mobile and hit the wrong button without realizing

We now have (with https://github.com/nodejs/release-cloudflare-worker/pull/34 and https://github.com/nodejs/release-cloudflare-worker/pull/36) a list of redirects inside this repo, we should find the best way to use that.

I'd assume that updating the redirect config @jbergstroem pointed out could be done with another js file that interacts with Cloudflare's v4 api.
I wonder if we could add another action that fires only on pr merge and only if the json file gets updated. So for example whenever we merge a pr like #36, it'd fire and call the script to update the redirect config. Would require some more human input though to merge the pr unless we'd like setup something to auto-merge them. Regardless production releases are still manual

Iv opened a PR handling this.

I wonder if we could add another action that fires only on pr merge and only if the json file gets updated. So for example whenever we merge a pr like https://github.com/nodejs/release-cloudflare-worker/pull/36, it'd fire and call the script to update the redirect config. Would require some more human input though to merge the pr unless we'd like setup something to auto-merge them. Regardless production releases are still manual

I'm not sure what you mean, but what I'v had in mind is adding a step in the process of a release, manually deploying the worker
https://github.com/nodejs/node/blob/092fb9f541ce8cc07289b5a69eb93892445739f5/doc/contributing/releases.md?plain=1#L1127

I'm not sure what you mean

I was referring to this comment on doing the redirects (or a rewrite if possible) via Cloudflare rules. By doing so we'd avoid the need to handle it in the worker

Thought about it a bit more and curious as to if the Cloudflare rules approach would overlap with nodejs/build#3270, I think so since it's modifying config. Wonder if we should go with handling the redirects in the worker (#39) and then revisit the Cloudflare rules approach sometime in the future after Terraform is fully integrated to see if it'd be worth moving over?

I think https://github.com/nodejs/build/issues/3270 is going to take some while before it happens.
additionally, the amount of rules is > 700 that update on each release. this makes converting them into Cloudflare config much more complicated.
even if we convert the rules into wildcard rules they will be pretty complex and won't follow the exact logic of https://github.com/nodejs/nodejs-latest-linker.
so for me its a +1 on redirecting inside the worker


re-organized and expanded creating your own image topic

Signed-off-by: Victoria Bialas<EMAIL_ADDRESS>

@ManoMarks @SvenDowideit Take a look, and see what you think. I reorganized the topics on building your first image and learning about Dockerfiles.

yup LGTM :)

LGTM


index.html.erb:150-152: This is something to do later in...

The puzzle 234-6f82e2e2 from #234 has to be resolved:
https://github.com/cqfn/0capa/blob/adb3574b8d8658ac8a5f82ac95185654a8f21312/app/views/report/index.html.erb#L150-L152
The puzzle was created by @rultor on 27-Oct-22.
Estimate: 15 minutes,  role: DEV.
If you have any technical questions, don't ask me, submit new tickets instead. The task will be "done" when the problem is fixed and the text of the puzzle is removed from the source code. Here is more about PDD and about me.

The puzzle 234-6f82e2e2 has disappeared from the source code, that's why I closed this issue.


Fix errors from rustty bump and appease rustc warnings

src/drawing.rs

fix compile error because rustty switched its error type from rustty::Error to std::io::Error

src/main.rs

fix compile error because tty.get_event(..) now takes a std::time::Duration instead of an int for its timeout

src/radio/hackrf/mod.rs

change static result to static RESULT to appease rustc

Thanks!


Get list of Tabs for a template

I want to return all the tabs for a template by recipient, which I can achieve with the following REST request:
restapi/v2/accounts/{accountId}/envelopes/{templateId}/recipients?include_tabs=true&include_extended=true
Which API method do I use for this, I've searched through the code but couldn't find a method for this request type.

I found it, the reason I missed it was that the templateId is called a envelopeId in the API. Im guessing that maybe this accepts either a tempalteId or an envelopeId? If so, the parameter should be renamed accordingly.
However! The method doesn't work. Using postman I get the desired response, however when calling via the API I get a JSON parsing error:
Data: {System.Collections.ListDictionaryInternal}
    ErrorCode: 500
    ErrorContent: null
    HResult: -2146233088
    HelpLink: null
    InnerException: null
    Message: "Unexpected character encountered while parsing value: {. Path 'signers[0].tabs.dateTabs', line 56, position 11."
    Source: "DocuSign.eSign"
    StackTrace: "   at DocuSign.eSign.Client.ApiClient.Deserialize(IRestResponse response, Type type)\r\n   at DocuSign.eSign.Api.EnvelopesApi.ListRecipientsWithHttpInfo(String accountId, String envelopeId, ListRecipientsOptions options)\r\n   at DocuSign.eSign.Api.EnvelopesApi.ListRecipients(String accountId, String envelopeId, ListRecipientsOptions options)"
    TargetSite: {System.Object Deserialize(RestSharp.IRestResponse, System.Type)}

For reference, here's the JSON for the same call via postman:
"dateTabs": [
          {
            "validationPattern": "",
            "validationMessage": "",
            "shared": "false",
            "requireInitialOnSharedChange": "false",
            "requireAll": "false",
            "name": "someDate1ToolTip",
            "value": "",
            "width": 168,
            "required": "true",
            "locked": "false",
            "concealValueOnDocument": "false",
            "disableAutoSize": "false",
            "tabLabel": "someDate1",
            "bold": "true",
            "italic": "false",
            "underline": "false",
            "fontSize": "size24",
            "documentId": "1",
            "recipientId": "1",
            "pageNumber": "1",
            "xPosition": "43",
            "yPosition": "183",
            "tabId": "e68d65d6-e280-4e64-8a30-902c6b1eeb63",
            "templateLocked": "false",
            "templateRequired": "true",
            "tabOrder": "0"
          }
        ]

I also ran the entire JSON response through a JSON validator and it's all good. Must be some issue with your custom deserialiser in the c# sdk.

I'm not quite sure what you're wanting to do. Are you wanting to see the tabs of an actual document that was completed by a recipient? Or are you wanting to see the empty placeholder tabs of a template that would be used to populate instances of that template?
Based on your original equivalent endpoint, you're wanting the 2nd option. However, the SDK method you're referring to actually does the 1st option (that's why it takes EnvelopeID and not TemplateId, because it's actually pulling down tabs from an actual document that was signed by a recipient) - I use this all the time, and it works perfectly for pulling out data the recipient entered.
I honestly don't know if the SDK supports pulling down tabs for a template that aren't actually tied to a real document.

Yes, you are correct, I want to get a list of all the tabs within a template so I know what values are expected to be sent in the envelope to populate a document.
I found the endpoint I was using from this stack overflow response:
http://stackoverflow.com/questions/20249190/getting-all-tabs-from-a-template-using-rest-docusign-api
And it does appear to do exactly what I need, but I agree it doesn't seem right to be passing in a templateId where the api appears to expect an envelopeId. But it does work (via the rest api) just not when using the sdk.
Is there's a better way to achieve the same goal?

That's an interesting method I hadn't thought of.
We have one table that holds all merge-able template fields across all of our templates (about 50 fields total) along with the query that needs to be run to get the value. Whenever we generate a new doc from a template, we actually send ALL of those fields with their values. It seems kind of overkill, but DocuSign handles it well and only grabs the fields that apply to that current doc and throws away all the others.
It keeps it easy on us by not having to track (or look up every time) which template fields apply to which template.

@miketonguk This issue with the dateTabs should now be fixed with version 2.1.7. Please try it out.


Identify tasks for repo migration to kubernetes-sigs

[ ] Update go.mod reference and any other reference to vmware-tanzu in import statements
[ ] Token used by codecov also is based on current org (vmware-tanzu) -> get / update token
[ ] Make sure all  files mentioned in https://github.com/kubernetes/kubernetes-template-project are present
[ ] as pointed out in the sig mailing list, we will change all copyright license headers from VMware, Inc. to The Kubernetes Authors. Refer this message. Refer to any existing copyright header - https://github.com/kubernetes-sigs/cluster-api-provider-vsphere/blob/134604ab96e2e52901dda1b6414de21e67674616/controllers/svcdiscovery_controller_intg_test.go#L1-L15
[ ] identify all dependencies that don't have acceptable licenses.
[ ] need to verify that all contributors to the repo have signed the CNCF CLA

After all the above are done (with Draft PRs), raise repo migration request here

I will start off with the first one:

Update go.mod reference and any other reference to vmware-tanzu in import statements

[ ] Update badges on Readme.md and register on 3rd party sites as required.

[ ]  Update badges on Readme.md and register on 3rd party sites as required.

@NilanjanDaw Edit the main issue description itself. Will be easier to track all tasks.

Hi Team 😄
I was exploring the project and was curious if there were any updates for the repo migration to k8s-sigs?

Hey @pokearu, This is put on hold for the time being but we definitely aspire to migrate this over to k8s-sigs in the future.

Hey @pokearu, This is put on hold for the time being but we definitely aspire to migrate this over to k8s-sigs in the future.

Ah got it 👍 Thanks for the update.


SSL Errors some times

Hello,
2/3 connections are working fine, 3rd (sometimes alse 2nd) has the following error.
HTTP error: write EPROTO<PHONE_NUMBER>08104:error:14094438:SSL routines:ssl3_read_bytes:tlsv1 alert internal error:../deps/openssl/openssl/ssl/record/rec_layer_s3.c:1544:SSL alert number 80
But there is no way to change SSL settings inside the configuration-node.
Port is set to 443 and "Use SSL?" is activated.
Do you know how to solve this issue?
Thanks and have a nice evening!

What version do you have of the Ignition NodeRED nodes?

Closing since there is no activity


Liveness & Readiness endpoints

In kubernetes, you typically expose a set of probes to allow the system to know when it's live, when it's ready to serve traffic, and when it has started up. As far as I can tell Edgedb currently doesn't have any documented way to check for it's liveness and readiness. It would be nice if there was documented way to check for Edgedb's liveness and readiness.
https://kubernetes.io/docs/tasks/configure-pod-container/configure-liveness-readiness-startup-probes/

Documentation is, indeed, lacking for these.
The (HTTP) endpoints are: /server/status/ready and /server/status/alive


Documentation missing for newest features.

Missing especially reference audio event detection and timeseries ingestion.

This is done now.


Suggestion / request: use Dependabot to keep dependencies up-to-date

First of all, thanks for iNaturalist! Seriously cool project, and amazing to have it open source.
I've got a suggestion / request: would you be up for using Dependabot to automatically create dependency update PRs for this repo? I ran it against my fork and it generated these PRs. I'll port the devise one across to this repo now because it's security related. I think it should be fine to merge, despite the major version bump (which was just to drop support for Ruby 2.0 and Rails 4.0).
I built Dependabot, but I'm honestly only suggesting it because I hope it can save you some time. I'd love any feedback, and obviously having open source repos using Dependabot helps boost its profile, but if it's not helpful to you then it's not really worth anything.
You can install it from here or here if you decide to give it a try. It's been through GitHub's security testing (to be allowed in the GitHub Marketplace) and is used by a few thousand organisations, and the source code is here.
:octocat:

Thanks for the PR, and the suggestion. We're going to give Dependabot a try. If you're using iNaturalist, I hope you're enjoying it!

Awesome! I’ll take a look through all the initial PRs and make sure everything goes smoothly from my side, too 🙂

Closing since you're giving Dependabot a try - thanks! 🎉


add len method

Codecov Report

Merging #9 into v1.0.0 will decrease coverage by 0.28%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           v1.0.0       #9      +/-   ##
==========================================
- Coverage   80.88%   80.59%   -0.29%
==========================================
  Files           9        8       -1
  Lines          68       67       -1
==========================================
- Hits           55       54       -1
  Misses         13       13

Impacted Files
Coverage Δ

collection/array.go
100% <100%> (ø)
:arrow_up:

collection/each.go

Continue to review full report at Codecov.

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


Cannot import MatrixOperator

Hi,
Thank you for developing the Qiskit package. I have used Qiskit in the paper published about a year ago. Also, I have provided a google colab notebook example which uses JARVIS-Tools and AtomQC packages.
Recently, while re-running the notebook, I found many of the modules have been either moved/renamed or not available. For example I use MatrixOperator from qiskit.aqua.
I tried to search where it could have been moved but I couldn't find any documentation of migrations. This migration page seems outdated as well.
Could you please help me find the MatrixOperator or its latest updated equivalent?
Thanks in advance.

This issue seems to have cover three things that are related, but maybe helps to answer them independently.

How can I reproduce the result in a paper using a-year-old Qiskit?
Where is MatrixOperator that used to be in Aqua?
the migration-guide is outdated.

The first question is probably the one with the most direct answer: You can install the dependencies fixing the versions to the point in which the paper was published. That is means that your first cell should look something like this: !pip  install "jarvis-tools~=2021.7.5" "qiskit~=0.27". This will pin the version to July 2021 and probably would do the trick if the point is just to reproduce the results you used to have.
For the third part of the issue, do you mind opening a different issue with details on how the migration page needs to be updated?
In the meanwhile, let's jump to the second point, which it seems to be the core of your question. Here, we are going to assume you want to bring your code up to speed with the latest Qiskit, that used to work with Aqua and you want to migrate that. We can keep the topic of this issue on that point. Feel free to open new issues on the others if you would like to follow up that conversation.

Where is MatrixOperator now?
https://github.com/usnistgov/jarvis/blob/217eb3719f9ffce8f2bde9ae73f3961e42660217/jarvis/io/qiskit/inputs.py#L52
Your code usesMatrixOperator from the module  aqua.operators in Aqua. This module used to live here, in the legacy operator module.
According with the migration guide the module aqua.operators was replaced by qiskit.opflow in qiskit-terra. In your case, you are converting those MatrixOperators into WeightedPauliOperators:
https://github.com/usnistgov/jarvis/blob/217eb3719f9ffce8f2bde9ae73f3961e42660217/jarvis/io/qiskit/inputs.py#L57
so maybe you can use Pauli operators directly. Here is the documentation about it.

@knc6 if the issue on MatrixOperator is solved/explained, can we close it?

Yes, I was able solve the issue by pinning the following packages:
!pip  install jarvis-tools==2021.7.10 qiskit-terra==0.18.0 qiskit-aer==0.8.2 numpy==1.19.5 matplotlib==3.2.2 requests==2.23.0 scipy==1.4.1 joblib==1.0.1 tqdm==4.41.1 toolz==0.11.1 xmltodict==0.12.0 qiskit-terra==0.18.0 qiskit-aer==0.8.2 qiskit-ibmq-provider==0.15.0 qiskit-ignis==0.6.0 qiskit-aqua==0.9.4


Advance Sorting "Page is not" found after linking on it

Description
Bug
Right now, if you go to Advance Sorting page under DSA in Javascript it shows "Page is not found", which is bug.
Screenshot

Solution
We have to add the page link correctly into the code of this project.

Hey Please Assigned Me To This ,I also solved the issue

Hey I Solved This Issue and also raise a PR

I that case
Brother Raise a pr

Hey Also Raised a PR

Already this issue is completed by @Tanmay-Joshi73  & also reviewed by @Vishal-raj-1. So I am closing this issue.


Add:  RaveOne

It's my Private Channel, I would like you to add it. It mostly have old cartoons (cartoon network) and Animes , tv series and, movies. Its like 60%cartoons, 20%tv series 10%anime 10%movies.
Here's the link: http://<IP_ADDRESS>:8409/iptv/channel/1.m3u8
Name: RaveOne
Genre : entertainment
Give it a try

https://github.com/iptv-org/iptv/blob/master/CONTRIBUTING.md#how-to-add-a-new-stream-link-to-a-playlists


Project description on PyPI is blank

The adventurelib project description on the new PyPI is currently showing up simply as "UNKNOWN".
Additionally, it does not offer a link to this repository - where this is given, pypi.org can show Github stars etc.

FWIW, this is still the case.  "Last released: Feb 13, 2018"

Should be closed after #26. :)


Lighthouse looking for password somewhere on clean Dawn theme

I am working right off a download of a clean Dawn theme following https://shopify.dev/themes/getting-started/create which has a password page, however, the password is not active, and I am getting the following error:
Configuring shopify CLI
Will run Lighthouse CI on https://***
==============================
Creating development theme
{"theme":{"id":128112328866,"name":"Development (a77b97-d351661ce68d)","role":"development","shop":"***","editor_url":"https://***/admin/themes/128112328866/editor","preview_url":"https://***/?preview_theme_id=128112328866"}}
==============================
Configuring Lighthouse CI
==============================
Running Lighthouse CI
✅  .lighthouseci/ directory writable
✅  Configuration file found
✅  Chrome installation found
✅  GitHub token set
Healthcheck passed!

Getting a new page...
Getting password cookie...
TimeoutError: waiting for selector `form[action*=password] input[type="password"]` failed: timeout 30000ms exceeded
    at new WaitTask (/.node/lib/node_modules/puppeteer/lib/cjs/puppeteer/common/DOMWorld.js:509:34)
    at DOMWorld.waitForSelectorInPage (/.node/lib/node_modules/puppeteer/lib/cjs/puppeteer/common/DOMWorld.js:420:26)
    at Object.internalHandler.waitFor (/.node/lib/node_modules/puppeteer/lib/cjs/puppeteer/common/QueryHandler.js:31:77)
    at DOMWorld.waitForSelector (/.node/lib/node_modules/puppeteer/lib/cjs/puppeteer/common/DOMWorld.js:313:29)
    at Frame.waitForSelector (/.node/lib/node_modules/puppeteer/lib/cjs/puppeteer/common/FrameManager.js:841:51)
    at Page.waitForSelector (/.node/lib/node_modules/puppeteer/lib/cjs/puppeteer/common/Page.js:2238:33)
    at module.exports (/github/workspace/setPreviewCookies.js:9:16)
    at processTicksAndRejections (internal/process/task_queues.js:95:5)
    at async PuppeteerManager.invokePuppeteerScriptForUrl (/.node/lib/node_modules/@lhci/cli/src/collect/puppeteer-manager.js:108:5)

    at async Object.runCommand (/.node/lib/node_modules/@lhci/cli/src/collect/collect.js:240:7)

Originally posted by @Avex-Dev in https://github.com/Shopify/lighthouse-ci-action/issues/15#issuecomment-1004291159

Hey @Avex-Dev I've worked out how to fix this...
If your store isn't password protected (via Shopify Admin) - don't set the 'SHOP_PASSWORD' secret in your repo, this will then bypass the password page check in lighthouse check.
It would also help if Shopify updated their documentation to explain this!
The below explains why...
  if ('$shop_password' !== '') {     console.error('Getting password cookie...');     await page.goto('$host/password');     await page.waitForSelector('form[action*=password] input[type="password"]');     await page.\$eval('form[action*=password] input[type="password"]', input => input.value = '$shop_password');     await Promise.all([       page.waitForNavigation(),       page.\$eval('form[action*=password]', form => form.submit()),     ])   }

Hey guys, I'm also facing the same issue. In my case, I have a development store, and it's password-protected. I've added the correct password, but I'm still getting the following error log. Has this issue been resolved by any chance? or this is only for stores that are not having password protection ? 🤔
==============================
Running Lighthouse CI
✅  .lighthouseci/ directory writable
✅  Configuration file found
✅  Chrome installation found
✅  GitHub token set
Healthcheck passed!

Getting a new page...
Getting password cookie...
Getting preview cookie...
Running Lighthouse 3 time(s) on https://***/?preview_theme_id=16[25](https://github.com/Adnandeardigital/another-shopify-repo/actions/runs/6561257071/job/17821113831#step:4:26)73812014&_fd=0&pb=0
Run #1...done.
Run #2...done.
Getting a new page...
Run #3...done.
Getting password cookie...
TimeoutError: waiting for selector `form[action*=password] input[type="password"]` failed: timeout 30000ms exceeded
    at new WaitTask (/.node/lib/node_modules/puppeteer/lib/cjs/puppeteer/common/DOMWorld.js:509:34)
    at DOMWorld.waitForSelectorInPage (/.node/lib/node_modules/puppeteer/lib/cjs/puppeteer/common/DOMWorld.js:420:[26](https://github.com/Adnandeardigital/another-shopify-repo/actions/runs/6561257071/job/17821113831#step:4:27))
    at Object.internalHandler.waitFor (/.node/lib/node_modules/puppeteer/lib/cjs/puppeteer/common/QueryHandler.js:31:77)
    at DOMWorld.waitForSelector (/.node/lib/node_modules/puppeteer/lib/cjs/puppeteer/common/DOMWorld.js:313:[29](https://github.com/Adnandeardigital/another-shopify-repo/actions/runs/6561257071/job/17821113831#step:4:30))
    at Frame.waitForSelector (/.node/lib/node_modules/puppeteer/lib/cjs/puppeteer/common/FrameManager.js:841:51)
    at Page.waitForSelector (/.node/lib/node_modules/puppeteer/lib/cjs/puppeteer/common/Page.js:2238:[33](https://github.com/Adnandeardigital/another-shopify-repo/actions/runs/6561257071/job/17821113831#step:4:34))
    at module.exports (/github/workspace/setPreviewCookies.js:9:16)
    at processTicksAndRejections (internal/process/task_queues.js:95:5)
    at async PuppeteerManager.invokePuppeteerScriptForUrl (/.node/lib/node_modules/@lhci/cli/src/collect/puppeteer-manager.js:108:5)

    at async Object.runCommand (/.node/lib/node_modules/@lhci/cli/src/collect/collect.js:2[40](https://github.com/Adnandeardigital/another-shopify-repo/actions/runs/6561257071/job/17821113831#step:4:41):7)

This error is shown when there isn't a form with an input of type password on your theme. e.g.
<form action="...password">
  <input type="password">
</form>

Did any of you change it? Did the platform change it? The above code made the assumption that this would be there on the password page.

Can also confirm that my store has a password with the proper elements:

and

Not sure if anyone else has seen a change today, but just as suddenly and inexplicably as this stopped working last week it started working again this morning. We haven't made any changes that I can imagine would impact this at all — certainly no changes to the password form since it's not something we have access to. Either way, just really grateful to see that little green checkmark again 😆

No worries at all @charlespwd , and thanks for the additional info here! I appreciate you closing the loop for us!

Was this issue ever resolved? I'm using v.1.2.1, however still getting the same TimeOut error:
TimeoutError: Waiting for selector `form[action*=password] input[type="password"]` failed: Waiting failed: 30000ms exceeded


Stop deniel logs

What this PR does / why we need it:
Currently the coredns generates a lot of logs from type NOERROR(around 30 000 per hour) which do not bring us any valuable information. We have established that these logs are coming with class denial from the log plugin.  This is a diagram which shows the difference before and after the fix.

Note that for busy servers logging will incur a performance hit.
/cc @ialidzhikov  @vlvasilev  @vpnachev
Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:
Release note:
NONE

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

/lgtm


BZ-1277962 - REST Query API: taskId_min and taskId_max parameters are reversed

Not a blocker at the moment, only for master.

looking good, there is a test so +1, will merge in a minute

@mswiderski Could you please cherry-pick to 6.3.x?
https://bugzilla.redhat.com/show_bug.cgi?id=1277962 has gotten the blocker+ flag.
Thanks!

done


Fix help output for requirements container command

cli.generateCommandsHelp was removed with some of the Framework refactors. So currently sls requirements produces a programmer error.
This patch fixes the command, with a fallback to old Framework version where generateCommandsHelp will remain to be used

Thanks @pgrzesik for approval. Unfortunately CI fail doesn't allow to merge it at this stage. I'll rebase once we have that sorted

Unfortunately due to repository settings (which we're unable to change at this point), PR's cannot be merged if they're not update with master, so until that's the case, we need to go with PR's 1 by 1 :/

@pgrzesik I've just noticed that actually one test failed here, and again it looks as some quirk. Can you check it? Is this anything we can fix globally so won't return?


Update analytics lib

@pedrofneves thanks for the PR, I have two questions:

why do you want to update all of that?
The apps flyer integration was already added on the last release, check https://github.com/la-haus/flutter-segment/pull/19

PR useless


Edit typo

Codecov Report

Merging #378 (ae29e43) into master (b5e9163) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #378   +/-   ##
=========================================
  Coverage     67.17%   67.17%
  Complexity      500      500
=========================================
  Files            90       90
  Lines          1785     1785
  Branches        190      190
=========================================
  Hits           1199     1199
  Misses          524      524
  Partials         62       62

Continue to review full report at Codecov.

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


[stable/prometheus-rabbitmq-exporter] Bump to v0.29.0

What this PR does / why we need it:

Bumps rabbitmq-exporter version to v0.29.0
Adds new include_vhost and skip_vhost options that were added in v0.29.0
Reduces container privileges to run as non-root and with a readOnlyRootFilesystem
Use named ports to reduce duplicated templating

Which issue this PR fixes
Special notes for your reviewer:
Checklist

[x] DCO signed
[x] Chart Version bumped
[x] Variables are documented in the README.md

/ok-to-test
/retest
/assign @Juanchimienti

/unassign @Juanchimienti
/assign @unguiculus

/assign @viglesiasce

/lgtm

/retest


Improve Success and Error output from rpc / hivemind-cli

The formatting of output from the CLI is not in the best 'human' readable form.
For example (creating a trade):
cryptaxe@computer:~/hivemind$ ./src/hivemind-cli createtrade 1AwXR5AFfR8PFCPB8cUDQpLF5CnkrsTZQC 926827a6d80340d5c810239eeecc624c68c68ef495a2278017302b4a5c8322e5 buy 1  1 0 0
{
"txid" : "0d2e9186e7ca0e219c0e0ac75e307bd6631cb9e800e557430320ba866c14869b",
"tradeid" : "82d57de1336bb86021015f02e603348f1edcb50cf13395241620b31b1b1a78ca",
"B" : 0.10000000,
"buy_or_sell" : "buy",
"nShares" : true,
"price" : 5000000.50000002,
"total" : 5000000.50000002
}
In this example, the number of shares is printing as "true" instead of the number of shares which is 1, the price is not formatted (the data needs to be passed through the FormatMoney() function before being printed) and the same goes for the total value.
Most of the functions in the CLI have similar issues, and are located in src/rpcwallet.cpp
Same goes for error messages, for example:
order of parameters in error message is wrong, compare message to the list of params:
{"code":-1,"message":"createdecision branchid prompt address eventoverby [scaled min max]}
Correct order:

address             (base58 address)\n
branchid            (uint256 string)\n
prompt              (string)\n
eventoverby         (block number)\n
answer optionality  (false=mandatory to answer, true=optional to answer)\n
is_scaled           (boolean)\n
scaled min          (if scaled, numeric)\n
scaled max          (if scaled, numeric)"}

Several error messages have the same formatting issues, most likely they were written while the order of params was different.

Alright now price formatting just needs to be fixed,
18:08:00
￼
Error: price needs to be at least<PHONE_NUMBER>.50000095 (code -4)

Fixed with https://github.com/CryptAxe/hivemind/commit/3a1c0bd30475d7b1fbe43bd602c5ec5a11e3c86a


An option to make cards transparent when dragging them

How does it look like? Check here.

Easy enough to make the actively dragged cards translucent, I'll add that in when I tackle user settings. It would be hard to implement the left-behind "ghost" cards from your example, though.


Overriding some config values is impossible

Issue Summary
If I specify a custom URL in my config.production.js for the portal:url config value, the portal URL is not updated on the site.
To Reproduce

Add a custom script portal.min.js to your theme root folder
Upload the theme
Add the following JSON to config.production.json:

"portal": {
  "url": "https://<your ghost site URL>/portal.min.js"
}

Restart Ghost with ghost restart
Check the portal URL in the , and verify that it's the default one instead of the custom one.

Technical details:

Ghost Version: Ghost-CLI 1.17.0, Ghost 4.4.0
Node Version: 14.16.1
Browser/OS: Firefox, Chrome
Database: MySQL
Environment: production (confirmed with ghost doctor)

I'm not able to reproduce this at all - I've tried several times. The config is designed to be overridden and I've also heard several people talking about having done it successfully on the forum.


Add basic layout of Vision Layer

Added Gate and Buoy Task
Added Dynamic Reconfiguration files
Added the OpenCV basic library

It is normally advised that you break your PR to smaller PRs each having a fewer files changes since reviewing them would be easy. Few general styling issues:

the metadata has to be cleaned of all comments. Keep only the main comments required over there (see previous packages in the repo to understand).
README needs to be added for each package that is being sent to the PR so that it is easy to understand them in future
I suppose you are using 8 spaces tab in your IDE. It would be better to switch it to 4 instead since rest of the code uses that
The bracketing is an issue. Follow the style-guide here
The source code is lacking comments a lot! You would need to add them otherwise it ould be difficult in future to understand the code. Try to keep the openCV and the ros part of the code separate as we had discussed that day.

Make these changes first. After that I'll check for compilation and other issues (if any).

There are several .orig files that have been created. Also, do we even need level mode controller/ Why not remove it? I think Rithvik opened an issue on the same. Also, I am closing this PR. Please send in your pull requests with a small number of files at a time. Typically package wise. There are 24 commits already which is too much in my opinion.
@ksakash Comments?

Yes, I am going to close this one. No point creating such a big PR. We'll send in two smaller PRs for both the packages. We'll also fix the redundant code issue.


Addon compiled by prebuildify does not work, but the one compiled by node-gyp-build does

I'm seeing this behavior on this project. I can reproduce this all on a single Windows machine. Everything should be 64-bit (including my node.js installation), and the addon uses NAPI, so there should not be a platform mismatch.
Reproduction steps:

git clone https://github.com/REVrobotics/node-can-bridge
cd node-can-bridge
git checkout v2.0.5
npm install
npm run prepublishOnly (runs prebuildify)
npm install --verbose

When you run npm install --verbose, node-gyp-build sees the prebuilt addon that was built in the previous step, but it prints the following error and goes on to rebuild the addon.
node:internal/modules/cjs/loader:1280
  return process.dlopen(module, path.toNamespacedPath(filename));
                 ^

Error: \\?\C:\Users\Noah\node-can-bridge\prebuilds\win32-x64\node.napi.node is not a valid Win32 application.
\\?\C:\Users\Noah\node-can-bridge\prebuilds\win32-x64\node.napi.node
    at Object.Module._extensions..node (node:internal/modules/cjs/loader:1280:18)
    at Module.load (node:internal/modules/cjs/loader:1074:32)
    at Function.Module._load (node:internal/modules/cjs/loader:909:12)
    at Module.require (node:internal/modules/cjs/loader:1098:19)
    at require (node:internal/modules/cjs/helpers:108:18)
    at load (C:\Users\Noah\node-can-bridge\node_modules\node-gyp-build\node-gyp-build.js:22:10)
    at Object.<anonymous> (C:\Users\Noah\node-can-bridge\node_modules\node-gyp-build\build-test.js:19:19)
    at Module._compile (node:internal/modules/cjs/loader:1196:14)
    at Object.Module._extensions..js (node:internal/modules/cjs/loader:1250:10)
    at Module.load (node:internal/modules/cjs/loader:1074:32) {
  code: 'ERR_DLOPEN_FAILED'
}

I'd appreciate any guidance for debugging this. While it's good that at least the rebuild is working correctly, it would be nice to get to the bottom of this.

It actually has something to do with one of the DLL files that I had set up to be placed in prebuilds, but not the node-gyp-build Release folder. I get the same behavior on the fresh build if I copy the DLLs over.


Implement Email Verification

Generate a random secret link, then send it to the user on sign up. Once the user clicks the link, enable their account.

[x] Add Unique Link Generation to User Creation API
[x] Add Verification API
[x] Add Verification Page

Closed with 2975875ddac7445bf73a72deffdca7599d441c6c!


Proxy: Fix rewrite to api route

When a rewrite to an API-page (e.g. /sitemap/test.xml -> /api/sitemap/test) is added to the next.config.js the response results in an error.
This PR fixes this misbehavior.

Superseded by #190.


wiki: SRTP not on DTLS

Update the WebRTC Stack image on the wiki to reflect that SRTP doesn't run fully on top of DTLS.

Image in the Wiki has been adjusted.


Materialized view RelationType is view, so dbt tries to drop it each time and fails

Dbt 1.5.0
I am trying to use the materialized view materialization.
The materialized view is created as expected. But on the second run dbt tries to drop it, and fails with error "SQL compilation error: Object found is of type 'MATERIALIZED_VIEW', not specified type 'VIEW'."
I guess this is because dbt uses SHOW TERSE OBJECTS IN SCHEMA to determine objects types, and according to doc https://docs.snowflake.com/en/sql-reference/sql/show-objects :
For materialized views, this returns “VIEW”, not “MATERIALIZED VIEW”.

Fixed on revision <IP_ADDRESS>


Docs(Discoverer) - added offerings file and instructions

This PR add the list of prefetched offerings used in case of troubles with remote resources. It also adds instructions on how to use such list.

Current coverage is 41.16%

Merging #287 into master will decrease coverage by -6.88% as of 35d0864

@@            master    #287   diff @@
======================================
  Files          215     361    +146
  Stmts         9009   13170   +4161
  Branches      1316    1656    +340
  Methods          0       0
======================================
+ Hit           4328    5422   +1094
- Partial        352     430     +78
- Missed        4329    7318   +2989

Review entire Coverage Diff as of 35d0864

Powered by Codecov. Updated on successful CI builds.


Group degree fix

Signed-off-by: Eugenio Angriman<EMAIL_ADDRESS>

The Windows build fails because of the data type of a parallel for loop. Please use the new omp_index type as in the rest of the code base.

The remaining problems are not related to the group degree algorithm. I would prefer to rebase (i.e. squash) this patch series into a single commit. (With a single commit message to explain all changes.) Can you do that or do you want me to merge it as-is?

I would also prefer to squash it, let me do that before the merge.


Add Nix Dev environment for easier reproducible dev envs

Is there an existing issue or pull request for this?

[X] I have searched the existing issues and pull requests

Feature description
I would like to just type nix develop to begin contributing and get parity with CI and other devs. By defining the dependencies of this project in a flake you could ensure everyone on any OS is building as the project expects.
Will submit a draft PR for testing.
Desired solution
Use Nix
Alternatives considered
None
Additional context
No response

Unfortunately I don't have experience with Nix but I'm totally fine with it if anyone wants to add it.


Direct torrent (without debrid) support?

Does this addon support torrenting directly without a debrid service? If not, is this planned? Thanks

Unplanned, this addon is debrid only.


HIVE-27601 : Backport of HIVE-21924 to branch-3

JIRA link - https://issues.apache.org/jira/browse/HIVE-27601

Taken care here - https://github.com/apache/hive/pull/4524


Update Peruvian entries

This PR is for update channels in Peru

Cancel for conflicting entries in this database.