Skip to content

Commit

Permalink
Merge pull request CircuitVerse#147 from vedant-jain03/final_evaluation
Browse files Browse the repository at this point in the history
[documentation] : final report GSoC'22 Vedant Jain
  • Loading branch information
gr455 authored Sep 12, 2022
2 parents 78707dc + 99ff704 commit 1bb8187
Show file tree
Hide file tree
Showing 4 changed files with 153 additions and 0 deletions.
153 changes: 153 additions & 0 deletions content/posts/Vedant_jain_final_report.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,153 @@
---
title: "CircuitVerse Platform Improvement: Final Report (GSoC 2022)"
date: 2022-09-11T09:25:55+05:30
draft: false
author: Vedant Jain
type: post
---

![gsoc22_final_phase](/images/vedant_gsoc22/final_phase/final_phase.png)

This blog is about the work we have done and goals we have accomplished in the GSoC'22.

Let's Dive in...

## Project Description
CircuitVerse is a Digital Logic Simulator on web. It is an educational tool for students interested in electronics to simulate and make circuits on their browser.

This project aimed to improve the main platform by integrating new features. So intially the project consist of 6 features but after mentor discussion we have added more features into the project context.

## Project goals
The features we have successfully implemented:
- gitlab authentication integration
- sso integration
- Support multiple mentors for groups
- Improve embed feature
- Assignment restriction
- noticed integration (Notification Page)
- notification api(Mobile-app).

### Feature: gitlab authentication integration
This feature enables user to authenticate using thier gitlab account in CircuitVerse. For achieving this, I used `omniauth-gitlab` gem. Initially, I faced issues because of the old version of omniauth gem in CircuitVerse, so I first update the gem along with it's dependencies and configurations. After omniauth update, gitlab authentication works smoothly. I also updated the documentation to integration application for config.

**PR**
- [build(deps): bump omniauth from 1.9.1 to 2.1.0](https://github.com/CircuitVerse/CircuitVerse/pull/3071)
- [feat: gitlab authentication integration](https://github.com/CircuitVerse/CircuitVerse/pull/3154)

### Feature: SSO Integration

SSO stands for Single-Sign On. I have used `devise_saml_authenticatable` gem. For the testing I used Okta IDP. I faced issues during integration but as always mentor guided me the way, and I was able to solve it.

**PR**
- [feat: sso integration](https://github.com/CircuitVerse/CircuitVerse/pull/3167)

### Feature: Support Multiple Mentors Support in Group

This feature enables group to have multiple mentor. There will be 1 primary mentor(owner), and primary mentor can add mentors, make member a mentor and vise-versa. Mentors can grade, CRUD, and reopen assignments. Ruturaj has already raised a PR for it and had implemented mot of the stuffs, but there is still need to rafactor, cleanup and stimulus JS migration.

- [Documentation](https://docs.circuitverse.org/#/chapter2/2cvforeducators?id=add-group-mentors)

**Owner view:**
![Screenshot from 2022-06-21 21-43-00](https://user-images.githubusercontent.com/76901313/174847736-fa1a2706-7072-41b9-9809-d8892e2b6522.png)

**Mentor View:**
![Screenshot from 2022-06-21 21-43-18](https://user-images.githubusercontent.com/76901313/174847726-b36fb375-88d4-49a9-bb38-3992eda4b112.png)


**PR**
- [Feature: Support multiple mentors for groups](https://github.com/CircuitVerse/CircuitVerse/pull/2096)
- [[documenation] : Support Multiple Mentors](https://github.com/CircuitVerse/CircuitVerseDocs/pull/334)

### Feature: Improve embed feature
Initially the embed feature was basic. This feature advance it by providing the customization options. When a user click on embed button, a popup will be displayed with the options to customize your embed.
The customization options:
- Iframe width and height.
- Border width and color.
- Optionally Disable stuff like Clock Enable/Clock-Time, Interval/Zoom, Display Name.
- Change theme.

![embed](/images/vedant_gsoc22/embed_img1.png)

Kartikay already have made a PR with the integration for Iframe width & height, border with & color. I worked on integrating advance options, revamp and cleanup.


**PR**
- [Improve embed feature](https://github.com/CircuitVerse/CircuitVerse/pull/2768)

### Feature: Assignment restriction
We already have element restrictions for the assignment i.e., to restrict the group members with the use of elements for the Project.
But we have got quite a few complaints from professors that CircuitVerse allows cheating easily. For example, users can easily copy/paste any public circuit. They can give collaborators access to their own circuit and copy-paste etc. So it has been requested by professors to disable this feature. This PR enables that feature into CircuitVerse.

![assignment](/images/vedant_gsoc22/assignment_restriction_img1.png)

**PR**
- [feat: assignment restriction](https://github.com/CircuitVerse/CircuitVerse/pull/3091)

### Feature: Improve Notification Page

Notifications are very important aspect, we were using `activity_notification` gem before, but it makes the page lacky and the gem is not maintained any more, so we decided to shift to new gem, and we found `noticed` gem for it.

Noticed gem is really very easy to use and had really cool and amazing features. You can learn about the noticed gem [here](https://github.com/excid3/noticed).

We have covered 2 events for notification:
- Notify the author when someone forks the project.
- Notify the author when someone stars the project.

We have implemented a section in the navbar to get quick access to your notification.

![notification_2](/images/vedant_gsoc22/final_phase/notification_2.png)

#### Notification Page

Notification page for a respective user shows all the notification for the user logged in. You can mark a notification as read by clicking to it, or you can click **mark_all_as_read** to mark all notifications as read.

![notification_2](/images/vedant_gsoc22/final_phase/notification_1.png)

#### Challenges
The noticed implementation was comparatively easy, but the major challenge was to migrate the old data under the `activity_notifications` as both gem have different implementation, different schemas and methods. So we finally made a data migration file in which we made a script to migrate the data from `notification` table to `noticed_notification` table.

**PR**
- [feat: noticed integration (Notification Page)](https://github.com/CircuitVerse/CircuitVerse/pull/3243)

> We also reimplemented the web push notification, for which the PR is under review! There are some production issue we are facing, we are trying to solve it!
### Feature: Notification api(Mobile-app).

Notification api is used to provide the different endpoints for the Mobile-app.

#### Endpoints:

- GET `/api/v1/notifications`

To get all the notification for current authorized user.
- PATCH `/api/v1/notifications/mark_as_read/:notification_id`

To mark a particular notification as read.
- PATCH `/api/v1/notifications/mark_all_as_read`

To mark all notifications as read.

**PR**
- [feat: notifications api](https://github.com/CircuitVerse/CircuitVerse/pull/3259)

### Future Goals
We initially had some more features in the bucket list:
- Weekly Contest.
- Suggest tags for Project.

We will research and will work on these goals after gsoc!

### Blog posts written

- [Community Bonding Period at CircuitVerse — Google Summer of Code 2022](https://dev.to/vedantjain03/community-bonding-period-at-circuitverse-google-summer-of-code-2022-43cd)
- [GSoC 2022 CircuitVerse | Week 1 Report](https://dev.to/vedantjain03/gsoc-2022-circuitverse-week-1-report-4h88)
- [GSoC 2022 CircuitVerse | Week 2 Report](https://dev.to/vedantjain03/gsoc-2022-circuitverse-week-2-report-519a)
- [GSoC 2022 CircuitVerse | Week 3 Report](https://dev.to/vedantjain03/gsoc-2022-circuitverse-week-3-report-4af1)
- [GSoC 2022 CircuitVerse | Week 4 Report](https://dev.to/vedantjain03/gsoc-2022-circuitverse-week-4-report-bl0)
- [GSoC 2022 CircuitVerse | Week 5 and 6 Report](https://dev.to/vedantjain03/gsoc-2022-circuitverse-week-5-and-6-report-2kpp)
- [GSoC 2022 CircuitVerse | Week 7 and 8 Report](https://dev.to/vedantjain03/gsoc-2022-circuitverse-week-7-and-8-report-2a71)
- [GSoC 2022 CircuitVerse | Week 9 and 10 Report](https://dev.to/vedantjain03/gsoc-2022-circuitverse-week-9-and-10-report-3e4p)
- [GSoC 2022 CircuitVerse | Week 11 and 12 Report](https://dev.to/vedantjain03/gsoc-2022-circuitverse-week-11-and-12-report-4n41)

### Conclusion
By far we have accomplished 7/9 goals and that was under 12 weeks. We have man things lined up next.
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.

0 comments on commit 1bb8187

Please sign in to comment.