add support for uploading distribution files

i.e. a new option:

zip: zip and upload dist using files array from package.json

caveats:

may require a build step, as some files in files may be .gitignore'd
may require force adding some files and pushing up a temporary release branch to ensure built files are included in the release but kept out of master

This supports two use cases:

A package needs to be distributed with a build but you don't want to check that build into source control over and over (solution: automate this).
A release needs to include a zip of the repo (solution: automate that).

This is something @kneemer & @paulcpederson want in particular (PRs welcome :smile_cat:)

I could try putting this together if I can see someone do it all by hand first.

@ngoldman @bcomnes I think this is almost 2 options:

Do you want to create a zip and attach it to the release? --zip -z
Do you want to only include files listed in the files key in your distribution? --files -f maybe...

Also, as far as a build step, would it be safe to just run npm prebulish and tell people to make sure all of their compilation happens there?

@paulcpederson what is your ideal workflow in your specific situation?

Here's a situation I'm in with several different libraries:

I'm writing code that needs to be compiled (sass -> css, es6 -> es5, etc...)
Committing the compiled files is a pain because it leads to lots of merge conflicts (especially in minified files) and the pr's are really noisy because every time you change anything it is reported in two or three places.
Locally there is some task runner (npm, grunt, gulp, etc) that compiles these files on the fly

When it comes to releases, not committing the built file is problematic because then users of your library can't just npm install and instantly have everything ready to go.

The ideal workflow is to have the best of both worlds:

Don't add compiled files to version control
run gh-release -f and have that automatically include, tag, and push the built files

A workflow for this would be the following (shell version plagarized from the leaflet build script):
# Get current version (from package.json)
 VERSION=$(node --eval "console.log(require('./package.json').version);")

# Checkout a new temporary branch (probably named gh-release or some such)
git checkout -b gh-release

# Build and test (probably optional for this module, though maybe configurable)
npm test
npm run prepublish

# Force add files from package.json 'files' key
git add dist/whatever.js dist/whatever.min.js -f

# Commit changes with a versioned commit message
git commit -m "build $VERSION"

# Tag with the version (eg v0.0.3)
git tag $VERSION

# Push the tags to GitHub
git push --tags git@github.com:yourname/yourrepo.git $VERSION

# this is where you would npm publish (not sure we want to do that..)

# Cleanup
git checkout master
git branch -D build

Ahhhrg.  I wrote a response and didn't save it.
To specify this a bit more:

When packaging a release with build results, gh-release will only (at least to start with) release on a tag branch like in the leaflet example.
gh-release will take care of creating that branch and tag and pushing to origin
gh-release will test to see the build step is completed before it is run (how?)
How will gh-release know which files to force add?  flags, gh-release options in package.json?

I need to review this article again, but I have it noted as describing a sane packaging strategy that include build steps: https://medium.com/@brianleroux/es6-modules-amd-and-commonjs-c1acefbe6fc0
I'll try and extract possibly useful information from it soon.

After reading @ngoldman 's goals for this project and talking to him a bit, I almost wonder if all this isn't a bit much for gh-release. Maybe this is a separate module? Does this violate the 'do one thing and do it well' philosophy? I don't know. Do other people even need this or am I the weirdest person in the world?
:confused:

@paulcpederson since github releases can include files it makes sense for gh-release to cover that too. However maybe supporting complex build steps for front-end packages is out of the scope of the project.
I think if we can support the zip+upload step, we're already offering something useful enough to meet the latter use case halfway.
A second package could use gh-release programmatically and handle the use case specific to front-end packages that want distribution files in their releases but not in version control.

The build step should probably be handled by something else, or gh-release is a step in some kind of in a release build.
I'm thinking there would be 3 scenarios:

A regular release (like how it works now).  The release download is just an archive of the source code.
A built release.  An array of files is provided to gh-release as an option.  Gh-release creates a tag-branch and force commits these files to it.  The release download is a snapshot of the source plus the built files.
If some kind of zip option is also set along side the file release array in option two, take only those files and zip/tar them and use that for the release download.

What do you all think?

@ngoldman right, that makes sense.
So there could be another module that depends on gh-release that essentially kicks off the branching gymnastics and then once everything is tagged and push correctly, it just calls gh-release. I think that makes sense.
If that's the case, then we could definitely add --zip to gh-release. I think we should still use the files array in package.json for the content of the zip, because that would make the zip mirror what will be distributed on npm.
The other options for the zip are:

Just zip the entire project
Zip a specific folder
Zip everything in the files array
Zip everything except what is precluded in the .npmignore

++ for mirroring what's distributed on npm by default (regardless of scenario). we can start with that then add options as necessary.

I think npm also automatically includes LICENSE, CHANGELOG, and README as well, which is probably something we should figure out and do as well.

I bet we can just find the code or module npm is using for this and use that.

lol, my browser is currently at https://github.com/npm/npm/blob/master/lib/publish.js

Wow, this is very interesting: https://github.com/npm/npm/blob/448efd0eaa6f97af0889bf47efc543a1ea2f8d7e/doc/misc/npm-developers.md#keeping-files-out-of-your-package
Apparently adding files that are gitignored to your npm package is as easy as adding a blank .npmignore file. Very interesting.

@paulcpederson how is https://github.com/paulcpederson/unignored coming along? I'm guessing the point is to reliably get the files npm would publish as an array or something like that, possibly for use in a gh-release-dependent project?

I think the consensus is gh-release should support adding assets to a release (reference here: https://developer.github.com/v3/repos/releases/#upload-a-release-asset, tested here: https://github.com/bcomnes/gh-release-test/releases/tag/v1.0.4) and leave the branching, building, committing, and pushing of a release build to another project. I'll try to get to this soon (PRs welcome).

I'm also realizing part of this use case is totally unnecessary since github releases automatically generate a zip (https://github.com/ngoldman/gh-release/archive/v1.1.2.zip)  and tarball (https://github.com/ngoldman/gh-release/archive/v1.1.2.tar.gz) which are listed right under the release.

Being able to upload one or more release assets is still a valid feature request though.

Yeah, I agree it doesn't make sense to zip the whole repo. If we add an attachment capability it should be basically a dist folder.
Unignored is fully functional. I have a theory about how all this should work, but it is far outside the scope of gh-release. Maybe in versioneer?

@paulcpederson there are issues, check https://github.com/paulcpederson/unignored/issues/1 & https://github.com/paulcpederson/unignored/issues/2

Cool. =>  latest release ... :grimacing:

Here is another example of a build release process: https://github.com/maxogden/monu/blob/master/publish-release.js

Looks like it was turned into its own module https://github.com/remixz/publish-release

hmm dang

he's doing a lot of stuff very similarly to us, we had worked together :(

yeah, that doesn't really do anything that gh-release doesn't do other than uploading assets.

@paulcpederson there's some differences in methodology and publish-release's programmatic version is emitting events. It's also doing the $EDITOR thing we were discussing.
@remixz :wave:

I think it came from helping out @maxogden with github.com/maxogden/monu which requires a build with the release for an app kind of thing

@ngoldman hi! dang, i wish we did work together. you're totally correct in that it came from monu, max wanted to use the publishing workflow on some other projects, so i made it into a module.
i'd say we could try to combine them, but gh-release differs from publish-release a bit, especially with the strong defaults on gh-release. however, i'm totally open to PRing the asset uploading from publish-release, so at least both modules have the functionality :smile:

"you can't share opinions, but you can share submodules" - me, just now
On Tue, Mar 31, 2015 at 6:07 PM, Zach Bruggeman<EMAIL_ADDRESS>wrote:

@ngoldman https://github.com/ngoldman hi! dang, i wish we did work
together. you're totally correct in that it came from monu, max wanted to
use the publishing workflow on some other projects, so i made it into a
module.
i'd say we could try to combine them, but gh-release differs from
publish-release a bit, especially with the strong defaults on gh-release.
however, i'm totally open to PRing the asset uploading from
publish-release, so at least both modules have the functionality [image:
:smile:]
—
Reply to this email directly or view it on GitHub
https://github.com/ngoldman/gh-release/issues/1#issuecomment-88300564.

@remixz++ @maxogden++ agreed!

We've got support for asset uploads is 2.0.0:
https://github.com/ngoldman/gh-release/releases/tag/v2.0.0
Thanks to @paulcpederson :+1:
submodule: https://github.com/paulcpederson/gh-release-assets
should be usable by https://github.com/remixz/publish-release


Update usage.mdx

Some suggested changes that address accessibility and other considerations, to be discussed at the Guild call.

@aagonzales Agreed. I wasn't sure if this was a welcome change, and would still like to discuss. As well, there are a number of other potential changes to make to the Style page, which I'd like to go over on the Carbon call.

@laurenmrice a reminder that the image of Drag and drop file upload, which currently reads "Drag and drop files here or upload" needs to applied across both Usage and Style to read:

Drag and drop files here or click to upload"

This matches the implementation.

Just thought I'd capture what I put in slack. The wording and style was based what is already used on the Carbon website (such as modal)


please can you make a tutorial for this? This is  will be helpful for all dotnet developer in bangladesh

please can you make a tutorial for this? This is  will be helpful for all dotnet developer in bangladesh

Please make a video tutorial on  it for web api and mvc

hello @zakariabijoy, I am not sure when I will have free time for a tutorial. In the meantime, check out the documentation from SSLCommerz => https://developer.sslcommerz.com/


Update Non-Administrative Install Guide

The Chocolatey non-administrative installed guide may need updating based on newer versions of Chocolatey. We should look at the guidance we provide and update it if necessary.

Basically it says to use "C:\ProgramData\ chocoportable" not the default install directory of Chocolatey - I know it was stated it was useless, but it is not. Just an FYI when you get to this rabbit hole.


Ios intercept request and replace with local assets

Environment
Flutter version:
Plugin version:
Android version:
iOS version:
Xcode version:
Device information:
Description
What you'd like to happen:
Alternatives you've considered:
Images:

See https://github.com/pichillilorenzo/flutter_inappwebview/issues/600


KOGITO-9566: Dynamic data input schema generation plus some improvements

Hey, I just made a Pull Request!

:heavy_check_mark: Checklist

[ ] A changeset describing the change and affected packages. (more info)
[ ] Added or updated documentation
[ ] Tests for new functionality and regression tests for bug fixes
[ ] Screenshots attached (for UI changes)
[ ] All your commits have a Signed-off-by line in the message. (more info)

I just missed this epic piece of work :cry:


python3Packages.aioesphomeapi: 9.1.4 -> 9.1.5

Motivation for this change
https://github.com/esphome/aioesphomeapi/releases/tag/v9.1.5
Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.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

[x] Fits CONTRIBUTING.md.

@ofBorg build home-assistant


Update index.js

Edited some Device Info descriptions.

First ever pull request! Fingers crossed!

Thanks for the PR


How to use with python3

I have written a module in python3. I can see the plugin can import from only from python 2 folder. I have trouble setting the python path.  Any help will be appreciated.

Hi! Python 3 is now supported. Check out README. All you need to do is make sure you have python3 headers installed (eg. apt install python3-dev under Ubuntu) and compile with make PYTHON_VERSION=3.6 (or whatever version you use).
I hope it works fine for you. Enjoy!


1.17.x breaks elasticsearch module

We're currently trying to leverage testcontainers-java with the elasticsearch module and noticed that the most recent version 1.17.1 breaks our use of the elasticsearch module. I think it's due to this commit https://github.com/testcontainers/testcontainers-java/commit/c7449edda49ff780921067c4a03014ce68b2aa41
We're currently trying to leverage elasticsearch version 7.4.2 but our containers fail to start with the error
org.testcontainers.containers.ContainerLaunchException: Container startup failed
Caused by: org.rnorth.ducttape.RetryCountExceededException: Retry limit hit with exception
Caused by: org.testcontainers.containers.ContainerLaunchException: Could not create/start container
Caused by: com.github.dockerjava.api.exception.NotFoundException:
Status 404: {"message":"Could not find the file /usr/share/elasticsearch/config/certs/http_ca.crt in container df2a5216dc041fb543e4d156a05ec268f1db4cadb5efef170200b841d712b2a9"}

Please advise?

Hey @spinscale, do you have any clue what could be happening?

This looks, as if the check for version 8 is somehow returning that this is indeed a version 8 running...
This check seems to return the version is version 8...
new ComparableVersion(dockerImageName.getVersionPart()).isGreaterThanOrEqualTo("8.0.0");

@brentryan can you share some code how you are initializing the container and what version you are using?
The following snippet worked for me, but maybe there is more to it:
        try (ElasticsearchContainer container = new ElasticsearchContainer(DockerImageName.parse("docker.elastic.co/elasticsearch/elasticsearch").withTag("7.4.2"))) {
            // Start the container. This step might take some time...
            container.start();
        }

It looks like this is because the underlying image has a tagged version of "latest" which then triggers this code which fails.
The code we're using to initialize our container is loading some elasticsearch plugins so we have to do it slightly differently. Let us know if there's a better way, but here's what the init code looks like that fails:
                ImageFromDockerfile dockerImage = new ImageFromDockerfile()
                        .withDockerfileFromBuilder(builder -> {
                            builder.from(getDockerImageName() + ":" + getDockerImageVersion());
                            builder.run(getEsPluginName());
                            if (getEsPluginPartsName().length > 0) {
                                builder.run(getEsPluginPartsName());
                            }
                            builder.build();
                        });

                container = new ElasticsearchContainer(
                        DockerImageName.parse(dockerImage.get())
                                .asCompatibleSubstituteFor(DEFAULT_ELASTICSEARCH_DEFAULT_IMAGE)
                );

can you try the following
try (ElasticsearchContainer container = new ElasticsearchContainer("docker.elastic.co/elasticsearch/elasticsearch:8.1.0"){
            @Override
            protected void containerIsStarted(InspectContainerResponse containerInfo) {
            }
        }) {

and see if that works?

This seems indeed the culprit as  new ComparableVersion("latest").isGreaterThanOrEqualTo("8.0.0") returns true.
It might make sense to just log a warning, if the CA cert file is not available instead of throwing a hard exception, as the version can be anything with custom images. I'll open a PR.

is there a better way to load custom plugins and use custom images like this?
It seems like I'd want ElasticsearchContainer to have a constructor that takes a type ImageFromDockerfile and does the right thing.


Update PPP

Codecov Report
Base: 73.49% // Head: 73.49% // No change to project coverage :thumbsup:

Coverage data is based on head (e769b6b) compared to base (160d60f).
Patch has no changes to coverable lines.

Additional details and impacted files
@@            Coverage Diff            @@
##             master     #358   +/-   ##
=========================================
  Coverage     73.49%   73.49%
  Complexity     1415     1415
=========================================
  Files           191      191
  Lines          4407     4407
  Branches        532      532
=========================================
  Hits           3239     3239
  Misses         1005     1005
  Partials        163      163

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 at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


VS2017 Error A server with the specified hostname could not be found while adding an Apple Account

Trying to add an Apple Developer account [Tools > Options > Xamarin > Apple Accounts] to Visual Studio Professional 2017 Version 15.9.16
will result in a dialog saying:
Error adding account
There was an error while trying to log in: A server with the specified hostname could not be found.
At the paired mac's Keychain Access app, we can see in the Passwords section every attempt to try to add the account with a corresponding entry like:
<EMAIL_ADDRESS>Please advise

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 6d17e2db-437f-d45f-f78f-f4b32a40c3f3
Version Independent ID: 44ff0294-e19c-a926-1aa2-338c3b61371a
Content: Apple Account Management - Xamarin
Content Source: docs/cross-platform/macios/apple-account-management.md
Product: xamarin
Technology: xamarin-crossplatform
GitHub Login: @davidortinau
Microsoft Alias: daortin

Hi @fperez2511 this sounds like something where more system information and logs would be helpful in diagnosing.
Please use the Report a problem feature in Visual Studio, to include the additional information that might help us identify the problem. Thanks!

Also experiencing this issue.

I also have this same issue, anyone have any solutions?

upgrading to vs 2019 worked for me.

The same problem. No option to upgrade to VS2019.

Same problem. And no intention to upgrade to VS2019.


Approve onboarding via signature

It should no longer be needed to make two transactions for a user to onboard. Replace the approval transaction with a signature, which will be verified when the user performs the onboarding transaction.

Will be merged in with #152


Introduce separate client for Oracle

Description:
Currently, interacting with a Oracle database has to be done with JDBC client. Need to provide a separate client for Oracle similar to MySQL, H2, along with the refactoring done to provide a richer feature set focusing on vendor-specific areas

Refer the comment in https://github.com/ballerina-platform/ballerina-lang/issues/13818


Docker process failing at task 'Clean previous docker-py package if installing docker'

Hi, I'm running into the following problem:
TASK [angstwad.docker_ubuntu : Clean previous docker-py package if installing docker.] ******************************************************************************
fatal: [host]: FAILED! => {"changed": false, "cmd": ["/usr/local/bin/pip", "uninstall", "-y", "docker-py"], "msg": "\n:stderr: Traceback (most recent call last):\n  File \"/usr/local/bin/pip\", line 7, in <module>\n    from pip._internal.cli.main import main\n  File \"/usr/local/lib/python2.7/dist-packages/pip/__init__.py\", line 1, in <module>\n    from typing import List, Optional\nImportError: No module named typing\n"}

From what I can see, people have previously have issued at this task that have been resolved. Unsure what to do, as I have typing installed (see pip list below), and not sure what else the issue is? Sorry if this is a obvious question, I'm pretty new to all of this.
Package      Version
------------ -------
ansible      4.0.0
ansible-core 2.11.0
cffi         1.14.5
cryptography 3.4.7
Jinja2       3.0.1
jmespath     0.10.0
MarkupSafe   2.0.1
packaging    20.9
pip          21.1.1
pycparser    2.20
pyparsing    2.4.7
PyYAML       5.4.1
resolvelib   0.5.4
setuptools   49.2.1
typing       <IP_ADDRESS>

I thought I'd got past this - set my interpreter_python = python3, and it managed to get past this task, but then failed on the next. I reran it (without changing anything) and it's now failed on the task above again, with the error:
 fatal: []: FAILED! => {"changed": false, "cmd": ["/usr/local/bin/pip3", "uninstall", "-y", "docker-py"],
"msg": "\n:stderr: Traceback (most recent call last):\n
File \"/usr/local/bin/pip3\", line 7, in <module>\n    from pip._internal.cli.main import main\n
File \"/usr/local/lib/python3.5/dist-packages/pip/_internal/cli/main.py\", line 58\n    sys.stderr.write(f\"ERROR: {exc}\")\n                                   ^\nSyntaxError: invalid syntax\n"}

See #240


refactor(IDRegistry): audit according to solcurity

see: https://github.com/transmissions11/solcurity
addresses all identified issues in the IDRegistry contract

coverage should be fixed in the next release: https://github.com/foundry-rs/foundry/issues/3072


UWP and CS 8.0

Is there a way to clarify whether CS 8.0 can work with UWP projects?  I have tested it using the "Configure multiple projects" settings of this document.  Seems to work well.
[Enter feedback here]

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 5694e62b-c756-1f62-4014-2ceafa54c011
Version Independent ID: 3f01c378-22e4-197c-a862-ab4a924506c7
Content: C# language versioning - C# Guide
Content Source: docs/csharp/language-reference/configure-language-version.md
Product: dotnet-csharp
Technology: csharp-language-reference
GitHub Login: @BillWagner
Microsoft Alias: wiwagn

Thanks for noting this @keithnicholson We'll look into it.
@adegeo Do you know what TFMs are allowed in UWP apps?

According to the TFM doc, uap10.0 is the UWP TFM that targets .NET 5 and I would assume uses the same version of C# that .NET 5 does. But really we would need someone from the UWP doc team to answer that.

@mcleanbyron Perhaps you can answer this question.

I'm following up with some engineers on the Windows developer platform team, and will reply back as soon as I can with some details.

Here's the info I received from some engineers in the Windows developer experience team:

UWP supports .NET Standard 2 and C# 7.3 by default. You can use the  element in your project file to change to a later version of C# (as demonstrated in Sergio's blog), but there is no guarantee that all new language features will work properly. In particular, you should expect limitations when the newer language feature requires newer .NET runtime support.
As for TFMs supported in UWP projects, UWP apps use .NET Native and support .NET Standard 2; they do not support .NET 5. Apparently the 'uap10.0' TFM is an older moniker for earlier versions of .NET Native. Current UWP projects use the simpler 'uap' TFM (in the  element in the project file).

I hope this helps.

@keithnicholson This has been a great discussion. Thanks for sparking it.
I like how Sergio points to both the potential benefits, and where the pitfalls are in this unsupported scenario.
I'll close this as an answered question.


Dropdown component - not making any api call when render form

I add a dropdown control with rest request pointing to my local api http://localhost:3000/dropdown-values and on form render it throw this error
vue.runtime.esm.js?2b0e:1888 TypeError: [Dropdown] Rest-API Endpoint must be valid http/https URL.

Hello @dhimiterbundo ,
Please provide the configuration of the dropdown datasource. Also the result-data of your API. Thanks

hello @sethsandaru
this is the dropdown configuration:
        "control-ba824229-0571-4eff-b51c-19e7005764f0": {
          "uniqueId": "control-ba824229-0571-4eff-b51c-19e7005764f0",
          "type": "dropDown",
          "name": "",
          "label": "Dropdown",
          "subLabel": "",
          "isShowLabel": true,
          "placeholderText": "",
          "containerClass": "col-md-4",
          "additionalContainerClass": "",
          "additionalFieldClass": "",
          "additionalLabelClass": "",
          "defaultValue": "",
          "validations": [],
          "dataMode": "api",
          "multiple": false,
          "items": [],
          "apiURL": "http://localhost:3000/dropdown-values",
          "apiTextKey": "label",
          "apiValueKey": "value"
        }
      },

And this is the response from api call:
[
  {
    "value": "option 1",
    "label": "1"
  },
  {
    "value": "option 2",
    "label": "2"
  },
  {
    "value": "option 3",
    "label": "3"
  },
  {
    "value": "option 4",
    "label": "4"
  },
  {
    "value": "option 5",
    "label": "5"
  },
  {
    "value": "option 6",
    "label": "6"
  }
]

Hello @dhimiterbundo ,
Thanks for the full detail. I'll look into this!

Okay, 100% is a bug from the logic that I made. More detail here:
https://github.com/sethsandaru/vue-form-builder/commit/f2a0301e19eb42ca106a6499dce30c4f2e68b64b#diff-9fc44001afef0c8e62cb509fec58429bL85-L88
The fixes will come up soon, it will be on @2.0.1. Thanks. I'll let you know when it's released.

@sethsandaru Trying to make api request to 3rd party api https://support.travelpayouts.com/hc/en-us/articles/203956163-Travel-insights-with-Travelpayouts-Data-API#data_of_city
In form config:
Value Key of your Object : name
Text Key of your Object : name
Getting error:
Access to fetch at 'http://api.travelpayouts.com/data/en/cities.json?token=7ef5867b5c424588c63ac72fd48f1221' from origin 'http://localhost:3000' has been blocked by CORS policy: No 'Access-Control-Allow-Origin' header is present on the requested resource. If an opaque response serves your needs, set the request's mode to 'no-cors' to fetch the resource with CORS disabled.

Along with:
TypeError: Cannot read property 'containerClass' of undefined

and
Invalid prop: type check failed for prop "control". Expected Object, got Undefined

and
app.js:146148 [DROPDOWN-Control-control-a84467ce-7bcd-47de-b3fa-ee637f2d23a9] Request API to get data failed. TypeError: Failed to fetch


Update demo to use CodeMirror

Change the text entry to use CodeMirror so we can see lint errors inline. Code donated from here:
https://github.com/angelozerr/codemirror-lint-eslint
We just need to integrate it.

It looks like it should be possible to do with Orion as well, They have Annotations that would do exactly the same thing. Unfortunately their documentation is not really great. @mamacdon would you be able to help us with this? As far as I understand it should be possible to add annotations through editor.getAnnotationsModel().addAnnotation() but I don't fully understand what goes into it, and what to do with it later. I'm not sure if you know anything about it.

Hey guys. There's an easier way: editor.showProblems(Problem[]) where each Problem looks like this.
Here's a fix that worked for me: https://github.com/mamacdon/eslint.github.io/commit/fc0670351a0dcdbb88233f3d987e226d46045b31
It's not perfect:

I set the end offset for each annotation to start + 1. Ideally you'd want to show the annotations on the actual source range of the relevant AST node/token that was flagged by the rule -- that's what we do in the full-blown Orion product. The eslint.verify() API doesn't seem to expose detailed range info, but you would know better than I.
The Orion editor doesn't support displaying annotations at EOL+1 (like Sublime Text has), so "trailing whitespace" errors get dropped.
"Missing newline at eof" is in a weird spot.

Also, the Orion jsdoc in this area is poor, I'll try to improve it. Thanks for pointing it out.

Awesome! Thanks for the help @mamacdon I'll create a pull request for this.


 They should be recompiled and deployed.
I have done this for myself, but to my own docker images, not the myetherwallet docker images.

Thanks recompiling them now!

Cool!


Inheriting from core error classes in C output incorrect name

It appear that YARD isn't resolving the names of core error classes when the classes are defined in C source.
I'm guessing there needs to be a a lookup table to map the built in error classes to their real names.
Not sure where that's best done though. (Ideas?)
I see that the error class ends up as a proxy:

Is it perhaps possible to pre-populate YARD with definitions of the C error classes?
I see the classes are defined in error.c in Ruby's source. https://github.com/ruby/ruby/blob/930e637c0e2ffd6fcb46205d473b6f4c05607471/error.c#L2478
Maybe a tool to scrape this file and periodically update the lookup table? Not that I'd expect this to update often.
Steps to reproduce
https://github.com/thomthom/yard/commit/e064775e1f2dba0b3c6c96a5959958cf6b6d99b8
    # spec/parser/c_parser_spec.rb

    describe "Class inherited from core error class" do
      it "resolves correct name" do
        parse <<-eof
          void
          Init_Mask(void)
          {
              rb_cMyError = rb_define_class("MyError", rb_eArgError);
          }
        eof
        klass = Registry.at('MyError')
        expect(klass.name).to eq :MyError
        expect(klass.title).to eq 'MyError'
        expect(klass.superclass.name).to eq :ArgumentError
        expect(klass.superclass.title).to eq 'ArgumentError'
      end
    end

Actual Output
rb_eArgError ends up as ArgError instead of ArgumentError.
Can be seen in production here: http://ruby.sketchup.com/Layout/LockedEntityError.html

Expected Output
[Describe what you expected to happen.]
Environment details:

OS: Windows 10 (1905)
Ruby version (ruby -v): ruby 2.5.3p105 (2018-10-18 revision 65156) [x64-mingw32]
YARD version (yard -v): yard 0.9.20

I have read the Contributing Guide.

I prototyped a way to address this:
YARD::Handlers::C::Base - the #namespace_for_variable method could lookup a map of names.
For our yard-sketchup template I monkey patched this solution:
https://github.com/SketchUp/sketchup-yard-template/blob/dev-stubs-autoloader/lib/yard-sketchup/patches/c_base_handler.rb
The list was made by scraping the Ruby C source with a small script:
https://github.com/SketchUp/sketchup-yard-template/blob/dev-stubs-autoloader/tools/update_error_map.rb
Would such a solution be acceptable? Adding a name lookup to #namespace_for_variable?

FYI I don't seem to have access to those links.

Oops. I linked to the dev branch which was later merged into master and then deleted. I've updated the links to the master branch.
The script will only scrape the error.c in the Ruby repo.
Could wire it up to a Rake task if that's of any use.
The Ruby branch namings looks consistent enough that the script could potentially accept a Ruby version number and then try to fetch from a specific branch. But maybe its enough to simply scrape from head on master?

You may want to scope this lookup to exception classes only,
Not sure if you can determine that from the parser.... ? Or, do you mean filtering rb_e* prefix?


Evaluation failed, issues with egl_renderer (?)

Good day,
when finishing the training of GDRNPP I am encountering an error with the egl-renderer. It throws the info, that a .txt flies does not exist (gdrnpp_bop2022/lib/egl_renderer/robotPose/panda_arm_models/models.txt), followed by an info, that the pose error could not be determined. Then, it goes through two tests (both called "ape-iter0_lmo-test"), one per objects "mspd_5:50" and "mssd_0.050:0.500"
, which is followed by another runtime error specifying the output folder "RuntimeError: output/gdrn/lmoPbrSO/convnext_AugCosyAAEGray_DMask_amodalClipBox_lmo/ape/inference_/lmo_bop_test/ape-iter0_lmo-test/error:vsd_ntop:1_delta:15.000_tau:0.050 does not exist."
I do not know, how to handle this error, any tips? Am I missing something from the bop_toolkit perhaps? (trying "cpp" as render option in the lmo config file produces a similar issue, where the module bop_renderer was not found)
Kind regards,

Check if bop_renderer is rightly compiled.

Thanks for the swift reply. I reinstalled the bop_toolkit and compiled the bop_renderer, too. And, this seems to have fixed the renderer issue.
The other issue prevails, though. It has something to do with the "vsd" evaluation metric, since the corresponding folder (as detailed in my original post) is not existent in the output folder after training. I did not change anything else in the main training script, except reducing the number of folders processed in the dataset as well as the batch size and epochs, since I wanted to test the pipeline for errors, first.

I was able to find the issue, I linked the bop_renderer in a different file than in lib/physixd/config.py

Hello @shanice-l ,
I have an error that is similar to what @beastGrendel shared above. When I load the pre-trained weights for LMO dataset and try to do testing for Pose Estimation, then I am getting this error .
RuntimeError: output/gdrn/lmo_pbr/convnext_a6_AugCosyAAEGray_BG05_mlL1_DMask_amodalClipBox_classAware_lmo/inference_model_final_wo_optim/lmo_bop_test/convnext-a6-AugCosyAAEGray-BG05-mlL1-DMask-amodalClipBox-classAware-lmo-test-iter0_lmo-test/error:vsd_ntop:1_delta:15.000_tau:0.050 does not exist.
I am unable to get past this issue. Kindly guide me what to do. I have also re installed and compiled bop_renderer.


Remove spurious docs jobs / workflows / executions

Issue #8068
Moved docs-latest-public.yaml into all-post-commit-workflows.yaml'
Removed the doc step from fast-dispatch-build-and-unit-tests.yaml and also from the secret steps hidden within slow and fast dispatch tests (run_pre_post_commit_regressions_fast_dispatch.shandrun_pre_post_commit_regressions_slow_dispatch.sh` )

I would recommend running both post commit and the docs workflows separately and ensuring they work

Build Docs - https://github.com/tenstorrent/tt-metal/actions/runs/8941441478
All Commits - https://github.com/tenstorrent/tt-metal/actions/runs/8941437613

let's not submit this until the post-commit time is back to normal; starting docs step after tests increases post-commit time by quite a bit

Many many thanks for the review!
I have updated to use the build runner for docs and removed the gate in all_commit tests on the unit-tests.
Here are the successful runs of all-commit and build-docs workflows:

https://github.com/tenstorrent/tt-metal/actions/runs/8950827437
https://github.com/tenstorrent/tt-metal/actions/runs/8950828950

You can see that the build of docs take 6 mins.

I am going to test the standalone workflow after the merge since it is a new one and it does not yet appear in the list of available workflows.


findutils: disable a bogus test

findutils fails to pass checkPhase on my Linux system that has a sysconf(_SC_OPEN_MAX) limit set. The test itself appears to be a common copypasta between gnu projects. Poking limits of available fd's is irrelevant to what findutils is designed to do, so I consider the test to be bogus.
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[ ] NixOS
[ ] macOS
[x] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[x] Fits CONTRIBUTING.md.

Success on x86_64-linux (full log)
Attempted: findutils
Partial log (click to expand)
shrinking /nix/store/pb8rmhpjskvxqgnnh0cgwxniwnlrxgxk-findutils-4.6.0/bin/find
shrinking /nix/store/pb8rmhpjskvxqgnnh0cgwxniwnlrxgxk-findutils-4.6.0/bin/xargs
shrinking /nix/store/pb8rmhpjskvxqgnnh0cgwxniwnlrxgxk-findutils-4.6.0/bin/locate
gzipping man pages under /nix/store/pb8rmhpjskvxqgnnh0cgwxniwnlrxgxk-findutils-4.6.0/share/man/
strip is /nix/store/n9acaakxahkv1q3av11l93p7rgd4xqsf-bootstrap-tools/bin/strip
stripping (with command strip and flags -S) in /nix/store/pb8rmhpjskvxqgnnh0cgwxniwnlrxgxk-findutils-4.6.0/libexec  /nix/store/pb8rmhpjskvxqgnnh0cgwxniwnlrxgxk-findutils-4.6.0/bin
checking for references to /build in /nix/store/pb8rmhpjskvxqgnnh0cgwxniwnlrxgxk-findutils-4.6.0...
shrinking RPATHs of ELF executables and libraries in /nix/store/bjwdb2yx1vdlw178g440bhz449al9a2l-findutils-4.6.0-info
strip is /nix/store/n9acaakxahkv1q3av11l93p7rgd4xqsf-bootstrap-tools/bin/strip
checking for references to /build in /nix/store/bjwdb2yx1vdlw178g440bhz449al9a2l-findutils-4.6.0-info...

Success on aarch64-linux (full log)
Attempted: findutils
Partial log (click to expand)
shrinking /nix/store/7pvrvimvn8jd3zmcvs6cpsayfnbky422-findutils-4.6.0/bin/xargs
shrinking /nix/store/7pvrvimvn8jd3zmcvs6cpsayfnbky422-findutils-4.6.0/bin/find
gzipping man pages under /nix/store/7pvrvimvn8jd3zmcvs6cpsayfnbky422-findutils-4.6.0/share/man/
strip is /nix/store/8zbl35kgg8v7gircl3pvy9027vslll1q-bootstrap-tools/bin/strip
stripping (with command strip and flags -S) in /nix/store/7pvrvimvn8jd3zmcvs6cpsayfnbky422-findutils-4.6.0/libexec  /nix/store/7pvrvimvn8jd3zmcvs6cpsayfnbky422-findutils-4.6.0/bin
checking for references to /build in /nix/store/7pvrvimvn8jd3zmcvs6cpsayfnbky422-findutils-4.6.0...
shrinking RPATHs of ELF executables and libraries in /nix/store/sfvmbz4zfpi442xdph8iywckbwg3vp89-findutils-4.6.0-info
strip is /nix/store/8zbl35kgg8v7gircl3pvy9027vslll1q-bootstrap-tools/bin/strip
checking for references to /build in /nix/store/sfvmbz4zfpi442xdph8iywckbwg3vp89-findutils-4.6.0-info...
/nix/store/7pvrvimvn8jd3zmcvs6cpsayfnbky422-findutils-4.6.0

Timed out, unknown build status on x86_64-darwin (full log)
Attempted: findutils
Partial log (click to expand)
cannot build derivation '/nix/store/nppq35402zrgdqiwh4sbzyir1lkgmwrl-binutils-wrapper-2.30.drv': 5 dependencies couldn't be built
cannot build derivation '/nix/store/dz9qibb8fdin5vqhm85crv8jdhyvknx7-bash-4.4-p23.drv': 3 dependencies couldn't be built
cannot build derivation '/nix/store/prwxx61njc9zmagvyzsvhij67ha39mj0-bootstrap-stage3-binutils-wrapper.drv': 4 dependencies couldn't be built
cannot build derivation '/nix/store/qhk8z0am0mh3z0101k3mi64kpzk52nfy-bootstrap-stage3-clang-wrapper.drv': 6 dependencies couldn't be built
cannot build derivation '/nix/store/a95409hpgycqv8qmxcmvs44ig7r9cnvs-bootstrap-stage3-stdenv-darwin.drv': 9 dependencies couldn't be built
cannot build derivation '/nix/store/qf02k13w4p4g0l467dvhm6rqjvf5i03z-gmp-6.1.2.drv': 3 dependencies couldn't be built
cannot build derivation '/nix/store/zc9dqlh6dlfndgxhz86zc9c4cdkfpivp-xz-5.2.4.drv': 2 dependencies couldn't be built
cannot build derivation '/nix/store/rcqzrdb2dn52ghqwwizm60wpg1hh3gxs-coreutils-8.29.drv': 4 dependencies couldn't be built
cannot build derivation '/nix/store/al7q6g46y9gn197dl2bpm4ppp9k7s08w-findutils-4.6.0.drv': 3 dependencies couldn't be built
error: build of '/nix/store/al7q6g46y9gn197dl2bpm4ppp9k7s08w-findutils-4.6.0.drv' failed

After some more investigation I found that findutils ships their own gnulib which implements getdtablesize() using getrlimit(). Latter doesn't work very well with older kernels with our glibc (perhaps too new). These two conditions seem to produce the test fail. I still think that the test itself is unnecessary, because the getdtablesize() is not used anywhere in the code that is actually run by user.

Success on x86_64-linux (full log)
Attempted: findutils
Partial log (click to expand)
/nix/store/pb8rmhpjskvxqgnnh0cgwxniwnlrxgxk-findutils-4.6.0

Success on aarch64-linux (full log)
Attempted: findutils
Partial log (click to expand)
/nix/store/7pvrvimvn8jd3zmcvs6cpsayfnbky422-findutils-4.6.0

Timed out, unknown build status on x86_64-darwin (full log)
Attempted: findutils
Partial log (click to expand)
cannot build derivation '/nix/store/nppq35402zrgdqiwh4sbzyir1lkgmwrl-binutils-wrapper-2.30.drv': 5 dependencies couldn't be built
cannot build derivation '/nix/store/dz9qibb8fdin5vqhm85crv8jdhyvknx7-bash-4.4-p23.drv': 3 dependencies couldn't be built
cannot build derivation '/nix/store/prwxx61njc9zmagvyzsvhij67ha39mj0-bootstrap-stage3-binutils-wrapper.drv': 4 dependencies couldn't be built
cannot build derivation '/nix/store/qhk8z0am0mh3z0101k3mi64kpzk52nfy-bootstrap-stage3-clang-wrapper.drv': 6 dependencies couldn't be built
cannot build derivation '/nix/store/a95409hpgycqv8qmxcmvs44ig7r9cnvs-bootstrap-stage3-stdenv-darwin.drv': 9 dependencies couldn't be built
cannot build derivation '/nix/store/qf02k13w4p4g0l467dvhm6rqjvf5i03z-gmp-6.1.2.drv': 3 dependencies couldn't be built
cannot build derivation '/nix/store/zc9dqlh6dlfndgxhz86zc9c4cdkfpivp-xz-5.2.4.drv': 2 dependencies couldn't be built
cannot build derivation '/nix/store/rcqzrdb2dn52ghqwwizm60wpg1hh3gxs-coreutils-8.29.drv': 4 dependencies couldn't be built
cannot build derivation '/nix/store/al7q6g46y9gn197dl2bpm4ppp9k7s08w-findutils-4.6.0.drv': 3 dependencies couldn't be built
error: build of '/nix/store/al7q6g46y9gn197dl2bpm4ppp9k7s08w-findutils-4.6.0.drv' failed

Success on x86_64-linux (full log)
Attempted: findutils
Partial log (click to expand)
shrinking /nix/store/rwdpcb15h2hc2ar3a8jw21ipyzwahkl3-findutils-4.6.0/bin/locate
shrinking /nix/store/rwdpcb15h2hc2ar3a8jw21ipyzwahkl3-findutils-4.6.0/bin/find
shrinking /nix/store/rwdpcb15h2hc2ar3a8jw21ipyzwahkl3-findutils-4.6.0/bin/xargs
gzipping man pages under /nix/store/rwdpcb15h2hc2ar3a8jw21ipyzwahkl3-findutils-4.6.0/share/man/
strip is /nix/store/n9acaakxahkv1q3av11l93p7rgd4xqsf-bootstrap-tools/bin/strip
stripping (with command strip and flags -S) in /nix/store/rwdpcb15h2hc2ar3a8jw21ipyzwahkl3-findutils-4.6.0/libexec  /nix/store/rwdpcb15h2hc2ar3a8jw21ipyzwahkl3-findutils-4.6.0/bin
checking for references to /build in /nix/store/rwdpcb15h2hc2ar3a8jw21ipyzwahkl3-findutils-4.6.0...
shrinking RPATHs of ELF executables and libraries in /nix/store/v42vbj0czq954a86717xnnmk18rjisy7-findutils-4.6.0-info
strip is /nix/store/n9acaakxahkv1q3av11l93p7rgd4xqsf-bootstrap-tools/bin/strip
checking for references to /build in /nix/store/v42vbj0czq954a86717xnnmk18rjisy7-findutils-4.6.0-info...

Success on aarch64-linux (full log)
Attempted: findutils
Partial log (click to expand)
shrinking /nix/store/lkknkfq4nc6xgkqkmlfvsclcv9fmw7sm-findutils-4.6.0/bin/xargs
shrinking /nix/store/lkknkfq4nc6xgkqkmlfvsclcv9fmw7sm-findutils-4.6.0/bin/find
gzipping man pages under /nix/store/lkknkfq4nc6xgkqkmlfvsclcv9fmw7sm-findutils-4.6.0/share/man/
strip is /nix/store/8zbl35kgg8v7gircl3pvy9027vslll1q-bootstrap-tools/bin/strip
stripping (with command strip and flags -S) in /nix/store/lkknkfq4nc6xgkqkmlfvsclcv9fmw7sm-findutils-4.6.0/libexec  /nix/store/lkknkfq4nc6xgkqkmlfvsclcv9fmw7sm-findutils-4.6.0/bin
checking for references to /build in /nix/store/lkknkfq4nc6xgkqkmlfvsclcv9fmw7sm-findutils-4.6.0...
shrinking RPATHs of ELF executables and libraries in /nix/store/qwzn8gyzscp9cjffrk5zlnhal50ygx2z-findutils-4.6.0-info
strip is /nix/store/8zbl35kgg8v7gircl3pvy9027vslll1q-bootstrap-tools/bin/strip
checking for references to /build in /nix/store/qwzn8gyzscp9cjffrk5zlnhal50ygx2z-findutils-4.6.0-info...
/nix/store/lkknkfq4nc6xgkqkmlfvsclcv9fmw7sm-findutils-4.6.0

Timed out, unknown build status on x86_64-darwin (full log)
Attempted: findutils
Partial log (click to expand)
cannot build derivation '/nix/store/nppq35402zrgdqiwh4sbzyir1lkgmwrl-binutils-wrapper-2.30.drv': 5 dependencies couldn't be built
cannot build derivation '/nix/store/dz9qibb8fdin5vqhm85crv8jdhyvknx7-bash-4.4-p23.drv': 3 dependencies couldn't be built
cannot build derivation '/nix/store/prwxx61njc9zmagvyzsvhij67ha39mj0-bootstrap-stage3-binutils-wrapper.drv': 4 dependencies couldn't be built
cannot build derivation '/nix/store/qhk8z0am0mh3z0101k3mi64kpzk52nfy-bootstrap-stage3-clang-wrapper.drv': 6 dependencies couldn't be built
cannot build derivation '/nix/store/a95409hpgycqv8qmxcmvs44ig7r9cnvs-bootstrap-stage3-stdenv-darwin.drv': 9 dependencies couldn't be built
cannot build derivation '/nix/store/qf02k13w4p4g0l467dvhm6rqjvf5i03z-gmp-6.1.2.drv': 3 dependencies couldn't be built
cannot build derivation '/nix/store/zc9dqlh6dlfndgxhz86zc9c4cdkfpivp-xz-5.2.4.drv': 2 dependencies couldn't be built
cannot build derivation '/nix/store/rcqzrdb2dn52ghqwwizm60wpg1hh3gxs-coreutils-8.29.drv': 4 dependencies couldn't be built
cannot build derivation '/nix/store/al7q6g46y9gn197dl2bpm4ppp9k7s08w-findutils-4.6.0.drv': 3 dependencies couldn't be built
error: build of '/nix/store/al7q6g46y9gn197dl2bpm4ppp9k7s08w-findutils-4.6.0.drv' failed


Update stable version in repo from 1.4.0 to 1.4.2

Think about update stable version to 1.4.2
Version: stable=1.4.0 (347 days old) => latest=1.4.2 (15 days old)
Installs: stable=1068 (83.05%), latest=99 (7.7%), total=1286
Click to use developer portal
Click to edit
Note: This is an automatically generated message and not personally authored by bluefox!

@christianh17
Is there any reason to delay release to stable any longer?
If there is no comment until mid of November (15.11.) we might publish 1.4.2 at stable anyway.

@mcm1957 To be honest: The requirements I have to do are difficult for me to achieve, so I did not take the time to dive in deeply. I am only "hobby"-programmer. If the only "leftover" is the splitting of the readme.md it should be no problem to do this and I could do a version 1.4.3 hopefully next weekend.

Thanks for the info about the readme.md and your offers to help me :-).
As far as I understand the comments from the users: the issue #207 is resolved with the new version. That issue was the reason for the new version ;-). And: Lots of people installed already the beta version, I think they did it because of the incompatibility between the new js controller and my adapter.
The dependapot gave me a green checkmark with the newest commit - I commited your PRs.
What shall I do now? I really do not know what's the best to do.
You do not stress me - I know I should care about the new version going into the stable repository and sometimes I need a reminder.
Regards
Christian

And: Lots of people installed already the beta version
Not quite sure what you refer to. As you did not yet create a release which beta do you refer to? Direct github installation?

If you think your code is OK now the next steps would be to create a new release. Your installation of @alcalzone/releasescript seems to be incomplete. If it's ok I will create a PR to fix this.

It would be great if you could fix the releasescript with a PR :-). Thanks!

I've created https://github.com/christianh17/ioBroker.bydhvs/pull/239
So the next steps would be

merge the PR and check that no error are reported from tests
update README.md and add a ne changelog section using tag ### WORK IN PROGRESS
add your fixes to README.md
add a note that now nodejs 16 is required as minimum to README.md
create a new release by running 'npm run release'

please note that this hould be a minor release (1.5.0) minimum as the required node version is changed.

check the results of the workflow.

If the woirkflow raises any errors and you do not see the reason at the first glance, feel free to contact me again.

JUST HOLD ON - I just noticed that I missed update of pacakge-lock

Thanks again. Today I am too tired, I can't do anything reasonable any more.

OK, PR is ready.
NO Problem - continue whenever you have time.
If I do not react please ping my again - I could likely miss some mail.

Hopefully all went good now, I managed with your help to create the new release. Thanks!
If there are some successful installs the next step will be a PR to get the release into the stable repo. :-)

@christianh17
Just noticed that you did not yet activate the automatic deploy to npm. (or it got lost during one of the latest updates)
I would suggest to uncomment the are at test-and-release.yml (https://github.com/christianh17/ioBroker.bydhvs/blob/c7e521208469de55ac94ce08c3393a0456c357bd/.github/workflows/test-and-release.yml#L79) unless you prefer manual deploment
1.5.0 is NOT yet deployed to npm and thus will not be available at latest. You may do manual npm deploy at the adapter directory or

uncommnet the linked section
ensure that a nom token is stores at github secrets (for instructions see referenced location at test-and.release)
create a new 1.5.1 release ( or a 1.5.1-alpha.0 to test it before)

@mcm1957 : Thanks again, it is so kind from you to support me!
I did a manual deploy and for the future: I did as you recommended, so the next release will be pushed automatically to npm :-).
Today I wanted to install the 1.5 version on my productive ioBroker, but this has to wait till the new version will be accessible via the beta repository. Of course it worked on my test installation.
Nice weekend!

version should be available at latest repo  this afternoon

This issue seems to be outdated.
This issue suggests to update the stable version of this adapter to 1.4.2 but in the meantime an update to version 1.5.0 is suggested.
So this issue will be closed and replaced by an updated one.


Would it be possible to use an external prompt, such as startship?

Hey again,
I am not sure if you are familiar with the prompt starship, but it's written in rust, cross-platform, and says it should be compatible with most any shell. I am not sure if it would be possible to just pop it in there, though, in a manner similar to other shells? I made a post in their discussions area as well, as I wasn't sure exactly who would be the best to ask. https://github.com/starship/starship/discussions/2905
If you have any thoughts on this, I definitely would appreciate it.
Thanks,
-MH
-- Edit
I was looking more through their FAQ and came across this bit:

Basic Implementation
The way Starship is built, it should be possible to add support for virtually any shell. The starship binary is stateless and shell agnostic, so as long as your shell supports prompt customization and shell expansion, Starship can be used.
Here's a small example getting Starship working with bash:
# Get the status code from the last command executed
STATUS=$?

# Get the number of jobs running.
NUM_JOBS=$(jobs -p | wc -l)

# Set the prompt to the output of `starship prompt`
PS1="$(starship prompt --status=$STATUS --jobs=$NUM_JOBS)"

Done a quick test and replacing the existing prompt handler with this seems to work fine.
Shell.Prompt = () =>
    var prompt = `starship prompt --status=$Shell.LastExitCode$ --jobs=$Shell.BackgroundProcesses.Count()$`;
    return ColorString.FromRawANSI(prompt);
};

I tried swapping out the default for your example, but for some reason, I keep getting the following:
zsh > dotnet-shell
An error occured loading core.nsh
StartIndex cannot be less than zero. (Parameter 'startIndex')
# >

I tried to add a few slight checks, but I still end up getting the same result:

core.nsh snippet
// bool? powerline = null;
#region c#
Shell.Prompt = () =>
{
    var exitCode = `Shell.LastExitCode`;
    if (exitCode == null || exitCode < 0)
    {
        exitCode = 0;
    }

    var bgJobs = `Shell.BackgroundProcesses.Count()`;
    if (bgJobs == null || bgJobs < 0)
    {
        bgJobs = 0;
    }

    var prompt = `starship prompt --status=$exitCode$ --jobs=$bgJobs$`;
    return ColorString.FromRawANSI(prompt);
};

// Shell.Prompt = () =>
#endregion
// {
//       if (powerline == null && !RuntimeInformation.IsOSPlatform(OSPlatform.Windows))
//       {
//           var powerlineHelpOut=`which powerline`;
//           powerline = !string.IsNullOrWhiteSpace(powerlineHelpOut);
//       }

//       if (powerline == true)
//       {
//           string powerLinePrompt=`powerline-render shell left --last-exit-code=$Shell.LastExitCode$`;
//           return new ColorString("!", Color.Green, Color.LightBlue) + ColorString.FromRawANSI( powerLinePrompt );
//       }
//       else
//       {
//           return new ColorString("!" + Environment.UserName + "@" + Environment.MachineName, Color.Green) + new ColorString(" " + Shell.WorkingDirectory + ">", Color.Blue) + " ";
//       }
// };

I just edited core.nsh directly. It seemed like that would be the proper way to go about it, am I mistaken on that?
Thanks,
-MH

There are problems:
var exitCode = `Shell.LastExitCode`; should actually be var exitCode = Shell.LastExitCode;
Putting text in backticks means execute this shell command, take the stdout and put in a variable. But Shell.LastExitCode is already a piece of C# -- see LastExitCode. That means we can just assign it.
Same for the bgJobs variable assigned as well - drop those backticks.
bgJobs and exitCode will also never be null or < 0 so you can drop that check as well.
When executing the starship prompt helper you are correctly using $VARIABLE$ notation so that dotnet-shell knows to replace that text with a variable held in C#.
Make those changes and your code should work.
My guess is that you getting that error as some bash error message is being evaluated when starship is run. Personally I would stick with my code, it works and its a bit more concise.

Ah yes, that definitely makes sense. Unfortunately, though, the original code you had shared is what was giving the An error occured loading core.nsh. StartIndex cannot be less than zero. (Parameter 'startIndex') message. That was the reason I tried to check to make sure one of the values wasn't coming back as less than 0. The error message didn't change, though between the original code you shared and the modification I had tried.

This is my core.nsh that i've just tested on Ubuntu and works. Could I ask that you give this a go. If there is still a bug there is a much more exciting issue.

I have reproduced the issue. The dotnet-shell preprocess isn't taking out all the comments correctly. Removing the large comment blog (as in my paste) will fix your issue.

I was just in process of typing that out, you beat me to it. :+1:

Latest release will fix the issue


Expression throws error

When I have my expression like such:
{{ :refresh:user.name }}
I receive this error:
Syntax Error: Token ':' not a primary expression at column 1 of the expression [:refresh::user.name] starting at [:refresh::user.name].
However, if I remove the leading space, it works. Is this intended behaviour?
e.g.
{{:refresh:user.name}}

Also, should this expression still work:
{{:refresher::user.name}}

However, if I remove the leading space, it works. Is this intended behaviour?

That is not the intended behaviour, no. I'll see if I can get on that in a timely fashion (basically means I'll pester @tregusti about some assistance), I've got a feeling it has to do with some of the regexes in use.

Also, should this expression still work: {{:refresher::user.name}}

No, that will not work 😄

You solved that nicely! Though you could remove the brackets around the \s in a future commit. Not needed, but no diff.


compose files are misinterpreted as version 2 when there is no version header - and it's not possible to do anything with them

Bug description
I can't start, edit, or delete an old stack because the format is wrong
Expected behavior
I would expect to be able to start, edit or delete an existing stack
Portainer Logs
2021/03/25 13:31:21 http error: Unable to stop stack (err=The Compose file '/data/compose/95/docker-compose.yml' is invalid because:
Unsupported config option for tk4: 'mem_reservation'
Steps to reproduce the issue:

Go to any older stack that was created when portainer didn't support compose v2 syntax, it has to be one where the compose file does not specify the compose version, and with some statements that do not exist in other compose versions.
try to do anything to it - stop, start, delete, edit
It will not work. The error will complain about a syntax error since the default compose version seems to be 2 now, but the old compose file would be version 1 and have different commands.

Technical details:

Portainer version: 2.1.1
Docker version (managed by Portainer):
Platform: linux
Command used to start Portainer (docker run -p 9000:9000 portainer/portainer):
Browser: Firefox

Additional context
according to the docker-compose documentation, a compose file without version header is interpreted as version 1 - but inside portainer it is misinterpreted as something else, most likely version 2, which breaks every pre-existing stack.

here's my compose file:
tk4:
  mem_reservation: 96M
  mem_limit: 128M
  container_name: tk4
  hostname: akari
  image: rattydave/docker-ubuntu-hercules-mvs:latest
  net: bridge
  ports:
    - 3270:3270/tcp
    - 8038:8038/tcp
  privileged: true
  tty: true
  stdin_open: true
  restart: unless-stopped
  volumes:
    - /home/mathias/hercules_mode:/opt/hercules/tk4/unattended/mode
  environment:
    - NUMCPU=1
    - MAXCPU=1

Up to portainer 2.0.x it worked fine - now in portainer 2.1.1 I can't do anythign with it, I can't start the stack, and without the stack running i can't edit the compose file either.

Ticket update by Bala Natarajan<EMAIL_ADDRESS>Hi
pl confirm if the setup is on ARM/ARM64.

nope, X86_64, docker running on openSUSE Leap 15.2

Remarking this for dev investigation. @lemmy04 , have you got any other log files you are able to sharE?

I was able to deploy the above stack to Portainer v 2.0.0 but not into 2.1.1
I believe in 2.1.1 when a version is not specified, its interpreted as Version 3 (not 2). In this case the YML that 's used is not valid for Docker Compose Version 3

portainer 2.5.1 - this bug is still around.

Hi, you'll need to specify the version.
(libcompose always assumes v2, but docker compose doesn’t)
But I assume that you can't edit the stack (to add the version in) because it's stopped?

exactly. Also, the default behaviour for the docker-compose command is to "assume" Version 2 when there's no version info in a compose file, so I don't think portainer should deviate from that... and there is no way to "specify" version 1.

It's even in the portainer logs:
2021/05/31 06:28:24 http error: Version in "/data/compose/25/docker-compose.yml" is invalid. You might be seeing this error because you're using the wrong Compose file version. Either specify a supported version (e.g "2.2" or "3.3") and place your service definitions under the services key, or omit the version key and place your service definitions at the root of the file to use version 1.

@lemmy04 yes, we're going to work on https://github.com/portainer/portainer/issues/4944 which will allow you to update the version of a stopped stack to solve this problem.

Since this ticket was opened, the version element has been deprecated. The compose specification has this to say:

Top-level version property is defined by the specification for backward compatibility but is only informative.
A Compose implementation SHOULD NOT use this version to select an exact schema to validate the Compose file, but prefer the most recent schema at the time it has been designed.

I too have run into this issue now. And not sure how to fix it. Portainer version 2.18.4. Omitting the version, I get a notification unsupported Compose file version: 1.0. Which sucks, because I need to manipulate the startup order of containers. The depends_on keyword can be used in stack deploy on the latest compose version, according to Docker docs.

Still an issue on 2.19.5


Improve the garage-sign targets push resilience

image_types_ostree: introduce GARAGE_PUSH_RETRIES

This add the possibility to change the number of retries
of the "garage-sign targets push"

image_types_ostree: introduce GARAGE_PUSH_RETRIES_SLEEP

This add the possibility to sleep a maximum defined random
time after "garage-sign targets push" fails

Can this be backported to kirstone https://github.com/uptane/meta-updater/pull/57 ?
Thanks

Absolutely, you can always just create PRs for every branch you need. It's worth starting with master just to centralize the discussion, but once that's clear, go right ahead!

Absolutely, you can always just create PRs for every branch you need. It's worth starting with master just to centralize the discussion, but once that's clear, go right ahead!

That's what I expected but it's always better to ask first.


algebraic sorting in sparse.linalg.eigsh fails for hermitian input

I am using scipy.sparse.linalg.eigsh on hermitian matrices, asking for the (necessarily real) eigenvalues to be sorted algebraically, starting from the smallest eigenvalue (which='SA').
It seems that for certain matrix sizes and for certain (comparatively large) number of requested eigenvalues/eigenvectors, the sorting breaks down (and it does so without warning).
The code below sets up a sparse real symmetric matrix, once using a real and once a complex representation (with zero imaginary part). It then computes the first k eigenvalues (printing only the first three).
The real version works fine.
For the complex version, eigenvalue ordering breaks down for large enough k (but still significantly below the dimension of the matrix).
import scipy.sparse as sp
import scipy.sparse.linalg as spla

n=101

print("First three eigenvalues for real symmetric input")
t_real = sp.diags([ 1, -2, 1],[-1,0,1], shape=(n,n))
for k in [n/3,n/2,n-3,n-2]:
    eva, evc = spla.eigsh(t_real, k=k, which='SA')
    print("k = {}: {}".format(k, eva[:3]))

print("First three eigenvalues for real symmetric input")
print("(but stored as complex array).")
t_cplx = sp.diags([ 1+0J, -2+0J, 1+0J],[-1,0,1], shape=(n,n))
for k in [n/3,n/2,n-3,n-2]:
    eva, evc = spla.eigsh(t_cplx, k=k, which='SA')
    print("k = {}: {}".format(k, eva[:3]))

outputs
$ python mwe.py
First three eigenvalues for real symmetric input
k = 33: [-3.99905144 -3.99620666 -3.99146835]
k = 50: [-3.99905144 -3.99620666 -3.99146835]
k = 98: [-3.99905144 -3.99620666 -3.99146835]
k = 99: [-3.99905144 -3.99620666 -3.99146835]
First three eigenvalues for real symmetric input
(but stored as complex array).
k = 33: [-3.99905144 -3.99620666 -3.99146835]
k = 50: [-2.06159012 -2.12312181 -2.18453672]
k = 98: [-0.01515898 -0.02366906 -0.0340538 ]
k = 99: [-0.00853165 -0.01515898 -0.02366906]
```

Note: When using complex arrays, the problem seems to be just the *ordering* of the eigenvalues (i.e. the actual smallest algebraic eigenvalues are still contained in the list).

I am using python 2.7.12, scipy 0.17.1

The order is different since complex-valued matrices are computed via
the nonsymmetric ZNAUPD/ZNEUPD mode 'SR', which behaves differently.
Note that the documentation of eighs does not claim the eigenvalues are
ordered.

Note that the documentation of eighs does not claim the eigenvalues are
ordered.

Oh, you are right - sorry I didn't notice! I guess I was just used to this behavior from linalg.eigh and then found the same behavior with eigsh with which='SA' for some choices of parameters, while not for others.
One might consider adding a warning to the documentation of which (something like "Note: eigenvalues will not be ordered in general.") but I admit that just considering the other options for which should probably lead the reader to this conclusion as well.
Thanks for the quick feedback, the issue can be closed.


Number type support?

Hello, I wonder how to specify the schema so that the value is parsed into number?

I implemented a Number type in 0.9.2 -   I started explaining how to do it,
and well... Let me know if this works.
On Sat, Jul 18, 2015 at 8:14 AM Big Drum<EMAIL_ADDRESS>wrote:

Hello, I wonder how to specify the schema so that the value is parsed into
number?
—
Reply to this email directly or view it on GitHub
https://github.com/jspears/subschema/issues/11.

Thanks for quick response. But seems it doesn't support input integers.
The following branch matches integer and prevent it being set to the value manager.
//Not a valid number but valid to become a number
if (/^(-\.|-|)[0-9]*\.?$/.test(value)) {

yeah, was a little quick to release.  I think its fixed.   But don't have
time to do anything thorough until tonight.
On Sun, Jul 19, 2015 at 12:11 AM Big Drum<EMAIL_ADDRESS>wrote:

Thanks for quick response. But seems it doesn't support input integers.
The following branch matches integer and prevent it being set to the value
manager.
//Not a valid number but valid to become a number
if (/^(-.|-|)[0-9]*.?$/.test(value)) {
—
Reply to this email directly or view it on GitHub
https://github.com/jspears/subschema/issues/11#issuecomment-122634325.

I think its fixed.  Let me know if you have any issues with this.


[Improvement] fixed code quality issues

Hi :wave:
I ran the DeepSource static analyzer on the forked copy of iris repo and found some interesting code quality issues.
This PR fixes a few of them:

Removed unnecessary blank (_) identifier
Replaced t.Sub(time.Now()) with time.Until
Removed unnecessary use of fmt.Sprintf
Removed unnecessary use of slice
Added .deepsource.toml config

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

Hey @wadey :wave:
Please review this PR! :sparkles:

@wadey :wave: Please have a look :eyes:

@wadey @JohnMaguire :wave: please review this PR :sparkles:


Set transparent navigation bar on Android 10

Squash and merged at 7da68c78a8024213413d5a899c6f1c64a75b6f84, Thanks.
Already released at Google Play.


How to Run a JNLP File At a Local Folder Location

I am trying to use FunStart to run a tool that depends on JWS. We did not create this tool, and we have no control over it.  It is pushed down from a website (normally powered with JWS), but it does not have a link that ends with ".jnlp"
The FunStart GUI does not accept a link without the ".jnlp" ending. It does not accept a folder path directly to the JNLP file (it seems to think a colon is an illegal character).
Is there any way around this?

Interesting — just to verify — the link itself does contain a jnlp file
even though it doesn’t have the extension?  I will take a look and get back
to you.  I’ll also look at the local file reference issue.
Could you give me a mock-up of what the URL looks like to make sure I fix
your issue?
On Mon, Aug 12, 2019 at 3:27 PM RDavies2019<EMAIL_ADDRESS>wrote:

I am trying to use FunStart to run a tool that depends on JWS. We did not
create this tool, and we have no control over it. It is pushed down from a
website (normally powered with JWS), but it does not have a link that ends
with ".jnlp"
The FunStart GUI does not accept a link without the ".jnlp" ending. It
does not accept a folder path directly to the JNLP file (it seems to think
a colon is an illegal character).
Is there any way around this?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/mainstringargs/FunStart4j/issues/11?email_source=notifications&email_token=ABVXM4Q5ZOELCNNFF3WV3LLQEHBS5A5CNFSM4ILEXDEKYY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4HEZWMOA,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABVXM4UDATRAC5GTB3SKXSTQEHBS5ANCNFSM4ILEXDEA
.

I just push an updated release, please give it a try


Investigate Linux and Mac support

Investigate if we can build the .Net core project on Linux and Mac.

It is possible to compile using the same code but you need to also compile the oqs library natively for the os and architecture you want to support and copy them into the final product when you want to run them. Though the current code probably resolve to a dll file and will need some modifications to support the “so” and “dylib” for linux and Mac respectively.

Closed by #11.


Question: does zero-copy deserialization change object performance?

I have looked carefully in the manual, but I couldn't find whether there is an impact on the performance of a zero-copy deserialized object WRT a normal object of the same type with the same content.
The author of zerovec, for example, shows that there is definitely a gap in his implementation. I guess in the case of rkyv resolving relative pointers might have some impact, as the immutability of the backing storage make it necessary to solve them at each access, but it would be nice to know if the effect is negligible or not.

rkyv strives to have exactly the same formal runtime for serialized types as regular ones - you should expect there to only be a constant factor differentiating the runtimes of the two. I think it's worth mentioning that runtime may vary in the docs, but I really cannot say what the impact is likely to be. In most cases (x86?) you'll probably pay a small tax, other targets (ARM? it seems likely that relative pointers will cost more on RISC architectures) may pay a higher tax. Performance is a highly composite metric though, and I don't want to make claims that are hard to back up or require updating over time.


fix: deploy issue while zipping the folder

Description
Today plugins use bot-deploy as local dependency in package.json. But the archiver doesn't work in this pattern(I don't know the reason now. I have added archiver to test in the plugins and composer server, it works).
The bot-deploy is only used by plugins. So I move it to plugins' lib folder and add a yarn workspace for plugins.
I also merged #3556 to fix the profile save issue, this will break the publish workflow.

Task Item
fixes #3301
refs #3536

Screenshots

Coverage increased (+0.009%) to 53.244% when pulling eb69617132159d177a40dea5775d2cb1dce4d53d on lei9444:test into 9bd3ecd67772bda4ec508c92cff9e43d8d17af34 on microsoft:master.

@cwhitten this is the fix that I believe eventually resolve the "archiver" not working issue without have to switching to "zip-folder" (which depends on a lower version of lodash, thus not passing secuirty check).
we tested this both in web app and in electron and it's working as expected E2E.

Are the other changes un-releated to the workspace change meant to be in this PR? Specifically the runtime settings changes?

Are the other changes un-releated to the workspace change meant to be in this PR? Specifically the runtime settings changes?

Hi, @a-b-r-o-w-n I merged @VanyLaw 's PR about profile can not be saved issue. Remove this change will make the publish fail. The PR is https://github.com/microsoft/BotFramework-Composer/pull/3556.
Do you think we need to move un-releated changes to another PR? Maybe I can drop the commit and reopen @VanyLaw 's PR #3556.

Are the other changes un-releated to the workspace change meant to be in this PR? Specifically the runtime settings changes?

Hi, @a-b-r-o-w-n I just merged @VanyLaw 's PR about profile can not be saved issue. Remove this change will make the publish fail. The PR is #3556.
Do you think we need to move the un-releated changes to another PR? Maybe I can drop the commit and reopen @VanyLaw 's PR #3556.

I think this one is now considered as an fix to the E2E publishing workflow, it fixes the zipping stucking issue and fix the publishing profile not saving issue. I suggest we take this one. @a-b-r-o-w-n  what do you think

@boydc2014 I haven't had a chance to test this. So if you feel like it is good, let's merge.


added: Support for max-pixels attribute

#578

@gushil, would you like me to make the little tweak to enketo-xslt (and enketo-transformer) to pass the max-pixel value set in the XForm, or are you interested in doing this?
I'm asking because xsl is such an obscure language.

@gushil, would you like me to make the little tweak to enketo-xslt (and enketo-transformer) to pass the max-pixel value set in the XForm, or are you interested in doing this?
I'm asking because xsl is such an obscure language.

I think for now it would be best if you do the tweak so I can focus on this issue. Is it ok?

resizeImage has been moved into utils.js.
Now still working on how to save resized image blob. Thinking maybe saving into localstorage?

I will make the enketo-xslt change and let you know when done.

Thinking maybe saving into localstorage?

Would like to avoid a browser db solution if we can. Could we use the preview perhaps? (objectURl or dataURI?)

done, see https://github.com/enketo/enketo-core/issues/578#issuecomment-507350216


Add collection insight distribution showing number items owned and unique owners

The first number means how many items are owned and then we could add unique owners

From these two numbers, we could distil ownership distribution, which says how much is owned collection

Tested on https://kodadot.xyz/rmrk/collection/dac5c7f54029d0e73c-HXAI8
Inspired by https://solanart.io/collections/thetower

On "ITEMS" activity, you would like to display:
(owned items) / (unique owners) ?
What will be displaying on "OWNED" activity?
In solarnart it's displaying:
(listed nfts in collection) / (collection length)
(Preview from Apes collection with solarnart implementation)

Distribution was solved in #1037
So still we can add this there

On "ITEMS" activity, you would like to display: (owned items) / (unique owners) ? What will be displaying on "OWNED" activity?
In solarnart it's displaying: (listed nfts in collection) / (collection length)

Oh, I was a bit confused when doing research, yes this is what I've meant originally
Items
(listed nfts in collection) / (collection length)
Owners
(unique owners)

I've expanded bit @roiLeo hope it make sense now! :)
From owners currently we count distribution index but it can give you insight on "how" is collection distributed if someone is interested in hard numbers.
Re items
make sense to show listed and total as when you buy some ppl don't list it etc and that could show another collections dynamics etc

Odds that it will be fixed in #1124

Okay closing in favour #1124


Review crud mission

Fix #45
Also fix #95

Actually it also fixes #35

@cescoffier Thank you, merging:)

LGTM, merged: https://github.com/openshiftio/appdev-documentation/commit/1f0225b77d883b4bd9eda1c41cb851bc5c111abc


Add to array(fix + unit tests)

This PR is the same code as https://github.com/aspnet/Mvc/pull/2131. That was based against a different branch. I am merging it with jsonpatch branch.
cc @KevinDockx, @rynowak

:shipit: after fixing minor comments


"theme: fancy" not rendering icons

os: MacOS Sonoma 14.0, current as of time of this post
lsd --version: 1.0.0, installed 10 mins ago via Homebrew
echo $TERM: xterm-256color
echo $LS_COLORS: none -- empty return value (see pic, below)

Expected behavior
Display "fancy" icons.
Actual behavior
This image is with theme: fancy inside ~/.config/lsd/config.yaml:

To compare, this image is with theme: unicode:

Note re nerd-font
I followed the link in LSD's readme, then following that guide's Homebrew install instructions. This image shows that the nerd font is installed:

While fiddling around trying to resolve, I figured out that my terminal app was no longer using the nerd-font I'd previously selected. Didn't even notice the change. Anyway, that's the issue. Sorry for the unnecessary ticket. 😊


Guilds marked as undefined without "?" afterwards.

I tried to recreate the "guildinfo.ts" command in the test directory but it guild was always showed up as undefined. Whenever I add a ? afterwards, the error goes away. But whenever I run !guildinfo everything is either marked as 0 or "undefined".

Hi there, thanks for your issue. Can you send the code?

I deleted the code as I was just testing the Library. ANerdThatNoOneLikes was watching the stream at the time and he saw everything. Everything was working, it was just that guild had to be marked as optional. I even copy-pasted the code from the github and it still gave me the error.

Hm no idea


add browser.d.ts

Fix an issue where lightstep-tracer/browser does not have types:

To work around this, I've added this same declaration in our own declare.d.ts for now.

@bg451 Can you review this?


[DUOS-2951][risk=no] Post DAA API

Addresses
https://broadworkbench.atlassian.net/browse/DUOS-2951
Summary

Requires https://github.com/DataBiosphere/consent/pull/2256
Adds a new resource class for DAAs along with api documentation

Have you read CONTRIBUTING.md lately? If not, do that first.

Label PR with a Jira ticket number and include a link to the ticket
Label PR with a security risk modifier [no, low, medium, high]
PR describes scope of changes
Get a minimum of one thumbs worth of review, preferably two if enough team members are available
Get PO sign-off for all non-trivial UI or workflow changes
Verify all tests go green
Test this change deployed correctly and works on dev environment after deployment

manually tested on the Swagger, looks good to me besides the test failures in DaaResourceTest

There are some actions that don't seem to be executing ... I didn't see the failing DaaResourceTest here, but it is failing locally

There are some actions that don't seem to be executing ... I didn't see the failing DaaResourceTest here, but it is failing locally

Ok, that's due to the base branch not being develop. Once https://github.com/DataBiosphere/consent/pull/2256 is finalized, I'll rebase this and clean up any test failures


Missing dependency on install instructions

libdrm-dev and  libgl1-mesa-dev are dependencies I had to install in order to compile, they should be added on the instructions.

what OS are you on?

Lubuntu 18.04

I added this in #73


Password Protected Vimeo Albums

Hello There,
I've been trying to play a password-protected album on my Chromecast. Omitting the --video-password option catt asks me to provide the password with "--video-password" option:
$  catt cast "https://vimeo.com/showcase/xyz"
ERROR: This album is protected by a password, use the --video-password option
Error: Remote resource not found.

When providing the password using this option it says the option does not exist:
$ catt cast --video-password MY_PASSWORD "https://vimeo.com/showcase/8551988"
Usage: catt cast [OPTIONS] VIDEO_URL
Try 'catt cast -h' for help.

Error: no such option: --video-password

Am I doing something wrong here?
I verified it with youtube-dl and it works fine. But cannot make it work with catt.

No, this is because that's a YouTube-dl option, not a CATT option. I'm not at the computer currently but I think we have a ytdl command line option.

Try this:
catt cast -y videopassword=MY_PASSWORD "https://vimeo.com/showcase/8551988"

Try this:
catt cast -y videopassword=MY_PASSWORD "https://vimeo.com/showcase/8551988"

Thank you.
It verifies the password now as youtube-dl does, but results in errors:
[vimeo:album] 8551988: Downloading JSON metadata
[vimeo:album] 8551988: Downloading JSON metadata
[vimeo:album] 8551988: Verifying the password
[vimeo:album] 8551988: Downloading page 1
Traceback (most recent call last):
  File "/usr/bin/catt", line 8, in <module>
    sys.exit(main())
  File "/usr/lib/python3.9/site-packages/catt/cli.py", line 644, in main
    return cli(obj=get_config_as_dict())
  File "/usr/lib/python3.9/site-packages/click/core.py", line 829, in __call__
    return self.main(*args, **kwargs)
  File "/usr/lib/python3.9/site-packages/click/core.py", line 782, in main
    rv = self.invoke(ctx)
  File "/usr/lib/python3.9/site-packages/click/core.py", line 1259, in invoke
    return _process_result(sub_ctx.command.invoke(sub_ctx))
  File "/usr/lib/python3.9/site-packages/click/core.py", line 1066, in invoke
    return ctx.invoke(self.callback, **ctx.params)
  File "/usr/lib/python3.9/site-packages/click/core.py", line 610, in invoke
    return callback(*args, **kwargs)
  File "/usr/lib/python3.9/site-packages/click/decorators.py", line 33, in new_func
    return f(get_current_context().obj, *args, **kwargs)
  File "/usr/lib/python3.9/site-packages/catt/cli.py", line 219, in cast
    cst, stream = setup_cast(
  File "/usr/lib/python3.9/site-packages/catt/controllers.py", line 82, in setup_cast
    stream = StreamInfo(video_url, device_info=cast_device.info, ytdl_options=ytdl_options) if video_url else None
  File "/usr/lib/python3.9/site-packages/catt/stream_info.py", line 62, in __init__
    self._entries = list(self._preinfo["entries"])
TypeError: 'OnDemandPagedList' object is not iterable

This Vimeo URL (showcase) has 2 videos in it. Could it be the reason for this failure?

This Vimeo URL (showcase) has 2 videos in it. Could it be the reason for this failure?

It most likely is. If its possible, can you try video-urls from that showcase?

It most likely is. If its possible, can you try video-urls from that showcase?

I tried a single link (https://vimeo.com/showcase/8551988/video/561679574) from the showcase but doesn't work.
Apparently, Youtube-dl doesn't support it either. It gives me 404.

Hi,
I have similar issue - is there some way to log in to google account when starting a cast? I would like to cast my calendar to my "not so smart display" - Google Nest Hub Max.
When casting https://calendar.google.com. - it shows log in page, which is useless without keyboard.
Any ideas?
Thank you!


Op2dTensorLite doesn't handle the last element in some cases (OpTensor3d test failed)

When we reach number of workgroups hard limit (int max_num_wg  = 4096;) Op2dTensorLite stops to handle all the elements beyond total threads limit (4096*256 = 1048576):
./bin/test_tensor_ops --float --a 1 1 1048577 --b 1 1 1048577 --sa 1 1 1 --sb 1 1 1 --sc 1 1 1 --offsets 0 0 0 --alpha-beta 1 1 1 --packed 1
...
MIOpen(HIP): Info2 [Log] Kernel MIOpenTensorKernels.cl Compile Time, ms: 208.425
MIOpen(HIP): Info2 [run] kernel_name = Op2dTensorLite, global_work_dim = { 1048576, 64, 1 }, local_work_dim = { 256, 1, 1 }
...
c[1048576](16.000000) = a[1048576](16.000000) * b[1048576](16.000000)
...
Mismatch at 1048576: 272 != 16

When I set max_num_wg limit to 16, I encountered this error much earlier, at the elements beyond 4096.
This kernel is widely used in RNN and we were just lucky that there were small tensors.

Related to #2221


[AutoPR eventgrid/data-plane] EventGrid StorageBlobCreatedEventData: Make contentLength field long instead of int to account for blob sizes greater than 2 GB.

Created to sync https://github.com/Azure/azure-rest-api-specs/pull/5179

(message created by the CI based on PR content)

This PR has been merged into https://github.com/Azure/azure-sdk-for-python/pull/3629

Codecov Report

Merging #4338 into restapi_auto_eventgrid/data-plane will increase coverage by 1.18%.
The diff coverage is n/a.

@@                          Coverage Diff                          @@
##           restapi_auto_eventgrid/data-plane    #4338      +/-   ##
=====================================================================
+ Coverage                              55.02%    56.2%   +1.18%
=====================================================================
  Files                                  10947     9183    -1764
  Lines                                 240081   183633   -56448
=====================================================================
- Hits                                  132094   103216   -28878
+ Misses                                107987    80417   -27570

Impacted Files
Coverage Δ

...ventgrid/models/storage_blob_created_event_data.py
26.66% <ø> (ø)
:arrow_up:

...grid/models/storage_blob_created_event_data_py3.py
26.66% <ø> (ø)
:arrow_up:

.../hdinsight/operations/script_actions_operations.py
25.75% <0%> (-71.22%)
:arrow_down:

.../hdinsight/operations/configurations_operations.py
32.14% <0%> (-64.29%)
:arrow_down:

.../operations/script_execution_history_operations.py
31.91% <0%> (-63.83%)
:arrow_down:

...t/hdinsight/models/runtime_script_action_detail.py
38.46% <0%> (-61.54%)
:arrow_down:

...insight/models/runtime_script_action_detail_py3.py
38.46% <0%> (-61.54%)
:arrow_down:

...mt/hdinsight/operations/applications_operations.py
21.49% <0%> (-56.08%)
:arrow_down:

...hdinsight/azure/mgmt/hdinsight/models/usage_py3.py
44.44% <0%> (-55.56%)
:arrow_down:

...gmt-hdinsight/azure/mgmt/hdinsight/models/usage.py
44.44% <0%> (-55.56%)
:arrow_down:

... and 2527 more

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 beed1a0...e42ccb8. Read the comment docs.


radiotunes 320kbps MP3?

I'm not sure why this is excluded, as the URLs are certainly there. Maybe this was different when you originally wrote this up?
For example, here's the 128kbps AAC:
http://prem1.radiotunes.com:80/80srock?[snip]
Contrast this with the 320kbps MP3:
http://prem1.radiotunes.com:80/80srock_hi?[snip]
It seems the URL structure is the same as DI.FM.

Fortunately it's an easy search/replace on the generated playlist, so not the end of the world...

Hi draeath,
Thanks for pointing this out, I should have some time soon (been very busy) to take a look at this.


SCVMM - Correctly handle VMs without snapshots

Up to this point, we have used the SCVMM Virtual Machine property 'LastRestoredVMCheckpoint' to determine if a given checkpoint (snapshot) is current. However, it can be nil if no checkpoints were ever created for a given VM and therefore can cause the SCVMM parser to bomb out.
Instead, we should query the LastRestoredCheckpointID to determine if a
checkpoint is current which is set to either:
The most current checkpoint OR the Virtual machine ID if no checkpoints are present.
https://bugzilla.redhat.com/show_bug.cgi?id=1213939

@blomquisg - over to you, this is a blocker bug

LGTM

@chessbyte - white space cleaned up.

@chessbyte @blomquisg - Can we get this merged? QE cannot run a SCVMM refresh until this is fixed. Seems to be a problem in the newer verison of VMM Server.


databaseAccountOfferType in Microsoft.DocumentDB/databaseAccounts@2021-04-15 is an enum, but missing values.

Looks like there is only one value - 'Standard' so this should be easy to add.

Link to the REST object definition.  https://docs.microsoft.com/en-us/azure/templates/microsoft.documentdb/databaseaccounts?tabs=json#databaseaccountcreateupdateproperties-object

Can you add this to #784?

I took from the community call the other day that these issues should be now separated one-by-one.  Maybe I misunderstood.  @anthony-c-martin, did I misunderstand?

We are planning to separate them, but haven't started the process yet. I suppose it's fine to leave it here and i'll tag it so we can keep track of it.

cc @markjbrown FYI

This value is always standard. The enum is a legacy artifact.

If the value is an enum, isn't it worth saving infrastructure devs the time to look this up by adding it to the swagger?
@alex-frankel, in my experience, this tends to mean that a "Premium" version is in the works although that may not be true in this case.

Also - there is currently a label of "Provider Bug" on this Issue.  I have also see the label "types: swagger inaccuracy".
Do these two labels mean the same thing?

Let's just go with both for now :)

Hey @MehaKaushik FYI. Maybe something we can address in an upcoming api version.


Alternative next slide

Use case: I have a big presentation I'll use regularly. Sometimes I want to go into the details of a topic, sometimes not. Editing the presentation constantly or skipping all slides by hand is possible, but not really convenient. If you press Alt + arrow key, the presentation will jump to the referenced slide instead of the next slide.
As far as I have understood impress.js something like this doesn't exist, does it?

Hi @naraesk
Something like this is now supported in master branch. Thanks for a great suggestion!
Demo presentation: http://impress.github.io/impress.js/examples/2D-navigation/


fix akka-more on jdk9

seen at https://scala-ci.typesafe.com/view/scala-2.12.x/job/scala-2.12.x-jdk9-integrate-community-build/325/consoleFull: [akka-more] Caused by: java.lang.ClassNotFoundException: javax.xml.bind.annotation.XmlRootElement

test run:
https://scala-ci.typesafe.com/view/scala-2.12.x/job/scala-2.12.x-jdk9-integrate-community-build/328/ (404 for now)

[akka-more:error] java.lang.NoClassDefFoundError: javax/xml/bind/annotation/XmlRootElement

ah, so this is reproducible outside dbuild, actually, by running ; ++2.12.6 ; akka-camel/test on JDK 9

oh, there's already a fix: https://github.com/akka/akka/pull/25417

gah, but we can't use latest akka master, because they've upgraded to sbt 1.2.0, which dbuild doesn't support yet. but sbt 1.2.0 support is expected to land soon in dbuild 0.9.13, so I just need to be a little patient

yay, this got fixed in 2cd392196c313ae01cd7edd31f93f00336d52f8f


create topline impact docs

🚀 Deployed on https://62990618060dce0543b9f70f--cozy-fox-0defec.netlify.app


Implement an async version of ToSocketAddrs

Resolve #50

Force pushd for rebase master.
@stjepang Hi, after a long day, I've created a commit to replace std::net::ToSocketAddrs with async_std::net::ToSocketAddrs, would you mind review again. Thanks.

Could you perhaps incorporate changes from https://github.com/DCjanus/async-std/pull/1 here? In particular, let's remove the new re-exports and move tests into the tests directory. At the same time, let's also delete unit tests that are not relevant to ToSocketAddrs.

@stjepang is this ready to merge?

bors r+


[BUG] - Tooltip does not work with component function and forwardRef

NextUI Version
2.2.9
Describe the bug
I am using a function component which returns two siblings, and I was trying to use <Tooltip> to wrap this component, but it did not work. I also tried to use forwardRef, but although the ref is successfully passed to the Tooltip, the Tooltip did not work correctly. Please find my sample code below.
I am not really familiar with how the ref works in Tooltip, and I am not sure if this the correct way.
Your Example Website or App
No response
Steps to Reproduce the Bug or Issue

Create the component function:

// custom-button.tsx

export const CustomButton = forwardRef((props, ref) => {
  return (
    <>
      <Button ref={ref}>
        This is a button
      </Button>
      <Modal>
        <ModalContent isOpen={false}>
          <ModalBody>
            This is a modal
          </ModalBody>
        </ModalContent>
      </Modal>
    </>
  );
});

Wrap this component by <Tooltip>:

// page.tsx

export default function Page() {
  return (
    <Tooltip content="This is a tooltip">
      <CustomButton />
    </Tooltip>
  );
}

Expected behavior
When hovering on the Button inside the CustomButton component, the tooltip should show.
Screenshots or Videos
No response
Operating System Version
Linux
Browser
Chrome

Thanks! Any althernative to fix it?
My current fix is to move the Tooltip inside CustomButton.

@darkpursuer , i resolved this problem by adding a div around my component. Maybe this solution will work for you too.

@darkpursuer , i resolved this problem by adding a div around my component. Maybe this solution will work for you too.

This also worked for me when using the Popover component and having the button inside the PopoverTrigger


[FEAT]: Project Download/Upload

We need to add a download project button and an upload button.
Those endpoints exists and are usable via Swagger/curl.

Do we know where we want these?
For example
download being project overview + here?


update recipe for 1.0

@conda-forge-admin, please rerender
Checklist

[x ] Used a personal fork of the feedstock to propose changes
[x] Bumped the build number (if the version is unchanged)
[x] Reset the build number to 0 (if the version changed)
[x] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[x] Ensured the license file is being packaged.

Not tested on Windows

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.


Way to disable some ISA groups

If one core supports only imc ( -march=rv32imc), what is the best way to disable other ISA groups?

You can configure the selected the ISA group here:
https://github.com/google/riscv-dv/blob/master/src/riscv_instr_gen_config.sv#L34
riscv_instr_group_t supported_isa[] = {RV32I, RV32M, RV64I, RV64M, RV32C, RV64C};

I did the following change
// ISA supported by the processor
riscv_instr_group_t supported_isa[] = {RV32I, RV32M, RV32C};
But then got these errors
run.int ./run -tool irun
Your DISPLAY variable is set to :3.0
Your job has been submitted to NC - Interactive Queue
Fairshare= /class/gui
Resources= guiHost CORES/0 Limit:gui_riscv_@USER@
Env      = SNAPPROP(@JOBID@)+D(DISPLAY=uls-ep-vhpcetx09:3.0)
Command  = vw ./run -tool irun
Logfile  = /dev/null
JobURL   = http://uls-ep-vhpcnc02:8271/cgi/node.cgi?id=001652034
JobId    = 001652034
<<<>>>
instr_name == SD; rs2 == saved_regs[i]; rs1 == SP; imm == 8 * (i+1);)
|
ncvlog: *E,UNDIDN (./src/riscv_directed_instr_lib.sv,216|78): 'SD': undeclared identifier [12.5(IEEE)].
instr_name == LD; rd == saved_regs[i]; rs1 == SP; imm == 8 * (i+1);)
|
ncvlog: *E,UNDIDN (./src/riscv_directed_instr_lib.sv,282|77): 'LD': undeclared identifier [12.5(IEEE)].
allowed_instr = {LW, SW, LWU, allowed_instr};
|
ncvlog: *E,UNDIDN (./src/riscv_load_store_instr_lib.sv,110|35): 'LWU': undeclared identifier [12.5(IEEE)].
allowed_instr = {LD, SD, allowed_instr};
|
ncvlog: *E,UNDIDN (./src/riscv_load_store_instr_lib.sv,116|26): 'LD': undeclared identifier [12.5(IEEE)].
allowed_instr = {LD, SD, allowed_instr};
|
ncvlog: *E,UNDIDN (./src/riscv_load_store_instr_lib.sv,116|30): 'SD': undeclared identifier [12.5(IEEE)].
allowed_instr = {C_LD, C_SD, allowed_instr};
|
ncvlog: *E,UNDIDN (./src/riscv_load_store_instr_lib.sv,118|30): 'C_LD': undeclared identifier [12.5(IEEE)].

It seems to be unrelated to the supported_isa change, do you have this compile failure before the change?

BTW, please set XLEN = 32
https://github.com/google/riscv-dv/blob/master/src/riscv_instr_pkg.sv#L42

I can generate a test now but it seems still generate 64-bit instructions since I got error when I ran the test
ascom: Running as on 0.cpp.s (0)..
vascom: Executing /wdc/apps/riscv/bin/riscv64-unknown-elf-as   -march=rv32imc 0.cpp.s -o 0.o
0.s: Assembler messages:
0.s:8139: Error: unrecognized opcode ld x22,0(x7)' 0.s:8148: Error: unrecognized opcode sd x22,0(x7)'
0.s:8160: Error: unrecognized opcode sd x22,0(x7)' 0.s:8171: Error: unrecognized opcode sd x22,0(x7)'
vascom: ERROR > assembler failed (ret=256)

pt_fault_handler:
sfence.vma x0, x0
li x27, 2
li x17, 0x0
csrr x15, 0x343 # mtval
srli x15, x15, 12
slli x15, x15, 37
la x7, page_table_0
fix_pte:
srli x23, x15, 23
slli x23, x23, 2
add x7, x7, x23
ld x22, 0(x7)
slli x23, x22, 28
srli x23, x23, 29
bne zero, x23, fix_leaf_pte
beq zero, x27, fix_leaf_pte
srli x22, x22, 10
slli x22, x22, 10
li x23, 0xf1
or x22, x22, x23
sd x22, 0(x7)
srli x7, x22, 10
slli x7, x7, 12

Have you changed XLEN to 32?

Yes, I did. Just did a grep to make sure and got this
riscv_instr_pkg.sv:  parameter int XLEN = 32;

Can you check your SATP setting in riscv_instr_pkg? It should be SV32 or BARE for RV32IMC

It works now. Thanks. BTW, do you have a way to specify number of instructions we want to genererate  per test?
Thanks,

You can check the option here:
https://github.com/google/riscv-dv/blob/master/src/riscv_instr_gen_config.sv#L287