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.

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

Reinstate the documentation generation routines #3132

Open
krzywon opened this issue Oct 24, 2024 · 5 comments
Open

Reinstate the documentation generation routines #3132

krzywon opened this issue Oct 24, 2024 · 5 comments

Comments

@krzywon
Copy link
Contributor

krzywon commented Oct 24, 2024

Describe the bug
Due to some late issues related to the documentation generation processes, they were disabled for the final v6.0.0 release. This is to track and note this all should be reinstated, as early as v6.0.1.

@ehb54
Copy link

ehb54 commented Nov 25, 2024

So is this why I don't have working help in the v6.0.0 linux release packages?
(Installed in a container ubuntu:20.04, also tried ubuntu:22.04, same results)

This is the response from Help->Documentation
image

And below, clicking Help on the Fit Panel
image

Here's what I get from flatpak on Rocky 8.10
Help->Documentation
image

Help on the Fit Panel
image

@ehb54
Copy link

ehb54 commented Dec 18, 2024

Note - I also had a documentation issue on OSX at the SAS2024 tutorial.
Windows seemed to work.

@jamescrake-merani
Copy link
Contributor

Note - I also had a documentation issue on OSX at the SAS2024 tutorial.

What specifically was the problem? I just tried it on the Mac we have in our team, and I could access the documentation from Help -> Documentation but I couldn't access the model documentation from clicking the Help button in the fitting window. I got the following error instead

08:03:14 - ERROR: Traceback (most recent call last): File "sas/qtgui/Perspectives/Fitting/FittingWidget.py", line 1891, in onHelp File "sas/qtgui/Perspectives/Fitting/FittingWidget.py", line 1903, in getHelpLocation KeyError: 'Plugin Models'

I was curious as to whether you got the same problem, something additional that doesn't work, or if its completely different on your machine?

@butlerpd
Copy link
Member

In principle this should not cause the problem pointed out by @ehb54. This should only have prevented plugin model docs from generating (which has always been the case - this was to add a new feature that compiled plugin model docs on the fly). But it is possible that it is due to some of the folder location changes in 6.0. In which case, were this "doc regeneration" working, it would have dealt with that problem? @krzywon would be most likely to know.

I'm curious @jamescrake-merani - the error you show is looking at Plugin Models which I think is where this leads to a problem? but I'm guessing you actually were clicking on help for a built in model? Is that correct?

@jamescrake-merani
Copy link
Contributor

but I'm guessing you actually were clicking on help for a built in model? Is that correct?

Yes I was looking at the sphere model which should be a builtin model to my understanding.

Looking at the error, this is the line in FittingWidget.py where it is happening

plugin_names = [name for name, enabled in self.master_category_dict[CATEGORY_CUSTOM]]

This seems to be the function where that self.master_category_dict is actually set.

    def readCategoryInfo(self):
        """
        Reads the categories in from file
        """
        self.master_category_dict = defaultdict(list)
        self.by_model_dict = defaultdict(list)
        self.model_enabled_dict = defaultdict(bool)

        categorization_file = CategoryInstaller.get_user_file()
        if not os.path.isfile(categorization_file):
            categorization_file = CategoryInstaller.get_default_file()
        with open(categorization_file, 'rb') as cat_file:
            self.master_category_dict = json.load(cat_file)
            self.regenerateModelDict()

        # Load the model dict
        models = load_standard_models()
        for model in models:
            self.models[model.name] = model

        self.readCustomCategoryInfo()

I don't have a debugger to play around with on the Mac but I wonder if it is something to do with this categorization_file. But yes its still also strange that its looking for plugin models.

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

No branches or pull requests

4 participants