Seiver 0 Posted February 20, 2018 Share Posted February 20, 2018 ok new issue, Quote Link to post Share on other sites
SpaceBeeGaming 3 Posted February 20, 2018 Share Posted February 20, 2018 I'm also getting this MC 1.7.10 Quote Link to post Share on other sites
Gavote 1 Posted February 21, 2018 Share Posted February 21, 2018 Oooh. Good news is it seems to be consistent and repeatable...bad news is that it seems to be repeatable. Quote Link to post Share on other sites
XyFreak 36 Posted February 21, 2018 Author Share Posted February 21, 2018 Very bad news is that i can't fix that right off the bat. Looks like the calculations on the calibration data is taking too much time. I have an idea but... going to take a bit Quote Link to post Share on other sites
XyFreak 36 Posted February 22, 2018 Author Share Posted February 22, 2018 @Seiver@SpaceBeeGaming I've updated the "polynomial" component of BRGC to be more efficient with transponating matrixes. I know this is not going to fix the core issue here but it might help you getting over the issue for now. Can you please run the installer again and check if it's still occuring for you? If not please post me the error message again so I can figure out whether this actually helped a little or not. I need more time for a real/proper fix. Quote Link to post Share on other sites
Seiver 0 Posted February 22, 2018 Share Posted February 22, 2018 Quote Link to post Share on other sites
XyFreak 36 Posted February 22, 2018 Author Share Posted February 22, 2018 Looks like that didn't get it any further... thanks. Quote Link to post Share on other sites
XyFreak 36 Posted February 23, 2018 Author Share Posted February 23, 2018 Ok guys I have a working solution in the pipeline... It might break old OpenOS instances though so you better be prepared to upgrade that / reinstall the computer I'll see if i can squeeze in another smaller bugfix as well so stay tuned - fix should be out either in an hour or tomorrow Quote Link to post Share on other sites
XyFreak 36 Posted February 23, 2018 Author Share Posted February 23, 2018 Big Reactors Grid Control Version 4.2.8 has been released! Changelog: - Use threads/coroutines for heavy calculations on calibration data. This'll fix the "too long without yielding" errors. - Hopefully fixed an issue where the controller gets confused by the massive internal energy storage of draconic evolutions energy relays. NOTE: If you see errors regarding the controller not being able to find "thread" you'll have to upgrade your OpenOS! If you're upgrading, you'll have to re-download the installer because it is now fetching an additional package! Quote Link to post Share on other sites
Heavymetal100 0 Posted February 24, 2018 Share Posted February 24, 2018 Getting this when I try running the program any thoughts? Already tried re downloading the wget link and re installing still getting this Quote Link to post Share on other sites
Michiyo 28 Posted February 24, 2018 Share Posted February 24, 2018 Looks like your trying to run the program on an older OpenOS version or an older version of OpenComputers if the OC version is recent but you haven’t upgraded the OpenOS install you don’t have the new thread library. XyFreak 1 Quote Link to post Share on other sites
Heavymetal100 0 Posted February 24, 2018 Share Posted February 24, 2018 Yeah Im playing on the FTB Skyfactory 3 on version 1.10 i just update oc so ill see what happens Quote Link to post Share on other sites
Heavymetal100 0 Posted February 24, 2018 Share Posted February 24, 2018 Yup fixed it working just fine now. Thank you Quote Link to post Share on other sites
XyFreak 36 Posted February 24, 2018 Author Share Posted February 24, 2018 /me points at the last line of the changelog Quote Link to post Share on other sites
ZeroNoRyouki 7 Posted February 25, 2018 Share Posted February 25, 2018 @XyFreak so.... I'm testing a new implementation of the ER computers API and when I run your program GUI my reactor (the only one connected to the computer) shows up as DISCONNECTED. brgcctrl list show one reactor in the CALIBRATING state. I've update BRGC to the last version. I've also update OpenOS, broken and replaced my computer port, etc etc The major logical change in the API implementation is that all the set/do methods are not executed right away but they exit immediately after scheduling themselves to be executed on the next tick on the server thread while the get ones are still carried away on the LUA thread. So, for example, if you call getActive() right after a setActive(true) you will get false back if the LUA thread was not preempted in favor of the server thread. All this is done to get back in a thread-safe environment without incurring again the "yeld problem" I've also renamed the getMinimumCoordinate/getMaximumCoordinate methods but grep is telling me your are not using them. EDIT: problem found. I forgot that I've renamed getConnected too Quote Link to post Share on other sites
XyFreak 36 Posted February 25, 2018 Author Share Posted February 25, 2018 @ZeroNoRyouki Uh..yeah^^ that function is kinda important^^ If you intend to rename that, could you let me know in advance? I'll just look for either of those methods then and call whatever is available to me. Thanks ^^ Quote Link to post Share on other sites
ZeroNoRyouki 7 Posted February 26, 2018 Share Posted February 26, 2018 @XyFreak Yeah don't worry, I've put in an alias for that method so for a couple of version it should be OK. I'm going to send you and preview copy of the mod when I'm done with the API rewrite to test BRGC against it. I've also added a couple of new methods to grab more data in a single call Gavote 1 Quote Link to post Share on other sites
XyFreak 36 Posted February 26, 2018 Author Share Posted February 26, 2018 @ZeroNoRyouki Sounds nice Thanks Quote Link to post Share on other sites
evan1026 0 Posted February 28, 2018 Share Posted February 28, 2018 Having an issue running the installer. It does fine with the first few packages but when it gets to br_grid_control it just exits without outputting any errors or anything Attached a screenshot of the computer, if that helps. OpenComputers version is MC1.12.1-1.7.1.43 Quote Link to post Share on other sites
XyFreak 36 Posted February 28, 2018 Author Share Posted February 28, 2018 Your computer propably doesn't have enough RAM to process the br_grid_control package. Quote Link to post Share on other sites
evan1026 0 Posted March 1, 2018 Share Posted March 1, 2018 Yep, that was it. Would it be possible to add something to the installation page to mention you need at least two Tier 2 RAM sticks (or equivalent) to be able to install? Quote Link to post Share on other sites
ZeroNoRyouki 7 Posted March 1, 2018 Share Posted March 1, 2018 @XyFreak and everyone else interested in the ER Computer API: please check https://github.com/ZeroNoRyouki/BigReactors/issues/162 Quote Link to post Share on other sites
SpaceBeeGaming 3 Posted March 2, 2018 Share Posted March 2, 2018 These changes to the API mean that the program needs to have two version, one for ER and one for BR (1.7.10). Quote Link to post Share on other sites
XyFreak 36 Posted March 2, 2018 Author Share Posted March 2, 2018 @SpaceBeeGaming Nothing that can't be solved by some abstraction I can just check for the presence of certain functions and then use them if they're available. No big deal. Quote Link to post Share on other sites
SpaceBeeGaming 3 Posted March 3, 2018 Share Posted March 3, 2018 New crash. Quote Link to post Share on other sites