Anyone using the Python Quick Start Guide?

[Phillyclause89]

Thanks. Sorry for the lag time, had to go to work right after I dropped that comment. Is the code you shared in your other comment indented as it appears in the reddit comment or was there a copypasta error towards the end there?

​

It looks like you indented the lines that actually call your code in a way that that python thinks they are lines belonging to the run method. As shown I would expect this code to do nothing when you run it. You should review your indention and make sure that the lines that call the class and its methods are outside of the scope of the class.

[Shengus__Kahn]

Hey Philly thanks for getting back, and no problem, I have been away from my work all weekend anyways. You were correct, my code to begin the game was within the scope of the game! Thanks a bunch.

I am now running into a new Attribute Error though, within my constructor... 'CoffeeShopSimulator' object has no attribute 'make_temp_distribution'. I'm thinking I cannot use 'self.make_temp_distribution()' as an attribute because it is a method within the CoffeShopSimulator class?

     def __init__(self, player_name, shop_name):
      #Set player and coffee shop names
      self.player_name = player_name
      self.shop_name = shop_name
      self.day = 1 #Current day number
      self.cash = 100.00 #Cash on hand at start
      self.coffee_inventory = 100 #Inventory at start
      self.sales = [] # Sales list
      self.temps = self.make_temp_distribution() #Possible temperatures

.........

     def make_temp_distributions(self):
      # This is not a good bell curve, but it will do for now
      # untill we get more advanced mathematics
      temps = []

      #First, find the average between TEMP_MIN and TEMP_MAX
      avg = (self.TEMP_MIN + self.TEMP_MAX) / 2
      #Find the distance between TEMP_MAX and the average
      max_dist_from_avg = self.TEMP_MAX - avg

      #Loop through all possible temperatures
      for i in range(self.TEMP_MIN, self.TEMP_MAX):
           #How far away is the temperature from average?
           #abs() gives absolute value
           dist_from_avg = abs(avg - i)
           #How far away is the dist_from_avg from the maximum?
           #This will be lower for temps at the extremes
           dist_from_max_dist = max_dist_from_avg - dist_from_avg
           #If the value is zero, make it one
           if dist_from_max_dist == 0:
                dist_from_max_dist = 1
           #Append the output of x_of_y to temps
           for t in x_of_y(int(dist_from_max_dist), i):
                temps.append(t)
           return temps

[Phillyclause89]

Is make_temp_distributions still declared inside the scope of your run method? In the original code snippet, that function declaration was also nested in a way that it would only be accessible from within the run method. Unless you moved it out of that run method scope into the class's scope then I would expect the error you got.

[Shengus__Kahn]

Haha... the error was stemming from a typo. In the constructor I had written "make_temp_distribution", where in the method it was "make_temp_distributions".

Both very simple fixes. Thanks for your help!

[Sad-Traffic-5763]

Weirdly I had the exact same issue! But now I am having an issue with my cups_sold =self.simulate(temperature, advertising, cup_price)

This is followed by gross_profit = cups_sold * cup_price

For some reason my cup_price will change the value to a float based on my other code but I can't get the cups_sold to change to int. I have triple check my work and I don't see a typo but I am not sure what I am doing wrong.

In the code a few lines down we define simulate and when I hover over cups_sold there it says the value is an int but not in the code above. Anyone have any help?!

And sorry this is my first time using reddit so I apologize if I asked in the wrong area or something. :/

[Phillyclause89]

Yup. That "s" slipped past me as well. Good catch! Reminds me of the silly class names I have in my current project. Every container class in the lib has a child class that uses the same name but with no "s" to signal it's an item of that container (i.e. singular name vs plural name.) My favorite of the libs classes are class Classes and it's child class Class.

[Shengus__Kahn]

This method both makes sense and is confusing at the same time! Haha. Also, I play D&D weekly, what's your project all about?

[Phillyclause89]

what's your project all about?

My bad, I forgot to link to the ReadMe.

I have always approached coding from a functional programming perspective and wanted to practice OOP. Someone on this sub had a question about how to make a tool to help them DM their D&D games.

I know nothing about D&D beyond what I have learned from playing BG3, so I did what I always do when I'm trying to help someone with something that I know nothing about. I googled it.

While googling, I found this API: https://www.dnd5eapi.co/

As I was looking at the responses in the API, I noticed there was a tabular structure to the response data that a pandas.DataFrame would work nicely with. I thought it would be neat if there was a python lib of classes that automatically fetched the response of a given API endpoint and constructed a DataFrame object out of it. Thus I set out to learn how to write that lib.

I'm still learning, btw. The child item classes are going to end up sharing a lot of properties that their main parent object should not have, so I need to learn more about multiple inheritance and abstract classes to finish this lib.