amuse icon indicating copy to clipboard operation
amuse copied to clipboard

Bugfix issue for #850

Open rieder opened this issue 3 years ago • 3 comments

  • [x] Test issue 850
  • [x] Add test to workflow for this branch
  • [ ] Fix issue
  • [ ] Remove workflow
  • [ ] Merge & Remove branch

rieder avatar Jul 05 '22 13:07 rieder

The test_issue_850 workflow tests issue #850 - it currently fails so the bug is still there (of course, since no fix has been made yet). I'm not sure what causes the issue, but it is clear that SeBa does not evolve star[1] from the test if a binary of stars[0,2] is added. Not sure if this extends to SeBa only evolving stars that are also in a binary (if any binary is added), perhaps also add this to the test? (cc @silviatoonen)

rieder avatar Jul 05 '22 13:07 rieder

This issue has been automatically marked as stale because it has not had recent activity. It will be closed in 28 days if no further activity occurs. Thank you for your contributions.

stale[bot] avatar Sep 06 '22 15:09 stale[bot]

see comment i made in #850

ipelupessy avatar Sep 08 '22 11:09 ipelupessy

This issue has been automatically marked as stale because it has not had recent activity. It will be closed in 28 days if no further activity occurs. Thank you for your contributions.

stale[bot] avatar Nov 07 '22 13:11 stale[bot]

this can be fixed see the comment on #850

ipelupessy avatar Nov 07 '22 15:11 ipelupessy

Hi all, this goes above my knowledge of amuse - if the suggestion of Inti works, let's go ahead and implement it. Otherwise, Simon would be the best person, knowing amuse & SeBa both? Cheers Silvia


Dr. Silvia Toonen Assistant professor Anton Pannekoek Institute University of Amsterdam Email: @.@.> @.@.> will soon be deactivated. Website: https://staff.fnwi.uva.nl/s.g.m.toonen/https://eur04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fstaff.fnwi.uva.nl%2Fs.g.m.toonen%2F&data=04%7C01%7Cservicedesk-icts%40uva.nl%7Ce2bdbbd0a70043e9f36f08d8ccef8185%7Ca0f1cacd618c4403b94576fb3d6874e5%7C1%7C0%7C637484676327089939%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C1000&sdata=89YxQBjtPoJ%2BC3gAfS9IC6HGNPTNQPeMg47HQL0UMoo%3D&reserved=0


From: Inti Pelupessy @.> Sent: 07 November 2022 15:47 To: amusecode/amuse @.> Cc: Silvia Toonen @.>; Mention @.> Subject: Re: [amusecode/amuse] Bugfix issue for #850 (PR #869)

this can be fixed see the comment on #850https://eur04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Famusecode%2Famuse%2Fissues%2F850&data=05%7C01%7CS.G.M.Toonen%40uva.nl%7Cc353139a950040289bb508dac0d761a4%7Ca0f1cacd618c4403b94576fb3d6874e5%7C0%7C0%7C638034328530297521%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000%7C%7C%7C&sdata=vxzrh0XpXuvuN2aRM%2BmCFzevR9UP0CEsYq7oh31%2F3%2Bw%3D&reserved=0

— Reply to this email directly, view it on GitHubhttps://eur04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Famusecode%2Famuse%2Fpull%2F869%23issuecomment-1305807528&data=05%7C01%7CS.G.M.Toonen%40uva.nl%7Cc353139a950040289bb508dac0d761a4%7Ca0f1cacd618c4403b94576fb3d6874e5%7C0%7C0%7C638034328530297521%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000%7C%7C%7C&sdata=CJv7RiV0s7FbeYJVh6yw8ttpUWS4yMKQngCw2qLHNHI%3D&reserved=0, or unsubscribehttps://eur04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAGUXULA6UEECKZRSM2LE54TWHEQBFANCNFSM52WFK64Q&data=05%7C01%7CS.G.M.Toonen%40uva.nl%7Cc353139a950040289bb508dac0d761a4%7Ca0f1cacd618c4403b94576fb3d6874e5%7C0%7C0%7C638034328530453743%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000%7C%7C%7C&sdata=eZ7OQWpuWoW21lalwG35eVJ0mM8VbOsl9bIvYJwZKAA%3D&reserved=0. You are receiving this because you were mentioned.Message ID: @.***>

silviatoonen avatar Nov 07 '22 16:11 silviatoonen

if the test cases are ok, its prob ok

ipelupessy avatar Nov 30 '22 14:11 ipelupessy

Test is successful, so are the regular SeBa tests. Merging.

rieder avatar Nov 30 '22 14:11 rieder

thanks!


Dr. Silvia Toonen Assistant professor Anton Pannekoek Institute University of Amsterdam Email: @.@.> @.@.> will soon be deactivated. Website: https://staff.fnwi.uva.nl/s.g.m.toonen/https://eur04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fstaff.fnwi.uva.nl%2Fs.g.m.toonen%2F&data=04%7C01%7Cservicedesk-icts%40uva.nl%7Ce2bdbbd0a70043e9f36f08d8ccef8185%7Ca0f1cacd618c4403b94576fb3d6874e5%7C1%7C0%7C637484676327089939%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C1000&sdata=89YxQBjtPoJ%2BC3gAfS9IC6HGNPTNQPeMg47HQL0UMoo%3D&reserved=0


From: Steven Rieder @.> Sent: 30 November 2022 14:14 To: amusecode/amuse @.> Cc: Silvia Toonen @.>; Mention @.> Subject: Re: [amusecode/amuse] Bugfix issue for #850 (PR #869)

Test is successful, so are the regular SeBa tests. Merging.

— Reply to this email directly, view it on GitHubhttps://eur04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Famusecode%2Famuse%2Fpull%2F869%23issuecomment-1332212583&data=05%7C01%7CS.G.M.Toonen%40uva.nl%7C906a6fb8fb8e44d0a58308dad2dd2f72%7Ca0f1cacd618c4403b94576fb3d6874e5%7C0%7C0%7C638054144672049225%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000%7C%7C%7C&sdata=wQMLbArFWFdZrS9mKGjf32RPU8Poz0UK3aY6Z%2BbL7Ks%3D&reserved=0, or unsubscribehttps://eur04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAGUXULGKF6QURRCCL2MWQ7TWK5OL7ANCNFSM52WFK64Q&data=05%7C01%7CS.G.M.Toonen%40uva.nl%7C906a6fb8fb8e44d0a58308dad2dd2f72%7Ca0f1cacd618c4403b94576fb3d6874e5%7C0%7C0%7C638054144672049225%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000%7C%7C%7C&sdata=rXpMdXx0S1JGDN5f6EIjhyL9yv9F%2F8CPpOuIPeehAR4%3D&reserved=0. You are receiving this because you were mentioned.Message ID: @.***>

silviatoonen avatar Nov 30 '22 14:11 silviatoonen