Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Picture in readme shows a error. #6091

Open
Atharva-77 opened this issue Dec 6, 2019 · 5 comments
Open

Picture in readme shows a error. #6091

Atharva-77 opened this issue Dec 6, 2019 · 5 comments

Comments

@Atharva-77
Copy link

Atharva-77 commented Dec 6, 2019

Overview Description

The picture in readme cannot be seen.

Actual Results

PictureError
The picture besides sentence 'Throughput Graph' shows error.

Expected Results

I will fix the error. The reason is that picture may be uploaded by github server. So if github changes where it wants to store it's images, the image would be broken down. Another problem can be that if someone clones this repository, then he may get the broken image as image is not directly stored in the repository. I will make a separate folder for the images to sort this issue.

Additional Information:

Can you provide me the image or link to it.

@arishta
Copy link

arishta commented Dec 7, 2019

The image link is actually broken. There's two way to proceed.

First - Commit the image and then use the raw link of the image to replace the broken URL.
Second - Replace the broken URL by a working URL.

Broken URL - " https://graphs.waffle.io/coala/coala/throughput.svg"

@Krythz43
Copy link

Krythz43 commented Dec 7, 2019

@arishta waffle.io shut down their website in May this year. Info on what this graph contained would be helpful. Maybe we could replicate the result or if it's not that important just remove it. What do you think?

@abhishalya
Copy link
Member

We don't need it, best to remove it.

@himan-21
Copy link

@abhishalya @Krythz43 may I remove this Throughput Graph image by editing the README.rst?

@rajpratyush rajpratyush mentioned this issue Mar 12, 2021
2 tasks
@RakeshPotnuru
Copy link

@Atharva-77

This issue needs to be closed.

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

Successfully merging a pull request may close this issue.

6 participants