Yeah, I agree. I find just a little bit confusing to have models.py `model/sql/`. It's not a big deal, but we could have for example `model/models.py` with all the django models and then another python module containing SqlDataSource and all the other derived classes. In that way under sql we would only have files containing straight sql or json/sql