examples: Fix linalg operator construction

Codecov Report

Merging #784 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #784   +/-   ##
=======================================
  Coverage   88.78%   88.78%
=======================================
  Files         104      104
  Lines       10990    10990
  Branches     2192     2192
=======================================
  Hits         9758     9758
  Misses        948      948
  Partials      284      284

Continue to review full report at Codecov.

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

Merged.


Bad use of null-like value: logCreator

If createLogger is called with { logCreator: undefined }, the default value of null will be assigned to logCreator.  The value of logCreator is not checked prior to calling logCreator(logLevel) resulting in   an exception such as a TypeError.
https://github.com/tulios/kafkajs/blob/3c3a91c94f3b32b9c5e35ff6af671244eee45487/src/loggers/index.js#L38
This is probably a low priority issue, but it's detected by Coverity scanning as a CWE-476.

Do you have a suggestion for better handling of this? Although it's not intentional, raising a TypeError seems like a pretty reasonable thing to do when being passed an invalid type.

I think the default assignment of null is a bit unusual, given that you wouldn't expect a "default" case to lead to a TypeError.  I can think of 3 options:

Removing the null default.  It will still throw a TypeError if undefined, but at least it's not implied that null is ok,
Defaulting the logFunction if logCreator is null/undefined (instead of calling logCreator and triggering a TypeError)
Defaulting the logCreator to something more like a noop function, that returns a noop function to logFunction.

But with those last 2 options now you'd be changing the logic to indicate that undefined is a valid type for logCreator.  If logCreator is required, then I think option 1 (no default) is the best option.


"Bigger Map"

Player count reduced  to 8 to give more space in the map

In the future the idea is to 6
Spawnpoints re-adjusted
For the time being, spawnpoints 9 and 10 are located in the center.

Power-ups

Nerfed effect, since there will be more of them.
Doubled their amount in the map
Reduced crate HP by 20%

Reduced player speed by 10% (through configurator)

Based on what I talked to @manucamejo I've lowered the players amount to 7.
Also you'll had to change the initial positions in the configurator localhost:4100/map_configurations


Backward compatibility for Object.entries in safari browser

I found that Object.entries was not defined in safari. It worked fine in Chrome and Firefox.
Found https://github.com/es-shims/Object.entries and incorporated that. Then it worked.
Attaching the diff, please check. If you suggest a different way (I even tried babel-polyfill) please let me know. Else, I can provide a PR.
// In methods.js
// npm install --save object.entries
import { entries } from 'object.entries'

if (!Object.entries) {
  entries.shim();
}

So react table doesn't work in safari for you?
On Fri, Feb 24, 2017 at 11:17 AM Vaithiyanathan Sundaram <
<EMAIL_ADDRESS>wrote:

I found that Object.entries was not defined in safari. It worked fine in
Chrome and Firefox.
Found https://github.com/es-shims/Object.entries and incorporated that.
Then it worked.
Attaching the diff, please check. If you suggest a different way (I even
tried babel-polyfill) please let me know. Else, I can provide a PR.
// In methods.js// npm install --save object.entriesimport { entries } from 'object.entries'
if (!Object.entries) {
entries.shim();
}
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/tannerlinsley/react-table/issues/96, or mute the
thread
https://github.com/notifications/unsubscribe-auth/AFUmCXmdHhiHeC5b0XRIg1gdH3BC1mMTks5rfx6ygaJpZM4MLhBy
.

Yes I am getting Object.entries is undefined error. I use Version 10.0.3 (116<IP_ADDRESS>.1). I have a private copy that works with the above diff. If you wish to see the console debugger I can send.
Thanks again.

Interesting.  It works in my safari, v10.0.2.  Are you on a mac or windows?
On Sat, Feb 25, 2017 at 1:59 AM Vaithiyanathan Sundaram <
<EMAIL_ADDRESS>wrote:

Yes I am getting Object.entries is undefined error. I use Version 10.0.3
(116<IP_ADDRESS>.1). I have a private copy that works with the above diff. If
you wish to see the console debugger I can send.
Thanks again.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/tannerlinsley/react-table/issues/96#issuecomment-282471046,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AFUmCbx_bklCpHofEQtLduy2_fsILIbSks5rf-1XgaJpZM4MLhBy
.

I tried to clean the cache, and cleared my modules, then tried again.
Like you said, it started working. I still don't understand why I got the Object.entries issue or if babel-polyfill is somehow helping.
In any case I would like to close this for now until I see that issue again.
Thanks for your help and wrong alarm.

No worries! 😊


Allow any impl Spanned for quote_spanned!

In almost all cases I find myself using the span of an existing AST node, so needing to write .span()=> everywhere is a lot of visual noise. And since Span is impl Spanned, I believe this would be backward compatible (though there could very well be some sort of special case I have not thought of). Would this be a reasonable change, or is there some reason it is not so?

I think this is all right as currently implemented, with .span()=> required if the desired span is the span of an AST node.
In my own macros I prefer avoiding Spanned, so adding a new implicit application of it would be counterproductive.


report enum and get_reportid

example of new functionality
report_api = mws.apis.Reports(**mwscred)
request_response = report_api.request_report(AllReports.ACTIVE_LISTINGS.report)
report_id = report_api.get_reportid(request_response)
if report_id:
    report = report_api.get_report(report_id)
    if AllReports.ACTIVE_LISTINGS.isutf8:
        result = report.original.decode('utf8')
    else:
        result = report.original.decode('latin-1')

Codecov Report

Merging #74 into develop will increase coverage by 0.48%.
The diff coverage is 80.48%.

@@             Coverage Diff             @@
##           develop      #74      +/-   ##
===========================================
+ Coverage    74.77%   75.25%   +0.48%
===========================================
  Files           18       18
  Lines          880      962      +82
  Branches        93       97       +4
===========================================
+ Hits           658      724      +66
- Misses         215      231      +16
  Partials         7        7

Impacted Files
Coverage Δ

mws/apis/reports.py
88.57% <80.48%> (-11.43%)
:arrow_down:

Continue to review full report at Codecov.

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

beforehand i used a wrapper to get new reports.
This would replace the wrapper or at least it will be a simple function.
The report enum is not tested and not complete. I think it's a good place to put some useful, easy accessible information about reports.
I can try to test the encoding for more reports.
The date part of the tuple we might drop.

to-do: remove tuples.
Encoding gets now handled by #75

How long is a good wait time for the report to finish?
Probably look every 30 seconds, up to 4 times?

FYI, some post-merge changes committed on setup.py to streamline Py version installs.
Future changes: we'll test out the ReportType enumerations and try to tie them into request methods more closely. Also need documentation to point them out and how to make use of them.


Add additional example for keyword lists

I've noticed on several codebases that there's always some confusion when trying to execute nested preloading. In my opinion it is because the order of examples that are given on
# preloading a single association, nothing new
posts = Repo.preload posts, :comments

# preloading several associations looks quite straight forward
posts = Repo.preload posts, [:comments, :authors]

# But here is where I've found confusion among developers.
 posts = Repo.preload posts, [comments: [:replies, :likes], authors: []]

On the last example, the :comments nested preloads are just an extension of the previous example. Yet the :authors field does not require any nested preloading at all. Which in fact makes sense, because Ecto will not have to preload anything where there is an empty list, which is quite convenient for dynamic preloading. However when a given entity is considerably large, this example can give the impression that all of the fields on the same level of preloading where nested preloading took place, have the need to require an empty list. Such as:
# This is what I usually encounter
 posts = Repo.preload posts, [comments: [replies: [:reactions], likes: []], authors: []]

As you can see the :replies field has now a nested preload, which based on the example I've been talking before gives that impression that :likes should have the empty list because another field from the same level of preloading actually had some nested preloading taking place. We could obviously have a smaller list, such as:
# This is what I would prefer
 posts = Repo.preload posts, [:authors, comments: [:likes, replies: :reactions]]

Just a minor thing I've encountered way too many times on different codebases with different developers that I will like to know what's the actual reason to use way to many empty lists on the preloads list. Also to clarify the confusion when using nested preloads.

:green_heart: :blue_heart: :purple_heart: :yellow_heart: :heart:


YOLOX-X performs on average 0.05 - 0.10 mAP worse than YOLOV5-X

Hi,
I use the default training settings to train a YOLOX-X, but since I am only able to train with 1 GPU I set the batch size to 8.
By these default training settings after training, my best YOLOX-X model performs on average 0.05 - 0.10 mAP worse than a trained YOLOV5-X model, but one difference is that for yolov5 they provide some fine-tune (training) hyperparameters.
I have two questions:

Should change the (training) hyperparameters when using only 1 GPU? And how to scale the parameters? (e.g. divide by 8?)
What can I do to outperform YOLOV5-X (that is trained with standard fine-tune parameters?) on the same dataset?

Some hyperparams of YOLOX-X is not tuned well in our released version. Any advice ? @Joker316701882 @GOATmessi7

Some hyperparams of YOLOX-X is not tuned well in our released version. Any advice ? @Joker316701882 @GOATmessi7

I got a YOLOX-S trained by using standard settings, of which the performance is also worse than YOLO-S. What do you recommend to change to get better performance?

Hi
Waiting for more answer!
Best regards,
PeterPham

I met the same problem. Maybe because the hyper params or data augmentations?


tails-screen-locker just sort of appears on the screen without any explanation during descriptor importing

This step can be either really fast or slow depending on the blockchain progress and how busy the node is.
I should say more obviously, this will be slow, lock your screen if you have to step away from the computer. BEFORE showing the dialog for tails-screen-locker. Otherwise it just sits there and the notification with explanation expires.

Lets make a file called "remind-lock-screen" that checks if the user has a screen lock passphrase set and notifies them every 3 +/- minutes if not, else clears its notifications and exits. This is for chainstate-preload.
Press the lock screen keys to lock your computer if you need to walk away from it.

This also defines the scope of bails-wallet for seed recovery, it should build the descriptor and stop there. (This means in L2, we can also let it display a QR of that descriptor.) That way Bails can offer some entertainment during long steps like descriptor importing, as well as me not need to worry about teaching bails-wallet how to detect and import descriptors.

 But we do try to set the screen lock password during each boot. If this needs to be reopened we will replace it with a dialog since notifications create spam that reduces privacy on the lock screen.


New PR to test SignCLA at 2020-05-18 11:09:49.645507459 -0700 PDT m=+2.922547764

Body of PR

unsigned-fail-fork-staging: test succeeded


More documentation on "Custom Highlighting"

Before recent updates, it was really easy to change the colors of  "special groups"
vim.cmd([[highlight LspSagaWinbarFileName gui=NONE, guifg=#7c6f64]])
vim.cmd([[highlight LspSagaWinbarFileIcon gui=NONE, guifg=#7c6f64]])
vim.cmd([[highlight LspSagaWinbarSep gui=NONE, guifg=#7c6f64]])

After updates, its unclear how to apply these custom highlights using the api.
It would be wiser to add an example.

by using api you can do it in a for loop .

local res = require('lspsaga.lspkind').get_kind_group()
for i , group in pairs(res) do
  if group:find("FileName') then
     api.nvim_hi_set_hl(0,group , { fg = xxx, bg=xxx})
  end
  if i == #res then
      group is winbarsep
      api.nvim_hi_set_hl(0,group , { fg = xxx, bg=xxx})
  end
end

local res = require('lspsaga.lspkind').get_kind_group()
for i , group in pairs(res) do
  if group:find("FileName') then
     api.nvim_hi_set_hl(0,group , { fg = xxx, bg=xxx})
  end
  if i == #res then
      group is winbarsep
      api.nvim_hi_set_hl(0,group , { fg = xxx, bg=xxx})
  end
end

local res = require('lspsaga.lspkind').get_kind_group()
for i , group in pairs(res) do
  if group:find("FileName") then
     vim.api.nvim_hi_set_hl(0,group , { fg = "#3c3c3c", bg="#3c3c3c"})
  end
  if i == #res then
      group is winbarsep
      vim.api.nvim_hi_set_hl(0,group , { fg = "#3c3c3c", bg="#3c3c3c"})
  end
end

i fixed the " and ' and also the vim.api* but it still throws on error on line-3
group is winbarsep {Unexpected }

even then, neovim will throw an error, ```lua:1: attempt to index field 'config' (a nil value``

run this after lspasga.setup . if i == #res is wibarsep no need do find

run this after lspasga.setup . if i == #res is wibarsep no need do find

nope :/

does it locally work for you?

you have typo it's nvim_set_hl

local res = pcall(require, 'lspsaga.lspkind').get_kind_group()
for i, group in pairs(res) do
  if group:find("FileName") then
    vim.api.nvim_set_hl(0, group, { fg = "#3c3c3c", bg = "#3c3c3c" })
  end
  -- group is winbarsep
  if group:find("winbarSep") then
    vim.api.nvim_set_hl(0, group, { fg = "#3c3c3c", bg = "#3c3c3c" })
  end
end

no typos

can you reproduce E5113 on your side?

local res = pcall(require, 'lspsaga.lspkind').get_kind_group()

I don't think this line is correct ? you do pcall here ? it return two value .

local res = require('lspsaga.lspkind').get_kind_group()

yes that was the issue, no more errors but colors are not applying

maybe using vim.api like in the docs, isn't a good solution

it works fine in min config. i already test it . and new group is SagaWinbarxxxx you can also do it like before what you do .

read #867

this is api for some vimer want custom some groups or some colorscheme authors.

Awesome! Thanks man..


Azure - New ResourceGroup model

This PR will add a new model for the Azure Resource Group.
A migration for this new model was already merged here: #5026
A separate PR will be made to add a resource_group_id to the Vm model.

@bronaghs Is a migration needed to create resource_groups table? Or was that done in a previous PR?

@chessbyte the migration was merged in #5026.
@bronaghs from the migration, why the table has ems_ref and ems_id? I think ems_ref is sufficient. Also why there is no column for location which is included in the return from the API?

ems_ref != ems_id .  The former is a key from Azure, the latter is the reference to record in the ext_management_systems table.

@bzwei  - Azure locations will be called Regions in MIQ. Regions in Azure will be mirror Regions in AWS where a Region is its own EMS. Therefore the association between the ResourceGroup and the Region will be represented by the existing ems_id column.


APT whitelist request for gfortran in trusty

The version-specific package gfortran-4.8 is already in the whitelist, but gfortran itself is not

The source package is gcc-defaults according to the ubuntu website:
http://packages.ubuntu.com/trusty/devel/gfortran

Building with gfortran now works, closing.


FuncUiAnalyzer server is not Active.

Hi, followed the steps here  SilkyFowl.Avalonia.FuncUI.LiveView.Demo.Paket
but when i launch the preview never gets rendered and on the bottom right " FuncUiAnalyzer server is not Active" is displayed. I tried changing the code,  restarting the f# extension, and restarting vs code and  unstalling docker and restarting my machine , but no luck
Any hints on what i could do?
Thanks in advance!

I tried to use https://github.com/SilkyFowl/SilkyFowl.Avalonia.FuncUI.LiveView.Demo but it did not work either.
Perhaps, we need a specific version of Inonide to have the same Analyzer SDK version used.
I also tried to use, the CLI tool documented their https://ionide.io/ionide-analyzers/ and using version 0.26.0 allows to run it but still don't start the server. I suspect this is because FuncUiAnalyzer only works when used via Ionide? Perhaps, it could be nice to check if it is possible to make it work with fsharp-analyzers CLI tool as it would be easier to keep the SDK version in check?


feat: adds getter function to provider class for retrieving total executed batches

Still unsure if this is needed, as its being tested to resolve https://github.com/matter-labs/dapp-portal/issues/193 based on https://github.com/matter-labs/dapp-portal/pull/194

What :computer:

First thing updated with this PR
Second thing updated with this PR
Third thing updated with this PR

Why :hand:

Reason why first thing was added to PR
Reason why second thing was added to PR
Reason why third thing was added to PR

Evidence :camera:
Include screenshots, screen recordings, or console output here demonstrating that your changes work as intended

Notes :memo:

Any notes/thoughts that the reviewers should know prior to reviewing the code?

Hey Dustin, we can keep it like this and if you can just give us the heads up before it can be merged so that we can prepare other SDKs

Thanks @petarTxFusion ! Still unsure if this change is useful or not. I have the ethers v6 version here fyi.

Then we should wait and discuss it since it could be troubling if we endup needing to remove it and result in breaking change.


RuntimeError: Dimension out of range (expected to be in range of [-2, 1], but got 2) with CornerNet_Saccade

I'm running
from train_detector import Detector
gtf = Detector();
root_dir = "/home/SK00495085/monk/Monk_Object_Detection/data";
coco_dir = "training_menu"
img_dir = "/"
set_dir = "Images"
gtf.Train_Dataset(root_dir, coco_dir, img_dir, set_dir, batch_size=4, num_workers=4)
root_dir = "/home/SK00495085/monk/Monk_Object_Detection/data";
coco_dir = "validation_menu"
img_dir = "/"
set_dir = "Images"
gtf.Val_Dataset(root_dir, coco_dir, img_dir, set_dir)
gtf.Model(model_name="CornerNet_Saccade")
gtf.Hyper_Params(lr=0.00025, total_iterations=6900000, val_interval=10000)
gtf.Setup();
gtf.Train();
I got this error:
loading annotations into memory...
Done (t=0.59s)
creating index...
index created!
loading annotations into memory...
Done (t=0.22s)
creating index...
index created!
Loading Model - core.models.CornerNet_Saccade
Model Loaded
start_iter       = 0
distributed      = False
world_size       = 0
initialize       = False
batch_size       = 1
learning_rate    = 0.00025
max_iteration    = 6900000
stepsize         = 5520000
snapshot         = 3450000
val_iter         = 10000
display          = 100
decay_rate       = 10
Process 0: building model...
total parameters: 116967797
start prefetching data...
shuffling indices...
setting learning rate to: 0.00025
training start...
start prefetching data...
shuffling indices...
0%|                                               | 0/6900000 [00:00<?, ?it/s]
Traceback (most recent call last):
File "training_saccade.py", line 31, in
gtf.Train();
File "/home/SK00495085/monk/Monk_Object_Detection/6_cornernet_lite/lib/train_detector.py", line 298, in Train
training_loss = self.system_dict["local"]["nnet"].train(**training)
File "/home/SK00495085/monk/Monk_Object_Detection/6_cornernet_lite/lib/core/nnet/py_factory.py", line 93, in train
loss = self.network(xs, ys)
File "/home/SK00495085/.conda/envs/monk_env/lib/python3.6/site-packages/torch/nn/modules/module.py", line 489, in call
result = self.forward(*input, **kwargs)
File "/home/SK00495085/monk/Monk_Object_Detection/6_cornernet_lite/lib/core/models/py_utils/data_parallel.py", line 68, in forward
return self.module(*inputs[0], **kwargs[0])
File "/home/SK00495085/.conda/envs/monk_env/lib/python3.6/site-packages/torch/nn/modules/module.py", line 489, in call
result = self.forward(*input, **kwargs)
File "/home/SK00495085/monk/Monk_Object_Detection/6_cornernet_lite/lib/core/nnet/py_factory.py", line 20, in forward
loss  = self.loss(preds, ys, **kwargs)
File "/home/SK00495085/.conda/envs/monk_env/lib/python3.6/site-packages/torch/nn/modules/module.py", line 489, in call
result = self.forward(*input, **kwargs)
File "/home/SK00495085/monk/Monk_Object_Detection/6_cornernet_lite/lib/core/models/py_utils/losses.py", line 150, in forward
pull, push = self.ae_loss(tl_tag, br_tag, gt_mask)
File "/home/SK00495085/monk/Monk_Object_Detection/6_cornernet_lite/lib/core/models/py_utils/losses.py", line 26, in _ae_loss
dist = tag_mean.unsqueeze(1) - tag_mean.unsqueeze(2)
RuntimeError: Dimension out of range (expected to be in range of [-2, 1], but got 2)

Are you training on multiple GPUs?

yes

try to increase the batch size to 8. if the error still persists please let us know.

I changed the batch size to 8. its working fine.
The reason why I changed to 1 is because of the below error.
"RuntimeError: CUDA out of memory. Tried to allocate 399.88 MiB"
Now it's running.Thanks you

It ran for some time, But now it's throwing this below error
batch_size       = 8
learning_rate    = 0.00025
max_iteration    = 6900000
stepsize         = 5520000
snapshot         = 3450000
val_iter         = 10000
display          = 100
decay_rate       = 10
Process 0: building model...
Traceback (most recent call last):
File "training_saccade.py", line 31, in
gtf.Train();
File "/home/SK00495085/monk/Monk_Object_Detection/6_cornernet_lite/lib/train_detector.py", line 232, in Train
self.system_dict["local"]["model"], distributed=distributed, gpu=gpu)
File "/home/SK00495085/monk/Monk_Object_Detection/6_cornernet_lite/lib/core/nnet/py_factory.py", line 51, in init
self.network = DataParallel(self.network, chunk_sizes=system_config.chunk_sizes)
File "/home/SK00495085/monk/Monk_Object_Detection/6_cornernet_lite/lib/core/models/py_utils/data_parallel.py", line 61, in init
self.module.cuda(device_ids[0])
File "/home/SK00495085/.conda/envs/monk_env/lib/python3.6/site-packages/torch/nn/modules/module.py", line 260, in cuda
return self._apply(lambda t: t.cuda(device))
File "/home/SK00495085/.conda/envs/monk_env/lib/python3.6/site-packages/torch/nn/modules/module.py", line 187, in _apply
module._apply(fn)
File "/home/SK00495085/.conda/envs/monk_env/lib/python3.6/site-packages/torch/nn/modules/module.py", line 187, in _apply
module._apply(fn)
File "/home/SK00495085/.conda/envs/monk_env/lib/python3.6/site-packages/torch/nn/modules/module.py", line 187, in _apply
module._apply(fn)
[Previous line repeated 7 more times]
File "/home/SK00495085/.conda/envs/monk_env/lib/python3.6/site-packages/torch/nn/modules/module.py", line 193, in _apply
param.data = fn(param.data)
File "/home/SK00495085/.conda/envs/monk_env/lib/python3.6/site-packages/torch/nn/modules/module.py", line 260, in
return self._apply(lambda t: t.cuda(device))
RuntimeError: CUDA out of memory. Tried to allocate 5.12 MiB (GPU 0; 7.93 GiB total capacity; 146.31 MiB already allocated; 5.56 MiB free; 839.00 KiB cached)

It ran for the first time, but the same code running for the second time throws this Runtime error.

The issues are with Cuda. I got it.
Thanks.

once again runtime error
11,  1.55s/it]^M  0%|                            | 9976/6900000 [20:38:55<2966:40:26,  1.55s/it]^M  0%|                            | 9977/6900000 [20:38:56<2967:57:09,  1.55s/it]^M  0%|                            | 9978/6900000 [20:38:58<2955:07:32,  1.54s/it]^M  0%|                            | 9979/6900000 [20:38:59<2944:00:52,  1.54s/it]^M  0%|                            | 9980/6900000 [20:39:01<2937:11:48,  1.53s/it]^M  0%|                            | 9981/6900000 [20:39:02<2937:09:15,  1.53s/it]^M  0%|                            | 9982/6900000 [20:39:04<2951:29:57,  1.54s/it]^M  0%|                            | 9983/6900000 [20:39:05<2943:31:21,  1.54s/it]^M  0%|                            | 9984/6900000 [20:39:07<2937:01:24,  1.53s/it]^M  0%|                            | 9985/6900000 [20:39:08<2935:26:05,  1.53s/it]^M  0%|                            | 9986/6900000 [20:39:10<2931:41:56,  1.53s/it]^M  0%|                            | 9987/6900000 [20:39:12<2928:14:04,  1.53s/it]^M  0%|                            | 9988/6900000 [20:39:13<2951:58:24,  1.54s/it]^M  0%|                            | 9989/6900000 [20:39:15<2983:16:16,  1.56s/it]^M  0%|                            | 9990/6900000 [20:39:16<3001:54:02,  1.57s/it]^M  0%|                            | 9991/6900000 [20:39:18<2993:27:21,  1.56s/it]^M  0%|                            | 9992/6900000 [20:39:19<2987:51:46,  1.56s/it]^M  0%|                            | 9993/6900000 [20:39:21<2970:26:55,  1.55s/it]^M  0%|                            | 9994/6900000 [20:39:22<2996:07:47,  1.57s/it]^M  0%|                            | 9995/6900000 [20:39:24<2977:08:39,  1.56s/it]^M  0%|                            | 9996/6900000 [20:39:26<2961:41:57,  1.55s/it]^M  0%|                            | 9997/6900000 [20:39:27<2948:27:08,  1.54s/it]^M  0%|                            | 9998/6900000 [20:39:29<2940:58:51,  1.54s/it]^M
^M  0%|                            | 9999/6900000 [20:39:32<2965:10:45,  1.55s/it]^M  0%|                           | 9999/6900000 [20:39:33<14235:41:00,  7.44s/it]
^MProcess 0: training loss at iteration 10000: 3.72452449798584
Traceback (most recent call last):
File "training_saccade.py", line 32, in
gtf.Train();
File "/home/SK00495085/monk/Monk_Object_Detection/6_cornernet_lite/lib/train_detector.py", line 307, in Train
validation_loss = self.system_dict["local"]["nnet"].validate(**validation)
File "/home/SK00495085/monk/Monk_Object_Detection/6_cornernet_lite/lib/core/nnet/py_factory.py", line 105, in validate
loss = self.network(xs, ys)
File "/home/SK00495085/.conda/envs/monk_env/lib/python3.6/site-packages/torch/nn/modules/module.py", line 489, in call
result = self.forward(*input, **kwargs)
File "/home/SK00495085/monk/Monk_Object_Detection/6_cornernet_lite/lib/core/models/py_utils/data_parallel.py", line 68, in forward
return self.module(*inputs[0], **kwargs[0])
File "/home/SK00495085/.conda/envs/monk_env/lib/python3.6/site-packages/torch/nn/modules/module.py", line 489, in call
result = self.forward(*input, **kwargs)
File "/home/SK00495085/monk/Monk_Object_Detection/6_cornernet_lite/lib/core/nnet/py_factory.py", line 20, in forward
loss  = self.loss(preds, ys, **kwargs)
File "/home/SK00495085/.conda/envs/monk_env/lib/python3.6/site-packages/torch/nn/modules/module.py", line 489, in call
result = self.forward(*input, **kwargs)
File "/home/SK00495085/monk/Monk_Object_Detection/6_cornernet_lite/lib/core/models/py_utils/losses.py", line 134, in forward
focal_loss += self.focal_loss(tl_heats, gt_tl_heat, gt_tl_valid)
File "/home/SK00495085/monk/Monk_Object_Detection/6_cornernet_lite/lib/core/models/py_utils/losses.py", line 57, in _focal_loss_mask
pos_pred = pred[pos_inds]
RuntimeError: The shape of the mask [8, 18, 64, 64] at index 1 does not match the shape of the indexed tensor [8, 79, 64, 64] at index 1

Which dataset are you working on? Please point to that dataset and share your code so that we can reproduce the errors

I'm working on a real-time project so I'm unable to share the data.
But the steps which I have done till now are listed below.

I set up the required environment to run CornerNet_Saccade
Converted POC XML to Coco dataset using VOC Type to Coco - Via Monk Type Annotation.ipynb
I have train and valid data so I considered Train With Validation Data.ipynb
So kept the model running for the whole night, but suddenly it stopped throwing that run time error.

Please check if there is any discrepancy within annotation files. Since it started and and ran for certain  hours then the issue could be traced back to an image maybe with no labels and bounding boxes or box shapes crossing the image boundaries.

yes, I crosschecked both train and valid data a couple of times regarding the bounding boxes and the labels. All are good. But when I go with only training data without validation data, everything works fine.
Seems there is an issue with the validation part.

Thank you for the detailed analysis. Will check validation codes.

We have run multiple tests on corner-net pipeline, yet the error wasn't reproduced. Have you reached a solution yet?

Closing due to inactivity


User with subset of corpus permissions unable to Query corpus

When a user has access to a corpus with only the 'Can enrich' and 'Can access database' permissions (i.e. the permissions in django we want to give for any restricted corpora), Enrichments all work fine, but Queries never complete. The user gets stuck waiting for query results.
The user is apparently unauthorized to see the query results. This is a permissions issue (we know because super-users can see the query results that the restricted user cannot).

I believe this should be fixed now with the permissions update, but can you test it again Vanna?


add sticky and stretch keywords

The stretch keyword is not mentioned in the working draft of the spec, but it is supported with a prefix by some browsers. Unfortunately, I couldn't see a good way to do a special case with it for -moz-available.
For part of #6130

Hello, thanks for submitting a PR!
We don't add non-standard things to -prefix-free, the whole point is that if it's non-standard, it should have a prefix in the code to make that fact obvious.
I would be delighted to merge this PR if you only leave sticky in it.

Well, you said in the issue that you would consider it.


Exported variable 'X' has or is using name 'Y' from external module "Z" but cannot be named.

TypeScript Version:  2.7.0-dev.20171221
Code
// foo.ts
import { WatchOptions } from "cpx";
export let x: WatchOptions;

// bar.ts
import { x } from "./foo";
export const x2 = x;

Just for the sake of the above you can install only @types/cpx
Expected behavior:
No error and the external module interface is named.
Actual behavior:
Exported variable 'x2' has or is using name 'WatchOptions' from external module "C:/git/xxxx/node_modules/@types/cpx/index" but cannot be named.

Related to https://github.com/Microsoft/TypeScript/issues/20657 but it seems this only solved local non node_modules imports

Duplicate of #9944


Spring - Add integration test scenario for Service Bus binder

Bug reported in https://github.com/Azure/azure-sdk-for-java/issues/20454, we need to add integration test for this scenario.

Add integration test scenario for Service Bus binder #21330


promql: Fix subtle bug where label lists could be mutated when printed

Fixes UTF-8 aggregator label list items getting mutated with quote marks when String-ified.
Fixes quoted metric names not supported in metric declarations.
Fixes UTF-8 label names not being quoted when String-ified.
Fixes https://github.com/prometheus/prometheus/issues/15470
Fixes https://github.com/prometheus/prometheus/issues/15528

any ideas why my TestParseSeriesDesc test is failing on CI but not on my machine? I have Go 1.23.3 locally

ok I can get it to fail with --tags=dedupelabels

ok it's probably because label equality is not the right way to test

Why does the parser change?

the parser change is the second item -- quoted metric names were not supported in the "series description" path, which is separate from the vector selector path that had been updated.


React v15

Thank you for this lib !
Using react v15.x, a warning comes up :
warning.js?8a56:36 Warning: Accessing PropTypes via the main React package is deprecated. Use the prop-types package from npm instead.
Here is a link for possible solution : https://facebook.github.io/react/warnings/dont-call-proptypes.html
Is your dependency on react v0.14 really necessary ? I could try to check this if you want ( though not really confident).

Now, the version 0.9.0 is using react v15.x.


Add support for starting services in a specified VRF.

The venerable iproute2 utility has recently introduced support
for executing programs in specific VRFs which are virtualized
routing tables.  These are typically used to isolate different
networking planes from each other, for security or flexibility
reasons.
Services which use the normal supervisor/start-stop-daemon
pattern can be configured by setting the vrf variable in the
/etc/conf.d tree for the service.
This allows for things like configuring the sshd service to
run in a management VRF, which is useful for high assurance
environments where the management plane is intended to be
isolated.
Signed-off-by: Ariadne Conill<EMAIL_ADDRESS>

The variable used for enabling this should start with "rc_" to avoid potential clashing with runscript-specific variables.
Also note that this only applies when using the "declarative" style for runscripts (ie, settings some variables and not defining a start function).  This will not apply to runscripts that use their own start function.

Everything in the provided reviews is correct. The response to the comment about this not applying to scripts that have their own start/stop functions is correct, but if they have their own start/stop functions we assume they know what they are doing.
I would also add a couple of things.
Since this is  linux specific, you would also want to check $RC_UNAME to make sure we are on Linux.
Also, please add documentation for this variable in etc/rc.conf.
Thanks much.
William


Route Constraints break source code generation

This might be a PEBKAC issue, but I noticed that the source generation breaks when adding a route constraint (something valid with ASP.NET routing).
using Marten;
using Wolverine.Http;
using WolverineMartenSample.Models;

namespace WolverineMartenSample.Endpoints.Todos;

public static class ShowEndpoint
{
    // HERE route constraint of :int
    [WolverineGet("/todos/{id:int}")]
    public static async Task<IResult> Get(int id, IQuerySession session)
    {
        var todo = await session.LoadAsync<Todo>(id);
        return todo is null ? Results.NotFound() : Results.Ok(todo);
    }
}

I get this error:
System.InvalidOperationException: Compilation failures!

CS1514: { expected
CS1513: } expected
CS1022: Type or namespace definition, or end-of-file expected
CS0116: A namespace cannot directly contain members such as fields, methods or statements
CS1022: Type or namespace definition, or end-of-file expected
CS0116: A namespace cannot directly contain members such as fields, methods or statements
CS1022: Type or namespace definition, or end-of-file expected
CS0116: A namespace cannot directly contain members such as fields, methods or statements
CS0116: A namespace cannot directly contain members such as fields, methods or statements
CS1022: Type or namespace definition, or end-of-file expected
CS8124: Tuple must contain at least two elements.
CS1022: Type or namespace definition, or end-of-file expected
CS0116: A namespace cannot directly contain members such as fields, methods or statements
CS1022: Type or namespace definition, or end-of-file expected
CS0116: A namespace cannot directly contain members such as fields, methods or statements
CS1022: Type or namespace definition, or end-of-file expected
CS0116: A namespace cannot directly contain members such as fields, methods or statements
CS1022: Type or namespace definition, or end-of-file expected
CS0116: A namespace cannot directly contain members such as fields, methods or statements
CS1022: Type or namespace definition, or end-of-file expected
CS1022: Type or namespace definition, or end-of-file expected
CS1022: Type or namespace definition, or end-of-file expected
CS0116: A namespace cannot directly contain members such as fields, methods or statements
CS0116: A namespace cannot directly contain members such as fields, methods or statements
CS0509: 'GET_todos_id': cannot derive from sealed type 'int'
CS0115: '<invalid-global-code>.Handle(HttpContext)': no suitable method found to override
CS0103: The name '_sessionFactory' does not exist in the current context

Code:

// <auto-generated/>
#pragma warning disable
using Marten;
using Microsoft.AspNetCore.Routing;
using System;
using System.Linq;
using Wolverine.Http;

namespace Internal.Generated.WolverineHandlers
{
    // START: GET_todos_id:int
    public class GET_todos_id:int : Wolverine.Http.HttpHandler
    {
        private readonly Wolverine.Http.WolverineHttpOptions _options;
        private readonly Marten.ISessionFactory _sessionFactory;

        public GET_todos_id:int(Wolverine.Http.WolverineHttpOptions options, Marten.ISessionFactory sessionFactory) : base(options)
        {
            _options = options;
            _sessionFactory = sessionFactory;
        }

        public override async System.Threading.Tasks.Task Handle(Microsoft.AspNetCore.Http.HttpContext httpContext)
        {
            await using var querySession = _sessionFactory.QuerySession();
            if (!int.TryParse((string)httpContext.GetRouteValue("id"), out var id))
            {
                httpContext.Response.StatusCode = 404;
                return;
            }

            var result = await WolverineMartenSample.Endpoints.Todos.ShowEndpoint.Get(id, querySession).ConfigureAwait(false);
            await result.ExecuteAsync(httpContext).ConfigureAwait(false);
        }

    }

    // END: GET_todos_id:int

}

   at JasperFx.RuntimeCompiler.AssemblyGenerator.Generate(String code)
   at JasperFx.RuntimeCompiler.AssemblyGenerator.Compile(GeneratedAssembly generatedAssembly, IServiceVariableSource services)
   at JasperFx.RuntimeCompiler.CodeFileExtensions.InitializeSynchronously(ICodeFile file, GenerationRules rules, ICodeFileCollection parent, IServiceProvider services)
   at Wolverine.Http.HttpChain.<>c__DisplayClass66_0.<BuildEndpoint>b__0()
   at System.Lazy`1.ViaFactory(LazyThreadSafetyMode mode)
   at System.Lazy`1.ExecutionAndPublication(LazyHelper executionAndPublication, Boolean useDefaultConstructor)
   at System.Lazy`1.CreateValue()
   at Wolverine.Http.HttpChain.<>c__DisplayClass66_0.<BuildEndpoint>b__1(HttpContext c)
   at Microsoft.AspNetCore.Routing.EndpointMiddleware.Invoke(HttpContext httpContext)
--- End of stack trace from previous location ---
   at Swashbuckle.AspNetCore.SwaggerUI.SwaggerUIMiddleware.Invoke(HttpContext httpContext)
   at Swashbuckle.AspNetCore.Swagger.SwaggerMiddleware.Invoke(HttpContext httpContext, ISwaggerProvider swaggerProvider)
   at Microsoft.AspNetCore.Diagnostics.DeveloperExceptionPageMiddlewareImpl.Invoke(HttpContext context)

HEADERS
=======
Accept: application/json
Connection: keep-alive
Host: localhost:5279
User-Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_7) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/<IP_ADDRESS> Safari/537.36
Accept-Encoding: gzip, deflate, br
Accept-Language: en-US,en;q=0.9
Referer: http://localhost:5279/swagger/index.html
sec-ch-ua: "Not.A/Brand";v="8", "Chromium";v="114"
DNT: 1
sec-ch-ua-mobile: ?0
sec-ch-ua-platform: "macOS"
Sec-Fetch-Site: same-origin
Sec-Fetch-Mode: cors
Sec-Fetch-Dest: empty

Fixed for 1.2.


Changes for new pandoc version, Text instead of String

I made some changes for compliance with newer pandoc version. It would be great if this could be merged into the main repo.

This has been merged.


Cannot augment Value to process.

With pre 0.9.0 versions of Ditto I could use a TypeConverter to easily process NuPicker values.
public override object ConvertFrom(ITypeDescriptorContext context, CultureInfo culture, object value)
{
    Picker picker = value as Picker;
    if (picker != null)
    {
        if (picker.PickedKeys.Any())
        {
            return base.ConvertFrom(context, culture, picker.PickedKeys);
        }

        return null;
    }

    return base.ConvertFrom(context, culture, value);
}

This was possible as I was able to pass a different value to the ConvertFrom method.
In 0.9.0 the Value property setter is internal as is the ProcessValue(value, context) method. This looks like it means I would have to implement all the code contained within UltimatePickerAttribute in order to implement that processor.

Hey Jim,
You'll probably want to make a NuPicker attribute like this:
public class NuPickerAttribute : DittoProcessorAttribute
{
    public override object ProcessValue()
    {
        var picker = this.Value as Picker;
        if (picker != null && picker.PickedKeys.Any())
        {
            return picker.PickedKeys;
        }
        return this.Value;
    }
}

Then you'll want to chain them on your property like:
[UmbracoProperty(Order = 0)]
[NuPicker(Order = 1)]
[UmbracoPicker(Order = 2)]
public IPublishedContent MyPicker { get; set; }

If you find you are defining that on your properties a lot, you could also wrap it in a multi processor for convenience:
public class MyNuPickerAttribute : DittoMultiProcessorAttribute
{
    public MyNuPickerAttribute ()
        : base(new DittoProcessorAttribute[]
        {
            new UmbracoPropertyAttribute(),
            new NuPickerAttribute(),
            new UltimatePickerAttribute()
        })
    { }
}

then your property would be like:
[MyNuPicker]
public IPublishedContent MyPicker { get; set; }

That second one's the bad boy I'm after. Ace!

I don't suppose it would be possible to add this to the documentation? I went through hell today trying to figure out the answer to my problem, haha. And here it was, a beautiful example!

@naepalm We'll add it for sure 👍
Getting all the docs up to date is planned for our v1.0 rollup release - in terms of when that will happen, we're currently neck-deep in client work, (aren't we all, right? 😃) - but yes, it's still very much on our todo list.

@leekelleher @naepalm
Agreed... Though I would say we're pretty much feature complete now. (Once my PR is merged) so we should concentrate on (code cleanup/performance) and testing a few final things in preparation for that release. (What happens when someone uses List<T> instead of IEnumerable<T> against the UmbracoPicker?)
Oh and here's up-to-date NuPicker examples
Picker
/// <summary>
/// Provides a unified way of converting the <see cref="Picker"/> type to the given <see cref="Type"/>
/// </summary>
public class NuPickerAttribute : DittoMultiProcessorAttribute
{
    /// <summary>
    /// Initializes a new instance of the <see cref="NuPickerAttribute"/> class.
    /// </summary>
    public NuPickerAttribute()
        : base(new DittoProcessorAttribute[] { new NuPickerConverterAttribute(), new UmbracoPickerAttribute() })
    {
    }

    /// <summary>
    /// Returns the correct value to pass to the <see cref="UmbracoPickerAttribute"/>
    /// </summary>
    private class NuPickerConverterAttribute : DittoProcessorAttribute
    {
        /// <inheritdoc/>
        public override object ProcessValue()
        {
            Picker picker = this.Value as Picker;
            if (picker != null && picker.PickedKeys.Any())
            {
                return picker.PickedKeys;
            }

            return this.Value;
        }
    }
}

EnumPicker
/// <summary>
/// Provides a unified way of converting the <see cref="Picker"/> type to the given enum. />
/// </summary>
public class NuPickerEnumAttribute : DittoMultiProcessorAttribute
{
    /// <summary>
    /// Initializes a new instance of the <see cref="NuPickerEnumAttribute"/> class.
    /// </summary>
    public NuPickerEnumAttribute()
        : base(new DittoProcessorAttribute[] { new NuPickerEnumConverterAttribute(), new EnumAttribute() })
    {
    }

    /// <summary>
    /// Returns the correct value to pass to the <see cref="EnumAttribute"/>
    /// </summary>
    private class NuPickerEnumConverterAttribute : DittoProcessorAttribute
    {
        /// <inheritdoc/>
        public override object ProcessValue()
        {
            Picker picker = this.Value as Picker;
            return picker != null ? picker.SavedValue : this.Value;
        }
    }
}

PROTIP: Ditto automatically adds the UmbracoProperty attribute so you never really have to.

@JimBobSquarePants Thank you SO MUCH for including those. Until the docs get updated, I'll just bookmark this for myself as a reminder ;)

No worries, always happy to help 😄  Gonna try to get stuck into the docs soon.

Really excited about this release [docs] :-)


Hosting on Deno Deploy

How would one go about deploying a relay on deno deploy?

This repo becomes a pure lib for relay, you can start directly with:
deno run -A scripts/start.ts

I added extra features with this lib on another repo: https://github.com/mdzz-club/relay66


Trigonometric functions for EDecimal

Includes sin, cos, tan, asin, acos, atan, atan2.
Also including sinh, cosh, tanh should be easy.
Here is a starting point: https://github.com/raminrahimzada/CSharp-Helper-Classes/blob/master/Math/DecimalMath/DecimalMath.cs
Also doing them for EFloat is okay but we only use EDecimal so it is more important to us.

There are also a couple of tests that can be ported directly: https://github.com/raminrahimzada/CSharp-Helper-Classes/blob/master/Math/DecimalMath/DecimalMathUnitTests.cs

The reason why it is not implemented might be because it is hard to implement trigonometric functions with correct rounding and because there are few resources on correctly rounded trigonometric algorithms.
Here is a resource on correct rounding:
https://www.mpfr.org/algorithms.pdf


[5.3] Controller closure middleware

This will allow registering a closure middleware in the controller's constructor, so that any auth/session stuff can be called and set as properties on the controller:
public function __construct()
{
    $this->middleware(function ($request, $next) {
        $this->user = $request->user();

        return $next($request);
    });
}

Notes:

To allow middleware closures, I had to rework the router's sortMiddleware method to not use collections, since contains and search do something else (invoke it) when passed a closure. That method looks a little nasty now :confused:

There's a small change which I'm not sure would be considered breaking: the middleware property on the controller is now stored in a different format. I don't think this is meant to be tinkered with directly, but who knows?

Another option would be to create an before/after action hook like this:
public __construct() {
 $this->beforeAction('index', ['handleSessions']);
}

private handleSessions() {
 //do your thing
}

public index() {
 //handle sessions already executed
}

But the closure on the middleware should do the trick on most cases, not sure of how it would work in all cases probably

Hi @JosephSilber just a quick question, is it reasonable to make use of a middleware class in this new way:
public function __construct()
{
$this->middleware(function ($request, $next) {
return new MyMiddleware()->handle($request, $next);
});
}

That function has always accepted a string of a middleware class, which still works:
$this->middleware(MyMiddleware::class);

Ha thanks @JosephSilber I did not know that! I've just done a PR do update the Controller page to demo both of these.


Exploration API Implementation

In gitlab by @sylvaingaudan on Apr 12, 2017, 11:07
Implement the Exploration API with the following steps:

Step
Count
Search
Aggregate

Check Params
[x]
[x]
[x]

Create Filter Request Part
[x]
[x]
[x]

Create Size Request Part
[ ]
[x]
[x]

Create Sort Request Part
[ ]
[x]
[x]

Create Projection Request Part
[ ]
[x]
[ ]

Create Aggregation Request Part
[ ]
[ ]
[x]

Execution
[x]
[x]
[x]

Extract Result
[ ]
[x]
[x]

Extract Count
[x]
[ ]
[ ]

Transform GeoJSON
[ ]
[x]
[x]

Human and Pretty
[x]
[x]
[x]

In gitlab by @MohamedHamouGisaia on Apr 12, 2017, 14:11
created branch 3-exploration-api-implementation

In gitlab by @MohamedHamouGisaia on Apr 19, 2017, 14:40
mentioned in merge request !10

In gitlab by @sfalquier on Apr 19, 2017, 15:24
closed via commit 787b0e783d5a2beec90402610d122c563c5c7128

In gitlab by @MohamedHamouGisaia on Apr 19, 2017, 15:31
reopened

In gitlab by @sfalquier on Apr 20, 2017, 11:53
mentioned in commit 787b0e783d5a2beec90402610d122c563c5c7128

In gitlab by @sylvaingaudan on Apr 26, 2017, 12:09
changed milestone to %2

In gitlab by @sylvaingaudan on Apr 26, 2017, 12:23
added ~1918368 label

In gitlab by @sylvaingaudan on Apr 28, 2017, 13:33
closed

In gitlab by @sylvaingaudan on May 3, 2017, 17:37
removed ~1918368 label


hw03 dbernhard tests: bitcast_{1,3,4} should not pass

The bitcast_1, bitcast_3 and bitcast_4 tests use the i8 type, which is specified to be not part of LLVMLite - LLVMLite only supports i8* pointers according to the documentation on moodle...

You're right. I'll fix them :+1:

@dbernhard-0x7CD I just merged your PR since we are running out of time, however I would prefer to still have some tests with i8* type. I would say that to provide a meaningful test with i8* we should for example setting up a global string, then maybe index in it with a gep, which would retrieve an i8*, then convert the pointer with a bitcast to i64* and load a full quadword from memory. At this point either shift the result to take only first character or return the full quadword if the memory we load was initialiased already (since gep allows out of bound indexing).
What do you think? Unfortunately I am having lessons now that's why I am asking (currently from mobile) :)


Map Group Icon - Change Color

Make the group icon a color that is distinct from any of the dots.

Changed the marker color of one of the marker to black, none of the marker color is yellow. The clustering color is retained as yellow.

Legislative district map

QA tested and confirmed done. Reference: https://drive.google.com/open?id=1bQqsp0BJVHhz0xUFZYeApBnDuBBvDr7vkWL-2d4dUNc


Firmware 3.5.62-95 - unable to get username/password

Hello there. Is the firmware (3.5.62-95) support? I can't get username and password, neither via iobroker web ui nor via npm or docker. And yes, i followed the instructions exactly ;)
My iobroker instance is running in docker container. My robot is connected to my wifi, iobroker and roombar are in the same net / subnet. It is accessable via the iRobot-App but iobroker can't receive username and password. Other pieces of information (when not in wifi setup mode (flashing wifi symbol)) like model, blid, serialnumber can be retrieved.
When I press and hold "spot" and "clean" while it the roomba is at the base, wifi starts blinking and the get-password npm says to press one key. After doing that it says sth. like "undefined" and times out after a while with that output:

Make sure your robot is on the Home Base and powered on (green lights on). Then press and hold the HOME button on your robot until it plays a series of tones (about 2 seconds). Release the button and your robot will flash WIFI light.
Then press any key here...
node:events:304
throw er; // Unhandled 'error' event
^
Error: connect ETIMEDOUT <IP_ADDRESS>:8883
at TCPConnectWrap.afterConnect [as oncomplete] (node:net:1129:16)
Emitted 'error' event on TLSSocket instance at:
at emitErrorNT (node:internal/streams/destroy:194:8)
at emitErrorCloseNT (node:internal/streams/destroy:159:3)
at processTicksAndRejections (node:internal/process/task_queues:80:21) {
errno: -110,
code: 'ETIMEDOUT',
syscall: 'connect',
address: '<IP_ADDRESS>',
port: 8883
}

The roomba also is pingable from the docker container.
What can I do/test? Thanks in advance.

Roomba Model: Roomba 675
Roomba Software Version: 3.5.62-95

Hi, any news or isn't this repo supported any more? Tell me, how I can help and I'll try....

hi, have you tried it with the dorita way?
to detect the correct connection settings use the "getpassword" command from
https://github.com/koalazak/dorita980#readme
and paste it to the adapter settings.

Hi and sorry for the delay. Didn't expect to get an answer after that time. The mail may be ended in nirvana..... I tried all the ways (via iobroker, via docker, via general notebook with node.js, the javascript way, the npm way).
Today, I did a factory reset at the roomba and the most I can see is the blid, but an error occurs while getting the password. Here is my output:
Make sure your robot is on the Home Base and powered on (green lights on). Then press and hold the HOME button on your robot until it plays a series of tones (about 2 seconds). Release the button and your robot will flash WIFI light.
Then press any key here...
Robot Data:
{
  ver: '3',
  hostname: 'Roomba-3162C21C60127610',
  robotname: 'Roomba',
  ip: '<IP_ADDRESS>',
  mac: 'xx:xx:xx:xx:xx:xx',
  sw: '3.5.62',
  sku: 'R675040',
  nc: 1,
  proto: 'mqtt',
  cap: { ota: 1, eco: 1, svcConf: 1 },
  blid: '3162C21C601******' <--
}
node:events:353
      throw er; // Unhandled 'error' event
      ^

Error: read ECONNRESET
    at TLSWrap.onStreamRead (node:internal/stream_base_commons:213:20)
Emitted 'error' event on TLSSocket instance at:
    at emitErrorNT (node:internal/streams/destroy:188:8)
    at emitErrorCloseNT (node:internal/streams/destroy:153:3)
    at processTicksAndRejections (node:internal/process/task_queues:80:21) {
  errno: -4077,
  code: 'ECONNRESET',
  syscall: 'read'
}

That's all I got. Please, I somebody has some information. I also adjusted my mail settings to react in time ;)
Thanks

Hi and sorry for the delay. Didn't expect to get an answer after that time. The mail may be ended in nirvana..... I tried all the ways (via iobroker, via docker, via general notebook with node.js, the javascript way, the npm way).
Today, I did a factory reset at the roomba and the most I can see is the blid, but an error occurs while getting the password. Here is my output:
Make sure your robot is on the Home Base and powered on (green lights on). Then press and hold the HOME button on your robot until it plays a series of tones (about 2 seconds). Release the button and your robot will flash WIFI light.
Then press any key here...
Robot Data:
{
  ver: '3',
  hostname: 'Roomba-3162C21C60127610',
  robotname: 'Roomba',
  ip: '<IP_ADDRESS>',
  mac: 'xx:xx:xx:xx:xx:xx',
  sw: '3.5.62',
  sku: 'R675040',
  nc: 1,
  proto: 'mqtt',
  cap: { ota: 1, eco: 1, svcConf: 1 },
  blid: '3162C21C601******' <--
}
node:events:353
      throw er; // Unhandled 'error' event
      ^

Error: read ECONNRESET
    at TLSWrap.onStreamRead (node:internal/stream_base_commons:213:20)
Emitted 'error' event on TLSSocket instance at:
    at emitErrorNT (node:internal/streams/destroy:188:8)
    at emitErrorCloseNT (node:internal/streams/destroy:153:3)
    at processTicksAndRejections (node:internal/process/task_queues:80:21) {
  errno: -4077,
  code: 'ECONNRESET',
  syscall: 'read'
}

That's all I got. Please, I somebody has some information. I also adjusted my mail settings to react in time ;)
Thanks

Any progress on this?

Please check with newest version of adapter (1.2.1).
If issue still occure, please report your configuration (Adapter version, js-controller version, node, npm, OS).
If no response received within 30 days this issue might be closed.

no reaction received - lets close here.
If the problem still exists please post a commengt or open a new issue.


Generate a more minimal tsconfig paths output for default scenario

For each folder tsconfig.json is updated with a new paths entry. This can get a bit noisy to check in anytime a new folder is created. For the default config which is one level deep the following configuration can be used:
{
  "compilerOptions": {
    "paths":  {
      "@*": ["src/*"]
    }
  }
}

That single configuration can handle any number of folders one level deep

This has been reported multiple times but its not part of the library scope.
vite-aliases is specifically build for bigger projects where organisation and management is key.

vite-aliases is specifically build for bigger projects where organisation and management is key

My suggestion was to reduce the amount of code output to tsconfig.json to a version that is more terse, to improve developer experience. I don't see how that suggestion runs counter to your stated goal.
As an aside, it is common to post enhancement or feature requests to repos. If you don't want that kind of feedback, it might be worth posting something in your readme.
Cheers

Ah okay, sorry, must've misunderstood your issue then!
So are you suggesting if the user sets the option depth to 1, it wont generate any of the folders as aliases?

Correct, that is my suggestion exactly. It's not a big deal, functionally they are equivalent but I think using this provides a slightly better developer experience because the tsconfig only needs to be updated once whereas with the current implementation adding or removing folders will change the file and require updates to be checked in.
If you think this is a good enhancement I'd be happy to make a pull request. If not, no worries, appreciate your consideration either way.

Hm, i think might fit well into the current Changes that i am working on!
Will post something in this Thread, if i am gonna include it directly in the upcoming changes.

Will be included in the next upcoming v0.12.0


Native integration with Kafka

Having native integration/libraries for Kafka would be incredibly useful.
AWS provides a managed Kafka service, and Kafka is a popular messaging service for those on the JVM.
P.S. I searched through the issues and there wasn't a mention of a native Kafka integration.
Sample Implemenations
Here is the outline for both producer/consumer interceptors as an example:
import org.apache.kafka.clients.producer.ProducerInterceptor
import org.apache.kafka.clients.consumer.ConsumerInterceptor

class KafkaProducerInterceptor<K, V> : ProducerInterceptor<K, V> {
    override fun onSend(record: ProducerRecord<K, V>?): ProducerRecord<K, V> = TODO("Not yet implemented")

    override fun configure(configs: MutableMap<String, *>?) = TODO("Not yet implemented")

    override fun onAcknowledgement(metadata: RecordMetadata?, exception: Exception?) = TODO("Not yet implemented")

    override fun close() = TODO("Not yet implemented")
}

class KafkaConsumerInterceptor<K, V> : ConsumerInterceptor<K, V> {
    override fun onConsume(records: ConsumerRecords<K, V>?): ConsumerRecords<K, V> = TODO("Not yet implemented")

    override fun onCommit(offsets: MutableMap<TopicPartition, OffsetAndMetadata>?) = TODO("Not yet implemented")

    override fun configure(configs: MutableMap<String, *>?) = TODO("Not yet implemented")

    override fun close() = TODO("Not yet implemented")
}

These interceptors are managed by the Kafka client library, potentially in a separate thread context. How should the Segment be passed in or fetched?

Hi @KushalP,
We currently do not have it on our roadmap to add native support for Kafka. For handling segments in separate threads, you could check out the utility executors implemented in this PR which do a pretty good job of handling the context for async work. I'd need a little more information about how Kafka uses threads to better answer that.
We are always happy to review PRs for new instrumentation support, potentially in a new community-contributed section of the repository.

I'm happy to look into this, if you're able to elaborate on how these utility executors should be used.
It's common to use a Kafka producer as follows:
val config = mapOf(
    ProducerConfig.BOOTSTRAP_SERVERS_CONFIG to "kafka:9094",
    ProducerConfig.ACKS_CONFIG to "all",
    ProducerConfig.KEY_SERIALIZER_CLASS_CONFIG to StringSerializer::class.java,
    ProducerConfig.VALUE_SERIALIZER_CLASS_CONFIG to StringSerializer::class.java
)

val topic = "some-topic-name"
val producer = KafkaProducer<String, String>(config)

// This is where we trigger the internal I/O Thread and Sender Thread within the Kafka library.
producer.send(ProducerRecord(topic, "key", "value"))

The send method on KafkaProducer has the following two signatures if it's useful:
public Future<RecordMetadata> send(ProducerRecord<K, V> record)
public Future<RecordMetadata> send(ProducerRecord<K, V> record, Callback callback)

The following libraries  exist for similar tracing tools for comparison:

Zipkin: https://github.com/openzipkin-contrib/brave-kafka-interceptor
OpenTracing: https://github.com/opentracing-contrib/java-kafka-client

I see, I'm not actually sure  the utility executors fit this use case then. You'll probably just want to wait until the future resolves and capture the metadata in a subsegment.
FWIW, the X-Ray team is currently working on interoperability with OpenTelemetry, which means instrumenting with OT and using the X-Ray backend exporter should be able to support a Kafka interceptor made for OT as well. See here for the PoC.

In addition, X-Ray does actually document a workaround to trace Kafka producers & consumers. See this blog post for a step-by-step process for instrumenting both the producer and consumer side of messages, and linking them on the X-Ray Service map. There is also a sample app containing working source code from the blog post.
If you would still like to make a pull request to effectively turn this example code into a library, you are welcome to.


Also add mongo-tools

This allows for mongodump to be available when running backup.
Otherwise, you get:

Warning: /tmp/backup_Zx8aoj: line 52: mongodump: command not found

This is also needed for running yunohost backup restore.

:shark:

Cheers !


Adding attributes file to ignore HTML test files

This should exclude them from future diffs and Git linguist calculations.
No review is requested for this.

Codecov Report

Merging #62 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #62   +/-   ##
=======================================
  Coverage   94.77%   94.77%
=======================================
  Files          85       85
  Lines         670      670
=======================================
  Hits          635      635
  Misses         35       35

Continue to review full report at Codecov.

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


Add support for full offline mode

To develop locally with Fn, one must have internet connection due to the hard dependency with Docker Hub as per README.md:
# Set your Docker Hub username
export FN_REGISTRY=<DOCKERHUB_USERNAME>

It is crucial to allow developers to work on offline mode for many reasons.
Would be nice to have a feature like:
fn start --go-offline

This call would download all available FDKs and latest Docker images.

If you've run fn start once already, then you can work offline. fn deploy --local keeps deploys local too.

Maybe having fn start --local would make more sense, and then any fn deploy would automatically attempt local initially (with pretty fast timeout) and then remote. Having to do --local on every fn deploy seems odd.

fn start simply starts the server. I don't think it makes sense for the server to have an offline mode. That said the confusion might be because fn deploy is doing both a build and a push, whereas --local skips the push. Can think about this one but more urgent work to be done.

I have not had any issues doing development offline (damn you, sightglass!), for what it's worth. That being said, it's because I have all the images downloaded from working online previously. One test fails, but as far as building the fn image itself and using the cli, with fn deploy --local it has been pretty painless to work offline. Big +1 for airplane mode working tho, a good thing to keep in mind :)
a list of images:

fnproject/functions (this can be easily built with make docker-build)
mysql
postgres
fnproject/hello
fnproject/error

possibly others (mostly underneath the above images, i believe), but running make test while online will get most stuff necessary for dev; i.e. it's close to the desired fn start --go-offline, fdk's are a notable exclusion.

I can only add that you might need following images for development/testing:

redis
quay.io/goswagger/swagger
funcy/timeout:0.0.1
funcy/multi-log:0.0.1
denismakogon/os.environ
funcy/log:0.0.1

Regarding FDKs, AFAIK we don't have images for each of supported. We insist on to use programming language-specific distribution/consumption model (wheels for python, gem for ruby, etc.)
@treeder can we move those (from mine and funcy spaces) images to fnproject space?

Closing since this already works.


Elasticsaerch and redis driver

bugfix for mysql driver with spaces in a databse name
elasticsaerch driver
redis driver

Just to leave some feedback. This service and supporting components are going to be changing. It's likely that the db package ends up using go-os/db which there's an open issue for since it was called go-platform. So not sure about merging it. Also if the redis driver cannot support the search method its not really useful as its just more of a glorified key value store.

Thanks for the feedback.
I've removed the Redis driver.
I've seen a few repositories where people use or seek Elasticsearch driver, so it might be interesting even if the db-srv changes. Elasticsearch driver is well tested on our site and just a small addition to the current settings.
Also there is a small bugfix in the mysql driver.

Thanks for the PR but its likely that db-srv will no longer be a proxy for databases but rather something standalone.


Alternative Typeclass Implementation?

Pardon my ignorance if there's an obvious reason not to do this, I'm really new to functional programming, but I was playing around with porting Haxl to different environments (in particular, Purescript to try to ease in the nodejs community to the idea), and I realized that it's pretty straightforward to implement a <|> operator. This implementation is Purescript so I'm not sure how well it translates to real Haskell
https://github.com/jqyu/purescript-rad/blob/master/src/Rad/Core/Monad.purs#L236-L243
I don't know much about the Haskell runtime but I suppose the fact that it's a recursive implementation may make memory bounds hard to predict, but this allows for a graceful way of recovering from errors, as well as straightforward implementations of retried requests (i.e., chaining a fixed number of <|> calls to catch failures)
In particular, I'm working on a GraphQL implementation using Haxl and it would be really useful for me to be able to catch errors and return a null result for a branch of a query so I figured other people might be able to benefit from this as well.

<|> could be fairly straight-forward, but doesn't fully fit in with our intended exception semantics. We use withDefault more to ensure that CriticalErrors bubble up appropriately when the framework fails.
https://github.com/facebook/Haxl/blob/master/Haxl/Prelude.hs#L198-L203
<|> will also not save you from pure exceptions in code (someone calls error).
Is there a use-case that fits using an implementation of <|> over withDefault?

On closer inspection it looks like withDefault and catchAny cover all my use cases, I just need to restructure some of my own code.
Thanks for addressing this!

Ok, I'm going to go ahead and close this. @jqyu thanks for bringing it up, and let us know if there's anything else we can help with.


[Bug] package.json could not be resolved in analytics constants

Bug Report
Describe the bug
When importing the package:

It seems like this is fixed by changing this to:
import pkg from '../../../../package.json';

Though thats me playing inside node_modules, so whether or not that compiles like that is a different story

@matiasfha im stil seeing a similar issue on latest


Add Retrying for Internal Server Errors?

We have found from time to time the APIs will fail due to an internal server error. These are rare, but they do occur. For example, from a test today:
PHPUnit 4.8.36 by Sebastian Bergmann and contributors.
....ESSSS
Time: 20.39 seconds, Memory: 14.00MB
There was 1 error:
1) Google\Cloud\Samples\Monitoring\monitoringTest::testWriteTimeseries
Google\ApiCore\ApiException: {
    "message": "One or more TimeSeries could not be written:\nAn internal error occurred: timeSeries[0]",
    "code": 13,
    "status": "INTERNAL",
    "details": [
        {
            "@type": "google.monitoring.v3.createtimeserieserror-bin",
            "data": "<Unknown Binary Data>"
        },
        {
            "@type": "grpc-status-details-bin",
            "data": "<Unknown Binary Data>"
        }
    ]
}
/home/[secure]/build/GoogleCloudPlatform/php-docs-samples/monitoring/vendor/google/gax/src/ApiCore/ApiException.php:98
/home/[secure]/build/GoogleCloudPlatform/php-docs-samples/monitoring/vendor/google/gax/src/ApiCore/Transport/GrpcTransport.php:160
/home/[secure]/build/GoogleCloudPlatform/php-docs-samples/monitoring/vendor/guzzlehttp/promises/src/Promise.php:246
/home/[secure]/build/GoogleCloudPlatform/php-docs-samples/monitoring/vendor/guzzlehttp/promises/src/Promise.php:223
/home/[secure]/build/GoogleCloudPlatform/php-docs-samples/monitoring/vendor/guzzlehttp/promises/src/Promise.php:62
/home/[secure]/build/GoogleCloudPlatform/php-docs-samples/monitoring/vendor/google/cloud-monitoring/V3/Gapic/MetricServiceGapicClient.php:853
/home/[secure]/build/GoogleCloudPlatform/php-docs-samples/monitoring/src/write_timeseries.php:84
/home/[secure]/build/GoogleCloudPlatform/php-docs-samples/monitoring/monitoring.php:128
/home/[secure]/build/GoogleCloudPlatform/php-docs-samples/monitoring/vendor/symfony/console/Command/Command.php:250
/home/[secure]/build/GoogleCloudPlatform/php-docs-samples/monitoring/vendor/symfony/console/Tester/CommandTester.php:79
/home/[secure]/build/GoogleCloudPlatform/php-docs-samples/monitoring/test/monitoringTest.php:143
/home/[secure]/build/GoogleCloudPlatform/php-docs-samples/monitoring/test/monitoringTest.php:91
FAILURES!
Tests: 6, Assertions: 8, Errors: 1, Skipped: 4.

Should we add internal server errors to the list of retry codes by default? We could possibly implement a separate flag for it, e.g. 'retryServerErrors' => true. Or, if it continues to happen, we can have it added to metric_service_client_config.json like it is for pubsub, although this would require a change all the way up the chain.

I had another occurrence of this in our test suite, this time for the Tasks API. This would randomly cause 500 failures in our customer's applications, so gracefully handling this internally is really important.

The question seems to be how big of a risk we introduce if we do retry non idempotent methods. Could there be cases where an internal error could be returned, but the request did in fact process? I lean towards retrying server errors in what we could consider safe circumstances, but wonder if this is a discussion that has been hashed out before. @michaelbausor would you have any insight?

I think that this needs to be assessed on an API-by-API basis. I know that there are some APIs where retrying on Internal is safe (and in fact highly desirable) but I don't think it is safe to do so on all APIs all the time.


UnicodeEncodeError: 'charmap' codec can't encode characters in position 6-19: character maps to

Describe the bug
Traceback (most recent call last):
  File "dflauncher.py", line 242, in <module>
    automate_script(3)
  File "dflauncher.py", line 167, in automate_script
    'Username:'], timeout=4)
  File "C:\Users\myusername\git\test\wexpect.py", line 1366, in expect
    return self.expect_list(compiled_pattern_list, timeout, searchwindowsize)
  File "C:\Users\myusername\git\test\wexpect.py", line 1380, in expect_list
    return self.expect_loop(searcher_re(pattern_list), timeout, searchwindowsize)
  File "C:\Users\myusername\git\test\wexpect.py", line 1433, in expect_loop
    c = self.read_nonblocking(self.maxread, timeout)
  File "C:\Users\myusername\git\test\wexpect.py", line 1793, in read_nonblocking
    self.logfile_read.write (s)
  File "C:\Python36\lib\encodings\cp1252.py", line 19, in encode
    return codecs.charmap_encode(input,self.errors,encoding_table)[0]
UnicodeEncodeError: 'charmap' codec can't encode characters in position 6-19: character maps to <undefined>

To Reproduce
I've got no clue... use a colon in the expectation?
Expected behavior
Not crash.
Environment:

[windows version] 10
[Python version] 3.6
[wexpect version] latest from pypi source, v2.3.6

At the first sight it seems that the logging fails. Please try to remove the logging from the line 1793 as a workaround.
I will try to reproduce as soon as it is possible.

Here is a related StackOverflow post.
I have created a new branch: i11  for this issue.
unicode_parent can reproduce a very similar issue.

I just encountered the error again.
For me it was self.logfile_read, not self.logfile... I applied what the stackoverflow post said, changing the wexpect.py line to:
self.logfile_read.write (s.encode("utf-8"))
and things seem to work now :)

I had to update my local copy of wexpect.py with this line today, in place of the previous "fix":
self.logfile_read.write (s.encode("utf-8") if not isinstance(s, str) else s)
It was after using this line:
shell.logfile_read = sys.stdout

that I got the strange error:
Traceback (most recent call last):
  File "main.py", line 11, in <module>
    shell.expect(unique_prompt)
  File "C:\Users\nmz787\git\wexpect_issues\wexpect.py", line 1388, in expect
    return self.expect_list(compiled_pattern_list, timeout, searchwindowsize)
  File "C:\Users\nmz787\git\wexpect_issues\wexpect.py", line 1402, in expect_list
    return self.expect_loop(searcher_re(pattern_list), timeout, searchwindowsize)
  File "C:\Users\nmz787\git\wexpect_issues\wexpect.py", line 1455, in expect_loop
    c = self.read_nonblocking(self.maxread, timeout)
  File "C:\Users\nmz787\git\wexpect_issues\wexpect.py", line 1816, in read_nonblocking
    self.logfile_read.write (s.encode("utf-8") if isinstance(s, str) else s)
TypeError: write() argument must be str, not bytes

I beleive this is the same as #47


Understanding the outcomes of Google Summer of Code

Dear Christian Bruckmayer,
I am a Ph.D. student at the University of São Paulo (Brazil). We would like to understand how and how much Open Source Communities benefit from participating in Google Summer of Code (GSoC). We believe that your experience in GSoC is valuable and could help us achieve that. In a 1-minute questionnaire, we would like to know:
Concerning the project entitled "Open Source Event Manager (OSEM): Implemention Organizer Dashboard" developed in GSoC 2014.

Do you have a GitHub repository for the GSoC project? If so, could you provide us with the link to the project?

Have you continued committing code or opening issues to the GSoC project after GSoC? (Yes / No) If so, code, issues or both?

Have you continued committing code or opening issues to other OSS projects after GSoC? (Yes / No) If so, code, issues or both?

Could we use aggregate data from your participation in GSoC and other OSS projects in our research?

The following link contains a sample of our work and how we would use your data:
https://ieeexplore.ieee.org/document/8094410/
Regards,
Jefferson

Unrelated and outdated!


pluck('id') doesn't work

Model gives me fine collection results
$this->subcategory->search('query');

But when I use pluck on it, it throws me this error, am I missing anything here?
$this->subcategory->search('query')->pluck('id');

ErrorException in ElasticquentResultCollection.php line 22:
Missing argument 2 for Elasticquent\ElasticquentResultCollection::__construct(), called in /usr/share/nginx/zapstore/vendor/laravel/framework/src/Illuminate/Support/Collection.php on line 444 and defined

Will have a look here https://github.com/elasticquent/Elasticquent/issues/6

What version of Elastiquent are you on, @kevincobain2000?

1.0.3

Possibly related to #47

I get the same error with
$this->subcategory->search('query')->only('id');
and
$this->subcategory->search('query')->except('id');

@helloiamlukas What version of Elasticquent are you using?

@chuangbo 1.0.6

@helloiamlukas pluck fixed at #59, which hasn't been published and only available at dev-master.

@chuangbo just updated to dev-master: pluck works fine and no exception thrown for only and except
but still only returns an empty collection and except has no effect on the given collection


Does not work at all in V10

Just letting you know. I'm super grateful you keep this module working. It's the only character sheet i can stand. If i had the skills to take it over for you i would

I can confirm. Does not show up as a Character Sheet option in v10. Hope someone picks up this project, my players loved using it.

Yes, my players and I have been using this after transferring from roll20 early this year. I find all the other available character sheets to be too spread out with their categories and information tabs. My players and I appreciate the work you've done on this module, and I hope it'll become available for v10.

#40 in case anyone is needing this, check the releases of the repository for the pull request.

I had this module working on Foundry Version 9, after updating to the latest patch of Version 10, the OGL sheet no longer appears as a sheet option. I have attempted deactivating every module but this sheet, and it still does not appear, so I think it is something with Version 10. Thank you for continuing the work on this, hope it can get working again!

I would love to see this working with the new version of foundry it looks like they have landed on a stable build if that helps for development. I hope they come back to this project and update it a lot of my group loves the old school sheets, and hates the new ones lol. Thank you devs in advance for your help.

From what I can tell, the code in master includes a fix for V10. I even saw a message from the author saying it's fixed for v10. Still doesn't show.
The manifest.json still references the most latest release on github, which does not include the fix. If @akrigline will create a new release using the latest master branch, then the manifest.json can be updated and the fix applied. At least that's what I suspect at the moment.

Thank you for the reply, I took a look at the fix and must admit coding skills suck, I hope that we can get a new release soon. Thank you again for your reply.
ThanksGreg
On Saturday, January 7, 2023 at 08:07:01 PM EST, André Scott @.***> wrote:
From what I can tell, the code in master includes a fix for V10. I even saw a message from the author saying it's fixed for v10. Still doesn't show.
The manifest.json still references the most latest release on github, which does not include the fix. If @akrigline will create a new release using the latest master branch, then the manifest.json can be updated and the fix applied. At least that's what I suspect at the moment.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you commented.Message ID: @.***>

does anyone know if the proposed changes in #40  fix the issues and allow running this on v10?


Replace float arrays over the wire with integer coded arrays

Reviewers
Functional:
Readability:

Changes

added a new flatbuffer type for encoding floats as integer for smaller over-the-wire memory footprints

[ ] Add unit tests for new flatbuffer type for varying number of bins.

One overall readability concern is that it's hard to tell what client-requested data is getting encoded and what the bin counts are. Mostly because the reader has to follow the call hierarchy to see where the bin counts are specified. I don't think it's a huge deal, but maybe it's worth documenting at the API level which API calls are affected by this new encoding.

Great point. I think I'm going to add the bin counts as an optional parameter to the get requests for layout/obs, data/var and summarize/var requests. If not specified, no encoding is used. If specified, we use the encoding. That way, default value don't need to be hardcoded in two separate places (BE and FE) and the BE makes no assumptions about how the client wants its data returned.

@atolopko-czi I decided to go through with it and build it into the API. On the client side, I needed to add an optional nBins argument to the AnnoMatrix.fetch interface and downstream functions. I initially wanted to create a more complex query type for obs, var, and emb queries that would include nBins and leave the fetch interface unchanged, but that started getting even more complicated.


Cant operate device

Hello, I cant operate my HS105, I get an error that:
Microsoft.CSharp.RuntimeBinder.RuntimeBinderException: ''Newtonsoft.Json.Linq.JValue' does not contain a definition for 'model'' on line 67 of TPlinkDiscovery

Could you provide a full text dump for the sys_info JSON-object and/or the contents of the message variable?

How would I send a text dump?

Can anyone help with this?

Bump! Anyone have this error before?

Bump! Anyone have this error before?

 Dude this is preventing me from finishing my project. I hope they help us soon. I hope we get help soon.

Bump! Anyone have this error before?

 Dude this is preventing me from finishing my project. I hope they help us soon. I hope we get help soon.

Also, I am not having it in my code. I am having it on execution.

Bump! Anyone have this error before?

Just use this. This is based off the one your having issues with. Will fix the issues.
https://www.nuget.org/packages/tplink-smartdevices/

I unfortunately no longer use this series of devices in my smart home so I don't have anything to test against. I'm happy to take a pull request, but I see the alternative library is pretty actively maintained; I'm thrilled to see someone has made good use of this as a base and has since picked this up in the meantime, though! :)

Bump! Anyone have this error before?

Just use this. This is based off the one your having issues with. Will fix the issues.
https://www.nuget.org/packages/tplink-smartdevices/

that new API works, thank you!


Not working with rich text editor fields

Hi, first of all, thank you very much for the handy plugin. It is really useful and I found it from a stack overflow answer.
My form has a rich text editor fields (CK Editors). Your plugin can detect the dirtiness of all other fields but those rich text editors. Is there any attributes to be set to make it work with rich text editors or it just isn't implemented? If it an unimplemented feature, is there a work around?
Thanks!

After CKeditor is initialised, you need to add an event:
// add change event to set is dirty on the form
editor.model.document.on('change:data', () => {
    // find closest form to the textarea element
    element.closest('form').dirty('setAsDirty');
});


Cleanup

Optimizations and cleanup:

replace local constants with global constants
remove unnecessary declarations
create adapter for util method

Codecov Report

Merging #304 into develop will increase coverage by 0.24%.
The diff coverage is 1.88%.

@@              Coverage Diff              @@
##             develop     #304      +/-   ##
=============================================
+ Coverage      19.38%   19.63%   +0.24%
+ Complexity       289      288       -1
=============================================
  Files             91       95       +4
  Lines           3637     3586      -51
  Branches         566      553      -13
=============================================
- Hits             705      704       -1
+ Misses          2857     2807      -50
  Partials          75       75

Impacted Files
Coverage Δ
Complexity Δ

.../dubbo/admin/controller/LoadBalanceController.java
84.21% <ø> (ø)
14 <0> (ø)
:arrow_down:

.../apache/dubbo/admin/common/util/OverrideUtils.java
0% <ø> (ø)
0 <0> (ø)
:arrow_down:

...che/dubbo/admin/controller/AccessesController.java
88.88% <ø> (ø)
14 <0> (ø)
:arrow_down:

...o/admin/model/adapter/WeightToOverrideAdapter.java
0% <0%> (ø)
0 <0> (?)

.../org/apache/dubbo/admin/DubboAdminApplication.java
33.33% <0%> (ø)
1 <0> (ø)
:arrow_down:

.../dubbo/admin/service/impl/OverrideServiceImpl.java
0.57% <0%> (ø)
1 <0> (ø)
:arrow_down:

...el/adapter/BalancingDTO2OverrideConfigAdapter.java
0% <0%> (ø)
0 <0> (?)

...min/model/adapter/LoadBalance2OverrideAdapter.java
0% <0%> (ø)
0 <0> (?)

...l/adapter/DynamicConfigDTO2OverrideDTOAdapter.java
0% <0%> (ø)
0 <0> (?)

...va/org/apache/dubbo/admin/config/ConfigCenter.java
92.3% <100%> (-0.15%)
17 <0> (ø)

... and 4 more

Continue to review full report at Codecov.

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


Add CI workflow

There are two workflows:

Test code base when PR is created
Release code to NPM repo when merged to main branch

:robot:
Changes in the patch:

Added a new file .github/workflows/pr.yml with 38 lines of code.
Added a new file .github/workflows/release.yml with 23 lines of code.
Modified package-lock.json file with 670 insertions and 798 deletions.
Modified package.json file with 7 changes.

Analysis:

The addition of .github/workflows/pr.yml and .github/workflows/release.yml files suggests the addition of Continuous Integration (CI) workflows for testing the codebase when a pull request is created and releasing the code to the NPM repository when merged to the main branch. This is a positive change as it improves the development process by automating testing and release tasks.

The modifications in package-lock.json indicate changes in the dependencies of the project. The version of the package @opentelemetry/api has been updated from ^1.3.0 to ^1.4.1. This suggests that there might be bug fixes or new features in the updated version. However, it is important to review the release notes or documentation of the updated package to ensure compatibility and avoid potential issues.

The modifications in package.json are not specified in the patch, so it is unclear what changes have been made. It is recommended to review the changes in package.json to ensure that the dependencies and scripts are correctly updated.

Optimizations:

The workflows defined in the .github/workflows directory can be optimized by using matrix strategies. Currently, the test job in pr.yml runs on three different versions of Node.js (18, 19, and 20). Instead of defining separate steps for each version, a matrix strategy can be used to run the steps in parallel for all specified versions. This can improve the efficiency of the workflow and reduce the execution time.

Potential Problems:

The modifications in package-lock.json suggest changes in the dependencies of the project. It is important to review these changes and ensure that they do not introduce any compatibility issues or breaking changes. It is recommended to thoroughly test the project after these changes to ensure its stability.

The modifications in package.json are not specified in the patch, so it is unclear what changes have been made. It is important to review these changes and ensure that they are correct and do not introduce any issues.

Overall, the addition of CI workflows and updates to dependencies can improve the development process and stability of the project. However, it is important to review the changes in package-lock.json and package.json to ensure compatibility and avoid potential problems.
> gpt-3.5-turbo-16k

:robot:
Summary of changes:

Added CI workflows:

Created .github/workflows/pr.yml file for testing the code base when a pull request is created.
Created .github/workflows/release.yml file for releasing the code to the NPM repository when merged to the main branch.

Modified package-lock.json:

Updated the version of @opentelemetry/api from "^1.3.0" to "^1.4.1".
Updated the version of @opentelemetry/exporter-collector-grpc from "^0.25.0" to the latest version.
Updated the version of @opentelemetry/instrumentation from "^0.40.0" to the latest version.
Updated the version of @opentelemetry/instrumentation-aws-sdk from "^0.34.3" to the latest version.
Updated the version of @opentelemetry/semantic-conventions from "^1.14.0" to the latest version.
Removed the devDependency on mocha.

Optimizations:

The package-lock.json file should be regenerated to reflect the updated dependencies.

Potential problems:

The workflows defined in .github/workflows/pr.yml and .github/workflows/release.yml may have incorrect or missing steps. A thorough review of the workflows is required to ensure they are correctly configured.

Example of optimization:

Instead of manually updating the package-lock.json file, a package manager like npm or yarn can be used to update the dependencies automatically. For example, running npm update will update the dependencies in the package-lock.json file to their latest compatible versions.

> gpt-3.5-turbo-16k

:robot:
Summary of changes:

Added CI workflows:

Created .github/workflows/pr.yml file for testing the code base when a pull request is created.
Created .github/workflows/release.yml file for releasing the code to the NPM repository when merged to the main branch.

Modified package-lock.json:

Updated the version of @opentelemetry/api from "^1.3.0" to "^1.4.1".
Updated the version of @opentelemetry/exporter-collector-grpc from "^0.25.0" to the latest version.
Updated the version of @opentelemetry/instrumentation from "^0.40.0" to the latest version.
Updated the version of @opentelemetry/instrumentation-aws-sdk from "^0.34.3" to the latest version.
Updated the version of @opentelemetry/semantic-conventions from "^1.14.0" to the latest version.
Removed the devDependency on mocha.

Optimizations:

The package-lock.json file should be regenerated to reflect the updated dependencies.

Potential problems:

The workflows defined in .github/workflows/pr.yml and .github/workflows/release.yml may have incorrect or missing steps. A thorough review of the workflows is required to ensure they are correctly configured.

Example of optimization:

Instead of manually updating the dependencies in package-lock.json, a package manager like npm or yarn can be used to automatically update the dependencies based on the version ranges specified in package.json. This can help ensure that the dependencies are always up to date without manual intervention.

> gpt-3.5-turbo-16k

:robot:
Summary of changes:

Added CI workflows:

Created .github/workflows/pr.yml file for testing the code base when a pull request is created.
Created .github/workflows/release.yml file for releasing the code to the NPM repository when merged to the main branch.

Modified package-lock.json:

Updated the version of @opentelemetry/api from "^1.3.0" to "^1.4.1".
Updated the version of @opentelemetry/exporter-collector-grpc from "^0.25.0" to the latest version.
Updated the version of @opentelemetry/instrumentation from "^0.40.0" to the latest version.
Updated the version of @opentelemetry/instrumentation-aws-sdk from "^0.34.3" to the latest version.
Updated the version of @opentelemetry/semantic-conventions from "^1.14.0" to the latest version.
Removed the devDependency on mocha.

Optimizations:

The package-lock.json file should be regenerated to reflect the updated dependencies.

Potential problems:

The workflows defined in .github/workflows/pr.yml and .github/workflows/release.yml may have incorrect or missing steps. A thorough review of the workflows is required to ensure they are correctly configured.

Example of optimization:

Instead of manually updating the package-lock.json file, a package manager like npm or yarn can be used to update the dependencies automatically. For example, running npm update will update the dependencies in the package-lock.json file to their latest compatible versions.

> gpt-3.5-turbo-16k

:robot:
Summary of changes:

Added CI workflows:

Created .github/workflows/pr.yml file for testing the code base when a pull request is created.
Created .github/workflows/release.yml file for releasing the code to the NPM repository when merged to the main branch.

Modified package-lock.json:

Updated the version of @opentelemetry/api from "^1.3.0" to "^1.4.1".
Updated the version of @opentelemetry/exporter-collector-grpc from "^0.25.0" to the latest version.
Updated the version of @opentelemetry/instrumentation from "^0.40.0" to the latest version.
Updated the version of @opentelemetry/instrumentation-aws-sdk from "^0.34.3" to the latest version.
Updated the version of @opentelemetry/semantic-conventions from "^1.14.0" to the latest version.
Removed the devDependency on mocha.

Optimizations:

The package-lock.json file should be regenerated to reflect the updated dependencies.

Potential problems:

The workflows defined in .github/workflows/pr.yml and .github/workflows/release.yml may have incorrect or missing steps. A thorough review of the workflows is required to ensure they are correctly configured.

Example of optimization:

Instead of manually updating the dependencies in package-lock.json, a package manager like npm or yarn can be used to automatically update the dependencies based on the version ranges specified in package.json. This can help ensure that the dependencies are always up to date without manual intervention.

> gpt-3.5-turbo-16k

:robot:
Summary of changes:

Added CI workflows:

Created .github/workflows/pr.yml file for testing the code base when a pull request is created.
Created .github/workflows/release.yml file for releasing the code to the NPM repository when merged to the main branch.

Modified package-lock.json:

Updated the version of @opentelemetry/api from "^1.3.0" to "^1.4.1".
Updated the version of @opentelemetry/exporter-collector-grpc from "^0.25.0" to the latest version.
Updated the version of @opentelemetry/instrumentation from "^0.40.0" to the latest version.
Updated the version of @opentelemetry/instrumentation-aws-sdk from "^0.34.3" to the latest version.
Updated the version of @opentelemetry/semantic-conventions from "^1.14.0" to the latest version.
Removed the devDependency on mocha.

Optimizations:

The package-lock.json file should be regenerated to reflect the updated dependencies.

Potential problems:

The workflows defined in .github/workflows/pr.yml and .github/workflows/release.yml may have incorrect or missing steps. A thorough review of the workflows is required to ensure they are correctly configured.

Example of optimization:

Instead of manually updating the dependencies in package-lock.json, a package manager like npm or yarn can be used to automatically update the dependencies based on the version ranges specified in package.json. This can help ensure that the dependencies are always up to date without manual intervention.

> gpt-3.5-turbo-16k


Listen to UDP broadcast traffic

Is this a bug report or a feature request?:
not sure
What happened:
I run a server inside a Raspberry PI Cluster that listens to UDP broadcast traffic on a specific port to discover clients. This does not work. I'm not sure if I just misconfigured something or if this is an actual  bug.
What you expected to happen:
I would expect that UDP Broadcast traffic to be routed to the service pod.
How to reproduce it (as minimally and precisely as possible):
My service description:
apiVersion: v1
kind: Service
metadata:
  name: lms-web
  namespace: lms
  annotations:
    metallb.universe.tf/allow-shared-ip: lms
spec:
  type: LoadBalancer
  ports:
  - name: http
    protocol: TCP
    port: 9000
    targetPort: 9000
  selector:
    app: lms

---
apiVersion: v1
kind: Service
metadata:
  name: lms-cli
  namespace: lms
  annotations:
    metallb.universe.tf/allow-shared-ip: lms
spec:
  type: LoadBalancer
  ports:
  - name: cli
    protocol: TCP
    port: 9090
    targetPort: 9090
  selector:
    app: lms

---
apiVersion: v1
kind: Service
metadata:
  name: lms-control-udp
  namespace: lms
  annotations:
    metallb.universe.tf/allow-shared-ip: lms
spec:
  type: LoadBalancer
  ports:
  - name: control
    protocol: UDP
    port: 3483
    targetPort: 3483
  selector:
    app: lms

---
apiVersion: v1
kind: Service
metadata:
  name: lms-control-tcp
  namespace: lms
  annotations:
    metallb.universe.tf/allow-shared-ip: lms
spec:
  type: LoadBalancer
  ports:
  - name: control
    protocol: TCP
    port: 3483
    targetPort: 3483
  selector:
    app: lms

My metallb configmap:
kind: ConfigMap
metadata:
  namespace: metallb-system
  name: config
data:
  config: |
    address-pools:
    - name: my-ip-space
      protocol: layer2
      addresses:
      - <IP_ADDRESS>-<IP_ADDRESS>

Anything else we need to know?:
If it helps, I'm trying to run a Logitech Media Server on Kubernetes which needs the UDP traffic to discover its payers.
Environment:

MetalLB version: 0.7.3
Kubernetes version: v1.11.1
OS (e.g. from /etc/os-release):

PRETTY_NAME="Raspbian GNU/Linux 9 (stretch)"
NAME="Raspbian GNU/Linux"
VERSION_ID="9"
VERSION="9 (stretch)"
ID=raspbian
ID_LIKE=debian
HOME_URL="http://www.raspbian.org/"
SUPPORT_URL="http://www.raspbian.org/RaspbianForums"
BUG_REPORT_URL="http://www.raspbian.org/RaspbianBugs"
HYPRIOT_OS="HypriotOS/armhf"
HYPRIOT_OS_VERSION="v2.0.1"
HYPRIOT_DEVICE="Raspberry Pi"
HYPRIOT_IMAGE_VERSION="v1.9.0"

Kernel (e.g. uname -a):
Linux k8s-node04 4.14.34-hypriotos-v7+ #1 SMP Sun Apr 22 14:57:31 UTC 2018 armv7l GNU/Linux

Kubernetes's LoadBalancer logic (which MetalLB relies on) does not support forwarding broadcast traffic into pods, so this simply will not work in k8s, sorry. The only option you have is to run the pods with hostNetwork=true and try to wrangle something with that, but you won't get stable IPs with that setup.
Nothing I can do about this, it's a shortcoming burned into k8s.

@danderson Do you mind elaborating on that? I think this its worth making a feature request to kubernetes so I want to make sure I understand the problem before I propose a solution

@danderson I'd also like to follow up the main k8s issue with this, can you please point us to any existing issue which documents this limitation ?