Add converter for dashboard JSON in "export format" representation

Hi there,
@jeremybz / @jeremybanzhaf asked at ^1:

My goal is to use the API to get the JSON representation of a dashboard, including __inputs and named datasources.

@jangaraj answered:

HTTP API for dashboard export for external use doesn’t exist. This export feature is implemented in the UI (frontend). I would say the only option is to use standard dashboard API and make that output transformed into “dashboard format for external use”. Unfortunately, that format is not documented, so you have to use reverse engineering.

Torkel Ödegaard said at ^2:

This is not possible, the data is captured by the frontend so you would have to replicate what all the panel & data sources do (issue metric queries etc).

In order to implement this transformation, I would be happy to provide a place within this package, if possible. Would that make any sense / be helpful in any way?
The implementation should adhere to the specifications defined by the corresponding TypeScript code.

DashboardModel::getSaveModelClone()
DashboardModel::getPanelSaveModels()
DashboardExporter.ts
DashboardExporter.test.ts

With kind regards,
Andreas.
/cc @jangaraj

I agree. That DashboardExporter.ts needs a corresponding Python implementation.

Hi,
grafana-snapshot-tool as mentionned in its documentation, can import, export, and generate what is called in GRAFANA UI "snapshots": they are dashboards with corresponding datas incorporated.
My opinion is that snapshots, can't be part of the API because they are composed of calls to others parts of the api:

exporting a dashboard
collection datasources names
parsing the datasources used by the dashboard
query datasources from grafana
incorporating results into the dashboard
then generating a file or importing it to a grafana server.

A snapshot is a picture of a dashboard too, as a consequence it has parameters:

a time range (timefrom, time to)
value for each variables used by the dashboard.

Grafana-snapshot-tool is my implementation of these functionnalities.
It uses the python api (panadata/grafana-client), to perform all above actions.
My contribution to this repository, concerns datasources, particullary  the ability to query them, as required and done by Grafana UI (reverse engineering on API call with browser dev tool).
Hope my opinion can help.
Anyway feel free to use the code or to incorpore it into contribs, if you think it should help :)

Dear Jean-Francois,
apologies for the late reply. I am very happy that we finally got closer in touch at https://github.com/peekjef72/grafana-snapshots-tool/issues/2#issuecomment-1251008404 ff. If I can find some time to work on another iteration of grafana-client, I will also be happy to share further thoughts about our topics of shared interest.
In the meanwhile, I will be looking forward to all contributions and improvements coming from your pen which may be needed as we go. Thank you very much already for making a start with https://github.com/panodata/grafana-client/compare/main...peekjef72:grafana-client:main.
With kind regards,
Andreas.

@amotl I don't know current state of this but with some reverse engineering,
I tried to solve my externally shareable dashboard export...  I think grafana client should provide some more generic implementation for shareable dashboards.
class Grafana:
    """A class to interact with Grafana's API for managing dashboards."""

    def __init__(self, host: str, api_key: str = None):
        """Initializes the Grafana instance.

        Args:
            host (str): The base URL of the Grafana instance.
            api_key (str, optional): The API key for authenticating requests.
        """
        self.host = host
        self.api_key = api_key

    @property
    def headers(self):
        """Returns the headers for API requests."""
        return {"Authorization": f"Bearer {self.api_key}", "Content-Type": "application/json"}

    @cached_property
    def datasources(self) -> dict | None:
        """Fetches and caches the available datasources from Grafana."""
        datasources = {}
        url = urljoin(self.host, "/api/datasources")
        response = requests.get(url, headers=self.headers)

        if response.status_code == 200:
            for ds in response.json():
                datasources[ds["uid"]] = {
                    "name": ds["name"].replace("-", "_").upper(),
                    "label": ds["name"],
                    "description": "",
                    "type": "datasource",
                    "pluginId": ds["type"],
                    "pluginName": ds["typeName"],
                }
            return datasources
        else:
            print(f"Failed to fetch datasources: {response.status_code} - {response.text}")
            return None

    def _external_sharable(self, dashboard: dict) -> dict:
        """Prepares a dashboard for external sharing by updating datasource references.

        Args:
            dashboard (dict): The dashboard JSON data.

        Returns:
            dict: The updated dashboard JSON data.
        """
        datasources = self.datasources
        applicable_ds = []

        def _search_replace_datasource(obj):
            if isinstance(obj, dict):
                for key, value in obj.items():
                    if key == "annotations":
                        continue
                    # Replace datasource UID with variable.
                    if key == "datasource" and isinstance(value, dict):
                        if uid := value.get("uid"):
                            if uid in datasources:
                                applicable_ds.append(uid)
                                ds_name = datasources[uid]["name"]
                                value["uid"] = f"${{{ds_name}}}"
                    # Replace sitreps API's UIL with Variable.
                    elif key == "url" and re.search(SITREPS_API_REGEX, value):
                        obj[key] = re.sub(SITREPS_API_REGEX, "${SITREPS_API}", value)
                    else:
                        _search_replace_datasource(value)
            elif isinstance(obj, list):
                for item in obj:
                    _search_replace_datasource(item)

        _search_replace_datasource(dashboard)
        inputs = [datasources.get(ds_uid) for ds_uid in set(applicable_ds)]

        # If sitreps apis used by dashobard then include input asking hostname for sitreps apis.
        if "APIS" in [ds["name"] for ds in inputs]:
            inputs.append(
                {
                    "name": "SITREPS_API",
                    "label": "sitreps_api",
                    "description": "Sitreps API endpoint.",
                    "type": "constant",
                }
            )

        dashboard["__inputs"] = inputs
        return dashboard

Dear @digitronik,
your contribution is well received. Currently, I am a bit short on time, but if you see a chance to slot your code into (on top of) GH-23 in one way or another, a corresponding patch will be even more welcome.
With kind regards,
Andreas.


Unreliable connection, scroll bar moving of own accord

Product and Version : vscode
OS Version : linux fedora 29 (on my end, mac on the other)
Live Share Extension Version: v0.3.1150
Sorry I can't be more exact, but my last session using this extension was very bad.
It seemed like the connection was lost, but no error given. The follow feature that worked to start with seemed to just stop working. Also the scrolling of the JavaScript code was jerking up and down on both ends.
We had to log in and out 3 times in an hour to continue.
I hope you can stop adding features and just make it reliable. I have been using it for about 6 weeks once a week, and it is getting less stable each time it seems.
When I am using it I am paying the other party, so I can't really use that time for debugging. I hope you can add to the changelog when you get more reliable connections. Cheers, it would be AMAZING if it were bug free :)

@David-Else Thanks for reporting this, and apologies to hear that you’ve had some poor experiences with the product lately.
I’d love to make sure we fully understand the issues you ran into, so we can investigate them properly:

Regarding following stopping work: were you trying to explicitly follow the other developer and it wasn’t working, or did you notice that you were following at one point, and then it stopped? I just ask since we automatically unfollow when you switch files, so I’m curious if that’s the behaviornyou saw, or if you hit a bug. You see can more info of how following behaves here, and I’d love to hear any feedback.

Regarding the file scrolling jerking up and down: Was the issue you encountered the same as this?

Regarding having to log in and out: did you need to do that because the connection was dropped and you needed to reconnect? Or were you trying to “reset” the session to hopefully correct the other issues you mentioned?

Thanks again for using the product, and helping us improve it!

@lostintangent Hi, thanks for the reply.

One of us would be following the other and then it would stop. I checked the docs and read it stopped following if:

You start editing the currently active file
You open a different file
You close the currently active file
This may have been the case, I can't remember. I will tell my friend about this and we will see next time.

It was something like this, but much less dramatic. The jerking was slower.

We had to log in and out as the connection was dropped.

 At one point that totally froze chrome and vs-code for me and I had to reset the whole thing. Maybe that was involved? We really need that feature though! :)

@David-Else Just to confirm, is this the extension you're using? We actually did work to integrate with that extension, so it should definitely work :)
Does it cause Chrome/VS Code to freeze every time you use it, or only sometimes? Also, do you recall when it caused the freeze? Was it when you clicked the Go Live button? After you starting sharing? Something else? We'll try to repro it on our end, but any details you could share would be greatly appreciated. Thanks!

@loskinski Yes, it is that extension. It had never frozen before, but did the last time I used it. The result was that it froze on Chromium Version 71.0.3578.98 (Developer Build) Fedora Project (64-bit) and in vs code you could not click the go live button to cancel.
It was not when I clicked the go live button that is froze, it just happened at some random time. Afraid I can't remember any more details. If it happens again I will return and add it to this thread.

Now you are out of beta I will try it again when I have an opportunity and see If I get more luck. Cheers.


Write-DbaDbTableData - Added ColumnMap like in Import-DbaCsv

Type of Change

[ ] Bug fix (non-breaking change, fixes # )
[x] New feature (non-breaking change, adds functionality, fixes #6683 )
[ ] Breaking change (effects multiple commands or functionality, fixes # )
[ ] Ran manual Pester test and has passed (`.\tests\manual.pester.ps1)
[ ] Adding code coverage to existing functionality
[ ] Pester test is included
[ ] If new file reference added for test, has is been added to github.com/sqlcollaborative/appveyor-lab ?
[ ] Nunit test is included
[ ] Documentation
[ ] Build system

Purpose
Have the same mapping feature like in Import-DbaCsv
Approach
Just copied the code, that was easy.
Commands to test
$server = Connect-DbaInstance -SqlInstance SRV1
$server.Invoke("CREATE TABLE tempdb.dbo.test (col1 INT, col2 VARCHAR(100))")
$data = Invoke-DbaQuery -SqlInstance $server -Query "SELECT 123 AS value1, 'Hello world' AS value2" -As DataSet
$data | Write-DbaDbTableData -SqlInstance $server -Table 'tempdb.dbo.test' -ColumnMap @{ value1 = 'col1' ; value2 = 'col2' }

Screenshots

I think the datatype is wrong: [hashtable[]]$ColumnMap should be [hashtable]$ColumnMap, because only one hashtable is allowed. Shall I add this fix in both Write-DbaDbTableData and Import-DbaCsv with this pull request or open a new issue?

Yeah, that makes sense @andreasjordan. You can fix in the other commands and add it to this PR.

Looks fab 🤩  Thank you!


Sieve Save button wrap

Hi,
When editing Sieve filters I noticed the Save button wraps to the next line.

This is not an issue when I select English as language.

It's not really a bug, so I'd suggest you open a request at https://rainloop.uservoice.com/ :)
Ralf.

Thanks!


auto upload to common repo

It would be cool to connect between versions of Julia and different systems (specific combinations of hardware (GPU, CPU, SSD, etc) and software (OS, locally compiled libs, etc)). To do that in a more scalable fashion (apart from making sure you harvest all the specs on the local system this runs on) you'll need to upload the results to some open database.
Then people would be able to see what specific system configurations are relatively suffering and maybe what could be done about it. If enough people submit, we would even be able to say how many people are affected by relatively poor performance, which would depend on how many people have that specific configuration. And if this gets really big then the testsuite might attract more and more package authors as a means for testing their code on tons and tons of very different configurations. They would then be able to use the feedback to possibly adjust their code to cater to more systems.
Of course, to make this as real as possible, it would be imperative to make it as easy as possible:
using SystemBenchmark
runsubmit() # runs the banchmark AND submits it online

Agreed. I couldn't figure out a non-vulnerable way to have an anonymous bucket, so for now I've set up #8 with a note in the readme. We can pull the data in as people submit it for now.
Please feel free to try automating it more!

Looks good enough for now. If this gets really big then we'll need to revisit this feature.

It'd be great to let people easily upload raw benchmark data. FWIW https://github.com/simeonschaub/ArtifactUtils.jl has upload_to_gist function to upload an artifact to gist using git and gh which could be useful for this. Maybe it's also possible to post an URL and the artifact SHA1 on #8 automatically using gh. It needs people to log in via gh CLI though.


Set NoDelay as default

Resolves  #374

You're taking all my bugs! Not that I'm complaining :smile:

Was looking for something to test the Travis with that wouldn't unnessarily clog AppVeyor :grin:


Import Error: import mmcv

when i have a try to run test.py, there is an import error as below:
from .cv2 import *
ImportError: /lib64/libcairo.so.2: undefined symbol: FT_Get_Var_Design_Coordinates
After i checked the information, I found that this is a problem that the cairo version is too low.  but I have no idea how to install the right cairo version under non-root permissions on linux. So does any one have the same problem?

You may try to install it through conda. Anaconda maintains this package here.

You may try to install it through conda. Anaconda maintains this package here.

You may try to install it through conda. Anaconda maintains this package here.

it does work, thank you very much!

You may try to install it through conda. Anaconda maintains this package here.

it does work, thank you very much!


-source=go has no effect

Commit f637716 removed the special handling to only check Go comments (commit 1aaafc0 is the next commit that passed CI).
With commit 49cf02c715,
./misspell -source=go
void mispell()
{
// mispell
}
stdin:3:3:found "mispell" a misspelling of "misspell"

This appears to be the correct behaviour. The misspelling in a comment is identified and the misspelling in the function name is ignored.
With commit 1aaafc0,
./misspell -source=go
void mispell()
{
// mispell
}
/home/joel/sample.go:1:5:found "mispell" a misspelling of "misspell"
/home/joel/sample.go:3:4:found "mispell" a misspelling of "misspell"

Now the misspelling in the function name is also identified.

Hello, sorry for the delay.  I'm getting back into the swing of things.
I'll need to revisit the logic here.  Originally it was doing something very different for go, and not sure if the -source=go makes sense anymore or what.
In one case it was just treating it like a text file (check everything), later it just checked comments via parsing the source file.
Thoughts welcome on what you'd like to see for go.
I just learned about some linters allow go build tags as well. https://github.com/alecthomas/gometalinter/issues/275#issuecomment-294010321

See #47 as well

I'm actually using it for non-Go sources (mostly C++, Java and JavaScript). It's important that it doesn't check strings because I have resource strings that I don't want changed, and changing function and variable names can obviously be a problem.
I threw together https://github.com/carnegie-jwilliamson/misspell/commit/5bb422c63e6131347491a507375a200e0eaadd04 , which is basically the old version of ReplaceGo updated to follow a similar structure to how you changed Replace.

ah righto.  I'm happy with a PR on this actually since i think its the Right Thing to do.

https://github.com/client9/misspell/pull/104

coolio.. some minor merge issues (client9 vs. 	carnegie-jwilliamson) but I'll sort it out.

ok it's in.
thanks so much!
n

Thanks.


Setting for only showing sources in languages that I can check

I personally don't speak Russian and therefore can't check whether the Primary Source tool correctly parsed a statement. I would prefer it when the tool only shows me sources in languages I can check (and takes the information from Babel).

In order to ease us (the Wikidata dev team) to track the issues of the primary sources tool, we have created a Phabricator ticket for this issue: https://phabricator.wikimedia.org/T148144
Moving forward, please drop your comments onto that Phabricator ticket.


Fix video replay issue in iOS

Details
Fix video replay issue in iOS.
Fixed Issues
$ https://github.com/Expensify/App/issues/37249
PROPOSAL: https://github.com/Expensify/App/issues/37249#issuecomment-1968594898
Tests

Open video attachment in iOS.
Once the video completes, try to replay it.

[x] Verify that no errors appear in the JS console

QA Steps

Open video attachment in iOS.
Once the video completes, try to replay it.

[x] Verify that no errors appear in the JS console

PR Author Checklist

[x] I linked the correct issue in the ### Fixed Issues section above
[x] I wrote clear testing steps that cover the changes made in this PR

[x] I added steps for local testing in the Tests section
[x] I added steps for the expected offline behavior in the Offline steps section
[x] I added steps for Staging and/or Production testing in the QA steps section
[x] I added steps to cover failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[x] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)
[x] I tested this PR with a High Traffic account against the staging or production API to ensure there are no regressions (e.g. long loading states that impact usability).

[x] I included screenshots or videos for tests on all platforms
[x] I ran the tests on all platforms & verified they passed on:

[x] Android: Native
[x] Android: mWeb Chrome
[x] iOS: Native
[x] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[x] MacOS: Desktop

[x] I verified there are no console errors (if there's a console error not related to the PR, report it or open an issue for it to be fixed)
[x] I followed proper code patterns (see Reviewing the code)

[x] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick)
[x] I verified that the left part of a conditional rendering a React component is a boolean and NOT a string, e.g. myBool && <MyComponent />.
[x] I verified that comments were added to code that is not self explanatory
[x] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[x] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method

[x] If any non-english text was added/modified, I verified the translation was requested/reviewed in #expensify-open-source and it was approved by an internal Expensify engineer. Link to Slack message:

[x] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[x] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is either coming verbatim from figma or has been approved by marketing (in order to get marketing approval, ask the Bug Zero team member to add the Waiting for copy label to the issue)
[x] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[x] I verified the JSDocs style guidelines (in STYLE.md) were followed

[x] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[x] I followed the guidelines as stated in the Review Guidelines
[x] I tested other components that can be impacted by my changes (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar are working as expected)
[x] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[ x I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[x] I verified that if a function's arguments changed that all usages have also been updated correctly
[x] If any new file was added I verified that:

[x] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[x] If a new CSS style is added I verified that:

[x] A similar style doesn't already exist
[x] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(theme.componentBG))

[x] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[x] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[x] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[x] If the PR modifies a component or page that can be accessed by a direct deeplink, I verified that the code functions as expected when the deeplink is used - from a logged in and logged out account.
[x] If the PR modifies the UI (e.g. new buttons, new UI components, changing the padding/spacing/sizing, moving components, etc) or modifies the form input styles:

[x] I verified that all the inputs inside a form are aligned with each other.
[x] I added Design label and/or tagged @Expensify/design so the design team can review the changes.

[x] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[x] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.

Screenshots/Videos

Android: Native
https://github.com/Expensify/App/assets/35566748/c3b4195c-1f76-4d01-9c30-cbf9cdb4b0b6

Android: mWeb Chrome
https://github.com/Expensify/App/assets/35566748/493ecf91-3f14-43c7-8bb2-2ba71a1b6c1c

iOS: Native
https://github.com/Expensify/App/assets/35566748/aa3464db-35f3-4d22-a270-dfbbdc8b30a2

iOS: mWeb Safari
https://github.com/Expensify/App/assets/35566748/771c07d1-5f2f-4e81-b1b7-765ec0e14ca0

MacOS: Chrome / Safari
https://github.com/Expensify/App/assets/35566748/c9be9be5-f7ae-4030-97c3-7c06ed66a2e7

MacOS: Desktop
https://github.com/Expensify/App/assets/35566748/4ee41835-a43d-4f42-9c22-ddc16930a708

@ShridharGoel I understand the impacted platform is just iOS. But considering the change is in BaseVideoPlayer, can we please add test videos for all the platforms? We don't want any regressions because of this change.

Thanks for the update @ShridharGoel. While the code changes look good, I need to check if adding isPlaying dependency causes any further issues. Please allow a day for me to test these scenrios.

@ShridharGoel Can we elaborate the test steps to also add the full-screen mode?

Reviewer Checklist

[x] I have verified the author checklist is complete (all boxes are checked off).
[x] I verified the correct issue is linked in the ### Fixed Issues section above
[x] I verified testing steps are clear and they cover the changes made in this PR

[ ] I verified the steps for local testing are in the Tests section
[x] I verified the steps for Staging and/or Production testing are in the QA steps section
[x] I verified the steps cover any possible failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[x] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)

[x] I checked that screenshots or videos are included for tests on all platforms
[x] I included screenshots or videos for tests on all platforms
[ ] I verified tests pass on all platforms & I tested again on:

[ ] Android: Native
[ ] Android: mWeb Chrome
[ ] iOS: Native
[ ] iOS: mWeb Safari
[ ] MacOS: Chrome / Safari
[ ] MacOS: Desktop

[x] If there are any errors in the console that are unrelated to this PR, I either fixed them (preferred) or linked to where I reported them in Slack
[x] I verified proper code patterns were followed (see Reviewing the code)

[x] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick).
[x] I verified that the left part of a conditional rendering a React component is a boolean and NOT a string, e.g. myBool && <MyComponent />.
[x] I verified that comments were added to code that is not self explanatory
[x] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[x] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method
[x] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[x] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is either coming verbatim from figma or has been approved by marketing (in order to get marketing approval, ask the Bug Zero team member to add the Waiting for copy label to the issue)
[x] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[x] I verified the JSDocs style guidelines (in STYLE.md) were followed

[x] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[x] I verified that this PR follows the guidelines as stated in the Review Guidelines
[x] I verified other components that can be impacted by these changes have been tested, and I retested again (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar have been tested & I retested again)
[x] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[x] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[x] If a new component is created I verified that:

[x] A similar component doesn't exist in the codebase
[x] All props are defined accurately and each prop has a /** comment above it */
[x] The file is named correctly
[x] The component has a clear name that is non-ambiguous and the purpose of the component can be inferred from the name alone
[x] The only data being stored in the state is data necessary for rendering and nothing else
[x] For Class Components, any internal methods passed to components event handlers are bound to this properly so there are no scoping issues (i.e. for onClick={this.submit} the method this.submit should be bound to this in the constructor)
[x] Any internal methods bound to this are necessary to be bound (i.e. avoid this.submit = this.submit.bind(this); if this.submit is never passed to a component event handler like onClick)
[x] All JSX used for rendering exists in the render method
[x] The component has the minimum amount of code necessary for its purpose, and it is broken down into smaller components in order to separate concerns and functions

[x] If any new file was added I verified that:

[x] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[x] If a new CSS style is added I verified that:

[x] A similar style doesn't already exist
[x] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(theme.componentBG)

[x] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[x] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[x] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[x] If the PR modifies a component or page that can be accessed by a direct deeplink, I verified that the code functions as expected when the deeplink is used - from a logged in and logged out account.
[x] If the PR modifies the UI (e.g. new buttons, new UI components, changing the padding/spacing/sizing, moving components, etc) or modifies the form input styles:

[x] I verified that all the inputs inside a form are aligned with each other.
[x] I added Design label and/or tagged @Expensify/design so the design team can review the changes.

[x] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[x] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.
[x] I have checked off every checkbox in the PR reviewer checklist, including those that don't apply to this PR.

Screenshots/Videos

Android: Native

Android: mWeb Chrome

iOS: Native

iOS: mWeb Safari

MacOS: Chrome / Safari

MacOS: Desktop

Yes the center play button to be clicked in the full screen mode after the video is complete.

fyi I just followed the steps mentioned in the issue reproduction steps.

@ShridharGoel I had tried in your branch only with the latest main at that point of time. I'll take a look at my end again, but can we merge the latest main and try once again?
Can you upload the screencast of the iOS along with the steps you performed?

Hmm. Thanks for the video, I'll take a look.

@ShridharGoel It worked this time. I am working on the checklist, but I have one last comment. Can we improve the content of the Test steps?
Let's include, the part for fullscreen and also mention that we should hit Play-Pause a few times when the video is played.

@mananjadhav shouldReplayVideo always returns false for web. This change shouldn't affect that.

I tested this against the staging env and latest main. Can you confirm at your end?

I checked on Web it worked fine, but I'll need time to test all platforms.

I think for all the platforms, we should be doing a thorough testing for all play-pause controls, the buttons, the screen, full screen mode, and play-pause end of the video, middle of the video.

Can you please update your screencasts to cover. This is to ensure we don't add any regression.

Also, can we improve the content of the Test steps? Let's include, the part for fullscreen and also mention that we should hit Play-Pause a few times when the video is played.

And can you also update the Test steps as requested earlier?

Thanks, just post a comment once the videos are uploaded. I think I'll also be able to test by that time. Thank you for your patience.

Reviewer Checklist

[x] I have verified the author checklist is complete (all boxes are checked off).
[x] I verified the correct issue is linked in the ### Fixed Issues section above
[x] I verified testing steps are clear and they cover the changes made in this PR

[x] I verified the steps for local testing are in the Tests section
[x] I verified the steps for Staging and/or Production testing are in the QA steps section
[x] I verified the steps cover any possible failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[x] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)

[x] I checked that screenshots or videos are included for tests on all platforms
[x] I included screenshots or videos for tests on all platforms
[x] I verified tests pass on all platforms & I tested again on:

[x] Android: Native
[x] Android: mWeb Chrome
[x] iOS: Native
[ ] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[x] MacOS: Desktop

[x] If there are any errors in the console that are unrelated to this PR, I either fixed them (preferred) or linked to where I reported them in Slack
[x] I verified proper code patterns were followed (see Reviewing the code)

[x] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick).
[x] I verified that the left part of a conditional rendering a React component is a boolean and NOT a string, e.g. myBool && <MyComponent />.
[x] I verified that comments were added to code that is not self explanatory
[x] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[x] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method
[x] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[x] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is either coming verbatim from figma or has been approved by marketing (in order to get marketing approval, ask the Bug Zero team member to add the Waiting for copy label to the issue)
[x] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[x] I verified the JSDocs style guidelines (in STYLE.md) were followed

[x] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[x] I verified that this PR follows the guidelines as stated in the Review Guidelines
[x] I verified other components that can be impacted by these changes have been tested, and I retested again (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar have been tested & I retested again)
[x] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[x] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[x] If a new component is created I verified that:

[x] A similar component doesn't exist in the codebase
[x] All props are defined accurately and each prop has a /** comment above it */
[x] The file is named correctly
[x] The component has a clear name that is non-ambiguous and the purpose of the component can be inferred from the name alone
[x] The only data being stored in the state is data necessary for rendering and nothing else
[x] For Class Components, any internal methods passed to components event handlers are bound to this properly so there are no scoping issues (i.e. for onClick={this.submit} the method this.submit should be bound to this in the constructor)
[x] Any internal methods bound to this are necessary to be bound (i.e. avoid this.submit = this.submit.bind(this); if this.submit is never passed to a component event handler like onClick)
[x] All JSX used for rendering exists in the render method
[x] The component has the minimum amount of code necessary for its purpose, and it is broken down into smaller components in order to separate concerns and functions

[x] If any new file was added I verified that:

[x] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[x] If a new CSS style is added I verified that:

[x] A similar style doesn't already exist
[x] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(theme.componentBG)

[x] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[x] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[x] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[x] If the PR modifies a component or page that can be accessed by a direct deeplink, I verified that the code functions as expected when the deeplink is used - from a logged in and logged out account.
[x] If the PR modifies the UI (e.g. new buttons, new UI components, changing the padding/spacing/sizing, moving components, etc) or modifies the form input styles:

[x] I verified that all the inputs inside a form are aligned with each other.
[x] I added Design label and/or tagged @Expensify/design so the design team can review the changes.

[x] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[x] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.
[x] I have checked off every checkbox in the PR reviewer checklist, including those that don't apply to this PR.

Screenshots/Videos

Android: Native
https://github.com/Expensify/App/assets/3069065/caad55b1-9efb-46a9-9611-8da94b6be026

Android: mWeb Chrome
https://github.com/Expensify/App/assets/3069065/63f87c41-ff8c-4846-9c8b-320e896fc603

iOS: Native
https://github.com/Expensify/App/assets/3069065/f20e895b-dea2-4abf-866f-35606f897b54

iOS: mWeb Safari

MacOS: Chrome / Safari
https://github.com/Expensify/App/assets/3069065/6906868e-4b4a-4a1f-90e2-dd46a694e6a8

MacOS: Desktop
https://github.com/Expensify/App/assets/3069065/e340ef99-4f3a-4a31-a2d6-ce8e333b1dfe

:hand: This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release.

🚀 Deployed to production by https://github.com/Julesssss in version: 1.4.60-13 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅


Unable to verify certificate

Nothing out of the ordinary, used yt-dl to grab https://www.youtube.com/watch?v=pWxJEIz7sSA and it kicked back that it could not verify the certificate.  Using latest build 11/10.
ERROR: Unable to download webpage: (urlopen error [SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed (_sll.c:590 (caused by URLError(1, u'[SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed (_ssl.c:590)'),)) ```

Which operating system are you using? Windows XP, 7 or Linux, Mac OS X? Please post the full log from the following command:
youtube-dl --verbose "https://www.youtube.com/watch?v=pWxJEIz7sSA"

Windows 7 latest service pack.
Enter youtube url: youtube-dl --verbose https://www.youtube.com/watch?v=pWxJEIz7
sSA
Downloading video...
WARNING: The url doesn't specify the protocol, trying with http
WARNING: Could not send HEAD request to http://youtube-dl --verbose https://www.
youtube.com/watch?v=pWxJEIz7sSA: <urlopen error [Errno 11004] getaddrinfo failed
>
ERROR: Unable to download webpage: <urlopen error [Errno 11004] getaddrinfo fail
ed> (caused by URLError(gaierror(11004, 'getaddrinfo failed'),))

Couldn't download the youtube video. Did you copy the url correctly?
Press any key to continue . . .

Looks like you're using a wrapper but not youtube-dl itself. Could you run the command given above in Command Prompt (cmd.exe)?

i'm using youtube-dl on .net web service it also get this error
Traceback (most recent call last): File "main.py", line 19, in  File "youtube_dl_init_.pyo", line 410, in main File "youtube_dl_init_.pyo", line 377, in _real_main File "youtube_dl\YoutubeDL.pyo", line 338, in init File "youtube_dl\YoutubeDL.pyo", line 1969, in _setup_opener File "youtube_dl\utils.pyo", line 492, in make_HTTPS_handler File "ssl.pyo", line 440, in create_default_context File "ssl.pyo", line 391, in load_default_certs File "ssl.pyo", line 378, in _load_windows_store_certs WindowsError: [Error 5] Access is denied

I have same issue.  Using latest build 11/10:
Downloading video... ERROR: Unable to download webpage: (urlopen error [SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed (_sll.c:590 (caused by URLError(1, u'[SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed (_ssl.c:590)'),))
Using command line prompt, the following command is issued:
youtube-dl -i --batch-file ./ImportChoons.txt --format "best" --extract-audio --audio-format "best" --audio-quality 0
where ImportChoons.txt is a file containing misc links to video clips on YouTube.
OS:  Windows 7 Professional, 64 bit, Service Pack 1
Only change from youtube-dl working and not working is upgrading youtube-dl to 11/10/2015 version

Duplicate of #7309.


Fixes for dgl example

Description of changes:
Minor fixes for DGL training scripts.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@ChuyangDeng


[Feature Request] Reusing component with multiple data in one Vue instance

Existing Component
Yes
Component Name
Select
Description
Similar components for one page but individual data populate
<el-select v-model="values" placeholder="Select Account">
              <el-option
                      v-for="item in optionss"
                      :key="item.values"
                      :label="item.label"
                      :values="item.values">
              </el-option>
 </el-select>

  <el-select v-model="value" placeholder="Select Account">
              <el-option
                      v-for="item in options"
                      :key="item.value"
                      :label="item.label"
                      :value="item.value">
              </el-option>
</el-select>

One component works fine but other components select not work, but data populated correctly, Why?

I think you seem to be wrong.
In first component, the attr should be 'value', not be 'values'
<el-select v-model="values" placeholder="Select Account">
              <el-option
                      v-for="item in optionss"
                      :key="item.values"
                      :label="item.label"
                      :value="item.values">
              </el-option>
 </el-select>

Okay fine ! but on select items will populate, but why are others not working?

Okay fine ! but on select items will populate, but why are others not working?

Okay fine ! but on select items will populate, but why are others not working?

Yes! Here is the link
https://codepen.io/momin-ctg/pen/QWLgOpL


I'll push the extra commit enabling actual cross-compilation if the CI is fine with the current changes.

Weird, I wouldn't have expected the build to pass at https://github.com/lihaoyi/upickle/pull/267/commits/e6959567d67aaef1ed22cee7ea156c3177a9a8b9, without https://github.com/lihaoyi/upickle/pull/267/commits/756f1cce80dfda6b5084b0bfb5cb7426e6830150.


Fix headache checkbox value

I noticed what's probably a copy-paste bug in the code, which gives the headache checkbox the wrong checked state. This should fix that, though I haven't tested this locally.

Good catch!


Required checks are stuck at pending with auto_fix

Hello and thank you for this great action
Since GITHUB_TOKEN is designed for not triggering other workflows, if there is a fix done by the linter, the required checks are stuck at a pending state. I tried using the following snippet which creates a temporary token from a GitHub app with required scopes but it is not triggering the workflow although I successfully use the same snippet to trigger other workflows.
Is there a workaround for this?
Thanks!
  lint:
    name: Lint
    if: ${{ !github.event.pull_request.draft }}
    runs-on: ubuntu-latest
    strategy:
      matrix:
        node-version: [14.x]
    steps:
    - name: Get Token
      id: get_workflow_token
      uses: peter-murray/workflow-application-token-action@v1
      with:
        application_id: ${{ secrets.APPLICATION_ID }}
        application_private_key: ${{ secrets.APPLICATION_PRIVATE_KEY }}

    - name: Check Out Repo
      uses: actions/checkout@v2

    - name: Use Node.js ${{ matrix.node-version }}
      uses: actions/setup-node@v3
      with:
        node-version: ${{ matrix.node-version }}

    - name: Install dependencies
      run: yarn install

    - name: Run linters
      uses: wearerequired/lint-action@v2
      with:
        github_token: ${{ steps.get_workflow_token.outputs.token }}
        eslint: true
        prettier: true
        auto_fix: true

What happens if you use the same token for actions/checkout?


fix consistent hashing test for multicluster

Select the network in the service to avoid hitting the gateway which makes consistent hashing unpredictable (we only perform it at the client, not the TCP gateway)

@stevenctl: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

release-notes_istio
6b0aa10a3a4c3643b06b53ae316ce30c06de3232
link
/test release-notes_istio

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 kubernetes/test-infra repository. I understand the commands that are listed here.

@stevenctl: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

release-notes_istio
6b0aa10a3a4c3643b06b53ae316ce30c06de3232
link
/test release-notes_istio

integ-pilot-multicluster-tests_istio
6b0aa10a3a4c3643b06b53ae316ce30c06de3232
link
/test integ-pilot-multicluster-tests_istio

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 kubernetes/test-infra repository. I understand the commands that are listed here.

@stevenctl: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

integ-pilot-multicluster-tests_istio
6b0aa10a3a4c3643b06b53ae316ce30c06de3232
link
/test integ-pilot-multicluster-tests_istio

release-notes_istio
24aec9c8455dc63d2ea3df9c3235423972aadab2
link
/test release-notes_istio

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 kubernetes/test-infra repository. I understand the commands that are listed here.

/retest

@stevenctl: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

integ-pilot-multicluster-tests_istio
24aec9c8455dc63d2ea3df9c3235423972aadab2
link
/test integ-pilot-multicluster-tests_istio

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 kubernetes/test-infra repository. I understand the commands that are listed here.


cant enable mysqli_*

Output
PHP Fatal error:  Uncaught Error: Call to undefined function mysqli_connect()

Expected Result
No error at all

Platform
OS: "Linux Mint 18.1 Serena"
Stuff I already tried:

install extension using phpbrew ext install mysqli

Result:
myuseraccount@localhost / $ tail /home/myuseraccount/.phpbrew/build/7.1.2-zts-enabled/ext/mysqli/build.log
make: Entering directory '/home/myuseraccount/.phpbrew/build/7.1.2-zts-enabled/ext/mysqli'
/bin/bash /home/myuseraccount/.phpbrew/build/7.1.2-zts-enabled/ext/mysqli/libtool --mode=compile cc -DZEND_ENABLE_STATIC_TSRMLS_CACHE=1 -I. -I/home/myuseraccount/.phpbrew/build/7.1.2-zts-enabled/ext/mysqli -DPHP_ATOM_INC -I/home/myuseraccount/.phpbrew/build/7.1.2-zts-enabled/ext/mysqli/include -I/home/myuseraccount/.phpbrew/build/7.1.2-zts-enabled/ext/mysqli/main -I/home/myuseraccount/.phpbrew/build/7.1.2-zts-enabled/ext/mysqli -I/home/myuseraccount/.phpbrew/php/7.1.2-zts-enabled/include/php -I/home/myuseraccount/.phpbrew/php/7.1.2-zts-enabled/include/php/main -I/home/myuseraccount/.phpbrew/php/7.1.2-zts-enabled/include/php/TSRM -I/home/myuseraccount/.phpbrew/php/7.1.2-zts-enabled/include/php/Zend -I/home/myuseraccount/.phpbrew/php/7.1.2-zts-enabled/include/php/ext -I/home/myuseraccount/.phpbrew/php/7.1.2-zts-enabled/include/php/ext/date/lib  -DHAVE_CONFIG_H  -g -O2   -c /home/myuseraccount/.phpbrew/build/7.1.2-zts-enabled/ext/mysqli/mysqli.c -o mysqli.lo
mkdir .libs
 cc -DZEND_ENABLE_STATIC_TSRMLS_CACHE=1 -I. -I/home/myuseraccount/.phpbrew/build/7.1.2-zts-enabled/ext/mysqli -DPHP_ATOM_INC -I/home/myuseraccount/.phpbrew/build/7.1.2-zts-enabled/ext/mysqli/include -I/home/myuseraccount/.phpbrew/build/7.1.2-zts-enabled/ext/mysqli/main -I/home/myuseraccount/.phpbrew/build/7.1.2-zts-enabled/ext/mysqli -I/home/myuseraccount/.phpbrew/php/7.1.2-zts-enabled/include/php -I/home/myuseraccount/.phpbrew/php/7.1.2-zts-enabled/include/php/main -I/home/myuseraccount/.phpbrew/php/7.1.2-zts-enabled/include/php/TSRM -I/home/myuseraccount/.phpbrew/php/7.1.2-zts-enabled/include/php/Zend -I/home/myuseraccount/.phpbrew/php/7.1.2-zts-enabled/include/php/ext -I/home/myuseraccount/.phpbrew/php/7.1.2-zts-enabled/include/php/ext/date/lib -DHAVE_CONFIG_H -g -O2 -c /home/myuseraccount/.phpbrew/build/7.1.2-zts-enabled/ext/mysqli/mysqli.c  -fPIC -DPIC -o .libs/mysqli.o
In file included from /home/myuseraccount/.phpbrew/build/7.1.2-zts-enabled/ext/mysqli/mysqli.c:34:0:
/home/myuseraccount/.phpbrew/build/7.1.2-zts-enabled/ext/mysqli/php_mysqli_structs.h:42:33: fatal error: ext/mysqlnd/mysqlnd.h: No such file or directory
compilation terminated.
Makefile:194: recipe for target 'mysqli.lo' failed
make: *** [mysqli.lo] Error 1
make: Leaving directory '/home/myuseraccount/.phpbrew/build/7.1.2-zts-enabled/ext/mysqli'

install the extension using sudo apt-get install php-mysqli

Result:
Reading package lists... Done
Building dependency tree
Reading state information... Done
Note, selecting 'php7.0-mysql' instead of 'php-mysqli'
php7.0-mysql is already the newest version (7.0.15-0ubuntu<IP_ADDRESS>).
0 upgraded, 0 newly installed, 0 to remove and 28 not upgraded.

manually add the extension in the php.ini file:
first sudo locate php.ini, gave me:

/etc/php/7.0/apache2/php.ini
/etc/php/7.0/apache2/php.ini.ucf-dist
/etc/php/7.0/cli/php.ini
/home/myuseraccount/.phpbrew/build/7.1.2-phpbrew-test/php.ini-development
/home/myuseraccount/.phpbrew/build/7.1.2-phpbrew-test/php.ini-production
/home/myuseraccount/.phpbrew/build/7.1.2-zts-enabled/php.ini-development
/home/myuseraccount/.phpbrew/build/7.1.2-zts-enabled/php.ini-production
/home/myuseraccount/.phpbrew/php/7.1.2-phpbrew-test/etc/php.ini
/home/myuseraccount/.phpbrew/php/7.1.2-zts-enabled/etc/php.ini
/usr/lib/php/7.0/php.ini-development
/usr/lib/php/7.0/php.ini-production
/usr/lib/php/7.0/php.ini-production.cli

second php --info | grep php.ini told me that:

Configuration File (php.ini) Path => /home/atekin/.phpbrew/php/7.1.2-zts-enabled/etc
Loaded Configuration File => /home/atekin/.phpbrew/php/7.1.2-zts-enabled/etc/php.ini

went into the file and added the line extension=mysqli.so

reboot doesnt work

php --info | grep mysql gives me this output:

PHP Warning:  PHP Startup: Unable to load dynamic library '/home/myuseraccount/.phpbrew/php/7.1.2-zts-enabled/lib/php/extensions/no-debug-zts-20160303/mysqli.so' - /home/myuseraccount/.phpbrew/php/7.1.2-zts-enabled/lib/php/extensions/no-debug-zts-20160303/mysqli.so: cannot open shared object file: No such file or directory in Unknown on line 0
Warning: PHP Startup: Unable to load dynamic library '/home/myuseraccount/.phpbrew/php/7.1.2-zts-enabled/lib/php/extensions/no-debug-zts-20160303/mysqli.so' - /home/myuseraccount/.phpbrew/php/7.1.2-zts-enabled/lib/php/extensions/no-debug-zts-20160303/mysqli.so: cannot open shared object file: No such file or directory in Unknown on line 0

I wish i had not installed phpbrew because previously everything worked....
If there is no solution to this:
How can i remove every trace of phpbrew and have my native php back?

the log you pasted of phpbrew ext install mysqli indicates that you didn't have the header files of mysql, so the build process failed.
you can simply phpbrew off to get your native php back.

@jhdxr it looks like a PHP bug or at least lack of documentation. The ext/mysqlnd/mysqlnd.h header file is part of the PHP distribution and is not expected to be found anywhere outside. But in my experience, I was never able to compile the mysqli extension apart from compiling PHP itself.
@keltik85 please look at https://github.com/phpbrew/phpbrew/issues/322#issuecomment-57940643 and see if it helps. As a workaround, you can re-compile PHP with mysqli linked statically.


Type-mismatch error boxed Trait

Hi,
The latest nightly 3-25-2022 ( 0.3.989) on Mac vscode gives me this error. Cargo build this is ok. Also the RA build 0.3.986 don't
give this error. rustc -V
rustc 1.58.0 (02072b482 2022-01-11)
[{
"resource": "/Users/ville.mattila/src/smc_umbrella/backend/src/manager/policy.rs",
"owner": "rustc",
"code": {
"value": "type-mismatch",
"target": {
"$mid": 1,
"external": "https://rust-analyzer.github.io/manual.html#type-mismatch",
"path": "/manual.html",
"scheme": "https",
"authority": "rust-analyzer.github.io",
"fragment": "type-mismatch"
}
},
"severity": 8,
"message": "expected Box<dyn Foo<FooItem = ::BarItem>, Global>, found Box<F, Global>",
"source": "rust-analyzer",
"startLineNumber": 33,
"startColumn": 26,
"endLineNumber": 33,
"endColumn": 39
}]
Here is source code. I grant this may be most correct rust, but it works for me. You can give suggestion better code too :-) ,
but RA should not give error, where rustc is happy.

trait Foo {
    type FooItem;
}

trait Bar {
    type BarItem: Foo;
    fn add_foo(&self, bar: Box<dyn Foo<FooItem=Self::BarItem>>) {

    }
}

struct Baz<B> {
    bar: B,
}

impl<B:Bar>  Baz<B> {
    fn add_rule<F: 'static>(&mut self, mut foo: F)
    where F: Foo<FooItem = <B as Bar>::BarItem> {
        self.bar.add_foo(Box::new(foo)); // <<-- THIS line give RA Error now
    }
}

This will be fixed by https://github.com/rust-lang/chalk/pull/754 once that gets merged. Also, https://github.com/rust-analyzer/rust-analyzer/pull/11809 will hide the diagnostic unless you explicitly enable experimental errors :slightly_smiling_face:

Ok thanks!


Failed to allocate memory, size 200: no suitable memory region found

Hey, I just tested your hook library by slightly modifying the example. However, I keep getting the same error regardless of the type of hook I use:
callhookexample\log\CallHook: Allocating 200 bytes near module x2.exe
callhookexample\log\CallHook: Failed to allocate memory, size 200: no suitable memory region found
callhookexample\log\CallHook: Error: failed to hook call at<PHONE_NUMBER>7E21DD - unable to allocate memory.
Does anyone know what could be causing this? It's strange that the same memory size is allocated for different types of hooks. Thanks in advance.

I see, it's weird that the call to VirtualAlloc fails here. The piece of code you removed makes sure that the memory region allocated for the hook is as close as possible to the end of the checked memory region. This is done by subtracting the size of the allocation cb from the pointer to the end of the region end:
alloc = VirtualAlloc(alloc, cb, MEM_COMMIT | MEM_RESERVE, PAGE_EXECUTE_READWRITE);

This has never failed for me (or the many others who have used my mods - for example, memory allocations in ELDEN RING for hooks will often happen below the base address of the application image).
According to Microsoft, the address given to VirtualAlloc will be rounded down to the nearest page/allocation granularity boundary. If you have any ideas why it fails in the environment you run it in (please provide some details like your OS, arch etc.), please let me know so I can fix it.

Oh I see, yes that is pretty specific. It's not an AC I'm familiar with the workings of. Anyhow, seems like the issue is in the measures in place and not in my code. If you ever figure out which part made it fail (and why your fix worked) let me know, I am curious.
Good luck with your project!


Turn off mix-manifest override

Laravel Mix Version: 0.10.0 (npm list --depth=0)

This is not a bug but a feature that would be nice to see in Mix.
I'm using Mix to process a lot of assets (mostly one sass and one js file per page and some global assets) and when the list of assets gets really long I often comment out the assets I don't need at the moment to speed up the webpack initialization.
The problem is: When I do that, all previously compiled files (from the mix-manifest) are deleted before the new compilation, so I manually have to edit the manifest before running Mix (so that the files I still need are not deleted) and that can get quite annoying.
It would be nice if we could do something like mix.keepManifest() to instruct Mix to only replace the entries in the manifest.

There is Stackoverflow topic
Laravel Mix multiple entry points generates one manifest.js where is discussed splitting webpack.mix.js approach which is more appropriate in this case.
Otherwise, there are some disadvantages:
1.
2.


Make a benign edit to test CODEOWNERS

I am going to test the split-line aspect of this by first getting a Github review from @evankanderson who is a member of the TOC and release leads, but not serving-writers (the bottom-most match).

whoops was just hovering - didn't mean to click it


Contour layer is overflowed with red color on Mac devices

I have an app which facilitates Cornerstone API (ver. 2.2.8) to render Contour image over an NM image (2 layers).
Everything works flawlessly on Windows devices (Chrome and Firefox) but there is an edge cases on Mac devices (Safari and Chrome as well) where one of the images is shown as a red rectangle.
At first, I thought, may be it related to color-map I've defined for Contour images, but after removing the color-map definition (the Contour image still exists but invisible) the issue has still remained.

Fix https://github.com/cornerstonejs/cornerstone/pull/572


Replace colored with owo-colors

#605

This looks so much cleaner!

I agree, thanks for suggesting this library, it does exactly what we need and much more 💯 .


Waste/summary avalanches.

Tibialyzer doesn't count avalanches to my waste. They do pop up in the "used items" sections but are not added to waste. Other runes and potions work fine.
Screenshot: https://scr.hu/zw2Yed

I don't get it. 175 gmp + 48 SDs are worth around 26k. Your waste seems to be over 66k so it counts some avalanches... I would say the majority of them. But the accurate value should be around 76k after we add all the avalanches. Something is definitely wrong but not as you described.

Those are ultimate mana potions 350 GP each.

Sorry then but they look exactly as gmp on this screenshot.

Yea but belive me those are UMP :)

Check which price you have set for Avalanch runes. It calculates your waste with the price you set at used item.
item@avalanche rune

Is there 45 at the top right side?

@KuerbisGnu OMG you were right. Idk how did the price changed... Thanks mate :)


Nested insert fails, when json schema has { additionalProperties: false }

Looks like #id variables are also validated
 1) Nested populator simple nested sample data:
     Error: {
  "#id": "Additional properties not allowed"
}

The #id properties are removed from the models by setting them to undefined. You can change this to delete by overriding the omitImpl method of a model:
class BaseModel extends Model {
  static omitImpl(obj, key) {
    delete obj[key];
  }
}

got it working with omitImpl 👍

I'll change the default omitImpl to use delete.


Through does not fetch all pivot fields

I've got 3 models setup. For simplicity sake, I'll use the example of a document having several authors. So the relationship would basically be:
bookshelf.model('Author', {
  documents() {
     return this.belongsToMany('Document').through('AuthorDocument');
  }
})

bookshelf.model('Document', {
  authors() {
     return this.belongsToMany('Author').through('AuthorDocument');
  }
})

bookshelf.model('AuthorDocument', {
  document() {
     return this.belongsTo('Document');
  }

  author() {
    return this.belongsTo('Author');
  }
})

And let's say that the authors_documents table has the following data to further describe the relationship:
id | author_id | document_id | last_accessed | role
1  | 1         | 1           | 2016-10-26    | 'OWNER'

According to the docs, the through function used to specify the belongsToMany relationship should fetch last_accessed and role and put it into the model with _pivot_ prefix; however, that doesn't happen. The only _pivot_ properties that show up are id, author_id, and document_id. And I'm not sure why.
I had to use withPivot along with through in order to get that data by manually specifying the columns which makes through completely useless in this context.
Did I misunderstand something or is this not working correctly?

As far as I understand the documentation (it is very confusing on this topic), you need to use withPivot in order to fetch extra columns. So if this is working for you, it's as expected.

@rikkit thanks!
I don't use the library anymore so I'll just go ahead and close this out :)


Deployment Shield for Readme

Issue

The only way to really see the deployment besides going to deployments is to go to the commit. It would be nice to have some kind of shield that points to the Pipeline that deploys to Docker Hub and know it is passing.

Solution

Add the proper shield to the top of the README.md file
The name of the of the shield can Deploy with the same format as the Test and Build Pipelines.

Screenshot

Deploy Shield merged from #107, closing.


Bump up DJL version to 0.23.0-SNAPSHOT

Description
Brief description of what this PR is about

If this change is a backward incompatible change, why must this change be made?
Interesting edge cases to note here

Codecov Report
Patch coverage: 72.32% and project coverage change: +1.55 :tada:

Comparison is base (bb5073f) 72.08% compared to head (1acd551) 73.64%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@             Coverage Diff              @@
##             master    #2564      +/-   ##
============================================
+ Coverage     72.08%   73.64%   +1.55%
- Complexity     5126     6939    +1813
============================================
  Files           473      684     +211
  Lines         21970    30296    +8326
  Branches       2351     3142     +791
============================================
+ Hits          15838    22312    +6474
- Misses         4925     6456    +1531
- Partials       1207     1528     +321

Impacted Files
Coverage Δ

api/src/main/java/ai/djl/modality/cv/Image.java
69.23% <ø> (-4.11%)
:arrow_down:

...rc/main/java/ai/djl/modality/cv/MultiBoxPrior.java
76.00% <ø> (ø)

.../main/java/ai/djl/modality/cv/output/Landmark.java
100.00% <ø> (ø)

...i/djl/modality/cv/translator/BigGANTranslator.java
21.42% <0.00%> (-5.24%)
:arrow_down:

.../modality/cv/translator/ImageFeatureExtractor.java
0.00% <0.00%> (ø)

.../ai/djl/modality/cv/translator/YoloTranslator.java
27.77% <0.00%> (+18.95%)
:arrow_up:

...ain/java/ai/djl/modality/cv/util/NDImageUtils.java
67.10% <0.00%> (+7.89%)
:arrow_up:

api/src/main/java/ai/djl/modality/nlp/Decoder.java
63.63% <ø> (ø)

api/src/main/java/ai/djl/modality/nlp/Encoder.java
66.66% <ø> (ø)

.../main/java/ai/djl/modality/nlp/EncoderDecoder.java
64.00% <ø> (ø)

... and 228 more

... and 352 files with indirect coverage changes
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Don't Merge! Regression PR perf testing with metadata validation

Docs Build status updates of commit 4d5bf3f:
:clock10: Full build: the average full build time is 19 min(s) 10 sec(s), this is based on the last 216 build(s) for this repository.

Docs Build status updates of commit 4d5bf3f:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details
Dependent Files

TestingPullRequest.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Fix user menu item styling

Small fix for the user menu. I'm not exactly sure but I think Semantic-UI is adding a display:flex !important inline style to the menu which destroys the item alignment. Workaround by explicitely setting a width and removing the useless border-radius on the items.
Before

After

Codecov Report

Merging #4985 into master will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff           @@
##           master   #4985   +/-   ##
======================================
  Coverage    37.4%   37.4%
======================================
  Files         306     306
  Lines       45343   45343
======================================
  Hits        16959   16959
  Misses      25931   25931
  Partials     2453    2453

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 3973572...a3792fb. Read the comment docs.

For maintainers: I've tagged this as 1.6 because it is a bug. Although if someone disagrees with me adding this into the release please feel free to change it to 1.7

Force-pushed another tweak. The border radius removment didn't actually work, now that I've added a !important to it.

@silverwind merged. Thanks for the PR 😄


Error: Cannot find module 'fs/promises'

Electron.NET CLI 9.31.2
Node version v14.17.6
Dotnet core 3.1
electron-builder 22.14.5
Windows 10

I have been building an application with Electron.NET for several years that works well. Recently, I rebuilt the application successfully but when I launch the app, I get the following error message in a popup:
Error: Cannot find module 'fs/promises'
Stack:
...\AppUpdater.js
...\main.js
...\autoUpdater.js
I can't use the file browser or access the dev tools of the app.
I can't solve my problem. I tried to use older versions of electron builder, 22.10.5 in particular by installing it globally, but this does not solve my bug.
I went back to the package.json of my application at a point where I was sure that it worked, but I still have this problem. I regenerated the lock package, removed the node_modules, reinstalled node js 14.17.1.
I'm going in circles. I don't know how to debug this problem.

We have same problem, someone looking for a solution

you have any soluction? in my work team we have the same problem, from yesterday we could not compile the project.

We can compile, but when it starts error message box appears.
I fixed it in the following way:

manually built electron with fixed, as in frninja commit, package.json(instructions below)
copy resource/app.asar
after electronize build I replace produced resource/app.asar with previously copied good one

How to create app.asar.
When you call ecectronize build you can see build directory and commands, in my case it's App\obj\desktop\custom:

go to this folder, here you will find package.json, open it and change electron-updater from ^4.3.5 to 4.3.9
then run  npm install --production
after it successfully completes run npx electron-builder --config=./bin/electron-builder.json --win --ia32 -c.electronVersion=11.1.1 exact command line you can see in electronize build output.
in the last line of the output you will see appOutDir=...\App\bin\Desktop\win-ia32-unpacked then you can find your app.asar in App\bin\Desktop\win-ia32-unpacked\resources\app.asar

you have any soluction? in my work team we have the same problem, from yesterday we could not compile the project.

We can compile, but when it starts error message box appears. I fixed it in the following way:

manually built electron with fixed, as in frninja commit, package.json(instructions below)
copy resource/app.asar
after electronize build I replace produced resource/app.asar with previously copied good one

How to create app.asar.
When you call ecectronize build you can see build directory and commands, in my case it's App\obj\desktop\custom:

go to this folder, here you will find package.json, open it and change electron-updater from ^4.3.5 to 4.3.9
then run  npm install --production
after it successfully completes run npx electron-builder --config=./bin/electron-builder.json --win --ia32 -c.electronVersion=11.1.1 exact command line you can see in electronize build output.
in the last line of the output you will see appOutDir=...\App\bin\Desktop\win-ia32-unpacked then you can find your app.asar in App\bin\Desktop\win-ia32-unpacked\resources\app.asar

this solution is good for to do the executable file, but isn't our case because this problem occurs when fun electronize start :(

Thx frninja
In my opinion, the best way to fix the bug is to regenerate ElectronNet.CLI.

Clone the repo
Checkout version 9.31.2
Patch the package.json of ElectronNET.Host by pinning the version of electron-updater to 4.3.9. Other versions should work. I also fixed the versions of the other packages to avoid new surprises
Use the buildReleaseNuGetPackages.cmd to regenerate ElectronNet.CLI nuget package. I changed the version to 9.31.3 to facilitate deployement
Install your new nuget from your local repo and use your favorite electron app generation command again

For Electron.NET 9.31.2, I uninstalled ElectronNet.CLI 9.31.2, then installed ElectronNet.CLI 13.5.1. It works for me so far.
But if the specific version of electron-builder can be used during building, it would be able to solve the root issue.

I am also getting errors and need help. I can run electronize start with no issues, but when running the executable I get this error as well.  Please help.  I plan to try the above workaround for now.  BLOCKED

Ok I was able to get it working...I upgraded package.json in the following way.
"electron-updater": "4.6.1"
"electron": "^16.0.3",
And following Cedric-LE example.  I ran "buildAll" instead of buildReleaseNuGetPackages.cmd.  Latest update to the latest electron build.

I have fixed this issue by doing following changes to my project

Upgraded my project from dotnet core 3.1 to dot net 5.0
Upgraded ElectronNET API from 9.32.1 to 13.5.1
Upgraded ElectronNET CLI from 9.32.1 to <IP_ADDRESS>
Modified package.json dependencies and devDependencies to below

"dependencies": {
   "electron-updater": "4.3.9",
   "image-size": "1.0.0",
   "portscanner": "^2.2.0",
   "socket.io": "^2.4.0",
   "dasherize": "2.0.0"
 },
 "devDependencies": {
   "@types/node": "^15.14.0",
   "electron": "^13.1.5",
   "tslint": "^6.1.3",
   "typescript": "^4.3.5"
 }

Ok I was able to get it working...I upgraded package.json in the following way. "electron-updater": "4.6.1" "electron": "^16.0.3", And following Cedric-LE example. I ran "buildAll" instead of buildReleaseNuGetPackages.cmd. I will update to the latest electron build.

Yes, I just only upgraded electron-updater to 4.3.9 and ran "buildAll.cmd", It works fine!
Thanks a lot!

Hi, I've tried every solution in here and I still get this error, both when running the built Windows .exe and when running "electronize start".
I've encountered this before and found that upgrading Node.js resolved it, but not this time.
Currently unable to release any builds.

I did more testing and what worked for me was setting ElectronNET.Host's package.json to electron-updater 4.6.1 and electron ^16.0.3, running buildAll.cmd, running buildReleaseNuGetPackages.cmd, and using that new API package. The buildAll installs the CLI package. This was vsmike's suggestion.
Other steps I took before reaching this fix:
Tested my project in a clean virtual machine and this error still occurs, so it's not an environment issue.
My regular desktop that produces this error is running Windows 10, the VM runs Windows 11.
I have tried setting electron-updater in my project's package.json to 4.3.9. I have tried doing it in ElectronNET.Host's package.json and then running buildAll.cmd to install this new CLI version.
I've used buildReleaseNuGetPackages.cmd and used this new API NuGet package with my project.
I've deleted every node_modules and package-lock.json file in my project, then ran "npm install" or "npm install --production".
I've tried using the 13.5.1 versions of the CLI and API.
I've cleaned and rebuilt my solution.

rajeshs-logicmatter
Thank you ,
I am also facing this kind of issue but as you suggested the steps I followed them and everything working smoothly

Solution:

Install SDK .Net 5.0
Install ElectronNET.CLI --version 13.5.1
Command : dotnet tool install --global ElectronNET.CLI --version 13.5.1

That's it


Update to - reusable workflows

As discussed earlier today, please incorporate https://github.com/Azure/AzOps/issues/383 in this PR as well.

Please consider updating also the 'update' workflow - specifically on this line https://github.com/SomilGanguly/AzOps-Accelerator/blob/sogangul-AzOpsAccelerator-pipelineRevamp/.github/workflows/update.yml#L86
or we will miss out on the .templates folder. Similar to https://github.com/Azure/AzOps-Accelerator/pull/106/files I assume.

@Jefajers, please take a look at this from a doc perspective.

I took the liberty of incorporating the change described in https://github.com/Azure/AzOps/issues/384 for both the Actions and ADO pipelines.


Bug: Race / Nil pointer panic getSegmentWithUpperBoundary -> Flaky test

There seems to be a race-like condition which can be experienced through a flaky test that fails with something like:
panic: runtime error: invalid memory address or nil pointer dereference
[signal SIGSEGV: segmentation violation code=0x1 addr=0x0 pc=0x15ea6a5]

goroutine 52 [running]:
github.com/semi-technologies/weaviate/adapters/repos/db/lsmkv.(*segment).get(0x0, 0xc0008fe005, 0x4, 0x98967b, 0x15dfcd3, 0x15dec11, 0x15e361a, 0x15b6d6c, 0x15b638f)
        /Users/etiennedilocker/go/src/github.com/semi-technologies/weaviate/adapters/repos/db/lsmkv/segment_replace_strategy.go:23 +0x65
github.com/semi-technologies/weaviate/adapters/repos/db/lsmkv.(*SegmentGroup).getWithUpperSegmentBoundary(0xc0003af200, 0xc0008fe005, 0x4, 0x98967b, 0x11, 0x10b7122, 0x108cdb4, 0xc0008fefaa, 0x4, 0x1094bc0)
        /Users/etiennedilocker/go/src/github.com/semi-technologies/weaviate/adapters/repos/db/lsmkv/segment_group.go:164 +0x13f
github.com/semi-technologies/weaviate/adapters/repos/db/lsmkv.(*SegmentGroup).makeExistsOnLower.func1(0xc0008fe005, 0x4, 0x98967b, 0x1, 0x2cf5, 0x4)
        /Users/etiennedilocker/go/src/github.com/semi-technologies/weaviate/adapters/repos/db/lsmkv/segment_group.go:125 +0xe8
github.com/semi-technologies/weaviate/adapters/repos/db/lsmkv.(*segment).initCountNetAdditions.func1(0xc0008fe005, 0x4, 0x98967b, 0x2b00)
        /Users/etiennedilocker/go/src/github.com/semi-technologies/weaviate/adapters/repos/db/lsmkv/segment.go:163 +0x99
github.com/semi-technologies/weaviate/adapters/repos/db/lsmkv.(*bufferedKeyAndTombstoneExtractor).flushAndCallback(0xc0008cf200)
        /Users/etiennedilocker/go/src/github.com/semi-technologies/weaviate/adapters/repos/db/lsmkv/segment_key_and_tombstone_extractor.go:149 +0x123
github.com/semi-technologies/weaviate/adapters/repos/db/lsmkv.(*bufferedKeyAndTombstoneExtractor).do(0xc0008cf200)
        /Users/etiennedilocker/go/src/github.com/semi-technologies/weaviate/adapters/repos/db/lsmkv/segment_key_and_tombstone_extractor.go:68 +0x139
github.com/semi-technologies/weaviate/adapters/repos/db/lsmkv.(*segment).initCountNetAdditions(0xc0005c6000, 0xc00000e180, 0x16, 0x714e)
        /Users/etiennedilocker/go/src/github.com/semi-technologies/weaviate/adapters/repos/db/lsmkv/segment.go:180 +0x305
github.com/semi-technologies/weaviate/adapters/repos/db/lsmkv.newSegment(0xc0000ac570, 0x2e, 0x1f23d10, 0xc0004568c0, 0xc00000e180, 0xc0000ac59b, 0x1cb7ab5, 0x3)
        /Users/etiennedilocker/go/src/github.com/semi-technologies/weaviate/adapters/repos/db/lsmkv/segment.go:136 +0x891
github.com/semi-technologies/weaviate/adapters/repos/db/lsmkv.(*SegmentGroup).add(0xc0003af200, 0xc0000ac570, 0x2e, 0x0, 0x0)
        /Users/etiennedilocker/go/src/github.com/semi-technologies/weaviate/adapters/repos/db/lsmkv/segment_group.go:140 +0x15a
github.com/semi-technologies/weaviate/adapters/repos/db/lsmkv.(*Bucket).atomicallyAddDiskSegmentAndRemoveFlushing(0xc000407340, 0x0, 0x0)
        /Users/etiennedilocker/go/src/github.com/semi-technologies/weaviate/adapters/repos/db/lsmkv/bucket.go:577 +0x1ac
github.com/semi-technologies/weaviate/adapters/repos/db/lsmkv.(*Bucket).FlushAndSwitch(0xc000407340, 0x1, 0xc000000015)
        /Users/etiennedilocker/go/src/github.com/semi-technologies/weaviate/adapters/repos/db/lsmkv/bucket.go:555 +0x32f
github.com/semi-technologies/weaviate/adapters/repos/db/lsmkv.(*Bucket).initFlushCycle.func1(0xc000407340)
        /Users/etiennedilocker/go/src/github.com/semi-technologies/weaviate/adapters/repos/db/lsmkv/bucket.go:508 +0x5a5
created by github.com/semi-technologies/weaviate/adapters/repos/db/lsmkv.(*Bucket).initFlushCycle

I believe the underlying cause to be a race-like condition where the segment state changes while this method runs, thus leading to a nil-segment. Even though the segment access is thread-safe, thus not making this a classical race, there still seems to be a temporarily invalid state, probably from a missing lock.

The above theory was incorrect and the actual cause was in the shutdown method:

shutdown closes each disk segment
shutdown sets each segment to nil
shutdown did not previously change the slice, so as of now ig.segments is filled with nil, nil, nil, ...
the next flush (memtable is flushed after disk is closed) would try to read from ig.segments which has a non-zero length, but all contents are nil
simply setting the slice to nil fixes the problem


[Mention The Feature]: Add complete tutorial page for Stock Prediction using time series analysis in projects page website.

Is your feature request related to a problem? Please describe.
create a tutorial page explaining creation of advanced time series forecasting model for stock prediction. Adding features to dataset such as moving averages for 3 days back and in future to todays point and dividing date into parts to take day , month and year into account
Describe the solution you'd like.
No response
Describe alternatives you've considered.
No response
Additional context.
No response
Show us the magic with screenshots
No response
Checklist

[X] I have checked the Existing Issues
[X] I have read the Contributing Guidelines
[X] I want to work on this issue. (optional)

@DarshAgrawal14 Assigning this issue to you.
From now onwards i am gonna assign level 2 for these tutorials once, as there were instructions provided by organisers for Project Admins.
Make sure to complete this.

@vivekvardhan2810 ,is it going to be level 2 for all following tutorial pages?

from now @DarshAgrawal14 it's going to be level 2


⚠ Jenkins Plugins Artifactory has degraded performance

In 6ce7e51, Jenkins Plugins Artifactory (https://get.jenkins.io/plugins/artifactory/) experienced degraded performance:

HTTP code: 200
Response time: 1221 ms

Resolved: Jenkins Plugins Artifactory performance has improved in ba4b83a after 12 minutes.


Introducing SGLang Guru on Gurubase.io

Hello team,
I'm the maintainer of Anteon. We have created Gurubase.io with the mission of building a centralized, open-source tool-focused knowledge base. Essentially, each "guru" is equipped with custom knowledge to answer user questions based on collected data related to that tool.
I wanted to update you that I've manually added the SGLang Guru to Gurubase. SGLang Guru uses the data from this repo and data from the docs to answer questions by leveraging the LLM.
In this PR, I showcased the "SGLang Guru", which highlights that SGLang now has an AI assistant available to help users with their questions. Please let me know your thoughts on this contribution.
Additionally, if you want me to disable SGLang Guru in Gurubase, just let me know that's totally fine.

This looks very cool! Would you be interested in adding the guru for https://github.com/linkedin/Liger-Kernel too? I am the maintainer of liger-kernel.

Sure, but I can't find detailed documentation for Liger-Kernel. Are there any PDFs, websites, or YouTube videos I can use as data sources for it?

Are you on sglang slack https://join.slack.com/t/sgl-fru7574/shared_invite/zt-2ngly9muu-t37XiH87qvD~6rVBTkTEHw? We can chat there

Hey everyone, have you had a chance to check this out?

We have moved from readthedocs to https://sgl-project.github.io/. Can you update the docs link in https://gurubase.io/g/sglang? It still points to the old address, which is unavailable now.

Hey @merrymercy I updated it.

@kursataktas Thanks for the contribution. It is merged.

Hi everyone,
I’d like to update you on the release of the Maintainer Panel feature on Gurubase. With this panel, you can add, remove, or update data sources, as well as change the logo and more. You can find the details here.
In the near future, I’m planning to include analytics insights in this panel, such as the number of questions asked, the most frequently asked ones, and more. I’ll be sure to update this thread once it’s available. However, in case I miss it, I highly recommend joining our Discord channel to stay updated.
If you’d prefer that I don’t update this thread anymore, please let me know.


PDF JIS Z 8301: inner cover page styles

Source issue: https://github.com/metanorma/metanorma-jis/issues/9
Original PDF:

Generated PDF:

To do: remove space between paragraphs in the inner-cover-page section.

Done:


demo github repo?

you have mentioned repo link but It is is hard to figure out how you have implemented it, I'm trying to implement but it keeps failing

we will release the demo site for cms part.
we will achieve this project since filament3 support tabs now.
https://beta.filamentphp.com/docs/3.x/infolists/layout/tabs


nginx升级出错

make -f objs/Makefile
make[1]: Entering directory `/root/lnmp/src/nginx-1.13.0'
cd ../pcre-8.39 \
	&& if [ -f Makefile ]; then make distclean; fi \
	&& CC="cc" CFLAGS="-O2 -fomit-frame-pointer -pipe " \
	./configure --disable-shared  --enable-jit
/bin/sh: 1: cd: can't cd to ../pcre-8.39
make[1]: *** [../pcre-8.39/Makefile] Error 2
make[1]: Leaving directory `/root/lnmp/src/nginx-1.13.0'
make: *** [build] Error 2
Upgrade Nginx failed!

What Are You Doing?
	1. Upgrade Nginx/Tengine/OpenResty
	2. Upgrade MySQL/MariaDB/Percona
	3. Upgrade PHP
	4. Upgrade Redis
	5. Upgrade Memcached
	6. Upgrade phpMyAdmin
	q. Exit

Please input the correct option:

修改version.txt里的pcre_version为8.39就能升级了


Clean differences with Numba 0.51.2 we did not do

During merge with Numba 0.51.2 like #50 there could be changes that are not in upstream Numba and was not added by us.
This code could be searched by showing all diffs and see who are author of the code. It author is not from IntelPython then code should be reverted to upstream Numba code.

@PokhodenkoSA @reazulhoque already started some of the changes and has created the initial set of patches.


Error: Illegal character in opaque part at index 47

java -jar jardiff.jar file1.jar file2.jar
java.lang.IllegalArgumentException: Illegal character in opaque part at index 47: jar:file:file1.jar
        at java.net.URI.create(Unknown Source)
        at scala.tools.jardiff.IOUtil$.rootPath(IOUtil.scala:16)
        at scala.tools.jardiff.JarDiff.$anonfun$diff$1(JarDiff.scala:35)
        at scala.collection.immutable.List.foreach(List.scala:312)
        at scala.tools.jardiff.JarDiff.renderAndCommit$1(JarDiff.scala:34)
        at scala.tools.jardiff.JarDiff.$anonfun$diff$3(JarDiff.scala:54)
        at scala.collection.Iterator$$anon$9.next(Iterator.scala:554)
        at scala.collection.Iterator$GroupedIterator.takeDestructively(Iterator.scala:205)
        at scala.collection.Iterator$GroupedIterator.go(Iterator.scala:220)
        at scala.collection.Iterator$GroupedIterator.fill(Iterator.scala:257)
        at scala.collection.Iterator$GroupedIterator.hasNext(Iterator.scala:261)
        at scala.collection.IterableOnceOps.foreach(IterableOnce.scala:576)
        at scala.collection.IterableOnceOps.foreach$(IterableOnce.scala:574)
        at scala.collection.AbstractIterator.foreach(Iterator.scala:1196)
        at scala.tools.jardiff.JarDiff.diff(JarDiff.scala:55)
        at scala.tools.jardiff.Main$.run(Main.scala:84)
        at scala.tools.jardiff.Main$.main(Main.scala:20)
        at scala.tools.jardiff.Main.main(Main.scala)
Caused by: java.net.URISyntaxException: Illegal character in opaque part at index 47: jar:file:file1.jar
        at java.net.URI$Parser.fail(Unknown Source)
        at java.net.URI$Parser.checkChars(Unknown Source)
        at java.net.URI$Parser.parse(Unknown Source)
        at java.net.URI.<init>(Unknown Source)
        ... 18 more

How would I (or anyone else seeing this ticket) reproduce the problem on my own computer?

Sorry, I cannot give you the compare jars.

Illegal character in opaque part at index 47: jar:file:file1.jar

What character is at index 47 that could be causing the error?

@nebur43 it needn't be the jars you used there.  A minimised reproduction would be necessary, otherwise we can't action this ticket and we'll just have to close it.

Ok. I close it.

Actually, I think I found it: it's because of the space in the (original) file path.
I'll send a fix.

I also encountered this problem. By removing the space in the path of the JAR and it worked fine.

Thank you a lot for investigating this. This helped me fixing a completely unrelated error just because I found your issue here.


Update WindowsBuild.md

Be more explicit about the Visual Studio installer bootstrapper.  This was pointed out as being unclear by @slavapestov.

Replace this paragraph with a description of your changes and rationale. Provide links to external references/discussions if appropriate.

Resolves SR-NNNN.

@swift-ci please smoke test

CC: @slavapestov

@swift-ci please smoke test macOS platform

@swift-ci please smoke test Linux platform

@swift-ci please smoke test Linux platform


Configuration File

Describe the solution you'd like

How about adding a configuration file for WinUtil, that will only be created/kept if a checkbox "keep personalisation" or smth in option list is checked.
Kinda adding the feature to erase any winutil leftovers or keep some stuff to have custom themes/configuration/other leftovers to make winutil loadup even faster.

Cool idea, the removal of leftovers (preferences, which currently there's none, and log files) is a plus IMO, as some people don't like bloating their system, or just want any future user to know you used WinUtil (.. for whatever reason), and it's possible to implement it without much hassle.
Cool suggestion @MyDrift-user

Update: There actually are leftovers now. The theme and chocolatey settings are "saved" as a file under %localappdata%/winutil
It's still to be evaluated where and how we want to store persistent content in a manageable, centralized, standardized way. Ideas are Registry, config/ini file in localappdata


/bin/bash

I'm a newbie at decompiling, but that looks like it's attempting to read /bin/bash on 1.109.0
I was going to try reverse engineering further, but I'm stuck since the phone is jail broken. So getting ingress to run is the first step to attaching it to a disassembler.

Ryan what is a good way to get in touch with you?

Any chance you update this and compile it for the new path Qusic?

Any access to the whole /bin directory has already been blocked.

Weird something changed for sure. I know ryan was getting the OS Integrity check failed. They have recently started banning users since November for just having their phone Jailbroken so the detection must have changed in someway.

I'm a newbie at decompiling, but that looks like it's attempting to read /bin/bash on 1.109.0
I was going to try reverse engineering further, but I'm stuck since the phone is jail broken. So getting ingress to run is the first step to attaching it to a disassembler.


python3Packages.pytorch: add pybind11 to buildInputs

Description of changes
This allows for the headers to be available, so that
pytorch doesn't try and install pybind11 into its own
includes directory.
https://www.reddit.com/r/NixOS/comments/v3655n/collision_trying_to_write_my_first_flake/
Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
22.11 Release Notes (or backporting 22.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[ ] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 175962 run on x86_64-linux 1

8 packages marked as broken and skipped:

python310Packages.pytorch-metric-learning
python310Packages.pywick
python310Packages.shap
python310Packages.skorch
python39Packages.pytorch-metric-learning
python39Packages.pywick
python39Packages.shap
python39Packages.skorch

2 packages failed to build:

python310Packages.pymc
python39Packages.pymc

62 packages built:

openai (python39Packages.openai)
python310Packages.arviz
python310Packages.boxx
python310Packages.bpycv
python310Packages.ffcv
python310Packages.functorch
python310Packages.ignite
python310Packages.openai
python310Packages.pyro-ppl
python310Packages.pytorch (python310Packages.pytorchWithoutCuda)
python310Packages.pytorch-lightning
python310Packages.pytorch-pfn-extras
python310Packages.pytorchWithCuda
python310Packages.qiskit
python310Packages.qiskit-machine-learning
python310Packages.rising
python310Packages.slicer
python310Packages.spacy
python310Packages.spacy-loggers
python310Packages.spacy-transformers
python310Packages.stanza
python310Packages.tensorboardx
python310Packages.test-tube
python310Packages.textacy
python310Packages.torch-tb-profiler
python310Packages.torchgpipe
python310Packages.torchinfo
python310Packages.torchmetrics
python310Packages.torchvision
python310Packages.wandb
python310Packages.zcs
python39Packages.arviz
python39Packages.boxx
python39Packages.bpycv
python39Packages.dalle-mini
python39Packages.elegy
python39Packages.ffcv
python39Packages.functorch
python39Packages.ignite
python39Packages.pyro-ppl
python39Packages.pytorch (python39Packages.pytorchWithoutCuda)
python39Packages.pytorch-lightning
python39Packages.pytorch-pfn-extras
python39Packages.pytorchWithCuda
python39Packages.qiskit
python39Packages.qiskit-machine-learning
python39Packages.rising
python39Packages.slicer
python39Packages.spacy
python39Packages.spacy-loggers
python39Packages.spacy-transformers
python39Packages.stanza
python39Packages.tensorboardx
python39Packages.test-tube
python39Packages.textacy
python39Packages.torch-tb-profiler
python39Packages.torchgpipe
python39Packages.torchinfo
python39Packages.torchmetrics
python39Packages.torchvision
python39Packages.wandb
python39Packages.zcs

the pymc failures are not related
https://github.com/NixOS/nixpkgs/pull/175962

8 packages marked as broken and skipped:
python310Packages.pytorch-metric-learning python310Packages.pywick python310Packages.shap python310Packages.skorch python39Packages.pytorch-metric-learning python39Packages.pywick python39Packages.shap python39Packages.skorch

2 packages failed to build:
python310Packages.pymc python39Packages.pymc

62 packages built:
openai python310Packages.arviz python310Packages.boxx python310Packages.bpycv python310Packages.ffcv python310Packages.functorch python310Packages.ignite python310Packages.openai python310Packages.pyro-ppl python310Packages.pytorch python310Packages.pytorch-lightning python310Packages.pytorch-pfn-extras python310Packages.pytorchWithCuda python310Packages.qiskit python310Packages.qiskit-machine-learning python310Packages.rising python310Packages.slicer python310Packages.spacy python310Packages.spacy-loggers python310Packages.spacy-transformers python310Packages.stanza python310Packages.tensorboardx python310Packages.test-tube python310Packages.textacy python310Packages.torch-tb-profiler python310Packages.torchgpipe python310Packages.torchinfo python310Packages.torchmetrics python310Packages.torchvision python310Packages.wandb python310Packages.zcs python39Packages.arviz python39Packages.boxx python39Packages.bpycv python39Packages.dalle-mini python39Packages.elegy python39Packages.ffcv python39Packages.functorch python39Packages.ignite python39Packages.pyro-ppl python39Packages.pytorch python39Packages.pytorch-lightning python39Packages.pytorch-pfn-extras python39Packages.pytorchWithCuda python39Packages.qiskit python39Packages.qiskit-machine-learning python39Packages.rising python39Packages.slicer python39Packages.spacy python39Packages.spacy-loggers python39Packages.spacy-transformers python39Packages.stanza python39Packages.tensorboardx python39Packages.test-tube python39Packages.textacy python39Packages.torch-tb-profiler python39Packages.torchgpipe python39Packages.torchinfo python39Packages.torchmetrics python39Packages.torchvision python39Packages.wandb python39Packages.zcs

Yeah, those seem to be caused by
error: hash mismatch in fixed-output derivation '/nix/store/m604ai7h3qn0f973drm3bjkpl0d1122k-source.drv':
         specified: sha256-NJxklOpIbSbi/SB/rafBNllpnNb1yWLVpyB2f/U0i78=
            got:    sha256-vIDyqYj9HuH5gVcc6KPkB2NNTAggiPHyAR2fCXOQzsE=
error: 1 dependencies of derivation '/nix/store/whd26jmzyy9pp21pf2ar0zm89v068f9g-python3.10-aesara-2.7.2.drv' failed to build
error: 1 dependencies of derivation '/nix/store/ix2a43rb86bqpmmy3wvx2kf12lgj72fx-python3.9-aesara-2.7.2.drv' failed to build
error: 1 dependencies of derivation '/nix/store/h0cmjf5ai4xjicw65injlf9aa62zibpz-python3.10-aeppl-0.0.31.drv' failed to build
error: 1 dependencies of derivation '/nix/store/7n45ig3jsyh57l09zdrd2y4scigh80p6-python3.9-aeppl-0.0.31.drv' failed to build
error: 2 dependencies of derivation '/nix/store/241ym0x696c5djb6g32j5w3ss2d0i369-python3.10-pymc-4.0.0.drv' failed to build
error: 2 dependencies of derivation '/nix/store/mghlxad1xcwli8jjpk16i5al351r39gl-python3.9-pymc-4.0.0.drv' failed to build


There are two logo urls for an applicaion- update the descriptions according to the proper usage

Describe the issue:

How to reproduce:
One under application general settings, and one under branding.
Expected behavior:
Remove if there are redundant configs
update the descriptions properly
How will this fit when application-level branding comes into the picture?

This issue is being closed due to extended inactivity. Please feel free to reopen it if further attention is needed. Thank you for helping us keep the issue list relevant and focused!