Speaking endpoints and polling fixed

This commit is contained in:
Carlos-Mesquita
2024-11-27 08:04:18 +00:00
parent a2a513077f
commit a96d4c6e52
5 changed files with 215 additions and 107 deletions

View File

@@ -0,0 +1,95 @@
import { UserSolution } from '@/interfaces/exam';
import useExamStore from '@/stores/exam';
import { StateFlags } from '@/stores/exam/types';
import axios from 'axios';
import { SetStateAction, useEffect, useRef } from 'react';
type UseEvaluationPolling = (props: {
pendingExercises: string[],
setPendingExercises: React.Dispatch<SetStateAction<string[]>>,
}) => void;
const useEvaluationPolling: UseEvaluationPolling = ({
pendingExercises,
setPendingExercises,
}) => {
const {
flags, sessionId, user,
userSolutions, evaluated,
setEvaluated, setFlags
} = useExamStore();
const pollingTimeoutRef = useRef<NodeJS.Timeout>();
useEffect(() => {
return () => {
if (pollingTimeoutRef.current) {
clearTimeout(pollingTimeoutRef.current);
}
};
}, []);
useEffect(() => {
if (!flags.pendingEvaluation || pendingExercises.length === 0) {
if (pollingTimeoutRef.current) {
clearTimeout(pollingTimeoutRef.current);
}
return;
}
const pollStatus = async () => {
try {
const { data } = await axios.get('/api/evaluate/status', {
params: {
sessionId,
userId: user,
exerciseIds: pendingExercises.join(',')
}
});
if (data.finishedExerciseIds.length > 0) {
const remainingExercises = pendingExercises.filter(
id => !data.finishedExerciseIds.includes(id)
);
setPendingExercises(remainingExercises);
if (remainingExercises.length === 0) {
const evaluatedData = await axios.post('/api/evaluate/fetchSolutions', {
sessionId,
userId: user,
userSolutions
});
const newEvaluations = evaluatedData.data.filter(
(newEval: UserSolution) =>
!evaluated.some(existingEval => existingEval.exercise === newEval.exercise)
);
setEvaluated([...evaluated, ...newEvaluations]);
setFlags({ pendingEvaluation: false });
return;
}
}
if (pendingExercises.length > 0) {
pollingTimeoutRef.current = setTimeout(pollStatus, 5000);
}
} catch (error) {
console.error('Evaluation polling error:', error);
pollingTimeoutRef.current = setTimeout(pollStatus, 5000);
}
};
pollStatus();
return () => {
if (pollingTimeoutRef.current) {
clearTimeout(pollingTimeoutRef.current);
}
};
});
};
export default useEvaluationPolling;

View File

@@ -11,17 +11,17 @@ import Reading from "@/exams/Reading";
import Selection from "@/exams/Selection";
import Speaking from "@/exams/Speaking";
import Writing from "@/exams/Writing";
import { Exam, ExerciseOnlyExam, LevelExam, PartExam, UserSolution, Variant } from "@/interfaces/exam";
import { Stat, User } from "@/interfaces/user";
import { Exam, LevelExam, UserSolution, Variant } from "@/interfaces/exam";
import { User } from "@/interfaces/user";
import { evaluateSpeakingAnswer, evaluateWritingAnswer } from "@/utils/evaluation";
import { defaultExamUserSolutions, getExam } from "@/utils/exams";
import { getExam } from "@/utils/exams";
import axios from "axios";
import { useRouter } from "next/router";
import { toast, ToastContainer } from "react-toastify";
import { v4 as uuidv4 } from "uuid";
import ShortUniqueId from "short-unique-id";
import { ExamProps } from "@/exams/types";
import useExamStore from "@/stores/exam";
import useEvaluationPolling from "@/hooks/useEvaluationPolling";
interface Props {
page: "exams" | "exercises";
@@ -128,7 +128,8 @@ export default function ExamPage({ page, user, destination = "/", hideSidebar =
if (exercise.type === "writing")
await evaluateWritingAnswer(user.id, sessionId, exercise, index + 1, userSolutions.find((x) => x.exercise === exercise.id)!);
if (exercise.type === "interactiveSpeaking" || exercise.type === "speaking")
if (exercise.type === "interactiveSpeaking" || exercise.type === "speaking"){
console.log(userSolutions.find((x) => x.exercise === exercise.id)!);
await evaluateSpeakingAnswer(
user.id,
sessionId,
@@ -136,6 +137,7 @@ export default function ExamPage({ page, user, destination = "/", hideSidebar =
userSolutions.find((x) => x.exercise === exercise.id)!,
index + 1,
);
}
}),
)
})();
@@ -143,53 +145,7 @@ export default function ExamPage({ page, user, destination = "/", hideSidebar =
}
}, [exam, showSolutions, userSolutions, sessionId, user?.id, flags]);
useEffect(() => {
if (!flags.pendingEvaluation || pendingExercises.length === 0) return;
const pollStatus = async () => {
try {
// Will fetch evaluations that either were completed or had an error
const { data } = await axios.get('/api/evaluate/status', {
params: {
sessionId,
userId: user.id,
exerciseIds: pendingExercises.join(',')
}
});
if (data.finishedExerciseIds.length > 0) {
const remainingExercises = pendingExercises.filter(id => !data.finishedExerciseIds.includes(id));
setPendingExercises(remainingExercises);
if (remainingExercises.length === 0) {
const evaluatedData = await axios.post('/api/evaluate/fetchSolutions', {
sessionId,
userId: user.id,
userSolutions
});
const newEvaluations = evaluatedData.data.filter(
(newEval: UserSolution) => !evaluated.some(
existingEval => existingEval.exercise === newEval.exercise
)
);
setEvaluated([...evaluated, ...newEvaluations]);
setFlags({ pendingEvaluation: false });
return;
}
}
if (pendingExercises.length > 0) {
setTimeout(pollStatus, 5000);
}
} catch (error) {
console.error(error);
setTimeout(pollStatus, 5000);
}
};
pollStatus();
}, [sessionId, user.id, userSolutions, setFlags, setEvaluated, evaluated, flags, pendingExercises]);
useEvaluationPolling({pendingExercises, setPendingExercises});
useEffect(() => {
if (flags.finalizeExam && moduleIndex !== -1) {

View File

@@ -20,27 +20,51 @@ async function handler(req: NextApiRequest, res: NextApiResponse) {
await form.parse(req, async (err: any, fields: any, files: any) => {
if (err) {
console.log(err);
res.status(500).json({ ok: false });
console.error('Error parsing form:', err);
res.status(500).json({ ok: false, error: 'Failed to parse form data' });
return;
}
try {
const formData = new FormData();
if (!fields.userId || !fields.sessionId || !fields.exerciseId || !fields.task) {
throw new Error('Missing required fields');
}
formData.append('userId', fields.userId);
formData.append('sessionId', fields.sessionId);
formData.append('exerciseId', fields.exerciseId);
Object.keys(files).forEach(fileKey => {
const index = fileKey.split('_')[1];
for (const fileKey of Object.keys(files)) {
const indexMatch = fileKey.match(/^audio_(\d+)$/);
if (!indexMatch) {
console.warn(`Skipping invalid file key: ${fileKey}`);
continue;
}
const index = indexMatch[1];
const questionKey = `question_${index}`;
const audioFile = files[fileKey];
const binary = fs.readFileSync((audioFile as any).path);
formData.append(`audio_${index}`, binary, 'audio.wav');
formData.append(questionKey, fields[questionKey]);
fs.rmSync((audioFile as any).path);
});
if (!audioFile || !audioFile.path) {
throw new Error(`Invalid audio file for ${fileKey}`);
}
if (!fields[questionKey]) {
throw new Error(`Missing question for audio ${index}`);
}
try {
const buffer = fs.readFileSync(audioFile.path);
formData.append(`audio_${index}`, buffer, `audio_${index}.wav`);
formData.append(questionKey, fields[questionKey]);
fs.rmSync(audioFile.path);
} catch (fileError) {
console.error(`Error processing file ${fileKey}:`, fileError);
throw new Error(`Failed to process audio file ${index}`);
}
}
await axios.post(
`${process.env.BACKEND_URL}/grade/speaking/${fields.task}`,
@@ -54,9 +78,26 @@ async function handler(req: NextApiRequest, res: NextApiResponse) {
);
res.status(200).json({ ok: true });
} catch (error) {
console.error('Error processing request:', error);
res.status(500).json({
ok: false,
error: 'Internal server error'
});
Object.keys(files).forEach(fileKey => {
const audioFile = files[fileKey];
if (audioFile && audioFile.path && fs.existsSync(audioFile.path)) {
try {
fs.rmSync(audioFile.path);
} catch (cleanupError) {
console.error(`Failed to clean up temp file ${audioFile.path}:`, cleanupError);
}
}
});
}
});
}
export const config = {
api: {

View File

@@ -25,6 +25,7 @@ async function handler(req: NextApiRequest, res: NextApiResponse) {
return;
}
try {
const formData = new FormData();
formData.append('userId', fields.userId);
formData.append('sessionId', fields.sessionId);
@@ -32,9 +33,13 @@ async function handler(req: NextApiRequest, res: NextApiResponse) {
formData.append('question_1', fields.question);
const audioFile = files.audio;
const binary = fs.readFileSync((audioFile as any).path);
formData.append('audio_1', binary, 'audio.wav');
fs.rmSync((audioFile as any).path);
if (!audioFile || !audioFile.path) {
throw new Error('Audio file not found in request');
}
const buffer = fs.readFileSync(audioFile.path);
formData.append('audio_1', buffer, 'audio_1.wav');
fs.rmSync(audioFile.path);
await axios.post(
`${process.env.BACKEND_URL}/grade/speaking/2`,
@@ -48,6 +53,17 @@ async function handler(req: NextApiRequest, res: NextApiResponse) {
);
res.status(200).json({ ok: true });
} catch (error) {
console.error('Error:', error);
if (files.audio?.path && fs.existsSync(files.audio.path)) {
try {
fs.rmSync(files.audio.path);
} catch (e) {
console.error('Failed to cleanup file:', e);
}
}
res.status(500).json({ ok: false });
}
});
}

View File

@@ -64,8 +64,8 @@ const evaluateSpeakingExercise = async (
formData.append("exerciseId", exercise.id);
const evaluationQuestion = `${exercise.text.replaceAll("\n", "")}` + (exercise.prompts.length > 0 ? `You should talk about: ${exercise.prompts.join(", ")}` : "");
formData.append("question_1", evaluationQuestion);
formData.append("audio_1", audioFile, "audio.wav");
formData.append("question", evaluationQuestion);
formData.append("audio", audioFile, "audio.wav");
const config = {
headers: {