Skip to content
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

Stops highlighting in long paragraphs #210

Open
leosenko opened this issue May 17, 2020 · 4 comments
Open

Stops highlighting in long paragraphs #210

leosenko opened this issue May 17, 2020 · 4 comments

Comments

@leosenko
Copy link

Prerequisites

  • [ X] Put an X in this box if you have done the following:
    • Ensured the issue has not already been raised
    • Ensured the issue is caused by this package

Description

Steps to reproduce & Minimum working example

In latex source, create a paragraph of:
Isdadsa sdadas \cite{ds231} sadasds Fig.~\ref{fig:sss} by \cite{sdads12} dasda \SI{23}{\celsius}, dsad \SI{22222}{\celsius} adasd sad asd asd asd sa \SI{22222}{\celsius} dsadsa dnd \ce{Mg^2+} sds. \cite{sd223aa} sd \SI{21323123232}{\celsius} dsad \gls{sss} dasda \SI{2123322}{\celsius}.

The last \SI is not highlighted. If you delete/shorten the first cite to \cite{31}, it becomes highlighted.


#### Expected behaviour
all the primitives should be highligeted

#### Actual behaviour
After some amount of text, highlighting stops working.

### Additional information
Atom 1.46.0 x64 OSX 10.14.6
@AOS55
Copy link

AOS55 commented Jun 28, 2020

Also getting this problem, stays highlighted after math mode. For example after second inline math $ all is highlighted in $$, colour.

Conveniently $\frac{\nabla_{\bm{\theta}} \pi(A_{t}|S_{t}; \bm{\theta})}{\pi(A_{t}|S_{t}; \bm{\theta})}$ from equation\ref{eq:deriveScore} is the score function $\nabla_{\bm{\theta}} \ln(\pi(A_{t}|S_{t}; \bm{\theta}))$. This forms the basis of the REINFORCE algorithm \cite{RL41} and the update step samples $G_{t} \nabla_{\bm{\theta}} \ln(\pi(A_{t}|S_{t}; \bm{\theta}))$. If the return $G_{t}$ increases then the weight of the parameter is increased, therefore weighting policys with higher return. But reduces the size of the update if the probability of selecting an action is high, discounting policies that suffer from maximisation bias. REINFORCE is shown in algorithm \ref{algo:T-REINFORCE}, and best thought of as a parameterised MC method because the return is calculated based on the discounted sum for the whole episode.

@AOS55
Copy link

AOS55 commented Jun 28, 2020

I am also in OSX 10.15.4 atom is latest version 1.48.0 x64

@bbgra
Copy link

bbgra commented Jul 21, 2020

I experience the same two issues in similar scenarios!
Atom version 1.48.0 x64 on Windows 10.

@bbgra
Copy link

bbgra commented Jul 21, 2020

Some googling helped me solve the issue.

It appears that Atom has a maximum number of tokens per line (100), see atom/atom#1667
There is a package, allowing you to change the number of tokens per line, https://github.com/postcasio/atom-grammar-token-limit
This restores the syntax highlighting.

Hope that helps you fix your issues is as well!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants