Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[RFC]: add console/log-each-map #13

Open
kgryte opened this issue Feb 14, 2025 · 4 comments · May be fixed by stdlib-js/stdlib#5357
Open

[RFC]: add console/log-each-map #13

kgryte opened this issue Feb 14, 2025 · 4 comments · May be fixed by stdlib-js/stdlib#5357
Assignees
@headlessNode
Labels
difficulty: 1 Low degree of difficulty. Should be straightforward to implement and/or resolve. estimate: 2-4hrs Task which should take between 2 to 4 hours. Feature Task to add a new feature. ❌ No AI Not allowed to use AI.

Comments

@kgryte
Copy link
Member

kgryte commented Feb 14, 2025 •
edited by headlessNode
Loading

Signature:

logEachMap( str[, ...args], clbk[, thisArg] )

Example usage:

function add( a, b ) {
    return a + b;
}

var x = [ 1, 2, 3 ];
var y = [ 4, 5, 6 ];

logEachMap( '%d + %d = %d', x, y, add )

Expected results:

1 + 4 = 5
2 + 5 = 7
3 + 6 = 9

Time estimate without AI: 3hrs
Time estimate with AI: 1-2hrs
@kgryte kgryte added difficulty: 1 Low degree of difficulty. Should be straightforward to implement and/or resolve. estimate: 2-4hrs Task which should take between 2 to 4 hours. Feature Task to add a new feature. labels Feb 14, 2025
@naterush naterush added 🤖 AI Allowed to use AI. ❌ No AI Not allowed to use AI. and removed 🤖 AI Allowed to use AI. labels Feb 17, 2025
@headlessNode headlessNode linked a pull request Feb 21, 2025 that will close this issue
Open
1 task
@headlessNode
Copy link
Member

headlessNode commented Feb 21, 2025 •
edited
Loading

Loom recording

PR resolving this issue:

Time till opening PR:

  • 4hr 17mins.

@headlessNode
Copy link
Member

As this issue was recently changed from AI to No-AI and me having implemented most of the function with AI prior to label change. We decided to continue the use of AI for this issue.

@headlessNode
Copy link
Member

Notes:

  • A heads-up, couldn't record the last few mins of time spent working on this issue. During those mins the last remaining files related to typescript definitions were added.
  • The function implementation using AI was done fairly quickly. Most of the time spent was on ensuring complete testing of the implementation and making everything inline with stdlib-js conventions.
  • Due to the complex signature of the function, AI was not able to accurately generate the typescript definitions. So that was done without the use of AI after a discussion with a project maintainer.

@kgryte
Copy link
Member Author

kgryte commented Feb 22, 2025

@naterush Should the label also be updated to reflect that AI was used?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@headlessNode headlessNode

Labels
difficulty: 1 Low degree of difficulty. Should be straightforward to implement and/or resolve. estimate: 2-4hrs Task which should take between 2 to 4 hours. Feature Task to add a new feature. ❌ No AI Not allowed to use AI.
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

feat: add @stdlib/console/logEachMap headlessNode/stdlib
3 participants
@kgryte @naterush @headlessNode