M-Labs
/
web2019
5
0
Fork 2
Code Issues 20 Pull Requests 3 Releases Wiki Activity

Add warnings if phaser has next card too long and increase it's virtual HP consumption #151

Open
esavkin wants to merge 2 commits from esavkin/web2019:148-phaser-fix into master
pull from: esavkin/web2019:148-phaser-fix
merge into: M-Labs:master
Conversation 0 Commits 2 Files Changed 4 +94 -35
4 changed files with 94 additions and 35 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
static/js/shop.bundle.js
View File

File diff suppressed because one or more lines are too long

2
static/js/shop/shop_store.js
View File

@ -480,7 +480,7 @@ const useCart = ((set, get) => ({
let itemsCopy = Array.from(crate.items);
const disabled = !!get().crateParams(crate.crate_mode).warnings_disabled;
itemsCopy = FillResources(itemsCopy, disabled);
itemsCopy = TriggerWarnings(itemsCopy, disabled);
itemsCopy = TriggerWarnings(itemsCopy, disabled, crate);
const [crate_warnings, occupied] = TriggerCrateWarnings(crate);
return {
...crate,

41
static/js/shop/warnings.js
View File

@ -30,14 +30,14 @@ const find_next_source_index = (data, index, source) => {
}
const not_enough_resource_trigger = (name) => {
return (_data, _index, counters) => {
return (_data, _index, counters, _crate) => {
const resource = find_in_counters(counters, name);
return resource.occupied > resource.max;
}
}
const no_source_trigger = (name) => {
return (data, index, _counters) => {
return (data, index, _counters, _crate) => {
const occupied = item_occupied_counters[name](data[index]);
if (occupied > 0)
return !find_previous_source(data, index, name);
@ -46,12 +46,20 @@ const no_source_trigger = (name) => {
}
const wiring_constraint = (name) => {
return (data, index, _counters) => {
return (data, index, _counters, _crate) => {
const next = find_next_source_index(data, index, name);
return next - index === 1;
}
}
const phaser_next_card_long = (data, index, counters, crate) => {
const nbrOccupied = resource_counters.hp(crate.items, -1);
const nbrHP = useShopStore.getState().crateParams(crate.crate_mode).hp;
return (index < data.length - 1) ? (
(data[index+1].consumes && data[index+1].consumes.depth && data[index+1].consumes.depth >= 8.5)
) : (nbrOccupied >= nbrHP && nbrHP > 0)
}
const Types = {
"eem_resource": {
level: "warning",
@ -88,22 +96,27 @@ const Types = {
trigger: wiring_constraint("eem"),
message: "Due to wiring constraints, the carrier can only connect to EEM cards immediately at its right, without crossing another carrier."
},
"phaser_next_card_long": {
level: "reminder",
trigger: phaser_next_card_long,
message: "Next card may interfere with this Phaser. Consider placing a short card after this Phaser."
},
"default": {
level: "warning",
trigger: (_a, _b, _c) => {
trigger: (_a, _b, _c, _d) => {
return true;
},
message: 'This item has unimplemented warning'
}
}
export function TriggerWarnings(data, disabled) {
export function TriggerWarnings(data, disabled, crate) {
return data.map((element, index) => {
if (!element.warnings) return element;
element.show_warnings = disabled ? [] :element.warnings
.map((warning, _) => {
if (!!Types[warning])
return Types[warning].trigger(data, index, element.counted_resources) ? {trigger: undefined, name: warning, ...Types[warning]} : null;
return Types[warning].trigger(data, index, element.counted_resources, crate) ? {trigger: undefined, name: warning, ...Types[warning]} : null;
else
return Types.default;
})
@ -143,6 +156,22 @@ const crate_warnings = {
const nbrHPDesktop = useShopStore.getState().crate_modes.desktop.hp;
return crate.crate_mode === useShopStore.getState().crate_modes.rack.id && occupied < nbrHPDesktop;
}
},
"phaser_not_fit": {
message: "Since one or more Phasers may interfere with their next cards, they may additional space, that wouldn't fit into the crate. Consider removing cards, placing short cards after Phaser(s), or choosing larger crate.",
sb10q commented 2024-10-16 11:22:42 +08:00
Review
Copy Link

"they may additional space"?

"they may additional space"?
sb10q commented 2024-10-16 11:23:14 +08:00
Review
Copy Link

And why "may interfere"? It's pretty clear whether they do or not.

And why "*may* interfere"? It's pretty clear whether they do or not.
level: "warning",
trigger: (crate, occupied) => {
const nbrHP = useShopStore.getState().crateParams(crate.crate_mode).hp;
const stacked_phasers = crate.items.filter((elem, index, data) => {
return (elem.name_codename === "Phaser") && ((index < data.length - 1) ? (
(data[index + 1].consumes && data[index + 1].consumes.depth && data[index + 1].consumes.depth >= 8.5)
) : (occupied >= nbrHP && nbrHP > 0))
}
).length;
// #!debug
console.log(stacked_phasers, occupied)
return stacked_phasers > 0 && (occupied + stacked_phasers*2 >= nbrHP) && nbrHP >0;
}
}
}

84
static/js/shop_data.js
View File

@ -221,7 +221,8 @@ const shop_data = {
"eem_wiring_constraint"
],
consumes: {
hp: 8
hp: 8,
depth: 20
}
},
'kaslisoc': {
@ -297,7 +298,8 @@ const shop_data = {
"eem_wiring_constraint"
],
consumes: {
hp: 8
hp: 8,
depth: 20
}
},
'vhdcicarrier': {
@ -322,7 +324,8 @@ const shop_data = {
"eem_wiring_constraint"
],
consumes: {
hp: 8
hp: 8,
depth: 16
}
},
'bnc-dio': {
@ -393,7 +396,8 @@ const shop_data = {
],
consumes: {
hp: 8,
eem: 1
eem: 1,
depth: 7
}
},
'sma-dio': {
@ -458,7 +462,8 @@ const shop_data = {
],
consumes: {
hp: 4,
eem: 1
eem: 1,
depth: 7
}
},
'mcx-dio': {
@ -549,7 +554,8 @@ const shop_data = {
],
consumes: {
hp: 4,
eem: 2
eem: 2,
depth: 23
}
},
'rj45-dio': {
@ -635,7 +641,8 @@ const shop_data = {
],
consumes: {
hp: 4,
eem: 2
eem: 2,
depth: 7
}
},
'urukul': {
@ -700,7 +707,8 @@ const shop_data = {
consumes: {
hp: 4,
eem: 2,
clk: 1
clk: 1,
depth: 16
}
},
'urukul_4412': {
@ -738,7 +746,8 @@ const shop_data = {
consumes: {
hp: 4,
eem: 2,
clk: 1
clk: 1,
depth: 16
}
},
'phaser': {
@ -767,12 +776,14 @@ const shop_data = {
size: 'small',
warnings: [
"no_eem_source",
"no_clk_source"
"no_clk_source",
"phaser_next_card_long"
],
consumes: {
hp: 4,
eem: 1,
clk: 1
clk: 1,
depth: 16 // heatsink 8.5
}
},
'zotino': {
@ -814,7 +825,8 @@ const shop_data = {
consumes: {
hp: 4,
eem: 1,
tec: 1
tec: 1,
depth: 16
},
resources: [
{name: "idc", max: 4}
@ -842,7 +854,8 @@ const shop_data = {
],
consumes: {
hp: 4,
eem: 1
eem: 1,
depth: 17
},
resources: [
{name: "idc", max: 4}
@ -866,7 +879,8 @@ const shop_data = {
],
consumes: {
hp: 8,
idc: 1
idc: 1,
depth: 6
}
},
'hvamp32': {
@ -903,7 +917,8 @@ const shop_data = {
],
consumes: {
hp: 4,
idc: 4
idc: 4,
depth: 16
}
},
'idc-sma-adapter': {
@ -924,7 +939,8 @@ const shop_data = {
],
consumes: {
hp: 4,
idc: 1
idc: 1,
depth: 6
}
},
'idc-mcx-adapter': {
@ -967,7 +983,8 @@ const shop_data = {
"idc_resource"
],
consumes: {
hp: 4
hp: 4,
depth: 5
},
resources: [
{name: "idc", max: 4}
@ -1020,7 +1037,8 @@ const shop_data = {
],
consumes: {
hp: 8,
eem: 2
eem: 2,
depth: 16
}
},
'koster': {
@ -1046,7 +1064,8 @@ const shop_data = {
],
consumes: {
hp: 4,
eem: 2
eem: 2,
depth: 7.5
}
},
'clocker': {
@ -1077,7 +1096,8 @@ const shop_data = {
consumes: {
hp: 4,
eem: 1,
clk: 1
clk: 1,
depth: 7
},
resources: [
{name: "clk", max: 6}
@ -1114,7 +1134,8 @@ const shop_data = {
],
consumes: {
hp: 4,
eem: 1
eem: 1,
depth: 16
},
},
'fast_servo': {
@ -1148,7 +1169,8 @@ const shop_data = {
],
consumes: {
hp: 4,
eem: 1
eem: 1,
depth: 16
},
},
'mirny': {
@ -1179,7 +1201,8 @@ const shop_data = {
consumes: {
hp: 4,
eem: 1,
clk: 1
clk: 1,
depth: 16
},
},
'almazny': {
@ -1207,7 +1230,8 @@ const shop_data = {
consumes: {
hp: 8,
eem: 1,
clk: 1
clk: 1,
depth: 16
},
},
'thermostat-eem': {
@ -1235,6 +1259,7 @@ const shop_data = {
consumes: {
hp: 4,
eem: 1,
depth: 16
}
},
'thermostat2ch': {
@ -1253,7 +1278,8 @@ const shop_data = {
],
size: 'small',
consumes: {
hp: 4
hp: 4,
depth: 12
},
resources: [
{name: "tec", max: 2}
@ -1282,7 +1308,8 @@ const shop_data = {
consumes: {
hp: 8,
eem: 1,
clk: 1
clk: 1,
depth: 8
},
},
'pounder': {
@ -1316,7 +1343,8 @@ const shop_data = {
consumes: {
hp: 8,
eem: 1,
clk: 1
clk: 1,
depth: 16
}
},
'eem_pwr_mod': {
@ -1341,6 +1369,7 @@ const shop_data = {
warnings: [],
consumes: {
hp: 8,
depth: 20
},
},
'kirdy': {
@ -1372,6 +1401,7 @@ const shop_data = {
warnings: [],
consumes: {
hp: 8,
depth: 16
},
},
},