Skip to content

Conversation

@drammock
Copy link
Member

cc @cbrnr

@drammock drammock added the backport-candidate on-merge: backport to maint/1.11 label Nov 30, 2022
@cbrnr
Copy link
Contributor

cbrnr commented Dec 1, 2022

Looks great! Thanks @drammock!

@drammock
Copy link
Member Author

drammock commented Dec 1, 2022

https://output.circle-artifacts.com/output/job/4fac4c92-3241-4dfb-9dad-3e7dd75eb178/artifacts/0/dev/index.html

Failed CI is a windows permissions error on test_search_light (plus someone cancelled the compat/old job?)

@cbrnr cbrnr changed the title Add mastadon link Add mastodon link Dec 1, 2022
@larsoner larsoner merged commit 452d51d into mne-tools:main Dec 1, 2022
@larsoner
Copy link
Member

larsoner commented Dec 1, 2022

Thanks @drammock ! Now we just need some content 😄

@cbrnr
Copy link
Contributor

cbrnr commented Dec 1, 2022

Do/did we already have someone in charge of our Twitter account (not the bot)?

@drammock drammock deleted the add-mastadon-link branch December 1, 2022 23:15
drammock added a commit that referenced this pull request Dec 2, 2022
@cbrnr
Copy link
Contributor

cbrnr commented Dec 3, 2022

I just checked our Mastodon profile, and we don't get the green checkmark if I link to https://mne.tools/. However, I do get it when I link to https://mne.tools/stable/index.html, which I'd like to avoid (after all, this URL is what people see as our official homepage). @drammock do you know why this is the case (some redirection?) and if it can be fixed?

@drammock
Copy link
Member Author

drammock commented Dec 3, 2022

Yeah Probably mastodon's script doesn't follow redirects. Does it work to just specify /stable (without the index.html)? That seems an ok compromise to me.

@cbrnr
Copy link
Contributor

cbrnr commented Dec 4, 2022

Yes, adding /stable works!

larsoner added a commit to larsoner/mne-python that referenced this pull request Dec 6, 2022
* upstream/main:
  Revert "[ENH] Add tutorial on time-frequency source estimation with STC viewer GUI" (mne-tools#11350)
  [ENH] Add tutorial on time-frequency source estimation with STC viewer GUI (mne-tools#10920)
  FIX: Fix example (mne-tools#11348)
  Add mastodon link (mne-tools#11347)
  fix default n_fft for spectrum plots (mne-tools#11345)
@larsoner larsoner added backported and removed backport-candidate on-merge: backport to maint/1.11 labels Feb 24, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants