Confusing Guidance

Not sure why Excel is mentioned here?  It might be beneficial to explain a little more around this.
Having some trouble with a Web linked service.  It appears no URL is allowed/displayed and the "advanced" does not show the properties above.  Seen here when testing connection:

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

ID: a51b3335-6434-a98a-28b1-75f8e14aa9cd
Version Independent ID: 6fd36a6f-71a5-4adb-c8e5-da18b43cc176
Content: Copy data from Web Table using Azure Data Factory
Content Source: articles/data-factory/connector-web-table.md
Service: data-factory
GitHub Login: @linda33wj
Microsoft Alias: jingwang

@Mike-EEE Thanks for the feedback! I have assigned the issue to the content author to investigate further and update the document as appropriate.

@Mike-EEE i didn't get your Q #1 - where did you see Excel?
For #2, currently Web connector doesn't have UI-based authoring experience, you need paste the config in JSON into Advanced tab. Try below.
Another reminder: make sure you do need Web connector instead of HTTP connector, which has different capabilities though both retrieve data from an endpoint.
{
    "properties": {
        "typeProperties": {
            "url" : "<input your url>",
            "authenticationType": "Anonymous"
        }
    }
}

#in-process

Thanks for your reply and additional information, @linda33wj.  In my estimation, as someone who has no context/orientation around your product, I would recommend putting a little more context around the Excel as it seems to arrive out of nowhere and is an abrupt transition.
I did not know about web connector and now I am truly confused on the difference between the two. 😉   In my case I am calling an HTTP REST WebAPI which has both "Web" and "Http" in there, so having a resource somewhere that delineates the two would be incredibly valuable to a newb such as myself who is learning about your product.

@Mike-EEE thanks for the feedback on Excel. I'll add more notes there.
I saw you were trying the Web connector on UI thus want to raise the diff, as in majority cases we observe, HTTP connector is what customer need. I think it's the case for your scenario as well. I'd suggest you try the HTTP connector to evaluate, it has UI-based authoring and it's supported in the copy data tool wizard as well.
Regarding the difference between HTTP connector vs Web connector, the links I sent in previous reply describe the respective capability. Let me clarify here again:

HTTP connector is used to retrieve data in response/file-stream from HTTP/REST endpoint.
Web table connector is used to extract data from HTML table in a web page.

OK, so to add to my confusion here, from your first comment I thought it was a Web connector, but it is actually a Web table connector. 😆 And yes, after about a day of fighting with Azure Data Factory, I did end up using the Http connector.
FWIW, in my case, I could not for the life of me get my http connector to work until I put in a User-Agent header for the request.  It must have been about an hour or so doing searches online to figure out what was the problem, which was compounded by the fact that it was a public REST endpoint that was working just fine in my browser.  Ultimately, I ended up copying/pasting all the headers used in my browser session and finally reduced it to that one header.
I am not sure if this is applicable, or if this is a common/standard/assumed practice, but having a note somewhere in the Http connector documentation to use headers from your browser as a template might be worth mentioning.  It's something that I will be doing from now on, at least. 😛
In any case, knowing that there isn't public-UI is valuable here.  Thank you for clarifying that.  It's actually very surprising that the most critical part of this UI is not exposed and made available to the user, resulting in the poor and confusing experience above.  Preview, indeed. 😉

I've add tips for HTTP connector and Web connector in doc based on your feedback. It will go live soon.
#please-close

Great, thank you @linda33wj !


Moving names of primitive atomic tactics from Coq.Init.Notations to Coq.Init.Ltac

There is a Coq PR coq/coq#12023 which introduces a specific file Ltac.v to activate the ltac plugin. This implies a change of qualification of the primitive atomic tactics from Coq.Init.Notations to Coq.Init.Ltac.
This PR adapts MetaCoq to the change.
Maybe a backward-compatible fix is possible but the proposed PR is not: I just adapted the code word by word.

Please merge, thanks!


Additional logging for Distributed event bus

Logging for the event bus configuration

This change is

Reviewed 2 of 3 files at r1, 1 of 1 files at r2, 1 of 1 files at r3.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable


RangeError (index): Invalid value: Valid value range is empty: 0

I/flutter (18543): ══╡ EXCEPTION CAUGHT BY RENDERING LIBRARY ╞═════════════════════════════════════════════════════════
I/flutter (18543): The following RangeError was thrown during paint():
I/flutter (18543): RangeError (index): Invalid value: Valid value range is empty: 0
I/flutter (18543):
I/flutter (18543): Widget creation tracking is currently disabled. Enabling it enables improved error messages. It can
I/flutter (18543): be enabled by passing `--track-widget-creation` to `flutter run` or `flutter test`.
I/flutter (18543):
I/flutter (18543): When the exception was thrown, this was the stack:
I/flutter (18543): #0      List.[] (dart:core-patch/array.dart:166:52)
I/flutter (18543): #1      LineBarVisualizer.paint (package:flutter_visualizers/Visualizers/LineBarVisualizer.dart:39:50)
I/flutter (18543): #2      RenderCustomPaint._paintWithPainter (package:flutter/src/rendering/custom_paint.dart:528:13)
I/flutter (18543): #3      RenderCustomPaint.paint (package:flutter/src/rendering/custom_paint.dart:566:7)
I/flutter (18543): #4      RenderObject._paintWithContext (package:flutter/src/rendering/object.dart:2109:7)
I/flutter (18543): #5      PaintingContext.paintChild (package:flutter/src/rendering/object.dart:172:13)
I/flutter (18543): #6      _RenderFlex&RenderBox&ContainerRenderObjectMixin&RenderBoxContainerDefaultsMixin.defaultPaint (package:flutter/src/rendering/box.dart:2501:15)
I/flutter (18543): #7      RenderFlex.paint (package:flutter/src/rendering/flex.dart:948:7)
I/flutter (18543): #8      RenderObject._paintWithContext (package:flutter/src/rendering/object.dart:2109:7)
I/flutter (18543): #9      PaintingContext.paintChild (package:flutter/src/rendering/object.dart:172:13)
I/flutter (18543): #10     RenderShiftedBox.paint (package:flutter/src/rendering/shifted_box.dart:70:15)
I/flutter (18543): #11     RenderObject._paintWithContext (package:flutter/src/rendering/object.dart:2109:7)
I/flutter (18543): #12     PaintingContext.paintChild (package:flutter/src/rendering/object.dart:172:13)
I/flutter (18543): #13     _RenderProxyBox&RenderBox&RenderObjectWithChildMixin&RenderProxyBoxMixin.paint (package:flutter/src/rendering/proxy_box.dart:123:15)
I/flutter (18543): #14     RenderObject._paintWithContext (package:flutter/src/rendering/object.dart:2109:7)
I/flutter (18543): #15     PaintingContext.paintChild (package:flutter/src/rendering/object.dart:172:13)
I/flutter (18543): #16     _RenderCustomMultiChildLayoutBox&RenderBox&ContainerRenderObjectMixin&RenderBoxContainerDefaultsMixin.defaultPaint (package:flutter/src/rendering/box.dart:2501:15)
I/flutter (18543): #17     RenderCustomMultiChildLayoutBox.paint (package:flutter/src/rendering/custom_layout.dart:361:5)
I/flutter (18543): #18     RenderObject._paintWithContext (package:flutter/src/rendering/object.dart:2109:7)
I/flutter (18543): #19     PaintingContext.paintChild (package:flutter/src/rendering/object.dart:172:13)
I/flutter (18543): #20     _RenderProxyBox&RenderBox&RenderObjectWithChildMixin&RenderProxyBoxMixin.paint (package:flutter/src/rendering/proxy_box.dart:123:15)
I/flutter (18543): #21     _RenderInkFeatures.paint (package:flutter/src/material/material.dart:548:11)
I/flutter (18543): #22     RenderObject._paintWithContext (package:flutter/src/rendering/object.dart:2109:7)
I/flutter (18543): #23     PaintingContext.paintChild (package:flutter/src/rendering/object.dart:172:13)
I/flutter (18543): #24     _RenderProxyBox&RenderBox&RenderObjectWithChildMixin&RenderProxyBoxMixin.paint (package:flutter/src/rendering/proxy_box.dart:123:15)
I/flutter (18543): #25     PaintingContext.pushLayer (package:flutter/src/rendering/object.dart:369:12)
I/flutter (18543): #26     RenderPhysicalModel.paint (package:flutter/src/rendering/proxy_box.dart:1724:15)
I/flutter (18543): #27     RenderObject._paintWithContext (package:flutter/src/rendering/object.dart:2109:7)
I/flutter (18543): #28     PaintingContext.paintChild (package:flutter/src/rendering/object.dart:172:13)
I/flutter (18543): #29     _RenderProxyBox&RenderBox&RenderObjectWithChildMixin&RenderProxyBoxMixin.paint (package:flutter/src/rendering/proxy_box.dart:123:15)
I/flutter (18543): #30     RenderObject._paintWithContext (package:flutter/src/rendering/object.dart:2109:7)
I/flutter (18543): #31     PaintingContext.paintChild (package:flutter/src/rendering/object.dart:172:13)
I/flutter (18543): #32     _RenderProxyBox&RenderBox&RenderObjectWithChildMixin&RenderProxyBoxMixin.paint (package:flutter/src/rendering/proxy_box.dart:123:15)
I/flutter (18543): #33     RenderObject._paintWithContext (package:flutter/src/rendering/object.dart:2109:7)
I/flutter (18543): #34     PaintingContext._repaintCompositedChild (package:flutter/src/rendering/object.dart:127:11)
I/flutter (18543): #35     PaintingContext.repaintCompositedChild (package:flutter/src/rendering/object.dart:95:5)
I/flutter (18543): #36     PipelineOwner.flushPaint (package:flutter/src/rendering/object.dart:876:29)
I/flutter (18543): #37     _WidgetsFlutterBinding&BindingBase&GestureBinding&ServicesBinding&SchedulerBinding&PaintingBinding&SemanticsBinding&RendererBinding.drawFrame (package:flutter/src/rendering/binding.dart:348:19)
I/flutter (18543): #38     _WidgetsFlutterBinding&BindingBase&GestureBinding&ServicesBinding&SchedulerBinding&PaintingBinding&SemanticsBinding&RendererBinding&WidgetsBinding.drawFrame (package:flutter/src/widgets/binding.dart:761:13)
I/flutter (18543): #39     _WidgetsFlutterBinding&BindingBase&GestureBinding&ServicesBinding&SchedulerBinding&PaintingBinding&SemanticsBinding&RendererBinding._handlePersistentFrameCallback (package:flutter/src/rendering/binding.dart:285:5)
I/flutter (18543): #40     _WidgetsFlutterBinding&BindingBase&GestureBinding&ServicesBinding&SchedulerBinding._invokeFrameCallback (package:flutter/src/scheduler/binding.dart:1033:15)
I/flutter (18543): #41     _WidgetsFlutterBinding&BindingBase&GestureBinding&ServicesBinding&SchedulerBinding.handleDrawFrame (package:flutter/src/scheduler/binding.dart:975:9)
I/flutter (18543): #42     _WidgetsFlutterBinding&BindingBase&GestureBinding&ServicesBinding&SchedulerBinding._handleDrawFrame (package:flutter/src/scheduler/binding.dart:891:5)
I/flutter (18543): #46     _invoke (dart:ui/hooks.dart:236:10)
I/flutter (18543): #47     _drawFrame (dart:ui/hooks.dart:194:3)
I/flutter (18543): (elided 3 frames from package dart:async)
I/flutter (18543):
I/flutter (18543): The following RenderObject was being processed when the exception was fired: RenderCustomPaint#70ce9 relayoutBoundary=up4:
I/flutter (18543):   creator: CustomPaint ← Visualizer ← Column ← Padding ← ConstrainedBox ← Container ← StateBuilder ←
I/flutter (18543):     MediaQuery ← LayoutId-[<_ScaffoldSlot.body>] ← CustomMultiChildLayout ← AnimatedBuilder ←
I/flutter (18543):     DefaultTextStyle ← ⋯
I/flutter (18543):   parentData: offset=Offset(0.0, 50.0); flex=null; fit=null (can use size)
I/flutter (18543):   constraints: BoxConstraints(0.0<=w<=392.7, 0.0<=h<=Infinity)
I/flutter (18543):   size: Size(392.7, 0.0)
I/flutter (18543): This RenderObject had the following descendants (showing up to depth 5):
I/flutter (18543):     child: RenderLimitedBox#93b58 relayoutBoundary=up5 NEEDS-PAINT
I/flutter (18543):       child: RenderConstrainedBox#581d2 relayoutBoundary=up6 NEEDS-PAINT
I/flutter (18543): ════════════════════════════════════════════════════════════════════════════════════════════════════
V/AudioTrack(18543): ~AudioTrack, releasing session id 47801 from 18543 on behalf of 18543
I/ExoPlayerImpl(18543): Init 778e5b1 [ExoPlayerLib/2.10.3] [whyred, Redmi Note 5, Xiaomi, 28]

it was fine at first, everything work, but when I stop the player then play it again , the visualizer won't show up. reproduce repo https://github.com/karnadii/flutter_visualizers_test. I have to hot restart to make it work again.

i manage to make it work.

i manage to make it work.

How did you do ? I'm facing the same error


Moving to Go modules, bumping k8s libraries, bumping prometheus client

Changes

Biggest and most dangerous change: bumped version number of kubernetes related packages k8s.io/api, k8s.io/apimachinery, k8s.io/client-go. I bumped them up to around 1.10 (https://github.com/kubernetes/client-go#compatibility-matrix). Open to discussing this. This was done because I needed to regenerate generated codes and could not go get the old versions (admittedly my fault).

Also bumped the prometheus client to support the migration to go modules.

Using the improbable fork of mwitkow/go-flagz, because it wasn't compatible with go modules.

Removed a test because Go 1.12+ net/url.Parse does not support non numeric ports anymore.

Verification
I ran the test suite locally and in CI.
I am yet to run e2e tests or anything fancier.

You'll need to update line 20 of resolver.go too

@thommay Thanks, done.


Fixes unnecessary change in animation length property while using the slider

Fixes #37072
After this fix the length of animation updates only when the values are finally changed (either a mouse release event of slider/grabber or confirmation of keyboard input)
Changes made:

Added EditorSpinSlider::is_grabbing to check the grabbing status of EditorSpinSlider
Added EditorSpinSlider::_ungrabbed to trigger signals when mouse is released after grabbing
value_changed is triggered at mouse release event.
EditorSpinSlider::pre_grab_value is also updated for changes through grabber
AnimationTimelineEdit::_anim_length_changed skips changes if slider is grabbed, in this way the issue is fixed

I think this attempts to do something similar to what https://github.com/godotengine/godot/pull/69530 did. cc @TokageItLab to confirm.

I remember that I solved a similar problem to this with a signal instead of a bool value expose. I will check later to see if this is still necessary.

In this case, since there is no other process occurring during the value change, simply adding the MERGE flag should be sufficient. I sent #71139.

Superseded by #71139.

Superseded by #71139


estimate_timlag & estimate_dimension

I noticed that the estimate_timelag function in delay_embeding/estimate _timelag requires the transposed version of the data array e.g. (TxN) instead of (NxT) like everywhere else.
Also the estimate_dimension function in delay_embedding/estimate dimension gives the following error for some datasets while functioning perfectly for others.
RayTaskError(ValueError): ray::_dim_fnn() (pid=9294, ip=<IP_ADDRESS>)
File "/home/jentsch/FCF/delay_embedding/estimate_dimension.py", line 19, in _dim_fnn
X_embedded = helpers.create_delay_vector(X,delay=time_delay,dim=dimension)
File "/home/jentsch/FCF/delay_embedding/helpers.py", line 58, in create_delay_vector
dvs[idx,:] = vec(sequence[idx:tShift+idx+1:delay,:])
ValueError: could not broadcast input array from shape (0,) into shape (372,)

Furthermore, I think it would be beneficial if the save part of the ccm.connectivity() function would also save the timelag and dimension used.

Dear Terry, thanks for your interest in my our code package. I just fixed both issues, now time (T) is the leading dimension of all functions. Please check it again and let me know if there are any issues. I also added delay and dimension to the saved properties of the ccm.connectivity function. Regarding the error you're getting, the use case of the false nearest neighbors method is on a per-dimension basis. Although you can input the function with the full multivariate signal (T,N) but you should be careful about the interpretation. Please give it another try and let me know if the error you're getting persists. Notice that the delay time needs to be smaller than the total number of time points in the signal.

Dear Terry, thanks for your interest in my our code package. I just fixed both issues, now time (T) is the leading dimension of all functions. Please check it again and let me know if there are any issues. I also added delay and dimension to the saved properties of the ccm.connectivity function. Regarding the error you're getting, the use case of the false nearest neighbors method is on a per-dimension basis. Although you can input the function with the full multivariate signal (T,N) but you should be careful about the interpretation. Please give it another try and let me know if the error you're getting persists. Notice that the delay time needs to be smaller than the total number of time points in the signal.

Resolved


Can't get Jint when no script tag is included

I am trying to get Jint engine from the javascript service. However, when there is no script tag with a body, GetJint method returns null. I can get Jint engine after adding a script dynamically or including a script tag with body at start. It can be tested with following method.
[TestMethod]
public async Task JintTest()
{
    var cfg = Configuration.Default.WithJavaScript().WithDefaultLoader();
    var svc = cfg.Services.OfType<ScriptingService>().FirstOrDefault(x => x.Engine.Type == "text/javascript");

    var htmlEmpty = "<!doctype html><script type=\"text/javascript\"></script>";
    var htmlNonEmpty = "<!doctype html><script type=\"text/javascript\">a=5</script>";

    var document = await BrowsingContext.New(cfg).OpenAsync(m => m.Content(htmlNonEmpty));
    Assert.IsNotNull(svc.Engine.GetJint(document), "Can't get Jint when the script tag is not empty");

    document = await BrowsingContext.New(cfg).OpenAsync(m => m.Content(htmlEmpty));
    Assert.IsNotNull(svc.Engine.GetJint(document), "Can't get Jint when the script tag is empty");  // Currently error is here

    var script = document.CreateElement("script");
    script.SetAttribute("type","text/javascript");
    script.TextContent = "a=5";
    document.Head.AppendChild(script);

    Assert.IsNotNull(svc.Engine.GetJint(document), "Can't get Jint even after dynamically adding the script");
}

Hm the question is now if this is an AngleSharp problem or an AngleSharp.Scripting problem.
I think AngleSharp should support the notion of appending a script later on, where the script execution is triggered once the element is appended. So if the script is really executed then AngleSharp is behaving as it should be.
That being said Jint is only active for a document if it has been invoked for this document. Otherwise, no JavaScript engine is created. Potentially, the GetJint should be changed to GetOrCreateJint, but right now its simply a GetJint and results in null if no JavaScript has yet been executed.
One more thing to try in your code: As the script execution is asynchronous have you tried, e.g., inserting a await Task.Delay(1000); between appending the script element and getting the engine? 1 second is definitely over the top here, but if that is not working then no script is executed for sure.

I tried delaying at it still doesn't work. Note that I can get Jint after running a script on the page through a tag.
I considered the Jint is created lazily (for performance purposes maybe). The problem is I can't run a Javascript code before atleast one javascript code is run through <script> tag. So should I go out of the way to dynamically add a dummy script tag in case GetJint might return null?
Example counter case in browsers: Go to a plain text link which makes sure no script is included. Open console (F12). Write your script in console and it works.
Of course that doesn't mean a browser includes the scripting engine in any case before you open the developer tools. I am just inquiring if the current GetJint works as intended.

The scenario you describe is still possible. How else what script elements evaluate their input? GetJint only gets an already created context; what you want is the EvaluateScriptAsync functionality. This is also the functionality that would be executed by a browser.
I don't know if the previously described behavior (adding a script tag does not add an engine) is a bug or not yet, but GetJint does what its asked for: Get Jint if its already available. It does not create one for you (otherwise it should be called GetOrCreateJint as mentioned).
Long story short: The GetJint works as designed.

Available in latest version.


More copyediting

@jpsim @tgoyne @bdash
Changes:

Copyedited RLMObject and RLMObjectSchema
Changed certain repeated copy throughout the source to read the same

I updated based on feedback. Also fixed more shared copy.

FYI two small SwiftLint warnings: https://travis-ci.org/realm/realm-cocoa/jobs/132426310
Object.swift:93: warning: Trailing Whitespace Violation: Lines should not have trailing whitespace. (trailing_whitespace)
Realm.swift:289: warning: Line Length Violation: Line should be 120 characters or less: currently 121 characters (line_length)

Thanks @jpsim! SwiftLint is super useful.

If there are no further comments I'll merge this as soon as the most recent CI build passes.


Can now compute the sum of a list of space-separated integers.

Added some documentation and the ability to compute the sum of a list of space-separated integers. Requires Python 3 though.

thanks for pr @aryan-mann
code works fine

"print" in client.py has an error @aryan-mann

changed a line in server.py and code works well
num_list = list(map(lambda x: int(x), data.decode("utf-8").split( )))
#split( ) works this way for int
thanks for PR and changes @aryan-mann


Fix duplicate DB entries, equal staging/production

Set redis workers concurrency to 1 to eliminate duplicate DB entries
Staging should as close as possible to the production build to
detect issues caused by different software versions and configurations

I think setting the celery worker concurrency to 1 is not the correct fix in this case. The problem was actually that the schedule was not being defined properly and by chance it was working on staging because there the workers are only 1 (due to it having 1 core).
I properly address this issue in https://github.com/TheTorProject/ooni-measurements/pull/18 and added the pins you suggest in there for the DB version and MQs

Closing as this has been addressed in #18.


Propose projects for Community Bridge Summer and Summer of Docs

Ticket for maintainers, but anyone else in the community please feels free to suggest projects :hugs:
Details: https://twitter.com/CloudNativeFdn/status/1250801240503652352
Deadline: 24th April.
Ideas so far:
Community Bridge Summer
https://github.com/thanos-io/thanos/issues/1598
Summer of Docs:
https://github.com/thanos-io/thanos/issues/2041
cc @thanos-io/thanos-maintainers

https://github.com/thanos-io/thanos/issues/2454

Done!


ISCSI LUN Expansion issue - csi.trident.netapp.io  can't resize a non-NFS PV

Hello All,
Currently having isssue expanding the ISCSI LUN.
Trident Version : 20.04.0
K8s Version : v1.16.14
Ontap : 9.7
I have edited the PVC to new LUN size, this is what i have been getting:
Events:
Type     Reason             Age                       From                   Message

Warning  ExternalExpanding  33m                       volume_expand          Ignoring the PVC: didn't find a plugin capable of expanding the volume; waiting for an external controller to process this PVC.
Warning  ExternalExpanding  19m (x2 over 20m)         volume_expand          Ignoring the PVC: didn't find a plugin capable of expanding the volume; waiting for an external controller to process this PVC.
Warning  ExternalExpanding  9m11s                     volume_expand          Ignoring the PVC: didn't find a plugin capable of expanding the volume; waiting for an external controller to process this PVC.
Warning  ResizeFailed        (x13 over 31m)  csi.trident.netapp.io  can't resize a non-NFS PV.
Also tried editing : manifests/kube-apiserver.
- --feature-gates=ExpandCSIVolumes=true
- --feature-gates=ExpandInUsePersistentVolumes=true
also tried this:
kube-apiserver --disable-admission-plugins=PersistentVolumeClaimResize
Thanks
Chaitan

@Chaitan123 based on our earlier conversations it looks like this was a k8s upgrade (from 1.15 to 1.16). Can you try bouncing your Trident Controller pod? If you have installed Trident with tridentctl the k8s version change won't be picked up unless the Trident Controller pod inits again. You can get around this by using the Trident operator

Hello Thanks Balaramesh for your response, i have bounced the trident pods, still getting the error:
Type     Reason             Age                 From           Message

Warning  ExternalExpanding  2m20s (x2 over 2d)  volume_expand  Ignoring the PVC: didn't find a plugin capable of expanding the volume; waiting for an external controller to process this PVC.
Also edited: - kube-apiserver --disable-admission-plugins=PersistentVolumeClaimResize
Thanks
Chaitan

That "error" is a normal message from Kubernetes stating that k8s has no built-in plugin that can handle it. The external controller is says it's waiting for is Trident. LUNs cannot resize until the pod(s) mounting them are started (or restarted if they're currently in use) because the filesystem must be resized as well and that requires data access to the LUN while nothing is using it, so k8s will not inform trident of the resize until the re/start of the pod.

Thanks for the responses, i have re-installed trident and it worked as expected.
Thanks

Thanks for the responses, i have re-installed trident and it worked as expected.


import from within a template's script

I'd need to import something from a local module in order to implement a custom element.
Something like:
<template name="xxx-yyy">
  <script>
    import XElement from './x.js';
    export default class extends XElement {
    }
  </script>
</template>

but it fails with:

TypeError: Failed to resolve module specifier "./x.js". Invalid relative url or base scheme isn't hierarchical.

It seems like the problem is that you can't import relative paths from within a script imported via a data URL.
How would you suggest to handle this using Ponys?

Hi @BlueNebulaDev, thanks for posting the issue - I'll look into it, and will get back to you with my best attempt at a fix or a workaround. My initial thought is to try respecifying the path to the local module using import.meta.resolve(), but I don't yet see a way to get this to work for more deeply nested relative imports; it may be that no better outcome is possible before the browsers implement HTML modules. In any case, I'll see what I can do; if you have any ideas, please let me know! :slightly_smiling_face:

I'm short of ideas. The only working solutions I can think of is to pre-process the file. But that seems quite against the philosophy of Ponys.
Are JS modules finalized? I believe it might be worth bringing up this issue with the committee working on them. The clean solution would be for dynamic import to accept as a parameter its own URL (import("data:...", {url:"."}))...

Hi @BlueNebulaDev, I think I figured out a solution to this issue: using the URL constructor, I was able to reliably rewrite any relative import paths found during parsing of the component setup script as absolute URLs, with respect to both location.origin and the location of the component class.
Hopefully that's fixed it; please do let me know if you encounter any further unexpected behavior. Thanks again for raising the issue! :slightly_smiling_face:


Added PWM support for Raspberry Pi Pico

Summary
Added code to the Raspberry Pi Pico to support the PWM driver code.
Impact
This should have no impact on existing functionality.
Testing
Have tested the ability to generate wave forms using existing PWM demo application.

@curuvar please remove the merge patch from the PR.

@curuvar in order to fix the merge issue reported by @xiaoxiang781216 you can do it:
leave your branch and move to master branch (or upstream branch if you followed our documentation: https://nuttx.apache.org/docs/latest/contributing/making-changes.html )
$ git checkout upstream
$ git pull
$ git checkout your_branch_name
$ git rebase upstream
$ git push -f

I'm trying to figure out the rebase.  I (probably foolishly) made the changes in the master branch.  I still don't really have the zen of git.  Should I delete my clone of the repository and build a new one from scratch or is there some other easy way to recover?

I'm going to close and re-open the pull request to try to get this done right.

Sure, thank you @curuvar


Running test fails trying to access non-existing file

Using vscode-python 2020.6.88468
VSCode 1.46.0
When I run a test using the pytest framework, there is always an error at the end of the log:
Error: Error: cannot open file:///c%3A/Code/pixelcrunch/C. Detail: Unable to read file 'c:\Code\pixelcrunch\C' (Error: Unable to resolve non-existing file 'c:\Code\pixelcrunch\C')
regardless if the test itself actually passes or fails:

I don't know what is causing it to look for this file.

We're still trying to figure out automatic conda activation - it's a very complicated problem, unfortunately. You can track that work via #5344.

Understandable.....

Could this be at all related to getting this error whenever I try to debug my python code? My files definitely exist, and the configurations were working yesterday. Not sure what's changed since then. Possibly an extension update I suppose.

@tenspd137 Unfortunately, I am not able to reproduce the issue you see regarding
Error: Error: cannot open file:///c%3A/Code/pixelcrunch/C. Detail: Unable to read file 'c:\Code\pixelcrunch\C' (Error: Unable to resolve non-existing file 'c:\Code\pixelcrunch\C')

I am assuming c:\Code is where you have your VSCode installed? Can you send me a file list in that directory? And also, of this c:\Code\pixelcrunch directory?
Also, can you try installing an older version of the extension (for example 2020.4.58038), and see if your problem happens?
You can follow the prompts from "Extensions: Install Specific Version of Extension..." in the command palette:

Or you can open the extensions side bar, click on the settings for the Python extension and pick "Install Another Version...":

I have a repro of this, and potentially some extra info (ping me internally if you want to more interactively debug)
I get this error message:
Error: Error: cannot open file:///c%3A/Projects/multiindex/X. Detail: Unable to read file 'c:\Projects\multiindex\X' (Error: Unable to resolve non-existing file 'c:\Projects\multiindex\X')

The interesting part here is my C:\Projects folder is actually a mount point to my X: drive. So here's my theory:

something in the PyTest/test support is expecting a relative file path like tests\my_test_file.py
PyTest resolved the path to the real drive
the path is now absolute, and looks like X:\multiindex\tests\my_test_file.py
PVSC is treating : as a separator (probably because it normally is on Unix?) and splitting the "X" off by itself
it then combines the "relative" path X to the workspace root, which does not exist

Probably the best way to deal with this is to detect the absolute path, then fully resolve the workspace root ("realpath" equiv.) and recalculate the relative path. If it's still not within the workspace, then raise that as an error. Otherwise, you've found the file. Recalculating the relative path avoids opening the file in the editor twice, which is what will happen if you don't.

@zooba Great guess. It seems this line is the one throwing the error. Right above the line we're resolving file path similar to how you described. This is some old code so I am not sure of its purpose.

Recalculating the relative path avoids opening the file in the editor twice, which is what will happen if you don't.

I am not sure how opening the file comes into the picture. So do you get this error when opening the file (navigating a suite/function/file etc.)?  Note the original issue reports the error on just running the file - not opening it.

For instance, this is my Python test log,
================================== FAILURES ===================================
_______________________ FailingTests.test_failure_still _______________________

self = <hello_test.FailingTests testMethod=test_failure_still>

    def test_failure_still(self):
>       self.assertEqual("I'm right!", "no, I am!")
E       AssertionError: "I'm right!" != 'no, I am!'
E       - I'm right!
E       + no, I am!

Tests\steve\hello_test.py:6: AssertionError
- generated xml file: C:\Users\karraj\AppData\Local\Temp\tmp-4172kPl1yrfyWOkV.xml -
============================== 1 failed in 0.12s ==============================

Have a look at this string which appears in the traceback: "Tests\steve\hello_test.py:6: AssertionError". We are separating on ":" to get the relative path to the test file Tests\steve\hello_test.py. I am suspecting you have the absolute path appearing here, but I am not sure why. What are the settings you're using?

Anyways, the fix seems clear enough. Don't expect only relative paths at this line and do a file exists check before opening document.

I believe this is a duplicate of https://github.com/microsoft/vscode-python/issues/10658 - both issues report the same bug with not being able to open a file and in the output both of the paths have a c at the end.
Given that the other issues has more 👍 can we close this issue? I can post @karrtikr 's comment on the other issue explaining how it needs to be fixed.

There is super-weird workaround (Windows 10) that I use now, hope this will be fixed in the repo soon.
But if you want to use pytest now, you can do the following.
Go to settings and add a line to Python > Testing: Pytest Args:
--junit-xml=./project-name-tests/results.xml

After that use elevated Power Shell to create a Junction from root of your disk to this subfolder:
cd /
New-Item -Type Junction -Path ./project-name-tests/ -Target .\Path\To\Working\Dir\project-name-tests\

What happens here?
I guess the main problem is that Windows paths are not detected correctly as absolute paths. So test runner tries to read C:\your\project\dir\C:\path\to\temp\file.xml and fails.
However if XML file is generated under your project folder, there is another bug 😫 And it makes test runner to try to open C:\subfolder\file.xml (I guess it's not always C: drive, but the drive you are using for your code).
So this junction point creates a link from C:\subfolder\ to C:\path\to\project\subfolder and the file is now readable even on incorrect path.

+1 for me, sadly.
I never had this problem until I installed the Pytest IntelliSense extension. Although I immediately removed it, the damage was done and my installation appears to be in a permanently broken state until I go through hundreds of hoops to force a removal of all VS code related files as I've had to do once before (and never want to do again).
This has cost me hours of productivity, and as someone who tends to only get a few hours on Saturday to code in my personal time, has really bummed me out (this happened on my personal dev env, not my professional one).

+1 for me, sadly.
I never had this problem until I installed the Pytest IntelliSense extension. Although I immediately removed it, the damage was done and my installation appears to be in a permanently broken state until I go through hundreds of hoops to force a removal of all VS code related files as I've had to do once before (and never want to do again).
This has cost me hours of productivity, and as someone who tends to only get a few hours on Saturday to code in my personal time, has really bummed me out (this happened on my personal dev env, not my professional one).

Wow, this was so incredibly obscure (for me). I've finally come up with a minimal way to reproduce this bug!
Steps to reproduce

Install a fresh environment with pyest. I made a folder named vscode_python_12403 and used pipenv: pipenv install --dev pytest
Create a file, test_12403.py
Paste the following code into the file:

from datetime import datetime
def test_expected_behavior():
# Test fails, but fails in the way we expect both tests to fail
data_string = '2021'
format = '%Y-'
raise ValueError("time data %r does not match format %r" %
(data_string, format))
def test_unexpected_behavior():
# Exact same code (extracted from datetime.strptime code), but breaks and
# causes file lookup failure
datetime.strptime('2021', '%Y-')
4. Click python interpreter on bottom-left and select the venv that pipenv generated
5. Configure tests.
  a. Open command pallete with Ctrl+Shift+P
  b. Enter `Python: Configure Tests` and click
  c. Click `pytest`
  d. Click `. Root directory`
6. Run tests and observe behavior.

> NOTE: You will need to comment out the unexpected behavior test entirely before the expected behavior begins to happen

Wow, this was so incredibly obscure (for me). I've finally come up with a minimal way to reproduce this bug!
Steps to reproduce

Install a fresh environment with pyest. I made a folder named vscode_python_12403 and used pipenv: pipenv install --dev pytest
Create a file, test_12403.py
Paste the following code into the file:

from datetime import datetime
def test_expected_behavior():
# Test fails, but fails in the way we expect both tests to fail
data_string = '2021'
format = '%Y-'
raise ValueError("time data %r does not match format %r" %
(data_string, format))
def test_unexpected_behavior():
# Exact same code (extracted from datetime.strptime code), but breaks and
# causes file lookup failure
datetime.strptime('2021', '%Y-')
4. Click python interpreter on bottom-left and select the venv that pipenv generated
5. Configure tests.
  a. Open command pallete with Ctrl+Shift+P
  b. Enter `Python: Configure Tests` and click
  c. Click `pytest`
  d. Click `. Root directory`
6. Run tests and observe behavior.

> NOTE: You will need to comment out the unexpected behavior test entirely before the expected behavior begins to happen

I have same error when running pytest.
The tests are discoverable and I'm able to run them, but when I 'm running the test in output I see error: "cannot open file:///Users/yurib/Projects/files_uploader/tests/test_system_tests_linux.py"
Worth to mention that previously I moved the project to the new path and in the error vscode somehow refers to the old path of the test file. I tried to clear the cache, deleted python extension, but still same error.

Another way to reproduce:
Create test_main.py with contents:
import unittest
import unittest.mock

class TestCrash(unittest.TestCase):
    def test_simple(self):
        m2 = unittest.mock.Mock(name='mymock')
        # m2() # no error if this is called
        m2.assert_called()

Setup testrunner:

run command 'Python: Configure Tests'
select 'pytest'
select '. Root directory'

Run all tests

'There was an error running the tests' notification is displayed
'Python Test Log' output ends with an error message:

Error: Error: cannot open file:///d%3A/dev/py/1/c. Detail: Unable to read file 'd:\dev\py\1\c' (Error: Unable to resolve non-existing file 'd:\dev\py\1\c')

If I create the missing file ('d:\dev\py\1\c'), the error message is replaced:
Error: Error: Illegal value for `line`

Notes:
My testing environment:
VSCode: 1.58.1
pytest version: 6.2.4
python: 3.7.2
python extension: v2021.6.944021595
The error occurs every time on windows, I've only managed to reproduce it in WSL once.
Deleting pycache fixed WSL, no visible effect on windows.

Am also getting this error, seems to have not been resolved although it is reporducable and the issue was raised over a year ago. I guess I can go back to using test.http

I was able to reproduce this error using the stable build, and verified that https://github.com/microsoft/vscode-python/pull/16769 will fix it 😊

The issue can still be reproduced with the steps in my previous comment:
https://github.com/microsoft/vscode-python/issues/12403#issuecomment-879876830

@rulerofthehuns the issue is still occurring on the stable version of the Python extension but it should be fixed in the next release. It's also now available in our insiders program (View > Command Palette... and run "Python: Switch to Insiders Weekly Channel")

@luabud Thank you for the clarification. I've assumed the python extension was also updated with VSCode's update, and didn't check the version.
I can confirm that the bug cannot be reproduced with the new version.


Cant change language

Hello, maybe I'm doing something wrong but I follow the readme and add this line to my application.js
//= require languages/ro.js
And I get this error after restarting server
couldn't find file 'languages/ro.js'
What I'm missing?
Regards

You should also pass the language as an option when initializing the editor. Including the JS file is not enough. https://www.froala.com/wysiwyg-editor/languages

How can I change it dynamically? Like you do on this page: https://www.froala.com/wysiwyg-editor/languages

@evpozdniakov
I do this by adding a data attribute with the locale that I need (given by rails, like with user preference or with a link to change locale) the I retrieve this before froala initialization. The data attribute is on the text-area tag used by froala.
I'll post some exemple code if I can.


Fix a few typos and add a translation entry for unknown algorithm

Split off from #51.

Merged with master


ovirt: fix ovirt-populator build

Fix oVirt populator build when done in a git workspace
Resolves #411
Signed-off-by: Miguel Martín<EMAIL_ADDRESS>

yeah, we thought about having this Containerfile as a temporary solution until we build the ovirt populator pod with bazel as well, but it can take time until we get there. @liranr23 ?

Yes, it probably will take time.
The change looks OK. Did you try an import with the new image?

edit: we have the CI :)

long live the CI!


[libosmium] Update to 2.18.0

Describe the pull request

What does your PR fix?
Fixes #26316

Which triplets are supported/not supported? Have you updated the CI baseline?
Same as before, Yes

Does your PR follow the maintainer guide?
To the best of my knowledge.

Yes

Thank You!


Superlinter workflow fail to download module error.

Is there an existing issue for this?

[X] I have searched the existing issues

Current Behavior
Summary
We have 2 repositories one is for infra code and another one is for tf modules. When we're running superlinter workflow in infra repo, getting fail to download module. see the below error -
Error: Failed to download module
│
│ Could not download module "bastonhost_ec2" (baston-host.tf:86) source code
│ from
│ "git::ssh://[git@github.com](mailto:git@github.com)/Legal-and-General-Group-Confidential/ptx-pg-modules.git?ref=v1.0.3":
│ error downloading
│ 'ssh://[git@github.com](mailto:git@github.com)/Legal-and-General-Group-Confidential/ptx-pg-modules.git?ref=v1.0.3':
│ /usr/bin/git exited with 128: Cloning into
│ '.terraform/modules/bastonhost_ec2'...
│ Host key verification failed.
│ fatal: Could not read from remote repository.
│
│ Please make sure you have the correct access rights
│ and the repository exists.

Details
Give all details on the vulnerability. Pointing to the incriminated source code is very helpful for the maintainer.
Another problem statement-
Unable to fetch the superlinter workflow report in github.
Impact
Delivery
Expected Behavior
It should not throw error for download the module and run linting, also it should output the report and store in github.
Anything else?
NA

Hi @MSS2114 .
Based on the Host key verification failed. error, I would try setting SSH_SETUP_GITHUB to true in your super-linter configuration and try again.

Can you please suggest where I need to give this value in below workflow file -
name: Terraform Linter
on:
workflow_dispatch:
inputs:
deployment_environment:
required: true
type: string
description: Please type the region you want to deploy in
jobs:
lint_and_analyze:
name: Linting and source code analysis
runs-on: fixed-ip-runner-4core
environment:
name: ${{inputs.deployment_environment}}
permissions:
contents: read
pull-requests: write
  steps:
    - name: Checkout code
      uses: actions/checkout@v2

    - name: Lint Code Base
      uses: github/super-linter@v5
      with:
        github_token: ${{ secrets.GITHUB_TOKEN }}

      env:
        VALIDATE_ALL_CODEBASE: true
        DEFAULT_BRANCH: main
        GITHUB_TOKEN: ${{ github.token }}
        VALIDATE_PYTHON_BLACK: false
        VALIDATE_PYTHON_ISORT: false
        VALIDATE_PYTHON_MYPY: false
        VALIDATE_NATURAL_LANGUAGE: false
        VALIDATE_MARKDOWN: false
        VALIDATE_JSCPD: false
        VALIDATE_GITHUB_ACTIONS: false # Fails when scanning tf security scan action erroneously
        FILTER_REGEX_EXCLUDE: .*.app-servers.tf
        #FILTER_REGEX_EXCLUDE: '.*[.](providers.tf|git@github.com)$' # Due to errors scanning providers containing terraform configuration_aliases

Could it be one of the two downtimes on GitHub in the last hours? Most of my actions were failing, especially when installing dependencies.
https://www.githubstatus.com/incidents/7txcg0j03kkg
https://www.githubstatus.com/incidents/r7641l07b67v

0s
Current runner version: '2.315.0'
Runner name: 'fixed-ip-runner-4core_427c841bb2a9'
Runner group name: 'Default'
Machine name: 'runner'
Operating System
Runner Image
GITHUB_TOKEN Permissions
Secret source: Actions
Prepare workflow directory
Prepare all required actions
Getting action download info
Error: Unable to resolve action github/super-linter@v6.[4](https://github.com/Legal-and-General-Group-Confidential/ptx-pg-infra/actions/runs/8817793163/job/24205060122#step:1:4).1, unable to find version v6.4.1

0s
Current runner version: '2.315.0'
Runner name: 'fixed-ip-runner-4core_427c841bb2a9'
Runner group name: 'Default'
Machine name: 'runner'
Operating System
Runner Image
GITHUB_TOKEN Permissions
Secret source: Actions
Prepare workflow directory
Prepare all required actions
Getting action download info
Error: Unable to resolve action github/super-linter@v6.[4](https://github.com/Legal-and-General-Group-Confidential/ptx-pg-infra/actions/runs/8817793163/job/24205060122#step:1:4).1, unable to find version v6.4.1

The repo changed organization, from github to super-linter. See https://github.com/super-linter/super-linter/issues/5570#issuecomment-2075002794

Error
<EMAIL_ADDRESS>is not allowed to be used in Legal-and-General-Group-Confidential/ptx-pg-infra. Actions in this workflow must be: within a repository that belongs to your Enterprise account, created by GitHub, or matching the following: aquasecurity/tfsec*, aquasecurity/tfsec-pr-commenter-action*, aws-actions/, docker/, github/, hashicorp/, microsoft/, nuget/setup-nuget, veracode/, azure/, helm/chart-releaser-action*, liquibase/liquibase-github-action*<EMAIL_ADDRESS><EMAIL_ADDRESS>sonarsource/sonarqube-scan-action*, sonarsource/sonarcloud-github-action*, octokit/, bmc-compuware/TTT-CLI-Local, nrwl/nx-set-shas*, grafana/, errata-ai/vale-action.

getting an startup failure

Please find the latest log
5_Lint Code Base.txt
The workflow file worked with github/super-linter@v6

Hi! This looks like a configuration issue rather than a bug. Please, follow the instructions in the README.

<EMAIL_ADDRESS>is not allowed to be used in Legal-and-General-Group-Confidential/ptx-pg-infra

It looks like you need to allowlist this action to be used in your repository.

Hey @ferrarimarco ,
Can we setup a call and see the issue there?


WW3 fails to compile on gnu with gcc>9

From gcc10,  gfortran requires the -fallow-argument-mismatch option to compiler MPI programs, as this has not been fixed upstream by MPI implementations.
In line cmplr.env line 184, change the general options to
optc='-c -J$path_m -p -g -fno-second-underscore -ffree-line-length-none -fconvert=big-endian -fallow-argument-mismatch'

Conditional upon gcc version > 9 @awest-noaa

Better yet, make use of ESMF_F90COMPILEOPTS ...

This has been fixed in the WW3 version included in CoastalApp.


Performance issues under large-scale

Describe the bug
I didn't use the count method, but he executed the command, and the query was still slow.
I want to know why.
Why not use db.User. count()?
Stack trace
{ "op" : "command", "ns" : "qxg.User", "command" : { "aggregate" : "User", "pipeline" : [ { "$match" : {  } }, { "$group" : { "_id" : 1, "n" : { "$sum" : 1 } } } ], "cursor" : {  }, "lsid" : { "id" : UUID("e1b53a94-8acc-4973-9e21-e044009fa8d3") }, "$db" : "qxg" }, "keysExamined" : 0, "docsExamined" : 104561, "cursorExhausted" : true, "numYield" : 105, "nreturned" : 1, "locks" : { "ReplicationStateTransition" : { "acquireCount" : { "w" : NumberLong(1) } }, "Global" : { "acquireCount" : { "r" : NumberLong(108) } }, "Mutex" : { "acquireCount" : { "r" : NumberLong(2) } } }, "flowControl" : {  }, "storage" : { "data" : { "bytesRead" : NumberLong(6446414), "timeReadingMicros" : NumberLong(213104) } }, "responseLength" : 121, "protocol" : "op_msg", "millis" : 441, "planSummary" : "COLLSCAN", "ts" : ISODate("2022-03-15T12:16:08.402Z"), "client" : "212xx", "allUsers" : [ { "user" : "xxx", "db" : "qxg" } ], "user" : "xxx" }
Versions

Dependency
Version

node
16

typescript
?

mikro-orm
4 laster

your-driver
mongo

I found that the problem lies in the dependency of mongodb


[SPARK-20109][MLlib] Rewrote toBlockMatrix method on IndexedRowMatrix

What changes were proposed in this pull request?

I added the method toBlockMatrixDense to the IndexedRowMatrix class. The current implementation of toBlockMatrix is insufficient for users with relatively dense IndexedRowMatrix objects, since it assumes sparsity.

EDIT: Ended up deciding that there should be just a single toBlockMatrix method, which creates a BlockMatrix whose blocks may be dense or sparse depending on the sparsity of the rows. This method will work better on any current use case of toBlockMatrix and doesn't go through CoordinateMatrix like the old method.
How was this patch tested?
I used the same tests already written for toBlockMatrix() to test this method. I also added a new additional unit test for an edge case that was not adequately tested by current test suite.
I ran the original IndexedRowMatrix tests, plus wrote more to better handle edge cases ignored by original tests.

Can one of the admins verify this patch?

Addressed comments where everything was clear, replied to the last one about only having one toBlockMatrix. Back to you @viirya . Thanks for feedback.

ok to test.

oh, seems only the committers can trigger jenkins test. cc @jkbradley @MLnick

ok to test

Test build #75475 has started for PR 17459 at commit 06c2b3a.

Test build #75475 has finished for PR 17459 at commit 06c2b3a.

This patch fails Scala style tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/75475/
Test FAILed.

Thanks for the feedback guys. All comments addressed, though if anyone else has feedback on discussion me and @viirya are having about whether there should be a separate toBlockMatrixDense method or we should just have an argument to specify Dense or Sparse in the default toBlockMatrix method, please weigh in. Thanks.

Test build #75482 has started for PR 17459 at commit 12e78bf.

Test build #75482 has finished for PR 17459 at commit 12e78bf.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Did some thinking about this, and I think that to make the API cleaner maybe we could deprecate the regular toBlockMatrix method and add toBlockMatrixSparse. Until it's removed, we could just have toBlockMatrix call toBlockMatrixSparse. I think that'd be more explicit and make it clearer for users what kind of BlockMatrix they're creating. After that I think it'd be easy enough for me to abstract out the DenseMatrix creation step of toBlockMatrixDense and make it a general purpose helper method that toBlockMatrixDense and toBlockMatrixSparse can call to create the specific type of BlockMatrix they want.
I think this explicitness is important since it seems a lot of users create a BlockMatrix through these methods as opposed to with a BlockMatrix constructor since it's kind of a hard constructor to use (official Spark docs also suggest to users that it's easier to use toBlockMatrix than to attempt to use constructors: https://spark.apache.org/docs/latest/mllib-data-types.html#blockmatrix).
If we don't want to go the deprecation route, we could have toBlockMatrix take an argument specifying whether the data is sparse or dense, but I think that should be an explicitly required argument since it's otherwise easy to create something unintended.

I've done some prototype locally to generalize this change to SparseMatrix. During that, I have a thought that do we have the limit that all Matrix in BlockMatrix need to be the same kind of Matrix (i.e., DenseMatrix or SparseMatrix)?
Actually we can easily have only one toBlockMatrix method which creates a BlockMatrix including both DenseMatrix or SparseMatrix, depending if the blocks are sparse or not.
From the external view of this API, we don't have an explicit difference between SparseMatrix-backed and DenseMatrix-backed BlockMatrixs. We don't have subclasses for it, nor any property can be used to know about it. Doesn't it mean we don't really care about it?

@viirya I think we definitely care about giving users the ability to make either dense or sparse Block matrices. I made a 100k by 10k IndexedRowMatrix of random doubles, then converted it to a BlockMatrix to multiply it by its transpose. With the current toBlockMatrix implementation, that took 252 seconds on 128 cores. With my implementation, that took 35 seconds. The backing of a BlockMatrix matters a lot, and we need to let users be explicit about it.
I considered having toBlockMatrix check if the rows of IndexedRowMatrix were dense or sparse, but there is no guarantee of consistency. Like, an IndexedRowMatrix could be a mix of Dense and Sparse Vectors. In that case, it would not be clear what type of BlockMatrix to create. A decent approximation of this would be to just decide the matrix type based on the first vector we look at in the iterator we get from groupByKey, but I still think it's best to be explicit.

I considered having toBlockMatrix check if the rows of IndexedRowMatrix were dense or sparse, but there is no guarantee of consistency. Like, an IndexedRowMatrix could be a mix of Dense and Sparse Vectors. In that case, it would not be clear what type of BlockMatrix to create. A decent approximation of this would be to just decide the matrix type based on the first vector we look at in the iterator we get from groupByKey, creating a mix of Dense and Sparse matrices in a BlockMatrix, but I still think it's best to be explicit. Also, we currently have the description of toBlockMatrix promising to make a BlockMatrix backed by instances of SparseMatrix, so we have made promises to users about the composition of the BlockMatrix before.

I don't mean we don't care about it. I meant there is no guarantee that BlockMatrix is purely consisted of DenseMatrix or SparseMatrix. It could be a mix of them.
Thus, we can have a toBlockMatrix which creates a BlockMatrix which is a mix of DenseMatrix and SparseMatrix. A block in a BlockMatrix can be a DenseMatrix and SparseMatrix, depending on the ratio of values in the block.
For a IndexedRowMatrix completely consisted of DenseVector, this toBlockMatrix definitely returns a BlockMatrix backed by DenseMatrix. For other cases, DenseMatrix might not be best choice for all blocks in the BlockMatrix, as many blocks will be sparse.
About the promise that toBlockMatrix makes a BlockMatrix backed by instances of SparseMatrix, as I said it is not explicitly bound to the API level. I think it is not a big problem.

Alright, I agree with this. We'll switch off Dense or Sparse matrix backings based on what the type of the first vector in the iterator is. I'd be happy to take on making these adjustments.

@johnc1231 The prototype I did: https://github.com/apache/spark/compare/master...viirya:general-toblockmatrix?expand=1
Maybe you can take a look and see if it is useful to you.

I think your prototype looks good. I'm pretty much just gonna do exactly that then.

@viirya I made changes exactly as you did in your prototype, plus a few style edits. But yeah, I think this is a good, easy to use implementation that will be better in all use cases than current implementation.

Test build #75518 has started for PR 17459 at commit 4582a7e.

Test build #75518 has finished for PR 17459 at commit 4582a7e.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/75518/
Test PASSed.

Merged build finished. Test PASSed.

Please modify the PR title also. Thanks.

@viirya Added appropriate tests, back to you.

Test build #75575 has started for PR 17459 at commit a38851c.

Test build #75575 has finished for PR 17459 at commit a38851c.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/75575/
Test PASSed.

@viirya Do you have any more comments on this, or are you happy with it?

Test build #75754 has started for PR 17459 at commit 3fe21cf.

Only one remaining comment left for the test.
Btw, @johnc1231 Do you think it is possible you can run a simple benchmark, so we know whether this change improves the performance too?

Test build #75754 has finished for PR 17459 at commit 3fe21cf.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/75754/
Test PASSed.

Merged build finished. Test PASSed.

@viirya
Gonna fix that last test tonight, but just doing a quick timing on my laptop, (2014 Macbook Pro with 2.2Hz i7), the difference was as follows on a 10k by 10k matrix of random doubles between 0 and 1:
New method: 113837 ms
Old method: 167853 ms
So it's clearly better in the dense case. Will benchmark sparse tonight as well.

Test build #76107 has started for PR 17459 at commit d692d30.

Test build #76107 has finished for PR 17459 at commit d692d30.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/76107/
Test PASSed.

Merged build finished. Test PASSed.

@viirya I fixed the test as you asked, so please take a look when you get a chance. I'm having a little bit of trouble with my local spark build for some reason, but I'll do that other benchmark when it's resolved.

@viirya Any more feedback on this?

@johnc1231 Thanks for updating this. I'll review it in the weekend.

Except for few comments regarding style, the code changes LGTM.
And it'd be good if we can have benchmark for sparse case too.
cc @MLnick @jkbradley for review.

Did a sparse benchmark (2014 Macbook Pro with 2.2Hz i7) 60 partitions, 10k by 10k matrix with mostly 0's, 10% 1's, made of SparseVectors. Both old method and new method took about 7.5 seconds.

@viirya Addressed style nitpicks and did spark benchmarks. Think that should be everything.

Test build #76504 has started for PR 17459 at commit 994b457.

Test build #76504 has finished for PR 17459 at commit 994b457.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/76504/
Test PASSed.

Merged build finished. Test PASSed.

@viirya Now that style nitpicks and sparse benchmarks are done, are you good with this? Also, per your recommendation, CCing @MLnick and @jkbradley for review of this. Should be easy to review, since we've iterated on it a lot.

Also, made changes suggested by @srowen . Don't know if he now has to sign off on those changes being done.

This has been reviewed pretty thoroughly at this point. Can a committer give this a quick look? @srowen @MLnick @jkbradley I think it's basically ready to go in.

@srowen Addressed comments, back to you. And thanks for taking time to look this over.

Test build #77437 has started for PR 17459 at commit a7a03dc.

@srowen Fixed both, back to you

Test build #77439 has started for PR 17459 at commit 289dbdb.

Test build #77437 has finished for PR 17459 at commit a7a03dc.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/77437/
Test PASSed.

Test build #77439 has finished for PR 17459 at commit 289dbdb.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/77439/
Test PASSed.

@srowen @viirya All comments addressed, back to you guys. Hopefully we've just about reached something ready to commit.

Test build #77606 has started for PR 17459 at commit f9c5506.

Test build #77606 has finished for PR 17459 at commit f9c5506.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/77606/
Test PASSed.

Merged to master


[Plugin] Plugin names

In this issue, I'd like to decide what the names of all plugins will be and their respective priority.
We should be able to find each plugin at github.com/vigour-io/<name>

priority
name
other possible names
description & scope

1
status-bar
bars
native status bar control (and the menu bar on android)

1
env
environment, info, device, device-info, platform, client
provides static info about the device and platform's name and version, bundle id, etc.

1
network
connectivity
provides an observable for connectivity (wifi, 3G, 4G, none, etc.)

1
orientation

listens to the orientation of the device (pure js?)

1
pay
purchasing, money, payment, store, purchase
native purchasing and plugin architecture for web payments (pay pal, credit card, ...)

1
auth
login
login via facebook, google +, others

1
chromecast

Everything needed for Chromecast sender apps

1
share

Share via other apps (e-mail, facebook, twitter, google +, etc.).

1
infonline

InfOpen stuff

2
dial

everything DIAL

3
notify

native notifications

3
remote

Remote controls

4
fs

Native file system access including file transfer (upload/download)

4
settings
user-settings
Observable for device settings, e.g. preferred language, date & time format, etc.

For now, let's just agree on a name & priority for each plugin. The scope of each plugin is way bigger than what we need immediately, and prioritizing specific features will happen in that plugin's repo. For example, we won't care about the user's preferred date & time format for a very long time, but that plugin is indeed a # 1 priority because we will be interested in connectivity.
I'm not particularly attached to any of these names, so please propose alternatives for all of them. I'd like to have all these names settled by Sunday morning (no beating around the bush).
/cc @jimdebeer @youzi @gideonvd

@jimdebeer We discussed the following changes:

create a facebook plugin with

facebook.share
facebook.login

demote "share" plugin priority

remove "auth" plugin

I don't like it.
Devices have this general share thing where you share stuff and they propose a list of all the share-capable apps on your device (whatsapp, slack, mail, hipchat, message, iCloud, twitter, save PDF to iBooks, snapchat, messenger, etc, etc, etc). So I don't see why the share feature sould be in the facebook plugin, as you may want to share with other apps, and we don't want one plugin per app (we'll never see the end of it).
Same goes for login. We currently only support logging in via facebook or e-mail, but it's reasonable to think we'll be logging in via oauth, twitter, etc, in the future. Do we really want one plugin per login-capable app?
Of course, my argument is less strong for login, as there is no general login via xyz dialog offered by native devices. But for sharing, this seems quite obvious to me.

About the naming thing... I'd say let's use npm scopes. New NPM has already
cost us quite a bit of extra effort so let's enjoy the perks at least.
https://docs.npmjs.com/getting-started/scoped-packages
On Thursday, October 15, 2015, Shawn Inder<EMAIL_ADDRESS>wrote:

The repo name doesn't have to be the same as the npm name, but yes
we'llhave to prefix for npm :( probably "vigour-"
—
Reply to this email directly or view it on GitHub
https://github.com/vigour-io/vigour-native/issues/86#issuecomment-148381703
.

@ninjatux you asked for a rough estimate of the prio 1 plugins:

plugin
estimation
remark

status-bar
3h

env
1h

network connectivity
3h

orientation
2

pay
8h
just play store, right? no native sdk for paypall I hope...

auth
8h
depends on how many varieties

chromecast
8h
never done this before, I don't know...

share
8h
also here it depends on what we need as a minimum

infonline
--
AFAIK not possible (see comment above)

@michielibiza  don't forget facebook, it was included later but is needed for a specific implementation so we want to give it max priority

8h for the pay plugin, you are so cute @michielibiza ! I guess you weren't there yet last time we did this, but there was a shitstorm of fuckness sourrounding this whole purchasing stuff: Incorrect documentation, broken test apis, blocked google accounts, lack of control over the actual store account, etc. Plus things have most probably changed considerably since then as well.. I hope you're right about this, but I would advise planning a little more time more this.
The other estimates make sense however :) Thanks

@shawninder haha, thanks for the feedback :)
I think a lot here is about scope: 8 h for the pay plugin should be enough to get something working on a test account (since I already have something that worked beginning of the year, I think it shouldn't take too long to get that up and running). But indeed that doesn't include debugging play store setup in actual products!
So I think we can gat a plugin that purchases from a test app in our own play store account. But indeed there is always some nightmarish version/key twiddling for products where we don't have access to the play store that can take days. This however should be part of the planning of the product itself.
Does that make more sense or do you still think 8h is too little?

@ninjatux So if we pull facebook out of the plugins auth and share what is left in auth and share (as a minimum requirement)? I think a facebook plugin should take about a day. But perhaps the estimates for auth and share would come down after that.

Oh and just to make one more thing clear: All my estimates are for the native side only.  Do we need more ideas on how to limit breakage of dependencies given that none of the dependencies are done?
Furthermore I might need to do a tiny bit more on the android build process so in total it's about 7 to 8 days


fix: added the ability to inform the worker's client that the Http Se…

I ran into a problem when deno-vm is used in a Docker container in which the Deno process does not start, callback in this._httpServer.listen crashes, http server not started. In this case, there is no way for the client code of the application to find out about this and somehow handle the situation.
Messages that are sent via postMessage are stored in _pendingMessages and will never be processed.
My solution is not the best, but I implemented it so as not to change the library interface.

@chudnyi Thanks for your contribution! It looks pretty good, but there's no test in your pull request covering this scenario. Could you add a test case to ensure that the proper errors are thrown and are able to be handled?


React to LastPass RCE.

Tavis Ormandy is back again with a serious vulnerability, but this time in LastPass: https://bugs.chromium.org/p/project-zero/issues/detail?id=1209
This issue is to thoroughly investigate how this security vulnerability affects Gratipay.
Summary
Tavis discovered that 1minsignup/chrome/websiteConnector.js will proxy unauthenticated window messages to the LastPass extension.
window.addEventListener("message", function(e) {
    e.data.fromExtension || chrome.runtime.sendMessage(e.data, function(e) {});
});

Two lines of JavaScript is all it takes to run calc.exe:
win = window.open("https://1min-ui-prod.service.lastpass.com/");
win.postMessage({fromExtension: false, cmd: "openattach", attachkey: "d44479a4ce97554c24399f651ca76899179dec81c854b38ef2389c3185ae8eec", data: "!8uK7g5j8Eq08Nr86mhmMxw==|1dSN0jXZSQ51V1ww9rk4DQ==", mimetype: "other:.bat"}, "*");

Funny enough the following was noted in Tavis' report:

They also said they couldn't get my exploit to work, but I checked my apache access logs and they were using a Mac. Naturally, calc.exe will not appear on a Mac.

LastPass have not released an official security advisory yet, but they believe to have resolved the issue: https://twitter.com/LastPass/status/844176201392504834

Does Gratipay use LastPass?
TL;DR: Yes.
As noted on http://inside.gratipay.com/howto/keep-secrets, some less-sensitive team member passwords are stored in and shared through LastPass.
How will Gratipay deal with this issue?
Gratipay will assume that any password, private key and anything that is stored in a LastPass vault as compromised and will take the necessary precautions to remediate the issue.
We will also follow Tavis' advice:

Additionally, if any corporate intercepting ssl proxy is returning custom error pages for NXDOMAIN then this might still be exploitable, you should test the exploit if you think this might apply to you and contact your administrator if necessary.

Conclusion
This report reveals that Gratipay's team is affected. We will roll all credentials stored in any LastPass vault and ensure that all team members are using the latest version of the LastPass extension.
In the meantime, we will discuss whether we should start using a different password manager. Tavis noted that there is a different vulnerability that affects the Firefox extension that does not appear to be fixed yet:

(Please note, issue 1188 which affects LastPass on firefox is not fixed, and still works)

Link to tweet: https://twitter.com/taviso/status/844251926233579520
We would like to thank Tavis Ormandy for another fantastic report. We might need to set up a reminder for every time Tavis tweets, because you can bet it is going to be big! ;)

EDIT 1: Please note that this had to be fixed server-side and does not require any update to the client.

Apparently Tavis has found another vulnerability in LastPass: https://twitter.com/taviso/status/844312124541186048

LastPass's initial response:

On the night of March 20th, we received a report of an issue in our Chrome <IP_ADDRESS> extension. We immediately investigated and released a server-side workaround within a few hours. The exploit applied to all LastPass clients – Chrome, Firefox, Edge – in which an experimental user onboarding feature was released.
Later on March 21st, another report came in related to Firefox 4.1.35a. In fact, this vulnerability is largely the same as the one reported the prior day, and affecting the 4.x Firefox addon. While this issue would have been addressed by our full fix to follow our workaround, this report was received before this could be released. We issued an update, Firefox 4.1.36a, around 12:15am ET today to specifically address that report.
The fixes are being pushed to all users and most should be updated automatically.
We have no indication that any of the reported vulnerabilities were exploited in the wild, but we’re doing a thorough review at this time to confirm. We will soon provide a more comprehensive summary of the events and what our community needs to know. No password changes are required of users at this time.

Article: LastPass: Security done wrong

And just to add fuel to the fire: http://www.martinvigo.com/design-flaws-lastpass-2fa-implementation/

We've migrated away from LastPass to 1Password: https://github.com/gratipay/inside.gratipay.com/issues/175.


🔧 Use manifest v3

Would be nice to move to manifest version3 -> https://developer.chrome.com/docs/extensions/mv3/intro/mv3-migration/

@johannchopin is it a standard or only a push from chrome ?

Sadly there is no standard for extension's  manifest so most of the browser try to implement the Google Chrome convention (like the actual one). Since the new convention won't really impact this extension and firefox is already trying to implement most changes https://blog.mozilla.org/addons/2019/09/03/mozillas-manifest-v3-faq/ I guess it's just an advantage to update it.


feature: make menu bar icon fit macos theme

Most MacOS icons are background-transparent, so this commit removes the
background of the icon svg and sets the image to be a template (so the
colors can be reversed)
imo this shouldn't be squash merged because the first commit updates the
Charts dependency so it works with xcode 14

@gaoDean How did you make it work?
I don't have any toolbar or something. Just running process in background:

Should I change something in my corne config maybe?
CONFIG_ZMK_KEYBOARD_NAME="Chocofi ZMK"
CONFIG_USB_DEVICE_VID=0x13BA
CONFIG_USB_DEVICE_PID=0x3E7
CONFIG_ZMK_BLE=y
CONFIG_BT=y
CONFIG_ZMK_IDLE_TIMEOUT=30000
CONFIG_ZMK_SLEEP=y
CONFIG_ZMK_IDLE_SLEEP_TIMEOUT=900000
CONFIG_ZMK_MOUSE=y

@xbladesub The app shouldn't depend on the keyboard working, when the keyboard is turned off, the app should still run in the background. Just to confirm cus I don't fully understand you, does the normal main branch work for you? Or is it just this version that doesn't show up?

Mine looks like this

Mine looks like this

I don't see menu bar item. (there is no zmk icon there, but process is running)

Also check the Assets.xassets for the icon, see if it is not there or smth

I'm also on that version

Also are you on the latest Xcode?

Also are you on the latest Xcode?

yes, 14.2 for now

@gaoDean As you can see everything is ok with assets, app is running, but there is no icon in menu bar.

Merged, thanks! I think @xbladesub issues are independent of this change, we might try to address it in a different PR.

@Katona @gaoDean Solved, other apps didn't allow me to see menu bar item


Conditionally use flashes in failure_app

Fixes #3574

:heart: :green_heart: :blue_heart: :yellow_heart: :purple_heart:

@alex88 Would we then be able to get error messages from somewhere else, if flash messages are no longer being populated in flash?

@serene you already get the error In the response afaik

@alex88 Would this be the case even in the event of a remote/JS sign in call?

@serene I think so, I don't remember the case, but I think I was exactly calling the API to sign in via ajax


fix(drawer): Fix trapped focus in the drawer

Issue: #1241
What I did

A filtering function was added to the useFocusLock hook, which filters out invisible elements that were being included in the focusableItems.current array and preventing the focus from being properly locked on the modal window.
Screenshots:

Checklist

[x] changeset has been added
[x] Pull request description is descriptive
[ ] I have made corresponding changes to the documentation
[x] New and existing unit tests pass locally with my changes
[ ] I have added tests that prove my fix is effective or that my feature works

How to test

Go to our documentation site.
Navigate to Components > Drawer
Click "Show Drawer" under "Navigation Example"
Tab through the opened Drawer

This solution also addresses #970


ImportsAnalyzer -> ImportSyntaxReader. Also create StandardAnalyzer

Migrates the import analyzer underlying tooling down into the Visitor module. Deletes the remainder of ImportAnalyzer in favor of a new StandardAnalyzer

I don't get the CodeCov complaint... the method it says we aren't hitting, we are hitting (and I have the breakpoints to prove it!)

I don't get the CodeCov complaint... the method it says we aren't hitting, we are hitting (and I have the breakpoints to prove it!)


Add S3 tags and update github description

I noticed there are no S3 tags and the descriptions says "Standalone alternative to Kafka JDBC Connector"
I propose to update it to "Standalone alternative to Kafka JDBC Connector and S3 Kafka Connector"

Amended description, left it more general as we intend provide more than just doobie and s3

Awesome!


Windows - Script error at start up

Script error at start up

This is a problem with older versions of IE than 11. Since Wails on windows rely on IE libraries. With Wails 2.0 they this will be moved as they're moving to a different library.


Installation Problem with NVM

Probably a low priority, but I ran into an issue when installing in conjunction with nvm.
The install seems to be using the headers from my system node<EMAIL_ADDRESS>) instead of my nvm selected version (node@v4.0.0). The obvious workaround is just to upgrade the system level node.
Just thought I'd go ahead and document this, since I ran into it.
waylonflinn@cecil:~/Development/js-vec-collection/vectorious$ npm install
-
><EMAIL_ADDRESS>install /home/waylonflinn/Development/js-vec-collection/vectorious/node_modules/nblas
> node-gyp configure && node-gyp rebuild

make: Entering directory `/home/waylonflinn/Development/js-vec-collection/vectorious/node_modules/nblas/build'
  CXX(target) Release/obj.target/addon/addon.o
  CXX(target) Release/obj.target/addon/functions/dot.o
../functions/dot.cc:3:11: error: ‘v8::Float64Array’ has not been declared
 using v8::Float64Array;
           ^
../functions/dot.cc:4:11: error: ‘v8::Float32Array’ has not been declared
 using v8::Float32Array;
           ^
In file included from /home/waylonflinn/.node-gyp/0.10.25/src/node_buffer.h:28:0,
                 from ../node_modules/nan/nan.h:46,
                 from ../functions/dot.h:4,
                 from ../functions/dot.cc:1:
../functions/dot.cc: In function ‘Nan::NAN_METHOD_RETURN_TYPE ddot(Nan::NAN_METHOD_ARGS_TYPE)’:
../functions/dot.cc:13:19: error: ‘class v8::Value’ has no member named ‘IsFloat64Array’
   assert(info[1]->IsFloat64Array());
                   ^
../functions/dot.cc:14:19: error: ‘class v8::Value’ has no member named ‘IsFloat64Array’
   assert(info[3]->IsFloat64Array());
                   ^
../functions/dot.cc:16:29: error: ‘Float64Array’ was not declared in this scope
   void *x_data = info[1].As<Float64Array>()->Buffer()->GetContents().Data(),
                             ^
../functions/dot.cc:16:43: error: no matching function for call to ‘v8::Local<v8::Value>::As()’
   void *x_data = info[1].As<Float64Array>()->Buffer()->GetContents().Data(),
                                           ^
../functions/dot.cc:16:43: note: candidate is:
In file included from /home/waylonflinn/.node-gyp/0.10.25/src/node.h:62:0,
                 from ../node_modules/nan/nan.h:45,
                 from ../functions/dot.h:4,
                 from ../functions/dot.cc:1:
/home/waylonflinn/.node-gyp/0.10.25/deps/v8/include/v8.h:291:38: note: template<class S> v8::Local<S> v8::Local<T>::As() [with S = S; T = v8::Value]
   template <class S> inline Local<S> As() {
                                      ^
/home/waylonflinn/.node-gyp/0.10.25/deps/v8/include/v8.h:291:38: note:   template argument deduction/substitution failed:
../functions/dot.cc:16:43: error: template argument 1 is invalid
   void *x_data = info[1].As<Float64Array>()->Buffer()->GetContents().Data(),
                                           ^
../functions/dot.cc:20:41: error: ‘y_data’ was not declared in this scope
          *y = reinterpret_cast<double*>(y_data);
                                         ^
In file included from /home/waylonflinn/.node-gyp/0.10.25/src/node_buffer.h:28:0,
                 from ../node_modules/nan/nan.h:46,
                 from ../functions/dot.h:4,
                 from ../functions/dot.cc:1:
../functions/dot.cc: In function ‘Nan::NAN_METHOD_RETURN_TYPE sdot(Nan::NAN_METHOD_ARGS_TYPE)’:
../functions/dot.cc:32:19: error: ‘class v8::Value’ has no member named ‘IsFloat32Array’
   assert(info[1]->IsFloat32Array());
                   ^
../functions/dot.cc:33:19: error: ‘class v8::Value’ has no member named ‘IsFloat32Array’
   assert(info[3]->IsFloat32Array());
                   ^
../functions/dot.cc:35:29: error: ‘Float32Array’ was not declared in this scope
   void *x_data = info[1].As<Float32Array>()->Buffer()->GetContents().Data(),
                             ^
../functions/dot.cc:35:43: error: no matching function for call to ‘v8::Local<v8::Value>::As()’
   void *x_data = info[1].As<Float32Array>()->Buffer()->GetContents().Data(),
                                           ^
../functions/dot.cc:35:43: note: candidate is:
In file included from /home/waylonflinn/.node-gyp/0.10.25/src/node.h:62:0,
                 from ../node_modules/nan/nan.h:45,
                 from ../functions/dot.h:4,
                 from ../functions/dot.cc:1:
/home/waylonflinn/.node-gyp/0.10.25/deps/v8/include/v8.h:291:38: note: template<class S> v8::Local<S> v8::Local<T>::As() [with S = S; T = v8::Value]
   template <class S> inline Local<S> As() {
                                      ^
/home/waylonflinn/.node-gyp/0.10.25/deps/v8/include/v8.h:291:38: note:   template argument deduction/substitution failed:
../functions/dot.cc:35:43: error: template argument 1 is invalid
   void *x_data = info[1].As<Float32Array>()->Buffer()->GetContents().Data(),
                                           ^
../functions/dot.cc:39:39: error: ‘y_data’ was not declared in this scope
         *y = reinterpret_cast<float*>(y_data);
                                       ^
make: *** [Release/obj.target/addon/functions/dot.o] Error 1
make: Leaving directory `/home/waylonflinn/Development/js-vec-collection/vectorious/node_modules/nblas/build'
gyp ERR! build error
gyp ERR! stack Error: `make` failed with exit code: 2
gyp ERR! stack     at ChildProcess.onExit (/usr/local/lib/node_modules/npm/node_modules/node-gyp/lib/build.js:269:23)
gyp ERR! stack     at ChildProcess.EventEmitter.emit (events.js:98:17)
gyp ERR! stack     at Process.ChildProcess._handle.onexit (child_process.js:797:12)
gyp ERR! System Linux 3.13.0-46-generic
gyp ERR! command "node" "/usr/local/lib/node_modules/npm/node_modules/node-gyp/bin/node-gyp.js" "rebuild"
gyp ERR! cwd /home/waylonflinn/Development/js-vec-collection/vectorious/node_modules/nblas
gyp ERR! node -v v0.10.25
gyp ERR! node-gyp -v v1.0.3
gyp ERR! not ok
npm ERR! Linux 3.13.0-46-generic
npm ERR! argv "node" "/usr/local/bin/npm" "install"
npm ERR! node v0.10.25
npm ERR! npm  v2.7.4
npm ERR! code ELIFECYCLE

npm ERR<EMAIL_ADDRESS>install: `node-gyp configure && node-gyp rebuild`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>install script 'node-gyp configure && node-gyp rebuild'.
npm ERR! This is most likely a problem with the nblas package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     node-gyp configure && node-gyp rebuild
npm ERR! You can get their info via:
npm ERR!     npm owner ls nblas
npm ERR! There is likely additional logging output above.

npm ERR! Please include the following file with any support request:
npm ERR!     /home/waylonflinn/Development/js-vec-collection/vectorious/npm-debug.log

I use n for node version control and no problems. Could it be nvm-specific?

It probably is nvm specific. I'll check out n, and see what happens.

Think I can close this?

This looks like the relevant nvm issue:
https://github.com/creationix/nvm/issues/397
close away.


show explicit -t  in 'bosh task ' error command

Previously, error messages from a failed bosh command said to run bosh task <num> --debug. Now it includes the explicit target: bosh -t <target> task <num> --debug

Closing - ruby CLI is feature frozen in favor of gocli, so I'm going to close this. Since bosh commands require the environment/deployment options, this shouldn't be an issue.


backport: Merge bitcoin#20816,20844,20584,14501,18669

waiting for https://github.com/dashpay/dash/pull/5490 to merge

-std::vector<LogCategory> LogCategoriesList();
+std::vector<LogCategory> LogCategoriesList() const;
-std::string LogCategoriesString()
+std::string LogCategoriesString() const

from https://github.com/bitcoin/bitcoin/pull/20584 for https://github.com/bitcoin/bitcoin/pull/18669
@vijaydasmp

Hello @knst , I had missed const after std::vector BCLog::Logger::LogCategoriesList(), added that and pushed changes , please have a look


Unnecessary Flutter / Dart code inteli-sense or suggestions when adding documentation or TODOs.

Hi Team,
Whenever I put a fullstop/period (.) in my code documentation or todo comments, android studio unnecessarily showing Dart code suggestions. It is very frustrating. I need to press escape key immediately after typing Period sign (.) every time.
Attached the screenshot for your reference. I was coding for a flutter application on Mac when taking this screen shot.

Additional Notes: I reported this issue in Google Issues tracker for Android Studio. But, they asked me to report this in Flutter. Here is the Google issues tracker Link : https://issuetracker.google.com/125088418

cc @devoncarew

Issue moved to flutter/flutter-intellij #3231 via ZenHub


where is a desktop GUI?

where is a desktop GUI and how to install it?

uh… probably somewhere in the future :) I haven't worked on that yet


OctopOS Prime Mascot Design-6

5 costume designs for OctopOS Prime. The concept is completely left to your imagination.
The design size of their costumes must be 1080x1080.

OctopOS Prime Mascot Design


Turn Variant const again

When the new version of Mix was released, it came with break changes, one of them is the class Variant hasn't a constant constructor anymore, what don't allow to use the new format of enums with properties.
In this PR I fix this and Variant constructor become constant again.
Changing this:
enum SomaButtonAppearance {
  primary(Variant("primary")),
  secondary(Variant("secondary")),
  ghost(Variant("ghost"));

  const SomaButtonSize(this.variant);

  final Variant variant;
}

To
enum SomaButtonAppearance{
  primary,
  secondary,
  ghost;

  Variant get variant {
    switch (this) {
      case primary:
        return Variant("primary");
      case secondary:
        return Variant("secondary");
      case ghost:
        return Variant("ghost");
    }
  }
}

@all-contributors please add @tilucasoli for code


Add dynamic driver loading function

Update the current driver adaptation and usage method, from fixed
use of Hisilicon device driver to automatic registration of the
driver according to the algorithm.
When the algorithm API layer uses the driver, it is no longer
bound to the fixed device driver, but dynamically obtained
and stored according to the algorithm query to use.
Update the driver and API layer of the zip module, keep the
function of the init interface unchanged, update the implementation
of the init2 interface and match the dynamic loading function.

can we pick different algorithms by packet size, like a big packet chooses hardware and small packet select software?


DEC-327: Auth check v1 items update

Why: Permissions check for editing the item was temporarily disabled, needed to be restored.
How: For the most part this was the same as any other permissions check, but we determined that there needed to be additional info in the response so that the front end could better report why the user received the response that it did. The ItemMetaDataForm already had mechanisms for rendering errors encountered with item validation. I expanded this to also handle generic server error messages and changed APIController to provide a permission error message when the user is forbidden to make the change.

:+1:

:+1:


Annotations disable not working

I am using 419 version of developer version YTC, but it happens on stable version too. Even if i have unticked "enable annotations" option annotation appears.

Duplicate/related to #432?


Add Dockutil Support

This adds support for dockutil to auto add apps to the dock, note the dock will restart when set to yes.
to use set ADDTODOCK to yes or pass ADDTODOCK=yes as an argument

One more question: what happens if you have ADD_TO_DOCK enabled and run installomator multiple times? will it add multiple icons for the same app?

If ADD_TO_DOCK is enabled but the item already exists dockutil fails silently and does not modify the dock.
I don't see much use trying to log that outcome though as the end result of the icon being in the dock is still true.
I will modify the help text to reflect this.

I would think that this should be moved to a separate script. That way we can keep Installomator free from 3. party software requirements.
Actually this suggestion was an inspiration for me to add this functionality to the MDM scripts.


Rename lplr to lpl

The lplr name is to be replaced by lpl. Including the "r" at the end of a register has fallen out of favor, since the "r" follows from it being a CSR. This CSR does not need to be unprivileged.

PR #45 to address the issue.


I want to zero to come to most bottom so don't have any gap

I want the zero to come to bottom so I don't have a gap here how can I do it

Can't find a way

You can use the scaleMargins option within the PriceScaleOptions to adjust the margin placed above and below the highest and lowest points on the chart.


Implement betweeness centrality

Welcome to Codecov :tada:
Once merged to your default branch, Codecov will compare your coverage reports and display the results in this comment.
Thanks for integrating Codecov - We've got you covered :open_umbrella:


(Android) Bug When Swipe Pages

Which OS ?
Android
Version
Which versions are you using:

react-native-swiper v?
1.6.0-nightly.3

react-native v0.?.?
0.59.9

Actual behaviour
I write an example
my swiper ...
<View style={styles.container}>
                <Swiper
                    loop={false}
                    showsPagination={false}
                    ref={ref => (this._swiper = ref)}
                    onTouchEnd={() => (this.inMotion = false)}
                    onTouchStart={() => (this.inMotion = true)}
                    onMomentumScrollEnd={() => (this.inMotion = false)}
                >
                    <Text> Page 1 </Text>
                    <Text>Page 2 </Text>
                </Swiper>
</View>

When I swipe to page 2 and then back to page 1 it shows me page 2 but the index is still page 1.
I enabled theloop = {true}and this bug stopped, but when I disable the loop it happens = / plz can help me?
iOs still normal it doesn't happen!
reproduction

i have same issue :(

We can temporary use 1.5.14 version that works fine on android

I got same issue here with 1.6.0-nightly.3. @danielgssp how did you solve this issue? Downgrading version back to 1.5.14 as yurkagon said?


fix: rock from fork

Applicable spec: N/A
Overview
GHCR does not allow rocks to be upload to ghcr.io from forks due to org permission settings (write).
This PR mitigates the use of upload-artefacts action on PRs from forks by using actions/cache, which do not require org write permissions.
The release charm library depends on upload-artefacts. This PR explicitly creates a workaround to not use upload-artifacts action, hence the publish_charm workflow will fail.
We should consider whether this should be refactored or the workaround should be accepted.
Passing workflow run from fork using this workflow example.
TODO: Check whether the existing workflow works.
Rationale
To allow PRs from forks to run.
Workflow Changes

Checklist

[x] The contributing guide was applied
[x] The PR is tagged with appropriate label (urgent, trivial, complex)

Closing in favor of #285


Update delete-columns-from-a-table.md

Good day team,

I do not like the use of "DELETE Columns" in the page name, when it should be "DROP Columns". I recomend to change it, but I do nothing regarding this point.

I edited the first comment since it is not accurate

When you drop a column you do not delete the data from the disk, but only change the metadata of the table structure. The Data is still there and can be restored by reading the binary data from the disk theoretically (I show it is lectures on "SQL Internals" from time time). In fact unless you do another action like REBUILD, then you do not reclaim the space from the row structure.
I gave full explanation and demo in this thread in the forum:
https://social.msdn.microsoft.com/Forums/en-US/61f1666a-f233-4572-b647-97c67846e2ab/some-guidelines-for-update-largetable-set-c1-0?forum=transactsql
Therefore, I removed the statement "This action cannot be undone", and the "from the database" since it is still in the database file but it's not in the table (the virtual element we call table).

@pituach : Thanks for your contribution! The author, @stevestein, has been notified to review your proposed change.

@pituach - Thanks for the feedback. For the title, drop or delete columns are both appropriate and searching for either returns this article. The title is related to the filename and URL so we're going to leave it as is for now.
'This action cannot be undone' is ok to remove.  Thanks for the update!
#sign-off - This PR is ok to merge.


Document needs to be updated to reflect Modern Auth

[Enter feedback here]
This command in the article--
Connect-SPOService -Url https://contoso-admin.sharepoint.com -Credential<EMAIL_ADDRESS>--should become--
Connect-SPOService -Url https://contoso-admin.sharepoint.com
SPOService will no longer (by default) allow pre-Modern Auth.

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

ID: 05b23d72-33ed-a6a1-a442-a5e5cf039f9f
Version Independent ID: 4247cf37-3667-2007-7251-46ed80f58ef5
Content: Get started with the SharePoint Online Management Shell
Content Source: sharepoint/docs-conceptual/sharepoint-online/connect-sharepoint-online.md
Service: sharepoint-powershell
GitHub Login: @pamgreen-msft
Microsoft Alias: pamgreen

Hello @JRVcr how are you? The

To connect with multifactor authentication (MFA)

section already uses the method you are proposing.
Thanks!!

@get-itips Thank you very much for the contribution and sharing this explanation. @JRVcr Hope this comment is helpful for you. If you see a documentation update is required, please feel free to open an issue for the same. We proceed here to close it. Thanks for taking out some time to open the issue. Appreciate and encourage you to do the same in future also.


Compile errors with TS 2.4

RxJS version: 5.4.1
Importing rxjs in TS 2.4 gives this error:
node_modules/@reactivex/rxjs/dist/cjs/Subject.d.ts(16,22): error TS2415: Class 'Subject<T>' incorrectly extends base class 'Observable<T>'.
  Types of property 'lift' are incompatible.
    Type '<R>(operator: Operator<T, R>) => Observable<T>' is not assignable to type '<R>(operator: Operator<T, R>) => Observable<R>'.
      Type 'Observable<T>' is not assignable to type 'Observable<R>'.
        Type 'T' is not assignable to type 'R'.
node_modules/@reactivex/rxjs/dist/cjs/observable/dom/WebSocketSubject.d.ts(24,22): error TS2415: Class 'WebSocketSubject<T>' incorrectly extends base class 'AnonymousSubject<T>'.
  Types of property 'lift' are incompatible.
    Type '<R>(operator: Operator<T, R>) => WebSocketSubject<R>' is not assignable to type '<R>(operator: Operator<T, R>) => Observable<T>'.
      Type 'WebSocketSubject<R>' is not assignable to type 'Observable<T>'.
        Types of property 'operator' are incompatible.
          Type 'Operator<any, R>' is not assignable to type 'Operator<any, T>'.
            Type 'R' is not assignable to type 'T'.

dupe of #2699 , #2539, #2700 . #2701 #2702 #2703 https://github.com/ReactiveX/rxjs/issues/2704 https://github.com/ReactiveX/rxjs/issues/2705 https://github.com/ReactiveX/rxjs/issues/2706

Sorry, only looked through open issues

One issue is opened for tracking purpose. Rest of them are closed by dupe.


Check CI status in build release script

@keybase/react-hackers
@oconnor663 and I tested the Linux change.  @gabriel, please can you test the OS X one?

dm me to test this after resolve comment.. i'm slow to read email

Also instead of doing this in node.. we could add command to go release tool and use this travis library: https://github.com/Ableton/go-travis
But I'm just a node hater... so it's ok :)

But we use more CI services than just Travis.

the npm install is only in this packaging dir tree... its for appdmg and electron-package only used in building... This is not npm install for keybase/client/desktop :)

appdmg fails to install on Linux (EBADPLATFORM), so we can't put the new module in that package.json file.  I'd like to stick with the temp file stuff, I think cleaning up the install afterwards is the right idea.

lg2m whichever way you decide is ok with me


core: WIP factory-builder

@chriskilding I've had some time to work on a solution for the factory builder implementation. You might rebase your PR on this branch as it does most of the heavy lifting, but is still a bit rough around the edges. If you have some time to polish the new interfaces and update the examples, that'd be awesome.

Thanks! I've now incorporated these changes into my PR.
We should keep this PR around while we get the other one over the line, after that we can close this one.


⚠ SceneTime has degraded performance

In 90827ea, SceneTime ($ST) experienced degraded performance:

HTTP code: 200
Response time: 9453 ms

Resolved: SceneTime performance has improved in 35519f5 after 6 minutes.