-
Notifications
You must be signed in to change notification settings - Fork 32
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
Submission #10
Open
rohith10
wants to merge
123
commits into
CIS565-Fall-2013:master
Choose a base branch
from
rohith10:master
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Submission #10
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
…y code. Use GLFW, and the WinMain entry point.
…Hello member variables! Next step - separate out all the renderer/API related functions into separate class.
On to much needed refactoring/tidying up now.
… Renderer. Also, refactored/renamed a whole bunch of stuff with more on the way.
… now. GL facing ops now in Renderer. App holds all the models. Now uses draw lists, where objects are grouped by material. There's only one shader switch per list. Now uses logging routines to log messages as OutputDebugStrings (VS' Output window) or to File. Miles more to go before this can be called an optimized renderer by any definition. Next steps - get in a Shader Constant Manager, fix how lights are handled, and restore all of the original functionality (all the debug views from the original are temporarily disabled).
…ader; ShaderConstantManager now handles bool as ints.
…ners and should be deleted first. Removed a default framebuffer bind in render loop since it's followed by another bind immediately afterward.
…xture bindings for that program. Will hold Constant Buffer bindings as well, once all the uniforms are packaged into uniform buffers; Moved common includes/Enums to Common.h.
…! Need to be more careful when copy-pasta'ing)
…chnique. This class will hold the bind points for all the uniforms, so that all shader constants and textures can be set via this class. Puts an end to per-frame calls to glGetUniformLocation. Also includes the following changes: Moved common includes/Enums to Common.h.
- Don't normalize vertex normals in vert shader since we're doing it already to the interpolated normal in the frag shader.
…eed to get new version of tinyobjloader and rectify the mtl file.
- Overall: better handling of cmd line args. - GLApp: tinyobjloader is isolated to this module. - tinyobjloader: Update to version as of 4/12/2015. Now header only. - GLProgram: If a texture is not associated with a slot, it will be bound to 0. For example, if an item has a t2DDiffuse binding, but not a t2DNormal binding, the t2DNormal slot will be bound to 0. This allows correct sampling of unbound textures to (0,0,0,0). - TextureManager: will only try to load and create a texture if the filename is not empty. - Utility: new methods to log a complete line so there's no longer to a need to do LogFile/LogOutput followed by LogFile/LogOutput("\n").
…ay in the range [-100, 100] along any axis.
…rming the former only once a program is set active.
…l only if the debugger is present, in effect directing the output to the debugger. Otherwise, it logs to file. Now ignoring the logfile thus generated.
… and use DSA wherever possible.
GL 4.5 methods have a lot more driver overhead than GL 4.1/2 or even D3D11, even though on paper they have direct parallels to the D3D11 way of doing stuff. There really is no excuse for this in 2015.. oops, 2016.
…ions and named object functions (Direct State Access) wherever possible; Vertex Specification is separate from the VB binding.
…, instead of being in the lib directory shared32, since it's a source-only library.
…ll projects to target Win SDK 10.
… reference the libs built by 2015; VS2015 projects now emit their intermediate and output files to separate directories for better co-existence (with 2013).
…se files define a namespace which holds the hash codes of various shader variable names so that we don't end up running a hash function on every single variable that needs to be set every single frame, which is terribly inefficient. At startup, we initialize the hash codes by running the hash function on all shader variable names that we need, and then use them every single frame. Declared wrapper types ShaderConstantReference, TextureReference and ConstantBufferIndex to ensure some degree of type safety. All of these wrap uint32s and provide implicit conversions to/from uint32 so that they can be used as before.
[minor] Upgrade to Windows 10 SDK version 14393.
All projects build only in 64-bit now. GLFW upgraded to 3.2.1, and compiled locally to get VS 2017 lib/dll. Next up: Upgrade GLEW.
Remove multi-context build configurations from the GLEW project.
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.
No description provided.