bug: python return annotations

[quantum-booty]

class Test:
  def __str__(self) -> str:
      """
  
      Parameters
      ----------
      self: 
          
  
      Returns
      -------
      config_str : 
          
      strip : 
          
      """
      config_str = ""
      for row in self.tile:
          row_str = "".join(row)
          config_str += row_str + "\n"
      return config_str.strip()

return config_str.strip() should be anonymous, and the self should not be a parameter, as it is a method.
This is what I expect:

class Test:
  def __str__(self) -> str:
      """
      Returns
      -------
      str:
          
      """
      config_str = ""
      for row in self.tile:
          row_str = "".join(row)
          config_str += row_str + "\n"
      return config_str.strip()

[danymat]

Related: #41 (comment)

[danymat]

I think I just added support for type hints, and fixed the bug at the same time
Can you try it out ?

[quantum-booty]

Thanks a lot @danymat!
This is what I'm getting, I think self should not be a parameter

    def __str__(self) -> str:
        """

        Parameters
        ----------
        self: 
        	

        Returns
        -------
        str
        	
        """
        config_str = ""
        for row in self.tile:
            row_str = "".join(row)
            config_str += row_str + "\n"
        return config_str.strip()

[quantum-booty]

I think the point about self should also apply to classmethods

    @classmethod
    def test(cls) -> None:
        """

        Parameters
        ----------
        cls: 
        	

        Returns
        -------
        None
        	
        """
        pass

I expect that cls should not be a parameter.

[quantum-booty]

Also I think -> None should not generate a return type, because that is the default value thats being returned by a function.
So rather than this:

def test(a: int) -> None:
    """

    Parameters
    ----------
    a: int
    	

    Returns
    -------
    int
    	
    None
    	
    """
    pass

I don't expect it to generate a doc for None returns.

def test(a: int) -> None:
    """

    Parameters
    ----------
    a: int

    """
    pass

[danymat]

• None should now be hidden
• reference to self is now removed

[danymat]

I think the point about self should also apply to classmethods

    @classmethod
    def test(cls) -> None:
        """

        Parameters
        ----------
        cls: 
        	

        Returns
        -------
        None
        	
        """
        pass

I expect that cls should not be a parameter.

This seems to be a little bit difficult to do, I don't think I will do it tonight

[danymat]

Hello, I think that commit d46c438 did provide a very great handling of such self, clscases. Can you take a look and try again ?

[quantum-booty]

class Test:
    @staticmethod
    def test2(a: int, b: str):
        """

        Parameters
        ----------
        a: int
            
        b: str
            

        Returns
        -------
        int
            
        str
            
        """
        pass
        

    @classmethod
    def test(cls, a: int, b: str):
        """

        Parameters
        ----------
        a: int
            
        b: str
            

        Returns
        -------
        int
            
        str
            
        """
        pass
        

    def test1(self, a: int, b: str):
        """

        Parameters
        ----------
        a: int
            
        b: str
            

        Returns
        -------
        int
            
        str
            
        """
        pass
       

Thanks @danymat I think that did fix the self and cls cases.
I think I've found another bug, some how its generating return int and str types.

[danymat]

Last commit should fix it, a single line that did this lol. Is everything ok ?

[quantum-booty]

nice xD good work!