Updating to latest quic-go pulls in a testing framework

In the process of releasing a Caddy update, I was updating our vendored quic-go dependency but I realized that it pulled in a LOT of new dependencies, including ginkgo and gomega, which I believe shouldn't be pulled in except for tests? (I was not using -t, so it shouldn't pull in test dependencies.) I admit I am in the middle of some other things so I haven't investigated where they are used in the code.
Before I commit 340+ new files to our repo, I decided to defer updating quic-go for now until we figure out if this is in error (instead of delaying the Caddy release entirely). If it is a mistake, great -- we can get it fixed. If not, then can we find a way not require test frameworks for production builds?
Any way to lighten the dependency tree at this point would be greatly appreciated.
PS. I would be remiss to omit another thank-you for this project. You guys are doing great work.

Hi @mholt, it would be great if you can hold off the release for another half an hour or so. I'd like to see the recent fixes (especially #1340) included in the next Caddy release.
ginkgo and gomega should definitely not be needed anywhere else than in the test code. I'm investigating right now.

Update: I think the culprit is https://github.com/lucas-clemente/quic-go/blob/master/integrationtests/tools/testserver/server.go. The whole integrationtests directory only contains test code, and doesn't need to be vendored. But afaik, it's not possible to exclude directories from being vendored, is it?

I'm using gvt to manage the vendor folder -- it should only pull in dependencies that are needed to compile, not test (unless -t is used). I wonder if it has something to do with the dot-imports?
https://github.com/lucas-clemente/quic-go/blob/240896a4dd6fc56c42698bb0855166858c47fa27/integrationtests/tools/testserver/server.go#L15-L16
I'm not sure though, it's just a guess.
And no, I'm not sure of a way to prevent a subfolder from being vendored.
If the files all end with _test.go, that should do it; because there's nothing about the folder or file structure as-is that would cause them to be treated as test-only. (There might also be a provision for a testdata folder? Worth looking into.)
I'll hold the release for a few more hours. It's after midnight here, so I'm gonna get some sleep and return to this in the morning. :+1:

Thanks, @mholt!
I tried two things:

replace the dot imports by normal imports
rename the integrationtests directory to integration_test (hoping that Go would recognize that)

Both attempts don't work. gvt still vendors ginkgo.
@lucas-clemente: Any ideas?

@marten-seemann I'm gonna release Caddy 0.11 today but we can get quic-go updated on the next release; I have a bunch of PRs in the backlog I need to go through anyway. :+1:

Having thought about this, I don’t really see what a good solution would be.
The only thing that would actually get rid of the ginkgo files would be moving the integrationtests directory in another repo, but that would create a strong coupling between those two repos, so I’d like to avoid that.

I will have a similar dilemma with Caddy as I plan to build out its testing suite, but I think the tests will live in a separate repo, since the test suite really is a different application entirely. I don't think it's bad to have to repos, personally...

Hello @mholt, I recently discussed this issue with @lucas-clemente. We both agree that splitting the integration tests into a separate repository will complicate things quite a bit, since we're often adding new integration tests when fixing a bug.
I see the fact that govendor doesn't have an option to ignore certain directories is a shortcoming of the go tooling itself. On the other hand, while aesthetically unsatisfying, this shouldn't have any effect on the Caddy binary, since ginkgo and gomega are only used for the test code, and therefore won't be compiled when building the regular binary.
I haven't had the time to take a closer look at the vgo changes coming up with Go 0.11, but I hope that will finally solve the problem.

Hey Marten! Nice to hear an update about this. :) Thanks for looking into it.
My main concern isn't so much binary size (although that is mildly a concern, but not something this library really contributes to in a significant way) -- it's the size of the code repository.
I will take another look and see if we can get away without vendoring those particular packages and manually wrangle the vendor folder a bit, to work around the issue in the meantime.

Closing this issue since it seems there's nothing more I can do about it.


Update KeeperStateProvider.java

This should resolve https://github.com/ClickHouse/clickhouse-kafka-connect/issues/375

Nice!

@Paultagoras I don't see much difference except just some very minor lag of updated values between table in cluster. Not sure if thats going to affect any events

@Paultagoras i see one of our cloud tests is falling please take a look before


training dataset

could you share the training dataset?

In this project, we use Matterport3D dataset as our training / testing set.

You need to sign the Term os Use at Matterport to get matterport3D dataset, containing undistorted_color_images and undistorted_depth_images.
By sending the email to yindaZ attached with Term of Use mentioned above, you can get render depth, estimate_boundary and estimate_normal.
For sobel_edge, it's easy for you to use Sobel in opencv package to generate sobel edge from render depth. (Sorry I just can't find out where my code is now...)
Last, resize all of them to size 240x320

ps. It might take many time on data preprocessing...
Hopes that my reply can help you ~


New icon animation: review

Name of icon to be animated as shown in Carbon Icon Library:
review
Category of icon:
operations
Team/product requesting:
icon motion group
Contacts:
@kristastarr @johnbister
When is this needed?

optional - is there a specific deadline

Additional context

optional - is there anything else we should know?

@jagathgj Hey! For this one, I would scale up the star shape at 10% then have it go back to normal at 20%. Let me know if you need me to separate layers in illustrator or anything else.  Thanks again! 😄


feat: 国密前端支持(closed #1660)

Codecov Report

:exclamation: No coverage uploaded for pull request base (v2.3.3-dev@d571361). Click here to learn what that means.
The diff coverage is n/a.

@@              Coverage Diff              @@
##             v2.3.3-dev    #1666   +/-   ##
=============================================
  Coverage              ?   73.38%
=============================================
  Files                 ?      389
  Lines                 ?    26301
  Branches              ?        0
=============================================
  Hits                  ?    19300
  Misses                ?     7001
  Partials              ?        0


"Undefined global function" in PyTorch

I'm experimenting with building PyTorch against ROCm, which in turn built with ebuilds from here on Gentoo. I realize, that this is completely unsupported configuration, but could you please suggest me, where to find the inconsistency in the build with the following problem. The problem is that every call to pretty much any function I tried in torch.nn.functional fails with RuntimeError: Undefined __global__ function.. Minimal reproducing code is just:
import torch
import torch.nn.functional as F

x = torch.tensor([1, 2], device="cuda")
v = F.relu(x)

I've found some similar error output in rocFFT issue, but can't really tell if it's connected to rocFFT or not.
Thanks in advance for any help or information!

@aclex, do you still see the issue with the latest Rocm?

Can't confirm it for 5.0, haven't built PyTorch against it yet, unfortunately. Feel free to close it for now, I'll reopen if it's still there.

Closing.  Please re-open if it occurs with latest ROCm 6.0.2 (HIP 6.0.32831)


DO NOT MERGE: testing presubmit prow jobs

Issue #, if available:
Description of changes:
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/hold

/woof

/retest

Looks like things are working again.


cleanup test properly for passing case

https://github.com/ethereumjs/merkle-patricia-tree/pull/28 looks like a promising fix for a part of the issue - 'get' of undefined
I've updated the test cases to clean up the error listener and pass properly when the issue is fixed

Thanks, @0xNPE - I'll have a look at this with ethereumjs/merkle-patricia-tree#28 in the mix and see how it goes!

Blocked until the #453 is finished

@benjamincburns is this safe to merge now that https://github.com/trufflesuite/ganache-core/pull/209 is merged?

Closing as these tests require significant refactoring at this point due to underlying vm changes.


feat(JavaScript): Added a few new code examples

feat(JavaScript): Added the decimalToBinary, binaryToDecimal and solveX to the code examples

Added JavaScript/decimalToBinary.js
Added JavaScript/binaryToDecimal.js
Added JavaScript/solveX.js

feat(JavaScript): Removed decimalToBinary and binaryToDecimal

Removed JavaScript/decimalToBinary.js
Removed JavaScript/binaryToDecimal.js

tbh the code didnt worked for me (no err but yea)

don't know about you but I think it's totally working. Don't get me wrong, there are no errors, and it does solve for the value x.


Add support to build a snap.

This change introduces snapcraft.yaml to produce a snap named
"charmhub-lp-tool". It uses the password-manager-service interface to
allow launchpadlib to store the store the auth token in the keyring (see
LP: #1923727).

Something we need to decide is if we want to build the snap using launchpad + a recipe, or using build.snapcraft.io/ , or via github actions ( https://github.com/marketplace/actions/snapcraft-action , https://github.com/snapcore/action-build )

Hey @freyes - I tried building the snap using snap pack and go the error:
error: cannot pack ".": open meta/snap.yaml: no such file or directory

Has something changed in the world of snapcraft or am I simple doing-things-wrong :)
Love the idea of it becoming a snap btw; we can then add a config file for 'default' config of charms and then simplify using the tool!

The command to build the snap is "snapcraft" (e.g. snapcraft --provider lxd)

Ah, I thought I was doing something wrong, and you've confirmed it.  I wonder why there is a snap pack command.  Oh, well, I'll go read the man pages!


Friday 26-01-2024 roll call

Roll Call!


Implement SwapChain for CUDA and ISPC

Probably use OpenGL for display? Maybe we need to re-design the related interfaces before that...

Implemented with Vulkan.


Uncapitalize "Could not compile" error message

"could not compile ..." matches other Cargo and rustc errors and
warnings better.

Thanks for the pull request, and welcome! The Rust team is excited to review your changes, and you should hear from @nrc (or someone else) soon.
If any changes to this PR are deemed necessary, please add them as extra commits. This ensures that the reviewer can see what has changed since they last reviewed the code. Due to the way GitHub handles out-of-date commits, this should also make it reasonably obvious what issues have or haven't been addressed. Large or tricky changes may require several passes of review and changes.
Please see the contribution instructions for more information.

@bors: r+
Thanks!

:pushpin: Commit 26229cd8ad6a80b24133d87fc8719c5cae1122c0 has been approved by alexcrichton

:hourglass: Testing commit 26229cd8ad6a80b24133d87fc8719c5cae1122c0 with merge 3b735c6305ae7fa0d9921b38c521a5ba64c87223...

:sunny: Test successful - checks-azure
Approved by: alexcrichton
Pushing 3b735c6305ae7fa0d9921b38c521a5ba64c87223 to master...


JSON.parse of string giving Error, cant reply

Logging the JSON works great, but when trying to pass it through bot.replywithtyping(message, myJSON); im getting a "UnhandledPromiseRejectionWarning: Unhandled promise rejection (rejection id: 1): TypeError: Cannot read property 'length' of undefined"
I've verified the JSON with Slacks message builder and it runs great, but my bot can't seem to use it.
Any ideas ?
Tried: Stringify, parse, raw string but i either get the raw string in slack from the bot or the error mentioned above.

bot.reply did the trick


Logbook HttpRequest/HttpResponse TCK

Detailed Description

Define and document the contract of HttpRequest/HttpResponse's withBody, withoutBody and getBody opoerations and provide a "Technology Compatibility Kit" for implementations to rely on.
Context

Possible Implementation

Possible states and transitions

Ideas
Buffering is an internal state which we don't expose right now. For a TCK it would be helpful to verify the desired effect (not buffering unless needed) was actually applied. Maybe that can be achieved by delegating back to the implementation-specific part of the test suite.

[ ] State pattern within request/response implementations?
[ ] TCK based on one or more Strategy implementations which verify that all transitions work as expected (including negative tests)
[ ] Is it even possible to have withBody and withoutBody return new instances with modified state?

Your Environment

Version used: 2.0.0-SNAPSHOT

The transition from buffering to buffered is one of two things:

buffering of already available body is delayed until getBody is actually used (which is when RequestWritingStage.write() is called)
decorated output steam has been exposed

Maybe Buffering should be Exposed? Or the state transition is called buffer, rather than expose.


Unused Code removal

Removed unused map variable from the code. This variable was declared but never used. I removed the code and verified again from the find and found no impact.
What's in the PR

...

How to test manually

...

Automatic testing

[ ] PR includes unit tests

Documentation

[ ] PR updates documentation

Same comment as in #169.


Inform end user if account creation takes some time

This issue is imported from pivotal
	Not all provisioning can be real time. \

Add a `user-wait-time` option to the provisoinings (default 0 minutes).\
Inform the user on welcome it wil take N minutes to activate his account (if >0)\
register the time the provisioning message was send\
disable the button to proceed to the application until the wait time expires

Voor nu: Per provisioning instellen of deze real time is


Fix: Update readme to match current folder structure

Issue
Flowing instruction in readme faced error.

Acceptance Criteria
Current readme is showing
Run kscript new_project.kts to create a new project with the following arguments:

while the path to new_project.kts is /scripts/new_project.kts.

Include instruction to open the scripts folder
or
Change readme to match the folder.

@ryan-conway Is this issue fixed by this PR? 🤔

@lydiasama Not exactly. That issue fixed a bug that occurred when calling kscript scripts/new_project.kts. I think the issue here is that the readme says to run kscript new_project.kts but does not specify where exactly to run it 🤔

@ryan-conway I see. I found this issue too when I generated my new app but I think everyone would know that we need to run the script on the script file path. So I didn't create an issue for it. 😅  So I voted this issue +1

@ryan-conway
From my understanding, there's no need to specify where exactly to run it 💭 Because when you made this fix, it can be run from both root or /script folder

My bad, I misunderstood. It's not possible to run from root 🙏
To keep it simple, I agree we should just simply update README.md to change to scripts folder first

@Tuubz Can I take this one? It seems everyone has their task. 😬

@lydiasama Yes please, go for it 😊


Show Next Event Functionality Not Working in macOS Sonoma

The Show Next Event functionality seems to be broken in macOS Sonoma. Even though I have an event in the next couple of hours, the Next Event doesn't show up in the menu bar even when it is set to 24-hour visibility.

That happened to me before, even before upgrading to macOS Sonoma.
Fortunately (or maybe not), it didn't happen ever again so I don't know how to reproduce it 🔮
Just quit and reopen the app. If that happens again in the future, try to remember what you did before and please tell me.

In this case, I've tried quitting and reopening the app multiple times, but it doesn't seem to pick up the next event I have which is in about 10 hours or so, even when set to display the next event in 24 hours. Maybe there is another problem that I can't seem to put my finger on?

Try un/checking the clock icon in the calendar list and see if that changes something.

Yeah, that seemed to fix it. I guess disabling reminders from showing up adds back the events. I'm assuming this is a bug of some sort?

Probably. If you enable the reminders again, does the problem come back?

Yeah enabling reminders again removes the next event from the menu bar. So unchecking reminders shows the event and checking the reminders makes the event disappear.

They should both show in the menu bar. Maybe there's not enough space?

I usually place the one with the higher priority to me on the right, so when I'm using my macbook and there's not enough space (because of the notch), the left one disappears.

I guess the weird part of this is that I don't have any active reminders, so in theory there shouldn't be anything to display or render.

Well... that's... so weird 😅

Are you using the checkbox, to completely disable the reminder, or the clock/moon icon?
What's the effect when you toggle each one?

So using the checkbox to completely disable the reminders and the clicking on the clock turning it into a moon, both result in the next event showing up. If the clock icon is there or reminders is checked, the event doesn't show up anymore.

Yeah, I'll give this a try in the morning, it's about 2 AM here, will completely uninstall with AppCleaner (to purge the config files) and then will try again.
On Sep 28 2023, at 2:10 AM, Carlos Enumo  @.***>  wrote:

Geez... I have no idea. Try to toggle all calendars off and back on. Maybe your saved settings are corrupted or something.
—
Reply to this email directly,     view it on GitHub (https://github.com/pakerwreah/Calendr/issues/152#issuecomment-1738763929), or     unsubscribe (https://github.com/notifications/unsubscribe-auth/AGAXKZOCQKWX6PLO5LNDTJLX4U5GVANCNFSM6AAAAAA5KRL5AI).
You are receiving this because you authored the thread.

Yup this seemed to fix it. I'm confused which specific config file might have been corrupted, but redownloading the app, and enabling the "Shorten if notch present" feature seemed to have fixed it for now.

Well, I'm glad that it got solved in the end.
It's very hard to know what was going on without being able to look into the data.
It can be that the upgrade to macOS Sonoma messed up the calendar IDs in your user preferences. 🤷🏻♂
If that happens again, feel free to open a new issue and we can investigate it further.


xps13-9380: enable fwupd

Description of changes
Just enabling fwupd service like on any other xps13.
Also changed the wiki mentioning how to update firmware.
Things done

[ ] Tested the changes in your own NixOS Configuration
[ ] Tested the changes end-to-end by using your fork of nixos-hardware and
importing it via <nixos-hardware> or Flake input

This is my first contribution to the repo.
Tried to follow the testing part of CONTRIBUTING.md but did not manage to successfully run a test locally.
The test command is broken. I also tried: nix run ./tests#run . but my xps13 could not handle the stress.

Need to fix documentation here as I had to refactor the test.

@mergify queue


GitHub Issue #206: Bug in DataBuilder reversing frames

rollbar expects most recent call to be last, not first - this was lost somewhere in development of 1.0.0. This PR brings back the expected order of frames as requested in #206

Ready for review

LGTM

👍  We will need to note this in the changelog; for anyone upgrading from 1.x, this will cause their events to be fingerprinted differently by Rollbar.

Good point, it will restore the fingerprints from <= 0.18 but be different than >= 1.0.0 until this is released


Managing External Domain spoofing

Under the section for legitimate External domain spoofing you mention this scenario:
The sender is on a mailing list (also known as a discussion list), and the mailing list relays email from the original sender to all the participants on the mailing list.
Are there any examples on how to remedy issues with this? If there are many members of the discussion list, that's a lot of individual "allow to spoof" toggles.

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

ID: f1c0cbdb-062e-08e1-7c7c-cedab550f01b
Version Independent ID: 2791a840-3284-bc0d-7b9e-026b852797e6
Content: Configure spoof intelligence - Office 365
Content Source: microsoft-365/security/office-365-security/learn-about-spoof-intelligence.md
Service: o365-seccomp
GitHub Login: @chrisda
Microsoft Alias: chrisda

@MarcLaf, the other option is to use PowerShell. The important information is in a CSV file.

Right I see that option but again, that's a lot of manual combing through and adding to the allow list. Was just looking for an alternate solution or suggestions on how to deal with it.

@chrisda Thank you very much for sharing this explanation. @MarcLaf If you see a documentation update is required, please feel free to open an issue for the same. We proceed here to close it. Thanks for taking out some time to open the issue. Appreciate and encourage you to do the same in future also.


Avoid page fault due to partially initialised memory sections

If module memory sections exist (e.g. when using initrd), and there's a usable memory larger than 2Mi between the end of the kernel memory and the start of the module, harvey would try to use it as user memory, despite it being found to be earlier than sys->pmstart (as set in setphysmembounds).  This would cause a page fault when physinit was later called for that memory section.
This commit will avoid using any PamMEMORY sections found earlier than any PamMODULE sections.  This has the downside that we lose useful memory, but probably not very much.  In my case the memory section was just under 6Mi.
Maybe this can be recovered with a more sophisticated change (probably involving physalloc.c)?
This change should have no effect if modules (initrd) are not used.
Signed-off-by: Graham MacDonald<EMAIL_ADDRESS>

Fixes #1096

I've tested the change on an apu2 and it boots ok, so let's give it a shot.


Read hosted cluster creds from secret in target ns

Issue: https://github.com/stolostron/backlog/issues/26218
Signed-off-by: Jakob Gray<EMAIL_ADDRESS>

/override "ci/prow/sonar-pre-submit" "SonarCloud Code Analysis"
Connection to remote cluster can't be reasonably unit tested

added "controllers/hosted.go" to sonar exclusion list. seeing if that fixes the sonar scan issues.

Excluding controllers/hosted.go helped, but didn't completely fix the issue. Need to bring this up with the delivering faster working group and CICD.

I'm not too worried about the coverage count, but looking for a code review.

/lgtm


Add extra tools to the toolbar

When adding the annotorious-selector-pack and/or annotorious-tilted-box plugins, the extra tools don't get added to the toolbar.

Yes, that's right. The toolbar is definitely intended to provide icons/buttons for plugin tools in the future. But only the two standard tools (rect/poly) are implemented so far.

I just released a new version of the toolbar which includes icons for selector pack and tilted box tools.


Let Client.attach wait until stream initialization is finished

What this PR does / why we need it:
This PR is the same as the issue https://github.com/yorkie-team/yorkie-js-sdk/pull/440 in JS.

Client.attach() wait until the attached document has been initialized or timed out.
The default timeout is 5 sec.
Due to the sync duration being 1 sec, Change the delay time for document sync in Test Codes from 1 sec to 1.5 sec.

Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?:

Additional documentation:

Checklist:

[x] Added relevant tests or not required
[x] Didn't break anything

Thank you for your contribution. I have a question.
Would it be possible to remove non-deterministic codes(Task.sleep) from tests in iOS SDK? We removed non-deterministic codes like time.delay(Promise version of setTimeout) from JS SDK after waiting for the initialization event.
https://github.com/yorkie-team/yorkie-js-sdk/pull/440/files#diff-e189bdd138ade553af55c7c6000357b18d2480646720e9659a64f27b771ef963L11

The extra delays for initialization have been removed. The rests are for auto-sync. I'll remove them by changing the manual sync in the following work.


Error in the last example of userguide/dockervolumes

In the last example of userguide/dockervolumes we want to restore backed up data into a dbdata2 container:
sudo docker run -v /dbdata --name dbdata2 ubuntu /bin/bash
sudo docker run --volumes-from dbdata2 -v $(pwd):/backup busybox tar xvf /backup/backup.tar

This snippet has 2 issues:

If we dig into the logic of the snippet, we'll see that restored data should get into /dbdata directory of dbdata2 container. Instead the data is left in / directory of a temporary container. The second line should like this:

sudo docker run --volumes-from dbdata2 -v $(pwd):/backup busybox cd /dbdata && tar xvf /backup/backup.tar

(Added cd /dbdata && )

As it was previously stated in the userguide, we'd like to use the same base containers for most operations to reduce network and disk usage. So that in the second line busybox should be replaced by ubuntu. Though considering the size of busybox this may be not a very serious issue :smile:.

Closing, because https://github.com/docker/docker/issues/12054 was merged.
Thanks @eluck!


Fix support for EIP in EC2-Classic

For EC2-Classic, the allocation_id is nil, which causes an error, so the ID and the module name should be derived from IP address rather than the allocation_id.

Coverage increased (+0.0004%) to 99.786% when pulling ef877fc2cb258d969a6df9dae0a5e501a7498233 on yn:master into caf9c915bd6ab316b88f1ce406e7010b8833b0b2 on dtan4:master.

LGTM 👍  Thanks ❗


cli: anchor new does not use new unique program id and does not add new program id to Anchor.toml

I think what we want to do here is first complete https://github.com/project-serum/anchor/issues/695. And then use that new macro instead of hardcoding a keypair.

Fixed in #2509 and #2516


Fix mkdir concurrency creating

Issue:
Two PHP processes try to create directory concurrently.
Both process successfully pass  if (null !== $dir && !is_dir($dir)) { condition, but only one process successfully creates directory and second process will throw Exception, but directory is created.
clearstatcache() is not required due to PHP documentation http://php.net/manual/en/function.clearstatcache.php

You should also note that PHP doesn't cache information about non-existent files. So, if you call file_exists() on a file that doesn't exist, it will return FALSE until you create the file.

Same for is_dir()
Please also check https://github.com/kalessil/phpinspectionsea/blob/master/docs/probable-bugs.md#mkdir-race-condition

Thanks, but the same fix was applied already in c0eeca76fdb1b5410e7af36bbfe5af265eeb5bf6 so closing this.


pull request for e2e test at 01/12/2017 04:26:26

:white_check_mark: Validation status: passed
For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Ingress resource not reachable when installed with dapr

In what area(s)?

/area runtime

/area operator

/area placement

What version of Dapr?

1.10.5

Expected Behavior

I have created a new aks cluster with kubernetes version 1.26.3. I have installed a dapr version 1.10.5 and then nginx ingress with dapr annotations. I'm not able to access the ingress resource. I tried with load balancer ip and also with host.
However, the same steps works well with a different cluster with kubernetes version 1.23.15
Steps followed

Install dapr

helm upgrade --install dapr dapr/dapr --version=1.10.5 --namespace dapr-system --create-namespace --wait

Install nginx ingress controller

helm upgrade --install nginx-ingress ingress-nginx/ingress-nginx -f dapr-annotations.yaml --create-namespace --namespace nginx

# dapr-annotations.yaml
controller:
  podAnnotations:
    dapr.io/enabled: "true"
    dapr.io/app-id: "nginx-ingress"
    dapr.io/sidecar-listen-addresses: "<IP_ADDRESS>"
    dapr.io/app-port: "80"

After this I tried to access http://{{LOAD_BALANCER_IP}}, but I cannot reach default nginx NOT FOUND page.
The url http://{{LOAD_BALANCER_IP}} should load nginx NOT FOUND page
Actual Behavior

The url http://{{LOAD_BALANCER_IP}} is not accessible.
Steps to Reproduce the Problem

Install dapr in the k8s cluster with version 1.26.3
Install nginx controller

helm upgrade --install nginx-ingress ingress-nginx/ingress-nginx -f dapr-annotations.yaml --create-namespace --namespace nginx

Access the ingress with load balancer ip

Can you provide logs from the daprd container?


运行RNN-T会报错

拉下最新代码执行/home/lsj/zdb/wenet/examples/aishell/rnnt/run.sh第四步的时候报错如下：
Traceback (most recent call last):
File "wenet/bin/train.py", line 448, in
main()
File "wenet/bin/train.py", line 283, in main
script_model = torch.jit.script(model)
File "/home/lsj/.conda/envs/wenet/lib/python3.8/site-packages/torch/jit/_script.py", line 1286, in script
return torch.jit._recursive.create_script_module(
File "/home/lsj/.conda/envs/wenet/lib/python3.8/site-packages/torch/jit/_recursive.py", line 476, in create_script_module
return create_script_module_impl(nn_module, concrete_type, stubs_fn)
File "/home/lsj/.conda/envs/wenet/lib/python3.8/site-packages/torch/jit/_recursive.py", line 542, in create_script_module_impl
create_methods_and_properties_from_stubs(concrete_type, method_stubs, property_stubs)
File "/home/lsj/.conda/envs/wenet/lib/python3.8/site-packages/torch/jit/_recursive.py", line 393, in create_methods_and_properties_from_stubs
concrete_type._create_methods_and_properties(property_defs, property_rcbs, method_defs, method_rcbs, method_defaults)
File "/home/lsj/.conda/envs/wenet/lib/python3.8/site-packages/torch/jit/_recursive.py", line 863, in try_compile_fn
return torch.jit.script(fn, _rcb=rcb)
File "/home/lsj/.conda/envs/wenet/lib/python3.8/site-packages/torch/jit/_script.py", line 1343, in script
fn = torch._C._jit_script_compile(
RuntimeError:
'torch.wenet.transducer.predictor.RNNPredictor (of Python compilation unit at: 0x51e2a80)' object has no attribute or method 'call'. Did you forget to initialize an attribute in init()?:
File "/home/lsj/zdb/wenet/wenet/transducer/transducer.py", line 483
ys_in_pad = add_blank(text, model.blank, model.ignore_id)
# predictor
predictor_out = model.predictor(ys_in_pad)
~~~~~~~~~~~~~~~ <--- HERE
if not use_k2:
# joint
'compute_loss' is being compiled since it was called from 'Transducer.forward'
File "/home/lsj/zdb/wenet/wenet/transducer/transducer.py", line 116
    # compute_loss
    loss_rnnt = compute_loss(
    ~~~~~~~~~~~~~~~~~~~~~~~~~
        self,
        ~~~~~
        encoder_out,
        ~~~~~~~~~~~~
        encoder_out_lens,
        ~~~~~~~~~~~~~~~~~
        encoder_mask,
        ~~~~~~~~~~~~~
        text,
        ~~~~~
        text_lengths,
        ~~~~~~~~~~~~~
        steps=steps,
        ~~~~~~~~~~~~
        use_k2=self.enable_k2,
        ~~~~~~~~~~~~~~~~~~~~~ <--- HERE
    )

fix ， 可以试下 这个 https://github.com/wenet-e2e/wenet/pull/1990

fix ， 可以试下 这个 #1990

谢谢大佬，已经解决了


Save As - Overwrite Protection (Prompt)

As it relates to Issues #152 and #153 (Save As overlay), add a feature that protects against accidental overwriting.
On the Save As overlay, when the user clicks the Save As button, the system should always prompt with a confirmation overlay if the filename matches an already existing file in the target folder.  The prompt should say, "File 'filename' already exists. Overwrite anyway?" and should have an Overwrite and a Cancel button.

I think this is a bit too much to add to this sprint.  Let's slate it for the next sprint

Okay.

@monteslu to address this issue you are going to want to look at the following files:

src/views/save-overlay.js
src/plugins/overlays.js
client.js

I suspect that the actual logic will go in src/views/save-overlay.js

Verified done.


Updated cmake install action.

As of 9/26/2023, the old version can no longer install CMake 3.16.x and gives the following error:
Run<EMAIL_ADDRESS>with:
cmake-version: 3.16.x
github-api-token: ***
Error: Unable to find version matching 3.16.x

@axlan FYI


Mht sdk 1107

Make sure that:

[ ] You have read the contribution guidelines.
[ ] You submit test cases (unit or integration tests) that back your changes.
[ ] Your local test passed ./mvnw clean install -Dmaven.javadoc.skip=true.

Codecov Report

Merging #4166 (2df48f3) into master (54f233d) will decrease coverage by 0.22%.
The diff coverage is n/a.

:exclamation: Current head 2df48f3 differs from pull request most recent head 6cdd231. Consider uploading reports for the commit 6cdd231 to get more accurate results

@@             Coverage Diff              @@
##             master    #4166      +/-   ##
============================================
- Coverage     70.21%   69.99%   -0.23%
+ Complexity     7143     7106      -37
============================================
  Files           964      964
  Lines         26863    26863
  Branches       2390     2390
============================================
- Hits          18862    18802      -60
- Misses         6550     6607      +57
- Partials       1451     1454       +3

Impacted Files
Coverage Δ

...controller/ShenyuClientHttpRegistryController.java
77.77% <0.00%> (-22.23%)
:arrow_down:

...ava/org/apache/shenyu/admin/utils/SessionUtil.java
78.94% <0.00%> (-15.79%)
:arrow_down:

...ache/shenyu/plugin/grpc/cache/GrpcClientCache.java
87.50% <0.00%> (-12.50%)
:arrow_down:

...ruptor/RegisterClientServerDisruptorPublisher.java
52.94% <0.00%> (-11.77%)
:arrow_down:

...henyu/plugin/grpc/resolver/ShenyuNameResolver.java
54.25% <0.00%> (-9.58%)
:arrow_down:

.../apache/shenyu/admin/model/dto/BatchCommonDTO.java
50.00% <0.00%> (-7.15%)
:arrow_down:

...apache/shenyu/admin/model/dto/AuthPathWarpDTO.java
57.14% <0.00%> (-7.15%)
:arrow_down:

...enyu/admin/service/impl/PermissionServiceImpl.java
34.61% <0.00%> (-6.42%)
:arrow_down:

...org/apache/shenyu/admin/model/dto/AuthPathDTO.java
58.82% <0.00%> (-5.89%)
:arrow_down:

...va/org/apache/shenyu/common/timer/TimingWheel.java
85.71% <0.00%> (-4.77%)
:arrow_down:

... and 29 more

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


Internal speakers on MacBook Pro sounds thin and tinny

I have no bass any more after installing emac 2.1 which I also noticed on the previous version as well. When I use a flat profile with no eq the internal speakers sound very tinny and nothing like they used to sound before I installed the software.

I don't experience such issue on my 2015 MacBook Pro. You can always adjust the sound to your liking (kinda the purpose of the app) It might be the case that Apple does some post-processing to make the sound better coming out of the speakers.

There is a difference when I use eqmac in flat to the sound when I quit the app and use the Mac sound. Shouldn’t the flat response sound the same as  Mac’s audio since nothing is supposed to be eq’d as yet?

On 26 Nov 2017, at 20:38, Roman Kisil<EMAIL_ADDRESS>wrote:
I don't experience such issue on my 2015 MacBook Pro. You can always adjust the sound to your liking (kinda the purpose of the app) It might be the case that Apple does some post-processing to make the sound better coming out of the speakers.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/romankisil/eqMac2/issues/101#issuecomment-347036578, or mute the thread https://github.com/notifications/unsubscribe-auth/AgV4biJ4hYfp89IRvsIlPd6YiYgT0m-2ks5s6cxRgaJpZM4Qq4fV.

Yeah it should, but I won't look into this as I can't fix it if I don't experience the problem :)

Hey everyone. Excuse me for the lack of updates. I have been really busy on the new version of eqMac. I am happy to announce an open beta release.
You can download it here: https://eqmac.app
There are a lot of known bugs that I could not fix just yet:

Driver not installing from the first try. Sometimes need to restart eqMac a couple times.
Complex Device switching logic is not perfect (for example when switching off the device that's currently playing, plugging in new device etc.)
UI takes long to load (a lot can be done to improve that)
UI dimensions are not synching properly
There's a noticeable audio delay in the playback, so not suitable for video yet (I know where the problem lies just haven't had the time to fix yet)
Sometimes the playback doesn't start, in that case please try to switch the output to the device you want and eqMac should try and build the audio pipeline again.

So as you can see there's still some work to be done but at least it's a completely modern tech stack that I am fully willing to work on: 100% Swift, User Space driver and a Web UI, instead of C++, Kernel Drivers. As I said, should work on Catalina and should not have any pesky ear drum blasting volume bugs (at least I didn't come across any).
I'm open to feedback and issue reports here: https://github.com/bitgapp/eqmac
That repo will become the main repo for eqMac and this repo will be archived eventually.
And yes it's called just eqMac, not eqMac3 :) And it will stay that way forever.

Hey everyone 👋
I will be open sourcing the new version of eqMac, therefore, I am closing this issue and repository as I will be transfering nodeful/eqMac2 repo to bitgapp/eqMac
Please bear with me for the next couple of days as I setup that new repo. The eqMac2 code will be living in the old-eqMac2 branch if anyone is interested in referencing it for whatever reason.
Please try out the new version and report bugs to the new repo.
Thanks for using eqMac.


⚠ Bofrost.de has degraded performance

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

HTTP code: 200
Response time: 1368 ms

Resolved: Bofrost.de performance has improved in bf46e64 after 26 minutes.


Remove overall padding

I want to remove the padding on the left:

and have it like so:

But when I try to apply the following :
[treecontrol] li {
    padding: 0;
}

It sets the padding of the children to 0px too:

Is it possible to achieve desired result?

[treecontrol] > ul > li {
    padding: 0;
}

SO question

Just do
treecontrol treeitem li {
padding-left: 20px;
}
treecontrol li {
padding: 0;
}


False positive of Layout/MultilineOperationIndentation in methods as argument

When it just in method — everything is OK. But if we add class-method before method definition — there is false positive.

Expected behavior
No offenses.
Actual behavior
test.rb:3:5: C: Layout/MultilineOperationIndentation: Align the operands of an expression spanning multiple lines.
    another
    ^^^^^^^

Steps to reproduce the problem
bar def foo
  one &&
    another
end

RuboCop version
$ rubocop -V
0.52.1 (using Parser <IP_ADDRESS>, running on ruby 2.5.0 x86_64-linux)

The bug is still here.


Requested file format unknown

I have added the addon, and it is working so far, however I get the following message when playing:

Is this something I can ignore? If so, is there a way to disable the window from showing?

Seems I just needed to create a new inherited scene. That made the error go away.


Applying redis version bump.

Bump PR for redis version.

This is a duplicate with https://github.com/hashgraph/hedera-json-rpc-relay/pull/1742
We should close this and continue work on the first ticket.


39 carousel component

acf-export-2024-07-22.json

Uploading acf-export-2024-07-26.json…

acf-export-2024-08-01.json


i want to show timepicker in spinner mode . can you guide for same ?

You could set the styling attribute TimePicker.timePickerMode to "spinner" that forces the dialog to use the spinners instead of the clock face.
Here's an example (haven't tested it):
<resources>
    <style name="AppTheme" parent="Theme.MaterialComponents.Light.DarkActionBar">
        <item name="timePickerStyle">@style/MyTimePicker</item>
    </style>

    <style name="MyTimePicker"<EMAIL_ADDRESS>        <item name="timePickerMode">spinner</item>
    </style>
</resources>


Cannot generate Kotlin models

Tutorial - https://www.apollographql.com/docs/android/tutorial/03-write-your-first-query/
I am on the "initial" branch and cannot generate Kotlin models at the end of step 3.
I have tried rebuilding the project, running generateApolloSources , ran the app on a device.
Apollo version - 2.5.5

Hi 👋 ! What is the output of generateApolloSources? Do it find sources or does it show NO-SOURCE?
Also could you push  your fork to github, that might just be easier to investigate.

Hi,
Output for generateApolloSources
Selected primary task ':app:generateApolloSources' from project :app file or directory 'AndroidStudioProjects/apollo-android-tutorial/app/src/test/graphql', not found file or directory 'AndroidStudioProjects/apollo-android-tutorial/app/src/testDebug/graphql', not found file or directory 'AndroidStudioProjects/apollo-android-tutorial/app/src/main/graphql', not found file or directory 'AndroidStudioProjects/apollo-android-tutorial/app/src/debug/graphql', not found file or directory 'AndroidStudioProjects/apollo-android-tutorial/app/src/main/graphql', not found file or directory 'AndroidStudioProjects/apollo-android-tutorial/app/src/release/graphql', not found file or directory 'AndroidStudioProjects/apollo-android-tutorial/app/src/androidTest/graphql', not found file or directory 'AndroidStudioProjects/apollo-android-tutorial/app/src/androidTestDebug/graphql', not found file or directory 'AndroidStudioProjects/apollo-android-tutorial/app/src/test/graphql', not found file or directory 'AndroidStudioProjects/apollo-android-tutorial/app/src/testRelease/graphql', not found
The command to download the schema creates the graphql directory outside the app module and the gradle task expects it to be inside the app module.
mkdir -p app/src/main/graphql/com/example/rocketreserver/ ./gradlew :app:downloadApolloSchema --endpoint='https://apollo-fullstack-tutorial.herokuapp.com/' --schema='src/main/graphql/com/example/rocketreserver/schema.json'
I moved the graphql directory to app and it works now.


Add: Added JavaScript book

DOM Enlightenment - JavaScript Book Added
What does this PR do?
Add resource(s)
For resources
Description
Added DOM Enlightenment - JavaScript book
Why is this valuable (or not)?
This book is based on modern JavaScript.
How do we know it's really free?
you can check out the book here
For book lists, is it a book? For course lists, is it a course? etc.
yes, it is a book
Checklist:

[x] Read our contributing guidelines.
[x] Search for duplicates.
[x] Include author(s) and platform where appropriate.
[x] Put lists in alphabetical order, correct spacing.
[ ] Add needed indications (PDF, access notes, under construction).
[x] Used an informative name for this pull request.

Follow-up

Check the status of GitHub Actions and resolve any reported warnings!

Thanks, and welcome to free-programming-books!


Proposal: allow for selectable sets of datasets/configuration

Right now we store all defaults and configuration tables in the ctapipe-extra/ctapipe_resources package, and the user can override them by setting the $CTAPIPE_SVC_PATH environment variable (which takes a directory or list of directories where to look for service files).
This is fine for simple use cases, but is getting cumbersome when we switch simulation versions, or even later when we have multiple sites/configs.  I propose that we think of developing a more flexible system, which can allow sub-sets to be enabled by the user using a single name describing the full set of data.
The simplest realization is to start to structure the ctapipe_resources directories, and allow for these different data versions (note in CTA parlance, these are SVC (service) data).  Perhaps:
ctapipe_resources/
    prod3b/
    prod4/
    ...

In each sub-dir, we could further organize things to make it more clear:
prod3b/
     cameras/
          NectarCam.camgeom.fits.gz
          FlashCam.camgeom.fits.gz
          ...
          gain_channel_thresholds.ecsv
          optics.ecsv.txt
    atmosphere/
          paranal.atmprof.fits.gz
          atmprof26.ecsv.txt
    data/
          gamma_test.simtel.gz
          gamma_test_large.simtel.gz

Then we provide a method to set the current data sub-set ctapipe.set_svc_data("prod3b") or something like that if they want to switch from the default. From an API standpoint, not much would change, since utils.get_dataset_path() and related functions would remain the same, just automatically look in whichever one is specified, and they already have support for searching subdirectories.
Users could still specify custom paths using $CTAPIPE_SVC_PATH, but those should point to paths with sub-dirs in them with new names (e.g my_custom_svc_data/, which would be enabled by ctapipe.set_svc_data("my_custom_svc_data").
Any better ideas?

Note: this concept is not so different from how the IRFs are being organized for the science tools (currently using a caldb structure, but that's a bit cumbersome)

Hi Karl,
With the configurations, would it make any sense to connect with the yaml files currently being constructed for the MC? Although I guess the problem might be that parameters are stored in non-ideal formats...
https://forge.in2p3.fr/projects/cta_analysis-and-simulations/repository/show/Simulations/MCModelDescription/trunk/configReports

Would it be possible to have the FACT open data release integrated into this? There aren't many 'real' datasets that are publicly available and that would be a useful thing to have for MC/simulation comparison purposes.

A comment on this proposal: we should also look at what is done in gammapy,  where they use an index table on a server to point to datasets, which makes it easy to move the data to other servers, etc, without changing the code.

Should use the reference data sets to generate the data


No such file or directory: 'checkpoints/dummy

When I firstly run train.py, it keep reminding me No such file or directory: 'checkpoints/dummy.  How to skip verify_checkpoint_directory()

you can comment it out or just create this directory


Request textDocument/documentSymbol failed. - Adobe ColdFusion Builder extension

Does this issue occur when all extensions are disabled?: No

VS Code Version:  v 1.80.1
OS Version: Windows 10.0.19045

Steps to Reproduce:

With ColdFusionBuilder extension installed, open compatible file (.cfm/.cfc).
Start typing.
Not immediately but at somepoint, there appears to be a memory error or limit (within a minute or 2). Then many errors are sent to
the output window - for each keystroke. (see attached)
vscode_cf_builder_err_output.txt

Thanks for creating this issue! It looks like you may be using an old version of VS Code, the latest stable release is 1.80.2. Please try upgrading to the latest version and checking whether this issue remains.
Happy Coding!

That was a typo. My version is 1.80.2 (the latest)
The problem persist.

/causedByExtension

This issue is caused by an extension, please file it with the repository (or contact) the extension has linked in its overview in VS Code or the marketplace for VS Code. See also our issue reporting guidelines. If you don't know which extension is causing the problem, you can run Help: Start extension bisect from the command palette (F1) to help identify the problem extension.
Happy Coding!


Refactored the Groundhog attack, implemented other set features, and added tests

Major changes

Refactored the SetRepresentation classes (renamed  SetFeature) needed by GroundhogAttack, and their respective subclasses. Major changes are in the set_classifiers.py file.
Added histogram features from Stadler et al., as a HistSetFeature object.
Added tests for the features (specifically NaiveSetFeature and HistSetFeature) and GroundhogAttack in tests/test_attacks.py.

Minor changes:

Renamed Groundhog  -> GroundhogAttack
Linting with black.
Removed classes for individual classifiers (e.g. RFClassifier) that were essentially wrappers of sklearn classifiers.
Added typing and comments in several files.

Merging this into main as current reviewers are away.
Note that all changes made in these commits can and should be overturned if the resulting structure is found to be less usable or general than previous code.


[BUG] from foo import * imports should be ignored in stub files

Hi!
I see pycln recently added support for .pyi files. That's awesome!
I'm interested in possibly using pycln in typeshed's CI. However, it seems like there's one part of PEP 484 that pycln isn't quite compliant with when it comes to stub files, namely this:

[...] all objects imported into a stub using from ... import * are considered exported. (This makes it easier to re-export all objects from a given module that may vary by Python version.)

If I run pycln on typeshed at the moment, it erroneously deletes a large number of "unused" from foo import * imports. These imports are all deliberate -- if a type checker sees a from foo import * import in a stub file bar.pyi, it understands all symbols from module foo to be re-exported in module bar.
Playing around with pycln, it looks like this one-line patch solves the problem:
diff --git a/pycln/utils/refactor.py b/pycln/utils/refactor.py
index 4739e72..a241fc8 100644
--- a/pycln/utils/refactor.py
+++ b/pycln/utils/refactor.py
@@ -279,7 +279,7 @@ class Refactor:

                 # Expand any import '*' before checking.
                 node, is_star = self._expand_import_star(node)
-                if is_star is None:
+                if is_star is None or (is_star and self._path.is_stub):
                     continue

                 # Get set of used names.

I'd file a PR, but I'm struggling a little bit with the test suite 😅 (I'm not too familiar with the advanced features of pytest, I'm afraid.)

Hi @AlexWaygood, this is a good catch.. actually, I planned to implement this within PR #150 but forgot to do so.

@AlexWaygood after doing some investigation, your patch looks good to me, so please open a PR, and I'll help you finalize it even if you are unsure where/how to write the unit tests.

+ @AlexWaygood  I'm delighted to see you supporting the project. Thanks a lot bro!

@AlexWaygood I'm delighted to see you supporting the project. Thanks a lot bro!

Thank you for the very cool tool!

@AlexWaygood I've just published a new release including the fix for this bug: v2.0.2.

@AlexWaygood I've just published a new release including the fix for this bug: v2.0.2.

Awesome, thanks so much for your help!


Problem encountered on https://dotnet.microsoft.com/en-us/learn/dotnet/hello-world-tutorial/run

Problem encountered on https://dotnet.microsoft.com/en-us/learn/dotnet/hello-world-tutorial/run
Operating System: windows
Provide details about the problem you're experiencing. Include your operating system version, exact error message, code sample, and anything else that is relevant.

Duplicate of #8701


KAFKA-10891:The control plane needs to force the validation of requests from the controller

…ect the control plane to be safe and stable.
At present,data requests and control requests use different endpoints,these endpoints are all registered with ZooKeeper.This results in the client business still having access to the metadata that controls the endpoint,this returns the control endpoint back to the data endpoint.One option is to turn on the firewall to disable client access to the control port.But does Kafka need to do something on its own to handle data request access control endpoints to deny those requests.I modified part of the code to deny access control endpoints for data requests.This is the first time I've submitted code to the community,I hope to get some advice, if possible, I will continue to work on this.

@MayureshGharat Can you take a look at this?

@showuon This is my first time to submit pr to the community.Could you give me some suggestions?Thank you very much!

@wenbingshen , thanks for the PR. Some comments left, but most importantly, we don't know which JIRA ticket you're fixing. Please check the contributing code change page. Thank you.

The PR title should usually be of the form KAFKA-xxxx: Title, where KAFKA-xxxx is the relevant JIRA id and Title may be the JIRA's title or a more specific title describing the PR itself. For trivial cases where a JIRA is not required (see JIRA section for more details) MINOR: or HOTFIX: can be used as the PR title prefix.

@showuon  Thank you very much for your suggestions and comments,In order to meet the requirement of submitting PR, do I need to submit a PR again?I'm going to create a new jIRA ID for this PR and add test cases.

@showuon Thank you very much for your suggestions and comments,In order to meet the requirement of submitting PR, do I need to submit a PR again?I'm going to create a new jIRA ID for this PR and add test cases.

No, just update in this PR directly.

@showuon Hello,I made some changes and added a test case.

@showuon  Thank you,Luke,I did accidentally use the formatting tool.I submitted the latest code.Thank you very much for your help.

@chia7712 Can you take a look at this PR?Thank you.

@chia7712 Can you take a look at this PR?Thank you.

@wenbingshen Thanks for your patch. This is an interesting issue. Please take a look at following questions.

Is there a case that client sends controller-unrelated request to controller plane ? I guess that users define "bootstrap servers" as controller listener name can reproduce this issue?
How to keep the compatibility if our kafka users do have this cases (This PR can break such behavior)?
Does inter.broker.listener.name have similar issue?

@wenbingshen Thanks for your patch. This is an interesting issue. Please take a look at following questions.

Is there a case that client sends controller-unrelated request to controller plane ? I guess that users define "bootstrap servers" as controller listener name can reproduce this issue?
How to keep the compatibility if our kafka users do have this cases (This PR can break such behavior)?
Does inter.broker.listener.name have similar issue?

@chia7712  Thanks for your comments and questions.I think we just need to completely separate control requests from data requests, and provide the user with an optional configuration to control whether the control request is forced to validate, which can be disabled by default.There is no way to do this now, when the user needs to fully protect control from possible client requests.This patch can do that.On the KIP-291 basis,we can ignore inter.broker.listener.name.

@chia7712  Thanks for your comments and questions.I think we just need to completely separate control requests from data requests, and provide the user with an optional configuration to control whether the control request is forced to validate, which can be disabled by default.There is no way to do this now, when the user needs to fully protect control from possible client requests.This patch can do that.On the KIP-291 basis,we can ignore inter.broker.listener.name.

@wenbingshen Thanks for your response.

I think we just need to completely separate control requests from data requests, and provide the user with an optional configuration to control whether the control request is forced to validate, which can be disabled by default.There is no way to do this now, when the user needs to fully protect control from possible client requests.

I feel this needs a KIP as it proposed a new config and it is related to compatibility.
for another, was my comment valid?

Is there a case that client sends controller-unrelated request to controller plane ? I guess that users define "bootstrap servers" as controller listener name can reproduce this issue?

I'm curious whether there are other ways which can produce this issue.

@wenbingshen Thanks for your response.

I think we just need to completely separate control requests from data requests, and provide the user with an optional configuration to control whether the control request is forced to validate, which can be disabled by default.There is no way to do this now, when the user needs to fully protect control from possible client requests.

I feel this needs a KIP as it proposed a new config and it is related to compatibility.
for another, was my comment valid?

Is there a case that client sends controller-unrelated request to controller plane ? I guess that users define "bootstrap servers" as controller listener name can reproduce this issue?

I'm curious whether there are other ways which can produce this issue.

@chia7712 Thanks for your comment.I'm sorry that I forgot to reply to the first question. During my use, I found no other way to cause this problem except to define the configuration "bootstrap.servers" as the control plane.I haven't created KIP yet. If possible, could you give me some guidance?Thanks.

@wenbingshen Thanks for your response.

I think we just need to completely separate control requests from data requests, and provide the user with an optional configuration to control whether the control request is forced to validate, which can be disabled by default.There is no way to do this now, when the user needs to fully protect control from possible client requests.

I feel this needs a KIP as it proposed a new config and it is related to compatibility.
for another, was my comment valid?

Is there a case that client sends controller-unrelated request to controller plane ? I guess that users define "bootstrap servers" as controller listener name can reproduce this issue?

I'm curious whether there are other ways which can produce this issue.

@chia7712 Thanks for your comment.I'm sorry that I forgot to reply to the first question. During my use, I found no other way to cause this problem except to define the configuration "bootstrap.servers" as the control plane.I haven't created KIP yet. If possible, could you give me some guidance?Thanks.

@wenbingshen Thanks for your response.

I think we just need to completely separate control requests from data requests, and provide the user with an optional configuration to control whether the control request is forced to validate, which can be disabled by default.There is no way to do this now, when the user needs to fully protect control from possible client requests.

I feel this needs a KIP as it proposed a new config and it is related to compatibility.
for another, was my comment valid?

Is there a case that client sends controller-unrelated request to controller plane ? I guess that users define "bootstrap servers" as controller listener name can reproduce this issue?

I'm curious whether there are other ways which can produce this issue.

@chia7712 Thanks for your comment.I'm sorry that I forgot to reply to the first question. During my use, I found no other way to cause this problem except to define the configuration "bootstrap.servers" as the control plane.I haven't created KIP yet. If possible, could you give me some guidance?Thanks.

@wenbingshen Thanks for your response.

I think we just need to completely separate control requests from data requests, and provide the user with an optional configuration to control whether the control request is forced to validate, which can be disabled by default.There is no way to do this now, when the user needs to fully protect control from possible client requests.

I feel this needs a KIP as it proposed a new config and it is related to compatibility.
for another, was my comment valid?

Is there a case that client sends controller-unrelated request to controller plane ? I guess that users define "bootstrap servers" as controller listener name can reproduce this issue?

I'm curious whether there are other ways which can produce this issue.

@chia7712 Thanks for your comment.I'm sorry that I forgot to reply to the first question. During my use, I found no other way to cause this problem except to define the configuration "bootstrap.servers" as the control plane.I haven't created KIP yet. If possible, could you give me some guidance?Thanks.

@wenbingshen Thanks for your response.

I think we just need to completely separate control requests from data requests, and provide the user with an optional configuration to control whether the control request is forced to validate, which can be disabled by default.There is no way to do this now, when the user needs to fully protect control from possible client requests.

I feel this needs a KIP as it proposed a new config and it is related to compatibility.
for another, was my comment valid?

Is there a case that client sends controller-unrelated request to controller plane ? I guess that users define "bootstrap servers" as controller listener name can reproduce this issue?

I'm curious whether there are other ways which can produce this issue.

@wenbingshen see https://cwiki.apache.org/confluence/display/KAFKA/Kafka+Improvement+Proposals for guidance. In short, you have to complete following tasks.

require KIP permission (post to<EMAIL_ADDRESS>write KIP
post KIP to<EMAIL_ADDRESS>to get discussion and votes

@wenbingshen see https://cwiki.apache.org/confluence/display/KAFKA/Kafka+Improvement+Proposals for guidance. In short, you have to complete following tasks.

require KIP permission (post to<EMAIL_ADDRESS>write KIP
post KIP to<EMAIL_ADDRESS>to get discussion and votes

@chia7712  Thanks for your guidance. I will apply for KIP permissions and learn how to create KIP.

@wenbingshen see https://cwiki.apache.org/confluence/display/KAFKA/Kafka+Improvement+Proposals for guidance. In short, you have to complete following tasks.

require KIP permission (post to<EMAIL_ADDRESS>write KIP
post KIP to<EMAIL_ADDRESS>to get discussion and votes

@chia7712  Thanks for your guidance. I will apply for KIP permissions and learn how to create KIP.

@wenbingshen see https://cwiki.apache.org/confluence/display/KAFKA/Kafka+Improvement+Proposals for guidance. In short, you have to complete following tasks.

require KIP permission (post to<EMAIL_ADDRESS>write KIP
post KIP to<EMAIL_ADDRESS>to get discussion and votes


set config defaults and update example configmap

if the bind, metrics section is not defined in the configmap, set the defaults.

Sorry for the huge delay - completely missed the notification

Rearranged the imports @mrIncompetent

Thank you very much for the PR :)


Max size for payload?

Hi,
If I try to send a payload with a size > 254 bytes then the payload is not send (or truncate) and the Topic is good.
There is a limit for the message size that is set to 254 bytes?
Is it possible to change it?
Thanks for your answer.
Gilles
***  I have a Mega with UIPEthernet.h and a 28J60.
***  I have make the change on pubsubclient from MrDunk for increase the buffer size (here for see it : https://github.com/knolleary/pubsubclient/pull/282/commits/a257083a66433e71fc43fb714da499016589ece4 )

Hi
Sorry I make a mistake in my code and I declare a byte to the length of the payload.
So now it's good and pubsubclient send over than 254 bytes.
Thanks a lot for this very util library.
Gilles


update compare in the homepage to show metro areas

Show metros by default in the homepage
Sort metro areas by population (larger population first)

 [SEO] Switch compare on homepage to "metro" by default and sort by population

 [SEO] Switch compare on homepage to "metro" by default and sort by population


Using variable values within the column labels

Good morning!
First of all, I love this new table features. I have to work on it frequently and it was the last not very versatile element of grafana.
I would like to ask you to add the possibility of using the values of the variables within the column labels. As you can see they are not loaded correctly.

@cgravita Thank you for the feature request. We will implement in the upcoming release.


Script update new version

all scripts combined to manage resources and make features available to all scripts

updated to improve script performance.
all scripts now located in one directory.
crypto tickers also added to script


quic: HTTP/3 upstream initial checkin

This is working end to end for a number of the protocol integration tests, but still needs a bunch of work before it's production ready (I triaged some of the excludes but not all, watermarks not working etc).
I think given this works for the happy path it's worth checking in, and I can iterate on getting individual tests working and doing the TODOs.
Risk Level: low (minor core refactors, major changes are all behind hidden config)
Testing: integration tests, some unit tests
Docs Changes: n/a
Release Notes: n/a
#14829

cc @DavidSchinazi

comments should be addressed, PTAL

ping!

(mac failure is the known timeout issue)


Accessibility Project

Activities in this branch:-
-Created an accessibility branch following the GitHub workflow.
-Checked for and solved accessibility issues as follows:
I) Page titles - Window titles edited to adequately describe the content of the page.
ii) Images text alternatives - All images are edited with appropriate alt text.
iii) Text headings - All text headings are set to follow the accessibility standard.
iv) Color contrast - No issues found.
v) Resize - No issues found.
vi) Interaction - Labels added and made invisible to fit Figma design.
vii) Moving content -  No issues found.
viii) Multimedia - No issues found.
ix) The basic structure of the page - No issues found.

@franclobo
I really appreciate your constructive review, it took us time to actually get the contrast using the Aditus tool you recommended. at first, the wave tool passed our color choice before your review, I must say we learned more using the aditus tool. Fortunately,
we have passed the contrast test using the aditus tool and here is evidence for that:

Thanks!


Support for embedding

I gave a shot at embedding JPype in a host JVM.  The pattern for this was actually pretty easy.  The good news is we already have 95% of the machinery required to make it work, so the delta from a Python host running an embedded Java to a Java host running an embedded Python is actually very small.  I also poked at some of the existing embedded solutions such as jep.   They are missing some of the finer points of interacting between Java like lifetime control and utf encoding, etc, so moving JPype in this direction will actually be a worthy improvement.
We just need to identify the location of the Python shared library from an external copy of the JPype jar.  Then we add a few symbols to the shared library so that Java can successfully name bind.  Drop in the routine to start CPython (with a PYTHONHOME set), call the JPype module init and we have an embedded copy of Python to bridge to.
Based on the preliminary work, here are the tasks required to complete it.

[ ] Wrap the object protocol in a Java object.
[ ] Set up the converse from a raw Python object to Java as a explicit conversion.
[ ] Complete a round trip with Python object.
[ ] Come up with a scheme to keep the JPype jar separated from the shared library (install scripts and bootloader)
[ ] Come up with a scheme to find the embedded info with the dll, (version of CPython, default location of home, name of JPype shared library).
[ ] Define a start up routine for the JPype module from within embedded.
[ ] Disable functions that do not apply in embedded context (startJVM, shutdownJVM)
[ ] Register our ScriptEngine with Java.
[ ] Add a test suite for embedded Python from Java.

Several of these tasks were already in the plans, so the delta for this feature is modest.  Most of the work is going to be in the bootloader and test suite.
The one big difficulty is that if we are providing both directions to the bridge we end up in a very odd place in terms of our packaging system.  When embedding Java we would need to look like a Python package for an installer for Pypi.  But if we are providing embedded Python we would be best to look like a maven repo.  I don't see how we can accomplish looking like both easily and maven is so picky about how it works in terms of structure it would force reorganization which we really do not want to do.

Closed due to JVM limitations.  Will give another shot at this later.


cannot print images, barcode, QRCODE,

I am using the code from the TSC.JS example and it does not print the expected result correctly.
I am working with react native 0.63 and with an Android 8 device

Try using BluetoothEscposPrinter instead of BluetoothTscPrinter

was fixed? I have the same problem.

oh this BluetoothEscposPrinter work to me: await BluetoothEscposPrinter.printPic(logo, {width: 250, height: 50});


Feature Request: Add cell command (with keystroke)

Jupyter has a command to add a new cell. Be nice if we had one too.

This is being done with native jupyter support.


reordering of sticky headers in Z direction when embedded within a scrollview

I have a vertical scrollview that uses sticky headers.  I have placed this vertical scrollview within a horizontal, paginated scrollview, so my app can switch tabs between two vertical scrollviews.  For some reason, the ordering of the sticky headers in the Z direction is acting funky only when it is within the horizontal scrollview.  As I scroll, it constantly reorders the surfaces within the sticky header in the Z direction.  I even tried to translate the surfaces in the sticky header itself in the Z direction an enormous amount, something like 200 pixels, and it still is getting overlapped by other surfaces that it shouldn't.  When I put the vertical scrollview in my view without the paginated horizontal scrollview, it works fine and the ordering of the surfaces is perfect.  Any ideas what the problem might be?

Actually I just isolated the error.  The actual problem is when my horizontal scrollview used a container to hide the overflow.  When I took out the container for the horizontal scrollview, it stopped the sticky headers from doing the weird reordering.

Did you use overflow:hidden on your container? There is a common bug when using oveflow:hidden on your surfaces. See https://famo.us/guides/pitfalls

Overflow: hidden does indeed cause z-indexing issues, be careful using it:
https://github.com/Famous/famous/issues/493
You could still use useContainer: true, but I would advise disabling 'overflow: hidden', by also setting 'overflow: {}', like this:
var scrollView = new FlexScrollView({
  useContainer: true,
  container: {}
});

And then add a background surface behind the scrollviews and make sure that the scrollviews have different z-index so that the surfaces properly underlap and overlap. You have to be a bit creative with this, because of this annoying browser bug.

Thank you I found a workaround in my app.


how to get xPub from account/code

On the Bitsurance page I get the account code from the account selector. With that code O can filter the account object, but that object does not contain the xPub I need in the next step to call the bitsurance widget.

OK got it


Add an MTRDevice_Concrete method for checking whether a data-value satisfies some expectation.

Fast-tracking platform-specific change with platform owner review.


Fix maximum entry tracking for hash map type maps.

Signed-off-by: Alan Jowett<EMAIL_ADDRESS>Partially resolves: #557
Description
Correctly track maximum entries in hash_maps.
Testing
CI/CD
Documentation
No.

Converting to draft until we know why it's triggering NMR crash.


Enable explicit API mode

This PR enables the explicit API mode in the Kotlin compiler. This means that all public API must be explicitly annotated as such, and provide explicit return types, or the compilation will break. This ensures better API stability and avoids unexpected return types (it already caught something, which is fixed in this PR).
Since this PR is immense, know that it is:

Enabling the explicit API mode
Adding public modifiers and explicit return types until the project runs again
Reformatting everything
Cleaning up some minor style problems in code that was changed anyway
Providing better error messages for missing composition locals, that point to the lack of theme (that code was impacted anyway from the rest of the changes)

Closes #228

Switching reviewer to @devkanro since @lamba92 is on vacation

All fixed :)

Hopefully it's all fixed for real now :P


get rid of the hamburger menu

it only has one option, also boo hamburger menus

https://github.com/mrjohnmorrow/mrjohnmorrow.github.io/commit/838ef26552073d0e5d2317d9b669e02d7da89773


New cheatcode interface

Which components does the task require to be changed? (think hard pls)
snforge
Description
We should redo the cheating interfaces so that they make more sense

One global variant with _global suffix (no span, since it doesn't make much sense)
One granular variant to target only one contract
Removal of Multiple because it can be combined all of the above
Change naming of span to include ContractCalls instead of TargetCalls
For example:

set_caller_address_global(value)
set_caller_address(contract_address, value, span)

A few questions come up:

How should cancelling the cheatcodes be implemented?

restore_caller_address? revert / reset / undo / rollback?

with the proposed approach we'd need also a cheatcode for cancelling the cheatcode globally (so 4 functions per cheatcode in total)

Why not have the cheatcodes be traits on ContractAddress? Or perhaps on dispatcher but it'd require creating one previous to cheating an address.

address.set_caller_address(new_caller)

address.caller_address().set(new_caller)

I've got this feeling that right now with the start_ and stop_ prefixed functions it's kind of easy to just glance at the test and more or less know the lifetimes of cheats. Were they named similarly it might be a bit harder.


Let ProjectConnection extend Closeable interface

ProjectConnection should extend Closeable interface for use of try-with-resources pattern.
Signed-off-by: Stefan Lehmann<EMAIL_ADDRESS>Context

Contributor Checklist

[x] Review Contribution Guidelines
[x] Make sure that all commits are signed off to indicate that you agree to the terms of Developer Certificate of Origin.
[x] Check "Allow edit from maintainers" option in pull request so that additional changes can be pushed by Gradle team
[x] Provide integration tests (under <subproject>/src/integTest) to verify changes from a user perspective
[x] Provide unit tests (under <subproject>/src/test) to verify logic
[x] Update User Guide, DSL Reference, and Javadoc for public-facing changes
[x] Ensure that tests pass locally: ./gradlew <changed-subproject>:check

Gradle Core Team Checklist

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation
[ ] Recognize contributor in release notes

I've added the signoff manually as GitHub doesnt seem to have that feature in the Web GUI.

Hope you can accept my full name ;)

You can edit the line with git commit --amend. That way we don't need to make am exception from the legal process :)

Closed it. I've opened a new one using my feature branch.


Is it possible to the make the scrolling vertical instead of horizontal?

Would like to try this out in my project with all of the same methods but in a vertical scrolling manner!
Thanks :) :+1:

Implemented in v4.2.0, you can set verticalScroll to YES, see demo for detail usage.


Allow Broker authors to define the JSON Schema for request output.

Fixes https://github.com/openservicebrokerapi/servicebroker/issues/116
We found it useful to define the full output of responses vs defining just the credentials or disk info sections. This gives more flexibility to Broker authors.

I like this :slightly_smiling_face:
One typo is breaking the build so I'll wait until it's fixed.

LGTM

LGTM

@n3wscott I think we need a proper proposal for this one, outlining what the benefits are to both service broker authors and platform authors, guidance of how platforms could use this schema, etc.

outlining what the benefits are to both service broker authors and platform authors, guidance of how platforms could use this schema, etc.

There is a number of things we would like to do with this mechanism at google, and it all revolves around understanding the components that are returned from instance creation and binding. Binding more so. Because brokers are allowed to return anything at all from a Binding, we require a way to programmatically understand what the contents of the binding is.
It comes down to simply understanding what the response body is from the broker.

A simple Design Doc for this proposal: https://docs.google.com/document/d/1pSLntUlkcQ7mqYE4BnGH3saFg4IbSGkYHlu_2-AgTmg/edit?usp=sharing

I've just noticed that this PR doesn't include the necessary changes to https://github.com/openservicebrokerapi/servicebroker/blob/master/openapi.yaml

I've just noticed that this PR doesn't include the necessary changes to the OpenAPI schema

The OpenAPI doc did not exist when this was created. I will update.

@luksa

The spec implies that you can add the response schema to service_instance.create and service_instance.update in addition to service_binding.create, which was the only place where we actually need the response schema. The spec does not say whether a broker author can/should/must add the response schema in the two places I mentioned. Specifying the schema for those responses obviously shouldn't be required, since the OSBAPI spec itself defines what the response should be.

It was my explicit intention to allow service_instance.update and  service_binding.create to have defined responses. At the moment, current implementations do not use the response for update, but that will not be true in the future. None of this should be required, but it is my intention to allow some advanced behavior from platform/broker interactions with these hooks.

On a related note, the response schema defines the schema for the whole response, not just the returned credentials. This means there's overlap between what the OSBAPI spec prescribes and the broker-returned schema, so the question arises whether a broker that returns both credentials and e.g. syslog_drain_url must return the complete schema or just the schema for the credentials part?

It was also intended to encompass the entire response, not just credentials. Bindings do more than just credentials and it would be a mistake to allow defining only the sup-resources for response.credentials.

Technically, the schema in the reference catalog response in the OSBAPI spec states that the response will not contain anything other than a credentials field (this is implied by the type:object property and the lack of either a properties.syslog_drain_url or an additionalProperties:true

This was also intentional, it is an example response that does not have any other fields.

@luksa I decided on RequestResponseSchemaObject. 👍 ?
Also updated a little wording.
PTAL

It was my explicit intention to allow service_instance.update and service_binding.create to have defined responses. At the moment, current implementations do not use the response for update, but that will not be true in the future. None of this should be required, but it is my intention to allow some advanced behavior from platform/broker interactions with these hooks.

I don't think that we should add response to service instance create or update. They don't have meaning in the spec currently, there are no use cases presented for them, etc. If you feel strongly they should be part of the spec, I would suggest splitting them out of this PR and making the case for them separately. We have agreement on the other changes in this PR, as far as I can tell, so we should be able to get this merged once those changes are moved to another PR.


Validation on foreign key fields

Foreign key fields that are marked as not-null need to be validated in the UI before the data is submitted to ermrest.

Issue moved to informatics-isi-edu/chaise #736 via ZenHub


Missing Requirement.txt file

I noticed that the requirements.txt file is missing from the repository. This file is essential for setting up the project environment as it specifies all the necessary dependencies.
Could you please add the requirements.txt file to the repository? This will ensure that all contributors can easily install the required packages and maintain a consistent development environment.

@hacker-saran Great, thank you for pointing out the missing file!


I have an issue with the image loading. Images available but not displayed!!! Repository is stored in Gitlab. Site is deployed on Netlify. This bug was described before. Was it fixed?

Describe the bug

netlify-cms.js:24 TypeError: Cannot read property 'id' of undefined
at t.default.getMediaDisplayURL (netlify-cms.js:302)
at getLargeMediaClient.then.o (netlify-cms.js:302)

To Reproduce

Expected behavior

Screenshots

Applicable Versions:

Netlify CMS version: [e.g. 2.0.4]
Git provider: [Gitlab]
OS: [Linux]

Node.JS version:

CMS configuration
backend:
  name: git-gateway
  branch: master

media_folder: "src/images"
public_folder: "images"

collections:
  - name: "services"
    label: "Services"
    folder: "src/content/services"
    create: true
    fields:
      - { name: "title", label: "Title", widget: "string" }
      - { name: "order", label: "Order", widget: "number" }
      - { name: "number", label: "Number", widget: "string" }
      - { name: "image", label: "ServiceImage", widget: "image"}
      - { name: "body", label: "Body", widget: "markdown" }

Additional context

The fix hasn't been released yet, it will come out in 2.9.1. In the meantime you can use 2.9.1-beta.2.

Thanks)


Unifi Firewall Repo

@dan-hill2802 that was the plan, however it appears to be broken. Will take a look at fixing it next week

@TomTucka If the NorthOps module isn't working for you, then maybe this one?
https://registry.terraform.io/modules/dwp/repository/github/latest


:sparkles: feat: enable theming

Summary
Create POC for theming a component by doing it with the bal-badge
Use Case

Preferred Solution
No response
Additional Information
No response
Will this feature introduce breaking changes?
None
Code of Conduct

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

/cib


TypeScript compiler support

I may chew on this for unrelated reasons. Adding an addendum that with the upcoming TS 1.5, there's now a tsconfig.json which can contain many of these settings.

Cool @RichiCoder1 I would suggest you start as an addin to try out the concept.

Absolutely. I've got something prototyped out.

Probably will grab this with my Node and NPM items.

I am stronly interested in this feature. Does anybody plan to implement this (soon)?

I'm using this build task to transpile typescript file using the config file tsconfig.json
Task("Build")
.Does(() =>
{
var settings = new DotNetCoreBuildSettings
{
//OutputDirectory = config.Path.Output,
ToolPath = dotnet,
Configuration = "Release",
NoRestore = true,
NoIncremental = true // force to rebuild solution and compile ts file
//Framework = "netcoreapp2.1"
};
DotNetCoreBuild(config.Solution, settings);

});


digest: remove unnecessary error pointer formats in test

The linter was complaining about these and they are quite unnecessary.

Sorry for the premature-merge making this one necessary.
For the record: LGTM by all means.


[Feature Request] Cancellation

All methods should provide the additional optional parameter cancellationToken to cancel the corresponding HTTP request.

Done.
Nuget package: https://www.nuget.org/packages/Microsoft.Azure.Databricks.Client/1.1.1613.2


client id support + reload + configsetdir

Can you add a description on what is this PR about?

This PR was solely for testing and validation of some changes and is not intended for merging.


Maint/docker health checks

Add a Docker health check to the backend container (presence of the pid file gunicorn makes on startup)
Require a healthy backend container for the frontend/supervisor containers to start
Explicitly check for HLL_DB_PASSWORD to be set on startup and shut down with an error message if not (if the database hasn't been initialized, missing the password will cause the postgres container to fail the health check, but it will pass on subsequent startups if the database was ever seeded successfully)

When I originally added the health checks for redis/postgres I stupidly only set the supervisor container to check for it, but really we should check in the backend container and make everything else depend on a successful backend startup I think.
One downside is it is going to give scary looking error messages to people who don't have multiple servers configured even though there isn't any sort of actual problem.
Example where postgres won't start (no password set on initial setup):

Example with a healthy backend 1 but unhealthy (unconfigured backend 2 and 3):

Example with an unhealthy backend 1 and unconfigured backend 2 and 3:

Sorry, I do not dare to approve this, as I'm pretty lost with anything docker related :/

One downside is it is going to give scary looking error messages to people who don't have multiple servers configured even though there isn't any sort of actual problem.

What error message are you referring to? 🤔

When you recreate/restart containers (?)
Sounds OK, though... We may add some explanation in the README to warn this is normal as long as there is only one game server configured.

One downside is it is going to give scary looking error messages to people who don't have multiple servers configured even though there isn't any sort of actual problem.

What error message are you referring to? 🤔

Image 2 in the PR shows what it looks like, right now on master and all the releases if the server isn't configured it'll just shut down, on this branch it will actively show you an error which I think might confuse some users, but it will also make it more apparent if they have configuration problems and are trying to use those containers.
Now that I think of it, I'd like to see how this output change would interact with ansible as I use that do updates.

One downside is it is going to give scary looking error messages to people who don't have multiple servers configured even though there isn't any sort of actual problem.

What error message are you referring to? 🤔

Image 2 in the PR shows what it looks like, right now on master and all the releases if the server isn't configured it'll just shut down, on this branch it will actively show you an error which I think might confuse some users, but it will also make it more apparent if they have configuration problems and are trying to use those containers.
Now that I think of it, I'd like to see how this output change would interact with ansible as I use that do updates.

I find the extraneous containers annoying so I edit the docker-compose.yml to only spin-up the containers that are needed for the hll servers that we have.
Would it be less scary for people to choose between pre-canned versions of the docker-compose.yml that match their situation?

I find the extraneous containers annoying so I edit the docker-compose.yml to only spin-up the containers that are needed for the hll servers that we have.
Would it be less scary for people to choose between pre-canned versions of the docker-compose.yml that match their situation?

That would add files and a configuration step, though. Now it's working OK without any need to change anything.
Users that can rent a VPS and pull a github project on it shouldn't be scared of some warning messages, as the'yre supposed to have read the documentation, where it's been said "these messages are OK if you're not running more than one server".
... I would leave it as it is...

One downside is it is going to give scary looking error messages to people who don't have multiple servers configured even though there isn't any sort of actual problem.

What error message are you referring to? 🤔

Image 2 in the PR shows what it looks like, right now on master and all the releases if the server isn't configured it'll just shut down, on this branch it will actively show you an error which I think might confuse some users, but it will also make it more apparent if they have configuration problems and are trying to use those containers.
Now that I think of it, I'd like to see how this output change would interact with ansible as I use that do updates.

I find the extraneous containers annoying so I edit the docker-compose.yml to only spin-up the containers that are needed for the hll servers that we have.
Would it be less scary for people to choose between pre-canned versions of the docker-compose.yml that match their situation?

We could certainly do that yeah but then we'd have more of a maintenance burden like elguillermo noted, so I'd prefer not. It's already tough for us that to remember to update the arm versions of stuff.
I think it's okay if we have it report as an error, but I'm going to do some more homework because I use ansible to update RCONs and if it's going to cause ansible playbooks to fail, I'll find some other way to do this.

One downside is it is going to give scary looking error messages to people who don't have multiple servers configured even though there isn't any sort of actual problem.

What error message are you referring to? 🤔

Image 2 in the PR shows what it looks like, right now on master and all the releases if the server isn't configured it'll just shut down, on this branch it will actively show you an error which I think might confuse some users, but it will also make it more apparent if they have configuration problems and are trying to use those containers.
Now that I think of it, I'd like to see how this output change would interact with ansible as I use that do updates.

I find the extraneous containers annoying so I edit the docker-compose.yml to only spin-up the containers that are needed for the hll servers that we have.
Would it be less scary for people to choose between pre-canned versions of the docker-compose.yml that match their situation?

We could certainly do that yeah but then we'd have more of a maintenance burden like elguillermo noted, so I'd prefer not. It's already tough for us that to remember to update the arm versions of stuff.
I think it's okay if we have it report as an error, but I'm going to do some more homework because I use ansible to update RCONs and if it's going to cause ansible playbooks to fail, I'll find some other way to do this.

NP. Just a thought.