jasig-ssp IRC Logs-2013-04-22

[13:39:43 CDT(-0500)] <Patty_> Unicon Press Release with SPC

[13:39:44 CDT(-0500)] <Patty_> http://www.unicon.net/content/st-petersburg-college-goes-live-student-success-plan-enlisting-unicon-services-and-support

[17:24:12 CDT(-0500)] <TonyUnicon> dmccallum54 question for you

[17:24:21 CDT(-0500)] <dmccallum54> ok

[17:24:31 CDT(-0500)] <TonyUnicon> Do we have some other mechanism besides the i0000XX-test.xml 's

[17:24:35 CDT(-0500)] <TonyUnicon> to seed external data?

[17:24:44 CDT(-0500)] <dmccallum54> oh boy do we ever

[17:24:52 CDT(-0500)] <TonyUnicon> heh

[17:24:56 CDT(-0500)] <TonyUnicon> can't wait to hear about it(smile)

[17:25:19 CDT(-0500)] <dmccallum54> ./src/SSP/scripts/testDataGenerator

[17:25:32 CDT(-0500)] <dmccallum54> that thing is going to need to be augmented to add MAP data

[17:25:40 CDT(-0500)] <JasonElwood> I built the latest on the linux CI instance, and it did seed the data either

[17:26:19 CDT(-0500)] <dmccallum54> we do not have existing liquibase scripts, other than tony's test changesets, that will populate MAP-related external tables

[17:26:40 CDT(-0500)] <dmccallum54> what we do have is a script that generates sample data in general

[17:26:54 CDT(-0500)] <dmccallum54> which at a minimum needs to be extended to add support for our new data types

[17:27:22 CDT(-0500)] <dmccallum54> but then i believe also might need some TLC if it's supposed to be able to generate data for specific, existing users

[17:27:38 CDT(-0500)] <TonyUnicon> no, this is external data

[17:28:23 CDT(-0500)] <dmccallum54> right. that script generates external data as well.

[17:28:44 CDT(-0500)] <dmccallum54> just not the external data you need at the moment

[17:29:36 CDT(-0500)] <dmccallum54> tyler's been working on that script most recently

[17:30:45 CDT(-0500)] <dmccallum54> and, oh, i think i see what you were meaning by "no this is external data"… i.e. that the external MAP data isn't user-specific?

[17:31:08 CDT(-0500)] <TonyUnicon> yes

[17:31:54 CDT(-0500)] <dmccallum54> yeah, so in that case, it should be relatively straightforward to add new data generators. for some of the items jason was talking about on the phone this morning tho, which are user-specific, there will probably need to be bigger changes to the script

[18:07:55 CDT(-0500)] <JasonElwood> The external_course.number column type is integer. This will cause problems for schools with course numbers like 190L, 210E, etc

[18:09:53 CDT(-0500)] <js701> specification says char pretty sure.

[18:09:57 CDT(-0500)] <dmccallum54> can you open a ticket for that, jason?

[18:10:16 CDT(-0500)] <JasonElwood> yeah. just wanted to confirm

[18:11:21 CDT(-0500)] <JasonElwood> it is char

[18:12:52 CDT(-0500)] <dmccallum54> 4:12p, that's the ideal time to start poking around in GPS code, right?

[18:14:08 CDT(-0500)] <JasonElwood> IMHO, any time is a good time

[18:14:49 CDT(-0500)] <dmccallum54> that didn't feel like sympathy

[18:16:00 CDT(-0500)] <JasonElwood> IMHO, anybody who gets started in GPS code at 4:12p is a friggin genius worthy of Queen playing Flash Gordon every time he walks in a room

[18:17:24 CDT(-0500)] <dmccallum54> i feel like if i just "walk" into such a room i'd be letting the room down

[18:17:59 CDT(-0500)] <JasonElwood> "He'll save every one of us!"

[18:18:20 CDT(-0500)] <dmccallum54> yeah…. should probably just give up now.

[18:19:48 CDT(-0500)] <JasonElwood> I forgot you guys are all like 30

[18:20:07 CDT(-0500)] <js701> not me, oldest man in the room