text:
Expose reference count increment

Original message from Xuezhao:

Just to discuss with you about one small issue about HG internal refcount.
Now HG_Get_input implemented as  calling hg_core_incr_ref, HG_Core_get_input and some other core APIs.
As in Cart we added an internal header above HG and we want to unpack the RPC request header first and then unpack the boy when serving a RPC request, so we use the HG_Core_ APIs.
One problem is that the HG_Core_get_input does not take refcount, so we will meet a problem later when calling HG_Free_input/HG_Destroy as it will cause refcount issue.
We could have some hacking in CaRT to avoid the refcount issue, at the other hand do you think does it reasonable to move the calling of hg_core_incr_ref to be inside HG_Core_get_input?
(one related change is adding a hg_core_destroy to hg_set_input, it seems reasonable as it can protect the handle be valid during hg_set_input).

It seems that exposing the reference count increment function would just solve the issue.

@liuxuezhao Let me know if you have any more problems.

@soumagne thanks very much, it looks good, I'll make some change to cart to use it.

authors: liuxuezhao soumagne

repo: mercury-hpc/mercury


text:
Admin > Countries: Disaggregation categories appear editable after they are in use

As a rule, a disaggregation category should not be editable after there is result data associated with it. Today, categories appear to remain editable after they are in use.
Steps to reproduce

In Admin > Countries > Tolaland > Country disaggregations, add a new category any of the existing disaggregations.

Submit a new result for any Tolaland program, entering data into the new disaggregation category you just created.

Navigate back to the admin page where you added the category.

Expected: Category text field is inactive.
Observed: It is active. You can edit the text and submit it, and see a green success message. In fact, the changes are NOT saved. It's all lies.
Acceptance criteria

[x] 1. When a disaggregation category is in use by at least one saved result, you cannot edit its text in Admin > Country. The text field is inactive.

It's probably just a UI store setting somewhere that got missed.

@camgnostic It looks like ALL categories become non-editable after any category is used in any result. My intention was that the specific category used would become non-editable and non-deletable, not that ALL categories for the entire disaggregation be non-editable and non-deletable.

Working as expected. After I disaggregate a result, the categories I used are no longer editable in the Admin. I also see a lock icon with a tool-tip explaining why.

Couple of UI tweaks:

Vertically center-align lock icon with number menus.

Reduce gap between lock icon and tool tip.

Looks good!
Here's what I'm seeing now -- everything in alignment, no gap.

source: gharchive/issue

authors: jennyhavah sanjuroj

repo: mercycorps/TolaActivity

license_type: permissive

license_source: bigquery


text:
Single tile for Property Control

Hello, I would like to change between values of property within single tile (expected is Light tile, not Switch tile) instead of having multiple tiles with one value. Maybe it is already possible but my programming skills are none.
I have mijia.vacuum.v2 and if I use Property Control for “vacuum:fan-level”property it shows as many tiles as possible Values for that property. In this case 4 values, therefore Homebridge creates 4 new tiles for each option, instead I would like to use 1 tile showed as Light, maybe somewhere I can manually specify range of values from 0-4. This way maybe I could control suction power from single tile. This way I can create separated tile for Water releasing when mopping.
Can we use something like Action Button and we would specify range of values? I have tried to use this property for Action Button, Property Control and Property monitor but none of them result in expected state. I can reach expected result with different plugin but I would like to use this one, because it is more stable. I have red this page and it was not useful for my case https://github.com/merdok/homebridge-miot/blob/main/docs/robotcleaner.md

The problem here is that, how do you display 4 values on a switch which only accepts 2 values(on/off)? Even if would let you to specify that you want to set mode 1 to the ON state and mode 2 to OFF state, then what would the switch display in case when mode 3 is set on the vacuum?
This is unfortunately not an easy to solve problem because HomeKit can only have switches with 2 states.
But there is a way to reduce the number of switches, simply disable modeControl and create yourself switches using the propertyControl. There you can create switches which when turned on will set a specified value, hence you could only add those switches which you really need. Will that work for you?

You are right, I assume it cannot be Switch tile but Light or Fan tile. This way there is slider on a tile which can output number and this number changes the property value for "vacuum:fan-level". But it needs to be somehow handled properly because this specific property does not support values from 0-100 but only from 0-4 included (0 means fan is off and 4 is fan is on maximum). In video is expected result how it behaves in the other plugin, we can see that the Fan slider is able to do steps of 25, the number is visible on HomeBridge tiles on right side of video.
https://user-images.githubusercontent.com/7964597/189893267-05ee6df8-ebc5-433f-9711-406d5c70bb07.mov

You can specify a value in propertyControl which will then generate you a single switch for the specified value. When you not specify a value then it will generate all switches. Please have a look at the example in the README.
As for the slider, it is fine for speed based controls and it is already like that, that the plugin will emulate fan levels with a slider.
But that does not make sense for modes since then you do not know what mode is behind the 50% value and what is behind 75%.

yep exactly how you said, for modes generally it does not make sense to be controlled with slider but it makes sense for modes which control speed/power in levels (as in my case).

Fan levels are already emulated as a slider if present.
For modes you simply need to use the the propertyControl property and configure it yourself as you wish.

Thats correct, not using all of the values per single vacuuming but definitely from time to time I need all of the values. Having 1 or 2 Switch tiles for most used values are OK solution but not satisfying enough because it is not ready for all of the values. Also having dedicated Switch tiles for all values for Fan on vacuum would mean 4 new tiles, and for Water it would mean 3 tiles. It means users Accessory list bloats bigger for 7 new tiles while there could be only 2 new tiles, thats why I started this issue.
But no worry, I will figure out smtn.
Thanks for clarification. You can close this issue.

The biggest issue is here that homekit does not support vacuum accessories or many configuration possibilities, hence we have to do tricks to make all the functionally to work...

authors: lastrebel merdok

repo: merdok/homebridge-miot

license: MIT

license_source: github-api


text:
The devices seem to be missing features

I have a few devices and some buttons etc. seem to be missing. how can I add them? Some features come automatically, but still many things seem to be missing.
my devices :
yeelink.light.color3
dmaker.fan.p39
roborock.vacuum.a27
zhimi.heater.zb1a
zhimi.airp.vb4
for example : https://www.merdok.org/miotspec/
this address shows more features for my devices, while the home app doesn't have as many features and buttons.

You can use the propertyControl and propertyMonitor config properties to expose anything that you want. You can have a look at the examples here:
https://github.com/merdok/homebridge-miot?tab=readme-ov-file#configuration

Yes, that's exactly what I wanted to ask. how do I know the settings here and how do I adjust them? a lot of people get stuck on this. The explanation in the example is not very clear for beginners, to be honest.

You just go to the mentioned website https://www.merdok.org/miotspec/ type in your device and then you just take the property name or id and put it in the configs as in the example.

You just go to the mentioned website https://www.merdok.org/miotspec/ type in your device and then you just take the property name or id and put it in the configs as in the example.

First of all, thank you very much for your answers, but I am very new to these things and it seems really complicated. Actually, as far as I understand, all the features of the device are already specified on https://www.merdok.org/miotspec/. Is there any way to convert them to config automatically? When we export it, it will convert it to config and we can edit the properties related to the device ourselves. Like IP, etc.

No, there is no way to convert them. You simply go to the website pick which properties you would like and put them in your config, and that's it. The example in the README shows pretty clearly how they should be added, there is also a description to the config properties if you need more info.

I tried but it's not working.

Then you have to be more specific what is nothing and post your config.

Okay, I solved the problem.
But there is a small issue left.for example, I want to add fan:horizontal-angle for my dmaker.fan.p39 device, but it adds it as toggle. I want to add two arrows that trigger a fan. As I press left, the angle decreases and the fan is observed as a decrease in the fan. Likewise, when I press the right arrow, the angle increases and the fan indicator increases. is such a thing possible?

Unfortunately that is not possible, HomeKit only allows limited control possibilities, in order to overcome that issues some tricks are necessary. So if you want to change the horizontal angle then you would need to add a button for your favorite angles. Also you can try to use the fan:motor-control property on your device, which might be eventually what you are looking for?
The toggle name i fetched from the miot spec, if necessary then you can override it by specifying your own name in the config as per example.

Unfortunately that is not possible, HomeKit only allows limited control possibilities, in order to overcome that issues some tricks are necessary. So if you want to change the horizontal angle then you would need to add a button for your favorite angles. Also you can try to use the fan:motor-control property on your device, which might be eventually what you are looking for? The toggle name i fetched from the miot spec, if necessary then you can override it by specifying your own name in the config as per example.

Thanks a lot, fan:motor-control this is what I was looking for, I wish it was like button or smtn not toggle but better than anything. Also I can give Name for my own additions, but I am not sure how to add them in the config since I do not know the ids of the default ones.

The default ones you unfortunately cannot change, they are being read out from the spec. You could try to modify the spec file which is cached on your device but that would be something rather complex i would say.

The default ones you unfortunately cannot change, they are being read out from the spec. You could try to modify the spec file which is cached on your device but that would be something rather complex i would say.

So I should edit them when its back to original ?

Whenever you restart homebridge you will need to edit them currently, so i guess during your config modifications don't edit them until you are done.

Whenever you restart homebridge you will need to edit them currently, so i guess during your config modifications don't edit them until you are done.

Thanks a lot for all your effort and helps


text:
Wake Up On LIne

HI All,
I installed the plugin.
I can turn off my LG TV. But I can't turn on it.
I have WebOS 2 Firmware 03.15.55
ON my log I found
Jan 30 19:20:32 master homebridge[26782]: [2019-1-30 19:20:32] [My webOS tv] webOS - Error: connect ECONNREFUSED 192.168.123.36:3000
Can anyone help me?

Hi,
I try to send a magic packet from some computer but nothing to do.
Also, I have installed the official app of LG WebOS and I can't turn on my TV. So I think is not possible to do.

Ok, well in that case your TV does not support it...

repo: merdok/homebridge-webos-tv


text:
error: require is not a function

getting this error after successful npm install and starting homebridge:
(running on node v11.9.0)
[7/9/2019, 9:08:52 AM] ====================
[7/9/2019, 9:08:52 AM] ERROR LOADING PLUGIN homebridge-webos-tv:
[7/9/2019, 9:08:52 AM] TypeError: require(...) is not a function
    at Object.<anonymous> (/usr/local/lib/node_modules/homebridge-webos-tv/node_modules/websocket/lib/WebSocketServer.js:20:29)
    at Module._compile (internal/modules/cjs/loader.js:734:30)
    at Object.Module._extensions..js (internal/modules/cjs/loader.js:745:10)
    at Module.load (/usr/local/lib/node_modules/homebridge-broadlink-rm/node_modules/coffeescript/lib/coffee-script/register.js:45:36)
    at tryModuleLoad (internal/modules/cjs/loader.js:566:12)
    at Function.Module._load (internal/modules/cjs/loader.js:558:3)
    at Module.require (internal/modules/cjs/loader.js:663:17)
    at require (internal/modules/cjs/helpers.js:20:18)
    at Object.<anonymous> (/usr/local/lib/node_modules/homebridge-webos-tv/node_modules/websocket/lib/websocket.js:2:22)
    at Module._compile (internal/modules/cjs/loader.js:734:30)
    at Object.Module._extensions..js (internal/modules/cjs/loader.js:745:10)
    at Module.load (/usr/local/lib/node_modules/homebridge-broadlink-rm/node_modules/coffeescript/lib/coffee-script/register.js:45:36)
    at tryModuleLoad (internal/modules/cjs/loader.js:566:12)
    at Function.Module._load (internal/modules/cjs/loader.js:558:3)
    at Module.require (internal/modules/cjs/loader.js:663:17)
    at require (internal/modules/cjs/helpers.js:20:18)
    at Object.<anonymous> (/usr/local/lib/node_modules/homebridge-webos-tv/node_modules/websocket/index.js:1:80)
    at Module._compile (internal/modules/cjs/loader.js:734:30)
    at Object.Module._extensions..js (internal/modules/cjs/loader.js:745:10)
    at Module.load (/usr/local/lib/node_modules/homebridge-broadlink-rm/node_modules/coffeescript/lib/coffee-script/register.js:45:36)
    at tryModuleLoad (internal/modules/cjs/loader.js:566:12)
    at Function.Module._load (internal/modules/cjs/loader.js:558:3)
    at Module.require (internal/modules/cjs/loader.js:663:17)
    at require (internal/modules/cjs/helpers.js:20:18)
    at Object.<anonymous> (/usr/local/lib/node_modules/homebridge-webos-tv/node_modules/lgtv2/index.js:12:23)
    at Module._compile (internal/modules/cjs/loader.js:734:30)
    at Object.Module._extensions..js (internal/modules/cjs/loader.js:745:10)
    at Module.load (/usr/local/lib/node_modules/homebridge-broadlink-rm/node_modules/coffeescript/lib/coffee-script/register.js:45:36)
    at tryModuleLoad (internal/modules/cjs/loader.js:566:12)
    at Function.Module._load (internal/modules/cjs/loader.js:558:3)
    at Module.require (internal/modules/cjs/loader.js:663:17)
    at require (internal/modules/cjs/helpers.js:20:18)
    at Object.<anonymous> (/usr/local/lib/node_modules/homebridge-webos-tv/index.js:1:77)
    at Module._compile (internal/modules/cjs/loader.js:734:30)
    at Object.Module._extensions..js (internal/modules/cjs/loader.js:745:10)
    at Module.load (/usr/local/lib/node_modules/homebridge-broadlink-rm/node_modules/coffeescript/lib/coffee-script/register.js:45:36)
    at tryModuleLoad (internal/modules/cjs/loader.js:566:12)
    at Function.Module._load (internal/modules/cjs/loader.js:558:3)
    at Module.require (internal/modules/cjs/loader.js:663:17)
    at require (internal/modules/cjs/helpers.js:20:18)
    at Plugin.load (/usr/local/lib/node_modules/homebridge/lib/plugin.js:72:23)
    at Server.<anonymous> (/usr/local/lib/node_modules/homebridge/lib/server.js:153:14)
    at Array.forEach (<anonymous>:null:null)
    at Server._loadPlugins (/usr/local/lib/node_modules/homebridge/lib/server.js:145:22)
    at new Server (/usr/local/lib/node_modules/homebridge/lib/server.js:57:24)
    at module.exports (/usr/local/lib/node_modules/homebridge/lib/cli.js:32:16)
    at Object.<anonymous> (/usr/local/lib/node_modules/homebridge/bin/homebridge:17:22)
    at Module._compile (internal/modules/cjs/loader.js:734:30)
    at Object.Module._extensions..js (internal/modules/cjs/loader.js:745:10)
    at Module.load (internal/modules/cjs/loader.js:626:32)
    at tryModuleLoad (internal/modules/cjs/loader.js:566:12)
    at Function.Module._load (internal/modules/cjs/loader.js:558:3)
    at Function.Module.runMain (internal/modules/cjs/loader.js:797:12)
    at executeUserCode (internal/bootstrap/node.js:526:15)
    at startMainThreadExecution (internal/bootstrap/node.js:439:3)

[7/9/2019, 9:08:52 AM] ====================

Honestly i do not know why you are getting this error. Maybe try to remover your node_modules directory and try to to install your packages again.

removing mode_modules and reinstalling fixed the issue

authors: InteNs merdok


text:
Ability to apply an action to a set of VMs

I.e. start instances corresponding to some SQL query, e.g. all instances with state = shutoff.

Possible solution:
\L - output as list (one by line)
\E - output as enumeration (comma separated)
E.g.:
perspective>select name from instances limit 10\E

repo: meridor/perspective-backend


text:
[UI][Catalog] HTML Entities displaying inappropriately

Screenshots/Logs

Environment

Browser: Chrome Safari Firefox
Host OS: Mac Linux Windows
Meshery Server Version: stable-v
Platform: Docker Kubernetes

Contributor Guides and Handbook

🎨 Wireframes and designs for Meshery UI in Figma (open invite)
🖥 Contributing to Meshery UI
🙋🏾🙋🏼 Questions: Discussion Forum and Community Slack

This issue is now fixed

repo: meshery/meshery


text:
fix: failing golint-ci test for mesheryctl

Notes for Reviewers
This PR fixes #

golangci-lint checks (example: https://github.com/meshery/meshery/actions/runs/7993330368/job/21828900989?pr=10365)

Desired Result:

Signed commits

[x] Yes, I signed my commits.

No more undefined error messages from docs_test.go.

@alphaX86 does this look good to you?

source: gharchive/pull-request

authors: RipulHandoo leecalcote


text:
Update to providers doc clarifying URLs

Notes for Reviewers
This PR primarily updates the configurable-oauth-callback-url section of the document clarifying Custom URL and provider URL. It also has corrections to sentence, grammar etc.
I have also updated the provider selection screenshot.
This was requested by Lee in slack.
Signed commits

[x] Yes, I signed my commits.

🚀 Preview for commit 335831f4962608698644dce9c49f856a8c65af94 at: https://660a59eeb18ed865cf608aaf--meshery-docs-preview.netlify.app

@sangramrath Let's discuss this in Meshery dev call today. I'm adding this as an agenda item to the meeting minutes.

Thanks @leecalcote I will make the required changes.
@vishalvivekm I am unable to attend the call today but do leave me a comment of any other changes that are required if this is discussed on the call.

🚀 Preview for commit 59a606fb8661b6aefdb18660769733f2d2ab4e6c at: https://66139ae419634e637218b90f--meshery-docs-preview.netlify.app

🚀 Preview for commit d0525636b3575d8707a51d66b8704517eb0382f1 at: https://66139e94914aad663689c81e--meshery-docs-preview.netlify.app

authors: l5io sangramrath vishalvivekm


text:
[mesheryctl]add subcommands for system config command

Description
mesheryctl system config command throws an error when config type of either aks/eks/gke/minikube is not specified.
changed the mesheryctl system config to have subcommands for each config type.
Also removed the dependency on users providing auth token as input, these subcommands try to fetch the auth token from constants.GetCurrentAuthToken() if its not specified on the CLI
This PR fixes #3758 #3282
Notes for Reviewers
Tested the refactored peices, e2e testing has not been performed on all the platforms.
Signed commits

[ ] Yes, I signed my commits.

Codecov Report

Merging #4183 (3d03789) into master (2227a06) will decrease coverage by 7.43%.
The diff coverage is 11.61%.

:exclamation: Current head 3d03789 differs from pull request most recent head 6d3c410. Consider uploading reports for the commit 6d3c410 to get more accurate results

@@            Coverage Diff             @@
##           master    #4183      +/-   ##
==========================================
- Coverage   26.91%   19.47%   -7.44%
==========================================
  Files          59       59
  Lines        5031     5063      +32
==========================================
- Hits         1354      986     -368
- Misses       3262     3779     +517
+ Partials      415      298     -117

Flag
Coverage Δ

gointegrationtests
?

unittests
19.47% <11.61%> (+0.29%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

mesheryctl/internal/cli/root/system/config.go
8.61% <11.61%> (+7.48%)
:arrow_up:

mesheryctl/internal/cli/root/system/status.go
0.00% <0.00%> (-62.83%)
:arrow_down:

mesheryctl/internal/cli/root/system/start.go
0.99% <0.00%> (-53.98%)
:arrow_down:

mesheryctl/internal/cli/root/system/stop.go
0.85% <0.00%> (-36.76%)
:arrow_down:

mesheryctl/internal/cli/root/system/update.go
1.05% <0.00%> (-34.74%)
:arrow_down:

mesheryctl/internal/cli/root/system/restart.go
1.78% <0.00%> (-32.15%)
:arrow_down:

mesheryctl/internal/cli/root/system/check.go
1.39% <0.00%> (-22.38%)
:arrow_down:

mesheryctl/internal/cli/root/system/logs.go
0.00% <0.00%> (-21.60%)
:arrow_down:

mesheryctl/internal/cli/root/config/config.go
12.79% <0.00%> (+2.90%)
:arrow_up:

Continue to review full report at Codecov.

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

@hexxdump There is a lint error.

Wouldn't adding each of the platforms as sub commands cause a lot of overhead?
When we support a new platform in the future, instead of adding a new case in a switch statement, we have to create a new command.
Also we would need to ensure parity between all these subcommands and this can become a bigger task than it needs to be.
Instead of new subcommands, can we just show a custom error message showing the available platforms? It would provide the same UX as far as usability and documentation is our concern.
Am I missing any other detail on why subcommands are needed?

@navendu-pottekkat, Going forward, it may not make sense to continue with a single config command and switch cases beneath it. We need to migrate to go-client SDKs for each platform. It would bloat each switch case and the parent configCmd's code(could become a big monolithic function).

Okay. Got it.

@alphaX86 it would be good reflect on how this changes the docs in the mesheryctl reference guide. @hexxdump, can we include a docs update in this PR? https://github.com/meshery/meshery/blob/master/docs/_data/mesheryctlcommands/cmds.yml

@alphaX86 it would be good reflect on how this changes the docs in the mesheryctl reference guide. @hexxdump, can we include a docs update in this PR? https://github.com/meshery/meshery/blob/master/docs/_data/mesheryctlcommands/cmds.yml

Sounds good to me @leecalcote, let's wait for @hexxdump reply

@alphaX86 it would be good reflect on how this changes the docs in the mesheryctl reference guide. @hexxdump, can we include a docs update in this PR? https://github.com/meshery/meshery/blob/master/docs/_data/mesheryctlcommands/cmds.yml

Sounds good to me @leecalcote, let's wait for @hexxdump reply

Yes, cmds.yml should change accordingly. @alphaX86 will you be able to take care of this once this PR gets merged?

@navendu-pottekkat I think @hexxdump is still working on issue 3282

I am still getting this for the perf command

You gotta login first, with mesheryctl system login

@hexxdump Are you addressing any other concerns in this PR? Aren't we ready to merge this?

@hexxdump Are you addressing any other concerns in this PR? Aren't we ready to merge this?

I am not addressing any other issues other than the ones mentioned in the description. I think we can merge this.

Thank you for clarifying. Merging this.

authors: alphaX86 codecov-commenter hexxdump leecalcote navendu-pottekkat piyushsingariya


text:
[UI] changes the color of  "X" from red to the theme color of UI

Signed-off-by: Gaurav Chadha gauravchadha1676@gmail.com
Description
changes the color of  "X" from red to the theme color of UI in :

Filters
Patterns
Applications

This PR fixes # Style
Notes for Reviewers
We don't have any issues with red but it's not in our game. :P
Signed commits

[x] Yes, I signed my commits.

@Chadha93 some screenshots would be great😅.

@Chadha93 some screenshots would be greatsweat_smile.

Sure adding with few more commits

Codecov Report

Merging #4315 (2acbb1d) into master (dac33b5) will not change coverage.
The diff coverage is n/a.

:exclamation: Current head 2acbb1d differs from pull request most recent head 9656c64. Consider uploading reports for the commit 9656c64 to get more accurate results

@@           Coverage Diff           @@
##           master    #4315   +/-   ##
=======================================
  Coverage   20.15%   20.15%
=======================================
  Files          59       59
  Lines        5037     5037
=======================================
  Hits         1015     1015
  Misses       3727     3727
  Partials      295      295

Flag
Coverage Δ

unittests
20.15% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out 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 22a3ba0...9656c64. Read the comment docs.

@Chadha93 some screenshots would be greatsweat_smile.

Added @warunicorn19

@leecalcote I don't think we have a central theme yet🤔.

Reuse elements from central theme, please.

We don't have defined central theming yet ig. // @sudo-NithishKarthik


text:
Add RJSF docs

Signed-off-by: Utkarsh Srivastava srivastavautkarsh8097@gmail.com
Description
This PR add RJSF docs.
Notes for Reviewers
Signed commits

[x] Yes, I signed my commits.

🚀 Preview for commit 8cbd2b471a0f444514bccbcea9aa42fc9ccdeb82 at: https://6197b561f11cc10071e4834a--meshery-docs-preview.netlify.app


text:
Don't cache failed getMesonTasks() result.

I don't think it should rely on changeHandler being invoked to clear the cache.

Could you explain the problem this fixes?

@lukester1975 if it fails, what hope so you have that it will succeed next time? I think only a reconfigure has a chance to fix the error potentially.

Yeah, this is mainly about UX. Caching a failed getMesonTasks means further attempts to run a task do nothing, no showErrorMessage dialog etc.
But also reasoning about whether changeHandler is going to fire in every case that might "fix" the failure is too difficult (impossible?). Simpler to just not cache on failure.
Thanks

@xclaesse should we just merge this and spin a release?

@tristan957 TBH I was not convinced about this PR, if getting tasks failed once, it won't magically succeed next time. The only thing this could achieve is slowing down everything (load tasks is slow) and spamming user with the error message. Unless I missed something?
AFAIK, the only chance we have to have loading tasks succeed again is after a reconfigure, in that case we do that already.

Maybe I was unclear so let me try again: it's not about magically succeeding next time. It's about the lack of any feedback running tasks after caching the failure.
Not caching the failed result was the smallest mod I could see to achieve this. Such feedback is not spam... spam is unsolicited.
Worrying about "slowing down everything" isn't an issue for the super rare case of getMesonTasks failing.

Seems pretty compelling to me after that explanation.

I should give it a try to see how it goes in practice. I'm in phone ATM, can't test until Monday.
The reason I'm prudent here is because when I implemented that cache it was because vscode calls that method a lot to populate menus, and parsing interpretation JSON files is super slow. So I would like to be sure it shows error only when user really does an action and not spuriously just because vscode refresh a menu.
Maybe it should cache the error state to not retry parsing and show error immediately?

repo: mesonbuild/vscode-meson


text:
Failed to detect a master: Failed to parse data of unknown label 'json.info'

Below is the json file which I am using
{
   "id": "mist-logstash",
   "cpus": 1,
   "mem": 1024.0,
   "instances": 1,
   "container": {
     "type": "DOCKER",
     "docker": {
       "image": "mesos/logstash-scheduler:0.0.6",
       "network": "HOST"
     }
   },
   "env": {
     "JAVA_OPTS": "-Dmesos.logstash.framework.name=logstash -Dmesos.zk=zk://zkhost:2181/mesos"
   }
 }

still getting below error
2016-01-16 01:12:19,956:5(0x7f80a1ffb700):ZOO_INFO@check_events@1703: initiated connection to server [172.31.4.72:2181]
2016-01-16 01:12:19,959:5(0x7f80a1ffb700):ZOO_INFO@check_events@1750: session establishment complete on server [172.31.4.72:2181], sessionId=0x1516706148feac2, negotiated timeout=10000
I0116 01:12:19.959508    34 group.cpp:313] Group process (group(1)@172.31.4.28:59624) connected to ZooKeeper
I0116 01:12:19.959650    34 group.cpp:790] Syncing group operations: queue size (joins, cancels, datas) = (0, 0, 0)
I0116 01:12:19.959738    34 group.cpp:385] Trying to create path '/mesos' in ZooKeeper
I0116 01:12:19.980105    34 detector.cpp:138] Detected a new leader: (id='85')
I0116 01:12:19.980309    34 group.cpp:659] Trying to get '/mesos/json.info_0000000085' in ZooKeeper
Failed to detect a master: Failed to parse data of unknown label 'json.info'

Mesos version is 0.25.

We are in the process updating the docs. Please take a look at #79.

authors: abhishekamralkar mwl

repo: mesos/logstash


text:
Probe modules

https://jira.mesosphere.com/browse/DCOS_OSS-4792
Probe returns an error if required modules are not loaded:
--- PASS: TestProbe_MissingRequiredModule (0.92s)
    csilvm_test.go:2211: rpc error: code = FailedPrecondition desc = One or more kernel modules are missing: [no_such_module]

@gpaul @nfnt PTAL

repo: mesosphere/csilvm


text:
Update Tests to Run Against DCOS Cluster

I believe that current tests only run against Mesos clusters with Marathon running.  I do not believe they actually run against DCOS clusters.  The test infrastructure for the dcos-cli tests needs to provision a DCOS cluster.

It runs against a Vagrant VM generated from the DCOS Image.

Then tests are not appropriately setting the dcos.core_url, or something else is going wrong.  http://<dcos.cor_url>/metadata does not return the correct information in the test context.  Some issue should be used to track this.  Either this one should be modified for clarity or a new one should be created.

ssh to DCOS nodes is disabled by design

authors: gabrielhartmann jsancio

repo: mesosphere/dcos-cli


text:
ci: changed runner labels

There were some workflows that still had the wrong labelled runners or were using githubs runners, we need to change them to org-wire-runners so we have a more coherent understanding of our infrastructure. This pr will be used to understand which workflows require which size or if they need kind cluster. Please check the the runners in the change made and use this as a criteria. Once the sizes have been confirmed the pr we can merge.
Which issue(s) does this PR fix?:
https://jira.d2iq.com/browse/D2IQ-97191
Special notes for your reviewer:

Does this PR introduce a user-facing change?:

@shaeed-noordeen The jobs are not getting any self-hosted runners. can you please take a look.

authors: shaeed-noordeen supershal

repo: mesosphere/konvoy-image-builder


text:
Fix the formatting on restart service section & Remove marathon hostname configuration for agent nodes

For the restart service commands to be formatted correctly on the page I neede to inclue a newline before ```.
Also we do not need to set hostname for marathon on agent nodes since they do not run marathon

Thanks @surdy

repo: mesosphere/open-docs


text:
The reference space type is not updated until the frame after the session started

In the actual quest, the new reference type is already local floor when the session start, but with the emulator is not which is annoying and requires me to have special cases to handle the emulator if I want to test with it.
As u can see in the video, if I ask for it after the session started is properly updated to local-floor in this case.
https://github.com/meta-quest/immersive-web-emulator/assets/16962655/f44ba115-a93b-4dc3-97a8-b76452f917f8
This is with Wonderland Engine

Good find, will fix in v1.4

authors: SignorPipo felixtrz

repo: meta-quest/immersive-web-emulator


text:
chore(deps): update dependency jest-environment-jsdom to v27.4.6

This PR contains the following updates:

Package
Type
Update
Change

jest-environment-jsdom
devDependencies
minor
27.0.6 -> 27.4.6

Release Notes

facebook/jest
v27.4.6
Compare Source
Fixes

[jest-environment-node] Add AbortSignal (#12157)
[jest-environment-node] Add Missing node global performance (#12002)
[jest-runtime] Handle missing mocked property (#12213)
[@jest/transform] Update dependency package pirates to 4.0.4 (#12002)

Performance

jest-config perf: only register ts-node once when loading TS config files (#12160)

v27.4.4
Compare Source
Fixes

[babel-jest] Add process.version chunk to the cache key (#12122)
[jest-environment] Add mocked to jest object (#12133)
[jest-worker] Stop explicitly passing execArgv (#12128)

Chore & Maintenance

[website] Fix the card front that looks overlapping part of the card back

v27.4.3
Compare Source
Fixes

[jest-environment-jsdom] Remove @types/jsdom dependency (and make its dom property private) (#12107)

v27.4.2
Compare Source
Fixes

[jest-worker] Add additional execArgv to filter (#12103)

v27.4.1
Compare Source
Fixes

[jest-worker] Filter execArgv correctly (#12097)

v27.4.0
Compare Source
Features

[expect] Enhancing the toHaveProperty matcher to support array selection (#12092)
[jest-core] Add support for testResultsProcessor written in ESM (#12006)
[jest-diff, pretty-format] Add compareKeys option for custom sorting of object keys (#11992)
[jest-mock] Add ts-jest mock util functions (#12089)

Fixes

[expect] Allow again expect.Matchers generic with single value (#11986)
[jest-circus, jest-jasmine2] Avoid false concurrent test failures due to unhandled promise rejections (#11987)
[jest-config] Add missing slash dependency to package.json (#12080)
[jest-core] Incorrect detection of open ZLIB handles (#12022)
[jest-diff] Break dependency cycle (#10818)
[jest-environment-jsdom] Add @types/jsdom dependency (#11999)
[jest-environment-jsdom] Do not reset the global.document too early on teardown (#11871)
[jest-transform] Improve error and warning messages (#11998)
[jest-worker] Pass execArgv correctly to worker_threads worker (#12069)

Chore & Maintenance

[docs] CLI options alphabetized (#11586)
[jest-runner] Add info regarding timers to forcedExit message(#12083)
[*] Replaced substr method with substring (#12066)
[*] Add types entry to all export maps (#12073)

v27.3.1
Compare Source
Fixes

[expect] Make expect extension properties configurable (#11978)
[expect] Fix .any() checks on primitive wrapper classes (#11976)

Chore & Maintenance

[expect] BigInt global is always defined, don't check for its existence at runtime (#11979)
[jest-config, jest-util] Use ci-info instead of is-ci to detect CI environment (#11973)

v27.3.0
Compare Source
Features

[jest-config] Add testEnvironmentOptions.html to apply to jsdom input (#11950)
[jest-resolver] Support default export (.) in exports field if main is missing (#11919)

Fixes

[expect] Tweak and improve types (#11949)
[jest-runtime] Ensure absolute paths can be resolved within test modules (#11943)
[jest-runtime] Fix instanceof for ModernFakeTimers and LegacyFakeTimers methods (#11946)

v27.2.5
Compare Source
Features

[jest-config] Warn when multiple Jest configs are located (#11922)

Fixes

[expect] Pass matcher context to asymmetric matchers (#11926 & #11930)
[expect] Improve TypeScript types (#11931)
[expect] Improve typings of toThrow() and toThrowError() matchers (#11929)
[jest-cli] Improve --help printout by removing defunct --browser option (#11914)
[jest-haste-map] Use distinct cache paths for different values of computeDependencies (#11916)
[@jest/reporters] Do not buffer console.logs when using verbose reporter (#11054)

Chore & Maintenance

[expect] Export default matchers (#11932)
[@jest/types] Mark deprecated configuration options as @deprecated (#11913)

v27.2.4
Compare Source
Features

[expect] Add equality checks for Array Buffers in expect.ToStrictEqual() (#11805)

Fixes

[jest-snapshot] Correctly indent inline snapshots (#11560)

v27.2.3
Compare Source
Features

[@jest/fake-timers] Update @sinonjs/fake-timers to v8 (#11879)

Fixes

[jest-config] Parse testEnvironmentOptions if received from CLI (#11902)
[jest-reporters] Call destroy on v8-to-istanbul converters to free memory (#11896)

v27.2.2
Compare Source
Fixes

[jest-runtime] Correct wrapperLength value for ESM modules. (#11893)

v27.2.0
Compare Source
Features

[jest-resolver, jest-runtime] Pass conditions to custom resolvers to enable them to implement support for package.json exports field (#11859)
[jest-runtime] Allow custom envs to specify exportConditions which is passed together with Jest's own conditions to custom resolvers (#11863)

Fixes

[@jest/reporters] Use async transform if available to transform files with no coverage (#11852)
[jest-util] Return correct value from process.send stub (#11799)

v27.1.1
Compare Source
Features

[jest-runtime] Add experimental, limited (and undocumented) support for mocking ECMAScript Modules (#11818)

Fixes

[jest-resolver] Support node: prefix when importing Node core modules with ESM (#11817)
[jest-types] Export the PrettyFormatOptions interface (#11801)

v27.1.0
Compare Source
Features

[jest-haste-map] Use watchman suffix-set option for faster file indexing. (#11784)
[jest-cli] Adds a new config options snapshotFormat which offers a way to override any of the formatting settings which come with pretty-format. (#11654)
[jest-reporters] Expose the getSummary util (#11695).
[jest-resolver] Support node: prefix when importing Node core modules (#11331)

Fixes

[jest-each] Relaxed the validation to allow multibyte characters in headings (#11575)
[jest-environment-jsdom] Add support for userAgent option (#11773)
[jest-environment-node] Add Event and EventTarget to node global environment. (#11727)
[jest-mock] Fix spyOn to use Object.prototype.hasOwnProperty (#11721)
[jest-resolver] Add dependency on jest-haste-map (#11759)
[jest-types] Compat with @types/node v16 (#11645)

Chore & Maintenance

[docs] Correct expects.assertions documentation by adding async/await for asynchronous function.

Configuration
📅 Schedule: At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about this update again.

[ ] If you want to rebase/retry this PR, check this box.

This PR has been generated by Renovate Bot.

Renovate Ignore Notification
As this PR has been closed unmerged, Renovate will now ignore this update (27.4.6). You will still receive a PR once a newer version is released, so if you wish to permanently ignore this dependency, please add it to the ignoreDeps array of your renovate config.
If this PR was closed by mistake or you changed your mind, you can simply rename this PR and you will soon get a fresh replacement PR opened.

repo: metabreak/grpc-lib


text:
no issue

just testing. I love YAMZ! xo

issue fixed

repo: metadata-research/yamz


text:
TrafficGen Input  without Agents

Hi,
Is there a way to generate Scenarios with only Map features as Input ?
Because according to the code , the vehicle's placement model depends on existing agent with the map .
Thank you

This is a long lasting problem and according to some feedback from other user I believe it's not trivial modification to make it work without the first agent. That can be considered to be a limitation of this work. Sorry for this. I remember a paper citing TrafficGen talked about this but unfortunately I can't find it out.
The intuition behind this is that the first agent's velocity actually makes a strong prior knowledge to other agents.

repo: metadriverse/trafficgen


text:
Allow retry when error And let error event get more info

When error, the "infinite-scroll-error" element could show "Error, <a onlick="infScroll.loadNextPage()>Click here Retry", give a chance to user for retry, if success, Reset canLoad to true, everything go on.

Rather than handling this within Infinite Scroll, you could add this functionality outside by listening for the error event

@desandro How?

// jQuery
$container.on( 'error.infiniteScroll', function( event, error, path ) {
  // add functionality here
})

// vanilla JS
infScroll.on( 'error', function( error, path ) {...})

You not understood this issue, please see the changed.

Thanks I have decided to pass on this contribution. I appreciate your effort here, and hope you continue getting involved in open source.

authors: ahdung desandro

repo: metafizzy/infinite-scroll


text:
Post: Introducing BMO e2e

Add a blog post introducing the BMO e2e suite.
Fixes: #371

/cc @kashifest @tuminoid

Oh come on! What is the logic that determines how that cspell file should be formatted? It seems to change all the time

Oh come on! What is the logic that determines how that cspell file should be formatted? It seems to change all the time

LOL, I had the same. Not sure what triggers it. At some point I reasoned to myself it is about line length but then it just decided to reformat it anyways in different way on next save.


text:
Revert "Hook execve and execv to allow shared sockets. (#2610)"

Reverts broken #2610

The bug was fixed with #2620

repo: metalbear-co/mirrord


text:
Remove the manifest generation

Although it's a valid idea to use metallb as source of truth, in practice it's hard to maintain.
The generate_manifests.sh file is now complex and hard to maintain.
It would make more sense to keep the manifests of metallb indipendent from the operator template, and having a way to validate they are not drifting by filling values of the template and comparing them.

This was done by moving to the helm based approach, closing it

repo: metallb/metallb-operator


text:
Need to update netname in some e2e test cases

Signed-off-by: Mohamed Mahmoud mmahmoud@redhat.com

LGTM


text:
Support for macOS Development

I think this belongs more in this repo - https://github.com/metaparticle-io/metaparticle-ast/issues/2 based on https://github.com/metaparticle-io/package/blob/master/javascript/metaparticle-package/index.js#L5-L18

You're right, the bug is in this repo. This PR helped me to make it work with Docker for Mac.

This is now fixed (I think)

authors: brendandburns chanezon christopherhein

repo: metaparticle-io/package


text:
Add mint ui example

How this example can and should be used is further described in this PR to the docs https://github.com/metaplex-foundation/docs/pull/391
The docs will explain everything, nevertheless we should allow the user to just clone the page to allow them to focus on understanding instead of copy and pasting code.
This page is based on the connect-wallet example

should be all done.
Thank you very much for reviewing!

@lorisleiva from my point of view merging this should be fine.
I will add additional functions (LUT, Mint multiple NFTs at once...) in a future revision, but i would like to merge this first so that we can reference it already.

Thank you! 🥳

authors: MarkSackerberg lorisleiva

repo: metaplex-foundation/js-examples


text:
metarhia-common fail to require a submodule in browser environment

Stacktrace:
Uncaught Error: Cannot find module './lib/utilities'
    at webpackEmptyContext (eval at ../metaschema/node_modules/metarhia-common/dist sync recursive (app.js:127), <anonymous>:2:10)
    at Array.map (<anonymous>)
    at eval (common.js?3e0f:24)
    at Object.../metaschema/node_modules/metarhia-common/dist/common.js (vendors.js:11)
    at __webpack_require__ (runtime.js:783)
    at fn (runtime.js:151)
    at eval (introspection.js?307f:11)
    at Object.../metaschema/dist/lib/introspection.js (app.js:68)
    at __webpack_require__ (runtime.js:783)
    at fn (runtime.js:151)

FWIW, I encountered the problem trying to use metaschema in browser, although that's not really relevant to the issue.
/cc @tshemsedinov @nechaido @lundibundi

This seems to be the exact same problem that #178 should have fixed, but metaschema uses an ancient version of metarhia-common.

Closing this since that's not really an issue with metarhia-common itself anymore.

repo: metarhia/common


text:
Get request to root static directory generates 404

Impress and Node.js versions
Impress 3.0.0-alpha.5, Node 18.4.0
Platform
Windows
Describe the bug
GET request to root static directory, like http://localhost:8000/ redirects to http://localhost:8001/  and generates 404 error.
{"error":{"message":"Not Found","code":404}}
GET request to http://localhost:8001/index.html works fine
To Reproduce
No response
Expected behavior
No response
Screenshots
No response
Additional context
No response

Can't reproduce this for 3.0.0-alpha.8 @leonpolak

Fixed in v3.0.0-alpha.14

authors: leonpolak tshemsedinov

repo: metarhia/impress


text:
Paginate dataset list

Quickly figured this out, no UI changes but allows showing more datasets with ?page=2.

Looks good but I'm not sure those who initially raised the issue will be happy to edit URL parameters to get pagination working.

No worries, it's just meant to be a quick hack before the UI is
implemented. At least it provides access to data which is currently not
accessible at all.
On Fri, 12 Jun 2020 at 12:55, Vitaly Kovalev notifications@github.com
wrote:

Looks good but I'm not sure those who initially raised the issue will be
happy to edit URL parameters to get pagination working.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/metaspace2020/metaspace/pull/603#issuecomment-643209504,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAK5BWM6TYCDWEFGZWGQGM3RWICSXANCNFSM4NSPQGZA
.

authors: intsco richardgoater


text:
Add guide for Discovery api into jupyter cells

Description

Simple one.
Add a guidance comment for R functions into auto-generated Jupyter cells.
Related Issue :

#2343
metatron-app/discovery-jupyter-r-utils#2
How Has This Been Tested?

Add a unit test and already using in the own notebooks.
Need additional checks?
Types of changes

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

Checklist:

[x] I have read the CONTRIBUTING document.
[x] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[x] I have added tests to cover my changes.

Additional Context
If you want to use the functions described in the comment, you need to upgrade RMetis module to the latest version.

run build
deploy to 4

Share result :

repo: metatron-app/metatron-discovery


text:
fix column type of metadata to be same as filter type

Description
Fix column type of metadata to be same as filter type
Related Issue :

How Has This Been Tested?

Management - Metadata - Column Details
Check that the column type of the metadata is the same as the type of the filter.

Need additional checks?
Types of changes

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

Checklist:

[ ] I have read the CONTRIBUTING document.
[ ] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have added tests to cover my changes.

Additional Context

deploy to 3

deploy to 3


text:
[Bug] 调用的函数不存在

Verify steps

[X] Is this something you can debug and fix? Send a pull request! Bug fixes and documentation fixes are welcome.
[X] I have read the Wiki, especially the FAQ page.
[X] I have searched on the issue tracker for a related issue.

MetaTube Plugin Version
2023.503.910.0
MetaTube Server Version
https://jav.mcjoker.xyz/
What OS are you seeing the problem on?
Linux
Description
刷新元数据时失败
MetaTube Server
No response
Jellyfin/Emby Logs
[2023-06-04 11:19:49.681 +08:00] [ERR] Error in "MetaTube"
System.MissingMethodException: Method not found: 'Void MediaBrowser.Controller.Entities.PersonInfo.set_Type(System.String)'.
   at Jellyfin.Plugin.MetaTube.Providers.MovieProvider.GetMetadata(MovieInfo info, CancellationToken cancellationToken)
   at System.Runtime.CompilerServices.AsyncMethodBuilderCore.Start[TStateMachine](TStateMachine& stateMachine)
   at Jellyfin.Plugin.MetaTube.Providers.MovieProvider.GetMetadata(MovieInfo info, CancellationToken cancellationToken)
   at MediaBrowser.Providers.Manager.MetadataService`2.ExecuteRemoteProviders(MetadataResult`1 temp, String logName, TIdType id, IEnumerable`1 providers, CancellationToken cancellationToken)

MetaTube Server Logs
No response
How to Reproduce
No response

Jellyfin版本 10.9.0


text:
Redirect after login / registration

Hello, what is the proper / recommended method for redirecting a user to a specific page after they login or register?
Thank you in advance
Eric

Is this still an issue for you? If you are using FlowRouter, this is what I've been using that works:
AccountsTemplates.configure({
    onSubmitHook: ( error, state ) => {
        if ( !error && state === 'signIn' ) {
            // login successful, route to index
            FlowRouter.go('index');
        }
    },
    onLogoutHook: ( error, state ) => {
        FlowRouter.go('signin');
    }
});

repo: meteor-useraccounts/core


text:
Ancillary generation: E-W and N-S orography gradient field

As a developer I want access to ancillary datasets for the E-W and N-S orography gradient field for spot data and neighbourhood processing routines so that developers don't have to regenerate these fields each time they are required.
Related issues: #43
Acceptance criteria:

Develop a flexible script(s) that can be called from the command line, in order to generate the required ancillary files
Develop plugins to generate the required ancillary with unit tests
Generate an ancillary file for E-W and N-S orography gradient field

This issue has been superseded by the Epic: Ancillary Generation for Improver (#97) and will be included in a User Story under this Epic.

authors: gavinevans meabhnicguidhir

repo: metoppv/improver


text:
temporal-interpolate CLI to clize

CLI now outputs CubeList into a single file.
Need to update one of the KGOs to reflect this change

Testing:

[x] Ran tests and they passed OK  (except known failure)

 Here is an overview of what got changed by this pull request:

Complexity decreasing per file
==============================
+ lib/improver/tests/acceptance/test_temporal_interpolate.py  -3

See the complete overview on Codacy

 Here is an overview of what got changed by this pull request:

Complexity decreasing per file
==============================
+ lib/improver/tests/acceptance/test_temporal_interpolate.py  -3

See the complete overview on Codacy

 Here is an overview of what got changed by this pull request:

Complexity decreasing per file
==============================
+ lib/improver/tests/acceptance/test_temporal_interpolate.py  -3

See the complete overview on Codacy


text:
Switch GitHub Actions back on for PRs

Switch GitHub Actions back on for PRs - need this to get the check at the bottom of pull request pages.

You get duplicate runs ^
..because you are on upstream, but it's useful for forked PRs ;)

authors: LaurenceBeard benfitzpatrick


text:
Profile

Currently, figures take about 30 minutes to build, for 200 data points.
Profiling them to see which functions and which calls are taking up the most time can yield invaluable information on how to speed up the code. Speeding up the parts of the code that take up the most time (and sometimes memory access calls) can speed up the code. A speed of 3x or more is very noticeable and worthwhile.
See the messages here to learn how to do so:
https://github.com/meyer-lab/tHMM/issues/227#issuecomment-590603889

@aarmey Here's the profile for figure 3.  Any advice on where we can optimize? Shakthi said the pdf call was a good candidate.

That looks like the only clear possibility to me. Though, it's only ~20% of the time so it's not going to make everything 3X faster (~20% faster at best).
It's getting called a lot so turning most of it into a numba function can help. The best thing would be if the pdf could be called for all the cells at once in a vectorized fashion (i.e. you pass a matrix with all the observations, and get back all the probabilities). I'm not sure how feasible that is with how the emissions are stored right now.

...
    for lineageObj in tHMMobj.X:  # for each lineage in our Population
        lineage = lineageObj.output_lineage  # getting the lineage in the Population by lineage index
        EL_array = np.zeros((len(lineage), tHMMobj.num_states))  # instantiating N by K array for each lineage

        for current_cell_idx, cell in enumerate(lineage):  # for each cell in the lineage
            for state_k in range(tHMMobj.num_states):  # for each state
                EL_array[current_cell_idx, state_k] = E[state_k].pdf(cell.obs)

        EL.append(EL_array)  # append the EL_array for each lineage
    return EL

EL_array is holding the probabilities.
E is a list of state distributions.
This is the function that utilizes the pdf. You can see the double for loop. I'm assuming we can at least vectorize the outer for loop over the cells, so the only for loop is over the states. The emissions are stored in the cells as a tuple. Do you think this can be vectorized @aarmey ?

Yep. Just vectorizing the loop over cells would be a huge boost.

authors: aarmey lkargi shak360

repo: meyer-lab/tHMM


text:
Beaucoup de "Unhandled exception, please reload"

Quasiment à chaque fois que je clique sur "generate" dans "Analytics" j'ai cette erreur.
blazor.webassembly.js:1 GitHub dabatabase sha 'e0a6ac7f87b63a4d5a3be87d8bd6bf0f3d01a2d8' is equals to the current db sha 'e0a6ac7f87b63a4d5a3be87d8bd6bf0f3d01a2d8' => Do not import
blazor.webassembly.js:1 crit: Microsoft.AspNetCore.Components.WebAssembly.Rendering.WebAssemblyRenderer[100]
Unhandled exception rendering component: Arg_IndexOutOfRangeException
System.IndexOutOfRangeException: Arg_IndexOutOfRangeException
at Meziantou.Moneiz.Pages.Analytics.BalanceHistoryChart.OnParametersSet()
at Microsoft.AspNetCore.Components.ComponentBase.RunInitAndSetParametersAsync()
at Microsoft.AspNetCore.Components.RenderTree.Renderer.GetErrorHandledTask(Task , ComponentState )
(anonymous) @ blazor.webassembly.js:1
service-worker.js:47 Serving from cache: https://moneiz2.meziantou.net/_content/MudBlazor/MudBlazor.min.css
service-worker.js:47 Serving from cache: https://moneiz2.meziantou.net/lib/fa/css/all.css
service-worker.js:47 Serving from cache: https://moneiz2.meziantou.net/css/site.min.css
service-worker.js:47 Serving from cache: https://moneiz2.meziantou.net/Meziantou.Moneiz.styles.css
service-worker.js:47 Serving from cache: https://moneiz2.meziantou.net/manifest.json

blazor.webassembly.js:1 crit: Microsoft.AspNetCore.Components.WebAssembly.Rendering.WebAssemblyRenderer[100]
Unhandled exception rendering component: Arg_IndexOutOfRangeException
System.IndexOutOfRangeException: Arg_IndexOutOfRangeException
at Meziantou.Moneiz.Pages.Analytics.BalanceHistoryChart.OnParametersSet()
at Microsoft.AspNetCore.Components.Rendering.ComponentState.SupplyCombinedParameters(ParameterView )

Fixed by 935de98d0b3244c88cc2b7abeed2fc17695f9451


text:
Proposition to add note about trackBy and ngDoCheck

Hi @mgechev i noticed that there is no notes about using trackBy function for *ngFor and nothing about using ngDoCheck to reduce heavy calculation in multiple places in template.
Would you be open to accept pull request on that?
If so - i would be happy to send it this weekends.

Sure, if you want you can draft the suggestions here and later move them to the list?
Thanks for the suggesting the improvements!

@mgechev there is my first iteration on trackBy topic.
I also would like to add persistence advantage of using tracking function. The idea is simple: given that rendered collection of video players, each one has the "like" button. User starts to play one video and in the middle of the process clicks "like". If an immutable collection is used (which is true in most cases, as it's suggested practice) then without trackBy DOM tree re-renders again to alter collection state. Which causes that player is stopped because it is totally new player has been rendered on its place.
But i am afraid describing this in good words and complete example will dramatically increase paragraph length which makes it less chance to be finished by the reader.
Do you think it worth adding it?
trackBy option for *ngFor directive
The *ngFor directive is used for rendering a collection.
By default *ngFor doesn't know how to identify each item of the given collection.
When the collection is changed in some way then the whole DOM tree is destroyed and recreated again. This is acceptable if the collection is totally different. But most times we have just a slightly modified collection (e.g. collection is sorted or one item is added/removed/changed). Anyway DOM tree is destroyed and recreated from the scratch again. This is unnecessary and, what even more important, may lead to poor performance of the whole app, particularly on a big collection of data.
To modify this behavior it is up to developer to provide custom tracking function as trackBy option for the *ngFor directive. Tracking function takes two arguments: index and item. Angular uses the value returned from tracking function to track items identity.
Example
@Component({
  selector: 'yt-feed',
  template: `

Your video feed

  <yt-player *ngFor="let video of feed; trackBy: trackById" [video]="video"></yt-player>
`
})
export class YtFeedComponent {
  feed = [
    {
      id: 3849, // note "id" field, we refer to it in "trackById" function
      title: "Angular in 60 minutes",
      url: "http://youtube.com/ng2-in-60-min",
      likes: "29345"
    },
    // ...
  ];

  trackById(index, item) {
    return item.id;
  }
}

Sorry for the delay.
Yes, I agree that this example may get a bit too long. The draft of trackBy sounds good to me! Maybe you can provide an external reference of the advantages of trackBy in case of immutable data?
Do you want to open a PR with the suggestion?

Yes i will open PR in the middle of the week (i did not get much free time these weekends).
Could you please clarify what do you mean under

an external reference of the advantages of trackBy in case of immutable data

?

I also would like to add persistence advantage of using tracking function. The idea is simple: given that rendered collection of video players, each one has the "like" button. User starts to play one video and in the middle of the process clicks "like". If an immutable collection is used (which is true in most cases, as it's suggested practice) then without trackBy DOM tree re-renders again to alter collection state. Which causes that player is stopped because it is totally new player has been rendered on its place.

I think by itself, this deserves a short blog post. I think it's worth mentioning something like:

Introducing a trackBy function will not only improve performance but also help us determine the unchanged entries in immutable data structures. For further reference take a look here.

authors: mgechev tytskyi

repo: mgechev/angular-performance-checklist


text:
🛑 Merchant APP (API Version API) is down

In e44c1c8, Merchant APP (API Version API) (https://gateway.wavemoney.io/v2/merchant-api/url-version/entity/merchant) was down:

HTTP code: 502
Response time: 308 ms

Resolved: Merchant APP (API Version API) is back up in 441bc2f after 2 hours, 51 minutes.

repo: mgmgpyaesonewin/strapi-upptime


text:
Uncaught SyntaxError: Unexpected token U

[Enter steps to reproduce below:]

...
...

Atom Version: 1.3.2
System: Microsoft Windows 10 Pro
Thrown From: Remote-FTP package, v0.7.15
Stack Trace
Uncaught SyntaxError: Unexpected token U
At file:///C:/Users/gpopa.TAYCOR/AppData/Local/atom/app-1.3.2/resources/app.asar/static/index.html#%7B%22locationsToOpen%22%3A%5B%7B%22pathToOpen%22%3A%22C%3A%5C%5CUsers%5C%5CGPOPA~1.TAY%5C%5CAppData%5C%5CLocal%5C%5CTemp%5C%5Cscp22471%5C%5Chome%5C%5Cdesktop%5C%5Csalesapp%5C%5Cmeteor%5C%5Cclient%5C%5Ctimeline.js%22%2C%22initialLine%22%3Anull%2C%22initialColumn%22%3Anull%7D%2C%7B%22pathToOpen%22%3A%22C%3A%5C%5CUsers%5C%5Cgpopa.TAYCOR%5C%5C.atom%22%2C%22initialLine%22%3Anull%2C%22initialColumn%22%3Anull%7D%5D%2C%22windowInitializationScript%22%3A%22C%3A%5C%5CUsers%5C%5Cgpopa.TAYCOR%5C%5CAppData%5C%5CLocal%5C%5Catom%5C%5Capp-1.3.2%5C%5Cresources%5C%5Capp.asar%5C%5Csrc%5C%5Cinitialize-application-window.js%22%2C%22resourcePath%22%3A%22C%3A%5C%5CUsers%5C%5Cgpopa.TAYCOR%5C%5CAppData%5C%5CLocal%5C%5Catom%5C%5Capp-1.3.2%5C%5Cresources%5C%5Capp.asar%22%2C%22devMode%22%3Afalse%2C%22safeMode%22%3Afalse%2C%22profileStartup%22%3Afalse%2C%22appVersion%22%3A%221.3.2%22%2C%22atomHome%22%3A%22C%3A%5C%5CUsers%5C%5Cgpopa.TAYCOR%5C%5C.atom%22%2C%22shellLoadTime%22%3A282%2C%22projectDirectoryPaths%22%3A%5B%22C%3A%5C%5CUsers%5C%5CGPOPA~1.TAY%5C%5CAppData%5C%5CLocal%5C%5CTemp%5C%5Cscp22471%5C%5Chome%5C%5Cdesktop%5C%5Csalesapp%5C%5Cmeteor%5C%5Cclient%5C%5Ctimeline.js%22%2C%22C%3A%5C%5CUsers%5C%5Cgpopa.TAYCOR%5C%5C.atom%22%5D%7D:10

SyntaxError: Unexpected token U
    at Object.parse (native)
    at C:\Users\gpopa.TAYCOR\.atom\packages\Remote-FTP\lib\client.js:78:21
    at FSReqWrap.readFileAfterClose [as oncomplete] (fs.js:380:3)

Commands
     -7:25.3.0 tree-view:remove (li.file.entry.list-item.selected)
  2x -7:18.6.0 tree-view:remove-project-folder (span.name.icon.icon-file-directory)
     -7:03.3.0 remote-ftp:create-sftp-config (ol.tree-view.full-menu.list-tree.has-collapsable-children.focusable-panel)
  2x -6:56.8.0 remote-ftp:toggle (ol.tree-view.full-menu.list-tree.has-collapsable-children.focusable-panel)
     -6:51.7.0 application:open-folder (atom-pane.pane.active)
     -6:40.5.0 remote-ftp:toggle (atom-pane.pane.active)
     -5:14.5.0 remote-ftp:connect (ol.tree-view.full-menu.list-tree.has-collapsable-children.focusable-panel)
     -5:00.7.0 remote-ftp:toggle (atom-pane.pane.active)
     -3:15 tree-view:remove-project-folder (span.name.icon.icon-file-directory)
     -2:55.1.0 remote-ftp:toggle (ol.tree-view.full-menu.list-tree.has-collapsable-children.focusable-panel)
     -2:27.2.0 remote-ftp:create-sftp-config (ul.list-inline.tab-bar.inset-panel)
     -2:12.3.0 core:backspace (atom-text-editor.editor.is-focused)
     -1:57.7.0 core:move-left (atom-text-editor.editor.is-focused)
  2x -1:56.8.0 core:paste (atom-text-editor.editor.is-focused)
  3x -1:41.5.0 core:save (atom-text-editor.editor.is-focused)
     -0:01.3.0 remote-ftp:connect (atom-text-editor.editor.is-focused)

Config
{
  "core": {}
}

Installed Packages
# User
color-picker, v2.0.14
highlight-selected, v0.11.1
minimap-highlight-selected, v4.3.1
remote-edit, v1.8.23
Remote-FTP, v0.7.15
smart-tab-name, v1.0.5

# Dev
No dev packages

Please post your .ftpconfig without credentials

Please post your .ftpconfig without credentials

Fixed this bug. I had to escape the back-slashes in the path to the file.

Fixed this bug. I had to escape the back-slashes in the path to the file.

@radupopa2010 what do you mean escape backslashes? thanks
does the whats the key file format? is it .ppk?

repo: mgrenier/remote-ftp

license: mit


text:
Add Timeout to refresh Repo

In order to update from repo with putting to much pressure on the storage, a timeout can be set on repo assignment. This timeout in seconds allows update of flags in distributed deployments.

i guess the travis needs some updates.
The updated rubocop version in the current runs had more complains that before. I satisfied rubocop also for the old code

Hi,
I took your code and tweaked it a little, see f4795a37b3c0408951a78ccf20bf6a398c9df105
Thanks for the input

authors: enc mgsnova

repo: mgsnova/feature


text:
Support for objective c and to use .plist

Hello mhergon,
First of all great job you have done with this library
Can you please provide update in Objective c and suggest me to a way for use .plist file instead of .srt file

Hi @vanditmehta
You can use Swift version on your Objective-C projects. Please, refer to this issue for instructions (last message).
https://github.com/mhergon/MPMoviePlayerController-Subtitles/issues/17
About .plist, what do you want to do exactly?

We want to create dynamic control where user can touch and add some text from some time stamp to some another time stamp

Hi @vanditmehta
The library supports strings and .srt files. If you create string manually, you will be able to read the string and show with the library.

authors: mhergon vanditmehta


text:
Multiple vms in one vagrant

Vagrant.configure("2") do |config|
    config.vm.define "debbie" do |debbie|
        debbie.vm.box = "debian/jessie64"
        debbie.vm.network :forwarded_port, host: 80, guest: 80
        debbie.vm.network :forwarded_port, host: 3306, guest: 3306
        debbie.vm.network :forwarded_port, host: 2299, guest: 22
        debbie.vm.provision "shell", inline: "echo Hello"
    end

    config.vm.define "cindy" do |cindy|
        cindy.vm.box = "bento/centos-7.2"
        cindy.vm.network :forwarded_port, host: 8888, guest: 88
        cindy.vm.network :forwarded_port, host: 8080, guest: 80
        cindy.vm.network :forwarded_port, host: 3333, guest: 3306
        cindy.vm.network :forwarded_port, host: 2298, guest: 22
        cindy.vm.provision "shell", inline: "echo Hello"
    end
end

$ vagrant up # both machines boot up. Then ...
$ vagrant ssh cindy
# … or:
$ vagrant ssh debbie

Then if someone was using windows and wanted to use sftp netdrive to mount the drives so they could open via explorer
@echo off
Start "" "C:\Program Files (x86)\SFTP Net Drive 2017\Sftpnetdrive.exe" start /profile:"debbie"
Start "" "C:\Program Files (x86)\SFTP Net Drive 2017\Sftpnetdrive.exe" start /profile:"cindy"
Start "" "C:\Program Files (x86)\SFTP Net Drive 2017\Sftpnetdrive.exe" start /profile:"s1"

automates the connection process

This is good info. Need to move it to WIKI and close this issue out.

Added to wiki. Closing.

repo: mhulse/vagrant-latmp


text:
Not editable mentions support

This is rather question than issue. How/If it is possible to do so mentions themselves are not editable? So when user presses backspace (for example), whole mention is removed.

Thank you for the suggestion, I'm working on this feature.
If not in the 1.3.0 update, perhaps in the 1.4.0 one.
Thank you ;)

has been added in the 1.4.0 release, Check out the 'deleteOnBack' functionality!
Thank you!

authors: micazi radzish


text:
Correct docs.py

'j' is actually down, and 'k' is up.

Nice catch, thanks!

authors: crsaracco michael-lazar

repo: michael-lazar/rtv


text:
Improve EditorToolBar UX for iOS

Don't hide toolbar when de-activating soft keyboard, move it around based on logic in frontend.
This solution can be read about here: sticky-header-ios-keyboard-fix.
I have still found a few edge cases from my phone so it is not perfect 100% of times but might at least be a good start towards fixing this. Mainly my concern is that it kind of flickers when you hit save, I guess it jumps with scroll restoration.
Mitigates the problem mentioned in issue: #1

Aww this is such a difficult thing to get right. I have failed with many attempts myself. I tested your version on iPhone and while in some cases the toolbar re-appears in the correct location, in other cases it disappears or gets moved to the middle or bottom of the screen. It's great to have a starting point, but we're not quite there yet. Happy to review new takes on this if you have ideas. Would really be keen to have this solved at some point. Ideally it would stick there with a stable position without the need for any transitions.

Yea it is a heck of an issue and a lot harder then it seemed at first. What makes it hard for iOS is that there is no event triggered with the soft keyboard transition. If one could find and fire that event this problem would be trivial. I will do some more research, this was the first solution that came to mind :)

We are in the middle of the war native apps vs progressive web apps. :) There's no reason why the mobile browser should shift the fixed elements out of the viewport by default. And if there are valid arguments for doing that (can't see them), we should have a way to control this behavior. Hopefully this changes some time soon, so progressive web apps are straight-forward to do.
Anyways... I was just using Notes on the iPhone and maybe this pattern might also be interesting for us: No formatting tools by default until you enter a "Formatting mode" where instead of the keyboard there are formatting controls.

Needs a different implementation for mobile, but maybe this gives us more control for a smooth experience.
@nilskj would be up for meeting in person to discuss ideas and their feasibility? :)

Oh that is interesting, another thing I found is that you can long press on the text itself and access a little menu with formatting options. I can only see Bold, Underline and Italic options there though. Let's discuss this further on :)

authors: michael nilskj

repo: michael/editable-website


text:
Callback specified in on-page-change attribute is not getting called

I am working on a mobile app using Ionic Framework.
I want Prev, Next based navigation, so I have removed any code belonging to numbered navigation,
This how the pagination template looks,
<div class="tabs tabs-icon-only pagination" ng-if="1 < pages.length || !autoHide">

        <button class="button icon ion-arrow-left-c"></button>

        <button class="button icon ion-arrow-right-c"></button>

Pagination is working fine, but the on-page-change attribute on <dir-pagination-controls> does not seem to work.
Here is the code,
View
<dir-pagination-controls on-page-change="pageChanged(newPageNumber)" template-url="app/pagination/next-prev-pagination.tpl.html"></dir-pagination-controls>

Controller
```javascript
$scope.pageChanged = function(newPage) {
    console.log("inside pageChanged");
    if (audioService.getMediaStatus() == 1 || audioService.getMediaStatus() == 2 || audioService.getMediaStatus() == 3) {
        vm.isPlaying = false;
        audioService.stop();
    }
}
```
I have added the console statement for debugging. Though the navigation is working, nothing gets printed in the console.

Even tried with the alternative using $scope.$watch(), but seems to work only during page load.
Code for the same,

    ...
    ...

Controller
$scope.$watch('currentPage', function(newValue, oldValue) {
    console.log("inside watch");
    $scope.pageChanged();
});

watch() is fired only once during page load and not while navigating through pagination.
Any help on this.

Hi,
Thanks for the detailed post. Looking at the code you posted, I don't see any obvious issues that could explain this. I've never used Ionic so I don't know if it could be something peculiar to that framework.
Is it possible to reproduce the error in a plunker?

It was a syntax problem.
I had used ngController directive to bind controller class with the view and <dir-pagination-controls> was outside the controller div.
Code with error,

    ...
    ...

<dir-pagination-controls on-page-change="pageChanged(newPageNumber)" template-url="app/pagination/next-prev-pagination.tpl.html"></dir-pagination-controls>

After changing it to this, it started working

    ...
    ...
    <dir-pagination-controls on-page-change="pageChanged(newPageNumber)" template-url="app/pagination/next-prev-pagination.tpl.html"></dir-pagination-controls>

As I am new to angular, I was not aware of the concept of scope.

Ok, no worries and thanks for the update.

repo: michaelbromley/angularUtils


text:
Song is skipped after few seconds and make players out of sync.

Hi @mherger,
After upgrading squeezebox server and spotty, I have this issue described in title.
When I ask to play the whole album, first song plays for 4-5 seconds and it skips to next one. But players don't start the second one at the same time, so all others songs are played out of sync on the players.
If I do skip it the first one manually before the errors occurs, all other songs play just fine on each player.
It can be reproduced with this album: L'étrange pays, artist:  Jean Leloup
Logitech Media Server Version: 7.9.2 - 1565967976
Spotty: 2.8.4
3 players:

Odroid C1 with Max2Play / SqueezeLite
Raspberry Pi with Max2Play / SqueezeLite
Old Atom desktop with Ubuntu / SqueezeLite

Spotify Connect: checked for all players.
Before the upgrade I had
Logitech Media Server Version: 7.9.0
Spotty: 2.2.x

Please go to Settings/Advanced/File Types and disable Spotty -> OGG transcoding.

Thanks a lot. That was it.
Does it mean I miss some codecs on my ubuntu server?

It should be there, but unfortunately has proven to be buggy on some platforms.

authors: michaelherger noliveleger

repo: michaelherger/spotty


text:
Printer: I can only add items to the top bar, not to the buttons on the page

I have added the following printer creation callback function to the PostScript Printer Applications:
static void   ps_printer_extra_setup(pappl_printer_t *printer,
				     void *data)
{
  char                  path[256];      // Path to resource

  //snprintf(path, sizeof(path), "%s/getdefaults", printer->uriname);
  snprintf(path, sizeof(path), "/ipp/printers/%s/getdefaults",
	   papplPrinterGetName(printer));
  //papplSystemAddResourceCallback(system, path, "text/html",
  //			       (pappl_resource_cb_t)ps_printer_web_get_defaults,
  //			       printer);
  papplPrinterAddLink(printer, "Defaults from Printer", path, true);

  //snprintf(path, sizeof(path), "%s/installable", printer->uriname);
  snprintf(path, sizeof(path), "/ipp/printers/%s/installable",
	   papplPrinterGetName(printer));
  //papplSystemAddResourceCallback(system, path, "text/html",
  //			       (pappl_resource_cb_t)ps_printer_web_installable,
  //			       printer);
  papplPrinterAddLink(printer, "Installable Options", path, true);
}

There are several lines commented out, as I did not write the code for the extra web interface pages yet and also issue #97 is not solved yet (so the way to create the paths is most probably wrong for now).
The few active lines (the papplPrinterAddLink()calls) create items in the individual printer's web interfaces, but only on the printer's pages (not in the printer entries on the system's main page) and only in the top bar. For a consistent user interface I would like to add these items also to the buttons of both the printer's page and the printer entry in the system's main page.
Is this possible? If not, I appreciate the addtion of an extra function which does exactly this and which I could add to te above function.
Screenshot

Here one sees the items "Defaults from Printer" and "Installable Options" in the black top bar but not under the buttons near the icon.

OK, I have updated papplPrinterAddLink and papplSystemAddLink to use a enum bitfield that specifies where the links are shown, in addition to the old HTTPS option. This gets used in most places to insert buttons as needed (with three exceptions: supply levels, identification, and test page since those depend on the current printer driver data...)
[master 17896d4] Update papplPrinter/SystemAddLink to accept an options bitfield (Issue #98)

authors: michaelrsweet tillkamppeter

repo: michaelrsweet/pappl


text:
图一含义

（1）不明白图一和论文的关系。（2）图一左右两边想表达什么意思。右边能表达更复杂的动力学行为？

Dear Xiao Zhang,
Could you please post your question in English? None of the authors understand Chinese.

authors: XiaoZhang-NN pierreablin

repo: michaelsdr/momentumnet


text:
View preferences

Prepares the app for visual customisations by adding a 'View' menu item as well as some placeholder code to save and load preferences from UserDefaults. Added an isMainController field so that only changes made in the first window are saved.
This PR serves as a base for some other PRs I'm about to open.

That's for the series of PRs @vin047. Could you attach some screenshots to all the PRs?

Done (in case you missed the notifications 😅).

repo: michaelvillar/timer-app


text:
http.nonProxyHosts system property ignored

Hi,
It seems the plugin ignores the properties http.nonProxyHosts and https.nonProxyHosts.
For example, I get a HttpHostConnectException if I try to download a file from Google and I pass non-existent proxy settings, while ignoring *.google.com at the same time:
gradle downloadFile -Dhttp.proxyHost=localhost -Dhttp.proxyPort=999 -Dhttp.nonProxyHosts=*.google.com -Dhttps.nonProxyHosts=*.google.com

Thanks!

Yes, I can confirm this. Thanks for reporting.
I'll try to implement this feature when I've got some time, but it might take a couple of days/weeks. Pull requests are very appreciated :-)

Yeah, I'm already starting to fiddle around with the code. I'm not a very big expert on HttpClient, but hopefully I'll have something to send you for review soon :blush:

I'm closing this one in favour of #63.

@michel-kraemer I think we should keep this open as #63 seems to be closed :(

@timotei What do you mean? There was a pull request #68 which has been merged. So both issues can be closed now. If you have a problem related to this feature please open a new issue.

Sorry. I meant #72. This was the PR that was merged at the end.

@michel-kraemer Oh. OK. I haven't seen anywhere the link to #72, that's why I was confused.
Thanks for the clarification!

authors: Ithildir michel-kraemer timotei

repo: michel-kraemer/gradle-download-task


text:
When kwargs are used, option values cannot have "="

I know this is painful to fix, but I call it a bug anyway because it is very unexpected: users would think it easy to know that "--opt", requiring one option, will take the "abc=" below.
$ cat test_plac.py
#!/usr/bin/env python
def func(opt: ('opt', 'option', 'o') = 'foo', **kw: ('kw',)):
    print(opt, kw)
import plac
plac.call(func)
$ python test_plac.py --opt abc=
usage: test_plac.py [-h] [-o foo] [kw [kw ...]]
test_plac.py: error: argument -o/--opt: expected one argument

Fixed in master, https://github.com/micheles/plac/commit/f3e28a6868004d3a424c6bd14d3327f216733bdf

authors: isaacto micheles

repo: micheles/plac


text:
API docs?

Hi, I'm new to Typescript.
Is there a way to generate API docs?
I see https://github.com/michitaro/healpix#progress but to be able to use it I have to look at the code what the inputs / outputs are. Is that how it's usually done for Typescript?
If there is a way to add docs and you want it here, this could be something I contribute to a bit while I familiarise myself with TS and this package?

I've generated docs by typedoc.
https://michitaro.github.io/healpix/typedoc/modules/_index_.html
There are no comments yet.

Thanks!

authors: cdeil michitaro

repo: michitaro/healpix


text:
Unexpected behaviour using different QoS settings

Unexpected behaviour using different QoS settings

Hardware description: Crazyflie 2.1 w/ flow deck and LPS deck
RTOS: freeRTOS
Installation type: micro_ros_setup for both the agent and the firmware
Version or commit hash: foxy

Steps to reproduce the issue
I have pushed the necessary files to this temporary repo.

Build the workspace and agent using the micro_ros_setup scripts (I have tried with docker as well and the results were the same).
Configure firmware to use the custom uros app that can be found here . This app basically subscribes to a /microROS/ping topic and publishes a response to a /microROS/pong topic. The message type is a custom type which is an array of uint8_t that i have defined in cf_messages and added to mcu_ws. A ROS2 node to measure the latency (RTT) is present in the eval_ws
In order to avoid issues due to radio communication i am using USB, therefore i have modified the microros_transports.c file and the bridge to use the USB llink but the results still apply to radio.
Build the firmware and flash it to the crazyflie.
Run the python bridge, i have used the no joystick version with modifications to print the console as well.
Build the agent according to the micro_ros_setup repo and run it ros2 run micro_ros_agent micro_ros_agent serial --dev /tmp/ttyS10 --baudrate 115200 -v6 im using tmp to avoid needing sudo
Build, source and run the latency_test (making sure the QoS matches the on in the app)in the eval_ws ros2 run csv_creator latency_test --ros-args -p num_samples:=100 -p num_bytes:=256 -p frequency:=1 -p id:=2 -p filename:="random_name" or use the automated script with variable number of bytes.
Change the qos in both app and ROS2 node and compare the results using RELIABLE vs BEST_EFFORT QoS

Expected behavior
The latency using both profiles should be similar or slightly lower in one of them (probably lower in best effort) and increases with the number of packets sent (CRTP uses 31 bytes of payload per packet and has an overhead due to the XRCE serial framing)
Actual behavior
The latency with RELIABLE QoS increases with the payload  as expected but the BEST_EFFORT results are random, with significantly higher latency ~110ms vs ~25ms for 8B payload using USB with no clear change with respect to payload size and with significant message loss (i suspect the crazyflie receives but doesnt send back the pong)
Additional information
I also think the timeout to read from serial should be lower as the 200ms that are used are significantly higher than the RTT of the radio link which takes around 7ms meaning it is better to send more packets than wait for sufficient data from the serial stream.

Is this issue still valid? Maybe https://github.com/eProsima/Micro-XRCE-DDS-Agent/pull/262 is related.
@ruimscarvalho98 can you give us an update?

Maybe eProsima/Micro-XRCE-DDS-Agent#262 is related.

Hey @pablogs9, I am a bit busy this week but I'll try to test this when I find some time and give you an update on it

Closing due to inactivity, reopen if needed.


text:
Improve CI build

Uses a new base image: microros/ci:dashing to build firmware clients.
This base image has the toolchains pre-downloaded it should avoid the spurious download error.

Looks like it's not finding the gcc?

Looks like it's not finding the gcc?

True. Some path issues. I will push some changes.

TODO: add flashing tool check to CI in order to check if e.g. dfu-utils dependency is resolved

@jfm92 are you aware of this CI? I'm removing my assignment and assigning you

This PR was addressed at #86.

authors: BorjaOuterelo iluetkeb julibert pablogs9

repo: micro-ROS/micro-ros-build


text:
Update README.md

grammar

thanks!

authors: BrianHe melvincarvalho


text:
Add explanation for agglomerateByPrevalence

This PR aims to solve the issue (https://github.com/microbiome/mia/issues/407) by making the function agglomerateByPrevalence easier to understand :

Specify the returned value
Add details on the way the function works
Clarify the examples

Very nice. I think we could still move it under agglomerate-methods with agglomerateByRank and agglomerateByVariable
I think this is more "agglomeration" than "prevalence" function, so that would be the best place for it

If you check the man page of agglomerate-methods, it does not describe detection and prevalence and as_relative parameters. (They are passed through with ...). You could add to agglomerate.R file those parameters under ... parameter.
" Passed to getPrevalence and getPrevalentTaxa functions. Used in agglomeratebyPrev function..." (Along with the current text.
As agglomerate.R already has ... parameter, you cannot ... above agglomerateByPrevalence function (otherwise it overwrites the one in agglomerate.R).

See the man page. You could modify description so that it describes the agglomeration functions in more general (it now only says about agglomerateByRank) --> Agglomeration functions can be used to sum-up to rank or variable or prevalence...

authors: TuomasBorman thpralas

repo: microbiome/mia


text:
Implement migrator that renames the has_calibration field of MassSpectrometry documents to generates_calibration

Write a migrator that does the following:

For each document in the data_generation_set collection, if that document represents an instance of the MassSpectrometry class (i.e. its type field consists of nmdc:MassSpectrometry) and that document has a field named has_calibration, rename that field to generates_calibration.

Note: The associated schema change (PR linked below) introduces a uses_calibration slot into the MetabolomicsAnalysis class and the MetaproteomicsAnalysis class. No migration is necessary for the introduction of this slot because it is not taking the place of an existing slot and its value is not required.
The associated schema change is being introduced via the following PR: https://github.com/microbiomedata/nmdc-schema/pull/2282
CC: @aclum

I'll implement this now and open a PR. If PR #2282 is still open (unmerged) by the time I open a PR for this migrator, I'll make that PR branch be the base branch of the migrator's PR branch.

repo: microbiomedata/nmdc-schema


text:
The introduction is now always portable text.

Tidying after migration for https://github.com/microbit-foundation/python-editor-next/issues/332

This looks good locally and the code change seems fine. I'll review the PR branch in the morning.

repo: microbit-foundation/python-editor-next


text:
Could you add going to the gym as a scenario?

It would be great if you could add going to a gym or exercising to your scenario choices. Variables could include:

indoors or out
ventilation -- no windows or windows that don't open with 1 door; door(s) and windows that open; fans (overhead or floor); hepa filter/air cleaner (whatever you call it)
activity -- high (cardio), medium (weight work), low (yoga)
masked or not
approximate size of the room
if taking a class, number of people in the class
how much space between you and people near you

I found this article in the NY Times very helpful when I was deciding which activities to continue at my gym --
https://www.nytimes.com/2021/03/22/well/move/exercise-classes-gym-coronavirus-covid.html
Thanks to the article I'm still going to my chisel (weights) class but not to my spinning class, which is held in a small room with windows that don't open, 1 door and overhead fans that mostly move the air around.
Thanks --
Amy Mayers

Hi Amy,
Looking at this already over in #962 :)

Great! Thank you!
On Thu, Aug 12, 2021 at 6:58 PM Jenny @.***> wrote:

Hi Amy,
Looking at this already over in #962
https://github.com/microcovid/microcovid/issues/962 :)
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/microcovid/microcovid/issues/965#issuecomment-898024896,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AVF6RKF6EFVPLAPZ6SN3BELT4RGYXANCNFSM5CCKJ2NQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&utm_campaign=notification-email
.

--
Amy M. Mayers
o: (202) 363-2537
c: (202) 236-7328
amymayers.com

I'm going to close this as a duplicate to keep things tidy; if you're interested in sharing "field reports", I'd appreciate them over in the other open issue!

Thanks!
On Fri, Aug 13, 2021 at 2:33 PM Jenny @.***> wrote:

I'm going to close this as a duplicate to keep things tidy; if you're
interested in sharing "field reports", I'd appreciate them over in the
other open issue!
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/microcovid/microcovid/issues/965#issuecomment-898647680,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AVF6RKH2UP4AU6EHFWVBXLDT4VQQ7ANCNFSM5CCKJ2NQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&utm_campaign=notification-email
.

--
Amy M. Mayers
o: (202) 363-2537
c: (202) 236-7328
amymayers.com

repo: microcovid/microcovid


text:
Android O+ account restrictions message

@mar-v-in
On 30ed2720a0ecd3067fe760e801ecbdbe1b5ff5be the message is "Allow to bypass Android O+ account restrictions for Google Apps" but inside the code there is "When enabled, all applications on this device will be able to see email address of your Google Accounts without prior authorization."
So, does it apply to all apps or only to Google Apps?

The functionality technically applies to all applications that have GET_ACCOUNTS (aka Contacts) permission. It is mostly useful for Google Apps, as other apps do not tend to blindly believe they have permissions to access the account when they don't. However with this option enabled there is nothing preventing them from accessing the email address (if they have GET_ACCOUNTS permission). This was the default behavior on Android before Oreo and still is the default behavior for apps that target any older version at compile time, so the impact on privacy is limited (as there are other ways to get the account email address and you still need the permission anyway).

Does this affect #790?

Yes, this is a workaround for the issue that arises with Google Apps that are not able to discover the Google Account.

@mar-v-in, could I ask if it is possible to have instant push notifications instead of "Ping interval 60 seconds, Automatic: 5 minutes and other"?

Are the Google apps that could need this bypass different from those in KNOWN_GOOGLE_PACKAGES?  If it's deemed appropriate to whitelist them, perhaps a similar approach could be used?
@caneleex Your question seems completely off topic to me.  If you're interested in notifications that are not delayed, you may be interested in Doze support that will be in the next version.  See #732.

@mar-v-in: It is possible to have an option to limit it to KNOWN_GOOGLE_PACKAGES?

Many Google Apps are affected by this issue, not only those currently in KNOWN_GOOGLE_PACKAGES, but also some non-Google apps (chromium forks). Also it's really a mess to implement this on a per package basis. As this only recovers the original Android before Oreo behavior (including check of GET_ACCOUNTS permission), it's not a huge issue to allow it for all apps. And after granting access to an app to use a specific account, you should also be able disable the option again (is only required to discover the account, not to use it).

@voidstarstar any eta?

@mar-v-in
Ok, it sounds like a per package whitelist would not be feasible then.
From what I understood:

Lollipop and earlier would blindly grant all permissions at install time without a way to revoke them.
Starting with Marshmallow, permissions were separated into "normal" and "dangerous" categories.  A normal permission would be granted blindly, similar to Lollipop.  A dangerous permission would prompt the user at run time to allow it at a granular level and the user would later be able to revoke it.  The GET_ACCOUNTS permission is in this "dangerous" category due to the privacy issues with giving out this info to apps.
Unfortunately the Marshmallow permission changes were only enforced if apps targeted Marshmallow or newer, so any apps targeting Lollipop or earlier would have their run time permissions blindly accepted.  So starting with Oreo, the permission prompts were actually enforced regardless of the targeted version to prevent this loophole.

Maybe I misunderstand what this patch does.  Would this new option prevent the user from getting prompted for the GET_ACCOUNTS permission and instead blindly allow it for all apps (including apps that target Marshmallow and newer)?  If so, isn't this the same behavior as pre-M, not pre-O?  As long as this is implemented as an option that is off by default, I think it would be fine for now, but it could possibly be improved in the future.
@caneleex I do not know.  You can still compile it yourself to get it today, however.

Since Android O, apps targeting O+ will not be able to see accounts even after GET_ACCOUNTS was granted by default. Apps targeting pre-O are not affected and will be able to see accounts as soon as GET_ACCOUNTS is granted. Apps targeting pre-M will be granted GET_ACCOUNTS automatically and be able to see accounts directly.
This feature only affects Android O+ and apps targeting O+ after they were granted GET_ACCOUNTS permission.

Thanks for clarifying.  I misunderstood the issue.  Sorry about that!

@voidstarstar how?

@caneleex https://github.com/microg/android_packages_apps_GmsCore/wiki/Building
This is off topic.  If you have questions about when the next release will be, you should take them to #788.

Since the issue is now clear and the new version with the fix is released today I'm closing it now.

authors: Eerovil ale5000-git caneleex mar-v-in voidstarstar


text:
🔧 Fixing the demo build

The demo doesn't work.
If we run the npm run build:demo command and if we open the resulting demo in the browser, we get a blank page with an error.

It doesn't work because:

We had a bug in the demo component, we were trying to spread a null value.
CDN links were referring to react 15, this version doesn't have forwardRef yet. It was introduced in React 16.3 - https://github.com/facebook/react/blob/main/CHANGELOG.md#react-17

In this PR I'm fixing the demo build.

https://github.com/user-attachments/assets/75124729-f995-4e70-b182-54965f447c28
An example of the fix

repo: microlinkhq/react-json-view