Skip to content

UofT-DSI | Shell - Assignment 1 #1

Open
alamfaiy wants to merge 8 commits intomainfrom
assignment
Open

UofT-DSI | Shell - Assignment 1 #1
alamfaiy wants to merge 8 commits intomainfrom
assignment

Conversation

@alamfaiy
Copy link
Owner

@alamfaiy alamfaiy commented Jan 12, 2025

What changes are you trying to make?

I am going through the required assignment steps as listed.
This includes creating directories, moving directories, listing contents,
creating multiple directories into a (not originally existing) parent directory,
copying server/user/event logs into their respective directories that I just created,
deleting files with sensitive information,
and finally outputting the files from all subdirectories into a text file.
Overall, these changes help to initially organize the client's data files.

What did you learn from the changes you have made?

The main thing I took away from this was how to go about doing these changes in the first place.
Before, coding and git and terminals etc. have always seemed so...intangible...particularly merging,
but now I understand how to go about it.

Was there another approach you were thinking about making?

For this assignment, not particularly. Very straightforward. Makes sense as this is the first assignment.

Were there any challenges?

Hmm, for the script itself, it was a relatively simple script.
I did learn about the brace expansion "{ }" notation which makes the code more concise.
I also learned how to merge changes and deal with conflicts. It was a bit weird to commit, or makes changes.
I struggled with the pull request. This is because I accidentally did everything in the main branch.
I had no idea how to reverse this, and tried, but ultimately decided to delete the repository and start from scratch.
In future, I will be much more careful with the branching and stuff. Once I fixed it, the pull request worked.

How were these changes tested?

First I uncommented that "remove data folder and everything in it" line, so I could actually look through data.
I followed the assignment_test_clean steps to test in a different folder.
I used bash in the clean directory to run the script, then I listed stuff within each of the folders to check that everything was contained that needed to be.
Specifically, that the directories were in the correct locations,
the main.py file was created in the correct location,
the server/user/event folders had the server/user/event files as needed,
and there were no ipaddr files.
--> EDIT: keeping the remove data folder line wasn't good, so once I saw the x's on the autograder, I went back and fixed it.

Checklist

  • I can confirm that my changes are working as intended
  • I have tested the script in a clean directory and verified that the desired outputs were achieved

@github-actions
Copy link

Autograder results

question status comment
1 data directory does not exist
2 rawdata not moved to data/raw
3
4 Missing directories: data/processed/server_logs, data/processed/user_logs, data/processed/event_logs
5 No server log files in data/raw
6 No user log files in data/raw; No event log files in data/raw
7
8 data/inventory.txt does not exist
9

@github-actions
Copy link

Autograder results

question status comment
1
2
3
4
5
6
7
8 data/inventory.txt does not exist
9

@github-actions
Copy link

Autograder results

question status
1
2
3
4
5
6
7
8
9

@ananyajha2000
Copy link

Assignment marked complete

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants