search

SangeethKarunaratne / building-sat-

0 stars 0 forks source link

Results generation failed #88

Closed Dilshi830 closed 2 years ago

Dilshi830 commented 2 years ago

Describe the bug When I enter an entry and save it the following error message saying " Results generation failed" and Results do not get generated ![Uploading Screenshot (126).png…]()

To Reproduce Steps to reproduce the behavior:

  1. Go to '...'
  2. Click on '....'
  3. Scroll down to '....'
  4. See error

Expected behavior A clear and concise description of what you expected to happen.

Screenshots If applicable, add screenshots to help explain your problem.

Desktop (please complete the following information):

Smartphone (please complete the following information):

Additional context Add any other context about the problem here.

Dilshi830 commented 2 years ago

Also the results are only generated when I empty the cache and hard reload it every single time after a new entry. Else the same message stating " Results cannot be generated" is shown

SangeethKarunaratne commented 2 years ago

@Dilshi830 screenshot not attached and cannot recreate, attach screenshots and the output in console

Dilshi830 commented 2 years ago

Screenshot (126)

@SangeethKarunaratne I created a new project and this issue kept happening. Every time i create a new entry and press Ctrl+s this error message appears. When I scroll down to the results nothing is displayed there. Only when I emty the cache and hard reload, it is fixed.

I will try and screen record.

SangeethKarunaratne commented 2 years ago

@Dilshi830 there appears to be an issue when a new project is created will look into it.

SangeethKarunaratne commented 2 years ago

@Dilshi830 there appears to be an issue with chart generation, the code will be revert back to previous state, where those issues were not fixed, until this issue is fixed

Dilshi830 commented 2 years ago

Dear Mr Sangeeth,

The screen recording is too big to attach in Github.

I have attached here.

Yes, this occurs when a new project is created. EarthWorks - Google Chrome 2022-01-31 02-01-46.mp4 https://drive.google.com/file/d/1SAHd2pDGwy6Sbj6YhGY2ePyi2g1ErfZg/view?usp=drive_web

On Mon, Jan 31, 2022 at 1:42 AM Sangeeth Karunaratne < @.***> wrote:

@Dilshi830 https://github.com/Dilshi830 there appears to be an issue when a new project is created will look into it.

— Reply to this email directly, view it on GitHub https://github.com/SangeethKarunaratne/building-sat/issues/88#issuecomment-1025547745, or unsubscribe https://github.com/notifications/unsubscribe-auth/ASR2ODZ3YTWWAHOS3HII6WLUYZKRJANCNFSM5NE7PJ4A . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

You are receiving this because you were mentioned.Message ID: @.***>

Dilshi830 commented 2 years ago

Dear Mr Sangeeth,

I understand completely that this needs to be done to rectify what is happening. But as of now there are close to 80 fixed and closed Github entries. I sincerely hope and appreciate that you take responsibility that none of these closed and fixed issues will resurface or occur again in the final product when the code is reverted (once this specific issue is fixed) as it is very hard to recheck and spot errors that we have discussed and closed that are assumed to be fixed as of now.

We plan to keep checking Building-SAT for errors by running a new project everyday until the 4th of February. Kindly inform us on how to proceed on coming across errors that have already been entered and closed as Github entries. I hope we do not have to look for or try to spot issues that were already closed as this maybe very confusing.

Would very much appreciate your advice and thoughts on this.

Thank you, Dilshi

On Mon, Jan 31, 2022 at 2:04 AM Sangeeth Karunaratne < @.***> wrote:

@Dilshi830 https://github.com/Dilshi830 there appears to be an issue with chart generation, the code will be revert back to previous state, where those issues were not fixed, until this issue is fixed

— Reply to this email directly, view it on GitHub https://github.com/SangeethKarunaratne/building-sat/issues/88#issuecomment-1025565123, or unsubscribe https://github.com/notifications/unsubscribe-auth/ASR2OD3E7KR5DY3SCOGXMPTUYZNEPANCNFSM5NE7PJ4A . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

You are receiving this because you were mentioned.Message ID: @.***>

SangeethKarunaratne commented 2 years ago

@Dilshi830 closed issues will not resurface unless you check and close such issues, there were some issue that were closed without even testing the functionality, then it would resurface. And these issue were supposed be checked long time ago create issues now will only affect the final hand over timeline

SangeethKarunaratne commented 2 years ago

@Dilshi830 this issue is fixed

Dilshi830 commented 2 years ago

I will check and close the issue. Thank you very much

Dilshi

On Mon, Jan 31, 2022 at 11:52 PM Sangeeth Karunaratne < @.***> wrote:

@Dilshi830 https://github.com/Dilshi830 this issue is fixed

— Reply to this email directly, view it on GitHub https://github.com/SangeethKarunaratne/building-sat/issues/88#issuecomment-1026560954, or unsubscribe https://github.com/notifications/unsubscribe-auth/ASR2OD3UMKQF7H2LYAYGANLUY6GKDANCNFSM5NE7PJ4A . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

You are receiving this because you were mentioned.Message ID: @.***>

Dilshi830 commented 2 years ago

Mr Sangeeth,

I have used Building-SAT almost everyday continuously and I did not come across this results generation issue. As soon as I came across it I put the entry.

So do you mean that this issue is fixed by reverting the code to the previous state. If so, when can we check Building-SAT completely when the earlier issues do not resurface?

Thank you, Dilshi

On Tue, Feb 1, 2022 at 4:33 AM dilshi dharmarathna @.***> wrote:

I will check and close the issue. Thank you very much

Dilshi

On Mon, Jan 31, 2022 at 11:52 PM Sangeeth Karunaratne < @.***> wrote:

@Dilshi830 https://github.com/Dilshi830 this issue is fixed

— Reply to this email directly, view it on GitHub https://github.com/SangeethKarunaratne/building-sat/issues/88#issuecomment-1026560954, or unsubscribe https://github.com/notifications/unsubscribe-auth/ASR2OD3UMKQF7H2LYAYGANLUY6GKDANCNFSM5NE7PJ4A . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

You are receiving this because you were mentioned.Message ID: @.***>

SangeethKarunaratne commented 2 years ago

@Dilshi830 what I meant was not for this specific issue, these were what I referred too #91 #92 #86 #85 What I did was I reverted the latest code to previous state so that you can work on it, the fixes I add will be on top of the code with the issue, so when I add the fix it will be there with the one that created the issue before, closed tickets will not resurface with regard to what happened with this result generation issue. If the tickets were closed after testing the functionality none of them will resurface because it is the same code that was there from before.