statusline and tabline can't be made transparent

environment

vim: 8.1.1137
vim-airline: master branch as of 2019-05-02 02:14:57
vim-airiine-themes: master branch as of 2019-04-11 01:53:43
OS: Fedora 29 (Linux 5.0.9-200.fc29.x86_64)
Have you reproduced with a minimal vimrc: yes; see below
What is your airline configuration: see below
if you are using terminal:
terminal: gnome-terminal (for screenshot below); also urxvt256c-ml
$TERM variable: xterm-256color (for screenshot below); also rxvt-unicode-256color
color configuration (:set t_Co?): t_Co=256
if you are using Neovim: not using nvim

vim config (complete file!)
" ~/.vimrc
call plug#begin('~/.vim/plugged')
Plug 'vim-airline/vim-airline'
Plug 'vim-airline/vim-airline-themes'
call plug#end()
let g:airline_theme = 'base16color'
let g:airline_powerline_fonts = 1
let g:airline#extensions#tabline#enabled = 1
actual behavior
I want the airline statusline and tabline to be able to be made transparent (specifically, the background of section C) in a transparent terminal.
I have tried 2 different terminals (gnome and urxvt), and both exhibit the same behavior. I have also tried 2 airline themes (base16color.vim and term.vim).
In each case, if I edit the theme vim file by changing the statusline setting from 234 to 'NONE', I get the desired behavior of a transparent statusline and tabline. However, thick arrow powerline glyphs show up in a weird color, and I don't know how to make them go away.
When not using powerline fonts, of course there is no issue, but those glyphs should be transparent themselves (they seem more like spacers).
I am providing two different images here so you can see what I mean. The screenshots were saved to .png files to maintain transparency, but then I opened them up again in gimp to screenshot them again so that you can see the transparent (checkerboard) areas.
(sorry in my screenshots below I wrote 'tagline' instead of 'tabline')
Original:

Transparent (showing the problem):

expected behavior
There should be a way to make the statusline and tabline background of section C transparent (while still using powerline fonts), without weird-colored glyphs showing up.
Thank you! Your help is very much appreciated. :)

The whole idea of the separators is, that they automatically show a nice transition from foreground to background color so that the next sections foreground looks nice. This does not really make sense for a transparent background. I suggest to not use separators if you want to use a transparent terminal.

Hi @chrisbra , thank you for the quick response. :)
I certainly value and respect your opinion, however I think my opinion (which others might share) is still a valid one. I personally feel that the powerline arrows in the foreground on top of a (semi) transparent background looks nice. (Aside, the example I'm showing you here is very basic with the checkerboard, but with my own background and colors it's much nicer.)
For example, I changed the right separator myself here:

The green left separator arrows look fine, and I'd love for the right separator arrows to look similar.
Anyway, if you look at example sections A, B, and C, there are things I don't understand...
In example Section A:

, why is separator 1 and separator 2 two different colors? In my original screenshot, sep 2 displayed just fine, and it was sep 1 that didn't make sense even being there (this is part of why I considered it a bug). Sep 2 is the arrow, and sep 1 is what made things look bad. What's the purpose of sep 1 being there, other than it being a "spacer" of sorts? (considering sep 2 is the arrow)
In example Section B:

, if there was a way for me to remove these separators - or better yet, be able to assign them a color or transparency, then there wouldn't be a problem. But if I set the single "g:airline_right_sep" property, it gets applied everywhere. If I could make just those seps go gray when inactive, then it wouldn't be a problem.
In example Section C:

, things are even more strange. Separators 1, 2, and 3 are all different colors (foregrounds and backgrounds). Sep 1 is white with transparent bg, Sep 2 is black with transparent bg, and Sep 3 is green with black bg. But really, it's the white Sep 1 causing the most trouble. If it could match either sep 2 or 3, that would be a lot better. Or even if it could be overriden separately from the rest, but again... only the one "g:airline_right_sep" property gets applied everywhere.
Anyways, thanks for reading this far. Would you please consider leaving this issue open, so that others might be inclined to weigh in with their opinions, or perhaps it can be discussed with your fellow maintainers? I do think a transparent background statusline and tagline's attractiveness is a valid opinion.  Thanks once again for your time.

Just thought I'd drop in a screenshot of how it actually looks for me, and the couple glyphs I'd like to get rid of (and how nice it would look without them). You can see I also use tmuxline in concert with airline, and it does the transparency nicely.

It won't be on my priority list very high, as there is a lot of other things I have to take care, but let me re-open it and see if others chime in.
Please show the output of :echo eval(&statusline[2:]). Also if you need a special theme to make this work, please consider adding one to the vim-airline-themes repository.

Thank you @chrisbra ; much appreciated.
Output of :echo eval(&statusline[2:]) ...
%{airline#check_mode(1)}%#airline_a#%( %#airline_a_bold#%{airline#util#wrap(airline#parts#mode(),0)}%#airline_a#%{airline#util#ap
pend(airline#parts#crypt(),0)}%{airline#util#append(airline#parts#paste(),0)}%{airline#util#append(airline#extensions#keymap#stat
us(),0)}%{airline#util#append(airline#parts#spell(),0)}%{airline#util#append("",0)}%{airline#util#append("",0)}%{airline#util#app
end(airline#parts#iminsert(),0)} %)%#airline_a_to_airline_b#%#airline_b#%#airline_b_to_airline_c#%#airline_c#%( %<%f%m %#airlin
e_c_red#%{airline#util#wrap(airline#parts#readonly(),0)}%#airline_c# %)%=%#airline_c_to_airline_x#%#airline_x#%( %{airline#util#
prepend("",0)}%{airline#util#prepend("",0)}%{airline#util#wrap(airline#parts#filetype(),0)} %)%#airline_x_to_airline_y#%#airline
_y#%( %{airline#util#wrap(airline#parts#ffenc(),0)} %)%#airline_y_to_airline_z#%#airline_z#%( %3p%% %#airline_z_bold#%{g:airline
_symbols.linenr}%4l%#airline_z#%#airline_z_bold#/%L%{g:airline_symbols.maxlinenr}%#airline_z# :%3v %)%#airline_z_to_airline_warni
ng#%#airline_warning#%( %{airline#util#wrap(airline#extensions#whitespace#check(),0)} %)
Also, good idea about me adding a theme. I'll see what I can do, given it would be my first attempt. :)

Hmmm... That copy/paste of the echo doesn't include the glyphs. I'll post another screenshot here:


Add RefGet support

RefGet is a GA4GH standard for retrieving reference genomes by name or checksum. It seems natural for RefGenie to support this protocol given that it can serve fasta references.

Thanks for the feedback, we're thinking the same thing! I was thinking about adding refget checksums as an attribute for the genomes.
There are two issues at the moment: First,  the refget protocol is currently concerned with individual sequences rather than collections, but this is apparently on the docket for refget v2.  The other thing is... by my understanding of the refget protocol, you can retrieve sequences themselves given a checksum, but actually not by name (which will be hard to do because of non-congruence of names...). So, our use case is again slightly different than the current protocol. I am in communication with @andrewyatz about how we can link the ideas of refgenie and refget. I'm really hopeful that we'll be able to link these two projects together, which I think will go a long way toward standardizing genome sequences and accompanying assets. But we're kind of just starting to image the best path forward.

Hi. To give some background here. The aim of refget will always be retrieval by checksum as it’s one of the few ways we’ve found to make a totally unambiguous call.
What we are looking at supporting though is sequence collections (the concept of a genome or transcriptome or proteome) and checksum discovery using other known identifiers or aliases. My hope is both of these new protocols will eventually boil down to “and here are the checksums”.
I can imagine others building far more complex protocols and utilities on top of this but these feel like the building blocks we need.

Thanks @andrewyatz
I think refgenie will be able to easily build on top of refget to add in the capability of retrieving things by name, which I'm starting to brainstorm about in #81. Because of the 'config file' approach of refgenie this seems to fit naturally here, so I think it's a great complement to refget.
I have also started brainstorming a few ideas about the collection-level checksums in #82. I will be interested in hearing more about your approach to that.

I've implemented basic 'getseq' support now in this commit: ea877584e8ad91eab369343466d2ca5248dd51ca.
it works like this on dev:
refgenie getseq -g hg38 -l chr1:50000-50200
AAACAGGTTAATCGCCACGACATAGTAGTATTTAGAGTTACTAGTAAGCCTGATGCCACTACACAATTCTAGCTTTTCTCTTTAGGATGATTGTTTCATTCAGTCTTATCTCTTTTAGAAAACATAGGAAAAAATTATTTAATAATAAAATTTAATTGGCAAAATGAAGGTATGGCTTATAAGAGTGTTTTCCTATTGTT

at the moment it is not yet using the refget/hashing...it just uses the refgenie local human-readable identifier for genome.

@jdidion maybe of interest: I've spun off refget functionality into a separate python package called refget -- we're now working to incorporate this into refgenie for sequence identifiers.


Break up Labcam after short press for voice note recording

Testserver: qa-gluons
Test date: 22.02.2017
Device: Tablet Nexus 9
User: standard user
Action: record voice note
-> open voice note window -> short press on microphone icon
Observation: ->"unfortunately, Labcam has stopped"
Expected result: The correct handling would be "long press", so I would expect an info box with something like "recording wasn't successful" and return to microphone icon

I have tested it. Now if user shortly press the button, there will be a pop-up showing up to remind user, that he has to long press to record voice note. LabCam will not crash if user only shortly press the button.


Issuing credits to Omo

Issue 2000 credits to @Omoaholo.

 Follow up
On Wednesday, 27-Mar-24 13:19:28 UTC by Gov4Git dev
Issued 2000 credits to member @omoaholo.


Replace tempita with Jinja2

Fixes #10204
WIP:

[x] Render tutorial
[x] Remove tempita from HTML templates for Jupyter
[ ] Changelog entry

Nice: VS Code has debugging support for Jinja templates:

One circle job failed because MNE-BIDS depends on our censored Tempita. This will require an update of MNE-BIDS and can be ignored here for now.
All other checks passed.
I will update the changelog ASAP; in any case, this PR is ready for review.

Removed Tempita in favor of Jinja2
HTML repr of Info is more robust
I could remove some manual string escaping from the Python code, as Jinja2 will do that work for us now
All HTML templates are now located under mne/html_templates

This broke mne-bids :)
In [1]: import mne_bids
---------------------------------------------------------------------------
ImportError                               Traceback (most recent call last)
<ipython-input-1-b553cc368f26> in <module>
----> 1 import mne_bids

~/Documents/github_repos/mne-bids/mne_bids/__init__.py in <module>
      3 __version__ = '0.10.dev0'
      4 from mne_bids import commands
----> 5 from mne_bids.report import make_report
      6 from mne_bids.path import (BIDSPath, get_datatypes, get_entity_vals,
      7                            print_dir_tree, get_entities_from_fname,

~/Documents/github_repos/mne-bids/mne_bids/report.py in <module>
      9
     10 import numpy as np
---> 11 from mne.externals.tempita import Template
     12 from mne.utils import warn, logger, verbose
     13

ImportError: cannot import name 'Template' from 'mne.externals.tempita' (unknown location)

This broke mne-bids :)

Already fixed: https://github.com/mne-tools/mne-bids/pull/944


% and # in the URL

The package did not work with files and folders in the name of which there are the characters '#' and '%'.
Correction checked in work with webdav from cloud.mail.ru

hi artem.
thank you for your pull request!
i think it is not enough, there are more reserved characters.
it would be nice to have a test.
i need to do more research on this topic.
stay tuned.

escaping the full path in commit 375f391c8ad875587e81f1584f5fe4a4f30ee034
thanks

Excellent! Thank you!


TypeError: Object # has no method 'restore' at Gulp.

When trying to build, the aforementioned error is output.
$ npm run build

> vanilla-bootstrap@ build /media/www/forums/themes/bootstrap
> gulp

[10:07:53] Using gulpfile /media/www/forums/themes/bootstrap/gulpfile.js
[10:07:53] Starting 'styles'...
[10:08:01] 'styles' errored after 8.59 s
[10:08:01] TypeError: Object #<StreamFilter> has no method 'restore'
    at Gulp.<anonymous> (/media/www/forums/themes/bootstrap/gulpfile.js:23:18)
    at module.exports (/media/www/forums/themes/bootstrap/node_modules/gulp/node_modules/orchestrator/lib/runTask.js:34:7)
    at Gulp.Orchestrator._runTask (/media/www/forums/themes/bootstrap/node_modules/gulp/node_modules/orchestrator/index.js:273:3)
    at Gulp.Orchestrator._runStep (/media/www/forums/themes/bootstrap/node_modules/gulp/node_modules/orchestrator/index.js:214:10)
    at Gulp.Orchestrator.start (/media/www/forums/themes/bootstrap/node_modules/gulp/node_modules/orchestrator/index.js:134:8)
    at /media/www/forums/themes/bootstrap/node_modules/gulp/bin/gulp.js:129:20
    at process._tickCallback (node.js:448:13)
    at Function.Module.runMain (module.js:499:11)
    at startup (node.js:119:16)
    at node.js:935:3

npm ERR! Linux 3.13.0-58-generic
npm ERR! argv "node" "/usr/bin/npm" "run" "build"
npm ERR! node v0.10.40
npm ERR! npm  v2.13.3
npm ERR! code ELIFECYCLE
npm ERR! vanilla-bootstrap@ build: `gulp`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the vanilla-bootstrap@ build script 'gulp'.
npm ERR! This is most likely a problem with the vanilla-bootstrap package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     gulp
npm ERR! You can get their info via:
npm ERR!     npm owner ls vanilla-bootstrap
npm ERR! There is likely additional logging output above.

npm ERR! Please include the following file with any support request:
npm ERR!     /media/www/forums/themes/bootstrap/npm-debug.log

Duplicate of #177.


plugin broken?

Hi,
somehow world news plugin does not work any longer. Entries text is partially scrambled, many live tv links don't work, some point to different channels than written.
Several deinstallation/reinstall attempts did not help, nor did deleting the world news live folder in .kodi/user_data/addon_data.
All other plugins work.

There is a new version out now but i don't think your issue with the text is coming form the plug-in. If you still see this problem post back with Kodi version.

Thanks for the reply!
Funny thing is, the problem appears on two laptops, kodi 15.2 installed on top of xubuntu15.10.
It even happens on a complete new .kodi subfolder, so probably has something to do with the underlying system...

I just installed kodi from launchpad on a different computer, installed world news plugin via the fusion installer, and it looks the same - scrambled text (partially). In BBC world news, the three stream that are there look
xe
xea
xeam

sry, closed unintentionally...

I'm seeing this as well.
Ubuntu 15.10, WWN 1.5.1, Kodi 15.2.
Start of news list is:
[My Custom Streams]
a
ABC News 24
a
Al Jazeera Arabic
Al Jazeera English
antena3.
...

My list looks like this:
[My Custom Streams]

ABC News 24

Al Jazeera Arabic
Al Jazeera English
antena3.
etc.
So there are different letters instead of the original entries. Could that point to an encoding issue?

I tried different LANG=, LC_ALL etc. settings, but they didn't help.

Seems that both of you are running it on top of Ubuntu 15.10.
I'll need to install and test it on the new Ubuntu version.

Thank you, and thank you for a great plugin, by the way.
Please tell me what I can do to help diagnose or test.

@nikolajsheller If you can dump a log in xbmc when starting WNL that could help.

Kodi+WNL start:
kodi.log.zip

Same problem with the source code in git.
System details:
Starting Kodi (15.2 Git:Unknown). Platform: Linux x86 64-bit
Using Release Kodi x64 build
Kodi compiled Dec  7 2015 by GCC 5.3.0 for Linux x86 64-bit version 4.1.4 (262404)
Running on Arch Linux rolling, kernel: Linux x86 64-bit version 4.2.5-1-ARCH
FFmpeg statically linked, version: 2.6.4-kodi-2.6.4-Isengard
Host CPU: AMD A10-7850K APU with Radeon(TM) R7 Graphics, 4 cores available
Logfile: https://gist.github.com/sgargbugreporter/b784f63da10a462f6313
Preliminary: the URLS are wrong for BBC. Return a 403 (Forbidden)

For me, the problem is now gone on Yarvis (Kodi 16).
The plugin just works now :)


Stop/Start commands

It would be useful to set the backrest process as active/inactive state to aid in failover scenarios.  This would allow cron jobs to be running on a primary or standby and only run on the server that is currently active.  It's also useful for a corosync / pacemaker environment to inactivate the primary and activate a standy.
There will be two new commands:

start - allows all commands to run.
stop - returns an error if any commands are run.  The --abort option will also abort any currently running processes.  --stanza can be used to limit this to a single stanza while the absence of that option will affect all backrest processes running on the system (but not affect other processes in the enterprise even if they share a backup server).

Implemented in ddd4c14.


Expose a "retransmitted bytes" counter

Exposes a counter for the number of bytes retransmitted due to resending after receiving NAKs.
Caveats (shown in 44e89dfd7539ea2fa1f6a82655c119ef2681ab97):

Only counts bytes sent to multiple MDC destinations once
Retransmitted bytes are not included in total bytes

I looked into addressing those caveats, but I couldn't decide on the best approach.
Would it be best to revert the disabled tests? Whilst it is useful documentation, it will probably rot.

LGTM


You can never delete a exec process before it's started.

Sometimes we need to delete a exec process when it's in created state, e.g. doing cleanup when start fails for some reason.
However, currently there is no way to delete a exec process in created state.

If you use process.Delete(ctx), it will tell you process must be stopped before deletion.
Even if you Kill the process, because the pid is 0, it will return directly and the process will still be created.
If you use process.Delete(ctx, containerd.WithProcessKill), because the process is in created state, Wait will start waiting for TaskExit event. However, because there is no such event generated, it will hang forever until context deadline exceeded.

/cc @crosbymichael @mlaventure
/cc @abhinandanpb

@crosbymichael Thanks!
I'm not sure whether we want to make process exactly the same with task, because created state of process doesn't have a corresponding process running, which is different from task.
I've not idea how to properly fix this yet, :p Wait for your PR then.

@Random-Liu done.  Its just a validation issue for execs.


Implement passwordLength field for service hosting generated accounts

Hey guys. First off, I implemented a working testConnection function for DirectAdmin, as the previous implementation did not work.
Secondly, I ran into an issue where my DirectAdmin installation has a minimum password length of 12 characters, while the FOSSBilling password generation only generates a string up to 10-characters long. I thought about a few different implementations, and ultimately landed on implementing a generic passwordLength field, usable by all of the service hosts.
I've made the change everywhere it seemed necessary (including the SQL migrations); however, it's clear that I've missed something as the value does not get submitted to the database, and therefore does not get retrieved either (regardless of it being done in the creation/or update phase).
I'd appreciate an overall review of this PR. Any suggestions/recommendations on a better implementation would be greatly appreciated! I haven't written PHP in approximately 5 years, so apologies if there's something obvious I've missed.

@BelleNottelling
I've left it as a draft as the passwordLength field doesn't currently get properly stored in the database, which I'm sure is something simple I'm overlooking.
To clarify my stance on alternative implementations, DirectAdmin allows you to configure a minimum and maximum password length. Similarly, you can do the same for usernames (which I suspect will be my next issue). This is why I ultimately landed on this implementation, as no default values would work. The only alternative in my eyes would be to note the maximum length returned by the API on error, regenerate the password at that stage, and then re-attempt the request. That feels incredibly messy to me, though.

I've just noticed an edge case in the existing DirectAdmin implementation. As it replaces the stored user account's password with asterisks, uncancelling an order results in the password being set on DirectAdmin as such. It looks like WHM and Plesk have different implementations, which I'm looking over now. Otherwise, I'll just re-gen the password on uncancel

TL;DR for the above:

Fixed issue with all service host adapters where upon an order being uncancelled, the account's password would be remotely set as asterisks
Implemented DirectAdmin connection testing that works
Implemented DirectAdmin package select (you can manually specify a custom package value, which will now be checked against the DirectAdmin server)
Implemented passwordlength configurable value for all service host adapters

Fixed issue with all service host adapters where upon an order being uncancelled, the account's password would be remotely set as asterisks

That's a good find & fix, thank you.
I'm honestly not sure why we haven't deleted that DB row entirely yet, but that can be done in a separate PR

Thanks for the fast review! I really appreciate it :)
I believe I've satisfied your comments, though please do double check! I completely agree on moving to a field containing JSON approach, and I'd be happy to help with that as/when needed.
As a minor sidenote, for context I would have preferred naming the field passwordLength instead, but the only other field on the model containing multiple words (accesshash) didn't follow camelcase, so 🤷 I noticed that there's a real mix of camel and snake case throughout the codebase. I did change a few things here and there, and modified a few single letter variable names to be more descriptive. I'd love to know (and apologies if you've already noted it elsewhere) which naming convention the codebase should be using.
As I integrate FOSSBilling more tightly into our infrastructure, I suspect I'll be spending more time helping out. Thanks again!

I'm confused about the CI tests failing, as I can't see an obvious issue within this PR. Again, my apologies for my lack of familiarity with PHP, it's been a while since I've worked with it like this. Any insight would be greatly appreciated!

I noticed that there's a real mix of camel and snake case throughout the codebase

Indeed, you can thank BoxBilling for that. For new code / heavy rewrites I'd definitely go for camelCase, however in cases where you're updating code I think it would be fine to do either camelCase or follow what's already in the surrounding code.

I'm confused about the CI tests failing

Pulled the branch. It looks like this is a great example of the many poorly written tests from the BoxBilling project.
Many of them either don't test much, are implemented wrong, or I've even seen cases of a test that required broken behavior to pass.
In this case, it's a test that's not correctly implemented and it's not mocking all of the required data for it to pass.
I typically have been removing these tests as they've proven to be largely unhelpful. Feel free to remove it or to correct the testcase itself if you want, either one is fine.
The longer-term goal is to actually deprecate and eventually remove these tests entirely and instead perform tests via live docker instances where there's an actual config deployed with a real database as the current ones simply mock everything and are nearly useless for actually finding problems.

Hah, go figure! In which case, I may rework this and switch over to camelCase. Might as well get it right the first time for new code! I may spend some time tonight/tomorrow doing so, as well as reworking more of the DirectAdmin implementation.
I'll also apologize for doing many different things within one PR. I tend to do that when first contributing to a project, since I end up stumbling across 101 things that could be improved (such as docs).
Once again I appreciate your fast reviews and responses! I'll get those changes done tonight/tomorrow, and hopefully you and another person will be able to review them soon :) Thanks!

@BelleNottelling sorry to tag! Were you waiting on me to do anything else?

Oh, @wolveix
I honestly wasn't sure if it was ready for review or not, but if it is I'll give it a review now and let you know if I find anything

If possible I would prefer where we store the password length in a text while json encode. Instead storing it it the table directly as it would allow plugin developers to expend other server plugins where it is needed any way...
It would solve a lot of duplicated work in the future ...

@jaapmarcus @BelleNottelling mentioned this earlier in the PR:

3. It's not really specific to this PR and is just generally how things are setup right now, but I think we need to stop doing separate DB fields for these configs and instead move to using JSON in a single column. It's more flexible for server managers and would allow us to continue what was started in

I completely agree that that would be a better approach, though given Belle's comment, I figured it'd make more sense to split that off into a new PR. Happy to rework otherwise!

@jaapmarcus @BelleNottelling mentioned this earlier in the PR:

It's not really specific to this PR and is just generally how things are setup right now, but I think we need to stop doing separate DB fields for these configs and instead move to using JSON in a single column. It's more flexible for server managers and would allow us to continue what was started in

I completely agree that that would be a better approach, though given Belle's comment, I figured it'd make more sense to split that off into a new PR. Happy to rework otherwise!

If we going to change the behaviour any way we should do it now. As it also requires database structure updates that need to be altered again with the other PR.  It doesn't make sense to have the changes for maybe a few releases present ...

If we going to change the behaviour any way we should do it now. As it also requires database structure updates that need to be altered again with the other PR. It doesn't make sense to have the changes for maybe a few releases present ...

As I said, I'm happy to work on this :) Though given my long break from PHP and my freshness to this repo, it may make more sense for @BelleNottelling or someone else to tackle the task. But I'm happy to give it a go otherwise!

If we going to change the behaviour any way we should do it now. As it also requires database structure updates that need to be altered again with the other PR. It doesn't make sense to have the changes for maybe a few releases present ...

While I understand the point here, I think there's also the argument that @wolveix is following the existing implementation and that it's not their responsibility to completely refactor legacy code for us, most certainly not as one of their first contributions to the code base.
Furthermore, there's an argument that such a change would need to be delegated to at the soonest a 0.7.0 release as that change would be deleting at least 10 columns from the service_hosting_server table and would be a very breaking change that any code that directly references any of these columns in the database.
It also needs to be handled and implemented very carefully as any error in the migration step could potentially leave any hosting related service in a at best broken state or at worst accidentally result in essentially a complete lost to the configurations for all of someone's hosting servers within FOSSBIlling.
I really don't think we want to nor should be putting this onto a brand new contributor and I also don't think we should be preventing the merge of this pull request over it. At best, such a refactor is likely months away


Include nested template with string

Hi, I’m a little puzzled, when including a template can I not pass a simple string inside it using the with keyword?
#[derive(Serialize)]
struct Template {
    colors: Vec<&'static str>,
}

Main template:
{% for color in colors %}
    {% include "color" with color %}
{% endfor %}

Color template:
My color is: {{ color }}

The error I get:
  --> color:1:17
   |
 1 | My color is: {{ color }}
   |                 ^^^^^
   |
   = reason: not found in this scope

In order to compile I must do the following but it feels verbose.
#[derive(Serialize)]
struct Template {
    colors: Vec<Color>,
}

#[derive(Serialize, Clone)]
struct Color {
    color: &'static str,
}

Currently with {value} makes {value} the root context for the nested template. So the first error is because it is trying to lookup the field color from the string. The way you are doing it is currently the best way to do, but I agree it is a bit verbose.
You should actually not have not specify with at all, but it appears there is a bug with accessing loop variables in the parent template 😬. I have fixed it in d9e46eaa41c30123cafcfc08be61f966281d577d, so the following should work in the next release.
{% for color in colors %}
    {% include "color" %}
{% endfor %}

With regards to using with I can see a few potential improvements here:
Improve passing of value into the nested template

We could support map literals and allow constructing the literal inline before passing, for example{% include "color" with { color: color } %}

This is still a bit verbose

Improve referring to the root context

We could support referring to the current root context with .My color is: {{ . }}

Fix is released, I'm going to close this issue, but I have opened #23, and #24


SoloV2 dealing with empty images at Train Time

Prerequisite

[X] I have searched Issues and Discussions but cannot get the expected help.
[X] I have read the FAQ documentation but cannot get the expected help.
[X] The bug has not been fixed in the latest version (master) or latest version (3.x).

💬 Describe the reimplementation questions
Our use case has a custom dataset, where we want to use SoloV2 for instance segmentation.
For some of the images in the training set, there are no objects to be detected, and so the COCO dataset json file we create has no annotations to correspond to that image. If we run any SoloV2 model with such a training set, it gives an error.
Does current SoloV2 implementation have support for empty training images? If not then how should we implement it?
Environment
sys.platform: linux
Python: 3.9.11 (main, Mar 29 2022, 19:08:29) [GCC 7.5.0]
CUDA available: True
GPU 0,1,2,3: NVIDIA RTX A4000
CUDA_HOME: /usr/local/cuda
NVCC: Cuda compilation tools, release 11.1, V11.1.105
GCC: gcc (Ubuntu 9.3.0-17ubuntu1~20.04) 9.3.0
PyTorch: 1.12.0
PyTorch compiling details: PyTorch built with:

GCC 9.3
C++ Version: 201402
Intel(R) oneAPI Math Kernel Library Version 2021.4-Product Build 20210904 for Intel(R) 64 architecture applications
Intel(R) MKL-DNN v2.6.0 (Git Hash 52b5f107dd9cf10910aaa19cb47f3abf9b349815)
OpenMP 201511 (a.k.a. OpenMP 4.5)
LAPACK is enabled (usually provided by MKL)
NNPACK is enabled
CPU capability usage: AVX2
CUDA Runtime 11.3
NVCC architecture flags: -gencode;arch=compute_37,code=sm_37;-gencode;arch=compute_50,code=sm_50;-gencode;arch=compute_60,code=sm_60;-gencode;arch=compute_61,code=sm_61;-gencode;arch=compute_70,code=sm_70;-gencode;arch=compute_75,code=sm_75;-gencode;arch=compute_80,code=sm_80;-gencode;arch=compute_86,code=sm_86;-gencode;arch=compute_37,code=compute_37
CuDNN 8.3.2  (built against CUDA 11.5)
Magma 2.5.2
Build settings: BLAS_INFO=mkl, BUILD_TYPE=Release, CUDA_VERSION=11.3, CUDNN_VERSION=8.3.2, CXX_COMPILER=/opt/rh/devtoolset-9/root/usr/bin/c++, CXX_FLAGS= -Wno-deprecated -fvisibility-inlines-hidden -DUSE_PTHREADPOOL -fopenmp -DNDEBUG -DUSE_KINETO -DUSE_FBGEMM -DUSE_QNNPACK -DUSE_PYTORCH_QNNPACK -DUSE_XNNPACK -DSYMBOLICATE_MOBILE_DEBUG_HANDLE -DEDGE_PROFILER_USE_KINETO -O2 -fPIC -Wno-narrowing -Wall -Wextra -Werror=return-type -Wno-missing-field-initializers -Wno-type-limits -Wno-array-bounds -Wno-unknown-pragmas -Wno-unused-parameter -Wno-unused-function -Wno-unused-result -Wno-unused-local-typedefs -Wno-strict-overflow -Wno-strict-aliasing -Wno-error=deprecated-declarations -Wno-stringop-overflow -Wno-psabi -Wno-error=pedantic -Wno-error=redundant-decls -Wno-error=old-style-cast -fdiagnostics-color=always -faligned-new -Wno-unused-but-set-variable -Wno-maybe-uninitialized -fno-math-errno -fno-trapping-math -Werror=format -Werror=cast-function-type -Wno-stringop-overflow, LAPACK_INFO=mkl, PERF_WITH_AVX=1, PERF_WITH_AVX2=1, PERF_WITH_AVX512=1, TORCH_VERSION=1.12.0, USE_CUDA=ON, USE_CUDNN=ON, USE_EXCEPTION_PTR=1, USE_GFLAGS=OFF, USE_GLOG=OFF, USE_MKL=ON, USE_MKLDNN=OFF, USE_MPI=OFF, USE_NCCL=ON, USE_NNPACK=ON, USE_OPENMP=ON, USE_ROCM=OFF,

TorchVision: 0.13.0
OpenCV: 4.6.0
MMCV: 1.6.0
MMCV Compiler: GCC 9.3
MMCV CUDA Compiler: 11.3
MMDetection: 2.25.2+7ad77a9
Expected results
Functioning SoloV2 training
Additional information
Here is the traceback of the error:
  File "/deep2/u/mahmedc/mmdetection/tools/train.py", line 257, in <module>     main()   File "/deep2/u/mahmedc/mmdetection/tools/train.py", line 246, in main     train_detector(   File "/deep2/u/mahmedc/mmdetection/mmdet/apis/train.py", line 244, in train_detector     runner.run(data_loaders, cfg.workflow)   File "/deep/group/aicc-bootcamp/packages/miniconda3/envs/cloud-pollution/lib/python3.9/site-packages/mmcv/runner/epoch_based_runner.py", line 136, in run     epoch_runner(data_loaders[i], **kwargs)   File "/deep/group/aicc-bootcamp/packages/miniconda3/envs/cloud-pollution/lib/python3.9/site-packages/mmcv/runner/epoch_based_runner.py", line 53, in train     self.run_iter(data_batch, train_mode=True, **kwargs)   File "/deep/group/aicc-bootcamp/packages/miniconda3/envs/cloud-pollution/lib/python3.9/site-packages/mmcv/runner/epoch_based_runner.py", line 31, in run_iter     outputs = self.model.train_step(data_batch, self.optimizer,   File "/deep/group/aicc-bootcamp/packages/miniconda3/envs/cloud-pollution/lib/python3.9/site-packages/mmcv/parallel/data_parallel.py", line 77, in train_step     return self.module.train_step(*inputs[0], **kwargs[0])   File "/deep2/u/mahmedc/mmdetection/mmdet/models/detectors/base.py", line 248, in train_step     losses = self(**data)   File "/deep/group/aicc-bootcamp/packages/miniconda3/envs/cloud-pollution/lib/python3.9/site-packages/torch/nn/modules/module.py", line 1130, in _call_impl     return forward_call(*input, **kwargs)   File "/deep/group/aicc-bootcamp/packages/miniconda3/envs/cloud-pollution/lib/python3.9/site-packages/mmcv/runner/fp16_utils.py", line 116, in new_func     return old_func(*args, **kwargs)   File "/deep2/u/mahmedc/mmdetection/mmdet/models/detectors/base.py", line 172, in forward     return self.forward_train(img, img_metas, **kwargs)   File "/deep2/u/mahmedc/mmdetection/mmdet/models/detectors/single_stage_instance_seg.py", line 127, in forward_train     mask_loss = self.mask_head.forward_train(   File "/deep2/u/mahmedc/mmdetection/mmdet/models/dense_heads/base_mask_head.py", line 62, in forward_train     loss = self.loss(   File "/deep/group/aicc-bootcamp/packages/miniconda3/envs/cloud-pollution/lib/python3.9/site-packages/mmcv/runner/fp16_utils.py", line 205, in new_func     return old_func(*args, **kwargs)   File "/deep2/u/mahmedc/mmdetection/mmdet/models/dense_heads/solov2_head.py", line 545, in loss     loss_mask = torch.cat(loss_mask).mean() RuntimeError: torch.cat(): expected a non-empty list of Tensors
We are able to train MaskRCNN models with the same dataset

Thanks for your bug report! We will fix this in the next version.
Before that, you can try to set filter_empty_gt=True to your dataset and see if this can solve your problem.

Hi, you can try to use https://github.com/open-mmlab/mmdetection/pull/9185. This may fix your bug.

resolved in #9185


feat: React Navigation primitives

We need to expose React Navigation components that you can use within Rise mobile app to alter appearance of the UI. For example, you should be able to declaratively change screen title or header buttons.
Right now, it's tied to the screen key.
We may need a better abstraction for this, but this is one of the ideas.

Closing in favour of actions concept and built-in actions in Rise


Error defining MIN value on a DATETIME property

Hi,
Im getting an error on OrientDB 2.0.5 when creating a DATETIME property with MIN value on a Studio:

There error in the log is as follows:
2015-03-25 07:37:58:928 SEVERE Internal server error: com.orientechnologies.orient.core.exception.OSchemaException: Invalid datetime format while formatting date 'Mon Mar 02 2015 00:00:00 GMT+0700 (SE Asia Standard Time)' --> java.text.ParseException: Unparseable date: "Mon Mar 02 2015 00:00:00 GMT+0700 (SE Asia Standard Time)" [ONetworkProtocolHttpDb]
However, if I manually try to set the MIN value afterwards to, say, "00:00:00 03/25/2015", then it seems to work (i.e. no error is thrown), but then when I refresh the screen, value goes away:

Also, I havent tested doing this from console. So I am not sure if the error is in database itself of just the Studio.

I can confirm that this is not just an issue with the STUDIO applications form:  Executing this query:
DROP CLASS DateMinTest
CREATE CLASS DateMinTest
CREATE PROPERTY DateMinTest.dt DATETIME
ALTER PROPERTY DateMinTest.dt MIN DATE('2015-02-02 12:00:00')

or
DROP CLASS DateMinTest
CREATE CLASS DateMinTest
CREATE PROPERTY DateMinTest.dt DATETIME
ALTER PROPERTY DateMinTest.dt MIN '2015-02-02 12:00:00'

Results in an exception:
"com.orientechnologies.orient.core.exception.OSchemaException: Invalid datetime format while formatting date ''2015-02-02 12:00:00''
--> java.text.ParseException: Unparseable date: "'2015-02-02 12:00:00'"
If some one  the OrientDB team can let me know what the intended syntax is, then I can fix this bug.  I am trying to update the CREATE Property syntax to allow setting the MIN, MAX, and DEFAULT.  It is working, but this bug is preventing setting MIN, MAX.
Note that the following query works:
DROP CLASS DateMinTest
CREATE CLASS DateMinTest
CREATE PROPERTY DateMinTest.dtDefault DATETIME
ALTER PROPERTY DateMinTest.dtDefault DEFAULT '2015-02-02 12:00:00'

The issues seems to be in the OPropertyImpl class.  The "setDefaultValueInternal" method does not check that the value itself is exactly a parsable date.  Wherease setMaxInternal and setMinInternal call checkForDateFormat.  It seems like Min and Max what the string itself to be a parseable date, whereas Default simply requires that the expression eventually can be resolved to a date / datetime.
One obvious issue is that is seems like we must "quote" the datetime string.  If we do not, then the parser will see the space between the date and time, and only parse the former.  However, when you quote the string, then the OPropertyImpl tries to parse the string  '2015-02-02 12:00:00' (containing the quotes).  The quotes cause the unparseable date issue.
Again if @lvca @luigidellaquila or some one else on the Orient team lets me know what the indented valid syntaxes are, I will fix this issue.
Thanks.

Hi @andreyvk @mmacfadden
IMHO for consistency the second syntax should be supported  (ALTER PROPERTY DateMinTest.dt MIN '2015-02-02 12:00:00'), but I like the first one as well (even more, actually)
Thanks a lot!
Luigi

Fixed in Studio develop branch


New RPM package doesn't work for AmazonLinux

We have a dockerfile for AmazonLinux (docker/release/amazonlinux), and I found this when validating docker files. It turns out that AmazonLinux doesn't have the package openssl-libs.

@SteveL-MSFT Amazon Linux is not an officially supported distro for .NET Core 2.0 (see https://github.com/dotnet/core/blob/master/release-notes/2.0/2.0-supported-os.md#linux). Do we need to support it?

No, it's not one we officially support either.

OK, I will remove the dockerfile then.

Could we document PowerShell Core package dependences to help users to install PowerShell on unsupported distributives?

Hello @SteveL-MSFT  and @daxian-dbw  just hit this issue myself on AWS linux, this was working fine a few days ago. do you have any recommendations to address the dependency issue of openssl-libs ? i realy realy realy dont want to change to a different distro so any help to get this working again would be awesome. (PS support AWS linux ;) )

@r2690698 just to be clear, I want to enable running PSCore6 on as many systems/devices as possible, but we can only officially support what dotnetcore supports.  For other platforms, we'll make a best effort (community support like for ArchLinux is always appreciated!).

The powershell package dependencies are the same as those of dotnet-runtime-2.0.0 packages (DEB, RPM) (available at here). You can find the dependencies in the PR description of #4902. Or better, look at the source of truth in New-UnixPackage from packaging.psm1.
The RPM package of dotnet-runtime-2.0.0 depends on openssl-libs and thus cannot be installed through yum install on Amazon Linux.
However, the dotnet team provide a .tar.gz package of dotnet-runtime-2.0.0 for users to deploy, similar to the .zip package for windows. I think powershell should also provide a .tar.gz package for users to use on not officially supported distros. I will add this task to #3961.


Integrate ECharts JS plot library in DFLib

Let's integrate Apache ECharts to DFLib. The charts will be rendered as JS snippets that can be displayed in the UI. The first UI target will be Jupyter notebooks.
Note that ECharts has an enormous number of configuration options. In this version, we will cover a small subset of them, and we'll expand it as we go.

Let's integrate Apache ECharts to DFLib. The charts will be rendered as JS snippets that can be displayed in the UI. The first UI target will be Jupyter notebooks.

How do you plan to generate the JS? Do you use https://echarts.icepear.org/ or do you create your own library?

Ah, interesting. Wasn't aware of this lib. I already have something working here, though the API is work in progress and is going to change:

Not using the library saves some extra dependencies and makes things more "native" (able to work directly with the DataFrame), so I may stay on this path, but I really like how they classified ECharts coordinate types, and otherwise made dealing with "options" more sane. So thanks a lot for the pointer.

Just merged the initial integration. It supports basic Cartesian charts (line, scatter, bar) and a number of useful configuration options. We will continue extending it in this and other releases. An example of time series:
t                     v1 v2
------------------- ---- --
2024-01-01             3 20
2024-01-02             5  6
2024-02-03             2 11
2024-02-10             4 11
2024-02-12             2 15
2024-02-14             3 14
2024-02-15             4 13
2024-02-15T03:05:06    5 12
2024-02-20          11.5  2

ECharts
    .chart("temp, °C")
    .sizePixels(600, 350)
    .series(SeriesOpts.line().smooth().areaStyle(), "v1", "v2")
    .xAxis("t", Axis.time())
    .yAxis(Axis.value().boundaryGap(false).label(AxisLabel.create().formatter("{value} °C")))
    .toolbox(Toolbox.create().featureDataZoom().featureSaveAsImage())
    .legend()
    .plot(df)


Module not found

Describe the bug
Getting a Module not found: Can't resolve 'https://unpkg.com/@jsquash/jpeg?module'
To Reproduce
Just the very basic - added an import statement as given here https://github.com/jamsinclair/jSquash?tab=readme-ov-file#usage-in-the-browser
Environment
React 17 + CRA 4 + Craco with config:
module.exports = {
  webpack: {
    configure: {
      module: {
        rules: [
          {
            type: 'javascript/auto',
            test: /\.mjs$/,
            include: /node_modules/,
          },
        ],
      },
    },
  },
}

Happy to provide any further info. I find this part about the JS toolchaining quite confusing, and my knowledge in this area is poor.

@zehawki I am not overly familiar with create react app. I suspect this happens during the webpack build. Webpack will need the module installed locally as it cannot resolve URL imports.
Installing with npm install -S @jsquash/jpeg and importing it as shown in the docs should solve the issue 👍


VAE + Classification

Hi all,
I am trying to modify the VAE example  for classification purposes. I want to use the latent variables of the VAE as input to the MLP classifier. Are the z_mean and z_log_var be concatenated and fed as an input to the MLP classifier? Or should it just be z?
Thanks!

It should just be z. Actually z is calculated by z =  z_mean + K.exp(z_log_var) * epsilon, see the sampling function defined in the code:
def sampling(args):
    z_mean, z_log_var = args
    epsilon = K.random_normal(shape=(batch_size, latent_dim),
                              mean=0., stddev=epsilon_std)
    return z_mean + K.exp(z_log_var) * epsilon

Then z is what we mean by "latent variables". You will want to put z as the input of your MLP.


Update ui_template.html

Hello Alex, first of all thanks for this wonderfull api. Currently I am using it in my scada project with node-red.
I had problems adding two or more comma separated types like "myID@cx_color,cx_stroke" so I deeg into the code and made some changes.
The compatibility with the api is still the same. Only I added the posibility to use two diferents payload to change the color so you can change the cx_color and the cx_stroke for a single object. The two types of payloads are:
unique_id@+cx_color - payload is a string or payload is an object {color: string, width: number}.
The added code also allows any kind of combination of different types., like @cx_status,cx_color,cx_hide etc.

Hello, yes of course.

Done

Hi Juan, just noticed that it wasn't addressed.
I cannot Merge the pull request as the proposed changes violate the design functionality. I'll try explain below, however correct me if I'm wrong.
In your proposed code line 117 you added cxTypeParts. It isn't correct as the topic in the message should not have "parts". For example if I have a SVG group <g id="T-101_level@cx_move,cx_hide">...</g> then a message from upstream node cannot be {"payload":{"x":0,"y":0,"deg":17},"topic":"T-101_level@cx_move,cx_hide"} it needs to be {"payload":{"x":0,"y":0,"deg":17},"topic":"T-101_level@cx_move"} or {"payload":true,"topic":"T-101_level@cx_hide"}
I'm working on version 2 where I'm going away from using topic property and will add cxId and cxClass properties.

ok I understand. I will be attentive to the new version. Thanks.


filter realtime logs by source address

but only if all sources filter by address and none use factory contracts

Hi @holic thanks for opening! For posterity, wanted to note that this was proposed as a workaround for RPC providers that require specifying an address for eth_getLogs requests. Today, Ponder's realtime sync does not specify one.
We plan to solve this, but would rather do it alongside a full review of our realtime sync algorithm (https://github.com/ponder-sh/ponder/issues/228) rather than patch it in. Nonetheless thanks for raising this!


Fix URL for downloading R-devel on macOS

The current URL 404s: https://travis-ci.org/nealrichardson/arrow-r-ci/jobs/554464322#L58
https://r.research.att.com/el-capitan/R-devel/ redirects to https://mac.r-project.org/el-capitan/R-devel/ and shows that the correct file name is R-devel-el-capitan.pkg
cc @jimhester @jeroen

LGTM, thanks @nealrichardson

Well, actually LGTM except you need to update the specs to reference the new location, they are located at https://github.com/travis-ci/travis-build/blob/6214d3e51e186d75d7488bede859c8142d0222a8/spec/build/script/r_spec.rb#L78

Fixed @jimhester, PTAL

Ok looks fine now and the tests are now passing as well.


Some packages are missing a flat-container nupkg

Most of these packages are also missing their registration1 page.
All packages are found in search except for a couple older packages that are all unlisted. This suggests the issue is in Catalog2Dnx itself because Catalog2Lucene and often Catalog2Registration are running correctly.
Here is a zip containing reports of all packages with this issue.

Replaced by #3991


gridCreator_Web_design_#335_AkshithaGunupati

#335

Hi @NtemKenyor, Could you please take a moment to review my pull request? I'm currently working on a university assignment, and I need to complete this task before the deadline. Your time and assistance are greatly appreciated.
Thank you!


Defaults to config too quickly after wifi connection loss

Hi Liebman,
Great project, works great. I have encountered an issue in my environment that could possibly enhance the project for others. I am looking to use this as a highly available NTP server for my environment.
My wifi reboots weekly in the middle of the night. When this happens, the ESP detects the loss and before the wifi is reconnected the ESP goes to broadcasting it's own SSID for config mode. This takes the NTP server offline until I manually restart it. Is there a way to incorporate a configurable delay in seconds for the wifi reconnect when there is an existing config in place. Or another idea is a configurable parameter that just keeps attempting reconnects and blocks config mode from being entered unless the button is pressed.
Thanks for considering these suggestions.

This should be doable.  I'm using WiFiManager and I know it has options to set the connect timeout.

OK - I've created a branch disable-auto-captive-portal can you give that a try?  This is what "should" happen:

the captive portal will start if the config button is held at power up/reset
on a fresh (erased) unit the config file will not exist in SPIFFS and captive portal will start
if it has a config file it will connect but NOT start the portal. (if its failing to connect it will delay one second and try again forever)

Let me know if this works for your use case and if so I'll merge to master.

Yes, it waited out for the wifi to return and then successfully reconnected, awesome thanks. Actually rebooted 3 times and it recovered each so appears to be in great shape. All other functions appear at they were prior. When wifi comes back it does delay the 120 seconds before becoming valid again.


[Servicebus] Sb changelog for oct release

API change check
API changes are not detected in this pull request.


make sure patchFindDir works with Next.js 15 and add patching validation check

resolves #172

The patchFindDir function: https://github.com/opennextjs/opennextjs-cloudflare/blob/acec1c3ce61602a5bbb3f1330b5d1e0d0110b3f6/packages/cloudflare/src/cli/build/patches/to-investigate/patch-find-dir.ts#L12
Stopped working since a new name variable got introduced in the Next.js source code, causing the bundled code to rename the function's name parameter to name2 instead:

I fixed this by updating the code replacement to also accept the variables with a numerical suffix. I also added a check to make sure that the patching is applied (if we already had the check in place that would have saved me lots of time here! 😭)
A few notes:

I wanted to update the code to use AST manipulation, but since this patch is in the to-investigate directory I guess that we assume that this is temporary, so maybe it's not worth investing time to update the code to use ASTs? (if it is I am more than happy to do so!)
again, having the patchedCode !== code check would have saved me lots of time here, we need to add it to all the patch functions to avoid wasting time again, I will do that in a followup PR
potentially we can hit this issue with other patching functions, I can update those as well to accept numerical suffixes in their variables/paramenters, but I think that adding the patchedCode !== code check would be good enough for now

Does this also resolve the warnings (eval(require)...?)

no 🥲
I can look into those if you want, but since they are only warnings I assumed that they are lower priority
Regarding warnings we also get this one:

Does this also resolve the warnings (eval(require)...?)

no 🥲
I can look into those if you want, but since they are only warnings I assumed that they are lower priority
Regarding warnings we also get this one:

Maybe create an issue for eval(require)... so that we look into it at some point.


Add Award winners for 2024

Add Award winners for 2024

/assign @tokt

/lgtm
/approve

@mfahlandt  Thank you so much for working on this, appreciate it a lot!! <3


App styling - Add style to the application

Hi, in this PR, I have implemented the following in the project. Please review and provide feedback.

Add styling to the applciation based on the design provided in the Zeplin template.
Document project changes in readme file.

Implemented the changes requested by the previous reviewer. Removed the extra space outside the App component.


Make OptimizerConfig a builder style API

Which issue does this PR close?
re  https://github.com/apache/arrow-datafusion/issues/3524
Rationale for this change
I am preparing to improve the API between the optimizer and the rest of the system and I want to get the basic pattern established before more major changes
What changes are included in this PR?
This PR moves from modifying public fields to a builder interface for OptimizerConfig
Are there any user-facing changes?
To set execution start time, must use builder interface now

Codecov Report

Merging #3525 (543f8c2) into master (12f047e) will decrease coverage by 0.00%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master    #3525      +/-   ##
==========================================
- Coverage   85.77%   85.76%   -0.01%
==========================================
  Files         299      299
  Lines       55334    55336       +2
==========================================
- Hits        47460    47459       -1
- Misses       7874     7877       +3

Impacted Files
Coverage Δ

datafusion/core/src/execution/context.rs
79.29% <100.00%> (-0.03%)
:arrow_down:

datafusion/optimizer/src/optimizer.rs
91.66% <100.00%> (+0.75%)
:arrow_up:

datafusion/optimizer/src/simplify_expressions.rs
82.80% <100.00%> (-0.04%)
:arrow_down:

datafusion/optimizer/src/type_coercion.rs
93.56% <100.00%> (ø)

datafusion/expr/src/window_frame.rs
92.43% <0.00%> (-0.85%)
:arrow_down:

datafusion/expr/src/logical_plan/plan.rs
77.68% <0.00%> (-0.34%)
:arrow_down:

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Sprite.touching doesn't seem to work, at least in the demo

In the demo you just fall continuously, this seems to be because this.touching('Ground') never returns to true, even after I disabled gravity and just moved into the black to the right, it still never returns true. This also happened regardless of whether or not fast collision is enabled.

this seems to be because this.touching('Ground') never returns to true, even after I disabled gravity and just moved into the black to the right, it still never returns true.

That's odd. I just cloned a fresh copy of master, and everything works as expected. What's your browser/OS?

Just made a bunch of changes to the way the touching block works in #6. Would you be willing to git pull and try the new one out instead? Hopefully it'll fix the issue (despite not knowing the cause of the issue in the first place). :crossed_fingers:

Woah! It looks like there's a difference between npm run watch and npm run build. The first works; the second doesn't.
I have no idea why there would be any difference. :confused:

I just retested, it has to the with the --mode=production, when I switched it to --mode=development it started working

Ah! Looks like there's some code that relies on sprite.constructor.name to check which type of sprite it is. In production mode, webpack shortens the names of everything to be as small as possible. Thus, the check doesn't work.
I should have known better than to rely on that type of thing... Give me a few minutes.

Should be all fixed now. git pull and give it another try. ;)

Still doesnt work with the production tag

Odd. Hopefully you're pulling from the master branch. Would you be willing to double-check that you actually have the latest code? Look in the touching function (line 80) in [git repo]/scratch-js/Sprite.js. The second line should say...
// We want this (new)
const matching = spr => spr.name === sprName

as opposed to...
// Not this (old)
const matching = spr => spr.constructor.name === sprName

Which one do you have?

https://i.imgur.com/HKfpF6h.png, yeah I have the new code, this is so strange

That's crazy! You also have the other changes from e5a5dd5, right? If Ground.js or Player.js are missing the new this.name lines, you would get the same falling through the ground effect.
I'm completely confident that the code works, as long as you have it all. I was able to replicate your issue and solve it with 100% consistency. My only other idea is some sort of caching problem either in the browser or in webpack itself.

Ok, I just redeleted everything and git cloned, and now it works. That was so strange.

Okay! That was odd, but I'm glad it's taken care of now. :smile:


Using 2 space indentation instead of 4 for middleware_topology_controller.js

GitHub diff is not perfect for these white space changes, so here is the output of
git diff -w (ignoring the white spaces)
http://pastebin.com/g7t9YDMf
@miq-bot add_labels providers/hawkular, ui

@himdel Could you please look It's a followup on this https://github.com/ManageIQ/manageiq/pull/8630#discussion_r68199297

@Jiri-Kremser if you look here this will show you the diffs, ignoring whitespace.

@chessbyte ?w=1 nice

@Jiri-Kremser great, thanks! But.. missed a few 4space indents in there..

LGTM
@himdel @martinpovolny merge please?

Thanks @Jiri-Kremser , will merge when green..

@himdel it's green :green_heart:

..aand merged :)


It is possible to continue registration on the site after entering invalid data in the verification code field

Description
It is possible to continue registration on the site after entering invalid data in the verification code field
Preconditions:

The site is opened https://goodqa.netlify.com
The phone number verification is completed
The “Sign up” page with “And which email address..” text is displayed

Steps to reproduce:

Click the “Back” button
Click the “Back” button
Click the “Next” button
Enter invalid data in the verification code field (e.g.111111)
Pay attention to the screen

Actual result: It is possible to continue registration on the site after entering invalid data in the verification code field.The same after starting again from the “Full Name” and entering another phone number
Expected result: It is impossible to continue registration on the site after entering invalid data in the verification code field
Supporting data
Env https://goodqa.netlify.com
Samsung Galaxy S8 // Android 8.0 (Google Chrome 75)
iPhone6 12.3.1, Safari
https://www.screencast.com/t/aPr2qIgmyU0o

@qatests1001 this doesnt seem like a bug to me. If user already verified his phone there's no need to require any code at all. maybe we can hide this page once phone is validated. only if he changes his phone number should we require a new valid code

https://www.pivotaltracker.com/story/show/167485222


Building the binary for NXP TWR-LS1021A board

According to the "mbed.htm", the firmware version is 0203. Using this version, I have faced some issues regarding UART (UART-USB converter). I have also noticed the latest release (0242) has some fixes related to UART (#97). So I want to use the latest version 0242 for TWR-LS1021A board which is based on MK20DX128VFM5. I have followed the guidelines mentioned by DEVELOPERS-GUIDE.md. The problem is that I could not find this board specific code in the "source/board" directory. As this is based on the K20DX target, I have tried with "k20dx_bl.bin" but the board did not boot up.
Has anyone tried the DAPLink source for TWR-LS1021A board?
Do I need to port this source for the TWR-LS1021A board?

Hi @skhasan, I haven't  tried DAPLink with this board, although DAPLink does support two other tower boards - twrkl28z72m and twrke18f. The file k20dx_bl.bin should work on other tower boards. How are you loading it?

Could you please let me know how to flash the bootloader (k20dx_bl.bin)? Currently my board has CMSIS-DAP (not DAPLink ) bootloader preinstalled. That is why the DAPLink interface firmware (build 203) is not working for my board. In my understanding, I should flash the DAPLink bootloader first but I do not know how to flash it. Please help me.

You'll need to program the K20 over swd with an external debugger using uVision.

Closing due to inactivity. @skhasan please reopen if you have further questions.


Update README URLs based on HTTP redirects

Created with https://github.com/dkhamsing/frankenstein
GitHub Corrected URLs

Was
Now

https://github.com/bboyfeiyu/android-tech-frontier
https://github.com/hehonghui/android-tech-frontier

https://github.com/bboyfeiyu/android-tech-frontier/issues
https://github.com/hehonghui/android-tech-frontier/issues

https://github.com/bboyfeiyu/android-tech-frontier/issues/1
https://github.com/hehonghui/android-tech-frontier/issues/1

https://github.com/bboyfeiyu/android-tech-frontier/wiki
https://github.com/hehonghui/android-tech-frontier/wiki

This change is

Melanie Bennell

On 21 Jul 2018, at 2:19 pm, 1Bennell<EMAIL_ADDRESS>wrote:
@1Bennell commented on this pull request.
Reviewable status: 0 of 1 files reviewed, 2 unresolved discussions (waiting on @1Bennell)
README.md, line 85 at r1 (raw file):
免费优秀的电子书

软件架构模式
D

README.md, line 88 at r1 (raw file):

已完成列表
请移步项目Wiki 。
P
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.


Issue with design decisions

Just a couple of compliments, then some complains, bear with me 😅.
First, thank you. The ReadTheDocs project is awesome and fantastic! It's greatly improved the state of my documentation teams documentation. The web hosting is very generous, and the Sphinx theme is fabulous.
Now on to the complains... Is it going to be another 5-9 months before things get reviewed again? I assumed that #234 being solved would cause this project to gain momentum, but besides one instance, it just died again with no PR reviews and issues just constantly being marked as "Design Decision". Speaking of which, based on the amount this is put in, it seems like it's just a catch-all for a bunch of issues.
Are "Design Decisions" ever going to be made, or only once every 2 years? (A little bit of an exaggeration)
...
Just trying to get some momentum back in this project again, I love you guys and what you do, and I'm assuming it's an incredibly small team. I just wonder what the stance is on things, before I fork and merge a couple pending PRs myself for my organizations use. In particular, #986 is very very important to us

Hi, I just reviewed and replied to most issues a couple of weeks ago https://github.com/readthedocs/sphinx_rtd_theme/issues/997. And, yes the main cause is because we are a small team (5) managing several projects, and the team that's familiar with the theme is working in the re-design of the readthedocs.org site, so reviews here and decisions will be a little slow till that is done, or we are able to hire more people (this may not happen so soon).

Thanks for the kind words, hopefully we can focus on the theme soon


Move reactpy.backend.hooks module into reactpy.core.hooks

Description
Fixes #1182 - Moves the hooks in reactpy.backend.hooks into reactpy.core.hooks
Checklist
Please update this checklist as you complete each item:

[X] Tests have been developed for bug fixes or new functionality.
[X] The changelog has been updated, if necessary.
[X] Documentation has been updated, if necessary.
[X] GitHub Issues closed by this PR have been linked.

By submitting this pull request I agree that all contributions comply with this project's open source license(s).

I'm having trouble with adding the # nocov comment at the top of backend.hooks. It seems to only be working on the first import line instead of the whole file.

Hmm I thought there was a way to do it for the whole file. Worst case you could add that comment to each individual hook in the file.

I decided to add nocov comments to every import/function. Let me know if there's a more concise way to do it, because I couldn't figure it out.

Still not passing lint:
reactpy/core/hooks.py:504: error: Incompatible return value type (got "Sequence[Any] | EllipsisType | None", expected "Sequence[Any] | None")  [return-value]****

Why do you mean by pin mypy?

I don't see how my changes could have caused the lint error we're seeing, so I think I'm good to add mypy to the most recent version that doesn't break tests (once I know how to do that).


Error and I can't save

Player.log
Hi, I'm not sure what's going on but Solasta gives me an error with the mod and won't let me save or load until I exit the game and re-enter.
Which is kind of annoying to me because I'm playing in iron mode.
I had never thrown any errors.
I attached the player.log that the error says has the information, I think.
Sorry if I didn't understand well, English is not my native language.

Can I also have the save? I speak Spanish, Portuguese and Italian if you prefer any other language.

Save.zip
Ahí esta el ultimo autosave. El error sale siempre que cambio de mapa. No se que puede ser porque casi siempre uso la misma configuración.

Unfortunately cannot reproduce. Try RESPEC'ing your bard.


[Suggestion]

Config to disable squid spawning
Config to disable bat spawning

These are good ideas. I will look into adding them.

Implemented in 1eae0fab65c8de3aff5fb4416e3d2ff76d92f013


Should avoid opening files in windows containing special buffers (&buftype != '')

Is your feature request related to a problem? Please describe.
It seems that currently flatten.nvim will open files in users' previous windows no matter what buffer is originally associated  in that window. This can be an issue when users have two terminal windows opening side by side and try to open a file inside one of the terminals since the file opened will be displayed in the window for another terminal.
Describe the solution you'd like
Find the first window that is not associate with a special buffer (vim.bo.bt == '') and open the file in that window.
Describe alternatives you've considered
N/A
Additional context
There are some screenshots that explain the issue:

Two terminals opened side by side:

Opening files from the terminal on the right (with the window on the left as the previous window accessed):

File opened on the left window, taking the place of the first terminal, which is not expected:

That's a good point, I'll look into implementing that soon.


Make sure that the XGA LT pipeline is resilient to failed generation of spectra

These things happen, it would be good to make sure that the pipeline can deal with it without erroring out completely.

Initially I thought that I would just remove the entire source if something went wrong, but I should probably be a little less broad and try to remove only the data which is causing an issue

I considered removing only those spectra which fail generation steps, and maybe I will come back to that idea in the future, but I'm not convinced it's the right approach yet.


Use sys.prefix to determine if inside a virtualenv

Fixes #30

Just need to double check that the test doesn't pass in a Conda environment.

When in a conda env, sys.prefix is the same as sys.base_prefix, so we still need the extra logic for conda.

Thanks!


support config->transcribe run in background

https://github.com/jambonz/jambonz-feature-server/issues/568

published in v0.0.50


Make sure to start a gRPC client call after the initialization of an Endpoint

Motivation:
Although, #3876 addressed java.lang.IllegalStateException: Should call init(endpoint) before invoking this method in 1.13.0, we still got a report the
problem from Armeria Slack.
The problem could occur if the following two conditions are met:

EndpointGroup was resolved and has non-empty endpoints when creating an ArmeriaClientCall.
https://github.com/line/armeria/blob/c21a88ff59daa2101712a8e1da41a6281f2c0878/grpc/src/main/java/com/linecorp/armeria/internal/client/grpc/ArmeriaClientCall.java#L173
The endpoints in the EndpointGroup become empty somehow when the ArmeriaClientCall initializes ClientRequestContext.
https://github.com/line/armeria/blob/d78b52f3bc62ed1c4f5037f092ea1f514156d880/core/src/main/java/com/linecorp/armeria/client/DefaultClientRequestContext.java#L302-L311
https://github.com/line/armeria/blob/c21a88ff59daa2101712a8e1da41a6281f2c0878/grpc/src/main/java/com/linecorp/armeria/internal/client/grpc/ArmeriaClientCall.java#L234

Modifications:

Assume that DynamicEndpointGroup can be empty anytime.

Mark endpointInitialized after ctx.whenInitialized()

Result:

You no longer see IllegalStateException when an EndpointGroup is empty.

EmptyEndpointGroupException will be raised instead after a connection timeout.

Side note: I'm not 100% sure that this is the actual cause of your problem. Because it might be rare that endpoints are disappeared in the micro/nanoseconds.


Native module problem

In my purely native module I get the following error:
error: Project file must contain declare-project and declare-source
But in native modules only requires declare-project and declare-native to jpm to accept it

@GrayJack The files referenced in your declare-native map are C files, right? Unfortunately, Documentarian is only intended to work with Janet files.

@GrayJack It took a few months but I added experimental support for native modules. It's working with Markable (example) but I haven't tested it extensively.
I'm not sure if you're still interested in this but if you try it out and notice any problems, please let me know! I'll close for now on the expectation that this works but can always reopen if that's not the case.

@GrayJack It took a few months but I added experimental support for native modules. It's working with Markable (example) but I haven't tested it extensively.
I'm not sure if you're still interested in this but if you try it out and notice any problems, please let me know! I'll close for now on the expectation that this works but can always reopen if that's not the case.


Switch to GCC 13

To reduce the size of the binary a bit we add -mno-unaligned-access because that made struct initialization generate quite a lot more code.
In the new toolchain assert is defined as a macro. That interferes with the ASF function called assert so we undefined it right before that function is defined.
We also have to provide the search path to the linker to our dummy files libssp and libssp_nonshared. The actual implementation of the stack protector functions are in common_main.c.

I will wait this this PR until I have some other CI related things merged and I've tested the device.

Firmware doesn't work. It doesn't pop up the "unlock" screen.


Fix canceling animations.

What does the pull request do?
As described in #6111, canceling an animation didn't do anything, which after the value store refactor in #8600 caused RoutedViewHost to stop working as described in #9514 (and maybe #9621).
The problem was that a DisposeAnimationInstanceSubject was being created but then just discarded and so was never disposed (which I assume is the purpose of this class, given its name). Make Animator<T>.Apply actually return the DisposeAnimationInstanceSubject so that it gets disposed when the animation gets canceled.
Because ControlCatalog doesn't have a dependency on Avalonia.ReactiveUI I had no way to test whether RoutedViewHost was fixed, so I also added a ReactiveUIDemo sample project which currently just contains a simple RoutedViewHost example for testing.
@jmacato you wrote this code, so I'd appreciate a 👀 .
Fixed issues
Fixes #6111
Fixes #9514
Might fix #9621 (need to check)
Depends on #9625

You can test this PR using the following package version. 11.0.999-cibuild0027179-beta. (feed url: https://pkgs.dev.azure.com/AvaloniaUI/AvaloniaUI/_packaging/avalonia-nightly/nuget/v3/index.json) [PRBUILDID]

You can test this PR using the following package version. 11.0.999-cibuild0027193-beta. (feed url: https://pkgs.dev.azure.com/AvaloniaUI/AvaloniaUI/_packaging/avalonia-nightly/nuget/v3/index.json) [PRBUILDID]


Shim for pywheels

CI test included

please retest

please retest

@cattabiani : CVF is failing but I believe that's separate issue.


Add support for no buffer upload

Brief Summary of Changes
Hi,
I'm using this SDK in a web service that uploads potentially large video files, sometimes multiple files at once. The SDK buffers chunks in memory, so if there are a lot of concurrent clients, the memory consumption can become so large that the entire service will crash. This PR fixes that by replacing buffer with pipe, so now bytes can be sent as soon as they're read.
What does this PR address?

[ ] GitHub issue (Add reference - #XX)
[ ] Refactoring
[ ] New feature
[x] Bug fix
[ ] Adds more tests

Are tests included?

[x] Yes
[ ] No

Reviewer, please note:

Checklist:

[x] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[x] I ran the full test suite before pushing the changes and all the tests pass.

@const-cloudinary can you take a look at this when you'll have a minute? 🙂

@kjarmicki sure, check your slack

@kjarmicki thank you for contribution!


Checks links rot with CI

From time to time, we have rot links on the documentation. To prevent that, we could have a workflow in the CI pipeline to checks all links. This workflow would not need to run in every PR, but daily for example because external can break at anytime.
Impediment:
Storybook is dynamically generated and needs more time to fully load a page. Does a link checker support this situation?
Some options to analyse:
https://github.com/lycheeverse/lychee
https://github.com/untitaker/hyperlink#options
https://github.com/marketplace/actions/check-links-with-linkcheck

Two issues:

Most (if not all) of the link checkers don't support websites with dynamically rendered content
Storybook doesn't output regular 404 when a story does not exist (e.g. https://next.design-system.post.ch/?path=/docs/foundations-layout-fake--docs)

The first issue could be resolved by using a checker which looks at the source: the MDX files. But then we need to parse links to make a difference between external and internal links, and the check for internal links would still need to be done dynamically. We could use a headless browser, but it makes all the things more complex and prone to error and flakiness…

Please check first, if there is a plugin which allows us to use something like permalinks for our stories.

For example: https://storybook.js.org/addons/storybook-redirect/

There is as well a possibility to set an id attribute: https://storybook.js.org/docs/configure/sidebar-and-urls#permalink-to-stories

@imagoiq, is this really a blocker for the storybook migration? If not, please remove the milestone (the ticket will just be with the others in the backlog) so we have a better overview of when we can migrate.

https://storybook.js.org/addons/storybook-redirect has not been updated for 5 years and looks rather complicated.
With https://github.com/swisspost/design-system/pull/2603, we get kind of permalinks. Nothing prevents us from changing those ids, but we are safe if we can keep ourselves from changing them, or at least do it carefully and add a redirect with a middleware as in https://github.com/storybookjs/storybook/discussions/23093#discussioncomment-7063363
This PR doesn't prevent issue when a story is deleted, but we can keep the same conclusion and try to remind ourselves to add a manual redirect.
The only remaining part is external URLs. If we omit Figma, ng-boostrap, badge-fury, archive.design-syste.post.ch, MDN, caniuse (which should be rather stable) we have currently:
1 link on card component (picsum.photos to generated examples)
17 links for logos dependencies on home page
Therefore, it might be overkilled to check external links with a solution involving e2e for example.

@gfellerph I added this ticket again on the migration milestone because having those ids will stop the old link from working, so ideally it should be done when the documentation is publicly available (or before).


question: adjusting target ldcast

Hi @jleinonen,
more an approach question.
Say, you are just interested in the rainfall at one particular spot (e.g. discharge at one point in a hydrological basin). Do you think it is worthwhile to adjust the target to the time series of the future precipitation (instead of the time series of the radar images)? This implies, among others, skipping the encoder part of the future precipitation.
Look forward to your thoughts on this.
Regards,
Tomas

Hi Tomas,
My first guess is that if you specialize the model specifically for one basin, you may well get better results. But in that case you probably don't even need a generative model at all, it's easier to use a simpler predictive model since you're only interested in one point.
The nice thing about generative models is that they give you the flexibility to look into spatial and temporal distribution properties of the rainfall probabilistically over a wide range of scales. But if you are interested in one point only you don't really need that flexibility.
Cheers,
Jussi

Hi Jussi,
I have to ponder a bit on this, and will come back to it later.
Regards,
Tomas


Bump undici version and minimum node version

To target security vulnerabilities in undici, we update to the latest version.
From undici v5.7.0, we require a node >=16.8.0 - here we update the minimum to 16.13.0 (the first Node 16 LTS release)

Just worth noting that this will require some people to bump their version of node - I assume we have some precedent for a release like this

Would you be ok using 16.8.0 as the minimum version instead? Makes the breaking change radius a little smaller.

That's fair. Let's land this?

wait it was landed already ha


Deployment Issues/questions

When following the deployment guide steps 4.1 & 5.1 to prepare the app the admin permission request related to https://actions.office365.com/ActionPackage.ReadWrite.All does this have something to do with the Kaizala service?
We understand there are two packages(action package and teams app) which has the following web application info about where the apps will be hosted once we deploy them.
"webApplicationInfo": {
"id": "9bb724a5-4639-438c-969b-e184b2b1e264",
"resource": "https://actions.office365.com"
}
The bolded id above is the APPID of KaizalaActionsPlatform (https://www.rickvanrousselt.com/azure-default-service-principals-reference-table/) and also the action package that we get while deploying has the similar structure of Kaizala action. So we would like to know more about these details.

These Apps does not related to kaizala.


provide example data

Provide a link to some small example data that can be used to test that the workflow works and does what it is intended to do (perhaps also a way to check the output).

Fixed


Fix issue where longer logic does not match CSS spec

It appears that Color.js "longer" hue adjust logic does not match the CSS specification: https://www.w3.org/TR/css-color-4/#hue-longer.
The important difference is that while the spec states in pseudo code:
if (0 < θ₂ - θ₁ < 180) {
  θ₁ += 360;
}
else if (-180 < θ₂ - θ₁ < 0) {
  θ₂ += 360;
}

It appears Color.js is using something similar to:
if (0 < θ₂ - θ₁ < 180) {
  θ₂ += 360;
}
else if (-180 < θ₂ - θ₁ <= 0) {
  θ₁ += 360;
}

Thanks for spotting that! Which behavior is better? I wrote both that part of the spec and that part of color.js, so either could be changed :P

Which behavior is better?

I guess that depends on the intention 🙂.
The handling of equivalent angles is what threw me off and made me take a closer look. Any time we have a scenario where two colors with exact hues (or equivalents, think 0 and 360 or 180 and 540), the unpatched implementation gives you an interpolation of all hues while the patched version keeps the interpolated hue constant:
Unpatched:

Patched:

In a way, I can see how both could be interpreted as valid. You either interpret the long way around the two identical hues, or you interpret the long way around the two identical hues. I honestly don't know which is right 🙃.

The latter seems more correct to me. @svgeesus?
Is this the only difference?

I kind of agree that I think the latter is the intended.

I'll see if I can get a CSS Color 4 issue raised later today.

Since the CSS spec is not properly describing the intent, and the intent is being followed here, I will close this PR.

CSS now properly describes the intent, so color.js continues to be correct


Cool...?

Hi!
This seems like a really great idea! Where did you get the idea from initially?
Also it would be great if the contribution guidelines would be a bit more clear.

Is matikkaeditori.fi related to this project? They look pretty similar...

Hey @Esinko!
Thanks for your comment, I'll be updating the contribution guidelines and improving the docs this week. The idea came from Eira high school to build an open source editor that expands upon the features of the Abitti Rich Text Editor. This is not related to matikkaeditori.fi. Most importantly, this is open source so that other schools can roll out their own versions easily.

Alright! That makes sense.


#185 - Eliminate "error TS6064" while executing tsc --watch

Below command given in QuickStart.html, throws error when executed. Therefore suggesting an extra option to eliminate it.
tsc --watch -m commonjs -t es5 --emitDecoratorMetadata app.ts
error TS6064: Option 'experimentalDecorators' must also be specified when option 'emitDecoratorMetadata' is specified.
message TS6042: Compilation complete. Watching for file changes.
Works correctly with additional option.
tsc --watch -m commonjs -t es5 --emitDecoratorMetadata --experimentalDecorators app.ts
message TS6042: Compilation complete. Watching for file changes.

Closing because this document has been completely rewritten and replaced, obviating the need for your changes. But many thanks to you for engaging and helping. We need that!


How does it relate to others backport libraries like "aiocontextvars"?

Hi. :)
While looking for a Python 3.5 backport of PEP 567, I come across your better-contextvars library but also the aiocontextvars (which itself relies on the contextvars backport).
Basically, I would like to know if there is any difference between your library and others ones? Did you decide to go with your own because you noticed problems in the alternative implementations?

Hello Delgan, from what I can tell, there isn't much difference. I was writing this to get myself a better understanding of how the contextvars module works internally and to challenge myself as I had no experience with context variables and it wasn't really a pleasure to read its original source code.
Since all of these modules are striving for similar functionality, they should be fairly similar. From a neutral viewpoint, it shouldn't matter which implementation you choose in the end.

Thanks for the clarification! Looking at the source code, your two libraries are very similar, so it should not make much difference as you said. Just wanted to be sure. Thanks. :+1:


[Misc] Update CoinsList symbol and unit font colour

Symbol and unit font colour should be light-secondary in Running mode.

Figma:

App:

Also, coin units text should vertically align with amount text.

Text vertical alignment corrected in #214.


Adds new device dump service

This adds a new "action" to Home Assistant that'll allow a user to view the list of devices in their setup.

Bump version to patch since this is added to help debug an issue.

Hey,
would "diagnostics" be a good place to put this?
e.g. like in https://github.com/p0l0/hagruenbeck_cloud/blob/main/custom_components/gruenbeck_cloud/diagnostics.py


qt_advanced_docking_system: 3.8.2-3 in 'noetic/distribution.yaml' [bloom]

Increasing version of package(s) in repository qt_advanced_docking_system to 3.8.2-3:

upstream repository: https://github.com/githubuser0xFFFF/Qt-Advanced-Docking-System
release repository: https://github.com/tesseract-robotics-release/qt_advanced_docking_system-release.git
distro file: noetic/distribution.yaml
bloom version: 0.10.7
previous version for package: null

@hidmic These are fixed in PR #32648


Distributed training: Speed extremely slow

Environment info
Operating System: Centos 7.0
Package used (Python/R/Scala/Julia): Python
MXNet version: 0.93
Python version and distribution: 2.7
We are testing the distributed train over multi-machine mxnet.  We compile Mxnet with parameter "USE_DIST_KVSTORE=1" , and successful run train a MLP on mnist.   But we found that the speed of distributed training was unusually slow compared to the speed of stand-alone training.
Stand-alone commands and speed：
python train_mnist.py
INFO:root:start with arguments Namespace(batch_size=64, disp_batches=100, gpus=None, kv_store='device', load_epoch=None, lr=0.05, lr_factor=0.1, lr_step_epochs='10', model_prefix=None, mom=0.9, monitor=0, network='mlp', num_classes=10, num_epochs=20, num_examples=60000, num_layers=None, optimizer='sgd', test_io=0, top_k=0, wd=0.0001)
INFO:root:Epoch[0] Batch [100]  Speed: 21875.54 samples/sec     Train-accuracy=0.771040
INFO:root:Epoch[0] Batch [200]  Speed: 21260.40 samples/sec     Train-accuracy=0.913906
INFO:root:Epoch[0] Batch [300]  Speed: 21302.58 samples/sec     Train-accuracy=0.933750
INFO:root:Epoch[0] Batch [400]  Speed: 21216.02 samples/sec     Train-accuracy=0.936875
INFO:root:Epoch[0] Batch [500]  Speed: 22835.21 samples/sec     Train-accuracy=0.933594
INFO:root:Epoch[0] Batch [600]  Speed: 21612.71 samples/sec     Train-accuracy=0.947500
INFO:root:Epoch[0] Batch [700]  Speed: 23362.35 samples/sec     Train-accuracy=0.954375
INFO:root:Epoch[0] Batch [800]  Speed: 21683.75 samples/sec     Train-accuracy=0.954219
INFO:root:Epoch[0] Batch [900]  Speed: 21656.14 samples/sec     Train-accuracy=0.955781
INFO:root:Epoch[0] Train-accuracy=0.958615
INFO:root:Epoch[0] Time cost=2.804
INFO:root:Epoch[0] Validation-accuracy=0.957803
Command and speed of distributed training on local machine by using two workers:
launch.py -n 2 --launcher local python train_mnist.py --kv-store dist_sync
INFO:root:start with arguments Namespace(batch_size=64, disp_batches=100, gpus=None, kv_store='dist_sync', load_epoch=None, lr=0.05, lr_factor=0.1, lr_step_epochs='10', model_prefix=None, mom=0.9, monitor=0, network='mlp', num_classes=10, num_epochs=20, num_examples=60000, num_layers=None, optimizer='sgd', test_io=0, top_k=0, wd=0.0001)
INFO:root:start with arguments Namespace(batch_size=64, disp_batches=100, gpus=None, kv_store='dist_sync', load_epoch=None, lr=0.05, lr_factor=0.1, lr_step_epochs='10', model_prefix=None, mom=0.9, monitor=0, network='mlp', num_classes=10, num_epochs=20, num_examples=60000, num_layers=None, optimizer='sgd', test_io=0, top_k=0, wd=0.0001)
INFO:root:Epoch[0] Batch [100]  Speed: 2580.90 samples/sec      Train-accuracy=0.104425
INFO:root:Epoch[0] Batch [100]  Speed: 2574.24 samples/sec      Train-accuracy=0.096689
INFO:root:Epoch[0] Batch [200]  Speed: 4408.62 samples/sec      Train-accuracy=0.093594
INFO:root:Epoch[0] Batch [200]  Speed: 4399.50 samples/sec      Train-accuracy=0.099531
INFO:root:Epoch[0] Batch [300]  Speed: 3343.81 samples/sec      Train-accuracy=0.095469
INFO:root:Epoch[0] Batch [300]  Speed: 3342.38 samples/sec      Train-accuracy=0.097344
INFO:root:Epoch[0] Batch [400]  Speed: 3247.69 samples/sec      Train-accuracy=0.096250
INFO:root:Epoch[0] Batch [400]  Speed: 3245.97 samples/sec      Train-accuracy=0.104531
INFO:root:Epoch[0] Batch [500]  Speed: 3364.46 samples/sec      Train-accuracy=0.102188
INFO:root:Epoch[0] Batch [500]  Speed: 3364.97 samples/sec      Train-accuracy=0.098437
INFO:root:Epoch[0] Batch [600]  Speed: 3729.89 samples/sec      Train-accuracy=0.097500
INFO:root:Epoch[0] Batch [600]  Speed: 3732.76 samples/sec      Train-accuracy=0.097812
INFO:root:Epoch[0] Batch [700]  Speed: 5105.08 samples/sec      Train-accuracy=0.087969
INFO:root:Epoch[0] Batch [700]  Speed: 5097.00 samples/sec      Train-accuracy=0.104375
INFO:root:Epoch[0] Batch [800]  Speed: 3931.05 samples/sec      Train-accuracy=0.099062
INFO:root:Epoch[0] Batch [800]  Speed: 3930.63 samples/sec      Train-accuracy=0.101406
INFO:root:Epoch[0] Batch [900]  Speed: 3763.65 samples/sec      Train-accuracy=0.098906
INFO:root:Epoch[0] Batch [900]  Speed: 3758.21 samples/sec      Train-accuracy=0.099531
INFO:root:Epoch[0] Train-accuracy=0.104307
INFO:root:Epoch[0] Time cost=16.560
INFO:root:Epoch[0] Train-accuracy=0.099662
INFO:root:Epoch[0] Time cost=16.584
INFO:root:Epoch[0] Validation-accuracy=0.098029
INFO:root:Epoch[0] Validation-accuracy=0.098029
Command and speed of distributed training on two computers(with same hardware configration):
launch.py -n 2 --launcher ssh -H hosts python train_mnist.py --kv-store dist_sync
INFO:root:start with arguments Namespace(batch_size=64, disp_batches=100, gpus=None, kv_store='dist_sync', load_epoch=None, lr=0.05, lr_factor=0.1, lr_step_epochs='10', model_prefix=None, mom=0.9, monitor=0, network='mlp', num_classes=10, num_epochs=20, num_examples=60000, num_layers=None, optimizer='sgd', test_io=0, top_k=0, wd=0.0001)
INFO:root:start with arguments Namespace(batch_size=64, disp_batches=100, gpus=None, kv_store='dist_sync', load_epoch=None, lr=0.05, lr_factor=0.1, lr_step_epochs='10', model_prefix=None, mom=0.9, monitor=0, network='mlp', num_classes=10, num_epochs=20, num_examples=60000, num_layers=None, optimizer='sgd', test_io=0, top_k=0, wd=0.0001)
INFO:root:Epoch[0] Batch [100]  Speed: 614.03 samples/sec       Train-accuracy=0.097463
INFO:root:Epoch[0] Batch [100]  Speed: 613.66 samples/sec       Train-accuracy=0.096225
INFO:root:Epoch[0] Batch [200]  Speed: 577.24 samples/sec       Train-accuracy=0.097812
INFO:root:Epoch[0] Batch [200]  Speed: 576.24 samples/sec       Train-accuracy=0.096250
INFO:root:Epoch[0] Batch [300]  Speed: 609.78 samples/sec       Train-accuracy=0.094531
INFO:root:Epoch[0] Batch [300]  Speed: 610.73 samples/sec       Train-accuracy=0.098125
INFO:root:Epoch[0] Batch [400]  Speed: 624.24 samples/sec       Train-accuracy=0.100000
INFO:root:Epoch[0] Batch [400]  Speed: 623.74 samples/sec       Train-accuracy=0.100625
INFO:root:Epoch[0] Batch [500]  Speed: 642.81 samples/sec       Train-accuracy=0.096719
INFO:root:Epoch[0] Batch [500]  Speed: 643.28 samples/sec       Train-accuracy=0.099531
INFO:root:Epoch[0] Batch [600]  Speed: 613.65 samples/sec       Train-accuracy=0.097187
INFO:root:Epoch[0] Batch [600]  Speed: 613.63 samples/sec       Train-accuracy=0.096406
INFO:root:Epoch[0] Batch [700]  Speed: 626.13 samples/sec       Train-accuracy=0.106875
INFO:root:Epoch[0] Batch [700]  Speed: 626.15 samples/sec       Train-accuracy=0.096562
INFO:root:Epoch[0] Batch [800]  Speed: 621.21 samples/sec       Train-accuracy=0.098437
INFO:root:Epoch[0] Batch [800]  Speed: 620.83 samples/sec       Train-accuracy=0.099687
INFO:root:Epoch[0] Batch [900]  Speed: 589.75 samples/sec       Train-accuracy=0.098281
INFO:root:Epoch[0] Batch [900]  Speed: 589.72 samples/sec       Train-accuracy=0.104844
INFO:root:Epoch[0] Train-accuracy=0.101351
INFO:root:Epoch[0] Time cost=98.129
INFO:root:Epoch[0] Train-accuracy=0.097551
INFO:root:Epoch[0] Time cost=98.144
INFO:root:Epoch[0] Validation-accuracy=0.098029
INFO:root:Epoch[0] Validation-accuracy=0.098029
INFO:root:Epoch[1] Batch [100]  Speed: 639.28 samples/sec       Train-accuracy=0.097308
INFO:root:Epoch[1] Batch [100]  Speed: 640.32 samples/sec       Train-accuracy=0.096844
We do not know why the speed so slow in distributed training , someone can help us?
Thanks

please try a larger workload, such as train_cifar10. and also there are some suggestions: https://github.com/mli/mxnet/tree/master/example/image-classification#speed

@mli Let's add a warning somewhere to tell people training small networks on multiple machines doesn't make sense.

good idea.

I am experimenting with MXNMT from https://github.com/magic282/MXNMT.
The training data is 1million. I have 2 machines each with 8 GPUs (I only use the first 6).
When I launch the job with /path/to/launch.py -n 2 --launcher ssh -H hosts python main.py, and batch_size=360(each gpu 60), kvstore=dist_sync, the performance is:

When with /path/to/launch.py -n 2 -s 4 --launcher ssh -H hosts python main.py, the performance is:

When with /path/to/launch.py -n 2 -s 6 --launcher ssh -H hosts python main.py, the performance is:

My question is how to set server number, and among these servers, which are level 1 servers, which are level 2 servers (According to https://github.com/dmlc/mxnet/issues/797).


Definitions

Reading this, I want Vendor and Customer defined so there is no doubt.
Maybe other definitions could be included.

Feel free to send a PR


Table of content highlight don't work properly when you scroll

Table of content highlight don't highlight the few last content you scroll to the bottom of the website, highlight match the first item visible on the screen.
It is visible when you scroll down on your home website, Contributing will never be highlighted.

Hi, @kevlarcade. Thanks for the feedback. This has been fixed via https://github.com/huhu/juice/commit/c2ba14f473dccaa040af5034b9d76543b57e69df.


added main file to bower.js to support automatic linking

Having the main js file in your bower.js allows generators (see yeoman.io) to automatically link dependancies to your project.

@Mikhus Can you merge this?  This really is a no-brainer one-line change.


Member updates

Pre-submission checklist:
Please check each of these after submitting your pull request:

[ ] Are you only including a repo_url if your project is 100% open source? If so, you need to pick the single best GitHub repository for your project, not a GitHub organization.
[ ] Is your project closed source or, if it is open source, does your project have at least 300 GitHub stars?
[ ] Have you picked the single best (existing) category for your project?
[ ] Does it follow the other guidelines from the new entries section?
[ ] Have you included a URL for your SVG or added it to hosted_logos and referenced it there?
[ ] Does your logo clearly state the name of the project/product and follow the other logo guidelines?
[ ] Does your project/product name match the text on the logo?
[ ] Have you verified that the Crunchbase data for your organization is correct (including headquarters and LinkedIn)?
[ ] ~15 minutes after opening the pull request, the CNCF-Bot will post the URL for your staging server. Have you confirmed that it looks good to you and then added a comment to the PR saying "LGTM"?

Build failed because of:
No cached entry, and Hyve Managed Hosting (member) has issues with twitter: https://twitter.com/HyveSecureCloud, 404 - {"errors":[{"code":34,"message":"Sorry, that page does not exist."}]}

Deploy preview for landscape failed.
Built with commit 6f959939234c8b5cd3311b6119edabc43eb9d7e3
https://app.netlify.com/sites/landscape/deploys/5f653ba17460c90009554564

Build failed because of:
CloudThat (member)  has a crunchbase https://crunchbase.com/organization/cloudthat which does not start with 'https://www.crunchbase.com/organization'

LGTM


fix typo in document

Release`` or Debug->ReleaseorDebug`.

Thank you for the patch, but as we explained in #382, we are not currently accepting any external contributions since it would require us to move more than 2,000 file in our internal repository.  Good news is that the typo is already addressed by 280e38fe3d9db4df52f0713acf2ca65898cd697a so we can just close this thread you don't need to sign our CLA (Contributor License Agreement) anymore.
Thanks again for your patch and we sincerely apologize for not being able to accept your patch.


✨ Enhance Reward Page with Customizable Timer ⏲, Soothing Music 🎵, and Header Fix 🛠Issue/Suggestion

Before creating an issue, go through all open/closed issues to avoid getting your issue/suggestion marked as duplicate##
Issue and Steps to Reproduce:

I would like to propose enhancements to the Reward Page that would improve the user experience and customization options.

The CSS of the timer needs improvement to allow for more customization features, such as the ability to select the timer duration.
The timer should display a large pop-up or notification while it is running, ensuring that users are aware of the ongoing countdown.
It would be great to have relaxing music options available on the Reward Page, providing users with a soothing atmosphere during their tasks with various options and customization options

The navigation bar needs fixing to ensure smooth and consistent navigation across the page.

Screenshots:

Expected:

1. Users should be able to customize the timer duration, allowing them to set specific time intervals according to their preferences.
2. The timer should prominently display a pop-up or notification to draw users' attention while it is active.
3. The Reward Page should offer various relaxing music options, enabling users to choose their preferred background ambiance.
4. The navigation bar should be fixed to ensure seamless and consistent navigation throughout the page.
Actual:

Currently, the Reward Page does not offer sufficient customization options for the timer, lacks a prominent pop-up or notification while the timer is running, does not provide relaxing music options, and has issues with the navigation bar.

@akshitagupta15june Please take a moment to review the issue I've raised. I am confident that you will find it valuable and not regret considering it.