scanpy icon indicating copy to clipboard operation
scanpy copied to clipboard

Fix diffmap plotting

Open Koncopd opened this issue 3 years ago • 12 comments

Fixes https://github.com/scverse/scanpy/issues/2254 https://github.com/scverse/scanpy/issues/2227

Before there was a hard coded offset for the diffmap basis (https://github.com/scverse/scanpy/blob/8736fc3b02a6af3b7214481c2cf78a3f8f5e06bd/scanpy/plotting/_tools/scatterplots.py#L1077) and later it disappeared. This restores the offset for diffmap, and also provides an option in embedding to select basis columns before components, dimensions etc.

Koncopd avatar May 12 '22 14:05 Koncopd

@Koncopd can you get the CI to pass, please? Happy to review then.

Zethson avatar May 17 '22 14:05 Zethson

I am not sure, what happens with CI. I don't think it is because of the changes in this PR.

Koncopd avatar May 17 '22 17:05 Koncopd

/azp run

ivirshup avatar May 23 '22 14:05 ivirshup

Azure Pipelines successfully started running 1 pipeline(s).

azure-pipelines[bot] avatar May 23 '22 14:05 azure-pipelines[bot]

@Koncopd, there was a bugged release of pip. I think this should work now that there has been a bugfix

ivirshup avatar May 23 '22 14:05 ivirshup

Codecov Report

Merging #2255 (9759293) into master (8e5fedb) will decrease coverage by 0.00%. The diff coverage is 75.00%.

@@            Coverage Diff             @@
##           master    #2255      +/-   ##
==========================================
- Coverage   71.82%   71.82%   -0.01%     
==========================================
  Files          98       98              
  Lines       11539    11542       +3     
==========================================
+ Hits         8288     8290       +2     
- Misses       3251     3252       +1     
Impacted Files Coverage Δ
scanpy/plotting/_tools/scatterplots.py 88.09% <75.00%> (-0.16%) :arrow_down:

codecov[bot] avatar May 23 '22 14:05 codecov[bot]

Ok, i will merge and check, thx.

Koncopd avatar May 23 '22 15:05 Koncopd

@giovp Could you check why the visium test fails? I don't think it is related to this PR.

Koncopd avatar Jun 01 '22 14:06 Koncopd

ah it seems a really minor tolerance thingy, try to increase it to 16 should be fine

save_and_compare_images = image_comparer(ROOT, FIGS, tol=15)

giovp avatar Jun 01 '22 16:06 giovp

/azp run

Koncopd avatar Jun 02 '22 10:06 Koncopd

Azure Pipelines successfully started running 1 pipeline(s).

azure-pipelines[bot] avatar Jun 02 '22 10:06 azure-pipelines[bot]

@ivirshup is this the accepted solution for https://github.com/scverse/scanpy/pull/2280?

Just trying to figure out if we can close the PR and fix the bug.

dburkhardt avatar Feb 07 '23 17:02 dburkhardt