Panic on machine create

(digitalocean, but I don't think it's unique that driver)
What seems to be the relevant bit:
  time="2015-04-28T14:09:50-07:00" level="info" msg="Creating SSH key..."
   time="2015-04-28T14:09:52-07:00" level="info" msg="Creating Digital Ocean droplet..."
   panic: runtime error: invalid memory address or nil pointer dereference
   [signal 0xb code=0x1 addr=0x20 pc=0x82fc8]

   goroutine 16 [running]:
   runtime.panic(0x6d5280, 0xb31d84)
   	/usr/src/go/src/pkg/runtime/panic.c:279 +0xf5
   bytes.(*Buffer).ReadFrom(0x208f89ce0, 0x0, 0x0, 0x0, 0x0, 0x0)
   	/usr/src/go/src/pkg/bytes/buffer.go:169 +0x208
   github.com/docker/machine/libmachine/provision.installDockerGeneric(0x2208c51590, 0x208cd9400, 0x0, 0x0)
   	/go/src/github.com/docker/machine/libmachine/provision/utils.go:30 +0xe0
   github.com/docker/machine/libmachine/provision.(*UbuntuProvisioner).Provision(0x208cd9400, 0x0, 0x0, 0x0, 0x0, 0x0, 0x0, 0x208cb4c40, 0x12, 0x0, ...)
   	/go/src/github.com/docker/machine/libmachine/provision/ubuntu.go:103 +0x288
   github.com/docker/machine/libmachine.(*Host).Create(0x208c37ee0, 0x7fff5fbff973, 0x16, 0x0, 0x0)
   	/go/src/github.com/docker/machine/libmachine/host.go:121 +0x2ba
   github.com/docker/machine/libmachine.(*Machine).Create(0x208cad460, 0x7fff5fbff973, 0x16, 0x208cac5f0, 0xc, 0x208cab300, 0x2208c3a250, 0x208c7a480, 0x0, 0x0, ...)
   	/go/src/github.com/docker/machine/libmachine/machine.go:59 +0x565
   github.com/docker/machine/commands.cmdCreate(0x208c7a480)
   	/go/src/github.com/docker/machine/commands/create.go:89 +0xcae
   github.com/codegangsta/cli.Command.Run(0x77b690, 0x6, 0x0, 0x0, 0x7c2ab0, 0x10, 0x0, 0x0, 0x0, 0x0, ...)
   	/go/src/github.com/docker/machine/Godeps/_workspace/src/github.com/codegangsta/cli/command.go:113 +0xeb2
   github.com/codegangsta/cli.(*App).Run(0x208c368f0, 0x208c36000, 0xd, 0xd, 0x0, 0x0)
   	/go/src/github.com/docker/machine/Godeps/_workspace/src/github.com/codegangsta/cli/app.go:156 +0xaea
   main.main()
   	/go/src/github.com/docker/machine/main.go:73 +0x5a3

The offending code seems to be the buf.ReadFrom line:
func installDockerGeneric(p Provisioner) error {
        // install docker - until cloudinit we use ubuntu everywhere so we
        // just install it using the docker repos
        if output, err := p.SSHCommand("if ! type docker; then curl -sSL https://get.docker.com | sh -; fi"); err != nil {
                var buf bytes.Buffer
                if _, err := buf.ReadFrom(output.Stderr); err != nil {
                        return err
                }

                return fmt.Errorf("error installing docker: %s\n", buf.String())
        }

        return nil
}

It seems not to like to try and read from Stderr when something has gone wrong.  I'm not sure that trying to read from STDERR on the command if the right thing to do, really - we might want to just return the error.  If you look at the code for RunSSHCommandFromDriver, which GetSSHCommand wraps, there are many cases that might cause you to return errors from that function that don't necessarily guarantee getting a cmd to read STDERR from successfully e.g. we can't establish the SSH hostname. cc @ehazlett @sthulb

I'm not sure if it's related, but I'm getting some (not infrequent) SSH timeouts on create as well:
✗ digitalocean: create with supported engine options
   (in test file test/integration/scratch.bats, line 38)
     `[ $status -eq 0 ]' failed
   time="2015-04-28T15:03:48-07:00" level="info" msg="Creating SSH key..."
   time="2015-04-28T15:03:48-07:00" level="info" msg="Creating Digital Ocean droplet..."
   time="2015-04-28T15:05:54-07:00" level="error" msg="Error creating machine: Error getting SSH command: dial tcp <IP_ADDRESS>:22: operation timed out"
   time="2015-04-28T15:05:54-07:00" level="warning" msg="You will want to check the provider to make sure the machine and associated resources were properly removed."
   time="2015-04-28T15:05:54-07:00" level="fatal" msg="Error creating machine"

@nathanleclaire the problem with just returning from an error is that the install script outputs stuff to stderr which appears to be not err.  I believe this is still the case -- I will do a quick test with curl to check.

Haven't duped this


Apache Cassandra vector store

Implement Apache Cassandra vector store

[x] metadata fields
[x] similarity threshold
[ ] filter expressions
[ ] filter expression cql validation
[ ] test driver config files
[ ] id --> multiple partition keys
[ ] schema with clustering keys

This is starting to take shape.  I'm still working through getting the tests working and fleshing out the filter expression code.
Not really for a detailed review yet (I will take the PR out of draft when that happens), but high-level eye-balling and hallway testing is more than welcome.

This PR is ready for review @tzolov @markpollack
All further changes will be added as additional checkpoint commits (that will be squashed after the review is finished).

Rebased, squashed and merged at 656d238285838461d7f9f4ebc67fe1f8dc740750


Fix PyPI deploy workflow

[ ] Closes #xxxx
[ ] I am familiar with the contributing guidelines
[ ] Tests added
[ ] Updates entries in docs/sphinx/source/reference for API changes.
[ ] Adds description and name entries in the appropriate "what's new" file in docs/sphinx/source/whatsnew for all changes. Includes link to the GitHub Issue with :issue:`num` or this Pull Request with :pull:`num`. Includes contributor name and/or GitHub username (link with :ghuser:`user`).
[ ] New code is fully documented. Includes numpydoc compliant docstrings, examples, and comments where necessary.
[x] Pull request is nearly complete and ready for detailed review.
[x] Maintainer: Appropriate GitHub Labels (including remote-data) and Milestone are assigned to the Pull Request and linked Issue.

@wholmgren tagged v0.9.2-alpha.1 yesterday (https://github.com/pvlib/pvlib-python/pull/1483#issuecomment-1216828879) but the deploy workflow run for that commit skipped the PyPI upload step.  I think, due to an error I made in #1306, the problem is that the workflow was triggered only by the push, not the tag, so the github.ref was master and not v0.9.2-alpha.1 and that step was skipped.  This PR introduces version tags as another trigger for that workflow, so the workflow should be run both on the initial push to master (skipping PyPI upload) and the subsequent version tag (activating the PyPI upload).
Would like to get this merged ASAP to not further delay evaluation of the build/install changes ahead of the final 0.9.2 release.

Seems to have worked: https://github.com/pvlib/pvlib-python/runs/7886150625?check_suite_focus=true and https://pypi.org/project/pvlib/0.9.2a2/
The logs reveal a new issue:

Warning:  You are using "pypa/gh-action-pypi-publish@master". The "master" branch of this project has been sunset and will not receive any updates, not even security bug fixes. Please, make sure to use a supported version. If you want to pin to v1 major version, use "pypa/gh-action-pypi-publish@release/v1". If you feel adventurous, you may opt to use use "pypa/gh-action-pypi-publish@unstable/v1" instead. A more general recommendation is to pin to exact tags or commit shas.

I'll just piggyback that one on #1483.


Upgrade to target-based CMake. Added option for shared libraries.

The Travis failures seem to have to deal with a difference in Boost versions. It looks like Boost 1.54.0 doesn't have the CMake targets defined. I can switch that easily enough. The AppVeyor tests, on the other hand, I'm not so sure what the issue actually is.

Looks good. Can you fix Windows build?

I'll give it a shot. I have basically no Visual Studio experience (or Visual Studio for that matter). I'll see if I can reproduce the error in the free version.

OK, so it looks like it was expecting static libraries instead of DLLs, I added the static library option in the appveyor.yml file and it seems to work.

Ahh, I forgot about that one, I was getting a couple of compile errors about redisclient::Pipeline being an incomplete type. I'll go back and fix that.
-Phil

On Oct 20, 2017, at 11:23 AM, Alex Nekipelov<EMAIL_ADDRESS>wrote:
@nekipelov commented on this pull request.
In examples/CMakeLists.txt https://github.com/nekipelov/redisclient/pull/46#discussion_r145993010:

 async_pubsub2.cpp

 async_pubsub.cpp
 async_set_get2.cpp
 async_set_get.cpp
 sync_set_get.cpp

sync_pipeline.cpp
Why did you delete this line?

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/nekipelov/redisclient/pull/46#pullrequestreview-70877904, or mute the thread https://github.com/notifications/unsubscribe-auth/AMchtus1K1JowHQXs8fekLn6tfrfecLAks5suLrfgaJpZM4P6x_D.

Thank you!


typo: lable -> label

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

BingMaps/articles/create-a-custom-map-url.md
:white_check_mark:Succeeded

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

@DRMap this are still issues on the live docs, are you planning to fix this elsewhere or are these docs just abandoned now?

Cleaning up the documentation repository. The ‘lable’ typo should be fixed now. Would you mind double checking?
From: Nick Schonning<EMAIL_ADDRESS>Sent: Friday, March 6, 2020 10:14
To: MicrosoftDocs/bingmaps-docs<EMAIL_ADDRESS>Cc: Dietmar Rabel<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [MicrosoftDocs/bingmaps-docs] typo: lable -> label (#14)
@DRMaphttps://github.com/DRMap this are still issues on the live docs, are you planning to fix this elsewhere or are these docs just abandoned now?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/MicrosoftDocs/bingmaps-docs/pull/14?email_source=notifications&email_token=ALZ6MASJGCKDK5IAXVART7DRGE4NPA5CNFSM4F6JKYV2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEOCJ2SA#issuecomment-595893576, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ALZ6MARTO643NH5SOD52BITRGE4NPANCNFSM4F6JKYVQ.

Looks like it's OK on the live site now.
FYI, closing a bunch of PRs without a comment  after they've sat open for 2 years isn't a good way to run on OSS project. Closing the is fine, but you should mention that you've already fixed them on an internal branch. I'm not sure if the rest of the PRs you closed are actually addressed, but i'm not spending any more energy here


fixing minor ui changes on app

renaming some home page components
fixing influncer list and brand list
adding dark theme to app
renaming in influencer registartion form

@RohitGupta1237
Assign me the issue


Move pt_lib.jl into Banyan.jl

Idea: Move res/pt_lib.jl to src/pt_lib.jl and just include the file in Banyan.jl and export all important the split/merge/cast functions as part of Banyan.jl. Then, any user will have Banyan in their environment and be using it so it should get usinged and the generated code should be able to use all the functions in it.
Then, we shouldn't have to have special parameters in create_job, just a files, code_files, pf_dispatch_table (path to the pt_lib_info.json).

Closed with #36


Draft: Fix negated option value when followed by argument

I'm facing a boolean option parsing error: when using negated option (e.g. --no-foo) followed by argument, the parsed value is not the expected one.
Context: In modulesync, we use thor. I expect to contribute a new command to be used like this:
msync execute --no-fail-fast -- /path/to/script --script-option

But the options hash returned by thor after parsing this command line is:
{
"fail_fast" => true
}

Which is not the expected value ;-)
Please note that if --no-fail-fast option is the last one, its OK:
$ msync execute --no-fail-fast
{
"fail_fast" => false
}
$ msync execute /path/to/script --no-fail-fast
{
"fail_fast" => false
}

First, I commited a small test, attempting to target the problem.

This bug was a side-effect of system's thor load (through system's bundle command).
Solved uninstalling ruby-thor package (and ruby-bundler) and install them through gem.


OpenStack providers: migrate to gophercloud/gophercloud

This is a first refactoring of the OpenStack integration. The main goal is migrating Gophercloud (the SDK) to gophercloud/gophercloud because it moved years ago to its own organization and repo. A working implementation has been tested with an OpenStack based cloud provider (entercloudsuite.com).

@f-minzoni Thank you so much for the contribution! Sorry for the delay, it has been very busy for us. I took a quick look, and this is great! Great job!
I think I'll have time to do a detailed review over the weekend.
If you're able, can you just rebase the PR to two commits: one commit with changes to the code base, and another with the vendor directory and with changes to the Gopkg.* files? This is not required, but could make it easier to review the PR.
Thanks once again!


Add 2D SDF Nodes

This PR is a continuation of https://github.com/RodZill4/material-maker/pull/508 and includes various fixes(naming mostly) and adds missing documentation for all 2d sdf nodes.

Excellent! Definitely in next release. :)

Added sdArrow node
https://user-images.githubusercontent.com/830253/236122454-7ce12a88-be7c-4c18-8f1b-20749db3e385.mp4

Finalized, no more changes planned.
In summary this adds 20 2D SDF's to the library:

Arrow
Cross
Cut Disk
Egg
Equilateral Triangle
Heart
Hexagram
Isosceles Triangle
Moon
Parallelogram
Pie
Rounded Box
Rounded Cross
Rounded X
Regular Star
5-point Star
Trapezoid
Triangle
Uneven Capsule
Versica


Add LoopiaSE as provider

Add support for Loopia(.se), follows the same schema as Google Domains

Loopia also operates in Norway, UK, Serbia and likely uses the same platform there but it's not something I can confirm.

Fixed, thanks

Thanks for your contribution!


GaussianBlur with UseOptimized 64 bit adds a band to the top

Summary of your issue
When doing a GaussianBlur on a 32F1C Mat (32.0 sigma), I get a banding effect at the top of my resulting image when Cv2.SetUseOptimized(true) and using 64 bit code
Environment
Windows 7, .NET 4.6.1, OpenCvSharp <IP_ADDRESS>80319
What did you do when you faced the problem?
Was able to work around it by ensuring UseOptimized(false) before the call to GaussianBlur
Banding goes away if using Emgu, or if you set "prefer 32bit code" to true
Example code:
var mat2 = new Mat(@"d:\aa.tif");

Mat floatImage2 = new Mat();
mat2.ConvertTo(floatImage2, MatType.CV_32FC1);
floatImage2 = new Mat(floatImage2, new Rect(0, 0, image.Image.PixelWidth, image.Image.PixelHeight));

Cv2.SetUseOptimized(true);
var blurredImage2 = floatImage2.GaussianBlur(new Size(101, 101), 32.0, 32.0);
blurredImage2.SaveImage(@"d:\blurred2Opt.png");
Cv2.SetUseOptimized(false);
var blurredImage = floatImage2.GaussianBlur(new Size(101, 101), 32.0, 32.0);
blurredImage.SaveImage(@"d:\blurred2NoOpt.png");

Output:
left image

What did you intend to be?
right image

I translated your program into original OpenCV (C++) and tried it. I could reproduce your problem.
When I upgraded the version of OpenCV from 3.4.1 to 3.4.3, the problem seems to be solved.
int main()
{
    cv::Mat mat = cv::imread(R"(C:\lenna.png)", cv::IMREAD_GRAYSCALE);
    cv::Mat floatImage;
    mat.convertTo(floatImage, CV_32FC1);

    cv::setUseOptimized(true);
    cv::Mat blurredImage1;
    cv::GaussianBlur(floatImage, blurredImage1, cv::Size(101, 101), 32.0, 32.0);
    cv::imwrite(R"(C:\temp\blur1.png)", blurredImage1);

    cv::setUseOptimized(false);
    cv::Mat blurredImage2;
    cv::GaussianBlur(floatImage, blurredImage2, cv::Size(101, 101), 32.0, 32.0);
    cv::imwrite(R"(C:\temp\blur2.png)", blurredImage2);

    return 0;
}

It seems that OpenCV4 will be released shortly, so I am planning to not update to 3.4.3.


Reference to use of paper certificate for validation unclear / not always true

Where to find the issue
https://www.coronawarn.app/en/faq/#vac_cert_verification "How is the proof of vaccination verified?"
which says:
"Alternatively, proof with the paper-based vaccination document is possible."

https://www.coronawarn.app/de/faq/#vac_cert_verification "Wie soll der Nachweis einer Impfung geprüft werden können?"
which says:
"Alternativ ist ein Nachweis mit dem analogen Impfausweis möglich."

Describe the issue

In the English version it is not clear in the article what type of paper-based vaccination document is being described. It could be a paper-based digital vaccination certificate or it could be the yellow booklet "International Certificates of Vaccination / Internationale Bescheinigungen über Impfungen und Impfbuch".
In both language versions, according to issue https://github.com/corona-warn-app/cwa-website/issues/1805 Berlin only accepts a digital vaccination certificate, so it is not always true that a non-digital vaccination certificate will be accepted.

Suggested change
The Corona-Warn-App does not use the yellow vaccination document at all, so in my view it is better not to make statements about it because this is the responsibility of other authorities.
Remove the sentence:

"Alternatively, proof with the paper-based vaccination document is possible."
"Alternativ ist ein Nachweis mit dem analogen Impfausweis möglich."

otherwise it will be necessary to state clearly what type of paper-based document is meant, that it is not always accepted, that holder will need to check in advance depending on where they intend to present their proof of vaccination and that they can get a digital certificate from a pharmacy.

The FAQ articles referred to here have not yet been changed:

https://www.coronawarn.app/en/faq/#vac_cert_verification "How is the proof of vaccination verified?"
https://www.coronawarn.app/de/faq/#vac_cert_verification "Wie soll der Nachweis einer Impfung geprüft werden können?"

although in PR #1811 it says

Fixes #1805 and #1810

This issue here (#1810) could be fixed by adding a link from the above FAQs to the updated FAQs from PR #1811:

https://www.coronawarn.app/en/faq/#vac_cert_voluntary "Am I obliged to use the digital certificate of vaccination in the Corona-Warn-App? Will there still be the yellow paper-based vaccination booklet? Can I continue to use my yellow vaccination booklet? What about people who don't own a smartphone?"
https://www.coronawarn.app/de/faq/#vac_cert_voluntary "Bin ich verpflichtet, den digitale Impfnachweis in der Corona-Warn-App zu nutzen? Wird es trotzdem noch den gelben analogen Impfausweis geben? Kann ich meinen gelben Impfausweis auch weiterhin benutzen?

I will submit a PR to do this.

I expect this issue to be resolved by:

[ ] #2083

Resolved through PR #2083.


ObservableList doesn't override clear(), which removes all the items without notifying observers

When calling clear() on an ObservableList, the ListMixin implementation is called:
    void clear() { this.length = 0; }

This removes all the items, without notifying the observers.
This is breaking the ObservableList contract which states that the observers are notified if any items are added, removed or replaced.

Setting length to 0 will notify observers, right?
https://github.com/dart-lang/observe/blob/8c6e8f958e6c6931215a541838d79519975d62b9/lib/src/observable_list.dart#L72

Oh, then it means I have a bug in my code :-(. Let me double check tomorrow, but I think you are right.

The problem was in my code, indeed. Sorry for the trouble, and thanks for your help!


Discrepancy between alerts firing in prometheus and those reported in alertmanager

Bug Description
In a fresh deployment of an environment, we have a COS with Charmed K8s and Charmed Kubeflow.
Currently we see 13 alerts firing in prometheus, however only 4 show up in alertmanager.
To Reproduce
N/A
Environment
This is with Charmed K8s 1.29/beta and Charmed Kubeflow 1.8/stable on top of Juju 3.1.6
App                       Version  Status  Scale  Charm                         Channel        Rev  Address         Exposed  Message
alertmanager              0.25.0   active      1  alertmanager-k8s              stable          96  REMOVED   no
catalogue                          active      1  catalogue-k8s                 stable          31  REMOVED   no
grafana                   9.2.1    active      1  grafana-k8s                   stable          93  REMOVED  no
loki                      2.7.4    active      1  loki-k8s                      stable         105  REMOVED  no
prometheus                2.47.2   active      1  prometheus-k8s                stable         156  REMOVED   no
prometheus-scrape-config  n/a      active      1  prometheus-scrape-config-k8s  latest/stable   44  REMOVED  no
traefik                   2.10.4   active      1  traefik-k8s                   stable         166  REMOVED  no

Relevant log output
N/A

Additional context
No response

This might be due to https://github.com/canonical/alertmanager-k8s-operator/issues/207; if not, we should investigate further :)

Hi @nishant-dash,
Now with #212 merged, would you be able to re-test with revision 98 from edge?

it seems like upgrading to rev 98 did the trick,  almost certain because of https://github.com/canonical/alertmanager-k8s-operator/issues/207 like @lucabello mentioned


Add TOC to topic list page

Should be collapsed on mobile view.
Should be expanded on desktop view.

Experimented with it.
But this would steal space from content.
For now, the "TOC" is sufficient as-is.
In the future, I have to think about a general UI overhaul if this comes up again.


strictOrder() not honored when using Mock Name (aka namespace, scope)

Hey Bas / James / others:
In the midst of authoring some tests, I came across an unexpected situation. It seems as though strictOrder() is not honored when a mock name is supplied. (And yes, I am installing the MockPlugin in my main).
Is this expected behavior?
If this is a bug, I'd even be willing to help fix this, if that is of interest. Is there something you can point me to as a place to start?
I'll let my tests do the talking from here:
TEST_GROUP(FooTests)
{
};

TEST(FooTests, StrictOrderViolatedWhenNotUsingMockName)
{
	mock().strictOrder();

	mock().expectOneCall("A");
	mock().expectOneCall("B");

	mock().actualCall("B");
	mock().actualCall("A");

	// THIS FAILS AS EXPECTED
}

TEST(FooTests, StrictOrderViolatedWhenUsingMockName)
{
	mock("foo").strictOrder();

	mock("foo").expectOneCall("A");
	mock("foo").expectOneCall("B");

	mock("foo").actualCall("B");
	mock("foo").actualCall("A");

	// THIS DOES NOT FAIL AS EXPECTED
}

Update:
Just to clarify, I don't leave failing tests laying around or anything -- I just always start my TDD cycle with a failure :)
So I actually just tried the following (see below), and things now work. Perhaps the issue is with the mock plugin not checking certain expectations for things that use mock names?
I poked around in tests/CppUTestExt/MockSupportTest.cpp and actually saw that CppUTest covers itself for similar behavior. Not sure what I am missing. The [MockPlugin section of the docs] (https://cpputest.github.io/mocking_manual.html#mock_scope) state that using the MockPlugin calls "checkExpectations at the end of every test (on global scope, which goes recursive over all scopes)".
TEST_GROUP(FooTests)
{
};

TEST(FooTests, StrictOrderViolatedWhenNotUsingMockName)
{
	mock().strictOrder();

	mock().expectOneCall("A");
	mock().expectOneCall("B");

	mock().actualCall("B");
	mock().actualCall("A");
}

TEST(FooTests, StrictOrderViolatedWhenUsingMockName)
{
	mock("foo").strictOrder();

	mock("foo").expectOneCall("A");
	mock("foo").expectOneCall("B");

	mock("foo").actualCall("B");
	mock("foo").actualCall("A");

	mock("foo").checkExpectations();	// <-- added this line, test now fails as expected
}

I think the strictOrder only works on the same namespace. It is the same with other calls...

yes, but the guide statement After installing the plugin, you don’t have to worry too much anymore about calling checkExpectations or cleaning your MockSupport. still might lead the confusion, may be adding some additional remarks is good.

@basvodde I suspect this might be a bug in the Plugin; does it really call checkExpectations on all namespaces? Or just on the gloabl namespace?

For now, my workaround is to simply place mock("foo").checkExpectations(); after each test that i care about strict ordering for a certain namespace.

@arstrube , the code inside MockSupportPlugin::postTestAction() has said the check expectations only for the global namespace, no?
    if (!test.hasFailed())
        mock().checkExpectations();

I think you can omit using checkExpections explicit by creating the same namespace mock plugin with your test and install it.

Yeah, you guys are right. This is definitively wrong.

@basvodde - I took a look at the code. In the case of mock().clear(), simple recursion is taking place.
In the case of mock().checkExpectations() though, yes, it is called recursively across namespaces, but it's all somehow tied to and conditional on lastActualFunctionCall so it's easy to see that it might not work across namespaces for some reason.
However, I find it hard to understand this whole lastActualCall business -- I mean, why not simply check expectations across namespaces recursively and be done?

@matthewpiatt I believe this has been fixed and can be closed?

@arstrube yes. I tested my original snippets against a pull of 434496a and things run as expected. thanks for fixing this!


[Bug]: New Heroku deployment crashes

Is there an existing issue for this?

[X] I have searched the existing issues

Description
I've been trying to use Appsmith self-hosted for the first time. I followed every step to deploy on Heroku. However when I try to deploy and run the app it crashes with the following error:
2023-09-11T14:39:29.428377+00:00 heroku[web.1]: Starting process with command `/usr/bin/supervisord -n`
2023-09-11T14:39:30.231895+00:00 app[web.1]:   % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
2023-09-11T14:39:30.231921+00:00 app[web.1]:                                  Dload  Upload   Total   Spent    Left  Speed
2023-09-11T14:39:30.495617+00:00 app[web.1]:
2023-09-11T14:39:30.495619+00:00 app[web.1]: {
2023-09-11T14:39:30.495620+00:00 app[web.1]:   "success": true
2023-09-11T14:39:30.516568+00:00 app[web.1]: }nginx: [warn] the "user" directive makes sense only if the master process runs with super-user privileges, ignored in /etc/nginx/nginx.conf:1
2023-09-11T14:39:30.529555+00:00 app[web.1]: nginx: [emerg] bind() to <IP_ADDRESS>:80 failed (13: Permission denied)
2023-09-11T14:39:30.610089+00:00 heroku[web.1]: Process exited with status 1
2023-09-11T14:39:30.640284+00:00 heroku[web.1]: State changed from starting to crashed

Steps To Reproduce
Follow steps here: https://docs.appsmith.com/getting-started/setup/installation-guides/heroku
Public Sample App
No response
Environment
Production
Issue video log
No response
Version
Cloud / Self Hosted

@ric2z thanks for the report. We're looking into this and we'll come up with a solution. Hope you can still run Appsmith with one of our other deployment modes


Client still accessible but keylogger has stopped

I've been testing quasarrat for a while now, and after almost exactly 10 days the keylogger is no longer outputting any new data.  I know the PC is in use, remote desktop surveillance, etc. all work.  Is there any way to reset the program remotely to try and fix this?  Will remotely executing the executable again work?
Update:  I had local access to the desktop in question so I just closed the process and ran the executable again, it's working now.  Would still like to know why it stopped in the first place.  I know the user sometimes uses a japanese keyboard (in windows) but it never had problems handling that input before (since the key presses are the same regardless).
Update 2:  Just noticed today that the system clock on the PC seems to be failing, it was showing a time quite a bit behind what it should be.  Would this screw up the keylogger?

It's possible that the failing system clock caused this issue. It's a bit hard for me to debug it cause of the 10 days runtime needed. Maybe you can find out if it's working with corrected system clock. Japanese keyboard should be no problem.

Nope, nothing to do with the clock

I'm assuming now that maybe it was writing to a log file from previous day or something if the clock was actually off, it wasn't something I was paying attention to at the time so I can't really check it now.  It hasn't had any problems since I reset it though so I'm not sure what the issue was.  I know this wasn't much info to go on, I was mostly just posting this to see if anyone had any thoughts.  One thing I should note was that the computer was put in sleep mode every day and not shut down or reset, and it also suffered from some pretty severe network connectivity issues, but I don't think either of those would have affected it (The logging stopped in the middle of a day).

Another update:  I don't think this is the same issue as last time, but when I tried downloading logs today, the window would never update.  I checked with the file manager and the log file on the client pc was at 0 bytes, even though I know it had data as of the previous day (in the early AM).  I deleted the file and logging started working again.  Sadly I lost the data between then and last I checked but it wasn't a big deal.

So this keeps happening and I'm not sure what's causing it, the logs never show anything that would be unusual keyboard activity right before it stops logging.  Since it's such a nebulous problem I had two suggestions, the first would be a way to get the PID of the process on the remote machine (probably something to implement in an enhanced task manager), and the second would be a way to restart the process remotely.  Right now my method is to find the PID and put it in a script that kills the current process and runs the executable again, it would be nice if this was a feature built into quasar just in case the program stops functioning client side.


Force overcomes rigidbodies

Depending on the shape of the collider, a high mass and gravity pushes it through a static collider. I assume, that happens with other forces as well. I created a little test, see attachment.
MassTest.zip
Manipulate lines 33 and 38 of Internal.json to adjust start conditions, adjust mass with the input field in the upper left corner at runtime when the body is at rest.
This is a problem, since creating e.g. airplanes or tanks with realistic masses cannot park on the ground...

The simulation can be improved by increasing the solverIterations of the physics scene, or choose a different correction algorithm as described in the physics settings. The defaults are not good enough when it comes to calculate complex simulations where forces overcome a threshhold within one frame since there is no continous collision but discrete steps.
The  quality does reach it's limits, i noticed, at around 50k iterations vs. 1000 tons, there is no increase in quality and other problems start to occure, especially in rotations. But these values would be enough for planes (maximum 640 tons and tanks 60 tons), where not too many objects are in the scene.
Now, that the precision in general is lacking, because of calculation shortcuts that allow it to not recalculate too many steps fully, but partical, is a caveat of OIMO that i mentioned initially in choosing the engine. There are certain "improvements" in the inner workings that allow it to be fast and accurate in most use cases, but nearly useless in others.
In case you can to make a realistic flight simulator based on accurate values, then i should probably look into bullet physics (binding library) again, or give physx (binding library) another try, since it got open sourced.
Within OIMO is suspect it's an issue of small values not being discarded, but kinda accumulate. With high mass, things like friction/restitution come close to 0, same goes for rotations (since when a supermassive body is falling of the edge it's also not correctly calculated in OIMO).
I will look into it, but i'm starting to suspect that you expect more from a simple javascript engine than it can handle, and than FUDGE initially was in need of.
I can see the potential problem here in the described use case, but there is the option of scaling the whole system to a smaller factor to circumenvent most of the issue.

I tinkered around with the solverIterations before and noticed the impact. I was hesitant to jank it up that much, expecting performance problems.
With "choosing a different algorithm" do you mean Physics.settings.defaultConstraintSolverType? That look applicable to joints.

settings.defaultCorrectionAlgorithm, it's either 0,1,2 for Baumgarte, Split-Impulse, non-linear Gaus-Seidel. Each with their own caveats. The jointConstraintType is only for joints, the other one is for general constraints.
OIMO is calling everything influencing the body outside of forces a constraint, since bodies "constrain" another body to not go through it. A contact is therefore a "contactConstraint" and if bodies are touching a default "PositionCorrectionAlgorithm" is used unless, the penetration is too deep, then it seems to always fallback to Split_Impulse.
The  fallback is influenced by
OIMO.Setting.alternativeContactPositionCorrectionAlgorithm, which is not exposed in FUDGE since it's normally not needed.
OIMO solver iterations is tricky, since it does not actually do full iterations of the whole world, which is one of the reasons the precision is lacking, it's doing pseudo steps per island (group of bodies). So increasing the iterations in any other engine is severly influencing the performance. In OIMO it can safely be cranked up by a lot more until you notice a better simulation and a worse performance.
As said OIMO is lacking, but is to this day, the best option for easy integration, since the way it's programmed are not so biased towards specific ways things need to be set up to work, but it takes a lot of shortcuts to achieve what it does, which result in wrong behaviour beyond a certain point, partically because of floating point  precision problems within javascript, which need to be caught but don't seem to be.
I will look into it, but it's really easier to work with small numbers, and scale whole scenes towards those. Meaning treat 10 maybe as 1 ton, same goes for restitutions/friction.

Update:
Currently not so sure anymore that it is an OIMO problem or a problem of the communication between OIMO and FUDGE.
Since the OIMO demo scenes seem stable with masses up to 300 tons * Math.random and also random restitutions. They also run on javascript so it's not a floating preceision problem, but they run only on OIMO+OIMO Debug Draw, so no outside communication with an external Framework/Engine.
Could be that FUDGE is somehow returning wrong values to OIMO that is introducing these problems, because of the interaction of transforms/rigidbodies.
Still investigating.

Update: I'm still on it, but this seems to be a tough one.
I can not reproduce it within the OIMO Demos no matter the weight, or shape.
I can reproduce it on FudgeBall_Example from April 2021, although with higher values than in the initial example.
Also it only happens on shapes that are not spheres. Also the player was unaffected while the "ball", as a box with heigh weight did fall through.
I can easily reproduce it in your example also on every shape that is not a sphere.
In all 3 i use the same OIMO Version i used in the demos.
Which leads to two conclusions, there are probably miscalculations within the implementation and as a result of it in the handling of the collision shape / inertia. That is not the case for spheres which can not be squished in their shape, they are always a perfect ball and therefore the inertia is always in the middle, which also makes a lot of physics calculations easier.

So when I understand you correctly, you can take my MassTest and run it with different versions of FUDGE with different results. In that case, it should be possible (e.g. using Git Bisect) to find the commit that introduced the problem.

Did that, but it was never worked the way OIMO Demos (Demos that are independent of Fudge, therefore no outside transform is interfering). There was always a problem above certain weights, only the weight was higher, going back to 2020-2021, because of that have to find the actual root of the problem.
Since it sits somewhere between OIMO and Fudge, and it's a shape specific problem, i at least get closer to solving it.
But since i can only spare so much time for this, it takes time, just wanted to keep you posted.

No haste...


create an auto incrementing table

In New-MySQLiteDBTable see how to create a table with an autoincrementing column.

https://www.sqlite.org/autoinc.html

I think I will not implement this and instead rely on the intrinsic ROWID property.


Fix minAllowed of VPA resource for vpn-seed-server containers.

How to categorize this PR?

/area networking
/area auto-scaling
/area robustness
/kind enhancement
What this PR does / why we need it:
Fix minAllowed of VPA resource for vpn-seed-server containers.
Initially, the minimum allowed memory requests for the reverse vpn containers seem to have been mixed up. The container hosting OpenVPN never uses 20 MB of memory while the envoy proxy container regularly occupies 40 MB. Therefore, this change switches the minimum settings to reduce the amount of autoscaling required. Thereby it should stabilize the VPN connection.
In addition to that, the initial memory requests of OpenVPN container and its metrics exporter (only in HA scenario) are scaled down to be closer to their needs.
Which issue(s) this PR fixes:
None.
Special notes for your reviewer:
As the highly-available VPN does not use envoy proxy there is not the same amount of autoscaling happening in the HA VPN. The additional container for metrics is not auto-scaled at all.
Release note:

The `vpn-seed-server` now has better minimum memory settings so that less auto-scaling should occur.

/cc @DockToFuture @axel7born

https://github.com/gardener/gardener/blob/bd60605aaa94e227992c84a8112c44fa8fbe42da/pkg/component/networking/vpn/seedserver/seedserver.go#L463
Shouldn't we change this one too?

/lgtm

/approve

/retest-required


Simple change for demo.

This is just a simple change to demo the triggering in Azure Pipelines.

/azp run java - keyvault


feat : 修复 java.naming.ClassNamingShouldBeCamelRule.rule.msg 中英文本不一致问题。

As shown in the following code：
messages.xml
<entry key="java.naming.ClassNamingShouldBeCamelRule.rule.msg">
    <![CDATA[类名使用UpperCamelCase风格，必须遵从驼峰形式，但以下情形例外：（领域模型的相关命名）DO / BO / DTO / VO / DAO]]>
</entry>

messages_en.xml
<entry key="java.naming.ClassNamingShouldBeCamelRule.rule.msg">
    <![CDATA[Class names should be nouns in UpperCamelCase except domain models: DO, BO, DTO, VO, etc]]>
</entry>

so，I want to fix a mistake：
<entry key="java.naming.ClassNamingShouldBeCamelRule.rule.msg">
    <![CDATA[Class names should be nouns in UpperCamelCase except domain models: DO, BO, DTO, VO, DAO, etc]]>
</entry>

 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.liangguizhao seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

 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.liangguizhao seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


Errors for manifest.bioimage.yaml

Hi again,
now the error has changed but I cannot really know where it comes from: https://github.com/deepimagej/models/runs/3963723727#step:7:34
Is it that I should call this one rdf.yaml as well?
Requirement already satisfied: zipp>=0.5 in /opt/hostedtoolcache/Python/3.7.12/x64/lib/python3.7/site-packages (from importlib-metadata->click<9.0.0,>=7.1.1->typer->bioimageio.spec==0.3.3.post5) (3.6.0)
Errors for manifest.bioimage.yaml
Traceback (most recent call last):
  File "/opt/hostedtoolcache/Python/3.7.12/x64/bin/bioimageio", line 8, in <module>
    sys.exit(app())
  File "/opt/hostedtoolcache/Python/3.7.12/x64/lib/python3.7/site-packages/typer/main.py", line 214, in __call__
    return get_command(self)(*args, **kwargs)
  File "/opt/hostedtoolcache/Python/3.7.12/x64/lib/python3.7/site-packages/click/core.py", line 1128, in __call__
    return self.main(*args, **kwargs)
  File "/opt/hostedtoolcache/Python/3.7.12/x64/lib/python3.7/site-packages/click/core.py", line 1053, in main
    rv = self.invoke(ctx)
  File "/opt/hostedtoolcache/Python/3.7.12/x64/lib/python3.7/site-packages/click/core.py", line 1659, in invoke
    return _process_result(sub_ctx.command.invoke(sub_ctx))
  File "/opt/hostedtoolcache/Python/3.7.12/x64/lib/python3.7/site-packages/click/core.py", line 1395, in invoke
    return ctx.invoke(self.callback, **ctx.params)
  File "/opt/hostedtoolcache/Python/3.7.12/x64/lib/python3.7/site-packages/click/core.py", line 754, in invoke
    return __callback(*args, **kwargs)
  File "/opt/hostedtoolcache/Python/3.7.12/x64/lib/python3.7/site-packages/typer/main.py", line 500, in wrapper
    return callback(**use_params)  # type: ignore
  File "/opt/hostedtoolcache/Python/3.7.12/x64/lib/python3.7/site-packages/bioimageio/spec/__main__.py", line 27, in validate
    pprint(summary["errors"])
KeyError: 'errors'
Error: Process completed with exit code 1.

Thank you!

0.3.3post6 is now available via pip


Bump Version to 0.3.0

This is required because the Publish Action on v.0.3.0 failed:

ERROR    HTTPError: 400 Bad Request from https://test.pypi.org/legacy/
File already exists. See https://test.pypi.org/help/#file-name-reuse
for more information.

This is an easy fix to the problem. In the long term we should solve it by reading the version from the tag. One approach is described in #25.

Closed as #25 was merged before.


Multi region deployment request

I am not sure if this is a miss on my side to get it to deploy in multiple regions or its a feature that is not available just yet but the ability to deploy this as multi region would be super helpful
following something like this from AWS
https://www.wellarchitectedlabs.com/reliability/200_labs/200_deploy_and_update_cloudformation/6_multi_region_deploy/
and or this
https://docs.aws.amazon.com/solutions/latest/multi-region-infrastructure-deployment/template.html

Hello @frankscalzo, and thanks for reaching out to the LZA team!
I believe you are looking for the enabledRegions property available in global-config.yaml. LZA is multi-region aware when this value is set to more than one region. Please note that any resources utilizing the deploymentTargets property will be automatically deployed to the secondary region if you do not have excludedRegions configured for those resources. Resources that use the region property (such as VPCs) will need to be defined in new configuration definitions in their respective config files to take advantage of the new region.
I hope this helps! Please let us know if there's anything more we can do to help on this issue.

that was it i missed that since there i not a great spot for clear docs
i had to disable/ ignore new region for  audit manager and detective while deploying thanks


WIP: Mesos Documentation Project Reorg Proposal

This is an initial proposal for reorganizing the Mesos Documentation.
Our goal is to satisfy the needs of Operators, Developers, and Contributors by:

Revising, restructuring, and expanding existing topics.
Authoring new topics, such as Quick Start and What is Mesos?.
Reorganizing the table of contents.
Providing role-specific views of the table of contents.

Closing since we have reorganized the docs in the meantime.


Websocket server shuts down connection as soon as browser starts polling

Godot version
4.0.dev
System information
Windows 11, NVIDIA 3060, Vulkan Mobile
Issue description
When running a web assembly from inside the browser, the web socket server will shut down the connection as soon as the client starts polling with the following error message: WebSocket connection to 'ws://<IP_ADDRESS>:4100/' failed: WebSocket is closed before the connection is established.
Simple code for client\server logic:
func start_network(server: bool):
	# Connect base signals to get notified of new client connections
	multiplayer.peer_connected.connect(_connected)

	# disconnections
	multiplayer.peer_disconnected.connect(_disconnected)

	var peer : WebSocketMultiplayerPeer = WebSocketMultiplayerPeer.new()

	if server:
		# Start listening on the given port.
		error = peer.create_server(PORT)
		if error != OK:
			print("Unable to start server, code: " + str(error))
			set_process(false)
		else:
			print("Server Unique ID: " + str(peer.get_unique_id()))
			print("Server is listening")
	else:
		error = peer.create_client("<IP_ADDRESS>:4100")
		if error != OK:
			console.log("Error message: " + str(error))
	multiplayer.multiplayer_peer = peer`

Steps to reproduce
Create a project and run as an HTML export.  This code was used to start the server\client:
func start_network(server: bool):
	# Connect base signals to get notified of new client connections
	multiplayer.peer_connected.connect(_connected)

	# disconnections
	multiplayer.peer_disconnected.connect(_disconnected)

	var peer : WebSocketMultiplayerPeer = WebSocketMultiplayerPeer.new()

	if server:
		# Start listening on the given port.
		error = peer.create_server(PORT)
		if error != OK:
			print("Unable to start server, code: " + str(error))
			set_process(false)
		else:
			print("Server Unique ID: " + str(peer.get_unique_id()))
			print("Server is listening")
	else:
		error = peer.create_client("<IP_ADDRESS>:4100")
		if error != OK:
			console.log("Error message: " + str(error))
	multiplayer.multiplayer_peer = peer`

Minimal reproduction project
Easy to duplicate

PS: Code blocks should use triple backticks like this (with an optional language name for syntax highlighting):
```gdscript
code here
```
I edited your post accordingly, but remember to do this in the future :slightly_smiling_face:

I'm having the exact same issue with a similar project of mine. Did anyone find a solution to this?


fix: use wrapped roundtripper for exceptions

Use the wrapped roundripper for IPs that are marked as exceptions.

Related Issue or Design Document

Checklist

[ ] I have read the contributing guidelines and signed the CLA.
[ ] I have referenced an issue containing the design document if my change introduces a new feature.
[ ] I have read the security policy.
[ ] I confirm that this pull request does not address a security vulnerability.
If this pull request addresses a security vulnerability,
I confirm that I got approval (please contact<EMAIL_ADDRESS>from the maintainers to push the changes.
[ ] I have added tests that prove my fix is effective or that my feature works.
[ ] I have added the necessary documentation within the code base (if appropriate).

Further comments

I'm working on a proper patch with tests

https://github.com/ory/x/pull/657


does not work at flutter-pi

Hi!
I'm using the [1.0.0-rc.4](https://pub.dev/packages/audioplayers/versions/1.0.0-rc.4) that works fine in an Linux ubuntu desktop.
but when I use it in a flutter-pi project, it shows me the below error:
[compositor] Preferred framebuffer depth for hardware cursor is not supported by flutter-pi.
[ERROR:flutter/lib/ui/ui_dart_state.cc(209)] Unhandled Exception: MissingPluginException(No implementation found for method setSourceUrl on channel xyz.luan/audioplayers)
#0      MethodChannel._invokeMethod (package:flutter/src/services/platform_channel.dart:165:7)
<asynchronous suspension>
#1      AudioPlayer.play (package:audioplayers/src/audioplayer.dart:112:5)
<asynchronous suspension>

Do I have to do something to work in raspberrypi by flutter-pi?
Note: sound is enabled in my raspberry pi.

No implementation found for method setSourceUrl

That is very interesting, it appears Flutter is not recognizing your flutter-pi as "linux" in order for audioplayers to use the correct binding.
Could you please try using any other flutter dependencies with native linux code?
I wonder if this is an issue with Flutter itself.

That is very interesting, it appears Flutter is not recognizing your flutter-pi as "linux" in order for audioplayers to use the correct binding.

But why this has to be happened?

Could you please try using any other flutter dependencies with native linux code? I wonder if this is an issue with Flutter itself.

What do you mean exactly? Do you mean I use audioplayers_linux package directly in my project? or use another flutter package with a native linux code?
if the second is right, I'm using other packages such as flutter_libserialport in my flutter-pi project and it is working fine.

or use another flutter package with a native linux code

yes that is what I meant. interesting, I don't understand why flutter_libserialport works and AP doesn't. I compared our configurations for the linux binding; the only difference is that libserialport does not use federation and we do. But that should not affect anything.
I just tried running our example (packages/audioplayers/example) on my Arch Linux and it worked. Could you please try clonning the repo and running that on your raspberry pi? I wonder if something else is causing interference.

Hey, I ran into a similar issue using flutter-pi and audioplayers package.

Flutter 3.19.5 stable
audioplayers: ^4.0.0, ^5.0.0 and ^6.0.0
Raspberry Pi 3B with flutter-pi
Raspbian 11 (Bullseye)

I tried my own code - running on all platforms except Linux using flutter-pi pretty fine - and I also deployed the audioplayers/example project using flutter-pi. The example's UI works properly, when pressing the play button the same error is thrown:
[ERROR:flutter/runtime/dart_vm_initializer.cc(41)] Unhandled Exception: MissingPluginException(No implementation found for method create on channel xyz.luan/audioplayers)
#0   MethodChannel._invokeMethod (package:flutter/src/services/platform_channel.dart:332) <asynchronous suspension>
#1   AudioplayersPlatform.create (package:audioplayers_platform_interfaces/src/audioplayers_platform.dart:18) <asynchronous suspension>
#2   AudioPlayer._create (package:audioplayers/src/audioplayer.dart:175) <asynchronous suspension>

In addition I checked the platform detected by flutter. The result is 'linux'.
While writing this comment, I found this issue in the flutter-pi repo which might explain the problems. I will try to understand what's happening there in the next days. But as I am new to the platform channel mechanic I can't promise any valuable results 😄


[NEW ALGORITHM] Adding Basic And Standard Bit manipulation Algorithms

##Description
I propose to add a new section on bit manipulation to the repository. This section will cover the basics of bit manipulation, include standard algorithms, and provide comprehensive documentation to aid understanding and usage.
Checklist:

[x] Contributor in GSSoC-ext

[x] Want to work on it

[x] I want to add a new section
Additional Information
Plan for Bit Manipulation Section:-

Introduction to Bit Manipulation
Explanation of bits and binary representation
Common bitwise operators (AND, OR, XOR, NOT, shifts)

Standard Algorithms
Counting the number of set bits (Hamming weight)
Finding the only non-repeating element in an array where every other element repeats twice
Detecting if a number is a power of two
Swapping two numbers without using a temporary variable

I am eager to take on this task and am confident in my ability to meet and exceed expectations.

Is this task assigned to me?
@pankaj-bind

@sumankr15 you can start working on it

no response


Home page content improvements from user tests

User story
As a ReportStream user, I want to understand if ReportStream can help me so I can decide what to do.
Background & context
User tests revealed some opportunities to improve clarity and usability of the home page, particularly around the CTA and how we help sections.
Open questions

Working links
Acceptance criteria

[x] Iterations drafted and approved by Mo
[x] Next steps planned

Document approved with changes and context


manual open/close?

I have a bit that was "click to open" > "click or mouseout to close" that I'm wanting to replace.
Is there a flag to set or a method that can be fired to manually open/close the item?
I expect not, so therefore, how might the plugin might be modified to do so?
Thanks either way.

I'll add this feature request to the next release.
Feature Description
Expose two new interface properties that define the showAction and hideAction of a tooltip. These properties will accept one of the following DOM event assignments:

showAction:  click, mousedown, mouseenter.
hideAction: click, mouseout, mouseleave.

That's pretty cool and useful.
Might it be simpler to make methods to call using and svelte/html events? That way it can be dealt with directly.
like for example

 tooltip.close()} />

I had that thought initially. I could combine all of them so everything is out there. You can be explicit with events or control behavior via callbacks

That sounds great. Thanks and well done.

After thinking about it, I'm just going to make the show prop bindable and you can control the initial or close behavior by just setting that to true or false

you'll have something like this now

 This tooltip should appear to the
  <Tooltip
    bind:show={isVisible}
    action="click"
    align="center"
    content="I should appear to the right."
    position="right">
    right
  </Tooltip>
  when clicked.

<button on:click={hideOnClick}>Close the tooltip</button>

One of the things I dislike about github is that notifications are very out of the way. I'm always missing stuff.
This looks fabulous thanks for your work!


Refactor to use keyStoreAgent, capabilityAgent, & other breaking changes

Tons of breaking changes are incoming to the demo-issuer which will require using keystoreAgent and capabilityAgent in place of controllerKey among other breaking changes to numerous functions.

@aljones15 the changes you are referencing are related to switching from webkms-client@1 to webkms-client@2.  This software does not interact with those APIs.  There is no webkms-client dep here.

Yeah my bad here, but regardless we do need to retest after next major changes:
  "dependencies": {
    "apisauce": "^1.1.1",
    "async-handler": "0.0.3",
    "body-parser": "^1.19.0",
    "did-context": "^2.0.0",
    "did-method-key": "^0.4.0",
    "ejvm": "^1.0.5",
    "express": "^4.17.1",
    "express-async-handler": "^1.1.4",
    "jsonld-signatures": "^4.6.0",
    "schema.org": "^3.1.1",
    "uuid-random": "^1.3.0",
    "vc-js": "^0.3.0"
  },

vc-js is already out of date I believe in self-signing.


18 make reports

#20 needs to get merged first - this built off a small CSS addition made in that branch.

Sorry I misclicked, meant to add as reviewer


Company Custom field page - Showing true/false instead of On/off in need attention view

Describe the bug

When some client make bool value on/off then it shows true/false in company custom field page it shoud show On/Off

True->On, False->Off

currently its ok
If client persists then will change


Swiftlint: non_optional_string_data_conversion

Enable non_optional_string_data_conversion and make the code comply to that.

Thanks for submitting a issue!
to assign this issue to yourself please comment @aurora-care-bear please assign me


SDKs should get the token exchange endpoint from the issuer's configuration

Originally reported by @stefanmilivojevic91 for the .NET SDK

Token endpoint path should be fetched from issuer's openid configuration (.well-known/openid-configuration) endpoint. AzureAD has oauth2/token for instance and OAuth2Client is unable to fetch access token using client credentials flow

[ ] JS SDK
[ ] Go SDK
[ ] Python SDK
[ ] .NET SDK: https://github.com/openfga/dotnet-sdk/issues/30
[ ] Java SDK

Instead of reading it from the OpenID discovery endpoint, we can let users configure it directly, to avoid querying the endpoint.

Duplicated by a newer issue: #238, closing in favour of that


Passcode creation

“Your app storage is encrypted by the passcode. You will not be able to log in if you forget the passcode and will have to recover your wallet using the recovery flow”
So recovery is done only if one have done the backup what if he has not done the backup ?
refer https://bithyve-workspace.slack.com/archives/C018J9CUR1N/p1693831901203969?thread_ts=1693829108.477109&channel=C018J9CUR1N&message_ts=1693831901.20396

duplicate https://github.com/bithyve/bitcointribe/issues/6743


use current node-version 16

What does this PR do?
Add resource(s) | Remove resource(s) | Add info | Improve repo
For resources
Description
Why is this valuable (or not)?
How do we know it's really free?
For book lists, is it a book? For course lists, is it a course? etc.
Checklist:

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

Followup

Check the output of Travis-CI for linter errors!

needs review

Worth noting you can tell the change doesn't break anything because the workflow for this very PR used Node 16.

Thanks, and welcome to free-programming-books!


[Hotfix] Move npm version to dockerfile #3109

Move npm version to dockerfile in rest server and webportal.

Coverage decreased (-0.01%) to 51.225% when pulling c02316a6448c83b9dd8e665b2bddefc4af13d4a0 on xiongyf/hotfix-npm-version into 89cce98b12a3a4bf48f954b8733ef7351ec79a9a on pai-0.14.y.


Polish C++ Wsman* classes

Hi,
I did some polishing work in C++ bindings.
What's new:

new class WsmanOptions
refactored classes:

WsmanFilter
WsmanEPR

simplified calls in OpenWsmanClient

Thanks for review.

Thanks a lot for giving the C++ bindings some love !
Please add a "License:" field to the header of newly added files.

Ah, thank you. Will do so.

Things changed:

added BSD license to newly added files
did some code cleanup in (Open)WsmanClient to make the code look more C++-ish

Thanks Peter.
However, "BSD" is not sufficient. Please make it "BSD-3-Clause" to match the Openwman license.

Changed to BSD-3-Clause.

Perfect. Thanks a lot for your contribution !

These changes look great. Thanks Peter!
Klaus, are you planning any release soon that will include these changes (v2.4.13)?


Reduce number of points of interest for testing

One idea to increase performance for testing on local machines (laptops) would be to take a limited number of points of interest for each file/scan. You could randomly select say a 100 or 1000 points of interest from each file. In this way you can test all scans and also have a nice diversity of data.

Done in #91


chore(multicluster): Remove cluster domain

Signed-off-by: Allen Leigh<EMAIL_ADDRESS>
Description:
Remove clusterDomain configuration and usages.

Affected area:

Functional Area

New Functionality
[X]

Documentation
[ ]

Install
[ ]

CLI Tool
[ ]

Certificate Management
[ ]

Control Plane
[ ]

Ingress
[ ]

Egress
[ ]

Networking
[ ]

Observability
[ ]

SMI Policy
[ ]

Sidecar Injection
[ ]

Security
[ ]

Upgrade
[ ]

Tests
[ ]

CI System
[ ]

Demo
[ ]

Performance
[ ]

Other
[ ]

Please answer the following questions with yes/no.

Does this change contain code from or inspired by another project?

Did you notify the maintainers and provide attribution?

No

Is this a breaking change?

No

Codecov Report

Merging #3854 (cc02902) into main (1c0f8ff) will increase coverage by 0.08%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main    #3854      +/-   ##
==========================================
+ Coverage   67.73%   67.82%   +0.08%
==========================================
  Files         199      199
  Lines       11088    11063      -25
==========================================
- Hits         7511     7503       -8
+ Misses       3528     3511      -17
  Partials       49       49

Flag
Coverage Δ

unittests
67.82% <100.00%> (+0.08%)
:arrow_up:

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

Impacted Files
Coverage Δ

pkg/envoy/lds/inmesh.go
78.59% <ø> (-1.06%)
:arrow_down:

pkg/catalog/outbound_traffic_policies.go
85.00% <100.00%> (-0.24%)
:arrow_down:

pkg/catalog/service.go
90.38% <100.00%> (-0.27%)
:arrow_down:

pkg/envoy/eds/response.go
42.85% <100.00%> (-1.01%)
:arrow_down:

pkg/envoy/registry/services.go
95.42% <100.00%> (-0.08%)
:arrow_down:

pkg/envoy/secrets/secrets.go
64.86% <100.00%> (-0.93%)
:arrow_down:

pkg/providers/kube/client.go
65.00% <100.00%> (-0.22%)
:arrow_down:

pkg/service/service.go
100.00% <100.00%> (ø)

pkg/service/types.go
75.00% <100.00%> (+25.00%)
:arrow_up:

pkg/utils/service.go
100.00% <100.00%> (ø)

... and 10 more

Continue to review full report at Codecov.

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


Fixes #3564

JS : Added error image for broken/unloaded emoji

@rishig can you tell why the build failed?
Thanks for guidance


Snackbar transitionDuration enter, exit must both be set

[x] This is not a v0.x issue.
[x] I have searched the issues of this repository and believe that this is not a duplicate.

Expected Behavior 🤔

When using <Snackbar> with the transitionDuration param set to { exit: 0 } the snackbar should appear with its default enter duration (duration.enteringScreen), and exit with a duration of 0.
Current Behavior 😯

A warning is thrown:

Warning: Material-UI: argument "duration" must be a number or a string but found NaN.

Steps to Reproduce 🕹

Link: https://codesandbox.io/embed/create-react-app-n71s0

Create a <Snackbar> with the following prop: transitionDuration={{ exit: 0 }}
Observe console.

Context 🔦

The <Snackbar> API docs list the transitionDuration type as number | { enter?: number, exit?: number } which leads me to believe that it's possible to set either enter or exit but the warning is thrown if only one of them are set.
I think we can change the docs to read number | { enter: number, exit: number } or change the implementation to check for the value and use the default if not set on each of enter and exit.
Your Environment 🌎

Tech
Version

Material-UI
v4.2.1

React
16.9.0

Browser
Chrome Version 75.0.3770.142 (Official Build) (64-bit)

TypeScript
3.4.1

@danada Thanks for raising the problem. The TypeScript type is imported from https://reactcommunity.org/react-transition-group/transition#Transition-prop-timeout. I think that it would make sense to fix the implementation. What do you think of defaulting to 0?
diff --git a/packages/material-ui/src/transitions/utils.js b/packages/material-ui/src/transitions/utils.js
index c43f9da67..963865f64 100644
--- a/packages/material-ui/src/transitions/utils.js
+++ b/packages/material-ui/src/transitions/utils.js
@@ -5,7 +5,7 @@ export function getTransitionProps(props, options) {

   return {
     duration:
-      style.transitionDuration || typeof timeout === 'number' ? timeout : timeout[options.mode],
+      style.transitionDuration || typeof timeout === 'number' ? timeout : timeout[options.mode] || 0,
     delay: style.transitionDelay,
   };
 }

It looks like defaulting to 0 is pretty standard. Proposed change looks good :smiley:

@danada Is this something you would like to contribute? :)

If you don't mind waiting till Monday I'll submit a PR ☺

@oliviertassinari take a look at #16839

@danada Oops, I had submitted one already.

@danada Oops, I had created one already.


Coverage doesn't load on Firefox

I'm trying to use the Firefox addon but the coverage doesn't load. I see "Loading coverage report..." on the page, but that's it. Checking the Network tab, I see that drop.min.js tries to load from Cloudflare, but is blocked due to CSP.
Addon version 0.2.0, Firefox 113.0.2 (64-bit), Ventura (13.4). uBlock is disabled on Github.

Thanks for letting us know, we don't see this issue in Chrome, but can reproduce in firefox. So we will work on this and get a fix out in our next release.

@bmulholland I believe we've shipped an update that we think will address this problem. Are you still seeing issues?

Yes, same issue. Now there's no network request in devtools Network tab, but console does show "Error: NetworkError when attempting to fetch resource." No other info is output.

Experienced same as @bmulholland on 0.2.1 =)

What I noticed afterwards is that there was an unset permission in the extension settings for "Access your data for https://api.codecov.io". After enabling, it changed from "Loading coverage report" to "Coverage report not available" and no longer had the network error

Also seeing the below white space for preferences but that might be unrelated!

Oh nice find. I can confirm I never saw a prompt for that, and that after enabling that option I also see "report not available." I still see no requests to Codecov in the Network tab of devtools.

Okay actually that looks to be happening because Codecov isn't actually receiving my coverage at all. It's saying we're throttled because we're on a free plan (though we're actually paying...). It's also silently failing in CI. Not cool. Will write support.

I'm not sure what ye changed over the last few hours but it's now working for me so thanks =)

What I noticed afterwards is that there was an unset permission in the extension settings for "Access your data for https://api.codecov.io/". After enabling, it changed from "Loading coverage report" to "Coverage report not available" and no longer had the network error

Both these permissions are actually specified as required, so it's very strange that they don't show up during installation and instead here under optional permissions.
I can reproduce this locally, and also see a (old) comment on Mozilla Bugzilla that mentions this bug, so will do some more digging to find the root cause.

I got the issue with my Codecov license resolved and can now see coverage report stats on PRs. I don't see inline coverage indicators though, is that supposed to work?

@bmulholland are you a dark mode user? I'm noticing that I do not get PR line coverage in dark mode. We're currently discussing this in #26

Issue appears to be resolved, inline coverage indicators on PR issue is tracked in #26, so we will be closing this.

Thank you! No, I'm not a dark mode user.

@eliatcodecov No, it was not fixed, #50 shows it and I just had it now fresh too.


Connect Power BI to Azure Blob Storage Account

[Enter feedback here]
There is a note which states that PBI service must be located in the same region as the Blob Storage account in order for PBI to connect. This does not seem to be accurate since PBI should be able to connect to Blob Storage accounts in different regions. If this note is referring to the use of an Azure trial account, then it would be best to change the wording to point this out.

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

ID: 4a352aea-fca9-8a16-f5f7-533244f037ef
Version Independent ID: 4a352aea-fca9-8a16-f5f7-533244f037ef
Content: Azure Blob Storage - Power Query
Content Source: powerquery-docs/connectors/azure-blob-storage.md
Service: powerquery
GitHub Login: @bezhan-msft
Microsoft Alias: bezhan

Hi @Friedduc
Thanks for providing feedback that helps improve our documentation. This issue has been resolved and we are closing the issue. #please-close


Chore: Use illustrations from artwork repository

Fix #518
Pull request checklist
Make sure you:

[x] Signed the Contributor License Agreement
[x] Followed the commit message guidelines

Short description of the change(s)

Added postinstall.js
Added postinstall script to package.json
Added @hint/artwork dependency
Removed files from repo that would be copied from @hint/artwork during postinstall
Added .gitignore for files that would be copied from @hint/artwork during postinstall

Looks like I have some code style changes to make for the build.

@RoryDuncan Thank you!

@RoryDuncan Changes are now live, thanks again!


czprint-fn, zprint-fn don't work for fns with specs in 0.5.0

The various -fn and -fn-str variants don't work for fns which have specs in zprint 0.5.0, because while {:style :spec] was deprecated, it was still used internally in one place.  The temporary workaround for this is to use an options map {:spec {:docstring? false}}, which will inhibit the addition of specs to docstrings.

Fixed in 0.5.1.


Fix "return_statements" issue in src/Controllers/Reports/GenerateReportController.php

From what I can tell most of the multiple return statements have to do with the various array_map functions being used, returning their values to the assigned variable. I did refactor the createResponse method to have only a single return statement.   To do so I assign the formatted response to a variable $response and then return that variable.

Thank you for your contribution @mikemiles86 !


Favicons

I haven't used Eleventy before so thought a Favicon PR would let me have a look through the code and see how it all works. I've also added .DS_Store to the .gitignore and fixed a typo on the styleguide.

@trys This is perfect, thanks for contributing!


Improve default azure.yaml when azd init empty template is run

Run azd init empty template and you get this azure.yaml
# yaml-language-server: $schema=https://raw.githubusercontent.com/Azure/azure-dev/main/schemas/v1.0/azure.yaml.json

name: TheCatSaidNo
infra:
    provider: ""
    path: ""
    module: ""
pipeline:
    provider: ""

Let's get 'services' and potentially other useful info in there.

We should also investigate improving the existing directory structure (e.g. generate infra/)

Related #135

Closing this in favor of #135


1.6.10 - solr rebuild_index / update_index error

UPDATE:
@zhiyuli - Decided to scrap the database and start over with the copy of www database from Sunday which seemed to deploy as expected in regards to solr rebuild index and update index.
The issue denoted before was in regards to drew collection 1, of which there are two versions of. Neither would render as valid end points previously, they now seem to resolve on alpha.

Using copy of www resources (as seen on alpha.hydroshare.org) there is an error in reference to BaseResource: drew collection 1, @zhiyuli may have an idea...
./hsctl rebuild_index
[hydrobuild@alpha hydroshare]$ ./hsctl rebuild_index
/usr/local/lib/python2.7/site-packages/mezzanine/utils/conf.py:55: UserWarning: TIME_ZONE setting is not set, using closest match: Etc/UTC
  warn("TIME_ZONE setting is not set, using closest match: %s" % tz)
/usr/local/lib/python2.7/site-packages/django/contrib/contenttypes/models.py:161: RemovedInDjango19Warning: Model class django.contrib.contenttypes.models.ContentType doesn't declare an explicit app_label and either isn't in an application in INSTALLED_APPS or else was imported before its application was loaded. This will no longer be supported in Django 1.9.
  class ContentType(models.Model):

/usr/local/lib/python2.7/site-packages/autocomplete_light/widgets.py:19: RemovedInDjango19Warning: The django.forms.util module has been renamed. Use django.forms.utils instead.
  from django.forms.util import flatatt

/usr/local/lib/python2.7/site-packages/autocomplete_light/fields.py:8: RemovedInDjango19Warning: django.contrib.contenttypes.generic is deprecated and will be removed in Django 1.9. Its contents have been moved to the fields, forms, and admin submodules of django.contrib.contenttypes.
  from django.contrib.contenttypes.generic import GenericForeignKey

System check identified some issues:

WARNINGS:
?: (mezzanine.core.W01) Please update your settings to use the TEMPLATES setting rather than the deprecated individual TEMPLATE_ settings. The latter are unsupported and correct behaviour is not guaranteed. Here's a suggestion based on on your existing configuration:

TEMPLATES = [{u'APP_DIRS': True,
              u'BACKEND': u'django.template.backends.django.DjangoTemplates',
              u'DIRS': (u'/home/docker/hydroshare/hydroshare/templates',),
              u'OPTIONS': {u'builtins': [u'mezzanine.template.loader_tags'],
                           u'context_processors': (u'django.contrib.auth.context_processors.auth',
                                                   u'django.contrib.messages.context_processors.messages',
                                                   u'django.core.context_processors.debug',
                                                   u'django.core.context_processors.i18n',
                                                   u'django.core.context_processors.static',
                                                   u'django.core.context_processors.media',
                                                   u'django.core.context_processors.request',
                                                   u'django.core.context_processors.tz',
                                                   u'mezzanine.conf.context_processors.settings',
                                                   u'mezzanine.pages.context_processors.page')}}]

?: (mezzanine.core.W02) TEMPLATE_DEBUG and DEBUG settings have different values, which may not be what you want. Mezzanine used to fix this for you, but doesn't any more. Update your settings.py to use the TEMPLATES setting to have template debugging controlled by the DEBUG setting.
hs_tools_resource.SupportedResTypes.supported_res_types: (fields.W340) null has no effect on ManyToManyField.
Removing all documents from your index because you said so.
All documents removed.
Indexing 377 Generics
Traceback (most recent call last):
  File "manage.py", line 8, in <module>
    execute_from_command_line(sys.argv)
  File "/usr/local/lib/python2.7/site-packages/django/core/management/__init__.py", line 354, in execute_from_command_line
    utility.execute()
  File "/usr/local/lib/python2.7/site-packages/django/core/management/__init__.py", line 346, in execute
    self.fetch_command(subcommand).run_from_argv(self.argv)
  File "/usr/local/lib/python2.7/site-packages/django/core/management/base.py", line 394, in run_from_argv
    self.execute(*args, **cmd_options)
  File "/usr/local/lib/python2.7/site-packages/django/core/management/base.py", line 445, in execute
    output = self.handle(*args, **options)
  File "/usr/local/lib/python2.7/site-packages/haystack/management/commands/rebuild_index.py", line 26, in handle
    call_command('update_index', **options)
  File "/usr/local/lib/python2.7/site-packages/django/core/management/__init__.py", line 120, in call_command
    return command.execute(*args, **defaults)
  File "/usr/local/lib/python2.7/site-packages/django/core/management/base.py", line 445, in execute
    output = self.handle(*args, **options)
  File "/usr/local/lib/python2.7/site-packages/haystack/management/commands/update_index.py", line 183, in handle
    return super(Command, self).handle(*items, **options)
  File "/usr/local/lib/python2.7/site-packages/django/core/management/base.py", line 623, in handle
    label_output = self.handle_label(label, **options)
  File "/usr/local/lib/python2.7/site-packages/haystack/management/commands/update_index.py", line 188, in handle_label
    self.update_backend(label, using)
  File "/usr/local/lib/python2.7/site-packages/haystack/management/commands/update_index.py", line 233, in update_backend
    do_update(backend, index, qs, start, end, total, verbosity=self.verbosity, commit=self.commit)
  File "/usr/local/lib/python2.7/site-packages/haystack/management/commands/update_index.py", line 96, in do_update
    backend.update(index, current_qs, commit=commit)
  File "/usr/local/lib/python2.7/site-packages/haystack/backends/solr_backend.py", line 56, in update
    docs.append(index.full_prepare(obj))
  File "/usr/local/lib/python2.7/site-packages/haystack/indexes.py", line 212, in full_prepare
    self.prepared_data = self.prepare(obj)
  File "/usr/local/lib/python2.7/site-packages/haystack/indexes.py", line 203, in prepare
    self.prepared_data[field.index_fieldname] = field.prepare(obj)
  File "/usr/local/lib/python2.7/site-packages/haystack/fields.py", line 159, in prepare
    return self.convert(super(CharField, self).prepare(obj))
  File "/usr/local/lib/python2.7/site-packages/haystack/fields.py", line 103, in prepare
    raise SearchFieldError("The model '%s' combined with model_attr '%s' returned None, but doesn't allow a default or null value." % (repr(obj), self.model_attr))
haystack.exceptions.SearchFieldError: The model '<BaseResource: drew collection 1>' combined with model_attr 'first_creator' returned None, but doesn't allow a default or null value.

./hsctl update_index
[hydrobuild@alpha hydroshare]$ ./hsctl update_index
/usr/local/lib/python2.7/site-packages/mezzanine/utils/conf.py:55: UserWarning: TIME_ZONE setting is not set, using closest match: Etc/UTC
  warn("TIME_ZONE setting is not set, using closest match: %s" % tz)
/usr/local/lib/python2.7/site-packages/django/contrib/contenttypes/models.py:161: RemovedInDjango19Warning: Model class django.contrib.contenttypes.models.ContentType doesn't declare an explicit app_label and either isn't in an application in INSTALLED_APPS or else was imported before its application was loaded. This will no longer be supported in Django 1.9.
  class ContentType(models.Model):

/usr/local/lib/python2.7/site-packages/autocomplete_light/widgets.py:19: RemovedInDjango19Warning: The django.forms.util module has been renamed. Use django.forms.utils instead.
  from django.forms.util import flatatt

/usr/local/lib/python2.7/site-packages/autocomplete_light/fields.py:8: RemovedInDjango19Warning: django.contrib.contenttypes.generic is deprecated and will be removed in Django 1.9. Its contents have been moved to the fields, forms, and admin submodules of django.contrib.contenttypes.
  from django.contrib.contenttypes.generic import GenericForeignKey

System check identified some issues:

WARNINGS:
?: (mezzanine.core.W01) Please update your settings to use the TEMPLATES setting rather than the deprecated individual TEMPLATE_ settings. The latter are unsupported and correct behaviour is not guaranteed. Here's a suggestion based on on your existing configuration:

TEMPLATES = [{u'APP_DIRS': True,
              u'BACKEND': u'django.template.backends.django.DjangoTemplates',
              u'DIRS': (u'/home/docker/hydroshare/hydroshare/templates',),
              u'OPTIONS': {u'builtins': [u'mezzanine.template.loader_tags'],
                           u'context_processors': (u'django.contrib.auth.context_processors.auth',
                                                   u'django.contrib.messages.context_processors.messages',
                                                   u'django.core.context_processors.debug',
                                                   u'django.core.context_processors.i18n',
                                                   u'django.core.context_processors.static',
                                                   u'django.core.context_processors.media',
                                                   u'django.core.context_processors.request',
                                                   u'django.core.context_processors.tz',
                                                   u'mezzanine.conf.context_processors.settings',
                                                   u'mezzanine.pages.context_processors.page')}}]

?: (mezzanine.core.W02) TEMPLATE_DEBUG and DEBUG settings have different values, which may not be what you want. Mezzanine used to fix this for you, but doesn't any more. Update your settings.py to use the TEMPLATES setting to have template debugging controlled by the DEBUG setting.
hs_tools_resource.SupportedResTypes.supported_res_types: (fields.W340) null has no effect on ManyToManyField.
Indexing 377 Generics
Traceback (most recent call last):
  File "manage.py", line 8, in <module>
    execute_from_command_line(sys.argv)
  File "/usr/local/lib/python2.7/site-packages/django/core/management/__init__.py", line 354, in execute_from_command_line
    utility.execute()
  File "/usr/local/lib/python2.7/site-packages/django/core/management/__init__.py", line 346, in execute
    self.fetch_command(subcommand).run_from_argv(self.argv)
  File "/usr/local/lib/python2.7/site-packages/django/core/management/base.py", line 394, in run_from_argv
    self.execute(*args, **cmd_options)
  File "/usr/local/lib/python2.7/site-packages/django/core/management/base.py", line 445, in execute
    output = self.handle(*args, **options)
  File "/usr/local/lib/python2.7/site-packages/haystack/management/commands/update_index.py", line 183, in handle
    return super(Command, self).handle(*items, **options)
  File "/usr/local/lib/python2.7/site-packages/django/core/management/base.py", line 623, in handle
    label_output = self.handle_label(label, **options)
  File "/usr/local/lib/python2.7/site-packages/haystack/management/commands/update_index.py", line 188, in handle_label
    self.update_backend(label, using)
  File "/usr/local/lib/python2.7/site-packages/haystack/management/commands/update_index.py", line 233, in update_backend
    do_update(backend, index, qs, start, end, total, verbosity=self.verbosity, commit=self.commit)
  File "/usr/local/lib/python2.7/site-packages/haystack/management/commands/update_index.py", line 96, in do_update
    backend.update(index, current_qs, commit=commit)
  File "/usr/local/lib/python2.7/site-packages/haystack/backends/solr_backend.py", line 56, in update
    docs.append(index.full_prepare(obj))
  File "/usr/local/lib/python2.7/site-packages/haystack/indexes.py", line 212, in full_prepare
    self.prepared_data = self.prepare(obj)
  File "/usr/local/lib/python2.7/site-packages/haystack/indexes.py", line 203, in prepare
    self.prepared_data[field.index_fieldname] = field.prepare(obj)
  File "/usr/local/lib/python2.7/site-packages/haystack/fields.py", line 159, in prepare
    return self.convert(super(CharField, self).prepare(obj))
  File "/usr/local/lib/python2.7/site-packages/haystack/fields.py", line 103, in prepare
    raise SearchFieldError("The model '%s' combined with model_attr '%s' returned None, but doesn't allow a default or null value." % (repr(obj), self.model_attr))
haystack.exceptions.SearchFieldError: The model '<BaseResource: drew collection 1>' combined with model_attr 'first_creator' returned None, but doesn't allow a default or null value.

Closing as the error doesn't seem to be reproducible on a clean deploy of the environment.


feedback: split screen gets replaced with cody chat

Areas for Improvement

[X] UI/UX
[ ] Onboarding
[ ] Docs
[X] Chat
[ ] Commands
[ ] Context
[ ] Response Quality
[ ] Other

What needs to be improved? Please describe how this affects the user experience.
If I have 2 files open side-by-side and I go to open a new Cody chat, one of my files in the split gets replaced with the Cody chat.
Why is this feedback important? Please describe the rationale.
This is a real blocker for me using Cody currently. I love everything else you've done. It's too bad of a UX for me to be able to use Cody with this issue. Every other option allows chat in the sidebar (possibly for this reason) - Phind, Codeium, Cursor, Copilot Chat. I can't understand this decision. Maybe I'm missing something?
Describe the solution you'd like to see
Allow the chat to be in the right or left side bar.
Describe any alternatives that could be considered
No response
Additional context
No response

Ah I can see this is a duplicate of https://github.com/sourcegraph/cody/issues/2828
I'll leave my comments there


Vehicle.RadioStation

I found a bug that the implementation of Vehicle.RadioStation misses vehicle handle parameter.
Sample source code
if (!Game.Player.Character.IsInVehicle()) {
    return;
}

Vehicle currentVehicle = Game.Player.Character.CurrentVehicle;

// Crash
currentVehicle.RadioStation = RadioStation.RadioOff;

// Success
Function.Call(Hash.SET_VEH_RADIO_STATION, new InputArgument(currentVehicle), new InputArgument("OFF"));

Script Hook
void SET_VEH_RADIO_STATION(Vehicle vehicle, char* radioStation)

Current implementation
https://github.com/crosire/scripthookvdotnet/blob/master/source/scripting/Vehicle.cpp
867 void Vehicle::RadioStation::set(GTA::RadioStation value)
868 {
869     if (value == GTA::RadioStation::RadioOff)
870     {
871         Native::Function::Call(Native::Hash::SET_VEH_RADIO_STATION, "OFF");
872     }
873     else if (System::Enum::IsDefined(value.GetType(), value))
874     {
875         Native::Function::Call(Native::Hash::SET_VEH_RADIO_STATION, Game::_radioNames[static_cast<int>(value)]);
876     }
877 }

Closed issue. But strange... Why didn't the commit close this issue?


enhancement: remove already selected candidates from selection list

create an election and add some candidates to its list
now try to add a person again that is already on the list

result: OpenSlides correctly issues an error message that the person is already on the list:

But it would be more convenient if already selected users will automatically be removed from the selection list so that they cannot be selected accidentally and pollute the list unnecessarily.

Same user selection field is used also for list of speakers, motion submitters/supporters which should work like the candidate field for elections.

This can in my opinion be postponed to 2.2

The motion submitters/supporters form fields already support it - selected users are disabled in drop down list.
Still open: drop down lists for candidates and list of speakers.

This issue was addressed in OpenSlides 3


Useless vector declaration

The vector isn't referenced in the small code block it's declared. I don't see any "useful" side effect in its declaration.
Taken from #1179

Haha see #217 ;)

Whoops


Error with using external heap and having message buffers use the heap allocator

Observing compilation errors when enabling OPENTHREAD_CONFIG_MESSAGE_USE_HEAP_ENABLE. Specifically, the error is due to

'GetHeap' is not a member of 'ot::Instance'	message.cpp

On first inspection this appears to be due to the usage of the external heap configuration (OPENTHREAD_CONFIG_HEAP_EXTERNAL_ENABLE). Un-enabling this feature causes the compilation errors to go away.
Looking at the source, if OPENTHREAD_CONFIG_MESSAGE_USE_HEAP_ENABLE is enabled then MessagePool::GetFreeBufferCount in  message.cpp will use GetHeap() to determine the number of buffers available.
uint16_t MessagePool::GetFreeBufferCount(void) const
{
    uint16_t rval;

#if OPENTHREAD_CONFIG_MESSAGE_USE_HEAP_ENABLE
    rval = static_cast<uint16_t>(Instance::GetHeap().GetFreeSize() / sizeof(Buffer));
#elif OPENTHREAD_CONFIG_PLATFORM_MESSAGE_MANAGEMENT
    rval = otPlatMessagePoolNumFreeBuffers(&GetInstance());
#else
    rval = mNumFreeBuffers;
#endif

    return rval;
}

However, it looks like the GetHeap method is not declared if OPENTHREAD_CONFIG_HEAP_EXTERNAL_ENABLE  is true (instance.hpp).
#if !OPENTHREAD_CONFIG_HEAP_EXTERNAL_ENABLE
    /**
     * This method returns a reference to the Heap object.
     *
     * @returns A reference to the Heap object.
     *
     */
    static Utils::Heap &GetHeap(void) { return sHeap; }
#endif

The expectation would be that it is possible to use external heap and have message buffers use the heap allocator.

Thanks @silabs-nino . I expect it should possible to use such combination of configs (exteranl heap along with msg buffer using heap).
The GetHeap() seems to be used in Message methods which get an estimate of total/remaining buffers (which are used when getting buffer info ). We can consider adding checks to return zero or some specific "invalid" value indicating no estimate is available when we have such a combination.

Just submitted PR:

https://github.com/openthread/openthread/pull/7933


3664 map layers and overlays endpoint fails requesting datasets shared within an organization

@Kartones , @rafatower CR this fix for #3664, please. It's covered more than what issue reflected because many new endpoints were broken for organization shared vizs.

Frontend tests were OK :+1: (details)

In general ok with the code, though there are some broken tests (did not look intp them) and I think it should deal with RecordNotFound exceptions rather than just letting them bubble up

Frontend tests were OK :+1: (details)

:+1: from my side

Don't CR until tests passes. I'll ask you later.

Frontend tests were OK :+1: (details)

Now, finally, @rafatower, @Kartones take a new look at this, please. You can test it in ded03 if you want.

ok, and I need this for my Layer changes so... go ahead please!

:+1: to latest changes


Clean up tests

This prevents a hang in the case of a regression, and reduces the test suite running time by 1 second.

Coverage decreased (-0.1%) to 80.282% when pulling bee0ea498a7c20fbe37bdc03cc54b16fdc10b612 on jonathanhefner:clean-up-tests into 3cee3331033a9d2cb642076de8aefac3bf0f4e3b on guard:master.


Use TimerOutputs.jl to measure test performance

Coverage remained the same at 100.0% when pulling 78f981a882e514232e0bf5d23d72600b6f5d9aa2 on test/performance into 9dcaa6c71b9726e2214d84322ca92c6d7be7367a on master.

Coverage remained the same at 100.0% when pulling 78f981a882e514232e0bf5d23d72600b6f5d9aa2 on test/performance into 9dcaa6c71b9726e2214d84322ca92c6d7be7367a on master.


Full support for air gapped environments

When CI is setup in an internet-less environment, then tools or resources need to be pre-configured. For example, the cf cli is downloaded from the internet but would like to also be able to set it up as either a resource (Concourse) or a tool (Jenkins). In Jenkins I'm thinking it would be done like this using the declarative syntax:
pipeline {
    agent any
    tools {
        cf 'cf-cli-6.29.1+d5129d651.2017-08-17'
    }
    stages {
        stage('Example') {
            steps {
                sh 'cf -v'
            }
        }
    }
}

Then, pipeline shell scripts would need to know or be told when something is preconfigured or needs to be downloaded.

As a workaround, for now, I guess you'll need to remove the curl command from the scripts. I think we could provide a switch so as not to curl anything.


Better testing for chunked uploads.

We have a whole code branch that does chunked uploads that is not and has never been tested. That's a problem, because sometimes it breaks and we don't notice (#2861). I'd like to add more testing for chunked uploads.
However, we can't use httpbin to do this testing. This is because the WSGI spec staunchly refuses to let applications see chunked transfer encoding at the app layer. For this reason, to test chunked transfer encoding requires a new style of testing, one that doesn't involve running a WSGI server but instead running something that can see the bytes as they hit the wire. Basically, something like the socket level tests of urllib3.

One possibility would be adapting urllib3's solution to our case. This new server we'd make doesn't need to support the whole HTTP protocol, because we just need it for the chunked transfer encoding thing, right?

I think we wouldn't actually use a proper server at all, just a pair of sockets.

Г
вторник, 24 ноября 2015 г. пользователь Cory Benfield написал:

I think we wouldn't actually use a proper server at all, just a pair of
sockets.
—
Reply to this email directly or view it on GitHub
https://github.com/kennethreitz/requests/issues/2866#issuecomment-159189923
. Тштххшгшшззжзхмц

--
С уважением,
Рыскулов Чингиз
WWW.WEEKENDS.KG http://www.weekends.kg/
email<EMAIL_ADDRESS><EMAIL_ADDRESS>тел: +996 777 808888
Skype: k http://weekends.kg/gstan

Right, ok.
Both #2860 and this issue come from the same problem: pytest-httpbin's server doesn't seem to support the whole spectrum of our testing needs properly (not saying it is bad software, it's great). We could solve both issues at the same time just by making our socket server able to listen to listen to parallel requests.

Even that is overdesigning it. All we need is to be able to receive a request and send a response. Parallel requests aren't a concern, just one is totally fine. We need a listening socket that can send a pre-canned 200 response when a request is received then read all the data. That's literally all we need to do.

Yes, I understand. My point was that we could solve #2860 too by making this server able to listen to parallel requests. It's ok though if you think it would be better to address those two problems separately.

Oh, I see. Frankly, I think we need to write a better test for #2860, because the approach we had been using (brute force running a lot of threads) is not really the most effective way to test the function.

Ok!
Would it be ok if I start a PR trying to adapt SocketDummyServer and SocketDummyServerTestCase from urllib3?

You are welcome to, though it'll probably require quite a bit of work.

May it make sense to get this into urllib3 and also get rid of tornado there?

@ml31415 Given that we're going to try to move the chunked encoding logic to urllib3, that's probably true. However, the time investment here is still useful, because this kind of focused socket-based testing is better than using httpbin. =)

Agreed, that it makes sense to have this kind of testing in requests. But the testserver itself may be interesting for urllib3 too. So maybe put the testserver there, and requests just reuses it for testing? Instead of duplicating code around.

urllib3 already has its own equivalent of this. =) That's where we stole the idea from. However, it's a little bit nicer to avoid importing urllib3's test framework, because generally speaking testing functionality is not covered by versioning constraints, and can change at any time. Having that change under our feet without warning would be...annoying.

 This is quite some overhead imho, and could be done with a much simpler - own - test server implementation, e.g. extending the dummyserver, or adding some features here.

@ml31415 If urllib3 switches away from Tornado we'd like to switch to a socket-level test approach as you saw with the chunked encoding tests. That's something we're interested in doing, but it's hard to find the time to do the migration. If you're volunteering, then go for it!

Hmm, just had a closer look on it. Moving a bunch of the dummyserver tests to the socketserver (e.g. the retry tests), and replacing tornado with some pimped SimpleHTTPServer for the remaining tests seems quite possible... anyways, I guess that's another discussion then.


Senses

Just suggesting putting senses somewhere on the sheet. Low light, darkvision, blindsight/sense, etc.

A good thought, I can add this.
Initial thoughts are to add it to the Basics section. Any thoughts for where else this addition might live?

Basics is what I thought as well. Only other place it would maybe fit is abilities?

Sounds right, Basics it is. I've added it to my project board: https://github.com/zombieFox/awesomeSheet/projects

Feature is now live


Added separate Github page

Fixes Issue
This PR fixes the following issues:
https://github.com/NSCC-BPIT/NSCC-BPIT-Website/issues/104

Check List (Check all the boxes which are applicable)

[x] My code follows the code style of this project.
[x] My change requires a change to the documentation.
[x] I have updated the documentation accordingly.
[x] All new and existing tests passed.
[x] This PR does not contain plagiarized content.
[x] The title of my pull request is a short description of the requested changes.

Screenshots and video
pic1

pic2

@0-mayurkaretha pls check the commit is failing some build checks

Hey @pradeeptosarkar
Please review.