Implement Cloud-Init NoCloud datasource

[X] basic support for passing meta-data and user-data from flexvolume definition
[x] get rid of getvolumename
[x] move volume population code to virtlet so the nodes don't need genisoimage
[x] fix flexvolume unmount (kubelet fails to detect flexvolume mountpoint, deleted pods stuck in Terminating)
[x] add integration test
[x] update e2e test
[x] update documentation

This change is

Reviewed 14 of 21 files at r1, 4 of 6 files at r3, 21 of 21 files at r4.
Review status: all files reviewed at latest revision, 6 unresolved discussions.

Dockerfile.build, line 18 at r4 (raw file):

                libguestfs-tools \
                genisoimage \
                p7zip-full \

Is this package still needed? I can not find where it's used... (genisoimage is used in integration tests, so this is clear for me).

pkg/flexvolume/flexvolume.go, line 51 at r4 (raw file):

	// XXX: invent a better wat to decide whether debugging should
	// be used for flexvolume driver. For now we only enable it if
	// Docker-in-Docker env is used

Imo this should be documented fixed location declared as constant value. Can be passed to pod as volume, to deliver info about wanted debugging logs.

pkg/utils/files.go, line 44 at r4 (raw file):

}

func GenIsoImage(isoPath string, volumeId string, srcDir string) error {

Exported func without docstring (i know that we have a lot of them, but would be nice to not introduce new).

pkg/utils/json.go, line 2 at r4 (raw file):

/*
Copyright 2016 Mirantis

2017

pkg/utils/json.go, line 24 at r4 (raw file):

)

func MapToJson(m map[string]interface{}) string {

Missing docstring.

tests/e2e/e2e.sh, line 29 at r4 (raw file):

SCRIPT_DIR="$(cd $(dirname "$(readlinkf "${BASH_SOURCE}")"); pwd)"
virsh="${SCRIPT_DIR}/../../examples/virsh.sh"
vmssh="${SCRIPT_DIR}/../../examples/vmssh.sh"

Looks like we should have a follow up which will move these scripts into other directory... utils, contrib, scripts?

Comments from Reviewable

Review status: all files reviewed at latest revision, 6 unresolved discussions.

Dockerfile.build, line 18 at r4 (raw file):
Previously, jellonek (Piotr Skamruk) wrote…
Is this package still needed? I can not find where it's used... (genisoimage is used in integration tests, so this is clear for me).

It's used by pkg/utils/testing/files.go to unpack an iso image for testing purposes w/o loop mounting it. It could be documented but I think in this case we must document all the packages used here

pkg/flexvolume/flexvolume.go, line 51 at r4 (raw file):
Previously, jellonek (Piotr Skamruk) wrote…
Imo this should be documented fixed location declared as constant value. Can be passed to pod as volume, to deliver info about wanted debugging logs.

Well I actually think we may just try and invent some better way of turning on the debugging here, this one's a hack (although quite handy one). E.g. convert virtlet daemonsets etc. to templatable helm and make an option to enable the flexvolume debugging (that will create a magic file under the hood). I'm not sure documenting this hack is a good thing...

pkg/utils/files.go, line 44 at r4 (raw file):
Previously, jellonek (Piotr Skamruk) wrote…
Exported func without docstring (i know that we have a lot of them, but would be nice to not introduce new).

Will fix, thanks

pkg/utils/json.go, line 2 at r4 (raw file):
Previously, jellonek (Piotr Skamruk) wrote…
2017

Will fix.

pkg/utils/json.go, line 24 at r4 (raw file):
Previously, jellonek (Piotr Skamruk) wrote…
Missing docstring.

Will fix.

tests/e2e/e2e.sh, line 29 at r4 (raw file):
Previously, jellonek (Piotr Skamruk) wrote…
Looks like we should have a follow up which will move these scripts into other directory... utils, contrib, scripts?

At the moment they're tied to examples one way or another (e.g. vmssh.sh uses predefined ssh key by default). I think we may do this cleanup in a followup PR.

Comments from Reviewable

Reviewed 3 of 3 files at r5.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable

Reviewed 13 of 21 files at r1, 3 of 6 files at r3, 20 of 21 files at r4, 3 of 3 files at r5.
Review status: all files reviewed at latest revision, 5 unresolved discussions, all commit checks successful.

build/cmd.sh, line 132 at r5 (raw file):

      start_libvirt="VIRTLET_DISABLE_KVM=${VIRTLET_DISABLE_KVM:-} /start.sh -novirtlet && "
    fi
    vcmd "${start_libvirt}cd '${subdir}' && CGO_CFLAGS='-g -O2 -I/usr/include/glib-2.0 -I/usr/lib/x86_64-linux-gnu/glib-2.0/include' CGO_LDFLAGS='-lglib-2.0 -lvirt' go test $*"

CGO flags now can be omitted.

build/cmd.sh, line 136 at r5 (raw file):

function gobuild {
    vcmd "cd '$(virtlet_subdir)' && CGO_CFLAGS='-g -O2 -I/usr/include/glib-2.0 -I/usr/lib/x86_64-linux-gnu/glib-2.0/include' CGO_LDFLAGS='-lglib-2.0 -lvirt' go build $*"

As above.

docs/volumes.md, line 143 at r5 (raw file):

Here we set hostname for the VM, inject an ssh public key and provide a script that executed by cloud-init. There's helper script named [examples/vmssh.sh](../examples/vmssh.sh) that can be used to access VMs over ssh after providing ssh keys for them (it defaults to using a sample key from examples/ directory).

Virtlet's cloud-init mechanism is not finalized yet and will change in future so as to support dynamic metadata generation and passing metadata via the metadata server instead of `NoCloud`.

Why instead? I thought "along with". Aren't we going to keep 'nocloud' still in place after adding support for metadata server as well?

examples/README.md, line 38 at r5 (raw file):

Besides [cirros-vm.yaml](cirros-vm.yaml), there's also [ubuntu-vm.yaml](ubuntu-vm.yaml) that can be used to start an Ubuntu Xenial VM. It can also be accessed using `vmssh.sh` after it boots:

./vmssh.sh root@ubuntu-vm

missed [command...]

tests/integration/container_test.go, line 183 at r5 (raw file):

func (ct *containerTester) verifyNoContainers(filter *kubeapi.ContainerFilter) {
	if len(ct.listContainers(filter).Containers) != 0 {
		ct.t.Errorf("expected no containers to be listed")

I suppose, the error msg should contain info about set filter set for List request.

Comments from Reviewable

Review status: 33 of 37 files reviewed at latest revision, 5 unresolved discussions.

build/cmd.sh, line 132 at r5 (raw file):
Previously, vefimova (Victoria Efimova) wrote…
CGO flags now can be omitted.

Done.

build/cmd.sh, line 136 at r5 (raw file):
Previously, vefimova (Victoria Efimova) wrote…
As above.

Done.

examples/README.md, line 38 at r5 (raw file):
Previously, vefimova (Victoria Efimova) wrote…
missed [command...]

Done.

tests/integration/container_test.go, line 183 at r5 (raw file):
Previously, vefimova (Victoria Efimova) wrote…
I suppose, the error msg should contain info about set filter set for List request.

Done.

Comments from Reviewable

Review status: 33 of 37 files reviewed at latest revision, 5 unresolved discussions.

docs/volumes.md, line 143 at r5 (raw file):
Previously, vefimova (Victoria Efimova) wrote…
Why instead? I thought "along with". Aren't we going to keep 'nocloud' still in place after adding support for metadata server as well?

Done.

Comments from Reviewable

Review status: 33 of 37 files reviewed at latest revision, all discussions resolved.

Comments from Reviewable

Reviewed 4 of 4 files at r6.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable


Create AzureStorageBlob.framework

iOS framework for Storage Blob service

[x] Download blob
[x] Upload blob #214
[x] List blobs in container
[x] Remove blob #215

All items complete. Closing.


[docs-only] Update the PROXY_TLS envvar description

This PR clarifies that the setting PROXY_TLS needs to be set to true when using the embedded IDP service. Also adds a note in the proxy readme.
The envvar description should be backported to docs-stable-2.0

@micbar I am just working on changing this. Will ping you soon.

@micbar ping, changes made, pls review


KVM Host-HA using CEPH RBD

I was able to use KVM Host-HA using NFS as primary storage it works great.
If I use CEPH RBD as primary storage KVM Host-HA is not working.
Does it mean KVM Host-HA only works with NFS ?

I was able to use KVM Host-HA using NFS as primary storage it works great.
If I use CEPH RBD as primary storage KVM Host-HA is not working.
Does it mean KVM Host-HA only works with NFS ?

it is also supported for storpool storage. #8045
There is a PR to support ceph storage. #5862 , it is not merged yet.

@weizhouapache Thank you


Omit base model types by default

The properties of a model's base types are folded into the super type. If the base types aren't referenced, omit them.
Fixes https://github.com/Azure/autorest.go/issues/1333

The comment is very focused on languages that use OO inheritance to describe these relationships.


Code Coverage badge in README.md is broken

I see the badge is trying to load from master branch.
I tried using main instead and it's working.

:tada: This issue has been resolved in version 1.8.1 :tada:
The release is available on:

GitHub release
npm package (@latest dist-tag)

Your semantic-release bot :package::rocket:


VideoJS Element is not loading the example streams

When attempting to run the exampels in the Hugo VideoJS site it is not working with the examples provided.
https://hugomods.com/docs/media/video-js/#videojs-hls-example
https://hugomods.com/docs/media/video-js/#videojs-mpeg-dash-example
In my .md the regular video is loading fine but no the stream.
//working fine
{{< video-js "https://heishenhua.com/video/preview/video_PV03_Music.mp4" >}}
//not working
{{< video-js
src="https://devstreaming-cdn.apple.com/videos/streaming/examples/adv_dv_atmos/main.m3u8"
type="application/x-mpegURL"

}}

Let me know if more information is needed or if the issue is just me.

Does it work on the docs site? Is there any error on Chrome Console tab?

It would be better if you can provide a public repo for reproducing this issue.

Yes it is working in the Doc Site.
https://hugomods.com/docs/media/video-js/
And here is my repo https://github.com/cbellosoto/TestSite/blob/main/content/index.md
Let me know if you have any questions.

There are several issues:

Your theme doesn't implement the HugoPress, you've to include the CSS and JavaScript on your templates, tweak your template and check the page's source code.

You're copying the source code to your own project, please kind in mind that you're maintaining the code yourself, you won't got updated if there is bugfix on this repo.
Please append -D flag on hugo server to build draft content.

Closed per above comment.


script to test filters

The filters are being applied and the simulation, I think, works as expected but the results don't show up for some reason. It always shows "no matching records"
I should've added a line in the end (sorry about that!)

works as expected but the results don't show up for some reason

The filter configuration set up by hand gives no results too, so that's not the test script fault.

The ones set by hand in the Chrome started by Selenium/WebDriver don't work. However, if it's a normal Chrome tab that we open, it seems to work. For eg: coverage = 50 and type is preprocessing gives results in the normal Chrome tab but not by Selenium.

This works for me (results in one row):
driver.implicitly_wait(10)
driver.find_element_by_tag_name('details').click()

addEntries(2)
fillEntry(0, "coverage", "=", "50")
fillEntry(1, "type", "is", "preprocessing")

addGlobalOp("and")
applyFilters()

Ahh I'm not sure why it wasn't working earlier - I see 2 rows now. Also, it could be the "is not" that isn't working because it worked fine when I changed it to:
fillEntry(0, "coverage", ">", "50") fillEntry(1, "type", "is", "simulation") fillEntry(2, "tool", "is", "icarus")

Please let me know if I have to change something, thanks!

Oh, and one more thing: we should target already merged filter version (i.e. without global operator). Can you adapt it? Looks like the only required changes are fillEntry adjustment and removal of addGlobalOp.

Sorry it took me so long and thanks for being patient! (I was finishing up my mid-terms and some other stuff)

LGTM, please make it "ready for a review" to allow merging.

How will that be used? Should we add a CI job for this?

How will that be used? Should we add a CI job for this?

This is a simple script to help developers/reviewers test the changes in filter UI. Instead of manually entering the filter settings after each small code change/full refresh, or describing what exactly did you click in what order to trigger some error, you can run/share a code like:
#!/usr/bin/env python3
import filter_ui_test as ft

ft.setDriver()
ft.openPage('https://chipsalliance.github.io/sv-tests-results/') # or URL to your local copy
ft.addEntries(1); ft.fillEntryType(0, 'coverage'); ft.fillSpan(0, '>', '50', 'and', '<=', '60')
ft.addEntries(1); ft.fillEntryType(1, "type");     ft.fillSpan(1, "is", "preprocessing")
ft.addEntries(1); ft.fillEntryType(2, "tool");     ft.fillSpan(2, "is", "icarus")
# Test: apply - remove one - apply
ft.applyFilters()
ft.removeIcon(2)
ft.applyFilters()

The result has to be manually verified, so it isn't suitable for CI in its current form.

Is conf the right directory for this script?

It's quite good - it is near the stuff it is inteded to be used with. We can think about another directory (e.g. development) when we get more scripts like this one.

If so, can we make sure it is also taken into account when checking the formatting? see: https://github.com/chipsalliance/sv-tests/blob/master/Makefile#L145-L147

@Prahitha please add wildcard conf/report/*.py in the Makefile linked above

@Prahitha
Just one more thing: please fix formatting errors (extra newlines: https://github.com/chipsalliance/sv-tests/pull/1743/checks?check_run_id=3805424990).


Fix use of module.parent

module.parent is deprecated https://github.com/marvingrieger/ioBroker.haassohn/blob/c3dfc6c00485e27594ad94de982e6d9826221990/main.js#L487, better check module === require.main
Originally posted by @foxriver76 in https://github.com/ioBroker/ioBroker.repositories/issues/1054#issuecomment-761801121

Opened in wrong repository

Opened in wrong repository


Add monitoring and alerting to management cluster

Linked to https://github.com/stackhpc/azimuth-config/pull/16

Closed via #58


[feature request] Archive whole file

First of all, thank you very much for such a helpful plugin!
I've been using the plugin as it is for a while, and there's only one missing thing I needed on some occasions.
I can guess that the plugin itself is not exactly about archiving in general (it's task-archiver, isn't it), but then again, there's an option to archive headings.
Do you think it would make sense to add the command to archive the whole file?

Hey, @jrogov,
Please describe the workflow you'd want. Should the plugin archive files based on date or source path, etc.


TypeError: content.matchAll(...) is not a function or its return value is not iterable

On npm start I am getting the following error:

edit: updating node to 15.x works fine...

Elder.js requires node v12+. Glad you got it figured out. :)


IRC clients lose some messages right after disconnect

In the following log, user @halfak was on IRC and user @yuvipanda was on the mattermost web interface. 'yuvipanda' sent numbers from 1 to 30, and 2-6 were missed...
[13:10:27] <yuvipanda> @halfak ok next
[13:10:36] <yuvipanda> @halfak is the 1 to 30 disconnection thing
[13:10:45] <halfak> OK.  Ready to disconnect
[13:10:52] <yuvipanda> 1
[13:10:55] <yuvipanda> @halfak go! :)
[13:10:56] <yuvipanda> 2
[13:10:59] * Disconnected ()
[13:11:22] * Now talking on #town-square
[13:11:22] * subbu (3j5u8mfb57rg8c3dsozno8a1qh@https://mattermost.wmflabs.org) has joined
[13:11:22] * tgr (6ezx1bib83898dh6x9g3cz71iw@https://mattermost.wmflabs.org) has joined
[13:11:22] * joe (akr6t7k3ubdhmnit96cazh1xth@https://mattermost.wmflabs.org) has joined
[13:11:22] * robla (idxrsxk1ubd95p4pjq83rqzu4e@https://mattermost.wmflabs.org) has joined
[13:11:22] * luisv (n5fbhrmwwibifjaerouxqracyr@https://mattermost.wmflabs.org) has joined
[13:11:22] * stu (pgj4t7yuffbq3xdks773kijqdc@https://mattermost.wmflabs.org) has joined
[13:11:22] * hashar (tfr5g9rscj8hxbazod4qrcaibr@https://mattermost.wmflabs.org) has joined
[13:11:22] <yuvipanda> 6
[13:11:22] <yuvipanda> 7
[13:11:22] <yuvipanda> 8
[13:11:22] <yuvipanda> 9
[13:11:22] <yuvipanda> 10
[13:11:22] <yuvipanda> 11
[13:11:22] <yuvipanda> 12
[13:11:22] <yuvipanda> 13
[13:11:22] <yuvipanda> 14
[13:11:22] <yuvipanda> 15
[13:11:22] <yuvipanda> 16
[13:11:22] <yuvipanda> 17
[13:11:22] <yuvipanda> 18
[13:11:22] <yuvipanda> 19
[13:11:22] <yuvipanda> 20
[13:11:22] <yuvipanda> 21
[13:11:22] <yuvipanda> 22
[13:11:22] <yuvipanda> 23
[13:11:23] <yuvipanda> 24
[13:11:24] <yuvipanda> 25
[13:11:25] <yuvipanda> 26
[13:11:26] <yuvipanda> 27
[13:11:26] <halfak> Back
[13:11:27] <yuvipanda> 28
[13:11:32] <yuvipanda> @halfak what did you see
[13:11:43] <halfak> So, it looks like 2-6 dropped.

Can't reproduce it, probably can't type fast enough. Is it possible to try again with -debug on ?
Pipe it through grep -v "PRIVMSG mattermost :LOGIN" so that you're credentials aren't visible.

I already had debug on for this so let me find the logs.

Any logs or reproduced again ? otherwise I'll close this.

I'll try repro this the coming week whwen more of my victims come back
online from vacation
On Sat, Nov 28, 2015 at 2:35 PM, @42wim<EMAIL_ADDRESS>wrote:

Any logs or reproduced again ? otherwise I'll close this.
—
Reply to this email directly or view it on GitHub
https://github.com/42wim/matterircd/issues/11#issuecomment-160343097.

--
Yuvi Panda T
http://yuvi.in/blog

ping timeout, reopen when it happens again :-)


Few updates to make kue a bit more up to date

Also question, tests really looks like they were written by 3 different mens. Will it be ok if i will update all them to one look (plain js files and should.js)?

Please merge this.


LESQ-368 Feat/launch

Description
This PR contains the changes needed to make the new teacher experience live. DoO NOT MERGE until ready to launch.

Updates the app to use AppHeader across all pages
remove old SiteHeader and related components (MenuLinks & MenuLink, menuLinks fixtures)
Update spelling in hamburger to "Early-release units"
Uncomment key stage links in the footer
Remove copyright notice from the footer
Remove /teachers from deployment test urls
moved teachers homepage the root url
update links to teachers

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


add Code of Conduct

Needs to be updated with an email address for a maintainer.
Found this issue https://github.com/rtfd/readthedocs.org/issues/2323 as part of looking for something to do today for #100DaysofCommits but saw Read The Docs has no COC yet. Adding COC first.

Any reason this wasn't just added to our documentation? I'm not finding if this is a file that GitHub supports in some sort of native fashion

+1 on a more explicit code of conduct. It should be included in the docs, and I'd be more inclined to probably use Django's or the Python community one. Though we should figure out which to use.

It actually didn't occur to me to put it straight in the documentation as that's not the convention I'm familiar with. I'm used to seeing it as a specific file (see https://github.com/search?q=code_of_conduct.md&ref=searchresults&type=Code&utf8=✓ with 30k+ results).
What is the best way to get a decision on which code of conduct will apply to this repo?

I think this is much better suited for our documentation, that is where all of our other contributing docs are. Also, GitHub doesn't seem to handle these files natively, such as it does with the .github/CONTRIBUTING.rst file.
Our current contributing docs should link to the CoC, they live at:
http://docs.readthedocs.io/en/latest/contribute.html
On the copy of the agreement, I prefer the tone of Django's code of conduct to the one supplied here, and don't think Python's defines expectations explicitly enough. It's also important the CoC sets the expectation on all users, not just maintainers/contributors.
My vote is for Django's CoC here.

People, do you have some pragmatical reasons to add this nonsense?  Do you care if CoC will stop someone from contributing to your project?
What about simple NoCoC instead?

@skirpichev the internet has shown time and time again that it isn't always capable of acting like an adult, and so yes, we whole-heartedly agree it's necessary that we explicitly state what is unacceptable behavior. This has never been a problem though, and we hope it is never a problem.
If anyone lost interest in contributing because they would rather use hateful language/actions, it's probably for the best anyways.

This has never been a problem though

So, you don't have practical reasons (from the history of the project) to introduce this, right?

because they would rather use hateful language/actions

No, because they learn that such documents - provoke instead "hateful actions" by hiding conflicts, not stop them.  Anyway, thank you for explanation of your decision.


fix: Disable browser autocomplete on MultiSelect input element

This change prevents browser autocomplete being invoked when clicking on the search input element in the MultiSelect component by adding autocomplete="off" to the input element.
Related issue(s)

Fixes #708
What kind of change does this PR introduce? (check at least one)

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New Feature (non-breaking change which adds functionality)
[ ] Refactor
[ ] Build-related changes
[ ] Other, please describe:

Does this PR introduce a breaking change? (check one)

[ ] Yes
[x] No

How has This been tested?

Tested the MultiSelect component in Firefox using Storybook. Before adding my code, the browser autocomplete appeared when double-clicking the search input within the MultiSelect. After adding my code, it did not. See the before and after screenshots below.
Screenshots (if appropriate):
Before:

After:

The PR fulfills these requirements:

[x] It's submitted to the main branch
[x] When resolving a specific issue, it's referenced in the related issue section above
[ ] My change requires a change to the documentation. (Managed by Tremor Team)
[ ] I have added tests to cover my changes
[x] Check the "Allow edits from maintainers" option while creating your PR.
[x] Add refs #XXX or fixes #XXX to the related issue section if your PR refers to or fixes an issue.
[x] By contributing to Tremor, you confirm that you have read and agreed to Tremor's CONTRIBUTING.md guideline. You also agree that your contributions will be licensed under the Apache License 2.0 license.

Oops, sorry, I forgot about everything needing to be cased differently in React 😅 I have pushed an update.


Update PULL_REQUEST_TEMPLATE.md

Summary
Update pull request template to use comments as in other mattermost repositories
Ticket Link
None
Checklist
None
Test Information
This PR was tested on: None

Thanks @larkox !!

Thank you very much @larkox :+1:


Add comments through the source code

Currently, the whole codebase lacks comments that describe each component and their associate method or functions. it would be nice to provide comments/descriptions that can serve as documentation for existing or new contributors

Hi, Can i work on it

sure

you'll find some examples here :  https://github.com/JeanPhilippeKernel/RendererEngine/blob/develop/ZEngine/include/ZEngine/Rendering/Shaders/ShaderReader.h
Doxygen is our reference  :  https://www.doxygen.nl/manual/docblocks.html#cppblock


Add new jurisdiction code to employment tribunal decisions

The new jurisdiction code of 'Notice Appeal' has been added to the
Employment Tribunal Decisions finder.
The following rake tasks will also need to be run:

[ ] rake publishing-api:publish_finders
[ ] rake rummager:publish_finders

Trello card

Looks good to me 👍 do you want to try deploy that branch on integration and give those tasks a spin?

Deploying the branch then running the rake tasks works in integration - "Notice Appeal" appears in both Specialist Publisher and the Finder:
Specialist Publisher

Employment Tribunal Decisions finder

Awesome 👍  Thanks for the screengrabs.


[LIBCLOUD-936] fixed AWS ALB/ELB driver init method to instantiate ne…

Fixed AWS ALB/ELB driver init method to instantiate nested connection object properly
Description
AWS ELB/ALB Driver connection object initialization is broken.
Error:
/Users/irvan/py27venv/bin/python test_token.py Traceback (most recent call last):   File "test_token.py", line 33, in <module>     data = driver.list_balancers()   File "/Users/irvan/Repos/GIT/github/libcloud/libcloud/loadbalancer/drivers/elb.py", line 68, in list_balancers     data = self.connection.request(ROOT, params=params).object   File "/Users/irvan/Repos/GIT/github/libcloud/libcloud/common/base.py", line 603, in request     headers=headers, stream=stream)   File "/Users/irvan/Repos/GIT/github/libcloud/libcloud/http.py", line 215, in request     verify=self.verification   File "/Users/irvan/py27venv/lib/python2.7/site-packages/requests/sessions.py", line 518, in request     resp = self.send(prep, **send_kwargs)   File "/Users/irvan/py27venv/lib/python2.7/site-packages/requests/sessions.py", line 639, in send     r = adapter.send(request, **kwargs)   File "/Users/irvan/py27venv/lib/python2.7/site-packages/requests/adapters.py", line 502, in send     raise ConnectionError(e, request=request) requests.exceptions.ConnectionError: HTTPSConnectionPool(host='elasticloadbalancing.%s.amazonaws.com', port=443): Max retries exceeded with url: /2012-06-01/?Action=DescribeLoadBalancers&Version=2012-06-01 (Caused by NewConnectionError('<requests.packages.urllib3.connection.VerifiedHTTPSConnection object at 0x10760f4d0>: Failed to establish a new connection: [Errno 8] nodename nor servname provided, or not known',))
Issue:
https://issues.apache.org/jira/browse/LIBCLOUD-936
Status
done, ready for review
Checklist (tick everything that applies)

[ ] Code linting (required, can be done after the PR checks)
[ ] Documentation
[ ] Tests
[ ] ICLA (required for bigger changes)

Coverage decreased (-0.0003%) to 88.86% when pulling 0a03bede146c1984daa0c5be2431335c5cedc550 on lrvan:LIBCLOUD-936 into 3dbf50b07e151e3a0960962bd6b896a38065fd3a on apache:trunk.

build [3789.9|https://travis-ci.org/apache/libcloud/jobs/262180991] failure seems to be not related to the change introduced by PR
build [3789.10|https://travis-ci.org/apache/libcloud/jobs/262180994] failed with some tests invocation error:
...................................................................................ERROR: InvocationError: '/home/travis/build/apache/libcloud/.tox/pypypy/bin/python setup.py test'
___________________________________ summary ____________________________________
ERROR:   pypypy: commands failed
build [3789.11|https://travis-ci.org/apache/libcloud/jobs/262180996] failed with an error related to test environment setup:
File "/home/travis/virtualenv/pypy3-2.4.0/site-packages/pkg_resources/init.py", line 805, in resolve
raise DistributionNotFound(req)
pkg_resources.DistributionNotFound: setuptools-scm>=1.15.0
the rest are fine.

build 3789.9 failure seems to be not related to the change introduced by PR
build 3789.10 failed with some tests invocation error:
...................................................................................ERROR: InvocationError: '/home/travis/build/apache/libcloud/.tox/pypypy/bin/python setup.py test'
___________________________________ summary ____________________________________
ERROR:   pypypy: commands failed
build 3789.11 failed with an error related to test environment setup:
File "/home/travis/virtualenv/pypy3-2.4.0/site-packages/pkg_resources/init.py", line 805, in resolve
raise DistributionNotFound(req)
pkg_resources.DistributionNotFound: setuptools-scm>=1.15.0
the rest are fine.

@lrvan the tests should pass now

Merging 👍


add circuit breaker sample

Purpose

Sample to demonstrate the Circuit Breaker functionality in Ballerina

Automation tests

Unit tests

done

Integration tests

N/A

Security checks

Ran FindSecurityBugs plugin and verified report? no
Confirmed that this PR doesn't commit any keys, passwords, tokens, usernames, or other secrets? yes

Test environment

Ballerina version 0.962
JDK 1.8.0_161

Learning

Release it! book

I have made the changes according to the recommendations. Please point out if something is still missing.


Balanced/unbalanced training

Are the models trained on balanced trainset? Did you use unbalanced data for training?

The models are trained on full (bal + unbal) dataset. If we only train on balanced dataset, the performance is worse, (mAP around 0.25 - 0.30)


Create govuk-search-relevance-tool job for running tests

We have a number of Dependabot PRs (https://github.com/alphagov/govuk-search-relevance-tool/pulls)
which are waiting on Jenkins CI responses and never receiving a response, since no job exists.

Closed in favour of archiving the repo (after checking with Jessica Jones).

Closed in favour of archiving the repo (after checking with Jessica Jones).


Support to remove table prefix from entities when "--removePrefix table_prefix_ " is mentioned

First of all, I've started using this tool recently and it's wonderful, great job.
Would be good an option to remove a table_prefix before generating entities.
I decided to delete my entities files after i changed them alot, my case looks like this:
All good with the following:
TypeORM:

entityPrefix = web_

Entities:

Account
User
Session

Tables:

web_account
web_user
web_session

As i said before, i deleted all the entities and used typeorm-model-generator to generate new ones, to my surprise the output was:
Not good, different files:
Entities:

WebAccount
WebUser
WebSession

What is should have been:
Entities:

Account
User
Session

Which turned my problem into another problem since my code was bound to the previous -unprefixed- entities, and this could be fixed with a new option to specify a table prefix and extract ONLY those tables. In the meantime i managed to fix it with a custom naming strategy, but it would be nice to label issue this as an enhancement, and the --removePrefix used should end in ormconfig.json.
Here is the code i used in the strategy:
let entityPrefix = getFromArgv('--removePrefix') || '';

function entityName(oldEntityName, entity) {
    let ep = entityPrefix;
    if (ep && ep.trim().length && oldEntityName.startsWith(ep)) {
        entity.sqlName = entity.sqlName.substr(ep.length);
        entity.columns = entity.columns.map(item => {
            if (item.hasOwnProperty('default') && item.default) {
                item.default = item.default.replace(/"'(.*)'"/gi, '"$1"')
            }
            return item;
        })
        return oldEntityName.substr(ep.length);
    }
    return oldEntityName;
}

$ typeorm-model-generator --namedStrategy=./CustomFile.js --removePrefix web_   ...connectionProps
<EMAIL_ADDRESS>node v14.17.0

This is not likely to be implemented - #329.
Custom naming strategy is a proper way to achieve the behavior you want.


Fuzz tests - tuf

Fuzz tests for tuf

Signed-off-by: naveensrinivasan<EMAIL_ADDRESS>

@asraa Ran into this while running fuzz tests.
warning: starting with empty corpus
fuzz: elapsed: 0s, execs: 0 (0/sec), new interesting: 0 (total: 0)
fuzz: elapsed: 3s, execs: 0 (0/sec), new interesting: 0 (total: 0)
fuzz: minimizing 31-byte failing input file
fuzz: elapsed: 4s, minimizing
--- FAIL: FuzzInitializeTufFromEnv (3.87s)
    --- FAIL: FuzzInitializeTufFromEnv (0.02s)
        fuzz_test.go:25: open /var/folders/pf/bq3t19j505xg4pprmb35ls7r0000gn/T/FuzzInitializeTufFromEnv3710900219/001/targets/artifact.pub: no such file or directory

    Failing input written to testdata/fuzz/FuzzInitializeTufFromEnv/771e938e4458e983a736261a702e27c7a414fd660a15b63034f290b146d2f217
    To re-run:
    go test -run=FuzzInitializeTufFromEnv/771e938e4458e983a736261a702e27c7a414fd660a15b63034f290b146d2f217

FAIL
exit status 1
FAIL	github.com/sigstore/sigstore/pkg/tuf	4.044s

Process finished with the exit code 1

Am I going in the right direction? Wanted some feedback. I took inspiration from the test code.

@asraa  Friendly ping

Are we ok to close this? I’d like to fuzz TUF, but this requires more thought around which APIs to target.

Created an issue, will close this for now.


Fix lint task failure because of third party libs

What is this PR trying to achieve?
Linting Python that doesn't include 3rd party libraries and consequently error incorrectly.
Who is this change for?
💻

Worth noting we didn't see any failures in CI because of this - we should investigate.


issues

Error while getting releases. Cannot read property 'tag' of undefined
when i want to get releases from rapid7/metasploit-framework show that error

Hello @mandofski23
This issue has been resolved as of version 1.2.0 Please, let me know if there's any issue with this!
Thanks,
Marouane


Adding the Concepts page for Streaming Integrator

Description:
Adding the Concepts page for the Streaming Integrator profile.
Suggested Labels:
Streaming-Integration docs
Affected Product Version:
EI 7.0.0
Link
https://github.com/wso2/docs-ei/blob/master/en/streaming-integrator/docs/concepts.md

Added [1] for the EI 7.0.0 release.
[1] https://ei.docs.wso2.com/en/latest/streaming-integrator/concepts/concepts/


136 check smoothest line solution

Closes #136

@Constantino-Carreto-Romero When you get the chance, do you think you can run the plots test file and make sure everything works for you?

@jorpppp I ran the plot test file without problems.

Thanks @Constantino-Carreto-Romero . Merging now.


Tweet Alert for OSDC Telegram Bot

Fixes #5  .
New Features: Adds a tweet alert on the OSDC telegram channel , and that includes the replies on the tweet, and anyone who Retweeted it.

Here's the GIF to demonstrate how it works, I used my own twitter account for this.


about the evaluation

thank you for your repo. I reproduce your experiments, however, I find that PESQ, CD and LLR aren't consistent with the paper. For CD and LLR , I used the matlab code involved in the book "Speech Enhancement: Theory and Practice".  Would you like to provide your evaluation code?

I used the evaluation scripts from the REVERB challenge. You can download them from here. Let me know if you still find issues reproducing the results.
For PESQ evaluation, I used the PESQ executable and compiled it on my machine. Then I used a matlab wrapper to run the evaluation (Downloaded from here). The comment section of this link also has other useful PESQ versions (in python, mex-file,..).


[BUG] - Differentiate the responsibility between showPreview and showOpenInCodeSandbox

NextUI Version
2.2.10
Describe the bug
In the CodeDemo component, we have showOpenInCodeSandbox property but showOpenInCodeSandbox is not going to be false unless we set showPreview to false as well.
https://github.com/nextui-org/nextui/blob/5a8768a4b209004185b7cf6f652f3d16ee876415/apps/docs/components/docs/components/code-demo/code-demo.tsx#L145
Your Example Website or App
No response
Steps to Reproduce the Bug or Issue
https://github.com/nextui-org/nextui/blob/5a8768a4b209004185b7cf6f652f3d16ee876415/apps/docs/components/docs/components/code-demo/code-demo.tsx#L145
Expected behavior
If showOpenInCodeSandbox set to false, “Open In Sandbox” button will be hidden.
If showPreview set to false, the preview will be hidden.
Screenshots or Videos
No response
Operating System Version
macOS
Browser
Chrome

I will work on this! thanks, guyss!

This issue will be taken care of by https://github.com/nextui-org/nextui/issues/2517


Toil dependency should be removed

The Toil docs say you shouldn't have your module depend on Toil.

Fixed in #184


Update Node.js 14, 16 versions to 20 in building-and-testing-nodejs.m…

…d building-and-testing-nodejs.md
Removed Node.js versions 14.x and 16.x. Added version 20.x. Updated Node.js 14.x with 18.x, as 18 is still supported.

Why:
GitHub Actions show warnings with Node.js 14.x version when running nodejs.yaml workflow file, which refers to the documentation https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/
Closes:

What's being changed (if available, include any code snippets, screenshots, or gifs):
Removed 14.x and 16.x, added 20.x.
Replaced 14 to 18.

Check off the following:

[x] I have reviewed my changes in staging, available via the View deployment link in this PR's timeline (this link will be available after opening the PR).

For content changes, you will also see an automatically generated comment with links directly to pages you've modified. The comment won't appear if your PR only edits files in the data directory.

[x] For content changes, I have completed the self-review checklist.

@ani-hovhannisyan Thanks so much for opening a PR! I'll get this triaged for review ✨

You're welcome! 🎉

Thanks very much for contributing! Your pull request has been merged 🎉 You should see your changes appear on the site in approximately 24 hours. If you're looking for your next contribution, check out our help wanted issues ⚡

Thank you very much for spotting and fixing this! I'll merge this now.

You're welcome! 🎉


Dependency directory kinds when building dependent crates

I'm cross-compiling a (very) bare-metal program from osx to arm (along a reasonably well-travelled path!).
[dependencies]
core = { path = "core" }
rlibc = "0.1.4"

Error:
$ cargo build --target=arm-none-eabi --verbose
   Compiling rlibc v0.1.4
     Running `rustc [...]/rlibc-0.1.4/src/lib.rs \
                    --crate-name rlibc --crate-type lib -g \
                    -C metadata=f21067c861e5133f \
                    -C extra-filename=-f21067c861e5133f \
                    --out-dir [...]/target/arm-none-eabi/debug/deps \
                    --emit=dep-info,link --target arm-none-eabi \
                    -L dependency=[...]/target/arm-none-eabi/debug/deps \
                    -L dependency=[...]/target/arm-none-eabi/debug/deps \
                    --cap-lints allow`
       Fresh core v0.1.0 (file:///Users/alilee/arm/core)
[...]rlibc-0.1.4/src/lib.rs:1:1: 1:1 error: can't find crate for `core` [E0463]

The -L dependency= kind is not being used to find libcore.rlib and if I run the rustc command manually changing to -L crate= or -L all= then the command finishes and rlibc.rlib is built.
Is there a way to get cargo to emit different kinds of -L in a dependency? What is the correct way to direct the dependent crate rlibc to find core?
I've tried adding the -L as a parameter to my package's cargo call, however these cargo rustc parameters don't seem to propagate to the dependency crates.
	@cargo rustc --target $(TARGET) -- -L all=./target/$(TARGET)/debug/deps

Happy to write this up if you could give me a clue! Many thanks!

Ah this is actually intentional, currently, because this is compiling rlibc which doesn't have a dependency on core. Crates aren't currently allowed to depend on anything they don't explicitly depend upon. As such, I'm going to close this as working as intended.
Currently when compiling for a new target the best way to do this is to create your own sysroot with libcore/libstd (as appropriate) inside it and then make sure --sysroot is passed to the compiler somehow. We hope to make this a better situation in the future!

Ok, so this issue lives at the nexus of cross-compiling, cargo and multirust. Don't mind me, just setting up the SEO ring:
http://stackoverflow.com/questions/28031806/cannot-link-against-core-library-when-cross-compiling
https://github.com/brson/multirust/issues/20

@alexcrichton Thanks very much for the comment.
Can I get your sense that this is simply a matter of copying my libcore.rlib into a rustlib/arch directory?
$ cd ~/.multirust/toolchains/nightly/lib/rustlib/arm-none-eabi/
$ ls
libcore-47c0e7f81fd07753.rlib

(You can guess that this didn't work on its own!)

Yeah in theory that's all you need to do, but did that not work out?

Great! I was just missing the final lib:
$ cd ~/.multirust/toolchains/nightly/lib/rustlib/arm-none-eabi/lib
$ ls
libcore-47c0e7f81fd07753.rlib
$

Thanks heaps.

https://github.com/rust-lang/rfcs/pull/1133 would make rlibc depend on core.


Add support for interactive transitioning

https://github.com/material-motion/transitioning-objc/issues/3

Am proposing the following changes to consolidate the code paths here (these affect public APIs which is why I'm suggesting we make these changes now before the code lands):

[ ] Make MDMInteractiveTransitionContext conform to MDMTransitionContext.
[ ] Delete transitionContext from MDMInteractiveTransitionContext.
[ ] Rename MDMInteractiveTransition to MDMTransitionInteractionController.
[ ] Delete MDMViewControllerInteractiveTransitionContext and implement MDMInteractiveTransitionContext in MDMViewControllerInteractiveTransitionContext.


Array empty element converts to null

stringify(new Array(3)) /* "[null,null,null]" */
[,,][0] === null /* false */

I would accept a pull request to fix this, but it requires us to manually do the stringifying of Arrays and Objects. That would let us properly handle undefined and Dates even when they are properties of objects/elements of arrays.

Is it really needed?

I'm confused. This was your issue. If you don't think the problem needs solving, why did you open the issue?

Just to state that issues exists

btw fix is quite easy
if (Array.isArray(obj))
  return '[' + obj.map(x => stringify(x)).join(',') + ']';

That example is wrong in a more subtle way:
[,,,,]
// => [empty x 4]
eval('[' + [,,,,].join(',') + ']')
// => [empty x 3]

I don't know whether this is super important to fix or not. I suspect not as sparse arrays are a pretty rare occurrence, but it would be nice to support undefined in arrays and Date objects in deeply nested locations.

Yes, as mentioned before trailing comma consumes last delimetr
Here is easy fix :)
"engine": { "node": ">6" }


Enhance information status

If I uploaded a video and I am logged in, it should rather denote "Uploaded by: You" rather than my username.

Okay.

FIXED!!!


Implement Authentication with Google and Facebook

Description

As of right now, we have the ability to sign-in using an email and password

We are going to be adding two new sign-in methods: Google and Facebook

To do this, we are going to leverage Firebase's implementation of OAuth, which provides helpful built-ins for signing in with both Google and Facebook
Fortunately, both methods are very similar

Authentication with Google

Create a new file in our frontend utils called googleAuthUtils.js
Reference the documentation on how to obtain the needed helper functions
Use the signInWithRedirect method because it is the most compatible for both web and mobile devices
Create the following util functions:

createGoogleUserInFirebase

This should use the built-in function signInWithRedirect
Utilize the getRedirectResult to track the signed-in user and to also perform error-handling

For now, you can simple store the user in a const
For error handling, retrieve the error code and error message and console log them

Annoyingly, the page on the documentation which should show all the error codes is outdated so for now, just log the error code and message

logGoogleUserOut

Very straightforward, just use the signOut built-in
Perform error handling by catching any errors and then console logging the error code and error message

Authentication with Facebook (This is pretty much a copy paste of the Google Auth :P)

Create a new file in our frontend utils called facebookAuthUtils.js
Reference the documentation on how to obtain the needed helper functions
Use the signInWithRedirect method because it is the most compatible for both web and mobile devices
Create the following util functions:

createFacebookUserInFirebase

This should use the built-in function signInWithRedirect
Utilize the getRedirectResult to track the signed-in user and to also perform error-handling

For now, you can simple store the user in a const
For error handling, retrieve the error code and error message and console log them

Surprise, surprise, the documentation on the error codes is still outdated!

logFacebookUserOut

Very straightforward, just use the signOut built-in
Perform error handling by catching any errors and then console logging the error code and error message

For authenticating with Facebook, this requires a Facebook Developer account. I should have everything set up with all the permissions but if anything is missing, please let me know!

Matching the hi-fi design for the login, sign up, and forgot password pages

We have the following pages: LoginV2, SignupV2, and ForgotPasswordV2
Edit the following pages so that they match the hi-fi design
Implement the appropriate util functions in the LoginV2 page to implement logging in with Google and Facebook

Acceptance Criteria

The specified util functions have been implemented and utilize the specified built-ins and provide the specified error-handling
The util functions have been appropriately implemented into the LoginV2 page

Resources

How to authenticate with Google in Firebase Auth
How to authenticate with Facebook in Firebase Auth
Hi-fi Wireframes

Me when auth:


NotFoundError (see above for traceback): Tensor name "edge_rnn/gru_cell/candidate/bias" not found in checkpoint files checkpoints/dual_graph_vrd_final_iter2.ckpt

when loading checkpoint files, the error occors.
Loading model weights from checkpoints/dual_graph_vrd_final_iter2.ckpt
Traceback (most recent call last):
File "./tools/test_net.py", line 91, in
viz_net(args.network_name, args.model, imdb, args.test_mode)
File "/home/yzzc/Work/csl/scene-graph-TF-release/tools/../lib/fast_rcnn/visualize.py", line 114, in viz_net
saver.restore(sess, weight_name)
File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/training/saver.py", line 1802, in restore
{self.saver_def.filename_tensor_name: save_path})
File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/client/session.py", line 900, in run
run_metadata_ptr)
File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/client/session.py", line 1135, in _run
feed_dict_tensor, options, run_metadata)
File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/client/session.py", line 1316, in _do_run
run_metadata)
File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/client/session.py", line 1335, in _do_call
raise type(e)(node_def, op, message)
tensorflow.python.framework.errors_impl.NotFoundError: Tensor name "edge_rnn/gru_cell/candidate/bias" not found in checkpoint files checkpoints/dual_graph_vrd_final_iter2.ckpt
[[Node: save/RestoreV2 = RestoreV2[dtypes=[DT_FLOAT, DT_FLOAT, DT_FLOAT, DT_FLOAT, DT_FLOAT, ..., DT_FLOAT, DT_FLOAT, DT_FLOAT, DT_FLOAT, DT_FLOAT], _device="/job:localhost/replica:0/task:0/device:CPU:0"](_arg_save/Const_0_0, save/RestoreV2/tensor_names, save/RestoreV2/shape_and_slices)]]
[[Node: save/RestoreV2/_13 = _Recvclient_terminated=false, recv_device="/job:localhost/replica:0/task:0/device:GPU:0", send_device="/job:localhost/replica:0/task:0/device:CPU:0", send_device_incarnation=1, tensor_name="edge_18_save/RestoreV2", tensor_type=DT_FLOAT, _device="/job:localhost/replica:0/task:0/device:GPU:0"]]
Caused by op u'save/RestoreV2', defined at:
File "./tools/test_net.py", line 91, in
viz_net(args.network_name, args.model, imdb, args.test_mode)
File "/home/yzzc/Work/csl/scene-graph-TF-release/tools/../lib/fast_rcnn/visualize.py", line 113, in viz_net
saver = tf.train.Saver()
File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/training/saver.py", line 1338, in init
self.build()
File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/training/saver.py", line 1347, in build
self._build(self._filename, build_save=True, build_restore=True)
File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/training/saver.py", line 1384, in _build
build_save=build_save, build_restore=build_restore)
File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/training/saver.py", line 835, in _build_internal
restore_sequentially, reshape)
File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/training/saver.py", line 472, in _AddRestoreOps
restore_sequentially)
File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/training/saver.py", line 886, in bulk_restore
return io_ops.restore_v2(filename_tensor, names, slices, dtypes)
File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/ops/gen_io_ops.py", line 1463, in restore_v2
shape_and_slices=shape_and_slices, dtypes=dtypes, name=name)
File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/framework/op_def_library.py", line 787, in _apply_op_helper
op_def=op_def)
File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/framework/ops.py", line 3392, in create_op
op_def=op_def)
File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/framework/ops.py", line 1718, in init
self._traceback = self._graph._extract_stack()  # pylint: disable=protected-access
NotFoundError (see above for traceback): Tensor name "edge_rnn/gru_cell/candidate/bias" not found in checkpoint files checkpoints/dual_graph_vrd_final_iter2.ckpt
[[Node: save/RestoreV2 = RestoreV2[dtypes=[DT_FLOAT, DT_FLOAT, DT_FLOAT, DT_FLOAT, DT_FLOAT, ..., DT_FLOAT, DT_FLOAT, DT_FLOAT, DT_FLOAT, DT_FLOAT], _device="/job:localhost/replica:0/task:0/device:CPU:0"](_arg_save/Const_0_0, save/RestoreV2/tensor_names, save/RestoreV2/shape_and_slices)]]
[[Node: save/RestoreV2/_13 = _Recvclient_terminated=false, recv_device="/job:localhost/replica:0/task:0/device:GPU:0", send_device="/job:localhost/replica:0/task:0/device:CPU:0", send_device_incarnation=1, tensor_name="edge_18_save/RestoreV2", tensor_type=DT_FLOAT, _device="/job:localhost/replica:0/task:0/device:GPU:0"]]

Make sure you have used the correct version of tensorflow (0.11)


GORA-424 Cache cursor size

MongoDBResult.getProgress calls cursor.size() which in turn issues count() request on underlying collection. This patch fixes this.

Is this PR file to reduce cost to calculate size from underlying collection?

This is major improvement if not critical bug.
We experienced very low performance of nutch fetch mapper as this progress meter issues count request on each url of thousands.
Also I will upgrade this patch to fix integer division bug later.

LGTM

+1

Looks good to me too.
I'll try to merge this bugfix later tonight.


Missing colon ( ' : ' ) in your ftplugin file. Makes autocommand not work properly.

Hello. I noticed that there seems to be a missing colon in the omnisharp_restart_all_servers auto command. The code is:
nnoremap <buffer> <Plug>(omnisharp_restart_server) OmniSharpRestartServer<CR>

I think it should be:
nnoremap <buffer> <Plug>(omnisharp_restart_server) :OmniSharpRestartServer<CR>

(Notice the colon before OmniSharpRestartServer)

You're right, thanks for pointing that out


chrome autocomplete hide background image of has-success, has-warning, has-danger

Using Chrome, if the user autocomplete an input with chrome feature the background become yellow and hide the background image from this css -> .has-success .form-control-success
How can we make the image appear over the autofill background-color ?
tried this hack but it still hide the background-image:
input:-webkit-autofill, textarea:-webkit-autofill, select:-webkit-autofill {
-webkit-box-shadow: 0 0 0px 1000px white inset !important;
}

Please provide a JS Bin demo of the bug.

should be easy to check with this simple demo
https://jsbin.com/haxemuzile/edit?html,css,output  (on chrome)

it's default browser behavior (show custom autocomplete background instead of any author-defined background) that i don't think can (or should) be overridden. only solution would be to explicitly suppress the behavior with autocomplete="off" on the input, or to reengineer the validation image/background to in bootstrap to not be on the form control itself, but on a separate wrapper

Relevant Stackoverflow

Closing as a won't fix given @patrickhlauke's comment.

@bkdotcom  this doesn't fix the problem it hide the yellow background but it still hide the image from bootstrap4.
Putting autocomplete to off is bad for a lot of situation where we want the autocomplete for UX.
So we cannot use the bootstrap 4 feature...
So there is useless CSS in bootstrap4 ... i though it would be better suited for cross-broswer, it seem's not,
Mobile first equal cross-browser in thoughts because the different device uses Safari, Chrome, Bing, ect ...
Bootstrap4 doesn't come with Mobile-First as they say.... google chrome is the most used broswer still in our days... and you fail at showing new feature that will not work for LOT of dev out there.

This work for me:
@-webkit-keyframes autofill-valid {
to {
/* Background image copied from: node_modules/bootstrap/dist/css/bootstrap.css selector: .was-validated .form-control:valid, .form-control.is-valid */
background-image: ($form-feedback-icon-valid);
}
}
@-webkit-keyframes autofill-invalid {
to {
/* Background image copied from: node_modules/bootstrap/dist/css/bootstrap.css selector: .was-validated .form-control:invalid, .form-control.is-invalid */
background-image: ($form-feedback-icon-invalid);
}
}
input.is-valid:-webkit-autofill {
-webkit-animation-name: autofill-valid;
-webkit-animation-fill-mode: both;
}
input.is-invalid:-webkit-autofill {
-webkit-animation-name: autofill-invalid;
-webkit-animation-fill-mode: both;
}

This webkit autofill animation trick doesn't seem to work anymore (April 2022, Chrome v100).


Trading divide by zero bug

Troubleshooting
Loot at attachment
log.txt
Version
v0.0.75
(c8bf045)
Description
Slack message give me this notification
Execution failed (14:14:29.481)
Job: Trailing Trade Indicator
Code: 16608
Message:
can't $divide by zero
- Current API Usage: 120
white_check_mark
eyes
raised_hands

4:14
Execution failed (14:14:32.928)
Job: Trailing Trade Indicator
Code: 16608
Message:
can't $divide by zero
- Current API Usage: 135`

To Reproduce
Do nothing :) when i stop and start the container the problem remain

I am having the same issue. I did two things and it stoped, not sure which of the two resolved it.

I switched Closed Trades view to D (from All)
Where applicable, I changed stop-loss from 1 to 0.999.

@BramFr  Hm, that is because of closed trade history.
Are you able to go to mongo container and get the following data and paste here?
$ docker exec -it binance-mongo /bin/bash
root# mongo
> use binance-bot
switched to db binance-bot
> db['trailing-trade-grid-trade-archive'].find();

And for the interim, you can clear grid trade history to make it work.
> db['trailing-trade-grid-trade-archive'].deleteMany({});

@BramFr Hm, that is because of closed trade history.
Are you able to go to mongo container and get the following data and paste here?
$ docker exec -it binance-mongo /bin/bash
root# mongo
> use binance-bot
switched to db binance-bot
> db['trailing-trade-grid-trade-archive'].find();

And for the interim, you can clear grid trade history to make it work.
> db['trailing-trade-grid-trade-archive'].deleteMany({});

Mongo cli log:
> use binance-bot
switched to db binance-bot
> db['trailing-trade-grid-trade-archive'].find();
{ "_id" : ObjectId("611a69b6d9afbea0e3c380a2"), "key" : "SHIBUSDT-2021-08-16T13:35:50+00:00", "symbol" : "SHIBUSDT", "baseAsset" : "SHIB", "quoteAsset" : "USDT", "buyGridTradeExecuted" : false, "sellGridTradeExecuted" : true, "allExecuted" : false, "totalBuyQuoteQty" : 0, "totalSellQuoteQty" : 1315.84907391, "buyGridTradeQuoteQty" : 0, "buyManualQuoteQty" : 0, "sellGridTradeQuoteQty" : 1315.84907391, "sellManualQuoteQty" : 0, "stopLossQuoteQty" : 0, "profit" : 1315.84907391, "profitPercentage" : 0, "buy" : [ { "triggerPercentage" : 1.02, "stopPercentage" : 1.01, "limitPercentage" : 1.0105, "maxPurchaseAmount" : 1500, "executed" : false, "executedOrder" : null } ], "sell" : [ { "triggerPercentage" : 1.03, "stopPercentage" : 0.95, "limitPercentage" : 0.9489, "quantityPercentage" : 1, "executed" : true, "executedOrder" : { "symbol" : "SHIBUSDT", "orderId" : 171583542, "orderListId" : -1, "clientOrderId" : "CMwinSbEhrG0hL1MRAdz7t", "price" : "0.00000888", "origQty" : "148014519.00", "executedQty" : "148014519.00", "cummulativeQuoteQty" : "1315.84907391", "status" : "FILLED", "timeInForce" : "GTC", "type" : "STOP_LOSS_LIMIT", "side" : "SELL", "stopPrice" : "0.00000889", "icebergQty" : "0.00", "time" :<PHONE_NUMBER>818, "updateTime" :<PHONE_NUMBER>017, "isWorking" : true, "origQuoteOrderQty" : "0.00000000", "currentGridTradeIndex" : 0, "nextCheck" : "2021-08-16T13:35:46.386Z" } } ], "archivedAt" : "2021-08-16T13:35:50+00:00" }
> db['trailing-trade-grid-trade-archive'].deleteMany({});
{ "acknowledged" : true, "deletedCount" : 1 }
> db['trailing-trade-grid-trade-archive'].find();


Eliminate warnings reported by Clang with -Weverything

I've tried to reduce the number of warnings reported, testing on Gentoo Linux amd64 (with Clang and GCC) and macOS 10.12 (with Clang from Xcode 8.2.1). When compiling internal tests with Clang (using clang -Weverything main.c mytests.c -O3 -o test), number of warnings went from 12 (main.c) + 46 (mytests.c) down to none (main.c) + 2 (mytests.c); GCC (using gcc -Wall -Wextra main.c mytests.c -O3 -o test) reported 7 warnings (in mytests.c only) before the changes and no warnings after.
Warnings still reported by Clang are in mytests.c and not directly related to ctest itself:
% clang -Weverything main.c mytests.c -o test
mytests.c:58:1: warning: empty struct has size 0 in C, size 1 in C++ [-Wc++-compat]
CTEST_DATA(fail) {};
^
./ctest.h:79:27: note: expanded from macro 'CTEST_DATA'
#define CTEST_DATA(sname) struct CTEST_IMPL_NAME(sname##_data)
                          ^
mytests.c:58:1: warning: empty struct is a GNU extension [-Wgnu-empty-struct]
./ctest.h:79:27: note: expanded from macro 'CTEST_DATA'
#define CTEST_DATA(sname) struct CTEST_IMPL_NAME(sname##_data)
                          ^
2 warnings generated.

Not exactly sure on the naming changes, so if you have other suggestions...

Unfortunately, I'm still seeing a lot of warnings from GCC related to CTEST2 in my own tests:
.../ctest.h:66:31: warning: redundant redeclaration of ‘ctest_blocklist_setup’ [-Wredundant-decls]
 #define CTEST_IMPL_NAME(name) ctest_##name
                               ^
.../ctest.h:91:26: note: in expansion of macro ‘CTEST_IMPL_NAME’
     void CTEST_IMPL_WEAK CTEST_IMPL_NAME(sname##_setup)(struct CTEST_IMPL_NAME(sname##_data)* data); \
                          ^
.../ctest.h:113:5: note: in expansion of macro ‘CTEST_SETUP’
     CTEST_SETUP(sname); \
     ^
.../ctest.h:126:30: note: in expansion of macro ‘CTEST_IMPL_CTEST2’
 #define CTEST2(sname, tname) CTEST_IMPL_CTEST2(sname, tname, 0)
                              ^
.../foo-test.c:79:1: note: in expansion of macro ‘CTEST2’
 CTEST2(foo, bar)
 ^
.../ctest.h:66:31: note: previous definition of ‘ctest_blocklist_setup’ was here
 #define CTEST_IMPL_NAME(name) ctest_##name
                               ^
.../ctest.h:92:26: note: in expansion of macro ‘CTEST_IMPL_NAME’
     void CTEST_IMPL_WEAK CTEST_IMPL_NAME(sname##_setup)(struct CTEST_IMPL_NAME(sname##_data)* data)
                          ^
.../foo-test.c:69:1: note: in expansion of macro ‘CTEST_SETUP’
 CTEST_SETUP (foo)
 ^

very nice work! merged with pleasure.

Ouch :) Thanks, I just thought I'd have time to fix those -Wredundant-decls warnings mentioned above, especially since it seems that I've added them myself in one of the commits here. Will open another PR once I figure it out then (or maybe someone else will sooner) ;)

Ok


Refactor MultipleAvatars

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

coming from https://github.com/Expensify/App/pull/19104#pullrequestreview-1436520238 :
We talked briefly 1:1 about a possible way to clean this up so that the styles and the props make more sense. Right now, the <MultipleAvatars> is very coupled to whatever is rendering it (ie. it needs to know that it's rendered in a report action, the search, the LHN, or an IOU preview, etc.). This makes it hard to reuse. Let's extract this out a little bit by adding these separate props to <MultipleAvatars>:

borderColor
borderColorHovered
borderColorPressed

NAB: at some point it would be good to split <MultipleAvatars> into two separate components:

<MultipleAvatars>
<MultipleAvatarsHorizontal>

Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~01f97faf1fade05575
Upwork Job ID:<PHONE_NUMBER>414840832
Last Price Increase: 2023-05-24

Started

not overdue

not overdue

not overdue

not overdue, refactor

^

I'm still planning on doing this refactor

not a priority

not a priority until after wave 8

^ making monthly

NAP

^


survival-table with big numbers

I am pretty new to survival analysis and I thought about using this library. My data is stored in "survival-table" format, and the values of "observed deaths" are big (millions). Thus it is impractical to convert into lifelines format: the resulting array will typically contain hundreds of millions of rows. Is there anything can be done in this case for KaplanMeierFitter?

Hey @vmarkovtsev, yes there is a way using the weights kwargs. Here's an example. Steps:

First I'm going to generate some data,
and then convert is to a survival table format,
"expand" the data again, but also consider how many observations we saw
and then use KaplanMeierFitter with weights

from lifelines.utils import survival_table_from_events
from lifelines import KaplanMeierFitter

T = np.random.randint(1, 75, size=1000)
E = np.random.binomial(1, 0.8, size=1000)
table = survival_table_from_events(T, E).reset_index()

table.head(10)
"""
   event_at  removed  observed  censored  entrance  at_risk
0       0.0        0         0         0      1000     1000
1       1.0       18        13         5         0     1000
2       2.0       11         9         2         0      982
3       3.0       12         7         5         0      971
4       4.0       12         5         7         0      959
5       5.0        8         7         1         0      947
6       6.0       14        12         2         0      939
7       7.0       14        11         3         0      925
8       8.0       14        14         0         0      911
9       9.0       14        11         3         0      897

"""

T_ = []
E_ = []
W_ = []

for _, row in table.iterrows():
    T_ = T_ + [row['event_at']] * 2
    E_ = E_ + [0, 1]
    W_ = W_ + [row['censored'], row['observed']]

kmf = KaplanMeierFitter()
kmf.fit(T_, E_, weights=np.array(W_))

This isn't an obvious solution, and I want to build some support for making this easier.
PS Great question, I'm going to add this to the docs.

Here's a better function:

def survival_events_from_table(survival_table, observed_deaths_col="observed", censored_col="censored"):
    """
    This is the inverse of the function ``survival_table_from_events``.

    Parameters
    ----------
    survival_table: DataFrame
        a pandas DataFrame with index as the durations and columns "observed" and "censored", referring to
           the number of individuals that died and were censored at time t.
    observed_deaths_col: str, optional (default: "observed")
        the column in the survival table that represents the number of subjects that were observed to die at a specific time
    censored_col: str,  optional (default: "censored")
        the column in the survival table that represents the number of subjects that were censored at a specific time

    Returns
    -------
    T: array
      durations of observation -- one element for observed time
    E: array
      event observations -- 1 if observed, 0 else.
    W: array
      weights - integer weights to "condense" the data

    Example
    -------
    >>> # Ex: The survival table, as a pandas DataFrame:
    >>>
    >>>                  observed  censored
    >>>    index
    >>>    1                1         0
    >>>    2                0         1
    >>>    3                1         0
    >>>    4                1         1
    >>>    5                0         1
    >>>
    >>> # would return
    >>> T = np.array([ 1.,  2.,  3.,  4.,  4.,  5.]),
    >>> E = np.array([ 1.,  0.,  1.,  1.,  0.,  0.])
    >>> W = np.array([ 1,  1,  1,  1,  1,  1])

    """
    columns = [observed_deaths_col, censored_col]

    T_ = []
    E_ = []
    W_ = []

    for t, row in survival_table.iterrows():
        if row[observed_deaths_col] > 0:
            T_.append(t)
            E_.append(1)
            W_.append(row[observed_deaths_col])
        if row[censored_col] > 0:
            T_.append(t)
            E_.append(0)
            W_.append(row[censored_col])

    return np.asarray(T_), np.asarray(E_), np.asarray(W_)

Thank you, this helped.


DiagnosticsStore folder on one of the nodes too big

Hello,
I have standalone SF cluster with 10 nodes (including 3 seed nodes) working on Service Fabric v.6.5.676.9590.
Several months ago I ran into a problem:
I deploying our application to this cluster every few months. After deployment in several weeks occurs the problem with disk space on one of the nodes. I discovered that DiagnosticsStore folder is too big, specifically this folder: DiagnosticsStore\fabriclogs-43466ece-0e6c-41af-8c87-33632debcfbc\VEC4PAAW_W2K12_NODE_NC4\Fabric. In this folder appears 5 files (4Mb each) every 5 minutes or about 5Gb per day. In contrast to this, on other nodes, I have only 1 or 2 files (100Kb - 1.5Mb) every 5 minutes:

Besides, the problem disappears by itself a few weeks later. Also, our SF-application works fine and I can't found the cause of the problem because I don't know how to analyze these *.dtr files.
One of zipped *.dtr files:
f962dc86d5d1af93b32db0e6ba8d879d_fabric_traces_6.5.676.9590_132344105321281492_26236_00637276824596925059_2147483647.dtr.zip
Can you help with resolving that issue?

I can tell this node is processing many FM queries which could be related to service resolutions. It's expected that system services that are being actively utilized would be logging at a faster rate than other nodes.

2020-6-13 22:00:01.503
FM.Query
5560
10308
[9ac86b82-4208-4ebc-b299-7d6461c2e62e:0] Query completed with error S_OK.

2020-6-13 22:00:01.504
FM.Query
5560
10308
[d8f1aa59-56e0-441e-a2cd-199a3aca0a28:0] Processing Query

2020-6-13 22:00:01.504
FM.Query
5560
10308
[d8f1aa59-56e0-441e-a2cd-199a3aca0a28:0] Query completed with error S_OK.

2020-6-13 22:00:01.506
FM.Query
5560
4496
[cc770f8f-902d-46ad-ba14-50e2eeb3515b:0] Processing Query

2020-6-13 22:00:01.506
FM.Query
5560
4496
cc770f8f-902d-46ad-ba14-50e2eeb3515b:0: Cache nodes: continuation token=, count=10

2020-6-13 22:00:01.506
FM.Query
5560
4496
[cc770f8f-902d-46ad-ba14-50e2eeb3515b:0] Query completed with error S_OK.

2020-6-13 22:00:01.509
FM.Query
5560
10164
[bfe568f0-27e4-4076-ad69-620cf007ae4f:0] Processing Query

2020-6-13 22:00:01.509
FM.Query
5560
10164
[bfe568f0-27e4-4076-ad69-620cf007ae4f:0] Query completed with error S_OK.

2020-6-13 22:00:01.510
FM.Query
5560
10164
[1dcdbb89-a42f-4619-868c-3684ce553a3a:0] Processing Query

2020-6-13 22:00:01.510
FM.Query
5560
10164
[1dcdbb89-a42f-4619-868c-3684ce553a3a:0] Query completed with error S_OK.

2020-6-13 22:00:01.511
FM.Query
5560
10164
[72a28208-a39c-488d-ac41-083477ac8a45:0] Processing Query

2020-6-13 22:00:01.512
FM.Query
5560
10164
[72a28208-a39c-488d-ac41-083477ac8a45:0] Query completed with error S_OK.

2020-6-13 22:00:01.513
FM.Query
5560
4496
[d74b30bf-ff74-46b2-af16-a016089d067a:0] Processing Query

2020-6-13 22:00:01.513
FM.Query
5560
4496
[d74b30bf-ff74-46b2-af16-a016089d067a:0] Query completed with error S_OK.

2020-6-13 22:00:01.514
FM.Query
5560
10164
[18b8fc75-e1dc-4144-8e72-3981be6f4f2b:0] Processing Query

2020-6-13 22:00:01.514
FM.Query
5560
10164
[18b8fc75-e1dc-4144-8e72-3981be6f4f2b:0] Query completed with error S_OK.

2020-6-13 22:00:01.515
FM.Query
5560
4496
[f5f3d93e-dc77-43bf-8e8a-fef572d80c03:0] Processing Query

2020-6-13 22:00:01.516
FM.Query
5560
4496
[f5f3d93e-dc77-43bf-8e8a-fef572d80c03:0] Query completed with error S_OK.

2020-6-13 22:00:01.588
FM.Query
5560
10308
[c672a968-e444-44c7-8f16-9b761d4cab0d:0] Processing Query

2020-6-13 22:00:01.588
FM.Query
5560
10308
c672a968-e444-44c7-8f16-9b761d4cab0d:0: Cache nodes: continuation token=, count=10

2020-6-13 22:00:01.588
FM.Query
5560
10308
[c672a968-e444-44c7-8f16-9b761d4cab0d:0] Query completed with error S_OK.

2020-6-13 22:00:01.591
FM.Query
5560
6596
[d027d444-5d52-4a93-8b41-1a47ba871e82:0] Processing Query

2020-6-13 22:00:01.591
FM.Query
5560
6596
[d027d444-5d52-4a93-8b41-1a47ba871e82:0] Query completed with error S_OK.

Hello @maburlik ,
So, if this expected behavior for Failover Manager then It is ok that we have log messages like this? (example from table above):
[d8f1aa59-56e0-441e-a2cd-199a3aca0a28:0] Query completed with error S_OK.

Query completed with error S_OK really means the error code for success, which is what we expect.

The diagnostics management process FabricDCA regularly monitors, organizes, and trims log data based on size quotas. If you are concerned with the amount of space used for logging data you can update your Diagnostics quota in the cluster configuration:
https://github.com/microsoft/service-fabric/issues?page=1&q=is%3Aissue+is%3Aclosed+sort%3Aupdated-desc
Diagnostics : DiskFullSafetySpaceInMB, MaxDiskQuotaInMB

Thank you for explanation. I'll close this issue.


Write "How to configure performers with plans" guide

Objective-C equivalent: https://github.com/material-motion/material-motion-runtime-objc#how-to-configure-performers-with-plans

Fixed in http://codereview.cc/D1791


docs: update cloudflare deploy instructions

The example TOML config is incorrect, updated based on the latest docs: https://developers.cloudflare.com/workers/cli-wrangler/configuration/#build

Thanks for the PR @manigandham but I'm afraid these changes are not necessary unless you have your own custom deployment flow which might differ in requirements.

@pi0 I think the issue here is that setting the build.command to True works fine for Linux systems, but throws an error on Windows. Using the updated config that this PR suggested works for me on Windows and seems like a good solution imo (altough I am facing some other issues with github actions specificially, but those seem unrelated).
Apologies for the lack of logs here, I just stumbled upon this while on my phone. Hopefully it makes sense at least 👌

PS C:\dev\ntest> wrangler publish
 Running true
'true' is not recognized as an internal or external command,
operable program or batch file.
Error: Build failed! Status Code: 1

PS C:\dev\ntest> wrangler publish
Error:  Workers Sites requires using a bundler, and your configuration indicates that you aren't using one. You can fix this by:
* setting your project type to "webpack" to use our automatically configured webpack bundler.
* setting your project type to "javascript", and configuring a build command in the `[build]` section if you wish to use your choice of bundler.

@pi0

(the guide is not meant for windows PC but github action with linux)

The guide has a "Deploy from your local machine using wrangler" section:

The guide uses the same config for both local and github actions. I don't understand the issue here. Why deviate from the official format in the Cloudflare docs only to be less compatible for people?

You can build command to "" for windows compatibility issue (docs updated in https://github.com/unjs/nitro/commit/91f53398c77613206367fae501e648fc40a41e04 thanks @PixelBoii for the hint!)

Why deviate from the official format in the Cloudflare docs only to be less compatible for people?

Dear @manigandham  There isn't really an issue with config! With nitro, unlike platform-specific CLIs (vercel, netlify, wrangler), nitro has a platform-agnostic dev/build stage + platform-specific deploy. We use wrangler only for deployment which is why not using build option in wrangler.toml. Nitro's usecase simply doesn't falls under basic usage guide of wrangler.


Client termination problem

I am calling the DefaultDockerClient from within a class that is run via the java goal of the exec-maven-plugin. All of the operations I do works fine but the procedure does not terminate. It seems as if the DefaultDockerClient creates some resource (e.g. thread) that do not allow my procedure to terminate. After 15 seconds of activity the exec plugin forcefully terminates. Any idea what this is and what could be done?

@helander is this still an issue? If not I can close this.

Please go ahead and close it.
Skickat från min iPhone

06 Jan 2015 kl. 21:36 skrev David Xia<EMAIL_ADDRESS>@helander is this still an issue? If not I can close this.
—
Reply to this email directly or view it on GitHub.


It is hard to see the pane divider in dark mode

Environment
Windows build number: 18362.239
Windows Terminal version (if applicable): 0.3.2142.0

Steps to reproduce
Open Windows Terminal in dark mode.
Add keybindings to split a tab into multiple panes. Ex:
            {
                "command": "splitHorizontal",
                "keys": [
                    "ctrl+shift+-"
                ]
            },
            {
                "command": "splitVertical",
                "keys": [
                    "ctrl+shift+|"
                ]
            },

Open a profile that has a dark background, like black or something close to that, ex, Campbell theme, where background is #0C0C0C.
Split the tab.
Expected behavior
To be able to easily distinguish the pane separator.
Actual behavior
In dark mode the pane separator is hard to see due to low contrast with surrounding background. Example:

In comparison, in light mode the separator is very visible:

Ideally, the color of the separator should take into consideration the surrounding background in order to show up with enough contrast to be visible in any combination.

Make it color configurable. Because I'd like red for dark mode but somebody might want blue, and another person green, and so on.

I am not of the opinion that we need a configuration option for every pixel that appears on the screen. It's a UI element: when we have a configurable UI, it'll "follow". :smile:

Ha yeah I don't believe there should be a config option for every pixel but with the configurable UI, I'd like control over each of the UI elements if possible. Probably can just make it match light mode for now as a default (though I'd appreciate you more if you made it red by default).

In related manner, it'd be nice to have a visual border between the tabs, too. At least for me I can't see any border at all.

In distant (unrelated?) topic, I can barely tell which tab is active.

The tab separator issue was already fixed upstream; we just need to ingest a new version of the package.
I'm going to close this out in favor of #1000, the epic for panes.

I'd like control over each of the UI elements if possible

@WSLUser Currently, the color of the separator in dark mode seems to match your highlight color setting under the system-wide Themes and related settings (find that via the Start menu), so if you want to customize it, that's the way to go :)


[HotFix][CheckStyle] Fix import related checkstyle

Be sure to do all of the following checklist to help us incorporate
your contribution quickly and easily:

[x] Any interfaces changed?
NO

[x] Any backward compatibility impacted?
NO

[x] Document update required?
NO

[x] Testing done
Please provide details on
- Whether new unit test cases have been added or why no new tests are required?
NO, only checkstyle fixed
- How it is tested? Please attach test report.
NO, only checkstyle fixed
- Is it a performance related change? Please attach the performance test report.
NO, only checkstyle fixed
- Any additional information to help reviewers in testing this change.
NO, only checkstyle fixed

[x] For large changes, please consider breaking it into sub-tasks under an umbrella JIRA.
NO, only checkstyle fixed

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/3580/

Build Success with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder/2342/

SDV Build Fail , Please check CI http://<IP_ADDRESS>:8080/job/ApacheSDVTests/3430/

retest this please

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/3592/

Build Success with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder/2354/

LGTM

merged into carbonstore branch


Nested shortcodes are actually what makes it break

@aurelienpierre Please rebase..

Not sure how it could already be non fast-forward, but anyway… rebased.


Custom middleware misses type in dashboard

Do you want to request a feature or report a bug?

Bug

What did you do?
Added a custom plugin using custom middleware and opened the Traefik dashboard to view the custom middleware.

What did you expect to see?
A custom middleware type defined by the plugin developer or at least something like 'Custom'
What did you see instead?
Empty middleware type

Output of traefik version: (What version of Traefik are you using?)

Version:      2.3.7
Codename:     picodon
Go version:   go1.15.6
Built:        2021-01-11T18:03:02Z
OS/Arch:      linux/amd64

Closed by #7782.

Closed by #7782.


add Getter/Setter of the service, characteristic, and descriptor.

From previous issue, I found noble uses two tiny C programs to read/write HCI and L2CAP, and works on Intel Edison(kernel 3.10).
So that, I created noblechild which provides almost same interface to paypal/gatt via noble's C programs.
Currently, noblechild can read/write attributes. But I have to make some changes to paypal/gatt.
This PR includes some setter/getter functions to use unexported variables in Server, Characteristic, and Descriptor. No other functions are changes.
Would you please merge this PR?
Thank you!

Sorry for the late response.
In the go gatt stack, we manage at the connections at L2CAP level, and we would like exclusive control.
This could be achieved only with the HCI USER CHANNEL, which was introduced since linux v3.14. Without it, user may feel some inconsistent issue due to both kernel and our stack try to manage the connections.
For minor changes that could help us get along with older kernels are fine. But changing API to expose more implementation might not worth it in my opinion.
Thanks,
Roy

Thank you for the response.
I understand and agree about you are not support without HCI USER CHANNEL kernel. So I don't include my proxy itself into this PR. I think my proxy should be independent. This PR only includes the missing piece to get interoperability with your great stack. I try to not change your code, but I couldn't. I think this PR has only minimal changes and it does not expose so much.
I know my approach is not usual and ugly. Even so, I want to support kernel 3.10 people like Intel Edison.
I would appreciate if you would think again.
Thank you,
Shirou

Though the intention was linux-specific, but the changes in this PR are pretty platform-independent.
It somewhat promotes (exposes) implementation to the API. But I think those handles are commonly presented in other BLE stack's API as well.
Thanks!

Thank you!!


Annotation fields missing

There are some more fields missing from annotations - I'm trying to reproduce the following exported json with the SDK, but I can't see how to set the following fields: tagKeys, titleFormat, step, textFormat, useValueForTime
      {
        "datasource": {
          "type": "loki",
          "uid": "${loki_datasource}"
        },
        "enable": true,
        "expr": "{cluster=~\"$cluster\",instance=~\"$instance\", transport=\"kernel\", level=\"emerg\"}",
        "hide": true,
        "iconColor": "light-purple",
        "name": "Critical system event",
        "tagKeys": "cluster,job,instance,level",
        "titleFormat": "Critical system event"
      },
      {
        "datasource": {
          "type": "prometheus",
          "uid": "${datasource}"
        },
        "enable": true,
        "expr": "changes(\nsum by (instance) (\n    group by (instance,release) (node_uname_info{cluster=~\"$cluster\",instance=~\"$instance\"})\n    )\n[$__interval:1m] offset -$__interval) > 1\n",
        "hide": true,
        "iconColor": "light-blue",
        "name": "Kernel update",
        "step": "5m",
        "tagKeys": "cluster,job,instance",
        "textFormat": "",
        "titleFormat": "Kernel update"
      },
      {
        "datasource": {
          "type": "prometheus",
          "uid": "${datasource}"
        },
        "enable": true,
        "expr": "node_boot_time_seconds{cluster=~\"$cluster\",instance=~\"$instance\"}*1000 > $__from < $__to",
        "hide": true,
        "iconColor": "light-yellow",
        "name": "Reboot",
        "tagKeys": "cluster,job,instance",
        "titleFormat": "Reboot",
        "useValueForTime": "on"
      },

For the moment, I have subclassed the models.dashboard.AnnotationQuery and builders.dashboard.AnnotationQuery classes, and added the extra fields there.

Thanks for reporting this!
I'll look into it :)


Make update deck for Tuesday 10/6 check-in

https://docs.google.com/presentation/d/1S98APjRdGHr_lPzIyR-dPoZa2G21g81Lhqm7eXDj2PI/edit#slide=id.g9c68f2b15_0_0


Pin doc-utils version

This PR fixes the style of TOC titles and unordered lists. It seems that it comes from a bug in docutils 0.17.0, which is a Sphinx dependency. Pinning docutils version to an older goes around the issue.
Related issues and PRs upstream:

https://github.com/sphinx-doc/sphinx/issues/9051
https://github.com/readthedocs/sphinx_rtd_theme/issues/1111
https://github.com/sphinx-doc/sphinx/pull/9053

Signed-off-by: EduPonz<EMAIL_ADDRESS>

Buildstatus:


Add dependencies for linux

This issue is caused by the absence of gnome-keyring and libsecret packages in Linux. Added these as dependencies to avoid this issue.

I believe the code Arch package is managed by a third party. They have added libsecret as a dependency by now, but not gnome-keyring. I'll close this issue as out-of-scope, for we do not manage that package.


Fix docs for client DSL changes in #3903

Fixes #3933.  Some of the regular crossbuilds failed with the regular flakes, which failed fast, and we never successfully ran the doc job before we merged #3903.

Post-reviews welcome, but things are horribly broken without this.


[Storage] az storage fs directory upload/download: Add back --auth-mode login as AzCopy supports Oauth now

Related command

Description

Testing Guide

History Notes

[Component Name 1] BREAKING CHANGE: az command a: Make some customer-facing breaking change
[Component Name 2] az command b: Add some customer-facing feature

This checklist is used to make sure that common guidelines for a pull request are followed.

[ ] The PR title and description has followed the guideline in Submitting Pull Requests.

[ ] I adhere to the Command Guidelines.

[ ] I adhere to the Error Handling Guidelines.

Storage


start e2e before all clusters are ready

Currently watcher will wait for all clusters to move to ready state and once it happens then notifies all threads to run e2e testing. If the user wants the created threads to start e2e before all clusters are moved to ready state, he can create a file named "e2e" dynamically in test directory. Watcher will check for this file and notify all threads to start running e2e tests.
With this new change, e2e testing will start in either

all clusters moved to ready state
or user created a file name "e2e" in test directory i.e /tmp//e2e

Fixes
https://github.com/cloud-bulldozer/perfscale-managed-services/issues/126

What are the next steps for this patch?
@morenod Please review and merge as appropriate


Trying to access Google Pixel 4 without USB Debugging?

Is it at all possible to use this tool WITHOUT USB Debugging enabled? My phone's screen is broken and this issue is PCB level. Would love to access my 3D scans using this tool!
Perhaps other scripts or methods that would allow this first and then I could use the scrcpy tool?
Thanks

Use an usb hub and connect mouse, screen, etc. Using mouse to turn on usb debug for scrcpy?

This is a good idea but the screen is broken.
Is there a USB OTG screen perhaps?

On Nov 17, 2021, at 8:08 AM, Yu-Chen Lin @.***> wrote:

Use an usb hub and connect mouse, screen, etc. Using mouse to turn on usb debug for scrcpy?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications on the go with GitHub Mobile for iOS or Android.

Maybe try these steps? Connect your device to a screen via HDMI, and plugin a mouse/keyboard to control the device?
If this doesn't work, I would say you're SOL.
https://www.alphr.com/access-android-broken-screen/


How to use in Docker Compose?

Hey thanks a lot for sharing this. I did not find out how to use Sherpa in Docker Compose, i'm not sure where the Flag --allow belongs to.
Could someone provide an example? Right now my compose file looks like this:
docker-api:
    ports:
      - "4550:4550"
    volumes:
      - /var/run/docker.sock:/tmp/docker.sock
    environment:
            - 'CONFIG=[ { "Path" : "*", "Access": "allow" } ]'
    image: "djenriquez/sherpa"

bump


fix(sql): workaround for filtering in query history api

fix(sql): workaround for filtering in query history api
This patch is intended to be a temporary workaround for the
inconsistency of the REST API [1]. The REST API states that all inputs
are query parameters, but in reality, they must be in the body.
This also works around the behavior of filter_by ignoring the
page_token when provided together by dropping the filter_by in
subsequent requests.
Tested: Ran the Python SDK locally and confirmed I was able to filter
and use the other parameters. Example code which filters by the
query start time.
from databricks.sdk import WorkspaceClient
from databricks.sdk.service.sql import QueryFilter, QueryInfo, TimeRange
w = WorkspaceClient(host=redacted, token=redacted)

filter = QueryFilter(
    query_start_time_range=TimeRange(
        start_time_ms=1689626553081,
        end_time_ms=1689626553083,
    )
)
assert len(list(w.query_history.list(filter_by=filter))) == 1
filter = QueryFilter(
    query_start_time_range=TimeRange(
        start_time_ms=1689626553083,
        end_time_ms=1689626553083,
    )
)
assert len(list(w.query_history.list(filter_by=filter))) == 0

Closes: https://github.com/databricks/databricks-sdk-py/issues/99

[x] make test run locally
[x] make fmt applied
[x] relevant integration tests applied

Thanks for raising this PR @lukehsiao! Unfortunately we won't be able to accept this PR as this file is autogenerated and your change will be reverted on the next generation pass. We will need to change the program used to generate these clients to handle this case. You can see which files are auto-generated in our SDK by looking at the first line of the file. If you want to address this change, the right place to do it is actually in the ApiClient implementation itself. You can inspect the request endpoint and modify it as needed there. Feel free to give that a shot!
FYI: internally, we're working on fixing the handling of parameters for GET and DELETE APIs across the entire API surface of Databricks. Most endpoints do handle parameters in a compliant manner, but this endpoint doesn't yet. I'll double check on the progress of this. However, the issue with filter_by and next_page_token will still need separate handling.

If you want to address this change, the right place to do it is actually in the ApiClient implementation itself. You can inspect the request endpoint and modify it as needed there. Feel free to give that a shot!

Apologies, can you say more here? I see the ApiClient class in core.py, but...it's not immediately obvious to me how that affects this file.

internally, we're working on fixing the handling of parameters for GET and DELETE APIs across the entire API surface of Databricks. Most endpoints do handle parameters in a compliant manner, but this endpoint doesn't yet. I'll double check on the progress of this. However, the issue with filter_by and next_page_token will still need separate handling.

An update here would be great! We would like to rely on Query History via this SDK in our application and having this work in the SDK would be very helpful.


HashToCurve and Hasher: Add new() method

The current HashToCurve / Hasher API's don't let one template over the Hash type. The issue at the moment is that the hash functions require operating on a struct instantiated elsewhere.
I suggest refactoring both of these to provide a new() -> Result<Self, Self::Error> API. The digest Trait, which allows for generalizing over cryptographic hash function, does something similar with just having a new() method.
I'm currently working on such a change, and would be happy to upstream it
EDIT: Its done in this branch (https://github.com/sikkatech/celo-bls-snark-rs/tree/make_hashes_have_new_fn ), sans for fixing the same doctests that are blocking #202

I like the idea in general a lot, though keep in mind the following (which was the main reason it was designed like this) - when instantiating a large BH hasher, we need to generate a lot of base points, which can take noticable time.

That makes sense. In those cases, I believe that we could make them return copies of a lazily evaluated singleton instance.
Another question is should the new() method take in the domain? I imagine in the general case, you can cache the work needed for domains / personalizations. (For instance if you set a personalization string to be the first block of what you hash.

That makes sense. In those cases, I believe that we could make them return copies of a lazily evaluated singleton instance.
Another question is should the new() method take in the domain? I imagine in the general case, you can cache the work needed for domains / personalizations. (For instance if you set a personalization string to be the first block of what you hash.

Yep, singleton instances might work here, which is similar to what we do now, but more explicitly.
About domain, I don't have a strong preference either way. In the use-cases I'm imagining this optimization would be hard to apply anyway.


No longer working over LTE

iOS 14.6 - Doesn’t seem to be working over LTE. Checking my DNS while connected to any DNA server through the app just comes back as my cellular service provider. Furthermore, Adguard’s Adblock DNS-over-HTTPS doesn’t work on WiFi or cellular with the app.

Hmm,
running iOS 14.6 on LTE and I get the the servers selected in DNSecure (1.1.1), both on DoT and DoH.
Maybe a local glitch or problems with the test site? Try this: https://browserleaks.com/ip


Webbrowser spammer plugin

If i have in the config:
node:
"enableProofOfWork": true
"enablePlugins"  "Spammer"
spammer:
"autostart": false
and want to start the spammer in the browser, all is fine.
But i if i want to disable the spammer again, that's not working, it is starting it again.
System: ubuntu
Browser: ungoogled chrome

The url /api/plugins/spammer/?cmd=settings is also always called even if the spammer is disabled via configuration.

The url /api/plugins/spammer/?cmd=settings is also always called even if the spammer is disabled via configuration. But I guess it is a dashboard problem...

That is currently the way the dashboard checks for the spammer plugin. Will be refactored soon(tm)


Apparent issues with build time dependencies inducing errors in running trestle.

Closing as not occuring any more.


A guide to node.js e-commerce

Great list! Thanks for keeping it clean and useful. Hopefully you find this guide to node.js valuable!
By submitting this pull request, I promise I have read the contribution guidelines twice and ensured my submission follows it. I realize not doing so wastes the maintainers' time that they could have spent making the world better. 🖖
⬆⬆⬆⬆⬆⬆⬆⬆⬆⬆

The blog post is too much of a Snipcart promotion.


fix(input-search): Fix type error in input search placeholder prop ts…

…x mode

Types of changes

[ ] New feature
[x] Bug fix
[ ] Enhancement
[ ] Component style change
[ ] Typescript definition change
[ ] Documentation change
[ ] Coding style change
[ ] Refactoring
[ ] Test cases
[ ] Continuous integration
[ ] Breaking change
[ ] Others

Background and context

Solution

How is the change tested?

Changelog

Component
Changelog(CN)
Changelog(EN)
Related issues

Checklist:

[ ] Test suite passes (npm run test)
[ ] Provide changelog for relevant changes (e.g. bug fixes and new features) if applicable.
[ ] Changes are submitted to the appropriate branch (e.g. features should be submitted to feature branch and others
should be submitted to main branch)

Other information

@ssjshengjie 这个问题会在 #3076 中统一解决掉


Net Framework v4.0.30319

Type of issue
Other (describe below)
Description
[Escriba sus comentarios aquí]net framework v4.0.30319. Tengo problemas

Page URL
https://learn.microsoft.com/es-es/dotnet/core/porting/upgrade-assistant-install#install-the-visual-studio-extension
Content source URL
https://github.com/dotnet/docs/blob/main/docs/core/porting/upgrade-assistant-install.md
Document Version Independent Id
af696b7d-2a60-7b0d-9318-bf7479974e2e
Article author
@adegeo
Metadata

ID: ac41b6b8-dc98-b63c-c2c3-57c38ff5dcf5
Service: dotnet-fundamentals

Sorry, this isn't a support forum. Please use Microsoft Q&A or some other forum for help. Looking briefly, it seems like you're missing a specific .NET Framework version. Contact Nero Vision for support. Thanks and sorry!