Inconsistent variable highlighting?

OS X 10.11.2, Sublime Text 3113, Gravity theme, latest MagicPython.

Trying out MagicPython from PythonImproved. In the first Python file I opened, I see:

X_train and X_test are highlighted purple, but y_train and y_test are white. Capitalizing the Y vars turns them purple.

I’m assuming that any word starting with a capital is treated as a “constant” and colored accordingly. Is this intentional? Wouldn’t checking for ALL_CAPS be better?

[vpetrovykh]

After months of deliberations (believe it or not, we actually talked about this many times) we’ve come to the conclusion that single-upper-case-letter constants are probably more of an exception that the typical use case in generic code.

The module math defines its constants as lower-case anyways (so it’s e and not E). In general, in sciences and math single-letter constants are frequently lower-case, so we can assume that for readability that will be the style used in code for established constants (c, e, g, h, etc.), likewise there are some established single-letter constants that are traditionally upper-case (G, R, H_0). These are very specific domains, though. It’s likely that within them there is no strong need to add emphasis to these constants in code as people using them are already conditioned to pay attention to these special symbols.

In a generic program, however, single-letter variables are more likely indicating a temporary intermediate computation result or similarly a result that is not meriting extra emphasis. It is considered good style to give important global concepts (classes, functions or constants) a meaningful human-readable name. If the name is long and readable, but only contains a single upper-case letter, it’s more reasonable to conclude that this doesn’t merit extra emphasis and is probably not a constant (e.g. X_train or even single-letter class X).

[motjuste]

Hey! recently discovered this cool project. Awesome work!

Regarding this issue, I actually use this ‘bug’ to highlight some variables. It helps with a quick high level view of the flow.

Coincidently, I am apparently viewing the same script as @pikeas and really like how I can keep track of X_train and X_test.

Just adding my two cents.