Drop-in Replacement of pychallonge

I get the following error in my script while trying to run on a Raspberry pi:

Traceback (most recent call last): File "Challonge Ticker.py", line 11, in import pychallonge ImportError: No module named pychallonge

I've installed pychallonge using pip install pychallonge

I'm trying to modify the following for use on my raspberry pi, but it keeps failing saying p1 = challonge.participants.show(tournamentURL, currentMatch["player1-id"])["name"]; KeyError: 'player1-id'

 `# Carmen "Beanwolf" Condeluci
 # 7/10/14

 from __future__ import print_function
 from shutil import move
 import challonge
 import sys
 import time
 import os
 import dateutil
 import pychallonge'

 print("Barebones Challonge Match Ticker Utility, ver 1.00")
 print("Made by Carmen \"Beanwolf\" Condeluci, most recent build published on 7/10/2014.\n")'

 print("This program pulls recently completed and/or upcoming matches from your Challonge hosted tournament and outputs a summary of the match to a text file. This can then be used with either OBS or XSplit's internal text scrolling tools, or an HTML/Flash source that can process text from and external file.\n")

 print("This file is saved as \"output.txt\" in the same directory as this program.")
 print("It will not appear until after one iteration has completed.\n")

 print("The number of completed and upcoming matches that will fit on the standard XSplit or OBS text scrolls is not too much. OBS, for example, tops out after about 4 completed matchs and 2 upcoming matches with a normal size font and some leading text. Keep this in consideration when working with these utilities.\n")

  challongeUsername = "MYUSERNAME"
  challongeAPIKey = "MYAPIKEY"

 challonge.set_credentials(challongeUsername, challongeAPIKey)

 # Retrieve a tournament by its id (or its url).
 tournamentURL = raw_input("\nPlease enter the tournament's URL indentifier.\n(\"challonge.com/xzr2pwvh\" would have a URL indentifier of \"xzr2pwvh\"): \n")

 tournament = challonge.tournaments.show(tournamentURL)

 matchesToShow = int(raw_input("\nEnter the number of completed matches you would like to show: "))
 upcomingToShow = int(raw_input("\nEnter the number of upcoming matches you would like to show: "))

 leadingText = raw_input("\nEnter any leading text that you would like to have on your ticker: \n")

 # Tournaments, matches, and participants are all represented as normal Python dicts.

 # Retrieve the participants for a given tournament.
 participants = challonge.participants.index(tournament["id"])

 print("\nThe program will now start in 5 seconds. It will continue to update indefinitely every minute until manually stopped with Ctrl + C.")
 time.sleep(5);

 #Overall loop would begin here
 while True:

print("Starting an interation...")

output = open('output.txt.tmp', 'w')

output.write(leadingText)

#shows user controlled amount of completed matches (higher rounds first)

output.write("  <|>  Most Recent Completed Matches:  ")

completedMatches = challonge.matches.index(tournament["id"], state="complete")
comLength = len(completedMatches)

# print(completedMatches)

if matchesToShow > comLength:
	for x in range(comLength, 0, -1):
		if x != comLength:
			output.write(" | ")

		currentMatch = completedMatches[x-1]

		comRound = str(currentMatch["round"])
		if "-" not in comRound:
			comRoundString = "W" + comRound
		else:
			comRoundString = "L" + comRound[1:]

		p1 = challonge.participants.show(tournamentURL, currentMatch["player1-id"])["name"];
		p2 = challonge.participants.show(tournamentURL, currentMatch["player2-id"])["name"]
		output.write(comRoundString + ": " + p1 + " vs. " + p2 + ", " + currentMatch["scores-csv"])
else:
	for x in range(comLength, (comLength - matchesToShow), -1):
		if x != comLength:
			output.write(" | ")

		currentMatch = completedMatches[x-1]

		comRound = str(currentMatch["round"])
		if "-" not in comRound:
			comRoundString = "W" + comRound
		else:
			comRoundString = "L" + comRound[1:]

		p1 = challonge.participants.show(tournamentURL, currentMatch["player1-id"])["name"];
		p2 = challonge.participants.show(tournamentURL, currentMatch["player2-id"])["name"]
		output.write(comRoundString + ": " + p1 + " vs. " + p2 + ", " + currentMatch["scores-csv"])

#Shows upcoming matches

output.write("  <|>  Upcoming Matches:  ")

pendingMatches = challonge.matches.index(tournament["id"], state="open")
pendLength = len(pendingMatches)

# print(pendingMatches)

if upcomingToShow > pendLength:
	for x in range(pendLength, 0, -1):
		if x != pendLength:
			output.write(" | ")

		pendingMatch = pendingMatches[x-1]

		penRound = str(pendingMatch["round"])
		if "-" not in penRound:
			penRoundString = "W" + penRound
		else:
			penRoundString = "L" + penRound[1:]

		p1 = challonge.participants.show(tournamentURL, pendingMatch["player1-id"])["name"];
		p2 = challonge.participants.show(tournamentURL, pendingMatch["player2-id"])["name"]
		output.write(penRoundString + ": " + p1 + " vs. " + p2)
else:
	for x in range(pendLength, (pendLength - upcomingToShow), -1):
		if x != pendLength:
			output.write(" | ")

		pendingMatch = pendingMatches[x-1]

		penRound = str(pendingMatch["round"])
		if "-" not in penRound:
			penRoundString = "W" + penRound
		else:
			penRoundString = "L" + penRound[1:]

		p1 = challonge.participants.show(tournamentURL, pendingMatch["player1-id"])["name"];
		p2 = challonge.participants.show(tournamentURL, pendingMatch["player2-id"])["name"]
		output.write(penRoundString + ": " + p1 + " vs. " + p2)

output.write("  <|>  ")

output.close();

move('output.txt.tmp', 'output.txt')

print("Finished an iteration!\n")

time.sleep(60)`

I modified the code from: https://github.com/CCondeluci/Simple-Challonge-Ticker

Any ideas on why this would not be working?

https://github.com/ZEDGR/pychal/blob/ebefcc8fc05798ce71f1199eb2481a4b99c84608/challonge/api.py#L121

The value of key 'display_name' in the dictionary returned from participants.index() is a float when the tournament participant's name passes float(v) without throwing ValueError. Thus in the returned dictionary every participant's name is of type str except those that can be interpreted as numbers, which are of type float.

FWIW, pychallonge does not have this issue.

Since the new update (1.11.1), I have been seeing a FAIL on main.ParticipantsTestCase, now.

Traceback (most recent call last):
  File "tests.py", line 213, in test_update
    self.assertTrue(p1["updated_at"] >= self.ps[0]["updated_at"])
AssertionError: False is not true

I've not yet dug in to it, however.

https://github.com/ZEDGR/pychal/issues/3

_parse now never casts certain response fields which are always strings to non-string types.

Version changed to 1.8.1.

After finally updating my fork of pychallonge to the latest most-official version, I think this change should be implemented in the official version. In challonge/api.py, I believe that line 157 should read:

params.append(("%s[%s][]" % (prefix, k), val))

When I originally wrote this section of code in June of 2015 (see this commit), the goal of it was to allow for tiebreaker rules (and the order of tiebreaker rules) for round-robin tournaments to be set via the API. To my knowledge, the attribute "tie_breaks" is the only attribute in the API where the value is a list. I have tested out my version of the code, and it does allow for the setting of tiebreaker rules. I have not tested out ZEDGR's code specifically, but I did attempt a more direct method of updating the API using ZEDGR's version of line 157 and I got an error. I would suggest attempting to run the following lines of code and seeing what happens:

import challonge t = challonge.tournaments.create("Test RR", "hut5wr9w", tournament_type="round robin", ranked_by="match wins", tie_breaks=["match wins vs tied","points difference","points scored"])

My current commit also contains a bulk_add unit test, which ZEDGR's doesn't, but that's less important.

Same program ran twice the timezone is different both times.

[ryan@ryanvile fgc]$ ./get_tournament.py | grep start 'group_stages_were_started': False, 'public_predictions_before_start_time': False, 'start_at': datetime.datetime(2017, 2, 28, 11, 35**, tzinfo=<FixedOffset '-05:00' datetime.timedelta(-1, 68400)>),** 'started_at': datetime.datetime(2017, 2, 28, 11, 43, 51, 214000, **tzinfo=<FixedOffset '-05:00' datetime.timedelta(-1, 68400)>),** 'started_checking_in_at': None, 'group_stages_were_started': False, 'public_predictions_before_start_time': False, 'start_at': datetime.datetime(2017, 2, 28, 13, 30, tzinfo=**<FixedOffset '-05:00' datetime.timedelta(-1, 68400)>),** 'started_at': datetime.datetime(2017, 2, 28, 16, 58, 58, 792000, **tzinfo=<FixedOffset '-05:00' datetime.timedelta(-1, 68400)>),** 'started_checking_in_at': None, 'group_stages_were_started': False, 'public_predictions_before_start_time': False, 'start_at': datetime.datetime(2017, 2, 28, 17, 35, tzinfo=**<FixedOffset '-05:00' datetime.timedelta(-1, 68400)>),** 'started_at': datetime.datetime(2017, 2, 28, 18, 2, 39, 53000, **tzinfo=<FixedOffset '-05:00' datetime.timedelta(-1, 68400)>),** 'started_checking_in_at': None, [ryan@ryanvile fgc]$ ./get_tournament.py | grep start 'group_stages_were_started': False, 'public_predictions_before_start_time': False, 'start_at': datetime.datetime(2017, 2, 28, 17, 35, tzinfo=**<FixedOffset '+01:00' datetime.timedelta(0, 3600)>),** 'started_at': datetime.datetime(2017, 2, 28, 17, 43, 51, 214000, **tzinfo=<FixedOffset '+01:00' datetime.timedelta(0, 3600)>),** 'started_checking_in_at': None, 'group_stages_were_started': False, 'public_predictions_before_start_time': False, 'start_at': datetime.datetime(2017, 2, 28, 19, 30, **tzinfo=<FixedOffset '+01:00' datetime.timedelta(0, 3600)>),** 'started_at': datetime.datetime(2017, 2, 28, 22, 58, 58, 792000, **tzinfo=<FixedOffset '+01:00' datetime.timedelta(0, 3600)>),** 'started_checking_in_at': None, 'group_stages_were_started': False, 'public_predictions_before_start_time': False, 'start_at': datetime.datetime(2017, 2, 28, 23, 35, tzinfo=**<FixedOffset '+01:00' datetime.timedelta(0, 3600)>),** 'started_at': datetime.datetime(2017, 3, 1, 0, 2, 39, 53000, **tzinfo=<FixedOffset '+01:00' datetime.timedelta(0, 3600)>),** 'started_checking_in_at': None,

In api.py the list of string values to treat as text do not include 'misc', which is provided in response to the participant.bulk_add() and index() calls. The API documentation states that 'misc' is user generated string value. Attempts to parse this as a datetime field can alter the fields value.

I might be missing something but in all but your GET requests, you don't send back any response. There is still useful information in the response for PUT and POST requests that at the very least would eliminate having to run a second API call to get the information and confirm what you sent.

Also always returning a response at least let's you write error handling if the PUT/POST fails or returns something you don't expect.