nbgrader: lowercase spelling, address most of #8283

Author: haraldschilly

src/packages/frontend/course/commands.ts

@@ -156,7 +156,7 @@ export const COMMANDS: { [name: string]: Command } = {
     icon: "graduation-cap",
     label: defineMessage({
       id: "course.commands.nbgrader.label",
-      defaultMessage: "Configure Nbgrader",
+      defaultMessage: "Configure nbgrader",
     }),
     button: labels.nbgrader,
     title: defineMessage({
@@ -185,7 +185,7 @@ export const COMMANDS: { [name: string]: Command } = {
   "network-file-systems": {
     icon: "database",
     label: labels.cloud_storage_remote_filesystems,
-    button: "Nbgrader",
+    button: labels.nbgrader,
     title: defineMessage({
       id: "course.commands.network-file-systems.tooltip",
       defaultMessage:

src/packages/frontend/course/configuration/actions.ts

@@ -289,7 +289,7 @@ export class ConfigurationActions {
         });
       }
 
-      // we also make sure all teachers have access to that project – otherwise NBGrader can't work, etc.
+      // we also make sure all teachers have access to that project – otherwise nbgrader can't work, etc.
       // this has to happen *after* setting the course field, extended access control, ...
       const ps = redux.getStore("projects");
       const teachers = ps.get_users(store.get("course_project_id"));

src/packages/frontend/course/configuration/nbgrader.tsx

@@ -3,6 +3,8 @@
  *  License: MS-RSL – see LICENSE.md for details
  */
 
+import { Card, InputNumber, Radio } from "antd";
+
 import { Checkbox } from "@cocalc/frontend/antd-bootstrap";
 import {
   CSS,
@@ -11,9 +13,8 @@ import {
   useRedux,
 } from "@cocalc/frontend/app-framework";
 import { A, Icon } from "@cocalc/frontend/components";
-import { InputNumber } from "antd";
 import { SelectProject } from "@cocalc/frontend/projects/select-project";
-import { Card, Radio } from "antd";
+
 import { CourseActions } from "../actions";
 import {
   NBGRADER_CELL_TIMEOUT_MS,
@@ -26,7 +27,7 @@ const radioStyle: CSS = {
   display: "block",
   whiteSpace: "normal",
   fontWeight: "inherit",
-};
+} as const;
 
 interface Props {
   name: string;
@@ -106,7 +107,7 @@ export function Nbgrader({ name }: Props) {
         You can create a new project dedicated to running nbgrader, upgrade or
         license it appropriately, and copy any files to it that student work
         depends on. This new project will be shared will all collaborators of
-        this instructorproject.
+        this instructor project.
         <br />
         You can also grade all student work in the student's own project, which
         is good because the code runs in the same environment as the student
@@ -126,7 +127,7 @@ export function Nbgrader({ name }: Props) {
         }}
       >
         <h6>
-          Nbgrader hidden tests:{" "}
+          nbgrader hidden tests:{" "}
           {settings?.get("nbgrader_include_hidden_tests")
             ? "Included"
             : "NOT included"}
@@ -163,7 +164,7 @@ export function Nbgrader({ name }: Props) {
           borderRadius: "5px",
         }}
       >
-        <h6>Nbgrader timeouts: {timeout} seconds</h6>
+        <h6>nbgrader timeouts: {timeout} seconds</h6>
         <i>Grading timeout in seconds:</i> if grading a student notebook takes
         longer than <i>{timeout} seconds</i>, then it is terminated with a
         timeout error.
@@ -213,7 +214,7 @@ export function Nbgrader({ name }: Props) {
           borderRadius: "5px",
         }}
       >
-        <h6>Nbgrader output limits: {Math.round(max_output / 1000)} KB</h6>
+        <h6>nbgrader output limits: {Math.round(max_output / 1000)} KB</h6>
         <i>Max output:</i> if total output from all cells exceeds{" "}
         {Math.round(max_output / 1000)} KB, then further output is truncated.
         <InputNumber
@@ -258,7 +259,7 @@ export function Nbgrader({ name }: Props) {
         }}
       >
         <h6>
-          Nbgrader parallel limit:{" "}
+          nbgrader parallel limit:{" "}
           {parallel > 1
             ? `grade ${parallel} students at once`
             : "one student a time"}
@@ -282,9 +283,10 @@ export function Nbgrader({ name }: Props) {
   return (
     <Card
       title={
-        <A href="https://doc.cocalc.com/teaching-nbgrader.html">
-          <Icon name="graduation-cap" /> Nbgrader
-        </A>
+        <>
+          <Icon name="graduation-cap" /> nbgrader (
+          <A href="https://doc.cocalc.com/teaching-nbgrader.html">Docs</A>)
+        </>
       }
     >
       {render_grade_project()}

src/packages/frontend/course/nbgrader/nbgrader-button.tsx

@@ -56,7 +56,7 @@ export function NbgraderButton({ name, assignment_id }: Props) {
     const n = settings.get("nbgrader_parallel") ?? PARALLEL_DEFAULT;
     return (
       <Tip
-        title={`Nbgrader parallel limit: grade ${n} students at once`}
+        title={`nbgrader parallel limit: grade ${n} students at once`}
         tip="This is the max number of students to grade in parallel.  Change this in course configuration."
       >
         <div style={{ marginTop: "5px", fontWeight: 400 }}>
@@ -150,10 +150,10 @@ export function NbgraderButton({ name, assignment_id }: Props) {
     <span>
       {" "}
       <Icon name="cocalc-ring" spin />
-      <Gap /> Nbgrader is running
+      <Gap /> nbgrader is running
     </span>
   ) : (
-    <span>Nbgrader...</span>
+    <span>nbgrader...</span>
   );
   return (
     <div style={{ margin: "5px 0" }}>

src/packages/frontend/cspell.json

@@ -3,8 +3,11 @@
   "language": "en",
   "enabled": true,
   "words": [
+    "autograde",
+    "autograding",
     "Bioconductor",
     "CoCalc",
+    "conat",
     "dplyr",
     "ggplot",
     "ipython",
@@ -24,12 +27,14 @@
     "rclass",
     "rereturn",
     "respawns",
+    "revealjs",
     "Rmarkdown",
     "rtypes",
     "Sagemath",
     "sagetex",
     "sagews",
     "scikit",
+    "SocketIO",
     "statsmodels",
     "syncdb",
     "syncdoc",
@@ -41,14 +46,14 @@
     "timetravel",
     "tolerations",
     "undelete",
-    "undeleting",
-    "revealjs",
-    "conat",
-    "SocketIO"
+    "undeleting"
   ],
   "ignoreWords": [
     "antd",
+    "buttonbar",
     "collab",
+    "flyoutdragbar",
+    "flyouts",
     "immutablejs",
     "ipynb",
     "isdir",
@@ -60,21 +65,18 @@
     "mesg",
     "mintime",
     "mistralai",
+    "noconf",
+    "nprocs",
+    "pchildren",
+    "pids",
     "Popconfirm",
     "PoweroffOutlined",
+    "ptree",
     "reuseinflight",
     "sidechat",
     "vertexai",
     "vfill",
-    "xsmall",
-    "flyouts",
-    "buttonbar",
-    "noconf",
-    "flyoutdragbar",
-    "ptree",
-    "pchildren",
-    "nprocs",
-    "pids"
+    "xsmall"
   ],
   "flagWords": [],
   "ignorePaths": ["node_modules/**", "dist/**", "dist-ts/**", "build/**"],

src/packages/frontend/i18n/common.ts

@@ -689,7 +689,7 @@ export const labels = defineMessages({
   },
   nbgrader: {
     id: "labels.nbgrader",
-    defaultMessage: "NBgrader",
+    defaultMessage: "nbgrader",
   },
   name: { id: "labels.name", defaultMessage: "Name" },
   description: { id: "labels.description", defaultMessage: "Description" },