More complete & correct usage example

Blindly copy-pasting the one-liner in the earlier example doesn't work :)

@saurabhnanda thanks! Merged


Update beam_fn_api.proto to specify allowed split points.

allowed_split_points will allow the runner to specify indices where the SDK may split.
Please add a meaningful description for your change here

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Choose reviewer(s) and mention them in a comment (R: @username).
[ ] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

Post-Commit Tests Status (on master branch)

Lang
SDK
Apex
Dataflow
Flink
Gearpump
Samza
Spark

Go

---
---
---
---
---
---

Java

Python

---

---
---
---

Pre-Commit Tests Status (on master branch)

---
Java
Python
Go
Website

Non-portable

Portable
---

---
---

See .test-infra/jenkins/README for trigger phrase, status and link of all Jenkins jobs.

@angoenka
@robertwb

@angoenka @robertwb @lostluck

Run Python PreCommit

Please also revert the release-2.11.0 branch modifications (will require another PR).

Revert PR for release-2.11.0 branch https://github.com/apache/beam/pull/8475


Increase topology node size & provide alternative solution for interior icons

Determine base size for topology nodes.
Provide alternative solution for interior icons.  Alissa will provide more information as to what th

ese are indicating.
FYI @abonas

There are 2 cases in which currently the icon is presented inside a graph node:

traffic flows to/from an object in another namespace (if that namespace is not selected in the namespace selector dropdown on top of the graph)
traffic flows to/from an object in another namespace and that namespace is not accessible to the user

cc @lucasponce @jmazzitelli @jshaughn

Here's the initial stab at the Kiali graph using PF4 topology designs: https://marvelapp.com/41jgfd4/screen/59366802

cc @pilhuhn @jotak @jshaughn @aljesusg @lucasponce @josejulio @mwringe   have a look at the new design :)
@serenamarie125 this is really nice.
I wonder regarding the following text:
"Kiali currently uses edge color only to denote transfer error rate, this is a no-no for accessibility (from what I can tell from the legend), so I utilize the dashed line concept I came up with earlier here.
The idea is that... (kiali)
green + small dashes + fast animation = good (0.1% error)
orange + medium dashes + slow animation = ok (0.1-20% error)
red + long dashes + no animation = bad (20%+ error)
"
I completely agree regarding the needed change in dashes not to count only on the color.
But I'm not sure I understand regarding the animation speed. Traffic animation is sometimes switched on/off, it could be that there are errors among "normal" traffic, so unless I understand this suggestion the wrong way, it will be difficult and strange to play with the speed for single errors.

Yes, that looks very nice.
Do we have an idea how this could look when we have a much larger mesh?

+1, it looks very nice!
About traffic animation, adding to what Alissa says, it's also to be noted that today animation speed varies according to the response time.
It might not be an extremely important feature, but still, I believe it gives a positive feeling about the graph when we see it all animated in function of criterias that directly relate to the data. (I know it's very subjective)

It looks nice.
About the grouping shapes:
Cytoscape uses rectangle by default for grouping; I think we can hide it and draw our custom shape instead, but it needs to be an inscribed polygon.

I think we can only use rectangle, round-rectangle, bottom-round-rectangle, or cut-rectangle for the compound nodes.
From: http://js.cytoscape.org/#style/node-body
"Only *rectangle shapes are supported by compounds, because the dimensions of a compound are defined by the bounding box of the children"

True, but:

You could do something visually similar by hiding the node body and using an ellipse as an SVG background image.

Still not really sure if it would work as good.
https://github.com/cytoscape/cytoscape.js/issues/2129#issuecomment-398042779

I like the condensed toolbar line although note that it is currently sized to not wrap on lower-res screens (I can't quite recall what resolution we targeted).  We should maintain that goal.
I don't think we can do any of the html-based labeling without some additional support in Cy.  Matt tried and it didn't work out.  We are currently limited to a text of one color with icon fonts.
The form-fitting app box shape is likely challenging (read: not doable).
Perhaps we should offer node-size options (default: small or maybe use some sort of "smart" default).
Animation speed should continue to reflect response times imo.
If we do go with dashes (and we need to check cy's options here) I'd suggest solid lines for edges without appreciable error.  That will make dashed lines stand out all the more. I'd still like to see colors applied. As a note, edge labels show error rate.  And also graph find can highlight edges with error rates.  One more idea, we could offer a display option that displays error rates only.

I like the condensed toolbar line although note that it is currently sized to not wrap on lower-res screens (I can't quite recall what resolution we targeted).  We should maintain that goal.
I don't think we can do any of the html-based labeling without some additional support in Cy.  Matt tried and it didn't work out.  We are currently limited to a text of one color with icon fonts.
The form-fitting app box shape is likely challenging (read: not doable).

I am wondering whether form fitting is critical in the first place, compared to current options. I mean - was there UX research done that indicates that form fitting is preferred by users over the current state?
@serenamarie125

Perhaps we should offer node-size options (default: small or maybe use some sort of "smart" default).
Animation speed should continue to reflect response times imo.
If we do go with dashes (and we need to check cy's options here) I'd suggest solid lines for edges without appreciable error.  That will make dashed lines stand out all the more. I'd still like to see colors applied. As a note, edge labels show error rate.

those labels also show percentage of requests

And also graph find can highlight edges with error rates.  One more idea, we could offer a display option that displays error rates only.

I don't think we can do any of the html-based labeling without some additional support in Cy. Matt tried and it didn't work out. We are currently limited to a text of one color with icon fonts.

Yeah, the whole problem here is that the graph is done in the canvas and the labels are done using canvas text (which doesn't support much styling). If we want to do more styling of labels then there are libraries that will put html on top of the canvas.
But these html labels are not part of the graph itself and the size of these html labels are not used when the graph layout is being calculated. So the graph isn't properly spaced for the labels anymore (ie labels overlap with the app boxes, labels overlap with other nodes since they are not spaced far enough apart, etc).
There was also an issue with getting events to work properly with these html labels. So if you were to hover over a node or select a node, the html labels didn't get notified.
There might be a way to get around these technical issues, but at the time it started to become one hack after another to get it to work and we decided back then that it might not have been worth the effort.
Even if we could get the labels to work properly, there is also the other styling of the graph that might not be possible (eg the drop shadows on the nodes, the rounded triangle nodes, the non-rectangular bounding boxes around grouped nodes, etc)

One of the things we are thinking about from a PF4 topology perspective is that when hovering over a node, the node label will be enlarged so that it's readable.  @mceledonia do we have any updates on that ?

In Kiali today we do something like this already when hovering.  We highlight the hovered object and it's "neighborhood":

@serenamarie125 is there any more to do on this one?


Fix kill path in systemd's unit file

Totally my fault for this one, I should have tested in the Vagrant box (that I will definitely do from now on!).
Sorry for the trouble. (Should close #29)

...huh, I didn't catch this. How is it that this has been working all morning as I've deployed stuff?

@PeterFaiman Totally agreed on the Unicorn bit in the config to support the reload. I read a bit some exchanges between systemd devs and unicorn devs and it seems that the cohesion of both solutions to have a perfect reload is not yet implemented.
To be honest, I don't have any idea that could explain why the reloads worked for you @pushcx ...
The proposed changed of @PeterFaiman seem great to me, so I'll create another PR (not to pollute this one that solves the linked issue), with his proposed changes.


Sorting of union types is non-deterministic

This issue is related to https://github.com/TypeStrong/typedoc/issues/2502 but applies to non-literal types. Sorting of union types is non-deterministic.
Steps to reproduce the bug
export type Type1 = 'type1';
export type Type2 = { prop1: string; prop2: boolean };

/**
 * Some Interface
 */
export interface SomeInterface {
  someUnionType: Type1 | Type2;
}

Expected

Actual
Most of the times it shows

But sometimes it shows

(This is causing our CI pipeline to fail as we have a job to ensure the API Doc changes are included in pull requests)
Environment

Typedoc version: v0.26.10

I can't reproduce this. TypeDoc is using the the type as it is reported from TypeScript, which in the ~30 times I've run it, always puts "type1" second in the union.

You can work around this by forcing Type1 to introduce a new type:
export type Type1 = 'type1' & {};


⚠ Bofrost.lu has degraded performance

In f7044c9, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 1480 ms

Resolved: Bofrost.lu performance has improved in 6b231d5 after 6 minutes.


DPL-439 QC data

Closes #
Changes proposed in this pull request:

...

Closed as not required at the moment.


Collect and enrich commits and projects from gitlab

Why:
We need to collect all github issues and projects so we can create metrics in grafana for this data
What:
Commits are stored in psql with stats of added lines etc.
Projects are stored based on project details API request from gitlab
How:

Fetch from the gitlab API
Save in mongo
Enrich by fetching from mongo
format for PSQL
Save in PSQL

@UltimateBeaver feel free to review this after the fact and I will make a new PR to fix the comments you make here


correct 10. multiple dispatch exercise 9.1 from "prints" to "returns"

I was a bit confused when doing the 10. Multiple dispatch exercise 9.1.
I think the wording should be changed from prints to returns as the later assertion `@assert foo(true) == "foo with one boolean!"``will not work if the function is not returning?
I understand the Jupyter notebook will show it's output and that is what is possibly meant by "prints".

Indeed, thanks! (And sorry for the delay)


No learning rate

Hi sir, could you please explain how can I set a learning rate for this code. I have been scrutinizing the code but was unable to find any parameter for this. If you haven't implemented this how how should I do that. I want this because after some epochs I get 2 - 4 loss for each batch and the overall mean of this is not reduced. I want to set a learning rate scheduler in order to make this more converged. Thank you in advance

HI , there is  a default learning rate for Adam Optimizer see its syntax its 0.01 , if you look into the earlier versions of SimplHtr decaying learning rate was used ( for 1000 epochs its 0.01 and later its 0.001 ) you can manually pass the learning rate parameter as well in the AdamOptimizer . Is this what are you asking ???

Thank dear Harald. So sorry for being late in the thread. You helped me a lot.


Unable to initiate a push notification from iOS 10.1.1 device

I have a chat feature built in my ionic application using firebase as my backend. I'm using node-gcm and node-apn on my node server hosted on Heroku.
Expected Behaviour
Send and receive push notifications between iOS and Android builds of my application when user A sends a chat message to user B, or vice versa.
Actual Behaviour

Android users can send and receive notifications.
iOS users receive notifications from Android users.
Android users do not receive notifications from iOS users
iOS users may or may not receive notifications from other iOS users. (I only have one iOS device I am able to test with.)

Platform and Version (eg. Android 5.0 or iOS 9.2.1)
Actual iOS 10.1.1
Build targets tried 8, 9 and 10
(Android) What device vendor (e.g. Samsung, HTC, Sony...)
iphone 5
Oneplus One
Nexus 5
Cordova CLI version and cordova platform version
cordova --version                          6.4
cordova platform version android           6.1
cordova platform version ios           4.3

Plugin version
cordova plugin version | grep phonegap-plugin-push   1.9.0

Sample Push Data Payload
    $scope.sendMessage = function() {
        // Get the users auth jwt to verify them on the node server
        firebase.auth().currentUser.getToken(true)
        .then(function(userToken){
            $http({
                method: 'POST',
                url:'MY_HEROKU_URL',
                data:{
                    reg_id: pushRegId,
                    token: userToken,
                    message: $scope.IM.textMessage,
                    sender_name: $scope.user.name,
                    topic: '/topics/' + $state.params.roomId,
                    state: 'app.room',
                    roomId: $state.params.roomId,
                    sender_imgURL: $scope.user.pic,
                    chatters: chatters,
                    platform:  platform
                }
            })
            .then(function(res){
                // All chats are getting sent and received between iOS and Android builds
                Chats.send($scope.user, $scope.IM.textMessage);
                $scope.IM.textMessage = "";
            })
            .catch(function(err){
                //Haven't hit this debugger so no errors here
                debugger;
            });
        });
    };

Sample Code that illustrates the problem
app.js
$ionicPlatform.ready(function() {
    $cordovaPushV5.initialize({
      android: {
          senderID: "MY_SENDER_ID",
      },
      ios: {
          senderID: "MY_SENDER_ID",
          alert: "true",
          badge: "true",
          sound: "true",
          "gcmSandbox": "true"
      }
    })
    .then(function(push){
      $cordovaPushV5.Push = push;
      $cordovaPushV5.onNotification();
      $cordovaPushV5.onError();
      $cordovaPushV5.register()
      .then(function(regId){
        var oldRegId = $localstorage.get('registrationId');
        if (oldRegId !== regId) {
          // Save new registration ID
          $localstorage.set('registrationId', regId);
          // Post registrationId to your app server as the value has changed
        }
      });
    });

Post route on node router
var firebase = require('firebase');
var bodyParser = require('body-parser');
var jsonParser = bodyParser.json();
var gcm = require('node-gcm');
var sender = new gcm.Sender('<Server key (auto created by Google Service)>');
var apn = require('apn');
var apnProvider = new apn.Provider({
  token: {
    key:"apns.p8", // in root of my node app
    keyId: "MY_APNS_KEY",
    teamId: "MY_TEAM_ID"
  },
  production: false
});

router.post('/', jsonParser, function(req, res){
  if(req.body.platform === "Android"){
    var message = new gcm.Message({
      priority: 'high',
      notification: {
        click_action: "FCM_PLUGIN_ACTIVITY",
        title: req.body.sender_name,
        body: req.body.message
      },
      data: {
        state: req.body.state,
        roomId: req.body.roomId,
        sender_imgUrl: req.body.sender_imgURL
      }
    });
    sender.send(message, { topic: req.body.topic }, function(err, response){
      if(err){
        res.status(500).json({ error: err });
      } else {
        res.status(200).json({ response: 'Push notification sent' });
      }
    });
  } else if(req.body.platform === "iOS"){
    var notification = new apn.Notification({
      aps:{
        priority: 10, // tried with and without priority set
        contentAvailable: true, //I've tried both true and 1 here
        alert: {
          title: req.body.sender_name,
          body: req.body.message
        },
        topic: req.body.topic,
        state: req.body.state,
        roomId: req.body.roomId,
        sender_imgUrl: req.body.sender_imgURL,
        token: req.body.token,
        sound: "default"
      }
    });
    apnProvider.send(notification, req.body.reg_id)
    .then(function(sendResolve){
      if(sendResolve.failed.rejected){
        res.status(500).json({ error: 'The send failed with status ' + sendResolve.failed.rejected.status + ". Reason: " + sendResolve.failed.rejected.response.reason });
      } else if(sendResolve.failed.error){
        res.status(500).json({ error: sendResolve.failed.error});
      } else {
        res.status(200).json({ response: 'Notifications sent to: ' + sendResolve.sent  });
      }
      apnProvider.shutdown();
    });
  }
});

Logs taken while reproducing problem
200 status code from heroku logs
heroku[router]: at=info method=POST path="/" host=<MY_HEROKU_URL>request_id=xxxxxxxx-xxxx-xxxx-xxxx-xxxxxxxxxxxx fwd="<FWD_IP>" dyno=web.1 connect=1ms service=350ms status=200 bytes=240
xcode build logs
2016-11-16 12:14:43.766363 Apparent[269:19289] Apache Cordova native platform version 4.3.0 is starting.
2016-11-16 12:14:43.770190 Apparent[269:19289] Multi-tasking -> Device: YES, App: YES
2016-11-16 12:14:44.317036 Apparent[269:19289] Using UIWebView
2016-11-16 12:14:44.324867 Apparent[269:19289] [CDVTimer][handleopenurl] 0.504017ms
2016-11-16 12:14:44.334965 Apparent[269:19289] Unlimited access to network resources
2016-11-16 12:14:44.335570 Apparent[269:19289] [CDVTimer][intentandnavigationfilter] 10.241985ms
2016-11-16 12:14:44.336608 Apparent[269:19289] [CDVTimer][gesturehandler] 0.694036ms
2016-11-16 12:14:44.408310 Apparent[269:19289] [CDVTimer][file] 71.398973ms
2016-11-16 12:14:44.443287 Apparent[269:19289] [CDVTimer][splashscreen] 34.744978ms
2016-11-16 12:14:44.477346 Apparent[269:19289] [CDVTimer][statusbar] 33.713996ms
2016-11-16 12:14:44.481371 Apparent[269:19289] [CDVTimer][keyboard] 3.619969ms
2016-11-16 12:14:44.481589 Apparent[269:19289] [CDVTimer][TotalPluginStartup] 157.431960ms
2016-11-16 12:14:45.274793 Apparent[269:19289] createNotificationChecker
2016-11-16 12:14:45.274938 Apparent[269:19289] not coldstart
2016-11-16 12:14:45.280931 Apparent[269:19289] active
2016-11-16 12:14:45.281472 Apparent[269:19289] PushPlugin skip clear badge
2016-11-16 12:14:45.611214 Apparent[269:19289] Resetting plugins due to page load.
2016-11-16 12:14:51.846240 Apparent[269:19391] WF: === Starting WebFilter logging for process Apparent
2016-11-16 12:14:51.846374 Apparent[269:19391] WF: _userSettingsForUser mobile: {
    filterBlacklist =     (
    );
    filterWhitelist =     (
    );
    restrictWeb = 1;
    useContentFilter = 0;
    useContentFilterOverrides = 0;
    whitelistEnabled = 0;
}
2016-11-16 12:14:51.846654 Apparent[269:19391] WF: _WebFilterIsActive returning: NO
2016-11-16 12:14:52.291093 Apparent[269:19289] Finished load of: file:///var/containers/Bundle/Application/xxxxxxxx-xxxx-xxxx-xxxx-xxxxxxxxxxxx/Apparent.app/www/index.html#/login
2016-11-16 12:14:52.618440 Apparent[269:19289] [HockeySDK] WARNING: Detecting crashes is NOT enabled due to running the app with a debugger attached.
2016-11-16 12:14:52.634299 Apparent[269:19289] THREAD WARNING: ['HockeyApp'] took '104.316406' ms. Plugin should use a background thread.
2016-11-16 12:14:52.659939 Apparent[269:19289] THREAD WARNING: ['StatusBar'] took '24.387939' ms. Plugin should use a background thread.
2016-11-16 12:14:52.724460 Apparent[269:19357] Push Plugin register called
2016-11-16 12:14:52.724617 Apparent[269:19357] PushPlugin.register: setting badge to false
2016-11-16 12:14:52.724694 Apparent[269:19357] PushPlugin.register: clear badge is set to 0
2016-11-16 12:14:52.724756 Apparent[269:19357] PushPlugin.register: better button setup
2016-11-16 12:14:52.736952 Apparent[269:19357] GCM Sender ID <MY_SENDER_ID>
2016-11-16 12:14:52.737089 Apparent[269:19357] Using GCM Notification
2016-11-16 12:14:52.737225 Apparent[269:19357] Using GCM Sandbox
2016-11-16 12:14:52.765167 Apparent[269:19289] Push Plugin register success: <xxxxxxxx xxxxxxxx xxxxxxxx xxxxxxxx xxxxxxxx xxxxxxxx xxxxxxxx xxxxxxxx>
2016-11-16 12:14:52.857: GCM | GCM library version 1.1.4
2016-11-16 12:14:52.879: GCM | Invalid key in checkin plist: GMSInstanceIDDeviceDataVersion
2016-11-16 12:14:52.883409 Apparent[269:19289] GCM Registration Token: xxxxxxxxxxx:xxxxxxxxxxxxxxxxxxxxxx_xxxxxxxxxxxxxxxxxxx_xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx_xxxxxxxxxxxxxxxxxxxxx_xxxxxxxxxxxxxxxxxxx_xxxxxxx_xxxxxxxxxxxxxxx
2016-11-16 12:30:30.892178 Apparent[269:19289] subscribe from topic: -KWZfeQxUsNGJyjIlX3V
2016-11-16 12:30:30.896079 Apparent[269:19289] Already subscribed to -KWZfeQxUsNGJyjIlX3V
2016-11-16 12:30:31.106081 Apparent[269:19289] success:  Already subscribed to -KWZfeQxUsNGJyjIlX3V
2016-11-16 12:30:39.710497 Apparent[269:19289] [MC] System group container for systemgroup.com.apple.configurationprofiles path is /private/var/containers/Shared/SystemGroup/systemgroup.com.apple.configurationprofiles
2016-11-16 12:30:39.713902 Apparent[269:19289] [MC] Reading from public effective user settings.

You may try to replace the following code:
android: {
senderID: "MY_SENDER_ID",
},
ios: {
senderID: "MY_SENDER_ID",
alert: "true",
badge: "true",
sound: "true",
"gcmSandbox": "true"
}
with
"android": {
"senderID": "XXXXXXXX"
},
        "ios": {
           "senderID": "XXXXXXXX"
            "sound": true,
            "vibration": true,
            "badge": true,
           "gcmSandbox": "true"
        },

I struggled two days with my code. After making these changes, it worked. Give a try.

I was hopeful but this doesn't seem to be it. I tried several variations of quoting and not quoting the boolean values. Did you mean to leave the comma off after the senderID property inside the "ios" object?

OK I have an update.
I removed senderID and gcmSandbox from the ios config object and added a console.log(res) inside the .then(res) promise resolve for my $http POST to my heroku hosted node server.
I also moved the topic property outside the aps object but still within the notification object on my node server under new apn.Notification({ per node-apn docs.
Below is the output logs in Xcode;
2016-11-17 11:54:41.660420 Apparent[413:165145] Push Plugin register called
2016-11-17 11:54:41.660634 Apparent[413:165145] PushPlugin.register: setting badge to false
2016-11-17 11:54:41.660851 Apparent[413:165145] PushPlugin.register: clear badge is set to 0
2016-11-17 11:54:41.661129 Apparent[413:165145] PushPlugin.register: better button setup
2016-11-17 11:54:41.668489 Apparent[413:165145] GCM Sender ID (null)
2016-11-17 11:54:41.668619 Apparent[413:165145] Using APNS Notification
2016-11-17 11:54:41.720594 Apparent[413:165084] Push Plugin register success: <xxxxxxxx xxxxxxxx xxxxxxxx xxxxxxxx xxxxxxxx xxxxxxxx xxxxxxxx xxxxxxxx>
2016-11-17 11:54:50.692051 Apparent[413:165084] subscribe from topic: -KWZfeQxUsNGJyjIlX3V
2016-11-17 11:54:50.694056 Apparent[413:165084] Failed to subscribe to topic -KWZfeQxUsNGJyjIlX3V: Error Domain=com.google.gcm Code=3004 "(null)"
2016-11-17 11:54:50.899760 Apparent[413:165084] Error:  Failed to subscribe to topic -KWZfeQxUsNGJyjIlX3V - The operation couldn’t be completed. (com.google.gcm error 3004.)
2016-11-17 11:54:57.745157 Apparent[413:165084] [MC] System group container for systemgroup.com.apple.configurationprofiles path is /private/var/containers/Shared/SystemGroup/systemgroup.com.apple.configurationprofiles
2016-11-17 11:54:57.749777 Apparent[413:165084] [MC] Reading from public effective user settings.
2016-11-17 11:55:12.412907 Apparent[413:165084] {"data":{"response":"Notifications sent to: "},"status":200,"config":{"method":"POST","transformRequest":[null],"transformResponse":[null],"url":"https://murmuring-fjord-75421.herokuapp.com/","data":{"reg_id":"MY_REG_ID","token":"SUPER_LONG_TOKEN","message":"Hi again","sender_name":"Brad Wertman","topic":"/topics/-KWZfeQxUsNGJyjIlX3V","state":"app.room","roomId":"-KWZfeQxUsNGJyjIlX3V","sender_imgURL":"MY_IMGURL","chatters":["bf4b25b0-79d0-4b47-9bbc-1828ece621ed"],"platform":"iOS"},"headers":{"Accept":"application/json, text/plain, */*","Content-Type":"application/json;charset=utf-8"}},"statusText":"OK"}

I'm still not receiving push notifications on my Android device sent by my iOS device though. I tried setting a break point at $rootScope.$on('$cordovaPushV5:notificationReceived',function(event, notification){ in my .ready() and it doesn't get hit.
So I'm really not sure what else to check.

You have these errors:
2016-11-17 11:54:50.694056 Apparent[413:165084] Failed to subscribe to topic -KWZfeQxUsNGJyjIlX3V: Error Domain=com.google.gcm Code=3004 "(null)"
2016-11-17 11:54:50.899760 Apparent[413:165084] Error:  Failed to subscribe to topic -KWZfeQxUsNGJyjIlX3V - The operation couldn’t be completed. (com.google.gcm error 3004.)

Yes, it also says subscribe from topic: -KWZfeQxUsNGJyjIlX3V I had Googled for that error message before and didn't find much information on it. Does subscribe from topic: -KWZfeQxUsNGJyjIlX3V mean it subscribed?

Sounds like what this SO post is referring to. But I don't understand why GCM is involved in a push notification initiated by my iOS device. Shouldn't APN being handling it?
This is the factory I have set up for the topic subscription $$added gets called when the user's chat rooms load for each room.
Pta.factory('pushSubscribe', [
  '$firebaseArray',
  '$cordovaPushV5',
  function ($firebaseArray, $cordovaPushV5) {
  return $firebaseArray.$extend({
    $$added: function(room){
      // Room topic is the $id of the chat room
      $cordovaPushV5.Push.subscribe(room.key,
        function(success){
          console.log('success: ', success);
      },
      function(err){
        console.log('Error: ', err);
      });
    },
    $$removed: function(room){
      $cordovaPushV5.Push.unsubscribe(room.key,
        function(success){
          console.log('success: ', success);
      },
      function(err){
        console.log('Error: ', err);
      });
    }
  });
}]);

It seems you have successfully got to APN and got the token.
2016-11-17 11:54:41.668619 Apparent[413:165145] Using APNS Notification
2016-11-17 11:54:41.720594 Apparent[413:165084] Push Plugin register success:
Check your business logic.  If you want an iOS device to subscribe to topics you'll need to use GCM on iOS. That is you'll need to pass the sender ID and gcmSandbox as part of the init options. This is what you were initially doing. Go back to that.
It looks like everything was setup correctly in your original post.
So how are you sending the message from iOS to Android? Actually, I'm confused at this point if you want to send via GCM only or GCM and APNS.

@macdonst I need push notifications between iOS and Android devices. I'm using the unique id created by firebase when I make a $firebaseArray for the topic to subscribe to.
I check for what platform the message gets sent from and pass a corresponding "Android"  or "iOS"  string with the notification. I use that string on my node server to determine whether to push the notification through node-gcm or node-apn
As you said, I originally had the senderId and gcmSandbox set on my iOS init configuration but it didn't seem to be working then. I was receiving notification on the IOS  device sent from Android but not the other way around. This is still the case though with those configurations removed.
Am I understanding that APN doesn't support topic subscription and that it has to be done through GCM? I'm not against doing that. But from we what I had read in the docs here and elsewhere I was under the impression that it is preferable to go through APN directly since GCM had to do some type of look up to get the registration from APN anyway.

Thanks @macdonst,
I found that if I added senderId and gcmSandBox back into the ios config object passed to .initialize() AND removed the conditional logic for running either node-gcm or node-apn on my node server, just pushing everything through node-gcm I get push notifications going both ways between the two platforms.
Are there any caveats or drawbacks to doing it this way?

@bmwertman great, glad you figured it out.
If you are sharing notifications between Android and iOS then I would use GCM to send all of the messages and I don't really see any drawbacks to that.
As for your last issue where clicking on the message in Android it doesn't open the app I'm sure it is your payload. When sending to an Android device use:
data: {
        click_action: "FCM_PLUGIN_ACTIVITY",
        title: req.body.sender_name,
        body: req.body.message,
        state: req.body.state,
        roomId: req.body.roomId,
        sender_imgUrl: req.body.sender_imgURL
      }

When sending to iOS use:
notification: {
        click_action: "FCM_PLUGIN_ACTIVITY",
        title: req.body.sender_name,
        body: req.body.message
      },
      data: {
        state: req.body.state,
        roomId: req.body.roomId,
        sender_imgUrl: req.body.sender_imgURL
      }

Now you may ask why the difference? Well I've explained it here before but when you put stuff in the notification section of the payload the Android OS picks it up and does not inform the push plugin that a new message has arrived. I know it is a pain and I hope we can do better in the 2.0.0 release where we fully move to FCM.

Great explanation and thanks again for the help!


where is the RLIP_caption_coco.py?

Where is your script for generating pseudo-labels?

Apologies for forgetting to upload the code for BLIP. I am uploading it right now.

I update the readme with a code link. @ZHUXUHAN

I've updated the readme with a code link. @ZHUXUHAN

ok thanks for your answer


v4.x - rebase and merge, don't squash

Once ready, this PR should be rebased and merged, not squashed and merged.
fixes #64 (namespaces)
fixes #213 (all elements have renderAsync())
fixes #135 (don't render twice)
fixes #231 (transferAttributes mutates the element)
fixes #229 (async everywhere)

Probably best to review 1 commit at a time. Before merging, I'll fork v3.x and set up daily testing of that branch too.
Think we're at the point where master is 4.x is sensible.


Align padding between timeline and graph

Is your feature request related to a problem? Please describe.
Our visualization for labs recently was affected by https://github.com/cerner/carbon-graphs/commit/b1584e2d45c301536d34176e7573466b4b82d13c. Previously our graph-timeline combo looked like this:

After the commit above was released, it looked like this:

Describe the solution you'd like
We'd like to be able to remove our custom padding and have the axes still be in alignment with one another.
Describe alternatives you've considered
We might decide on locking carbon-graphs in our module to minor versions to prevent these issues going forward. Long term we'd like to avoid doing this.
Additional context
Here are the current custom paddings we apply when the timeline and graph are shown together:
timelinePadding = {
	top: 3,
	bottom: 5,
	left: 50,
	right: 50,
};
graphPadding = {
	top: 0,
	bottom: 5,
	left: 25,
	right: 67,
};

When the custom padding is not applied, the timeline graph looks like this:

I don't think the possible solution provided in the issue description can be done as they originate from different constructs and Timeline does not have Y label whereas, graph constructs require space for the label as well. What do you think? @abhijit945
@jeffmerten In my opinion the possible solution you can go with is
timelinePadding = {
        top: 3,
        bottom: 5,
-        left: 50,
+        left: 100,
        right: 50,
      };

Reason:
If you see the code at line 718, we have added config.padding.left to consider the spacing between y axes labels and axes for the graph construct. So for this reason timeline construct requires double the padding of graph construct. And in your test file, you are not displaying the label otherwise, you would also be required to consider LabelWidth while providing padding.

+1

@pranav300 I agree it might not make sense to hold the padding the same between the time and graph due to the y axis difference. Is it safe for us to assume that timeline padding = 2 * graph padding going forward?

@jeffmerten I may not be able to guarantee about timeline padding = 2 * graph padding as label width should also be considered. But currently, as much as I know there is no enhancement that would change it. But again I can't guarantee about future enhancements.

Can the label width be exported for us to use?

The padding property is always going to be content padding i.e. padding relative to the content.

If you want a graph level padding then I think solely depending on the above property to position your timeline is not desirable in your scenario. For your scenario, wouldn't it be a better if we applied CSS-style padding (px. rem etc.) on the container instead i.e. Graph level padding?

I think that probably would be a better way, but if we apply that css style padding on our own container we still have the same problem: how do we ensure alignment is maintained in our module in the future without locking carbon-graphs to a specific version?

Good point.
The issue is that Carbon has a default padding property for content and we have exposed it to consumers to be modified.
In this case, the consumer is entirely responsible for managing the padding left and right. Carbon sets out with initial setting of having no padding or margin.
Proposal
It goes without saying that this has not been implemented; padding and margin are not accounted for when the width of the container is calculated. This however can be done by doing something like this. When consumer doesnt provide the padding and/margin, the width will take 100% of the container.

@jeffmerten Created https://github.com/cerner/carbon-graphs/issues/121
Is what you were looking for?

I think this will be fixed with above issue, please feel free to reopen this if you have additional concerns.


Ensure that List title/description cannot contain markup or code

Lists can be made public. Ensure that these fields are using Django's built-in mechanisms for preventing JS or arbitrary HTML from being included.

Looks good, it's automatically escaped by Django:


Add DesignerRuntimeImplementationProjectOutputGroup

Contributes to https://github.com/dotnet/sdk/issues/2707
This adds a stub target for the new output group to common targets, which the SDK will override.
@davkean Is this the correct pattern to follow here?
FYI, @chabiss

https://github.com/dotnet/sdk/pull/2895 is up with the concrete implementation. I will wait for that to be reviewed before merging this.


[TextField/Input] focus/error underline are not bounded by the input element

Description
The current demo site doesn't have a layout that exemplifies the problem (tweaked code below).

Problem description
The focused/error underline width should match that of the unfocused underline.  You can see from the screenshot that the input element is considerably smaller than the container.  I think this somehow changed with @nathanmarks recent refactoring, though I don't have a recommended solution or I would just PR it.  Do we want the input to grow to the container width by default?  If so, that would solve this case.
Link to minimally-working code that reproduces the issue
Reproduce with a couple of tweaks to the demo:
// @flow weak

import React, { PropTypes, Component } from 'react';
import { createStyleSheet } from 'jss-theme-reactor';
import TextField from 'material-ui/TextField';

const styleSheet = createStyleSheet('TextFields', () => ({
  container: {
    // display: 'flex',
    // flexWrap: 'wrap',
    minWidth: 700,
  },
  input: {
    margin: 10,
  },
}));

export default class TextFields extends Component {
  static contextTypes = {
    styleManager: PropTypes.object.isRequired,
  };

  state = {
    name: 'Cat in the Hat',
  };

  render() {
    const classes = this.context.styleManager.render(styleSheet);

    return (

        <TextField
          id="name"
          label="Name"
          className={classes.input}
          value={this.state.name}
          onChange={(event) => this.setState({ name: event.target.value })}
        />

        <TextField
          id="uncontrolled"
          label="Uncontrolled"
          defaultValue="foo"
          className={classes.input}
        />

        <TextField
          required
          id="required"
          label="Required"
          defaultValue="Hello World"
          className={classes.input}
        />

        <TextField
          error
          id="error"
          label="Error"
          defaultValue="Hello World"
          className={classes.input}
        />

    );
  }
}

Versions

Material-UI: next 7df873a19455bd39cad1d3dbb99becee48b08377
React:
Browser:

The width on this stylesheet is causing the mismatch:
    inkbar: {
      '&:after': {
        backgroundColor: theme.palette.accent.A200,
        left: 0,
        bottom: -1,
        // Doing the other way around crash on IE11 "''"" https://github.com/cssinjs/jss/issues/242
        content: '""',
        height: 2,
        position: 'absolute',
        width: '100%',
        transform: 'scaleX(0)',
        transition: theme.transitions.create(
          'transform',
          '200ms',
          null,
          easing.easeOut,
        ),
      },
      '&$focused:after': {
        transform: 'scaleX(1)',
      },
    },

As I see it, we could either:
a) grow the input
b) stop styling the input with a border and only use &:after for all states

a) grow the input

@rosskevin I'm all 💯  for that option.
The very fact that the following example isn't rendering correctly is embarrassing.
<Input
  style={{width: 400px}}
  defaultValue="Hello world"
  className={classes.input}
/>

I've been locally using in my react-formal-material-ui:
    input: {
      width: '100%'
    }

Seems to work well.


Yii aliases in windows. Strange behavior.

/**
    * @param User   $user
    * @param string $newPassword
    */
   public function sendMail(User $user, $newPassword)
   {
       /** @var MailgunMailer $mailer */
       $mailer = \Yii::$app->mailer;

       var_dump(\Yii::getAlias('@runtime'));
       echo '
';
       var_dump(\Yii::getAlias('@runtime/mail'));
       echo '
';
       var_dump(\Yii::getAlias('@backend/mail'));
       echo '
';
       var_dump($mailer->viewPath);
       echo '
';
       die();
       /** @var MailgunMessage $message */
       $message = $mailer->compose('auth/recover-password', [
           'user' => $user,
           'newPassword' => $newPassword
       ]);

       $message
           ->setFrom('no-reply')
           ->setTo($user->email)
           ->setSubject($message->domain.' Восстановление доступа.')
           ->setTags(['recover-password'])
           ->send();
   }

gives me such result
string(46) "C:\xampp\htdocs\global\frontend_resume\runtime"
string(51) "C:\xampp\htdocs\global\frontend_resume\runtime/mail"
string(35) "C:\xampp\htdocs\global/backend/mail"
string(35) "C:\xampp\htdocs\global/backend\mail"

this is Yii2 Advanced and called from frontend namespace which has own config (not merged with backend config).
Also it working on Win7 so namespaces (slashes) is not unix like... But when I called working script at hosting (ubuntu) it throws me an error, in localhost (win7) script works fine (because win7 understand "" slashes).
Error in unix hosting:
The view file does not exist: ...<hidden>.../backend\mail/auth/recover-password.php
because of namespace, wrong slash in path.
So I debug aliases that see routes (code is on top).
Main point that I have namespaces frontend and backend, in frontend I created form that extends from form from backend, and it fires code below...
So question is - why slash is wrong? \Yii::$app->mailer->viewPath is default value, it should create "/" slashes. Where I should search for this issue?

Sorry, works everything fine.
Found redefine in front config...

        'mailer' => [
            'viewPath' => \Yii::getAlias('@backend').'\mail'
        ],

Here from come error. Can be closed, again sorry. ;)


Kraken send a "buy" limit order

Bitfinex:    7,401.00 / 7,408.90
Kraken:      7,298.80 / 7,298.80
—------------------------—
Kraken/Bitfinex:      1.40% [target  0.80%, min -0.31%, max  1.59%]   trailing  1.51%  1/1
 Looking for a limit price to fill 0.001351 BTC...
 order book: 1.594400@$7,402.40
[ ENTRY FOUND ]
Date & Time:       11/16/2017 10:56:15
Exchange Long:     Kraken (id 1)
Exchange Short:    Bitfinex (id 0)
Fees:              0.25% / 0.20%
Price Long:        7,296.50 (target)
Price Short:       7,402.40 (target)
Spread:            1.40%
Cash used:         10.00 on each exchange
Exit Target:       0.00%
 Trying to send a "buy" limit order: 0.001370 @ $7,296.50...
{"error": ["EGeneral:Invalid arguments:volume"]}

I run into same problems, what happened? What is the reason for this?

It looks like kraken is placing a minimum volume size of 0.002 on orders. Volume < 0.002 returns that json error.


Fixin quirks

This commit fixes #13 + #14:

Handles enter key press in cases where no result is highlighted
Adds an optional loading property to the Geocoder. This attaches a .loading class to the results list ul and leaves it to the user to decide what to do with it. I played with an additional div or attaching it to the input class, but this allows for the most flexibility.
Newest versions of React also don't require the line /** @jsx React.DOM */ declaration, so I've removed that from the example code.

@yhahn -- would love your eyes on this before merge.

Few more tweaks:

set focus on enter keypress
set focus when clicking result
keep focus on input when clicking result


Support GenericInitialValue (InitialValues.INIT)

https://github.com/tkf/InitialValues.jl/pull/29

[ ] Require InitialValues 0.2.4

Codecov Report

Merging #288 into master will decrease coverage by 3.78%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #288      +/-   ##
==========================================
- Coverage   89.75%   85.96%   -3.79%
==========================================
  Files          20       20
  Lines        1396     1297      -99
==========================================
- Hits         1253     1115     -138
- Misses        143      182      +39

Impacted Files
Coverage Δ

src/Transducers.jl
90.00% <ø> (-0.91%)
:arrow_down:

src/interop/onlinestats.jl
76.19% <ø> (-9.53%)
:arrow_down:

src/interop/dataframes.jl
0.00% <0.00%> (-100.00%)
:arrow_down:

src/basics.jl
35.00% <0.00%> (-50.72%)
:arrow_down:

src/simd.jl
78.12% <0.00%> (-21.88%)
:arrow_down:

src/unordered.jl
79.59% <0.00%> (-17.02%)
:arrow_down:

src/core.jl
74.37% <0.00%> (-15.57%)
:arrow_down:

src/reduce.jl
69.47% <0.00%> (-10.73%)
:arrow_down:

src/lister.jl
80.48% <0.00%> (-7.32%)
:arrow_down:

... and 10 more

Continue to review full report at Codecov.

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


Security scan

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


Beckn One - P2P Energy Trading Reference App Implementation

Description

Implement the Beckn One P2P Energy Trading Reference App based on defined buyer, seller, admin for buyer, and admin for seller flows, as well as the finalized UI/UX designs.
This task encompasses both frontend and backend development, with Strapi used as the BPP.
The application will be deployed on dev, staging, and production environments and hosted on the Experience Center across all three environments.

Related ticket:
#2365
#276

Goals

[ ] Develop the frontend and backend for the P2P energy trading reference app based on UI/UX designs and defined user flows.
[ ] Use Strapi as the backend (BPP) for data management and API interactions.
[ ] Set up infrastructure in dev, staging, and production environments.
[ ] Deploy the application on the Experience Center across all environments (dev, staging, prod).

Expected Outcome
A fully functional Beckn One P2P Energy Trading Reference App implemented according to specified user flows and UI/UX designs, deployed on the Experience Center across dev, staging, and production environments.
Acceptance Criteria

[ ] Frontend and backend development is completed based on the user flows and designs.
[ ] Strapi is configured and integrated as the BPP for backend functionality.
[ ] Infrastructure is set up in dev, staging, and production environments.
[ ] The application is deployed on the Experience Center across all three environments.

Mockups / Wireframes
Available (refer to finalized designs)
Product Name
Beckn One
Domain
P2P Energy Trading
Tech Skills Needed

Frontend Development
Backend Development
Strapi Configuration
Deployment and Infrastructure Management

Complexity
High
Category
Full Stack Development
Sub Category
App Implementation

Related ticket:
#2365
#276


differentiate struct_extract instructions using VJP

Differentiates struct_extract instructions using the VJP of the corresponding getter.
Float._value has no VJP defined so AD can't differentiate public static prefix func - (x: ${Self}). This could be fixed by defining a VJP for Float._value or by adding a custom adjoint for -. The latter sounded less invasive so I did that.

@swift-ci please test tensorflow

@swift-ci please test tensorflow

@swift-ci please test tensorflow

@swift-ci please test tensorflow

@swift-ci please test tensorflow

@swift-ci please test tensorflow


This information should be represented as a warning in documentation .

In current documentation of session and app state here - Session and app state in ASP.NET Core.
There are few session state exhibits and last one is the following:
Session state cookies aren't marked essential by default, so session state isn't functional unless tracking is permitted by the site visitor. For more information, see General Data Protection Regulation (GDPR) support in ASP.NET Core.
This information is very critical for developers and they should be aware that their session state in .net core 2.1 onward will not work, so this information should be represented as warning instead of an information only.

Thanks @gurmeetsinghdke ... This isn't going to be as much of an issue as of 3.0 Preview 8. They're going to remove GDPR from the templates. See: https://github.com/aspnet/AspNetCore/issues/10346
The GDPR doc speaks to this scenario at the top. https://docs.microsoft.com/aspnet/core/security/gdpr
btw -- There's a This page feedback button at the bottom of the topic that tags issues with metadata. Just noting it for future issues that you might need to open. It helps us track work on these issues.

Thanks @gurmeetsinghdke ... This isn't going to be as much of an issue as of 3.0 Preview 8. They're going to remove GDPR from the templates. See: aspnet/AspNetCore#10346
The GDPR doc speaks to this scenario at the top. https://docs.microsoft.com/aspnet/core/security/gdpr
btw -- There's a This page feedback button at the bottom of the topic that tags issues with metadata. Just noting it for future issues that you might need to open. It helps us track work on these issues.

@guardrex Thanks for providing the information about elimination of GDPR from template.
Should it not be there for documentation pertaining to .Net Core 2.1+ and <3.0? Please suggest.

When the topic author returns, he'll take a look.

@gurmeetsinghdke  I'm closing this for now given GDPR is going away in 3.0. If we get more complaints, I'll reconsider.


Pass props to Component

How can i pass row values to example component?

Hey,
In your example component, you are able to accept the data prop which corresponds to the row of data in question. This is automatically parsed to your component by default.
Try adding the following to your component:
props: {
    data: {
        type: Object,
        default: () => ({})
    }
},

By default, the row's data will be parsed as an object.
Let me know if that solves your issue.


Recurrent Neural Networks

Recurrent Neural Networks
So far, we've been looking at convolutional neural networks and models that allows us to analyze the spatial information in a given input image. CNN's excel in tasks that rely on finding spatial and visible patterns in training data.
In this and the next couple lessons, we'll be reviewing RNN's or recurrent neural networks. These networks give us a way to incorporate memory into our neural networks, and will be critical in analyzing sequential data. RNN's are most often associated with text processing and text generation because of the way sentences are structured as a sequence of words!
RNN's and Text Generation
At the end of this lesson, you will be tasked with creating a TV script generator; a model that takes in a series of words as input and outputs a likely next word, forming a text, one word at a time.
Similarly, RNN's can be used to generate text given any other data. For example, you can give an RNN a feature vector from an image, and use it to generate a descriptive caption. Image captions are used to create accessible content and in a number of cases where one may want to read about the contents of an image.
Sketch RNN
One of my favorite use cases for RNN's is in generating drawings. Sketch RNN (demo here) is a program that learns to complete a drawing, once you give it something (a line or circle, etc.) to start!
Sketch RNN example output. Left, Mona Lisa. Right, pineapple.
It's interesting to think of drawing as a sequential act, but it is! This network takes a starting line or squiggle and then, having trained on a number of types of sketches, does its best to complete the drawing based on your input squiggle.
Next, you'll learn all about how RNN's are structured and how they can be trained! This section is taught by Ortal, who has a PhD in Computer Engineering and has been a professor and researcher in the fields of applied cryptography and embedded systems.

Recurrent Neural Networks
So far, we've been looking at convolutional neural networks and models that allows us to analyze the spatial information in a given input image. CNN's excel in tasks that rely on finding spatial and visible patterns in training data.
In this and the next couple lessons, we'll be reviewing RNN's or recurrent neural networks. These networks give us a way to incorporate memory into our neural networks, and will be critical in analyzing sequential data. RNN's are most often associated with text processing and text generation because of the way sentences are structured as a sequence of words!
RNN's and Text Generation
At the end of this lesson, you will be tasked with creating a TV script generator; a model that takes in a series of words as input and outputs a likely next word, forming a text, one word at a time.
Similarly, RNN's can be used to generate text given any other data. For example, you can give an RNN a feature vector from an image, and use it to generate a descriptive caption. Image captions are used to create accessible content and in a number of cases where one may want to read about the contents of an image.
Sketch RNN
One of my favorite use cases for RNN's is in generating drawings. Sketch RNN (demo here) is a program that learns to complete a drawing, once you give it something (a line or circle, etc.) to start!
Sketch RNN example output. Left, Mona Lisa. Right, pineapple.
It's interesting to think of drawing as a sequential act, but it is! This network takes a starting line or squiggle and then, having trained on a number of types of sketches, does its best to complete the drawing based on your input squiggle.
Next, you'll learn all about how RNN's are structured and how they can be trained! This section is taught by Ortal, who has a PhD in Computer Engineering and has been a professor and researcher in the fields of applied cryptography and embedded systems.


Qrcode on windows terminal

https://github.com/parvardegr/sharing/issues/20
After checking the issues of qrcode-terminal, seems windows os can't support small option to output the qrcode in native terminal.
In this PR just adding -w to ignore the small will resolve this issue.
Example:
sharing f:\demo\demo\pom.xml -w

Before:

After:

@parvardegr updated.


secded_gen.py

def verify(cfgs):
    error = 0
    for cfg in cfgs['cfgs']:
        if (cfg['k'] <= 1 or cfg['k'] > 120):
            error += 1
            log.error("Current tool doesn't support the value k (%d)", cfg['k'])

        if (cfg['m'] <= 1 or cfg['m'] > 20):
            error += 1
            log.error("Current tool doesn't support the value m (%d)", cfg['m'])

        # Calculate 'm' (parity size)
        min_m = min_paritysize(cfg['k'])
        if (cfg['m'] < min_m):
            error += 1
            log.error("given \'m\' argument is smaller than minimum requirement " +
                      "using calculated minimum (%d)", min_m)

        # Error check code selection
        if (cfg['code_type'] not in CODE_OPTIONS):
            error += 1
            log.error("Invalid code {} selected, use one of {}".format(
                cfg['code_type'], CODE_OPTIONS))

    return error

At which step the error occurs when I enter K more than 120? For example K=266, this function def _hsiao_code(k, m) may be error ?

could you show your error message? i'm assuming the first case?
large k don't work too well with hsiao codes, it takes awhile to converge last i checked.
if you want to do large k, probably want to do hamming instead, since that's deterministic.

let's close this question for now. Let us know if there are additional questions.


[release-0.58]: Fix: Align Reenlightenment flows between converter.go and template.go

What this PR does / why we need it:
Manual backport for https://github.com/kubevirt/kubevirt/pull/8740.
The reason a manual backport is required is because the functional tests are added to hyperv_test.go which didn't exist in release-58 branch because the PR that created this file wasn't backported.
Release note:

Fix: Align Reenlightenment flows between converter.go and template.go

/cc @vladikr @acardace

/hold
waiting for https://github.com/kubevirt/kubevirt/pull/8793 to get in

/hold cancel

/retest-required

/test pull-kubevirt-build-release-0.58

/test pull-kubevirt-build-0.58

/retest-required

/retest-required

/retest-required

/retest-required

/retest-required


New freezeAllDragEvents configuration property to prevent node drag-and-drop

Hello and happy Hacktoberfest!
I implemented the resolution to the feature request in #308 that @danielcaldas described. It seems to work in that the nodes are frozen, but the entire graph still can be panned around and zoomed into. Let me know if the idea was to block those drag events as well (I can't tell with certainty from the discussion), and I can try to tackle this further.

Ok I think I achieved what was originally asked for. The new prop also disables panning and zooming, while leaving graph elements clickable.

Sorry I could not make it during Hacktoberfest.

That's fine, with the new restrictions, I wasn't really going for it.
I'll try to update the PR later today.

I tried manually pulling out older versions of those sandbox/ and tools/ auto-generated files from previous commits, but for some reason, two of them still appear modified in "Files changed".

I tried manually pulling out older versions of those sandbox/ and tools/ auto-generated files from previous commits, but for some reason, two of them still appear modified in "Files changed".


[BUG] Support Powershell in azure-spring-boot-samples

Current the scripts in azure-spring-boot-samples does not work for PowerShell.
We should make it work for PowerShell.
For example:

Closing in favor of https://github.com/Azure/azure-sdk-for-java/issues/26533.


Tap linuxbrew/xorg by default

Closes Linuxbrew/brew#440.
Signed-off-by: Bob W. Hogg<EMAIL_ADDRESS>

Ensure $HOMEBREW_PREFIX/bin is in PATH so Git is usable after installation

I don't fully follow this comment. What is the purpose of this line?
system "env", "PATH=#{HOMEBREW_PREFIX}/bin:#{ENV["PATH"]}", …

I don't fully follow this comment. What is the purpose of this line?

brew tap will install Git (using Utils.ensure_git_installed!) if it's not available already.
Assume Git is not available from the host already. If so, brew install git will be run. After the installation finishes, $HOMEBREW_PREFIX/bin has to be in the PATH, or else Homebrew will think Git still isn't installed. Instead, you would get paradoxical output like this:
🍺  /home/bob/.linuxbrew/Cellar/git/2.14.1: 1,408 files, 42.5MB, built in 35 seconds
Error: Git must be installed and in your PATH!
==> Installing git
Warning: git 2.14.1 is already installed
Error: Git is unavailable
Failed during: /home/bob/.linuxbrew/bin/brew tap linuxbrew/xorg

Thanks for the explanation, Bob!

HI, Bob. I'd like to retain the behaviour that the installation script itself does not install any packages. Some older distributions and other use cases require installing packages in a particular order or other hacky workarounds to bootstrap a system. If brew tap tries to install git, it breaks this constraint. So though I appreciate the PR, I think this code for tapping linuxbrew/xorg at the appropriate time should probably live in linuxbrew/brew rather than in linuxbrew/install. Thanks all the same!


[question][service bus] Current state against java sdk

Hello,
we are starting new project. As mentioned here library is moved to sdk and sdk has currently only beta version. So my question is is this going to be implemented here in spring-cloud-azure project? Do we have to start with older version an will there be some migration guide to newer version?
Thanks

Which module of Service Bus do you want to use? We're planning to upgrade our service bus SDK to track2 (the one in the azure-sdk-for-java repo) later.

well I want to use quue/topic so probably amqp. Or are we talking here about other modules?
What does mean track2 later?

Sorry for the late response. Are you going to use the Service Bus directly or use the Spring integration?

@saragluna we'd like to use spring integration

So we'll also upgrade the underlying Service Bus SDK once they got a released version.


Build.exe doesn't work without webhook.txt

                                                                                work

                                                                                doesnt work

will fix this later

fixed


Rebase atom deprecations

This is a follow on to #278, completing more of the deprecations. It involved a rebase, so I'm submitting a new PR with @erran's commits intact.

It's been so long. This is new and slick:

Except of course, we didn't catch the error:

Which means we no longer have our nifty dialog with an Issue link. :cry:

The trickiest one is going to be working with a new way of displaying our little viewer.
atom.workspaceView is no longer available. In most cases you will not need the view. See the Workspace docs for alternatives: https://atom.io/docs/api/latest/Workspace. If you do need the view, please use atom.views.getView(atom.workspace), which returns an HTMLElement.

Hopping on a plane, hope I get to reinvigorate this soon.

I'm going to be busy at Strata for a few, feel free to add on if you get time @erran or just merge as is. Everything is working but we're still not living in the future with Atom here.

I'm going to be busy at Strata for a few, feel free to add on if you get time @erran or just merge as is. Everything is working but we're still not living in the future with Atom here.

I've personally being using good ol' vim a lot recently. I've been in and out of Atom. I'll probably have some time this weekend to hack away on some of my own projects in which case I'll definitely dedicate some time to atom-script.

Ok if we put this much in at the very least? Should cut down on reports quite a bit.

Yeah, these changes look good.
Sent from my iPhone

On Feb 23, 2015, at 08:23, Kyle Kelley<EMAIL_ADDRESS>wrote:
Ok if we put this much in at the very least? Should cut down on reports quite a bit.
—
Reply to this email directly or view it on GitHub.

I was about to release this too then I realized you had a flow for how you were populating the changelog.

Shipped.
Preparing and tagging a new version ✓
Pushing v2.17.0 tag ✓
Publishing<EMAIL_ADDRESS>✓


Incompatibility with Towny (and potentially more protection plugins)

It seems that the latest introduction of the config option
delete-invalid-items: true

has also introduced a little problem.
If I'm using a plugin like Towny along with AxShulkers and I leave this config option true, it'll lead to a user placing a shulker box in a town they do not have access to, for some weird reason it then messes something up with AxShulkers which leads to all items inside being deleted if you try to open it afterwards.
So to reproduce:

Install AxShulkers
Install Towny
Get a shulker box, put things in it through AxShulkers plugin
Place shulker box in a town you can not place blocks in
Open shulker box through AxShulkers plugin and see it is now empty

I hope there is a fix for this as I like the idea behind the config option but it's currently quite a problem with other plugins 😕

For some reason towny is using high for an event priority (not sure why a protection plugin would do that), and axshulkers used normal, which caused this issue, https://github.com/Artillex-Studios/AxShulkers/releases/tag/1.9.1 should solve this issue!

I just now got to verify it, seems to be working fine. Thanks once again for the quick response <3


toMatchText | Add method to verify that an element or page text matches the given pattern.

This PR, adds a new method to verify that the selected element's text matches a given pattern. This is really useful to validate elements which text is dynamic for example.
2021 Copyright vs 2020 Copyright  matches 202[0-9] Copyright
I was not sure how to call this method, my options were:

toMatchText
toHaveTextMatching

Can we just improve toHaveText and toEqualText to accept both a string or regex?  That would improve both functions without needed an extra matcher.
expect(page).toEqualText("#foo", "bar")
expect(page).toEqualText("#foo", /202\d/)
expect(page).toEqualText("#foo", new RegExp("202\\d"))

That sounds good as well.. Either way works just fine.
I think a new matcher is cleaner when looking at possible tests:
...
expect(page).toEqualText("#foo", "bar")
expect(page).toMatchText("#bar", /thismatch/)
expect(page).not.toMatchText("#bar", /notthismatch/)

I'm torn.  Part of me thinks using the existing would be better, part of me see's your point.  If we do add a new matcher, I'd recommend we deprecate toHaveText so then there are only two matchers.  toEqualText for exact text matching and toMatchText for regex based matching.  That way you can do all the following:
expect(page).toMatchText('#bar', /^Copyright 202\d$/) // regex match for the full text content
expect(page).toMatchText('#bar', /202\d/) // regex match of partial text content
expect(page).toMatchText('#bar', 'Copyright') // string matches partial content like toHaveText

That would help with migration, toHaveText becomes toMatchText with more powerful regex matching.
Thoughts?

I'm torn. Part of me thinks using the existing would be better, part of me see's your point. If we do add a new matcher, I'd recommend we deprecate toHaveText so then there are only two matchers. toEqualText for exact text matching and toMatchText for regex based matching. That way you can do all the following:
expect(page).toMatchText('#bar', /^Copyright 202\d$/) // regex match for the full text content
expect(page).toMatchText('#bar', /202\d/) // regex match of partial text content
expect(page).toMatchText('#bar', 'Copyright') // string matches partial content like toHaveText

That would help with migration, toHaveText becomes toMatchText with more powerful regex matching.
Thoughts?

I think this sounds like the right way to proceed. Would you let me know what is the right way to deprecate the other method?

Remove it from the docs.
Add @deprecated in the jsdocs?
...

I think this sounds like the right way to proceed. Would you let me know what is the right way to deprecate the other method?

I'm a newer member of this project, so before proceeding with my suggestions, I'd check in with @mxschmitt and @mmarkelov to ensure our discussion aligns with their goals for the project.  If it does, your two items are correct: remove from docs, add @deprecated with a note to use toMatchText.

Deprecating it sounds good, but I would not remove it in the near future. Sounds like a good plan. :+1:

Codecov Report

Merging #81 (aed1537) into master (c531fe1) will increase coverage by 0.64%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master      #81      +/-   ##
==========================================
+ Coverage   90.97%   91.61%   +0.64%
==========================================
  Files          10       11       +1
  Lines         144      155      +11
  Branches       26       27       +1
==========================================
+ Hits          131      142      +11
  Misses         13       13

Impacted Files
Coverage Δ

src/matchers/toHaveText/index.ts
100.00% <ø> (ø)

src/matchers/toMatchText/index.ts
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

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


[ICD] Refresh key to avoid check-in counter rollover problems.

fixes: https://github.com/project-chip/connectedhomeip/issues/28235

@thivya-amazon conditional approval assume you address the rest of comments, also please work on unit test on it at the same time, thanks


[Extension] Use loading component from extension point

Signed-off-by: Abhishek-kumar09<EMAIL_ADDRESS>Description
Removes loading component from the Meshery-UI instead use a loader componenent provided from extension point.
Notes for Reviewers
Signed commits

[x] Yes, I signed my commits.

Appropriate to list a short description here? https://docs.meshery.io/extensibility/ui

Codecov Report

Merging #5372 (0bc68d1) into master (a9e0deb) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #5372   +/-   ##
=======================================
  Coverage   17.47%   17.47%
=======================================
  Files         107      107
  Lines        9882     9882
=======================================
  Hits         1727     1727
  Misses       7733     7733
  Partials      422      422

Flag
Coverage Δ

gointegrationtests
6.00% <ø> (ø)

integrationtests
∅ <ø> (?)

unittests
13.16% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Continue to review full report at Codecov.

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

I will add notes for Custom Extension components and loader

I will add notes for Custom Extension components and loader

Thanks much.


Only works while dragging / FPS use case

Hey, thanks for this amazing library!
I've used a while ago for this FPS-style camera implementation.
However, upon testing it more recently, it appears that with new Flutter versions it now only works while the mouse button is down, i.e., while dragging.
While not dragging, the mouse pointer does "disappear", but I don't get mouse events via onPointerDataPacket.
It seems by the readme that this is an "expected" issue, and that this packet is not really meant for what I thought it was (first person camera). And the events are intentionally only sent when the mouse button is down. Is that correct?
And, more broadly, is FPS-camera a use case that you wish to support with this library? Or is this out-of-scope?

I also have tried this simplified version based on the example on the repo, which also does not work:
mouse.dart
class Mouse {
  static final _lock = PointerLock();

  static Offset _delta = Offset.zero;
  static StreamSubscription<Offset>? _lockStream;

  static Offset getDelta() {
    final delta = _delta;
    _delta = Offset.zero;
    return delta;
  }

  static Future<void> init() async {
    //
  }

  static Future<void> dispose() async {
    //
    return await unlock();
  }

  static Future<void> reset() async {
    _delta = Offset.zero;
  }

  static Future<void> lock() async {
    if (_lockStream != null) return;
    _lockStream = _lock
        .startPointerLockSession(windowsMode: WindowsPointerLockMode.clip)
        .listen((delta) => _delta = delta, onDone: unlock);
    await _lock.lockPointer();
    await _lock.hidePointer();
  }

  static Future<void> unlock() async {
    if (_lockStream == null) return;
    await _lockStream?.cancel();
    _lockStream = null;
    await _lock.showPointer();
    await _lock.unlockPointer();
  }
}

Hey, thanks for this amazing library! I've used a while ago for this FPS-style camera implementation.
However, upon testing it more recently, it appears that with new Flutter versions it now only works while the mouse button is down, i.e., while dragging. While not dragging, the mouse pointer does "disappear", but I don't get mouse events via onPointerDataPacket. It seems by the readme that this is an "expected" issue, and that this packet is not really meant for what I thought it was (first person camera). And the events are intentionally only sent when the mouse button is down. Is that correct?

With the streaming mode, that's correct. It only works with button being down. With that other mode, it should also work with other "triggers". Are you on Windows?

And, more broadly, is FPS-camera a use case that you wish to support with this library? Or is this out-of-scope?

I think it would be a nice addition to this library and it somehow fits in, but I currently don't have the time to add it. But if someone else wants to give it a go, sure!

I also have tried this simplified version based on the example on the repo, which also does not work:

Does it work when you download this repo and start the example app as is?

Thanks for the response!
I am on macos (sadly flutter_gpu only works on macos for now), but I did try both "capture modes".
I will try the example app in a bit - I thought it only had the lock while dragging use case.
Either way - I would be happy to try to contribute, if you are open to a PR :)


Sometimes the "on" event needs to be listed first

App example:
http://gist.github.com/946d24720603b170c826
It would not work if the on event was listed after the record, it had to be listed immediately after the Tropo:Generator.new
This seems to be almost arbitrary when it's needed, I can't tell why it's happening.

resolved


Set shell script interpreter to bash instead of sh

On many systems sh points to bash. In that case the scripts work fine. However, double square brackets won't necessarily work when sh does not point to bash but to a POSIX sh. This commit changes the interpreter to bash. Alternatively, one can change the double square brackets to single square brackets to achieve portability.
Fixes #495.

I'm not too experienced with bash or sh scripting... but would this change break for someone using zsh? (which is now the default in macOS, but I haven't bothered to switch over)

This a good question, I'll have the possibility to test this at home in the next couple of days if @corneliusludmann is not able to verify or test this.

As long as bash is installed it does not matter whether the default shell is zsh or any other shell. The first line (shebang) says exactly that, that bash should be used for executing the script. I assume that MacOS will still have a bash installed. Since I don't use MacOS myself, I can't really say much about it.

Just heads up - I went ahead and reverted this commit as it broke the automated docker builds.
The docker image uses Alpine linux, which doesn't use bash but instead uses ash?
Cloning into '.'...
Warning: Permanently added the RSA host key for IP address '<IP_ADDRESS>' to the list of known hosts.
Switched to a new branch 'v3.5.2'
Pulling cache layers for index.docker.io/sqlpad/sqlpad:3.5.1...
Done!
KernelVersion: 4.4.0-1060-aws
Components: [{u'Version': u'18.03.1-ee-3', u'Name': u'Engine', u'Details': {u'KernelVersion': u'4.4.0-1060-aws', u'Os': u'linux', u'BuildTime': u'2018-08-30T18:42:30.000000000+00:00', u'ApiVersion': u'1.37', u'MinAPIVersion': u'1.12', u'GitCommit': u'b9a5c95', u'Arch': u'amd64', u'Experimental': u'false', u'GoVersion': u'go1.10.2'}}]
Arch: amd64
BuildTime: 2018-08-30T18:42:30.000000000+00:00
ApiVersion: 1.37
Platform: {u'Name': u''}
Version: 18.03.1-ee-3
MinAPIVersion: 1.12
GitCommit: b9a5c95
Os: linux
GoVersion: go1.10.2
Starting build of index.docker.io/sqlpad/sqlpad:3.5.2...
Step 1/8 : FROM node:12.3.1-alpine
---> 91acf04599c4
Step 2/8 : ENV NODE_ENV production
---> Using cache
---> 37abc5f5c2dc
Step 3/8 : EXPOSE 3000
---> Using cache
---> 02564c4020ca
Step 4/8 : ENTRYPOINT ["/docker-entrypoint"]
---> Using cache
---> 1a788d8ab755
Step 5/8 : WORKDIR /sqlpad
---> Using cache
---> f44a9933cedd
Step 6/8 : COPY . .
---> bdec28c21aa1
Step 7/8 : RUN scripts/build.sh && npm cache clean --force && cp -r /sqlpad/server /usr/app && cp /sqlpad/docker-entrypoint / && chmod +x /docker-entrypoint && rm -rf /sqlpad
---> Running in 0fb5e2099e1c
env: can't execute 'bash': No such file or directory
Removing intermediate container 0fb5e2099e1c
The command '/bin/sh -c scripts/build.sh && npm cache clean --force && cp -r /sqlpad/server /usr/app && cp /sqlpad/docker-entrypoint / && chmod +x /docker-entrypoint && rm -rf /sqlpad' returned a non-zero code: 127

Actually @corneliusludmann @deemoowoor is the existing shebang #!/usr/bin/env sh path an issue? Does #! /bin/sh work in your environment?

In this particular case #!/usr/bin/env bash should work best, I think. Using something like #!/bin/bash is usually fine, too, unless you're running the script on some very specific setups/systems where shell might be installed somewhere else than /bin (somewhat rarely, it might be).

Unless someone is willing to rewrite the script to POSIX shell syntax, changing it to use #!/usr/bin/env bash shebang should suffice, I think.


Fix: typos

Fix: typos
CLA: trivial

This would be acceptable with CLA: trivial. Could you please amend the commit message to include CLA: trivial on a separate line in the commit message body?

@t8m just added the mention :)

@omahs you need to amend the commit message not just the PR text. Use git commit --amend and git push --force for that.

24 hours has passed since 'approval: done' was set, but this PR has failing CI tests.  Once the tests pass it will get moved to 'approval: ready to merge' automatically, alternatively please review and set the label manually.

@hlandau assuming you're OK with CLA: trivial.

I've amended the commit message to contain the CLA: trivial when merging the PR.
Merged to master branch. Thank you.


Tooltips with class argument

The HtmlHelper::tooltip() method wraps content in an element and adds attributes necessary for bootstrap tooltips.
I would expect the method to also accept 'class' etc. options and pass them through so they end up in the HTML (as most methods do, e.g. tag(), div(), para(), etc.).
However I didn't really understand how to change the code of tooltip() to make it work.

The tooltip() method should already do that, but I have just noticed the template is incorrect:
'tooltip' => '<{{tag}} data-toggle="{{toggle}}" data-placement="{{placement}}" title="{{tooltip}}">{{content}}</{{tag}}>'

There is a missing {{attrs}} placeholder, so the template should be:
'tooltip' => '<{{tag}} data-toggle="{{toggle}}" data-placement="{{placement}}" title="{{tooltip}}">{{content}}</{{tag}}>'

I closed this by mistake, sorry...

No worries, pull request is coming up.


Add a form component

We should include a form component for rendering forms.
Here is a list of MyCase requirements:

Renders a form based on a data structure
It should not be tied to a state management solution
It should be able to include client-side validations as max length, required fields
It should allow plugging in custom validations for fields
It should allow rendering a form with already invalid fields
Uses the xanthous components for rendering the fields

One solution I found is React-Formal:

Uses yup schemas for setting structure
Form has onChange prop that is called when a value changes
It will validate the yup schema and add those messages to the DOM
Yup is used for validation and it allows custom transformations and tests.
Form allows passing in defaultErrors to start with errors in uncontrolled mode, or use errors in controlled mode.
Each component has a component prop that will be used for rendering the Form.Field, Form.Message etc.


⚠ Roblox Devforum has degraded performance

In e2f449c, Roblox Devforum (https://devforum.roblox.com) experienced degraded performance:

HTTP code: 500
Response time: 1171 ms

Resolved: Roblox Devforum performance has improved in d84c87f after 21 minutes.


Some plant titles do not appear on title bar due to limited scroll

Screenshot from Pixel 3 XL with page scrolled up all the way.

Could not reproduce on Pixel 3XL :/

Hi ,
Tested on Pixel 3XL Emulator ,When Scrolled Up Title is properly displaying ,attached screen shot for reference.

Hi,

Hi,

HI,
There was issue with minHeight in plant desc textView.
I have tested with  Pixel 3XL/4XL Emulator working as expected.

#729


Project Name Change : Sonar Tool Request

Please always provide the Jira Issue your PR is for, as well as test URLs where your change can be observed (before and after):
Jira ID:
Test URLs:

Before: https://main--exlm--adobe-experience-league.hlx.live/?martech=off
After: https://project-name-update--exlm--adobe-experience-league.hlx.live/?martech=off

Code Quality Check not passed and queued for long time. Hence closed the PR.


How to include ebpf_xdp.h

When compiling a .p4 file as described in the readme, clang cannot find the ebpf_xdp.h header:
/tmp/xdp1.h:6:10: fatal error: 'ebpf_xdp.h' file not found

The header file is included in this repository and says

This file contains all functions and definitions necessary for the xdp target C code to compile. It must be included with any file generated by the p4c-xdp compiler.

But how am I supposed to include it? Of course, I could hack this together somehow, but I'm wondering what is the correct way you intended for this. 😄 Maybe the Readme needs an update?

The C compiler can be given a -I flag to indicate a path where headers are expected to be found.
So probably adding clang -I tests .... would work (assuming you are invoking clang in the root directory of the p4c-xdp project, since the ebpf_xdp.h file is there.) If this works for you we can update the README.

Thank you for your quick response.
Yes, that's what I meant with hack this together. 😄 I managed to build it now by including these directories:
-I ../p4c/backends/ebpf/runtime/ \
-I ../p4c/extensions/p4c-xdp/tests/ \
-I ../p4c/backends/ebpf/runtime/contrib/libbpf/src/ \

Is it really necessary to include them manually or is there something wrong with my install (debian 10, built from sources) annd normally the headers get installed into the system?
Anyway, adding a hint to the Readme would probably be helpful for others.

This package does not install anything on your system, so you will need to specify the path to the headers.


Make write functions

Detailed Description

Write function for a forecast
Write function for PV data

Context
Good to have generic write function, which can save to many tables, but the thing doing the writing doesnt need to know about
Possible Implementation

Write Forecasts, One write to the Forecast and forecast_value table and writes to the forecast_value_latest table, being upserting.
Add a PV site
Update a PV site
Add a PV Generation value
TODO defined inputs to these objects

The input to save the forecast could be a list if dataframes. Each of the dataframes will have

"target_datetime_utc"
"forecast_kw"
"pv_uuid"

This will keep things in dataframes in the forecaster, which is handy.
we could use https://pandera.readthedocs.io/en/stable/ to ensure the dataframes have the correct columns

Would we be able to add an option where we pass ml_id instead of pv_uuid This would keep the old database cleaner - https://github.com/openclimatefix/nowcasting_datamodel/issues/140

So the dataframe could either be
"target_datetime_utc"
"forecast_kw"
"pv_uuid"

Or
"target_datetime_utc"
"forecast_kw"
"pv_ml_id"

for instance?

yea, that would be great


kernel: activate arch interface headers

These new headers are now pulled into the appropriate #include spaces.
Some documentation tweaks as well. There is one more PR before this effort is completely done, to renamespace z_arch_ to arch_, as doxygen currently skips everything beginning with z_.

@aescolar I had to touch some posix headers, please review.
I think there is more work to be done to fully disentangle things, but that can wait until we start work on #19666

Can we get this merged soon? And have sufficient time until the release to test possible issues with this refactoring.

@ioannisg waiting for @aescolar to review, I emailed him, once that's done I'll rebase for merge


[NFC] Performance guide: Mention that AdaptiveCpp can target CPUs through OpenCL too

Interesting. Does DPC++ set those vectorization hints by itself? Usually the generated SPIR-V should be pretty similar between DPC++ and AdaptiveCpp.

Does DPC++ set those vectorization hints by itself?

Maybe, but it looks like it does. I'm not sure what exactly icpx is doing with the code, and I haven't tested it with the open source DPC++ yet.


Not working with Vue

Expected
0 lines  of code
What happened?
49 lines of code
Steps to reproduce

vue init webpack nocode
find nocode -name '*' | wc -l

Works great on my Saturn (with no transmission). Sony NoStation Vue already has no channel for it. Have you tried them?


Break conventions for the demo app

This allows the new version of core-ktx without breaking the build. This does this by breaking the conventions for the demo-app -- basically letting it do its own thing and duplicating some things from the conventions.

(the image was only updated to reduce its size after seeing the app crash once)

Bah, closing for now.


webview: Error: Could not register service worker: InvalidStateError

I am now getting the above error after updating to ollama v0.4.4 (preview) is related to this issue.
It occurs as soon as you try to launch Continue. I tried uninstalling a reinstalling, but that didn't fix it either.
Here's the error vscode throws:
Error loading webview: Error: Could not register service worker: InvalidStateError: Failed to register a ServiceWorker: The document is in an invalid state..
Originally posted by @shashanknimje in https://github.com/continuedev/continue/issues/3048#issuecomment-2495518538

Same problem
@shashanknimje
I temporarily installed version 0.9.228 and it works for me

@shashanknimje I run in the same problem -> https://github.com/continuedev/continue/issues/3062
I resolved the issue by modifying a single letter in the extension's code.

Hi all, thanks for the quick feedback here 🙂  we've shipped a fix but our CI is broken right now due to an upstream outage in GH Actions. We'll post an update once the new version is out here: https://github.com/continuedev/continue/issues/3065

fixed by #3059


DPDK Backend : Minor fixes

Error if learner table exists with non-exact match keys
Removed explicit from constructor with more than one parameters
Typo in condition check in context json generation
`4) Added new test

stderr file is empty. The testcase added is an error-case ?

Yes . Working on fixing the test infra as well. Currently, the stderr is empty for all error cases as well.  stderr file is actually for errors/warnings from p4test backend. For dpdk, we use a file called -error which is also empty in most cases.
Current way of adding negative tests is to add them in the Xfail list with expected regex in the error output

Closing as upstreamed


add use techstack logo

Describe the changes that you want to make in the documentation
i will add use techstack logo in the readme file
Significance of the changes made
it will look good and easily anyone can find it

can you assign it to me . i am very new at gssoc


Versions with working physics fluid is still not imported

Even though there is no fluid I feel like the 2 working versions of the working physics engine is a step in the right direction which would potentially lead me into a working fluid simulation.


Allow intialization using an instance of winston

Thanks for creating this library. I have a small concern about the setup.
The current setup forces me to initialize winston via the express-winston options. In my opinion it would be neat if it let's me pass the instance of winston within options in addition to the express-winston specific options.

Just realized that this is already supported. I will show myself out. Thank you!


JSDoc3 ~ namepaths are not supported

JSDoc3 allows to use ~ character to refer to the inner member of the class/namespace (docs), but jsdoctypeparser does not recognize such declarations as a valid types:
>parser.parse('Name~property')
Unexpected token: "":
Name~property
   ^

Thanks for your reporting, @SevInf .
I'm gonna enable it on v2.0.0 (related 9e60e6d).
And after v2.0.0, we can get new AST that is not comatible to v1.x.

It will be released with v2.0.0


Adds Cancun configuration and fixes issues related to eip1559 and eip2929

The PR adds a configuration for Cancun fork.
Includes a fix related to EIP-2929, where some addresses were incorrectly considered cold and consumed extra gas. I identified this issue when running the tests located at ethtests/GeneralStateTests/Cancun/stEIP1153-transientStorage, which were failing because the gas costs did not match.
Also adds a new configuration property eip_1159 which when set to true, burns the base fee, instead of sending it to the coinbase address.

@sorpaas can you have a look at these changes?
I splitted these changes from the #278 to make the review easier.

@RomarQ could you split this PR into three PRs?

PR1: fix eip1559
PR2: fix eip2929 (and update jsontests in CI)
PR3: add cancun config

Could you add ethtests/GeneralStateTests/Cancun/stEIP1153-transientStorage into jsontests in CI

Could you add ethtests/GeneralStateTests/Cancun/stEIP1153-transientStorage into jsontests in CI

This depends on https://github.com/rust-ethereum/evm/pull/288.
We can test it this way:
cargo run --release --verbose -p jsontests -- jsontests/res/ethtests/GeneralStateTests/Cancun/stEIP1153-transientStorage --debug

Once we enable tests for Cancun fork here:
https://github.com/rust-ethereum/evm/blob/8168ca150abce577a4fb78d2da0918d229939b9e/jsontests/src/run.rs#L106
if we run all the tests, some will fail, mainly because there are still a few incompatibilities after berlin (gas related). I only fixed a few, there are more.

Once we enable tests for Cancun fork here:
https://github.com/rust-ethereum/evm/blob/8168ca150abce577a4fb78d2da0918d229939b9e/jsontests/src/run.rs#L106
if we run all the tests, some will fail, mainly because there are still a few incompatibilities after berlin (gas related). I only fixed a few, there are more.

We could fix those incompatibilities in future PRs, let's merge this PR first


How to config the redis trigger in the http trigger context

Hi, I read the document: https://spin.fermyon.dev/rust-components/, there is a phrase:
This HTTP component can be paired with a Redis component, triggered on new messages on the messages Redis channel.

But there is only one trigger config item in the application level, so how to understand this sentence and how to config this feature, because I want to build an application which simultaneously requires http component and redis component.
Thank you.

Hi, @miketang84!
You are right, that particular phrase is a bit loose, apologies.. (a git blame reveals I am to blame here).
You currently cannot have an application that contains two separate component types / triggers.
We hope to solve this in the future (#40) but currently, you need two separate applications in order to achieve this.

Got, thank you.


Resolution does not scale with pixel density, resulting in blurry pictures on high-density screens

Hello!
I am using your (amazing) integration for my lovelace wallpanel screensaver on a Pixel Tablet.
I tried to troubleshoot why my pictures were blurry, and after some tests I believe it is because the integration is downloading pictures with a resolution based on the CSS resolution (1280x800) instead of the device pixel resolution (2560x1600)
I confirmed this by enabling the debug log and looking at the picture downloaded.
Is that something you can fix on your end? I'm not sure if it is an issue with camera entity in HA or with your camera code, as I do not know how the width/height of the screen is retrieved.
Either way, I'm open to suggestions if a fix is not possible in the integration.
Thanks!

Not stale, good bot o/

I've not yet been able to reproduce your issue. The dimension to be requested is set by the HA frontend code, which looks like it is taking the pixel ratio into account. Do you have other integrations that display the resolution correctly?


Add GitHub workflow to automate tests

This adds a GitHub workflow that uses my action-janet-test GitHub Action. That action bundles together a number of different steps:

downloading Janet
installing Janet
downloading JPM
installing JPM
checking out this repository
installing any dependencies (in this case there are none)
running jpm test

You can see similar workflows in use in some of my Janet libraries (e.g. Documentarian and Testament).

Thanks, much appreciated!


Added new sample to List all application customizers resolves #2074

New sample to list all application customizers in a tenant.

Merged manually. Thank you! 👏

Merged manually. Thank you! 👏