[Feature] add --network flag

addresses #111
. I enabled the --network option on create command which take a string value
. created new struct networkConfig which include ID of the network and the EndpointSettings builder coupled with it, the builder take containerName as parameter
. As it's not allowed to create container with multiple network, I settled the connection to the bridge network just before starting the container assuming that networkConfig is defined
That's all done so far, probably there is some point to improve, any feedbacks would be appreciated

make sense. indeed deleting cluster will not delete the network at all

@andyz-dev I just pushed the update.
About deletion concern, it should not affect what networks the user has created

Tested with the following command sequence:
$ docker network create mynet
$ k3d create -n test --network mynet
Using 'docker network ls'  I found that the default network 'k3d-test' is still created.
Should we not creating the default network when the '--network' option is provided?

Tested with the following command sequence:
$ docker network create mynet
$ k3d create -n test --network mynet
Using 'docker network ls' I found that the default network 'k3d-test' is still created.
Should we not creating the default network when the '--network' option is provided?

I can make the updates but I'm not really sure if it's needed. I Thought that this network is used somewhere else, as it has been labelled "app=k3d" and more
Do you really think the network should not be created in this specific case @andyz-dev ?

@malaDev  Did the following command sequence work for you?  I got the following error message.
azhou@Andys-MBP:~/projs/review/k3d$ docker network create xyz
c9ff3126294764daa6ae8b2a9885de8f61bcab0b9d583ac6dd8ffc092b09e55d
azhou@Andys-MBP:~/projs/review/k3d$ ./bin/k3d create -n test --network xyz
INFO[0000] Created docker volume  k3d-test-images
INFO[0000] Creating cluster [test]
INFO[0000] Creating server using docker.io/rancher/k3s:v1.0.0...
INFO[0000] Pulling image docker.io/rancher/k3s:v1.0.0...
INFO[0009] Removing cluster [test]
INFO[0009] ...Removing server
INFO[0009] ...Removing docker image volume
INFO[0009] Removed cluster [test]
FATA[0009]  Couldn't create container k3d-test-server
Error response from daemon: network k3d-test not found

@malaDev  Did the following command sequence work for you?  I got the following error message.
azhou@Andys-MBP:~/projs/review/k3d$ docker network create xyz
c9ff3126294764daa6ae8b2a9885de8f61bcab0b9d583ac6dd8ffc092b09e55d
azhou@Andys-MBP:~/projs/review/k3d$ ./bin/k3d create -n test --network xyz
INFO[0000] Created docker volume  k3d-test-images
INFO[0000] Creating cluster [test]
INFO[0000] Creating server using docker.io/rancher/k3s:v1.0.0...
INFO[0000] Pulling image docker.io/rancher/k3s:v1.0.0...
INFO[0009] Removing cluster [test]
INFO[0009] ...Removing server
INFO[0009] ...Removing docker image volume
INFO[0009] Removed cluster [test]
FATA[0009]  Couldn't create container k3d-test-server
Error response from daemon: network k3d-test not found

seems there is another creation statement during removal. is it normal?

seems there is another creation statement during removal. is it normal?

It seems to me the creation failed because k3d-test network was not found. But the creation should not require it since the --network option is provided.  Is possible to fix this case?

@andyz-dev I'm not sure to get what you are expecting here

@andyz-dev are you saying that k3d should create the network if it's missing?

Closing this due to inactivity and in favor of k3d v3 :heavy_check_mark:


360 Video Resolution Bug in Chrome.

Changing to fullscreen and back in chrome causes the resolution of the non-fullscreen video to distort.
You can see this in the video example, https://pannellum.org/documentation/examples/video/

The issue is that Chrome reports the wrong size for the viewer when it is checked in the fullscreenchange event handler after existing full screen. I'd argue that this is a browser bug, but I was able to work around it using the relatively new ResizeObserver interface and fix this in 44e878f30763a51cbbbf5ab4c5b97fa4c3b2dfd9.

Awesome! That was a crazy fast fix, thanks.


Released version 2.2.1

New Version contains following fixes

Fix Pods with latest versions to avoid carthage update crash
Implementation to capture all crash scenarios

Coverage decreased (-0.2%) to 82.045% when pulling 3803a7fcc402968d56ed7598a8cc9002c8a216f5 on development into c36ccc38e3a0c38ba891eca3f5e9517b4346ef28 on master.

Coverage increased (+0.5%) to 82.759% when pulling 8f0e685befe045131975e5fe67d54bd75d3125f9 on development into c36ccc38e3a0c38ba891eca3f5e9517b4346ef28 on master.


Define a main file and only compile the main file.

I have searched the issues of this repo and believe that this is not a duplicate.
I have searched the discussions and believe that my question is not already covered.

Feature Request

It would be really nice if it would be possible to define a main file per project.

The lsp would only compile the main file.
The lsp would recompile the main file on changes of #included or #imported files.

At the moment if I write for example a template.typ file it will be compiled on save. This just results in an empty pdf which is useless. In turn I will have to change back to my main.typ and save it manually to see the changes in my template.typ take effect.
I would also suggest that if there is a main.typ in the root folder that it will be recognized as the main file by default. Or is there a more common filename for the main file? Maybe there is also some other mechanism from which the main file could be deteremined.

I'm very new to typst, so please excuse if something about the request is stupid.

Hey i think that there is no solution for this at the moment. Its related to this issue i think: https://github.com/nvarner/typst-lsp/issues/285
my workflow at the moment is just to use the typst cli watch command on your "main" file:
typst watch <filename>

Additionally there seems to be an issue in disabling the lsp to generate a pdf: https://github.com/nvarner/typst-lsp/issues/265
To work around this i just add *.pdf to my .gitignore
Hope that helps

Thanks for the answer and the link to the other issue.
You're tip helps for not generating empty files but it won't recompile the main file on changing the template file.

hm as long as the template is included in "main" it should work.
Could you maybe provide an example setup?

hm as long as the template is included in "main" it should work. Could you maybe provide an example setup?

Ah sorry, my bad. I didn't try it out, I just read the help

$typst watch --help
Watches an input file and recompiles on changes

and inffered from that that it wouldn't recompile on changes of imported files.
Well thanks a lot for the help. I'll close since there is already another issue.


Lifetime of CacheService

The CacheService is added to AutoFac with the Lifetime scope as InstancePerRequest. Should the DI create new MemoryCache service every request & dispose the cached objects ?

The CacheService is a wrapper around the static:
this.memoryCache = MemoryCache.Default;

So in this case it doesn't really matter too much. However, I'll change this to a singleton in the next update.

Done


[RFC]: improve README examples of math/special namespace

Description
This RFC proposes to improve the README examples of the math/special namespace package. Currently, namespace package READMEs only include minimal examples which are not particularly informative and do not showcase namespace functionality.
Historically, namespace examples were not given much effort due to the rapidly changing nature of the project; however, now that development and organization has settled, it would be good to revisit these packages and add proper examples to better communicate to users how to use namespace contents.
For those wanting to contribute to this issue, you can use the following steps:

Browse through the contents of the math/special namespace and identify interesting functions and packages to showcase.
Update the example code block in the "Examples" section of the README for the math/special namespace and open a PR.
When updating the README, examples should follow existing project conventions (e.g., ES5).
Examples may require functionality from other packages to help make examples more interesting.

Related Issues
None.
Questions
No.
Other
No.
Checklist

[X] I have read and understood the Code of Conduct.
[X] Searched for existing issues and pull requests.
[X] The issue name begins with RFC:.

Does math/special have only one function?

@suredesm Good call! It's currently missing a lot of functions because we didn't have time to roll them out yet. It's probably better to revisit this issue once the namespace is more populated.

May I know what types of functionality expected under math/special because already there is math/base/special providing math functionalities. That'd be very helpful to comeup with some ideas for this.


have a bug,redraw the wall,can not add new item

for new update,it has new bug,if i redraw the wall,then i take the design view ,can't add item in it,or it is only center ,can't move to anywhere.please check it,i want to take part in your project ,but my english do not good . i come from china ,thank you.

I tried to reproduce what you're doing to cause the issue, but failed. Can you please describe it in more detail... e.g. as a step-by-step approach maybe with a few screen shots?

Yes I also face the same problem ,if i redraw the wall, from edit floor plan and make changes in the layout. And after that if i add something through add element then in design, the element is not working proper. Please check. Thank You.

@Abhishekamol
I Get it! the bug is solved in earlear,and Ekki is update to it.but forgot to fix a position:
static pointInPolygon(x: number, y: number, corners, startX?: number, startY?: number): boolean {       //startX = startX || 0;       //startY = startY || 0;
fix it , it works ok.

@Abhishekamol
it is in src/core/utils.ts

sorry don't get it... the default initialization should be disabled? really?

Thanks @hsmart . It is working now.


feat(node_framework): Add a task to handle sigint

What ❔
Adds a task that handles SIGINT (aka ctrl+c), propagating the signal to the ZkStackService instead of shutting the node down immediately.
Why ❔

Graceful shutdown.
Replicating the current behavior.

Checklist

[ ] PR title corresponds to the body of PR (we generate changelog entries from PRs).
[ ] Tests for the changes have been added / updated.
[ ] Documentation comments have been added / updated.
[ ] Code has been formatted via zk fmt and zk lint.
[ ] Spellcheck has been run via zk spellcheck.
[ ] Linkcheck has been run via zk linkcheck.

I wonder if in the future we can add different timeouts as well

Basic support of timeouts is already present in the framework (e.g. after sending a stop signal, the service will wait 30 seconds (hardcoded timeout for now) for all the components to stop, then it'll exit forcefully).


update onfocus event

Rationale
The crawler hits this view frequently
Related Pull Requests

Changes

Why not use the input taglib now that it supports onFocus?

I was just trying to un-link the various PRs


removed hardcoded references to "_wixAB3", made that value configurab…

with @adi-kopelevich
made that value configurable in laboratory.properties (part of EX-316)

merged including master rebase at https://github.com/wix/petri/commit/ad54a6783b1752e07667a1c3661ffe1385e67a92


smtp: Add test to match on attachment with md5

Based on the filemd5 test but using smtp attachment instead. The SMTP transaction contains the EICAR file as an attachment and the expected md5 to match used is the standard md5 for the EICAR.
This test currently fails because the smtp/mime-decoder is including the \r\n delimiter in the attachment buffer which causes a different md5 to be calculated compared to the standard EICAR md5.

Thanks for this test.

This test currently fails because the smtp/mime-decoder is including the \r\n delimiter in the attachment buffer which causes a different md5 to be calculated compared to the standard EICAR md5.

Is there a redmine ticket for this ?

And does https://github.com/OISF/suricata/pull/8886 solve the issue ?

@catenacyber redmine ticket is here https://redmine.openinfosecfoundation.org/issues/6023
No, your branch does not fix the issue. EICAR md5 is still calculated to e7e5fa40569514ec442bbdf755d89c2f.

Thank you Thomas
Are you planning to do a Suricata fix as well ?
It looks like the issue is with SMTP supplying data to MIME line after line, so MIME decoder cannot know that it should or should not take the end of line, because it cannot know wether the next line is a boundary...

@catenacyber sorry, I am not planning to do a Suricata fix as well.
I attempted a fix here https://redmine.openinfosecfoundation.org/attachments/2786 but this in resulted in some extra suricata-verify failures https://redmine.openinfosecfoundation.org/attachments/2804
I'm not familiar enough with smtp/mime encoding to understand what to do here. It may be the case sometimes you need to include the delimiter and sometimes you shouldn't.

No worries, I will take it, just wanted to avoid duplicating the work if you were already on it.
Thanks for the test

Replaced by #1265


Fix TestNamespaceTargeting testing.T passing

Prior to this, the parents testing.T was passed, so when a test failed
it would indicate the parent failed, but not which child, which was
confusing.
Please provide a description for what this PR is for.
And to help us figure out who should review this PR, please
put an X in all the areas that this PR affects.
[ ] Configuration Infrastructure
[ ] Docs
[ ] Installation
[ ] Networking
[ ] Performance and Scalability
[ ] Policies and Telemetry
[ ] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure

@howardjohn: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

integ-new-install-k8s-presubmit-tests-master
2c55c551468ba32ebcca26611b0cd3f78a5edfe2
link
/test integ-new-install-k8s-presubmit-tests-master

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

Prior to this, the parents testing.T was passed, so when a test failed
it would indicate the parent failed, but not which child, which was
confusing.

Do we see some failures somewhere?

Yes but it was fixed - https://github.com/istio/installer/pull/335
On Thu, Aug 15, 2019 at 2:43 PM Jianfei Hu<EMAIL_ADDRESS>wrote:

Prior to this, the parents testing.T was passed, so when a test failed
it would indicate the parent failed, but not which child, which was
confusing.
Do we see some failures somewhere?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/istio/istio/pull/16323?email_source=notifications&email_token=AAEYGXIPY3S4BF2ASDWG3TLQEXEZZA5CNFSM4IMB45E2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD4NCSFY#issuecomment-521808151,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAEYGXPB26MEEAHQJOKU3JTQEXEZZANCNFSM4IMB45EQ
.

@howardjohn: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

integ-new-install-k8s-presubmit-tests-master
2c55c551468ba32ebcca26611b0cd3f78a5edfe2
link
/test integ-new-install-k8s-presubmit-tests-master

integ-security-k8s-presubmit-tests-master
2c55c551468ba32ebcca26611b0cd3f78a5edfe2
link
/test integ-security-k8s-presubmit-tests-master

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

@howardjohn: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

integ-new-install-k8s-presubmit-tests-master
2c55c551468ba32ebcca26611b0cd3f78a5edfe2
link
/test integ-new-install-k8s-presubmit-tests-master

integ-security-k8s-presubmit-tests-master
2c55c551468ba32ebcca26611b0cd3f78a5edfe2
link
/test integ-security-k8s-presubmit-tests-master

istio-pilot-multicluster-e2e-master
2c55c551468ba32ebcca26611b0cd3f78a5edfe2
link
/test istio-pilot-multicluster-e2e-master

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

/test all

@howardjohn: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

integ-security-k8s-presubmit-tests-master
2c55c551468ba32ebcca26611b0cd3f78a5edfe2
link
/test integ-security-k8s-presubmit-tests-master

istio-pilot-multicluster-e2e-master
2c55c551468ba32ebcca26611b0cd3f78a5edfe2
link
/test istio-pilot-multicluster-e2e-master

integ-new-install-k8s-presubmit-tests-master
2c55c551468ba32ebcca26611b0cd3f78a5edfe2
link
/test integ-new-install-k8s-presubmit-tests-master

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

@howardjohn: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

istio-pilot-multicluster-e2e-master
2c55c551468ba32ebcca26611b0cd3f78a5edfe2
link
/test istio-pilot-multicluster-e2e-master

integ-new-install-k8s-presubmit-tests-master
2c55c551468ba32ebcca26611b0cd3f78a5edfe2
link
/test integ-new-install-k8s-presubmit-tests-master

istio-racetest-master
2c55c551468ba32ebcca26611b0cd3f78a5edfe2
link
/test istio-racetest-master

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

/retest

@howardjohn: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

integ-new-install-k8s-presubmit-tests-master
2c55c551468ba32ebcca26611b0cd3f78a5edfe2
link
/test integ-new-install-k8s-presubmit-tests-master

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

/cherrypick release-1.3

@Monkeyanator: new pull request created: #16680

In response to this:

/cherrypick release-1.3

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


Getting a 403 error

I'm not sure if this is something that I've changed on the Cloudflare side, or if something changed in the API. I'm getting a 403 error when I try to generate new certs with dehydrated -c.
Traceback (most recent call last):
  File "/opt/dehydrated/hooks/cloudflare/hook.py", line 206, in <module>
    main(sys.argv[1:])
  File "/opt/dehydrated/hooks/cloudflare/hook.py", line 203, in main
    ops[argv[0]](argv[1:])
  File "/opt/dehydrated/hooks/cloudflare/hook.py", line 167, in create_all_txt_records
    create_txt_record(args[i:i+X])
  File "/opt/dehydrated/hooks/cloudflare/hook.py", line 105, in create_txt_record
    zone_id = _get_zone_id(domain)
  File "/opt/dehydrated/hooks/cloudflare/hook.py", line 82, in _get_zone_id
    r.raise_for_status()
  File "/home/user/.conda/envs/acme/lib/python3.6/site-packages/requests/models.py", line 840, in raise_for_status
    raise HTTPError(http_error_msg, response=self)
requests.exceptions.HTTPError: 403 Client Error: Forbidden for url: https://api.cloudflare.com/client/v4/zones?name=domain.tld

Gah! I had changed my Cloudflare login email.
Sorry for this.


Ping keep-alive error.

Hello!
I just updated to the latest pull of Chiadog (v0.7.5-3-gc7f2991) I am getting the following error on the keep-alive monitor"
[   ERROR] --- Failed to ping keep-alive: 'Subview' object has no attribute 'type' (keep_alive_monitor.py:116)
Checked the config.yaml and even used the new one and moved over my settings.  I did not see anything amiss and there was no change.

Hey! Did you re-run ./install.sh after pulling? (see readme)
We added a new Python dependency so it’ll be strictly necessary to install dependencies for upgrades. If that doesn’t help, you can go back to the stable release with git checkout v0.7.5.
cc @Artanicus
P.S.: We should improve guides for docker and move everyone to docker-based installs since it’s more secure and dependencies are managed automatically. This always points to the latest stable release: docker pull ghcr.io/martomi/chiadog:latest

Hello, and thank you for the reply.  I did run the ./install.sh script once I pulled the new version.  I didn't pay attention to the output, but I will do so and report back.
Thanks again for Chiadog, it is invaluable.

I did another fetch and pull, then re-ran  './install.sh'.  It said that all the requirements were satisfied successfully.  The issue remains.  I will try using the docker version and see if it persists.  For now I have disabled the keep-alive ping and everything else seems fine.
I should note that this is on both of my farms.  One is running Ubuntu 22.04 LTS and the other is running Ubuntu 22.10.  Both installs are having the same issue.

I think I know what's going on. The keepalive notifier is getting passed a confuse config instead of a dict, while it still at this revision expects a dict. The keepalive PR would solve it, alternatively I can patch in a quick conversion on the receiving side. On mobile now but will PR that patch as an alternative when I get a chance.
Not sure why type checking or my real world testing didn't catch this, will also try to figure out that flaw and see if anything else is similarly broken. Unittests feed a config forcefully, hence why they could not catch it.

Oddly enough I can't replicate the error in my testing, which explains how it got through. Could you please provide a sanitized copy of your config.yaml @bradyjarvis ? I want to make sure existing configs continue to work as-is, so it might be there's some peculiarity there that triggers the issue. Mind you, there is a clear error in the code, but for some reason my test config is not triggering it:
(this has sections the main HEAD does not yet read, but those should not affect things)
notification_title_prefix: 'fullnode'
log_level: DEBUG

chia_logs:
  file_log_consumer:
    enable: true

keep_alive_monitor:
  enable_remote_ping: true
  ping_url: 'https://hc-ping.com/xxx'
  thresholds:
    WALLET: 120

daily_stats:
  enable: true
  time_of_day: 12

monitored_services:
  - FULL_NODE
  - WALLET
  - FARMER

notifier:
  pushover:
    enable: true
    daily_stats: true
    wallet_events: true
    credentials:
      api_token: xxx
      user_key: 'xxx'

Actually, it might be just stupid enough that the problem only surfaces when the ping is actually attempted, and I never left it running long enough at the current revision to trigger that. shamecube

That fixed it, thanks guys!  Appreciate the quick response and glad to have the keep-alive working again.


add pr template, copied from tidb repo and changed a little

see issue: https://github.com/pingcap/parser/issues/26

Is there something wrong with this pr?

@lnhote Thanks for your contribution and sorry for the late reply.
LGTM


Port Ozaria Resource Hub to CodeCombat

Copied the index and the button files to coco teacher-dashboard. (Onboarding video removed, because it was all about ozaria. Do we have a coco one?)
Copied the required functions from classrooms.ozar.js to classrooms.coco.js (There was only one: setMostRecentClassroomId)
Updated routing for route teachers/resources
Updated fetchClassroomsForTeacher  to  match the ozaria one (receive object type argument { teacherId: string } instead of just teacherId: string), because it was called from a common file (teacherDashboard), and the ozaria one already had the object argument type.
Didn't update anything on the UI, because it seems to be ok for me:

@nwinter Now the coco one extends the ozaria one. It's much DRYer now, but not sure if this the correct way to do.
The email address was moved to utils.
I've copied isPaidTeacher function to coco.
The margins are fixed, screenshot in description updated.


train with background in maskrcnn

I did a comparison test, and I got more accurate results when I tested the background images, although the ap didn't improve.
If there are some bugs, please tell me.
The main references are #169 and #853

Hi @abcxs
Thanks for the PR.

I did a comparison test, and I got more accurate results when I tested the background images, although the ap didn't improve.

Could you share your findings?

Surely, i'm doing detection on buildings. For example, buildings don't appear in the mountains, so there are no training background pictures like this during the training, but the test will encounter these pictures, there will be some false check. When I add background pictures to the training, the false check of similar pictures will be greatly reduced.

I would very much love to see support and more discussion on this. I have a similar issue with negative samples that I like to train to reduce false positives for object detection in aerial images.


Fix clone screening page issues

https://franklin.moleculardevices.com/products/clone-screening#qpix-400-series

[x] On the customer breakthrough carousel, the images should be in a circle, globally
[x] For the products section, the title should be h3: 40px and the subtitle h4: 30px.
Like this -

[x]  For the "Applications & Research" and "Related Products & Services" sections, left align the titles and text coming in the cards, globally.

@alexcarol Yes, you are right.

Another example page for my 1st point - https://franklin.moleculardevices.com/products/3d-biology

More example pages For my last point
https://franklin.moleculardevices.com/products/cellular-imaging-systems
https://franklin.moleculardevices.com/products/3d-biology

https://franklin.moleculardevices.com/products/microplate-readers/multi-mode-readers

https://franklin.moleculardevices.com/products/microplate-readers/absorbance-readers

Could you clarify what you mean by "globally" in the first and last points? Do you have any other examples of pages where this should be applied too (if that's the case)?

@alexcarol This point is not done. Can you please check? The title and the text are still center-aligned for me.

@MeghnaTewari it should be fixed now, could you check again?

fixed now. Thanks.


Revert "[flutter_tools] partial revert of start app change (#68128)"

This reverts commit 767fb98a7b3b75bb6545cd707b9ad5103ae12df7.
Description
This commit appears to have caused a performance regression.  We are going to revert it as a trial to figure out if it did in fact cause the regression or if there is some problem with the test.
Related Issues
https://github.com/flutter/flutter/issues/68316
Tests
Checklist
Before you create this PR, confirm that it meets all requirements listed below by checking the relevant checkboxes ([x]). This will ensure a smooth and quick review process.

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I signed the CLA.
[x] I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
[x] I read the Tree Hygiene wiki page, which explains my responsibilities.
[x] I updated/added relevant documentation (doc comments with ///).
[x] All existing and new tests are passing.
[x] The analyzer (flutter analyze --flutter-repo) does not report any problems on my PR.
[x] I am willing to follow-up on review comments in a timely manner.

Breaking Change
Did any tests fail when you ran them? Please read Handling breaking changes.

[x] No, no existing tests failed, so this is not a breaking change.
[ ] Yes, this is a breaking change. If not, delete the remainder of this section.

[ ] I wrote a design doc: https://flutter.dev/go/template Replace this with a link to your design doc's short link
[ ] I got input from the developer relations team, specifically from: Replace with the names of who gave advice
[ ] I wrote a migration guide: https://flutter.dev/go/breaking-changes-template Replace this with a link to a pull request that adds the migration guide to https://flutter.dev/docs/release/breaking-changes

cc @jonahwilliams

Closed because the source of the problem was found.


Upgrade to egui v0.25.0 and miniquad v0.4

This includes:

@not-fl3's (i.e. your) https://github.com/not-fl3/egui-miniquad/pull/63

It would be much easier to review this PR if that PR were merged first. I don't think anything much I've done here depends on that PR (so feel free to rebase and drop this commit), but I wanted all this to work against the latest miniquad version, so I included your PR because of that. Also, that PR has been open for 6 months now, so I figured why not include it :)

@emilk's https://github.com/not-fl3/egui-miniquad/pull/64

(well, not the exact commit from that PR: I copied over the 2 lines of changes from it, bumped to latest miniquad 4.x alpha, and fixed the 2 broken examples in this crate)

and 2 more commits to update to egui 0.24.1 and 0.25.0.

I've tested a little bit and "things seem to work" - at least the obvious stuff like rendering, mouse and keyboard input handling, etc.
Things I'm not sure about:

The flickering between the color-test gradients that @emilk noted in https://github.com/not-fl3/egui-miniquad/pull/64 is still there. It only appears between two gradients.. is that a big deal? Not for me, but it might be for others.
High DPI screens support. Egui 0.24.0 revamped how pixels per point is calculated and used. I made a best effort attempt to integrate this correctly but I couldn't find any big picture docs and I don't use a high DPI screen so I'm not sure it's done correctly.

And it looks like egui can suggest DPI changes back to the egui backend now as well, based on the user zooming in or something like that? I didn't implement this side of things at all because I don't need it.

So.. it's not a rock solid PR, but I'm raising it anyway to save others doing the same work if they don't care about these limitations.

Give me about a week, I am about to publish miniquad-0.4 without alpha.
(yes, I was about to publish it since forever, but now I really am!)

Thanks for the pointers @emilk - got the points per pixel stuff fixed now. (I suspected it was meant to be that simple, but when I tried it nothing happened, which was because of the next bullet)
@not-fl3 I've also added support for punctuation keys that were missing, most notably plus and equals!
I've also replaced the manual override of pixels per point in the demo example with manually setting zoom factor instead; otherwise changing zoom factor in egui with its built in keyboard bindings will immediately be reset to 1.0 zoom next frame.

With all 3 of those things fixed, you can now zoom the demo example in and out (and reset zoom) as expected with Ctrl/Command +/-/0.
So this PR is now B+ quality, with the only remaining thing being the gap between the gradients (which, again, I can live with).

Btw, a new egui is coming next week too :)

Give me about a week, I am about to publish miniquad-0.4 without alpha.
(yes, I was about to publish it since forever, but now I really am!)

commented 3 weeks ago

anyway, miniquad is released, no more github/crates differences nor -alpha versions!
Most changes are internal for metal backend, so this PR should(mostly) just work.

Yay!
I've updated this PR to target miniquad 0.4.0 now, so: time to hit merge?
(I know there's egui 0.26 out already, but maybe let's just get this merged first)


Fix ClassCastException and refactor strategy of obtaining current authn

Fixes #112

This definition really needed to be moved out of the child servlet context definition as it was not available to the parent ctx components. How else? This file could serve for future common parent ctx bean defs.
Cheers,
D.

Ah, got it. Excellent. On to the next release and beyond :)


Fix bad uses of mutex causing deadlocks

When I translated old-style uses of Mutex I did it wrong in some cases.
This fixes a few cases I found that causes the editor to freeze on saving a scene.

Thanks!


Unwrap cube command (Suggestion)

It would be cool to add a command that could "unwrap" a cube,
It would unwrap it by putting a portal on the edge that changes gravity that then leads to another edge, this will make it feel like the cube is flattened.

The commands syntax could be like this "/portal unwrapcube x1 y1 z1 x2 y2 z2 height"
The height would be the height of each portal that is made.

It's being considered.

The feature has been added, will be in 2.2.0

Thank you!


[Questions] - How to send error level logs in crashlytics ?

Hi, I would like to know if it is possible and how to implement this fucntionality:
Every time I use logger.e for error level, I would like the logger to send this information in crashlytics so I can get report about it.
Right now I don't get any information about logger.e when app is in production env and in clients devices.
I can see console output only when I develop

I too am interested see documentation added for how I can send my logs to sentry.io

It looks like we can use the LogOutput class to send events to a network target.

Here it’s also mentioned that you can log events anywhere e.g file, firebase or logcat etc.
I hope it helps.

I've using different approach use Logger with Firebase Crashlytics. Maybe this example give an idea for you.
Extend Default Logger (mm_logger.dart)
I only override error method because I just want to log errors in Firebase Crashlytics you can override another methods if you want.
class MMLogger extends Logger {
  /// Create a new instance of MMLogger.
  ///
  /// You can provide a custom [printer], [filter] and [output]. Otherwise the
  /// defaults: [PrettyPrinter], [DevelopmentFilter] and [ConsoleOutput] will be
  /// used.
  MMLogger({
    LogFilter? filter,
    LogPrinter? printer,
    LogOutput? output,
    Level? level,
  }) : super(filter: filter, printer: printer, output: output, level: level);

  @override
  void e(dynamic message, [dynamic error, StackTrace? stackTrace]) {
    super.e(message, error, stackTrace);
    FirebaseCrashlytics.instance.recordError(error, stackTrace,
        // reason: 'a fatal error',
        // Pass in 'fatal' argument
        fatal: true);
  }
}

Create instance from MMLogger instead of Default Logger (main.dart):
final logger = MMLogger(
  filter: null, // Use the default LogFilter (-> only log in debug mode)
  printer: PrettyPrinter(), // Use the PrettyPrinter to format and print log
  output: null, // Use the default LogOutput (-> send everything to console)
);


Greenkeeper/@types/node 11.12.3

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Add support for sorting keys that are arrays

The String method of reflect.Value always returns the same result for two different arrays, regardless of their content (e.g. [4]int always produces "<[4]int Value>"). This causes inconsistent sorting for maps whose keys are arrays.

Coverage decreased (-0.63%) to 96.25% when pulling 59a3ca0a768183d38b51c2b99687b8c48ba0771a on Anaminus:master into 1aaf839fb07e099361e445273993ccd9adc21b07 on davecgh:master.

Coverage increased (+0.04%) to 96.92% when pulling 2bab0bbc40b9293b0d4bd4d8c1fed03705b8763f on Anaminus:master into 1aaf839fb07e099361e445273993ccd9adc21b07 on davecgh:master.

Thanks for the PR.  I'll review it this weekend.

I've squashed this down to a single commit (a4b53deb9bf78b69198092ddafa308d3be2bd002) and merged it.


Build regression tests for ClueWeb data

We don't have regression tests for ClueWeb data yet...

See PR #273


Topic Builder matching one in spine-client

This PR adds a TopicBuilder returned by TopicFactory#select() to construct subscription topics in a flexible manner.
The TopicBuilder API matches that of a QueryBuilder introduced in PR #21.
An example of building a Topic:
requestFactory.topic()
    .select(CUSTOMER_TYPE)
    .byIds(getWestCoastCustomerIds())
    .withMask(["name", "address", "email"])
    .where([
        ColumnFilters.eq("type", "permanent"),
        ColumnFilters.eq("discountPercent", 10),
        ColumnFilters.eq("companySize", Company.Size.SMALL)
    ])
    .build()

@armiol, PTAL.


Add scorecard to ci

Issue link

Closes #205
What changes have been made

Addition of operator-sdk scorecard check to the CI and make target.
I've had to remove a failing test, olm-crds-have-resources-test from the CI. This appears to be a known bug
Verification steps

Look at CI run
Checks

[ ] I've made sure the tests are passing.
Testing Strategy

[ ] Unit tests
[ ] Manual tests
[ ] Testing is not required for this change

@anishasthana can you LGTM this, you're requested changes have been implemented

/lgtm


[Feature Request] Graph View

Just like the graph view on Obsidian as shown below, there should be a way to export it (and even if it isn't dynamic), it will be a nice touch to the digital garden.
Thanks.

Agreed, this is one of many features I want to implement. It's on my todo list, but as I'm only one person, and doing this in my free time, it will probably be a while until its implemented. Thanks for the suggestion though!

Hey! I found this repo of someone doing something similar, but with Jekyll.  It has side bar support and graph view. For anyone needing that now, they can change.
But @oleeskild, this could help get an idea of how to do it. It's easier to copy ideas that figure it out yourself, lol. On my side, I love the simplicity of what you've done (Plugin instead of having to do outside cmd commits, easy template change through Obsidian's templates, etc), so I'll wait and even help if you need any.
Keep up the good work!
https://github.com/maximevaillancourt/digital-garden-jekyll-template

I've just released a new version that adds a local graph and backlinks. You'll need the latest version of both the plugin and template.
The template can be updated via Obsidian by going to "Settings > Digital Garden > Site Template > Manage site template > Create PR"
Any feedback on what works and what you'd like to be improved would be super helpful 😄


WIP!! feat($location): basic matrix parameter support

This is a WIP to provide a focus for discussion around how to help ensure that in hybrid (ngUpgrade) apps $location can work with the Angular router without URL collision issues.
The backstory is that $location tries to encode/decode matrix parameter delimiters causing the URL to be become invalid.

This PR creates a new Location type called LocationMatrixUrl. This understands how to parse and compose URLs that contain matrix parameters.  This is very much a WIP - the real implementation would involve some significant refactoring to reuse code from the other Location types. But for simplicity I just copied the code over.
The new specs are basically a direct copy from the New Url -> encoding section, which tests the encoding of the LocationHtml5Url, with the results tweaked to reflect the new behaviour of the LocationMatrixUrl approach.


HTTP redirects breaks libvirt_volume.source

System Information
Linux distribution
$ lsb_release -a
No LSB modules are available.
Distributor ID: Ubuntu
Description:    Ubuntu 22.04.1 LTS
Release:        22.04
Codename:       jammy

Terraform version
$ terraform -v
Terraform v1.3.7
on linux_amd64
+ provider registry.terraform.io/dmacvicar/libvirt v0.7.1

Description of Issue/Question
I've been trying to have terraform create some archboxes using the qcow2 images provided by vagrant - these URLs redirect a few times and ultimately end up in some S3 bucket at AWS I think it was.
Setup
terraform {
  required_providers {
    libvirt = {
      source = "dmacvicar/libvirt"
    }
  }
}

provider "libvirt" {
}

resource "libvirt_volume" "archlinux-qcow2" {
  name   = "archlinux-qcow2"
  pool   = "default"
  source = "https://app.vagrantup.com/archlinux/boxes/archlinux/versions/20230104.116125/providers/libvirt.box"
  format = "qcow2"
}

Steps to Reproduce Issue
$ terraform apply

Result
libvirt_volume.archlinux-qcow2: Creating...
╷
│ Error: error while getting Content-Length of "https://app.vagrantup.com/archlinux/boxes/archlinux/versions/20230104.116125/providers/libvirt.box": strconv.Atoi: parsing "": invalid syntax - got
│
│   with libvirt_volume.archlinux-qcow2,
│   on main.tf line 15, in resource "libvirt_volume" "archlinux-qcow2":
│   15: resource "libvirt_volume" "archlinux-qcow2" {
│
╵

Looking at libvirt/volume_image.go#L94
response, err := http.Head(i.url.String())
if err != nil {
    return 0, err
}

I think I got the title wrong, as the documentation clearly states that http.Head will follow up to 10 redirects.
As far as I can see, the mentioned URL does 4 redirects:
$ curl -v -L https://app.vagrantup.com/archlinux/boxes/archlinux/versions/20230104.116125/providers/libvirt.box 2> >(grep "HTTP/1.1")
> GET /archlinux/boxes/archlinux/versions/20230104.116125/providers/libvirt.box HTTP/1.1
< HTTP/1.1 302 Found
> GET /archlinux/boxes/archlinux/versions/20230104.116125/providers/download/libvirt.box HTTP/1.1
< HTTP/1.1 302 Found
> GET /v1/object/eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJrZXkiOiJib3hlcy82NGNjZGNiMy01YzZkLTQ3YWUtOWNlOS1lNzgyMzcxZTgwM2QiLCJtb2RlIjoiciIsImV4cGlyZSI6MTY3MzQ3Nzc5NX0.z34E8jDTIKLZH7q8vjrAmKUoSXmKGs38Hdh90Tb4iGo HTTP/1.1
< HTTP/1.1 307 Temporary Redirect
> GET /archivist/boxes/64ccdcb3-5c6d-47ae-9ce9-e782371e803d?X-Amz-Algorithm=AWS4-HMAC-SHA256&X-Amz-Credential=ASIA6NDPRW4BQPUMO3GT%2F20230111%2Fus-east-1%2Fs3%2Faws4_request&X-Amz-Date=20230111T224136Z&X-Amz-Expires=900&X-Amz-Security-Token=FwoGZXIvYXdzEGgaDC0Fikpyw%2FSOkdJ5aSK3ARFecWXHJqqZ95ZgyOAIpFdwqoKIcqqlZms3kTIA5ehUdJGmIQ6O36Z2nhTKgbIWMIP9If2ZQl577ULIAjzWL3J4%2BJDK89JQL7hM%2Fve7F0r0z6we0l%2F3VCQdu51KCcNAXBTJ7DCXjSHsCJ%2FFHYWoLcGljkPR5bHuOj6xCZoikuNr2qICjGxqwwCOEtkHMQnMBSqn%2BG95VHMvEr%2FwR7rRgeJXjATg2NavYe2%2BAkqJZXISfYjiTet9Aijm9PydBjItrG2a5h916gwFhPvFi617WYQifF4XYAQSQTmfPRvsE%2BgZ570vCGvnhVaOn4uo&X-Amz-SignedHeaders=host&X-Amz-Signature=2893c009b97f9ce63ba02cd4f038b3397a9c06b0db0ca7969a3de55f08f17281 HTTP/1.1
< HTTP/1.1 200 OK

And this last 200 OK does indeed include a Content-Length header.
< HTTP/1.1 200 OK
< Content-Type: binary/octet-stream
< Content-Length: 495197110
< Connection: keep-alive
< x-amz-id-2: t0xymyPQVNGUC0lh0/Jc8cgKGNRLeKt7uuFB9kCa10hyDR1nOUcA5CGHojTATyBEK/lKaP7YBwQ=
< x-amz-request-id: JWTQNTXN4EXPNPSD
< Date: Wed, 11 Jan 2023 22:41:10 GMT
< Last-Modified: Wed, 04 Jan 2023 17:31:59 GMT
< ETag: "d1b84cae0adfe7f02930f0d6a1e90f4e"
< Accept-Ranges: bytes
< Server: AmazonS3
< X-Cache: Miss from cloudfront
< Via: 1.1 c2c75215aa2ab067e062055fa68a3fde.cloudfront.net (CloudFront)
< X-Amz-Cf-Pop: CPH50-C1
< X-Amz-Cf-Id: YxSxrJab2i2VDU3BDowXeBgii0zCDTYVA8VYMDA8EzFVbbORtHM0xw==

So I'm not quite sure whats going on

Think same issue here. Since yesterday evening, on multiple systems, without a local code change (i rebuild terraform and providers, with terraform init). My scripts keep hanging:
libvirt_volume.w-base-vol[6]: Creating...
libvirt_volume.second_disk[3]: Creating...
libvirt_volume.second_disk[4]: Creating...
libvirt_volume.second_disk[5]: Creating...
libvirt_volume.second_disk[2]: Creating...
libvirt_volume.second_disk[5]: Creation complete after 0s [id=/vm/w6.k106-second-disk]
libvirt_volume.w-base-vol[0]: Creating...
libvirt_volume.second_disk[6]: Still creating... [10s elapsed]
libvirt_volume.w-base-vol[5]: Still creating... [10s elapsed]
libvirt_volume.second_disk[0]: Still creating... [10s elapsed]
libvirt_volume.w-base-vol[6]: Still creating... [10s elapsed]
libvirt_volume.second_disk[3]: Still creating... [10s elapsed]
libvirt_volume.second_disk[4]: Still creating... [10s elapsed]
libvirt_volume.second_disk[1]: Still creating... [10s elapse
.....
libvirt_volume.second_disk[3]: Still creating... [17m10s elapsed]
libvirt_volume.w-base-vol[6]: Still creating... [17m10s elapsed]
libvirt_volume.second_disk[2]: Still creating... [17m10s elapsed]
libvirt_volume.second_disk[1]: Still creating... [17m10s elapsed]
libvirt_volume.second_disk[6]: Still creating... [17m10s elapsed]
libvirt_volume.w-base-vol[0]: Still creating... [17m10s elapsed]
Without succeeding

@n-able-consulting, while annoying - don't think its the same issue - when using the arch box vagrant images, terraform fails instantly;
Terraform will perform the following actions:

  # libvirt_volume.archlinux-qcow2 will be created
  + resource "libvirt_volume" "archlinux-qcow2" {
      + format = "qcow2"
      + id     = (known after apply)
      + name   = "archlinux-qcow2"
      + pool   = "default"
      + size   = (known after apply)
      + source = "https://app.vagrantup.com/archlinux/boxes/archlinux/versions/20230104.116125/providers/libvirt.box"
    }

Plan: 1 to add, 0 to change, 0 to destroy.

Do you want to perform these actions?
  Terraform will perform the actions described above.
  Only 'yes' will be accepted to approve.

  Enter a value: yes

libvirt_volume.archlinux-qcow2: Creating...
╷
│ Error: error while getting Content-Length of "https://app.vagrantup.com/archlinux/boxes/archlinux/versions/20230104.116125/providers/libvirt.box": strconv.Atoi: parsing "": invalid syntax - got
│
│   with libvirt_volume.archlinux-qcow2,
│   on main.tf line 15, in resource "libvirt_volume" "archlinux-qcow2":
│   15: resource "libvirt_volume" "archlinux-qcow2" {
│
╵

Digging around with mitmproxy, the problem lies in the vagrantup.com web service, which directs users to the s3 buckets. When presented with a HEAD request, it eventually sends a 200 OK and includes a "Location: " header. It should of course send a 307 TemporaryRedirect together with the "Location: " header - which it does when presented with a GET request.
I don't think there's much for this provider to do about that
Furthermore, the builds provided by vagrantup are actually gzipped tarballs and need to be extracted in order for libvirt to use so - this is going essentially nowhere the way I wanted it to :)
Feel free to close this issue

Hi @fasmide - could you close the issue yourself? I know it's been long time but I would appreciate it.

absolutely :)


SSLPeerUnverifiedException when migrating from Brixton.M3 to Brixton.RELEASE

Using spring-boot 1.3.5 and org.springframework.cloud
when using zuul proxy
with version: Brixton.M3
everything works fine.
When we upgraded to Brixton.RELEASE
we get the following exception
javax.net.ssl.SSLPeerUnverifiedException: Host name 'localhost' does not match the certificate subject provided by the peer (CN=test, OU=test County, O=test, L=Miami, ST=Florida, C=us)
What do you suggest we do in order to migrate the version.
Thanks

Can you provide some steps to reproduce the issue? Maybe a sample project?

I'm encountering the same problem.
I can't share the service though...
I get:
14:36:14.652 [main] DEBUG org.apache.http.wire - http-outgoing-5 << "{"timestamp":1464957374647,"status":500,"error":"Internal Server Error","exception":"javax.net.ssl.SSLPeerUnverifiedException","message":"Host name 'localhost' does not match the certificate subject provided by the peer (CN=Tom Akehurst, OU=Unknown, O=Unknown, L=Unknown, ST=Unknown, C=Unknown)"}"
The setup is pretty simple, I have a zuul service that just relays the request to a wiremock instance via https.
This is the important part of the test:
    @Rule
    public TestCasePrinterRule pr = new TestCasePrinterRule(System.out);
    @Rule
    public WireMockRule wireMockRule = new WireMockRule(wireMockConfig()
            .port(7777)
            .httpsPort(Integer.valueOf(System.getProperty("targetPort", "48443"))));
    private String serverUrl;

    @Before
    public void before() throws IOException {
        System.out.println("Wiremock Port: " + System.getProperty("targetPort", "48443"));
        serverUrl = "https://" + hostname + ":8443/";
    }

    @Test
    public void jsonContentIsNotTouched() throws IOException {
        String content = getFile("json-content.json");
        String uri = "/userservice-userservice-impl/my/json/test";
        stubFor(post(urlEqualTo(uri))
                .willReturn(aResponse()
                        .withStatus(200)
                        .withHeader("Content-Type", "application/json")
                        .withBody("{message:'Okily dokily!'}")));

        RestResponse<byte[]> response = simpleRestClient.post(serverUrl + uri)
                .entity(new StringEntity(content, org.apache.http.entity.ContentType.create((MediaType.APPLICATION_JSON))))
                .execute();
        assertThat(response.getStatus(), Matchers.is(200));

        verify(postRequestedFor(urlMatching(uri))
                .withRequestBody(equalTo(content))
                .withHeader("Content-Type", matching("application/json")));
    }

Does this help?

Tom Akehurst is the author of Wiremock. It looks like the certificate is his. Maybe you need to ask him how to configure the server or client to avoid trust issues?

Hmm, well, this is a self signed certificate which is not unusual at least in development environments and after Brixton.M3 we are getting these errors from the zuul based spring cloud  service. When you do the calls yourself you can usually just configure a HostnameVerifier for the ConnectionFactory which ignores the hostname, but I wasn't successful yet to find a way to configure this for the http client zuul/ribbon uses.
Maybe it's a zuul problem because the zuul version changed from .M3 to .RELEASE. I haven't investigated this more, but how should we know where to fix this? I would hope that the spring-cloud team or the netflix guys can shed some light on this issue :-)

It looks like you have some experience solving this kind of problem, so maybe you can dig a bit more to find out how to configure the HTTP client?

Hmmm, I may be able to spend a few hours, but honestly I currently have more work than time already ;-)

Was this ever resolved? The HTTPClient object in RibbonLoadBalancingHttpClient is a private final field that is initialised inside the class, theres no access to even call the setDefaultHostnameVerifier override that would allow to bypass the verification check - the only way I can see to do it would be to create your own RibbonLoadBalancingClient that allows you to set the client, but that seems like a horrible hack and will add a lot of redundant code

The issue is still open, and still marked as "help wanted". I know that @ryanjbaxter is looking at updating the way the client gets created, so maybe he could take a look when he gets to it.

@ali2992 9eed2490288a2d218d80e7498de861d851ef9c19 just introduced a constructor that allows you to pass an HttpClient object.

I think @spencergibb's solution should work based on the code he introduced.  Can we close this issue then?  If not I think we should move it to Spring Cloud Netflix instead of here.

Please open a new issue here if you think this is still a problem.

Is there a quick solution to this issue when using "the whole stack", meaning Eureka, Ribbon and Zuul within Spring Boot?
I believe that adding:
ribbon:
CustomSSLSocketFactoryClassName: com.netflix.http4.ssl.AcceptAllSocketFactory
To the application.yml file does it, but I'm wondering if people have any better ideas.


Feature: Adds parallel GitHub actions for e2e tests to get rid of flaky tests

[x] I added CHANGELOG entry for this change.
[x] Change is not relevant to the end user.

Changes
I believe quite a lot of our e2e tests simply hit a deadline due to -timeout 10m
Now, we could increase that, but I personally dislike builds > 10 min :)
This feature implements a pretty cool golang bin via gotesplit - which we fetch via scripts/gotesplit.sh. We place it in the /tmp/bin just like the Shellcheck bin.
What this tool does is it splits the tests based on the amount of parallelism you want. At the moment it does 0 on local dev work, which is exactly the same behavior as it already is. Mostly because our own computers do not benefit from the parallelism in our "runner", because that's our PC. We already do parallelism in the tests itself, which is perfect for that.
However for cloud based runners, it helps to have n-amount of runners, as it does not scale well vertically, but actually does it pretty good horizontally.
An example from my fork:

We increase the total amount of build minutes by roughly 40%, but we reduce the build time by 30-40% and actually remove quite a lot of flaky tests. Probably due to lack of GitHub runners and/or busy servers(?).
It can be further improved to actually cache the docker build stage so that each parallel testing job has access to the freshly prepared container image of Thanos.
Verification
Tested it locally and on my fork.

p.s. I have no idea why I had to format the docs, but.. yea it's included as well!

I wonder why this still has
Thanos end-to-end tests Expected — Waiting for status to be reported

Even though they have been removed :/
Failing docs check seems related to your changes. Why there are so many whitespace changes? :smile:

Regarding docs, if you ran make lint before committing, it clashes with make check-docs - both format in different ways but currently check-docs is the authoritative command since that's used by CI. See my forgotten PR here: https://github.com/thanos-io/thanos/pull/4662

@GiedriusS
The docs are actually better, but there is some quirk that the check-docs does not clear white noise.
I've talked with @matej-g  and we are going to address this in https://github.com/thanos-io/thanos/pull/4662 were we reached consensus of how it should look like. That gets fixed tomorrow.
In other words, the changed docs files are okay for now, even though it should not have been part of this PR. I think when we merged the other PR, it should have 0 diff anymore on it.
As for Bingo. I was implementing this, but hit this issue that gets fixed as well: https://github.com/bwplotka/bingo/pull/106

[ ] wait for #4662
[ ] wait for Bingo implementation

Indeed it looks like you are correct - Thanos end-to-end tests is specified in the "checks that need to pass before merging" page and it probably waits forever for those results even though they will never come in this pull request. So, I will just adjust the list of those checks after merging this.

@wiardvanrij awesome work & a clever hack :rocket:


Why:
Closes [issue link]

What's being changed:

Check off the following:

[x] I have reviewed my changes in staging (look for the latest deployment event in your pull request's timeline, then click View deployment).
[x] For content changes, I have completed the self-review checklist.

Writer impact (This section is for GitHub staff members only):

[ ] This pull request impacts the contribution experience

[ ] I have added the 'writer impact' label
[ ] I have added a description and/or a video demo of the changes below (e.g. a "before and after video")

Ok

@kernc
Thanks so much for opening an issue! I'll triage this for the team to take a look :eyes:


Vagrant-Workflow-tests fail on release 2.3 on win7

I downloaded the release and did the git clone for tests and these are the errors I'm getting - is there a way I can build native ruby to avoid these native gem builds failing?
Failures:

usage scenarios developing top-level cookbooks installs gem dependencies via bundle install
Failure/Error: cmd_succeeds("cd #{APP_COOKBOOK_DIR} && bundle install")
expected true
got false
./spec/helpers.rb:33:in `cmd_succeeds'
./spec/acceptance/usage_scenarios_spec.rb:46:in `block (3 levels) in <top (required)>'

usage scenarios developing top-level cookbooks runs the unit tests via rake test
Failure/Error: cmd_succeeds("cd #{APP_COOKBOOK_DIR} && rake test")
expected true
got false
./spec/helpers.rb:33:in `cmd_succeeds'
./spec/acceptance/usage_scenarios_spec.rb:49:in `block (3 levels) in <top (required)>'

usage scenarios developing top-level cookbooks runs the integration tests via rake integration
Failure/Error: cmd_succeeds("cd #{APP_COOKBOOK_DIR} && rake integration")
expected true
got false
./spec/helpers.rb:33:in `cmd_succeeds'
./spec/acceptance/usage_scenarios_spec.rb:52:in `block (3 levels) in <top (required)>'

Finished in 13 minutes 52 seconds (files took 0.33793 seconds to load)
15 examples, 3 failures
Failed examples:
rspec ./spec/acceptance/usage_scenarios_spec.rb:45 # usage scenarios developing top-level cookbooks inst alls gem dependencies via bundle install
rspec ./spec/acceptance/usage_scenarios_spec.rb:48 # usage scenarios developing top-level cookbooks runs  the unit tests via rake test
rspec ./spec/acceptance/usage_scenarios_spec.rb:51 # usage scenarios developing top-level cookbooks runs  the integration tests via rake integration
W:\repo\vagrant-workflow-tests>
also is there a way to do this for the 'rake build' from source? I love the tail on acceptance.log, and was wondering if there is one for the rake build in the source file?

@paul42 hard to say what's going wrong here. Can you provide the contents of the acceptance.log (e.g. via gist) so I can have a further look?
And are you using powershell or plain cmd shell?
Not sure if I got your last question right, but what I'm usually doing is this:

git clone https://github.com/tknerr/bills-kitchen.git
git checkout master
rake build
(going for a coffee)
rake acceptance
(...and another coffee)

No, for the rake build there is not a more detailed logfile... :-/

P.S.: you can also use the Ruby DevPack for building bill's kitchen:
https://github.com/tknerr/ruby-devpack/releases/tag/v0.3

I'm going to close this issue because I made a giant mistake and did rake acceptance against the wrong folder, I had to terminate the batch for a rake build and in that folder I ran the acceptance test, mistakenly thinking it had finished. I'm still not successfully getting past rake build

@paul42 let me know how it goes and please reoepen if you have further problems with the acceptance tests
Btw this is what rake acceptance does:
https://github.com/tknerr/bills-kitchen/blob/master/Rakefile#L65-69
And same here: the first time after a rake build of the bills-kitchen devpack it will take quite long for the acceptance tests to finish. If you run them a second time they will be much faster since all the vagrant baseboxes are already downloaded, bundled gems already installed, and much more will be already cached. (see target/build/home/vagrant.d/boxes and target/build/home/vagrant.d/cache for example)


[Core] L2 chain icons logic

Hey @flypaper0 , so when user are choosing a chain ,  chain selection list , can we show L2 with L2 chain logo?

In this case , for arbitrum , can we just show arbitrum logo?


[Q] summarize() function explanation

When a summarize() function with 1day interval and sum aggregation function is used like in example below:
summarize(some.metric, '1d', 'sum')
so everything is OK if data range interval is set up to 30 days but for a longer intervals an Y axis values are changed in strange way.
See attached pictures for better imagination:
30 days
31 days
Is this  expected behaviour?

This is most likely related to your carbon storage-schemas.conf and storage-aggregation.conf - your storage-schemas.conf tells carbon to keep a higher resolution for just 30 days, and a lower resolution for a longer period, and your storage-aggregation.conf tells carbon to average values when copying from the higher-resolution to lower-resolution series. Then, when you tell graphite-web to go back further than 30 days, it skips the higher resolution series completely, and only uses the lower-resolution series (because it wants to use a single resolution variant to cover the entire period).
If you configure your storage-aggregation.conf to aggregate to the lower resolution using "sum" for this series, that should  fix this problem. (These settings automatically apply only when the whisper storage file is initially created for a series, so you'll have to manually apply the change to the whisper file using the whisper utilities, or blow away the data and allow it to be re-created).

Thank you for explanation. I will check the configuration files.


Visual Studio 2015 TypeScript won't transpile

TypeScript Version:  2.0.3
This is my tsconfig file:
{
  "compileOnSave": true,
  "compilerOptions": {
    "target": "es5",
    "module": "commonjs",
    "moduleResolution": "node",
    "sourceMap": true,
    "emitDecoratorMetadata": true,
    "experimentalDecorators": true,
    "removeComments": false,
    "noImplicitAny": false
  },
  "exclude": [
     "node_modules"
 ]
}

I have a project in Visual Studio 2015 Update 3 with TypeScript version 2.0.3 and none of my .ts files are compiling on save anymore. I can run the command tsc (file path) which will transpile the .ts file into a .js, but it does not create a map, which makes it so I cannot debug in Chrome. Simply saving a .ts file does not have any effect.
The machine I am working on is Microsoft Windows Server 2012 as User A. Any Visual Studio project I open has the same symptoms and will not compile TypeScript, however, Users B and C on the same machine are not experiencing any issues (also in Visual Studio 2015 Update 3).
It appears to me that it may somehow be related to the user account I am using - the evidence being that other users on the same machine, using the same Visual Studio are not having the same problems I am.
Is there something that is user specific that could have been changed on my account that would cause this type of behavior and not have been changed for other users?
Thank you in advance for any help

this could possibly be related to https://github.com/Microsoft/TypeScript/issues/11110. do you see other features broken?

What is your global tsc version that you run from command line? That should not be related to VS though.

@mhegazy I will do what that post suggests and get back to you, thanks!
@zhengbli tsc -v Version 2.0.3 as returned from the Package Manager Console

@mhegazy I will do what that post suggests and get back to you, thanks!

Wait.. there is nothing suggested in this post yet. we originally thought it was a setup issue. but it is not.

Do you see other features broken as well? colorization, completion, find all references, nav bar, etc..?

as far as I can tell, no, it is just when I save a .ts file nothing happens. well that's not true, on the bottom left it says "Item Saved".
also, in my project properties there is no typescript tab. not sure if that is related.
other than that, I do not see anything else glaring at me, although I would be happy to check something specific if you would like

in your help\about what version of TS do you see?

<IP_ADDRESS> and I changed my csproj file to be 2.0 for typescript

what version is your OS specifically? is it R2?
also what version do you see onC:\Program Files (x86)\Microsoft Visual Studio 14.0\Common7\IDE\CommonExtensions\Microsoft\TypeScript\Microsoft.VisualStudio.LanguageServices.TypeScript.dll ?

And do you have HKEY_CURRENT_USER\SOFTWARE\Microsoft\VisualStudio\14.0\TypeScriptLanguageService\EnableDevMode by any chance?

correct, Windows Server 2012 R2 Standard
C:\Program Files (x86)\Microsoft Visual Studio 14.0\Common7\IDE\CommonExtensions\Microsoft\TypeScript\Microsoft.VisualStudio.LanguageServices.TypeScript.dll -> properties -> details -> file version = 14.30921.1.0
I do not. HKEY_CURRENT_USER\SOFTWARE\Microsoft\VisualStudio\14.0\ does not even contain a folder named TypeScriptLanguageService

I can tell you that I was wrong, the other users on the same machine are experiencing the same issue. I am going to try and uninstalling 2.0 so that they can continue to work.
I will say that the reason I upgraded to 2.0 originally is because I was having these symptoms in my project even with 1.8

I will say that the reason I upgraded to 2.0 originally is because I was having these symptoms in my project even with 1.8

this is a different issue then. does this happen for different projects? or just one? does compile on save work for a new project ? if not can you share your project?

it happens for all projects on this machine for just me.
compile on save does not work on a new project.
what do you mean share my project?

the main problem manifests itself in that the .map files are not created, so I don't have any typescript files in my google chrome, so I can't debug it

also, now I my Visual Studio does not show any color coding in typescript files

also, now I my Visual Studio does not show any color coding in typescript files

What version of ts is this? 2.0.3 or 1.8?

we downgraded to 1.8. in help -> about it says 1.8.36 and tsc -v says 1.8.10

How did you downgrade? uninstall 2.0 + repair on VS?

uninstalled and reinstalled 1.8 from an installer. it works for other people on the same box now. we did not do a repair on VS

I do not know how it would work without a repair. the version of TS installed by update 3 (1.8.36) is different from that one that was last published on the download page (1.8.6). so i am almost certain it will not work as there are dependencies on newer binaries that were shipped with VS 2015.
I would recommend, first uninstalling all versions of TS on your machine, then install latest Update 3 if you have not, or running a repair on Update 3 if you already have it.

I did that, but then there is no version of TS on the computer

i guess i have the same issue
Visual Studio 2015 update 3 updated Typescript from 1.8.* to 2.0.3 and now my ts is not transpiling and i get warnings about experimental decorators (but they are enabled in my tsconfig).
when i run tsc -v i get 2.0.3
but when i run tsc with a file that contains decorators i get a warning that i should enable them in tsconfig.
when i run tsc without options nothing happens
do you have any hint how i can diagnose why tsc is not picking up the tsconfig file?

@JanEggers can you share a project i can try with?

not that project but i have a playground solution with the same issue.
https://github.com/JanEggers/Playground
in
https://github.com/JanEggers/Playground/blob/master/Playground.core/wwwroot/app/app.component.ts
i see the experimental decorator warning and it was fine with ts 1.8.4

@JanEggers, this looks like the issue in https://github.com/Microsoft/TypeScript/issues/10578. your tsconfig.json include clause really does not include anythign. so the file does not "belong" to the tsconfig.json, and hence the error.
changing the tsconfig.json to:
{
  ...
  "include": [
    "wwwroot/app/**/*"
  ],
  ...
}

solves the issues

thx its working again.
but why did it work with 1.8.4 and with 2.0 and a gulp task
for the next version i really would appreciate if you can  provide some diagnostic output option so anyone can figure this out more easy themselves.
i was really paranoid if the tsconfig got loaded at all as it was working fine with the last version

I deleted my user account on the server and created a new one and it is now working with TS 1.8


Wire new dqlite implementation

This branch wires the new Go bindings for dqlite into the lxd code base. The commits are mostly organized by sub-package, I believe it should not be too bad to backport.

jenkins: test this please

jenkins: test this please

jenkins: test this please

jenkins: test this please

Doesn't run on 32bit arches.

Error messages need to be updated to match our pattern of starting with a capital and not ending with a period

As for the DB being flushed to disk thing, can we add a .sync command or something to lxd sql and use that instead of having to either skip the empty table checks or have to bounce the daemon to have it sync the database?

Also, should I kill the go-grpc-sql and go-sqlite3 repos in the Canonical namespace?

Those two runners show the same error:

https://jenkins.linuxcontainers.org/job/lxd-github-pull-test/4625/arch=amd64,backend=lvm,compiler=golang-1.10,restrict=lxd-priv/console
https://jenkins.linuxcontainers.org/job/lxd-github-pull-test/4625/arch=amd64,backend=zfs,compiler=golang-1.10,restrict=lxd-priv/console

==> Spawned LXD (PID is 243993)
==> Confirming lxd is responsive
INFO[07-30|23:22:43] LXD 3.3 is starting in normal mode       path=/lxc-ci/build/tmp.iTEVMYRIt7/go/src/github.com/lxc/lxd/test/tmp.Git/Ktw
INFO[07-30|23:22:43] Kernel uid/gid map:
INFO[07-30|23:22:43]  - u 0 0<PHONE_NUMBER>
INFO[07-30|23:22:43]  - g 0 0<PHONE_NUMBER>
INFO[07-30|23:22:43] Configured LXD uid/gid map:
INFO[07-30|23:22:43]  - u 0<PHONE_NUMBER>000000
INFO[07-30|23:22:43]  - g 0<PHONE_NUMBER>000000
WARN[07-30|23:22:43] CGroup memory swap accounting is disabled, swap limits will be ignored.
INFO[07-30|23:22:43] Initializing local database
INFO[07-30|23:22:43] Updating the LXD database schema. Backup made as "local.db.bak"
Error: LXD still not running after 300s timeout (Get http://unix.socket/1.0: dial unix /lxc-ci/build/tmp.iTEVMYRIt7/go/src/github.com/lxc/lxd/test/tmp.Git/Ktw/unix.socket: connect: no such file or directory)

So looks like a hang on initial startup, this needs to be tracked down as it hit two different machines using two different storage backends on the first test run, so chances are we'll see a lot more of that.

I capitalized all error messages introduced or modified by this branch.

I added a the "lxd sql global .sync" command since it seems a good idea in general.
However, I think that the fact that the test for activateifneeded now actually bounces the daemon is a good thing because it ensures that the shutdown logic does sync to disk. If we were to manually run .sync in the test, we might miss a regression in the shutdown logic.

 As for go-sqlite3, we currently don't needed but if we want "go get" to work without passing the "-tags libsqlite3" flag, we probably have to keep it. I don't like it much, because CanonicalLtd/go-sqlite3 would be essentially a copy of mattn/go-sqlite3 but with the vendored sqlite3 code replaced with the one that contains our patch. Also, in order to make "get run" I'll need to vendor libuv as well (in go-dqlite). I'm not entirely sure we want to go down this path, but I don't see other alternatives if we want "go get" to work.

jenkins: test this please

Regarding the two jobs that got stuck during startup, I believe that's fixed now.

I've re-enabled the empty table checks also for the clustering case. It seems to actually work properly. But let's see what CI has to say.

jenkins: test this please

looking good but lets make sure

jenkins: test this please

Thanks everyone!

Thanks!


Adding HighlightRange to the selection

moving the issue from https://github.com/dlibby-/highlight/issues/14
@Reinmar as FYI
My understanding is that since HighlightRange inherits from Range, it can be passed to Selection#addRange().
According to https://www.w3.org/TR/selection-api/#dom-selection-addrange that range is passed there by a reference and it should be updated once the selection changes.
Does it mean that someone can add a highlight range to the selection? And, if so, could it cause some problems?
Also, we could imagine that the selection is highlighting its ranges internally (so ::selection ~== ::highlight(selection)). I wonder if this could have some interesting implications.

@Reinmar As mentioned in #69 , we have removed HighlightRange from this API and use a HighlightRangeGroup to apply highlight styling to a group of ranges. Therefore, this shouldn’t be an issue anymore. Please let me know if you disagree.

Closing this issue as it seems to have been addressed. Please feel free to reactivate if you have any additional concerns.


[documentation] Changing the dev activity note in Iroha 1 documentation to a warning

Documentation URL(s)
https://github.com/hyperledger/iroha/blob/main/docs/source/index.rst
Description
Currently, the documentation claims that I1 and I2 development happens in parallel, while the core team is focused on Iroha 2. This needs an update, so the users are not misdirected towards using I1 in new projects.
Your suggestions
I have an updated text in mind. First of all, it should be a warning, not an info block, secondly, it has to be on the top.
Finally, I believe it should look like this to reflect the current state of development:
Please note that support for Hyperledger Iroha v1 is limited because it's no longer actively developed.

The core team focuses on Hyperledger Iroha v2, a complete rewrite of Iroha in Rust.

These versions are incompatible, so you will have to use Iroha 2 instead of Iroha 1 for the new projects.
You can read about the differences in the Iroha 2 documentation.

Who can help?
@outoftardis

Addressed in #3071


Nesting documents results in error msg

When nesting this:
https://docs.google.com/spreadsheets/d/1vdpVb1WpAPu47Zsp3_YbHZ-nG_rrZb5gCUUXFQhLQDk/pubhtml
into -
https://docs.google.com/spreadsheets/d/1kAcdCnwglW_fIVk1XCg2yURN6m7Y3SyQOKEJYIhHOUI/
I receive an error message indicating the document cannot be found -
Document is missing (perhaps it was deleted?)
Can anyone else confirm this behaviour?
-Damo

Update: I  wasn't using a copy of the the latest Google CMS master.
All good now, when using ID instead of complete URL

Working, when using Spreadsheet ID only..

Damo, if you are still struggling with this, try boiling the problem down to the minimum needed to test, then share with me with full edit capability.  Contact me with your e-mail here: http://www.seaver99.com/demo/php/contactform.php and I'll give you my e-mail for sharing purposes.

Thanks for the follow up, have got it working using the full URI..


The repo start count on the top is missing - the side shows

Description
I know about the pinned issue, and I have read it, BUT my top star count is missing

How to replicate the issue + URL
https://github.com/refined-github/refined-github
Extension version
23.9.21
Browser(s) used
Version 116.0.5845.187 (Official Build) (arm64)

See

https://github.com/refined-github/refined-github/pull/6941
https://github.com/refined-github/refined-github/issues/6940

@fregante may I suggest to make the description of this option clearer?
Because I expected to find how to disable it by searching for "star" or "fork", and couldn't. That's why I also asked there.

I have disabled it but still no stars besides the repo name

Let me be clear, I'm fine with duplication of the stars on the side and header. The whole purpose of the start count (which was originally proposed by me) was to show the stars on each repo page.


Package is not working

I've set my private token and I keep clicking on sync/create but nothing is happening, I've also restarted Atom few times...

That package not developed


Add ability to set a contextual gutter

I have a website which uses the default settings for columns and gutters. But one block on my site is a grid of 8 | 4, where I don't need any gutters and the columns need to have the exact same width because there's a responsive image that defines the height.
The 8 column grid is again divided in two times 4 of 8 that need to have the same height again. This way I get a grid of 8 squares in which I can group an image (blue squares) with a description (grey blurred squares)

Because of the default gutter however, the columns are never the same width. Setting block-collapse on span-colums removes the spaces, but adds the margin to the column, which makes the width of my columns different and prevents the images from lining up correctly.
If I were able to define a gutter within a certain context this wouldn't be a problem.

This is interesting @ChezFre, I need to delve into it a bit more to understand the example. Is it possible for you to make a repo setting up the example? If not, I will give it a shot to re-create the situation next week when I have a minute.

Hi @wardpenney, thank you for your time and input. I've set up an example of what I'm trying to achieve at https://github.com/ChezFre/neat-example.
The problem is resolved when I set the $gutter to 0. However, since this part of the site is an exception it would be undesirable to do so. At the moment I've just abandoned the use of the grid for this particular block.

+1

We will be adding support for custom grid sections.

However, support for this will ont be present in any 1.X version.


Updated crossnamespace to mention that dashboards need spec.allowCrossNamespaceImport as well

Inspired by https://github.com/grafana/grafana-operator/issues/1142.

 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.


This relative module was not found:  * ./lib-cov/fluent-ffmpeg in ./node_modules/fluent-ffmpeg/index.js

Version information

fluent-ffmpeg<EMAIL_ADDRESS>ffmpeg version:
OS:Mac

Code to reproduce
const ffmpeg = require('fluent-ffmpeg');

Electron + Vue 框架，在js中引用ffmpeg。
package.json中
"electron:serve": "vue-cli-service electron:serve",
在终端执行 npm run electron:serve 提示错误
ERROR  Failed to compile with 1 errors                                                                                11:10:12
This relative module was not found:

./lib-cov/fluent-ffmpeg in ./node_modules/fluent-ffmpeg/index.js
ERROR  Build failed with errors.
npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR<EMAIL_ADDRESS>electron:serve: vue-cli-service electron:serve
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>electron:serve script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.

npm ERR!      /Users/liukun/.npm/_logs/2020-04-17T03_10_12_322Z-debug.log
The terminal process terminated with exit code: 1
(note: if the problem only happens with some inputs, include a link to such an input file)
Expected results
Observed results
This relative module was not found:

./lib-cov/fluent-ffmpeg in ./node_modules/fluent-ffmpeg/index.js

Checklist

[ ] I have read the FAQ
[ ] I tried the same with command line ffmpeg and it works correctly (hint: if the problem also happens this way, this is an ffmpeg problem and you're not reporting it to the right place)
[ ] I have included full stderr/stdout output from ffmpeg

Same problem here (Electron + Vue + macOS + electron-builder).
I found that fork https://github.com/pietrop/ffmpeg-static-electron that should solve this issue, but the author doesn't seem to maintain it anymore.

I had the same issue - Electron,  vue/cli and electron-builder - and here's how I solved the issue:
In vue.config.js I added this to the webpack configuration in the plugins property:
const webpack = require('webpack');

module.exports = {
  configureWebpack: {
    plugins: [
      new webpack.DefinePlugin({
        'process.env.FLUENTFFMPEG_COV': false
      })
    ]
  }
}

That fixed my issue.

@ghost I tried the same thing but the Rollup approach and it's not working for me in Rollup. Can you suggest some way?
I am trying to bundle a nodejs app fully.
Command I ran: rollup -c --environment FLUENTFFMPEG_COV:false OR rollup -c --environment FLUENTFFMPEG_COV:0

修改 node_modules/fluent-ffmpeg/index.js

@haoranisg does that fix the bundling issue, or is it just a suggestion?
@max-programming - I've reached the exact same point as you, trying to bundle a node package using this library with rollup and getting this error, did you every solve this or have a workaround?


index with $or operator not working

Description
I am trying to create an index for a query with "$or" operator but the query plan does not recognize the index.
my question is, how to create a working index for the following query ?

{
   "selector": {
      "type": "project",
      "$or": [
         {
            "pmId": "8b1cee44bf16ccced9163adee20008a8"
         },
         {
            "sponsor._id": "8b1cee44bf16ccced9163adee20008a8"
         },
         {
            "AccessMembers": {
               "$elemMatch": {
                  "$and": [
                     {
                        "id": "8b1cee44bf16ccced9163adee20008a8"
                     },
                     {
                        "ACL": {
                           "$in": [
                              "View",
                              "Edit"
                           ]
                        }
                     }
                  ]
               }
            }
         }
      ]
   }
}

Steps to Reproduce
Expected Behaviour
Your Environment

CouchDB Version used:
v. 2.1.1
Browser name and version:
Operating System and version:
Windows Server 2016 Datacenter

Additional context

What is your Mango index definition?
What does the `/{db}/_explain' endpoint say when you use it?
I would think in this situation you'd need every single field you're referencing in your index: pmId, sponsor._id, and ACL.

thank you @wohali for the reply.
the index is as following:

{
	"index": {
		"fields": [
			"type",
			"pmId",
			"sponsor._id",
			"AccessMembers.[].id",
			"AccessMembers.[].ACL"
		]
	},
	"name": "pm-projects-index",
	"type": "json"
}

the "explain" result as following:

{
 "dbname": "pm",
 "index": {
  "ddoc": null,
  "name": "_all_docs",
  "type": "special",
  "def": {
   "fields": [
    {
     "_id": "asc"
    }
   ]
  }
 },
 "selector": {
  "$and": [
   {
    "type": {
     "$eq": "project"
    }
   },
   {
    "$or": [
     {
      "pmId": {
       "$eq": "8b1cee44bf16ccced9163adee20008a8"
      }
     },
     {
      "sponsor._id": {
       "$eq": "8b1cee44bf16ccced9163adee20008a8"
      }
     },
     {
      "AccessMembers": {
       "$elemMatch": {
        "$and": [
         {
          "id": {
           "$eq": "8b1cee44bf16ccced9163adee20008a8"
          }
         },
         {
          "ACL": {
           "$in": [
            "View",
            "Edit"
           ]
          }
         }
        ]
       }
      }
     }
    ]
   }
  ]
 },
 "opts": {
  "use_index": [],
  "bookmark": "nil",
  "limit": 25,
  "skip": 0,
  "sort": {},
  "fields": "all_fields",
  "r": [
   49
  ],
  "conflicts": false,
  "stale": false,
  "update": true,
  "stable": false,
  "execution_stats": false
 },
 "limit": 25,
 "skip": 0,
 "fields": "all_fields",
 "mrargs": {
  "include_docs": true,
  "view_type": "map",
  "reduce": false,
  "start_key": null,
  "end_key": "",
  "direction": "fwd",
  "stable": false,
  "update": true,
  "conflicts": "undefined"
 }
}

The simplest is just creating an index on type. The add in the execution stats to see how many docs are ignored or not.

From: brifai<EMAIL_ADDRESS>Sent: Saturday, June 29, 2019 12:19 PM
To: apache/couchdb
Cc: Subscribed
Subject: Re: [apache/couchdb] index with $or operator not working (#2063)
thank you @wohalihttps://github.com/wohali for the reply.
the index is as following:
{
"index": {
"fields": [
"type",
"pmId",
"sponsor._id",
"AccessMembers.[].id",
"AccessMembers.[].ACL"
]
},
"name": "pm-projects-index",
"type": "json"
}
the "explain" result as following:
{
"dbname": "pm",
"index": {
"ddoc": null,
"name": "_all_docs",
"type": "special",
"def": {
"fields": [
{
"_id": "asc"
}
]
}
},
"selector": {
"$and": [
{
"type": {
"$eq": "project"
}
},
{
"$or": [
{
"pmId": {
"$eq": "8b1cee44bf16ccced9163adee20008a8"
}
},
{
"sponsor._id": {
"$eq": "8b1cee44bf16ccced9163adee20008a8"
}
},
{
"AccessMembers": {
"$elemMatch": {
"$and": [
{
"id": {
"$eq": "8b1cee44bf16ccced9163adee20008a8"
}
},
{
"ACL": {
"$in": [
"View",
"Edit"
]
}
}
]
}
}
}
]
}
]
},
"opts": {
"use_index": [],
"bookmark": "nil",
"limit": 25,
"skip": 0,
"sort": {},
"fields": "all_fields",
"r": [
49
],
"conflicts": false,
"stale": false,
"update": true,
"stable": false,
"execution_stats": false
},
"limit": 25,
"skip": 0,
"fields": "all_fields",
"mrargs": {
"include_docs": true,
"view_type": "map",
"reduce": false,
"start_key": null,
"end_key": "",
"direction": "fwd",
"stable": false,
"update": true,
"conflicts": "undefined"
}
}
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://github.com/apache/couchdb/issues/2063?email_source=notifications&email_token=AABL2AUWVGULUJD6MCB3U3DP44ZKPA5CNFSM4H3Z4I5KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODY3WGBY#issuecomment-506946311, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AABL2ASF4FIGXFGCE3SVY6DP44ZKPANCNFSM4H3Z4I5A.

Thank you @garrensmith for your input, but can this query be indexed ?

If you use an $or operator then all the fields in the index need to be on both sides of the or otherwise it won’t work. That’s why I think keep the index fields to a minimum or use a partial_filter_selector when building the index.

From: brifai<EMAIL_ADDRESS>Sent: Saturday, June 29, 2019 2:44 PM
To: apache/couchdb
Cc: garren smith; Mention
Subject: Re: [apache/couchdb] index with $or operator not working (#2063)
Thank you @garrensmithhttps://github.com/garrensmith for your input, but can this query be indexed ?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/apache/couchdb/issues/2063?email_source=notifications&email_token=AABL2ATFHY6I7F7SHCGPQTTP45KKNA5CNFSM4H3Z4I5KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODY3YENY#issuecomment-506954295, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AABL2AWY4INWQQWLLX7XTBDP45KKNANCNFSM4H3Z4I5A.

Thank you @garrensmith , @wohali do you have any input about it ?

no, garren is the expert here


v2 tag is not set to the latest tag of the v2.x series

Can you update the v2 tag to be linked to 2.0.7 tag?

Done. Thanks for pointing this out!


PHP 8.2 Deprecated Warnings

Creation of dynamic property QuickBooksOnline\API\Diagnostics\LoggerBase::$ServiceRequestLoggingLocation is deprecated in vendor/quickbooks/v3-php-sdk/src/Core/Configuration/LocalConfigReader.php on line 417.
Creation of dynamic property QuickBooksOnline\API\Data\IPPBatchItemRequest::$operationSpecified is deprecated in vendor/quickbooks/v3-php-sdk/src/DataService/Batch.php on line 230.
Creation of dynamic property QuickBooksOnline\API\Data\IPPBatchItemRequest::$operationSpecified is deprecated in vendor/quickbooks/v3-php-sdk/src/DataService/Batch.php on line 230.

Same issue when trying to get the oauth link.
Getting: Can't Config Environments from passed parameters but it's a blanket error for the actual error that is a dynamic property creation by the sdk.
It does not seem php 8.2 compatible at all.

I've found when the exception is triggered... It's in the file LocalConfigReader.php on line 124 when the logger is setup:
LocalConfigReader::setupLogger($ippConfig, $defaultLoggingLocation, "TRUE");
The exception is :
Creation of dynamic property QuickBooksOnline\API\Diagnostics\LoggerBase::$ServiceRequestLoggingLocation is deprecated
So what I would propose is simply add this in the LoggerBase.php file before the class creation:
use QuickBooksOnline\API\Core\CoreConstants;
use \AllowDynamicProperties;
#[AllowDynamicProperties]
class LoggerBase
{

We are waiting for the same fix

Getting the same error; hoping for a fix soon.

Same error here - what is taking so long to fix this?  The fix is rather simple.

@poolsimon We will update you soon

The fix will be released soon

Fixed in version 6.1.2


typo in landcover.json

 Schreib mal wieder was

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
5 unchanged lines in 4 files lost coverage.
Overall coverage decreased (-0.001%) to 89.185%

Files with Coverage Reduction
New Missed Lines
%

services/src/handlers/upload.rs
1
99.05%

services/src/pro/handlers/datasets.rs
1
95.7%

services/src/pro/users/postgres_userdb.rs
1
96.47%

services/src/pro/contexts/postgres.rs
2
96.24%

Totals

Change from base Build<PHONE_NUMBER>:
-0.001%

Covered Lines:
105663

Relevant Lines:
118476

💛  - Coveralls


[Improvement] Add key mapping to disable decoupled camera

Hi, thank you for this amazing mod.
I would like to sugest an improvement, for a key that disables the decoupled camera, as this will allow for better compatibility with some combat mods.
Thanks!

Thank you for the suggestion. Have you tried solving the issue with the adaptive item configs (see adaptive_crosshair_hold_items and adaptive_crosshair_use_items)?

Wow, it worked great!
However the idea of a key to enable/disable the decoupled camera would still be useful when building, as it allows for more precise block placement as the block's rotation depends on the direction the player is facing (I.E. placing a chest in the wrong direction because the player is facing another direction).
Anyway, for the  combat the adaptive_crosshair_hold_items option works great!
Thank you!

But how do you do that?

However the idea of a key to enable/disable the decoupled camera would still be useful when building, as it allows for more precise block placement as the block's rotation depends on the direction the player is facing (I.E. placing a chest in the wrong direction because the player is facing another direction).

This has been fixed in the latest release. Please let me know if this issue can be closed, as there should no longer be a need for such a feature.

Yo! I was also looking for something like this. Cuz its kinda uncomfortable having the camera not turn with the player. Could this be an actual feature of the mod?

I was also looking for something like this. Cuz its kinda uncomfortable having the camera not turn with the player. Could this be an actual feature of the mod?

This sounds like you want to permanently disable the decoupled camera mode, which is already possible by setting decoupled_camera to false in the config file (shouldersurfing-client.toml). Does this already solve your problem?

Hey! I was just going to come here to suggest the same feature. A toggle key for the decoupled camera would be great. I use this on a server with a very intricate spell system and the game looks great with the decoupled camera but in combat its necessary to have it disabled. A toggle for this feature would save a lot of time from going through 5+ menus just to toggle it!

Although the next version will have a dedicated keybind for toggling the camera coupling, I still recommend properly setting up the adaptive_crosshair_hold_items and adaptive_crosshair_use_items config options, which will automatically enable and disable the camera coupling depending on the item held or used.


bugs

Describe your problem here.
Warning: Could not find Visual Studio 2017 VsDavCmd
Missing: HXCPP_VARS
Error Could not automatically setup MSVC
What does this mean? Did I do something wrong? its been like that for about an hour. should leave the command to run or do something
Are you modding a build from source or with Lua?
Source
What is your build target?
Windows x64
Did you edit anything in this build? If so, mention or summarize your changes.
no

haxe issue. not a psych issue


3.1.0 RE3 seed run out of free item slots

Version
3.1.0
Seed
R730-NY2V-ZTM885XZ3VY20QNHMHLK0C00H200H200ZZEXZYZZZZ
Scenario
RE 3 (Jill)
Player Character
kendo
What happened?

I have changed this message in dev. builds now to explain and just say "try another seed". I am no longer taking reports of this issue, as it is just down to a non-perfect door randomizer.


Error when importing Draw or PandasTools

I am getting the following error when using Python 3.10.11 on Ubuntu 2022:
Python 3.10.11 (main, Apr 20 2023, 19:02:41) [GCC 11.2.0] on linux
Type "help", "copyright", "credits" or "license" for more information.

import rdkit
from rdkit.Chem import Draw
Traceback (most recent call last):
File "", line 1, in
File "/home/polyakovv/anaconda3/envs/rdk3_10/lib/python3.10/site-packages/rdkit/Chem/Draw/init.py", line 14, in
from rdkit.Chem.Draw import rdMolDraw2D
ImportError: libXrender.so.1: cannot open shared object file: No such file or directory

it happens on most recent and older versions of rdkit
Thanks for you help,
Valery
Configuration (please complete the following information):

RDKit version: 2022.03.5 and 2023
OS: [e.g. Ubuntu 20.04]: 2023
Python version (if relevant): Python 3.10.11
Are you using conda? yes
If you are using conda, which channel did you install the rdkit from? default

Additional context
Add any other context about the problem here.

Looks like the package libxrender1 is missing.
sudo apt install libxrender1
should hopefully resolve the ImportError.


Show-if-true in a list does not respond correctly to a join-field

Describe the bug
A delete-action of a row in a list of my Items table should only be allowed (using Show-if-true for the delete-button) when a join-field prijslijst.status is equal to "Ontwikkeling".
The prijslijst.status join-field is correctly shown in the Items-list buy still the action-button does not respond correctly.
To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior
The action-row button should behalve correctly on a Show-if-true formula using a join-field.
Screenshots

Additional context
Add any other context about the problem here.

But if i create a formula value instead of a  joined field value then the show-if-true works as expected ;-)

fixed in #2250

Confirmed. Thanks


Running PublishBuildArtifacts on Linux results in error

I am trying to utilize the PublishBuildArtifacts task within a build definition that is designed to build some artifacts within a Linux environment.
1 2015-07-22T20:42:57.498Z: Cleaning staging folder: /home/msimons/BuildAgents/MultiDevTest/_work/build/ce2c019942eac2ab3e1bceb308a15a1f1789b1e89e2218e12a6aecbde54b4869/staging/CoreClr
2 2015-07-22T20:43:00.109Z: found 25 files
3 ##[Error] 2015-07-22T20:43:00.305Z: TypeError: undefined is not a function
4 ##[Error] 2015-07-22T20:43:00.389Z: Return code: 1
The inputs I have set are the following
Copy Root = ""
Contents = "bin/Product/**"
Artifact Name = "CoreClr"
Artifact Type = "Server"
I have also tried to user the File share type with the same result.  My expected outcome would be for this to work or to have more assistance from the log output in determining the source of the failure.

Opened issue in wrong repo

even though closed - on the build definition variables tab, set
system.debug to true
that should output many more details


SVGImagesource flickering

Today I started a new test to see if I can animate SVG raw and then change the source using
 svgsource = new SvgImageSource();
 Image.Source = svgsource;
 svgsource.SetSourceAsync(memoryStream);

https://github.com/juanpaexpedite/SVGAnimations
Of course I have to learn how to optimize to edit just parts of the svg string, and recreating the source stream.
But at the moment the issue I have is when I call SetSourceAsync again it flickers too much. The svg is pretty simple, And would be great if its possible to avoid that.
Will be possible to do something in this way in a near future?

You can try setting and awaiting the source on the  SvgImageSource before setting it as the image source. It should avoid the flickering, tho you still won't get real time animation because of the asynchronous nature of that flow.
(Though if you've got too much time on your hands you can drop down into Win2D levels and do it)

Thank you for your feedback. I work on the XAML team at Microsoft. We are aware of the issue and plan to improve SetSourceAsync in a future release so that the flickering no longer occurs.
In the meantime, you can workaround the issue by using two SvgImageSources and alternating the Canvas.ZIndexProperty of each one. On each frame you can call SetSourceAsync on the SvgImageSource that is lower in the z-order and then switch the two SvgImageSources to go to the next frame.

Really thank you for the answer @samschiferl (hopefully not a Win2D workaround answer :)).
I will try the workaround. Is there a roadmap or clue about SVG?.
I would like to implement a library to animate paths, strokes, etc. and one day something like Lottie with Composition, because I understand there is no roadmap for 'XAML stroke and path animation' on the way isn't it?
Thank you..

@juanpaexpedite Support for stroke and path animations is on the public backlog: as discussed in #193, this is an area that is being investigated currently. This feature has been requested repeatedly and as soon as we have more clarity on actual timelines, we'll let you know.

It sounds like a workaround has been provided for this issue.


[DOC] API reference should point to code/import location

Currently, the API reference does not clearly point to the code location of the class/method documented.
Some classes will have examples with an import statement, but in general this is not the case.
Hence, a typical user who does not have a developer install will be unable to find the correct import command from looking at the API reference alone.
This should be addressed, e.g., by a link provided to source and/or a printed full import path at the top of each method/class page in the API reference.

yes good point, that is why I recently added docstring examples to the datasets. Ideally there can be a section Import in the docstring where the path is given? Not sure if this can work with Sphinx

this is channeling user frustration from users at my workplace 😄

Can you give an example of what you mean or point to a specific place in the docs and describe the expected outcome?

ok, let's say I'm browsing the docs and find AutoARIMA
https://www.sktime.org/en/latest/api_reference/auto_generated/sktime.forecasting.arima.AutoARIMA.html
and say, "nice, this would be a great forecaster to use!"
Then, the next step would be to write sth like from ???? import AutoARIMA, with the missing information being the ????.
It is unclear from a user perspective how to get from the doc page to the ????, and to my understanding there is no general way.
This is how the user experience looks like:

The "natural" places where a user might look, per expectation:

on the left, in the navigation bar. That is not ordered by module name level, so doesn't help.
at the top, e.g,. for a header forecasting.arima.AutoARIMA

The shortest path to the ???? information is clicking on the [source], then scrolling up until the file path gets visible in the GitHub interface which is not at all obvious imo.

sklearn and pandas have it in header, so might be worth to check how they did it

what we want it this:

I made a draft PR but this headline did not yet show up 😞