probcomp/bayeslite

latent variables cannot be inputs to CGPM foreign models

Open

#445 aberto em 12 de jul. de 2016

Ver no GitHub
 (0 comments) (0 reactions) (0 assignees)Python (63 forks)batch import
help wantedk min/hr

Métricas do repositório

Stars
 (915 stars)
Métricas de merge de PR
 (Nenhuma PRs mesclada em 30d)

Description

create generator g for p using cgpm (
    model foo given err using venturescript (sp = errfoo),
    latent err numerical
)

This will not work because cgpm_metamodel._create_schema assumes that every input to a foreign model exists as a manifest variable of the population, and passes them to core.bayesdb_variable_stattype to determine the statistical type.

The logic in _create_schema must be changed to defer whatever processing requires the statistical types of inputs to foreign models until after all clauses have been ingested, and then update that processing to use latents (the mapping from latent variable names to their respective statistical types) instead of core.bayesdb_variable_stattype when appropriate.

Guia do colaborador

Direção de pesquisa
Examine o método `cgpm metamodel. create schema` e modifique a lógica para adiar o processamento dos tipos estatísticos de entrada até depois que todas as cláusulas forem ingeridas, usando o mapeamento latents para modelos estrangeiros quando apropriado.
Pilha de tecnologia
python
Domain
database
Tipo Issue
Bug
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-3 horas
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
PythonGit
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
75