text:
[HPRO-555] Change id columns for awardees and organizations table to VARBINARY

HPRO-555
MySQL VARCHAR columns ignore trailing whitespace. This led to an issue in the test environment where the test awardee API returned two organizations with the same id but one with a trailing space, resulting in a duplicate key collision. This change treats the id as a binary column to avoid the MySQL VARCHAR padding behavior.
The side affect of this is that comparisons are now case sensitive, but the IDs should be case sensitive anyways. Also, sorting will be affected, but we are not sorting organizations or awardees by ID anywhere.
There are places where we join to the awardees.id or organizations.id column from columns that are still VARCHAR. We should determine whether we want to make this change across all tables.

Is there any need to fix existing data, or has that already been handled?

@stephenyeargin Not in HealthPro. I am unclear on the state of things in the upstream systems, but our sync will handle any changes that are made.

repo: all-of-us/healthpro


text:
v0.13.x 版本说明以及建议

Emby API 高级设置再次提速
在 v0.12.x 基础上，支持了 Emby 每个用户观看视频记录的查询，也就是可以做到，只要你的 Emby 服务器上任意一个人看过一部视频，就可以排除这个视频继续去搜索字幕。
使用的是 SkipWatched 字段控制，开启后，搜索效率会更高。文档见：高阶设置 - Emby API 支持
EmbyConfig:
    Url: http://192.168.50.x:8096
    ApiKey: 123456789
    LimitCount: 3000
    SkipWatched: false

修复 ass 字幕识别问题
同时修复了 ass 字幕某些情况下识别字幕语言类型错误的问题，这个问题可以直接导致视频字幕无法正确下载。

支持jellyfin吗

支持jellyfin吗

基础的字幕搜索功能是支持的。
没有搭建 jellyfin 的测试环境，它的 API 是否与 Emby 一致不清楚。估计大致是一样的，但是新的 API 未知。

source: gharchive/issue

license: MIT

license_type: permissive

license_source: github-api


text:
Link to old tutorial in README.md is broken

404 link to "old tutorial" in README.md:

https://github.com/allenai/allennlp/blob/master/tutorials/getting_started/semantic_parsing.md

It looks like that it's moved to the "docs" directory:

https://github.com/allenai/allennlp/blob/master/docs/tutorials/getting_started/semantic_parsing.md

is gone from there too now

authors: Mistobaan Tmr

repo: allenai/allennlp-semparse


text:
How to use Text2Sql model?

Is there any pre-trained model for Text to Sql? If not then how can I train the  model with allennlp-semparse, please guide me!

I have one at https://github.com/OhadRubin/SmBop


text:
Documentation/Description

Hi team,
It was mentioned in the ProcTHOR paper that the dataset .json files would be released as open source, when can we expect it?
If not, is there a documentation of the toolset to create a dataset using ProcTHOR? I'd really like to know if there is one, as it's a bit confusing to deal with the raw code right now.

The dataset is released :)
Please see the website for a demo on how to use it: https://colab.research.google.com/drive/1Il6TqmRXOkzYMIEaOU9e4-uTDTIb5Q78
The JSON files of the houses are available here: https://github.com/allenai/procthor-10k

authors: mattdeitke sarthakchittawar

repo: allenai/procthor


text:
How to visualize named entities in custom colors

There's an options in Spacy which allows us to use custom colors for named entity visualization. I'm trying to use the same options in scispacy for the named entities. I simply created two lists of entities and randomly generated colors and put them in options dictionary like the following:
options = {"ents": entities, "colors": colors}
Where entities is a list of NEs in scispacy NER models and colors is a list of the same size. But using such an option in either displacy.serve or displacy.render (for jupyter) does not work. I'm using the options like the following:
displacy.serve(doc, style="ent", options=options)
I wonder if using the color option only works for predefined named entities in the Spacy or there's something wrong with the way I'm using the option?

colors should be a dictionary mapping {"TAG": "color"}, not a list, see here:
https://spacy.io/usage/visualizers#ent

colors should be a dictionary mapping {"TAG": "color"}, not a list, see here:
https://spacy.io/usage/visualizers#ent

Sorry, I should correct myself, colors is a dictionary with the same names in my entities list as keys. And, still it does not work.

Can you provide an actual code snippet, because doing this with custom colours has actually worked for me before, so I know that it works :)
The entities that scispacy detects are called ENT, do you have that in your colour dict?

I just realized what the problem was. When I use the full spaCy pipeline model like the following:
nlp = spacy.load("en_core_sci_md")
Named entities won't be visualized, but when I use a specific NER model, like the following:
nlp = spacy.load("en_ner_bionlp13cg_md")
I do see the visualized named entities.
then I think my question now is, if I want to visualize all the available named entities in scispacy (27 named entities,) should I call the NER models separately? Isn't there a way to use the full pipeline model and visualize all the named entities at once?

Again, if you provide me with a code snippet of what you did for the en_core_sci_md model, I can help - I know for a fact that it is possible to visualize entities using this model.
In terms of combining the other models together, this is a bit tricky in Spacy as the NER model is stateful (meaning that if you run more than one NER model in a pipeline it doesn't work properly). So unfortunately that might be a bit tricky, sorry!

This is the exact code I'm using:
import scispacy
import spacy
from spacy import displacy

# nlp = spacy.load("en_ner_jnlpba_md")
nlp = spacy.load("en_core_sci_md")

text = """The purpose of our study was to learn the distribution characteristics of cancer stem cell markers (CD24, CD44) in invasive carcinomas with different grade and molecular subtype. For research was used 1324 postoperative breast cancer samples, from which were selected 393 patient with invasive ductal carcinoma samples examined 2008-2012 in Laboratory of "Pathgeo Union of Pathologist" is and N.Kipshidze Central University Hospital. The age range is between 23-73 year. For all cases were performed immunohistochemical study using ER, PR, Her2, Ki67, CK5- molecular markers (Leica Microsystems). For identify cancer stem cells mononuclear antibodies CD24 (BIOCARE MEDICAL, CD44 - Clone 156-3C11; CD24 - Clone SN3b) were used. Association of CD44/CD24 expression in different subtypes of cells, between clinicopathological parameters and different biological characteristics were performed by Pearson correlation and usind X2 tests. Obtained quantitative statistical analyses were performed by using SPSS V.19.0 program. Statistically significant were considered 95% of confidence interval. The data shows, that towards G1-G3, amount of CD44 positive cases increased twice. CD44 positive cases are evenly distributed between Luminal A, Luminal B, HER2+, triple negative basal like cell subtypes and in significantly less (4,8 times) in Her2+ cases. Maximum amount of CD44 negative cases is shown in Luminal A subtype, which could be possible cause of better prognosis and high sensitivity for chemotherapy. For one's part such aggressive subtypes of breast cancer as Luminal B and basal like cell type, are characterized by CD44 positive and antigen high expression, which can be reason of aggressive nature of this types and also reason of chemotherapy resistance. As well as amount of CD24 positive cases according to malignancy degree, also antigen expression features does not show any type of correlation between malignancy degree and CD24 positivity or with CD24 expression features, or presence of stem cells. That can be the reason of tumor aggressivity and chemoresistance. exceptions are Her2 positive tumors because they have different base of carcinogenesis."""

doc = nlp(text)
options = get_entity_options()
displacy.render(doc, style='ent', options=options)

Where get_entity_options() is a method I wrote for getting the color options like the following (everybody, feel free to use it if you find it useful):
import random

def get_entity_options(random_colors=False):
    """
    generating color options for visualizing the named entities
    """
    def color_generator(number_of_colors):
        color = ["#"+''.join([random.choice('0123456789ABCDEF') for j in range(6)])
                 for i in range(number_of_colors)]
        return color

    entities = ["GGP", "SO", "TAXON", "CHEBI", "GO", "CL",
                "DNA", "CELL_TYPE", "CELL_LINE", "RNA", "PROTEIN",
                "DISEASE", "CHEMICAL",
                "CANCER", "ORGAN", "TISSUE", "ORGANISM", "CELL", "AMINO_ACID", "GENE_OR_GENE_PRODUCT", "SIMPLE_CHEMICAL", "ANATOMICAL_SYSTEM", "IMMATERIAL_ANATOMICAL_ENTITY", "MULTI-TISSUE_STRUCTURE", "DEVELOPING_ANATOMICAL_STRUCTURE", "ORGANISM_SUBDIVISION", "CELLULAR_COMPONENT"]

    colors = {"ENT":"#E8DAEF"}

    if random_colors:
        color = color_generator(len(entities))
        for i in range(len(entities)):
            colors[entities[i]] = color[i]
    else:
        entities_cat_1 = {"GGP":"#F9E79F", "SO":"#F7DC6F", "TAXON":"#F4D03F", "CHEBI":"#FAD7A0", "GO":"#F8C471", "CL":"#F5B041"}
        entities_cat_2 = {"DNA":"#82E0AA", "CELL_TYPE":"#AED6F1", "CELL_LINE":"#E8DAEF", "RNA":"#82E0AA", "PROTEIN":"#82E0AA"}
        entities_cat_3 = {"DISEASE":"#D7BDE2", "CHEMICAL":"#D2B4DE"}
        entities_cat_4 = {"CANCER":"#ABEBC6", "ORGAN":"#82E0AA", "TISSUE":"#A9DFBF", "ORGANISM":"#A2D9CE", "CELL":"#76D7C4", "AMINO_ACID":"#85C1E9", "GENE_OR_GENE_PRODUCT":"#AED6F1", "SIMPLE_CHEMICAL":"#76D7C4", "ANATOMICAL_SYSTEM":"#82E0AA", "IMMATERIAL_ANATOMICAL_ENTITY":"#A2D9CE", "MULTI-TISSUE_STRUCTURE":"#85C1E9", "DEVELOPING_ANATOMICAL_STRUCTURE":"#A9DFBF", "ORGANISM_SUBDIVISION":"#58D68D", "CELLULAR_COMPONENT":"#7FB3D5"}

        entities_cats = [entities_cat_1, entities_cat_2, entities_cat_3, entities_cat_4]
        for item in entities_cats:
            colors = {**colors, **item}

    options = {"ents": entities, "colors": colors}

    return options

Using the full model, I can't see any visualization, but when I switch to a specific NER model I do see the visualization.

@phosseini : VERY cool, thank you!  I added your code (for my own use / tests) as a method, giving the following results!  :-)

entity_options.py
## Source: https://github.com/allenai/scispacy/issues/141#issuecomment-518274586
## Author: https://github.com/phosseini
## File: /mnt/Vancouver/apps/spacy/entity_options.py
## Use (Python 3.7 venv):
##     import entity_options
##     from entity_options import get_entity_options
##     displacy.serve(doc, style="ent", options=get_entity_options(random_colors=True))

import random

def get_entity_options(random_colors=False):
    """ generating color options for visualizing the named entities """

    def color_generator(number_of_colors):
        color = ["#"+''.join([random.choice('0123456789ABCDEF') for j in range(6)]) for i in range(number_of_colors)]
        return color

    entities = ["GGP", "SO", "TAXON", "CHEBI", "GO", "CL", "DNA", "CELL_TYPE", "CELL_LINE", "RNA", "PROTEIN", \
                "DISEASE", "CHEMICAL", "CANCER", "ORGAN", "TISSUE", "ORGANISM", "CELL", "AMINO_ACID", \
                "GENE_OR_GENE_PRODUCT", "SIMPLE_CHEMICAL", "ANATOMICAL_SYSTEM", "IMMATERIAL_ANATOMICAL_ENTITY", \
                "MULTI-TISSUE_STRUCTURE", "DEVELOPING_ANATOMICAL_STRUCTURE", "ORGANISM_SUBDIVISION", "CELLULAR_COMPONENT"]

    colors = {"ENT":"#E8DAEF"}

    if random_colors:
        color = color_generator(len(entities))
        for i in range(len(entities)):
            colors[entities[i]] = color[i]
    else:
        entities_cat_1 = {"GGP":"#F9E79F", "SO":"#F7DC6F", "TAXON":"#F4D03F", "CHEBI":"#FAD7A0", "GO":"#F8C471", "CL":"#F5B041"}
        entities_cat_2 = {"DNA":"#82E0AA", "CELL_TYPE":"#AED6F1", "CELL_LINE":"#E8DAEF", "RNA":"#82E0AA", "PROTEIN":"#82E0AA"}
        entities_cat_3 = {"DISEASE":"#D7BDE2", "CHEMICAL":"#D2B4DE"}
        entities_cat_4 = {"CANCER":"#ABEBC6", "ORGAN":"#82E0AA", "TISSUE":"#A9DFBF", "ORGANISM":"#A2D9CE", "CELL":"#76D7C4", \
                          "AMINO_ACID":"#85C1E9", "GENE_OR_GENE_PRODUCT":"#AED6F1", "SIMPLE_CHEMICAL":"#76D7C4", "ANATOMICAL_SYSTEM":"#82E0AA", \
                          "IMMATERIAL_ANATOMICAL_ENTITY":"#A2D9CE", "MULTI-TISSUE_STRUCTURE":"#85C1E9", "DEVELOPING_ANATOMICAL_STRUCTURE":"#A9DFBF", \
                          "ORGANISM_SUBDIVISION":"#58D68D", "CELLULAR_COMPONENT":"#7FB3D5"}

        entities_cats = [entities_cat_1, entities_cat_2, entities_cat_3, entities_cat_4]
        for item in entities_cats:
            colors = {**colors, **item}

    options = {"ents": entities, "colors": colors}
    # print(options)
    return options

Python 3.7 venv
(py3.7) [victoria@victoria spacy]$ date; pwd; ls -l

  Tue 26 Nov 2019 01:45:28 PM PST
  /mnt/Vancouver/apps/spacy
  total 20
  -rw-r--r-- 1 victoria victoria 2287 Nov 26 13:42 entity_options.py
  drwxr-xr-x 2 victoria victoria 4096 Nov 26 13:34 __pycache__
  -rw------- 1 victoria victoria 3560 Nov 26 12:02 readme-victoria-spacy.txt
  drwxr-xr-x 3 victoria victoria 4096 Nov 19 11:41 scispacy
  -rw-r--r-- 1 victoria victoria 2624 Nov 26 11:59 spacy_srl.py

(py3.7) [victoria@victoria spacy]$ pip install https://s3-us-west-2.amazonaws.com/ai2-s2-scispacy/releases/v0.2.4/en_ner_craft_md-0.2.4.tar.gz
  ...
  Successfully installed blis-0.4.1 catalogue-0.0.8 en-ner-craft-md-0.2.4 preshed-3.0.2 spacy-2.2.3 thinc-7.3.1

(py3.7) [victoria@victoria spacy]$ env | grep -i virtual
VIRTUAL_ENV=/home/victoria/venv/py3.7

(py3.7) [victoria@victoria spacy]$ python --version
Python 3.7.4

(py3.7) [victoria@victoria spacy]$ python
  Python 3.7.4 (default, Nov 20 2019, 11:36:53)
  [GCC 9.2.0] on linux
  Type "help", "copyright", "credits" or "license" for more information.

>>> import spacy
>>> from spacy import displacy

>>> text = "26902145. Breast cancer susceptibility gene 1 (BRCA1) is a tumor suppressor protein that functions to maintain genomic stability through critical roles in DNA repair, cell-cycle arrest, and transcriptional control. The androgen receptor (AR) is expressed in more than 70% of breast cancers and has been implicated in breast cancer pathogenesis. However, little is known about the role of BRCA1 in AR-mediated cell proliferation in human breast cancer. Here, we report that a high expression of AR in breast cancer patients was associated with shorter overall survival (OS) using a tissue microarray with 149 non-metastatic breast cancer patient samples. We reveal that overexpression of BRCA1 significantly inhibited expression of AR through activation of SIRT1 in breast cancer cells. Meanwhile, SIRT1 induction or treatment with a SIRT1 agonist, resveratrol, inhibits AR-stimulated proliferation. Importantly, this mechanism is manifested in breast cancer patient samples and TCGA database, which showed that low SIRT1 gene expression in tumor tissues compared with normal adjacent tissues predicts poor prognosis in patients with breast cancer. Taken together, our findings suggest that BRCA1 attenuates AR-stimulated proliferation of breast cancer cells via SIRT1 mediated pathway. | 30714292. Breast cancer susceptibility gene 1 (BRCA1) has been implicated in modulating metabolism via transcriptional regulation. However, direct metabolic targets of BRCA1 and the underlying regulatory mechanisms are still unknown. Here, we identified several metabolic genes, including the gene which encodes glutamate‐oxaloacetate transaminase 2 (GOT2), a key enzyme for aspartate biosynthesis, which are repressed by BRCA1. We report that BRCA1 forms a co‐repressor complex with ZBRK1 that coordinately represses GOT 2 expression via a ZBRK1 recognition element in the promoter of GOT2. Impairment of this complex results in upregulation of GOT2, which in turn increases aspartate and alpha ketoglutarate production, leading to rapid cell proliferation of breast cancer cells. Importantly, we found that GOT2 can serve as an independent prognostic factor for overall survival and disease‐free survival of patients with breast cancer, especially triple‐negative breast cancer. Interestingly, we also demonstrated that GOT2 overexpression sensitized breast cancer cells to methotrexate, suggesting a promising precision therapeutic strategy for breast cancer treatment. In summary, our findings reveal that BRCA1 modulates aspartate biosynthesis through transcriptional repression of GOT2, and provides a biological basis for treatment choices in breast cancer. | BRCA1/2. BRCA1 and BRCA2 (BRCA1/2) are human genes that produce tumor suppressor proteins."

>>> nlp = spacy.load("en_ner_craft_md")
>>> doc = nlp(text)

>>> import entity_options
>>> from entity_options import get_entity_options

>>> get_entity_options()                               ## default: (random_colors=False)
{'colors': {'AMINO_ACID': '#85C1E9',
            'ANATOMICAL_SYSTEM': '#82E0AA',
            'CANCER': '#ABEBC6',
            'CELL': '#76D7C4',
            'CELLULAR_COMPONENT': '#7FB3D5',
            'CELL_LINE': '#E8DAEF',
            'CELL_TYPE': '#AED6F1',
            'CHEBI': '#FAD7A0',
            'CHEMICAL': '#D2B4DE',
            'CL': '#F5B041',
            'DEVELOPING_ANATOMICAL_STRUCTURE': '#A9DFBF',
            'DISEASE': '#D7BDE2',
            'DNA': '#82E0AA',
            'ENT': '#E8DAEF',
            'GENE_OR_GENE_PRODUCT': '#AED6F1',
            'GGP': '#F9E79F',
            'GO': '#F8C471',
            'IMMATERIAL_ANATOMICAL_ENTITY': '#A2D9CE',
            'MULTI-TISSUE_STRUCTURE': '#85C1E9',
            'ORGAN': '#82E0AA',
            'ORGANISM': '#A2D9CE',
            'ORGANISM_SUBDIVISION': '#58D68D',
            'PROTEIN': '#82E0AA',
            'RNA': '#82E0AA',
            'SIMPLE_CHEMICAL': '#76D7C4',
            'SO': '#F7DC6F',
            'TAXON': '#F4D03F',
            'TISSUE': '#A9DFBF'},
 'ents': ['GGP',
          'SO',
          'TAXON',
          'CHEBI',
          'GO',
          'CL',
          'DNA',
          'CELL_TYPE',
          'CELL_LINE',
          'RNA',
          'PROTEIN',
          'DISEASE',
          'CHEMICAL',
          'CANCER',
          'ORGAN',
          'TISSUE',
          'ORGANISM',
          'CELL',
          'AMINO_ACID',
          'GENE_OR_GENE_PRODUCT',
          'SIMPLE_CHEMICAL',
          'ANATOMICAL_SYSTEM',
          'IMMATERIAL_ANATOMICAL_ENTITY',
          'MULTI-TISSUE_STRUCTURE',
          'DEVELOPING_ANATOMICAL_STRUCTURE',
          'ORGANISM_SUBDIVISION',
          'CELLULAR_COMPONENT']}

>>> get_entity_options(random_colors=True)
{'colors': {'AMINO_ACID': '#30CBF7',
            'ANATOMICAL_SYSTEM': '#6DF980',
            'CANCER': '#1AE0F9',
            'CELL': '#5813C7',
            'CELLULAR_COMPONENT': '#0D350E',
            'CELL_LINE': '#1AA436',
            'CELL_TYPE': '#F837CC',
            'CHEBI': '#54B69E',
            'CHEMICAL': '#BADCA1',
            'CL': '#D845FB',
            'DEVELOPING_ANATOMICAL_STRUCTURE': '#0D9CB4',
            'DISEASE': '#78A2E5',
            'DNA': '#CAD406',
            'ENT': '#E8DAEF',
            'GENE_OR_GENE_PRODUCT': '#EC2144',
            'GGP': '#A6AA7D',
            'GO': '#8312F0',
            'IMMATERIAL_ANATOMICAL_ENTITY': '#F7E433',
            'MULTI-TISSUE_STRUCTURE': '#221891',
            'ORGAN': '#786BC0',
            'ORGANISM': '#43534C',
            'ORGANISM_SUBDIVISION': '#B6F342',
            'PROTEIN': '#4454D9',
            'RNA': '#64C158',
            'SIMPLE_CHEMICAL': '#F8616A',
            'SO': '#344E4D',
            'TAXON': '#63B69D',
            'TISSUE': '#0DE67C'},
 'ents': ['GGP',
          'SO',
          'TAXON',
          'CHEBI',
          'GO',
          'CL',
          'DNA',
          'CELL_TYPE',
          'CELL_LINE',
          'RNA',
          'PROTEIN',
          'DISEASE',
          'CHEMICAL',
          'CANCER',
          'ORGAN',
          'TISSUE',
          'ORGANISM',
          'CELL',
          'AMINO_ACID',
          'GENE_OR_GENE_PRODUCT',
          'SIMPLE_CHEMICAL',
          'ANATOMICAL_SYSTEM',
          'IMMATERIAL_ANATOMICAL_ENTITY',
          'MULTI-TISSUE_STRUCTURE',
          'DEVELOPING_ANATOMICAL_STRUCTURE',
          'ORGANISM_SUBDIVISION',
          'CELLULAR_COMPONENT']}

## default: get_entity_options(random_colors=False)
## displacy.serve(doc, style="ent", options=get_entity_options())

>>> displacy.serve(doc, style="ent", options=get_entity_options(random_colors=True))

  Using the 'ent' visualizer
  Serving on http://0.0.0.0:5000 ...
  127.0.0.1 -- -- [26/Nov/2019 13:43:47] "GET / HTTP/1.1" 200 20529

Screenshots
random colors = False:

random colors = True:

@victoriastuart Nice comparison! I tried to define colors in a way so that different categories are distinguishable and also entities in the same category are in the same color range. Definitely it can be improved but better than random colors I think :)

Again, very appreciative of your code: so much fun to try and also quite useful I think!  :-)

how to save this result into a data frame?

authors: DeNeutoy kirandiquery phosseini victoriastuart

repo: allenai/scispacy


text:
🛑 Simply Greek, Hershey is down

In 93d72b0, Simply Greek, Hershey (https://simplygreekhershey.com) was down:

HTTP code: 406
Response time: 118 ms

Resolved: Simply Greek, Hershey is back up in 85aa3f6 after 12 minutes.

repo: alliancewebteam/uptime


text:
AM_INIT_AUTOMAKE: command not found

I'm on Centos 7
I'm trying to compile for on-disk:

# ./configure --enable-utf8 --enable-tcb=btree
./configure: line 2319: AM_INIT_AUTOMAKE: command not found
checking for gcc... gcc
checking whether the C compiler works... yes
checking for C compiler default output file name... a.out
checking for suffix of executables...
checking whether we are cross compiling... no
checking for suffix of object files... o
checking whether we are using the GNU C compiler... yes
checking whether gcc accepts -g... yes
checking for gcc option to accept ISO C89... none needed
./configure: line 3120: AM_PROG_CC_C_O: command not found
./configure: line 3122: syntax error near unexpected token `external'
./configure: line 3122: `AM_GNU_GETTEXT(external)'

I installed tokyo cabinet
but am getting the above.  Do you know what is wrong? OS thing?

You probably need to install "Development Tools" and  gettext-devel

I ran
yum groupinstall "Development tools"
yum install gettext-devel
Same error as before

I don't have a CentOS machine to test this out but based on the errors you are getting, it looks like you are missing some development tools such as automake autoconf, gettext. Did  you build from development or did you download v1.3?

v1.3
It seems I don't need the on-disk though, so I'll close this issue.

authors: allinurl javadan

repo: allinurl/goaccess

license: mit

license_source: bigquery


text:
Display stats from an unclosed STDIN

Hello,
Apparently goaccess already supports displaying stats from cat/tail and if you pass in a file, it will update the stats as new lines are added to the file.
Currently however it is not possible to do something such as tail -f | goaccess -a -p goaccess.conf as it will stay in parsing mode indefinitely.
At this point I'd like to ask, since you already support live updating of stats, is the reason that stats are not displayed as parsing is done is that it is a lot less efficient (a lot less lines parsed/second)? If it has little impact, it would be nice to see the stats displayed as parsing occurs.
Second, it would be nice to support live stats through something such as tail -f | goaccess -a -p goaccess.conf, which would allow users to quickly diagnose issues by monitoring logs since goaccess was started.

Hello,
Good question, currently the live or tail implementation makes use of fseeko to change the file position and achieve a 'follow mode'. This was shipped in one of the early version of goaccess. As you said, being able to read directly from stdin  would be great, especially for the use-case you mentioned above.
I'll go back and take a look at this again to see what can be done and how feasible  it is. I appreciate you bringing this up.

I've pushed a commit that enables the ability to output live stats from a stream or an unclosed STDIN, i.e., tail -f. This allows to do:
tail -f access.log | goaccess --log-format=COMBINED -

or even
tail -f access.log | goaccess --log-format=COMBINED -o report.html --real-time-html -

It also opens the possibility for live data filtering from the pipe such as:
tail -f access.log | grep -i 'firefox' | goaccess --log-format=COMBINED -

One thing to note is that  tail -f will keep the pipe opened even when goaccess already has exited. For instance, tail -f syslog | grep -q 'cron' tail will exit on SIGPIPE, however it will only get SIGPIPE until an extra byte is written to the tail'd file. SIGTERM, SIGINT should close tail fine though.
Feel free to build from master to test this out, otherwise it will be pushed out in the upcoming release. Thanks.

authors: allinurl tomzx


text:
php api doesn't have such annotation as TestCaseID

Hi guys,
Please add such Annotation type to allure-php-adapter-api(not php unit adapter) as java api has
ru.yandex.qatools.allure.annotations.TestCaseId

Hello, feature requests about PHP implementation should be reported in appropriate repository: https://github.com/allure-framework/allure-php-adapter-api
Also, if you have request for some specific framework, there are following repositories:

Codeception
PHPUnit

sorry for that, please close my issue

authors: Gordiievskyi just-boris

repo: allure-framework/allure-core


text:
16 bit parallel mode incorrectly implemented

The driver is currently using write_word to send associated command data, however this is implemented as
    // Writes a data word to the display.
    fn write_word(&mut self, value: u16) -> Result<(), Error<PinE>> {
        use display_interface::DataFormat::U16;
        self.di
            .send_data(U16(&[value.to_be()]))
            .map_err(|_| Error::DisplayError)
    }

which means it'll try to send data as 16 bit value, however the only display commands accepting 16 bit values are RAMWR and RAMWRC, i.e. the ones used to send pixel data. Everything else must be sent as multiple 8bit values, even in 16bit parallel mode.

I might be wrong but I think this is actually OK. write_word is only used for RAMWR and CASET/RASET but those expect "16 bit" values too...

No, because they always need to be sent in two separate 8 bit values but if you specifiy a 16 bit Datasize, a 16 bit DI impl will not split it up but send it as a single 16 Bit value.

You can clearly see that the column D8-15 is empty and there're 4 rows with 8 bit transfers.

Ah yes you're right. I'll switch it to U8

Fixed

Can we get a release? ;)

We did, v0.4.2

authors: almindor therealprof


text:
use case: metronome

I have yet to find a flutter package that can keep time accurately enough for use as a metronome. I think we should add such an example to flutter_soloud to both ensure the package maintains best-in-industry playback scheduling accuracy, but to also show people what techniques are needed to play with accuracy

Hi @lukepighetti, thanks for your feedback!
Some brief examples are expected soon. We will open an issue to ask which one could be more relevant for the devs.
Meanwhile, I would refer you to my reply of #99 regarding this issue.

do you have any immediate recommendations for how to get the best metronome like performance out of flutter_soloud?

do you have any immediate recommendations for how to get the best metronome like performance out of flutter_soloud?

IIRC a good choice is to set a buffer of 512 (instead of the default 2048) here.
Of course, there is some troubles because you need to clone this repo locally and point it to your app's pubspec.yaml after modifying player.cpp.
I also think you could try using the SoLoud.play([your-short-metronome-sound]) inside a Timer.periodic().
Let me know!

I tried to make a metronome using a buffer of 512 and since the web platform is almost ready, I made the example available here.

I tried something like this

void main() async {
  WidgetsFlutterBinding.ensureInitialized();

  /// Initialize the player
  await SoLoud.instance.init();

  runApp(
    const MaterialApp(
      home: Metronome(),
    ),
  );
}

class Metronome extends StatefulWidget {
  const Metronome({super.key});

  @override
  State<Metronome> createState() => _MetronomeState();
}

class _MetronomeState extends State<Metronome> {
  /// delay between ticks.
  final delay = ValueNotifier<int>(100);

  /// duration of the tick sound.
  final tickDurationMs = ValueNotifier<int>(45);

  Timer? timer;
  AudioSource? tick;
  SoundHandle? tickHandle;

  @override
  void initState() {
    super.initState();
    SoLoud.instance
        .loadWaveform(WaveForm.fSquare, true, 0.25, 1)
        .then((value) async {
      final octave = 2;
      final noteIndex = 0;
      final startingFreq = 55.0 * (pow(2, (12 * octave) / 12));
      final freq = startingFreq * (pow(2, noteIndex / 12));

      tick = value;

      /// start playing the tick in a paused state, so it can be
      /// unpaused/paused in the `Timer` callback.
      tickHandle = await SoLoud.instance.play(tick!, paused: true);

      SoLoud.instance.setWaveformFreq(tick!, freq);
    });
  }

  @override
  Widget build(BuildContext context) {
    return Scaffold(
      body: Stack(
        children: [
          Padding(
            padding: const EdgeInsets.all(16),
            child: Center(
              child: Column(
                mainAxisSize: MainAxisSize.min,
                children: [
                  const Text('Metronome example',
                      textScaler: TextScaler.linear(3)),
                  const SizedBox(height: 32),
                  ValueListenableBuilder<int>(
                    valueListenable: delay,
                    builder: (_, ms, __) {
                      return Row(
                        children: [
                          Text('delay ms: $ms  BPM: ${60000 ~/ ms}'),
                          Expanded(
                            child: Slider.adaptive(
                              min: 15,
                              max: 500,
                              value: ms.toDouble(),
                              onChanged: (value) {
                                delay.value = value.toInt();
                                start();
                              },
                            ),
                          ),
                        ],
                      );
                    },
                  ),
                  ValueListenableBuilder<int>(
                    valueListenable: tickDurationMs,
                    builder: (_, duration, __) {
                      return Row(
                        children: [
                          Text('tic duration ms $duration'),
                          Expanded(
                            child: Slider.adaptive(
                              min: 15,
                              max: 500,
                              value: duration.toDouble(),
                              onChanged: (value) {
                                tickDurationMs.value = value.toInt();
                                start();
                              },
                            ),
                          ),
                        ],
                      );
                    },
                  ),
                ],
              ),
            ),
          ),
        ],
      ),
    );
  }

  void start() {
    timer?.cancel();
    timer = Timer.periodic(Duration(milliseconds: delay.value), (_) {
      if (tickHandle != null) {
        SoLoud.instance.setPause(tickHandle!, false);
        SoLoud.instance.schedulePause(
          tickHandle!,
          Duration(milliseconds: tickDurationMs.value),
        );
      }
    });
  }
}

Soon there will be there will be the option to choose the buffer size while initializing the player.

Some things I noticed using this on iOS simulator

400 bpm clearly shows drift issues
tic duration cannot be less than 47 ms

authors: alnitak lukepighetti


text:
Update joda-time to 2.10

Updates joda-time:joda-time from 2.9.9 to 2.10.
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention @scala-steward in the comments below.
Have a nice day!

Codecov Report

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

@@           Coverage Diff           @@
##           master     #105   +/-   ##
=======================================
  Coverage   91.64%   91.64%
=======================================
  Files          63       63
  Lines         993      993
  Branches       53       53
=======================================
  Hits          910      910
  Misses         83       83

Continue to review full report at Codecov.

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

source: gharchive/pull-request

authors: codecov-io scala-steward


text:
Use BigQuery for User Feedback analytics

Instead of fetching data from UA we now use the BigQuery API which
allows us to query the table for the specific metrics we need. It
already returns the information we need in the correct format and
reduces the amount of work required post query.
https://trello.com/c/6OkkMnm0/3411-5-migrate-ua-metrics-for-user-feedback-to-bigquery-in-content-data-api

Merging as @MuriloDalRi is on leave for 2 weeks.

authors: AgaDufrat MuriloDalRi

repo: alphagov/content-data-api


text:
Rename framework_framework variable

Summary
framework_framework is an unnecessarily obtuse variable name and is
not overly descriptive of what it actually means. When used, this
describes the 'family' of framework being referenced. For example,
G-Cloud 10 is in the 'G-Cloud' family of frameworks. Digital Outcomes
and Specialists 2 is in the 'Digital Outcomes and Specialists' family.
Let's change all the references of framework_framework to
framework_family.
Ticket
https://trello.com/c/iqFcVKpd/14
Dependencies
https://github.com/alphagov/digitalmarketplace-utils/pull/397

@samuelhwilliams - tests need re-running and branch needs updating. Dependencies seem to be merged now.

Yep. The idea is to start using api_stubs from utils, so hopefully these json file fixtures can go away 'soon'

~@samuelhwilliams is there a tech debt ticket for moving across to the utils stubs?~
This one right?
https://trello.com/c/FYPvix7N/183-update-api-stubs-in-dm-utils

Yep

authors: DilwoarH benvand samuelhwilliams


text:
Replace in-app beta banner with new toolkit version

Replaces the in-app beta banner with the new toolkit version.
Means we can delete a couple of files.
Largely the same as I'm doing in the buyer frontend.

👍


text:
[DO NOT MERGE] Use component for search results

This PR updates finder-frontend to use the document list component for search results.
Before:

After:

This involves some changes to the UI, specifically:

changes to the document list component (done)
metadata about search results is now output in a ul element, not a dl (more on this below)
links in search results will now be underlined (this seemed like such a small difference it wasn't worth adding an option to the document list component to not underline its links)
minor reduction of spacing between search results

Changes to the code
The documents method in result_set_presenter.rb has been modified to present the data in the format the document list component expects. This has involved taking most of the logic out of the _search_results.html.erb template and putting it into search_result_presenter.rb.
Because the logic is now in the presenter and not the template some use of html_safe has been required. I don't believe this could cause any problems but please check this when reviewing.
It might be that it would be better to update all this to pass the data to the presenter in a better format in the first place, and apparently advanced search is going to be removed soon, so we may need to revisit this.
Metadata output
Metadata for search results includes a label and a value. There is a flag for whether the label should be shown (or visually hidden). This was previously done in the template by optionally adding the govuk-visually-hidden class to the dt, which contained the label.
The document list component outputs metadata in a ul, so the change is that both the label and value are inside a li but the label can be optionally wrapped in a span with the visually hidden class. I'm happy to discuss whether this is semantically appropriate but the benefit is that when the label appears with the value it guarantees they appear together on the same line, i.e.
Before:

After:

Additionally, search results can also output "First published during the " beneath the metadata. This is now also being output by the document list component using the 'subtext' option, but the styling for it is controlled by finder-frontend.
The outputting of this text was recently fixed in this PR.
Highlighted or 'top' search results
Some searches have a top search result that is highlighted. In order to see this you may need to set the parameter GOVUK-ABTest-FinderAnswerABTest to B in your browser (I use this browser extension) and do a keyword search for eori.
They look like this:

This is now being handled with an optional parameter to the document list component to alter the appearance of a search result. The component is also able to output metadata with this result but we currently only show the link and description for these results.
Grouped results
The code for grouped results has largely remained the same.
Other information
If &debug_score=1 is appended to the parameters of a finder, further information about search results is output beneath them, as shown below. This is used by PAs.

This is also now being output by the document list component. It is a chunk of markup generated in the presenter and passed to the component as part of the 'subtext' option, but styled by finder-frontend.
Supersedes https://github.com/alphagov/finder-frontend/pull/1194
Trello card: https://trello.com/c/DAblviIN/793-investigate-using-more-components-in-finder-frontend

Specific examples to sanity check:

Advanced search pages, should have metadata
A finder with debug
a page with a highlighted result
a page with First published by on it

Other search page examples to sanity check:

http://finder-frontend-pr-1196.herokuapp.com/search/all
http://finder-frontend-pr-1196.herokuapp.com/search/research-and-statistics
http://finder-frontend-pr-1196.herokuapp.com/search/advanced?group=guidance_and_regulation&topic=%2Feducation
http://finder-frontend-pr-1196.herokuapp.com/drug-device-alerts
http://finder-frontend-pr-1196.herokuapp.com/find-eu-exit-guidance-business

Other finders

We should also add new unit tests to the search_results_presenter_spec for the new logic.

@bilbof have added.

repo: alphagov/finder-frontend


text:
Add child ordering to mainstream browse page

Currently, all 2nd level mainstream browse columns have the label "A to Z" on them. When a page has been curated, we shouldn't display this label, because it won't be true.
Therefore, collections,  needs to know the child ordering type for a top level mainstream browse pages in order to display related labels.
https://trello.com/c/bcTrheDZ/244-don-t-show-a-to-z-on-mainstream-browse-columns-which-aren-t-in-alphabetical-order

@alext, you merged it already but I was wondering, wouldn't this need to be called second_level_ordering? How am I going to refer to this parameter if Ruby does not allow variables starting with a number?

I'm pretty sure the ruby side of things can be worked around - but I do think it would be good to rename this to "second_level_ordering", for consistency with the existing "second_level_browse_pages" link field name.

Sounds good on the naming front.
As for strings vs symbols, I prefer to use string keys for data that's going to be JSON encoded to make things clearer.  Symbols will be converted to strings in the round-trip to JSON and back, and therefore the source hash doesn't match the JSON decoded version of it.

authors: alext jackscotti rboulton

repo: alphagov/govuk-content-schemas


text:
Run an AMA session to recruit DSDay organisers

What
Run an Ask Me Anything session to recruit DSDay organisers from GDS
Why
We need more organisers
Who needs to work on this
Epic Lead
Who needs to review this

Done when

[x] Session scheduled
[x] Organisers recruited
[x] Session promoted
[x] Session planned
[x] Session run successfully

Imran ran this week commencing 31/07/2023

authors: Ciandelle ImranH-GDS

repo: alphagov/govuk-design-system


text:
Problem with tunnelling instructions

I've noticed that other developers are not able to use the ssh commands in some of the manual pages, eg the integration command here https://docs.publishing.service.gov.uk/manual/rabbitmq.html#header
This is because ssh integration doesn't work for them.
In my ssh config I have an integration host defined as
Host integration
  Hostname jumpbox.integration.publishing.service.gov.uk

But this is not in the example config here:
https://github.com/alphagov/govuk-puppet/blob/master/development-vm/ssh_config
We could fix this by changing the example and/or changing the docs. I'm just creating the issue to keep track of this as I don't have time to fix right now. Does anyone have any opinions on how this should work?

Sounds like that file in govuk-puppet needs to be made up to date with https://docs.publishing.service.gov.uk/manual/ssh-config.html#header or the other way around...

Ah I wasn't even aware of that one. The getting started guide links directly to the file in github: https://docs.publishing.service.gov.uk/manual/get-started.html#6-access-remote-environments

Looks like this was fixed in 2018. Closing.

authors: MatMoore issyl0 thomasleese

repo: alphagov/govuk-developer-docs


text:
Set parent page for apps by team page

This will highlight the "Apps" link in the main navigation.

:man_facepalming:


text:
Update Fastly TLS certificate doc

This process requires having the superuser role in Fastly, which can be configured in govuk-user-reviewer.

Hold up a moment. If you need to get two PRs reviewed to temporarily elevate your permissions, isn't it gonna be more straightforward just to assign the ticket to someone who has Fastly superuser already?

Also, are you sure it requires superuser? Doesn't it just require the TLS management permission? (which unfortunately doesn't seem to be settable via Terraform, but any superuser can grant it to you in Fastly)

Hold up a moment. If you need to get two PRs reviewed to temporarily elevate your permissions, isn't it gonna be more straightforward just to assign the ticket to someone who has Fastly superuser already?

@sengi yeah that does make sense, I'm not sure if it's just Senior Tech members that have superuser permissions?

Also, are you sure it requires superuser? Doesn't it just require the TLS management permission? (which unfortunately doesn't seem to be settable via Terraform, but any superuser can grant it to you in Fastly — and it's fine for anyone with engineer access to have the TLS permission)

@ChrisBAshton are you able to confirm the above? In hindsight maybe I should have triaged it to Senior Tech.

@deborahchua there are a few people who have superuser access, including most of Senior Tech. I think it's fair to point people to Senior Tech to ask them to grant the permission 👍

authors: ChrisBAshton deborahchua sengi


text:
Update GOV.UK Frontend repo docs to link to new tech docs

What
After the new Tech Docs Template (TDT) tech docs are published, we need to update Frontend to: remove documentation that's been moved to the TDT, and update links so they point to the TDT.
Why
So there's no duplicate documentation, and user journeys work from the Frontend repo to the TDT.
Who needs to know
Mark, devs
Done when
Updated existing test branch to:

[ ] removed documentation that's been moved to the TDT
[ ] updated links so they point to the TDT

Then:

[ ] published

Not sure why this is closed – think this still needs a review.

repo: alphagov/govuk-frontend


text:
Gather performance stats with rollup-plugin-visualiser

What
Use rollup-plugin-visualiser to gather stats on code performance.
This will build off work done in:

https://github.com/alphagov/govuk-frontend/pull/3109
https://github.com/alphagov/govuk-frontend/pull/3115

We want to initially just get rollup-plugin-visualiser working with our code and outputting info from our build. This will likely require creating a standalone stats-gathering rollup build, since the plugin requires rollup 3, and we're pinned to 0.59.4 for IE8 compatibility.
Why
As we're making changes to the way our JavaScript works, and other architectural changes, we'd like to keep track of how our changes impact the files we distribute.
Who needs to work on this
Developer
Who needs to review this
Developer
Done when

[ ] We can output stats and visual data from rollup-plugin-visualiser: https://github.com/alphagov/govuk-frontend/pull/3239
[ ] We've decided which files and modules we want to track
[ ] Made issues for next steps:

[ ] Compare diff between pull requests and base branches
[ ] Github action to comment on PRs with these stats
[ ] Set up a way to store the data gathered to use for analysing trends
[ ] Decide what "good" looks like (after monitoring stats for a while)

Closing in favour of #3279


text:
Catch up with Pay to discuss how they do their API documentation

What
Meet with folks of GOV.UK Pay to get information on how they do their API documentation
Why
This will help guide what we could do for our own API documentation
Who needs to work on this
Technical writers (developers may want to listen in as well)
Who needs to review this
Anyone?
Done when

[x] We've met with folks of GOV.UK Pay and shared their information with the squad

Discussion summary (internal link)


text:
Hide Accordion content (again) during .js-enabled page load

This PR investigates an Accordion change to avoid a visual layout shift
During a slow page load (with js-enabled class on <body>) we now see Accordion sections jump from open to closed after component initialisation. We're not sure if this is preferred for users.
For example, shown when adding a 500ms delay before our code runs
<script src="/public/all.js"></script>
<script>
  // Add 500ms delay before our code runs
  // (For example, analytics might run first !!)
  setTimeout(function () {
    window.GOVUKFrontend.initAll()
  }, 500)
</script>

For discussion, I've included a quick tweak which keeps all the enhancements from:

https://github.com/alphagov/govuk-frontend/pull/3053

Still antsy about this, really, as although it reduces the layout shift on load, it's still shipping something that doesn't work properly if JavaScript fails. Doesn't feel very progressive enhancement of us.
As that's a bigger problem to fix across multiple components though, I'll just sit on my hands for now.

I agree with @querkmachine that this isn't ideal. The risk of users not being able to see the content is, IMO, more damaging than them seeing a flash of content. We do do this elsewhere where we interact with js-enabled and sort of assume that the component will initialise so there's precedence for it but in those instances eg: the header. In that instance though it defaults the header menu to be open on mobile, meaning users can still access content.
After ticking this over I unfortunately don't have any quick alternatives. It is technically the same as v4.4.0 and it allows us to ship a neat enhancement, so whilst it's a problem it's not explicitly a regression. I'm gonna wait and see if anyone else has opinions but the options I see right now are:

Merge this and make a note on https://github.com/alphagov/govuk-frontend/issues/999 as additional evidence
Close this and revert https://github.com/alphagov/govuk-frontend/pull/3053

Im leading towards 1. If nobody has any other feelings I'll approve this in the afternoon.

Thanks @owenatgov
Knowing that this could go either way we did a minor padding fix in this one:

https://github.com/alphagov/govuk-frontend/pull/3094

(So should we keep the flicker, at least it's a beautiful flicker 😊)
Had a quick run through Tabs and there's another interesting middle ground. With slow-loading JS you can still click a tab and it'll (eventually) catch up via window.location.hash
Will be a shame to merge this PR but gives us more time to fully consider:

JavaScript off
JavaScript on but crashed
JavaScript on but skipped checks
JavaScript on but took too long

https://github.com/alphagov/govuk-frontend/issues/999

@colinrotherham That PR you've linked needs a changelog entry as well!
I initially wondered if this should have a change entry on the basis that this was technically a revision of an existing PR, however I'm not quite confident that it does.

@colinrotherham That PR you've linked needs a changelog entry as well!

@owenatgov Ahh the padding fix will soon be hidden, can't do a change entry if the fix never gets rendered 😉
I'll do a quick one though for the mobile height, button margin and (maybe) trailing attribute space?

IMO this PR doesn't need a separate changelog entry as it's basically a continuation of the hidden="until-found" PR. Could reference both PRs against the same feature, I suppose?

@owenatgov @querkmachine @claireashworth Can you check these two new commits for me please?

ec27262 Add CHANGELOG note about accordion section visibility during load
6d08bd Hide Accordion content (again) during .js-enabled page load

Or see the full CHANGELOG.md on this branch

We're pausing this PR in favour of getting a fix release out in order to get a critical chrome bug released and give ourselves a bit more time to discuss the tradeoff's of this PR and it's interaction with https://github.com/alphagov/govuk-frontend/pull/3053.

We're pausing this PR in favour of getting a fix release out in order to get a critical chrome bug released and give ourselves a bit more time to discuss the tradeoff's of this PR and it's interaction with #3053.

We discussed this team today and agreed that we want to take a proper look at #999, but in the meantime we're happy to merge this as it basically maintains the 'status quo'.

authors: 36degrees colinrotherham owenatgov querkmachine


text:
Add .tmp folder to starter .gitignore

We've had reports from users that the .tmp folder is shown as not ignored by their IDE (IntelliJ types it seems), even though that folder contains a gitignore. Let's add .tmp to the starter .gitignore to stop people being confused by their IDE, but keep the .tmp/.gitignore behaviour to cover situations where the .gitignore file is missing for some reason. This is a bit belt-and-braces, but doesn't really have a downside.

Closing until #1711 is updated.

repo: alphagov/govuk-prototype-kit


text:
Avoid memoising meta tags

If we use with_variant more than once in a test case, we would keep
the old value set and the tests could potentially fail.
Trello: https://trello.com/c/t4n8kPpx/434-make-sure-collections-can-be-toggled-with-the-a-b-test-and-react-to-it
This is the cause of a bug reported here: https://github.com/alphagov/government-frontend/pull/264#discussion_r103697916

This makes sense. Thanks for tracking it down! 🚀

authors: carvil suzannehamilton


text:
Amend GOVUK.StickAtTopWhenScrolling

Add a check for a .js-sticky-resize class
If it exists on the sticky element, then find the width of the parent
element (as we’re assuming by having set this class that it’s in a
grid-column that will change it’s width when resized).
Get the width of the parent element and set the width of the shim and
the width of the sticky element to match when the window is resized.
In the tests, create a wrapper element with a width defined (300px) to
check that the width is also set for the sticky element and shim.
This is needed for service-manual-frontend and can be previewed over here.

Yep, I’m happy with this.

authors: gemmaleigh robinwhittleton


text:
Add GA4 tracking to devolved nations banners

What

Adds GA4 tracking to the devolved nations banner component.

More specifically, it adds devolved_nations_banner: 'England'/'England, Scotland, Wales'/'Scotland, Wales'/etc/undefined to the pageview if the banner is rendered. It grabs this value based on the value of the data-ga4-devolved-nations-banner attribute, for example: data-ga4-devolved-nations-banner=England, Scotland, Wales. This PR also adds the ga4-link-tracker to the banner.

Both of these rely on ga4_tracking: true so we can toggle the tracking on and off on the banner.

I had to write a Ruby function to grab the nations for data-ga4-devolved-nations-banner, and for the link tracker's section. This is because I needed to ensure the English text for the countries was always used. I also used gsub to replace  and with , so that the pageview value is England, Scotland, Wales instead of England, Scotland and Wales.

I've checked it in a local version of static.

I also added a commit to ensure the other banners don't render any GA4 attributes when ga4_tracking is false, as it felt wrong not to check for this.

Why

https://trello.com/c/XtFFZH3C/664-banners-devolved-nations-banner-element-visibility-and-navigation

Visual Changes

Added a with_ga4_tracking_example to the docs for the component.

Thanks @andysellick - I've made the changes, but had to modify your solution slightly because:

I need the two_words_connector and last_word_connector to also be forced to be English, otherwise and would get translated to another language.
I need the section to also be forced English, but not have the  and replaced as that GA4 value needs to be the texst that is rendered on the page rather than our modified custom pageview value.


text:
Make layout footer support spawning new windows for links

https://trello.com/c/TiZgaPxw/537-open-support-links-in-a-new-tab
Enhancing the layout footer component so that it can make pressing a link open a new window
or change the page of the current window.

Making use of link_to in _layout_footer.html.erb to handle the target attribute and
explicitly setting target to '' if no target attribute is passed to the component.
Adding a new example to layout_footer.yml so users can see examples of using the
layout footer component to open links in either the same or a new window.

Remember to update the CHANGELOG if your change needs to be listed in the next version of the gem.

I've generalised the component to accept a collection of anchor attributes rather than just focus on target.  I've also ensured that if target="_blank" then it'll set rel="noopener".

@kgarwood these commits seem to be associated with your VM's email address

authors: kevindew kgarwood


text:
Report errors to Sentry for failed Rake tasks

We are getting no errors when these tasks fail.
This will report the errors to sentry.

Nice one. Did you manage to see this working locally?

authors: emileswarts kjdchapman

repo: alphagov/govwifi-logging-api


text:
Export .aws_credentials in standard and terraform forms

Adjust .aws_credentials file (defined in the vault) so that it

Exports AWS credentials in a standard form (AWS_ACCESS_KEY_ID and AWS_SECRET_ACCESS_KEY). Adjust ansible-deploy job to consume them directly.
Exports AWS credentials in a terraform form (TF_VAR_aws_access_key and TF_VAR_aws_secret_key). Adjust terraform job to consume them directly.

LGTM :+1:

authors: actionjack mtekel

repo: alphagov/multicloud-deploy


text:
Update _java.md

Checked and updated links in the doc:

styling for external links (removed [external link] as not consistent with design system)
updated link to trial mode page in the error codes

Thanks @klssmith, great spot! I've updated them now. We're ready to merge/deploy these changes :)

repo: alphagov/notifications-tech-docs


text:
Revert "Bump winston from 3.10.0 to 3.11.0"

Reverts alphagov/pay-products-ui#2624

We no longer think that this dependency introduced a problem. Closing

repo: alphagov/pay-products-ui


text:
[DO NOT MERGE] PP-4822 Add text explaining $reference in email custom paragraph

Add some text to the page where the custom paragraph for email notification is configured, explaining that “$reference” will be replaced by the actual payment reference.

The build may fail because I had to reboot CI. Re-run it if it goes red.

@alexbishop1 I think we can close this PR?

closing due to lack of activity

repo: alphagov/pay-selfservice


text:
Support optimistic locking on v2 endpoints.

This allows publisher applications to detect conflicts
when (for example) a user has left their edit page
open for a long time and someone else has updated
the item in the meantime.
If the version the app thinks it's updating does
not match the version in the Publishing API, a 409
Conflict is returned.
If the previous_version key is not provided, no
checking is performed.  This makes this locking
opt-in on the part of the publisher application.
https://trello.com/c/s3dN4eDP/336-optimistic-locking
Client changes here: https://github.com/alphagov/gds-api-adapters/pull/383

@alext @tuzz I've refactored the command objects.

I feel this could do with some integration-level tests that cover things like nothing should be sent to the content-store etc when a version conflict is detected.

Thanks @elliotcm – looks good!

This is now updated to include request specs.

:+1:

authors: alext elliotcm tuzz

repo: alphagov/publishing-api


text:
Make QuestionPresenterTest more realistic

Since phrase lists are now only interpolated for question nodes, these tests
didn't make much sense. Especially as they were using the QuestionPresenter
with an Outcome node.
Also I'm not sure why the log statement was only being executed for outcome
nodes. If anything the opposite is now true and in any case the code in
SmartAnswer::I18nRenderer#value_for_interpolation will only ever be executed
for question nodes.

Looks good to me.

authors: chrisroos floehopper

repo: alphagov/smart-answers


text:
Maintain the status quo for sandbox admins.

Part of / related to: https://github.com/alphagov/gsp/pull/855

We changed direction slightly so this is unnecessary now.

repo: alphagov/tech-ops-cluster-config


text:
[WIP] - Send Editions' content_id to gds-api-adapters

We want to use content IDs instead of slugs to represent items in curated
lists, so that things like slug changes don't break the linkage. In order to do
this, we need to ensure that all content in publisher has a content-id, and
these content-ids are published to the publishing API.
This newly added test check that RegisterableEdition has a content_id.
I can't manually set a content_id. This test checks only for its presence.
Part of:
https://trello.com/c/rmBoBPm9/290-ensure-that-all-content-in-publisher-has-a-content-id

It's not worth merging this until we can bump gds-api-adapters to include the change that actually sends the content-id through to panopticon.

Closing this, going to re-open it after the work in gds-api-adapters is done

:+1: Looks good to me.

authors: alext jackscotti

repo: alphagov/whitehall


text:
Add endpoint to export documents information

This endpoint takes two arguments, lead_organisation and
type as well as pagination options like items_per_page and page_number.
lead_organisation is needed to find all the documents where
the lead_organisation is present on the latest edition of each
document. type is used to specify the whitehall edition type of the documents
you want returning, e.g NewsArticle or Publication.
The endpoint returns the following information for every document:

the document_id
all the locales the document has had
all the subtypes the document has had (news_article_type, speech_type, publication_type, corporate_information_page_type)
all the lead_organisations of the most recent edition

Trello:
https://trello.com/c/Tmq0PbYg/1037-create-an-endpoint-for-getting-a-list-of-documents-from-a-publisher

@kevindew I've made the changes you've requested, hopefully all the tests pass as the VPN doesn't seem to work on the train which means I can't see CI and my whitehall tests don't run properly locally! 😆

@kevindew PR is ready for another review


text:
Replace the prev/next and document list patterns with components gem on "latest" view

What
Adds 2 components to the "latest" view:

The prev/next component
The document list component

Why
This is part of work by the govuk accessibility team to replace bespoke components with our supported publishing components in frontend apps to reduce the risk of tech debt, bugs and accessibility issues creeping into our apps.
Page developed against
Card
Visual changes
Before

After

Just need an integration link for this review

authors: chris-gds owenatgov


text:
Avoid Oracle-specific collation in db:create.

utf8mb4_0900_ai_ci is specific to Oracle MySQL and doesn't work with MariaDB. utf8mb4_unicode_520_ci is, for all practical purposes, identical* and works with both MySQL and MariaDB.
This makes it easier to run Whitehall locally and helps avoid us getting locked into Oracle's flavour of MySQL.
Doesn't affect existing tables, only db:create, db:reset etc. i.e. testing/development.
* citation needed 🙃

Been thinking on this in the background and now I'm much less certain that it's worth it 🤔
Mainly because I forgot that db/schema.rb is supposed to be generatable from the actual database, i.e. Rails kinda considers the database itself to be the source of truth for defining the schema.
I'm also not 100% sure there isn't whacky data in production Whitehall that might get sorted/compared differently between these two very similar collations.
So I'm now leaning towards "ehh the whole Unicode situation with MySQL/MariaDB is messed up but this probably isn't a clean enough workaround on balance". Sorry for the noise!

Argument for I guess would be: "Well it doesn't break any tests, so it's still WAI." I wouldn't buy that though haha.


text:
Fully undocumented readme file

Hi alphamikle!
I do respect your efforts on this topic but I do not have a slittiest idea of how to use your library. Your source of code with an example is useless as I don't know how to navigate thru it. Sorry for saying this, but this is the fact that I have to study this provided project in order to figure out how to use your package - it is too complicated way to do this and not everyone have a time to explore the whole project and neither do I.
Moreover, in this bloc of code (package:high_low/domain/main/logic/main_backend.dart) I got this type of error: "The argument type 'Future<List> Function({void data, required MainEvent event})' can't be assigned to the parameter type 'FutureOr<ActionResponse> Function({required dynamic data, required MainEvent event})'."
@override
  void initActions() {
    whenEventCome(MainEvent.loadStocks).run(_loadStocks);
    whenEventCome(MainEvent.filterStocks).run(_filterStocks);
  }

Unless you provided a simple example with usage both Frontend and Backend  on example of  BLoC or another state manager I can't use this lib, unfortunately.
Please provide a simple example on both repo and pub as all other publishers do.
Thanks in advance.

Hi @fufylev! A big thanks for your feedback. Really appreciate it! I will update the example project this weekend and trigger you.

authors: alphamikle fufylev

repo: alphamikle/isolator


text:
How to add some secret static files?

How to copy secret static files from the root of the folder.

Railway automatically copies all root files to the root file of the docker image that it builds. If it is not showing there may be an error in the name of your file.

repo: alphasecio/flask


text:
Appears to hang if no components found

It should probably have a "nothing found" state, or even just end the loading animation, if we are keeping that.
(I'm starting a new project so no components yet)

This is what I came up with feel free to PR/push onto that PR.
Note that this only happens if Alpine is on the page but there are not components 😄 .

Looks fine to me. Yeah I just started a project and this page has no components, but I just load Alpine in globally anyway :)

authors: HugoDF KevinBatdorf

repo: alpine-collective/alpinejs-devtools


text:
Broken scrolling and background in FireFox for starter project

Describe the bug
When running starter project in FireFox custom scrolling of only content part is broken. Site works with default scroll behavior

Also that brakes background filling. So full page looks like that:

No problem in Chrome.
Also, main project (from main branch) works as expected in FireFox.

To Reproduce
Steps to reproduce the behavior:

Create "starter" project by running git clone -b starter https://github.com/altence/lightence-admin.git my-awesome-project && cd my-awesome-project
Run it with yarn install && yarn start
Open http://localhost:3000/profile/personal-info in FireFox
Scroll to the bottom of the page

Expected behavior
Expected that starter project will work same as main one. Without broken styles.
Desktop (please complete the following information):

OS: Windows
Browser FireFox
Version 109.0

Additional context
It is really important for people who want to use this project as base for their project.

Hello @Shultc, thank you for your issue.
We are sorry, but starter branch is now deprecated and soon will be deleted. Please, use master branch instead.

Created Pull Request for removing this section from README.md
https://github.com/altence/lightence-ant-design-react-template/pull/254

authors: Shultc leineveber


text:
TabControl UIXML not works

@generalloki
OS : Windows 11
Version : 3c3a8e0f364e10e9854f124cb414ea4b53837e30
https://docs.alternet-ui.com/api/Alternet.UI.TabControl.html
This code doesn't works in UIXML :
<TabControl SuggestedHeight="150" Name="tabControl" Margin="8">
  <TabPage Title="Page 1"></TabPage>
  <TabPage Title="Page 2"></TabPage>
  <TabPage Title="Page 3"></TabPage>
  <TabPage Title="Page 4"></TabPage>
  <TabPage Title="Page 5"></TabPage>
</TabControl>

I have test lot of things without solution
C# code need to be modified into Documentation :
var tc = new Alternet.UI.TabControl();
tc.SuggestedSize = (250, 300);
tc.Pages.Add(new TabPage { Title = "Page 1"});
tc.Pages.Add(new TabPage { Title = "Page 2" });
tc.Pages.Add(new TabPage { Title = "Page 3" });
tc.Pages.Add(new TabPage { Title = "Page 4" });
tc.Pages.Add(new TabPage { Title = "Page 5" });
tc.Parent = mainPanel;

With this on :
 var tc = new Alternet.UI.TabControl();
 tc.SuggestedSize = (250, 300);
 tc.Add("Page 1");
 tc.Add("Page 2");
 tc.Add("Page 3");
 tc.Add("Page 4");
 tc.Add("Page 5");
 tc.Parent =  mainPanel;

Thanks. I returned the old behavior. There is no need to call Add anymore if tabs are declared in uixml. We switched to own TabControl instead of native one, so some new bugs are possible.

@generalloki
TabControl with UIXML always not works with   version with this simple code :
<Window xmlns="http://schemas.alternetsoft.com/ui/2021"
        xmlns:x="http://schemas.alternetsoft.com/ui/2021/uixml"
        x:Class="testTabPages.MainWindow"
        Title="testTabPages">
	<StackPanel Orientation="Vertical" Name="mainPanel">
		<TabControl SuggestedHeight="150" Name="tabControl" Margin="8">
			<TabPage Title="Page 1"></TabPage>
			<TabPage Title="Page 2"></TabPage>
			<TabPage Title="Page 3"></TabPage>
			<TabPage Title="Page 4"></TabPage>
			<TabPage Title="Page 5"></TabPage>
		</TabControl>
	</StackPanel>
</Window>

No it doesn't work with the lastest version available, always blank window.
Here is test project with latest version 135058b3c7b2fff4e120293e229f941d5c92f971 :
testTabPages_New.zip

There is smth with you nugets version. I beleive they have the old code.
If I compile with Alternet.UI latest master directly without using nugets it works fine.
Here is the pic. Just in case I updated build version in master so it could help.

@generalloki
Sorry. I will have a look and try to understand what's happening.
Thanks a lot again

Test again with 0.9.508 now, and all works fine. :-)
I don't understand why my previous tests are KO because I pull last version, run "Install.bat", Sample show so compile is OK, update nugate package in my project.
Excuse me again to take your time for this error.

I will try to increase build number each time I upload to master (in any case it sounds right).
This would solve such kind of problems.

authors: generalloki neoxeo

repo: alternetsoft/AlternetUI


text:
Add support for creating pipelines without an estimator as the final component

Currently, our pipelines assume that the final component in the component graph is an estimator. We should allow for pipelines that are just a series of transformations (ex: preprocessing pipeline) that do not have an estimator.
This issue tracks adding support for pipelines without an estimator. Right now, fit does not error out, but it doesn't make sense to call predict or predict_proba when we don't have an estimator. Perhaps we should define or allow a transform method?
This came up in #2058, where we want to create pipelines from data check actions. These pipelines would only consist of transformers, and no estimators. However, until #2103 is complete, there's not a good way to create these valid preprocessing pipelines based on actions.

@angela97lin can you update this issue with a little more of the "why?" and use case?  I know we talked about it, but I can't for the life of me remember why.

@chukarsten Updated the description! 😁

I think my spike in #2110 addresses this! I added a transform method on ComponentGraph.

Thanks @angela97lin ! To confirm, this is a "nice to have" for data check actions, and our workaround of using a list of components will suffice for now, correct?

@dsherry Yeah, it'll be a little tricky to deal with target transformations, but it should be doable (just very hacky and not elegant at all).

For whoever picks this up: see #2151

I just closed #2151 because it was stale, but the branch still exists
@angela97lin : @chukarsten has expressed interest in working on this too. I am fine with whatever you two want to do, just let me know!

authors: angela97lin chukarsten dsherry

repo: alteryx/evalml


text:
Bump up minimum kaleido version

Pull Request Description
(replace this text with your description)

After creating the pull request: in order to pass the release_notes_updated check you will need to update the "Future Release" section of docs/source/release_notes.rst to include this pull request by adding :pr:123.

No linked issues found. Please add the corresponding issues in the pull request description.  Use GitHub automation to close the issue when a PR is merged

authors: christopherbunn machineFL


text:
[Suggestion] Need Generic Fail(T object) method too

It's hard to always implement IError and being bounded to the IError contract standard. With the generic fail method, developer can set their own implementation of errors.

You can also pass a string as error message which will be converted automatically to an Error object internally. Which class would you want to use for your Error?

authors: altmann gcbishal

repo: altmann/FluentResults


text:
matlab stall in exporting pdf of mesh

I'm not sure if this is an issue or if my file size is just too large, but every time I try to export a particular mesh figure to a pdf file, my matlab cpu usage goes to 100% and then nothing happens for up to 10 minutes (at which point I force quit).  I tried turning off aliasing and reducing resolution but neither seemed to help.
I'm working on a MacBook Pro with OS X version 10.11.4 and using matlab R2015b.
Code used:
filename='Laser Baseline';
rawdata=csvread(['CSV files-selected/',filename,'.ascii_0001.ascii.csv']);
figure(1)
clf
mesh(rawdata)
figure(1)
export_fig(['plots/',filename,'.pdf'],'-nocrop','-transparent','-pdf');

I've attached a screen copy of the mesh figure and can send a copy of the data file.  The size of raw data is 2192  by 2753.

Such problems are often due to the OpenGL figure renderer. Try using a different renderer i.e., export_fig(..., '-painters')

Thanks for the fast response!  Tried -painters and that didn't work.  Tried removing -transparent and using -painters but that didn't work either.  I'll try -painters with lower resolution and see if that works.

also try saveas(gcf, ['plots/',filename,'.pdf'])

authors: altmany matlaw


text:
EncodePipelineVAAPI: Don't allocate extra VA surface

This doesn't really matter all that much since it's only one extra surface, but it would be shame if someone took inspiration here and copied this bad code somewhere else (as I did).

May I ask what did you copy this for?

In wf-recorder.

authors: Vixea nowrep

repo: alvr-org/ALVR


text:
Drag & drop!

Added drag & drop functionality
Had to fight with reset button

russian youtube tutorials are the best


text:
Orthogonal initialization

Orthogonal initialization thats typically used in hidden-hidden weights in RNNs

Hi, Thanks for the PR.
It would be great if this can be changed to also support non-square matrices and matrices of dimension > 2 (i.e. convolutional layer weights). For example with a weight tensor of size [a, b, c, d], this can be reshaped to [a, bcd] and svd computed on this. Also, for non-square matrices, look at np.linalg.svd with the FullMatrices=False option.

Hey, I found some free time in the morning so put this in (to work with n-d tensors too), so closing this PR.

Great! Thanks

authors: MaximumEntropy alykhantejani

repo: alykhantejani/nninit


text:
🛑 stamp.itove.com is down

In b9b066e, stamp.itove.com (https://stamp.itove.com) was down:

HTTP code: 0
Response time: 0 ms

Resolved: stamp.itove.com is back up in 6f3774a.

repo: alzee/up


text:
Add skipLast(int)

Adds a new method that is the natural counterpart to skip(long)

Coverage decreased (-0.2%) to 98.444% when pulling b23a588ff426c0b2526a1934f8c9779cebfaa422 on epkugelmass:elan/skiplast into 59d5cee1e364c76ff258684899897da7213d520f on amaembo:master.

Coverage decreased (-0.2%) to 98.277% when pulling 2607371742d071b7388f1399c0ee0f6c56410d04 on epkugelmass:elan/skiplast into 5fbb114bafcfec7cde71ff2323d0b38598176767 on amaembo:master.

authors: coveralls epkugelmass

repo: amaembo/streamex


text:
auto complete problem's solution in cpp

I would like to contribute to this issue during the hacktoberfest. Please assign this issue to me. Thanks.

Can I be assigned to work on some Linked Lists problems for this issue?
Coz I don't see any Linked Lists implementations in C++ in your Repository.
P.S: Thanks in Advance :)

@AkshadK7
Yes, you can work on creating linked list implementation in C++. 😃
Create a PR once you are done with the changes.

Sure !

Can I work on implementing Dijkstra, Prims and Kruskal's algorithms in graphs?

I would also like to add a few algorithms to this repo, can you assign me too?

@Dhruv-VINT simply create a PR with the algorithms you want to contribute.
I'll review and merge them!

I would like to contribute please assign something to me

@bobbysaprey420
Yes, you can work on creating linked list implementation in C++. 😃
Create a PR once you are done with the changes.
(Also have a look at: https://github.com/amanmehara/programming/blob/master/CONTRIBUTING.md)

Can I be assigned to work on some more problems in competitive programming for this issue?
P.S: Thanks in Advance :)

yes sure
On Fri, Oct 16, 2020 at 2:42 PM Shubham Kumar notifications@github.com
wrote:

Can I be assigned to work on some more problems in competitive programming
for this issue?
P.S: Thanks in Advance :)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/amanmehara/programming/issues/78#issuecomment-709927858,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AJVPL5XLFH5EDUZRSTMZEZLSLAFBBANCNFSM4R25F7PQ
.

okay thanks a lot

@amanmehara is their anything I can contribute to earn some PR for Hacktoberfest.
Thanks in Advance

repo: amanmehara/programming


text:
It's possible determinate if a Position is inside a Polygon ?

There is a way with this plugin , to determinate if a Position results inside a Polygon defined on the map ?
Thanks.

Not supported.
You can use NetTopologySuite what is powerful geometry library.


text:
Update dependabot.yml

Issue #, if available:
None
Description of changes:
Updates the dependabot.yml file to limit dependabot to only one open PR at a time (for general updates—security updates are exempt from this limit) and make dependabot to group all github action updates into one PR.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Thanks Matt!

authors: jobarr-amzn popematt

repo: amazon-ion/ion-java


text:
Make SchemaSystem thread-safe

Right now, since SchemaSystem is !Send and !Sync, it can't be used from multiple threads concurrently or even passed between multiple threads.  If we want to validate an Ion document in a multithreaded environment, we need to instantiate a new SchemaSystem for each thread, which isn't great for performance.
Ideally, we should be able to load an ISL schema once and re-use it from multiple threads without having to (re)parse it every time.

I think there is some overlap between this issue and #150.
This is pretty off-the-cuff, but I wonder if we can solve them both by decoupling the loading of a schema from the management/caching of the schemas.
Suppose we define two traits:
// The point is that this trait should be some sort of unification across things like Arc<Schema>,
// Box<Schema>, Rc<Schema>, etc. so that regardless of how the user chooses to handle the
// memory management of the cached schemas, the other parts of the ISL implementation will still work.
trait SchemaRef: Clone {
    fn new(schema: Schema) -> Self;
    fn as_schema(self) -> &Schema
}

trait SchemaCache {
    type SchemaType: SchemaRef;

    fn get_or_put(&mut self, id: String, schema: fn() -> Schema) -> Self::SchemaType;
    fn get(&self, id: String) -> Option<Self::SchemaType>;
    fn invalidate(&mut self, id: String) -> IonSchemaResult<()>;
}

And we define a separate loader (function), such as this:
/// Loads the requested schema, along with any direct and transient imports
fn load_schema(id: String, authorities: &Vec<Authority>) -> IonSchemaResult<(Schema, Vec<Schema>)>;

Then with a bit of glue code, we can put together a SchemaSystem that wraps this loader function and a SchemaCache. Depending on the SchemaCache being used, the SchemaSystem could be Send/Sync, I think.
Obviously this idea needs some refinement if it's going to work, but it might be a decent place to start.

authors: popematt sadderchris

repo: amazon-ion/ion-schema-rust


text:
Biased Shapley Value Implementation

Thanks for the nice work!
The Shapley values implementation gives a biased estimate of the formula given in the paper as the weighting factor is missing. This leads to biased Shapley value estimates, though I do not believe it would change the gist of your results.
Some relevant pointers to how Shapley values are currently computed in the codebase:

The Shapley value (positive,negative) pairs are prepared here for example for the neighbors.
Since each feature is dropped/replaced with a 50% chance (here and here), there is an equal probability for each coalition size (i.e., feature subset size), |S|.
The Shapley value is then computed as the average of those pairs here.

My back-of-the-envelope expectation computation suggests that in such case a weighting factor of $\frac{2^{n-1}}{n {n - 1 \choose |S|}}$ needs to be added to unbias the estimate of the Shapley value, where n is the number of features for the current datapoint (i.e., the history and visible neighbors) and $|S|$ is the number of features (history/neighbors) that will be dropped/replaced. The numerator is needed to take into account the sampling probability of each sample, as each of the n-1 features will be dropped/replaced with a probability of $\frac{1}{2}$.
Do you possibly have some insights on the missing weighting factor? Was the weighting factor left out on purpose?

Hi,
Thanks for your question and interest in our paper.
Yes, you are right, but in our experimental results, we observe no difference between adding the weights or not, hence we opt for the simpler version in the code.
Best,
Osama

Thank you for the quick response and clarification, Osama. Closing the issue in that case!


text:
Ecr database.yml

${} syntax is unique to this project. Something more ubiquitous
should be used like ECR.

Evaluate database.yml as ECR before parsing as YAML
Switches the pg database config to use ECR
Add notes to README about using ECR, +example

Additionally, adding a deprecation notice for the ${} syntax may be in order?

Would love to alleviate this problem with just documentation and nix this PR.
This is why I created this PR:
I created my first Amber app. I generated some scaffolding and created a few records. Then I ran crystal spec, only to find that it was using my development db for tests.
In Rails, I'm used to tests running against a test database. I don't see a way to manage environment dependent databases, so I had planned to do something like:
# config/database.yml
pg:
  database: "my_database_<%= ENV["AMBER_ENV"] %>"

In production, I would just use a DATABASE_URL.
Perhaps this simply isn't needed with Amber? If so, I'd love to document it, I just need some guidance.

@danielwestendorf
currently, you can do:
pg:
  database: "my_database_${AMBER_ENV}"

This will pull the AMBER_ENV environment variable.  This is the syntax used by Docker and Docker Compose.
If we document this and change the templates in amber_cmd, will this satisfy your concerns?

@drujensen
Yes, I think it would satisfy my needs from a technical standpoint.
I don't use Docker and Docker Compose. Is that a requirement to use the framework? Why not both methods?
I spent about 30 minutes Googling crystal-lang ${} coming up with no results until I finally dug into the source to find it was a self-implemented convention. More documentation would have hopefully helped me figure it out sooner, but I'd personally prefer a convention already available in Crystal (humble opinion of language convention over implementation configuration).

@danielwestendorf thanks for the feedback.
I wrote this ${} a couple years ago.   I was learning docker and really liked the syntax.  I can see if someone is not familiar with it, it may look odd.
Regarding documentation, yes, that will always be a battle with new frameworks.  A lot of work is still experimental and you don't want to document until you feel confident it's going to stick.

Unfortunately, the yaml standard does not define how to interpolate variables.
After searching for "yaml environment variables" and "yaml interpolation", there are quite a few frameworks using the ${} syntax:
https://docs.docker.com/compose/environment-variables/
https://docs.puppet.com/hiera/3.3/variables.html
https://www.elastic.co/guide/en/beats/filebeat/1.2/using-environ-vars.html
https://discuss.circleci.com/t/cant-access-env-variables-from-my-yml-file/5463

@danielwestendorf I opened #48 to address your concerns and we will update amber-cmd to add the environment to the database configuration.
At this time, we don't feel that processing the database.yml with ECR is the right way of handling this.  If we find that we need more than just environment variables, we may consider re-opening this PR.
Again, thanks for the contribution.

authors: danielwestendorf drujensen

repo: amber-crystal/granite-orm


text:
Add support for rendering doubles using Grisu 2

Moving this from icicle to here so we can call it from Haskell.
Also, we finally have a roundtrip test for the double parsing / rendering!
! @amosr @tranma

👍

authors: amosr jystic

repo: ambiata/anemone


text:
Link libsodium statically

For use in anatomy-ambiata, among other reasons. Consider whether it's
worth switching to the latest version at the same time.

#67

repo: ambiata/tinfoil


text:
🛑 monitorss.ambler.kr is down

In fc855c8, monitorss.ambler.kr (https://monitorss.ambler.kr/) was down:

HTTP code: 503
Response time: 215 ms

Resolved: monitorss.ambler.kr is back up in a82dda1 after 15 minutes.

repo: amblerkr/upptime


text:
Thread was interrupted from a waiting state

Thread was interrupted from a waiting state

How? What's the scenario?

authors: amibar shupming

repo: amibar/SmartThreadPool

license: MS-PL


text:
/usr/bin/bash: line 167: gcovr: command not found

installing gcovr from spec 'gcovr==6.0'...
⚠  Note: '/root/.local/bin' is not on your PATH environment variable. These
    apps will not be globally accessible until your PATH is updated. Run `pipx
    ensurepath` to automatically add it, or manually modify your PATH in your
    shell's config file (i.e. ~/.bashrc).
done! ✨ 🌟 ✨
  installed package gcovr 6.0, installed using Python 3.10.12
  These apps are now globally available
    - gcovr

see https://gitlab.com/ClausKlein/cmake_template/-/jobs/5032411110#L2349

You should source ~/.cpprc. It should be added to the PATH automatically.

done, but
Successfully installed argcomplete-3.1.1 click-8.1.7 packaging-23.1 pipx-1.2.0 userpath-1.9.0
WARNING: Running pip as the 'root' user can result in broken permissions and conflicting behaviour with the system package manager. It is recommended to use a virtual environment instead: https://pip.pypa.io/warnings/venv
[command]/usr/bin/python3 -c import sys;print(sys.base_exec_prefix);
/usr
"/root/.local/bin" was added to "/root/.cpprc"
/root/.local/bin is already in PATH.
⚠  All pipx binary directories have been added to PATH. If you are sure you
want to proceed, try again with the '--force' flag.
Otherwise pipx is ready to go! ✨ 🌟 ✨
Installing venv  via pip
ERROR: Could not find a version that satisfies the requirement venv (from versions: none)
ERROR: No matching distribution found for venv
Installing python3-venv  via apt-get

.setup_cpp: &setup_cpp |
  curl -LJO "https://github.com/aminya/setup-cpp/releases/download/v0.35.2/setup-cpp-x64-linux"
  chmod +x setup-cpp-x64-linux
  echo -e "section_end:"`date +%s`":.setup_cpp\r\e[0Ksection_start:"`date +%s`":hidden_section\r\e[0K setup cpp"
  export PATH="$HOME/.local/bin:$PATH"
  ./setup-cpp-x64-linux --compiler $compiler --cmake true --ninja true  --ccache true \
    --clangtidy true --clangformat true --cppcheck true --gcovr true
  source ~/.cpprc
  echo -e "section_end:"`date +%s`":hidden_section\r\e[0Ksection_start:"`date +%s`":.setup_cpp\r\e[0K"
  clang-tidy --version
  cmake --version
  ninja --version
  gcc --version

but the most problems with v0.35.2 are using github-action:
https://github.com/cpp-best-practices/cmake_template/actions/runs/6102132450
      - name: Setup Cpp
        uses: aminya/setup-cpp@v1
        with:
          compiler: ${{ matrix.compiler }}
          vcvarsall: ${{ contains(matrix.os, 'windows' )}}

          cmake: true
          ninja: true
          vcpkg: false
          ccache: true
          clangtidy: ${{ env.CLANG_TIDY_VERSION }}

          cppcheck: true

          gcovr: true
          opencppcoverage: true

Could you try with the new version? @ClausKlein

Could you try with the new version? @ClausKlein

Works fine on gitlib-ci:
Installing collected packages: ninja, cmake, MarkupSafe, lxml, jinja2, gcovr
Successfully installed MarkupSafe-2.1.3 cmake-3.27.4.1 gcovr-6.0 jinja2-3.1.2 lxml-4.9.3 ninja-1.11.1
WARNING: Running pip as the 'root' user can result in broken permissions and conflicting behaviour with the system package manager. It is recommended to use a virtual environment instead: https://pip.pypa.io/warnings/venv
cmake version 3.27.4
CMake suite maintained and supported by Kitware (kitware.com/cmake).
1.11.1.git.kitware.jobserver-1
/usr/local/bin/gcovr
gcovr 6.0
Copyright (c) 2013-2023 the gcovr authors
Copyright (c) 2013 Sandia Corporation.
Under the terms of Contract DE-AC04-94AL85000 with Sandia Corporation,
the U.S. Government retains certain rights in this software.

authors: ClausKlein aminya

repo: aminya/setup-cpp


text:
Add helpful books section

Please comment out the books you know about computational neuroscience
It would glad if you can submit the PR on your own :)

make sure the book title is correct
make sure there is a link (e.g. amazon) for buying or reading the book

https://github.com/amirHossein-Ebrahimi/awesome-computational-neuro-science/pull/8


text:
Previous Release Tag Stuck at certain date

Somehow the action ignores any release since the 28th of February. Before that it was working perfectly fine...
The logs:

Note: These logs pertain to the second instance of the workflow being triggered today
Although there were multiple releases since then...

Hi @mcquenji , thanks for raising this issue.
Can you confirm if these tags after Feb are part of draft/pre-release or regular release. I checked the same action working fine on my repositories. Although I suspect the action is not able to fetch tags after Feb for your repo for some reason.

The releases are a regular release.
However, we've discovered that the action stops working every time a month passes (since 2024) im not quite sure what may cause this issue. We've observed that the action always thinks the latest release is from the last month even when there are new ones. Deleting all releases but the latest one from the current month fixes this issue (until the next month).

That's strange and not happening in my repos.
Is your repository public so that I can take a look? If not, then can you post how you are using this action?

My repo is private but here's the workflow:

name: Create Release

on:
  push:
    branches:
      - main

jobs:
    build:
        uses: ./.github/workflows/build.yaml
        secrets: inherit
    release:
        runs-on: ubuntu-latest
        needs: [build]
        steps:

            - name: Getting pdf from build action
              uses: actions/download-artifact@v4
              with:
                name: mainpdf-${{ github.sha }}

            - name: Auto Generate Next Release Tag
              uses: amitsingh-007/next-release-tag@v5.1.0
              id: tag
              with:
                github_token: ${{ secrets.GITHUB_TOKEN }}
                tag_prefix: ''
                tag_template: dd.mm.yy.i

            - name: Rename PDF
              run: mv main.pdf "LB Planner Thesis v${{ steps.tag.outputs.next_release_tag }}.pdf"

            - name: Create Release
              id: create_release
              uses: ncipollo/release-action@v1.14.0
              with:
                name: PDF Build ${{ steps.tag.outputs.next_release_tag }}
                tag: ${{ steps.tag.outputs.next_release_tag }}
                artifacts: "LB Planner Thesis v${{ steps.tag.outputs.next_release_tag }}.pdf"

Thanks for the snippet, I will take a look on the weekends

Hey @mcquenji, can you try using uses: amitsingh-007/next-release-tag@8fab8fd559ffb987e33cf9c3761c219f900d9854 this unpublished version. It has one debug log. Let me know if you see last release or some old release in the API response from GitHub.

i've changed the version of your action in my repo. will follow up when it gets stuck again


text:
Not workign

fsfaf

its' working now

still not working

right, now should be fine

repo: amitvaghela/TestOBJC


text:
Preventing deleted (not isEnabled) apps from being added to _listings…

…ByOwnerCache - The 'My Listings' apps list.\
@lsemesky @mannyrivera2010

Closing in favor of a code change that will handle this and other deleted listing bugs/requests/updates.

repo: aml-development/ozp-center


text:
link Swift Alps Event to right place

Correct link to swift alps ...

Readme is generated ... edit contents.json right away ...

repo: amobconf/awesome-mobile-conferences


text:
Building a docker images encounter some error (envirnment: arm,ubuntu )

Step 5/14 : RUN wget https://doris-thirdparty-repo.bj.bcebos.com/thirdparty/ldb_toolchain_gen-v0.4.sh     -q -O /tmp/ldb_toolchain_gen.sh     && sh /tmp/ldb_toolchain_gen.sh /var/local/ldb-toolchain/     && rm /tmp/ldb_toolchain_gen.sh
---> Running in 1805d086ad7b
bin/gcc: line 7: /var/local/ldb-toolchain/bin/gcc-11: cannot execute binary file
bin/gcc: line 7: /var/local/ldb-toolchain/bin/gcc-11: Success
Generated toolchain cannot compile a simple program.
Please file an issue at https://github.com/amosbird/ldb_toolchain_gen/issues with current setup information.
The command '/bin/sh -c wget https://doris-thirdparty-repo.bj.bcebos.com/thirdparty/ldb_toolchain_gen-v0.4.sh     -q -O /tmp/ldb_toolchain_gen.sh     && sh /tmp/ldb_toolchain_gen.sh /var/local/ldb-toolchain/     && rm /tmp/ldb_toolchain_gen.sh' returned a non-zero code: 1

Currently only x86 architecture is supported.

Thanks

authors: Andydufresneguohao amosbird


text:
Stop suppressing stream_select errors

I spent some hours today debugging why do my loops hang. There are numerous occasions when stream_select does not work. Instead of throwing the error, loop is going forever, because for some unknown reason, you throw only for one single type of error.
Example 1:
Loop::run(function() {
    $stream = fopen('php://memory', 'rwb');

    Loop::onWritable($stream, function() {
        throw new \Exception('I should have been thrown');
    });

    fwrite($stream, 'wat');
});

Example 2:
Loop::run(function() {
    $stream = tmpfile();
    stream_filter_append($stream, "string.rot13", STREAM_FILTER_WRITE);

    Loop::onWritable($stream, function() {
        throw new \Exception('I should have been thrown');
    });

    fwrite($stream, 'wat');
});

Hereby I request you to stop ignoring stream_select errors as it leads to hard to debug issues.

I'm not entirely sure why only a single type of error is being thrown if stream_select() returned false. @kelunik Do you remember why this was done?
@ostrolucky Note that the streams you are passing to the loop in the examples above cannot be used with the event loop. Only socket can be polled. Memory streams and file streams will never invoke the watcher, thus the exceptions in your watchers can never be thrown. The error suppression was stopping you from seeing the error from stream_select() that the streams were invalid, not catching any exceptions.

Yes I know now, see

There are numerous occasions when stream_select does not work

I'm not blaming you, that's stream_select limitation. But I didn't that when I was trying it. I would realize that much sooner if error wasn't silenced :)

I guess 'I should have been thrown' as the exception message made me think you were expecting that to be thrown but your complaint was that it was not. We're on the same page now, no worries.

Actually rereading your comment, this is not entirely true

Memory streams and file streams will never invoke the watcher

Second example will work when removing stream_filter_append. That's what's causing stream_select to not work. File resources are ok.

File resources appear to work, but will always report as readable or writable. So no, they don't actually work.

Yep, except those with stream_filter :)

@kelunik we are waiting for your input here

See #160. It was the initial attempt to unignore errors which we don't want to suppress.
https://github.com/amphp/amp/commit/a2477575355d1871779974c9e5902b70955071fb
We need to inspect which warnings we want to suppress and which we want exceptions for.

I suggest to whitelist errors rather than blacklist. Blacklists are never complete.

authors: kelunik ostrolucky trowski

repo: amphp/amp


text:
[TACHYON-618] Optimize max free allocator policy

https://tachyon.atlassian.net/browse/TACHYON-618
Currently the max free allocator just looks at the absolute number of bytes available in each directory. A better heuristic would be to always allocate to the highest tier possible. While it is not purely max free, it makes more sense as the user would expect higher tiers to be filled before lower tiers (since lower tiers generally are larger but slower).

+1 LGTM

+1 LGTM

authors: apc999 calvinjia carsonwang

repo: amplab/tachyon


text:
[TACHYON-1270] Remove deprecated client code

https://tachyon.atlassian.net/browse/TACHYON-1270

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Tachyon-Pull-Request-Builder/6871/
Test PASSed.

@calvinjia Thanks for cleaning this up! I like the new API a lot better - it's simpler and more consistent with our filesystem API. I've left some minor comments.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Tachyon-Pull-Request-Builder/6884/
Test FAILed.

LGTM

Jenkins, test this please.

Removing over 2k lines of code. Nice!

LGTM

LGTM

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Tachyon-Pull-Request-Builder/6896/
Test PASSed.

authors: AmplabJenkins aaudiber calvinjia gpang yupeng9


text:
getUsedBytes() assert in deleteFileTest

deleteFileTest can be invoked after other tests. If prior tests have written bytes into worker storage, getUsedBytes() will return non-zero

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Tachyon-Pull-Request-Builder/2694/
Test PASSed.

Thanks!

authors: AmplabJenkins haoyuan rootfs