Using a label without a text controller throws "toDouble called on null"

Steps to Reproduce
    return new Scaffold(
        appBar: new AppBar(
          title: new Text('Enter text to read'),
        ),
        body: new Center(
          child: new TextField(
              maxLines: null,
              decoration: new InputDecoration(
                hintText: 'Enter some text for paced reading',
              ),
              onChanged: _action,
              onSubmitted: (String str) {
                Navigator.of(context).push(new MaterialPageRoute<Null>(
                    builder: (_) => new MyReadPage(text: str)));
              }),
        ));

Though, it seems to happen somewhat sporadically. First it happened on first load, later it happened upon deleting some characters.
Logs
I/flutter (28170): ══╡ EXCEPTION CAUGHT BY WIDGETS LIBRARY ╞═══════════════════════════════════════════════════════════
I/flutter (28170): The following NoSuchMethodError was thrown building _TextSelectionHandleOverlay(dirty, state:
I/flutter (28170): _TextSelectionHandleOverlayState#8b639):
I/flutter (28170): The method 'toDouble' was called on null.
I/flutter (28170): Receiver: null
I/flutter (28170): Tried calling: toDouble()
I/flutter (28170):
I/flutter (28170): When the exception was thrown, this was the stack:
I/flutter (28170): #0      Object.noSuchMethod (dart:core-patch/dart:core/object_patch.dart:46)
I/flutter (28170): #1      double./ (dart:core-patch/dart:core/double.dart:41)
I/flutter (28170): #2      _TextSelectionHandleOverlayState.build (package:flutter/src/widgets/text_selection.dart:512:49)
I/flutter (28170): #3      StatefulElement.build (package:flutter/src/widgets/framework.dart:3640:27)
I/flutter (28170): #4      ComponentElement.performRebuild (package:flutter/src/widgets/framework.dart:3550:15)
I/flutter (28170): #5      Element.rebuild (package:flutter/src/widgets/framework.dart:3451:5)
I/flutter (28170): #6      ComponentElement._firstBuild (package:flutter/src/widgets/framework.dart:3530:5)
I/flutter (28170): #7      StatefulElement._firstBuild (package:flutter/src/widgets/framework.dart:3668:22)
I/flutter (28170): #8      ComponentElement.mount (package:flutter/src/widgets/framework.dart:3525:5)
I/flutter (28170): #9      Element.inflateWidget (package:flutter/src/widgets/framework.dart:2890:14)
I/flutter (28170): #10     Element.updateChild (package:flutter/src/widgets/framework.dart:2693:12)
I/flutter (28170): #11     SingleChildRenderObjectElement.mount (package:flutter/src/widgets/framework.dart:4533:14)
I/flutter (28170): #12     Element.inflateWidget (package:flutter/src/widgets/framework.dart:2890:14)
I/flutter (28170): #13     Element.updateChild (package:flutter/src/widgets/framework.dart:2693:12)
I/flutter (28170): #14     ComponentElement.performRebuild (package:flutter/src/widgets/framework.dart:3562:16)
I/flutter (28170): #15     Element.rebuild (package:flutter/src/widgets/framework.dart:3451:5)
I/flutter (28170): #16     ComponentElement._firstBuild (package:flutter/src/widgets/framework.dart:3530:5)
I/flutter (28170): #17     StatefulElement._firstBuild (package:flutter/src/widgets/framework.dart:3668:22)
I/flutter (28170): #18     ComponentElement.mount (package:flutter/src/widgets/framework.dart:3525:5)
I/flutter (28170): #19     Element.inflateWidget (package:flutter/src/widgets/framework.dart:2890:14)
I/flutter (28170): #20     Element.updateChild (package:flutter/src/widgets/framework.dart:2693:12)
I/flutter (28170): #21     ComponentElement.performRebuild (package:flutter/src/widgets/framework.dart:3562:16)
I/flutter (28170): #22     Element.rebuild (package:flutter/src/widgets/framework.dart:3451:5)
I/flutter (28170): #23     ComponentElement._firstBuild (package:flutter/src/widgets/framework.dart:3530:5)
I/flutter (28170): #24     StatefulElement._firstBuild (package:flutter/src/widgets/framework.dart:3668:22)
I/flutter (28170): #25     ComponentElement.mount (package:flutter/src/widgets/framework.dart:3525:5)
I/flutter (28170): #26     Element.inflateWidget (package:flutter/src/widgets/framework.dart:2890:14)
I/flutter (28170): #27     Element.updateChild (package:flutter/src/widgets/framework.dart:2693:12)
I/flutter (28170): #28     RenderObjectElement.updateChildren (package:flutter/src/widgets/framework.dart:4325:32)
I/flutter (28170): #29     MultiChildRenderObjectElement.update (package:flutter/src/widgets/framework.dart:4648:17)
I/flutter (28170): #30     Element.updateChild (package:flutter/src/widgets/framework.dart:2682:15)
I/flutter (28170): #31     _TheatreElement.update (package:flutter/src/widgets/overlay.dart:507:16)
I/flutter (28170): #32     Element.updateChild (package:flutter/src/widgets/framework.dart:2682:15)
I/flutter (28170): #33     ComponentElement.performRebuild (package:flutter/src/widgets/framework.dart:3562:16)
I/flutter (28170): #34     Element.rebuild (package:flutter/src/widgets/framework.dart:3451:5)
I/flutter (28170): #35     BuildOwner.buildScope (package:flutter/src/widgets/framework.dart:2251:33)
I/flutter (28170): #36     BindingBase&SchedulerBinding&GestureBinding&ServicesBinding&RendererBinding&WidgetsBinding.drawFrame (package:flutter/src/widgets/binding.dart:580:20)
I/flutter (28170): #37     BindingBase&SchedulerBinding&GestureBinding&ServicesBinding&RendererBinding._handlePersistentFrameCallback (package:flutter/src/rendering/binding.dart:200:5)
I/flutter (28170): #38     BindingBase&SchedulerBinding._invokeFrameCallback (package:flutter/src/scheduler/binding.dart:713:15)
I/flutter (28170): #39     BindingBase&SchedulerBinding.handleDrawFrame (package:flutter/src/scheduler/binding.dart:649:9)
I/flutter (28170): #40     _invoke (file:///b/build/slave/Linux_Engine/build/src/flutter/lib/ui/hooks.dart:91)
I/flutter (28170): #41     _drawFrame (file:///b/build/slave/Linux_Engine/build/src/flutter/lib/ui/hooks.dart:80)
I/flutter (28170): ════════════════════════════════════════════════════════════════════════════════════════════════════

analyze reported no issues
Flutter Doctor
[✓] Flutter (on Linux, locale en_US.UTF-8, channel alpha)
    • Flutter at /home/mfairhurst/dart/flutter
    • Framework revision e8aa40eddd (4 weeks ago), 2017-10-17 15:42:40 -0700
    • Engine revision 7c4142808c
    • Tools Dart version 1.25.0-dev.11.0

[✗] Android toolchain - develop for Android devices
    ✗ Unable to locate Android SDK.
      Install Android Studio from: https://developer.android.com/studio/index.html
      On first launch it will assist you in installing the Android SDK components.
      (or visit https://flutter.io/setup/#android-setup for detailed instructions).
      If Android SDK has been installed to a custom location, set $ANDROID_HOME to that location.

[✗] Android Studio (not installed)
    • Android Studio not found; download from https://developer.android.com/studio/index.html
      (or visit https://flutter.io/setup/#android-setup for detailed instructions).

[-] IntelliJ IDEA Ultimate Edition (version 2016.2)
    ✗ Flutter plugin not installed; this adds Flutter specific functionality.
    • Dart plugin version 162.2485
    • For information about installing plugins, see
      https://flutter.io/intellij-setup/#installing-the-plugins
    ✗ This install is older than the minimum recommended version of 2017.1.0.

[-] IntelliJ IDEA Ultimate Edition (version 2017.2)
    ✗ Flutter plugin not installed; this adds Flutter specific functionality.
    • Dart plugin version 172.2791
    • For information about installing plugins, see
      https://flutter.io/intellij-setup/#installing-the-plugins

[-] IntelliJ IDEA Ultimate Edition (version 2016.3)
    ✗ Flutter plugin not installed; this adds Flutter specific functionality.
    • Dart plugin version 163.8651
    • For information about installing plugins, see
      https://flutter.io/intellij-setup/#installing-the-plugins
    ✗ This install is older than the minimum recommended version of 2017.1.0.

Looks like this might occur on long press?

Looks like a duplicate of https://github.com/flutter/flutter/issues/11573


Cannot make a GigE camera work with a direct connection

Hi,
As a part of a project, I need to use a DMK 23GM021 camera with a Raspberry Pi.
After spending several days trying all the things I could think about, I managed to have the camera detected by the tiscamera tools (and also the Aravis tools) when plugged to my home router.
It also works well when plugged directly to a Windows computer.
However, I just can't get it to be detected when plugged directly to the Raspberry Pi, or to any other computer running Linux actually.
It may very well be a network issue, but I'm far from being competent in networking and I just can't tell what could me missing here.
I tried to setup a DHCP server on the Raspberry but it never receives any DHCPREQUEST messages from the camera (also, do I even need a DHCP server?)
The only packets I see with Wireshark are DISCOVERY_CMD, plus the DHCPDISCOVER and DHCPOFFER messages.
I also tried to ping the whole subnet (<IP_ADDRESS>/16) using the fping and nmap commands, but nothing came out.
I am a bit stuck here :/
The camera was bought under the pretense that it was perfectly compatible with Linux, but I had mixed results so far...
Some help would be great.

Hello
"DMK 23GM021 camera with a Raspberry Pi." I am very sorry, but that wont work, because the PI has no GigE network controller. The camera needs the GigE speed, otherwise it denies to work.
Also the camera must be powered. Either by a PoE device or an external 12V power supply.
Yes, you can use the camera with Linux, but you must use compatible hardware. If you are bound to a PI and can not change to e.g. Odroid with GigE interface, then you should consider an USB camera, e.g.  DFK 42BUC03 or DFG 42AUC03.
Stefan

I must admit that I never considered the speed of the Ethernet port...
I was using a PoE device by the way.
Well, we should have thought of that during the design step.
Thank you for your answer, you can close this issue.

You may consider using

Twisted pair network cable
Then the camera should at least ask for DHCP and the yellow LED on the camera should blink.
lowest possible frame rate
That could do the job, but must not be stable.

Stefan


Added Ubuntu-18.04 as test environment

This uses Ruby-2.5 and gcc-4.8.
In adding it, I have grouped the same description together and used matrix.
The reason I limited myself to just matrix.include is that there are several environments that need to be specialized, and the whole description is easy to understand.
For example, the following description is not immediately comprehensible to me even when I write it myself. It is even more so if there are a lot of them.
  matrix:
    os: ubuntu-2022.04, ubuntu-2020.04
    cc: gcc, clang
    include:
    - { cc: gcc, cxx: g++ }
    - { cc: clang, cxx: clang }
    - { os: ubuntu-latest, cc: gcc-4.8, cxx: g++-4.8, container: ubuntu:18.04 }

This is the same as:
  matrix:
    include:
    - { os: ubuntu-2022.04, cc: gcc, cxx: g++ }
    - { os: ubuntu-2022.04, cc: clang, cxx: clang++ }
    - { os: ubuntu-2020.04, cc: gcc, cxx: g++ }
    - { os: ubuntu-2020.04, cc: clang, cxx: clang++ }
    - { os: ubuntu-latest, cc: gcc-4.8, cxx: g++-4.8, container: ubuntu:18.04 }

Ruby-2.3 is available for Ubuntu-2016.04.
But I did not add it because it is already a build requirement for mruby to be Ruby-2.5 or higher.
# However, it is a bit surprising that it even succeeds to rake test.

For now, let me summarize the whitespace that makes it look right...

I don't think the version number 2.5 is that important. It was the oldest maintained version at the time of the update (2021-02-09). Probably, we should update here regularly (or remove version number, since it's hard to find unsupported version e.g. 1.8).

I understand about the version number.
I will now change this PR to just a commit that uses matrix.


Text Guide should recommend metadata format

Was https://code.google.com/p/darwincore/issues/detail?id=180
Reported by gtuco.btuco, May 24, 2013
==Documentation Error Report==
URL:
http://rs.tdwg.org/dwc/terms/guides/text/index.htm
Error description:
Not actually an error, just a request to make a recommendation that isn't there already. The Darwin Core Text Guide does not make recommendations about the metadata document.
Recommended remedy:
Add a recommendation about the metadata in the archive, that it conform minimally to a standard, for example Dublin Core Application Profile.
Submitter: John Wieczorek, Markus Döring

How to proceed here? The standard is silent on the inclusion of a metadata file describing the data set just as it is on the compression options (see https://github.com/tdwg/dwc/issues/80). That is an innovation of a Darwin Core archive, which is an implementation, documented here.  Should these be part of the standard or only part of the implementation?

Hmm. There was at one point a desire to maintain "ancillary" pages that contained stuff outside the standard like recommended sources of URIs. I think that is still in the RDF Guide. So I suppose one could create a separate page, perhaps hosted on the DwC GitHub site or the TDWG website, where implementation suggestions are kept. If we did this, we should probably figure out where those RDF ancillary pages ended up and put this in the same place.

Found the ancillary page here. It's lurking in the defunct RDF task group's repo, just as it was when exported from Google Code. Probably not the best place for it...

Wouldn't you put the metadata format recommendation as non-normative
content in the Text Guide, similar to the suggestion for
https://github.com/tdwg/dwc/issues/80?
On Wed, Sep 9, 2020 at 4:41 PM Steve Baskauf<EMAIL_ADDRESS>wrote:

Found the ancillary page here
https://github.com/tdwg/rdf/blob/master/DwCAncillary.md. It's lurking
in the defunct RDF task group's repo, just as it was when exported from
Google Code. Probably not the best place for it...
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/tdwg/dwc/issues/33#issuecomment-689776210, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AADQ724GWM37Q2GS7MF57O3SE7K6DANCNFSM4AXK3RIA
.

I think it could go either way. The whole ancillary page thing dates from before the VMS was written and we were struggling to figure out what the process was for adding a document to a standard. I think the desire to keep that other content out of the standards document was an attempt to have latitude in making changes without invoking a burdensome process. We now have that latitude from the VMS.
In looking at that ancillary page, I'm not sure what is there that needs to be maintained. A lot of it relates to material that may not be so relevant any more. I think the most useful bit there is the list of recommended IRIs, which could probably be put into the RDF Guide as a non-normative section. The other modeling bits could probably be added as references at the end of the RDF Guide. DSW now has a published paper that's citable. The taxon model is being worked on by the new TCS group, so that falls out of the hands of DwC. I'm not aware that the other models discussed are being used by anyone anymore.
The bottom line is that probably the idea of ancillary pages has outlived its usefulness. This kind of reference material and examples could just be included in the document itself as you suggested.  Also referring people to a page in the repo of a dead task group isn't that great - whatever we maintain should probably be somewhere that can be maintained by the DwC Maintenance Group (another thing that didn't exist prior to the VMS).
I am so happy at the number of issues that are being closed that I hate to suggest this, but I think I'm going to open an issue to decommission the ancillary RDF page and just revise it into the RDF Guide.

Punting this issue for later work that also includes the RDF ancillary pag.

With the update to the Text Guide to mention Darwin Core Archive as an implementation of the Darwin Core Text recommendation and link to the documentation of Darwin Core Archives, the issue of the meta.xml file is taken care of by that implementation documentation, as it should be.


Multiple Clocks

In my quest to learn Meteor I'm trying to build a simple stopwatch app. So I have multiple projects that I want to track time to. Each project needs its own timer/clock. Could I use to this package to tie individual timers to each project and update those times to the server?

You probably could use it but not quite in the way you describe. I think you would just store a startTime in the database when they click "Start" for a project timer, and then you would use this package only for displaying the elapsed time reactively in the browser. To do that you'd do something like
Template.myTimerPage.onCreated(function () {
  this.clock = new ReactiveClock();

  this.autorun(() => {
    let project = Projects.findOne(someId);
    let elapsed = (Date.now() - project.startTime.getTime()) / 1000;
    this.clock.setElapsedSeconds(elapsed);
  });

  this.clock.start();
});

Template.myTimerPage.helpers({
  displayTime: () => {
    return Template.instance().clock.elapsedTime({format: '00:00:00'});
  }
});


feat(ui): update theme colors #466

Updated colors within the ThemeColors.ts file.
The issue requested that the colors off-white-1 and off-white-2 be added. It's worth mentioning that there is already a theme called offwhite, so we may want to change these offwhite-1 and offwhite-2 to keep naming consistent.
We could also change the name to off-white instead, but this would require us to replace the CSS variable across the project.

This change is

It appears that UnoCSS doesn't support kebab-cased colors. The hyphen is reserved for color prefixes e.g. "ut-..."


css: Move bootstrap-focus-style from id to class

The commit af36e9f added a bug that breaks new user invite. The CSS class  bootstrap-focus-style was added to id hence breaking the value extraction.
Fixes: #24249
Screenshots and screen captures:

Self Review checklist

[x] Self-reviewed the changes for clarity and maintainability
(variable names, code reuse, readability, etc.).

Communicate decisions, questions, and potential concerns.

[ ] Explains differences from previous plans (e.g., issue description).
[x] Highlights technical choices and bugs encountered.
[ ] Calls out remaining decisions and concerns.
[ ] Automated tests verify logic where appropriate.

Individual commits are ready for review (see commit discipline).

[x] Each commit is a coherent idea.
[x] Commit message(s) explain reasoning and motivation for changes.

Completed manual review and testing of the following:

[x] Visual appearance of the changes.
[ ] Responsiveness and internationalization.
[ ] Strings and tooltips.
[ ] End-to-end functionality of buttons, interactions and flows.
[ ] Corner cases, error conditions, and easily imagined bugs.

@prashantvithani Thanks for fixing this. Can you please update the line wrapping in commit message as per guidelines?

Thanks for review @sahil839. I've updated the commit message as per the guidelines.

Thanks! You should also add a period at the end of commit summary (first line) as the guidelines mention.

I have checked other instances of this class as well. There was one case where the class was added to div instead of select, fixed it in #24253. All other things are fine.

Merged, thanks for catching and fixing this!


As a pack user, I only see a single copy of the generated image SHAs

Remove "wall of SHAs" coming from ggcr (but keep exporter SHAs).
Acceptance Criteria
When I build an OCI image with pack build and any combination of valid options
Then I don't see any SHAs that originate from the go-containerregistry
And I see the SHAs intentionally output by the exporter

Not seeing SHAs generated from go-containerregistry when I run pack build, accepting.


Filtered stream fixes

streamConnect should not get any parameter on line 139, 141
the third param of needle.get is not an object, but the options are the second param, where timeout should be too

 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.

Thank you - fixed in 521c873


Added the "multiFileDrag" binding to allow multiple files in Drag and Drop

The knockout-file-bindings library does not currently support multiple files (see issue #5 on the original repo). This PR should allow users to manage multiple files through the multiFileDrag binding, and should not interfere with any single file functionality.
Also made some minor changes to remove commented out code and updated the documentation to reflect the changes

@safrazik - I'm interested in this feature could you review this?

So basically you want multiple file upload support? I think supporting only
"drag and drop" is not enough. I will be working on the issue:
https://github.com/adrotec/knockout-file-bindings/issues/16 and will give
an update soon

Yeah, that would be great. I did start to have a look last night but probably won't get a chance for a few days to get anything done.

multiple file uploads support added recently in a proper way, including previews (This pull request added support only for drag and drop). Check the updated docs https://github.com/adrotec/knockout-file-bindings#multiple-file-uploads


Adding Windows machine support to set timezones

Tests added for calls to Windows class
Updates for Gemfile for easier install of gem

Thanks a lot Robert, this looks great! I'm not going to be able to test this, but I'll add some Ruby convention fixes and merge later today.

Released in v1.2.0. Thanks again!


[7.x.x] Bump Dependencies #4873717853

Bumps dependencies for product-is. Link : https://github.com/wso2/product-is/actions/runs/4873717853

 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.jenkins-is-staging seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


what happened to picard-tool submodule?

Hi David,
I have repaired the submodule link for picard-tool. Please let me know if you experience further problems with it.
Kami Chiotti


Preferred names for manual categories

Is your feature request related to a problem? Please describe.
We can use the dendrogram to select rows/cols and assign custom categories. However, we would like to have a list of preferred categories. This would make it easier for users to assign cell type using a common vocabulary and prevent issues like "CD8 T Cells", "CD8 T Cell", "CD8" all being names for the same thing.
Describe the solution you'd like
We will build in a argument in the manual_category method that takes a list of preferred categories. We will then pass this into the dendrogram input list as a HTML5 datalist.

[x] load metadata for preferred categories that includes colors
[x] have these preferred categories show up under autocomplete input text box

Describe alternatives you've considered
None
Additional context
None

this has been implemented as of version 0.13.0


Great Job.  Great big THANK YOU!!

This is FANTASTIC.
Thank you for providing this immense collection of snippets and examples.

thx ParkinT :-)


Unnecessary tab index on pre elements?

The audit feature in Astro dev toolbar says tabindex=0 doesn't belong on the <pre>-elements. Thoughts?

Do we need it to be focusable or is the copy button enough? Doesn't look like CTRL+C works on <pre> focus either way. Any other benefits? Relevant if scrollable maybe?

The attribute is intentional. According to our a11y testing, the attribute is required for screen readers to properly read the code. The warning is therefore incorrect and can be ignored.

Thanks for the quick response. Found a relevant discussion for reference 🙂
IMO the rule should be fixed.

Thank you for linking this! I wanted to look it up as well and didn't find it quickly. :) Here's another one that I remember as well: https://github.com/withastro/astro/issues/8001
I'm not sure if we can do anything to avoid that supposedly incorrect flagging of the attribute by the toolbar. If more users report this in the future, we might need to investigate further.

The audit appears to be smart enough that it doesn't flag elements with overflow (and therefore a scrollbar).


feat: framework agnostic core

Is your feature request related to a problem? Please describe.
create-t3-app is currently coupled to React as a consequence of using Next for SSR. However, I think there's value in create-t3-app's best practices for full stack TypeScript that don't necessarily need to be coupled to React or Next.
Here are some advantages to supporting multiple framework backends over just React:

Migrating to and from other frameworks
Ease of learning for users and contributors experienced with frameworks
Flexibility of not coupling your project to React and Next, but also not having to switch to an alternative like Vite or Astro that may not have as much type safety

One of the things I like about Vite and Astro is knowing I can change my mind later, but as it currently stands if I want to make a Next, Turbopack, or T3 app, I'm forced to choose React

Type safety by using a framework written in TypeScript (Angular/Solid/Svelte/Vue/etc.)

Potentially better if you use a framework that doesn't use JSX due to https://github.com/Microsoft/TypeScript/issues/14729

Future proofing by allowing the lifespan of create-t3-app to potentially be longer than React's or Next's

Describe the solution you'd like to see
For now, I think the easiest way to achieve this would be to refactor some of the internals to use Vite and the SSR plugin. This would make SSR support framework agnostic and should make it easy to support Preact, Vue, Svelte, Solid, and Angular. For back compatibility, we would either need to figure out how to bridge the gap between missing/different Next features, or move the current internals into a separate feature flag or plugin package.
Describe alternate solutions
Turbopack could be an option in the future when it's no longer exclusively supported as a build system for Next.
Alternatively, if this is too high level or out of scope for the T3 maintainers, it could be maintained as a separate project, and create-t3-app could adapt parts of it for Next or React.
Additional information
No response

There are some Create T3 App-inspired init tools based on other frameworks already
Solid: https://github.com/OrJDev/create-jd-app
Nuxt: http://sidebase.io/
Svelte: https://github.com/zach-hopkins/create-t3svelte-app
These are great, but I think they should remain separate projects. Even when using a similar set of packages other than the framework itself, the codebases end up being so different that I don't see much value in unifying them as one project. It would also multiply (exponentially?) the work we'd need to do - we're often the first project to hit issues in the libraries that we use, which means we need to debug them, open issues, decide what to do in the meantime before it's fixed, etc.
Who knows what the future will bring, but for the foreseeable future we will be tied to Next. That's not the right choice in all cases, but it's a great choice in many cases, and we don't need to be everything for everyone.


Allow versions of requests > 2.24.0

Installing drone-python==1.0.0 causes a dependency conflict on the version of requests (I'm using requests==2.32.x elsewhere). Suggest changing the requirement spec to allow v2.24.0 or later versions

I'm forgetting if there was a reason I had pinned requests to a specific version. I'll take a look again, Thanks for the PR!


Use vcf2zarr in GWAS tutorial notebook

[ ] Fixes #1257
[ ] Tests added
[ ] User visible changes (including notable bug fixes) are documented in changelog.rst
[ ] New functions are listed in api.rst

Thanks @jeromekelleher. Here's the new notebook (updates in the "Importing data from VCF" section):
https://github.com/sgkit-dev/sgkit/blob/7094d3cf192dfc25ff69456ec7f1e71e7df2c264/docs/examples/gwas_tutorial.ipynb
I'll merge it later today.

LGTM - short and sweet!


fix: typing isSelected, isPointed props for option, groupLabel slots

Fix #403

@adamberecz ping :)


Python InteractiveRunner works when evaluated as a global but not in a function

The following snippet works when I run it in a Jupyter notebook (e.g., Google Colab):

import apache_beam as beam

transform = beam.Create([1, 2, 3])

p = beam.Pipeline('InteractiveRunner')
pcoll
= p | transform
result = p.run()
result.wait_until_finish()
print(result.get(pcoll))  # [1, 2, 3]

However, if I try to put creating/evaluating the pipeline into a helper function it fails with a mysterious error:

import apache_beam as beam

def evaluate(transform):
  p = beam.Pipeline('InteractiveRunner')

 pcoll = p | transform
  result = p.run()
  result.wait_until_finish()
  return result.get(pcoll)

transform
= beam.Create([1, 2, 3])
evaluate(transform)

results in

---------------------------------------------------------------------------
AttributeError
                    Traceback (most recent call last)
<ipython-input-33-271cfe7be46f> in <module>()

     9
     10 transform = beam.Create([1, 2, 3])
---> 11 evaluate(transform)

<ipython-input-33-271cfe7be46f>
in evaluate(transform)
      4   p = beam.Pipeline('InteractiveRunner')
      5   pcoll = p | transform
---->
6   result = p.run()
      7   result.wait_until_finish()
      8   return result.get(pcoll)

/usr/local/lib/python3.7/dist-packages/apache_beam/pipeline.py
in run(self, test_runner_api)
    557         finally:
    558           shutil.rmtree(tmpdir)
-->
559       return self.runner.run_pipeline(self, self._options)
    560     finally:
    561
shutil.rmtree(self.local_tempdir, ignore_errors=True)

/usr/local/lib/python3.7/dist-packages/apache_beam/runners/interactive/interactive_runner.py
in run_pipeline(self, pipeline, options)
    134
    135     # Make sure that sources without a user
reference are still cached.
--> 136     inst.watch_sources(pipeline)
    137
    138     user_pipeline
= ie.current_env().user_pipeline(pipeline)

/usr/local/lib/python3.7/dist-packages/apache_beam/runners/interactive/pipeline_instrument.py
in watch_sources(pipeline)
   1006           ie.current_env().watch({'synthetic_var_' + str(id(pcoll)):
pcoll})
   1007
-> 1008   retrieved_user_pipeline.visit(CacheableUnboundedPCollectionVisitor())

AttributeError:
'NoneType' object has no attribute 'visit'

It would be nice if this worked, or failing that, if at least an instructive error message was printed :)

Imported from Jira BEAM-12233. Original Jira may contain additional context.
Reported by: shoyer.

@KevinGG could you help triage this? Thank you!

This is intended behavior.
When a pipeline is not defined in the __main__ module, it's not known by Beam to do any magic for it because it's local.
To make it work inside non-main modules/scopes, add ib.watch(locals()) or ib.watch({'pipeline_name': pipeline, 'pcoll_name': pcoll}) to notify Beam to watch them.
And use ib.collect and ib.show instead of the result = p.run() + result.get(pcoll) pattern because you don't need to run the whole pipeline to materialize a single pcollection.

Feel free to reopen if the answer does not work for you.


Update to the latest monaco-editor

Hey, @rw-access thank you for your support. I'll try to publish the new version this week, but before that, you can use the config method to download your desired version. It should look like this:
import loader from '@monaco-editor/loader';
// or import { loader } from '@monaco-editor/react';

loader.config({ paths: { vs<EMAIL_ADDRESS>} });


gNOI-3.1: Cancel reboot on test failure

When a reboot test fails, it could leave a pending reboot active scheduled for 2 hours in the future. For each reboot action, defer a CancelReboot to stop all future reboots.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 46.471%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
586

Relevant Lines:
1261

💛  - Coveralls

/gcbrun


https://detekt.dev/docs/next/rules/potential-bugs/#unnamedparameteruse

I am trying to add UnnamedParameterUse rule as mentioned here.
Expected Behavior
No error when executing ./gradlew detekt.
Observed Behavior
Execution failed for task ':app:detekt'.
> Run failed with 1 invalid config property.
        - Property 'potential-bugs>UnnamedParameterUse' is misspelled or does not exist.

Steps to Reproduce
Detekt version
[versions]
plugin-detekt = "1.23.6"
detekt-formatting = "1.23.6"
detekt-rules-compose = "1.3.0"

[libraries]
detekt-formatting = { group = "io.gitlab.arturbosch.detekt", name = "detekt-formatting", version.ref = "detekt-formatting" }
detekt-rules-compose = { group = "ru.kode", name = "detekt-rules-compose", version.ref = "detekt-rules-compose" }

[plugins]
detekt = { id = "io.gitlab.arturbosch.detekt", version.ref = "plugin-detekt" }

Context
Trying to enable UnnamedParameterUse rule in existing project.
Your Environment

Version of detekt used:  Shared above
Version of Gradle used (if applicable):
Gradle scan link (add --scan option when running the gradle task):
Operating System and version:
Link to your project (if it's a public repository): https://github.com/Abhimanyu14/finance-manager

That's because the rule you're referring to is included only in Detekt 2.0.0 (you can see it as the URL you provided has next in it).
The problem is that you're using Detekt 1.23.6 where this rule does not exist

Hi @cortinico,
I see that the detekt latest version is v1.23.6.
How can I get detekt 2.0.0?

There are only snapshots of that version. It's in development right now.


Cannot see add-on after adding repository

Select Add-On (place a lowercase x for the relevant add-on)

[ ] Xiaomi Mi Scale

Describe the bug
When I add https://github.com/lolouk44/hassio-addons to repositories, nothing appears. No error when adding repository.
To Reproduce
just add https://github.com/lolouk44/hassio-addons to repositories
Expected behaviour
to see addon
Screenshots

Scale (please complete the following information):
Does not matter now.
Device running Home Assistant (please complete the following information):

Device used to run the Script/Container [e.g. Raspberry Pi, NUC]
Bluetooth device used [e.g. Built-in, USB Dongle]

Home Assistant Version:
2023.4.6
Additional context
no

What do the logs show ?

Thanks for confirming. Closing issue...


Bug: cannot set return value of DI'ed pipe

HI there!
Description of the bug
I try to set the return value of a DI'ed pipe's transform() function, but it doesn't. I also expect() the function .toHaveBeenCalled(), but it's never called. It's as if my test and my component have different instances of the pipe.
It could simply be that I'm doing something wrong or just forgot something somewhere. Couldn't find this specific use case in the docs and I believe I'm doing everything correctly, so I'd thought I'd file a bug.
An example of the bug
I have this relatively boring pipe:
@Pipe({ name: 'bla' })
export class BlaPipe implements PipeTransform {
  transform(value: any, ...args: any[]) {
    return `hi there ${value}`;
  }
}

It resides in this module, where I provide the pipe for use with DI:
@NgModule({
  declarations: [BlaPipe],
  exports: [BlaPipe],
  providers: [BlaPipe],
})
export class PipesModule {}

I then try to use this pipe:
@Component({
  selector: 'app-root',
  standalone: true,
  imports: [PipesModule],
  templateUrl: './app.component.html',
})
export class AppComponent {
  constructor(private blaPipe: BlaPipe) {}

  doSomething() {
    let val = this.blaPipe.transform('Frank');
    console.log('result of pipe transform:', val);
    return val;
  }
}

Which all works as expected. But then I try to test it. I have the autoSpy() set:
ngMocks.autoSpy('jasmine');

getTestBed().initTestEnvironment(BrowserDynamicTestingModule, platformBrowserDynamicTesting());

and then the test itself:
describe('AppComponent', () => {
  let blaPipe: jasmine.SpyObj<BlaPipe>;
  let sut: AppComponent;

  beforeEach(() => MockBuilder(AppComponent));

  beforeEach(() => {
    blaPipe = TestBed.inject(BlaPipe) as jasmine.SpyObj<BlaPipe>;
    // blaPipe = ngMocks.findInstance(BlaPipe) as jasmine.SpyObj<BlaPipe>;

    blaPipe.transform.and.returnValue('test');

    sut = TestBed.createComponent(AppComponent).componentInstance;
  });

  it('should use the blaPipe', () => {
    let result = sut.doSomething();

    expect(blaPipe.transform).toHaveBeenCalled();
    expect(result).toBe('test');
  });
});

And I ran into assertion failures:
Expected spy MockOfBlaPipe.transform to have been called.
Expected undefined to be 'test'.

Link: https://github.com/JPtenBerge/ng-mocks-mock-pipe-bug
Expected vs actual behavior
Expected: that the pipe's transform function would've returned my configured return value.
Actual: it just returns undefined. It seems to not have been setup. It's as if my component receives a different instance than when I use TestBed.inject() or ngMocks.findInstance().

I can't get the example from the 'Auto Spy' documentation to work either.
This test passes. It's clear here that findInstance() gets it right.
You tried finding the instance before creating the component though.
describe('AppComponent', () => {
  let blaPipe: jasmine.SpyObj<BlaPipe>;
  let blaPipe_found: jasmine.SpyObj<BlaPipe>;
  let sut: AppComponent;

  beforeEach(() => MockBuilder(AppComponent));

  beforeEach(() => {
    sut = TestBed.createComponent(AppComponent).componentInstance;

    blaPipe = TestBed.inject(BlaPipe) as jasmine.SpyObj<BlaPipe>;
    blaPipe_found = ngMocks.findInstance(BlaPipe) as jasmine.SpyObj<BlaPipe>;

    blaPipe.transform.and.returnValue('test');
    blaPipe_found.transform.and.returnValue('test_found');

    console.log("injected: ", blaPipe);
    console.log("findInstance: ", blaPipe_found);
  });

  it('should use the blaPipe', () => {
    let result = sut.doSomething();

    expect(blaPipe_found.transform).toHaveBeenCalled();
    expect(result).toBe('test_found');
  });
});

I noticed here that the injectors are different. Not sure if this is relevant

Tested on Angular 17.0.10


Payment Observer Got Stuck

Investigate why Payment Observer got stuck.  This instance payment observer was not running as a separate service.  Part of this investigation should make sure this scenario would be covered by payment observer running as a separate k8s healthcheck.
Logs included in Jira. ANCHOR-129

In order for the /health endpoint to report error for the observer, it must be running in a separated service.
I had also investigated the cause of the long catch-up time. The observer reads the last cursor (eg:<PHONE_NUMBER>49242920) stored in the database, then tries to catch up from the last cursor. The longer the observer is stuck, the longer it takes to read from the production network.
We reduced the problem in the develop branch: StellarPaymentObserver.java#277, so that the observer only reads up to the most recent MAX_RESULT (200) transactions instead of from the last known cursor.
The fix also handles the testnet reset. (edited)


[opentelemetry-demo] Issue with DATABASE_URL in Feature flag service

Database URL set in feature flag service is connecting to ffs-postgres, without taking into consideration the installation name.
https://github.com/open-telemetry/opentelemetry-helm-charts/blob/7421d9abecae37c5b9e04e31489a4d08188fb466/charts/opentelemetry-demo/values.yaml#L92-L93
Say we run
helm install mydemo ./opentelemetry-demo
the postgres service would be mydemo-ffs-postgres
we see error 16:07:22.958 [error] Postgrex.Protocol (#PID<0.1854.0>) failed to connect: ** (DBConnection.ConnectionError) tcp connect (ffs-postgres:5432): non-existing domain - :nxdomain

/cc @wph95


readme is gone?

I'm getting an error on the readme link.

I used a google code url when moving from there to github, should be better now.


[api-docs] Remove embed visualizers in community resources

A new commit was made into the api-docs repo. This issue was created automatically. Need to review if we made any change to the library for this commit.
https://github.com/discord/discord-api-docs/commit/af649d421ad0247eddb2484034c893409600edeb

This commit will not effect our lib's code. No changes required.


Implement basic skeleton & networking

I've stripped out everything from the draft project by @Geod24 and left in the networking. It's basic stuff, but it works, and we can get started implementing other things like interacting with SCP, etc.
I can demonstrate how multiple nodes connect with each other in person.
Edit: I wish to make everything unit-testable, perhaps with the framework @Geod24 previously implemented.

Update:

Fixed documented config to mention Agora.
Fixed extra blank line
Added config verification which should be called both after parsing, and after creating a manual config for unittests
Made array fields in the config immutable (see next line)
Added shared aliasing tests to make sure these configs can always be used with the LocalRest library
Removed unecessary key_pair field which can just be part of the Config file instead now that it's immutable
Changed retry-delay from integer to floating-point. This allows setting it to a few milliseconds which is useful in order to speed up unittests.
Fixed uneccessary "() ()" template
Removed unused Hash module
Fixed a node from trying to connect to itself during discovery
Fixed fiber sleep calls to consistently use the configured sleep time
Moved creation of the initial task directly into the Node class. This allows the node to auto-connect in unittests without having to manually run its node.start() method.

And:

Added a commit implementing a very basic network discovery test-case. Much more will be added, this was just a proof-of-concept. But the test already uncovered a few bugs that are now fixed.

@AndrejMitrovic AndrejMitrovic force-pushed the AndrejMitrovic:basic-networking  branch from ace94d4 to 04b9a1d 1 minute ago

Unfortunately this shows the wrong diff because I force-pushed the wrong changes at first.

I fixed most stuff except the dub test target which still fails. Why does dub have to be so difficult..

There is also some missing documentation. Can be added soon or later after merging if I don't make it today.

Ok, managed to fix the dub test target.

I also renamed Client to RemoteNode to make it a bit clearer.

Ok, managed to fix the dub test target.

Nope..

So the reason why I was confused my tests didn't always build/link is because in the first commit dub test fails, but in the second one it doesn't. It's really weird. But with both commits in it works so I dunno.


Cannot read property 'authorId' of undefined

i got this from logs
TypeError: Cannot read property 'authorId' of undefined     at exports.pushNotificationLikes.functions.database.ref.onWrite.event (/user_code/index.js:24:38)     at Object.<anonymous> (/user_code/node_modules/firebase-functions/lib/cloud-functions.js:112:27)     at next (native)     at /user_code/node_modules/firebase-functions/lib/cloud-functions.js:28:71     at __awaiter (/user_code/node_modules/firebase-functions/lib/cloud-functions.js:24:12)     at cloudFunction (/user_code/node_modules/firebase-functions/lib/cloud-functions.js:82:36)     at /var/tmp/worker/worker.js:710:26     at process._tickDomainCallback (internal/process/next_tick.js:135:7)
the deploy was successful

@Jombreng, The problem was obtained from the update of Firebase SDK for Cloud Functions  to version 1.0.
Cloud function in this repository were written for sdk v. 0.5. Version 1.0 has different syntax. You should migrate to the new version using this guide.
Updated version of the functions will be able in the next version of the project.

Thank you,  i fix it already


sqlite/chinook returns PANIC: Could not parse stored DateTime string: 2009-01-01 00:00:00: ParseError(Invalid) on findMany()

chinook.db.zip
With alpha 845
/home/runner/work/introspection-engine-output/introspection-engine-output/clients/sqlite/chinook.db
11 models
- albums findMany({}) Error:
Invalid `prisma.albums.findMany()` invocation in
/home/runner/work/introspection-engine-output/introspection-engine-output/clients/sqlite/chinook.db/index.js:568:31

PANIC: Could not parse stored DateTime string: 2009-01-01 00:00:00: ParseError(Invalid)
- artists findMany({}) Error:
Invalid `prisma.artists.findMany()` invocation in
/home/runner/work/introspection-engine-output/introspection-engine-output/clients/sqlite/chinook.db/index.js:851:32

PANIC: Could not parse stored DateTime string: 2009-01-01 00:00:00: ParseError(Invalid)
- customers findMany({}) Error:
Invalid `prisma.customers.findMany()` invocation in
/home/runner/work/introspection-engine-output/introspection-engine-output/clients/sqlite/chinook.db/index.js:1126:34

PANIC: Could not parse stored DateTime string: 2009-01-01 00:00:00: ParseError(Invalid)
- employees findMany({}) Error:
Invalid `prisma.employees.findMany()` invocation in
/home/runner/work/introspection-engine-output/introspection-engine-output/clients/sqlite/chinook.db/index.js:1409:34

PANIC: Could not parse stored DateTime string: 2009-01-01 00:00:00: ParseError(Invalid)
- genres findMany({}) Error:
Invalid `prisma.genres.findMany()` invocation in
/home/runner/work/introspection-engine-output/introspection-engine-output/clients/sqlite/chinook.db/index.js:1700:31

PANIC: Could not parse stored DateTime string: 2009-01-01 00:00:00: ParseError(Invalid)
- invoices findMany({}) Error:
Invalid `prisma.invoices.findMany()` invocation in
/home/runner/work/introspection-engine-output/introspection-engine-output/clients/sqlite/chinook.db/index.js:1975:33

PANIC: Could not parse stored DateTime string: 2009-01-01 00:00:00: ParseError(Invalid)
- invoice_items findMany({}) Error:
Invalid `prisma.invoice_items.findMany()` invocation in
/home/runner/work/introspection-engine-output/introspection-engine-output/clients/sqlite/chinook.db/index.js:2258:38

PANIC: Could not parse stored DateTime string: 2009-01-01 00:00:00: ParseError(Invalid)
- media_types findMany({}) Error:
Invalid `prisma.media_types.findMany()` invocation in
/home/runner/work/introspection-engine-output/introspection-engine-output/clients/sqlite/chinook.db/index.js:2541:36

PANIC: Could not parse stored DateTime string: 2009-01-01 00:00:00: ParseError(Invalid)
- playlists findMany({}) Error:
Invalid `prisma.playlists.findMany()` invocation in
/home/runner/work/introspection-engine-output/introspection-engine-output/clients/sqlite/chinook.db/index.js:2816:34

PANIC: Could not parse stored DateTime string: 2009-01-01 00:00:00: ParseError(Invalid)
- playlist_track findMany({}) Error:
Invalid `prisma.playlist_tracks.findMany()` invocation in
/home/runner/work/introspection-engine-output/introspection-engine-output/clients/sqlite/chinook.db/index.js:3084:39

PANIC: Could not parse stored DateTime string: 2009-01-01 00:00:00: ParseError(Invalid)
- tracks findMany({}) Error:
Invalid `prisma.tracks.findMany()` invocation in
/home/runner/work/introspection-engine-output/introspection-engine-output/clients/sqlite/chinook.db/index.js:3352:31

PANIC: Could not parse stored DateTime string: 2009-01-01 00:00:00: ParseError(Invalid)

generator client {
    provider = "prisma-client-js"
    output = "../clients/sqlite/chinook.db"
  }
datasource db {
  provider = "sqlite"
  url      = "file:../sqlite_dbs/chinook.db"
}

model albums {
  AlbumId  Int      @default(autoincrement()) @id
  Title    String
  ArtistId artists
  tracks   tracks[]

  @@index([ArtistId], name: "IFK_AlbumArtistId")
}

model artists {
  ArtistId Int      @default(autoincrement()) @id
  Name     String?
  albums   albums[]
}

model customers {
  Address      String?
  City         String?
  Company      String?
  Country      String?
  CustomerId   Int        @default(autoincrement()) @id
  Email        String
  Fax          String?
  FirstName    String
  LastName     String
  Phone        String?
  PostalCode   String?
  State        String?
  SupportRepId employees?
  invoices     invoices[]

  @@index([SupportRepId], name: "IFK_CustomerSupportRepId")
}

model employees {
  Address    String?
  BirthDate  DateTime?
  City       String?
  Country    String?
  Email      String?
  EmployeeId Int         @default(autoincrement()) @id
  Fax        String?
  FirstName  String
  HireDate   DateTime?
  LastName   String
  Phone      String?
  PostalCode String?
  State      String?
  Title      String?
  ReportsTo  employees?  @relation("employeesToemployees_ReportsTo")
  customers  customers[]
  employees  employees[] @relation("employeesToemployees_ReportsTo")

  @@index([ReportsTo], name: "IFK_EmployeeReportsTo")
}

model genres {
  GenreId Int      @default(autoincrement()) @id
  Name    String?
  tracks  tracks[]
}

model invoices {
  BillingAddress    String?
  BillingCity       String?
  BillingCountry    String?
  BillingPostalCode String?
  BillingState      String?
  InvoiceDate       DateTime
  InvoiceId         Int             @default(autoincrement()) @id
  Total             Float
  CustomerId        customers
  invoice_items     invoice_items[]

  @@index([CustomerId], name: "IFK_InvoiceCustomerId")
}

model invoice_items {
  InvoiceLineId Int      @default(autoincrement()) @id
  Quantity      Int
  UnitPrice     Float
  InvoiceId     invoices
  TrackId       tracks

  @@index([TrackId], name: "IFK_InvoiceLineTrackId")
  @@index([InvoiceId], name: "IFK_InvoiceLineInvoiceId")
}

model media_types {
  MediaTypeId Int      @default(autoincrement()) @id
  Name        String?
  tracks      tracks[]
}

model playlists {
  Name       String?
  PlaylistId Int     @default(autoincrement()) @id
}

model playlist_track {
  /// This used to be part of a relation to playlists
  PlaylistId Int
  /// This used to be part of a relation to tracks
  TrackId    Int

  @@id([PlaylistId, TrackId])
  @@index([TrackId], name: "IFK_PlaylistTrackTrackId")
}

model tracks {
  Bytes         Int?
  Composer      String?
  Milliseconds  Int
  Name          String
  TrackId       Int             @default(autoincrement()) @id
  UnitPrice     Float
  AlbumId       albums?
  GenreId       genres?
  MediaTypeId   media_types
  invoice_items invoice_items[]

  @@index([MediaTypeId], name: "IFK_TrackMediaTypeId")
  @@index([GenreId], name: "IFK_TrackGenreId")
  @@index([AlbumId], name: "IFK_TrackAlbumId")
}

More context with DEBUG=*
The error message needs to follow the error spec besides the bug.


[Matter v1.4 SVE] TC_SWTCH_2_6 Fails on 7b: Not Applicable for My Device

Describe the bug
I have executed TC_SWTCH_2_6 . 4a, 5a and 6a step passed. step from 7a isn't applicable for esp_generic_switch because we are not supporting MSL feature. So it is expected that the TC should pass but it is failing.please find the failure logs.
Steps to reproduce the behavior
commission generic switch and execute TC_SWTCH_2_6
Expected behavior
It should pass because steps from 7a NA for my DUT
Log files
TC-SWTCH-2-6-sep-27-count3-esp32c3.log
TC-SWTCH-2-6-sep-27-count3-TH.log
PICS file
No response
Screenshots

Environment
No response
Additional Information
No response

@raul-marquez-csa , could you take a look on this?


Improve 403 error page for logging in

Describe the bug
If students try to log in with their AAD account but our app isn't approved by their admin, they'll see a blank page with no information on what happened. We should have some documentation on how admins can add our app to their AAD, and also explain what students can do after seeing this page
This is what the page looks like:

@eanders-ms do we plan to address this for hour of code release?

No plan to address it. Issue was on our side. We were blocking the tenant in staging. It worked fine in prod. We don't have a real-world instance of this happening.

@livcheerful were you trying to connect to staging subscription instead of prod?

I think so, I can't get it to repro on live

Ok thanks! I am closing this bug as won't fix.


fix: Add ADT URL query parameter support

Add optional support to define the ADT endpoint as query parameter with the name adtUrl=...
Example: http://localhost:3000/?adtUrl=https://prod-wus-sample-adt.api.wus2.digitaltwins.azure.net

Abandoning this as its supported in the hosted version.  We are considering changes that will contribute hosted version improvements back into the public repo so that the deltas between the hosted version and the public repo remain minimal


Implement nospec code blocks for alternate syntax

This applies to the alternate_syntax parser only
If a ruby code block is preceded by a <!-- :nospec: --> (and optional whitespace lines)
than it shall not be included as a Context::Include node into the AST and be ignored

As HTML comments do not work in Bitbucket's converter :angry: we need another trigger,
e.g.
Let us trigger includes with keywords like Given and When

Done in 0.4.0


How read custom file classes?

I created my model on Google Colab to enjoy the GPU mode with a custom class.
After, I saved the model model.model.save('model.h5') and downloaded only this file.
In my machine, I loaded the model model = deepforest.deepforest(saved_model=MODEL_PATH) and I can predict the custom class, but the class name is only Tree.
I created a classes.csv file in the root folder, but how read this file to save the correct label on the predict?
Thank's

I'm not sure how I've missed this issue. Probably during the holiday break. My apologies. I'm sure its too late, but I just ran into the same issue myself and will update the code base. The key was model.read_classes().

model.classses

be careful its in the same order, probably safer to use the same tool as during training
from deepforest import utilities
model.classes_file = utilities.create_classes(annotations)

Added example to docs
A couple notes on multi-class, if you trained a multi-class model and need to reload the model, make sure to reload the classes file, or else all objects will be labeled (tree)[https://github.com/weecology/DeepForest/issues/150]

from deepforest import deepforest
from deepforest import utilities
m = deepforest.deepforest("/orange/ewhite/everglades/Zooniverse/predictions/20210211_072221.h5")
m.classes_file = utilities.create_classes("/orange/ewhite/everglades/Zooniverse/parsed_images/test.csv")
m.read_classes()

Also note there is likely some integration errors with the comet dashboard, I recommend not using comet for multi-species models, as there are too many assumptions for single tree species.

Thank's for your reply.

Thank you, sorry for the very late reply, normally i'm on top of these. I
pushed 0.3.6 to make sure that custom classes are sorted alphanumeric, this
will help among runs. I still see some issues restarting a saved model with
custom classes. I expect to address this at the beginning of next week.
Feel free to open another issue.
On Fri, Feb 12, 2021 at 9:25 AM Anderson<EMAIL_ADDRESS>wrote:

Thank's for your reply.
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/weecology/DeepForest/issues/150#issuecomment-778330611,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAJHBLEZBMX6NZB7G3HIIYLS6VQCPANCNFSM4UVDBUKA
.

--
Ben Weinstein, Ph.D.
Postdoctoral Fellow
University of Florida
http://benweinstein.weebly.com/

from deepforest import utilities
model.classes_file = utilities.create_classes(<path_to_annotations>)
In the above, what is the classes file contain? Is it same as the annotation file with image_path,xmin,ymin,xmax,ymax,label?
Usecase:
I have a custom data, for which I want to identify 2 classes (cropname and weed).
Steps followed as per documentation:

split raster into smaller tiles.
Prepared train and test.
Trained for few epochs
initialized deepforest model and changed the corresponding config file
created pytorch lighting trainer
loaded the lastest release model
Saved the prediction result
Result:
The above works fine for single class
problem:
the identified class is tree and not cropname
How to change this task from single to multiclass problem


Bitterly Dissapointed. Does not work in Chrome or Chromium Edge

The title says it all really. Followed all step and even tried the download versions. They do n ot work in either Chrome or Edge.
We get as far as seeing in IMG placeholder that is approx. 10x10px in an active (online) tab but it errors thereafter,

Now that's interesting. It does work - without me changing or reloading anything... Potential bug in chromium!

@Unkind-Chris since it looks like the issue was resolved, I'll go ahead and close this issue. Thanks.

Thanks @Reezaali


feat: logger and emit errors

Fixes #38
Fixes #37

In your opinion should we remove async from start and emit the listening event instead? Like we do for error? Like a normal server does @balloob @Michsior14

In your opinion should we remove async from start and emit the listening event instead? Like we do for error? Like a normal server does @balloob @Michsior14

Porque no los dos?

Porque no los dos?

"Normal server" 🤷
Async awaiting a task is absolutely preferred. It means you know the error that is raised is related to you calling that function. An error event can be emitted by any other action that is currently in progress.

"Normal server" 🤷
Async awaiting a task is absolutely preferred. It means you know the error that is raised is related to you calling that function. An error event can be emitted by any other action that is currently in progress.


Error running the comand "az ad sp create-for-rbac"

This was following the guide in the step 2
ARM_CLIENT_SECRET=$(az ad sp create-for-rbac \
  --name http://tf-sp-$UNIQUE_ID \
  --role Contributor \
  --scopes "/subscriptions/$ARM_SUBSCRIPTION_ID" \
  --query password \
  --output tsv)

This is autogenerated. Please review and update as needed.
Describe the bug
Command Name
az ad sp create-for-rbac
Errors:
400 Client Error: Bad Request for url: http://localhost:50342/oauth2/token
Traceback (most recent call last):
python3.6/site-packages/knack/cli.py, ln 215, in invoke
    cmd_result = self.invocation.execute(args)
cli/core/commands/__init__.py, ln 631, in execute
    raise ex
cli/core/commands/__init__.py, ln 695, in _run_jobs_serially
    results.append(self._run_job(expanded_arg, cmd_copy))
cli/core/commands/__init__.py, ln 686, in _run_job
    cmd_copy.exception_handler(ex)
...
python3.6/site-packages/msrestazure/azure_active_directory.py, ln 486, in get_msi_token
    result.raise_for_status()
python3.6/site-packages/requests/models.py, ln 940, in raise_for_status
    raise HTTPError(http_error_msg, response=self)
requests.exceptions.HTTPError: 400 Client Error: Bad Request for url: http://localhost:50342/oauth2/token

To Reproduce:
Steps to reproduce the behavior. Note that argument values have been redacted, as they may contain sensitive information.

Put any pre-requisite steps here...
az ad sp create-for-rbac --name {} --role {} --scopes {} --query {} --output {}

Expected Behavior
Environment Summary
Linux-4.15.0-1082-azure-x86_64-with-debian-stretch-sid
Python 3.6.5
Installer: DEB

azure-cli 2.5.1 *

Additional Context

hi @JaviRpo just to confirm, did you see the error in Cloudshell?

@yungezz yes, it was in Cloudshell.
The error is intermittent. I closed the Cloudshell, open a new one, and the error didn't happen anymore.

hi @JaviRpo sorry for late response, this is an known issue in Cloudshell #11749. As a workaround, pls try run az login.

Hi @JaviRpo , do it work after you run az login. If no more concern, I will close this first, CloudShell problem will be traced in #11749.


Coordinator descriptors

Re-implementation of this original WIP PR from the legacy repo: https://github.com/unchained-capital/caravan/pull/340
There are a couple new UI elements:

A switch to choose if you want to import by xpubs/devices or descriptors

The descriptor importer itself

A button to download the descriptors as a txt file from the wallet overview

The UX is still a little rough around the edges (for example, you have to select the network and client FIRST before importing the descriptor), but that can be cleaned up later (ideally with a more comprehensive rewrite that does a wizard style wallet setup flow).
Here's a descriptor from one of the test suite fixtures to test with:
wsh(sortedmulti(2,[f57ec65d/48'/0'/100'/2']xpub6DcqYQxnbefzFkaRBK63FSE2GzNuNnNhFGw1xV9RioVG7av6r3JDf1aELqBSq5gt5487CtNxvVtaiJjQU2HQWzgG5NzLyTPbYav6otW8qEc/1/*,[00000006/48'/0'/100'/2']xpub6EwJjKaiocGvqSuM2jRZSuQ9HEddiFUFu9RdjE47zG7kXVNDQpJ3GyvskwYiLmvU4SBTNZyv8UH53QcmFEE23YwozE61V3dwzZJEFQr6H2b/1/*))#kmvcnunj

Annoyingly I had messed up the build pipeline which broke the ci workflow. Should be working now.


Build for MacOS (arm64,amd64) not working

Describe the bug:

The build for MacOS is not working with version 1.5.0.
I have tested with different SDK versions.
The following error is shown:
flag provided but not defined: -w"

Device and debug info (please complete the following information):
Device info

OS: Linux -->
Target-OS: MacOS
Version: Fedora 39
Go version: 1.21.
fyne-cross version: 1.5.0
Fyne version: 2.4.4

Do you have any go flags at in your environment? Can you share the command line you are using?

I have not set any go flags.
The commands are:
fyne-cross darwin -arch=amd64 --macosx-sdk-path ${SDK_DIR} ./cmds/gui
fyne-cross darwin -arch=arm64 --macosx-sdk-path ${SDK_DIR} ./cmds/gui

SDK_DIR points to MacOSX11.3.sdk.
If fyne-cross is fixed to v1.4.0 it build succeeds.

I am able to reproduce the problem. Will investigate.

It seems this does not affect build on darwin host targeting darwin.


Handlers follow ups

Here are the follow-ups we discussed in the earlier PR.

I should probably add some tests to this.

Trying it against gdb reveals some issues.

Aha, it now checksums the $ and the checksum (lol) which is wrong.

I made the changes and tacked on a couple more things.  Now gdb can attach and get through several commands before having problems.


feat(integrations): Add Keras3 Callbacks

Description

[x] Add WandbMetricsLogger for Keras3 which works with Keras2 and the Tensorflow, PyTorch, and JAX backends of Keras3.
[x] Add WandbModelCheckpoint for Keras3 which works with Keras2 and the Tensorflow, PyTorch, and JAX backends of Keras3.
[x] Add deprecation warning for the old callbacks compatible only with Keras2.

Usage Example for WandbMetricsLogger
from wandb.integration.keras3 import WandbMetricsLogger
model.fit(
    X_train,
    y_train,
    validation_data=(X_test, y_test),
    callbacks=[WandbCallback()],
)

Usage Example for WandbModelCheckpoint
from wandb.integration.keras3 import WandbModelCheckpoint
model.fit(
    X_train,
    y_train,
    validation_data=(X_test, y_test),
    callbacks=[WandbModelCheckpoint(filepath="model.keras")],
)

+1; this seems like a really nice PR and I was also looking for this functionality.

Hello
When will this feature be implemented and are there other solution than downgrading to keras 2 for using WandbMetricsLogger ?


TypeInitializationExtensions - use DependencyResolver?

@JimBobSquarePants - Following on from #20 - do you see any issue with us swapping out using Activator with DependencyResolver?

As log as it falls back to Activator as the default then I think it's fine. In 99% cases I very much doubt anyone would be using dependency injection.
On the note of TypeConverters I wanted to as whether we should establish a collection of built in converters for the various Umbraco datatypes. I already have working code for most of the main ones.

I agree, but now that the DI topic has been raised, it's how far do we go with it?
// @stevetemple - What are your thoughts on this? Is it an over-generalisation to be replacing every occurrence of Activator.CreateInstance with DependencyResolver.Current.GetService?

My spelling in that last comment was appalling!
I'd like to also consider performance as a factor in feature design so any insight on the impact of the change from that perspective would be most useful.

@JimBobSquarePants - It appears that the GetService of the default resolver (DefaultDependencyResolver) literally just calls Activator.CreateInstance (wrapped in a try/catch):
see here.
re: Built-in TypeConverters (to ship with Ditto) ... sounds good, open a separate issue/PR.
Would this mean having built-in POCOs? (not as keen on that)

@leekelleher That's good to know. I would definitely us DependencyResolver in that instance then but leave any other Activator.CreateInstance usages throughout the codebase intact since they don't involve anything that we'd want too many people messing with.
re: Built-in TypeConverters. No POCO's are used in any implementations I have done but I couldn't be sure about if we tried to do grid or related links.

A good built in TypeConverter would be one for the new Grid stuff in 7.2

OK, let's open a new PR and see where we go with built-in TypeConverters :muscle:

One of the guys here is actually writing that grid TypeConverter already so we'll send that over once it's done

Cool, thanks @stevetemple!

Ace. I'll open a PR in an hour or so with a few.


Trailing slash on void elements has no effect and interacts badly with unquoted attribute values.

hi
Can I remove trail slash from meta tags ?
I send my site to html validator and get next
Info: Trailing slash on void elements [has no effect](https://github.com/validator/validator/wiki/Markup-%C2%BB-Void-elements#trailing-slashes-in-void-element-start-tags-do-not-mark-the-start-tags-as-self-closing) and [interacts badly with unquoted attribute values](https://github.com/validator/validator/wiki/Markup-%C2%BB-Void-elements#trailing-slashes-directly-preceded-by-unquoted-attribute-values).

From line 9, column 9; to line 9, column 100
<meta property="og:title" content="site title" />

Yeah, feel free to send a PR


Adapt secure vault config changes from puppet base

Applying secure vault configurations via puppet templates is removed in puppet base. Instead, we simply copy the secure vault files via puppet file resources. This change needs to be adapted in order to ensure the functionality of secure vault with the latest puppet base.
Please do note that secure vault will not work with previous puppet-bases after this change is adopted.

Refer to PR wso2/puppet-apim#41 when adapting the changes


[Bug]: Add Query option is not disaplyed to the user on Firestore

Is there an existing issue for this?

[X] I have searched the existing issues

Description
When the user establishes Firestore DS connection it is observed and the user navigates to creating query it is observed that the schema on the right side must show "add Query" option as per walkthrough but schema pane does not the the the
Steps To Reproduce

Establish a Firestore connection
Click on "Add Query"
Navigate to Query page
and observe the schema on the right side
a) Does not display (add icon) and drop to create Queries
b) Walkthrough is displayed to the user

Public Sample App
No response
Environment
Production
Issue video log
No response
Version
Cloud

@prapullc reduce prio to Low according to this.


Add support for statsd export

The goal of this PR is to enable collection of proxy stats. This PR focuses on generating envoy configs to direct statsd output.
This PR updates the proxy agent and the manager to support a new field in the mesh config: statsd_udp_address. The initial default option is to generate this address from the IP address of Mixer and the configured statsd-udp port (defaults to 9125).
Tested on new deployments. A PR to istio/istio that updates the deployment configs for mixer, pilot, and prometheus will follow.
This has a current weakness: if mixer service is deleted and restarted, export will stop working, as the statsd_udp_address is not regenerated in response.

There is a linter warning (run bin/check.sh)

Jenkins job pilot/presubmit passed

Jenkins job pilot/e2e-smoketest passed

Jenkins job pilot/presubmit passed

@kyessenov I've updated to do resolve before agent instantiation and added to ingress/egress per your comments. I believe this matches what you were asking for. PTAL.

Jenkins job pilot/presubmit passed

Jenkins job pilot/presubmit passed

Jenkins job pilot/e2e-smoketest passed

Thanks!

Jenkins job pilot/presubmit passed

Jenkins job pilot/e2e-smoketest passed

Thanks for the good review, and the merge!


Omnisharp should check and log the presence of the .NET Core SDK on every launch

At present, omnisharp checks for the presence of the .NET Core SDK only upon initial install. A user may uninstall the .NET Core SDK but not the C# extension. In that scenario, upon opening C# files in VS Code, they should be presented with the dialog that reports that they do not have a .NET Core SDK installed, and given the opportunity (via the dialog) to install it.

This is checked on each run.


feat(be): add api of getting contest' users information

Description
closes TAS-460
특정Contest에 참가한 '모든 참여자들의 성적 및 유저정보' 를 가져오는 api

Additional context

Before submitting the PR, please make sure you do the following

[ ] Read the Contributing Guidelines
[ ] Read the Contributing Guidelines and follow the Commit Convention
[ ] Provide a description in this PR that addresses what the PR is solving, or reference the issue that it solves (e.g. fixes #123).
[ ] Ideally, include relevant tests that fail without this PR but pass with it.

https://github.com/skkuding/codedang/pull/2029 에서 해결


TCE repository generation API

What this PR does / why we need it

Expands the proposal for creating TCE releases by defining a Go API for automation of the tasks.
Details for the Release Notes (PLEASE PROVIDE)

NONE

Which issue(s) this PR fixes

Fixes: #2717

Closing this out in favor of #3186, which has actual Go code.


feat: change encryption key

Also added some tests.
I did not test it on web, as I am not sure how to.
I have left an unimplemented block, I will add support for removing encryption later on.
That block should not currently be reachable from the dart api.
Closes #1541

Please also raise this PR for isar-community to main branch

Why is taking too long to merge anything nowadays @simc

Because Isar is unmaintained for months. You should switch to Isar-Community fork...
Am 25. März 2024 16:37:57 MEZ schrieb Muragijimana @.***>:

Why is taking too long to merge anything nowadays @simc
--
Reply to this email directly or view it on GitHub:
https://github.com/isar/isar/pull/1567#issuecomment-2018300020
You are receiving this because you commented.
Message ID: @.***>

--
Martin Clauss
Meine Contactulater Card - All meine Kontaktinfo immer aktuell und vollständig:
https://contactulater.app/me/mc

Not right now, check out the documentation on Https://Isar-Community.dev
Am 25. März 2024 18:44:11 MEZ schrieb Muragijimana @.***>:

Yes is isar community released on pub.dev?
On Mon, Mar 25, 2024, 19:39 mrclauss @.***> wrote:

Because Isar is unmaintained for months. You should switch to
Isar-Community fork...
Am 25. März 2024 16:37:57 MEZ schrieb Muragijimana @.***>:

Why is taking too long to merge anything nowadays @simc
--
Reply to this email directly or view it on GitHub:
https://github.com/isar/isar/pull/1567#issuecomment-2018300020
You are receiving this because you commented.
Message ID: @.***>

--
Martin Clauss
Meine Contactulater Card - All meine Kontaktinfo immer aktuell und
vollständig:
https://contactulater.app/me/mc
—
Reply to this email directly, view it on GitHub
https://github.com/isar/isar/pull/1567#issuecomment-2018550581, or
unsubscribe

https://github.com/notifications/unsubscribe-auth/ACBCOJH4RS5E3H7YKPQSZBDY2BOONAVCNFSM6AAAAABDYGN26GVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDAMJYGU2TANJYGE

.
You are receiving this because you commented.Message ID:
@.***>

--
Reply to this email directly or view it on GitHub:
https://github.com/isar/isar/pull/1567#issuecomment-2018557797
You are receiving this because you commented.
Message ID: @.***>

--
Martin Clauss
Meine Contactulater Card - All meine Kontaktinfo immer aktuell und vollständig:
https://contactulater.app/me/mc


Kata Containers 1.1.0

agent: honour CPU constraints when agent is the init process
mount: Remove redundant mkdir
network: Setup localhost when running as init
agent: update container cpuset cgroup parents
Fix update cpus - cpuset
agent: Add support for ephemeral volumes
namespace: Share pid namespace of containers
grpc: signal frozen containers
Get cgroup mounts info from /proc/cgroups
namespaces: Create persistent UTS and IPC namespaces
grpc: implement pause and resume commands

4018add agent: honour CPU constraints when agent is the init process
1cff660 mount: Remove redundant mkdir
cc74b5d network: Setup localhost when running as init
dc06ec7 network: Add extra sanity checks
b580ae7 network: Check all function parameters
ee106e9 main: Exit(1) on error
5bf3909 network: Reduce cyclomatic complexity
cdf9bce namespace: Simplify code
0d7f36f agent: update container cpuset cgroup parents
4cf1809 agent: update container cpuset cgroup before setting the new configuration
d70e5bb agent: don't modify container's config
a30395a grpc: signal frozen containers
08674c0 agent: Add support for ephemeral volumes
c517125 ci: Refactor to pass metalinter checks
ad9c33d tests: Modify tests to take into account if pid ns needs to be shared
e334596 namespace: Share pid namespace of containers
1e72fbb agent: Get cgroup mounts info from /proc/cgroups
ab6c4ff namespaces: Create persistent UTS and IPC namespaces
75c6fc0 grpc: implement pause and resume commands

lgtm


Added optimizer support regarding hash partitions.

This PR added optimizer support for hash partitions in CreateTable / Insert / InsertSelection / UpdateTable.
Assigned to @hbdeshmukh. Thanks!

Hi @zuyu, Would you mind assigning @jianqiao to review this PR? I believe he will be a better reviewer for this PR.

LGTM except some minor style issues.

Fixed the comments. Thanks!


Show Import Wizard context menu option when connected directly to a database

Is your feature request related to a problem? Please describe.
The Import Wizard context menu option shows if you're connected to the <Default> database and then you right click on the database name. However, it does not appear if you specify a database when adding the connection, then right click on the server/database.
I believe this might have been what was reported in #7043 but that issue is now locked for comment.
Describe the solution or feature you'd like
Would like the Import Wizard option when right clicking on a database-specified connection.
Describe alternatives you've considered
None, as that is the only way I'm aware of to access the Import Wizard. To workaround, I have to add a new connection to <Default>.
Additional context

This issue was addressed by #16700


[flutter_tools] try_merge_upstream script should fail if there were upstream cirrus config changes

If there were upstream updates to our cirrus config, we probably shouldn't bother running CI with the old config as it's a waste of resources and probably a waste of time for the engineer, who may end up troubleshooting a bug that's already been solved.
Currently blocked until https://github.com/flutter/flutter/pull/52840 lands.

Obsolete.


Metal pagination should offer early escape criteria

When using client-side filtering against a paginated resource list, the pagination helpers could accept criteria functions to determine if pagination can be concluded early.
If there are 1000 devices that I need to sift through to find the one with a particular set of tags (https://github.com/kubernetes-sigs/cluster-api-provider-packet/issues/630), I should be able to take advantage of pagination helpers and not request 100 pages of data if my single desired matching result was found on page 1.
See related packngo issue https://github.com/packethost/packngo/issues/206

In the event that a user doesn't want the automatic pagination provided by ExecuteWithPagination, they always have the option of using the Execute method instead and navigating the pages as they see fit.


yargs 5

reference: https://github.com/yargs/yargs/blob/master/CHANGELOG.md
No breaking API changes introduced.

Awesome! If you merge it, I will try it on my current CLIs :-)

Just waiting for travis. Merging right after that.


"SyntaxError: invalid syntax" from running latest diff-cover

diff-cover just started failing in our CI environment, perhaps as a result of the latest release. The same command seemed to work yesterday. Note that the change I made that triggered the latest build did change coverage, and I have not tried to verify whether this error occurs with a different version of diff-cover or with the older version of my project.
Running "shell:diff-cover" (shell) task
Traceback (most recent call last):
  File "/usr/bin/diff-cover", line 9, in <module>
    load_entry_point('diff-cover==1.0.7', 'console_scripts', 'diff-cover')()
  File "/usr/lib/python2.6/site-packages/pkg_resources/__init__.py", line 542, in load_entry_point
    return get_distribution(dist).load_entry_point(group, name)
  File "/usr/lib/python2.6/site-packages/pkg_resources/__init__.py", line 2570, in load_entry_point
    return ep.load()
  File "/usr/lib/python2.6/site-packages/pkg_resources/__init__.py", line 2230, in load
    return self.resolve()
  File "/usr/lib/python2.6/site-packages/pkg_resources/__init__.py", line 2236, in resolve
    module = __import__(self.module_name, fromlist=['__name__'], level=0)
  File "/usr/lib/python2.6/site-packages/diff_cover/tool.py", line 18, in <module>
    from diff_cover.report_generator import (
  File "/usr/lib/python2.6/site-packages/diff_cover/report_generator.py", line 18
    violation.line for violation in violations
                     ^
SyntaxError: invalid syntax

Oh jeez. I'm sorry to cause you all issues on a weekend....
Im a bit shocked my CI did not catch this. Ill take a look right away... (I have to see if I still support python2.6 officially actually...)

Yeah, looking over this. I dropped 2.6 support a while back which would explain why CI would not have caught the error... though I am a tad skeptical that this is our first use of 2.6 invalid syntax.
but let me do a quick check to confirm

Ok, yeah
So python2.6 support broke around november of 2017 and I decided it was not worth continuing to support it.
https://github.com/Bachmann1234/diff-cover/commit/75efbbac12f38c0491b0cf738053e750113917fe#diff-b91f3d5bd63fcd17221b267e851608e8
If upgrading to 2.7 (or 3 ) is not an option try pinning to https://github.com/Bachmann1234/diff-cover/releases/tag/v1.0.6
Hope that helps!

@Bachmann1234 Sounds good. Thanks Matt! Someone else had updated the version of diff-cover (from some significantly earlier version) without me realizing it. The release of 1.0.7 seems to have just been a coincidence. Thanks for maintaining a great project that continues to be very valuable to our team!

This is a dead issue but talking somewhere I just realized there is a fairly easy work around if you wanted you update diff cover without updating your other python stuff
Diff cover is a tool not a library. So it's not like it would break your code to remove diff cover from the dependencies of your project and have a separate and more modern python interpreter installed just to run tools.
Your call if it's worth it but just thought I'd put it here for the record


Error "has no properties in common with type 'FormlyFieldConfig'" with Angular 4.4

I'm submitting a ...  (check one with "x")
[x] bug report => search github for a similar issue or PR before submitting
[ ] feature request
[x] support request

Current behavior
I just copied the instructions from the README 1:1 into my application and started the app. The complete error message is:
ERROR in .../ui/src/app/routes/users/users/users.component.ts (17,5): Type '{ className: string; fieldGroup: ({ className: string; key: string; type: string; templateOptions...' has no properties in common with type 'FormlyFieldConfig'.

Expected behavior
Expected to get a form ;)
Minimal reproduction of the problem with instructions
As i said, just copied the instructions from the README
Please tell us about your environment:

Ubuntu 16.04.3 LTS
npm 5.4.0

Angular version:
4.4.0-RC.0
Language:
Typescript: 2.5.2

Args, solved it myself, sorry still learning angular2+/typescript. Error was the type definition for userFields. Instead of
userFields: FormlyFieldConfig = ...

it should be
userFields: FormlyFieldConfig[] = ...

Maybe this should be reflected in the README


breakpoint destroys settings

Hello,
I've been using your gallery for my project, and it works great!
I've noticed that
  this.galleryOptions = [
      {
         breakpoint: 800
      }
];

doesn't work. In fact, when combined with other settings, none of them apply.

Hi, could you give me more info? Maybe some real example?

Lukasz,
this.galleryOptions = [
{
breakpoint: 960,
thumbnailsColumns: 3,
width: '95%',
height: '90%'
},
{
width: '52vw',
height: '90vh',
thumbnailsColumns: 6,
thumbnailsRows: 2,
imageAnimation: NgxGalleryAnimation.Fade,
imageSize: 'contain',
imagePercent: 80,
thumbnailsPercent: 20,
thumbnailsMargin: 0,
thumbnailMargin: 1,
imageArrowsAutoHide: true,
thumbnailsArrowsAutoHide: true,
preview: false
}
];
anything less than 960px, the gallery hides when I added width and height to the default settings. When I remove this, the 960 breakpoint settings work, but the gallery doesn't look good on Desktop. I also tried setting multiple breakpoints to no avail.


Convolution: Dilation

Feature description
Support dilation in convolution layers.

Related PR: https://github.com/burn-rs/burn/pull/301
@nathanielsimard I assume this ticket can be closed now?


add static built binaries

Fix: #475
This patch enables build statically linked binaries with the following command:
make STATIC=1

Tested on alpine docker image for both commands:

soci
/soci # ./soci create docker.io/library/ubuntu:latest
layer sha256:dbf6a9befcdeecbb8813406afbd62ce81394e3869d84599f19f941aa5c74f3d1 -> ztoc sha256:372417343d4b4b741e6b940c5829a7cb5d934c3edcae0af504e6ff9d5e029cdd

soci-snapshotter-grpc
/soci # ./soci-snapshotter-grpc
{"level":"info","msg":"starting soci-snapshotter-grpc","revision":"3f21ac241646e219b4986578a6f6e59a98d94f64","time":"2023-05-12T05:28:57.964955078Z","version":"3f21ac2"}
{"error":"failed to mount overlay: operation not permitted","level":"fatal","msg":"snapshotter is not supported","time":"2023-05-12T05:28:57.965572192Z"}

The snapshotter failed to due to missing overlay fs. But the binary was able to run.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Just for comparison
Without STATIC=1
total 65M
-rwxr-xr-x 1 admin admin 25M May 15 18:56 soci
-rwxr-xr-x 1 admin admin 40M May 15 18:56 soci-snapshotter-grpc

With STATIC=1
total 60M
-rwxr-xr-x 1 admin admin 20M May 15 18:55 soci
-rwxr-xr-x 1 admin admin 41M May 15 18:55 soci-snapshotter-grpc


 If you use cask one, you cannot use this QSyncthingTray because of a conflict:
❯ Installing qsyncthingtray (qsyncthingtray) into /Applications ...
==> Satisfying dependencies
==> Installing Formula dependencies from Homebrew
syncthing ... ==> Error: The `brew link` step did not complete successfully
Error: Command failed to execute!

==> Failed command:
["#<Pathname:/usr/local/bin/brew>", "install", "syncthing"]

==> Output of failed command:
==> Downloading https://homebrew.bintray.com/bottles/syncthing-0.12.11.el_capitan.bottle.tar.gz
Already downloaded: /Library/Caches/Homebrew/syncthing-0.12.11.el_capitan.bottle.tar.gz
==> Pouring syncthing-0.12.11.el_capitan.bottle.tar.gz
The formula built, but is not symlinked into /usr/local
Could not symlink bin/syncthing
Target /usr/local/bin/syncthing
already exists. You may want to remove it:
  rm '/usr/local/bin/syncthing'

To force the link and overwrite all conflicting files:
  brew link --overwrite syncthing

To list all files that would be deleted:
  brew link --overwrite --dry-run syncthing

Possible conflicting files are:
/usr/local/bin/syncthing -> /opt/homebrew-cask/Caskroom/syncthing/0.12.11/syncthing-macosx-amd64-v0.12.11/syncthing
==> Caveats
To have launchd start syncthing at login:
  ln -sfv /usr/local/opt/syncthing/*.plist ~/Library/LaunchAgents
Then to load syncthing now:
  launchctl load ~/Library/LaunchAgents/homebrew.mxcl.syncthing.plist
==> Summary
🍺  /usr/local/Cellar/syncthing/0.12.11: 6 files, 13.1M

==> Exit status of failed command:
#<Process::Status: pid 65269 exit 1>

I guess removing the dependency should make sense and allow users to use brew or cask one (not brew one does not provide upgrade via the web interface, which sucks).

The cask version will be removed, so this makes sense.

Humm...

I don’t get what you mean. What’s the difference?

The default UX to upgrade syncthing is a button in the web GUI.
brew recipe disable this ability (which make sense because brew handle its own version) so you must use a brew command to update syncthing.
The UX of a button and a command to type is not the same :)

@MoOx That’s actually a relevant point and seems like a case worthy or consideration. Care to submit that concern on https://github.com/caskroom/homebrew-cask/issues/15603?
Also, completely removing the dependency here isn’t the best solution. At beast it should be changed to depends_on cask:.

Good point. This PR was a way to start a discussion, which is a success to me :)
I will wait for some feedback before sending another PR according to your last comment, if this is relevant at this moment.

This is useful to you now, while to us it currently makes no difference, so feel free to make the change to depends_on cask: (link back to this issue for context).
When we get closer to the time of making a decision in #15603, we can revisit this.


mailgun.messages().send gives error 404 Not found.

{ Error:
<title>404 Not Found</title>

404 Not Found

nginx

    at IncomingMessage.<anonymous> (/Users/shashi/Workspace/mailgundemo/node_modules/mailgun-js/lib/request.js:309:15)
    at emitNone (events.js:91:20)
    at IncomingMessage.emit (events.js:185:7)
    at endReadableNT (_stream_readable.js:926:12)
    at _combinedTickCallback (internal/process/next_tick.js:74:11)
    at process._tickCallback (internal/process/next_tick.js:98:9) statusCode: 404 }

Please provide sample source code to reproduce the issue.

Sample Code
var api_key = 'api:APIKEY';
var domain = 'https://api.mailgun.net/v3/DOMAINNAME/messages';
var Mailgun = require('mailgun-js');

//We pass the api_key and domain to the wrapper, or it won't be able to identify + send emails
    var mailgun = new Mailgun({apiKey: api_key, domain: domain});

    var data = {
    //Specify email data
      from<EMAIL_ADDRESS>    //The email to contact
      to<EMAIL_ADDRESS>    //Subject and text data
      subject: 'Hello from Mailgun',
      html: 'Hello,There I am using mailgun'
    }

    //Invokes the method to send emails given the above data with the helper library
    mailgun.messages().send(data, function (err, body) {
        //If there is an error, render the error page
        if (err) {
            //res.render('error', { error : err});
            console.log("got an error: ", err);
        }
        //Else we can greet    and leave
        else {
            //Here "submitted.jade" is the view file for this landing page
            //We pass the variable "email" from the url parameter in an object rendered by Jade
           // res.render('submitted', { email : req.params.mail });
            console.log(body);
        }
    });

Sometimes It gives 301 Moved Permanently error too. is there any issue with my system ports??

What version of the module is the issue happening on? Does the issue happen on latest version?

What platform and Node.js version? (For example Node.js 0.12 on Mac OS X)

Does the action work when you manually perform request against mailgun using curl (or other means)?

I ran into this issue too. I fixed it by replacing
var domain = 'https://api.mailgun.net/v3/DOMAINNAME/messages';
with
var domain = 'DOMAINNAME';
mailgun-js will append and prepend everything else. :)

i found same issue bt not resolve

k

sidenote: if you are using EU servers, you should also define host. I spend too much time debugging that. RTFM 😂

sidenote, might be helpful: if you are using EU servers, you should also define host. I spend too much time debugging that. RTFM 😂

sidenote, might be helpful: if you are using EU servers, you should also define host. I spend too much time debugging that. RTFM 😂

Thanks so much for hint, and just to remind others.
change the original code to the following one.
const mailgun = require('mailgun-js')({apiKey: apiKey, domain: domain,host:"api.eu.mailgun.net"})

Thank you very much!

@Tymek still it's mid-2020 already, that hint must have saved me hours of debugging. I still wonder why it's not mentioned in the docs

@null9beta here npmjs.com/package/mailgun-js#options. That's why I remarked RTFM :wink:
Maybe this should be more prominent. If you think so, please PR such a change.


Statamic 5 support

Hi,
It would be great to have Statamic 5 support for this plugin!

Hey @JoeriE I'm working on it!


Display start date on Webinar pages over published and updated date

 Webinars: Remove date created and updated and add time and date of webinar


ca.serial.txt missing kills the install with 3.6

BUG REQUEST INFO:
Environment:

Cloud provider: AWS

OS (printf "$(uname -srm)\n$(cat /etc/os-release)\n"):

All the machines are in AWS: installer which hosts this repo and installer tools is a micro instance, and it creates the OCP cluster by this repo using rhel 7.4 template.
$ uname -srm; cat /etc/os-release
Linux 3.10.0-693.1.1.el7.x86_64 x86_64
NAME="Red Hat Enterprise Linux Server"
VERSION="7.4 (Maipo)"
ID="rhel"
ID_LIKE="fedora"
VARIANT="Server"
VARIANT_ID="server"
VERSION_ID="7.4"
PRETTY_NAME="Employee SKU"
ANSI_COLOR="0;31"
CPE_NAME="cpe:/o:redhat:enterprise_linux:7.4:GA:server"
HOME_URL="https://www.redhat.com/"
BUG_REPORT_URL="https://bugzilla.redhat.com/"

REDHAT_BUGZILLA_PRODUCT="Red Hat Enterprise Linux 7"
REDHAT_BUGZILLA_PRODUCT_VERSION=7.4
REDHAT_SUPPORT_PRODUCT="Red Hat Enterprise Linux"
REDHAT_SUPPORT_PRODUCT_VERSION="7.4"

Version of Ansible: (ansible --version):

$ rpm -q ansible
ansible-<IP_ADDRESS>-3.el7.noarch

Version of Jinja: (pip freeze | grep -i jinja)

$ pip freeze | grep -i jinja
Jinja2==2.7.2

Version of Shade: (pip freeze | grep -i shade)

no such installed
Openstack-ansible-contrib version (commit) (git rev-parse --short HEAD):

$ git rev-parse --short HEAD
1ce66d0

Copy of used inventory files and custom variables (please omit your secrets!):
nothing extra
Command used to invoke ansible:
./ose-on-aws.py --keypair=$AWS_KEY --rhsm-user=$RHUSER --rhsm-password=$RHPWD \
  --public-hosted-zone=$DOMAIN --rhsm-pool=$RHPOOL \
  --deployment-type=openshift-enterprise --region=eu-west-1 \
  --ami=ami-bb9a6bc2  --app-dns-prefix=apps  \
  --containerized=true \
  --github-client-id $GH_CID --github-client-secret $GH_CSEC \
  --github-organization $GH_ORG --app-node-count 2

Output of ansible run:
TASK [openshift_master_certificates : Create the master server certificate] ****************************
skipping: [ose-master01.xxx] => (item=ose-master01.xxx)
failed: [ose-master01.xxx -> ose-master01.xxx] (item=ose-master02.xxx) => {
    "changed": true,
    "cmd": [
        "/usr/local/bin/oc",
        "adm",
        "ca",
        "create-server-cert",
        "--hostnames=<IP_ADDRESS>,kubernetes.default,internal-openshift-master.xxx,10.20.xxx,op
enshift.default,openshift-master.xxx,openshift.default.svc,ip-10-20-xxx.eu-west-1.compute.int
ernal,kubernetes,openshift.default.svc.cluster.local,kubernetes.default.svc,kubernetes.default.svc.clust
er.local,openshift",
        "--cert=/etc/origin/generated-configs/master-ip-10-20xxx.eu-west-1.compute.internal/master.se
rver.crt",
        "--key=/etc/origin/generated-configs/master-ip-10-20-xxx.eu-west-1.compute.internal/master.ser
ver.key",
        "--expire-days=730",
        "--signer-cert=/etc/origin/master/ca.crt",
        "--signer-key=/etc/origin/master/ca.key",
        "--signer-serial=/etc/origin/master/ca.serial.txt",
        "--overwrite=false"
    ],
    "delta": "0:00:00.175529",
    "end": "2017-08-18 20:03:23.015585",
    "failed": true,
    "item": "ose-master02.xxx",
    "rc": 1,
    "start": "2017-08-18 20:03:22.840056"
}

STDERR:

error: --signer-serial, "/etc/origin/master/ca.serial.txt" must be a valid file
See 'oc adm ca create-server-cert -h' for help and examples.

Anything else do we need to know:
I had to run the installer several times, as it stopped to different reasons. E.g. not being able to allocate memory, possibly micro instance is too small for ansible. But however, it got past all those, and ansible here didn't make sure if the ca.serial.txt file exists as it should. If the file didn't get created because of earlier failures, it should still be ensured to exist with some dummy value.
E. Rich from Red Hat found the spot where this fails, and his comments were:

I think you have a bug that you need to file: https://github.com/openshift/openshift-ansible/blob/master/roles/openshift_master_certificates/tasks/main.yml#L48-L72 is the code your erroring on.
However nowhere before this do we check / validate that we have the serial file. Using https://github.com/openshift/openshift-ansible/search?utf8=✓&q=openshift_ca_serial&type= I can't see where we ensure that file is created. To work around this just touch the file, or echo 1 > file on the server and re-run it should fix the issue.

I now echoed the 1 into serial file, I'll report back how it succeeded

This is a known downfall of the dynamic inventory. It has to do with no master being officially defined as the "first master" unlike the static inventory. In the appendix of the ref arch we have it defined how to work around this. I do not see us fixing this before the long term solution that is coming in 3.7.

Also seeing this on 3.6, its a total pain to be honest as it adds manual steps in to adding new nodes. Is there anything that could be done in Ansible to keep the files in sync until 3.7 is released? Surely its just a case of reading all 3 files and pushing the highest number out to all masters?

For the record, this has bitten Openshift Ops.  Since there isn't a "primary master" or "first master" ever defined, re-arranging the order of the masters in the inventory is breaking scale-ups.

Azure ref. arch. contains a workaround in the post installation steps (just copying ca.serial.txt from the first master to the masters)

This needs to be fixed, not a workaround. Thanks :)


WORKFLOWS: Diagnose why the e2e workflow is failing

The GitHub action for spinning up AWX and running playbooks seems to fail each time. I noticed it very occasionally being flaky in tower_configuration but it has never worked here. It seems to fail when waiting for AWX to come alive.
We need to diagnose and fix this issue

This seems to be working most of the time now since it was all merged into master. I can't see why that would make any difference but it seems to have 🤷♂ Will close this for now

Of course I close this and it starts failing again 😔


Client Simulator

The Client Simulator allows user to test Parameter Framework configurations. It allows to write scenario, mixing criterion change and arbitrary script execution.
It ease the development of automatic Parameter Framework configuration functional test.

Coverage remained the same at 70.13% when pulling cd1a7467c5d7f672baf2e697d09f56ca4776a9c6 on krocard:client_simulator into 9d1bcf046870f2c3bafede70dbb86c977b93c360 on 01org:master.

:+1: @dawagner @clero @sguiriec @Makohoek

:+1:

I'll review this at some point in time :)

We should add sample files for both documentation and testing (CI) purposes.


Windows builds missing

Comment:
If someone is interested, it should not take too much effort. I think all it takes is mirroring the patches in setup.py currently only for Unix to the Windows part.

Done: #14


ZUI: use new favicon

Update the favicon used in zui

@raulkele  I don't see the favicon in the browser. Is this done?


Bump chart's appVersion and fix chart repo on release job

Motivation
The appVersion was false and the helm release job was pushing to a wrong repo.
Changes
See commits.
Tests done
None, seems pretty straightforward.
TODO

[x] I've assigned this PR to myself
[x] I've labeled this PR correctly

Can you also adjust the latest version in the README?
https://github.com/caas-team/sparrow/blob/main/README.md?plain=1#L84

Can you also adjust the latest version in the README?
https://github.com/caas-team/sparrow/blob/main/README.md?plain=1#L84

Sorry, I don't think this is something we should do. Manual updates of the README, just to show how to install a specific version, don't really add any value to sparrow. It's nice updating everything and polishing the documentation every now and then, but this is where we should draw a line. Showing once how it's done is more than enough.
We already have to update the chart version manually each time we change something, and we need to remember to change the appVersion as well when we bump the sparrow's version. That's enough busywork for me.


If CrashPlan exceeds inotify's max watch limit - Read-only FS

I am attempting to follow these steps, https://support.code42.com/CrashPlan/4/Troubleshooting/Linux_real-time_file_watching_errors
However, step two is unable to be completed due to the container being in part a read-only FS.
/tmp # echo 1048576 > /proc/sys/fs/inotify/max_user_watches
sh: can't create /proc/sys/fs/inotify/max_user_watches: Read-only file system

This needs to be done on the host, not the container.
See https://github.com/jlesage/docker-crashplan-pro#inotifys-watch-limit

Doh, looks like I wasn't thinking forth dimensionaly. That fixed it. Thanks


Modify ErrorHandlerMiddleware constructing

https://github.com/spiral/framework/blob/master/src/Http/Middleware/ErrorHandlerMiddleware.php#L55
Having a bool param breaks the injection of such middleware bypassing the ErrorHandlerBootloader because container can't construct it. I suggest applying a class wrapper for suppressError param or assign an injector

resolved already