This repository was archived by the owner on Feb 18, 2025. It is now read-only.

TIP-45: IOTA 2.0 Transaction Payload - #147

Draft
PhilippGackstatter wants to merge 128 commits into
mainfrom
tip45
Draft

TIP-45: IOTA 2.0 Transaction Payload#147
PhilippGackstatter wants to merge 128 commits into
mainfrom
tip45

Conversation

@PhilippGackstatter

@PhilippGackstatterPhilippGackstatter commented Oct 23, 2023

Copy link
Copy Markdown

Rendered Version

Note that:

  • This TIP is in Draft mode and as such is still being updated.
  • Links to other new TIPs don't work and will only work once the TIP is merged.

daria305and others added 30 commits May 8, 2023 11:37
@daria305
Add TX TIP layout with updated motivation
4ae35fc
@daria305
Extend tx payload with allottedAccounts
3bec6cf
@PhilippGackstatter
Adapt TIP-47 from TIP-19
cfbe787
@muXxer
Merge pull request#12from iotaledger/tip47
4f67068
TIP-47: Storage Deposit
@PhilippGackstatter
Replace TIP-19, add TIP-47 in Readme
54d3faa
@daria305
Rename allotments
d07c4ff
@daria305
Further tip adjustments
fa28fcf
@PhilippGackstatter
Polish wording
e109330
@oliviasaa
Update tip-0045.md
06eb01d
Updated decays
@oliviasaa
Update tip-0045.md
a24a35f
Changed to a single lookup table
@daria305
Time and allotments description
9c9af7c
@oliviasaa
Add decay algorithms
e519051
@oliviasaa
minor change
7b15c0f
@oliviasaa
remove algorithms
0da6c05
@jkrvivian
Add context inputs
b69379c
@PhilippGackstatter
Add Context Input syntactical validation
d6ca1fb
@PhilippGackstatter
Rename Creation Time to Creation Slot
7dc17d1
@PhilippGackstatter
Format TIP-45
77acd99
@roman1e2f5p8s
Adapt changes from9c9af7c
89843cb
@roman1e2f5p8s
Adapt changes fromb69379c
e054452
@roman1e2f5p8s
Adapt changes fromd6ca1fb
468c6b3
@roman1e2f5p8s
Adapt changes from7dc17d1
81b204f
@roman1e2f5p8s
Adapt changes from77acd99
0c434d2
@roman1e2f5p8s
Fix conflicts with tip45withdecays
77764c6
@roman1e2f5p8s
Fix formatting, typos, Latex formulas
a2d70b4
@roman1e2f5p8s
Improve backquote formulas
e4a1dbc
@PhilippGackstatter
Add Table of Contents and format
fb1be1f
@PhilippGackstatter
Rename BIC Input
614f450
@PhilippGackstatter
Add reward input index <= max inputs count rule
68f81df
@PhilippGackstatter
Fix Public Key casing
254311d
@Alex6323Alex6323 mentioned this pull request Nov 24, 2023
Closed
6 tasks
thibault-martinez
thibault-martinez reviewed Dec 1, 2023
View reviewed changes
Comment threadtips/TIP-0045/tip-0045.md Outdated
thibault-martinez
thibault-martinez reviewed Jan 15, 2024
View reviewed changes
Comment threadtips/TIP-0045/tip-0045.md
A _Block Issuance Credit Input_ provides the BIC balance of a specific account as context to transaction execution of a
specific slot. It is required for any _Account_ transition and destruction if the account contains a _Block Issuer
Feature_, as any operation on the _Account_ can only be allowed if the BIC balance is non-negative at a specific point
in time. A _Block Issuance Credit Input_ always requires a _Commitment Input_. The input will be resolved to the BIC

@thibault-martinezthibault-martinezJan 15, 2024•
edited
Loading

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

A Block Issuance Credit Input always requires a Commitment Input.

This doesn't seem to be true, the node accepts BIC input without a commitment input.

@PhilippGackstatterPhilippGackstatterJan 15, 2024

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do you happen to have an example or a alphanet transaction that shows that?
I don't see how it could get past this check: https://github.com/iotaledger/iota-core/blob/de711606b447c36733ee60d3b72a0932166d07c1/pkg/protocol/engine/ledger/ledger/vm.go#L96.

thibault-martinez
thibault-martinez reviewed Jan 15, 2024
View reviewed changes
Comment threadtips/TIP-0045/tip-0045.md
thibault-martinez
thibault-martinez reviewed Feb 1, 2024
View reviewed changes
Comment threadtips/TIP-0045/tip-0045.md
Sign up for freeto subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

2 more reviewers
@thibault-martinezthibault-martinezthibault-martinez left review comments
@Alex6323Alex6323Alex6323 left review comments
Reviewers whose approvals may not affect merge requirements

At least 3 approving reviews are required to merge this pull request.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

8 participants

@PhilippGackstatter@thibault-martinez@Alex6323@daria305@muXxer@oliviasaa@jkrvivian@roman1e2f5p8s
, 'i'); if (__m === '*' || __re.test(location.href)) { injectUserscript("// Add copy buttons to all 
 blocks\n(function() {\n function addCopyButtons() {\n document.querySelectorAll('pre code').forEach(function(codeBlock) {\n if (codeBlock.parentElement.hasAttribute('data-copy-added')) return;\n codeBlock.parentElement.setAttribute('data-copy-added', 'true');\n \n var btn = document.createElement('button');\n btn.textContent = 'Copy';\n btn.style.cssText = 'position:absolute;top:4px;right:4px;padding:2px 8px;font-size:11px;background:#4ecdc4;border:none;border-radius:4px;color:#1a1a2e;cursor:pointer;opacity:0.7;transition:opacity 0.2s;';\n btn.onmouseover = function() { this.style.opacity = '1'; };\n btn.onmouseout = function() { this.style.opacity = '0.7'; };\n btn.onclick = function() {\n navigator.clipboard.writeText(codeBlock.textContent).then(function() {\n btn.textContent = 'Copied!';\n setTimeout(function() { btn.textContent = 'Copy'; }, 1500);\n });\n };\n codeBlock.parentElement.style.position = 'relative';\n codeBlock.parentElement.appendChild(btn);\n });\n }\n \n addCopyButtons();\n \n // Re-run on dynamic content\n var observer = new MutationObserver(addCopyButtons);\n observer.observe(document.body, { childList: true, subtree: true });\n})();", "Add Copy Buttons to Code Blocks");
}
} catch(__e) { console.warn('[Userscript:Add Copy Buttons to Code Blocks]', __e); }
})();
(function(){
try {
var __m = "github.com";
var __re = new RegExp('^' + "github\\.com" + '

Skip to content

This repository was archived by the owner on Feb 18, 2025. It is now read-only.

TIP-45: IOTA 2.0 Transaction Payload - #147
Draft
PhilippGackstatter wants to merge 128 commits into
mainfrom 
tip45
Draft
TIP-45: IOTA 2.0 Transaction Payload#147
PhilippGackstatter wants to merge 128 commits into
mainfrom 
tip45
Conversation
@PhilippGackstatter
@PhilippGackstatterPhilippGackstatter
commented
Oct 23, 2023

Copy link


Copy Markdown

Rendered Version
Note that:
  • This TIP is in Draft mode and as such is still being updated.
  • Links to other new TIPs don't work and will only work once the TIP is merged.
daria305and others
added 30 commits
May 8, 2023 11:37
@daria305
Add TX TIP layout with updated motivation
4ae35fc
@daria305
Extend tx payload with allottedAccounts
3bec6cf
@PhilippGackstatter
Adapt TIP-47 from TIP-19
cfbe787
@muXxer
Merge pull request#12from iotaledger/tip47
4f67068
TIP-47: Storage Deposit
@PhilippGackstatter
Replace TIP-19, add TIP-47 in Readme
54d3faa
@daria305
Rename allotments
d07c4ff
@daria305
Further tip adjustments
fa28fcf
@PhilippGackstatter
Polish wording
e109330
@oliviasaa
Update tip-0045.md
06eb01d
Updated decays
@oliviasaa
Update tip-0045.md
a24a35f
Changed to a single lookup table
@daria305
Time and allotments description
9c9af7c
@oliviasaa
Add decay algorithms
e519051
@oliviasaa
minor change
7b15c0f
@oliviasaa
remove algorithms
0da6c05
@jkrvivian
Add context inputs
b69379c
@PhilippGackstatter
Add Context Input syntactical validation
d6ca1fb
@PhilippGackstatter
Rename Creation Time to Creation Slot
7dc17d1
@PhilippGackstatter
Format TIP-45
77acd99
@roman1e2f5p8s
Adapt changes from9c9af7c
89843cb
@roman1e2f5p8s
Adapt changes fromb69379c
e054452
@roman1e2f5p8s
Adapt changes fromd6ca1fb
468c6b3
@roman1e2f5p8s
Adapt changes from7dc17d1
81b204f
@roman1e2f5p8s
Adapt changes from77acd99
0c434d2
@roman1e2f5p8s
Fix conflicts with tip45withdecays
77764c6
@roman1e2f5p8s
Fix formatting, typos, Latex formulas
a2d70b4
@roman1e2f5p8s
Improve backquote formulas
e4a1dbc
@PhilippGackstatter
Add Table of Contents and format
fb1be1f
@PhilippGackstatter
Rename BIC Input
614f450
@PhilippGackstatter
Add reward input index <= max inputs count rule
68f81df
@PhilippGackstatter
Fix Public Key casing
254311d
@Alex6323Alex6323
mentioned this pull request
Nov 24, 2023

Closed

6 tasks
thibault-martinez
thibault-martinez
reviewed
Dec 1, 2023
View reviewed changes
Comment threadtips/TIP-0045/tip-0045.md
Outdated

thibault-martinez
thibault-martinez
reviewed
Jan 15, 2024
View reviewed changes
Comment threadtips/TIP-0045/tip-0045.md
A _Block Issuance Credit Input_ provides the BIC balance of a specific account as context to transaction execution of a
specific slot. It is required for any _Account_ transition and destruction if the account contains a _Block Issuer
Feature_, as any operation on the _Account_ can only be allowed if the BIC balance is non-negative at a specific point
in time. A _Block Issuance Credit Input_ always requires a _Commitment Input_. The input will be resolved to the BIC
@thibault-martinezthibault-martinezJan 15, 2024•

edited

Loading

Copy link


Copy Markdown

Member
Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

A Block Issuance Credit Input always requires a Commitment Input.
This doesn't seem to be true, the node accepts BIC input without a commitment input.
@PhilippGackstatterPhilippGackstatterJan 15, 2024

Copy link


Copy Markdown

Author
Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do you happen to have an example or a alphanet transaction that shows that?

I don't see how it could get past this check: https://github.com/iotaledger/iota-core/blob/de711606b447c36733ee60d3b72a0932166d07c1/pkg/protocol/engine/ledger/ledger/vm.go#L96.
thibault-martinez
thibault-martinez
reviewed
Jan 15, 2024
View reviewed changes
Comment threadtips/TIP-0045/tip-0045.md
thibault-martinez
thibault-martinez
reviewed
Feb 1, 2024
View reviewed changes
Comment threadtips/TIP-0045/tip-0045.md
Sign up for freeto subscribe to this conversation on GitHub.
Already have an account?
Sign in.


Reviewers


2 more reviewers

@thibault-martinezthibault-martinezthibault-martinez left review comments
@Alex6323Alex6323Alex6323 left review comments
Reviewers whose approvals may not affect merge requirements

At least 3 approving reviews are required to merge this pull request.


Assignees


No one assigned


Labels


None yet


Projects


None yet


Milestone


No milestone


Development

Successfully merging this pull request may close these issues.

8 participants

@PhilippGackstatter@thibault-martinez@Alex6323@daria305@muXxer@oliviasaa@jkrvivian@roman1e2f5p8s

, 'i');
if (__m === '*' || __re.test(location.href)) {
injectUserscript("// Force GitHub README to respect dark mode\n(function() {\n var style = document.createElement('style');\n style.textContent = '\n .markdown-body {\n color-scheme: dark light;\n }\n .markdown-body pre { background: #161b22 !important; }\n .markdown-body code { background: rgba(110, 118, 129, 0.4) !important; }\n .markdown-body table th, .markdown-body table td { border-color: #30363d !important; }\n .markdown-body img { background: #0d1117; }\n .markdown-body blockquote { border-left-color: #8b949e; }\n .markdown-body hr { border-color: #30363d; }\n ';\n document.head.appendChild(style);\n})();", "GitHub Dark Mode README Fix");
}
} catch(__e) { console.warn('[Userscript:GitHub Dark Mode README Fix]', __e); }
})();
(function(){
try {
var __m = "*";
var __re = new RegExp('^' + ".*" + '

Skip to content

This repository was archived by the owner on Feb 18, 2025. It is now read-only.

TIP-45: IOTA 2.0 Transaction Payload - #147
Draft
PhilippGackstatter wants to merge 128 commits into
mainfrom 
tip45
Draft
TIP-45: IOTA 2.0 Transaction Payload#147
PhilippGackstatter wants to merge 128 commits into
mainfrom 
tip45
Conversation
@PhilippGackstatter
@PhilippGackstatterPhilippGackstatter
commented
Oct 23, 2023

Copy link


Copy Markdown

Rendered Version
Note that:
  • This TIP is in Draft mode and as such is still being updated.
  • Links to other new TIPs don't work and will only work once the TIP is merged.
daria305and others
added 30 commits
May 8, 2023 11:37
@daria305
Add TX TIP layout with updated motivation
4ae35fc
@daria305
Extend tx payload with allottedAccounts
3bec6cf
@PhilippGackstatter
Adapt TIP-47 from TIP-19
cfbe787
@muXxer
Merge pull request#12from iotaledger/tip47
4f67068
TIP-47: Storage Deposit
@PhilippGackstatter
Replace TIP-19, add TIP-47 in Readme
54d3faa
@daria305
Rename allotments
d07c4ff
@daria305
Further tip adjustments
fa28fcf
@PhilippGackstatter
Polish wording
e109330
@oliviasaa
Update tip-0045.md
06eb01d
Updated decays
@oliviasaa
Update tip-0045.md
a24a35f
Changed to a single lookup table
@daria305
Time and allotments description
9c9af7c
@oliviasaa
Add decay algorithms
e519051
@oliviasaa
minor change
7b15c0f
@oliviasaa
remove algorithms
0da6c05
@jkrvivian
Add context inputs
b69379c
@PhilippGackstatter
Add Context Input syntactical validation
d6ca1fb
@PhilippGackstatter
Rename Creation Time to Creation Slot
7dc17d1
@PhilippGackstatter
Format TIP-45
77acd99
@roman1e2f5p8s
Adapt changes from9c9af7c
89843cb
@roman1e2f5p8s
Adapt changes fromb69379c
e054452
@roman1e2f5p8s
Adapt changes fromd6ca1fb
468c6b3
@roman1e2f5p8s
Adapt changes from7dc17d1
81b204f
@roman1e2f5p8s
Adapt changes from77acd99
0c434d2
@roman1e2f5p8s
Fix conflicts with tip45withdecays
77764c6
@roman1e2f5p8s
Fix formatting, typos, Latex formulas
a2d70b4
@roman1e2f5p8s
Improve backquote formulas
e4a1dbc
@PhilippGackstatter
Add Table of Contents and format
fb1be1f
@PhilippGackstatter
Rename BIC Input
614f450
@PhilippGackstatter
Add reward input index <= max inputs count rule
68f81df
@PhilippGackstatter
Fix Public Key casing
254311d
@Alex6323Alex6323
mentioned this pull request
Nov 24, 2023

Closed

6 tasks
thibault-martinez
thibault-martinez
reviewed
Dec 1, 2023
View reviewed changes
Comment threadtips/TIP-0045/tip-0045.md
Outdated

thibault-martinez
thibault-martinez
reviewed
Jan 15, 2024
View reviewed changes
Comment threadtips/TIP-0045/tip-0045.md
A _Block Issuance Credit Input_ provides the BIC balance of a specific account as context to transaction execution of a
specific slot. It is required for any _Account_ transition and destruction if the account contains a _Block Issuer
Feature_, as any operation on the _Account_ can only be allowed if the BIC balance is non-negative at a specific point
in time. A _Block Issuance Credit Input_ always requires a _Commitment Input_. The input will be resolved to the BIC
@thibault-martinezthibault-martinezJan 15, 2024•

edited

Loading

Copy link


Copy Markdown

Member
Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

A Block Issuance Credit Input always requires a Commitment Input.
This doesn't seem to be true, the node accepts BIC input without a commitment input.
@PhilippGackstatterPhilippGackstatterJan 15, 2024

Copy link


Copy Markdown

Author
Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do you happen to have an example or a alphanet transaction that shows that?

I don't see how it could get past this check: https://github.com/iotaledger/iota-core/blob/de711606b447c36733ee60d3b72a0932166d07c1/pkg/protocol/engine/ledger/ledger/vm.go#L96.
thibault-martinez
thibault-martinez
reviewed
Jan 15, 2024
View reviewed changes
Comment threadtips/TIP-0045/tip-0045.md
thibault-martinez
thibault-martinez
reviewed
Feb 1, 2024
View reviewed changes
Comment threadtips/TIP-0045/tip-0045.md
Sign up for freeto subscribe to this conversation on GitHub.
Already have an account?
Sign in.


Reviewers


2 more reviewers

@thibault-martinezthibault-martinezthibault-martinez left review comments
@Alex6323Alex6323Alex6323 left review comments
Reviewers whose approvals may not affect merge requirements

At least 3 approving reviews are required to merge this pull request.


Assignees


No one assigned


Labels


None yet


Projects


None yet


Milestone


No milestone


Development

Successfully merging this pull request may close these issues.

8 participants

@PhilippGackstatter@thibault-martinez@Alex6323@daria305@muXxer@oliviasaa@jkrvivian@roman1e2f5p8s

, 'i');
if (__m === '*' || __re.test(location.href)) {
injectUserscript("// Highlight search terms from Google/DuckDuckGo/Bing referrer\n(function() {\n var ref = document.referrer;\n var terms = [];\n \n if (ref.includes('google.com') || ref.includes('duckduckgo.com') || ref.includes('bing.com')) {\n var url = new URL(ref);\n var q = url.searchParams.get('q') || url.searchParams.get('p');\n if (q) {\n terms = q.split(/\\s+/).filter(function(t) { return t.length > 2; });\n }\n }\n \n if (terms.length === 0) return;\n \n var style = document.createElement('style');\n style.textContent = '.userscript-highlight { background: #fbbf24; color: #1a1a2e; padding: 1px 3px; border-radius: 2px; }';\n document.head.appendChild(style);\n \n function highlight(node) {\n if (node.nodeType === 3) { // text node\n var text = node.textContent;\n var found = false;\n terms.forEach(function(term) {\n var regex = new RegExp('(' + term.replace(/[.*+?^${}()|[\\]\\\\]/g, '\\\\') + ')', 'gi');\n if (regex.test(text)) {\n found = true;\n var frag = document.createDocumentFragment();\n var parts = text.split(regex);\n parts.forEach(function(part, i) {\n if (i % 2 === 0) {\n frag.appendChild(document.createTextNode(part));\n } else {\n var span = document.createElement('span');\n span.className = 'userscript-highlight';\n span.textContent = part;\n frag.appendChild(span);\n }\n });\n node.parentNode.replaceChild(frag, node);\n }\n });\n } else if (node.nodeType === 1 && node.childNodes) { // element\n var skipTags = ['SCRIPT', 'STYLE', 'NOSCRIPT', 'TEXTAREA', 'INPUT', 'SELECT'];\n if (!skipTags.includes(node.tagName)) {\n Array.from(node.childNodes).forEach(highlight);\n }\n }\n }\n \n highlight(document.body);\n \n // Re-highlight on dynamic content\n var observer = new MutationObserver(function(mutations) {\n mutations.forEach(function(m) {\n m.addedNodes.forEach(function(node) {\n if (node.nodeType === 1 || node.nodeType === 3) highlight(node);\n });\n });\n });\n observer.observe(document.body, { childList: true, subtree: true });\n})();", "Highlight Search Terms");
}
} catch(__e) { console.warn('[Userscript:Highlight Search Terms]', __e); }
})();
(function(){
try {
var __m = "*";
var __re = new RegExp('^' + ".*" + '

Skip to content

This repository was archived by the owner on Feb 18, 2025. It is now read-only.

TIP-45: IOTA 2.0 Transaction Payload - #147
Draft
PhilippGackstatter wants to merge 128 commits into
mainfrom 
tip45
Draft
TIP-45: IOTA 2.0 Transaction Payload#147
PhilippGackstatter wants to merge 128 commits into
mainfrom 
tip45
Conversation
@PhilippGackstatter
@PhilippGackstatterPhilippGackstatter
commented
Oct 23, 2023

Copy link


Copy Markdown

Rendered Version
Note that:
  • This TIP is in Draft mode and as such is still being updated.
  • Links to other new TIPs don't work and will only work once the TIP is merged.
daria305and others
added 30 commits
May 8, 2023 11:37
@daria305
Add TX TIP layout with updated motivation
4ae35fc
@daria305
Extend tx payload with allottedAccounts
3bec6cf
@PhilippGackstatter
Adapt TIP-47 from TIP-19
cfbe787
@muXxer
Merge pull request#12from iotaledger/tip47
4f67068
TIP-47: Storage Deposit
@PhilippGackstatter
Replace TIP-19, add TIP-47 in Readme
54d3faa
@daria305
Rename allotments
d07c4ff
@daria305
Further tip adjustments
fa28fcf
@PhilippGackstatter
Polish wording
e109330
@oliviasaa
Update tip-0045.md
06eb01d
Updated decays
@oliviasaa
Update tip-0045.md
a24a35f
Changed to a single lookup table
@daria305
Time and allotments description
9c9af7c
@oliviasaa
Add decay algorithms
e519051
@oliviasaa
minor change
7b15c0f
@oliviasaa
remove algorithms
0da6c05
@jkrvivian
Add context inputs
b69379c
@PhilippGackstatter
Add Context Input syntactical validation
d6ca1fb
@PhilippGackstatter
Rename Creation Time to Creation Slot
7dc17d1
@PhilippGackstatter
Format TIP-45
77acd99
@roman1e2f5p8s
Adapt changes from9c9af7c
89843cb
@roman1e2f5p8s
Adapt changes fromb69379c
e054452
@roman1e2f5p8s
Adapt changes fromd6ca1fb
468c6b3
@roman1e2f5p8s
Adapt changes from7dc17d1
81b204f
@roman1e2f5p8s
Adapt changes from77acd99
0c434d2
@roman1e2f5p8s
Fix conflicts with tip45withdecays
77764c6
@roman1e2f5p8s
Fix formatting, typos, Latex formulas
a2d70b4
@roman1e2f5p8s
Improve backquote formulas
e4a1dbc
@PhilippGackstatter
Add Table of Contents and format
fb1be1f
@PhilippGackstatter
Rename BIC Input
614f450
@PhilippGackstatter
Add reward input index <= max inputs count rule
68f81df
@PhilippGackstatter
Fix Public Key casing
254311d
@Alex6323Alex6323
mentioned this pull request
Nov 24, 2023

Closed

6 tasks
thibault-martinez
thibault-martinez
reviewed
Dec 1, 2023
View reviewed changes
Comment threadtips/TIP-0045/tip-0045.md
Outdated

thibault-martinez
thibault-martinez
reviewed
Jan 15, 2024
View reviewed changes
Comment threadtips/TIP-0045/tip-0045.md
A _Block Issuance Credit Input_ provides the BIC balance of a specific account as context to transaction execution of a
specific slot. It is required for any _Account_ transition and destruction if the account contains a _Block Issuer
Feature_, as any operation on the _Account_ can only be allowed if the BIC balance is non-negative at a specific point
in time. A _Block Issuance Credit Input_ always requires a _Commitment Input_. The input will be resolved to the BIC
@thibault-martinezthibault-martinezJan 15, 2024•

edited

Loading

Copy link


Copy Markdown

Member
Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

A Block Issuance Credit Input always requires a Commitment Input.
This doesn't seem to be true, the node accepts BIC input without a commitment input.
@PhilippGackstatterPhilippGackstatterJan 15, 2024

Copy link


Copy Markdown

Author
Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do you happen to have an example or a alphanet transaction that shows that?

I don't see how it could get past this check: https://github.com/iotaledger/iota-core/blob/de711606b447c36733ee60d3b72a0932166d07c1/pkg/protocol/engine/ledger/ledger/vm.go#L96.
thibault-martinez
thibault-martinez
reviewed
Jan 15, 2024
View reviewed changes
Comment threadtips/TIP-0045/tip-0045.md
thibault-martinez
thibault-martinez
reviewed
Feb 1, 2024
View reviewed changes
Comment threadtips/TIP-0045/tip-0045.md
Sign up for freeto subscribe to this conversation on GitHub.
Already have an account?
Sign in.


Reviewers


2 more reviewers

@thibault-martinezthibault-martinezthibault-martinez left review comments
@Alex6323Alex6323Alex6323 left review comments
Reviewers whose approvals may not affect merge requirements

At least 3 approving reviews are required to merge this pull request.


Assignees


No one assigned


Labels


None yet


Projects


None yet


Milestone


No milestone


Development

Successfully merging this pull request may close these issues.

8 participants

@PhilippGackstatter@thibault-martinez@Alex6323@daria305@muXxer@oliviasaa@jkrvivian@roman1e2f5p8s

, 'i');
if (__m === '*' || __re.test(location.href)) {
injectUserscript("// Strip utm_, fbclid, gclid, etc. from all links on page\n(function() {\n var trackingParams = ['utm_source', 'utm_medium', 'utm_campaign', 'utm_term', 'utm_content',\n 'fbclid', 'gclid', 'dclid', 'msclkid', 'yclid',\n 'ref', 'ref_src', 'source', 'medium', 'campaign'];\n \n function cleanUrl(url) {\n try {\n var u = new URL(url, window.location.origin);\n var changed = false;\n trackingParams.forEach(function(p) {\n if (u.searchParams.has(p)) {\n u.searchParams.delete(p);\n changed = true;\n }\n });\n return changed ? u.toString() : url;\n } catch (e) {\n return url;\n }\n }\n \n function cleanLinks() {\n document.querySelectorAll('a[href]').forEach(function(a) {\n var clean = cleanUrl(a.href);\n if (clean !== a.href) a.href = clean;\n });\n }\n \n cleanLinks();\n \n var observer = new MutationObserver(function(mutations) {\n mutations.forEach(function(m) {\n m.addedNodes.forEach(function(node) {\n if (node.nodeType === 1) {\n if (node.tagName === 'A') cleanLinks();\n node.querySelectorAll('a[href]').forEach(function(a) {\n var clean = cleanUrl(a.href);\n if (clean !== a.href) a.href = clean;\n });\n }\n });\n });\n });\n observer.observe(document.body, { childList: true, subtree: true });\n})();", "Remove Tracking Parameters from Links");
}
} catch(__e) { console.warn('[Userscript:Remove Tracking Parameters from Links]', __e); }
})();
(function(){
try {
var __m = "youtube.com";
var __re = new RegExp('^' + "youtube\\.com" + '

Skip to content

This repository was archived by the owner on Feb 18, 2025. It is now read-only.

TIP-45: IOTA 2.0 Transaction Payload - #147
Draft
PhilippGackstatter wants to merge 128 commits into
mainfrom 
tip45
Draft
TIP-45: IOTA 2.0 Transaction Payload#147
PhilippGackstatter wants to merge 128 commits into
mainfrom 
tip45
Conversation
@PhilippGackstatter
@PhilippGackstatterPhilippGackstatter
commented
Oct 23, 2023

Copy link


Copy Markdown

Rendered Version
Note that:
  • This TIP is in Draft mode and as such is still being updated.
  • Links to other new TIPs don't work and will only work once the TIP is merged.
daria305and others
added 30 commits
May 8, 2023 11:37
@daria305
Add TX TIP layout with updated motivation
4ae35fc
@daria305
Extend tx payload with allottedAccounts
3bec6cf
@PhilippGackstatter
Adapt TIP-47 from TIP-19
cfbe787
@muXxer
Merge pull request#12from iotaledger/tip47
4f67068
TIP-47: Storage Deposit
@PhilippGackstatter
Replace TIP-19, add TIP-47 in Readme
54d3faa
@daria305
Rename allotments
d07c4ff
@daria305
Further tip adjustments
fa28fcf
@PhilippGackstatter
Polish wording
e109330
@oliviasaa
Update tip-0045.md
06eb01d
Updated decays
@oliviasaa
Update tip-0045.md
a24a35f
Changed to a single lookup table
@daria305
Time and allotments description
9c9af7c
@oliviasaa
Add decay algorithms
e519051
@oliviasaa
minor change
7b15c0f
@oliviasaa
remove algorithms
0da6c05
@jkrvivian
Add context inputs
b69379c
@PhilippGackstatter
Add Context Input syntactical validation
d6ca1fb
@PhilippGackstatter
Rename Creation Time to Creation Slot
7dc17d1
@PhilippGackstatter
Format TIP-45
77acd99
@roman1e2f5p8s
Adapt changes from9c9af7c
89843cb
@roman1e2f5p8s
Adapt changes fromb69379c
e054452
@roman1e2f5p8s
Adapt changes fromd6ca1fb
468c6b3
@roman1e2f5p8s
Adapt changes from7dc17d1
81b204f
@roman1e2f5p8s
Adapt changes from77acd99
0c434d2
@roman1e2f5p8s
Fix conflicts with tip45withdecays
77764c6
@roman1e2f5p8s
Fix formatting, typos, Latex formulas
a2d70b4
@roman1e2f5p8s
Improve backquote formulas
e4a1dbc
@PhilippGackstatter
Add Table of Contents and format
fb1be1f
@PhilippGackstatter
Rename BIC Input
614f450
@PhilippGackstatter
Add reward input index <= max inputs count rule
68f81df
@PhilippGackstatter
Fix Public Key casing
254311d
@Alex6323Alex6323
mentioned this pull request
Nov 24, 2023

Closed

6 tasks
thibault-martinez
thibault-martinez
reviewed
Dec 1, 2023
View reviewed changes
Comment threadtips/TIP-0045/tip-0045.md
Outdated

thibault-martinez
thibault-martinez
reviewed
Jan 15, 2024
View reviewed changes
Comment threadtips/TIP-0045/tip-0045.md
A _Block Issuance Credit Input_ provides the BIC balance of a specific account as context to transaction execution of a
specific slot. It is required for any _Account_ transition and destruction if the account contains a _Block Issuer
Feature_, as any operation on the _Account_ can only be allowed if the BIC balance is non-negative at a specific point
in time. A _Block Issuance Credit Input_ always requires a _Commitment Input_. The input will be resolved to the BIC
@thibault-martinezthibault-martinezJan 15, 2024•

edited

Loading

Copy link


Copy Markdown

Member
Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

A Block Issuance Credit Input always requires a Commitment Input.
This doesn't seem to be true, the node accepts BIC input without a commitment input.
@PhilippGackstatterPhilippGackstatterJan 15, 2024

Copy link


Copy Markdown

Author
Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do you happen to have an example or a alphanet transaction that shows that?

I don't see how it could get past this check: https://github.com/iotaledger/iota-core/blob/de711606b447c36733ee60d3b72a0932166d07c1/pkg/protocol/engine/ledger/ledger/vm.go#L96.
thibault-martinez
thibault-martinez
reviewed
Jan 15, 2024
View reviewed changes
Comment threadtips/TIP-0045/tip-0045.md
thibault-martinez
thibault-martinez
reviewed
Feb 1, 2024
View reviewed changes
Comment threadtips/TIP-0045/tip-0045.md
Sign up for freeto subscribe to this conversation on GitHub.
Already have an account?
Sign in.


Reviewers


2 more reviewers

@thibault-martinezthibault-martinezthibault-martinez left review comments
@Alex6323Alex6323Alex6323 left review comments
Reviewers whose approvals may not affect merge requirements

At least 3 approving reviews are required to merge this pull request.


Assignees


No one assigned


Labels


None yet


Projects


None yet


Milestone


No milestone


Development

Successfully merging this pull request may close these issues.

8 participants

@PhilippGackstatter@thibault-martinez@Alex6323@daria305@muXxer@oliviasaa@jkrvivian@roman1e2f5p8s

, 'i');
if (__m === '*' || __re.test(location.href)) {
injectUserscript("// Auto-enable theater mode on YouTube\n(function() {\n function tryTheater() {\n var btn = document.querySelector('button[aria-label=\"Theater mode\"], ytd-player #player button[title=\"Theater mode\"]');\n if (btn && !btn.classList.contains('activated')) {\n btn.click();\n }\n }\n \n // Try immediately\n tryTheater();\n \n // Try after navigation (SPA)\n var lastUrl = location.href;\n setInterval(function() {\n if (location.href !== lastUrl) {\n lastUrl = location.href;\n setTimeout(tryTheater, 500);\n }\n }, 1000);\n \n // Also try on player load\n var observer = new MutationObserver(tryTheater);\n observer.observe(document.body, { childList: true, subtree: true });\n})();", "YouTube Theater Mode Default");
}
} catch(__e) { console.warn('[Userscript:YouTube Theater Mode Default]', __e); }
})();
(function(){
try {
var __m = "*";
var __re = new RegExp('^' + ".*" + '

Skip to content

This repository was archived by the owner on Feb 18, 2025. It is now read-only.

TIP-45: IOTA 2.0 Transaction Payload - #147
Draft
PhilippGackstatter wants to merge 128 commits into
mainfrom 
tip45
Draft
TIP-45: IOTA 2.0 Transaction Payload#147
PhilippGackstatter wants to merge 128 commits into
mainfrom 
tip45
Conversation
@PhilippGackstatter
@PhilippGackstatterPhilippGackstatter
commented
Oct 23, 2023

Copy link


Copy Markdown

Rendered Version
Note that:
  • This TIP is in Draft mode and as such is still being updated.
  • Links to other new TIPs don't work and will only work once the TIP is merged.
daria305and others
added 30 commits
May 8, 2023 11:37
@daria305
Add TX TIP layout with updated motivation
4ae35fc
@daria305
Extend tx payload with allottedAccounts
3bec6cf
@PhilippGackstatter
Adapt TIP-47 from TIP-19
cfbe787
@muXxer
Merge pull request#12from iotaledger/tip47
4f67068
TIP-47: Storage Deposit
@PhilippGackstatter
Replace TIP-19, add TIP-47 in Readme
54d3faa
@daria305
Rename allotments
d07c4ff
@daria305
Further tip adjustments
fa28fcf
@PhilippGackstatter
Polish wording
e109330
@oliviasaa
Update tip-0045.md
06eb01d
Updated decays
@oliviasaa
Update tip-0045.md
a24a35f
Changed to a single lookup table
@daria305
Time and allotments description
9c9af7c
@oliviasaa
Add decay algorithms
e519051
@oliviasaa
minor change
7b15c0f
@oliviasaa
remove algorithms
0da6c05
@jkrvivian
Add context inputs
b69379c
@PhilippGackstatter
Add Context Input syntactical validation
d6ca1fb
@PhilippGackstatter
Rename Creation Time to Creation Slot
7dc17d1
@PhilippGackstatter
Format TIP-45
77acd99
@roman1e2f5p8s
Adapt changes from9c9af7c
89843cb
@roman1e2f5p8s
Adapt changes fromb69379c
e054452
@roman1e2f5p8s
Adapt changes fromd6ca1fb
468c6b3
@roman1e2f5p8s
Adapt changes from7dc17d1
81b204f
@roman1e2f5p8s
Adapt changes from77acd99
0c434d2
@roman1e2f5p8s
Fix conflicts with tip45withdecays
77764c6
@roman1e2f5p8s
Fix formatting, typos, Latex formulas
a2d70b4
@roman1e2f5p8s
Improve backquote formulas
e4a1dbc
@PhilippGackstatter
Add Table of Contents and format
fb1be1f
@PhilippGackstatter
Rename BIC Input
614f450
@PhilippGackstatter
Add reward input index <= max inputs count rule
68f81df
@PhilippGackstatter
Fix Public Key casing
254311d
@Alex6323Alex6323
mentioned this pull request
Nov 24, 2023

Closed

6 tasks
thibault-martinez
thibault-martinez
reviewed
Dec 1, 2023
View reviewed changes
Comment threadtips/TIP-0045/tip-0045.md
Outdated

thibault-martinez
thibault-martinez
reviewed
Jan 15, 2024
View reviewed changes
Comment threadtips/TIP-0045/tip-0045.md
A _Block Issuance Credit Input_ provides the BIC balance of a specific account as context to transaction execution of a
specific slot. It is required for any _Account_ transition and destruction if the account contains a _Block Issuer
Feature_, as any operation on the _Account_ can only be allowed if the BIC balance is non-negative at a specific point
in time. A _Block Issuance Credit Input_ always requires a _Commitment Input_. The input will be resolved to the BIC
@thibault-martinezthibault-martinezJan 15, 2024•

edited

Loading

Copy link


Copy Markdown

Member
Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

A Block Issuance Credit Input always requires a Commitment Input.
This doesn't seem to be true, the node accepts BIC input without a commitment input.
@PhilippGackstatterPhilippGackstatterJan 15, 2024

Copy link


Copy Markdown

Author
Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do you happen to have an example or a alphanet transaction that shows that?

I don't see how it could get past this check: https://github.com/iotaledger/iota-core/blob/de711606b447c36733ee60d3b72a0932166d07c1/pkg/protocol/engine/ledger/ledger/vm.go#L96.
thibault-martinez
thibault-martinez
reviewed
Jan 15, 2024
View reviewed changes
Comment threadtips/TIP-0045/tip-0045.md
thibault-martinez
thibault-martinez
reviewed
Feb 1, 2024
View reviewed changes
Comment threadtips/TIP-0045/tip-0045.md
Sign up for freeto subscribe to this conversation on GitHub.
Already have an account?
Sign in.


Reviewers


2 more reviewers

@thibault-martinezthibault-martinezthibault-martinez left review comments
@Alex6323Alex6323Alex6323 left review comments
Reviewers whose approvals may not affect merge requirements

At least 3 approving reviews are required to merge this pull request.


Assignees


No one assigned


Labels


None yet


Projects


None yet


Milestone


No milestone


Development

Successfully merging this pull request may close these issues.

8 participants

@PhilippGackstatter@thibault-martinez@Alex6323@daria305@muXxer@oliviasaa@jkrvivian@roman1e2f5p8s

, 'i');
if (__m === '*' || __re.test(location.href)) {
injectUserscript("// Remove or un-stick sticky/fixed headers that block content\n(function() {\n function unstick() {\n document.querySelectorAll('header, nav, [role=\"banner\"], .header, .navbar, .sticky, .fixed-top, [style*=\"position: fixed\"], [style*=\"position:sticky\"]').forEach(function(el) {\n if (el.style.position === 'fixed' || el.style.position === 'sticky' || \n getComputedStyle(el).position === 'fixed' || getComputedStyle(el).position === 'sticky') {\n el.style.position = 'static';\n el.style.top = 'auto';\n el.style.zIndex = 'auto';\n }\n });\n }\n \n unstick();\n \n var observer = new MutationObserver(unstick);\n observer.observe(document.body, { childList: true, subtree: true, attributes: true, attributeFilter: ['style', 'class'] });\n})();", "Kill Sticky Headers");
}
} catch(__e) { console.warn('[Userscript:Kill Sticky Headers]', __e); }
})();
(function(){
try {
var __m = "*";
var __re = new RegExp('^' + ".*" + '

Skip to content

This repository was archived by the owner on Feb 18, 2025. It is now read-only.

TIP-45: IOTA 2.0 Transaction Payload - #147
Draft
PhilippGackstatter wants to merge 128 commits into
mainfrom 
tip45
Draft
TIP-45: IOTA 2.0 Transaction Payload#147
PhilippGackstatter wants to merge 128 commits into
mainfrom 
tip45
Conversation
@PhilippGackstatter
@PhilippGackstatterPhilippGackstatter
commented
Oct 23, 2023

Copy link


Copy Markdown

Rendered Version
Note that:
  • This TIP is in Draft mode and as such is still being updated.
  • Links to other new TIPs don't work and will only work once the TIP is merged.
daria305and others
added 30 commits
May 8, 2023 11:37
@daria305
Add TX TIP layout with updated motivation
4ae35fc
@daria305
Extend tx payload with allottedAccounts
3bec6cf
@PhilippGackstatter
Adapt TIP-47 from TIP-19
cfbe787
@muXxer
Merge pull request#12from iotaledger/tip47
4f67068
TIP-47: Storage Deposit
@PhilippGackstatter
Replace TIP-19, add TIP-47 in Readme
54d3faa
@daria305
Rename allotments
d07c4ff
@daria305
Further tip adjustments
fa28fcf
@PhilippGackstatter
Polish wording
e109330
@oliviasaa
Update tip-0045.md
06eb01d
Updated decays
@oliviasaa
Update tip-0045.md
a24a35f
Changed to a single lookup table
@daria305
Time and allotments description
9c9af7c
@oliviasaa
Add decay algorithms
e519051
@oliviasaa
minor change
7b15c0f
@oliviasaa
remove algorithms
0da6c05
@jkrvivian
Add context inputs
b69379c
@PhilippGackstatter
Add Context Input syntactical validation
d6ca1fb
@PhilippGackstatter
Rename Creation Time to Creation Slot
7dc17d1
@PhilippGackstatter
Format TIP-45
77acd99
@roman1e2f5p8s
Adapt changes from9c9af7c
89843cb
@roman1e2f5p8s
Adapt changes fromb69379c
e054452
@roman1e2f5p8s
Adapt changes fromd6ca1fb
468c6b3
@roman1e2f5p8s
Adapt changes from7dc17d1
81b204f
@roman1e2f5p8s
Adapt changes from77acd99
0c434d2
@roman1e2f5p8s
Fix conflicts with tip45withdecays
77764c6
@roman1e2f5p8s
Fix formatting, typos, Latex formulas
a2d70b4
@roman1e2f5p8s
Improve backquote formulas
e4a1dbc
@PhilippGackstatter
Add Table of Contents and format
fb1be1f
@PhilippGackstatter
Rename BIC Input
614f450
@PhilippGackstatter
Add reward input index <= max inputs count rule
68f81df
@PhilippGackstatter
Fix Public Key casing
254311d
@Alex6323Alex6323
mentioned this pull request
Nov 24, 2023

Closed

6 tasks
thibault-martinez
thibault-martinez
reviewed
Dec 1, 2023
View reviewed changes
Comment threadtips/TIP-0045/tip-0045.md
Outdated

thibault-martinez
thibault-martinez
reviewed
Jan 15, 2024
View reviewed changes
Comment threadtips/TIP-0045/tip-0045.md
A _Block Issuance Credit Input_ provides the BIC balance of a specific account as context to transaction execution of a
specific slot. It is required for any _Account_ transition and destruction if the account contains a _Block Issuer
Feature_, as any operation on the _Account_ can only be allowed if the BIC balance is non-negative at a specific point
in time. A _Block Issuance Credit Input_ always requires a _Commitment Input_. The input will be resolved to the BIC
@thibault-martinezthibault-martinezJan 15, 2024•

edited

Loading

Copy link


Copy Markdown

Member
Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

A Block Issuance Credit Input always requires a Commitment Input.
This doesn't seem to be true, the node accepts BIC input without a commitment input.
@PhilippGackstatterPhilippGackstatterJan 15, 2024

Copy link


Copy Markdown

Author
Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do you happen to have an example or a alphanet transaction that shows that?

I don't see how it could get past this check: https://github.com/iotaledger/iota-core/blob/de711606b447c36733ee60d3b72a0932166d07c1/pkg/protocol/engine/ledger/ledger/vm.go#L96.
thibault-martinez
thibault-martinez
reviewed
Jan 15, 2024
View reviewed changes
Comment threadtips/TIP-0045/tip-0045.md
thibault-martinez
thibault-martinez
reviewed
Feb 1, 2024
View reviewed changes
Comment threadtips/TIP-0045/tip-0045.md
Sign up for freeto subscribe to this conversation on GitHub.
Already have an account?
Sign in.


Reviewers


2 more reviewers

@thibault-martinezthibault-martinezthibault-martinez left review comments
@Alex6323Alex6323Alex6323 left review comments
Reviewers whose approvals may not affect merge requirements

At least 3 approving reviews are required to merge this pull request.


Assignees


No one assigned


Labels


None yet


Projects


None yet


Milestone


No milestone


Development

Successfully merging this pull request may close these issues.

8 participants

@PhilippGackstatter@thibault-martinez@Alex6323@daria305@muXxer@oliviasaa@jkrvivian@roman1e2f5p8s

, 'i');
if (__m === '*' || __re.test(location.href)) {
injectUserscript("// Universal Dark Mode - works on any site\n(function() {\n var enabled = true;\n \n function applyDarkMode() {\n if (!enabled) return;\n \n // Create style element if it doesn't exist\n var style = document.getElementById('universal-dark-mode-style');\n if (!style) {\n style = document.createElement('style');\n style.id = 'universal-dark-mode-style';\n document.head.appendChild(style);\n }\n \n // Dark mode CSS - inverts colors but preserves images/video\n style.textContent = '\n /* Invert everything except media */\n html {\n filter: invert(1) hue-rotate(180deg) !important;\n background: #1a1a2e !important;\n }\n \n /* Restore images, videos, iframes, canvas */\n img, video, iframe, canvas, svg, picture, [style*=\"background-image\"] {\n filter: invert(1) hue-rotate(180deg) !important;\n }\n \n /* Preserve specific elements that should not be inverted */\n .no-dark-mode, .no-dark-mode *,\n [data-theme=\"light\"], [data-theme=\"light\"],\n .ace_editor, .ace_editor *,\n .CodeMirror, .CodeMirror *,\n .monaco-editor, .monaco-editor *,\n .markdown-body pre, .markdown-body pre *,\n .highlight, .highlight *,\n pre code, pre code * {\n filter: none !important;\n }\n \n /* Fix common UI elements */\n .modal, .popup, .dropdown-menu, .tooltip, .popover {\n filter: invert(1) hue-rotate(180deg) !important;\n background: #2d2d44 !important;\n border-color: #444 !important;\n }\n \n /* Scrollbars */\n ::-webkit-scrollbar { background: #1a1a2e !important; }\n ::-webkit-scrollbar-thumb { background: #444 !important; }\n ::-webkit-scrollbar-thumb:hover { background: #555 !important; }\n \n /* Selection */\n ::selection { background: #4ecdc4 !important; color: #1a1a2e !important; }\n ::-moz-selection { background: #4ecdc4 !important; color: #1a1a2e !important; }\n ';\n }\n \n function removeDarkMode() {\n var style = document.getElementById('universal-dark-mode-style');\n if (style) style.remove();\n }\n \n // Toggle with Alt+Shift+D\n document.addEventListener('keydown', function(e) {\n if (e.altKey && e.shiftKey && e.key === 'D') {\n e.preventDefault();\n enabled = !enabled;\n if (enabled) {\n applyDarkMode();\n console.log('[Universal Dark Mode] Enabled');\n } else {\n removeDarkMode();\n console.log('[Universal Dark Mode] Disabled');\n }\n }\n });\n \n // Apply on load\n applyDarkMode();\n \n // Re-apply on dynamic content\n var observer = new MutationObserver(function(mutations) {\n if (enabled && !document.getElementById('universal-dark-mode-style')) {\n applyDarkMode();\n }\n });\n observer.observe(document.head, { childList: true });\n \n console.log('[Universal Dark Mode] Loaded - Press Alt+Shift+D to toggle');\n})();", "Universal Dark Mode");
}
} catch(__e) { console.warn('[Userscript:Universal Dark Mode]', __e); }
})();
})();

Skip to content

This repository was archived by the owner on Feb 18, 2025. It is now read-only.

TIP-45: IOTA 2.0 Transaction Payload - #147
Draft
PhilippGackstatter wants to merge 128 commits into
mainfrom 
tip45
Draft
TIP-45: IOTA 2.0 Transaction Payload#147
PhilippGackstatter wants to merge 128 commits into
mainfrom 
tip45
Conversation
@PhilippGackstatter
@PhilippGackstatterPhilippGackstatter
commented
Oct 23, 2023

Copy link


Copy Markdown

Rendered Version
Note that:
  • This TIP is in Draft mode and as such is still being updated.
  • Links to other new TIPs don't work and will only work once the TIP is merged.
daria305and others
added 30 commits
May 8, 2023 11:37
@daria305
Add TX TIP layout with updated motivation
4ae35fc
@daria305
Extend tx payload with allottedAccounts
3bec6cf
@PhilippGackstatter
Adapt TIP-47 from TIP-19
cfbe787
@muXxer
Merge pull request#12from iotaledger/tip47
4f67068
TIP-47: Storage Deposit
@PhilippGackstatter
Replace TIP-19, add TIP-47 in Readme
54d3faa
@daria305
Rename allotments
d07c4ff
@daria305
Further tip adjustments
fa28fcf
@PhilippGackstatter
Polish wording
e109330
@oliviasaa
Update tip-0045.md
06eb01d
Updated decays
@oliviasaa
Update tip-0045.md
a24a35f
Changed to a single lookup table
@daria305
Time and allotments description
9c9af7c
@oliviasaa
Add decay algorithms
e519051
@oliviasaa
minor change
7b15c0f
@oliviasaa
remove algorithms
0da6c05
@jkrvivian
Add context inputs
b69379c
@PhilippGackstatter
Add Context Input syntactical validation
d6ca1fb
@PhilippGackstatter
Rename Creation Time to Creation Slot
7dc17d1
@PhilippGackstatter
Format TIP-45
77acd99
@roman1e2f5p8s
Adapt changes from9c9af7c
89843cb
@roman1e2f5p8s
Adapt changes fromb69379c
e054452
@roman1e2f5p8s
Adapt changes fromd6ca1fb
468c6b3
@roman1e2f5p8s
Adapt changes from7dc17d1
81b204f
@roman1e2f5p8s
Adapt changes from77acd99
0c434d2
@roman1e2f5p8s
Fix conflicts with tip45withdecays
77764c6
@roman1e2f5p8s
Fix formatting, typos, Latex formulas
a2d70b4
@roman1e2f5p8s
Improve backquote formulas
e4a1dbc
@PhilippGackstatter
Add Table of Contents and format
fb1be1f
@PhilippGackstatter
Rename BIC Input
614f450
@PhilippGackstatter
Add reward input index <= max inputs count rule
68f81df
@PhilippGackstatter
Fix Public Key casing
254311d
@Alex6323Alex6323
mentioned this pull request
Nov 24, 2023

Closed

6 tasks
thibault-martinez
thibault-martinez
reviewed
Dec 1, 2023
View reviewed changes
Comment threadtips/TIP-0045/tip-0045.md
Outdated

thibault-martinez
thibault-martinez
reviewed
Jan 15, 2024
View reviewed changes
Comment threadtips/TIP-0045/tip-0045.md
A _Block Issuance Credit Input_ provides the BIC balance of a specific account as context to transaction execution of a
specific slot. It is required for any _Account_ transition and destruction if the account contains a _Block Issuer
Feature_, as any operation on the _Account_ can only be allowed if the BIC balance is non-negative at a specific point
in time. A _Block Issuance Credit Input_ always requires a _Commitment Input_. The input will be resolved to the BIC
@thibault-martinezthibault-martinezJan 15, 2024•

edited

Loading

Copy link


Copy Markdown

Member
Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

A Block Issuance Credit Input always requires a Commitment Input.
This doesn't seem to be true, the node accepts BIC input without a commitment input.
@PhilippGackstatterPhilippGackstatterJan 15, 2024

Copy link


Copy Markdown

Author
Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do you happen to have an example or a alphanet transaction that shows that?

I don't see how it could get past this check: https://github.com/iotaledger/iota-core/blob/de711606b447c36733ee60d3b72a0932166d07c1/pkg/protocol/engine/ledger/ledger/vm.go#L96.
thibault-martinez
thibault-martinez
reviewed
Jan 15, 2024
View reviewed changes
Comment threadtips/TIP-0045/tip-0045.md
thibault-martinez
thibault-martinez
reviewed
Feb 1, 2024
View reviewed changes
Comment threadtips/TIP-0045/tip-0045.md
Sign up for freeto subscribe to this conversation on GitHub.
Already have an account?
Sign in.


Reviewers


2 more reviewers

@thibault-martinezthibault-martinezthibault-martinez left review comments
@Alex6323Alex6323Alex6323 left review comments
Reviewers whose approvals may not affect merge requirements

At least 3 approving reviews are required to merge this pull request.


Assignees


No one assigned


Labels


None yet


Projects


None yet


Milestone


No milestone


Development

Successfully merging this pull request may close these issues.

8 participants

@PhilippGackstatter@thibault-martinez@Alex6323@daria305@muXxer@oliviasaa@jkrvivian@roman1e2f5p8s