CodeClimate badges returning "Invalid response data" for one repo

Are you experiencing an issue with...
shields.io
🐞 Description
I help run a software development course in which we provide a dashboard for the code quality of all teams.
One, and only one, team currently has badges that return "Invalid response data" despite CodeClimate being set up properly.
This was also happening to another team on Sunday, but starting Monday only one team has this issue.
FWIW, we set up those badges before some of the teams had fully set up their repos, so perhaps there's some caching issue, but I would not expect errors to be cached >24h.
🔗 Link to the badge

For comparison, working ones:

💡 Possible Solution
No response

I haven't had a chance to properly look at this, but if anyone else comes to it, this is failing with ValidationError: "data" must contain less than or equal to 1 items when we call https://api.codeclimate.com/v1/repos?github_slug=swift-wallet%2Fsdp2022

The API response for the Swift Wallet team contains two elements in the data array. We're being a bit restrictive and only expecting one:
https://github.com/badges/shields/blob/53cb1da830900697e61b58f6780f9e632504a397/services/codeclimate/codeclimate-common.js#L10
In this case, one of the two elements contains the id we're looking for, the other doesn't. We'll need to change our implementation to loop through all elements, and find one that does contain latest_default_branch_snapshot or latest_default_branch_test_report data, respectively for the analysis and coverage badges.
I'll aim at picking this up during the weekend if no one does in the meantime.


Improve PHPUnit assertions

Description

Letting assertion equals be strict.

Motivation and context

For improvements.

How has this been tested?
N/A
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[X] Improvements

PR checklist

[X] My change requires a change to the documentation.
[X] I have updated the documentation accordingly.
[X] I have read the CONTRIBUTING.md document.
[X] I have added tests to cover my changes.

Thank you for contributing! 🎉
I've just committed this in a0eff507723fa49e1f725f2d2ff8a75c8b1e07bf. Better late than never. ;-)


Get-EntraUser | Add-EntraGroupMember doesn't work

Get-EntraUser -UserId <GUID>| Add-EntraGroupMember -GroupId <GUID> fails with error New-MgGroupMember_CreateExpanded: Invalid object identifier 'Microsoft.Graph.PowerShell.Models.MicrosoftGraphUser'.
Instead of the whole object you need to pipe just user's ID to make pipeline work.

Thanks, @alexandair, for bringing this up!
We have prioritized breaking the module into modular components. This will be fixed as part of a larger work item for pipelining patterns described in this issue - https://github.com/microsoftgraph/entra-powershell/issues/746 (you are familiar with this item 💯)


Update API info for labels, Conditional AdUnits

Related to issue https://github.com/prebid/prebid.github.io/issues/491 and PR https://github.com/prebid/prebid.github.io/pull/481

Just in case we need the reference, this work was previously happening in the now-closed https://github.com/prebid/prebid.github.io/pull/481

Thanks for this PR @bretg !
I found a few things that I think we need to address.
High level comments for each file are below, followed by some patches attached with suggested/possible changes that you can snarf if you want.
conditional-ad-units.md

The anchor links to the page sections don't work - I tried rewriting them to use a few different techniques, but couldn't get them to work.  I was however able to just slap in a TOC (see patch).
The Label Targeting section that you added to the API reference was still in this file, too.  I think we should just maintain the one copy in the API reference (see also patch)

conditional-ad-units.patch.txt
publisher-api-reference.md

Just one thing: a broken link back to the Conditional Ad Units page because Jekyll is expecting the .html suffix, and barfs on .md.

publisher-api-reference.patch.txt

Thanks @rmloveland - patches applied.

thanks @rmloveland


Movie Info Page

Issue for movie information page - will display movie info based on user search

[ ] Pull data from streaming API
[ ] Utilize localstorage
[ ] Create search bar
[ ] Create form (?)
[ ] Display movie info (include title, main cast, length, year of release, movie poster, director)
[ ] Say "Movie not found" if search doesn't work


Camera controller resets its IP address and brings down the church's internet

This has happened twice so far. The default value of the camera controller IP address (IIRC <IP_ADDRESS>) conflicts with the IP address of the internet gateway (in the store room). When the camera controller "wins" the church is thereby disconnected from the internet.
A workaround is to reset the IP address of the camera controller to its preferred value (see below) of <IP_ADDRESS> as per the instructions in the basic manual (see the hardware page of these tech notes).
I contacted Panasonic to ask if this is a known issue and what a fix might involve. They responded advising us to upgrade the camera controller firmware to the latest version, 5.02, which I did today. Let's leave this issue open and log any further occurrences of the problem here.
The preferred value of the camera controller IP address was chosen by SFL who left the following note of IP addresses:
<IP_ADDRESS> - datavideo encoder
<IP_ADDRESS> - Camera 1
<IP_ADDRESS> - Camera 2
<IP_ADDRESS> - Camera 3
<IP_ADDRESS> - Camera 4
<IP_ADDRESS> - Panasonic RP-50 Camera Controller
<IP_ADDRESS> - Blackmagic ATEM

I checked with Panasonic and they said that this issue is not a problem they are familiar with.

I sent the following sales inquiry to CVP.com (recommended by Tim Matthews)

Hi. I lead a team which does live streaming at a large church in Winchester. We currently have a Panasonic AW-RP50 remote camera controller which is starting to become unreliable. It controls 4 PTZ cameras including 3 Panasonic AW-HE40SWEJ units.
What replacement camera controller(s) would you recommend? Compatibility with the cameras is essential. A similar operator interface to the AW-RP50 is very desirable to make it easy to introduce the unit to the live streaming team. I think we should also allow for future expansion with more cameras.

CVP.com came back to me with the following helpful information:

It looks like the direct replacement to your model would be the AW-RP60GJ, while the AW-RP150 seems to be the big brother offering compatibility with 4k50p so if you wanted to upgrade the cameras in the future for 4k acquisition, this would be the best choice. Though I don't believe it will handle more than 5 cameras.
you can find the full specs here https://business.panasonic.co.uk/professional-camera/Touchscreen-Remote-Camera-Controller/aw-rp150
Though from your current set up either of these would be the easiest and most streamline options to upgrade as the software and UI should be very similar, if not identical, to what you and your team are already used to
Hope this all makes sense!

So, at the time of writing, the AW-RP60GJ looks like a suitable replacement unit, if the current problem recurs. CVP.com's list price is  £1,875.80 (£2,250.96 inc VAT).

After upgrading the firmware, the problem has not reproduced for several weeks.


Lesson Titles along with lesson number for Lessons Panel

In course-v3/fastai-video-browser/src/components/LessonsPanel.jsx
Do you think it would be good idea to also give lesson name like it has been done for Notes Panel,
'Lesson 1 : Image Classification'
instead of
'Lesson 1'

I'm unable to setup the project. Any link/instructions?


Archive Papertrail logs in S3

See: http://help.papertrailapp.com/kb/how-it-works/permanent-log-archives/#create-and-share-an-s3-bucket


kubectx: 0.9.1 -> 0.9.2

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/ahmetb/kubectx/releases.
meta.description for kubectx is: "Fast way to switch between clusters and namespaces in kubectl!"
meta.homepage for kubectx is: "https://github.com/ahmetb/kubectx"
meta.changelog for kubectx is: ""
Updates performed

Golang update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS

The tests defined in passthru.tests, if any, passed

0 of 0 passed binary check by having a zero exit code.

0 of 0 passed binary check by having the new version present in output.

found 0.9.2 with grep in /nix/store/d7n2lwz4l0ahy8bj1ldhl2d9cimxph9z-kubectx-0.9.2

directory tree listing: https://gist.github.com/b0aa3de229b6fa306343d541fee3808d

du listing: https://gist.github.com/5d6ed539da248442704393950a2a8681

Rebuild report (if merged into master) (click to expand)

4 total rebuild path(s)

1 package rebuild(s)

1 x86_64-linux rebuild(s)
1 i686-linux rebuild(s)
1 x86_64-darwin rebuild(s)
1 aarch64-linux rebuild(s)

First fifty rebuilds by attrpath
kubectx

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/d7n2lwz4l0ahy8bj1ldhl2d9cimxph9z-kubectx-0.9.2 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A kubectx https://github.com/r-ryantm/nixpkgs/archive/0ae7c2a141a77ad4384d9e6225b2e40200ff7ebd.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/d7n2lwz4l0ahy8bj1ldhl2d9cimxph9z-kubectx-0.9.2
ls -la /nix/store/d7n2lwz4l0ahy8bj1ldhl2d9cimxph9z-kubectx-0.9.2/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

kubectx

Maintainer pings
cc @jlesquembre @kalbasit for testing.

Result of nixpkgs-review pr 114345 run on x86_64-linux 1

1 package built:

kubectx


vue 2.1.10 npm run build err

webpack
ERROR in ./~/vue-cropperjs/VueCropper.js
Module not found: Error: Cannot resolve module 'lodash/omit' in /Users/busyhe/workspace/web/company_project/zhinengpingtai/node_modules/vue-cropperjs
 @ ./~/vue-cropperjs/VueCropper.js 4:11-33

This library doesn't use lodash anymore, which version are you using ?

thanks, I updated. but IE is error,
SCRIPT1002: 语法错误

I don't know what's  wrong

Can you describe it in english? Also try to delete node_modules and reinstall the package with latest version.


import docs do not explain setting api version

this feedback was submitted by way of docs feedback
this person mentioned that this docs page does not talk about api version, and how to use them
https://www.pulumi.com/docs/guides/adopting/import/

Closing as it's not clear what's needed here.


Versioned helpers (pkg/api/v1/helpers) are missing

pkg/api/v1/helpers.go was removed but pkg/api/v1/helpers/helpers.go was not added in its place. I noticed in staging/copy.sh the line mkcp "pkg/api/helper" "pkg/api" was explicitly added: https://github.com/kubernetes/kubernetes/pull/44361/files#diff-79c7edfbbe6b22642e05ac8c9cd431e7. Why did it need to be added and do we need to add  mkcp "pkg/api/v1/helper" "pkg/api/v1"too?
https://github.com/kubernetes/kubernetes/pull/44361
https://github.com/kubernetes/client-go/commit/bb2e2ad83cfeab769ec660af19256ba6b0f2c6ef

It's intentional. The entire pkg/api and pkg/apis will be gone after https://github.com/kubernetes/kubernetes/pull/44784. After that, we will selectively introduce back the helpers, but in a more appropriate directory.

Closing, since it looks like this issue is resolved.


[connection].session.sql file opens very eagerly

Not sure this is a bug as such - more a UX thing
I normally have SQL files open already that I want to run when I start using SQLtools.
Then I want to check for a table name or something so I open the sidebar explorer and the [connection].session.sql file opens over the top of them.
Repro

set up a connection
open the sidebar
click on the connection to explore the db

Snowflake driver
Expected

No [connection].session.sql  file opens

Actual

[connection].session.sql  file opens and I have to close it before running my SQL

Proposals

Setting to configure this file opening or not
?Turn this off by default

Does https://github.com/mtxr/vscode-sqltools/pull/1161 (just merged and not yet published) help you?

I don't think so, I still want to be able to see the query for any queries I run, I just don't want to be presented with a blank file when I connect to the db

Please try with "sqltools.autoOpenSessionFiles": false

Ah that's the setting I need, thanks.
I'd propose that this be set to false by default, but if other people find it helpful I understand!

Changing defaults can be unpopular, so I'm going to close this. Doing that won't prevent further feedback though.


Fixes Dockerfile to support new TeamTalk5 SDK

Base image was changed to slim-bookworm because the new TeamTalk5 SDK is known to not work on Debian 11 bullseye.
Also libmpv1 dependency was replaced with libmpv2 because the former is not available in bookworm anymore.

The Docker image with these changes is available by tag cyrmax/ttmediabot:dev on Docker Hub.


Weapon builds being duplicated

General information
Project issue is for:
Server
Expected result:
Updating a weapon build should create a new one
Actual result:
In some cases, updating a weapon build appears to duplicate it in the profile
How to reproduce
Currently unknown, reported by multiple users in Discord
Log files
Attached
Profile
Attached

Attachments

server-2024-04-03(2).log (507 KiB)
LogOutput(123).log (9 KiB)
2024.04.03_20-44-35_<IP_ADDRESS>.29197_traces.log (48 KiB)
660bc56d0003708ac67a0226.zip (52 KiB)

Originally written by DrakiaXYZ

I was not able to reproduce this.
Steps I tested:
Saving a preset, changing it, overwriting it then manually checking the profile after closing client + server.
Saving a preset, changing it, overwriting it, and then running a raid as PMC (exfil failed)
Saving a preset, changing it, overwriting it, and then running a raid as scav (exfil success)
I'll try more another time!

Originally written by acidphantasm

It appears to also be happening to equipment presets - just got an example today from Discord

Attachments

660b484300057ed9c9971d6f.zip (108 KiB)

Originally written by acidphantasm

We've tracked this issue down to Profile editor adjusting capitalization of properties:

Attachments

image.png (34 KiB)

Originally written by chomp


Include backport branch in workflow to trigger CI

Signed-off-by: Heemin Kim<EMAIL_ADDRESS>Description
Add backport branch in workflow to trigger CI
Issues Resolved
Backport branch is not added in workflow to tigger CI which does not run any validation check before merge
Check List

[X] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Codecov Report

Merging #147 (80d2f10) into main (89b676a) will decrease coverage by 0.28%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##               main     #147      +/-   ##
============================================
- Coverage     85.13%   84.85%   -0.29%
+ Complexity      320      319       -1
============================================
  Files            44       44
  Lines          1063     1063
  Branches         86       86
============================================
- Hits            905      902       -3
- Misses          124      126       +2
- Partials         34       35       +1

Impacted Files
Coverage Δ

...earch/geospatial/index/mapper/xypoint/XYPoint.java
68.96% <0.00%> (-5.18%)
:arrow_down:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

I'm curious how does this work in k-NN repo? We don't have "backport" branches in CI github actions, but checks are still running, like in this PR - https://github.com/opensearch-project/k-NN/pull/547.

I'm curious how does this work in k-NN repo? We don't have "backport" branches in CI github actions, but checks are still running, like in this PR - opensearch-project/k-NN#547.

What I wanted was having minimum number of reviews before merging. Seems like it need to be configured in branch level in github itself by maintainer. https://github.blog/2018-03-23-require-multiple-reviewers/ Let me close this PR.


Leaderboard Improvements

Subject of the issue
Connected wallets are currently pinned to the top of the leaderboard, leaving a gap in the ranking where your position would be.
Deliverables

[x] Pin current user to top AND show in ranking
[x] Hide leaderboard search bar on mobile

We decided to show the search bar on mobile, but display it below the selection buttons


[octavia-ingress-controller] Include CA certs into Barbican secret

Is this a BUG REPORT or FEATURE REQUEST?:

Uncomment only one, leave it on its own line:
/kind bug
/kind feature

What happened:
When you enable TLS encryption on an ingress, the secret created in Barbican does not contain the CA certificates, which can produce SSL problems like this:
$ curl https://mydomain.com

curl: (60) SSL certificate problem: unable to get local issuer certificate

What you expected to happen:
The Barbican secret created by the controller should contains all the certificates found in the Kubernetes secret.
How to reproduce it:
Create a Let's Encrypt certificate (using acme.sh) and use the full chain certificate to create the Kubernetes secret.
kubectl create secret tls mydomain.com --key=mydomain.com.key --cert=fullchain.cer

Then use this secret to create an Ingress with TLS encryption.
$ cat <<EOF | kubectl apply -f -
---
apiVersion: networking.k8s.io/v1
kind: Ingress
metadata:
  name: test-octavia-ingress
  annotations:
    kubernetes.io/ingress.class: "openstack"
    octavia.ingress.kubernetes.io/internal: "false"
spec:
  tls:
    - secretName: mydomain.com
  rules:
    - host: mydomain.com
      http:
        paths:
        - path: /
          pathType: Exact
          backend:
            service:
              name: webserver
              port:
                number: 8080
EOF

And then,
curl https://mydomain.com

Anything else we need to know?:
We can verify that the Ingress is only returning the client certificate with the following command.
openssl s_client -connect mydomain.com:443 -showcerts

Environment:

openstack-cloud-controller-manager(or other related binary) version:
OpenStack version:
Others:

Hi @hedinasr, thanks for reporting this issue, could you please provide more detailed steps to help us understand how to reproduce, e.g. How you generated the certificates? How you created the Barbican secretes, etc.

I updated the issue, let me know if you want more detail.

Thanks for the update.


"Dictionary changed size during iteration" in client.PubSub.on_connect()

Hi!
I'm using:
celery==4.1.0
redis==2.10.6
On a fairly busy backend, and I'm seeing occasional 'dictionary changed size during iteration' (full trace below) errors during times of peak load. My guess is that this happens when PubSub.subscribe() and PubSub.on_connect() are called concurrently (thereby changing self.channels as on_connect() iterates over it.
https://github.com/andymccurdy/redis-py/blob/5109cb4f6b610e8d5949716a16435afbbf35075a/redis/client.py#L2376

RuntimeError: dictionary changed size during iteration
File "celery/app/task.py", line 413, in delay
return self.apply_async(args, kwargs)
File "celery/app/task.py", line 536, in apply_async
**options
File "celery/app/base.py", line 736, in send_task
self.backend.on_task_call(P, task_id)
File "celery/backends/redis.py", line 189, in on_task_call
self.result_consumer.consume_from(task_id)
File "celery/backends/redis.py", line 76, in consume_from
self._consume_from(task_id)
File "celery/backends/redis.py", line 82, in _consume_from
self._pubsub.subscribe(key)
File "redis/client.py", line 2482, in subscribe
ret_val = self.execute_command('SUBSCRIBE', *iterkeys(new_channels))
File "redis/client.py", line 2404, in execute_command
self._execute(connection, connection.send_command, *args)
File "redis/client.py", line 2415, in _execute
connection.connect()
File "redis/connection.py", line 502, in connect
callback(self)
File "redis/client.py", line 2374, in on_connect
for k, v in iteritems(self.channels):

I am having the same issue on edis==2.10.5 and celery==4.0.0

django-redis==4.9.0
redis==2.10.6
celery==4.1.0
recently gave me the same exception during the peak load.

Seeing the same exception:
redis==2.10.5
celery==4.0.2
With high load.

Also seeing this:
redis==2.10.5
celery==4.1.0

Same with redis 3.2.6, celery 4.2.0

redis==2.10.5
celery==4.2.1

This is probably race condition. PubSub object is not thread safe. Instance of PubSub class should be used in one thread only.
I think the same rule applies to celery.backends.redis.RedisBackend object.

Any suggestions on how to tackle this?

I am experiencing the same issue with Celery & Redis, very repetitively. Has anyone been able to figure out a work around?


MultiTouch Identifiers Misidentification

First of all, thank you for this cool project.
I'm not at all in app development and have only experience in python and a tiny bit of web dev. Excuse me in advance if I don't understand every subtility of the code.
I compiled and installed the mapper with no issue, managed to grant permissions but the app doesn't map the trackpad at all when started. The menu is displaying well so I guessed it wasn't a swift problem but rather a C/C++ issue.
I edited a bit the util file to add some dummy prints to try to have some clues about the place where the problem occurs:
    for (CFIndex i = 0; i < CFArrayGetCount(deviceList); i++) {
        MTDeviceRef device = (MTDeviceRef)CFArrayGetValueAtIndex(deviceList, i);
        int familyId;
        MTDeviceGetFamilyID(device, &familyId);

        // Dummy print 1
        printf("familyId: %d\n", familyId);
        printf("Trackpad found\n");

        if (
            familyId >= 98
            && familyId != 112 && familyId != 113  // Magic Mouse 1&2 / 3
        ) {

            // Dummy print 2
            printf("Starting Callbacks\n");

            MTRegisterContactFrameCallback(device, (MTFrameCallbackFunction)trackpadCallback);
            MTDeviceStart(device, 0);
        }
    }

Here is the output of these small modifications when I run the code, during the listing of devices part:
familyId: 176
Trackpad found
Starting Callbacks
*** Recognized (0xb0) family*** (60 cols X  2 rows)
familyId: 113
Trackpad found

However, I had disconnected my external MagicTrackpad for the experimentation to avoid disturbing anything, and what a surprise ! I still had 2 multitouch devices listed.
One of them is (n°176) and (in theory) is able to use absolute positioning
The other one (n°113) is, according to the comments, a Magic Mouse 2 (spoiler, it's not)
I finally managed to identify the problem.
I'm running the code on a MacBook Pro M2 13" 2022 with macOS Sonoma 14.1.2
On this device, the TouchBar is also a MultiTouch compatible device.
Here, my TouchBar is the device n°176 whereas my actual internal trackpad is n°113.
The fun part being that while the actual trackpad is not taken into account by the app due to the anti MagicMouse update (#14), the app is working really well on the TouchBar.
I can fully control my cursor from the touchbar, the small height and wide width of the device trying to match the screen size.
I will definitely break records on osu! with this setup (no).
For my usage, I simply removed the MagicMouse catch and it's working like a charm. I remember that I might encounter problem if I own one of these devices one day, but that's not that important.
Thank you once again for creating this nice app

I have been super busy in the past few months (till now), sorry for only noticing your issue after 8 days.
A solution to this problem is to have an updated list of familyId of all multitouch compatible devices, such that this app can hook callback to the correct devices. With the list, we may be able to

have multiple devices connected
be able to choose which device to hook callback

I wonder is there a predefined list of devices available to developers. Tbh I think we may need to reverse engineer familyID of all existing devices. It requires a community to do it but obviously there is no community surrounding this project.
Nevertheless, I should really spend some time fixing some of the issues relating to (re)connecting devices later.


Fix build on unsupported OS

src/lib.rs: replace conditional compilation with cfg_attr by cfg to define platform module + fix for unsupported OS

build/tests OK with an example code on Linux (OS supported) and OpenBSD (OS unsupported)

fix Arc-blroth/memory-stats#7

Nice! Looking forward to a new release with this 🙏


Submitting a 500k bytes PFD causes the chain to get stuck in rounds

Summary of Bug
When submitting a PFD with 500k bytes, the chain get stuck in rounds and can not proceed with generating a new block on a timely manner (test-case falls to fail after aprox 4 min of wait time for new block)
Ref: https://github.com/celestiaorg/test-infra/pull/48
Logs: https://gist.github.com/Bidon15/56997a852e8cce67768be50c138e2a7c
Version
v0.6.0
Steps to Reproduce
config.toml for each of the validators
mempool.max-txs-bytes =<PHONE_NUMBER>
mempool.max-tx-bytes = 1048576
mempool.size = 5000
consensus.timeout-commit = 30s

Note
If only 1 of 3 validators submit PFD containing 500k bytes of data, then it takes aprox. 142 seconds and 9 rounds to produce a block
More info here: https://gist.github.com/Bidon15/008e7b2952a363c3a117649617ee3d9c
Some logs to search
Jul 26 14:52:52.968538	INFO	62.3979s    MESSAGE << validators[001] (c6e79b) >> Submitting PFD with 500k bytes random data
Jul 26 14:55:39.132887	INFO	228.5622s    MESSAGE << validators[001] (c6e79b) >> latest size of the block is - 501173
Jul 26 14:55:35.893631	INFO	225.3235s      OTHER << validators[002] (d28ccb) >> 2:55PM INF committed state app_hash=3EB1A85C2A11CDACE719A6B74A22FDE89425C64AFE373578FE5A500C171DCD8C height=3 module=state num_txs=1

For Admin Use

[ ] Not duplicate issue
[ ] Appropriate labels applied
[ ] Appropriate contributors tagged
[ ] Contributor assigned/self-assigned

I'll be updating with the new data on validators set quantity(let's say 30) vs amount of 500k txs (starting with 1tx in block and moving forward)
Behaviour is pretty much similar to what we see on mamaki
As well as this needs to be retested once we downgrade to 0.34.x tendermint

Behaviour is pretty much similar to what we see on mamaki

can you elaborate on this?

I've tested this on mamaki, and large blocks do seem to increase the time it takes to produce a block, but the results vary. Note that this also seems to occur with other tx types besides PayForData.

can you elaborate on this?

Sure. I do remember that during the start of the mamaki chain, we had very big delays for creating new blocks (i don't know have we found the root cause and fix it) and not seeing any panics messages. And block times we had was like 4-5 min blocks
I've caught some of it again after playing around with pfd on mamaki (7 minutes block between 150940 to 150941)

some of the logs on 150940 block
https://gist.github.com/Bidon15/9fecfb69ea183770fe261884ddddc1e9
and .json output of the block rest request/response
https://gist.github.com/Bidon15/fd9a7863ec7735c4d1730a671431578d

We should re-test this once the downgrade to tendermint 0.34 is complete. (re #472)

Results on downgraded tendermint and celestia-app cosmos sdk(both sdk/app are on my forks) based on
https://github.com/celestiaorg/celestia-app/pull/472
https://github.com/celestiaorg/cosmos-sdk/pull/243
Network setup:
3 validators
consensus.timeout_commit = 60 seconds
Case 1⃣ : All 3 validators submits
Outcome -> 2/3 have error : RPC error -32603 - Internal error: timed out waiting for tx to be included in a block
https://gist.github.com/Bidon15/16e6fac365c63f7c8f323d2a4d2f650e
Case 2⃣  : One validator submits pfd with block flag
Outcome ->  Validator error : RPC error -32603 - Internal error: timed out waiting for tx to be included in a block
https://gist.github.com/Bidon15/63ad39d35c0b8033eb728a9b6c08cf75
Case 3⃣  : One validator submits pfd with sync flag
Outcome -> No data was included
https://gist.github.com/Bidon15/3b5a0afaf9a3945b079421ef0137c9c1

This issue can be closed after we finish testing on the cluster with 15+ validators set and 4MB blocks
So far, local docker testing have shown that we can produce 1.5 MB blocks constantly having only 3 validators (which is the original for creating this issue)
Proof: https://gist.github.com/Bidon15/5a906b597cc0075ff89f639e11004ba4

Closing as with 20 validators' set and 4mb blocks - we are safe and sound on producing blocks.
The different issue is the bandwidth to withstand the network load, which is out-of-scope of this issue
https://gist.github.com/Bidon15/276f5abf7e102d67c2ac967d904bb5d2

So the downgrade fixed this?

So the downgrade fixed this?

Yeap!

Can you let the tendermint team now via an issue as discussed, please?


Re:identify device supports fingerprint or not

Let anyone tell how to check whether the device support finger print sensor or not using javascript

That is what the isAvailable() method is for.  Please refer to README.  Result will indicate if hardware is available or not.


add trustServerCertificate if the parameter exists

Fixes #3572

I have this same problem. Do you have a planned date for when this PR will be integrated? Than you

Totally forgot about this - thought it was merged.
Will ask for a review.

Should be included in the preview now.

It works! Thanks!


"Berești-Tazlău", "Boșoteni", "Enăchești", "Prisaca", "Românești", "Tescani", "Turluianu" are villages in "Berești-Tazlău" common, "BACĂU" county

Description : When user is typing the villages in the search bar, the locations should appear in the dropdown menu correctly spelled and with localities information (as in Law nr. 2 din 16 februarie 1968)
Preconditions : the website (https://peviitor-ro.github.io/ui.orase/) is up and runing.
After every step delete the text you typed from the search bar.

Steps 2: Click on the search bar and write "Boșoteni"
Expected result: In the drop down menu you find "Boșoteni, BACĂU (Berești-Tazlău)"

Steps 3: Click on the search bar and write "Enăchești"
Expected result: In the drop down menu you find "Enăchești, BACĂU (Berești-Tazlău)"

Steps 5: Click on the search bar and write "Românești"
Expected result: In the drop down menu you find "Românești, BACĂU (Berești-Tazlău)"

Steps 6: Click on the search bar and write "Turluianu"
Expected result: In the drop down menu you find "Turluianu, BACĂU (Berești-Tazlău)"


WIP: feat: add HKTAN Segment Version 7

This pull request addresses #13 and adds HKTAN Segment Version #7. And resolves the issue I had while using clients.accounts(), however the use of the tan methods is not yet implemented.
See Section B.5.2 c) of the [PIN/TAN] Specification for the added fields

Hey Daniel, any progress or ideas on this?
Would love to help 😀

I didn't mean to implement the method at all, it was just a fix because my app crashed when Tan method version 7 was just available, even tough I didn't use it.

hm... any guidance on how to get this implementation going?
how are you guys testing btw? (cc @Prior99 ) just with your personal accounts?
last time i worked on some FinTS code, I quickly locked my banking account 🥴


用BootStrap setting文件的方式启动，如果json文件改错一次后，服务将不可用，一定要重启

1、我在应用代码中直接使用了BootStrap来启动服务，如下：
bootstrap.run(new String[]{serverType, "-p", port, "-g", filePath});
2、服务正常启动，然后估计改错JSON文件报如下错：
zaip-csbc-operation-service|WARN |2021-09-28 10:30:08,434|pool-1-thread-1|c.g.d.m.r.watcher.WatcherFactory|WatcherFactory.java:42|########com.fasterxml.jackson.databind.JsonMappingException: Unexpected character ('C' (code 67)): was expecting double-quote to start field name  at [Source: (String)"[ {         **CC**"request": {
3、把JSON文件改回来后，依然报错，无法启动
########Fail to load configuration in pbs.getUserInfoByIdNo.json.

Try to build your own Moco to check if it is fixed.
./gradlew proguard

Recent,i use standalone mock server as a docker service and i find  that the configuration change auto restart is fail.So i read the code about it,i find the auto restart service is based on java7's watchservice.And in docker environment,it won't trigger any file change events.
So can we use the basic way to monitor the file change such as use a schedule file scanner to do this.
Watchservice is out-of-date in cloud service.


Shell form ENTRYPOINT should be more intelligent

Why make the shell form for ENTRYPOINT a second class citizen?
Why not make it accept arguments, and interpret CMD additions?
Then their would be no need to document its deficiencies.
If docker is to become the best syntax to configure systems, it must support the most simple and intuitive syntax.
Are their any issues with this change besides backwards compatibility to consider?

what do you mean a second class citizen?
any ideas @tiborvass

@MetaMemoryT could you give an example of what you're suggesting?

As specified in the docs https://docs.docker.com/reference/builder/#entrypoint
"shell form" without the limitations currently listed, rather than "the preferred exec form"

it is not a second class citizen, it just is listed second under the preferred way of doing it. Projects will always have best practices and ours include this "preferred exec form". The "shell form" still works tho.


Introduce "resolved" flag for graph updates

Let the consumer of "update" event know that things like currentPorts and currentMetadata are resolved.

Duping into 4002


Issue with downloading YouTube live stream

I am trying to download a video that was originally part of a live stream but youtube-dl gives the error "ERROR: unable to download video data: HTTP Error 400: Bad Request". The specific video in question is https://www.youtube.com/watch?v=gTMnJNEWip4
The issue seems to be related to the format type since downloading the .3gp format type seems to work, but the mp4 dash do not. On the actual YouTube page, watching it with the mp4 dash formats work, so it seems to be an issue when obtaining the links.
Below is the output from my download script
[debug] System config: []
[debug] User config: []
[debug] Command-line args: ['--verbose', '--ignore-errors', '--no-continue', '--no-overwrites', '--keep-video', '--no-post-overwrites', '--download-archive', 'archive.txt', '--write-description', '--write-info-json', '--write-annotations', '--write-thumbnail', '--all-subs', '--output', '%(uploader)s (%(uploader_id)s)/%(id)s/%(title)s - %(upload_date)s.%(ext)s', '-f', 'bestvideo+bestaudio/best', 'https://www.youtube.com/watch?v=gTMnJNEWip4']
[debug] Encodings: locale UTF-8, fs utf-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2015.04.26
[debug] Python version 3.2.3 - Linux-3.8.13-bone47-armv7l-with-debian-7.4
[debug] exe versions: none
[debug] Proxy map: {}
[youtube] gTMnJNEWip4: Downloading webpage
[youtube] gTMnJNEWip4: Extracting video information
WARNING: video doesn't have subtitles
[youtube] gTMnJNEWip4: Searching for annotations.
[youtube] gTMnJNEWip4: Downloading DASH manifest
[info] Video description is already present
[info] Video annotations are already present
[info] Video description metadata is already present
[youtube] gTMnJNEWip4: Thumbnail is already present
WARNING: You have requested multiple formats but ffmpeg or avconv are not installed. The formats won't be merged
[debug] Invoking downloader on 'https://r9---sn-o097zne6.googlevideo.com/videoplayback/id/gTMnJNEWip4.1/itag/136/source/yt_live_broadcast/requiressl/yes/pmbypass/yes/gcr/us/playlist_type/CLEAN/ratebypass/yes/cmbypass/yes/mime/video%2Fmp4/live/1/gir/yes/signature/4E6CC3C889827CB8F5408B61C3214E911332A1C6.1E39BF49133E395D341567362CDAFCB91AA3730B/key/cms1/fexp/900720,903945,907263,924637,934954,936122,938028,9406013,9406631,9406705,9407115,9407433,9408093,9408142,9408347,9408705,9408807,9412491,943611,947233,948124,948703,948913,951703,952612,952637,952640,957201/sver/3/upn/ni-xyYIvMQY/ip/<IP_ADDRESS>/ipbits/0/expire/1430238687/sparams/cmbypass,expire,gcr,gir,id,ip,ipbits,itag,live,mime,mm,ms,mv,pl,playlist_type,pmbypass,ratebypass,requiressl,source/mm/32/ms/lv/mt/1430216757/mv/u/pl/17'
ERROR: unable to download video data: HTTP Error 400: Bad Request
Traceback (most recent call last):
  File "/usr/local/bin/youtube-dl/youtube_dl/YoutubeDL.py", line 1408, in process_info
    partial_success = dl(fname, new_info)
  File "/usr/local/bin/youtube-dl/youtube_dl/YoutubeDL.py", line 1363, in dl
    return fd.download(name, info)
  File "/usr/local/bin/youtube-dl/youtube_dl/downloader/common.py", line 342, in download
    return self.real_download(filename, info_dict)
  File "/usr/local/bin/youtube-dl/youtube_dl/downloader/http.py", line 64, in real_download
    data = self.ydl.urlopen(request)
  File "/usr/local/bin/youtube-dl/youtube_dl/YoutubeDL.py", line 1701, in urlopen
    return self._opener.open(req, timeout=self._socket_timeout)
  File "/usr/lib/python3.2/urllib/request.py", line 376, in open
    response = meth(req, response)
  File "/usr/lib/python3.2/urllib/request.py", line 488, in http_response
    'http', request, response, code, msg, hdrs)
  File "/usr/lib/python3.2/urllib/request.py", line 414, in error
    return self._call_chain(*args)
  File "/usr/lib/python3.2/urllib/request.py", line 348, in _call_chain
    result = func(*args)
  File "/usr/lib/python3.2/urllib/request.py", line 496, in http_error_default
    raise HTTPError(req.full_url, code, msg, hdrs, fp)
urllib.error.HTTPError: HTTP Error 400: Bad Request

And the formats available
root@beaglebone:~# youtube-dl -F gTMnJNEWip4 -v
[debug] System config: []
[debug] User config: []
[debug] Command-line args: ['-F', 'gTMnJNEWip4', '-v']
[debug] Encodings: locale UTF-8, fs utf-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2015.04.26
[debug] Python version 3.2.3 - Linux-3.8.13-bone47-armv7l-with-debian-7.4
[debug] exe versions: none
[debug] Proxy map: {}
[youtube] gTMnJNEWip4: Downloading webpage
[youtube] gTMnJNEWip4: Extracting video information
[youtube] gTMnJNEWip4: Downloading DASH manifest
[info] Available formats for gTMnJNEWip4:
format code  extension  resolution note
140          m4a        audio only DASH audio  144k , m4a_dash container, aac  @128k (48000Hz)
141          m4a        audio only DASH audio  272k , m4a_dash container, aac  @256k (48000Hz)
160          mp4        256x144    DASH video  124k , 15fps, video only
133          mp4        426x240    DASH video  258k , 30fps, video only
134          mp4        640x360    DASH video  616k , 30fps, video only
135          mp4        854x480    DASH video 1116k , 30fps, video only
136          mp4        1280x720   DASH video 2216k , 30fps, video only
17           3gp        176x144
36           3gp        320x240    (best)

I'm also having the same issue, which I cannot resolve.
ERROR: unable to download video data: HTTP Error 400: Bad Request

A livestream is only available in 3gp, I want it in 1080p. Is there a workaround or fix for this issue?
Luke$ youtube-dl -F nBwAYT_ogj4 -v
[debug] System config: []
[debug] User config: []
[debug] Command-line args: [u'-F', u'nBwAYT_ogj4', u'-v']
[debug] Encodings: locale UTF-8, fs utf-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2015.04.26
[debug] Python version 2.7.6 - Darwin-14.3.0-x86_64-i386-64bit
[debug] exe versions: ffmpeg 2.6.2, ffprobe 2.6.2, rtmpdump 2.4
[debug] Proxy map: {}
[youtube] nBwAYT_ogj4: Downloading webpage
[youtube] nBwAYT_ogj4: Extracting video information
[youtube] nBwAYT_ogj4: Downloading DASH manifest
[info] Available formats for nBwAYT_ogj4:
format code  extension  resolution note
140          m4a        audio only DASH audio  144k , m4a_dash container, aac  @128k (48000Hz)
141          m4a        audio only DASH audio  272k , m4a_dash container, aac  @256k (48000Hz)
160          mp4        256x144    DASH video  124k , 15fps, video only
133          mp4        426x240    DASH video  258k , 30fps, video only
134          mp4        640x360    DASH video  616k , 30fps, video only
135          mp4        854x480    DASH video 1116k , 30fps, video only
136          mp4        1280x720   DASH video 2216k , 30fps, video only
137          mp4        1920x1080  DASH video 4141k , 30fps, video only
17           3gp        176x144
36           3gp        320x240    (best)

Seems like YouTube had to process the video into additional formats. The new format list is shown below. Downloading the highest quality dash video seems to work.
root@beaglebone:~# youtube-dl -F gTMnJNEWip4 -v
[debug] System config: []
[debug] User config: []
[debug] Command-line args: ['-F', 'gTMnJNEWip4', '-v']
[debug] Encodings: locale UTF-8, fs utf-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2015.04.26
[debug] Python version 3.2.3 - Linux-3.8.13-bone47-armv7l-with-debian-7.4
[debug] exe versions: none
[debug] Proxy map: {}
[youtube] gTMnJNEWip4: Downloading webpage
[youtube] gTMnJNEWip4: Extracting video information
[youtube] gTMnJNEWip4: Downloading DASH manifest
[info] Available formats for gTMnJNEWip4:
format code  extension  resolution note
140          m4a        audio only DASH audio  134k , m4a_dash container, aac  @128k (44100Hz), 82.05MiB
141          m4a        audio only DASH audio  260k , m4a_dash container, aac  @256k (44100Hz), 162.94MiB
160          mp4        256x144    DASH video  130k , 13fps, video only, 69.86MiB
133          mp4        426x240    DASH video  271k , 26fps, video only, 156.50MiB
134          mp4        640x360    DASH video  528k , 26fps, video only, 196.95MiB
135          mp4        854x480    DASH video 1038k , 26fps, video only, 418.88MiB
298          mp4        1280x720   DASH video 2548k , h264, 53fps, video only, 1.19GiB
17           3gp        176x144
36           3gp        320x240
5            flv        400x240
18           mp4        640x360
22           mp4        1280x720   (best)


TextField

Closes #114

Shot Type
Count
Status

Added
0
🟡

Deleted
0
❌

Regression
5
❗

❗ storybook/components-icons--icons__[w1440px]:

Before
After

❗ storybook/components-icons--icons__[w1920px]:

Before
After

❗ storybook/components-icons--icons__[w390px]:

Before
After

❗ storybook/components-icons--icons__[w900px]:

Before
After

❗ storybook/utility-responsive-image--responsive-image__[w1440px]:

Before
After

Shot Type
Count
Status

Added
0
🟡

Deleted
0
❌

Regression
1
❗

❗ custom/widgets-user-profile--is-opened__[w1920px]:

Before
After

Shot Type
Count
Status

Added
0
🟡

Deleted
0
❌

Regression
4
❗

❗ storybook/components-icons--icons__[w1440px]:

Before
After

❗ storybook/components-icons--icons__[w1920px]:

Before
After

❗ storybook/components-icons--icons__[w390px]:

Before
After

❗ storybook/components-icons--icons__[w900px]:

Before
After

Shot Type
Count
Status

Added
0
🟡

Deleted
0
❌

Regression
5
❗

❗ custom/widgets-meal-nutrients--hint-opened__[w1440px]:

Before
After

❗ custom/widgets-meal-nutrients--hint-opened__[w390px]:

Before
After

❗ custom/widgets-navbar-opened__[w1920px]:

Before
After

❗ custom/widgets-progressInfo__[w1920px]:

Before
After

❗ custom/widgets-user-profile--is-opened__[w1920px]:

Before
After

Shot Type
Count
Status

Added
0
🟡

Deleted
0
❌

Regression
6
❗

❗ storybook/components-icons--icons__[w1440px] (click to expand)

Before
After

❗ storybook/components-icons--icons__[w1920px] (click to expand)

Before
After

❗ storybook/components-icons--icons__[w390px] (click to expand)

Before
After

❗ storybook/components-icons--icons__[w900px] (click to expand)

Before
After

❗ storybook/utility-responsive-image--responsive-image__[w1440px] (click to expand)

Before
After

❗ storybook/utility-responsive-image--responsive-image__[w1920px] (click to expand)

Before
After

Shot Type
Count
Status

Added
0
🟡

Deleted
0
❌

Regression
6
❗

❗ custom/widgets-collapsiblenavbar-opened__[w1439px]_statistics_submenu_popover (click to expand)

Before
After

❗ custom/widgets-collapsiblenavbar-opened__[w899px]_statistics_submenu (click to expand)

Before
After

❗ custom/widgets-meal-nutrients--hint-opened__[w1920px] (click to expand)

Before
After

❗ custom/widgets-meal-nutrients--hint-opened__[w900px] (click to expand)

Before
After

❗ custom/widgets-navbar-opened__[w1920px] (click to expand)

Before
After

❗ custom/widgets-progressInfo__[w900px] (click to expand)

Before
After

Shot Type
Count
Status

Added
0
🟡

Deleted
0
❌

Regression
1
❗

❗ storybook/utility-responsive-image--responsive-image__[w1440px]:

Before
After

Shot Type
Count
Status

Added
0
🟡

Deleted
0
❌

Regression
7
❗

❗ custom/widgets-collapsiblenavbar-opened__[w1439px]_statistics_submenu_popover (click to expand)

Before
After

❗ custom/widgets-collapsiblenavbar-opened__[w899px]_statistics_submenu (click to expand)

Before
After

❗ custom/widgets-linkstabbar--current-focus__[w1920px] (click to expand)

Before
After

❗ custom/widgets-meal-nutrients--hint-opened__[w1920px] (click to expand)

Before
After

❗ custom/widgets-meal-nutrients--hint-opened__[w900px] (click to expand)

Before
After

❗ custom/widgets-user-profile--is-opened__[w1920px] (click to expand)

Before
After

❗ storybook/widgets-heart-rate--default__[w1920px] (click to expand)

Before
After


Add functionality for copying a file

Coverage remained the same at 100.0% when pulling 03b43f7e6a4b6a89053632be1cd4eabcde3f6db6 on copyFile into 7ac43100f4c7c523856cd8c755fad3b71e51af16 on master.


Search stops with error when it comes along a broken reparse point.

Problem
Search stops with error when it comes along an illegal reparse point. (This is not a dangling reparse point, but an illegal!)
An illegal reparse point contains illegal reparse data.
Comments on the code
Basically it is an extension of the error handling in two places, which deals with error codes that come from illegal reparse data.
I accidentally stumbled across such a situation, when the search for a file stopped with the message 'The data present in the reparse point buffer is invalid.` This is ERROR_INVALID_REPARSE_DATA
Then I started to create illegal reparse points via the debugger with an extra tool, and found ERROR_CANT_ACCESS_FILE to be malicious too. Both are handled now.

You probably saw this; it also means selecting a bad reparse point will succeed with "The specified path is invalid" in the right hand pane.  The old code would display a dialog and refuse to select the folder.  I tried a few other operations and they appear to fail gracefully enough.

I know this, but a dialog is totally unusable during daily work.
Furthermore this dialog sometimes freezes the whole winfile. Don't ask me why.
So the point was to end up in 'invalid path specified' for all strange situations and have no extra confirmation.


More retries and extended delays for running iptables commands

Retry more times in case of iptables command failure, and introduce an incremental delay for consecutive failures
@lennardseah @giantballofyarn

:+1:

:+1:


Add "nightly" feature

Due to serde-rs/quasi#54, it is impossible to include crates that use serde-codegen with quasi alongside crates that do not.
To solve this, I implement a "nightly" feature that, if enabled, compiles the serde code without codegen. This will only work on the nightly compiler.
Also, updates the code to use serde_derive, as serde_macros is now deprecated.
Should the travis config be updated to test this? It's the same code as tested by stable.

Rebased.

Ping @softprops. Travis is failing due to https://github.com/rust-lang/rustup.sh/issues/72.

Going to close this now that serde macros and rust stable play nicely together


Update dependencies

This is a WIP PR.

@ilanbiala have you looked into PM2?
https://www.reddit.com/r/node/comments/2dwsee/forever_vs_pm2/cjuixsb

@simison I don't really see any major benefits over forever.

@simison Unitech doesn't credit their contributors anymore, so I prefer not to support a project that doesn't give credit to the authors.

we're using PM2 at work as well.

@lirantal Node 0.12 only has failing tests. However, I'd prefer to wait for us to up the test coverage before finalizing this because then we'll have to do less testing before merging.

Cool, fine by me.

Not a great idea to do this at once, so I'm closing and we'll do it incrementally as much as we can.


Adding Helix and LSDelux in Development

Type of Change

[x] New feature
[ ] Bug fix
[ ] Documentation update
[ ] Refactoring
[ ] Hotfix
[ ] Security patch
[ ] UI/UX improvement

Description

Helix is a awesome neovim alternative and lsd is very good looking ls replacment.
Testing

Impact

Issue related to PR

Resolves #

Additional Information

Checklist

[x] My code adheres to the coding and style guidelines of the project.
[x] I have performed a self-review of my own code.
[ ] I have commented my code, particularly in hard-to-understand areas.
[ ] I have made corresponding changes to the documentation.
[ ] My changes generate no errors/warnings/merge conflicts.

@dhruvmistry2000  It seems they are not accepting addition of new apps according to the documentation page

I heard about that but i didnt knew about it. I will close the PR.


[Message] Proposal: Add 'fluid' and change 'compact' behaviour

I propose:

that a fluid class is added to Message to enable full-width and that it by default is only the size of its contents
that the compact class reduces the padding of the Message instead

What do you think?

Messages are typically block. What would be the purpose of fluid?

fluid and compact are intertwined with each other. If compact were to be changed, the fluid option would have to be added since there would otherwise not be a way for the Message to become "fit-to-contents" if the Message is display: block/fluid by default. Therefor the Message also have to be "fit-to-contents" by default.
Does that make sense?


tests: Don't start DotcomWeb.Endpoint as a server in tests

Summary of changes
Start DotcomWeb.Endpoint without the server: true flag in tests.
Starting DotcomeWeb.Endpoint with server: true causes the tests to fail if anything is already running on the port supplied (in this case 4002). That conflicts with the docker config, which runs dotcom-1 on the same port, so if you try to run mix test while the docker setup is running, they will reliably fail.
Completely removing DotcomWeb.Endpoint runs afoul of one test, which has localhost:4002 hard-coded in its assertion, which is why I didn't completely obliterate it the DotcomWeb.Endpoint invocation. Another option, which I'm open to, would be to remove the hard-coded localhost:4002 from that test.

General checks

[x] Are the changes organized into self-contained commits with descriptive and well-formatted commit messages? This is a good practice that can facilitate easier reviews.
[x] Testing. Do the changes include relevant passing updates to tests? This includes updating screenshots. Preferably tests are run locally to verify that there are no test failures created by these changes, before opening a PR.
[x] Tech debt. Have you checked for tech debt you can address in the area you're working in? This can be a good time to address small issues, or create Asana tickets for larger issues.

New UI, or substantial UI changes

[x] Cross-browser compatibility is less of an issue now that we're no longer supporting IE, but changes still need to work as expected in Safari, Chrome, and Firefox.
[x] Are interactive elements accessible? This includes at minimum having relevant keyboard interactions and visible focus, but can also include verification with screen reader testing.
[x] Other accessibility checks such as sufficient color constrast, or whether the layout holds up at 200% zoom level.

New endpoints, or non-trivial changes to current endpoints

[x] Have we load-tested any new pages or internal API endpoints that will receive significant traffic? See load testing docs
[x] If this change involves routes, does it work correctly with pertinent "unusual" routes such as the combined Green Line, Silver Line, Foxboro commuter rail, and single-direction bus routes like the 170?

Which test checks that value? We can probably remove it too.

@anthonyshull https://github.com/mbta/dotcom/blob/6ba2846bf84efcf545d42b51d08f3860d8ec9d64/test/cms/paragraph_test.exs#L323-L328

There are a few tests that reference localhost:4002. Pretty sure they are just looking at URLs. Not sure they actually expect anything to be running.


[Contrib] Workspace for cuBLAS backend

This PR adds a 32MB workspace for cuBLAS backend, so that functions like cublasLtMatmul can take the workspace as input.
The workspace is managed under CuBlasThreadEntry so that it will be allocated only once in each thread.

cc @vinx13 @masahi @junrushao @tqchen

There is a pass to allocate such workspace and append it to arguments of a BYOC function: https://github.com/apache/tvm/blob/unity/python/tvm/relax/transform/transform.py#L1305-L1317
If this can be used, I think that would be preferred.

@masahi Thank you for the great suggestion! Yes I think this one can be a next step, so that the workspace size can be adjustable. For now, given I am not particular familiar with the BYOC flow, I may not be able to quickly enable the workspace to work with the AllocateWorksapce pass. Adding a fixed size workspace in thread entry is the fastest way to enable workspace. I agree that we can leverage the pass later on.

@masahi would you mind sharing more guidance to @MasterJH5574?


Added townlong-yak

This contains the mapping for adding townlong-yak. Please let me know if I need to map the download_url too.
Closes #15

https://raw.githubusercontent.com/ajour/ajour-catalog/3ce1414498adbc7167f11fe0dac7cd58e7b55391/townlong_yak.json
https://raw.githubusercontent.com/ajour/ajour-catalog/3ce1414498adbc7167f11fe0dac7cd58e7b55391/catalog.json

Super nice @petergp - thank you!

Please let me know if I need to map the download_url too.

We won't need it so I think this is just fine. I'll let this live until we have implemented the change in the frontend.

Updated the PR so we strip HTML tags from the summary
Further GitHub Actions now generate two catalogs.

catalog.json - Deprecated catalog that does not contain townlong-yak
catalog-2.0.json - New catalog with townlong-yak add-ons


Bug same GUID

I played the game with my friends in co-op mode.
I created the server and I, obviously, tried to trasfer the saving file from my friend's world inside my server.
My old GUID and my new GUID is the same.
So when i runned the script I got an error.
I opened "fix-host-save.py" and I noticed the problem at line 131.
So then I modified the script like that:
# We must rename the patched save file from the old GUID to the new GUID for the server to recognize it.
    if os.path.exists(new_sav_path):
        if new_sav_path != old_sav_path:
            os.remove(new_sav_path)
            os.rename(old_sav_path, new_sav_path)

    print('Fix has been applied! Have fun!')

This will resolve the issue.
If you want you can update your script.
This is my first time I use Python language, and I changed the script quickly, maybe you can change it in an better way than mine.

Wrong repository. You can delete this issue.


SwitchListTile and CheckboxListTile cannot be styled as a ListTile, missing shape parameter

Trying to round corners of the SwitchTileList and CheckboxListTile
There is a missing parameter for adding shape. I can't make the tiles in a Drawer look the same.
I also tried to wrap the whole SwitchListTile with ClipRRect without success,
Proposal
I would imagine that exposing the shape parameter of ListTile would do the job. Most of the props are already forwarded to ListTile like dense, selected, contentPadding etc.

Leaving this for reference : custom shape to ListTile was added in this issue.

@shihaohong I would like to work on this issue.

@shihaohong I have submitted a PR, PTAL whenever you get some time :)


pythonPackages.fs-s3fs: init at 1.0.0

Motivation for this change
fs is a python-based file system abstraction layer. The new package
fs-s3fs is an implementation of it which stores files inside an S3
bucket.
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/)
[x] Determined the impact on package closure size (by running nix path-info -S before and after)
[x] Fits CONTRIBUTING.md.

cc @ironpinguin

@GrahamcOfBorg build pythonPackages.fs-s3fs python3Packages.fs-s3fs

Success on x86_64-linux (full log)
Attempted: pythonPackages.fs-s3fs, python3Packages.fs-s3fs
Partial log (click to expand)
reading manifest file 'fs_s3fs.egg-info/SOURCES.txt'
writing manifest file 'fs_s3fs.egg-info/SOURCES.txt'
running build_ext

----------------------------------------------------------------------
Ran 0 tests in 0.341s

OK
/nix/store/zm0z25gsn7n4z9navvcpigs1dh8xrgf8-python2.7-fs-s3fs-1.0.0
/nix/store/spjm31qfhcz5syrkbj8acixc4l7q28d6-python3.7-fs-s3fs-1.0.0

Timed out, unknown build status on aarch64-linux (full log)
Attempted: pythonPackages.fs-s3fs, python3Packages.fs-s3fs
Partial log (click to expand)
AssertionError: [psutil.Process(pid=2272, name='python2.7', started='12:17:54'), psutil.Process(pid=2274, name='python2.7', started='12:17:54'), psutil.Process(pid=2275, name='python2.7', started='12:17:54'), psutil.Process(pid=2212, name='python2.7', started='12:17:51'), psutil.Process(pid=2277, name='python2.7', started='12:17:54'), psutil.Process(pid=2278, name='python2.7', started='12:17:54'), psutil.Process(pid=2273, name='python2.7', started='12:17:54'), psutil.Process(pid=2280, name='python2.7', started='12:17:54'), psutil.Process(pid=2281, name='python2.7', started='12:17:54'), psutil.Process(pid=2282, name='python2.7', started='12:17:54'), psutil.Process(pid=2279, name='python2.7', started='12:17:54'), psutil.Process(pid=2276, name='python2.7', started='12:17:54'), psutil.Process(pid=2267, name='python2.7', started='12:17:53'), psutil.Process(pid=2268, name='python2.7', started='12:17:53'), psutil.Process(pid=2269, name='python2.7', started='12:17:53'), psutil.Process(pid=2270, name='python2.7', started='12:17:53'), psutil.Process(pid=2271, name='python2.7', started='12:17:53')]

----------------------------------------------------------------------
Ran 764 tests in 31.563s

FAILED (failures=6, errors=344, skipped=55)
building of '/nix/store/1jjs8zwd48qlfm7z87rlwzljvdfy2ggs-python2.7-pyftpdlib-1.5.4.drv' timed out after 3600 seconds
cannot build derivation '/nix/store/2hfabsmc2gqnsazj78fagsxzhkbl4rhs-python2.7-fs-2.1.1.drv': 1 dependencies couldn't be built
cannot build derivation '/nix/store/9lanjncb4d0npbvlmj7s4ai6ckqlxs4r-python2.7-fs-s3fs-1.0.0.drv': 1 dependencies couldn't be built
error: build of '/nix/store/9lanjncb4d0npbvlmj7s4ai6ckqlxs4r-python2.7-fs-s3fs-1.0.0.drv' failed

Timed out, unknown build status on x86_64-darwin (full log)
Attempted: pythonPackages.fs-s3fs, python3Packages.fs-s3fs
Partial log (click to expand)
cannot build derivation '/nix/store/v269j0cs7x3lvf6c36j22kdiyjjpfzhp-python3.7-botocore-1.10.84.drv': 1 dependencies couldn't be built
cannot build derivation '/nix/store/xmg03ck8h1aw0bx3mlm0n58dgqsv4qki-python3.7-s3transfer-0.1.13.drv': 2 dependencies couldn't be built
cannot build derivation '/nix/store/j8anqgk22rs0r5m4dx65ms3s82fswng3-python3.7-boto3-1.7.84.drv': 2 dependencies couldn't be built
building of '/nix/store/pp8j2di24va5z0rfs5crbhg8a1ix4h7m-python2.7-pyftpdlib-1.5.4.drv' timed out after 3600 seconds
building of '/nix/store/p6dd7haagsvbwvg3ng5zq07f3h8r5g2y-python3.7-pyftpdlib-1.5.4.drv' timed out after 3600 seconds
cannot build derivation '/nix/store/4500nxmpafjf0pdfsqhbpm82wxnvhzmn-python2.7-fs-2.1.1.drv': 1 dependencies couldn't be built
cannot build derivation '/nix/store/8mmwyrx7zx2bqg2ck32lck7dj0rmh2jz-python3.7-fs-2.1.1.drv': 1 dependencies couldn't be built
cannot build derivation '/nix/store/bf8wjsi239kh904rr9avhnb80dmhh9mm-python2.7-fs-s3fs-1.0.0.drv': 1 dependencies couldn't be built
cannot build derivation '/nix/store/vpid6za0g5i10z29r03gj62hnn5vmh09-python3.7-fs-s3fs-1.0.0.drv': 2 dependencies couldn't be built
error: build of '/nix/store/bf8wjsi239kh904rr9avhnb80dmhh9mm-python2.7-fs-s3fs-1.0.0.drv', '/nix/store/vpid6za0g5i10z29r03gj62hnn5vmh09-python3.7-fs-s3fs-1.0.0.drv' failed

rebuilding due to timeouts (I guess that's been an issue with borg as the package is actually trivial)
@GrahamcOfBorg build pythonPackages.fs-s3fs python3Packages.fs-s3fs

Success on x86_64-linux (full log)
Attempted: pythonPackages.fs-s3fs, python3Packages.fs-s3fs
Partial log (click to expand)
reading manifest file 'fs_s3fs.egg-info/SOURCES.txt'
writing manifest file 'fs_s3fs.egg-info/SOURCES.txt'
running build_ext

----------------------------------------------------------------------
Ran 0 tests in 0.340s

OK
/nix/store/zm0z25gsn7n4z9navvcpigs1dh8xrgf8-python2.7-fs-s3fs-1.0.0
/nix/store/spjm31qfhcz5syrkbj8acixc4l7q28d6-python3.7-fs-s3fs-1.0.0

Timed out, unknown build status on x86_64-darwin (full log)
Attempted: pythonPackages.fs-s3fs, python3Packages.fs-s3fs
Partial log (click to expand)
cannot build derivation '/nix/store/h1gwncrradj44jig8m4k90dnzr8lp5xw-python3.7-botocore-1.10.84.drv': 1 dependencies couldn't be built
cannot build derivation '/nix/store/5snh28m89jkxqcwzb5r2n95a4isssqqz-python3.7-s3transfer-0.1.13.drv': 2 dependencies couldn't be built
cannot build derivation '/nix/store/lr28gdp9qnja0pxd421jxgvn1gizd2dz-python3.7-boto3-1.7.84.drv': 2 dependencies couldn't be built
building of '/nix/store/qhwh1fnv5b38b7pb6b180w9rpvxy73xd-python2.7-pyftpdlib-1.5.4.drv' timed out after 1800 seconds
building of '/nix/store/s790nqhii0bpzilvhaylj4sqzn4qglr1-python3.7-pyftpdlib-1.5.4.drv' timed out after 1800 seconds
cannot build derivation '/nix/store/7as6c1yy2hrx7axmzmw1si3mrr624hmn-python2.7-fs-2.1.1.drv': 1 dependencies couldn't be built
cannot build derivation '/nix/store/8pyy447wb5j1a7apdxllpr9pykfidrxg-python3.7-fs-2.1.1.drv': 1 dependencies couldn't be built
cannot build derivation '/nix/store/i2i3fwk2sbm0ynx7072x0vga37p37ri2-python2.7-fs-s3fs-1.0.0.drv': 1 dependencies couldn't be built
cannot build derivation '/nix/store/d4ii59f8gc2n052qzjd665m6xszdn776-python3.7-fs-s3fs-1.0.0.drv': 2 dependencies couldn't be built
error: build of '/nix/store/d4ii59f8gc2n052qzjd665m6xszdn776-python3.7-fs-s3fs-1.0.0.drv', '/nix/store/i2i3fwk2sbm0ynx7072x0vga37p37ri2-python2.7-fs-s3fs-1.0.0.drv' failed

Timed out, unknown build status on aarch64-linux (full log)
Attempted: pythonPackages.fs-s3fs, python3Packages.fs-s3fs
Partial log (click to expand)
AssertionError: [psutil.Process(pid=2272, name='python2.7', started='03:04:55'), psutil.Process(pid=2274, name='python2.7', started='03:04:56'), psutil.Process(pid=2211, name='python2.7', started='03:04:53'), psutil.Process(pid=2276, name='python2.7', started='03:04:56'), psutil.Process(pid=2277, name='python2.7', started='03:04:56'), psutil.Process(pid=2278, name='python2.7', started='03:04:56'), psutil.Process(pid=2273, name='python2.7', started='03:04:56'), psutil.Process(pid=2280, name='python2.7', started='03:04:56'), psutil.Process(pid=2281, name='python2.7', started='03:04:56'), psutil.Process(pid=2279, name='python2.7', started='03:04:56'), psutil.Process(pid=2275, name='python2.7', started='03:04:56'), psutil.Process(pid=2266, name='python2.7', started='03:04:55'), psutil.Process(pid=2267, name='python2.7', started='03:04:55'), psutil.Process(pid=2268, name='python2.7', started='03:04:55'), psutil.Process(pid=2269, name='python2.7', started='03:04:55'), psutil.Process(pid=2270, name='python2.7', started='03:04:55'), psutil.Process(pid=2271, name='python2.7', started='03:04:55')]

----------------------------------------------------------------------
Ran 764 tests in 49.522s

FAILED (failures=6, errors=345, skipped=55)
building of '/nix/store/mvgdnsikmk6iqpwkgfc9mqj8vvfs8kj4-python2.7-pyftpdlib-1.5.4.drv' timed out after 3600 seconds
cannot build derivation '/nix/store/fa0s38038gjfc6cp6kng75cxslvgkm17-python2.7-fs-2.1.1.drv': 1 dependencies couldn't be built
cannot build derivation '/nix/store/ni2hfcwq1hxgnry1dfz2wdb1m84r2ml0-python2.7-fs-s3fs-1.0.0.drv': 1 dependencies couldn't be built
error: build of '/nix/store/ni2hfcwq1hxgnry1dfz2wdb1m84r2ml0-python2.7-fs-s3fs-1.0.0.drv' failed

The original problem seems to be pyftpdlib which has been packaged some months ago and seems to break on Hydra. Currently investigating....

I decided to skip the tests in pyftpdlib for now. I'm not that surprised that an FTP library has a certain set of impure tests (that behave differently in Hydra builds and Borg tests, locally the simply worked).

Thanks for the reminder! I decided to skip the tests as they heavily rely on s3 buckets and such tests arehighly impure.

Thanks for the reminder! I decided to skip the tests as they heavily rely on s3 buckets and such tests arehighly impure.

All these impurities and we'll need a software filter :smile:
I'm testing locally. Should be good though.


[MINOR] Dead code from BrokerEndPoint removed

I found that several BrokerEndPoint methods nod used in production code.
It seems methods can be safely removed:

createBrokerEndPoint(Int, String)
readFrom(buffer: ByteBuffer)
connectionString(): String
writeTo(buffer: ByteBuffer)
sizeInBytes: Int

Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

@showuon
@hachikuji
@dengziming
Thanks for the review and merge!


sample app yaml needs a small tweak

I just tried to deploy the sample ingress app from the yaml referenced in the doc:
kubectl apply -f https://raw.githubusercontent.com/Azure/application-gateway-kubernetes-ingress/master/docs/examples/aspnetapp.yaml
And the image it is using (mcr.microsoft.com/dotnet/core/samples:aspnetapp) no longer exists. I change it to (mcr.microsoft.com/dotnet/samples:aspnetapp) and was able to deploy it.
Can someone fix the yaml to reflect this?
Thanks
Ed

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

ID: 94bdeded-846e-526c-8ce3-77ee36ffa852
Version Independent ID: 7d768d5c-0318-25f1-38f8-7a2ee15c1c41
Content: Tutorial: Enable ingress controller add-on for existing AKS cluster with existing Azure application gateway
Content Source: articles/application-gateway/tutorial-ingress-controller-add-on-existing.md
Service: application-gateway
GitHub Login: @greg-lindsay
Microsoft Alias: greglin

@eoq
Thanks for your feedback! We will investigate and update as appropriate.

@eoq , We have assigned this to the content owner/author for review.
@greg-lindsay , for your review.

Hi - I will update the article as I am able. I am closing this issue as duplicate because we have others saying the same thing. Thanks for bringing this up. #please-close


Code Quality: Improved UserControls codebase

Summary
This namespace was one of the places where there were leftovers from UWP.
Re-organized codebase in UserControls to enhance maintainability and removed some XAML files(OpacityIcon and DataGridHeader), so it must be little but the app should be faster a bit.
Task Checklist

[x] Rename IAddressToolBar to IAddressToolbarViewModel
[x] Rename ISearchBox to ISearchBoxViewModel
[x] Rename ISearchBox to ISearchBoxViewModel
[x] Rename Views.LayoutModes to Views.ContentLayouts
[x] Rename ViewModels.LayoutModes to ViewModels.ContentLayouts
[x] Move IAddressToolbarViewModel from UserControls to ViewModels.UserControls
[x] Move ISearchBoxViewModel from UserControls to ViewModels.UserControls
[x] Move ViewModels.Previews to ViewModel.UserControls.Previews
[x] Move ViewModels.Widgets to ViewModel.UserControls.Widgets
[x] Change base class of DataGridHeader from UserControl to ButtonBase
[x] Ensure we folllow our codebase guideline
[x] Clean up
[ ] Apply requested changes

Known Issues
Following issues would be fixed in the upcoming PR.
N/A
PR Checklist

[x] Related: #4180
[x] Approval: Have discussed with and got approval from the team[^1]
[x] Tests: Tested accessibility on the local end
[x] Deployment: Deployed the app on the local end
[ ] Localization: Modified en-US string resources[^2]
[ ] Co-authors: N/A

Steps To Validate Changes
There're some classes that were changed codebase largely. Please have look at:

See that DataGridHeader is working well
See that OpacityIcon is working well

Screenshots
N/A
[^1]: If the request hasn't been agreed by the team, this work might be rejected. Make sure that you get approval from the team before opening any PR related the request.
[^2]: If you removed any en-US string resources, make sure that there are no references of those resources. When you add a new en-US string resources, do not make any changes on other languages' resources; Crowdin will do, instead.

This is ready for review.

I'm not so sure about renaming LayoutModes to ContentLayouts, can you clarify where this is coming from?

Which steps were used to verify this changes?


Provide Option to Save Reading to a File

Most likely implemented via a -o / --output option; make each filename unique via something like a date/time (vs a random hash).

https://github.com/beeankha/command_line_tarot/pull/14

"Output readings to a text file" functionality is complete!


Docs: add infra config reference tables

@lulf, Can you please review again with that change? Thanks!

@ncbaratta, Can you re-review please?


Updated marked to 0.3.9

#63

@Simply007 Sure. I will make changes and update this pr by today.

@Simply007 Travis build is passing now. ;) Thanks for the guidance. :+1:

@ankurg22: Thank you for you contribution. Hope that this sample app is useful for you.
Ondřej


Reservations collection

Source code: reservation.js
const ReservationSchema = mongoose.Schema(
    {
        userId: {
            type: mongoose.ObjectId,
            required: [true, 'User ID is required']
        },
        railRouteId: {
            type: mongoose.ObjectId,
            required: [true, 'Train route ID is required']
        },
        seats: [
            {
                seatId: {
                    type: String,
                    required: [true, 'Seat ID is required']
                }
            }
        ]
    },
    {
        timestamps: true
    }
);

Finished.


Attempt at WinForms project part 2

Hi guys,
After the fixes in https://github.com/dotnet/corert/issues/7995#issuecomment-602634375 and https://github.com/dotnet/corert/issues/7995#issuecomment-602679256 I continued my attempts to run a minimum viable WinForms app. I had to create a minimal rd.xml file:
<Directives>
    <Application>
      <Assembly Name="System.Resources.Extensions"  Dynamic="Required All"/>
      <Assembly Name="System.Windows.Extensions" Dynamic="Required All"/>
    </Application>
</Directives>

which brought me to the next issue. There's an exception thrown from this code from WinForms:
Gdip.GdipLoadImageFromStream(new GPStream(stream), out image))

See https://github.com/dotnet/runtime/blob/master/src/libraries/System.Drawing.Common/src/System/Drawing/Image.Windows.cs#L35
and
internal static partial class Interop
{
    internal static partial class Ole32
    {
 [ComImport,
            Guid("0000000C-0000-0000-C000-000000000046"),
            InterfaceType(ComInterfaceType.InterfaceIsIUnknown)]
        internal unsafe interface IStream
....

See https://github.com/dotnet/runtime/blob/master/src/libraries/Common/src/Interop/Windows/Ole32/Interop.IStream.cs#L19
Considering this code talks about interface and Ole32, I assume I hit the missing COM support from #4219 . At any rate, I am opening this issue in case you can think of some smart way to continue forward.
This is my minimal repro project.

Yes, this is COM. The only workaround that comes to mind is not using the images on the toolstrip, or finding a different way to load the bitmaps. They're currently loaded from a resource stream. Image.FromFile would likely work, for example.

Removing the images actually showed the main window (TADA!). However, a second later the app crashed with UIAutomation related exception:
System.Exception: Method '[System.Windows.Forms.Primitives]Interop+UiaCore.UiaRaiseAutomationEvent(IRawElementProviderSimple,UIA)' requires marshalling that is not yet supported by this compiler.
   at Interop.UiaCore.UiaRaiseAutomationEvent(Interop.UiaCore.IRawElementProviderSimple, Interop.UiaCore.UIA) + 0x40
   at System.Windows.Forms.AccessibleObject.RaiseAutomationEvent(Interop.UiaCore.UIA) + 0x39
   at System.Windows.Forms.ToolStripItem.ToolStripItemAccessibleObject.RaiseFocusChanged() + 0x6b
   at System.Windows.Forms.ToolStripItem.Select() + 0x24c
   at System.Windows.Forms.ToolStripItem.HandleMouseEnter(EventArgs) + 0x16b
   at System.Windows.Forms.ToolStripItem.FireEvent(EventArgs, ToolStripItemEventType) + 0x14b
   at System.Windows.Forms.ToolStrip.OnMouseMove(MouseEventArgs) + 0x186
   at System.Windows.Forms.Control.WmMouseMove(Message&) + 0xea
   at System.Windows.Forms.Control.WndProc(Message&) + 0xab5
   at System.Windows.Forms.ScrollableControl.WndProc(Message&) + 0x8a
   at System.Windows.Forms.ToolStrip.WndProc(Message&) + 0x296
   at System.Windows.Forms.MenuStrip.WndProc(Message&) + 0xfa
   at System.Windows.Forms.Control.ControlNativeWindow.OnMessage(Message&) + 0x31
   at System.Windows.Forms.Control.ControlNativeWindow.WndProc(Message&) + 0x141
   at System.Windows.Forms.NativeWindow.Callback(IntPtr, Interop.User32.WM, IntPtr, IntPtr) + 0xee

So ...

 If you're building just something for your personal use, you can probably get rid of the accessibility stuff. Of course, don't ship like that (or have fallback on CoreCLR ready for users with special needs).
I usually look at stack traces like this and try to find an extensibility point somewhere that lets me get rid of the problem. Making your own ToolStripItem descendant that overrides ToolStripItem.CreateAccessibilityInstance might work. There might be more things like this. It depends how dedicated you are. WinForms won't work great without COM.

I just wanted to check if there are issues which could be fixed to make WinForms projects (as produced by Visual Studio designer) usable with CoreRT. While the COM issue is unfortunate, at least we now know that if one wants to use WinForms, they need to have a particular rd.xml, so my research was not completely in vain. I really appreciate your help and support - I learned much along the way, so I am happy. Closing this issue now.


x/tools/gopls: bad completion candidate for variadic call where method returns slice

What version of Go are you using (go version)?

$ go version
go version devel +869e2957b9 Mon Nov 16 22:24:14 2020 +0000 linux/amd64
$ go list -m golang.org/x/tools
golang.org/x/tools v0.0.0-20201111213328-5794f8bd7a57
$ go list -m golang.org/x/tools/gopls
golang.org/x/tools/gopls v0.0.0-20201111213328-5794f8bd7a57

Does this issue reproduce with the latest release?
Yes
What operating system and processor architecture are you using (go env)?
go env Output
$ go env
GO111MODULE=""
GOARCH="amd64"
GOBIN=""
GOCACHE="/home/myitcv/.cache/go-build"
GOENV="/home/myitcv/.config/go/env"
GOEXE=""
GOFLAGS=""
GOHOSTARCH="amd64"
GOHOSTOS="linux"
GOINSECURE=""
GOMODCACHE="/home/myitcv/gostuff/pkg/mod"
GONOPROXY=""
GONOSUMDB=""
GOOS="linux"
GOPATH="/home/myitcv/gostuff"
GOPRIVATE=""
GOPROXY="https://proxy.golang.org,direct"
GOROOT="/home/myitcv/gos"
GOSUMDB="sum.golang.org"
GOTMPDIR=""
GOTOOLDIR="/home/myitcv/gos/pkg/tool/linux_amd64"
GOVCS=""
GOVERSION="devel +869e2957b9 Mon Nov 16 22:24:14 2020 +0000"
GCCGO="gccgo"
AR="ar"
CC="gcc"
CXX="g++"
CGO_ENABLED="1"
GOMOD="/home/myitcv/.vim/plugged/govim/go.mod"
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -m64 -pthread -fmessage-length=0 -fdebug-prefix-map=/tmp/go-build353352024=/tmp/go-build -gno-record-gcc-switches"

What did you do?
Consider:
package main

import (
	"fmt"
)

type S struct {
	vals []string
}

func (s S) Vals() []string {
	return s.vals
}

func X(vs ...string) {
	fmt.Println(vs)
}

func main() {
	var s S
	X(s._)
}

with the cursor at the _ position. Attempt completion and you get the following candidates:
&protocol.CompletionList{
    IsIncomplete: true,
    Items:        {
        {
            Label:            "vals",
            Kind:             5,
            Tags:             nil,
            Detail:           "[]string",
            Documentation:    "",
            Deprecated:       false,
            Preselect:        true,
            SortText:         "00000",
            FilterText:       "vals...",
            InsertText:       "",
            InsertTextFormat: 1,
            TextEdit:         &protocol.TextEdit{
                Range: protocol.Range{
                    Start: protocol.Position{Line:20, Character:5},
                    End:   protocol.Position{Line:20, Character:5},
                },
                NewText: "vals...",
            },
            AdditionalTextEdits: nil,
            CommitCharacters:    nil,
            Command:             (*protocol.Command)(nil),
            Data:                nil,
        },
        {
            Label:            "Vals",
            Kind:             2,
            Tags:             nil,
            Detail:           "func() []string",
            Documentation:    "",
            Deprecated:       false,
            Preselect:        false,
            SortText:         "00001",
            FilterText:       "Vals...",
            InsertText:       "",
            InsertTextFormat: 1,
            TextEdit:         &protocol.TextEdit{
                Range: protocol.Range{
                    Start: protocol.Position{Line:20, Character:5},
                    End:   protocol.Position{Line:20, Character:5},
                },
                NewText: "Vals...",
            },
            AdditionalTextEdits: nil,
            CommitCharacters:    nil,
            Command:             (*protocol.Command)(nil),
            Data:                nil,
        },
    },
}

Note that the new text for the Vals candidate is wrong: it applies the ... without the parentheses required to first call the method.
What did you expect to see?
The candidate Vals()...
What did you see instead?
As above.

cc @stamblerre @muirdm
FYI @leitzler

This is because the non-snippet insert text doesn't get the invocation () parens, but it does tack on the ....
It would be easy to fix the insert text to yield Vals()..., but the below behavior would also change:
func foo() []int {
  return nil
}
var _ []int = f<> // would complete to "foo()" instead of "foo"

Do non-snippet users want the invocation parens when the function takes no arguments?

Thanks.

Do non-snippet users want the invocation parens when the function takes no arguments?

Would another option be to not add the ... when a call expression is involved?

Would another option be to not add the ... when a call expression is involved?

Yes, that is definitely an option too.

I'm trying to fix this issue, and I would like to know if the behavior of ignoring ... in variadic functions apply to Completion and CompletionSnippets. I'm asking this because I have a wip implementation that ignore ... if the candidate is a function, but the tests of CompletionSnippets has been failed because they expected to have ()... completion. I was in doubt if I should ask here on the issue or open a pr.

You want to suppress "..." if it is a function call and snippets are disabled. If snippets are enabled then there is no problem with the "...".


Program goes blank using Samsung Galaxy S10+ (Example Images Attached)

I went ahead and downloaded the latest version of the program today and I'm currently trying to connect it to my MacBook Pro (running MacOS Catalina 10.15.7), and for some reason the program goes blank when I hit the refresh button. I know it's connected properly because I ren Samsung Smart Switch and it was detected. I plugged in my Moto G3 just to try it and It works fine with that. Not sure what's up with it, but I attached screenshots for examples (censored my name on Smart Switch for privacy reasons).
Edit: I've tried making an error report for this occurrence, but each time I try it generates a block error log.

I will take a look. Thanks for reporting

I will take a look. Thanks for reporting

I'm having a similar issue on my S8+ w/ MacOS 10.15.7
I want to say the program gets stuck while refreshing, so that might be a place to look.
I seem to recall reading somewhere that Smart Switch interferes with MTP, but as I don't have it installed on my devices I can't say that's the cause.

I'm having a similar issue on my S8+ w/ MacOS 10.15.7
I want to say the program gets stuck while refreshing, so that might be a place to look.
I seem to recall reading somewhere that Smart Switch interferes with MTP, but as I don't have it installed on my devices I can't say that's the cause.

@xNS5 @atkeeler
Could you guys send me the error log by going to Help menu -> Report Bugs -> Email error logs -> Attach the log file to the email.
Cheers

@xNS5 @atkeeler
Could you guys send me the error log by going to Help menu -> Report Bugs -> Email error logs -> Attach the log file to the email.
Cheers

I was right that smart switch interferes with MTP!
If you check out this link, I found a few kext files leftover from when I had smart switch (which was a while ago). Once I uninstalled them, I was able to use the program without issue.

I was right that smart switch interferes with MTP!
If you check out this link, I found a few kext files leftover from when I had smart switch (which was a while ago). Once I uninstalled them, I was able to use the program without issue.

@ganeshrvel
After removing SmartSwitch and all the Samsung drivers from my MacBook Pro (as suggested by @xNS5), I am able to access my S10+ through OpenMTP. I would recommend adding into the description that if a user has SmartSwitch installed on their Mac that it might interfere with the program due to the Samsung drivers that are installed on the system.

@ganeshrvel
After removing SmartSwitch and all the Samsung drivers from my MacBook Pro (as suggested by @xNS5), I am able to access my S10+ through OpenMTP. I would recommend adding into the description that if a user has SmartSwitch installed on their Mac that it might interfere with the program due to the Samsung drivers that are installed on the system.

Reopening this issue until its fixed.

Reopening this issue until its fixed.

The issue has been fixed. The solution is to uninstall Smart Switch and remove the com.devguru.* kext files. I don't think there is anything you can do on your end to prevent these drivers from interfering with OpenMTP other than adding a disclaimer to the project README.

The issue has been fixed. The solution is to uninstall Smart Switch and remove the com.devguru.* kext files. I don't think there is anything you can do on your end to prevent these drivers from interfering with OpenMTP other than adding a disclaimer to the project README.

i will have to make a note inside the faqs sections of the app

i will have to make a note inside the faqs sections of the app

Updated the FAQ section in the latest version and README file. Closing this issue.


Path doesn't get added automatically during setup

VSCode Version: 1.44.2 (user setup)
Commit: ff915844119ce9485abfe8aa9076ec76b5300ddd
Date: 2020-04-16T16:36:23.138Z
Electron: 7.1.11
Chrome: 78.0.3904.130
Node.js: 12.8.1
V8: 7.8.279.23-electron.0
OS: Windows_NT x64 10.0.18363

Steps to Reproduce:

Download and install using 64-bit Windows User Installer
Run code in the Powershell window.

error logs

code : The term 'code' is not recognized as the name of a cmdlet, function, script file, or operable program.
Check the spelling of the name, or if a path was included, verify that the path is correct and try again.
At line:1 char:1
 +code
 +~~~~
    + CategoryInfo          : ObjectNotFound: (code:String) [], CommandNotFoundException
   + FullyQualifiedErrorId : CommandNotFoundException

I know it's a simple issue and can be resolved just by adding the VSCode installation path to the environment variables.
But, it should happen automatically during installation, right?

Show me your PATH: echo $env:PATH.


istio bump to 1.11.4

Istio bump to 1.11.4
See also: kyma-project/kyma#12498

 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.


[cmd] CHECKPOINT - create a checkpoint/tag/version  (or backup to another dir)

Through our meeting and presentations it seems like version history is pretty important, even something basic. Basic versioning would require:

Storing historic files somewhere
Keeping a log of tags/checkpoints
Ability to look at files from a specific tag/checkpoint

We have a pending PR in hyperdrive to add checkpoints. But that will still require saving the files somewhere to checkout.
I made dat-backup to try to solve the storage issues using hypercore-archiver.  The cool part (and why I was so motivated to start this with hypercore-archiver) is that we can use the Svalbard type infrastructure to selectively store backups on different machines. I could give my changes feed to a Svalbard-type server and give criteria on what to backup!!
Usage & Implementation
Most of this is already implemented in dat-backup, its just a matter of cleaning up and changing the
commands to suit dat-next.
You'd run backup in a directory with an existing archive:
dat-next backup --name "first backup"

This will copy the files and create a snapshot archive inside dir/.dat-backups/<static-key>. It will also append to the changes feed:
{ "key": "<static-key>", "type": "add", "data": { "name": "first backup"}}

When hyperdrive checkout PR merges, we can append the same data the hyperdrive feed.
To list backups you could do:
dat-next backup --list

Prints out:
Backups

First backup
  4f86658db9f1a738832c7488a9d3c096dc4f3af1633b43cfa581184dbfaffdcd

-----------
Backup Feed
  1 Active Backup
  Key: 50fdc4987ea2989417e393ff029129c4377cc8a93295da3c61cbeda7930d51fb
-----------

Where backup feed is the hypercoe-archiver changes feed and the 4f866... key is the snapshot key for the backup.
This would allow us to serve the backups over the Dat network:
dat-next backup --serve

And someone else could run
dat-next clone 4f86658db9f1a738832c7488a9d3c096dc4f3af1633b43cfa581184dbfaffdcd

Users can clone the backup as if it was a regular archive (since it is). This would allow a user to clone their own backups as a minimal way checkout old files. But we could also alias a dat-next checkout type command to the clone command.

This issue was moved to datproject/dat#650


First review

@Recmo please review my commits
I significantly simplified RoT and TimeSource contracts please review again

I like that the contracts are more explicit, with inheriting Ownable and all the safeMath. Makes it easier to review!

I think next step would be to apply Solidty Style Guide. Solium is a great tool for linting Solidity files.


chore: update dependencies

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


inline styling

what do you think?
http://jamesknelson.com/why-you-shouldnt-style-with-javascript/

Hi @plandem, we have thought about this a lot and most recently started an experiment to explore how Belle could evolve. If you have time feedback here is very much appreciated: https://github.com/nikgraf/future-react-ui/

done

thx :+1:


[NDMII-2654][snmp] store devices tags on disk with mock directory

What does this PR do?

Motivation

Additional Notes

Possible Drawbacks / Trade-offs

Describe how to test/QA your changes

/merge


Add timestamp

Echoing @f-gate it looks good but lets add a test
Also I  think because this is a pr to the rebase branch the tests didn't run because we do have a test against max cap contributions that should have failed with using old_contribution. Can you please verify if that is the case @cuteolaf ?

fixed & pushed a commit.

@cuteolaf im merging in but can you please add a pr to add a test that a new contribution gets assigned a timestamp value


Tuple arity and formatter removing ()

For some reason x is a u64 and not a tuple containing a single element
When running the formatter the type annotation on y gets changed from (u64) to u64 however this is documentation therefore we need the ()

contract;

abi MyContract {
    fn test_function();
}

impl MyContract for Contract {
    fn test_function() {
        // x is of type u64
        let x: u64 = (42);

        // y is of type u64
        let y: (u64) = (42);

        // z is of type (u64), i.e. a one-arity tuple
        let z: (u64, ) = (42, );
    }
}

What should happen here? Is the compiler incorrect; is the formatter incorrect?

cc @eureka-cpu @mohammadfawaz

If this is happening after formatting it is the formatter, I'll take a look at how we are formatting ty at some point this weekend if nobody gets to it first

I've looked into this and if the ty of x was indeed (u64) on the compile side, it would parse and automatically get parenthesis from the formatter... @mohammadfawaz I'm not really sure where I should be looking on the compile side of things if it is indeed a compiler issue. If you point me in the right direction, I'll gladly tackle it tonight. To be more clear, any time a parsed ty is matched on in the formatter, it will always get parenthesis. https://github.com/FuelLabs/sway/blob/master/swayfmt/src/utils/language/ty.rs#L34

What I think may be happening is that the parsed type is u64 and not (u64), otherwise the formatter would have added the parens

What I think may be happening is that the parsed type is u64 and not (u64)

Yeah, the AST probably just have a u64 instead of a (u64). Emily implemented that here: https://github.com/FuelLabs/sway/pull/1774 if that helps. Interestingly, Rust has the same behavior and gives a warning that the parentheses are not used but the formatter does not drop them. Since our formatter is based strictly on the AST without any side information, I'm not sure how we would go about this.

What I think may be happening is that the parsed type is u64 and not (u64)

Yeah, the AST probably just have a u64 instead of a (u64). Emily implemented that here: #1774 if that helps. Interestingly, Rust has the same behavior and gives a warning that the parentheses are not used but the formatter does not drop them. Since our formatter is based strictly on the AST without any side information, I'm not sure how we would go about this. We will need some additional information in the AST to describe what's happening. How critical is this and do we really need to talk about this case in the Sway Book? Seems like one of those trivial things that no one cares about.

It is in the current sway book therefore it is one of the sections that I have ported for consistency. I had to look up what arity was when I saw this. I don't find this to be helpful and I'd rather remove it from the new book anyway.

What I think may be happening is that the parsed type is u64 and not (u64)

Yeah, the AST probably just have a u64 instead of a (u64). Emily implemented that here: #1774 if that helps. Interestingly, Rust has the same behavior and gives a warning that the parentheses are not used but the formatter does not drop them. Since our formatter is based strictly on the AST without any side information, I'm not sure how we would go about this. We will need some additional information in the AST to describe what's happening. How critical is this and do we really need to talk about this case in the Sway Book? Seems like one of those trivial things that no one cares about.

Personally, now that I understand what's happening, I don't think any change is necessary as it's just a nit and seems inconsequential to any case I can think of.

Maybe adding that the formatter will remove them in the warning message would be okay, though overly verbose in my opinion.

Yeah so that's a non-issue and we can remove that example in the book.

@Braqzen I'm closing this issue if that's okay with you. I think we all agreed that the example should just be removed from the new book as it's not particularly useful.


Make setup.py install the scripts for you

Right now the setup.py script puts all three Python files in the /anaconda3/lib/python3.7/site-packages/finder_maker/ directory (or something similar), but there should be a way to have only the functions.py go there and have the other two files (which are executable scripts) go to /anaconda3/bin/. Then they would automatically be in your path and you wouldn't have to do the aliasing you suggest in the README.

And then you should also make the scripts actually executable by adding the line #!/usr/bin/env python at the top.


TransformSynchronizer not sharing and receiving information

Overview
I'm trying to share the transform position, rotation and scale of a gameobject that is not spawned at runtime, but already exists in the scene of the two hololens.
In my project I have:
Manager GameObject with:

SharingStage
SharingWorldAnchorManager

WorldAnchor with:

AutoJoinSessionAndRoom

Sphere (Child of WorldAnchor) with:

TransformSynchronizer
Where I changed the Start method with:

private void Start()
        {
            Initialize();
            transformDataModel = new SyncTransform("transform");
        }

The session and room are created but no information is synched between the two hololens.
Expected Behavior
When I move a sphere in the scene of one Hololens, the sphere moves in the scene of the second Hololens.
Actual Behavior
When moving the sphere in the Hololens 1, in the Hololens 2 there is no movement.
No OnPositionChanged events are called in TransformSynchronizer of the second hololens.
Unity Editor Version
Unity 2017.3.1f
Mixed Reality Toolkit Release Version
2.1.1

Closing issues older than 180 days. If this is still an issue, please reactivate with recent information.


Docs are not updated to 0.8.0

https://mesosphere.github.io/marathon/ features 0.7.5.

Thanks, changed. Should be visible shortly.

@drexin rest api docs seems to be outdated too. I don't see maxLaunchDelaySeconds, for example.

updated

:+1:


Go back to regular pip

Hahaha and of course after weeks of this bug, everything gets resolved the day I press the merge button. This should fix the issues where

pip install scvelo downgrades a flit install -s installed scanpy (now the dist-info dir name contains the correct version)
The wheel built by flit now corresponds to the freshly-changed spec’s mangling rules

But that’s no problem, git blame won’t be affected much, this almost exclusively deletes lines.

Just checked using this dockerfile, works flawlessly:
FROM continuumio/miniconda

RUN conda install python=3.8
RUN pip install flit>=3.1
RUN git clone https://github.com/theislab/scanpy.git
WORKDIR /scanpy
RUN git checkout mainline-pip || true  # go to the mainline-pip branch if it hasn’t been merged int master yet
RUN FLIT_ROOT_INSTALL=1 flit install -s --dep=develop  # Make development install of scanpy
# make sure the dist-info folder has a plus in its name
RUN SCANPY_VERSION=$(python -c 'from importlib.metadata import version; print(version("scanpy"))') && \
    test -d /opt/conda/lib/python3.8/site-packages/scanpy-$SCANPY_VERSION.dist-info
RUN pip install scvelo  # Install project that depends on scanpy
RUN test -L /opt/conda/lib/python3.8/site-packages/scanpy