matcom / dashboard

MIT License
7 stars 7 forks source link

Publications graph #20

Closed cmglezpdev closed 1 year ago

cmglezpdev commented 1 year ago

Cambios solicitados en el issue #15


Creo que sería buena práctica en mi implementación del grafo en el archivo de publicaciones.py refactorizar esto:

publications = {
    "papers": {
        "title": "Artículos",
        "data": papers,   
    },
    "presentations": {
        "title": "Presentaciones",
        "data": presentations,   
    },
    "books": {
        "title": "Libros",
        "data": books,   
    },
    "chapters": {
        "title": "Capítulos",
        "data": chapters,   
    }   
}

Esto es un diccionario que cree en base a los tipos de publicaciones que existen el cual lo veo adecuado para poder añadir el selector de subgrupos en el grafo pero es poco escalable. Quiero confirmar que son los únicos tipos de publicaciones que se manejarán, o si existiran mas previamente, añadir un método para poder buscar todos los tipos automáticamente y devolver este diccionario.


for publ in data:
    for author in publ.authors:
        if author.department != '':
            sections[ author.department ] = True
        if author.department != '':
            sections[ author.institution ] = True
        if author.department != '':
            sections[ author.faculty ] = True

También sería bueno poder abstrater este método en un método propio del modelo Person(o si cree conveniente de otra forma), para que automáticamente me devuelva las instituciones, departamentos y tal a los que pertenece, ya que esto lo uso mucho también