make diamond_collector_test should set up a virtualenv

By default, running make diamond_collector_test (or, by extension, make test), fails, unless you create a virtualenv:
krall@dev5-devc:~/pg/go/src/github.com/Yelp/fullerite (master) $ make diamond_collector_test
Failed to import module: testzookeeper. Traceback (most recent call last):
  File "src/diamond/test.py", line 261, in getCollectorTests
  File "/nail/home/krall/pg/go/src/github.com/Yelp/fullerite/src/diamond/collectors/zookeeper/test/testzookeeper.py", line 9, in <module>
    from zookeeper import ZookeeperCollector
ImportError: cannot import name ZookeeperCollector

Failed to import module: test_scribe. Traceback (most recent call last):
  File "src/diamond/test.py", line 261, in getCollectorTests
  File "/nail/home/krall/pg/go/src/github.com/Yelp/fullerite/src/diamond/collectors/scribe/test/test_scribe.py", line 8, in <module>
    from scribe import ScribeCollector
ImportError: cannot import name ScribeCollector

Failed to import module: testmesos. Traceback (most recent call last):
  File "src/diamond/test.py", line 261, in getCollectorTests
  File "/nail/home/krall/pg/go/src/github.com/Yelp/fullerite/src/diamond/collectors/mesos/test/testmesos.py", line 13, in <module>
    from mesos import MesosCollector
ImportError: cannot import name MesosCollector

..............FF.Unable to convert 277.0xs: unit xs
......................need more than 1 value to unpack
.need more than 1 value to unpack
.need more than 1 value to unpack
.......................................Unable to read JSON response.
Unable to retrieve MBean listing.
.......FFF...FFFFF...................Unable to parse response from elasticsearch as a json object
........................Couldnt parse json: No JSON object could be decoded
.....FFFFF.FFF..........EF.......................Cannot load json data: No JSON object could be decoded: line 1 column 0 (char 0)
..............GearmanCollector Error:
...Couldnt parse json: No JSON object could be decoded
...Unable to parse response from solr as a json object
....Unable to parse response from mx4j
.http://<IP_ADDRESS>:8082/path?template=identity: <urlopen error BOOM>
.......
======================================================================
ERROR: test_instance_names (testpgbouncer.TestPgbouncerCollector)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/usr/lib/pymodules/python2.6/mock.py", line 1190, in patched
    return func(*args, **keywargs)
  File "/nail/home/krall/pg/go/src/github.com/Yelp/fullerite/src/diamond/collectors/pgbouncer/test/testpgbouncer.py", line 72, in test_instance_names
    collector.collect()
  File "/nail/home/krall/pg/go/src/github.com/Yelp/fullerite/src/diamond/collectors/pgbouncer/pgbouncer.py", line 93, in collect
    host, port, user, password).iteritems():
AttributeError: 'NoneType' object has no attribute 'iteritems'

======================================================================
FAIL: test_instances (testslony.TestSlonyCollector)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/usr/lib/pymodules/python2.6/mock.py", line 1190, in patched
    return func(*args, **keywargs)
  File "/nail/home/krall/pg/go/src/github.com/Yelp/fullerite/src/diamond/collectors/slony/test/testslony.py", line 79, in test_instances
    self.assertPublished(publish, 'bar', 14)
  File "/nail/home/krall/pg/go/src/github.com/Yelp/fullerite/src/diamond/test.py", line 157, in assertPublished
    self.assertEqual(actual_value, expected_value, message)
AssertionError: bar: actual number of calls 0, expected 1

======================================================================
FAIL: test_override_user_password_nodestr (testslony.TestSlonyCollector)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/usr/lib/pymodules/python2.6/mock.py", line 1190, in patched
    return func(*args, **keywargs)
  File "/nail/home/krall/pg/go/src/github.com/Yelp/fullerite/src/diamond/collectors/slony/test/testslony.py", line 108, in test_override_user_password_nodestr
    'postgres', '_postgres', '(.*)'
  File "/usr/lib/pymodules/python2.6/mock.py", line 880, in assert_any_call
    '%s call not found' % expected_string
AssertionError: _get_stats_by_database('localhost', 5432, 'postgres', 'postgres', 'postgres', '_postgres', '(.*)') call not found

======================================================================
FAIL: test_should_ignore_unneeded_collections (testmongodb.TestMongoDBCollector)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/usr/lib/pymodules/python2.6/mock.py", line 1190, in patched
    return func(*args, **keywargs)
  File "/nail/home/krall/pg/go/src/github.com/Yelp/fullerite/src/diamond/collectors/mongodb/test/testmongodb.py", line 125, in test_should_ignore_unneeded_collections
    self.connection['db1'].collection_names.assert_called_once_with()
  File "/usr/lib/pymodules/python2.6/mock.py", line 834, in assert_called_once_with
    raise AssertionError(msg)
AssertionError: Expected to be called once. Called 2 times.

======================================================================
FAIL: test_should_ignore_unneeded_databases (testmongodb.TestMongoDBCollector)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/usr/lib/pymodules/python2.6/mock.py", line 1190, in patched
    return func(*args, **keywargs)
  File "/nail/home/krall/pg/go/src/github.com/Yelp/fullerite/src/diamond/collectors/mongodb/test/testmongodb.py", line 106, in test_should_ignore_unneeded_databases
    assert call('baddb') not in self.connection.__getitem__.call_args_list
AssertionError

======================================================================
FAIL: test_should_publish_nested_keys_for_db_stats (testmongodb.TestMongoDBCollector)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/usr/lib/pymodules/python2.6/mock.py", line 1190, in patched
    return func(*args, **keywargs)
  File "/nail/home/krall/pg/go/src/github.com/Yelp/fullerite/src/diamond/collectors/mongodb/test/testmongodb.py", line 68, in test_should_publish_nested_keys_for_db_stats
    self.connection['db1'].command.assert_called_once_with('dbStats')
  File "/usr/lib/pymodules/python2.6/mock.py", line 834, in assert_called_once_with
    raise AssertionError(msg)
AssertionError: Expected to be called once. Called 2 times.

======================================================================
FAIL: test_should_ignore_unneeded_collections (testmongodb.TestMongoMultiHostDBCollector)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/usr/lib/pymodules/python2.6/mock.py", line 1190, in patched
    return func(*args, **keywargs)
  File "/nail/home/krall/pg/go/src/github.com/Yelp/fullerite/src/diamond/collectors/mongodb/test/testmongodb.py", line 255, in test_should_ignore_unneeded_collections
    self.assertPublishedMany(publish_mock, metrics)
  File "/nail/home/krall/pg/go/src/github.com/Yelp/fullerite/src/diamond/test.py", line 184, in assertPublishedMany
    self.assertPublished(mock, key, value, expected_value)
  File "/nail/home/krall/pg/go/src/github.com/Yelp/fullerite/src/diamond/test.py", line 157, in assertPublished
    self.assertEqual(actual_value, expected_value, message)
AssertionError: localhost_27017.databases.db1.collection1.key: actual number of calls 0, expected 1

======================================================================
FAIL: test_should_ignore_unneeded_databases (testmongodb.TestMongoMultiHostDBCollector)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/usr/lib/pymodules/python2.6/mock.py", line 1190, in patched
    return func(*args, **keywargs)
  File "/nail/home/krall/pg/go/src/github.com/Yelp/fullerite/src/diamond/collectors/mongodb/test/testmongodb.py", line 226, in test_should_ignore_unneeded_databases
    assert call('baddb') not in self.connection.__getitem__.call_args_list
AssertionError

======================================================================
FAIL: test_should_publish_nested_keys_for_db_stats (testmongodb.TestMongoMultiHostDBCollector)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/usr/lib/pymodules/python2.6/mock.py", line 1190, in patched
    return func(*args, **keywargs)
  File "/nail/home/krall/pg/go/src/github.com/Yelp/fullerite/src/diamond/collectors/mongodb/test/testmongodb.py", line 195, in test_should_publish_nested_keys_for_db_stats
    self.assertPublishedMany(publish_mock, metrics)
  File "/nail/home/krall/pg/go/src/github.com/Yelp/fullerite/src/diamond/test.py", line 184, in assertPublishedMany
    self.assertPublished(mock, key, value, expected_value)
  File "/nail/home/krall/pg/go/src/github.com/Yelp/fullerite/src/diamond/test.py", line 157, in assertPublished
    self.assertEqual(actual_value, expected_value, message)
AssertionError: localhost_27057.db_keys.db_nested_key: actual number of calls 0, expected 1

======================================================================
FAIL: test_should_publish_nested_keys_for_server_stats (testmongodb.TestMongoMultiHostDBCollector)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/usr/lib/pymodules/python2.6/mock.py", line 1190, in patched
    return func(*args, **keywargs)
  File "/nail/home/krall/pg/go/src/github.com/Yelp/fullerite/src/diamond/collectors/mongodb/test/testmongodb.py", line 170, in test_should_publish_nested_keys_for_server_stats
    'localhost_27057.key': 2
  File "/nail/home/krall/pg/go/src/github.com/Yelp/fullerite/src/diamond/test.py", line 184, in assertPublishedMany
    self.assertPublished(mock, key, value, expected_value)
  File "/nail/home/krall/pg/go/src/github.com/Yelp/fullerite/src/diamond/test.py", line 157, in assertPublished
    self.assertEqual(actual_value, expected_value, message)
AssertionError: localhost_27017.more_keys.nested_key: actual number of calls 0, expected 1

======================================================================
FAIL: test_should_publish_stats_with_long_type (testmongodb.TestMongoMultiHostDBCollector)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/usr/lib/pymodules/python2.6/mock.py", line 1190, in patched
    return func(*args, **keywargs)
  File "/nail/home/krall/pg/go/src/github.com/Yelp/fullerite/src/diamond/collectors/mongodb/test/testmongodb.py", line 213, in test_should_publish_stats_with_long_type
    'localhost_27057.key': 2
  File "/nail/home/krall/pg/go/src/github.com/Yelp/fullerite/src/diamond/test.py", line 184, in assertPublishedMany
    self.assertPublished(mock, key, value, expected_value)
  File "/nail/home/krall/pg/go/src/github.com/Yelp/fullerite/src/diamond/test.py", line 157, in assertPublished
    self.assertEqual(actual_value, expected_value, message)
AssertionError: localhost_27057.more_keys: actual number of calls 0, expected 1

======================================================================
FAIL: test_should_ignore_unneeded_collections (testtokumx.TestMongoMultiHostDBCollector)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/usr/lib/pymodules/python2.6/mock.py", line 1190, in patched
    return func(*args, **keywargs)
  File "/nail/home/krall/pg/go/src/github.com/Yelp/fullerite/src/diamond/collectors/tokumx/test/testtokumx.py", line 259, in test_should_ignore_unneeded_collections
    self.assertPublishedMany(publish_mock, metrics)
  File "/nail/home/krall/pg/go/src/github.com/Yelp/fullerite/src/diamond/test.py", line 184, in assertPublishedMany
    self.assertPublished(mock, key, value, expected_value)
  File "/nail/home/krall/pg/go/src/github.com/Yelp/fullerite/src/diamond/test.py", line 157, in assertPublished
    self.assertEqual(actual_value, expected_value, message)
AssertionError: localhost_27017.databases.db1.collection1.key: actual number of calls 0, expected 1

======================================================================
FAIL: test_should_ignore_unneeded_databases (testtokumx.TestMongoMultiHostDBCollector)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/usr/lib/pymodules/python2.6/mock.py", line 1190, in patched
    return func(*args, **keywargs)
  File "/nail/home/krall/pg/go/src/github.com/Yelp/fullerite/src/diamond/collectors/tokumx/test/testtokumx.py", line 229, in test_should_ignore_unneeded_databases
    assert call('baddb') not in self.connection.__getitem__.call_args_list
AssertionError

======================================================================
FAIL: test_should_publish_nested_keys_for_db_stats (testtokumx.TestMongoMultiHostDBCollector)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/usr/lib/pymodules/python2.6/mock.py", line 1190, in patched
    return func(*args, **keywargs)
  File "/nail/home/krall/pg/go/src/github.com/Yelp/fullerite/src/diamond/collectors/tokumx/test/testtokumx.py", line 198, in test_should_publish_nested_keys_for_db_stats
    self.assertPublishedMany(publish_mock, metrics)
  File "/nail/home/krall/pg/go/src/github.com/Yelp/fullerite/src/diamond/test.py", line 184, in assertPublishedMany
    self.assertPublished(mock, key, value, expected_value)
  File "/nail/home/krall/pg/go/src/github.com/Yelp/fullerite/src/diamond/test.py", line 157, in assertPublished
    self.assertEqual(actual_value, expected_value, message)
AssertionError: localhost_27057.db_keys.db_nested_key: actual number of calls 0, expected 1

======================================================================
FAIL: test_should_publish_nested_keys_for_server_stats (testtokumx.TestMongoMultiHostDBCollector)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/usr/lib/pymodules/python2.6/mock.py", line 1190, in patched
    return func(*args, **keywargs)
  File "/nail/home/krall/pg/go/src/github.com/Yelp/fullerite/src/diamond/collectors/tokumx/test/testtokumx.py", line 173, in test_should_publish_nested_keys_for_server_stats
    'localhost_27057.key': 2
  File "/nail/home/krall/pg/go/src/github.com/Yelp/fullerite/src/diamond/test.py", line 184, in assertPublishedMany
    self.assertPublished(mock, key, value, expected_value)
  File "/nail/home/krall/pg/go/src/github.com/Yelp/fullerite/src/diamond/test.py", line 157, in assertPublished
    self.assertEqual(actual_value, expected_value, message)
AssertionError: localhost_27017.more_keys.nested_key: actual number of calls 0, expected 1

======================================================================
FAIL: test_should_publish_stats_with_long_type (testtokumx.TestMongoMultiHostDBCollector)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/usr/lib/pymodules/python2.6/mock.py", line 1190, in patched
    return func(*args, **keywargs)
  File "/nail/home/krall/pg/go/src/github.com/Yelp/fullerite/src/diamond/collectors/tokumx/test/testtokumx.py", line 216, in test_should_publish_stats_with_long_type
    'localhost_27057.key': 2
  File "/nail/home/krall/pg/go/src/github.com/Yelp/fullerite/src/diamond/test.py", line 184, in assertPublishedMany
    self.assertPublished(mock, key, value, expected_value)
  File "/nail/home/krall/pg/go/src/github.com/Yelp/fullerite/src/diamond/test.py", line 157, in assertPublished
    self.assertEqual(actual_value, expected_value, message)
AssertionError: localhost_27057.more_keys: actual number of calls 0, expected 1

======================================================================
FAIL: test_should_ignore_unneeded_collections (testtokumx.TestTokuMXCollector)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/usr/lib/pymodules/python2.6/mock.py", line 1190, in patched
    return func(*args, **keywargs)
  File "/nail/home/krall/pg/go/src/github.com/Yelp/fullerite/src/diamond/collectors/tokumx/test/testtokumx.py", line 128, in test_should_ignore_unneeded_collections
    self.connection['db1'].collection_names.assert_called_once_with()
  File "/usr/lib/pymodules/python2.6/mock.py", line 834, in assert_called_once_with
    raise AssertionError(msg)
AssertionError: Expected to be called once. Called 2 times.

======================================================================
FAIL: test_should_ignore_unneeded_databases (testtokumx.TestTokuMXCollector)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/usr/lib/pymodules/python2.6/mock.py", line 1190, in patched
    return func(*args, **keywargs)
  File "/nail/home/krall/pg/go/src/github.com/Yelp/fullerite/src/diamond/collectors/tokumx/test/testtokumx.py", line 108, in test_should_ignore_unneeded_databases
    assert call('baddb') not in self.connection.__getitem__.call_args_list
AssertionError

======================================================================
FAIL: test_should_publish_nested_keys_for_db_stats (testtokumx.TestTokuMXCollector)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/usr/lib/pymodules/python2.6/mock.py", line 1190, in patched
    return func(*args, **keywargs)
  File "/nail/home/krall/pg/go/src/github.com/Yelp/fullerite/src/diamond/collectors/tokumx/test/testtokumx.py", line 69, in test_should_publish_nested_keys_for_db_stats
    self.connection['db1'].command.assert_called_once_with('dbStats')
  File "/usr/lib/pymodules/python2.6/mock.py", line 834, in assert_called_once_with
    raise AssertionError(msg)
AssertionError: Expected to be called once. Called 2 times.

======================================================================
FAIL: test_override_user_password (testpgbouncer.TestPgbouncerCollector)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/usr/lib/pymodules/python2.6/mock.py", line 1190, in patched
    return func(*args, **keywargs)
  File "/nail/home/krall/pg/go/src/github.com/Yelp/fullerite/src/diamond/collectors/pgbouncer/test/testpgbouncer.py", line 100, in test_override_user_password
    '<IP_ADDRESS>', '6433', 'postgres', 'foobar')
  File "/usr/lib/pymodules/python2.6/mock.py", line 880, in assert_any_call
    '%s call not found' % expected_string
AssertionError: _get_stats_by_database('<IP_ADDRESS>', '6433', 'postgres', 'foobar') call not found

----------------------------------------------------------------------
Ran 222 tests in 2.744s

FAILED (failures=19, errors=1)
make: *** [diamond_collector_test] Error 1
krall@dev5-devc:~/pg/go/src/github.com/Yelp/fullerite (master) $

krall@dev5-devc:~/pg/go/src/github.com/Yelp/fullerite (master) $ virtualenv venv
New python executable in venv/bin/python
Installing setuptools, pip...done.
krall@dev5-devc:~/pg/go/src/github.com/Yelp/fullerite (master) $ . venv/bin/activate
(venv)krall@dev5-devc:~/pg/go/src/github.com/Yelp/fullerite (master) $ pip install -r requirements-dev.txt
Downloading/unpacking configobj (from -r requirements-dev.txt (line 1))
  Downloading configobj-5.0.6.tar.gz
  Running setup.py (path:/nail/home/krall/pg/go/src/github.com/Yelp/fullerite/venv/build/configobj/setup.py) egg_info for package configobj
Downloading/unpacking mock (from -r requirements-dev.txt (line 2))
  Downloading mock-2.0.0-py2.py3-none-any.whl (56kB): 56kB downloaded
Downloading/unpacking six (from configobj->-r requirements-dev.txt (line 1))
  Downloading six-1.10.0-py2.py3-none-any.whl
Downloading/unpacking funcsigs>=1 (from mock->-r requirements-dev.txt (line 2))
  Downloading funcsigs-1.0.2-py2.py3-none-any.whl
Downloading/unpacking pbr>=0.11 (from mock->-r requirements-dev.txt (line 2))
  Downloading pbr-1.10.0-py2.py3-none-any.whl (96kB): 96kB downloaded
Downloading/unpacking ordereddict (from funcsigs>=1->mock->-r requirements-dev.txt (line 2))
  Downloading ordereddict-1.1.tar.gz
  Running setup.py (path:/nail/home/krall/pg/go/src/github.com/Yelp/fullerite/venv/build/ordereddict/setup.py) egg_info for package ordereddict
Installing collected packages: configobj, mock, six, funcsigs, pbr, ordereddict
  Running setup.py install for configobj
  Running setup.py install for ordereddict
Successfully installed configobj mock six funcsigs pbr ordereddict
Cleaning up...
(venv)krall@dev5-devc:~/pg/go/src/github.com/Yelp/fullerite (master) $ make diamond_collector_test
.................Unable to convert 277.0xs: unit xs
.......................need more than 1 value to unpack
.need more than 1 value to unpack
.need more than 1 value to unpack
.......................................Unable to read JSON response.
Unable to retrieve MBean listing.
.....................................Unable to parse response from elasticsearch as a json object
....................Unable to read JSON response: No JSON object could be decoded
.......Couldnt parse json: No JSON object could be decoded
.................................................Cannot load json data: No JSON object could be decoded
.................Couldnt parse json: No JSON object could be decoded
...Unable to parse response from solr as a json object
....Unable to parse response from mx4j
.http://<IP_ADDRESS>:8082/path?template=identity: <urlopen error BOOM>
.......
----------------------------------------------------------------------
Ran 226 tests in 2.570s

OK
(venv)krall@dev5-devc:~/pg/go/src/github.com/Yelp/fullerite (master) $

The reason the test suite succeeds in virtualenv and fails without it is because - some of those tests flat out don't run if certain libraries (such as mongo-client drivers) are not present. So when a virtualenv is created - many of those Python libraries don't exist and those problematic tests don't fail.
Most of the collectors that are failing - we aren't using, so I am thinking of further blacklisting them.

Rather than skipping tests if certain modules aren't available, I think it would be better use a virtualenv to make sure those modules exist in the testing environment. This would also ensure that everybody is testing in a consistent environment.


custom start and end for route_migration

I think it could maybe be useful to also be able to pass custom start and end inputs to route_migration, that would be processed by lookup_timestep to replace the season argument.  Running into this with a species where I disagree with the eBird S&T assigned dates for the end of migration.  I could also see this being useful for plenty of other applications, like shorter-term predictions of where birds go.

We could also add a function to customize the species metadata in a BirdFlow object, and/or you could edit the bf$species list directly.  I have mixed feelings about this as it would be nice to make it clear that the resulting dates weren't from eBird S&T.  We could for instance allow optional "custom_" versions in the species list that would be preferentially used (if present) when looking up the migration dates, along with a function  set_custom_date(bf,  name, date).  I'm not pushing for this, just brainstorming.

An alternative approach keeping things more modular could be to wrap this messy part into a helper function:
  locations <- sample_distr(get_distr(bf, start, from_marginals = TRUE),
    n = n)
  if (n == 1) {
    ind <- which(as.logical(locations))
  }
  else {
    ind <- apply(locations, 2, function(x) which(as.logical(x)))
  }
  x <- i_to_x(ind, bf)
  y <- i_to_y(ind, bf)

And return list(x = ..., y= ...)
Then user could do something like this:
sample_distrib_new_fn(bf, start, n, from_marginals = TRUE) -> out
route(bf, out$x, out$y, lookup_timestep(mystart), lookup_timestep(myend))

I think I'd rather think of the ebirdST data (prebreeding start, etc) within the bf object as coming directly from S&T and not modify it.

An alternative approach keeping things more modular could be to wrap this messy part into a helper function:
...

sample_distr(d, bf = bf, format = "xy") now handles the messy part.
https://birdflow-science.github.io/BirdFlowR/reference/sample_distr.html

I think I'm going to merge route_migration() into route() so one function handles everything.

[ ] Drop row and col arguments (I don't think anyone uses them and c_to_x() and  r_to_y() can handle conversion if necessary.
[ ] Make x and y optional.  If they are missing then sample n_each  starting locations from the starting timestep distribution.
[ ] Replace start, end, direction, season_buffer, n with  ... passed to lookup_timestep_sequence()

After #110 we then can replace route_migration(bf, n = 5) with  route(bf, season = "prebreeding", n_each = 5)  and still be able to specify specific starting positions e.g. with route(bf, x_coord = x, y_coord = y,  ...)
@slager does that work for you?  After the discussion above the only remaining difference between the two functions was whether the starting locations are sampled from the distributions.

That works.  It could be nice to have a flag to sample from ebirdst distribution or sample from model marginal distribution.

You can now do everything with route().  To have the function sample the starting locations leave x_coord and y_coord NULL:
rts <- route(bf, n = 10, season = "prebreeding")

You can use any of the time arguments supported by lookup_timestep_sequence() so  custom date or timestep ranges also work:
rts <- route(bf, n=10, start = "2023-04-07", end = "2023-07-14")


Fix stub errors not throwing properly

PublicClientApplication stub errors were throwing the constructor rather than the error message. This PR fixes this and also:

Adds tests
Adds explanation to the errors doc
Adds aka.ms vanity link to the errors doc in the error message (open to other suggestions for the link name)

Coverage remained the same at 92.632% when pulling 6a0c28447f198eaa3f02cd450d8f181c02536bcf on fix-stub-errors into 18d8bd5d458d313e8229ea0557db28474ed4ec9f on dev.


PacReq: Thunderbird

Types
-bin (Binary release of the package)
URL
https://www.thunderbird.net/en-US/thunderbird/releases/ (Releases)
https://archive.mozilla.org/pub/thunderbird/releases/ (Sources)
Building guide
I'll copy firefox-bin pacscript and change it into thunderbird-std-bin. For ESR, I'll add it on next request which called thunderbird-esr-bin. I'll create new commit on my forks.
Additional context
No response

Cause I am now taking the rest and have issues for my health, I'll working on it in free time I had.

For ESR builds (thunderbird-esr-bin), it will be working on it after main release had been merged.


feat: Adds RestTemplate default ConnectTimeout / ConnectRequestTimeout

Fixes gh-4110

Wrote related code to explicitly declare custom timeouts in eureka.client.restTemplate.

I'm also paying attention to the issue in-house, so I'll do my best. Please show a lot of interest.

Because Configuration Properties declaration moved, I removed the test in the existing test class. Is it a big problem for the 'codecov/project' error to occur as you can see?

Because Configuration Properties declaration moved, I removed the test in the existing test class. Is it a big problem for the 'codecov/project' error to occur as you can see?

No, the delta threshold is set too high. We should change it in future. I don't think this change requires additional tests.

Thank you. It was a good experience.

Thanks for contributing.


Caltrans training: dbt content changes

Slide on data layer concepts
Slide/example on CTEs
Swap out data to focus on PEMS
Update content in training doc to reflect additional day
Update links in training doc to point to Caltrans repos, etc. instead of CARB

Reference: Caltrans training plan

Ian has some exploratory models here: https://github.com/cagov/caldata-mdsa-caltrans-pems/tree/exploratory-models


MambaRoll: Physics-Driven Autoregressive State Space Models for Medical Image Reconstruction

Hi,
Please consider including our recent work, MambaRoll, in your collection. Here are the links to the preprint and the GitHub repository:
ArXiv preprint: https://arxiv.org/abs/2412.09331
GitHub repo: https://github.com/icon-lab/MambaRoll
Thank you for this collection.

@bilalkabas  Hi, we will update this paper in the paper list.


Ctrl-c: Copy doesn't work

there is no way to select text by keyboard command. Therefore Ctrl-c is pretty useless with keyboard.

What you're saying is that there is no way to select text with the keyboard, not that Control C to copy doesn't work. See #79.


Build Failure on OSX 10.6.8

Buiding makes this error:
lposix.c: In function ‘luaopen_posix_c’:
lposix.c:2078: error: ‘SIGPOLL’ undeclared (first use in this function)
lposix.c:2078: error: (Each undeclared identifier is reported only once
lposix.c:2078: error: for each function it appears in.)
make[2]: *** [posix_c_la-lposix.lo] Error

signal.h looks like this on this system:
#if  (defined(_POSIX_C_SOURCE) && !defined(_DARWIN_C_SOURCE))
#define	SIGPOLL	7	/* pollable event ([XSR] generated, not supported) */
#else	/* (!_POSIX_C_SOURCE || _DARWIN_C_SOURCE) */
#define	SIGIOT	SIGABRT	/* compatibility */
#define	SIGEMT	7	/* EMT instruction */
#endif	/* (!_POSIX_C_SOURCE || _DARWIN_C_SOURCE) */

_DARWIN_C_SOURCE is enabled by normal configure script in config.h
Simply commenting out the setting of SIGPOLL helps.

SIGPOLL is POSIX-mandated, so it's not a bug in luaposix.
You're welcome to supply a workaround patch for Darwin.

(I'd prefer a patch that takes into account however Darwin gets around the lack of SIGPOLL, and hence provides equivalent functionality, but if that's too much work or not possible, then simply not having SIGPOLL on Darwin is OK.)

I suppose thats why they provide it if "_DARWIN_C_SOURCE" is not set.
How about this workaround. Judging from their signal.h SIGABRT and SIGEMT seem to be their replacement, whatever it means.
#if (!defined(_DARWIN_C_SOURCE))
set_const("SIGPOLL", SIGPOLL);
#else
set_const("SIGABRT", SIGABRT);
set_const("SIGEMT",  SIGEMT);
#endif
Or we teach configure to not set _DARWIN_C_SOURCE, then the header seem to try to be more POSIX compliant.

SIGABRT is already defined. (Again, it's POSIX-mandated.) The bit of header file you quote is defining SIGIOT to be SIGABRT, not defining SIGABRT.
SIGEMT seems to do something else entirely (it's about an emulated instruction; is this to do with Rosetta?) So it's not a replacement. I have no desire to support non-POSIX signals in luaposix.
As to not setting _DARWIN_C_SOURCE, what are the downsides of doing that? I presume it's set by default for a reason...
I'm sorry, I don't have time to learn about an OS I don't use; perhaps you could seek guidance elsewhere? I suggest the autoconf or gnulib mailing lists, as there seems to be a general point of interest here, namely, what is one supposed to do about the lack of SIGPOLL on Darwin, and hence there should really be a patch to autotools, or at least a gnulib module to handle the problem.

I got my stuff sorted with commenting out SIG_POLL I don't use it eitherway, only wanted to help to get luaposix be more compatible and make the life for others easier. If you don't care, then lets just stop and forget it.

I certainly do care, but I would rather this issue were fixed where it should be fixed, not in luaposix. (Of course, really it should be fixed in Darwin, but in the absence of that, a fix that helps all GNU autotools-based software is the best we can do.)
If you don't want to do that, then can you please just supply a simple patch that tests #ifdef SIGPOLL, with a comment explaining this is a workaround for Darwin, and then at least you've helped Darwin users of luaposix.

I don't think its apple nor gnulibs problem.
Gnulibs point of view: By default lets turn on all System extenstions ( http://fossies.org/unix/privat/metalog-2.tar.gz:a/metalog-2/gnulib/m4/extensions.m4 )  Whatever that got them into this autoconf default policy. But they turn on _DARWIN_C_SOURCE like they turn on everything the system can do. Similar with happens all GNU extensions.
Apples point of view: programs can either chose to have a standards compliant environment emulation, or can announce they know of our dirty tricks and extensions and say so with Macro (_DARWIN_C_SOURCE)
I suppose configure.ac needs to be taught not to include AC_USE_SYSTEM_EXTENSIONS if our full aim is to stick to the standard and not do with system extensions. Thats what we wand, no? I just don't yet see how.

Thanks for the analysis. I'll ask the gnulib folk what they think.

Turns out that SIGPOLL (and also SIGPROF) are marked obsolescent, so I've removed them from luaposix in git master. This should cure your build problem. Thanks again for the report.


Receive notification in Javascript but not in the device

Hi:
I've installed everything alright. I've got the registrationId, and form a server I send a message ... and the function:
notification.push.on('notification', function(data) {
           // here the data
});

is triggered correctly when it's received.
But I don't see the Android notification itself, in the notification bar.
Any idea ?
thank you !

@adantart you should only see the notification in the shade when the app is in the background and it has a message and/or title.


ERROR: CUDA RT call cudaFuncSetAttribute. Failed with invalid device function (98).

I run into this error when I train the model with use_fast_fftconv.
ERROR: CUDA RT call "cudaFuncSetAttribute(kernel, cudaFuncAttributeMaxDynamicSharedMemorySize, shared_memory_size )" in line 809 of file /***********/H3/csrc/fftconv/fftconv_cuda.cu failed with invalid device function (98).

This error actually doesn't stop the training process, but the result of the conv op is wrong. I also run PYTHONPATH=$(pwd) pytest tests/
PYTHONPATH=$(pwd) pytest tests/
========================================================================================================================= test session starts =========================================================================================================================
platform linux -- Python 3.8.16, pytest-7.4.0, pluggy-1.2.0
rootdir: /mnt/cache/wangzerui/H3-origin/H3
plugins: anyio-3.6.2
collected 4160 items

tests/ops/test_fftconv.py FFFFFFFFFFFFFFFFFFFFssssFFFFFFFFFFFFFFFFFFFFssssFFFFFFFFFFFFFFFFFFFFssssFFFFFFFFFFFFFFFFFFFFssssFFFFFFFFFFFFFFFFFFFFssssFFFFFFFFFFFFFFFFFFFFssssFFFFFFFFFFFFFFFFFFFFssssFFFFFFFFFFFFFFFFFFFFssssFFFFFFFFFFFFFFFFFFFFssssFFFFFFFFFFFFF [  5%]
FFFFKilled

Envs:

torch: 2.0.0+cu117
gpu: A100-80G × 8
cuda: 11.7

I have installed the fftconv by running
cd csrc/cauchy && pip install . && cd ../../ \
    && cd csrc/fftconv && pip install . && cd ../../ \
    && cd .. && rm -rf csrc

I haven't seen this, but googling seems to suggest it's because there’s a mismatch between the CUDA version the binary was compiled to and the CUDA version of the device. Maybe the solution is to uninstall the fftconv extension, then make sure to reinstall it with the right CUDA version.

Solved after I use another cluster with a newer driver.
current output of  nvidia-smi:
Every 1.0s: nvidia-smi                                                                                Mon Aug  7 19:15:39 2023

Mon Aug  7 19:15:39 2023
+-----------------------------------------------------------------------------+
| NVIDIA-SMI 525.60.13    Driver Version: 525.60.13    CUDA Version: 12.0     |
|-------------------------------+----------------------+----------------------+

Previous output:
Every 1.0s: nvidia-smi                                                                                Mon Aug  7 19:19:25 2023

Mon Aug  7 19:19:25 2023
+-----------------------------------------------------------------------------+
| NVIDIA-SMI 470.129.06   Driver Version: 470.129.06   CUDA Version: 11.4     |


I find a better plugin in clipboard

I always use clipboard
but sometime , the clipboard is not friendly for me
i find a clipboard use telescope to brower my clipboard history
and i want to some text just delete not put in registers
so i use backspace like i begin use editor
maybe this is friendly for newer vim user
i try it become let this become better
because i really like this nvim config

I use clipmenud and copyq to manager my clipboard.
I don't know how this experience is convenient. How about your own feeling?

I not experience clip menu ,maybe it is better for just at nvim.
because clib menus should be glboal.
I want to experience that you said menu

thanks for that answer
the neo cliblike git file just fzf search maybe not different from clibmenu
so it is good for me sometime


Dev Box configuration constantly gives "log in required" and "resume" button without working

Dev Home version
No response
Windows build number
No response
Other software
No response
Steps to reproduce the bug
When trying to configure Dev Box, an error saying "log in required" pops up 5-6 times, prompting the user to hit Resume. After 5-6 times of the error and hitting resume, configuration finally works.
This entire UX is extremely confusing and took a few times before I realized I should continue pressing resume until it works. There needs to be clearer action prompts (telling the user explicitly what to do and how to do it, similar to how Hyper-V has a login prompt) and this should not take 5-6 prompts simply to work.
Expected result
No response
Actual result
Configuration only works once I figured out that pressing resume 5-6 times when prompted and waiting would do it. The text in-product is not informative and the experience is blocking.
Included System Information
No response
Included Extensions Information
No response

Issue is fixed in the latest build


How to set a cloud variable with CloudEvents

I've been searching on the wiki and on Scratch for a while but I have no clue how to set a variable with the CloudEvents thing. When looking into _requests.py I see you just did it manually... How am I supposed to do it ?

You can't set cloud variables with CloudEvents, use CloudRequests for that: https://github.com/TimMcCool/scratchattach/wiki/Cloud-Requests


Beta

Codecov Report
Patch coverage: 100.00% and no project coverage change.

Comparison is base (ed1062f) 100.00% compared to head (483c9d3) 100.00%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@            Coverage Diff            @@
##              main        #2   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            5         5
  Lines          151       146    -5
  Branches         7         6    -1
=========================================
- Hits           151       146    -5

Impacted Files
Coverage Δ

deps.ts
100.00% <100.00%> (ø)

utils.ts
100.00% <100.00%> (ø)

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.


eraser: Add version <IP_ADDRESS>93

See https://github.com/ScoopInstaller/Extras/pull/10447.

/verify


Can't log in with newest build

3.2.0-68-gf713d88
Incorrect username/pass errors with correct username+pass and with correct email+pass

I am not sure I understand the description of the problem. Can you be more verbose?

Are you saying you can't log in at ALL?

I am inputting the correct username and pass and also the correct email and pass, but i am getting "incorrect login" errors.

I can't replicate this. Maybe DM me more sensitive details in IRC

Is this still a thing?

nope!


mention Ali's "Programming in D" prominently on "Gettting Started"

We should take newbies by the hand on "Getting Started". And as far as I'm
aware, Ali's book is the best thing we have in that category.
Also, drive-by fix of bad parenthesis.

change the word and let's pull please - thanks!

updated to "beginners"

Ping?


Dependency issue with python 3.6 with click 8.0.0 ?

It seems that click has some deps issue with python 3.6 with the new release 8.0.0.
But still it seems supported : https://github.com/pallets/click/blob/main/setup.cfg#L33
Is it expected behavior ? Or should python_requires be bump to 3.7 ?
how to reproduce :
pyenv install 3.6.0
pyenv local 3.6.0
pip install click
python -c "import click"

AttributeError: module 'typing' has no attribute 'NoReturn'

Looks like it was added in 3.6.2...

I’m seeing this failure as well, via black. Still have 3.6.0 on some of our CI servers (for unfortunate reasons). It would be really nice if we could get a release that works with 3.6.0 even if the medium term solution is to focus on 3.6.2+.
The fix may be simple: add a TYPE_CHECKING guard. I’d be happy to contribute a PR.

Note that Python 3.6 (all versions) goes end of life in December 2021. You need to start upgrading your entire system very soon, for more important reasons than a typing import in Click.

Thanks!


Added support for feature = preserve_order

This adds support for the preserve_order feature that is also provided by serde_json. The behavior is similar and comes with the same general downsides that it's a global feature.

That's a good point - or alternatively, would it be appropriate to make indexmap the only option if the overhead is comparable?


Install pulseaudio deps and start pulseaudio on Linux Desktop

Found by https://github.com/mozilla/cubeb-rs/pull/91#issuecomment-2417760184

Copied from https://github.com/mozilla/cubeb/blob/master/.github/workflows/build.yml#L20-L26

Yeah.


SVG with pattern embedded in PDF not getting rendered

Link to PDF file (or attach file here):
SVG Pattern in PDF.pdf
Configuration:

Web browser and its version:
FireFox 45.8.0, Google Chrome 57.0.2987.98, IE 11
Operating system and its version:
PDF.js version:
PDFJS.version = '1.0.907';
Is an extension:

Steps to reproduce the problem:

Application has logic of inserting SVG with various patterns in PDF.
Application uses PDF JS (V 1.0.907) for rendering this PDF on UI.
But pdf.js simply skips the rendering of the pattern (

Looks like TilingPattern is expecting bbox details in x0, y0, x1, y1
But the pattern from attached PDF is x0, y1, x1, y0
Note: this.bbox = PDFJS.Util.normalizeRect(IR[4]); change seams to fix this issue

What is the expected behavior? (add screenshot)
SVG Pattern in PDF.pdf
What went wrong? (add screenshot)

Link to a viewer (if hosted on a site other than mozilla.github.io/pdf.js or as Firefox/Chrome extension):
Same issue is reproducible if we upload attached pdf on PDF JS demo link : https://mozilla.github.io/pdf.js/web/viewer.html

Closing as duplicate of issue #8117.


Allow passing the value of the value property directly

If a class has a value property then it should be possible to pass this directly, without the surrounding object.
For instance, instead of:
"propertyName" : {
    "content" : "ows:BoundingBox"
}

you should be able to do simply
"propertyName" : "ows:BoundingBox"

If the passed value is not an object (not an instance of the class), but the class has a value property and the value is an instance of that value property type (easy to check) then it gets autowrapped and marshalled.
See tomkralidis/csw4js#1.

Documentation:
https://github.com/highsource/jsonix/wiki/Shortcutting-Structures

Done.


prevent user history view in demo mode

Demo mode prevents history viewing but the specific API is leaking history. This is a security issue.

Thanks for pointing this out, but I'm not sure its a security issue in demo mode.

Please check out #2320.


Single click folden in column view

What's the Problem?
No response
Solution/Idea
It would be awesome if you could open the folders with single click when in Column view!
Alternatives
Option on the settings menu so the user can decide if they want column view to open folders in one click or double
Priorities

Capability
Priority

This proposal will allow developers to accomplish W
Must

This proposal will allow end users to accomplish X
Should

This proposal will allow developers to accomplish Y
Could

This proposal will allow end users to accomplish Z
Won't

Files Version
Version: <IP_ADDRESS> OS Version: 10.0.19043.1237
Windows Version
Edition	Windows 10 Pro Version	21H1 Installed on	31/1/2021 OS build	19043.1237 Experience	Windows Feature Experience Pack 120.2212.3530.0
Additional comment
No response

We have that! Files Settings > Files and Folders > Open items with one click. Are you experiencing problems with it?

We have that! Files Settings > Files and Folders > Open items with one click. Are you experiencing problems with it?

I think they are requesting for the column layout to automatically work with single click while still using double click in the other layouts.

We have that! Files Settings > Files and Folders > Open items with one click. Are you experiencing problems with it?

If I enable "Open items with one click", executeables and doc files will be opened with one click too. Can we add a new option to only enable opening folder with one click? When in column mode, it would a very productive way to navigate ;-)

What I mean is exactly what Zhiqiang and Yaichenbaum are saying.
I am aware of the option that lets you open folders with one click, but this also opens executable files, which is not convenient if I am being honest.
In column view, the open with one click option can be enabled but only work with folders, not other files or exe.
Also, in that view, it doesn't remember the sorting, it would be nice to have it remember that I sorted files by type or date etc when I open it again.
Thanks for all the hard work, files is pretty awesome!

Yes, a "single-click-open on folders only" is the only thing that keeps me from switching over completely at the moment.

Merging with #6308.


How to export logs to a elasticsearch?

Unless I go down the route of logging to both a discrete log file and the console at the same time using an extension method, I dont see a way to export the logs to elasticsearch.
Is there a real-time event I could use?  I am not sure the best way to send logs in real-time to elastic.

I guess, the best way is to .UseNLog() with Hangfire itself and configure NLog to use NLog.Targets.ElasticSearch. But you still have to log to both places.


Pr android10 blank screen fix and custom icons fix

Pull request guide
Thank you for considering to improve this cordova-plugin-googlemaps.
When you create a pull request, please make it to multiple_maps branch instead of master branch.
Because the multiple_maps branch is edge version.
Thank you for your understanding.

Can somebody please confirm its functionality on Cordova Android 11?


chore: add test for multidimensional array inputs

Description

Problem*

Resolves #2808
Summary*
This PR adds a multidimensional array input to the 2d_simple_array test
Documentation

[x] This PR requires documentation updates when merged.

[ ] I will submit a noir-lang/docs PR.

[x] I will request for and support Dev Rel's help in documenting this PR.

Multidimensional arrays are now supported int the program ABI.
Additional Context

PR Checklist*

[x] I have tested the changes locally.
[x] I have formatted the changes with Prettier and/or cargo fmt on default settings.

This supposedly went into v0.14.0 correct? Weird that it wasn't captured in the release PR & changelogs 🤔


Update Python versions run in CI, update tests to use cffconvert 2

Recent PRs against main failed because Python 3.6. is no longer supported in Ubuntu latest, which is 22.04.

Related issues
Refs: https://github.com/actions/setup-python/issues/544
Describe the changes made in this pull request

Remove Python 3.6 (end of life 2021-12-23) and add newer versions up to 3.11
Update script for testing examples to use cffconvert (latest, i.e., >=2.0)
Except two links from link checking as they continuously fail, which may be because they're behing a proxy that doesn't allow this kind of access

Review checklist

[ ] Please check if the pull request is against the correct branch
(format/schema/semantic documentation changes: develop; typos, meta files, etc.: main)
[ ] Please check if all changes are recorded in CHANGELOG.md and adapt if necessary.
[ ] Please run tests locally.

cd $(mktemp -d --tmpdir cff.XXXXXX)
git clone https://github.com/citation-file-format/citation-file-format .
git checkout update-python-versions-for-CI
python3 -m venv env
source env/bin/activate
pip install --upgrade pip wheel setuptools
pip install -r requirements.txt
pytest
<do other things>

[ ] Check if correct Python versions are run and test successfully

Some of the links appear dead but are actually ok when I check them; the testing/test jobs require a bit more investigation though, they generate a "The namespace_packages parameter is deprecated." error, seemingly during pip install of a dependency, ruamel.yaml

Some of the links appear dead but are actually ok when I check them;

I've included link checking exceptions for two alive URLs in this PR.

the testing/test jobs require a bit more investigation though, they generate a "The namespace_packages parameter is deprecated." error, seemingly during pip install of a dependency, ruamel.yaml

The error is gone, probably because it has been fixed in a newer release of ruamel.yaml.
I'd be very happy for you to do a quick re-review of this @jspaaks. Thanks 🙏.


Problems showing long text in limited space

Thanks for creating parrotsay!
Working on tiny-care-terminal I noticed that long text are not displayed very nicely there is not enought space:

Maybe some better word-wrap could be introduced!?
It's not a big think but I thought It should be tracked here.

Sorry, tiny-care-terminal is is actually using ansi-art.
My mistake :blush:


Mark mapping closures @noescape

A lot of functions in the swift standard library have this attribute as well.

:+1: Good call. Thank you @JaSpa!


Translate tasks/administer-cluster/enabling-endpoint-slices into Japanese

This is a Feature Request

What would you like to be added
Translate tasks/administer-cluster/enabling-endpoint-slices into Japanese
Why is this needed
This document is not translated into Japanese.
Comments

/language ja

/assign

/milestone 1.16-ja.1

/milestone clear

#18873 resolved this issue.
/close


Updating POM File for next preview snapshot

Codecov Report

Merging #557 into dev will decrease coverage by 0.3%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##                dev     #557      +/-   ##
============================================
- Coverage     46.71%   46.41%   -0.31%
+ Complexity     2224     2206      -18
============================================
  Files           108      108
  Lines         25321    25321
  Branches       4175     4175
============================================
- Hits          11828    11752      -76
- Misses        11472    11550      +78
+ Partials       2021     2019       -2

Flag
Coverage Δ
Complexity Δ

#JDBC41
46.19% <ø> (-0.32%)
2196 <ø> (-14)

#JDBC42
46.31% <ø> (-0.3%)
2203 <ø> (-19)

Impacted Files
Coverage Δ
Complexity Δ

...om/microsoft/sqlserver/jdbc/ReaderInputStream.java
41.57% <0%> (-5.62%)
14% <0%> (-2%)

...rc/main/java/com/microsoft/sqlserver/jdbc/DDC.java
43.34% <0%> (-2.26%)
104% <0%> (-1%)

...om/microsoft/sqlserver/jdbc/SimpleInputStream.java
51.11% <0%> (-1.49%)
11% <0%> (-1%)

...a/com/microsoft/sqlserver/jdbc/PLPInputStream.java
53.01% <0%> (-1.21%)
29% <0%> (-1%)

...c/main/java/com/microsoft/sqlserver/jdbc/Util.java
60.75% <0%> (-1.11%)
87% <0%> (-2%)

...m/microsoft/sqlserver/jdbc/SQLServerResultSet.java
32.41% <0%> (-0.86%)
237% <0%> (-8%)

...in/java/com/microsoft/sqlserver/jdbc/IOBuffer.java
53.39% <0%> (-0.83%)
0% <0%> (ø)

...n/java/com/microsoft/sqlserver/jdbc/Parameter.java
62.44% <0%> (-0.22%)
63% <0%> (-1%)

...om/microsoft/sqlserver/jdbc/SQLServerBulkCopy.java
52.58% <0%> (-0.2%)
238% <0%> (-3%)

...rc/main/java/com/microsoft/sqlserver/jdbc/dtv.java
63.05% <0%> (-0.19%)
0% <0%> (ø)

... and 1 more

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 91e7d8a...b247869. Read the comment docs.


[aws efs] ProvisioningFailed - no volume plugin matched

I followed the deployment steps and deployed efs-provisioner on kubernetes. But I am getting below error,
$ kubectl describe pvc efs -n dev
Name:		efs
Namespace:	dev
StorageClass:	aws-efs
Status:		Pending
Volume:
Labels:		<none>
Annotations:	volume.beta.kubernetes.io/storage-class=aws-efs
Capacity:
Access Modes:
Events:
  FirstSeen	LastSeen	Count	From				SubObjectPath	Type		Reason			Message
  ---------	--------	-----	----				-------------	--------	------			-------
  8m		23s		11	persistentvolume-controller			Warning		ProvisioningFailed	no volume plugin matched

I'm having the same issue on EKS, and don't know how to debug it. There are no any erros in efs-provisioner pod logs, EFS is mounted by efs-provisioner, but pvcs are pending with that no volume plugin matched message.

Looks like it's smth with latest versions. Works fine with chart version 0.1.3 (which uses 0.1.2 image) In order to provision PVC with 0.2.0 chart and v2.1.0-k8s1.11 image you'll need to add volume.beta.kubernetes.io/storage-provisioner: %your provisionerName% annotation to your PVC.
provisioner name could be checked in storage class:
➜  ~ kubectl get storageclasses.storage.k8s.io efs -o yaml --export
apiVersion: storage.k8s.io/v1
kind: StorageClass
metadata:
  creationTimestamp: null
  labels:
    app: efs-provisioner
    chart: efs-provisioner-0.1.3
    env: dev
    heritage: Tiller
    release: efs-provisioner
  name: efs
  selfLink: /apis/storage.k8s.io/v1/storageclasses/efs
parameters:
  gidAllocate: "true"
  gidMax: "50000"
  gidMin: "40000"
provisioner: kubernetes.io/aws-efs
reclaimPolicy: Delete
volumeBindingMode: Immediate

what version of kubernetes?
/area aws/efs

We are using EKS one
Server Version: version.Info{Major:"1", Minor:"11+", GitVersion:"v1.11.5-eks-6bad6d", GitCommit:"6bad6d9c768dc0864dab48a11653aa53b5a47043", GitTreeState:"clean", BuildDate:"2018-12-06T23:13:14Z", GoVersion:"go1.10.3", Compiler:"gc", Platform:"linux/amd64"}

what does the StorageClass aws-efs look like?

nvm, I see it above. The issue is that provisioner is "kubernetes.io/aws-efs". It can't begin with kubernetes.io as that is reserved by kubernetes

nvm, I see it above. The issue is that provisioner is "kubernetes.io/aws-efs". It can't begin with kubernetes.io as that is reserved by kubernetes

I have the same issue and using kubernetes.io/aws-efs, what should I use then? Thanks.

IIUC you just need to change provisioner name, as kubernetes.io is reserved. i.e. if you are using efs-provisioner helm chart – set efsProvisioner.provisionerName to some other value.


SMP - lading upgrade from 0.20.8 -> 0.20.9

What does this PR do?
Updates the version of lading used in regression detector experiments to pick up on the latest improvements.
Motivation
See lading release notes
Additional Notes
Possible Drawbacks / Trade-offs
Describe how to test/QA your changes

/merge


Fix a typo in getConfigFiles_AND_Connect.md

Thanks for the PR.  The changes have been integrated into https://github.com/input-output-hk/cardano-node/pull/2399

Thanks for the PR.  The changes have been integrated into https://github.com/input-output-hk/cardano-node/pull/2598


MAFFT's usage of stdout + direct file handle redirection in run_command doesn't play nicely with some environments

mafft stdout/stderr implementation does not work in a Jupyterhub deployment.
Running external command line application. This may print messages to stdout and/or stderr.
The command being run is below. This command cannot be manually re-run as it will depend on temporary files that no longer exist.

Command: mafft --preservecase --inputorder --thread 1 /tmp/qiime2-archive-xr3zed18/1fe37b20-35c7-4a07-b0d7-f9e7a3d6b30f/data/dna-sequences.fasta

/home/jorden/dev/mc3/envs/biota/lib/python3.5/site-packages/skbio/io/registry.py:548: FormatIdentificationWarning: <_io.BufferedReader name='/tmp/q2-AlignedDNAFASTAFormat-4i04u915'> does not look like a fasta file
  % (file, fmt), FormatIdentificationWarning)

---------------------------------------------------------------------------
AssertionError                            Traceback (most recent call last)
<ipython-input-5-59f24acdb7a5> in <module>()
----> 1 biota.util.make_tree(f.DenoisedSequenceVariant)

~/dev/biota_code/biota/util.py in make_tree(seqs, tree_path, overwrite, n_threads)
    347     if not os.path.isfile(tree_path) or overwrite:
    348         q_seqs = qiime2.Artifact.import_data("FeatureData[Sequence]", seqs)
--> 349         aligned_seqs, = alignment.methods.mafft(q_seqs, n_threads=n_threads)
    350         filtered_aligned_seqs, = alignment.methods.mask(aligned_seqs)
    351         tree, = fasttree(filtered_aligned_seqs, n_threads=n_threads)

<decorator-gen-410> in mafft(sequences, n_threads)

~/dev/mc3/envs/biota/lib/python3.5/site-packages/qiime2/sdk/action.py in bound_callable(*args, **kwargs)
    226                 # Execute
    227                 outputs = self._callable_executor_(scope, callable_args,
--> 228                                                    output_types, provenance)
    229
    230                 if len(outputs) != len(self.signature.outputs):

~/dev/mc3/envs/biota/lib/python3.5/site-packages/qiime2/sdk/action.py in _callable_executor_(self, scope, view_args, output_types, provenance)
    361
    362     def _callable_executor_(self, scope, view_args, output_types, provenance):
--> 363         output_views = self._callable(**view_args)
    364         output_views = tuplize(output_views)
    365

~/dev/caporasolab/q2-alignment/q2_alignment/_mafft.py in mafft(sequences, n_threads)
     68     # while aligning, which would be a bug on mafft's end. This is just a
     69     # sanity check and is not expected to trigger in practice.
---> 70     assert len(ids) == len(msa)
     71     for id, seq in zip(ids, msa):
     72         seq.metadata['id'] = id

AssertionError:

Changing to subprocess.run(cmd, stdout=subprocess.PIPE, stderr=subprocess.PIPE) resolves the issue.

Updated to reflect the actual error encountered. Got the other ones resolved, but this is the one that showed once and is now the final issue with the way mafft's call is set up.


Update dependencies

Don't think we can bump those versions until we ditch Node 0.6.x/0.8.x support; is there a particular reason you wanted to update them?

No particular reason, just a bad habit I have to check outdated packages. But I forgot about express-winston node 0.6+ support.


PassTLSClientCert info fields are not placed in header

Bug
What did you expect to see?
Expected to see the full details of certificate.
The field serialNumber is not passed in the header.
The dashboard shows that these options are all set to true.
What did you see instead?
"X-Forwarded-Tls-Client-Cert-Info":["Subject%3D%22C%3DNL%2CST%3DZuid-Holland%2CL%3DRotterdam%2CO%3DStedin%2CCN%3Dstdcap-tbwa-101.sg.stedingroep.nl%22%2CIssuer%3D%22DC%3Dnl%2CDC%3Dstedingroep%2CDC%3Dsg%2CCN%3DSGEnterpriseRootCA%22%2CNB%3D1524071915%2CNA%3D1587143915%2CSAN%3Dstdcap-tbwa-101.sg.stedingroep.nl%2Cstdcap-tbwa-101.ee.intern%2Cstdcap-tbwa-101"]
Output of traefik version: (What version of Traefik are you using?)
traefik:v2.0.5

What is your environment & configuration (arguments, toml, provider, platform, ...)?
apiVersion: traefik.containo.us/v1alpha1
kind: Middleware
metadata:
  name: traefik-middleware-mtls
  namespace: stedin-esb-o
spec:
  passTLSClientCert:
    pem: true
    info:
      notAfter: true
      notBefore: true
      sans: true
      subject:
        country: true
        province: true
        locality: true
        organization: true
        commonName: true
        serialNumber: true
        domainComponent: true
      issuer:
        country: true
        province: true
        locality: true
        organization: true
        commonName: true
        serialNumber: true
        domainComponent: true

If applicable, please paste the log output in DEBUG level (--log.level=DEBUG switch)
time="2019-11-21T10:04:06Z" level=debug msg="vulcand/oxy/roundrobin/rr: completed ServeHttp on request" Request="{\"Method\":\"GET\",\"URL\":{\"Scheme\":\"\",\"Opaque\":\"\",\"User\":null,\"Host\":\"\",\"Path\":\"/\",\"RawPath\":\"\",\"ForceQuery\":false,\"RawQuery\":\"\",\"Fragment\":\"\"},\"Proto\":\"HTTP/1.1\",\"ProtoMajor\":1,\"ProtoMinor\":1,\"Header\":{\"Accept\":[\"*/*\"],\"User-Agent\":[\"curl/7.29.0\"],\"X-Forwarded-Host\":[\"traefik.dev.io.stedin.net\"],\"X-Forwarded-Port\":[\"443\"],\"X-Forwarded-Proto\":[\"https\"],\"X-Forwarded-Server\":[\"traefik2-58858db94d-wx6hp\"],,\"X-Forwarded-Tls-Client-Cert-Info\":[\"Subject%3D%22C%3DNL%2CST%3DZuid-Holland%2CL%3DRotterdam%2CO%3DStedin%2CCN%3Dstdcap-tbwa-101.sg.stedingroep.nl%22%2CIssuer%3D%22DC%3Dnl%2CDC%3Dstedingroep%2CDC%3Dsg%2CCN%3DSGEnterpriseRootCA%22%2CNB%3D1524071915%2CNA%3D1587143915%2CSAN%3Dstdcap-tbwa-101.sg.stedingroep.nl%2Cstdcap-tbwa-101.ee.intern%2Cstdcap-tbwa-101\"],\"X-Real-Ip\":[\"<IP_ADDRESS>\"]},\"ContentLength\":0,\"TransferEncoding\":null,\"Host\":\"traefik.dev.io.stedin.net\",\"Form\":null,\"PostForm\":null,\"MultipartForm\":null,\"Trailer\":null,\"RemoteAddr\":\"<IP_ADDRESS>:42362\",\"RequestURI\":\"/\",\"TLS\":null}"

The SerialNumber field is not part of the subject field. Therefore the config setting is never checked. I will create a pull request for this.


Adding Consumable bidder dev docs

Please see corresponding PR in Prebid.js repo for Consumable bidder adapter.

@mkendall07 sorry for the span, but does this look ok for a merge once the release is scheduled? I want to make sure we get any necessary edits in before that date so we don't miss it. Thanks.


Start scan from next block

Avoids querying and upserting the same cType/attestation every 10 minutes.

There is a small risk that events can be missed if there are multiple events in the same block. Since the only downside to the safe approach is ugly logs, we will stick with it for now.
One solution would be to switch to events API v2, but that would involve significant effort.


Upload to the chrome web store

Hi @andresusanto, can we upload this to the chrome web store? For some reason it keeps getting automatically uninstalled out of my extensions. Not sure whether it's something to do with the company's GSuite policies or not, but maybe if it wasn't sideloaded it might not get nuked :)
I'm happy to manage the store listing if you want to add me as a maintainer to the repo.

Hi @jgibo, sounds good! Thanks for volunteering, I wanted to upload this to chrome ext store for a long time, but I haven't got the time to read the docs on how to do it 😂 I've invited you to be a maintainer of the repo. Thanks!!


Force Heating ON/OFF

I want to know if I can force starting/stop heating boiler via EMS bus without original thermostat, just by EMS bus data. My boiler is : Bosch Condens 2500W

case solved


Jfm/chef18 update and version bump

Description
Updating gem dependencies and bumping us to 18.2.0 in preparation for a release. Also updating the inspec version to 5.x and removing AIX support temporarily while we resolve some hardware build/test issues
Related Issue

Types of changes

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

Checklist:

[ ] I have read the CONTRIBUTING document.
[ ] I have run the pre-merge tests locally and they pass.
[ ] I have updated the documentation accordingly.
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.
[ ] All commits have been signed-off for the Developer Certificate of Origin.

This pr is borked, hence closing.
Also correct dependencies are getting pulled in here https://github.com/chef/chef/pull/13643


Fix stable/unstable Node.js detection for Node.js >=4.0.0

Examples:

Node.js 4.1.0 is now considered stable.
Node.js 4.2.0-alpha.1 is now considered unstable.
Node.js pre-releases are now included in all but not stable,
e.g. v0.10.16-isaacs-manual.

Note that newer semver doesn't match pre-release versions so eagerly, e.g.
"^4.2.0-alpha" will match version "4.2.0-alpha.4" but will not match
"4.2.1-alpha.4".
Fix #39

Really fantastic PR, textbook, thanks @mzgol!

:clap:


Update toolbar.ts

Short description of what this resolves:
Changes proposed in this pull request:

Ionic Version: 1.x / 2.x
Fixes: #
Just a typo

thank you!


MH-12711 Make XACML parser more robust

If an element of an ACL is malformed, for example when the role is missing (which shouldn't happen but currently can happen when setting the access rights via some rest endpoints - this will be fixed with another PR), the XACML parser currently fails completely and other things with it, making the affected mediapackages almost unusable (for example when trying to view the event details or - ironically - when trying to fix the broken ACL via some rest endpoints).
This patch proposes that the parser logs an error when trying to parse a malformed ACE and skips this element instead of not catching the error at all. This is in the spirit of the existing check for role == null (which falls short if the role is missing altogether, hence the need for this patch).
This work is sponsored by SWITCH.

Does this mean, that if an XACML attachment is malformed, Opencast will fall back to the global default ACL?

@lkiesow Only if all the ACE are malformed (since the global default currently is an empty AccessList, right?). Otherwise the list will contain the ACEs that were wellformed.
Though I'm not really sure if this is the best way to go about this. Maybe we can discuss later.


[support PHP7] first attempt to commit for supporting php7

@stanley-cheung Please review this PR
I have not use the follow code style:
#if PHP_MAJOR_VERSION < 7
   // all supports php5 codes
#else
   // all supports php7 codes
#endif

But use the follow code style:
function()
{
#if PHP_MAJOR_VERSION < 7
   // all supports php5 codes in function
#else
   // all supports php7 codes in function
}

So, I only commit timeval code, we should agree on code style.
If we use second method, it will prepare to use php7_wrapper in the future.
What do you think?

Thanks for your pull request. The automated tests will run as soon as one of the admins verifies this change is ok to test.

Thanks for your pull request. The automated tests will run as soon as one of the admins verifies this change is ok to test.

Thanks for your pull request. The automated tests will run as soon as one of the admins verifies this change is ok to test.

LGTM. Please proceed with the rest. Thanks

Thanks for your pull request. The automated tests will run as soon as one of the admins verifies this change is ok to test.

Fixes #4337

Now unit test is OK on php5 and php7, please review this PR @stanley-cheung thanks!

Hi, @stanley-cheung I have squashed all commits to one commit to be good for review.
About the PR:

Supported PHP5.*, PHP7.0 and 7.1.0alpha1 by marco

Run right for all unit test cases on all PHP version(>=5.5)

Added some unit test cases

Added some TODO(like: crash, unit test)

Fixed all PHP files format by run_php_cs_fixer.sh

And then we should do this.... https://github.com/thinkerou/grpc-php7/issues/4

@murgatroid99 Can you please see if you have any major objection?
@thinkerou Thanks for the great work here. Let's hold the PR in this state for now for the review. Let's hold off on making more changes for now. Thanks!

I skimmed through it, and I don't see anything particularly out of place. I do hope that in the future, we can decrease the code duplication and us a compatibility library (like php7_wrapper appears to be).

LGTM. Let me wait for the tests to finish. Can you also make a similar PR for the v1.0.x branch please? Thanks.

Yeth, OK.

I believe at this time master and v1.0.x for PHP is at the exact same footing. So just apply the same change to the v1.0.x branch should be fine.

@stanley-cheung Done, please review #7487

Verified this compiles and installs properly on both php5 and php7 and both manually and as a pecl package. Also passed unit tests and generated code tests on both platform


Adding the liconic docker container

Hey Doga, instead of merging this here, could you actually create a new liconic_module repo for this version of the code? That way we can preserve the legacy ROS version and keep consistency with how we've handled the other REST/ROS repos.

Hey Doga, instead of merging this here, could you actually create a new liconic_module repo for this version of the code? That way we can preserve the legacy ROS version and keep consistency with how we've handled the other REST/ROS repos.

Disregard this comment, this module doesn't actually have any ROS bits, just needs to be renamed. Will do after tornado drill.

Hey Doga, left some comments and suggestions for improvements. One of the things that I figured out is that you can put the liconic_driver and liconic_rest_node.py in a directory named src to get a proper layout that creates proper python modules. Basically, we can run python -m liconic_rest_node instead of python liconic_module/scripts/liconic_rest_node.py. You can see examples in the brooks and a4s modules of how that works.


Add specs for #970

We wanna make sure it doesn't happen again

Fixed by https://github.com/atom-community/linter/pull/972


Add pick_inlaps() and pick_outlaps() functions (from #453)

It's actually unexpected that we have laps with both a pit-in and a pit-out timestamp.
Top 3 tracks where this happen in the 2023 season;
Dutch Grand Prix         20
Australian Grand Prix    17
Japanese Grand Prix       9

I checked the track map at each one and the pit lane start and end are all on different sides of the start/finish line.
One explanation is that the car pitted on two consecutive laps but that is a rather rare occurrence that doesn't explain the frequency. Any other obvious reasons that I am overlooking?

One explanation is that the car pitted on two consecutive laps but that is a rather rare occurrence that doesn't explain the frequency. Any other obvious reasons that I am overlooking?

I think this mainly happens in Free Practice or Quali. In quali, a driver could be starting a lap but then aborts, so it re-enters the pit lane in the out lap.
In Free Practice sometimes drivers do an out-in lap to "prime" the rubber, or when doing a constant speed test with aero rakes on.
Not super common, but it happens sometimes

I forgot the first lap is always an out lap quirk. That could explain why the tracks where this happens change from year to year.
FWIW I only looked at laps from the actual races

I manually looked at some of the laps that are both an inlap and an outlap in a race. Apart from the first lap, this seems to always happen around safety cars, red flags and changing weather conditions and seems reasonable.
For the first lap, actually, I think that it should have no PitOutTime except when a car starts from the pit lane. But that's a separate problem. I wouldn't try to handle the first lap separately in this new function. I think PitOutTimes should be removed from the first lap instead.
The PR looks good. If you agree that first lap should have no PitOutTime, then I'll merge this.

If you agree that first lap should have no PitOutTime, then I'll merge this.

Yeah, I agree with you. Like you said, it would make sense to only have PitOutTime for a first lap if a car starts from the pit lane. But this is possibly a more complex fix, we should maybe open a new issue for that and gather ideas?

I opened #461 to keep a reminder that the PitOutTime for first laps in a race should not exist


Expose Razor LSP specific options in VS UI

We currently have no way for the user to customize Razor options in VS LSP world. This is necessary for things like enable/disable Razor formatting.

It would be great to be able to configure these within VS.
But are there currently any Razor LSP options that can be configured outside of the VS UI?
How are these currently configured?


Go implementation Buffer.GetBytes panics

I'm digging into Go's codebase and found a bug in Buffer.GetBytes.
Here's a playpen to see it in action:
https://play.golang.org/p/3-1uLvXD83
I'll send a PR with a fix for this and the corresponding test.

woops yeah, probably want to check the Pos + length.. my bad, thanks for finding this!


[#4404] improvement: Add startup ASCII logo

What changes were proposed in this pull request?
Add startup ASCII logo to start the service script.

Why are the changes needed?
Fix: #4404
Does this PR introduce any user-facing change?
Users will see the logo when ./bin/gravitino.sh start.
How was this patch tested?
N/A

@xunliu What do you think about the style of the ASCII logo?

@zivali can you please also add this to gravitino-iceberg-rest-server.sh, maybe we can add this to a public script, what do you think?

@jerryshao Ok! How about we move this to bin/common.sh. Both gravitino.sh and gravitino-iceberg-rest-server.sh use the common.sh script.

@jerryshao Ok! How about we move this to bin/common.sh? Both gravitino.sh and gravitino-iceberg-rest-server.sh use the common.sh script.

Yeah, that's what I mean.

Can the spacing between letters be increased a little? It seems that each letter is a bit too close together from the image.

@mchades I increased the spacing a bit. What do you think?

LGTM.

@zivali would you please make the CI pass?

Overall LGTM, @zivali would you please fix the conflicts, thanks.


[stable/sentry] Bump dependencies and allow dependency naming

Is this a new chart
No
What this PR does / why we need it:
These charts have been updated a lot, since we've introduced
these dependencies.
It mainly introduces new version of Postgres and doesn't seem to break
the general functionality. It does introduce some template changes and
fixes a single problem, where policy has an empty value and causes
kubernetes to fall out in persistant volume claim.
  annotations:
    "helm.sh/resource-policy":

We can also benefit from not having Persistent Volumes.
Whilst being here, it would be nice to bump redis too.
On separate note:
It may be a case, when a user would like to install two different
instances of postgres or redis in the same namespace with a single
release. This would cause one of the instances being overwritten by the
other and lead to unwanted behaviour.
Adding this support and prefixing sentry's dependencies, may solve that
problem, and doesn't really strike as a bad practice.
Special notes for your reviewer:
Only briefly tested.
Checklist

[x] DCO signed
[x] Chart Version bumped
[x] Variables are documented in the README.md
[x] Title of the PR starts with chart name (e.g. [stable/chart])

Hi @paroxp!
I'd bump the chart version to 3.0.0 because the changes brake compatibility.
Have you the chart tested with new requirements?

Hey @okgolove,
I've bumped the version.
I have tested the chart. But not to the fullest. (Issues with my cluster not related to the chart).
It would appear, it's all scheduling up correctly, independently running. I didn't however see the full Sentry running as I got into the rabbit hole of doom not related :D
I'll try to run it somewhere, but it may be quicker for you to run it 🤔
I fully expect it to carry on working, as per https://github.com/getsentry/sentry/issues/12673

Nice catch! 😱
Also resolved some conflicts.

@paroxp
Would you bump the chart version to latest please?

hey @zanhsieh, bumped it now.

/ok-to-test

As you can see something is going wrong.

Sure. I've had a brief look into this, and to be honest I'm not sure what's wrong or if it's even related...
It seems like it fails to run the migrations. It starts with:

FATAL ERROR - The following SQL query failed: CREATE EXTENSION IF NOT EXISTS citext

If I run that docker image locally and psql into it, the above command works just fine...
I know that THIS CHANGE introduced it, but I'm lost as to where to look for clues :O

/test pull-charts-e2e

/retest

The last run succeeded. I've changed the postgresUsername from sentry to postgres... Will update the README and I think it's ready?

OK. I'm done here. Sorry if you started the tests before I made my change :(
Just waiting for the two flags :)

/retest

/lgtm

This commit caused our deployment to break. sentry-postgresql  became sentry-sentry-postgresql, same with redis.

@paroxp i think i can set nameOverride: postgresql for it to be fixed, but would this cause teh current instance of postgres to be overwritten?

also just realize, all of the vars were renamed without warning... postgresHost vs postgresqlHost


healthcare: write test for hl7v2_message_list.go

Add test for https://github.com/GoogleCloudPlatform/golang-samples/blob/master/healthcare/hl7v2_message_list.go.

Hey @noerog, any updates on this issue?

Not yet, I will add it to my next block of work and try to get it done within 2 weeks.

Thanks!


Feat: get all pokemons from API

Objective and Motivation

Using Celery Beat to download all the Pokemon at a specific day of the week.

Periodic tasks should be added in the following way: https://docs.celeryproject.org/en/stable/userguide/periodic-tasks.html#introduction. I recommend you to set the periodic tasks manually since it's the way we do it on our projects

This should be made by adding the tasks on celerybeat_schedule.py

TODO:  'offset' to move to the next page
Add a 'limit' query parameter to the GET request: ?limit=60&offset=889


Add setting to control how long "currently playing" popup lasts

It's too short to read more than a few words.

Quite honestly, I think this is more of a chrome feature--and I doubt it would be able to be messed with by plugins. As of now, you can disable them altogether, or display them. There is no option to change the delay at which they are displayed. More reading here.

Gotcha. If nobody says anything to the contrary I'll close this tomorrow.

The notification close timeout is indeed customizable, see here: https://github.com/MeoMix/StreamusChromeExtension/blob/master/src/js/background/model/chromeNotificationsManager.js#L92
I haven't had any complaints about 3 seconds in the past, but I can consider making it customizable. It's important to question whether >20% of users would customize a setting. If not, Google suggests that you should try to make an educated guess yourself. Ref:  http://www.google.com/design/spec/patterns/settings.html#settings-settings

var closeNotificationTimeout = setTimeout(this._onCloseNotificationTimeout.bind(this), 3000);

Thanks! I had no idea that you could bind a timeout.


FAILURE: Build failed with an exception.

/MainActivity.java:41: error: constructor StatusBarPackage in class StatusBarPackage cannot be applied to given types;
new StatusBarPackage(this)
^
required: no arguments
found: MainActivity
reason: actual and formal argument lists differ in length
1 error
:app:compileDebugJavaWithJavac FAILED
FAILURE: Build failed with an exception.
I have followed all the instruction of installing.
import me.neo.react.StatusBarPackage;
.
.
.
@Override
protected List<ReactPackage> getPackages() {
return Arrays.<ReactPackage>asList(
new MainReactPackage(),
new StatusBarPackage(this)
);
}
.
.
.

I'm having the same problem trying to build https://github.com/react-native-material-design/demo-app

Same issue here :/
React-native v0.32

This probably fixes your issue
https://github.com/NishanthShankar/react-native-android-statusbar/pull/18
In RN >= 0.30 you no longer need to pass this to the package
Cheers,
Miguel

works for me after remove this


[cosmetic][bar chart] bar value overlap and not in the correct bar section when stacked

Screenshot

Can we get some clarification of how this should behave when the bars are shorter than the text size?

probably no show text in this case, otherwise is gonna be hard to read(too small) or hard to refer to it(text live outside the bar) The information is available, as long as hovering to show tooltip works. @suddjian @jinghua-qa we will implement this(or better alternative) in new echarts, giving up the NVD3 😛

Closing this as stale since it's been silent for so long, and if there IS an issue with the current situation, it's different from the bug initially reported. Anyone is welcome to open a new Issue with updated context, or raise a PR to address the problem. Thanks!


Support for Google's OAuth2

new Google2Api
added Request#addParameter() to support adding parameter on GET or POST request in regards to the request's verb (Google is expecting a POST for requesting access token)
new Google2Example.

Hacked thanks to tbruyelle on https://github.com/fernandezpablo85/scribe-java/issues/117

OAuth 2.0 support is going to be improved in Scribe 2.0
Until that, we can't keep patching the (already suboptimal) OAuth2 code. Sorry


Chapter 16.1 External Commands missing crate link - error-chain

The first section of chapter 16.1 is missing the crate error-chain:

So does section 2: Run an external command passing it stdin and check for an error code

switched out to anyhow


use grep to find all a/an typos

use grep to find all a/an typos

@thaJeztah @albers
Do you know what is the wrong with the documentation test?

@allencloud I've seen it fail on other PR's, but should not be related, let me try to restart it
LGTM

retest this please

Looks like only builds with "no docs changes" succeed.

@albers I pinged @SvenDowideit, who may know more about it

retest this please

LGTM

retest this please


Sluggish cursor when used with nvim installed in wsl

Doesn't happen with windows version of neovim
Extension explicitly defines kind as "ui" but using wsl's neovim is useful
If someone has this problem with linux (non wsl)/mac version - please report here.

because of the sluggish in vscodeVim and emVim, I found this project.
I tried use nvim with vscodeVim, the sluggish is still there.
but I can't use vscode-neovim, because ESC not work, I trapped in Insert mode and can't return Normal

I tried use nvim with vscodeVim, the sluggish is still there.

vscodeVim is a completely different project: https://github.com/VSCodeVim
@transtone are you sure you chose the right extension?
@asvetliakov btw this reminds me, would you consider a different name? "vscode-neovim" is a very generic name, and rather confusing for people looking through the VScode extensions list: it is too similar to (1) vscodevim and (2) https://github.com/Chillee/VSCodeNeovim.

I tried use nvim with vscodeVim, the sluggish is still there.

vscodeVim is a completely different project: https://github.com/VSCodeVim
@transtone are you sure you chose the right extension?
@asvetliakov btw this reminds me, would you consider a different name? "vscode-neovim" is a very generic name, and rather confusing for people looking through the VScode extensions list: it is too similar to (1) vscodevim and (2) https://github.com/Chillee/VSCodeNeovim.

I use it follow this guide: https://github.com/VSCodeVim/Vim#neovim-integration

I use it follow this guide: https://github.com/VSCodeVim/Vim#neovim-integration

Again, that is a completely different project than the one you're commenting in.

I use it follow this guide: https://github.com/VSCodeVim/Vim#neovim-integration

Again, that is a completely different project than the one you're commenting in.

Again, I don't want solve VSCoceVim's bug in vscode-neovim's issue.
I said this to let author know that other vscode vim extension have sluggish cursor problem too.
it maybe upstream bug from vscode itself.

@transtone I'm aware of some performance problems, working on it
@justinmk
VSCodeNeovim is not released at all so it's not a problem. The extension is called "Neo Vim" in vscode marketplace. vscode-neovim is only for the git repo

I want use wsl's nvim, how can I set the path?

@transtone
This will be supported once 0.0.36 will be released (sometime soon)

0.0.36 has "Use WSL" config flag. When enabled specify neovim path in WSL
Alternative approach it to mark extensionKind as workspace in local config, but not recommended since there will be 2 round trips neovim <-> remote extension host <-> vscode instead one

0.0.36 has "Use WSL" config flag. When enabled specify neovim path in WSL. Required wsl.exe and wslpath binaries
Alternative approach it to mark extensionKind as workspace in local config, but not recommended since there will be 2 round trips neovim <-> remote extension host <-> vscode instead one

Is there a settings example?

@transtone
for useWSL:
"vscode-neovim.neovimPath": "/usr/sbin/nvim",
"vscode-neovim.useWSL": true

for extension kind:
    "remote.extensionKind": {
        "vscode-neovim": "workspace"
    }

Warning: you might need to install additional software in WSL and configure neovim to allow windows <-> wsl system clipboard exchange. I haven't dig into it yet.


autoFix doesn't work with markdown files

eslint v6.2.2
vscode v1.37.1
vscode-eslint v1.9.1
Repro repo here

You need to tell the extension to accept auto fixes from the markdown plugin. Simply use the following line in the validate setting: { "language": "markdown", "autoFix": true }


[FR] Add CLI

I like to use this as a CLI, without Obsidian. Something like:
kindle-clippings file.txt -to /some/dir -template someTemplate

This is out of scope for this plugin that operates only within Obsidian.
With that said, I will consider splitting out the kindle scraping and parsing engine into a separate library that can be used as CLI or a separate app outside of Obsidian.


Bug - material-icons not showing on Blackberry 10

The material-icons are not visible on BlackBerry 10.  Tested on Z10 using 10.3.1.  Here's a screenshot of the Android demo page, with the invisible material-icon menu and material-icon search on the left and right, respectively.
I can click on the menu icon, and it the menu opens.

I thought I'd send this out there in case someone has an idea what's going on.  Otherwise, I'll put a little time into debugging this, and update this week-end with results.

I’d guess that this is related to https://github.com/google/material-design-icons/issues/179
I don’t have any Blackberries to test nor I know much about their browser engine, so any help with this is greatly appreciated, @listedegarde :) Maybe you can test the workaround mentioned in the issue above.

Thanks for that suggestion!  You saved me a lot of debugging time.  Indeed, the workaround from that post works.  .material-icons {text-rendering: optimizeLegibility; }
I'm not sure how you deal with issues, but I suppose this could be closed as a duplicated of issue 179.

Yeah, we ran into the same issue wie Edge. Thanks for testing and letting us now :) I’m optimistic that this issue will be fixed upstream soon-ish.


not able to run VSCode python on Raspian (Rpi 3 B)

VSCode Version: Code - OSS 1.14.0 (05b254dcfb44422238c6fdb665721a41ee4ce6ac, 2017-06-20T20:22:03.289Z)
OS Version: Linux arm 4.14.34-v7+
Extensions:
Extension|Author (truncated)|Version
---|---|---
python|don|0.7.0
python|tht|0.2.3

Installed the CODE-OSS on raspian but not able to run test code:
import  numpy as np
x=1
y=2
z=(x+y)
print(Z)
Debugging receiving variables errors! with a long list of "ArithmeticError" and others.
Does somebody got VSCode running on Ranpian?

Steps to Reproduce:

That just sounds like a Python, or maybe an issue with the Python extension.

It was,
I've been programing with VS code on W10 and there the "shebang" line is not necessary. Debian runuing python on a armf processor needs!!!
begginers mistake, Thanks anyway!


Pornhubpremium (and regular pornhub maybe) Pornstar page and playlist not working

Checklist

[ x] I'm reporting a broken site support
[ x] I've verified that I'm running youtube-dl version 2020.03.24
[ x] I've checked that all provided URLs are alive and playable in a browser
[ x] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[x ] I've searched the bugtracker for similar issues including closed ones

Verbose log

C:\Users\M>youtube-dl.exe -f best https://www.pornhubpremium.com/pornstar/gabbie-carter -o C:/PH/Pornstar/GabbieCarter/%(uploader)s/Uploaded/%(title)s-%(id)s-%(duration)s.%(ext)s -i --restrict-filenames --write-description --write-info-json --add-metadata --embed-thumbnail --download-archive C:\archive.txt --cookies "C:\Users\M\Downloads\cookies.txt" -v
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: ['-f', 'best', 'https://www.pornhubpremium.com/pornstar/gabbie-carter', '-o', 'C:/PH/Pornstar/GabbieCarter/%(uploader)s/Uploaded/%(title)s-%(id)s-%(duration)s.%(ext)s', '-i', '--restrict-filenames', '--write-description', '--write-info-json', '--add-metadata', '--embed-thumbnail', '--download-archive', 'C:\\archive.txt', '--cookies', 'C:\\Users\\M\\Downloads\\cookies.txt', '-v']
[debug] Encodings: locale cp1252, fs mbcs, out cp850, pref cp1252
[debug] youtube-dl version 2020.03.24
[debug] Python version 3.4.4 (CPython) - Windows-10-10.0.18362
[debug] exe versions: ffmpeg 3.4, ffprobe 3.4
[debug] Proxy map: {}
[PornHubPagedVideoList] pornstar/gabbie-carter/videos: Downloading page 1
[download] Downloading playlist: pornstar/gabbie-carter/videos
[PornHubPagedVideoList] playlist pornstar/gabbie-carter/videos: Collected 0 video ids (downloading 0 of them)
[download] Finished downloading playlist: pornstar/gabbie-carter/videos

Description

For some reasons, some pornstars pages do not have a '/videos' page. Going to this URL comes up with an error about the page not existing. It seems that the pornstar name acts as the videos page. To see the premium videos the URL looks like
https://www.pornhubpremium.com/pornstar/gabbie-carter?premium=1&o=mr
Putting this into youtube-dl results in it trying to look for /videos again and then failing
I tried to work around by clciking play all as playlist then download the playlist but it just downloads the first one and not the rest even with --yes-playlist .
The output for that is
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: ['-f', 'best', 'https://www.pornhubpremium.com/view_video.php?viewkey=ph5e90c1c172d99&pkey=sys%3Apornstar%3A306792%3Amr%3A1%3Apremium%3A1', '-o', 'C:/PH/Pornstar/GabbieCarter/%(uploader)s/Uploaded/%(title)s-%(id)s-%(duration)s.%(ext)s', '-i', '--restrict-filenames', '--write-description', '--write-info-json', '--add-metadata', '--embed-thumbnail', '--download-archive', 'C:\archive.txt', '--cookies', 'C:\Users\M\Downloads\cookies.txt', '-v', '--yes-playlist']
[debug] Encodings: locale cp1252, fs mbcs, out cp850, pref cp1252
[debug] youtube-dl version 2020.03.24
[debug] Python version 3.4.4 (CPython) - Windows-10-10.0.18362
[debug] exe versions: ffmpeg 3.4, ffprobe 3.4
[debug] Proxy map: {}
[PornHub] ph5e90c1c172d99: Downloading pc webpage
[PornHub] ph5e90c1c172d99: Downloading m3u8 information
[PornHub] ph5e90c1c172d99: Downloading m3u8 information
[PornHub] ph5e90c1c172d99: Downloading m3u8 information
[PornHub] ph5e90c1c172d99: Downloading m3u8 information
[PornHub] ph5e90c1c172d99: Downloading m3u8 information
WARNING: There's no description to write.
[info] Writing video description metadata as JSON to: C:\PH\Pornstar\GabbieCarter\Primal_s_Edging_Handjobs_and_Blowjobs\Uploaded\2_Gorgeous_Cum_Hungry_Babes_Share_Cock_and_Cum_-Alex_Coal_Gabbie_Carter-ph5e90c1c172d99-1368.info.json
[PornHub] ph5e90c1c172d99: Downloading thumbnail ...
[PornHub] ph5e90c1c172d99: Writing thumbnail to: C:\PH\Pornstar\GabbieCarter\Primal_s_Edging_Handjobs_and_Blowjobs\Uploaded\2_Gorgeous_Cum_Hungry_Babes_Share_Cock_and_Cum-Alex_Coal_Gabbie_Carter-ph5e90c1c172d99-1368.jpg
[debug] Invoking downloader on 'https://dv2.phprcdn.com/videos/202004/10/302297312/1080P_4000K_302297312.mp4?ttl=1587553227&ri=51200000&rs=400000&hash=8c5a931f4d25f450467783915b363604'
[download] Destination: C:\PH\Pornstar\GabbieCarter\Primal_s_Edging_Handjobs_and_Blowjobs\Uploaded\2_Gorgeous_Cum_Hungry_Babes_Share_Cock_and_Cum-Alex_Coal_Gabbie_Carter-ph5e90c1c172d99-1368.mp4
[download] 100% of 362.69MiB in 03:27
[ffmpeg] Adding metadata to 'C:\PH\Pornstar\GabbieCarter\Primal_s_Edging_Handjobs_and_Blowjobs\Uploaded\2_Gorgeous_Cum_Hungry_Babes_Share_Cock_and_Cum-Alex_Coal_Gabbie_Carter-ph5e90c1c172d99-1368.mp4'
[debug] ffmpeg command line: ffmpeg -y -loglevel "repeat+info" -i "file:C:\PH\Pornstar\GabbieCarter\Primal_s_Edging_Handjobs_and_Blowjobs\Uploaded\2_Gorgeous_Cum_Hungry_Babes_Share_Cock_and_Cum-Alex_Coal_Gabbie_Carter-ph5e90c1c172d99-1368.mp4" -c copy -metadata "date=20200410" -metadata "title=2 Gorgeous Cum Hungry Babes Share Cock and Cum - Alex Coal & Gabbie Carter" -metadata "purl=https://www.pornhubpremium.com/view_video.php?viewkey=ph5e90c1c172d99&pkey=sys%3Apornstar%3A306792%3Amr%3A1%3Apremium%3A1" -metadata "artist=Primal's Edging Handjobs and Blowjobs" "file:C:\PH\Pornstar\GabbieCarter\Primal_s_Edging_Handjobs_and_Blowjobs\Uploaded\2_Gorgeous_Cum_Hungry_Babes_Share_Cock_and_Cum-Alex_Coal_Gabbie_Carter-ph5e90c1c172d99-1368.temp.mp4"
[atomicparsley] Adding thumbnail to "C:\PH\Pornstar\GabbieCarter\Primal_s_Edging_Handjobs_and_Blowjobs\Uploaded\2_Gorgeous_Cum_Hungry_Babes_Share_Cock_and_Cum-Alex_Coal_Gabbie_Carter-ph5e90c1c172d99-1368.mp4"
[debug] AtomicParsley command line: AtomicParsley "C:\PH\Pornstar\GabbieCarter\Primal_s_Edging_Handjobs_and_Blowjobs\Uploaded\2_Gorgeous_Cum_Hungry_Babes_Share_Cock_and_Cum-Alex_Coal_Gabbie_Carter-ph5e90c1c172d99-1368.mp4" --artwork "C:\PH\Pornstar\GabbieCarter\Primal_s_Edging_Handjobs_and_Blowjobs\Uploaded\2_Gorgeous_Cum_Hungry_Babes_Share_Cock_and_Cum-Alex_Coal_Gabbie_Carter-ph5e90c1c172d99-1368.jpg" -o "C:\PH\Pornstar\GabbieCarter\Primal_s_Edging_Handjobs_and_Blowjobs\Uploaded\2_Gorgeous_Cum_Hungry_Babes_Share_Cock_and_Cum-_Alex_Coal_Gabbie_Carter-ph5e90c1c172d99-1368.temp.mp4"

You must provide account credentials.

that wasn't an issue for the initial request to support pornhubpremium...
As this same issue (looking for /videos that doesn't exist) is applicable to pornhub.com also, no account credentials are not required.


drop babel

I don't see any reason to use here babel.
BTW:  +1 to allow issues in this repo, because they are not related to the email templates itself and also not to panini. For instance why do you use babel?

We keep the issue trackers on all template repos closed (this one, one for Apps, and two for Sites) to keep the issues consolidated in one place. Unlike Panini, which is used by two of the frameworks, the templates are all framework-specific. We maintain over a dozen repositories for the Foundation project now, so anywhere we can consolidate helps.
As for using an ES6 Gulpfile, it's partially a "because we can" thing, and partially because we're trying to stay on the edge of web standards. We're using Gulp 4, we use ES6 in Foundation for Sites, and we'll be using TypeScript for Foundation for Apps 2.
We also want all of the Gulpfiles for each project to follow the same format, naming conventions, and so on. And part of that is writing it in ES6.


Propagate method in python API

I saw that in the python api we cannot use the propagate function component. One may want to propagate all constraints without search for a solution, as it is described in the docs. Conveniently, in Java we can do it, using the propagate function via the solver. Can this be part of the python API too?

At first sight, I would say that this requires to provide also two methods :

pushLevel() (or saveSate())
popLevel() (or restoreState())
to deal with failures.

That is not undoable but should be used carefully.

I agree with Charles, if it's possible in the Java API it should be doable in the Python API, but this is kind of an unusual and advanced usage of a CP solver, so if we add this possibility we should be sure to do it in a way that will not confuse CP beginners.

Thanks, no rush, was just something I found useful when using in in Java and think that it can be useful also in Python. Not a priority

Fixed in ee568f93682d1814afcf05e18d3eb69857f5863b
The methods are in the solver API, and named _propagate, _push_state, and _pop_state (I used the underscore character to highlight that these are kind of "private" methods)