Contributors of a specific file

I want to display contributors of a specific file in a repo. How can I do it?

@AndreyAkinshin The current version doesn't support it. You could have a deeper look at the Github API. I had brief look & only found contributors for Repos: https://developer.github.com/v3/repos/collaborators/

http://stackoverflow.com/questions/13430769/getting-github-individual-file-contributors

Nice! Are you interested in working on it & creating a PR?

Unfortunately, right now I have no time.

@AndreyAkinshin too bad, but similar situation for me. I currently focus on other things.


bam_clean issue

Hi!
Using bam files as input with the bam_clean: picard setting in the config file produces an error when bcbio_nextgen is run (see below). Maybe I'm doing something wrong here, or is this a bug?
[genetik@v01s979 work]$ bcbio_nextgen.py -n 30 ../config/config.yaml
[2016-04-04T07:21Z] Resource requests: sambamba, samtools; memory: 2.00, 2.00; cores: 16, 16
[2016-04-04T07:21Z] Configuring 1 jobs to run, using 16 cores each with 32.1g of memory reserved for each job
[2016-04-04T07:21Z] Timing: organize samples
[2016-04-04T07:21Z] multiprocessing: organize_samples
[2016-04-04T07:21Z] Using input YAML configuration: /home/genetik/irina/bams/calling_2016.04.01-14.32.17/config/config.yaml
[2016-04-04T07:21Z] Checking sample YAML configuration: /home/genetik/irina/bams/calling_2016.04.01-14.32.17/config/config.yaml
[2016-04-04T07:21Z] Testing minimum versions of installed programs
[2016-04-04T07:21Z] Timing: alignment preparation
[2016-04-04T07:21Z] multiprocessing: prep_align_inputs
[2016-04-04T07:21Z] multiprocessing: disambiguate_split
[2016-04-04T07:21Z] Timing: alignment
[2016-04-04T07:21Z] multiprocessing: process_alignment
[2016-04-04T07:21Z] Timing: callable regions
[2016-04-04T07:21Z] multiprocessing: prep_samples
[2016-04-04T07:21Z] Prepare cleaned BED file : bams/11-7401_S6_bam
[2016-04-04T07:21Z] Traceback (most recent call last):
[2016-04-04T07:21Z]   File "/home/genetik/bcbio/share/bcbio/anaconda/lib/python2.7/site-packages/bcbio/variation/bedutils.py", line 16, in <module>
[2016-04-04T07:21Z]     from bcbio.variation import vcfutils
[2016-04-04T07:21Z]   File "/home/genetik/bcbio/share/bcbio/anaconda/lib/python2.7/site-packages/bcbio/variation/vcfutils.py", line 19, in <module>
[2016-04-04T07:21Z]     from bcbio.pipeline import config_utils, shared, tools
[2016-04-04T07:21Z]   File "/home/genetik/bcbio/share/bcbio/anaconda/lib/python2.7/site-packages/bcbio/pipeline/shared.py", line 9, in <module>
[2016-04-04T07:21Z]     import pybedtools
[2016-04-04T07:21Z]   File "/home/genetik/bcbio/share/bcbio/anaconda/lib/python2.7/site-packages/pybedtools/__init__.py", line 12, in <module>
[2016-04-04T07:21Z]     from . import contrib
[2016-04-04T07:21Z]   File "/home/genetik/bcbio/share/bcbio/anaconda/lib/python2.7/site-packages/pybedtools/contrib/__init__.py", line 4, in <module>
[2016-04-04T07:21Z]     from . import long_range_interaction
[2016-04-04T07:21Z]   File "/home/genetik/bcbio/share/bcbio/anaconda/lib/python2.7/site-packages/pybedtools/contrib/long_range_interaction.py", line 7, in <module>
[2016-04-04T07:21Z]     import pandas
[2016-04-04T07:21Z]   File "/home/genetik/bcbio/share/bcbio/anaconda/lib/python2.7/site-packages/pandas/__init__.py", line 7, in <module>
[2016-04-04T07:21Z]     from pandas import hashtable, tslib, lib
[2016-04-04T07:21Z]   File "pandas/src/numpy.pxd", line 157, in init pandas.hashtable (pandas/hashtable.c:38262)
[2016-04-04T07:21Z] ValueError: numpy.dtype has the wrong size, try recompiling
[2016-04-04T07:21Z] Prepare merged BED file : bams/11-7401_S6_bam
[2016-04-04T07:21Z] Prepare cleaned BED file : bams/12-304_S8_bam
[2016-04-04T07:21Z] Traceback (most recent call last):
[2016-04-04T07:21Z]   File "/home/genetik/bcbio/share/bcbio/anaconda/lib/python2.7/site-packages/bcbio/variation/bedutils.py", line 16, in <module>
[2016-04-04T07:21Z]     from bcbio.variation import vcfutils
[2016-04-04T07:21Z]   File "/home/genetik/bcbio/share/bcbio/anaconda/lib/python2.7/site-packages/bcbio/variation/vcfutils.py", line 19, in <module>
[2016-04-04T07:21Z]     from bcbio.pipeline import config_utils, shared, tools
[2016-04-04T07:21Z]   File "/home/genetik/bcbio/share/bcbio/anaconda/lib/python2.7/site-packages/bcbio/pipeline/shared.py", line 9, in <module>
[2016-04-04T07:21Z]     import pybedtools
[2016-04-04T07:21Z]   File "/home/genetik/bcbio/share/bcbio/anaconda/lib/python2.7/site-packages/pybedtools/__init__.py", line 12, in <module>
[2016-04-04T07:21Z]     from . import contrib
[2016-04-04T07:21Z]   File "/home/genetik/bcbio/share/bcbio/anaconda/lib/python2.7/site-packages/pybedtools/contrib/__init__.py", line 4, in <module>
[2016-04-04T07:21Z]     from . import long_range_interaction
[2016-04-04T07:21Z]   File "/home/genetik/bcbio/share/bcbio/anaconda/lib/python2.7/site-packages/pybedtools/contrib/long_range_interaction.py", line 7, in <module>
[2016-04-04T07:21Z]     import pandas
[2016-04-04T07:21Z]   File "/home/genetik/bcbio/share/bcbio/anaconda/lib/python2.7/site-packages/pandas/__init__.py", line 7, in <module>
[2016-04-04T07:21Z]     from pandas import hashtable, tslib, lib
[2016-04-04T07:21Z]   File "pandas/src/numpy.pxd", line 157, in init pandas.hashtable (pandas/hashtable.c:38262)
[2016-04-04T07:21Z] ValueError: numpy.dtype has the wrong size, try recompiling
[2016-04-04T07:21Z] Prepare merged BED file : bams/12-304_S8_bam
[2016-04-04T07:21Z] Prepare cleaned BED file : bams/13-6849_S1_bam
[2016-04-04T07:21Z] Traceback (most recent call last):
[2016-04-04T07:21Z]   File "/home/genetik/bcbio/share/bcbio/anaconda/lib/python2.7/site-packages/bcbio/variation/bedutils.py", line 16, in <module>
[2016-04-04T07:21Z]     from bcbio.variation import vcfutils
[2016-04-04T07:21Z]   File "/home/genetik/bcbio/share/bcbio/anaconda/lib/python2.7/site-packages/bcbio/variation/vcfutils.py", line 19, in <module>
[2016-04-04T07:21Z]     from bcbio.pipeline import config_utils, shared, tools
[2016-04-04T07:21Z]   File "/home/genetik/bcbio/share/bcbio/anaconda/lib/python2.7/site-packages/bcbio/pipeline/shared.py", line 9, in <module>
[2016-04-04T07:21Z]     import pybedtools
[2016-04-04T07:21Z]   File "/home/genetik/bcbio/share/bcbio/anaconda/lib/python2.7/site-packages/pybedtools/__init__.py", line 12, in <module>
[2016-04-04T07:21Z]     from . import contrib
[2016-04-04T07:21Z]   File "/home/genetik/bcbio/share/bcbio/anaconda/lib/python2.7/site-packages/pybedtools/contrib/__init__.py", line 4, in <module>
[2016-04-04T07:21Z]     from . import long_range_interaction
[2016-04-04T07:21Z]   File "/home/genetik/bcbio/share/bcbio/anaconda/lib/python2.7/site-packages/pybedtools/contrib/long_range_interaction.py", line 7, in <module>
[2016-04-04T07:21Z]     import pandas
[2016-04-04T07:21Z]   File "/home/genetik/bcbio/share/bcbio/anaconda/lib/python2.7/site-packages/pandas/__init__.py", line 7, in <module>
[2016-04-04T07:21Z]     from pandas import hashtable, tslib, lib
[2016-04-04T07:21Z]   File "pandas/src/numpy.pxd", line 157, in init pandas.hashtable (pandas/hashtable.c:38262)
[2016-04-04T07:21Z] ValueError: numpy.dtype has the wrong size, try recompiling
[2016-04-04T07:21Z] Prepare merged BED file : bams/13-6849_S1_bam
[2016-04-04T07:21Z] Prepare cleaned BED file : bams/14-5639_S4_bam
[...]

Here is an exerpt from the config file:
---
upload:
  dir: ../final
details:
  - files: /home/genetik/irina/bams/11-7401_S6.bam
    description: bams/11-7401_S6.bam
    metadata:
      batch: batch1
      sex: male
    analysis: variant2
    genome_build: hg19
    algorithm:
      aligner: false
      bam_clean: picard
      mark_duplicates: false
      variantcaller: [mutect2, varscan, scalpel, freebayes]
      min_allele_fraction: 1
      quality_format: Standard
      coverage_interval: amplicon
      recalibrate: false
      realign: gatk
      variant_regions: /home/genetik/irina/myeloid_targets.s.m.bed
      clinical_reporting: true
      effects: false
      ensemble:
        numpass: 1

Pär;
Sorry about the install issue. Is this a slightly older version of bcbio? There was a problem late last year with this due to having incompatible versions of numpy packages (there was a switch from numpy 1.9 to 1.10). There is a workaround here which should hopefully get your packages up to date and in sync:
https://github.com/chapmanb/bcbio-nextgen/issues/1105#issuecomment-155372386
Hope this fixes it for you. If not, feel free to re-open and we can investigate more.

Hi!
I updated to the latest devel release of bcbio-nextgen. The error looks somewhat different now: no complaints about numpy this time. Instead, it complains about bcbio.variation.bedutils.remove_bad(x) and AttributeError: 'module' object has no attribute 'bedutils'.

Sorry about the continued problems. This looks like you've gotten a working set of numpy libraries now and moved past the initial error. The new error looks like you're getting an old version of bcbio without the bedutils module. This could either be due to having your PYTHONPATH or PYTHONHOME set to an older version. Is it possible you're exporting these or have another version of bcbio on the system that might cause the conflict? Hope this helps.

PYTHONPATH was indeed set. Now everything seems to work, thanks for your help!


Can't get OnText handle to work

Hi,
I'm trying to learn go lang by porting my telegram bot from C# to go, and I'm using Telebot because it seems quite easy to use.
I could manage to get the "/hello" handle to work; after that I added the "OnText" handle because I want to check text messages for spam but, while the "/hello" handle works the OnText handle seems to never get fired. I added the bot to a group and I expect the OnText to get fired each time I write a text message.
Here is my simple code:
package main

import (
	"time"
	"log"

	tb "github.com/tucnak/telebot"
)

func main() {
	log.Println("Starting the bot...")
	b, err := tb.NewBot(tb.Settings{
		Token:  "MY TOKEN",
		Poller: &tb.LongPoller{Timeout: 10 * time.Second},
	})

	if err != nil {
		log.Fatal(err)
		return
	}

	// This doesn't get fired!!
	b.Handle(tb.OnText, func(m *tb.Message) {
		b.Send(m.Chat, "Got a text message")
	})

	// This works
	b.Handle("/hello", func(m *tb.Message) {
		options := new(tb.SendOptions)
		options.ReplyTo = m
		b.Send(m.Chat, "Hi "+m.Sender.FirstName+", I'm the Bot!", options)
	})

	b.Start()
}

Ok, I did some more tests and it seems the OnText handle gets actually fired if my message starts with "/" (assuming it's not "/hello"). Any other text message not starting by "/" does not trigger the OnText hendle. I configured my bot privacy setting to off (I get "Has access to messages" under the bot user).
Is this normal behavior? How am I supposed to catch any group text message?

@simonerom just to make sure: same bot works as expected with C#? Does it work if bot is mentioned (by username)?

Yep, that's true.

@GTANAdam what is true?

@GTANAdam what is true?

The bot isn't reading the messages posted in groups, but only commands.

@GTANAdam then disable bot privacy in botfather

@stek29 That somehow didn't solve it.

same problem

i had the same issue lately with my bot in a chat. i could trigger the OnText handler only by a text message starting with "/", even though my bot was configured to have access to all messages in a chat.
the problem was resolved by removing the bot from a chat and adding it back after making privacy off.
i guess it's just how telegram works


rmp: Adds support to create an abc standard cell library from opensta

Creates a factory class to map an sta library into an abc one. This library can be used to unmap a netlist in ABC into an AIG for optimization, and then back into a mapped netlist

Needs https://github.com/berkeley-abc/abc/pull/294 to be merged here and then in the openroad copy

@maliberty Can you merge this


Background color in matplotlib - white vs. transparent

Glen Peters pointed out in this tweet that copy-pasting pyam/matplotlib figures into social media leads to suboptimal results. Any thoughts on whether to set the background color to white for the entire figure, or add an option to do this easily?

I wonder if we could use a default theme, e.g., seaborn's white grid that is more to our liking? Not sure if this would fix the background issue or not...

I agree that it would be odd to have white background within axes but transparent outside for default.
Might be useful in some cases, but generally non-transparent background is better i think.


Dockerfile robustness improvements

What have you changed?
Make dockerfiles/artifact generation more robust against changes like https://github.com/tikv/tikv/pull/6137.
What is the type of the changes?

Engineering (engineering change which doesn't change any feature or fix any issue)

How is the PR tested?

Manual test (add detailed scripts or steps below)

make dist_artifacts

Does this PR affect documentation (docs) or should it be mentioned in the release notes?
no
Does this PR affect tidb-ansible?
no
Refer to a related PR or issue link (optional)
Benchmark result if necessary (optional)
Any examples? (optional)

/test

LGTM

/merge

/retest

/test

/merge


Rename KCell.vinsts to KCell.insts

Is your feature request related to a problem? Please describe.
In some develop in gdsfactory it would be nice for KCell and VKCell to have more similar method / property names
Describe the solution you'd like
Rename vinsts or add a @property for insts in the VKCell

I have just misread some code, apologies


gantt x-axis display

Hi,
First off many thanks for such useful tool.
I was using the online live editor and I encountered a weird issue. The gantt I was trying to create was as shown below:
gantt
dateFormat DD/MM/YY
title PostNord Deployment Plan

Section Common
Task 1 :done,10/03/17, 13/03/17
Task 2 :done, 02/12/16, 13/03/17
Task 3 :done, 02/12/16, 13/12/16
Section Section_1
Design :02/12/16, 22/02/17
Design 2 :crit, 02/12/16, 06/03/17
Design 3 :crit, 02/12/16, 18/03/17

Order wave 1 :20/03/17, 14/04/17
Order wave 2 :crit, 27/03/17, 21/04/17
Order wave 3 :crit, 03/04/17, 28/04/17

Deployment Wave 1 :17/04/17, 23/06/17
Deployment Wave 2 :24/04/17, 09/06/17
Deployment Wave 3 :crit,01/05/17, 07/07/17

Section Section_2
Design :crit, 23/02/17, 15/03/17
Order and Deliver : 20/03/17, 28/04/17
Deployment :01/05/17, 16/08/17

When I use this the Week changes to year for first tick of x-axis (Jan) and then changes to mm-yy for March and April, then changes back to W-17 for May and then again to mm-yy for rest of the months.
I also found that changing the duration of last task to June or July changes everything back to W-nn format. I am also trying to find out a way to change the resulting image to show whole of x-axis as just mmm (Jan, Feb, Mar etc.) and reading some of the previous issues but haven't really understood the axisFormat thing that people have referred to and would really appreciate some guidance on it.

+1

@knsv
actually while I was not getting anywhere with my problem, I came across timeline from vis js library which appears to be doing the same thing and then an R-markdown package based on this js library called timevis which can help create a timeline fairly easily. I was able to create some really nice visuals of gantt using variations of text below on rstudio:
{r echo = FALSE}
library(timevis)

dataGroups <- read.csv(header = TRUE, text = "
id,content,group,start,end,style
1,Task 1,Common,2016-12-02,2017-03-17,NA
2,Task 2,Common,2016-12-02,2017-02-17,background-color: red;color: white
3,Task 3,Common,2016-12-02,2017-03-17,NA
4,Task 4,Section 1,2016-12-02,2017-02-17,background-color: red,NA
5,Task 5,Section 1,2017-03-17,2017-04-07,NA,NA
6,Task 6,Section 2,2017-03-17,2017-04-27,NA,NA
7,Task 7,Section 2,2017-04-07,2017-04-27,NA,NA
8,Task 8,Section 2,2017-05-07,2017-05-17,background-color: red,NA
9,Task 9,Section 2,2017-06-07,2017-06-20,NA,NA
10,Task 10,Section 3,2017-07-07,2017-07-31,background-color: red,NA
")

groups <- read.delim(header = TRUE, sep = ":", text = "
id: content
Common:Common
Section 1:Section 1
Section 2:Section 2
Section 3:Section 3
")

timevis(
  dataGroups,
  groups,
  showZoom = TRUE,
  zoomFactor = 1,
  fit = TRUE
)

Now as it is all being done using javascript library much like mermaid does using phantomjs, I was wondering if someone can look into creating the solution / fork of timevis into mermaid but using vis.js to create the gantt.
My apologies in advance if this is a very naive expectation or makes no sense but I would much rather be able to use mermaid on atom than open R-Studio.
Thanks again!!!

That is quite nice! An integration with vis.js for would open new possibilities for diagrams. Good idea!
Get Outlook for iOS
On Wed, Apr 12, 2017 at 3:53 PM +0200, "pubmania"<EMAIL_ADDRESS>wrote:
@knsv
actually while I was not getting anywhere with my problem, I came across timeline from vis js library which appears to be doing the same thing and then an R-markdown package based on this js library called timevis which can help create a timeline fairly easily. I was able to create some really nice visuals of gantt using variations of text below on rstudio:
{r echo = FALSE}
library(timevis)
dataGroups <- read.csv(header = TRUE, text = "
id,content,group,start,end,style
1,Task 1,Common,2016-12-02,2017-03-17,NA
2,Task 2,Common,2016-12-02,2017-02-17,background-color: red;color: white
3,Task 3,Common,2016-12-02,2017-03-17,NA
4,Task 4,Section 1,2016-12-02,2017-02-17,background-color: red,NA
5,Task 5,Section 1,2017-03-17,2017-04-07,NA,NA
6,Task 6,Section 2,2017-03-17,2017-04-27,NA,NA
7,Task 7,Section 2,2017-04-07,2017-04-27,NA,NA
8,Task 8,Section 2,2017-05-07,2017-05-17,background-color: red,NA
9,Task 9,Section 2,2017-06-07,2017-06-20,NA,NA
10,Task 10,Section 3,2017-07-07,2017-07-31,background-color: red,NA
")
groups <- read.delim(header = TRUE, sep = ":", text = "
id: content
Common:Common
Section 1:Section 1
Section 2:Section 2
Section 3:Section 3
")
timevis(
dataGroups,
groups,
showZoom = TRUE,
zoomFactor = 1,
fit = TRUE
)
Now as it is all being done using javascript library much like mermaid does using phantomjs, I was wondering if someone can look into creating the solution / fork of timevis into mermaid but using vis.js to create the gantt.
My apologies in advance if this is a very naive expectation or makes no sense but I would much rather be able to use mermaid on atom than open R-Studio.
Thanks again!!!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

Hi, with my extremely limited knowledge I did manage to create an html page where if I put CSV for data it will create Gantt chart using timeline module of vis js. I used papaparse js to parse CSV and create json dataset. Code is here and demo is here.
I am hoping someone knowledgeable can now help integrate it with mermaid or atleast give me some pointers on how to go about doing it.

I think the orignal issue has been fixed: https://mermaidjs.github.io/mermaid-live-editor/#/edit/Z2FudHQgCmRhdGVGb3JtYXQgREQvTU0vWVkKYXhpc0Zvcm1hdCAlWS0lbQp0aXRsZSBQb3N0Tm9yZCBEZXBsb3ltZW50IFBsYW4KClNlY3Rpb24gQ29tbW9uClRhc2sgMSA6ZG9uZSwxMC8wMy8xNywgMTMvMDMvMTcKVGFzayAyIDpkb25lLCAwMi8xMi8xNiwgMTMvMDMvMTcKVGFzayAzIDpkb25lLCAwMi8xMi8xNiwgMTMvMTIvMTYKU2VjdGlvbiBTZWN0aW9uXzEKRGVzaWduIDowMi8xMi8xNiwgMjIvMDIvMTcKRGVzaWduIDIgOmNyaXQsIDAyLzEyLzE2LCAwNi8wMy8xNwpEZXNpZ24gMyA6Y3JpdCwgMDIvMTIvMTYsIDE4LzAzLzE3CgpPcmRlciB3YXZlIDEgOjIwLzAzLzE3LCAxNC8wNC8xNwpPcmRlciB3YXZlIDIgOmNyaXQsIDI3LzAzLzE3LCAyMS8wNC8xNwpPcmRlciB3YXZlIDMgOmNyaXQsIDAzLzA0LzE3LCAyOC8wNC8xNwoKRGVwbG95bWVudCBXYXZlIDEgOjE3LzA0LzE3LCAyMy8wNi8xNwpEZXBsb3ltZW50IFdhdmUgMiA6MjQvMDQvMTcsIDA5LzA2LzE3CkRlcGxveW1lbnQgV2F2ZSAzIDpjcml0LDAxLzA1LzE3LCAwNy8wNy8xNwoKU2VjdGlvbiBTZWN0aW9uXzIKRGVzaWduIDpjcml0LCAyMy8wMi8xNywgMTUvMDMvMTcKT3JkZXIgYW5kIERlbGl2ZXIgOiAyMC8wMy8xNywgMjgvMDQvMTcKRGVwbG95bWVudCA6MDEvMDUvMTcsIDE2LzA4LzE3
And by the way you can configure the axis format: https://github.com/knsv/mermaid/issues/269#issuecomment-373229136

@pubmania  I like the work you have done and I created a new issue to track it: https://github.com/knsv/mermaid/issues/640


Check overload resolution docs against new rules

There are three new steps to overload resolution that are documented here
My first reading gives me the impression that the 1st rule doesn't change any observable behavior. My first read says the 2nd and 3rd would result in a better method being found in some instances where pre 7.3 versions would result in an ambiguity. (That's after a first reading, not to be taken without verification on the proposal).
Note: This might be best handled in the context of #4729

I reviewed our docs on overload resolution, and we don't state anything that violates the new rules.
Closing this, as the only location where it will be covered is in the "what's new" topic.


Expanded sensor graphs not displaying correctly

Home Assistant release with the issue:
0.77.3

Last working Home Assistant release (if known):
Unknown
Operating environment (Hass.io/Docker/Windows/etc.):
Hassio

Component/platform:
Frontend

Description of problem:
Click on a sensor for a graph view (all good so far):
https://i.imgur.com/JLam5tV.png
After clicking on the title to expand the view the bottom of the graph is cut off. This seems to be because the size of the sensor text under the graph is not taken into account:
https://i.imgur.com/7v2csgN.png
This problem only occurs when expanding individual sensor graphs, not for the history graph component as there is no text under these.
It also occurs in all web browsers I've tested. Firefox, Chrome and Safari.
Problem-relevant configuration.yaml entries and (fill out even if it seems unimportant):

Traceback (if applicable):

Additional information:

Instead of doing a forced refresh, actually close the browser and all tabs.  Then hit CTRL+ALT+DEL and clear all cache.  Then restart browser and try again.

I cleared all my browser history (CTRL+SHIFT+DEL) and restarted it. The same problem persists.

Ok good to know those steps were performed.  Also for front end issues like this the best place to file them is here:  https://github.com/home-assistant/home-assistant-polymer

Should I close this and open an issue over there?

Have confirmed another user has the same problem after clearing his cache using Chrome on Windows 10 and iOS.

@tomlut taking a closer look I see it too, looks like the last row is removed on desktops.  Yea I would file it over there as its a front end issue and not backend.  I don't think I have access to the move bot.

/move to home-assistant/home-assistant-polymer


Small breadcrumb update

Removing "documentation" from the breadcrumb to align with guidelines and use space more efficiently.

@kfollis
Can you review the proposed changes? IMPORTANT: When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
#assign:@MicrosoftDocs/public-repo-pr-review-team

The following users don't have valid GitHub IDs, or are not collaborators on this repo: microsoftdocs/public-repo-pr-review-team

#sign-off


Vulnerability roundup 84: file-roller-3.32.1: 1 advisory

search, files

[ ] CVE-2020-11736 CVSSv3=3.9 (nixos-19.09)

Scanned versions: nixos-19.09: 31dcaa5eb67. May contain false positives.

19.09 is EOL
Closing


ParagraphCell out of border

Hi,
when I insert a paragraph inside a table, sometimes the text, if too long, goes out of border. Attached is the code with a little example, the pom.xml file and the output.
what am I doing wrong?
Thank you
pom.xml.txt
Report.java.txt
example.pdf

Looks like a bug to me. From what I can see from a quick test it only happens if row spanning is active. I will look into it.

The problem here is that cells with row spanning do not have their "own" height – it will just be calculated by the other "regular" cells of the rows it spans.
That means: Because the two cells in the second column are not very high their sum does not exceed the height of the cell that spans two rows.
So what can you do about it? You could set the minWidth() of the cells on the right, e.g. like so:
private Table createRegularTable() {
    return Table.builder()
            .addColumnsOfWidth(50, 50)
            .addRow(Row.builder()
                    .add(TextCell.builder().minHeight(70).borderWidth(1).text("1").build())
                    .add(TextCell.builder().borderWidth(1).text("fu bza asd fad fda dsafa afa fsdfs fdsfs fds").rowSpan(2).build())
                    .build())
            .addRow(Row.builder()
                    .add(TextCell.builder().minHeight(70).borderWidth(1).text("2").build())
                    .build())
            .build();
}

What you also could do (but this is a bit hacky): calculate the height of the cell upfront by building a table containing only this cell and then set the minWidth() relative to this calculated value:
private Table createRegularTableWithCalculation() {
    TextCell textCell = TextCell.builder().borderWidth(1).text("fu bza asd fad fda dsafa afa fsdfs fdsfs fds").build();

    Table.builder()
            .addColumnsOfWidth(50)
            .addRow(Row.builder()
                    .add(textCell)
                    .build())
            .build();

    System.out.println(textCell.getHeight());

    Table realTable = Table.builder()
            .addColumnsOfWidth(50, 50)
            .addRow(Row.builder()
                    .add(TextCell.builder().minHeight(textCell.getHeight() / 2).borderWidth(1).text("1").build())
                    .add(textCell.toBuilder().rowSpan(2).build())
                    .build())
            .addRow(Row.builder()
                    .add(TextCell.builder().minHeight(textCell.getHeight() / 2).borderWidth(1).text("2").build())
                    .build())
            .build();

    return realTable;
}

Hope this helps.
Best,
Stefan

I tried the solution with the method that calculates the height. In the example I posted the solution works because the case is very simple. Now I have to try it in the project where the tables are dynamically generated.
Thank you per the answer.

It's really an interesting case, I have to admit.
I honestly didn't think of this issue when I implemented row spanning (and believe me: supporting row spanning and column spanning at the same time creates a huge load of complexity which can easily be underestimated).
Since it would be really nice to have this feature I will try to look into it if I may find enough time.

I created a fix (which still needs a bit of testing) and I would like to ask you to try it out @buzzer83: It's in the branch bugfix/row-spanning-sizing. Could you install it locally and run your code with this version (i.e. 0.6.3-SNAPSHOT) and tell me if it worked for you?
PS: When installing locally just run mvn clean install -DskipTests -Dgpg.skip -Ddependency-check.skip=true (in order to speed things up).

I installed version 0.6.3 locally (not with pom.xml) but it gives me several errors (look at them in the image). I certainly did something wrong.

You can install it locally by running the following commands:
git clone https://github.com/vandeseer/easytable.git
git checkout -b snapshot-branch origin/bugfix/row-spanning-sizing
mvn clean install -DskipTests -Dgpg.skip -Ddependency-check.skip=true

Afterwards you can reference the locally installed version in your pom.xml with version 0.6.3-SNAPSHOT.
PS: In your IDE you will get compile errors in case you don't have the Lombok plugin installed. But it's not necessary that you open it in your IDE.

I installed the Lombok plugin and managed to resolve. your solution works correctly.  TY
Now I have to understand why the lines in paragraphs go wrong. For example in the third column. To indicate the end of the line I put the label "end".
example.pdf

Glad to hear the solution works. For the other issue (or question) you're facing, please open another issue in order not to mix up things. Thanks.

obviously. ty.

I will leave this issue open, because the fix is not yet released. As soon as version 0.6.3 is published, I'll close it.
Thanks for testing it upfront @buzzer83! 👍

Released with version 0.6.3.


Send messages from trader via transit-movements-router to EIS/ERMIS

I've just updated the tests to use property based testing and to fix a comment that had been missed. The tests all pass, and I tested using this branch of the API layer to check that it works with the header, and I'm pleased to say it does.
This either needs to await the XML arrivals endpoint PR, or for the linked branch to be PRd and merged -- one for standup tomorrow.

If we're not checking for Unit do we even need to do this?

Yes, we're still checking that the future was succesful and not a failure, an exception will be thrown by whenReady if it's a failure. (I took out checking for Unit because it was causing compiler warnings.)


Fix 404 links in documentation site

Many links in the documentation site are broken.
I could find those intended destinations on the site, but the URL is a little bit different.
Probably have to walk through the docs and update those outdated links.
Some of the broken links

404 Error

Bump, hope someone could spend some time on updating the docs!


Support for test command

So I stumped upon some problem with auto test with Gitlab and found this
https://gitlab.com/gitlab-org/gitlab-ce/issues/26941
Basically gitlab said that python buildpack does not support test command. And then I also found this from Nov 2016 https://github.com/heroku/heroku-buildpack-python/commit/89125df805a8a942dbb18d7fa413da7cc28359c0 which we removed test command because it's deprecated
The test command is pretty simple, should I add it back?

we have tests.

they just don't exist there.


Add a password component

Add a password component for both bootstrap and materialize.
Based on the Default (text) input, I just added a type='password'.

Nice! For your next PRs you can write "Closes #2257" where 2257 is the related issue number at the end of your message, then github can close the issue automatically when the PR is merged.
Thanks!


[MB-3498] Adds the LeftNav component w/ shipments to the Move Task Order page

Description
This PR adds the LeftNav component (similar to the one on the Move Details page) to the Move Task Order page in the TOO interface.
It does not order them in any particular way at the moment nor do we distinguish between shipments of the same type or family.  Meaning HHGs aren't necessarily going to be displayed together nor could you tell them apart from their labels.
I also did a bit of refactoring because this component has been around for a while and doesn't get much attention.  I tried to reduce the amount of repeated searching we were doing of the service items.  I also fixed the react-query cache invalidation because it was using the wrong query key.  I tried to beef up the tests a bit as well, the nested components go a bit deep.
Reviewer Notes
Is there anything you would like reviewers to give additional scrutiny?
Setup
Add any steps or code to run in this section to help others prepare to run your code:
make db_dev_e2e_populate
make server_run
make office_client_run

Click on any move that is in the status Approvals Requested or Move Approved from the TOO queue
Navigate to the MTO tab
The Left Nav should be visible and contain all of the approved shipments for the move
Clicking on a shipment in the LeftNav should scroll you to the top of the shipment container
Also scrolling should update the border in the LeftNav of the current shipment in view if the page is large enough.

Code Review Verification Steps

[ ] If the change is risky, it has been tested in experimental before merging.
[ ] Code follows the guidelines for Logging
[ ] The requirements listed in Querying the Database Safely have been satisfied.
Any new migrations/schema changes:

[ ] Follow our guidelines for zero-downtime deploys (see Zero-Downtime Deploys)
[ ] Have been communicated to #g-database
[ ] Secure migrations have been tested following the instructions in our docs

[ ] There are no aXe warnings for UI.
[ ] This works in Supported Browsers and their phone views (Chrome, Firefox, IE, Edge).
[ ] Tested in the Experimental environment (for changes to containers, app startup, or connection to data stores)
[ ] User facing changes have been reviewed by design.
[ ] Request review from a member of a different team.
[ ] Have the Jira acceptance criteria been met for this change?

References

Jira story for this change

Screenshots

@amanda5iu @brooksr4 Do you all know what the right color is for the LeftNav link text when not selected? I see some variations in this Zeplin collection https://zpl.io/VxNNkXk


in React 16, PropTypes will not be supported

the error occured at bannerSize: React.PropTypes.string,
when you create a new react native app, the package.json will be like
  "dependencies": {     "react": "16.0.0-alpha.12",     "react-native": "0.48.3",
.....................
so cannot use this library.

OK, I solved it by npm install<EMAIL_ADDRESS>the same issue

For React 16 support I recommend using the 2.0.0-alpha version, currently being developed in #165


Ensure response body is fully read on context errors

Issue #, if available:
https://github.com/twitchtv/twirp/issues/384
Description of changes:
As mentioned in the GH issue above,  the Golang docs specify that:

If the Body is not both read to EOF and closed, the Client's underlying RoundTripper (typically Transport) may not be able to re-use a persistent TCP connection to the server for a subsequent "keep-alive" request.

The original poster QueenCitySmitty mentions that

Both doProtobufRequest and doJSONRequest only defer closing the body, not also reading to EOF. This has the potential to cause issues with TCP.

marioizquierdo makes a good suggestion to do this only for context errors as it is:

[...] safer to implement, and will not cause a second read on the response body for regular successful requests.

[X] By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Can you say a bit more about when this would make a difference? It seems to me that once the Context value used for a Request expires or is canceled, the http.Transport will try to clean up any remaining state as soon as possible. For HTTP/1.1, that means it's going to close the TCP connection, thereby preventing its reuse. I'm not as familiar with this part of Go's HTTP/2 support, but I'd guess that the underlying TCP connection would be reusable with or without this change.
Is there an integration test that confirms the description of the problem, and that the proposed change solves the problem?

Can you say a bit more about when this would make a difference? It seems to me that once the Context value used for a Request expires or is canceled, the http.Transport will try to clean up any remaining state as soon as possible. For HTTP/1.1, that means it's going to close the TCP connection, thereby preventing its reuse. I'm not as familiar with this part of Go's HTTP/2 support, but I'd guess that the underlying TCP connection would be reusable with or without this change.

We are correcting an oversight in the twirp implementation to always read response bodies to EOF for cases where we experience context errors - this better aligns the twirp implementation with the go docs:
// If the returned error is nil, the Response will contain a non-nil
// Body which the user is expected to close. If the Body is not both
// read to EOF and closed, the Client's underlying RoundTripper
// (typically Transport) may not be able to re-use a persistent TCP
// connection to the server for a subsequent "keep-alive" request.

Is there an integration test that confirms the description of the problem, and that the proposed change solves the problem?

AFAICT, there are no benchmark or integration tests in this package. Testing these changes is the difficult and time consuming part, and it's what we're working on right now.

Thanks. I see that the code isn't meeting the letter of the net/http doc's requirements for connection reuse. I see that the current patch addresses only one of the ways the code doesn't meet those requirements. I don't see how it would matter in practice.
On the one hand, I have doubts that this is a real problem; it looks like the window (between no error from http.Client.Do, followed by error from ctx.Err) would "usually" be "quite small". And again, how would consuming the Body make a difference in whether the http.Transport is able to reuse the connection? It seems that either the Body is already sufficiently consumed for the Transport to have removed the association between the Request+Response.Body and the TCP connection (so failing to consume the whole body wouldn't make a difference in the reusability of the TCP connection), or the association is still there and so the Transport is also in the process of detecting that the Context has been canceled/expired and will (for HTTP/1.1) discard the TCP connection anyway.
On the other hand, if it's really important for us to consume the entire body on every code path (when http.Client.Do doesn't return an error), then maybe it's time to do that in a way that's more obviously complete (and which will remain complete as the code continues to evolve), rather than addressing these one at a time. The body is also not consumed when the response's status code is a redirect.
If the time has come to bring the caller of http.Client.Do into compliance with that method's docs, I'd advocate for making that alignment obvious and permanent rather than adding to the patchwork. If the goal is a targeted change to address a specific operational issue, then a demonstration / tests would help to drive that point home.


Add ping pong health, remove a timeout interval, fix up network events

Need to wait on a kittycad/lib update so pong request type comes through...

@Irev-Dev ready for review

Oh actually the vitest that failed looks relevant, maybe touch base with @franknoirot if it's not easy to fix, I think he wrote the networkIndicator test?


404 on logout when LANDING_PAGE=login

Gitea version (or commit ref): 1.14.1
Git version: 2.20.1
Operating system: Linux raspberrypi 5.10.17-v7l+ #1403 SMP Mon Feb 22 11:33:35 GMT 2021 armv7l GNU/Linux
Install binary Link: https://github.com/go-gitea/gitea/releases/download/v1.14.1/gitea-1.14.1-linux-arm-6
Database (use [x]):

[ ] PostgreSQL
[x] MySQL
[ ] MSSQL
[ ] SQLite

Can you reproduce the bug at https://try.gitea.io:

[ ] Yes (provide example URL)
[x]  No

Log gist:
I canot change try.gitea.io configuration to using login as homepage.

Description

using login as homepage;
# ${gitea_home}/custom/conf
[server]
LANDING_PAGE     = login

start gitea;

visit homepage, and login;

click some repository;

click right upper account icon, select "logout" from dropdown menu;

404 return;
...

Screenshots
this issue can be reproduced.

Although I think I now understand how to replicate this issue - it really would have been much more helpful if you could have provided logs.

Although I think I now understand how to replicate this issue - it really would have been much more helpful if you could have provided logs.

Tonight, I want to reproduce this issue, but using other computer in my home(last time using the one in the office), but annoying, I changed log level to debug and restart server, it cannot be reproduced.
So, I think maybe the reason is browser cache?
I will temporary close this issue, and remaining log level to debug; when it occured again, I will issue log with it.

Maybe the repository is private? Then you can't access it after logging out.

Yes, it's private repository, but it just only for me, and I can access it, promise :)


➕ [RESOURCE] - Adding Computer Networks Course

Idea Contribution

[X] I have read all the resource request issues.
[X] I'm interested in working on this issue
[X] I'm part of GSSOC organization

Where do you want to add the resource? Specify the category and section.
Computer Networks -> Paid Courses Section
Provide the URLs of the resources you are adding
https://www.coursera.org/specializations/computer-network-security
Any alternative approaches/features
No response
Additional Context
No response
Code of Conduct

[X] I follow the CONTRIBUTING GUIDELINE of this project.

Closr this issue for now. Compile all the resources you want to add and then create PR

okay @jfmartinz I'll do that


stat: PCA fails with noasm or appengine build tags

The failure is due to numerical stability and only really shows up in the lowest valued column.

What shall we do with this?
The smallest singular value is practically zero and the SVD (and thus the principal vectors) is not unique in such case (if I'm not wrong). I think that the "numerical instability" is just the noasm tag finding another acceptable vector, which is also supported by the fact that Mul(vecs.T(), vecs) gives the identity matrix. Maybe the test should be changed to handle this or we can just continue ignoring the test case.


Either implementation

This links to https://github.com/MonacsLib/Monacs/issues/23
This is a very first implementation. Didn't actualy played a lot with it. I'm creating PR to show you what I did and hear what can be improved.
There are no unit tests beacuse I simply didn't write them yet :P

I added function signatures quicker understanding what a function does. It helps me a lot with developing and using the functions later. Maybe creating some sort of metadata (attributes?) with such information would be beneficial.

I assume that two Either instances should be equal when they have the same generic parameters and underlying values. We shouldn't do casting to object, I think the interface is unnecessary here.
The fact that in F# you can omit the type params and have object for the missing ones is the result of stronger type inference than in C#, where you cannot skip parameters in such case. There is no need to worry about it.

Okay, I'll delete IEither and bring back previous equality check.

I approved the changes and will merge and test them. In meanwhile you can start working on the API docs for the newly created classes and methods.


REQ: Please add restored_external source

Please add the source of restored_external thanks

Ok, it's xerubs source but slightly modified


First spawn

On first entry to the server, the player appears in randomly place on the map, and not teleported to the spawn point.

base.lua:17
Function tvRP.teleport(x,y,z) works good.

But after a while, they are teleported to their last position right ?

Yep

Should be fixed.


bash-it slows down mac terminal

Open a new or tab in  terminal/iterm2 and compare the time to show the prompt with bash-it and without bash-it.
With bash-it it is taking way longer.
Issue #2: This is slow when we execute any command, like ls or any other command. The prompt takes time to come back to terminal.

Well, that's a trade-off when using a tool like this. When you open a new shell, all of your enabled plugins, aliases and completions are evaluated and loaded into the shell - this obviously takes time.
If you want to make things faster, I suggest that you only enable the things that you really need, and disable everything else.
Please also see #517 for some stats on the individual plugins' load times.
Other than that, what do you suggest to solve this?

Thanks for the response, I think we can live with terminal loading for first time.
I have updated it with issue #2 where prompt takes longer to come back after executing any command. Looks like it is running some python script after each execution or reloading the bash.

This is probably caused by the theme that you're using. The themes gather a good deal of info, e.g. about the Git repo you're in, about the Ruby/Python/etc. version that you're using, and then displaying that.
Which theme are you using? Some of the themes provide options to exclude some of the info. Or it might make sense to customize or patch the theme so it doesn't gather that info.

@nwinkler It might be correct. I was using the default theme "bobby". Now I tested bunch of the themes and found nwinkler_random_colors and powerline-multiline really good in terms of looks and speed.
I added one extra line in nwinkler_random_colors, just for my taste and style so that It looks clean and easy on eyes to make out between commands.

Glad to hear that you found something that works for you!


Sik-Ho Tang | Review -- BEiT: BERT Pre-Training of Image Transformers.

Sik-Ho Tang. Review — BEiT: BERT Pre-Training of Image Transformers.

Overview
BEiT, Pretraining ViT, Using Masked Image Modeling (MIM).
BEiT: BERT Pre-Training of Image Transformers. BEiT, by Microsoft Research. 2022 ICLR, Over 300 Citations.
Self-Supervised Learning, BERT, Transformer, Vision Transformer, ViT, DALL·E.

Bidirectional Encoder representation from Image Transformers (BEiT) is proposed, where a masked image modeling (MIM) task to pretrain Vision Transformers.
BEiT first “tokenizes” the original image into visual tokens. Then some image patches are randomly masked and fed into the backbone Transformer.
The pre-training objective is to recover the original visual tokens based on the corrupted image.

BEiT Architecture

Overview of BEiT pre-training.
Overall Approach

Inspired by BERT, a pre-training task is proposed, namely, masked image modeling (MIM).
MIM uses two views for each images, i.e., image patches, and visual tokens.
The image is split into a grid of patches that are the input representation of backbone Transformer.
The image is “tokenized” to discrete visual tokens by the latent codes of discrete VAE, where discrete VAE is from DALL·E.

During pre-training, some proportion of image patches are randomly masked, and fed the corrupted input to Transformer.

The model learns to recover the visual tokens of the original image, instead of the raw pixels of masked patches.

Image Representation
During pre-training, the images have two views of representations, namely,

image patch: input representations,
visual tokens output representations.

Image Patches

Image Patches (Cut from the first figure).

The 2D image of the size $H\times W\times C$ is split into a sequence of patches $x_{p}$ ($p$ is from $1$ to $N$) of the size $P^{2}$, with the number of patch  $N=\frac{HW}{P^{2}}$  patches.
The image patches $x_{p}$ are flattened into vectors and are linearly projected which is similar to word embeddings in BERT.

Particularly, BEiT splits each $224\times 224$ image into a $14\times 14$ grid of image patches, where each patch is $16\times 16$.
Visual Tokens

Visual Tokens (Cut from the first figure).

The image is represented as a sequence of discrete tokens obtained by an “image tokenizer”, instead of raw pixels.

Specifically, the image of the size $H\times W\times C$ is tokenized into $z=[z_{1},…,z_{N}]$, where the vocabulary $V={1,…,|V|}$ contains discrete token indices.

The image tokenizer learned by discrete variational autoencoder (dVAE), by DALL·E, is directly used.
There are two modules during visual token learning, namely, tokenizer and decoder.
The tokenizer $q(z|x)$ maps image pixels $x$ into discrete tokens $z$ according to a visual codebook (i.e., vocabulary).
The decoder $p(x|z)$ learns to reconstruct the input image $x$ based on the visual tokens $z$.
The vocabulary size is set to $|V| = 8192$.

ViT Backbone

Following ViT, the Transformer backbone network is used.
The input of Transformer is a sequence of image patches $x_{i}^{p}$.
The patches are then linearly projected to obtain patch embeddings $Ex_{i}^{p}$.
The standard learnable 1D position embeddings $E_{pos}$ are added to patch embeddings:

The encoder contains $L$ layers of Transformer blocks:

The output vectors of the last layer is:

which are used as the encoded representations for the image patches, where $h_{i}^{L}$ is the vector of the $i$-th image patch.
ViTBase is used, which is a 12-layer Transformer with 768 hidden size, and 12 attention heads. The intermediate size of feed-forward networks is 3072.

BEiT Pretraining: Masked Image Modeling (MIM)
Masked Image Modeling (MIM)

BEiT Masked Image Modeling (MIM) (Cut from the first figure).

After splitting the image into image patches, as described above, approximately 40% image patches are randomly masked, where the masked positions are denoted as $M$. The masked patches are replaced with a learnable embedding $e[M]$. In BEiT, At most 75 patches are masked.
Then, the good and masked image patches are input into the L-layer Transformer.
A softmax classifier is used to predict the corresponding visual tokens:

The pre-training objective is to maximize the log-likelihood of the correct visual tokens $z_{i}$ given the corrupted image:

BEiT is pretrained on the training set of ImageNet-1K.
The pre-training runs for about 500k steps (i.e., 800 epochs) with 2k batch size. The 500k training steps take about five days using 16 Nvidia Tesla V100 32GB GPU cards.

From VAE Perspective

The BEiT pre-training can be viewed as variational autoencoder training:

In the first stage, the image tokenizer is obtained as a discrete variational autoencoder. Specifically, the first stage minimizes the reconstruction loss, with an uniform prior.
In the second stage, the prior $p_{\theta}$ is learnt while keeping $q_{\phi}$ and $q_{\Psi}$ fixed.
Thus, the above equation is re-written as:

where the second term is the proposed BEiT pre-training objective.


[5.1] Add setCollection method to AbstractPaginator

This functionality has already been added to L5.2.

Sorry, we do not backport features to 5.1.

The LTS release will only receive bug fixes.


fix: default build creation

Changes

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#582  👈
#578 : 1 other dependent PR (#580 )
#573
#572
#571
#570
#566
#565
#579
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @MasterPtato and the rest of your teammates on  Graphite


Order classes by recent failures and modification date when feeding them to javac compiler

Expected Behavior
Repeated compileJava should provide fast turnaround.
For instance, if compilation fails, and user edits a file, then Gradle should feed the problematic file first to the compiler, so user sees earlier if the error is fixed or not.
At the same time, if some of the files were modified, it makes sense to schedule them first for the compilation so the user sees if the changed files compile at all.
So it would be nice to have the following order:

Order files with known failures during previous compilation
Then order by modification date (most recently modified files should go first)

Current Behavior
Gradle schedules files in unknown order, and it might take significant time for the first error from javac to appear.
Context
javac might take significant time, especially when formal verification tools like https://checkerframework.org/ or https://github.com/google/error-prone are used.
I just checked with Java 11 on the command line, and it seems to work:
$ javac A.java B.java
A.java:1: error: class Aa is public, should be declared in a file named Aa.java
public class Aa {
       ^
B.java:1: error: class Bb is public, should be declared in a file named Bb.java
public class Bb {}
       ^
2 errors

$ javac B.java A.java
B.java:1: error: class Bb is public, should be declared in a file named Bb.java
public class Bb {}
       ^
A.java:1: error: class Aa is public, should be declared in a file named Aa.java
public class Aa {
       ^
2 errors

The issue is still relevant


Issue with back button not unmounting and remount tab scene

Version
"react": "16.0.0-alpha.12",
"react-native": "0.46.4",
"react-native-router-flux": "4.0.0-beta.14",
When I click the back button and go to my teams view to select a different team my photos view which are under a different tab is not re mounting, so I can't load the new data. I need to be able to re mount a tab view it seems that it's always cached
You can see the issue here

It is how tabbar works - it doesn't remount anything. Use Actions.refresh to update props.

I tried Actions.refresh it doesn't help in this case

Okey, it is about your app state management, not navigation.
You could try to use onEnter handler (as static function) for Photos screen to get photos for 'active' screen. Also you could (or even should) use Redux or Mobx to re-render photos once home is changed.

Yea I'm using Redux, the core issue is with the EasyListView package that I'm using, which is not rendering when props

So it is not problem of RNRF.

No I don't think so, my idea was to see if I can force an unmount and mount which would make it work

I thought RNRF back button should unmount the tab compount

No. It is how react-navigation works and how tabbar works. Why you can't use onEnter, onExit?

Btw, always try to use LATEST beta (or even master) before creating an issue.

Ok I thought I was on the latest version


Adding IoT Demo support to the client

Temperature and Light charts on the home view are now charts instead of images.
When a Room Devices Endpoint and a Room Id are defined, the values for the Temperature and Light are "live" and coming from devices/sensors.

My Room view shows just Light and Temp sliders
Changing slider values will cause the IoT Devices to change their desired and current values. After a short period of time those changes will propogate through the system for the user to see.

By default (no Room Devices Endpoint and Room Id defined) the values for the Temperature and Light are faked.

My Room view will show the Light, Temp, Window, and Music sliders along with a message informing the user that they need to configure the Room Devices Endpoint to enable room controls

Documentation will need another update once the IoT Demo backend is published out to Github
Includes fix for the CustomSlider where it wasn't setting the Value property when moving the slider thumb.

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.:x: willyt150 sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.


Clarify MRP flag setting SLA

@JohanFreelancer9 - kindly do the copy editor review for this PR. Thanks

@KCCross Please merge this PR. Thanks

@KCCross Gentle reminder


Generated DCOS bootstrap url is invalid for 1.10

The recently added 1.10 support for dcos cluster generates a url that does not exist:
https://dcosio.azureedge.net/dcos/stable/bootstrap/e38ab2aa282077c8eb7bf103c6fff7b0f08db1a4.bootstrap.tar.xz
Returns: "The specified blob does not exist."
All other versions return a valid file so running using the 1.10 example does not install dcos
https://github.com/Azure/acs-engine/blob/master/examples/dcos-releases/dcos1.10.json
https://github.com/Azure/acs-engine/commit/8420ce6214086188c46386e5f27a059894728cc3#diff-8edb2653431a5da9ad1f4286097702a7R30
This effects master and any other branch that contains this url key - can this be updated to a valid blob url

@julienstroheker Are you the caretaker of DC/OS images @ https://dcosio.azureedge.net/dcos/stable/bootstrap/? Or do you know who is?
Looks like we need to update a working tarball for 1.10. (Assuming the artifact name is different, we'll also need to update the references in acs-engine.
Thanks!

I am not unfortunately and I don't know who is.
It is weird this was working when we merged.


pycall : unable to share/expose char array (Unexpected python type descriptor: U)

In GitLab by @youldrouis on Feb 23, 2021, 11:49
code example :
metadata:
  pope_acronym_size: int
  pope_acronym: { type: array, subtype: char, size: '$pope_acronym_size' }
plugins:
  pycall:
    on_event:
      POPE_local_1fpp:
        with:
        exec: |
          import numpy as np
          acronym_size = 4
          acronym = 'toto'
          print(' * [debug] pope_acronym_size =',acronym_size)
          print(' * [debug] pope_acronym =',acronym)
          pdi.expose( 'pope_acronym_size' , acronym_size , pdi.OUT )
          arr = list(acronym)
          arr2 = np.array(arr)
          print(' * [debug] arr =', arr)
          print(' * [debug] arr2 =', arr2)
          pdi.expose( 'pope_acronym'      , arr2 , pdi.OUT )

I get this as output,
 * [debug] pope_acronym_size = 4
 * [debug] pope_acronym = toto
 * [debug] arr = ['t', 'o', 't', 'o']
 * [debug] arr2 = ['t' 'o' 't' 'o']

Exposing acronym, arr or arr2 (that are resp. string, list and numpy chararray) causes the same error : unable to interpret Unicode type :
  *** [PDI/PyCall] Error: while calling python, caught exception: Error: Impl_error: Unexpected python type descriptor: U

At:
  /usr/lib/python3/dist-packages/pdi/__init__.py(41): share
  /usr/lib/python3/dist-packages/pdi/__init__.py(44): expose
  <string>(24): <module>

I also tried bytes(arr, 'utf-8'), I get the same error with type descriptor: S ...

In GitLab by @ksiero on Feb 24, 2021, 12:45
mentioned in commit c41607f0ae73549921ac5102bfe97e58f3286f3a

In GitLab by @ksiero on Feb 26, 2021, 11:16
mentioned in commit a3523346f547899dc59d196333e8ab844c9d7d0c

In GitLab by @ksiero on Feb 24, 2021, 16:16
mentioned in commit c2f740c26f2b649e328b7fc45b30b7bd4d8ce0f9

In GitLab by @ksiero on Feb 24, 2021, 13:54
mentioned in commit 3e48ad5bed1ea2e218ace1bd21aa51179076d1e8

In GitLab by @ksiero on Mar 5, 2021, 12:01
mentioned in commit 9c281efc9ccfe13fb38952b98e3bd32d8d9334f0

In GitLab by @ksiero on Mar 30, 2021, 09:51
unassigned @ksiero

In GitLab by @ksiero on Mar 5, 2021, 14:51
mentioned in commit 51e5ad924d56af5610551aab44d315deaa032ef4

In GitLab by @ksiero on Mar 5, 2021, 12:03
mentioned in commit 53490c2c284606a6a9d232d9c2cfb7b38b7a4516


Question: How to detect empty content in editor?

I want to check if the editor has no content or only whitespace characters before the user submits the form. I have tried the following but it does not work in some cases, is there an easier way?
export const getTextContent = async (editorState: EditorState) => {
  return new Promise<string>((resolve) => {
    editorState.read(() => {
      const root = $getRoot();
      const text = root.getTextContent();
      resolve(text);
    });
  });
};

export const isEmptyContent = async (editorState: EditorState) => {
  const data = editorState.toJSON();
  // @ts-ignore
  if (data.root.children?.length === 1 && !data.root.children[0].children?.length) {
    return true;
  }
  const textContent = await getTextContent(editorState);
  return !textContent?.trim();
};

look at $isRootTextContentEmpty, from @lexical/text

I'm gonna close this but please re-open if you can't find something workable from there. I opened a separate issue to revisit our emptiness semantics.


STORM-2236: Kafka Spout with manual partition management.

As the title says.

@hmcl You might want to take a look at this too

@liurenjie1024 once the review is complete and it incorporates the feedback from all the reviewers (I am going through the review now), please squash the commits into one commit, and make sure that this commit has the title of the JIRA it is addressing, as well as a summary of its contents.

+1 once the merge conflict is resolved. It would probably be good to add some unit tests of the new classes, but I think this can be merged without them.

@srdo Do I have the access to resolve the conflicts? Or it should be resolved by those who merge the code?

@liurenjie1024 You have access. You need to open your branch in the Git client you usually use and pull the latest commits from master on this repo (https://github.com/apache/storm.git). If you like you can use git pull --rebase to avoid a merge commit in the history. Once your branch is up to date, push it back to your fork. The PR will update automatically.
Here's an example of how it can be done from command line (I have a remote called "upstream" that points to this repository, if you don't have one like it use git remote add upstream https://github.com/apache/storm.git to add it).
git checkout STORM-2236
git pull upstream master --rebase //you'll get a merge conflict
git mergetool //resolve merge conflict (you can also use whatever git GUI you usually use)
git add <the files you resolved conflicts for>
git rebase --continue
//do the above 3 lines until the rebase is over
//At this point you should verify that the branch diff to master looks reasonable. If it looks fine, push to your branch with the following.
git push --force

@liurenjie1024 @srdo lets wait till all the pending comments are resolved.

Hi, all:
I've resolved the comments and when this patch can be merged into master? I've read trident source code and it seems that there exists some bugs there, and the bug fix may rely on codes in this branch, so I hope it can be merge ASAP.

Please copy the Apache license into all the new files. You can just grab it from the top of KafkaSpout.

@harshach @srdo @hmcl Any updates?

LGTM. +1

@liurenjie1024 thanks for the updates. Overall looks good to me. Can you squash your commits into one commit. +1 after that.

+1. Just one minor nit on a name. can you please squash the commits and put a little summary of the patch in the commit message. Thanks.

Hi, all:
All commits have been squashed into one.

Any update about the merge process of this patch?

@liurenjie1024 thanks for you patience. Merged into master. I would like to merge this into 1.x-branch as well. You are using FunctionalInterface and 1.x branch we are still on java 7 . Do you want to change that open another PR for 1.x-branch?

@harshach OK I'll submit a PR for that later.


Speech_Recognition

The videos are really great but let's assume that I'm a reader in the bus and I can't get my headphones out for the moment, but I would still like to hear and enjoy the content, I would suggest to add the relevant information from the video in the article as well maybe in a sectioned thing so that the user could only stick to the text in case of need.


Jcolomb metadatacreator

working on the medata creation documentation.

it also brings the new packrat stuff in this branch...


Publish to Hex.pm

The repository has already a package.exs file, there is a reason that has not been published in http://hex.pm yet?

I would also like to see this published to Hex.  I even forked it, just so I could publish the fork on Hex.
If you like, I can publish yamerl on Hex myself.  I didn't want to do so without your blessing!

it'd be nice if projects like Yamlelixir could install directly from Hex, instead of cloning this repo on GitHub :)

Hi! Sorry, I have little time to work on this project now that I'm no longer employed by Yakaz.
So I tried to publish the package this morning but I get a 500 HTTP error. Investigating...

Ok, it's fixed now:

I had to remove non-US-ASCII characters in src/yamerl.app.src (several "é" in my name).
I had rebar3_hex 1.19.0 installed. rebar3 plugins upgrade rebar3_hex said it upgraded the plugin to 2.5.1 but 1.19.0 was still the only one in ~/.cache/rebar3/plugins. After removing that copy and running rebar3 again, it fetched the expected 2.5.1 version.

The package is now published to Hex.pm.
Thank you for reminding me to finally do this! It was easier than I imagined.

Thanks!


v0.38.1 released on crates.io but not tagged

I added a comment on the last release commit but I'm not really sure of the visibility of it to the author or maintainers, so pinging @ivmarkov here just in case.
This commit changes the version of the package and seems to have been published to crates.io, but the git tag isn't present/hasn't been pushed here.

Yes, the CI pipeline was unfortunately half-broken at the time, and it managed to release, while the tagging (and docu failed). I'll just re-run it and release 0.38.2 shortly.

I tagged v.38.1 manually in the meantime (note: the published release was the next commit). Also, the current Publish action runs cargo publish in a build matrix which probably isn't the best idea.


Doesn't allow to track just in one Environment

I tried just setting up for a production environment in production.rb inside deploy.
But It keeps asking me the key for another environments.

cc @jamiepenney

Hey @pragmaticivan, sorry about taking so long to get back to you! Can you check this PR out? https://github.com/MindscapeHQ/capistrano-raygun-deployment/pull/3
I haven't had a chance to test it yet, does that look like it would do what you need?


Deprecated function used in Tutorial for rasterio data parsing

What is your issue?
in the documentation tutorial section
https://xarray.pydata.org/en/v2023.05.0/examples/visualization_gallery.html#Parsing-rasterio-geocoordinates
the xarray.open_rasterio function is used, that is deprecated since v0.20.0.
This causes the Jupiter notebook to throw an error message.
It might be nice to either remove the example or adapt it to provide a way to parse rasterio data without the function in question.

Hi @paradx , thanks for raising this.
The fix would be to use open_dataset(file, engine='rasterio') and ensure rioxarray is installed. (see https://github.com/pydata/xarray/issues/8003) Would be you be interested in submitting a small PR to fix this?

Solved in #7671.


APIFY_LOCAL_STORAGE_DIR Ignored

Running on Windows 8 and consuming the latest release through npm, it appears that the Apify.CheerioCrawler is ignoring the ENV configuration on which directory should be written to (APIFY_LOCAL_STORAGE_DIR).
In my case, it is - by default - currently writing to the current user directory.
While my code adheres very closely to that outlined in the Apify SDK Guide, I am executing the program using node <file_name>.
Note(s)

I am not using the Apify CLI to execute the program.
I have only worked with 0.16.*, I have not tried to reproduce this issue in previous releases.
I am - at this stage - only working with CheerioCrawler and have not verified that this issue exists with the others.

Are you setting the APIFY_LOCAL_STORAGE_DIR environment variable before including apify package? Can you ensure the value is set by running the following command?
console.log(process.env.APIFY_LOCAL_STORAGE_DIR);

Are you setting the APIFY_LOCAL_STORAGE_DIR environment variable before including apify package? Can you ensure the value is set by running the following command?
console.log(process.env.APIFY_LOCAL_STORAGE_DIR);

Will take a look on both. Give me a few minutes to check.

Hm. By removing (deleting) the local ./apify_storage directory and re-running the issue seems to have been resolved.
Did the above (the log and verify package order). Dunno what was happening.

Strange. Anyway, I'm glad it works now, let us know if you see any other problem.


[stable/telegraf] Added imagePullSecrets (see #15703)

Added imagePullSecrets to be able to pull docker image from private repository.
Signed-off-by: Nicolas Dascanio<EMAIL_ADDRESS>
What this PR does / why we need it:
Added imagePullSecrets to be able to pull docker image from private repository
Which issue this PR fixes
(optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged)

fixes #15703

Special notes for your reviewer:
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[x ] DCO signed
[x] Chart Version bumped
[ ] Variables are documented in the README.md
[x] Title of the PR starts with chart name (e.g. [stable/chart])

/assign @naseemkullah

/ok-to-test

/lgtm


resource link index reuse

when we reuse resource links, all columns are properly updated except the old source path was not updated

Fixed in #2007


⚠ Bofrost.lu has degraded performance

In 9e845ee, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 2211 ms

Resolved: Bofrost.lu performance has improved in 6cdb34e after 2 hours, 26 minutes.


fix: dark mode

Fixes

Fixes #2 by @zackkrida
Description

Fixes dark mode
Technical details

Change the svg colors when the theme is set to dark mode
Screenshots

Checklist

[x] My pull request has a descriptive title (not a vague title like Update index.md).
[x] My pull request targets the default branch of the repository (main or master).
[x] My commit messages follow best practices.
[x] My code follows the established code style of the repository.
[ ] I added tests for the changes I made (if applicable).
[ ] I added or updated documentation (if applicable).
[ ] I tried running the project locally and verified that there are no
visible errors.

Developer Certificate of Origin

Developer Certificate of Origin
Developer Certificate of Origin
Version 1.1

Copyright (C) 2004, 2006 The Linux Foundation and its contributors.
1 Letterman Drive
Suite D4700
San Francisco, CA, 94129

Everyone is permitted to copy and distribute verbatim copies of this
license document, but changing it is not allowed.

Developer's Certificate of Origin 1.1

By making a contribution to this project, I certify that:

(a) The contribution was created in whole or in part by me and I
    have the right to submit it under the open source license
    indicated in the file; or

(b) The contribution is based upon previous work that, to the best
    of my knowledge, is covered under an appropriate open source
    license and I have the right under that license to submit that
    work with modifications, whether created in whole or in part
    by me, under the same open source license (unless I am
    permitted to submit under a different license), as indicated
    in the file; or

(c) The contribution was provided directly to me by some other
    person who certified (a), (b) or (c) and I have not modified
    it.

(d) I understand and agree that this project and the contribution
    are public and that a record of the contribution (including all
    personal information I submit with it, including my sign-off) is
    maintained indefinitely and may be redistributed consistent with
    this project or the open source license(s) involved.

Hey @zackkrida, can you please guide me about where I am doing wrong. This PR is still in progress.

Hi @rajat2502, you had a small typo in the css class and one incorrect value in the filter that caused your changes to not work. I fixed and merged though! Excellent work and research; I didn't think this feature was going to be possible using filter but you figured it out 😸

Hey @zackkrida, thanks for helping me out!!!
It's great to know that I was able to help. I am surely looking forward to contributing more to the organization. I would love to suggest more features and fix more bugs.

I was thinking if we should add a download option there as well. Let me know if it's required or will be of some help then I will be happy to work on it as well.

Another thing that came into my mind while working on this issue is that there should be an option to change the height & width of the images (just like the font size). Otherwise, some images might overflow from the image-wrapper.

@rajat2502 Please, make issues for any of your ideas! I already did for one of them: https://github.com/creativecommons/og-image-generator/issues/4


Ability to share one photo

Is your feature request related to a problem? Please describe.
I like the ability so temporary share content from the gallery, but it would be nice if the feature was available for single items as well.
Describe the solution you'd like
A way to share single items the same way you can share entire albums.

Hi,
This would need a significant feature improvement. Currently only full directories are possible to share.
Dont think I will have time for this in the near future, but feel free to create a PR. I can try to give some pointers.

Definitly vote this up!
My common usecase of PG2 is to collect my photos and share them with my family and friends. Often I have a lot of photos of an event or situation, but only a few of them are really worth to share.
I don't like to duplicate them in another folder just for sharing. Instead I do mark them with tags to be able to search for them.
It would be really nice to create an album (e.g. "all 5-Stars pictures from folder A") and just share it.
Being able to share albums would also solve the request of @kardo22 as he could just create one for this specific file.
Regards


How do you tackle nested wheres

things like where foo=1 and (where bar=0 or baz=0) ?

How to use IN operation?
SELECT * FROM table WHERE id IN (1,2,3,4,5)

@amdrade
This can help you:
https://github.com/Respect/Relational/blob/master/tests/library/Respect/Relational/SqlTest.php#L72-L78

Thanks @felipecwb !!


Please Include an extenson list in each .code-workspace file

Issue Type: Feature Request
It would be very helpful if VSCode could Include an extenson list in each .code-workspace file, especially, when setting up a new environment or sync settings on a new computer.
VS Code version: Code 1.54.1 (f30a9b73e8ffc278e71575118b6bf568f04587c8, 2021-03-04T22:38:31.419Z)
OS version: Windows_NT x64 10.0.19042

What is this extension list is for? Can you please elaborate.

Thank you for your reply.  The extension list is a list of extensions enabled in the workspace. Let me explain in detail. Sometimes, when I create (or update) a .code-workspace file, I have to enable and disable the extensions manually, again and again, it is really time-consuming. What I would like to get is a function that

Generate a list of the enabled extensions in the .code-workspace file
When VSCode reads this file, the listed extensions will be enabled, and the un-listed extensions will be disabled automatically
The list can be a grouped list (with 'name': '' and 'items': []) to serve the same as the extension packs

It would be helpful when users set up a new workspace.

/duplicate
There is an existing issue already for this. Hence closing this.


1.4.16

1.4.16

Coverage increased (+0.2%) to 97.348% when pulling c7ff06d517838f262ec9691909414a2968b4f1f8 on development into 00abab512cc42366d635a2e0d656a8201e437800 on master.


Extension host terminated unexpectedly

code-server version: Latest 1.408-vsc1.32.0
OS Version: Linux VM, Chrome 73.0.3683.86

Description
Tired to open the developer console in chrome. Lost connection to the socket and keeps on trying again and again.

It looks like working fine in local setup.

Please find the attachments for more details.
Steps to Reproduce

Deploy your code-server in Gcloud
Try to open your developer console (CMD + Shift + c)

same issue

@yogeshvar Do you know what version of linux your VM is running?

We have a fix for the socket disconnects, but it looks like the disconnects here were a result of the terminal spawn error. Thanks for reporting!

I too am getting the same error,
I cannot load or download any extensions too. I'm currently self-hosting it

@yogeshvar Do you know what version of linux your VM is running?

Yes, I use Linux version 4.15.0-1028-gcp

Same issue here, I'm using Ubuntu 18.04 on Digital Ocean.

I noticed that this issue goes away upon specifying the data directory.
./code-server --data-dir /root/code-server works fine but without the --data-dir argument code-server crashes within a few minutes of launch.

Same issue here, I'm using Ubuntu 18.04 on Vultr. After sometime it just crashes.

Same issue i am using Digital Ocean, following is the error i am getting on terminal

INFO  http://localhost:80/
INFO
INFO  Starting shared process [1/5]...
WARN  stderr {"data":"(node:1260) [DEP0005] DeprecationWarning: Buffer() is deprecated due to security and usability issues. Please use the Buffer.alloc(), Buffer.allocUnsafe(), or Buffer.from() methods instead.\n"}
INFO  Starting shared process [2/5]... {"error":"Exited unexpectedly"}
WARN  stderr {"data":"(node:1274) [DEP0005] DeprecationWarning: Buffer() is deprecated due to security and usability issues. Please use the Buffer.alloc(), Buffer.allocUnsafe(), or Buffer.from() methods instead.\n"}
internal/child_process.js:366
throw errnoException(err, 'spawn');
^

Error: spawn ENOMEM

at ChildProcess.spawn (internal/child_process.js:366:11)
at Object.spawn [as _spawn] (child_process.js:538:9)
at t.async (/home/travis/build/codercom/code-server/packages/server/out/cli.js:504:34598)
at e.exports (/home/travis/build/codercom/code-server/packages/server/out/cli.js:504:34099)
at e (/home/travis/build/codercom/code-server/packages/server/out/cli.js:504:32722)
at Timeout.u [as _onTimeout] (/home/travis/build/codercom/code-server/packages/server/out/cli.js:504:33015)
at ontimeout (timers.js:436:11)
at tryOnTimeout (timers.js:300:5)
at listOnTimeout (timers.js:263:5)
at Timer.processTimers (timers.js:223:10)

root@test-ubuntu-1-1:~/myapp# packet_write_wait: Connection to <IP_ADDRESS> port 22: Broken pipe


Body text color is too low-contrast

The font color for the paragraph text is too light and therefore inaccessible for visually-impaired users. See the WebAIM Color contrast tool here: http://webaim.org/resources/contrastchecker/
Font color is #7a7c7c, on background #ffffff which fails the checker.
Solution would be to choose a darker grey that passes the tests on the color checker tool.

I don't have a copy of the build running, but I suggest #2d2e2e

want me to apply it to my build? In a new branch of course :)

Fixed with PR #114 - thanks @minamarkham!

Thanks @minamarkham!


Allow more responsive (climate) data exchange

closes #51
A functionality to process input climate data (*.nc) that covers greater or smaller time periods than 12 months still needs to be added.

Ready for another review.


Regarding RefAllele and Default Allele in gene-table.csv

Hi, this question is mostly for better understanding of pypgx data structure.
I tried to figure out the meaning of RefAllele, but it's not quite right actually. I thought RefAllele is the allele represented on the Human Reference genome (the fasta file), but GRCh37Default and GRCh38Default are already represented that. I also saw case where GRCh37Default and GRCh38Default flip (I think it's because of changes between GRCh37 and GRCh38), but I found 5 cases where GRCh37Default and GRCh38Default are the same, but they are different from RefAllele
Gene	RefAllele	GRCh37Default	GRCh38Default
ABCB1	*1          	*2                   	*2
NAT2	*4          	*12                   	*12
SLC22A2	*1          	*3                   	*3
UGT2B7	*1          	*2                   	*2
UGT2B15	*1          	*2                   	*2

I found this logic check to assign allele where no candidate is found, but still, I'm not fully understand the role of RefAllele
if ref_allele != default_allele and ref_allele not in candidates and default_allele not in candidates:
    candidates.append(default_allele)
if not candidates:
    candidates.append(default_allele)

Could you please explain what is RefAllele please? And how to assign it in gene-table? Thank you very much.

@NTNguyen13,
Good question! The RefAllele column in the gene-table.csv file gives you reference STAR allele for the given gene (some people refer it as "wild-type" allele, but reference allele is the preferred term).
For example, the CYP2D6 gene has *1 as reference allele and therefore RefAllele is *1. Now, if you look at the CYP2D6 sequence of GRCh37, you will find that it actually matches that of *2; therefore, the GRCh37Default column is *2. Finally, when you do the same for GRCh38, its CYP2D6 sequence matches that of *1 and so GRCh37Default is *1.
Let me know if you have more questions.
P.S. You will see that the NAT2 gene has *4 as reference allele instead of *1. That's for historical reasons. See the official NAT2 alleles page for more details (http://nat.mbg.duth.gr/Human NAT2 alleles_2013.htm).

Thank you for the quick response! So if I find a new gene to add to pypgx, I can assign  RefAllele based on literature review, GRCh37Default and GRCh38Default based on the human genome sequence, depended on assembly versions, am I right?

That's correct! Though I would strongly advise that if you have a PGx gene you'd like to add to PyPGx, please first open a new issue in the repository for discussion before making a PR 😄

yes, I'm definitely gonna follow that!


Split Datepicker component off to its own repository

The datepicker has been a big pain - the UI Bootstrap team has floated the idea of splitting it off to its own repository so it could get the maintenance that it deserves without bogging down the release cycle.
Thoughts?

Like discussed in last meeting I agree with splitting it out to it's own repo. That way we can channel efforts to the right components and keep a clear issue tracker for instance.
We also mentioned porting a full featured jQuery datepicker like the one from eterniecode to Angular without the jQuery dependency. I'm just tossing that in here as well.

I also agree with what was said during the meeting obviously the datepicker
has become a noose around this projects neck
On Mar 30, 2015 7:47 AM, "Robin van Baalen"<EMAIL_ADDRESS>wrote:

Like discussed in last meeting
https://github.com/angular-ui/bootstrap/wiki/Meeting-minutes#meeting-thursday-march-26th
I agree with splitting it out to it's own repo. That way we can channel
efforts to the right components and keep a clear issue tracker for
instance.
We also mentioned porting a full featured jQuery datepicker like the one
from eterniecode to Angular without the jQuery dependency. I'm just tossing
that in here as well.
--
Reply to this email directly or view it on GitHub
https://github.com/angular-ui/bootstrap/issues/3472#issuecomment-87707860
.

Interesting. I can definitely see even as a casual observer how this would be helpful. Almost half of the issues on the front page are datepicker-related.
datepicker also seems to draw more help/support tickets than its companions, so perhaps this would be an opportunity to address that (expanded docs, easier to browse issues, single meeting area for people knowledgeable in it). And smaller repos are less intimidating for people to begin contributing to.
That said there are the obvious issues of user confusion and module proliferation, and the possibility that datepicker won't stay as active as the parent project.

I'm actually waiting for the pickers to be spun off into their own modules so I can tackle them.

I agree, but would this new project take a dependency on ui-bootstrap?  I would hate to see a separate implementation to support what the tooltip and position service in ui-bootstrap are doing of the datepicker..

Good point @RobJacobs that is indeed one of the many issues we have to tackle before deciding what to do with it.
@wesleycho I took the liberty to add a checklist to your original post to keep track of possible issues involved with splitting off the datepicker project. Feel free to do as you please with it.

Dateparser shouldn't be a problem, since it's only used in the datepicker. Thus is should just follow the datepicker into it's own repository.

For the time being, we are not doing any split.


Can you fix this bug? Bad url for filebrowser browse.html

Tested
Django 1.11
django-ckeditor 5.4.0
Use his solution to download the browse.html template to make it work. it has been out for years..
Thanks

The template should be in the package.

The template is in the ckeditor_uploader application. Make sure you include it in your INSTALLED_APPS.


Async tests fail in browser

This is an issue with tests themselves, library code is fine.
http://rpominov.github.io/kefir/test/in-browser/SpecRunner.html

Looks related to this: https://github.com/mhevery/jasmine-node/issues/371

I'm fairly familiar with Karma for running unit tests in a browser. Any interest in using that as a test runner? We'd also be able to solve the "ES6 in tests" problem, as we can use a browserify preprocessor on the tests before running them.

I used Karma and I like how it works in watch mode and creates quick response loop. So maybe it would make sense to use Karma to improve dev experience. But while developing we probably don't want to run tests in browser, and I'm not sure Karma supports running in node (does it?).
As for this particular bug, I don't think Karma will help directly. I'm guessing that problem is that we use different versions of Jasmine on node and in browser. One supports async tests and one don't. Btw we do use browserify already to build browser tests (see https://github.com/rpominov/kefir/blob/master/Gruntfile.coffee#L29).
Also I kinda like the "Run tests in your browser" link. Will it be possible to create such bundle with Karma?
I think Karma is cool, but just don't know if it will solve our problems and won't create new ones. I think main problem with current setup is dev experience, it's takes too long to run the tests every time, a good watch mode would help. And this bug is kinda minor, probably not too hard to fix, I just didn't look at it to be honest.

But while developing we probably don't want to run tests in browser, and I'm not sure Karma supports running in node (does it?).

Not sure of that exactly, but it does support running in PhantomJS, which would be headless, so node-like (CLI-only).

Also I kinda like the "Run tests in your browser" link. Will it be possible to create such bundle with Karma?

No reason we couldn't continue to generate that bundle with grunt.
Up to you if you want it, but since I'm bouncing around this repo, figured I'd try and solve as many problems as possible :).

Did a bit of research. The tests fail because we use Jasmine 1.3 which doesn't support it('', done => ...) functionality. But in jasmine-node a modified version of Jasmine 1.3 is used, and it supports done. The  jasmine-node package seems abandoned and it doesn't support Jasmine 2.x, which makes it harder for us to upgrade to Jasmine 2.x. Seems like the most correct way of running Jasmine on node is to use jasmine package, but it doesn't seem to support preprocessing out of the box (we need it for coffee and es6+). Not sure what we should do yet, but two thoughts I have so far: 1) This particular issue is harder to fix than I thought 2) The test environment is very outdated and needs overall upgrade.
As for Karma, it doesn't support node, and if we add Karma it won't be able to replace any tools that we use (we still will need something to run on node, and to create "Run tests in your browser" bundle). So Karma will be just additional tool in the testing environment, not sure it's worth it.

It's actually a good thing you did this research, because I was considering switching from Mocha -> Jasmine for another project. Mocha supports preprocessing, but I don't know how much work it would be to switch over. The describe / it constructs are the same, but the assertion syntax would different, and I don't know how easily the test helpers will convert.
That's definitely a bunch of work though, so if you want completely overhaul the testing suite, it may be worth considering what you would want to do as if you were doing it from scratch.

Looking back over this convo, you don't need Karma to "support" node; you just run them in with node, and use Karma only when you're trying to run the tests in the browser.
I'm revisiting this issue because I'm currently refactoring my test suite for brookjs, and I'd love to be able to use the matchers / assertions you wrote here. Any interest in extracting them to a separate package? If so, couple other questions:

Do you want to stick w/ Jasmine? I'd get some benefits / harmony from switching to mocha / chai and extracting the matchers to a chai plugin, but I don't want to impose my requirements on you.
Do you want to leave them in coffeescript or can we "decaffeinate" them?

Oh, I've deleted the browser tests build, so it should be easier for any further improvements. #1 and #2 both sound good for me, feel free to open a PR!
Closing this issue.


Allow installation of plugins from archive (zip or tar.gz) files (Trac #28)

Migrated from http://microfluidics.utoronto.ca/ticket/28
{
    "status": "closed",
    "changetime": "2014-04-17T19:39:01",
    "description": "",
    "reporter": "cfobel",
    "cc": "",
    "resolution": "fixed",
    "_ts": "1397763541728826",
    "component": "microdrop",
    "summary": "Allow installation of plugins from archive (zip or tar.gz) files",
    "priority": "major",
    "keywords": "",
    "version": "0.1",
    "time": "2012-01-04T03:27:29",
    "milestone": "Microdrop 1.0",
    "owner": "cfobel",
    "type": "enhancement"
}

Trac update at 20120104T03:27:38: cfobel changed type from "defect" to "enhancement"

Trac update at 20120220T04:09:20: ryan changed summary from "allow installation of plugins from archive (zip or tar.gz) files" to "Allow installation of plugins from archive (zip or tar.gz) files"

Trac update at 20120309T05:44:30:

<EMAIL_ADDRESS>commented:

commit 381d12a46f1299d19fdf6c9c5711ebfd93ef9de7
Author: Christian Fobel<EMAIL_ADDRESS>Date:   Fri Mar 9 00:39:10 2012 -0500
Add support for installing a plugin from a zip

Fixes #32.

<EMAIL_ADDRESS>changed resolution from "" to "fixed"
<EMAIL_ADDRESS>changed status from "new" to "closed"

Trac update at 20120309T05:46:32: cfobel commented:

Currently, there is no way to uninstall a plugin from within microdrop.  Should there be?  This might cause problems since the plugin will likely be imported already.  There might be a way to 'un-import' the plugin...

Trac update at 20120309T06:00:34<EMAIL_ADDRESS>commented:

commit fcc31778c384ce9bf85dc6527ee0e007b2b436f5
Author: Christian Fobel<EMAIL_ADDRESS>Date:   Fri Mar 9 00:55:26 2012 -0500
Add support for installing plugins from tar.gz

See #32.

Trac update at 20120309T14:54:56:

ryan commented:

I'm reopening this ticket with a few proposed changes:

Installed plugins should show their version number in the plugin manager. This may require a change to the plugin API or just a requirement that plugins have a version member (similar to the requirement that they have a name).
If a user tries to install a plugin that already exists, instead of just skipping, check if the version numbers are different. If so, ask the user if they would like to replace the plugin.
It seems that when new plugins are installed they are not available until the program restarts. Either, import them if possible to make them available immediately, or prompt the user and restart the program.
We need to have an easy way to uninstall plugins. Since it's probably not easy/possible to unimport the python modules, could we save a list of plugins marked for removal and check this list on startup (before doing the import)?

ryan changed resolution from "fixed" to ""
ryan changed status from "closed" to "reopened"

Trac update at 20120312T20:41:09:

<EMAIL_ADDRESS>commented:

commit 53d2c9ba630993de58de131596fef61cd70fe49a
Merge: 206741e93da0fa7c27f7c1985e81f38885af05fb a27792f377ae3e0007a19ab5a44d21128d612044
Author: Christian Fobel<EMAIL_ADDRESS>Date:   Mon Mar 12 16:34:07 2012 -0400
Merge branch 'plugin_manager', Post-pone deleti...

Post-pone deletion of uninstalled plugins until next program launch.  This
prevents problems during deletion due to Windows file locking.  Fixes #32.

This change addresses proposed changes (1)-(4) from ticket #32.

<EMAIL_ADDRESS>changed resolution from "" to "fixed"
<EMAIL_ADDRESS>changed status from "reopened" to "closed"

Trac update at 20120312T20:41:11<EMAIL_ADDRESS>commented:

commit a27792f377ae3e0007a19ab5a44d21128d612044
Author: Christian Fobel<EMAIL_ADDRESS>Date:   Mon Mar 12 16:34:07 2012 -0400
Post-pone deletion of uninstalled plugins until...

Post-pone deletion of uninstalled plugins until next program launch.  This
prevents problems during deletion due to Windows file locking.  Fixes #32.

This change addresses proposed changes (1)-(4) from ticket #32.

Trac update at 20120312T20:41:13<EMAIL_ADDRESS>commented:

commit 40dc2d1df75a3069037aa67c21aa52377c1ab5b0
Author: Christian Fobel<EMAIL_ADDRESS>Date:   Fri Mar 9 16:35:38 2012 -0500
Add uninstall buttons to plugin manager, see #32

Add an uninstall button beside each plugin in the plugin manager.
Add version number label beside each plugin in the plugin manager.

These changes address proposed changes (1) and (4) from ticket #32.
Note that upgrading and uninstalling works now under Linux, but will likely
have to be updated for Windows, since Windows is more picky about deleting
files that might be in use.

Trac update at 20120312T20:41:15<EMAIL_ADDRESS>commented:

commit 4d00af01b0a3dd473fb2d72b0a3797dcbadaa452
Author: Christian Fobel<EMAIL_ADDRESS>Date:   Fri Mar 9 16:35:38 2012 -0500
Add version checking to plugin install, see #32

Read yaml encoded dictionary of the form:
    {'name': <plugin module name>, 'version': <string Version>}
from file named "properties.yml" in the plugin directory to obtain version of
plugin to be installed and from the currently installed version (if present).

The following is an example plugin directory layout an plugin archive:
    video_recorder.tgz
        \->video_recorder
            \->__init__.py
            \->microdrop
                \->__init__.py
            \->properties.yml

Example contents of the properties.yml file are shown below:
$ cat plugins/video_recorder/properties.yml
{name: video_recorder, version: '0.1'}

If a plugin is installed/upgraded inform the user that the application needs to
be restarted, then quit the application.

These changes address proposed change (2)-(3) from ticket #32.

Trac update at 20120313T23:00:10: cfobel commented:

Change the yes/no to use dialog from main_window_controller

Trac update at 20140417T19:39:01: ryan changed component from "core" to "microdrop"


Add a missing not-supported marker

7d208b73ad06ea4f31bedcca111a45d9561f06ec removed support not only for setting "exclude", but also for overriding "compilerOptions"

This is fixed in the new plugin now.


Don't use insertionSort when more than 300 markers are added

Uses the old 20% rule up to 300 markers and then always does the built-in (n log n) sort.
This is a big performance win for large datasets when eg. 19000 new markers could trigger an
insertionSort on a 100k dataset.
Measured on a real application: sorting time during app startup dropped from 70seconds(!!) to <200ms.
Both sorts are also now wrapped in the if (_nbChanges) check to avoid redundant sorts (even if they're fast and stable, I think this logic is simpler).

Thanks a lot for the pull-request. I will run some tests and release a new version later this week (I hope).


[MRTK Shell Parity] [Cursor] Asset: Cursor and Contextual icons

Overview
Adding new shell cursor and contextual icon assets & prefabs.
Changes

Fixes: #4417 #4418

Note
We need a code update to properly display contextual icons and new cursor assets. #4332
e.g.) Object with ManipulationHandler should display CursorMoveArrowsMove.prefab icon around the cursor
e.g.) Bounding Box's upper right corner's scale handle should display CursorMoveArrowsNortheastSouthwest icon around the cursor

Example of contextual icons displayed around the cursor

Cursor and icon prefabs

Cursor and icon assets - Mesh, materials

/azp run mrtk_pr

Do the example scenes need to be updated to have the new shell cursor?

@julenka Yes, this PR is for the asset only. I'll open a separate PR for the cursor prefab/animation updates. (blocked by incorrect Cursor state issue currently #4510 #4511 #4483)
Assets are coming from the shell.


tf.keras.models.load_model takes forever to load

System information

Have I written custom code: yes
OS Platform and Distribution: Windows 10 Version 1607
TensorFlow installed from: pip
TensorFlow version:

Git-Version: v1.12.1-8794-ge36271a61d
version: 1.15.0-dev20190814

CUDA/cuDNN version: 10.0/7
GPU model and memory: NVIDIA GTX 1050 Ti, 4096 MB

Describe the current behavior
The code below produces this output:
Save successfull
loading ...
W0815 15:02:11.705600 17844 deprecation.py:506] From C:\tensorflow_anduin\lib\site-packages\tensorflow_core\python\ops\init_ops.py:97: calling GlorotUniform.__init__ (from tensorflow.python.ops.init_ops) with dtype is deprecated and will be removed in a future version.
Instructions for updating:
Call initializer instance with the dtype argument instead of passing it to the constructor
W0815 15:02:11.706604 17844 deprecation.py:506] From C:\tensorflow_anduin\lib\site-packages\tensorflow_core\python\ops\init_ops.py:97: calling Zeros.__init__ (from tensorflow.python.ops.init_ops) with dtype is deprecated and will
be removed in a future version.
Instructions for updating:
Call initializer instance with the dtype argument instead of passing it to the constructor

even after 30 mins it doesn't print the expected output Load successfull
Describe the expected behavior
Succesful loading in under 1 min
Code to reproduce the issue
import tensorflow as tf
from tensorflow.keras.layers import (Dense, Input, Lambda)
from tensorflow.keras.models import Model, Sequential
from scipy import sparse
import numpy as np

def layer_lambda(input_x):
    sparse = input_x[0]
    dense = input_x[1]
    dense = tf.transpose(dense)
    y = tf.sparse.sparse_dense_matmul(sparse, dense)
    return tf.transpose(y)

dense_mat = np.eye(30, 30, dtype=np.float32)
sparse_mat = sparse.coo_matrix(dense_mat)
sparse_indices = np.mat([sparse_mat.row, sparse_mat.col]).transpose()
sparse_tensor = tf.SparseTensor(sparse_indices, sparse_mat.data, sparse_mat.shape)

model = Sequential()
model_input = Input(shape=(20,))
x = Dense(20)(model_input)
x = Dense(30)(x)
x = Lambda(layer_lambda, output_shape=(None, 30, 30))([sparse_tensor, x])
model = Model(model_input, x)

model.predict([[np.ones(20)]])

model.save("model.h5")

print("Save successfull")
print("loading ...")
model_load = tf.keras.models.load_model("model.h5", custom_objects={'layer_lambda': layer_lambda})

print("Load successfull")

This error occurred after fixing issue 31607.

I was able to replicate the issue with TF nightly version,kindly find the gist of colab.Thanks

Is there a possible fix to this bug?

Moving all of the sparse ops into the lambda solves this issue:
def layer_lambda(input_x):
  dense_mat = np.eye(30, 30, dtype=np.float32)
  sparse_mat = sparse.coo_matrix(dense_mat)
  sparse_indices = np.mat([sparse_mat.row, sparse_mat.col]).transpose()
  sparse_tensor = tf.SparseTensor(sparse_indices, sparse_mat.data, sparse_mat.shape)
  dense = input_x
  dense = tf.transpose(dense)
  y = tf.sparse.sparse_dense_matmul(sparse_tensor, dense)
  return tf.transpose(y)

model = Sequential()
model_input = Input(shape=(20,))
x = Dense(20)(model_input)
x = Dense(30)(x)
x = Lambda(layer_lambda, output_shape=(None, 30, 30))(x)
model = Model(model_input, x)