updating cluster ran into a hiccup, proceeded to succeed, warning message from UI wont go away

What kind of request is this (question/bug/enhancement/feature request):
bug
Steps to reproduce (least amount of steps as possible):

updated a cluster via the UI (there may have been a temporary communication issue with a specific node)
subsequent updates to the cluster succeed, even for the host reporting a warning against

Result:
error message in ui wont go away:
Provisioning incomplete, host(s) [host_ip] skipped because they could not be contacted
Other details that may be helpful:
Environment information

Rancher version (rancher/rancher/rancher/server image tag or shown bottom left in the UI):
2.1.1
Installation option (single install/HA):
HA

I encountered the same issue when changing a cluster setting and keeping the "edit cluster" window open for too long. As a result, 2 of my 3 etcd/control nodes were listed as unable to contact.
Any following change to the cluster was only carried out on the first etcd/control node and did not carry on past the two "unreachable" nodes.
I cordoned, drained and rebooted the nodes each individually and eventually the cluster did recover and finish its configuration change.
Environment

Rancher: v2.1.6
Kubernetes: 1.11.6
Docker-CE: 18.09.2
Single node Rancher setup with custom clusters on Ubuntu 16.04.5 LTS.


Potential bug in conjugate_gradient and general_conjugate_gradient

Hello,
I was using ACGD for a constrained optimization problem with the Lagrange multiplier method. I let the primal parameters be the min player and the Lagrange multiplier be the max player.
With the Lagrange multiplier having a dimension of 1, after the first iteration my nsteps equals 0 after after line 168. This creates an error in line 200 if i > 100:.
I currently solved it by inserting in line 178: i = 0. The results I got make sense.
Without going too much into the theory while using this, I was wondering if ACGD is the correct choice and if my fix makes sense.
Thank you!

Yes! ACGD is the right choice for Lagrange Multipliers for constrained optimization problem.
Thanks for pointing out this corner case. I just fix it in the latest version. Please let me know if it solves the problem.
Best

Hi thank you it worked for ACGD. I believe you did not change conjugate_gradient so CGD might run into the same problem.


Add which checking in gvm-installer

Some minimal installed Linux distributions didn't bundled which package,
So gvm-installer will complain a mistake error - Could not find git
The pull request fix this problem.

So pity this problem still exists.

@phuslu Apologies! We are closing stale PRs. If you want to open another one... I think the actual solution would be to use command -v which, or use command -v ... in place of which


Examples does not work.

First of all, thanks for implementing this project. It has become one of the standard packages I have been using.
(I installed dependencies through requirements.txt)
I ran the examples and got the following error:
(hotreloadtest) Huseyins-MacBook-Air-2:hot-reload huseyin$ python manage.py runserver
Traceback (most recent call last):
  File "manage.py", line 8, in <module>
    from django.core.management import execute_from_command_line
ImportError: No module named django.core.management
(hotreloadtest) Huseyins-MacBook-Air-2:hot-reload huseyin$

I am guessing that problem is wrong django version . I tried to find the correct django version by trying out each one, but I was not successful.
I think examples might need version pinned requirements and python version noted on README.

It seems you haven't activated the virtualenv. This error can't have anything to do with django-webpack-loader, when not even Django is found.

Oups. You are absolutely right. This was a problem with my environment. Upon closer inspection I found the problem. Thanks for pointing that out.

@huseyinyilmaz  Excuse me, can you tell me how to solve this problem?

@Bongsopha It seems like that was happening because I did not install the dependencies correctly.


firezone version 0.3.3-1 when changing default['enterprise']['name'] = 'example' firezone-ctl reconfigure failed

HI,
on firezone version 0.3.3-1 when I change default['enterprise']['name'] = 'firezone' to default['enterprise']['name'] = 'example' firezone-ctl reconfigure failed:
# firezone-ctl reconfigure
Starting Chef Infra Client, version 16.17.51
Patents: https://www.chef.io/patents
resolving cookbooks for run list: ["firezone::default"]
Synchronizing Cookbooks:
  - packagecloud (1.0.1)
  - firezone (0.0.1)
  - enterprise (1.2.0)
  - runit (5.1.6)
  - line (4.4.4)
  - yum-epel (4.4.1)
Installing Cookbook Gems:
Compiling Cookbooks...

================================================================================
Recipe Compile Error in /var/opt/firezone/cache/cache/cookbooks/firezone/recipes/default.rb
================================================================================

Chef::Exceptions::ValidationFailed
----------------------------------
install_path is a required property

Cookbook Trace: (most recent call first)
----------------------------------------
  /var/opt/firezone/cache/cache/cookbooks/enterprise/recipes/runit.rb:32:in `block in from_file'
  /var/opt/firezone/cache/cache/cookbooks/enterprise/recipes/runit.rb:28:in `from_file'
  /var/opt/firezone/cache/cache/cookbooks/firezone/recipes/postgresql.rb:23:in `from_file'
  /var/opt/firezone/cache/cache/cookbooks/firezone/recipes/default.rb:13:in `from_file'

Relevant File Content:
----------------------
/var/opt/firezone/cache/cache/cookbooks/enterprise/recipes/runit.rb:

 25:  node.override['runit']['sv_dir']       = "#{install_path}/sv"
 26:  node.override['runit']['lsb_init_dir'] = "#{install_path}/init"
 27:
 28:  component_runit_supervisor node['enterprise']['name'] do
 29:    ctl_name node[node['enterprise']['name']]['ctl_name'] ||
 30:             "#{node['enterprise']['name']}-ctl"
 31:    sysvinit_id node[node['enterprise']['name']]['sysvinit_id']
 32>>   install_path node[node['enterprise']['name']]['install_path']
 33:  end
 34:

System Info:
------------
chef_version=16.17.51
platform=ubuntu
platform_version=20.04
ruby=ruby 2.7.5p203 (2021-11-24 revision f69aeb8314) [x86_64-linux]
program_name=/opt/firezone/embedded/bin/chef-client
executable=/opt/firezone/embedded/bin/chef-client

Running handlers:
[2022-04-08T11:19:16+01:00] ERROR: Running exception handlers
Running handlers complete
[2022-04-08T11:19:16+01:00] ERROR: Exception handlers complete
Chef Infra Client failed. 0 resources updated in 04 seconds
[2022-04-08T11:19:17+01:00] FATAL: Stacktrace dumped to /var/opt/firezone/cache/cache/chef-stacktrace.out
[2022-04-08T11:19:17+01:00] FATAL: Please provide the contents of the stacktrace.out file if you file a bug report
[2022-04-08T11:19:17+01:00] FATAL: Chef::Exceptions::ValidationFailed: install_path is a required property

Best regards.

Thanks for the report @rchekhina ! Looks like the install directory needs to match the enterprise cookbook name. So I can understand a little better -- can you shed some light on what you're aiming to achieve by changing the enterprise cookbook name? It's used more as an internal configuration variable thus far.

HI Jamil,
Ok I was thinking that the parameter was to define the name of my company. So I will close this issue.

HI Jamil,
Ok I was thinking that the parameter was to define the name of my company.
So you can close this issue.
Best regards.
On Fri, Apr 8, 2022 at 7:38 PM Jamil @.***> wrote:

Thanks for the report @rchekhina https://github.com/rchekhina ! Looks
like the install directory needs to match the enterprise cookbook name.
So I can understand a little better -- can you shed some light on what
you're aiming to achieve by changing the enterprise cookbook name? It's
used more as an internal configuration variable thus far.
—
Reply to this email directly, view it on GitHub
https://github.com/firezone/firezone/issues/530#issuecomment-1093123655,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ASWWRPCTCI3I4N5ABD4BOQDVEBVHNANCNFSM5S4DFP7Q
.
You are receiving this because you were mentioned.Message ID:
@.***>

--
Reynald CHEKHINA
SA, Square1
e: @.** @.**>
Company Registration Number: 528714
VAT number IE3178050PH


lifepower driver issue

Describe the bug
I see that my (previously working) revov driver has been replaced with the lifepower one;
Unfortunately It doesn't work on my older revov tianpower unit though.
@40000000647dc05509b3aa5c INFO:SerialBattery:Testing Lifepower
@40000000647dc05518895964 ERROR:SerialBattery:Unexpected err=IndexError('list index out of range'), type(err)=<class 'IndexError'>
Will do a bit of debugging and see whats going on.
From a cursory look, I see he's calling 7E vs 7C on the "modbus" call, first stop I guess will be to change to 7C calls see if that resolves, then go from there.
How to reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior
Expect working code ;)
Driver version
latest stable
Venus OS device type
Raspberry Pi
Venus OS version
3.0
BMS type
Revov
Cell count
16
Connection type
Serial USB adapter to RS485
Config file
.

Relevant log output
.

Any other information that may be helpful
.

snuck mine back in for some debugging.  Mine definitely prefers the 7C call, not the Lifepower driver 7E modbus call, so ultimately this is going to need a different driver type than the LifePower one unfortunately.

For reference a sample packet reply is below;
i.e. a full reply from a 7C:01:01:00:02:0D request -
[7C:01:01:76:01:10:8D:5E:0D:5F:8D:62:0D:6A:8D:78:0D:5B:8D:C3:0D:59:8D:6B:0D:47:8D:59:0D:47:8D:7E:0D:52:8D:3B:0D:3A:02:01:70:ED:03:01:27:10:04:01:46:50:05:03:00:42:00:42:20:44:06:05:00:00:00:01:00:00:00:00:00:00:07:01:00:C9:08:01:15:67:09:01:27:10:0A:01:00:00:0B:01:00:00:E2:F7:0C:01:00:00:3D:D0:0D:01:00:40:00:00:0E:01:00:40:00:00:0F:01:00:24:EE:9A:10:01:00:09:C6:B9:80:0D]
Breaking that down;
id under 10 are 2 x byte value length, id values > 10 are 4 x byte value length.
in format of     ID: Length: Data
Type, 1,1 length (hex)
7C:01:01:76:
first data type, length (16 x 2 byte value)
01:10
Data 16 x 2 bytes
8D:3C:0D:39:8D:41:0D:47:8D:4E:0D:58:8E:3A:0E:76:8D:49:0D:27:8D:65:0D:73:8D:30:0D:48:8D:35:0D:36
02:01
75:30
03:01
27:10
04:01
46:50
05:03:
00:42:00:42:20:44
06:05
00:00:00:00:00:00:00:00:00:01
07:01
00:C9
08:01
15:73
09:01
27:10
0A:01
00:00
---- From here 0xB onward 01 length indicates 4 bytes not 2!
0B:01
00:00:E2:F7
0C:01
00:00:3D:D0
0D:01
00:40:00:00
0E:01
00:40:00:00
0F:01
00:24:EE:9A
10:01
00:09:C6:B9
END
7E:0D

Liked PR https://github.com/Louisvdw/dbus-serialbattery/pull/694

Any updates on this?

I had issues getting this LifePower.py driver working with Eg4-LL 12v BMS. Found that my BMS is using different commands, then what was written by other in the Revov or the LifePower Driver. Spent some time writing up something that works on my device. Have issues on the Test.connect still (errors out), but the driver still picks it up, and the daemon is able to pull the stats with out fail. Reboots do not require any extra work to get this driver restarted.
https://github.com/tuxntoast/eg4-ll/blob/main/egll.py

@tuxntoast feel free to open a PR for the dev branch.


Use codecov v4, single upload step, and non-codecov reports

This PR solves https://github.com/napari/napari-graph/pull/13#issuecomment-2141196170 and moves coverage upload to a separate step to make the CI logs cleaner.
It also adds support for GHA-only coverage reports using the method from
hynek.me/articles/ditch-codecov-python

Nice! Thanks @Czaki! However it is currently failing in a weird way. 😅

Nice! Thanks @Czaki! However it is currently failing in a weird way. 😅

It is not verid. I hit this every time I migrate to this schema. it is only problem with paths between operating systems...

@jni it is ready to review.
Also contains hynek.me/articles/ditch-codecov-python

It looks like branch coverage is handled differently on codecov.io and normal report.

Amazing! Thank you @Czaki!


Option to not close the Safari tab

I use a redirect rule to open YouTube pages in Video Lite. Often I will search for something in Safari and open a link from the Google results. If that link is a YouTube video then I can’t get back to the search results because Redirect has closed the Safari tab.

Hi @patrickrushton, I believe it's because you selected the corresponding option on the redirection screen here:

You can disable it either on the screen above or on the following Edit screen:

Whould you please take a look?
By the way, there is an issue that the navigation history is cleared when opening another app using Redirect Web. This is caused by Safari and it's tracked in this issue.: https://github.com/mshibanami/redirect-web/issues/52

I don't seem to have a checkbox "Close the tab automatically" on iOS. I also don't have the "Application" select box. I'm just directing to a videolite:// url scheme.

@patrickrushton I see. Then it must be the same issue as #52.
Yes, that option is only for macOS, and the tab you believe is closed is actually the same tab displaying a blank page bundled with the extension, which clears the history, making it seem like a new tab.
Currently, there is no workaround. Sorry for the inconvenience. However, as you can see in #52, Apple acknowledged this issue a week ago. So it could be fixed soon.

Ah okay. Thanks for the explanation. Feel free to mark this one as a duplicate then.


config: override brand for learner dashboard MFE in MITxOnline QA

What are the relevant tickets?
https://github.com/mitodl/hq/issues/1499, Specifically https://github.com/mitodl/brand-mitol-residential/pull/1#issuecomment-1584648648
Description (What does it do?)

Adds an override for brand-edx.org and uses our own custom brand
The override should only affect the learner dashboard MFE and Residential MITx-CI for now.

There are a lot of details mentioned in https://github.com/mitodl/hq/issues/1499

Screenshots (if appropriate):
Screenshots can be seen in https://github.com/mitodl/hq/issues/1499
How can this be tested?

Once deployed, We should not see any Upgrade button or Looking for New Challange card

Additional Context

@blarghmatey This is ready for review now.


Can´t open details of query / mutation

Great tool, but since yesterday can´t open details of query / mutation, just empty window..
While inspecting the app, the console shows:
Uncaught TypeError: Cannot convert undefined or null to object
at Function.keys ()
at main.84ea91ac.chunk.js:1
at Array.map ()
at re (main.84ea91ac.chunk.js:1)
at Xi (153.37de9f8c.chunk.js:2)
at wl (153.37de9f8c.chunk.js:2)
at gu (153.37de9f8c.chunk.js:2)
at mu (153.37de9f8c.chunk.js:2)
at lu (153.37de9f8c.chunk.js:2)
at 153.37de9f8c.chunk.js:2
Tried on more than one website, restarted the computer, etc..
chrome Version 92.0.4515.107 (Official Build) (64-bit)

Thanks for providing the error details. I'm looking into this.

@pbsmultimedia could you provide a URL for a website this error occurred?


Missing inherited methods sections

I think this is a regression from https://github.com/crystal-lang/crystal/pull/14910, but haven't expcliitly confirmed it yet. Basically a repeat of https://github.com/crystal-lang/crystal/pull/14878#issuecomment-2288744060.
On latest 1.13 release: https://crystal-lang.org/api/1.13.3/Array.html, you can see the Included Modules section and related methods inherited from ... sections from those modules at the end of the method summaries.
However on master: https://crystal-lang.org/api/master/Array.html, Included Modules and the inherited methods sections of those modules are missing.

Yes, seems to be related to #14910, that it's exposing a bug for Crystal::Doc::Generator#must_include?. I don't think it's handling generic types correct.
Adding
  def must_include?(type : Crystal::GenericInstanceType)
    type = self.type(type.generic_type)
    must_include? type
  end

seems to do the trick.

For future reference: I compared the doc output from #15055 against master using diff on index.json.
$ git checkout master
$ make docs
$ mv docs docs-master
$ git checkout $branch
$ make crystal docs
$ diff --color <(jq 'del(.. | .url?)' docs-master/index.json) <(jq 'del(.. | .url?)' docs/index.json)

jq 'del(.. | .url?)' pretty prints the JSON for line-based diffing, and it removes all url attributes because they differ between every commit (they are permalinks to the specific commit version of the source file).
I haven't found an easy way to compare the HTML output. There's a lot more noise there and harder to filter out. index.json should usually be fine to assess doc generator content output.


use tqdm to display progress over bags

Compare with the content changes in PR #23 .  In that PR I got into a bit of a mess with incorrectly resolved merge conflicts.  To be safe, I'm just recreating the changes here.


JSqlParser can't parse DATABASE function in mysql.

        select count(*)
        from information_schema.columns
        where table_name = 'extend_value_info'
          and column_name = 'extend_value_str'
          and table_schema = DATABASE();

throws:
Caused by: net.sf.jsqlparser.parser.ParseException: Encountered unexpected token: "=" "="
    at line 5, column 28.

Was expecting one of:

    "&"
    "::"
    ";"
    "<<"
    ">>"
    "COLLATE"
    "CONNECT"
    "EMIT"
    "GROUP"
    "HAVING"
    "START"
    "["
    "^"
    "|"
    <EOF>

	at net.sf.jsqlparser.parser.CCJSqlParser.generateParseException(CCJSqlParser.java:33398) ~[jsqlparser-4.5.jar:na]
	at net.sf.jsqlparser.parser.CCJSqlParser.jj_consume_token(CCJSqlParser.java:33231) ~[jsqlparser-4.5.jar:na]
	at net.sf.jsqlparser.parser.CCJSqlParser.Statement(CCJSqlParser.java:167) ~[jsqlparser-4.5.jar:na]
	at net.sf.jsqlparser.parser.CCJSqlParserManager.parse(CCJSqlParserManager.java:23) ~[jsqlparser-4.5.jar:na]
	... 54 common frames omitted

Is there any workround?
I try to around the DATABASE with "``" like this:
        select count(*)
        from information_schema.columns
        where table_name = 'extend_value_info'
          and column_name = 'extend_value_str'
          and table_schema = `DATABASE()`;

Then mysql can't parse it.

Greetings!
Please download/use the Keywords PR #1382 which supports proper Whitelisting of non-reserved keywords. Your statement will work as shown here

@manticore-projects  When this pr will be merged? It seems that almost a year passed and this pr hasn't been merged. What' problem?

@manticore-projects When this pr will be merged? It seems that almost a year passed and this pr hasn't been merged. What' problem?

I can't say for sure. Maybe @wumpz has many things on his plate. Unfortunately I can't merge it as I am just an interested user.


Has anyone tried without Docker?

Has anyone else got this error too and successfully resolved it?

Reduce batch size. If you don't use GPU, put use_gpu = False


Older torch versions torch<2.0.0 are not supported by metatensor python modules

I am not sure if this is relevant but I tested it with python 3.8.
With torch version 1.13.1+cpu
================================================================================================
FAILED tests/atomistic/ase_calculator.py::test_torch_script_model - RuntimeError: The following operation failed in the TorchScript interpreter.
FAILED tests/atomistic/ase_calculator.py::test_exported_model - RuntimeError: The following operation failed in the TorchScript interpreter.
FAILED tests/atomistic/ase_calculator.py::test_serialize_ase - RuntimeError: The following operation failed in the TorchScript interpreter.
FAILED tests/operations/equal_metadata.py::test_operations_as_python - pytest.PytestUnraisableExceptionWarning: Exception ignored in: <_io.FileIO [closed]>
FAILED tests/operations/sort.py::test_operation_as_torch_script - RuntimeError: The following operation failed in the TorchScript interpreter.
FAILED tests/operations/sort.py::test_operation_as_python - IndexError: tensors used as indices must be long, byte or bool tensors
ERROR tests/learn/linear.py::test_linear_single_block_tensor - AttributeError: module 'torch' has no attribute 'set_default_device'
ERROR tests/learn/linear.py::test_linear_from_weight - AttributeError: module 'torch' has no attribute 'set_default_device'
ERROR tests/learn/linear.py::test_torchscript_linear - AttributeError: module 'torch' has no attribute 'set_default_device'
ERROR tests/learn/module_map.py::test_module_map_single_block_tensor[None] - AttributeError: module 'torch' has no attribute 'set_default_device'
ERROR tests/learn/module_map.py::test_module_map_single_block_tensor[out_properties1] - AttributeError: module 'torch' has no attribute 'set_default_device'
ERROR tests/learn/module_map.py::test_torchscript_module_map - AttributeError: module 'torch' has no attribute 'set_default_device'
======================================================================================= 6 failed, 158 passed, 6 errors in 15.62s ========================================================================================
torch-tests: exit 1 (15.97 seconds) /home/alexgo/code/metatensor/python/metatensor-torch> pytest --cov=/home/alexgo/code/metatensor/.tox/torch-tests/lib/python3.8/site-packages/metatensor --cov-append --cov-report= --import-mode=append -W error --tb=short pid=1298569
  torch-tests: FAIL code 1 (86.13=setup[1.23]+cmd[67.03,0.69,0.66,0.56,15.97] seconds)
  evaluation failed :( (86.19 seconds)

Tests work fine with torch version 2.0.0+cpu.
Since metatensor-torch in principle supports Torch 1.11 according to the CMake file, version ``1.13.1+cpu` could be in principle supported. We probably need to check how easy it is to fix these and decide by that on a minimum version for the modules and then add these to the CI jobs.

So we do run CI with pytorch 1.11, but only for the C++ tests. I think we should keep supporting old-ish torch versions for a bit (maybe moving the minimal version to 1.13), and add the corresponding tests to CI!


Misunderstanding the problems with room reverberation (RIR)

Misunderstanding the problems with room reverberation (RIR) and how easily they can be created

Room Impulse Response (RIR) is an audio signal processing task that involves capturing and analyzing the acoustic characteristics of a room

https://ohf-voice.github.io/wake-word-collective/ 'the wake word while you walk around the room'
Its extremely hard (beamforming) to remove room impulse response, but very easy to create with many github projects to do this  https://github.com/LCAV/pyroomacoustics is one of many that can accurately add RIR
You only need <0.3m recordings that have very low RIR that can create many distances accurately with tools such as pyroomacoustics.
This allows you to provide datasets for certain devices as for a device with beamforming walking around a room including large RIR (large rooms) will greatly increase dataset entropy and resultant models will be less accurate as the beamforming will attenuate RIR.
Also any model created with samples containing RIR due to the nature of RIR sound will bounce off walls and arrive at the mic at different time periods due to differing distances.
These mix at the mic and the more distant the mic the more the recorded spectra will differ as more reverberation and mixing will happen creating very different harmonics.
At least if your recording at a distance supply metadata of that distance so a dataset can be filtered to only include near <0.3 recordings and also allow you to be sure you have an even spread than create dataset bias.
Metadata is hugely important, Recording Device, Recording distance, Gender, Age Band, Lang, Region (Reginal accents), is essential so that you can filter and create evenly spread datasets or tailor a dataset to a type of metadata for more accuracy.
From what I can see https://ohf-voice.github.io/wake-word-collective/ is going to create datasets only for devices without any form of RIR attenuation (beamforming and such) and create innacurate models as there is a limit to how much RIR you can include in samples as further distances to the mic in big rooms give big differences in spectra and greatly increasing dataset entropy and hence lowering model accuracy.
You can include a bit of RIR with non beamforming devices to increase accuracy of more distant wakewords but the tradeoff is overall accuracy and without the ability to filter too distant RIR for this type of device the dataset is pure potluck of what environment users submit.
Bemused as always.
Stuart

Common voice almost got things correct but failed in a few areas.
1... Meta data is crucial (recording device, record distance, gender, age group, Lang, region (accent), native speaker) all allow filtering to create balanced datasets. Even though none of the above is specific or private Common Voice failed to enforce and do has very little meta data.
2... Likely over enthusiasm of non native English speaking universities created a huge bias at least with the English dataset with a singular non native regional accent distinctly different to native speakers. Metadata filtering can fix this.
3... Validating recordings for some didn't work well as bad or wrong samples still exist.
Apart from that common voice gave a pretty much perfect way to collect speech sentences.
The only difference is here we are capturing words not sentences so on screen it should prompt for words.
Only a single dataset is needed for all types of keyword models and ! Keywords of the same voice are extremely important to some model types.
https://clagnut.com/blog/2380/#English_phonetic_pangrams
Phonetic tangrams are short sentences in any language that contain the distinct soundings of that language and make a great balanced !kwdataset that can be easily and quickly recorded especially interspersed with kw.
Having an open source keyword dataset of any keyword would allow many different model types to be created for different platforms. It does need meta data, ! Keywords and doesn't need distant recordings as they can be accurately recreated. https://github.com/DavidDiazGuerra/gpuRIR is another.
Also capturing KW @ the KWS is very easy as you are sampling that window and your KWS authenticates as does the subsequent actions of the smart speaker .
Having an option in after setting the metadata could quickly garner much data through use and provide the 1st open source dataset to get anywhere the volume and quality big data have without asking for option in.
On device recording hugely increases accuracy for datasets filtered for that device.

Thanks for the feedback

Just to check you did understand the 'feedback'. The way you are collating the current dataset will result in for most an inaccurate and relatively useless one.
Opensource has never had a comparative KW dataset comparative to bigdata.
To actually get an opensource iniative to collect a KW dataset but get it sorted wrong is extremely disheartening when the fixed are relatively easy to fix.
You can not include large RIR in a dataset hence why beamforming is such an important tech for smart speaker tech.
!KW of the same voice are also important and also the metadata to create even none skewed datasets.
The issue has been closed and brushed off as feedback whilst what you have will create a pretty useless dataset that could be fixed in a normal issue and reply manner


Add subchannel update damper

This adds a configurable option for ServiceProxy to debounce requests to update service channels by a minimum period (default 0). The period can be altered by an external agent (remote configuration).
The updateServiceChannels function is not covered by our tests currently. None of our tests appear to exercise ring changes.
Without configuring the min update period, this retains the current behavior, except using setTimeout instead of setImmediately, effectively introducing an implicit minimum delay of 4ms.
r @Raynos

lgtm.
Recommend hooking up minUpdatePeriod to flipr before deploying.
Recommend spamming restart in travis until you get the "lucky green".

@Raynos Added remote config.

Still lgtm :P


WABS Backup failing with "unprocessed exception has avoided all error handling"

The logs look like this:
Jul 25 21:36:26 sv-db1 requests.packages.urllib3.connectionpool INFO     Starting new HTTPS connection (1): svproddbbackup.blob.core.windows.net
Jul 25 21:36:27 sv-db1 wal_e.worker.upload INFO     MSG: completed archiving to a file#012        DETAIL: Archiving to "wabs://db-backup/wal_005/0000000500003313000000D5.00000028.backup.lzo" complete at 00KiB/s.#012        STRUCTURED: time=2016-07-26T01:36:27.035835-00 pid=28070 action=push-wal key=wabs://db-backup/wal_005/0000000500003313000000D5.00000028.backup.lzo prefix= rate=00 seg=0000000500003313000000D5.00000028.backup state=complete
Jul 25 21:36:27 sv-db1 wal_e.blobstore.wabs.wabs_util WARNING  MSG: retrying file upload from unexpected exception#012        DETAIL: The exception type is <class 'select.error'> and its value is (4, 'Interrupted system call') and its traceback is   File "/usr/local/lib/python2.7/dist-packages/wal_e/retries.py", line 62, in shim#012            return f(*args, **kwargs)#012          File "/usr/local/lib/python2.7/dist-packages/wal_e/blobstore/wabs/wabs_util.py", line 90, in upload_chunk#012            block_id, content_md5=check_sum)#012          File "/usr/local/lib/python2.7/dist-packages/azure/storage/blob/blobservice.py", line 2366, in put_block#012            self._perform_request(request)#012          File "/usr/local/lib/python2.7/dist-packages/azure/storage/storageclient.py", line 171, in _perform_request#012            resp = self._filter(request)#012          File "/usr/local/lib/python2.7/dist-packages/azure/storage/storageclient.py", line 160, in _perform_request_worker#012            return self._httpclient.perform_request(request)#012          File "/usr/local/lib/python2.7/dist-packages/azure/storage/_http/httpclient.py", line 181, in perform_request#012            self.send_request_body(connection, request.body)#012          File "/usr/local/lib/python2.7/dist-packages/azure/storage/_http/httpclient.py", line 143, in send_request_body#012            connection.send(request_body)#012          File "/usr/local/lib/python2.7/dist-packages/azure/storage/_http/requestsclient.py", line 81, in send#012            self.response = self.session.request(self.method, self.uri, data=request_body, headers=self.headers, timeout=self.timeout)#012          File "/usr/local/lib/python2.7/dist-packages/requests/sessions.py", line 475, in request#012            resp = self.send(prep, **send_kwargs)#012          File "/usr/local/lib/python2.7/dist-packages/requests/sessions.py", line 585, in send#012            r = adapter.send(request, **kwargs)#012          File "/usr/local/lib/python2.7/dist-packages/requests/adapters.py", line 403, in send#012            timeout=timeout#012          File "/usr/local/lib/python2.7/dist-packages/requests/packages/urllib3/connectionpool.py", line 578, in urlopen#012            chunked=chunked)#012          File "/usr/local/lib/python2.7/dist-packages/requests/packages/urllib3/connectionpool.py", line 385, in _make_request#012            httplib_response = conn.getresponse(buffering=True)#012          File "/usr/local/lib/python2.7/dist-packages/wal_e/cmd.py", line 32, in monkey#012            return original(*args, **kwargs)#012          File "/usr/lib/python2.7/httplib.py", line 1051, in getresponse#012            response.begin()#012          File "/usr/lib/python2.7/httplib.py", line 415, in begin#012            version, status, reason = self._read_status()#012          File "/usr/lib/python2.7/httplib.py", line 371, in _read_status#012            line = self.fp.readline(_MAXLINE + 1)#012          File "/usr/lib/python2.7/socket.py", line 476, in readline#012            data = self._sock.recv(self._rbufsize)#012          File "/usr/local/lib/python2.7/dist-packages/requests/packages/urllib3/contrib/pyopenssl.py", line 207, in recv#012            [self.socket], [], [], self.socket.gettimeout())#012          There have been 1 attempts to upload  file wabs://db-backup/basebackups_005/base_0000000500003313000000D5_00000040/tar_partitions/part_00000228.tar.lzo so far.#012        STRUCTURED: time=2016-07-26T01:36:27.306487-00 pid=20812
Jul 25 21:36:27 sv-db1 requests.packages.urllib3.connectionpool INFO     Starting new HTTPS connection (2): svproddbbackup.blob.core.windows.net
Jul 25 21:36:27 sv-db1 requests.packages.urllib3.connectionpool INFO     Starting new HTTPS connection (3): svproddbbackup.blob.core.windows.net
Jul 25 21:36:27 sv-db1 requests.packages.urllib3.connectionpool INFO     Starting new HTTPS connection (4): svproddbbackup.blob.core.windows.net
Jul 25 21:36:28 sv-db1 wal_e.main   CRITICAL MSG: An unprocessed exception has avoided all error handling#012        DETAIL: Traceback (most recent call last):#012          File "/usr/local/lib/python2.7/dist-packages/wal_e/cmd.py", line 578, in main#012            pool_size=args.pool_size)#012          File "/usr/local/lib/python2.7/dist-packages/wal_e/operator/backup.py", line 197, in database_backup#012            **kwargs)#012          File "/usr/local/lib/python2.7/dist-packages/wal_e/operator/backup.py", line 502, in _upload_pg_cluster_dir#012            pool.put(tpart)#012          File "/usr/local/lib/python2.7/dist-packages/wal_e/worker/upload_pool.py", line 108, in put#012            self._wait()#012          File "/usr/local/lib/python2.7/dist-packages/wal_e/worker/upload_pool.py", line 65, in _wait#012            raise val#012        error: (4, 'Interrupted system call')#012        #012        STRUCTURED: time=2016-07-26T01:36:28.635565-00 pid=20812

The interrupted system call is 4 which seems to indicate a timeout has occurred?

The error message could afford to be improved, but to retry is at least the right behavior. Does it ever succeed?

It actually does not succeed. We actually hadn't had a backup complete in about 10 days without realizing it and we triggered a backup every night. I switched to the S3 integration and that seems to be going smoothly.

Are you hosting in Azure? I am wondering if your link to WABS was slow enough somehow that it caused timeouts.

Hosted in digital ocean and they don't have an equivalent service so we have to go outside.

We are seeing this very often backing up to S3. Is there a way to get more detailed information about the nature of the failure?

I don't have any specific advice on the matter, no.


SW-5230 Remove shapefile validation options

Early on in the development of the tracking features, all our test shapefiles were
invalid in one way or another, so we needed to be able to disable the various
validation checks in order to test the code.
Now that we have shapefiles whose geometries are valid and whose only "problem" is
that they sometimes lack target planting densities for zones, there's no need to
continue supporting invalid files. Remove the concept of "validation options" from
the importer and start treating target planting density as an optional value with
a default.

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#1889  👈
#1887
#1886
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @sgrimm and the rest of your teammates on  Graphite

Merge activity

Apr 26, 6:19 PM EDT: @sgrimm started a stack merge that includes this pull request via Graphite.


Differences with ABAP Formatter

I think this has come up before, but it would be good if abap-cleaner doesn't unintentionally contradict the basics of what the ABAP formatter will do. Some findings:

Text Elements: The Formatter considers TEXT a keyword so if you are using uppercase it does this TEXT-01 but abap cleaner forces lowercase text-01.
Program selection screen definitions: The Formatter will force everything to have no indentation, but abap-cleaner adds indentation. Assuming this is this intended?

SELECTION-SCREEN BEGIN OF BLOCK block WITH FRAME TITLE text-01.
  PARAMETERS param1 RADIOBUTTON GROUP r1.

  SELECTION-SCREEN BEGIN OF LINE.
    PARAMETERS param2 RADIOBUTTON GROUP r2.
  SELECTION-SCREEN END OF LINE.
SELECTION-SCREEN END OF BLOCK block.

Sometimes I see nested types used, and the indentation isn't quite right. I'd want this, which is what the Formatter does:

TYPES:
  BEGIN OF some_type.
    INCLUDE TYPE nested_type.
    TYPES: field TYPE some_field,
  END OF some_type.

But abap-cleaner does this:
TYPES: BEGIN OF some_type.
         INCLUDE TYPE nested_type.
TYPES:   field TYPE some_field,
       END OF some_type.

The ABAP formatter actually does format those examples of yours like that, on a very recent release... Unfortunately the results of the ABAP formatter are backend release dependant.

The ABAP formatter actually does format those examples of yours like that, on a very recent release... Unfortunately the results of the ABAP formatter are backend release dependant.

Id be fine with that for examples 1 & 2. But does it format like abap-cleaner does even for 3? That behavior seems very odd.

Hi Josh,
in my ADT, ABAP Formatter indeed puts this to upper case TEXT-01 – and shows an error saying '"TEXT-01" is not a numbered text (for example: TEXT-001).' Doesn't it expect three digits on your system?
Example 2 is formatted by ABAP Formatter exactly as shown in your code, so it is indented, which I think makes sense and is consistent with formatting of BEGIN OF ... END OF blocks in other cases like TYPES.
Regarding example 3 – yes, it is really unfortunate that we have to break a TYPES chain for an INCLUDE! In my ADT, ABAP Formatter does this:
TYPES:
  BEGIN OF some_type.
    INCLUDE TYPE nested_type.
TYPES: field TYPE farr_contract_id,
  END OF some_type.

… which looks really messy to me. Your example …
TYPES:
  BEGIN OF some_type.
    INCLUDE TYPE nested_type.
    TYPES: field TYPE some_field,
  END OF some_type.

… actually looks better for my taste, but then, isn't it a bit inconsistent to have the TYPES keyword in two completely different positions? The second one is even indented further than END OF!
That's why ABAP cleaner tries to have the TYPES keywords at the same vertical position (because they are at the same level of indentation), while moving the whole BEGIN OF ... END OF block a bit to the right, so it can be formatted as if it all were a single chain:
TYPES: BEGIN OF some_type.
         INCLUDE TYPE nested_type.
TYPES:   field TYPE some_field,
       END OF some_type.

So, you could say it was a conscious decision against what ABAP Formatter does…
Kind regards,
Jörg-Michael

Hi Josh,
in my ADT, ABAP Formatter indeed puts this to upper case TEXT-01 – and shows an error saying '"TEXT-01" is not a numbered text (for example: TEXT-001).' Doesn't it expect three digits on your system?

Yes, this is just a typo.

Example 2 is formatted by ABAP Formatter exactly as shown in your code, so it is indented, which I think makes sense and is consistent with formatting of BEGIN OF ... END OF blocks in other cases like TYPES.

I would prefer the indentation also, so this is just a difference of versions apparently.

Regarding example 3 – yes, it is really unfortunate that we have to break a TYPES chain for an INCLUDE! In my ADT, ABAP Formatter does this:
TYPES:
  BEGIN OF some_type.
    INCLUDE TYPE nested_type.
TYPES: field TYPE farr_contract_id,
  END OF some_type.

… which looks really messy to me. Your example …
TYPES:
  BEGIN OF some_type.
    INCLUDE TYPE nested_type.
    TYPES: field TYPE some_field,
  END OF some_type.

… actually looks better for my taste, but then, isn't it a bit inconsistent to have the TYPES keyword in two completely different positions? The second one is even indented further than END OF!
That's why ABAP cleaner tries to have the TYPES keywords at the same horizontal position (because they are at the same level of indentation), while moving the whole BEGIN OF ... END OF block a bit to the right, so it can be formatted as if it all were a single chain:
TYPES: BEGIN OF some_type.
         INCLUDE TYPE nested_type.
TYPES:   field TYPE some_field,
       END OF some_type.

So, you could say it was a conscious decision against what ABAP Formatter does…
Kind regards, Jörg-Michael

Point 3 is certainly a unique situation. On one hand, yes, I would like all TYPES keywords in the same position, but I feel like the nested type is an exception, since it belongs inside the BEGIN/END block. I'm not sure I like it, but I have found that there is a "solution" which satisfies both abap-cleaner and the ATC checks we have for indentation in abapOpenChecks:
TYPES BEGIN OF some_type.
        INCLUDE TYPE nested_type.
TYPES   field TYPE some_field.
TYPES END OF some_type.

Not sure if others use this type of syntax, but I suppose it's a (not very obvious) compromise.

Closing this since there's a workaround and it's system version-dependent.

Hi Josh,
alright, thanks! Just to have some label assigned, I think this is somewhere in between "won't fix" and "no defect" …
Kind regards,
Jörg-Michael


feat: add opentelemetry 1.1.0 release

update opentelemetry with release 1.1.0

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


Replace ESCAPE by DOWN key to re-show picker 2

Replace ESCAPE by DOWN key to re-show picker, rebase for #827

Thanks @fabdouglas see #1305 for the full PR!


Consider more renaming for clarity

If cv() is now convert_encoding(), it probably makes sense to rename sani() into sanitize()...
Also - doesn't cv_string now need to be converted_encoding_string ?
More renaming ideas:

underlying_outbuf::pos(), outbuf::pos() etc. - very confusing! Readers will tend to assume this method returns a numeric value. But - I don't have a great idea for an alternative name... at_current_pos()/at_current_position() ? Hmm.
outbuf -> output_buffer - these days, I think streambuf would not have made it past the language standard committee...

The documentation still says cv() and sani().

... and, actually, some places in the code still use sani() or cv(). Example.

sani still exits, but it is an alias to sanitize_encoding or sanitize_from_encoding.
cv renamed to conv, which is an alias to convert_encoding or convert_from_encoding.
outbuf renamed to outbuff.
outbuf::pos renamed to outbuff::pointer.


Call into Keplr Extension via OfflineSigner interface

Depends on #264
A web dApp should be able to use this same Wallet interface that it uses to access local key material stored in the browser, to send a signing request to the Keplr extension. Ideally work with @thunnini to get the client code working. This has two steps:

[ ] Submit a simple x/bank SendMsg to the Keplr extension to be properly signed
[ ] Add support for x/wasm CosmWasm 0.9 messages to Keplr and sign them with the extension (requires many changes to Keplr)

Bonus:

[ ] Figure out how to test such integration in CI - node/webapp calling extension that posts tx to local blockchain in CI.

I was thinking about how to demo this and do we need to build a minimal wallet as a frontend app to show integration.
Then I came across https://github.com/chainapsis/keplr-example/blob/master/src/main.js which is used to demo calling the extension from a webapp using cosmosjs. We can port this code over to use cosmjs and provide this as a proof of concept that the integration works

There is a great start at https://github.com/chainapsis/keplr-extension/commit/15b2d37523c6046a8acd6f797af93cb5208819b0 and I was thinking we could do 2 things to move this forward:

make a PR off of that branch to update to latest cosmjs commits (with launchpad support) and get this to compile
try to test it out with a sample script

1 should be pretty straight forward. (at least following our api changes and keplr build process).
For 2, how would you test it? (in develop mode)

I was thinking to do something like this: https://github.com/chainapsis/keplr-example/blob/master/src/main.js but using cosmjs interfaces rather than cosmosjs.
I was thinking of something like this:

ensure cosmjs works with some deployed network supported by keplr (cosmos-hub?)

if not, we set one up ourselves and fork keplr for this...

fork keplr-example
modify the code to use cosmjs not cosmosjs to sign the message
yarn dev
open the page in my browser without keplr plugin (get error message)
open the page in my browser with keplr plugin (see signing request)

We don't actually need to submit the tx to the cosmoshub (I doubt you have the tokens), but we can see that the request makes it properly to keplr and is displayed there.
Anything I am missing?

I made the cosmjs branch for testing cosmjs integration before merging to master. https://github.com/chainapsis/keplr-example/blob/comsjs/src/main.js
This uses cosmjs and its offline signer capability. It probably works with the current keplr extension on cosmjs branch.
And, you can test it by running npm run dev command.
Right now, I am working to add some more test codes to keplr extension repo, and the current main aim is to support the multiple accounts and Ledger, and support the feature to be able to add new chains dynamically. I expect the feature to add chains dynamically to be implemented within 2 weeks (released with a warning the users that it is still an experimental feature). After this feature is implemented, we expect that people will no longer have to build custom keplr extension versions to support their custom chains. But until then, you will have to build a separate Keplr extension that supports specific custom chains.
The issue I can see is that the cosmjs wallet interface hasn’t been fixed yet, so we are not able to release cosmjs Keplr into production yet. So even though the ‘dynamically add new chains’ feature is complete, we may have to build custom versions at least temporarily.

Thank you @Thunnini
I believe @willclarktech will soon make a PR into your cosmjs branch that gets it working with the most recent cosmjs code. (yes, pre-release). I assume there are no automated tests for the injectors (he's manually trying the custom build to send tokens on the cosmos hub).
And yes, obviously this is not going into master until (1) CosmJS has a stable release working with this launchpad. Also, if I understand correctly, (2) we need to implement the OnlineSigner interface to allow the Keplr extension to set gas, sign and post. Correct?

See

https://github.com/chainapsis/keplr-extension/pull/21
https://github.com/chainapsis/keplr-example/pull/3

Thanks, done


Realsense R200 error: */camera/driver - Error calling rs_device_supports_option

Hi,
I get this error when trying to use the turtlebot3 waffle Realsense R200 camera using the command : "roslaunch turtlebot3_bringup turtlebot3_realsense.launch" or the command: "roslaunch realsense_camera r200_nodelet_default.launch"
[ERROR] [1517504560.255244866]: /camera/driver - Error calling rs_device_supports_option ( device:0x2369600, option:R200_AUTO_EXPOSURE_RIGHT_EDGE ):
UVCIOC_CTRL_QUERY:UVC_GET_CUR error 2, No such file or directory
I followed all the instructions available here (http://wiki.ros.org/librealsense) and here (http://emanual.robotis.com/docs/en/platform/turtlebot3/appendix_realsense/#installation)
This is what i get:
livre@livre-570x-DVT3:~$ roslaunch turtlebot3_bringup turtlebot3_realsense.launch
... logging to /home/livre/.ros/log/75eae038-0763-11e8-abc6-74e6e2d140c0/roslaunch-livre-570x-DVT3-15405.log
Checking log directory for disk usage. This may take awhile.
Press Ctrl-C to interrupt
Done checking log file disk usage. Usage is <1GB.
started roslaunch server http://<IP_ADDRESS>:40131/
SUMMARY
PARAMETERS

/camera/driver/base_frame_id: camera_link
/camera/driver/camera_type: R200
/camera/driver/color_fps: 30
/camera/driver/color_frame_id: camera_rgb_frame
/camera/driver/color_optical_frame_id: camera_rgb_optica...
/camera/driver/depth_frame_id: camera_depth_frame
/camera/driver/depth_optical_frame_id: camera_depth_opti...
/camera/driver/fisheye_frame_id: camera_fisheye_frame
/camera/driver/fisheye_optical_frame_id: camera_fisheye_op...
/camera/driver/imu_frame_id: camera_imu_frame
/camera/driver/imu_optical_frame_id: camera_imu_optica...
/camera/driver/ir2_frame_id: camera_ir2_frame
/camera/driver/ir2_optical_frame_id: camera_ir2_optica...
/camera/driver/ir_frame_id: camera_ir_frame
/camera/driver/ir_optical_frame_id: camera_ir_optical...
/camera/driver/mode: manual
/camera/driver/serial_no:
/camera/driver/usb_port_id:
/rosdistro: kinetic
/rosversion: 1.12.12

NODES
/camera/
driver (nodelet/nodelet)
nodelet_manager (nodelet/nodelet)
ROS_MASTER_URI=http://<IP_ADDRESS>:11311
process[camera/nodelet_manager-1]: started with pid [15414]
process[camera/driver-2]: started with pid [15415]
[ INFO] [1517504505.609302519]: Initializing nodelet with 4 worker threads.
Intel RealSense F200_camera ; <IP_ADDRESS>
Intel RealSense LR200_camera ; <IP_ADDRESS>
Intel RealSense R200_camera ; <IP_ADDRESS>
Intel RealSense SR300_camera ; <IP_ADDRESS>
Intel RealSense ZR300_adapter ; <IP_ADDRESS>
Intel RealSense ZR300_camera ; <IP_ADDRESS>
Intel RealSense ZR300_motion_module ; <IP_ADDRESS>
[ INFO] [1517504507.857575789]: /camera/driver - Detected the following camera:
- Serial No:<PHONE_NUMBER>, USB Port ID: 2-2-4, Name: Intel RealSense R200, Camera FW: <IP_ADDRESS>
[ WARN] [1517504507.857724890]: /camera/driver - Detected unvalidated firmware:
-<PHONE_NUMBER>'s current camera firmware is <IP_ADDRESS>, Validated camera firmware is <IP_ADDRESS>
[ INFO] [1517504507.857843712]: /camera/driver - Connecting to camera with Serial No:<PHONE_NUMBER>, USB Port ID: 2-2-4
[ INFO] [1517504550.035593896]: /camera/driver - Setting static camera options
[ INFO] [1517504560.255059257]: /camera/driver - Enabling Depth in manual mode
[ERROR] [1517504560.255244866]: /camera/driver - Error calling rs_device_supports_option ( device:0x2369600, option:R200_AUTO_EXPOSURE_RIGHT_EDGE ):
UVCIOC_CTRL_QUERY:UVC_GET_CUR error 2, No such file or directory
[FATAL] [1517504591.531324716]: Failed to load nodelet '/camera/driverof typerealsense_camera/R200Nodeletto managernodelet_manager'
[camera/driver-2] process has died [pid 15415, exit code 255, cmd /opt/ros/kinetic/lib/nodelet/nodelet load realsense_camera/R200Nodelet nodelet_manager depth:=depth color:=color ir:=ir ir2:=ir2 fisheye:=fisheye imu:=imu __name:=driver __log:=/home/livre/.ros/log/75eae038-0763-11e8-abc6-74e6e2d140c0/camera-driver-2.log].
log file: /home/livre/.ros/log/75eae038-0763-11e8-abc6-74e6e2d140c0/camera-driver-2.log
[camera/nodelet_manager-1] process has died [pid 15414, exit code -11, cmd /opt/ros/kinetic/lib/nodelet/nodelet manager __name:=nodelet_manager __log:=/home/livre/.ros/log/75eae038-0763-11e8-abc6-74e6e2d140c0/camera-nodelet_manager-1.log].
log file: /home/livre/.ros/log/75eae038-0763-11e8-abc6-74e6e2d140c0/camera-nodelet_manager-1.log**
all processes on machine have died, roslaunch will exit
shutting down processing monitor...
... shutting down processing monitor complete
done
livre@livre-570x-DVT3:~$

I have no idea what to do about it.
Thanks.

Hi,
I followed this instructions  "http://wiki.ros.org/realsense_camera/Tutorials/Building_from_Sources" and after that I used this command "sudo apt-get --reinstall install 'ros-*-librealsense'" And the error disappeared.

@RubensQRZ, please feel free to reopen if you encounter this issue again.


saving empty profiler

<!---   *** BEFORE SUBMITTING: PASTE CLIPBOARD HERE TO COMPLETE YOUR REPORT ***   ---!>
1. Profiler not showing any data
2. Clicked "Save data"
3. Tried to save on desktop
4. This error

Hey @Czimchik, thanks for reporting. Please post the error you're getting to understand what's happening in your case.

Hey @Czimchik, thanks for reporting. Please post the error you're getting to understand what's happening in your case.

Closing due to lack of response.

Closing due to lack of response.


Deploy functions needs update to aws lambda runtime nodejs 6.10 > nodejs8.10

For bug reports, please fill in the next sections:
Current behavior
Creating service service in cluster shared-eu-west-1... ✔
Bundling functions... ⣟
Bundling functions... 13.1s
Deploying... 776ms
There are issues with the new service definition:

  Global
    ✖ software.amazon.awssdk.services.lambda.model.InvalidParameterValueException: The runtime parameter of nodejs6.10 is no longer supported for creating or updating AWS Lambda functions. We recommend you use the new runtime (nodejs8.10) while creating or updating functions. (Service: null; Status Code: 400; Error Code: InvalidParameterValueException; Request ID: ee0cd923-76b8-11e9-a12b-2349b42e70b4)

Reproduction
I don't know how to reproduce or fix  this bug, because in the graphcool.yml there isn't a runtime variable to just set to nodejs8.10.
Expected behavior?
It should just deploy as I do before in the pasts months without problems.
What can I do to solve this bug during the deploy of a graphcool service?

It is fixed now

I also get this problem..

How to solve it...

Yep, same error here. Just appeared out of the blue today.

I'm also getting the same issue.

So what is the fix?

So what is the fix?

I've found "the fix" unfortunately it only applies to new deployments not updating existing lambda functions. If you are trying to update production code like me I think you're out of luck.
It's from @dpetrick
https://github.com/prisma/graphcool-framework/commit/617ce95e2a4b68bc025e0d9206b5abdaf201df22#diff-0df7b5f86ea502ce516847555a3a9bed
Clearly I'm no expert on the internals but if you look at the rest of the code update seems to ignore the node runtime which would explain what's happening. Line 113 and beyond.
https://github.com/prisma/graphcool-framework/blob/master/server/backend-shared/src/main/scala/cool/graph/shared/functions/lambda/LambdaFunctionEnvironment.scala
It seems like this could be inserted between lines 128 and 129 to fix the issue
.runtime(runtime)
@dpetrick Are you able to help?

This started to happen again today out of the blue.
software.amazon.awssdk.services.lambda.model.InvalidParameterValueException: The runtime parameter of nodejs6.10 is no longer supported for creating or updating AWS Lambda functions.

Received and advice from graphcool support and it solved the issue:
"You will need to redeploy the function in a new name so that it can reset the runtime."

"You will need to redeploy the function in a new name so that it can reset the runtime."

Thanks for relaying that here @romanmandryk ... and it makes complete sense as the runtime is included in the create code but not the update code but that doesn't help when someone can't redeploy code as quickly that is calling the named functions such as in a mobile phone app.
Didn't get an ETA on it being resolved?

Can any

Received and advice from graphcool support and it solved the issue:
"You will need to redeploy the function in a new name so that it can reset the runtime."

Please could you mind giving a bit more detail on how to do this? I'm stuck and cant figure out what needs to be done. I've renamed the function but the issue still occurs.

You need to rename all your functions. Not just one. The deployment fails if even one fails.
Unfortunately for me and I'm sure many others I cant update the place the functions are called from all at once so it is not a good enough solution.

@shokimble Oh wow! That's hectic! At the very least is it a one time thing or would you need to rename your functions every time you need to deploy?

@shokimble Thank you for your help with this 😄

@shokimble @romanmandryk
I've gone through and tried to rename all my functions but unfortunately, it's not worked. I'm guessing we're going to have to wait till this is rectified in a new graphcool-framework cli release

Any news on this?

We have deployed a fix for this: https://github.com/prisma/graphcool-framework/commit/dc274418ba663caad8c157a8d8690127e64e6541
Please test this out now 🙏

Phew! Finally, thanks.
I can confirm the deploy worked without errors. I haven't checked it actually updates your function but let's assume it does. If not I'll report back.

Hello,
I have the same issue from node6 to node 12 with a fresh install of graphcool. How do I install the fix?
Thanks

Same issue here with v8.10 and renaming functions does not work for me.


[FLINK-32123][kafka] Fix E2E kafka packages download timeout

What is the purpose of the change
Fix E2E kafka packages download timeout
Brief change log
Fix E2E kafka packages download timeout
Verifying this change
This change is a test package download fix without any test coverage.
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (no)
The serializers: (no)
The runtime per-record code paths (performance sensitive): (no)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn, ZooKeeper: (no)
The S3 file system connector: (no)

Documentation

Does this pull request introduce a new feature? (no)
If yes, how is the feature documented? (not documented)

Duplicate of https://github.com/apache/flink/pull/22603


feat: create article new feature

Drill4J Bot - Change Testing Report
Comparing 9088800 (current) to [d1cc5b5](http://localhost:8095/dashboard/2?build=realworld%3Aspring-webflux%3Ad1cc5b5917597b8b407f13189c3299911ff888ff) (baseline).
Changes
1 method (0 new, 1 modified)
Risks
All changes tested
100% coverage
Recommended tests
19 tests
See details in Drill4J


Error running logger

This is the error I get after installing with luarocks and configuring openresty with the example:
2022/10/22 20:37:00 [error] 328#328: *9 failed to run body_filter_by_lua*: ...ocal/share/lua/5.1/usagelogger/http_logger_for_nginx.lua:25: bad argument #1 to 'pairs' (table expected, got string)
stack traceback:
	[C]: in function 'pairs'
	...ocal/share/lua/5.1/usagelogger/http_logger_for_nginx.lua:25: in function 'bodyfilter'
	body_filter_by_lua:3: in main chunk while sending to client, client: 10.######, server: demo.######.com, request: "GET /?test=value HTTP/2.0", upstream: "https://origin/?test=value", host: "demo.######app"
2022/10/22 20:37:00 [error] 328#328: *11 lua entry thread aborted: runtime error: /usr/local/share/lua/5.1/usagelogger/http_message.lua:11: request required
stack traceback:
coroutine 0:
	[C]: in function 'assert'
	/usr/local/share/lua/5.1/usagelogger/http_message.lua:11: in function 'send'
	...ocal/share/lua/5.1/usagelogger/http_logger_for_nginx.lua:46: in function <...ocal/share/lua/5.1/usagelogger/http_logger_for_nginx.lua:41>, context: ngx.timer, client: 10.######, server: <IP_ADDRESS>:443

The same error occurs with the Resurface container:  https://github.com/resurfaceio/test-openresty

reverting to 2.0-0 only fixes the first error in bodyfilter.  the second error still remains.

further testing shows that ngx.ctx.req and ngx.ctx.res are nil

Hi there @ne4u, thank you for taking the time to open this issue. I saw it earlier today but it was just now that I was able to reply.
I believe the issue had to do with two main things: not parsing the qs query string into a table and not initializing the req.params table before attempting to index a key. With those two things fixed, I made a new release 2.1.1-1in order to test it with resurfaceio/test-openresty, and it works! (on my end at least). Could you try installing the latest version of this package and running the example test again, please?
The dockerfile in that repo needs to be modified that so that a whole new release is not required just to test a specific commit, tag or branch of this one. That and writing more unit tests are priorities for this logger.

Thanks!  I can confirm the error persists.  It took either several tries or several minutes to get the error though. That could be why you didn't notice it.  But before I got the error, I noticed that no data was being sent by doing packet captures.  Here's the same error with the update line numbers:
2022/10/25 02:24:49 [error] 7#7: *4 lua entry thread aborted: runtime error: ...nresty/luajit/share/lua/5.1/usagelogger/http_message.lua:11: request required
stack traceback:

coroutine 0:
[C]: in function 'assert'
...nresty/luajit/share/lua/5.1/usagelogger/http_message.lua:11: in function 'send'
...ajit/share/lua/5.1/usagelogger/http_logger_for_nginx.lua:69: in function <...ajit/share/lua/5.1/usagelogger/http_logger_for_nginx.lua:64>, context: ngx.timer, client: 10.###.###.###, server: <IP_ADDRESS>:80
Again it seems that ngx.ctx.req and ngx.ctx.res are nil.  I verified by altering http_logger_for_nginx.lua to write to nginx error.log if they are nil and not call ngx.timer.at()
Hopefully, that helps.

Part of my problem not seeing the traffic with tcpdump was due to not specifying a resolver in nginx when using a FQDN for the logger url.  When adding a resolver I see the traffic.  However, the POST is nil since the res and req are nil as previously noted.

This may explain the why and how to fix:
https://forum.openresty.us/d/6211-ef6e7fd3ec824a040bfc7143acc7f153

Hi @ne4u! I'm just seeing your updates —is the 2nd error still showing up for you? If it isn't anymore after modifying your env config, could you share what was it that you changed? I'd like to provide better error handling for cases like that (at least provide more useful error messages)

i'm not sure what exactly i changed that fixed the issue.  I do know that adding the resolver directive helped.  but i also know that shouldn't have impacted the nil value I know was coming from res and req variables.  As I noted I added debug logging to validate that they where nil.  I do have some quite complete ngnix configurations that I suspect may have been the culprit.  After stripping the configs to a bare minimum things started working. now nginx/openresty is strickly a reverse proxy.
I'm not sure if it makes sense for Resurface platform.  However, it does make sense to me to include some additional details about the request which i'm putting in body_filter_by_lua_block:
ngx.ctx.res.headers['Connnection-ID'] = ngx.var.connection
ngx.ctx.res.headers['Request-Count'] = ngx.var.connection_requests
ngx.ctx.res.headers['Pipelining'] = ngx.var.pipe
ngx.ctx.res.headers['Client-Address'] = ngx.var.remote_addr
ngx.ctx.res.headers['Request-URI'] = ngx.var.request_uri
ngx.ctx.res.headers['Query-String'] = ngx.var.query_string
From a security and ML perspective that data seems relevant.


I couldn't edit using the iD editor on FF 53.02.

I couldn't edit using the iD editor on FF 53.02.
Also, the browser control reports the iD link is to "http://preview.ideditor.com/"

Hi @govvin, this is an issue with iD being served over http. One workaround for now is to enable loading unsafe scripts on your browser.

cc: @bhousel

In Firefox, even after disabling protection (which I suppose is the equivalent for this) didn't work for me.

@ajithranka
I just tried this on Chrome (Linux), and got this error from the browser console after allowing it load unsafe scripts:

This should be fixed once iD moves to https.

cc @iandees and @miccolis
It would be really great to get https working on the preview.ideditor.com domain.


lxd-unix.socket systemd file does not set mode

I'm not sure where the lxd packages are created, hence the bugreport here.
The lxd-unix.socket systemd unit file needs 'SocketMode=0660' to prevent the socket from being world writable.

Also see #1308 which should be combined with this fix.

As mentioned in the pull request, I don't think it's LXD's job to check and enforce the right mode on inherited sockets. However this is a critical issue with the Ubuntu packaging which I'm now fixing.
Thanks!

Fix uploaded to the Ubuntu archive for xenial and trusty-backports, wily security fix is being tested and will land within the hour.
Closing this issue.
Thanks!


[Go Live][May 24]

Push to live

Docs Build status updates of commit bd42f66:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

msteams-platform/TOC.yml
:white_check_mark:Succeeded
View

msteams-platform/assets/images/teams-toolkit-v2/teams toolkit fundamentals/codelens.PNG
:white_check_mark:Succeeded
View

msteams-platform/assets/images/teams-toolkit-v2/teams toolkit fundamentals/deploy-command.PNG
:white_check_mark:Succeeded
View

msteams-platform/assets/images/teams-toolkit-v2/teams toolkit fundamentals/deploy-manifest.PNG
:white_check_mark:Succeeded
View

msteams-platform/assets/images/teams-toolkit-v2/teams toolkit fundamentals/hover.PNG
:white_check_mark:Succeeded
View

msteams-platform/assets/images/teams-toolkit-v2/teams toolkit fundamentals/manifest preview -3.png
:white_check_mark:Succeeded
View

msteams-platform/assets/images/teams-toolkit-v2/teams toolkit fundamentals/manifest preview-1.png
:white_check_mark:Succeeded
View

msteams-platform/assets/images/teams-toolkit-v2/teams toolkit fundamentals/pre.png
:white_check_mark:Succeeded
View

msteams-platform/assets/images/teams-toolkit-v2/teams toolkit fundamentals/preview-23.png
:white_check_mark:Succeeded
View

msteams-platform/assets/images/teams-toolkit-v2/teams toolkit fundamentals/select-environment.PNG
:white_check_mark:Succeeded
View

msteams-platform/assets/images/teams-toolkit-v2/teams toolkit fundamentals/update-to-teams.PNG
:white_check_mark:Succeeded
View

msteams-platform/assets/images/teams-toolkit-v2/teams toolkit fundamentals/validate.PNG
:white_check_mark:Succeeded
View

msteams-platform/mstdd-landing.yml
:white_check_mark:Succeeded
View

msteams-platform/toolkit/TeamsFx-manifest-customization.md
:white_check_mark:Succeeded

msteams-platform/toolkit/TeamsFx-manifest-preview.md
:white_check_mark:Succeeded

msteams-platform/toolkit/TeamsFx-multi-env.md
:white_check_mark:Succeeded
View

msteams-platform/toolkit/TeamsFx-preview-and-customize-app-manifest.md
:white_check_mark:Succeeded
View

msteams-platform/whats-new.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


Fix bug on missing backpropagation in WeightDropout

In WeightDropout, it applies dropout mask on target weight specified in  layer_names. Current implementation accidentally disables target weights to do back-propagation. Here is a simple code to show the bug:
import torch
import torch.nn as nn
from fastai2.text.models.awdlstm import WeightDropout

lstm = nn.LSTM(3, 5, batch_first = True)
# target weight: weight_hh_l0
# non-target weight: weight_ih_l0
lstm_dp = WeightDropout(lstm, weight_p = 0.8, layer_names='weight_hh_l0')

test_input = torch.randn(8, 20, 3)  # (batch size, seq length, input dim)
test_h = torch.randn(1, 8, 5)
test_c = test_h.data
output, (h, c) = lstm_dp(test_input, (test_h, test_c))

loss = output.sum()
loss.backward()

#show that non-target weight have gradients computed but target weight hasn't
assert lstm_dp.module.weight_ih_l0.grad is not None
assert lstm_dp.weight_hh_l0_raw.grad  is None

This PR makes two changes:

to fix the above-mentioned bug
include additional tests on nbs/32_text.models.awdlstm.ipynb to verify that gradients are computed for target weights

This is great! Thanks so much :)


Fix Omada - Crawling latest version

✍ Description
The script was always downloading the older version because the grep command was extracting all links containing linux_x64 in the HTML content. However, the extraction wasn't sorted, and the head -n1 command was picking the first match, which might not always be the latest version. Additionally, the HTML content could include multiple links with similar patterns, such as older versions or different file types (e.g., .tar.gz), causing the script to select the wrong one.
By adding a sort command, we ensured that the list of URLs was ordered, and then using tail -n 1 ensured the script always selects the most recent version, based on the sorted list.

Related Issue: #889

🛠 Type of Change
Please check the relevant options:

[x] Bug fix (non-breaking change that resolves an issue)
[ ] New feature (non-breaking change that adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change unexpectedly)
[ ] New script (a fully functional and thoroughly tested script or set of scripts)

✅ Prerequisites
The following steps must be completed for the pull request to be considered:

[x] Self-review performed (I have reviewed my code to ensure it follows established patterns and conventions.)
[x] Testing performed (I have thoroughly tested my changes and verified expected functionality.)
[x] Documentation updated (I have updated any relevant documentation)

📋 Additional Information (optional)
Provide any extra context or screenshots about the feature or fix here.

Hi this fix seems to have a typo in the grep regex.

I did some testing and found a trailing double quote.
root@pve:~# a2=$(curl -fsSL "https://www.tp-link.com/en/support/download/omada-software-controller/" | grep -oP 'href="([^"]+linux_x64[^"]+\.deb)"' | sed 's/href="//' | sort | tail -n 1)
root@pve:~# echo $a2
https://static.tp-link.com/upload/software/2024/202412/20241205/omada_v<IP_ADDRESS>_linux_x64_20241128140044.deb"
root@pve:~# a1=$(curl -fsSL "https://www.tp-link.com/en/support/download/omada-software-controller/" | grep -oP 'href="([^"]+linux_x64[^"]+\.deb)' | sed 's/href="//' | sort | tail -n 1)
root@pve:~# echo $a1
https://static.tp-link.com/upload/software/2024/202412/20241205/omada_v<IP_ADDRESS>_linux_x64_20241128140044.deb

done

Worked! Thanks!


Fix calculation of the resource part in "files" URIs not featuring trailing slash

Without the / at the end of e.g. /v1/<tenant>/files/export, the value of the resource attribute in preparation for handling the request with FileRequestHandler, becomes identical to the URI. While not critical with current code, the condition does then cascade into additional computation in e.g. any_path_islink (and potentially other places that depend on the value of the resource attribute), and there's been cases of any_path_islink raising errors which would abort the request. Again, it's a matter of interpretation whether such requests where any_path_islink aborts, should be aborted since no listing would succeed for these anyway, but in any case I thought the resource attribute should be computed "correctly". The latter in quotes because I can't be sure there's not intention for the attribute to ever have valid values like v1/<tenant>/files/export, but what this change has going for it is that resource value will be the same between URIs that only differ by presence of the terminating slash. And if resource should bear empty string value for /v1/<tenant/files/export/ URIs, then one could argue it should also hold the same value for /v1/<tenant>/files/export (no trailing slash), an assertion which will hold with this change.
This correction should also prove beneficial longer term -- with the rest of the code that may potentially depend on "sane" values of resource.
Of course all this hinges on allowing URIs like .../export without the trailing slash, but current documentation lists these explicitly, so I assume they are indeed allowed.
An alternative is adding the trailing slash with e.g. a reverse proxy ahead of the API, so the API can simply assume the trailing slash. But since the API is more of an authority on the semantics and category of the URIs that, after all, originate with it, I thought it's more fitting to be handling this in the API and not in a reverse proxy.
Should fix #256.

(Pushed the signed equivalent of the commit, the one pushed first wasn't signed because it was created from a patch I mailed myself since I am at work, aka "reasons")

Would it make sense to add a test for this case?


regression: texture load timeout

It looks like the update for elm-0.15 added a hardcoded 30ms timeout to all texture-loading HTTP requests, which makes textures unusable for all but local testing.

Looks like this is fixed by https://github.com/johnpmayer/elm-webgl/commit/874bb6777b08ec8815c8f7acb1bc89414ccdefcb


switch travis from python 3.5-dev to 3.5

@rgommers
I changed 3.5-dev to 3.5.  Should 3.5-dev or 3.6-dev still be present as well?

Not sure 3.6-dev exists, at least it's not in the docs (http://docs.travis-ci.com/user/languages/python/). 3.5-dev is definitely not needed.
We could consider adding nightly, but given how little we use of the Python C API I don't think that it's very valuable.


Deleting Main.hx as it causes conflicts with Haxe

As of the latest commit, The current Main.hx file is conflicting with a discord bot i am making, my own main file is not being executed and instead hxdiscord's main file is.
This pull request should remove the conflicting file.

That Main.hx file was added by accident, I was testing something then I ended up creating it by accident. I've already removed it so there's nothing to worry about


clean up longruns

Purpose

Clean up longruns
To-do

Content

Remove unnecessary longruns and add some new ones
Move current failing longruns to experimental

bors r+


Declared class static-named properties are rejected as reserved identifiers

The following class declaration complains, "Use of future reserved word in strict mode".
declare class U {
  static: string;
}

The logic goes way back to an initial commit: https://github.com/facebook/flow/commit/896442b0450e48cfd8a4100c3642237a0603f3d5#diff-5c9b93a1febb707483495f384257a069R687. Issue https://github.com/facebook/flow/issues/1169 seems kinda relevant.
A quick experiment in Node shows that this error is mistaken. There's a note at the tail of my spec's <IP_ADDRESS> that references the relevant sections: "In strict mode code, let and static are treated as reserved words through static semantic restrictions (see 12.1.1, <IP_ADDRESS>, <IP_ADDRESS>, and 14.5.1) rather than the lexical grammar."

Duplicate of #3129


DLC visualization tool

What's in this PR:

A new schema (dlc_report) to compute & store plotly figures for the dlc body part predictions.
A widget to display these figures

Should the scope of this PR include either of the following ... ?

pytests in the workflow
documentation on using the widget

Should the scope of this PR include either of the following ... ?

pytests in the workflow
documentation on using the widget

The tables are computed. It's a very simple schema. We can postpone the pytest to a date after SfN.
Yes, it's pretty simple. The widget consists of a dropdown menu and a plot button. User first selects a result and then clicks the plot button to see a trajectory plot and the trace of the positions side by side.

A caveat. The trajectory figure will respond slowly to zoom in/out processes if the number of points are huge. Right now, the plots don't have the sliders to visualize a portion of the data. This feature will be added later to mitigate the slow response problem.

Thanks @tdincer. Could you please add a screenshot of the widget to this pull request?

@kabilar I added a screenshot of the widget in the first comment.

Hi @tdincer - What should happen next here? Do we have the bandwidth to add those pytests now? Are there other features you want to add before we merge?


Migrating from AI SDK RSC to UI

Description
Morphic is built using AI SDK RSC. However, AI SDK RSC currently has several limitations and is not recommended for production use.
Replace the parts built with RSC and make it work stably.
Reference

Migration Guide: https://sdk.vercel.ai/docs/ai-sdk-rsc/migrating-to-ui
Discussion: https://github.com/vercel/ai/discussions/3251

@miurla any updates?

We're working on migration. It'll probably be released as a different version from RSC since it's being almost completely rebuilt.

Will it be in this repo or a different new repo?

@miurla can one help? And do you have a time estimate for the rough structure? Would like to contribute to help speeding it up

@samuelkarani Yes, it will be in this repo, not a new one.

@JimVincentW Thank you. The implementation of custom hooks needed for migration is complete. We're currently turning it into an SDK. It will start with minimal functionality.


Put region check after bundle creation

Issue #, if available:
Description of changes:
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/approve
/lgtm


NTR for concentration/count units

Hi UO team!
I was wondering if some concentration/count terms could be added as they would be useful for a wastewater specification I'm working on?
Here is the ROBOT table with the requested fields if that is useful. Let me know if you don't think these terms should go here/anything doesn't make sense.

Preferred term label
Synonyms
Textual definition
Suggested parent term

nanograms per liter
ng/L
A mass unit density which is equal to mass of an object in nanograms divided by the volume in liters.
gram per liter based unit

log10 gene copies per 100 milliliter
log10 GC/100 mL
A count unit which is the logarithmic (base 10) of the number of gene copies present in a 100 milliliter volume of a sample.
count unit

gene copies per liter
GC/L
A count unit in which the defined unit reflects the number of gene copies present in a liter volume of a sample.
count unit

log10 gene copies per nanogram total DNA

A count unit which is the logarithmic (base 10) of the number of gene copies within a nanogram of deoxyribonucleic acid genetic material.
count unit

population density unit

A density unit which is a measure of the population in a given area.
density unit

ion concentration unit

A concentration unit which represents a standard measurement of ions in a substance.
concentration unit

milliequivalent per liter
meq/L
A ion concentration unit that represents the chemical activity per volume.
ion concentration unit

weight for weight
% w/w
A dimensionless unit which is a comparison based on weight rather than volume, which is equivalent to per hundred.
dimensionless unit

gram per total solids
g/g
A dimensionless unit which is a comparison of the weight of a substance as a proportion to the weight of total solids.
dimensionless unit

colony forming units per 100 milliliter
CFU/100 mL
A colony forming unit per volume which is used to assess microbial density and describes the number of colony forming units within 100 milliliters.
colony forming unit per volume

colony forming units per grams total solids
CFU/gTS
A colony forming unit which is used to assess microbial density and that describes the number of colony forming units within a gram of total solids.
colony forming unit

most probable number per milliliter
MPN/mL
A colony forming unit per volume which describes an estimate of the most probable number of microbes within one milliliter of material.
colony forming unit per volume

most probable number per 100 milliliter
MPN/100 mL
A colony forming unit per volume which describes an estimate of the most probable number of microbes within 100 milliliters of material.
colony forming unit per volume

Thank you!

Hi all, just wondering if these are appropriate for UO or if I should request elsewhere?


Integration Test - Release - 2019/03/18 07:40:17

Auto create for integration test. IssueCommandFlow

#pri1

Just type '#backlog'

#pri2

#backlog

#assign PRMergerTest1

#assign NonAllowedUser

NonAllowedUser is not a valid GitHub ID, or is not a collaborator on this repo.

#reassign PRMergerTest2

GitHub user PRMergerTest1 has been removed from the MicrosoftDocs organization, so they were automatically removed as an assignee.

#reassign NonAllowedUser

NonAllowedUser is not a valid GitHub ID, or is not a collaborator on this repo.

#please-close


OUTREACHY SPRING 2023 STARTER ISSUE #3 - Do this third

GOAL: Install Docker on your computer to start becoming familiar.
MAIN TASK: Find something that needs clarification or a mistake in the ODK-X documentation: https://docs.odk-x.org/ and fix it.
To do this, you need to create a pull request for https://github.com/odk-x/docs with your documentation improvement. Please follow the Docker hosted build environment to test your changes on your local computer.
https://github.com/odk-x/docs#using-docker-hosted-build--environment
It is important to use the Docker method, so you gain some familiarity with Docker.

@wbrunette Hi mentor, do we need to be assigned to be recorded as the contributor? If so, can you please assign me? Thank you

@wbrunette Hi mentor, do we need to be assigned to be recorded as the contributor? If so, can you please assign me? Thank you

No @S1Y333, you don't need to be assigned. Please go ahead and work on the issue.

Share a great learning resource to get started. This tutorial covers all the Docker knowledge and the Linux command line, which is very helpful. ->

Hello @wbrunette and  @Cveman1, I will also be working on this issue.

Hello @wbrunette and @Cveman1, I will also be working on this issue.

Hey Dorcas, you don't need to be assigned, so please go ahead.

Hello, @wbrunette please review when available, thank you.
[https://github.com/odk-x/docs/pull/224](https://github.com/odk-x/docs/pull/224

@nuilewis have you encountered an error when running ./run-task.sh odkx-autobuild ? I have installed docker and built the image successfully. Any idea? Thank you!

@nuilewis have you encountered an error when running ./run-task.sh odkx-autobuild ? I have installed docker and built the image successfully. Any idea? Thank you!

Hi @S1Y333, I did encounter an error when trying to run the command, wasn't quite sure yet what it was, I'm still trying to figure it out. Will keep you posted if I succeed

@nuilewis have you encountered an error when running ./run-task.sh odkx-autobuild ? I have installed docker and built the image successfully. Any idea? Thank you!

@nuilewis Problem solved. Here's the debugging process:

when running ./run-task.sh odkx-autobuild , it shows the problem because it using the python version on my laptop, which is 3.10, so I tried to upgrade the python version on my laptop to 3.11 but the problem continue.
Then I noticed in the  dockerfile, it didn't specify which python version, which can cause problem "FROM python", so I changed it to "FROM python:3.9.16-buster" specific version ( the version later than 3.9 all don't work)
next, rebuild the docker image and then run  ./run-task.sh odkx-autobuild, it's working now. So it might need to fix the version problem later on.

@S1Y333 nice work. Create an issue in the ODK-X issue queue and then make a matching pull request to fix it :)

@nuilewis have you encountered an error when running ./run-task.sh odkx-autobuild ? I have installed docker and built the image successfully. Any idea? Thank you!

@nuilewis Problem solved. Here's the debugging process:

when running ./run-task.sh odkx-autobuild , it shows the import python problem because it using the python version on my laptop, which is 3.10, so I tried to upgrade the python version on my laptop to 3.11 but the problem continue.
Then I noticed in the  dockerfile, it didn't specify which python version, which can cause problem "FROM python", so I changed it to "FROM python:3.9.16-buster" specific version ( the version later than 3.9 all don't work)
next, rebuild the docker image and then run  ./run-task.sh odkx-autobuild, it's working now. So it might need to fix the version problem later on.

Okay, that's nice 🎉

Hi @Dee-Tyan,
As @S1Y333 explained, it might be necessary for you to modify the Dockerfile to the specified version.

@nuilewis have you encountered an error when running ./run-task.sh odkx-autobuild ? I have installed docker and built the image successfully. Any idea? Thank you!

@nuilewis Problem solved. Here's the debugging process:

when running ./run-task.sh odkx-autobuild , it shows the import python problem because it is using the python version on my laptop, which is 3.10, so I tried to upgrade the python version on my laptop to 3.11 but the problem continues.
Then I noticed in the  dockerfile, it didn't specify which python version, which can cause problem "FROM python", so I changed it to "FROM python:3.9.16-buster" specific version ( the version later than 3.9 all didn't work)
next, rebuild the docker image and then run  ./run-task.sh odkx-autobuild, it's working now. So it might need to fix the version problem later on.

Thanks @S1Y333, helped me out a lot!

Hi @wbrunette Please help me review this pull request at https://github.com/odk-x/docs/pull/228

Hii @wbrunette, here's my submission https://github.com/odk-x/docs/pull/230 . Thank you!


weave reset while using proxy leaves a stopped container behind

vagrant@host1:~$ weave launch
vagrant@host1:~$ eval $(weave env)
vagrant@host1:~$ weave reset
WARNING: It appears that your environment is configured to use the Weave Docker API proxy. Stopping it will break this and subsequent docker invocations. To restore your environment, run 'eval $(weave env --restore)'.
Error deleting container: Cannot connect to the Docker daemon. Is the docker daemon running on this host?
vagrant@host1:~$ eval $(weave env --restore)
vagrant@host1:~$ weave reset
vagrant@host1:~$ docker ps -a
CONTAINER ID        IMAGE                         COMMAND                  CREATED             STATUS                      PORTS               NAMES
157c72d557f1        weaveworks/weaveexec:latest   "/home/weave/sigproxy"   36 seconds ago      Exited (1) 35 seconds ago                       berserk_shockley
vagrant@host1:~$

The left-over container is the one that executed the weave script as weave --local reset. It is run with --rm but the Docker client cannot remove it once the proxy has stopped.
Possible ways out:

don't kill the proxy if DOCKER_HOST is pointing at the proxy
do docker rm -f from the outer-level script.


scipy.spatial.distance.pdist coerces to double

scipy.spatial.distance.pdist coerces to double here. The issue is that I'd like to use pdist with a method written in Cython which is typed to only accept int64 data. Is this coercion necessary? I'd be happy to issue a PR, but before doing so, I wanted to check first if it would be a waste of time.

there is only code for double and bool, though bool is unused due to the coercion (see https://github.com/scipy/scipy/issues/5027). There is probably no reason there aren't more types besides nobody wrote the code.
Though why do you need int64 specifically? I don't think the conversion from double back to int should be very expensive, do you need the full 64 bit range?

Good point -- I retooled it to allow an untyped numpy array and there really wasn't an appreciable difference in runtime. So, perhaps this is moot. The motivation for int64 here is that the data are counts, so the natural type is unsigned int.


Please add new model 358K

I have two Dyson Purifier+Cool Formaldehyde. One with device type 358E and one with device type 358K.
I could connect the device type 358E in Home Assistant, but for the device type 358K I'm getting the error "Failed to connect".

Tacking on to what @mtjweb is asking for as well - there are a number of new models that are quite similar but lacking the code correlation in the library.  It's a matter of mainly updating the code const to include them.  Please do check it out and add.

This has been added in https://github.com/libdyson-wg/ha-dyson, an actively maintained fork - more information in this thread if you're curious: ha-dyson #185.


Feature : Hazel Script File

Hi There, I Have Been Following Hazel From the Start, But Missing the Most important Feature Scripting. Scripting is Essential For Any Language, So I Came up With this Idea Hazel Script File (.hzs).
Syntax
The Syntax is Basically Like the HTML,XML Stuff to Keep it Simple.
like

<Cube "Mention properties here or inside it">
height = 10px

Somewhat Like This
 Control
It Should Basically Have Full Control Over the Engine, From Drawing Cube to Adding Textures.

Maybe implementing a functional  language like Javascript that is the most popular   language of 2020 according to stack overflow can be better tan a static approach lets say your example could be something like this
`
const hazel = require('hazel-core')
hazel.createCube(width:50, heigth:50);
`

Well a syntax like HTML would honestly be more annoying to write and modify than something like C# or C++. HTML is used for just writing content on a webpage, meaning it would be better off as a way to make objects on the screen and easily modify their looks than something like an actual scripting language, however you may as-well use CSS at that point.

At least that can be the starting point for this or some thing else like javascript or C# syntax somewhat like an modern programming language ?

At least that can be the starting point for this or some thing else like javascript or C# syntax somewhat like an modern programming language ?

Meh. We should use an actual language than some markup language like HTML. How will you make a reference to a component in an HTML-like language? How can you make a variable? I think what you are looking for is a CSS sort-of language for stylizing the objects in a scene, which we could just as easily do in a normal language.

At least that can be the starting point for this or some thing else like javascript or C# syntax somewhat like an modern programming language ?

Meh. We should use an actual language than some markup language like HTML. How will you make a reference to a component in an HTML-like language? How can you make a variable? I think what you are looking for is a CSS sort-of language for stylizing the objects in a scene, which we could just as easily do in a normal language.

But thats not an Easy Task 🤯 but that would be better :)

Javascript Syntax is a lot better i guess ? 🤔

Javascript Syntax is a lot better i guess ? 🤔

That completely depends on who you ask lol since people'd disagree and their is no "Best" language. Whatever language is requested the most should be added so most people can easily write scripts.

Javascript Syntax is a lot better i guess ? 🤔

That completely depends on who you ask lol since people'd disagree and their is no "Best" language. Whatever language is requested the most should be added so most people can easily write scripts.

Well Then Cherno Should have some answers for this @TheCherno

I saw one of his videos were he states that c# is going to be the scripting language i guess that's the answer

I saw one of his videos were he states that c# is going to be the scripting language i guess that's the answer

Ya But that is still unimplemented

I saw one of his videos were he states that c# is going to be the scripting language i guess that's the answer

Nice. Then I think this issue should be closed.

I thin Hazel-dev has already implemented a C# scripting system using the Mono runtime. I guess Hazel will implement that stuff.

Yes, Hazel-dev uses C#, which is also what Hazel will use. You can always use the NativeScriptComponent as well, but no other ones will be supported for now. I'll close this as a question being answered.


MultiplyWaitStrategy - incorrect name

MultiplyWaitStrategy should be named ExponentialWaitStrategy or something similar.
See https://en.wikipedia.org/wiki/Exponential_backoff

We might change it later but is used in other solutions as well. If you want to use a name that complies to your naming conventions you can fork the repo and change its name

your naming conventions

The whole world's naming convention. This naming is plainly a bug. Nobody calls it that way.

you can fork the repo and change its name

I don't care, you can close the issue. I just looked at the code and reported a bug.

Thanks for for feedback, we’ll consider it for future development

@arieleeuw You suggestion to fork the code is interesting and completely unproductive. I agree that there naming and convection issues, of confusing or things that are understood flat out wrong.  MultiplyWaitStrategy and ExponentialWaitStrategy don't mean the same things.  In fact, MultiplyWaitStrategy means nothing to me.

@maykays, it would be nice to have a reason for closing this issue.

@ilyash - this issue was closed based on your comment above "I don't care, you can close the issue." However, it seems you still feel strongly about this issue. I'm re-opening the issue and we'll take another look.

I'll summarize my position.

This is a bug
Denying that this is a bug - not OK
Saying that I suggest "name that complies to your naming conventions" - not OK
Suggesting fork - not OK
Closing the issue without a comment - meh
Closing the issue saying "It's a bug but we won't fix it because ..." - OK (it's your project)
Fixing the issue - great

My company is saving $$$ on our EC2 instances using this great solution (thanks AWS).
Don't get the point of arguing, or even bother, about some internal function names that do not affect the functionality of the program at all (so why is this a bug?). Functionality is what matters to us.
AWS please keep coming with more solutions like this.
Alex
WSG Prague


Confused about the performance of Pose2mesh on Human3.6M

The performance of Pose2mesh on Human3.6M:
Training with Human3.6M:
MPJPE:64.9
PA-MPJPE:48.0
Training with Human3.6M and COCO:
MPJPE:67.9
PA-MPJPE:49.9
Best result:
MPJPE:64.9
PA-MPJPE:46.3
As mentioned in the paper，using more datasets to train Pose2mesh will decrease the performance on Human3.6M. I wonder the best result on Human3.6M is supposed to be trained with Human3.6M dataset only? In this case, the best result should be the same with the one trained with Human3.6M. Or it should be trained with Human3.6M+COCO+MuCo?  Would you please show me the training settings on Human3.6M？

the best result should be the same with the one trained with Human3.6M
I think so, as train and test sets of Human3.6M have the same action category and thus similar poses. The discussion is in the paper.

The training settings are at asset/yaml/

the best result should be the same with the one trained with Human3.6M

I think so, as train and test sets of Human3.6M have the same action category and thus similar poses. The discussion is in the paper.

Thanks for your quick reply. And as the best result should trained with Human3.6M. I think Table 5 and Table 8 should report the same PA-MPJPE？Is this a miswriting?

No. Actually the explanation is also in the paper.
When computing PA-MPJPE of Table 5, I used all camera images (4 in Human3.6M), which I think natural.
In Table 8, for PA-MPJPE, I used only the frontal camera image for fair comparison with previous works.

We measured the PA-MPJPE of Pose2Mesh on Human3.6M by
testing only on the frontal camera set, following the previous works [23, 27, 28].

No. Actually the explanation is also in the paper.
When computing PA-MPJPE of Table 5, I used all camera images (4 in Human3.6M), which I think natural.
In Table 8, for PA-MPJPE, I used only the frontal camera image for fair comparison with previous works.

We measured the PA-MPJPE of Pose2Mesh on Human3.6M by
testing only on the frontal camera set, following the previous works [23, 27, 28].
That solved my confusion. So in Table 8, the training set of Human3.6M is still [1,5,6,7,8] and the testing set is [9, 11] but only use the frontal camera set(camera 3)? Would you mind sharing the code to get this test set? Or how should I modify the code in Pose2mesh?

So in Table 8, the training set of Human3.6M is still [1,5,6,7,8] and the testing set is [9, 11] but only use the frontal camera set(camera 3)?

Yes.

Would you mind sharing the code to get this test set? Or how should I modify the code in Pose2mesh?

You just need to skip the cam annotation '4', when loading test set.
Like this
if cam != '4':  # front camera (Table 6)
    continue

So in Table 8, the training set of Human3.6M is still [1,5,6,7,8] and the testing set is [9, 11] but only use the frontal camera set(camera 3)?

Yes.

Would you mind sharing the code to get this test set? Or how should I modify the code in Pose2mesh?

You just need to skip the cam annotation '4', when loading test set. Like this
if cam != '4':  # front camera (Table 6)
    continue

Thanks, I have got the paper result followed your suggestion.


ng new hello Error: Path "/app/app.module.ts" does not exist. Path "/app/app.module.ts" does not exist.

Versions
npm -v
5.5.1

I am facing problem in creating new project in angular.
When I run ng new myapp command I get following command

ng new hello
Error: Path "/app/app.module.ts" does not exist.
Path "/app/app.module.ts" does not exist.

I am also facing the same problem can anyone help me out???

After searching for hours.
I did not find the solution anywhere.
But I reinstalled my node.js and npm, the problem was solved.

Duplicate of #8765

I reinstalled Node and Angular, did it several times. cleaning cache and so on.
but still get this error
Can anyone please help?
When I create a new project "ng new project-name" , I get this error:
Error: Path "/app/app.module.ts" does not exist.
Path "/app/app.module.ts" does not exist.

+1

Same issue on Windows 10. Tried to uninstall globally, clear cache and reinstall with no luck.

Same issues here...Any workaround guys?
I am new to angularjs and just get angularcli installed and run
"ng new my-app" which from official getting started doc. And i got this issue.

@bradkt @carlchandev
please, see my last answer above

I'm also faced same issue ... i find the solution : 1. change the directory of your working and execute the code ng new "your app name" 2. other uninstall all the app(node,angular) and again reinstall................./

Hi Friends i found the answer
for this error
Error: Path "/app/app.module.ts" does not exist.
Path "/app/app.module.ts" does not exist.
while creating the angular project
Solution : Follow the steps :
Please go to the directory where you have installed the node then see in your project directory where you are creating the projects there you see node_modules folder simply delete that because its overwriting the your node modules directory
Cheers
https://github.com/adnanaslamit/

@adnanaslamit thanks brother it really worked for me but I delete node_modules outside of my projects directory


Gazebo model of the panda robot not responding to velocity inputs

When using a simple JointGroupVelocityController with gazebo, the robot does not respond to command but start slowly drifting from original position even when commands is set to zero. After checking the controller, commands are received and set in the "update" function but ignored by the model. However, things behave as expected with real hardware so not sure if the problem could come from the ros_control_gazebo plugin or the franka_hw_sim. Note that JointGroupPositionController works well.

Great ! Thank you. Closing this issue.


Suggested variable usage for DRY code

Suggested variable usage for DRY code and changed to plurals which I found can be changed

Added minor changes to variables and comments

I appreciate that you're trying to help, but your changes are not substantial enough to constitute a pr.