Improper progress during search

See attached gif, the progress bar quickly goes to ~90% and then takes a long while until it reaches 100%.

@jrieken can you zip the workspace and attach and include your settings?

unsure if that is needed: https://github.com/Microsoft/vscode/blob/master/src/vs/workbench/parts/search/browser/searchViewlet.ts#L1405
tho, it is the workspace described here: Microsoft/TypeScript#7444

@jrieken the progress is only fake until I know how many files to search. since file walking can take forever, and I still want to show something, there can be cases where the fake progress goes to 90% and only then a lot of more files are discovered.
There is really no good way of showing real progress without knowing the number of files to search in upfront (e.g. an index would help).

I wonder if we should show the infinite progress as long as we don't know how many files there are?

Yeah, that could work.

Actually I notice that 2 files in this workspace are very large (> 6 MB) and essentially the search is really busy processing those 2 files in the end and the file based progress is correct. I pushed a change to adjust the progress based on the size of the file. Let me know how it goes.


Add globals for exec from plaintext secrets YAML

Allows drone exec to use the plaintext of the secrets YAML to populate environment globals for local builds.

I'm writing some tooling to wrap drone-cli (most of our non-engineer users don't care what they're using, they just want to do a final test run before pushing) and this would be very useful for that. My wrapper could point to an existing file with variables, or even generate and crap one out into /tmp if it needed to be a dynamic decision.
Will have to stop by again later to read the changes over in more detail, but conceptually I really like this!

LGTM but about to leave for lunch. Will review when I get back!

Take your time!

@benschumacher Can I bug you to update this? https://github.com/drone/docs/blob/master/content/devs/cli.md#local-testing

@gtaylor Absolutely. Will do.

@benschumacher Did this ever end up happening?


version: add error handling

Signed-off-by: hwdef<EMAIL_ADDRESS>
Ⅰ. Describe what this PR did
add error handling
Ⅱ. Does this pull request fix one issue?

Ref: #952
Ⅲ. Why don't you add test cases (unit test/integration test)? (你真的觉得不需要加测试吗？)
Ⅳ. Describe how to verify it
Ⅴ. Special notes for reviews

Codecov Report

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

@@           Coverage Diff           @@
##           master    #1003   +/-   ##
=======================================
  Coverage   46.54%   46.54%
=======================================
  Files         116      116
  Lines        6903     6903
=======================================
  Hits         3213     3213
  Misses       3434     3434
  Partials      256      256

Continue to review full report at Codecov.

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

LGTM，code format is the way you talk, communicate. We are always bringing convenience when delivering code. Thanks for your update. Wish to write strict and friendly code together. @hwdef


Error loading 2.1.5

Hi,
I'm getting no devices and the following logs on loading 2.1.5:
Logger: homeassistant.components.climate
Source: custom_components/mitsubishi/climate.py:68
Integration: Climate (documentation, issues)
First occurred: 8:26:33 PM (5 occurrences)
Last logged: 8:26:33 PM

Error while setting up mitsubishi platform for climate
Traceback (most recent call last):
  File "/usr/src/homeassistant/homeassistant/helpers/entity_platform.py", line 250, in _async_setup_platform
    await asyncio.shield(task)
  File "/config/custom_components/mitsubishi/climate.py", line 47, in async_setup_platform
    entities.append(MitsubishiClimate(config.get(CONF_NAME),
  File "/config/custom_components/mitsubishi/climate.py", line 68, in __init__
    self._uid = echonet_hvac.getIdentificationNumber()["identification_number"]
AttributeError: 'HomeAirConditioner' object has no attribute 'getIdentificationNumber'

I have AP series heat pumps.

Can you try updating the 'mitsubishi_echonet' library to 0.5?

Thanks Scotty, yes am running HACs and yes have 0.5.1 echonet library (and no others). Might try restarting the docker container.

Yes I would try restarting everything to begin with, and then we can validate which libraries are installed to the core home assistant container with pip, do you know how to run an interactive session to a running docker container?
Regards,
Scott
From: Alfiegerner @.>
Reply-To: scottyphillips/mitsubishi_hass @.>
Date: Tuesday, 15 June 2021 at 6:49 pm
To: scottyphillips/mitsubishi_hass @.>
Cc: Scott Phillips @.>, Comment @.***>
Subject: Re: [scottyphillips/mitsubishi_hass] Error loading 2.1.5 (#27)
Thanks Scotty, yes am running HACs and yes have 0.5.1 echonet library (and no others). Might try restarting the docker container.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or unsubscribe.

Yep restarting docker container fixed it. Bit odd as I did an interactive session into the container to confirm that 0.5.1 was there, but hey ho, all working now. Thanks for everything.

No problem, im an network engineer by trade so all troubleshooting begins with 'lets reboot' lol


Review of Dopa-responsive dystonias

I am trying to align MedGen concepts with Mondo/Orpha/OMIM/SNOMED (and eventually UMLS CUI mapping) and am running into some issues with names, which will probably take review by multiple groups to potentially sort out...
MONDO:0016812  dopa-responsive dystonia (DRD) is currently parent to:

Mondo ID
Mondo Name
OMIM Xref (name)
Orpha xREF (name)
SNOMED xREF(name)
Associated Gene

MONDO:0011551
TH-deficient DRD
605407 (SEGAWA SYNDROME, AUTOSOMAL RECESSIVE)
101150 (Autosomal recessive dopa-responsive dystonia, aka DYT5a)
715827001 (Autosomal recessive dopa responsive dystonia)
TH  (Orphanet also has  TSPOAP1)

MONDO:0012994
(DRD) due to sepiapterin reductase deficiency
612716 (Dystonia, dopa-responsive, due to sepiapterin reductase deficiency)
70594 (Dopa-responsive dystonia due to sepiapterin reductase deficiency)
45116002 (Sepiapterin reductase deficiency)
SPR

MONDO:0007495
Dystonia 5
128230 (DYSTONIA, DOPA-RESPONSIVE)
98808 (Autosomal dominant dopa-responsive dystonia, aka DYT5b)
715768000 (Autosomal dominant dopa responsive dystonia)
GCH1, GCH1 - AD or AR in OMIM; GCH1, IMPDH2 NR4A2 in Orpha*

Other OMIM records that may be relevant:
233910 INCLUDED entry:  DYSTONIA, DOPA-RESPONSIVE, WITH OR WITHOUT HYPERPHENYLALANINEMIA, AUTOSOMAL RECESSIVE, Gene on this record in OMIM is GCH1.
The SNOMED CT data, as represented in UMLS has:
230332007 “Diurnal dystonia” with three children:
-Autosomal Dominant (AD) DRD
-Autosomal recessive(AR) DRD
-DRD due to Sepiapterin reductase deficiency
So it looks like there are really 4 children concepts… Maybe?
I.	Dopa-responsive dystonia (aka Diurnal dystonia)
a.	AD DRD due to GCH1 deficiency  (primary concept on OMIM 128230?)
b.	AR DRD due to GCH1 deficiency (included concept on OMIM 233910?)
c.	AR DRD due to TH deficiency (aka AR Segawa syndrome, OMIM 605407)
d.	(DRD) due to sepiapterin reductase deficiency (AR, ?AD) (OMIM and OrphaIDs seem OK on this one)
@ahamosh Is OMIM Moving all the GCH1-associated DRDs to 128230 ? Right now the AR concept is 'included' on 233910,  but the Gene:Phenotype table on 128230 has both AD and AR inheritance modes. Would OMIM consider updating the preferred name on 128230 so that 'Dopa responsive dystonia' is a synonym since it seems there are at least 3 genes that contribute to DRD phenotypes?
I'm not sure where "DTY5a" and "b" come from and if DYT5 (dystonia 5) should really be a grouping term? OMIM links this to GCH1 associated DRD; but Orpha's "a" and "b" forms are associated with two different genes.
This article in the Genetics section of MedLinePlus suggests "Dystonia 5, dopa responsive type" describes all the DRDs, regardless of genetic cause: https://medlineplus.gov/genetics/condition/dopa-responsive-dystonia/
*side/final notes Regarding Orpha’s other genes associated with 98808:

Gene NR4A2 (a nuclear receptor) is associated with OMIM phenotype 619911 (Dopa-responsive parkinsonism as a feature but also with intellectual disability)
-Gene IMP DEHYDROGENASE 2; IMPDH2 is associated with variants that affect enzyme activity but no DRD or movement disorder type condition seems to be specifically associated with this gene in OMIM.

Other OMIM records that may be relevant:
233910 INCLUDED entry: DYSTONIA, DOPA-RESPONSIVE, WITH OR WITHOUT HYPERPHENYLALANINEMIA, AUTOSOMAL RECESSIVE, Gene on this record in OMIM is GCH1.

this is GTP cyclohydrolase I deficiency with hyperphenylalaninemia, which is a sibling of dystonia 5

Would OMIM consider updating the preferred name on 128230 so that 'Dopa responsive dystonia' is a synonym since it seems there are at least 3 genes that contribute to DRD phenotypes

OMIM:128230 is equiv to MONDO:0007495 Dystonia 5

@kanems I'm not sure I follow everything you said above. This is what we currently have in Mondo. Could you rephrase the specific action items needed, thanks!

Thanks for the clarification @kanems! I'll create the new term: dystonia, dopa-responsive, with or without hyperphenylalaninemia, autosomal recessive
I'm unsure if AR, BH4 deficient should be a child of DRD, so I will just leave it for now, unless you or someone else tells me otherwise.


WIP: Add New Info Unit Groups

Do not merge yet. This is WIP because it has a couple major todos, but I wanted to get it up in case folks wanted to start reviewing the basics.
Additions

A new Info Unit Group Wagtail module
A new Info Unit Group organism template to render the above
A new Info Unit molecule template to render Info Units belonging to the above
Three new heading blocks that get grouped together in Wagtail (using the new struct-with-block-wrapper-classes.html form_template and the new heading-blocks.css for a simpler interface for creating headings at different levels, optionally with icons

Changes

Minor updates to the existing m-info-unit_image styling that help my new Info Units but don't break anything with existing info units

Testing

Pull branch
gulp styles
./runserver.sh
Create a page, add an Info Unit Group to the content area, and play around with the various options.
Preview the page

Screenshots

Notes

I created brand new templates and left the old Image and Text and Link Blob stuff in place, so that we can get this in production without immediately worrying about migrating the old stuff. We can either migrate them by hand or write a script to do it in another PR.

Todos

Finish tidying up the styling, most notably vertical spacing
Update/write tests
Take another pass at the templates' docstrings and make sure I didn't miss anything
Determine if we're using the stroke between 25/75s enough to warrant keeping that option, or if we use it enough that we may as well always use it

Checklist

[ ] PR has an informative and human-readable title
[ ] Changes are limited to a single goal (no scope creep)
[ ] Code can be automatically merged (no conflicts)
[ ] Code follows the standards laid out in the development playbook
[ ] Passes all existing automated tests
[ ] New functions include new tests
[ ] New functions are documented (with a description, list of inputs, and expected output)
[ ] Placeholder code is flagged
[ ] Visually tested in supported browsers and devices
[ ] Project documentation has been updated
[ ] Reviewers requested with the Assignee tool :arrow_right:

Added a todo to figure out what to do about lines between 25/75 items.

@Scotchester, what are chief differences between the existing info-unit and this one?

what are chief differences between the existing info-unit and this one?

Visually, there shouldn't really be any (pending the outcome of the last todo item about rule lines).
Behind the scenes, just a bit of adjustment to the template, the primary effect of which is that decorative images are now correctly output as divs width a background-image if they have no alt text set.
On the Wagtail side, rather than having separate H3 & H4 fields, you can now choose the heading level you want and then optionally add an icon. I forgot to add a screenshot of the Wagtail interface originally; I've just added that to the OP.

This looks awesome

@Scotchester, why no make the icons a dropdown?

why no make the icons a dropdown?

Well, I didn't think about it too hard, to be honest. I just followed the example of the existing H4 icon text field. There are a lot of icons, though. Unless we tried to pare it down to some subset allowed for this use case, a dropdown would be massive.

How hard is it to make this a feature flag so we can test the change on build before making it available more widely? Platform team folks including myself @anselmbradford @nataliafitzgerald @ielerol can test this.

Alternately: give us a heads up when this is merged and we volunteer to test it heavily on build!

Here are some screenshots illustrating the margin that needs to be changed from 15 to 30px:

How it looks with 30px margin above the blobs:

Another visual bug - when adding more than 3 items to 33/33/33 layout, the new row doesn't have a margin-top or whitespace. I would expect it to have the same whitespace that a new row in 50/50 layout would have:

For each link blob, there is this checkbox option in Wagtail:
"Show links as button:"

What's the use case for this? Is there a scenario in which we want content editors to add some links as regular links, and some as buttons, within the same info unit? If not, can we improve the UI so you can make all links a button, otherwise default to regular link?

@Scotchester
First off, great work! This will solve a number of spacing issues we've been having as well as giving users more flexibility in terms of heading sizes and such. Merging the "Image text 50/50 group," "Half width link blob group," "Image text 25/75 group," and "Image text 33/33/33 group" into one mega module makes total sense.
One thing I noticed is that we are adding functionality to the "Image text 25/75 group," and "Image text 33/33/33 group" that didn't exist before: The option to use a button instead of a link. Before, the option to pick between a button and a link was only available for the "Image text 50/50 group," and "Half width link blob group."

Have users (our team) asked for this added functionality or had a need for this in their projects? I don't know of any use cases that exist thus far (for any of the modules that are wrapped up into this mega module).
@ielerol - Any thoughts?

@Scotchester @sebworks
In terms of the icons, if a dropdown is too massive, how about a live/clickable link in the interface that takes you to the page where all the icon names are? This is something that is included in the existing interface but the link isn't clickable. We could use a link that will take you to the part of the icon page that is relevant to users that are looking for the names: https://cfpb.github.io/capital-framework/components/cf-icons/#icons

in regards to showing links as buttons, our current guidance is to use them sparingly to keep their impact high. I can't think of many cases where it would be desirable to use a whole series of buttons in third-width link blobs, or to add a single button in a group of link blobs that otherwise have text links. In fact I haven't even found any examples where we're using buttons in 50/50s on current pages. The closest I've found is the CCDB page, which is being replaced with the new public complaint search, and in any case isn't in Wagtail.
I want a little more time to discuss with other designers in case there are examples I'm missing, but for now I'm leaning towards eliminating buttons as an option from this module altogether, and just leaving the full-width CTA as the main way to add buttons to the body of a page.

Thanks for all the feedback! I will be working on addressing it today.

In terms of the icons, if a dropdown is too massive, how about a live/clickable link in the interface that takes you to the page where all the icon names are?

@nataliafitzgerald Thanks for nudging me to dig deeper on that. I found an easy way to enable rendering of links in a field's help text! Fixed in 1c9a814fc1d863afc413591755457aca68ca2468.

@cfarm Vertical spacing issues have been addressed in 0605626.

@ielerol @nataliafitzgerald @cfarm
I would be more than happy to remove the option to show links as buttons. What do we need to do / who else do we need to discuss with in order to be comfortable with that decision?

@Scotchester - I think the new process is that we consult with @schaferjh or John Paul when there is a lack of consensus. It seems like there's consensus here but if we want to be safe I'd say we reach out to @schaferjh. I don't exactly know what consensus means in a technical sense (as in how many people need to agree on something to reach this consensus).

I've asked @jimmynotjim and @keelerr if they know of any places where 50/50s are currently using buttons, and they haven't used them in any of their work, and have no objections to removing the option. @schaferjh might know some more context around the original decision to include buttons in that module.
I do think it would be a good idea to let the design team know generally that we want to make this change in case anyone does have a need, I can bring it up.

I did a full review of just about every page Wagtail page on the site (excluding things like individual blog posts and report pages and such) and couldn't find a single example of a button at the bottom of an Image and Text or Link Blob module.

Also, during my review, I had the opportunity to observe the usage of the horizontal rule lines between 25/75 items. I didn't take notes or anything, but it felt like they're used about half the time.
In my personal opinion, the spacing of items without the horizontal rules feels a bit tight. I'd like to propose that we always place a horizontal rule between 25/75 items. Anyone strongly against doing that?

@Scotchester
Regarding making the horizontal rule line the default for 25/75's I'd want us (designers) to have a chance to take a look at this pattern across the site before deciding whether to move in that direction.

@nataliafitzgerald Absolutely, please do!

Quick note on timing before I leave for the day:
This needs to get merged and deployed no later than Tuesday afternoon in order for Rara to have the time they'll need to build the pages they're going to launch on Thursday. Monday morning I will finish the tests.
If the design decisions are not settled by Tuesday morning, I will move forward with what seems to be the majority opinion at the time. Note again that deploying this code will not affect any existing Image and Text or Link Blob modules on the site, so if we need to change course from what happens here, we can do that before migrating the existing modules to this new module.

I took a look at the 25/75 pattern across a collection of pages. After looking at a variety of examples, I do not feel that we should always place the horizontal rule below 25/75 items. We have a lot of instances across the site where we are currently using the 25/75 without the rule line. Continuing to give designers the flexibility to determine whether a rule line makes sense or not for a specific situation makes the most sense to me (and will make designers feel less locked in).
Instances where no rule line makes sense:
There are a lot of examples of places where we're currently using the 25/75 without the rule line. Here are some examples.
Set of things:
Instances where the 25/75 items are part of a group of similar things. For example, on the "Everyone has a story" page where we include a group of stories. These stories are part of a set and there is no need from a visual design perspective to emphasize the visual separation. Including rule lines here would be unnecessary and create undue visual clutter.

I have noticed when using the 25/75 pattern (and most of our patterns) that guiding our stakeholders as to word counts can be very beneficial. You can see how word counts helped to create equal units here (so each unit has the same level of importance on the page):

Numbered steps

Single 25/75

Landing page with variety of sections
This example works without the rule because the two blocks both have 4 lines of supporting copy. In general these patterns work best when they have a similar or the same number of lines of text (within a group).

Here are the guidelines from the DM:

Instances where including a rule line makes sense:
Things that are grouped together but which may benefit from some visual separation
This example would benefit from word count guidance for stakeholders. The varying lengths of content create spacing that looks uneven and gives some of the units more visual impact/dominance than the others. Additionally the heading "What is a prepaid card" (H2) doesn't fulfill the parent relationship I would expect. If the 25/75's are all recommended steps or things to do then the H2 heading should ideally be written to reinforce the common thread of the set. This would also help users that skim content by reading headings. It is possible that with content parameters and a clearly unifying heading that the horizontal rule would be less necessary here.

This example would also benefit from word count guidance for stakeholders for the same reasons mentioned in the previous example. Additionally, this example doesn't have a unifying heading which makes it a little harder to digest from a content perspective. It leaves me asking, what is the unifying idea for all of these things (25/75's). If they are steps then the heading could have the word steps in it.

In looking through a variety of examples, I notice that there is a point where the content gets so long that the spacing between the units starts to look off. I haven't identified what this breaking point is but talking through it with @Scotchester it's when we exceed the minimum height. That is what causes the spacing to look like it does on this example (look at spacing after the third section before the rule line):

If you change the blocks to a max of 3 lines each, you get the ideal spacing (for using the rule line):

With that being said, once the content is wrangled the rule line isn't absolutely necessary:

@Scotchester
I will be out on Monday -Wednesday of next week but my official position is:

Button option should be removed from the 25/75, 50/50, 33/33/33 (we can always revisit if a use case happens to come up in the future)
Horizontal rule lines should be optional not mandatory when using the 25/75's

@Scotchester, let's go with the drop down.

Awesome work on this module, @Scotchester! It looks so great. And thanks for pushing it through this quickly to help team Rara.
With the horizontal rules, I lean toward making them optional for the reasons @nataliaf posted regarding items' relatedness. Rara isn't currently stacking 25/75 modules on portal pages. But more broadly speaking, we're creating dual topic pages (i.e. Mortgages having two large sections— origination and servicing), and we're using a horizontal rule as a means to separate the two large sections on the page. So, I lean toward offering flexibility with rule use in modules with the thought that this might help teams down the road ensure the right points of separation/relation are in place for the whole of the page.

Overall looking good. I made a strategic comment over in the Hub issue, addressing specific issues about implementation raised here:

Links as buttons: this was included originally to give the modules flexibility. At the time there was an example on either Press Resources or About Us that needed it, though clearly they are not implemented now. I do not see the button option documented in the DM, so let's remove it.
Please maintain the rule line as an option.
Would the mincon drop down be a filter-as-you-type drop? If not, I don't support a drop down for 90 items.

Pushed some updates that should address all of the remaining issues. @cfarm and @jimmynotjim please re-review.
I need to add a few more tests for the custom validator that I just added to ensure that 25/75s have images, but if you could give this a look tomorrow morning, @chosak, I'd appreciate it!

Wrapped up the test stuff, and I think this is good to go, pending Travis's final check!

:tada:

Nice work @Scotchester!


Fixed Redundant empty Bookmarks Page

Fixes #338.
Added a simple check once the user presses the Bookmarks option on NavDrawer. If no bookmarks exist, it shows up a prompt rather than lead him to an empty screen, thus saving time of the user. So the BookmarksFragment gets loaded only when some sessions are bookmarked.
Also, if we un-bookmark a session and the Bookmarks list becomes empty again, it shows up the dialog on pressing the back button on that Session Detail, and clicking on OK loads the TracksFragment back again.
This also happens to be one of the layout changes suggested in my proposal, to reduce the blankiness in the application.
@championswimmer @mananwason @juslee

@shivenmian Screenshots please

@the-dagger Had deleted my clone because I'm on a different project rn, but you can merge it. Works. Instead of showing the present "No bookmarks yet" screen, it just shows a dialog if no bookmarks there (saving the user from a blank screen which does nothing). You can pull it and check.

Cool. 👍


[MXNET-72] Improve sparse.adam_update

Description
Fix https://discuss.mxnet.io/t/lazy-update-with-adam-optimizer-is-much-slower-for-sparse-input/724
Many kernels written for row_sparse ndarrays are slow on GPU. For example, in sparse SGD, the kernel has the following logic using kernel::launch:
# pragma parallel for
for (i = 0 ... num_nonzero_rows) {
    # thread i computes an entire row in the output
    row = indices[i]
    for (col = 0 .. num_columns)
        output[row][col] = weight - lr * grad
}

Such "parallelization by rows"(A) works fine on CPU but is problematic on GPU, because:

when num_nonzero_rows is small, only a small number of cuda threads are launched. GPU utilization will be low
the memory access pattern is horrible. GPU threads are launched as groups of warps. Each thread in the same warp are accessing different memory location, resulting 32x more unnecessary memory transactions.

Instead, the kernels on GPU should be "parallelized by the number of elements"(B) to update. This way, all threads in the same warp are accessing the same chunk of memory.
On the other hand, if I apply B on CPU with openmp, I see the performance is 3-4x slower. Hence I only applied B for GPUs in this PR. (I didn't dig deeper why this happens - the performance should be comparable if static openmp scheduling is used.I didn't check what default omp scheduling strategy is on my instance. Maybe @cjolivier01 has more insight on cpu performance?).
time(s) for 300 iterations for lazy_update=True (26x improvement)

nnr
1280
12800

Before
1.31863999367
13.2637498379

After
0.050815820694
0.430017948151

time(s) for 30 iterations for lazy_update=False (34x improvement)

nnr
1280
12800

Before
22.2087020874
21.9239070415

After
0.658735990524
0.678998947144

(Just want to trigger the CI. Will update destination branch to master later.)
Checklist
Essentials

[ ] Passed code style checking (make lint)
[ ] Changes are complete (i.e. I finished coding on this PR)
[ ] All changes have test coverage:
Unit tests are added for small changes to verify correctness (e.g. adding a new operator)
Nightly tests are added for complicated/long-running ones (e.g. changing distributed kvstore)
Build tests will be added for build configuration changes (e.g. adding a new build option with NCCL)
[ ] Code is well-documented:
For user-facing API changes, API doc string has been updated.
For new C++ functions in header files, their functionalities and arguments are documented.
For new examples, README.md is added to explain the what the example does, the source of the dataset, expected performance on test set and reference to the original paper if applicable
[ ] To the my best knowledge, examples are either not affected by this change, or have been fixed to be compatible with this change

Changes

[ ] Feature1, tests, (and when applicable, API doc)
[ ] Feature2, tests, (and when applicable, API doc)

Comments

If this change is a backward incompatible change, why must this change be made.
Interesting edge cases to note here

import time
import mxnet as mx
import numpy as np
import argparse

mx.random.seed(0)
np.random.seed(0)

parser = argparse.ArgumentParser(description='Bench updater')
parser.add_argument('--dim-in', type=int, default=240000, help='weight.shape[0]')
parser.add_argument('--dim-out', type=int, default=512, help='weight.shape[1]')
parser.add_argument('--nnr', type=int, default=5000, help='grad.indices.shape[0]')
parser.add_argument('--repeat', type=int, default=1000, help='num repeat')
parser.add_argument('--dense-grad', action='store_true')
parser.add_argument('--dense-state', action='store_true')
parser.add_argument('--cpu', action='store_true')
args = parser.parse_args()
dim_in = args.dim_in
dim_out = args.dim_out
nnr = args.nnr
ctx = mx.cpu() if args.cpu else mx.gpu()

ones = mx.nd.ones((dim_in, dim_out), ctx=ctx)

if not args.dense_grad:
    weight = ones.tostype('row_sparse')
    indices = np.arange(dim_in)
    np.random.shuffle(indices)
    indices = np.unique(indices[:nnr])
    indices = mx.nd.array(indices, ctx=ctx)
    grad = mx.nd.sparse.retain(weight, indices)
else:
    weight = ones.copy()
    grad = ones.copy()

if args.dense_state:
    mean = ones.copy()
else:
    mean = ones.tostype('row_sparse')

var = mean.copy()

# warmup
for i in range(10):
    mx.nd.sparse.adam_update(weight, grad, mean, var, out=weight,
                         lr=1, wd=0, beta1=0.9, beta2=0.99, rescale_grad=0.5, epsilon=1e-8)
weight.wait_to_read()
a = time.time()
for i in range(args.repeat):
    mx.nd.sparse.adam_update(weight, grad, mean, var, out=weight,
                         lr=1, wd=0, beta1=0.9, beta2=0.99, rescale_grad=0.5, epsilon=1e-8)
weight.wait_to_read()
b = time.time()
print(b - a)

Very nice catch! Do you have an estimation how much overall speedup this could bring? We could highlight this in the release notes

By the way, could you add the benchmark at tests/python/benchmark so we can use them leter on?

Ah we already got benchmarks at https://github.com/apache/incubator-mxnet/tree/master/benchmark

does this include the fix for wd & states?
Have we decided what to do with wd in general?

@piiswrong no it doesn't. It only supports wd=0. No decision for wd in general yet

@haojin2 @cjolivier01 @reminisce @anirudh2290 @rahul003 @ZiyueHuang could you help review?

LGTM!


feat(dbm-services): dbha monitor report event should include cluster_type

关联研发信息
dev_branch: feat/dbha-event-opt/#6073
target_branch: v1.5.0


YouTube API key verification failed

My error log shows the following errors:
jun 19 20:30:19 raspberrypi mopidy[27280]: ERROR    [YouTubeBackend-3] mopidy_youtube Failed to verify YouTube API key, disabling API
jun 19 20:30:19 raspberrypi mopidy[27280]: INFO     [YouTubeBackend-3] mopidy_youtube using bs4API
I created an API key for 'YouTube Data API v3' on https://console.developers.google.com/apis/
[youtube]
enabled = true
youtube_api_key = ***************************************
api_enabled = true
search_results = 30
The amount of stars in my configuration above represents the length of the API key which I copied from Google's API library.
I tried to delete the API key and create another one. I also tried to use another email address to create the API key. Both with no luck.
Do you have any suggestions?
Thanks in advance

Strange. How would you feel about sending me an API key that you have created and that you have confirmed doesn't work. I could test it, and then you can cancel it.

Thank you for your quick reply. I sent you an email with extra information including the full API key. Please let me know if this is sufficient for you to test.

Hi @Archiefkast - thanks for the above and for your email. I have tested your API key; the error I get is:
2020-06-20 20:46:46,691 ERROR [16374:YouTubeBackend-6] mopidy_youtube: search error "{'code': 403, 'message': 'The request cannot be completed because you have exceeded your quota.', 'errors': [{'message': 'The request cannot be completed because you have exceeded your quota.', 'domain': 'youtube.quota', 'reason': 'quotaExceeded'}]}"
2020-06-20 20:46:46,693 ERROR [16374:YouTubeBackend-6] mopidy_youtube: Failed to verify YouTube API key, disabling API

I'm not sure what your quota is, or what you might use it for, or how much using mopidy-youtube uses it up, but it looks like it has been exceeded?  Maybe you can have a look at your usage in the google console?

Thank your for your research!
The strange thing is, I see that there are a few requests on the API key that I generated yesterday. All these request results in an error:

In the quota section, the usage indicates 0 and it seems that no quota has been reached:

The limit of 0 queries per day could be your problem.  Mine has a limit of 10000 per day, and mopidy-youtube does use them.  You can see the small number of queries I ran with my API key before I tried with your API key:

This should be the cause. I will look for more information about this 0 requests per day limit. I like the API connection because of the personal playlists.
This issue can be closed now, it looks like it has nothing to do with the plugin.
Thank you very much for your effort @natumbri !

I deleted my project on https://console.developers.google.com/apis/ and created a new one. Then I created a new API key, entered it in my Mopidy configuration and restarted Mopidy. Now it is working! See the log below:
jun 20 14:13:57 raspberrypi mopidy[7249]: INFO     [YouTubeBackend-3] mopidy_youtube YouTube API key verified
Again, thank you very much @natumbri ! You made my day :)

Glad it is working for you! Would you mind just explaining a bit how you use personal playlists? I'm interested to know how people actually use the plugin, especially if they are using it in a way different to how I use it.

My setup:

Mopidy with the YouTube plugin
Icecast: a free, open source media streaming server (https://icecast.org/)
Home Assistant: a free, open source home automation solution (https://www.home-assistant.io/)
Google chromecast audio (in several rooms of my home)

Streaming through Icecast
I have configured the Mopidy audio output to stream through Icecast:
(https://docs.mopidy.com/en/release-2.2/audio/#streaming-through-icecast)
Select a playlist in Home Assistant to play in a room
In Home Assistant I created a dropdown for selecting a playlist (that will automatically load a YouTube playlist in Mopidy):

In Home Assistant I also created a dropdown for each room with a Google chromecast audio to select the Mopidy input:

The only thing I have to figure out is how I can let Home Assistant shuffle Mopidy's playlist after loading a YouTube playlist.
Please let me know if you have any further questions.

Thanks for the detailed description.  I'm doing something similar, but with snapcast instead of icecast, and domoticz instead of home assistant.
I don't know how home assistant works, but if it can do this
curl -d '{"jsonrpc": "2.0", "id": 1, "method": "core.tracklist.shuffle"}' -H 'Content-Type: application/json' http://localhost:6680/mopidy/rpc

... your playlist will be shuffled.

You're welcome. Thank you for the suggestion, I will try that one.

Also, if you want random play mode, instead of shuffling the playlist, you could do this:
curl -d '{"jsonrpc": "2.0", "id": 1, "method": "core.tracklist.set_random", "params": {"value": true}}' -H 'Content-Type: application/json' http://localhost:6680/mopidy/rpc

One good thing about snapcast is the client runs on android (so you can use an old android phone as a receiver) and on openwrt, so you can use just about any device that is supported by openwrt and has a usb port as a client.
I've done it with some success using a TL-WR703N (which is a very, very basic router) with a very cheap CM108 usb sound card.

That's really a good thing, it offers a way to reuse old devices. This is a nice one for my holiday :)
Thank you very much for your help and your suggestions!

Random play mode works like a charm! :)


Remove init() from discovery/metrics.go

What did you do?
Imported github.com/prometheus/prometheus/model/rulefmt to perform rule validation, in a project that's also using k8s.io/client-go and expects to register its own workqueue metrics.
Instead, because of a dependency chain, github.com/prometheus/prometheus/discovery ends up imported, and because it has an init() function which registers these metrics, it overwrites the metrics I'm using.
What did you expect to see?
Valid e.g. workqueue_depth metrics from my controller.
What did you see instead? Under which circumstances?
No e.g. workqueue_depth metrics from my controller, as they get stomped on by the discovery package.
System information
No response
Prometheus version
No response
Prometheus configuration file
No response
Alertmanager version
No response
Alertmanager configuration file
No response
Logs
No response

Prometheus is not designed for use as a library. Any difficulties you have using it as such are not an issue for Prometheus.
You are welcome to propose a PR which makes things easier for you, but I will close this.


Switching locales with url_for(locale: "en") on devise pages loads wrong devise scope when using multiple devise models.

Situation
A rails application with two locales ('en' and 'nl').
Devise on two models: Client and User.
A link in the application layout that allows you to switch locales on any page.
This link is generated in a helper method like so:
link_to("EN", url_for(locale: "en"))
(and vice versa for link to the NL version of the page).
Switching locales works as expected on every non-devise page in the application.
Problem
On any devise related page (sign up, sign in, forgot password, etc.) it always generates the url to the first devise scope declared in the routes.rb file. In my case devise_for :user.
This means that when I am on the English /en/clients/sign_in page, the locale switch link points to the Dutch /nl/users/sign_in page instead of the Dutch /nl/clients/sign_in page.
When I change the order of the devise_for calls in the routes.rb file, the problem reverses: the locale links on any user related devise page incorrectly brings you to the /clients/... devise pages.
Inspecting devise_mapping on the page shows that it does know of the correct scope.
I post this issue after scouring the web for an hour or so. Does anyone understand what is going on here and possibly have a solution?
Many thanks!

Interesting:
The problem disappears when I override the sessions controller for each Devise model (e.g. `devise_for :clients, controllers: { sessions: "clients/sessions" } with a simple blank clients/sessions_controller.rb that inherits from Devise::SessionsController.
Could it be that somewhere at the time of the call to url_for(), the application does not know of the current resource_name and just uses the default one?

I am facing the same issue. On /users/sign_in changing the locale with a link also changes url to /admins/sign_in, and device_for :admins is defined first.

I'm closing this issue because it has not had recent activity.
If you're still facing this on the latest version, please open a new one with all the information requested in the template.
Thank you!


Error while building android with

Take a look at following error. It occurs when I run react-native run-android.
I have configured everything mentioned in documentation.
My react-native version is RN 0.44
error: constructor OrientationPackage in class OrientationPackage cannot be applied to given types;
          new OrientationPackage()
          ^
  required: Activity
  found: no arguments
  reason: actual and formal argument lists differ in length
1 error

got it worked with #93

@karan101292 Seems like you got it working. Going to close this then!


slacker.Error: invalid_auth

Hey guys,
I want to delete all messages from the #general channel.
I am the owner of my slack workspace.
I installed the slack cleaner using pip install slack-cleaner.
I created an app with bot scopes channels:history, channels:read, chat:writeandusers:read. I copied the bot user oauth token and using it it slack-cleaner --token  --message --channel general --user "*"`.
But getting error
Running slack-cleaner v0.7.4
Traceback (most recent call last):
  File "/usr/local/bin/slack-cleaner", line 5, in <module>
    from slack_cleaner.cli import main
  File "/usr/local/lib/python2.7/site-packages/slack_cleaner/cli.py", line 84, in <module>
    init_user_dict()
  File "/usr/local/lib/python2.7/site-packages/slack_cleaner/cli.py", line 66, in init_user_dict
    res = slack.users.list().body
  File "/usr/local/lib/python2.7/site-packages/slacker/__init__.py", line 331, in list
    return self.get('users.list', params={'presence': int(presence)})
  File "/usr/local/lib/python2.7/site-packages/slacker/__init__.py", line 120, in get
    api, **kwargs
  File "/usr/local/lib/python2.7/site-packages/slacker/__init__.py", line 102, in _request
    raise Error(response.error)
slacker.Error: invalid_auth

Expected behavior
It should work as expected.
Context

Version: Version 0.7.4
Python Version: Python 2.7.17

I had the exact same invalid_auth problem (even when granting all bot and user OAuth permissios to the app at Slack out of desperation) and ended up using https://github.com/sgratzl/slack_cleaner2 to overcome it.

I had the same problem.
But, it seems slack-cleaner2 is working
https://github.com/sgratzl/slack_cleaner2

My commands

$ python --version
Python 3.6.10

$ python
Python 3.6.10 (default, Aug 22 2020, 00:49:22)
[GCC 8.3.0] on linux
Type "help", "copyright", "credits" or "license" for more information.
>>> from slack_cleaner2 import *
>>>
>>> s = SlackCleaner('xoxp-***')
>>>
>>> s.users
this will list all users in workspace
[xxx, xxx...]

>>> [msg for msg in s.c['YOUR CHANNEL NAME'].msgs(with_replies=True)]
this will list all message in a channel

thanks all.  Been getting the same error repeatedly. Will just give 2 a go...

The problem seems to be with the upstream Slacker module, which has been discontinued.
Authorization token has to be sent inside HTTP headers or inside the POST body, not inside the GET query params as Slacker does.
https://api.slack.com/web#slack-web-api__authentication

Patching /usr/local/lib/python3.7/site-packages/slacker/__init__.py with changes from the following fork has saved my day, at least for my temporary use case:
https://github.com/ismith/slacker/commit/8ff928c59ec77b047b7270d96936e3df7e2bc4fb

slack deprecated legacy token.

please migrate to https://github.com/sgratzl/slack_cleaner2 since I don't have the capacity to fix the underlying/dependent slacker library


add clear log button

fixes #348

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

0 of 5 (0.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.2%) to 62.872%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

pkg/sync/http.go
0
5
0.0%

Totals

Change from base Build<PHONE_NUMBER>:
-0.2%

Covered Lines:
1226

Relevant Lines:
1950

💛  - Coveralls


Remove Binding

Hi,
I am using bidirection binding between strings and  textFields. Theses bindings are done in a subview. I want to remove these bindings once the subview is removed:
        TimelineViewModel._actStr.bidirectionalBindTo(actTextField.bnd_text)
        TimelineViewModel._placeStr.bidirectionalBindTo(placeTextField.bnd_text)
        TimelineViewModel._roomStr.bidirectionalBindTo(roomTextField.bnd_text)

Thanks!!

Couldn't you use dispose() in your willRemoveSubview on the parent view? (Just throwing ideas.)

Binding are automatically disposed (removed) when targets (text fields in your case) are deallocated. If you need to do that sometime earlier, store the disposable:
let disposable = TimelineViewModel._actStr.bidirectionalBindTo(actTextField.bnd_text)

And later when you need it dispose it as @clooth said:
disposable.dispose()

Thanks @clooth and @srdanrasic  for the solution. I am very new to reactive programming and I have started with SwiftBond to learn it properly. It's amazing and I hope to get more support from here.


CQL: Add integration tests for system.local and system.peers

It looks like CI is hitting #365 (not related to these changes, apparently)


[Clang importer] Infer @_downgrade_exhaustivity_check for new-to-2017 enum elements in Swift 3

In Swift 3 mode, infer @_downgrade_exhaustivity_check for any enum
elements that were introduced in the 2017 SDKs (macOS 10.13, iOS 11.0,
tvOS 11.0, watchOS 4.0). This is a stop-gap until we get "open" enums,
but serves an important source-compatibility use case.
Fixes rdar://problem/32824207.

@swift-ci please smoke test

@swift-ci please smoke test and merge

@swift-ci please smoke test and merge


New package: PTYQoL v0.0.1

Registering package: PTYQoL
Repository: https://github.com/putianyi889/PTYQoL.jl
Created by: @putianyi889
Version: v0.0.1
Commit: 6e6b5cf0648ddcaf2dfda2370da0b999a72d87d9
Git reference: HEAD
Description: Julia bad

Please don't register personal packages. See guidelines in the README

Please close this PR for now


Could be good? Add appendBase64()

Ethier's DynamicBuffer has a function called appendSafeBase64() that appends and base64-izes data simultaneously. I don't know assembly but this seems potentially more efficient than one at a time!
Interestingly Solady is given an author credit on the function.

Wow no love for this one? Am I finally falling off?
Use case here is this.
Old way:

Generate big SVG or BMP image
Base64 it
Append it to a JSON string using a DynamicBuffer

End result being a JSON string with the image in B64 format at the end.
"Proposed new way"

Generate big SVG or BMP image
Base64 it while simultaneously appending the B64 data to the buffer!

End result is the same!
In my informal testing it was up to a 10% gas savings for large BMPs.
Mainnet Etherscan has a ~100M read gas limit so if you want the "cred" of an "on-chain" NFT that shows a ton of data for tokenURI there, you need to think about gas!
Thoughts?

I think keep as separate functions for now.
If someone is actually hitting the gas limit, try using LibString.directReturn first.


Add custom exception for some errors

https://github.com/nnstreamer/nntrainer/pull/308#discussion_r454772057

Declare custom errors if needed (to nntrainer_error.h)
Handle it at least at capi exception boundary

:octocat: cibot: Thank you for posting issue #315. The person in charge will reply soon.


Role path incorrect when using requirements_file on 1.9 molecule

The role path is not set correctly when using a requirements_file in ansible.cfg.  Ansible is unable to find a path to the role.
ERROR! the role 'yatesr.timezone' was not found in /Users/jodewey/git/molecule/test/functional/driver/docker/scenarios/command_test/roles:/Users/jodewey/git/molecule/test/functional/driver/docker/scenarios/command_test:/Users/jodewey/git/molecule/test/functional/driver/roles:/Users/jodewey/git/molecule/test/functional/roles

The error appears to have been in '/Users/jodewey/git/molecule/test/functional/driver/docker/scenarios/command_test/playbook.yml': line 4, column 7, but may
be elsewhere in the file depending on the exact syntax problem.

The offending line appears to be:

  roles:
    - role: yatesr.timezone
      ^ here

  syntax: commands succeeded
  py27-ansible19-unit: commands succeeded
  py27-ansible19-functional: commands succeeded
  py27-ansible20-unit: commands succeeded
  py27-ansible20-functional: commands succeeded
  py27-ansible21-unit: commands succeeded
  py27-ansible21-functional: commands succeeded
  doc: commands succeeded
  congratulations :)


fix(host): optimize storage register failed with dup name

What this PR does / why we need it:

Does this PR need to be backport to the previous release branch?:
release/3.11

/lgtm


Update README file with configuring MongoDB for local development

Issue

At present, the README does not contain any detailed documentation on how we can configure MongoDB for local development

What can be done

Update the README with documentation setting up MongoDB for local development

@rafat17 good observation.

readme updated.


[Question] Tenancy on Single Domain?

Just discovered Tenancy and I think it will solve an issue in an upcoming project.
Can I use Tenancy v5.6 (with Laravel 7) without utilizing hostnames and subdomains? I would use just the website model and my own identification code but does anyone see any issues long term? Just trying to get input before I dive deep into this project and find out it won't be possible to do "that one critical thing my app needs and now I have to start over since I can't use Tenancy" situtation.
For my test, I used explicit route model binding like below and that seemed to work.
// app/Providers/RouteServiceProvider.php

Route::bind('tenant', function ($value) {
    $environment = app(\Hyn\Tenancy\Environment::class);
    $website = \Hyn\Tenancy\Models\Website::whereUuid($value)->firstOrFail();
    $environment->tenant($website);
    //ddd(app(\Hyn\Tenancy\Environment::class)->tenant());
    return $environment->tenant();
});

My URLs would look like this: https://laravel7.test/app/{tenant}/blah => https://laravel7.test/app/418b8053-99aa-4064-a557-6d884ea3e213/blah
Super simple middleware saves me from having to add the tenant parameter to each route() call in my blade files:
public function handle($request, Closure $next)
{
    URL::defaults(['tenant' => $request->tenant]);

    return $next($request);
}

I mostly need database table separation for my project (storing audit log data from third-party sources) and Tenancy provides that functionality but I don't need separate storage, domains, etc. I am using OAuth instead of the primary Laravel Auth scaffold (this app is a consumer, I own the IDP app which is also in Laravel) so subdomain authentication would be a pain with redirect URLs. Based on #527 I think my answer would be yes? I will be utilizing background jobs so would TenantAwareJob still function properly? (I'll admit, I don't yet know how that trait works so it may be a mute point for my situation)
Thoughts? Thanks!

Thoughts?

Yeah, if it works for you, it can't be wrong.
Your approach seems reasonable to me.


select function not trigger observe

The selection function cannot trigger traitlets observe.
I am using select(), trying to realize control table both with manual and with function.
The observe function of table is only triggered when click, but not after select() function.
Do you know how to trigger it?
Example:
def func(change):
...
datagrid.observe(func, name=['selections'])
datagrid.select(row1=rows,column1=1,row2=rows, column2=1)
*func should be run based on new selections, but not.

I am also interested in resolving this. When onclick function is triggered, the selection returned in the field .selected_cell_values shows previous selection. I found no way so far to track current selection once it changed.

I'm facing the same issue. A solve or workaround would be great.


more flexible generate_attention_heatmaps()

Feature
It is confusing when using generate_attention_heatmaps() as it requires a list of feature files but takes individual attention arrays for the function to work. Is it possible to make it taking list of files for both 'bags=' and 'attention=' parameters?
Pitch
Now the function is very confusing and takes multiple steps before you can run it. It'd be more useful if it's simplified.
Alternatives

Additional context

after reading into the code, I just did this:
slide_path = 'xxx.svs'
locations = np.load('path_to_the_index_file_of_the_feature.index.npz')['arr_0']
attention = np.load('path_to_the_attention_score_numpy_array_file._att.npz')['arr_0']
sf.util.location_heatmap(
    locations=locations,
    values=attention,
    slide=slide_path,
    tile_px=224,
    tile_um=112,
    filename='xxx.png'
)

I believe it'll be much easier if you use slideflow-studio on a gpu instance with a desktop, which I don't have.
Hope this helps.


Copy number axis are not being displayed properly

Describe the bug
https://genome.med.nyu.edu/external/imielinskilab/mskiweb/xanthc01/case-report/?dataset=clinical-samples&tab=2&report=1160696&location=1:1-M:16571

To Reproduce
Open pan-genome view, then switch to another tab or patient. Once you get past #5 the left axis are all missing for the plots that do load and the right axis overlaps with the plot making it difficult to read.
Expected behavior
Bot axis should be visible and non-overlapping.
Additional context
Most likely related to #5 since the axis will appear on the initial load but disappears after changing patient or tab state.

@shihabdider , I have added a warning notification on the scatterplots header to indicate when the slope and intercept attributes are missing from the metadata.json file, which may cause the left hand axis not rendering


Ss/hdfs touchup

retest this please

retest this please

retest this please

retest this please

retest this please

retest this please

retest this please

retest this please

retest this please


Rocm using kernel 4.18.11 and Blender3d Fury X and Firepro w8100

After installing Rocm 1.9.0 with kernel 4.18.11, I noticed that out of the 2 cards I only see my Fury X. Furthermore upon entering blender I don't see any opencl cards unlike amdgpu-pro. I'm trying to figure out if Rocm 1.9 currently work with blender 2.79, and does Rocm even support Firepro w8100? Or is the kernel version too new? I don't see great documentation on what actually has been tested officially.

Documentation for supported and enabled hardware is available here. We have code that enables FirePro W8100 (Hawaii) GPUs, but we do not guarantee support for these devices.
Note that to enable Hawaii (Sea Islands) support, you may need to pass the kernel boot parameter amdgpu.cik_support=1. cik_support defaults to 1 in our custom ROCK kernel drivers, but depending on how your drivers were built in your 4.18.11 kernel (I don't know what distribution you're using, nor what their kernel compile-time parameters are), Sea Islands support may be disabled. You may also need to blacklist the older radeon driver to ensure it doesn't also load and try to take control of your Hawaii GPU.
echo 'blacklist radeon' | sudo tee /etc/modprobe.d/blacklist-radeon.conf
Could you show me the output of the following commands?

dmesg | grep amd
dmesg | grep radeon
dmesg | grep kfd
lsmod
cat /sys/module/amdgpu/parameters/cik_support

Blender questions have been raised in other issues, so I would recommend following those for your other question.

Am using Kubuntu 18.04 sorry. I switched back to pro, since I was able to get amdgpu-pro installed and working again after figuring out what kernel version it needed to be installed under.
xorg.log doesn't list Firepro w8100 so it is turned off. I didn't know it was a matter of turning off and on, thanks. When you guys get blender working in cycles I will be back.

So I recently had problems trying to  upgrade the amdgpu-pro drivers from 18.20 to 18.30 where it couldn't install correctly on kernel 4.15.0-39, so I gave rocm another chance. It works.
I was following some set of instructions to install on kernel 4.18, where I didn't install rocm-dkms or rock-dkms which somewhat worked as they displayed opencl properties but did not display in blender. I encountered the same problem when I went to install opencl only with rock-dkms. The rocm-dkms gave me an error when installing that kernel 4.18.20 was not a valid kernel or something but after restarting it registered in clinfo and shows up in blender and is able to render.

Hawaii is no more officially supported with ROCm, please check: https://github.com/RadeonOpenCompute/ROCm#hardware-and-software-support
Thank you.

Hawaii is no more officially supported with ROCm, please check: https://github.com/RadeonOpenCompute/ROCm#hardware-and-software-support
Thank you.


Evil Picture

Summary of the new feature / enhancement
Did you use an evil-looking character because all administrators are evil?

Proposed technical implementation details (optional)
No response

I believe the character's name is "Nanoman".  I'm guessing the main goals were to make a character that was "edgy" and gender ambiguous (which I appreciate for inclusivity).  I do agree that the artwork does look a bit "evil" and probably could use a refresh or just be eliminated altogether. After all, PowerShell gets a very bad reputation as being some sort of loner hacking tool to the point where organizations allow any user-downloaded executables and macros to run but PowerShell is locked down hard because it has the perception of being so much more dangerous.

This isn't really an issue with PowerShell itself so I am converting this to a discussion


add bydhvs adapter to official repository

discussion see: https://forum.iobroker.net/topic/43931/neuer-adapter-byd-hvs/51

RE-CHECK!

Automated adapter checker
No changed adapters found
Add comment "RE-CHECK!" to start check anew

@christianh17 The adapter is not in the latest repo (the other file) ... please add there too ... normally it goes into latest first ...

RE-CHECK!

Automated adapter checker
No changed adapters found
Add comment "RE-CHECK!" to start check anew

I just did some small bug fixes, did a new version and added this one to the latest-repository. The RE-CHECK-Posting was deleted automatically :-(. Sorry, I am new with this kind of stuff, can you please give me some advice what to do next? Thanks.

All good, I will do review later todaay

RE-CHECK!

Automated adapter checker
No changed adapters found
Add comment "RE-CHECK!" to start check anew

Hi,
some review comments:

Do you have a callback after socket close in https://github.com/christianh17/ioBroker.bydhvs/blob/master/main.js#L178 ? Then best use this to call unload callback
please add roles for all objects https://github.com/christianh17/ioBroker.bydhvs/blob/master/main.js#L199
this timout in https://github.com/christianh17/ioBroker.bydhvs/blob/19b8a7d75c852fbdc9279b3d5d64124d8e8a9f75/main.js#L611 is not enable in unload (rare case but could be)
Please also subscribe to the error event of the IPClient, else adapter might crash in error cases
What happens on disconnect? I do not see any reconnecton mechanism?
Thank you for checking and fixing the issues

Hi!
Thanks for the review. I tried to resolve all points:
Do you have a callback after socket close in https://github.com/christianh17/ioBroker.bydhvs/blob/master/main.js#L178 ? Then best use this to call unload callback
My idea was different: I open the connection to read the data and immediately close it afterwards. So I close the socket at the unload event. Reason: The BYD battery allows only one connection at a time.
please add roles for all objects https://github.com/christianh17/ioBroker.bydhvs/blob/master/main.js#L199
fixed
this timout in https://github.com/christianh17/ioBroker.bydhvs/blob/19b8a7d75c852fbdc9279b3d5d64124d8e8a9f75/main.js#L611 is not enable in unload (rare case but could be)
fixed
Please also subscribe to the error event of the IPClient, else adapter might crash in error cases
fixed
What happens on disconnect? I do not see any reconnecton mechanism?
As mentiononed above. If the connections drops - the socket is closed and there will be the next try for the next pull.
For now,  i did a new prepatch-version, I am now at 1.2.2-0 . I did not change the PR, at first I would like to get an O0 for the adapter from you, afterwards I will change the PR.
Kind regards
Christian

RE-CHECK!

Automated adapter checker
No changed adapters found
Add comment "RE-CHECK!" to start check anew

RE-CHECK!

Automated adapter checker
No changed adapters found
Add comment "RE-CHECK!" to start check anew

Latest shouldn’t have a version attribute.

@foxriver76: Sorry, I do not understand what to do. All other adapters do have a version like:
{
"admin": {
"meta": "https://raw.githubusercontent.com/ioBroker/ioBroker.admin/master/io-package.json",
"icon": "https://raw.githubusercontent.com/ioBroker/ioBroker.admin/master/admin/admin.png",
"type": "general",
"version": "4.2.1"
},
Shall I really delete the line with the version-number in my PR?

For sources-dist-stable this is correct, because the version is hard defined by the developer. For sources-dist (Beta) the latest npm version is automatically taken.


issue while login user.

i am using xrdp0.9.5 version in ubuntu server.
i have created 5 users named user1,user2,user3,user4,user5.
i have face the issue of user login some times.
for e.g. suddenly user4 is not able to login. at the same times other users can login properly.
then i deleted that user4 and again create new user with the same name but issue remains same.
if i create another user named user44 then this user able to login.
so i checked logs of xrdp.log and xrdp-sesman.log at the time of user4 login.
i have removed all socket files related to that user which is located at /tmp/.xrdp/
but when i login user4 user it will only show reconnected in xrdp-sesman.log file .
below are the logs,

xrdp.log
[20180328-16:27:42] [INFO ] An established connection closed to endpoint: NULL:NULL - socket: 7
[20180328-16:27:42] [CORE ] WARNING: Invalid x.509 certificate path defined, default path will be used: /etc/xrdp/cert.pem
[20180328-16:27:42] [CORE ] WARNING: Invalid x.509 certificate path defined, default path will be used: /etc/xrdp/key.pem
[20180328-16:27:43] [DEBUG] xrdp_00003ee9_wm_login_mode_event_00000001
[20180328-16:27:43] [WARN ] local keymap file for 0x4009 found and dosen't match built in keymap, using local keymap file
[20180328-16:27:50] [DEBUG] returnvalue from xrdp_mm_connect 0
[20180328-16:27:50] [INFO ] An established connection closed to endpoint: NULL:NULL - socket: 12
[20180328-16:27:51] [INFO ] An established connection closed to endpoint: NULL:NULL - socket: 12
[20180328-16:27:51] [INFO ] An established connection closed to endpoint: NULL:NULL - socket: 12
[20180328-16:27:52] [INFO ] An established connection closed to endpoint: NULL:NULL - socket: 12
[20180328-16:27:52] [INFO ] An established connection closed to endpoint: NULL:NULL - socket: 12
`

xrdp-sesman.log

[20180328-16:30:07] [INFO ] A connection received from: <IP_ADDRESS> port 36570
[20180328-16:30:07] [INFO ] scp thread on sck 8 started successfully
[20180328-16:30:08] [INFO ] ++ reconnected session: username user4, display :18.0, session_pid 25674, ip <IP_ADDRESS>:38661 - socket: 8
[20180328-16:30:08] [INFO ] An established connection closed to endpoint: <IP_ADDRESS>:36570 - socket: 8

Do you have Xorg process with display number :18.0? Do ps uaxww and see.
If not, somehow sesman is corrupted state. You need to restart sesman.
Restarting sesman will lose other user's session, however, there's no other way right now.


⚠ Bofrost.be has degraded performance

In 52a91a1, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 1463 ms

Resolved: Bofrost.be performance has improved in bf82ac8 after 6 minutes.


⚠ Bofrost.de has degraded performance

In 6bb0c75, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1316 ms

Resolved: Bofrost.de performance has improved in aef8041.


No speed up when using patch_sklearn with PyCaret

Hi,
We are planning to integrate scikit-learn-intelex project with PyCaret
The issue is as following:
from pycaret.datasets import get_data
data = get_data('poker')

from pycaret.classification import *
s = setup(data, target = 'CLASS', session_id = 123)

from sklearnex import patch_sklearn
patch_sklearn()

%%time
knn = create_model('knn')

This took 10 minute despite of patch_sklearn command.
However when I explicitly import the model it gives great results from acceleration:
from pycaret.datasets import get_data
data = get_data('poker')

from pycaret.classification import *
s = setup(data, target = 'CLASS', session_id = 123)

from sklearnex import patch_sklearn
patch_sklearn()

from sklearn.neighbors import KNeighborsClassifier
knn_intel = KNeighborsClassifier()

%%time
knn_intel = create_model(knn_intel)

Expected Action:
What can we do to make the first attempt gives result of acceleration so that users won't have to import estimator explicitly. I am thinking we can add a parameter in the setup function called use_intel_acceleration. When that is set to True by user we should run patch_sklearn command in our code base so that users won't have to do any thing explicitly outside of PyCaret code base.
The file which creates model container in PyCaret repo are located here.

Hi @moezali1
Thanks for reporting the issue.
The problem is that patching must be applied before calling sklearn. patching replaces some methods from sklearn with their optimizions from the intel extension for scikit-learn.
In your case, you are importing pycaret first, where sklearn is already imported.
However, I found that if you call the patch before all other code, your code that you have attached will not work. This is not expected behavior.
The reason for this is a bug on our side, which is related to train_test_split. I already created an issue
I already have a fix and I will create a PR with it in the near future. After the fix everything seems to work as expected:

We also think that the patching option may not be the easiest for integration and perhaps the best option would be to import the required methods directly from sklearnex. The autogluon team integrated sklearnex in a similar way.

@moezali1
Also like an option you can use patching for some algorithms.
For example this line will work for your code:
from sklearnex import patch_sklearn
patch_sklearn("knn_classifier")

from pycaret.datasets import get_data
data = get_data('poker')

from pycaret.classification import *
s = setup(data, target = 'CLASS', session_id = 126)

To get map of algorithms:
import daal4py as d4p
d4p.sklearn.monkeypatch.dispatcher._get_map_of_algorithms().keys()

>>
dict_keys(['pca', 'kmeans', 'dbscan', 'distances', 'linear', 'ridge', 'elasticnet', 'lasso', 'svm', 'logistic', 'log_reg', 'knn_classifier', 'nearest_neighbors', 'knn_regressor', 'random_forest_classifier', 'random_forest_regressor', 'train_test_split', 'fin_check', 'roc_auc_score', 'tsne', 'svc', 'logisticregression', 'kneighborsclassifier', 'nearestneighbors', 'kneighborsregressor', 'randomrorestclassifier', 'randomforestregressor'])

I can also help you create a PR for integration sklearnex into pycaret.

@PivovarA Thanks for your detailed response. I look forward to receiving your PR on pycaret. Excited for this integration.

@PivovarA We are releasing 3.0-rc in 2 weeks. Do you think its possible to send a PR our way on develop branch before May 15th?
Thanks.


wrap url parsing exceptions from urllib3's PoolManager

This should address #4746 by catching and wrapping all LocationValueErrors and rewrapping them under the Requests namespace as InvalidURL.

Codecov Report

Merging #4765 into master will increase coverage by 0.02%.
The diff coverage is 80%.

@@            Coverage Diff             @@
##           master    #4765      +/-   ##
==========================================
+ Coverage   66.79%   66.81%   +0.02%
==========================================
  Files          15       15
  Lines        1563     1567       +4
==========================================
+ Hits         1044     1047       +3
- Misses        519      520       +1

Impacted Files
Coverage Δ

requests/adapters.py
70.27% <80%> (+0.08%)
:arrow_up:

Continue to review full report at Codecov.

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


Bugfix with undefined children_hooks when package is None

Fix the code path in touch_import_top() where package is None (e.g. import six instead of from six import string_types). Fixed a minor bug in this code path, so that a custom fixer can use it.
(touch_import_top() is always called with a package within the futurize codebase.)

Thanks @azjps !


[Story] Operator should handle target namespace deletion

I guess for this to work the Operator would also need to be watching for any namespaces to be created/deleted in the cluster, so that it can reconcile by deleting the original secrets if namespaces are deleted, or recreating/re-copying them if a matching namespace is recreated too?

I suppose other objects that are placed in target namespaces would suffer from the same issue (e.g. role bindings, CA secrets). This might be a bit tricky to handle. Since we can't have cross-namespace owner references, maybe we could do something like this:

Apply a label to all cross-namespace objects pointing to the namespace/name of the CR
Add a custom controller watch on cross-namepsace object types (e.g. secret, rolebinding) that checks for the label(s)
If the label exists, enqueue a reconcile request for that CR's namespace/name
Controller recreates any missing objects

As for deleting the certificate object from the install namespace for deleted namespaces, we could issue a get request for the namespace and, if it doesn't exist, delete the certificate and secret. I think the namespace deletion event should be captured with the above custom controller watch.
Originally posted by @ebaron in https://github.com/cryostatio/cryostat-operator/issues/938#issuecomment-2339128336

@andrewazores I suppose with the above implementation, that CR would fail to reconcile until the namespace is recreated. The operator would keep trying to create the objects in a non-existent namespace. Perhaps that's okay, the CR does specify an invalid spec if the target namespace doesn't exist (currently).

Perhaps that's okay, the CR does specify an invalid spec if the target namespace doesn't exist (currently).

I think that makes enough sense - we just have to make sure we document this. But I agree that listing target namespaces that don't exist should be considered an invalid spec, and it's probably best that this results in some kind of a failure rather than a silent acceptance. In particular, if we are doing things like copying secrets into the target namespaces, then silently failing if the namespace doesn't exist, only to then perform the copy at a later date if it does get created, seems like it might be surprising behaviour and potentially a security issue - the user might have typo'd a namespace name by accident, not noticed because the deployment succeeded, and then their secret gets inadvertently created and shared into a different namespace later on which they did not intend.
We have talked about this kind of problem before, where the Operator should be proactive about checking RBAC for each of the target namespaces - is there a particular Issue we can link this to?

We have talked about this kind of problem before, where the Operator should be proactive about checking RBAC for each of the target namespaces - is there a particular Issue we can link this to?

Do you mean this sort of multi-tenancy issue: https://github.com/cryostatio/cryostat-operator/issues/579?

Yes that looks like it's probably what I'm thinking of.


My scrypy isn't duplicating my audio.

Hi there, I am a noob so please help me.
Environment

OS: Windows 11 Home
Scrcpy version: 2.6.1
Installation method: Windows release
Device model: Samsung A33
Android version: 14

The bug
I was trying to duplicate the audio from my phone, but it isn't forwarding the audio with the recommended command.
C:\Users\mshal\Desktop\Warned\scrcpy-win64-v2.6.1>scrcpy --max-fps=90 --video-codec=h265 --audio-dup --video-bit-rate=110M
scrcpy 2.6.1 <https://github.com/Genymobile/scrcpy>
INFO: Audio duplication enabled: audio source switched to "playback"
INFO: ADB device found:
INFO:     -->   (usb)  RZCT6088JVT                     device  SM_A336E
C:\Users\mshal\Desktop\Warned\scrcpy...d. 84.2 MB/s (71112 bytes in 0.001s)
[server] INFO: Device: [samsung] samsung SM-A336E (Android 14)
INFO: Renderer: direct3d
INFO: Texture: 1080x2400

I have found that it still forwards the audio with:
C:\Users\mshal\Desktop\Warned\scrcpy-win64-v2.6.1>scrcpy --max-fps=90 --video-codec=h265 --video-bit-rate=110M
scrcpy 2.6.1 <https://github.com/Genymobile/scrcpy>
INFO: ADB device found:
INFO:     -->   (usb)  RZCT6088JVT                     device  SM_A336E
C:\Users\mshal\Desktop\Warned\scrcpy.... 110.5 MB/s (71112 bytes in 0.001s)
[server] INFO: Device: [samsung] samsung SM-A336E (Android 14)
INFO: Renderer: direct3d
INFO: Texture: 1080x2400

I was trying to duplicate the audio from my phone, but it isn't forwarding the audio with the recommended command.

What does it do instead? Just playing it on the device?
With which app? In this mode, as explained in the documentation:

apps can opt-out (so they are not captured)

Call of duty: mobile and Spotify

Spotify and Instagram. They were working fine two days ago now. Now only IG works after I restarted my phone after my post.

It just plays on the device.

What does it do instead? Just playing it on the device?

Spotify and Instagram. They were working fine two days ago now. Now only IG works after I restarted my phone after my post.

What does it do instead? Just playing it on the device?

It plays on my device but not in the laptop while mirroring

have you tried simply scrcpy -m720?

CODM, Spotify and Instagram.

For spotify at least, it is expected that it is not mirrored wkth --audio-dup (--audio-source=playback), because with this capture mode apps can opt-out.
It is forwarded with the default audio output capture mode, i.e. when you don't pass --audio-dup (so it will play only on your computer).

have you tried simply scrcpy -m720?

How do I do that?

CODM, Spotify and Instagram.

For spotify at least, it is expected that it is not mirrored wkth --audio-dup (--audio-source=playback), because with this capture mode apps can opt-out.
It is forwarded with the default audio output capture mode, i.e. when you don't pass --audio-dup (so it will play only on your computer).

Thanks for the insight


Error: in i2p lib

hacker@localhost  ~  python3 my_connecter.py
Traceback (most recent call last):
File "my_connecter.py", line 2, in
from i2p import socket
File "", line 991, in _find_and_load
File "", line 975, in _find_and_load_unlocked
File "", line 655, in _load_unlocked
File "", line 618, in _load_backward_compatible
File "", line 259, in load_module
File "/usr/local/lib/python3.8/site-packages/i2p-0.1.0-py3.8.egg/i2p/socket/init.py", line 84, in
File "", line 991, in _find_and_load
File "", line 975, in _find_and_load_unlocked
File "", line 655, in _load_unlocked
File "", line 618, in _load_backward_compatible
File "", line 259, in load_module
File "/usr/local/lib/python3.8/site-packages/i2p-0.1.0-py3.8.egg/i2p/socket/sam/init.py", line 5, in
File "", line 991, in _find_and_load
File "", line 975, in _find_and_load_unlocked
File "", line 655, in _load_unlocked
File "", line 618, in _load_backward_compatible
File "", line 259, in load_module
File "/usr/local/lib/python3.8/site-packages/i2p-0.1.0-py3.8.egg/i2p/socket/sam/simple.py", line 8, in
File "", line 991, in _find_and_load
File "", line 975, in _find_and_load_unlocked
File "", line 655, in _load_unlocked
File "", line 618, in _load_backward_compatible
File "", line 259, in load_module
File "/usr/local/lib/python3.8/site-packages/i2p-0.1.0-py3.8.egg/i2p/datatypes.py", line 11, in
File "", line 991, in _find_and_load
File "", line 975, in _find_and_load_unlocked
File "", line 655, in _load_unlocked
File "", line 618, in _load_backward_compatible
File "", line 259, in load_module
File "/usr/local/lib/python3.8/site-packages/i2p-0.1.0-py3.8.egg/i2p/crypto.py", line 10, in
File "/usr/local/lib/python3.8/site-packages/libnacl-1.7.1-py3.8.egg/libnacl/init.py", line 85, in
nacl = _get_nacl()
File "/usr/local/lib/python3.8/site-packages/libnacl-1.7.1-py3.8.egg/libnacl/init.py", line 81, in _get_nacl
raise OSError(msg)
OSError: Could not locate nacl lib, searched for libsodium.so, libsodium.so.23, libsodium.so.18, libsodium.so.17, libsodium.so.13, libsodium.so.10, libsodium.so.5, libsodium.so.4,
I think i need to install some packages, but i don't know what packages need to install

Decision: install nacl and libsodium


Column value validation fails with ODBC (Postgresql) table backend

Prework

[x] Read and agree to the code of conduct and contributing guidelines.
[x] If there is already a relevant issue, whether open or closed, comment on the existing thread instead of posting a new issue.
[ ] Post a minimal reproducible example so the maintainer can troubleshoot the problems you identify. A reproducible example is:

[ ] Runnable: post enough R code and data so any onlooker can create the error on their own computer.
[ ] Minimal: reduce runtime wherever possible and remove complicated details that are irrelevant to the issue at hand.
[ ] Readable: format your code according to the tidyverse style guide.

Description
When using an ODBC (Postgres) backend for validations, using the col_vals_lte function with a date value errors:
ERROR: operator does not exist: date <= numeric;

It appears the value is being sent to the backend as a number.
This happens regardless of how the value is constructed, e.g. as.Date(), as.Posixlt(), lubridate::today() etc.
Reproducible example
I'm sorry, I don't know how to create a reprex for a database backend. I would love to know how though!
Expected result
A valid check of whether values in column are of a date less than or equal to today.

Per usage in this article, I found that things work if you convert the date to a character. This is a great workaround, but it would be nice to be able to pass something like lubridate::today() directly.
I might could help implement this with a couple of pointers on where to begin!

Closing this because it would be fixed  by a database driver like {odbc} or {RPostgres}, not {pointblank}

@brancengregory hi! can you try with the development version of RPostgres?

@brancengregory hi! can you try with the development version of RPostgres?

RPostgres handles this case well, on release and development. My comment was misleading, sorry. I only meant that the issue would be in whatever driver was used, and in my case that was odbc, so if there is still an issue it would be there. I haven't checked the status in a couple of months since I've already moved away from odbc this year. I'll definitely report back if I do, though!


No Package Published Yet

From today verdaccio (v3.8.3) stopped showing packages in local registry.
Opening on browser this message is shown: No Package Published Yet
The storage folder specified in configuration (~/.local/share/verdaccio/storage) contains all my packages.
But database file is empty ~/.local/share/verdaccio/storage/.sinopia-db.json.
Indeed running verdaccio from terminal I get this:
 warn --- config file  - /home/bssadmin/.config/verdaccio/config.yaml
 error--- Package database file corrupted (invalid JSON), please check the error printed below.
File Path: /home/bssadmin/.local/share/verdaccio/storage/.sinopia-db.json SyntaxError: Unexpected end of JSON input
    at JSON.parse (<anonymous>)
    at LocalDatabase._parseDatabase (/home/bssadmin/.npm-packages/lib/node_modules/verdaccio/node_modules/@verdaccio/local-storage/lib/local-database.js:298:19)
    at LocalDatabase._fetchLocalPackages (/home/bssadmin/.npm-packages/lib/node_modules/verdaccio/node_modules/@verdaccio/local-storage/lib/local-database.js:272:23)
    at new LocalDatabase (/home/bssadmin/.npm-packages/lib/node_modules/verdaccio/node_modules/@verdaccio/local-storage/lib/local-database.js:49:22)
    at LocalStorage._loadStorage (/home/bssadmin/.npm-packages/lib/node_modules/verdaccio/build/lib/local-storage.js:809:14)
    at new LocalStorage (/home/bssadmin/.npm-packages/lib/node_modules/verdaccio/build/lib/local-storage.js:54:27)
    at Storage.init (/home/bssadmin/.npm-packages/lib/node_modules/verdaccio/build/lib/storage.js:64:25)
    at /home/bssadmin/.npm-packages/lib/node_modules/verdaccio/build/api/index.js:145:19
    at Generator.next (<anonymous>)
    at step (/home/bssadmin/.npm-packages/lib/node_modules/verdaccio/build/api/index.js:53:191)
 error--- Database is locked, please check error message printed during startup to prevent data loss.
(node:9242) ExperimentalWarning: The fs.promises API is experimental
 error--- Database is locked, please check error message printed during startup to prevent data loss.
 warn --- Plugin successfully loaded: htpasswd
 warn --- Plugin successfully loaded: audit
 warn --- http address - http://<IP_ADDRESS>:4873/ - verdaccio/3.8.3

The configuration is almost untouched
#
# This is the default config file. It allows all users to do anything,
# so don't use it on production systems.
#
# Look here for more config file examples:
# https://github.com/verdaccio/verdaccio/tree/master/conf
#

# path to a directory with all packages
storage: /home/bssadmin/.local/share/verdaccio/storage
# path to a directory with plugins to include
plugins: ./plugins

web:
  # WebUI is enabled as default, if you want disable it, just uncomment this line
  #enable: false
  title: Verdaccio

auth:
  htpasswd:
    file: ./htpasswd
    # Maximum amount of users allowed to register, defaults to "+inf".
    # You can set this to -1 to disable registration.
    #max_users: 1000

# a list of other known repositories we can talk to
uplinks:
  npmjs:
    url: https://registry.npmjs.org/

packages:
  '@*/*':
    # scoped packages
    access: $all
    publish: $authenticated
    proxy: npmjs

  '**':
    # allow all users (including non-authenticated users) to read and
    # publish all packages
    #
    # you can specify usernames/groupnames (depending on your auth plugin)
    # and three keywords: "$all", "$anonymous", "$authenticated"
    access: $all

    # allow all known users to publish packages
    # (anyone can register by default, remember?)
    publish: $authenticated

    # if package is not available locally, proxy requests to 'npmjs' registry
    proxy: npmjs

# To use `npm audit` uncomment the following section
middlewares:
  audit:
    enabled: true

# log settings
logs:
  - {type: stdout, format: pretty, level: http}
  #- {type: file, path: verdaccio.log, level: info}

listen:
  - http://<IP_ADDRESS>:4873

What's happened? How can I resolve?

Your database seems corrupted. The DB is a JSON file, so won't be hard to validate the content. If the is a *.lock file aside of the database, remove it.

In the same folder (/home/bssadmin/.local/share/verdaccio/storage) I have no .lock file.
I've no doubt about the validity of .sinopia-db.json. It's empty so clearly it's not a valid JSON file.

So, I guess that solves your issue, right?

Absolutely not. Verdaccio tells me that I've not package. In storage folder all my packages are present.
Running verdaccio I can see only an error related to .sinopia-db.json corrupted. It's empty and remains empty, also gave 777 permissions nothing change.
I don't know if this is the reason why I've no packages for Verdaccio. And I don't know how to resolve.

I've no doubt about the validity of .sinopia-db.json. It's empty so clearly it's not a valid JSON file.

Ok, remove the database, restart server. Verdacio will create a new db for you. If the data were lost I'm afraid you will need to add the packages names by your own.

Ok no db has been recreated with
{"list":[],"secret":xxx}
To add names, I've just to republish my packages. Isn't it?

To add names, I've just to republish my packages. Isn't it?

Yes, that would take some time. I'd suggest do it manually. eg:
{"list":[
"pkg-1",
"pkg-2",
"pkg-3"
],"secret":xxx}

worked perfectly, thanks

Cool :) I close it then.


Emphasis on  decorated  git log instead if showbranch

The out put of
git log --graph --decorate --oneline --abbrev-commit
is easier to comprehend than showbranch
So define an alias for this and use it more.
E.g.
git config --global alias.graph "log --graph --decorate --oneline --abbrev-commit"

Yeah, let's get rid of the showbranch, it is not intuitive at all.


TS features for setFeatures in geojson-vt example

Describe the bug
Property 'setFeatures' does not exist on type 'Tile'.ts(2339)

To Reproduce
Steps to reproduce the behavior:

Go to geojson-vt example

Fork and change main.js to main.ts
See error

Expected behavior
setFeatures to exist

The examples are in js and generally lack ts typecasting.  tileLoadFunction is common to all UrlTile sources and therefore specifies the base ol/Tile class which does not have methods specific to its subclasses.  For a VectorTile source the ol/VectorTile subclass is used and would need a typecast in ts.  The same would apply to numerous examples which use methods specific to Circle, Polygon or LineString geometries.

Then this would be an enhancement ticket where LoadFunction takes a generic so ol/VectorTile can improve its type?


Support of Python 3.7

Hi!
is a release planned in the future which supports Python 3.7?
Thanks,
Steffen

Maybe anyone is interested to fork this repo and continue updating it on pypi?

This seems to install via pip and run without issue for me on python 3.7.6 on Ubuntu.


dolphinEmuMaster: update to the latest upstream version

Motivation for this change
dolphinEmuMaster failed to compile. Updating to the latest version made it work.
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nix-review --run "nix-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Question:
What does the "9976" in version mean and by what should I replace it ?
Notify maintainers
cc @MP2E

LGTM, though make sure to change the version number too. I went ahead and did that and merged it in as of
f2068d3


fixed modules not beeing added to vsproj

fixes #36384

Thanks for the workaround! This fix does the job, but it seems it causes each module to link against all previous module sources when vsproj is enabled. I ended up with a bunch of 500MB lib files after a full build.
Maybe we could use 2 separate variables modules_sources for all modules and module_sources for the current one to avoid this problem?

Okay then it was a bug, that my hard drive was full. I have an idea how to fix it, but don't know how fast I can implement it.

Okay @pouleyKetchoupp could you retry it? It shouldn't longer pollute the module libs with all modules sources

Maybe we could use 2 separate variables modules_sources for all modules and module_sources for the current one to avoid this problem?

Yeah kinda, but this isn't possible because of https://github.com/godotengine/godot/blob/c5f71ecc6a60e5a92a1f711a1368e051d4824b25/modules/SCsub#L31-L33

@HaSa1002 Yes it's working fine, thanks!
@akien-mga Could you please review this fix? It's simple and it would be good to have on master for contributors working with Visual Studio :)

Note to self: not relevant for the 3.2 branch, unless #35982 is cherrypicked (but I don't plan to).

Thanks!


[FR] Tie scatter points and image together on click in visualize embedding setup

Proposal Summary
When visualizing embeddings (https://voxel51.com/docs/fiftyone/tutorials/image_embeddings.html) I want to be able to click on a single scatter point and be taken to the associated image. Or when scrolling images click and on one and be taken to the location in 2d embedding space.
Motivation

What is the use case for this feature?

I work in unsupervised learning of single cell imaging data. GT labels are uncommon. When exploring a scatter plot of embeddings (e.g. https://voxel51.com/docs/fiftyone/tutorials/image_embeddings.html) it's useful to click on a point and view the associated image. Or when scrolling through images it's useful to see where that image is in embedding space.

Why is this use case valuable to support for FiftyOne users in general?

A lot of users probably won't have labels either and need more robust support for exploring the data embeddings

Why is this use case valuable to support for your project(s) or
organization?

This would be very useful for many people in medical imaging where cross expertise between computation / modeling and biology is necessary. Biological-focused collaborators need good tools to investigate results of computational counterparts. Typically in academic settings datasets are unlabelled due to $ constraints.

Why is it currently difficult to achieve this use case? (please be as
specific as possible about why related FiftyOne features and components are
insufficient)

At present the best you can do (I think) is lasso a single point or two. It also does not appear to be possible to go from image in the session to highlight where the image is in the embedding space.
What areas of FiftyOne does this feature affect?

[x ] App: FiftyOne application
[ ] Core: Core fiftyone Python library
[ ] Server: FiftyOne server

Willingness to contribute
The FiftyOne Community encourages new feature contributions. Would you or
another member of your organization be willing to contribute an implementation
of this feature?

[ ] Yes. I can contribute this feature independently.
[x] Yes. I would be willing to contribute this feature with guidance from
the FiftyOne community.
[ ] No. I cannot contribute this feature at this time.

Hi @lstrgar :wave:
When you attach an interactive plot to an App session as shown in that tutorial, the interactivity is bidirectional, it probably just takes one more click than you wish:

In the scatterplot: like you mentioned, you must lasso select points rather than directly clicking on them. But, if you lasso select, the corresponding images will be loaded in the App
In the App: you can select individual images, but you must then click "Show only selected samples" in the menu in order for the corresponding points to be highlighted in the scatterplot


Add a link to Capybara in the documentation

This PR is add a link to Capybara in the documentation.

Before submitting the PR make sure the following are checked:

[x] Feature branch is up-to-date with main (if not - rebase it).
[x] Squashed related commits together.
[-] Added tests.
[x] Updated documentation.
[-] Added an entry to the CHANGELOG.md if the new code introduces user-observable changes.
[x] The build (bundle exec rake) passes (be sure to run this locally, since it may produce updated documentation that you will need to commit).

Thank you!


Different font sizes for write and preview of comments.

I've noticed that SN uses different font sizes for Write and Preview of comments. This is a problem when writing more than a few lines of text. For instance, see the highlighted word in the first picture below and try to find that exact word in the second picture:

I've tested this in Chromium and Firefox, and got the same results.

Nice catch. Yeah it's definitely smaller. Line height is also different.


Decreases time needed to run tests.

Codecov Report

Merging #737 (8aa17e3) into master (05f512d) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #737   +/-   ##
=======================================
  Coverage   81.02%   81.02%
=======================================
  Files          55       55
  Lines        4347     4347
=======================================
  Hits         3522     3522
  Misses        825      825

Continue to review full report at Codecov.

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


increase attempts by 1

People should feel happy... increase allowable attempts by 1

Fix submitted - waiting for test and deply


pipewire: apply patch for missing declarations

Description of changes

Things done
Upstream patch https://gitlab.freedesktop.org/pipewire/pipewire/-/merge_requests/1205
This fixes build of xdg-desktop-portal-wlr after https://github.com/NixOS/nixpkgs/pull/166441

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
22.05 Release Notes (or backporting 21.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[ ] Fits CONTRIBUTING.md.

I'll defer to @jansol