-
Notifications
You must be signed in to change notification settings - Fork 112
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
Add numerical support of other real types (helper) #1948
Conversation
Review checklistThis checklist is meant to assist creators of PRs (to let them know what reviewers will typically look for) and reviewers (to guide them in a structured review process). Items do not need to be checked explicitly for a PR to be eligible for merging. Purpose and scope
Code quality
Documentation
Testing
Performance
Verification
Created with ❤️ by the Trixi.jl community. |
Redo #1929 to keep consistency. I may start my project soon so the whole type issue here may slow down (but won't stop). Let me know if you consider this urgent, and I may adjust my priority :) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks! For now, I suggest making the TODO notes more concrete. Besides that, this is open-source software, so we're happy about all contributions and know that other things are happening in real life.
Co-authored-by: Hendrik Ranocha <[email protected]>
What do you mean by other things happening in real life? |
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #1948 +/- ##
==========================================
- Coverage 96.15% 96.12% -0.03%
==========================================
Files 454 454
Lines 36509 36521 +12
==========================================
Hits 35103 35103
- Misses 1406 1418 +12
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
I can't spend as much time on this open-source project as I would like since I also have to other things |
I thought it means something more abstract and high level, but if you are talking about this, no worries. Does that mean I have total control over my project? As I was always thinking about making this a totally separate package within the ecosystem, like https://github.com/SciML/DiffEqGPU.jl, instead of integrating it directly into the original package. At least keep an eye on my project - sometimes it moves fast. |
This is open-source software, so we can not disallow this (as long as you respect the license). We would, of course, be happy if you continued contributing to the main Trixi.jl repo - in particular with fixes for floating point types like in this PR. |
I will try my best as I work full time during daytime.
Why not say, "This is open-source software, so you are allowed to do so"? You are saying something that seems exclusive to me. I believe everyone comes to open-source out of interest and all kinds of contribution are encouraged. This is just so wired for me, I just don’t get the same feeling from other places... Sorry for above if you are not feeling well, but I am also not feeling well 🥲 |
This is also offensive. This implies that you already consider the possibility that I may not respect the license. Would you say something like this to your colleagues? Probably no. I'm feeling a sense of inequality and hierarchy from you. Please mind your words even if you don't mean that way. I need some time to adjust myself - sorry for that I clearly know that MIT license is somehow the most flexible one - for me, only a copycat would be likely to violate it. |
I seldom feel a sense of inclusion here. Decisions are made by people who are not actually doing the work, instead of through an open and transparent vote. I feel like I’m just doing tasks that are assigned to me, and independent thinking and open-mindedness are discouraged. Maybe I chose the wrong place - this is an open-source place but makes me feel like a school, some people (includes me) are even tagged with the label of student |
I'm sorry if my words above sound aggressive to you - that was not my goal. Please do whatever you like to do and work on the open-source projects that you enjoy |
Extra part of #1909. Better saved as draft as it may take longer time.
Tasks: