Skip to content

오류사항 수정#78

Merged
rktclgh merged 3 commits into
mainfrom
develop
Mar 21, 2026
Merged

오류사항 수정#78
rktclgh merged 3 commits into
mainfrom
develop

Conversation

@rktclgh

@rktclgh rktclgh commented Mar 21, 2026

Copy link
Copy Markdown
Owner

📢 기능 설명

필요시 실행결과 스크린샷 첨부

연결된 issue

연결된 issue를 자동으로 닫기 위해 아래 {이슈넘버}를 입력해주세요.

close #{이슈넘버}



🩷 Approve 하기 전 확인해주세요!

  • 리뷰어가 확인해줬으면 하는 사항 적어주세요.
  • [ ]

✅ 체크리스트

  • PR 제목 규칙 잘 지켰는가?
  • 추가/수정사항을 설명하였는가?
  • 이슈넘버를 적었는가?
  • Approve 하기 전 확인 사항 체크했는가?

@coderabbitai

coderabbitai Bot commented Mar 21, 2026

Copy link
Copy Markdown

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 3ab41d56-ac45-4044-bcfd-09b72c5a6702

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch develop

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@rktclgh rktclgh merged commit c9feff2 into main Mar 21, 2026
2 of 3 checks passed
@github-project-automation github-project-automation Bot moved this from Backlog to Done in Vlainter_BackEnd Mar 21, 2026
@gemini-code-assist

Copy link
Copy Markdown

Summary of Changes

Hello, I'm Gemini Code Assist1! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed!

이 Pull Request는 AI 기반 면접 질문 생성 및 강의 자료 처리와 관련된 여러 개선 사항을 포함합니다. AI가 생성하는 질문의 품질을 높이기 위해 의미론적 중복 질문을 감지하는 기능을 도입했으며, 생성된 요약문에서 전문 용어 보존을 검증하는 로직을 원문 자료의 길이에 따라 유연하게 조정하도록 개선했습니다. 또한, 강의 자료에서 시각 자료를 추출할 때 적용되던 개수 제한을 없애 자료 처리의 완전성을 높였습니다.

Highlights

  • 의미론적 중복 질문 감지: AI 생성 질문 수집 시, 단순히 동일한 텍스트가 아닌 의미론적으로 유사한 질문을 감지하여 중복을 방지하는 로직이 추가되었습니다.
  • 용어 보존 검증 로직 개선: 생성된 요약문에서 원문 전문 용어 보존 여부를 검증하는 로직이 개선되었습니다. 특히, 원문 입력 길이가 짧을 경우 검증 기준이 완화됩니다.
  • 시각 자료 추출 제한 제거: PDF, PPTX 등 강의 자료에서 시각 자료(이미지)를 추출할 때, 기존의 최대 개수 제한(12개)이 제거되어 모든 시각 자료를 추출할 수 있게 되었습니다.

🧠 New Feature in Public Preview: You can now enable Memory to help Gemini Code Assist learn from your team's feedback. This makes future code reviews more consistent and personalized to your project's style. Click here to enable Memory in your admin console.

Using Gemini Code Assist

The full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips.

Invoking Gemini

You can request assistance from Gemini at any point by creating a comment using either /gemini <command> or @gemini-code-assist <command>. Below is a summary of the supported commands on the current page.

Feature Command Description
Code Review /gemini review Performs a code review for the current pull request in its current state.
Pull Request Summary /gemini summary Provides a summary of the current pull request in its current state.
Comment @gemini-code-assist Responds in comments when explicitly tagged, both in pull request comments and review comments.
Help /gemini help Displays a list of available commands.

Customization

To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a .gemini/ folder in the base of the repository. Detailed instructions can be found here.

Limitations & Feedback

Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for GitHub and other Google products, sign up here.

Footnotes

  1. Review the Privacy Notices, Generative AI Prohibited Use Policy, Terms of Service, and learn how to configure Gemini Code Assist in GitHub here. Gemini can make mistakes, so double check it and use code with caution.

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

이 PR은 강의 자료 요약 시 전문용어 보존 검증 로직을 완화하고, 시험 문제 생성 시 의미적으로 중복되는 질문을 필터링하는 기능을 추가했습니다. 또한 문서에서 시각적 자료를 추출할 때 최대 개수 제한을 제거했습니다. 전반적으로 코드 품질을 개선하고 오류를 수정하는 좋은 변경입니다. 한 가지 제안 사항으로, courseExamIntentSignals 함수를 리팩토링하여 가독성과 유지보수성을 높일 수 있습니다.

Comment on lines +2058 to +2068
private fun courseExamIntentSignals(text: String): Set<String> {
val lowered = text.lowercase()
val intents = linkedSetOf<String>()
if (listOf("정의", "무엇", "define", "what is", "state").any { lowered.contains(it) }) intents += "DEFINITION"
if (listOf("설명", "서술", "기술", "explain", "describe").any { lowered.contains(it) }) intents += "EXPLAIN"
if (listOf("비교", "차이", "compare", "contrast", "difference").any { lowered.contains(it) }) intents += "COMPARE"
if (listOf("구하", "계산", "calculate", "compute", "find").any { lowered.contains(it) }) intents += "CALC"
if (listOf("구현", "작성", "코드", "implement", "write", "code").any { lowered.contains(it) }) intents += "BUILD"
if (listOf("맞으면", "틀리면", "고르", "선택", "choose", "select", "true or false").any { lowered.contains(it) }) intents += "SELECT"
return intents
}

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

medium

courseExamIntentSignals 함수는 여러 if 문을 사용하여 반복적인 로직을 가지고 있습니다. Map을 사용하여 의도(intent)와 키워드를 매핑하면 코드를 더 간결하고 유지보수하기 쉽게 만들 수 있습니다. 아래와 같이 리팩토링하는 것을 제안합니다.

    private fun courseExamIntentSignals(text: String): Set<String> {
        val lowered = text.lowercase()
        val intentKeywords = mapOf(
            "DEFINITION" to listOf("정의", "무엇", "define", "what is", "state"),
            "EXPLAIN" to listOf("설명", "서술", "기술", "explain", "describe"),
            "COMPARE" to listOf("비교", "차이", "compare", "contrast", "difference"),
            "CALC" to listOf("구하", "계산", "calculate", "compute", "find"),
            "BUILD" to listOf("구현", "작성", "코드", "implement", "write", "code"),
            "SELECT" to listOf("맞으면", "틀리면", "고르", "선택", "choose", "select", "true or false")
        )

        return intentKeywords
            .filter { (_, keywords) -> keywords.any { lowered.contains(it) } }
            .keys
    }

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

1 participant