Skip to content

add feature:dump record to excel#14732

Open
InterKnight wants to merge 1 commit intoApolloAuto:masterfrom
InterKnight:dump_record_to_excel
Open

add feature:dump record to excel#14732
InterKnight wants to merge 1 commit intoApolloAuto:masterfrom
InterKnight:dump_record_to_excel

Conversation

@InterKnight
Copy link
Copy Markdown

add a feature:dump record to excel

@CLAassistant
Copy link
Copy Markdown

CLAassistant commented Dec 31, 2022

CLA assistant check
All committers have signed the CLA.

@daohu527
Copy link
Copy Markdown
Contributor

daohu527 commented Jan 3, 2023

@InterKnight Thank you for your contribution! Pls make a english version of readme, and we'd better change the name to readme.md.

In addition, please explain the application scenario of the function, can we not directly output to csv format

@InterKnight
Copy link
Copy Markdown
Author

Pls make a english version of readme, and we'd better change the name to readme.md.

no problem

please explain the application scenario of the function, can we not directly output to csv format

As far as I know,if we want to parse some topic to file,we need write code for each topic because any topic’s proto is different,so I write this script to parse any topic.
I haven't support csv format now, but it can be added to TODO list

And I have two quesetions:
1, Do I need to make a new pull_requset for the changes above?
2, I see the apollo_iscan is faild, what can I do to make it pass?

@daohu527
Copy link
Copy Markdown
Contributor

@InterKnight Thank you for your contribution, we need to evaluate whether this tool need to merge into the main branch, in fact we may want to create a separate repository for tools.

Actually Apollo now supports packages, like python packages. The merging of projects will also be graded, and it will be decided whether to merge into apollo according to the usage of the tools. During this period, you can promote your tools.

A bit like an incubation mechanism for new projects.

  1. Create a project, and it is really useful, and has a certain number of users
  2. Submit the package and merge it into the official repository or recommended list

I think we are at the first stage, I think you can keep your repo and promote it in the community, we are always here to help!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants