Voice control not working mac

OS:
Mac OSX Sierra
Issue Descriptions:
experimental voice controls fail
Steps to Reproduce:
enable voice control
"hey music" etc doesn't do anything
I've also tested the google speech api example which also rendered nothing. I am using a headset with mic.
Looks like it fails then turns off, am happy to test myself and do pr, just need to validate that it is indeed a bug or my voice just isn't US sounding enough being a Kiwi
also looks like we should be setting the dialect too recognition.lang = select_dialect.value;

@mikey0000 The language should be pulled in from your system locale.
(I am a Kiwi and it works for me 👍 )
However the voice API is severely rate limited due to Electron's API key so it only works during certain random hours 😆

Ah one of those issues.
voice API is severely rate limited due to Electron's API -- which API is
this? can we allow personal API keys?
On 2017-01-27 11:30, Samuel Attard wrote:

@mikey0000 [1] The language should be pulled in from your system
locale.
(I am a Kiwi and it works for me 👍 )
However the voice API is severely rate limited due to Electron's API
key so it only works during certain random hours 😆
--
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub [2], or mute the
thread [3].

Links:
[1] https://github.com/mikey0000
[2]
https://github.com/MarshallOfSound/Google-Play-Music-Desktop-Player-UNOFFICIAL-/issues/2171#issuecomment-275534502
[3]
https://github.com/notifications/unsubscribe-auth/ABY3PWRCZjMIwT6lnNZ3kIO0emiEYBWTks5rWR5rgaJpZM4LvKML

@mikey0000 I believe the API key is baked in during the Electron build process.  But you can try launching the application from the command line with the GOOGLE_API_KEY environment variable set.

@MarshallOfSound so i should have something like these?

and why should i launch app from command line?

@gosteev Typically environment variables aren't picked up on macOS unless the app is launched from the command line.  Safer to suggest launching from the command line so I don't get "but it doesn't work on macOS" later down the line 👍
But yeah, that should work.

v4.3.0, win10 x64, no results.
is that possible to create some type of 'sandbox' to test this feature?

I activated the feature for the first time (I tried a restart also) and "Hey music, etc" doesn't seem to do anything.
I can't see any other explanation of how to enable or use this feature in the docs.....and so assume no additional setup or Mac OS enabling is needed. Although I'd be slightly surprised if it's listening all the time without having to enable somewhere else?
Mac OS Sierra
10.12.6
Chrome
Version 59.0.3071.115 (Official Build) (64-bit)


feat(Anomaly-Detection): add Online Isolation Forest

Hi @ineveLoppiliF , thanks for your contribution! I started the tests, should get back to you soon


Update Reset-AppxPackage.md

Made changes to the syntax formatting
Fixes https://github.com/MicrosoftDocs/windows-powershell-docs/issues/3071

@JohanFreelancer9 Please copy review

Fixes all versions of the doucment

@tiburd
PR has been copyedited and is ready for final review, could you please check and merge? Thanks!


Conversion for Floating Base Systems

Hello,
I was wondering if the URDF to CasADi conversion works for floating base systems like mobile robots?

The current implementation hasn't really dealt with floating base systems, but everything in this library is based on "Rigid Body Dynamics Algorithms" by Roy Featherstone (2008), so it should be possible.
If you look into it, I'd be more than happy to review and accept pull requests.


Dispose method of a service is called twice on child container dispose.

If i register something like this:
Container.RegisterType<IChildContainerDisposeService, ChildContainerDisposeService>(new HierarchicalLifetimeManager());
works fine;
but, if i register something like this:
Container.RegisterType<IChildContainerDisposeService, ChildContainerDisposeService>(new HierarchicalLifetimeManager());
Container.RegisterType< ChildContainerDisposeService >(new HierarchicalLifetimeManager());
then, the dispose method of ChildContainerDisposeService is called twice.
UnitTest
using System;
using Microsoft.VisualStudio.TestTools.UnitTesting;
using Unity;
using Unity.Lifetime;
namespace Unity.Test
{
[TestClass]
public class ChildContainerDisposeTest
{
IUnityContainer Container;
    [TestInitialize]
    public void Init()
    {
        Container = new UnityContainer();
        Container.RegisterSingleton<ChildContainerDisposeLogger>();
        Container.RegisterType<IChildContainerDisposeService, ChildContainerDisposeService>(new HierarchicalLifetimeManager());
        Container.RegisterType<ChildContainerDisposeService>(new HierarchicalLifetimeManager());
    }

    [TestCleanup]
    public void Clean()
    {
        Container?.Dispose();
    }

    [TestMethod]
    public void ChildContainerDisposeServiceTest()
    {
        var logger = Container.Resolve<ChildContainerDisposeLogger>();
        logger.Clear();

        var rootService = Container.Resolve<IChildContainerDisposeService>();

        Assert.AreEqual(1, logger.Construtor, "Construtor log must be 1. First resolve on Root Container.");
        Assert.AreEqual(0, logger.Dispose, "Dispose log must be 0. First resolve on Root Container.");

        var childContainer = Container.CreateChildContainer();
        var childService = childContainer.Resolve<IChildContainerDisposeService>();

        Assert.AreEqual(2, logger.Construtor, "Construtor log must be 2. First resolve on Child Container.");
        Assert.AreEqual(0, logger.Dispose, "Dispose log must be 0. First resolve on Child Container.");

        var childService2 = childContainer.Resolve<IChildContainerDisposeService>();

        Assert.AreEqual(2, logger.Construtor, "Construtor log must be 2. Second resolve on Child Container.");
        Assert.AreEqual(0, logger.Dispose, "Dispose log must be 0. Second resolve on Child Container.");

        childContainer.Dispose();

        Assert.AreEqual(2, logger.Construtor, "Construtor log must be 2. Disposed Child Container.");
        Assert.AreEqual(1, logger.Dispose, "Dispose log must be 1. Disposed Child Container.");

        Container.Dispose();

        Assert.AreEqual(2, logger.Construtor, "Construtor log must be 2. Disposed Root Container.");
        Assert.AreEqual(2, logger.Dispose, "Dispose log must be 2. Disposed Root Container.");
    }
}

#region logger

public class ChildContainerDisposeLogger
{
    public int Construtor { get; private set; }

    public int Dispose { get; private set; }

    public void ConstrutorCalled()
    {
        Construtor++;
    }

    public void DisposeCalled()
    {
        Dispose++;
    }

    public void Clear()
    {
        Construtor = 0;
        Dispose = 0;
    }
}

#endregion

#region IChildContainerDisposeService

public interface IChildContainerDisposeService { }
public class ChildContainerDisposeService : IChildContainerDisposeService, IDisposable
{
    private ChildContainerDisposeLogger _logger;

    public ChildContainerDisposeService(ChildContainerDisposeLogger logger)
    {
        _logger = logger;
        _logger.ConstrutorCalled();
    }

    public void Dispose()
    {
        _logger.DisposeCalled();
    }
}

#endregion

}

When you register types like this:
container.RegisterType<IChildContainerDisposeService, ChildContainerDisposeService>(new HierarchicalLifetimeManager());
container.RegisterType<ChildContainerDisposeService>(new HierarchicalLifetimeManager());

You create two distinct registrations which handled separately. Read this article for more info.
To fix your issue register it like this:
container.RegisterType<IChildContainerDisposeService, ChildContainerDisposeService>();
container.RegisterType<ChildContainerDisposeService>(new HierarchicalLifetimeManager());


is this active?

Nope. It was never quite finished, and nobody's working on it anymore.

I hope to get back to it sometime... but if you want to contribute, please
do!
Addison
On Thu, Aug 27, 2015 at 10:31 AM Trevor Dixon<EMAIL_ADDRESS>wrote:

Nope. It was never quite finished, and nobody's working on it anymore.
—
Reply to this email directly or view it on GitHub
https://github.com/backside/backside/issues/2#issuecomment-135485967.

I would like to do that, not sure how much i can assist you with though,
but would you mind giving me a small overview on the tools needed and what
parts are missing?
lördag 29 augusti 2015 skrev Addison Higham<EMAIL_ADDRESS>
I hope to get back to it sometime... but if you want to contribute, please
do!
Addison
On Thu, Aug 27, 2015 at 10:31 AM Trevor Dixon<EMAIL_ADDRESS><javascript:_e(%7B%7D,'cvml','notifications@github.com');>>
wrote:

Nope. It was never quite finished, and nobody's working on it anymore.
—
Reply to this email directly or view it on GitHub
https://github.com/backside/backside/issues/2#issuecomment-135485967.

—
Reply to this email directly or view it on GitHub
https://github.com/backside/backside/issues/2#issuecomment-135933858.

Is there much parts missing?
lördag 29 augusti 2015 skrev Stellan Haglund<EMAIL_ADDRESS>
I would like to do that, not sure how much i can assist you with though,
but would you mind giving me a small overview on the tools needed and what
parts are missing?
lördag 29 augusti 2015 skrev Addison Higham<EMAIL_ADDRESS><javascript:_e(%7B%7D,'cvml','notifications@github.com');>>:

I hope to get back to it sometime... but if you want to contribute, please
do!
Addison
On Thu, Aug 27, 2015 at 10:31 AM Trevor Dixon<EMAIL_ADDRESS>wrote:

Nope. It was never quite finished, and nobody's working on it anymore.
—
Reply to this email directly or view it on GitHub
https://github.com/backside/backside/issues/2#issuecomment-135485967.

—
Reply to this email directly or view it on GitHub
https://github.com/backside/backside/issues/2#issuecomment-135933858.

I have given it some thought and im very interested in making a project like this to work, would it be better to start from scratch with a new project with its own license or what do you think?

Sorry, me AGAIN.. :)
I hope this isnt a rude question, but would it be possible that i can take
over the project and try to move it forward?
And if that is not an option could I use what you have done to make another
project?
Im just asking this because it seems like a really interesting project, and
since there has been no activity in such a long time then maybe I would be
able to take a bit further..
lördag 29 augusti 2015 skrev Stellan Haglund<EMAIL_ADDRESS>
Is there much parts missing?
lördag 29 augusti 2015 skrev Stellan Haglund<EMAIL_ADDRESS><javascript:_e(%7B%7D,'cvml','sh@idly.se');>>:

I would like to do that, not sure how much i can assist you with though,
but would you mind giving me a small overview on the tools needed and what
parts are missing?
lördag 29 augusti 2015 skrev Addison Higham<EMAIL_ADDRESS>
I hope to get back to it sometime... but if you want to contribute,
please
do!
Addison
On Thu, Aug 27, 2015 at 10:31 AM Trevor Dixon<EMAIL_ADDRESS>wrote:

Nope. It was never quite finished, and nobody's working on it anymore.
—
Reply to this email directly or view it on GitHub
<https://github.com/backside/backside/issues/2#issuecomment-135485967
.

—
Reply to this email directly or view it on GitHub
https://github.com/backside/backside/issues/2#issuecomment-135933858.

Hey, no worries... sorry for the delay in getting back to you!
So to be a bit more concrete on the state of this project, it is mostly working on the backend side, with a few larger-ish bugs/changes away from cutting an initial beta release. You can see most of those as issue in the individual projects, for example: https://github.com/backside/backside-api/issues/6 and https://github.com/backside/backside-mongo-store/issues/1 are the two biggest issues on the persistence layer. With those fixed, the persistence layer and API are in a decent shape with most of the core firebase API being supported, but with probably a few unknown bugs.
With that work, the biggest remaining chunk of work is on the client side library. We have some basic stuff done (here: https://github.com/backside/backside-client), but its really the barebones. And probably has lots of bugs.
As far as how to involve you, if you are interested, it would be great if you wanted to help flesh out the functionality and get things in a better state and have you become a maintainer. However, I put a lot of time into this last year and often think about getting back to work on it and am still interested myself in continuing on with it (and have plans for it being the backing store for a number of other projects I am interested in) but real life has been a problem. So rather than just drop it all in your lap, how does having a google hangout or something and going over the architecture and deciding for a way to get your feet wet sound? If that all works well, we can make you a maintainer on it and hopefully also serve as the momentum I need to get back into it.
If that sounds like a decent plan, let me know and we can set something up. If meeting sounds like too much effort, feel free to just fork and hack away and I will review PRs, give feedback and whatever else you need to keep pushing forward.
Glad to see your interest! Hope we can figure something out!

Oh also, as far as license, if we didn't add one, just assume MIT/ISC.

I would be happy to do a hangout, so should we move this conversation to
gmail first maybe?
måndag 31 augusti 2015 skrev Addison Higham<EMAIL_ADDRESS>
Hey, no worries... sorry for the delay in getting back to you!
So to be a bit more concrete on the state of this project, it is mostly
working on the backend side, with a few larger-ish bugs/changes away from
cutting an initial beta release. You can see most of those as issue in the
individual projects, for example: backside/backside-api#6
https://github.com/backside/backside-api/issues/6 and
backside/backside-mongo-store#1
https://github.com/backside/backside-mongo-store/issues/1 are the two
biggest issues on the persistence layer. With those fixed, the persistence
layer and API are in a decent shape with most of the core firebase API
being supported, but with probably a few unknown bugs.
With that work, the biggest remaining chunk of work is on the client side
library. We have some basic stuff done (here:
https://github.com/backside/backside-client), but its really the
barebones. And probably has lots of bugs.
As far as how to involve you, if you are interested, it would be great if
you wanted to help flesh out the functionality and get things in a better
state and have you become a maintainer. However, I put a lot of time into
this last year and often think about getting back to work on it and am
still interested myself in continuing on with it (and have plans for it
being the backing store for a number of other projects I am interested in)
but real life has been a problem. So rather than just drop it all in your
lap, how does having a google hangout or something and going over the
architecture and deciding for a way to get your feet wet sound? If that all
works well, we can make you a maintainer on it and hopefully also serve as
the momentum I need to get back into it.
If that sounds like a decent plan, let me know and we can set something
up. If meeting sounds like too much effort, feel free to just fork and hack
away and I will review PRs, give feedback and whatever else you need to
keep pushing forward.
Glad to see your interest! Hope we can figure something out!
—
Reply to this email directly or view it on GitHub
https://github.com/backside/backside/issues/2#issuecomment-136217123.


tf.diag support

It would be useful if we could get a square diagonal matrix from a vector of values similar to the diag command from tensorflow. Currently to do this I utilize broadcasting; I create an identity matrix and multiply(*) with the vector. However that depends on the backend function "eye" which is implemented via numpy and I run into other problems when I do that. The main one is that I can't pass the batch size as an argument to "eye" because it is variable("None").
(Disclaimer: I've only used the Tensorflow backend so I'm not sure how the Theano version would play out)

Theano has the same function, so this could easily be implemented for both backends. T.diag(K.variable(np.arange(5))).eval() produces the expected result.

How to support for tensorflow backend, because I am getting this error
AttributeError: module 'keras.backend' has no attribute 'diag_part'

def backendwise_diag(*args, **kwargs):
    backend = keras.backend.backend()
    if backend == 'tensorflow':
        import tensorflow
        return tensorflow.diag_part(*args, **kwargs)
    elif ...

@eIGato I literally just searched for this. Great timing on your result.
This should definitely be included in Keras.


Cactbot Timers in Castrum Lacos Litore

Description
Cactbot Timers not stopping
Additional Details / Steps to Reproduce
In Castrum Lacos Lictore after the 3rd Boss and the Final Boss the Timers arent stopping and still ongoing after leaving the Castrum or after finished the 3rd Boss Fight.
Cactbot Module
Raidboss (alerts & timelines)
Configuration Info
Plugin Name          Enabled Version  Path
---------------------------------------------------------------------------------------------------------------------------------------
FFXIV_ACT_Plugin.dll True    <IP_ADDRESS>  C:\Users\AppData\Roaming\Advanced Combat Tracker\Plugins\FFXIV_ACT_Plugin.dll
OverlayPlugin.dll    True    0.19.27  C:\Users\AppData\Roaming\Advanced Combat Tracker\Plugins\OverlayPlugin\OverlayPlugin.dll
CactbotOverlay.dll   True    <IP_ADDRESS> C:\Users\AppData\Roaming\Advanced Combat Tracker\Plugins\cactbot\cactbot\CactbotOverlay.dll
Triggernometry.dll   False   <IP_ADDRESS>  C:\Users\AppData\Roaming\Advanced Combat Tracker\Plugins\Triggernometry.dll

Overlay Name URL
---------------------------------------------------------------------------------------------------------------------------------
Raidbossy    file:///C:/Users/AppData/Roaming/Advanced%20Combat%20Tracker/Plugins/cactbot/cactbot/ui/raidboss/raidboss.html
DPS Meter    https://hibiyasleep.github.io/kagerou/overlay/
Hunt Radar   file:///C:/Users/AppData/Roaming/Advanced%20Combat%20Tracker/Plugins/cactbot/cactbot/ui/radar/radar.html
Fishing      file:///C:/Users/AppData/Roaming/Advanced%20Combat%20Tracker/Plugins/cactbot/cactbot/ui/fisher/fisher.html
Eureka       file:///C:/Users/AppData/Roaming/Advanced%20Combat%20Tracker/Plugins/cactbot/cactbot/ui/eureka/eureka.html

Various Settings                                Value
------------------------------------------------------------------------------------------------------------------------------------
Plugin Language                                 German
Machina Region                                  Global
Game Version                                    2<IP_ADDRESS>0.0000
Screen Mode                                     Borderless Windowed
Inject and use Deucalion for network data       True
Hide Chat Log (for privacy)                     False
Use WinPCap-compatible library for network data False
Disable high-performance network filter         False
Disable Combine Pets with Owner                 False
Disable Damage Shield estimates                 False
(DEBUG) Enable Debug Options                    False
(DEBUG) Log all Network Packets                 False
(DEBUG) Also Show 'Real' DoT Ticks              False
(DEBUG) Simulate Individual DoT Crits           False
(DEBUG) Graph Potency, not Damage               False
(DEBUG) Enable Benchmark Tab                    False
Cactbot User Dir                                C:\Users\AppData\Roaming\Advanced Combat Tracker\Plugins\cactbot\cactbot\user\

Log & Screenshots
No response
Confirmation

[X] I reviewed  Common Issues and Workarounds and verified that my settings are correct.
[X] I used the search function to confirm this issue doesn't already exist.

Per discussion in ACT discord: no log file attached because the log splitter apparently doesn't recognize CLL or Bozja CEs as exportable encounters, and the web version of the splitter doesn't allow for an entire log to be anonymized and exported.  I'm going to PR an option to the web splitter to allow for full anonymize/export (mirroring functionality of existing CLI tool) so we can troubleshoot/analyze logs where the splitter isn't correctly recognizing the start/end of certain encounters.

What I also noticed is that as soon as the timers were bugged once, they no longer appear at all in the next CLL, but the information messages with "Right" or "left" do.

@ZenithAether Apologies for the delay. The log splitter has now been updated to allow the entire log file to be anonymized/exported without being split:
http://overlayplugin.github.io/cactbot/util/logtools/splitter.html
If you could do that and upload your anonymized log file that includes the CLL run, we'd be able to take a closer look at this.  Thanks for your patience!

Hello here is now the logfile. I had to zip it because the Original file had 197 MB.
processed.zip

@JLGarber, I've got a theory about this, but this is a part of the code you know much better than I do, so would appreciate your thoughts.
In the Bozja timeline, the resets use a window 100000 command:
https://github.com/OverlayPlugin/cactbot/blob/83e4de243c44acdc540f1aecdc685efc8def4750/ui/raidboss/data/05-shb/eureka/bozjan_southern_front.txt#L10-L12
I haven't run into a window command that only has a single param before (as opposed to the more familiar [start,end] format), but looking through timeline_parser.ts, I think the windowCommand regex would match this value as the end and leave start undefined (at least, it did when I did a mockup test):
https://github.com/OverlayPlugin/cactbot/blob/83e4de243c44acdc540f1aecdc685efc8def4750/ui/raidboss/timeline_parser.ts#L158-L159
In that case, with start being undefined, it looks like the parser treats 100000 as a single window (in other words, it adds half of the window as a lookbehind and half as a lookahead).  Since the time (seconds) for these entries is 0, it would set a start of -50000, and an end of 50000:
https://github.com/OverlayPlugin/cactbot/blob/83e4de243c44acdc540f1aecdc685efc8def4750/ui/raidboss/timeline_parser.ts#L609-L614
Which makes perfect sense.... except that the CLL timeline for the final boss (Dawon/Lyon) has entries at a time of 60000+:
https://github.com/OverlayPlugin/cactbot/blob/83e4de243c44acdc540f1aecdc685efc8def4750/ui/raidboss/data/05-shb/eureka/bozjan_southern_front.txt#L357-L359
If I'm right about the way the unbounded window works, then I think it wouldn't match the ActorConrol at the end of CLL (which is definitely in the log) because it would occur after the +50000 sync window.  I think that would be pretty easy to fix, by just changing window 100000 to be window 0,100000.
Does that sound right to you?

That sounds correct, although I think in this case it would be better to have an explicit window 100000,100000 just to keep things clear. I wonder whether it might be worth adding a test for whether all explicit window parameters have two arguments, just to avoid this ever happening elsewhere.

I wonder whether it might be worth adding a test for whether all explicit window parameters have two arguments, just to avoid this ever happening elsewhere.

Although the behavior isn't documented, it does seem by-design given the code, and there are examples of single-param windows in TimelineGuide:
https://github.com/OverlayPlugin/cactbot/blob/83e4de243c44acdc540f1aecdc685efc8def4750/docs/TimelineGuide.md?plain=1#L423-L435
That, plus the possibility people may be doing it in user files - rather than break it or warn about it, maybe we could just add a note to the TimelineGuide documenting what the behavior is when only one param is specified?  (FWIW, I took a quick look, and while it does exist elsewhere in the bozja/eureka stuff, the sync window amply covers the entire timeline).

Oh, I absolutely agree we shouldn't modify the behavior itself. I'm mostly thinking here that we might want to require any usages within the repository to define both start and end of a window, as a matter of style. (I sort of see it as analogous to how the raidboss module will accept bare regexes in triggers, since users might do that for whatever reason, but we do not allow it for repository triggers.)

Oh, I absolutely agree we shouldn't modify the behavior itself. I'm mostly thinking here that we might want to require any usages within the repository to define both start and end of a window, as a matter of style. (I sort of see it as analogous to how the raidboss module will accept bare regexes in triggers, since users might do that for whatever reason, but we do not allow it for repository triggers.)

Yeah, makes sense.  I'll drop a PR to fix this one for now (and add some documentation to TimelineGuide), and then do a follow-up PR to clean up any stray single-param references and add a linting test.

I recently ran the Dalriada for comparison. Everything seems to fit with the timers and there is no overlap like between Adrammelech and Lyon in CLL.


Server hosting not working after latest Steam update

I'm unable to host a server with the latest build. I believe the latest Steam update may have broken the server hosting and/or friend invitation system through the Shift+Tab Steam overlay.

It works for me.


Not finding definitions

Hi! Having a little trouble getting this to recognise libraries/ requires. I've reinstalled twice, just in case, no errors in any of the builds.
Also had the same in vim. Not too sure what I'm doing wrong?
Current settings:
"Lua.runtime.version": "Lua 5.3",
    "Lua.runtime.path": [ // Pulled straight from awesome-client 'return package.path'
        "/usr/share/lua/5.3/?.lua",
        "/usr/share/lua/5.3/?/init.lua",
        "/usr/lib/lua/5.3/?.lua",
        "/usr/lib/lua/5.3/?/init.lua",
        "./?.lua",
        "./?/init.lua",
        "/home/erik/.config/awesome/?.lua",
        "/home/erik/.config/awesome/?/init.lua",
        "/etc/xdg/awesome/?.lua",
        "/etc/xdg/awesome/?/init.lua",
        "/usr/share/awesome/lib/?.lua",
        "/usr/share/awesome/lib/?/init.lua",
        "/home/erik/.luarocks/share/lua/5.3/?.lua",
        "/home/erik/.luarocks/share/lua/5.3/?/init.lua"
    ],
    "Lua.workspace.ignoreSubmodules": false,
    "Lua.workspace.library": {
        "/usr/share/awesome/lib": true
    },
    "Lua.diagnostics.disable": [
        "lowercase-global"
    ],
    "Lua.diagnostics.globals": [
        "client",
        "awesome",
        "screen",
        "root",
        "awful"
    ]

So where is your gears.lua ?

That'd be /usr/share/awesome/lib/gears/init.lua

Try remove the setting Lua.runtime.path

That works very well in VSC! Thanks.
Unfortunately running the exact same settings in Vim doesn't seem to do it for some reason. I've made sure my CocConfig is actually applying, settings like  "Lua.diagnostics.globals" and "Lua.diagnostics.globals" are working as expected.

Please check the log in extensionpath/server/log, it will display the settings I read (search Config update:).
Is there any difference between VSC and Vim?

Solved it!
It looked as though the config hadn't updated in the standalone server, but after a closer look at the filename of the log it occurred to me I'd stopped actually changing directory into .config/awesome.
Tested it out and its working completely as expected.
Thanks again!


What variables should I change to change the number of retrieved search results.

Minimal steps to reproduce

I want to retrieve 5 search results for every query made. How can I change the number in the code apart from changing the number it in the UI. I have tried changing at multiple place in the source code , but wasn't able to retrieve more than set 3 .
And also is there a way to dynamically decide the number of search results based on the query ,because it would be helpful when we have a lot of documents.
Thank you in advance

Mention any other details that might be useful

Thanks! We'll be in touch soon.

Have you tried changing this line?
https://github.com/Azure-Samples/azure-search-openai-demo/blob/a1170562395ad122d2be0e8e29d33cac7ff169c2/app/backend/approaches/chatreadretrieveread.py#L121
To:
top = 10 or some other number?
In terms of dynamically deciding search results based on query, you'd need to change it in that function. I'm not sure what criteria you're using it for changing it, but you could try changing the function call to have a second parameter for number of sources, and then modify the query prompt few shots to demonstrate when it'd call different numbers of sources:
https://github.com/Azure-Samples/azure-search-openai-demo/blob/a1170562395ad122d2be0e8e29d33cac7ff169c2/app/backend/approaches/chatreadretrieveread.py#L132

Hi @pamelafox  I have tried making the changes in the following line of code:  top = overrides.get("top", 3)  but the changes are not being reflected. Do you think I should change else where , as of now I am able to control the number of document from the visual interface of the application only.
In the case of dynamically deciding the search results, I am still trying to figure it out. If you have any repos that are using functions or any of these properties would be of great help.
Thanks in advance

any update on this issue ?


Split the per serer dashboard by subsystem

This issue is a result of discussions on #22 #242 #234
scylla-per-server-metrics-master has become too big and hard to use.
The following will be metrics removed, duplicate the cluster view:

total and unreachable nodes
total storage

The rest will be split into the following dashboard
Operations

Requests Served
All of the Proxy metrics: read, write, timeouts and error
CQL
All of the replica metrics: read/write etc (should we move this to storage?)
compaction

Memory

cache
LSA
OS level memory

Storage

Scylla Disk metrics
Disk activity (currently in the IO dashboard)
IO Qouue (currently in the IO dashboard)
OS level disk metrics (free space)

Networking

CQL connection #241
OS level network (NIC) Rx, Tx
Future: streaming metrics

Each of the dashboards above will have the same templating as the current per server dashboard: per cluster, per node and per shard.
Once complete, we can have a table of all nodes, with links to each node dashboard in the cluster dashboard.

@glommer @amnonh @eyalgutkind please comment

@glommer @amnonh @eyalgutkind any comment?

@glommer @amnonh @eyalgutkind  last attempt for a feedback before I send a PR ;)

I agree with the proposed split. I need to see the actual graphs that land into each to comment if they are complete or not.

One problem with this approach, is you lose the time range when moving from a dashboard to the other

I still think that the lack of automation in generating these dashboards is
one of our biggest problems.
If we did have automation, it would be easier to generate all of those
dashboards plus one with all of them together in a single screen for those
who prefer it and / or want to correlate them.
It would also make it easier to take screenshots, since we now have to take
screenshot of just one page.
On Tue, Oct 17, 2017 at 1:52 PM, Tzach Livyatan<EMAIL_ADDRESS>wrote:

One problem with this approach, is you lose the time range when moving
from a dashboard to the other
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/scylladb/scylla-grafana-monitoring/issues/245#issuecomment-337308388,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAUNve0irMjhPPeaHjhHy1LRpDm8FBdZks5stOlCgaJpZM4PsDEw
.

@tzach is this still relevant?


[stable/superset] Allow to override secret by using existing one

What this PR does / why we need it:
This PR will allow to use an existing secret file containg the init_superset.sh and superset_config.py scripts.
This is to allow users to create files/init_superset.sh and files/superset_config.py, and then use the helm templates with a custom secret file to take advantage of Helm templates.
This is not possible using values.yaml directly, because only files within the templates/ folder can.
Which issue this PR fixes

none

Special notes for your reviewer:

none

Checklist

[x] DCO signed
[x] Chart Version bumped
[x] Variables are documented in the README.md
[x] Title of the PR starts with chart name (e.g. [stable/chart])

That's something we are waiting for :-) We are using this chart as a dependency and would like to use our own set of secret instead of the proposed default name.

/ok-to-test

Hello,
Did the tests run as expected ?
Thanks for your feedbacks !
Regards

@davidkarlsen @scottrigby @jlegrone
Would you guys LGTM this PR please?
@link-webcreations
If this PR passed, would you mind to volunteer as one of this chart owners? Current this chart does not have owner. See:
https://github.com/helm/charts/issues/13869


Basic distributed processing with Hydra

Implemented a basic script that demonstrates distributed processing with Hydra, as mentioned in #42.
The command to run the script is:
python ddp_00.py -m rank=... init_method=...

where rank is a list of the ranks (either a comma-separated list of integers or range(start, stop)) and init_method is a string that specifies one of the two possible initialization methods: TCP initialization and shared file-system initialization (environment variable initialization is not related to init_method).
I'll add documentation (Markdown) that explains the distributed processing in PyTorch as well as how Hydra can kick off distributed processes as demonstrated in the script.

rank can also be a range(start,stop).

rank can also be a range(start,stop).

Not sure if the user even needs to provide the rank argument explicitly. rank has to vary from 0 to num_gpus- 1 (for standard usecases). So we might just infer it ourselves. I understand why does it have to be specified manually right now but this could be a useful example for callbacks.

rank can also be a range(start,stop).

Not sure if the user even needs to provide the rank argument explicitly. rank has to vary from 0 to num_gpus- 1 (for standard usecases). So we might just infer it ourselves. I understand why does it have to be specified manually right now but this could be a useful example for callbacks.

I agree that it's not the best user experience but we don't have calllback right now and even if we did, we need to make sure the design would actually support this. (It's not obvious that this is the case).

rank can also be a range(start,stop).

Not sure if the user even needs to provide the rank argument explicitly. rank has to vary from 0 to num_gpus- 1 (for standard usecases). So we might just infer it ourselves. I understand why does it have to be specified manually right now but this could be a useful example for callbacks.

I agree that it's not the best user experience but we don't have calllback right now and even if we did, we need to make sure the design would actually support this. (It's not obvious that this is the case).

@shagunsodhani What would it take to implement a callback?

rank can also be a range(start,stop).

Not sure if the user even needs to provide the rank argument explicitly. rank has to vary from 0 to num_gpus- 1 (for standard usecases). So we might just infer it ourselves. I understand why does it have to be specified manually right now but this could be a useful example for callbacks.

I agree that it's not the best user experience but we don't have calllback right now and even if we did, we need to make sure the design would actually support this. (It's not obvious that this is the case).

@shagunsodhani What would it take to implement a callback?

rank can also be a range(start,stop).

Not sure if the user even needs to provide the rank argument explicitly. rank has to vary from 0 to num_gpus- 1 (for standard usecases). So we might just infer it ourselves. I understand why does it have to be specified manually right now but this could be a useful example for callbacks.

I agree that it's not the best user experience but we don't have calllback right now and even if we did, we need to make sure the design would actually support this. (It's not obvious that this is the case).

@shagunsodhani What would it take to implement a callback?

Oops sorry missed this comment :) @omry will have a better insight about that. ccing @jieru-hu who is working on callbacks.

rank can also be a range(start,stop).

Not sure if the user even needs to provide the rank argument explicitly. rank has to vary from 0 to num_gpus- 1 (for standard usecases). So we might just infer it ourselves. I understand why does it have to be specified manually right now but this could be a useful example for callbacks.

I agree that it's not the best user experience but we don't have calllback right now and even if we did, we need to make sure the design would actually support this. (It's not obvious that this is the case).

@shagunsodhani What would it take to implement a callback?

Oops sorry missed this comment :) @omry will have a better insight about that. ccing @jieru-hu who is working on callbacks.

Callbacks will likely be pushed back to Hydra 1.2.

Callbacks will likely be pushed back to Hydra 1.2.

Callbacks will likely be pushed back to Hydra 1.2.

We will have callbacks in 1.1, but I am no longer sure we should use them here.
This is not documented yet, but will be before 1.1 is released (Example app).
I am no longer sure we should leverage callbacks here, but it should now be possible to play with it on master.


Updated BeaconBlock.js

Due to the recent changes made to the states in the beacon chain, we will need to update the beacon block implementation. Here's a link: https://github.com/ethereum/eth2.0-specs/blob/master/specs/core/0_beacon-chain.md#beacon-chain-blocks

Closing once #39 , no longer relevant.


[Tabs] Prop to hide whitespace when scrollable tabs don't need scrolling

[X] This is not a v0.x issue.
[X] I have searched the issues of this repository and believe that this is not a duplicate.

Our app uses a lot of tabs for opening different items to edit. Because the number of open tabs can be wider than the viewport we enabled the Tabs.scrollable property. One things we all noticed is when scrolling isn't needed and the left icon is hidden, there is a bunch of whitespace to the left of the first tab. Looking at the source for TabScrollButton there is a placeholder  when visible=false but it doesn't appear there's a way to change how it behaves. What would be nice is to have a 2nd prop like Tabs.hideScrollPlaceholder that would hide the whitespace when the scroll icon isn't needed.
Here is the whitespace to the left of Home I'm talking about.

Now I did write my own component as a workaround but it isn't perfect. What happens is when you click the left scroll button, it scrolls, the icon (and whitespace) disappears, which causes all the tabs to suddenly shift left. Since Tabs.updateScrollButtonState() calculates the visible property for TabScrollButton it could probably handle the scroll then jump better.
import React from 'react';
import PropTypes from 'prop-types';

import MuiTabScrollButton from '@material-ui/core/Tabs/TabScrollButton';
import { withStyles } from '@material-ui/core/styles';
import styles from './TabScrollButton-styles';

function TabScrollButton(props) {
  const { classes, ...other } = props;

  if (other.visible) {
    return (
      <div className={classes.root}>
        <MuiTabScrollButton {...other} />
      </div>
    );
  }

  return <span />;
}

TabScrollButton.propTypes = {
  classes: PropTypes.object.isRequired
};

export default withStyles(styles, { name: 'TabScrollButton' })(TabScrollButton);

export default () => ({
  root: {
    display: 'flex',
    flex: '0 0 56px'
  }
});

@MikeSuiter I'm moving the discussion to #12358


should force load react-components which send over turbo-stream

Summary
Support React On Rails w/Hotwire and Turbo Streams https://github.com/shakacode/react_on_rails/issues/1493, https://github.com/shakacode/react_on_rails/issues/1508
All turbo events (turbo:before-render, turbo:render,.. ) will not dispatch with TurboStream, except turbo:before-stream-render.
And it looks like turbo-rails did not support a after-stream-render event, i don't know why yet.
But hotwired send html, react_on_rails send script, right ?
So my solution: send a script that will trigger load react component along with it.
Pull Request checklist

i add a new render option: force_load which if true then the trigger load script will be concat with the component itself.

i not use reactOnRailsPageLoaded which will query and reload all current react-components (not effective), i introduce new method reactOnRailsComponentLoaded(react-component-dom-id) to load directly the react-component that the script be sent along with.

Other Information

This change is

Summary by CodeRabbit

New Features

Introduced force_load configuration for React components, enabling immediate loading through Turbo Streams.
Added new Turbo Stream templates and routes for dynamic React component rendering.
Integrated a new React component HelloTurboStream for enhanced user interactions.

Bug Fixes

None

Documentation

None

Refactor

None

Style

None

Tests

Added tests for force_load option and Turbo Stream component rendering.

Chores

Updated dependencies to include @hotwired/turbo-rails.
Added @app_props_hello_from_turbo_stream in PagesController.

Revert

None

@theforestvn88 Please

Check the coderabitai suggestions
fix the CI failure on liniting
Rebase on master, and check CI failures.

@Judahmeek did you see anything requiring changes?

@theforestvn88 can you check if your PR solves the problem in https://github.com/shakacode/react_on_rails/issues/1508#issuecomment-1499730248?

Looks good.

@G-Rath any opinions on this PR?

@theforestvn88 can you fix the lint errors?

@theforestvn88 can you fix the lint errors?

i don't know what/where the lint errors are, the command nps format.listDifferent just return exit code 1.
Please help me !

Need new task to update https://github.com/shakacode/react_on_rails/blob/master/docs/rails/turbolinks.md#using-turbo
Merging...


fix(pageserver): fire layer eviction alert only when it's visible

Problem
The pull request https://github.com/neondatabase/neon/pull/8679 explicitly mentioned that it will evict layers earlier than before. Given that the eviction metrics is solely based on eviction threshold (which is 86400s now), we should consider the early eviction and do not fire alert if it's a covered layer.
Summary of changes
Record eviction timer only when the layer is visible + accessed.
Checklist before requesting a review

[x] I have performed a self-review of my code.
[ ] If it is a core feature, I have added thorough tests.
[ ] Do we need to implement analytics? if so did you add the relevant metrics to the dashboard?
[ ] If this PR requires public announcement, mark it with /release-notes label and add several sentences in this section.

Checklist before merging

[ ] Do not forget to reformat commit message to not include the above checklist

Merging it so that we can make it into the next release; I can resolve further comments/concerns in another patch.


Web Developer Skills folder is missing with some Python sub-folders

Fixed! It was due to issue #65.


Plug in Looses Room on restart

Whenever homebridge is restarted, the soil humidity sensors default back to "Default Room" when they had previously been set to a different room

I am having the same problem.
Perhaps, it has something to do with an issue that also causes the home bridge to restart twice because the first restart causes the following:
`[12/3/2022, 4:15:19 PM] [Ecowitt] Data source address: ::ffff:<IP_ADDRESS>
[12/3/2022, 4:15:19 PM] [Ecowitt] Request: POST /data/report HTTP/1.1
host: <IP_ADDRESS>
connection: Close
content-type: application/x-www-form-urlencoded
content-length: 354
[12/3/2022, 4:15:19 PM] [Ecowitt] Data report: {
"PASSKEY": "",
"stationtype": "GW1100B_V2.2.0",
"runtime": "1443881",
"dateutc": "2022-12-04 00:15:19",
"tempinf": "61.16",
"humidityin": "57",
"baromrelin": "29.961",
"baromabsin": "29.961",
"soilmoisture1": "32",
"soilmoisture2": "34",
"soilmoisture3": "44",
"soilmoisture4": "59",
"soilmoisture5": "34",
"soilbatt1": "1.5",
"soilbatt2": "1.6",
"soilbatt3": "1.6",
"soilbatt4": "1.6",
"soilbatt5": "1.6",
"freq": "915M",
"model": "GW1100B"
}
[12/3/2022, 4:15:19 PM] [Ecowitt] stationTypeInfo: null
[12/3/2022, 4:15:19 PM] [Ecowitt] modelInfo: null
[12/3/2022, 4:15:19 PM] [Ecowitt] Discovering sensors
[12/3/2022, 4:15:19 PM] [Ecowitt] Adding sensor: WH51 channel: 1
[12/3/2022, 4:15:19 PM] [Ecowitt] Adding sensor: WH51 channel: 2
[12/3/2022, 4:15:19 PM] [Ecowitt] Adding sensor: WH51 channel: 3
[12/3/2022, 4:15:19 PM] [Ecowitt] Adding sensor: WH51 channel: 4
[12/3/2022, 4:15:19 PM] [Ecowitt] Adding sensor: WH51 channel: 5
[12/3/2022, 4:15:19 PM] [Ecowitt] StationInfo: {
"model": "GW1100B",
"deviceName": "",
"serialNumber": "",
"hardwareRevision": "",
"softwareRevision": "",
"firmwareRevision": "",
"frequency": "915M",
"PASSKEY": "****",
"sensors": [
{
"type": "WH51",
"channel": 1
},
{
"type": "WH51",
"channel": 2
},
{
"type": "WH51",
"channel": 3
},
{
"type": "WH51",
"channel": 4
},
{
"type": "WH51",
"channel": 5
}
]
}
[12/3/2022, 4:15:19 PM] [Ecowitt] sensorId: ****-WH51-1 uuid: ****
[12/3/2022, 4:15:19 PM] [Ecowitt] Restoring existing accessory from cache: WH51
[12/3/2022, 4:15:19 PM] Error: Cannot find the bridged Accessory to remove.
at Bridge.Accessory.removeBridgedAccessory (/homebridge/node_modules/homebridge/node_modules/hap-nodejs/src/lib/Accessory.ts:717:13)
at Bridge.Accessory.removeBridgedAccessories (/homebridge/node_modules/homebridge/node_modules/hap-nodejs/src/lib/Accessory.ts:733:12)
at BridgeService.handleUnregisterPlatformAccessories (/homebridge/node_modules/homebridge/src/bridgeService.ts:404:17)
at HomebridgeAPI.emit (node:events:527:28)
at HomebridgeAPI.unregisterPlatformAccessories (/homebridge/node_modules/homebridge/src/api.ts:355:10)
at EcowittPlatform.registerAccessories (/homebridge/node_modules/homebridge-ecowitt/src/EcowittPlatform.ts:267:18)
at EcowittPlatform.onDataReport (/homebridge/node_modules/homebridge-ecowitt/src/EcowittPlatform.ts:136:12)
at /homebridge/node_modules/homebridge-ecowitt/src/EcowittPlatform.ts:82:14
at nextTick (/homebridge/node_modules/homebridge-ecowitt/node_modules/restify/lib/chain.js:167:13)
at processTicksAndRejections (node:internal/process/task_queues:78:11)
[12/3/2022, 4:15:19 PM] Got SIGTERM, shutting down Homebridge...`

+1 @cypherpork I've got the same issue as you - that's issue #3

Was there ever a solution to fixing this? I am not able to rename the sensors either.

Hi @dxdr, @abrainwood, @askseb.  I have a new Ecowitt plugin available (v2), check out Homebridge Ecowitt Weather Sensors which can be installed through the Homebridge UI.  Not sure what exactly is causing the issue you mentioned with the rooms, but I went ahead and tested for that case and didn't see that issue.  Would appreciate any feedback if you have time to give this new version a try, and let me know if you hit any issues.

Thanks @rhockenbury . I’m not using home bridge at the moment but if I spin it up again I’ll give this a go. Cheers


[WebApp] Delete command should be more transparent about deleting plans

Description
If you call the az webapp delete command, and the specified webapp instance is the only one within the App Service plan, then it will delete the plan as well. Personally, I found this behavior pretty confusing (since you have no way of knowing it's happening), but more importantly, it wasn't what I wanted at all, so I had to re-create the plan again.
The portal makes this behavior pretty clear when you're attempting to delete a web app instance, and it also allows you to prevent deleting the plan. It would be great if the CLI allowed suppressing the plan deletion behavior (some new flag?) and/or maybe it would make sense to display a prompt that notifies the user that they're about to delete the only web app, and ask whether they'd like to also delete the plan. That way, the behavior is clear and predictable.

Environment summary
Install Method: How did you install the CLI? (e.g. pip, interactive script, apt-get, Docker, MSI, nightly)
Answer here: nightly
CLI Version: What version of the CLI and modules are installed? (Use az --version)
Answer here: nightly
OS Version: What OS and version are you using?
Answer here: macOS
Shell Type: What shell are you using? (e.g. bash, cmd.exe, Bash on Windows)
Answer here: bash

Yes, we can expose a flag for this.

This seems similar in concept to #3922. We should aim for a consistent solution.

It is a bit different, that the API supports this through a flag but CLI doesn't expose it


Topic recognition

Hello,
This pull request contains my implementation of the YOLO network detecting lesions on the ISIC 2017/8 dataset.
The mean IoU for the model on testing data is 97.6%.
The pull request contains the following files:

README.md - Project documentation
modules.py - functions for setting up training
dataset.py - script that will setup data for training
train.py - script for training the model
data.yaml - YOLO file for training
slurm.sh - script for running on HPC
ReadmeSupport - directory with supporting images for README.md

For further details and results, please refer to the README.
Riley Clark
45296831

This is an initial inspection, no action is required at this point
Difficulty: Normal
Readme:
Good

appropriate plots/images are shown for results

Commit messages:
OK
Code:

no predict.py file
uses only one class for the full dataset (this would be an incorrect way of solving the problem then as the dataset has multiple classes)

Functionality/Performance:

for one class of labels, achieves good performance

Marking
Good Practice (Design/Commenting, TF/Torch Usage)
Adequate design and implementation
Good spacing and comments
Header blocks missing -1
Recognition Problem
Solves problem, could use multiple classes -1
Driver Script present
File structure present
Shows Usage & Demo & Visualisation & Data usage
Module present
Commenting
No Data leakage
Difficulty: Normal -5
Commit Log
Meaningful commit messages
Progressive commits used
Documentation
ReadMe acceptable
Model/technical explanation, model could be explained better -1
Good Description and Comments
Markdown used and PDF submitted
Pull Request
Successful Pull Request (Working Algorithm Delivered on Time in Correct Branch)
No Feedback required
Request Description good


bcp -G Option not working as described

For Azure Active Directory Integrated authentication, provide the -G option without a user name or password. This configuration assumes that the current Windows user account (the account the bcp command is running under) is federated with Azure AD.
This is not working. If you don't provide a username: you get the error:
SQLState = FA004, NativeError = 0
Error = [Microsoft][ODBC Driver 17 for SQL Server][SQL Server]Failed to authenticate the user '' in Active Directory (Authentication option is 'ActiveDirectoryIntegrated').
Error code 0x800401F0; state 10
CoInitialize has not been called.
If you provide user and password you get the error:
SQLState = FA004, NativeError = 0
Error = [Microsoft][ODBC Driver 17 for SQL Server][SQL Server]Failed to authenticate the user 'xxxxxx' in Active Directory (Authentication option is 'ActiveDirectoryPassword').
Error code 0x800401F0; state 10
CoInitialize has not been called.
For details also see:
https://social.msdn.microsoft.com/Forums/en-US/e6269455-2090-432f-be27-363e9fd486e6/bcp-login-failed-using-activedirectorypassword-authentication?forum=sqltools

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

ID: 86bd520a-7468-fd55-c357-b0f645a917ef
Version Independent ID: 3dc9dee5-afcb-8287-4441-e84ff717186a
Content: bcp Utility - SQL Server
Content Source: docs/tools/bcp-utility.md
Product: sql
Technology: tools-other
GitHub Login: @markingmyname
Microsoft Alias: maghan

@HuubvdP -- thank you for your feedback.  Please file this as a product issue at SQL Server Feedback.
@markingmyname -- please look into this issue #4102.

I've filed it there:
https://feedback.azure.com/forums/908035-sql-server/suggestions/39643693-bcp-g-option-not-working-as-described

@HuubvdP
Thank you for your feedback! We will investigate and get back to you with the relevant info.
Thank you for adding your experience in UserVoice to find for others.

I reworked the "Remarks" section a bit.
I made it more of a bulleted section, and I added an extra note of using special characters from the article you mentioned in this Issue.
I also added how to determine where to see where the bcp utility is installed.
By the end of the week, the article will show the changes.

We will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply. We will gladly continue the discussion, and we will reopen the issue. To see what the community is wanting, please visit the UserVoice site.

@HuubvdP
I was able to test this issue.
When testing on my machine, it did not work.  I got the error stated in this issue.  However, when I built a new clean VM, it did work.  Please make sure you are using the latest version of the BCP utility tool.
The bottom line of my finding is that this issue is generally due to an environmental issue.
Here are the steps and tools I used to make it work.
I reviewed this article to help me set up an AAD Admin to my Azure SQL DB server - https://docs.microsoft.com/en-us/azure/sql-database/sql-database-aad-authentication-configure?tabs=azure-powershell.
Then once I assigned an admin, I had to make sure that I am an admin for the entire tenant to have the ability to assign the admin. So if you are not an admin for your actual Azure tenant, ten, I would have that admin assign an admin to your Azure SQL DB.
I would try and test by using SSMS to see if you can log in to the Azure SQL DB server using Azure Active Directory - Password first.

If that works, then you should be to try and test the example in the article
Then you should be ok.
If you can't get it to work using -P ten leave off the -P and have the tool request a Password or PIN code when the script is executed.
I hope this helps.
Thank you.

We will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply. We will gladly continue the discussion, and we will reopen the issue. To see what the community is wanting, please visit the UserVoice site.

@markingmyname I've tested thes steps but still didn't get this working. Can you confirm that you tested bcp with only -G option (NO -U and-P) and that you get data back from a Azure sql server?

Hey!
I'm facing same problem: bcp Version: 15.0.1300.359 gives message: Failed to authenticate the user '' in Active Directory, like it couldn't get proper authorization info from the system. If I use -U and put my id explicitly, it displays browser window with password prompt and lets me in.
SSMS works fine with Active Directory - Integrated. I can also create ODBC datasource with the driver 17 using Azure Active Directory Integrated. So it's only BCP not working on my machine.

Reopening this issue to investigate further.

Microsoft has found the error and fixed this. It will be released in the next odbc version.
For more information you need to contact Microsoft.

Perfect, guys, thanks!
śr., 10 cze 2020, 08:51 użytkownik HuubvdP<EMAIL_ADDRESS>napisał:

Microsoft has found the error and fixed this. It will be released in the
next odbc version.
For more information you need to contact Microsoft.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/MicrosoftDocs/sql-docs/issues/4102#issuecomment-641767912,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AEPB6H46NAQEEQQZZYY6EYLRV4UOJANCNFSM4KRMP64Q
.

Currently, there is no specific release notes article for cmd line tools for SQL separately.
I will take this feedback to the parties involved.
Thank you.


Implement Dragonfly using a DHT

Feature request:
I propose to shift Dragonfly from being a tracker based p2p network to a trackerless p2p network. We could use a DHT to implement a trackerless p2p, where in effect every peer will become a tracker.
In the last few months I have been reading multiple white papers on the topic and worked on multiple POCs. Dragonfly could use something similar to Kademlia or a variation of it to improve efficiency and reliability of the p2p.
Please have a look at the official white paper: https://pdos.csail.mit.edu/~petar/papers/maymounkov-kademlia-lncs.pdf
You might find this also useful: http://www.bittorrent.org/beps/bep_0005.html
These are other resources you can use to dig further into the topic:
https://www.youtube.com/watch?v=w9UObz8o8lY
https://www.youtube.com/watch?v=_kCHOpINA5g&t=86s
https://medium.com/princeton-systems-course/implementing-kademlia-in-go-65ec9e3c1735#:~:text=Kademlia defines a notion of,keys as 160-bit IDs.
https://github.com/nictuku/dht
I believe Dragonfly would benefit enormously from implementing a DHT in particular in terms of scaling and efficiency.
I am keen to work or collaborate on this feature, unless someone else is already working on it?
Let me know please :)
Thanks.

BT's DHT and PEX features have been considered when dragonfly project initialization. Seeking addresses is slower. Tracker mode can be quickly find parent addresses, it meets the scenario of image acceleration and fast download.


Ability to authentificate to harbor using Bearer Token

Is your feature request related to a problem? Please describe.
When harbor is configured with OIDC authentification, all the requests needs to be authenticated using bearer token.
Related:

https://github.com/goharbor/harbor/issues/10597#issuecomment-596642414
https://github.com/goharbor/harbor/issues/13683
https://github.com/goharbor/harbor/issues/16647#issuecomment-1094819541

Describe the solution you'd like
Add the ability to inject bearer token within the provider configuration
Describe alternatives you've considered

create a robot account with username/password and use it with the provider. didn't work due to lack of permissions: https://github.com/goharbor/harbor/issues/14145

Any chance, to be able soon, to authenticate within the provider at a Harbor instance via a bearer token?


Disable all keyboard shortcuts (such as F2)

I have a tree instance, which I want to disable all browser-based modifications on it.  I have used "check_callback" to disable all actions already, which is great.
However, I'm unable to disable the F2 keyboard shortcut, or any other keyboard shortcuts (if there are any I didn't know of).  When I press F2 on a selected node, the node item still gets changed into an  field, though I know it won't get saved because I have prevented it via "check_callback"
Is there any way of completing disabling the keyboard shortcuts, so that it won't even have a change to change to an <input text field?
Your help is greatly appreciated.

Oh and just to add, I did read one of your previous issues regarding on keydown.jstree,  and tried to return false in that callback, but seemed like there wasn't any effect.
Thanks!!

You should set check_callback to boolean false (or do not set it at all) - that way the rename input won't be shown. Or you can .off the keydown event on the tree container.

Reopening. This will also disable editing the tree elsewhere when I just want to block F2.
Also, offing the keydown event will unbind all other shortcuts too.

I'm having the same problem.
I'd like to be able to disable just the F2 key.
I don't wanna lose the features where I can "edit" the tree neither lose all others key shortcuts.
Could you reopen the case, please?

The only thing I can think of is remove the f2 binding all together. Otherwise I would have to create several config options, which is not desired.
How does that sound? Any ideas?

remove the f2 binding all together, and if anyone wants it - they will have to manually create a listener

I support this suggestion


[SPARK-21274][SQL] Implement INTERSECT ALL clause

What changes were proposed in this pull request?
Implements INTERSECT ALL clause through query rewrites using existing operators in Spark.  Please refer to Link for the design.
Input Query
SELECT c1 FROM ut1 INTERSECT ALL SELECT c1 FROM ut2

Rewritten Query
   SELECT c1
    FROM (
         SELECT replicate_row(min_count, c1) AS (min_count, c1)
         FROM (
              SELECT c1,
                     vcol1_cnt,
                     vcol2_cnt,
                     IF (vcol1_cnt > vcol2_cnt, vcol2_cnt, vcol1_cnt) AS min_count
              FROM (
                   SELECT   c1, count(vcol1) as vcol1_cnt, count(vcol2) as vcol2_cnt
                   FROM (
                        SELECT c1, true as vcol1, null as vcol2 FROM ut1
                        UNION ALL
                        SELECT c1, null as vcol1, true as vcol2 FROM ut2
                        ) AS union_all
                   GROUP BY c1
                   HAVING vcol1_cnt >= 1 AND vcol2_cnt >= 1
                  )
              )
          )

How was this patch tested?
Added test cases in SQLQueryTestSuite, DataFrameSuite, SetOperationSuite

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/1363/
Test PASSed.

Test build #93611 has started for PR 21886 at commit 1039e47.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/1368/
Test PASSed.

Test build #93617 has started for PR 21886 at commit 6392469.

Typo in the PR description: IF (vcol1_cnt > vcol1_cnt, vcol2_cnt, vcol1_cnt) -> IF (vcol1_cnt > vcol2_cnt, vcol2_cnt, vcol1_cnt).

Test build #93611 has finished for PR 21886 at commit 1039e47.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
case class Intersect(

Merged build finished. Test FAILed.

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

@gatorsmile I see this failure in other PRs as well. Is this introduced by some recent changes ?

retest this please.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/1375/
Test PASSed.

Merged build finished. Test PASSed.

Test build #93629 has started for PR 21886 at commit 6392469.

Test build #93617 has finished for PR 21886 at commit 6392469.

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

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/1383/
Test PASSed.

Merged build finished. Test PASSed.

Test build #93642 has started for PR 21886 at commit 7268736.

Test build #93629 has finished for PR 21886 at commit 6392469.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

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

Test build #93642 has finished for PR 21886 at commit 7268736.

This patch fails MiMa tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

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

retest this please

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/1391/
Test PASSed.

Test build #93651 has started for PR 21886 at commit 7268736.

Test build #93651 has finished for PR 21886 at commit 7268736.

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

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

Merged build finished. Test PASSed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/1412/
Test PASSed.

Test build #93677 has started for PR 21886 at commit bfe7030.

Test build #93677 has finished for PR 21886 at commit bfe7030.

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

Merged build finished. Test PASSed.

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

cc @dilipbiswal  Could you resolve the conflicts? I will start the review after the rebase.

@gatorsmile Rebased.

Test build #93708 has started for PR 21886 at commit 67b15ee.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/1434/
Test PASSed.

Test build #93708 has finished for PR 21886 at commit 67b15ee.

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

The code looks good to me. Let us improve the test cases.

@gatorsmile Thank you.. I will make the changes.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/1446/
Test PASSed.

Merged build finished. Test PASSed.

Test build #93725 has started for PR 21886 at commit 8ba4b71.

Test build #93725 has finished for PR 21886 at commit 8ba4b71.

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

Merged build finished. Test PASSed.

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

Test build #93760 has started for PR 21886 at commit 5d5461a.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/1466/
Test PASSed.

LGTM pending Jenkins

Test build #93760 has finished for PR 21886 at commit 5d5461a.

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

Merged build finished. Test PASSed.

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

@dilipbiswal Please address the style issues in your other PRs.

Thanks! Merged to master.

Build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/1474/
Test PASSed.

@dilipbiswal The merged PR does not pick up your last commit.

@gatorsmile Ok Sean..

Test build #93769 has started for PR 21886 at commit 89d03af.

Test build #93769 has finished for PR 21886 at commit 89d03af.

This patch fails due to an unknown error code, -9.
This patch does not merge cleanly.
This patch adds no public classes.

Build finished. Test FAILed.

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


Dev

#607 edate is set for TODAY in the start menu
(app.js,)

Hi Yosi, please squash your commits to one commit (see contribution guidelines).

how do I squash all the commit to one commit?

http://stackoverflow.com/questions/5189560/squash-my-last-x-commits-together-using-git
On Tue, Jun 7, 2016 at 10:02 PM yosinv<EMAIL_ADDRESS>wrote:

how do I squash all the commit to one commit?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/hasadna/anyway/pull/607#issuecomment-224381376, or mute
the thread
https://github.com/notifications/unsubscribe/AAQEw6wGZu931Y1m9i0NiPtitvZn1NlEks5qJcBSgaJpZM4IvQLn
.


Heap out of memory error when running tests in my local machine

When running the test locally,  I am getting a heap out of memory error

 RUNS   graphql-serve  packages/graphql-serve/tests/GraphbackServer.test.ts

<--- Last few GCs --->

[100208:0x4b85480]    55571 ms: Mark-sweep 2031.0 (2064.1) -> 2025.9 (2052.6) MB, 463.3 / 0.1 ms  (+ 128.7 ms in 28 steps since start of marking, biggest step 5.2 ms, walltime since start of marking 606 ms) (average mu = 0.191, current mu = 0.096) allocat[100208:0x4b85480]    56193 ms: Mark-sweep 2030.6 (2052.6) -> 2027.4 (2051.8) MB, 610.8 / 0.1 ms  (average mu = 0.112, current mu = 0.018) allocation failure scavenge might not succeed

<--- JS stacktrace --->

==== JS stack trace =========================================

    0: ExitFrame [pc: 0x13cb519]
Security context: 0x08039c9008d1 <JSObject>
    1: new Map(aka Map) [0x8039c90a1f9](this=0x3deebcc00591 <the_hole>)
    2: ConstructFrame [pc: 0x134c7fa]
    3: createSymbolTable [0x18b36a0c4409] [/home/manyanda/Documents/Projects/aerogear/graphback/node_modules/typescript/lib/typescript.js:~12970] [pc=0x3082f17874e4](this=0x07d449cfcf79 <Object map = 0x3cf48f738d9>,0x3deebcc004b1 <undefined>)
    4: decla...

FATAL ERROR: Ineffective mark-compacts near heap limit Allocation failed - JavaScript heap out of memory
 1: 0xa07f90 node::Abort() [/usr/local/bin/node]
 2: 0xa0839c node::OnFatalError(char const*, char const*) [/usr/local/bin/node]
 3: 0xb80d9e v8::Utils::ReportOOMFailure(v8::internal::Isolate*, char const*, bool) [/usr/local/bin/node]
 4: 0xb81119 v8::internal::V8::FatalProcessOutOfMemory(v8::internal::Isolate*, char const*, bool) [/usr/local/bin/node]
 5: 0xd2d875  [/usr/local/bin/node]
 6: 0xd2df06 v8::internal::Heap::RecomputeLimits(v8::internal::GarbageCollector) [/usr/local/bin/node]
 7: 0xd3a785 v8::internal::Heap::PerformGarbageCollection(v8::internal::GarbageCollector, v8::GCCallbackFlags) [/usr/local/bin/node]
 8: 0xd3b635 v8::internal::Heap::CollectGarbage(v8::internal::AllocationSpace, v8::internal::GarbageCollectionReason, v8::GCCallbackFlags) [/usr/local/bin/node]
 9: 0xd3e0ec v8::internal::Heap::AllocateRawWithRetryOrFail(int, v8::internal::AllocationType, v8::internal::AllocationOrigin, v8::internal::AllocationAlignment) [/usr/local/bin/node]
10: 0xd04cbb v8::internal::Factory::NewFillerObject(int, bool, v8::internal::AllocationType, v8::internal::AllocationOrigin) [/usr/local/bin/node]
11: 0x10464be v8::internal::Runtime_AllocateInYoungGeneration(int, unsigned long*, v8::internal::Isolate*) [/usr/local/bin/node]
12: 0x13cb519  [/usr/local/bin/node]

This was also observed in CI and fixed by https://github.com/aerogear/graphback/commit/4a16de7ea20accb8daf0b4922d95e862ab999770, and can be fixed locally using a simalar approach
NODE_OPTIONS="--max_old_space_size=4096" yarn test

I am wondering what could have happened to cause the memory increase, also should we increase the heap by default (for ci and locally) or document it the option ?
/cc @craicoverflow @wtrocki

need to update metadata library

need to update metadata library

Why? You think that's where the memory increase is coming from?

need to update metadata library

It is up to date I believe.

I think it could be how jest tests are executed.
jest -w 1 is executed from root as one large test suite. If we use yarn workspaces to execute each package test sequentially it might not run out of memory

See #1420
Since this is not a blocker, and the initial fix https://github.com/aerogear/graphback/commit/4a16de7ea20accb8daf0b4922d95e862ab999770 means master is building fine, i will leave this PR for now. There is a deeper problem in our tests that is causing some errors.


New release 0.19.24

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.


Add a couple tsconfig options to atom-typescript

This adds support for noImplicitThis and noUnusedLocals tsconfig options.

Whoops, let me clean this up.


add common error fix to tutorials README

Pull request information

Status: Ready to merge
Kind of changes: Doc
Related PR or issue: -

Description

Add another fix in the tutorials README for the common error ImportError: no module named flow... when running tutorials in jupyter notebook.

Pull Request Test Coverage Report for Build 3907

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 90.614%

Totals

Change from base Build 3898:
0.0%

Covered Lines:
8467

Relevant Lines:
9344

💛  - Coveralls


Fix Hub build and sign flow

Sign should wait for build to be completed before syncing.
Signed-off-by: Michail Resvanis<EMAIL_ADDRESS>

Codecov Report
Base: 77.76% // Head: 77.90% // Increases project coverage by +0.13% :tada:

Coverage data is based on head (3816800) compared to base (5d0daa6).
Patch coverage: 100.00% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #268      +/-   ##
==========================================
+ Coverage   77.76%   77.90%   +0.13%
==========================================
  Files          26       26
  Lines        2789     2792       +3
==========================================
+ Hits         2169     2175       +6
+ Misses        513      511       -2
+ Partials      107      106       -1

Impacted Files
Coverage Δ

internal/cluster/cluster.go
88.35% <100.00%> (+2.34%)
:arrow_up:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

/ok-to-test
/lgtm
/assign @yevgeny-shnaidman

/approve


Running with Pycharm debugger

I'm trying to run a script with accelerate on Pycharm debugger. I've come across issue-535 and followed the steps. However, I'm getting the following lines of error.
Traceback (most recent call last):   File "/home/envs/HF/lib/python3.7/site-packages/accelerate/commands/launch.py", line 280, in simple_launcher     mixed_precision = PrecisionType(args.mixed_precision.lower()) AttributeError: 'NoneType' object has no attribute 'lower' python-BaseException
Do I need additional parameters when running with module accelerate.commands.launch?

You need to still configure accelerate. Run accelerate config from thr CLI. This error will be much more clear in the next release.

Thanks for the comment!
I configured running accelerate config. However, I'm getting another error as follows. Though it seems like either python subprocess error or a compatibility problem between accelerate and subprocess, I'm sharing the error message here in case you've seen something similar before.
Traceback (most recent call last):   File "/home/.conda/envs/ltr/lib/python3.7/subprocess.py", line 800, in __init__     restore_signals, start_new_session)   File "/home/.conda/envs/ltr/lib/python3.7/subprocess.py", line 1551, in _execute_child     raise child_exception_type(errno_num, err_msg, err_filename) FileNotFoundError: [Errno 2] No such file or directory: 'torchrun': 'torchrun' python-BaseException

Is pytorch installed? How did you install accelerate?

Pytorch is installed. I installed accelerate through pip install accelerate and ran accelerate config right after.

@SeoHyeong
Has you solved the "FileNotFoundError: [Errno 2] No such file or directory: 'torchrun': 'torchrun'" error?


searchPaths not working as expected

I have Config Server app that reads from a git repo that looks like this:
RedTeam
  gps
    gps.yml
    gps.properties
GoldTeam
  gps
    gps.yml
    gps.properties

I'm on RedTeam and the app I'm working on is gps. However ConfigServer gives gps config files from both GoldTeam and RedTeam when I only want configs under RedTeam/gps. Is there a way I can have my Config Server only pick up configs in the RedTeam/gps folder without having to change the app name?
I thought spring.cloud.config.server.git.searchPaths would work I read the documentation I've tried:
searchPaths: 'ReadTeam'
searchPaths: 'ReadTeam/*'
searchPaths: 'ReadTeam/{application}'
but it keeps searching all the folders. Whenever I query Config Server localhost:8080/gps or  localhost:8080/RedTeam/gps I still get the config files from both RedTeam/gps and GoldTeam/gps.
Read these but didn't help
https://github.com/spring-cloud/spring-cloud-config/issues/252
https://stackoverflow.com/questions/35026446/spring-cloud-config-searchpaths

Any suggestions please. Should I try a different value for searchPaths or is there something idk about that might work?

removed the configs programatically


Sites: Only display geographic sites from country or countries selected in Profile settings

Affects:

Workflow > Sites page
Project initiation and tracking forms > Site menu
Collected Data form > Site menu
Anywhere else?

Requested enhancement
I only see sites from the country or countries selected in my profile settings.
Note: I am not sure what the experience is for users who only have access to one country. But even as an Admin with access to all programs everywhere, the system should always respect profile settings. If I choose to only view programs from the US and Afghanistan, I should only see geographic sites from those countries.

From Mahmood: Sometimes, we have program that are implemented across countries. I am trying to think if there will be cases when you do not have a country in your profile but you may still be entering data against a program in sites that belong to a country that is not in your profile.
I will investigate with the PaQ team.

Addressed in Papaya.


Fix typo

Thanks!

@swift-ci please smoke test

@aciidb0mb3r - Np.  Not a very useful PR, but just wanted to get the dev environment setup.  Hopefully more to come soon!

Great, let me know if you need any help!


Mongo fixes

This is a set of changes that brings mongo/moped/bson support up to the latest versions of those gems, and fixes a few bugs as well.  Note however that older versions of those gems will no longer work.

This represents further work towards #103


 Let me know if you have any questions or concerns.

At first glance this PR looks good. I will try to find some time to test it out and merge in the next few days.  Thanks!

I'll probably add another commit to allow people to configure which environments we are sending error messages to raygun after this gets merged.
Note this PR removes the report_exception function on the Raygun module due to the warning that elixir emits regarding __STACKTRACE__. People will need to update their code to call report_stacktrace passing in the __STACKTRACE__ directly as is directed. Therefore we should require a bump to the minor version of the library at least.


generate compile_commands.json for use by language servers like clangd

Improving Vere developer experience
This PR enables a bazel run //bazel:refresh_compile_commands command that generates a compile_commands.json file in the root of the repo. This file can be used by popular C/C++ editor plugins/extensions to automatically enable modern IDE features like auto-completion, cross-references, hovering, refactoring, etc.
To enable this, we add this "compile commands extractor" program to our BUILD.bazel file.
If you want to try this yourself with your own installation of vscode with the official clangd extension (for example)...
git checkout i/195/better-ide-support

# uninstall any c/c++ extensions you might already have
code --install-extension llvm-vs-code-extensions.vscode-clangd
code --uninstall-extension ms-vscode.cpptools

# add these `clangd` settings to your *user* json:
  "clangd.arguments": [
    "--header-insertion=never",
    "--compile-commands-dir=${workspaceFolder}/",
    "--query-driver=**",
  ],

# generate the `compile_commands.json` file
bazel run //bazel:refresh_compile_commands

# finally, reload your vscode window

Enjoy no red squigglies, plus lots more.
Resolves #195.

I looked into an alternative to hedronvision's tool called bear which, on first glance, seemed like it could be preferable due to its generic nature (i.e., it's not tightly coupled with bazel like hedronvision's is) but it doesn't actually work with bazel at all (without tedious modification which betrays bazel's attempts at hermetic builds). This comment confirms the same.

Thanks for the quick review @mcevoypeter, just pushed a commit to address your comments.

Thanks for the speedy changes. Sorry I forgot to mention it on the first pass, but mention of this target and its purpose should be made in CONTRIBUTING.md. After that, it's good to go.

Thanks for the speedy changes. Sorry I forgot to mention it on the first pass, but mention of this target and its purpose should be made in CONTRIBUTING.md. After that, it's good to go.

Good idea, done. Thanks again.


(Unirest\Exception) Received HTTP code 407 from proxy after CONNECT

When trying to use a proxy, an error appears. However, if I use proxies in other projects, everything works fine. Any ideas for solving the problem?
$instagram = new \InstagramScraper\Instagram();
Instagram::setProxy([
    'address' => '<IP_ADDRESS>',
    'port'    => '8080',
    'tunnel'  => true,
    'timeout' => 30,
     'auth' => [
        'user' => 'Vdsk4',
        'pass' => 'XcnR3c',
    ],
]);
// Request with proxy
$account = $instagram->getAccount('kevin');

Hello @vprivaloff,
This is a proxy error, I suggest you review your provider's docs.
Cheers.

Hello @vprivaloff,
This error originates from your proxy provider, I suggest you review their docs.
Cheers.

Thank. I will deal with the problem!

How to fix?


Fix syntax error when checking whether comment is supported

The check failed with "Error: syntax error, unexpected '}', expecting newline or semicolon" even when comment is supported.

great catch. thanks
/lgtm
/approve


Query NoSQL DB

Is it possible to run a query on a NoSQL DB? I do not have access to Azure Portal Data Explorer (because of security reasons) and want to run a query search for a specific document in my container.

@zack-docusign We currently don't offer querying CosmosDB NoSQL databases. This can be a new feature for us to look into. Could you elaborate on what security reasons prevent you from using Azure Portal? At the end of the day, Azure Portal and us make similar network calls and I cannot think of reasons making the extension more secure than Azure Portal.


always use dask_array_type for isinstance calls

This makes sure we use dask_array_type instead of dask_array.Array for isinstance calls.

[x] Closes #3779
[ ] Tests added
[x] Passes isort -rc . && black . && mypy . && flake8
[ ] Fully documented, including whats-new.rst for all changes and api.rst for new API

given that we seem to have quite a few bugs in the dask compatibility code (which only gets called if dask is not installed): would it make sense to add a new CI that works like py38 without dask?

the failing tests should be unrelated

LGTM!

the failing tests still are unrelated, they are either #3783, #3751 or a number of distributed errors I don't understand.
Should we merge this?

LGTM!

distributed failures are https://github.com/dask/distributed/issues/3491. We should do something I guess :)


Markdown location

Introduce location in markdown parser so tools can easily retrieve the original position of a block or a token in the input file/string.
Commits:

2d139eb introduces de Location concept and update the parser
fda150d isn't interesting since it only update signature in the test file
dacc7ba adds some test suites for the locations

Avoid to denotes commit's sha1 because they are not stable by reroll. Moreover, the descriptions of individual commits should be in the commits, not in the cover letter.

+1

:shipit:

I will wait for the addition of source paths before merging (unless you complain)

@privat There is no source path since the MarkdowProcessor::process method only take a string.

Ok, i will merge then


Update images with screenshots fom new WebUI

Compress all images to save space.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/2404/
Test PASSed.

@WilliamZapata thanks for the update. It seems like the image quality is worse as a side effect of compression?

@calvinjia The images for the webui were simply too small, I took the screenshots in retina resolution and updated them.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/2416/
Test FAILed.

Jenkins, test this please.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/2426/
Test FAILed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/2467/
Test PASSed.


fix: malformed environment variable JSON

This Pull Request is a

[ ] New feature
[x] Bug fix
[x] Documentation change
[ ] Performance optimization
[ ] Code refactoring
[ ] Other: please add a description

Problem
Our suggested .env.sample had an invalid JSON value assigned to the TALON_ONE_ATTRIBUTES_MAPPINGS environment variable
Solution
Fixed that default value for the environment variable on the sample file

@ArthurFlageul this made me realize how many environment variables we have there that expect JSON values assigned to them.
Do you think we could / should add a general comment somewhere in the readme to point the users to validate their JSON values when changing / assigning them?
(excluding the current malformed default value 🙈 )

Good idea! I've added notes and updated a couple of other minor things.

@ArthurFlageul Thanks a lot - that looks great 💯


Inline functions are broken after formatting

I think inline functions should stay inline after formatting:
like in my code:
    return {
        normal = pipe.new
        * func.map2(tracks1, tracks2,
            function(t1, t2)
                local seg = {junction.generateArc(t1)[1], junction.generateArc(t2)[2]}
                seg[1][2], seg[2][1] = average(seg[1][2], seg[2][1])
                seg[1][4], seg[2][3] = average(seg[1][4], seg[2][3])
                return pipe.new
                    * seg
                    * pipe.map(pipe.map(coor.vec2Tuple))
                    * pipe.zip({{false, false}, {false, false}}, {"edge", "snap"})
            end)
        * pipe.flatten()
        * station.prepareEdges
        * function(r) return func.with(r, {edges = coor.applyEdges(trans.mpt, trans.mvec)(r.edges)}) end,
        ext = pipe.new
        * func.map2(tracks1, tracks2,
            function(t1, t2)
                local seg = {junction.generateArc(t1)[3], junction.generateArc(t2)[4]}
                return pipe.new
                    * seg
                    * pipe.map(pipe.map(coor.vec2Tuple))
                    * pipe.zip({{true, false}, {false, true}}, {"edge", "snap"})
            end)
        * pipe.flatten()
        * station.prepareEdges
        * function(r) return func.with(r, {edges = coor.applyEdges(trans.mpt, trans.mvec)(r.edges)}) end
    }

after formatting, it's not good as before, since inline functions are broken, and my pipe operators goes to the end of the line
I think the user should be able to configure on some formatting options.
    return {
        normal = pipe.new *
            func.map2(
                tracks1,
                tracks2,
                function(t1, t2)
                    local seg = {junction.generateArc(t1)[1], junction.generateArc(t2)[2]}
                    seg[1][2], seg[2][1] = average(seg[1][2], seg[2][1])
                    seg[1][4], seg[2][3] = average(seg[1][4], seg[2][3])
                    return pipe.new * seg * pipe.map(pipe.map(coor.vec2Tuple)) *
                        pipe.zip({{false, false}, {false, false}}, {"edge", "snap"})
                end
            ) *
            pipe.flatten() *
            station.prepareEdges *
            function(r)
                return func.with(r, {edges = coor.applyEdges(trans.mpt, trans.mvec)(r.edges)})
            end,
        ext = pipe.new *
            func.map2(
                tracks1,
                tracks2,
                function(t1, t2)
                    local seg = {junction.generateArc(t1)[3], junction.generateArc(t2)[4]}
                    return pipe.new * seg * pipe.map(pipe.map(coor.vec2Tuple)) *
                        pipe.zip({{true, false}, {false, true}}, {"edge", "snap"})
                end
            ) *
            pipe.flatten() *
            station.prepareEdges *
            function(r)
                return func.with(r, {edges = coor.applyEdges(trans.mpt, trans.mvec)(r.edges)})
            end
    }

I think this might be difficult to address in a broad and generic way, as it would definitely have knock-on effects with legitimate code. What kind of options would you like to see to address this?
I do agree that this is a drastic change to the obviously preferred operator * concatenation in your original sample, though.

Don't know how your formatter works, but if it exists AST, the CR should be considered, and I think you can put some switches in options for different styles, for example left { should be in a new line or not, or just keep, or in my case, a line feed before an operator should be conserved or not. Don't know if you are using VS (the big one) there are many such options like this in formatting.
I am currently using the formatter in this extension, https://marketplace.visualstudio.com/items?itemName=gccfeli.vscode-lua, it works but it's slow, however it respects the line feed,  the formatter or it can be found here:
https://github.com/GCCFeli/LuaFormat


Unable to login

Subject: Unable to login
Description:
This error page when I log in
Results:
I should be able to log in
Backend version: develop-0023fa73
Frontend version:
User Agent: Mozilla/5.0 (Linux; Android 8.1.0; HOME6) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/70.0.3538.110 Mobile Safari/537.36
Page: https://couchers.org/login
User: gobia22 (22892)

already logged in


GetClaim() returning interface is awkward to use. Use output parameter instead?

Is your feature request related to a problem? Please describe.
The following code is awkward as I need to guess that the structure that GetClaim() returns is map[string]string.
		val, ok := introspectionResponse.GetClaim("cnf").(map[string]string)
		if !ok {
			http.Error(rw, "", http.StatusUnauthorized)
			return
		}

		expectedClientCertificateHash, err := base64.URLEncoding.DecodeString(val["x5t#S256"])
		if err != nil {
			http.Error(rw, "", http.StatusUnauthorized)
			return
		}
		if !bytes.Equal(clientCertificateHash[:], expectedClientCertificateHash) {
			http.Error(rw, "", http.StatusUnauthorized)
			return
		}

Describe the solution you'd like
I'd prefer GetClaim() to take an output parameter instead. so that I can marshal safely to my own struct type:
func (r IntrospectionResponse) GetClaim(key string, out interface{}) error {...}

type ConfirmationClaim struct {
        CertificateThumbprint string `json:"x5t#S256"`
}

var cnf ConfirmationClaim
err := introspectionResponse.GetClaim("cnf", &cnf)
if err != nil { ... }
expectedClientCertificateHash, err := base64.URLEncoding.DecodeString(cnf.CertificateThumbprint)
if err != nil { ... }
if !bytes.Equal(clientCertificateHash[:], cnf.CertificateThumbprint {
	http.Error(rw, "", http.StatusUnauthorized)
	return
}

Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

No rush. The code works as is.  It's just awkward.

#283 should improve this. If you have any other feedback, please let us know


Improved code for space age

I improved the code as suggested in the last  pull request.
So there is only one function now, where you give the planet name (the value is defined in the header) and the number of seconds to get the result.
Is it a problem if the #define are in the header ?

This makes it a lot cleaner, but I have to say that it seems like it eliminates the reason for the exercise. Particularly because the interface nearly forces this to be implemented with a one liner, I'm not sure that this is an improvement from a learning standpoint.

@ryanplusplus In the current implementation they're all just one liners too. Multiple one-liners sure, but they all do the same thing so nothing new is learned with each function.

Idea! perhaps if the function accepts a simple enum planets, which the implementation should use to lookup the constant value in some way? There's an opportunity, at a minimum, to introduce the use of enums, but also lookups and so on, and potentially X macros. Learners could make the implementation as simple or as complex as they desire with such an interface.

For eg, something like:
typedef enum planets
{
    mercury = 0,
    venus,
    earth,
    mars,
    jupiter,
    saturn,
    uranus,
    neptune,
    max_planet = neptune
};

float convert_planet_age(planets planet, const long input);

@wolf99 With a bunch of one-liners, you don't learn something new with each function, but you do have to contend with how to manage all of that duplication. I think finding a clean solution to the duplication is the primary challenge of this exercise.
I think your idea to use an enum is a good one as it introduces new concepts in addition to simply eliminating the old.

@ryanplusplus Not sure which approach you're advocating for now :smile:
Have seen that one submission does use an approach to reduce the duplication. but the deduplication lesson is not forced - submissions can pass all tests without reducing the duplication.
With the enum approach at least the enum and lookups of some form are required to pass tests...

@wolf99 I'm fine with either the original or with your suggestion.
You're right that the de-duplication is not forced since we can't do that with tests. Luckily Exercism encourages feedback from other users :)

I'll do the changes within a few days, i've a lot of work to do at the moment 👍

👍


System Labels must be excluded from queries

Every queried resource must strip the system labels
Querying by labels must append system labels + user labels

Unnecessary. The namespaces will prevent the users from using selectors.


Discord link is invalid

Description
I clicked the 'Welcome you all' button in the website that redirects to Discord invite page link, but it says the link is invalid.
Steps to reproduce
I opened the website in my browser
I clicked on the Welcome you all button
The link says it's invalid
Expected behavior
I expect to be taken to the Discord channel after clicking the link.
Actual behavior
I am shown a message that says the link is invalid.
Device/browser
Device: Windows
Browser: Google Chrome
Screenshots

Would like to fix this. Please assign me to this issue.
And please mention the correct link of discord server.

@SahityaRoy can u assign this issue to me

hye assign me this issue


⚠ Bofrost.es has degraded performance

In 7f5fe86, Bofrost.es (https://www.bofrost.es) experienced degraded performance:

HTTP code: 200
Response time: 1628 ms

Resolved: Bofrost.es performance has improved in 265511c after 23 minutes.


fftw3_mpi

For the builds using MPI, we should now be able to use fftw and fftw_mpi:
https://github.com/conda-forge/fftw-feedstock/pull/52
Something that may not be so easy: the configuration file has to be different with and without MPI.

fttw_mpi support add in https://github.com/conda-forge/fluidfft-feedstock/pull/4


Release with Metrics filter

I find your library really useful, especially the recent addition - metrics filter. I have following questions:

do you plan to release it in nearby future? I would like to use new filter and I don't want to have a dependency to a SNAPSHOT
do you plan to publish artifacts on some public repository? I think that would be great and ease using your spi

@dchrzascik Yes, I think these are good ideas and we will try to release to a public repo ASAP.

@dchrzascik There is a new release containing the response time filter now: https://github.com/aerogear/keycloak-metrics-spi/releases/tag/1.0.1
Currently we only have the artifact hosted on Github, we might release it to Maven Central in the future.

Great, thanks.