No, it's wrong. Your indentation for the output method makes it part of the constructor. Also you are creating objects of the class inside one of its methods (output). And you should use commas in print calls to separate fields rather than plus.
It should be something like:
class Nse():
def __init__(self, name, value):
self.name = name
self.value = value
def output(self):
print("Your stock name is", self.name, "Your stock value is", self.value)
j = Nse('TATA','34')
m = Nse('EDUCOMP','1')
j.output()
m.output()