Formatting 0.01 to 10 decimal places just prints zeroes

I don't know if you're interested in these formatting problems or not.
From my logs it looks as if a lot of small numbers are being formatted as 0. I can reproduce it with the value 0.01 (I use 1% often in some code) and the format string '%.10f' (which I pretty much use for all the values I print).
I've updated my number format program to show you exactly what I mean:
package main

import (
	"fmt"

	"github.com/ericlagergren/decimal"
)

func testPrint(floatValue float64, decimalValue string) {
	fmt.Printf("F: %.10f\n", floatValue)
	decimal, _ := new(decimal.Big).SetString(decimalValue)
	fmt.Printf("D: %.10f\n", decimal)
}

func main() {
	testPrint(0.1234567891, "0.1234567891")
	testPrint(0.123456789, "0.123456789")
	testPrint(0.123, "0.123")
	testPrint(0.0000000000000000000000000000000000000000000000000000000000001, "0.0000000000000000000000000000000000000000000000000000000000001")
	testPrint(0.01, "0.01")

	fmt.Printf("F: %f\n", 6.0)
	testDecimal, _ := new(decimal.Big).SetString("6.0")
	fmt.Printf("D: %f\n", testDecimal)
}

The current code outputs:
F: 0.1234567891
D: 0.1234567891
F: 0.1234567890
D: 0.1234567890
F: 0.1230000000
D: 0.1230000000
F: 0.0000000000
D: 0.0000000000
F: 0.0100000000
D: 0.0000000000
F: 6.000000
D: 6.00

(The last lines show a small inconsistency with '%f' but that's less of a concern for me.)
Many thanks,
Geoff

I appreciate the issues. I need to get %f fixed before the next release.

Thanks. Also, I wondered about %d too - it would be nice (for me) if it truncated to an integer, since it (and %x, %o etc.) are integer verbs. Maybe all integer verbs (%b, %c, %d, %o, %q, %x, %X, %U) could do this and format the same way they would an int?
I've no idea if that's a good idea for anyone else though but I thought I'd mention it.

While I'm sympathetic to the idea, I feel like that's a big—but also subtle—breaking change. For example, changing a function signature is a breaking change, but it's also noisy—the code won't compile. Silently truncating decimals is quiet and easy to miss until you realize you're charging a customer $3 instead of $3.14 and production is on fire. (Granted, that requires multiple failures along the way but still...) Even for v4 (a major version, so breaking changes are allowed) I'd be a little nervous making the change.
I'm open to using a verb other than one that's already taken by either this package or the fmt package (to reduce confusion). If you wanted to create a PR for that I'd be more than happy to consider it.

Yeah, I take your point but I figured I'd mention it. I guess I just assumed fmt integer verbs would truncate to integers here, but I'm not sure why. I also thought about maybe capitalised versions truncating to int (so %D etc.) but that wouldn't work so well with the %x/%X distinction. Oh well - thanks for your thoughts.

#87

Not sure if this is the desired behaviour in the Go operating mode, but this test fails (gives 0 instead of 0.00). Leaving it in GDA mode will make it pass.
func TestDecimal_FormatGo(t *testing.T) {
	z := new(Big)
	z.Context.OperatingMode = Go
	fs := "%.2f"
	want := "0.00"
	got := fmt.Sprintf(fs, z)
	if got != want {
		t.Fatalf("printf(%s)\ngot   : %s\nwanted: %s", fs, got, want)
	}
}

Another one (again, only in Go mode, gives '0.00000' instead of ' 0.00')
func TestDecimal_FormatGo(t *testing.T) {
	z := new(Big)
	z.Context.OperatingMode = Go
	fs := "'%5.2f'"
	want := "' 0.00'"
	got := fmt.Sprintf(fs, z)
	if got != want {
		t.Fatalf("printf(%s)\ngot   : %s\nwanted: %s", fs, got, want)
	}
}

It looks like the above two are intentional.
	if x.compact == 0 && o == Go {
		// Go mode prints zeros different than GDA.
		if f.width == noWidth {
			f.WriteByte('0')
		} else {
			f.WriteString("0.")
			io.CopyN(f, zeroReader{}, int64(f.width))
		}
		return
	}

fmt.Sprintf("'%5.2f'", 0.) gives ' 0.00', so I'm not sure I see the rationale for this. It certainly surprised me. If it's here to stay it would be worth adding another item to the list of operating mode differences.

@mpwalkerdine hi I just wanted to let you know I saw your comments and am taking a look. Sorry for the delay.

No worries, and thanks for the great library! :smiley:

Update: I’m rewriting part of the Format code because it’s too complicated right now. That’ll fix the issues.


Prevent infinite loop on connection close

Hi.
I have Ratchet working 24/7 and sometimes it starting to use 100% of CPU. In this case I need to restart it and this helps. But it happend very often, so I debug it and found that foef($this->stream) do not return true when client disconnects while event loop tick is not ended.
Same with is_resource($this->stream) it returns true.
I'm using:

PHP 5.5.13 / Win8 x64
PHP 5.4.10 / Win7 x64
react/event-loop 0.4.1
react/socket 0.4.2
react/stream 0.4.2
cboden/ratchet 0.3.3

In this case after
$sent = fwrite($this->stream, $this->data);

$this->lastError['message'] contains an error:

There is no way to catch such behavior using best practices, so I decide to check $this->lastError['message'] on 'errno=10054'.
Maybe it will be more efficient if we will check $this->lastError['number'] on any error > 0. But to prevent just on this case I decide to check only errno=10054 in message.
I know this is not good resolution but it works.

Excellent troubleshooting @lysenkobv! I agree with you it doesn't look like a good resolution but PHP socket error handling probably doesn't allow something more elegant.
Are you able to provide a unit test proving and fixing this or steps to reproduce so we can see/test this manually?

Impressive find :+1: , it isn't not neat but it's the best we can get, LGTM :+1:

I wrote a test and found that BufferTest::testWritingToClosedStream() have the same logic I wrote.
I decide to not duplicate it. But it is required to make some changes in the code.
When $sent = fwrite($this->stream, $this->data); executes we need to check errors.
Better solution is to check errno. set_error_handler always handle error if one occurred.
if ($this->lastError['number'] > 0) {

In this case, that part of code is not needed anymore:
if (0 === $sent && feof($this->stream)) {
     $this->emit('error', array(new \RuntimeException('Tried to write to closed stream.'), $this));
     return;
}

...because in test testWritingToClosedStream() this type of error is catched by set_error_handler().
Error message contains fwrite(): send of 3 bytes failed with errno=32 Broken pipe, errno=8.
To be sure I replaced part of code I mentioned above by checking $sent === false
if ($sent === false) {
    $this->emit('error', array(new \RuntimeException('Send failed'), $this));
    return;
}

In conclusion error checking looks like that:
if ($this->lastError['number'] > 0) {
	$this->emit('error', array(
		new \ErrorException(
			$this->lastError['message'],
			0,
			$this->lastError['number'],
			$this->lastError['file'],
			$this->lastError['line']
		),
		$this
	));

	return;
}

if ($sent === false) {
	$this->emit('error', array(new \RuntimeException('Send failed'), $this));
	return;
}

and testWritingToClosedStream asserion looks like that:
$this->assertSame('fwrite(): send of 3 bytes failed with errno=32 Broken pipe', $error->getMessage());

All 66 tests have executed successfully.

I anybody here? :smile:

:+1: Again, just awesome work on this @lysenkobv!   :-)


Wrap user-defined widgets

_zsh_highlight_bind_widgets uses the following code to decide which widgets to wrap:
  for cur_widget in ${${(f)"$(builtin zle -la)"}:#(.*|_*|orig-*|run-help|which-command|beep|yank-pop)}; do
    case $widgets[$cur_widget] in

      # Already rebound event: do nothing.
      user:$cur_widget|user:_zsh_highlight_widget_*);;

User-defined widgets, such as edit-command-line or even a trivial f() {}; zle -N f (which both fall into the first branch of that case pattern), should be wrapped to recompute highlighting like other widgets.

The redrawhook work (https://github.com/zsh-users/zsh-syntax-highlighting/issues/154#issuecomment-144541575) solves the problem.

Right now, the code looks like this:
https://github.com/zsh-users/zsh-syntax-highlighting/blob/82f5eb17ecee1f18b3ce3d8a93761a51235cab78/zsh-syntax-highlighting.zsh#L206-L212
  206         # Already rebound event: do nothing.
  207         user:$cur_widget|user:_zsh_highlight_widget_*);;
  208
  209         # User defined widget: override and rebind old one with prefix "orig-".
  210         user:*) eval "zle -N orig-$cur_widget ${widgets[$cur_widget]#*:}; \
  211                       _zsh_highlight_widget_$cur_widget() { _zsh_highlight_call_widget orig-$cur_widget \"\$@\" }; \
  212                       zle -N $cur_widget _zsh_highlight_widget_$cur_widget";;

Why does it behave differently for zle -N foo foo (takes the user:$cur_widget case) and zle -N foo bar (takes the user:* case)?

Fixed some time ago in 487b122c480d00befc3f80a8cc2e53ad99593b2c.


Add Separable Conv1D layer

Test and build pass.
Just a question, do we hope to eventually support NCWH formats? A lot of code would need to be modified in that case and I'm realizing that writing these layers.
Could i open an issue for that or would you want to support all layers first before supporting multiple formats?

@Shashi456 A slight nitpick; could you please move the Separable layers below the DepthwiseConvs since the two are closely related and ZeroPaddings stands between them.
It'd then also match the testing order.

@jon-tow alright. But would a layer reorg be appropriate in this PR?

@rxwei done. With this and transposed conv1d and 3d, we are done with basically supporting all convolutional layers :). Extensive testing and more functionality remains.

Can someone trigger a build on this ? And sorry for the persistence but #288 and #174 can be merged.


GitHub issue templates

I have added 3 issue templates: help, report a bug and feature request. This will hopefully allow Github issues to be more informative. This is my first PR xD, do give feedback on the template to see what we can add to improve it.

Alright thanks! I will make similar issue templates for the other MV sub modules when I have time (Just copy and change the name from mv-core to mv- and some links).


Checking on H5179, is it supported?

Can you please confirm if GoveeBTTempLogger supports the H5179 device?
It seems like you've listed it in models, but the actual algorithm to decrypt the data is not done. Just wanted to confirm if my assumptions are correct?
Please note that the decryption algorithm is available in another project:
https://github.com/Thrilleratplay/GoveeWatcher/blob/master/debugger/govee_advertisement.py#L108-L114

I see where I added the type name in the code, but I don't think I added support for that device.  I don't think it's listed in the Readme.md file as being supported.
I generally haven't tried to claim the code I created worked for a device if I haven't seen it working, which requires my having one of the devices. If someone with the particular device wants to create a patch supporting the H5179 and test it, I'd be willing to accept the patch into the master branch, but right now I have more thermometers than I need sitting around and don't feel the need to spend money on more.

@wcbonner, understandable, thanks for the speedy reply, I appreciate that.
Unfortunately, I don't own the device as well, otherwise, I would've given it a shot. It's just noticed some possible advantages over H5074, which's battery life is approximately 12 months, while the H5179 is advertised by Govee as a "500 days" (12 + 11 months) battery life.

I think the 5074 was the small one with the button battery, and I've been incredibly impressed with the battery life.
I've had one running since I started this project, zip tied to the balcony on my apartment in Seattle, and it's still reporting 74% battery.  I've also got one in my fridge and one in my freezer and they are holding up surprisingly well considering they are constantly going below freezing.

I think the 5074 was the small one with the button battery, and I've been incredibly impressed with the battery life.

Yes, it's that one, the H5074 it's using 1x CR2477 battery which is 3V, while the H5179 is using 3x AAA batteries (4.5v together). Considering its batteries are quite common to find in stores and it has a longer battery life, makes it worth some specific scenarios, but of course, it depends on the use case and relevancy.

I've had one running since I started this project, zip tied to the balcony on my apartment in Seattle, and it's still reporting 74% battery. I've also got one in my fridge and one in my freezer and they are holding up surprisingly well considering they are constantly going below freezing.

Regarding battery life, that's impressive, I somehow didn't though zip-tying sensors, but so doing that you'll have "incorrect" results of humidity of what would've been expected.
I have a sensor hanging on my balcony too (since March 12th), but it somehow keeps showing battery life 100%, and now I'm not sure if it's working that great, or it's just reporting wrong data or it's the decryptor that's deciphering the data incorrectly.
Regarding doing testing inside the freezer/fridge, I've considered using SensorPush HTP.xw which is:

advertised as 24-month battery life,
has a better temperature range (-40°C - 60°C (-40°F - 140°F)),
and as advertised can work under low temperature very well (without of zipping).

Unfortunately, this device is:

way expensive,
there is no decypher found in open source,
Sampling Interval is 1m (I believe Govee ones are 2s)
and currently, the only way to receive data on your computer is to purchase a separate gateway device, which needs to connect to internet and store the data on seller's servers, from where you can retrieve the data using APIs.

The humidity issues in the graph I attribute mostly to unrealistic expectations when I created the particular graph and included min/max. Each data point is an entire day, and humidity is relative humidity. It's quite often that at some point in the 24 hour period it's going to reach a maximum relative humidity of 100%, whether it's cold and hits the dew point, or raining, making the graph itself nearly useless.
The battery life in the units initially led me to believe that I was interpreting the data incorrectly. What I've really found is that they seem to be incredibly efficient as long as they are only broadcasting data. If you use the phone app to query the data, it goes into a connected Bluetooth protocol that seems to use significantly more power than the BTLE announcements. It's especially true of the units with a display.
The unit in my freezer dropped it's estimated battery quickly, then didn't drop much more, making me think that the voltage that the device is using to monitor battery state hasn't changed much, but obviously was lower than the expected and calibrated factory voltage should be. I also have a feeling that they are going to work until at some point the voltage drops rapidly and the unit just stops.

The H5177 unit has a much larger display than the H5075 units and takes three batteries instead of two, but seems to last much more than 150% of the time of the smaller unit.

The H5177 unit has a much larger display than the H5075 units and takes three batteries instead of two, but seems to last much more than 150% of the time of the smaller unit.

H5075's battery life is quite bad.. it doesn't even last a month in my case, mostly 20-25 days...

The humidity issues in the graph I attribute mostly to unrealistic expectations when I created the particular graph and included min/max. Each data point is an entire day, and humidity is relative humidity. It's quite often that at some point in the 24 hour period it's going to reach a maximum relative humidity of 100%, whether it's cold and hits the dew point, or raining, making the graph itself nearly useless.

Yeah, that's a problem, I would suggest you to try-out interactive graphs too, it can be web-based, i.e. Chart.js, you can set dynamic min-max, pick your desired duration, zoom-in-out (you need to attach a plugin for this), and even export it as an image file too.

The battery life in the units initially led me to believe that I was interpreting the data incorrectly. What I've really found is that they seem to be incredibly efficient as long as they are only broadcasting data. If you use the phone app to query the data, it goes into a connected Bluetooth protocol that seems to use significantly more power than the BTLE announcements. It's especially true of the units with a display.

oof, yeah, that's a problem.. it would've been so nice to finally figure out how to get the historical data from the Govee devices.. so there won't be a necessity to continuously monitor LE beacons constantly..


feat(otlp-transformer)!: remove internal types and functions from public API

Which problem is this PR solving?
Removes all types that don't need to be exported anymore from the public interface. They were previously private as the serialization was handled by exporters, but that has changed since #4581 and #4542 were merged. Now we don't need to make the intermediate representation public anymore since the interface takes the SDKs representation and then directly returns an Uint8Array that can immediately be sent.
Reducing the public API in this way is a key part to making @opentelemetry/otlp-transformer stable, see #4582
Fixes #4583
Breaking changes

(user-facing): the following types and functions were intended for internal use and have been removed from exports

OtlpEncodingOptions
IKeyValueList
IKeyValue
IInstrumentationScope
IArrayValue
LongBits
IAnyValue
Fixed64
SpanContextEncodeFunction
toLongBits
OptionalSpanContextEncodeFunction
getOtlpEncoder
Encoder
HrTimeEncodeFunction
encodeAsLongBits
encodeAsString
hrTimeToNanos
IValueAtQuantile
ISummaryDataPoint
ISummary
ISum
IScopeMetrics
IResourceMetrics
INumberDataPoint
IHistogramDataPoint
IHistogram
IExponentialHistogramDataPoint
IExponentialHistogram
IMetric
IGauge
IExemplar
EAggregationTemporality
IExportMetricsServiceRequest
IBuckets
IResource
IStatus
EStatusCode
ILink
IEvent
IScopeSpans
ISpan
IResourceSpans
ESpanKind
IExportTraceServiceRequest
IScopeLogs
IExportLogsServiceRequest
IResourceLogs
ILogRecord
ESeverityNumber

Type of change

[x] Breaking change (fix or feature that would cause existing functionality to not work as expected)

How Has This Been Tested?

[x] Unit tests

@trentm good point - addressed in https://github.com/open-telemetry/opentelemetry-js/pull/5200/commits/fb0d6dc847737c9de61a0e1d3bdaee4bac9881cf 🙂
I'll also hold of on merging this until I have a PR for https://github.com/open-telemetry/opentelemetry-js-contrib/issues/2565 so that we don't run into trouble when updating the contrib repo where these types are used in some tests.

where these types are used in some tests.

Ah, I hadn't checked for usage in the contrib repo.

added the blocked label so we don't merge this before https://github.com/open-telemetry/opentelemetry-js-contrib/issues/2565


Install fails in Mac with "certificate rejected: unknown public key OID: [1,3,14,3,2,12]" reason

OS: Mac OS X (El Capitan) 10.11.6
Elm Version: 0.17.1
The following is an example, but it fails with every package.
❯❯❯ elm package install elm-lang/html
Some new packages are needed. Here is the upgrade plan.

  Install:
    elm-lang/core 4.0.5
    elm-lang/html 1.1.0
    elm-lang/virtual-dom 1.1.1

Do you approve of this plan? [Y/n] y
Starting downloads...

  ✗ elm-lang/core 4.0.5
  ✗ elm-lang/html 1.1.0
  ✗ elm-lang/virtual-dom 1.1.1
Error: The following HTTP request failed.
<https://github.com/elm-lang/core/zipball/4.0.5/>

TlsExceptionHostPort (HandshakeFailed (Error_Protocol ("certificate rejected: unknown public key OID: [1,3,14,3,2,12]",True,CertificateUnknown))) "github.com" 443

So, I am not a haskell guy but googling I found this is definitively a Haskell TLS exception.
So, I installed gcc, stack, cabal and buddies and did the following, getting the exact same error. Any Ideas?
❯❯❯ cabal install tls-debug
❯❯❯ cabal exec tls-retrievecertificate -- github.com 443 --verify                                                    ⏎
connecting to github.com on port 443 ...
serial:<PHONE_NUMBER>8981134066197160231674485287
issuer:   DistinguishedName {getDistinguishedElements = [([2,5,4,6],ASN1CharacterString {characterEncoding = Printable, getCharacterStringRawData = "US"}),([2,5,4,10],ASN1CharacterString {characterEncoding = Printable, getCharacterStringRawData = "DigiCert Inc"}),([2,5,4,11],ASN1CharacterString {characterEncoding = Printable, getCharacterStringRawData = "www.digicert.com"}),([2,5,4,3],ASN1CharacterString {characterEncoding = Printable, getCharacterStringRawData = "DigiCert SHA2 Extended Validation Server CA"})]}
subject:  DistinguishedName {getDistinguishedElements = [([2,5,4,15],ASN1CharacterString {characterEncoding = UTF8, getCharacterStringRawData = "Private Organization"}),([1,3,6,1,4,1,311,60,2,1,3],ASN1CharacterString {characterEncoding = Printable, getCharacterStringRawData = "US"}),([1,3,6,1,4,1,311,60,2,1,2],ASN1CharacterString {characterEncoding = Printable, getCharacterStringRawData = "Delaware"}),([2,5,4,5],ASN1CharacterString {characterEncoding = Printable, getCharacterStringRawData = "5157550"}),([2,5,4,9],ASN1CharacterString {characterEncoding = Printable, getCharacterStringRawData = "88 Colin P Kelly, Jr Street"}),([2,5,4,17],ASN1CharacterString {characterEncoding = Printable, getCharacterStringRawData = "94107"}),([2,5,4,6],ASN1CharacterString {characterEncoding = Printable, getCharacterStringRawData = "US"}),([2,5,4,8],ASN1CharacterString {characterEncoding = Printable, getCharacterStringRawData = "California"}),([2,5,4,7],ASN1CharacterString {characterEncoding = Printable, getCharacterStringRawData = "San Francisco"}),([2,5,4,10],ASN1CharacterString {characterEncoding = Printable, getCharacterStringRawData = "GitHub, Inc."}),([2,5,4,3],ASN1CharacterString {characterEncoding = Printable, getCharacterStringRawData = "github.com"})]}
validity: DateTime {dtDate = Date {dateYear = 2016, dateMonth = March, dateDay = 10}, dtTime = TimeOfDay {todHour = 0h, todMin = 0m, todSec = 0s, todNSec = 0ns}} to DateTime {dtDate = Date {dateYear = 2018, dateMonth = May, dateDay = 17}, dtTime = TimeOfDay {todHour = 12h, todMin = 0m, todSec = 0s, todNSec = 0ns}}
### certificate chain trust
tls-retrievecertificate: unknown public key OID: [1,3,14,3,2,12]
CallStack (from HasCallStack):
  error, called at ./Data/X509/PublicKey.hs:135:23 in x509-1.6.3-B8QQl20I2pFHRBKHO0jkKo:Data.X509.PublicKey

In the end this was a corrupted keychain on the mac!!!  all working now!


Proposal: escape, quote and other options for CSV formatter plugin

Hi, I heard that built-in CSV formatter plugin doesn't support escape and quote now.
If you have no plan to do, I'm thinkng about writing it and send pull request.
Because I need CSV formatter for local-file output or load into Google BigQuery (need CSV or JSON).
I'm author of embulk-input-gcs and embulk-output-bigquery plugins.
How do you think about following specs?
Options already exists
http://www.embulk.org/docs/built-in.html#csv-formatter-plugin

delimiter string Delimiter character such as , for CSV, \t for TSV, | or any single-byte character	, by default
header_line boolean If true, write the header line with column name at the first line
newline enum Newline character (CRLF, LF or CR)	CRLF by default
charset enum Character encoding (eg. ISO-8859-1, UTF-8)	UTF-8 by default

Options I think need to implement

quote string any-single-byte-character used to quote fields containing special characters. " by default
quote_policy enum (ALL, MINIMAL, NONNUMERIC or NONE) MINIMAL by default

https://docs.python.org/3.4/library/csv.html#csv.QUOTE_ALL
ALL quote all fields
MINIMAL  only quote those fields which contain special characters such as delimiter, quote or any of the characters in lineterminator
NONNUMERIC quote all non-numeric fields
NONE never quote fields.  When the current delimiter occurs in value it is preceded by the current escape character. If escape is not set, skip line and output log like CSV parser.

escape string escape character to escape a special character. \\ by default
newline_in_field enum Newline character in the field. (CRLF, LF, CR). LF by default
skip_empty_lines boolean skip empty lines. false by default

not support
I think it's better to control for each columns using filter plugins.

trim
null_string
padding

Reference

CSV File Reading and Writing — Python 3.4.3 documentation
pandas.DataFrame.to_csv — pandas 0.16.0 documentation
CSV Library Features Matrix - OrangeSignal CSV

Very good start point.

delimiter, quote, escape, header_line, charset, newline

No-brainer. They're necessary.

quote_policy

Well designed. Sounds good.

newline_in_field

I don't have good ideas whether LF is good for default or not. But It's good start anyways.

skip_empty_lines

How does this work? (what does "empty lines" mean?)
As you mention, null_string is tricky because if null_string is "" (empty string, which will be the default value, probably), actual empty string needs to be quoted even if quote_policy is MINIMAL.

Thanks for comment.

I don't have good ideas whether LF is good for default or not. But It's good start anyways.

I'm also wondering LF is best or not.
CSV format written by MS Excel for Windows seems standard.

newline: CRLF
newline in field: LF (Alt+Enter input)

How does this work? (what does "empty lines" mean?)

This means if all column in current-line is empty, CSV formatter doesn't output current-line.
But I'm wondering this option is really need or not.

actual empty string needs to be quoted even if quote_policy is MINIMAL

I think if quote_policy is MINIMAL, empty string doesn't need to be quoted.
MS Excel for Windows seems same.
#coding:utf-8
import csv

writer = csv.writer(open('test.csv', 'wb'), delimiter=',', quotechar='"', quoting=csv.QUOTE_MINIMAL)

writer.writerow(['ABC', '', 'DE,F', 'GH"I'])

result is...
ABC,,"DE,F","GH""I"

Thank you for your good working.
I checked newline and newline_in_field characters  on OSX Platform.
I wrote CSV from MS-Excel file.
result

newline: CR
newline_in_field: CR

Test data

@sakama

actual empty string needs to be quoted even if quote_policy is MINIMAL

this is if null_string is "" (empty string)

Although I'm not fully able to understand that your comment.
Anyway I think it's better that CSV formatter plugin will not support null_string option.

I welcome pull-requests :+1:

Thanks! Maybe I can send pull-request within a week.


Question: Running tests locally.

What is the easiest way to run the unit tests locally as they appear to require API_KEY, API_BASE_URL. Using the sandbox values does not appear to function.

Hi @kamidzi, unfortunately the tests currently cannot be run outside of our environment – we hope to fix this soon. Thank you for trying to run them, and I apologize for the frustration!


Create using-breakout-rooms.md

I'm trying to add a new page to the Microsoft Teams documentation for using Breakout Rooms, a new feature. Does not seem to exist anywhere yet.

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

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


Lazy error evaluation of Result.OkIf and Result.FailIf

As far as I am aware, the factory methods OkIf and FailIf need the Error instances to be provided at the moment they are called.
This may pop up in scenarios, where a reported Error instance requires access to the data which might not exist:
var list = Enumerable.Range(1, 9).ToList();

// will throw an InvalidOperationException
var result = Result.FailIf(
    list.Any(IsDivisibleByTen),
    $"Item {list.First(IsDivisibleByTen)} should not be on the list!");

// snip

static bool IsDivisibleByTen(int i) => i % 10 == 0;

The proposed feature would be to implement following method overloads:

Result.OkIf(bool condition, Func<string>)
Result.OkIf(bool condition, Func<Error>)
Result.FailIf(bool, Func<string>)
Result.FailIf(bool, Func<Error>)

which would enable lazy evaluation of the errors; modifying the previous snippet:
var list = Enumerable.Range(1, 9).ToList();

// will not throw, as the Func<string> should not be called
var result = Result.FailIf(
    list.Any(IsDivisibleByTen),
    () => $"Item {list.First(IsDivisibleByTen)} should not be on the list!");

// snip

static bool IsDivisibleByTen(int i) => i % 10 == 0;

The workarounds to this are:

use safe methods, which do not throw exceptions in such cases or
use ternary / if statements.

Looks good. Would be happy if you create a pr for this feature. I would be more happy if you also add some tests and extend the readme file with this feature. I will encrease the version number, write the changelog and generate the package.
Thanks in advance

Hi @altmann, thanks for a quick reply! The pull request, including overloads implementation, unit tests and a short section in the README is opened and ready for review.

thank you for the great pr. I merged it already into the master branch. New package will be generated hopefully tomorrow.

New package is out: https://www.nuget.org/packages/FluentResults


A few wording touchups to CONTRIBUTING.md

Signed-off-by: TomSweeneyRedHat<EMAIL_ADDRESS>while reviewing CRI-O's CONTRIBUTING.md, we spotted a few small grammatical changes that needed to be made here too.

LGTM

LGTM


doc(browsers): Add page with instructions for Brave Shields

Goal is to have a page that we can link to on GitHub with visual explanations of what to do to allow third-party service workers.
Related to #14 but does not close it: we need to also find ways to direct Brave users to this page.

Can be reviewed by reading the rendered page at:
https://github.com/stackblitz/webcontainer-core/blob/2e5bc8c298dcc982a251411fa21969a04bff222b/browsers/brave/brave.md
@KwintenP @SamVerschueren Want to check the technical bits?

@SamVerschueren I also switched the Opera logo for Brave on the main README, and linked to this new page there.
I think showing Brave rather than Opera makes sense given the market share of both browsers with our target audience.

Nice work on the README for Brave!


replication refactoring

Refactor UG/replication.cpp into a full-fledged RG client library.  It's almost there; the task is mostly a matter of renaming some methods.

Get this done sooner rather than later...

This is pretty much done in my uncommitted UG refactoring.


Hyperion "Light", Build improvements and minor fixes

Summary
Added

Allow to build a "light" version of Hyperion, i.e. no grabbers, or services like flat-/proto buffers, boblight, CEC
Allow to restart Hyperion via Systray

Changed

Started on using Smart Pointers

Fixed

Fixed auto discovery of Hyperion server in hyperion-remote
Fixed Qt version override, e.g. set via QTDIR
Addressed Warnings during builds
Disabled QNetworkConfigurationManager deprecation warnings (for Qt 5.15 builds)

Removed

The dependency zlib was removed

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

[X] Bugfix
[X] Feature
[ ] Code style update
[ ] Refactor
[ ] Docs
[X] Build-related changes
[ ] Other, please describe:

If changing the UI of web configuration, please provide the before/after screenshot:
Does this PR introduce a breaking change? (check one)

[ ] Yes
[X] No

If yes, please describe the impact and migration path for existing setups:
The PR fulfills these requirements:

[ ] When resolving a specific issue, it's referenced in the PR's body (e.g. Fixes: #xxx[,#xxx], where "xxx" is the issue number)

If adding a new feature, the PR's description includes:

[ ] A convincing reason for adding this feature
[ ] Related documents have been updated (docs/docs/en)
[ ] Related tests have been updated

PLEASE DON'T FORGET TO ADD YOUR CHANGES TO CHANGELOG.MD

[ ] Yes, CHANGELOG.md is also updated

To avoid wasting your time, it's best to open a feature request issue first and wait for approval before working on it.
Other information:

Let's go.


Request for indoor weights or indoor inference script

Hi, Congrats for the really cool work on speeding up LoFTR !!!
My question is similar to previously raised issue
I am running this demo_single_pair.ipynb notebook . Currently, this notebook only shows outdoor example and loads outdoor weights eloftr_outdoor.ckpt. But the provided google drive has indoor checkpoints as well, but when i try to load indoor checkpoints in the notebook:
matcher.load_state_dict(torch.load("weights/indoor_ds_new.ckpt")['state_dict'])
then i got error like:
Error(s) in loading state_dict for LoFTR:
	Missing key(s) in state_dict:
Unexpected key(s) in state_dict

Can you tell me correct way to load these indoor checkpoints?
Though I have used these outdoor checkpoints, eloftr_outdoor.ckpt in indoor texture less surface and got good results, is this expected, cause you have mentioned that outdoor checkpoints are trained on megadepth dataset and it a huge domain transfer to indoor texture less surface
Also what weights were being used to create this indoor texture less surface demo

Thanks
Udit

Hello,
Thank you for your interest in our work!
The indoor checkpoints available in the Google Drive link are indeed the checkpoint of LoFTR, but we provided the link primarily for downloading datasets. We apologize for any confusion this may have caused.
Generally, checkpoints trained on outdoor datasets will not perform as well on indoor scenes. For better results, it is recommended to use checkpoints specifically trained for indoor environments.
For the indoor demo, we actually used a checkpoint that was trained on the ScanNet dataset with the same training data and strategy as LoFTR. However, we currently do not plan to provide this checkpoint.


Coloring by pLDDT

Hi @mhekkel I just want to let you know that Mol* has a native ability to color AlphaFold cif files by pLDDT score.
The example app demonstrates how to activate this behavior: https://github.com/molstar/molstar/blob/master/src/apps/viewer/app.ts
Relevant lines:
https://github.com/molstar/molstar/blob/master/src/apps/viewer/app.ts#L136-L139
https://github.com/molstar/molstar/blob/master/src/apps/viewer/app.ts#L188
https://github.com/molstar/molstar/blob/master/src/apps/viewer/app.ts#L200-L202
https://github.com/molstar/molstar/blob/master/src/apps/viewer/app.ts#L489-L517C1
I'm no expert, I just recently experimented with implementing this logic into Nightingale, you can check the PR here https://github.com/ebi-webcomponents/nightingale/pull/259/files

You can try it out by drag-and-dropping your CIF file here: https://molstar.org/viewer/


No pagination options for ListOriginCertificates

Confirmation

[X] My issue isn't already found on the issue tracker.
[X] I have replicated my issue using the latest version of the library and it is still present.

cloudflare-go version
0.80.0 and 0.86.0
Go environment
GO111MODULE='on'
GOARCH='arm64'
GOBIN=''
GOCACHE='/Users/whoami/Library/Caches/go-build'
GOENV='/Users/whoami/Library/Application Support/go/env'
GOEXE=''
GOEXPERIMENT=''
GOFLAGS=''
GOHOSTARCH='arm64'
GOHOSTOS='darwin'
GOINSECURE=''
GOMODCACHE='/Users/whoami/go/pkg/mod'
GONOPROXY=''
GONOSUMDB=''
GOOS='darwin'
GOPATH='/Users/whoami/go'
GOPRIVATE=''
GOPROXY='https://proxy.golang.org,direct'
GOROOT='/opt/homebrew/opt/go/libexec'
GOSUMDB='sum.golang.org'
GOTMPDIR=''
GOTOOLCHAIN='auto'
GOTOOLDIR='/opt/homebrew/opt/go/libexec/pkg/tool/darwin_arm64'
GOVCS=''
GOVERSION='go1.21.5'
GCCGO='gccgo'
AR='ar'
CC='cc'
CXX='c++'
CGO_ENABLED='1'
GOMOD='/Users/whoami/github/ory/cloud/go.mod'
GOWORK=''
CGO_CFLAGS='-O2 -g'
CGO_CPPFLAGS=''
CGO_CXXFLAGS='-O2 -g'
CGO_FFLAGS='-O2 -g'
CGO_LDFLAGS='-O2 -g'
PKG_CONFIG='pkg-config'
GOGCCFLAGS='-fPIC -arch arm64 -pthread -fno-caret-diagnostics -Qunused-arguments -fmessage-length=0 -ffile-prefix-map=/var/folders/cb/tjw9rzkd0y95s9xwkm4gfwlw0000gq/T/go-build195787705=/tmp/go-build -gno-record-gcc-switches -fno-common'

Expected output
When calling ListOriginCACertificates I would prefer to be able to get a list of all certificates, or be able to go page-by-page.
Actual output
When calling ListOriginCACertificates there is no option to add pagination options, which results in only getting 20 results from the API.  The ListParams only supports zoneID
Code demonstrating the issue
# client is a wrapper for cloudflare.API
certs, err := client.API.ListOriginCACertificates(ctx, cloudflare.ListOriginCertificatesParams{ZoneID: client.ZoneID})
if err != nil {
    cmd.PrintErrf("Could not list certificates: %v\n", err)
    return err
}
cmd.Printf("Certs: %d", len(certs))

Steps to reproduce

Create some sample certs (+20)
List them in the API
Only get first 20 results

References
No response

Pagination options are also missing for ListCertificatePacks
cf: https://github.com/cloudflare/cloudflare-go/blob/a410c28b480f97defd30372eb6ef4f91121170e2/certificate_packs.go#L78


Replaced script.js with loadjs

To avoid the problems caused by scriptjs not stating its licence, replaced it with loadjs which has an MIT licence

Perfect ! Thanks :)


numpy feature test

Type of Change
feature test issue fix
Description
AttributeError: module 'numpy' has no attribute 'bool'
Expected Behavior & Potential Risk
feature test pass
How has this PR been tested?
feature test
Dependency Change?
no

test pass https://inteltf-jenk.sh.intel.com/job/lpot-feature-test-top/610/


Flagged submissions: Replace score with flag notes

Currently, reviewers are able to add several flags to a submission that should prevent it from being eligible for payment. These flags are stored in the tables "Grading Observation" (v1 -> v7)
Flags are:
Incorrect Analysis / Answer
Empty Submission
Plagiarized
Duplicate
These flags should render a submission not eligible for payment. We should replace the grade that was previously displayed on the page with the flag. If there are multiple flags, please display them in a list, separated by commas.

Showcase page is located here: https://metricsdao.xyz/showcase


Adjusts PDF-creation to work

Fixes #203.

I will merge this with the hint, that, when OpenSlides/openslides-autoupdate-service#260 is closed, the mentioned lines of code have to be adjusted.


📝 Add gene symbol usage and  mapping FAQ

See: https://github.com/laminlabs/pfizer-lamin-usage/issues/76 and https://github.com/laminlabs/bionty/issues/143

Fixes a Future warning in can_validate
Adds a FAQ section on the usage of Gene symbols and mapping to ENSEMBL IDs

I do like this FAQ and in particular the whole context story, but this PR and the entry in the changelog doesn't get this across at all. It gives off the impression of a random FAQ.
Can you motivate a bit in the PR description why this was added?

It looks like ruff no longer validates strikt indendation.
We shouldn't have hanging indendation anywhere in the codebase and the docs. This throws me off:

For as long as we don't cover this with linting, let's watch out, please @sunnyosun @Zethson @Koncopd


Checking in draft ready autonat documentation.

Description

Fixes pyrsia/pyrsia#943
This PR documents our bootstrap process and the libp2p autonat negotiation for developers.
Screenshots (optional)
PR Checklist

[x] I've read the contributing guidelines.
[x] I've read "What is a Good PR?"
[x] I've included a good title and brief description along with how to review them.
[x] I've linked any associated an issue.

Code Contributions

N/A

@efrisch Is it possible to use a mermaid markdown instead of the png?  That way we can change it easily down the road if needed.

@efrisch this could be as simple as this (although not 100% the same)
graph TD;
node[Pyrsia Node] --HTTP GET /status--> boot[boot.pyrsia.link]
boot --> boot1[Boot node 1
PeerId A]
boot --> boot2[boot node 2
PeerId B]
boot --> boot3[Boot node 3
PeerId C]
boot --> boot4[Boot node 4
PeerId D]
boot1 --> json[JSON
Containing peer
addresses and
basic metrics]
boot2 --> json
boot3 --> json
boot4 --> json
json ----> node

@efrisch I read what you have so far and that's looking good. What I would maybe like to read in such documentation is how things work after bootstrapping. Maybe also with a little (mermaid) diagram.


NEW FORK

⚠ Deprecation Notice for v0.0.11 and older ⚠
Both this and the underlying push-prebuilt-action repositories seem to be abandoned and this repo is throwing a couple deprecation warnings. I have addressed those and published a new release. Would be happy to add others on as maintainers as well if I wind up lacking time.
New repo
https://github.com/jpribyl/action-docker-layer-caching
Usage
- name: Docker Layer Caching2
  uses<EMAIL_ADDRESS>
Marketplace
https://github.com/marketplace/actions/docker-layer-caching2
Underlying build action
https://github.com/jpribyl/push-prebuilt-action

Many thanks your action works!  I've submitted a PR to update the README/marketplace listing if that's OK with you:  https://github.com/jpribyl/action-docker-layer-caching/pull/5


[DOCS] Create script to setup a new server

Estimated time: 2hs
Create a document with:

server requirements
dependencies
steps
env vars
ports to open
etc.
Everything necessary to set up a server from scratch

https://www.notion.so/lambdaclass/How-to-set-up-a-new-Mirra-server-c88e0ebd99824a2abcda04b449e94033?pvs=4


QUESTION: What does each line mean ?

Hi,
Please forgive what is probably obvious to most , but what does each line that’s reported mean (exactly)? I’m using the whoami Docker container with others (gluetun) to help me validate that I’ve been given an IP provided by my vpn provider, but looking at what’s reported there is information I don’t recognise, even the host name reported is not familiar, where does that come from. Some of them I can work out, but if by any chance you have a explanation for each one, that would be awesome - thanks !
Hostname: ba38f324df5f2
IP: <IP_ADDRESS>
IP: <IP_ADDRESS>
IP: <IP_ADDRESS>
RemoteAddr: <IP_ADDRESS>:51630
GET / HTTP/1.1
Host: <IP_ADDRESS>:89
User-Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_6) AppleWebKit/605.1.15 (KHTML, like Gecko) Version/14.0.3 Safari/605.1.15
Accept: text/html,application/xhtml+xml,application/xml;q=0.9,*/*;q=0.8
Accept-Encoding: gzip, deflate
Accept-Language: en-gb
Connection: keep-alive
Cookie: CS_SESS_ID=1e7234ed-80df-4374-84a0-6b79d504462c; QT=1621932594983; NAS_PW_STATUS=0; NAS_SID=n04fa23sd; NAS_USER=admin; home=1
Referer: http://<IP_ADDRESS>:8080/containerstation/?t=16219325903892/
Upgrade-Insecure-Requests: 1

If there scope for a future enhancement , then perhaps consider adding environment variables to allow people to customise the labels, so rather than have 3 called IP, I could describe them differently to know which is which?

Hello,
The hostname you have is basically your container name. You can set it to foo by running this container with --name foo option or to your real hostname by running it onto host network with --net=host.
The IPs you see are basically the IPs associated to your container / host IPs (if your container runs on the same network namespace). So in order:

your loopback
your docker host public IP
your vtun IP

The RemoteAddr is then machine private IP the request comes from through your tunnel.
The rest of what you see is basically your request with the headers the requester set.
Rémi

Many thanks,
QQ: sometime i only two IP address reported, how do I know which is which ?
Feature request: Are there any plans to add ‘where am I’ feature p so I can the geographical location of the public IP address I have when running it through a VPN?

You can set it to foo by running this container with --name foo option

I believe the correct key is --hostname.


Update documentation to include data versioning

Update documentation to explain how data versioning behaves. Include examples that demonstrate using the data version parameter.

I also updated our internal documentation with a checklist of items we need to modify to support new collections. @torimcd - Let me know if you see any gaps!


Support multiple indices for one detector

User can use multiple indices as detector's input.  Currently we only support use one index or index pattern on Kibana. We need to support multiple indices on
1.Create/edit detector page
2.Detector detail page
3.Detector list page
4.Dashboard page

If this can be done, I believe this will open a lot more doors for writing correlation rules between different log sources. We are currently using ELK as a SIEM solution, this feature can be used to enrich a particular index with more information from a different index.


Add: TVP Mazatlán

Channel Name
XHMZ-TDT1
Alternative Names (optional)
TVP Mazatlán
Network (optional)
No response
Owners (optional)
No response
Country
MX
Subdivision (optional)
No response
City (optional)
Mazatlán
Broadcast Area
c/MX
Languages
spa
Categories (optional)
general
NSFW
FALSE
Launched (optional)
No response
Closed (optional)
No response
Replaced By (optional)
No response
Website (optional)
https://tvpacifico.mx/tv-en-linea/mazatlan
Logo
https://scontent-yyz1-1.xx.fbcdn.net/v/t39.30808-1/271755827_5321375487891095_2432776156282125041_n.jpg?stp=dst-jpg_p200x200&_nc_cat=109&ccb=1-7&_nc_sid=c6021c&_nc_ohc=UYUE80UtS2gAX-L-nPe&_nc_oc=AQmEmLD6ZaCgGG_xOi3pK5ekU1snWjsjuC_ianbulGULE8j76EUy2VpU0fujGvPPPNo&_nc_ht=scontent-yyz1-1.xx&oh=00_AfDv9IeMHVbBKfy4rh83_wZyH4v-G-vpetBQ8QBekD0sPQ&oe=6475C189
Notes
https://5ca3e84a76d30.streamlock.net/gpacifico4/smil:mazatlan.smil/playlist.m3u8
[Geo-blocked]

Please update your request:

the call sign should be used as the channel name
logo must be in JPEG or PNG format

https://github.com/iptv-org/database/blob/master/CONTRIBUTING.md#channels

Corrected


chart line doesn't match with value when use logarithmic in yaxis option

Description

Steps to Reproduce

set { logarithmic: true } in yaxis option
line didn't match with value-Y
please see Reproduction Link

Expected Behavior
Actual Behavior
Screenshots

Reproduction Link
https://codesandbox.io/p/sandbox/log-chart-error-s4yk8v
 <!-- Modify this codepen (https://codepen.io/apexcharts/pen/bxzgZJ) to clearly describe the issue, then fork it and paste the resulting codepen into the issue. Remember, this is a minimal example with only the bare minimum of code to help resolve the issue. Issues/bug reports without reproducible examples will be closed, so be sure to include an example.
<p>If you're using vue-apexcharts and want to create a demo in a Vue environment, use the CodeSandbox Vue template - https://codesandbox.io/s/pwwz8009n0 If you're using react-apexcharts and want to create a demo in a React environment, use the CodeSandbox React template - https://codesandbox.io/s/6yoqonyo7r -->

Remove the negative numbers from your data series'. Or perhaps a logarithmic chart is not the appropriate one to use for this data.


Udemy D.load fail macOS Catalina

[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: ['--geo-bypass', '--verbose', '--username', 'PRIVATE', '--password', 'PRIVATE', '-o', '~/MyVideos/%(playlist)s/%(chapter_number)s - %(chapter)s/%(title)s.%(ext)s', 'https://www.udemy.com/course/scor-350-701/learn/']
[debug] Encodings: locale UTF-8, fs utf-8, out utf-8, pref UTF-8
[debug] youtube-dl version 2020.09.06
[debug] Python version 3.8.5 (CPython) - macOS-10.15.6-x86_64-i386-64bit
[debug] exe versions: ffmpeg 4.3.1, ffprobe 4.3.1
[debug] Proxy map: {}
[udemy:course] Downloading login popup
ERROR: Unable to download webpage: HTTP Error 403: Forbidden (caused by <HTTPError 403: 'Forbidden'>); please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
File "/opt/local/bin/youtube-dl/youtube_dl/extractor/common.py", line 627, in _request_webpage
return self._downloader.urlopen(url_or_request)
File "/opt/local/bin/youtube-dl/youtube_dl/YoutubeDL.py", line 2238, in urlopen
return self._opener.open(req, timeout=self._socket_timeout)
File "/opt/local/Library/Frameworks/Python.framework/Versions/3.8/lib/python3.8/urllib/request.py", line 531, in open
response = meth(req, response)
File "/opt/local/Library/Frameworks/Python.framework/Versions/3.8/lib/python3.8/urllib/request.py", line 640, in http_response
response = self.parent.error(
File "/opt/local/Library/Frameworks/Python.framework/Versions/3.8/lib/python3.8/urllib/request.py", line 569, in error
return self._call_chain(*args)
File "/opt/local/Library/Frameworks/Python.framework/Versions/3.8/lib/python3.8/urllib/request.py", line 502, in _call_chain
result = func(*args)
File "/opt/local/Library/Frameworks/Python.framework/Versions/3.8/lib/python3.8/urllib/request.py", line 649, in http_error_default
raise HTTPError(req.full_url, code, msg, hdrs, fp)

Issue should be closed, account needed


Client pointers: make ID types generic

For user-defined client pointers, we want them to return an ID. But not just any ID! It should be an ID that comes from the selection set.
So, the param_type for a client pointer should have a generic type IdWrapper<T> in place of ID fields. But not all ID fields — just the ID fields that are valid to be returned (e.g. if you're returning a User ID, then User ID's should have type IdWrapper<T>).
In practice, these will be strings, but maybe not! Maybe they should be objects.
There should also be a function that turns IdWrapper<any> into a string.

I like using abstract classes for this as they have clearer error message than when using branded types, but branded types allow for using the value as string directly playground for both


Detect self-removal during triggering and handle it

Not sure if the fns[i] existence check is needed though.

https://github.com/muut/riotjs/issues/118

Not sure if I have the courage to merge this, since users expect the callbacks to be triggered in order. But this PR definitely gives me some tools to understand the issue.

Seems to work nicely and passes the test suite. Thank you!


ipyvolume.pylab.scatter: KeyError    Traceback (most recent call last)

When trying to run this notebook: https://github.com/rockestate/point-cloud-processing/blob/master/notebooks/point-cloud-processing.ipynb
I encountered this error when reaching this command:
# Color ground in grey
df['ground'] = df['Classification']!=1
#ground = p3.scatter(df.loc[df['ground'],'Y'], df.loc[df['ground'],'Z'], df.loc[df['ground'],'X'], color='red', size=.2)
non_ground = p3.scatter(df.loc[~df['ground'],'Y'], df.loc[~df['ground'],'Z'], df.loc[~df['ground'],'X'], color='red', size=.2)

resulting in such error message:
---------------------------------------------------------------------------
KeyError                                  Traceback (most recent call last)
<ipython-input-17-98903417dd52> in <module>()
      3 p3.scatter(df.loc[~df['ground'],'Y'],
      4            df.loc[~df['ground'],'Z'],
----> 5            df.loc[~df['ground'],'X'])

/usr/local/lib/python3.5/dist-packages/ipyvolume/pylab.py in scatter(x, y, z, color, size, size_selected, color_selected, marker, selection, grow_limits, **kwargs)
    417     fig = gcf()
    418     if grow_limits:
--> 419         _grow_limits(x, y, z)
    420     s = ipv.Scatter(x=x, y=y, z=z, color=color, size=size,
    421                     color_selected=color_selected, size_selected=size_selected,

/usr/local/lib/python3.5/dist-packages/ipyvolume/pylab.py in _grow_limits(x, y, z)
    164 def _grow_limits(x, y, z):
    165     fig = gcf()
--> 166     xlim(*_grow_limit(fig.xlim, x))
    167     ylim(*_grow_limit(fig.ylim, y))
    168     zlim(*_grow_limit(fig.zlim, z))

/usr/local/lib/python3.5/dist-packages/ipyvolume/pylab.py in _grow_limit(limits, values)
    147     else:
    148         try:
--> 149             values[0]  # test if scalar
    150         except TypeError:
    151             newvmin = values

/usr/local/lib/python3.5/dist-packages/pandas/core/series.py in __getitem__(self, key)
    765         key = com._apply_if_callable(key, self)
    766         try:
--> 767             result = self.index.get_value(self, key)
    768
    769             if not is_scalar(result):

/usr/local/lib/python3.5/dist-packages/pandas/core/indexes/base.py in get_value(self, series, key)
   3116         try:
   3117             return self._engine.get_value(s, k,
-> 3118                                           tz=getattr(series.dtype, 'tz', None))
   3119         except KeyError as e1:
   3120             if len(self) > 0 and self.inferred_type in ['integer', 'boolean']:

pandas/_libs/index.pyx in pandas._libs.index.IndexEngine.get_value()

pandas/_libs/index.pyx in pandas._libs.index.IndexEngine.get_value()

pandas/_libs/index.pyx in pandas._libs.index.IndexEngine.get_loc()

pandas/_libs/hashtable_class_helper.pxi in pandas._libs.hashtable.Int64HashTable.get_item()

pandas/_libs/hashtable_class_helper.pxi in pandas._libs.hashtable.Int64HashTable.get_item()

KeyError: 0

I cannot figure out why and it seems related to pandas as well, so please let me know if it's better to post an issue for this latter package instead. :)
Here are my pandas and ipyvolume versions:
print("ipyvolume.__version__: {}".format(ipyvolume.__version__))
print("pandas.__version__: {}".format(pd.__version__))
ipyvolume.__version__: 0.5.1
pandas.__version__: 0.23.4

I am using Ubuntu 16.04 with Python 3.5.2.

Try passing grow_limits=False to scatter, I don't 'offically' support pandas, make sure you pass a numpy array to ipyvolume, that should always work.

Tried, but didn't work, as points stayed out of view. Downcasting to numpy worked, thanks!


Remove Back text btn from step 1 of forms

@samclaridge all forms? or just give?

Maybe just give for now? But maybe all eventually?
Samuel Claridge
— Sent from
https://polymail.io/
On Mon, Mar 07, 2016 at 9:24 PM James Baxley
<
mailto:James Baxley<EMAIL_ADDRESS>
wrote:

https://github.com/samclaridge
all forms? or just give?
—
Reply to this email directly or
https://github.com/NewSpring/Apollos/issues/381#issuecomment-193560975
.
https://github.com/NewSpring/Apollos/issues/381#issuecomment-193560975


Unable to update cluster to new release

Pretty self-explanatory

Closing...
Forcing the upgrade as per Vadim's comments

Closing...
Forcing the upgrade as per Vadim's comments


trying to link on conda

Using conda I ran:
conda config --add channels defaults
conda config --add channels conda-forge
conda config --add channels bioconda
conda install bamtools
this is all fine, I can compile but not link:
undefined reference to BamTools::BamAlignment::FindTag(std::__cxx11::basic_string<char, std::char_traits, std::allocator > const&, char*&, unsigned int const&, unsigned int&) const'
It seems that there is a discrepancy between the names in the libbamtools.a file and in the one I compile:
nm -A ReconsReferenceBAM.o  |grep -i findtag ReconsReferenceBAM.o:                       U _ZNK8BamTools12BamAlignment7FindTagERKNSt7__cxx1112basic_stringIcSt11char_traitsIcESaIcEEERPcRKjRj
But this symbol is there:
nm -A /home/gabriel/.conda/envs/test_gabriel/lib/libbamtools.a |grep -i findtag /home/gabriel/.conda/envs/test_gabriel/lib/libbamtools.a:BamAlignment.cpp.o:00000000000012d0 T _ZNK8BamTools12BamAlignment7FindTagERKSsRPcRKjRj
Notice the missing basic_stringIcSt11char
However, if I compile the program using my local build, it links
without any problem because the symbol has the basic_stringIcSt11char:
nm -A /tmp/bamtools/build/src/api//libbamtools.a  |grep -i findtag /tmp/bamtools/build/src/api//libbamtools.a:BamAlignment.cpp.o:0000000000001690               T _ZNK8BamTools12BamAlignment7FindTagERKNSt7__cxx1112basic_stringIcSt11char_traitsIcESaIcEEERPcRKjRj

undefined reference to BamTools::BamAlignment::FindTag(std::__cxx11::basic_string<char, std::char_traits, std::allocator > const&, char*&, unsigned int const&, unsigned int&) const'
That is related to the GCC 5 ABI switch, your compilers and paths are munged.


Write and check project specific configuration cache fingerprint entries

Fixes #?
Context
Project specific inputs still invalidate the whole cache entry in this change.
Also introduce a test fixture for verifying the result of a build with project isolation enabled.
Contributor Checklist

[ ] Review Contribution Guidelines
[ ] Make sure that all commits are signed off to indicate that you agree to the terms of Developer Certificate of Origin.
[ ] Make sure all contributed code can be distributed under the terms of the Apache License 2.0, e.g. the code was written by yourself or the original code is licensed under a license compatible to Apache License 2.0.
[ ] Check "Allow edit from maintainers" option in pull request so that additional changes can be pushed by Gradle team
[ ] Provide integration tests (under <subproject>/src/integTest) to verify changes from a user perspective
[ ] Provide unit tests (under <subproject>/src/test) to verify logic
[ ] Update User Guide, DSL Reference, and Javadoc for public-facing changes
[ ] Ensure that tests pass sanity check: ./gradlew sanityCheck
[ ] Ensure that tests pass locally: ./gradlew <changed-subproject>:quickTest

Gradle Core Team Checklist

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation
[ ] Recognize contributor in release notes

OK, I've already triggered a build for you.

Pre-tested commit build failed.

@bot-gradle test and merge


Add a URL module to implement standardized URL processing for HTTP request targets

Can probably port the one from NodeJS: https://nodejs.org/api/url.html#the-whatwg-url-api
Source: https://github.com/nodejs/node/blob/v19.3.0/lib/url.js
Some of the more esoteric features (internationalization) may not be very useful here. I'm mostly interested in the test cases, really.
Roadmap:

[ ] Port the existing test cases from NodeJS
[ ] Port the NodeJS implementation
[ ] Streamline interfaces where necessary (should be idiomatic)

Required for #137

After looking at the NodeJS code, it seems that porting their WHATWG URL module directly might be more trouble than it's worth:

Their test suite for the new URL API is basically just comparing the URL constructor against the web platform test data
There's some (very few) additional tests, which may or may not be worth porting as well
The implementation uses their (separately-tested) legacy URL module for many things
In typical NodeJS fashion, this implementation is rather messy and likely difficult to port (if that even is desirable...)
javaScript also uses UTF16 and the test data (being JSON) appears to be encoded such that it would have to be transformed

Instead, there may be some alternatives worth considering:

Port the somewhat simpler, but equally-JS specific implementation at https://github.com/jsdom/whatwg-url
Reimplement the required logic (basic URL parser etc.) based on the rather detailed spec and skip all JS-specific parts

Lastly, I wonder how important having this module right now actually is.  So, perhaps shelving this until later is the better approach here.


Specify and implement SSDP location validation

This bug is to make sure that when we get a location header over SSDP that the header can't point us in a random direction. We need to first off make sure it is not a DNS address since we should only be using SSDP for local discovery. Then we need to make sure it is a properly formed IPV4 or IPV6 address. Then ideally we would check the local subnet mask and make sure that the address is within that range. Although that last check makes me slightly nervous if the WiFi infrastructure is really fancy. The point of all this checking is to make sure that we don't get the kind of amplification attacks we saw with UPnP.


[MIST-1058] Implement minimum load query allocator

This PR resolves #1058 by

implementing MinLoadQueryAllocationManager that allocates queries to the task that has the minimum load.

LGTM. I will merge this PR.