"remote" should be an animation.

_make_remote just returns an animation.  From reading this code several times, I see no reason why we shouldn't just have a new sort of animation, a RemoteAnimation, which contains other animations.

The reason this comes up is if we don't do this, we're going to need a new section in the project for each different type of externally-controlled animation - and that might be a lot.

Well it returns something that has start and cleanup functions. But yeah...
I get what you mean. Should be easy... But Id have to look at how that
might complicate the config...it will certainly me less simple but prob not
too bad. I'll see if I can make that work before 3.1 release. Shouldn't be
too hard...
On Jul 20, 2017 6:34 AM, "Tom Swirly"<EMAIL_ADDRESS>wrote:

The reason this comes up is if we don't do this, we're going to need a new
section in the project for each different type of externally-controlled
animation - and that might be a lot.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/ManiacalLabs/BiblioPixel/issues/298#issuecomment-316664787,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA6a6udodcNko12TM4TSeGFO0dBRcyNKks5sPy1IgaJpZM4Od6Gs
.

:+1:

I looked at this some more.
It's not so much code I think.
You derive Remote from Sequence and simply construct everything as if it were a Sequence - you only override step.
Let me do an introductory tiny change to threading to make this easy for you.  Moment pls.

OK, no change coming.
The one issue is ensuring that child animations of a remote animation are forced to be threaded.
There's no slick way to do this right now.  The easiest way is to override Animation.set_runner in Remote and after calling super().set_runner, "manually" setting this value on each child animation.

No! Remote IS NOT a Sequence.
Remote animations are individual.
On Thu, Jul 20, 2017 at 8:14 AM, Tom Swirly<EMAIL_ADDRESS>wrote:

I looked at this some more.
It's not so much code I think.
You derive Remote from Sequence and simply construct everything as if it
were a Sequence - you only override step.
Let me do an introductory tiny change to threading to make this easy for
you. Moment pls.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/ManiacalLabs/BiblioPixel/issues/298#issuecomment-316684844,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA6a6iLhYp7KXhz_SHD4IQG3GNWTr5mzks5sP0SZgaJpZM4Od6Gs
.

Then derive from the new bp.animations.collection.Collection class.  :-D

Why? That just complicates what I have.
On Thu, Jul 20, 2017 at 8:34 AM, Tom Swirly<EMAIL_ADDRESS>wrote:

Then derive from the new bp.animations.collection.Collection class. :-D
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/ManiacalLabs/BiblioPixel/issues/298#issuecomment-316689145,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA6a6so1VxQYqBuIw595AZU6gbOO9ur3ks5sP0lZgaJpZM4Od6Gs
.

Why does it make it more complex?
All that code you have constructing subanimations for remotes shouldn't be there in that code body.
We are already construction subanimations - for Sequences.  We should be doing it the same way every time, not having two different ways of doing it depending on the class.

Instead of yakking about it, I can just do it.  :-D  Moment pls.

See comment on your PR.
On Thu, Jul 20, 2017 at 8:43 AM, Tom Swirly<EMAIL_ADDRESS>wrote:

Why does it make it more complex?
All that code you have constructing subanimations for remotes shouldn't be
there in that code body.
We are already construction subanimations - for Sequences. We should be
doing it the same way every time, not having two different ways of doing it
depending on the class.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/ManiacalLabs/BiblioPixel/issues/298#issuecomment-316691086,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA6a6hhYlD8TotpdewFgVB2NthWDeg_Pks5sP0tggaJpZM4Od6Gs
.

This is fixed, yes?

Yup, of course :) One of these days I'm going to make a project file with a
remote that starts a remote that starts a remote that starts a remote (all
on diff ports of course).
On Mon, Jul 24, 2017 at 6:54 AM, Tom Swirly<EMAIL_ADDRESS>wrote:

Closed #298 https://github.com/ManiacalLabs/BiblioPixel/issues/298.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/ManiacalLabs/BiblioPixel/issues/298#event-1175888834,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA6a6sqG9qRT-76LFy-aYHff5RKIwVnHks5sRHfTgaJpZM4Od6Gs
.


Add support for Travis CI

I've added the correct .travis.yml to get things going along with a sample test case for it to play. It now requires the test_db from (https://github.com/datacharmer/test_db) to be setup before we can harness things. This will need a script to clone the repo, import the .sql data into mysql and finally be able run the test.

Just added the build status logo.

Added the datacharmer/test_db build setup scripts here. The build is now green!


Top Of Book

I really like what you have done so far. Thank you. I'm using the tutorials as a base and trying to expand from there. Although I've programmed before (Python most recently), I'm new to typescript/javascript. Perhaps this is not the forum to ask for some help but I don't see a forum for help -- if there is, please point me to it.
My specific problem is that I'd like to simply see top of book as it updates. I am trying to use the LiveOrderBook example #2: https://github.com/coinbase/gdax-tt/blob/master/tutorials/t002_liveOrderbook.ts
I've added to the FeedFactory streamhandler (please instruct me if I have used the terminology incorrectly) a new book.on message handler from the LiveOrderBook:
   book.on('data',() => {`
             console.log(`${book.highestBid.toFixed(3)}`);
   });

This does not work. I get "undefined". When I look at the API docs, it appears that book does have the accessor highestBid (https://coinbase.github.io/gdax-tt/apiref/classes/src_lib_bookbuilder.bookbuilder.html)
I've also tried using the state:
const state = book.state();
console.log(`highest bid:  ${state.highestBid}`);

In addition, should I be using the message "data" to check for top of book updates? or should I use "ticker"? Or something else?
So I feel like I'm not reading the API docs correctly. Please advise on my questions and, perhaps, how I ought to be accessing the top of book rather than the directions I have taken.
edit: fixed formatting problem.

I (finally) figured this out. For those that are interested, here is how to solve the problem:
const state = ltc_book.state();
const bid: PriceLevel = state.bids[0];
console.log('bid : ', bid.price);
console.log(`here : ${padfloat(bid.price, 7, 2)}`);

I've included the raw console.log of bid.price, but it won't work they way that you want it to as it is a "BigJS" number.
output looks like this:
bid :  BigNumber { s: 1, e: 2, c: [ 148,<PHONE_NUMBER>0000 ] }
here :  148.62

Note that I've hardcoded the index to the bids to 0. I based this on code in
utils/printer.ts
https://github.com/coinbase/gdax-tt/blob/master/src/utils/printers.ts

If you have a LiveOrderbook:
const live = new LiveOrderbook(config);

// later to fetch the prices
live.book.highestBid.price.toNumber()
live.book.lowestAsk.price.toNumber()


Change Functional tests to use attribute instead of compiler directive

Added attribute [TestCategory("FunctionalTests")] to functional tests. Builds then reference that attribute for running tests instead of compiling different versions.

Pull Request Test Coverage Report for Build 95347

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 78.531%

Totals

Change from base Build 95237:
0.0%

Covered Lines:
13154

Relevant Lines:
16750

💛  - Coveralls

:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.dll compared against version 4.6.3
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.AI.Luis.dll compared against version 4.6.3
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.AI.QnA.dll compared against version 4.6.3
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.ApplicationInsights.dll compared against version 4.6.3
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.Azure.dll compared against version 4.6.3
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.Dialogs.dll compared against version 4.6.3
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.Integration.ApplicationInsights.Core.dll compared against version 4.6.3
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.Integration.AspNet.Core.dll compared against version 4.6.3
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.TemplateManager.dll compared against version 4.6.3
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.Testing.dll compared against version 4.6.3
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Configuration.dll compared against version 4.6.3
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Connector.dll compared against version 4.6.3
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Schema.dll compared against version 4.6.3
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Streaming.dll compared against version 4.6.3


Indexed functions can break Int-keyed maps

Currently, the library contains these functions for accessing and looking up keys by index:
findIndex :: Ord k => Key ph k -> Map ph k a -> Key ph Int
elemAt :: Key ph Int -> Map ph k v -> (Key ph k, v)

It was a mistake to re-use the Key type for this purpose. Suppose you had a singleton map with Int keys, containing the mapping [17 |-> _]. findIndex (Key 17) will give you a Key 0 with the same phantom type as the original map. This means that you can use the Key 0 with (!), which will then fail.
Instead, the index-based functions should just use some Index type rather than re-using the Key type.

Fixed in 9001b06fbccdb48d7c


Support for version 3

@FalseDev I wrote this without testing because no time
kindly see the code lol

It's never going to happen is it


controller: Clean up the instance manager when the node is disconnected

Fix the 1st scenario in longhorn/longhorn#1545

Need an explanation for the PR.


Add config options for dataset write_in_background and write_periode

Make it possible to set these globally.
Along the way

Remove unsupported plottr subscriber.

@ThorvaldLarsen

Both the documentation on config in general and the write options should be improved. I will do that in a subsequent pr

@jenshnielsen @astafan8 this is a great solution to changes in doNd defaults in my opinion that allows everyone the settings they want rather than being stuck with the defaults in qcodes core as what most (vocal) users would like at any given time.


adding support for .NET 4.6.1

Thanks for your contribution!  I will get this pulled into a release over the next couple days.  Just waiting for the permissions needed to push the package to nuget.org and chocolatey.org.  Thanks!

+1
On 8 Dec 2015 8:51 pm, "Gary Ewan Park"<EMAIL_ADDRESS>wrote:

Thanks for your contribution! I will get this pulled into a release over
the next couple days. Just waiting for the permissions needed to push the
package to nuget.org and chocolatey.org. Thanks!
—
Reply to this email directly or view it on GitHub
https://github.com/psake/psake/pull/141#issuecomment-162995339.

@damianh I am good to push this tonight, but I am still considered about the failing test that I am seeing.  Any thoughts on why this is failing for me.  Is this environment specific do you think?

sorry lad no clue
On 12 Dec 2015 8:13 pm, "Gary Ewan Park"<EMAIL_ADDRESS>wrote:

@damianh https://github.com/damianh I am good to push this tonight, but
I am still considered about the failing test that I am seeing. Any thoughts
on why this is failing for me. Is this environment specific do you think?
—
Reply to this email directly or view it on GitHub
https://github.com/psake/psake/pull/141#issuecomment-164176870.

push it anyway
On 12 Dec 2015 9:02 pm, "Damian Hickey"<EMAIL_ADDRESS>wrote:

sorry lad no clue
On 12 Dec 2015 8:13 pm, "Gary Ewan Park"<EMAIL_ADDRESS>wrote:

@damianh https://github.com/damianh I am good to push this tonight,
but I am still considered about the failing test that I am seeing. Any
thoughts on why this is failing for me. Is this environment specific do you
think?
—
Reply to this email directly or view it on GitHub
https://github.com/psake/psake/pull/141#issuecomment-164176870.

Assert ($output -NotLike "14.0") '$output should contain 14.0'

Should it be -like?


Colored background of plot area for WPF plots

I would like to:
Color plot background of a WPF plot.
This is what I have tried:
Issuing a wpfPlot1.plt.Colorset(Colorset.OneHalfDark); has no effect.  I have also tried to change the internal ScottPlot rendering just to see something other than white. No joy:)
What am I missing?
Thanks,
Mark

Hi @Scrat9876, check out the arguments available with wpfPlot1.plt.Style().
You can pass in individual colors:
https://swharden.com/scottplot/cookbooks/4.0.38/#customize-figure-background-colors
or pass in a Style which contains a collection of pre-selected colors which match:https://swharden.com/scottplot/cookbooks/4.0.38/#customize-plotstyles-plot-style-seaborn
I see where you got confused by Plot.Colorset() method. A color set is like a color palette... but I recognize this name makes it seem like this is how you "set the color". I'll consider improving this name in ScottPlot 4.1!
Let me know if this does the trick

wpfPlot1.plt.Style(figBg: Color.LightBlue);
wpfPlot1.plt.Style(dataBg: Color.LightYellow);

https://swharden.com/scottplot/cookbooks/4.0.38/#customize-figure-background-colors

I have just tried that and it won't work.
`            double[] dataX = new double[] { 1, 2, 3, 4, 5 };
double[] dataY = new double[] { 1, 4, 9, 16, 25 };
WpfPlot1.Plot.AddScatter(dataX, dataY);
        WpfPlot1.plt.Style(figBg: Color.FromRgb(222,27,27));
        WpfPlot1.plt.Style(dataBg: Color.FromRgb(27, 222, 157));
        `WpfPlot1.Refresh();``

This is my code and error says: "The best overload for 'Style' does not  a parameter named dataBg"...

Hi @stefan7189,
It looks like that code/documentation is for an old version of ScottPlot.
This code example can help you achieve your desired effect:
WpfPlot1.plt.Style(figureBackground: Color.FromRgb(222,27,27));
WpfPlot1.plt.Style(dataBackground: Color.FromRgb(27, 222, 157));

@stefan7189 also note that there are collections of predefined styles you may find useful:
https://scottplot.net/cookbook/4.1/category/style/


Line to install

Hey everyone. I'm super new to cordova, phonegap and CLI. Can anyone tell me what I have to write in my CLI to install this plugin?  Thanks! :)

@paulamourad you can read https://github.com/Telerik-Verified-Plugins/ImagePicker/blob/master/README.md

@sfaizanh I did, and I tried all the lines shown there, and I get the same error that says I don't have the package.json file:
`
Paulas-MacBook-Pro:plugins DevCore$ cordova plugin add https://github.com/Telerik-Verified-Plugins/ImagePicker.git
(node:4676) UnhandledPromiseRejectionWarning: CordovaError: Failed to fetch plugin https://github.com/Telerik-Verified-Plugins/ImagePicker.git via registry.
Probably this is either a connection problem, or plugin spec is incorrect.
Check your connection and plugin name/version/URL.
Error: npm: Command failed with exit code 254 Error output:
npm WARN checkPermissions Missing write access to /Users/DevCore/Desktop/vpay phonegap/Vpay Test/node_modules/cordova-plugin-barcodescanner
npm WARN enoent ENOENT: no such file or directory, open '/Users/DevCore/Desktop/vpay phonegap/Vpay Test/package.json'
npm WARN Vpay Test No description
npm WARN Vpay Test No repository field.
npm WARN Vpay Test No README data
npm WARN Vpay Test No license field.
npm ERR! path /Users/DevCore/Desktop/vpay phonegap/Vpay Test/node_modules/cordova-plugin-barcodescanner
npm ERR! code ENOENT
npm ERR! errno -2
npm ERR! syscall access
npm ERR! enoent ENOENT: no such file or directory, access '/Users/DevCore/Desktop/vpay phonegap/Vpay Test/node_modules/cordova-plugin-barcodescanner'
npm ERR! enoent This is related to npm not being able to find a file.
npm ERR! enoent
npm ERR!      /Users/DevCore/.npm/_logs/2018-04-25T15_14_41_203Z-debug.log
at /usr/local/lib/node_modules/cordova/node_modules/cordova-lib/src/plugman/fetch.js:173:37
at _rejected (/usr/local/lib/node_modules/cordova/node_modules/q/q.js:864:24)
at /usr/local/lib/node_modules/cordova/node_modules/q/q.js:890:30
at Promise.when (/usr/local/lib/node_modules/cordova/node_modules/q/q.js:1142:31)
at Promise.promise.promiseDispatch (/usr/local/lib/node_modules/cordova/node_modules/q/q.js:808:41)
at /usr/local/lib/node_modules/cordova/node_modules/q/q.js:624:44
at runSingle (/usr/local/lib/node_modules/cordova/node_modules/q/q.js:137:13)
at flush (/usr/local/lib/node_modules/cordova/node_modules/q/q.js:125:13)
at _combinedTickCallback (internal/process/next_tick.js:131:7)
at process._tickCallback (internal/process/next_tick.js:180:9)
(node:4676) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). (rejection id: 1)
(node:4676) [DEP0018] DeprecationWarning: Unhandled promise rejections are deprecated. In the future, promise rejections that are not handled will terminate the Node.js process with a non-zero exit code.
`
I've been struggling with this for the past week, and I really need to implement a barcode scanner in my app. Any other thoughts? Thanks.

Go in your config.xml file and replace with this one
https://github.com/Telerik-Verified-Plugins/ImagePicker/pull/114#issuecomment-383308577

@sfaizanh should I copy the code that is showing in the picture?

in config.xml
    <plugin name="cordova-plugin-telerik-imagepicker" spec="git+https://github.com/Sjerd/ImagePicker.git">
        <variable name="PHOTO_LIBRARY_USAGE_DESCRIPTION" value="Allow to add photos" />
    </plugin>

Here's what I've done:

I added the code you shows above to my config.xml
I created a package.json file in my plugins folder and added this code to the .json file:
Ran the command lines shown here.

Nothing works.  Here is my app setup. Is it correct? I've been trying to install the plugin to my plugins folder.

This is what my config.xml looks like:

And this is what my package.json file looks like (I created it):

And finally, here's how I'm writing the command:

What am I doing wrong? :'(

open plugins/fetch.json find com.synconset.imagepicker and remove it (if you find it), remove platforms
cordova platform rm android and add again cordova platform add android

and what is your cordova and cordova-android version?

I didn't find com.synconset.imagepicker in the plugins/fetch.json
Should I remove and add platforms inside the plugins folder or inside the project folder?

@sfaizanh How can I know what is my cordova and cordova-andoid version? :)

Here's my version:

My cordova version

Ok, so I did what you told me and I already got an error when running cordova platform rm android. I got the same mistake when running cordova platform add android

Read first for creating cordova project https://cordova.apache.org/docs/en/latest/guide/cli/

Ok, I'm following it step by step. It tells me to use sudo npm install -g cordova to install cordova. Do I have to install it in my app folder?

I really appreciate all your help, by the way :)

-g is for global install so you can run from anywhere
Cordova needs to globally available.
Sent from my iPhone

On Apr 25, 2018, at 22:59, paulamourad<EMAIL_ADDRESS>wrote:
I really appreciate all your help, by the way :)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

Ok, so I followed each step and I got this message. Does it mean it has been correctly installed?

Ok, I added the wrong screen shot. Here's what it showed when I ran the install line:

For plugin list cordova plugin ls
cordova-plugin-barcodescanner doesn't support cordova 7.0 you can use phonegap-plugin-barcodescanner

I figured out that phonegap-plugin-barcodescanner is the wrong plugin :)
I've installed the cordova-plugin-qrscanner and I hope this will work.
Thank you so much for all your help!!!!! 😃

Awesome!, You can close this issue now

Will do! Thanks! Best of luck!


Update version number to 1.1.0

previous version update only changed the suffix

@ryanbrandenburg I think we got lucky because no new 1.0.0-preview1 packages are listed in the MyGet gallery e.g. see https://dotnet.myget.org/feed/aspnetcore-dev/package/nuget/Microsoft.AspNetCore.WebHooks.Receivers


⚠ Bofrost.es has degraded performance

In 2f24688, Bofrost.es (https://www.bofrost.es) experienced degraded performance:

HTTP code: 200
Response time: 1737 ms

Resolved: Bofrost.es performance has improved in 3c4f08a after 42 minutes.


[FALSE-POSITIVE] bancointer.com.br

Domains or links
Please list any domains and links listed here which you believe are a false positive.
bancointer.com.br
More Information
How did you discover your web site or domain was listed here?
We were warned by our digital risk monitoring tool.
Have you requested removal from other sources?
Please include all relevant links to your existing removals / whitelistings.

Additional context
Add any other context about the problem here.
We are a brazilian bank institution and this is our main website.
Trademark number registered at INPI: 912833130
If you require any additional info, please reach us at<EMAIL_ADDRESS>

Thanks @lfbrand90 for reporting. It has been added to the whitelist and will be stripped out on every build from hereon. Will do a removal from the source lists in the morning which will eventually clear it out of all lists here.

Whitelisted also on our combined input sources files we use.


Callback on dynamic route

Callbacks don't pass the parameters in a dynamic route. Don't know if this is a bug or an unimplemented feature, either way it is an issue.
Take the snippet below (crudely written, apologize for any syntax errors)
get('/user/$user' function($id) {
    // your code here n stuff
});

This has been fixed. Now this works:
get('/callback/$name', function($name){
echo "Callback executed. The name is $name";
});


How to use with Webpack?

I can't work out how to import this into an existing ES6 project using webpack. Could you provide an example? The nearest I've got so far is:
import * as d3 from "d3";
import { xyzoom } from "d3-xyzoom"

however when doing this I encounter this error https://github.com/d3/d3-zoom/issues/27 .

Unfortunately, I don't use this plugin with a Webpack configuration.
However, this webpack plugin might be a good start to find a solution.

@etiennecrb thanks! I'll take a look at that and see if it solves my problem.

so i just wanted to make a note about this particular situation. I'm using D3 v5 and webpack. The way i got it working was import D3 and this package as normal.
import * as d3 from "d3"; import * as d3xyzoom from "d3-xyzoom";
Now d3-xyzoom is using independently of my normal D3 modules, but works as expected just fine.
//example code let xy = d3xyzoom.xyzoomTransform(this.some-selection.node())

One more note.
To import d3-xyzoom as d3 module I used these lines:
import * as d3 from "d3";
import * as d3xyzoom from "d3-xyzoom";
Object.assign(d3, d3xyzoom);

This way it will work like in examples.
const zoom = d3.xyzoom()


Transfer gets stuck forever

I noticed that transfer can get stuck forever. When this happens TCP connection remains open on the client side for many days without transmitting any data and without any active TCP timers:
# ss -e -4 -i -s
[...]
Netid State      Recv-Q Send-Q  Local Address:Port Peer Address:Port
[...]
tcp   ESTAB      0      0        <IP_ADDRESS>:58076       <IP_ADDRESS>:8056
 ino:157759 sk:3 <->
         ts sack cubic wscale:7,7 rto:8120 rtt:2540.5/1392.48 ato:40 mss:1340 cwnd:10 bytes_acked:112 segs_out:7 segs_in:4 send 42.2Kbps lastsnd:1284511270 lastrcv:1284517070 lastack:1284510250 pacing_rate 84.4Kbps retrans:0/2 rcv_space:29200

Same problem reproduced some other time:
# ss -e -4 -i -s
[...]
Netid State      Recv-Q Send-Q  Local Address:Port Peer Address:Port
[...]
tcp   ESTAB      0      0    <IP_ADDRESS>:33177 <IP_ADDRESS>:8056
         ino:1441666 sk:d520e400 <->
         ts sack cubic wscale:7,7 rto:950 rtt:213.75/177.5 mss:1340 cwnd:2 ssthresh:7 send 100.3Kbps lastsnd:388096380 lastrcv:388220470 lastack:388096300 retrans:0/7 rcv_space:14600

I have no idea how to reproduce this problem. I've spent a lot of time trying to understand this issue, but I failed. It looks like some callback did not fire and new data was not added to outgoing buffer, but I can't prove that. I hope that maybe someone has seen something similar and might help.
Here is some more information about the case:

When problem occurs, I can see that server logs successful POST, but I don't know at what exactly stage upload gets stuck.
My TUS client's software runs on Linux with node v6.12.3. Transfer is done over VPN and unstable 3G/2G connection.
Problem occured with tus-js-client in version 1.4.5 with small patch that allows usage of custom node local storage (I don't think it is related to the issue),  but also in the same version with another extra patch (https://github.com/tus/tus-js-client/pull/125) and keepalive enabled in agent.
Data passed to TUS upload is kept in Buffer.

As a workaround I've added a timer that calls upload.abort() if transfer does not finish within some defined period of time, but that revealed https://github.com/tus/tus-js-client/issues/138.

Do you have another idea how this problem could be fixed/prevented? Because I can't think of any other solution right now.

I'm having a very similar problem/bug. In Firefox 77, when I toggle the debug console (F12) while a file is uploading, the upload suddenly stucks forever. Sometimes it's stucking directly after the first toggle, sometimes after the 3 or 4th toggle. But it stalls. forever. No event, no transmission... nothing. I haven't tested this effect on other browsers yet, but it's a way I'm able to reproduce the problem. My uploader configuration is as follows:
var myworker = new tus.Upload(file, {
  endpoint: './upload.php',
  chunkSize: 1048576,
  headers: {
    'Upload-Length': filesize,
    'Upload-Filename': filename
  },
  onError: errorcb,
  onProgress: progresscb,
  onSuccess: successcb
});

myworker.start()

So, nothing special here... Any suggestions how to fix this?

@mstimvol That's interesting. I am also running Firefox 77 but I am not able to reproduce this. Frankly, it more sounds like a bug in Firefox for me and not something we can do much about.

Closing this issue due to inactivity. Feel free to leave a comment if you want to continue the discussion :)


All text is white on the documentation site

What version of Turborepo are you using?
x
What package manager are you using / does the bug impact?
npm
What operating system are you using?
Mac
Describe the Bug
What I see:

Disabling the color-scheme rule (what I'd expect to see):

Expected Behavior
Text is not white over white background
To Reproduce

MacOS - Set either light/dark-mode on the system settings
Chrome v102
Open https://turborepo.org/docs

Hmm, I can't seem to reproduce this, and I think I have the same setup. Did you have to relaunch chrome to trigger the issue? Or did it occur right after switching macos system settings?

Interesting, I think this is some extension's fault. I'm sorry for the noise.


add feature decideMediaCapturePermissions on ios15+

Connection with issue(s)
Add support onPermissionRequest on iOS 15+.
You can decide grant or deny request permission camera or microphone as android had.
Used inside androidOnPermissionRequest.
To Do
refactor name androidOnPermissionRequest to onPermissionRequest

[ ] double check the original issue to confirm it is fully satisfied
[ ] add testing notes and screenshots in PR description to help guide reviewers
[ ] request the "UX" team perform a design review (if/when applicable)

@pichillilorenzo Can this feature appear in the next release?

@pichillilorenzo May I ask has this feature appeared in the latest release?

it requires plugin version 6.x.x


User Label aka avatar-letter and Cyryllic encoding

Expected behaviour
avatar-letter should include first letters of username parts.
Actual behaviour
There are avatar-letters now include two "?" for all users with the same color in all tasks.
Screenshot is attached.
Steps to reproduce
Cyryllic usernames work wrong.
Latinic usernames work perfectly.
Configuration

Kanboard version:1.0.27
Database type and version:3.6.20  sqlite
PHP version:5.5.30
OS:Linux 2.6.32-431.11.2.el6.x86_64
Browser:Mozilla/5.0 (Windows NT 6.1) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/49.0.2623.110 Safari/537.36

I tested with several Russian names and it's working fine for me. Which operating system do you have on the server?

Hi,
-bash-4.1$ uname -a
Linux saswiki 2.6.32-431.11.2.el6.x86_64 #1 SMP Mon Mar 3 13:32:45 EST 2014 x86_64 x86_64 x86_64 GNU/Linux

method Kanboard\Helper\UserHelper:getInitials() is used mb_substr.
you must add
mb_internal_encoding ( "UTF-8");

That should be fixed in the development branch. Thanks.

You can now use Kanboard 1.0.29, reopen this issue if necessary.


Text Size

Been customising my buttons quite heavily and using custom SF Icons and creating icons using the stream deck key creator the issue I have is the the text size no matter what I change it to either disappears completely as its set too high or it cant be seen as text_size: 12 goes microscopic. Is there something that can be tweaked to say leave text alone every time the deck refreshes.

Did you manage to fix it? And if so, how?

I thought I had by sizing the icons below where I wanted the text but unfortunately after checking today the text again has automatically changed size. I have a few theories and I've a few more adjustments to make to see if I can find what exactly is causing it.

It must be the automatic resizing of icons when loading an icon from file.

Previously we resized when downloading a file but not when loading a file from disk.

The text shows up perfectly then the file refreshes and it shows up microscopic text. I'll keep at it to see if I can find a solution or try figure out what's causing it

Can you share the button config? And the image file? I'll try to reproduce and fix it.

Right now the images (after update of yesterday) are automatically resized to the native size of the StreamDeck.
So text_size should be uniform across all images.

I am away at the moment once I'm back I will be able to have a look at it and I'll update you. Thanks for your help :)

Both icons have same text size and offset
text_size: 14
text_offset: -15

Thanks for sharing.
And are both loaded via a local file?

Also, which version are you using?
You'll see it printed as one of the first things in your log.

It may have been the fact that I was using icons I had saved on my server I switched them all over locally to the pi that's running the add on and the text seems to appear correctly now. The add on is updated to 2023.8.5.

Happy to hear that it’s fixed now!
Which installation method did you use BTW?


animation breaks after switching orientation

The background color of the toolbar is not shown after switching the orientation of the device.

What do you mean by that? It makes only background color disappear? Or are you refering to hiding FAB to normal after orientation change?


[Fabric] Shim RCTPixelValue to take scale factor

Please select one of the following

[ ] I am removing an existing difference between facebook/react-native and microsoft/react-native-macos :thumbsup:
[x] I am cherry-picking a change from Facebook's react-native into microsoft/react-native-macos :thumbsup:
[ ] I am making a fix / change for the macOS implementation of react-native
[ ] I am making a change required for Microsoft usage of react-native

Summary
Fabric on macOS implementation:
Shim RCTPixelValue to take scale factor
Changelog
[macOS][Fabric] - Shim RCTPixelValue to take scale factor
Test Plan
[x] Build RNTester-macOS w/ Fabric - doesn’t run yet, but no RCTPixelValue errors

Build errors:
Build RNTester-macOS_2022-11-28T16-13-01.txt
[x] Build RNTester - iOS w/ Fabric

[x] Build RNTester-macOS w/ Paper - should work

[x] Build RNTester - iOS w/ Paper - should work

53587dce8e636d4c7c3d55d02dc88befa2cc6255 will be removed once it's merge into main

closes #1555 #1561


ARROW-3273: [Java] Fix checkstyle for Javadocs

Enable checkstyle rules for Javadoc and apply fixes. Suppressing missing Javadocs for now because there are to many to add here. Also, changed the checkstyle DTD links to point to more official files located at checkstyle.org. This was done because to suppress a check with a specific message required an update to the suppress filter DTD.

+1 from me.

Thanks @icexelloss! I'll leave this open for a bit longer in case @jacques-n or @siddharthteotia have any comments.

Java tests passed, CI errors look like they are due to unrelated C++ compilation

merged to master


Some datamodules use mnist_transforms variable name in code and docs

Description
CIFAR10, TinyCIFAR10DataModule, SSLImagenet, STL10 data modules use mnist_transforms variable name in documentation. When I was reading the docs for CIFAR10 that caused a little confusion. When I was inspecting the code, I noticed that for SSLImageNetDataModule, the variable used to store transformations is also named mnist_transforms. I believe a typo was introduced by copy-paste.
Files with the typo:

pl_bolts\datamodules\cifar10_datamodule.py
pl_bolts\datamodules\ssl_imagenet_datamodule.py
pl_bolts\datamodules\stl10_datamodule.py

How to fix
The most straightforward fix would be updating the docs and updating the variable's name to transforms. We are already in the class that names the dataset, so there is no need to prefix variables. That would also help to avoid similar errors in the future.

I can fix it but I cannot assign myself and create branch. So if someone can help me with that I would appreciate.


intword added to documentation

It adds the definition of intword function in the readme file. This obvious part was missing in the documentation.

Closing this, since the library is not maintained at present.


Delta file not applied when use VMware Vsphere VM snapshots

Dear Sirs,
we are using Backupvss writer to backup Domino V14.0 FP2 IF1 on Windows server 2022 VM, VMWARE V7 and VEEAM V12.
We configured Veeam as describe in work-arounds number 2 of this comment https://github.com/HCL-TECH-SOFTWARE/domino-backup/issues/23#issuecomment-2348588115.
We modified VMware Tools on VM adding Volume Shadow Copy Services Support but Domino don't write delta information directly into the snapshot as if auto-recovery option was not enabled:
28/11/2024 23:30:44   Backup: Domino Database Backup
28/11/2024 23:30:44   Backup: Started
28/11/2024 23:30:44   BackupVSS: Domino Agent VSS Freeze Event received
28/11/2024 23:30:44   Backup: Pruning backups
28/11/2024 23:31:02   Backup: BackupNode: [], BackupName: [default], Translog Mode: [CIRCULAR],  Backup Mode: [SNAP]
28/11/2024 23:31:02   Backup: LastBackupTime: 26/11/2024 23:31:14
28/11/2024 23:31:06   Backup: Starting backup for 340 database(s) [SnapShotMode]
28/11/2024 23:31:10   Backup: Ready for VSS Snapshot
28/11/2024 23:31:10   BackupVSS: DominoVSSWriter::OnFreeze Domino in Freeze Mode after 23 seconds!
28/11/2024 23:31:15   Backup: VSS Post Snapshot status reached after 4 seconds!
28/11/2024 23:31:15   Backup: Capturing changes in 340 database(s) [SnapShotMode]
28/11/2024 23:31:15   BackupVSS: Sending unfreeze to Domino Backup
28/11/2024 23:31:16   Backup: VSS Snapshot Backup finalized
28/11/2024 23:31:16   BackupVSS: DominoVSSWriter::OnPostSnapshot Post backup changes applied after 1 seconds.
28/11/2024 23:31:25   BackupVSS: DominoVSSWriter::OnBackupComplete Updating backup completed time
28/11/2024 23:31:25   BackupVSSCompletedTimeUpdate: waiting for backup to complete
28/11/2024 23:31:27   Backup:
28/11/2024 23:31:27   Backup: --- Backup Summary ---
28/11/2024 23:31:27   Backup: Previous Backup  : 26/11/2024 23:31:14
28/11/2024 23:31:27   Backup: Start Time       : 28/11/2024 23:31:02
28/11/2024 23:31:27   Backup: End Time         : 28/11/2024 23:31:16
28/11/2024 23:31:27   Backup: Runtime          : 00:00:14.31
28/11/2024 23:31:27   Backup:
28/11/2024 23:31:27   Backup: All              :   340
28/11/2024 23:31:27   Backup: Processed        :   340
28/11/2024 23:31:27   Backup: Excluded         :     0
28/11/2024 23:31:27   Backup: Pending Compact  :     0
28/11/2024 23:31:27   Backup: Compact Retries  :     0
28/11/2024 23:31:27   Backup: Backup Errors    :     0
28/11/2024 23:31:27   Backup: Not Modified     :     0
28/11/2024 23:31:27   Backup: Delta Files      :     2
28/11/2024 23:31:27   Backup: Delta applied    :     0
28/11/2024 23:31:27   Backup:
28/11/2024 23:31:27   Backup: Total DB Size    :     1,0 TB
28/11/2024 23:31:27   Backup: Total DeltaSize  :     1,4 KB
28/11/2024 23:31:27   Backup: Data Rate        : 75.546,5 MB/sec
28/11/2024 23:31:27   Backup: --- Backup Summary ---
28/11/2024 23:31:27   Backup:
28/11/2024 23:31:27   Backup: Finished
In the log we see that there are 2 Delta Files but 0 Delta applied.
Why delta files are not applied to snapshot? Where am i wrong?
Please help
Roberto

You should not have any delta files which are generated. Delta Files should be applied into the Auto Recovery snapshot.
Can you please be very specific which of the configuration you used from the mentioned other issue?
I outlined multiple options which work.
Issue #23 describes the background of the problem we are currently facing with Veeam in one configuration option and describes in detail the different configuration options that work.
You can't just reference another issue which has multiple config options and you say you have configured it like that.
Which configuration are you using exactly of the outlined configuration options?

Hy Daniel,
we used this work-around:
"2. Use VMware Vsphere VM snapshots
The other solution currently is to use a native VMware snapshot.
This will only work with a virtual machine where the disks to snapshot are configured as dependent disks.
Because only dependent disks are snapshotted along with the VM.
In addition VMware tools need to be installed on the machine (which should be the case on all Windows machines).
For this type of configuration the standard configuration for application aware processing needs to be disabled and the VMware snapshot configuration needs to be enabled.
Disable Enable application-aware processing on Guest Processing tab of the backup job.
On the Storage tab select Advanced...
Open the vSphere tab
Select Enable VMware Tools quiescence
Enable all Change blog tracking options below on the same screen (should be already enabled in most cases)
On the VMware side make sure the disk is a dependent disk.
In addition make sure the setting Quiesce guest file system (needs VMware tools installed). needs to be selected in the VM configuration. "
With this method, 0 delta files are applied to snapshot.
Thanks
Roberto

OK if you are using this option, the next part to check is if you are running as the Local Service user instead of the System account.
Domino 14.0 starts as Local Service without admin rights. But the server needs to be able to mount the disk.
Can you clarify if you are using an account with admin rights?
There is no other way on Windows to have admin rights to mount the snapshot.

I'm running HCL Domino Server service as Local Service.
The windows account I use to access the operating system has admin rights.

You need to use the Local System account. The server has to perform the snapshot mount in the Post Snapshot even.
There should be logging that it failed. But clearly there should be a way to tell backup to only use Auto Recovery snapshots and fail if this does not success. Right now it continues and fails back to try a delta file snapshot.

In the next few days i change Local Service in Local System to run HCL Domino Server service and and I'll let you know.
One question on domino backup configuration document, i use Windows Default Configuration where i changed Config type=Server and backup target, are there other parameter to setting?

you should use the Domino 12.0.2 Veeam integration DXL config file.
The VSS backup does not need anything special. It will just work because the backupvss server task does all the work.
backupvss is a hybrid process implementing the VSS Writer and is a Domino server task on the other side.
Once the data is backed-up with Auto Recovery snapshots, the data is consistent in the backup.
So you could just mount it via Veeam and copy it back.
But there is an end to end restore option, which needs the right configuration and the right scripts in place to mount the Veeam backup.
Please take a look at this document. I rewrote this with just the VSS requirements.
The earlier document is still relevant for Linux.
But for Windows VSS in Domino 12.0.2 made our life much easier on the backup side.
So we are recommending VSS for Windows and don't support the older approach we had for earlier versions.
The Auto Recovery part of VSS is really a game changer to get a consistent backup.

I imported Domino 12.0.2 Veeam integration DXL config file and i changed Local Service in Local System to run HCL Domino Server service and now delta files are applied successfully:
06/12/2024 23:30:44   BackupVSS: Domino Agent VSS Freeze Event received
06/12/2024 23:30:44   BackupVSS: backup command line: Binary path: [] Param: [-vss]
06/12/2024 23:30:44   Backup: Domino Database Backup
06/12/2024 23:30:44   Backup: Started
06/12/2024 23:30:44   Backup: Pruning backups
06/12/2024 23:30:45   Backup: BackupNode: [SRV_CIMA], BackupName: [default], Translog Mode: [CIRCULAR],  Backup Mode: [SNAP]
06/12/2024 23:30:45   Backup: LastBackupTime: 05/12/2024 23:30:59
06/12/2024 23:30:48   Backup: Starting backup for 340 database(s) [SnapShotMode]
06/12/2024 23:30:53   Backup: Ready for VSS Snapshot
06/12/2024 23:30:53   BackupVSS: DominoVSSWriter::OnFreeze Domino in Freeze Mode after 7 seconds!
06/12/2024 23:30:57   Backup: VSS Post Snapshot status reached after 3 seconds!
06/12/2024 23:30:57   Backup: Capturing changes in 340 database(s) [SnapShotMode]
06/12/2024 23:30:57   BackupVSS: Sending unfreeze to Domino Backup
06/12/2024 23:31:00   Backup: Applied 350,0 Bytes to Backup File [C:\backupvss_snapshot]
06/12/2024 23:31:02   Backup: Delta file apply completed.
06/12/2024 23:31:03   Backup: VSS Snapshot Backup finalized
06/12/2024 23:31:03   BackupVSS: DominoVSSWriter::OnPostSnapshot Post backup changes applied after 5 seconds.
06/12/2024 23:31:10   BackupVSS: DominoVSSWriter::OnBackupComplete Updating backup completed time
06/12/2024 23:31:10   BackupVSSCompletedTimeUpdate: waiting for backup to complete
06/12/2024 23:31:12   Backup:
06/12/2024 23:31:12   Backup: --- Backup Summary ---
06/12/2024 23:31:12   Backup: Previous Backup  : 05/12/2024 23:30:59
06/12/2024 23:31:12   Backup: Start Time       : 06/12/2024 23:30:45
06/12/2024 23:31:12   Backup: End Time         : 06/12/2024 23:31:03
06/12/2024 23:31:12   Backup: Runtime          : 00:00:18.734
06/12/2024 23:31:12   Backup:
06/12/2024 23:31:12   Backup: All              :   340
06/12/2024 23:31:12   Backup: Processed        :   340
06/12/2024 23:31:12   Backup: Excluded         :     0
06/12/2024 23:31:12   Backup: Pending Compact  :     0
06/12/2024 23:31:12   Backup: Compact Retries  :     0
06/12/2024 23:31:12   Backup: Backup Errors    :     0
06/12/2024 23:31:12   Backup: Not Modified     :     0
06/12/2024 23:31:12   Backup: Delta Files      :     0
06/12/2024 23:31:12   Backup: Delta applied    :     1
06/12/2024 23:31:12   Backup:
06/12/2024 23:31:12   Backup: Total DB Size    :     1,0 TB
06/12/2024 23:31:12   Backup: Total DeltaSize  :     0,0 Bytes
06/12/2024 23:31:12   Backup: Data Rate        : 56.991,7 MB/sec
06/12/2024 23:31:12   Backup: --- Backup Summary ---
06/12/2024 23:31:12   Backup:
06/12/2024 23:31:12   Backup: Finished
Thank you very match Daniel.

Great! thanks for coming back to me! I am removing the bug label and close it.
-- Daniel


Refactor common types

As titled.
There are still three folders that need to be reorganized, still trying to figure out where those should go.

[ ] __common/analysis
[ ] __common/index
[ ]  __common/mapping

Currently, __common/common.ts contains a lot of unrelated definitions, we could reorganize them better in the future.

+1 for moving away lots of things out of commons and aggregating into larger files.
Regarding __commons/mapping: there are lot of nested folders that don't bring additional value as they only contain one type definition that also contains the name of their parent folder.
For example, __commongs/mapping/meta_fields/all/AllFields.ts should end up in __commongs/mapping/meta_fields/AllFields.ts (remove the intermediate all directory that is redudant)


make ts_options_validator output deterministc

the program path was absolute, similar to rules_nodejs #3462

always make separate PRs for orthogonal things! :)


Tooltips callback doesn't work in Blazor Server

Describe the bug
Does not work in .NET 6.0 Blazor Server.
To Reproduce
Can reproduce by creating a fresh .NET 6 Blazor Server project, importing latest nuget version, and copying over relevant files from Demo site. I have replicated in this repo here: https://github.com/riley-cochrane/ChartJSTest
Run site
Go to https://localhost:7245/testgraph
Hover over bar on bar graph and no tool tip displays.
Console displays error:
Uncaught Error: The current dispatcher does not support synchronous calls from JS to .NET. Use invokeMethodAsync instead.     at y (blazor.server.js:1:1840)     at e.invokeMethod (blazor.server.js:1:2834)     at config.options.plugins.tooltip.callbacks.title (Chart.js:81:27)     at Tooltip.getTitle (chart.js:11425:35)     at Tooltip.update (chart.js:11510:25)     at Tooltip.handleEvent (chart.js:11875:14)     at Object.afterEvent (chart.js:11936:25)     at callback (chart.js:53:15)     at PluginService._notify (chart.js:6316:11)     at PluginService.notify (chart.js:6303:25)
Any fix for this? Can make local changes to create a new version if necessary.
Thanks,
Riley

On NET 8 working but on NET 9 not


Make flow view scrollable in vertical

I wasn't able to scroll it despite a scrollbar is displayed at right side.

Fixed with commit https://github.com/solettaproject/soletta-dev-app/commit/fa24e972f13e288668565cf3081b834df62f5b71


Use relative paths for non-lib imports

Resolved Issues

Fixes issues with library resolution when importing Hyperdrive from another foundry project.
Seems to speed up LSP (using vscode-solidity-server in nvim)

Pull Request Test Coverage Report for Build<PHONE_NUMBER>7
Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 91.635%

Totals

Change from base Build<PHONE_NUMBER>6:
0.0%

Covered Lines:
2147

Relevant Lines:
2343

💛  - Coveralls


Batch Copy Items Support

Hello, for the Copy Items workflow, an enhancement would be to allow the selection, and drag and drop of multiple items from source to target portal.  Thanks!

@sam-berg Great idea! I'll definitely be investigating that!

Agree this could be useful for mass-copy operations.


Issue 31716 no class def found error for com.nimbusds.jose.shaded.json.json array

Description
As title, Fix #31716
All SDK Contribution checklist:

[x] The pull request does not introduce [breaking changes]
[x] CHANGELOG is updated for new features, bug fixes or other significant changes.
[x] I have read the contribution guidelines.

General Guidelines and Best Practices

[x] Title of the pull request is clear and informative.
[x] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

Testing Guidelines

[x] Pull request includes test coverage for the included changes.

API change check
API changes are not detected in this pull request.

Hi, @saragluna , @moarychan , please help to review this PR.


deepeval and api key

Is your feature request related to a problem? Please describe.
i want to use deepeval with my local model on ollama, but it seems not possibile
Describe the solution you'd like
i want the possibility to use my api key instead of the Open AI key , i want to maintain my data private

open your terminal and shell
deepeval set-local-model --model-name=llama3:8b --base-url="http://localhost:11434/v1/" --api-key="ollama"

In addition to the local model, a support for local embedding would be great.
This is especially relevant for Synthesizer api

Duplicate:

https://github.com/confident-ai/deepeval/issues/1007


feat: SSE WIP

https://github.com/DevCycleHQ/test-harness/pull/420 - this PR needs to be merged first.


freeipmi: 1.6.7 -> 1.6.8

See https://lists.gnu.org/archive/html/info-gnu/2021-05/msg00013.html
for release information

Motivation for this change
Things done

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

[X] NixOS
[ ] macOS
[ ] other Linux distributions

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

Result of nixpkgs-review pr 123844 at af5882a9 run on x86_64-linux 1

2 packages marked as broken and skipped:

python38Packages.pyslurm
python39Packages.pyslurm

5 packages built successfully:

freeipmi
netdata
nut
slurm
slurm-spank-x11


How to define your own task

If i want to define my own simulation task and don't need dataset. Is this all I need to design my own YAML file in the tasks folder and run it?

Right. You can create a custom YAML file based on any existing one in agentverse/tasks/simulation. Then, execute your code using python agentverse_command/main_simulation_cli.py --task your_task_name. If you run into any issues, feel free to reach out for assistance.

OK, I understand. thanks so much.


Add tailRecM and MonadError instance for Decoder

I based the tailRecM and handleErrorWith implementations on the instances of Either and io.circe.Decoder.

Codecov Report

Merging #53 into master will increase coverage by 0.22%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master      #53      +/-   ##
==========================================
+ Coverage   90.31%   90.54%   +0.22%
==========================================
  Files          12       12
  Lines         289      296       +7
  Branches        4        4
==========================================
+ Hits          261      268       +7
  Misses         28       28

Impacted Files
Coverage Δ

modules/cats/package.scala
95.83% <100%> (+1.71%)
:arrow_up:

Continue to review full report at Codecov.

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

I also have an Arrow instance for Decoder but I ran into https://github.com/typelevel/cats/issues/1567, so we can either add it after that's fixed in the next Cats version or add it untested.

Awesome, thank you, and LGTM!


Build in Ubuntu 18.04 container in 20.04 image

This is due to GitHub deprecating the Ubuntu 18.04 image. This commit fixes #1667.
Checklist

[x] Each commit message has a non-empty body, explaining why the change was made
[ ] Methods or procedures are documented
[ ] Regression or unit tests are included, or existing tests cover the modified code
[x] My PR is restricted to a single feature or bugfix

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 and MIT licenses.

Is it actually a good idea to continue doing this? 18.04 is not going to see security updates either.

Can we stop supporting Ubuntu 18.04?

@tautschnig @celinval building on 18.04 is still required for some use cases. There is also the fairly urgent issue of Diffblue no longer shipping 18.04 binaries, which this PR also seeks to address.


error installing platformio platform

platformio platform uninstall https://github.com/kuba2k2/libretuya

WARNING: This command is deprecated and will be removed in the next releases.
Please use `pio pkg uninstall` instead.

Error: Could not find the package with 'https://github.com/kuba2k2/libretuya' requirements for your system 'linux_x86_64'
carmelo@carmelo-desktop:~/Desktop/tuya-cloudcutter$ platformio platform install -f https://github.com/kuba2k2/libretuya

WARNING: This command is deprecated and will be removed in the next releases.
Please use `pio pkg install` instead.

Platform Manager: Installing git+https://github.com/kuba2k2/libretuya
git version 2.34.1
Cloning into '/home/carmelo/.platformio/.cache/tmp/pkg-installing-i8188zqg'...
remote: Enumerating objects: 650, done.
remote: Counting objects: 100% (650/650), done.
remote: Compressing objects: 100% (446/446), done.
remote: Total 650 (delta 188), reused 529 (delta 173), pack-reused 0
Receiving objects: 100% (650/650), 438.17 KiB | 207.00 KiB/s, done.
Resolving deltas: 100% (188/188), done.
Platform Manager<EMAIL_ADDRESS>has been installed!
Installing/updating ltchiptool
Defaulting to user installation because normal site-packages is not writeable
Requirement already satisfied: ltchiptool in /home/carmelo/.local/lib/python3.10/site-packages (1.7.0)
Collecting ltchiptool
  Downloading ltchiptool-2.0.2-py3-none-any.whl (118 kB)
     ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 118.9/118.9 KB 213.1 kB/s eta 0:00:00
Requirement already satisfied: prettytable<4.0.0,>=3.3.0 in /home/carmelo/.local/lib/python3.10/site-packages (from ltchiptool) (3.5.0)
Requirement already satisfied: click<9.0.0,>=8.1.3 in /home/carmelo/.local/lib/python3.10/site-packages (from ltchiptool) (8.1.3)
Requirement already satisfied: importlib-metadata<5.0.0,>=4.12.0 in /home/carmelo/.local/lib/python3.10/site-packages (from ltchiptool) (4.13.0)
Requirement already satisfied: pycryptodomex<4.0.0,>=3.15.0 in /home/carmelo/.local/lib/python3.10/site-packages (from ltchiptool) (3.16.0)
Requirement already satisfied: colorama<0.5.0,>=0.4.5 in /home/carmelo/.local/lib/python3.10/site-packages (from ltchiptool) (0.4.6)
Collecting bk7231tools<2.0.0,>=1.2.1
  Downloading bk7231tools-1.2.3-py3-none-any.whl (27 kB)
Collecting xmodem<0.5.0,>=0.4.6
  Downloading xmodem-0.4.6.tar.gz (32 kB)
  Preparing metadata (setup.py) ... done
Requirement already satisfied: pyserial<4.0,>=3.5 in /home/carmelo/.local/lib/python3.10/site-packages (from bk7231tools<2.0.0,>=1.2.1->ltchiptool) (3.5)
Requirement already satisfied: zipp>=0.5 in /usr/lib/python3/dist-packages (from importlib-metadata<5.0.0,>=4.12.0->ltchiptool) (1.0.0)
Requirement already satisfied: wcwidth in /home/carmelo/.local/lib/python3.10/site-packages (from prettytable<4.0.0,>=3.3.0->ltchiptool) (0.2.5)
Building wheels for collected packages: xmodem
  Building wheel for xmodem (setup.py) ... done
  Created wheel for xmodem: filename=xmodem-0.4.6-py3-none-any.whl size=34564 sha256=18ed5de0122c8065d5317eb90efbe230352103e469b2e0bbdaf84843659b14e9
  Stored in directory: /home/carmelo/.cache/pip/wheels/19/00/c7/cff2d8b8981235aa42bcd09109a401169f9726df47518ada6b
Successfully built xmodem
Installing collected packages: xmodem, bk7231tools, ltchiptool
  Attempting uninstall: bk7231tools
    Found existing installation: bk7231tools 1.1.5
    Uninstalling bk7231tools-1.1.5:
      Successfully uninstalled bk7231tools-1.1.5
  Attempting uninstall: ltchiptool
    Found existing installation: ltchiptool 1.7.0
    Uninstalling ltchiptool-1.7.0:
      Successfully uninstalled ltchiptool-1.7.0
Successfully installed bk7231tools-1.2.3 ltchiptool-2.0.2 xmodem-0.4.6
Error: Traceback (most recent call last):
  File "/home/carmelo/.local/lib/python3.10/site-packages/platformio/__main__.py", line 102, in main
    cli()  # pylint: disable=no-value-for-parameter
  File "/home/carmelo/.local/lib/python3.10/site-packages/click/core.py", line 1130, in __call__
    return self.main(*args, **kwargs)
  File "/home/carmelo/.local/lib/python3.10/site-packages/click/core.py", line 1055, in main
    rv = self.invoke(ctx)
  File "/home/carmelo/.local/lib/python3.10/site-packages/platformio/cli.py", line 71, in invoke
    return super().invoke(ctx)
  File "/home/carmelo/.local/lib/python3.10/site-packages/click/core.py", line 1657, in invoke
    return _process_result(sub_ctx.command.invoke(sub_ctx))
  File "/home/carmelo/.local/lib/python3.10/site-packages/click/core.py", line 1657, in invoke
    return _process_result(sub_ctx.command.invoke(sub_ctx))
  File "/home/carmelo/.local/lib/python3.10/site-packages/click/core.py", line 1404, in invoke
    return ctx.invoke(self.callback, **ctx.params)
  File "/home/carmelo/.local/lib/python3.10/site-packages/click/core.py", line 760, in invoke
    return __callback(*args, **kwargs)
  File "/home/carmelo/.local/lib/python3.10/site-packages/click/decorators.py", line 26, in new_func
    return f(get_current_context(), *args, **kwargs)
  File "/home/carmelo/.local/lib/python3.10/site-packages/platformio/commands/platform.py", line 177, in platform_install
    ctx.invoke(
  File "/home/carmelo/.local/lib/python3.10/site-packages/click/core.py", line 760, in invoke
    return __callback(*args, **kwargs)
  File "/home/carmelo/.local/lib/python3.10/site-packages/platformio/package/commands/install.py", line 65, in package_install_cmd
    install_global_dependencies(options)
  File "/home/carmelo/.local/lib/python3.10/site-packages/platformio/package/commands/install.py", line 77, in install_global_dependencies
    pm.install(
  File "/home/carmelo/.local/lib/python3.10/site-packages/platformio/package/manager/platform.py", line 52, in install
    p = PlatformFactory.new(pkg)
  File "/home/carmelo/.local/lib/python3.10/site-packages/platformio/platform/factory.py", line 78, in new
    cls.load_platform_module(
  File "/home/carmelo/.local/lib/python3.10/site-packages/platformio/platform/factory.py", line 37, in load_platform_module
    return load_python_module("platformio.platform.%s" % name, path)
  File "/home/carmelo/.local/lib/python3.10/site-packages/platformio/compat.py", line 79, in load_python_module
    spec.loader.exec_module(module)
  File "<frozen importlib._bootstrap_external>", line 883, in exec_module
  File "<frozen importlib._bootstrap>", line 241, in _call_with_frames_removed
  File "/home/carmelo/.platformio/platforms/libretuya/platform.py", line 48, in <module>
    ltchiptool.util.lt_set_path(dirname(__file__))
AttributeError: module 'ltchiptool.util' has no attribute 'lt_set_path'

ubuntu 22 not virtualized! worked before latest update

Post the output of python -V, pip -V and ltchiptool -V.

Confirming I get the same error.
PS C:\Dev\libretuya-esphome> python -V
Python 3.10.8
PS C:\Dev\libretuya-esphome> pip -V
pip 22.3.1 from C:\Users\Cossid\AppData\Local\Programs\Python\Python310\lib\site-packages\pip (python 3.10)
PS C:\Dev\libretuya-esphome> ltchiptool -V
ltchiptool v2.0.2

Strange, ran the install a second time and it looks like it worked fine.

Most likely importlib.reload() didn't work properly (it didn't load the new version of ltchiptool).

i can confirm rerunning the installer fix the problem


Dependency package puppeteer has a new version available

We have identified a dependency on version 13.7.0 of puppeteer. A new version (14.0.0) is available for upgrade.
Following are the steps to upgrade package dependency.

Understand the breaking changes between the version being used and the version you want to upgrade to.

Identify all packages that take a dependency on this package.

Go to the root folder for each such package (/sdk/service-name/package-name) and update package.json to have the new version.

Run rush update to ensure the new version is pulled in.

Make relevant changes to absorb the breaking changes.

Repeat steps 3 to 5 for each of the packages that have a dependency on this package.

Label prediction was below confidence level 0.6 for Model:ServiceLabels: 'Azure.Core:0.28339764,Service Bus:0.17050031,Azure.Identity:0.05797889'

@jeremymeng can you look into how big this is?

I am not sure what do we use this for now. Some packages depend on it, but not all
sdk/batch/batch/package.json:56:    "puppeteer": "^10.1.0",
sdk/core/core-rest-pipeline/package.json:130:    "puppeteer": "^13.5.1",
sdk/core/core-http/package.json:168:    "puppeteer": "^13.5.1",
sdk/core/core-amqp/package.json:112:    "puppeteer": "^13.5.1",
sdk/core/logger/package.json:99:    "puppeteer": "^13.5.1",
sdk/web-pubsub/web-pubsub-express/package.json:96:    "puppeteer": "^13.5.1",
sdk/web-pubsub/web-pubsub/package.json:112:    "puppeteer": "^13.5.1",
sdk/eventhub/event-hubs/package.json:175:    "puppeteer": "^13.5.1",
sdk/identity/identity-cache-persistence/package.json:85:    "puppeteer": "^13.5.1",
sdk/identity/identity-vscode/package.json:85:    "puppeteer": "^13.5.1",
sdk/identity/identity/package.json:152:    "puppeteer": "^13.5.1",
sdk/servicebus/service-bus/package.json:171:    "puppeteer": "^13.5.1",
sdk/keyvault/keyvault-keys/package.json:146:    "puppeteer": "^13.5.1",
sdk/keyvault/keyvault-secrets/package.json:143:    "puppeteer": "^13.5.1",
sdk/keyvault/keyvault-certificates/package.json:148:    "puppeteer": "^13.5.1",
sdk/storage/storage-file-share/package.json:170:    "puppeteer": "^13.5.1",
sdk/storage/storage-blob-changefeed/package.json:139:    "puppeteer": "^13.5.1",
sdk/storage/storage-file-datalake/package.json:166:    "puppeteer": "^13.5.1",
sdk/storage/storage-queue/package.json:162:    "puppeteer": "^13.5.1",
sdk/storage/storage-blob/package.json:180:    "puppeteer": "^13.5.1",
sdk/storage/storage-internal-avro/package.json:102:    "puppeteer": "^13.5.1",

The previous upgrade looked easy: https://github.com/Azure/azure-sdk-for-js/pull/20782

Seems like karma pulls this in for browser testing? I wonder if we could eventually migrate away to playwright

Seems like karma pulls this in for browser testing?

Right, we basically use it to install chrome for browser testing.  all browser tests need it but probably due to how rush/pnpm links to shared downloads, those packages that don't depend on puppeteer could still load the module somehow.

while investigating this I went ahead and opened PR https://github.com/Azure/azure-sdk-for-js/pull/21837.  We need to drop nodejs v12 from our test matrix first though.

We're dropping support for node 12 in October, right? April + 6 months? Though presumably we could stop testing in it before then.

It turns out that we actually install and build using node v14, then runs on v12.  So the pipelines are passing


BART Summary length

❓ Questions and Help
Before asking:

search the issues.
search the docs.

What is your question?
Hi
I am trying to play with the length of the generated summary. It seems that there are several parameters relevant to the length in bart.sample(): min_len, max_len_a, max_len_b.
Am I correct?
I have played with different values max_len_a, but I get the same summary for different values of max_len_a.
Thanks
Code

What have you tried?
What's your environment?

fairseq Version (e.g., 1.0 or master):
PyTorch Version (e.g., 1.0)
OS (e.g., Linux):
How you installed fairseq (pip, source):
Build command you used (if compiling from source):
Python version:
CUDA/cuDNN version:
GPU models and configuration:
Any other relevant information:

As you note there are three flags: min_len, max_len_a and max_len_b. The final output size will be constrained by these:
min_len <= output_len <= (max_len_a * input_len) + max_len_b

The default value for max_len_b is 200, so you probably want to increase that. You can also increase min_len if you want to force the model to generate longer outputs.
Finally you can try adjust lenpen. Values > 1 encourage the model to produce longer outputs, so you can try lenpen=1.5.


"CentOS-7" option missing

"CentOS-7" is missing as possible option

Closing as CentOS-7, and some other legacy builds, have been included as archived versions that can also be selected in the config.
If you find any other build that was supported at some point, and is not present in the latest releases, please open a new issue.


Improve error reporting

Two separate commits, which I suggest reviewing separately:

Prunes some unused Jira integration logic that we don't support yet.
Simplify and standardize Jira error messaging so we don't attempt to render entire HTTP response.body to the UI.

Great questions!

When an unhandled exception occurs, it will bubble up to the root handler.  This is okay because it provides about the same amount of information and I want to actively track that code path and cull its number of occurrences to zero.  Judging from review of the logs, the path that I removed is never actually called.

Correct, I'm deliberately removing the response body.  In cases where there's something valid in it, it gets surfaced via the errors JSON handling.  In other cases, it is usually just a ton of unreadable markup, or even worse, unrenderable bytes.

:+1:


feat(deployment.py): add support fo annotations to deployments and carry forward

closes https://github.com/teamhephy/workflow/issues/73
Requires manual edit on the deployment:
kubectl edit deployment rocksolidapp-web -n rocksolidapp

Then edit the part:
...
spec:
  (...skipping top parts...)
  template:
    metadata:
      annotations:
        iam.amazonaws.com/role: role-arn-pods
        nginx.ingress.kubernetes.io/app-root: /rootfs
        sidecar.istio.io/inject: "true"
      creationTimestamp: null
      labels:
        app: rocksolidapp
        heritage: deis
        type: web
        version: v39
      name: rocksolidapp-web
      namespace: rocksolidapp

and add the annotations on the deployment['spec']['template']['metadata']['annotations'] and the controller preserves these annotations for all releases after that. :anchor:
After a new release or scale event the Pods of an app will have this yaml:
apiVersion: v1
kind: Pod
metadata:
  annotations:
    iam.amazonaws.com/role: role-arn-pods
    nginx.ingress.kubernetes.io/app-root: /rootfs
    sidecar.istio.io/inject: "true"
  creationTimestamp: 2018-11-07T14:45:50Z
  generateName: rocksolidapp-web-5b68dc7d85-
  labels:
    app: rocksolidapp
    heritage: deis
(...skipping bottom parts...)

:+1:  :rocket: :shipit:

I have tested this a lot. Works during scale and config:set events as well as new code pushes and any other new releases. You can use this image for testing:
cryptophobia/controller:test_annotations

Merging this in now! Thanks for reviewing guys :+1:


bug: intermittent ArgumentError from ignore_before_process on JRuby

I can't tell what causes this since it's internal code to rollbar which looks fine. Sharing in case this happens to anyone else.

railties <IP_ADDRESS>
sidekiq 6.4.0
rollbar 3.4.0

JRuby <IP_ADDRESS>, ruby 3.1.0
backtrace
2023-08-25T21:11:52.546856+00:00 app[scheduler.7589]: ArgumentError: wrong number of arguments (given 3, expected 4)
2023-08-25T21:11:52.546856+00:00 app[scheduler.7589]: ignore_before_process? at /app/vendor/bundle/jruby/3.1.0/gems/rollbar-3.4.0/lib/rollbar/notifier.rb:150
2023-08-25T21:11:52.546856+00:00 app[scheduler.7589]: log at /app/vendor/bundle/jruby/3.1.0/gems/rollbar-3.4.0/lib/rollbar/notifier.rb:138
2023-08-25T21:11:52.546856+00:00 app[scheduler.7589]: error at /app/vendor/bundle/jruby/3.1.0/gems/rollbar-3.4.0/lib/rollbar/notifier.rb:197
2023-08-25T21:11:52.546857+00:00 app[scheduler.7589]: error at /app/vendor/ruby-3.1.0-jruby-<IP_ADDRESS>/lib/ruby/stdlib/forwardable.rb:238
2023-08-25T21:11:52.546857+00:00 app[scheduler.7589]: display_error_message_with_rollbar at /app/vendor/bundle/jruby/3.1.0/gems/rollbar-3.4.0/lib/rollbar/plugins/rake.rb:21
2023-08-25T21:11:52.546857+00:00 app[scheduler.7589]: standard_exception_handling at /app/vendor/bundle/jruby/3.1.0/gems/rake-13.0.6/lib/rake/application.rb:195
2023-08-25T21:11:52.546857+00:00 app[scheduler.7589]: load_rakefile at /app/vendor/bundle/jruby/3.1.0/gems/rake-13.0.6/lib/rake/application.rb:103
2023-08-25T21:11:52.546857+00:00 app[scheduler.7589]: run at /app/vendor/bundle/jruby/3.1.0/gems/rake-13.0.6/lib/rake/application.rb:82
2023-08-25T21:11:52.546857+00:00 app[scheduler.7589]: standard_exception_handling at /app/vendor/bundle/jruby/3.1.0/gems/rake-13.0.6/lib/rake/application.rb:186
2023-08-25T21:11:52.546857+00:00 app[scheduler.7589]: run at /app/vendor/bundle/jruby/3.1.0/gems/rake-13.0.6/lib/rake/application.rb:80
2023-08-25T21:11:52.546857+00:00 app[scheduler.7589]: <main> at /app/vendor/bundle/jruby/3.1.0/gems/rake-13.0.6/exe/rake:27
2023-08-25T21:11:52.546858+00:00 app[scheduler.7589]: load at org/jruby/RubyKernel.java:1091
2023-08-25T21:11:52.546858+00:00 app[scheduler.7589]: <main> at bin/rake:29
2023-08-25T21:11:52.546858+00:00 app[scheduler.7589]: ... 14 levels...

2023-08-25T21:11:52.547113+00:00 app[scheduler.7589]: ActiveRecord::StatementInvalid: ActiveRecord::JDBCError: org.postgresql.util.PSQLException: This connection has been closed.
2023-08-25T21:11:52.547114+00:00 app[scheduler.7589]: log at /app/vendor/bundle/jruby/3.1.0/gems/activerecord-<IP_ADDRESS>/lib/active_record/connection_adapters/abstract_adapter.rb:699

Rails 7.0.6
Rollbar 3.4.0
jruby <IP_ADDRESS>
ruby 3.1.4
We experienced the same thing. We can't create a minimal example to reproduce the issue but we got around it with this patch.
Rollbar::Notifier.class_eval do
  def use_exception_level_filters?(options)
    option_value = options && options[:use_exception_level_filters]

    return option_value unless option_value.nil?

    configuration.use_exception_level_filters_default
  end
end

The difference is options[:use_exception_level_filters] vs options.delete(:use_exception_level_filters)
These args are passed to log . At this point extra is a hash.
[1] pry(#<Rollbar::Notifier>)> extra
=> {:use_exception_level_filters=>true}

After use_exception_level_filters? is called extra returns a weird error.
[1] pry(#<Rollbar::Notifier>)> extra
ArgumentError: wrong number of arguments (given 0, expected 1)
from /usr/local/rvm/gems/default/gems/pry-0.14.2-java/lib/pry/ring.rb:43:in `<<'
Caused by RuntimeError: foo
from /workspaces/Profile/app/controllers/name_controller.rb:8:in `edit'

When ignore_before_process? is called it's with extra it produces this error.
[2] pry(#<Rollbar::Notifier>)> ignore_before_process?(level, exception, message, extra)
ArgumentError: wrong number of arguments (given 3, expected 4)
from /usr/local/rvm/gems/default/gems/rollbar-3.4.0/lib/rollbar/notifier.rb:158:in `ignore_before_process?'
Caused by RuntimeError: foobar
from /workspaces/Profile/app/controllers/name_controller.rb:8:in `edit'

When plugging in the actual value all is good.
[5] pry(#<Rollbar::Notifier>)> ignore_before_process?(level, exception, message, {:use_exception_level_filters=>true})
=> false

Side note: The method use_exception_level_filters? is a bit misleading as it also mutates the options.

While many consider avoiding mutation of method arguments preferable, this bug appears to be rooted in a Jruby issue. I've submitted Unexpected ArgumentError calling Forwardable delegated single splat method with keyword arguments

Thanks for getting this fixed in JRuby!


⚠ Turnos 24x7 has degraded performance

In 04465b1, Turnos 24x7 (https://reservas-dev.alternativasinteligentes.com/) experienced degraded performance:

HTTP code: 200
Response time: 998 ms

Resolved: Turnos 24x7 performance has improved in c2c6dc0.


Chose a global default for operation mode

DP Requirement ID: APM-02
The aggregation and proxy management process shall allow the selection of Active or Passive operation as a global setting for all connections on the CBSD Proxy Interface.

@jpad-freedomfi should the variable be set while the deployment of DP instance or changeable in runtime (i.e. via API)?

[Joey] variable should be set while the deployment


fix: don't assume bash when displaying interactive CLI prompts

Closes #21838
~Requires influxdata/influx-cli#192 to merge first so we can re-update go.mod to point at main~

Ugh, now needs influxdata/influx-cli#195

Reset and force-pushed to clean up the git history, not sure what happened with all the merges from master


WootingTwo support?

Any plans for WootingTwo support? I tried it with my Wooting Two and it doesn't seem to work; I have a "red box" next to HID Status. Thanks!

I can see that the fork  seems to have WootingTwo support.

yeah that fork is the one converted to the official wooting sdk instead of pure HID. It's also used for an sdk example afaik


chore: upgrade platform-plugin-aspects to v0.9.4

Description
This PR bumps the version of platform-plugin-aspects to v0.9.4

Thanks for the pull request, @Ian2012! Please note that it may take us up to several weeks or months to complete a review and merge your PR.
Feel free to add as much of the following information to the ticket as you can:

supporting documentation
Open edX discussion forum threads
timeline information ("this must be merged by XX date", and why that is)
partner information ("this is a course on edx.org")
any other information that can help Product understand the context for the PR

All technical communication about the code itself will be done via the GitHub pull request interface. As a reminder, our process documentation is here.
Please let us know once your PR is ready for our review and all tests are green.

@Ian2012 Even though your pull request wasn’t merged, please take a moment  to answer a two question survey so we can improve your experience in the future.


No storage display

Description
Sometimes when you open Files the storage isn't shown for the drive.
Steps To Reproduce

Open Files
Storage bar is not shown

Expected behavior
Storage bar should be shown for the drives
Files Version
Version: <IP_ADDRESS> OS Version: 10.0.22000.918
Windows Version
Edition	Windows 11 Home Single Language Version	21H2 Installed on	10-05-2022 OS build	22000.918 Experience	Windows Feature Experience Pack 1000.22000.918.0
Relevant Assets/Logs
debug_fulltrust.log
debug.log

Merging with #9289


Issue with multiple components

Hi @evansmwendwa, thanks for your amazing work. I did an component, using daterangepicker. it seems that if i multiple instances of my component, all the instances are being setup with the same config, even if i pass different inputs for each one. Take a look at my gist.
https://gist.github.com/BenevidesLecontes/7f828eef7a4e6a7aee8b0253079aba6b

The picker is initialized using the options object passed and not the bound input, the bound input has to be manually updated every time the picker changes.
If you share a gist to your template ui-kit.components/ui-date-rangepicker/ui-date-rangepicker.component.html I can try and reproduce the issue.

@evansmwendwa here's my template https://gist.github.com/BenevidesLecontes/10f2dc4f8b492e4480b9be0d29f4d7eb

Try passing your to the datepicker using the options property [options]="{}"

@evansmwendwa it doesn't work as expected neither, it's applying the same settings for all components instances in the page. Did you tried to reproduce the issue? What i'm trying to do is to use my component like this, with slight different settings passed as inputs.

Ex: See this usage:

          <app-date-rangepicker [control]="searchFilterForm" [opens]="'right'" [itemName]="'periodo_criacao'" [id]="'periodo_criacao'"
                                [label]="'Período da criação da ocorrência'"></app-date-rangepicker>

          <app-date-rangepicker [control]="searchFilterForm" [opens]="'left'" [itemName]="'periodo_registro'" [id]="'periodo_registro'"
                                [label]="'Período de registro'"></app-date-rangepicker>

          <app-date-rangepicker [opens]="'center'" [control]="searchFilterForm" [itemName]="'periodo_prazo'" [id]="'periodo_prazo'"
                                [label]="'Período do prazo'"></app-date-rangepicker>

          <app-date-rangepicker (selectedDate)="teste($event)" [control]="searchFilterForm"
                                [itemName]="'periodo_encerramento'"
                                [opens]="'center'"
                                [id]="'periodo_encerramento'"
                                [label]="'Período de encerramento'"></app-date-rangepicker>

Kindly allow me sometime to create an example for you. I will be back online in a few hours. Hopefully I will get it to work with your sample

Your component is not monitoring any change detection, causing all the instances to be created using the default values and ignoring changes in the property binding.
To fix this implement OnChanges and then track the new values there so the datepicker get's updated every time the values change
import { Component, OnInit, OnChanges, OnDestroy, Input  } from '@angular/core';

// implement OnChanges
export class UiDateRangepickerComponent implements AfterViewInit, OnChanges {
  // your input properties
  @Input() id: string;
  @Input() itemName: string;
  @Input() label: string;
  @Input() control: FormGroup;

  // add config object
  public options: any = {};

  // bind all input to the config object here
  // this should support change detection
 ngOnChanges() {
    this.options.opens = this.opens;
    this.options.drops = this.drops;
   // bind all the properties to the options object
   // NB: if you have many inputs consider using a single object and binding to it
  }
}

Then in your ui-date-rangepicker.html template bind [options] to your config object
NB: this is very important in order for the picker to see the different options
NB: since you are using multiple instances no need to use the global config constructor(private daterangepickerOptions: DaterangepickerConfig) unless just for skipping the bundled css

    <input [id]="id" [name]="id"
           [placeholder]="placeholder"
           [formControl]="control.get(itemName)"
           class="form-control uneditable-input input-layout"/>

Example See this plunker for a simple example on the above

@evansmwendwa it's working fine, thanks for your help and your time.


tf_upgrade_v2 needs explicit encodings on Windows to upgrade utf-8

System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow): No
OS Platform and Distribution (e.g., Linux Ubuntu 16.04): Windows 10
Mobile device (e.g. iPhone 8, Pixel 2, Samsung Galaxy) if the issue happens on mobile device:
TensorFlow installed from (source or binary): binary x64
TensorFlow version (use command below): v2.2.0-rc4-8-g2b96f3662b 2.2.0
Python version: 3.8.2
CUDA/cuDNN version: 10.1
GPU model and memory: Tesla K80, 1080ti (11GB per gpu)

Describe the current behavior
Running tf_upgrade_v2 yields encoding errors on Windows, even with defaults set to utf-8 (though no explicit encodings in each file -- as this isn't required in Python 3).
Adding explicit encoding to tensorflow\tools\compatibility\ast_edits.py as shown below resolved this for me -- if a default of utf-8 isn't desirable maybe a command line option?
    # Write to a temporary file, just in case we are doing an implace modify.
    # pylint: disable=g-backslash-continuation
    with open(in_filename, "r", encoding='utf-8') as in_file, \
        tempfile.NamedTemporaryFile("w", delete=False, encoding='utf-8') as temp_file:
      ret = self.process_opened_file(in_filename, in_file, out_filename,
                                     temp_file)
    # pylint: enable=g-backslash-continuation

@sbuser
Please share simple stand alone code for us to replicate the issue faced or is possible share a colab gist for us to analyse the issue.


Fix mimebody DKIM body-hash computation

This PR fixes the DKIM body-hash computation of singlepart and multipart bodies.
I opted for a fn body_raw(&self) -> Vec<u8> method for these bodies, but if it would be preferable with a fn format_body(&self, out: &mut Vec<u8>) instead I could do that change.
This fixes #922

Thanks for the PR, and sorry for the delay in reviewing. I don't mind the extra allocations, although it shouldn't be too much hard to remove them. Can you share the document that you used to determine that this is the right way of signing the bodies?


Use span attributes to enrich stack traces

What is the problem this feature would solve?
Currently stack traces already include span names to make the trace easier to understand, however, it's still lacking some context. I propose to enable a way to enrich printed stack traces to also include the some span attributes (e.g. for a certain span name).

What is the feature you are proposing to solve the problem?
.
What alternatives have you considered?
No response

would printing all attributes be prohibitive?


yml support for configuration files

Yaml will become the preferred serialization format instead of json (perhaps the unique serialization format if we decide to drop the json format).
Pro:

more user friendly
no brackets
less prone to errors (compared to json trailing commas for non technical users)
always parseable by default ({"property": $something} is not parseable and no easy way to implement a validator, where for $something we could have 1, "a string", null for example). property: $something can be validated

Cons:

a large amount of documentation (and third party plugins documentation) will need to be updated
in case in future releases we'll decide to drop json, no more backwards compatible (any strong opinion for maintaining json?)
play_json fixture soft deprecation and fixture rename

Considered useful by users, reopen

Feedback from users: deprecate json and use yaml.
play_json fixture deprecated and it will removed in version >=2.0.0


Getting the following errors on trying to build the project

If you look at the Output window it should tell you what the error is with the 'dotnet tool restore' command.

Closing because no response.


hard data issue

Hi,
I am using the ImageQuilting toolbox to generate realizations of subsurface.
On the attached image, the left figure if the training image, the right figure is the borehole location and the center is one of the realizations.
The windows used in iqsim is 40x40. Apparently, this is much bigger than the actual distance between two boreholes.  How can I use more efficiently the simulation ? Reducing the window or am I doing something wrong?

I am attaching the two files I use as taring image and borehole information.
The code looks like this

TI1=npzread("flummy.npy")
TI1=TI1[:,:,4]
TI1= reshape(TI1,201,201,1)
geo=npzread("real_geo2.npy")
shape = HardData()
for i=1:size(geo,1)
push!(shape, (geo[i,1]+1,geo[i,2]+1,1)=>geo[i,3])
end
reals= iqsim(TI1, 40, 40, 1, size(TI1)..., hard=shape, nreal=1)

Data_for_Marios.zip

Hi @mariosgeo, thank you for reaching out.
I wouldn't recommend image quilting for dense hard data configurations. We investigated this issue in our paper, which I am sharing here. It can work if you reduce the template size, but it becomes more computationally intensive. For dense hard data configurations, pixel-based algorithms like direct sampling are recommended. I didn't have the time to implement direct sampling yet, but meanwhile I think there is a Python implementation on the internet that one of my colleagues used.
If you are actually simulating a 3D domain or a very large 2D domain, then direct sampling and other existing pixel-based algorithms become too computationally intensive. I would try to reduce the template size and activate the GPU with image quilting to see if it helps: http://juliohm.github.io/ImageQuilting.jl/stable/gpu-support.html
Another parameter of image quilting that is set by default for the user, and that also impacts performance, is the overlap configuration. By default the overlaps are set to 1/6 of the template size, in your case ceil(1/6 * 40) --> 7. You may want to reduce this overlap region by decreasing the fraction further.
Please let me know if that clarifies the issue.
Best,

Thanks for the reply and advice.
I tried various  square template sizes (5,8,10,20,40,50, attached the one with template=10), with similar results.

I tried also several overlap settings with the following loop (the title indicates the factor)
`for i=2:20
reals= iqsim(TI1, 10, 10, 1, size(TI1)..., hard=shape,nreal=1,gpu=true,overlapx=1/i,overlapy=1/i,overlapz=1/i)
end`

Is there something more I can try?

I am closing the issue. If you have questions again in the future, please ask in our gitter channel. The issue tracker on GitHub is mainly for bug reports and feature requests.
Thanks,

Thanks for the help and reply


Include base parameters in json file for testing

To be used in test for PR #80

@blakecaldwell is this different from default.json?

@jasmainak yes, this is different. it's just the base params from get_params_default()

@jasmainak please merge for testing of #80 to succeed

Thanks @blakecaldwell


Remove unused package

I want to use this package standalone but it install yajra/laravel-datatables-buttons too.

I think the buttons can be removed, but the validation is actually needed to validate the request.

Can you please share some snippets on how to use this as standalone? Haven't tried it yet. Thanks!

@yajra  I mean use yajra/laravel-datatables-oracle with yajra-laravel-datatables-editor. Don't use html builder just write javascript code

thank's @yajra everything work pretty sweet. One more thing, how i can use editor with tinymce or ckeditor ?


Implement support for 'unsubscribe'

Thanks for the PR! Could you please add a scenario here: https://github.com/s12v/sns/blob/master/features/subscription.feature ? This should improve code coverage.

Codecov Report

Merging #28 into master will decrease coverage by 2.05%.
The diff coverage is 11.11%.

@@            Coverage Diff             @@
##           master      #28      +/-   ##
==========================================
- Coverage   38.81%   36.75%   -2.06%
==========================================
  Files          21       21
  Lines         286      302      +16
  Branches       10       11       +1
==========================================
  Hits          111      111
- Misses        175      191      +16

Impacted Files
Coverage Δ

...scala/me/snov/sns/response/SubscribeResponse.scala
48% <0%> (-6.29%)
:arrow_down:

.../main/scala/me/snov/sns/actor/SubscribeActor.scala
0% <0%> (ø)
:arrow_up:

src/main/scala/me/snov/sns/api/SubscribeApi.scala
77.77% <28.57%> (-17.23%)
:arrow_down:

Continue to review full report at Codecov.

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

I tried adding some tests ... but this was my first exposure to cucumber and scala so please do have a look :)


pre-login Handshake

Issue Type: Bug
A connection was successfully established with the server, but then an error
occurred during the pre-login handshake. (provider: SSL Provider, error: 0 -
The wait operation timed out.)
previous version i'm ok to connect to the server. but after update lately, that server cannot login anymore while i'm login with other software it's work
Azure Data Studio version: azuredatastudio 1.20.1 (fe6312306647441b30102a64bb43abb06ebcf2be, 2020-07-17T02:32:14.131Z)
OS version: Darwin x64 19.5.0

System Info

Item
Value

CPUs
Intel(R) Core(TM) i5-7360U CPU @ 2.30GHz (4 x 2300)

GPU Status
2d_canvas: enabledflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmetal: disabled_offmultiple_raster_threads: enabled_onoop_rasterization: disabled_offprotected_video_decode: unavailable_offrasterization: enabledskia_renderer: disabled_off_okvideo_decode: enabledviz_display_compositor: enabled_onviz_hit_test_surface_layer: disabled_off_okwebgl: enabledwebgl2: enabled

Load (avg)
2, 3, 3

Memory (System)
8.00GB (0.02GB free)

Process Argv
-psn_0_57358

Screen Reader
no

VM
0%

Extensions: none

This looks likely to be caused by https://github.com/microsoft/azuredatastudio/issues/11367#issuecomment-659614111 - please try either installing the server certificate locally or setting TrustServerCertificate to true in the advanced properties and see it that solves your issue.

@Charles-Gagnon thanks boss it's working


If PR #4207 is merged, ensure its generating script is added to the /util directory

Said program has been added to PR #4207.


Getting started page not found

I tried to find some documentation to getting started and the Getting Started page returns me a http 404 error.

Link:
https://instagram.github.io/IGListKit/Guides/Getting Started.md

https://instagram.github.io/IGListKit/getting-started.html

This is how I get that link.
First I click here in the release page:

Then I click on the documentation page the Getting Started link:

And finally I get the http 404 error

🤔 I wonder if this is a Jazzy issue? The sidebar has the correct link
https://instagram.github.io/IGListKit/getting-started.html vs the link in the readme that 404s which is https://instagram.github.io/IGListKit/Guides/Getting%20Started.md

@jessesquires

lol
The README link is relative, so it works when clicked from the README.
jazzy generates the html docs, so then that relative link is broken. will fix.


Added test for unhealthy readiness probe

Approach
In order to be as realistic as possible, the new test utilizes the same setup as the healthy tests in order to properly apply the database migrations and register authentication. It then shuts down the database just prior to calling the readiness probe in order to make it fail.
Changes

Lowered the default timeout for the SQL Server health check to 10 seconds
Removed obsolete IOptionsSectionDefinition interface
Split up health checks into healthy and the new unhealthy checks
Added the ability to shut down the database in the NoPlanApiFactory

Codecov Report

:exclamation: No coverage uploaded for pull request base (main@d4fcfc1). Click here to learn what that means.
 Consider uploading reports for the commit f6e3ef9 to get more accurate results

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #391   +/-   ##
=======================================
  Coverage        ?   85.33%
=======================================
  Files           ?       37
  Lines           ?      607
  Branches        ?        0
=======================================
  Hits            ?      518
  Misses          ?       89
  Partials        ?        0

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

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


pre-commit: PR110880

Link: https://github.com/llvm/llvm-project/pull/110880
Requested by: @nikic

/add-label reviewed
/close


Fix bug of listing vms && add contract Id.

https://github.com/threefoldtech/tfgrid-sdk-ts/issues/1291
Fix bug of deploying solution and find it in failed deployments while it also appears in list.
Add contracts of failed deployments.

There is one case where we won't be able to get the contract id which is if encryption didn't happen. In this case, we need to show a more meaningful message like "unknown contract id as it was not decrypted".

There is one case where we won't be able to get the contract id which is if encryption didn't happen. In this case, we need to show a more meaningful message like "unknown contract id as it was not decrypted".

will it be good to show msg next to deployment name or just not to show with contract as we do with nodeId.

There is one case where we won't be able to get the contract id which is if encryption didn't happen. In this case, we need to show a more meaningful message like "unknown contract id as it was not decrypted".

will it be good to show msg next to deployment name or just not to show with contract as we do with nodeId.

Sure we can do that.

There is one case where we won't be able to get the contract id which is if encryption didn't happen. In this case, we need to show a more meaningful message like "unknown contract id as it was not decrypted".

will it be good to show msg next to deployment name or just not to show with contract as we do with nodeId.

Sure we can do that.

done


routing table is missing entries

When trying to run a pod in a pod manifest, I found the routing table missed a default routing rule.
To reproduce:
Run a pod using pod manifest:
$ sudo ./build-rkt-0.7.0+git/bin/rkt --insecure-skip-verify fetch docker://ubuntu
...
sha512-8f9dc7cea55b7256e33fb4959cd02d4f

$ cat manifest
{
    "acVersion":"0.6.1+git",
    "acKind":"PodManifest",
    "apps":[{
        "name":"worker",
        "image":{"id":"sha512-8f9dc7cea55b7256e33fb4959cd02d4f"},
        "app":{
                "exec":["/bin/sleep", "1000"],
                "user":"0",
                "group":"0"
        }}]
}

$ sudo rkt prepare --pod-manifest manifest
...
bd843b21-0fde-448d-9946-1fc6fce03382

$ sudo rkt run-prepared --mds-register=false --private-net=true bd843b21-0fde-448d-9946-1fc6fce03382

Enter the pod and check routing table:
$sudo rkt enter $UUID
No command specified, assuming "/bin/bash"
root@rkt-bd843b21-0fde-448d-9946-1fc6fce03382:/# ping <IP_ADDRESS>
PING <IP_ADDRESS> (<IP_ADDRESS>): 48 data bytes
ping: sending packet: Network is unreachable
root@rkt-bd843b21-0fde-448d-9946-1fc6fce03382:/# route -n
Kernel IP routing table
Destination     Gateway         Genmask         Flags Metric Ref    Use Iface
<IP_ADDRESS>   <IP_ADDRESS>         <IP_ADDRESS> U     0      0        0 eth0

For some image(docker://yifan/route:v1), even I run without pod manifest, the routing table still misses the default entry.

/cc @eyakubovich

I also ran into some weird cases:

run with --private-net=true

$ sudo ./build-rkt-0.7.0+git/bin/rkt --insecure-skip-verify run --mds-register=false --private-net=true docker://yifan/route:route
...
2015/07/30 17:20:57 Execing /init
[14628.037624] sh[4]: Kernel IP routing table
[14628.037839] sh[4]: Destination     Gateway         Genmask         Flags Metric Ref    Use Iface
[14628.038047] sh[4]: <IP_ADDRESS>   <IP_ADDRESS>         <IP_ADDRESS> U     0      0        0 eth0

run with --private-net

$ sudo ./build-rkt-0.7.0+git/bin/rkt --insecure-skip-verify run --mds-register=false --private-net docker://yifan/route:route
...
015/07/30 17:23:12 Execing /init
[14763.409575] sh[4]: Kernel IP routing table
[14763.409789] sh[4]: Destination     Gateway         Genmask         Flags Metric Ref    Use Iface
[14763.409917] sh[4]: <IP_ADDRESS>         <IP_ADDRESS>   <IP_ADDRESS>         UG    0      0        0 eth0
[14763.410032] sh[4]: <IP_ADDRESS>   <IP_ADDRESS>         <IP_ADDRESS> U     0      0        0 eth0

This actually works as expected with the exception that it should have issued an error instead of running.
Recent changes have given the following semantics to --private-net:

--private-net=foo,bar -- join networks foo and bar
--private-net -- join all networks in /etc/rkt/net.d and special default network (built-in)
--private-net=all -- same as 2
--private-net=foo,default -- join foo in /etc/rkt.net.d/ and special default network (built-in)

The default network adds a default route (<IP_ADDRESS>/0). In cases where --private-net specifies some network but does not mention default, a special default-restricted network is added. This default-restricted network does not add the default route but does add a /31 route (to make metadata service reachable).
--private-net=true just says to join network true. Since default was not specified, it tries to join true and default-restricted. It does not find true network and so just joins default-restricted, thus producing
[14628.037839] sh[4]: Destination     Gateway         Genmask         Flags Metric Ref    Use Iface
[14628.038047] sh[4]: <IP_ADDRESS>   <IP_ADDRESS>         <IP_ADDRESS> U     0      0        0 eth0

The bug here is that it should have issued an error that it failed to find true and failed to run.

@eyakubovich Thank you! This makes sense. But I still failed to run a pod manifest with only --private-net..

@yifan-gu Can you explain it a little more how what leads to that failure? What networks are defined are defined in your configuration folder (default /etc/rkt/net.d)?

@steveeJ The --pod-manifest case is due to https://github.com/coreos/rkt/pull/1225/files?diff=unified#diff-e4511c1550e4abd8b839c17d256b1464L45


Add JSON Visio to Community Projects

PR Checklist
Please check if your PR fulfills the following requirements:

[ ] Tests for the changes have been added (for bug fixes / features)
[x] Docs have been added / updated (for bug fixes / features)

PR Type
Added a new community project

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] CI related changes
[x] Documentation content changes
[ ] Other... Please describe:

Awesome and thanks for sharing!


Add Cell Count - HumanGumsNormalandDiseased

Make sure Cell Count is both in Ingest and Staging area
Add cell count and export
https://contribute.data.humancellatlas.org/projects/detail?uuid=783c9952-a4ae-4106-a6ce-56f20ce27f88
Acceptance criteria for the task:
Cell count is present in ingest and the staging area

[ ] Ingest
[ ] Staging area

Closing as UCSC is already looking into it