[roi/profile] Polish

Polish things all around ROIs and profiles

I've just tested this and having the last profile properties stored is indeed quite nice. I've noticed however that initially, the profile appears on top of the plot widget which makes things a bit weird when drawing a line profile.
In #3047 you mentioned that it is possible to change the default of the last button to be "Compute profile sum". How exactly? ProfileOptionToolButton seems very hard to reach.
Finally, would it be possible to move the setProfileWindow method to the ProfileManager? This way I can set own type of profile window.

the profile window appears on top of the plot widget which makes things a bit weird

It is supposed to be displayed on the side (if you have enough space). That's the exact same algorithm as before. But at home i use a very small screen, And i suspect the profile window is bigger than previously. We have also added a Dialog window flag to the profile, this can be custom if you inherit the profile manager, see below.

you mentioned that it is possible to change the default of the last button

I have also mentioned it in the issue. The way a profile is computed (sum/mean, line width) is part of the ROI itself, so when you custom the name, you also custom that properties.
If it is not working, cause of the StorePropertystuff, maybe... I think we can had default values to this object (setDefaultMethod...). But if the first approach is working for you, i think it is enough.

Finally, would it be possible to move the setProfileWindow method to the ProfileManager?

Definitely no. It is also a per ROI concept. To be able to display many ROIs at the same time. That is how the cross ROI is working. But you have 2 solutions

You can use the previous ProfileToolBar (from silx.gui.qt.plot.Profile) which provide a backward compatibility (btw i don't know how it behave right now with the cross ROI).
You can inherit the Profile manager and create it on your own. It is what i am doing for Flint (to dock the profile windows). You can take a look at the module silx.gui.qt.plot.Profile which also inherit the profile to implement the backward compatibility feature (else i can show you a Flint commit).

you mentioned that it is possible to change the default of the last button

I have also mentioned it in the issue. The way a profile is computed (sum/mean, line width) is part of the ROI itself, so when you custom the name, you also custom that properties.
If it is not working, cause of the StorePropertystuff, maybe... I think we can had default values to this object (setDefaultMethod...). But if the first approach is working for you, i think it is enough.

Maybe you did not understand what I meant before. See how the method is set when the button is instantiated. I want to change that default to sum. I don't even need the average, but I don't mind if I can set the sum as the default.
https://github.com/silx-kit/silx/blob/c25787b07b2fdeca4c4226440dbae800a242c4ae/silx/gui/plot/PlotToolButtons.py#L269-L275

Finally, would it be possible to move the setProfileWindow method to the ProfileManager?

Definitely no. It is also a per ROI concept. To be able to display many ROIs at the same time. That is how the cross ROI is working. But you have 2 solutions

You can use the previous ProfileToolBar (from silx.gui.qt.plot.Profile) which provide a backward compatibility (btw i don't know how it behave right now with the cross ROI).
You can inherit the Profile manager and create it on your own. It is what i am doing for Flint (to dock the profile windows). You can take a look at the module silx.gui.qt.plot.Profile which also inherit the profile to implement the backward compatibility feature (else i can show you a Flint commit).

I don't understand this. Fine, it is a ROI concept, but I should be able to change the way all those windows look, what buttons have etc. Before I was using the ProfileTooBar, but if I want to customize the toolbar, I first need to instantiated it, then clear it, which is convoluted.
    profileWindow = PlotWidget()
    # toolBar = QToolBar("Profile", parent=widget)
    toolBar = ProfileToolBar(parent=widget, plot=widget, profileWindow=myProfileWindow)
    toolBar.clear()

    profileManager = manager.ProfileManager(parent=widget, plot=widget)
    profileManager.setItemType(image=True)
    profileManager.setActiveItemTracking(True)

    roiManager = profileManager.getRoiManager()

    horizontalLine = profileManager.createProfileAction(
        ProfileImageHorizontalLineROI, parent=toolBar
    )
    toolBar.addAction(horizontalLine)

    verticalLine = profileManager.createProfileAction(
        ProfileImageVerticalLineROI, parent=toolBar
    )
    toolBar.addAction(verticalLine)

"It's a ROI concept", means you can edit it from the ROI. So:
def configureMyRois(roi):
    roi.setName("I prefer writing more things here")
    roi.setProfileMethod("mean")

roiManager = profile.getRoiManager()
roiManager.sigInteractiveRoiCreated.connect(configureMyRois)

And that's it.

I don't really understand your problem with the tool bar.

If you use the ProfileToolBar from silx.gui.plot.Profile, obviously it is already feed with content. It was the case before and it is still the case now. It is provided for retro compatibility.
You can use the tool bar from silx.gui.plot.tools.profile.toolbar which is the new one.
And if you are still not fine with it copy-paste it and create your own tool bar qt.QToolBar.

Anyway:
myProfileWindow = PlotWidget()

This can't work. The profile window have to be a silx.gui.plot.tools.profile.manager.ProfileWindow (or simply a silx.gui.plot.ProfileMainWindow (which should be deprecated)). Or you have to implement it's API to manage new profile computation result.

Ok, let me try differently. Below is a minimal working example. I want two things:

When I do a profile to have a profile window without any buttons.
The "sum" should be selected for the new profile, and this should be reflected in the toolbar above.

import sys
import numpy as np

from silx.gui.qt import QApplication, QMainWindow, QToolBar
from silx.gui.plot import PlotWidget
from silx.gui.plot.Profile import manager
from silx.gui.plot.tools.profile.rois import (
    ProfileImageHorizontalLineROI,
    ProfileImageVerticalLineROI,
    ProfileImageCrossROI,
    ProfileImageLineROI,
)

class BasePlotWidget(PlotWidget):
    def __init__(self, parent=None, **kwargs):
        super(BasePlotWidget, self).__init__(parent, **kwargs)

def configureRois(roi):
    roi.setName("Cut")
    roi.setProfileMethod("mean")

def main():
    app = QApplication([])

    window = QMainWindow()
    widget = BasePlotWidget(parent=window)

    toolBar = QToolBar("Profile", parent=widget)

    profileManager = manager.ProfileManager(parent=widget, plot=widget)
    profileManager.setItemType(image=True)
    profileManager.setActiveItemTracking(True)

    roiManager = profileManager.getRoiManager()
    roiManager.sigInteractiveRoiCreated.connect(configureRois)

    horizontalLine = profileManager.createProfileAction(
        ProfileImageHorizontalLineROI, parent=toolBar
    )
    toolBar.addAction(horizontalLine)

    verticalLine = profileManager.createProfileAction(
        ProfileImageVerticalLineROI, parent=toolBar
    )
    toolBar.addAction(verticalLine)

    crossLine = profileManager.createProfileAction(ProfileImageCrossROI, parent=toolBar)
    toolBar.addAction(crossLine)

    freeLine = profileManager.createProfileAction(ProfileImageLineROI)
    toolBar.addAction(freeLine)

    cleanAction = profileManager.createClearAction(toolBar)
    toolBar.addAction(cleanAction)
    editorAction = profileManager.createEditorAction(toolBar)
    toolBar.addAction(editorAction)

    x = np.outer(np.linspace(-10, 10, 200), np.linspace(-10, 5, 150))
    data = np.sin(x) / x
    widget.addImage(data)

    window.setCentralWidget(widget)
    widget.addToolBar(toolBar)
    window.show()

    sys.exit(app.exec_())

if __name__ == "__main__":
    main()

If you want to show your own profile window you have to:
class MyProfileWindow(ProfileWindow):
    def __init__(self, parent=None, backend=None):
        super.__foo__()
        self._plot1D = here you can create the PlotWidget you want to use for 1D profiles
        # We should create a createPlot1D so what it is obvious, so let me know

class MyProfileManager(ProfileManager):

    def createProfileWindow(self, plot, roi):
        # and here you can create it
        return MyProfileWindow(plot)

That's the cleanest way.
If you was using PlotWidget with the old toolbar, we still can profile a compatibility for that. Let me know.

I will create now _createPlot1D to the ProfileWindow so what it is done.

I will create now _createPlot1D to the ProfileWindow so what it is done.

So I should rely on a private method to implement the behavior I want. Isn't that risky? Before I used the public API to define a custom plot widget for the profiles and this is now deprecated.

2. The "sum" should be selected for the new profile, and this should be reflected in the toolbar above.

Is this possible?

I would find it much easier to use to just allow for another argument to be passed to the initializer of the ProfileWindow like this:
def __init__(self, parent=None, backend=None, plotWidget=None)```

So I should rely on a private method

That's in fact a protected method. One you can inherit, but that you should not call from outside. It is explicitly documented, so i don't think there is a problem. But it could be public, sometimes i put public sometimes protected, usually i try to match the current class implementation.

The "sum" should be selected for the new profile

I already answer there: https://github.com/silx-kit/silx/pull/3048#issuecomment-628572015
What works for the name also works for the method.

I would find it much easier to just allow for another...

 And here concepts do not match. The new profile expect many profile windows, so anyway it must require a factory of window and not an instance of window. And as you also see it exists many kind of profile, in 1D or 2D. So this stuffs was complex.

The "sum" should be selected for the new profile

I already answer there: #3048 (comment)
What works for the name also works for the method.

I'm sorry but that doesn't work. Please see the code that I've posted. I don't see the QMenu changing.

I would find it much easier to just allow for another...

 And here concepts do not match. The new profile expect many profile windows, so anyway it must require a factory of window and not an instance of window. And as you also see it exists many kind of profile, in 1D or 2D. So this stuffs was complex.

Ok, a setter then. I will try to work a bit on this. I find the current way to customize the plot window of the profile very convoluted.

I'm sorry but that doesn't work.

Then that's a bug, I will take a look.

I copy-pasted your code, everything is working. I think i don't understand your problem. Do you use this this branch?

Yes, I am using this branch. Here is a recording. I want that menu to be on the "sum".

In your example you have used mean.
I have tested it with both. Both are working.

In your example you have used mean.
I have tested it with both. Both are working.

Well, this is embarrassing. Sorry about that. :disappointed:

You'r welcome. But i really was questioning myself about the way i explain things :-?

There is an issue when setting the labels of the profile window.

Below is part of the code I'm using:
class BasePlotWidget(PlotWidget):
    def __init__(self, parent=None, **kwargs):
        super(BasePlotWidget, self).__init__(parent, **kwargs)
        self.setGraphXLabel("BaseX")
        self.setGraphYLabel("BaseY")

class CustomProfileWindow(ProfileWindow):
    def _createPlot1D(self, parent, backend):
        plot = PlotWidget(parent=parent, backend=backend)
        plot.setGraphXLabel("ProfileX")
        plot.setGraphYLabel("ProfileY")
        return plot

class CustomProfileManager(ProfileManager):
    def createProfileWindow(self, plot, roi):
        return CustomProfileWindow(plot)

The axis label is part of the profile data. So if you want to enforce something, you have to set it while you receive the profile data. So you have to inherit ProfileWindow.setProfile() or one of the protected class rendering one of the kind of profile data.
BTW, this setProfile could be renamed to setProfileData, for clarity. What do you think @t20100?
For me it is not a bug. Except you have a generic way, using the main plot information, through the computed profile, to provide maybe a better axis by default. But if you want to custom it here, i don't think there is choices.

BTW, to avoid to inherit the ProfileManager we can create a class setter. So you can use the default one. Let me know what you think.
profileManager.setProfileWindowClass(CustomProfileWindow)

But that's mostly all we can do.

The axis label is part of the profile data. So if you want to enforce something, you have to set it while you receive the profile data. So you have to inherit ProfileWindow.setProfile() or one of the protected class rendering one of the kind of profile data.
BTW, this setProfile could be renamed to setProfileData, for clarity. What do you think @t20100?
For me it is not a bug. Except you have a generic way, using the main plot information, through the computed profile, to provide maybe a better axis by default. But if you want to custom it here, i don't think there is choices.

So, in my case, I would have to reimplement all the logic in _setImageProfile just to set the label of the plot? :anguished:

BTW, to avoid to inherit the ProfileManager we can create a class setter. So you can use the default one. Let me know what you think.
profileManager.setProfileWindowClass(CustomProfileWindow)

But that's mostly all we can do.

Thanks!

So I should rely on a private method

That's in fact a protected method. One you can inherit, but that you should not call from outside. It is explicitly documented, so i don't think there is a problem. But it could be public, sometimes i put public sometimes protected, usually i try to match the current class implementation.

Could you please make it public? I hope it will not be problematic if in my implementation the function does not return an instance of Plot1D.

For the example shown by @mretegan somehow I recall a discussion where we said that when using horizontal or vertical profiles the X axis should take the corresponding axis label associated to the original image.

For the example shown by @mretegan somehow I recall a discussion where we said that when using horizontal or vertical profiles the X axis should take the corresponding axis label associated to the original image.

This is actually what I am trying to achieve 😄. Whenever possible use the labels of the original plot, and even set the title of the cut accordingly.

Then that is clearly a regression:
https://github.com/silx-kit/silx/pull/2395

Perhaps not a regression but a half-way implementation in #2395. The actual coordinates are used but effectively it was not explicitly requested to use the labels.

So, in my case, I would have to re-implement all the logic in _setImageProfile just to set the label of the plot? anguished

No, you just have to call super, and set the label after that.
But i agree with @vasole, it is what i tried to explain. If the information is already known, there is no reason to custom it. In short, fix the code, instead of convoluting it.

Looking at the code, i think it is not a regression. But we can easily display the right label from the parent plot. So i will do it.

Nice! Just a final idea: maybe the label of the y-axis could be the name of the method instead of simply "Profile".

That's a good idea. Is something have an advice here?

Here is the result. I have also specified the line width in the yLabel. I think it's a good place, but i feel like it is not convenient to read. Then maybe the method alone if better. Let me know.

My ultimate commit, if there is no more feedback.
I will switch to another PR to fix a specific issue.

It's fine now. This PR can be reviewed.

Oh i forget about this width info... I will try to make it consistent it the diagram title

It's fine now

Alleluia

I've just tested it and all works as expected. Very nice work Valentin, and thanks for your patience :smile:

Thanks. I was expecting to finish it earlier. But that's nice you spend time to test it. It is really nice to be sure everything is working now, and not in a bug fix release.

Just to add one last thing (maybe not very important), this is the most commended pull request in the silx repository.

Also the only one with a video!

hmmm weird error...

omgwtfitrulzbro

Finally tests look also fine with PySide2.


[Question] Can I activate an eSIM without being a carrier?

Hi,
Sorry if this is the wrong place to ask, but... Considering that my company resells eSIMs, is it possible for an app to activate other carriers' eSIMs?
Currently, we provide the QRCode for the activation, it works for desktop users but is not a good experience for mobile users.
I read through Android docs and it seems to me that only carriers and their own signed apps can do it, is that correct? Does the same apply to Apple? Do they only approve the entitlement if you are the carrier activating your own eSIMs?
Thanks!

in iOS iOS 17.4 now you can with an URL


Improved methods for saving results

User can now save a lighter version of results dictionary to accommodate post-processing.
Lighter version removes chains from results dictionary to keep json file a manageable size.
Methods added to ChainProcessing module for loading serial and parallel MCMC simulations.  These methods load the results dictionary from json, then look to see if the chains are included.  If not, then it searches for log files matching the specified extension.  If none are found, then it returns an empty list for each type of chain.
This should help to address #55 .

Pull Request Test Coverage Report for Build 410

60 of 72 (83.33%)  changed or added relevant lines in 5 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.4%) to 97.565%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

pymcmcstat/MCMC.py
3
4
75.0%

pymcmcstat/structures/ResultsStructure.py
2
13
15.38%

Totals

Change from base Build 404:
-0.4%

Covered Lines:
2324

Relevant Lines:
2382

💛  - Coveralls


Use uv for dependencies handling

Closes #30
This PR implements the following changes :

Recommends to install uv and use it to run cookiecutter using uvx
Uses uv venv to create the virtual env
Uses uvx to build the package using build.
Updates the ruff version
Restricts the ruff checks to only src/ and test/ folders
Update the GH Actions to run tests using uv too

Potential improvements :

Rename the default virtual env name to .env and use the environment auto detection feature from uv.

Note to myself: not yet ready because workflows are not installing uv yet


Run sdist and wheel building with network isolation

Prevent sdist and wheel build steps from accessing the internet. externalcommand.run() can use now run commands with unshare. unshare --net --map-current-user creates a new network namespace without access network.
Fixes: #263

fromager enables network isolation by default. I could make it opt-in.


Sink refactor

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.Nir Tsruya seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


Fix for [Doubly nested subcommand creates wrong help output]

Related this issue https://github.com/rails/thor/issues/627.
If we don't put command name (basename) there, it makes sense as arguments after the command.
Since it seems to take time to get subcommand names there, I think it's the solution for now.

@KishiTheMechanic can you add a test for this behavior so we can validate it works as expected?

I made #756 and close #755 now.

Little bit late on that, but I monkeypatched that here if somebody else has that issue.


Add C++ demo

This is my attempt at creating demo for C++. I will flesh it out further if there's any issue :grin:

This is great. Thank you!


VarAccess raises TraitErrorsException for accesses in lambda expressions

At the moment, there is no support in OpenRewrite for handling variable accesses within lambda expressions, causing the following test cases to fail:
@Test
void lambdaException() {
    rewriteRun(
      java(
        """
          import java.util.concurrent.FutureTask;

          class Test {
              void foobar() {
                  FutureTask<Exception> task = new FutureTask<>(() -> {
                      try {
                          return null;
                      } catch (Exception e) {
                          return e;
                      }
                  });
              }
          }
          """
      )
    );
}

with the exception:
Caused by: org.openrewrite.analysis.trait.util.TraitErrorsException: TraitErrors:
	- Method must be created from class org.openrewrite.java.tree.J$MethodDeclaration but was class org.openrewrite.java.tree.J$ControlParentheses
	- InstanceInitializer must be created from class org.openrewrite.java.tree.J$Block but was class org.openrewrite.java.tree.J$Lambda
	- Field must be declared in a class, interface, or anonymous class
	at org.openrewrite.analysis.trait.util.TraitErrors.doThrow(TraitErrors.java:43)
	at fj.data.Either$RightProjection.on(Either.java:531)
	at fj.data.Validation.on(Validation.java:146)
	at org.openrewrite.analysis.trait.expr.VarAccessBase$1.visitVariable(VarAccess.java:190)
	at org.openrewrite.analysis.trait.expr.VarAccessBase$1.visitVariable(VarAccess.java:178)
	at org.openrewrite.java.tree.J$VariableDeclarations$NamedVariable.acceptJava(J.java:5627)
	at org.openrewrite.java.tree.J.accept(J.java:59)
	at org.openrewrite.TreeVisitor.visit(TreeVisitor.java:278)
	...

and the test:
@Test
void lambdaNewInstance() {
    rewriteRun(
      java(
        """
          class A {
              static A newInstance() {
                  return new A();
              }
          }

          class Test {
              ThreadLocal<A> foobar =
                  ThreadLocal.withInitial(() -> {
                      A a = A.newInstance();
                      return a;
                  });
          }
          """
      )
    );
}

with the exception:
Caused by: org.openrewrite.analysis.trait.util.TraitErrorsException: TraitErrors:
	- Method must be created from class org.openrewrite.java.tree.J$MethodDeclaration but was class org.openrewrite.java.tree.J$Block
	- No parent Method found
	- Field must be declared in a class, interface, or anonymous class
	at org.openrewrite.analysis.trait.util.TraitErrors.doThrow(TraitErrors.java:43)
	at fj.data.Either$RightProjection.on(Either.java:531)
	at fj.data.Validation.on(Validation.java:146)
	at org.openrewrite.analysis.trait.expr.VarAccessBase$1.visitVariable(VarAccess.java:190)
	at org.openrewrite.analysis.trait.expr.VarAccessBase$1.visitVariable(VarAccess.java:178)
	at org.openrewrite.java.tree.J$VariableDeclarations$NamedVariable.acceptJava(J.java:5627)
	at org.openrewrite.java.tree.J.accept(J.java:59)
	at org.openrewrite.TreeVisitor.visit(TreeVisitor.java:278)
	...

In the future, this may involve adding a LambdaExpr trait to the trait API in rewrite-analysis to distinguish lambda expressions.

As noted in the linked PR the issue was mostly linked to the handling of anonymous subclasses.


why the amount is not transferred to POOL

Describe the bug
A clear and concise description of what the bug is.
I have transferred the amount to the user but it is not received by the user..hence the txid is generated but showing nothing in the explorer...the txid is not showing in the explorer..please help..
Transaction ID  : 77c9e32cff44e3a36d80ac9619687b12184df0d00a7c5370a655439cfe374fa1
Address : SPW9J5KWAWS7QJ759KHTYM70S6TQHNB17GWCWY9N
Block#
Time stamp : 20 Jul 2023 10:18:38
To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior
A clear and concise description of what you expected to happen.
the expectation is that the pool transferred the amount to user and after that the amount will again transferred to POOL account..but the pool is able to transferred and user received it..but user cannot transferred the amount to pool ..but the txid is generated which is not showing in the explorer
Screenshots
If applicable, add screenshots or consol.log to help explain your problem.

Desktop (please complete the following information):

OS: [e.g. iOS]
Browser [e.g. chrome, safari]
Version [e.g. 22]

Smartphone (please complete the following information):

Device: [e.g. iPhone6]
OS: [e.g. iOS8.1]
Browser [e.g. stock browser, safari]
Version [e.g. 22]

Additional context
Add any other context about the problem here:

Is this an Explorer issue or stacking pool issue?

This isn't an Explorer issue.
Please ask for community support on the #support channel in Discord.


Use correct visibility value in addGroup(Group)

The implementation org.gitlab4j.api.GroupApi.addGroup(Group) has in small bug:
The parameter visibility is filled with the value of group.getDescription() instead of  group.getVisibility()
Hence the request is failing, if description is set. Also it's not possible to change the default visibility private.

@mdeknowis
Once again, thank you for your contribution.

@gmessner Thanks!


Split function returns error for valid Jsonata expression

For the JSON below:
{
  "predictions": [
    {
      "fields": [
        "Response_message_field"
      ],
      "values": [
        [
          "#Inventory #Signature"
        ]
      ]
    }
  ]
}

The expression predictions.values[0][0].$split ($,"#") returns the error:
Argument 3 of function $split does not match function signature

The $ parameter isn't needed, but it should still work. I'll see if I can get this fixed. In the meantime, just use $split("#") and you'll get the same results as https://try.jsonata.org/LbQMukM1R

I've pushed a fix to the Split code to handle if a $ is passed...


Pr/doc adjustments

Small adjustments to the documentation
@obackhouse @alejandrosantanabonilla
I added the docs/source/apidoc directory for sphinx-apidoc files, since I think it's good to separate automatically created files in their own directory. I changed the call in workflows/ci.yaml to sphinx-apidoc -o source/apidoc -t templates ../vayesta/ is this all that needs to be done?
@ghb24 I reverted/changed some minor things of your corrections, in case you want to have a look

Let's waut for the tests to finish and see if the passed, including creating correctly the Docs. @obackhouse , the github.ref is for running the docs only for the master, right? Maybe, the branches workflow keyword might be useful here?
on:
push:
branches:
- '*'
- '!master'
This is running all tests but in the master one. If not, then we can just leave the github.ref, might be easier to follow the .yml file.

I think actually the changes will be deployed directly from this PR once the CI gets past installation and tests so you can double check online before merging. That's a bad idea but probably fine for now because it means we can preview the changes made by PRs.

However, if there are multiple PRs, the latter will overwrite the docs of the former?
Can we have two worksflows, once for the master branch deploying the main documentation, and another one for PRs, deploying to a PR specific link?

Let's waut for the tests to finish and see if the passed, including creating correctly the Docs. @obackhouse , the github.ref is for running the docs only for the master, right? Maybe, the branches workflow keyword might be useful here?
on: push: branches: - '*' - '!master'
This is running all tests but in the master one. If not, then we can just leave the github.ref, might be easier to follow the .yml file.

not sure what you mean here, but the current ci.yaml runs on pushes and PRs to master branch, but we only want the docs to deploy on pushes to master (and not PRs). We could of course have a separate workflow for the docs with a separate schedule but it is nice to have them combined. But that's easily controlled by the change I stated above.

Indeed, that is the point. So the PRs and pushes for all branches except master are carried out by this command. So, I think this command could be used to fork the PRs and push events using the different branches. we can split the workflow and say, all PRs and push events of all other branches will not build the docs. The PRs and Push events of the docs are only built using master....

Indeed, that is the point. So the PRs and pushes for all branches except master are carried out by this command. So, I think this command could be used to fork the PRs and push events using the different branches. we can split the workflow and say, all PRs and push events of all other branches will not build the docs. The PRs and Push events of the docs are only built using master....

yes, that is what my suggested change does

alright, so i understood correctly what you wanted to do. We can have also the option of the branches, giving us the freedom to decide what to push for the different branches...

yeah, that's one issue. I don't think you can have two on the same repo but I suppose we could open a new repo for testing the deployment. I think I could then set up a workflow in this main repo that deploys onto the testing repo without having to PR to the testing repo manually - if you think it's worth the effort

I think for the time being we can just prevent PRs from deploying the docs, if thats easier

Could you add that change? Thanks

done, please just double check that it all passes before merging

So the onlien docs atm don't seem quite right:
in https://boothgroup.github.io/Vayesta/quickstart/ewf.html the code snippets are missing - in the DMET quickstart they are present.
Are these docs generated from the master branch or this PR? If I build them locally from either branch, the code snippets are present

The docs should be created from this PR, since it is not merged yet. I think the problem is the relative path we are using for referencing the .py files. In ewf, we have the following path:
/../../../vayesta/example/....
whereas in edmet, we have:
/../../examples/ewf/molecules/01-simple-ccsd.py
I remember I made a comment to be merged in tutorials for changing this relative path. In ay case, it is easy to change, we need to change all the relative paths in ewf to the latter form and the correct file will be called.

I have changed the relative paths in the main/docs folder. I have commited those changes. This should solve the problem


Write a JavaScript program to convert centimeters to inches

Description
Write a JavaScript program to convert centimeters to inches
1 inch = 2.54 centimeters
Input  : 12.7
Output : 5

How to contribute

Save the solution in program/ConvertCentimetersToInches.js file
Add ConvertCentimetersToInches.js file in convert-centimeters-to-inches folder

assign me

assign me

!assign

Hey @Robert-Capra
You can comment !assign on any good first issue you want to contribute to, to be auto-assigned
!assign


Maintained?

Hi @sandfox,
We are looking for a way to parse any/all formats of ARN.
At present, this module only supports the older ARNs (with 6 segments) but would not understand a  Lambda ARN (which can have 7 or 8 segments):
http://docs.aws.amazon.com/general/latest/gr/aws-arns-and-namespaces.html#arn-syntax-lambda
Would you accept a PR updating it to support more ARN types?

Hi @nelsonic,
Totally down for improvements etc and more than happy to publish them! I've been meaning to de-coffee script this for ages, so if you can wait a few hours I should have the non-cs branch version on master to work from.

@sandfox I can de-coffee it if you like ...

@nelsonic ahh, just saw this after I de-cs'd it! I've just pushed up a new version. The changes are minor.

updates on this?


1.4.2 - Release notes

Vc version / revision
Operating System
Compiler & Version
Compiler Flags
Assembler & Version
CPU

     1.4.2             |        all          |         all           |                |                     |

Testcase

Actual Results
Expected Results
Release notes summarizing the full changelog and closed issues

The relevant closed issues since 1.4.1 seem to be: #221 , #226 , #228 , #231 , #232,  #247 , #250 , #253

A pointer to previous release notes would be useful to have.

I drafted the release notes on the release page here based on the statements of previous releases: https://github.com/VcDevel/Vc/releases


fix: upgrade to version 3.1.259

from: https://github.com/jenkins-x/jx-cli

@jenkins-x-bot-test: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

vault
68de8e28caf8831b6a839aea290bc3f8f0d9a0c7
link
/test vault

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the jenkins-x/lighthouse repository. I understand the commands that are listed here.

@jenkins-x-bot-test: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

vault
68de8e28caf8831b6a839aea290bc3f8f0d9a0c7
link
/test vault

tls
68de8e28caf8831b6a839aea290bc3f8f0d9a0c7
link
/test tls

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the jenkins-x/lighthouse repository. I understand the commands that are listed here.

@jenkins-x-bot-test: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

vault
68de8e28caf8831b6a839aea290bc3f8f0d9a0c7
link
/test vault

tls
68de8e28caf8831b6a839aea290bc3f8f0d9a0c7
link
/test tls

gsm
68de8e28caf8831b6a839aea290bc3f8f0d9a0c7
link
/test gsm

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the jenkins-x/lighthouse repository. I understand the commands that are listed here.

@jenkins-x-bot-test: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

vault
68de8e28caf8831b6a839aea290bc3f8f0d9a0c7
link
/test vault

tls
68de8e28caf8831b6a839aea290bc3f8f0d9a0c7
link
/test tls

gsm
68de8e28caf8831b6a839aea290bc3f8f0d9a0c7
link
/test gsm

kube
68de8e28caf8831b6a839aea290bc3f8f0d9a0c7
link
/test kube

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the jenkins-x/lighthouse repository. I understand the commands that are listed here.

@jenkins-x-bot-test: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

tls
68de8e28caf8831b6a839aea290bc3f8f0d9a0c7
link
/test tls

gsm
68de8e28caf8831b6a839aea290bc3f8f0d9a0c7
link
/test gsm

kube
68de8e28caf8831b6a839aea290bc3f8f0d9a0c7
link
/test kube

vault
68de8e28caf8831b6a839aea290bc3f8f0d9a0c7
link
/test vault

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the jenkins-x/lighthouse repository. I understand the commands that are listed here.

@jenkins-x-bot-test: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

gsm
68de8e28caf8831b6a839aea290bc3f8f0d9a0c7
link
/test gsm

kube
68de8e28caf8831b6a839aea290bc3f8f0d9a0c7
link
/test kube

vault
68de8e28caf8831b6a839aea290bc3f8f0d9a0c7
link
/test vault

tls
68de8e28caf8831b6a839aea290bc3f8f0d9a0c7
link
/test tls

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the jenkins-x/lighthouse repository. I understand the commands that are listed here.

@jenkins-x-bot-test: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

kube
68de8e28caf8831b6a839aea290bc3f8f0d9a0c7
link
/test kube

vault
68de8e28caf8831b6a839aea290bc3f8f0d9a0c7
link
/test vault

tls
68de8e28caf8831b6a839aea290bc3f8f0d9a0c7
link
/test tls

gsm
68de8e28caf8831b6a839aea290bc3f8f0d9a0c7
link
/test gsm

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the jenkins-x/lighthouse repository. I understand the commands that are listed here.

@jenkins-x-bot-test: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

vault
68de8e28caf8831b6a839aea290bc3f8f0d9a0c7
link
/test vault

tls
68de8e28caf8831b6a839aea290bc3f8f0d9a0c7
link
/test tls

gsm
68de8e28caf8831b6a839aea290bc3f8f0d9a0c7
link
/test gsm

kube
68de8e28caf8831b6a839aea290bc3f8f0d9a0c7
link
/test kube

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the jenkins-x/lighthouse repository. I understand the commands that are listed here.


Added ALC892 layout-id 28 for Clevo P751DMG

Tested on my Clevo P751DMG.
layout-id = 28
HDMI and DisplayPort Audio also tested if HDAU is correct.
Maybe also work for P751DM, P750DMG, P750DM, not tested.
Only difference between P751DMG and P751DM is G-Sync.

Thanks


[CS2103-F09-1] MyGM

MyGM is a desktop app for high school basketball team trainers to manage players’ contacts and data. It is optimized for use via a Command Line Interface (CLI) so that frequent tasks can be done faster by typing in commands.

Codecov Report

Merging #60 (3c85e85) into master (4f7660a) will not change coverage.
The diff coverage is n/a.

:exclamation: Current head 3c85e85 differs from pull request most recent head 8440b59. Consider uploading reports for the commit 8440b59 to get more accurate results

@@            Coverage Diff            @@
##             master      #60   +/-   ##
=========================================
  Coverage     72.15%   72.15%
  Complexity      399      399
=========================================
  Files            70       70
  Lines          1232     1232
  Branches        125      125
=========================================
  Hits            889      889
  Misses          311      311
  Partials         32       32

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 4f7660a...8440b59. Read the comment docs.


Could you provide a code example of adding VGG-16 into backbone and using it in faster rcnn?

I need to implement the Faster-rcnn + VGG16. But I dont find the VGG backbone of faster rcnn. There is only a file about VGG under mmdet/models/backbones, that is ssd_vgg,py. It can be used directly in fasterrcnn config file? I hope the tutorial can be more detailed. Could you provide a more detailed tutorial about add a VGG backbone into Faster rcnn? A colab example is more appreciated. Thanks.

what should i pay attention to when implementing my task based on the config file of mmdetection/configs/faster_rcnn/faster_rcnn_r50_fpn_1x_coco.py. For example, the input channel of FPN or else.

We don't support Faster R-CNN with backbone VGG16. But you can implement it by yourself. The logic should be similar to ResNet which receives an image and output a sequence of features. You can refer to other public implementations of VGG16 Faster R-Cnn and try to migrate it to MMDetection.

Hi,bro!~
Have you implemented Faster R-CNN with backbone VGG16 successfully?

Hi,bro!~
Have you implemented Faster R-CNN with backbone VGG16 successfully?

Hi,bro!~ Have you implemented Faster R-CNN with backbone VGG16 successfully?

I build the backbone based on mmcv.cnn.VGG

Hi,bro!~ Have you implemented Faster R-CNN with backbone VGG16 successfully?

Hi,bro!~ Have you implemented Faster R-CNN with backbone VGG16 successfully?

Hi, could you provide a code of VGG16 for reference?

https://github.com/tianxianhao/FasterRCNN_VGG16_DC5.git

Hi,bro!~ Have you implemented Faster R-CNN with backbone VGG16 successfully?

Hi,bro!~ Have you implemented Faster R-CNN with backbone VGG16 successfully?

Hi, could you provide a code of VGG16 for reference?

https://github.com/tianxianhao/FasterRCNN_VGG16_DC5.git

Hi,bro!~ Have you implemented Faster R-CNN with backbone VGG16 successfully?

Hi,bro!~ Have you implemented Faster R-CNN with backbone VGG16 successfully?

Hi, could you provide a code of VGG16 for reference?

https://github.com/tianxianhao/FasterRCNN_VGG16_DC5.git

Thanks for your help.


tooltips have disapearred

Hi,
Yesterday I started to use meteor-kitchen and there was tooltips in the GUI editor. Today they have disapeared.
I see that last commit on this repo was on april so I don't know if it is related to this repo ?

@prisonier2 thanks.
That's mistake made by me yesterday when I published latest version. Will fix it later today.

Ok thanks Petar.
Your tool is A W E S O M E !

@prisonier2 thank you! BTW, new better gui is coming soon, stay tuned!

I guess this is why the sitemap looks broken ?

@prisonier2 sitemap is broken for other reason, but that will not be fixed - moving to new GUI in ~1 weeks (I hope).

@prisonier2 fixed.

@perak  Thanks ;)


Change number flag, Mason County WA

The number flag currently uses the ADDRESS field, which contains the full street address, rather than the STREETNUMB field, which contains only the address number.
cc @ingalls

Could you add STRNUMSUFF as the unit field in the conform?


Machine Learning on AWS

A small group of us are working on a short guide to ML on AWS via a shared Quip document.  If you have domain expertise and would like to join LMK

I would like to help. How can I start?

@jlevy would it be possible for you to invite @niranjv to our shared Quip document?  -- thanks

@lynnlangit sorry for delay; looks like it's your doc and you've disabled sharing on the doc — can you enable link sharing? (Quip is a bit confusing on this. :( )

@niranjv in any case can you ping me (jlevy) in the Slack group and we'll sort it out? There are a few areas for ML coverage where your help would be super appreciated.

@lynnlangit sorry for delay but  https://quip.com/hmAKA6bCQPy0

@jlevy @lynnlangit Thanks for granting me access to the doc. Now how do I get started? Is there any specific item I can work on?

I've been framing cloud-based ML in terms of two offer types - serverless APIs (i.e. Vision, Speech, etc...) and custom model hosting (both regular and dnns, such as MxNet, Tensorflow etc...).  I would start by reframing the existing document this way.  Also I think comparisons to other cloud vendors, particularly GCP ML offerings is important.


Fix reward bug in the CleanRL example

Feel free to merge :) Thanks.

@vwxyzjn I am wondering why this change is not part of envpool_ppo_atari.py in the CleanRL repo? Is this bug specific to 0.26+ code (since the example in this repo is able to handle both legacy_gym and 0.26+, but the example in cleanRL repo can only handle less than 0.26)

@rfali  this change was part of https://github.com/vwxyzjn/cleanrl/blob/f62ad1f942890db7b752f4ad79b65be53917d3c2/cleanrl/ppo_atari_envpool.py#L99.
Ehh maybe we are talking about different things but basically you could have lives==0 but the episode has not terminated yet. This is the reason I used the info["termintaed"]

@vwxyzjn @Trinkle23897 I am also curious to know why a RecordEpisodeStatistics wrapper for envpool has
self.episode_returns += infos["reward"]
whereas gym, gymnasium's RecordEpisodeStatistics and SB3's VecMonitor use
self.episode_returns += reward
in their step() to calculate rewards.

@vwxyzjn @Trinkle23897 I am also curious to know why a RecordEpisodeStatistics wrapper for envpool has
self.episode_returns += infos["reward"]
whereas gym, gymnasium's RecordEpisodeStatistics and SB3's VecMonitor use
self.episode_returns += reward
in their step() to calculate rewards, especially since this PR mentions this is a reward bug.

It doesn’t look right. rewards is the clipped reward.

@vwxyzjn @Trinkle23897 I am also curious to know why a RecordEpisodeStatistics wrapper for envpool has
self.episode_returns += infos["reward"]
whereas gym, gymnasium's RecordEpisodeStatistics and SB3's VecMonitor use
self.episode_returns += reward
in their step() to calculate rewards, especially since this PR mentions this is a reward bug.

It doesn’t look right. rewards is the clipped reward.

hey @vwxyzjn , can you please specify which one is correct for envpool usage (with Atari and for other envs like Procgen). the first one is used in envpool (and we used it and saw drastically lower scores in ALE) whereas the second one is the original wrapper from gym (see here) and for gymnasium (see here)

self.episode_returns += infos["reward"]
self.episode_returns +=reward

When you said

It doesn’t look right. rewards is the clipped reward.

I thought you were saying that using reward is not right (as it is clipped reward) and hence I also stuck with it when using envpool, but I think this (option 1) maybe is incorrect.
What do envpool authors say about this? @Trinkle23897 @vwxyzjn


grunt.template not working

I am trying to specify paths for postcss-assets with my variables but with no luck:
…
require('postcss-assets')({
	loadPaths:['<%=yeogurt.client%>/images/'],
	basePath:'<%=yeogurt.client%>/',
	cachebuster:true
})
…

Warning: /Users/grawl/sites/***/dist/style.css:372:5: Asset not found or unreadable: inline/icon-key-minor.png
Load paths:
  /Users/grawl/sites/***/<%=yeogurt.client%>
  /Users/grawl/sites/***/<%=yeogurt.client%>/<%=yeogurt.client%>/images Use --force to continue.

Yep, Grunt doesn't process the options you pass to the PostCSS plugin as a function argument. Not much we can do. =(


groups: Received undefined is not an object (evaluating 'e.isNew') while trying to change the name of a section

This issue is what it says on the tin — I was attempting to edit a channel section's name in a group I am the host of:
wgt@https://fabled-faster.arvo.network/apps/groups/assets/index.d96bf496.js:1137:42341
qH@https://fabled-faster.arvo.network/apps/groups/assets/index.d96bf496.js:93:53298
RG@https://fabled-faster.arvo.network/apps/groups/assets/index.d96bf496.js:93:62835
Oie@https://fabled-faster.arvo.network/apps/groups/assets/index.d96bf496.js:97:967
M6e@https://fabled-faster.arvo.network/apps/groups/assets/index.d96bf496.js:97:890
Aw@https://fabled-faster.arvo.network/apps/groups/assets/index.d96bf496.js:97:740
UG@https://fabled-faster.arvo.network/apps/groups/assets/index.d96bf496.js:95:10949
UG@[native code]
@https://fabled-faster.arvo.network/apps/groups/assets/index.d96bf496.js:93:39073
@https://fabled-faster.arvo.network/apps/groups/assets/index.d96bf496.js:82:3901
lie@https://fabled-faster.arvo.network/apps/groups/assets/index.d96bf496.js:93:39020
vh@https://fabled-faster.arvo.network/apps/groups/assets/index.d96bf496.js:93:38954
Xd@https://fabled-faster.arvo.network/apps/groups/assets/index.d96bf496.js:95:8497
nR@https://fabled-faster.arvo.network/apps/groups/assets/index.d96bf496.js:93:58628
nR@[native code]
@https://fabled-faster.arvo.network/apps/groups/assets/index.d96bf496.js:1137:44664
@https://fabled-faster.arvo.network/apps/groups/assets/index.d96bf496.js:1137:41510
asyncFunctionResume@[native code]
@[native code]
promiseReactionJobWithoutPromise@[native code]
promiseReactionJob@[native code]

@urcades Was unable to repro this locally. I tried creating a group, created a couple sections, edited the titles, moved a channel back and forth, and edited again. Everything worked as expected. Any other tips on how to reproduce?

Unfortunately none to my knowledge — fwiw, I have not encountered this issue since, and it may have somehow been fixed with the update we recently shipped!


Slow loading time for navbar links: search, events, and map

Description
When clicking on the search, events, or map links in the navbar, there is a delay of approximately 1.5 or less seconds before the page loads. This issue does not occur when accessing the docs or playground links, which load quickly and smoothly.
Steps to Reproduce:
Navigate to the website
Click on the search, events, or map links in the navbar
Observe the delay in page loading time
Expected behavior: All navbar links should load quickly and without delay.
Screenshots
https://user-images.githubusercontent.com/129386460/235110616-050f38e2-9038-41d6-aeb6-ab06aeb1c59a.mp4
Additional information
No response

Related issue #6376  - perhaps a performance epic is needed?

Yes this pages are slow because they load all the data before displaying the page. As we move to the database usage rather the json files (from the app perspective) these page load times will improve https://github.com/EddieHubCommunity/LinkFree/issues/6244
I suggest we close this for now and revisit in the near future after the refactor


Harbor Integration

Checklist

[ ] Affected Issues have been mentioned in the Closing issues section
[ ] Documentation has been written/updated.
[x] Changelog entry has been written

This PR enables the use of Harbor as a secure docker registry within Lagoon, complete with vulnerability scanning provided by Clair.
Changelog Entry
Feature - Integrate Harbor/Clair into Lagoon
Closing issues

[ ] remove chartmuseum from core config, and also all containers, services, etc
[ ] rename EXT_ENDPOINT to HARBOR_NGINX_ENDPOINT
[ ] update startscripts to check LAGOON_ROUTES for checking the harbor nginx endpoint, see https://github.com/amazeeio/lagoon/blob/master/services/logs-db-ui/entrypoints/80-keycloak-url.bash

All the code review tasks from above have been completed. I now need to revert the change which disables PVC persistence, and this will be ready for the integration phase.


[stable/Kube-ops-view] Add securityContext

/assign @hjacobs
What this PR does / why we need it:
This PR is to give the opportunity to put a securityContext to let the pod run into a K8S with PodSecurityPolicy who only allow RunAsNonRoot user
in value.yaml file user can add:
securityContext:
runAsUser: 1000
To be compliant this kind of psp
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[X] DCO signed
[X] Chart Version bumped
[X] Variables are documented in the README.md

/assign @hjacobs

/ok-to-test
/assign
/lgtm

Thanks!


Adding comments from meeting to organizing inner topics

beyond just math, want to match w/e domain naming conventions (e.g., if doing SIR model, name the compartments and rate parameters according to that convention)


Remove links to qiskit.org from API docs

We have a couple of these still, such as in the migration guides to tutorials on qiskit.org. Search for qiskit.org.
Some of these require fixing the original API docs.
Also deal with qisk.it links.
--
Update Nov 21, post https://github.com/Qiskit/documentation/pull/381, we only need to update API docs. Tracked by https://github.com/Qiskit/qiskit/issues/11289 and https://github.com/Qiskit/qiskit-ibm-runtime/issues/1235

Done. Thanks @javabster for fixing this repo and @gluonhiggs for fixing Qiskit!


Fix issue with historgram statistic set being converted by duration

removed covert by duration and added test for historgram min,max and sum reporting in statistic set

Thank you! Left some comments

Addressed the comments and added another test for a sliding window histogram to sanity check that subsequent report methods correctly update the min/max etc

Thank you for your input with this!
On Tue, Aug 15, 2017 at 3:07 AM, williedoran<EMAIL_ADDRESS>wrote:

Addressed the comments and added another test for a sliding window
histogram to sanity check that subsequent report methods correctly update
the min/max etc
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/azagniotov/codahale-aggregated-metrics-cloudwatch-reporter/pull/6#issuecomment-322430106,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA8azKjDPo0S3JAIfKrIOI2dH3GK-eU5ks5sYW3ggaJpZM4O2taK
.


npm start issues

I have set up the whole thing up to when you type npm start but when I do I get these errors:

<EMAIL_ADDRESS>start /home/pi/alexavoiceserviceexamples/samples/companionService
node ./bin/www

This node service needs to be running to store token information memory and vend them for the AVS app.
fs.js:584
  return binding.open(pathModule._makeLong(path), stringToFlags(flags), mode);
                 ^

Error: ENOENT: no such file or directory, open '/home/pi/alexvoiceserviceexamples/samples/javaclient/certs/server/node.key'
    at Error (native)
    at Object.fs.openSync (fs.js:584:18)
    at Object.fs.readFileSync (fs.js:431:33)
    at Object.<anonymous> (/home/pi/alexavoiceserviceexamples/samples/companionService/bin/www:21:13)
    at Module._compile (module.js:413:34)
    at Object.Module._extensions..js (module.js:422:10)
    at Module.load (module.js:357:32)
    at Function.Module._load (module.js:314:12)
    at Function.Module.runMain (module.js:447:10)
    at startup (node.js:142:18)

npm ERR! Linux 4.1.19-v7+
npm ERR! argv "/usr/bin/nodejs" "/usr/bin/npm" "start"
npm ERR! node v5.9.0
npm ERR! npm  v3.7.3
npm ERR! code ELIFECYCLE
npm ERR<EMAIL_ADDRESS>start: `node ./bin/www`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>start script 'node ./bin/www'.
npm ERR! Make sure you have the latest version of node.js and npm installed.
npm ERR! If you do, this is most likely a problem with the alexa-voice-service-sample-companion-service package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     node ./bin/www
npm ERR! You can get information on how to open an issue for this project with:
npm ERR!     npm bugs alexa-voice-service-sample-companion-service
npm ERR! Or if that isn't available, you can get their info via:
npm ERR!     npm owner ls alexa-voice-service-sample-companion-service
npm ERR! There is likely additional logging output above.

npm ERR! Please include the following file with any support request:
npm ERR!     /home/pi/alexavoiceserviceexamples/samples/companionService/npm-debug.log

My debug log is:
0 info it worked if it ends with ok
1 verbose cli [ '/usr/bin/nodejs', '/usr/bin/npm', 'start' ]
2 info using<EMAIL_ADDRESS>3 info using<EMAIL_ADDRESS>4 verbose run-script [ 'prestart', 'start', 'poststart' ]
5 info lifecycle<EMAIL_ADDRESS><EMAIL_ADDRESS>6 silly lifecycle<EMAIL_ADDRESS>no script for prestart, continuing
7 info lifecycle<EMAIL_ADDRESS><EMAIL_ADDRESS>8 verbose lifecycle<EMAIL_ADDRESS>unsafe-perm in lifecycle true
9 verbose lifecycle<EMAIL_ADDRESS>PATH: /usr/lib/node_modules/npm/bin/node-gyp-bin:/home/pi/alexavoiceserviceexamples/samples/companionService/node_modules/.bin:/usr/bin:/usr/local/sbin:/usr/l$
10 verbose lifecycle<EMAIL_ADDRESS>CWD: /home/pi/alexavoiceserviceexamples/samples/companionService
11 silly lifecycle<EMAIL_ADDRESS>Args: [ '-c', 'node ./bin/www' ]
12 silly lifecycle<EMAIL_ADDRESS>Returned: code: 1  signal: null
13 info lifecycle<EMAIL_ADDRESS>Failed to exec start script
14 verbose stack Error<EMAIL_ADDRESS>start: `node ./bin/www`
14 verbose stack Exit status 1
14 verbose stack     at EventEmitter.<anonymous> (/usr/lib/node_modules/npm/lib/utils/lifecycle.js:239:16)
14 verbose stack     at emitTwo (events.js:100:13)
14 verbose stack     at EventEmitter.emit (events.js:185:7)
14 verbose stack     at ChildProcess.<anonymous> (/usr/lib/node_modules/npm/lib/utils/spawn.js:24:14)
14 verbose stack     at emitTwo (events.js:100:13)
14 verbose stack     at ChildProcess.emit (events.js:185:7)
14 verbose stack     at maybeClose (internal/child_process.js:850:16)
14 verbose stack     at Process.ChildProcess._handle.onexit (internal/child_process.js:215:5)
15 verbose pkgid<EMAIL_ADDRESS>16 verbose cwd /home/pi/alexavoiceserviceexamples/samples/companionService
17 error Linux 4.1.19-v7+
18 error argv "/usr/bin/nodejs" "/usr/bin/npm" "start"
19 error node v5.9.0
20 error npm  v3.7.3
21 error code ELIFECYCLE
22 error<EMAIL_ADDRESS>start: `node ./bin/www`
22 error Exit status 1
23 error Failed at the<EMAIL_ADDRESS>start script 'node ./bin/www'.
23 error Make sure you have the latest version of node.js and npm installed.
23 error If you do, this is most likely a problem with the alexa-voice-service-sample-companion-service package,
23 error not with npm itself.
23 error Tell the author that this fails on your system:
23 error     node ./bin/www
23 error You can get information on how to open an issue for this project with:
23 error     npm bugs alexa-voice-service-sample-companion-service
23 error Or if that isn't available, you can get their info via:
23 error     npm owner ls alexa-voice-service-sample-companion-service
23 error There is likely additional logging output above.
24 verbose exit [ 1, true ]

Hope someone can help!

Hi! Looks like this error is from something you have modified from the base generated template. Can you paste your entire bin/www file here so we can see what the contents are? It looks like from the error you are trying to read the file /home/pi/alexvoiceserviceexamples/samples/javaclient/certs/server/node.key from line 21, but that files does not exist.

I don't have a www file? :P

It's in the bin folder. The full path is also in your error: /home/pi/alexavoiceserviceexamples/samples/companionService/bin/www

Oh sorry thought you meant from root directory.
#!/usr/bin/env node
console.log('This node service needs to be running to store token information memory and vend them for the AVS app.\n');
/**

Module dependencies.
*/
var app = require('../app');
var debug = require('debug')('companion:server');
var https = require('https');
var fs = require('fs');
var config = require("../config");

/**

Get port from environment and store in Express.
*/
var port = normalizePort(process.env.PORT || '3000');
app.set('port', port);

var options = {
key: fs.readFileSync(config.sslKey),
cert: fs.readFileSync(config.sslCert),
ca: fs.readFileSync(config.sslCaCert),
requestCert: true,
rejectUnauthorized: false,
};
/**

Create HTTP server.
*/
var server = https.createServer(options, app);

/**

Listen on provided port, on all network interfaces.
*/
server.listen(port);
server.on('error', onError);
server.on('listening', onListening);

/**

Normalize a port into a number, string, or false.
*/
function normalizePort(val) {
var port = parseInt(val, 10);

if (isNaN(port)) {
// named pipe
return val;
}
if (port >= 0) {
// port number
return port;
}
return false;
}

Thanks! Yea, it looks like that file was modified from what we generated. I don't have much more information about your error more than I said in https://github.com/expressjs/generator/issues/106#issuecomment-203029835 You just need to create the files for SSL that your code needs and you should be all-set.
If you still are confused or need help, please drop by our support channel: https://gitter.im/expressjs/express :)

Wait sorry about this. Thought I was posting an issue on https://github.com/amzn/alexa-avs-raspberry-pi/issues ??!?


Set memcache_up to 0 on failure to scrape

This commit sets memcache_up to 0 instead of just omitting the metric.

:+1:

Good catch!

:+1: Nice, yea, I had a feeling that was where the bug was.


Test PR Branch1

PR test for SCA

⛔ Snyk has found 4 issues.

Icon
Severity
Issues

Critical
0

High
4

Medium
0

Low
0

⛔ code/snyk check is completed. Snyk found 4 issues. (View Details)


FMD2 Crashes - Access Violation

The Wine / FMD2 exe crashes with an Access Violation. Im using kasm-v3.
It sometimes also just stops working, without any crash, like it just freezes and then after a restart shows that error.

More Details:
OS: Ubuntu 22.04
Ram: 8GB
CPU: 4xXeon
Similar DL Tasks: 2
Downloads per Task: 30

maybe this is something due to the script changes from @Jerrk ?

Something new i just observed, when i keep the KasmVNC window in focus this doesnt occur.
This only happens when i close the window or unfocus it.
Also if i lower my Downloads per Task to 20, it doesnt seem to occur anymore (atleast atm).

yeah ok 20 seems to be more stable, but it still freezes but it catches itself.

This might be an cpu power issue as with a stronger vps thos doesnt happen anymore

Nevermind it just took longer but it still crashed

I also upgraded my vps and this is still happening

I also noticed that whenever fmd2 freezes in Kasm, the actual docker container is still running and eating my cpu & ram.

So after the last freeze / crash, now after a restart im stuck on this instead of fmd2 loading and showing me an error...

@Banh-Canh


"output" does not show any map

I have installed IBM SPSS Modeler 18.2.1 on my Mac machine (mac 10.14.6) , R essential for SPSS Modeler 18.2.1, R 3.6.2. installed the extension from the extension hub. In the output I have node "googleMaps" and Heatmaps" node which I am using the modeler stream file. when I execute these nodes the "output" does not show any map - it is blank. Any idea why it isn't working? This was working on my Windows Machine. Any known issue?

any luck with solving the issue?


Build pipeline

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.:x: paulsoumi96 sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.


Question on: Reference database

Can I use MiDAS database for the taxonomic classification? If so, could you direct me to the right reference files that I should be using with the MiSeq SOP pipeline?

Sorry, but I'm not sure what the MIDAS database is. I'm sure you can use
it, it just might be a matter of you getting it into the right format
On Tue, Sep 20, 2022 at 2:23 PM mdmahbub118 @.***>
wrote:

Can I use MiDAS database for the taxonomic classification? If so, could
you direct me to the right reference files that I should be using with the
MiSeq SOP pipeline?
—
Reply to this email directly, view it on GitHub
https://github.com/mothur/mothur.github.io/issues/94, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAJUUBHADGBNCTL22RIV7R3V7H6IFANCNFSM6AAAAAAQRKV32E
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>


Dutch & Belgian public tv now fail

Please follow the guide below

You will be asked some questions and requested to provide some information, please read them carefully and answer honestly
Put an x into all the boxes [ ] relevant to your issue (like this: [x])
Use the Preview tab to see what your issue will actually look like

Make sure you are using the latest version: run youtube-dl --version and ensure your version is 2018.10.05. If it's not, read this FAQ entry and update. Issues with outdated version will be rejected.

[x] I've verified and I assure that I'm running youtube-dl 2018.10.05

Before submitting an issue make sure you have:

[x] At least skimmed through the README, most notably the FAQ and BUGS sections
[x] Searched the bugtracker for similar issues including closed ones
[x] Checked that provided video/audio/playlist URLs (if any) are alive and playable in a browser

What is the purpose of your issue?

[x] Bug report (encountered problems with youtube-dl)
[ ] Site support request (request for adding support for a new site)
[ ] Feature request (request for a new functionality)
[ ] Question
[ ] Other

The following sections concretize particular purposed issues, you can erase any section (the contents between triple ---) not applicable to your issue

If the purpose of this issue is a bug report, site support request or you are not completely sure provide the full verbose output as follows:
Add the -v flag to your command line you run youtube-dl with (youtube-dl -v <your command line>), copy the whole output and insert it here. It should look similar to one below (replace it with your log inserted between triple ```):
[debug] System config: []
[debug] User config: []
[debug] Command-line args: [u'-v', u'http://www.youtube.com/watch?v=BaW_jenozKcj']
[debug] Encodings: locale cp1251, fs mbcs, out cp866, pref cp1251
[debug] youtube-dl version 2018.10.05
[debug] Python version 2.7.11 - Windows-2003Server-5.2.3790-SP2
[debug] exe versions: ffmpeg N-75573-g1d0487f, ffprobe N-75573-g1d0487f, rtmpdump 2.4
[debug] Proxy map: {}
...
<end of log>

If the purpose of this issue is a site support request please provide all kinds of example URLs support for which should be included (replace following example URLs by yours):

Single video: https://www.youtube.com/watch?v=BaW_jenozKc
Single video: https://youtu.be/BaW_jenozKc
Playlist: https://www.youtube.com/playlist?list=PL4lCao7KL_QFVb7Iudeipvc2BCavECqzc

Note that youtube-dl does not support sites dedicated to copyright infringement. In order for site support request to be accepted all provided example URLs should not violate any copyrights.

Description of your issue, suggested solution and other information
Explanation of your issue in arbitrary form goes here. Please make sure the description is worded well enough to be understood. Provide as much context and examples as possible.
If work on your issue requires account credentials please provide them or explain how one can obtain them.

[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: [u'https://www.vrt.be/vrtnu/a-z/bordertown/1/bordertown-s1a2-a-doll-s-house-part-2/', u'--username', u'PRIVATE', u'--password', u'PRIVATE', u'--verbose']
[debug] Encodings: locale UTF-8, fs UTF-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2018.10.05
[debug] Python version 2.7.15rc1 (CPython) - Linux-4.15.0-36-generic-x86_64-with-Ubuntu-18.04-bionic
[debug] exe versions: ffmpeg 3.4.4-0ubuntu<IP_ADDRESS>, ffprobe 3.4.4-0ubuntu<IP_ADDRESS>
[debug] Proxy map: {}
[VrtNU] Logging in
[VrtNU] Requesting a token
[VrtNU] bordertown-s1a2-a-doll-s-house-part-2: Downloading webpage
WARNING: [VrtNU] bordertown-s1a2-a-doll-s-house-part-2: Failed to parse JSON Invalid \escape: line 6 column 18 (char 104)
[VrtNU] bordertown-s1a2-a-doll-s-house-part-2: Downloading JSON metadata
[Canvas] pbs-pub-926367f3-fb54-4062-97f6-a80239d1ec4c$vid-6d60a446-196d-47f3-875e-8fdbd3c1b092: Downloading JSON metadata
[Canvas] pbs-pub-926367f3-fb54-4062-97f6-a80239d1ec4c$vid-6d60a446-196d-47f3-875e-8fdbd3c1b092: Downloading m3u8 information
WARNING: Failed to download m3u8 information: HTTP Error 403: Forbidden
[Canvas] pbs-pub-926367f3-fb54-4062-97f6-a80239d1ec4c$vid-6d60a446-196d-47f3-875e-8fdbd3c1b092: Downloading MPD manifest
WARNING: Failed to download MPD manifest: HTTP Error 403: Forbidden
[Canvas] pbs-pub-926367f3-fb54-4062-97f6-a80239d1ec4c$vid-6d60a446-196d-47f3-875e-8fdbd3c1b092: Downloading ISM manifest
WARNING: Failed to download ISM manifest: HTTP Error 403: Forbidden
[Canvas] pbs-pub-926367f3-fb54-4062-97f6-a80239d1ec4c$vid-6d60a446-196d-47f3-875e-8fdbd3c1b092: Downloading f4m manifest
WARNING: Unable to download f4m manifest: HTTP Error 403: Forbidden
ERROR: No video formats found; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
Traceback (most recent call last):
  File "/usr/local/bin/youtube-dl/youtube_dl/YoutubeDL.py", line 792, in extract_info
    ie_result = ie.extract(url)
  File "/usr/local/bin/youtube-dl/youtube_dl/extractor/common.py", line 507, in extract
    ie_result = self._real_extract(url)
  File "/usr/local/bin/youtube-dl/youtube_dl/extractor/canvas.py", line 73, in _real_extract
    self._sort_formats(formats)
  File "/usr/local/bin/youtube-dl/youtube_dl/extractor/common.py", line 1286, in _sort_formats
    raise ExtractorError('No video formats found')
ExtractorError: No video formats found; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.


feature: support up/down insert of node drag

supports the insertion function up/down the target node when the node is dragged

Thank you for your contributions!


Can cpu run this code?

Can cpu run this code?
and
How to run ？

Yes, you can run the code on CPU by removing the gpu arg:
python exp.py --dir_data ../data/devdata
The training and inference can be slow on CPU though. Please have a try and let us know if you might come across any problems!

when I run the code, I come across this problems:
File "exp.py", line 28
self.exp_name = f'{consts.DIR_DATA.stem}-{consts.LM_NAME_SUFFIX}-{self.path_model_config.stem}'
^
SyntaxError: invalid syntax
could you please help me to fix the code and run it correctly?   Thx

Hi, which version of Python are you using? f-string is not supported in some previous versions, so please use python 3.6+ to see if the error goes away.

Thx a lot!
I use python3.8.5  successfully solved this problem, but when I run the code by cpu, it come across this problem:
Traceback (most recent call last):
File "exp.py", line 11, in
from preprocess import Preprocessor
File "/data/WORKSPACE/UCPhrase-exp-master/src/preprocess/init.py", line 3, in
from preprocess.annotator_wiki import WikiAnnotator
File "/data/WORKSPACE/UCPhrase-exp-master/src/preprocess/annotator_wiki.py", line 5, in
from match import KeywordProcessor
File "/data/WORKSPACE/UCPhrase-exp-master/src/match/init.py", line 1, in
from match.keywordprocessor import KeywordProcessor
ModuleNotFoundError: No module named 'match.keywordprocessor'
If you suspect this is an IPython 7.23.0 bug, please report it at:
https://github.com/ipython/ipython/issues
or send an email to the mailing list at<EMAIL_ADDRESS>You can print a more detailed traceback right now with "%tb", or use "%debug"
to interactively debug it.
Extra-detailed tracebacks for bug-reporting purposes can be enabled via:
%config Application.verbose_crash=True

Hi, did you successfully run the command bash build.sh? Ideally there should be a "src/match/keywordprocessor.so" file after compilation. Could you please confirm that?

@xgeric can we build keywordprocessor on windows?

Hi, we did not test the code on Windows, and we recommend to run the code on Linux.


Not passing new command to modules + tests

Fixes #43

Current coverage is 100% (diff: 100%)

Merging #44 into master will not change coverage

@@           master   #44   diff @@
===================================
  Files           3     3
  Lines         116   119     +3
  Methods         4     4
  Messages        0     0
  Branches       17    18     +1
===================================
+ Hits          116   119     +3
  Misses          0     0
  Partials        0     0

Powered by Codecov. Last update 0104dae...cfab4f1


"error":{"code":"504","message":"An error occurred with your deployment"}} #1033

"error":{"code":"504","message":"An error occurred with your deployment"}} #1033
even tried

but its showing this.
any idea how to fix. also how can i base url communication site documentation i know the drive id.

nvm saw the deployment section will check out now.

getting this error


Multiple marker selection via BoundsDriftMarker and SemanticMarkers

Note: targets https://github.com/VEuPathDB/web-monorepo/pull/534
Hi @moontrip and @dmfalke
I wanted to try my suggestion in https://github.com/VEuPathDB/web-monorepo/pull/513#issuecomment-1747598022 and I have found that it works pretty well and has removed quite a lot of code. I've also made it more 'reacty' and got rid of one jquery-like bit.
I have only sorted out the @veupathdb/components side of things - and fixed all the stories (hopefully reverting the old marker stories back to what they were before, so only the MarkerSelection story has selectedMarker logic).
I did introduce a useEffect to do pruning on selectedMarkers but I'd say this was OK. If you pan a selected marker off-screen, you would expect to trigger a side-effect maybe? Nothing is triggered if nothing needs pruning.
I've changed the spec of selectedMarkers back to string[] as per the original spec.
Some things to do

[x] test chart markers (oops I forgot)
[x] wire back into SAM and store the selectedMarkers in appState/markerConfiguration? - we could discuss with @dmfalke the best approach here (to avoid triplicate code in each MapType)
[x] if possible fix the highlighting behaviour so that after the click, the highlighted marker is not dimmed - or... add a highlight around the popup (I think I prefer the latter) --> new standalone ticket https://github.com/VEuPathDB/web-monorepo/issues/617
[ ] disable multiple-marker in SAM via commenting or other easily reversible approach --> new ticket for wiring https://github.com/VEuPathDB/web-monorepo/issues/618

If I understand your speaking correctly, it is not the issue of marker click, but it is due to the mouse-over event for both popup and rectangular gray box. I didn't look into this carefully, but I guess that there might be a CSS effect to make marker dim for the mouse-over marker.

Agreed, it's pre-existing functionality that dims the marker so that the bounding box grey rectangle is more visible while the popup is shown. I think a yellow border around the popup would solve this for me.  At the moment the only feedback I get from highlighting is a dull yellow. More like "lowlighting" lol...

I'll test the speed/responsiveness issue later. I wonder if we should have a snackbar that says "[Some of] your selected markers have been deselected because they have gone off-screen" or "Your marker selection has been cancelled because the marker aggregation level has changed due to zooming" (or similar!)

If I understand your speaking correctly, it is not the issue of marker click, but it is due to the mouse-over event for both popup and rectangular gray box. I didn't look into this carefully, but I guess that there might be a CSS effect to make marker dim for the mouse-over marker.

Agreed, it's pre-existing functionality that dims the marker so that the bounding box grey rectangle is more visible while the popup is shown. I think a yellow border around the popup would solve this for me. At the moment the only feedback I get from highlighting is a dull yellow. More like "lowlighting" lol...

@bobular I cannot think a quick solution off the top of my head for highlighting popup: it may be a bit tricky. Pop-up only shows when doing mouse-over a marker, so it may not be that visible in a general case.
And, dull yellow! 😄 Perhaps other color may also work, like (dull) orange 😄

By the way, I noticed a minor(?) disadvantage on your approach using useEffect. Since selectedMarkers are updated via useEffect, there is some delay to update selectedMarkers array. As a side effect, when marker drift happens (aggregation/deaggregation, i.e., when geohash level is changed), highlights remain for a while, and then disappear: this is more evident when the number of markers are large (e.g., chart marker at marker selection storybook). But it may just be okay.

I tested with the chart marker story too and I think it's OK - it only happens when aggregating on zooming out. It could even be seen as a feature? It shows you where your selected markers were for about 1 second. Then if a snackbar came up to say what had happened, I think that would be great. But Dave usually has other opinions on snackbars!
Tested also with donuts in SAM - aggregation behaviour is the same as in the story (though slower due to real-life data requests) - I think it's fine.
What does need addressing is the loss of marker double-click (should zoom-to-marker-bounds). Could you see if we can keep both click behaviours please?

I'll test the speed/responsiveness issue later. I wonder if we should have a snackbar, triggered by the effect, that says "[Some of] your selected markers have been deselected because they have gone off-screen" or "Your marker selection has been cancelled because the marker aggregation level has changed due to zooming" (or similar!)

@bobular I did some test with SAM. It is more noticeable when zooming-out (small video is attached). I wonder how it will work with larger number of markers like mega study etc. but we may live with that anyway? :) Live site will be faster than my VM, so that delay may not be that noticeable in there.
https://github.com/VEuPathDB/web-monorepo/assets/12802305/a88aaa49-acc3-4341-bef4-1413137bb83a

Thanks for the video - I've been recording one too.
I think what you demonstrate is absolutely fine. While I was recording I couldn't always replicate it. Sometimes the aggregation animation seemed broken (while the disaggregation was still animating fine). It may be related to https://github.com/VEuPathDB/web-monorepo/issues/575
Anyway, here's my video, highlighting (pun intended) another small issue, and discussing the dim yellow issue too.
https://github.com/VEuPathDB/web-monorepo/assets/308639/893d5844-2001-4f1e-b4bc-4f59f5a545d4

@bobular Just for a note. I think I did comment somewhere that I will look into the popup location bug(?) but can't find it. Anyway. I have been trying to resolve that bug.

Go ahead and merge this to the parent branch (and move the to-do list over to that PR), or re-target this PR to main.  Thanks!

After merging #610 into this, please go ahead and merge this to the parent branch (and move the to-do list over to that PR), or re-target this PR to main and close #534. Thanks!

@bobular thank you for your suggestion! I think that it is better to keep this PR as a candidate to be merged into main so that we can easily keep track of our progress/comments. I will close the parent PR and leave this to be merged into main 👍

@bobular FYI, I found a solution to enable marker-double click again at https://github.com/VEuPathDB/web-monorepo/pull/615. It would be great if you can check it out when you are available.

I've checked off the remaining items/made new tickets.
We could merge this when it is disabled (see new checkbox item in description).

I think we're in a relatively good place here, but we should not merge this before b66, which gives us time to address the remaining minor issues (minor to the user - major for us, probably!)


Create release hooks

The following hooks shall be created on creating new release:

Generate website and save raw code in /docs (delete and regenerate content).
The .nojekyll file needs to be created to the /docs folder.

Replace with #21


⚠ Roblox TextFilter Api has degraded performance

In 692506e, Roblox TextFilter Api (https://textfilter.roblox.com) experienced degraded performance:

HTTP code: 200
Response time: 4860 ms

Resolved: Roblox TextFilter Api performance has improved in b10e368.


update issue/pr templates

instead of basic MD templates, one can now have nice YML based templates that look nicer and will result in better issues/PR descriptions.
see: https://github.com/bids-standard/bids-specification/pull/1191

only relevant for issue templates, which we don't have.