Sentiments-of-Bundestag / graphenauswertung

0 stars 0 forks source link

SessionId missing in response #23

Closed abteilung6 closed 3 years ago

abteilung6 commented 3 years ago

Regarding your endpoint we miss the addtional field sessionId.

/factions/ranked

{
      "recipient": "F001",
      "sender": "F000",
      "sentiment": -1.399999976158142,
      "count": 10
      // missing
      sessionId: ""
    },

/factions/ranked

  {
    "name": "CDU/CSU", 
    "size": 246,
    "factionId": "F000",
    "rank": "0.7"
      // missing
      sessionId: ""
  }, 

etc.

Our interface expectations:


// Sentiment

// Completly Missing
export interface Session {
  sessionId: number;
  legislativePeriod: number;
  endDateTime: string;
  startDateTime: string;
}

export interface Faction {
  // Missing
  factionId: string;
  // Missing
  sessionId: number;
  name: string;
  size: number;
}

export interface FactionGraph {
  recipient: string;
  // Missing
  sessionId: number;
  sender: string;
  sentiment: number;
  // Missing
  count: number;
}

export interface FactionRanked {
  // Missing
  factionId: string;
  // Missing
  sessionId: string;
  name: string;
  size: number;
  rank: number;
}

export interface Person {
  speakerId: string;
  // Missing
  sessionId: number;
  name: string;
  rank: number;
  role: string;
}

// Completly missing
export interface PersonMessage {
  // Missing
  sessionId: number;
  sender: string;
  recipient: string;
  sentiment: number;
}

// Completly missing
export interface PersonGraph {
  // missing
  sessionId: number;
  sender: string;
  recipient: string;
  sentiment: number;
  count: number;
}

export interface PersonRanked {
  speakerId: number;
  // missing
  factionId: string;
  faction: string;
  name: string;
  role: string;
  rank: number;
}
glutengo commented 3 years ago

Thanks for your input. I understand most of the requirements and most of them are already in porgress.

Questions:

We noticed this morning that the group 4 database does currently not have any messages anymore, so I suppose, they are currently working on something there.

glutengo commented 3 years ago

@abteilung6

As mentioned in the other issues, we have deployed an updated version today. The endpoints are in line with the readme on the develop branch (https://github.com/Sentiments-of-Bundestag/graphenauswertung#endpoints).

I have also spoken to Group 4 and Group 5 about a few open issues:

I hope this helps you in some way. Please keep asking if there is anything else I can help with.

glutengo commented 3 years ago

will be resolved in #26