System.IdentityModel.Tokens.Jwt.dll has a hard dependency on Newtonsoft.Json 6.0

It looks like System.IdentityModel.Tokens.Jwt.dll version <IP_ADDRESS> has a hard dependency on Newtonsoft.Json version <IP_ADDRESS>.  It blocks us from using this dll because other components in our system has dependency on Newtonsoft.Json 8.0.  It would be great to remove this dependency so that consumer could plugin their own JSON serializer and deserializer.  Maybe the hard dependency could be removed from the JsonExtensions static constructor so that the custom JSON lib could be plugged in via properties.

@xinyu8 what error are you experiencing? Compile? Updating References? Runtime?

It's runtime error for not be able to load Newtonsoft 6.0 dll.
Background: we use pacman (instead of Nuget) to manage packages/libraries. The IdentityModel dll is packaged into one pacman package from Nuget without any external dependencies.  After encounter the runtime error, I tried to add dependencies to Newtonsoft pacman package within my component.  However, I was not able to add Newtonsoft 6.0 because the minimum version of Newtonsoft supported in pacman is 8.0.

@xinyu8 I am not familiar with 'pacman'. Where can I get it so I can play with it?

oh, that Bing or OSD depot. You could start from https://www.bingwiki.com/Pacman

Fixed with PR #355


Strip off ANSI Escapes from g:find_files_findprg command output

Output from g:find_files_findprg command may contain AnsiEsc sequence (e.g.^[[0m^[[32m./output/dir^[[0m). This pull request makes vim-find-files able to open files/directories in the "File List" buffer with AnsiEsc sequence in it (by stripping off the ansi escape sequence before following the link to the file).
This allows vim-find-files to have output being colored inside Vim using AnsiEsc plugin by adding the following setting to .vimrc:
function! SetHighlightFindFiles()
    if &filetype ==# 'findfiles'
        AnsiEsc
    endif
endfunction

augroup my_find_files_customization
    autocmd! FileType findfiles :call SetHighlightFindFiles()
augroup END " my_find_files_customization

And here is the result (with "File List" buffer with syntax highlight using AnsiEsc):

The current code in the master branch uses "gf" to navigate to the filenames in the "File List" window, which would be impossible if the output "File List" window contains ANSI Escape sequence like this:

(this screenshot is what things look like without AnsiEsc and the code in this pull request)


flex attribute compile error

Is this a bug report?
yes

Did you try recovering your dependencies?
yes
Which terms did you search for in User Guide?
Environment

node -v:  v9.3.0
npm -v: 5.6.0
yarn --version (if you use Yarn):
npm ls react-scripts (if you haven’t ejected): 1.1.0

Then, specify:

Operating system: macOS 10.13.2
Browser and version (if relevant): Google Chrome Canary 66

Steps to Reproduce

(Write your steps here:)

write this in css: flex: 1 1 0;, this mean flex-grow: 1; flex-shrink: 1; flex-basis: 0
save and compile

Expected Behavior

the compiled css should be: flex: 1 1 0;
Actual Behavior

the compiled css is: flex: 1 1;
because flex: 1 1; mean flex-grow: 1; flex-shrink: 1; flex-basis: 0%; the problem is:
flex-basis: 0% not equal to flex-basis: 0, and so produce style bug

because flex: 1 1; mean flex-grow: 1; flex-shrink: 1; flex-basis: 0%;

What do you base this information on? MDN claims the default value for flex-basis is 0.
https://developer.mozilla.org/en-US/docs/Web/CSS/flex
I believe the spec says the same.

@gaearon yes, you are right. But in chrome 64.0.3282.140 (64-bit)，the default value for flex-basis is 0%, I think this is a bug.

Please report it to Chrome 🙂


Detail of training

Hi, I'm trying to train yolact on cityscapes dataset with my code. However, my result just looks worst.
I've traced your cityscapes training config. Did you only modify the image size, learning rate and lr_steps on training compared to the original yolact config?
or there is some additional modification or trick? Thank you.

The flag USE_SQUARE_ANCHORS in the original yolact_v1 config is set to True, this makes it hard for the network to detect objects with large aspect ratios like pedestrains.


Consider making breaking change of making dart:io APIs return deeply immutable bytes.

The VM has support for sharing deeply immutable typed data across isolates. Though it is cumbersome to create such deeply immutable typed data - especially if one wants to avoid an additional copy (see e.g. dart-lang/sdk/issues/50068).
Many typed data instances originate from outside dart code (e.g. loaded from filesystem, socket / http, etc). When that data is read, it is very rarely modified.
So I propose making all dart:io APIs return deeply immutable bytes. It would be a breaking change due to a very small percent of use cases where such data is actually modified, those places would need to be updated.
Though the benefit (all dart:io returned bytes are sharable across isolates by-pointer) seems to outweigh the cost to me (very rare case of modifying such bytes).
The breaking change would not necessarily imply we have to change the actual types of the API (though that would be an option)
/cc @lrhn opinions?

If you can pull it off, I have no problem with the change.
It's a breaking change. Whether someone is actually broken depends on whether you're right that people don't actually change the data being read in.
Do we have a proper unmodifiable ByteBuffer to back this?

Do we have a proper unmodifiable ByteBuffer to back this?

We do - at least in the VM.
The trick is to generate a backing, a _UnmodifiableByteBufferView on top of it and ensure nobody can access the backing store (thereby ensuring nobody can write to it).
Our C API exposes this functionality, see NewExternalTypedData(..., unmodifiable)

We could consider if there is a path to making this opt-in or configurable in some way, e.g. through a separate set of APIs or
through some annotation or language version (somehow)? Then we don't have to bother about it being breaking.

I think doing so may be feasible, at the expense of clean API. Though for us to actually benefit, we'd need to go through dart:io and all code that reads from files and change that to opt-in to getting immutable bytes. That seems like a lot of changes.
If we on the other hand, decided to change the semantics to return immutable bytes, we only need to update very few places that actually need a writable bytes. The price we pay here is that for those use cases, we have to make mutable copies.
@lrhn wdyt?
Someone could do a trial of this in g3 and see how many places would need updating (one place I'm aware of is the WebSocketTransformer code)


Remote design values exercise

Design a remotely facilitated design values exercise. Need to decide which online tool to use: currently considering Boardthing, Google Slides, and Mural.

@pamela-drouin I think that Boardthing worked out really well. I would recommend it to other folks.

Yay! I agree, it worked well, especially for being free and having the
ability to invite guests. Also, I have to say, their export feature is
fantastic (multiple formats to choose from).
On Thu, Jun 16, 2016 at 4:03 AM, iamjessklein<EMAIL_ADDRESS>wrote:

@pamela-drouin https://github.com/pamela-drouin I think that Boardthing
worked out really well. I would recommend it to other folks.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/bocoup/opendesignkit/issues/41#issuecomment-226454631,
or mute the thread
https://github.com/notifications/unsubscribe/AFrJSa9mpvkUZ2kQhkR_HPoWToBmrxWiks5qMS2PgaJpZM4I1d3y
.

--
Pam Drouin
UX Designer @Bocoup

@pamela-drouin I'm going to close this issue so that we focus on the related issues #57  and #43


Increase the size of the memory map to 640KByte

Increase the size of the memory map to 640.
To solve problems that arise from large multi-transaction blocks. Unable to put 100 transactions in 64KByte's memory.
This problem will soon be fixed.

Where is the test-case?

And, I don't think this will solve the issue. A transaction can contain a large number of inputs and outputs. I don't see how this will allow supporting arbitrary "100" transactions in the block.

So from my understanding, this commit here should not have been included when being merged?

Thanks, looks good now!


[Android] handle CALL_FROM with null value

Hi,
On this commit, I added null checker before replace "client:" on handleCallIntent method in BackgroundCallJavaActivity class.

Looks good! could you explain in which scenario call_from can be null?

This addition is no longer relevant to Android.


feat: screening assessment shows changes

Implements NDT-219

@rafasdc @RRanath think we can get this reviewed?


jq: error

jq: error (at :0): Cannot index string with string "dns_names"
jq: error (at :5): Cannot iterate over null (null)
parse error: Invalid numeric literal at line 3, column 0
debian 9 and debian 10
sudo apt install jq
$ git clone https://github.com/vincentcox/bypass-firewalls-by-DNS-history
bash ./bypass-firewalls-by-DNS-history.sh -a -d dadadadada.de

Also seeing on Ubuntu 20.04. Is anyone maintaining this repo?

Also seeing on Ubuntu 20.04. Is anyone maintaining this repo?

Feel free to make a pull request! ;)

same here on OS X Catalina

I am facing this on mac, I too installed jq using brew


Fix update circleci api

I don't know when this happened, but it seems that "permission denied" without using this method (failed job).

Build log
Preview build with this PR (commit): https://output.circle-artifacts.com/output/job/b7bc06da-b0fb-45a9-96e2-5fb852e52ec8/artifacts/0/index.html

It was not that api post could only be done with this method.
When I saw the "Personal API Tokens" screen of circle ci, the registered "Personal API Tokens" key is deleted...why...?

THIS is the cause.


SENTRY_RELEASE and SENTRY_ENVIRONMENT

It looks like these two (SENTRY_RELEASE and SENTRY_ENVIRONMENT) are missing support in at least v2. I haven't checked v1.
See https://docs.sentry.io/error-reporting/configuration/?platform=php#release
I see support for it in sentry-python but not here.

I suppose that this should be supported across all SDKs. Would you mind opening a PR to fix the issue? The change should be pretty easy and require to just edit the defaults in the Options class


npm install -S?

I'm guessing -S is to save to save to your package.json -- but that's the default now.

It only became the default in March, I'll give it a while before changing the docs to support people stuck on old versions etc.


Enabled accurate quality profiles with CP, SB and Sonnar plugins

Enabled couchpotato plugin to send correct movie quality to movie queue:

Added option to get entry('quality_req') in movie_queue
Added helper method to correctly convert quality profile in CP to FlexGet format quality    requirement
Minor tweaks

Enabled Sonnar plugin to send accurate quality profile when using configure_series plugin

Will now send converted qualities using 'configure_series_qualities' and quality cutoff using 'configure_series_target'

Enabled Sickbeard plugin to send accurate quality profile when using configure_series plugin

Will now send converted qualities using 'configure_series_qualities'

@gazpachoking this is ready to be merges as far as i'm concerned


Best Practices NOT SHOWING

Thanks for stopping by to let us know something could be better!
PLEASE READ: If you have a support contract with Google, please create an issue in the support console instead of filing on GitHub. This will ensure a timely response.
Please run down the following list and make sure you've tried the usual "quick fixes":

Search the issues already opened: https://github.com/googleapis/google-api-php-client/issues
Search StackOverflow: http://stackoverflow.com/questions/tagged/google-cloud-platform+php

If you are still having issues, please be sure to include as much information as possible:
Environment details

OS: Linux
PHP version: 7.2.24
Package name and version:  google-api-php-client--PHP7.0

Steps to reproduce
Install the library, make a request, the SEO, PERFORMANCE AND ACCESSIBILITY data are showed, but not BEST_PRACTICES
Code example
    $pageSpeedService = new Google_Service_PagespeedInsights($client);

    $optParams = array(
        "category" => [
            "SEO",
            "PERFORMANCE",
            "BEST_PRACTICES",
            "ACCESSIBILITY"
        ],
        "strategy" => "DESKTOP"
    );

    $result = $pageSpeedService->pagespeedapi->runpagespeed("https://medium.com", $optParams)->getLighthouseResult();

Making sure to follow these steps will guarantee the quickest resolution possible.
Thanks!

Hello! It looks like you want help on how to use a specific API. The maintainers of this repository only provide support for the code in this repository. Make sure your client library is up-to-date by running composer update, and search for your question on Stack Overflow.
If you have a paid support plan with Google Cloud, you can open a support ticket.


Eris 0.15.0 has breaking changes

Pluris isn't affected significantly, but it affects probably the 2 most used plugins, awaitMessages and awaitReactions.
What's the breaking change? message.channel can now be uncached, therefore accessing the client property from both these plugins will cause Pluris to error. All other breaking changes in v0.15.0 shouldn't affect Pluris (I'll regret saying this).
I'll have this fixed in the near future and release this and the removal of deprecated Embed class (see the docs for info) in a semver minor update. Updates to endpoint URLs [Typescript] may also be featured in the semver minor but I won't guarantee it.

After reviewing my code, it seems like this isn't affected. I'll still update pluris to be UTD with eris


Refactor WebRTC events

This is a work item for #685.
Pages:

[x] https://developer.mozilla.org/en-US/docs/Web/Events/addstream
[x] https://developer.mozilla.org/en-US/docs/Web/Events/bufferedamountlow
[x] https://developer.mozilla.org/en-US/docs/Web/Events/close
[x] https://developer.mozilla.org/en-US/docs/Web/Events/datachannel
[x] https://developer.mozilla.org/en-US/docs/Web/Events/gatheringstatechange
[x] https://developer.mozilla.org/en-US/docs/Web/Events/icecandidate
[x] https://developer.mozilla.org/en-US/docs/Web/Events/iceconnectionstatechange
[x] https://developer.mozilla.org/en-US/docs/Web/Events/identityresult
[x] https://developer.mozilla.org/en-US/docs/Web/Events/idpassertionerror
[x] https://developer.mozilla.org/en-US/docs/Web/Events/idpvalidationerror
[x] https://developer.mozilla.org/en-US/docs/Web/Events/message
[x] https://developer.mozilla.org/en-US/docs/Web/Events/negotiationneeded
[x] https://developer.mozilla.org/en-US/docs/Web/Events/open
[x] https://developer.mozilla.org/en-US/docs/Web/Events/error (RTCDataChannel)
[x] https://developer.mozilla.org/en-US/docs/Web/Events/peeridentity
[x] https://developer.mozilla.org/en-US/docs/Web/Events/removestream
[x] https://developer.mozilla.org/en-US/docs/Web/Events/selectedcandidatepairchange
[x] https://developer.mozilla.org/en-US/docs/Web/Events/signalingstatechange
[x] https://developer.mozilla.org/en-US/docs/Web/Events/statechange
[x] https://developer.mozilla.org/en-US/docs/Web/Events/tonechange
[x] https://developer.mozilla.org/en-US/docs/Web/Events/track
[x] Add BCD entries for each event in RTCPeerConnection
[x] Add BCD entries for each event in RTCDataChannel
[x] Add BCD entries for each event in RTCIceTransport
[x] Add BCD entries for each event in RTCDTMFSender

@a2sheppy I've reviewed these pages! Looking really good mostly, with a few scraps to fix. I am impressed with the examples — they are really useful and interesting.
So, the niggles are as follows:

Some events were missing "event" from the end of the page title:

https://developer.mozilla.org/en-US/docs/Web/API/RTCPeerConnection/addstream_event
https://developer.mozilla.org/en-US/docs/Web/API/RTCPeerConnection/icecandidate_event
https://developer.mozilla.org/en-US/docs/Web/API/RTCPeerConnection/negotiationneeded_event
https://developer.mozilla.org/en-US/docs/Web/API/RTCPeerConnection/removestream_event
https://developer.mozilla.org/en-US/docs/Web/API/RTCIceTransport/selectedcandidatepairchange_event
https://developer.mozilla.org/en-US/docs/Web/API/RTCPeerConnection/signalingstatechange_event
https://developer.mozilla.org/en-US/docs/Web/API/RTCIceTransport/statechange_event
https://developer.mozilla.org/en-US/docs/Web/API/RTCDTMFSender/tonechange_event

the old close event page (https://developer.mozilla.org/en-US/docs/Web/Events/close) redirects to https://developer.mozilla.org/en-US/docs/Web/Events/close, which links to the RTCDatachannel.close() page for now; shouldn't it link to https://developer.mozilla.org/en-US/docs/Web/API/RTCDataChannel/close_event?

In the blue information box, the "Event handler" heading should be "Event handler property". You've got it as "Event handler" on most of them.

In the blue information box, you need to remove the "Target objects" row, as it is not needed now the event page is nicely under the object it is targetting. (In the case of multiple target objects, you should state the others in the "See also" links). You need to remove it from:

https://developer.mozilla.org/en-US/docs/Web/API/RTCPeerConnection/addstream_event
https://developer.mozilla.org/en-US/docs/Web/API/RTCPeerConnection/icecandidate_event
https://developer.mozilla.org/en-US/docs/Web/API/RTCPeerConnection/negotiationneeded_event
https://developer.mozilla.org/en-US/docs/Web/API/RTCPeerConnection/removestream_event
https://developer.mozilla.org/en-US/docs/Web/API/RTCPeerConnection/signalingstatechange_event
https://developer.mozilla.org/en-US/docs/Web/API/RTCIceTransport/statechange_event
https://developer.mozilla.org/en-US/docs/Web/API/RTCDTMFSender/tonechange_event

In the "Examples" section, the following pages show the event handler property version first, and then the addEventListener() version. It is the other way around in all other pages.

https://developer.mozilla.org/en-US/docs/Web/API/RTCIceTransport/gatheringstatechange_event
https://developer.mozilla.org/en-US/docs/Web/API/RTCPeerConnection/icecandidate_event
https://developer.mozilla.org/en-US/docs/Web/API/RTCPeerConnection/iceconnectionstatechange_event
https://developer.mozilla.org/en-US/docs/Web/API/RTCDataChannel/open_event
https://developer.mozilla.org/en-US/docs/Web/API/RTCIceTransport/selectedcandidatepairchange_event
https://developer.mozilla.org/en-US/docs/Web/API/RTCDTMFSender/tonechange_event
I'm not sure if this is a huge problem really, as long as both are shown. But ideally we should be consistent, and show the addEventListener() version first

https://developer.mozilla.org/en-US/docs/Web/API/RTCIceTransport/statechange_event has no example at all.

These pages are still using the old-style compat tables, so will need the new macro call added:

https://developer.mozilla.org/en-US/docs/Web/API/RTCPeerConnection/icecandidate_event
https://developer.mozilla.org/en-US/docs/Web/API/RTCPeerConnection/negotiationneeded_event
https://developer.mozilla.org/en-US/docs/Web/API/RTCPeerConnection/removestream_event
https://developer.mozilla.org/en-US/docs/Web/API/RTCIceTransport/selectedcandidatepairchange_event
https://developer.mozilla.org/en-US/docs/Web/API/RTCPeerConnection/signalingstatechange_event
https://developer.mozilla.org/en-US/docs/Web/API/RTCIceTransport/statechange_event
https://developer.mozilla.org/en-US/docs/Web/API/RTCDTMFSender/tonechange_event

@a2sheppy one more thing I noticed; in your interface page "Events" sections, you've written the event descriptions in a different style to the style that was chosen during @Elchi3 's research phase.
The event descriptions need to be updated to match the chosen style, and you need to also include the "Listen to these events using..." paragraph at the start of each section.
See https://developer.mozilla.org/en-US/docs/Web/API/Window#Events for the required styles.

@a2sheppy I've reviewed these pages! Looking really good mostly, with a few scraps to fix. I am impressed with the examples — they are really useful and interesting.
So, the niggles are as follows:
1. Some events were missing "event" from the end of the page title:

All fixed.

1. the old close event page (https://developer.mozilla.org/en-US/docs/Web/Events/close) redirects to https://developer.mozilla.org/en-US/docs/Web/Events/close, which links to the RTCDatachannel.close() page for now; shouldn't it link to https://developer.mozilla.org/en-US/docs/Web/API/RTCDataChannel/close_event?

Fixed.

2. In the blue information box, the "Event handler" heading should be "Event handler property". You've got it as "Event handler" on most of them.

Yeah, it changed in the template sometime after I started my work, and my copy and pasting was from one of my pages to another mostly, so I didn't catch the change. Fixed.

3. In the blue information box, you need to remove the "Target objects" row, as it is not needed now the event page is nicely under the object it is targetting. (In the case of multiple target objects, you should state the others in the "See also" links). You need to remove it from:

Fixed.

1. In the "Examples" section, the following pages show the event handler property version first, and then the addEventListener() version. It is the other way around in all other pages.

I'm not sure if this is a huge problem really, as long as both are shown. But ideally we should be consistent, and show the addEventListener() version first

Huh. OK. I shuffled them around intentionally, just to make the pages less monotonous.

1. https://developer.mozilla.org/en-US/docs/Web/API/RTCIceTransport/statechange_event has no example at all.

Fixed.

2. These pages are still using the old-style compat tables, so will need the new macro call added:

I looked at these and they all had current style compat tables when I looked at them just now...

https://developer.mozilla.org/en-US/docs/Web/API/RTCPeerConnection/icecandidate_event
https://developer.mozilla.org/en-US/docs/Web/API/RTCPeerConnection/negotiationneeded_event
https://developer.mozilla.org/en-US/docs/Web/API/RTCPeerConnection/removestream_event
https://developer.mozilla.org/en-US/docs/Web/API/RTCIceTransport/selectedcandidatepairchange_event
https://developer.mozilla.org/en-US/docs/Web/API/RTCPeerConnection/signalingstatechange_event
https://developer.mozilla.org/en-US/docs/Web/API/RTCIceTransport/statechange_event
https://developer.mozilla.org/en-US/docs/Web/API/RTCDTMFSender/tonechange_event

@a2sheppy one more thing I noticed; in your interface page "Events" sections, you've written the event descriptions in a different style to the style that was chosen during @Elchi3 's research phase.
The event descriptions need to be updated to match the chosen style, and you need to also include the "Listen to these events using..." paragraph at the start of each section.
See https://developer.mozilla.org/en-US/docs/Web/API/Window#Events for the required styles.

I was not aware that we were required to write these using the exact same format as this. I understood it to mean that we just had to include the same information, which I do. I find this format limiting and I am hesitant to redo all my event lists to chop them down to this form unless it really is required. Repetition of that type invites people's eyes to miss things, IMO.

Huh. OK. I shuffled them around intentionally, just to make the pages less monotonous.

OK, fair enough. I don't think this is a big deal.

I looked at these and they all had current style compat tables when I looked at them just now...

Ah, OK. Maybe they weren't refreshed when I looked at them ;-)

I was not aware that we were required to write these using the exact same format as this. I understood it to mean that we just had to include the same information, which I do. I find this format limiting and I am hesitant to redo all my event lists to chop them down to this form unless it really is required. Repetition of that type invites people's eyes to miss things, IMO.

Hrm, I dunno. I appreciate what you are saying, but I feel that the way you've written it sounds almost a bit biased towards using on... event handler properties. You don't mention addEventListener at all. Yes, you could argue this is implied by virtue of what it is, but this is not very helpful for less experienced devs.

Huh. I don't read it that way at all, but if you do, then I'm sure others
will as well. I guess I can live with redoing them to match the other
style, then. I'll do that today.
On Tue, Mar 5, 2019 at 3:03 AM Chris Mills<EMAIL_ADDRESS>wrote:

Huh. OK. I shuffled them around intentionally, just to make the pages less
monotonous.
OK, fair enough. I don't think this is a big deal.
I looked at these and they all had current style compat tables when I
looked at them just now...
Ah, OK. Maybe they weren't refreshed when I looked at them ;-)
I was not aware that we were required to write these using the exact same
format as this. I understood it to mean that we just had to include the
same information, which I do. I find this format limiting and I am hesitant
to redo all my event lists to chop them down to this form unless it really
is required. Repetition of that type invites people's eyes to miss things,
IMO.
Hrm, I dunno. I appreciate what you are saying, but I feel that the way
you've written it sounds almost a bit biased towards using on... event
handler properties. You don't mention addEventListener at all. Yes, you
could argue this is implied by virtue of what it is, but this is not very
helpful for less experienced devs.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/mdn/sprints/issues/952#issuecomment-469579450, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ABkL35EUPNmEl6HyI3UQAwEYlxhPXMBwks5vTiTGgaJpZM4alNBF
.

--
Eric Shepherd
Senior Technical Writer
Mozilla
Blog: http://www.bitstampede.com/
Twitter: http://twitter.com/sheppy

I've finished revising the event blurbs on the overview pages. Hopefully
this is all finished now.
On Tue, Mar 5, 2019 at 9:43 AM Eric Shepherd (Sheppy<EMAIL_ADDRESS>wrote:

Huh. I don't read it that way at all, but if you do, then I'm sure others
will as well. I guess I can live with redoing them to match the other
style, then. I'll do that today.
On Tue, Mar 5, 2019 at 3:03 AM Chris Mills<EMAIL_ADDRESS>wrote:

Huh. OK. I shuffled them around intentionally, just to make the pages
less monotonous.
OK, fair enough. I don't think this is a big deal.
I looked at these and they all had current style compat tables when I
looked at them just now...
Ah, OK. Maybe they weren't refreshed when I looked at them ;-)
I was not aware that we were required to write these using the exact same
format as this. I understood it to mean that we just had to include the
same information, which I do. I find this format limiting and I am hesitant
to redo all my event lists to chop them down to this form unless it really
is required. Repetition of that type invites people's eyes to miss things,
IMO.
Hrm, I dunno. I appreciate what you are saying, but I feel that the way
you've written it sounds almost a bit biased towards using on... event
handler properties. You don't mention addEventListener at all. Yes, you
could argue this is implied by virtue of what it is, but this is not very
helpful for less experienced devs.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/mdn/sprints/issues/952#issuecomment-469579450, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ABkL35EUPNmEl6HyI3UQAwEYlxhPXMBwks5vTiTGgaJpZM4alNBF
.

--
Eric Shepherd
Senior Technical Writer
Mozilla
Blog: http://www.bitstampede.com/
Twitter: http://twitter.com/sheppy

--
Eric Shepherd
Senior Technical Writer
Mozilla
Blog: http://www.bitstampede.com/
Twitter: http://twitter.com/sheppy

Yup, looks great; cheers mate.


initial file uploads implementation according to #77

Closes #77.
TODO:

[x] Send success message as a reply in local rmb (failure is handled) and send a local message command in remote rmb

Can be done later:

HttpApi needs a builder because it takes too many parameters now.

To be fixed:

The file should have a unique name (currently it can be overwritten by any new upload with the same file name).


Task Description is missing on the task page while available via /guide

Actions before raising this issue

[X] I searched the existing issues and did not find anything similar.
[X] I read/searched the docs

Is your feature request related to a problem? Please describe.
Having task description is a nice to have feature that I spotted in the cloud version of CVAT 2.10 (it might have been available before). This feature is not disclosed in the docs
https://opencv.github.io/cvat/docs/manual/basics/task-details/
Unfortunately, the community on-prem version has a slightly different behavior: the task description is not available from the task page, while one can open the editing interface via /guide endpoint.
This is confusing. Obviously it would be great having this feature available.
Describe the solution you'd like
It would be useful accessing and seing the descriptions through the web interface.
Getting them via CLI is relevant as well.
Describe alternatives you've considered
No response
Additional context
On cloud (at the time of writing)
Server version: 2.10.2
Core version: 14.1.0
Canvas version: 2.19.1
UI version: 1.61.3
On prem
Server version: 2.10.3
Core version: 14.1.1
Canvas version: 2.19.1
UI version: 1.61.4

Few more details
The data submitted via /guide UI successfully land in the engine_annotationguide table. However, there is no other option other than using API to get it
https://opencv.github.io/cvat/docs/api_sdk/sdk/reference/apis/guides-api/

There are not any restrictions of this feature in open source solution.
It is available, please, re-check.

No restriction would be the expected behaviour.
Unfortunately this is not the case: double-checked, please see the screenshots

The Task description button is available in the cloud version

There is no task decription in Task interface

The guide endpoint is still available

I see. The same scheme as with labels.
If a task in a project, setup guide on the project page.
If a task not in a project, setup guide on the task page.

Ok. Understood. Thanks.
I would like to be able to customize the description task-wise. Any thoughts of that being useful?


Synthesis failed for nodejs-cloud-container

Hello! Autosynth couldn't regenerate nodejs-cloud-container. :broken_heart:
Here's the output from running synth.py:
/.cache/synthtool/googleapis/WORKSPACE:42:1
DEBUG: Rule 'gapic_generator_python' indicated that a canonical reproducible form can be obtained by modifying arguments sha256 = "8d11f06b408ac5f1c01da3ca17f3a75dc008831509c5c1a4f24f9bde37792a57"
DEBUG: Call stack for the definition of repository 'gapic_generator_python' which is a http_archive (rule definition at /home/kbuilder/.cache/bazel/_bazel_kbuilder/a732f932c2cbeb7e37e1543f189a2a73/external/bazel_tools/tools/build_defs/repo/http.bzl:296:16):
 - <builtin>
 - /home/kbuilder/.cache/synthtool/googleapis/WORKSPACE:224:1
DEBUG: Rule 'com_googleapis_gapic_generator_go' indicated that a canonical reproducible form can be obtained by modifying arguments sha256 = "12bfed7873f085093cd60615bd113178ecf36396af0c2ca25e6cd4d4bebdd198"
DEBUG: Call stack for the definition of repository 'com_googleapis_gapic_generator_go' which is a http_archive (rule definition at /home/kbuilder/.cache/bazel/_bazel_kbuilder/a732f932c2cbeb7e37e1543f189a2a73/external/bazel_tools/tools/build_defs/repo/http.bzl:296:16):
 - <builtin>
 - /home/kbuilder/.cache/synthtool/googleapis/WORKSPACE:261:1
DEBUG: Rule 'gapic_generator_typescript' indicated that a canonical reproducible form can be obtained by modifying arguments sha256 = "ca322b5e7b0d03b3cc44a90444e3a7f944c9ba3345f0505ee48c8e715d19dd95"
DEBUG: Call stack for the definition of repository 'gapic_generator_typescript' which is a http_archive (rule definition at /home/kbuilder/.cache/bazel/_bazel_kbuilder/a732f932c2cbeb7e37e1543f189a2a73/external/bazel_tools/tools/build_defs/repo/http.bzl:296:16):
 - <builtin>
 - /home/kbuilder/.cache/synthtool/googleapis/WORKSPACE:280:1
DEBUG: Rule 'gapic_generator_csharp' indicated that a canonical reproducible form can be obtained by modifying arguments sha256 = "40ddae63d2729ef5ccbd8b60123327ea200ce9400d0629238193ff530dcaea18"
DEBUG: Call stack for the definition of repository 'gapic_generator_csharp' which is a http_archive (rule definition at /home/kbuilder/.cache/bazel/_bazel_kbuilder/a732f932c2cbeb7e37e1543f189a2a73/external/bazel_tools/tools/build_defs/repo/http.bzl:296:16):
 - <builtin>
 - /home/kbuilder/.cache/synthtool/googleapis/WORKSPACE:318:1
DEBUG: Rule 'bazel_skylib' indicated that a canonical reproducible form can be obtained by modifying arguments sha256 = "1dde365491125a3db70731e25658dfdd3bc5dbdfd11b840b3e987ecf043c7ca0"
DEBUG: Call stack for the definition of repository 'bazel_skylib' which is a http_archive (rule definition at /home/kbuilder/.cache/bazel/_bazel_kbuilder/a732f932c2cbeb7e37e1543f189a2a73/external/bazel_tools/tools/build_defs/repo/http.bzl:296:16):
 - <builtin>
 - /home/kbuilder/.cache/synthtool/googleapis/WORKSPACE:35:1
Analyzing: target //google/container/v1:container-v1-nodejs (1 packages loaded, 0 targets configured)
INFO: Call stack for the definition of repository 'npm' which is a yarn_install (rule definition at /home/kbuilder/.cache/bazel/_bazel_kbuilder/a732f932c2cbeb7e37e1543f189a2a73/external/build_bazel_rules_nodejs/internal/npm_install/npm_install.bzl:411:16):
 - <builtin>
 - /home/kbuilder/.cache/bazel/_bazel_kbuilder/a732f932c2cbeb7e37e1543f189a2a73/external/build_bazel_rules_nodejs/index.bzl:87:5
 - /home/kbuilder/.cache/synthtool/googleapis/WORKSPACE:293:1
Analyzing: target //google/container/v1:container-v1-nodejs (1 packages loaded, 2 targets configured)
ERROR: An error occurred during the fetch of repository 'npm':
   yarn_install failed: yarn install v1.19.1
[1/5] Validating package.json...
[2/5] Resolving packages...
[3/5] Fetching packages...
info If you think this is a bug, please open a bug report with the information provided in "/home/kbuilder/.cache/bazel/_bazel_kbuilder/a732f932c2cbeb7e37e1543f189a2a73/external/gapic_generator_typescript/yarn-error.log".
info Visit https://yarnpkg.com/en/docs/cli/install for documentation about this command.
 (error An unexpected error occurred<EMAIL_ADDRESS>Request failed \"404 Not Found\"".
)
ERROR: /home/kbuilder/.cache/synthtool/googleapis/google/container/v1/BUILD.bazel:218:1: //google/container/v1:container_nodejs_gapic depends on @gapic_generator_typescript//:protoc_plugin in repository @gapic_generator_typescript which failed to fetch. no such package '@npm//@bazel/typescript': yarn_install failed: yarn install v1.19.1
[1/5] Validating package.json...
[2/5] Resolving packages...
[3/5] Fetching packages...
info If you think this is a bug, please open a bug report with the information provided in "/home/kbuilder/.cache/bazel/_bazel_kbuilder/a732f932c2cbeb7e37e1543f189a2a73/external/gapic_generator_typescript/yarn-error.log".
info Visit https://yarnpkg.com/en/docs/cli/install for documentation about this command.
 (error An unexpected error occurred<EMAIL_ADDRESS>Request failed \"404 Not Found\"".
)
ERROR: Analysis of target '//google/container/v1:container-v1-nodejs' failed; build aborted: no such package '@npm//@bazel/typescript': yarn_install failed: yarn install v1.19.1
[1/5] Validating package.json...
[2/5] Resolving packages...
[3/5] Fetching packages...
info If you think this is a bug, please open a bug report with the information provided in "/home/kbuilder/.cache/bazel/_bazel_kbuilder/a732f932c2cbeb7e37e1543f189a2a73/external/gapic_generator_typescript/yarn-error.log".
info Visit https://yarnpkg.com/en/docs/cli/install for documentation about this command.
 (error An unexpected error occurred<EMAIL_ADDRESS>Request failed \"404 Not Found\"".
)
INFO: Elapsed time: 1.221s
INFO: 0 processes.

DEBUG:synthtool:Wrote metadata to synth.metadata.
Traceback (most recent call last):
  File "/home/kbuilder/.pyenv/versions/3.6.9/lib/python3.6/runpy.py", line 193, in _run_module_as_main
    "__main__", mod_spec)
  File "/home/kbuilder/.pyenv/versions/3.6.9/lib/python3.6/runpy.py", line 85, in _run_code
    exec(code, run_globals)
  File "/tmpfs/src/github/synthtool/synthtool/__main__.py", line 102, in <module>
    main()
  File "/tmpfs/src/github/synthtool/env/lib/python3.6/site-packages/click/core.py", line 829, in __call__
    return self.main(*args, **kwargs)
  File "/tmpfs/src/github/synthtool/env/lib/python3.6/site-packages/click/core.py", line 782, in main
    rv = self.invoke(ctx)
  File "/tmpfs/src/github/synthtool/env/lib/python3.6/site-packages/click/core.py", line 1066, in invoke
    return ctx.invoke(self.callback, **ctx.params)
  File "/tmpfs/src/github/synthtool/env/lib/python3.6/site-packages/click/core.py", line 610, in invoke
    return callback(*args, **kwargs)
  File "/tmpfs/src/github/synthtool/synthtool/__main__.py", line 94, in main
    spec.loader.exec_module(synth_module)  # type: ignore
  File "<frozen importlib._bootstrap_external>", line 678, in exec_module
  File "<frozen importlib._bootstrap>", line 219, in _call_with_frames_removed
  File "/home/kbuilder/.cache/synthtool/nodejs-cloud-container/synth.py", line 13, in <module>
    library = gapic.node_library('container', version, proto_path=f'google/container/{version}')
  File "/tmpfs/src/github/synthtool/synthtool/gcp/gapic_bazel.py", line 52, in node_library
    return self._generate_code(service, version, "nodejs", **kwargs)
  File "/tmpfs/src/github/synthtool/synthtool/gcp/gapic_bazel.py", line 183, in _generate_code
    shell.run(bazel_run_args)
  File "/tmpfs/src/github/synthtool/synthtool/shell.py", line 39, in run
    raise exc
  File "/tmpfs/src/github/synthtool/synthtool/shell.py", line 33, in run
    encoding="utf-8",
  File "/home/kbuilder/.pyenv/versions/3.6.9/lib/python3.6/subprocess.py", line 438, in run
    output=stdout, stderr=stderr)
subprocess.CalledProcessError: Command '['bazel', '--max_idle_secs=240', 'build', '//google/container/v1:container-v1-nodejs']' returned non-zero exit status 1.
2020-08-06 04:12:48,839 autosynth [ERROR] > Synthesis failed
2020-08-06 04:12:48,839 autosynth [DEBUG] > Running: git reset --hard HEAD
HEAD is now at a2f27ef docs: add links to the CHANGELOG from the README.md for Java and Node (#373)
2020-08-06 04:12:48,844 autosynth [DEBUG] > Running: git checkout autosynth
Switched to branch 'autosynth'
2020-08-06 04:12:48,849 autosynth [DEBUG] > Running: git clean -fdx
Removing __pycache__/
Traceback (most recent call last):
  File "/home/kbuilder/.pyenv/versions/3.6.9/lib/python3.6/runpy.py", line 193, in _run_module_as_main
    "__main__", mod_spec)
  File "/home/kbuilder/.pyenv/versions/3.6.9/lib/python3.6/runpy.py", line 85, in _run_code
    exec(code, run_globals)
  File "/tmpfs/src/github/synthtool/autosynth/synth.py", line 690, in <module>
    main()
  File "/tmpfs/src/github/synthtool/autosynth/synth.py", line 539, in main
    return _inner_main(temp_dir)
  File "/tmpfs/src/github/synthtool/autosynth/synth.py", line 670, in _inner_main
    commit_count = synthesize_loop(x, multiple_prs, change_pusher, synthesizer)
  File "/tmpfs/src/github/synthtool/autosynth/synth.py", line 375, in synthesize_loop
    has_changes = toolbox.synthesize_version_in_new_branch(synthesizer, youngest)
  File "/tmpfs/src/github/synthtool/autosynth/synth.py", line 273, in synthesize_version_in_new_branch
    synthesizer.synthesize(synth_log_path, self.environ)
  File "/tmpfs/src/github/synthtool/autosynth/synthesizer.py", line 120, in synthesize
    synth_proc.check_returncode()  # Raise an exception.
  File "/home/kbuilder/.pyenv/versions/3.6.9/lib/python3.6/subprocess.py", line 389, in check_returncode
    self.stderr)
subprocess.CalledProcessError: Command '['/tmpfs/src/github/synthtool/env/bin/python3', '-m', 'synthtool', '--metadata', 'synth.metadata', 'synth.py', '--']' returned non-zero exit status 1.

Google internal developers can see the full log here.

Autosynth passed, closing! :green_heart:


docker-machine ssh behaves differently to documented

Following the documentation on https://docs.docker.com/machine/#ssh with the download provided on the same page, running on OSX 10.10.
docker-machine -v
docker-machine version 0.1.0

docker-machine ls
NAME   ACTIVE   DRIVER         STATE     URL
dev             virtualbox     Running   tcp://...
do     *        digitalocean   Running   tcp://...

The documentation appears to suggest the following should work:
docker-machine ssh uptime
FATA[0000] Host "uptime" does not exist

The following example using the -- separator (straight from the documentation) also doesn't work
docker-machine ssh dev -- df -h
FATA[0000] Host "df" does not exist

Passing the name of the machine, followed by the command simply opens an ssh session without running the command.
docker-machine ssh dev uptime
                        ##         .
                  ## ## ##        ==
               ## ## ## ## ##    ===
           /"""""""""""""""""\___/ ===
      ~~~ {~~ ~~~~ ~~~ ~~~~ ~~~ ~ /  ===- ~~~
           \______ o           __/
             \    \         __/
              \____\_______/
 _                 _   ____     _            _
| |__   ___   ___ | |_|___ \ __| | ___   ___| | _____ _ __
| '_ \ / _ \ / _ \| __| __) / _` |/ _ \ / __| |/ / _ \ '__|
| |_) | (_) | (_) | |_ / __/ (_| | (_) | (__|   <  __/ |
|_.__/ \___/ \___/ \__|_____\__,_|\___/ \___|_|\_\___|_|
Boot2Docker version 1.6.2, build master : 4534e65 - Wed May 13 21:24:28 UTC 2015
Docker version 1.6.2, build 7c8fca2

Note that the option -c does the correct thing. This is documented in the man page but not in the site documentation.
docker-machine ssh do -c free                                                                                /tmp
             total       used       free     shared    buffers     cached
Mem:        501800     446144      55656        340      14540     375540

I'm not sure whether the documentation or the implementation is correct. Also worth noting the man page documentation differs from the site documentation a little as well.

Ah, for some reason I ended up using 0.1.0. Testing with 0.3.0-rc2 this works as expected and the documentation has been updated. Closing.


Jupyter becomes unresponsive on remote:WSL

"Starting Jupyter Server" And "executing cell" appear in the status bar. But no response.

Error Code in the Toggle developer tool:
ErrorInfo.txt

but work good in host system.

This is a known issue. It's the connection between your machine and the remote machine attempting to download the code for our notebook viewer.
https://github.com/microsoft/vscode-python/issues/7874


Buttons should not have fixed width and height

We should use min-width and min-height over width and height.

We are not using width, but we are using height. Will set it to be a min height and update the border-radius accordingly to be twice as big as the min hight to avoid weird shapes on buttons whose height grows due to long text and a fixed width

Fixed by #25


E3002, E3003, E3012 - AWS::SSM::MaintenanceWindowTarget errors

cfn-lint version: 0.19.1
cfn-lint -u has been run
I am unable to to get a clean lint on a SSM template file that seems alright and that has loaded properly in AWS.  Would really appreciate any assistance.
The documentation is here on MaintenanceWindowTarget:
https://docs.aws.amazon.com/AWSCloudFormation/latest/UserGuide/aws-resource-ssm-maintenancewindowtarget.html
And the associated targets:
https://docs.aws.amazon.com/AWSCloudFormation/latest/UserGuide/aws-properties-ssm-maintenancewindowtarget-targets.html
The below cf should be syntactically correct.....
  BaselinePatchDailySSMMaintenanceWindowTarget:
    Type: AWS::SSM::MaintenanceWindowTarget
    Properties:
      Name: BaselinePatchDailyTarget
      Description: Systems with Tag Key=Patch,Value=Daily.
      WindowId: !Ref BaselinePatchDailySSMMaintenanceWindow
      ResourceType: INSTANCE
      Targets:
      - Key: tag:Patch
        Values:
        - Daily
        - daily

Results in the following cfn-lint errors:
E0002 Unknown exception while processing rule E3002: 'AWS::SSM::MaintenanceWindowTarget.Target'
BaselineSSMConfig.yaml:1:1

E0002 Unknown exception while processing rule E3003: 'AWS::SSM::MaintenanceWindowTarget.Target'
BaselineSSMConfig.yaml:1:1

E3012 Property Resources/BaselinePatchDailySSMMaintenanceWindowTarget/Properties/Targets/0 should be of type String
BaselineSSMConfig.yaml:292:9

Edited file as follows to attempt to make target a string:
  BaselinePatchDailySSMMaintenanceWindowTarget:
    Type: AWS::SSM::MaintenanceWindowTarget
    Properties:
      Name: BaselinePatchDailyTarget
      Description: Systems with Tag Key=Patch,Value=Daily.
      WindowId: !Ref BaselinePatchDailySSMMaintenanceWindow
      ResourceType: INSTANCE
      Targets: Key=tag:Patch,Values=Daily,daily

Results in the following cfn-lint error:
E3002 Property Targets should be of type List for resource BaselinePatchDailySSMMaintenanceWindowTarget
BaselineSSMConfig.yaml:291:7

Attempting to make the string a list:
  BaselinePatchDailySSMMaintenanceWindowTarget:
    Type: AWS::SSM::MaintenanceWindowTarget
    Properties:
      Name: BaselinePatchDailyTarget
      Description: Systems with Tag Key=Patch,Value=Daily.
      WindowId: !Ref BaselinePatchDailySSMMaintenanceWindow
      ResourceType: INSTANCE
      Targets:
      - Key=tag:Patch,Values=Daily,daily

Results in the following errors:
E0002 Unknown exception while processing rule E3002: 'AWS::SSM::MaintenanceWindowTarget.Target'
BaselineSSMConfig.yaml:1:1

E0002 Unknown exception while processing rule E3003: 'AWS::SSM::MaintenanceWindowTarget.Target'
BaselineSSMConfig.yaml:1:1

Thanks!

Thank you for patching so quickly!    They are now E3002, and E3012.
The original cf that should be correct:
  BaselinePatchDailySSMMaintenanceWindowTarget:
    Type: AWS::SSM::MaintenanceWindowTarget
    Properties:
      Name: BaselinePatchDailyTarget
      Description: Systems with Tag Key=Patch,Value=Daily.
      WindowId: !Ref BaselinePatchDailySSMMaintenanceWindow
      ResourceType: INSTANCE
      Targets:
      - Key: tag:Patch
        Values:
        - Daily
        - daily

Still has one of the errors from before:
E3012 Property Resources/BaselinePatchDailySSMMaintenanceWindowTarget/Properties/Targets/0 should be of type String
BaselineSSMConfig.yaml:292:9

Changing that line to a string format as it asks:
  BaselinePatchDailySSMMaintenanceWindowTarget:
    Type: AWS::SSM::MaintenanceWindowTarget
    Properties:
      Name: BaselinePatchDailyTarget
      Description: Systems with Tag Key=Patch,Value=Daily.
      WindowId: !Ref BaselinePatchDailySSMMaintenanceWindow
      ResourceType: INSTANCE
      Targets: Key=tag:Patch,Values=Daily,daily

Leads to the same error as before:
E3002 Property Targets should be of type List for resource BaselinePatchDailySSMMaintenanceWindowTarget
BaselineSSMConfig.yaml:295:7

Note it was a list originally.  Changing the string to a list:
  BaselinePatchDailySSMMaintenanceWindowTarget:
    Type: AWS::SSM::MaintenanceWindowTarget
    Properties:
      Name: BaselinePatchDailyTarget
      Description: Systems with Tag Key=Patch,Value=Daily.
      WindowId: !Ref BaselinePatchDailySSMMaintenanceWindow
      ResourceType: INSTANCE
      Targets:
      - Key=tag:Patch,Values=Daily,daily

Has the following different from before error:
E3002 Expecting an object at Resources/BaselinePatchDailySSMMaintenanceWindowTarget/Properties/Targets/0
BaselineSSMConfig.yaml:296:9

Looking into this.

ok, I think I got what my issues are.  It may take a day or two to get a fix fully in if I don't run into anything major.  They did something new in the specs that we hadn't accounted for.

Thanks for your help!

I'm sure you tried a few of these options too (against CloudFormation)
This works.
Targets:
      - Key: tag:Patch
        Values:
        - Daily
        - daily

This does not even though the documentation says it should
Targets:
      - Key=tag:Patch,Values=Daily,daily

Hmm.  Both of those are coming up with the same errors as earlier for me.  The first one that you mention as working is saying:
'''
E3012 Property Resources/BaselinePatchDailySSMMaintenanceWindowTarget/Properties/Targets/0 should be of type String
BaselineSSMConfig.yaml:292:9
'''
Second one is still:
E3002 Expecting an object at Resources/BaselinePatchDailySSMMaintenanceWindowTarget/Properties/Targets/0
BaselineSSMConfig.yaml:292:9

I haven't been able to get any clean runs.  Just uninstalled the module and resinstalled in case there was some leftover cruft as well.

sorry, I meant I ran the string approaches directly against CloudFormation.  If you agree with this I will just remove the PrimitiveType from the spec and we will only allow for the object approach.
This
Targets:
      - Key=tag:Patch,Values=Daily,daily

results in
Property validation failure: [Value of property {/Targets/0} does not match type {Object}]
This
Targets: Key=tag:Patch,Values=Daily,daily

Property validation failure: [Value of property {/Targets} does not match type {Array}]

Doh! I misread.... I agree with this approach you have here. Thank you!


Author V3 microsite plan

As discussed today, let's work on a plan for how we're going to refresh the site at workboxjs.org for the V3 launch. It would be great to include more examples, be more beginner friendly and ease navigation.

See #102


refactor: update eslint rules and husky

any attached jira ticket?

any attached jira ticket?

Part of https://github.com/lifinance/data-types/pull/54


Add link in README to issues tagged as Question

Hi @dilijev, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

It looks like you're a Microsoft contributor (Doug Ilijev). If you're full-time, we DON'T require a Contribution License Agreement. If you are a vendor, please DO sign the electronic Contribution License Agreement. It will take 2 minutes and there's no faxing! https://cla.microsoft.com.

TTYL, MSBOT;


[Spark-14283][SQL] Add a option to avoid local sort in randomSplit

What changes were proposed in this pull request?
Add a option to avoid local sort in randomSplit.
Since randomSplit is used a fair amount in ML, it would be great to avoid the sort when possible.
How was this patch tested?
manual tests

Test build #56047 has started for PR 12453 at commit abbf4ce.

Test build #56047 has finished for PR 12453 at commit abbf4ce.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/56047/
Test FAILed.

Jenkins, test this please

Test build #56050 has started for PR 12453 at commit abbf4ce.

Test build #56050 has finished for PR 12453 at commit abbf4ce.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/56050/
Test FAILed.

Shouldn't the user-facing documentation explain the consequences of omitting this sort, especially its impact on correctness?

I misunderstand the JIRA. I will colse this PR


Add a sidetable cache in the module for VTable entries, replacing linear lookup

radar rdar://problem/28311199
This commits resolves a FIXME in SILVTable::getImplementation: Building a sidetable cache in the module for function lookup.
That, in turn, drastically reduces Swift's compilation time in programs with large VTables.

@swift-ci please smoke test

@eeckstein Can you please review?

@shajrawi Joe, do we also need the same optimization for witness tables?

@swiftix As far as I can tell we do not: SIL Modules have DenseMap(s) for witness table lookups, Witness table itself doesn't do any linear lookups and I don't see this as an issue in 'instruments'

@swift-ci please smoke test

@swift-ci please smoke test

@eeckstein requested changes / SILVerifier add-on are in amended PR

@swift-ci please smoke test

@swift-ci please smoke test

Very nice @shajrawi !

@swift-ci please smoke test

@swift-ci please smoke test

Looks good! = D

@swift-ci please smoke test

@swift-ci please smoke test


video ads

hello, thanks for the great work, which i've been using for a long time now.
i know you said that video ads cannot be muted.
the thing is, now that video ads have become pervasive (one every 10 minutes or so), and because they are particularly loud and nasty, spotifree becomes useless if we cannot mute them.
is it absolutely sure that there will never be a way to mute them?
thanks!!

Sorry, but Spotify makes it pretty hard to detect and mute video ads, and I still haven't figured out a way to do it. Maybe I would, but I've been finding it increasingly difficult to carve out time to work on Spotifree.


Fixing documentation issue #591

Deploying Typescript Chaincodes Fail using DeployCC.sh In CLI Container for Test Network
#591

Type of change

Documentation update

Description
In order to make the documentation easier for the user to read . I made a change a change to
documentation, to notify the reader to install the required compiler before going any feature with
documentation in order to facilitate less chances of errors

Additional details

Related issues
https://github.com/hyperledger/fabric/issues/3212

@AyushRaj8271 you'll need to sign the commit to get the DCO check passed, see instructions at https://github.com/hyperledger/fabric/pull/3309/checks?check_run_id=5742167876

@mbwhite @denyeart the only language that doesn't have any pre-req installation of a compiler is javascript . All the others (go/java/typescript) will require compilers installed before you can deploy a chaincode (confirmed when moving caliper benchmarks to use test-network).

The change currently picks up 52 files for update. You'll need to rebase on latest main and re-push so that just your changes show up in the commit.

No response from contributor, closing.


Pass kms_key as part of KeyService call (merge to develop)

Replaced behavior of passing None in the encrypt and decrypt
functions.

LGTM


Add compatibility with errors-2.0.*, resolves #33

@jsl @jsantos17 Could you take a look, please?

@jsl could you turn off circleCI tests, please? We didn't put the tokens needed for the tests so they are likely to fail and we already have TravisCI running tests too

LGTM. CPP is always ugly, but seems like a fairly good solution here.

For the record. I'm merging it now with circleCI tests failing because the tokens needed for the tests are not set. On the other hand we only need travisCI tests passing, since the are authentication tokens are properly set.

Just curious, any way that we could do something like this to release to a separate library? @juanpaucar


Allow user to override app and probe default port

Why?

Just in case port 4040 and 4030 are already in use locally.
I think it came up in conversation when I was encouraging people to give
feedback, but I don't remember who mentioned it.
On Tuesday, 2 June 2015, Peter Bourgon<EMAIL_ADDRESS>wrote:

Why?
—
Reply to this email directly or view it on GitHub
https://github.com/weaveworks/scope/issues/153#issuecomment-108020700.

Gotcha. How would you change that, via the scope script?

With the scope script, any extra arguments are just passed to the
container's entrypoint.sh script.
I'm thinking in entrypoint.sh, munging any arguments starting with -probe.
into a file /etc/weave/probe-args, and similar for the app.  Then in
run-app and run-probe, read these files and tack them onto the command,
similar to how we handle having hosts passed right now.
What do you think?
On Wed, Jun 3, 2015 at 8:14 AM, Peter Bourgon<EMAIL_ADDRESS>wrote:

Gotcha. How would you change that, via the scope script?
—
Reply to this email directly or view it on GitHub
https://github.com/weaveworks/scope/issues/153#issuecomment-108224213.

I like the idea of using a flag prefix and parsing it in the scope script, but there's got to be a nicer way than dumping them to text files in the container and dumping them out again in the run scripts?

Depends on what runit supports. Dumping the flags to a file seems the
easiest to me.
On Wednesday, 3 June 2015, Peter Bourgon<EMAIL_ADDRESS>wrote:

I like the idea of using a flag prefix and parsing it in the scope script,
but there's got to be a nicer way than dumping them to text files in the
container and dumping them out again in the run scripts?
—
Reply to this email directly or view it on GitHub
https://github.com/weaveworks/scope/issues/153#issuecomment-108254997.

Definitely need to change UI dev server port from 3000 to 404x.
:3000 is just too common.

I guess I'm wondering if there's a better way to pass information through the host—Docker membrane. Environment variables? Probably too finicky. Yeah, maybe files are the least-bad option.

The command line is used for the "host—Docker membrane".  Its within the container, passing variables from entrypoint.sh to the run-foo scripts that requires the file.

Sorry, yes, that's what I meant.


Autscroll not working on smaller devices

I have tried testing it on some smartphones and auto scroll ddnt work. I also tried testing it on a regular tablet device, also didnt work on portrait orientation but somehow works if landscaped. I thought it was a device issue but out of it i tried testing it on a desktop web browser and adjusted the window size as small as a regular smartphone then i replicated the problem.

Thanks for leaving this issue. Can you narrow down to what actions you take are not working, and for which browsers?  The following example sties have several links (between documents no hashtag, same documents, no hashtag), and if you can narrow down a reproducible scenario (action, browser combo) for us I'd really appreciate it !

http://iron-router-autoscroll.meteor.com
http://flow-router-autoscroll.meteor.com

What i did is that i have this rows of boxed items that doesnt fit on a page so i have to scroll down the page to see the other items below. The box has an onclick event that routes to the full details of the clicked item. On that page theres a back button that links back to the previous route. Initialy when i press back the page doesnt scroll down to where it was before

So i applied router-autoscroll and somehow solved the issue. But when i tried testing it on phones it didnt work. Test it on tablets works only if lanscaped. I thought it was a browser issue but tried it on desktop both on firefox and chrome and adjusted the window size as small as a regular smartphone then i somehow replicated the problem.
So im guessing theres a limitation on screen dimension. I believe im not using hashtags.
I used iron-router for my routes. Test it on chrome browsers on phone. On app its chromium33.
P.S. I accidentally pressed the close and comment button . :)

This seems to be the same issue I was having on the old iron-router-autoscroll where when I browse on my iPhone Safari and Chrome, the auto scroll doesn't work. Probably those mobile browsers handle scrolling differently....

Hello all, at OK GROW we no longer use this package in our projects. We will no longer be dedicating the resources to maintaining this package. If you would like to support this package, feel free to fork it.


Update Analytics to capture clicks

Since links are technically external to the page, they are not captured. This allows us to know whether users actually clicked the download link, whether they visited our page, whether they opened the license file, or whether they clicked the feedback form link.

lgtm


Add what people should look out for after gulp ran

What about adding this: Look at the css and see that x is no longer in the css...

@avermeulen good catch. Thanks!

What about changing this:
For the files included in this example, your trimmed CSS file in should look like this:
To something like:
A new main.css file will be created in the ./out folder that will look like this:
// css here
Note that the unused css elements have been removed.

Mmm, good, me like. Have updated README.


[Bug] Connect-DbaInstance reports platform incorrectly when using a Linux Container

Environmental information
PowerShell version:
Name                           Value

PSVersion                      6.1.0
PSEdition                      Core
GitCommitId                    6.1.0
OS                             Darwin 18.7.0 Darwin Kernel Version 18.7.0: Thu Jun 20 18:42:21 PDT 2019; root:xnu-4903.270.47~4/RELEASE_X86_64
Platform                       Unix
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0...}
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>
WSManStackVersion              3.0
dbatools Module version:
Name    : dbatools
Path    : /usr/local/share/powershell/Modules/dbatools/1.0.38/dbatools.psd1
Version : 1.0.38
Name    : dbatools
Path    : /usr/local/share/powershell/Modules/dbatools/0.9.477/dbatools.psd1
Version : 0.9.477
SQL Server:
select @@VERSION
Microsoft SQL Server 2019 (RC1) - 15.0.1900.25 (X64) 	Aug 16 2019 14:20:53 	Copyright (C) 2019 Microsoft Corporation
Developer Edition (64-bit) on Linux (Ubuntu 16.04.6 LTS)
Report
Connect-DbaInstance when targeting a  Linux container reports NT x64...see output below.
Connect-DbaInstance -SqlInstance $RestoreServer -SqlCredential $RestoreCredential
Name           Product              Version   Platform IsAzure IsClustered ConnectedAs
----           -------              -------   -------- ------- ----------- -----------
X.X.X.X Microsoft SQL Server 15.0.1900 NT x64   False   False       sa

Host used

[ ] powershell.exe
[ ] ISE
[X] VS Code
[ ] Other (please specify)

Expected Behavior
Expected to report the platform as Linux rather than NT
Actual Behavior
Reports NT x64 as the platform.

Ah, has mistype in the code. Corrected and it returns information but is showing NT x64.

$loc = Get-InstalledModule -Name dbatools | select -first 1 -Property InstalledLocation
Add-Type -Path "$($loc.InstalledLocation)\bin\smo\coreclr\Microsoft.SqlServer.Smo.dll"
Add-Type -Path "$($loc.InstalledLocation)\bin\smo\coreclr\Microsoft.SqlServer.ConnectionInfo.dll"
Add-Type -Path "$($loc.InstalledLocation)\bin\smo\coreclr\Microsoft.SqlServer.Management.Sdk.Sfc.dll"
$servername = "localhost,1411"
$server = New-Object Microsoft.SqlServer.Management.Smo.Server $servername
$message = "Enter the SQL Login credentials for the SQL Server"
$server.ConnectionContext.LoginSecure = $false
$sqllogin = Get-Credential -Message $message
$server.ConnectionContext.set_Login($sqllogin.username)
$server.ConnectionContext.set_SecurePassword($sqllogin.Password)
$server | select-object HostPlatform, Platform

Ok, so I think this is by design and not a bug. The platform of the machine I'm on and running Docker is NT x64.

When I run that code @wsmelton here's what I get:
HostPlatform Platform
------------   --------
Linux               NT x64

Correct.  The host platform for SQL Server is Linux, but the platform you have Docker running is Windows.
I run that same code against a local instance on my Windows machine, the Platform shows NT x64 and HostPlatform changes to Windows.

Nope, I'm on a Mac running Docker for Mac.

You will need to upgrade to PowerShell 6.2.x and then install sqlserver module and run this to see if you get the same result:
$loc = Get-InstalledModule -Name sqlserver | select -first 1 -Property InstalledLocation
Add-Type -Path "$($loc.InstalledLocation)\coreclr\Microsoft.SqlServer.Smo.dll"
Add-Type -Path "$($loc.InstalledLocation)\coreclr\Microsoft.SqlServer.ConnectionInfo.dll"
Add-Type -Path "$($loc.InstalledLocation)\coreclr\Microsoft.SqlServer.Management.Sdk.Sfc.dll"
$servername = "localhost,1411"
$server = New-Object Microsoft.SqlServer.Management.Smo.Server $servername
$message = "Enter the SQL Login credentials for the SQL Server"
$server.ConnectionContext.LoginSecure = $false
$sqllogin = Get-Credential -Message $message
$server.ConnectionContext.set_Login($sqllogin.username)
$server.ConnectionContext.set_SecurePassword($sqllogin.Password)
$server | select-object HostPlatform, Platform

Still no dice...
PS /Users/aen> $loc = Get-InstalledModule -Name sqlserver | select -first 1 -Property InstalledLocation
PS /Users/aen> Add-Type -Path "$($loc.InstalledLocation)\coreclr\Microsoft.SqlServer.Smo.dll"
PS /Users/aen> Add-Type -Path "$($loc.InstalledLocation)\coreclr\Microsoft.SqlServer.ConnectionInfo.dll"
PS /Users/aen> Add-Type -Path "$($loc.InstalledLocation)\coreclr\Microsoft.SqlServer.Management.Sdk.Sfc.dll"
PS /Users/aen> $servername = "localhost,1433"
PS /Users/aen> $server = New-Object Microsoft.SqlServer.Management.Smo.Server $servername
PS /Users/aen> $message = "Enter the SQL Login credentials for the SQL Server"
PS /Users/aen> $server.ConnectionContext.LoginSecure = $false
PS /Users/aen> $sqllogin = Get-Credential -Message $message

PowerShell credential request
Enter the SQL Login credentials for the SQL Server
User: sa
Password for user sa: ******************

PS /Users/aen> $server.ConnectionContext.set_Login($sqllogin.username)
PS /Users/aen> $server.ConnectionContext.set_SecurePassword($sqllogin.Password)
PS /Users/aen> $server | select-object HostPlatform, Platform

HostPlatform Platform
------------ --------
Linux        NT x64

Bug to Microsoft submitted....

My work here is done :)
jk, thanks for sticking with it and working on this with me.

yessssss thank you both. @wsmelton - you think they just forced the platform in SqlServer ?

I think in ssms it is looking at hostplatform and they just level it as platform.

Ticket was closed by MS as by design so we need to update our command(s) to use HostPlatform instead of Platform.


Write a Ruby program to convert miles to meters

Description
Write a Ruby program to convert miles to meters
1 mile = 1609 meters
Input  : 5
Output : 8046.72

How to contribute

Save the solution in program/ConvertMilesToMeters.rb file
Add ConvertMilesToMeters.rb file in convert-miles-to-meters folder

!assign


MRBrainS18 dataset

Hi @hjkuijf
I'm trying to download the MRBrainS18 dataset recently and I've sent my confidentiality agreement to<EMAIL_ADDRESS>However, no reply is received and the website seems to be unavailable now.
It would be great if you can check it out for me.
Thank you so much.

We process MRBrainS18 registration emails regularly. You have send your email one (1) business day ago. Please allow us some more time, since we also have other work to do.


Feedback for Samsung page

Has anyone looked into Android 12? I see that the latest version discussed is Android 11. Thanks in advance!

A12 Samsung seems to have gotten better and will respect your settings now.


Adds retry after header

Following https://developer.apple.com/documentation/appstoreserverapi/identifying_rate_limits, I added the retry after to the error so the client can respect the rate limit correctly.

Ok tweaked the constructor. Made this change a while ago in my fork, can't remember why I decided to add the object param instead of just an optional number.  🤷 Either way I think it's better your way!

Thank you. I've merged and released v0.15.3.


metrics-server failing to give pod stats

Works:
kubectl top nodes

NAME           CPU(cores)   CPU%   MEMORY(bytes)   MEMORY%
k3s-master     1417m        35%    1547Mi          9%
k3s-worker-a   664m         8%     4563Mi          14%
k3s-worker-b   645m         8%     4512Mi          14%
k3s-worker-c   285m         3%     4260Mi          13%
k3s-worker-d   466m         5%     18240Mi         28%
k3s-worker-e   5189m        64%    9272Mi          14%

Doesn't work:
kubectl top pods

W0213 12:17:04.017607   30395 top_pod.go:266] Metrics not available for pod default/sonarr-episode-prune-7776645f74-mt4s4, age: 170h45m57.017599s
error: Metrics not available for pod default/sonarr-episode-prune-7776645f74-mt4s4, age: 170h45m57.017599s

I saw mentioned this could be related to using Docker as the CRI, https://github.com/onedr0p/k3s-gitops/issues/13

Issue with k3s https://github.com/rancher/k3s/issues/1067#issuecomment-564007048

This has started working now, I do not think I did anything in order for this to work :/
❯ k top po
NAME                                    CPU(cores)   MEMORY(bytes)
blocky-558f8966b6-rp49r                 2m           11Mi
jackett-5cf784cbc7-2hwcw                2m           130Mi
nzbget-5c97f886d6-7mwzs                 2m           33Mi
nzbhydra2-85b9d4b7cb-ztchb              3m           424Mi
ombi-5dcdf9ffcf-rbfjx                   3m           133Mi
plex-kube-plex-5944f974cf-qr59c         9m           15331Mi
qbittorrent-fc68bdddd-w4sh6             4m           1138Mi
qbittorrent-prune-6958ff4756-f8nfr      1m           10Mi
radarr-74db59fb5-d9smj                  32m          587Mi
rclone-sync-85f655b974-lw2ct            1m           181Mi
sonarr-9fbff9c7d-nz6qq                  37m          857Mi
sonarr-episode-prune-7776645f74-wk8jg   1m           15Mi
tautulli-7d6b4b4b9d-2wgqj               6m           133Mi


Fixed auth token not getting refreshed on client cache reset

Token was being passed on server render from getInitialProps but if a client side redirect happened the token was not refreshed and thus queries after login were not working without a refresh.
Fixes #5246

Awesome 👍


OWLS-95638: WKO 4.0 AI documentation for runtime functionality and sample.

Changes for WKO 4.0 AI documentation for runtime functionality and sample. I have also updated the description in sample Dockerfile to match the current domain specs.

Looks fine, except for a mistake that's my fault. Please correct it and then I am fine with this PR.

Thanks Rosemary, I have fixed the section headings and contents section as you suggested.


Adverts for ujust commands in the terminal motd gives a false sense of their security.

Describe the bug
By default the terminal motd includes adverts or tips for executing random ujust commands without including any links to documentation about what these features do.
It instils a false sense of reliability in the command and there's no clear recourse for when they go wrong.

What did you expect to happen?
One of the suggestions is:
Using full disk encryption and tired of entering your password?  ujust setup-luks-tpm-unlock  can help you
  use your CPU's TPM to unlock your device.

But executing the suggested command is not reliable.
It can perform an unknown portion of the task, outputs a cryptic error and then the user if left without any clear path for reverting the process:  https://github.com/ublue-os/bluefin/issues/1469.
Instead it would make more sense to link to documentation for the ujust command.
Output of rpm-ostree status
N/A

Output of groups
N/A

Extra information or context
No response

ujust is a wrapper around just, which is a task runner.
ujust --choose will give you am fzf view with a preview window to inspect what each command will do.
A large amount of the commands have been moved from inline commands to scripts that can be viewed in /usr/libexec often prefixed with ublue.


Serialize AssetName to JSON as hex string

Fix (second attempt) of incorrect UTF-8 decoding of non-textual (non-UTF-8) asset names.
I also changed tx-view to test JSON repr.
BEFORE:
{
  "51f056b530aba5969bf996d1be51c7aa3581f6e0d7b9e0911a5c0c1d": {
    "TheDrifter13": 1,
    "GOLDtest": 5,
    "��\"3DUfw��������": 42 // UTF-8 fail, actually
  }
}

AFTER:
{
  "51f056b530aba5969bf996d1be51c7aa3581f6e0d7b9e0911a5c0c1d": {
    "546865447269667465723133": 1,
    "474f4c4474657374": 5,
    "00112233445566778899AABBCCDDEEFF": 42
  }
}

This is how it already looks like in UI (Cardano Explorer):

asset1...
Ticker:
Name:
Description:
Policy ID: 51f056b530aba5969bf996d1be51c7aa3581f6e0d7b9e0911a5c0c1d
Asset Name: 474f4c4474657374

Future Daedalus:

Name: GoodCoin
Asset name: 676f6f64636f696e (ASCII: goodcoin)

Just as a note: It should also be than <policyID><assetName> combined hexstring for the normal query utxo output too, also for the tx-out and mint parameters for CLI. like the "subject" parameter for the metadata-registry, same format for all.

Removed textual AssetName everywhere, hex only

Removed textual AssetName everywhere, hex only

do we really need the "." separator now than for policyID.assetName ?  polID is 56 chars long, rest is the assetName if present.

Removed textual AssetName everywhere, hex only

minted some tokens, looking good

I noticed in the JSON output, that the address key is now showing the bech32 address of the requested address. A few versions ago this showed the hex-representation of the address. Does not make a huge difference, but is this address field actively used somewhere? Looks like too me, that this is an additional information nobody really needs?

@cblp any updates here from your side? would be nice to see this in 1.31.0 😄

how does the output of "Friendly output multi-asset with ASCII version where possible" look like? i would leave it up to the software behind the CLI to decide to show it as readable or as hex/binary. makes it more complicated to post-process if the output is not hex all the time.

how does the output of "Friendly output multi-asset with ASCII version where possible" look like?

Here is an example:
mint:
  policy 52dc3d43b6d2465e96109ce75ab61abe5e9c1d8a3c9ce6ff8a3af528:
    asset 736b79 (sky): 142

i would leave it up to the software behind the CLI to decide to show it as readable or as hex/binary. makes it more complicated to post-process if the output is not hex all the time.

cardano-cli transaction view is an end-user software that uses Cardano API.
API version of cardano-cli is to be built in #3213

ah, so the normal utxo query is not affected by this, only in the transaction view output?

@gitmachtl utxo query is affected only in part of switching from text to hex.

@gitmachtl utxo query is affected only in part of switching from text to hex.

thats perfect, so hex output in the normal utxo query output, also in the JSON version of the output. and friendly names in the transaction view output.

@dcoutts should I fix docs (couttscoins etc.) in this PR?

ping 😄

@dcoutts should I fix docs (couttscoins etc.) in this PR?
Yes, please update the docs with respect to the serialization change.

@JaredCorduan  @newhoggy maybe someone of you is having the time to review it also?

do we have a change to get this into the 1.31.0 release?

@nc6 do we have a change to get this into the 1.31.0 release?

bors merge

wohoo! :-)


Is there a way to report the evap fan speed

I'd like to write an automation which takes into account the outside temperature and humidity (Aqara sensor) and the inside temperature and humidity (another Aqara sensor) and the current fan speed and adjust the comfort level in AUTO mode.
It looks like the underlying python library can extract the current fan speed from the status updates. Is it possible to expose that current fan speed as a sensor?

I checked that a while ago, but forgot to respond. Unfortunately, the fan speed is not reported back. I think I had the same thought when I initially did the module, but it's just not reported, same as the temperature is never reported back, sadly.


Update for FPP (5.29.4)

minor changes to satdb and MAPL fix for FPP - this also includes change to NH3

Ah ... wonderful ... right on time!! Thank you so much


Character dont save/load in qb-multicharacter

Hello again, now I have a problem, every time I create a character and disconnect, when I reconnect it does not show the character selection. im sorry for wasting your time with this errors but i re instal qbmultichar and change the codes and it doesnt load either :c

Please create a post in the support channel over my discord. This is a place for actual issues and not support.
https://discord.illenium.dev

Im in the discord but not veryfied yet

Closing as not an issue


Inventory Panel displaying/pulling wrong items

Inventory will show searched item but export different item(s)
Example:
Typed in hardened to find hardened blood shard
GUI shows hardened blood shard and amount
I click to pull shards out and instead BC Kinesis pipes come out
I dont believe its an issue with the mod items Im pulling out as this has happened dozens of times
with very random items from all sorts of mods.

Can you reproduce that in single player? And what steps are required to get this?

Haven't tried yet Ill test shortly.
Nothing special has to happen. I at any time go to my panel grab the item I want and before
it gets into my panel even it changes into a random item.
http://imgur.com/a/zcLiy
Here is some screen shots that just occurred on my server. It's not those specific items though as its happened numerous times with items of all types.

This happens randomly. Players are also reporting in the the pack i created.
There seems to be multiple issues with the inventory panel currently.
-Pulled item is not the item you receive (like shown above)
-Inventory panel sometimes not using Buckets of liquids in crafting recipes, it will return the full bucket back to you.
-Items randomly disappearing/getting deleted

I didn't realize EIO had already embedded support for Storage Drawers.
My guess is that EIO is either doing some level of caching that doesn't like how the order of slot enumeration changes, or it's mixing up "drawer" slots and "inventory" slots.  I kind of suspect that later.
I'm happy to answer more questions about drawer internals.

Sorry, I realized drawers aren't actually mentioned anywhere in this issue specifically.  I just got linked here related to it.  Ignore the above if not relevant.

Well - even if the indices got mixed up - I don't think this could be the cause here.
The inventory panel assigns an unique ID per item (on the server) which depends on the item ID, meta data and NBT tag. Server and client use this to communicate the item lists and requests.
On extracting the server verified that the item in a specific inventory/slot is what the server expects before extracting.
I assume that this is a kcauldron issue (as I think all reports had that in common) and the item IDs are not the same on server and client. If someone sees this issue on non kcauldron servers this information would be really helpful in combination with the logs generated by the debug build linked in #3129.

I can say for a fact that this issue is also present on a friends FTB Infinity server running straight forge.
Two people on that server experience the issue with drawers and I on my server experienced it using iron chests

"your server", @Cuppycake ? Does that mean you control it? If so, would you please use the test version that is linked in #3129 and send us the logs?

Yes I own and maintain the server and the pack on all my own hardware. Ill see what data I can gather.


Fancy -> Styled Components

HSDS V3 release
Still a work in progress
This update is trying to replace Fancy with the library styled-components. The way of generating css-in-js should be very similar, if not 99.9% the same so it should not impact our way of styling components. That being said, there is a lot of difference between the two libraries and this update is trying to close the gap to keep Help scout project working exactly the same way.
Styled-components offer a smaller bundle and a faster rendering engine compare to what we have right now with fancy (fork of Emotion 9). Moving away from fancy will allow us to leverage the community behind Styled-components that keep improving the library on a daily basis.
Enough talking, let's dive in what this PR is changing to our library:
Update `styled``
Originally we had a component called styled that was importing fancy and exposing it to our application.
import styled,
   { ThemeProvider, FrameProvider, ScopeProvider }
from "@helpscout/hsds-react/components/styled"

We update this component to import directly styled-components. It will also expose the ThemeProvider from this library we should work exactly like the one in fancy. FrameProvider and ScopeProvider will be removed from that component
Another update we did is to import directly from styled-component locally instead of doing it from our customize styled component (sorry I know it's confusing). Centralizing the import was nice, but it was a pain when refactoring or importing from a different directory. Now no need to deal with relative path:
import styled from "styled-components"

Frame component
Fancy did create a custom FrameProvider component to sync styles between iframe and the root document. That component is decrecated and being replace with a <Frame /> component. Instead of using a regular iframe in your code, simply use the <Frame /> component and it will take care of rendering the iframe and syncing the styles
import Frame from "@helpscout/hsds-react/components/frame"

class MyApp extends React.Component{
  render(){
    return (
      <Frame>
        <div id="App">
          <Card>
            <Text>
              <Tooltip title="Elf!" triggerOn="click">
                Buddy!
              </Tooltip>
            </Text>
          </Card>
      </div>
    </Frame>)
  }
}

Typescript
All our components are written using typescript. The good news is that styled-components support typing, but the bad news for us is that our code was not ready for that. This update fix a lot of typing issue from our css files.
When passing props to the styled component, we need a prop interface:
export interface Props {
  className?: string    className?: string
  innerRef: (node: HTMLElement) => void   ref: (node: HTMLElement) => void
}

const WrapperUI = styled.div<Props>`...`

When adding a ref prop, typescript was complaining for a reason still unknown to me. Solution was to force the typing of the method:
<ButtonWrapperUI align={align} ref={this.setNodeRef as any}>

innerRef deprecated
innerRef was a prop made available by Emotion to get access to the dom element node. With styled-components, we need to use directly ref. This might cause some issue with components using the hsds-react library
<ContentUI data-cy="ActionSelectContent" ref={this.setNodeRef}>

remove namespacedComponent
all components declaration in index.ts  was declaring himself in our namespacedComponent. It is no longer needed, except when there is propProvider involve. The good news is that PropProvider will push the component in the namespaced object if it was not already declared.
SC_ATTR
styled-components allow us to namespace our style element to not interfere with other styled-components instance on given page. This env variable needs to be set by the process that will compile the styles (storybook or webpack for example)
SC_ATTR=HSDS-react start-storybook -p 8900
Next steps to complete this PR

[ ] Fix testing
[ ] Fix ScopeProvider
[ ] Fix Test and validate that
[ ] Implement globalStyling
[ ] Deciding of PropProvider destiny
[ ] Validate that <Frame /> is working as expected (IE11 as well)
[ ] Test everywhere

This is looking great! I can't thank you enough for taking on this big task, the benefits are clear!

This is a serious undertaking, great job!
With styled-components, we need to use directly ref. This might cause some issue with components using the hsds-react library

^^ How do you think we can test for that?

This is looking really good. It's a big PR but I have faith in PL and our very large test suite. And I think the styled-components API is a big upgrade on what we have now.
I think it would be great to give QA a heads up on this, as it's probably going to require quite a bit of regression testing and I know Ryan at least likes to know about these things in advance

pushing another update that fixes almost all tests and coverage. There is component that would require more time to fix all the test:

[] Dropdown.Item
[] Frame
[] Tooltip Reference (if it used somewhere in hs-app/embed)
[] contextConnect
[] propConnect

merging in v3, to continue development of the next version. What's left to do or to fix will be done in this branch (there is a lot of cards covering what needs to be done)


AutoMigrate will not update changed default value for primary keys

GORM Playground Link
https://github.com/go-gorm/playground/pull/716
Description
I recently changed all of my tables to use a default for the UUID primary key so that if I forget to set a value, the default takes over and generates a UUID for me. Specifically, this is for the primary keys (I have tested my example with non primary keys and changing/adding a default works fine). I have tested this example with postgres 15.
//create a struct such as this (Notice the UUID primary key)
type Example struct {
	CreatedAt *time.Time      `json:"-"`
	DeletedAt *gorm.DeletedAt `gorm:"index" json:"-"`
	UpdatedAt *time.Time      `json:"UpdatedAt,omitempty,format:unix"`
	UUID      uuid.UUID       `gorm:"primarykey;type:uuid" json:"UUID"`
	Data string
}

//migrate the struct
lc.Gorm.AutoMigrate(&Example{})

//Update Example struct to the following (notice the addition of the default to UUID)
type Example struct {
	CreatedAt *time.Time      `json:"-"`
	DeletedAt *gorm.DeletedAt `gorm:"index" json:"-"`
	UpdatedAt *time.Time      `json:"UpdatedAt,omitempty,format:unix"`
	UUID uuid.UUID `gorm:"primarykey;type:uuid;default:gen_random_uuid()" json:"UUID"`
	Data string
}

lc.Gorm.AutoMigrate(&Example{})

After checking the database table data (via pgadmin), the default of UUID has not been changed, and is still empty.
From my perusal of the migrator code, this lack of change appears to be the case due to the !field.PrimaryKey check here, which makes it appear intentional, but I was not able to find any documentation of this limitation, or reasoning as to why this limit exists. Its even stranger as you can set the default for a primary key on table creation.
Is it possible this check was made to prevent setting default:someConst for the primary key?

Yes, this is intentional, migrating primary keys is a dangerous practice.
We should describe it clearly in the documentation

You can use AlterColumn to change it.
Migrating the primary key is a relatively complex and dangerous operation. It may be a foreign key to another table, which may require confirmation of the order of table changes and transactions involved. Even so, there may still be other tables that are not declared in the same process.
Usually in the early stages of development we can recreate it with DropTable and AutoMigrate. Your suggestion may be possible, but we may need to confirm that this will bring enough benefits.


Fix invite acceptance over federation

This PR fixes a race condition which caused accepting invites over federation to fail. It adds a function which runs as a background process which retries the update_room_membership call several times if an exception is raised by update_room_membership. Fixes #12.
The associated PR in Synapse is https://github.com/matrix-org/synapse/pull/15577, please note that the tests here will fail until that PR is merged and run_as_background_process is exposed in the module API.

I don't think the explicit import of run_as_background_process will be picked up by mypy until a new version of synapse is released (this change is currently merged in develop but has not been released) so I think this is ready to review as-is.


Fix Monitor Thresholds

This fixes the Monitor racing and the health poll marking Available
caches which were marked Unavailable by the Stat poll.
There was already logic to prevent that, but it wasn't accounting
for the Calculated stats, so the Health poll didn't check Calculated
stats for thresholds, but it does "have" those stats, so it thought
it could mark them available again.
This fixes the Health poll to also check thresholds for Calculated
stats, which not only fixes the race, but makes thresholds be
marked up and down faster with the quicker Health poll.
Includes unit tests.
Bug fix, no interface change, so no documentation change.
Includes changelog.

[x] This PR is not related to any other Issue

Which Traffic Control components are affected by this PR?

Traffic Monitor

What is the best way to verify this PR?
Set low availableKbps thresholds, run the monitor, verify in the Event Log and Cache States that caches are marked unavailable from kbps threshold, verify health poll does not mark caches that remain above the threshold as available, and that caches remain Unavailable (unless they drop below the threshold).
If this is a bug fix, what versions of Traffic Ops are affected?

2.2
3.0.0
3.0.1

The following criteria are ALL met by this PR

[x] This PR includes tests OR I have explained why tests are unnecessary
[x] This PR includes documentation OR I have explained why documentation is unnecessary
[x] This PR includes an update to CHANGELOG.md OR such an update is not necessary
[x] This PR includes any and all required license headers
[x] This PR ensures that database migration sequence is correct OR this PR does not include a database migration
[x] This PR DOES NOT FIX A SERIOUS SECURITY VULNERABILITY (see the Apache Software Foundation's security guidelines for details)

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/trafficcontrol-PR/3760/
Test FAILed.

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/trafficcontrol-PR/3762/
Test PASSed.

@ezelkow1 Done.

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/trafficcontrol-PR/3864/
Test PASSed.


DocsArchive-<IP_ADDRESS>Auto archive content

Auto archive content from https://github.com/OPS-E2E-PPE/docs-archive-test-source

Docs Build status updates of commit 8853ddb:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

:x:Error

Details

[Error]

Cannot sync git repo to specified commit because branch Archive_master_2019-10-22-03-40-16 has been deleted or has been force pushed: fatal: Couldn't find remote ref Archive_master_2019-10-22-03-40-16

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


Open Data Week presentation proposal

event link
due Nov 22 and proposal form is mostly short answer and multiple choice questions
ideas so far (per Amanda and group conversations)

use PLUTO to describe how we currently build data products
people will likely be most interested in the data (source data, new monthly frequency, QA)
interesting tech details (recipes yaml, DB schemas)
PLUTO use cases (city agencies, academic)

Link to draft (let Sasha know if you can't access it).

submitted


Snapshots under CI are failing where local is succeeding

I'm testing cypress with travis,
Tests are keep failing in CI where on local they succeeding.
I updated all snapshots to make sure that everything will success,
though most differences are around 5%,
but there are some (which are related to changes I did in that PR) where it's around 70% diff,
another fun fact, our Cypress Plan reached over 100% so no more recordings (so unfortunately can't see the differences), is it possible that CI is testing for some reason against the last snapshots that we're recorded into dashboard and not the one I just updated?
commands.js
import {addMatchImageSnapshotCommand} from 'cypress-image-snapshot/command';
addMatchImageSnapshotCommand();

plugins/index.js
const {addMatchImageSnapshotPlugin} = require('cypress-image-snapshot/plugin');

const webpack = require('@cypress/webpack-preprocessor');

const webpackOptions = require('../webpack.config');

module.exports = (on, config) => {
    const options = {webpackOptions};
    on('file:preprocessor', webpack(options));
    addMatchImageSnapshotPlugin(on, config);
};

Example test:
    it('can have error', () => {
        cy.get(soul(SANDBOX_ERROR)).type('Error Message');
        cy.get(soul(SANDBOX_SECTION)).matchImageSnapshot('error');
        cy.get(soul(SANDBOX_ERROR)).clear();
    });

base snapshots should be generated on the same environment as where they are compared. For example when I generate base snapshots in Chrome on my Mac, it will fail in Chrome on Linux in CI, because it renders fonts different.
I use linux docker to generate base snapshots on my Mac, the same linux version as on CI. Till now everything works ok.

I didn't thought about running it on linux using Docker thanks for the tip.
I found some issues that caused the differences, mostly falsy snapshots, closing it :)


update tsconfig

Summary
Describe the goal of this PR. Mention any related Issue numbers.
Requirements (place an x in each [ ])

[x] I've read and understood the Contributing Guidelines and have done my best effort to follow them.
[x] I've read and agree to the Code of Conduct.

 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.

Sorry, i accidentally open the PR. Please ignore this.


BUG: spatial/qhull: get HalfspaceIntersection.dual_points from the correct array

Qhull stores half-space intersection points in the points array, which
gives the canonical ordering for the dual hull points. qhalf Hx,y o indeed
prints exactly these points.
The order of points in vertex_list may be permuted, and should not be
used here.
Fixes gh-10501

cc @haudren

Very interesting bug. I must say I never used that feature extensively, I was mostly interested in getting the intersection points for my use cases (in any order). It's been a while since I touched that code, so I don't have any comments to add, thanks for fixing this!

Hello! Are there any plans to merge this?

@ilya-palachev: yes, but there's time to next release. If you need it now, git fetch origin refs/pull/10503/merge; git checkout FETCH_HEAD.

@ilya-palachev: yes, but there's time to next release. If you need it now, git fetch origin refs/pull/10503/merge; git checkout FETCH_HEAD (the patch may apply also on top last stable release, didn't check).

OK, thank you @pv !


🐞 Bug  Project(My circuits) description page has css broken

Is there an existing issue for this?

[X] I have searched the existing issues

Describe the bug
Project(My circuits) description page has css broken
its elements are at wrong places
Steps To Reproduce

Go to circuitverse and login
go to dashboard page
go to a project
in project go to more

Expected Behavior
its elements should be at correct places
Screenshots
No response
Device Information [optional]
- OS:
- Browser:
- version:

Additional context
No response
Are you working on this issue?
Yes

@tachyons assign it to me

I would like to work on this issue as it can just be done done in minutes by solving css please sir assign this issue to me i would make sure that i merge pr as soon as possible and solve this issue

Can I please work on this?


native in-memory filesystem

The memfs we were using has a lot of limitations that we had to work around: no support for symlinks or hardlinks, no support for mknod (devices), etc. We put many layers on top to get it to work, and finally hit the limit with https://github.com/wolfi-dev/os/actions/runs/4618853760/jobs/8171294258?pr=1074
This eliminates the external memfs and replaces it with a simple memfs that has full support.

Lots of CI to fix. We will get to it.

And green CI!

Lets get this in prior to merging: https://github.com/chainguard-dev/apko/pull/590

Just need someone to review and approve.

I enabled auto-merge; as soon as someone approves, it will merge in.

relies on #619

#619 is in

Aha! Now we got it!


Link from Security concept to related pages

Quick tweak prompted by https://kubernetes.slack.com/archives/CGW9JHPJN/p1559658529001100
Not useful? No worries; it didn't take long, I can /close it and move on.
/priority backlog

/lgtm
/approve


[BUG] virtual environment created from installer does not allow for Fluent 24R2 to be launched

🔍 Before submitting the issue

[x] I have tried the latest Ansys Python Installer version
[x] I have adapted properly the title of the issue

Description
Steps to reproduce:

Install latest installer
From installer, install Python (version doesn't matter)
From installer, create virtual environment
From installer, install pyfluent (version doesn't matter)
From installer, Launch console
From console, launch Fluent as follows: start "" "C:\Program Files\ANSYS Inc\v242\fluent\ntbin\win64\fluent.exe" 3ddp -t2 -sifile=serverinfo.txt -nm
Fluent opens and then shuts down. Error in the Fluent transcript is as follows:

legacy Provider loading failed
EVP_DecryptInit. could not load the shared library
Everything works fine if I launch from a virtual environment created manually.
Everything works fine if I launch a version of Fluent 24R1 and older.

@rploconnor: I believe this may be the result of a broader issue with pyinstaller that we can work around. See #376, I'm happy to answer any questions you may have about it.

Might be related like @phx-mkoninckx mentions, @rploconnor - can you give it a try to this version of the installer? https://github.com/ansys/python-installer-qt-gui/pull/377
You can download it from the workflow artifacts


Major refactoring

@gertsonderby I had this in the pipeline but wanted to wait for your changes to be merged. What do you think?

On the first glance, it looks reasonable. Am I seeing checked-in build artifacts, though? I'll take a closer look at it.

yeah, already got rid of the artifact...

LGTM.