GH14: Auto Detect Cake Tasks in VSCode

@cake-build/cake-team can you please all have a look at this, and let me know what you think?

In order to run this...

it is probably easier to remove the existing Cake extension that you will have installed
check out the latest source code
open project in VSCode
press F5
open a folder that contains some *.cake files
Press F1 and type task
Select Tasks: Run Task

The list of Cake Tasks should then be displayed.

@mholo65 If you could take it for a spin as well on your machine, to make sure that it isn't a case of "works on my machine" I would appreciate it. Thank you!

@gep13 will do when near computer :wink:

Verified on my computer with two different projects. Works like a charm!

@mholo65 said...
Verified on my computer with two different projects. Works like a charm!

Woot!
Was that a Windows machine?  Do you have access to a Mac or Linux box to test Task execution works as expected?

@gep13 said...
Was that a Windows machine? Do you have access to a Mac or Linux box to test Task execution works as expected?

Yes it was. No don't have access to either Linux or Mac. No need for Linux box anymore since Win10 and WSL. Maybe @redth want to try as he was originally interested in #26?

Good thinking!  Will wait and see if @redth can also provide some input here.

Going to give this a spin...

Finally got this going... I have one suggestion.  Let's make the regex pattern less rigid.
For instance, usually my old mono code habits lead me to declaring a task like this:
Task ("libs").Does (() => { /* ... */ });

Notice the space after Task and before (.
In theory, it's possible to have whitespace in between parenthesis and quotes, so how about we use this as the regex?
/Task\s?\(\s?"(.*?)"\s?\)/g

Thoughts?

@redth yes, that makes a lot of sense.
How about we go even further and add the regex as a configuration option, and set a default value.

Great!

@Redth said...
Great!

I will make the configuration additions, and then I might get you to take another look if that is ok.

Just let me know!

Correct me if I'm wrong, but wont the regex proposed by @redth mark zero or one whitespace. I think should throw in a star there to also match many whitespaces. Keep the question mark to make it ungreedy.
I.e whitespace regex should look like this \s*?

@mholo65 said...
I.e whitespace regex should look like this \s*?

Yes, I had had the same thought, but hadn't replied about it.  Yes, I believe you are correct.  Just done a quick test here:
https://regex101.com/
And your suggestion of \s*? is required.

Good catch, for some reason i was thinking \s would match one or more which clearly is not the case.

@Redth yeah, \s is just whitespace. \s+ is one or more whitespace. \s* is zero to many. Adding the ? will make the match nongreedy, i.e. will stop at first match.

@mholo65 yep, completely obvious after the fact ;) Thanks for noticing that one!

@mholo65 @Redth if either of you have a minute, can you have another look at this PR?  You should now have the following configuration options:

Which allow configuration of how the Tasks are detected.  The defaults should work for you as expected, but if you need something more the options are there.

Looks good to me!

@gep13 Would it be possible to group the settings like we've done with the bootstrappers? It makes it easier to see what settings are related. If not, could we add something to the setting namespace to group them?
"cake.taskRunner": {
  "autoDetect": "on",
  "scriptsGlobPattern": "**/*.cake",
  "scriptsExcludePattern": "**/format_C.cake",
  "taskRegularExpression": "lol"
}

or
"cake.taskRunner.autoDetect": "on",
"cake.taskRunner.scriptsGlobPattern": "**/*.cake",
"cake.taskRunner.scriptsExcludePattern": "**/format_C.cake",
"cake.taskRunner.taskRegularExpression": "lol"

@patriksvensson hmmm, it might be, but I will have to check.  The autodetect parameter is an enum, and I am not sure if you can next it inside something else.  But I will give it a try.
Agree on the renaming for consistency regardless.
@agc93 @patriksvensson were either of you actually able to take this for a spin?  Did it work on your machines?  Would love if someone could test this on a Mac.

@patriksvensson I have updated the extension to use the following:

Thoughts?

@gep13 I think it looks awesome 👍

@gep13 I can probably test it on my mac tonight if VSCode still runs on it (old OS).


Add Ethernet Network Diagnostics Cluster definition

Problem
Add the Ethernet Network Diagnostics cluster based on the CHIP-spec.
The specification of Ethernet Network Diagnostics Cluster:
https://github.com/CHIP-Specifications/connectedhomeip-spec/blob/master/src/service_device_management/DiagnosticsEthernet.adoc
To help you review the code, I put all manual added code into the first commit. The rest of  commits contain the changes either from the cluster GUI or codegen.

Summary of Changes
Add Ethernet Network Diagnostics Cluster definition

/rebase


This does not install

-bash: src/configure: No such file or directory
Braden:.rbenv bradynapier$ rm -fr ~/.rbenv
Braden:.rbenv bradynapier$ git clone https://github.com/rbenv/rbenv.git ~/.rbenv
fatal: Could not get current working directory: No such file or directory
Braden:.rbenv bradynapier$ touch ~/.rbenv
Braden:.rbenv bradynapier$ git clone https://github.com/rbenv/rbenv.git ~/.rbenv
fatal: destination path '/Users/bradynapier/.rbenv' already exists and is not an empty directory.
Braden:.rbenv bradynapier$
Braden:.rbenv bradynapier$ rm -f ~/.rbenv
Braden:.rbenv bradynapier$ git clone https://github.com/rbenv/rbenv.git ~/.rbenv
fatal: Could not get current working directory: No such file or directory
Braden:.rbenv bradynapier$
When trying to clone it says that directory doesnt exist... so I add it then it says error that file already exists... LOL!?

This appears to be a local issue with your system, not one with rbenv.
I hate to ask it (because I'll sound like a T1 at a mediocre network provider), but could you reboot (if you really can't, log out and back in again, and if that's not even possible, open a new terminal session, and please state which you did) your machine and provide the output of ls -Al.


Double services.AddMetricFactory() call

There is a double call of .AddMetricFactory() method in the WebHttpRequestDurations_3.1 exmample project.
Is it a mistake or intentional behavior?
        public void ConfigureServices(IServiceCollection services)
        {
            services.AddMetricFactory();
            services.AddControllers();
            services.AddMetricFactory();
        }

Fixed this line


Add set_angle 'immediate' parameter.

Fixes #340
Modified fade_in transition in main.gd to happen between :setup and :ready so that :ready don't start before fade_in is finished.

Although I'm effectively off Escoria for business reasons, I've been skimming
the emails occasionally.
I think I'm responsible for the :setup and :ready distinction, and the
intent wasn't to have anything trigger only for the first time. A global
variable can be used for that.
The big issue was how telon kept going so fast, that moving NPCs from
their original initial position to where they're now supposed to be was
done after the room had become visible.
So I added options to do things like keep the room black (eg. lights
being out) and such.
Of course as much as I still love Godot + Escoria, and appreciate your
work in it from a distance, you're free to redefine the semantics in
the rewrite. To me it made sense that you have a BTS :setup and then
action when :ready.
On Thu, Aug 05, 2021 at 11:43:41AM -0700, Dennis Ploeger wrote:

@dploeger commented on this pull request.

In [1]addons/escoria-core/game/core-scripts/esc/commands/change_scene.gd:

               if script.events.has("setup"):

                            escoria.event_manager.queue_event(script.events["setup"])
                            var rc = yield(escoria.event_manager, "event_finished")
                            while rc[1] != "setup":
                                    rc = yield(escoria.event_manager, "event_finished")
                            if rc[0] != ESCExecution.RC_OK:
                                    return rc[0]

Maybe we should add examples to the [2]event documentation about :setup
and :ready. Based on your description I don't get why ready should only be
called the first time the player visits a scene.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, [3]view it on GitHub, or [4]unsubscribe.
Triage notifications on the go with GitHub Mobile for [5]iOS or
[6]Android.
References
Visible links

https://github.com/godotengine/escoria/pull/357#discussion_r683703129
https://github.com/godotengine/escoria/blob/main/docs/esc.md#events
https://github.com/godotengine/escoria/pull/357#discussion_r683703129
https://github.com/notifications/unsubscribe-auth/AAATEWYYD2HURFE4AIAGMP3T3LLV3ANCNFSM5BR7AA7A
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
https://play.google.com/store/apps/details?id=com.github.android&utm_campaign=notification-email

Thanks for your contribution here, @mjtorn. That definition is much more coherent than the one I got from the previous code! I was actually only guessing there 😀
@StraToN based on this, do you want to remove the code that stores "readied" scenes and checks in this pr as well and correct the esc documentation part?

@mjtorn Thanks for the precision here! That's very helpful.
@dploeger

@StraToN based on this, do you want to remove the code that stores "readied" scenes and checks in this pr as well and correct the esc documentation part?

I'd rather do this in another dedicated PR, to avoid unclear commits.


Changing the button text

@praveenkumar-outlook , It's really helpful and valuable. but is there any trick to change the download button text or style?

Hi @buglessir , Thank you for using react-json-excel.
To answer your question,

To change download button text, there is a prop named "text"


Conv1D Example

Add Conv1D example with simple Sequential Model and input_shape

Add line to import Sequential and Conv1D Layer

I compared this to other examples, where it wasn't used. Is using imports canonical then?


fix the footer background spill up at 600px width

The footer background spills up and beyond the triangle mask by a few pixels when the website is around 600 pixels wide. This commit fixes that. I'm using hugo 0.80.0 on ubuntu to build.


Parallel of Spring Config Server in Kubernetes

This is just a question.
We are using Spring Config Server today, We have multiple apps, environments etc. Configs for which are activated using spring application name and spring profile active arguments.
What is the best approach that we can take to have the same flexibility over Kubernetes.
Config Map is definitely an option, but if we use fabric8 maven plugin to deploy our apps into a Kubernetes cluster, and we need different configs for different environment, how do we inject configs per environment based on spring profiles.
Any POC or a github repository would be useful.
Thanks in advance.
Cheers !!

I am also looking for a solution in which configMap properties are loaded based on the activated profile like dev ,prod.
@deepank-sharma Please post if you have got any solution. Thanks in advance.


comments on revisions requires page reload

This one is subtle but consistent and can be confusing.
If I review a suggested revision to an evidence item and make a comment.  When I submit the comment it appears.  I then accept or reject the revision and everything seems fine.  But if I navigate away somewhere and come back to this revision, my comment appears to be gone.  Until I do a page reload, then I see it again.

Nice catch. Probably I'm missing a cached list of comments that I need to flush when a comment is submitted on a suggested revision.

This appears to be fixed now.


support backups on replicas

in order to support gh-ost's need for a MySQL binlog, we want to optionally enable 1-day backups on a replica.

whoops, wrong repo. Actually happy to push here or our fork. you tell me, @dgtized

Happy to accept pull requests, would much prefer knowledge that current version is in active use, but pretty sure my version is ahead of your fork and slightly diverged. Specifically that the current version here also supports exporting a minimal IAM policy to allow stack-mitosis to run, which might be useful on your end, but on your fork it looks like you hardcoded a different timeout (which I intended to add as a flag at some point).
With regards to the requested feature, by default stack-mitosis enables backups on every replica that also has children. However, it can trivially enable backups on replicas if the original replica already has backups enabled. I believe that should be added to https://github.com/dgtized/stack-mitosis/blob/master/src/stack_mitosis/lookup.clj#L115 and then remove the special case in https://github.com/dgtized/stack-mitosis/blob/master/src/stack_mitosis/planner.clj#L58. That behavior is used now to transfer any existing modifications to any instance in the stack to any copies. If that behavior is also applied for BackupRetentionPeriod, then any replicas that have backups before the clone will have it after as well, even if they don't strictly require it for the clone operation.
Let me know how you would like to proceed.

Also I think putting it in post-create-replica-attributes would also correctly copy custom values of BackupRetentionPeriod like 2, or 30. So I think that's better anyway.


TPM udev rules for iotedge security daemon setup incomplete

Please change the documentation to incorporate the extra field in the udev rules listed under Solution.
Problem
I was receiving an error while starting up the IoT Edge Security Daemon, while using a TPM to authenticate with a DPS.
iotedged[3925]: 2020-05-08T16:22:34Z [INFO] - Starting Azure IoT Edge Security Daemon
iotedged[3925]: 2020-05-08T16:22:34Z [INFO] - Version - 1.0.9
iotedged[3925]: 2020-05-08T16:22:34Z [INFO] - Using config file: /etc/iotedge/config.yaml
iotedged[3925]: 2020-05-08T16:22:34Z [INFO] - Configuring /var/lib/iotedge as the home directory.
iotedged[3925]: 2020-05-08T16:22:34Z [INFO] - Configuring certificates...
iotedged[3925]: 2020-05-08T16:22:34Z [INFO] - Transparent gateway certificates not found, operating in quick start mode...
iotedged[3925]: 2020-05-08T16:22:34Z [INFO] - Finished configuring provisioning environment variables and certificates.
iotedged[3925]: 2020-05-08T16:22:34Z [INFO] - Initializing hsm...
iotedged[3925]: 2020-05-08T16:22:34Z [INFO] - Finished initializing hsm.
iotedged[3925]: 2020-05-08T16:22:34Z [INFO] - Provisioning edge device...
iotedged[3925]: 2020-05-08T16:22:34Z [INFO] - Starting provisioning edge device via TPM...
iotedged[3925]: 2020-05-08T16:22:34Z [INFO] - Starting DPS registration with scope_id "xxxxxxx", registration_id "l337h@xBr0_fosho"
iotedged[3925]: 2020-05-08T16:22:49Z [WARN] - Could not provision device
iotedged[3925]: 2020-05-08T16:22:49Z [WARN] -         caused by: DPS registration failed because the DPS operation is not assigned
iotedged[3925]: 2020-05-08T16:22:49Z [ERR!] - The daemon could not start up successfully: Could not initialize DPS provisioning client
iotedged[3925]: 2020-05-08T16:22:49Z [ERR!] -         caused by: Could not restore previous provisioning result
iotedged[3925]: 2020-05-08T16:22:49Z [ERR!] -         caused by: No such file or directory (os error 2)

OS info:
  Operating System: Ubuntu 18.04.4 LTS
            Kernel: Linux 4.15.0-99-generic
      Architecture: x86-64

Solution
Changing the udev rules to include the OWNER attribute solved my issue
Before
KERNEL=="tpm0", SUBSYSTEM=="tpm", GROUP="iotedge", MODE="0660"

After
KERNEL=="tpm0", SUBSYSTEM=="tpm", OWNER="iotedge", GROUP="iotedge", MODE="0660"

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

ID: a2113dfd-71ce-f982-35d2-528ec73c6a66
Version Independent ID: 5f043c8f-03ed-5d95-ec2d-4a02d80ecaec
Content: Provision device with a virtual TPM on Linux VM - Azure IoT Edge
Content Source: articles/iot-edge/how-to-auto-provision-simulated-device-linux.md
Service: iot-edge
GitHub Login: @kgremban
Microsoft Alias: kgremban

@SpaceKatt Thanks for bringing this to our notice. We are investigating into the issue and will update you shortly.

Yes, this request is correct. There are a few things that have gone wrong here.
The iotedge group is meant for ACLing the management socket, so strictly speaking the iotedged process's user is not required to run in that group. That's why the Debian / Ubuntu packages don't add the iotedge user to the iotedge group.
But the CentOS packages do add the user to the group. This is inconsistent. Furthermore, the service file for all distros set User=iotedge and Group=iotedge, even though that user isn't in that group.
Coming back to this doc, both this doc and the README-TPM.md in the iotedge repo suggest using the group in the udev rule. The rule ought to have OWNER since the iotedge user needs to access the device node. It doesn't strictly need to have GROUP at all, since the other users in the iotedge group don't need to access the device node.
So it should just be KERNEL=="tpm0", SUBSYSTEM=="tpm", OWNER="iotedge", MODE="0600" - note that only OWNER is set and that the MODE is 0600
(Of course, if the user decides they do need some other users to be able to access the tpm, they can make a new group themselves, and add a GROUP and modify the MODE on their own.)

@SpaceKatt From your thumbs-up, I take it that you approva of arsing's alternative update to just using the OWNER rule instead of GROUP and OWNER. I've updated the docs to reflect this, so will close this issue now. If you have any further questions or comments, please feel free to post here and we can reopen the issue if necessary.
Thanks for bringing this to our attention!
#please-close


Is there a way to format output to remove "(Official Video)" and "(Audio)" from title?

DO NOT REMOVE OR SKIP THE ISSUE TEMPLATE

[X] I understand that I will be blocked if I intentionally remove or skip any mandatory* field

Checklist

[X] I'm asking a question and not reporting a bug or requesting a feature
[X] I've looked through the README
[X] I've verified that I'm running yt-dlp version 2023.03.04 (update instructions) or later (specify commit)
[X] I've searched known issues and the bugtracker for similar questions including closed ones. DO NOT post duplicates
[X] I've read the guidelines for opening an issue

Please make sure the question is worded well enough to be understood
Is there a way to format output so that the words (Official Video) and (Audio) in content downloaded from youtube and being converted to mp3 from both the filename and metadata?
Provide verbose output that clearly demonstrates the problem

[ ] Run your yt-dlp command with -vU flag added (yt-dlp -vU <your command line>)
[ ] If using API, add 'verbose': True to YoutubeDL params instead
[ ] Copy the WHOLE output (starting with [debug] Command-line config) and insert it below

Complete Verbose Output
No response

See https://github.com/yt-dlp/yt-dlp#modifying-metadata

so is '--replace-in-metadata  " (Official Video)|(Audio)" "" '   sufficient?   I'll be adding to the command line parms in the metube docker container.

@famewolf asked "Is there a way to format output […] from both the filename and metadata" ?
Is there a way to only change the filename without changing the metadata ?
Should I open a new issue for this related question ?


update PBMTL,PGTML,PB0(bads),resample obs

fixes #5 some of #4

@aappling-usgs key addition is here


Regression failure upgrading from 2.5.0 to 2.5.1+, webpack fails to transpile new export statement for ES5.

We are attempting to upgrade to the latest version of tslib but ran into an issue with IE11.  We are targeting ["web", "es5"] in our webpack.config.js.  We are also have importHelpers set to true in our tsconfig.json.  With this configuration IE11 no longer runs in webpack development mode.
It appears that the breaking change was made in tslib 2.5.1. The following export was added to tslib.es6.mjs:
export default {
  __extends,
  __assign,
  __rest,
  __decorate,
  __param,
  __metadata,
  __awaiter,
  __generator,
  __createBinding,
  __exportStar,
  __values,
  __read,
  __spread,
  __spreadArrays,
  __spreadArray,
  __await,
  __asyncGenerator,
  __asyncDelegator,
  __asyncValues,
  __makeTemplateObject,
  __importStar,
  __importDefault,
  __classPrivateFieldGet,
  __classPrivateFieldSet,
  __classPrivateFieldIn,
};

This ES6 style of export is not transpiled by webpack when running in development mode and which causes the issue in IE11.  The issue does not occur when running in production mode.  One workaround is to manually include the es5 compliant tslib.js file on our pages and take webpack out of the picture.

If you're targeting es5 in your webpack config, then this file should be getting downleveled in syntax by webpack or babel or similar; this file is .mjs after all and is intended to contain ESM syntax like the above. This (to me) sounds like a bug in webpack or an incorrect configuration that isn't downleveling all of the files in a build.

Thank you so much for this issue!
I had to switch devices and was unable to reproduce an IE11-compatible build on my new machine.
Now I really know to appreciate the difference between "tslib": "^2.5.0" and "tslib": "2.5.0"  👍


49-Mile Scenic Drive shield

Added a shield for the 49-Mile Scenic Drive in San Francisco. This route includes segments of I-280 and U.S. 101, so it’s actually a bit more prominent on the map than its name would imply.

This shield image is based on a public domain graphic on Wikimedia Commons. It departs from this style’s standard blue color because the sky has to be a light blue to be recognizable. To ensure compatibility with the rest of the MUTCD-inspired color scheme, I reused the color of the blue jeans in this standard California MUTCD sign. I think this color will work well in other shields that depict the sky or anything else that’s much lighter than the standard blue. At high zoom levels, it has just enough contrast from water to avoid confusion with nearby ponds. I also took the custom orange from the same sign, which is intended to complement the light blue, but if desired, I can switch it to the standard orange, which is slightly lighter.
The original shield features a the seagull and the number 49 very prominently. I chose to omit the 49 because it would force us to either reduce the seagull’s head to the point of being illegible or expand the shield horizontally, giving it an unwieldy shape. A white 49 would also contrast poorly with the light blue I’ve chosen, which is slightly lighter than what’s on the actual signs.
Fixes #570.

I tried the orange from the existing color palette, and I agree that the original orange is better in this case.
I was worried about the light blue contrasting against the white, though maybe a black border looks too jarring:

The standard blue seems less jarring, and somehow less constraining too:


Missing area filter

Codecov Report

Merging #35 into master will decrease coverage by 3.77%.
The diff coverage is 20.45%.

@@            Coverage Diff             @@
##           master      #35      +/-   ##
==========================================
- Coverage   65.48%   61.71%   -3.78%
==========================================
  Files          14       14
  Lines         481      525      +44
==========================================
+ Hits          315      324       +9
- Misses        166      201      +35

Impacted Files
Coverage Δ

src/problemgenerator/filters.py
35.52% <20.45%> (-20.73%)
:arrow_down:

Continue to review full report at Codecov.

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


Use the normal devnet instead of our custom version

Currently we use a fork of starknet-devnet, with changes to the account.
We have added the functionality to deploy a new contract from the smart contract account, fork of OpenZeppelin's account contract.
With newer version of starknet-devnet, there is a cli argument to use custom implementation of account contracts, we should sunset our fork and use the new version with our custom account contract

closed in #38


Unexpected behavior with up arrow when newlines in history

In a basic Zsh install with nothing in my .zshrc, if I enter commands like the following:
% ls
% (
subsh> ls
subsh> )
% cd
%

and then push the up arrow repeatedly (with the prompt initially empty), I get shown cd, then the subshell with each line in turn, and then ls.
However if I do the same when using oh-my-zsh I get shown cd, then go through the subshell lines, and then when the ( is reached further presses of the up arrow go through any previous lines containing just ( (or does nothing if none of these exist).
To me at least this seems unexpected and I would expect it to work more like the zsh default. If I'm pressing up starting from an empty history I would expect to be shown each of my previous commands in turn rather than each up to a certain point and then not be able to view further.
I would guess this is to do with oh-my-zsh searching history based on the first input word but I'm not exactly sure what's causing it. I've also noticed the same thing in other situations where a command takes up multiple lines, which is why I think it's to do with the newlines in the history.

That is unexpected, and probably not what' s intended.
I can't reproduce this. On OS X 10.9/zsh 5.0.2 with OMZ loaded (with plugins=( osx themes nyan sublime brew vagrant emoji ) ), if I do this:
➜  ~  ls
AT.postflight.11860  Downloads  Movies    Public    local    stats
Desktop              Dropbox    Music     archives  luggage  tmp
Documents            Library    Pictures  bin       misc     var
➜  ~  (
subsh> echo foo
subsh> echo bar
subsh> )
foo
bar
➜  ~  cd

then when I use the up arrow, repeated presses show me:

cd
The

(
echo foo
echo bar
)

multi-line command, as a single history entry

ls

Maybe it's some additional configuration you have, or a different plugin? Could you run omz_diagnostic_dump (available in the latest OMZ update), and post the output as a gist?

Hi, thanks for checking this out. I thought this was probably something to do with some atypical configuration of mine or similar; to try and avoid this I tried the same in a fresh VM, with the same result. Here's the result of omz_diagnostic_dump in that VM: https://gist.github.com/BobWhitelock/219dc5924cc1eae8372e.

Okay, I can reproduce on an up-to-date non-Vagrant Ubuntu 14.04 VM (in VMware Fusion running on OS X 10.9). Works fine with normal zsh 5.1.1 and OMZ, but when I copied your .zshrc from that diagnostic dump over to mine, then I got the bad behavior. (Maybe I had custom settings that were avoiding it earlier.) Will look in to it more when I have a chance.

I recently ran into the same problem, what solved it for me was adding export LANG='en_US.UTF-8' at the beginning of .zshrc.

Can reproduce and @erlendfh's solution doesn't work. With zsh 5.1.1 and these two key bindings:
bindkey ^[OA up-line-or-search
bindkey ^[OB down-line-or-search

where ^[OA and ^[OB are the character sequences for up and down arrow (YMMV).
Unfortunately I think that's the documented behavior, see up-line-or-search:

Move up a line in the buffer, or if already at the top line, search backward in the history for a line beginning with the first word in the buffer.

You can use up-line-or-beginning-search which does the same but puts the cursor in the first line, therefore not triggering your use case.


Send receive reports

Description & motivation

Changes:

To-do before merge:

Screenshots:

Validation of changes:

Checklist:

[ ] My pull request follows the guidelines in the Contributing guide?
[ ] My pull request does not duplicate any other open Pull Requests for the same update/change?
[ ] My commits are related to the pull request and do not amend unrelated code or documentation.
[ ] My code follows a similar style to existing code.
[ ] I have added appropriate tests.
[ ] I have updated or added relevant documentation.

References

linked with https://github.com/specklesystems/speckle-server/pull/2288


Could not login using iOS example project

I have run example project from SDKSample. The code in the class IdentityViewController in the block never executes:

_platformManager.accountsPromise.then(^{
// ... code
});

So after success authentication to Microsoft account, _transitionToMainViewController method does not execute.

I've the same problem.
To correct that, on the func - (AnyPromise*)signInMsaAsync {
You need to remove the return on MCDConnectedDevicesPlatform: 468:
    MSAAccount* msaAccount = [[MSAAccount alloc] initWithClientId:CLIENT_ID scopeOverrides:@{}];

    return [AnyPromise promiseWithAdapterBlock:^(PMKAdapter _Nonnull adapter) {
        [msaAccount signInWithCompletionCallback:adapter];
    }].then(^{
        Account* account = [[Account alloc] initWithMSAAccount:msaAccount platform:self.platform apnsManager:self.apnsManager];
        account.state = AccountRegistrationStateInAppCacheOnly;
        [self.accounts addObject:account];
        [account prepareAccountAsync:self];
    }).then(^{
        [self accountListChanged];
    });
}


Sidebar minor UI improvements

I reviewed my formatting settings. Prettier is configured as needed but still see a mismatch. Can you share your settings.json?


Change the work display to show search result Files in Work (like collection show)

child of #996
Use the code the collection for showing works in a collection to show FileSets in the File.
Should update the Work show presenter to get the files out of the new predicate.
Does include sorting by title alphabetically by default.  Does not include visual ability to sort by title.

Focus of the project has changed and we are moving to a new ScholarSphere that may not be dependent on the current technology stack.


Add travis to hopper

make test-all fails for me, because it is not able to find a module called 'config'. Also, isort fails for 2 files. Odly enough: the imports are orderes properly!

I added a .isort.cfg file and also added a PYTHONPATH to tox.ini! Now the testsuite works flawless local and on travis as well.


Create a contact info block

Add links to social networks (LinkedIn, Facebook, Instagram), phone number, email, Telegram username, etc.

We will include links to our social networking profiles such as LinkedIn, Facebook, and Instagram, allowing others to connect with us on these platforms. Additionally, we will provide contact details such as a phone number, email address, and Telegram username to facilitate direct communication.


HTTPS is not supported

Hello,
I'm trying to use this module to send form data over HTTPS. Below you can see my code, which is similar to the test code and the error I'm receiving back.
Code:
var form = new FormData();
 form.append('file1', req.frontBuffer);
 form.append('file2', req.backBuffer);

 form.submit({
     headers: {'Authorization': 'LicenseKey xxxxxxx'},
     protocol: 'https',
     host: 'hostnamehere.com',
     method: 'post',
     path: '/Process/False',
     port: 443
     }, function(err, res) {
     if (err)
     {
         console.error('Error:' + err);
     }
     console.log(res.statusCode);
     res.on('data', function(chunk) {
         console.log('body:' + chunk);
     });
     console.log('\r\n\r\n');
     console.log(form);
 });

Error:
Error: Protocol "https" not supported. Expected "http:"
    at new ClientRequest (_http_client.js:55:11)
    at Object.exports.request (http.js:31:10)
    at FormData.submit (C:\Users\kylec\Documents\GitHub\project\node_modules\form-data\lib\form_data.js:411:20)

SOLUTION:
For anyone else struggling, the protocol is https: not https

Yeah, this is node thing, http.request expects it like that - https://nodejs.org/docs/latest-v4.x/api/http.html#http_http_request_options_callback
and url.parse returns it that way – https://nodejs.org/docs/latest-v4.x/api/url.html#url_url_parsing


RegionInput component

RegionInput
Overview / Summary
Summary description of UI component
The RegionInput component will render the same code as currently in AddressForm, where based on regionOptions length, either Select or TextInput is shown.
Rationale for why this component is necessary
There are forms outside of address entry where we need a region to be selected or entered.
Expected use cases
Region entry for shipping rules
React State
None
Data Fetching
None
State Store
None
Routing / Query String
None
Render Criteria
Based on regionOptions length, either Select or TextInput is shown. Use same props for these components as in AddressForm, unless any of them are specific to being used within the address form, in which case they should be passed in.
Breakpoints
Desktop, Tablet, Mobile
N/A
UI States
This is a simple switcher component, so UI states are managed by the child components.
Work

Create the component and add it to context
Update AddressForm to use it from components prop

:tada: This issue has been resolved in version 0.54.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Prevent use of ImmutAfterInit in the multithreaded environment

The initialization of ImmutAfterInitCell objects (and, by extension, ImmutAfterInitRef objects) is designed specifically only to safe when the system is still in a single-threaded environment, because the intialization checks are not thread-safe.  This change ensures that no attempt is made to initialize or reinitialize any of these objects once the system has started multiple processors.  These checks are debug-only, like the checks on initialization state.

Should we perhaps use Ordering::SeqCst when loading/storing into MULTI_THREADED? It should have no impact before bringing up the secondary CPUs anyway.

Should we perhaps use Ordering::SeqCst when loading/storing into MULTI_THREADED? It should have no impact before bringing up the secondary CPUs anyway.

Since this is just for debug checking, the ordering applied to these atomics is irrelevant.  Ordering rules simply define which values are observable before/after which other values in a weakly ordered memory system.  The whole point of this change is not to guarantee consistency of data, but simply to detect whether we have passed the point of multi-threaded init.  In other words, since ImmutAfterInitCell::init() is expressly designed NOT to be thread-safe, no changes to ordering rules could improve thread safety of its use, so sticking with relaxed ordering is the most sensible.

For some reason this triggers a boot failure on my side when bringing up secondary CPUs:
[SVSM] Launching AP with APIC-ID 21
[SVSM] Launching request-processing task on CPU 20
[SVSM] ERROR: Panic: CPU[0] panicked at kernel/src/cpu/smp.rs:38:14:
Failed to launch secondary CPU: Ghcb(VmgexitError(2, 5))
[SVSM] ---BACKTRACE---:
[SVSM]   [0xffffff800000c8f1]
[SVSM]   [0xffffff800000cb15]
[SVSM]   [0xffffff8000003738]
[SVSM]   Invalid frame
[SVSM] ---END---

KVM complains about a 2M-aligned VMSA:
kvm_amd: kvm [36813]: vcpu0, guest rIP: 0x0 vmgexit: AP VMSA address [8000c00000] from guest is unsafe as it is 2M aligned

Not sure yet how that can happen, as the VMSA allocation code makes sure the VMSA is not 2M-aligned.

For some reason this triggers a boot failure on my side when bringing up secondary CPUs:
[SVSM] Launching AP with APIC-ID 21
[SVSM] Launching request-processing task on CPU 20
[SVSM] ERROR: Panic: CPU[0] panicked at kernel/src/cpu/smp.rs:38:14:
Failed to launch secondary CPU: Ghcb(VmgexitError(2, 5))
[SVSM] ---BACKTRACE---:
[SVSM]   [0xffffff800000c8f1]
[SVSM]   [0xffffff800000cb15]
[SVSM]   [0xffffff8000003738]
[SVSM]   Invalid frame
[SVSM] ---END---

KVM complains about a 2M-aligned VMSA:
kvm_amd: kvm [36813]: vcpu0, guest rIP: 0x0 vmgexit: AP VMSA address [8000c00000] from guest is unsafe as it is 2M aligned

Not sure yet how that can happen, as the VMSA allocation code makes sure the VMSA is not 2M-aligned.

That's odd, I can boot a guest kernel fine with and without this PR.

That's odd, I can boot a guest kernel fine with and without this PR.

I seems not to be related to this PR. I found an issue where sometimes there is an off-by-one-page bug between the virtual and physical addresses where the heap starts. That causes the offset to not be 2M aligned anymore and the SVSM might allocate a 2m aligned virtual address which is 2m aligned physical. The bug originates somewhere in stage2, I am still investigating.

The issue has been fixed by PR #301


Can you please update the pod spec to include the latest?

Can you please  update the pod spec to include the latest?

Please Update the pod spec to reflect the changes on master . The pod version is crashing on loadView method. .
thanks

Done! Sorry for the delay

Thanks


Cool changes!

Replaced pygame.display.set_mode's values with the variables height and width, this should pave the way for customizable dimensions. Also added a isOffscreen function, useful for games where you don't want your character going off screen!

Oh yeah, I'm applab from scratch.

Looks good to me! I'll merge it now.


Tests fail with mono 5

When building fsharp with mono 5.0 build succeeds but test fail
Repro steps

Build and install fsharp from source

run tests/fsharp/core/run-opt.sh

Expected behavior
All tests succeed
Actual behavior
Several tests fail
Known workarounds
None
Related information

OS: Tested Ubuntu 16.04 x64 and Solus
Source built and tested: Release 4.1.18
Framework: Mono <IP_ADDRESS>

Full test output ouput.txt
Failures (as best as I can filter them out) failure.txt

@nosami All these failures appear to be to do with printing of Nullable values.  So it feels like something has changed with Mono on that?  There is probably a very simple repro using ToString or a basic reflection routine on Nullable values

Failures below
-------------------
Failures in queriesLeafExpressionConvert/test.fsx
    FAILED: test p2oin209v33x...failed, expected "(Convert(2) + new Nullable`1())" got "(Convert(2, Nullable`1) + new Nullable`1())"
    FAILED: test p2oin209v33x...failed, expected "(Convert(2) + Convert(3))" got "(Convert(2, Nullable`1) + Convert(3, Nullable`1))"
    FAILED: test p2oin209v33x...failed, expected "(Convert(2) + Convert(3))" got "(Convert(2, Nullable`1) + Convert(3, Nullable`1))"
    FAILED: test p2oin209v33x...failed, expected "(Convert(2) - new Nullable`1())" got "(Convert(2, Nullable`1) - new Nullable`1())"
    FAILED: test p2oin209v33x...failed, expected "(Convert(2) - Convert(3))" got "(Convert(2, Nullable`1) - Convert(3, Nullable`1))"
    FAILED: test p2oin209v33x...failed, expected "(Convert(2) - Convert(3))" got "(Convert(2, Nullable`1) - Convert(3, Nullable`1))"
    FAILED: test p2oin209v33x...failed, expected "(Convert(2) * new Nullable`1())" got "(Convert(2, Nullable`1) * new Nullable`1())"
    FAILED: test p2oin209v33x...failed, expected "(Convert(2) * Convert(3))" got "(Convert(2, Nullable`1) * Convert(3, Nullable`1))"
    FAILED: test p2oin209v33x...failed, expected "(Convert(2) * Convert(3))" got "(Convert(2, Nullable`1) * Convert(3, Nullable`1))"
    FAILED: test p2oin209v33x...failed, expected "(Convert(2) / new Nullable`1())" got "(Convert(2, Nullable`1) / new Nullable`1())"
    FAILED: test p2oin209v33x...failed, expected "(Convert(2) / Convert(3))" got "(Convert(2, Nullable`1) / Convert(3, Nullable`1))"
    FAILED: test p2oin209v33x...failed, expected "(Convert(2) / Convert(3))" got "(Convert(2, Nullable`1) / Convert(3, Nullable`1))"
    FAILED: test p2oin209v33x...failed, expected "(Convert(2) % new Nullable`1())" got "(Convert(2, Nullable`1) % new Nullable`1())"
    FAILED: test p2oin209v33x...failed, expected "(Convert(2) % Convert(3))" got "(Convert(2, Nullable`1) % Convert(3, Nullable`1))"
-------------------

-------------------
Failures in queriesOverIQueryable/test.fsx
    FAILED: test lqtcnewnc06yh9Q3...failed, expected
	"db.Join(db, i => i.Quantity, j => Convert(j.Quantity.GetValueOrDefault()), (i, j) => new AnonymousObject`2(Item1 = i, Item2 = j)).Select(_arg1 => (_arg1.Item1.Cost + _arg1.Item2.Cost))"
got
	"db.Join(db, i => i.Quantity, j => Convert(j.Quantity.GetValueOrDefault(), Nullable`1), (i, j) => new AnonymousObject`2(Item1 = i, Item2 = j)).Select(_arg1 => (_arg1.Item1.Cost + _arg1.Item2.Cost))"
    FAILED: test ltcnewnc06yh9Q6...failed, expected
	"db.GroupJoin(db, i => i.Quantity, j => Convert(j.Quantity.GetValueOrDefault()), (i, group) => new AnonymousObject`2(Item1 = i, Item2 = group)).Select(_arg1 => _arg1.Item2)"
got
	"db.GroupJoin(db, i => i.Quantity, j => Convert(j.Quantity.GetValueOrDefault(), Nullable`1), (i, group) => new AnonymousObject`2(Item1 = i, Item2 = group)).Select(_arg1 => _arg1.Item2)"
    FAILED: test ltcnewnc06yh9Q7...failed, expected
	"db.GroupJoin(db, i => Convert(i.Quantity.GetValueOrDefault()), j => j.Quantity, (i, group) => new AnonymousObject`2(Item1 = i, Item2 = group)).Select(_arg1 => _arg1.Item2)"
got
	"db.GroupJoin(db, i => Convert(i.Quantity.GetValueOrDefault(), Nullable`1), j => j.Quantity, (i, group) => new AnonymousObject`2(Item1 = i, Item2 = group)).Select(_arg1 => _arg1.Item2)"
    FAILED: test ltcjhnwec7eweww2...failed, expected
	"[1].Where(item => (item Is Item)).Select(item => Convert(item))"
got
	"[1].Where(item => (item Is Item)).Select(item => Convert(item, Item))"
-------------------

FAILURE: queriesLeafExpressionConvert/test.fsx - test p2oin209v33x...failed, expected "(Convert(2) + new Nullable`1())" got "(Convert(2, Nullable`1) + new Nullable`1())"
FAILURE: queriesLeafExpressionConvert/test.fsx - test p2oin209v33x...failed, expected "(Convert(2) + Convert(3))" got "(Convert(2, Nullable`1) + Convert(3, Nullable`1))"
FAILURE: queriesLeafExpressionConvert/test.fsx - test p2oin209v33x...failed, expected "(Convert(2) + Convert(3))" got "(Convert(2, Nullable`1) + Convert(3, Nullable`1))"
FAILURE: queriesLeafExpressionConvert/test.fsx - test p2oin209v33x...failed, expected "(Convert(2) - new Nullable`1())" got "(Convert(2, Nullable`1) - new Nullable`1())"
FAILURE: queriesLeafExpressionConvert/test.fsx - test p2oin209v33x...failed, expected "(Convert(2) - Convert(3))" got "(Convert(2, Nullable`1) - Convert(3, Nullable`1))"
FAILURE: queriesLeafExpressionConvert/test.fsx - test p2oin209v33x...failed, expected "(Convert(2) - Convert(3))" got "(Convert(2, Nullable`1) - Convert(3, Nullable`1))"
FAILURE: queriesLeafExpressionConvert/test.fsx - test p2oin209v33x...failed, expected "(Convert(2) * new Nullable`1())" got "(Convert(2, Nullable`1) * new Nullable`1())"
FAILURE: queriesLeafExpressionConvert/test.fsx - test p2oin209v33x...failed, expected "(Convert(2) * Convert(3))" got "(Convert(2, Nullable`1) * Convert(3, Nullable`1))"
FAILURE: queriesLeafExpressionConvert/test.fsx - test p2oin209v33x...failed, expected "(Convert(2) * Convert(3))" got "(Convert(2, Nullable`1) * Convert(3, Nullable`1))"
FAILURE: queriesLeafExpressionConvert/test.fsx - test p2oin209v33x...failed, expected "(Convert(2) / new Nullable`1())" got "(Convert(2, Nullable`1) / new Nullable`1())"
FAILURE: queriesLeafExpressionConvert/test.fsx - test p2oin209v33x...failed, expected "(Convert(2) / Convert(3))" got "(Convert(2, Nullable`1) / Convert(3, Nullable`1))"
FAILURE: queriesLeafExpressionConvert/test.fsx - test p2oin209v33x...failed, expected "(Convert(2) / Convert(3))" got "(Convert(2, Nullable`1) / Convert(3, Nullable`1))"
FAILURE: queriesLeafExpressionConvert/test.fsx - test p2oin209v33x...failed, expected "(Convert(2) % new Nullable`1())" got "(Convert(2, Nullable`1) % new Nullable`1())"
FAILURE: queriesLeafExpressionConvert/test.fsx - test p2oin209v33x...failed, expected "(Convert(2) % Convert(3))" got "(Convert(2, Nullable`1) % Convert(3, Nullable`1))"
FAILURE: queriesOverIQueryable/test.fsx - test lqtcnewnc06yh9Q3...failed, expected
	"db.Join(db, i => i.Quantity, j => Convert(j.Quantity.GetValueOrDefault()), (i, j) => new AnonymousObject`2(Item1 = i, Item2 = j)).Select(_arg1 => (_arg1.Item1.Cost + _arg1.Item2.Cost))"
got
	"db.Join(db, i => i.Quantity, j => Convert(j.Quantity.GetValueOrDefault(), Nullable`1), (i, j) => new AnonymousObject`2(Item1 = i, Item2 = j)).Select(_arg1 => (_arg1.Item1.Cost + _arg1.Item2.Cost))"
FAILURE: queriesOverIQueryable/test.fsx - test ltcnewnc06yh9Q6...failed, expected
	"db.GroupJoin(db, i => i.Quantity, j => Convert(j.Quantity.GetValueOrDefault()), (i, group) => new AnonymousObject`2(Item1 = i, Item2 = group)).Select(_arg1 => _arg1.Item2)"
got
	"db.GroupJoin(db, i => i.Quantity, j => Convert(j.Quantity.GetValueOrDefault(), Nullable`1), (i, group) => new AnonymousObject`2(Item1 = i, Item2 = group)).Select(_arg1 => _arg1.Item2)"
FAILURE: queriesOverIQueryable/test.fsx - test ltcnewnc06yh9Q7...failed, expected
	"db.GroupJoin(db, i => Convert(i.Quantity.GetValueOrDefault()), j => j.Quantity, (i, group) => new AnonymousObject`2(Item1 = i, Item2 = group)).Select(_arg1 => _arg1.Item2)"
got
	"db.GroupJoin(db, i => Convert(i.Quantity.GetValueOrDefault(), Nullable`1), j => j.Quantity, (i, group) => new AnonymousObject`2(Item1 = i, Item2 = group)).Select(_arg1 => _arg1.Item2)"
FAILURE: queriesOverIQueryable/test.fsx - test ltcjhnwec7eweww2...failed, expected
	"[1].Where(item => (item Is Item)).Select(item => Convert(item))"
got
	"[1].Where(item => (item Is Item)).Select(item => Convert(item, Item))"

@nosami This looks like some really minor difference in printing LINQ expression trees on Mono 5.0 and .NET Framework/Mono 4.x

@dsyme: just seen this, sorry... I really need to handle @mentions better :)
I'll investigate.

> Expression.Convert(Expression.Constant(Nullable 2), typeof<System.Nullable<int>>).ToString();;
val it : string = "Convert(2, Nullable`1)"

mono 4.8
> System.Linq.Expressions.Expression.Convert(System.Linq.Expressions.Expression.Constant(System.Nullable 2), typeof<System.Nullable<int>>).ToString();;
val it : string = "Convert(2)"

Bug filed https://bugzilla.xamarin.com/show_bug.cgi?id=57711

This actually seems to be by design in corefx (which Mono is now using) https://github.com/dotnet/corefx/blob/master/src/System.Linq.Expressions/src/System/Linq/Expressions/ExpressionStringBuilder.cs#L596

@nosami OK, thanks!

Thanks for looking at this @dsyme and @nosami. Any idea of when this is going to make it into an official release?

@rbnswartz For now we've just disabled these specific tests with Mono (which follows CoreFx behaviour) since the corresponding tests are run with .NET Framework and the differences are just cosmetic - there's no bug or change to F# here

@dsyme I was referring to the tests. I was hoping to run the tests that shipped with the release zip as part of the build process but I guess I could just run the tests from master if I had to.

@rbnswartz We can tag/release more or less straight away.  I did a direct commit to master and made a mistake, when it is green on Travis we will tag.

@dsyme Sounds good! Thanks again guys.

@rbnswartz No problem... I learned about nullable operators today while looking into this :)


Add a pull-request template so we almost always automatically run /lint.

We have these in other repos as well, so this makes things more even. it also includes running lint usually, to find out linting issues early.

/lgtm


Multi-GPU support

The demo notebook for multi-GPU support doesn't work anymore. The API has changed I presume;
https://github.com/clEsperanto/pyclesperanto/blob/main/demos/multi_gpu_demo.ipynb
The error is:
---------------------------------------------------------------------------
ImportError                               Traceback (most recent call last)
Cell In[1], line 2
      1 from skimage.io import imread
----> 2 from pyclesperanto import Clesperanto
      3 import time
      4 import numpy as np

ImportError: cannot import name 'Clesperanto' from 'pyclesperanto' (C:\Users\haase\mambaforge\envs\bio39\lib\site-packages\pyclesperanto\__init__.py)

Is something like this still supported?

Nope, this is a relicat of the old design.
Now import is following the same style as the prototype:
import pyclesperanto as cle

But how can have mutliple GPU-handlers at the same time then?

Ok, I just tried, it doesn't work. The device inside the Processor object is changed/overwritten:

Details: https://github.com/clEsperanto/pyclesperanto/blob/a28ad718eec8204fec4571bd743bc798c07e1cd8/demos/test_multigpu.ipynb

Okay :disappointed:
However you can still use it as follow:
buffer_rtx = cle.create( (100,100), device=cle.select_device('RTX'))
buffer_gtx = cle.create( (100,100), device=cle.select_device('GTX'))

The buffer will hold the correct device has this is handle in the C++ layer.
Same for the kernel call
cle.add_image_and_scalar(src, dst, scalar=1, device=cle.select_device('RTX'))

For the device managment, I kind of copied what was done in the prototype, see here. Feel free to make it better! :rocket:

update following

#69

Device selection
Multi device selection can be done as followed:
dev1 = cle.new_device('GTX')
dev2 = cle.new_device('RTX')

This approach is not fixed, we might change it in future for a better compatibility with the select_device() function.

Memory managment
Pushed images knows which device they are on. Operation on already pushed images will be automatically run on the same device, maintaining coherency.
Mixing Images from different devices will results in a kernel crash, we still need to introduce a runtime error management for this case.

https://github.com/clEsperanto/pyclesperanto/blob/main/demos/advance/multi_gpu_demo.ipynb

here is an example on how to use multiple GPU at same time


JBPM-5142: added test for generic task syntax checking

Cherry-pick of #280

Prepared for review.

+1


UNDERTOW-2221 Path param parsing not resumed correctly

Issue: https://issues.redhat.com/browse/UNDERTOW-2221
JBEAP issue: https://issues.redhat.com/browse/JBEAP-24415

Cc @fl4via


This page is out of date

The examples on this page are Azure Function V1 implementations that aren't necessarily compatible with Azure Function V2 implementations. (For example, functions with Event Hub input bindings are asynchronous, and aren't compatible with how a Service Bus output binding is defined here, as it uses out parameters).
This page should either be updated, or marked deprecated with a link to updated documentation.

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

ID: 9960577d-471e-c834-d87c-134a9de4cd9e
Version Independent ID: 4daa5650-7ac6-2a49-28e5-d114fa7cf14b
Content: Azure Functions Recipes - Service Bus
Content Source: sandbox/functions-recipes/service-bus.md
Service: functions
GitHub Login: @cecilphillip
Microsoft Alias: cephilli

+1


enable template config via msg.template for stored or generated templates

Before you hit that Submit button....
Please read our contribution guidelines
before submitting a pull-request.
Types of changes
What types of changes does your code introduce?
Put an x in the boxes that apply

[ ] Bugfix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)

If you want to raise a pull-request with a new feature, or a refactoring
of existing code, it may well get rejected if it hasn't been discussed on
the mailing list or
slack team first.
Proposed changes
Describe the nature of this change. What problem does it address?
Checklist
Put an x in the boxes that apply

[x] I have read the contribution guidelines
[x] For non-bugfix PRs, I have discussed this change on the mailing list/slack team.
[x] I have run grunt to verify the unit tests pass
[x] I have added suitable unit tests to cover the new/changed functionality

Coverage increased (+0.006%) to 86.329% when pulling 6c08225603e3d8425f746d958fa791fbc47d323b on rcruicks:master into 5e128f89f6f06565d53e631aeead56c0521fa709 on node-red:master.


Display Ledger incompatibility message for unsupported browsers

Currently we show the "Use Ledger" option in Firefox but upon click the page just refreshes. We should have a message that explains that it's not compatible.
https://github.com/leather-wallet/extension/assets/28991/d568b75f-1233-444f-9c89-b77db7d5a936

This is a regression, we used to have this.

I guess it doesn't make too much sense to have me assigned to this then? As we discussed yesterday, this just comes down to bringing a pre-existing modal back (for which the asset has already been updated)

@mica000 is the previous design in Figma somewhere for reference by chance?

I think it should be this one


feat: add forces to composer

Happy weekend everyone 😎
This pull request aims to support the use of forces in the experimental composer sub-package🤞
Please let me know if I have missed anything or have any questions!

Looks great!  Thank you!


EmrEtlRunner: bump AMI version in example config to 5.3.0

If we have to do all this testing, let's bump to the latest AMI at the same time...

Do you mean in the wiki? because it's already the case in eer.

Ah great, please split out into a dedicated commit to map onto this ticket...


Finding out if current request is a query or mutation at interceptor level

For my current backend, the client needs to send a special header to let the backend know if it's a mutation or a query, before the backend reads the content.
I've been trying to look into the source code and the only place I found this might be possible is DefaultHttpRequestComposer but it's final, and can't be overriden.
Also, been trying to do this in an interceptor, but I am not sure how to get this data from the request.
Thank you!

Hi!
Yes, HttpRequestComposer is a good way to do this. It is final but can still be used by composition like this:
private class MyHttpRequestComposer(url: String) : HttpRequestComposer {
    private val defaultHttpRequestComposer = DefaultHttpRequestComposer(url)

    override fun <D : Operation.Data> compose(apolloRequest: ApolloRequest<D>): HttpRequest {
        var httpRequest = defaultHttpRequestComposer.compose(apolloRequest)
        if (apolloRequest.operation is Mutation) {
            httpRequest = httpRequest.newBuilder()
                .addHeader("header-name", "header value")
                .build()
        }
        return httpRequest
    }
}

Then set it on your client like so:
    val apolloClient = ApolloClient.Builder()
        .networkTransport(
            HttpNetworkTransport.Builder()
                .httpRequestComposer(MyHttpRequestComposer("https://..."))
                .build()
        )
        .build()


Istio: multiple permission errors (bind/chown..)

What kind of request is this (question/bug/enhancement/feature request):
Bug
Steps to reproduce (least amount of steps as possible):
Upgrade Rancher to 2.3.2 with kubernetes 1.16.2. Enable Istio.
Result:
NAME                                      READY   STATUS             RESTARTS   AGE
istio-citadel-77659cfb8-4dwv8             1/1     Running            0          36s
istio-galley-87897f756-h864l              0/1     CrashLoopBackOff   2          37s
istio-init-delete-crd-10-8sstj            0/1     Completed          0          4s
istio-pilot-56866b7c5f-9swhx              1/2     Running            0          36s
istio-policy-687c45c5dc-xsk5n             2/2     Running            2          37s
istio-sidecar-injector-8565bfc879-prsph   0/1     CrashLoopBackOff   2          36s
istio-telemetry-6bd697dd4-mclrx           2/2     Running            1          37s
istio-tracing-79fbf487df-xfmd9            1/2     Error              2          36s
kiali-599d56ddf-pqn56                     0/2     CrashLoopBackOff   1          37s

~> k logs istio-galley-87897f756-h864l
...
2019-10-29T13:08:26.897820Z	fatal	validation	admission webhook ListenAndServeTLS failed: listen tcp :443: bind: permission denied

~> k logs kiali-599d56ddf-pqn56
2019/10/29 13:12:09 [emerg] 1#1: chown("/tmp/nginx", 101) failed (1: Operation not permitted)
nginx: [emerg] chown("/tmp/nginx", 101) failed (1: Operation not permitted)

Other details that may be helpful:
We have some projects with restrictive PSPs, but that shouldn't interfere with istio-system ns, right?
Environment information

Rancher version (rancher/rancher/rancher/server image tag or shown bottom left in the UI): 2.3.2
Installation option (single install/HA): HA

Cluster information

Cluster type (Hosted/Infrastructure Provider/Custom/Imported): Custom
Machine type (cloud/VM/metal) and specifications (CPU/memory): metal, >24CPU, >64GB
Kubernetes version (use kubectl version):

Server Version: version.Info{Major:"1", Minor:"16", GitVersion:"v1.16.2", GitCommit:"c97fe5036ef3df2967d086711e6c0c405941e14b", GitTreeState:"clean", BuildDate:"2019-10-15T19:09:08Z", GoVersion:"go1.12.10", Compiler:"gc", Platform:"linux/amd64"}

Docker version (use docker version):

sudo docker version
Client:
 Version:           18.06.2-ce
 API version:       1.38
 Go version:        go1.10.3
 Git commit:        6d37f41
 Built:             Sun Feb 10 03:47:56 2019
 OS/Arch:           linux/amd64
 Experimental:      false

Server:
 Engine:
  Version:          18.06.2-ce
  API version:      1.38 (minimum version 1.12)
  Go version:       go1.10.3
  Git commit:       6d37f41
  Built:            Sun Feb 10 03:46:20 2019
  OS/Arch:          linux/amd64
  Experimental:     false

regards,
strowi

Related issue - https://github.com/rancher/rancher/issues/22469

@strowi Do you have "restricted" PSP enabled at the cluster level ?

@sangeethah what do you mean at the cluster-level? We have multiple projects "ck.sys-privileged", "ck.dev-unprivileged", where we applied the psps.
Oh and the default "restricted"-psp seems to be the default.

@strowi If the PSP at the cluster level is set to "restricted" , we have a known issue with not being able to enable Istio which is tracked in #22469. There are workarounds mentioned in this issue , to get the Istio enabled successfully.

@sangeethah ahh thx;) i did a search but couldn't come up with anything here. Will follow that issue as well.


SPV Tracker GetSpendableCoins() Exception

Hey there,
I ran into this exception today regarding NBitcoin.SPV.Tracker.GetWalletTransactions(chain).GetSpendableCoins() source
The exception message: An item with the same key has already been added.
I haven't quite figured out what I did to raise the exception, but the message is clear as to what broke.
Thanks.
PS. Thank you for making this library, its fantastic.

Alright, so after a little tinkering, I was able to repeat the error.
I first built a transaction (with transaction builder) using the Coin elements found in GetSpendableCoins(). Then I used BroadcastTransactionAsync(myTx) to send the tBTC. I then waited a little while for the Tracker to pick up the transaction. It didn't, so I shutdown the program and tried rerunning it thinking it would pick up the transactions with fresh nodes. Instead, it crashed and I got the error I mentioned above.
I imagine it has something to do with the tracker picking up new wallet transactions and inserting the new wallet change outputs into the GetSpendableCoins() enum.
Cheers.

Thanks, I will try to reproduce.

I pushed version <IP_ADDRESS>. Can you Update-Package NBitcoin and see if the bug still exist ?

@JetJet13 did <IP_ADDRESS> fixed this issue ?

Yes. Since the update, the error has not come up when I tried to reproduce the exception. Thank you for taking the time to address this bug.
Cheers.


introduce nats transport

This PR introduces a new transport layer for inter-node connectivity based on NATS, in addition to the existing libp2p transport.
Integration Architecture
NATS is embedded into Bacalhau nodes. Each Orchestrator node (aka Requester node) also runs a NATS server and connects with other Orchestrator nodes to form a NATS cluster. Compute nodes on the other hand only run a NATS client and connect to any of the Orchestrator nodes to listen for job requests and publish their node info to the network.
With this approach, only the Orchestrator nodes need to be reachable, which simplies the network requirements for the Compute nodes. Also, the Compute nodes only need to know the address of only a single Orchestrator node, and will learn about the other Orchestrators at runtime. This simplifies bootstrapping, but also enables the Compute node to failover and reconnect to other Orchestrators at runtime.
For completeness, the Orchestrator nodes also run a NATS client to subscribe for Compute node info, publish job requests and listen for responses.
NATS Subjects
NATS follows subject based addressing where clients publish and subscribe to subjects, and the servers take care of routing of those messages. These are the current subject patterns used to orchestrate jobs:

node.compute.<node-id>.*
Every compute node subscribes to a topic in the format of node.compute.<node-id>.* to handle job requests from the orchestrator, such as AskForBid and BidAccepted. For example, if the Orchestrator selects node QmUg1MoAUMEbpzg6jY45zpfNo2dmTBJS6CzZCTMbniYsvC to run a job, it will send a message to node.compute.QmUg1MoAUMEbpzg6jY45zpfNo2dmTBJS6CzZCTMbniYsvC.AskForBid/1, which will land on the correct compute node. The suffix /1 is just for versioning in case we change the AskForBid message format

node.orchestrator.<node-id>.*
Similarly, Orchestrator nodes listen to a dedicate subject to handle compute callbacks, such as OnRunComplete and OnComputeFailure

node.info.<node-id>
Compute nodes periodically publish their node info to node.info.<node-id> subject, whereas Orchestrator nodes listen to node.info.* so their can handle node info from all compute nodes

Running with NATS
NATS is currently opt-in, and libp2p is still the default transport layer. To run a network with NATS, use the following commands:
# Run Orchestrator node
bacalhau serve --node-type=requester --use-nats

# Run a second Orchestrator node
bacalhau serve --node-type=requester --use-nats --cluster-peers=<HOST>:6222

# Run a compute node
bacalhau serve --node-type=compute --use-nats --orchestrators=<HOST>:4222

# Run devstack with NATS
bacalhau devstack --use-nats

Testing Done
Deployed and tested in development environment
Remaining Work

Logstream support for NATS transport
Auth, as currently any node can join the network
Devstack tests based on NATS and deployed as part of circleci checks

Maybe unrelated, but when I try the command below no ID is shown
bacalhau --api-host bootstrap.development.bacalhau.org node list

Maybe unrelated, but when I try the command below no ID is shown
bacalhau --api-host bootstrap.development.bacalhau.org node list

Yeah you are right. When calling node list using an older client, the node IDs will be empty since the older client relies on the value under PeerInfo.ID as the node id, which will be empty for NATS networks. I don't think this will be a blocker


Breakpoint set but not yet bound

I have followed the instructions on this page and the only thing I get when placing breakpoints is that the "Breakpoint set but not yet bound". Some information:

I tried both creating the project from the template dialogs in Visual Studio 2019, Version 16.5.4 - and from the command line by creating a new blazorwasm project.
I tried with both IIS Express and BlazorApp as debug targets.
WebAssembly Nuget versions are v3.2.1
Target framework appears to be .NET Standard 2.1
"Enable JavaScript debugging for ASP.NET (Chrome, Edge and IE)" is enabled.
In the template dialogs the .NET Core version is mentioned as 3.1.8.
Browser is Chrome.
Edge refuses to work, says it "Failed to launch debug adapter" with "ECONNREFUSED". Could it be related to my issues in Chrome?
Breakpoint is at "currentCount++" in Counter.razor from the example project.

Tried on two different computers, same issue and same parameters above.

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

ID: ec2f61c7-9f51-6735-e328-3c7b495d70c3
Version Independent ID: 4d06e565-282d-6387-aa90-cb2c2986d35a
Content: Debug ASP.NET Core Blazor WebAssembly
Content Source: aspnetcore/blazor/debug.md
Product: aspnet-core
Technology: aspnetcore-blazor
GitHub Login: @guardrex
Microsoft Alias: riande

Hello @StefanLundmark ... Open this for the product unit at ...
https://github.com/dotnet/aspnetcore/issues


I'm adding my info csv file to the student_info

It would be pretty cool to have my info here I guess. This fixes the lack of Mohammad Refat in this project.

awesome


CUDA 10 tensorflow installation

Hi,
I have NVDIA gtx 1080 and succesfully installed CUDA 10. I am having the following error.
Tensorflow installation does n ot warn me with anything but trying to import tensorflow I am getting the folllowing error.
`Python 3.6.5 |Anaconda, Inc.| (default, Apr 29 2018, 16:14:56)
[GCC 7.2.0] on linux
Type "help", "copyright", "credits" or "license" for more information.

import tensorflow as tf
Traceback (most recent call last):
File "/home/ayshine/anaconda3/lib/python3.6/site-packages/tensorflow/python/pywrap_tensorflow.py", line 58, in
from tensorflow.python.pywrap_tensorflow_internal import *
File "/home/ayshine/anaconda3/lib/python3.6/site-packages/tensorflow/python/pywrap_tensorflow_internal.py", line 28, in
_pywrap_tensorflow_internal = swig_import_helper()
File "/home/ayshine/anaconda3/lib/python3.6/site-packages/tensorflow/python/pywrap_tensorflow_internal.py", line 24, in swig_import_helper
_mod = imp.load_module('_pywrap_tensorflow_internal', fp, pathname, description)
File "/home/ayshine/anaconda3/lib/python3.6/imp.py", line 243, in load_module
return load_dynamic(name, filename, file)
File "/home/ayshine/anaconda3/lib/python3.6/imp.py", line 343, in load_dynamic
return _load(spec)
ImportError: libcublas.so.9.0: cannot open shared object file: No such file or directory

During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "", line 1, in
File "/home/ayshine/anaconda3/lib/python3.6/site-packages/tensorflow/init.py", line 22, in
from tensorflow.python import pywrap_tensorflow  # pylint: disable=unused-import
File "/home/ayshine/anaconda3/lib/python3.6/site-packages/tensorflow/python/init.py", line 49, in
from tensorflow.python import pywrap_tensorflow
File "/home/ayshine/anaconda3/lib/python3.6/site-packages/tensorflow/python/pywrap_tensorflow.py", line 74, in
raise ImportError(msg)
ImportError: Traceback (most recent call last):
File "/home/ayshine/anaconda3/lib/python3.6/site-packages/tensorflow/python/pywrap_tensorflow.py", line 58, in
from tensorflow.python.pywrap_tensorflow_internal import *
File "/home/ayshine/anaconda3/lib/python3.6/site-packages/tensorflow/python/pywrap_tensorflow_internal.py", line 28, in
_pywrap_tensorflow_internal = swig_import_helper()
File "/home/ayshine/anaconda3/lib/python3.6/site-packages/tensorflow/python/pywrap_tensorflow_internal.py", line 24, in swig_import_helper
_mod = imp.load_module('_pywrap_tensorflow_internal', fp, pathname, description)
File "/home/ayshine/anaconda3/lib/python3.6/imp.py", line 243, in load_module
return load_dynamic(name, filename, file)
File "/home/ayshine/anaconda3/lib/python3.6/imp.py", line 343, in load_dynamic
return _load(spec)
ImportError: libcublas.so.9.0: cannot open shared object file: No such file or directory
Failed to load the native TensorFlow runtime.
See https://www.tensorflow.org/install/install_sources#common_installation_problems
for some common reasons and solutions.  Include the entire stack trace
above this error message when asking for help.

`

@Ayshine, you need to build from sources to use cuda 10. If you want to use pip installation, you need to have cuda9.0 and cudnn7 installed. Please see installation instructions

Thanks, @joseortiz3. Trying to follow through your solution now. I have a little different configuration so I hope this would also work for me. I will get back here and update the issue after I completed my process.
~$ python --version
Python 3.6.5 :: Anaconda, Inc.
~$ nvcc --version
nvcc: NVIDIA (R) Cuda compiler driver
Copyright (c) 2005-2017 NVIDIA Corporation
Built on Fri_Nov__3_21:07:56_CDT_2017
Cuda compilation tools, release 9.1, V9.1.85
~$ cat /usr/local/cuda/version.txt
CUDA Version 10.0.130

ok wh'le runn'ng basel got this
ayshine@ayshine:~/tensorflow$ ./configure
/home/ayshine/bin/bazel: line 88: /home/ayshine/.bazel/bin/bazel-real: cannot execute binary file: Exec format error
/home/ayshine/bin/bazel: line 88: /home/ayshine/.bazel/bin/bazel-real: Success
Traceback (most recent call last):
File "./configure.py", line 1592, in
main()
File "./configure.py", line 1458, in main
check_bazel_version('0.15.0')
File "./configure.py", line 450, in check_bazel_version
curr_version = run_shell(['bazel', '--batch', '--bazelrc=/dev/null', 'version'])
File "./configure.py", line 141, in run_shell
output = subprocess.check_output(cmd)
File "/home/ayshine/anaconda3/lib/python3.6/subprocess.py", line 336, in check_output
**kwargs).stdout
File "/home/ayshine/anaconda3/lib/python3.6/subprocess.py", line 418, in run
output=stdout, stderr=stderr)
subprocess.CalledProcessError: Command '['bazel', '--batch', '--bazelrc=/dev/null', 'version']' returned non-zero exit status 1.

@samikama @joseortiz3 thanks for the help. I am going to downgrade from CUDA 10 to CUDA 9. Some other libraries I want to install does not have CUDA 10 version. But, I will keep in mind to come back installing tensorflow on CUDA 10 and make it work :)


CI failure: Test CNP Specs Test failed

Archive: test_results_master_511.tar.gz
Test flake seen in jenkins master build 511:
k8s-1.7.K8sValidatedServicesTest CNP Specs Test (from (k8s-1.7.xml))

Failing for the past 1 build (Since Failed#511 )
Took 4 min 25 sec.
add description
Stacktrace
/home/jenkins/workspace/cilium-ginkgo_cilium_master-QVKDPVJ2A6E272IRUX7X6DCB5FBLCQDKRTOMHPZS7ZECX3RM5C3A/src/github.com/cilium/cilium/test/ginkgo-ext/scopes.go:270
Was able to connect from reviews-v1-2690276606-qgcw0 to ratings:9080, but expected no connection: --2018-04-01 02:31:39--  http://ratings:9080/
Resolving ratings (ratings)... <IP_ADDRESS>
Connecting to ratings (ratings)|<IP_ADDRESS>|:9080... connected.
HTTP request sent, awaiting response... 200 OK
Length: unspecified
Saving to: 'index.html.1'

     0K                                                         118M=0s

2018-04-01 02:31:39 (118 MB/s) - 'index.html.1' saved [872]

--2018-04-01 02:31:39--  http://ratings:9080/
Resolving ratings (ratings)... <IP_ADDRESS>
Connecting to ratings (ratings)|<IP_ADDRESS>|:9080... connected.
HTTP request sent, awaiting response... 200 OK
Length: unspecified
Saving to: 'index.html.1'

     0K                                                         118M=0s

2018-04-01 02:31:39 (118 MB/s) - 'index.html.1' saved [872]

Expected
    <bool>: true
to be false
/home/jenkins/workspace/cilium-ginkgo_cilium_master-QVKDPVJ2A6E272IRUX7X6DCB5FBLCQDKRTOMHPZS7ZECX3RM5C3A/src/github.com/cilium/cilium/test/k8sT/Services.go:330

Relevant call: https://github.com/cilium/cilium/blob/f9000c886f1f1c29b7d2c2f8608c9d41278ac016/test/k8sT/Services.go#L459

A cursory glance at the test would suggest that after line 439 successfully applies the policy, it should be applied on all nodes:
https://github.com/cilium/cilium/blob/f9000c886f1f1c29b7d2c2f8608c9d41278ac016/test/k8sT/Services.go#L439-L459

Seen in the test results cilium-1z7pc_endpoint_list file:
...
    "state": "regenerating",
    "status": [
      {
        "code": "OK",
        "message": "Regenerating Endpoint BPF: endpoint policy updated \u0026 changes were needed",
        "state": "regenerating",
        "timestamp": "2018-04-01T02:33:08Z"
      }
...

Potentially the same root cause as #3467?

The other node's endpoints are ready.

Seen also in cilium_logs.txt:
level=warning msg="Generating BPF for endpoint failed, keeping stale directory." containerID=df331c8b24 endpointID=28219 file-path=/var/run/cilium/state/28219_next_fail policyRevision=120
level=warning msg="Regeneration of endpoint program failed" containerID=df331c8b24 endpointID=28219 error="Error while configuring proxy redirects: proxy state changes failed: context deadline exceeded" policyRevision=120
level=warning msg="Generating BPF for endpoint failed, keeping stale directory." containerID=cilium-loc endpointID=29898 file-path=/var/run/cilium/state/29898_next_fail policyRevision=119
level=warning msg="Regeneration of endpoint program failed" containerID=cilium-loc endpointID=29898 error="Error while configuring proxy redirects: proxy state changes failed: context deadline exceeded" policyRevision=119
level=warning msg="Generating BPF for endpoint failed, keeping stale directory." containerID=bfb790cd72 endpointID=42380 file-path=/var/run/cilium/state/42380_next_fail policyRevision=119
level=warning msg="Regeneration of endpoint program failed" containerID=bfb790cd72 endpointID=42380 error="Error while configuring proxy redirects: proxy state changes failed: context deadline exceeded" policyRevision=119
level=info msg="No request received to manage networking for container" containerID=3699524432 maxRetry=20 subsys=containerd-watcher
level=warning msg="Error while getting Kubernetes labels" error="pods \"reviews-v2-3456803578-q4c22\" not found" subsys=containerd-watcher
level=info msg="No request received to manage networking for container" containerID=58e72d7902 maxRetry=20 subsys=containerd-watcher
level=info msg="Endpoint policy recalculated" containerID=cilium-loc endpointID=29898 policyRevision=122
level=info msg="Endpoint policy recalculated" containerID=ed7ad1ceb1 endpointID=15066 policyRevision=118
level=warning msg="Unable to inspect container, retrying..." containerID=f20a8a50b5 error="unable to inspect container 'f20a8a50b5d359873d932ba1bff87dcf0c31f1327f4b4ad1dddcdaaa4511b1d2': Error: No such container: f20a8a50b5d359873d932ba1bff87dcf0c31f1327f4b4ad1dddcdaaa4511b1d2" maxRetry=20 retry=19 subsys=containerd-watcher
level=warning msg="Unable to inspect container, retrying..." containerID=f20a8a50b5 error="unable to inspect container 'f20a8a50b5d359873d932ba1bff87dcf0c31f1327f4b4ad1dddcdaaa4511b1d2': Error: No such container: f20a8a50b5d359873d932ba1bff87dcf0c31f1327f4b4ad1dddcdaaa4511b1d2" maxRetry=20 retry=20 subsys=containerd-watcher
level=info msg="No request received to manage networking for container" containerID=f20a8a50b5 maxRetry=20 subsys=containerd-watcher
level=info msg="No request received to manage networking for container" containerID=a6230ccaba maxRetry=20 subsys=containerd-watcher
level=warning msg="Error while getting Kubernetes labels" error="pods \"reviews-v1-2690276606-qgcw0\" not found" subsys=containerd-watcher
level=info msg="Endpoint policy recalculated" containerID=be33b3ae8e endpointID=11004 policyRevision=119
level=info msg="Endpoint policy recalculated" containerID=cilium-loc endpointID=29898 policyRevision=119
level=warning msg="Error while getting Kubernetes labels" error="pods \"productpage-v1-127201421-z0hx6\" not found" subsys=containerd-watcher
level=info msg="No request received to manage networking for container" containerID=8c72ad498c maxRetry=20 subsys=containerd-watcher

Unfortunately the /var/run/cilium/state dir doesn't appear to have been collected.

The result of running cilium policy wait isn't checked in this code:
https://github.com/cilium/cilium/blob/f9000c886f1f1c29b7d2c2f8608c9d41278ac016/test/helpers/kubectl.go#L739

Given that we now check the result of cilium policy wait, and we have fixed bugtool output for this test, I recommend we downgrade the priority of this from priority/1.0-testweek given that this has not reproduced since then.

@joestringer  I think that this issue is related to https://github.com/cilium/cilium/issues/3549 make sense to you?

Sure.

Unfortunately the /var/run/cilium/state dir doesn't appear to have been collected.

This is due to the fact that resources are deleted within the It :( Same issue as https://github.com/cilium/cilium/pull/3626 . I'll fix the test (tracked by https://github.com/cilium/cilium/issues/3816 ). Since we don't have enough logs, I'm closing this ticket. If this issue occurs again we can file a new ticket.


ExternalLabels in MSO

Test ExternalLabels
We test our ExternalLabels by sending metrics to a RemoteWrite endpoint and ensuring ExternalLabels defined on the MonitoringStack Operator exist in the metrics. We will deploy a custom Blue App, that has an endpoint /alerts that receives metrics. We can verify ExternalLabels are working by checking the logs of the Blue App.

Setup Kind Cluster
Deploy Blue Application
Deploy MSO
Verify ExternalLabels exist in RemoteWrite Metrics
Clean Up

Setup Kind Cluster
To setup kind cluster, read the README.md
Deploy Blue Application
kubectl apply -f -<<EOF
apiVersion: v1
kind: Service
metadata:
  labels:
    app: blue
    version: v1
  name: blue
  namespace: default
spec:
  ports:
    - port: 9000
      name: http
  selector:
    app: blue
  type: ClusterIP
---
apiVersion: apps/v1
kind: Deployment
metadata:
  labels:
    app: blue
    version: v1
  name: blue
  namespace: default
spec:
  selector:
    matchLabels:
      app: blue
      version: v1
  replicas: 1
  template:
    metadata:
      labels:
        app: blue
        version: v1
    spec:
      serviceAccountName: blue
      containers:
        - image: docker.io/cmwylie19/go-metrics-k8s:latest
          name: blue
          ports:
            - containerPort: 9000
              name: http
          imagePullPolicy: Always
      restartPolicy: Always
---
apiVersion: v1
kind: ServiceAccount
metadata:
  name: blue
EOF

Deploy MSO
We have defined our ExternalLabels and the RemoteWrite Endpoint.
kubectl apply -f -<<EOF
apiVersion: monitoring.rhobs/v1alpha1
kind: MonitoringStack
metadata:
  name: blue
spec:
  logLevel: debug
  prometheusConfig:
    externalLabels:
      tenant_id: "blue"
      user: "blue"
      clusterID: "blue"
    remoteWrite:
    - url: http://blue.default.svc.cluster.local:9000/alert # My app to read the metrics
  retention: 120h
EOF

Deploy ServiceMonitor
Tell Prometheus to Scrape the Blue App
kubectl apply -f -<<EOF
apiVersion: monitoring.coreos.com/v1
kind: ServiceMonitor
metadata:
  name: blue
  labels:
    app: blue
spec:
  selector:
    matchLabels:
      app: blue
  endpoints:
  - port: http
EOF

Verify ExternalLabels exist in RemoteWrite Metrics
Since Prometheus has been instruected to send RemoteWrite metrics to the blue app, we can check the logs of the blue app to ensure that the ExternalLabels we defined are present: tenant_id, user, and clusterID
kubectl logs -l app=blue | egrep 'clusterID|tenant_id|user'

output
scrape_duration_seconds{clusterID="blue", container="blue", endpoint="http", instance="<IP_ADDRESS>:9000", job="blue", namespace="default", pod="blue-b955b9f5f-s8kjd", prometheus="default/blue", prometheus_replica="prometheus-blue-0", service="blue", tenant_id="blue", user="blue"}
scrape_samples_scraped{clusterID="blue", container="blue", endpoint="http", instance="<IP_ADDRESS>:9000", job="blue", namespace="default", pod="blue-b955b9f5f-s8kjd", prometheus="default/blue", prometheus_replica="prometheus-blue-0", service="blue", tenant_id="blue", user="blue"}
scrape_samples_post_metric_relabeling{clusterID="blue", container="blue", endpoint="http", instance="<IP_ADDRESS>:9000", job="blue", namespace="default", pod="blue-b955b9f5f-s8kjd", prometheus="default/blue", prometheus_replica="prometheus-blue-0", service="blue", tenant_id="blue", user="blue"}
scrape_series_added{clusterID="blue", container="blue", endpoint="http", instance="<IP_ADDRESS>:9000", job="blue", namespace="default", pod="blue-b955b9f5f-s8kjd", prometheus="default/blue", prometheus_replica="prometheus-blue-0", service="blue", tenant_id="blue", user="blue"}

Clean Up
kubectl delete servicemonitor blue
kubectl delete monitoringstack blue
kubectl  delete svc,deploy,sa blue
kind delete cluster --name=kind
kind delete cluster --name=kind-kind

Closes #126

@cmwylie19 Thank you for the patch and it looks good to me.

I think we will benefit from adding an e2e test to ensure this feature works as expected.
Could you please fix the CI issues?

Please feel free to ping us should you need help with any of this.

@cmwylie19 Thank you for the patch and it looks good to me.

I think we will benefit from adding an e2e test to ensure this feature works as expected.
Could you please fix the CI issues?

For the commit lint git commit -s :
feat: support ExternalLabels prometheus config

A short description of what is changed ...

Fixes: #125
Signed-By: ...

Please feel free to ping us should you need help with any of this.

Absolutely, i will get to this. Thank you @sthaha

@sthaha would you be able to check this now? I have fixed the commits and the e2e tests. Thank you!


luci-app-frpc：add enable options

without the enable option, there is no control over running or stopping the plug-in. After the enable option is added, you can control whether the plug-in is enabled or not. adaptation frpc.

There are two PRs opened for the same thing. I think this PR can be closed and we can continue in https://github.com/openwrt/luci/pull/6133


Ubuntu stuck at sending path tree

I have the OSVR core running, but when I plug in the HMD and the tracker and run the server I get part of the way through but get stuck.
[OSVR Server] Using config file '/home/cmdann/vr/OSVR-Core/build/share/osvrcore/osvr_server_config.json'
[OSVR Server] Constructing server as configured...
[OSVR Server] Loading auto-loadable plugins...
[OSVR Server] Loading plugins...
[OSVR Server]
[OSVR Server] Instantiating configured drivers...
[OSVR Server]
[OSVR Server] Using OSVR HDK for display configuration. Did not find an alternate valid 'display' object in config file.
[OSVR Server] Triggering automatic hardware detection...
[OSVR Server] Registering shutdown handler...
[OSVR Server] Starting server mainloop: OSVR Server is ready to go!
[OSVR] Performing hardware auto-detection.
[OSVR] Added device: com_osvr_Multiserver/OSVRHackerDevKit0
[OSVR] Added device: com_osvr_Multiserver/OSVRHackerDevKitPrediction0
[OSVR] Path tree updated or connection detected
[OSVR] Sending path tree to clients.

Any way to force this path tree?

Could you share part of your dmesg after you connect both devices? I'm wondering if them were detected.

It doesn't necessarily display any other messages on the server after that
point, that looks like a fine log to me. You can run OSVR print tree to
view the path tree that's being sent. What are the symptoms of the actual
problem you're having, or is it just confusing log messages?
On Wed, Jun 29, 2016, 8:27 PM Marcelo Araujo<EMAIL_ADDRESS>wrote:

Could you share part of your dmesg after you connect both devices? I'm
wondering if them were detected.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/OSVR/OSVR-Core/issues/443#issuecomment-229535931, or mute
the thread
https://github.com/notifications/unsubscribe/AADuyX9TMyJyPjvDH1kOhVOD5aKYQ7Tbks5qQxt6gaJpZM4JBNCb
.

--
Ryan A. Pavlik, Ph.D.
CTO - OSVR Platform
Sensics, Inc.
www.sensics.com
--
Latest news and blog posts (subscribe here
http://sensics.com/subscribe-to-our-mailing-list/ to get weekly updates):
July 5: VRguy podcast: Matt Peckham, video game critic for TIME
http://sensics.com/vrguy-podcast-episode-17-matt-peckham-video-game-critic-for-time/
June 29: VRguy podcast: Tim Ruse, CEO of Zero Latency, on free-roam VR
http://sensics.com/vrguy-podcast-episode-16-tim-ruse-ceo-of-zero-latency-on-free-roam-vr/
June 22: The three (and a half) configurations of eye trackers
http://sensics.com/the-three-and-a-half-configurations-of-eye-trackers/


feat: change sendhostname

snippet.sendhostname - ga 수집방식 변경

ok to test

ok to test

ok to test


added numeric header for std::accumulate

fixed grpc patch command and added missing headers

Codecov Report

Merging #593 into develop will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff            @@
##           develop     #593   +/-   ##
========================================
  Coverage    44.85%   44.85%
========================================
  Files          271      271
  Lines        11198    11198
  Branches      5908     5908
========================================
  Hits          5023     5023
  Misses         750      750
  Partials      5425     5425

Impacted Files
Coverage Δ

irohad/ametsuchi/impl/postgres_wsv_command.cpp
32.22% <ø> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


Maintenance and Supersession

This package is fully functional, but may eventually become superceded by the new x/mobile/gl plan.

Yet, over 4 years after that sentence was written, I couldn't find any updates on WebGL support being added to the golang.org/x/mobile/gl package.
Is there actually any work at all being done with that (and I just missed it), or is supersession by x/mobile/gl entirely a hypothetical?
If it's the latter, it might be about time that the README sees another update to more accurately reflect the current state of this package..?

Thanks for raising this point.
The the x/mobile/gl plan was described at https://go-review.googlesource.com/c/mobile/+/8793/4#message-feac06fc2bd0ba1b6d2390d301030415a34d8a82 and https://groups.google.com/forum/#!topic/golang-dev/VCFDiU6Pc8M.
While x/mobile/gl has changed, I don't think adding support for GopherJS became in scope. This package will continue to exist with its current API until further developments, but there certainly isn't active work being done or planned now.
I should update the README accordingly.


REQUIRED_ORG filters out all private members

I set up a mention-bot with access to our organization and its private repositories using the following configuration:
GITHUB_USER=bot_username GITHUB_PASSWORD=bot_pass GITHUB_TOKEN=bot_token_with_repo_scope REQUIRED_ORG=YousicianGit npm start
However, if the REQUIRED_ORG is set, metion-bot cannot find any reviewers:
https://github.com/YousicianGit/Foobar/pull/1234
Reviewers: []
Skipping because there are no reviewers found.

If I remove that env variable it starts to work:
https://github.com/YousicianGit/Foobar/pull/123
Reviewers: [ 'user1', 'user2', 'user3' ]

I noticed that most of our users have their membership in the organization set to Private, like this:

https://help.github.com/articles/publicizing-or-hiding-organization-membership/
I assume that this is the reason no reviewers are found. Would be nice to fix this somehow. In the meantime we won't use the REQUIRED_ORG.

Seems like support was added for this in https://github.com/facebook/mention-bot/pull/106, so this can be a regression.

This does not appear to be a regression (although it is affecting us too). It appears that #106 only functions for private repositories and operates independently of the REQUIRED_ORG setting. Looking at the code it appears that the function used github.orgs.getForUser({ ... }); is only capable of iterating over the public organizations for a user.
It seems like it would be better to use github.orgs.getMembers which is already exposed inside of mention-bot as getMembersOfOrg, and then filter the list by who is returned by getMembersOfOrg instead of by calling github.orgs.getForUser for each potential reviewer. If the bot user is not a member of the org, this should still have the result of only seeing public members, but if they are a member of the org, it should see private members as well.


Don't merge back before traversing the additional_conf; close #17

The traverse function in cli.rb merged back before loading all of the higher-level @includes. Somehow the additional step for wrapped @includes remedied this.

do you think we should bump the release version for this?  I just realized I don't have ownership rights over the published gem unfortunately...

@LFDM please add me to the owners list for the published arethusa-cli gem

I’d like to fully understand the mechanics of the traverse function before doing that. I’ll step through the function tomorrow and hopefully will be able to confirm that the fix works as intended.

Am 21.02.2016 um 19:52 schrieb Bridget Almas<EMAIL_ADDRESS>do you think we should bump the release version for this? I just realized I don't have ownership rights over the published gem unfortunately...
—
Reply to this email directly or view it on GitHub https://github.com/alpheios-project/arethusa-cli/pull/18#issuecomment-186955009.


Allow merge only when tests are not worse

Measure Jest coverage and Stryker score on develop and the current branch and allow merge only when both values are not worse.
https://help.github.com/en/actions/reference/workflow-syntax-for-github-actions#jobsjobs_idoutputs

It might be better to leave it as it is. The coverage report is enough for now.


Test wasm

Pull Request Test Coverage Report for Build<PHONE_NUMBER>9
Details

131 of 231 (56.71%)  changed or added relevant lines in 5 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage decreased (-2.2%) to 73.524%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/datasource/wasm.ts
22
27
81.48%

pkg/eval_query.go
36
46
78.26%

src/datasource/datasource.ts
56
72
77.78%

src/datasource/helpers/index.ts
14
83
16.87%

Files with Coverage Reduction
New Missed Lines
%

src/datasource/datasource.ts
1
62.15%

Totals

Change from base Build<PHONE_NUMBER>9:
-2.2%

Covered Lines:
2628

Relevant Lines:
3463

💛  - Coveralls

Pull Request Test Coverage Report for Build<PHONE_NUMBER>9
Details

131 of 231 (56.71%)  changed or added relevant lines in 5 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage decreased (-2.2%) to 73.524%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/datasource/wasm.ts
22
27
81.48%

pkg/eval_query.go
36
46
78.26%

src/datasource/datasource.ts
56
72
77.78%

src/datasource/helpers/index.ts
14
83
16.87%

Files with Coverage Reduction
New Missed Lines
%

src/datasource/datasource.ts
1
62.15%

Totals

Change from base Build<PHONE_NUMBER>9:
-2.2%

Covered Lines:
2628

Relevant Lines:
3463

💛  - Coveralls


Stream keeps stalling on random moments, or on discontinuities in HLS

Shaka player version:
V.3.0.1
When the player runs our stream, it seems to be running ok-ish until it hit's a discontinuity in the HLS (marked with EXT-X-DISCONTINUITY).
We than seen console logging:

But i do not know what this means?
It eventually recovers and continues, but i don't understand why it would stall? other players, videoJS, seem to be playing the stream just fine

Hello @RonaldvB ,
It may stall for various reasons, and we'd love to find the root cause of this issue.

Could you please follow our bug report template here, and let us know which version of Shaka Player you're using?
https://github.com/google/shaka-player/issues/new?assignees=&labels=&template=bug_report.md
Could you please share with us the manifest you're testing with? It would help us to reproduce this issue and look into it. If it's private content, you can send an email to<EMAIL_ADDRESS>Thank you!

Have you read the FAQ and checked for duplicate open issues?
Yes
What version of Shaka Player are you using?
V.3.0.1
Can you reproduce the issue with our latest release version?
We're using the demo player online, so yes?
Can you reproduce the issue with the latest code from master?
NA
Are you using the demo app or your own custom app?
Demo app
What browser and OS are you using?
Chrome ( 83.0.4103.97 ) on Mac Catalina (10.15.5)
@michellezhuogg  i've asked our customer if we can share the stream, not sure if it has to be private, but i'll let you know :-)
thanx in advance

This is one of our streams:
https://streaming002.sniffercat.com/scheduler/scheduleMaster/118.m3u8

Thank you! Verified the bug, and we'll work on the fix.

Is there maybe a way we can fix something in our HLS to fix this?

Hello @RonaldvB ,
We have a potential fix for this! Could you please make your test manifest available again, so that we can verify it?

Hello @RonaldvB ,
We have a potential fix for this! Could you please make your test manifest available again, so that we can verify it? Thank you!

can i send it via the email you provided?

Yes please!

Ha @michellezhuogg  i've send a new link :)
In the meanwhile we've also done some updates


Fix phpstan bug

Bug https://github.com/phpstan/phpstan/issues/5372 was fixed and this is a showcase that it helped us - I will consider this done, when a phpstan is released.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
82

Relevant Lines:
113

💛  - Coveralls

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
82

Relevant Lines:
113

💛  - Coveralls


codes in README.md are not runalbe

I'm using Angular7 and installed this lib and keen-analysis, found that client.run returns undefined and client.query requires two parameters, first is the analysis_type, second is the config object. Should use the code in API document here.

Hi @linicam
Thank you for the feedback. We're currently working on the new API documentation website.
I'll let you know when it's done.
Regards


Document how to run a relay server

The current README is rather minimal. We'd like to have full documentation on how to run a relay server and the needed components.
For people using docker we need to provide a docker-compose file.

AFAIK this is done.


Broader tables cover contact box

If a table is too broad, it covers the contact box.

Thanks for reporting. :+1:
This could be fixed with a rather awkward word-break: break-all; or with shorter headings :wink:
The issue lies more in the fact that there are 3 columns, and this is not really optimal to display a lot of information.
A possible solution would be to wrap everything with overflow-x: scroll; or to implement something like Tablesaw.js which comes with a horizontal sliding function with a proper behaviour on both desktop and mobile.

https://jira.liip.ch/browse/SWG-106

Bootstrap’ responsive table component would be probably useful here.

It looks to me as an editorial and not a technical problem. Editors should use shorter words; this is allways possible. Shorter words also make it easier for users to understand the content of a table.

It also a technical issue, because the content of some tables comes from API and other systems. For this reason, it has to be solved.

Kann dieses Issue im neuen Release (Milestone) übernommen werden ?

Graciela und ihr Team schauen es an.

This is a known issue in the Web of non-breaking table cells and can be easily fixed using the Bootstrap's .table-responsive class in your implementation of the tables which grow broad.


.gitignore file not included in the built archive

Environment

Elixir version: 1.6.1

Expected behavior
When running the generate task from within this repository the generated includes a .gitignore file. This is the correct behaviour.
The file is part of the template in the priv directory https://github.com/CrowdHailer/raxx_kit/blob/master/priv/template/.gitignore.
Actual behavior
However when running the task from the installed archive
$ mix archive.install https://github.com/CrowdHailer/raxx_kit/raw/master/raxx_kit.ez
$ mix raxx.kit foo

The .gitignore file is missing from the generated project.
Are dotfiles not included in archives?

Fixed from version 0.4.1 by including dotfiles in priv directory to archive https://github.com/CrowdHailer/raxx_kit/commit/2b3a3566af3b724ffdc6128fa93f5e9b40a33e39#diff-04c6e90faac2675aa89e2176d2eec7d8R70

dotfile still not included if install from hex. see this discussion
https://elixirforum.com/t/is-there-a-way-to-pull-archives-from-hex-pm/13789/7

I have "fixed" this by having saving dotfile templates as _DOTFILE.filename. Works for the moment and makes the project no rely on any update to mix


fix(HyperlinkModal): fixes #325, #327, and #328

Signed-off-by: Diana Lease<EMAIL_ADDRESS>Issues
#325
#327
#328
Flags

First batch of bug fixes. Will continue working on the others separately.

Related Issues

Issue #326 (starts addressing this but still more work to be done)

If you select outside the text area and select the hyperlink button it crashes.
Also selecting a link, SHIFT + TAB twice, you can navigate via keyboard.
These could be opened as separate issues...

If you select outside the text area and select the hyperlink button it crashes.
Also selecting a link, SHIFT + TAB twice, you can navigate via keyboard.
These could be opened as separate issues...

I'll address the first one in this PR. Can you log the second one as a new issue?

I'll address the first one in this PR. Can you log the second one as a new issue?

@irmerk first one is now addressed


Enhance Compatibility with Rack 3 and Maintain Support for Rack 2

This PR introduces changes to the Yabeda::Prometheus::Mmap::Exporter class to ensure compatibility with both Rack 2 and Rack 3. It addresses the issue where Rack 3 moved WEBrick related code to a separate gem (rackup), which requires conditional handling based on the Rack version being used.

Thanks!

Released in 0.4.0. Enjoy!