[question] How to write the 'require' line in lakefile.lean to use the pre-installed Lean library?

I have the Std library installed as a package which provides these files.
They are the same files that 'lake build' left in the .lake/build folder when Std was built.
When some lake project has import Std, for example:
import Std

def main (args : List String) : IO Unit :=
  IO.println "Hello, world!"

lake build fails:
ld: error: undefined symbol: initialize_Std

This is because the require line is missing in lakefile.lean.
But how to write this require line?
This require line expects the checked out project with lakefile.lean:
require std from "/usr/local"

What would be the correct syntax for the require line that would pick up the pre-installed Lean library?

Could you explain why you are not just allowing lake to obtain Std by itself? The typical use case is
require std from git "https://github.com/leanprover/std4" @ "main"

If you use require std from "/usr/local" (which by the way I don't recommend because it should be a folder for std, not shared with other things - use /usr/local/lean4-std or similar instead), lake will expect the contents of the folder to be a lake project, meaning that it includes a lakefile.lean at the root and source files as well; and the files you put in the package should be located at .lake/build relative to this root. That is:
/usr/local/lean4-std/
  lakefile.lean
  Std.lean
  Std/
  .lake/build/
    ir/
      Std.c
      Std.c.hash
      ...
    lib/lean/
      Std.olean
      Std.olean.hash
      Std.ilean
      Std.ilean.hash
      ...

What we might need here is a concept of vendoring dependencies. We'll need this too once we do packaging for NixOS, for example.
I'd expect that lakefile and manifest stay untouched, but there is a way to tell lake ”I obtained the sources for you, they are unpacked here, do not run git”. Maybe a command line flag or environment variable.
Ideally lake would keep the mutable .lake directory still in the current directory.
Such an override is also useful to temporarily use a local checkout, e.g. when testing coupled bug fixes.
Does that direction sound reasonable to you, @tydeu? Or maybe you already have other ideas for this use case?

Using path dependencies already mostly works here, except that the .lake folder in the dependency may be written to if it is not already up to date. As long as you vendor the right version such that lake doesn't want to rebuild it I don't think there will be a problem, and even if it does rebuild it won't run git on path dependencies.

Lean should allow the syntax like 'require std' in lakefile.lean, in which case lake or lean should look for the pre-installed, pre-built library std in the same $PREFIX where lean is installed. For example, it can find it by looking for a $PREFIX/lib/lean/Std.olean file.

require std is coming, but it won't have the semantics you want - it will download the std package from https://reservoir.lean-lang.org/. There were some discussions about making this configurable to other package registries, but I don't think this feature is designed for system package managers like FreeBSD. (Nor indeed any part of lean.) As should be clear at this point you are carving a new path here with these questions so you may end up needing to munge the lakefiles yourself.

require std is coming, but it won't have the semantics you want [....] There were some discussions about making this configurable to other package registries, but I don't think this feature is designed for system package managers like FreeBSD.

The forthcoming require std (which will be tracked over at #3174) may work with @yurivict's use case if the package system can also ship an alternative registry. The idea is that it will use the default registry Lake is configured with (likely via environment variable now, but hopefully with a Lake-wide configuration file later). In essence, the goal is to accomplish what Cargo's alternative registries and source replacement does.

@yurivict, do maybe the new lake features already satisfy this request?


New version: Tenpi.VideoPlayer version 0.1.9

Pull request has been created with komac v2.6.0 :rocket:
Microsoft Reviewers: Open in CodeFlow

@wingetbot run


Create Travis-CI configuration for CI builds, add badge to README and provide Linux build instructions

Hi,
This is really three changes in one so it might make sense to merge this pull request as separate commits instead of squashing them. Here's a summary:

Adds a .travis.yml to enable Travis-CI builds for Tableau Log Viewer: currently this is limited to Linux only, but I intend to add OS X and Windows CI builds (most likely using Travis-CI and AppVeyor) soon
Adds a Travis-CI build badge in README.md
Adds Linux build instructions in INSTALL.md

Note that the Travis-CI build badge displays the status of the build in the dev branch. This seems to make most sense since this is the branch that pull requests are integrated into. To enable Travis-CI builds, you'll need to log into Travis-CI and enable builds on this repository. Since I'm not a member of the "tableau" team, I can't do this for you!
I think this will be useful as an additional pull request quality gate: I believe GitHub will automatically trigger a Travis-CI build for new pull requests once enabled and so we can tell at a glance if changes break the build or not. In the future, it should be possible to use the build output from Travis-CI as the official binary distribution of this tool too, which will eliminate the requirement for customers to build the code from source at all. When I get a chance, I'll look at how to publish artifacts from Travis-Ci back to GitHub.
To see an example of Travis-CI in action on this repository, check out my fork of this project at https://github.com/rcook/tableau-log-viewer and the build results at https://travis-ci.org/rcook/tableau-log-viewer.
-Richard

There are still conflicts, do you need to get the latest changes of the dev branch?

Closing as we'll get another version of this soon.


linuxPackages.akvcam: init at 1.1.1

Motivation for this change
Add this driver to let webcamoid support more applications as an output.
In order to work, it needs a configuration file.
Currently, I use a hardcoded file, it would require work to produce a flexible module.
Things done

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

[X] NixOS
[ ] macOS
[ ] 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 nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[X] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[X] Fits CONTRIBUTING.md.

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

2 packages marked as broken and skipped:

linuxPackages_hardkernel_4_14.akvcam
linuxPackages_hardkernel_latest.akvcam

17 packages built:

linuxPackages-libre.akvcam
linuxPackages.akvcam (linuxPackages_5_4.akvcam)
linuxPackages_4_14.akvcam
linuxPackages_4_19.akvcam
linuxPackages_4_4.akvcam
linuxPackages_4_9.akvcam
linuxPackages_5_8.akvcam
linuxPackages_5_9.akvcam (linuxPackages_latest.akvcam)
linuxPackages_hardened.akvcam
linuxPackages_latest-libre.akvcam
linuxPackages_latest_hardened.akvcam
linuxPackages_latest_xen_dom0.akvcam
linuxPackages_latest_xen_dom0_hardened.akvcam
linuxPackages_testing_bcachefs.akvcam
linuxPackages_xen_dom0.akvcam
linuxPackages_xen_dom0_hardened.akvcam
linuxPackages_zen.akvcam

Thanks @freezeboy for the contribution.


[Request] Add JabRef

JabRef is an open source bibliography reference manager written in Java. JabRef is a good tool for managing your reference when you write academic papers or prepare course projects.
homepage

Duplicate of #3164

Why did you opened it again as soon as the first one was closed as duplicate??


Adds JNumber conversions

Ref https://github.com/mdedetrich/scalajson/pull/38
This implementation is loosely based on that of Circe.
JNumber is internally subdivided into package private seven datatypes JStringDecimal, JBigDecimal, JBigInt, JLong, JInt, JDouble, and JFloat corresponding to the input values passed into JNumber.apply(...). This allows performant roundtrip back to the original number types, if needed.
Note that the parser implementer now has an option of constructing JStringDecimal via JNumber(value: String) or doing the number parsing upfront.

Codecov Report

Merging #41 into master will decrease coverage by 18.61%.
The diff coverage is 23.46%.

@@             Coverage Diff             @@
##           master      #41       +/-   ##
===========================================
- Coverage   44.68%   26.07%   -18.62%
===========================================
  Files           5        5
  Lines         687     1097      +410
  Branches      133      170       +37
===========================================
- Hits          307      286       -21
- Misses        380      811      +431

Impacted Files
Coverage Δ

js/src/main/scala/scalajson/ast/JValue.scala
0% <0%> (ø)
:arrow_up:

...s/src/main/scala/scalajson/ast/unsafe/JValue.scala
0% <0%> (ø)
:arrow_up:

...m/src/main/scala/scalajson/ast/unsafe/JValue.scala
47.29% <39.61%> (-12.22%)
:arrow_down:

jvm/src/main/scala/scalajson/ast/JValue.scala
48.86% <40.6%> (-24.53%)
:arrow_down:

shared/src/main/scala/scalajson/ast/package.scala
16.88% <0%> (-40.89%)
:arrow_down:

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 99ee084...5f215c9. Read the comment docs.

IMHO representing JSON numbers via an ADT like this is the way to go.
What's you rationale for making the sub types package private instead of opening up the ADT altogether?

@SethTisue I suppose if the internals change then you are not breaking it for end users?

IMHO this whole effort is about not changing anything, ever.
We need to get it almost 100% right from the beginning.
This is not a library that receives new features or other upgrades over time.
It's supposed to deliver a minimal and consistent JSON ADT, not more and not less.
That's why simplicity and clarity is quite important in my view. Any special construct, hidden fields, non-case-classes or other "tricks" should be extremely well justified.

This is not a library that receives new features or other upgrades over time.
It's supposed to deliver a minimal and consistent JSON ADT, not more and not less.

Completely agree, which is why I am being very careful about a release. Will need @eed3si9n , and possibly @travisbrown to clarify why they are private

What's you rationale for making the sub types package private instead of opening up the ADT altogether?

Unlike Json4s's AST where you get either case class JLong(num: Long) or case class JDecimal(num: BigDecimal) representing the actual number value, we have seven that represents how the JNumber was constructed. The reason why it's there is to avoid conversion cost, and it's almost irrelevant to the consumer of the AST.
For example, suppose I am getting "Twitter fave count", and I assume the number can be represented as Long, depending on the parser the number might be stored as JStringDecimal, JBigDecimal, JLong or JInt etc. The only thing I need to be concerned is can I safely convert it to Long.

IMHO this whole effort is about not changing anything, ever.
We need to get it almost 100% right from the beginning.

I agree with the general idea, but we should relax this to keeping strict binary compatibility of the exposed classes.
Other goals are correctness, user-friendliness, and performance. Hiding often allows us to do performance tricks without compromising on the usability of the surface API.

@eed3si9n Thanks, I will have a look at this today/tomorrow

Other goals are correctness, user-friendliness, and performance. Hiding often allows us to do performance tricks without compromising on the usability of the surface API.

Yes, I fully agree. However, in my view this project is a bit different from ordinary libraries as it provides almost only structure and very little logic (at least I would consider this a goal).
If there's no logic we don't really need to hide anything. Also, if the structure is right there is no need for "performance tricks".
My concern with hiding the internal ADT structure is that there will definitely be people that will want to access the underlying inner structure, if only to perf-optimize certain cases on their layer.
If we come up with right structure anyone who doesn't care about details can simply rely on the JNumber interface. But if I really wanted to I could drop down to the fully specialized JNumber ADT and apply dedicated handling to each case.
(One example that come to mind would be an optics library that provides AST transformation/manipulation facilities from the outside. Not being able to see the real underlying structure could be a disadvantage.)

@sirthias After talking with @Ichoran on Gitter, I am coming to the idea that both #38 and this PR have feature creeped.
I think you're right that we should try to keep the AST open, and in our case plain String (with regex check).

Regarding #38 , this was an attempt to provide performant JNumber conversions without altering the current API design too much, hence why the JNumber constructor was largely unchanged and we are using a bitflag

Closing this in favor of #42


play-scala-tls certificates expired on 25th Aug 2019

$ cd play-samples/play-scala-tls-example/scripts
$ openssl x509 -in example.com.crt  -text
Certificate:
    Data:
        Version: 3 (0x2)
        Serial Number: 63408077 (0x3c787cd)
    Signature Algorithm: ecdsa-with-SHA256
        Issuer: C=US, ST=California, L=San Francisco, O=Example Company, OU=Example Org, CN=exampleCA
        Validity
            Not Before: May 27 16:58:42 2019 GMT
            Not After : Aug 25 16:58:42 2019 GMT
        Subject: C=US, ST=California, L=San Francisco, O=Example Company, OU=Example Org, CN=example.com
        Subject Public Key Info:
            Public Key Algorithm: id-ecPublicKey
                Public-Key: (256 bit)
                pub:
                    04:a8:b1:a8:15:bd:df:88:f4:71:cf:e9:18:f1:70:
                    a3:fa:23:a5:d2:92:f3:65:04:fc:24:b0:89:9a:7e:
                    57:74:09:8e:e9:6f:ee:e1:8d:1f:e0:94:40:74:6e:
...

This causes tests to fail:

[info] ServerSpec:
[info] Server
[info] - should work fine over https *** FAILED ***
[info]   The future returned an exception of type: java.net.ConnectException, with message: General SSLEngine problem. (ServerSpec.scala:40)
[info] ScalaTest
[info] Run completed in 5 seconds, 545 milliseconds.
[info] Total number of tests run: 1
[info] Suites: completed 1, aborted 0
[info] Tests: succeeded 0, failed 1, canceled 0, ignored 0, pending 0
[info] *** 1 TEST FAILED ***
[error] Failed: Total 1, Failed 1, Errors 0, Passed 0

This only surfaces now because we haven't launched a single build in the last 26+ days.

Fixed in #54


Update device-onboarding-offboarding-macos-jamfpro.md

The link was broken, updated the correct link.
@chrfox

@JohanFreelancer9 - kindly do the copy editor review for this PR. Thanks

@tiburd Please merge this PR. Thanks

@nihendle
Hi, Nir,
This pull request was opened in the public repo. PMs and content developers should work in the private repo, as described in the authoring guide. We reviewed and merged this PR but encourage you to use the private repo for future contributions to Microsoft Docs.
Thanks!


NEW Get anchor links from elemental pages

Adds anchor links in HTMLText fields on an elemental block to the "anchor on a page" link form in WYSIWYG fields.
The new method should be overridden for blocks that need to add additional anchors or exclude some anchors from this base list.
An argument can be made that this should render the block and use that to parse out any anchors - but the approach this PR takes is more in line with how this is currently done in SiteTree::getAnchorsOnPage().
Also updated the theHasAContentElementWithContent fixture context regex to allow arbitrary html content in the test fixture.
Dependencies
The following related PRs must be merged in first in order for tests to pass

https://github.com/silverstripe/silverstripe-frameworktest/pull/106
https://github.com/silverstripe/silverstripe-cms/pull/2741

Parent Issues

https://github.com/silverstripe/silverstripe-elemental/issues/186

I've taken over this PR. Marking this as draft so nobody merges it just yet - the functionality is all there but I still need to add test coverage.

I'm testing these changes and I can't seem to get the result (the list of anchors on the page) in any way. Perhaps I'm doing something wrong. Here are the steps I follow.

I create pages using the following template.

<?php

namespace {

    use DNADesign\Elemental\Models\ElementalArea;
    use SilverStripe\Forms\HTMLEditor\HTMLEditorField;
    use SilverStripe\CMS\Model\SiteTree;

    class Page extends SiteTree
    {
        private static $db = [
            'Summary'     => 'HTMLText',
        ];
        private static $has_one = [
            'BlocksMain'         => ElementalArea::class,
            'BlocksRight'        => ElementalArea::class,
            'BlocksFooter'       => ElementalArea::class,
        ];

        public function getCMSFields()
        {
            $fields = parent::getCMSFields();

            $summary = HtmlEditorField::create('Summary', 'Summary', false);
            $summary->setRows(5);
            $summary->setDescription('Test Description');

            $fields->addFieldsToTab('Root.Main',$summary, 'Summary');
            return $fields;
        }
    }
}

In each ElementalArea I add a Content block.
In the Content block I add content
My_linkTest

Tests2
 via Source code.
Save and publish ElementalArea.
In the Content Editor for the Summary, I click on "Insert Link" and select "Anchor on a page".
In the popup window that opens, in the "Select a page" I select this page with which I am working at the moment.
But then In the "Anchor" drop-down list, I see "No results found".

I also tried adding an anchor, which is placed in the Summary, to the ElementalArea Content Block. And I even refreshed the page. Still don't have the anchors.

@sabina-talipova Note in the PR functionality added to the ElementalPageExtension - it is my expectation that people will be using that extension when adding elemental blocks to their pages. Perhaps this PR should also add some documentation that advises people who (for whatever reason) don't use that extension to also add this new functionality to their pages directly?

@sabina-talipova
Can you please try following these steps and let me know if that works?

Create a new page with elemental blocks
On the new page, create a content elemental block
Add some anchor link to the content elemental block.
Save the page
In that, or any other block (including on a different page), click "Insert Link", then "Anchor on a page" and select the above page.

The anchor link you added should be in the anchors dropdown list.
I suspect the issue you were having is that new anchors when you don't save the page aren't showing up - that's a defect with the way I implemented #2741 which I will need to fix - but in the meantime the main functionality here should work.

@GuySartorelli , it still doesn't work and it looks like it doesn't invoke updateAnchorsOnPage method from ElementalPageExtension. ElementalPageExtension adds ElementalArea on the Page, but doesn't run updateAnchorsOnPage. Probably, I should do something else.

As discussed on slack, that sounds like a problem with the way I've changed the javascript in https://github.com/silverstripe/silverstripe-cms/pull/2741, which means I'm not quite done yet. Once you've finished your review, assign this back to me and I'll see what I can do about that.

@GuySartorelli ,
I've finished testing in my local environment. Nice work! All works perfectly fine even with changes that I merged for TinyMCE_sslink-anchor.js. BUT with a few tiny enchincement that should be done.
First of all, in SiteTree class in the method getAnchorsOnPage() would be nice to implement your huck for DBHTMLText field. Since currently in this method we are looking for $this->Content (https://github.com/silverstripe/silverstripe-cms/blob/77b5bfa497d5026ffc806f1071a6c5d04d65bfb4/code/Model/SiteTree.php#L3338). But if I have other field's name , for example Summary, we skip it. That was the reason for my issue, I hadn't Content field at all. So if you implement your changes, something like this, it helps us to solve problem where we have more then one DBHTMLText on the page with different names.
        $anchors = [];
        $allFields = DataObject::getSchema()->fieldSpecs($this);
        $anchorRegex = "/\\s+(?<attr>name|id)\\s*=\\s*([\"'])(?<token>[^\\2\\s>]*?)\\2|\\s+(name|id)\\s*=\\s*([^\"']+)[\\s +>]/im";

        foreach ($allFields as $field => $fieldSpec) {
            $fieldObj = $this->owner->dbObject($field);
            if ($fieldObj instanceof DBHTMLText) {
                $parseSuccess = preg_match_all($anchorRegex, $fieldObj->getValue() ?? '', $matches);
                if ($parseSuccess >= 1) {
                    $fieldAnchors = array_values(array_filter(
                        array_merge($matches['token'])
                    ));
                    $anchors = array_merge($anchors, $fieldAnchors);
                }
            }
        }
        $anchors = array_unique($anchors);

        $this->extend('updateAnchorsOnPage', $anchors);

        return $anchors;

But it is just suggestion.

@GuySartorelli ,
I've finished testing in my local environment. Nice work! All works perfectly fine even with changes that I merged for TinyMCE_sslink-anchor.js. BUT with a few tiny enchincement that should be done.
First of all, in SiteTree class in the method getAnchorsOnPage() would be nice to implement your huck for DBHTMLText field. Since currently in this method we are looking for $this->Content (https://github.com/silverstripe/silverstripe-cms/blob/77b5bfa497d5026ffc806f1071a6c5d04d65bfb4/code/Model/SiteTree.php#L3338). But if I have other field's name , for example Summary, we skip it. That was the reason for my issue, I hadn't Content field at all. So if you implement your changes, something like this, it helps us to solve problem where we have more then one DBHTMLText on the page with different names.
        $anchors = [];
        $allFields = DataObject::getSchema()->fieldSpecs($this);
        $anchorRegex = "/\\s+(?<attr>name|id)\\s*=\\s*([\"'])(?<token>[^\\2\\s>]*?)\\2|\\s+(name|id)\\s*=\\s*([^\"']+)[\\s +>]/im";

        foreach ($allFields as $field => $fieldSpec) {
            $fieldObj = $this->owner->dbObject($field);
            if ($fieldObj instanceof DBHTMLText) {
                $parseSuccess = preg_match_all($anchorRegex, $fieldObj->getValue() ?? '', $matches);
                if ($parseSuccess >= 1) {
                    $fieldAnchors = array_values(array_filter(
                        array_merge($matches['token'])
                    ));
                    $anchors = array_merge($anchors, $fieldAnchors);
                }
            }
        }
        $anchors = array_unique($anchors);

        $this->extend('updateAnchorsOnPage', $anchors);

        return $anchors;

But it is just suggestion.

in SiteTree class in the method getAnchorsOnPage() would be nice to implement your solution for DBHTMLText field.

I won't make this change as a part of fixing the issue here. It could be a good enhancement so feel free to an issue for it in silverstripe/silverstripe-cms - but that is a separate problem to the issue this PR is fixing. That problem will exist for situations where people already have fields such as Summary even without elemental installed.

Note the duplicated anchor-in-page regex seems to pick up the "id" part of image-shortcodes (and possibly other shortcodes) as well.


Allow message or message_text to trigger iOS Alert

:+1:

I will roll this out later today after #83

thanks!

:+1:

how do I know if there is a new version and what that version is?


fix memory leak

Use co_delay() cause a circular reference between COTimeChan and CODispatchTimer

 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.


update dependencies

I've checked it with latest angular-bootstrap and it works fine
Also please don't forget to create tag (necessary for bower)

There were issues with 0.11
https://github.com/zhaber/angular-js-bootstrap-datetimepicker/issues/30

Thanks for letting me know. Updated PR.

Please create also tag 0.12.0 because rails-assets doesn't see new releases


Add redux-cut to middleware section

Is this really cheeky? 😜

Let’s wait for some more users, issues and versions?

Yeah fair I thought you might say something like that! No worries!

Don’t be shy to submit again. I just think some more real world usage would be needed so we don’t recommend something that might change the API soon.

FWIW, I do have it listed in my Redux addons catalog: https://github.com/markerikson/redux-ecosystem-links/blob/master/middleware.md#action-grouping-and-interception


[All Hosts] (testing) update Edge loopback instructions

Docs Build status updates of commit 527adf1:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/testing/sideload-office-add-ins-for-testing.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel


Train accuracy is about 20% when fine tune Roberta

I follow the fine tune scripts on the github, and the train accuracy about commonsense task is about 20%. What's wrong that I don't notice ?

having the same problem here...

Hi @myleott , is there any idea about it?  Thanks!


Improve accessibility

This adds alt texts and labels to images and links. In addition to that, the detail view of the individual projects can now be accessed using keyboard.

This looks good to me. Thank you for the PR!


⚠ Bofrost.fr has degraded performance

In 6a34c09, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 1484 ms

Resolved: Bofrost.fr performance has improved in 9789e7a after 6 minutes.


Invalid argument(s): An argument matcher (like any()) was either not used

Getting error:
Invalid argument(s): An argument matcher (like any()) was either not used as an immediate argument to Symbol("get") (argument matchers can only be used as an argument for the very method being stubbed or verified), or was used as a named argument without the Mocktail "named" API (Each argument matcher that is used as a named argument needs to specify the name of the argument it is being used in. For example: when(() => obj.fn(x: any(named: "x")))).
class MockHttpClient extends Mock implements http.Client {}

void main() {
  late MockHttpClient mockHttpClient;
  late HomeRemoteDataSourceImplementation homeRemoteDataSourceImplementation;

  setUp(() {
    mockHttpClient = new MockHttpClient();
    homeRemoteDataSourceImplementation =
        new HomeRemoteDataSourceImplementation(client: mockHttpClient);
  });

  group("getMedicalRecords", () {
    test(("should perform a GET request on a URL with application/json header"),
        () {
      when(() => mockHttpClient.get(Uri.parse(any()),
          headers: any(named: "headers"))).thenAnswer(
        (_) async =>
            http.Response(fixtures("Home/fixtures/medical_record.json"), 200),
      );

      homeRemoteDataSourceImplementation.getMedicalRecord();

      verify(
        () => mockHttpClient.get(
          Uri.parse(
              "https://run.mocky.io/v3/30579b01-75f8-4c9f-ac83-c98b5cfe1d2e"),
          headers: {
            "Content-Type": "application/json",
          },
        ),
      );
    });
  });
}

Your matchers on the mockHttpClient.get call are mixed. Replace Uri.parse(any()) with any() like so:
when(() => mockHttpClient.get(any(),
          headers: any(named: "headers"))).thenAnswer(
        (_) async =>
            http.Response(fixtures("Home/fixtures/medical_record.json"), 200),

Did this as per your suggestion:
when(() => mockHttpClient.get(
            // Uri.parse(// cant use any as null exception
            //     "https://run.mocky.io/v3/30579b01-75f8-4c9f-ac83-c98b5cfe1d2e"),
            any(),
            headers: any(named: "headers"),
          )).thenAnswer(
        (_) async => http.Response(
            fixtures("features/Home/fixtures/medical_record.json"), 200),
      );

Bad state: A test tried to use any or captureAny on a parameter of type Uri, but
registerFallbackValue was not previously called to register a fallback value for Uri.
To fix, do:
void main() {
  setUpAll(() {
    registerFallbackValue(/* create a dummy instance of `Uri` */);
  });
}

Yes, you need a "Fake" Uri as described in the README.  There's a good example in the bloc library weather example's meta_weather_api_client_test
To get it going you would want to add something like:
class FakeUri extends Fake implements Uri {}
void main() {
    setUpAll(() {
      registerFallbackValue(FakeUri());
    });

/* ... your tests ... */
}

Yes, you need a "Fake" Uri as described in the README. There's a good example in the bloc library weather example's meta_weather_api_client_test
To get it going you would want to add something like:
class FakeUri extends Fake implements Uri {}
void main() {
    setUpAll(() {
      registerFallbackValue(FakeUri());
    });

/* ... your tests ... */
}

Hey, how can I create a registerFallbackValue for an enum ?

Yes, you need a "Fake" Uri as described in the README. There's a good example in the bloc library weather example's meta_weather_api_client_test
To get it going you would want to add something like:
class FakeUri extends Fake implements Uri {}
void main() {
    setUpAll(() {
      registerFallbackValue(FakeUri());
    });

/* ... your tests ... */
}

Hey, how can I create a registerFallbackValue for an enum ?

You can just pass any enum value, no need to create a fake.

Yes, you need a "Fake" Uri as described in the README. There's a good example in the bloc library weather example's meta_weather_api_client_test
To get it going you would want to add something like:
class FakeUri extends Fake implements Uri {}
void main() {
    setUpAll(() {
      registerFallbackValue(FakeUri());
    });

/* ... your tests ... */
}

Hey, how can I create a registerFallbackValue for an enum ?

You can just pass any enum value, no need to create a fake.

Bad state: A test tried to use anyorcaptureAnyon a parameter of typeMETHOD, but registerFallbackValue was not previously called to register a fallback value for METHOD`.
To fix, do:
void main() {
  setUpAll(() {
    registerFallbackValue(/* create a dummy instance of `METHOD` */);
  });
}

This instance of METHOD will only be passed around, but never be interacted with.
Therefore, if METHOD is a function, it does not have to return a valid object and
could throw unconditionally.
If you cannot easily create an instance of METHOD, consider defining a Fake:
class MyTypeFake extends Fake implements MyType {}

void main() {
  setUpAll(() {
    registerFallbackValue(MyTypeFake());
  });
}

This is the error I am getting while trying to pass the enum value

@n1h41 can’t you provide a minimal reproduction sample?

@n1h41 can’t you provide a minimal reproduction sample?

as you mentioned before I passed the enum value without creating the fake. It's working. Thanks

@raxahai I see that you've re-opened the issue. Do you still have an error? If so, can you provide minimal reproductive code (without external package dependencies besides mocktail)? Else can the issue be closed?

it happens when i am using any() inside Record

it happens when i am using any() inside Record

Closing this for now since the issue is quite old and there aren't any actionable next steps. If anyone is still facing an issue please file a new bug and provide a minimal reproduction sample and I'm happy to take a closer look 👍


spark and cli output different model

I notice that the binary model produced by spark is different from cli and python produce, because it write a int into output stream before call c api which is called by cli and python without any post process.
It cause the model produced by spark can't be loaded by python.
Now I solve it by dropping front 4 bytes in model which produced by spark before load it.
Do you have other solutions?

Not sure why, but according to @CodingCat this is by design?


ProcessHeader(string package) - Default revision

Some messages from some controllers doesn't contains revision, for example MID 0015:
00420015____________0022017-10-03:14:12:30
In this case message parsing fails because MID revision parsed as 0 and DataField doesn't contains data for revision 0.
Possible fix is to change default revision to 1 (Mid.cs:100)
Revision = (package.Length >= 11 && !string.IsNullOrWhiteSpace(package.Substring(8, 3))) ? Convert.ToInt32(package.Substring(8, 3)) : 1,

As I've seen, even when parsing revision equals to 0, it does change it to rev 1 when parsing the "body" of the package, like in ProcessDataFields()
protected virtual void ProcessDataFields(string package)
        {
            if (!RevisionsByFields.Any())
                return;

            int revision = HeaderData.Revision > 0 ? HeaderData.Revision : 1; //This part
            for (int i = 1; i <= revision; i++)
                ProcessDataFields(RevisionsByFields[i], package);
        }

Can you provide a package example that is failing?

Example package is in my comment it's MID 0015. I forgot to mention it's necessary to replace underscores (_) with spaces. In comment preview extra spaces were removed so I replaced them with underscores.

I'm fixing it on mid 0015 override.
Since making changes on MID class will change every Mid behavior (which is more likely to cause problems/bugs for people).
By now, doing this it will help you out with this problem and I will also study the possibility to modify it to every mid.

Already published on nuget gallery, please update it and close the issue once you check it solved!

As far as I understand open protocol revision 0 doesn't exist. In this case it's about way to parse received message from controller and message with revision 0 is always necessary to parse based on revision 1 format.
Anyway, your solution is ok :)

I found similar issue in MID 0061 (version 3.2.0)
I switched parameters in Pack method by mistake and I substribed to tightening revision 0.
Tightening data as result:
02310061<PHONE_NUMBER>03XXXXXXXXXXXXXXXXXXXXXXXXX04XXXXXXXXXXXXXXXXXXXXXXXXX050006001070000080000091101111120008501300092014000870150008731600000170013018000051900040202020-09-15:04:43:09212017-03-01:08:43:16222230004720285

I fixed it by modify Mid0061.CS like this
        protected override void ProcessDataFields(string package)
        {
            if (HeaderData.Revision == 0 || HeaderData.Revision == 1 || HeaderData.Revision == 999)
            {
                int revision = HeaderData.Revision > 0 ? HeaderData.Revision : 1;
                ProcessDataFields(RevisionsByFields[revision], package);
            }
            else
            (...)

Offtopic but I don't want to open new issue for typo in enum file :)
You have typo in file TorqueValuesUnit.cs, value number 3 is LBF_LN but correct should be LBF_IN like Pound-force inch

There are some weird controllers out there, usually the default revision is always 1, but they send empty sometimes.
Anyway, I'm going to think about handling this behaviour in the next versions.
Also, for your code, better use if(HeaderData.Revision < 1 || HeaderData.Revision == 999), better doing 2 operations than 3.

About the enum, I'm going to change it. However it's may cause break changes.
Thanks for submitting both issues

I'm not sure that was probably PF 3000 with controller sw version 10.5 SR4. I think problem was with tightening subscription (MID 0060). I subscribed to tighttening data with revision 0, by mistake :)
And about my code, sorry i was bit tired. I agree your version is better

Makes sense! Anyway, better handle it in the lib too.
Oh, and you don't need to excuse for that, it was just a clue. Also, don't push yourself too much!

Please note that the specification says:
"If the initial MID Revision (revision 1) is required there is three different ways to get it, either send three spaces or 000 or 001."
I.e. there is no revision 0.
1 is the first revision.

@ekdahl and @yanick76, I'm afraid that this might cause more issues than fixing ours.
I'm sure that there is no rev 0, and that is a fact. But i've seen some controller to cause bugs when you send revision.
For example, I've seen some cases that when you send KeepAlive with revision set like<PHONE_NUMBER>1_________ they simply cause an exception and kills the connection with integrator, but it works when sending 00209999____________.
Also, OpenProtocolInterpreter allows you to set revision 000, so you can send empty spaces instead of the revision number.
But this will have to be set manually.

Enhancement made at version 5.0.0, please check release notes: https://github.com/Rickedb/OpenProtocolInterpreter/releases/tag/5.0.0


Support @bonfhir packages in Expo on Android devices

We are currently have an issue with loading bonfhir libraries on Android devices with the intent of using @bonfhir/gluestack-ui in Expo projects. In order to solve this issue, we need to discover how to make android compatible with one of the js engines.
Current error "Unexpected token '?'", without a stack trace.
What we know:

the issue comes when we set jsEngine to jsc in the app.json of an expo project.
using hermes is not possible. related issue #139
the issue is present when on simulator and physical devices
bonfhir is es2022 and expo uses babel to transpile it to es5
error is not present on android using  v8, but v8 is not compatible with @bonfhir/query at the moment.

Funny thing - if I enable debug mode (by using Cmd+m), then the app is happy and it runs correctly. 😤

Funny thing - if I enable debug mode (by using Cmd+m), then the app is happy and it runs correctly. 😤

This somewhat makes sense as I just learned than when debugging, the javascript then run in the chrome browser and not the app shell - and since bonfhir runs OK in Chrome, this makes sense. Not a good lead though, just confirming that jsc is not great on Android RN.


Move admin api from /admin/{collection} to /collections/{collection}

In gitlab by @sfalquier on Apr 19, 2017, 10:32

In gitlab by @sfalquier on Apr 19, 2017, 11:30
mentioned in merge request !9

In gitlab by @sfalquier on Apr 19, 2017, 10:58
created branch 10-move-admin-api-from-admin-collection-to-collections-collection

In gitlab by @MohamedHamouGisaia on Apr 19, 2017, 13:52
closed via merge request !9

In gitlab by @MohamedHamouGisaia on Apr 19, 2017, 14:02
mentioned in commit b3a3a73f79c37db3f7cd687bd8694b5f7e3472b9


Add instructions to use flow-typed

Followup from https://github.com/facebookincubator/create-react-app/issues/72.
Requires https://github.com/flowtype/flow-typed/pull/467 to be merged first.

Alright! https://github.com/flowtype/flow-typed/pull/467 is merged so what is written here is true!

@sachse I'm now concentrating on #1152 that will make this PR unnecessary.

Closing in favor of #1152 that is almost there!


Added missing authors for Flow Framework's Blogpost

Description
Fixed for https://github.com/opensearch-project/project-website/pull/2698
Issues Resolved
[List any issues this PR will resolve]
Check List

[ ] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the BSD-3-Clause License.

@nateynateynate thanks for merging this. Do you have an idea when will be the next deployment to verify the changes?


Investigate Electronite

Let's investigate if Electronite is a good fit for us.

Is it maintained well?
Is having Graphite support useful and worth it?
Can it fit into our ecosystem?
Is there an electronite builder tool?

https://github.com/unfoldingWord/electronite

electronite is a drop in replacement for electron. Specifically, it is a fork of Electron that has the chromium UI patched to support Graphite Fonts.

it is updated only when uW does a major release of translationCore (and they therefore use a later version of electron) and it is a manual process of a couple of days. E.g. for v22+, at the moment there are electronite releases for '22.0.0-graphite', '22.0.1-graphite',  '22.0.3-graphite' but electron has further releases of '22.0.2', '22.1.0', '22.2.0', '22.2.1', '22.3.0', '23.0.0', '23.1.0' that electronite doesn't yet support. Note they skipped '22.0.2'. However, they are open to help with this.
you also must use electronite-packager with it, which is a fork of electron-packager. There is only one version of this '15.4.0-graphite' so far whereas electron-packager currently has these additinal versions '15.5.0', '15.5.1', '15.5.2', '16.0.0', '17.0.0', '17.1.0', '17.1.1'.

So we would need to change our build system to use electron-packager first, then electronite and electronite-packager could be dropped it.

I asked for more info about the state of adding Graphite features into Open Type (OT) to the WSTech team in Slack. I record important comments here:
Chris noted - "The only scripts that I can think of off hand that still require Graphite for proper rendering are Nastaliq and Lanna"
Martin noted that progress is "going slowly but not completely stuck". But they don't have any idea on the timeframe. "Yes it is still the plan and our desire for Harfbuzz to fully integrate Graphite in a way that all can use it and Awami [font] can just be used wherever."
Bobby noted - "A Tamil font that I worked on has both OT and Graphite (GR). I thought of removing the GR, as the OT with HarfBuzz (HB) is sufficient. But Paratext seems to have an older version of HB, and the font did not work well there with just OT. So I kept the GR, and it works well in Paratext. So Tamil does not require GR, but Tamil in Paratext does. Keep in mind the integration of Graphite is not into OT per se, but into HarfBuzz, which is an OT engine. Other OT engines, such as DirectWrite from Microsoft, Lipka from Adobe, and Apple has one, are not getting the Graphite integration."

Conclusion: for now we won't introduce electronite into Paranext.

Thank you Ira!  This is really helpful.  Awesome.


'ConfigDict' object has no attribute 'train_cfg' in colab tutorial

Thanks for your error report and we appreciate it a lot.
Checklist

I have searched related issues but cannot get the expected help.
I have read the FAQ documentation but cannot get the expected help.
The bug has not been fixed in the latest version.

Describe the bug
I am getting started using mmdetection using de colab tutorial on github: mmdetection/demo/MMDet_Tutorial.ipynb
Without making any modifications to the code I get the following error in Train a new detector section:
AttributeError: 'ConfigDict' object has no attribute 'train_cfg'
Reproduction

What command or script did you run?

from mmdet.datasets import build_dataset
from mmdet.models import build_detector
from mmdet.apis import train_detector

# Build dataset
datasets = [build_dataset(cfg.data.train)]

# Build the detector
model = build_detector(
    cfg.model, train_cfg=cfg.train_cfg, test_cfg=cfg.test_cfg)
# Add an attribute for visualization convenience
model.CLASSES = datasets[0].CLASSES

# Create work_dir
mmcv.mkdir_or_exist(osp.abspath(cfg.work_dir))
train_detector(model, datasets, cfg, distributed=False, validate=True)

Did you make any modifications on the code or config? Did you understand what you have modified?
I did not make any modifications to the code.
What dataset did you use?

Environment

Please run python mmdet/utils/collect_env.py to collect necessary environment information and paste it here.
You may add addition that may be helpful for locating the problem, such as

How you installed PyTorch [e.g., pip, conda, source]
Other environment variables that may be related (such as $PATH, $LD_LIBRARY_PATH, $PYTHONPATH, etc.)

Error traceback
If applicable, paste the error trackback here.
AttributeError                            Traceback (most recent call last)
<ipython-input-38-63f0da5758a7> in <module>()
      9 # Build the detector
     10 model = build_detector(
---> 11     cfg.model, train_cfg=cfg.train_cfg, test_cfg=cfg.test_cfg)
     12 # Add an attribute for visualization convenience
     13 model.CLASSES = datasets[0].CLASSES

1 frames
/usr/local/lib/python3.7/dist-packages/mmcv/utils/config.py in __getattr__(self, name)
     41         else:
     42             return value
---> 43         raise ex
     44
     45

AttributeError: 'ConfigDict' object has no attribute 'train_cfg'

Bug fix
If you have already identified the reason, you can provide the information here. If you are willing to create a PR to fix it, please also leave a comment here and that would be much appreciated!

Please refer to #4688


Replace broken Arbitrum Sepolia RPC

Description
Replace broken Arbitrum Sepolia RPC
Backward compatibility
Yes

ty for doing this - i had this in a local branch but forgot to put the PR up!


sample code for demo

Hi, I found the demo here https://vue-tel-input.iamstevendao.com/ already usable for my usage.
However, I could not reproduce the same layout.
Could I have some sample code of the demo for my reference?
Thanks in advance

Here you go: https://github.com/iamstevendao/vue-tel-input/blob/master/docs/.vuepress/theme/components/Home.vue


Unable to access Lidarr (http / https mismatch?)

Yeah, i am using it, and it shows? Sorry to be such a PIA.
I am trying the "generate/artist_top_tracks/all/" API endpoint, and it seems it cannot connect to Lidarr because of an HTTP / HTTPS.
This is my docker-compose:
      - LIDARR_ENABLED=1
      - LIDARR_IP=xxx.mydomain.com
      - LIDARR_PORT=443
      - LIDARR_USE_SSL=1
      - LIDARR_BASE_API_PATH=/lidarr
      - LIDARR_TOKEN=xxx

(the LIDARR_USE_SSL=1 i found looking in the source code)
But it seems it's not working:
2024-09-18 10:20:35 ERROR    <class 'pyarr.exceptions.PyarrBadRequest'> subsonic_helper.py 153
Traceback (most recent call last):
  File "/home/user/spotisub/./spotisub/helpers/subsonic_helper.py", line 153, in write_playlist
    is_monitored = lidarr_helper.is_artist_monitored(artist_name_spotify)
  File "/home/user/spotisub/./spotisub/helpers/lidarr_helper.py", line 29, in is_artist_monitored
    artists_list = lidarr_client.get_artist()
  File "/home/user/.local/lib/python3.10/site-packages/pyarr/lidarr.py", line 116, in get_artist
    return self._get(
  File "/home/user/.local/lib/python3.10/site-packages/pyarr/request_handler.py", line 91, in _get
    response = _process_response(res)
  File "/home/user/.local/lib/python3.10/site-packages/pyarr/request_handler.py", line 233, in _process_response
    raise PyarrBadRequest(f"Bad Request, possibly a bug. {str(res.content)}")
pyarr.exceptions.PyarrBadRequest: Bad Request, possibly a bug. b'\r\n<title>400 The plain HTTP request was sent to HTTPS port</title>\r\n\r\n
400 Bad Req
uest
\r\nThe plain HTTP request was sent to HTTPS port\r\n
nginx/1.26.2\r\n\r\n\r\n'

It seems a mismatch in LidarrAPI?
I can see you append "https" correctly, but it seems the library still open an HTTP connection

Yeah, i am using it, and it shows? Sorry to be such a PIA.
I am trying the "generate/artist_top_tracks/all/" API endpoint, and it seems it cannot connect to Lidarr because of an HTTP / HTTPS.
This is my docker-compose:
      - LIDARR_ENABLED=1
      - LIDARR_IP=xxx.mydomain.com
      - LIDARR_PORT=443
      - LIDARR_USE_SSL=1
      - LIDARR_BASE_API_PATH=/lidarr
      - LIDARR_TOKEN=xxx

(the LIDARR_USE_SSL=1 i found looking in the source code)
But it seems it's not working:
2024-09-18 10:20:35 ERROR    <class 'pyarr.exceptions.PyarrBadRequest'> subsonic_helper.py 153
Traceback (most recent call last):
  File "/home/user/spotisub/./spotisub/helpers/subsonic_helper.py", line 153, in write_playlist
    is_monitored = lidarr_helper.is_artist_monitored(artist_name_spotify)
  File "/home/user/spotisub/./spotisub/helpers/lidarr_helper.py", line 29, in is_artist_monitored
    artists_list = lidarr_client.get_artist()
  File "/home/user/.local/lib/python3.10/site-packages/pyarr/lidarr.py", line 116, in get_artist
    return self._get(
  File "/home/user/.local/lib/python3.10/site-packages/pyarr/request_handler.py", line 91, in _get
    response = _process_response(res)
  File "/home/user/.local/lib/python3.10/site-packages/pyarr/request_handler.py", line 233, in _process_response
    raise PyarrBadRequest(f"Bad Request, possibly a bug. {str(res.content)}")
pyarr.exceptions.PyarrBadRequest: Bad Request, possibly a bug. b'\r\n<title>400 The plain HTTP request was sent to HTTPS port</title>\r\n\r\n
400 Bad Req
uest
\r\nThe plain HTTP request was sent to HTTPS port\r\n
nginx/1.26.2\r\n\r\n\r\n'

It seems a mismatch in LidarrAPI? I can see you append "https" correctly, but it seems the library still open an HTTP connection

I will try with https and report back, I haven't testet yet lidarr integration with https.

Yeah, i am using it, and it shows? Sorry to be such a PIA.
I am trying the "generate/artist_top_tracks/all/" API endpoint, and it seems it cannot connect to Lidarr because of an HTTP / HTTPS.
This is my docker-compose:
      - LIDARR_ENABLED=1
      - LIDARR_IP=xxx.mydomain.com
      - LIDARR_PORT=443
      - LIDARR_USE_SSL=1
      - LIDARR_BASE_API_PATH=/lidarr
      - LIDARR_TOKEN=xxx

(the LIDARR_USE_SSL=1 i found looking in the source code)
But it seems it's not working:
2024-09-18 10:20:35 ERROR    <class 'pyarr.exceptions.PyarrBadRequest'> subsonic_helper.py 153
Traceback (most recent call last):
  File "/home/user/spotisub/./spotisub/helpers/subsonic_helper.py", line 153, in write_playlist
    is_monitored = lidarr_helper.is_artist_monitored(artist_name_spotify)
  File "/home/user/spotisub/./spotisub/helpers/lidarr_helper.py", line 29, in is_artist_monitored
    artists_list = lidarr_client.get_artist()
  File "/home/user/.local/lib/python3.10/site-packages/pyarr/lidarr.py", line 116, in get_artist
    return self._get(
  File "/home/user/.local/lib/python3.10/site-packages/pyarr/request_handler.py", line 91, in _get
    response = _process_response(res)
  File "/home/user/.local/lib/python3.10/site-packages/pyarr/request_handler.py", line 233, in _process_response
    raise PyarrBadRequest(f"Bad Request, possibly a bug. {str(res.content)}")
pyarr.exceptions.PyarrBadRequest: Bad Request, possibly a bug. b'\r\n<title>400 The plain HTTP request was sent to HTTPS port</title>\r\n\r\n
400 Bad Req
uest
\r\nThe plain HTTP request was sent to HTTPS port\r\n
nginx/1.26.2\r\n\r\n\r\n'

It seems a mismatch in LidarrAPI? I can see you append "https" correctly, but it seems the library still open an HTTP connection

I haven't tested lidarr using https
I am using https://github.com/totaldebug/pyarr to connect to Lidarr, not sure if it supports HTTPS tho
Reading their docs I haven't found anything about https or ssl, guess i am gonna try to open an issue on their github

I have opened an issue on their github:
https://github.com/totaldebug/pyarr/issues/178

when i have time i will expose lidarr on http internally, i hope they will shed some light.

Update: i exposed the lidarr API directly on port 8686 on HTTP locally and it works, it seems it's really HTTPS not being supported by PyArr... hope they add support! Meanwhile, let's keep this open for future reference maybe?

Update: i exposed the lidarr API directly on port 8686 on HTTP locally and it works, it seems it's really HTTPS not being supported by PyArr... hope they add support! Meanwhile, let's keep this open for future reference maybe?

Let's just keep this issue open and wait an answer from the PyArr team, if they say that https isn't supported I'll just close this as "WON'T DO"


Feat/plmg-card

Closes: DEVELOP: Card

add Card component
add storybook stories
add accessibility tests
add spec tests
add React example

Oops I forgot storybook stuff, ok this isn't finished

GitHub checks will be happy once https://github.com/DuckyTeam/Plumage/pull/33 is merged

This is ready for review, waiting for an answer from the designer reagarding min-width and max-width


Text email with colon on first line results in empty email body

An email generated like this
val email = Email.text(
  From(...),
  To(...),
  Subject(...),
  Body.Ascii("foo: bar")
)
client.send(email)

results in an empty email message body because RFC822 interprets the body as a header. The workaround is to insert a newline before any body text that has a : on the first line:
val safeEmail = if (email.body.exists(_.value.takeWhile(_ != '\n').contains(':'))) {
  email.copy(body = email.body.map(b => Body.Ascii("\n" + b.value)))
} else {
  email
}
client.send(safeEmail)

@lukestewart13  yes sending a empty line after headers fixed this problem.  Fixed now 😉
thank you for reporting it

@lukestewart13  yes sending a empty line after headers fixed this problem.  Fixed now 😉
thank you for reporting it

fixed

fixed


Document how to get Faucet API key

As community of Cardano Developers grows, the need for testnet Ada may increase.
There is a faucet here: https://docs.cardano.org/cardano-testnet/tools/faucet
Unfortunately the process of issuing API keys is undocumented.
Would it be possible to add a direct link where to apply?
Possibly a form to fill in?

@mgajda yes it is definitely possible to add such a link & we could do this immediately, but I don't have that information & don't know who does.  There are a couple threads on this subject on the Cardano Forum which ask this question, but none provide an answer: going back over a year & remaining unanswered for the new testnets.
copying some people across the board who may know: @katomm @rdlrt @fill-the-fill @oversize @KtorZ

Hey,
unfortunately i don't know. I was in the same situation a few weeks ago (asking myself where would i get such a key). @mgajda are you needing the api key for a specific reason or just asking out of curiosity? You can get a bunch of ada once a day and the key would only allow you to request more than once a day not to receive more ada.I do agree though that if there is the option to provide a key, it should be also mentioned how/where to get one.
However, i think that having the faucet be available to get drained huge amounts of ada from by basically everyone all the time, might put the network in a struggle where more and more ada is locked in dev/test keys that are either lost or otherwise non accessible anymore. But that is a discussion that should be held by the people operating the testnet which currently is IOHK.

The API key is currently part of pinned message in IO testnet channel on discord and telegram. It is meant to be used only by participants of those networks, but agree that it's not obvious where to locate the groups.
As regards the actual funds, if the intention is to test creation of blocks - you have delegation option wherein upon requesting from faucet, a new account will be provisioned and provide you 1M delegation against your pool registered. Thus, there shouldn't really be a need to abuse faucets (has happened a lot in the past networks) unless doing tx spam tests and so on

My reading of the discussion so far is:

We could copy the API instructions from the "IO testnet channel" on Discord or Telegram into the testnet page here on the Dev Portal.
We should probably not, because that's not the way the faucet distribution was designed to work in the general case.

@mgajda are you satisfied with knowing the answer, enough for this to be closed?  I understand a case could be made to publicise a deeper level of the faucet API, but I believe it would encourage pathological behaviour like hoarding testnet tokens & therefore that things should be left the way they are.


Hide clips/snapshots button for Birdseye

I guess title says all. There won't be clips/snapshots for birdseye.

Hi @felipecrs -- thanks for the PR! This is reasonable, but my plan actually is to support clips and snapshots for birdesye. We can do this after https://github.com/dermotduffy/frigate-hass-card/issues/233 (which in turn depends on changes to the integration). Once that's done, you could configure the underlying cameras you want to show up on clips/snapshots (or all I suppose) and they would, theoretically, all merge together.
Make sense?
Regardless, I'm happy to merge this PR in the meantime as it's simple and could be reversed once the above was ready.

Oh, nice! That would be a lot better, and as you said we can take this for v3 and let that for let's say v4, or after.

@dermotduffy but if you prefer not to merge, I totally respect your decision. All fine! As always thanks for the amazing card!


Please Add Windows Platform Support

Thank you for developing this library. Could you please help add Windows support?

I found that the version on pub.dev doesn't support Windows, but the latest version on GitHub does support it.


expose bookmarks activity params

Task/Issue URL: https://app.asana.com/0/488551667048375/1206009971135588/f
Description
Expose bookmarks activity params to allow navigation from other modules
Steps to test this PR
Feature 1

[ ]
[ ]

UI changes

Before
After

!(Upload before screenshot)
(Upload after screenshot)

Current dependencies on/for this PR:

develop

PR #3904   👈

This stack of pull requests is managed by Graphite.


Your update yesterday made this plugin cause opening the bank slow down runelite

Your update yesterday made this plugin cause opening the bank slow down runelite,
Everytime i click on the bank there is a connection lost and my client lags until i restart it.

ok ill try and fix this, might have to revert back to previous update till solution... i wasnt having these issues prior to upload but now im noticing a few... probably gonna need to try to something better before this type of update thanks and sorry

its no big deal, i have just uninstalled the plugin <3


Second time opening context menu (when still shown) does not work correctly

If the context menu is open and then you try to open the context menu again on another item e.g., it will shortly be opened but directly closed afterwards... probably related to animation.

Will this be fixed?

Will this be fixed?

At least we will provide some workaround
Duplicate to #28


⚠ Bofrost.de has degraded performance

In 1c5424a, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 3006 ms

Resolved: Bofrost.de performance has improved in e173ba9 after 55 minutes.


Support for UIStoryboardSegue

Hi,
Is it possible to use an UIStoryboardSegue from a UIButton in Storyboard? It seems that InterfaCSS blocks my segue.
Code:
-(void)loadView
{
    [super loadView];
    self.view = [ISSViewBuilder rootViewWithStyle:@"sliderMainView" andSubViews:^NSArray *{
        return @[self.okBtn = [ISSViewBuilder buttonWithId:@"onboardingOkButton"]];
    }];

}

Well, what seems to happen here is that you are completely replacing
whatever you have set up in the storyboard (including segues) with a
different view hierarchy.
So the question is what your are tying to do here. If you only want to
assign an element id or style class to the views, you can do that directly
in the storyboard (via runtime attributes).
Cheers,
Tobias
On Friday, 16 October 2015, bangerang<EMAIL_ADDRESS>wrote:

Hi,
Is it possible to use an UIStoryboardSegue from a UIButton in Storyboard?
It seems that InterfaCSS blocks my segue.
Code:
-(void)loadView
{
[super loadView];
self.view = [ISSViewBuilder rootViewWithStyle:@"sliderMainView" andSubViews:^NSArray *{
return @[self.okBtn = [ISSViewBuilder buttonWithId:@"onboardingOkButton"]];
}];
}
—
Reply to this email directly or view it on GitHub
https://github.com/tolo/InterfaCSS/issues/46.

 I want to use it as a general style schema.
I was actually unfamiliar with runtime attributes, good stuff!
It works when I set color and font. But when I try to set the frame in my css:
frame: size(auto, 15%).bottom(0); nothing happens.
Is this the way to use this framework? I still want to use some storyboard features but avoid stuff like auto layout.
Best,
Johan

Hi Johan,
You can now (1.3.1) also use properties exposed using IBInspectable (there was a bug that prevented this from working correctly before).
There are basically two ways you can do layout with InterfaCSS:

Use absolute or parent relative rects to set the frame property
Using the slightly more powerful ISSLayout

Regardless of whatever approach you use, keep the following in mind:

Whenever you use a storyboard or xib, make sure to disable stuff that can interfere with layout handled by InterfaCSS (i.e. make sure auto layout is disabled for the storyboard/xib, and disable view autoresizing if not needed).
Using an ISSRootView as the root view of a view controller makes life easier (and is also required for ISSLayout).

Also, have a look at the sample code - that's usually a good place to start and to get inspiration (and then there is also the wiki of course).
/Tobias

Amazing Tobias, truly great work putting this together! The IBInspectable feature was just what I needed.

Thanks, glad you like it! :smiley:


Add theme: elementary OS Titlebar

Add theme: elementary OS Titlebar
This PR adds a new theme to the theme library.
Theme Details

Name: elementary OS Titlebar
Description: Respect elementary OS's default titlebar layout
Homepage: https://ark.sudovanilla.org/Korbs/zen-eos-fix
Author: @SudoVanilla

Thanks!


Lightweight Web Interface

Feature Request
Having a lightweight web interface would be good for dealing with all the information & easy configuration updates. Something similar to as plexpy uses, as the setup & information options increase a gui is a good thing for a clustering type product like this

I think we start with #9, then consider this.  A web interface would also be a lot more convenient for configuration and debugging.

I'm fine with that, I figured the web part at least a basic version should be pretty bolt on as there will be lots of standard ones out there


Add Algolia into Sponsors

Hey @redox,
Would you like to add Algolia into sponsors?
Currently, sponsors are listed here : https://cdnjs.com/about, maybe you'll like to send a PR to add it :)

Oh I would love to :) I'll try to submit a PR later this week end. Thank you!

:+1:

Looks good, close this now.


Add support for predefined hooks

I have been working with the NVIDIA container toolkit which generates a CDI yaml file, including hooks. The hooks all call nvidia-ctk to do 3 things:

create symlinks (relative to the root of a container)
chmod (container root prepended to target paths)
update ldcache (run ldconfig within the chroot of the container)

These are pretty basic standard activities. It seems like it would make sense to have containerEdits within the spec capable of handling these. The spec includes container edit things like mounts, env, deviceNodes, additionalGIDs, and of course hooks. I think that creating symlinks, changing permissions on dir/file, and maybe updating ldcache (I am mixed on that one) are the kinds of capabilities that would fit as first-class citizens within containerEdits.
Thoughts?

I’m working on CDI for Charliecloud, a lightweight, fully-unprivileged container implementation for HPC applications (hpc/charliecloud#1902). (In fact, it looks like @elezar was already helpful in some offline correspondence.) This would also be useful for us.
In fact, regardless of what the standard says, I anticipate not running nvidia-ctk hook to do these things, but rather interpreting such clauses as @deitch suggests and just do the things internally. Such a kludge seems better to me than calling external programs for these “basic standard” tasks, but I’d much rather follow documented behavior.
I would be happy to provide a draft PR for folks to look at.

@reidpr we did discuss this in our latest working group meeting (notes here) and will most likely not be implementing this as part of the spec for the time being.
One of the primary concerns is that even for "basic standard" tasks the applicable interfaces may not be applicable. Take a hook that "updates the ldcache" as an example. In the case of the NVIDIA Container Toolkit -- which implements this as the nvidia-ctk hook update-ldcache command (or nvidia-cdi-hook update-ldcache with the change from https://github.com/NVIDIA/nvidia-container-toolkit/pull/474) -- we take this to mean, run ldconfig from the HOST to update /etc/ld.so.cache IN THE CONTAINER using the specified folders. Here we need to make provision for different paths to ldconfig on the host (e.g. on ubuntu the correct path is ldconfig.real since ldconfig is a shell wrapper) as well as containers that don't have an ldcache (e.g. busybox).
If we need to capture this logic in some kind of binary, this still needs to be available on the host and would complicate CDI spec consumption since we would have to distribute this binary somehow.
Our current strategy for how to handle this would be to provide better documentation and examples for what these hooks do so that their behaviour can be replicated across various environments. This does place the burden on the hook vendors to provide this documentation though and cannot necessarily be enforced by the spec.
One question I have is whether there is some middle ground where we don't implement the hooks in the spec, but more strictly prescribe their behaviour? That is to say that as a vendor, NVIDIA would still distribute the nvidia-cdi-hook binary that would include an update-ldcache subcommand, but that the pre-and post conditions for this command are defined by a public spec. As the spec owners, we could even provide a set of test cases to validate this. This should in theory allow hooks to be more interchangeable and allow users to determine whether hooks are applicable given their environments.

One of the primary concerns is that even for "basic standard" tasks the applicable interfaces may not be applicable. Take a hook that "updates the ldcache" as an example. In the case of the NVIDIA Container Toolkit -- which implements this as the nvidia-ctk hook update-ldcache command (or nvidia-cdi-hook update-ldcache with the change from https://github.com/NVIDIA/nvidia-container-toolkit/pull/474) -- we take this to mean, run ldconfig from the HOST to update /etc/ld.so.cache IN THE CONTAINER using the specified folders.

I can see that as a concern. At the same time, I do think it is a narrowly prescribed set of activities. I originally thought 3, now there is a variant on 1, so total of 4.
Would we want to implement the truly standard ones, i.e. chmod and symlinks?

For clarification, the chmod hook that was added to nvidia-ctk was to workaround a crun bug and should be removed from this discussion.

Thank you @elezar.

Take a hook that "updates the ldcache" as an example ...  we take this to mean, run ldconfig from the HOST to update /etc/ld.so.cache IN THE CONTAINER .... Here we need to make provision for different paths to ldconfig on the host ...

I see your point. I wasn’t in that discussion obviously, but in my view that type of logic belongs more in the container implementation rather than vendor hooks. However you’ve been thinking about this longer than me.
I have a couple follow-ups.
Can you clarify the reasoning on why NVIDIA (or HPE, or ...) should be figuring out host ldconfig stuff rather than Charliecloud (or Podman, or ...)? What am I missing?
What if the host does not have ldconfig? (E.g., Alpine is based on musl, which doesn’t.)

as well as containers that don't have an ldcache...

Good point.
Do these containers need ldconfig run at all, though?
Is it reasonably common to have containers without an ldconfig executable within the container but still needing an ldcache update?

Our current strategy for how to handle this would be to provide better documentation and examples for what these hooks do .... [I]s some middle ground where we don't implement the hooks in the spec, but more strictly prescribe their behaviour?

I could work with this.
I really like the declarative aspect of CDI. Strictly prescribed hooks would be consistent with that, though (in my view) in kind of a roundabout way.

Are these meetings open to the public? It sounds like something I’d be interested in helping with.

Yes, the meetings are public. All the required details should be in the linked document. Ideally they are recorded too, but there is sometimes some lag between when these are available on YouTube.

I will comment to keep it open.

I have created #225 as an initial attempt to capture the intent of the "predefined" hooks.

Not stale.

Cool, thank you @elezar . What do we think about chmod? Or is that just an artifact going away anyways, as you commented above?

The chmod hook was added to our tooling as a workaround for a particular runc bug. I don't think we should formalize this and rather focus on the remaining hooks.


layer begin and end positions are invalid!

I am getting an error
[racon::Window::add_layer] error: layer begin and end positions are invalid!
I ran racon twice and I get the same error. Any help  or suggestion?

Hello,
which racon version are you using? Could you provide the data that is causing the error?
Best regards,
Robert

I am using version 1.2.0. The data that I am using is very huge (5.1GB compressed).
I aligned the reads using minimap. The output SAM output is then the input for racon. I am doing minimap-racon twice. In the first round, racon completed successfully. I get this error in the second round.

Could you please try the second iteration with the latest racon version (1.3.3)?

Here some more lines just before the error.
[racon::Polisher::initialize] loaded target sequences
[racon::Polisher::initialize] loaded sequences
[racon::Polisher::initialize] loaded overlaps
[racon::Polisher::initialize] aligned overlap 668099/668099
[racon::Window::add_layer] error: layer begin and end positions are invalid!

sure. I will try with the latest racon version and let you know. Thanks

I think the latest release is 1.3.2. I could not find v1.3.3 in github releases.

Versions 1.3.3 (latest commit on master, but not a release) and 1.3.2 are almost the same, the newer one has a smaller log while runnig.
Best regards,
Robert

We just found out that our genome assembly has a duplicated contig name. Can this duplicated naming error cause this error?

Probably! Did you manage to run the newer version? Any errors there?

Nope. Have not got chance to run. I just found out at there is a duplicated contig name this morning. I got new assembly without duplicated contigname. I will try with this new assembly now. Our HPC has v1.3.2 installed. I am thinking to test with v1.3.2 for now as , according to you, there is not much difference between v1.3.2 and v1.3.3, except the log filesize.

Sure, go ahead.

Racon ran successfully now. I ran racon-1.3.2 and with the no contig name duplication. Thanks for the help.


Fix the bamboozled printf format string

:o

! @amosr @jystic

👍 I'll see if it's trivial to add a test


Revert "add The Night Manager"

Reverts SickRage/sickrage.github.io#62, going to have to wait till PP exceptions or something similar gets added since the exception is in the filename and not the torrent itself.

Thanks for the pull request! Before a real human comes by, please make sure your PR has all of the below criteria checked

[x] Give a description on what the PR is for.
[ ] Make sure your PR is based on the DEVELOP branch
[x] Don't send big changes all at once. Split up big PRs into multiple smaller PRs that are easier to manage and review

Please make sure you also read contribution guide and followed all of the steps.
Thanks!

Approve!

Approve!

Approved!


Dragging problem / Minimal drag example

I'm trying to reproduce a minimal drag example, where the user can drag a component somewhere, and it stays there. Problem is: the first time the user drags the component, it stays there, but as soon as he tries to re-drag the component, it goes back to the (0,0) position and begins the next drag from there.

My code is the following:
<template>

    Widget

</template>

It's not so clear for me too...
Dragging use translate3D to style position on the element while dragging. But every time when start dragging again the values reset to 0,0,0... I've tried with other options from state and set (like values), but no success... :(

@rafaellehmkuhl have you got around this issue? were you able to use @vueuse/gesture in production or have you opted to use another solution?


Allow to obtain a permalink to the character sheet

Hello,
It exists for some random generators, so why not for this one?
For example, here you can generate a random character with stats and equipment, and using the random seed used to reset the random generator, you can recreate the character sheet. e.g: http://odd.pushingcows.se/?s=1496350790007 as a permalink.
You could then roll for a character on one device, being satisfied enough with it, send the permalink to yourself on another device, or share it with a friend of yours, etc.

It doesn't exist for this one because I didn't write the code to make that happen, and no one else has.


Feature request - show device names similar to nextdns

Hey Controld folks,
NextDns started showing device names in their logs especially when deployed on UDM PRO - the names that you specify in the UDM admin panel, instead of the device name that you specify on device.  This is cleaner way to show the devices that are currently attached to the UDM PRO and use the filtering options.
It wold be nice to add this to CtrlD in the future.
Thanks,

Do you mean device aliases? Can you screenshot what you're referring to?

Hey Yegors - that is correct.  For example: https://cloud.photom.me/hEoDHsJ2 and https://cloud.photom.me/0cUpIo38 thank you.

I'm a bit unfamiliar with this feature, however if you mirror your custom names to the hosts file they will be read by ctrld (v1.3.1)  and preference over "true hostnames" those devices may be advertising.
Are you using v1.3.1 and are those names mirrored in /etc/hosts on the UDM Pro, or these are stored in some other config file?

@yegors I think ubios store these mapping in mongodb, we need to read from there to get the mapping set by users.

Looks that way: https://github.com/nextdns/nextdns/blob/cd5b7f567b3b74fd1e42ac993e1f86306bca9139/discovery/ubios_linux.go#L82C1-L82C1

That is it ;) let me know if you need any additional info

This is done and available in v1.3.4


As a Software Developer of Open Source Software, I would like my continuous integration tests to run on Linux, MAC and Windows platforms so that I can verify platform neutralilty of the implementation

Travis CI does not currently support Windows builds.
[] investigate alternate build CI system for windows builds

I have created a branch for making the updates to the Travis CI configuration to enable LINUX builds ( ISSUE-5 ).  Presently the builds are failing because the libatlas-base-dev does not produce byte equivalent results.

[ ] investigate build differences in cblas/atlas libraries and identify solution.

INVESTIGATE BUILD DIFFERENCES IN CBLAS/ATLAS
Both the CBLAS and ATLAS libraries have substantial differences from each other and from the original BLAS library.   Most notably:

ATLAS uses excess precision (doubles instead of floats) for some of the calculations.
CBLAS appears to change its method when the size of the vectors is large, and uses a different loop unrolling strategy when the loops are small.

As a result, it is to be expected that they yield different results.   For the purpose of this work, I need a platform neutral implementation.

[ ] Investigate platform neutral BLAS library to use as basis for performance comparisons.

It is possible to compile the FORTRAN version of BLAS on both OSX and Linux.    Although it is not as fast on the mac (about 5x slower) than the Accelerate framework implementation of CBLAS, it does not make use of any hardware specific optimizations and is probably a better basis for comparison.   I have begun implementing the build scripts to install the FORTRAN library (building from source if necessary) on branch f77.

SCOPE CHANGE - Defer Window build to 1.0.0
The travis CI system does not yet support windows builds.   Rather than delay the completion of milestone 0.1.1 I am deferring support for the windows OS until version 1.0

Merged into master after approval of pull request


AppBarMobile is missing from the documentation - is there a way to override it?

We needed to override the default AppBarMobile (https://github.com/marmelab/admin-on-rest/search?utf8=✓&q=AppBarMobile&type=) since it seems that it appears on top of our custom AppBar when in mobile. I assume that this component is internally used .. how is it connected with the rest of the application so it can be overriden?
Thank you

It can't be overridden currently, unless you create your own versions of all the CRUD views (List, Create, Show, Edit, Delete).
The workaround would be to hide your custom AppBar on mobile.

Good to know! what I did is I used hacky z-index to enforce my custom AppBar to be on top of the Mobile one to work for both cases (mobile and desktop)

Fixed by #2155 (by deprecating it)


Poll System

Poll System
Features

Polls should be stored in case of bot shutdown
End time should be shown
Automatic result announcement
Duplicate votes should be removed, unless explicitly allowed

Closed by #14.


Fix backend Dockerfile

remove parsers.requirements.txt

Hi @falmanna , thanks for the update.
The above change is being fixed in another major PR coming up soon, hence wont be approving this.


appears to fail loading on 1.16.5

fails pretty much instantly with console reporting:
https://pastebin.com/W3ff4iHZ

Fixed in 1.1.4-1.16


How do you use this?

Can you show some examples of using this provider with resources?

provider "sumologic" {
    access_id = ""
    access_key = ""
    environment = "" # your region (eu, us, etc.)
}

resource "sumologic_collector" "collector" {
    name = ""
    description = ""
    category = ""
}

resource "sumologic_http_source" "http" {
    name = "" # string
    messagePerRequest = "" # bool
    collector_id = "${sumologic_collector.collector.id}"
}

That should work. I'm mainly waiting for Terraform to implement some way to deal with sensitive values so I can add more sources (the polling source comes to mind).
I'm not using this myself yet, so I can't really say if it really works (it works in my local tests).  :-)

How-to is in README.md now.


improving permissions and permission scopes

This PR enhances the permission system slightly by adding scopes and more logic that will need to be completed.

Thank you for the PR.
I have taken a look at the enhanced permissions logic and I think that the Scope addition makes sense. About the User, my initial idea was that the User was an abstract interface, then there would be implementations such as Administrator, Teacher, Student and so on, since that is easily expandable. However, having only the User class and decide the roles, or even use composition to get the User role, I think it would work too.

it was the best I could come up with right now. it will need some more work.  I would like to be able to use it to trap some errors and provide some checks and balances.  We can work on this.
theres some memory leaks but I think we need to test with Valgrind which i have limited experience.
I'm concerned with some memory leaks in dataframe.cpp and I might try and improve some of it.


Error after upgrading from version 3.11.12 -> 3.11.13

Not sure where this is coming from, but it was picked up by Dependabot - our app no longer compiled on CI.
== Compilation error in file lib/my_app/accounts/user.ex ==
** (RuntimeError) Exception in transformer AshAuthentication.Strategy.Custom.Transformer on MyApp.Accounts.User:

expected a map, got: nil
    (elixir 1.15.5) lib/map.ex:486: Map.take/2
    (ash_authentication 3.11.13) lib/ash_authentication/strategies/password/transformer.ex:78: anonymous fn/2 in AshAuthentication.Strategy.Password.Transformer.transform/2
    (ash_authentication 3.11.13) lib/ash_authentication/strategies/password/transformer.ex:75: AshAuthentication.Strategy.Password.Transformer.transform/2
    (ash_authentication 3.11.13) lib/ash_authentication/strategies/custom/transformer.ex:90: AshAuthentication.Strategy.Custom.Transformer.do_transform/3
    (ash_authentication 3.11.13) lib/ash_authentication/strategies/custom/transformer.ex:44: anonymous fn/2 in AshAuthentication.Strategy.Custom.Transformer.do_strategy_transforms/1
    (elixir 1.15.5) lib/enum.ex:4830: Enumerable.List.reduce/3
    (elixir 1.15.5) lib/enum.ex:2564: Enum.reduce_while/3
    /Users/me/Projects/work/my-app/lib/my_app/accounts/user.ex:1: (file)

I just pushed something to main that ought to resolve this. It will at least handle the nil value, but there may be a deeper issue.

Should be fixed in v3.11.15.  Can you confirm @sevenseacat?

We're running 3.11.15 now and it's all good!


TypeError at /admin/orders/R837725288.pdf - "no implicit conversion of nil into String"

My Gemfile has the below version :
gem 'spree_print_invoice' , :git => 'git://github.com/spree/spree_print_invoice.git' , branch: '2-3-stable'

While, I am clicking the button -- print invoice, getting the error -
Started GET "/admin/orders/R837725288.pdf?template=invoice" for <IP_ADDRESS> at 2015-04-13 13:50:57 +0530
Processing by Spree::Admin::OrdersController#show as PDF
  Parameters: {"template"=>"invoice", "id"=>"R837725288"}
#.....
Rendered<EMAIL_ADDRESS>(20.2ms)
  Rendered<EMAIL_ADDRESS>(27.7ms)
Completed 500 Internal Server Error in 47ms

TypeError - no implicit conversion of nil into String:
  prawn (1.0.0) lib/prawn/images.rb:144:in `verify_and_open_image'
  prawn (1.0.0) lib/prawn/images.rb:79:in `build_image_object'
  prawn (1.0.0) lib/prawn/images.rb:67:in `image'
   () Users/shreyas/.rvm/gems/ruby-2.1.2@collective/bundler/gems/spree_print_invoice-393bec83bfd6/lib/prawn_handler.rb:22:in `method_missing'
   () Users/shreyas/.rvm/gems/ruby-2.1.2@collective/bundler/gems/spree_print_invoice-393bec83bfd6/app/views/spree/admin/orders/_print.pdf.prawn:8:in `_411b913187c5f59f36c7bb34abf7ead6'
  actionview (4.1.4) lib/action_view/template.rb:145:in `block in render'
#...

Are you sure that Spree::PrintInvoice::Config[:logo_path] is actually a valid image?


Custom ContextMenu on NumericUpDown will not show on text input

Describe the bug
A custom ContextMenu will not shown on the text input for

NumericUpDown
HotKeyBox
DateTimePicker, TimePicker
DatePicker

To Reproduce
Create a ContextMenu and set it to one of these controls.
Expected behavior
The ContextMenu should be shown at the text input.
Environment(please complete the following information):

MahApps.Metro version >= v1.6.5

Additional context
Thx to @SR84 who found this issue.

Super, danke.


decide on language

some of the filters would need to talk to the github API, so probably something from this list: https://docs.github.com/en/rest/overview/libraries would be convenient
Python is probably convenient for combining with cffconvert validator
node would be good (but maybe not necesssary per se) if we want to run this in a scheduled GitHub Action workflow, for example if we're interested in collecting statistics over time.

Does it need to be one language? If necessary, we can dump to a JSON file and bridge the language gap, right?

We decided to use Node.js JavaScript with octokit library for making requests to the GitHub API

Octokit repo: https://github.com/octokit/octokit.js


C4 PlantUML elements with parameters passed with $ are not recognized

Describe the bug
When I add elements and pass the arguments like it's shown on the C4 PlantUML page with a $-character ($descr="description") these elements are not recognized.
In general nearly all examples on the C4 PlantUML page can't be displayed because of that issue.
To Reproduce
Steps to reproduce the behavior:

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

PlantUML
@startuml
Person(person_descr, "person", $descr="description")
Person(person_descr, "person", $descr="") ' an empty parameter also doesn't work
Person(person_tags_one, "person", $tags="one+two")
@enduml

Here more and it shows that it's a valid syntax
Expected behavior
The elements should be returned.

@dgassma : since all the c4 stuff is coming from you, do you have an idea why this is happening?

@philippsimon : you are right and I am sorry for the inconvenience.
I misinterpreted the meaning of the $.
The current implementation is partly optional parameter parsing. Take a look at the C4 tests in plantuml-parser:
@startuml

("C4 - Person & System")

Person(person, "Person::label", "Person::descr")
Person_Ext (person_ext, "Person_Ext::label", "Person_Ext::descr")

System(system, "System::label", "System::descr")
System_Ext (system_ext, "System_Ext::label", "System_Ext::descr")

SystemDb (systemDb, "SystemDb::label", "SystemDb::descr")
SystemDb_Ext (systemDb_ext, "SystemDb_Ext::label", "SystemDb_Ext::descr")

SystemQueue (systemQueue, "SystemQueue::label", "SystemQueue::descr")
SystemQueue_Ext (systemQueue_ext, "SystemQueue_Ext::label", "SystemQueue_Ext::descr")

System (system, "System::label")

System (systemDescr, "System::label", "System::descr")

System (systemDescrSprite, "System::label", "System::descr", "System::sprite")

System (systemDescrSpriteTags, "System::label", "System::descr", "System::sprite", "System::tags")

System (systemDescrSpriteTagsLink, "System::label", "System::descr", "System::sprite", "System::tags", "System::link")

@enduml

I already have an idea how to fix it in my mind. However, I am not sure, if the library should support the old (wrong) parsing as well?
What is your opinion on that @philippsimon & @Enteee
I am going to try to provide a fix by the end of week.

@dgassma I would actually allow both. I tested it and both works. So I guess others could have interpreted the same as you and it also works.
I tested it here:
https://www.plantuml.com/plantuml/uml/TOzF2u8m68VlVWgJ3WNf83e9WM3T2eBeBFlpaeDjolQExDitmmaHtiRps_F0Mpca5hrMoKeQhho0j4NiNKc8fQ_YAR7rp3kml3C81WlUQT8hQaOdToYKPAG-vBSHFQuNegb3i04rTJ2eD62BVkmdztTOz8bjtFJOfLbaPQae0vNz806EssmxYyrvm5bkOC3uUzfYv0_7q79flvXQV-NZKh428y9r7m00

I just saw that for mermaidjs they already created a parser for C4 diagrams:
https://github.com/mermaid-js/mermaid/blob/develop/packages/mermaid/src/diagrams/c4/parser/c4Diagram.jison
Maybe it can be reused or a collaboration can be started?

@dgassma i don't have a strong opinion if we should support the old parsing as well. I rarely use C4 so I can not really speak from experience. Sorry..
@philippsimon  looking at the parsing grammar in mermaid; looks cool and I would be super happy if I don't have to support C4 in a PEG in this project. But I am not quite sure if there's a working Jison to PEG translator and I am not even quite sure if our grammar isn't better, i think @dgassma  did quite a good job :)


what means the "ee"?

https://github.com/wenbowen123/catgrasp/blob/dc656c05da85e9e732c56a887bb665e1b2a52289/pybullet_env/env.py#L70

ee means end_effector, i.e. the last link of the robot.

great, thank you!


Omit notify on startup

We dislike the Jenkins spam by being notified when builds start. We've got to many builds going on for that. We'd like to only be notified if something breaks, and thus omitting the notification on startup.

Testing now:

Regression test: using ace (by branch feat/omit-startup-notification) without applying the setting, as to work the same as before
Happy case: explicitly omitting the startup notification by using the setting

Seems to not have worked as intended. Will have to rewrite the logic a bit, simplifying it to a less error prune implementation


Images in 'Top Picks' are poorly cropped

The section 'Top Picks' consists of a mosiac of images that are hardcoded. On a regular display the page looks like so:

First and foremost, I would love to see these images change with time. CC contains a huge collection of wonderful images and we have chosen to display a select 7 of them.
Something like a masonry layout (#225, #103) allowing every image to be shown in glorious entirety would do the page wonders. Currently Monocerotis (image no. 1), the bald eagle (image no. 4) and the NYC skyline (image no. 5) are all cropped in a way that the content is missing from view.
If the masonry layout is too much, at the very least we need to find dimension appropriate images for these divs. Also the vertical margin between the images is too much.
If we could decide on a direction in which to proceed, I would be more than happy to submit a PR patching this.

We're working on a redesign of the homepage which removes that section entirely so I'm going to close this ticket. Once the homepage redesign has shipped, feel free to open a new ticket if you have suggested improvements.


ag doesn't find the word in a range of files but grep does

$ grep -Rni state
Binary file Block4-graphtheory/Block-4.pdf matches
Binary file Block3-basiccombinatorics/Unit-2.pdf matches
Binary file Block3-basiccombinatorics/Unit-4.pdf matches
Binary file Block3-basiccombinatorics/Unit-1.pdf matches
Binary file Block3-basiccombinatorics/Block-3.pdf matches
Binary file Block3-basiccombinatorics/Unit-3.pdf matches
Binary file Block1-Elementry_logic/Unit-2.pdf matches
Binary file Block1-Elementry_logic/Unit-1.pdf matches
Binary file Block1-Elementry_logic/Block-1.pdf matches
Binary file Block1-Elementry_logic/Unit-3.pdf matches
$ ag state

$ ag -i state

ag doesn't search binary files by default. Try --search-binary.

The same thing happened to me:
1) grep -r "g_tr069_obj_func_list" romfs/
2) ag "g_tr069_obj_func_list" romfs/
grep displays all search results, but ag does not.
The ag version  is 2.1.0-1.el7.

Same for me, but I'm on latest (2.2.0).  These are source files, not binaries.  If I traverse down the tree closer to the matching files it works.


Added Packetization feature for ease in integration

Hi, I created a packetization feature to ease the process of adding different Slaves & addresses similar to SimpleModbus Library. 4 Steps of process:

Packet initialisation
Packet construction.
Configuration.
Loop & extraction of registers.

Hi!
I studied your modifications to learn what improvements these will bring into the library.
I am afraid I tend to not integrate the pull request, as the changes will implement a use case among many, too special for a more general application. The setup is RTU only, the construction of the repeating requests is not as general as I would like to have it.

Hi!
I studied your modifications to learn what improvements these will bring into the library.
I am afraid I tend to not integrate the pull request, as the changes will implement a use case among many, too special for a more general application. The setup is RTU only, the construction of the repeating requests is not as general as I would like to have it.
We even had a feature called "Jobs" in the pipeline, that did what you implemented, but on a more general level. We declined it, though, as it again was providing just one application for the Modbus.

No problem @Miq1 , Thanks.
As you mentioned, it is more application specific one, might not ideal for everyone. I will update you guyz, on implementing any new features in my side to make eModbus Library rocks 👍.


Multi-class loss_mmod

@davisking, thanks again for this awesome library.
You write here:

I haven't made the loss_mmod multi-class yet.  It's a minor change though
and will be in dlib in the near future though :)

I guess this does not exist yet? At least couldn't find anything.
I'd be happy to try to contribute, however it seems you already have a clear idea how it should be done, so I guess I'd be better off waiting for your implementation?

I haven't done it yet.  I'm working on the multi-aspect ratio update at the moment.  Which is itself not that complicated.  I'm fairly confident the code is right, but I haven't pushed it to github yet since I'm not done testing it.  And testing DNN tooling takes forever because training is so slow.
Anyway, I'll push the multi-aspect ratio mmod in a week or two at the most unless I find something wrong with it, which seems unlikely at this point.  Then I'll update it for multi-label support after that, which is a lot simpler and shouldn't require as much tedious testing.

The multi-label support is now in master, and seems to work great. Thanks a lot, Davis!


[#213] test public API open_or_create_with_attributes

Notes for Reviewer

Public API open_or_create_with_attributes should be tested.
update the doc of open_or_create_with_attributes.

Pre-Review Checklist for the PR Author

[X] Add sensible notes for the reviewer
[X] PR title is short, expressive and meaningful
[X] Relevant issues are linked in the References section
[X] Every source code file has a copyright header with SPDX-License-Identifier: Apache-2.0 OR MIT
[X] Branch follows the naming format (iox2-123-introduce-posix-ipc-example)
[X] Commits messages are according to this guideline

[X] Commit messages have the issue ID ([#123] Add posix ipc example)
[X] Commit author matches Eclipse Contributor Agreement (and ECA is signed)

[X] Tests follow the best practice for testing
[X] Changelog updated in the unreleased section including API breaking changes
[ ] Assign PR to reviewer
[X] All checks have passed (except task-list-completed)

Checklist for the PR Reviewer

[ ] Commits are properly organized and messages are according to the guideline
[ ] Unit tests have been written for new behavior
[ ] Public API is documented
[ ] PR title describes the changes

Post-review Checklist for the PR Author

[ ] All open points are addressed and tracked via issues

References

Relates to #213

@xieyuschen you forget to add the Relates to #213 at the end of the PR description - I added it for you!

Thank you very much for your contribution. As soon as the CI is green we merge it. I think you need to format the code.

my bad, I have forgot to add the fmt into my git hooks. i have amended the commit and pushed it. thanks a lot.

@elfenpiff , could you kindly help to re-run the failed pipeline? Only windows failed and irrelevant to the current PR. I don't have permission to re-run the failure ci and trigger the whole pipeline takes more time.
--- STDOUT:              iceoryx2-bb-threadsafe::trigger_queue_tests trigger_queue_one_pop_notifies_exactly_one_timed_push ---

running 1 test
test trigger_queue_one_pop_notifies_exactly_one_timed_push ... FAILED

failures:

failures:
    trigger_queue_one_pop_notifies_exactly_one_timed_push

test result: FAILED. 0 passed; 1 failed; 0 ignored; 0 measured; 11 filtered out; finished in 0.20s

--- STDERR:              iceoryx2-bb-threadsafe::trigger_queue_tests trigger_queue_one_pop_notifies_exactly_one_timed_push ---
thread '<unnamed>' panicked at iceoryx2-bb\threadsafe\tests\trigger_queue_tests.rs:267:17:
assertion failed: expr: sut.timed_push(0, TIMEOUT * 1000) == true;  value: false == true
stack backtrace:

could you kindly help to re-run the failed pipeline?

@xieyuschen done.
and its merge. Congratulations on your first of hopefully many contributions!

could you kindly help to re-run the failed pipeline?

@xieyuschen done.
and its merge. Congratulations on your first of hopefully many contributions!

Cool, it's an interesting project and actually it's helpful to eliminate massive ser/desel cpu resources inside company. I have thought it in golang before but haven't chosen this approach because of the challenges and the corresponding maintenance inside a company. This time in an open source project I believe I can participate more:)
Thanks for your patience to review.

@xieyuschen if you are interested in golang, there is #225. If you want, I think you could ask in the issue if help is wanted.


Cosmos DB - Merging feature branch into main

Moves all Cosmos SDK development to main branch

@RickWinter README added

@RickWinter / @jhendrixMSFT all comments are addressed and the PR is ready to merge. Unless there is a blocking concern, we'll move forward and merge it on monday.


Recognizing Horizontal Swiping

Hi @glepur. I am having trouble triggering onSwipe for horizontal swipe gestures. I have used default config parameters
const config = {
      velocityThreshold: 0.3,
      directionalOffsetThreshold: 100
    };

I have messed around altering params but still cannot reliable horizontal swipe when using ios simulator.
Could you provide more context on the range of velocityThreshold and directionalOffsetThreshold

Hi @SeanAvery.
velocityThreshold is minimum swipe speed required to trigger callbacks. I'm comparing it to vx and vy properties of gestureState internally.
directionalOffsetThreshold is used so callbacks would not trigger on diagonal swipes. i.e. when you are swiping the screen horizontally if your total vertical movement when the gesture finishes is higher than directionalOffsetThreshold callbacks will not trigger. Check dx and dy properties of gestureState.
So basically if you have hard time triggering callbacks on simulator you would want to reduce velocityThreshold and increase directionalOffsetThreshold.
If you have any ideas about how to improve documentation, or this repo in general, I'd be happy to receive PR's.