Identify and demo Sufia-specific features of interest for DAMS5

Following the current feature checklist: https://wiki.duraspace.org/display/hydra/Built-in+Feature+Checklist+-+Merge+Working+Document
It would be nice to demo some of the Sufia-only features that may be of interest to us. This could also help us provide feedback to the group working on which features merge between the two systems, which are turned into feature toggles, which are made into separate gems, etc.
Of potential specific interest (though we can discuss priorities as needed):

Batch Editing metadata
Fixity checking
Full-text indexing and searching
Google Analytics integration
Questioning Authority integartion
ResourceSync capability

I've followed the Sufia's instruction to create a Sufia-based app. And when run generator to install sufia, I got the following error:
"Users/hweng/git/my_sufia_app/config/environments/development.rb:53:in `block in <top (required)>': uninitialized constant ActiveSupport::EventedFileUpdateChecker (NameError)"
And after comment out the above line of config, this error was gone. But it comes up another error.

@mcritchlow I've tried different versions of Sufia and Rails. The following combination Rails <IP_ADDRESS> with Sufia 7.1.0 works fine to me to get the sufia based app up and running.
But I got issues when use sufia based app to create a new work object, which i posted the issue to hydra-tech list.

@hweng - Thanks for the update. Hopefully folks on the list will be able to assist. But glad to hear you at least got a combination working.

@mcritchlow To follow up the above issue, Mike Giarlo replied me in hydra-tech list:
"Hi Huawei,
OK, that helps. The error that you're seeing is resolved in the master branch of Sufia and will be released in 7.2.0, which should be released in the next week or so. Thanks for reporting this.
-Mike "

@mcritchlow Then I tried the master branch but there is some issues to install sufia using the current readme steps.
Mike Giarlo told me that there are draft release notes for upgrading to 7.2.0 that include required steps. Those will be published soon when the release is cut.

@hweng thanks for the update, that sounds about right. If there's a relevant Sufia ticket for this, would you mind referencing it here with a link?

@mcritchlow I didn't find particular relevant Sufia ticket for this issue. I get a reply from Justin said: this issue should be resolved with CurationConcerns 1.6.2, which I just released.
I will try it late.

@mcritchlow  Here is the hydra-tech list link regarding this issue: https://groups.google.com/forum/#!topic/hydra-tech/6zq9KrUmKDw

@mcritchlow The above error got resolved with CurationConcerns 1.6.2, which Justin Coyne just released on last Friday.
I've tried creating some different type of objects. It works well for me.

@mcritchlow  For the new released Sufia v7.2.0, I’ve tested and got errors with its installations which I created the issue here: https://github.com/projecthydra/sufia/issues/2764

@mcritchlow Tested batch uploading files in Sufia v7.1.0, it works fine to me. But I got errors when try batch editing, and not sure if it get fixed in v7.2.0. I will try it in v7.2.0 after the above install errors get fixed.

@mcritchlow The sufia 7.2.0 install issue is resolved for me, see https://github.com/projecthydra/sufia/issues/2764#issuecomment-252138293

Is this in progress? If so, please mark as such.

@hweng - I'm thinking we can perhaps mark this as Closed, would you agree? I know 7.3 is coming out in the near future, and I suspect that might be a good time to look into the ability to add in more complex metadata, extend batch editing, integrate QA, etc. What are your thoughts?

@mcritchlow Yes, I agree it will be a good investment looking into 7.3, which I suspect it would be more solid and extended features.


Can't click on Environments...  on 2.1.2

Click is not enabled but switching environments works:

I need to click on Docs tab to get Environments...  work.

Thanks

Looks like a bug a little z-index can fix. 😄 PR anyone?

I would love to pick this up but I can't replicate the issue on these browsers (Firefox, Opera, Chrome). Everything seems to work fine @Sparow199 @imolorhe

@mbao01 The problem happens when you have the dropdown over the query result. For that to happen, you would need to create some environments to make the dropdown long enough to get to the query result.
The same problem happens to the settings dropdown as well. The part of the dropdown over the query result isn't clickable.

Awesome. I've successfully replicated the issue and have corrected the behavior. I'll open a pr


API: source code link

It would be very convenient to have an optional field for link to source code for a particular application. Something like this:
$ cf app myapp
Showing health and status for app myapp in org rpaas / space dev as stas

name:              myapp
requested state:   started
instances:         1/1
usage:             256M x 1 instances
routes:            ...
last uploaded:     ...
stack:             cflinuxfs2
buildpack:         ...
source code:       https://git.domain.com/rpaas/myapp  # <====== HERE

     state     since                  cpu    memory          disk         details
#0   running   2017-09-20T04:58:00Z   0.0%   23.3M of 256M   109M of 1G

If this is implemented cf cli could be changed to detect source code link automatically when possible.

Hey @giner,
Thank you for this suggestion. We are in the early stages of design on an annotations feature similar to kubernetes' API. Would this meet your needs?
Thanks,
Greg

Hi @gerg,
It will do. Good to know you are working on this.

Hey @Gerg and @giner ,  I, also, would like to suggest an additional field on the next API version... I hope there is no problem to "get a ride" on this one (#904) since I believe it'similar... But I apologize if it's not the case, and let me know if you want me to open a new one just for my suggestion, will you?... :-)
So, my suggestion is to add a field, to the Service JSON, that would store the link for the OpenAPI (former Swagger) specification for the service - https://www.openapis.org/.
E.g.,
GET /v2/services
{
  "total_results": 1,
  "total_pages": 1,
  "prev_url": null,
  "next_url": null,
  "resources": [
    {
      "metadata": {
        "guid": "1993218f-096d-4216-bf9d-e0f250332dc6",
        "url": "/v2/services/1993218f-096d-4216-bf9d-e0f250332dc6",
        "created_at": "2016-06-08T16:41:31Z",
        "updated_at": "2016-06-08T16:41:26Z"
      },
      "entity": {
        "label": "label-57",
        "provider": null,
        "url": null,
        "description": "desc-134",
        "long_description": null,
        "version": null,
        "info_url": null,
        "active": true,
        "bindable": true,
        "unique_id": "4ab67d2e-18c0-4a36-8eed-fdee36fdd61b",
        "extra": null,
        "tags": [

        ],
        "requires": [

        ],
        "documentation_url": null,
        "openapi_url": null, # <====== HERE
        "service_broker_guid": "34b94a65-3cd3-4655-8c07-e2bd94ae21c5",
        "plan_updateable": false,
        "service_plans_url": "/v2/services/1993218f-096d-4216-bf9d-e0f250332dc6/service_plans"
      }
    }
  ]
}

Justification: As far as I understand, differently than "traditional" SOAP/XML webservices that you could just append "?wsdl" to an endpoint to retrieve all method signatures (operations available), for new RESTful services using Swagger/OpenAPI standard for documenting purpose, there is no way to automatically discover the url for the specification... :-( So, developers would benefit from knowing where to look for the OpenAPI specification, and applications could dynamically find and consume specific operations...
Ref.: https://apidocs.cloudfoundry.org/272/services/list_all_services.html

We are in the early stages of design on an annotations feature similar to kubernetes' API.

A couple of questions since we're on topic then:

will this work uniformly for all kinds of "objects" (apps, routes, spaces, orgs, services) or only for apps?
was it considered to control access to the metadata? e.g. I can imagine use cases for at least the following:

as operators we could use the ability to attach metadata that only admins can read/write and that only global auditors can read
as operators we could use the ability to attach metadata that only admins can read/write and that all users (with read access to the parent object) can read
as org/space manager we could use the ability to attach metadata that only org/space managers can read/write and that all users (with read access to the parent object) can read

this is scheduled and in progress! you can follow along here: https://www.pivotaltracker.com/story/show/160942050
Apps will be the first resource to support this sort of tagging, but the other resources you've mentioned are planned. It’s also important to note that, on our first pass, this labels+selectors api will NOT have access controls built in that are any different from access controls on the app itself. That means platform engineers & app developers using labels for chargebacks or other auditing will have to rely on an honor system.


Make handling of temp config dir better and tolerate exceptions during shutdown/cleanup

@TrevCraw Since the file that is locked is in the messaging\messageStore\Log directory, should we just skip messaging which would skip everything under that directory?


Make GHC base libraries dependent on GHC version.

See #84, #167.
Done for https://github.com/nh2/static-haskell-nix/issues/53

The CI fails with
Downloaded lts-13.15 build plan.
error: attribute 'ghc864' missing, at (string):1:69

so probably not my PR's fault.


Only package lib and docs in the npm published module.

Responding to #156
Sets files in the package.json to: lib, docs
I tested the package with npm pack. The resulting package contained the following:
CHANGELOG.md
LICENSE.md
README.md
docs
lib
package.json

Coverage remained the same at 100.0% when pulling 51f598e02b75db765f64f74ca4d0c5fbbe45166f on dont-publish-test-src into 9ee1242ccd03d45ffe5cc0b2457dd720a310f532 on master.


Plot all segments on a map

We currently have around 24,000 individual road and intersection segments in our data which is too much for Leaflet to handle.  It would be nice to find a way to be able to plot all of the segments in one map so that we can create some descriptive visualizations of the features in our dataset.

I was able to plot them by using geopandas .plot() method. It took quite a while (~5 min) and looked like a mess but it might be doable with some tweaking

Nice!  Does geopandas' plotting function make slippy maps?

I don't think so, it's basically just a matplotlib plot

Gotcha.  Yeah we'll want something that allows users to pan around and zoom in/out given the disparity between the size of the map and the level of detail people will want to be able to view it at.
Some very cursory research seems to point toward making our own maptiles but Uber just released a few tutorials on making maps using their own WebGL-based libraries which could be a solution as well: https://uber-common.github.io/vis-academy/#/

@alicefeng this is closed, is that right? Code is contained in the viz directory

@alicefeng this is closed, is that right? Code is contained in the viz directory

Haven't actually implemented this but yes, it does look like we've figured out a way to do it via Mapbox


Crash when attempting to quit again

Open the tray and click Quit
Wait, get impatient, reopen tray, and Quit again

Crash.
Specifications

OS: win32
IPFS Desktop Version: 0.6.0
Electron Version: 3.0.13
Chrome Version: 66.0.3359.181

Error
Error: connect ECONNREFUSED <IP_ADDRESS>:5001
    at TCPConnectWrap.afterConnect [as oncomplete] (net.js:1161:14)

Hello @da2x! Sorry for hearing this. Did it ever started up IPFS? Was the logo glowing?

@hacdias yes, I'd been using IPFS in the browser for a few hours. Then I wanted to quit the app (high CPU over several minutes). So opened the menu, clicked the quit icon, waited a minute, opened the menu again, and clicked quit again. The crash happen immediately following this. The state of IPFS was "exiting", so I don't really know whether it was running or not at the time.

I'm closing this since I think it is closed on the latest pre-release. If any of this happens again, please let me know.


Remove regression tests for trivially zero solutions

This PR removes regression tests for trivially zero solutions in the GABLS test.

Confirmed with @yairchn that this is the right thing to do. It should fix the remaining issue related to why CI is failing in #585.

bors r+


[Event Request] Codeunit 96 "Purch.-Quote to Order" - ArchivePurchQuote procedure

Hi,
We have this function on trigger OnRun on Codeunit 86 "Sales-Quote to Order":
ArchiveSalesQuote(Rec);
And this procedure has is own events with IsHandled boolean:
    local procedure ArchiveSalesQuote(var SalesHeader: Record "Sales Header")
    var
        ArchiveManagement: Codeunit ArchiveManagement;
        IsHandled: Boolean;
    begin
        IsHandled := false;
        OnBeforeArchiveSalesQuote(SalesHeader, SalesOrderHeader, IsHandled);
        if IsHandled then
            exit;

        case SalesSetup."Archive Quotes" of
            SalesSetup."Archive Quotes"::Always:
                ArchiveManagement.ArchSalesDocumentNoConfirm(SalesHeader);
            SalesSetup."Archive Quotes"::Question:
                ArchiveManagement.ArchiveSalesDocument(SalesHeader);
        end;
    end;

But on Codeunit 96 "Purch.-Quote to Order" we don't have the IsHandled boolean. We only have the case of Archive Quotes on the OnRun Trigger:
      PurchSetup.Get();
      case PurchSetup."Archive Quotes" of
          PurchSetup."Archive Quotes"::Always:
              ArchiveManagement.ArchPurchDocumentNoConfirm(Rec);
          PurchSetup."Archive Quotes"::Question:
              ArchiveManagement.ArchivePurchDocument(Rec);
      end;

We need the same procedure and event on Purchases. It would be something like:
    local procedure ArchivePurchQuote(var PurchHeader: Record "Purch Header")
    var
        ArchiveManagement: Codeunit ArchiveManagement;
        IsHandled: Boolean;
    begin
        IsHandled := false;
        OnBeforeArchivePurchQuote(PurchHeader, PurchOrderHeader, IsHandled);
        if IsHandled then
            exit;

       case PurchSetup."Archive Quotes" of
           PurchSetup."Archive Quotes"::Always:
               ArchiveManagement.ArchPurchDocumentNoConfirm(Rec);
           PurchSetup."Archive Quotes"::Question:
               ArchiveManagement.ArchivePurchDocument(Rec);
       end;
    end;

Thanks

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.
Build ID: 42716.


Add option to disallow intra word emphases only for underlines.

This solves an issue where asterisks are desired, but not underlines.
This is a common feature in a few engines, and noted by the CommonMark
spec.

Codecov Report

Merging #48 into master will increase coverage by 0.06%.
The diff coverage is 50%.

@@            Coverage Diff            @@
##           master     #48      +/-   ##
=========================================
+ Coverage   61.83%   61.9%   +0.06%
=========================================
  Files          14      14
  Lines        3448    3449       +1
  Branches     1056    1056
=========================================
+ Hits         2132    2135       +3
+ Misses        788     786       -2
  Partials      528     528

Impacted Files
Coverage Δ

bin/hoedown.c
42.73% <ø> (ø)
:arrow_up:

src/document.c
70.86% <50%> (+0.12%)
:arrow_up:

Continue to review full report at Codecov.

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

Thanks.


Stream cannot be consumed by StreamConsumer

Dart VM version: 2.3.2-dev.0.1 (Tue Jun 4 10:56:51 2019 +0200) on "macos_x64"

It seems that in newer versions of Dart, if I have a Stream<Uint8List>, I cannot use it with a StreamConsumer<List<int>>. This worked in the past, and in my head makes sense - because Uint8List is a descendant of List<int>, anything that accepts a List<int> should accept a Uint8List.
I've included two examples here:
https://github.com/thosakwe/uint8_repro/tree/master/bin
  error • The argument type 'AConsumer' can't be assigned to the parameter type 'StreamConsumer<B>' at bin/general_case.dart:16:21 • argument_type_not_assignable
  error • The argument type 'IntListConsumer' can't be assigned to the parameter type 'StreamConsumer<Uint8List>' at bin/uint8_list.dart:22:29 • argument_type_not_assignable

Both cases result in an argument_type_not_assignable error.

Why did this work before? A downcast from Uint8List to List<int> was already illegal in the analyzer before.
When were these semantics changed? I didn't see it in the changelog, and this worked as recently as Dart 2.2.0.
Stream.pipe is not very useful without support for polymorphism. I can also imagine that this change could break code for a lot of Dart users. For example, I ran into this error while reading files from package:file (using File.openRead), and piping the resulting stream to a StreamConsumer<List<int>>. In<EMAIL_ADDRESS>Uint8List is now returned instead of List<int>, and so anyone depending on that package (20 other packages on Pub) might already be running into this.

Thanks in advance.

Consider the general case:
import 'dart:async';

class A {}

class B implements A {}

class AConsumer implements StreamConsumer<A> {
  @override
  Future close() => null;

  @override Future<void> addStream(Stream<A> stream) async => null;
}

main() async {
  var stream = Stream.fromIterable([B()]);
  await stream.pipe(AConsumer());
}

The reason why this fails is that stream has type Stream<B>, so stream.pipe accepts an argument of type StreamConsumer<B>, and there is no subtype relationship between this type and the given type AConsumer. (We do have AConsumer <: StreamConsumer<A> and StreamConsumer<B> <: StreamConsumer<A>, but that's an up-then-down-cast and that doesn't amount to an upcast nor a downcast.)
You can avoid this conflict if you can make sure that stream has type Stream<A>:
main() async {
  var stream = Stream<A>.fromIterable([B()]);
  await stream.pipe(AConsumer());

  // or:
  var stream2 = Stream.fromIterable([B()]);
  await stream2.cast<A>().pipe(AConsumer());
}

However, there is an underlying reason for this conflict which is a bit deeper: The class StreamConsumer<S> uses its type argument just once, and that's in a method parameter type Stream<S>. This means that it could be declared in languages like C#, Kotlin, and Scala with a contravariant type argument (with different syntax, e.g., C# uses in as a type parameter modifier, and Scala uses -), but in Dart there is currently no support for declaring contravariant type arguments.
If it had been possible to declare the type argument of StreamConsumer as contravariant then we would have had AConsumer <: StreamConsumer<A> <: StreamConsumer<B>.
But given that we can't do that we can use the special case of invariance: Make sure that the same type argument is used everywhere (statically and dynamically).
In general, it fits much better for Dart to use a function type (where parameter types are actually contravariant) than a class type, in this kind of situation where there is a type parameter that should have been contravariant, but that's of course not an option when using a class like StreamConsumer which is required by the context.

Again, the situation doesn't allow for making changes to StreamConsumer, but just to illustrate the idea here's the core structure of the example, adjusted to use a function type such that we get the desired variance:
class A {}
class B implements A {}

class Stream<T> {
  Stream.fromIterable(Iterable<T> elements);
  Future pipe(StreamConsumer<Future Function(Stream<T>)> streamConsumer) =>
      null;
}

typedef F<X> = Future Function(Stream<X>);

abstract class StreamConsumer<F0 extends F<Null>> {
  F0 get addStream;
  Future close();
}

class AConsumer implements StreamConsumer<F<A>> {
  F<A> get addStream => null;
  Future close() => null;
}

main() async {
  var stream = Stream.fromIterable([B()]);
  await stream.pipe(AConsumer());
}

In this case we have no problem passing an AConsumer to Stream<B>.pipe, because AConsumer <: StreamConsumer<F<A>> <: StreamConsumer<F<B>>.

Thanks for the response! Seems like I'll have some reading to do...
Appreciate it.

Thanks!


Deferred interrupt handling for lock state

Goal: Use FreeRTOS capabilities for handling PIN_LOCK_LATCH_SWITCH and PIN_LOCK_ROTATION_SWITCH state changes.
Both pins should be attached to an ISR, which in turn notifies a waiting/blocking handler and takes care of anything from there on

Partly solved in #76f2806
Motor opening/parking should be put into another task


Question on HEVC bypass

Without editing the python files How could I allow bypassing on converting anything that's already in HEVC?  My aim is too bypass HEVC video but not the audio. As you can see I have tried getting around this.
Video: AVC -> HEVC = Yes
Video HEVC -> HEVC = audio only
As you can see I have tried getting around this.
[Video]
codec = h265vaapi, hevc
video-codec-fallback = hevc
conditional_copy = yes  # Enable conditional copying
conditional_copy_format = hevc  # Only copy if the format is HEVC
conditional_copy_streams = v  # Only copy the video stream
copy_hdr_metadata = yes
max-bitrate = 0
bitrate-ratio = 0.5
max-width = 0
preopts =
crf = -0
preset = medium
codec-parameters =
dynamic-parameters = True
profile = main
max-level = 5.0
pix-fmt = nv12, p010le
prioritize-source-pix-fmt = True
filter =
force-filter = False
crf-profiles =
x265-params = no-info=1

The option I am looking for would be
HEVC bypass =
If this can be done maybe it's a good idea to add others?
AV1 bypass =
AVC bypass =
Follow up question, if h265vaapi fails to work (sometimes it does) can a fallback formate be selected? I know this can be done if not using hwdevices.

The entire point of this script is to remux wherever possible which is essentially what you're requesting. Remuxing a stream copies it without any conversion. As long as your source material meets the criteria in your autoProcess.ini it will always remux and only convert if there's a reason to do so
My guess looking at your configuration is maybe due to you only having 2 pix-fmt options on your allowed list. The script output tells you why it failed the remux check and had to convert just look at the "debug" line in the json output for the stream

maybe due to you only having 2 pix-fmt options

I have tried to use others, however this always fails for h265vaapi. I think it only really supports nv12 ( when searching for answers, the top answer is to use -vf 'format=nv12,hwupload' and the script already does)
I understand how the script works. I believe that you're correct with the pix-fmt. I will try and retest with yuv420p and yuv420p10le maybe I missed something awhile ago.

You don't need to remove nv12 you just need to add the other formats later in the list. You can still keep nv12 first

From what I know about the script, if an option isn't listed it will choose the first option on the list.
I did some testing and found this to work, I think the issue I had was radarr and sonarr, I am yet to retest them.
codec = h265vaapi, hevc
max-bitrate = 0
bitrate-ratio = 0.5
max-width = 0
preopts =
crf = -0
preset = ultrafast
codec-parameters =
dynamic-parameters = True
profile = main10
max-level = 5.0
pix-fmt = p010le, yuv420p10le, yuv420p
prioritize-source-pix-fmt = False
filter =
force-filter = False
crf-profiles =
x265-params = no-info=1

Testing the script with HEVC main -> HEVC main10 (set as profile) causes error
Error while opening encoder - maybe incorrect parameters such as bit_rate, rate, width or height.
Error while filtering: Function not implemented

Everything works until there is a HEVC file that's profile is main.
Is there an option to bypass this or do I need to look at building a new server without using VAAPI?

This is just an error with your hardware encoder not liking one of your settings
I can't debug what FFMPEG command you should be using as this can be very hardware specific, but you can create an FFMPEG command that works for you and then post your full log output from SMA I can tell you what options to adjust to mirror your desired output if possible

Driver does not advertise encoder features, using guessed defaults.
Driver does not advertise encoder block size, using guessed defaults.
 No quality level set; using default (25).

From
"video": {
 "index": 0,
 "codec": "hevc",
 "bitrate": 5336565,
 "pix_fmt": "yuv420p10le",
 "profile": "main10",

To
"format": "mkv",
"video": {
"codec": "h265vaapi",
"map": 0,
"bitrate": 5154.686699999999,
"level": 5.0,
"profile": "main",
"pix_fmt": "nv12",: [

The main error is that iGPU doesn't have good support for HEVC to HEVC., this is why I wanted to skip anything in HEVC.
If the profile setting is different ( main or main10)  it will redo the HEVC video when it's  not necessary. HEVC videos are already smaller (25 to 50% less in size then AVC).

The profile options work the same way as the pix-fmt options, you can just add 'main' to your list of approved profiles to avoid transcoding
[Video]
profile = main10, main

That's good to know!
Would a video codec fallback work if the first option fails? codec = h265vaapi, hevc

No, it does not recover to alternate video codecs/run down the list when one fails
Gotta get your settings correct so its not failing, failures should not be commonplace

stream #0:0 -> #0:0 (h264 (native) -> hevc (hevc_vaapi))
No usable encoding profile found.
 Error while opening encoder - maybe incorrect parameters such as bit_rate, rate, width or height.

Removed pix-fmt  then  profile = main10, main, working again. I think your suggestion broke it.

I mean this is going from h264 to h265 so my suggestion wasn't relevant specifically to that. Posting the full logs would probably help

I no longer have the logs.
h264 to h265 no error
h265 to h265 error
It's when a video is h265 main10 with the profile settings being main. I think the igpu doesn't support someone of the pix formats. nv12 works for profile main and I think p010le works for main10.
Leave pix-fmt blank?

I think to best use the script, an upgrade is needed from QSV.

  Can't have it both ways though

Got it. No option to use srt subtitles as a fallback in-case devices/media players don't support.
Thank you.


Misbehaviour while showing dynamic content in the collapsible

Hi,
i'm searching for a way, to do collapsibles the right thing!
Heres my example:
Dynamic content in collapsible
Heres my problem:
I would like to display some dynamic content in the collapsible. In this example, there will be some times displayed. Don't worry about the same times. I only would like to show some dynamic content.
First of all, the collapse property is set to false in the initial state of redux, but the collapsible is closed (collapsible=true?!?!).
The second misbehaviour is, when the collapsible is shown (collapse=false) and a timeout is set. Then the collapsible closes itself.
What am i doing wrong???
I would like to display some times in the collapsible, without having this misbehaviour of closing the collapsible itself.
Any suggestions?

Hey @ralcon could you be a bit clearer? I'm not sure what you're asking exactly

Hi, i got the following problem:
My aim is to display several country times, updated by redux (every second via timeout), see my example. When i open the collapsible the first time, everything works fine and is displayed how it should be. but when i activate the timeout  to render the information, the collapsible callapses itself. the variable to identify, if the collapsible is collapsed, is false all the time (a redux property). why is the collapsible collapse itself?
A little bit clearer?

The target state should be:

Open callaspsibles
Press "Start" button
New times are rendered every second

=> Collapsibles should stay 'open' (collapse={false})!
within the given example:
1. git clone https://github.com/ralcon/dynamic-content-in-collapsible
2. cd dynamic-content-in-collapsible
3. npm install
4. react-native run-ios

@iRoachie : I am too facing the issue while showing dynamic controls in collapsible. Please see the video - https://hub.mangoapps.com/sf/MTAyNzA3XzEzODIxNzA

Having the same issue. When i update a collapsible that is open, it renders nothing even if the collapsed props is at false.

This solves this issue:
https://github.com/oblador/react-native-collapsible/pull/147

Just merged #147. Release will be out in a few mins.

Released in v0.11.0

Hello!
this bug is present in current version, pr #147 not merged

@dansanti I am still seeing issues when children inside Collapsible re-render. Did you manage to fix it?

PR #147 fix it

@dansanti It introduced flickering when my content changes height. Does that happen to you too?


Fix warnings

Codecov Report

Merging #28 (6bfed9f) into master (a8c13a7) will not change coverage.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master      #28   +/-   ##
=======================================
  Coverage   79.28%   79.28%
=======================================
  Files          27       27
  Lines         811      811
=======================================
  Hits          643      643
  Misses        168      168

Impacted Files
Coverage Δ

...ude/sliding_window/search/compute_simple_model.hpp
91.30% <ø> (ø)

...iding_window/search/write_output_file_parallel.hpp
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

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


Feat/signet component

Preflight Checklist

[X] I have read the Contributing Guidelines for this project.
[X] I agree to follow the Code of Conduct that this project adheres to.
[X] I have searched the pull request tracker for a Pull Request (PR) that matches the one I want to submit, without success.

Issue
This PR Closes #
Pull request checklist
Please check if your PR fulfills the following requirements:

[ ] Tests for the changes have been added (for bug fixes / features)
[X] Docs have been reviewed and added / updated if needed (for bug fixes / features)
[X] Build (npm run build) was run locally and any changes were pushed
[X] Lint (npm run lint) has passed locally and any fixes were made for failures

Changes
Changes in this pull request:

Adds Signet component

Browsers
I tested the build on the following browsers:

[X] Firefox Desktop: 92
[X] Chrome Desktop: 91
[X] Safari Desktop: iPhone 12, Safari 14
[X] Firefox Mobile: Pixel 5, Android 11
[X] Chrome Mobile: Pixel 5, Android 11
[X] Safari Mobile
[X] Opera: 79
[X] Brave: 93

Pull request type

Please check the type of change your PR introduces:

[ ] Bugfix
[X] Feature
[ ] Code style update (formatting, renaming)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] Documentation content changes
[ ] Other (please describe):

Does this introduce a breaking change?

[ ] Yes
[X] No

Other information

Mentions

@lyne-design-system/core

Codecov Report

Merging #761 (0d726ab) into master (31c6f9a) will decrease coverage by 0.38%.
The diff coverage is 0.00%.

:exclamation: Current head 0d726ab differs from pull request most recent head 08e6eb4. Consider uploading reports for the commit 08e6eb4 to get more accurate results

@@            Coverage Diff             @@
##           master     #761      +/-   ##
==========================================
- Coverage   29.69%   29.31%   -0.39%
==========================================
  Files          11       12       +1
  Lines         532      539       +7
  Branches      124      124
==========================================
  Hits          158      158
- Misses        355      362       +7
  Partials       19       19

Impacted Files
Coverage Δ

src/components/lyne-sbb-signet/lyne-sbb-signet.tsx
0.00% <0.00%> (ø)

src/components/lyne-title/lyne-title.tsx
81.25% <0.00%> (ø)

Continue to review full report at Codecov.

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


It's incorrect to resolve spread interpolation as intersect

🔎 Search Terms
object spread generic intersect
🕗 Version & Regression Information

~This is a crash:~
~This changed between versions ______ and _______~
This is the behavior in every version I tried, and I reviewed the FAQ for entries about: no mention in the FAQ
~I was unable to test this on prior versions because _______~

⏯ Playground Link
https://www.typescriptlang.org/play?ts=5.3.0-dev.20231005#code/GYVwdgxgLglg9mABAWwIYGsCmB1APAeUUwA8pMwATAZ0QG9FUAuBsAT0QF8AaRAIQD4AFHGb4eqAEzNeiAJR0AUIkQAnTFBAqk9AHR6445pJ4cA3ArMKFEBFSiIA7ogC8KDDkH0mdRACNmAIwMNDZgdpzcPhDMEsGIoeEcsubWtnAANpg66XAA5oIOOqg6vi7OrgHJQA
💻 Code
function makeW<O extends { a: any }, B>(o: O, a2: B ) {
  return { ...o, a: a2, };
};

const w = makeW({ a: { b: 1 as const }}, { c: 2 as const });

console.log(w.a.b === 1);

🙁 Actual behavior
Return type of makeW is resolved/inferred as O & { a: B } which is wrong.
the returned value { ...o, a: a2 } overrides any o.a.
🙂 Expected behavior
type of w is Omit<O, "a"> & { a: B }.
Additional information about the issue
Explicit typing works, but should be needed.
function makeW<O extends { a: any }, B>(o: O, a2: B ): Omit<O, "a"> & { a: B } {
  return { ...o, a: a2 };
};

const w = makeW({ a: { b: 1 as const }}, { c: 2 as const });

console.log(w.a.b === 1); // type error, no `b`.

Duplicate #10727


example 3 equations model is not running

@jaromir-benes , @nul0m , hey! I'm trying to run 3 equation model from the 'irispie examples' repo with irispie 0.54.8, but it returns an error:
Rule 'source' matched in its entirety, but it didn't consume all the text. The non-matching portion of the text begins
with '!transition-shocks`

Would appreciate any advice how to change the model code or anything else to make it work

In this file
https://github.com/IRIS-Solutions-Team/Iris-Pie-Examples/blob/master/linear_3eq_model/linear_3eq.model
!transition-shocks

    shk_y_gap, shk_diff_cpi, shk_rs

Should be
!unanticipated-shocks

    shk_y_gap, shk_diff_cpi, shk_rs

``


Dutch filters

Added Dutch versions of the phrases. Also added vegetarian versions, but left those commented out.
Not sure what the plan is with languages, but I decided to make a pull request anyways. Maybe filters other than the default should be disabled by default? That way merging in more languages won't affect @plorry's instance of the bot.

Just saw @squeek502's PR. I could convert these to regex as well, if you want to include Dutch filters at all.

@joriskcf This is great. I think we should have different bots doing different languages, rather than the one bot doing every language.
I'm thinking each language could be associated with its own bot's twitter credentials, and the task posts to the appropriate account, depending on which language got a match.
I'll work on this aspect of it - posting to different accounts based on language file.


"temporary value dropped while borrowed" with 1.41 beta

error[E0716]: temporary value dropped while borrowed
    --> src/wordlist_codegen.rs:1861:36
     |
1861 |        entries: ::phf::Slice::Static(&[
     |   ___________________________________-^
     |  |___________________________________|
     | ||
1862 | ||         ("expand", &EXPAND_STEM),
1863 | ||         ("queri", &QUERI_STEM),
1864 | ||         ("download", &DOWNLOAD_STEM),
...    ||
2087 | ||         ("creat", &CREAT_STEM),
2088 | ||     ]),
     | ||     ^
     | ||_____|
     | |______creates a temporary which is freed while still in use
     |        cast requires that borrow lasts for `'static`
2089 |    };
     |    - temporary value is freed at the end of this statement

error: aborting due to previous error

For more information about this error, try `rustc --explain E0716`.

$ rustup run beta rustc -V
rustc 1.41.0-beta.1 (eb3f7c2d3 2019-12-17)

[dependencies]
phf_codegen = "0.8"

@epage sorry for not getting to this sooner, but it looks like this should have been fixed by https://github.com/rust-lang/rust/pull/74945 which should be in the next Rust beta.

Presuming the above PR fixed as CI is now green, closing.


Fix: memory leak when publishing via realtime

#625

An attempt to mislead the problem.

@ricardopereira unfortunately not sure I can comment on this.  Will have to rely on @tcard here

I'm not really sure what is this. Is this WIP?

@tcard The client should try this with pod 'Ably', :git => 'https://github.com/ably/ably-ios.git', :branch => 'fix-625' and see if it helps.. I turned off the additional log data for Sentry.


Make it a meteor package

It would be nice to auto-run mgp when the app is started and whenever git-packages.json is changed.
Meteor does semantic analysis on package dependencies, so if there is anything missing the package might not run. However if we force the load order, we may be able to get around this.
If that does not work we will need to wait until meteor refactors their build tool.

https://github.com/xolvio/meteor-private-packages is this. Thanks @Sanjo!

Ooops nevermind. Seems like he ran into the same version constraint limitation I did https://github.com/xolvio/meteor-private-packages/commit/65569d6419202fd7c8442cd608ccc637db2f0fbc

This is not possible ATM. Will re-open if that changes.


mill: 0.10.0 → 0.10.1

Description of changes

Things done

Built on platform(s)

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

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

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

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

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

[x] Fits CONTRIBUTING.md.

/marvin opt-in
/status needs_reviewer

cc @scalavision

Thank you @kevincox , I have tested it locally with my builds, seems to work perfectly. Should be safe to merge this.

/status needs_merger


HHH-13364 Query.getSingleResult and getResultList() throw PessimisticLockException when pessimistic lock fails with timeout

https://hibernate.atlassian.net/browse/HHH-13364

it looks go to me


meson: 0.57.1 -> 0.59.2, gst-plugins-base 1.18.4 -> 1.18.5

https://mesonbuild.com/Release-notes-for-0-59-0.html
Updated meson for compatibility with newer packages such as wlroots 0.16
Updated gst-plugins-base for compatibility with meson >= 0.58
I just stumbled upon the meson version issue when building gamescope since it requires wlroots 0.16. https://github.com/Plagman/gamescope
nixpkgs-review is in progress although it might end up being more than practical to run on a single machine. i.e it seems this will trigger a rebuild of >40,000 packages just looking at nixpkgs-review now. :/
I have built meson itself & gst-plugins-base without issue.

Built on platform(s)

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

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

[x] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

I'm going to at least build systemd, sssd, and thunderbird as a sanity check. If anyone has any other packages that should be tested prior to merging let me know.

Near pkgs/development/tools/build-managers/meson/default.nix:3:3:
  |
3 | , writeTextDir
  |   ^

Due to this causing a mass rebuild, it needs to target staging.

I usually build fwupd and gnome-shell, or NixOS VM with GNOME if I can.

I've updated the PR to target staging. I'll rebuild the packages: fwupd, gnome-shell, systemd, sssd, and thunderbird as time allows.

fwupd: builds OK
sssd: fails but its also failing in hydra staging... I have a in-progress update to switch it to the 2.x branch.
systemd: builds OK
gnome-shell: fails with meson error but the error is : ERROR: Program 'dot' not found...
thunderbird: TODO

Checking now if gnome-shell fails in clean staging nixpkgs.

Is that in gnome-shell derivation? I do not see any reference to dot in the meson files.

The missing 'dot' error was in the libnice package which now needs graphviz. It looks like the Archlinux team had the same issue when meson was updated so it probably has something to do with documentation generation.
https://github.com/archlinux/svntogit-packages/commit/828b95566501b9918d85937a41da85709f506004#diff-37538beb61ff63edebbf735dfcf39e5d732f49183d6beb097169d971875ca422

Yeah, that is another instance of https://github.com/mesonbuild/meson/issues/8641. Adding graphviz is probably okay.

I am also getting some failed tests in some python packages:
In the most recent case: python3.9-sh
======================================================================
ERROR: test_general_signal (test.FunctionalTests)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/build/sh-1.14.2/test.py", line 1546, in test_general_signal
    p = python(py.name, _out=agg, _tee=True)
  File "/build/sh-1.14.2/sh.py", line 1566, in __call__
    return RunningCommand(cmd, call_args, stdin, stdout, stderr)
  File "/build/sh-1.14.2/sh.py", line 822, in __init__
    self.wait()
  File "/build/sh-1.14.2/sh.py", line 879, in wait
    self.handle_command_exit_code(exit_code)
  File "/build/sh-1.14.2/sh.py", line 905, in handle_command_exit_code
    raise exc
sh.ErrorReturnCode_1:

  RAN: /nix/store/g5rqxbchc59xrb3b5h41n8wh73l1sklk-python3-3.9.6/bin/python3.9 /build/tmp6xr_dtu1

  STDOUT:
0
1
2
3

  STDERR:
Traceback (most recent call last):
  File "/build/tmp6xr_dtu1", line 14, in <module>
    print(i)
  File "/build/tmp6xr_dtu1", line 8, in sig_handler
    print(10)
RuntimeError: reentrant call inside <_io.BufferedWriter name='<stdout>'>

----------------------------------------------------------------------
Ran 171 tests in 49.533s

FAILED (errors=1, skipped=3)
Test failed: <unittest.runner.TextTestResult run=171 errors=1 failures=0>
error: Test failed: <unittest.runner.TextTestResult run=171 errors=1 failures=0>

Let me know if there's a better way to get a concise output of failing tests instead of nixpkgs-review's output.

I do not use nix-review but would expect nix log -f . attributeThatFailed to work.

FYI, meson 0.60.1 has been released.

Looks like the meson patches need to be updated to work with meson 0.60.1. I should be able to have those done today. I'll create a new PR for meson 0.60.1 and close this one.
I am/was planning to wait until after the 20.11 release (or at least Nov 12 when staging is open again) before requesting another review since we are now already in ZHF. Since the only broken package to my knowledge without this update is the latest wlroots I doubt there's any reason to force this into 20.11.

I'll create a new PR for meson 0.60.1 and close this one.

Why can't we update this one?

I'll create a new PR for meson 0.60.1 and close this one.

Why can't we update this one?

I named the source branch meson-0.59.2 and I don't think there is a way to switch or rename the source branch. I guess it doesn't make a practical difference, I could just update this branch and update the title.

Closing as superseded by https://github.com/NixOS/nixpkgs/pull/144779.


Chore/update to v8

@NothingEverHappens Angular 8 now requires a node v10 "node": ">=10.9.0 <11.0.0", CI config needs to be updated.

Looks like CI is failing, something related to the node version?
Let me know if you need help figuring that out

@kirjs Yep, Angular now requires node v10 "node": ">=10.9.0 <11.0.0", but CI is configured for v8.

Amazing


Null still showing up for Lists

Class A {
.....
@EncodeNulls(false)
private List transList = null;
....
}
Even if transList is null, it still shows up in the document when inserted into Mongo.
I also did it in the registry and the same result.
My understanding would be that like other variables, it doesn't show up in the Mongo document if the value is null.

Am I not doing something correctly or my understanding is wrong? If so how do I not include transList if it's null like the other objects?
Thanks!

Hi @jklee14 ,
sorry for the late reply. I somehow didn't get noticed about your raised issue.
Could you provide the polymorphia version you have used?
About your field "transList": This PojoCodec is not capable to handle non-generic fields of List, Set or Map properly at the moment, when used within Pojos.
I am actually surprised that your given example actually wrote something into the database instead of throwing an exception. I would expect something like "org.bson.codecs.configuration.CodecConfigurationException".
Did you chain org.bson.codecs.IterableCodec before the PojoCodec when creating your CodecRegistry? Would be interesting to get more details about your setup.
To solve your problem you have two options:

use a generic List eg. List
use @de.bild.codec.annotations.Transient at you field if you don'T wnat that field stored to the database at all (somehow the name "transList" migh imply to me it is a transient list)

I now have added suport for non generic Lists, Sets and Maps within Pojos.
I still do not understand your setup.
Additionally your screen shot implies that your field "transList" was not null as an Array was wriiten to the database.
Have a look at
https://github.com/axelspringer/polymorphia/blob/3676effdc5a8c5c8fe58e8b1ddf04ecb4bfdb94d/src/test/java/de/bild/codec/NullHandlingTest.java#L84
This is a working example of what you need to achieve, I think?!
I will close this ticket. Let me know, if you need furhter support.

Hi From the looks of  your comment, I think you may have addressed my issue!
Thank you, let me try.
transList may or may not be null, and if it's null, we don't want it to appear in MongoDB which takes up space.
Polymorphia is working well for you, Thank you.

Everything is work as it should thank you so much for your support!


bug: type order in 'allOf' has an impact

Hello,
When validate with multiple schema using 'allOf', definition order has a behavior impact.
cf attached code to test
testSchemaValidator2.zip
Correct:
input value: "true"
schema: {
    "allOf": [
        {
            "type": "string"
        },
        {
            "type": "boolean"
        }
    ]
}
value is valid

reverse order lead to incorrect validation:
input value: "true"
schema: {
    "allOf": [
        {
            "type": "boolean"
        },
        {
            "type": "string"
        }
    ]
}
Boolean value found, but a string is required
Failed to match all schemas

I forget to mention version:
            "name": "justinrainbow/json-schema",
            "version": "v5.2.13",
            "source": {
                "type": "git",
                "url": "https://github.com/justinrainbow/json-schema.git",
                "reference": "fbbe7e5d79f618997bc3332a6f49246036c45793"
            },

@kroc702 thanks for the report.  Based on your schema provided it seems no input data will be valid ever as the a single value cannot be boolean and string at the same time. Using https://jsonschemalint.com/ you can see that both versions of you schema should raise errors whereas you mention that in both cases you expect them to be valid.

Well it's a choice, your choice ;)
As php is loosely typed, it doesn't hurt me to validate the strings "true" or "1" as boolean or number.
We just need a consistent behavior with all types and order shouldn't impact behavior of any/all/oneOf.


Unable to add OAUTH Azure Account

Dear deveolpers,
It's been months since I use this app succesfully for creating OTP token for Azure Accounts using OAuth Tokens feature. Despite the application argues that the algorithm allowed by Azure it is not as safe as it could be (SHA 1), it prompted a warning but still workded and offered good results.
Nowadays it has become impossible to add new tokens. The application prompts the following error:
"Token is invalid!
The token you are attempting to add is invalid. Please alert your token provider"
I've tried using other apps with the same TOPT parameters I was using for this setup. They worked as a charm.
May you confirm if there's any incompatibility for this kind of accounts or just a bug to be fixed?
Thank you in advance :)

Hi,
Could you post the OTP URI which Azure provides as a QR code image, such as:
otpauth://hotp/?secret=62tsfhk2k2gjybsk2idba3i4one2yzcj5dhwftsxk6xdwve4lram2gyq&algorithm=SHA256&digits=6&period=30&lock=false&counter=0

Please use a test account which does not contain actual sensitive OTP data in the secret value.

I have a similar error. With FreeOTP for Android version 2.0 (40) I get the error: "Token is invalid! The token you are attempting to add is invalid. Please alert your token provider."
I am trying to add the following account: otpauth://totp/test.Cloud%20Service:anna@test.com?secret=LMJ4BC2EF53ZO5VVYBSKE3VDLX6RR4DXRDQ3WOKVYRVYQSYGU6PZRZPXSJWOOSQ4LKCEIY6BEQRDTB7O3XEGKSHMIBDLLN6OOQBXDCI%3D&issuer=test.Cloud%20Service&digits=6&period=30&algorithm=SHA512
I get the error when scanning the QR code, when opening the URL and also when I enter the parameters manually.
In FreeOTP version 2.3.2 for iOS I can add the account without any problems.

Can you try removing the %3D from the end of the secret value, as mentioned in https://github.com/freeotp/freeotp-android/issues/301
The OTP URI can be edited with https://stefansundin.github.io/2fa-qr/

Thank you @justin-stephenson. Yes, it works without %3D.
The shared secret is a cryptographic random value as an array of bytes. This is encoded in Base64. Since Base64 can contain characters that already have a different meaning in an URL, the Base64 value is URL encoded.

Thank you @justin-stephenson. Yes, it works without %3D.
The shared secret is a cryptographic random value as an array of bytes. This is encoded in Base64. Since Base64 can contain characters that already have a different meaning in an URL, the Base64 value is URL encoded.

FreeOTP Android codebase contains its own base32 implementation. I created https://github.com/freeotp/freeotp-android/pull/305 to remove this code and use the apache commons codec base32 library, which also handles padding properly. I don't think we need to add any test for this as we should have no need to exercise the base32 library functionality itself.


Restored using .NET Framework 4.6.1 instead of .NET Core 2.2

Hi!
Thank You for this great library!
I would like to use the NuGet package (1.8.0) in a .NET Core 2.2 application but i get this warning:

D:\GitHub\---\---.csproj : warning NU1701: Package 'MediaDevices 1.8.0' was restored using '.NETFramework,Version=v4.6.1' instead of the project target framework '.NETCoreApp,Version=v2.2'. This package may not be fully compatible with your project.

I am not really familiar with multi-targeting but could this be fixed with that so NuGet could automatically restore as a .NET Core 2.1 library instead of a .NET Framework 4.6.1?
I added this repository as a submodule to temporary fix this warning.

This is all already prepared as you can see at MediaDevices / Nuget / MediaDevices.nuspec. I just want to fix a few little things before I put a new version on Nuget. Until then, you are welcome to download the finished files from MediaDevices / Signed /.

Thank You for the quick response!

Getting a similar warning in .NET 5.0. MediaDevices / Signed / seems to be missing and the package on Nuget was last updated in January 2019. Where can I find the .NET 5.0 compatible package?


add legacy comments (and reasoning) to a list of modules and functions

based on a discussion and recent work.

@Flix6x I can't find the list you gave me, but I believe it was quite short and I hope it's all included here.

Do you recall how you gave me that list? I searched chat and email but couldn't find it.

It was nothing special. Just the models Asset, AssetType, Market, MarketType, WeatherSensor, and WeatherSensorType, and the schemas AssetSchema and WeatherSensorSchema.

I guess you would add Power, Price and Weather?

Yes.

Some other things I ran into and I'm unsure how to handle:

CLI functions for adding/deleting structure, e.g. flexmeasures add weather-sensor. They could in principle live on.
populate_structure, also reachable by flexmeasures add structure. This really has an opinionated take at what structure should be on a FM server. I get that we want some default GenericAssetType instances to be there. So that is what this could become - default types.

I guess at least the latter point could become a new ticket. We might have a list item in a project, but this is already a bit more detailed.

I agree with those suggestions.


Fix the README to have correct links.

Make sure we are using youtube/spitfire and not spt/spitfire.

ptal @nicksay

LGTM


BUG: scipy.interpolate._bsplines._make_periodic_spline crashes the Python Interpreter after correctly raising ValueError

Describe your issue.
I find this issue when I'm using LLM to generate a call to scipy.interpolate._bsplines._make_periodic_spline.
For this code, the Python Interpreter should raise a ValueError since the input is not valid, and it does so.
I know it's often the case that LLM cannot generate valid inputs for functions, so I wrapped the call to scipy.interpolate._bsplines._make_periodic_spline with a "try...except" to avoid the program exit.
However, the Python Interpreter still crashes with a Segmentation fault.
I tested this code on both Python 3,10, 3.11, 3.12 and 3.13, but got the same issue.
Reproducing Code Example
import numpy as np
import scipy

x = np.linspace(0, 1, 100)
y = np.random.rand(100)
k = 5
t = np.linspace(-1, 4, 1 + 2 * k)[1:-1 + 2 * k]
axis = 0

# scipy.interpolate._bsplines._make_periodic_spline(x, y, t=t, k=k, axis=axis)

try:
    scipy.interpolate._bsplines._make_periodic_spline(x, y, t=t, k=k, axis=axis)
except ValueError as e:
    print(e)

Error message
# this is what I got without try...except
(py313) root@a475b702ecc0:~# python bug.py
Traceback (most recent call last):
  File "/root/bug.py", line 10, in <module>
    scipy.interpolate._bsplines._make_periodic_spline(x, y, t=t, k=k, axis=axis)
    ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/root/miniconda3/envs/py313/lib/python3.13/site-packages/scipy/interpolate/_bsplines.py", line 1214, in _make_periodic_spline
    ll += np.diag(ab[i, -kul - (k % 2): n - 1 + 2 * kul - i], k=-i)
ValueError: operands could not be broadcast together with shapes (2,2) (3,3) (2,2)
Segmentation fault (core dumped)

# this is what I got with try...except
(py313) root@a475b702ecc0:~# python bug.py
operands could not be broadcast together with shapes (2,2) (3,3) (2,2)
Segmentation fault (core dumped)

SciPy/NumPy/Python version and system information
1.14.1 2.2.0 sys.version_info(major=3, minor=13, micro=0, releaselevel='final', serial=0)
Build Dependencies:
  blas:
    detection method: pkgconfig
    found: true
    include directory: /opt/_internal/cpython-3.13.0rc1/lib/python3.13/site-packages/scipy_openblas32/include
    lib directory: /opt/_internal/cpython-3.13.0rc1/lib/python3.13/site-packages/scipy_openblas32/lib
    name: scipy-openblas
    openblas configuration: OpenBLAS 0.3.27.dev DYNAMIC_ARCH NO_AFFINITY Zen MAX_THREADS=64
    pc file directory: /project
    version: 0.3.27.dev
  lapack:
    detection method: pkgconfig
    found: true
    include directory: /opt/_internal/cpython-3.13.0rc1/lib/python3.13/site-packages/scipy_openblas32/include
    lib directory: /opt/_internal/cpython-3.13.0rc1/lib/python3.13/site-packages/scipy_openblas32/lib
    name: scipy-openblas
    openblas configuration: OpenBLAS 0.3.27.dev DYNAMIC_ARCH NO_AFFINITY Zen MAX_THREADS=64
    pc file directory: /project
    version: 0.3.27.dev
  pybind11:
    detection method: config-tool
    include directory: unknown
    name: pybind11
    version: 2.13.4
Compilers:
  c:
    commands: cc
    linker: ld.bfd
    name: gcc
    version: 10.2.1
  c++:
    commands: c++
    linker: ld.bfd
    name: gcc
    version: 10.2.1
  cython:
    commands: cython
    linker: cython
    name: cython
    version: 3.0.11
  fortran:
    commands: gfortran
    linker: ld.bfd
    name: gcc
    version: 10.2.1
  pythran:
    include directory: ../../opt/python/cp313-cp313/lib/python3.13/site-packages/pythran
    version: 0.16.1
Machine Information:
  build:
    cpu: x86_64
    endian: little
    family: x86_64
    system: linux
  cross-compiled: false
  host:
    cpu: x86_64
    endian: little
    family: x86_64
    system: linux
Python Information:
  path: /opt/python/cp313-cp31

You are trying to reach into non-public functionality, as indicated by the leading underscores, and that's not really supported.
Using the public interface:
In [1]: import numpy as np
   ...:
   ...: x = np.linspace(0, 1, 100)
   ...: y = np.random.rand(100)
   ...: k = 5
   ...: t = np.linspace(-1, 4, 1 + 2 * k)[1:-1 + 2 * k]
   ...: axis = 0

In [2]: from scipy.interpolate import make_interp_spline

In [3]: make_interp_spline(x, y, k=5, bc_type='periodic')
---------------------------------------------------------------------------
ValueError                                Traceback (most recent call last)
Cell In[3], line 1
----> 1 make_interp_spline(x, y, k=5, bc_type='periodic')

File ~/repos/scipy/scipy/build-install/lib/python3.12/site-packages/scipy/interpolate/_bsplines.py:1516, in make_interp_spline(x, y, k, t, bc_type, axis, check_finite)
   1514 # sanity check the input
   1515 if bc_type == 'periodic' and not np.allclose(y[0], y[-1], atol=1e-15):
-> 1516     raise ValueError("First and last points does not match while "
   1517                      "periodic case expected")
   1518 if x.size != y.shape[0]:
   1519     raise ValueError(f'Shapes of x {x.shape} and y {y.shape} are incompatible')

ValueError: First and last points does not match while periodic case expected

In [4]: y[-1] = y[0]

In [5]: make_interp_spline(x, y, k=5, bc_type='periodic')
Out[5]: <scipy.interpolate._bsplines.BSpline at 0x7f837c723bc0>

Also note that indeed, bc_type="periodic" does not really support passing in user-constructed knot vectors:
In [7]: make_interp_spline(x, y, k=5, t=t, bc_type='periodic')
---------------------------------------------------------------------------
NotImplementedError                       Traceback (most recent call last)
Cell In[7], line 1
----> 1 make_interp_spline(x, y, k=5, t=t, bc_type='periodic')

File ~/repos/scipy/scipy/build-install/lib/python3.12/site-packages/scipy/interpolate/_bsplines.py:1547, in make_interp_spline(x, y, k, t, bc_type, axis, check_finite)
   1544 k = operator.index(k)
   1546 if bc_type == 'periodic' and t is not None:
-> 1547     raise NotImplementedError("For periodic case t is constructed "
   1548                               "automatically and can not be passed "
   1549                               "manually")
   1551 # come up with a sensible knot vector, if needed
   1552 if t is None:

NotImplementedError: For periodic case t is constructed automatically and can not be passed manually

You are absolutely right. If the "guard" of the public function is able to ensure the security, and the internal usage is always right, then I believe there is no need to fix this issue. You can close this issue if you like.


Add ability to filter json_info results by type.

I'd like to be able to filter json_info results by type.
Examples:

Have it return results on just the string entries.
Have it return results on only number and boolean entries.
Have it return results on only scalar entries (string, number, boolean, or null).
Have it return results on only iterable entries (array, or object).

Here's what I'm thinking:
Add a --filter <types> option where  is a comma separated list of types to keep, e.g. 'strings,nulls'.
Valid types: strings, numbers, booleans, nulls, arrays, objects.
We also want to have two valid group types:

scalars (strings, numbers, booleans, nulls)
iterables (arrays, objects).

Then add some flags that are aliases for that option for the various types:

--strings = --filter 'strings'
--numbers = --filter 'numbers'
--booleans = --filter 'booleans'
--nulls = --filter 'nulls'
--arrays = --filter 'arrays'
--objects = --filter 'objects'
--scalars = --filter 'strings,numbers,booleans,nulls'
--iterables = --filter 'arrays,objects'

The flag versions would be additive, e.g. --strings --numbers would be like --filter 'strings,numbers'.
A --filter <types> argument would overwrite any previously provided --filter <types> or alias flags.
All of this should allow for both singular and plural versions of the words.

This functionality would also be handy in the json_search function.


integration with systemd - systemctl stop produces exit code 143

Issues and Contributing
Docker commands that you execute
# start
/usr/bin/docker run -d --user 1000 --privileged  --volume /home/jenkins:/var/jenkins_home --publish 443:8443 jenkins --httpPort=-1 --httpsPort=8443 --httpsKeyStore=/var/jenkins_home/.keystore/jenkins_keystore.jks  --httpsKeyStorePassword="$KEYSTORE_PASSWORD"

# test start stop
/usr/bin/docker start -a jenkins_server
/usr/bin/docker stop -t 5 jenkins_server

# Test with systemd
systemctl start jenkins-container && systemctl status jenkins-container
...
Loaded: loaded (/etc/systemd/system/jenkins-container.service; enabled; vendor preset: disabled)
Active: active (running) since Tue 2017-05-02 13:23:57 EDT; 5min ago

systemctl stop jenkins-container && systemctl status jenkins-container
...
Loaded: loaded (/etc/systemd/system/jenkins-container.service; enabled; vendor preset: disabled)
Active: failed (Result: exit-code) since Tue 2017-05-02 13:31:31 EDT; 2s ago

Actual result
Failed status from bad exit code 143
Expected outcome
Exit code 0 from /usr/bin/docker stop -t 5 jenkins_server
Have you tried a non-dockerized Jenkins and get the expected outcome?
Not applicable in our environment at the moment.
Output of docker version
Docker version 1.12.6, build 3a094bd/1.12.6
Other relevant information
I can avoid this with:
SuccessExitStatus=143

However, that is not optimal.
Docs referenced:

Similar issue on another container I found (tomcat)
Docker systemd integration
RedHat: Docker containers with systemd

as it is mentioned in https://github.com/docker-library/tomcat/issues/57#issuecomment-266556676

docker stop does exactly as the documentation states:

... a SIGTERM is sent to the main proces. ...

maybe tini config can be modified to handle that

This seems to be that the folder mounted was not created prior to initialising docker or that the user does not have permissions to the volume folder causing docker to crash.
try docker logs {container}

no answer, closing


Fix feynman technique responses

AI acting like he needs to be the one explaining. fix prompt engineering

Notice

No need for fine-tuning for now, the issue is how I send requests messages.

:tada: This issue has been resolved in version 1.0.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Update compatibility with Hexo 7

Hi @sergeyzwezdin ,
I've updated hexo-feed to ensure compatibility with hexo version 7.0.0. After testing, there don't seem to be any compatibility issues from my end.
Please review it on your side to confirm the compatibility. Let me know if you have any questions or need further information.
Thanks!

@liu-yihong  Merged, thank you for your efforts!


Adjusted for better 'readability'.

Adjusted for better 'readability'. #ATCP

@lukemurraynz : Thanks for your contribution! The author(s) have been notified to review your proposed change.

Hey @lukemurraynz, thanks for the suggestions.
The Azure Architecture Center has very specific editorial rules, and every article goes through multiple rounds of review and editorial adjustments before it's published. In general, we can't accept these kinds of PRs because it would result in articles no longer following those rules.
However, we're very happy to work with you to review PRs that relate to the technical content of articles, and to fix any invalid links or outdated information or mistakes.
Thanks again.
#please-close


Remove text specifying that app service plan can only be shared by ap…

…ps w/in same resource group
It is possible to have apps share an App Service Plan that is in a different resource group.
Scenario:
Resource Group: resourcegroup-01
App Service Plan: plan-in-resourcegroup-01
Web App: webapp-in-resourcegroup-01
New API App...
Resource Group: resourcegroup-02
API App: apiapp-in-resourcegroup-02
App Service Plan: plan-in-resourcegroup-01

@andresnava,
Thanks for having already signed the Contribution License Agreement. Your agreement was validated by Microsoft. We will now review your pull request.
Thanks,
Microsoft Pull Request Bot

@btardif - Please review the proposed changes.  Thanks

#sign-off


Chart md

I added the new chart.yaml fields, removed fields not supported in the current backend.  Also changed some of the language to reflect this is no longer a proposal but an existing thing.
edit: The intention here is to document existing behavior for users cloning the repo right now.

/cc @technosophos

I am good with the changes, but will leave the LGTM to @runseb, who had a few questions.

@runseb please take a look at last commit

Fixes #554

So I think I addressed all of these comments except the
Chart Developer: Responsible for building new charts.
Application Developer: Responsible for building applications that use charts.

suggestion which I'm not confident is a good idea.  These are roles that we would be defining here, rather than roles people already understand.  I agree that section could use more work though.  It is now in a separate file.

What's wrong with Chart Developer and Application Developer?
People definitely understand Application Developer.
Chart Developer was a proposal to replace Definition Developer, which is totally vague.

Ok I'll change

Service Dev: Responsible for developing new charts

to Chart developer

I'm not sure about application developer because we haven't defined what an application is, and the term is used in a lot of other contexts with very specific meanings.

OK. Then how about Chart User?

Looks like I did in fact change it to "App dev" and just forgot about it, and then argued against doing what I'd already done.  Kinda shows you how much context switching I'm doing right now :)
Ready to LGTM?

LOL. Taking another look...

A few final nits. Otherwise, LGTM.


benchmark switcher

Tiny quality of life improvement. Adds a posibility to run a specific benchmark, a group of benchmarks or to run everything with *

Thanks 👍


Adding SCTP support (Linux/FreeBSD)

I'm looking at adding SCTP support to the socket2 crate, and it would be useful to have definitions of constants and structs from netinet/sctp.h to do so. It seems like this crate is the most appropriate place to have those defined, given that definitions for similar protocols (TCP/UDP, or even the Linux-specific Netlink) are found here.
I would be happy to add the appropriate bindings and submit a PR for it here; I just wanted to check to see if such an addition would be desired.

I would suggest to submit a PR, makes sense.


Now, it completed 4171 asserts in 55 tests with 0 failures and 0 errors.

Thanks


Add tests for cache hook, using service calls.

Fixed via https://github.com/feathers-plus/feathers-hooks-common/pull/352
Published to npm as 4.5.3.


Domain randomization

Thank you for interesting challenge!
As you can see here, from the Simulation to the Real World, Domain Randomization is helpful. https://arxiv.org/abs/1703.06907
So, I would like to ask the parameters of quadrotor dynamics in Dodge drone challenge(https://github.com/uzh-rpg/flightmare/blob/092ff357139b2e98fc92bcdee50f38f85b55246d/flightpy/configs/vision/config.yaml#L45-L60). Do these parameters will change in the Selections and Finals? Or, do these parameters do not change?

the dynamic parameters will stay the same during the evaluation.
The only difference is that during training (if you are using Reinforcement learning), you are using a perfect simulation (no delay, no noise).
During the evaluation, we will be using a ROS script. The ROS message publication and subscriber will have some delay, which is negligible normally. But, the performance might vary during evaluation.
But, to answer your question. the dynamic parameters will not be changed. Not sure if Domain randomization will be a good idea in this case.

OK, I will check these delay, Thank you for your valuable information!


Add fetchival

fetchival is a small wrapper around window.fetch() to simplify writing JSON requests.

(Selected for inclusion in commit bundle)

Awesome, thank you! :)


Update CZML validation document.

Adds new properties from #5995.

Thanks for the pull request @shunter!

:heavy_check_mark: Signed CLA found.
:grey_question: CHANGES.md was not updated.

If this change updates the public API in any way, please add a bullet point to CHANGES.md.

Reviewers, don't forget to make sure that:

[ ] Cesium Viewer works.
[ ] Works in 2D/CV.
[ ] Works (or fails gracefully) in IE11.

Thanks @shunter


Not working on linux

Example from readme:
var fs = require('fs'),
    convertFactory = require('electron-html-to');

var conversion = convertFactory({
  converterPath: convertFactory.converters.PDF,
  strategy: 'electron-server'
});

conversion({ html: '
Hello World
' }, function(err, result) {
  if (err) {
    return console.error(err);
  }

  console.log(result.numberOfPages);
  result.stream.pipe(fs.createWriteStream('test.pdf'));
  conversion.kill();
});

doesn't work on linux:
{ [Error: electron manager could not start all workers..]
  workerErrors:
   [ [Error: Unable to reach electron worker - mode: server],
     [Error: Unable to reach electron worker - mode: server] ] }

@dbackowski yes, this is a limitation of electron, since it needs a GUI to be able to do its magic, but there are ways to be able to run electron inside linux (basically using xvfb) but i need to test for myself and see what problems occurs, see this SO question for more details

@bjrmatos yes i known this needs a GUI, I wrote a simple markdown editor in electron that use electron-html-to to produce pdf export from html.
On Mac OS X this works fine, on Linux this isn't working and got message "electron manager could not start all workers..".
This is my app https://github.com/dbackowski/md-editor

Great! Let me find some time to find the problem in linux and document the necessary steps to make it work!

Thx :smiley:

Any update on this? @bjrmatos

FYI I've tried running it on linux and it truly doesn't.
The trouble is not the pdf rendering which can be easily fixed with xvfb, but the IPC happening in the particular implementation of this package. The IPC communication between node and electron process doesn't seem to come through on linux. I've submitted question to the electron community, I'll ping back when I've some more information.

The link to the question describing the problem https://discuss.atom.io/t/node-js-to-electron-ipc-on-linux/31820

I've tried another channel to get an attention to this topic - https://github.com/electron/electron/issues/6868

hi @pofider , thnks for your time in this issue.
i was able to get electron ipc working in ubuntu 16.04 (using your example)
steps to make it work:

make sure to have installed the following libraries:

sudo apt-get install build-essential clang libdbus-1-dev libgtk2.0-dev libnotify-dev libgnome-keyring-dev libgconf2-dev libasound2-dev libcap-dev libcups2-dev libxtst-dev libxss1 libnss3-dev gcc-multilib g++-multilib curl gperf bison (listed in electron page) (i'm sure some of them are not necessary for final consumers but you know.. just in case 😄 )
sudo apt-get install libgtkextra-dev libnss3 libasound2 (listed here)
sudo apt-get install -y libgtk2.0-0 libnotify-bin libgconf-2-4 libnss3 xvfb to run xvfb (as detailed in this blog post)

run

export DISPLAY=':99.0'
Xvfb :99 -screen 0 1024x768x24 > /dev/null 2>&1 &
then just test it with node index.js
// index.js
var electron = require('electron-prebuilt')
var spawn = require('child_process').spawn

var child = spawn(electron, ['electron-script.js'], {
  stdio: ['ipc']
})

// this line is very helpful, it will tell you if there are missing dependencies in electron binary
child.stderr.pipe(process.stderr);

child.on('message', function (m) {
  console.log('Yes it works!')
})

// electron-script.js

process.send('I am ok')
process.exit(0);

can you verify if it works for you on ubuntu 14? maybe there is a missing dependency in your PC
make sure that /path/to/electron-prebuilt/dist/electron --help gives you some output first, if not there are problems in electron startup

now i'm testing electron-html-to and electron-workers in ubuntu, lets see how it goes..

@bjrmatos You rock!  The IPC works after installing the libraries you mentioned.
The electron-html-to doesn't work for me still, but we are definitely one step closer.

@pofider aha! i've found the problem.. in linux if i pass the env option for child_process.spawn, the electron process crash immediately. can you verify that? just comment the env option in this line, (it will crash even if you pass an empty env object, env: {}) seems like the entire env for the electron process is overridden and that's why it ended up crashed, definitely an issue in electron linux.
i will post the details in the electron issue you have opened

+1 on this, I would like to test electron-html-to in a Debian pre-production server... now I'm getting some sort of muted error:
    error: {
      "workerErrors": [
        {},
        {}
      ]
    }

I am getting the same issue on Windows 10 when I try to run the example code: "electron manager could not start all workers"...

Ok, problem fixed.  I ran:
npm install electron-prebuilt
And all was well.

The workaround mentioned [here])(https://github.com/electron/electron/issues/6868#issuecomment-252423414) seems to help. I'll have pull request tomorrow.

fixed in v2.2.0


Storybook + Typescript (Issue: Requires Babel "^7.0.0-0", but was loaded with "6.26.3")

I am following the Typescript documentation.
I am using version ^4.0.0-alpha.21 of storybook.
    "@storybook/addon-actions": "^4.0.0-alpha.21",
    "@storybook/addon-info": "^4.0.0-alpha.21",
    "@storybook/addon-links": "^4.0.0-alpha.21",
    "@storybook/addons": "^4.0.0-alpha.21",
    "@storybook/react": "^4.0.0-alpha.21",

The issue here is that the default webpack config is depending on an old version of babel-core, when I install<EMAIL_ADDRESS>it is still using<EMAIL_ADDRESS>ERROR in ./.storybook/config.js
Module build failed (from ./node_modules/babel-loader/lib/index.js):
Error: Requires Babel "^7.0.0-0", but was loaded with "6.26.3". If you are sure you have a compatible version of @babel/core, it is likely that something in your build process is loading the wrong version. Inspect the stack trace of this error to look for the first entry that doesn't mention "@babel/core" or "babel-core" to see what is calling Babel. (While processing preset<EMAIL_ADDRESS>    at throwVersionError (/Users/thomas.reggi/Desktop/git/github.com/alfredclub/admin-portal/node_modules/@babel/helper-plugin-utils/lib/index.js:65:11)
    at Object.assertVersion (/Users/thomas.reggi/Desktop/git/github.com/alfredclub/admin-portal/node_modules/@babel/helper-plugin-utils/lib/index.js:13:11)
    at _default (/Users/thomas.reggi/Desktop/git/github.com/alfredclub/admin-portal/node_modules/@babel/preset-react/lib/index.js:61:7)
    at /Users/thomas.reggi/Desktop/git/github.com/alfredclub/admin-portal/node_modules/@babel/helper-plugin-utils/lib/index.js:19:12
    at /Users/thomas.reggi/Desktop/git/github.com/alfredclub/admin-portal/node_modules/babel-core/lib/transformation/file/options/option-manager.js:317:46
    at Array.map (<anonymous>)
    at OptionManager.resolvePresets (/Users/thomas.reggi/Desktop/git/github.com/alfredclub/admin-portal/node_modules/babel-core/lib/transformation/file/options/option-manager.js:275:20)
    at OptionManager.mergePresets (/Users/thomas.reggi/Desktop/git/github.com/alfredclub/admin-portal/node_modules/babel-core/lib/transformation/file/options/option-manager.js:264:10)
    at OptionManager.mergeOptions (/Users/thomas.reggi/Desktop/git/github.com/alfredclub/admin-portal/node_modules/babel-core/lib/transformation/file/options/option-manager.js:249:14)
    at OptionManager.init (/Users/thomas.reggi/Desktop/git/github.com/alfredclub/admin-portal/node_modules/babel-core/lib/transformation/file/options/option-manager.js:368:12)
    at File.initOptions (/Users/thomas.reggi/Desktop/git/github.com/alfredclub/admin-portal/node_modules/babel-core/lib/transformation/file/index.js:212:65)
    at new File (/Users/thomas.reggi/Desktop/git/github.com/alfredclub/admin-portal/node_modules/babel-core/lib/transformation/file/index.js:135:24)
    at Pipeline.transform (/Users/thomas.reggi/Desktop/git/github.com/alfredclub/admin-portal/node_modules/babel-core/lib/transformation/pipeline.js:46:16)
    at transpile (/Users/thomas.reggi/Desktop/git/github.com/alfredclub/admin-portal/node_modules/babel-loader/lib/index.js:50:20)
    at /Users/thomas.reggi/Desktop/git/github.com/alfredclub/admin-portal/node_modules/babel-loader/lib/fs-cache.js:118:18
    at ReadFileContext.callback (/Users/thomas.reggi/Desktop/git/github.com/alfredclub/admin-portal/node_modules/babel-loader/lib/fs-cache.js:31:21)
    at FSReqWrap.readFileAfterOpen [as oncomplete] (fs.js:420:13)
 @ multi<EMAIL_ADDRESS><EMAIL_ADDRESS>./.storybook/config.js ./node_modules/webpack-hot-middleware/client.js?reload=true iframe[2]

ERROR in ./.storybook/addons.js
Module build failed (from ./node_modules/babel-loader/lib/index.js):
Error: Requires Babel "^7.0.0-0", but was loaded with "6.26.3". If you are sure you have a compatible version of @babel/core, it is likely that something in your build process is loading the wrong version. Inspect the stack trace of this error to look for the first entry that doesn't mention "@babel/core" or "babel-core" to see what is calling Babel. (While processing preset<EMAIL_ADDRESS>    at throwVersionError (/Users/thomas.reggi/Desktop/git/github.com/alfredclub/admin-portal/node_modules/@babel/helper-plugin-utils/lib/index.js:65:11)
    at Object.assertVersion (/Users/thomas.reggi/Desktop/git/github.com/alfredclub/admin-portal/node_modules/@babel/helper-plugin-utils/lib/index.js:13:11)
    at _default (/Users/thomas.reggi/Desktop/git/github.com/alfredclub/admin-portal/node_modules/@babel/preset-react/lib/index.js:61:7)
    at /Users/thomas.reggi/Desktop/git/github.com/alfredclub/admin-portal/node_modules/@babel/helper-plugin-utils/lib/index.js:19:12
    at /Users/thomas.reggi/Desktop/git/github.com/alfredclub/admin-portal/node_modules/babel-core/lib/transformation/file/options/option-manager.js:317:46
    at Array.map (<anonymous>)
    at OptionManager.resolvePresets (/Users/thomas.reggi/Desktop/git/github.com/alfredclub/admin-portal/node_modules/babel-core/lib/transformation/file/options/option-manager.js:275:20)
    at OptionManager.mergePresets (/Users/thomas.reggi/Desktop/git/github.com/alfredclub/admin-portal/node_modules/babel-core/lib/transformation/file/options/option-manager.js:264:10)
    at OptionManager.mergeOptions (/Users/thomas.reggi/Desktop/git/github.com/alfredclub/admin-portal/node_modules/babel-core/lib/transformation/file/options/option-manager.js:249:14)
    at OptionManager.init (/Users/thomas.reggi/Desktop/git/github.com/alfredclub/admin-portal/node_modules/babel-core/lib/transformation/file/options/option-manager.js:368:12)
    at File.initOptions (/Users/thomas.reggi/Desktop/git/github.com/alfredclub/admin-portal/node_modules/babel-core/lib/transformation/file/index.js:212:65)
    at new File (/Users/thomas.reggi/Desktop/git/github.com/alfredclub/admin-portal/node_modules/babel-core/lib/transformation/file/index.js:135:24)
    at Pipeline.transform (/Users/thomas.reggi/Desktop/git/github.com/alfredclub/admin-portal/node_modules/babel-core/lib/transformation/pipeline.js:46:16)
    at transpile (/Users/thomas.reggi/Desktop/git/github.com/alfredclub/admin-portal/node_modules/babel-loader/lib/index.js:50:20)
    at /Users/thomas.reggi/Desktop/git/github.com/alfredclub/admin-portal/node_modules/babel-loader/lib/fs-cache.js:118:18
    at ReadFileContext.callback (/Users/thomas.reggi/Desktop/git/github.com/alfredclub/admin-portal/node_modules/babel-loader/lib/fs-cache.js:31:21)
    at FSReqWrap.readFileAfterOpen [as oncomplete] (fs.js:420:13)
 @ multi ./.storybook/addons.js<EMAIL_ADDRESS><EMAIL_ADDRESS>manager[0]

You should also install babel-loader 8.

Might be related to https://github.com/storybooks/storybook/issues/4192

I am also seeing the similar errors when using version ^4.0.0-alpha.22 of the various storybook packages..
info @storybook/react v4.0.0-alpha.22
info
info => Loading presets
info => Loading custom addons config.
info => Loading custom webpack config (full-control mode).
 10% building modules 2/2 modules 0 activeℹ ｢wdm｣: wait until bundle finished:
webpack built 8a5184064a2b074299c8 in 8444ms
✖ ｢wdm｣: Hash: 8a5184064a2b074299c8
Version: webpack 4.19.1
Time: 8444ms
Built at: 09/21/2018 12:27:33 PM
                       Asset       Size   Chunks                    Chunk Names
     static/iframe.bundle.js    557 KiB   iframe  [emitted]  [big]  iframe
    static/manager.bundle.js   2.86 MiB  manager  [emitted]  [big]  manager
                 iframe.html   61.6 KiB           [emitted]
                  index.html  881 bytes           [emitted]
 static/iframe.bundle.js.map    565 KiB   iframe  [emitted]         iframe
static/manager.bundle.js.map   3.03 MiB  manager  [emitted]         manager
Entrypoint iframe [big] = static/iframe.bundle.js static/iframe.bundle.js.map
Entrypoint manager [big] = static/manager.bundle.js static/manager.bundle.js.map
[./.storybook/addons.js] 1.33 KiB {manager} [built] [failed] [1 error]
[./.storybook/config.js] 1.33 KiB {iframe} [built] [failed] [1 error]
<EMAIL_ADDRESS>366 bytes {manager} [built]
<EMAIL_ADDRESS>3.15 KiB {manager} [built]
<EMAIL_ADDRESS>93 bytes {iframe} [built]
<EMAIL_ADDRESS>113 bytes {iframe} {manager} [built]
<EMAIL_ADDRESS>2.19 KiB {manager} [built]
[./node_modules/global/window.js] 232 bytes {iframe} {manager} [built]
[./node_modules/webpack-hot-middleware/client-overlay.js] (webpack)-hot-middleware/client-overlay.js 2.16 KiB {iframe} [built]
[./node_modules/webpack-hot-middleware/client.js?reload=true] (webpack)-hot-middleware/client.js?reload=true 7.59 KiB {iframe} [built]
[./node_modules/webpack-hot-middleware/node_modules/strip-ansi/index.js] (webpack)-hot-middleware/node_modules/strip-ansi/index.js 161 bytes {iframe} [built]
[./node_modules/webpack-hot-middleware/process-update.js] (webpack)-hot-middleware/process-update.js 4.26 KiB {iframe} [built]
[./node_modules/webpack/buildin/module.js] (webpack)/buildin/module.js 497 bytes {iframe} {manager} [built]
[0] multi<EMAIL_ADDRESS><EMAIL_ADDRESS>./.storybook/config.js (webpack)-hot-middleware/client.js?reload=true 64 bytes {iframe} [built]
[1] multi ./.storybook/addons.js<EMAIL_ADDRESS><EMAIL_ADDRESS>52 bytes {manager} [built]
    + 728 hidden modules

ERROR in ./.storybook/addons.js
Module build failed (from ./node_modules/babel-loader/lib/index.js):
Error: Plugin/Preset files are not allowed to export objects, only functions. In /Users/zander/app/node_modules/babel-preset-react/lib/index.js
    at createDescriptor (/Users/zander/app/node_modules/@babel/core/lib/config/config-descriptors.js:178:11)
    at items.map (/Users/zander/app/node_modules/@babel/core/lib/config/config-descriptors.js:109:50)
    at Array.map (<anonymous>)
    at createDescriptors (/Users/zander/app/node_modules/@babel/core/lib/config/config-descriptors.js:109:29)
    at createPresetDescriptors (/Users/zander/app/node_modules/@babel/core/lib/config/config-descriptors.js:101:10)
    at presets (/Users/zander/app/node_modules/@babel/core/lib/config/config-descriptors.js:47:19)
    at mergeChainOpts (/Users/zander/app/node_modules/@babel/core/lib/config/config-chain.js:315:26)
    at /Users/zander/app/node_modules/@babel/core/lib/config/config-chain.js:278:7
    at buildRootChain (/Users/zander/app/node_modules/@babel/core/lib/config/config-chain.js:118:22)
    at loadPrivatePartialConfig (/Users/zander/app/node_modules/@babel/core/lib/config/partial.js:85:55)
 @ multi ./.storybook/addons.js<EMAIL_ADDRESS><EMAIL_ADDRESS>manager[0]

ERROR in ./.storybook/config.js
Module build failed (from ./node_modules/babel-loader/lib/index.js):
Error: Plugin/Preset files are not allowed to export objects, only functions. In /Users/zander/app/node_modules/babel-preset-react/lib/index.js
    at createDescriptor (/Users/zander/app/node_modules/@babel/core/lib/config/config-descriptors.js:178:11)
    at items.map (/Users/zander/app/node_modules/@babel/core/lib/config/config-descriptors.js:109:50)
    at Array.map (<anonymous>)
    at createDescriptors (/Users/zander/app/node_modules/@babel/core/lib/config/config-descriptors.js:109:29)
    at createPresetDescriptors (/Users/zander/app/node_modules/@babel/core/lib/config/config-descriptors.js:101:10)
    at presets (/Users/zander/app/node_modules/@babel/core/lib/config/config-descriptors.js:47:19)
    at mergeChainOpts (/Users/zander/app/node_modules/@babel/core/lib/config/config-chain.js:315:26)
    at /Users/zander/app/node_modules/@babel/core/lib/config/config-chain.js:278:7
    at buildRootChain (/Users/zander/app/node_modules/@babel/core/lib/config/config-chain.js:118:22)
    at loadPrivatePartialConfig (/Users/zander/app/node_modules/@babel/core/lib/config/partial.js:85:55)
 @ multi<EMAIL_ADDRESS><EMAIL_ADDRESS>./.storybook/config.js (webpack)-hot-middleware/client.js?reload=true iframe[2]
ℹ ｢wdm｣: Failed to compile.

here's my devDependencies:
{
  "devDependencies": {
    "@babel/core": "^7.1.0",
    "@storybook/addon-a11y": "4.0.0-alpha.22",
    "@storybook/addon-actions": "4.0.0-alpha.22",
    "@storybook/addon-backgrounds": "4.0.0-alpha.22",
    "@storybook/addon-knobs": "4.0.0-alpha.22",
    "@storybook/addon-links": "4.0.0-alpha.22",
    "@storybook/addon-options": "4.0.0-alpha.22",
    "@storybook/addons": "4.0.0-alpha.22",
    "@storybook/react": "4.0.0-alpha.22",
    "@types/jest": "^23.3.1",
    "@types/react": "^16.4.11",
    "@types/storybook__addon-a11y": "^3.3.1",
    "@types/storybook__addon-actions": "^3.4.1",
    "@types/storybook__addon-info": "^3.4.1",
    "@types/storybook__addon-knobs": "^3.4.1",
    "@types/storybook__react": "^3.0.9",
    "awesome-typescript-loader": "^5.2.0",
    "babel-loader": "^8.0.2",
    "babel-plugin-external-helpers": "^6.22.0",
    "babel-plugin-styled-components": "1.5.1",
    "babel-plugin-transform-class-properties": "6.24.1",
    "babel-plugin-transform-es2015-modules-commonjs": "6.26.2",
    "babel-plugin-transform-object-rest-spread": "6.26.0",
    "babel-preset-env": "^1.7.0",
    "babel-preset-react": "6.24.1",
    "chromatism": "3.0.0",
    "cross-env": "5.2.0",
    "depdoc": "1.0.3",
    "husky": "1.0.0-rc.13",
    "jest": "^23.5.0",
    "jest-dom": "1.10.0",
    "jest-junit": "5.1.0",
    "jest-styled-components": "^6.2.0",
    "lint-staged": "^7.3.0",
    "precise-commits": "1.0.2",
    "prettier": "1.13.7",
    "react": "^16.4.2",
    "react-chromatic": "0.8.4",
    "react-dom": "^16.4.2",
    "react-testing-library": "4.1.3",
    "release-it": "7.4.7",
    "rollup": "^0.63.2",
    "rollup-plugin-auto-external": "1.2.0",
    "rollup-plugin-babel": "^3.0.6",
    "rollup-plugin-commonjs": "^9.1.3",
    "rollup-plugin-jsx": "1.0.3",
    "rollup-plugin-node-resolve": "^3.3.0",
    "rollup-plugin-typescript2": "^0.16.1",
    "size-limit": "0.18.5",
    "storybook-addon-jsx": "^5.4.0",
    "storybook-readme": "^4.0.0-beta1",
    "styled-components": "^3.4.2",
    "ts-jest": "^23.1.4",
    "tslint": "^5.11.0",
    "tslint-config-airbnb": "^5.9.2",
    "tslint-config-prettier": "^1.14.0",
    "tslint-plugin-prettier": "^1.3.0",
    "tslint-react": "^3.6.0",
    "typescript": "^3.0.1"
  }
}

thanks @reggi, I just tried it with babel-bridge to no avail:
this line was added to my devDependencies:
"babel-core": "^7.0.0-bridge.0",

It's this error that consfuses me:
Error: => Create a storybook config file in "./.storybook/config.{ext}".

Why would it say that when there is a config in that directory??

@reggi, actually it's not a hacky wat to solve it. If you still want to use babel 6, you have to do this with a bridge.
@mrmartineau, is this a full error? can you maybe print script it ?

@igor-dv no it wasn't the full error. I'll get the full error soon. Cheers


Fix deprecated castShadows

Fix deprecated warning about using shadowCastingMode instead of castShadows which results in the following warning:
Assets/GoogleCardboard/Cardboard/Scripts/SkyboxMesh.cs(69,12): error CS0029: Cannot implicitly convert type string' to UnityEngine.Rendering.ShadowCastingMode'

Unity 5 correctly handles off-center projections for skyboxes, so the script is #if'ed to a nullity in Unity 5.  So compile issues no longer occur.


Update config field name

Update config field name of IOS_APP_STORE_URL
Jira: https://openedx.atlassian.net/browse/MA-2687

@BenjiLee @danialzahid94 Please review.

Current coverage is 43.87% (diff: 61.53%)

Merging #780 into master will increase coverage by 0.02%

@@             master       #780   diff @@
==========================================
  Files           356        357     +1
  Lines         25244      25250     +6
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
+ Hits          11070      11078     +8
+ Misses        14174      14172     -2
  Partials          0          0

Powered by Codecov. Last update 7a66fba...8f0d182

👍
Just to confirm, there should only be one URI in the APP_UPDATE_URIS for iOS?

@BenjiLee Yes


Figure out how to properly do functional testing

Specifically so MVC can remove their fallback provider code: https://github.com/aspnet/Mvc/issues/1612
cc @davidfowl @loudej

@muratg can you please triage so we can see when/if we can fix the MVC bug?

If this PR goes through, MVC functionals can probably just use the new Hosting.Create overload to add services for the functional tests I think: https://github.com/aspnet/Hosting/pull/146


Misleading error message in PadTargets

In reality, the columns missing were contig and stop.  Note that end and chr were in the target file.
***********************************************************************

A USER ERROR has occurred: Bad input: format error in '/home/lichtens/eval-gatk-protected/scripts/cromwell-executions/case_gatk_acnv_workflow/64b67442-4d5d-46dd-a2b2-f56b5581960a/call-PadTargets/home/lichtens/eval-gatk-protected/scripts/ice_targets_new.tsv' at line 1: Bad header in file.  Not all mandatory columns are present.  Missing: end, chr

***********************************************************************

snippet of the input file:
chr  start   end     name
1       12190   12227   CEX-chr1-12190-12227
1       12595   12721   CEX-chr1-12595-12721
1       13403   13639   CEX-chr1-13403-13639
1       30366   30503   CEX-chr1-30366-30503
1       35137   35174   CEX-chr1-35137-35174
1       35276   35481   CEX-chr1-35276-35481
1       35720   35736   CEX-chr1-35720-35736
1       69089   70010   CEX-chr1-69089-70010

I agree that this should be fixed.  I think changing Missing to Conflicting columns or something along those lines would do the trick.

I think this was fixed in https://github.com/broadinstitute/gatk/pull/2378.


simplify the UX of setting up a Ganglia node

The simplification would include a method to abstract over the steps below:

stale


[AIRFLOW-920] Allow marking tasks in zoomed in subdags

Please accept this PR that addresses the following issues:

https://issues.apache.org/jira/browse/AIRFLOW-920

Testing Done:

Unitests need to be updated

@aoen @saguziel : for testing purposes

I think I can simplify it a bit (the two walk sub dag functions do more or less the same), but it's a bit late her now.

@bolkedebruin are you still working on this?


[Contracts] Allow empty string in name & domain validation.

Fixes # https://github.com/microsoft/basis-planning/issues/925
Changes proposed:

Allow empty string in tunnel name & domain pattern
Tested consuming the change in the CLI

Other Tasks:

[x] If you updated the Go SDK did you update the PackageVersion in tunnels.go

What is the scenario when tunnel name or domain is empty string?

@IlyaBiryukov: it is when you're updating the tunnel name or domain. You should also be able to set it back to the default - no name or domain. Our service has logic in it that when it receives an empty string it sets it back to null. This was brought up in a PR comment by @jasongin in another PR. I filed a separate item to resolve this as it involved contract changes to the validation patterns.

Note: I didn't update IsValidTunnelName(string tunnelName)
This has a string.IsNullOrEmpty(tunnelName) check that will fail. It doesn't get used in the update scenario that we wish to support, and it used in other places (ex: tunnel plan names validation) where an empty string might not be desirable.


How to solve the problem of garbled characters?

As with the title

installed in unraid, version 6.11.5, browser Edge.

Well, this is odd. I have some questions that may help:

Is this a brand-new install or have you updated an existing one?
It looks like there is emoji content in the search box. How was that entered?
What language is your device set to?
Do you see the same issue viewing your instance on other devices?
Do you see the same issue viewing the demo site?
Are you able to try in a different browser?
If you read a note outside of flatnotes (i.e. opening up the .md file), what does the content look like?

Closing as stale.


Need to dump both routines, and any event(s) that trigger those routi…

It is not sufficient to export just the routines.  You also need to export any events that are used to trigger those routines.  This is especially key for those of us that have partitioned our history tables, as the routines are how we maintain the partitions.
This has been tested to work for both export and import on Zabbix 3.0.31 release.  As this is a function of the mysqldump command, I would expect this to work just fine with any other Zabbix version.

Also... Is there a reason you used --skip-extended-insert on the data export?  This option makes importing take FOREVER because it doesn't take advantage of bulk inserts.

Also... Is there a reason you used --skip-extended-insert on the data export?  This option makes importing take FOREVER because it doesn't take advantage of bulk inserts.


Allow Auto Activation When Bracket Matcher Automatically Inserts A Match

This takes part of #294 and separates it out for immediate merge. This allows providers to get a suggestion request if someone types (, [, etc. and bracket matcher inserts the corresponding ), ], etc.

This has other implications, say when someone pastes two characters. Also, this creates two extra garbage arrays on every buffer change. Overall this seems like a hack.

There's two ways to fix this.

Allow 2 characters to trigger
Prevent a bracket marcher match from causing a two character change to the buffer (it only needs to add a single character, which could occur as a separate event from the originally typed character's change event?)

We could avoid using arrays, for sure. I'm not sure solving for 1. In lieu of 2. is a hack though. Once 2. Is addressed, 1 becomes unnecessary and can be removed.


tolerations is not templated correctly

It looks like tolerations has no effect, I had defined it as below but it did not appeared in the daemonset or the pod
tolerations:
- key: somekey
  value: somevalue
  effect: NoSchedule

Thank you
Fixed in PR https://github.com/aquasecurity/aqua-helm/pull/447/


Support Angular 7

Please add support to angular 7.

I am getting the following error with Angular 7:
ERROR Error: StaticInjectorError(AppModule)[ToasterContainerComponent -> ToasterService]:
  StaticInjectorError(Platform: core)[ToasterContainerComponent -> ToasterService]:
    NullInjectorError: No provider for ToasterService!
    at<EMAIL_ADDRESS>(core.js:717)
    at resolveToken (core.js:954)
    at tryResolveToken (core.js:898)
    at<EMAIL_ADDRESS>(core.js:795)
    at resolveToken (core.js:954)
    at tryResolveToken (core.js:898)
    at<EMAIL_ADDRESS>(core.js:795)
    at resolveNgModuleDep (core.js:17739)
    at _createClass (core.js:17786)
    at _createProviderInstance (core.js:17756)

Is it something that should be addressed for Angular 7 or is it an issue I am having? Did anyone experienced the same problem?

I'm using the Toaster with the latest version of the Angular. Have you ever faced some problem?

This is now released under version 7.0.0, although the previous version should have worked just fine.  Apologies for the delay!


Add Relations for Nokia RAML

Added
->3G-3G Relations
->3G-2G Relations
->2G-2G Relations
->2G-3G Relations
for Nokia RAML

commits were added via migration, through the following commit
https://github.com/erssebaggala/bts-ce-lite/commit/a13c3339235d2c935fe33de649cf0e70fca66452