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
π Release 3.3.0 #250
Merged
Merged
π Release 3.3.0 #250
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Properly flush character in serial command buffer if 'p' is accepted and the following character is not.
* β¨ Refactor linker scripts In order to support custom linker scripts from user templates (e.g. libconsole), refactor linker scripts such that memory regions are defined in first linker script processed and standard section placement is performed last. * Move ENTRY points to correct file Forgot to move, hot/cold now works.
Change it from https://vexforums.dom/index.php/ which does not exist to https://vexforums.dom/ which is the homepage.
* Added an std::string overload of Controller::set_text() * Declared the set_text overload in the header file * std::string is now a const std::string& for performance reasons
* update vexos to 1.0.11 * add IMU data rate function * update imu rate setting documentation Co-authored-by: Theo Lemay <16546293+theol0403@users.noreply.github.com> * fix and clarify buggy rounding logic Co-authored-by: Theo Lemay <16546293+theol0403@users.noreply.github.com>
the future and we forget to change the clear string.
different library's kassert)
* β¨ Separate functions * π Move to apix.h
β¨Increasing the Max String Size on the Controller Display
This reverts commit f65dd47. Revert β¨ Multiple phases of initialization
Revert "β¨ Multiple phases of initialization (#209)"
* Added mutex_delete functionality * Made mutex delete with a shared pointer * Removed extraneous code * Changed mutex_delete code to reference sem_delete * Simplified constructor
* Create ext_adi.h * Created vdml_ext_adi.c and ext_adi.hpp * Create vdml_ext_adi.cpp * Add smart_port struct * Remove Port Config Typedef * Change smart_port variable * Fixed adi_encoder_t ext_adi_encoder_init * Remove smart_port struct and add get_ports function * Implemented get_ports * Removing pointer * Working on implementation for get_ports() * May have found proper solution to the issues with get_ports issue. * Changed params of gyro to directly reference smart points. * (Unfinished) changed a few more params to be include both smart port and adi port. * Fixed get_ports macro scoping issue * Fixed C api compiling errors, current state of cpp file is commented out, and the headers have been added to the api header file. * Fixed warning with unused adi port var, and fixed the smart_ports to be 0 indexed * Revert "Fixed warning with unused adi port var, and fixed the smart_ports to be" This reverts commit b0b1a6a. * Made smart ports be - 1 * ext gyro fix (we were passing in the adi port instead of the gyro int_32t object itself). * moved ext-adi #define functions into ext_adi.h, the c++ api is good for testing * Change _port to uint32_t * Fix ultrasonic calls * fixed return_ports on ultrasonic_get, needs testing * Update Comments * made adi into wrapper for ext_adi * Fix Ultrasonic C++ api * Combined external adi and internal adi, so it's all one api under the old syntax. * Deleted old unused hpp/cpp files for the seperate cpp files. * merged c++ api works with encoders, sonars, buttons, motors, gyros * code clean up * documentation clean up * made ports in vdml_adi.c from int32_t to uint8_t * removed unnecessary parenthesis * fixed docs to accurately reflect function and not just "configure" * reverted version * Deleted redundant (For external adi port) comments in adi expander header (comments existed since these macros used to be in the normal adi expander). * Update include/pros/adi.hpp Co-authored-by: Alex Brooke <akb.sbc@gmail.com> * Update include/pros/adi.hpp Co-authored-by: Alex Brooke <akb.sbc@gmail.com> * Alphabetized api header file, also deleted unnecessary empty lines on vdml header. * Added spaces back to multiplyer = 1 to adi gyro default param. Co-authored-by: Alex Brooke <akb.sbc@gmail.com> * fixed ambiguous initializations * fixed typos in the documentation * Update src/devices/vdml_adi.cpp Co-authored-by: Alex Brooke <akb.sbc@gmail.com> * Formatting Fix Co-authored-by: Alex Brooke <akb.sbc@gmail.com> * Comment Formatting Fix Co-authored-by: Alex Brooke <akb.sbc@gmail.com> * Param Formatting Fix Co-authored-by: Alex Brooke <akb.sbc@gmail.com> * Fix Param Formatting Co-authored-by: Alex Brooke <akb.sbc@gmail.com> * Comment Fix Co-authored-by: Alex Brooke <akb.sbc@gmail.com> * Changed comment spelling error Co-authored-by: Nick Mertin <nickmertin@gmail.com> * Update include/pros/adi.hpp Co-authored-by: Alex Brooke <akb.sbc@gmail.com> * Update include/pros/ext_adi.h Co-authored-by: Alex Brooke <akb.sbc@gmail.com> * Update src/devices/vdml_adi.cpp Co-authored-by: Alex Brooke <akb.sbc@gmail.com> * fixed documentation and formatting * Fixed formatting Co-authored-by: Theo Lemay <16546293+theol0403@users.noreply.github.com> * Fix Formatting Co-authored-by: Theo Lemay <16546293+theol0403@users.noreply.github.com> * revert version Co-authored-by: Theo Lemay <16546293+theol0403@users.noreply.github.com> * formatting fixes, documentation cleanup, fixed typos * removed constant function parameters, reset version string * reversed the smart port and adi port * removed empty ADIPort constructor * removed unnecessary type values * moved macros out of public, updated ENXIO warning * added explicit to port_pair constructors * Update include/pros/adi.hpp Co-authored-by: Alex Brooke <akb.sbc@gmail.com> * corrected encoder to gyro in comments Co-authored-by: Alex Brooke <akb.sbc@gmail.com> * fixed collum formating, reverted explicit * Add External ADI Test * Format Ext ADI Test Co-authored-by: Leo Riesebach <djava@users.noreply.github.com> Co-authored-by: WillXuCodes <xu1321@purdue.edu> Co-authored-by: mooreBrendan <moore.brendanm@comcast.net> Co-authored-by: mooreBrendan <37316094+mooreBrendan@users.noreply.github.com> Co-authored-by: Alex Brooke <akb.sbc@gmail.com> Co-authored-by: Will Xu <54247087+WillXuCodes@users.noreply.github.com> Co-authored-by: Nick Mertin <nickmertin@gmail.com> Co-authored-by: Theo Lemay <16546293+theol0403@users.noreply.github.com>
Co-authored-by: Kunwar Sahni <kunwar.sahni01@gmail.com>
* remove some weird library linkings * one more
* -Cleaned up repeating code using a macro in controller.c. -Added a test implementation for clearing controller text by passing a 0 length/size string (as jpearman said on VEXForum is supposed to do: https://www.vexforum.com/t/pros-controller-screen-problem/83186/5?u=ranchsauce). * Fixed compiling errors. * Another few fixes relating to the one above (and bringing the number of chars back up to 19 max) * Version Reversion * Fixed formatting issues * Reverted version properly (forgot to save file last time) * Added vexSystemVersion check to controller_clear_line * Added old fix back in case user is on older version * Version reversion * Update version * Update src/devices/controller.c Co-authored-by: Alex Brooke <akb.sbc@gmail.com>
kunwarsahni01
added this to In Progress
in PROS Non-Versioned Kernel Development
via automation
Sep 13, 2020
HotelCalifornia
approved these changes
Sep 13, 2020
PROS Non-Versioned Kernel Development
automation
moved this from In Progress
to Reviewer Approved
Sep 13, 2020
PROS Non-Versioned Kernel Development
automation
moved this from Reviewer Approved
to Done
Sep 13, 2020
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summary:
Motivation:
It has indeed been a while
References (optional):
see above
Test Plan:
tested in individual PRs