Skip to content

Conversation

@JuliaRegistrator
Copy link
Contributor

UUID: e5870fb4-0785-4c2b-93f1-21089c9623d1
Repo: https://github.com/stealth-lndrs/PoliSpectralTools.jl.git
Tree: ec250cd1a06b558bcf315d9aa5067ffdde9a3fa0

Registrator tree SHA: 50f504d641745716a5b3eabaf681d3a4937d2ae3
@github-actions
Copy link
Contributor

Hello, I am an automated registration bot. I help manage the registration process by checking your registration against a set of AutoMerge guidelines. If all these guidelines are met, this pull request will be merged automatically, completing your registration. It is strongly recommended to follow the guidelines, since otherwise the pull request needs to be manually reviewed and merged by a human.

1. New package registration

Please make sure that you have read the package naming guidelines.

2. AutoMerge Guidelines are all met! ✅

Your new package registration met all of the guidelines for auto-merging and is scheduled to be merged when the mandatory waiting period (3 days) has elapsed.

3. To pause or stop registration

If you want to prevent this pull request from being auto-merged, simply leave a comment. If you want to post a comment without blocking auto-merging, you must include the text [noblock] in your comment.

Tip: You can edit blocking comments to add [noblock] in order to unblock auto-merging.

@goerz
Copy link
Member

goerz commented Dec 13, 2025

Please note that registering a fork of another project is usually frowned upon. Can you clarify the relation to the original project? This would also need to be approved by @Vadinho70

The README looks a little bit like it might have been generated by an LLM, so I just want to make sure that you are aware of the LLM policies of the General registry

@JuliaTagBot JuliaTagBot added the AutoMerge: last run blocked by comment PR blocked by one or more comments lacking the string [noblock]. label Dec 13, 2025
@stealth-lndrs
Copy link

Please note that registering a fork of another project is usually frowned upon. Can you clarify the relation to the original project? This would also need to be approved by @Vadinho70

The README looks a little bit like it might have been generated by an LLM, so I just want to make sure that you are aware of the LLM policies of the General registry

Hello, @goerz.

Thanks for concerning about authorship.

Clarifying as requested: I am a PhD student at Polytechnic School of the University of São Paulo, who recently accomplished the Spectral Methods course, whose Professor is @Vadinho70. He proposed building a Julia package containing the methods we learned during the class as a final course project, and I was responsible for taking his original project, incrementing it with features, examples and tests, and finally shaping it into an actual package with detailed docs, so that we could publish for others to use. It is still a work in progress, but this first version is quite useful already, at least for the next students of the course.

I am going to talk to him in order to have his approval here. Is a comment from him in this pull request thread enough for an approval?

Regarding the LLM usage: indeed, I coded the package with the help of an LLM. Thanks for sharing the LLM Policies, I read them and understand. I have checked the code by eye and tested the features with a subset of the examples from the classes. There are still enhancements to be done, but the current features are correct under my limited knowledge. I still haven't written a section specifying how the LLM was used, can I do this in the next version of the package?

@goerz
Copy link
Member

goerz commented Dec 13, 2025

Thanks for the clarification! That sounds fine to me, and good enough to move forward (and, a valid reason for registering a fork)

I would recommend including this origin story briefly, somewhere in the README

@goerz goerz added the Override AutoMerge: ignore blocking comments Instructs automerge to ignore blocking comments label Dec 13, 2025
@JuliaTagBot JuliaTagBot removed the AutoMerge: last run blocked by comment PR blocked by one or more comments lacking the string [noblock]. label Dec 13, 2025
@Vadinho70
Copy link

This package was developed, with my approval, from some classroom scripts. The contributors have greatly expanded the functionality and documentation. It is very useful for anyone working with spectral methods, and I believe it would be interesting to make it public.

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

Labels

new package Override AutoMerge: ignore blocking comments Instructs automerge to ignore blocking comments

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants