From 18646dd5dc9368a8d2a99c2f68b2eccca63261ca Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Marek=20Noco=C5=84?= Date: Wed, 4 Dec 2024 09:20:31 +0100 Subject: [PATCH] Removed debug --- .github/workflows/preview_comment.yaml | 3 --- pim_feedbaxk | 34 ++++++++++++++++++++++++++ 2 files changed, 34 insertions(+), 3 deletions(-) create mode 100644 pim_feedbaxk diff --git a/.github/workflows/preview_comment.yaml b/.github/workflows/preview_comment.yaml index d91ebdaab6..2ef2072f35 100644 --- a/.github/workflows/preview_comment.yaml +++ b/.github/workflows/preview_comment.yaml @@ -27,7 +27,6 @@ jobs: - name: Create comment for changed files run: | - set -x #debug file_limit=100 build_url="https://ez-systems-developer-documentation--${{ github.event.pull_request.number }}.com.readthedocs.build/${{inputs.project}}en/${{ github.event.pull_request.number }}/" @@ -54,8 +53,6 @@ jobs: echo -e $comment > comment.md - cat comment.md #DEBUG - - name: Find comment id: find-comment uses: peter-evans/find-comment@v3 diff --git a/pim_feedbaxk b/pim_feedbaxk new file mode 100644 index 0000000000..3648e133a9 --- /dev/null +++ b/pim_feedbaxk @@ -0,0 +1,34 @@ +Attributes: +- Size ([XS, S, M, L, XL]) - a generic one +- Frame Shape/Features +- Material - a generic one +- Paint job/color - would probably require a couple of those (and a "Paints" attribute group) to take into account the various manufacturers and the parts +- + +Hi! + +I went through the PIM mnodeling exercise. + +I watch a couple of math youtube channels and they're sometimes doing what they call "building intuition" - going through a couple of handpicked examples so that the viewer understand what's going on under the hood and gains deeper knowledge of the subject. I think you're doing the same with the Product Types explanation - showing the implementation details so that the readers knows what's going on. Nicely done! + +About the modeling exercise: +- I've approached it in two phases: +1) Modeling attributes +2) Modeling Product Types + +About modelling attributes: +- It feels extremely tempting to model similar attributes as a single attribute. For example to have a single attribute for size: [XS, S, M, L, XL]. +But I think it's a trap - even though both the frame and the fork might be size XL, if at some point their actual dimensions might have to be displayed (like in tshirt, where you often get additional info when buying - chest diameter) then bundling these two would be a problem. + +Same for color - one manufacturer's red might be different from the red from another one. Bundling them together might be a problem (especially when one part comes in 5 colors and there's a million for the second one - it would be confusing for the PIM manager to choose the right one). + +About modelling product types: +- I didn't notice the series at first, so my initial idea was to model each model separately (Product Types: Fuji, Matterhorn, Annapurna, ...). I think it's mostly because the Series is not visible in the table - and started analyzing based on this :D +- Then I noticed the series: and I think the suggested solution makes sense, if the models in each series are similar to each other. + +I started to think how can the product type information be used. Let's assume: +- Series 4 might be targeted for families, and Series 5 might be targeted for semi-professionals + +My initial idea: +- so I'd use the product type to display the right templates etc. +