john
/
McRogueFace
1
0
Fork 0
Code Issues 29 Pull Requests Projects 1 Releases Wiki Activity

SFML Python Library #14

New Issue
Closed
opened 2024-03-09 18:38:09 +00:00 by john · 2 comments
john commented 2024-03-09 18:38:09 +00:00
Owner
Copy Link

notes say:
18 feat: Expose SFML library as a built-in

Is there an official wrapper for SFML? Can it be included as a built in?

Intentions:

  • Use SFML classes directly where they're used in McRogueFace (colors, vectors, textures, fonts)
  • Allow Python to instantiate and provide those objects, handle the handoff to get SFML pointers directly (i.e. follow the "C++ First" tenet, don't call Python every frame)
  • Prep for Window class which has more SFML objects that aren't of great interest to the rest of McRogueFace
  • If there's no library available, consider a mcrfpy.SFML submodule to expose the useful functionality
notes say: 18 feat: Expose SFML library as a built-in Is there an official wrapper for SFML? Can it be included as a built in? Intentions: * Use SFML classes directly where they're used in McRogueFace (colors, vectors, textures, fonts) * Allow Python to instantiate and provide those objects, handle the handoff to get SFML pointers directly (i.e. follow the "C++ First" tenet, don't call Python every frame) * Prep for Window class which has more SFML objects that aren't of great interest to the rest of McRogueFace * If there's no library available, consider a `mcrfpy.SFML` submodule to expose the useful functionality
john added the
Major Feature
 label 2024-03-09 18:38:09 +00:00
john added this to the Alpha Release Targets milestone 2024-03-13 14:54:50 +00:00
john commented 2025-11-26 10:12:41 +00:00
Author
Owner
Copy Link

Status: Superseded by alternative approach

  • Instead of exposing SFML directly, the codebase now:
    • Has mcrfpy.libtcod submodule for FOV/pathfinding (src/McRFPy_Libtcod.h)
    • Exposes SFML-like types natively: mcrfpy.Color, mcrfpy.Vector, mcrfpy.Texture, mcrfpy.Font
    • Uses PyColor, PyVector, PyTexture, PyFont wrappers throughout

The goal of "use SFML classes directly" is effectively achieved through native mcrfpy types that wrap SFML objects.

Status: Superseded by alternative approach - Instead of exposing SFML directly, the codebase now: - Has mcrfpy.libtcod submodule for FOV/pathfinding (src/McRFPy_Libtcod.h) - Exposes SFML-like types natively: mcrfpy.Color, mcrfpy.Vector, mcrfpy.Texture, mcrfpy.Font - Uses PyColor, PyVector, PyTexture, PyFont wrappers throughout The goal of "use SFML classes directly" is effectively achieved through native mcrfpy types that wrap SFML objects.
john closed this issue 2025-11-26 10:12:41 +00:00
john commented 2025-11-26 10:23:52 +00:00
Author
Owner
Copy Link

Superseded by Current Approach

Instead of exposing SFML directly, the codebase now:

  1. Native mcrfpy types wrap SFML objects:

    • mcrfpy.Color wraps sf::Color
    • mcrfpy.Vector wraps sf::Vector2f
    • mcrfpy.Texture wraps texture loading
    • mcrfpy.Font wraps font loading

  2. mcrfpy.libtcod submodule for algorithms:

    • FOV computation
    • A* and Dijkstra pathfinding
    • Line algorithms

This approach is cleaner than a separate SFML module because:

  • Types are integrated directly into the mcrfpy namespace
  • No need to import a separate module
  • Types are designed for McRogueFace's use cases

The goal of "use SFML classes directly" is effectively achieved through native mcrfpy types.

## Superseded by Current Approach Instead of exposing SFML directly, the codebase now: 1. **Native mcrfpy types** wrap SFML objects: - `mcrfpy.Color` wraps `sf::Color` - `mcrfpy.Vector` wraps `sf::Vector2f` - `mcrfpy.Texture` wraps texture loading - `mcrfpy.Font` wraps font loading 2. **`mcrfpy.libtcod` submodule** for algorithms: - FOV computation - A* and Dijkstra pathfinding - Line algorithms This approach is cleaner than a separate SFML module because: - Types are integrated directly into the mcrfpy namespace - No need to import a separate module - Types are designed for McRogueFace's use cases The goal of "use SFML classes directly" is effectively achieved through native mcrfpy types.
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
master
rogueliketutorial25
alpha_presentable
alpha_streamline_2
alpha_streamline_1
interpreter_mode
grid_entity_integration
reprs_and_member_names
break_up_ui_h
standardize_font_handling
standardize_vector_handling
standardize_color_handling
standardize_texture_handling
raii_pyobjects
rebase-for-7DRL_Feb-2024
engjam_uicollection
engjam_ui_overhaul
Labels
Clear labels   
Alpha Release Requirement

This issue is a blocker to 0.1 Alpha release of McRogueFace.

  
Bugfix

   
Demo Target

Functionality to demonstrate; Depends on new features, but this issue isn't for writing them.

  
Documentation

   
Major Feature

Significant time and effort required.

  
Minor Feature

Some effort required to create or overhaul functionality.

  
priority:tier1-active

Current development focus - critical path to v1.0

  
priority:tier2-foundation

Important foundation work - not blocking v1.0

  
priority:tier3-future

Future features - deferred until after v1.0

  
Refactoring & Cleanup

no new functionality, just improving the codebase.

  
system:animation

Animation and property interpolation

  
system:documentation

Documentation infrastructure

  
system:grid

Grid system and spatial containers

  
system:input

Input handling and events

  
system:performance

Performance optimization and profiling

  
system:python-binding

Python/C++ binding layer

  
system:rendering

Rendering pipeline and visuals

  
system:ui-hierarchy

UI component hierarchy and composition

  
Tiny Feature

quick and easy - a few lines or with little interconnection around the codebase

  
workflow:blocked

Blocked by other work - waiting on dependencies

  
workflow:needs-benchmark

Needs performance testing and benchmarks

  
workflow:needs-documentation

Needs documentation before or after implementation

No Label
Alpha Release Requirement
Bugfix
Demo Target
Documentation
Major Feature
Minor Feature
priority:tier1-active
priority:tier2-foundation
priority:tier3-future
Refactoring & Cleanup
system:animation
system:documentation
system:grid
system:input
system:performance
system:python-binding
system:rendering
system:ui-hierarchy
Tiny Feature
workflow:blocked
workflow:needs-benchmark
workflow:needs-documentation
Milestone
Clear milestone
No items
No Milestone
Alpha Release Targets
Projects
Clear projects
No project
Assignees
Clear assignees
No Assignees
1 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: john/McRogueFace#14
No description provided.
Delete Branch "%!s(<nil>)"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?