Deleting containers holds up the update bus on kubelet

This might just be WAI. We currently have something like (people who know better please correct):
apiserver
   |
reflector
   |
|(1) api  --(listen)>|
|(1) file --(listen)>|(merge)->(compress) ->[update BUS (50)] ->syncLoop
|(1) http --(listen)>|

Where (x) is a channel of capacity x

We can only get watch events as fast as we can compress in MergeState (which takes the lock on the pod storage and iterates through all its pods https://github.com/GoogleCloudPlatform/kubernetes/blob/master/pkg/kubelet/config/config.go#L337)
But still, that's mostly been ok in the little profiling I've done.

It's fairly easy to fill up that update bus with a few tries of:

cluster/kubectl.sh create rc --replicas=30
<wait for pods to start running>
cluster/kubectl.sh stop rc
cluster/kubectl.sh create rc --replicas=30

Bump replicas to 100 on a 3 core system  and it'll happen sooner :)
The stop and second create will fill the bus with 50 podupdates while SyncPods is still destroying containers from the delete. This causes merge to stall for the 51st update, which causes the reflector to stall. So essentially everything is held up on delete, which feels kind of backwards, because I usually don't care about deleted pods anymore?

There is a slight optimization to batch the [recent updates in 5 ms] (https://github.com/GoogleCloudPlatform/kubernetes/blob/master/pkg/kubelet/kubelet.go#L1618). Perhaps we can make it wait longer to batch more aggressively.
I think the kubelet's performance may have regressed since it switched to watching pods. Every pod status update kubelet sends would trigger an update from apiserver. We could potentially filter such updates when merging, or we could switch to using a more distributed per-pod merge/sync model. The latter is a bigger change and may not be needed for the 1.0. I will do some digging to see if this is truly needed.

The pod status shouldn't really be changing though unless you have crashy container right?

Yes, but if you do create and then stop rc, kubelet would still get the pod status update (to Running) in between those calls. These could fill up the channel as well, no?

Yes it would get the one set of pod status' that it just sent, those are usually snappy to process

Oh, those aren't sent to syncLoop if memory serves right, because the spec matches what you already have (def verify this)

I ran some tests on my local setup while setting GOMAXPROCS=1 for kubelet.
When replica count is 100, stop rc followed by create would fill up the channel, at which time kubelet could take up to a minute to finish the sync (mostly deleting the containers). This is because deleting containers are done in serial in SyncPods by one thread.  It reduced the sync time to <20 seconds. This could be lower if we let the go routines running int he background.
Can the performance be improved? Yes. However, given our v1.0 performance goal, I don't think the current performance is not tolerable.
Besides deleting, I think eventually we should tackle the problem that any pod worker (a go routine listening on a channel) may be slow (e.g. due to image pulling), and potentially dragging down the performance of other pods.

+1 on this probably being okay for v1.0.
On the pod worker front, we should definitely fix any issues that cause one pod worker to affect other pod workers. I don't believe we have any of those issues today though. The thread per pod worker model is intended to separate the effects of a slow job and if your job is "difficult" to start you're only hurting yourself.
I think the problem you're observing is that we don't actually perform all operations in pod workers. The deletes (of non-desired pods) are done globally as you mentioned. We can probably improve this by having the pod workers handle their deletion as well and potentially having some "unknown pod" pod workers for the rest. We'd be extending the lifetime of pod workers to more closely match the pod objects they track.
There are other global operations we do in SyncPods outside of pod workers (stale cleanups for almost everything), we'll probably need to move those too eventually as we start feeling their pain.

SGTM, do with this what you will :)

I think I've mistaken how pod worker worked. Each pod worker has a update channel of capacity one, but kubelet would not send any update if the pod worker is busy. Basically it would ignore/discard any incoming updates (with an extra buffer of one). That's why the syncing process would not be slowed down by a slow pod worker.
For v1.0, if we want to improve the bottleneck of deleting unwanted pods, I'd vote for simple parallelization of cleanup or a dedicated cleanup thread (as opposed to restructure pod worker for this purpose)

Simple parallelization SGTM

#6804 proposed to let user specify the grace period before SIGKILL the containers (i.e., essentially docker stop -t <timeout> <container_id). This may necessitate pushing the container killing to the per-pod go-routines since user can specify an arbitrary long timeout and docker stop would block until the operation is completed. Simple parallelization would still help, of course, but the longest timeout would dominate the sync time.
On the other hand, I wrote a simple micro-benchmark to test the effectiveness of paralleling killing the containers. The benchmark creates 100 containers, and then launches N workers to do docker inspect and docker stop; this mimics the docker operations in kubelet for killing a container. Take the results with a grain of salt since I ran it only on my desktop. With 1 worker, it took ~17s to kill the containers, while 5 workers took ~8s. Going beyond 5 workers didn't help. In short, parallelizing does help. In reality, the benefits might be more obvious because the containers could take longer to clean up (the default grace period is 10s), while in my benchmark the containers were doing nothing.

Closing this issue because #7048 parallelized the container killing in SyncPods.
Pushing container killing to pod workers would be easier and more meaningful after termination notice (#6804) is defined.

@yujuhong why would pushing to pod workers with the termination notice? If anything it seems like the other way around, termination would be easier if killing was done by pod workers.

I might have overstated by saying it'd be easier. What I really wanted to say was that "termination notice" and "killing by pod workers" should be designed together.  Termination notice definitely needs killing by pod workers so that it'd not unreasonably degrade kubelet's performance.
On the other hand, it also seemed like that termination notice would work better with the per-pod workers. Current pod-worker architecture relies on sending pod (specs) to per-pod workers. The lack of pod (specs) for the deleted pods makes killing by pod workers harder, or at least demands a big overhaul of existing structure. With the termination notice, I imagine the pod killing would work this way:
1. User sends a pod delete request to the apiserver.
2. Apiserver updates the pod to indicate that it should be terminated (note that the pod still exists)
3. Kubelet receives such a pod update and send it to the pod worker. Pod worker handles the killing which should be finished by the time the apiserver actually deletes the pod.
(There are of course other corner cases that we need to handle. This model just seems more compliant to how per-pod workers work today)
B assumes that we will use termination notice (or graceful termination) by default, which might be easier to implement. If kubelet needs to support both ways of killing (hard & graceful), then we'd probably want to wait until termination notice is mostly defined anyway. Either way, it doesn't seem worthy of making this change until we know the requirements.


how to display local pcap file?

hello Hesai team,
I have successfully catkin_make this repo.
I have a pcap file recorded by pandar40/40p, may I know how to use this repo to display the point cloud?
Thanks

rviz will help you.


Youtube "Scrobble only music category" never work

I noticed that the Youtube "Scrobble only music category" never works for me. For example this video (https://www.youtube.com/watch?v=Mb1ZvUDvLDY&feature=youtu.be) should scrobble but the web scrobblers icons simply stays saying "This site is supported fro scrobbling"

This is #590 again. Fixed in https://github.com/david-sabata/web-scrobbler/pull/594.

If this was fixed why I am still unable to scrollble from Youtube when the "Scrobble only music category" is checked. If I removed that setting and simply make Youtube scrobble everything, it works but that is not what I want.
Is it possible I am missing something about the "Scrobble only music category" feature. What is considered music category and how can I be sure I am looking at a music video that is from that category. For example every VEVO clip should be in that category right, but even those do not scrobble. I reinstalled the extension and its version is 1.47. Not sure what else I could do tbh.

@VladimirAmiorkov
Pull request is not merged yet. You can use this as a temporary solution:
https://github.com/aemxdp/web-scrobbler/releases/tag/v1.47-fixed

@aemxdp Thank you.


Junit5 enhancement

Update the project by using junit5 instead of junit4 for tests.

Isn't this a duplicate of #1710?

Correct!


docs/linux: fix trivial typing typo ida_rsa -> id_rsa

Here's my 2nd try.

Merged, thanks!

Thanks!
On Fri, Apr 15, 2022 at 19:23 Aleksandr Nogikh @.***>
wrote:

Merged, thanks!
—
Reply to this email directly, view it on GitHub
https://github.com/google/syzkaller/pull/3085#issuecomment-1100022691,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAHLBZ477CHFOLYVHL24MSDVFE7QNANCNFSM5TQCDPPQ
.
You are receiving this because you authored the thread.Message ID:
@.***>


conditional select fires on check

Using the conditional select there is no way to determine if the callback function was fired from the user clicking the checkbox or clicking the select area/text of the node
checkbox options
"checkbox": {
       "keep_selected_style": false,
       "three_state": false,
       "tie_selection": false,
       "whole_node": false
   },

It seems the target passed to conditional select event target is always the  tag.

well i was wrong.. event.target is class checkbox or class jstree-anchor.  oops


pull request for e2e test at 01/02/2017 08:26:07

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


unifont{,_upper}: 11.0.03 -> 12.0.01

Motivation for this change
http://lists.gnu.org/archive/html/info-gnu/2019-03/msg00002.html
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Assured whether relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

Looking good, thanks for the update. Rebased to master in 27fd948cf7a91c0d93f223423bc1c1c48be60eb4.

I also cherry-picked to the release-19.03 branch.


The test adapter ('Microsoft.VisualStudio.TestTools.TestTypes.Unit.UnitTestAdapter, Microsoft.VisualStudio.QualityTools.Tips.UnitTest.Adapter, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b03f5f7f11d50a3a') required to execute this test could not be loaded.

When I try to run the simple tests of the attached solution, I always get:
The test adapter ('Microsoft.VisualStudio.TestTools.TestTypes.Unit.UnitTestAdapter, Microsoft.VisualStudio.QualityTools.Tips.UnitTest.Adapter, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b03f5f7f11d50a3a') required to execute this test could not be loaded.

UnitTestProject2.zip
My VS version is:

@sailro - any ideas? I haven't been using VSTestHost in recent VS releases, so not sure if something has changed.

Yes, for several releases, the Test platform extension broken assembly resolution... I already created a bug for that, this should be resolved with 15.9.x
As a workaround you can try the following (this is what we are doing currently for our UI testing):
In [VS Studio Path]\Common7\IDE\Extensions\TestPlatform:
Create symbolic links to PrivateAssemblies and PublicAssemblies with the following (as admin):
mklink /D PrivateAssemblies ....\PrivateAssemblies
mklink /D PublicAssemblies ....\PublicAssemblies
Still in In [VS Studio Path]\Common7\IDE\Extensions\TestPlatform:
Create a Folder CommonExtensions\Platform, and copy Microsoft.VisualStudioTools.VSTestHost.15.0.dll into it.
And make sure that you used the latest InstallInVS2017.ps1 to install the test adapter, so that we are properly patching all configuration files.
Seb

Amazing.
Thanks @sailro .
With the currently latest version of InstallInVS2017.ps1 (I have used this version 4c5f2b7 on Feb 2) and your mentioned additional steps for a workaround work like a charm and the exceptions vanished.
As an additiontal hint for other people.
You have to set the testsettings file, otherwise incorrect paths are used for executing the devenv.exe.
Without the testsettings file:
TestContext Messages: Launching VisualStudio: "...\Microsoft Visual Studio 15.0\Common7\IDE\devenv.exe" Failed to launch devenv.exe (VisualStudio, 15.0) Restarting VS Launching VisualStudio: "...\Microsoft Visual Studio 15.0\Common7\IDE\devenv.exe" Failed to launch devenv.exe (VisualStudio, 15.0)
With one of the provided testsettings file in C:\Program Files (x86)\Microsoft Visual Studio\2017\Professional

You will get:

Thanks

I have hit this issue and followed the instructions to create the symbolic links and use the right .testsettings file, but I am still seeing this as the result of my test runs (tried my own project and the example at the start of this issue):

Launching VisualStudio: "...\Microsoft Visual Studio 15.0\Common7\IDE\devenv.exe"
Failed to launch devenv.exe (VisualStudio, 15.0)
Restarting VS
Launching VisualStudio: "...\Microsoft Visual Studio 15.0\Common7\IDE\devenv.exe"
Failed to launch devenv.exe (VisualStudio, 15.0)

Anyone have any ideas for what I can try / where I can look?


Remove separate license statement

with Premium, it is no longer a separate license. The customer does not need to separately purchase a user or Room License.

@JohanFreelancer9 Copy editing is needed for this PR. Thanks.

@LanaChin PR has been copyedited and is ready for final review, could you please check and merge? Thanks!


missing signature for release 0.6.7

Hi,
The release 0.6.7 is missing a signature. Would be nice if you could attach it, otherwise I am not able to release it in the arch linux repositories.

Haha thanks. I literally was just telling @dritter I needed to get this
done before you filed an issue against me for it! ;)
Sent from my mobile.
On Tue, Mar 19, 2019, 19:28 Syphdias<EMAIL_ADDRESS>wrote:

Assigned #1213 https://github.com/bhilburn/powerlevel9k/issues/1213 to
@bhilburn https://github.com/bhilburn.
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
https://github.com/bhilburn/powerlevel9k/issues/1213#event-2215299737,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAiw44yFGMQ6Uf2kxEcW1o_j55xRY3_rks5vYXKbgaJpZM4b4LES
.

Haven't forgotten! :)
On Tue, Mar 19, 2019 at 7:35 PM Ben Hilburn<EMAIL_ADDRESS>wrote:

Haha thanks. I literally was just telling @dritter I needed to get this
done before you filed an issue against me for it! ;)
Sent from my mobile.
On Tue, Mar 19, 2019, 19:28 Syphdias<EMAIL_ADDRESS>wrote:

Assigned #1213 https://github.com/bhilburn/powerlevel9k/issues/1213 to
@bhilburn https://github.com/bhilburn.
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
https://github.com/bhilburn/powerlevel9k/issues/1213#event-2215299737,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAiw44yFGMQ6Uf2kxEcW1o_j55xRY3_rks5vYXKbgaJpZM4b4LES
.


Changed "python.languageServer" to "Pylance" as per Issue #13007

Worked on Issue #13007 for OSD 2021.
Located the settings.json file after forking and cloning.
Replaced "python.languageServer" to "Pylance"

Hi @jasleen101010, thank you for your contribution!
Sorry for the unclear explanation in #13007, it was about changing the value of the python.languageServer setting. Pylance isn't a valid setting.
Remember to add a news entry thanking yourself as well!

Thanks for your help and quick reply @kimadeline ! Have made the changes as suggested and added a news entry. Thank you for helping me make my first official open-source contribution :D

Thank you again for your contribution @jasleen101010 🎉


tags of older versions

please don't delete older version tags. I had an issue when bower.json pointed to version 2.2.0 which doesn't exist anymore.
Thanks!

Can we add that tag back?  My Application is not working with the new update.

Re-added the v2.2.0 tag. Sorry for removing the previous one, I assumed it wasn't used. Lesson learned...


Dropdown: tempIndex is not reset when user enters text into filter

Steps to reproduce:

Go to State Dropdown example in docs site
Expand Dropdown
Hover over an option after the first item in the list
Type "Minnesota"
Hit Enter

Expected result: Dropdown closes as "Minnesota" option was already selected.
Actual result: Stack trace ...
  496 | },
  497 | () => {
  498 |   if (!this.state.active && this.state.tempIndex !== null) {
> 499 |     const newValue = this.state.options[this.state.tempIndex].props
      |                                                              ^  500 |       .value;
  501 |     this.optionOnClick(null, newValue, this.state.tempIndex);
  502 |   }

Suggested fix: Reset tempIndex to null in _handleChange setState call.

this issue is fixed by the latest commit in issue #762 since two issues are related.


No logging or compaction enabled for logs in snap internal /var/log/ folders

Steps to reproduce

Setup charmed-postgresql, connect an application to generate data. Deploying the default landscape-scalable bundle would be a good way to start generating logs.

Expected behavior
I expect the internal /var/log/ folder to start compacting old logs and eventually rotating them. I would like charm or snap level config knobs to alter this behavior (ie rotate logs after x days, keep logging directory under x GB, etc).
Actual behavior

files just build up in var/log/.

Versions

Operating system: 22.04.5 LTS

Juju CLI: 3.5.4

Juju agent: 3.5.0

Charm revision: 429

LXD: 5.0.3

Hi, @ethanmye-rs, at the moment the charm configures Postgresql to write a log every minute for every day of the week which will be overwritten at the turn of the week. So the charm will keep only 7 days worth of logs. Size of the logs will be variable, since the rotation is based on time, and there will always be around ~10k files, one for every minute of the week,

Thanks Dragomir for the context. Would you consider making this configurable at the charm/snap level? While this might be a reasonable config for some workloads, it is not ideal for others.

@dragomirp is it possible to re-use logrotated subordinate charm?
We typically use it for Openstack charms inside /var/log, but might be worth trying it for /var/snap...

Hi, @ethanmye-rs and @gustavosr98!
For that topic, I believe it's important to involve our PM here.
@7annaba3l, what's your thought about that?

Filed #696, which has some relation to this ticket as well I think.


Adding rounding extensions

@MadsKirkFoged let me know what you think about the implementation and naming of this. It's just something I needed a few times in a project so I though I'd add it to the library.
I chose to implement this using generic methods with T : BaseUnit, e.g.
public static UnknownUnit? RoundTo<T>(this T? value, T? roundVal) where T : BaseUnit

... so a user is forced to use the same Unit for value and roundValue, i.e. you can't round a Volume to a Mass etc. Alternatively, one could implement it like
public static UnknownUnit? RoundTo(this BaseUnit? value, BaseUnit? roundVal)

and rely on the runtime checks in Ratio to throw an exception.

I think we should try it out.
It could be nice if you didn't need to cast it back to the unit is was.
But I don't know if something like this would be able to compile?
public static T? RoundTo(this T? value, T? roundVal)

Yes this would be nice and I had something similar in mind at some point as well and played around a bit. I think there is no easy way to do this the way BaseUnit and the units inheriting from in is implemented right now.
This kind of shows you the issue:

... you cannot cast to T, and there is no constructor so you can't use new T().
I think this might work if BaseUnit were an abstract class that defined abstract constructors etc. But this would required quite a lot of changes as the current code uses BaseUnit quite a lot as a non-abstract class (I tried at some point and eventually decided is was not worth it). Maybe it's easy enough to create an abstract base class that BaseUnit inherits from. Not sure and I didn't try that.

The easier way to solve this would I think be to add such methods to the respective unit classes (not as extensions), using codegen.
You'd end up having the below, which compiles just fine:

This would do away with casting on the user side, which is nice, but it might not improve performance since you still do double * BaseUnit, which returns UnknownUnit, that is then cast to the result type (which might be fixable, see below).
Something like this might be useful for other methods as well. E.g. replacing the current AbsExtensions by non-static methods in the classes:

Similarly, there might be performance gained from creating overrides of quite a lot of methods defined in BaseUnit. For example plus, minus, equality operators etc where left and right are the same unit so we know the result is that unit, or multiplication by a double, which doesn't change the unit. Operations where left and right are different units should then still default to the methods in BaseUnit as currently.
This is not something I wanted to mess with for now but we can put it out there.


Prompt experiment data setup

Hi, thanks for sharing the code for this amazing work.
I am trying to run the prompt experiment, but cannot figure out the data setup. Could you please help me to create the needed data inputs (configs/data/prompt/gpt2_mixed.py)?
Thanks!

Thanks a lot! it helped.
Thanks for introducing the other paper too.


Product seo

The meta tags(Using for seo) are only available for the home page, the contact page and so on. I wonder how can I add proper meta tags which would hold a product characteristics. Like, when you share a product link on facebook or on whatsapp, so that the product image, the title, and a short part of the product description appears. Please, how can I acheive this?

@Grace-Endy
Which template you use? All products has individual title, description and keywords.


explicitly call the system bundler executable in bundler specs rather than depending on $PATH

What was the end-user problem that led to this PR?
A spec in the pristine command was failing in older versions of Ruby because the version of Bundler inside RubyGems was being loaded and executed instead of the current development version.
See: https://travis-ci.org/bundler/bundler/jobs/368955082
What was your diagnosis of the problem?
The $PATH variable when running bundle -v in this test had /path/to/bundler/exe listed as the first directory to search. This was causing /path/to/bundler/exe/bundle to be called which was loading Bundler in RubyGems to be loaded.
What is your fix for the problem, implemented in this PR?

Why did you choose this fix out of the possible options?
This seems like a simple solution and will prevent issues with making sure the $PATH is set correctly

This PR is not going to work because not all specs that execute the system Bundler have the stubbed Bundler gem installed, see travis.


0.4 fails to compile

Running Manjaro Linux. Got this when doing shards build crystalline --release --no-debug --progress -Dpreview_m on master:
Resolving dependencies
Fetching https://github.com/hugopl/version_from_shard.git
Fetching https://github.com/spider-gazelle/priority-queue.git
Fetching https://github.com/spider-gazelle/bisect.git
Fetching https://github.com/samueleaton/sentry.git
Installing version_from_shard (1.2.4)
Installing bisect (1.2.1)
Installing priority-queue (1.0.1)
Installing sentry (0.3.2 at cd86128)
Building: crystalline
Error target crystalline failed to compile:
/usr/bin/ld: cannot find /home/sardaukar/.asdf/installs/crystal/1.2.2/share/crystal/src/llvm/ext/llvm_ext.o: No such file or directory
collect2: error: ld returned 1 exit status
Error: execution of command failed with code: 1: `cc "${@}" -o /home/sardaukar/Code/FOREIGN/crystalline/bin/crystalline  -rdynamic -L/home/sardaukar/.asdf/installs/crystal/1.2.2/bin/../lib/crystal -lyaml  /home/sardaukar/.asdf/installs/crystal/1.2.2/share/crystal/src/llvm/ext/llvm_ext.o `"/usr/bin/llvm-config" --libs --system-libs --ldflags 2> /dev/null` -lstdc++ -lpcre -lm -lgc -lpthread -levent_pthreads -levent  -levent  -lrt -ldl`

I'm not sure why llvm_ext is not provided by the crystal installation here. You could either use docker to build crystalline or produce the llvm_ext object file manually.

Thanks!

https://github.com/crystal-lang/crystal/issues/12896


mintty: prepare for v3.6.1

This should fix the MinTTY 3.6.1 build failures.

Wrong repo? ;-)

Yes, also incomplete.


Size of checkpoint for pretrained RFID model doesn't match model size

I am trying to reproduce the results for inference using pretrained RFID model.
I have downloaded the checkpoint file from the One Drive.
I tried to run it with python nerf2_runner.py --mode test --config configs/rfid-spectrum.yml --dataset_type rfid
However it fails with:
RuntimeError: Error(s) in loading state_dict for NeRF2:
        size mismatch for attenuation_linears.0.weight: copying a param with shape torch.Size([256, 63]) from checkpoint, the shape in current model is torch.Size([256, 60]).
        size mismatch for attenuation_linears.5.weight: copying a param with shape torch.Size([256, 319]) from checkpoint, the shape in current model is torch.Size([256, 316]).
        size mismatch for signal_linears.0.weight: copying a param with shape torch.Size([256, 382]) from checkpoint, the shape in current model is torch.Size([256, 376]).

I believe that the model size is correct based on the paper but for some reason the checkpoint contains 3 extra columns whenever a 3D position vector is embedded.
I am working on a windows 11, so I am also wondering if new line encoding plays a role in the serialization / deserialization of the checkpoint.
Note that I have omitted the --gpu 0 switch because I haven't done a CUDA setup for this initial inference test. I removed the references to the GPU from the nerf2 runner and configured to run on CPU.
Any help or direction is much appreciated.

I met the same problem, so I had to retrain the model using the dataset set provided by the authors on GPU. Then, there's no problem with the test process.

For debugging purposes, I set include_input to False (refer to line 80 in model.py), which resulted in a bug preventing the loading of checkpoints. In the updated version, I have corrected this by setting include_input: True.

Forgot to close with the previous comment. Closing now.


Couchers User Survey

Google Form distributed by email(?) to active user base in order to prioritize feature development by alignment to users' interests.

@aapeliv Is this still active? Or should it move to to-do or backlog?

@aapeliv It is still active in terms of creating an insights document as a result for the community. I am on and off working a bit on it. Possibly we could move it also to a different ticket.


validation for assets

Tests
Bond + all fields less asset -> OK

Unbond all fields less asset -> OK

@enriquesouza

This message is not good , because there is no "red asterisk" on "Asset"

@enriquesouza  This message is not good , because there is no "red asterisk" on "Asset"

I will add the mandatory *

@johnnyluo the red start fix
UNBOND

BOND


(831) Handle errors in services

Context

We're writing a lot of very similar error-handling tests because of resolving client errors in controllers
Changes in this PR

Simplify organisationFromPrison util method
Handle errors in finding people/organisations in service
Fix an error code

Release checklist
Release process documentation
As part of our continuous deployment strategy we must ensure that this work is
ready to be released once merged.
Pre-merge

[ ] There are changes required to the Accredited Programmes API for this change to work...

[ ] ... and they have been released to production already

Post-merge

[ ] This adds, extends or meaningfully modifies a feature...

[ ] ... and I have written or updated an end-to-end test for the happy path in the Accredited Programmes E2E repo

[ ] This makes new expectations of the API...

[ ] ... and I have notified the API developer(s) of changes to the contract tests (Pact), or the API is already compliant

[ ] Manually approve release to preprod
[ ] Manually approve release to prod

 Tidy up the way we're handling errors in our service


Make sslheaders plugin better conform to documentation.

This PR is dependent on https://github.com/apache/trafficserver/pull/4700 .

@ywkaras Once it is approved it will be merged into master and will be on what will be the 9.0.x branch.  I will cherry pick it into the 8.0.x branch and we will need a new PR created for the 7.1.x branch.  Please made the PR for 7.1.x after this PR has been approved and merged to master.


fix: config file countdown counts from 20 to 0.

Previously, the countdown start from 0 to 20 seconds remaining, which is counterintuitive.

I noticed that this section of the code is using two separate timers, one uses a while loop and sleep() function while the other uses time(). The first one is used for timing and yields about 19.2 seconds time while the other (which is being fixed) is used to indicate the remaining time, which should be 20 seconds. So, this issue should be addressed soon.


How to disable Logic of Cleanup in Prompt

Survey will render a new line with Cleanup of Confirm by default.
It is not friendly to automation, so how can I config for skiping Cleanup invoke, just render raw response of user given?

I found Survey is not support reading from piped stdin or writing to piped stdout, so there is no need to deal with this problem.
Close it.


Create cloud function for rendering canvas as an image, use as token v2 URI

One notable aspect of aptos-canvas is each canvas is actually a v2 token. This means each canvas can benefit from the full token ecosystem, meaning it can be traded, sold, etc. One interesting aspect of this is, given each canvas is itself essentially an image, each token can be easily self described without linking out to external image data. A custom marketplace display module could use the canvas data itself rather than the metadata URI to show the token. For marketplaces that don't support this, the URI could be for a server / cloud function that looks up the canvas data on chain and returns it as an image.
So the URI would be like canvas-renderer.dport.me/0x123.png and the server looks up the Canvas at 0x123 and returns it as a png.

We now have an indexer based solution for this instead, though I did add support for the cloudflare worker at some point back when I represented the canvas using a 1d vector.


DF Manual Rolls: conflict with D&D Beyond Importer character features

** Conflicting Module(s)**
DF Manual Rolls v2.2.2
D&D Beyond Importer v2.4.8
Describe the conflict
DF Manual Rolls messes up some of character features descriptions on import of player character.
Expected behavior
It shouldn't interfere with importing features/items (didn't experiment further with what else it could break).
Screenshots (Optional)
Example of imported character(on left is the imported character, on right is feature from DDB Feature compendium, which was imported fine via feature munch in D&D Beyond Importer)

Settings on import

Link to character whose features descriptions breaks (both on Channel Divinity: Turn Undead and Mask of the Wild)
https://www.dndbeyond.com/profile/perrreira/characters/57084087

@Angry-Maid I'm unable to see how to get the character importer for this module. Could you provide some information on how to get that dialog?


Add support for (1) interleaved reads & (2) reading from pipes

I was exploring adding support for strobealign in NGLess and it would enable some optimizations if (1) it supports the interleaved reads format, including mixing paired and single-end reads and (2) reads can be piped in.
Adding support for the former is actually quite easy and I feel the result is slightly cleaner (less repeated code)
Adding support for piping in reads is a bit more complex because of the read length estimation. But the complexity is all hidden inside the InputBuffer/RewindableFile classes.

Cool, this would be nice to have! I’ll review the code as soon as I can. I’m happy you were able to get rid of some code duplication as well.
Just a note: I usually prefer smaller, self-contained commits (max. one refactoring, bug fix, added feature per commit), but it’s ok now. (No need to change anything now.)

Okay, I resolved the conflicts and cleaned up the history now. The design is cleaner too: there is a single RewindableFile and the KStream is embedded within it so that the two cannot get out of sync.

Thank you for this PR @luispedro ! Yes @marcelm, I am happy to merge this.

Feel free to add a "Thanks @luispedro" on these two bullet points.

Yes, I think its time to properly add credits to contributors in this repo (and elsewhere).

Thanks again!


Add solaris support

As solaris has a /proc this is pretty similar to linux. Restructured the code a little, so that only UnixProcess' Refresh method had to be implemented.
The assumption is that Executable() returns only the name of the binary started. On solaris this is truncated to 16 characters.

This looks pretty good, but we'd like to try to not introduce a cgo requirement for any OS on this library. It looks like the code though could probably be structured a lot like the FreeBSD one by recreating the psinfo struct in Go and loading it there. Would you be open to that?

I tried to do so, but failed. Problem seems to be, that the /proc/<pid>/psinfo file has 336 bytes (reported by ls and when reading with python and c). But whenever I read that file with golang, I get 416 bytes. Pretty clueless at the moment.

Well, the solution is that whatever I did to evaluate the size of the /proc/<pid>/psinfo file, I did with 32bit binaries. From man -s4 proc:

The format of the contents of any /proc file depends on the data model of the observer (the controlling process), not on the data model of the target process.

Now without the cgo dependency.

@mitchellh This LGTM and seems to work correctly on Illumos - merging will help port Nomad to Illumos.

@jen20 Sounds good, I just needed a verification + the removal of cgo. Both appear to now exist. :)


Fix missing cwd removal

@smatting removed dependencies on cwd with commit b21711f91b6c86f8d694200a87bca854ea31827c.
This commit removes one missing cwd usage.

Yep, I think I missed that one. Thanks! :+1:

Btw, are you able to get passing test? For me the same 2 tests fails (see log) on all branches: on the branch of this PR and on the two latest commits on master ( e43e6824c6ccf1d1d246f0cda7870c4a8d9d6dad and 5bc2ab54a5f20c7093d08d52354fa216ee2c94cd), haven't checked other branches. Maybe this is due do different git versions being used. I have git version 2.42.0 installed.

Btw, for me there are 2 additional tests failing which seem to be unrelated to the cwd isseu (see log). Are these failing for you as well? Maybe this is due do different git versions being used. I have git version 2.42.0 installed.

Yea, I have the same issue locally. Git 2.43.0. Haven't had time to dig into it yet 😢


Research Article | Introduction

Task to be done:  :octocat:

[x] Create a Research Article Introduction section In PDF format.
[x] Added the Article Introduction section file in the respective Docs folder.
[x] Use the Format Attached.

Attachment:
 Research Article_Format.docx :octocat:

Done :octocat:


Number of tracker server

Hi,
Can multiple tracker server help to increase the percent of media over p2p ?

Usually no but in some scenarios, it can increase P2P download ratio. I see to such cases:

High availability via redundancy. Using multiple trackers is good because if one fails/glitches/goes offline others may still function. Particularly if you use public trackers that are out of your control. In case of no trackers that function, you will get 0% P2P download ratio and 100% over CDN.

If for some reason different trackers contain a different list of peers for the same video stream using multiple trackers will result in more peers. More peers usually mean better P2P download ratio.


INSERT DEFAULT VALUES?

I have this table
CREATE TABLE users
(
	id serial NOT NULL,
	created_at timestamp DEFAULT CURRENT_TIMESTAMP NOT NULL,
	PRIMARY KEY (id)
) WITHOUT OIDS;

and want to create an user using default values.
I tried to define this structure:
#[derive(Insertable)]
#[table_name="users"]
pub struct NewUser {}

but it raises this error:
error: no rules expected the token `]`
  --> model/src/lib.rs:23:19
   |
23 | #[derive(Insertable)]
   |                   ^

It is reasonable because in SQL, inserting empty values is not allowed
db=# insert into users () values ();
ERROR:  syntax error at or near ")"
LINE 1: insert into users () values ();

However, with, DEFAULT VALUE you can insert values with default values.
actcast=# INSERT INTO users DEFAULT VALUES;
INSERT 0 1
actcast=# SELECT * FROM USERS;
 id |         created_at
----+----------------------------
  1 | 2017-04-26 01:12:56.90735
(1 row)

How can I use this feature through diesel? Or if not supported, could you support this (feature request)?

Of cause, I know I can do it with sql function
sql::<Integer>("INSERT INTO users DEFAULT VALUES RETURNING id")

This is a reasonable feature, but I'm unsure what I'd like the public API for it to be. If nothing else, #[derive(Insertable)] with no columns should fail to compile, as we're breaking our guarantees about runtime errors.

I suppose allowing empty structs to derive Insertable (in that case, empty structs will use DEFAULT VALUES) or providing another function like insert_defaults() will do.


How to run workers and pods on Secondary CIDR of VPC

We are running out of IP addresses for EKS workers and pods, and CNI custom networking seems complex.
Here in the blog post, option-2, they propose we can add another CIDER to VPC and use it for EKS workers and Pods without CNI usage.
https://aws.amazon.com/blogs/containers/optimize-ip-addresses-usage-by-pods-in-your-amazon-eks-cluster/
How can we do the AWS EKS-related changes using this module?

see https://github.com/aws-ia/terraform-aws-eks-blueprints/tree/main/examples/vpc-cni-custom-networking

@bryantbiggs thanks, that is the option-1 in the post, i am aiming for option-2

its the same but you need to ensure the control plane subnets are different than the subnets used by the nodes/pods. You can set the control plane subnet via control_plane_subnet_ids

@bryantbiggs is there some example that doesn't use ani CNI-related config, also currently we are using the module version 13.1.0, it is possible to do it if we currently stay on that version

Yes, I can leave the existing subnets as it is, but I want to create a new cider that can be used by workers and pods without having to do any CNI related stuff

@bryantbiggs Currently we are on version 13.2.1 , that looks very old and doesnt seem to have support for custom cider stuff, what should be the upgrade path if we need to upgrade to latest? upgrading directly from 13.2.1 to latest is possible?


Maybe it would be good to support simple mechanical structure development?

Maybe it would be good to support simple mechanical structure development?

Could you be more specific or share some reference?

Could you be more specific or share some reference?

Use case:
Train artificial intelligence to generate mechanical structures! (Complex examples: training generative models, generating electric generators, or designing robotic arms)
Simple example:
training generative models, generating new rocket nozzles, or exoskeleton structures (maybe complex?)

Just to make sure I understand correctly. You meant doing mechanical or robot design right? If so, there should be two major parts of it: (i) the optimization algorithm like topology optimization, evolution strategy, differentiable physics, etc and (ii) the simulation infrastructure to allow efficient evaluation (like parallelism), include interface for design, or provide gradients. The first point is sorta out of the scope of Genesis but the second one is on the plate. Stay tuned.
Some of our previous works did computational design for soft robots like this and this. Genesis should be sufficient to replicate some simpler algorithms (at least in the first work), e.g., using differentiable physics to optimize the geometry of a MPM body via a mask that represents the "existence" of a particle. For rigid body, there is an internal project working on it and may integrate the infra part once it's done.


[zh-cn]Fix bold is not displayed correctly

Bold is not displayed correctly

/language zh

/hold
This file is out of sync. Please don't do partial fix.
./scripts/lsync.sh content/zh-cn/docs/concepts/security/multi-tenancy.md

Hey @javadoors,
It's been a while since the last update, would you like to address issues in the previous comments so that we can move forward?

Hey @javadoors,
It's been a while since the last update, would you like to address issues in the previous comments so that we can move forward?

I think it should be able to merge now, right?


Improve BottomNavigationBar.title deprecation assertion message

Steps to Reproduce
Using the Bottom navigation bar, add some items using the new label property and one using the now deprecated title property,
Expected results:
Expected to see the bottom navigation bar working as normal
Actual results:
App crashes with exception items.every(BottomNavigationBarItem item) => item.label != null : Every item must have a non-null title or label
Bottom navigation bar doesn't allow for having a mismatch of items using either label or title, all items must use either title or label exclusively I'd like to use title for one of my items as i want it to look selected even when not officially selected, to be honest I think the title property shouldn't have been deprecated but maybe renamed, we the developers should just have the choice to use one or the other based on our requirements, as having a widget here is super useful

Hi @martipello
Can you please provide your flutter doctor -v and a minimal complete reproducible code sample?
Thank you

Hi @martipello
Can you please provide your flutter doctor -v and a minimal complete reproducible code sample?
Thank you

no, sorry I'm very busy, flutter doctor won't tell you anything its very easily reproducible even on the flutter example here https://api.flutter.dev/flutter/material/BottomNavigationBar-class.html , if i could do that I'd probably just make a pull request, if you don't want to so be it

The label and title assert/deprecation changes were made in https://github.com/flutter/flutter/pull/59127.
I think the problem is that the assert:
assert(
  items.every((BottomNavigationBarItem item) => item.title != null) ||
  items.every((BottomNavigationBarItem item) => item.label != null),
  'Every item must have a non-null title or label',
),

Is checking that either all of the items have a label or they all have a title. This seems reasonable, since the idea is for developers to migrate (all at once) to the label property. Unfortunately, the error message is misleading.

or you could let us use the title property or the label property, and just show a warning that it doesn't support tooltips

@martipello The idea is for developers to migrate all at once as said above, because deprecated properties will be removed in the future and you don't want to face this problem later.

@pedromassango im well aware of that im proposing that it shouldnt be deprecated as it serves a real purpose and going from using a widget to a string kills this functionality

BottomNavigationBar.title was removed a while ago after the deprecation period ended. I am going to close this issue since it does not exist anymore.


not work properly in vue single file components

as topic say

I'm not familiar with Vue, so you need to write a better description of the problem

Same issue with Vue.Js.
The *.vue files have a combination of HTML, JS, and CSS each in their own sections.
The file structure follows this pattern:
<template>
... html
</template>
<script>
... js
</script>
<style>
... css
</style>

Issue with Emoji Code snippets:
<template>
   <!-- Wrong format for HTML, should be 🐕 -->
   <span class="test"> \u{1F415} </span>
</template>
<script>
   // Correct in JS:
   let test = '\u{1F415}'
</script>
<style>
   .test::before {
      /* Correct in CSS: */
      content: '\1F415';
   }
</style>

Can you please place the cursor inside an empty pair of <template> tags and run the Developer: Inspect Editor Tokens and Scope command? I'm after the Textmate scope. Thanks!

Hi,
Same issue here.
Here is the result of the inspect into the template, script and style.
Template

language
vue-html

standard token type
Other

foreground
#ABB2BF

background
#282C34

contrast ratio
6.57

textmate scopes
text.html.vue-htmlsource.vue

foreground
No theme selector

Script

language
javascript

standard token type
Other

foreground
#ABB2BF

background
#282C34

contrast ratio
6.57

textmate scopes
source.jssource.vue

foreground
No theme selector

Style

language
css

standard token type
Other

foreground
#ABB2BF

background
#282C34

contrast ratio
6.57

textmate scopes
source.csssource.vue

foreground
No theme selector

Hope it helps to correct the bug.
Thanks and very useful extension btw 👍

Thanks @Srajdax, that's exactly what I was looking for. Does the new version fix this for you?

Yes,
It fixes the bug 😃
Thanks @idleberg


Added support for env vars

This allows tasks to have environment variables passed to them so you can do things like the following and ensure that the proper items are in place before running tests:
tasks: [
  {:cmd, "mix ecto.setup", env: [{"MIX_ENV", "test"}]},
  "mix test"
]

If no args are passed, and empty list is provided to System.cmd/3

Pull Request Test Coverage Report for Build 138

2 of 2 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.7%) to 78.462%

Totals

Change from base Build 137:
0.7%

Covered Lines:
51

Relevant Lines:
65

💛  - Coveralls

Friendly bump :)

Hey @akoutmos sorry I didn't have to take a look at this, I want to get ride of the TODO in the code instead of allowing it on credo.
I will merge it ASAP, thanks for the contribution! 🎉

Thanks for getting back to me! Are you talking about this one? https://github.com/qgadrian/elixir_git_hooks/blob/master/lib/mix/tasks/git_hooks/run.ex#L77

Want me to remove the TODO in this PR?

@akoutmos, please do. I updated master branch with the deprecation so no more TODOs pending anymore

Reverted the credo change. Let me know if there is anything else you would like me to do :)

Thanks for the contribution @akoutmos 🎉

No problem :). Thanks for the awesome lib!!


Do you have any update experiments?

Do you plan to update experiments? such as on widerface dataset for training and evaluation.

Sorry for late reply, I'll update code as soon as possible.


ros service to mqtt can't make it work

@marc-hanheide, thanks for this awesome tool
I'm trying to expose a ros service through a mqtt broker using your bridge. I'm unable to do it.
the ros-service-2-mqtt-broker side seems to be fine, I could see some activity on the mqtt-broker
On the local machine the service returns the following when is called:
rosservice call /rb1_base/image_provider/comm_enable "data: false"
success: True
message: "Picture send disabled"

but I cannot make it work, when I do on the remote machine:
rosservice call /rb1_base/image_provider/comm_enable "data: false"

or
rosservice call /remote/image_provider/comm_enable "data: false"

I won't get any response (I have ctrl+c the command)
The topics works really good with the same mqtt broker.
I use the following tag of the bridge
mqtt_bridge tag: 1.2.2
Server configuration yaml:
---
mqtt:
  client:
    protocol: 4      # MQTTv311
  connection:
    host: $(arg mqtt_host)
    port: $(arg mqtt_port)
    keepalive: 60
  private_path: device/001
serializer: json:dumps
deserializer: json:loads
bridge:
  - factory: mqtt_bridge.bridge:DynamicBridgeServer
    control_topic: '__dynamic_server'
  - factory: mqtt_bridge.bridge:LocalServiceProxy
    control_topic: '__dynamic_server'
    local_server: /rb1_base/image_provider/comm_enable
    remote_server: /remote/image_provider/comm_enable
    srv_type: std_srvs.srv:SetBool

Remote configuration yaml:
---
mqtt:
  client:
    protocol: 4      # MQTTv311
  connection:
    host: $(arg mqtt_host)
    port: $(arg mqtt_port)
    keepalive: 60
  private_path: device/001
serializer: json:dumps
deserializer: json:loads
bridge:
  - factory: mqtt_bridge.bridge:DynamicBridgeServer
    control_topic: '__dynamic_server'
  - factory: mqtt_bridge.bridge:RemoteService
    control_topic: '__dynamic_server'
    local_server: /rb1_base/image_provider/comm_enable
    remote_server: /remote/image_provider/comm_enable
    srv_type: std_srvs.srv:SetBool

on the mosquito broker I can the see the following:
mosquitto_sub -t '__dynamic_server/#' -d
Client mosq-0pRVZ6vcjoKjbbyO1T sending CONNECT
Client mosq-0pRVZ6vcjoKjbbyO1T received CONNACK (0)
Client mosq-0pRVZ6vcjoKjbbyO1T sending SUBSCRIBE (Mid: 1, Topic: __dynamic_server/#, QoS: 0, Options: 0x00)
Client mosq-0pRVZ6vcjoKjbbyO1T received SUBACK
Subscribed (mid: 1): 0
Client mosq-0pRVZ6vcjoKjbbyO1T received PUBLISH (d0, q0, r1, m0, '__dynamic_server/service/register/_rb1_base_image_provider_comm_enable_TO__remote_image_provider_comm_enable', ... (212 bytes))
{"args": {"local_server": "/remote/image_provider/comm_enable", "srv_type": "std_srvs.srv:SetBool", "remote_server": "/rb1_base/image_provider/comm_enable", "control_topic": "__dynamic_server"}, "op": "register"}

my remote node:
rosnode info /rb1_base/mqtt_bridge_test
--------------------------------------------------------------------------------
Node [/rb1_base/mqtt_bridge_test]
Publications:
 * /rosout [rosgraph_msgs/Log]

Subscriptions: None

Services:
 * /rb1_base/image_provider/comm_enable
 * /rb1_base/mqtt_bridge_test/get_loggers
 * /rb1_base/mqtt_bridge_test/set_logger_level
 * /remote/image_provider/comm_enable

contacting node http://<IP_ADDRESS>:34737/ ...
Pid: 63
Connections:
 * topic: /rosout
    * to: /rosout
    * direction: outbound
    * transport: TCPROS

Can you help me with that?
Thanks in advance

This is likely just a configuration problem you are having. You don't want to configure the service on both side, but only at one. The respective other side is automatically taken care of by the DynamicBridgeServer.
So, I think you'd want to remove the LocalServiceProxy from the server config:
---
mqtt:
  client:
    protocol: 4      # MQTTv311
  connection:
    host: $(arg mqtt_host)
    port: $(arg mqtt_port)
    keepalive: 60
  private_path: device/001
serializer: json:dumps
deserializer: json:loads
bridge:
  - factory: mqtt_bridge.bridge:DynamicBridgeServer
    control_topic: '__dynamic_server'
#  - factory: mqtt_bridge.bridge:LocalServiceProxy
#    control_topic: '__dynamic_server'
#    local_server: /rb1_base/image_provider/comm_enable
#    remote_server: /remote/image_provider/comm_enable
#    srv_type: std_srvs.srv:SetBool

Thanks a lot @marc-hanheide.
Removing the dynamic server on side made it work.
Thanks your support


Fix d6 model

Testing d6 model (Vansyngel 2016 arXiv:1611.02577) with the current presets configuration was not working. Two things were broken:

The initialization step was not passing some data members to the base class. Initialization of d6 model is fixed by https://github.com/healpy/pysm/commit/240b570cfafbfe6e1e8869ee82ffb04bc4102953. Some older data member (pixel_indices, for instance or MPI comm) have also been removed.

The emission computation was done for a list of frequencies and then returning an emission per frequency, I, Q, U and number of pixels. Commit https://github.com/healpy/pysm/commit/e5e5274e9b063b35a6f7ceb57835236116a65cc8 should address this, keeping the original code as much as possible (so, there might be room for optimization, numba for instance). The point is to integrate other frequencies at the end of d6 emission computation.

thanks! could you also contribute a unit test?

Ok do I create another PR in https://github.com/healpy/pysm-data/tree/master/pysm_2_test_data or I put fits files somewhere and you will push them into pysm-data repository ?

the first, thanks!
On Thu, Mar 12, 2020 at 12:29 PM Xavier Garrido<EMAIL_ADDRESS>wrote:

Ok do I create another PR in
https://github.com/healpy/pysm-data/tree/master/pysm_2_test_data or I put
fits files somewhere and you will push them into pysm-data repository ?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/healpy/pysm/pull/43#issuecomment-598374840, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAC5Q4VOZIRBLI3OR2HNXXLRHEZ2DANCNFSM4LGPLJWQ
.

Done. I have used the same cryptic name for test data files but I see that for d7 model, you use a different pattern. I leave you change the name as you wish. Note that the computation at 353 GHz is buggy and so I skip the check for this frequency (btw it's working à 352 or 354 GHz...)


23.3.0 broken

Expected Behavior / New Feature
the routing via consul broke in 23.3+
"ServiceDiscoveryProvider": {
	"Scheme": "http",
	"Host": "consul",
	"Port": 8500,
	"Type": "PollConsul",
	"PollingInterval": 1000
}

Actual Behavior / Motivation for New Feature
warn: Ocelot.Responder.Middleware.ResponderMiddleware[0]
      requestId: 0HN5I4V77ST2E:00000002, previousRequestId: No PreviousRequestId, message: 'Error Code: ConnectionToDownstreamServiceError Message: Error connecting to downstream service, exception: System.Net.Http.HttpRequestException: Connection refused (consul:81)
       ---> System.Net.Sockets.SocketException (111): Connection refused
         at System.Net.Sockets.Socket.AwaitableSocketAsyncEventArgs.ThrowException(SocketError error, CancellationToken cancellationToken)
         at System.Net.Sockets.Socket.AwaitableSocketAsyncEventArgs.System.Threading.Tasks.Sources.IValueTaskSource.GetResult(Int16 token)
         at System.Net.Sockets.Socket.<ConnectAsync>g__WaitForConnectWithCancellation|285_0(AwaitableSocketAsyncEventArgs saea, ValueTask connectTask, CancellationToken cancellationToken)
         at System.Net.Http.HttpConnectionPool.ConnectToTcpHostAsync(String host, Int32 port, HttpRequestMessage initialRequest, Boolean async, CancellationToken cancellationToken)
         --- End of inner exception stack trace ---
         at System.Net.Http.HttpConnectionPool.ConnectToTcpHostAsync(String host, Int32 port, HttpRequestMessage initialRequest, Boolean async, CancellationToken cancellationToken)
         at System.Net.Http.HttpConnectionPool.ConnectAsync(HttpRequestMessage request, Boolean async, CancellationToken cancellationToken)
         at System.Net.Http.HttpConnectionPool.AddHttp2ConnectionAsync(QueueItem queueItem)
         at System.Threading.Tasks.TaskCompletionSourceWithCancellation`1.WaitWithCancellationAsync(CancellationToken cancellationToken)
         at System.Net.Http.HttpConnectionPool.SendWithVersionDetectionAndRetryAsync(HttpRequestMessage request, Boolean async, Boolean doRequestAuth, CancellationToken cancellationToken)
         at System.Net.Http.DiagnosticsHandler.SendAsyncCore(HttpRequestMessage request, Boolean async, CancellationToken cancellationToken)
         at Ocelot.Requester.TimeoutDelegatingHandler.SendAsync(HttpRequestMessage request, CancellationToken cancellationToken)
         at Ocelot.Requester.MessageInvokerHttpRequester.GetResponse(HttpContext httpContext) errors found in ResponderMiddleware. Setting error response for request path:/some/path/to/accesss, request method: POST'

Steps to Reproduce the Problem

call something
Specifications

Version: 23.3.0,23.3.3
Platform: Linux alpine container
Subsystem:

Duplicate of #2119

Hello Leon,
Thank you for bringing the broken Service Discovery functionality in version 23.3.0 to our attention!
The Consul provider was indeed affected by a significant redesign of the provider's internal interfaces. We are aware of this issue and are diligently working on the v23.3 Hotfixes milestone.
However, we believe the stability of the provider in version 23.2.2 is quite solid.
We recommend rolling back to version 23.2.2 and awaiting the upcoming 23.3.4 release, which will include the necessary hotfixes.
Your patience is greatly appreciated.

Fixed in version 23.3.4 ✔


Cant import panda_model

When I try to import the panda_model_download function (or run it through the command line) the following error is yielded:

Hi,
thanks for pointing this issue out to me. This was kind of specific to Ubuntu 20.04 and not caught by the unittests, but should be fixed now 629395b.


fix: not closing channel header modal

modal now working properly on set header button

@nadalfederer Would you please fix the linter errors:
/root/mattermost-webapp/components/edit_channel_header_modal/index.js
  12:1  error  `./edit_channel_header_modal.jsx` import should occur after import of `../../utils/constants`  import/order
  12:1  error  There should be at least one empty line between import groups                                  import/order

Then I can bring this up for review.

This issue has been automatically labelled "stale" because it hasn't had recent activity.
A core team member will check in on the status of the PR to help with questions.
Thank you for your contribution!
/cc @jasonblais @hanzei

Hi @nadalfederer,
One test is still failing:
Summary of all failing tests
 FAIL  components/edit_channel_header_modal/edit_channel_header_modal.test.jsx
  ● components/EditChannelHeaderModal › should match snapshot, init

    Unexpected console logsWarning: Failed prop type: The prop `show` is marked as required in `EditChannelHeaderModal`, but its value is `undefined`.
        in EditChannelHeaderModal

      67 | afterEach(() => {
      68 |     if (logs.length > 0 || warns.length > 0 || errors.length > 0) {
    > 69 |         throw new Error('Unexpected console logs' + logs + warns + errors);
         |               ^
      70 |     }
      71 | });
      72 |

      at Object.<anonymous>.afterEach (tests/setup.js:69:15)

Test Suites: 1 failed, 449 passed, 450 total

I'm sorry that I've missed this error. It fixed now

/update-branch

Creating a new SpinWick test server using Mattermost Cloud.

Mattermost test server created! :tada:
Access here: https://mattermost-webapp-pr-4551.test.mattermost.cloud

Account Type
Username
Password

Admin
sysadmin
Sys@dmin123

User
user-1
User-1@123

Test server destroyed


Upgrade the upstream eclipse-che operator to the 7.0.0-rc-2.0 release

Updates to existing Operators

[x] Is your new CSV pointing to the previous version with the replaces property?
[x] Have you tested an update to your Operator when deployed via OLM?

Your submission should not

[x] Modify more than one operator
[x] Submit operators to both upstream-community-operators and community-operators at once
[x] Modify any files outside the above mentioned folders
[x] Contain more than one commit. Please squash your commits.

Operator Description must contain (in order)

[x] Description about the managed Application and where to find more information
[x] Features and capabilities of your Operator and how to use it
[x] Any manual steps about potential pre-requisites for using your Operator

Operator Metadata should contain

[x] Human readable name and 1-liner description about your Operator
[x] Valid category name1
[x] One of the pre-defined capability levels2
[x] Links to the maintainer, source code and documentation
[x] Example templates for all Custom Resource Definitions intended to be used
[x] A quadratic logo

@J0zi @dmesser @robszumski  Any idea why The OperatorHub.io eclipse-che page is still showing the 7.0.0-beta-5.0 as the current version ?
https://operatorhub.io/operator/eclipse-che
cc @ibuziuk

@davidfestal It is because of wrong filename for current version. File eclipse-che.v7.0.0-rc-2.0.clusterserviceversion-v7.0.0-rc-2.0.yaml has invalid name as it is a CSV file, it must end as .clusterserviceversion.yaml.


package devel

as part of the next generation of externpro, projects should build and package standalone
this issue is to track commits and design decisions related to packaging (with COMPONENT devel for development), including what is included in the package
also any cmake necessary to download and extract and find the devel package

xp_patch in cmake/pros.cmake as of commit https://github.com/externpro/externpro/commit/c1e5f71f83d300ea235d7851f0e8f2995628275b
is now coming from an externpro build artifact
set(xp github.com/externpro)
...
set(xp_patch REPO ${xp}/xppatch TAG v2.7 MD5_Linux a3987347f9f1717e768e84df33acb9c7 MD5_win64 45b20d653ab537735fb91ca82f9ca5c9)

but xp_patch can be overridden by a project to point to a local dist directory or locally built package, for example:
set(xp_patch DIST_DIR /bpvol/workspace/xppatch/_bld-Linux/dist)
set(xp_patch URL_Linux /bpvol/workspace/xppatch/_bld-Linux/patch-v2.7-Linux-devel.tar.xz MD5_Linux c2388acec3fb9d4d9374b0a424426999)

the xp_patch (or any other xp_<project>) override just needs to happen after the default is set -- which as of the latest commit happens in xpCommonFlags(), which is called from both include(flags) and xpSetFlags()

any future enhancements to package devel will be done under a new issue
this issue is complete with the commits to the main branch referenced above


Starcraft 2 achievements are very incorrect

I'm trying to use this plugin to view SC2 achievements, but although I seem to be able to connect and sync, there are a lot of inaccuracies in the data.
Specifically:

I am missing many achievements that I have unlocked in the game (51 reported in SuccessStory, vs. hundreds in the game client)
Many achievements incorrectly show as being held by 100% of players
Many achievements incorrectly show as being held by 0% of players

The plugin uses the API to get the data.
It's weird not having it all.
Is your profile public so I can check it out?

It should be public, yes. My tag is Sirrus233#1184


Two way communication using Aeron

I want to build a two way communication ( similar to a client - server ) rough code  for the client which first starts the transmission is like as follows
main()
{
    publisher();
}

publisher()
{
    // This has code similar to that of BasicPublisher in aeron-samples
    // except at the end we will call the subscriber

    // Code to send some messages

    // sending done now call listner

    listner();
}

listner()
{
    // This has code similar to that of Basicsubscriber in aeron-samples

    // Some additional code to processes received messages

    // now call the publisher

    publisher();

}

For the other client  the first will be listener and then reply
main()
{
    listner();
}

publisher()
{
   // This has code similar to that of BasicPublisher in aeron-samples
    // except at the end we will call the subscriber

    // Code to send some  reply messages

    // sending done now call listner

    listner();
}

listner()
{
    // This has code similar to that of Basicsubscriber in aeron-samples

    // Some additional code to processes received messages

    // now call the publisher

    publisher();

}

But it is not working as expected only first few messages are getting transmitted.
Any ideas how to implement a two way messaging system using Aeron?

You should look at the Ping/Pong examples to make sure you are retrying appropriately from back pressure on offer/tryClaim, etc.

In terms of how to do a request/response system with Aeron, you have some options.

Use a static back channel that is setup between components in your system. Fairly typical messaging system approach where each request is labeled with a response ID so that multiple requests can be differentiated with ease.

Use a dynamic back channel where the responder sets up a Publication to send responses back on as needed. Still need a response ID as with a static channel. Only real difference is that the response channel isn't always active.

Use a dual channel approach where each channel has a response channel and the act of setting up the request must also set up the response channel.

Several users have built request/response systems into their apps. Many different ways to go about it.
Someday, though, we will get time to build a generic method hopefully.

I will see those examples  and will get back here

Now I was able to Implement the 2 way communication system, but once the messages (total size) reaches near to that of allocated log buffer size the speed of send and receive drastically slowed down (Initially I was able to send and receive  >1 Lakh messages per-sec (using busy spin idle strategy and around 1K messages in SleepingIdleStrategy (my code is in C++)) later the rate is not even touching 1 message per second  any ideas what could be causing this delay ?
I am Sending using IPC and Sending on one stream ID and receiving on another stream ID

Is this OK to close now?

Yup I think we can close this issue now and i am closing this


Better timestamps

Currently, native Blizzard timestamps and Prat timestamps work. But they could be better.
Describe the solution you'd like
A few ideas:
Timestamps in its own "column"
When messages wrap, the text doesn't go underneath its timstamp
Timestamp only visible on mouseover
Same idea as above, but timestamps slide in from the left as the player hovers over each message line
Another idea is to show timestamps on hover, but on the right side of the message
Add headers on messages between days
When scrolling through chat history, it would be useful to see the date of the messages being scrolled. This is similar to how Slack/Discord/Telegram works.

One thing I would also consider is letting Glass handle timestamps itself, too. Using Blizzard timestamps, system messages and loot messages don't get a timestamp, for example. And it seems like a bit of overkill to install Prat just to get global timestamps. :P


Resolves #390

Fix case insensitive commands forcing parameters to return lowercase in SearchResult as specified in #390

This should also solve the base problem of #268 right? The option to toggle case sensitivity already exists which should mean that it can close both issues.


Linter suggesting to use exists instead of foldLeft, but there is no exists on the collection.

Hi, I'm getting a warning
Unless there are side-effects, this col.foldLeft can be replaced by col.exists.

which in principle would be correct, except there is no exists on my collection (scalaz.NonEmptyList).

Thanks for a quick fix!

And thank you for reporting.


BEST2+ goblet cell definition

Definition and references added

Closing as agreed we don't need separate cell type for BEST2 goblet cells in the colon as it marks all goblet cells in the colon


Empty sequence adds an undesired block

When an empty sequence is created, it adds an unnecessary 1-block. Here is a MWE where I additionaly remove the undesired block:
julia> using KomaMRI

julia> seq = Sequence()
Sequence[ τ = 0.0 ms | blocks: 1 | ADC: 0 | GR: 0 | RF: 0 | DEF: 0 ]

julia> seq += Sequence([Grad(1,1)])
Sequence[ τ = 1000.0 ms | blocks: 2 | ADC: 0 | GR: 1 | RF: 0 | DEF: 0 ]

julia> seq = seq[2:end]
Sequence[ τ = 1000.0 ms | blocks: 1 | ADC: 0 | GR: 1 | RF: 0 | DEF: 0 ]

This should't have implications in the simulations, however is not the behaviour we expect.

This could fix it
Sequence() = Sequence(Matrix{Grad}(undef, 3, 0))

or defne
struct EmptySequence end
Sequence() = EmptySequence()
+(::EmptySequence, seq::Sequence) = ...
+(::EmptySequence, seq::RF) = ...
+(::EmptySequence, seq::Grad) = ...
+(::EmptySequence, seq::ADC) = ...


Not authorized, even though I should be

Problem
I have an incredibly simple policy
class WeddingPolicy < ApplicationPolicy
  # to edit a wedding, a user must be an organizer
  def edit
    record.organizers.find(user).present?
  end
...
end

Which fails:

But, when I type the following in the failure page's console, it succeeds!
>>  record = @wedding
=> #<Wedding id: 3, created_at: "2016-10-16 19:59:36", updated_at: "2016-10-16 19:59:36", name: "john-jane-6118", bride_name: "John Smith", groom_name: "Jane Doe">
>>  user = current_user
=> #<User id: 1, auth0_id: "facebook|10153745000021851", created_at: "2016-10-16 19:03:50", updated_at: "2016-10-16 19:03:50">
>>      record.organizers.find(user).present?
=> true

Additional Details
I'm not actually using devise, I made my own helper for current_user and include it before pundit in the application controller
class ApplicationController < ActionController::Base
  include Auth0Helper
  include Pundit
  protect_from_forgery with: :exception
end

The code for the helper can be viewed here if needed.

My policy should be named edit? instead of edit sorry about that.


Raw query failed. Code: 42601 when loading page journey

Describe the Bug
We recently upgraded our instance to  v2.11.3 to v2.14.0 and set up the new page journey report.
When loading the journes for the last 24 hours, the page is loading indefinetly and an error is logged.

We upgraded in small steps (eg. v2.11.3 > v2.12.0 > v2.13.0 > v2.14.0) to avoid issues during database migration.
Database
PostgreSQL
Relevant log output
PrismaClientKnownRequestError:
Invalid `prisma.$queryRawUnsafe()` invocation:
Raw query failed. Code: `42601`. Message: `ERROR: syntax error at or near "event"`
    at _n.handleRequestError (/app/node_modules/@prisma/client/runtime/library.js:121:7749)
    at _n.handleAndLogRequestError (/app/node_modules/@prisma/client/runtime/library.js:121:7057)
    at _n.request (/app/node_modules/@prisma/client/runtime/library.js:121:6741)
    at async l (/app/node_modules/@prisma/client/runtime/library.js:130:9355)
    at async l (/app/.next/server/pages/api/reports/journey.js:1:2069)
    at async K (/app/node_modules/next/dist/compiled/next-server/pages-api.runtime.prod.js:20:16881)
    at async U.render (/app/node_modules/next/dist/compiled/next-server/pages-api.runtime.prod.js:20:17520)
    at async NextNodeServer.runApi (/app/node_modules/next/dist/server/next-server.js:598:9)
    at async NextNodeServer.handleCatchallRenderRequest (/app/node_modules/next/dist/server/next-server.js:269:37)
    at async NextNodeServer.handleRequestImpl (/app/node_modules/next/dist/server/base-server.js:813:17) {
  code: 'P2010',
  clientVersion: '5.17.0',
  meta: { code: '42601', message: 'ERROR: syntax error at or near "event"' }
}

Which Umami version are you using? (if relevant)
v2.14.0
Which browser are you using? (if relevant)
No response
How are you deploying your application? (if relevant)
Docker

Sorry, didn't see #2800. Will close.


Remove file change detection logic

This change removes the file change detection logic. I was planning to split this into a sub-package, but then I realized it actually doesn't really make sense to keep this anymore.
Currently we (1) set a timed reload, (2) ghostunnel checks if the files have changed when the timer fires, and (3) we reload the keystore if a change was detected. Idea was that we avoid a reload if file hasn't changed. But that doesn't work with certificates loaded from the keychain (there's no file to watch) so the timer flag is broken in that case then.
Rather than try to come up with something clever I just deleted the file change logic and set it to reload the cert whenever the timer fires. Reloads are cheap anyway so this optimization didn't make much sense to begin with. Anyone who is really concerned about extra reloads can just turn off the timer and instrument them manually via SIGUSR1.

Coverage decreased (-0.4%) to 92.439% when pulling ae5cc8d60f4c1c104e0c7a3700cea0549542976f on cs/simplify-reload into 3d7b6349e95103d932e56c8fa4a3689b46ff5f19 on master.


IVS-131 - Issue #279 - IFC102 improvements & cleanup validation_handling

Addresses https://github.com/buildingSMART/ifc-gherkin-rules/issues/279

Remove IfcPresentationStyleSelect -> is deleted in IFC4X3
Implement the following.

Alternatively we make the list checked as exact types, instead of type + subtypes. Most of these items are leaf entities anyway. This is probably the safest way forward.

We should probably also do this for attributes of entities (?). For this, there's a new small decorator to register enumeration types. Using this is quite intuitive; simply decorate the Enum-inherited class with @register_enum_type),. There is probably no need to use TypeBuilder (correct me if I'm wrong)
Furthermore, a testfile is added to ensure IfcTextLiteralWithExtent is passing.

Small cleanup for validation_handling:

Remove unused functions
Functions are in a more logical order now
Add some documentation

Another nuance I was not aware of .. Thanks :)
I'm assuming the same is true for IfcStyleAssignmentSelect, they're both deleted from the deprecated list.
We're now checking for exact types for deprecated entities. Just to be sure, I think this doesn't apply for the deprecated attributes of entities?
To give an example, for IfcRelAssigns, the RelatedObjectsType attribute is deprecated. This will then also be the case for its subtypes; IfcRelAssignsToActor, RelAssignsToGroup, and so forth. The same applies to IfcReinforcingElement -> IfcReinforcingBar, Mesh, Anchor, Conduit and IfcTendon,

To give an example, for IfcRelAssigns, the RelatedObjectsType attribute is deprecated. This will then also be the case for its subtypes; IfcRelAssignsToActor, RelAssignsToGroup, and so forth. The same applies to IfcReinforcingElement -> IfcReinforcingBar, Mesh, Anchor, Conduit and IfcTendon,

Correct!


RFC-WIP : Notebook for ConditionalDimension

Work in progress
Let's discuss.

btw it's ConditionalDimension not Conditional Dimension

btw it's ConditionalDimension not Conditional Dimension

btw it's ConditionalDimension not Conditional Dimension

Correct-fixed.

Starting to look through this now. Will post some comments as I go.
1st one: Name of the notebook: conditions is not descriptive enough. Why not change to conditional_dimensions.ipynb?

@FabioLuporini @mloubout @georgebisbas
Looking at the following:
condition = And(f[i,j] > 0, f[i,j] < 5 , evaluate=False)
ci = ConditionalDimension(name='ci', parent=j, condition=condition)

Why are we even calling this a 'conditional dimension' when we're putting if conditions on an equation? Do we need to have a think about some of this, noting that very similar conditions are needed for the implicit solver PR?

Looking good but needs some tidying + some additions.
Notebook currently finishes quite abruptly. What else can you do with 'conditional dimensions'? What are can't you do etc? Add a practical example that will be of interest to many users?

Starting to look good I think. Just needs to some up and tidying, and I'd also like to see the Use case B example to be expanded and hence made more informative.

Will do one more pass later. But one additional thing that is maybe is some asserts to ensure everything is working.

Some final comments:

Could you please give it one more thorough read through. There are still a reasonable of amount of grammatical errors present and some language could be further tidied. (I know I'm banning on about this sorry, but believe me it's good practice to be thorough with such things now since it'll pay off when it comes to writing up).
Could you add a couple of appropriate asserts to ensure nothing goes funny with the nb down the line.
Again, with regards to use base B - Most people are going to come across this functionality from wanting to subsample TimeFunctions for plotting/analysis purposes. Why not include a proper example of this? i.e. subsample and plot some iso acoustic wave-fields?

Some final comments:
1. Could you please give it one more thorough read through. There are still a reasonable of amount of grammatical errors present and some language could be further tidied. (I know I'm banging on about this sorry, but believe me it's good practice to be thorough with such things now since it'll pay off when it comes to writing up).

2. Could you add a couple of appropriate asserts to ensure nothing goes funny with the nb down the line.

3. Again, with regards to use base B - Most people are going to come across this functionality from wanting to subsample `TimeFunction`s for plotting/analysis purposes. Why not include a proper example of this? i.e. subsample and plot some iso acoustic wave-fields?

@rhodrin

I went through, I hope some of my fixes are fine.
You mean more than now?
I think the better solution is to redirect users to snapshotting tutorial. Let me know what do you think.

1. I went through, I hope some of my fixes are fine.

2. You mean more than now?

3. I think the better solution is to redirect users to snapshotting tutorial. Let me know what do you think.

Yeah, looking better thanks. I think Fabio has now left a few more comments also.
Number of asserts looks good.
Ah, I forgot about that tutorial sorry - yeah, a link to that is good. Nitpicking now, but what about before use case A creating the Examples heading - explain what examples you're going to demonstrate and introduce and put the link to the subsampling notebook there. And change Use Case A etc. to a subheading called Example A instead?

Merged, thanks


[Concurrency] await try -> try await

The try await ordering is both easier to read and indicates the order
of operations better, because the suspension point occurs first and
then one can observe a thrown error.

@swift-ci please test

@swift-ci please smoke test

@swift-ci please smoke test

@swift-ci please smoke test Linux

@swift-ci please smoke test Linux


Natural Selection 2 wont respond

Tried to get information about ns2 server (<IP_ADDRESS>:27015), but get nothing, always says "Server is offline"
What i did:

type: 'ns2', host: '<IP_ADDRESS>', with default games.txt
type: 'ns2', host: '<IP_ADDRESS>:27015', with default games.txt
type: 'ns2', host: '<IP_ADDRESS>', with port=27015 in games.txt

help please

Please feel free to re-open if this is still an issue 😄


Numba fails to understand float('inf')

Numba does not understand the code float('inf')
Running the following code
from numba import njit

@njit
def my_inf():
    return float('inf')

x = my_inf()

returns
TypingError: Invalid use of Function(<class 'float'>) with argument(s) of type(s): (Literal[str](inf))
 * parameterized
In definition 0:
    TypeError: float() only support for numbers
    raised from C:\Users\Luca.amerio\AppData\Local\Continuum\anaconda3\lib\site-packages\numba\typing\builtins.py:880
In definition 1:
    TypeError: float() only support for numbers
    raised from C:\Users\Luca.amerio\AppData\Local\Continuum\anaconda3\lib\site-packages\numba\typing\builtins.py:880
This error is usually caused by passing an argument of a type that is unsupported by the named function.
[1] During: resolving callee type: Function(<class 'float'>)
[2] During: typing of call at C:/Users/Luca.amerio/Documents/GitLab/Surface pressure mapping and modal integration/numba_bug.py (13)

The "problem" is that float('inf') is the only native python way to obtain infinity.
At the moment, the only workaround is to use the numpy.inf. The following code is indeed working
from numba import njit
from numpy import inf

@njit
def my_inf():
    return inf

x = my_inf()

This is a feature request for float(str) support.

@sklam The CPython function that handles this is PyFloat_FromString(PyObject *v)
My understanding is that this could be added to pythonapi.py just after this:
https://github.com/numba/numba/blob/82c7d37ad4900b01072c233da5ea97e95d5444db/numba/core/pythonapi.py#L443-L446
The logic of the Float class needs to change here:
https://github.com/numba/numba/blob/82c7d37ad4900b01072c233da5ea97e95d5444db/numba/core/typing/builtins.py#L888-L905
and a new implementation added here:
https://github.com/numba/numba/blob/82c7d37ad4900b01072c233da5ea97e95d5444db/numba/cpython/builtins.py#L245-L251
Is that right?
I'm happy to work on this but will need some guidance as I'm just getting familiar with the code base.

cmath.inf appears to function as a workaround.

@jagaudin, float('inf') should not rely on the Python interpreter function. We tried to avoid requiring the interpreter thus also the GIL as much as possible. The preferred approach is to implement  PyFloat_FromString logic in numba-compilable python code.

@sklam Thanks for the tip, I think I've spotted the relevant part of the documentation, just need to dive into it when I get a chance. Will post if I have questions.


Download Pipeline Artifact error

When using Download Pipeline Artifact within a release pipeline, I get the following error. This works fine when the artifact is published with "Publish pipeline artifact (0.140.0)", but this error is coming from a pipeline using "Publish build artifacts (1.158.1)". Doesn't this conflict with the fact the description below is "Download build and pipeline artifacts"?
2019-11-22T20:54:17.3018438Z ##[section]Starting: Download artifact - _configuration - configuration
2019-11-22T20:54:17.3141147Z ==============================================================================
2019-11-22T20:54:17.3141222Z Task         : Download Pipeline Artifacts
2019-11-22T20:54:17.3141254Z Description  : Download build and pipeline artifacts
2019-11-22T20:54:17.3141447Z Version      : 2.1.1
2019-11-22T20:54:17.3141478Z Author       : Microsoft Corporation
2019-11-22T20:54:17.3141524Z Help         : https://docs.microsoft.com/azure/devops/pipelines/tasks/utility/download-pipeline-artifact
2019-11-22T20:54:17.3141554Z ==============================================================================
2019-11-22T20:54:19.2887314Z Download from the specified build: #29383
2019-11-22T20:54:19.2898274Z Download artifact to: C:\vsts-agent-1_work\r31\a/_configuration/Configuration
2019-11-22T20:54:19.4756506Z ##[error]The container 1513296 could not be found.
2019-11-22T20:54:19.5045301Z ##[section]Finishing: Download artifact - _configuration - configuration

@richardgavel Thank you for the feedback, assigning this to the author for review.
Please see if https://github.com/microsoft/azure-pipelines-yaml/issues/215 addresses the issue, and that you're using the latest version with this fix.


Misc Cleanups

Motivation
Explain here the context, and why you're making that change.
What is the problem you're trying to solve.
Modifications
Describe the modifications you've done.
Result
After your change, what will change.

run integration tests


[[TEST]] Run Test262 tests

Include the Test262 project as a submodule and introduce a new test
script that attempts to parse every test file it contains. Because this
project currently fails a number of valid tests with both type I and
type II errors, include a text file temporarily white-listing those
tests.
Managing the Test262 project repository as a git submodule is somewhat
cumbersome for general development, so do not integrate this test in the
default npm test command.
Hey @caitp, would you mind taking a look? I'm really excited about this--it
should give us a much clearer understanding of what needs to be done
(especially as more features reach stage 3).

Coverage remained the same at 97.754% when pulling 61438e82c96828067bf673509f88ec3e016be267 on jugglinmike:test262 into 8cfe5adc9827114901db07cb983591737b70ce9a on jshint:master.

Thanks for the review, @caitp! If my latest changes look good to you, then I can
squash if you wouldn't mind merging for me. Or maybe you can do both from the
GitHub.com UI? I'm not sure.

Coverage remained the same at 97.754% when pulling 2d99b624460dd09a00ce760e15f18c7cf41364f8 on jugglinmike:test262 into 8cfe5adc9827114901db07cb983591737b70ce9a on jshint:master.

looks ok to me

Alrighty. I've squashed and force-pushed (original commits available here)--would you do the honors?

Coverage remained the same at 97.754% when pulling 9b0843a77437e1bf0d34983067754d238859d62d on jugglinmike:test262 into 8cfe5adc9827114901db07cb983591737b70ce9a on jshint:master.

@caitp Mind pushing that button for me?


Update vscode-setup.rst (IDFGH-6810)

Improved formatting and grammar, fixed links so that they render correctly

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

Hi @andypiper, thanks for the valuable contribution!
I see some issues on building this change using our docs-build. Please see my suggestions.
Thanks!

Thanks for the review and apologies for those errors, I like the suggestions as better ways to get uniform formatting in the list here. I've rechecked the links this time.

LGTM! Thanks @andypiper

I have taken this PR into the internal queue for merging.
Thank you @andypiper!

Merged in https://github.com/espressif/esp-idf/commit/6249b3d6718372c12ae96dd288181cd058abeca4


"enabling proxy protocol failed"

I got this when running provision-aws-cluster.sh:
Enabling proxy protocol
Error: 501: All the given peers are not reachable (failed to propose on members [http://<IP_ADDRESS>:4001] twice [last error: Unexpected HTTP status code]) [0]
# WARNING: Enabling proxy protocol failed.
# Ensure that the private key in cloudformation.json is added to
# your ssh-agent, then enable proxy protocol before continuing:
#
# deisctl config router set proxyProtocol=1

Your Deis cluster was deployed to AWS CloudFormation as stack deis0.

Now run this command in your shell:
export DEISCTL_TUNNEL=XX.XX.XX.XX
and continue to follow the documentation for "Installing the Deis Platform."

Running it manually does:
 deis/contrib/aws — master • →  deisctl config router set proxyProtocol=1
Error: 501: All the given peers are not reachable (failed to propose on members [http://<IP_ADDRESS>:4001] twice [last error: Put http://<IP_ADDRESS>:4001/v2/keys/deis/router/proxyProtocol: dial tcp <IP_ADDRESS>:4001: getsockopt: connection refused]) [0]

oh man, i think i screwed up by having a different deisctl version than what I was provisioning with.

The error message is telling you the likely cause-- that the private key for ssh'ing to nodes (required for tunneling) is not in your ssh agent.
I'll assume you have the correct .pem laying around somewhere.  If so:
$ ssh-add path/to/your.pem

I already did, and it shows up in my ssh-add -l. Lemme try again and paste any more debug info when I have them.

If the key is already in your agent, you can attempt troubleshooting the connection like so:
$ ssh -vvv<EMAIL_ADDRESS>

Alright. At the surface it seems like the problem might be the inability to connect to http://<IP_ADDRESS>:4001, but I'll let you know in a few hours :-)

@rstacruz since deisctl works through an SSH tunnel, that would be fleet attempting to connect to etcd. You might want to SSH into your cluster and troubleshoot why etcd is down. :)

Neat. Will there be a more polished provision-aws-cluster.sh tool in the
works?
On Thursday, January 7, 2016, Matthew Fisher<EMAIL_ADDRESS>wrote:

@rstacruz https://github.com/rstacruz since deisctl works through an
SSH tunnel, that would be fleet attempting to connect to etcd. You might
want to SSH into your cluster and troubleshoot why etcd is down. :)
—
Reply to this email directly or view it on GitHub
https://github.com/deis/deis/issues/4835#issuecomment-169564470.

Will there be a more polished provision-aws-cluster.sh tool in the works?

First we need to determine how etcd failed to bootstrap, then we can figure out how to fix this problem (or if it's a duplicate of previous troubleshooting problems). Can you do a ssh<EMAIL_ADDRESS>then a sudo journalctl -u --no-pager etcd, where xx.xx.xx.xx is the node used you censored as the target for DEISCTL_TUNNEL in the logs?

Ah, sorry, the question was a bit of a lateral thought—I meant, would there
be a separate AWS provisioning/maintenance tool released in the future that
doesn't require fetching the entire source tree to work?
Because that sounds pretty cool and Id love to help out.
On Thursday, January 7, 2016, Matthew Fisher<EMAIL_ADDRESS>wrote:

Will there be a more polished provision-aws-cluster.sh tool in the works?
First we need to determine how etcd failed to bootstrap, then we can
figure out how to fix this problem (or if it's a duplicate of previous
troubleshooting problems). Can you do a ssh<EMAIL_ADDRESS>then a sudo
journalctl -u --no-pager etcd, where xx.xx.xx.xx is the node used you
censored as the target for DEISCTL_TUNNEL in the logs?
—
Reply to this email directly or view it on GitHub
https://github.com/deis/deis/issues/4835#issuecomment-169565252.

We have one tool for that called rigger: https://github.com/deis/rigger
In v2 we have a tool called helm to provision Deis.

Okay, I got this:
Waiting for instances to pass initial health checks (460s) ...
Waiting for instances to pass initial health checks (450s) ...
Instances are available:
i-7fdeexxx      52.62.XX.XX     m3.medium       ap-southeast-2b running
i-78deeyyy      52.62.YY.YY    m3.medium       ap-southeast-2b running
i-9f155zzz      52.62.ZZ.ZZ    m3.medium       ap-southeast-2a running

and after ssh -vvv<EMAIL_ADDRESS>Welcome to Deis                 Powered by CoreOS
core@ip-10-21-2-187 ~ $

doing ps aux | grep etcd says its running. Then:
$ sudo journalctl -u --no-pager etcd
Failed to add match 'etcd': Invalid argument

but journalctl has these:
Jan 07 07:07:08 ip-10-21-2-187.ap-southeast-2.compute.internal sdnotify-proxy[1373]: E0107 07:07:08.117594 00001 network.go:53] Failed to retrieve network config: client: etcd cluster is unavai
Jan 07 07:07:08 ip-10-21-2-187.ap-southeast-2.compute.internal etcd2[797]: proxy: zero endpoints currently available
Jan 07 07:07:09 ip-10-21-2-187.ap-southeast-2.compute.internal etcd2[797]: proxy: zero endpoints currently available
Jan 07 07:07:09 ip-10-21-2-187.ap-southeast-2.compute.internal etcd2[797]: proxy: zero endpoints currently available

etcdctl output:
$ etcdctl get
Error:  client: etcd cluster is unavailable or misconfigured
error #0: client: etcd member http://<IP_ADDRESS>:2379 has no leader
error #1: client: etcd member http://<IP_ADDRESS>:4001 has no leader

From another node:

Jan 07 07:11:40 ip-10-21-XX-XX.ap-southeast-2.compute.internal sdnotify-proxy[1473]: E0107 07:11:40.361605 00001 network.go:53] Failed to retrieve network config: client: etcd cluster is unavailable or misconfigured
Jan 07 07:11:40 ip-10-21-XX-XX.ap-southeast-2.compute.internal etcd2[800]: could not get cluster response from http://10.21.XX.XX:2380: Get http://10.21.XX.XX:2380/members: dial tcp 10.21.XX.XX:2380: i/o timeout

I'm trying again now with 3 x m3.large and with a new discovery URL and seeing if that changes anything.

Okay, it turns out fine now! I guess m3.medium was a bad idea?

m3.medium is fine.  Since you have just explicitly mentioned creating a new discovery URL... were previous attempts all made without creating a new discovery URL each time?  That would definitely stop etcd from coming up clean.  Without etcd, fleet also fails, and thus so does deisctl.

ah, right. no, i didn't make a new discovery url. perhaps I should've ;)


Documentation for system module

Module overview docs

A lot of full-stops (.s) missing from element-level documentation.


Restored from backup and now dashboard does not update with existing repos activities.

Gitea version (or commit ref): 1.15.0+dev-65-g0bb8bd819
Operating system: gitea/gitea docker
Database: PostgreSQL
Can you reproduce the bug at https://try.gitea.io:

[ ] Yes (provide example URL)
[x] No

Log gist: gitea.log

Description
Gitea runs as a docker on unraid. For backup purposes I run tar the repos and data directories every night. About 1.5 months ago I had dual hard drive failure and had to restore data from backups. Since restoring, gitea's dashboard no longer shows activities for repos that existed before the restore point.
If I add a new repo now and do commits, it all shows up in the dashboard. However preexisting stuff does not. I can still push and pull all repos without an issue. I've logged into that docker console to also recursively restore permissions on all repo folders to git:git.
How can I get my dashboard back?
The only difference I can currently see is the ownership on the repo folders/files were not correct. To figure out what the permissions should've been I created a new repo and saw that everything was written as git:git so I can chown -R git:git  on the repos in hopes it would bring back the dashboard.

IIRC this bug could have been fixed, please try latest release.
It's caused by out-synced git hooks.


Adding an if clause for possible conflict error for r.client .

My first PR :)

/ok-to-test

Thanks @razo7
/approve
This makes me think that maybe using patch would be a better approach. Not sure.
anyway, not in this PR.
could you please just add a comment on the first conflict check that explains why we ignore it?
e.g.

conflicts are expected since all poison pill deamonset pods are competing on the same requests

/lgtm

/retest


Fix: #4555 display details attribute when folding / unfolding categories

Fixed: #4555 https://github.com/openfoodfacts/openfoodfacts-androidapp/issues/4555

thank you very much @krystelnguyen 👌


ContainerApps - Update error message and help to indicate need to use --registry-server argument

If the issue is to do with Azure CLI 2.0 in-particular, create an issue here at Azure/azure-cli

Related command

containerapp create
Extension name (the extension in question)
containerapp
Description of issue (in as much detail as possible)
I’m trying to deploy Metabase to ACA.
I pulled the docker image to my machine, tagged it and then pushed it to my ACR.
Unauthorized
I ran into was getting authentication error when running containerapp create.
az containerapp create -n metabase -g $RESOURCE_GROUP --environment $CONTAINERAPPS_ENVIRONMENT --image metabaseacr.azurecr.io/metabase:latest --target-port 3000 --ingress 'external'

The following field(s) are either invalid or missing. Invalid value: "metabaseacr.azurecr.io/metabase:latest": GET https:?scope=repository%3Ametabase%3Apull&service=metabaseacr.azurecr.io: UNAUTHORIZED: authentication required, visit https://aka.ms/acr/authorization for more information.: template.containers.metabase.image.

I confirmed that the ACR is admin-enabled:

After talking to one of the ACA PMs, I learned that I needed to specify the --registry-server argument.
Both the help and the error message should be updated to make this clearer to the user.

route to CXP team

@anthonychu FYI.
@haroonf I believe this is an error from the API that is surfaced on the client, can you confirm. If not lets check if there is some change on the client side that can be done. Thanks!

Adding Service team to look into this.


Missing something?

why is there a . in the first line?

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

ID: e0c31fb5-6109-2669-a774-9f8872dcb85a
Version Independent ID: 0b44b31b-b4ab-7334-c4a1-3fa61af31633
Content: Get-SPOMultiGeoExperience (sharepoint-online)
Content Source: sharepoint/sharepoint-ps/sharepoint-online/Get-SPOMultiGeoExperience.md
Service: sharepoint-powershell
GitHub Login: @
Microsoft Alias: ****

@officedocsbot assign @yogkumgit

Thanks for noticing.