Where communities thrive


  • Join over 1.5M+ people
  • Join over 100K+ communities
  • Free without limits
  • Create your own community
People
Activity
  • 21:28
    CircleCI success: oppia build (#129412) in https://github.com/oppia/oppia
    • updated explanation count limit function
      (3d52d958de187c08e5c32adbd5e5cfaa5a608173 by Hitesh Tomar)
  • 21:21
    lkbhitesh07 synchronize #14159
  • 21:14
    DubeySandeep unassigned #14022
  • 21:14
    DubeySandeep assigned #14022
  • 21:13
    DubeySandeep commented #14022
  • 20:41
    CircleCI success: oppia build (#129411) in https://github.com/oppia/oppia
    • changes
      (830f37e3eb2e349672cb8cb5f0810f2565cbc3b1 by heyimShivam)
  • 20:41
    oppiabot[bot] assigned #14359
  • 20:38
    heyimShivam commented #14359
  • 20:35
    heyimShivam synchronize #14359
  • 20:23
    CircleCI success: oppia build (#129410) in https://github.com/oppia/oppia
    • changes test case name
      (dab28e85ee1ab6ae0fd1aa1526e58de1bdbb46c3 by heyimShivam)
  • 20:20
    heyimShivam assigned #14347
  • 20:19
    heyimShivam review_requested #14347
  • 20:19
    heyimShivam commented #14347
  • 20:16
    heyimShivam synchronize #14347
  • 20:09
    CircleCI success: oppia build (#129409) in https://github.com/oppia/oppia
    • Fix Part of #9749: Migrates Collection-Player-Page to Angular (#12729)
      (2b718a0f2c33dc525606ee78d1867d041bbe98fc by GitHub)
  • 20:02

    DubeySandeep on develop

    Fix Part of #9749: Migrates Col… (compare)

  • 20:02
    DubeySandeep closed #12729
  • 19:25
    EricZLou commented #14159
  • 19:09
    oppiabot[bot] commented #14371
  • 19:09
    seanlip reopened #14371
ruinan-liu
@ruinan-liu
Hey Team,
I was working on adding the no .filter() linting rule and although it passes the python -m scripts.run_custom_eslint_tests test (Which means the lint I wrote is accurate?), it was showed error on the check python -m scripts.linters.pre_commit_linter --path core/tests/protractor_utils. It verified that some files in protractor_utils directory is using the .filter() function. Do we have an alternative for .filter()? And as part of the PR, should I also fix the files in /tests/protractor_utils?
Any guidance will be much appropriated.
Thanks Team!
2 replies
Mahendra
@mahendra1290

Screenshot from 2021-09-09 15-11-14.png

fron-tend tests failing while pushing can anybody help. I also tried to run tests on develop branch head these are also failing there

1 reply
Mahendra
@mahendra1290
Screenshot from 2021-09-09 18-37-49.png
Vishal Teotia
@vashuteotia123
chromeHeadless have not captured in 60000ms, killing
Facing this error and a bunch of lines after this, while pushing my code to my own branch.
10 replies
Shivam Jha
@ShivamJhaa
I am also getting the same error
ruinan-liu
@ruinan-liu
Same, getting the same error as above, attaching screenshot
I'm not sure, but the error seems to be that the expectations for the exceptions that are thrown are different?
ruinan-liu
@ruinan-liu
Screen Shot 2021-09-09 at 2.27.53 PM.png
3 replies
U8N WXD
@U8NWXD
A number of you have reported frontend test failures due to thrown errors that don’t match the expected errors. We are in the middle of trying to debug these problems right now. These errors are happening on lots of PRs and on develop, so if you see one of them, it’s probably not a problem with your code
1 reply
Shivam
@heyimShivam
done.png
project finally installed @U8NWXD thanks
1 reply
Vishal Teotia
@vashuteotia123
Those who were facing the error with frontend cases just update the develop branch and try again. As it is resolved for me.
Vishal Teotia
@vashuteotia123
I just created a PR, but I didn't had CLA signed. So I signed the CLA, now can anyone tell how I can trigger the checks again.
2 replies
Akshay Nandwana
@anandwana001

I am getting a python linter error on following line of code

                content=self.to_subtitled_html_proto(
                    self, state.content),

Error

Too many positional arguments for method call
6 replies
Akshay Nandwana
@anandwana001
    @classmethod
    def to_exploration_proto(
            cls, exploration_id, title, version, init_state_name, states):
        """Calculate the exploration size by setting exploration proto.

        Args:
            exploration_id: str. The id of the exploration.
            title: str. The exploration title.
            version: int. The version of the exploration.
            init_state_name: str. The name of the initial state.
            states: State. The state domain object.

        Returns:
            exploration_proto: Exploration. The exploration
                proto object.
        """
The above code giving error
4 space indentation in docstring.
5 replies
ruinan-liu
@ruinan-liu
Hey Team, Sorry to bring this up again, but I don't think the PR fixes my problem. I did a git fetch upstream develop and then git merge, so I do have the updated PR for the frontend test fix. However, when I run git push origin develop I encountered a similar error, I don't think this is related to my code, but could anyone double check?
Thanks!
1 reply
Screen Shot 2021-09-10 at 1.25.40 PM.png
Mahendra
@mahendra1290

Screenshot from 2021-09-11 02-46-05.png

unable to push even after updating with develop and passing all tests. Please help!

19 replies
Vishal Teotia
@vashuteotia123
image.png
Anyone know what this error states ?
1 reply
Vishal Teotia
@vashuteotia123
image.png
Shiqi Wu
@sophiewu6
Screen Shot 2021-09-11 at 5.44.07 PM.png
1 reply
I pulled the newest changes, but these two tests are still failing for me
Shivam Jha
@ShivamJhaa
image.png
How to remove this error?
2 replies
Mahendra
@mahendra1290
@ShivamJhaa just activate environment and start the project it will update any dependencies
Soumyajyoti Dey
@soumyo123-prog
Can someone please restart tests on (this PR)[oppia/oppia#13805]. Backend tests were cancelled after I merged the latest develop into my branch
1 reply
Akshay Nandwana
@anandwana001
Who could be a better person to reach out about exp_domain_test file on writing a new set of test cases?
1 reply
Srikanth K B
@srikanthkb
image.png
Hi,
Has anyone had the frontend test failing with the above error ?
2 replies
Shivam
@heyimShivam
error.png
while pushing branch to github this error appears
can any one help me with this ?
36 replies
Shivam
@heyimShivam
@U8NWXD
chrome Headless error
Shivam
@heyimShivam
I actually tried 5-6 times
Mahendra
@mahendra1290
okay
Shivam
@heyimShivam
4.png
5.png
1.png
2.png
3.png
0.png
@U8NWXD
Shivam
@heyimShivam
Okay i will try that tomorrow , thanks for your kind help .
Lucifer
@ashish-patwal
Hey guys i am getting these lint errors when pushing my code . It seems confusing all the files in scripts dir contains " quotes instead of ' quotes , yet it is particularly pointing errors to my modified files . I wanted to confirm if the linters have been updated for python , and we are using PEP8 principles from now on ?
Starting Linter....
----------------------------------------
----------------------------------------
05:45:18 FINISHED General Purpose Linter: 0.0 secs
----------------------------------------
----------------------------------------
Tasks still running:
  Custom Lint Checks Manager (started 11:15:18)
  Codeowner Lint Checks Manager (started 11:15:18)
  General Purpose Linter (started 11:15:18)
----------------------------------------
05:45:18 FINISHED General Purpose Linter: 0.0 secs
05:45:18 FINISHED Custom Lint Checks Manager: 0.3 secs
05:45:39 FINISHED Codeowner Lint Checks Manager: 21.8 secs
----------------------------------------
Tasks still running:
  Third Party Python Lint Checks Manager (started 11:15:39)
----------------------------------------
05:45:44 FINISHED Third Party Python Lint Checks Manager: 4.9 secs
----------------------------------------
Please fix the errors below:
----------------------------------------
************* Module scripts.common
scripts/common.py:110:0: C0301: Line too long (88/80) 
scripts/common.py:138:0: C0301: Line too long (83/80) 
scripts/common.py:182:0: C0301: Line too long (86/80) 
scripts/common.py:183:0: C0301: Line too long (82/80) 
scripts/common.py:184:0: C0301: Line too long (84/80) 
scripts/common.py:186:0: C0301: Line too long (88/80) 
scripts/common.py:187:0: C0301: Line too long (81/80) 
scripts/common.py:226:0: C0301: Line too long (81/80) 
scripts/common.py:287:0: C0301: Line too long (81/80) 
scripts/common.py:317:0: C0301: Line too long (84/80) 
scripts/common.py:325:0: C0301: Line too long (84/80) 
scripts/common.py:333:0: C0301: Line too long (87/80) 
scripts/common.py:351:0: C0301: Line too long (86/80) 
scripts/common.py:437:0: C0301: Line too long (84/80) 
scripts/common.py:491:0: C0301: Line too long (87/80) 
scripts/common.py:494:0: C0301: Line too long (81/80) 
scripts/common.py:505:0: C0301: Line too long (84/80) 
scripts/common.py:579:0: C0301: Line too long (86/80) 
scripts/common.py:582:0: C0301: Line too long (83/80) 
scripts/common.py:701:0: C0301: Line too long (84/80) 
scripts/common.py:36:0: C4001: Invalid string quote ", should be ' 
scripts/common.py:36:0: C4001: Invalid string quote ", should be ' 
scripts/common.py:36:0: C4001: Invalid string quote ", should be ' 
scripts/common.py:41:0: C4001: Invalid string quote ", should be ' 
scripts/common.py:42:0: C4001: Invalid string quote ", should be ' 
scripts/common.py:43:0: C4001: Invalid string quote ", should be ' 
scripts/common.py:44:0: C4001: Invalid string quote ", should be ' 
scripts/common.py:45:0: C4001: Invalid string quote ", should be ' 
scripts/common.py:46:0: C4001: Invalid string quote ", should be ' 
scripts/common.py:47:0: C4001: Invalid string quote ", should be ' 
scripts/common.py:48:0: C4001: Invalid string quote ", should be ' 
scripts/common.py:49:0: C4001: Invalid string quote ", should be ' 
scripts/common.py:50:0: C4001: Invalid string quote ", should be ' 
scripts/common.py:51:0: C4001: Invalid string quote ", should be ' 
scripts/common.py:52:0: C4001: Invalid string quote ", should be ' 
scripts/common.py:53:0: C4001: Invalid string quote ", should be ' 
scripts/common.py:54:0: C4001: Invalid string quote ", should be ' 
scripts/common.py:57:0: C4001: Invalid string quote ", should be ' 
scripts/common.py:60:0: C4001: Invalid string quote ", should be ' 
scripts/common.py:63:0: C4001: Invalid string quote ", should be ' 
scripts/common.py:66:0: C4001: Invalid string quote ", should be ' 
scripts/common.py:86:0: C4001: Invalid string quote ", should be ' 
scripts/common.py:87:0: C4001: Invalid string quote ", should be ' 
scripts/common.py:89:0: C4001: Invalid string quote ", should be '
3 replies
Pranshu Srivastava
@PranshuSrivastava
Hey, my PR hasn't been reviewed in a week van someone check it out please? oppia/oppia#13768
7 replies
Nihir Agarwal
@nihirag
Hey everyone! I am currently a pre-final year university student. I want to contribute to the oppia project. Can someone just show me the right direction from where I can start ?
1 reply
Pranshu Srivastava
@PranshuSrivastava
image.png
And can someone help me with this
2 replies