Merge pull request #2034 from codalab/cleanup_datagroup

Cleanup - Datagroup and related code removed

Author: Didayolo

src/apps/api/serializers/datasets.py

@@ -2,7 +2,7 @@
 from rest_framework.exceptions import ValidationError
 
 from api.mixins import DefaultUserCreateMixin
-from datasets.models import Data, DataGroup
+from datasets.models import Data
 from competitions.models import CompetitionCreationTaskStatus, CompetitionDump
 
 
@@ -223,14 +223,3 @@ def get_competition(self, obj):
 
     def get_owner_display_name(self, instance):
         return instance.created_by.display_name if instance.created_by.display_name else instance.created_by.username
-
-
-class DataGroupSerializer(serializers.ModelSerializer):
-    class Meta:
-        model = DataGroup
-        fields = (
-            'created_by',
-            'created_when',
-            'name',
-            'datas',
-        )

src/apps/api/urls.py

@@ -25,7 +25,6 @@
 router.register('phases', competitions.PhaseViewSet, 'phases')
 router.register('submissions', submissions.SubmissionViewSet)
 router.register('datasets', datasets.DataViewSet)
-router.register('data_groups', datasets.DataGroupViewSet)
 router.register('leaderboards', leaderboards.LeaderboardViewSet)
 router.register('submission_scores', leaderboards.SubmissionScoreViewSet, 'submission_scores')
 router.register('tasks', tasks.TaskViewSet)

src/apps/api/views/datasets.py

@@ -13,7 +13,7 @@
 
 from api.pagination import BasicPagination, LargePagination
 from api.serializers import datasets as serializers
-from datasets.models import Data, DataGroup
+from datasets.models import Data
 from competitions.models import CompetitionCreationTaskStatus
 from utils.data import make_url_sassy, pretty_bytes, gb_to_bytes
 
@@ -255,12 +255,6 @@ def public(self, request):
         return Response(serializer.data)
 
 
-class DataGroupViewSet(ModelViewSet):
-    queryset = DataGroup.objects.all()
-    serializer_class = serializers.DataGroupSerializer
-    # TODO: Anyone can delete these?
-
-
 @api_view(['PUT'])
 def upload_completed(request, key):
     # TODO: This view is weird. We have competitions, submissions, etc. that may not need to call this?

src/apps/datasets/admin.py

@@ -4,4 +4,3 @@
 
 
 admin.site.register(models.Data)
-admin.site.register(models.DataGroup)

src/apps/datasets/migrations/0012_delete_datagroup.py

@@ -0,0 +1,16 @@
+# Generated by Django 3.2 on 2025-10-14 09:38
+
+from django.db import migrations
+
+
+class Migration(migrations.Migration):
+
+    dependencies = [
+        ('datasets', '0011_auto_20250724_1050'),
+    ]
+
+    operations = [
+        migrations.DeleteModel(
+            name='DataGroup',
+        ),
+    ]

src/apps/datasets/models.py

@@ -142,10 +142,3 @@ def get_chahub_data(self):
             'is_public': self.is_public,
             'download_url': f'http{"s" if ssl else ""}://{site}{self.get_download_url()}'
         })
-
-
-class DataGroup(models.Model):
-    created_by = models.ForeignKey(settings.AUTH_USER_MODEL, on_delete=models.CASCADE)
-    created_when = models.DateTimeField(default=now)
-    name = models.CharField(max_length=255)
-    datas = models.ManyToManyField(Data, related_name="groups")

src/apps/profiles/views.py

@@ -24,7 +24,7 @@
 from oidc_configurations.models import Auth_Organization
 from .tokens import account_activation_token, account_deletion_token
 from competitions.models import Competition
-from datasets.models import Data, DataGroup
+from datasets.models import Data
 from tasks.models import Task
 from forums.models import Post
 from utils.email import codalab_send_mail
@@ -136,7 +136,6 @@ def send_user_deletion_notice_to_admin(user):
     competitions_participation = Competition.objects.filter(participants__user=user)
     submissions = user.submission.all()
     data = Data.objects.filter(created_by=user)
-    data_groups = DataGroup.objects.filter(created_by=user)
     tasks = Task.objects.filter(created_by=user)
     queues = user.queues.all()
     posts = Post.objects.filter(posted_by=user)
@@ -150,7 +149,6 @@ def send_user_deletion_notice_to_admin(user):
         'competitions_participation': competitions_participation,
         'submissions': submissions,
         'data': data,
-        'data_groups': data_groups,
         'tasks': tasks,
         'queues': queues,
         'posts': posts,

src/templates/profiles/emails/template_delete_account_notice.html

@@ -24,7 +24,6 @@ <h3>Summary</h3>
         <li>Competitions participation: {{ competitions_participation|length }}</li>
         <li>Submissions: {{ submissions|length }}</li>
         <li>Data: {{ data|length }}</li>
-        <li>DataGroups: {{ data_groups|length }}</li>
         <li>Tasks: {{ tasks|length }}</li>
         <li>Queues: {{ queues|length }}</li>
         <li>Posts: {{ posts|length }}</li>
@@ -83,13 +82,6 @@ <h4>Data created by the user</h4>
         {% endfor %}
     </ul>
 
-    <h4>DataGroups created by the user</h4>
-    <ul>
-        {% for data_group in data_groups.all %}
-            <li>{{ data_group }}</li>
-        {% endfor %}
-    </ul>
-
     <h4>Tasks created by the user</h4>
     <ul>
         {% for task in tasks.all %}

src/templates/profiles/emails/template_delete_account_notice.txt

@@ -24,7 +24,6 @@
         <li>Competitions participation: {{ competitions_participation|length }}</li>
         <li>Submissions: {{ submissions|length }}</li>
         <li>Data: {{ data|length }}</li>
-        <li>DataGroups: {{ data_groups|length }}</li>
         <li>Tasks: {{ tasks|length }}</li>
         <li>Queues: {{ queues|length }}</li>
         <li>Posts: {{ posts|length }}</li>
@@ -83,13 +82,6 @@
         {% endfor %}
     </ul>
 
-    <h4>DataGroups created by the user</h4>
-    <ul>
-        {% for data_group in data_groups.all %}
-            <li>{{ data_group }}</li>
-        {% endfor %}
-    </ul>
-
     <h4>Tasks created by the user</h4>
     <ul>
         {% for task in tasks.all %}