jasig-ssp IRC Logs-2013-04-30

[08:30:55 CDT(-0500)] <TonyUnicon> you're up early

[12:06:27 CDT(-0500)] <js70> so i need to do a ant initportal with the latest stuff that will solve the map loading issue?

[12:08:44 CDT(-0500)] <JasonElwood> for permissions, you can run the import described in SSP-920

[12:08:59 CDT(-0500)] <JasonElwood> ant -Dmaven.test.skip=true -Ddir=uportal-war/src/main/data/ssp_entities/patches-SSP-2-0-0/SSP-920/ data-import

[12:09:11 CDT(-0500)] <JasonElwood> but you have to sync the uPortal branch to get the change

[12:09:32 CDT(-0500)] <js70> k. thks.

[12:41:18 CDT(-0500)] <js70> jason, fixes are in. also adjusted the size of the semester/term panels. How difficult is it to make the work area longer. we really need that to get in a full 4 years.

[12:42:18 CDT(-0500)] <JasonElwood> first, you reduced the available courses in a term to 6 courses to save space?

[12:43:30 CDT(-0500)] <js70> yes

[12:44:26 CDT(-0500)] <JasonElwood> cool. second, I wouldn't be worried about seeing the full 4 years in the initial display. It will not be a primary focus and will shift up as it gets closer

[12:44:50 CDT(-0500)] <js70> k

[12:46:28 CDT(-0500)] <js70> just looking to get rid of as many scrollbars as possible…trying to save you money.:^)

[12:47:40 CDT(-0500)] <JasonElwood> It may save scrolling initially, but once courses are filled in scrolling will probably be inevitable. The only other possibility would be to collapse the years so years lower in the screen could move up

[12:47:42 CDT(-0500)] <TonyUnicon> hah

[13:01:05 CDT(-0500)] <js70> hey dan, just to confirm. In all cases, (except a specific bug to a branch) we are to do are work to master and then cherry pick to other branches?

[13:11:00 CDT(-0500)] <TonyUnicon> this behavior where hibernate commits state of all trackable objects inside a transaction is new to me

[13:12:15 CDT(-0500)] <TonyUnicon> its what is causing the plan dupes

[14:00:41 CDT(-0500)] <TonyUnicon> Jason, these remaining bugs I'm still working on, a little trickier than I thought but manageable

[14:00:50 CDT(-0500)] <TonyUnicon> I will have them in soon hopefully

[14:01:21 CDT(-0500)] <JasonElwood> cool. thanks man

[14:01:27 CDT(-0500)] <JasonElwood> Jim has been keeping me busy

[14:01:44 CDT(-0500)] <js70> in a good way.:^)

[14:01:53 CDT(-0500)] <TonyUnicon> he putting the pressure on me to wrap up he is so psyched for the next phase

[14:02:17 CDT(-0500)] <js70> I just lurve me some typing...

[14:02:42 CDT(-0500)] <js70> mine were pretty simple.

[14:04:30 CDT(-0500)] <TonyUnicon> Jason something else I noticed while working on this is

[14:04:45 CDT(-0500)] <TonyUnicon> in a scenario when we copy

[14:04:55 CDT(-0500)] <TonyUnicon> the plan name changes on the old plan too

[14:04:57 CDT(-0500)] <TonyUnicon> which is bad

[14:05:05 CDT(-0500)] <TonyUnicon> against because of this auto commit business

[14:05:12 CDT(-0500)] <JasonElwood> definitely bad

[14:05:21 CDT(-0500)] <JasonElwood> is that what happened instead of creating duplicates?

[14:06:07 CDT(-0500)] <TonyUnicon> yeah it creates the appearance of dupes because the modified time stamps and names are the same

[14:06:32 CDT(-0500)] <TonyUnicon> so now I have to code around this autocommit

[14:09:17 CDT(-0500)] <TonyUnicon> even marking the reference as final does not prevent the object from being modified

[14:09:23 CDT(-0500)] <TonyUnicon> probably because its done via reflection

[14:12:16 CDT(-0500)] <TonyUnicon> thiiink i might have it

[14:31:41 CDT(-0500)] <TonyUnicon> hooray

[14:32:05 CDT(-0500)] <js70> hiphip!

[14:32:11 CDT(-0500)] <tbain> hooray

[14:32:12 CDT(-0500)] <TonyUnicon> hooray!

[14:32:17 CDT(-0500)] <tbain> hip hip

[14:32:22 CDT(-0500)] <TonyUnicon> hooray!

[14:32:34 CDT(-0500)] <TonyUnicon> this is sans whiskey too

[14:32:41 CDT(-0500)] <js70> I love the smell of napalm in the morning...

[14:32:44 CDT(-0500)] <JasonElwood> nice

[14:33:18 CDT(-0500)] <TonyUnicon> alllllright jason, fix should be comin your way soon

[14:33:48 CDT(-0500)] <js70> just the way my mind works. best to stay out of it.

[14:39:11 CDT(-0500)] <TonyUnicon> checked in jason

[14:39:47 CDT(-0500)] <JasonElwood> Are you guys expecting to check anything in soon?

[14:41:25 CDT(-0500)] <TonyUnicon> not unless you find another bug

[14:41:28 CDT(-0500)] <TonyUnicon> I think I'm all in

[14:48:33 CDT(-0500)] <js70> got https://issues.jasig.org/browse/SSP-1101 done can check in if you want. but otherwise I'm done.

[14:48:43 CDT(-0500)] <JasonElwood> go ahead. thanks

[14:59:11 CDT(-0500)] <js70> checked in.

[15:28:07 CDT(-0500)] <JasonElwood> databasechangeloglock = my nemesis

[15:28:31 CDT(-0500)] <TonyUnicon> yeah

[15:28:40 CDT(-0500)] <TonyUnicon> i shake my first in the air everytime it happens

[15:53:51 CDT(-0500)] <JasonElwood> JS- the reduction in the number of courses in a term without scrolling change affected the term name and term cr hrs constant. They can now be hidden with the scroll

[15:54:46 CDT(-0500)] <JasonElwood> Also, the max number of courses, on my screen, is 4. on the 5th, the scroll appears. it the size based on number of courses or height

[15:59:25 CDT(-0500)] <js70> crap. looks good on my machine.

[16:00:02 CDT(-0500)] <js70> also seeing 6 courses.

[16:00:07 CDT(-0500)] <js70> weird.

[16:00:55 CDT(-0500)] <JasonElwood> emailed

[16:00:58 CDT(-0500)] <JasonElwood> this is in Firefox

[16:01:56 CDT(-0500)] <js70> k

[16:03:14 CDT(-0500)] <js70> 2 out of 3 ain't bad...

[16:05:07 CDT(-0500)] <JasonElwood> yeah. and this one is tricky

[16:33:50 CDT(-0500)] <dmccallum54> going to re-delete https://github.com/Jasig/SSP/tree/rel-2-0-0-b1-patches. looks like https://github.com/Jasig/SSP/tree/rel-2-0-0-b1-patches has all the commits that were pushed to the former today

[16:35:40 CDT(-0500)] <TonyUnicon> Dan, I am rolling back the legacy remarks, document tool commits on the branches and leave them on master

[16:35:45 CDT(-0500)] <TonyUnicon> and disable them in the javascript

[16:35:51 CDT(-0500)] <dmccallum54> ok

[16:36:45 CDT(-0500)] <TonyUnicon> what version of uportal are we using for this release? whatever is in master now?

[16:37:12 CDT(-0500)] <TonyUnicon> i just released we dont have a 2.0 uportal branch

[16:37:27 CDT(-0500)] <dmccallum54> the "beta 1" uportal branch is https://github.com/Jasig/uPortal/tree/ssp-rel-1-2-0-patches

[16:37:35 CDT(-0500)] <TonyUnicon> ok perfect

[16:37:42 CDT(-0500)] <dmccallum54> "perfect"

[16:37:52 CDT(-0500)] <TonyUnicon> heh

[16:39:10 CDT(-0500)] <TonyUnicon> luckily I've already reverted that commit because I thought it was not being used for this release

[16:40:14 CDT(-0500)] <TonyUnicon> still have to revert it on the ssp branches

[16:40:15 CDT(-0500)] <js70> Jason, fixed the panel sizes, should work ok now.

[16:40:43 CDT(-0500)] <dmccallum54> SSP-1097 is that done? i see it's been cherry picked

[16:41:31 CDT(-0500)] <JasonElwood> Tony did part of the ticket

[16:42:33 CDT(-0500)] <dmccallum54> so done as far as 2.0.0 (beta 1) is concerned?

[16:42:42 CDT(-0500)] <JasonElwood> yep

[16:42:47 CDT(-0500)] <dmccallum54> k thx

[16:43:03 CDT(-0500)] <js70> https://issues.jasig.org/browse/SSP-942 is still open, is there a reason. Says changed version. You just going to add requirements in next version.

[16:44:07 CDT(-0500)] <JasonElwood> I just closed it a few minutes ago

[16:44:46 CDT(-0500)] <js70> kool. speed of electrons and all that (switching and stuff) always a lag.

[16:45:09 CDT(-0500)] <js70> thks

[16:45:54 CDT(-0500)] <JasonElwood> does 920 need to be merged into b1?

[16:45:57 CDT(-0500)] <JasonElwood> Tony

[16:46:42 CDT(-0500)] <dmccallum54> 920 is already there

[16:46:48 CDT(-0500)] <JasonElwood> cool

[16:47:07 CDT(-0500)] <TonyUnicon> yeah ive been merging everything

[16:47:09 CDT(-0500)] <TonyUnicon> if thats ok?

[16:48:06 CDT(-0500)] <JasonElwood> yeah. I've just been adding labels for merge but don't need to for your's

[16:51:48 CDT(-0500)] <dmccallum54> yeah, thanks for doing that Tony

[16:52:16 CDT(-0500)] <dmccallum54> i'm going back through now and filling in other cherry-picks and removing Jira labels where it's already been done

[16:52:31 CDT(-0500)] <JasonElwood> I've seen a few emails

[16:52:37 CDT(-0500)] <dmccallum54> i bet

[16:52:50 CDT(-0500)] <TonyUnicon> heh

[16:53:25 CDT(-0500)] <TonyUnicon> hmm did just get an error when loading transcripts

[16:53:27 CDT(-0500)] <TonyUnicon> using the latest

[16:54:08 CDT(-0500)] <TonyUnicon> still an issue?

[16:54:09 CDT(-0500)] <TonyUnicon> java.lang.NullPointerException: null

[16:54:09 CDT(-0500)] <TonyUnicon> at org.jasig.ssp.transferobject.external.ExternalStudentRecordsLiteTO.<init>(ExternalStudentRecordsLiteTO.java:37) ~[ExternalStudentRecordsLiteTO.class:na]

[16:54:09 CDT(-0500)] <TonyUnicon> at org.jasig.ssp.web.api.external.ExternalStudentRecordsController.loadSummaryStudentRecords(ExternalStudentRecordsController.java:209) ~[ExternalStudentRecordsController.class:na]

[16:54:09 CDT(-0500)] <TonyUnicon> at org.jasig.ssp.web.api.external.ExternalStudentRecordsController$$FastClassByCGLIB$$f5960447.invoke(<generated>) ~[cglib-2.2.2.jar:na]

[16:54:09 CDT(-0500)] <TonyUnicon> at net.sf.cglib.proxy.MethodProxy.invoke(MethodProxy.java:204) ~[cglib-2.2.2.jar:na]

[16:56:20 CDT(-0500)] <TonyUnicon> Jason I'm only commit the code to disable the tools to the branch

[16:56:22 CDT(-0500)] <TonyUnicon> and not master

[16:56:24 CDT(-0500)] <TonyUnicon> that is ok?

[16:56:32 CDT(-0500)] <TonyUnicon> committing*

[16:56:38 CDT(-0500)] <dmccallum54> makes sense to me

[16:56:42 CDT(-0500)] <JasonElwood> yep

[16:56:45 CDT(-0500)] <TonyUnicon> ok

[16:57:17 CDT(-0500)] <JasonElwood> I'm not getting the NPE, and I just build from source 10 mins ago

[16:57:30 CDT(-0500)] <TonyUnicon> ok

[16:57:58 CDT(-0500)] <js70> thats weird.

[16:58:11 CDT(-0500)] <TonyUnicon> maybe something on my end, but it wasn't happening before

[16:58:14 CDT(-0500)] <TonyUnicon> of course I was building off master

[16:58:23 CDT(-0500)] <TonyUnicon> this is a build of branch

[16:58:25 CDT(-0500)] <JasonElwood> I built off master

[16:58:33 CDT(-0500)] <TonyUnicon> err

[16:58:44 CDT(-0500)] <TonyUnicon> you should probably build on branch

[16:58:49 CDT(-0500)] <TonyUnicon> no?

[16:59:18 CDT(-0500)] <JasonElwood> are there any differences right now?

[16:59:38 CDT(-0500)] <dmccallum54> Tony ssp-2-0-0-b1-patches is a zombie branch. i'm deleting it. will apply your SSP-1003 revert on the non-zombie ssp-2-0-0-patches

[17:00:17 CDT(-0500)] <TonyUnicon> yeah non-zombie is my current

[17:00:20 CDT(-0500)] <tbain> A question: On the Color screen the code field is a varchar of length 10, but some of the default data is longer than 10 characters (e.g. "Light Green"), should I abbreviate the codes on the default values, or should I lengthen it to, say, 15 or 20 characters?

[17:00:29 CDT(-0500)] <TonyUnicon> but if that's where we are cutting the release shouldn't we be building off that branch?

[17:01:01 CDT(-0500)] <dmccallum54> that a question for me?

[17:01:09 CDT(-0500)] <TonyUnicon> for anyone who is listening

[17:01:21 CDT(-0500)] <JasonElwood> go ahead and make the column larger. 35 or 50 to be safe

[17:01:57 CDT(-0500)] <tbain> Do you have a preference for 35 or 50, or some value twixt the two?

[17:02:06 CDT(-0500)] <dmccallum54> JasonElwood should probably switch to rel-2-0-0-patches

[17:02:18 CDT(-0500)] <js70> lumenisent mavue with a slight pinkish undertone...

[17:02:22 CDT(-0500)] <js70> will that fit?

[17:02:35 CDT(-0500)] <js70> pay no attention to the spelling

[17:02:41 CDT(-0500)] <JasonElwood> 50 is fine. I'm a big Grant fan.

[17:02:47 CDT(-0500)] <tbain> kk dokie

[17:03:27 CDT(-0500)] <dmccallum54> TonyUnicon ok. pushed the revert of 8e68313369e459bf7b1d7cbb78f4b2b682b61694 to rel-2-0-0-patches. deleting the zombie branch for reals now

[17:03:38 CDT(-0500)] <tbain> @js70 - that'll fit fine in the new size!

[17:03:40 CDT(-0500)] <TonyUnicon> thanks

[17:04:12 CDT(-0500)] <js70> then I'm good.

[17:06:17 CDT(-0500)] <JasonElwood> I can clone 1097 if it's easier to split the ticket

[17:06:27 CDT(-0500)] <JasonElwood> JS- scrolling looks better. thanks

[17:16:35 CDT(-0500)] <dmccallum54> ok. sourcetree thinks rel-2-0-0-patches is all caught up for now and my Jira search for that label is returning the empty set

[17:16:47 CDT(-0500)] <JasonElwood> I built from patches and I'm not getting the error

[17:17:58 CDT(-0500)] <dmccallum54> for 1097 … splitting is probably a good idea. else that ticket won't show up in 2.0.0's release notes.

[17:18:22 CDT(-0500)] <dmccallum54> (until it's completely resolved in 2.0.1, i mean)

[17:21:30 CDT(-0500)] <JasonElwood> 1097 is for b1, 1115 is for b2

[17:21:41 CDT(-0500)] <dmccallum54> sorry.. back to the "codes" question for colors...

[17:22:35 CDT(-0500)] <dmccallum54> codes are supposed to be terse. they're mainly to support references from loosely coupled subsystems or external systems entirely

[17:22:50 CDT(-0500)] <dmccallum54> so "Light Green" would be a good name

[17:23:18 CDT(-0500)] <dmccallum54> "LITE_GRN" or something like that would be a good code

[17:24:41 CDT(-0500)] <tbain> So should I go back and abbreviate the code(s) then, or just keep it as 50 chars?

[17:25:52 CDT(-0500)] <dmccallum54> unfortunately, that would be my vote, yes.

[17:26:09 CDT(-0500)] <dmccallum54> and it would match the convention established in the 'elective' table

[17:26:20 CDT(-0500)] <JasonElwood> yep

[17:27:47 CDT(-0500)] <dmccallum54> and no spaces in our default codes, pls. we place no restrictions on what you can enter via the UI. but in general i think we want to set a precedent where these things are trivial to set as a URL param

[17:28:34 CDT(-0500)] <tbain> kk dokie, that'll take all of a few minutes

[17:28:35 CDT(-0500)] <dmccallum54> sorry tyler, i should have caught that in my read through of the spec

[17:29:07 CDT(-0500)] <tbain> would lt_green and lt_blue be acceptable? And for that matter, should the other codes (e.g. marked as Orange or Purple) be lowercased?

[17:29:16 CDT(-0500)] <tbain> that's fine, it's hardly any effort to change it

[17:29:53 CDT(-0500)] <dmccallum54> um

[17:30:17 CDT(-0500)] <tbain> Here is the spec I'm looking at - https://wiki.jasig.org/display/SSP/SSP+v2.0+MAP+Admin-+Color+Management

[17:30:41 CDT(-0500)] <dmccallum54> shooting from the hip a bit on this one… but i think i'd say that upper case sends a better message that these are quasi-constants… basically enum entries

[17:31:27 CDT(-0500)] <dmccallum54> and at least in the external data where we're doing something like this already, our examples instruct you to use upper case ('Y' and 'N')

[17:32:27 CDT(-0500)] <tbain> kk, so LT_BLUE, ORANGE, PURPLE, LT_GREEN, etc.?

[17:32:32 CDT(-0500)] <dmccallum54> awesome

[17:32:43 CDT(-0500)] <TonyUnicon> "awesome"

[17:32:49 CDT(-0500)] <dmccallum54> was waiting for that...

[17:32:58 CDT(-0500)] <TonyUnicon> was waiting for my time to shine

[17:33:15 CDT(-0500)] <tbain> There is a reference here I'm not getting.

[17:33:17 CDT(-0500)] <dmccallum54> <blink>Tony</blink>

[17:33:50 CDT(-0500)] <TonyUnicon> now that may pay is being deducted because of those scroll bars, humor is all I have

[17:33:59 CDT(-0500)] <TonyUnicon> my*

[17:35:01 CDT(-0500)] <dmccallum54> wait? we're paying you for this stuff?

[17:35:37 CDT(-0500)] <TonyUnicon> haha

[17:35:47 CDT(-0500)] <dmccallum54> "haha"

[17:35:51 CDT(-0500)] <dmccallum54> damn. never gets old.

[17:39:03 CDT(-0500)] <js70> tyler is this going to fit?

[17:39:04 CDT(-0500)] <js70> LUM_MVE_SLT_PNK_UT

[17:39:20 CDT(-0500)] <tbain> No, sorry

[17:39:52 CDT(-0500)] <js70> its redundant anyway

[17:40:15 CDT(-0500)] <js70> http://www.amazon.com/Mauve-Invented-Color-Changed-World/dp/0393323137

[17:40:42 CDT(-0500)] <dmccallum54> you've been waiting a long time to spring that one on us haven't you, js70

[17:40:52 CDT(-0500)] <js70> you betcha.

[17:41:22 CDT(-0500)] <js70> the power of google and some spare time.

[17:41:28 CDT(-0500)] <tbain> !

[17:41:32 CDT(-0500)] <tbain> you have spare time?

[17:41:48 CDT(-0500)] <tbain> Sounds like someone just volunteered for more?

[17:41:49 CDT(-0500)] <js70> not any more.

[17:42:03 CDT(-0500)] <js70> :^)

[17:42:12 CDT(-0500)] <TonyUnicon> lol

[17:42:18 CDT(-0500)] <tbain> I'm trying to figure out how to fit lumenisent mavue with a slight pinkish undertone into 10 characters, but it's not working out :/

[17:42:21 CDT(-0500)] <tbain> Sorry

[17:42:51 CDT(-0500)] <js70> there a bigger fish to fry. we'll leave it for 3.0

[17:43:14 CDT(-0500)] <tbain> kk

[17:57:25 CDT(-0500)] <JasonElwood> I think the only item left is SSP-1114

[18:35:16 CDT(-0500)] <JasonElwood> The only remaining open are my documentation tasks. I we are there. I may spend some more time after a break walking through the app again. Thanks for all the work.

[18:35:28 CDT(-0500)] <dmccallum54> sweet

[18:36:16 CDT(-0500)] <TonyUnicon> "sweet"

[18:36:26 CDT(-0500)] <dmccallum54> damn. just got old.

[18:36:30 CDT(-0500)] <TonyUnicon>

[18:36:33 CDT(-0500)] <dmccallum54> heh

[18:36:51 CDT(-0500)] <TonyUnicon> !sweet

[21:38:37 CDT(-0500)] <TonyUnicon> I keep checking back in expecting to see 5 jira tickets added to my queue

[21:38:55 CDT(-0500)] <dmccallum54> i added one to my own queue.

[21:39:00 CDT(-0500)] <dmccallum54> gotta maintain. gotta maintain

[21:39:30 CDT(-0500)] <TonyUnicon> if there something you need help with?

[21:39:34 CDT(-0500)] <TonyUnicon> is*

[21:39:42 CDT(-0500)] <dmccallum54> nope. thx tho.

[21:39:47 CDT(-0500)] <TonyUnicon> ok, I

[21:40:06 CDT(-0500)] <TonyUnicon> I'll be hovering around my laptop, if there is something I can do to make lives easier tonight I'll be available

[21:40:53 CDT(-0500)] <dmccallum54> thx

[21:41:00 CDT(-0500)] <dmccallum54> i'm planning to cut out in about an hour

[21:42:07 CDT(-0500)] <TonyUnicon> ok