2025-09-08_inventory_model_pr_review_sync

Meeting Title: Inventory Model PR Review Sync Date: 2025-09-08 Meeting participants: Uttam Kumaran, Emily Giant

WEBVTT

1 00:01:14.640 ⇒ 00:01:16.240 Emily Giant: Hey, how’s it going?

2 00:01:17.320 ⇒ 00:01:19.899 Uttam Kumaran: Hello. Going well.

3 00:01:20.980 ⇒ 00:01:24.590 Emily Giant: It just got cold in my house. It’s, like, fall.

4 00:01:25.220 ⇒ 00:01:33.609 Uttam Kumaran: It is still pretty hot here, although getting… I don’t know, today’s not so bad, it’s like 90.

5 00:01:35.650 ⇒ 00:01:43.009 Uttam Kumaran: This has been a pretty… this has been a pretty, like, not-so-hot summer, actually, out here. It’s been raining a lot.

6 00:01:44.040 ⇒ 00:01:48.869 Emily Giant: Yeah, that’s kind of nice, though, like, it forces you to, like, hunker down.

7 00:01:49.340 ⇒ 00:01:49.990 Uttam Kumaran: Yeah.

8 00:01:50.520 ⇒ 00:02:01.129 Emily Giant: when it’s nice, or even just not raining out, I’m like, I should be outside. Yeah, yeah, yeah. Alright, I can go camera off. I know it sounds like you’re in, like, a coffee shop.

9 00:02:01.180 ⇒ 00:02:06.700 Uttam Kumaran: Yeah, I’m just, like, I’m in a little… just in a little coffee shop.

10 00:02:06.700 ⇒ 00:02:12.570 Emily Giant: Oh, it’s cute! But I can see. But I know sometimes it, like, helps the internet to go faster.

11 00:02:12.570 ⇒ 00:02:17.040 Uttam Kumaran: Yeah, that’s usually my thing, and I’m like, I have, like, that 2020 MacBook Air.

12 00:02:17.390 ⇒ 00:02:22.610 Uttam Kumaran: That I… I have not upgraded, and so, yeah, it will,

13 00:02:22.850 ⇒ 00:02:25.500 Uttam Kumaran: Yeah. We’ll lose steam at some time, so, yeah.

14 00:02:25.500 ⇒ 00:02:35.109 Emily Giant: Okay, yeah, I’ll go video off, because I know even, like, having it on for you to have to process it is not always the best. Okay, where do you want to start?

15 00:02:35.960 ⇒ 00:02:44.860 Uttam Kumaran: I’m kind of game to start anywhere. I mean, I looked at your PR, I… maybe if you want to just…

16 00:02:45.210 ⇒ 00:02:47.489 Uttam Kumaran: Share your screen and wall.

17 00:02:47.490 ⇒ 00:02:47.920 Emily Giant: Why?

18 00:02:47.920 ⇒ 00:02:50.720 Uttam Kumaran: Through it, and we can just merge it and sort of see.

19 00:02:51.640 ⇒ 00:02:53.689 Emily Giant: That is… Because I know…

20 00:02:53.930 ⇒ 00:02:54.870 Uttam Kumaran: Yeah.

21 00:02:54.870 ⇒ 00:02:57.029 Emily Giant: Okay, let me share…

22 00:02:57.330 ⇒ 00:02:58.080 Uttam Kumaran: Bye.

23 00:02:58.610 ⇒ 00:03:05.089 Uttam Kumaran: I’m trying to figure out the best one to share. It was good. Okay.

24 00:03:05.090 ⇒ 00:03:09.990 Emily Giant: So… Let me go to GitHub…

25 00:03:10.950 ⇒ 00:03:17.150 Emily Giant: Alright, so I have 1, 2, and 3 should also be there. I don’t know where… It is.

26 00:03:17.280 ⇒ 00:03:21.650 Emily Giant: Maybe I need to, like, resubmit it?

27 00:03:21.930 ⇒ 00:03:22.979 Emily Giant: Hold on.

28 00:03:23.340 ⇒ 00:03:26.930 Uttam Kumaran: I’m gonna push mine…

29 00:03:27.110 ⇒ 00:03:33.939 Uttam Kumaran: As well, because I’m tired of, like, fixing these tests, it’s just that it’s so long.

30 00:03:39.330 ⇒ 00:03:56.670 Emily Giant: Okay, there’s number 3. Okay, so the first one, it doesn’t really touch anything, but it could definitely be optimized with, like, some incremental stuff, which I can add. I think once, like, we’ve gone through the PRs.

31 00:03:56.850 ⇒ 00:04:02.340 Emily Giant: I can use cursor to create some of those tests to make them run faster.

32 00:04:02.930 ⇒ 00:04:05.499 Emily Giant: But… okay, I’m gonna be able to commit.

33 00:04:06.260 ⇒ 00:04:08.140 Emily Giant: It looks like a lot of files changed.

34 00:04:08.640 ⇒ 00:04:18.189 Emily Giant: But it’s really mostly files, created, because there were no existing, staging models for the legacy stuff.

35 00:04:18.350 ⇒ 00:04:21.489 Uttam Kumaran: And, it just went right into, like.

36 00:04:21.490 ⇒ 00:04:33.740 Emily Giant: a union with the new stuff, and it was getting really buggy, so this is just, like, the pulled-out Postgres models, the raw on top, and then,

37 00:04:34.660 ⇒ 00:04:36.619 Emily Giant: These might just be the Rob.

38 00:04:37.150 ⇒ 00:04:41.630 Uttam Kumaran: I’m just seeing a different screen, sorry, I was just pulling up on my side.

39 00:04:44.320 ⇒ 00:04:46.079 Emily Giant: You can’t see…

40 00:04:46.080 ⇒ 00:04:49.170 Uttam Kumaran: I’m seeing, like, Fellow?

41 00:04:49.170 ⇒ 00:04:52.349 Emily Giant: Oh, God, okay, hold on. I’m a memento.

42 00:04:52.910 ⇒ 00:04:55.190 Emily Giant: Not… not what was meant.

43 00:04:56.660 ⇒ 00:04:57.320 Emily Giant: Okay.

44 00:05:04.130 ⇒ 00:05:04.980 Emily Giant: Okay.

45 00:05:06.940 ⇒ 00:05:08.440 Emily Giant: Alright, can you see it now?

46 00:05:08.720 ⇒ 00:05:09.690 Emily Giant: GitHub.

47 00:05:10.850 ⇒ 00:05:12.210 Uttam Kumaran: Yes.

48 00:05:12.210 ⇒ 00:05:14.880 Emily Giant: Okay, so all of these are just raw bottles.

49 00:05:15.410 ⇒ 00:05:20.769 Emily Giant: that needed to be created. I don’t think any of them touch anything in production.

50 00:05:20.960 ⇒ 00:05:23.200 Uttam Kumaran: Okay. Yeah.

51 00:05:23.530 ⇒ 00:05:26.069 Emily Giant: And these were just file renamed without changes.

52 00:05:26.070 ⇒ 00:05:39.109 Uttam Kumaran: And so, one thing I’m gonna just look at to see, like, why this staging PR, I guess, didn’t… didn’t… it failed, and maybe let me just see if we can, like, get it.

53 00:05:39.450 ⇒ 00:05:43.360 Uttam Kumaran: If it’s… if it err for some weird reason, then I just want to just double check.

54 00:05:43.360 ⇒ 00:05:45.059 Emily Giant: Okay, let’s check that out.

55 00:05:45.930 ⇒ 00:05:47.050 Emily Giant: Hmm…

56 00:05:50.070 ⇒ 00:05:52.859 Uttam Kumaran: Oh, it looks like it just timed out.

57 00:05:52.860 ⇒ 00:05:53.700 Emily Giant: Oh.

58 00:05:54.230 ⇒ 00:06:01.949 Uttam Kumaran: It looks like overall… There was just, like, yeah, this test errors, whatever.

59 00:06:03.200 ⇒ 00:06:09.740 Uttam Kumaran: Okay, yeah, I’m not too… suspect about any of this. Did you happen to run…

60 00:06:10.250 ⇒ 00:06:13.070 Uttam Kumaran: All of these, like, locally.

61 00:06:13.350 ⇒ 00:06:14.250 Emily Giant: Yes, they’re fine.

62 00:06:14.250 ⇒ 00:06:22.430 Uttam Kumaran: Okay, okay, alright. Then I’m… I’m good to go, I mean, unless you want to… Implement any,

63 00:06:22.660 ⇒ 00:06:26.689 Uttam Kumaran: incremental now, like, I would just ship it.

64 00:06:26.690 ⇒ 00:06:41.469 Emily Giant: I’ll ship this one. Okay. It’s not gonna make any difference. And one of the things that I did do here, that you might want to review, I just remembered that, like, actually I changed a YAML file, or the project.yaml.

65 00:06:41.470 ⇒ 00:06:42.190 Uttam Kumaran: S.

66 00:06:42.190 ⇒ 00:06:45.150 Emily Giant: Well, I thought that this could potentially help.

67 00:06:45.150 ⇒ 00:06:50.009 Uttam Kumaran: No, no, so this is gonna help. I… this is on my to-do list to do, like, after…

68 00:06:50.130 ⇒ 00:06:58.150 Uttam Kumaran: as part of, like, my revenue work, I was going to just fix tags, like, overall, but this is fine, yeah.

69 00:06:58.150 ⇒ 00:07:01.319 Emily Giant: Do that now, so that none of these legacy tables ever run.

70 00:07:02.240 ⇒ 00:07:02.960 Uttam Kumaran: Okay.

71 00:07:02.960 ⇒ 00:07:04.639 Emily Giant: So that’s all set up here.

72 00:07:04.850 ⇒ 00:07:12.160 Emily Giant: That’s the only, like, actual… thing that touches production, is I’m trying to pull anything that’s gonna make

73 00:07:12.360 ⇒ 00:07:19.330 Emily Giant: the refreshes take longer out. Like, there’s no reason they’d ever need to refresh, right? Like, they’re not gonna change.

74 00:07:20.330 ⇒ 00:07:22.220 Uttam Kumaran: Okay. Yeah, I agree.

75 00:07:22.640 ⇒ 00:07:26.720 Emily Giant: Okay, cool. So, that is the only thing that’s an actual change to…

76 00:07:27.080 ⇒ 00:07:33.239 Emily Giant: the production environment, but it runs fine in my local. So, this one is good.

77 00:07:36.460 ⇒ 00:07:43.460 Emily Giant: Then, after that, it starts… and that was, like, the bulk of the, like, new files, so…

78 00:07:43.870 ⇒ 00:07:45.950 Emily Giant: After this, it gets more confusing.

79 00:07:46.070 ⇒ 00:07:49.390 Emily Giant: But that’s the bulk of, like, the junk.

80 00:07:49.850 ⇒ 00:07:59.629 Emily Giant: So this one, number 2, so these are the intermediate legacy files. Let’s open up the…

81 00:08:01.210 ⇒ 00:08:09.060 Uttam Kumaran: Yeah, and one thing we can do, actually, is we can bring in all the new changes from Master into this, so if you open up GitHub Desktop.

82 00:08:09.420 ⇒ 00:08:13.190 Uttam Kumaran: And you just have intermediate legacy fixes.

83 00:08:14.230 ⇒ 00:08:14.849 Emily Giant: So this guy?

84 00:08:14.850 ⇒ 00:08:20.190 Uttam Kumaran: Yes, so if you go to the branch intermediate legacy fixes…

85 00:08:21.490 ⇒ 00:08:24.930 Uttam Kumaran: If you just click on the middle one, the middle dropdown.

86 00:08:25.290 ⇒ 00:08:26.529 Emily Giant: Middle drop down.

87 00:08:26.530 ⇒ 00:08:27.990 Uttam Kumaran: This one, yeah.

88 00:08:28.490 ⇒ 00:08:31.559 Emily Giant: Where’s your… that… the…

89 00:08:34.190 ⇒ 00:08:36.130 Uttam Kumaran: This guy, right here.

90 00:08:37.250 ⇒ 00:08:38.510 Emily Giant: Oh, here.

91 00:08:39.360 ⇒ 00:08:41.710 Uttam Kumaran: And then you just click on, yeah…

92 00:08:41.710 ⇒ 00:08:42.030 Emily Giant: Gotcha.

93 00:08:42.039 ⇒ 00:08:45.389 Uttam Kumaran: And then now, if you go to Branch, in the top right.

94 00:08:45.769 ⇒ 00:08:49.659 Uttam Kumaran: And then you can just click on Update for Master.

95 00:08:49.799 ⇒ 00:08:51.289 Uttam Kumaran: It’s gonna bring in…

96 00:08:51.789 ⇒ 00:08:56.759 Uttam Kumaran: Oh, okay, so this should be… oh, actually, can you hit this? Because we just merged everything in the last 10 minutes, actually.

97 00:08:57.229 ⇒ 00:08:58.749 Emily Giant: What was that? Did you click fetch?

98 00:08:58.859 ⇒ 00:09:02.199 Uttam Kumaran: If you just click Fetch Origin, right here, yeah.

99 00:09:04.109 ⇒ 00:09:07.469 Uttam Kumaran: And then now we do the update for master thing.

100 00:09:09.649 ⇒ 00:09:12.979 Uttam Kumaran: There it is, okay, and then you just hit push again?

101 00:09:12.980 ⇒ 00:09:13.580 Emily Giant: Yes.

102 00:09:13.870 ⇒ 00:09:18.109 Uttam Kumaran: Yep. So this will basically bring in all the changes we just merged.

103 00:09:18.820 ⇒ 00:09:26.149 Uttam Kumaran: SPR should… Now, It’ll just run again in staging, and we can maybe just…

104 00:09:26.840 ⇒ 00:09:30.500 Uttam Kumaran: watch that. Maybe if you can walk me through the changes while that goes.

105 00:09:30.790 ⇒ 00:09:39.230 Emily Giant: Yeah, okay, so running it and staging, so I can switch… so say you’re saying switch to that branch here?

106 00:09:39.920 ⇒ 00:09:45.330 Uttam Kumaran: Yeah, and if you just… maybe we can just walk through the PR, you can tell me what the, kind of, the core changes are.

107 00:09:45.330 ⇒ 00:09:45.930 Emily Giant: Okay.

108 00:09:45.930 ⇒ 00:09:50.679 Uttam Kumaran: You can… we can even do that in the… in the… in GitHub, like, I don’t… you don’t have to do that in dbt.

109 00:09:50.680 ⇒ 00:09:56.180 Emily Giant: As long as you are down to, like, walk me through it, I really, really appreciate learning this stuff from you, because…

110 00:09:56.180 ⇒ 00:09:56.930 Uttam Kumaran: Oh, yeah!

111 00:09:56.930 ⇒ 00:09:59.389 Emily Giant: It, like, like, warm…

112 00:09:59.390 ⇒ 00:10:00.300 Uttam Kumaran: No, no, no, I’m down.

113 00:10:00.300 ⇒ 00:10:00.899 Emily Giant: And I’ll show you.

114 00:10:00.900 ⇒ 00:10:05.900 Uttam Kumaran: Kind of like my… Yeah, I’ll show you my workflow, and yeah, that’s fine.

115 00:10:05.900 ⇒ 00:10:11.739 Emily Giant: Okay, I really appreciate it. Okay, so if I want to go to the current branch, do I click into this again?

116 00:10:13.240 ⇒ 00:10:13.730 Emily Giant: Right, sure.

117 00:10:13.730 ⇒ 00:10:25.850 Uttam Kumaran: you are in… you are in an intermediate legacy file, so we can either do two things. We can go and review it within github.com, or if you want to show me locally, we can do that too. Either… either way works.

118 00:10:26.450 ⇒ 00:10:29.340 Emily Giant: Okay, so locally, when you say that, do you mean, like, in DBT?

119 00:10:29.340 ⇒ 00:10:31.550 Uttam Kumaran: Locally, I mean, like, within cursor.

120 00:10:31.980 ⇒ 00:10:33.789 Emily Giant: Oh, okay, yeah, let’s try it.

121 00:10:33.790 ⇒ 00:10:35.969 Uttam Kumaran: Right now, if you… right now, if you open cursor.

122 00:10:36.970 ⇒ 00:10:37.630 Emily Giant: Okay.

123 00:10:38.140 ⇒ 00:10:39.399 Emily Giant: I’m in the other branch.

124 00:10:39.910 ⇒ 00:10:40.950 Emily Giant: It’s like.

125 00:10:40.950 ⇒ 00:10:45.809 Uttam Kumaran: So, so now you’re actually… you… you will be in this branch, so you can see here, yep.

126 00:10:46.070 ⇒ 00:10:51.820 Uttam Kumaran: Okay. So you’re actually in the right one, and if you click on this in the top, in this middle, like, little branch thing.

127 00:10:52.230 ⇒ 00:10:54.820 Uttam Kumaran: Actually, not…

128 00:10:56.370 ⇒ 00:11:02.490 Uttam Kumaran: Maybe not, though. Actually, is there another, like, thing here, this little drop-down? Is there, like, something about…

129 00:11:02.490 ⇒ 00:11:03.479 Emily Giant: Yeah, you can also.

130 00:11:03.480 ⇒ 00:11:10.350 Uttam Kumaran: Can you actually click… Yeah, I forgot exactly where you go to see the list of changes.

131 00:11:10.900 ⇒ 00:11:16.160 Uttam Kumaran: Okay, just click, click that, and is there anything, like, here on this dropdown?

132 00:11:16.600 ⇒ 00:11:19.980 Emily Giant: I feel like changes are sometimes right here.

133 00:11:19.980 ⇒ 00:11:21.320 Uttam Kumaran: Or timeline?

134 00:11:23.010 ⇒ 00:11:26.560 Uttam Kumaran: Is there… oh, click the… if you click here and click Source Control.

135 00:11:27.370 ⇒ 00:11:33.779 Uttam Kumaran: Yeah. Oh, I don’t know what this guy… Can you… is there, like, a way to, like, get rid of this?

136 00:11:35.640 ⇒ 00:11:37.299 Uttam Kumaran: Like, it’s a three dot.

137 00:11:39.210 ⇒ 00:11:41.360 Uttam Kumaran: Nevermind.

138 00:11:41.590 ⇒ 00:11:45.940 Uttam Kumaran: Okay.

139 00:11:46.400 ⇒ 00:11:50.469 Uttam Kumaran: Usually, there is a… I wonder if my cursor what I have enough.

140 00:11:54.070 ⇒ 00:12:03.040 Uttam Kumaran: I see it on mine. Changes, commit… Wallet.

141 00:12:06.280 ⇒ 00:12:14.499 Uttam Kumaran: What the heck? Can you close and open cursor? Yeah.

142 00:12:16.190 ⇒ 00:12:16.930 Emily Giant: Okay.

143 00:12:21.070 ⇒ 00:12:25.250 Emily Giant: I’m not gonna… Oh, shit.

144 00:12:36.290 ⇒ 00:12:41.310 Uttam Kumaran: Okay, this is all fine, and then see if you can click back in the source control thing.

145 00:12:42.780 ⇒ 00:12:44.510 Uttam Kumaran: Yeah.

146 00:12:45.370 ⇒ 00:12:51.099 Uttam Kumaran: Okay, either… whatever, I don’t know why this is not sure. Oh, click on Gra… oh, sorry, at the bottom, click on Graph here.

147 00:12:54.020 ⇒ 00:12:55.199 Emily Giant: Oh, nice, okay.

148 00:12:55.670 ⇒ 00:13:00.960 Uttam Kumaran: Alright, this should show some of the changes, but… Shop.

149 00:13:01.350 ⇒ 00:13:05.530 Uttam Kumaran: I guess this may be now a little bit confusing.

150 00:13:05.530 ⇒ 00:13:10.680 Emily Giant: Okay, maybe the… probably the easiest is just to go to github.com, and you can just walk me through.

151 00:13:10.990 ⇒ 00:13:15.289 Emily Giant: That’s true. I don’t know why this is… I don’t know why it’s…

152 00:13:15.290 ⇒ 00:13:16.159 Uttam Kumaran: See, it…

153 00:13:16.160 ⇒ 00:13:17.870 Emily Giant: Part of the club.

154 00:13:17.870 ⇒ 00:13:22.680 Uttam Kumaran: Yeah, so I see int Legacy Inventory Reconciliations. Hmm.

155 00:13:23.250 ⇒ 00:13:27.769 Emily Giant: Okay, so in this one, I think it’s just new.

156 00:13:28.480 ⇒ 00:13:33.219 Emily Giant: Again, this is just parsing it out from what used to be inventory adjustments.

157 00:13:33.660 ⇒ 00:13:34.720 Uttam Kumaran: But…

158 00:13:34.720 ⇒ 00:13:52.609 Emily Giant: the way it was set up prior to this was that inventory adjustments was in the old format that we got from Hivo, and was never properly aligned to, like, the… the options we now have with Polytomic. So it was kind of a reductive version, until…

159 00:13:53.200 ⇒ 00:13:55.759 Emily Giant: We were able to, like,

160 00:13:56.560 ⇒ 00:14:08.480 Emily Giant: flesh out these intermediate models. So, nothing all that spectacular is going on here in the intermediate models, but this is when, because of just aligning, like.

161 00:14:09.090 ⇒ 00:14:10.709 Emily Giant: character type.

162 00:14:10.870 ⇒ 00:14:15.869 Emily Giant: Things will start failing downstream, but…

163 00:14:15.990 ⇒ 00:14:29.620 Emily Giant: they are already failing, so it’s not going to actually be different. So this one in Legacy Reconciliations, all this is, is, like, a row-by-row,

164 00:14:29.930 ⇒ 00:14:44.149 Emily Giant: detail of if inventory was removed from the system for a non-sale purpose. So, it’s meant to return, one row per, like, inventory adjustment. So it’s.

165 00:14:44.150 ⇒ 00:14:45.389 Uttam Kumaran: For adjustment type, okay.

166 00:14:45.390 ⇒ 00:14:53.329 Emily Giant: Yeah. So it’s the same thing as, like, the suborders. Like, in these tables, like, early on in the DAG, like.

167 00:14:53.420 ⇒ 00:15:06.839 Emily Giant: the suborder table is one row per item per suborder, and this is one row per item per adjustment. And I wish… or reconciliation, I should say. So I think of things like this.

168 00:15:07.130 ⇒ 00:15:23.480 Emily Giant: If it’s a sale, or if it’s getting delivered, it’s a suborder adjustment. If it’s a reconciliation, it’s leaving for some other reason, like spoilage, system mismatch, any of these. But it is the same level of granularity, so it’s still, like, one row per

169 00:15:23.730 ⇒ 00:15:29.010 Emily Giant: Adjustment per item, which you could say for both suborder tables and reconciliation tables.

170 00:15:29.950 ⇒ 00:15:30.569 Emily Giant: And then.

171 00:15:31.260 ⇒ 00:15:42.399 Emily Giant: This time, it’s mapped back to Demolade’s seed file with the updated adjustment type, so that the legacy adjustments are aligned to the type

172 00:15:42.740 ⇒ 00:15:44.160 Emily Giant: in polyatomic.

173 00:15:44.270 ⇒ 00:15:47.399 Emily Giant: not what was pulling from Hivo previously.

174 00:15:48.570 ⇒ 00:15:49.160 Uttam Kumaran: Okay.

175 00:15:49.710 ⇒ 00:15:50.179 Emily Giant: He works.

176 00:15:50.180 ⇒ 00:15:53.950 Uttam Kumaran: So my one… my one question…

177 00:15:54.200 ⇒ 00:16:00.370 Uttam Kumaran: Would be, one, if we can put your table description at the top of this file.

178 00:16:00.370 ⇒ 00:16:00.960 Emily Giant: Yep.

179 00:16:00.960 ⇒ 00:16:11.390 Uttam Kumaran: Any chance? What you can actually… yeah, you can decide it directly here. If you just write exactly what you told me, just as a comment, I think that’s… that is probably…

180 00:16:12.000 ⇒ 00:16:13.220 Uttam Kumaran: Great.

181 00:17:03.880 ⇒ 00:17:04.500 Emily Giant: Okay.

182 00:17:05.490 ⇒ 00:17:09.549 Emily Giant: Nope, nope, nope, hold on. That was…

183 00:17:24.900 ⇒ 00:17:28.309 Emily Giant: So, this one, in Legacy Item Adjustment.

184 00:17:28.319 ⇒ 00:17:38.089 Uttam Kumaran: I guess one more question I had about that one. There’s a hard-coded date, like, line… line items created at less than 2024-12-08.

185 00:17:38.670 ⇒ 00:17:40.529 Uttam Kumaran: Well, is there a significance there?

186 00:17:40.530 ⇒ 00:17:47.770 Emily Giant: There is. Pulled from HEVO. These are not polyatomic tables. None of them.

187 00:17:47.770 ⇒ 00:17:48.230 Uttam Kumaran: Okay, okay.

188 00:17:48.230 ⇒ 00:17:52.650 Emily Giant: not exist in polyatomic, so I want the most…

189 00:17:53.170 ⇒ 00:18:02.539 Emily Giant: If I can pull it from Polytomic, I will. But this is the first date that Polytomic pulls for inventory adjustments.

190 00:18:05.390 ⇒ 00:18:12.379 Emily Giant: And this never has to update after that date. That’s when they start, like, crossing over one another.

191 00:18:12.650 ⇒ 00:18:15.450 Uttam Kumaran: Okay, okay, so from Allhevo prior to…

192 00:18:17.490 ⇒ 00:18:18.819 Emily Giant: I’ll add that to the comments.

193 00:18:19.340 ⇒ 00:18:27.969 Uttam Kumaran: I just put in a little suggested thing. If you just refresh this, you should see, like, a little comment from me.

194 00:18:30.410 ⇒ 00:18:30.850 Emily Giant: Yeah.

195 00:18:32.590 ⇒ 00:18:35.030 Uttam Kumaran: If you scroll down…

196 00:18:35.900 ⇒ 00:18:37.340 Emily Giant: Yeah, yeah, yeah.

197 00:18:44.360 ⇒ 00:18:45.830 Uttam Kumaran: Cool. Okay.

198 00:18:46.020 ⇒ 00:18:47.250 Emily Giant: Yes, okay.

199 00:18:48.130 ⇒ 00:18:58.420 Emily Giant: So that’s the first intermediate. The second is, I don’t like this title. I wish I could change it, but here we are.

200 00:18:58.540 ⇒ 00:19:01.040 Uttam Kumaran: So, in Legacy Item Adjustment.

201 00:19:01.250 ⇒ 00:19:12.219 Emily Giant: as opposed to in legacy inventory reconciliation, if I had my druthers, I’d go back and change it to suborder adjustments, because.

202 00:19:12.660 ⇒ 00:19:13.700 Uttam Kumaran: Alright.

203 00:19:13.700 ⇒ 00:19:14.560 Emily Giant: these aren’t.

204 00:19:14.710 ⇒ 00:19:17.219 Emily Giant: Adjustments for items that were delivered.

205 00:19:17.510 ⇒ 00:19:21.460 Emily Giant: Versus items that… were not.

206 00:19:21.460 ⇒ 00:19:23.310 Uttam Kumaran: I don’t.

207 00:19:23.310 ⇒ 00:19:24.120 Emily Giant: So it looks…

208 00:19:24.520 ⇒ 00:19:32.390 Emily Giant: nearly exactly like the other table. It should have the exact same, like, gate logic applied here, but it’s for, adjustments…

209 00:19:33.780 ⇒ 00:19:41.830 Emily Giant: Sales, as opposed to… The unsellable inventory. But they’re nearly identical.

210 00:19:41.840 ⇒ 00:19:48.119 Uttam Kumaran: Okay, so I see that there is a filter for adjustment type.

211 00:19:48.770 ⇒ 00:19:49.460 Emily Giant: Walking.

212 00:19:49.460 ⇒ 00:19:49.860 Uttam Kumaran: Okay.

213 00:19:49.860 ⇒ 00:20:00.080 Emily Giant: because the data’s very, like, unnormalized, denormalized in the Hivo tables, like, there’s a lot of… adjustments…

214 00:20:00.810 ⇒ 00:20:18.000 Emily Giant: that will have a suborder ID, for example, but it will be a reconciliation because of how Hivo… or because of how NetSuite used to work when we did cash sales and re-deliveries. So, this is accounting for, all of that, like, very hard…

215 00:20:18.150 ⇒ 00:20:25.180 Emily Giant: to, pass on to Demolade business logic garbage.

216 00:20:25.180 ⇒ 00:20:26.530 Uttam Kumaran: That would…

217 00:20:26.530 ⇒ 00:20:36.490 Emily Giant: throw off historicals, because we did switch systems and how we use systems so many times in the past. So…

218 00:20:36.800 ⇒ 00:20:38.810 Emily Giant: Where’s the line that you’re referring?

219 00:20:38.810 ⇒ 00:20:43.240 Uttam Kumaran: That way I can… Yeah, exactly.

220 00:20:43.590 ⇒ 00:20:45.430 Emily Giant: These are the only…

221 00:20:45.990 ⇒ 00:21:02.880 Emily Giant: So, before, I believe that the model called Salesforce was, like, where suborder ID is not null, but that is not actually applicable to our historical data, because, when the system would remove and reconcile orders that didn’t exist, it would have suborders.

222 00:21:03.340 ⇒ 00:21:08.980 Emily Giant: And so it got really, like, duplicative and messy, using suborder ID.

223 00:21:09.320 ⇒ 00:21:13.299 Emily Giant: Instead of the actual adjustment type that’s here.

224 00:21:14.430 ⇒ 00:21:21.490 Uttam Kumaran: So, if we were to leave two comments on this, I think, one, it would be great to leave a comment on, like, why we did line 17.

225 00:21:21.940 ⇒ 00:21:28.300 Uttam Kumaran: Which is, like, yeah, so exactly for that. What you can do, actually, here, you can hit plus.

226 00:21:28.590 ⇒ 00:21:29.830 Uttam Kumaran: Next,

227 00:21:29.940 ⇒ 00:21:35.339 Uttam Kumaran: Yeah, you can hit plus, and then you can actually just, like, do… hit, this code button right here.

228 00:21:36.980 ⇒ 00:21:37.760 Emily Giant: You’re…

229 00:21:37.760 ⇒ 00:21:51.629 Uttam Kumaran: Yeah, and then you can actually just do, like, you can just put an enter and do, like, dash dash. Sorry, actually, in the previous one, I put hashtag, it’s actually just dash dash, and you can leave a comment, basically, just saying, like, why… why we… why we have this logic.

230 00:21:51.960 ⇒ 00:21:59.469 Emily Giant: Okay, there are… There are multiple tables.

231 00:21:59.590 ⇒ 00:22:04.909 Emily Giant: Used to backfill Order ID.

232 00:22:06.260 ⇒ 00:22:07.430 Emily Giant: Info.

233 00:22:13.930 ⇒ 00:22:16.040 Emily Giant: Well, yeah.

234 00:22:18.940 ⇒ 00:22:28.819 Emily Giant: Sorry, I’m trying to think of the best way to say this. Order ID, so I already use them all. Okay, there are multiple tables used to backfill order ID info, or legacy…

235 00:22:29.180 ⇒ 00:22:30.330 Emily Giant: Sales?

236 00:22:36.650 ⇒ 00:22:39.870 Emily Giant: Okay. Suborders…

237 00:22:44.300 ⇒ 00:22:45.430 Emily Giant: Idah D.

238 00:22:47.790 ⇒ 00:22:48.970 Emily Giant: sword.

239 00:22:50.280 ⇒ 00:22:58.630 Emily Giant: I’m trying to… oh, I was gonna try and think of the year. Suborder ID… is not… always indicative

240 00:23:00.610 ⇒ 00:23:02.710 Emily Giant: of a sale.

241 00:23:03.390 ⇒ 00:23:15.229 Emily Giant: Or… Several, not all. Several, but not all of the legacy tables used to backfill.

242 00:23:18.720 ⇒ 00:23:19.910 Emily Giant: Quarters.

243 00:23:21.500 ⇒ 00:23:26.470 Emily Giant: that, that… did not have… Enough.

244 00:23:26.790 ⇒ 00:23:33.130 Emily Giant: Inventory to be fulfilled. This could definitely be chat GBT’d into, like, a sentence that makes more sense in a short.

245 00:23:33.130 ⇒ 00:23:34.619 Uttam Kumaran: No, this is… this is fine.

246 00:23:34.620 ⇒ 00:23:38.260 Emily Giant: For now. Okay, orders that do not to be fulfilled.

247 00:23:38.740 ⇒ 00:23:44.270 Emily Giant: We’ll have a suborder present on the reconciliation.

248 00:23:44.680 ⇒ 00:23:45.680 Emily Giant: mine.

249 00:23:50.770 ⇒ 00:23:52.930 Emily Giant: Which is not the case.

250 00:23:54.640 ⇒ 00:23:56.020 Emily Giant: for active.

251 00:24:03.630 ⇒ 00:24:04.580 Emily Giant: goodness.

252 00:24:04.720 ⇒ 00:24:10.769 Emily Giant: Is there a special star section I can hit to make this… make English sense?

253 00:24:10.770 ⇒ 00:24:16.889 Uttam Kumaran: Let’s hit… I think you can just do comment, and it’ll be okay.

254 00:24:17.080 ⇒ 00:24:17.920 Uttam Kumaran: Yeah.

255 00:24:18.540 ⇒ 00:24:19.510 Uttam Kumaran: Perfect.

256 00:24:20.020 ⇒ 00:24:20.540 Emily Giant: Okay.

257 00:24:20.540 ⇒ 00:24:26.000 Uttam Kumaran: And then, yeah, you just leave… you could just leave it as is for now. Okay, so we can keep going down.

258 00:24:26.960 ⇒ 00:24:32.000 Emily Giant: I can literally hear them, for that moment, screaming milk. Like, the sound of the steamer.

259 00:24:32.000 ⇒ 00:24:34.130 Uttam Kumaran: Yes, yes.

260 00:24:34.790 ⇒ 00:24:39.910 Emily Giant: Okay, in, like, the bot details. So, before we used,

261 00:24:40.090 ⇒ 00:24:51.349 Emily Giant: the Kivo table or the Pokemon table, literally both, to fill in the NetSuite lot ID. This is the model that kind of

262 00:24:52.300 ⇒ 00:24:55.770 Emily Giant: Merges those two tables, and…

263 00:24:55.890 ⇒ 00:25:03.099 Emily Giant: aligns them to the structure that we use with polyatomic data, because there was no, like, inventory number ID, per se.

264 00:25:03.210 ⇒ 00:25:08.300 Emily Giant: not even per se, but it just wasn’t. This is the, like, unique ID for most of the,

265 00:25:08.680 ⇒ 00:25:10.650 Emily Giant: Most of the polyatomic tables.

266 00:25:10.830 ⇒ 00:25:20.039 Emily Giant: at a lot level, and, before it was just, like, NetSuite lot ID. So what this does is it pulls, because eventually.

267 00:25:20.380 ⇒ 00:25:36.230 Emily Giant: where we’ll get to is that the logic for unlotted goods, like vases and non-expiring inventory needs to be treated very differently than any product that has a NetSuite lot ID currently, but that wasn’t the case in the past.

268 00:25:36.650 ⇒ 00:25:40.539 Emily Giant: So, it gets very convoluted, Tom. I’m so.

269 00:25:40.970 ⇒ 00:25:53.840 Emily Giant: But the reason that this was necessary to split out is that, without recognizing that historical hard goods used to have lots,

270 00:25:53.980 ⇒ 00:25:57.790 Emily Giant: Up until… 2024?

271 00:26:00.050 ⇒ 00:26:13.700 Emily Giant: the sales numbers in aggregate were, incorrect, as was the on-hand data, because even if it was a base, it would have a lot. And…

272 00:26:13.800 ⇒ 00:26:29.219 Emily Giant: now they don’t, but if I don’t have a manner of pulling out historical hard goods that have lots… Okay. …to the current inventory balance. So, this is…

273 00:26:29.600 ⇒ 00:26:33.300 Emily Giant: the… the… Regulating of that.

274 00:26:34.870 ⇒ 00:26:41.890 Uttam Kumaran: I’m just leaving that as a comment. So historical hard goods, you said lots until 2024. Ag sales costs are impacted because now our goods don’t have lots.

275 00:26:42.710 ⇒ 00:26:52.269 Uttam Kumaran: This is how we can… so… Concile, and make sure historical.

276 00:26:59.810 ⇒ 00:27:04.519 Emily Giant: And then we have… oh, why is this one 07? Emily Giant.

277 00:27:04.630 ⇒ 00:27:06.250 Emily Giant: That’s supposed to be 08.

278 00:27:06.940 ⇒ 00:27:10.969 Uttam Kumaran: You can… you can hit plus and actually just edit it inline the same way.

279 00:27:10.970 ⇒ 00:27:12.170 Emily Giant: Okay.

280 00:27:12.170 ⇒ 00:27:12.750 Uttam Kumaran: Yeah.

281 00:27:13.320 ⇒ 00:27:14.070 Emily Giant: And then hit the code.

282 00:27:14.390 ⇒ 00:27:15.300 Uttam Kumaran: Yes.

283 00:27:16.350 ⇒ 00:27:18.339 Uttam Kumaran: You could literally just change it right there.

284 00:27:18.900 ⇒ 00:27:23.870 Uttam Kumaran: And then you can just hit comment, yeah. And then at the end, we’ll go through and just approve the command.

285 00:27:23.870 ⇒ 00:27:25.130 Emily Giant: All of them? Okay.

286 00:27:25.880 ⇒ 00:27:34.560 Emily Giant: Okay. Planter product fix. This… okay, so there was a ticket that came in 50 different times about, like.

287 00:27:34.920 ⇒ 00:27:36.120 Emily Giant: Fucked up.

288 00:27:36.620 ⇒ 00:27:47.660 Emily Giant: non-floral inventory. So, I realized somewhere along the way that it is because planters, get consolidated starting in

289 00:27:47.910 ⇒ 00:27:54.020 Emily Giant: the same timeframe that, we stopped using lots for hard goods, so…

290 00:27:55.310 ⇒ 00:28:05.179 Emily Giant: people were opening tickets for inaccurate planter data, planter sales data, and then, I guess at some point.

291 00:28:05.180 ⇒ 00:28:21.940 Emily Giant: Unbeknownst to me, the product planning team decided to make a pot and a plant only one SKU, since they can’t be sent without one another. And, what was happening downstream is that the planter was getting categorized as a plant.

292 00:28:22.320 ⇒ 00:28:27.999 Emily Giant: And then it was showing… duplicate plant sales, because it was like, oh, you sold 2 of the

293 00:28:29.000 ⇒ 00:28:33.940 Emily Giant: And a puff that the planter went in. So, this is the, like…

294 00:28:35.060 ⇒ 00:28:44.690 Emily Giant: like, changing out in historical data, like, doing what I have to do to our current data, just to make…

295 00:28:45.430 ⇒ 00:28:48.430 Emily Giant: So, like, this is really the crux of…

296 00:28:48.990 ⇒ 00:29:03.040 Emily Giant: where the issue was happening, and getting miscategorized as a plant. So it’s just, like, really superficial logic placed on top of plant components, so that we’re not oversaving sales of plants.

297 00:29:04.690 ⇒ 00:29:08.690 Uttam Kumaran: So that’s one thing, I think we can just leave a comment right on top of…

298 00:29:10.560 ⇒ 00:29:17.730 Uttam Kumaran: On top of 77, basically. Just to say, like, the reason for this categorization.

299 00:29:27.250 ⇒ 00:29:28.000 Uttam Kumaran: microphone.

300 00:29:41.680 ⇒ 00:29:42.729 Emily Giant: Does that make sense to you?

301 00:29:45.090 ⇒ 00:29:47.080 Uttam Kumaran: Yes.

302 00:29:47.730 ⇒ 00:29:50.830 Emily Giant: I’m gonna say reason for the case statement.

303 00:29:51.240 ⇒ 00:29:53.340 Emily Giant: for this logic.

304 00:29:53.460 ⇒ 00:29:56.130 Emily Giant: Implementation. Okay.

305 00:30:01.690 ⇒ 00:30:03.650 Uttam Kumaran: Cool.

306 00:30:03.650 ⇒ 00:30:05.810 Emily Giant: That’s really what’s going on in that one.

307 00:30:06.790 ⇒ 00:30:15.289 Emily Giant: int legacy suborder adjustment type. Okay, so this is… Let me look.

308 00:30:15.290 ⇒ 00:30:17.179 Uttam Kumaran: No, it’s not what it’s not important.

309 00:30:19.170 ⇒ 00:30:30.880 Emily Giant: Okay, so this is, once again, just like a downstream, because the, initial suborder adjustments was getting quite long. This is…

310 00:30:32.020 ⇒ 00:30:32.720 Uttam Kumaran: critique.

311 00:30:32.720 ⇒ 00:30:37.999 Emily Giant: The suborder adjustments and saying whether it was a re-delivery, subscription, or sale.

312 00:30:38.700 ⇒ 00:30:42.050 Emily Giant: Because only, like, care and sale

313 00:30:42.410 ⇒ 00:30:45.770 Emily Giant: We’re parsed out in that previous model, and…

314 00:30:45.890 ⇒ 00:30:59.780 Emily Giant: similar to, like, the polyatomic logic that exists. This is just layering on, like, what kind of suborder it was. But I think it is identical to the active adjustments.

315 00:30:59.950 ⇒ 00:31:01.010 Emily Giant: model.

316 00:31:01.210 ⇒ 00:31:02.460 Uttam Kumaran: Okay.

317 00:31:04.200 ⇒ 00:31:08.450 Uttam Kumaran: Okay, yeah, it looks, like, basically pretty similar.

318 00:31:08.450 ⇒ 00:31:09.290 Emily Giant: Yeah.

319 00:31:11.550 ⇒ 00:31:20.869 Emily Giant: This is the YAML file, which… not even worth looking at, because it gets added to in the next one down.

320 00:31:21.300 ⇒ 00:31:26.699 Emily Giant: It’s just adding tests for all of these to make sure that, each row that’s returned is unique.

321 00:31:27.470 ⇒ 00:31:28.190 Uttam Kumaran: Okay.

322 00:31:29.610 ⇒ 00:31:33.090 Uttam Kumaran: And are all of these columns being used? Yeah.

323 00:31:33.090 ⇒ 00:31:34.110 Emily Giant: Yeah.

324 00:31:34.110 ⇒ 00:31:34.740 Uttam Kumaran: Okay.

325 00:31:35.030 ⇒ 00:31:37.990 Emily Giant: I know, it’s bad, but yeah.

326 00:31:38.270 ⇒ 00:31:41.029 Emily Giant: The inventory is by far the largest.

327 00:31:41.150 ⇒ 00:31:44.180 Emily Giant: And it’s because it’s used by, like, planning.

328 00:31:44.180 ⇒ 00:31:45.600 Uttam Kumaran: finance.

329 00:31:45.650 ⇒ 00:31:56.479 Emily Giant: and, like, procurement. So they use, like, procurement uses ones that, like, I didn’t even know existed. But okay, this one I just deleted because it is…

330 00:31:56.890 ⇒ 00:32:01.480 Emily Giant: Completely unnecessary and not connected to anything.

331 00:32:01.590 ⇒ 00:32:02.650 Emily Giant: And it was…

332 00:32:02.760 ⇒ 00:32:14.860 Emily Giant: getting refreshed, and it just… it was never used, never connected. It was used as, like, a temporary test for ProductsXF. I promise to you, it was garbage. It does not…

333 00:32:15.540 ⇒ 00:32:25.189 Emily Giant: staging inventory adjustments. This was… Oh, this was just writing in… the, adjustment ID.

334 00:32:25.220 ⇒ 00:32:26.450 Uttam Kumaran: Yes.

335 00:32:26.450 ⇒ 00:32:31.179 Emily Giant: down the line, I can use the number instead of writing out the name. It’s just a little more fail-safe.

336 00:32:32.300 ⇒ 00:32:37.440 Uttam Kumaran: Okay. Not huge changes there. Inventory item.

337 00:32:39.220 ⇒ 00:32:41.660 Emily Giant: This is just adding,

338 00:32:42.150 ⇒ 00:32:51.370 Emily Giant: Mostly adding some granularity so that the model we already reviewed about the planters and the plants getting duplicated as plant.

339 00:32:51.370 ⇒ 00:32:52.480 Uttam Kumaran: Yeah.

340 00:32:52.480 ⇒ 00:32:58.520 Emily Giant: layering in… adjustment ID for the component, so that I could pull that out later.

341 00:32:58.940 ⇒ 00:32:59.540 Uttam Kumaran: Okay.

342 00:32:59.900 ⇒ 00:33:03.850 Emily Giant: And then, this is, the non-merchandise item.

343 00:33:04.290 ⇒ 00:33:14.980 Emily Giant: essentially, because we have to look at laotted and non-lotted items separately, the logic is different. There was a bunch of, like, stickers, boxes, stuff like that, getting

344 00:33:14.980 ⇒ 00:33:24.390 Emily Giant: added to the table that truly nobody uses. So, I just wanted, like, an easier, cleaner way to not have to list out every, like, non…

345 00:33:25.320 ⇒ 00:33:26.850 Emily Giant: Sellable item.

346 00:33:27.270 ⇒ 00:33:27.970 Emily Giant: And…

347 00:33:27.970 ⇒ 00:33:32.530 Uttam Kumaran: Can we list that in the comment, too, here? Just, like, what type of things are excluded here?

348 00:33:32.870 ⇒ 00:33:34.569 Uttam Kumaran: That way it’s… yeah.

349 00:33:44.240 ⇒ 00:33:54.180 Emily Giant: What’s the word? This… Combines, aggregates, combines all non-sellable non-website.

350 00:33:55.800 ⇒ 00:34:00.389 Emily Giant: packaging… and marketing materials.

351 00:34:01.610 ⇒ 00:34:06.110 Emily Giant: That are not… measured.

352 00:34:06.240 ⇒ 00:34:07.120 Emily Giant: Bye.

353 00:34:08.060 ⇒ 00:34:11.359 Emily Giant: Anyone? By stakeholders.

354 00:34:22.050 ⇒ 00:34:23.439 Emily Giant: It is being prompted.

355 00:34:24.170 ⇒ 00:34:42.160 Emily Giant: So the reason I added this logic is to set up a… Forced upgrades are pretty murky, in both polyatomic and historical data. What happens is a line gets returned for both items, because

356 00:34:42.429 ⇒ 00:35:02.149 Emily Giant: we window function… we, partition by item ID, and it will be a different item ID when they change it, so it looks as though, like, two main products were sent, but you cannot have two main products in a suborder. This is, like, a truth. So I added this, logic so that, like, eventually.

357 00:35:02.390 ⇒ 00:35:09.969 Emily Giant: We’ll be able to partition by whether or not it’s a main product, and use, like, the most recent update.

358 00:35:10.310 ⇒ 00:35:13.930 Emily Giant: as, like, the true main product that was sent. So…

359 00:35:14.500 ⇒ 00:35:20.150 Emily Giant: hopefully we won’t have to use this because of Shopify data, but, like, in…

360 00:35:20.830 ⇒ 00:35:30.150 Emily Giant: The interim will still be able to classify forced upgrades using his main product, for any suborders that have two.

361 00:35:31.980 ⇒ 00:35:32.650 Uttam Kumaran: Okay.

362 00:35:34.740 ⇒ 00:35:36.920 Emily Giant: That’s… I think that’s that whole one.

363 00:35:37.550 ⇒ 00:35:41.520 Emily Giant: And then… adjustment mapping, all it is is adding the numbers at the end.

364 00:35:42.170 ⇒ 00:35:51.869 Uttam Kumaran: So, for this, I guess I was gonna ask also about seed files. I was talking to Alex, I was asking if maybe I can move these to S3,

365 00:35:52.050 ⇒ 00:35:56.840 Uttam Kumaran: But you tell me, like, are… do any of the seed files change, like, really often?

366 00:35:57.160 ⇒ 00:36:07.649 Emily Giant: No, I don’t like them. They’re awful to create and upkeep, so I’m totally fine with having them in an S3.

367 00:36:08.270 ⇒ 00:36:13.179 Uttam Kumaran: Yeah, mainly it was one, so I can… we can… we don’t need to run…

368 00:36:13.320 ⇒ 00:36:18.590 Uttam Kumaran: dbt seed, like, we can actually just, like, create these as tables.

369 00:36:18.590 ⇒ 00:36:20.829 Emily Giant: Directly from S3.

370 00:36:21.380 ⇒ 00:36:28.289 Uttam Kumaran: And then… but the big change is, like, if you want to make a change, you just have to go replace the CSV and S3, versus doing it here.

371 00:36:28.580 ⇒ 00:36:31.050 Emily Giant: Yeah, that’s better for everyone, I think.

372 00:36:31.950 ⇒ 00:36:34.830 Emily Giant: Because ultimately, I don’t… ultimately, like, we have…

373 00:36:34.830 ⇒ 00:36:36.729 Uttam Kumaran: Because we have, like, Google Sheets.

374 00:36:37.160 ⇒ 00:36:48.509 Uttam Kumaran: That’s, like, some stuff there coming in fixed. We also have stuff that’s in the warehouse that’s, like, cut off from the rest of the world, like, it’s just there. And we have this, so I’m just trying to, like, reduce…

375 00:36:48.820 ⇒ 00:36:50.689 Uttam Kumaran: As many sources of, like.

376 00:36:51.630 ⇒ 00:36:59.800 Uttam Kumaran: potentially stale data, right? And, like, also, we’re running dbtseed every time, and some of this is not changing, ever.

377 00:36:59.930 ⇒ 00:37:01.370 Emily Giant: Oh, yeah, yeah.

378 00:37:01.830 ⇒ 00:37:04.899 Emily Giant: Yeah. The fiscal calendar is not gonna change.

379 00:37:04.900 ⇒ 00:37:06.400 Uttam Kumaran: I just don’t need to learn.

380 00:37:06.450 ⇒ 00:37:07.380 Emily Giant: Okay.

381 00:37:07.900 ⇒ 00:37:09.940 Emily Giant: Yeah, I’m totally down.

382 00:37:13.720 ⇒ 00:37:16.280 Emily Giant: Okay. Okay, so that’s part two.

383 00:37:17.120 ⇒ 00:37:19.909 Emily Giant: This is a much smaller one.

384 00:37:21.220 ⇒ 00:37:27.190 Uttam Kumaran: So, I’m gonna go ahead and just… let me just commit the changes that we made.

385 00:37:27.190 ⇒ 00:37:27.940 Emily Giant: Okay.

386 00:37:45.620 ⇒ 00:37:46.300 Uttam Kumaran: God.

387 00:38:03.900 ⇒ 00:38:04.590 Emily Giant: Okay?

388 00:38:04.880 ⇒ 00:38:09.230 Uttam Kumaran: I see. So I think these should all be, like, yeah, I’m just committing all of them.

389 00:38:29.460 ⇒ 00:38:32.150 Uttam Kumaran: Okay, I’m just one minute.

390 00:39:03.190 ⇒ 00:39:04.160 Uttam Kumaran: Okay.

391 00:39:23.780 ⇒ 00:39:32.160 Uttam Kumaran: Okay, great. So, let me… Take a look at some…

392 00:39:44.960 ⇒ 00:39:48.460 Uttam Kumaran: Oh, Excited.

393 00:40:08.550 ⇒ 00:40:13.160 Uttam Kumaran: I’m just gonna look at the jobs related to this PR and see what they say.

394 00:40:51.690 ⇒ 00:40:59.280 Uttam Kumaran: Okay, so there’s just a couple of warnings.

395 00:40:59.880 ⇒ 00:41:00.640 Emily Giant: Okay.

396 00:41:02.470 ⇒ 00:41:10.520 Uttam Kumaran: If you open… if you go to dbt, and you just go to orchestration, and you go to PR, go to the pull request environment.

397 00:41:11.870 ⇒ 00:41:14.239 Emily Giant: Or, yeah, that’s fine.

398 00:41:14.380 ⇒ 00:41:23.910 Uttam Kumaran: Yeah, you click on staging, and then you click on that one, and then you just go to the one for part 2.

399 00:41:24.570 ⇒ 00:41:32.569 Uttam Kumaran: So the latest job, if you go to that, the one minute ago, and you go to Invoke.

400 00:41:33.100 ⇒ 00:41:36.059 Uttam Kumaran: So, there’s a couple of warnings at the top here.

401 00:41:44.200 ⇒ 00:41:47.800 Uttam Kumaran: Sorry, if you go to the third… yeah, exactly.

402 00:41:48.120 ⇒ 00:42:01.480 Uttam Kumaran: So usually what I do is I just cop-take this, and I copy-paste this into Cursor, basically, and then I’m like, can you debug? So, I don’t know what these,

403 00:42:04.400 ⇒ 00:42:06.660 Uttam Kumaran: Yeah, I don’t know what these are.

404 00:42:08.450 ⇒ 00:42:09.610 Emily Giant: Hmm…

405 00:42:12.630 ⇒ 00:42:16.999 Emily Giant: I’m trying to figure out… I think these are just tests that are based on…

406 00:42:17.610 ⇒ 00:42:26.530 Emily Giant: Differently named Legacy Static Suborder Adjustments. I feel like that exists, but… hold on, I’m gonna copy and paste it real quick.

407 00:42:26.530 ⇒ 00:42:30.210 Uttam Kumaran: Yeah, one thing you just do… I would just copy it, you could just copy that top part.

408 00:42:36.210 ⇒ 00:42:37.020 Emily Giant: Okay.

409 00:42:39.480 ⇒ 00:42:42.139 Uttam Kumaran: And I just usually put it right into cursory.

410 00:42:42.790 ⇒ 00:42:43.530 Uttam Kumaran: That’s…

411 00:42:43.910 ⇒ 00:42:48.099 Emily Giant: Did it do it? I just see adjustments, period.

412 00:42:48.650 ⇒ 00:42:49.500 Emily Giant: Okay.

413 00:43:11.760 ⇒ 00:43:14.920 Uttam Kumaran: Okay, this is the most recent one.

414 00:43:15.030 ⇒ 00:43:17.590 Emily Giant: Am I confused?

415 00:43:17.760 ⇒ 00:43:18.630 Emily Giant: Next.

416 00:43:19.360 ⇒ 00:43:22.420 Emily Giant: I think this is 3 minutes ago, was previously.

417 00:43:22.420 ⇒ 00:43:23.139 Uttam Kumaran: Yeah, yeah, yeah.

418 00:43:23.140 ⇒ 00:43:24.000 Emily Giant: seconds ago.

419 00:43:24.450 ⇒ 00:43:27.410 Uttam Kumaran: No, this is… so this is a step in the job.

420 00:43:27.410 ⇒ 00:43:29.010 Emily Giant: Oh, gotcha, gotcha, gotcha, okay.

421 00:43:30.250 ⇒ 00:43:41.139 Uttam Kumaran: So if you click on just… I just usually click right here. You just click on that, and then I just, yeah, I’m just copied and space.

422 00:43:41.140 ⇒ 00:43:43.130 Emily Giant: I think these were just renamed.

423 00:43:43.340 ⇒ 00:43:45.219 Emily Giant: And it doesn’t like that.

424 00:43:45.560 ⇒ 00:43:46.450 Uttam Kumaran: Yeah.

425 00:43:46.660 ⇒ 00:43:49.540 Emily Giant: But these should have been in the first PR.

426 00:43:50.220 ⇒ 00:43:51.859 Emily Giant: Oh, I don’t know what.

427 00:43:52.660 ⇒ 00:43:55.750 Emily Giant: I don’t know why it’s doing this. Okay.

428 00:44:01.010 ⇒ 00:44:05.990 Uttam Kumaran: Yeah, you can just hit enter. Oh, it shouldn’t figure it out.

429 00:44:07.650 ⇒ 00:44:11.049 Uttam Kumaran: Cool, let’s see what it says.

430 00:44:19.520 ⇒ 00:44:25.750 Uttam Kumaran: Okay, yeah. I mean, it… basically what it’s saying is that they’re… There is a…

431 00:44:28.460 ⇒ 00:44:38.709 Uttam Kumaran: like, was… I guess what it’s saying is that this file just didn’t exist, but there were, like… there’s a YAML reference, so what it’s gonna try to do is gonna create it, but…

432 00:44:39.070 ⇒ 00:44:41.550 Uttam Kumaran: I don’t know, like, do we need it at all?

433 00:44:41.550 ⇒ 00:44:45.690 Emily Giant: Well… Let me see… okay, so there’s actually one error, what is this?

434 00:44:47.710 ⇒ 00:44:49.929 Uttam Kumaran: Salesforce inventory.

435 00:44:50.310 ⇒ 00:44:53.100 Emily Giant: That doesn’t matter. That is a…

436 00:44:53.210 ⇒ 00:44:56.850 Emily Giant: done… done and done model. That’s what this is replacing.

437 00:44:57.070 ⇒ 00:45:01.160 Emily Giant: but, if I go to Studio…

438 00:45:01.590 ⇒ 00:45:02.750 Uttam Kumaran: Yeah, just…

439 00:45:02.750 ⇒ 00:45:11.019 Emily Giant: models in the legacy file that should exist. Because prior to this, they were just, like.

440 00:45:11.330 ⇒ 00:45:15.160 Emily Giant: table references from nowhere, and

441 00:45:15.800 ⇒ 00:45:22.000 Emily Giant: I mean, they should be in this model, too, because I created them from those previous… Hold on.

442 00:45:22.650 ⇒ 00:45:25.869 Emily Giant: New model structure…

443 00:45:28.720 ⇒ 00:45:30.160 Uttam Kumaran: Oh, look, it’s crying!

444 00:45:33.910 ⇒ 00:45:37.049 Emily Giant: No, they just have different names. It’s static, like.

445 00:45:37.050 ⇒ 00:45:38.809 Uttam Kumaran: Thank you for staging legacy.

446 00:45:38.810 ⇒ 00:45:41.309 Emily Giant: Aging Legacy is the next exploration.

447 00:45:41.310 ⇒ 00:45:42.310 Uttam Kumaran: amusements.

448 00:45:42.350 ⇒ 00:45:46.499 Emily Giant: I feel as though, in Part 3, these are no longer there.

449 00:45:46.750 ⇒ 00:45:48.300 Emily Giant: But, let me…

450 00:45:48.300 ⇒ 00:45:48.930 Uttam Kumaran: Okay.

451 00:45:50.110 ⇒ 00:45:52.680 Emily Giant: I’m in Part 3 right now, so I can just check.

452 00:45:53.690 ⇒ 00:45:57.869 Uttam Kumaran: Well, if you’re in Part 3, then, you know, they’re there, then they’re… they will be there.

453 00:45:57.870 ⇒ 00:45:58.210 Emily Giant: Okay.

454 00:45:59.710 ⇒ 00:46:14.259 Uttam Kumaran: So what one option is, like, we either… because right now, I think what Cursor did is it’s gonna, it’s gonna create that table, so what you can… basically, if you go back to Cursor, I think we should just probably just change…

455 00:46:14.810 ⇒ 00:46:25.489 Uttam Kumaran: I don’t know what it did. It said… okay, so that’s… that’s fixed. Remove the unused configuration path.

456 00:46:25.840 ⇒ 00:46:32.270 Uttam Kumaran: Fine. And then what… for the first fix, if you click on,

457 00:46:32.650 ⇒ 00:46:38.440 Uttam Kumaran: If you click on this, this little files thing, we can go see, like, what files have changed.

458 00:46:40.780 ⇒ 00:46:46.169 Uttam Kumaran: So if you just scroll to go see, like, what else is yellow, yeah, so it looks like it changed something in inventory.

459 00:46:47.740 ⇒ 00:46:53.399 Uttam Kumaran: And inventory, yeah, so just check out what it changed here in IntoInventory.yaml.

460 00:46:54.860 ⇒ 00:47:04.099 Uttam Kumaran: Oh, right here. Yeah. So you just scroll down, we can just see, like, what it did.

461 00:47:04.240 ⇒ 00:47:07.750 Uttam Kumaran: It should be highlighted in yellow. Okay, so…

462 00:47:08.060 ⇒ 00:47:11.470 Uttam Kumaran: Oh, okay, yeah, this is missing an argument, so these you can keep.

463 00:47:12.190 ⇒ 00:47:14.710 Emily Giant: Okay. I think these are, these are just, like…

464 00:47:15.030 ⇒ 00:47:27.030 Uttam Kumaran: Warnings, so these are all fine. And then, it looks like… can you scroll up here? I think it decided to create a new file or something? If you click… yeah, on the right side, so…

465 00:47:27.190 ⇒ 00:47:30.909 Uttam Kumaran: It looks like created a missing model here, so if you click on this.

466 00:47:33.260 ⇒ 00:47:34.840 Uttam Kumaran: I don’t know what this is.

467 00:47:36.910 ⇒ 00:47:46.840 Uttam Kumaran: if you don’t… if we don’t need this, it’s probably… I guess it must have… it must have just, like, been like, oh, you don’t have it, it’s all created, but… Could just get rid of it.

468 00:47:47.830 ⇒ 00:47:48.320 Emily Giant: I just do.

469 00:47:48.320 ⇒ 00:47:49.989 Uttam Kumaran: Yeah, hit undo.

470 00:47:50.460 ⇒ 00:47:58.720 Uttam Kumaran: And then, basically, what you mentioned is that the na- the reason for the first one is that

471 00:47:58.860 ⇒ 00:48:02.090 Uttam Kumaran: It’s in legacy static suborder adjustments.

472 00:48:02.330 ⇒ 00:48:05.259 Uttam Kumaran: Oh, so what happened is, like, maybe you made it in…

473 00:48:05.560 ⇒ 00:48:08.470 Uttam Kumaran: Part 3, you just didn’t bring those in.

474 00:48:08.760 ⇒ 00:48:11.519 Uttam Kumaran: Or, like, you did it in Part 1, you didn’t bring it in?

475 00:48:11.830 ⇒ 00:48:13.360 Emily Giant: Yeah, it’s gotta be in part one.

476 00:48:13.360 ⇒ 00:48:21.779 Uttam Kumaran: Okay, okay, okay, that’s fine. Okay, so that’s fine. And then can we just go ahead, if that Salesforce thing doesn’t need to be there, we can just remove that model.

477 00:48:22.450 ⇒ 00:48:27.479 Uttam Kumaran: from Cursor, like, from the… from the Part 2 PR.

478 00:48:27.630 ⇒ 00:48:28.420 Emily Giant: Okay.

479 00:48:28.740 ⇒ 00:48:30.699 Emily Giant: I’m afraid.

480 00:48:31.250 ⇒ 00:48:32.680 Uttam Kumaran: Okay, alright.

481 00:48:33.200 ⇒ 00:48:41.189 Emily Giant: I’m afraid of removing it until this entire PR is done, because it connects two things that will cause your tests to fail.

482 00:48:41.670 ⇒ 00:48:45.650 Uttam Kumaran: Okay, so then what was… should we… is it worth, like, resolving that error at all?

483 00:48:45.650 ⇒ 00:48:46.420 Emily Giant: No.

484 00:48:46.620 ⇒ 00:48:47.529 Uttam Kumaran: Okay, okay.

485 00:48:47.530 ⇒ 00:48:50.909 Emily Giant: just remove the test, is what I would recommend.

486 00:48:51.790 ⇒ 00:48:53.590 Uttam Kumaran: Okay.

487 00:48:55.130 ⇒ 00:49:00.670 Emily Giant: So, I can remove that test. What was the… blah, blah, blah…

488 00:49:22.210 ⇒ 00:49:23.500 Emily Giant: 19 errors.

489 00:49:23.730 ⇒ 00:49:24.490 Emily Giant: What?

490 00:49:29.160 ⇒ 00:49:32.689 Emily Giant: Okay, that one’s… mmm… not needed.

491 00:49:34.520 ⇒ 00:49:37.829 Emily Giant: In Legacy Inventory Reconciliations.

492 00:49:39.810 ⇒ 00:49:43.489 Uttam Kumaran: So if you scroll all the way to the bottom, it should tell you why.

493 00:49:44.810 ⇒ 00:49:48.579 Uttam Kumaran: Oh, so this is currently running. Okay. Alright, well, we can…

494 00:49:49.610 ⇒ 00:49:55.989 Uttam Kumaran: What you can do is, I mean, we can wait until this finishes, or you can nix it, and then it’ll tell you why it failed.

495 00:49:57.780 ⇒ 00:50:05.000 Uttam Kumaran: Click cancel… And then if you just refresh… this page.

496 00:50:05.290 ⇒ 00:50:11.550 Uttam Kumaran: And you open… you may have to just wait a sec for it to just sink in this whole thing.

497 00:50:12.070 ⇒ 00:50:17.460 Uttam Kumaran: Let me just probably refresh again at this point. It should have been.

498 00:50:18.500 ⇒ 00:50:21.720 Uttam Kumaran: Cool. If you scroll all the way to the bottom here…

499 00:50:22.100 ⇒ 00:50:37.690 Uttam Kumaran: it should tell you, if you go all the way to the top, right after you turn to cancel, it’ll give you all these issues. So… one thing that I’ve done at this point is… so one is, like, I would just confirm that we should…

500 00:50:38.180 ⇒ 00:50:43.230 Uttam Kumaran: test running these locally, because I don’t know what these issues are.

501 00:50:43.230 ⇒ 00:50:50.650 Emily Giant: Yeah, these are not any… issues that I get in that branch locally, which is odd.

502 00:50:52.050 ⇒ 00:50:53.705 Uttam Kumaran: So…

503 00:50:55.510 ⇒ 00:51:01.469 Uttam Kumaran: Okay, so let’s take… let’s take this, like, this one for… okay, so this is a Salesforce one, that’s fine.

504 00:51:01.470 ⇒ 00:51:01.990 Emily Giant: Okay.

505 00:51:02.040 ⇒ 00:51:10.790 Uttam Kumaran: But this one… so one thing you can do now is let’s just copy-paste these two issues.

506 00:51:11.420 ⇒ 00:51:14.840 Uttam Kumaran: It’s, like, inventory reconciliations, this one.

507 00:51:16.210 ⇒ 00:51:20.390 Emily Giant: It’s really weird, like, I don’t… I don’t…

508 00:51:20.820 ⇒ 00:51:26.660 Emily Giant: I feel like this is somehow not taken into consideration any of…

509 00:51:26.810 ⇒ 00:51:31.319 Emily Giant: the part one changes. Like, I’ve tested these a million times.

510 00:51:31.710 ⇒ 00:51:33.570 Emily Giant: There’s just no way.

511 00:51:34.640 ⇒ 00:51:48.420 Uttam Kumaran: Yeah, we’ll see. I think, look, if you look… I mean, if we read this error code right here, if you scroll down a little bit, it looks like there’s… well, just the one we were looking at, so if we just start, at the… at this one, it looks like there’s some type of, like.

512 00:51:48.900 ⇒ 00:51:56.270 Uttam Kumaran: Like, there’s some hashtag or something here, so, like, I mean, we should just… we’ll just go really quickly. We’ll just…

513 00:51:58.110 ⇒ 00:51:59.209 Emily Giant: Is that from the notes?

514 00:51:59.210 ⇒ 00:52:02.849 Uttam Kumaran: Maybe, yeah, so that’s why, like, let’s just…

515 00:52:02.980 ⇒ 00:52:05.099 Uttam Kumaran: We can even just open this, if you just…

516 00:52:05.320 ⇒ 00:52:08.289 Uttam Kumaran: If you copy-paste this in the cursor, it’ll go fake account for us.

517 00:52:08.290 ⇒ 00:52:09.619 Emily Giant: Okay, that’s cool.

518 00:52:10.270 ⇒ 00:52:15.899 Uttam Kumaran: I guess we don’t care about the Salesforce one, so let’s just start with… let’s just start with this one.

519 00:52:16.070 ⇒ 00:52:18.350 Uttam Kumaran: I don’t really care, but yeah.

520 00:52:23.590 ⇒ 00:52:25.090 Uttam Kumaran: Yep, terrific.

521 00:52:25.790 ⇒ 00:52:31.120 Uttam Kumaran: So dbt Cloud is a weird thing where it’s not gonna let… if you scroll up, it’s not gonna have selected

522 00:52:31.670 ⇒ 00:52:35.419 Uttam Kumaran: yeah, so I would just go one by one.

523 00:52:36.040 ⇒ 00:52:39.960 Emily Giant: Okay, gotcha. Oh no, where did I… yeah, okay.

524 00:52:41.130 ⇒ 00:52:42.339 Emily Giant: It was the from.

525 00:52:42.720 ⇒ 00:52:48.449 Uttam Kumaran: Yeah, so there’s probably something from our comments in the system, which is okay, should be a quick fix for it.

526 00:52:52.230 ⇒ 00:52:52.910 Uttam Kumaran: Beautiful.

527 00:52:54.430 ⇒ 00:52:59.279 Uttam Kumaran: And what you can do is actually just put it at the bottom, so you can scroll this all the way down.

528 00:52:59.750 ⇒ 00:53:01.230 Uttam Kumaran: Yeah.

529 00:53:01.610 ⇒ 00:53:04.649 Uttam Kumaran: Should give you, like, a…

530 00:53:05.960 ⇒ 00:53:12.139 Uttam Kumaran: Is there, like, a new thing down here somewhere? Oh, it’s, it’s behind your, yeah, okay, yep.

531 00:53:15.860 ⇒ 00:53:22.719 Uttam Kumaran: Yeah, so it’ll go look, and it should find it. Maybe while it’s running, we can copy the next one. Yeah, it’s gonna fix that.

532 00:53:23.390 ⇒ 00:53:24.390 Uttam Kumaran: Cool.

533 00:53:25.230 ⇒ 00:53:31.180 Uttam Kumaran: Next. This one looks like, yeah, it’ll…

534 00:53:31.180 ⇒ 00:53:39.629 Emily Giant: I… I’ve allotted adjustments. These are not… they don’t need to be fixed. They’re all fixed in the next one, because we don’t use those anymore.

535 00:53:39.960 ⇒ 00:53:49.340 Emily Giant: So… Any of these that are popping up with, like, in ag, not relevant.

536 00:53:50.290 ⇒ 00:53:57.760 Uttam Kumaran: So then, let’s just fix… Let’s just fix this one. The one that’s just popped up is from.

537 00:54:00.350 ⇒ 00:54:02.850 Uttam Kumaran: And then it’ll… it should check and run it.

538 00:54:05.750 ⇒ 00:54:07.899 Uttam Kumaran: Okay, cool.

539 00:54:08.500 ⇒ 00:54:15.530 Uttam Kumaran: And then if you go to GitHub now… Yeah, yes.

540 00:54:16.510 ⇒ 00:54:21.450 Uttam Kumaran: To see, like, what were the… what were some of the changes that it made.

541 00:54:25.090 ⇒ 00:54:31.689 Uttam Kumaran: You just hit close, I would… let’s just see, like, what the change was here.

542 00:54:32.320 ⇒ 00:54:43.280 Uttam Kumaran: So you can just click on this… the second file. So yeah, let’s just… let’s just commit the second one. This is one of the fix we did for the warning.

543 00:54:43.430 ⇒ 00:54:51.230 Uttam Kumaran: And then this one… Like, what is this error?

544 00:54:52.920 ⇒ 00:54:56.560 Uttam Kumaran: Can you go back to cursory? Like, what did it say this error was?

545 00:54:57.100 ⇒ 00:55:03.390 Uttam Kumaran: Remove the stray comment from at the end of the file.

546 00:55:03.500 ⇒ 00:55:08.119 Uttam Kumaran: The comment was appearing after the file from…

547 00:55:10.830 ⇒ 00:55:25.540 Uttam Kumaran: Can you… can you tell cursor, can you say, just… can you try running this? Because, right, because one thing it’s doing, and I updated the cursor rules, but sometimes it just compiles, and it’s like, oh, it’s good, but you have to run the file.

548 00:55:29.250 ⇒ 00:55:32.809 Uttam Kumaran: Oh, let’s just see, it should take a sec.

549 00:55:44.250 ⇒ 00:55:46.300 Uttam Kumaran: of people hospital.

550 00:56:16.650 ⇒ 00:56:17.300 Emily Giant: Heck?

551 00:56:18.010 ⇒ 00:56:20.950 Uttam Kumaran: Cool.

552 00:56:20.950 ⇒ 00:56:22.560 Emily Giant: Yeah, status.

553 00:56:23.530 ⇒ 00:56:24.750 Uttam Kumaran: Cool.

554 00:56:24.750 ⇒ 00:56:25.390 Emily Giant: Alright.

555 00:56:25.390 ⇒ 00:56:36.200 Uttam Kumaran: So… then what you can do is we can go back to GitHub Desktop, and we can just commit the bottom… just the bottom two changes. So we can deselect whatever this change is, yeah, I don’t know.

556 00:56:40.020 ⇒ 00:56:45.010 Uttam Kumaran: Oh, I know what it is. So, there is no… there is no staging inventory.

557 00:56:45.250 ⇒ 00:56:48.280 Uttam Kumaran: there’s just… staging.

558 00:56:48.860 ⇒ 00:56:56.249 Uttam Kumaran: So, I think you can commit this too, because it’s gonna give you… these 3 should clear all the 1s.

559 00:56:57.580 ⇒ 00:56:58.070 Emily Giant: Boy.

560 00:56:58.910 ⇒ 00:56:59.809 Emily Giant: What do you mean?

561 00:56:59.810 ⇒ 00:57:09.579 Uttam Kumaran: Well, it… I feel like it made this change because there is no staging… Then inventory, and then legacy.

562 00:57:10.210 ⇒ 00:57:11.350 Emily Giant: There is, though.

563 00:57:11.700 ⇒ 00:57:12.250 Emily Giant: Right?

564 00:57:12.250 ⇒ 00:57:18.250 Uttam Kumaran: If you go back to cursor, we can probably scroll up and see, like, why it made this change, too.

565 00:57:18.910 ⇒ 00:57:20.909 Uttam Kumaran: So, yeah, if you go all the way up…

566 00:57:21.190 ⇒ 00:57:26.520 Emily Giant: That was the… folder that all of those new commitments from Part 1 were in.

567 00:57:28.090 ⇒ 00:57:30.020 Uttam Kumaran: But it looks like it’s inventory…

568 00:57:30.160 ⇒ 00:57:38.410 Uttam Kumaran: legacy, right? Is there… is there a staging? There’s no… this is just new model structure, intermediate, than inventory.

569 00:57:38.410 ⇒ 00:57:46.329 Emily Giant: But in staging, there’s also legacy. So, it’s here, and then inventory… oh, it’s uppercase.

570 00:57:46.900 ⇒ 00:57:47.540 Emily Giant: Not lower.

571 00:57:47.540 ⇒ 00:57:52.549 Uttam Kumaran: Wait, go back to… if you go back to GitHub Desktop, what is the change?

572 00:57:52.930 ⇒ 00:58:01.809 Uttam Kumaran: I just feel like it’s flipped. Like… staging, inventory, oh, could be.

573 00:58:02.340 ⇒ 00:58:03.250 Uttam Kumaran: Yeah, it could be.

574 00:58:03.250 ⇒ 00:58:05.520 Emily Giant: And this is not uppercase.

575 00:58:06.310 ⇒ 00:58:07.369 Uttam Kumaran: Yeah, it could be.

576 00:58:10.600 ⇒ 00:58:12.020 Uttam Kumaran: He’s a really positive.

577 00:58:12.020 ⇒ 00:58:16.849 Emily Giant: Okay, yeah, I mean, I can’t imagine why else, like, if I go to any of my…

578 00:58:17.950 ⇒ 00:58:25.270 Emily Giant: recent commits, there will be legacy folder, it’s just uppercase L. So, should I just not commit that?

579 00:58:25.270 ⇒ 00:58:32.850 Uttam Kumaran: Yeah, you can either go change it just to the lower… just the uppercase L, or… yeah.

580 00:58:33.410 ⇒ 00:58:34.520 Emily Giant: Can I do that?

581 00:58:34.520 ⇒ 00:58:38.020 Uttam Kumaran: One thing you could do is you can right-click this, and you can just click,

582 00:58:38.860 ⇒ 00:58:50.060 Uttam Kumaran: discard changes, and then now if you go to your dbt… yeah, now if you go to dbt project and cursor, you can just go change that to uppercase.

583 00:58:51.300 ⇒ 00:58:51.950 Uttam Kumaran: Okay.

584 00:58:51.950 ⇒ 00:58:52.930 Emily Giant: YAML.

585 00:58:53.380 ⇒ 00:58:54.380 Uttam Kumaran: Yeah.

586 00:59:00.310 ⇒ 00:59:01.210 Emily Giant: Amazing.

587 00:59:05.740 ⇒ 00:59:09.479 Emily Giant: If you see the YAML file before I do, Let me know.

588 00:59:09.870 ⇒ 00:59:11.419 Uttam Kumaran: To be all the way at the bottom.

589 00:59:11.420 ⇒ 00:59:11.930 Emily Giant: Okay.

590 00:59:12.760 ⇒ 00:59:15.519 Emily Giant: I know, yeah, it should be, but I’m not… okay.

591 00:59:15.520 ⇒ 00:59:18.790 Uttam Kumaran: Like, like, because this is the dbt project file, actually.

592 00:59:19.480 ⇒ 00:59:20.039 Uttam Kumaran: So it should be.

593 00:59:20.040 ⇒ 00:59:25.570 Emily Giant: Oh, oh, oh, oh, duh, sorry. Okay, I was looking in the wrong…

594 00:59:26.520 ⇒ 00:59:33.700 Emily Giant: the wrong everything. Okay, so staging… Inventory. Legacy. Heap.

595 00:59:36.150 ⇒ 00:59:37.850 Emily Giant: Should I say rerun?

596 00:59:39.450 ⇒ 00:59:43.060 Uttam Kumaran: Right?

597 00:59:56.080 ⇒ 01:00:00.010 Uttam Kumaran: You can… you can try to get it to… try to have it rerun, should be fine.

598 01:00:33.740 ⇒ 01:00:40.629 Uttam Kumaran: And then we can probably just push this to PR while we’re waiting, like, these changes.

599 01:00:41.190 ⇒ 01:00:43.219 Uttam Kumaran: Yeah, okay, cool. Great.

600 01:00:46.690 ⇒ 01:00:51.379 Emily Giant: configurations. I don’t… is that true? Are there duplicate staging configurations?

601 01:00:54.250 ⇒ 01:01:03.210 Uttam Kumaran: Scroll down to 145… oh, 145, 142… oh yeah, see, there’s staging…

602 01:01:03.880 ⇒ 01:01:07.160 Uttam Kumaran: The carrot chomps. The only thing chops?

603 01:01:07.370 ⇒ 01:01:08.140 Uttam Kumaran: Proceed.

604 01:01:08.590 ⇒ 01:01:17.575 Uttam Kumaran: Inventory, legacy, raw, staging, inventory, legacy…

605 01:01:21.830 ⇒ 01:01:23.130 Uttam Kumaran: I guess.

606 01:01:26.700 ⇒ 01:01:32.750 Emily Giant: I think it’s confused because it fixed something, and then I changed it, and it… Okay, let me keep.

607 01:01:32.940 ⇒ 01:01:35.879 Uttam Kumaran: Let’s scroll down now, let’s see if it’s giving us the issue.

608 01:01:50.740 ⇒ 01:01:55.530 Uttam Kumaran: I don’t know. I guess we… I mean, I don’t… maybe this is just not…

609 01:01:55.660 ⇒ 01:02:00.100 Uttam Kumaran: like, working for some reason, I don’t know, I have to just keep…

610 01:02:02.140 ⇒ 01:02:09.659 Uttam Kumaran: The other thing you could do is you could tell it, like, well, I do have… like, can you go up to here and show that… we can go back to just look at this…

611 01:02:10.180 ⇒ 01:02:13.399 Uttam Kumaran: Staging Inventory Legacy.

612 01:02:17.570 ⇒ 01:02:24.219 Emily Giant: Yeah, see, it’s just, right now, it’s just intermediate, it’s just… It’s staging, inventory, and then legacy.

613 01:02:25.870 ⇒ 01:02:34.910 Uttam Kumaran: Well then, can you… can you just… maybe we can tell that to Chris, or you can just do, like, I need to account for at… and then you can do at inventory.

614 01:02:35.470 ⇒ 01:02:41.280 Uttam Kumaran: the folder… That’s under… or just… yeah, if you go… if you just hit backspace until…

615 01:02:41.410 ⇒ 01:02:43.479 Uttam Kumaran: It was… I think it just popped up.

616 01:02:45.330 ⇒ 01:02:48.550 Uttam Kumaran: If you do inventory slash…

617 01:02:48.550 ⇒ 01:02:50.989 Emily Giant: hygiene inventory? Or…

618 01:02:51.470 ⇒ 01:02:54.130 Uttam Kumaran: If you do add inventory slash…

619 01:02:55.970 ⇒ 01:02:57.939 Uttam Kumaran: And then just select the one that…

620 01:02:58.930 ⇒ 01:03:02.499 Uttam Kumaran: It’s at the top. It’s this one.

621 01:03:02.750 ⇒ 01:03:04.619 Emily Giant: Yeah, I got you.

622 01:03:05.720 ⇒ 01:03:10.830 Uttam Kumaran: And then you can just say, add inventory,

623 01:03:11.910 ⇒ 01:03:18.229 Uttam Kumaran: Yeah, you’re gonna say I need to account for inventory in the dbt project, which I thought we were doing before.

624 01:03:19.590 ⇒ 01:03:25.309 Uttam Kumaran: That way we can have it figure out why it’s not working.

625 01:03:53.190 ⇒ 01:03:55.680 Uttam Kumaran: Oh, okay, maybe it was the capital.

626 01:04:13.510 ⇒ 01:04:14.710 Uttam Kumaran: Nice.

627 01:04:14.830 ⇒ 01:04:16.940 Uttam Kumaran: Okay, that’s probably it. It’s just the capital.

628 01:04:17.650 ⇒ 01:04:18.230 Emily Giant: Okay.

629 01:04:24.100 ⇒ 01:04:28.240 Emily Giant: Okay, and then… To commit that.

630 01:04:30.740 ⇒ 01:04:32.890 Emily Giant: Where do I go? I forget.

631 01:04:56.700 ⇒ 01:04:57.330 Emily Giant: Hang.

632 01:05:05.740 ⇒ 01:05:11.870 Uttam Kumaran: Okay, cool, yeah, I think we can just probably commit… commit these changes, and then we can merge this.

633 01:05:11.870 ⇒ 01:05:15.530 Emily Giant: when I commit, do I… where do I go?

634 01:05:15.530 ⇒ 01:05:17.260 Uttam Kumaran: We do this also here.

635 01:05:17.370 ⇒ 01:05:20.220 Uttam Kumaran: If you just click on Source Control.

636 01:05:20.540 ⇒ 01:05:22.430 Emily Giant: And then you click on Changes.

637 01:05:23.090 ⇒ 01:05:24.410 Emily Giant: Got it. Okay.

638 01:05:26.050 ⇒ 01:05:29.450 Uttam Kumaran: I feel like you can just… does it let you type in anything here?

639 01:05:29.650 ⇒ 01:05:30.340 Emily Giant: Yes.

640 01:05:30.340 ⇒ 01:05:32.610 Uttam Kumaran: Oh.

641 01:05:32.720 ⇒ 01:05:35.320 Uttam Kumaran: I just don’t know what this…

642 01:05:37.810 ⇒ 01:05:45.949 Uttam Kumaran: Yeah, I don’t know why in your GitHub it’s, like, not giving you all these changes. Are they showing up here in GitHub Desktop?

643 01:05:48.430 ⇒ 01:05:55.899 Uttam Kumaran: Yeah, I don’t know. Maybe just easier to just do it here. Not sure why it’s not populating there.

644 01:05:56.110 ⇒ 01:05:56.470 Emily Giant: Let me…

645 01:05:56.470 ⇒ 01:05:58.949 Uttam Kumaran: So these are all fixed, this should fix all the warnings.

646 01:05:58.950 ⇒ 01:06:00.079 Emily Giant: This is not…

647 01:06:00.480 ⇒ 01:06:06.620 Emily Giant: quite… I need to refresh something. Pull from origin, or what do I do? Because this is still lowercase.

648 01:06:06.980 ⇒ 01:06:13.640 Emily Giant: No, no, no, it, it, it, this is the change. Oh, okay, oh, duh, sorry, duh, okay, we’re good.

649 01:06:15.230 ⇒ 01:06:18.530 Uttam Kumaran: Yeah, so, if you just type in,

650 01:06:18.720 ⇒ 01:06:19.220 Emily Giant: Thank you.

651 01:06:19.220 ⇒ 01:06:23.110 Uttam Kumaran: Whatever, you just say, fixing warnings.

652 01:06:27.140 ⇒ 01:06:32.700 Uttam Kumaran: Perfect. And then… yep, you can pull down from Master…

653 01:06:33.340 ⇒ 01:06:44.130 Uttam Kumaran: And then, let’s see, it may give you some changes with the comments, so… hit X, and then we can just go through what the changes are.

654 01:06:48.870 ⇒ 01:06:58.060 Uttam Kumaran: Okay, can you, deselect this row, 50?

655 01:07:00.730 ⇒ 01:07:05.519 Uttam Kumaran: Just click on, yeah, that’s checked, yep, and then…

656 01:07:06.190 ⇒ 01:07:11.179 Uttam Kumaran: If you can just click on everything 132 and below.

657 01:07:14.030 ⇒ 01:07:16.260 Uttam Kumaran: And then also click on 130.

658 01:07:17.980 ⇒ 01:07:24.030 Uttam Kumaran: And then you can just type in, fixed conflict.

659 01:07:25.760 ⇒ 01:07:28.849 Uttam Kumaran: Yeah, and then you just commit.

660 01:07:31.100 ⇒ 01:07:32.210 Uttam Kumaran: Yes.

661 01:07:33.320 ⇒ 01:07:38.679 Uttam Kumaran: And then you can just push… Push this back up.

662 01:07:39.660 ⇒ 01:07:45.670 Uttam Kumaran: Yeah. And then, you could just discard these changes, you can just right-click and hit discard.

663 01:07:46.100 ⇒ 01:07:46.820 Uttam Kumaran: Yep.

664 01:07:47.410 ⇒ 01:07:48.100 Uttam Kumaran: Perfect.

665 01:07:48.220 ⇒ 01:07:52.209 Uttam Kumaran: Okay, so we can probably merge this now, this should have everything.

666 01:07:55.310 ⇒ 01:07:58.349 Emily Giant: Oh, where did I go? Yeah.

667 01:07:59.480 ⇒ 01:08:03.279 Emily Giant: Merge in, like, the desktop or the browser?

668 01:08:04.000 ⇒ 01:08:05.729 Uttam Kumaran: Yes.

669 01:08:18.729 ⇒ 01:08:19.399 Emily Giant: Cool.

670 01:08:30.189 ⇒ 01:08:31.240 Emily Giant: Okay.

671 01:08:34.590 ⇒ 01:08:37.419 Emily Giant: Next is… next and last.

672 01:08:38.050 ⇒ 01:08:46.990 Emily Giant: For today, anyway. Number 3, what checks were not successful? Let me see… It’s gonna.

673 01:08:46.990 ⇒ 01:08:52.499 Uttam Kumaran: Yeah, so one thing we’re gonna wanna do is just bring in all the newest changes into the…

674 01:08:52.950 ⇒ 01:08:58.419 Uttam Kumaran: into the third PR, so I can probably do it on mine, we’re probably gonna get some conflicts.

675 01:08:58.420 ⇒ 01:08:59.040 Emily Giant: Okay.

676 01:08:59.229 ⇒ 01:09:02.029 Uttam Kumaran: Maybe I can just share, and I can kind of show you…

677 01:09:07.109 ⇒ 01:09:08.240 Uttam Kumaran: Okay…

678 01:09:13.710 ⇒ 01:09:14.649 Uttam Kumaran: Okay.

679 01:09:19.510 ⇒ 01:09:23.479 Uttam Kumaran: So this is… which one are we looking at?

680 01:09:25.550 ⇒ 01:09:31.170 Uttam Kumaran: Or 3… we are looking at park 3… Great.

681 01:09:37.069 ⇒ 01:09:39.409 Uttam Kumaran: We get some merge conflicts…

682 01:09:44.620 ⇒ 01:09:45.950 Uttam Kumaran: For sure.

683 01:10:03.100 ⇒ 01:10:06.650 Uttam Kumaran: Okay, I’m gonna accept both.

684 01:10:07.290 ⇒ 01:10:10.039 Uttam Kumaran: And then let’s go to this one…

685 01:10:14.270 ⇒ 01:10:15.820 Uttam Kumaran: And…

686 01:10:22.130 ⇒ 01:10:24.650 Uttam Kumaran: Oh, okay, this is from…

687 01:10:28.730 ⇒ 01:10:31.470 Uttam Kumaran: like, my change, I think, so…

688 01:10:31.470 ⇒ 01:10:32.060 Emily Giant: Right.

689 01:10:32.520 ⇒ 01:10:38.259 Uttam Kumaran: Yeah, okay, so I’m gonna double that…

690 01:10:42.170 ⇒ 01:10:47.740 Uttam Kumaran: Delete that… Oh, okay.

691 01:10:50.890 ⇒ 01:10:53.520 Uttam Kumaran: And then… okay.

692 01:11:03.500 ⇒ 01:11:07.120 Uttam Kumaran: Okay, cool. Continue merge… perfect.

693 01:11:07.450 ⇒ 01:11:14.020 Uttam Kumaran: So, for this one… Maybe we can just walk through…

694 01:11:14.920 ⇒ 01:11:20.810 Uttam Kumaran: changes really quick, so some of these I saw are just, like, Adding in columns.

695 01:11:21.860 ⇒ 01:11:30.650 Emily Giant: Yeah, most of the… oh, sorry, my job. I think the major change was, that the MART models, the,

696 01:11:31.210 ⇒ 01:11:49.379 Emily Giant: the aggregate versions have some updated logic to account for, uncommitted orders that, like, was just unknown before. There was a lot of testing we had to do to, like, narrow down what that means, and that’s gonna… I’m starting at the end instead. Let me start at the beginning.

697 01:11:49.380 ⇒ 01:11:59.329 Emily Giant: Essentially, the new… Legacy format is just brought into the MART models in this, and because it’s…

698 01:11:59.930 ⇒ 01:12:08.829 Emily Giant: quite a bit of, like… Yeah, the one you’re looking at now, anything that doesn’t have, like, legacy in it is just updated columns.

699 01:12:09.120 ⇒ 01:12:09.560 Uttam Kumaran: Okay.

700 01:12:09.560 ⇒ 01:12:20.090 Emily Giant: This one? Updated columns. Like the main product one. There is a little bit of refinement of filtering, so I would say in both the in…

701 01:12:20.550 ⇒ 01:12:22.869 Emily Giant: Not the aggregate one, but,

702 01:12:23.120 ⇒ 01:12:37.519 Emily Giant: further upstream, with, like, the sub-orders… yep, and suborders unlotted, and suborders with lots, it’s the filtering for, like, what accounts for allotted suborder, and that’s related to, what we were chatting about earlier with the, like.

703 01:12:38.120 ⇒ 01:12:41.449 Emily Giant: hard goods did have

704 01:12:41.670 ⇒ 01:12:59.939 Emily Giant: a NetSuite lot ID prior to, like, 2024, so that was implementing that logic into the, subordin reconciliation table so that it didn’t, like, downstream distort the current on-hand loted, or unlotted goods.

705 01:13:01.460 ⇒ 01:13:02.010 Uttam Kumaran: Okay.

706 01:13:04.010 ⇒ 01:13:05.760 Emily Giant: So that’s what’s going on there.

707 01:13:06.040 ⇒ 01:13:13.939 Emily Giant: And that kind of carries through to both of the MART models, the adjustments and the,

708 01:13:14.680 ⇒ 01:13:28.910 Emily Giant: the lot balance, and then also, we never had a lot balance before for the unlotted hard goods, and that’s what we get with the into ag unlotted adjustments, and then,

709 01:13:30.100 ⇒ 01:13:39.680 Emily Giant: Yeah, I just changed the name, because I think I had spelled unlotted every possible way. Yeah. It’s not a real word, so, like.

710 01:13:40.070 ⇒ 01:13:45.380 Emily Giant: it looks like there’s a new model, it’s not, it’s just that I took away the hyphen. Yeah.

711 01:13:45.740 ⇒ 01:13:47.559 Uttam Kumaran: But it’s…

712 01:13:47.560 ⇒ 01:13:50.220 Emily Giant: Not big changes in this, really.

713 01:13:50.220 ⇒ 01:13:51.060 Uttam Kumaran: Yes. Yeah.

714 01:13:51.270 ⇒ 01:13:53.629 Uttam Kumaran: One question I had is there’s this…

715 01:13:54.290 ⇒ 01:14:00.660 Uttam Kumaran: Sort of, let me find it… yeah, this… like, nest logic?

716 01:14:01.840 ⇒ 01:14:08.559 Uttam Kumaran: How do you feel about this, or, like, because I feel like there’s a lot of logic in here.

717 01:14:08.560 ⇒ 01:14:15.329 Emily Giant: So, that is what is actually the deal with, like, committed versus not committed.

718 01:14:15.720 ⇒ 01:14:20.159 Emily Giant: Yeah, it’s… it’s pretty dense.

719 01:14:25.840 ⇒ 01:14:31.290 Uttam Kumaran: Yeah, because it’s, like, nested case. I mean, one thing we could do is, like… I guess, like.

720 01:14:32.420 ⇒ 01:14:35.689 Uttam Kumaran: I mean, I guess there’d be two questions I have. One is, like.

721 01:14:35.980 ⇒ 01:14:40.269 Uttam Kumaran: If it has to make it in this PR, then we can merge, but I would probably…

722 01:14:40.650 ⇒ 01:14:44.369 Uttam Kumaran: I can probably take a stab at rewriting some of it.

723 01:14:44.520 ⇒ 01:14:45.860 Uttam Kumaran: Because the.

724 01:14:45.860 ⇒ 01:14:46.459 Emily Giant: has to date.

725 01:14:46.460 ⇒ 01:14:47.769 Uttam Kumaran: This is a little bit tough.

726 01:14:48.700 ⇒ 01:14:53.819 Emily Giant: I would commit it and then go back and rewrite it, only because, like, stuff is gonna fail without this one.

727 01:14:53.820 ⇒ 01:14:54.750 Uttam Kumaran: Okay, okay.

728 01:14:56.060 ⇒ 01:14:58.360 Emily Giant: But I agree that it’s a little dense.

729 01:15:02.780 ⇒ 01:15:05.370 Uttam Kumaran: Yeah, I can try to rewrite it with cursor a little bit.

730 01:15:05.370 ⇒ 01:15:10.220 Emily Giant: Yeah. Even formatting it in a way that’s, like, readable would probably help a lot.

731 01:15:10.770 ⇒ 01:15:11.400 Uttam Kumaran: Okay.

732 01:15:12.600 ⇒ 01:15:16.599 Emily Giant: I think I was getting desperate around that era of…

733 01:15:21.590 ⇒ 01:15:22.550 Uttam Kumaran: Okay.

734 01:15:25.730 ⇒ 01:15:27.759 Uttam Kumaran: Okay, I’m fine with all this.

735 01:15:30.760 ⇒ 01:15:37.949 Emily Giant: I do think there are tests for every single non-staging model that was added.

736 01:15:39.030 ⇒ 01:15:39.790 Uttam Kumaran: Okay.

737 01:15:44.510 ⇒ 01:15:45.250 Uttam Kumaran: Thank you.

738 01:15:48.590 ⇒ 01:15:52.360 Emily Giant: Yeah, all that’s just things that didn’t exist in Legacy, I believe.

739 01:15:52.360 ⇒ 01:15:53.920 Uttam Kumaran: Yeah, okay.

740 01:16:07.520 ⇒ 01:16:12.230 Uttam Kumaran: Okay, this is fine.

741 01:16:12.390 ⇒ 01:16:17.450 Emily Giant: Yeah, I think those are just, like, character casting. Like, casting… My child.

742 01:16:19.770 ⇒ 01:16:22.910 Uttam Kumaran: Okay, let’s see what the,

743 01:16:32.940 ⇒ 01:16:37.499 Uttam Kumaran: Okay, so this one I want to just check, because some of these issues will then make it in.

744 01:16:47.980 ⇒ 01:16:51.509 Emily Giant: Using argument, property, generic test deprecation?

745 01:16:51.880 ⇒ 01:16:52.630 Emily Giant: Okay.

746 01:17:06.700 ⇒ 01:17:08.590 Uttam Kumaran: Let me see…

747 01:17:15.970 ⇒ 01:17:19.490 Uttam Kumaran: Okay, let’s see what it says for this guy.

748 01:17:34.470 ⇒ 01:17:35.520 Uttam Kumaran: Oh…

749 01:17:35.520 ⇒ 01:17:44.830 Emily Giant: int ag lauded adjustment versus adjustments. Okay, that’s, like, an old version it’s looking at, I think, because I updated that in my most recent commit, I thought.

750 01:17:48.760 ⇒ 01:17:50.050 Uttam Kumaran: Yeah, okay.

751 01:17:53.100 ⇒ 01:17:54.930 Uttam Kumaran: That’s fine.

752 01:18:04.160 ⇒ 01:18:10.139 Uttam Kumaran: It’s going to try to optimize, and I don’t want it to do that.

753 01:18:10.140 ⇒ 01:18:11.610 Emily Giant: Yep.

754 01:18:33.250 ⇒ 01:18:38.310 Uttam Kumaran: So that was the first… This one…

755 01:18:38.310 ⇒ 01:18:38.940 Emily Giant: Hmm.

756 01:18:38.980 ⇒ 01:18:44.760 Uttam Kumaran: Found… okay, so what is this one?

757 01:18:46.490 ⇒ 01:18:51.140 Uttam Kumaran: Gonna be another…

758 01:18:57.820 ⇒ 01:19:00.760 Uttam Kumaran: You know, just… just to work on this one.

759 01:19:10.690 ⇒ 01:19:11.510 Uttam Kumaran: That’s a winner.

760 01:19:15.410 ⇒ 01:19:19.560 Emily Giant: In ag… or lots with balances is not a… Thing.

761 01:19:19.560 ⇒ 01:19:25.760 Uttam Kumaran: Yeah… Jeez, I need to relax.

762 01:19:38.110 ⇒ 01:19:42.210 Uttam Kumaran: This combination of columns is, like, constantly one of the issues, I feel like.

763 01:19:42.210 ⇒ 01:19:50.659 Emily Giant: Yeah, it… I’m wondering if it is, like, deprecated functionality or something that’s going away that… wait a minute…

764 01:19:51.630 ⇒ 01:19:54.569 Uttam Kumaran: Yeah, it is, I think it’s just the fact that it will work.

765 01:19:54.870 ⇒ 01:19:55.359 Emily Giant: Oh my god.

766 01:19:55.360 ⇒ 01:19:59.790 Uttam Kumaran: The new… the new logic has its arguments.

767 01:20:00.380 ⇒ 01:20:04.800 Emily Giant: Yeah, okay. I mean, that’s fine. If it wants to update it, that’s great.

768 01:20:06.250 ⇒ 01:20:07.799 Uttam Kumaran: Okay, great.

769 01:20:11.120 ⇒ 01:20:15.020 Uttam Kumaran: Alright, now let’s see what it says…

770 01:20:56.060 ⇒ 01:20:58.610 Uttam Kumaran: Let’s see it, some sign of life.

771 01:21:10.680 ⇒ 01:21:12.619 Uttam Kumaran: Really, anything.

772 01:21:14.950 ⇒ 01:21:21.369 Uttam Kumaran: What is this? Oh… Oh, and then my other question is, I think the Zapier…

773 01:21:21.660 ⇒ 01:21:23.020 Emily Giant: Oh, that’s right!

774 01:21:23.180 ⇒ 01:21:29.250 Emily Giant: Yeah, I turned it off because it was going… Berserk on the weekend.

775 01:21:29.250 ⇒ 01:21:30.450 Uttam Kumaran: Okay, okay.

776 01:21:30.450 ⇒ 01:21:33.239 Emily Giant: Let me turn it back on and put it in that other channel.

777 01:21:33.240 ⇒ 01:21:34.759 Uttam Kumaran: Put it in the other channel, yeah.

778 01:21:34.760 ⇒ 01:21:38.100 Emily Giant: Yeah, I’m so sorry, I… I looked at…

779 01:21:38.100 ⇒ 01:21:39.429 Uttam Kumaran: That’s fine, I just, I.

780 01:21:39.430 ⇒ 01:21:42.659 Emily Giant: Because it had gone 50 times last weekend.

781 01:21:42.660 ⇒ 01:21:50.860 Uttam Kumaran: No, luckily, it is failing on one model, but it’s, like, something unused. I was just like… one day I looked, I’m like, oh, it’s failing, why did I not…

782 01:21:51.110 ⇒ 01:21:53.970 Uttam Kumaran: I was, like, freaking out.

783 01:21:54.490 ⇒ 01:21:58.790 Emily Giant: like an on-my-phone, this is gonna make Menakshi lose it.

784 01:21:58.790 ⇒ 01:22:01.139 Uttam Kumaran: I know, I know, I know, I know, okay.

785 01:22:03.080 ⇒ 01:22:04.629 Emily Giant: Okay, hold on…

786 01:22:04.630 ⇒ 01:22:08.639 Uttam Kumaran: I mean, you know, I was thinking, I wanna get, like, a days without data problem.

787 01:22:08.640 ⇒ 01:22:09.550 Emily Giant: Yes.

788 01:22:10.300 ⇒ 01:22:17.899 Emily Giant: we’re gonna be, like, the show, from… I don’t know if you’ve ever watched it, it’s really good, but, like, Days Without Incident.

789 01:22:18.380 ⇒ 01:22:22.540 Uttam Kumaran: It’s the Riders of Lost, it’s super spooky. Oh, okay.

790 01:22:22.540 ⇒ 01:22:29.079 Emily Giant: But they realized that they really messed up with Lost, because they didn’t have an ending when they started, so they.

791 01:22:29.080 ⇒ 01:22:29.730 Uttam Kumaran: Eastern.

792 01:22:29.730 ⇒ 01:22:31.400 Emily Giant: It’s an ending on this one.

793 01:22:33.650 ⇒ 01:22:38.540 Emily Giant: Alright, editing zap, I just need to change the channel to BI Infrastructure.

794 01:22:46.490 ⇒ 01:22:47.340 Emily Giant: Right.

795 01:22:50.120 ⇒ 01:22:54.289 Uttam Kumaran: Okay, this… we’re still getting this in Salesforce inventory, by the way.

796 01:22:54.530 ⇒ 01:22:59.610 Emily Giant: Oh, God. Can we do… is it fam… like, it doesn’t… just… It doesn’t…

797 01:22:59.610 ⇒ 01:23:00.580 Uttam Kumaran: what it is.

798 01:23:02.000 ⇒ 01:23:03.110 Uttam Kumaran: Let’s find out.

799 01:23:07.600 ⇒ 01:23:11.249 Uttam Kumaran: I mean, we’ve come a long way with some of these, like, logs, like…

800 01:23:12.020 ⇒ 01:23:18.220 Uttam Kumaran: to actually, like, care about him, so I want to make sure that, like, if we can fix it, we can fix it. I mean, I’ll…

801 01:23:18.220 ⇒ 01:23:22.429 Emily Giant: It is… I… it is for sure a waste of time, because…

802 01:23:22.910 ⇒ 01:23:29.740 Emily Giant: we’re not gonna use it anymore. Ever. It’s in an archive folder, in the most recent Commit.

803 01:23:30.900 ⇒ 01:23:31.860 Emily Giant: But…

804 01:23:31.860 ⇒ 01:23:35.210 Uttam Kumaran: Comment, Postgres ag inventory adjustment does not…

805 01:23:37.790 ⇒ 01:23:41.070 Emily Giant: That’s from the seed file, right?

806 01:23:41.750 ⇒ 01:23:43.529 Uttam Kumaran: Yeah, I think so, so let’s go…

807 01:23:43.530 ⇒ 01:23:45.210 Emily Giant: not be happening.

808 01:23:55.790 ⇒ 01:23:59.930 Uttam Kumaran: Oh, yeah, it was literally just, like, because it was quoted.

809 01:23:59.930 ⇒ 01:24:00.810 Emily Giant: Mmm.

810 01:24:01.700 ⇒ 01:24:02.819 Uttam Kumaran: Alright, you know.

811 01:24:05.370 ⇒ 01:24:07.479 Uttam Kumaran: What’s the significance of this state?

812 01:24:09.730 ⇒ 01:24:10.970 Emily Giant: One moment.

813 01:24:11.280 ⇒ 01:24:13.039 Emily Giant: I gotta find you. Okay.

814 01:24:13.810 ⇒ 01:24:19.740 Emily Giant: 2024, 11… it should be 08.

815 01:24:20.190 ⇒ 01:24:20.820 Emily Giant: This is me.

816 01:24:20.820 ⇒ 01:24:22.200 Uttam Kumaran: Should it 12.08?

817 01:24:22.200 ⇒ 01:24:28.329 Emily Giant: Yeah, what is… well, what model is this pulling from? Because it might be completely based on…

818 01:24:28.330 ⇒ 01:24:31.910 Uttam Kumaran: Salesforce Adjustments Base, Postgres, yes, it should be…

819 01:24:31.910 ⇒ 01:24:34.019 Emily Giant: Oh, yeah, it should be coming.

820 01:24:34.130 ⇒ 01:24:37.919 Emily Giant: Even though we’re never gonna use this model, I guess it’s worth updating.

821 01:24:42.570 ⇒ 01:24:46.550 Emily Giant: So the significance of that date is that it’s the literal date that we migrated.

822 01:24:48.000 ⇒ 01:24:49.489 Uttam Kumaran: Oh, yeah, yeah, yeah.

823 01:24:49.490 ⇒ 01:24:54.530 Emily Giant: But the… The tables were still running until 12.08, when.

824 01:24:54.530 ⇒ 01:24:55.399 Uttam Kumaran: I’ll talk about that.

825 01:24:55.400 ⇒ 01:24:57.390 Emily Giant: And account for the data.

826 01:25:03.470 ⇒ 01:25:06.169 Uttam Kumaran: Alright, let’s see what it finds out.

827 01:25:11.260 ⇒ 01:25:24.250 Uttam Kumaran: I’ve given it some nice solutions on how to, like, actually query the database, too, so… Nice. Sometimes it was hallucinating on, like, cool, I’m gonna go ahead and, like, fix your source, and then it’s, like, guessing at, like, what the source table has. I was like, I…

828 01:25:24.250 ⇒ 01:25:24.910 Emily Giant: Oh my god.

829 01:25:24.910 ⇒ 01:25:33.840 Uttam Kumaran: I changed the cursor rules, and I was like, run a dbt query, select the columns, like, go through the whole process, do not cheat.

830 01:25:33.840 ⇒ 01:25:39.299 Emily Giant: Thank you so much for doing that. That was, like, I think it was Thursday, I was, like.

831 01:25:39.300 ⇒ 01:25:40.150 Uttam Kumaran: Yeah.

832 01:25:40.150 ⇒ 01:25:42.150 Emily Giant: It has told me columns don’t exist.

833 01:25:42.150 ⇒ 01:25:48.029 Uttam Kumaran: I know, I was saying the same thing, so what I told it to do is, like, if you,

834 01:25:49.240 ⇒ 01:25:52.570 Uttam Kumaran: Yeah, I forgot where I put it. I put it in, like…

835 01:25:53.200 ⇒ 01:26:04.979 Uttam Kumaran: Maybe it’s in dbt development, I don’t know. Somewhere I put, like, go look… use dbt query to go look for the column names in…

836 01:26:05.260 ⇒ 01:26:11.790 Uttam Kumaran: like, in… like, and then I also put, like, compilation’s not enough, you must run the models with real data.

837 01:26:11.790 ⇒ 01:26:13.090 Emily Giant: Yeah. Out of here, and it’s like.

838 01:26:13.090 ⇒ 01:26:21.640 Uttam Kumaran: Query source, like, go look at the column name from the columns in information schema before, like… don’t make stuff up, basically.

839 01:26:21.810 ⇒ 01:26:23.209 Emily Giant: Do that. Wow.

840 01:26:23.210 ⇒ 01:26:31.579 Uttam Kumaran: Okay, just… Oh… Alright, let’s see what it says, not activated.

841 01:26:37.430 ⇒ 01:26:38.850 Uttam Kumaran: Nice.

842 01:26:40.130 ⇒ 01:26:41.030 Uttam Kumaran: Say hi.

843 01:27:06.770 ⇒ 01:27:11.630 Uttam Kumaran: Alright, great. Cool. Alright, let’s give it… let’s see what it does now.

844 01:27:11.630 ⇒ 01:27:12.280 Emily Giant: Pain.

845 01:27:13.490 ⇒ 01:27:16.069 Emily Giant: I do have to hop in a little bit, but…

846 01:27:16.070 ⇒ 01:27:16.850 Uttam Kumaran: Okay.

847 01:27:16.850 ⇒ 01:27:20.049 Emily Giant: Sorry. I’m gonna… I’m gonna be back later to, like.

848 01:27:20.050 ⇒ 01:27:22.649 Uttam Kumaran: push the looker changes. Okay.

849 01:27:22.650 ⇒ 01:27:23.150 Emily Giant: Nope.

850 01:27:23.150 ⇒ 01:27:26.779 Uttam Kumaran: Maybe let’s give it 2 more minutes, worst case, I’ll fix whatever’s remaining.

851 01:27:27.140 ⇒ 01:27:27.890 Emily Giant: Oh, man.

852 01:27:27.890 ⇒ 01:27:33.919 Uttam Kumaran: This will be the last chink. In other better news, though, like, inventory model is now running way faster.

853 01:27:34.790 ⇒ 01:27:36.040 Emily Giant: That’s great.

854 01:27:36.040 ⇒ 01:27:40.979 Uttam Kumaran: Yeah, although it is failing for something.

855 01:27:40.980 ⇒ 01:27:44.820 Emily Giant: It probably needs a full refresh, that’s…

856 01:27:45.200 ⇒ 01:27:51.689 Uttam Kumaran: Yeah, but you can see, like, it’s now taking roughly… 10 to 20 minutes, basically.

857 01:27:51.830 ⇒ 01:27:54.060 Emily Giant: That’s what it should be, like, 10 minutes.

858 01:27:54.060 ⇒ 01:27:55.100 Uttam Kumaran: That’s… Yeah.

859 01:27:55.580 ⇒ 01:28:02.569 Emily Giant: I think removing those legacy files will make a difference, too, but, I’m pretty sure it’s just failing because of needing,

860 01:28:02.570 ⇒ 01:28:07.070 Uttam Kumaran: Yeah, no, there’s this, like… this lot hourly changes.

861 01:28:07.070 ⇒ 01:28:08.390 Emily Giant: Oh…

862 01:28:08.390 ⇒ 01:28:14.110 Uttam Kumaran: It’s kind of an insane table, by the way, I guess I was gonna talk to you guys about it. But, like…

863 01:28:15.340 ⇒ 01:28:20.130 Uttam Kumaran: Yeah, I just, like… Made some incremental changes.

864 01:28:20.130 ⇒ 01:28:28.459 Emily Giant: Yeah, it never needs to be part of the full refresh, since it’s only getting one new row per hour, and it shouldn’t really depend on any…

865 01:28:28.460 ⇒ 01:28:29.230 Uttam Kumaran: Okay.

866 01:28:29.230 ⇒ 01:28:29.780 Emily Giant: God.

867 01:28:29.780 ⇒ 01:28:38.940 Uttam Kumaran: Yeah, I’ll fix it. That’s the only thing that’s failing, and some of these are now… were taking a long time before, and they’re finishing pretty fast, so, like…

868 01:28:38.940 ⇒ 01:28:39.560 Emily Giant: That’s fine.

869 01:28:39.560 ⇒ 01:28:45.709 Uttam Kumaran: For example, this is 4 minutes, so this should have finished in, like, 15 minutes, basically.

870 01:28:45.710 ⇒ 01:28:49.490 Emily Giant: That’s great. Yeah, I noticed even in my staging that they’re running a lot faster.

871 01:28:49.490 ⇒ 01:28:52.620 Uttam Kumaran: Yeah, okay, so let’s just… let me go look at this.

872 01:29:02.210 ⇒ 01:29:04.269 Uttam Kumaran: Ugh, okay, whatever.

873 01:29:04.270 ⇒ 01:29:05.150 Emily Giant: Oh, my God.

874 01:29:05.150 ⇒ 01:29:08.589 Uttam Kumaran: Alright, I’ll, I’ll, I’ll watch it finish, and I’ll watch it.

875 01:29:08.970 ⇒ 01:29:12.360 Emily Giant: In the run, just, like, don’t run that thing.

876 01:29:12.630 ⇒ 01:29:19.420 Uttam Kumaran: Yeah, yeah, I’ll, I’ll just… I’ll make sure everything outside of this is fine, and then I’ll…

877 01:29:19.840 ⇒ 01:29:21.100 Uttam Kumaran: I’ll figure it out.

878 01:29:21.280 ⇒ 01:29:26.779 Emily Giant: Cool. Especially because it’s legacy, it’s like, no, we’re not even using…

879 01:29:26.780 ⇒ 01:29:34.230 Uttam Kumaran: No, I don’t even know, and I don’t know why this is not even, like… well, I don’t know what, like, did I not ship this change? Like, I don’t know what…

880 01:29:36.480 ⇒ 01:29:42.240 Emily Giant: Yeah, and I’ll be available again, not that I expect you to continue working at a coffee shop, you need to go home.

881 01:29:42.240 ⇒ 01:29:44.969 Uttam Kumaran: No, no, no, I’m gonna go home, but I’ll be on later.

882 01:29:44.970 ⇒ 01:29:53.700 Emily Giant: I’ll be around, so just, if anything, it’s wild. I’m gonna be gone for, like, an hour, and then, I’m gonna come back and push the Looker changes.

883 01:29:53.700 ⇒ 01:29:55.240 Uttam Kumaran: Okay, okay.

884 01:29:55.240 ⇒ 01:29:59.960 Emily Giant: They’re all staged. I’ve done a lot, I left some notes, but other than that, they should be good to go.

885 01:29:59.960 ⇒ 01:30:01.140 Uttam Kumaran: Okay, okay, perfect.

886 01:30:01.140 ⇒ 01:30:08.709 Emily Giant: Thank you so much for doing this. There’s definitely gonna be, like, probably 2 more chunks, but they shouldn’t be…

887 01:30:09.430 ⇒ 01:30:14.750 Emily Giant: They shouldn’t be causing any additional failures as to what’s already in production.

888 01:30:14.750 ⇒ 01:30:22.160 Uttam Kumaran: Yeah, so today I just wanna, before we, like, hit the head, I just wanna make sure that the inventory job and the full

889 01:30:22.620 ⇒ 01:30:26.960 Uttam Kumaran: Like, the core… the, peak jobs.

890 01:30:27.400 ⇒ 01:30:29.130 Uttam Kumaran: I’ll run through at least once.

891 01:30:29.530 ⇒ 01:30:33.839 Uttam Kumaran: That’s what I’ll be kind of just, like, looking to make sure before… yeah.

892 01:30:33.840 ⇒ 01:30:43.230 Emily Giant: Anything… That, erase those tests. I’m serious, there’s a model, like, I think it’s inventory deliveries.

893 01:30:43.400 ⇒ 01:30:45.089 Emily Giant: That has, like, 50 tests.

894 01:30:45.780 ⇒ 01:30:52.739 Emily Giant: they’re not necessary. We’re not gonna be anything by the end of the day tomorrow, so it’s fine for it to fail.

895 01:30:52.740 ⇒ 01:30:53.670 Uttam Kumaran: Okay, okay.

896 01:30:53.670 ⇒ 01:30:55.360 Emily Giant: I know, I know that’s not…

897 01:30:55.460 ⇒ 01:31:02.499 Emily Giant: the most reassuring feedback, but I know exactly what you’re talking about, because it’s part of the big one PR.

898 01:31:02.500 ⇒ 01:31:03.790 Uttam Kumaran: Yeah, yeah.

899 01:31:03.790 ⇒ 01:31:05.879 Emily Giant: Those have been failing for…

900 01:31:06.240 ⇒ 01:31:09.720 Uttam Kumaran: as long as I can remember. So, yeah.

901 01:31:09.720 ⇒ 01:31:16.209 Emily Giant: And it’s because of the deprecated HEVO model that we’re completely getting rid of with all these, like, chunks of changes.

902 01:31:16.210 ⇒ 01:31:19.220 Uttam Kumaran: Okay.

903 01:31:19.220 ⇒ 01:31:22.720 Emily Giant: Yeah. Okay, I’ll just keep communicating with you about, like.

904 01:31:23.360 ⇒ 01:31:30.119 Emily Giant: anything… yeah, just any questions you have about, like, tests that you want to exclude, etc, I’m here.

905 01:31:30.470 ⇒ 01:31:31.760 Uttam Kumaran: Okay, okay.

906 01:31:31.760 ⇒ 01:31:33.549 Emily Giant: Thanks again so much for doing this.

907 01:31:33.550 ⇒ 01:31:34.670 Uttam Kumaran: Thank you.

908 01:31:34.670 ⇒ 01:31:35.669 Emily Giant: Talk to you later. Bye.

Brainforge Knowledge

Explorer

2025-09-08_inventory_model_pr_review_sync_e9019974

Graph View