Add suport for async validations

It would be VERY useful if the 'canexit' and 'canenter' methods accepted a promise, this way we would be able to make ajax calls for validation purposes.

Nevermind, you can validate the form synchronously using the angular build-in form property '$valid' and '$pending'.
For example you can add this function to validate an step:
$scope.validate = function () {

    //Only can change step if the form is valid and has no pending ajax requests
    return $scope.myForm.$valid && !$scope.myForm.$pending;
};

For more information please read forms documentantion:
https://docs.angularjs.org/guide/forms
NOTE: To use Angular's $asyncValidators you need angular version >= 1.3

Thanks for the great ideas, I am going to work on implementing this

@RyanDale thanks for the PR, I merged it and made the following changes in case you are interested:
Line 153 I moved it so canExitStep runs before canEnterStep
Line 188 and 211 I passed in $scope.context in as an argument, otherwise this context object was only available if you were using promises.
Thanks again!

@jacobscarter thanks for merging it in, good catch on both of those issues. The one thing I didn't do was update the documentation so I'm planning on doing that within the next few days.

@RyanDale I updated the readme file yesterday, go ahead and give it a read and let me know if you think it should have more detail or if I left anything out


Issue 14006 - "Contribute" button/option on the main site and sitemap

It looks like contribution information was present on the main index page but was commented out at some stage. I've re-instated the text and updated it.

Is this ok on the main index page?
Please comment on grammar or alternative wording.

All changes can be previewed here: http://nomad.so/misc/d/dlang.org/issue_14006/index.html
Bugzilla: https://issues.dlang.org/show_bug.cgi?id=14006

I think that's too much for the front page.
It mostly enhances information that is available here Getting Started.
Can you retarget your texts?
Also there already is a community menu button.

Also there already is a community menu button.

Ah, right so there is. I'll close this and take another look.


Expand Omeda promo code handling

This PR adds the following configuration options to the Omeda+IdentityX package. Each of these can be overridden by specifying a new value in the sites startServer handler.
@zarathustra323 I think this covers the conditions we talked about regarding sticky codes and site customization. Let me know if I missed something!

Param
Default Value
Description

omedaPromoCodeCookieName
oly_promo_src
The cookie/query parameter that should be checked for promo codes

omedaPromoCodeStripQueryParam
true
If enabled, remove the query param from the url after setting a cookie (redirect)

omedaPromoCodeDefault
undefined
The promo code that should be set if there is none available from the request (cookie/query param)

omedaPromoCodeCookieName
The specified cookie will be set from the same named URL parameter, similar to the oly_enc_id handling. If this cookie is present on the request, the value will be used as the promo code sent with the rapid identification calls.
omedaPromoCodeStripQueryParam
If enabled, the promo code query parameter will be stripped from the URL when detected.
omedaPromoCodeDefault
A global default promo code value to set, if the user has no specific promo code.
In practice, this means the following:

An anonymous visitor accessing a base-cms site with a URL parameter of ?oly_promo_src=foo.bar.baz will have the promo code FOO.BAR.BAZ set. If the visitor then attempts to log in via IdentityX (or returns later to do so), this promo code will be sent with the Omeda rapid identification call.
An authenticated visitor accessing the site with a new promo code will have the promo code stored. Each request that rapidly identifies them in the future will include this promo code.
If a visitor returns to the site with a different promo code (baz.bar.foo), the original promocode will be used and the new promo code will be discarded.

 Add promo codes to Identity-X configuration

@solocommand can you update the PR description to reflect the changes you made?


Image for legacy 32bit ARM SoC, such as RaspberryPI 3b 3 2b or zero..

As title, I found that  eclipse-leda-raspberrypi.tar.xz  in distro releases is still RaspberryPI4 image,  need 32bit ARM SoC support.

There is documentation on what needs to be done to support more hardware devices.
We don't see the need for 32-bit support, as this is very outdated platform.
Could you maybe elaborate more on why you need to support such an old SoC?


[Feature] Exclude Files

Is this feature missing in the latest version?

[X] I'm using the latest release

Is your feature request related to a problem? Please describe.
I would like to see which files in a particular folder have been modified.  This much is already possible, but I would like to exclude a certain list of files within that folder from being reported on.  Plus, I only want to look at that folder and not in folders within that folder.
Describe the solution you'd like?
I'd like to be able to not only specify a folder to monitor for changes but also

Exclude certain files within that folder to be monitored for changes
Exclude all folders within that folder to be monitored for changes

Describe alternatives you've considered?
No response
Anything else?
No response
Code of Conduct

[X] I agree to follow this project's Code of Conduct

Hi @SachinKainthSmarsh I believe that is already supported.
You can prefix patttern passed to the files input with ! or pass the patterns as is to the files_ignore input


[URFC] luci-app-ddns: Fix detection of wget in use

This is an Untested Request For Comments.
This pull request becomes relevant if https://github.com/openwrt/packages/pull/2739
(because of https://github.com/openwrt/packages/pull/2729, because of
https://github.com/openwrt/packages/issues/2728) is accepted in the packages feed.
It updates the detection algorithm for determining which version of wget (if any)
will be used by ddns-scripts and tweaks some capabilities related settings based
on the better information.
Signed-off-by: Daniel Dickinson<EMAIL_ADDRESS>

As stated, I see no need for openwrt/packages#2739 discussion.
For the offered optimisations I have to look at but I'm currently very short on time.

I've updated the patch to avoid using opkg to figure out what is wget, but for GNU wget, use wget -V to figure out if the ssl vs non-ssl version.  This patch doesn't actually depend on https://github.com/openwrt/packages/pull/2739 and work with or without it.

Fixed patch to avoid using opkg to figure out wget version in absence of -ssl -nossl separate binaries.

Does not actually depend on https://github.com/openwrt/packages/pull/2739; once tested it should be usable regardless of whether that patch is applied or not.

Based on the realization that the issue with wget symlink was the postrm/postinst mangling and not the symlink itself, I have modifed the wget PR to keep the wget-ssl and wget-nossl binaries, which makes the original purpose of this PR moot, however I also realized that the wget-nossl case was not actually detected by current logic and have therefore modified this PR to be about detecting wget-nossl.

Have fixes to push, then will un-URFC.

Pushed a fix for breakage in all cases, but still need to test with wget and wget-nossl


bug( ): Extra API calls when viewing a collection detail page directly

Describe the bug
When the user views a collection detail page directly e.g by pasting the URL into the address bar or following a link on an external site, extra requests to the back end are made.
To Reproduce
Steps to reproduce the behavior:

Go to https://cellxgene.cziscience.com/collections/296237e2-393d-4e31-b590-b03f74ac5070
in the network tab see the extra requests.

Expected behavior
The collection detail API (/v1/collections/{collection_id}) is only called once
Environment
All
Additional context
The fact that the collections endpoint is being hit at all for the detail page is addressed in a separate ticket:
bug( ): Extra API calls to collection endpoints when accessing portal collection detail page#4075

Adding to the Curators can filter Collections by curator and status epic as the FE caching and refresh strategy is related to this issue. Hoping we can fix this with the changes for the epic.


ENH/TST: add makedirs with mode fix, make_dir_tree

Closes #84

Rebased to get those test fixes from other PRs and modified to use os.makedirs.


Non blocking Channel#send?

Instead of having to write
select
when @channel.send 1 then true
else false
end

and paying the to_a price in select_impl when wanting to send to a Channel non blocking.
In one of our apps we use this pattern a lot:
@flow = true
@flow_change = Channel(Bool).new

def wait_for_flow
  return if @flow
  until @flow_change.receive
  end
end

def flow=(val)
  @flow = val
  while @flow_change.send?(val) # notify all flow change subscribers
  end
end

So the flow_change channel may or may not have receivers.

I think it might be good to add this behavioral variant.
But the name #send? is confusing. It would suggest similar semantics as #receive?, which returns nil instead of raising ClosedError. But that's something different. This method still raises ClosedError. The only difference to #send is that it returns instead of waiting when immediate delivery is impossible.
Not sure what would be a better name, tho. #try_send?? 🤔

I think instead we should change the code to avoid the to_a call. For example there's no need to call to_a if the Indexable has a single element.

I tried to optimize this a bit, but the main issue is that even with the optimization the code is translated to create a SendAction, which is a class so it allocates memory.
I think the right thing to do is, yes, to introduce some form of send?, but also change the expansion of select ... with just one channel.send to use send? instead.
In my mind select should be the only primitive to use (like in Go) and it's the compiler's responsibility to optimize it for the single case, not the user's responsibility.

I'm also surprised to learn that a select expression creates a class for each of the actions (like SendAction) so memory is allocated on every select. Really bad! I think it should be done in a way that no memory is ever allocated. Like, probably SendAction should be a struct, and we create a static array to hold those values in the stack. Then use pointers to refer to those. I think Go might be doing the same thing.
Then we can optimize for the case of 1 select action. In the end it will be as efficient as send?, I think.
It's more work for the compiler/us, but it's work that only needs to be done once by us, and not put the burden on developers.

Also it seems an Indexable is passed to select_impl. In the actual implementation that's a tuple. But that's not good if we need to sort the values. So I would change the implementation to use a Static Array and pass a slice to the method. Then the slice is mutable so the data can be sorted in-place. And maybe then the data can also be structs to avoid memory allocations. But I see some pointers being passed around, so it's a tricky refactor. But even though it'll take some time to get it right, it's probably the best thing to do.

In my mind select should be the only primitive to use

Considering that a very common use case is to wrap the select in a loop, I could see a second variant of select that comes with the loop built in and some predefined exit behavior semantics. That would allow for optimizations that is not really reasonable with select as it looks today. For example if you are selecting off several different things it would allow you to not tear down and then set up the waiting again each iteration of the loop - it would suffice to redo the one that triggered.

Hi @carlhoerberg , we'd like to move forward with this. What do you think of straight-shoota's observations? Do you want us to take it to completion? Thanks!

we'd like to move forward with this

Really? Could you show an equivalent API for Go? And if Go doesn't have one, why do we need one? I think it might be better to fix select. I know it's harder, but it's the right thing to do.

Isnt this less verbose than any select, fixed or not?

I guess that's right. My only concern is that eventually if we fix select to be as fast as try_send then there will be two ways to do the same thing. Maybe it's not a big deal. We could also eventually rewrite select to try_send if we detect they are the same.

The implementation seems correct, but I worry about increasing Channel's API. It's easy for things go wrong and not having a single code for sending things through the channel seems like a bad idea. In this case the whole method is blocking so it should be safe, but in general things like https://github.com/crystal-lang/crystal/blob/1b932187607adf7abb41968c8a36e62e2c2dbebd/src/channel.cr#L453-L457 makes me shiver.
We have receive/receive? which are both blocking, having try_send to be non-blocking seems confusing and opens the question whether we are going to add try_receive/send? for consistency. IMO it will not be trivial to remember that ? is blocking but try_ is not.
I also wonder if people would expect to write select; when ch1.try_send ...; when ch2.try_send ...; which will not compile because there is no matching _action method.
If select;when ch.send x then true; else false; end is a common pattern, I would prefer to put that in a macro or method and optimize the select_impl implementation a bit.
From what I see, it seems that the culprit of the to_a is only: https://github.com/crystal-lang/crystal/blob/1b932187607adf7abb41968c8a36e62e2c2dbebd/src/channel.cr#L426-L431
Did someone check if doing a special case to use a Slice to point to a single element when ops.size == 1 is enough to avoid the extra allocations?
If that is not enough, in order to keep the API as today we could have a private def self.select_impl_single(op : SelectAction, non_blocking), with more optimizations that can be applied for the single channel case, even at the cost of some internal code duplication that can be solved/refactored later.
A bonus of either last approaches is that will also apply to select; when ch.receive; else ... directly.

@bcardiff Sounds like a good assessment. 👍
Optimizing #select_impl should happen anyways. For a single action (or even a small number) we can avoid an array allocation. Additionally, we can try to refactor SendAction without heap allocations as suggested in https://github.com/crystal-lang/crystal/pull/12694#issuecomment-1296236424
Thus the lengthy form with a single select branch should be as performant as the custom implementation of #try_send.
I suppose at that point we could technically even implement #try_send based on that so that all Channel interactions go through a single point.
But then we could also consider other alternatives for this pattern in order to keep the Channel API succinct and avoid potential confusion about #try_send.

I was thinking that something like https://github.com/crystal-lang/crystal/compare/master...bcardiff:crystal:select-optimizations would avoid the array heap allocation, and can will cover single and dual action selects. WDYT?
Essentially it applies the following logic to keep a tuple of ops when possible for the locks.
def f(*ops)
  f_impl ops
end

def f(ops : Indexable)
  f_impl ops
end

def f_impl(ops : Indexable)
    if ops.is_a?(Tuple) && ops.size == 1
      f_impl_with_locks(ops, ops)
    elsif ops.is_a?(Tuple) && ops.size == 2
      ops0 = ops.fetch(0, nil).not_nil!
      ops1 = ops.fetch(1, nil).not_nil!

      case (ops0 <=> ops1)
      when 0
        f_impl_with_locks(ops, {ops0})
      when 1
        f_impl_with_locks(ops, {ops1, ops0})
      when -1
        f_impl_with_locks(ops, {ops0, ops1})
      else
        raise "unreachable"
      end
    else
      ops_locks = ops
        .to_a
        .uniq!
        .sort!

      f_impl_with_locks(ops, ops_locks)
    end
end

def f_impl_with_locks(ops : Indexable, ops_locks)
  puts({typeof(ops), typeof(ops_locks)})
end

f 1         # => {Tuple(Int32)              , Tuple(Int32)}
f 1, 2      # => {Tuple(Int32, Int32)       , Tuple(Int32, Int32)}
f 1, 2, 3   # => {Tuple(Int32, Int32, Int32), Array(Int32)}
f [1, 2]    # => {Array(Int32)              , Array(Int32)}

The runtime operations for the non blocking send is basically the proposal of this PR, so I expect it to work as efficient.

I think it's easier than that. We can just turn the tuple into a StaticArray for sorting. There's no need for special treatment for individual cases. This improves performance for all sizes of ops, as long as the size is known at compile time.
I have a patch for that ready that goes on top of #12814. It would work without it, but I'm batching it avoid conflicts.
On top of master the simplified diff would be this:
@@ -423,8 +423,12 @@ class Channel(T)
   private def self.select_impl(ops : Indexable(SelectAction), non_blocking)
     # Sort the operations by the channel they contain
     # This is to avoid deadlocks between concurrent `select` calls
-    ops_locks = ops
-      .to_a
+    if ops.is_a?(Tuple)
+      ops_locks = ops.to_static_array
+    else
+      ops_locks = ops.to_a
+    end
+    ops_locks
       .uniq!(&.lock_object_id)
       .sort_by!(&.lock_object_id)

The implementation of Tuple#to_static_array would be analog to StaticArray#[], returning StaticArray(Union(*T), {{ T.size}}). This can also be an internal helper method, but IMO it would be useful to have this publicly available. Doesn't matter though for select_impl.

Something I wanted to avoid is to have a union in ops_locks, so there is no multi-dispatch on each iteration.
Another thing that worries me is that Tuple#to_static_array will expand multiple defs. One for each combination of select actions used in the entire program. They should not be many but we are not forcing an upper bound unless we do something like I proposed.

The runtime operations for the non blocking send is basically the proposal of this PR, so I expect it to work as efficient.

I was wrong, we still have the allocation of the SelectContext ary pointed out. But I would start with optimizing how locks are computed. Seems we agree there.

Okay, yeah multi dispatch is indeed an issue. But we can't really prevent that on scale, unless we multiply all possible combinations.
I suppose we can combine both approaches and use the tuple sorting you suggested for small sizes.
But for the else branch I presume the StaticArray solution should be an improvement over to_a. For T.size == 1 it probably doesn't make any difference whatsoever. So we're effectively optimizing for the case T.size == 2. Perhaps we could do the same for T.size == 3, but the tuple combinations just explode from there.

Actually, your implementation doesn't avoid multi dispatch either, does it? The type of ops.fetch(0, nil).not_nil! is Union(*T).
Consequently:
f 1      # => {Tuple(Int32)              , Tuple(Int32)}
f 1, 2u32      # => {Tuple(Int32, UInt32), Tuple(Int32 | UInt32, Int32 | UInt32)}
f 1, 2u32, 3   # => {Tuple(Int32, UInt32, Int32), Array(Int32 | UInt32)}
f [1, 2u32]    # => {Array(Int32 | UInt32), Array(Int32 | UInt32)}

Tuple(Int32 | UInt32, Int32 | UInt32) is not an improvement over StaticArray(Int32 | UInt32, 2) (one could argue that it's pretty much the same thing).
To avoid those union types, we need to switch over the tuple size at compile time. This should work with macros:
def f_impl(ops : Tuple(*T)) forall T
  {% if T.size == 1 %}
    f_impl_with_locks(ops, ops)
  {% elsif T.size == 2 %}
      case (ops[0] <=> ops[1])
      when 0
        f_impl_with_locks(ops, {ops[0]})
      when 1
        f_impl_with_locks(ops, {ops[1], ops[0]})
      when -1
        f_impl_with_locks(ops, ops)
      else
        raise "unreachable"
      end
  {% else %}
    f_impl(ops.to_a)
  {% end %}
end

I suppose we could also look into alternative solutions to maybe avoid sorting altogether.
Considering that the size of ops is typically not that big, maybe iterating multiple times wouldn't be such a bad option.

I don't follow that latest example. Where would those methods be called?
Looks like they are overloads of the splat variant of non_blocking_select which are actually unused and proposed for removal in #12813.

I thought the compiler would generate those calls. Either way doing a dispatch on runtime depending on the indexable size to a method that will avoid unions in the ops_locks can still be done. That's the core part of the idea.

Yeah, sure that can be done. It's only feasible for very small sizes, though. Not even sure we would want to go higher than 2 or maybe 3.
And I wouldn't expect the actual performance improvement to be very noticeable. At least there are more effective measures, such as avoiding heap allocations.

I honestly wouldn't worry about multidispatch. We have LLVM here.
Here's an example:
struct Int32
  def foo
    self
  end
end

class String
  def foo
    bytesize
  end
end

def sum(values : Union(Int32, String)[2])
  values.sum(&.foo)
end

fun my_awesome_function : Int32
  values = uninitialized Union(Int32, String)[2]
  values[0] = 1
  values[1] = "hello"

  sum(values)
end

puts my_awesome_function

We are initializing a static array whose type is a union. We put an int in the first position and a string in the second one. Then we "sum" the values, which would in theory result in multidispatch.
Compile the above program with --release --emit llvm-ir and search for my_awesome_function in the output .ll file. Here it is:
define i32 @my_awesome_function() local_unnamed_addr #1 !dbg !25766 {
exit.1.i:
  ret i32 6, !dbg !25767
}

LLVM figured out what was in each position and completely avoided the multidispatch. Not only that, it computed everything at compile-time.

Unrelated to the comment above (I'll reply to it soon), look at this:
https://github.com/golang/go/blob/eaf0e3d4650fd223dec84ee52025c7a82bcb24bd/src/runtime/chan.go#L677-L715
So Go decides to implement different select patterns in different ways, probably optimal ones.
I think we could do the same. That is, instead of introducing try_send, detect this pattern and call a specific method (which would be nodoc.)
That way, at least for this issue, there's no need to optimize the general case.


resistFingerprinting appears to make windows size weird and unique

resistFingerprinting set to true:

set to false:

Should resistFingerprinting be turned on and is there a way to avoid this problem
tested with panopticlick

Do you have some big taskbar as a some launcher? You get an unfortunate screen size with RFP:

bits of identifying information
one in x browsers have this value

A solution is to set maxInnerWidth and maxInnerHeight:
https://github.com/ghacksuserjs/ghacks-user.js/issues/449#issuecomment-399765829
It's under 4502:
https://github.com/ghacksuserjs/ghacks-user.js/blob/master/user.js#L1633

issue 1: you're doing it wrong
When RFP = false, Panopticlick is getting your actual screen size. When RFP = true, it is getting your browser's inner window. Go here and see all the measurements (screen, available screen, outer  window, inner window, etc). This is what RFP is protecting you from - making sure all the measurements are reporting the same as your inner window.
RFP not only does that, but also sets the size of NEW windows to 1000 wide x 1000 max high (and reduced by 100s if not enough height available). Just toggling the pref to true will not resize your existing window(s) .. and by the look of it, you are maximized
issue 2: you're reading it wrong
This is pasted from a reply I did here

I’ll try to keep this short. At the end of the day, due to the nature of sites like Panopticlick, AmIUnique etc, the results are tainted. The reasons for this are:
– they interest and attract people who are interested in privacy (usually with tweaks and extensions etc), so it’s not representative of the “real world”. For example FF’s RFP hash for a 10×10 white canvas is one in 6.73 browsers. That’s almost 15%. That’s over double Firefox’s entire worldwide stats on all platforms. That’s not to say that extensions don’t also use this, but most of them by default fake randomly, and extensions aren’t even used that much (what was that figure? half of FF users have less than 2 extensions or non at all?). This is probably the best example, as canvas produces probably the highest entropy – you’re basically always unique, so it’s the first thing EVERYONE tries to beat – cogito ergo sum: now the results are so far off real world, they actually disportionately poison the overall result.
– results are tainted by repeat visits of said users, constantly trying to tweak their setups
– the datasets are small: that’s not the sites’ fault (yes half a million is small)
– the datasets can be outdated with old information, e.g user agents. Panopticlick only recently changed to a last 45 days (which is a good move), but this then plays into a smaller sampling (see previous point)
– the tests are outdated or limited
– some large scale real world tests in the last year confirm all of this
– PS: That’s not to say they are useless, they are great for seeing what your browser returns.
There’s a disconnect here from users understanding that while you might be in a pool of 1% of users (e.g say Tor Browser users), that the pool is massive. And what drives FP resistance is tightening up the bucket you are in – by enforcing the FP to be the same. Math is your friend. Lowering entropy (instead of raising it randomly, which just adds “information paradoxes” and breakage) is your friend. There is no point in hiding your browser (Firefox vs chrome), or OS (major platform) or UA as all they do is create more entropy (if trying to raise entropy randomly this is not an issue). And each spoof adds entropy in its own right, not just an overall “is this user using anti-FP’ing). Let’s say you spoof your OS and they can tell you’re not from the TCP/IP stack – that’s entropy. Lets say you spoof your browser from FF66 to FF60, that’s more entropy, and so on. It’s not more entropy when it’s enforced on a set of users. It is more entropy if you look at it worldwide.

 I shall refer to their face recognition algorithms as a test case for you.

k thanks


Faulty outline on method organization

Issue Report Checklist

[X] Searched the issues page for similar reports
[ ] Read the relevant sections of the Spyder Troubleshooting Guide and followed its advice
[X] Reproduced the issue after updating with conda update spyder (or pip, if not using Anaconda)
[ ] Could not reproduce inside jupyter qtconsole (if console-related)
[X] Tried basic troubleshooting (if a bug/error)

[X] Restarted Spyder
[X] Reset preferences with spyder --reset
[ ] Reinstalled the latest version of Anaconda
[ ] Tried the other applicable steps from the Troubleshooting Guide

[ ] Completed the Problem Description, Steps to Reproduce and Version sections below

Problem Description
Say you have a class with many methods, and the methods can be 'grouped' by adding
### Marker at the beginning of a line between 2 'sections' of methods, then the outline is picking up on this, however the indentation (and the distinguishing between method and function) is not correct.
What steps reproduce the problem?
copy the following code fragment in a file, and look at what 'Outline' is making of it
# -*- coding: utf-8 -*-

class demo(object):

    def __init__(self):
        pass

    def foo(self):
        pass

### bars 1
    def bar10(self):
        pass

    def bar11(self):
        pass

    def bar12(self):
        pass

### bars 2
    def bar20(self):
        pass

    def bar21(self):
        pass

    def bar22(self):
        pass

What is the expected output? What do you see instead?
# bars 1 and # bars2 should reside one level deeper, under their demo class, and recognized as methods instead of functions now.
Paste Traceback/Error Below (if applicable)

PASTE TRACEBACK HERE

Versions

Spyder version: 4.1.1
Python version: Python 3.7.5 (default, Oct 31 2019, 15:18:51) [MSC v.1916 64 bit (AMD64)]
IPython version: IPython 7.13.0 (but probably not related)
Qt version: 5.9.6
PyQt version: PYQT_CONFIGURATION = {'sip_flags': '-t WS_WIN -t Qt_5_9_6'}
Operating System name/version: WIN10 (probably the same on OSX and Linux, but not tested by me yet)

Dependencies

# Mandatory:
atomicwrites >=1.2.0           :  1.3.0 (OK)
chardet >=2.0.0                :  3.0.4 (OK)
cloudpickle >=0.5.0            :  1.3.0 (OK)
diff_match_patch >=20181111    :  20181111 (OK)
intervaltree                   :  None (OK)
IPython >=4.0                  :  7.13.0 (OK)
jedi =0.15.2                   :  0.15.2 (OK)
nbconvert >=4.0                :  5.6.1 (OK)
numpydoc >=0.6.0               :  0.9.2 (OK)
paramiko >=2.4.0               :  2.7.1 (OK)
parso =0.5.2                   :  0.5.2 (OK)
pexpect >=4.4.0                :  4.8.0 (OK)
pickleshare >=0.4              :  0.7.5 (OK)
psutil >=5.3                   :  5.7.0 (OK)
pygments >=2.0                 :  2.6.1 (OK)
pylint >=0.25                  :  2.4.4 (OK)
pyls >=0.31.9;<0.32.0          :  0.31.9 (OK)
qdarkstyle >=2.8               :  2.8 (OK)
qtawesome >=0.5.7              :  0.7.0 (OK)
qtconsole >=4.6.0              :  4.7.1 (OK)
qtpy >=1.5.0                   :  1.9.0 (OK)
rtree >=0.8.3                  :  0.9.3 (OK)
sphinx >=0.6.6                 :  2.4.0 (OK)
spyder_kernels >=1.9.0;<1.10.0 :  1.9.0 (OK)
watchdog                       :  None (OK)
zmq >=17                       :  18.1.1 (OK)

# Optional:
cython >=0.21                  :  0.29.15 (OK)
matplotlib >=2.0.0             :  3.1.3 (OK)
numpy >=1.7                    :  1.18.1 (OK)
pandas >=0.13.1                :  1.0.2 (OK)
scipy >=0.17.0                 :  1.4.1 (OK)
sympy >=0.7.3                  :  1.5.1 (OK)

Thanks for the bug report @nerohmot!
@steff456, please take a look at this one.

I didn't know you could do that.
Unfortunately the ### marker "grouping" isn't the feature(not by design at least) that's the bug. If you put the markers at the correct indentation level for the class it works as expected.
If desired, this type of arbitrary grouping would need to be done in a way that doesn't rely on indentation so it doesn't break python code. We would need to use beginning and end markers (maybe # ## and # \## or # << and # >> see #8855).
If python read comments, then the example code here would be an indentation error. This same problem happens with users using code cells inside classes and functions and so far my response has been please don't do that. Which is good advice because it leads to making broken code in your code cells.
One solution is making markers ignore indentation and never create groups. As for telling were they should go in the tree, I guess markers comments could always be siblings with the next line of code.

One solution is making markers ignore indentation and never create groups

I think this is the best solution for now. Could you help us with this @bcolsen?

I think the best solution might be to drop it entirely. ### (or more) is also only match for the marker and that's a code style error.  It's easy to implement dropping it and it could speed up the outline explorer instead of slowing it down.

One solution is making markers ignore indentation and never create groups. As for telling were they should go in the tree, I guess markers comments could always be siblings with the next line of code.

To implement this I would have to know what the level of the next tree is and match the indentation.

I'll have a look at the code.

I've noticed something similar...  this also produces a wrong grouping:
class abc():
    def f_1():
        return

        if True:
            def sub_f_1():
                return

        def sub_f_2():
                return

@steff456, please take a look at this one because I'd like to include a fix for it in our next release.

I worked on this today and I have a solution. I'll have a PR soon (tomorrow if all goes well)

Great to know!! Thanks a lot @bcolsen!
But at the end we decided to allow comments of the form ### to split the outline. Are you working along those lines?

But at the end we decided to allow comments of the form ### to split the outline. Are you working along those lines?

I don't have a great view of how that would be possible without using the tab level of the comments.
For example, I would like them to work as the original poster wants:
class demo(object):

    def __init__(self):
        pass

    def foo(self):
        pass

    # ## bars 1
    def bar10(self):
        pass

    def bar11(self):
        pass

    def bar12(self):
        pass

    # ## bars 2
    def bar20(self):
        pass

    def bar21(self):
        pass

    def bar22(self):
        pass

Would give:
C: demo
    M: __init__
    M: foo
    #: bars 1
        M: bar10
        M: bar11
        M: bar12
    #: bars 2
        M: bar20
        M: bar21
        M: bar22

Do we want this to do with:
# ## top level
class demo(object):

    def __init__(self):
        pass

    def foo(self):
        pass

Does it fold until the next comment?
If so, one simple way to implement that is to make half levels(I would implement as even and odd ints) have the comments on a half levels based on their tab level.
A class or function would have a tab level of 1,3,5,7,... and comments would have 0,2,4,6,... based on the indentation level of the comment. This would allow the code to have a lot of flexibility, but the indentation level would matter.
Without using indentation level there is a some ambiguity whether a comment is inside or outside a class or function.
What did you have in mind?

Here is another example for you guys...

@KardoPaska If you want to see how the fix is going. I could use some help in testing my https://github.com/spyder-ide/spyder/pull/12343


dev VM: bump k8s version to v1.14.0, CNI to 0.7.5 and etcd to v3.3.12

Signed-off-by: André Martins<EMAIL_ADDRESS>

This change is

test-me-please

Coverage increased (+0.05%) to 42.67% when pulling 3c013be8dfc9f230abec063f40d1368c297987f2 on pr/bump-k8s-dev-VM into 3a570b932dc54895546ce950075629f0f7d607e6 on master.


[Bug] Contacts import have some issues

Hi,
First, thank you for making this great stuff. I’ve imported my contacts using this, but I face several issues:

Groups (=Labels in Cozy) are not imported, which is very annoying. I have around 2500 contacts in ~50 groups (a lot of contacts being in more than one), and I can’t afford the time to relabel everyone.

Default field name for addresses (email+physical) are not translated. For instance, Home stays this way even if I’m using french. Maybe this is an issue for the contacts app though, please tell me if I need to report this over there.

Physical addresses are imported with all fields separated by comma, even if they are empty. For instance, I had:

,,1, Rue de Paris,Paris,,75000,France because only Street, City, ZIP and Country were filled in Google
,,1, Rue de Paris Paris 75000 France,,,, because this was a wrong entry in Google that had everything in Street

So, maybe empty field and the associated comma should not be added to the field on import. For the last example, I don’t expect you to fix it on import, but if you do by implementing some parsing together with https://github.com/cozy/cozy-contacts/issues/189, that would be wonderful. Speaking of that correlated issue, it might also change the way of importing addresses and make it a lot easier by just copying over the different fields in the detailed address view at Google to Cozy ones.
Finally, a way to remove the Google label created on import would be nice, but maybe that does already exists and I just wasn’t able to found it.

Note that 3. has been almost been fixed on the contacts app side (2.1.1). 1. and 2. are still valid though.

Thanks for your great feedback @ArchangeGabriel ☺
I opened a specific issue to add groups: #41
@m4dz what about the second point: should I open another issue here or in cozy/cozy-contacts ?

Thanks again for your precious feedback @ArchangeGabriel :smile:!
For the second point, I'm aware about the translation of datapoints' labels. I'll try to do it this sprint, no need to create a specific issue in Contacts.

Thanks, so I'll close this one and keep #41 for the last point.
Thanks to both of you!

@clochix, @m4dz: you’re welcome! It’s a pleasure to help your guys build this great piece of software the way I can. ;)


Clean up scaffolded code and show UseRouting position

I'd like to clean up the scaffolded code here, specifically the ILogger<T> and its injection. I'd also like to add UseRouting() to the middleware snippet. Can I go ahead and do that?

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

ID: 6f9d7ad5-47d6-80bf-88fe-fac3a74151a3
Version Independent ID: e64816f4-3391-5aab-31c2-ab4f915bc7f6
Content: Facebook, Google, and external provider authentication without ASP.NET Core Identity
Content Source: aspnetcore/security/authentication/social/social-without-identity.md
Product: aspnet-core
Technology: aspnetcore-security
GitHub Login: @Rick-Anderson
Microsoft Alias: riande

Yes, please do.


About ranking metric evaluation

Hi,
I'm wondering if it is possible to evaluate the ranking metrics of MAP(mean average precision) @ K and HR(hit ratio) @ K of GBMF/GBGCN under librecframework. If yes, how can I modify the code? Thanks.

You can modify the framework https://github.com/Sweetnow/librecframework/blob/master/librecframework/metric.py.
Just follow the current code,

create a new class derived from Metric
implement __call__ (score is the model output, ground_truth is the corresponding data from Dataset.__getitem__
plus the metric into self._sum
plus the count into self._cnt
register the metric class into _ALL_METRICS

And then, you can choose the new metric in config.json by its class name.

BTW, I think HR is the same as Precision.


[tests-only] Extend API test coverage for MOVE between spaces (#8113)

Port PR #8113

Requries PR https://github.com/owncloud/ocis/pull/8102 to be backported to stable-5
@ScharfViktor @micbar Will that happen?

403 was a mistake and will be rolled back to 502
Ref: https://github.com/owncloud/ocis/issues/8063#issuecomment-1883281975


doc: fix helm chart URL for the install process

Closes #149
Signed-off-by: Jonathan Gonzalez V<EMAIL_ADDRESS>

/ok-to-merge


[Definition] Avoid parsing lockfile twice on init

\c @chrismo

oh - yeah, good catch. thx

@homu r+

:pushpin: Commit d57220b has been approved by segiddins

:hourglass: Testing commit d57220b with merge 0f83e5a...

:sunny: Test successful - status


New version of SUMO

I know that Flow supports the 1.1.0 version of SUMO. However, the latest version of SUMO is 1.3.1.
I need some TraCI functions which are supported from 1.2.0.
Do you have a plan to upgrade the supported SUMO version?
If no, what problem is expected to occur if I integrate the latest version of SUMO to the current flow version?
Thank you in advance.

Hi, we are putting together our release schedule for the term and will put it out publicly shortly. In the meantime, I don't know of any a-priori upgrade issues but I'd bet on their being some. Upgrading for us is a slow process since we have to ensure that each RL environment is stable across SUMO releases; it's perfectly plausible that some new SUMO behavior causes the environments to become unsolvable or quite different than they were in our published papers.

Point is noted though and we will look into upgrading as we finalize our release schedule.

Just in case I haven't sent this to you, there's a flow slack users group for these sort of conversations: https://bit.ly/34L4IaW

Any update on this? Can Flow 0.5.0dev support SUMO 1.4?


How to get x coordinate from right side instead of left side?

Hi, I am using below function for getting X and Y coordinate from PDF:
As per current coding we are getting X coordinate from left side and Y coordinate from bottom.
Util.applyTransform = function Util_applyTransform(p, m) {
var xt = p[0] * m[0] + p[1] * m[2] + m[4];
var yt = p[0] * m[1] + p[1] * m[3] + m[5];
return [xt, yt];
};
Util.applyInverseTransform = function Util_applyInverseTransform(p, m) {
var d = m[0] * m[3] - m[1] * m[2];
var xt = (p[0] * m[3] - p[1] * m[2] + m[2] * m[5] - m[4] * m[3]) / d;
var yt = (-p[0] * m[1] + p[1] * m[0] + m[4] * m[1] - m[5] * m[0]) / d;
return [xt, yt];
};

Closing as INCOMPLETE, since there's unfortunately nowhere near enough information provided to make this actionable/valid; please note https://github.com/mozilla/pdf.js/blob/master/.github/ISSUE_TEMPLATE.md


Create .nuspec files from project file

create a nuspec from a project file, usable from commandline, can be integrated into paket restore

@forki could use some review

What does it change in the workflow?

right now nuget is creating the nuspec that's getting trimmed down before paket packs the output into a nupkg, so this will let us remove nuget from another point in the build pipeline.


Like gone after using the TumbleThree

Hi,
I have more than 2400 likes and yesterday I used TumbleThree. It stopped downloading the likes after 525 from over 2400. Today I tried again and looked into my account ... now I only have 337 likes.
How is that possible?
I have done nothing else to my likes and account.
In case it matters: it is mostly explicit material.
THX
C

Hi,
how old were your likes? Are you sure they all have been still valid? I guess that most of them obviously were already deleted or marked for deletion. It's not immediately updated in every user's likes list.
They are constantly removing explicit blogs.

The issue has been closed. You can still comment. Feel free to ask for reopening the issue if needed.


Screen shot is outdated

Screenshot of Update Compliance workspace Overview shows Windows Defender.  This is not available in the solution today

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

ID: 4cf1e851-9449-cd9d-0f8a-af797154ca88
Version Independent ID: 9ce3da25-2bfc-d558-1853-0d16d8f26c40
Content: Using Update Compliance (Windows 10) - Windows Deployment
Content Source: windows/deployment/update/update-compliance-using.md
Product: w10
Technology: windows
GitHub Login: @jaimeo
Microsoft Alias: jaimeo

@chinglis can you update?


add: back to home button

Related Issue
Closes: #510
Description of Changes
Added: back to home button with animations

in blogpost page
in about page
in contact us page
in log in page

Checklist:

[x] My code adheres to the established style guidelines of this project.
[x] I have conducted a self-review of my code.
[ ] I have included comments in areas that may be difficult to understand.
[ ] I have made corresponding updates to the project documentation.
[x] My changes have not introduced any new warnings.

Screenshots

Original
Updated

Please provide any necessary screenshots to illustrate the changes made.

Hey, Nice work..but I will suggest make a common component and use it wherever it needed of hardcoding everywhere :)

@rushabhpanbude  thank you for the suggestion. Will make sure to incorporate it next time.


Vendor dependencies

Let's not use godep pls

me running godep save -r
see the comment
drop my keyboard
@LK4D4  oookki :cry:

I've been using https://github.com/Masterminds/glide for a few smaller projects and I've been pretty happy with it.

gb?

I've been using https://github.com/Masterminds/glide for a few smaller projects and I've been pretty happy with it.

Glide + vendor/

@runcom I'll try glide today.

Ok, glide just copies whole dependency repo. I think I'll copy vendor script from docker :)

+1 I noticed that as well :P

@runcom I wrote https://github.com/LK4D4/vndr just for test. It behaves similarly to hack/vendor.sh from docker. It requires go tip, though :)

Gonna try it @LK4D4 :)

@LK4D4 Yes, the docker vendor script is so simple :)

@mrunalp not so simple if you'll take a closer look. It parsing arches from Dockerfile, rewriting imports and have one hardcoded package :)

@LK4D4 I meant simple to use ;)


ComponentSelectionReason#isExpected gives different results between Gradle 5.0 and 5.1

We are using org.gradle.api.artifacts.result.ComponentSelectionReason#isExpected API however we noticed that it gives different results when moving from Gradle 5.0 to 5.1.1
Expected Behavior
I would expect consistent behavior in both versions for the same situation.
Current Behavior
It returns false in Gradle 5.0 it returns true in Gradle 5.1.1
Context
We have this debugger view comparison which shows that in version 5.1.1 there is an additional description requested at the end of the descriptions list. The logic of isExpected is driven by that last entry. However, other reasons show that we applied rules which caused an upgrade of the version of my library.
Gradle 5.0

Gradle 5.1.1

So looking into this, I am failing to reproduce that 5.1.1 state showed in the report.
I can make the isExpected code more resilient to ordering in these descriptions but I wonder if there is not another issue hiding here.
Any chance you can provide a reproducer ?


remove need of quotes for key values

Currently, you must wrap key values in quotes to assign:
player = {
  'x': 0,
  'y': 0,
  'health': 100,
}

Instead I'd like to be able to define key values without, as long as there is no space in the key:
player = {
  x: 0,
  y: 0,
  health: 100,
}

much more convenient, thank you, please keep your good work!


Nodejs Error: read ECONNRESET

Hi, sorry for bothering you, I have some problems while trying your code.
While I'm running npm, I opened several web pages that connect to nodejs server, but when one web page closed or disconnected, the whole server will crush, and return this: Error: read ECONNRESET
I googled for these and someone said this is because there isn't an error handler, but I have tried to add an error handler, and nothing worked.
I'm wondering if I did anything wrong...
Thank you very much.
This is a screenshot of this problem

but when one web page closed or disconnected, the whole server will crush

Line 33 in app.js suppose to filter everyone who was disconnected:
https://github.com/valiafetisov/unity-webgl-multiplayer/blob/ab3a25cdf81b83ac26d8438d89854b1835689c7b/node/app.js#L32-L33
But if client was disconnected between line 33 and line 39 it will produce this error.
Thank you for catching this! I've added an error handler for that and also a general error handler in #6.
Would you be able to checkout this branch and test that it works now?

Hi, thank you for responding me!
It still crushes with same error messages, I think there may be something wrong in my test steps?
Here are my steps:

Open a terminal, execute npm.
Open the Unity project, build and run.
Wait for Unity finishes building and a web page which connects to my localhost will pop up.
Open several pages that connect to my localhost.
Check all the pages are connecting correctly, then randomly close one or more pages.

After step 5, the ECONNRESET error may occur, if not, I will randomly refresh some pages, and the error almost definitely happens after refreshing.
And my Unity version is 2019.4.3f1.
Thank you again!

I was able to reproduce the error. I've updated the ws package and the error disappeared, see https://github.com/websockets/ws/issues/1256.
Please pull changes from #6 (or download fresh fix-econnreset branch) and npm install && npm start as usual.
Let me know if it solves the problem for you

YES IT WORKS!!! (I never thought that ws package could cause this xD)
The problem has been totally solved, thank you so much!!!! :D


why do not zero “data32” before use?

https://github.com/DMTF/spdm-emu/blob/5dd34068a580e96e3dc64d10ab35289ad4e35828/spdm_emu/spdm_requester_emu/spdm_requester_spdm.c#L399C13-L399C13
In the code linked above, the libspdm_get_data() function shall only assign a value to the first byte in data32. If data32 is not zeroed in advance, an abnormal value may be obtained, causing the next line of assert to trigger

libspdm_connection_state_t is enum type, which is 32bit. Why it only assign the first byte?
Do you run into some real issue? or you just think there might be issue?

libspdm_connection_state_t is enum type, which is 32bit. Why it only assign the first byte?
Do you run into some real issue? or you just think there might be issue?

libspdm_connection_state_t is enum type, which is 32bit. Why it only assign the first byte?
Do you run into some real issue? or you just think there might be issue?

It's a real issue.
In some platforms, the compiler will optimize the size according to the value range in the enum. For example, in the arm platform, the armcc compiler will optimize the enum size below the range of 255 to 1 byte.
Of course, we can add compilation parameters to force all enum sizes to be fixed at 32 bit, but this will increase some RAM consumption, which is not very friendly to the operating environment with small memory

Can change it from sizeof(data32) to sizeof(libspdm_data_type_t).

C standard said: The expression that defines the value of an enumeration constant shall be an integer constant expression that has a value representable as an int.
I think this is the compiler should do.

Okay, I understand what you mean, but I think what Steven Bellock said above will make the code more stable. This is just a suggestion


Menu Options Symbols Overwritten by Menu text

What steps will reproduce the problem?

Spyder 5 Menu options have a symbol then text.
For example, Print shows a printer symbol then text that should be "Print".
On my system (Windows 10, Python 3.8 Anaconda), the printer symbol has "Pr" text on top of it then "int" follows.
Same issue throughout makes menu options very hard to read.
PS. Could the green triangle for "Run" be made bigger/ darker? Hard to find at first.

Hey @robert99, thanks for reporting. This is already fixed in our 5.0.1 version, released last Friday and part of Anaconda soon.


Add Navigator.pushReplacement() and Navigator.pushReplacementNamed()

Replace the current route by pushing a new route and disposing the old one.
Route.didPush() now returns a Future that completes when the push operation completes.
Fixes: https://github.com/flutter/flutter/issues/7365
Fixes: https://github.com/flutter/flutter/issues/6544


[GDScript] Fix incorrect compound assignment

Reverts in-place compound assignments
Added test to ensure correctness
Added tests for array assignments as well, as per #76041
Found that this causes further bugs and decided to just revert the change and leave it up for future fixes as it seems very tricky and many pitfalls
This reverts #72056
Fixes #75832
Fixes #76041
Regression from #72056

As this is a relatively destructive bug with very confusing effects and unpredictable I feel reverting is best, as it was a performance change

i can confirm that this fixes #76041

Thanks!

Thank you!


Have AbstractThrowableAssert extend AbstractObjectAssert?

Summary
Currently, it is only possible to assert on the message and cause of a throwable. As many custom exceptions have fields that we'd like to verify it'd be useful to have AbstractObjectAssert's assertions available.
Real life use case / example
assertThatThrownBy(() -> callService()).hasFieldOrPropertyWithValue("status", Status.ILLEGAL_ARGUMENT);

If combined with #662, can be even more type-safe
assertThatThrownBy(() -> callService()).asType(MyException.class).extracting(MyException::getStatus).isEqualTo(OK);

Java 8 specific ?
NO
If the issue Java 8 specific, the Pull Request must be based on the master, otherwise the 2.x branch must be used.

I agree we could have done that.


Fix clippy errors from 1.72

This fixes clippy errors with the version of clippy shipped with 1.72

I don't really mind either version - the newer version is arguably a bit clearer to me, but I can see why the more verbose might be preferable. Honestly, given the toss up, I'd prefer to just merge this as is and not have to wait for another CI run.


deal with U.S. units

When querying for how many X in Y, sometimes response are given in U.S. units, which was pronounced as "us." This separates the two letters when followed by one of the units listed in the regex expression.

Thanks for this - I've actually already put a handler to deal with the term U.S. more generally in the next version. Feel free to try it out here
https://github.com/tartanguru/alexa-google-search/tree/Version-2-final


vscode

Checklist

[X] I have checked the moderation queue that no new version has been submitted (only visible when logged in), or the submitted package version has failed the automated checks.

Please navigate to the following link to view the moderation queue.

[X] I have verified that the new version is a stable release.
[X] I have looked to see if there are any Bug reports that prevents a new version being submitted and that no existing Outdated Reports have been created.
[X] I have verified the new version has a Windows binary (EXE or MSI installer, Zip or other archive, etc).

New Software Version
1.81.0
Download location
https://az764295.vo.msecnd.net/stable/6445d93c81ebe42c4cbd7a60712e0b17d9463e97/VSCodeUserSetup-x64-1.81.0.exe
Package Page
https://community.chocolatey.org/packages/vscode

Simply updating the download URL for 64-bit isn't sufficient.  The core issue is that the URLs used to check for newer version availability is out of date in automatic/vscode.install/update.ps1.  The versions can easily be scraped from https://api.github.com/repos/microsoft/vscode/releases/latest; however, the download URLs are not present in the return.
More investigation is needed to determine where the setup files can be found (in an automated fashion going forward).

One option is to monitor github releases or
https://code.visualstudio.com/updates redirects to latest version (version is not in the same format)
Download url seems to be
https://update.code.visualstudio.com/1.81.0/win32-x64/stable
So download url https://update.code.visualstudio.com/{{ release_version }}/win32-x64/stable should work.
In the update check of visual studio i find the following code:
	) -> Result<Release, AnyError> {
		let update_endpoint =
			VSCODE_CLI_UPDATE_ENDPOINT.ok_or_else(UpdatesNotConfigured::no_url)?;
		let download_segment = target
			.download_segment(platform)
			.ok_or_else(|| CodeError::UnsupportedPlatform(platform.to_string()))?;
		let download_url = format!(
			"{}/api/versions/{}/{}/{}",
			update_endpoint,
			version,
			download_segment,
			quality_download_segment(quality),
		);

https://github.com/microsoft/vscode/blob/85daa35afb86202badbee79532537bf226961f25/cli/src/update_service.rs#L68
But so far i have been unable to find VSCODE_CLI_UPDATE_ENDPOINT

The download url is documented here https://code.visualstudio.com/docs/supporting/faq#_how-do-i-do-an-offline-installation-of-vs-code

https://update.code.visualstudio.com/api/versions/1.81.0/win32-x64/stable
source: https://stackoverflow.com/a/69810842
So workflow would be monitor gitlab. Check above url for release data of the binary. If not yet there try again after 1 day.

https://github.com/chocolatey-community/chocolatey-packages/pull/2264

One option is to monitor github releases or https://code.visualstudio.com/updates redirects to latest version (version is not in the same format)
Download url seems to be https://update.code.visualstudio.com/1.81.0/win32-x64/stable
So download url https://update.code.visualstudio.com/{{ release_version }}/win32-x64/stable should work.
In the update check of visual studio i find the following code:
	) -> Result<Release, AnyError> {
		let update_endpoint =
			VSCODE_CLI_UPDATE_ENDPOINT.ok_or_else(UpdatesNotConfigured::no_url)?;
		let download_segment = target
			.download_segment(platform)
			.ok_or_else(|| CodeError::UnsupportedPlatform(platform.to_string()))?;
		let download_url = format!(
			"{}/api/versions/{}/{}/{}",
			update_endpoint,
			version,
			download_segment,
			quality_download_segment(quality),
		);

https://github.com/microsoft/vscode/blob/85daa35afb86202badbee79532537bf226961f25/cli/src/update_service.rs#L68 But so far i have been unable to find VSCODE_CLI_UPDATE_ENDPOINT

I investigated in this as well.
The update URL is build in src/vs/platform/update/electron-main/abstractUpdateService.ts and the placeholders in there come from product.json which is only completed on an installation (the one from the repo itself is incomplete and missing this placeholders).
So with the placeholders replaced from an installation (1.80.1), I get the following url:
https://update.code.visualstudio.com/api/update/win32/stable/74f6148eb9ea00507ec113ec51c489d6ffb4b771
which indeed does point to the newest version.
So as you see, the URL is built with the commit from the currently installed version. What we could do is also store that commit hash somewhere and use that for checking for updates in in case of an update, also update the commit hash beside the version number.

As I mentioned over here, we should GitHub releases unless there is a reason not to. This would be preferable to an undocumented API call.

Thanks @Roemer and @pauby


Reference to tool GitVersion.CommandLine in tools.cake needs to be updated

Reference to GitVersion.CommandLine 3.6.5 in tools.cake should be updated to 5.0.0

This issue was created by mistake while I was investigating an issue in the Cake.AddinDiscoverer tool. Please ignore.


prover credential attribute tagging policy

Signed-off-by: sklump<EMAIL_ADDRESS>Credential attribute tagging policy for prover as per JIRA 1258 (https://jira.hyperledger.org/projects/IS/issues/IS-1258).

... I see there is a problem still. I will continue to work on it.

I don't understand what could cause these failures. Anyone with a good guess, kindly advise? Thanks.

Apparently a manual intervention stopped this build en route to perfecting 1.9.0. The update is fine as far as I know. I will update JIRA notes accordingly as well.

At present all the tests pass but there appears to be an error on tear-down:
test result: ok. 4 passed; 0 failed; 0 ignored; 0 measured; 0 filtered out
test result: ok. 1 passed; 0 failed; 0 ignored; 0 measured; 0 filtered out
test result: ok. 12 passed; 0 failed; 0 ignored; 0 measured; 0 filtered out
test result: ok. 68 passed; 0 failed; 0 ignored; 0 measured; 0 filtered out
test result: ok. 23 passed; 0 failed; 1 ignored; 0 measured; 0 filtered out
test result: ok. 22 passed; 0 failed; 0 ignored; 0 measured; 0 filtered out
test result: ok. 4 passed; 0 failed; 0 ignored; 0 measured; 0 filtered out
test result: ok. 27 passed; 0 failed; 0 ignored; 0 measured; 0 filtered out
test result: ok. 36 passed; 0 failed; 0 ignored; 0 measured; 0 filtered out
test result: ok. 0 passed; 0 failed; 0 ignored; 0 measured; 0 filtered out
test result: ok. 355 passed; 0 failed; 5 ignored; 0 measured; 0 filtered out
test result: ok. 1 passed; 0 failed; 4 ignored; 0 measured; 0 filtered out
test result: ok. 0 passed; 0 failed; 0 ignored; 0 measured; 0 filtered out
test result: ok. 225 passed; 0 failed; 2 ignored; 0 measured; 0 filtered out
test result: ok. 13 passed; 0 failed; 1 ignored; 0 measured; 0 filtered out
test result: ok. 638 passed; 0 failed; 1 ignored; 0 measured; 0 filtered out
test result: ok. 638 passed; 0 failed; 1 ignored; 0 measured; 0 filtered out
test result: ok. 173 passed; 0 failed; 0 ignored; 0 measured; 0 filtered out
test result: ok. 173 passed; 0 failed; 0 ignored; 0 measured; 0 filtered out
test result: ok. 28 passed; 0 failed; 1 ignored; 0 measured; 0 filtered out
test result: ok. 27 passed; 0 failed; 0 ignored; 0 measured; 0 filtered out
test result: ok. 58 passed; 0 failed; 0 ignored; 0 measured; 0 filtered out
test result: ok. 28 passed; 0 failed; 1 ignored; 0 measured; 0 filtered out
test result: ok. 27 passed; 0 failed; 0 ignored; 0 measured; 0 filtered out
test result: ok. 11 passed; 0 failed; 0 ignored; 0 measured; 0 filtered out
test result: ok. 58 passed; 0 failed; 0 ignored; 0 measured; 0 filtered out
test result: ok. 11 passed; 0 failed; 0 ignored; 0 measured; 0 filtered out
test result: ok. 79 passed; 0 failed; 0 ignored; 0 measured; 0 filtered out
test result: ok. 3 passed; 0 failed; 0 ignored; 0 measured; 0 filtered out
test result: ok. 1 passed; 0 failed; 0 ignored; 0 measured; 0 filtered out
test result: ok. 12 passed; 0 failed; 0 ignored; 0 measured; 0 filtered out
test result: ok. 638 passed; 0 failed; 1 ignored; 0 measured; 0 filtered out
test result: ok. 9 passed; 0 failed; 0 ignored; 0 measured; 0 filtered out

but on cleanup,
android_on_linux Test: stop pool
[Pipeline] sh
+ docker stop 0dd5e3225c606d18b6d2243ad28657c26a3ad498a7e32d18214455247ca40b5c
0dd5e3225c606d18b6d2243ad28657c26a3ad498a7e32d18214455247ca40b5c
+ docker rm -f 0dd5e3225c606d18b6d2243ad28657c26a3ad498a7e32d18214455247ca40b5c
0dd5e3225c606d18b6d2243ad28657c26a3ad498a7e32d18214455247ca40b5c
[Pipeline] sh
+ docker ps --format {{.ID}} --filter network=pool_network
+ xargs docker rm -f
"docker rm" requires at least 1 argument.
See 'docker rm --help'.

Usage:  docker rm [OPTIONS] CONTAINER [CONTAINER...]

Remove one or more containers
+ true
[Pipeline] echo
android_on_linux Test: remove pool network pool_network
[Pipeline] sh
+ docker network rm pool_network
pool_network
[Pipeline] sh
+ docker container prune -f
Total reclaimed space: 0B
[Pipeline] sh
+ docker network prune -f
[Pipeline] cleanWs
[WS-CLEANUP] Deleting project workspace...
[WS-CLEANUP] Deferred wipeout is used...
[WS-CLEANUP] done
[Pipeline] }
Terminated
[Pipeline] // node
Terminated
Terminated
[Pipeline] }
Terminated
[Pipeline] // parallel
[Pipeline] }
[Pipeline] // stage
[Pipeline] End of Pipeline

GitHub has been notified of this commitâ€™s build result

ERROR: script returned exit code<PHONE_NUMBER>
Finished: FAILURE

Similarly for Windows (same exit code).
I'd appreciate any advice on how to make it exit cleanly - I don't think this is anything to do with my update.

It looks like the CI suite still produces a timeout on an android test (anoncreds_revocation_interaction_test_issuance_by_demand), but all tests pass that the update touches.

@jovfer this is too much to merge. I will have to close this PR, recreate the update, and re-submit.
I do hope indy can merge this update soon so we can all move on.


Some code fragments refers to NavView tag, where shoould be NavigationView

The name of the control in XAML is NavigationView but some code fragments are referring to it as NavView as a short name for it.

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

ID: 745ab6c5-cfbe-a56f-dccd-ab55123f9223
Version Independent ID: 54455a9f-a92f-e462-558d-78fb696dba8c
Content: Navigation view - UWP app developer
Content Source: windows-apps-src/design/controls-and-patterns/navigationview.md
Product: windows
GitHub Login: @serenaz
Microsoft Alias: sezhen

Fixed


Error during compilation after modifying algorithms.jl

I am working on adding a histogram equalisation function to help me get started with my GSoC Proposal. I made the changes in src/algorithms.jl and added some tests in test/algorithms.jl. I made changes to the docs in docs/src/function_reference.md.
Now when I do using Images, the compilation fails. What am I doing wrong?
This is the error :
ERROR: LoadError: LoadError: UndefVarError: T not defined
 in include(::ASCIIString) at ./boot.jl:264
 in include_from_node1(::ASCIIString) at ./loading.jl:417
 in include(::ASCIIString) at ./boot.jl:264
 in include_from_node1(::ASCIIString) at ./loading.jl:417
 in eval(::Module, ::Any) at ./boot.jl:267
 [inlined code] from ./sysimg.jl:14
 in process_options(::Base.JLOptions) at ./client.jl:239
 in _start() at ./client.jl:318
while loading /home/jarvis/.julia/v0.5/Images/src/algorithms.jl, in expression starting on line 458
while loading /home/jarvis/.julia/v0.5/Images/src/Images.jl, in expression starting on line 39
ERROR: Failed to precompile Images to /home/jarvis/.julia/lib/v0.5/Images.ji
 in error(::ASCIIString) at ./error.jl:21
 in compilecache(::ASCIIString) at ./loading.jl:496
 in compilecache(::Symbol) at ./loading.jl:485
 in require(::Symbol) at ./loading.jl:355
 in eval(::Module, ::Any) at ./boot.jl:267

the error occurs on line 458 of algorithms.jl.  is that where you made changes?  show us if so by making a gist.

I got the error. My knowledge on julia abstract datatypes was not clear. Thanks!

@bjarthur Once I've added the function, how do I call it directly without using Images.function_name everytime ?


Create application that handles dev account creation

Create a program that runs in the terminal that automatically creates an account for development purposes.
When we run it, the terminal should look like:
Welcome to SCE's dev account creation!
Enter email name:
Enter password:
Enter account priviledge:
One moment...
Done! You may now sign in with <email they entered> with password <password they entered>.

Create a new object similar to [this] (https://github.com/SCE-Development/Core-v4/blob/c86c7d54542074ec449201dc3d50c91eee9c113c/api/routes/user.js#L58)

Some packages that are used for command line interfaces using node:
https://www.npmjs.com/package/inquirer
https://www.npmjs.com/package/prompts

Use readline
 npm in stall readine --save
const readline = require('readline');

const rl = readline.createInterface({
    input: process.stdin,
    output: process.stdout
});

rl.question('Enter email name: ', (answer) => {
    console.log(`Your email is: ${answer}`);
    rl.close();
});

Basically r1.question is equivalent to input('Enter email name') in python. It takes in 2 parameters: the question and a function of what you'll do with the answer (this is where you would call a separate function to create the account)
An example of the output can be seen below.


Setting HlSearchFloat has no effect

The bug was introduced due to this commit https://github.com/kevinhwang91/nvim-hlslens/commit/3b98b86c4693b7382e3ed06e5546ba04353c9a88
Thank you.

Forgot to remove HlSearchFloat which is redundant.


Added TextCat module

TextCat now inherits 'object' for consistent style with the rest of NLTK.
Added check so that NLTK can be loaded but only fails at runtime when the regex dependency isn't met.
Trivial variable name correction.

Thanks @abitalp


Submit form doesn't work

Hey,
I'm planning to use WKZombie in an upcoming project. I'm trying to login to a website, by setting the value attributes of some input elements and submit a form. I Have experience as an iOS developer, but I'm quite new to the web development (HTML/CSS/Javascript), so maybe I'm misunderstanding something. I'm using the same process as the WKZombie Demo Application.
let action = open(loginUrl)
    >>* get(by: SearchType.id("m_Content_username2"))
    >>> setAttribute("value", value: credentials.username)
    >>* get(by: SearchType.id("password2"))
    >>> setAttribute("value", value: credentials.password)
    >>* get(by: SearchType.id("aspnetForm"))
    >>> submit(then: PostAction.wait(3.0))
    >>* get(by: SearchType.id("s_m_HeaderContent_picctrlthumbimage"))

action.start { (result) in
    print(result)
}

This gives me the following Log:
REQUEST
https://www.lectio.dk/lectio/163/login.aspx
[....]

SCRIPT
getElementByXpath("//*[@id='m_Content_username2']").setAttribute("value", "xxx"); document.documentElement.outerHTML;
[]

SCRIPT
getElementByXpath("//*[@id='password2']").setAttribute("value", "yyy"); document.documentElement.outerHTML;
[]

SCRIPT
document.getElementById('aspnetForm').submit();
[..................................]

It find all elements and sets the right attributes. When it's has submitted the form and waited for 3 seconds, it can't find the last element. When looking at the snapshots, I see that the attributes get set right, but on the last screenshot I see that the browser is still on the login page, with the password reset. I sure it's the right credentials. It seems that the form isn't submitted right. The printet result is "Element Not Found".
Here is the HTML of the form:
<form method="post" action="./login.aspx" onsubmit="javascript:return WebForm_OnSubmit();" onkeypress="javascript:return WebForm_FireDefaultButton(event, 'm_defaultformbtn')" id="aspnetForm" class="ls-master-container2" autocomplete="off">
<!— Lots of HTML and stuff —>
</form>

I've also tried:

with a local instance of WKZombie, but with same result.
to execute the JavaScript, by running execute("WebForm_OnSubmit()"). The JavaScript result was a Success, but I could seem to go on from there. How would I load the following page?
finding the Login-button element and running click, but doesn't work either. It is executing for a while, and the I get the "Cancelling Rendering"-error.

This might not be an issue with WKZombie, but any clues would be much appreciated!! :)
Great work with this project 👍

Have you figured this out @Wiingaard ?


Replace JSONWebTokens.jl with JWTs.jl

I played around a bit with JWTs.jl (and contacted Tanmay regarding the maintenance status of the package). My impression is that it should be possible to use it instead of JSONWebTokens.jl, and AFAICT it already includes the key id in the header (https://github.com/tanmaykm/JWTs.jl/blob/dece86f6a9f74258c7c5b463e8f71992cefa6ebc/src/JWTs.jl#L94 and https://github.com/tanmaykm/JWTs.jl/blob/dece86f6a9f74258c7c5b463e8f71992cefa6ebc/src/JWTs.jl#L153) which would require https://github.com/felipenoris/JSONWebTokens.jl/pull/21 for JSONWebTokens.jl.
The PR is breaking due to changes in the configuration struct but the design is only very preliminary - I wonder e.g. if some of the setup in the test should be performed automatically (possibly as an alternative).
I tried to follow the specifications of the test server (also based the type and id of the signing key on https://launchsmarthealthit.org/v/r4/fhir/.well-known/openid-configuration or rather the jwks_uri specified therein) but unfortunately tests still fail. I wonder if some additional checks in https://github.com/smart-on-fhir/smart-launcher-v2/blob/3c79b5032adc576fba2e5d4258d61d4e0f189778/backend/routes/auth/token.ts#L252 are still violated but I tried to e.g. specify the same iss and sub (due to the check in https://github.com/smart-on-fhir/smart-launcher-v2/blob/3c79b5032adc576fba2e5d4258d61d4e0f189778/backend/routes/auth/token.ts#L309-L312) without success.
Never mind, I misunderstood the JWT/JWK design initially.
Any ideas @DilumAluthge?

Closed in favour of #9.


After moving account to from Level 1 OU to level 2 OU under a different Level 1 OU CfCt deletes stacksets deployed to account

After moving accounts to from Level 1 OU to level 2 OU under a different Level 1 OU CfCt deletes (or attempt to delete) stacksets deployed to account

To reproduce create and enroll an account using Account factory in a level 1 OU as shown. Use CfCT to deploy a stack within the account, using its parent OU as a deployment target.
Move the account to a level 2 OU as shown in diagram.
Ensure all OUs and accounts are registered with CT
Execute CfCT pipeline.
Review logs - search for Router FunctionName: delete_stack_instances, Deleting Stack Instance: CustomControlTower-<stacket name>

Root OU
|
+- Source Level 1 OU (CfCT stackset deployment target)
|  |
|  +- Account position before move
|
+- Dest Level 1 OU (CfCT stackset deployment target)
   |
   +- Dest Level 2 OU
      |
      + Account position after move

Expected behavior is the stack deployed within the account is not deleted because at all times it remains under a deployment target.
Please complete the following information about the solution:

[X] Version: [e.g. v1.0.0] v2.3.0

To get the version of the solution, you can look at the description of the created CloudFormation stack. For example, "(SO0089) - customizations-for-aws-control-tower Solution. Version: v1.0.0". You can also find the version from releases

[X] Region: [e.g. us-east-1] ap-southeast-2
[X] Was the solution modified from the version published on this repository? No
[ ] If the answer to the previous question was yes, are the changes available on GitHub?
[X] Have you checked your service quotas for the sevices this solution uses? Yes
[X] Were there any errors in the CloudWatch Logs?  Search for the name of the stackset in the logs. Yes

Closing this. I noticed the docs now require OUs to be addressed directly like
dest-ou-level-1:dest-ou-level-2
Thus specifying only the level OU won't work, and inheritance doesn't seem to work https://docs.aws.amazon.com/controltower/latest/userguide/cfct-nested-ou.html


Additional trace_transaction tests

Added some more trace_transaction tests generated from
https://github.com/ConsenSys/json-rpc-conformance-test-utils/pull/18
Note: some of these tests fail. I think it's just besu puts in an extra field for contract creation which OE does not
"[creationMethod" : "create",
Changelog

[x] I thought about the changelog and included a changelog update if required.

This failure is explained by: the version of openethereum which generated these test files (3.2.5 installed via homebrew) doesn't have this patch https://github.com/openethereum/openethereum/issues/114

these tests would pass with this change #3515


How to use keyDB with C++

where is KeyDB client library？

Since KeyDB is a Redis fork almost everyone uses existing Redis libraries to interact with us.  I don't have any specific recommendations, but here is a list: https://redis.io/clients#c--


Admission controller example fixes

Motivation

Fixes #943 and #944
Solution

Added commented-out code to certificate setup script for adding DNS name to certificate.
Added check around adding /metadata/labels to only add the field if it doesn't already exist.

Codecov Report

Merging #950 (0628d38) into master (55bea0b) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #950   +/-   ##
=======================================
  Coverage   72.43%   72.43%
=======================================
  Files          64       64
  Lines        4412     4412
=======================================
  Hits         3196     3196
  Misses       1216     1216


URL is over encoded

Let's say I navigate with params: {filter: JSON.srtringify({name: 'test'})},
and final path is /#/table?filter={name:'test'}
which gets decoded in url into /#/table?filter=%7B"name"%3A"test"%7D
Then, if I press refresh in browser, your code encodeURI(decodeURI(path))
will convert it to /#/table?filter=%7B%22name%22%253A%22test%22%7D
because decodeURI returns /#/table?filter={"name"%3A"test"} in this case
(while decodeURIComponent would decode it correctly)
Introduced in https://github.com/router5/router5/pull/384, would not happened if router5 was using qs https://github.com/router5/router5/issues/340
Seems like is related to #404

@troch I thought that setting the new option urlParamsEncoding to uriComponent after upgrading 7.x to 8.x would fix this issue. But seems like it doesn't.
I'm a bit confused, because I don't see this option is being used in actual code https://github.com/router5/router5/search?q=urlParamsEncoding&type=Code


Typo in extension method

applyintAttributesForTag instead of applyingAttributesForTag.
Nice library by the way!

Hey @kevinrenskers!
Thanks for filing the issue. I've managed to fix the type on #2 and migrated the project to Swift 5.1. I  also gave you maintainer access to file issues / improvements while using the app.
There's a new version available through Cocoapods and Swift PM.
Thanks for the contribution!


Sync upstream

@arielkru Let's try to resolve the codeql error like the example it gives
Bad:
^_(__|.)+_$

Good:
^_(__|[^_])+_$

Check out the right part of the parenthesis


I keep getting this issue after running the PMA_Scan

Exception in thread Thread-52:
Traceback (most recent call last):
File "/usr/lib/python2.7/threading.py", line 801, in __bootstrap_inner
self.run()
File "/usr/lib/python2.7/threading.py", line 754, in run
self.__target(*self.__args, **self.__kwargs)
File "/root/OWASP-Nettacker/lib/http_fuzzer/engine.py", line 312, in request_without_data
if rule_evaluator(response, condition):
File "/root/OWASP-Nettacker/lib/http_fuzzer/engine.py", line 337, in rule_evaluator
return eval(condition)
File "", line 1, in
AttributeError: 'int' object has no attribute 'status_code'

can you please copy the condition in code? I believe you write that wrong, correct example: response.status_code in [200, 401, 403]

I haven't edited the code yet.

Can you tell me the input?

python nettacker.py -i <IP_ADDRESS> -m pma_scan, are you working on an online target?

yes, I used an online target.

please check, I cannot figure that's the problem, but it kind of solve it for me,

@Ali-Razmjoo, I believe this is the better option. The error occurs when the request timeouts and so checking the response whether it is a valid response will eliminate the error I think!

thanks, fixed.


Removing the SQL command text from the dependency name for Framework

Removing the SQL command text from the dependency name for Framework processing to reduce event bloat.
Changes
When using the Microsoft.Data.SqlClient NuGet package for SQL statements, which now provides the SQL Command Text, the SQL dependency telemetry are all bloated with the SQL Command text for ad-hoc queries when running under .NET Framework.
For comparison, the .NET Core SQL dependency collector does not use the command text in the dependency name, and just relies on the Data property to provide that information, so this should bring more parity between .NET Framework and .NET Core SQL dependencies, and reduce the amount of data sent when running under the .NET Framework and executing ad-hoc statements (like when using Entity Framework).
Checklist

[X] I ran Unit Tests locally.
[X] CHANGELOG.md updated with one line description of the fix, and a link to the original issue if available.

Here is an example of what the dependency telemetry currently looks like in the Performance->Dependencies blade in AppInsights. The long blurred out segments are the SQL Command Text, which should only be needed in the "Data" property and shouldn't be a part of the name. This might be handy for stored procs, but it is dreadful for long ad-hoc queries.

Is this something that can be looked at @TimothyMothra? Don't know if I'm supposed to add a reviewer or if that's all on your end :)

Hi @stebet !
Our work schedules have been interrupted and we fell behind in reviews. Thanks for the reminder.
This looks straight forward to me, I'll queue the builds now.

/AzurePipelines run

@cijothomas I'm okay with this change. I just wanted to double-check with you... do you know if anything was dependent on this behavior?

Thanks. The full sql text was supposed to be part of .Data only . Thanks for fixing this.

/AzurePipelines run


[consensus] Eliminate the field enforce_increasing_timestamps

This is true everywhere (except fuzzing). Discussed many times on how
this should be deleted... this is an easy cleanup as I ponder ways to
make the code slightly easier to navigate and test.

:hourglass: Testing commit 35ca735292bc3fa6477a3e8335e11737e3e5d86f with merge<PHONE_NUMBER>2a8370b40bc285a4686de234f67ee6...

:sunny: Test successful - checks-circle_commit_workflow
Approved by: dmitri-perelman
Pushing<PHONE_NUMBER>2a8370b40bc285a4686de234f67ee6 to master...


CustomerIO.Builder::build() must be called before obtaining CustomerIO instance, null, null

I'm getting this crash:
I/flutter ( 8449): PlatformException(track, CustomerIO.Builder::build() must be called before obtaining CustomerIO instance, null, null)
I/flutter ( 8449):
I/flutter ( 8449): #0      StandardMethodCodec.decodeEnvelope
message_codecs.dart:653
I/flutter ( 8449): #1      MethodChannel._invokeMethod
platform_channel.dart:315
I/flutter ( 8449): <asynchronous suspension>

when trying to
      CustomerIO.track(
        name: 'view-mobile-app',
        attributes: {
          'page': _Tab.home.name,
        },
      );

with an uninitialized CustomerIO.
Happens with 1.2.4 only on Android.

Thanks for sharing the details. The SDK does require to be initialized before calling any other methods. To resolve this, please ensure you initialize the SDK correctly before tracking screen or using any other methods, and you should no longer see the error. If you encounter any difficulties, please share your use case with us for further assistance.

@mrehan27 thanks. Using CustomerIO is optional in my app, so it doesn't have to be initialized. It would be great if SDK could handle this case on its own, like FirebaseAnalytics does.

Thank you for sharing your use case with us. We appreciate your valuable feedback.
At the moment, the requested feature is not on our roadmap. We currently recommend avoiding calls to the SDK if tracking is not required. However, I've created an internal tracking and added a vote on your behalf so we can discuss and prioritize it internally. We'll keep you updated if there's any progress.
If you have any more ideas or suggestions, please don't hesitate to share them with us. Thanks again for sharing.

Hi @agordeev,
I'm a PM at Customer.io and the team forwarded your response as a use case that we could support in the future.  To help understand the flow, can you share more information about your app and how you're using Customer.io?  For the optional case, when (or who) would be able to enable the integration?
You can also reply to<EMAIL_ADDRESS>if you'd like to share more detail or jump on a call to discuss your use case in more detail.  I'm closing this thread for now, but feel free to reopen if you'd like to discuss this further here.
Cheers,
LJ


Minor typo

https://github.com/beatcracker/VSCELicense/blob/master/VSCELicense.psm1#L59
its written [dateime] instead of [datetime]

Well spotted! I'll fix this soon, thanks.

Fixed in 180082a!


Store Banned Methods Missing (.NET Framework)

In a fresh project compiling for .NET Framework 4.7.2 and installing from NuGet all store banned methods are missing. Reading #422 I assume that these should be available.

My apologies this was user error.


[IRC] Channel-specific Boosts

As currently implemented, ANY show that adds BoostBot to their splits gets to use BoostBot's functionality but there's currently no "filter" for the IRC module, meaning ANY channel the bot is in receives notification on ANY boost for ANY show. There's some low-hanging fruit here to adjust this so that only the show the channel is for receives their boost messages, I'm just not sure how to best implement it.
Example:

If the BoostIRC install instructions are followed and a user runs BoostIRC from their node they can get node-specific IRC notifications.
BoostBot (the mastodon bot) was designed to catch all the shows that opt in and toot their boosts over fedi. Right now the IRC bot in Podcasting 2.0 channel is merely relaying the mastodon boosts. If Adam or Dave were to follow the README for setting up the BoostIRC they'd be able to get just their own boosts coming in....given that the node they install it on to watch isn't also getting boosts for other shows too.

I understand that, and running it themselves would also avoid the less than minimum sat splits getting lost in the ether, but the IRC module should likely still have some sort of filtering enabled, there are hosts with nodes that appear on multiple podcasts, yourself included 🙂, and it'd be nice to have those announcements go to the proper IRC channels

This seems a natural feature to leverage the podcast:chat tag
https://github.com/Podcastindex-org/podcast-namespace/discussions/502


This fixes #24.

Looking at aioredis library, the client.py keyword for set is now 'ex', not 'expire'. Tested this fix and fast-cache now works without issue.

Thanks! Could you update version and changelog?

Thanks!


Using extends kills the subject

base.email
{% block html %}
    my html
    {% block body %}{% endblock %}
    more html
{% endblock %}

myemail.email
{% extends 'emails/base.email' %}
{% block subject %} banana {% endblock %}
{% block body %}my body{% endblock %}

I run DTE against myemail. The body/HTML side of things is working great, but the subject is completely ignored. If I add a subject tag to the base.email it will use that, but won't let me override it from myemail.email.
Am I structuring this incorrectly?

My guess is that you want to include the subject block in the base template too, just leave it empty if you don't have content for it there.


Supplying Credentials Bug: Header field "authorization" must only have a single value

Please run down the following list and make sure you've tried the usual "quick fixes":
✔ Search the issues already opened: https://github.com/googleapis/nodejs-dlp/issues
✔ Search the issues on our "catch-all" repository: https://github.com/googleapis/google-cloud-node
✔ Search StackOverflow: http://stackoverflow.com/questions/tagged/google-cloud-platform+node.js
If you are still having issues, please be sure to include as much information as possible:
Environment details

OS: OSX High Serria
Node.js version: v10.11.0
npm version: 6.4.1
@google-cloud/dlp version: 1.1.0

Steps to reproduce

Using sample https://github.com/googleapis/nodejs-dlp/blob/master/samples/redact.js
Supply service account credentials by replacing the following line in the sample

const dlp = new DLP.DlpServiceClient();

with
const dlp = new DLP.DlpServiceClient({
  projectId: serviceAccountJson.project_id,
  credentials: serviceAccountJson,
});

Calls such as:
dlp.listInfoTypes({})
dlp.projectPath('my-project')
work as expected with the service account
Calls such as: dlp.deidentifyContent break with the following error:
UnhandledPromiseRejectionWarning: Error: Getting metadata from plugin failed with error: Header field "authorization" must only have a single value
    at Http2CallStream.call.on (/Users/MY_PROJECT_PATH/node_modules/@grpc/grpc-js/build/src/client.js:101:45)
    at Http2CallStream.emit (events.js:187:15)
    at Http2CallStream.endCall (/Users/MY_PROJECT_PATH/node_modules/@grpc/grpc-js/build/src/call-stream.js:63:18)
    at /Users/MY_PROJECT_PATH/node_modules/@grpc/grpc-js/build/src/call-stream.js:295:18

It looks like it's a bug in @grpc/grpc-js v0.4.1 that is our dependency. This bad version already got unpublished. Please rm -rf node_modules and re-run npm install, double checking that grep version<EMAIL_ADDRESS>shows v0.4.0 and not v0.4.1. Thanks!


Support Rollup input extension .jsx

Current Behavior
The file src/index.jsx is not recognized as a Rollup input:
https://github.com/palmerhq/tsdx/blob/943648edb06cd131f2c1751120cd1301804b480b/src/index.ts#L64-L69
Expected behavior
It should be possible to have the file src/index.jsx as the Rollup input.
Suggested solution(s)
async function jsOrTs(filename: string) {
  const extension = (await isFile(resolveApp(filename + '.ts')))
    ? '.ts'
    : (await isFile(resolveApp(filename + '.tsx')))
    ? '.tsx'
    : (await isFile(resolveApp(filename + '.jsx')))
    ? '.jsx'
    : '.js';

  return resolveApp(`${filename}${extension}`);
}

Your environment

Software
Version(s)

TSDX
0.6.1

TypeScript
3.5.1

Browser
Chromium

npm/Yarn
npm

Operating System
Ubuntu 18.04

Related to the this issue, TSDX currently does not seem to support importing *.js(x) files from *.ts(x) files because rollup-plugin-typescript2 is not configured to process *.js(x) files.
https://github.com/palmerhq/tsdx/blob/943648edb06cd131f2c1751120cd1301804b480b/src/createRollupConfig.ts#L146-L162
The fixed configuration looks like this:
typescript({
  // everything from above, plus ...
  include: ['*.(t|j)s+(|x)', '**/*.(t|j)s+(|x)'] // default is ["*.ts+(|x)", "**/*.ts+(|x)"]
})

tsdx is quite evidently a typescript-first project. not a priority for me to support .js{x} but lets see what jared says

That's true, but I was thinking to align tsdx's features with CRA and CRA with --typescript set also resolves .js{x} files.

I'm not having this issue, but I think if tsdx supports .js it should also support .jsx. That would also add better support for gradual migrations to TS.

Isn't the problem here that Typescript doesn't support .jsx as a file extension for files containing jsx? As far as I know only .tsx is supported. More details: https://github.com/Microsoft/TypeScript/issues/30503


Update script.js to fix course data broken

@6ebeng Can you provide this PR again with the relevant changes, I will fix review and merge this. But a point to note is that you replaced the __NEXT_DATA with the API call. But we need to support both of these flows, so you need to have a conditional check, something like if __NEXT_DATA is undefined then get data from the API call


WLAN for OS mode visible, however not accessable

Hello,
I read from your project and wanted also to install in our RV. Unfortunatly I am only familiar with Arduino and not with Python.
Nevertheless I tried to programm with your description on an ESP32-WROOM-32 resp. ESP-WROOM-32UE (WOMOLIN).
It seems in the REPL the programming was successfully. Also the the ESP own WLAN is visible, however I get no connection.
May be you see on the protocoll why it is not working?
ets Jul 29 2019 12:21:46
rst:0xc (SW_CPU_RESET),boot:0x13 (SPI_FAST_FLASH_BOOT)
configsip: 0, SPIWP:0xee
clk_drv:0x00,q_drv:0x00,d_drv:0x00,cs0_drv:0x00,hd_drv:0x00,wp_drv:0x00
mode:DIO, clock div:2
load:0x3fff0030,len:4728
load:0x40078000,len:14888
load:0x40080400,len:3368
entry 0x400805cc
INFO:main:release no: 2.6.5
HW:  RP2
INFO:connect:Detected micropython 1.23.0. 11014 on port: esp32
INFO:main:OS mode activated
INFO:connect:set ap to: 1
INFO:connect:AP enabled: <IP_ADDRESS>
INFO:web_os_main:run lin:False inet:False webos:False naw:False file:False
INFO:web_os_main:run uart:1 tx:4 rx:5
INFO:web_os_main:HW-Check RP2 esp32
INFO:web_os:init
INFO:web_os_main:Start nanoweb server
INFO:web_os_main:Start OS command loop
Task exception wasn't retrieved
future:  coro= <generator object 'run' at 3ffd9690>
Traceback (most recent call last):
File "asyncio/core.py", line 1, in run_until_complete
File "/lib/nanoweb.py", line 236, in run
File "asyncio/stream.py", line 1, in start_server
OSError: -203
INFO:web_os_main:lin-loop is running
Best regards
Andreas

Hi Andreas,
I'm not quite sure what you have done. The WOMOLIN installation only works on WOMOLIN-HW. You have set the HW platform:RP2, which looks as if you have not used the ready-made image for ESP32. Then the software is not executable. Please use the image and follow the instructions, pls.
Best regards
Magnus

Hello Magnus,
thank you for your fast response. Unfortunately I was busy these days. Therefore I can answer first now.
First I installed micropython on the TI-CI-LIN Controller v2.1 by using the burnig tool of uPiCraft.
Then I tried to install inetbox2mqqt by using uPiCraft by using the sequence of your installation routine "Alternative 1: OTA-Installation with mip" as the esp32 has 16MB flash.
May be I have to use "Alternative 2: With esptool - only works with the ESP32".
Has the memory size of 16MB flash anyway to be respected when using flash_womolin_inetbox2mqtt_v264_4M.bin.zip?
Best regards
Andreas


Upgrade .NET SDK to 6.0.100

fixes #41
Not sure if I should include file scoped namespace and global using in this PR?
@patriksvensson also might not be ready yet due to Microsoft.Win32.Registry being preview.

@patriksvensson WDYT
Is it good to ship or should we wait for Microsoft.Win32.Registry non preview?
Created https://github.com/dotnet/runtime/issues/62009 to get clarification on what is expected.

Merged! Thank you for your contribution. Much appreciated! 👍


Add levels property for MultiIndex

Codecov Report

Merging #973 into master will decrease coverage by 2.18%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #973      +/-   ##
==========================================
- Coverage   94.57%   92.39%   -2.19%
==========================================
  Files          34       34
  Lines        6487     6520      +33
==========================================
- Hits         6135     6024     -111
- Misses        352      496     +144

Impacted Files
Coverage Δ

databricks/koalas/missing/indexes.py
100% <ø> (ø)
:arrow_up:

databricks/koalas/indexes.py
96.72% <100%> (+0.09%)
:arrow_up:

databricks/koalas/usage_logging/__init__.py
24.54% <0%> (-72.73%)
:arrow_down:

databricks/koalas/usage_logging/usage_logger.py
50% <0%> (-50%)
:arrow_down:

databricks/koalas/__init__.py
78.04% <0%> (-7.32%)
:arrow_down:

databricks/conftest.py
91.48% <0%> (-6.39%)
:arrow_down:

databricks/koalas/frame.py
93.84% <0%> (-2.19%)
:arrow_down:

databricks/koalas/groupby.py
89.34% <0%> (-2.05%)
:arrow_down:

databricks/koalas/namespace.py
85.71% <0%> (-1.44%)
:arrow_down:

databricks/koalas/plot.py
93.33% <0%> (-0.96%)
:arrow_down:

... and 8 more

Continue to review full report at Codecov.

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

LGTM.

Thanks! merging.

I had a short offline discussion with @ueshin .. let's take this out in the today's release.
I don't know how common levels and codes are. Maybe we should consider adding it back if this is used pretty much common ..?

I had a offline discussion with @ueshin. It requires to collect all data into driver side (similar discussion at https://github.com/databricks/koalas/pull/952). Let me take this out in today's release for now.


Benchmark spark.kryo.unsafe config performance

http://spark.apache.org/docs/latest/configuration.html#compression-and-serialization
spark.kryo.unsafe, available from Spark 2.1.0 defaults to false.
Whether to use unsafe based Kryo serializer. Can be substantially faster by using Unsafe Based IO.

Closing as WontFix


Add search button option?

Is there any option to add a search button to the input or any plan to do that in nearest future?

No plans for it yet. So far you can detect the search changes by using @search-change for it.

Sure, thank you.
I already use it to track search changes. Any chance to edit the input HTML? We only need this for call to action button and in fact the redirect will be triggered after user clicks on the list-item so the button is just to attract user and can be not functional.


Update _test.py

Update _test.py to accept all square matrices as inputs. Downside is that the function no longer accepts condensed forms.
According to the original paper [1], Mantel Test can be applied to any pair of square matrices. It is not constrained to be used only for symmetric matrices with 0's on the main diagonal.
I faced this issue in my research project. I'm working with distance matrices that indicate Hausdorff distance, which is not symmetric.
References:

Mantel, N. (1967). "The detection of disease clustering and a generalized regression approach". Cancer Research. 27 (2): 209–220

URL: https://aacrjournals.org/cancerres/article/27/2_Part_1/209/476508/The-Detection-of-Disease-Clustering-and-a

There seems to be something wrong with this implementation. It is grossly overestimating the correlation coefficient in some cases as compared to your original implementation.


Debug a problem with the bot

Please ignore...

/jsfiddle

/jsfiddle

/jsfiddle

/jsfiddle

/jsfiddle

/jsfiddle

/jsfiddle

/jsfiddle

/jsfiddle

/jsfiddle


C#: Godot cannot open the project of VisualStudio2022(17.X).

Godot version
3.x 4.x
System information
windows 11
Issue description
Open the script file in Godot.
It only opens VS2022(17.x), but VS does not load the project automatically.
With 16.x version it works fine.
在Godot中打开脚本文件.
只会打开VS2022(17.x),VS却不会自动加载项目.
使用16.x版本则正常.
Steps to reproduce
VS is not opened, use Godot to open CS
Minimal reproduction project
No response

It seems that the subsequent versions are normal again


Improve downloaded papers

constructing the query using NIH tool + downloading a new set of data with restricted query
Current criteria: Clinical + Brain
The number of papers downloaded is 717,601 (85 GB)
~ 460,000 with text body (the rest of the eliminated papers have only abstract or empty body)

Using the Medical Subject Headings (MeSH) terms for reseach in humans reduces this a bit to 377757:

Clinical AND brain AND ("Humans"[Mesh])

The download opeartion using
"Clinical AND brain AND ("Humans"[Mesh]) AND 1995[Pubdate] : 2023 [Pubdate]" ended up with the result of  350500 articles.


added maintainers 2) added pullapprove

approve

Current coverage is 30.55% (diff: 100%)

Merging #14 into master will not change coverage

@@             master        #14   diff @@
==========================================
  Files            26         26
  Lines           661        661
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
  Hits            202        202
  Misses          455        455
  Partials          4          4

Powered by Codecov. Last update aed1610...d8a2693

approved


Custom configurations are not available after migration

After migration, our custom configurations are no longer available in the project properties.
The configurations are still in the project files.
After adding the line <Configurations>Debug;Release;Release_CI</Configurations> in the first PropertyGroup in the project file, we got our custom configuration back.

Can you share an example?
I think what you mean is that you have this?
  <PropertyGroup>
    <Configuration>Debug;OtherConfiguration</Configuration>
  </PropertyGroup>

Is that correct?

Exactly.
All configurations remains in the project in the PropertyGroups but are not shown after upgrade.
  <PropertyGroup Condition=" '$(Configuration)|$(Platform)' == 'Debug|AnyCPU' "> ...
Before migration:

After migration:

After inserting manually the config they reappear:
 <PropertyGroup>     <Configuration**s**>Debug;Release;Release_CI</Configuration**s**>   </PropertyGroup>


Water sensors - AYR

middle sensor in Bare 2 is acting funny. there is a lot of noise in the data. will send kip a replacement for that sensor. I think it is related to the sensor being very old. it is a TDR 310S which are the first one we ever purchase.

this farm is working fine no issues.