aws-samples / aws-samples/sample-rvtools-processor

Does not support csv

Ouverte
#4 0 commentaires 0 réactions 0 personnes assignées Voir sur GitHub

Personne n'a encore pris cette issue.

Langage dominant
Python
Étoiles
9
Forks
4
Métriques de merge des PR
Aucune PR mergée en 30 j

Description

openpyxl doesn't support csv so it needs to be handled separately or simply converted to excel first.

While nothing on github indicates it does support CSV, the blog post does. It would be pretty simple to convert it and save temporary xlsx files to then process.
eg:

import csv
from openpyxl import Workbook

def csv_to_excel(csv_file, excel_file):
    wb = Workbook()
    ws = wb.active
    with open(csv_file, 'r', newline='', encoding='utf-8') as f:
        reader = csv.reader(f)
        for row in reader:
            ws.append(row)
    wb.save(excel_file)

Currently

> python rvtools_processor.py consolidate input1.csv input2.csv -o consolidated.csv

Starting consolidation of 2 files...
Processing: input1.csv
Error processing input1.csv: openpyxl does not support .csv file format, please check you can open it with Excel first. Supported formats are: .xlsx,.xlsm,.xltx,.xltm
Processing: input2.csv
Error processing input2.csv: openpyxl does not support .csv file format, please check you can open it with Excel first. Supported formats are: .xlsx,.xlsm,.xltx,.xltm
Creating consolidated file: input2.csv
Error: At least one sheet must be visible

Python 3.13.3

Package         Version
--------------- -----------   
openpyxl        3.1.5      
pandas          3.0.1

Guide de contribution

Ouvrir le guide de contribution

Par où commencer

  1. Lisez l'issue en entier, puis le guide de contribution du projet.
  2. Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
  3. Forkez le dépôt et travaillez sur une branche.
  4. Ouvrez une pull request qui référence le numéro de l'issue.

Piste de recherche

Commencez dans rvtools_processor.py, au niveau de la commande consolidate, en utilisant l’invocation CSV signalée et l’erreur openpyxl comme point d’entrée. Vérifiez comment les fichiers d’entrée sont actuellement traités et comment la sortie consolidée est créée. Le travail est terminé lorsque les entrées CSV peuvent être consolidées sans l’échec « At least one sheet must be visible ».

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Évaluation

Stack technique
pandas, python
Domaine
cli, data
Type d'issue
Fonctionnalité
Difficulté
3/5
Temps estimé
1-2 jours
Activité
Calme
Clarté
Plutôt claire
Accessibilité débutants
55/100

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.